Add aggregate performance test

A first pass at some performance tests for Aggregate. Would like opinions and suggestions please.

aww my comments! heh

Just like the reports_controller_performance_spec, the number of transfers makes no difference to the speed of the aggregation:
20,000 => 524ms
10,000 => 573ms
1,000   => 486ms
100     => 608ms
10       => 509ms
1         => 473ms

Looks good to me @macosgrove  +1
I'm sure we'll tweak these as we go

looks good


Request - Custom Title

One of the problems I run into with Codiad is that I may have multiple Codiad editors open for different sites. The problem there is that in the tab the title always reads Codiad.
Would be nice if we could have it say Codiad - customname or something to identify.
VERY VERY minor wish list item.

If you want to can bring that in with a pullrequest.
Just this line has to be changed with a variable:
https://github.com/Codiad/Codiad/blob/master/index.php#L29


Adding MyBinder launch ability [WIP]

The idea would be to allow the user to test labelImg without installing anything by using mybinder to run the tool virtually and connect with a VNC session

I appreciate the contribution, but we don't have the time to support another platform, particularly such an obscure one. However, if there's a sharp increase in interest/demand, I'd be happy to reconsider.


Improve subscription creation error message

When I create a subscription through pubsub.topic('topicName').subscription('subscriptionName').create(options), I receive the following error message:
{"error":{"code":3,"details":"You have passed an invalid argument to the service (argument=PushConfig::push_endpoint).","metadata":{"internalRepr":{},"options":{}},"note":"Exception occurred in retry method that was not classified as transient"},"message":"Could not create subscription","severity":"ERROR"}

After some investigation, I found out that when I'm creating a pull subscription, CreateSubscriptionOptions.oidcToken should be null or undefined. The problem is the message did not specify exactly what was wrong with my request, it actually was misleading me to think it was a problem with the pushConfig, even though what was invalid was my oidcToken property being present on a pull subscription request. Are there any plans to improve these error messages?
Reproducing
The following code should reproduce the issue:
import { PubSub,  CreateSubscriptionOptions } from '@google-cloud/pubsub';

const pubsub = new PubSub();
const options: CreateSubscriptionOptions = {
  pushConfig: {
    pushEndpoint: undefined,
  },
  oidcToken: {
    serviceAccountEmail<EMAIL_ADDRESS>    audience: '32555940559.apps.googleusercontent.com',
  }
};

const topic =  pubsub.topic('test-topic-123');

await topic.create();

await topic.subscription('my-test-subscription').create(options);

Thanks for the report. I'm not completely sure what the intent was for the oicdToken, but the generally recommended way to pass in auth these days is using an environment variable (GOOGLE_APPLICATIONS_CREDENTIALS) to point at a service key json. It can also be passed in on the PubSub constructor, but I don't think I've seen it passed in on the create() call. If that isn't helping, I can ask someone who's more familiar with that part.

I'm sorry if my issue sounded confusing. As far as I could understand, the oidcToken field is the equivalent of the "Enable authentication" checkbox when creating a push subscription through the portal. So maybe one way to fix it is moving it to the pushConfig object to make it more clear, and maybe just ignore it if pushConfig?.pushEndpoint is null/undefined
Thank you for answering.

This might actually be a question for our Pub/Sub meeting, @kamalaboulhosn ... this looks like it might be a server-side error message. I think there was some discussion going on about improving the error messages in general.


Error when starting and using a paloalto's GlobalProtext VPN/proxy

