Send branch on sync if available for rules. id:128

Code comment: DOING: Send branch on sync if available for rules. id:128 gh:135
in file: lib/services/imdoneio-client.coffee:273 has been modified.
Issue Comment Bot - @imdone. Get TODOBOTs to do your work at imdone.io

Code comment: DOING: Send branch on sync if available for rules. id:128 gh:135 gh:136
in file: lib/services/imdoneio-client.coffee:273 has been modified.
Issue Comment Bot - @imdone. Get TODOBOTs to do your work at imdone.io

Code comment: DOING: Send branch on sync if available for rules. id:128 gh:135
in file: lib/services/imdoneio-client.coffee:273 has been modified.
Issue Comment Bot - @imdone. Get TODOBOTs to do your work at imdone.io

Code comment: DOING: Send branch on sync if available for rules. id:128 gh:135 +now
in file: lib/services/imdoneio-client.coffee:273 has been modified.
Issue Comment Bot - @imdone. Get TODOBOTs to do your work at imdone.io

READY: Send branch on sync if available for rules. gh:135 +now id:44
lib/services/imdoneio-client.coffee:289
@imdone comment-bot. Get TODOBOTs to do your work at imdone.io

READY: Send branch on sync if available for rules. gh:135 +now id:36
lib/services/imdoneio-client.coffee:289
@imdone comment-bot. Get TODOBOTs to do your work at imdone.io


OPA - Not adding ApiGroups field makes the constraint to not work properly

What kind of request is this:
Bug/Enhancement
Steps to reproduce (least amount of steps as possible):

Create a cluster, navigate to dashboard
Navigate to OPA Gatekeeper
Click Enable
Add a New k8srequiredlabels constrain as Form

Edit Parameters as YAML:
labels:
- allowedRegex: ^$|^[a-zA-Z0-9]+
  key: gatekeep
message: All pods must have a `gatekeep` label

Add Kind Pod
Do not add an ApiGroup leave it there. Just add the Kind to match:

Result:
Yaml generated for this will be:
match:
    kinds:
    - kinds:
      - Pod

Expected:  If we try to create a workload without the gatekeep app gatekeeper should prevent it and raise an error. Also show violations for the existing workloads.
Actual: pods gets created without the mandatory label.
If we click in Add ApiGroup and leave it blank:

YAML is now:
match:
  kinds:
  - apiGroups:
    - ""
    kinds:
    - Pod

This makes the constraint work.
Environment information

Rancher version:

master-head (06/11/2020) b89c3a572

Installation option (single install/HA): Single
Cluster type: RKE (DO)

This was previously fixed #387

The behavior was recently changed as per request in this issue:
https://github.com/rancher/dashboard/issues/392#issuecomment-631074796

This affects all types of contraints in the Dashboard, repos and labels.
Other non-default Rancher types also include ApiGroups
https://github.com/open-policy-agent/gatekeeper/tree/master/library/general

With that information I'm planning to just add the empty apiGroups if it's not present when saving.

When not adding a n APIGroups field in the UI this is the generated kinds object in the YAML:
    kinds:
    - apiGroups:
      - '*'
      kinds:
      - Pod

If we add one APIGroups field in the UI and left empty we get the following YAML:
    kinds:
    - apiGroups:
      - ""
      kinds:
      - Pod

Which is expected as we're leaving the field empty and therefore the "".


[SPARK-17714][Core][maven]Avoid using ExecutorClassLoader to load Netty generated classes

What changes were proposed in this pull request?
Netty's MessageToMessageEncoder uses Javassist to generate a matcher class and the implementation calls Class.forName to check if this calls is already generated. If MessageEncoder or MessageDecoder is created in ExecutorClassLoader.findClass, it will cause ClassCircularityError. This is because loading this Netty generated class will call ExecutorClassLoader.findClass to search this class, and ExecutorClassLoader will try to use RPC to load it and cause to load the non-exist matcher class again. JVM will report ClassCircularityError to prevent such infinite recursion.
Why it only happens in Maven builds
It's because Maven and SBT have different class loader tree. The Maven build will set a URLClassLoader as the current context class loader to run the tests and expose this issue. The class loader tree is as following:
bootstrap class loader ------ ... ----- REPL class loader ---- ExecutorClassLoader
|
|
URLClasssLoader