Description
Zen fails to start when using a paloalto's GlobalProtext proxy (the proxy used by my work).
Logs from the macOS console app (the link to how to include the logs that Zen prints out was not helpful so let me know if there is someplace else to look at logs):
default	09:39:16.669517-0700	Zen	0x12e05ba40 - [PID=0] WebProcessCache::setApplicationIsActive: (isActive=1)
default	09:39:16.673858-0700	Zen	Gained inheritances: {(
    <RBSInheritance| environment:(none) name:com.apple.launchservices.userfacing origID:648-593-932837 0>,
    <RBSInheritance| environment:(none) name:com.apple.launchservices.userfacing origID:648-593-932837 0>
)}
default	09:39:16.690807-0700	Zen	Gained inheritances: {(
    <RBSInheritance| environment:(none) name:com.apple.frontboard.visibility origID:648-620-932839 0>
)}
default	09:39:17.654028-0700	Zen	parent WailsWindow 0x134010720 (17e06) remove NSWindow 0x134222260 (17e69) from group
default	09:39:17.654211-0700	Zen	parent WailsWindow 0x134010720 (17e06) add NSWindow 0x134222260 (17e69) to group
default	09:39:17.654601-0700	Zen	order window: 17e69 op: 1 relative: 17e06 related: 0
default	09:39:17.700637-0700	Zen	parent WailsWindow 0x134010720 (17e06) remove NSWindow 0x134222260 (17e69) from group
default	09:39:17.700861-0700	Zen	window NSWindow 17e69 finishing close
default	09:39:17.701174-0700	Zen	order window: 17e69 op: 0 relative: 0 related: 0
default	09:39:17.705393-0700	Zen	nw_path_libinfo_path_check [9FE9BF9E-727F-4850-915A-58C0FAA3193C Hostname#2af0d623:0 tcp, legacy-socket, attribution: developer]
	libinfo check path: satisfied (Path is satisfied), interface: utun4, ipv4, dns
default	09:39:17.705393-0700	Zen	nw_path_libinfo_path_check [FA2CB79E-AFE3-4197-B05C-614AB39E90D8 Hostname#782fce55:0 tcp, legacy-socket, attribution: developer]
	libinfo check path: satisfied (Path is satisfied), interface: utun4, ipv4, dns
default	09:39:17.705393-0700	Zen	nw_path_libinfo_path_check [E32CE374-59E9-4F8D-9B2E-EBC949C58FC4 Hostname#ea855251:0 tcp, legacy-socket, attribution: developer]
	libinfo check path: satisfied (Path is satisfied), interface: utun4, ipv4, dns
default	09:39:17.705450-0700	Zen	nw_path_libinfo_path_check [81BF410D-6CD6-4CEE-9508-BF4C57E450EF Hostname#80eab101:0 tcp, legacy-socket, attribution: developer]
	libinfo check path: satisfied (Path is satisfied), interface: utun4, ipv4, dns
default	09:39:17.705570-0700	Zen	nw_path_libinfo_path_check [27DE61DB-866E-4A5A-8B70-249D4CAFAC8C Hostname#326b1f12:0 tcp, legacy-socket, attribution: developer]
	libinfo check path: satisfied (Path is satisfied), interface: utun4, ipv4, dns
default	09:39:17.705550-0700	Zen	nw_path_libinfo_path_check [9B0DE8C8-338B-4439-B99A-2E93E68FD42E Hostname#cc9ddf31:0 tcp, legacy-socket, attribution: developer]
	libinfo check path: satisfied (Path is satisfied), interface: utun4, ipv4, dns
default	09:39:18.884031-0700	Zen	0x12e0500e0 - [PID=75693, throttler=0x12e01ce40] ProcessThrottler::Activity::Activity: Starting foreground activity / 'WebPageProxy::runJavaScriptInFrameInScriptWorld'
default	09:39:18.885795-0700	Zen	0x12e0500e0 - [PID=75693, throttler=0x12e01ce40] ProcessThrottler::Activity::invalidate: Ending foreground activity / 'WebPageProxy::runJavaScriptInFrameInScriptWorld'
default	09:39:20.864142-0700	Zen	0x12e05ba40 - [PID=0] WebProcessCache::setApplicationIsActive: (isActive=0)
default	09:39:21.001667-0700	Zen	Lost inheritances: {(
    <RBSInheritance| environment:(none) name:com.apple.frontboard.visibility origID:648-620-932839 0>,
    <RBSInheritance| environment:(none) name:com.apple.launchservices.userfacing origID:648-593-932837 0>,
    <RBSInheritance| environment:(none) name:com.apple.launchservices.userfacing origID:648-593-932837 0>
)}

Version
v0.3.1
Operating System
macOS 14.2
Steps to Reproduce

Start a paloalto's GlobalProtext VPN/proxy
Open the app and click start
See the "Failed to start proxy: set system proxy: no network service found" error
Stop a paloalto's GlobalProtext VPN/proxy
Open the app and click start
It works this time

Order does not matter: starting Zen before a paloalto's GlobalProtext proxy also fails.
Additional Context
No response

Hi, would you mind running a few terminal commands while GlobalContext is running and sharing their output? Here are the commands:

scutil --nwi
route -n get <IP_ADDRESS>
networksetup -listnetworkserviceorder

Also, were you using Zen before the last release? If so, was it working with the VPN?

Thanks! Looks like we'll need to do even more work to properly get the active network service. I have an idea for a fix in mind, I'll let you know once a working release gets published.

Hi @anfragment,
I'm just checking back in on this to see if there is any more information I can provide. I can also try out alpha/test builds if that helps recreate the issue or test fixes.
Also, you said that you would let me know one a working released is published. If you'd rather me not check-in like this let me know.
Thanks.

Hey @wburningham,
Thank you for reaching out again. I am keeping your issue in mind, but I just haven't gotten around to it yet. Thank you for offering your help in testing the solution—I'll drop an alpha build in this issue.
P.S. As an interim solution, I recommend starting Zen before launching the VPN. It should do the trick.


affected/package: golang.org/dl failed to download 1.19rc1@latest

What version of Go are you using (go version)?

$ go version
go1.18.3

Does this issue reproduce with the latest release?
I am trying to download 1.19rc1
 go install golang.org/dl/1.19rc1@latest
go: golang.org/dl/1.19rc1@latest: module golang.org/dl@latest found (v0.0.0-20220706180819-0ef8f243a9a9), but does not contain package golang.org/dl/1.19rc1

What operating system and processor architecture are you using (go env)?

go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/nomore/Library/Caches/go-build"
GOENV="/Users/nomore/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/nomore/go/pkg/mod"
GONOPROXY="*.dz11.com,code.aliyun.com,gitee.com,*.coldlake.cn"
GONOSUMDB="*"
GOOS="darwin"
GOPATH="/Users/nomore/go"
GOPRIVATE="*.dz11.com,code.aliyun.com,gitee.com,*.coldlake.cn"
GOPROXY="https://goproxy.cn/"
GOROOT="/usr/local/Cellar/go/1.18.3/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.18.3/libexec/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.18.3"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/dev/null"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/s8/w2xf8qxj6v712bq3_w_p7clw0000gp/T/go-build3115741191=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
 go install golang.org/dl/1.19rc1@latest
go: golang.org/dl/1.19rc1@latest: module golang.org/dl@latest found (v0.0.0-20220706180819-0ef8f243a9a9), but does not contain package golang.org/dl/1.19rc1

What did you expect to see?
download go1.19rc1 without any problem
What did you see instead?
not contain package golang.org/dl/1.19rc1

I followed by the install instruction.

you have a typo
Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For questions please refer to https://github.com/golang/go/wiki/Questions


Websocket streaming NAL Units

HI i already have a websocket setup with outputs NAL Units how i use the wfs to play the stream on browser ?
let wfs = new Wfs();
function onStreamDataReceived( nal )
{
//.....how do i pass the nal i receive to you wfs obj ??
}
Please help

https://github.com/ChihChengYang/wfs.js/blob/master/demo/index.html
var video1 = document.getElementById("video1"),
wfs = new Wfs();
wfs.attachMedia(video1,'ch1');
<===This will create mediaSource buffer and one websocket with "channel name" (ch1,here) connect to server ...
https://github.com/ChihChengYang/wfs.js/blob/master/src/controller/flow-controller.js
var client = new WebSocket( 'ws://' + window.location.host + '/' +  data.websocketName );
this.wfs.attachWebsocket(client,data.channelName);
<=== You can change your server's IP here. Once server receive client's connection that will start to send data matching "channel name". You may check out https://github.com/ChihChengYang/wfs.js/blob/master/server/wfs_server_lite.go
https://github.com/ChihChengYang/wfs.js/blob/master/src/loader/websocket-loader.js
<== You may check this if you would like to process NAL data...

First, You need to modify codes to escape from default websockt connection.
Once you receive the nal packets using this.wfs.trigger(Event.H264_DATA_PARSING, {data: copy }); to trigger h.264 parser that will demux h.264 into fmp4.

Would be able to create a js file which does not connect to websocket. where i can choose to input stream to be muxed. I tried to change you code but didnt have in any luck in getting to work.
It would be great help to me


Do we have to add Windows Service account to Resource Group for on-premise data gateway in Azure?

I am using Azure Gov cloud.
The documentation here  has stated to provide  contributor  privileges to  Windows service account   "NT SERVICE\PBIEgwService" in Azure Portal. Am I reading this right ? Is this mandatory ? It is very confusing because I was able to create a test connector (for on-premise SQL Server) and used it in my Logic app in hybrid scenario without adding NT Service account as Contributor in my Azure on-premise-data-gateway resource group.
If so, how do we add this Service Account to contributor privileges on my Azure Resource Group in Azure Gov/Commercial portal ? The example you have given is for how to use RBAC. But where is the user "NT SERVICE\PBIEgwService" in AAD in Gov or Commercial cloud? Because this user does not exist in my AAD tenant. Can I add "NT SERVICE\PBIEgwService" as a user in AAD? I don't think so as it does not work. Can you provide some more documentation and example to add this Service account to Azure Resource Group ?

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

ID: 0deaa8b2-66a2-b45c-3ecd-7ea2d08503ed
Version Independent ID: 364890ed-d3ba-9e01-c40a-ce78600319f1
Content: Install on-premises data gateway - Azure Logic Apps
Content Source: articles/logic-apps/logic-apps-gateway-install.md
Service: logic-apps
GitHub Login: @ecfan
Microsoft Alias: estfan

Hi @Anand-Moghe Thank you for your feedback! We will review and update as appropriate.

Hi @Anand-Moghe I don't believe you'd need to add the service process that's running on your on-prem to your AAD via Portal. Contributor During the installation of OPDG, as described in step 4 and 5, you provide an email address (work/school account) that's associated with your Azure Subscription (same tenant). Then, when you head over to Portal and create an On-Premises Data Gateway there, you should see the installation of your OPDG as an available option as show in my repro below:

Once you've got the OPDG running on Azure via Portal, whenever you configure Logic Apps connector that supports on-prem connection, you'll see the OPDG connection as an available option in the UI for data source.
To clarify this process better as well as any additional steps relating to MAG, as a next step, we have assigned this issue to the author to review further and take the right course of action.

@Anand-Moghe & @mike-urnun-msft, thank you, we have a content backlog work item for this now. #please-close


Compatibility with emojione plugin and the ASCII Smiley option

I wrote a plugin for emojis support with Shaarli. Everything works well but not one particular option with the Material theme.
https://github.com/NerosTie/emojione/issues/5
Material has a strange behavior with ASCII emojis like :) at the beginning of a line (that means they are not converted) if the markdown plugin is activated. And they are not converted in tag area too.
This issue do not exists with the default theme and I can't find where is the difference in your theme...
Could you help me? :pray:

I'll have a look (even though I have no idea how emojione works).

It looks like it's due to this bug: https://github.com/Ranks/emojione/issues/54
If, when looking at the HTML code, the emoji is right next to an HTML tag, it'll fail. So you can try for example, even on the default Shaarli theme, if you write this:
test :)
another line

the equivalent HTML code will be
test :)

another line

It'll fail as the emoji will be just next to the
 tag which makes the new line.
And therefore, it often fails in Shaarli Material because you target .link-tag which are  containing a , meaning that the first and last tags will be ignored (because just next to the opening or closing tag of a).
Same thing for the .link-description when enabling markdown because the markdown plugin created an extra div around the
 containing the description. It means that an emoji at the beginning of the description or at the end will be ignored. You will even have more issues if you use markdown syntax which will generate HTML tags, if those are located next to your emojis.
So to conclude, I would suggest to wait and see if they solve the issue. But as it was opened 1 year ago, it may never be fixed. Otherwise, it could be possible to fix the issue in their library by yourself.

With the default theme, this line:
test :)
another line

works if markdown is activated but it fails if markdown is deactivated and this the opposite behavior in the material theme...

So to conclude, I would suggest to wait and see if they solve the issue. But as it was opened 1 year ago, it may never be fixed. Otherwise, it could be possible to fix the issue in their library by yourself.

Thanks for the all explanation!
Well, this bug seems to be related to to ASCII and emojione, not the Material theme as I thought. Using ASCII is always risky in that case and this an option in a plugin, so I will not try to fix it by myself (maybe the emojione team will find a solution in their library).


Create notification on Product & Community Registries Check nightly run

Which area/kind this issue is related to?
/area ci
/area alizer
Issue Description
The nightly run which checks the starter projects of all stacks inside product and community registry should have an alerting mechanism in order to notify people for the status of the job. As a result a notification should be added after every run, as a message to the appropriate team channel so people can monitor easier the results of this job.
Acceptance Criteria

[ ] Every nightly run should send a message to team channel with the result of the task.
[ ] The message should follow the following format:
* Status: [{red light emoji} failure | {green light emoji} success]
* Severity: [for failure - medium | for success low]
* In case of failure mention @ app-devfile in order to take action.
* Title: Product & Community Registries Check
* Description: Mention Branch and Commit, add a short message and link to logs

Currently blocked from https://github.com/devfile/api/issues/1248

Similar to https://github.com/devfile/api/issues/1061

Blocked by https://github.com/devfile/api/issues/1026

Removing blocked label as I don't think is relevant anymore

Putting item back to "in review" cause I'm still looking for a proper way to test the related PR

PR https://github.com/devfile/alizer/pull/43 is now ready for review


User Action not being merged after sign in

Currently, if you clear all cache and app data to simulate a new user accessing SWC, do a random action like 'Follow us on X' and clicks on Sign In on the modal that will show up, there is a problem where the action that you just completed is not merged to that user after sign in.
The weird UX is that the user is seeing, after the steps above, the success screen saying the action was completed, but is isn't marked as complete on the list of user action. Prob the action was created but it was not assigned to the user after login.

TL;DR; There is no fix to this because it is not a problem, but could we change the logic to be able to merge only the user action that was completed?
After investigation, I believe that there is nothing wrong with the current flow. Here is why:
The scenario described in this issue will only happen if the users completes an action and then tries to log in with an existing user. This won't trigger the accounts merge logic and by not doing it, the user will experience the scenario described.
But, if the user tries to login, after completing the action, with a new user, the merge logic will happen and the completed action will be associated with this user record created.

I know that we don't merge user data in this situation because of security reasons, but what if we associate this specific user action with the authenticated user and remove it from the unauthenticated user if the user logged in immediately on the post action screen?
If would look something like this: Completes an action while unauthenticated -> sees the post action log in section -> log in -> we change the user action's associated user with the new authenticated user if the authenticated user already exists.
@travisbloom-cb


Unable to detect SecuROM 7.35.0007

BoS 3.1.0 was not able to detect SecuROM 7.35.0007 on Warhammer 40,000: Dawn of War: Soulstorm when scanning both the disc and the installed files on the hard drive.

@IcySon55 and @TheRogueArchivist it seems like this was taken care of?

Confirmed to be fixed.


Split http module into multiple files

Module structure remains the same for users (import paths are the same as before), but request and response have their own files to keep things easier to maintain as their complexity grows.

Just pushed , I removed the sys package since it's not needed now that we removed the Python Client/Server implementations

@saviorand Just a friendly ping that I recently finally got around to addressing your feedback

@bgreni thanks for the ping. Double-checked everything, looking good. Removed a couple print statements. Merging now.


Use shared instead of sharedInstance

Apple shanged all their API to used shared instead of sharedInstance
for all their variables that points to the signelton instance of an
class. This commit will ensure that the snippets for signelton classes
are doing the same.

@burczyk thanks for a great list of snippets. I hope you find this change useful 😃

Thanks for the contribution!


Prevent middleware from running on empty 'only'

When we set middleware with 'only' parameter we should expect the application to only load that middleware for the routes which are declared in the array we are passing it. Thus if we pass it an empty or (non instantiated) array it should not load that middleware.
However as is this is not the case. Take Laravel documentation example:
<?php

class BaseController extends Controller
{
  /**
  * If controller extending this BaseController needs to load this middleware it will declare this
  * array with names of routes/methods which require it.
  * @array Routes which should load some middleware.
  */
  public $middlewareRoutes;

  public function __construct()
  {
    // We will load this middleware ONLY for routes declared in $middlewareRoutes
    $this->middleware('some_middleware', ['only' => $this->middlewareRoutes]);

    // ^ This however does not behave as you would expect, yes ? Because if we pass it like this or
    // even if we declare $middlewareRoutes = [] the middleware will stil run when it should not.

    // If the controller does not declare $middlewareRoutes or if it's an empty array the middleware
    // should not be loaded.
  }

}

class IndexController extends BaseController
{
  // We do not want middleware to run for this method but it will regardless
  public function index()
  {
    return view('...');
  }
}

Now you could wrap the $this->middleware with an if but wouldn't that kinda defeat the purpose of 'only' ? With my change this works as expected.

This issue does not happen with except parameter. It works as expected if we pass it an empty array or non instantiated one.


Fix #510 and more (Abracadabra)

Fixing #510 on abracadabra / -ext
Also fix market revenue and daily borrow/deposit updates on normal/ -ext
Testing:

https://okgraph.xyz/?q=dmelotik%2Fabracadabra-ethereum
https://okgraph.xyz/?q=dmelotik%2Fabracadabra-ethereum-extended
https://okgraph.xyz/?q=dmelotik%2Fabracadabra-arbitrum-extended

strangely has other commits, closing and reopening with #594


TabBarIOS tintColor

Origin Pull request from cmcewen
All the work have been done by @cmcewen, I just rebased his work with the master.

Lucky number 1337!
Can you expand upon the tab bar example code to show this new functionality?

@ericvicenti do you know how launch UIExplorer e2e test directly on my desktop ?
I tried some commands
cd Examples/UIExplorer
xctool -scheme UIExplorer -sdk iphonesimulator8.1

./scripts/e2e-test.sh

./runXcodeTests.sh

But I can't reproduce the failing test
I have to retrieve the new screenshot and put it on the xamples/UIExplorer/UIExplorerTests/ReferenceImages/Examples-UIExplorer-UIExplorerApp/ folder, but I can't launch the test.
Anyone knows ?

Ah, so the snapshot tests can be finicky. Things will look different based on the device in the simulator, 32 vs 64 bit, OS version, etc. Are you re-recording the snapshot on iPhone 5 (32-bit) and iOS 8.x?
You should be able to run the tests locally with ./scripts/objc-test.sh

Are you re-recording the snapshot on iPhone 5 (32-bit) and iOS 8.x?

Yeah, I record the snapshot with iOS 8.1 and iPhone 5 but I can't specify 32-bit.
The difference between my screenshot and the oldest, its the text size, mine is more smaller than the oldest. @ericvicenti Do you know why ?

@ericvicenti it's ok ?
I changed the “recordMode” from NO to YES inside the UIExplorerTests, the “reference image” has been updated, and I can’t see my changes on the screenshot, but I see the changes when I run the UIExplorer project on Xcode, do you know why ?
What is the next steps to merge this feature ?

@nicklockwood Do you have something to add ?

Running tests on iPhone 5 automatically makes them 32-bit, iPhone 5s and above is 64-bit.
I don't know why you wouldn't be seeing the changes in the reference image. That's very strange.

@nicklockwood code updated ;-)
Can you try this branch if you see the tab bar in blue on the UIExplorer project ?

@a2 @nicklockwood @ericvicenti merging is possible ?

This would be nice to get merged! :)

Looks good to me

@brentvatne what is the next step ? This could be merged friday during the time afforded to those things ? -> https://gist.github.com/brentvatne/e1ca1e4beaf0135e3178

This was actually merged yesterday. Should be included in the next update. Sorry for the wait!

:+1: thanks @nicklockwood!

@tsunammis - looks like that won't be needed :smile: hope you join in on Friday!

@brentvatne thank you !! I hope I will be available friday, what is the timezone ? I'm in Paris.
@nicklockwood thank you, you are the best !!

@tsunammis - I'll be around at 10am - 3pm PT :smile:


Could you kindly provide a list of the environment configurations?

Such as an environment.yaml file for Conda, if possible.
It seems that there are some issues with my environment, preventing me from starting the training properly.
my environment（2080ti x 8）：
python                    3.8.16
cudatoolkit               11.8.0
cudnn                     <IP_ADDRESS>
pytorch                   1.12.1
pytorch-gpu               1.12.1
torchlight                0.0.1
torchlights               0.4.0
torchvision               0.14.1

I can train normally on my other codes and testing process runs fine, but there seem to be some bugs on training:
RuntimeError: Caught RuntimeError in replica 1 on device 1.
Original Traceback (most recent call last):
  File "/home/leo/.conda/envs/leo_Denoiser/lib/python3.8/site-packages/torch/nn/parallel/parallel_apply.py", line 61, in _worker
    output = module(*input, **kwargs)
  File "/home/leo/.conda/envs/leo_Denoiser/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/leo/Code/Denoiser/HSIR/hsir/model/hsdt/arch.py", line 111, in forward
    out = self.encoder(out, xs)
  File "/home/leo/.conda/envs/leo_Denoiser/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/leo/Code/Denoiser/HSIR/hsir/model/hsdt/arch.py", line 56, in forward
    x = self.layers[i](x)
  File "/home/leo/.conda/envs/leo_Denoiser/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/leo/.conda/envs/leo_Denoiser/lib/python3.8/site-packages/torch/nn/modules/container.py", line 139, in forward
    input = module(input)
  File "/home/leo/.conda/envs/leo_Denoiser/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/leo/Code/Denoiser/HSIR/hsir/model/hsdt/attention.py", line 244, in forward
    r, _ = self.attn(inputs)
  File "/home/leo/.conda/envs/leo_Denoiser/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/leo/Code/Denoiser/HSIR/hsir/model/hsdt/attention.py", line 85, in forward
    q = torch.matmul(attn, v)
RuntimeError: CUDA error: CUBLAS_STATUS_EXECUTION_FAILED when calling `cublasSgemmStridedBatched( handle, opa, opb, m, n, k, &alpha, a, lda, stridea, b, ldb, strideb, &beta, c, ldc, stridec, num_batches)`

or
packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/leo/Code/Denoiser/HSIR/hsir/model/hsdt/attention.py", line 246, in forward
    r = self.ffn(r)
  File "/home/leo/.conda/envs/leo_Denoiser/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/leo/Code/Denoiser/HSIR/hsir/model/hsdt/attention.py", line 183, in forward
    x2 = x * torch.sigmoid(w)
RuntimeError: CUDA error: misaligned address
CUDA kernel errors might be asynchronously reported at some other API call,so the stacktrace below might be incorrect.

or
RuntimeError: Caught RuntimeError in replica 1 on device 1.
Original Traceback (most recent call last):
  File "/home/leo/.conda/envs/leo_Denoiser/lib/python3.8/site-packages/torch/nn/parallel/parallel_apply.py", line 61, in _worker
    output = module(*input, **kwargs)
  File "/home/leo/.conda/envs/leo_Denoiser/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/leo/Code/Denoiser/HSIR/hsir/model/hsdt/arch.py", line 111, in forward
    out = self.encoder(out, xs)
  File "/home/leo/.conda/envs/leo_Denoiser/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/leo/Code/Denoiser/HSIR/hsir/model/hsdt/arch.py", line 56, in forward
    x = self.layers[i](x)
  File "/home/leo/.conda/envs/leo_Denoiser/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/leo/.conda/envs/leo_Denoiser/lib/python3.8/site-packages/torch/nn/modules/container.py", line 139, in forward
    input = module(input)
  File "/home/leo/.conda/envs/leo_Denoiser/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/leo/Code/Denoiser/HSIR/hsir/model/hsdt/attention.py", line 246, in forward
    r = self.ffn(r)
  File "/home/leo/.conda/envs/leo_Denoiser/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/leo/Code/Denoiser/HSIR/hsir/model/hsdt/attention.py", line 178, in forward
    x = F.gelu(x)
RuntimeError: CUDA error: misaligned address
CUDA kernel errors might be asynchronously reported at some other API call,so the stacktrace below might be incorrect.

or
  File "/home/leo/.conda/envs/leo_Denoiser/lib/python3.8/site-packages/torch/nn/functional.py", line 2438, in batch_norm
    return torch.batch_norm(
RuntimeError: cuDNN error: CUDNN_STATUS_EXECUTION_FAILED

This should be unrelated to memory and batch size since I still encounter the issue even with the smallest model.
Could you kindly update your environment configuration? It also might be related to the versions of torch and cudnn.
Or may stem from the use of this：sync_batchnorm

Well, I found that if I adjust batch_size to 4, it can run, but the cuda memory only takes up 1/6, and if I adjust it to 8 or bigger, an error will be reported.


[FEAT]: POC design for LEGO bottle conveyor

Description
Design a POC (proof of concept) in LDraw cad format for the "Bottle Translation Conveyor"
Additional Information

try to use LEGO parts from 42100
if needed more different LEGO parts check for parts owned by members of the B-AROL-O organization
last source for new parts can be the Bricklink marketplace

Note: using Bricklink Stud.io CAD each selected part shows its medium price on the marketplace.
No response

@alv67 based on our conversation I would assign the task to you, with planned resolution by the end of CW03.

This is the idea for the bottle conveyor I'm working on.

This is the idea for the bottle conveyor I'm working on.

Awesome! Looking forward to your PR with the LDR model

Reopening for @alv67 to complete his design


Crud plugin?

Sorry for the lazy question but would this work when combined with the Crud plugin?

Closing as it looks like this could indeed work. Tested while attempting a Crud JsonApiListener.

Thanks for checken, didn't know for sure but was expecting it because of the way its implemented :-)


[Galley] Combine Validation Webhook mode with Server Mode

Currently, galley executable can start in either mode: An MCP server, or a validation web-hook. To reduce the cognitive load on operators, we should simplify it and reduce it to one:

When Galley starts, it starts both the webhook module, and the server module.
Either can be optionally disabled through command-line.

One important thing worth noting here is, both have different liveness/readiness probe modes, so those needs to be aligned as well.
The steps to complete this task are:

[ ] Combine the liveness/readiness probes of both webhook and the server mode. We should standardize on istio.io/istio/pkg/probe, which server is already using.
[ ] Make it so that both modes start when just typing "galley" (i.e. not "galley validator"  or "galley server", just "galley"). It should be possible to disable either mode.
[ ] Update helm charts, so that they use the new form of the command-line.
[ ] Remove other command-line modes.

cc @irisdingbj

@ozevren ^^


Would it be possible to peg Ruby and Node to the specific version?

I'm using your image with GitLab CI, and would like to peg Ruby and Node to the current version.
I'm looking at the tags page on hub.docker.com, and it looks like you only release a few very generic versions. (e.g. 2-8 for Ruby 2.x.x and Node 8.x.x).
Would it be possible to release a more specific tag for the current version? E.g. 2.4.2-8.6.0?
I'd like to use something like image: "starefossen/ruby-node:2.4.2-8.6.0" in my .gitlab-ci.yml, so that I have a consistent environment.

Never mind! I figured out how to push my own docker image to hub.docker.com

👍 Custom image is way. More image combinations cause more time to get them update. It takes much time. I think mayor versions are enough.

This is actually what I need too, a guaranty that I'm always running the tests with the versions similar to the production environment. I really think this should have stayed open.

That could be fine but we don't have so time to maintain 24 combinations of Ruby 2 and Node 8 and 44 combinations of Ruby 2 and Node 6 for one Linux repository. There are 3 distributions (Full, Slim and Alpine) so in final 204 images :)
Mayor version is enough. When is minor version of Ruby or Node published we always update.
Minor versions are almost compatible with other lower minor versions.


Always skipped when remembering the last open buffers

Since 26d95031e2c624c5317dbdce386eb89f9356d4c0 Alpha is never shown with the 'shada (aka 'viminfo) += % which recovers the buffer list.