The SBT build uses the bootstrap class loader directly and ReplSuite.test("propagation of local properties") is the first test in ReplSuite, which happens to load io/netty/util/internal/__matchers__/org/apache/spark/network/protocol/MessageMatcher into the bootstrap class loader (Note: in maven build, it's loaded into URLClasssLoader so it cannot be found in ExecutorClassLoader). This issue can be reproduced in SBT as well. Here are the produce steps:

Enable hadoop.caller.context.enabled.
Replace Class.forName with Utils.classForName in object CallerContext.
Ignore ReplSuite.test("propagation of local properties").
Run ReplSuite using SBT.

This PR just creates a singleton MessageEncoder and MessageDecoder and makes sure they are created before switching to ExecutorClassLoader. TransportContext will be created when creating RpcEnv and that happens before creating ExecutorClassLoader.
How was this patch tested?
Jenkins

Test build #72598 has started for PR 16859 at commit 1c88474.

Test build #72598 has finished for PR 16859 at commit 1c88474.

This patch fails to build.
This patch merges cleanly.
This patch adds the following public classes (experimental):
public class TransportChannelHandler extends ChannelInboundHandlerAdapter

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72598/
Test FAILed.

Hi, @zsxwing .
Currently, SparkPullRequestBuilder is broken and HOTFIX is #16858 .

retest this please

Test build #72599 has started for PR 16859 at commit 1c88474.

Test build #72599 has finished for PR 16859 at commit 1c88474.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
public class TransportChannelHandler extends ChannelInboundHandlerAdapter

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72599/
Test PASSed.

cc @vanzin

btw I think you want [test-maven] to run the builder on maven. (last one ran on sbt.)

Test build #3570 has started for PR 16859 at commit 1c88474.

Test build #3570 has finished for PR 16859 at commit 1c88474.

This patch fails from timeout after a configured wait of `250m`.
This patch merges cleanly.
This patch adds the following public classes (experimental):
public class TransportChannelHandler extends ChannelInboundHandlerAdapter

Retest this please

Test build #72655 has started for PR 16859 at commit 1c88474.

Test build #72655 has finished for PR 16859 at commit 1c88474.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
public class TransportChannelHandler extends ChannelInboundHandlerAdapter

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72655/
Test FAILed.

Merged build finished. Test FAILed.

Test build #3570 has finished for PR 16859 at commit 1c88474.

This is the known OOM issue.

Test build #72655 has finished for PR 16859 at commit 1c88474.

The only failure test is the kafka flaky test.

I'm going to change the default value back since ReplSuite passed in Test build #72655 has finished for PR 16859 at commit 1c88474..

Test build #72660 has started for PR 16859 at commit 20b6fd4.

Test build #72660 has finished for PR 16859 at commit 20b6fd4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72660/
Test PASSed.

Test build #72665 has started for PR 16859 at commit 7fcb317.

Test build #72665 has finished for PR 16859 at commit 7fcb317.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72665/
Test PASSed.

Merged build finished. Test PASSed.

Thanks! Merging to master and 2.1.

@vanzin I just merged this one. If you have other comments, I will address them in a separate PR. Thanks!

Nope, don't have anything else.


Add alpha config for cmd line river_node tool

+add missing addrs for other envs.
For example
./core/node/env/alpha/run.sh registry streams --count

Prints stream count from alpha registry contract.

@sergekh2 reminding you to hold off on merging the alpha addresses for now - since they are about to change

updated now


Add Column Window (Create Table Button)

Describe the bug
Whenever I try to create a table then click submit for the new columns. The table and columns fails to be added.
To Reproduce
Steps to reproduce the behavior:

Go to 'A database'
Click on 'Create Table then finish the process of creating and click on submit'
It would exit you out of the application
Scroll down to 'the database that you have entered the new table'
See error

Expected behavior
It should have added the newly created table to the database
Screenshots

Desktop (please complete the following information):

OS: Windows
Browser: Chrome
Version: 10

Smartphone (please complete the following information):

Device: Xiaomi Note 10
OS: Android
Browser Chrome
Version 9.0

Additional context
I tried to rerun the application then signin and signout but it does not fix the problem. But all of the buttons rather than the submit button seems to work perfectly.

tried to reproduce the bug, but can't reproduce it

Everything works perfectly fine, however, The information tab was not updated, I'll create an issue for it

The problem will continue at #30. Please reopen and provide more details on how to reproduce the said problem again if ever that the problem persist.


Type is null if the first row has null value

Hello!
Thanks for the plugin, it's been super helpful. However, I think I've come across a bug. Let me explain.
When a table has a null value on the first row, the plugin assign null type to that column, but if it has a value, and the others rows have or not, it correctly assign the type.
Here is an example. I have the following table:

id (INT)
option (STRING)
order (INT)

1
(NULL)
1

2
(NULL)
2

3
A
1

4
A
2

5
(NULL)
3

When I try to get the option value like this.
Result = Statement->ExecuteSelect();
	for (FQueryResultRow row : Result.Rows){
		for (FQueryResultField field : row.Fields){
			if (field.ColName == "option")
                           step.option = field.StrVal;
LogTemp: Warning: Option:
LogTemp: Warning: Option:
LogTemp: Warning: Option:
LogTemp: Warning: Option:
LogTemp: Warning: Option:
field.StrVal is always empty because field.type is null. But if I assign a value on the first row.

id (INT)
option (STRING)
order (INT)

1
A
1

2
(NULL)
2

3
A
1

4
A
2

5
(NULL)
3

field.StrVal is empty when the value is null (obviously) but it is not on the others. And field.type is FString.
LogTemp: Warning: Option: A
LogTemp: Warning: Option:
LogTemp: Warning: Option: A
LogTemp: Warning: Option: A
LogTemp: Warning: Option:
Maybe I'm missing something but I don't know why this is happening.
Thanks in advance!

Well, you and I would consider that a bug... but unfortunately I don't think SQLite agrees.
Because (for better or worse) SQLite prides itself on its 'loosey-goosey' approach to data typing, which is a pretty big deviation from basically EVERY OTHER SQL database ever made.
Generally with any kind of resultset, either directly from a single table, or a synthetic one, you have 2 things; the data itself (a number of rows) and the definition that this data subscribes to.
The data rows in a proper SQL resultset cannot change datatype half way through the results - field 1, is always of whatever the type is, and NULL is obviously NOT a row type, I mean, what would be the purpose of having a database field designed to store only nulls?
Rather NULL is a (potentially) permissible value for any other datatype.
I wrote the function in the helper plugin to get the column names and types, and then get the data (because its generally 2 separate operations) and the column data I get using the sqlite3_column_type() function of the prepared statement.
And at the time, I guess I assumed it worked like a SQL database generally does. (My bad!)
Your evidence basically suggests, that rather than working as anyone familiar with SQL would expect, the results it returns are in fact NOT related to the prepared statement as a whole, but are actually influenced by each Step() while running it.
There are a few ways to approach 'fixing' this:

If using the newer versions of SQLite (the plugin has a fairly new version) you can use the STRICT TABLE option (required individually on EACH table!) This MIGHT make a difference to how things work/are reported (but I wouldnt guarantee it, and it would need to be tested)
You can manually fudge any query to ensure that it never returns NULLS, or, that there is a FAKE FIRST ROW with no nulls in it.
I could rewrite the code to preparse the entire resultset, filling in the column type for each column, when the value isnt null, and stopping once it has a complete set of column types with non null data.

That last one sounds workable until you consider the following:

What if all the data in a given column is null for a returned dataset? Actually quite possible with filtering, etc.
What if, due to the 'flexible' typing in SQLite, the first column of the first row returns an INTEGER, and the first column of the second row, returns TEXT? Also quite possible with SQLite.

OR WE FORGET ALL ABOUT 'NORMAL' SQL DATABASES...
And we just embrace the 'type fluidity' of SQLite, and I don't pre parse the columns at all, instead setting the fields of each row individually, for each row. Meaning that for a single resultset, you never really know what you are going to get until you look at it. And you cant assume anything, just because the previous row was that type.
I can make the changes for that last option, but I don't really have the time to test it right now.
As you have data which creates the error, I'm happy to commit the required changes if you want to have a go at testing them?

Hi there!
I Appreciate your detailed response. I experimented with the STRICT option, but it's basically doing the same thing.
About the first dummy row, I tried instead inserting the string "NULL" when the value is null, but like you said, it's not the best solution.
I believe the third option might be effective in my situation, because I'm not planning to use that flexibility of SQLite. If you could spare the time to provide the fix, I'd be more than happy to test it out.
Thanks!

Well, the change is there.
I re tested it on my base app, and it still worked (not testing your specific case though)
So let me know how it goes.
The main problem you are going to have now though, is that you will need to test every time
You could switch/case the 'type' property
or you could use the IsNull() method, something like:
FString ThisValue = ResultField.IsNull()?FString(TEXT("Nothing there!")):ResultField.StrVal;

Hello,
I forgot to mention that I wasn't using the latest version of the plugin. Since we're operating with UE5.1, I downloaded this release. It appears that the most recent version is not compatible, likely due to differences in the C++. If you know how to make it compatible, that would be great. However, for now, to address this issue, I have manually modified the file with the same changes you made.
Good news, it worked! Now, only the rows with null values are of null type, allowing me to easily check the type or if the string is empty.
Thanks!

Glad it works for you! 😄
There are different C++ versions supported, primarily by different compiler toolchains/coupled with different engines.
I was under the impression that C++20 was supported by UE5.0 and higher, as long as you are compiling it with something that uses C++20 (you can still use VS2019 hoever which probably isnt rocking C++20 as standard)
So perhaps, upgrade your IDE to VS2022 if you are still on VS2019 for example.
Either way, in this case the differences to the code are actually minimal, and you would find them yourself very quickly if you tried to build it.
(I have a friend using my plugin on UE4.27, and you need to change only 1 or 2 lines of code max)
In case you ever want to try it, it is just a couple of IF statements where a variable is declare INSIDE the IF parentheses as part of the statement, as opposed to having a single line declaration before the IF statement
(My IDE is Rider and it pushes such things on you!)
Was actually easy to just change those lines and rebuild


Update deploy-models-cohere-embed.md

Added a link to the index-add.md file

@msakande
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#please-close

@e-straight please create a PR in the private repo. Thanks


4 additional proposals from egallager

I am 1 of 400 state legislators in the state of New Hampshire, and one of the things I have been pushing for has been to get the legislature to start using GitHub (or something like it) for legislation. Unfortunately, GitHub currently lacks a few features that would be necessary before we could actually legislate here. I've outlined some of the ones I noticed with 3 new files in this pull request, as well as a 4th new file for another proposal that just comes from my personal usage of GitHub for other stuff.

These are also features that regular FOSS projects could also make use of, btw, for example, here's a proposal in the Wesnoth community that could make use of the "assign to committee and then approve via majority vote" proposal that I included in this PR: https://forums.wesnoth.org/viewtopic.php?t=58805


Lightning Network

Any plans on integrating/using Lightning Network for Bitcoin?

there are no such plans yet

Would consider working on a lightning integration because of high blockchain fees

Most people are not even able to pay with bitcoin bc of high blockchain fees - lightning which could make the app useful for everyday usage


[fix #508] restore previous optimization

[ ] confirm in big app
[ ] tests + related cleanup

@rwjblue @stefanpenner this optimization on maybeMergeTrees is ready? We just need to fix the tests?

@villander ya, I believe so. The above two checklist items should be accurate.

Ok @stefanpenner I'll help as soon as I can.

Ok @stefanpenner I'll help as soon as I can.

Awesome would love a hand, I keep getting pulled away before I can work on ^. So your welcome is much appreciated :)


Revert "Revert "fixup some unit tests""

Accidentally reverted this in #6898 but i has nothing directly to do with the attribute rewrite and is useful to keep.

(and mostly i'm reverting this here to hopefully keep the git timelines in closer synch so that patches to master that might build on top of this patch will hopefully not merge conflict on backporting)


[skip] Add explanation that roletoken is force true

Added an explanation for forced enable of roletoken.

Codecov Report

Merging #62 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #62   +/-   ##
=======================================
  Coverage   87.02%   87.02%
=======================================
  Files          14       14
  Lines         655      655
=======================================
  Hits          570      570
  Misses         75       75
  Partials       10       10

Impacted Files
Coverage Δ

config/config.go
100.00% <ø> (ø)

Continue to review full report at Codecov.

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


Blood Cell Detection

Deep Learning Simplified Repository (Proposing new issue)
:red_circle: Project Title : Blood Cell Detection
:red_circle: Aim : Create a model which will detect the blood cells.
:red_circle: Dataset : https://www.kaggle.com/datasets/adhoppin/blood-cell-detection-datatset
:red_circle: Approach : Try to use 3-4 algorithms to implement the models and compare all the algorithms to find out the best fitted algorithm for the model by checking the accuracy scores. Also do not forget to do a exploratory data analysis before creating any model.

📍 Follow the Guidelines to Contribute in the Project :

You need to create a separate folder named as the Project Title.
Inside that folder, there will be four main components.

Images - To store the required images.
Dataset - To store the dataset or, information/source about the dataset.
Model - To store the machine learning model you've created using the dataset.
requirements.txt - This file will contain the required packages/libraries to run the project in other machines.

Inside the Model folder, the README.md file must be filled up properly, with proper visualizations and conclusions.

:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Follow Contributing Guidelines & Code of Conduct before start Contributing.

:white_check_mark: To be Mentioned while taking the issue :

Full name :
GitHub Profile Link :
Email ID :
Participant ID (if applicable):
Approach for this Project :
What is your participant role? (Mention the Open Source program)

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

plz assign me this project @abhisheks008

Assigned it to you @sid-im! Go Ahead.

Can you share me the approach you are going to take while solving the issue. And please mention the required credentials written in the issue. @sid-im

Can you share me the approach you are going to take while solving the issue. And please mention the required credentials written in the issue. @sid-im

Yes @sid-im, It'd be great if you can kindly share your approach to work on this issue🙂

Full name : Siddarth Akkatangerhal
GitHub Profile Link :  https://github.com/sid-im
Email ID<EMAIL_ADDRESS>Participant ID (if applicable):
Approach for this Project :
i will use TensorFlow/Keras convolutional neural network to make our predictions using given data set.

@sid-im updates please.

Full name : Khushal Koli
GitHub Profile Link : https://github.com/khushaallll
Email ID<EMAIL_ADDRESS>Participant ID (if applicable):
Approach for this Project :

Planning to use different CNN models.

Planning on to use few pre-trained models, and YOLOv5

will try to construct a custom neural network as well
What is your participant role : -

Are you a participant from any open source program or, just wanna contribute here as a contributor?
@khushaallll

No I'm not a participant from open source program I just want to contribute as an independent contributor

Sounds great. Issue has been assigned to you @khushaallll. Go ahead!

Hey @khushaallll updates regarding this issue.

Hey @khushaallll updates regarding this issue

Yes working on it. Will be done by this weekend

Issue has been un-assigned from you due to inactivity.
@khushaallll


onHover event

I need to get id of row in on hover, right now I can achieve it using:
    const rowEvents = {
      onMouseEnter: (e) => this.onSelect(+e.target.parentNode.firstChild.textContent),
      onMouseLeave: (e) => this.onSelect(+e.target.parentNode.firstChild.textContent),
    };

Because of this event handler got only event argument I can get row id only in this curly way. Obviosly it will be more handy if this callbacks would receive row argument as onClick.

@abonec sorry for lately fixing, this requirement is fixed on<EMAIL_ADDRESS>check release note


How to use ND4j in android, please ??

Issue Description
Please describe your issue, along with:
ND4J is probably missing dependencies???
Version Information
Please indicate relevant versions, including, if relevant:

Deeplearning4j version
platform information: androind-arm64-v8a

Contributing
If you'd like to help us fix the issue by contributing some code, but would
like guidance or help in doing so, please mention it!

Please try to follow the instructions here for starters: https://deeplearning4j.org/android


Matplotlib Latex Font issue with Jupyter Notebooks and Interactive Window

Bug: Notebook Editor, Interactive Window, Editor cells

Steps to cause the bug to occur

Load matplotlib

2. Use latex for plots and load font in preamble

3. Plot something simple

Actual behavior
Error in rendering and outputting the plot for both Jupyter Notebook and interactive window
This seems to occur specifically due to the preamble : text.latex.preamble': r'\usepackage{libertine}'. Removing this the plot renders and shows properly.
Also, the error is in rendering the output in vscode. if you save the plot before you render it, it works properly and saves the plot with the libertine font.
Hence, this indicates that there is an issue when rendering the plot's output
Error message in vscode :
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
~/anaconda3/envs/wiki/lib/python3.7/site-packages/IPython/core/formatters.py in __call__(self, obj)
    339                 pass
    340             else:
--> 341                 return printer(obj)
    342             # Finally look for special method names
    343             method = get_real_method(obj, self.print_method)

~/anaconda3/envs/wiki/lib/python3.7/site-packages/IPython/core/pylabtools.py in (fig)
    252         jpg_formatter.for_type(Figure, lambda fig: print_figure(fig, 'jpg', **kwargs))
    253     if 'svg' in formats:
--> 254         svg_formatter.for_type(Figure, lambda fig: print_figure(fig, 'svg', **kwargs))
    255     if 'pdf' in formats:
    256         pdf_formatter.for_type(Figure, lambda fig: print_figure(fig, 'pdf', **kwargs))

~/anaconda3/envs/wiki/lib/python3.7/site-packages/IPython/core/pylabtools.py in print_figure(fig, fmt, bbox_inches, **kwargs)
    130         FigureCanvasBase(fig)
    131
--> 132     fig.canvas.print_figure(bytes_io, **kw)
    133     data = bytes_io.getvalue()
    134     if fmt == 'svg':

~/anaconda3/envs/wiki/lib/python3.7/site-packages/matplotlib/backend_bases.py in print_figure(self, filename, dpi, facecolor, edgecolor, orientation, format, bbox_inches, **kwargs)
   2054                         orientation=orientation,
   2055                         dryrun=True,
-> 2056                         **kwargs)
   2057                     renderer = self.figure._cachedRenderer
   2058                     bbox_artists = kwargs.pop("bbox_extra_artists", None)

~/anaconda3/envs/wiki/lib/python3.7/site-packages/matplotlib/backends/backend_svg.py in print_svg(self, filename, *args, **kwargs)
   1202                 detach = True
   1203
-> 1204             result = self._print_svg(filename, fh, **kwargs)
   1205
   1206             # Detach underlying stream from wrapper so that it remains open in

~/anaconda3/envs/wiki/lib/python3.7/site-packages/matplotlib/backends/backend_svg.py in _print_svg(self, filename, fh, dpi, bbox_inches_restore, **kwargs)
   1227             bbox_inches_restore=bbox_inches_restore)
   1228
-> 1229         self.figure.draw(renderer)
   1230         renderer.finalize()
   1231

~/anaconda3/envs/wiki/lib/python3.7/site-packages/matplotlib/artist.py in draw_wrapper(artist, renderer, *args, **kwargs)
     36                 renderer.start_filter()
     37
---> 38             return draw(artist, renderer, *args, **kwargs)
     39         finally:
     40             if artist.get_agg_filter() is not None:

~/anaconda3/envs/wiki/lib/python3.7/site-packages/matplotlib/figure.py in draw(self, renderer)
   1707             self.patch.draw(renderer)
   1708             mimage._draw_list_compositing_images(
-> 1709                 renderer, self, artists, self.suppressComposite)
   1710
   1711             renderer.close_group('figure')

~/anaconda3/envs/wiki/lib/python3.7/site-packages/matplotlib/image.py in _draw_list_compositing_images(renderer, parent, artists, suppress_composite)
    133     if not_composite or not has_images:
    134         for a in artists:
--> 135             a.draw(renderer)
    136     else:
    137         # Composite any adjacent images together

~/anaconda3/envs/wiki/lib/python3.7/site-packages/matplotlib/artist.py in draw_wrapper(artist, renderer, *args, **kwargs)
     36                 renderer.start_filter()
     37
---> 38             return draw(artist, renderer, *args, **kwargs)
     39         finally:
     40             if artist.get_agg_filter() is not None:

~/anaconda3/envs/wiki/lib/python3.7/site-packages/matplotlib/axes/_base.py in draw(self, renderer, inframe)
   2643             renderer.stop_rasterizing()
   2644
-> 2645         mimage._draw_list_compositing_images(renderer, self, artists)
   2646
   2647         renderer.close_group('axes')

~/anaconda3/envs/wiki/lib/python3.7/site-packages/matplotlib/image.py in _draw_list_compositing_images(renderer, parent, artists, suppress_composite)
    133     if not_composite or not has_images:
    134         for a in artists:
--> 135             a.draw(renderer)
    136     else:
    137         # Composite any adjacent images together

~/anaconda3/envs/wiki/lib/python3.7/site-packages/matplotlib/artist.py in draw_wrapper(artist, renderer, *args, **kwargs)
     36                 renderer.start_filter()
     37
---> 38             return draw(artist, renderer, *args, **kwargs)
     39         finally:
     40             if artist.get_agg_filter() is not None:

~/anaconda3/envs/wiki/lib/python3.7/site-packages/matplotlib/text.py in draw(self, renderer)
    712                     textrenderer.draw_tex(gc, x, y, clean_line,
    713                                           textobj._fontproperties, angle,
--> 714                                           mtext=mtext)
    715                 else:
    716                     textrenderer.draw_text(gc, x, y, clean_line,

~/anaconda3/envs/wiki/lib/python3.7/site-packages/matplotlib/backends/backend_svg.py in draw_tex(self, gc, x, y, s, prop, angle, ismath, mtext)
   1144     def draw_tex(self, gc, x, y, s, prop, angle, ismath='TeX!', mtext=None):
   1145         # docstring inherited
-> 1146         self._draw_text_as_path(gc, x, y, s, prop, angle, ismath="TeX")
   1147
   1148     def draw_text(self, gc, x, y, s, prop, angle, ismath=False, mtext=None):

~/anaconda3/envs/wiki/lib/python3.7/site-packages/matplotlib/backends/backend_svg.py in _draw_text_as_path(self, gc, x, y, s, prop, angle, ismath, mtext)
    960             if ismath == "TeX":
    961                 _glyphs = text2path.get_glyphs_tex(
--> 962                     prop, s, glyph_map=glyph_map, return_new_glyphs_only=True)
    963             else:
    964                 _glyphs = text2path.get_glyphs_mathtext(

~/anaconda3/envs/wiki/lib/python3.7/site-packages/matplotlib/textpath.py in get_glyphs_tex(self, prop, s, glyph_map, return_new_glyphs_only)
    305         # characters into strings.
    306         for x1, y1, dvifont, glyph, width in page.text:
--> 307             font, enc = self._get_ps_font_and_encoding(dvifont.texname)
    308             char_id = self._get_char_id_ps(font, glyph)
    309

~/anaconda3/envs/wiki/lib/python3.7/site-packages/matplotlib/textpath.py in _get_ps_font_and_encoding(texname)
    357             # FT_Load_Glyph/load_glyph.  (That charmap has a coverage at least
    358             # as good as, and possibly better than, the native charmaps.)
--> 359             enc = dviread._parse_enc(font_bunch.encoding)
    360         else:
    361             # If psfonts.map specifies no encoding, the indices directly

~/anaconda3/envs/wiki/lib/python3.7/site-packages/matplotlib/dviread.py in _parse_enc(path)
   1001     else:
   1002         raise ValueError(
-> 1003             "Failed to parse {} as Postscript encoding".format(path))
   1004
   1005

ValueError: Failed to parse /home/ananth/.miktex/texmfs/install/fonts/enc/dvips/libertine/lbtn_oexx6f.enc as Postscript encoding

Expected behavior
The plot renders properly inline on Jupyter Lab

Your Jupyter and/or Python environment
Please provide as much info as you readily know

Jupyter server running: Local
Extension version: 2020.5.80290
VS Code version: 1.45.1
Setting python.jediEnabled: false
Python and/or Anaconda version: 3.7.6
OS: Linux (Ubuntu):
Virtual environment: conda

Developer Tools Console Output

console.ts:137 [Extension Host] Starting WebSocket: ws://localhost:8888/api/kernels/7847d1eb-5c44-46ce-a477-5526cf43d183
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:40: Waiting for idle on (kernel): 7847d1eb-5c44-46ce-a477-5526cf43d183 -> unknown
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:40: Kernel switching to busy
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:40: Kernel switching to idle
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:40: Initial setup complete for file:///home/ananth/Research/minimum-similarity-experiments/plotting%20check.ipynb
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:40: restartKernel - initialSetup completed
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:40: Registering commtarget jupyter.widget
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:40: Kernel switching to busy
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:40: Kernel switching to idle
console.ts:137 [Extension Host] Kernel: connected (7847d1eb-5c44-46ce-a477-5526cf43d183)
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:41: Finished waiting for idle on (kernel): 7847d1eb-5c44-46ce-a477-5526cf43d183 -> idle
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:45: Executing cell NotebookImport#0
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:45: Submitting code for 3e339de9-41e0-4fe6-8ca8-6485c114e19f
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:45: Kernel switching to busy
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:45: Kernel switching to idle
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:45: Kernel switching to busy
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:46: Kernel switching to idle
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:46: Finished execution for NotebookImport#0
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:46: Finished executing cell NotebookImport#0
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:47: Executing cell NotebookImport#1
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:47: Submitting code for 3e339de9-41e0-4fe6-8ca8-6485c114e19f
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:47: Kernel switching to busy
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:47: Kernel switching to idle
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:47: Finished execution for NotebookImport#1
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:47: Finished executing cell NotebookImport#1
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:48: Executing cell NotebookImport#2
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:48: Submitting code for 3e339de9-41e0-4fe6-8ca8-6485c114e19f
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:48: Kernel switching to busy
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:50: Finished execution for NotebookImport#2
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:50: Finished executing cell NotebookImport#2
console.ts:137 [Extension Host] Info Python Extension: 2020-05-24 13:38:50: Kernel switching to idle

Microsoft Data Science for VS Code Engineering Team: @rchiodo, @IanMatthewHuff, @DavidKutu, @DonJayamanne, @greazer

@ananth1996 Thanks for reporting this. Your repro is very clear, so I suspect that we do have an issue here. But could you please tell me what you have in your conda environment for this repro? I'm running your sample code, but I don't believe that I have the right configuration to reach the error point that you are at.

Hi @IanMatthewHuff, to verify the error, I created a new Conda environment with only the basic packages, matplotlib and jupyter. I have attached the conda list for that env: conda_env.txt. The error still persists in the fresh env.
As I mentioned before, VSCode works well with the default Computer Modern of Latex, but when any font is loaded through the preamble it crashes while rendering it. I cross-checked with  r'\usepackage{comicneue}' and get a similar error.
Also, this is my first time reporting a bug, so please do let me know if you need more detailed information regarding the font caches etc to reproduce the error.

No I think I have enough to investigate on. I just didn't have a TeX install on my system, so I was getting issues with VS Code and Jupyter before getting to the preamble issue. Downloading TeX for my mac, now, but it's a big slow install so won't be able to look into this more until tomorrow.

@ananth1996 . So after just ditching the Mac and moving over to a Linux VM I was able to repro this.

We'll talk about this at our triage meeting. I will say that it looks from the matplotlibrc defaults that the support for this might be a bit funky, so I'm not sure if we'll be able to support this:
https://github.com/matplotlib/matplotlib/blob/a3d7a42d62fb0aaec58dbc906ebae909c7d410be/matplotlibrc.template#L286
Note for later investigation, error appears to be coming from here:
https://github.com/matplotlib/matplotlib/blob/a3d7a42d62fb0aaec58dbc906ebae909c7d410be/lib/matplotlib/dviread.py#L1032


tikv-ctl: make storage.block_cache.capacity dynamic changeable

What have you changed? (mandatory)
Allow the command ./tikv-ctl --host=<host:port> modify-tikv-config -m storage -n block_cache.capacity -v XXX to dynamic change block cache size if shared block cache is used. Also disallow chaning per-CF block cache size via tikv-ctl, to avoid confusion.
What are the type of the changes? (mandatory)
Improvment
How has this PR been tested? (mandatory)
start a tikv instance, load some data and warmup cache. Set cache size via the above tikv-ctl command. See cache size change in Grafana:

Note when shared block cache is used, only a single block cache size (with cf="all") is shown.
Does this PR affect documentation (docs) or release note? (mandatory)
will update tikv-ctl docs afterward.
Does this PR affect tidb-ansible update? (mandatory)
no
Refer to a related PR or issue link (optional)
shared block cache was added in #4563.
Benchmark result if necessary (optional)
Add a few positive/negative examples (optional)

Also show single block cache size in grafana when shared block cache is used (see updated summary for snapshot)

CI failure seems unrelated.

/run-all-tests

@zhangjinpeng1987 @Connor1996 CI passes. Mind take another look? Thanks.

/run-all-tests

ping @Connor1996

/run-all-tests


License?

Hi there,
I noticed there's no LICENSE file in the repo and none is set in Cargo.toml or pyproject.toml, so the licensing situation of this project is a bit unclear.
Could you clarify it?

Well there wasn't a License, we can make it Apache 2.0 so that's it's clear it's open.


DO NOT MERGE - ksr wiring POC

This commit is never intended to be merged.
Its a simple POC to try out wiring on ksr.
So far we have wiring for PluginName, Log, and PluginConfig
provided and testing for ksr.
There's more to be done.  This is a first step.
To do this, we introduce:
wiring/wiring_spi.go
wiring/wiring_impl.go
Neither of which in steady state belong here.
We also introduce:
plugin/ksr/wiring_impl.go
plugin/ksr/wiring_test.go
Signed-off-by: Ed Warnicke<EMAIL_ADDRESS>

Coverage decreased (-0.005%) to 76.542% when pulling 06c4a129d744d419567936e6ecd292e97f1fac1e on edwarnicke:wiring_ksr into 1dc0e94867b58e249d8ba0a8bfa90774f8fb9ab2 on contiv:master.


Initialisation of non-H2 database

I am trying to use Postgresql as my database engine and by correctly specifying the driver name, path and connectionstring I am able to connect to Postgresql. However, it is unable to continue because there is no database schema available. The default schema (data/initialize.sql) is only for the H2 database and it would be useful if the location of an alternative database schema could be specified as an additional command line parameter e.g dbSchemaFile.

While I can add this, my original intent was that if someone were going to use a more robust DB engine they would just create the database, create the user, modify the schema file as appropriate for their database and then execute dependency-check to update the db.
I always saw the use of a more robust database as a way to only have one server on the network need to connect out to the NVD to download the NVD CVE data. You could put dependency-check on a server that could connect to the Internet and use that to update the internal database. Then other instances of dependency-check could be running internally to perform scans. However, to increase the accuracy of dependency-check there are many times that Central is queried for information about the given JAR file. If most of your dependency-check scans cannot connect to Central you may have additional false negatives. This can be solved by using an internal Nexus server (or other repo server AFTER an analyzer has been built to query it).

Documentation pertaining to using a centralized database server has been added. For now see database.md. This will be added to the documentation site in the next release.


This doesn't support disabling tabs?

@akashdeep-singh Currently no, but that can be easily added 😄

@cristijora I'll try adding the functionality and PR if I get soemwhere

Don't worry. I almost finished it.

See this fiddle as an example
https://jsfiddle.net/b44cc4dq/386/
There is a disabled prop per tab
and 2 options to configure the colors for disabled tabs
 disabled-color="lightgrey"
 disabled-text-color="grey"

You can also do that via css if you want but you have to cover the hover case also.
If this is fine, I will publish to npm a new version

@cristijora Thanks! Could you publish to npm as well?

It's up on npm 0.5.4

It doesn't seem to be working on the jsfiddle you shared. Could you verify?

Maybe hard refresh? Do  you get something like this?

Yep, that worked. Thanks


Mast: Watch (?-i:^Melda Research$)

Mast requests the watch of the watch_keyword (?-i:^Melda Research$). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

(?-i:^Melda Research$) has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


Remove unimplemented and skipped specs

Context
These specs will not be implemented as we have moved away from a shared pages controller.
Changes proposed in this pull request
Remove some tests 💣
Guidance to review

Link to Trello card
https://trello.com/c/R6nEK8Hm/96-remove-skipped-specs
Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[ ] API release notes have been updated if necessary
[ ] If it adds a significant user-facing change, is it documented in the CHANGELOG?
[ ] Required environment variables have been updated or added to the Azure KeyVault

 Remove skipped specs


chore: update plunker template in readme.md

Along with @DevVersion's update to the plunker, fixes #7676

@josephperrott Plunker should be updated now.

Closing as plunker links have been updated.


feat: allow customizing the link fields

Description
This PR aims to support customizing the fields exposed and shown for Rich Text links (see #2167). I have implemented this suggestion (I think).

[x] I have read and understand the CONTRIBUTING.md document in this repository

Type of change

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

Checklist:

[x] I have added tests that prove my fix is effective or that my feature works
[x] Existing test suite passes locally with my changes
[x] I have made corresponding changes to the documentation

That sounds reasonable, done!

Hey there, thanks a lot for the approach. This is excactly what I was looking for. The only thing is, that my added custom fields are not getting saved anymore. For example, I want to replace the url field with custom one, that is validating the input for https/http. Moreover I'd like to extend the fields with a link-style-field:
 {       name: 'richTextContent',        type: 'richText',       required: true,       localized: true,       admin: {         link: {           fields: ({defaultFields}) => {             const theDefaultFields = defaultFields.filter((field) => (field as any).name != 'url')             const urlField = defaultFields.find((field) => (field as any).name === 'url')             return [               ...theDefaultFields,               {                 name: 'url',                 type: 'text',                 required: true,                 admin: {condition: urlField?.admin?.condition, placeholder: 'https://'},                 validate: (val, options) => {                   const errorText = options?.t('fieldValidation:invalidUrl')                     ? options.t('fieldValidation:invalidUrl')                     : 'Error'                   const valid = /^(http|https):\/\/[^ "]+$/.test(val)                   if (valid) {                     return true                   }                   return errorText                 },               },               {                 name: 'style',                 label: 'Link Style',                 type: 'select',                 options: ['button', 'icon-link'],               },             ]           },         },       },     }
Every Field will be shown correctly:

The only problem I have now is that as soon as I submit the link, my "link style" won't be saved. I tried it with other field types and field names - it's just not working. The new url field is working perfectly (guess because of the same name as the field before: "url") as well as the other default fields. Guess there is a problem with the payload state?


Multiple controllers in a single operator?

Type of question
Feature/implementation
Question
What did you do?
I have a single operator that has four different controllers. Once of the controllers is an abstraction that watches a single Custom Resource and creates three other Custom Resources, each of which is managed by a respective controller.
This seems to work just fine when the main abstraction CR is created. However, if I update a field in the main abstraction CR that does not result in a change to any of the child CRs, such as moving a value to  a new field thats replacing a deprecated one, I'm still seeing a Generation change in a child CR and a related reconcile trigger on the child CR controller.
What did you expect to see?
A change to the main CR that results in no changes to child/owned CRs results in a main controller reconcile trigger, but no triggering of child CR controllers.
What did you see instead? Under which circumstances?
I've added extensive logging and object dumps throughout the main abstraction controller to verify it never changes any of the child CRs and I've tried bypassing the caching client with an apiReader in the child CR controller to remove the cache from the picture.
After end of the main controller reconciliation loop, which makes no changes to owned CRs (and returns with reconcile.Result{}, nil), an unexpected reconciliation occurs in a controller for an owned resource and the Generation of the owned resource has increased from what it was in the object dumps in the main controller loop.
Environment

operator-sdk version:
operator-sdk version: "v0.17.1", commit: "6d108056f39fa19546cf9235fad6e84b0683114a", kubernetes version: "unknown", go version: "go1.13.10 linux/amd64"

Kubernetes version information:

Client Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.8", GitCommit:"ec6eb119b81be488b030e849b9e64fda4caaf33c", GitTreeState:"clean", BuildDate:"2020-03-12T21:00:06Z", GoVersion:"go1.13.8", Compiler:"gc", Platform:"linux/amd64"} Server Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.8", GitCommit:"ec6eb119b81be488b030e849b9e64fda4caaf33c", GitTreeState:"clean", BuildDate:"2020-03-12T20:52:22Z", GoVersion:"go1.13.8", Compiler:"gc", Platform:"linux/amd64"}

Kubernetes cluster kind:

vanilla k8s
Additional context
Add any other context about the question here.

Hi @walkamongus,
Let's see if I understood your scenario. You have a controller that manages a KIND. Then, due the reconciliation process, you are creating new CR's of the same KIND.

What did you expect to see?
A change to the main CR that results in no changes to child/owned CRs results in a main controller reconcile trigger, but no triggering of child CR controllers.

If you have a controller which was created to manage of a KIND it means that by default, you will find the watches for the Primary resources already implemented by each controller that you generated by using the SDK tool as follows.
// Watch for changes to primary resource Memcached
err = c.Watch(&source.Kind{Type: &cachev1alpha1.Memcached{}}, &handler.EnqueueRequestForObject{})
if err != nil {
    return err
}

So, if you create/update/delete a CR of the KIND that is managed by it (in the example Memcached ) then the reconcile will be re-trigged.
How works the Reconcile()?
The reconcile function is responsible for synchronizing the resources and their specifications according to the business logic implemented on them. In this way, it works like a loop, and it does not stop until all conditionals match its implementation. The following is pseudo-code with an example that clarifies it.
reconcile App {

   // Check if a Deployment for the app exists, if not create one
   // If has an error, then go to the beginning of the reconcile
   if err != nil {
       return reconcile.Result{}, err
   }
   ...

   // If it is at the end of the loop, then:
   // All was done successfully and the reconcile can stop
   return reconcile.Result{}, nil

}

Using the Watch to reconcile
By default, you will find the watches for the Primary resources already implemented by each controller that you generated by using the SDK tool as follows.
// Watch for changes to primary resource Memcached
err = c.Watch(&source.Kind{Type: &cachev1alpha1.Memcached{}}, &handler.EnqueueRequestForObject{})
if err != nil {
    return err
}

However, note that in the Mencached sample has watches implemented for each secondary object controlled/created by it.
https://github.com/operator-framework/operator-sdk-samples/blob/master/go/memcached-operator/pkg/controller/memcached/memcached_controller.go#L52-L70
Please, let us know if the above information clarifies how it works and if we can close this one. If not, please could you try to clarify better What did you do?, What did you expect to see?, What did you see instead? Under which circumstances? for we try to help you with. Maybe a snippet of the codes can help too.
Looking for your reply.

Hi @walkamongus,
Could you please let us know if still anything else that should be addressed here or if the information provided is enough already for you solve your case and we can close this one?

@camilamacedo86 thank you for the information. It turns out everything works exactly as expected. I ended up having a bad k8s master/etcd member that was causing my operator to spin out of control from resourceVersions jumping around.

hi @walkamongus,
really tks for letting us know.


Handle Redirects 301/302

Do you want to request a feature or report a bug?
Hello. I don't know if this is a feature or a bug.
I would say a bug.
What did you do?
I'm trying to reverse proxy a VMware Virtual Center appliance.
  http:
    routers:
      to-vmwarevc:
        entryPoints:
        - websecure
        rule: "Host(`vmwarevc.hostname.com`)"
        tls: {}
        service: vmwarevc-service
    services:
      vmwarevc-service:
        loadBalancer:
          passHostHeader:
          servers:
          - url: https://internalvc.domain.local

What did you expect to see?
After landing on the initial page, i choose for the html client.
after that i get redirected to https://vmwarevc.hostname.com/ui/login
from there i get a 302 redirect to https://vmwarevc. hostname.com /websso/SAML2/SSO/vsphere.local?SAMLRequest=zVRdb9owFH3fr4j(somereallylonghash)
What did you see instead?
I get redirected to https://internalvc. domain.local /websso/SAML2/SSO/vsphere.local?SAMLRequest=zVRdb9owFH3fr4j(somereallylonghash) instead of the proxied URL
Request header
Host: vmwarevc.hostname.com
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:68.9) Gecko/20100101 Goanna/4.4 Firefox/68.9 PaleMoon/28.8.0
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8
Accept-Language: en-US,en;q=0.5
Accept-Encoding: gzip, deflate, br
Referer: https://vmwarevc.hostname.com/ui/
Cookie: <redacted>
Connection: keep-alive

Response Header
HTTP/2.0 302 Found
X-Firefox-Spdy: h2
cache-control: no-store
content-length: 0
date: Tue, 31 Dec 2019 08:28:47 GMT
location: https://virtualcenter.domain.local/websso/SAML2/SSO/vsphere.local?SAMLRequest=zVRdb9owFH3fr4j(somereallylonghash)
pragma: no-cache
server: Anonymous
strict-transport-security: max-age=30758400; includeSubDomains
x-frame-options: deny

Output of traefik version: (What version of Traefik are you using?)
Version:      2.1.1
Codename:     cantal
Go version:   go1.13.5
Built:        2019-12-12T19:01:37Z
OS/Arch:      linux/amd64

Hi! I'm Træfiker :robot: the bot in charge of communication regulation.
Thanks for your interest in Traefik!
We dedicate the issue tracker to bug reports and feature requests only. My advanced AI has spotted that your issue might be a configuration problem or relates to something that doesn't look like a bug.
To confirm this, please join our Community Forum and reach out to us on the Traefik section.
In case I'm wrong (well, that would be embarrassing :sweat_smile:), my developers will re-open the issue and fix me!
In the meantime, you can double check Traefik's documentation.


Added Time line Slider

the time line slider is added from 1000 to 2000

can you pull the recent changes and then commit


socket reconnect?

server i am sending my rtmpt requests to (nginx) has a request count limit after limit is exceeded http keep-alive connection is being dropped. lf.swift treats it as connection was closed and stops streaming.
can you point me to the best place to look into it?
@shogo4405

Retrying failed request helps: https://github.com/shogo4405/lf.swift/pull/258
Not sure if it's the best solution though

How can I test it? I prepare nginx-rtmp-module(https://github.com/arut/nginx-rtmp-module)?

@shogo4405 should be reproducible with the max requests param enabled. We are using default value for http2_max_requests 1000;


Backport mattermost-desktop 5.5.1 -> 5.7.0 to release 23.11

Description of changes
Backport mattermost-desktop update to 5.7.0 with electron update to 28.x due electron 26 EOL
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

@ofborg build mattermost-desktop


Connections hang in CLOSED state

We have faced an issue when connections (ports and processes owning them) hang forever (we use these connections for infinite streams of events).
...
4071 inet_tcp<PHONE_NUMBER>7   <0.15260.394>                   *:*                    CLOSED       STREAM
4072 inet_tcp<PHONE_NUMBER>59  <0.17014.480>                   *:*                    CLOSED       STREAM
4078 inet_tcp<PHONE_NUMBER>09  <0.6376.600>                    *:*                    CLOSED       STREAM
4084 inet_tcp<PHONE_NUMBER>8   <0.22290.881>                   *:*                    CLOSED       STREAM
4087 inet_tcp<PHONE_NUMBER>32  <0.2095.695>                    *:*                    CLOSED       STREAM
4095 inet_tcp<PHONE_NUMBER>43  <0.23210.849>                   *:*                    CLOSED       STREAM

Our investigation showed it happens due to combination of configuration, error handling from cowboy side and inet_drv behavior. The interesting thing here is when send_timeout_close configuration parameter is used and inet_drv closes a connection due to timeout it does not send tcp_closed or tcp_error message to a port owner. The only thing we can do in this case is always check a return value of Transport:send/2.
UPD: I've made a test for this case in #1590

@essen could you react on this please?
We faced this issue too. Long running connections (MPEG-TS over HTTP in our case) tend to leak.
This leads to a growing number of useless processes spending CPU and memory.
On this picture there is a significant leak (about 5K processes which is about 1.5K cowboy_loop workers per day) on a quite low load (max 100 concurrent connections).

What I don't understand is why the idle/inactivity timeouts don't help.
Anyway have you tried the patch in the PR, does it fix your case?

why the idle/inactivity timeouts don't help

I should have mentioned, sorry for that.
These are long-living responses. Client connects and receives data as it appear, and it may last for hours (e.g. internet radio via shoutcast or SSE), without any data sent by a client. Dowloading 2-gigabyte file over slow 3G link is also affected.
Meaningful idle_timeout kills these connections because there is no client activity in the socket, leading to loss of data in perfect network conditions.
So, we have to disable it explicitly with cowboy_req:cast({set_options, #{idle_timeout => infinity}}, Req); for these long requests.

have you tried the patch in the PR

Not yet. Will check it soon.
I've traced ssl:send/2 -> {error,enotconn}, so the patch seems to address exactly our case.

Yes I've merged a similar patch for Gun, didn't think there was users disabling these timeouts so didn't follow up here (priorities and all that). But sounds like it'd be useful for the next release.

Looking forward for the release. Thank you!

This will now work properly on current master. I made a slightly different fix so you may want to test again before I release the new version. Closing, thanks!


[multistage] global index virtualID

first step for #10657

Codecov Report

Merging #10665 (79e45c6) into master (4481e1b) will decrease coverage by 56.48%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #10665       +/-   ##
=============================================
- Coverage     70.34%   13.86%   -56.48%
+ Complexity     6517      439     -6078
=============================================
  Files          2108     2054       -54
  Lines        113664   111178     -2486
  Branches      17126    16828      -298
=============================================
- Hits          79954    15418    -64536
- Misses        28110    94497    +66387
+ Partials       5600     1263     -4337

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
?

unittests2
13.86% <0.00%> (+0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

.../org/apache/pinot/query/routing/WorkerManager.java
0.00% <0.00%> (-86.87%)
:arrow_down:

...va/org/apache/pinot/query/runtime/QueryRunner.java
0.00% <0.00%> (-81.52%)
:arrow_down:

... and 1661 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Odd cleanups

Neat little app here, might look at trying a gtk4 port when the new machine finally arrives in a month...
....oh
My old name is everywhere in software I have written: its in config paths, its in ID's that can't be changed, its in URL's that aren't redirected. Kinda reduces my motivation to keep working on it 😕— 💜 Zoey 🌹🦀🏳🌈🏳⚧ (@zoeyfyi) November 28, 2020
well

Steve Austin, astronaut. A man barely alive. Gentlemen, we can rebuild him. We have the technology. We have the capability to build the world's first bionic man. Steve Austin will be that man. Better than he was before. Better, stronger, faster.

...If/when you are happy and merge this @bilelmoussaoui and I can help get the flathub EOL/redirect working
:crossed_fingers: WiX & snap Just Work:tm:, though I really don't know much about them
Also messed around with the about dialogue a tad, not seen anyone defining one in GtkBuilder for a while. You looked at GAction? It's a bit nicer than connecting all the signals by hand
:christmas_tree: Merry Christmas from the folk at GNOME
Zander, down in Tier 4
PS: tex-match.zoey.fyi seems broken

Neat little app here, might look at trying a gtk4 port when the new machine finally arrives in a month...

I haven't checked out gtk4-rs yet but I imagine this would be relatively easy except for the windows build since gvsbuild doesn't provide gtk4 (yet?).

...If/when you are happy and merge this @bilelmoussaoui and I can help get the flathub EOL/redirect working

Wow, that would be great! Would you mind doing the same with Boop-GTK?

PS: tex-match.zoey.fyi seems broken

Fixed :)
Thanks so much, and merry christmas! :christmas_tree:

Wow, that would be great! Would you mind doing the same with Boop-GTK?

Already working on it, unfortunately my poor little Pentium isn't enjoying building it much so may take a day or so


MSID: 92618 Version: 1 DOI: 2<IP_ADDRESS>1233

[PLACE MANUSCRIPT AND EDITOR DETAILS HERE WHEN AVAILABLE] See step 3
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=92618)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/previews/92618v1)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews and QC
Who can help: Production team

[ ] QC passed or QC not needed

Who can help: Editorial team

[x] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>1233)
[x] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[ ] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/92618/v1/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/92618)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/92618

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/92618

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/92618/v1/92618-v1.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/92618)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

Reviews not available yet.


preparing for 1.4.0 release stuff

tested building / using now and looks good with these changes

there was an extra yarn.lock in the km project that I deleted, it's not needed


Create Github VCR

Script to create Github VCR
To raise a new Pull Request, the following prerequisites need to be met. Please tick before proceeding:

[ ] a conversation was held in the appropriate DRMAN Slack channel.
[ ] a Github Issue was opened for this feature / bug.
[ ] test coverage was added (Cucumber or Spock as appropriate).

This addresses Issue #XXX

Please submit a fresh pull request  after integrating the code in https://github.com/DIDman/DRman/blob/master/src/main/bash/drman-main.sh


Graphics based masks are broken

package;

import openfl.display.Sprite;

class Main extends Sprite
{
	var ringC:Sprite;
	var ringM:Sprite;
	var ring:Sprite;

	public function new()
	{
		super();

		// Ring
		ringC = new Sprite();
		ring = new Sprite();
		ring.graphics.beginFill(0xFF700000);
		ring.graphics.drawCircle(0, 0, 150);
		ring.graphics.endFill();

		ringM = new Sprite();
		ringM.graphics.beginFill(0xFF000000);
		ringM.graphics.drawRect(-150, -150, 300, 300);
		ringM.graphics.beginFill(0xFFFFFFFF);
		ringM.graphics.drawCircle(0, 0, 120);

		ring.mask = ringM;
		ringC.addChild(ring);
		ringC.addChild(ringM);

		addChild(ringC);
		ringC.x = stage.stageWidth / 2;
		ringC.y = stage.stageHeight / 2;
	}
}

Mask is not rendered, but it has no effect.

This behaves the same in both Flash and other OpenFL targets for me. Seems okay for now


ui bugs at the moment(?)

i'd like to contribute but i'm not really sure where this is right now. here's the experience i'm getting:

after a fresh download of the git repo, after npm start the first thing i see is "no addons were found in /User/addons". but i would expect it to use ../../addons or similar on first startup.
i go to settings and change the OF root manually
i go to create/update and open a project with no addons. the addons field still says "No addons found, is OF path right?" even though i updated it already.
i switch to an example with an addon: example-simplePoints inside ofxDelaunay. now the addons field updates to show the addon.
i click "update project" but it gives me an error: "please select a platform first". it just says "Unknown" and i can't change it.
i go to settings and turn on advanced settings.
i come back and select "OS X" and hit "Update Project"
i get an error:

Command failed: /bin/sh -c /Users/kyle/Documents/openFrameworks/pgElectron/app/commandLinePG  -o"/Users/kyle/Documents/openFrameworks" -a"ofxDelaunay" -x"osx" "/Users/kyle/Documents/openFrameworks/addons/ofxDelaunay/example-simplePoints"
/bin/sh: /Users/kyle/Documents/openFrameworks/pgElectron/app/commandLinePG: Permission denied

i open terminal and cd to pgElectron/app/ and chmod +x commandLinePG
i do it again and it says "success". but when i look at the directory, there is no project file. from the i don't see any error messages in terminal.

one issue I see is that settings.json is in the gitignore, so I think the current settings.json isn't totally up to date (and it's hard for me to see it, since my settings.json is right).  I've just pushed my settings so that you don't get undefined for platform, but you'll obviously need to change the path.
also the PG that's in this repo seem out of date too (it's likely that @Daandelange updated the commands to the PG but not the projectGenerator app, which probably shouldn't be in this repo), so it's not happy with the parameters that are sent to it.  It also looks like our system needs some error checking, like looking if a folder is actually there.  (it's success right now is just if it ran the command line tool which seems wrong -- here, it's saying it succeeded when it didn't)
please keep in mind that I am doing a UI rewrite right now in another branch (which is fairly unusable at the moment but should be up v. soon) -- my head is in that and not in this at the moment so there are alot of visual / ui things but I think the next version should make things much cleaner.

(as a side note, if you run the command line PG and it says "no mode selected... please use create or update" that's a good sign that it's an older app, arturo and I worked through a bunch of changes to the command line to streamline the parameters, etc)

@ofTheo: Yes, I updated the cmdLinePG to the latest commit and indeed; as it's platform specific I didn't include the binary. (I should have removed the original one maybe).
@kylemcdonald You can just build it, rename it and place it in /app/app/commandLinePG

"No addons found, is OF path right?" even though i updated it already
This is an UI bug indeed. If you start typing the addons appear. I created #28 for this.

But it's weird that is says success while it's not been passed the right commands. Maybe the (new) command succeeds with the old cmdLinePG (but does something else as there are no more -c or -u flags) ?
Oh and I see /bin/sh -c in your command... which means you don't have the latest commit as it removes the -c flag.
As for platform Unknown, it reads it from settings.json which needs to be platform specific. When there's none it writes the settings from index.js (top) and platform is just set to "Unknown".
@ofTheo said there will be a platform-specific settings.json so we don't have to care about it yet.
For the moment you can open settings.json and set the platform variable to osx.
But yeah, to sum up it's still in a messy state but thanks for reporting your experience. :)
Should go better soon when Zach's semantic-ui branch gets merged.

I think it's fine to put in a platform specific json file for now, we are just testing now, it will help.  I pushed my json file if it's useful.

https://github.com/ofZach/pgElectron/commit/340b7a28d713fdbeea7c35d74b57a8502ffb81b4 I added a feature to detect the platforms too, only IOS remains to be detected. (that could be detected client-side (from the User-Agent string), or maybe server-side using the CPU info ?)

Cool! I'll keep watching for updates and testing. Daan, I don't think
platform detection will work for iOS because people will not run the
electron app on their iOS device, but from OSX.
On Wednesday, August 26, 2015, Daan de Lange<EMAIL_ADDRESS>wrote:

340b7a2
https://github.com/ofZach/pgElectron/commit/340b7a28d713fdbeea7c35d74b57a8502ffb81b4
I added a feature to detect the platforms too, only IOS remains to be
detected. (that could be detected client-side (from the User-Agent string),
or maybe server-side using the CPU info ?)
—
Reply to this email directly or view it on GitHub
https://github.com/ofZach/pgElectron/issues/27#issuecomment-134993076.

cool - the semantic-ui PR is more functional now (but not really working yet...) if you want to track that (I'm basically re-implementing alot of the stuff that was built on boostrap).  I think I am going to become a web designer now.   Jsfiddling my way to a working gui.

semantic ui is fairly clean and because we were merging two things before (select2 and bootstrap) the styling was wonky.  This feels more professional.  There's some some stuff to be done to make the chromeless window work well (it needs a close button, some info and dragging ability on the top bar) but it looks pretty cool to now have the window chrome.

Looks very nice! :+1:
I love the new window format!
Ping me if/when you need help getting js stuff to work again! :)

I think it's pretty straight forward, I just am moving more of the logic to JS and away from CSS (ie to show / hide elements) and semantic-ui is a little wonky in how it interacts with jquery (I find myself getting tripped up a little).  I have most buttons / fields hooked up now and the fields have data that is changing :) I will dig deeper later today on the overlays and some more state stuff / bugs.
the moniker stuff is pretty hilarious.

Hmm, well I did the CSS/hiding on purpose so that with JS you handle (or indicate) the app's state then the CSS handles all the rest depending on the app's state. This way you have cleaner code because all the appearance stuff is moved out of the JS files.
But I'll leave it up to you if you think it's easier the JS way. :)
PS: Yes it is ;D

ok, looks very nice. going to hold off on further testing/comments/tweaks till you think it's ready.

it's not 100% done but the semantic-ui PR branch can now do single project generation and updating (I need to hook up multiple / recursive updates and the modal dialogs are missing now, working on getting those back).   def worth taking a look at now, now that it's more functional....  more updates shortly.

Modal dialogs: :white_check_mark: eedeed6
I also discovered a better search-term-matchin-function for the addons dropdown: fcf35b2

thanks! I was starting to look at this for form validation http://semantic-ui.com/behaviors/form.html might be a little less intrusive then the modals, but it's good to have them back :)


Added remaining Monochrome / Ozone icons

Lots of new Monochrome / Ozone icons, in SVG and PNG (compressed using Pinga).
Combined with #432, this should complete the to-do list in #433.
Thumbnail of all icons (click for full size):

Below are the icons that use logos (a concern raised in #432): A decision needs to be made if these need to be changed.

A few of the '-content' icons might be a bit small and need tweaking (such as 'Mobile-PalmOS-content').

For now those look fine.
One small concern I have - all these files that are added with this PR, how many MBs does that add up to?

#435 (this commit) = 2.15 MB (2,259,328 bytes)
#432 = 0.5 MB (517,388 bytes)
retroarch-assets (2022-07-24, ozone & monochrome, png & src (not inc. ttf)) = 3.84 MB (3,842,521 bytes)
Above 3 combined = 6.62 MB (6,619,237 bytes)
...
retroarch-assets (complete repo 2022-07-24) = 613 MB (643,236,521 bytes)

@LibretroAdmin We'd like to tackle the MB bloat issue caused by retroarch-assets through moving to a Unified Theme format that will benefit all the UXs.  Part of that proposal is that SVG sources be required, and the PNG files in any needed size would be created by batch script.  The project is on the to-do list for @jdgleaver.

jdgleaver is on indefinite hiatus / retired, so we probably shouldn't wait on him :P

@hizzlekizzle That's good to know.  That means we'll need some help getting it implemented as outlined in #345.  It's a worthy cause that would greatly benefit the main project.


Improvement: add /readiness endpoint for Kubernetes hosting

Summary
To run Kong in Kubernetes a /readiness endpoint will tell Kubernetes whether a node is ready to receive traffic and make it part of a service by returning 200 OK. Failing to return 200 OK will - temporarily - remove the node from the service.
Combined with the /liveness endpoint - see https://github.com/Mashape/kong/issues/1677 - this makes running Kong in Kubernetes much more effective.

+1
For sure would be very usefull for the integration of Kong into Kubernetes, for a production grade level.

 swarm integration

@JorritSalverda Kong already comes with a kong health command to check its health, this can be used as a readiness/liveness probe:
        readinessProbe:
          exec:
            command:
            - kong
            - health
          initialDelaySeconds: 3

Kubernetes Ingress requires the readinessProbe to be an httpGet (without autentication), and I'm having a hard time finding an endpoint on the proxy port that returns http status 200 without calling another API...

I was able to solve my problem by adding a dummy API with a non-existent upstream uri and then using the request-termination plugin to always return status 200.

I would like to have a proper HTTP health check for Kong too. In our situation, we have Kong fronted by an AWS ELB which executes HTTP requests for health checking.
As I have the global http-log plugin configured,  @filotti's solution causes all the health checks (6 per minute per ELB instance per Kong instance to be logged.
the /liveness can be easily added to the kong template.
the /readiness would ready to determine the health state of Kong.


feat(nexus-shutdown): adding changes for nexus shutdown api call

This PR includes implementation to call io-engine nexus shutdown api from control plane.
Signed-off-by: Abhilash Shetty<EMAIL_ADDRESS>

bors try


Error code 404

I'm getting "Error code 404" when I enter my NVD API key.

If I remove the API key, everything works like a charm!

I have just realized that this glitch was due to my API key.

Hello @TURROKS @pjhavariotis
I have the same issue:

without API Key no error;

with NIST API Key 404 error.

I tried with different API Keys, same issue.
What could I check for?
Thank you


Custom error handler never called

Hello,
I am currently having trouble defining a simple custom error handler for some simple project. Here is my code:
var express = require('express');
var app = express();

app.get('/', function (req, res) {
  res.send('Hello World!');
});

app.use(function(err, req, res, next) {
  console.log('test');
  res.status(500).send("custom error handler called");
});

var server = app.listen(3000, function () {
  console.log('Example app started');
});

I am trying to trigger my handler by requesting '/toto' which isn't supported by my server but express default error handler seems to be always called instead of mine. Am I missing some point here?

Hi! So it looks like there is just some confusion, that's all :) So It sounds like you are trying to get res.status(500).send("custom error handler called"); called when requesting a route that does not exist (i.e. a 404). The condition of a route not existing is not actually considered an error by Express, which is why your code is not called when you request /toto in your example. What you want to do to catch 404 conditions is add a normal middleware as the last thing on your stack, which is effectively a "catch-all" handler and would get the requests that would end up being a 404. Here is an example:
var express = require('express');
var app = express();

app.get('/', function (req, res) {
  res.send('Hello World!');
});

app.use(function(err, req, res, next) {
  console.log('test');
  res.status(500).send("custom error handler called");
});

app.use(function(req, res, next) {
  console.log('test');
  res.status(404).send("custom not found handler called");
});

var server = app.listen(3000, function () {
  console.log('Example app started');
});

+1
I agree with author. Error handler is never called!
 So your example is expected to not work; the way Express works is that your handlers are always called top to bottom.  For your example to work, you need to swap the order:
var
    ex = require( 'express' ),
    app = ex();
app.all( '*', function( req, res ){
    throw 'TestError';
});
app.use(function(err, req, res, next){
    console.error( err.stack);
    res.status(500).send('Something broke!');
});
app.listen( 4000 );

If you find the information on the http://expressjs.com site lacking in this regard, please feel free to open an issue over at https://github.com/strongloop/expressjs.com :) !


Add ArchUnit - a Java architecture test framework

Found it in the ThoughtWorks technology radar 2018 tool section: https://www.thoughtworks.com/radar/tools/archunit.

Cool! Thanks. 🛋


Last plugin release date tracker probe

Description
We need to add a new probe that helps track the release status of the plugin, and how frequently the plugin is released.
Importance:

Help the Stakeholders to know How long it will take as on average to resolve their issues.
Time will the difference between the release of the plugin

I don't really think this would produce reliable data because:

a new maintainer can start to work on a previously abandonned plugin and the average of that plugin would to 10 times the reality
and the opposite is also true when a maintainer has not enough time to work on a plugin.

This is something we already introduced with https://github.com/jenkins-infra/plugin-health-scoring/blob/7b9a8d6a2453c53de5c0d4f00a7affeb38db6440/core/src/main/java/io/jenkins/pluginhealth/scoring/scores/AdoptionScoring.java#L96-L124 and it can be gamified (which is bad) or largely invalid.


exclusion of methods for next move and compiling with numba

To improve the runtime of the code we should try to pass the coordinates of the agents into an array so that we can exclude the methods to update the next step of the agents from the class agent. That way we could compile it with numba.

I have uploaded the code as far as I have come.
I have tried to extract all of the updating for the pheromones and agent moves out of the classes
that part is not done yet
also it is important that the code right now can only used within the simulation code and not within the GUI yet.
next steps would be:

check if values are given correctly to the arrays
check if calculations until line 107 are correct or if sth doesn't makes sense
is there an easier way to write the code until 107?
there are things that are not implemented yet: the max_value has to be found for each line in the array pheromone_values
then those indizes can be used to find the correct colum for each line that has the values for updating the original agent_array
then with this updated agent_array the pheromone array can be updated

branch #29_numba_prepared will not be used because most of the calculations are incorrect there will be another version that is also connected to Version2.0 Issue.

The first step is almost don, all the Method are outside of classes now and also most importet functions from already compiled libararys are in extra functions too, so that there should be no problems when trying to implement Numba


[cluster_test] Add start/stop container command

This diff add commands to stop and start containers for given validators:
cluster_test --stop [--peers hash1,hash2,...]
cluster_test --start [--peers hash1,hash2,...]

Unlike reboot/restart, --stop keeps container stopped, until --start is applied.
This is useful in multiple scenarios:

It helps to add stability to deployment process. We need to wipe db during deploy to protect against incompatibilities in code. However, wiping db when validators are running is very error prone. So new deployment process is this:
(a) stop containers
(b) wipe database
(c) update ECS task definitions
(d) start containers

It can be used to measure performance when 1,2,.. validators are down. Reboot effect can not be reliably used for this, because it brings validator back quite fast, we might not be able to reliably measure throughput in this short window

@bors-libra r+

:pushpin: Commit eb86b92 has been approved by dmitri-perelman

:hourglass: Testing commit eb86b928aff0545cffedbbaf45422ab5c3b0c521 with merge 182570f930153dda721e8f50e3f7aa560f5c9340...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: dmitri-perelman
Pushing 182570f930153dda721e8f50e3f7aa560f5c9340 to master...


Not supporting header containing plain language-country only, e.g., en-US

Hi,
if you switch language on an iPad (via Settings / General / Language & Region / iPad Language), Safari might not send the usual language-country + language combination (e.g., en-US,en) but a plain language-country only (e.g., en-US) without language fallback. Unfortunately, this is not supported by the accept language parser.
Expected behavior:
var parser = require("accept-language-parser"); // version 1.4.1
var language = parser.pick(['en', 'fr'], 'en-US'); // 'en'

Actual behavior:
var parser = require("accept-language-parser"); // version 1.4.1
var language = parser.pick(['en', 'fr'], 'en-US'); // null

Not sure if this should be supported.
Thank you.

In my use case I need to pick one of non-prefixed languages given prefixed accept-language. There should be at least an option to have this behavior :)


can it compile to 64bit dll?

Hello there,
thank you for creating such cool tool, I retargeted the project to compile for 64bit arch and I also self-compiled all the dependencies. Although the compilation succeeded and also can it connect to servers, but when I delete the pointers I created, weird errors happened.
I also retargeted to Windows 10.x API as VS2017 suggested.
delete this->dispatcher;
delete this->mc_client;

 	ntdll.dll!RtlReportCriticalFailure()	Unknown
 	ntdll.dll!RtlpHeapHandleError()	Unknown
 	ntdll.dll!RtlpHpHeapHandleError()	Unknown
 	ntdll.dll!RtlpLogHeapFailure()	Unknown
 	ntdll.dll!RtlFreeHeap()	Unknown
 	ucrtbase.dll!00007ffcab75d92b()	Unknown
 	mclib.dll!mc::core::Client::`vector deleting destructor'(unsigned int)	C++
>	Craft.dll!MCClient::Stop() Line 34	C++
 	[External Code]

Google said it might be a heap corruption.
Should I switch back to 32-bit or it was caused by my code?

I've never actually compiled it with 64-bit target, so I'm not sure if it works. I see that you're deleting the dispatcher before the client, so that could be the problem. A lot of the members of client have to unregister handlers in each of their destructors, so deleting the dispatcher first would definitely mess that up.
I'll try to compile it in 64-bit myself sometime when I have some free time to see if there's any issues.

ok thx soooo much! :D
I also tried not to delete dispatcher, the error also happened.

Yeah, it works absolutely fine as a 64-bit build. I tested it with the example client and with my larger Terracotta project.
I don't have Windows 10 to test if something goes wrong with retargeting the build. I'd say the best way to build it would be to use CMake to generate a 64-bit build instead of using the supplied solution.
If that doesn't change anything, then it's most likely an issue with your object destruction order.

okay thanks!
I'm gonna try to rebuild it. ;)
I think my project configuration might be wrong.


Add support for temporary credentials in AmazonSES

In case you use IAM role for authenticating AWS requests you can fetch
temporary access_key and secret_key from that role, but you also need to
include additional X-Amz-Security-Token header to that request.
Docs: (Http request signature)[https://docs.aws.amazon.com/general/latest/gr/sigv4-add-signature-to-request.html]

Looks fine to me. You can use your branch to test it out if it works correctly.

Closing PR, but using fork until 2 PR's merged:
https://github.com/swoosh/swoosh/pull/518
https://github.com/swoosh/swoosh/pull/519

FYI https://github.com/swoosh/swoosh/pull/518#issuecomment-704616717


Build error

When I run the command npm run watch or grunt server. it will give me the warning. Please see the warning in below screenshot.

Can anyone please help me to solve this?

Probably one of the file is missing or misnamed, try starting from a fresh clone of the repo and run grunt server on it.

Hi @limzykenneth  actually I find that .hbs file data lost. okay I will try to clone the repo.
Thanks


[Mastodon/Pawoo] Error while trying to login

Bug description
Logging in via OAuth2 produces an unknown client error. I might've logged in wrong though since I haven't added a new site in a long time and don't remember anything about OAuth2. You put your regular login information into the fields, and then Grabber gets the keys when it opens the website to authenticate it? Or am I supposed to get the consumer key/secrets somehow?
Client authentication failed due to unknown client, no client authentication included, or unsupported authentication method.
Steps to reproduce

Enter email and password into OAuth2, hit Test
Pawoo opens up, shows that I'm signed in, but rejects the authentication.

System information
Windows 7, Grabber 7.9.1.

You put your regular login information into the fields, and then Grabber gets the keys when it opens the website to authenticate it? Or am I supposed to get the consumer key/secrets somehow?

Yes exactly 😄
I just updated the docs to explain how: link
You need to have cURL or equivalent installed. Note that some user has reported that the login is not working for them for some reason 🤔

Yup my bad, I don't know what cat walked on my keyboard to add "shrc" (I guess I was editing my .bashrc file at the time?) but the correct link is "https://pawoo.net/api/v1/apps":
curl -X POST -F "client_name=Grabber" -F "redirect_uris=http://<IP_ADDRESS>/" https://pawoo.net/api/v1/apps


Using the package with Iron Router

I'm attempting to get the basic create example working in my app.
HTML:
<template name = "mapPanel">
  <div style="width: 100%; padding-right: 0px; height: 300px;" id="exampleMap">
  </div>
</template>

JS:
Template.mapPanel.onRendered(function() {
	GoogleMaps.create({
		name: 'exampleMap',
		element: document.getElementById('exampleMap'),
		options: {
			center: new google.maps.LatLng(-37.8136, 144.9631),
			zoom: 8
		}
	});
});

Everything works, except when I manually refresh the page (e.g. site.com/path).
Then the map doesn't load, and I'm getting "Exception from Tracker afterFlush function: ReferenceError: google is not defined"
Thinking that this is due to the JS running before template is ready. Does anyone know of any way to elegantly fix this?

Hey @yournewbase you'll need to use GoogleMaps.loaded() to make sure that the library has loaded before calling create.
Template.mapPanel.onRendered(function() {
  this.autorun(function() {
    if (GoogleMaps.loaded()) {
      GoogleMaps.create({
        name: 'exampleMap',
        element: document.getElementById('exampleMap'),
        options: {
          center: new google.maps.LatLng(-37.8136, 144.9631),
          zoom: 8
        }
      });
    }
  });
});

@dburles Ah, that's it! It works now, thank you.


Add Layers to the Model for Longer Training Times

Hello!
Would it be possible for you to make an update to the AI that adds layers to the model? It seems that it hit's a limit after about 4 days of training (the validation loss/training loss stagnates after about 4 days) (this is on a 650 pair dataset). I want to be able to train over the course of a longer time period to achieve better training/validation losses before a model reaches it's limit. Would this be possible?
Thank you in advance for your help!

Anjok 04 Did the sound quality become better in version 4? Is version 4 better than version 2.2?

This is a really important question. We've managed to achieve incredible results in our beta fork: https://github.com/Anjok07/ultimatevocalremovergui/tree/v5-beta-cml. However, the current size of the model does not allow us to move on.

We were able to increase the model size by doubling the channel size like so -
class CascadedASPPNet(nn.Module):
def __init__(self, n_fft):
    super(CascadedASPPNet, self).__init__()
    self.stg1_low_band_net = BaseASPPNet(2, 32)
    self.stg1_high_band_net = BaseASPPNet(2, 32)

    self.stg2_bridge = layers.Conv2DBNActiv(34, 16, 1, 1, 0)
    self.stg2_full_band_net = BaseASPPNet(16, 32)

    self.stg3_bridge = layers.Conv2DBNActiv(66, 32, 1, 1, 0)
    self.stg3_full_band_net = BaseASPPNet(32, 64)

    self.out = nn.Conv2d(64, 2, 1, bias=False)
    self.aux1_out = nn.Conv2d(32, 2, 1, bias=False)
    self.aux2_out = nn.Conv2d(32, 2, 1, bias=False)

    self.max_bin = n_fft // 2
    self.output_bin = n_fft // 2 + 1

    self.offset = 128

Were training model with these settings now. What do you think of this appraoch?

I think that's a good approach, but it increases GPU memory consumption significantly.
Memory-efficient approaches are as follows:

Add an additional convolutional layer to Decoder.
Increase the filter size of some layers of BaseASPPNet (e.g ksize=5, pad=2).

(Sorry, no guarantee for better accuracy)


Special characters in MeasureValue fields that involve # - #

Is your feature request related to a problem? Please describe.
EDH: Sometimes people put in "0-2" as a depth measurement. Currently, this is "Coerced to NA". I'd be interested to hear (a) how people use these ranges in their analyses and (b) how they would suggest handling it in TADA. My first instinct is to calculate an average so these values are usable, e.g. "0-2" becomes 1. But I may be misinterpreting the entry. For example, maybe result value is representative of all depths between 0 and 2, as opposed to representative of a depth somewhere between 0 and 2.
Describe the solution you'd like
CM: For making this field numeric, let's go with the avg (1m in this example) as the NUMERIC value for the depth field. There is also a metadata element where users can specify that this is a "surface" result, and we could/should probably make sure that is filled out here too in this step.
Describe alternatives you've considered
CM: The upper 2m is typically referred to as the "surface" layer, and I think it is safe to assume measure values with 0-2m here are representative of the "surface". However if we want to plot this on a depth profile, we could use the avg like you suggest, or assume the value is the same from x = 0 to 2m down (either option works depending on if we are drawing a line vs scatter plot).
Additional context
Add any other context or screenshots about the feature request here.

Somewhat related to this issue on depth profile data: https://github.com/USEPA/TADA/issues/149


Update toolbox.md icon

Replace old URI http://echarts.baidu.com/images/favicon.png -> https://echarts.apache.org/en/images/favicon.png.

Links to issue: https://github.com/apache/echarts/issues/16543


Update ACL pattern

The anti-corruption layer pattern is not limited to use with legacy systems. Updating the doc to clarify this.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/patterns/_images/anti-corruption-layer.png
:white_check_mark:Succeeded
View

docs/patterns/anti-corruption-layer.md
:white_check_mark:Succeeded
View

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