can you elaborate? I don't think I understand what 'shada is

Pardon my terseness.
'shada is an option, before ShaDA implementation in NeoVIM it was known as 'viminfo (actually ShaDA supersedes the 'viminfo but no matter).
It allows you to save various info about the NeoVIM sessions. One of the options (%) allows to remember the open buffer list.

oh I see, I had to close nvim with a listed buffer first. ok, I'll see if I can fix this

should be fixed in https://github.com/goolord/alpha-nvim/commit/234822140b265ec4ba3203e3e0be0e0bb826dff5 @naquad can you test?

Works for me! Thank you for the fix :)


Converting booleans to strings makes unparsable values

I have parsed some Json and I have a set of JValues that I'm looping through and eventually setting some properties in another document.
Unfortunately, when I take the JValue and I call .ToString() on it, it converts my JValue with a bool type and a value of true, to the string "True".  This is the same behavior as saying
bool b = true;
string s = b.ToString();
Unfortunately, I send this updated document elsewhere, and it's deserialized but it will fail because it outputs "True" instead of "true".  This seems like unexpected behavior and I can't seem to find any way to get a Boolean JValue to produce the actual JSON string that could be parsed later.
NB: I'm actually setting attributes on an XML document, but the same behavior would be occurring if I was constructing a JSON document the way I am.  I don't necessarily know the type of the value, I just take to the .ToString() and set it as the value of the attribute and it would be nice if I didn't have to special case Booleans.

Actually, let me amend that quickly:
In order to get a Boolean to produce true as the output, then I can use JsonConvert.SerializeObject(...), however this also produces quoted strings when serializing string objects which won't work in some cases.

That's by design. Strings in JSON aren't valid without quotes.


istioctl: allow setting remote istiod port

When running in KinD, istiod is exposed via NodePort rather than a LoadBalancer. For remote clusters to connect to this for configuration, they need to be able to change or map the port envoys use to get configuration from Pilot.
When values.global.remotePilotAddress is set, we create a Service/Endpoint istiod-remote with port 15012. This change adds to that with values.global.remoteIstiodPort. When that flag is set:

give the istiod-remote service a clusterIP so that it can perform port mapping
targetPort is set on the istiod-remote Service
istiod-remote endpoint uses the correct port

Example output:

With the new flag
landow@Stevens-MacBook-Pro:istioctl (master) $ ./istioctl manifest generate --set profile=remote --set values.global.remotePilotAddress=<IP_ADDRESS> --set values.global.remoteIstiodPort=32345 | grep "name: istiod-remote" --context=7
    namespace: istio-system
---

apiVersion: v1
kind: Endpoints
metadata:
  name: istiod-remote
  namespace: istio-system
subsets:
- addresses:
  - ip: <IP_ADDRESS>
  ports:
  - port: 32345
    name: tcp-istiod
--
--
    release: istio
---

apiVersion: v1
kind: Service
metadata:
  name: istiod-remote
  namespace: istio-system
spec:
  ports:
  - port: 15012
    name: tcp-istiod
    targetPort: 32345

Without the new setting
landow@Stevens-MacBook-Pro:istioctl (remote-istiod-port) $ ./istioctl manifest generate --set profile=remote --set values.global.remotePilotAddress=<IP_ADDRESS> | grep "name: istiod-remote" --context=7
    namespace: istio-system
---

apiVersion: v1
kind: Endpoints
metadata:
  name: istiod-remote
  namespace: istio-system
subsets:
- addresses:
  - ip: <IP_ADDRESS>
  ports:
  - port: 15012
    name: tcp-istiod
--
--
    release: istio
---

apiVersion: v1
kind: Service
metadata:
  name: istiod-remote
  namespace: istio-system
spec:
  ports:
  - port: 15012
    name: tcp-istiod
  clusterIP: None
---

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
b2ff25056d84b657797d51256626fdc84fa0c0af
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
b2ff25056d84b657797d51256626fdc84fa0c0af
link
/test gencheck_istio

unit-tests_istio
b2ff25056d84b657797d51256626fdc84fa0c0af
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl I strongly recommend against nodeport for any Istio use case. If you are running kind, I'd recommend using metallb. If our documentation is broken in some way, can you point it out, so we can remove any recommendations for nodeport.
There are many blogs on this topic - the first one that comes up in my google search is: https://mauilion.dev/posts/kind-metallb/

@stevenctl I strongly recommend against nodeport for any Istio use case. If you are running kind, I'd recommend using metallb. If our documentation is broken in some way, can you point it out, so we can remove any recommendations for nodeport.
There are many blogs on this topic - the first one that comes up in my google search is: https://mauilion.dev/posts/kind-metallb/

I've run through the MetalLB and it works fine. Right now in Prow, the kind clusters run with NodePort exposing tcp-istiod. As part of setting up multicluster (shared control plane) integration tests, we need to enable remote clusters to access the istiod/Pilot running in another cluster to get configs. Would you recommend setting up MetalLB as part of our KinD setup rather than providing this configuration override?

If no one is using nodeports for multicluster in prod and it is feasible to setup metallb in  our CI that option sounds good to me

Some prior work in this area: https://github.com/istio/operator/pull/379
Part of this work was removed at my suggestion because, at that time, the work occurred last minute (sort of like now ;) and I didn't have time to obtain an IP address range in the prow infrastructure.
See this removal PR for more context: https://github.com/istio/operator/pull/456
Cheers
-steve

Closing in favor of https://github.com/istio/istio/pull/23363


upgrade 0.12

superseded by https://github.com/cloudposse/terraform-aws-iam-chamber-s3-role/pull/7


Update response of charge state

I was also wondering if it's worth to describe each field. What do you think?

That would get rather tedious. Plus, most are fairly self-explanatory, so that would be a wasted effort.
Thanks for the update!


The PublisherIdentifier for "StartSubscription" test should use tenant Id

The PublisherIdentifier for "StartSubscription" test should use tenant Id instead of App Id. the "[Child] Admin | Sync Logs" flow is using Tenant Id for the "PublisherIdentifier" at "List audit log content" and "GetContentDetails" actions.

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

ID: 1855a9ee-8713-c8e6-2091-bc972f877c13
Version Independent ID: c87a0108-4de6-49df-8ba3-3bdb06216ad8
Content: Set up audit log components - Power Platform
Content Source: power-platform/guidance/coe/setup-auditlog.md
Service: power-platform
Sub-service: guidance
GitHub Login: @manuelap-msft
Microsoft Alias: mapichle

Will be updated in the December release, made the change locally for the next update. Thank you!


Update DistributionsAD and Distributions compatibilities

The latest release of DistributionsAD fixes the issues with Distributions >= 0.23.3.

Seems Bijectors has to be fixed as well.

See https://github.com/TuringLang/Bijectors.jl/pull/111.

It needs a new release of DynamicPPL that is compatible with Bijectors 0.8 first.


Export context folder from deployed odo component created with previous release

[kind/Enhancement]
Which functionality do you think we should update/improve?
There should be command to create context folder based on deployed component.
Why is this needed?
For binary and git components created with previous odo releases there would be no context folder. That means there would be no straightforward way to work with those components.

You should be able to just do odo component create java component-name --project project --app application in the directory with the source code and use component-name project and application of the already existing component. If I recall correctly odo is not validating names during the component creation so this should work.

We should not support backwards compatibility for this, as the context folder was introduced before beta. As @kadel mentions, if a component was created by a previous version of odo, you could still create the context folder that way.

@jorgemoralespou I am not asking backward compatibility, I am asking for the way out for developers, if any, who is using git and binary components. If we would just release odo-1.0.0 with vscode now all those component would not be usable without context folder. What @kadel suggested is basically 'do it again' for all git/binary components deployed. Would be much easier with 'odo import' to reconstruct context folder using deployed component.

@kadel

You should be able to just do odo component create java component-name --project project --app application in the directory with the source code and use component-name project and application of the already existing component. If I recall correctly odo is not validating names during the component creation so this should work.
What would happen for components with URLs, storage and links if after I use just odo create and then push it?

@dgolovin is this issue still relevant?


EasyAccess issues fixed

EasyAccess following issues are fixed :
#2, #3, #4, #7, #8, #10, #11, #12, #13, #14, #15

@justdroid - Can you please merge this pull request.

Merged.Releasing today in few hours.


Update THIRD_PARTY_LICENSES.txt

Description
With update of THIRD_PARTY_LICENSES.txt we need warning in each created PR, that updates setup.py. Warning message will request update THIRD_PARTY_LICENSES.txt, if any library was added or removed.
Jira: https://jira.oci.oraclecorp.com/browse/ODSC-40102

updated THIRD_PARTY_LICENSES.txt with list of libraries from setup.py. Made them alphabetically. Added license texts alphabetically
added Actions to post comment in open PR

Validation

checked when setup.py updated, Action with warning comment triggered
checked when closed and re-opnened PR, warning comment posted
checked formattin gof the warning message (last one in this PR is suggested one):

Hi.


added leys school

added The Leys school

@combinedactors Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

@combinedactors Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

Thank you, here are the details you requested:

https://www.theleys.net
The Leys School, Cambridge, CB2 7AD, United Kingdom
https://www.theleys.net/academic/subjects/computing/
https://www.theleys.net/contact-us/


eo-math 0.4.0

PR-Codex overview
This PR updates the version of the eo-math package and adds the tests package to all relevant files.
Detailed summary

Updated the version of the eo-math package to 0.4.0
Added the tests package to all relevant files
No other notable changes were made

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@yegor256 Please, have a look

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 13min)


Add pagination to UI.

Add pagination to UI.

Coverage remained the same at 16.849% when pulling 55243f274c531d863ca3ff9a50dce102e17542d7 on wknet123:dev into 08722e2166793232098d1e082ba4b83a14acb2f0 on vmware:dev.


Feature/hyperparameters search

2021-09-21 23:57:17,939 - embeddings.utils.flair_corpus_persister - INFO - Exception occured when persisting data: test dataset is None - This shouldn't be logged as exception as in HPS is expected behaviour

TODO:

[ ] Write tests
[ ] Add examples and readme
[ ] Add searchable document pooling type
[ ] Code cleanup


Inconsistent service availability

We use cloudmap controller in multiple clusters and have the same problem on them. Clusters are linked by vpc peering. Half of the time, an imported service cannot be reached in a pod.

Service name is correctly resolved in an internal kube adress.
If I use the endpointslice adress of the remote service, it works on every pod.
I created a DaemonSet for testing purposes to see if it can be a network/node problem. A pod that have worked on a node can fail after a restart and vice versa.
We used mcs-controller in v0.2.3 and updated in v0.3.0 and have the same problems.
Kubernetes 1.21 on all clusters.

I have a feeling that there is some trouble between the service and endpointslice resolution but it don't know how can I debug it.

Hello @ThomasBoutelier - Which version of coredns are you using?

Hello,
It's the version v1.8.4.
Here's the output of the logs:
{"level":"info","ts":1664418556.359755,"logger":"controllers.ServiceExport","msg":"no changes to export to Cloud Map","namespace":"bidder-es-inte","name":"matcher-es-http"}
{"level":"info","ts":1664423785.0764976,"logger":"controllers.ServiceExport","msg":"updating Cloud Map service","namespace":"bidder-inte","name":"redis-master"}
{"level":"info","ts":1664423785.0765486,"logger":"cloudmap","msg":"fetching a service","namespace":"bidder-inte","name":"redis-master"}
{"level":"info","ts":1664423785.0766342,"logger":"controllers.ServiceExport","msg":"no changes to export to Cloud Map","namespace":"bidder-inte","name":"redis-master"}
{"level":"info","ts":1664423785.076691,"logger":"controllers.ServiceExport","msg":"updating Cloud Map service","namespace":"bidder-es-inte","name":"matcher-es-http"}
{"level":"info","ts":1664423785.0766985,"logger":"cloudmap","msg":"fetching a service","namespace":"bidder-es-inte","name":"matcher-es-http"}
{"level":"info","ts":1664423785.076791,"logger":"controllers.ServiceExport","msg":"no changes to export to Cloud Map","namespace":"bidder-es-inte","name":"matcher-es-http"}
{"level":"info","ts":1664423785.0768418,"logger":"controllers.ServiceExport","msg":"updating Cloud Map service","namespace":"bidder-inte","name":"iasp"}
{"level":"info","ts":1664423785.0768507,"logger":"cloudmap","msg":"fetching a service","namespace":"bidder-inte","name":"iasp"}
{"level":"info","ts":1664423785.0769007,"logger":"controllers.ServiceExport","msg":"no changes to export to Cloud Map","namespace":"bidder-inte","name":"iasp"}

Hey - I can help you troubleshoot. Can you join our slack channel? https://github.com/aws/aws-cloud-map-mcs-controller-for-k8s#slack-community

Should be fixed by the https://github.com/aws/aws-cloud-map-mcs-controller-for-k8s/issues/254.


Make pre-hard-fork configs compatible with berkeley

These changes are needed to allow the node loading the genesis ledger for the hard fotk exported from mainnet. The export feature is being added in #13787 .
Explain your changes:

Simplify JSON encoding of Global_slot_span and Global_slot_since_genesis by removing the tag automatically added by deriving_yojson preprocessor. These tags are unnecessary, because the type of the number is usually apparent from the context / schema. Furthermore, they make pre-hard-fork config files incompatible with current daemons. This compatibility is required in order to transfer the state of the pre-hard-fork ledger into the post-hard-fork genesis ledger.

Add fallback parsing for token_ids in the runtime config. On mainnet token_id is simply a number, while on berkeley it's a base58-encoded binary identifier. For reasons explained above, we need the berkeley node to be able to parse pre-hard-fork config, so adding the fallback parsing to preserve that compatibility. This change can be reverted after the hard fork.

Explain how you tested your changes:

Started a sandbox node initialised with a config file containing a ledger exported from a mainnet node. Demonstrated that the node is capable of producing blocks. The code used to perform the export will be submitted shortly in another PR.

Cofig file used for testing (beware, it's quite large).
Checklist:

[x] Dependency versions are unchanged

Notify Velocity team if dependencies must change in CI

[ ] Modified the current draft of release notes with details on what is completed or incomplete within this project
[x] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[x] Serialized types are in stable-versioned modules
[x] Does this close issues? List them

Closes #23

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!approved-for-mainnet

!ci-build-me

!ci-build-me

!approved-for-mainnet


Add rictic as an owner in npm

Needed to be able to publish new versions.
@nevir help!

Also need to merge https://github.com/PolymerLabs/cleankill/pull/3 before we publish anything on npm.

@rictic done!


Strange border on the first and last  children of  when using the Chrome browser

What is the expected behavior?
Identical look and feel across all major/known/usual browsers.
What is the current behavior?
Currently when using the Chrome browser I get strange looking <mat-expansion-panel> borders, more specifically on a <mat-accordion>'s first and last <mat-expansion-panel> elements during their collapsed state.
What are the steps to reproduce?
The issue can be observed at https://material.angular.io/components/expansion/examples on the collapsed <mat-expansion-panel> elements when using the latest version of Chrome (71.0.3578.98 (Official Build) (64-bit)).
https://www.screencast.com/t/HWwmpIfA6ksg
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
The issue seems to be related to "@angular/material" 7.2.0 as it can be observed in the official documentation pages. I've only been able to detect this issue using Chrome (71.0.3578.98 (Official Build) (64-bit) on my PC. On the mobile Chrome browser the issue is not manifesting.
The issue is NOT manifesting on IE 11.472.17134.0 (update versions 11.0.101), Microsoft Edge 42.17134.1.0 and Firefox 64.0 (64-bit).
Is there anything else we should know?
It seems like the box-shadow of the first and last <mat-expansion-panel> elements is somehow shifted towards the bottom-right corner as it is too thick on the bottom and right side of those <mat-expansion-panel> elements and it is not present almost at all on the left side.

I experience a related issue of missing shadows on expansion panel elements in collapsed state except for first and last.
StackBlitz: https://stackblitz.com/edit/angular-material-expansion-panel-shadows

Chrome 71.0.3578.98 (Official Build) (64-bit)

In fact, the shadow is on the left on all but first and last items.

Did anyone find a related issue in the bug tracker of Chrome (https://bugs.chromium.org/p/chromium/issues/list) or perhaps file a new one? This case is easily reproducible and could help Chrome devs to fix the bug.

Did anyone find a related issue in the bug tracker of Chrome (https://bugs.chromium.org/p/chromium/issues/list) or perhaps file a new one? This case is easily reproducible and could help Chrome devs to fix the bug.

I didn't file a new bug as I don't know yet where the issue originates from, Chrome or Angular Material.

Here is the bug: https://bugs.chromium.org/p/chromium/issues/detail?id=878195.
Should be fixed in Chrome 72.

For reference, here is a screenshot taken programmatically in Chrome 66:

Script (index.js):
const puppeteer = require('puppeteer');

function timeout(ms) {
    return new Promise(resolve => setTimeout(resolve, ms));
};

(async() => {
    const browser = await puppeteer.launch();
    const page = await browser.newPage();
    await page.setViewport({width: 640, height: 480})
    await page.goto('https://angular-material-expansion-panel-shadows.stackblit$
    await timeout(20000)
    await page.screenshot({path: 'example.png'});
    browser.close();
})();

Requirements:

NodeJS
npm package puppeteer for the required version of Chrome (https://github.com/GoogleChrome/puppeteer/blob/master/docs/api.md)

Chrome 72 has landed and the issue seems fixed. Can anyone else confirm so I can close the issue?

it works for me (Chrome 72.0.3626.96)!


feat: add cloudtrail insight selector type specification

Hello
I added variable cloudtrail_insight_selector_type where you can set insight_selector type to ApiCallRateInsight which is so far the only value that is accepted as is documented in official terraform documentation.
Default value is set to null.

Thank you, LGTM!


react-test-renderer findAllByProps returns duplicate results for RN components

Is this a bug report?
Yes

Have you read the Contributing Guidelines?
Yes
Environment
Environment:
OS:  macOS Sierra 10.12.6
Node:  8.4.0
Yarn:  1.0.2
npm:  5.3.0
Watchman:  4.9.0
Xcode:  Xcode 8.3.3 Build version 8E3004b
Android Studio:  2.3 AI-162.4069837
Packages: (wanted => installed)
react: ^16.0.0 => 16.0.0
react-native: ^0.49.3 => 0.49.3
react-test-renderer ^16.0.0 => 16.0.0
Target Platform: Node (testing using Jest)

Steps to Reproduce

(Reproducible repo below)

Use react-test-renderer inside of a jest test to render a single View with a testID
Use the renderer's findAllByProps method to search for that testID

Expected Behavior

findAllByProps returns 1 instance, the View that was rendered
Actual Behavior

findAllByProps returns 2 instances, the View that was rendered and a child of that View with all of the same props.
Reproducible Demo

https://github.com/TAGraves/react-test-renderer-bug-example

Notes
I'm trying to use some of the new utilities provided by react-test-renderer to make assertions against a RN component hierarchy, but unfortunately the way RN mocks components when using Jest makes this a futile attempt. findAllByProps is returning twice as many elements as it should. I've attached a super simple example at the top (rendering just a single <View /> but getting two results from findAllByProps).
The issue stems from mockComponent.js. When it mocks a component out, its render returns another component with the same name and props. So if your intended tree looks like:
<View testID="a">
    <SomeComponent />
</View>

the outputted tree will be:
<View testID="a">
    <View testID="a">
      <SomeComponent />
    </View>
</View>

I believe this can be fixed by changing the mocked implementation just to:
const Component = class extends RealComponent {
  render() {
    return this.props.children || null;
  }
};

and I am happy to submit a PR with this change, but would like some guidance on if there's a reason for the existing approach before I go changing things.
Thank you!

This issue has not been fixed,, and I would love for it to receive some attention. As mentioned, I am happy to submit a fix as long as someone on the core team can confirm that this isn't an intentional design decision.

@TAGraves I've just encountered this also, thank you for the succinct explanation (I was scratching my head wondering why on earth the parent component had the same props as child). This issue would definitely be worth revisiting.

+1 to this as well: very frustrating that you can't test if the correct number of Components are being rendered, since they appear twice as often as they should

+1 I just encountered this issue as well

Facing the same issue with latest version, quite frustrating as well.
Did anybody find a workaround?


libplacebo: 4.208.0 -> 5.264.1

Diff: https://code.videolan.org/videolan/libplacebo/-/compare/v4.208.0...v5.264.1
Changelog: https://code.videolan.org/videolan/libplacebo/-/tags/v5.264.1
Description of changes

Things done

Built on platform(s)

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

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

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

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

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

[ ] Fits CONTRIBUTING.md.

I need to package libdovi but it seems to be using cargo-c which I don't know how to use properly as it isn't common in nixpkgs, I asked for help in the Rust Nix Matrix room, in the meantime I drafted this PR.

I already got those two sorted.

@PedroHLC I can't get it to detect glad2 and jinja2 properly, can you help out?

Their meson.build is overwriting our PYTHONPATH:
{
  postPatch = ''
    substituteInPlace meson.build \
      --replace 'python_env.append' '#'
  '';
}

PS: vulkan-headers is a nativeBuildInputs, and in src the preferred way is hash = "sha256-..." instead of sha256 = "sha256-...".

Result of nixpkgs-review pr 224795 run on x86_64-linux 1

46 packages failed to build:

arcanPackages.all-wrapped
arcanPackages.arcan
arcanPackages.arcan-wrapped
arcanPackages.cat9-wrapped
arcanPackages.durden-wrapped
arcanPackages.pipeworld-wrapped
arcanPackages.prio-wrapped
arcanPackages.xarcan
deepin.deepin-music
deepin.deepin-voice-note
eaglemode
kaffeine
kphotoalbum
libsForQt5.elisa
libsForQt5.elisa.dev
libsForQt5.phonon-backend-vlc
libvlc
megaglest
minitube
obs-studio
obs-studio-plugins.advanced-scene-switcher
obs-studio-plugins.droidcam-obs
obs-studio-plugins.input-overlay
obs-studio-plugins.looking-glass-obs
obs-studio-plugins.obs-gstreamer
obs-studio-plugins.obs-hyperion
obs-studio-plugins.obs-livesplit-one
obs-studio-plugins.obs-move-transition
obs-studio-plugins.obs-multi-rtmp
obs-studio-plugins.obs-ndi
obs-studio-plugins.obs-nvfbc
obs-studio-plugins.obs-pipewire-audio-capture
obs-studio-plugins.obs-source-record
obs-studio-plugins.obs-vaapi
obs-studio-plugins.obs-vkcapture
obs-studio-plugins.wlrobs
pympress
pympress.dist
python310Packages.python-vlc
python310Packages.python-vlc.dist
python311Packages.python-vlc
python311Packages.python-vlc.dist
reaper
strawberry
vlc
wtwitch

56 packages built:

adl
ani-cli
anime-downloader
anime-downloader.dist
anki
anki.doc
anki.man
ankisyncd
celluloid
cplay-ng
cplay-ng.dist
curseradio
curseradio.dist
deepin.deepin-movie-reborn
deepin.deepin-movie-reborn.dev
dmlive
ff2mpv
gonic
haruna
hydrus
hydrus.doc
hypnotix
jellyfin-media-player
jellyfin-mpv-shim
jellyfin-mpv-shim.dist
jftui
libdovi
libplacebo
libsForQt5.plasmatube
mnemosyne
mnemosyne.dist
mpc-qt
mpv
mpv-unwrapped
mpv-unwrapped.dev
mpv-unwrapped.man
mpvScripts.inhibit-gnome
mpvScripts.mpris
mpvpaper
plex-media-player
plex-mpv-shim
plex-mpv-shim.dist
python310Packages.glad2
python310Packages.glad2.dist
python310Packages.mpv
python310Packages.mpv.dist
python311Packages.mpv
python311Packages.mpv.dist
qimgv
radioboat
somafm-cli
stremio
sublime-music
sublime-music.dist
subtitleedit
ytfzf

libvlc failing.

libvlc failing.

Needs to bump, or backport this: https://code.videolan.org/videolan/vlc/-/merge_requests/3027

Result of nixpkgs-review pr 224795 run on x86_64-linux 1

1 package failed to build:

obs-studio-plugins.obs-ndi

101 packages built:

adl
ani-cli
anime-downloader
anime-downloader.dist
anki
anki.doc
anki.man
ankisyncd
arcanPackages.all-wrapped
arcanPackages.arcan
arcanPackages.arcan-wrapped
arcanPackages.cat9-wrapped
arcanPackages.durden-wrapped
arcanPackages.pipeworld-wrapped
arcanPackages.prio-wrapped
arcanPackages.xarcan
celluloid
cplay-ng
cplay-ng.dist
curseradio
curseradio.dist
deepin.deepin-movie-reborn
deepin.deepin-movie-reborn.dev
deepin.deepin-music
deepin.deepin-voice-note
dmlive
eaglemode
ff2mpv
gonic
haruna
hydrus
hydrus.doc
hypnotix
jellyfin-media-player
jellyfin-mpv-shim
jellyfin-mpv-shim.dist
jftui
kaffeine
kphotoalbum
libdovi
libplacebo
libsForQt5.elisa
libsForQt5.elisa.dev
libsForQt5.phonon-backend-vlc
libsForQt5.plasmatube
libvlc
megaglest
minitube
mnemosyne
mnemosyne.dist
mpc-qt
mpv
mpv-unwrapped
mpv-unwrapped.dev
mpv-unwrapped.man
mpvScripts.inhibit-gnome
mpvScripts.mpris
mpvpaper
obs-studio
obs-studio-plugins.advanced-scene-switcher
obs-studio-plugins.droidcam-obs
obs-studio-plugins.input-overlay
obs-studio-plugins.looking-glass-obs
obs-studio-plugins.obs-gstreamer
obs-studio-plugins.obs-hyperion
obs-studio-plugins.obs-livesplit-one
obs-studio-plugins.obs-move-transition
obs-studio-plugins.obs-multi-rtmp
obs-studio-plugins.obs-nvfbc
obs-studio-plugins.obs-pipewire-audio-capture
obs-studio-plugins.obs-source-record
obs-studio-plugins.obs-vaapi
obs-studio-plugins.obs-vkcapture
obs-studio-plugins.wlrobs
plex-media-player
plex-mpv-shim
plex-mpv-shim.dist
pympress
pympress.dist
python310Packages.glad2
python310Packages.glad2.dist
python310Packages.mpv
python310Packages.mpv.dist
python310Packages.python-vlc
python310Packages.python-vlc.dist
python311Packages.mpv
python311Packages.mpv.dist
python311Packages.python-vlc
python311Packages.python-vlc.dist
qimgv
radioboat
reaper
somafm-cli
strawberry
stremio
sublime-music
sublime-music.dist
subtitleedit
vlc
wtwitch
ytfzf


Add force-capture for Orbital for partial captures

cc @pierre for review.
This is to support partial captures to make sure that the transaction can go thru even if the underlying re-authorizations for subsequent partial captures fail. The workaround here is similar to the way we did for Cybs refund > 60 days

Just one question, otherwise :shipit:!
Note that releases are now automated: just submit a different PR similar to https://github.com/killbill/killbill-orbital-plugin/pull/10 against the release branch. Once we merge it, the plugin will be automatically released. 🎉

Great. Will do that once this one is merged.


code select

select all codes in a line by clicking the line number

Good candidate for an extension

You can also upvote this item on our feature backlog if you'd like this to be a built-in feature: Click/drag in gutter selects line(s), double-click selects block.


AMBARI-24639 Not able to turn off maintenance mode for a host.

How was this patch tested?
21815 passing (32s)
48 pending

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


Configure login page with an alternative branding

Replace the logo with https://unsplash.com/photos/ZQSs0YZUNfA
Name/Logo Text as SuperConnector
Back image with https://unsplash.com/photos/tR0jvlsmCuQ
Tag line: Shared seeing for coordinated action towards achieving outcomes and goals
Retain the background SVG as-is
Refer to the existing login page screenshot

New HTML File for login page - should be configurable (open source)

Can you please what on this page must be kept as-is? For example, this page must "contain login widget (user id, password, login button)".
Can we allow the user to change the position of the login widget, background image, text, etc. on the page? If so, we need to provide

a default template in which the admin can provide branding-related assets and get started
allow admin to provide an entirely custom HTML which "contains login widget" but everything else is custom.

@surendrasinghs
As of now the below configurations are available for the login screen. But the place holder should be the same. But it is possible to make the placeholder also configurable.
Fixed fields on the login page are,

User Name
Password
Login Button

Configured fields are,

Logo should be configurable
cQube heading is configurable
Tagline is configurable

Yet to do things are,

Configurable background color
Configurable water Mark
Configurable Images

NOTE: Requesting you to provide the Logo and Back images in PNG format.


#34 Validator/Check XSS for Title

#34 Validator/Check XSS for Title:

Add package react-hook-form for check validator
Add package xss for check xss
Check validator: Length from 1 to 256 characters
Check xss

@trinhlaosu Please fill missing field of issue.
Thank you


feat(chips): Add user defined tab index to chip list

Closes #6069

Add merge-ready when you're ready


Adds groups and a template for the Creative AI cluster

Hey, Dom.
I added a "Groups" section. We can link to them in the affiliations section of each user. The name and content is still to be debated. I wonder: is there a way of not rendering the section in the landing page? I should check the documentation more carefully.
Cheers.

Good news: going into content/home/groups.md and changing "active = true" to false means that Groups don't appear on the homepage, but you can still see the affiliations generated by it. That's how we want it, right?

Good news: going into content/home/groups.md and changing "active = true" to false means that Groups don't appear on the homepage, but you can still see the affiliations generated by it. That's how we want it, right?

Great!, yeah. That's exactly what I wanted. It'll depend on what everyone opines I guess. Besides the Creative AI Lab, do you think there'd be any other groups?

Great!, yeah. That's exactly what I wanted. It'll depend on what everyone opines I guess. Besides the Creative AI Lab, do you think there'd be any other groups?

There's MAD and CULT I think too. Probably a better question for the higher-ups.

There's MAD and CULT I think too. Probably a better question for the higher-ups.


lark vs. lark-parser

The lark name clash between
https://github.com/lark-parser/lark
and
https://github.com/voidfiles/lark
is currently causing some discussion inside Debian (see https://bugs.debian.org/941657#42 and the following message in that bug). There is no straightforward way to package both for Debian (see also the reference in my last sentence for details). Currently none of those two are included in Debian but there are plans to package your software for Debian. While there are - as far as I know - no plans to include the second at present, it would be nice to keep the door open to include both in Debian.
It also seems that this name similarity caused trouble for users of other projects (see e. g. https://github.com/hpc/charliecloud/issues/494).
For PyPI you seem to have solved this name clash by using lark-parser as name. Do you think it might be good (at least in the long run) to consistently rename lark throughout the code (although this would be a disruptive change)?

Thank you for allowing me to weigh in on this.
I think it's fair to assume that the other lark project is a dead project,
You already mentioned how it hasn't been updated in 6 years. In fact, its whole life-time was just shy of two weeks, between its first commit and last commit. (see here: https://github.com/voidfiles/lark/commits/master)
That is very problematic for a web project. In web, the entire tech stack changes every two years.
In fact, there are already many newer and more popular projects nowadays, that provide exactly what it provides, and much more.
Additionally, github's new and useful "Used By" feature reveals not a single popular repository uses the other lark. As for my Lark: It's actively maintained, and still gaining popularity. It's not even three years old, and it's used by 15 times more packages, several of which, each on their own, have more stars than the other lark. They will all be very inconvenienced if I change my import name.
In conclusion: That Lark is pining for the fjords. It is an ex-lark. If you haven't added it yet, chances are you never will.
Of course, I'm biased, but I think the reasonable choice is to add my package as-is.

@erezsh: Thanks for sharing your thoughts on this. You might be right that the problem that it is not easily possible to package both larks for Debian is rather a hypothetical than a real problem given the history of the other lark project.
Still I think that it is unfortunate to have two different projects using the same or similar names since this might confuse people as the above example shows. This danger will persist at least as long as both codes are available - even if one project is dead.
On the other hand I can perfectly understand your concerns to introduce such a breaking change - in particular since there are many dependent projects that could be upset by such a move.

l can see that neither choice is optimal. I hope you will choose the lesser of the two evils, whatever that is, in your eyes.
If you don't mind, I'll appreciate if you let me know what course of action you chose. And if there's anything I can do to help, that isn't changing the import name, please let me know.
Happy new year!

@wiene In case it is still relevant, pip install lark is now possible: #712 (and the other package is no longer on pypi)

@wiene In case it is still relevant, pip install lark is now possible: #712 (and the other package is no longer on pypi)

Thanks for letting me know. This goes well with the Debian package name. :-)


Use Localizer in Layouts

In LayoutEditor.Template.Toolbox.cshtml
who we can use T("") for {{item.toolboxLabel}}

This has to be solved by this issue: https://github.com/OrchardCMS/Orchard/issues/6750


Filter functionality doesn't seem to work

I am getting error as attached while performing a filter. Tree is being loaded correctly and selection, deselection works fine. Search text given is 6 . Have added comboTreePlugin.js & icontains.js. Please help.

$('#chkBoxTree').comboTree({
                            source: SampleJSONData,
                            isMultiple: true
                           });

I will check ASAP.
Thanks...

It is fixed on v1.2.1.

Thank you. Your help is greatly appreciated.
On Sat, Jan 11, 2020 at 12:23 PM Erhan FIRAT<EMAIL_ADDRESS>wrote:

Closed #22 https://github.com/erhanfirat/combo-tree/issues/22.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/erhanfirat/combo-tree/issues/22?email_source=notifications&email_token=AOHHIMTHDMHKQBIJAJORVDDQ5H575A5CNFSM4KEOULKKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOV5EA65I#event-2940735349,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOHHIMS2LOMYRCJMKGBKF33Q5H575ANCNFSM4KEOULKA
.


Allow alerting on external endpoints that do not receive a push within a configurable time frame

For step 3. , will it also include the negative "Said endpoint must automatically check if an alert has not been triggered" i.e. raise an alert if no result received within a specified period ?
(aka push-based monitoring)

Originally posted by @r3mi in https://github.com/TwiN/gatus/issues/722#issuecomment-2041015420

Thank you for creating the issue.
Now that external endpoints have been implemented (#722, #724), this should probably be the next feature related to external endpoints that gets implemented as they kind of go hand in hand, especially for those using external endpoints to test connectivity. If there's no connectivity, Gatus' API won't be reachable, which means that Gatus wouldn't be able to trigger an alert without this feature.
The feature in question should allow the user to configure a duration under which an update is expected to be received.
Should the duration elapse with no new status update, a status should be created to indicate a failure to receive an update within the expected time frame.
This should in turn cause https://github.com/TwiN/gatus/blob/28339684bfd9bb0e13bed4409f88e64bff31f3b2/watchdog/alerting.go#L13-L22 to be called, which would then lead to handleAlertsToTrigger being called (due to the new result indicating failure to receive an update having its Success field set to false), incrementing NumberOfFailuresInARow https://github.com/TwiN/gatus/blob/28339684bfd9bb0e13bed4409f88e64bff31f3b2/watchdog/alerting.go#L24-L27 & triggering whichever alerts should be triggered.
The only proper name I can think of for this feature is "dead man's switch", but as silly as it may sound, I don't like how that'd look on the configuration.
Just thinking out loud here
external-endpoints:
  - name: ...
    dead-man-switch:
      duration-until-trigger-automatic-failure: 1h
    alerts:
      - type: slack
        send-on-resolved: true

Another consideration to make is the interaction between this feature and maintenance. While the maintenance period should prevent alerts from being triggered, should failure status be pushed anyways? Perhaps this should be an additional parameter on the maintenance configuration (e.g. maintenance.silence-dead-man-switch)?
Anyways, just food for thoughts

The only proper name I can think of for this feature is "dead man's switch", but as silly as it may sound, I don't like how that'd look on the configuration:

I've seen other services call this heartbeat instead of dead man's switch

I would love to see a feature similar to this feature request, just like implemented in healthchecks.io
This would allow to provide a cron schedule and then monitor crons and alert for crons that do not run or jobs that take too long to complete.
An example configuration might look like this:
external-endpoints:
  - name: ...
    heartbeat:
      schedule: 0 0 15 * *
      grace-period: 2m


Add GKE validator for IP ranges

The primary and secondary IP ranges of a GKE cluster are immutable, so
they need to be correct when they are first provided. This change adds
some surface-level validation for the node, pod, and services IP ranges
if provided for the cluster. The valid ranges are based on the
documented requirements[1]. This will not be able to validate whether an
IP range is overlapping with the range of an existing cluster, and it
does not check whether the node range is valid for the number of nodes
in the cluster, which would need to take into account the number of
nodes in each node pool as well as the node autoscaling configuration.
[1] https://cloud.google.com/kubernetes-engine/docs/concepts/alias-ips
https://github.com/rancher/rancher/issues/32207

Wes is going to work on this in UI

Closing in favor of handling this in the UI


Donations from server not ordered by date

Making a GET request to the API for all of the user's donations pulls all donations except does not sort them by date. See picture.

<Hacksmiths {...props} />

@Gabri3l has this been sorted? Which commit?

I will fix this this weekend! Stay tuned ahaha

Thanks @Gabri3l


fix: Load image from gallery & iOS Swift callback

Fix: https://github.com/juliansteenbakker/mobile_scanner/issues/490
Following discussion https://github.com/juliansteenbakker/mobile_scanner/pull/492

Thanks for bringing this to my attention, however i did not found this solution the right way to do this. I've fixed this in #512


Add support for osu! profile migration when signed up for website with osu! OAuth

Close #90
Now when user signed up using osu! OAuth, the signals will get an information that can fill in Profile to user's profile.

Let's merge now!


[proto] Ported SimpleCopyPaste transform

Ported SimpleCopyPaste
Add tests

Example with MSCoco (batchsize = 2):

Seems like git merge has failed as on main we removed PIL and here I'm still using it.


Update Lab_26_AddTermsOfUseAcceptanceReporting.md

correct bolding
Module: 00
Lab/Demo: 00
Fixes # .
Changes proposed in this pull request:

This was fixed in another bug update.


Explain how to set up CORS headers on welcome page

We can't differentiate "api not running" from "api running but no CORS headers configured" so we'll have to add CORS header config messaging to the welcome page, as it's likely to be the second problem that users hit if they try and use the webui from https://webui.ipfs.io

Pervious discussion here https://github.com/ipfs-shipyard/ipfs-webui/issues/578

We could try requesting the empty dir CID from a gateway running on the default port http://<IP_ADDRESS>:8080/ipfs/QmUNLLsPACCz1vLxQVkXqqLX5R1X345qqfHbsf67hvA3Nn

That way we can infer if you have a daemon running or not.

We can use that info to focus on

no gateway - "download ipfs, set up cors, start the daemon"
gateway but no api - configure cors, or set the address your api is on

There is the case that we don't detect at gateway because it's on a custom port or disabled, so arguably we should just show all the steps that need to be done to get this working.


Paginating FieldSlip#index

Go to the /fieldslips page and play with the new pagination.

coverage: 93.508%. remained the same
when pulling 3a919e5e774b00e06d0f47a008ce90c89b4b6ca2 on njw-paginate-field-slip-index
into 93755bce4a9e58ac6eb29c87c7d20307ed938094 on main.

I'm debating whether to just submit this as is or if I should include allowing FiledSlips to be filtered by project and put a link to such URLs on the Project#show page.

Are you sure you can't just use our MO pagination?
We don't need any gems to paginate an index.

In the last MO Tech Meeting, @pellaea recommended finding whatever the standard way is to do pagination in Rails these days and do that.  Otherwise we need to add all the plumbing to Query to get that mechanism to work.  It ended up being less than 10 lines of code and it was only that much because I prefer having pagination on both the top and the bottom of index pages.  The gem I added appears to be well known, widespread, and very actively maintained.  It can do a lot more than I have it doing, but even what it's doing seems great to me.  Happy to hear other arguments, but this makes a lot more sense to me than delving into Query for this.


🔧 fix(lyrics): update Google Custom Search API endpoint

Related issue: #18
The site_restricted_api will soon be dead and can currently only be used by old customers, simple fix using the non restricted one by editing the endpoint url.
Tested a typical search engine on my end, restricted to all the websites made to be scraped and all the three tests passed.

Thank you for this.
It also seems like scrapers from genius don't work well, I fixed it by replacing line 35 in lyrics.py to all_extracts = self.source_code.select('div[class*="Lyrics-sc-"]'). It works for me with the changes in this PR and the before mentioned change at this moment of time.

@i-vis Can you please raise a PR for the same. I'll merge the same from my other account.


🐛 Bug Report: username and password is not trimmed when checking

👟 Reproduction steps
paste a password or username with a space at the end, this happens in copy-paste in Windows for example
👍 Expected behavior
the strings should remove spaces at start or end before the account is checked
👎 Actual Behavior
the login fails because the username is compared with the space
📜 Logs
No response

Fixed in v1.2.1


Docker sugar + permissions fix

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Adjust syntax to follow best practices (mainly line breaks & indents)
Use COPY for local files (not archives)
Switch to CMD, use ENTRYPOINT as preparation step
Use gosu to drop privileges, ensure proper folder permissions on startup
Avoid additional packages by specifying --no-install-recommends

#3374

Also broke the example from the README:
$ docker run -it --rm --link some-clickhouse-server:clickhouse-server yandex/clickhouse-client:18.14.11 --host clickhouse-server
docker: Error response from daemon: OCI runtime create failed: container_linux.go:348: starting container process caused "exec: \"--host\": executable file not found in $PATH": unknown.


fix: instantiate mvars of indices before instantiating fvars

WIP PR to fix the newly reported issue. The problem seems to be that some mvars are left uninstantiated before instantiating the free vars present in the indices of mutual inductive types.
Closes #3242 (again)

WIP

I haven't had time to make thorough tests to see if this really solves the issue for good. Knowing the issue popped up once after being already closed, I'd like to make sure this doesn't happen again. I haven't dived into how inductive headers are unified yet, and don't know if some expr mvars aren't expected to be unified later on here. If the expected behavior is that all expr mvars in the headers have already been solved at this point, then this certainly is the right solution, and I'd be happy to get this merged.
I'm not available to investigate on this right now, but plan on looking into this in the coming days.

Despite multiple rebases, the CI on mathlib/batteries seems to keep on failing, and I'm not sure why. Looking at the errors, I don't think the failures are due to the changes made in that PR. I'd like to see at least one CI run work "correctly" on this to ensure no breakage was made, but otherwise, I'm confident this PR makes the right fixes, and will be marking it as ready for review.

awaiting-review

Glad it's green now. Don’t forget to update the PR description, as it will become the commit message.

Looks good to me, I don’t think I understand the intricacies any better than you. Would you mind leaving a comment in the code with whatever insights you have so that the next person looking at the code won’t have to re-discover it?


Allow SDF model to be constructed in a single shot

🎉 New feature
Summary
This PR tweaks the behavior of the physics system so that when constructing a model it will first check if a ModelSdf component is available for the entity. If so, it uses that component to fill in all the model information, overriding the fields that are given by the other entity components (e.g. name and pose).
When creating a link, the physics engine will first check if a link of the same name for that model was already constructed. If so, the construction step is skipped, the link is queried from the model, and it gets added to the entity map. Same for joints and collision objects.
The net result of these behavioral changes is that physics engines which can only construct models all at once are able to be used for the typical use cases, while physics engines that can construct models incrementally can still do so when the user wants them to.
Test it
Try running with the dartsim plugin. Even though the behavior changed, there is no noticeable difference in the outcome.
The ability to test these changes with a bullet-featherstone plugin is forthcoming. The implementation is finished, but I am debugging some segfaults from inside of bullet at the moment.
Checklist

[ ] Signed all commits for DCO
[ ] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Joint is not placed in the right pose (bullet-featherstone), tested with log_record_dbl_pendulum.sdf

The seg fault issue with the classic bullet plugin is fixed by gazebosim/gz-physics#393 and the TPE failing to load is fixed by 4b01010.
I think the joint offset issue for bullet-featherstone shouldn't be a blocker for this PR since that will need to be fixed in gazebosim/gz-physics#373 and wouldn't involve any changes to this PR.

@osrf-jenkins retest this please

@mxgrey we have a whole slew of test failures here that aren't in main.  I spent a little bit of time, but didn't make much headway.

Removed the Garden label because we're past feature freeze ❄ .
This PR can either be merged into main and backported to gz-physics6 after the stable release, or retargeted to gz-physics6 and merged after the stable release (i.e. October).

Removed the Garden label because we're past feature freeze ❄ .
This PR can either be merged into main and backported to gz-sim7 after the stable release, or retargeted to gz-sim7 and merged after the stable release (i.e. October).

@osrf-jenkins retest this please

@osrf-jenkins retest this please

ackermann_steering
ok

acoustic_comms_demo
vehicles are not moving properly

acoustic_comms_moving_targets.sdf
wierd

acoustic_comms.sdf
ok

actor_crowd
ok

linear_battery_demo.sdf
ok

logical_audio_sensor_plugin.sdf
ok

logical_camera_sensor.sdf
ok

log_record_dbl_pendulum.sdf
ok

log_record_keyboard.sdf
ok

actor_crowd.sdf
ok

log_record_resources.sdf
ok

actor.sdf
ok

log_record_shapes.sdf
ok

actors_population.sdf
ok

mecanum_drive.sdf
not working

minimal_scene.sdf
ok

apply_joint_force.sdf
vel can be setted but when zero is setted the joint will not stop (damping is not applied)

model_photo_shoot.sdf
segfault

apply_link_wrench.sdf
ok

multicopter_velocity_control.sdf
small quad can takeoff but the hexacopter cannot

auv_controls.sdf
doing the wrong thing

multi_lrauv_race.sdf
the vehicle in the middle is not working properly the other two sometimes are doing wierd stuff too

boundingbox_camera.sdf
ok

nested_model.sdf
ok

breadcrumbs.sdf
box is getting some initial speed and the vehicle is not include in the physics engine

optical_tactile_sensor_plugin.sdf
ok

buoyancy_engine.sdf
boxes are moving up and down, not sure if this is ok

particle_emitter.sdf
ok

buoyancy.sdf
motor is rotating but the model is stuck in the same position

pendulum_links.sdf
removing the velocity plugin works well, commands are doing wierd things

camera_sensor.sdf
ok

perfect_comms.sdf
ok

camera_video_record_dbl_pendulum.sdf
ok

performer_detector.sdf
ok

physics_options.sdf
ok

collada_world_exporter.sdf
ok

plane_propeller_demo.sdf
plane doing wierd things, crazy since the beginning of the simulation

contact_sensor.sdf
ok

plot_3d.sdf
ok

conveyor.sdf
feature not implemented

polylines.sdf
polyline not supported as a collision shape

debug_shapes.sdf
ok

pose_publisher.sdf
ok

default.sdf
ok

quadcopter.sdf
ok

dem_monterey_bay.sdf
not supported

rf_comms.sdf
ok

dem_moon.sdf
not supported

rolling_shapes.sdf
ok

dem_volcano.sdf
not supported

segmentation_camera.sdf
ok

deprecated_ignition.sdf
ok

sensors_demo.sdf
ok

depth_camera_sensor.sdf
ok

sensors.sdf
ok

detachable_joint.sdf
ok

shader_param.sdf
segfautl

diff_drive.sdf
ok

shapes_bitmask.sdf
not implemented

diff_drive_skid.sdf
ok

elevator.sdf
ok (need to adjust PID)

shapes.sdf
ok

empty_gui.sdf
ok

skid_steer_mecanum.sdf
not turning properly

empty.sdf
ok

sky.sdf
ok

follow_actor.sdf
ok

spaces.sdf
ok

fuel.sdf
ok (heightmap not supported)

spherical_coordinates.sdf
when spawn a model, the model is crazy

fuel_textured_mesh.sdf
ok

thermal_camera.sdf
error reading world, might be related with the migration, this world should work

gpu_lidar_retro_values_sensor.sdf
ok

gpu_lidar_sensor.sdf
ok

touch_plugin.sdf
ok

graded_buoyancy.sdf
some physics errors

track_drive.sdf
models are crazy

grid.sdf
ok

tracked_vehicle_simple.sdf
segfault

heightmap.sdf
not supported

trajectory_follower.sdf
working but not doing the same as dartsim

import_mesh.sdf
ok

triggered_camera_sensor.sdf
ok

joint_controller.sdf
ok

triggered_publisher.sdf
ok

joint_position_controller.sdf
ok

trisphere_cycle_wheel_slip.sdf
ok

joint_trajectory_controller.sdf
ok

tunnel.sdf
segfault

kinetic_energy_monitor.sdf
not working (missing features?)

velocity_control.sdf
ok

levels_no_performers.sdf

video_record_dbl_pendulum.sdf
ok

levels.sdf
ok

visibility.sdf
ok

lift_drag_battery.sdf
joint rotating but not taking off

visualize_contacts.sdf
ok

lift_drag_nested.sdf
issues building model

visualize_lidar.sdf
ok (car jumps at the beginning)

lift_drag.sdf
joint rotates but the model is not taking off

wide_angle_camera.sdf
ok

lightmap.sdf
not able to see anything in the minimal scene (rendering issue?)

wind.sdf
ok

lights.sdf
ok

world_joint.sdf
segfault

NOTES:

There are some tutorials to record logs but it will be great to add some instructions to the sdf files

@osrf-jenkins retest this please

@mxgrey @ahcorde What needs to be done to merge this one?

@osrf-jenkins retest this please

@mxgrey @ahcorde What needs to be done to merge this one?

I believe that I got all the tests passing, but CI isn't reflecting that yet, giving it another go.

@osrf-jenkins retest this please


Add status field to Credential and Offer aggregates in agent_issuance

Description
This status field will be used to track whether a Credential/Offer is Pending, Rejected, Accepted, etc
Motivation
Improves UniCore's utility
Resources
n/a
To-do List

[ ] Add status field to Credential
[ ] Add status field to Offer

 Add status field to Credential and Offer aggregates in agent_issuance


Edge Pruning - user defined upper bound

Edge filtration is based on the distance field and has an explicit range of 0 -> 0.05.
However, users with a bacterial focus are likely to measure distance in integer values (SNPs = single nucleotide polymorphisms).
We'd like users to be able to arbitrarily define the upper bound of their pruning criteria.

Fixed a few commits ago, but started working well in b5a22ad2ed7af508fb38a6bb1a897c9340c47fd0


Change minimum and maximum salary fields

Jira ticket URL:
https://dfedigital.atlassian.net/secure/RapidBoard.jspa?rapidView=21&projectKey=TEVA&modal=detail&selectedIssue=TEVA-217
https://dfedigital.atlassian.net/secure/RapidBoard.jspa?rapidView=21&projectKey=TEVA&modal=detail&selectedIssue=TEVA-219
Screenshots of UI changes:
Before

After

Very good point, thanks @dushan-madetech


Update jhub apps related docs

This PR provides updates to all the jhub-apps related UI elements.
I've also removed some of the app creation workflow and instead pointed to the jhub-apps docs in order to reduce maintenance here.
Reference Issues or PRs
Resolves https://github.com/nebari-dev/nebari-docs/issues/441
What does this implement/fix?
Put a x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features not to work as expected)
[x] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Other (please describe):

Testing

[ ] Did you test the pull request locally?
[ ] Did you add new tests?

Documentation

Access-centered content checklist
Text styling

[x] The content is written with plain language (where relevant).
[x] If there are headers, they use the proper header tags (with only one level-one header: H1 or # in markdown).
[x] All links describe where they link to (for example, check the Nebari website).
[x] This content adheres to the Nebari style guides.

Non-text content

[x] All content is represented as text (for example, images need alt text, and videos need captions or descriptive transcripts).
[x] If there are emojis, there are not more than three in a row.
[x] Don't use flashing GIFs or videos.
[x] If the content were to be read as plain text, it still makes sense, and no information is missing.

Any other comments?

@kcpevey is this ready for review?

@viniciusdc yes, its ready for review


Update WelcomeView.swift

Description
Changes the welcome view options to be more intuitive and descriptive of the user action.

Instead of the welcome options being

Create a new file
Clone an existing project
Open a project or file

A more intuitive list would be:

Create a file
Open a file or folder
Clone an existing project

Moving "Clone an existing project" lower seems like a good idea since it's probably a less-used option. And changing the name of "Open a project or file" to "Open a file or folder" seems more descriptive of what the real action is.
Releated Issue
Issue #192

Checklist (for drafts):

[x] I read and understood the contributing guide as well as the code of conduct
[x] My changes generate no new warnings
[x] My code builds and runs on my machine
[x] SwiftLint Run
[ ] Review requested

Screenshots (if appropriate):

Before

After

I would approve this argumentation!
@austincondiff ?


Fix floating ip quota in network-design.md

Since we now have another subnet for floating ips, we should make a
note of this in the network design document.
Signed-off-by: Cullen Taylor<EMAIL_ADDRESS>

I think the wording is pretty fine. The subtle difference between "quota" and the ranges we pay for is probably not important.
However, we did recently add a second range of floating addresses, <IP_ADDRESS>/27. We need to account for this range in the doc.

@j2sol updated.