Instance Spawning Network Delay

When spawning instances, it's possible that high network delay may cause the instance to take a while to actually spawn, leading the Spawner to think it's dead before it's actually started listening for Ping commands to respond to.  The Spawner should put the instance in a Pending queue and either give it a long time to spawn or wait for a Successfully Spawned message from the instance.

Started a fix where instances broadcast a message once they're ready to process commands.  This must be implemented for all instance types, at which point this issue can be closed.


osb-checker runs test outside of the configurated API version

During usage of the osb-checker-kotlin to test compatibility of a service broker with API version 2.13, the osb-checker runs fetch binding tests although this is outside of the 2.13 spec. This is probably caused by the catalog of the broker, which contains binding_retrievable=true, because its actual supported version is >2.13. This means testing backwards compatibility is not really possible with the current state of the osb-checker-kotlin.
Will this behaviour be changed in the future?

This will be fixed, with the update to 2.15 spec.

API version dependent tested solved this issue.
Thank you.
Closing this issue.


Deserialization issue in Ghidra Server

Describe the bug
https://github.com/adrianbn and I are security researches. While doing some work on RMI, we discovered Ghidra Server is vulnerable to a deserialization attack via RMI that leads to SSRF. This is a partially blind SSRF, but requires no authentication.
To Reproduce
We have a proof of concept we can share. In the interest of responsible disclosure, we didn't want to make our github page public just yet, but please let us know if you'd like access, it's currently at https://github.com/ChrisJoinEngine/CVE-2021-40821 (private github as of posting this, we'll add you as collaborators).

The getRepositoryServer method in the GhidraServerHandle class takes unvalidated serialized objects
While these objects are not the trivial case of Java Deserialization, a proxy can be used to provide arbitrary objects to this method
A gadget chain exists in Ghidra that culminates in SSRF
By passing in a malicious object that invokes this gadget chain, Ghidra Server can be made to make an arbitrary request to a file or URL. This is mostly blind, but some stack traces can give some information to an attacker

Expected behavior
The easiest way is to use our proof of concept, and point the script to a Burp Collaborator instance. When you run the script against a Ghidra Server instance, you will see an HTTP request in your collaborator from the Ghidra Server. You can also request a file that is present, and one that is not on the Server's filesystem (e.g. /etc/passwd and /etc/notreal); while you won't be able to view the files, you will see corresponding error messages.
Environment (please complete the following information):

OS: Tested on an up to date OSX, as well as Windows 10 box. As it's Java it probably works on any OS
Java Version: 11.0.12
Ghidra Version: Pulled from master, 10.0.3 development. Probably other versions before that as well
Ghidra Origin: build from https://github.com/NationalSecurityAgency/ghidra

Additional context
Let us know who we give access to the proof of concept, or if there is a better way to report security issues. The PoC has additional details and walkthrough of the exploit.

Thanks for sharing this with us.  Can you please give access to @ghidra1?

Of course, collaborator invite has been sent

Apologies for the delay...we'll have some feedback for you tomorrow.  Thanks for your patience.

Can you please invite me again?

Sure thing, I sent you a new invite. Please don't hesitate to ask if the PoC gives you any issues at all. Thanks

While we were unable to demonstrate SSRF with your test code we have added a serialization filter to the Ghidra Server which has been merged for both patch and master: edbad325084c237b8ffa82e399e884ed42e6358c

Yea, I think it is fair to close this one. Thanks for your time, and looking into it

Thanks for bringing our attention to the potential issue.


cardinality of dcterms:source

If a Concept is based on multiple source, such as a definition and multiple sources of translations, or other annotations, this should be legal.

same is true of prov:derivedFrom
dcterms:provenance can be a single statement however.


How to store data in ES with all-in-one docker image?

Hi，i'm using docker.io/jaegertracing/all-in-one:1.5 docker image and I know set SPAN_STORAGE_TYPE as elasticsearch in env but i didn't find how to set ES_HOSTS.
Can you tell me how to set ES Store with docker.io/jaegertracing/all-in-one:1.5  image?

--es.server-urls or ES_SERVER_URLS

SPAN_STORAGE_TYPE=elasticsearch go run ./cmd/standalone/main.go help

@Yisaer you can join https://gitter.im/jaegertracing/Lobby to ask questions.


At least wait for the first value of untracked deps

This will fix #11

Tell me what you think of renaming Update.Complete.
I'm ready to merge in either case 👍

I'm ready to merge if you are
Only need a "go" from you :)

Hereby the go from me :P
I added the comment, so hopefully it is more understandable now.
Also thank you for the invitation into this repo!!

Merged! Thank you! 🙏


versioneer update

Hello @Olllom! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file lettuce/__init__.py:

Line 31:1: E402 module level import not at top of file

In the file lettuce/_version.py:

Line 342:80: E501 line too long (84 > 79 characters)
Line 432:80: E501 line too long (80 > 79 characters)
Line 435:80: E501 line too long (82 > 79 characters)

In the file versioneer.py:

Line 194:80: E501 line too long (87 > 79 characters)
Line 196:80: E501 line too long (86 > 79 characters)
Line 224:80: E501 line too long (85 > 79 characters)
Line 255:80: E501 line too long (87 > 79 characters)
Line 257:80: E501 line too long (83 > 79 characters)
Line 259:80: E501 line too long (86 > 79 characters)
Line 664:80: E501 line too long (81 > 79 characters)
Line 758:80: E501 line too long (85 > 79 characters)
Line 848:80: E501 line too long (80 > 79 characters)
Line 851:80: E501 line too long (85 > 79 characters)
Line 1276:80: E501 line too long (84 > 79 characters)
Line 1473:80: E501 line too long (80 > 79 characters)
Line 1476:80: E501 line too long (82 > 79 characters)
Line 1746:80: E501 line too long (83 > 79 characters)


进入使用EasyRefresh的页面,然后快速返回上一页报错

======== Exception caught by widgets library =======================================================
The following assertion was thrown while finalizing the widget tree:
_EasyRefreshState#5085c(tickers: tracking 2 tickers) was disposed with an active Ticker.
_EasyRefreshState created a Ticker via its TickerProviderStateMixin, but at the time dispose() was called on the mixin, that Ticker was still active. All Tickers must be disposed before calling super.dispose().
Tickers used by AnimationControllers should be disposed by calling dispose() on the AnimationController itself. Otherwise, the ticker will leak.
The offending ticker was: _WidgetTicker(created by _EasyRefreshState#5085c)
The stack trace when the _WidgetTicker was actually created was:
#0      new Ticker. (package:flutter/src/scheduler/ticker.dart:67:40)
#1      new Ticker (package:flutter/src/scheduler/ticker.dart:69:6)
#2      new _WidgetTicker (package:flutter/src/widgets/ticker_provider.dart:385:81)
#3      TickerProviderStateMixin.createTicker (package:flutter/src/widgets/ticker_provider.dart:295:34)
#4      new AnimationController.unbounded (package:flutter/src/animation/animation_controller.dart:279:21)
#5      IndicatorNotifier._initClampingAnimation (package:easy_refresh/src/notifier/indicator_notifier.dart:293:58)
#6      new IndicatorNotifier (package:easy_refresh/src/notifier/indicator_notifier.dart:44:5)
#7      new HeaderNotifier (package:easy_refresh/src/notifier/indicator_notifier.dart:779:8)
#8      _EasyRefreshState._initData (package:easy_refresh/src/easy_refresh.dart:327:23)
#9      _EasyRefreshState.initState (package:easy_refresh/src/easy_refresh.dart:287:5)
#10     StatefulElement._firstBuild (package:flutter/src/widgets/framework.dart:4942:57)
#11     ComponentElement.mount (package:flutter/src/widgets/framework.dart:4781:5)
...     Normal element mounting (45 frames)
#56     Element.inflateWidget (package:flutter/src/widgets/framework.dart:3817:16)
#57     MultiChildRenderObjectElement.inflateWidget (package:flutter/src/widgets/framework.dart:6350:36)
#58     MultiChildRenderObjectElement.mount (package:flutter/src/widgets/framework.dart:6362:32)
...     Normal element mounting (244 frames)
#302    Element.inflateWidget (package:flutter/src/widgets/framework.dart:3817:16)
#303    MultiChildRenderObjectElement.inflateWidget (package:flutter/src/widgets/framework.dart:6350:36)
#304    Element.updateChild (package:flutter/src/widgets/framework.dart:3551:18)
#305    RenderObjectElement.updateChildren (package:flutter/src/widgets/framework.dart:5883:32)
#306    MultiChildRenderObjectElement.update (package:flutter/src/widgets/framework.dart:6375:17)
#307    Element.updateChild (package:flutter/src/widgets/framework.dart:3530:15)
#308    ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:4832:16)

解决方法:

解决方法 easy_refresh.dart
@override
void dispose() {
super.dispose();
_headerNotifier.dispose();
_footerNotifier.dispose();
_userOffsetNotifier.dispose();
}
替换成
@override
void dispose() {
_headerNotifier.dispose();
_footerNotifier.dispose();
_userOffsetNotifier.dispose();
super.dispose();
}
@xuelongqy

这个问题在最新版已解决

这个问题在最新版已解决


Using a different backbone?

My question is how to load weights from a custom backbone (not from the set already supported by qubvel segmentation_models)?
I have trained ImageNet models such as ResNet50 for image classification using satellite data and would like to use the resulting weights as the ResNet50 encoder within a Unet model for segmentation rather than the default ResNet50 ImageNet weights. Does anyone know if this is possible, and, if so, any clues on how to go about this?
Thanks in advance...

Yes this is definitely possible. If you download imagenet weights from keras for resnet 50 and load them into the encoder portion of the unet model in the repo, it will work fine. That means, if you train your own resnet 50 from scratch, you can load those into the encoder of the unet model. I've done it before, I believe I had to create a custom function to do it, but it wasn't too difficult.

Thanks Jordan. Existing code looks like this:
BACKBONE = 'resnet50'
model = sm.Unet(BACKBONE, encoder_weights='imagenet', classes=n_classes, activation=activation, input_shape=(None, None, N))
which downloads encoder weights from: https://github.com/qubvel/classification_models/releases/download/0.0.1/resnet50_imagenet_1000_no_top.h5
How do you load weights from a local model .h5 file?

Like I said, I believe you need to create a custom function (though some might already exist on the internet, check stackoverflow), in which you go through each layer of the unet encoder and copy over the weights in the corresponding layers from the other encoder. The process is very similar to turning all of the layers in the encoder to frozen or unfrozen, please note that this is not working code, it's just meant to give you an idea of how one could accomplish what you're asking.
# non-working psuedo-code
unet_model = sm.Unet(BACKBONE, encoder_weights='imagenet', classes=n_classes, input_shape=(None, None, N))

# however you want to do this
custom_model = ResNet50(weights = 'custom_weights.h5')

# go through each layer in the unet_model,
# find the layers that have the same names as
# the layers in the custom ResNet50 model,
# copy over the weights.
layer_names = [layer.name for layer in custom_model.layers]

for layer in unet_model.layers:
    if(layer.name in layer_names):
        # swap the weights
        unet_model.layer.weights = custom_model.layer.weights

You get the basic idea..

Thanks Jordan - I get the idea but the implementation is more difficult!
For the segmentation_models unet model:
BACKBONE = 'resnet50'
model = sm.Unet(BACKBONE, encoder_weights='imagenet', classes=n_classes, activation=activation, input_shape=(None, None, N))
you can get the layer names using:
for layer in model.layers:      print (layer.name)
These names look like this:
data bn_data zero_padding2d_1 conv0 bn0 relu0 zero_padding2d_2 pooling0 stage1_unit1_bn1 stage1_unit1_relu1 stage1_unit1_conv1 stage1_unit1_bn2 stage1_unit1_relu2 zero_padding2d_3 stage1_unit1_conv2 stage1_unit1_bn3 stage1_unit1_relu3 stage1_unit1_conv3 stage1_unit1_sc add_1 stage1_unit2_bn1... etc
For the resNet50 model, the layer names are:
input_1 conv1_pad conv1 bn_conv1 activation_1 pool1_pad max_pooling2d_1 res2a_branch2a bn2a_branch2a activation_2 res2a_branch2b bn2a_branch2b activation_3 res2a_branch2c res2a_branch1 bn2a_branch2c bn2a_branch1 add_17... etc
So the names don't match, not sure how to link them and not sure whether all or just some of the weights need to be transferred over?
Any advice is appreciated...

Hi @tonyboston-au because the encode (i.e., resnet50) is within the Unet structure, keras treats it as a layer (thus ignoring the names of all of the resnet50 layer names). Can you confirm that the layer names you're showing are the resnet50 layers within the unet model, or just the layer names of the unet model?
It would be something like:
layer_index_for_resnet_50 = N

# print all resnet layers
for layer in unet.layers[layer_index_for_resnet_50].layers:
   print(layer.name)

Also, see #36, they discuss the same thing but for a different reason.

My apologies! I see now that the UNet model doesn't treat the encoder as a layer (derp, not enough coffee) and understand the difficulties with the differences in naming. However, although the names are different, they correspond to the same layers otherwise quvbel wouldn't have been able to load the imagenet weights into the encoders.

Can't work out how the layer names correspond as they look so different.
Unet layer names are neat and tidy but ResNet50 names are messy...
 I know the mapping to load weights is possible but it doesn't seem to be a simple one to one correspondence of the layers.

I think it might be easier to re-train your with the ResNet50 implementation of this repo. Alternatively you could add the decoder to your existing model. It's not that difficult.

Thanks for your help @JordanMakesMaps and @pluniak.
@pluniak - these sound like a more doable options. I have move on to other things for now but may come back to it later and try your suggestions...

Hi @tonyboston-au,
I don't know if you have already solved this by yourself, but I wanted to share a quick modification you could use, in case it helps:
from segmentation_models.models.unet import build_unet, DecoderUpsamplingX2Block, DecoderTransposeX2Block

def CustomBackboneUnet(backbone,
                       classes=1,
                       activation='sigmoid',
                       encoder_freeze=False,
                       encoder_features=None,
                       decoder_block_type='upsampling',
                       decoder_filters=[256, 128, 64, 32, 16],
                       decoder_use_batchnorm=True,
                       **kwargs):

  if decoder_block_type == 'upsampling':
    decoder_block = DecoderUpsamplingX2Block
  elif decoder_block_type == 'transpose':
    decoder_block = DecoderTransposeX2Block
  else:
    raise ValueError('Decoder block type should be in ("upsampling", "transpose"). '
                    'Got: {}'.format(decoder_block_type))

  if encoder_features is None:
    raise ValueError('Please provide encoder features')

  model = build_unet(
        backbone=backbone,
        decoder_block=decoder_block,
        skip_connection_layers=encoder_features,
        decoder_filters=decoder_filters,
        classes=classes,
        activation=activation,
        n_upsample_blocks=len(decoder_filters),
        use_batchnorm=decoder_use_batchnorm,
    )

  # lock encoder weights for fine-tuning
  if encoder_freeze:
      freeze_model(backbone)

  return model

So, assuming you use the ResNet50 provided by Tensorflow/Keras:
from tensorflow.keras.applications import ResNet50

backbone = ResNet50(weights=model_weights_path,
                    input_shape=(256, 256, 3),
                    include_top=False,
                    pooling=None)

backbone_features = ('conv4_block6_out', 'conv3_block4_out', 'conv2_block3_out', 'conv1_relu')

model = CustomBackboneUnet(backbone, encoder_features=backbone_features, encoder_freeze=True)

Thanks @adolfogc. I've moved on from this as the differences in performance using weights from different sources appear to be minor, but thanks for your suggestions.


Make doc and query count configurable in benchmark

Signed-off-by: John Mazanec<EMAIL_ADDRESS>Description
Makes the document and query count configurable in the benchmarking tool. With this functionality, users can now specify to only index or search a subset of the vectors in the data set. This is useful for indices that require training that may only need a subset of the data set for training.
A query or ingest step might look like this now:
steps:
...
  - name: ingest
    index_name: target_index
    field_name: target_field
    bulk_size: 500
    dataset_format: hdf5
    dataset_path: mydata.hdf5
    doc_count: 50000
...
  - name: query
    k: 100
    r: 1
    index_name: target_index
    field_name: target_field
    dataset_format: hdf5
    dataset_path: mydata.hdf5
    query_count: 5
    neighbors_format: hdf5
    neighbors_path: mydata.hdf5

Check List

[X] Commits are signed as per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #270 (a864a4a) into main (6e859f5) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #270   +/-   ##
=========================================
  Coverage     83.38%   83.38%
  Complexity      884      884
=========================================
  Files           127      127
  Lines          3833     3833
  Branches        361      361
=========================================
  Hits           3196     3196
  Misses          475      475
  Partials        162      162

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 6e859f5...a864a4a. Read the comment docs.

What are default values for new params?
And another question - does it make sense to reflect values of new params in test result, maybe with an average?

@martin-gaievski The default is the entire dataset. So, if you dont specify anything it will index each vector in the dataset.

And another question - does it make sense to reflect values of new params in test result, maybe with an average?

All metrics take into account the dataset size. For instance, querying produces the p50, p90 and p99 metrics. For ingest, it is the total time. In the future we could add an ingest metric that is docs/sec, but I think thats outside scope of this change for now.


Generic Python package discovery

This PR should add support to allow for the discovery of:

packages installed in virtual environments
packages installed in Python environments that are not in the PATH
Python environments setup as default ms-python.python

TODO:

Update description
Assign Issues
Add tests

I have a more advanced version of this branch staged locally, that wraps all existing Python functionality and some additional features into a Python class. However the challenges I am facing are the following.
I can't seem to be able to find a good SOLID approach to share a single instance of the Python class with all the different parts of the extension. Specifically the Single Responsibility Principle appears to be violated repeatedly with my proposed design.
There is a package called inversify (https://github.com/inversify/InversifyJS) that could help with the dependency inversion and SRP, but I would have to tear apart most of the extension. Using inversify would only make sense if I had the time to rewrite certain parts of the extension that deal with the whole set of Configuration options. Inverting the dependency just for Python but maintaining the linter and formatting dependencies is somewhat strange.
The second part is that all of this Python work feels somewhat redundant given that we could save ourselves a lot of grief just by using pipx run fortls or pipx run findent. My main concerns with pipx are that:

it spawns considerably slower than native calls to fortls (or any other binary)
It uses caching to improve performance but the cache expires after 14 days
It does not respect virtual environments, so pipx run fortls will install fortls in whichever place pipx installs binaries, even if pipx is called from within a virtual environment. I am sure there might be some settings that can help with this, but I haven't found them.

I would be fine using pipx if it weren't for the last point. I suspect users might complain that Modern Fortran installs stuff outside their venv.
As you can tell, my mind is not entirely set with the design or even approach that should be followed to solve the Python issues.
Let me know what you think.

Inverting the dependency just for Python but maintaining the linter and formatting dependencies is somewhat strange.
Yeah that seems strange to me as well.

I'm not sure I quite understand the motivation for this PR.
From my perspective, there's the problem discovering fortls on Windows, because it typically gets installed in a folder that's not on PATH, but is this the case on linux/mac as well?
Can you clarify?
The way I look at it now is there are basically two use cases:

we want to make the usage of the extension as seamless as possible,
which means being able to install, update, detect and run fortls automatically
the users should have the option to override this and run fortls from a custom location

Is this correct?
For the user override, do we have any idea on how often people actually use it?
If it were just a very small number, it might not be worth maintaining in the long term if it complicates things too much, but I have no idea how popular it is or how it's used.

I'm not sure I quite understand the motivation for this PR.
From my perspective, there's the problem of discovering fortls on Windows, because it typically gets installed in a folder that's not on PATH, but is this the case on linux/mac as well?
Can you clarify?

The motivation is to be able to interact with Python in a standardised and abstract manner.
Discovery of fortls on Windows is just one of the issues that would be resolved.
There can be similar issues with formatters, fypp preprocessor and fpm.
There is an entire class of user setups that would face auto-discovery issues.
Similarly, there are virtual environments and interactions with the ms-python.python extension that have been requested as features.
Manually specifying the paths, although well documented and trivial to do, seems to be a blocker for many users.
To me that is a clear indicator that the extension design of how Python package auto-discovery is implemented needs to be reworked.
Modern Fortran, as an extension that integrates with Python packages needs to:

Work with multiple Python and virtual environments
Install packages to the selected Python environment
Check if a Python package is installed in the environment
Get the location of installed Python binaries in the environment
Share the selected Python configuration within all parts of the extension that make use of it

This PR aims to address all of the above bullet points.
There are some minor issues, mostly related to points 3. & 4., but nothing that can't be solved with a bit of Python code.
The more pressing issues are 5. & 1. which carry technical difficulties and design considerations.

I see.
Regarding the point:

Work with multiple Python and virtual environments

Not with multiple ones at a time, though, right?
What's the idea about how one would select the venv that's supposed to be used?
If your local code isn't too much of a mess, maybe you can push it to a separate branch and I could have a look at it? Maybe I'll have some ideas. (It doesn't have to work, just to give me an idea...)

I have a more advanced version of this branch staged locally, that wraps all existing Python functionality and some additional features into a Python class. However the challenges I am facing are the following.
I can't seem to be able to find a good SOLID approach to share a single instance of the Python class with all the different parts of the extension. Specifically the Single Responsibility Principle appears to be violated repeatedly with my proposed design. There is a package called inversify (https://github.com/inversify/InversifyJS) that could help with the dependency inversion and SRP, but I would have to tear apart most of the extension. Using inversify would only make sense if I had the time to rewrite certain parts of the extension that deal with the whole set of Configuration options. Inverting the dependency just for Python but maintaining the linter and formatting dependencies is somewhat strange.
The second part is that all of this Python work feels somewhat redundant given that we could save ourselves a lot of grief just by using pipx run fortls or pipx run findent. My main concerns with pipx are that:

it spawns considerably slower than native calls to fortls (or any other binary)
It uses caching to improve performance but the cache expires after 14 days
It does not respect virtual environments, so pipx run fortls will install fortls in whichever place pipx installs binaries, even if pipx is called from within a virtual environment. I am sure there might be some settings that can help with this, but I haven't found them.

I would be fine using pipx if it weren't for the last point. I suspect users might complain that Modern Fortran installs stuff outside their venv.
As you can tell, my mind is not entirely set with the design or even approach that should be followed to solve the Python issues.
Let me know what you think.

Honestly, I think you could abandon this PR.  See my comment in #957


Contributing Guide- PR instructions inconsistency

Issue Description
I just submitted a PR for the first time, following the instructions in the Contributing Guide, and noticed a couple of conflicting points:
link to the Contributing Guide "Creating a PR" section

under Creating a PR

method 1/point 8 instructs you to use the tool to create a commit, which automatically adds an issue number and description to the commit message

under Common Steps

point 4 says "Do not add the issue number in the PR title or commit message."
point 5 says "at the end of your PR's description, append the keyword closes and #xxxx (where xxxx is the issue number)"

I guess the Common Steps/point 4 part should be removed ?

Yes, that is correct.

Hi! I'm completely new to contributing to open source projects so I'd like to work on this small issue as my first step. Am I correct in thinking that information for common step 4 needs to be completely removed and common steps 5 & 6 need to be re-numbered as common steps 4 & 5?

Yes.. that is correct @Komorebi-Girl !

Okay, great! Can I handle this tiny issue then?

is the issue resolved ?I am looking to work on it if not

Hi @sourabh1031 thanks for the interest. I suggest you look into the tracker, we have more open issues that we can use your help.

Okay, I'm a bit stuck. I'm trying to run the npm run commit command to use the commit tool but I get this error message from npm: "'git-cz' is not recognized as an internal or external command,
operable program or batch file."
I thought perhaps I should do an npm install but when I try to do that nothing happens. No error message. Nothing at all. Am I maybe missing something important?

@Komorebi-Girl can you reach out to the Contributors Chat room

Ok, will do. Thanks!


Add test for data linking self-correction code, turn on consistency checker

Add a test that ensures the self correction code is run when a problem is
detected in the version doc when linking data.
Also turn on the consistency checker, can't believe I missed that.

Coverage increased (+0.002%) to 97.59% when pulling b70017beecfb79127f657daf77875a73e25d8d84 on MrCreosote:master into b2de034bf7e631547ca7e6493e560a952f26e614 on kbaseIncubator:master.


Email sent confirmation

Screenshot

GOV.UK Guidance
There's guidance on this here https://www.gov.uk/service-manual/user-centred-
design/resources/patterns/email-confirmation-loop.html

@gavinwye This feels like something that should be in the email confirmation loop section of the service manual.

Agreed. This should definitely be a final step in a multi-step pattern, not a pattern in its own right.

This issue has been closed because it should be part of an email confirmation loop in the GOV.UK Design System.


Prepare Ravenpy v0.11.0

@huard You got to this much sooner than I expected! Safe travels!

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

7 of 13 (53.85%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 84.218%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

ravenpy/cli/collect_subbasins_upstream_of_gauge.py
3
6
50.0%

ravenpy/cli/generate_hrus_from_routing_product.py
3
6
50.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
3746

Relevant Lines:
4448

💛  - Coveralls


How to set a Project to keep a certain time Exceptionless data

How to set a Project to keep a certain time Exceptionless data, such as a week, other data is deleted automatically, or automatically stored elsewhere

Are you looking to have retention based on a specific project other than on the organization level?

Do you have based on organization-level settings

Find a solution ：https://github.com/exceptionless/Exceptionless/issues/139


Allow setting default locale for multi-language title

Instead of hardcoded 'English', one can choose other locales to work from

Looking nice, could you please add an example in the readme with a comment about the use of this option?
Any feedback @apurvis ?

Looks great to me just make sure to update the README before merging
On Thu, Dec 7, 2017 at 12:34 Adrien Jarthon<EMAIL_ADDRESS>wrote:

@jarthod https://github.com/jarthod requested your review on:
jarthod/survey-gizmo-ruby#101
https://github.com/jarthod/survey-gizmo-ruby/pull/101 Allow setting
default locale for multi-language title.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/jarthod/survey-gizmo-ruby/pull/101#event-1377693105,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOJDlvtKueK5hcF9q-ljlSqaO043klWks5s-EvOgaJpZM4Q5zEb
.

--
sent from space

Thanks @jarthod @apurvis, I added a little comment into the Readme, just edit it in case it isn't clear enough

Lgtm
Fwiw I'm going to be generally less active maintaining this gem since I am
no longer working for anyone using surveygizmo. Still happy to take a look
though
On Sun, Dec 17, 2017 at 15:01 Adrien Jarthon<EMAIL_ADDRESS>wrote:

@jarthod approved this pull request.
Good for me, anything else @apurvis https://github.com/apurvis ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jarthod/survey-gizmo-ruby/pull/101#pullrequestreview-84009592,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOJDhoUWlDIuB7jlWsUo4tcJ4kL44Cgks5tBXMFgaJpZM4Q5zEb
.

--
sent from space

Thanks for your contribution, I've merged this and released 6.5.1

not a huge deal but should be 6.6.0... https://semver.org
On Mon, Dec 18, 2017 at 3:10 AM, Adrien Jarthon<EMAIL_ADDRESS>wrote:

Thanks for your contribution, I've merged this and released 6.5.1
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jarthod/survey-gizmo-ruby/pull/101#issuecomment-352353772,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOJDnlqVMtb4VmS3QPXyMIaKpgbfsqVks5tBh3_gaJpZM4Q5zEb
.

Good point, I've updated to 6.6.0


Test Suite Issues

Here's a list of trouble I had getting the test suite running locally:

[ ] The test suite assumes too much about the host system.  I could not run it on the macOS system I use for all of my other docker development.  We should fix this with a build container.  Sample: https://gist.github.com/ryansch/388ce4ffdbdbe906c92dd339aa810ba9
[ ] The specs currently hardcode the image name to test against to kylemanna/openvpn.
[ ] The dual-proto specs require changes to the firewall and assume nothing else is running on 1194 and 443.  We should look into using a dedicated docker network here.

Yeah, these all need to be fixed.  The test cases were introduced in a hurry with the philosophy of "rough test cases are better then no test cases" after a number of extreme fringe features started breaking.
The biggest problem with OpenVPN is that for advanced configurations (i.e. IPv6 or dual-proto) it needs to mess with networking options that Docker just doesn't support now.
Also, I'm not familiar with the "build container" concept, can you explain more how it would help?

The specs currently hardcode the image name to test against to kylemanna/openvpn.
This isn't a problem since if you build locally and tag the build as kylemanna/openvpn it will use your local copy instead of remotely fetching.


Add search API functionality

Fixes

Partially addresses #249

Description

Incorporate search api per specifications for #249
Additionally replaces flask_restful with flask_restx invocation.

Testing

TODO:

Performance

Performance impact expected to be limited -- Largest impact is likely to depend on how many search results are returned.

Docs

Incorporating Flask_restx enables documentation to be automatically generated from docstrings in Resource.
To view documentation -- start app.py and navigate to server location. Then click on "Default namespace" to view the full docuemtnation for all methods.
Namespace can be eventually configured, but is beyond the scope of this PR.

@maxachis Can you reopen this on the v2 repo?

@mbodeantor Done!


Removing slashes that shouldn't be there

Removing slashes that shouldn't be there - the "" is wrong and launchpad will not start if it is there

@joshts0 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


[9.x] Adds attempt with callbacks

This PR expands on the discussion here about adding callbacks the attempt() authentication mechanisms.
Problem
There is no way to add additional checks at runtime when using attempt(). Any addition forces the developer to create a new Guard, or use nonchalantly validate() and getLastAttempted(), and fire Attempting manually.
if (Auth::validate($credentials) && $user = Auth::getLastAttempted()) {
    if ($user->isAwesomeAndCool()) {
        Auth::login($user, $request->filled('remember'));
        Event::fire(Attempting('session', $credentials, $request->filled('remember')));
        return $this->redirect();
    }
}

return $this->sendAttemptFailed();

Solution
This PR allows the SessionGuard::attempt() method to the receive a third argument that receives one or multiple callbacks. Each of these receive the validated User, and if one returns false, the whole attempt fails.
$result = Auth::attempt($credentials, false, function (Authenticatable $user) {
    return $user->isAwesomeAndCool();
});

return $result
    ? $this->redirect()
    : $this->sendAttemptFailed();

Since there are callables, the developer can add its own checks, or use checks made by external packages, all while controlling manually the flow and order of these.
How it works
The change adds an internal function shouldLogin() that executes each callback and returns false if one of them "fails". This is called after the user is properly validated:
if ($this->hasValidCredentials($user, $credentials) && $this->shouldLogin($callbacks, $user)) {
    $this->login($user, $remember);

     return true;
}

Breaking changes: extending the Session Guard

The above change is simple and non-breaking, since it doesn't changes the interface signature, but as a precaution I'm pushing it into the next version... unless it gets approved for 8.x. And this also includes an additional test.

Yeah. If someone is extending the class it will be terrible. 9.x it is.

Since it's for 9.x, why just change the contract? This would mean interoperability between all guards (included and custom ones).

It is in theory possible to solve this on 8.x by passing a reserved key in the $credentials array... something like __callbacks. An outside user can't submit a Closure through a form of course so we would simply spin the array of callbacks and invoke them if they are an instance of Closure. It would need to introduce a new method to make this easier like:
Auth::attemptSomeMethodName(array $arrayOfCallbacks, array $credentials, etc...)
{
    $credentials['__callbacks'] = $arrayOfCallbacks;
    return $this->attempt($credentials, etc...);
}

I really would prefer using callables because I can just reference an static method from somewhere.
I think that reserving a key it's kind of hassle because there is no guarantee the developer is not using it somewhere (I've seen some sh*t).
It could be that just adding a new method that accepts an array of callbacks could work for 8.x.
Auth::attemptAnd($credentials, fn($user) => $user->isCool());

If you add a new method, I'm not sure you have a way to get those callbacks called at the correct time during the authentication process without a breaking change? Unless I'm missing something?

If you add a new method, I'm not sure you have a way to get those callbacks called at the correct time during the authentication process without a breaking change? Unless I'm missing something?

I'm also missing something. Why it wouldn't be called at the correct time? The idea is to work on the validated user itself. Wrapping it up:

The dev calls attempt() (or whatever name is) with the callbacks.
The guard gets the user for these credentials.
The guard checks if the credentials are valid.
This is where the callbacks run for further checks, because it has now the Authenticatable instance to work with.

This is intended to work like an "AfterValidated" hook because, before validation, its pretty much like a black box.

Well - if you think you can add a new method in such a way that is not breaking feel free to adjust this PR to do that and I'll take a look.

Im on it. ‘attemptAnd()’ will be my choice.
Italo Baeza C.

El 22-04-2021, a la(s) 09:48, Taylor Otwell @.***> escribió:

Well - if you think you can add a new method in such a way that is not breaking feel free to adjust this PR to do that and I'll take a look.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Can you resubmit this to 8.x?

Can you resubmit this to 8.x?

Lol, of course.

https://github.com/laravel/framework/pull/37090


Sync stuck on 44

Sync stuck on 44

add node, port 18444
`  - <IP_ADDRESS>

<IP_ADDRESS>
<IP_ADDRESS>`

Any other node? I have been stuck at 66.

Any other node? I have been stuck at 66.

All Node new : 66. Wait 15m will update node

Still got sync issues unfortunately

Still got sync issues unfortunately

It's not a sync error. It was because the difficulty level that the mainnet set at the beginning was too high. Make the block no one can find. So it cannot continue to sync. More than 5 to 15 minutes block is not updated it will not sync. But the challenge part still jumps. If someone found the block it resumes syncing.

Somthing is worng. With 770 plots and 3 hours of farming I haven' win a single block. Estimated time to win was set to 15 min, not it's 17 min.

Something is wrong. With 770 plots and 3 hours of farming I haven't won a single block. Estimated time to win was set to 15 min, now it's 17 min. Challenges are running fine, plots are also ok, since with other forks I am winning it regularly.

I joined since the network only had 1 block until now! It took me 12 hours to find a block a few minutes away :(
I see that the dev is making the difficulty equal to the difficulty of Chia. So it is difficult to find blocks.

Something is wrong. With 770 plots and 3 hours of farming I haven't won a single block. Estimated time to win was set to 15 min, now it's 17 min. Challenges are running fine, plots are also ok, since with other forks I am winning it regularly.

I joined since the network only had 1 block until now! It took me 12 hours to find a block a few minutes away :(
I see that the dev is making the difficulty equal to the difficulty of Chia. So it is difficult to find blocks.

...but then, who is winning all those blocks if the difficulty is equal to Chia? If I have 2% of total net space, mathematically I would win something at least every couple of hours.

The difficulty of the seno network, not the equivalent difficulty of the split network. You still have the Challenge Plots Passed Filter and you will have a chance to found the block.

This blockchain is a joke. it is again not synced.

Just reinstall all

Just reinstall all

People are complaining about the win rate. I personally have 770 plots and did find only 1 block in more than 12 hours. Something is not ok here.


Should EasyTab handle context changes?

Changing Wacom settings, like the wacom input area or screen area, when the EasyTab client app is open requires the client to handle it.
Changing display settings (like turning off one display) also requires the client to handle it, since the Wacom context needs to be rebuilt.
In my case, I am currently not handling it. My app needs a restart whenever those two particular cases happen. Not a huge deal, but the other applications I test seem to handle these cases without problems.
I was debating whether to open this issue or not, since this is one of those "Drop the Easy from EasyTab" issues.
The applications I mentioned that handle these cases seem to work when the wacom driver crashes, so I have to assume that they are using the Windows Stylus API. Also, the Wacom sample code is not helpful at all for cases like these.
So I'm thinking that one way forward should be to support the Stylus API and leave the Wacom layer as a fallback for when the Stylus API is not available or when EasyTab is compiled as C99.
This could add a lot of complexity to the code, so it might be a bad idea? I just wanted to write my thoughts somewhere that wasn't email, since this is an open source project. I'm curious to read your thoughts, but feel free to close the issue, since there is no code to show yet :)

Settings change
I think switching to the Microsoft Stylus API will incur problems of its own, such as C99 compatibility. I'd vote to keep that out of EasyTab at least for now, until we find a more compelling reason to add it.
Can we auto-detect driver settings change somehow? If yes, we could add internal lazy reloading to the library to refresh the context.
If this adds too much code complexity, we might want to simply not add this functionality in here. Can we get in touch with someone who has written the apps that do The Right Thing™? There may be an undocumented way to get refreshed settings from the driver.
Driver crash
I'm okay with having to restart the application in case this happens, as long as we don't segfault in EasyTab. If we do decide to go the lazy reloading route for the settings refresh, this may get solved too. If not, I can live with it.

@ApoorvaJ an email to wacom would be very likely to return something helpful.

Good to hear your thoughts and know where you stand.
I don't know of a way to detect driver setting changes.  It will be useful to go looking into GTK and Qt and see what they are doing.
I mentioned the driver crash because on one occasion, my Wacom driver crashed without recovery. EasyTab didn't work because Wacom was FUBAR, yet Mischief and Krita(IIRC) had tablet support.
I'm closing the issue for now :)

Sorry to nag, but I had to correct myself. I must be remembering incorrectly as to what happened when the driver crashed. I attached a debugger to Krita and Mischief, and they are both calling WTPacketsPeek and WTPacketsGet respectively, so they are using the Wacom API, not the Windows API as I was assuming.


SSH Client

I'm opening another PR for this since I've split out the code from another PR and rebasing the other PR will take too long.

Looking forward to this.  Let me know when it's ready to test!

@ehazlett the SSH command works now. So that bit can be tested.

@ehazlett Fixed... ready to actually test now.

I'm super excited about this one, let's move this forward @sthulb :heart:

@ahmetalpbalkan I've addressed your latest concerns.
@ehazlett Back to testing I guess :)

@sthulb let me know if you need a windows box to test, I can give one :) my guess is unless we use docker/pkg/term (adding my comment which got lost in a commit below), ssh behavior on windows will probably get worse than what we got today.

@ahmetalpbalkan I've already switched out for the one from docker as you suggested.
I can use the windows vms from modern.ie or use one from AWS. :) thanks for the offer.

@sthulb ooh this is gonna be interesting.
we have some code to enable ANSI emulation on windows. check StdStreams method in docker/docker and use the streams it gives you instead of os.Stdin/os.Stderr (or you can modify os.Stdin/os.Stderr with those in main() or init() of main pkg too)

@ahmetalpbalkan the streams stuff sounds good. I can certainly look into this in another PR.
I think machine could do with some other changes for this to work. And I'm definitely in favour of better windows support.

Ugh. Again.

you like red buttons  :D :dancer:

GitHub has pretty bad usability on a phone. :)

GitHub has pretty bad usability on a phone. :)

Couldn't agree more. Huge buttons to "close" or "merge", but no button to edit your comment. To make things worse; (if you're on an iPhone), those buttons are directly beneath the "close" button to dismiss your keyboard.
(Recently pressed on "merge", and it was almost impossible to navigate/scroll away from the "confirm" button :fearful:).

This is causing a panic on Windows (msysgit w/bash):
$ docker-machine_windows-amd64.exe ssh test-vbox
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x2 pc=0x567486]

goroutine 16 [running]:
runtime.panic(0xaeb240, 0xf4e502)
        /usr/src/go/src/pkg/runtime/panic.c:279 +0x11f
github.com/docker/machine/ssh.(*Client).Shell(0xc0820086a0, 0x0, 0x0)
        /go/src/github.com/docker/machine/ssh/client.go:127 +0x476
github.com/docker/machine/libmachine.(*Host).CreateSSHShell(0xc08201ed00, 0x0, 0x0)
        /go/src/github.com/docker/machine/libmachine/host.go:169 +0x2e5
github.com/docker/machine/commands.cmdSsh(0xc082054240)
        /go/src/github.com/docker/machine/commands/ssh.go:66 +0x790
github.com/codegangsta/cli.Command.Run(0xb9a5f0, 0x3, 0x0, 0x0, 0xc64650, 0x2f, 0xca6970, 0x57, 0x0, 0x0, ...)
        /go/src/github.com/docker/machine/Godeps/_workspace/src/github.com/codegangsta/cli/command.go:113 +0xeb9
github.com/codegangsta/cli.(*App).Run(0xc08201e270, 0xc082005da0, 0x3, 0x3, 0x0, 0x0)
        /go/src/github.com/docker/machine/Godeps/_workspace/src/github.com/codegangsta/cli/app.go:156 +0xaf1
main.main()
        /go/src/github.com/docker/machine/main.go:73 +0x5aa

@ehazlett

LGTM
/cc @ahmetalpbalkan

@ehazlett I still have a few outstanding comments. I really didn't dig through the code btw, I just read a few quick passes on my bus rides several times. :)

@ahmetalpbalkan cool thx!  @sthulb could you address those?  I think after that it should be good.

@sthulb awesome things in that screenshot! thanks for moving this forward. ANSI emulation in windows console should be done once we use the StdStreams from pkg/term (maybe a separate pr?).

@ahmetalpbalkan exactly. Another PR will be incoming with StdStream, once I figure out how to use it everywhere in machine.

I agree - a second PR for that would be best.

All issues resolved hopefully. :) @ehazlett @ahmetalpbalkan

LGTM :+1:


Convert an execution listener like ${execution.setVariable("a", "B")} to an output mapping

Is your feature request related to a problem? Please describe.
A process contains a lot of execution listeners that are implemented as expressions like
${execution.setVariable("a", "B")}

These execution listeners are not available in Zeebe.
Describe the solution you'd like
The result of a conversion is an output mapping of the task or event:
    <bpmn:startEvent id="StartEvent_1">
      <bpmn:extensionElements>
        <zeebe:ioMapping>
          <zeebe:output source="="b"" target="a" />
        </zeebe:ioMapping>
      </bpmn:extensionElements>
    </bpmn:startEvent>

Describe alternatives you've considered
keep converting this by hand from the message.
Additional context
I have seen this in one process of a customer.

For now, i would leave this in the hands of a customer.

Even if we don't implement it now, we should keep the issue open to check if this modeling style in Camunda 7 happened any other time.


Make ad delection/checking a bit smarter

I have multiple ads with the same name under different categories, so everything was being deleted at once. This PR makes it so that when deleting/checking ad existence, it also respects the ads category

This is the kind of abuse that makes Kijiji more cluttered and pushes them to implement countermeasures against this project.

This is the kind of abuse that makes Kijiji more cluttered and pushes them to implement countermeasures against this project.

Having multiple ads under multiple catgories is abuse? Im renting a home and it falls under short-term rental, long-term rental, and roomate rental. Is this not allowed?

Also not sure why you're on this account and not ArthurG, must of been an accident? 😛

This seems like a very reasonable change. Thanks for the contribution!


Search a osd system

Check this osd
https://github.com/ErikReider/SwayOSD
or
https://github.com/heyjuvi/avizo

I decide go with avizo just because swayosd seems like the author is not an expert in the language and the aur package page shows many comments about issues to install


EIP712 iOS Not Working

As described in https://github.com/trustwallet/wallet-core/discussions/3049, Trust Wallet is having trouble signing EIP712 signed data on iOS.
I am using Web3Modal and wagmi. When I call signTypedData() on desktop, it works fine. When I attempt to call it with Trust Wallet iOS, it does not produce the same signature.
Here is my _app.tsx.
import { EthereumClient, w3mConnectors, w3mProvider } from '@web3modal/ethereum'
import { Web3Modal } from '@web3modal/react'
import { configureChains, createClient, WagmiConfig } from 'wagmi'
import { mainnet } from 'wagmi/chains'

const chains = [mainnet]
const projectId = 'febf8d9986a2cd637fa4004338dad39b'

const { provider } = configureChains(chains, [w3mProvider({ projectId })])
const wagmiClient = createClient({
    autoConnect: true,
    connectors: w3mConnectors({ projectId, version: 1, chains }),
    provider,
})
const ethereumClient = new EthereumClient(wagmiClient, chains)
const queryClient = new QueryClient()

and here is my sign function:
import { disconnect as disconnectWeb3, signMessage, signTypedData } from "@wagmi/core";
...
...
const sig = await signTypedData({
    value: txn.eipToSign.message,
    types: txn.eipToSign.types,
    domain: txn.eipToSign.domain
});

Android works as well

As described in #3049, Trust Wallet is having trouble signing EIP712 signed data on iOS.
I am using Web3Modal and wagmi. When I call signTypedData() on desktop, it works fine. When I attempt to call it with Trust Wallet iOS, it does not produce the same signature.
Here is my _app.tsx.
import { EthereumClient, w3mConnectors, w3mProvider } from '@web3modal/ethereum'
import { Web3Modal } from '@web3modal/react'
import { configureChains, createClient, WagmiConfig } from 'wagmi'
import { mainnet } from 'wagmi/chains'

const chains = [mainnet]
const projectId = 'febf8d9986a2cd637fa4004338dad39b'

const { provider } = configureChains(chains, [w3mProvider({ projectId })])
const wagmiClient = createClient({
    autoConnect: true,
    connectors: w3mConnectors({ projectId, version: 1, chains }),
    provider,
})
const ethereumClient = new EthereumClient(wagmiClient, chains)
const queryClient = new QueryClient()

and here is my sign function:
import { disconnect as disconnectWeb3, signMessage, signTypedData } from "@wagmi/core";
...
...
const sig = await signTypedData({
    value: txn.eipToSign.message,
    types: txn.eipToSign.types,
    domain: txn.eipToSign.domain
});

Doesn't seem to be related to wallet-core library, converting to a discussion.


fix typo in MarvelGraphFactory.java

Hello awslabs team.
I found a typo on the MarvelGraphFactory.java.
Thanks.

merged, thanks for the catch.


Ship dissapears on mount

DavincisVessels version: 1.12-6.340
MovingWorld version: 1.12-6.342
Minecraft version: 1.12.2
Forge version <IP_ADDRESS>2

So I build a ship, consisting of just basic vanilla blocks (glass, helm, and air balloons). I assemble ship and mount it. When I mount it, the entire thing disappears. When I try to pass through the blocks where the ship was previously placed, I glitch and am unable to pass through them, as if the game believes that the blocks are still there, even though they are completely invisible.
This is the exact same bug that was posted on July 25 by RobOnTheJob, which is still an open problem. Worth noting I am using ChickenChunks, BiomesOPlenty, Treecapitator, and Journeymap, but I am not building the ship within a chunk loaded by the ChickenChunks mod, as I have seen on this issue page that there could be some bugs related to doing so. And even though I am using only vanilla blocks, for giggles, I went and built the ship in a vanilla biome as well as one of the biomes from the mod, with no luck.
I really hope you will try your best to fix this bug! This mod is so awesome for world exploration, and I wish you luck in finding the problem! In the meantime, I guess I'll go explore with a horse :(

same here i have the same problem to

found a fix increase your render distance

try that it work for me


Exploit runs but games won't play

Hello, I have an SCPH-102 (PAL) console. I have tried both the slot 2 images for this model (0bad7ea9 and 76b880e5 which are for BIOS 4.4 and 4.5, respectively) and the exploit runs and I can get into Unirom. However, when I try to play a game, I get
"Waiting for CD to become ready & unlocked. . . .
Not a valid CD or the drive is locked
Press X to exit."
The console still works with legit PAL games so the optical drive is functional. Any help appreciated.

Hi, I bought another PS1 console (this one is a SCPH-9002) and it has absolutely no problem using FreePSXBoot or TonyHax to play my backup discs. So there is definitely something specific to my PSone console that prevents the exploit working. I should have also mentioned that the PSone has no problem playing original black disc games.
The warranty sticker is intact. Any ideas?

Don't know what to say. I tested 4 different PAL PSone and never had an issue...

I solved the problem: it was a jammed lid sensor. I opened the console up and added some lubricant to the lid switch mechanism, and now it works fine :)


Add GitHub Actions CI and remove Travis CI

This is ready to review.

Bump @ararslan

Bump @StefanKarpinski


Fix pytorch 1.12 tensor reshaping for anchors

Fixes #274.

PR is ready for review.

@mayeroa thanks!


Missing singlestat2 plugin with Grafana v6.6.0

Tested with Grafana 6.6.0-ubuntu
Steps to reproduce:

Enable 'Simple App' on plugins page
Go to the 'stats' dashboard

Looks like 'singlestat2' has been renamed to 'singlestat' some time ago:
https://github.com/grafana/grafana/pull/20651
Live editing the dashboard to use 'singlestat' instead seems to be fixing the issue.

Ditto for graph2 plugin on the streaming dashboard.

This plugins are available only with config option enabled. You can activate it via runing grafana with env GF_PLUGINS_ENABLE_ALPHA=true


Update readme on how to properly build

bazel build //plugin does not do anything as promised such as cross reference between java and protos since it does not include the build target that handles it.
please at least update it to something like
bazel build --define=ij_product=intellij-latest //plugin

This is unmaintained, as the note in the readme says. Try my fork instead.

I see but why not maintain this to the extend that it builds for most of the cases?
the current state wont build for anyone. at least the initial version should fix the maven http deprecation issue and call out the exact build command (as people may not use bazel)
you also did not call out the exact build command in your fork.

sorry did not see you fixed the build problem of selecting ij_product in your fork.


TC.3.3 A method checks if the draw pile and all Hand Cards are empty (and therefore 100 Cards have been played)

a05f8557073cfce42e22af36cb4982408a5ce1b3

This task has been finished on the day with posted SHA comment, in the interest of correctly closing the issue (and us having to first learn how to), it has been closed a while after the finishing of the task.

We have in the meantime figured out, that we can not correctly close it once it is done (otherwise the commit is kinda useless).


VanillaJS live update instructions appear to be incomplete

The instructions for the the live update instructions do not mention what the values for callback and data are, or what they represent. Looking into the type definitions do not help either, as data essentially resolves to a Record<any, any> and callback is essentially (data: Record<any, any>) => void
<!DOCTYPE html>
<html lang="en">
  <head>
    <title>Live Preview Example</title>
    <link
      rel="stylesheet"
      type="text/css"
<EMAIL_ADDRESS>    />
    <script type="module">
      import { ContentfulLivePreview } from<EMAIL_ADDRESS>
      const locale = 'en-US';

      ContentfulLivePreview.init({ locale });

      /**
       * Subscribe to data changes from the Editor, returns a function to unsubscribe
       * Will be called once initially for the restored data
       */
      const unsubscribe = ContentfulLivePreview.subscribe({
        data, // <- This is undefined within this script
        locale,
        callback, // <- This is undefined within this script
      });
    </script>
  </head>
  <body></body>
</html>

I will reopen this ticket, we will add a vanilla JS example soon and update the docs.

Please have a look at the updated documentation for vanilla js and our new example in the examples directory :)


Initial setup

Initial setup for project.
Installs gems for testing and organizes Gemfile

Ebert has finished reviewing this Pull Request and has found:

3 possible new issues (including those that may have been commented here).
4 fixed issues! 🎉

You can see more details about this review at https://ebertapp.io/github/ruan-brandao/banking/pulls/1.

Ebert has finished reviewing this Pull Request and has found:

4 fixed issues! 🎉

You can see more details about this review at https://ebertapp.io/github/ruan-brandao/banking/pulls/1.


jets server in Cloud9

Checklist

[x] Upgrade Jets: Are you using the latest version of Jets? This allows Jets to fix issues fast. There's a jets upgrade command that makes this a simple task. There's also an Upgrading Guide: http://rubyonjets.com/docs/upgrading/
[x] Reproducibility: Are you reporting a bug others will be able to reproduce and not asking a question. If you're unsure or want to ask a question, do so on https://community.rubyonjets.com
[x] Code sample: Have you put together a code sample to reproduce the issue and make it available?  Code samples help speed up fixes dramatically. If it's an easily reproducible issue, then code samples are not needed. If you're unsure, please include a code sample.

My Environment

Software
Version

Operating System
Amazon Linux

Jets
1.8.5

Ruby
2.5.1

Expected Behaviour
I would like to run jets server --port 8080 in Cloud9 to run a preview version of the site before deploy.
Current Behavior
The local server fails with a NoMethodError on lib/jets/controller/middleware/main.rb: in setup, line 35
Step-by-step reproduction instructions

Start a new Cloud9 host
Install and use Ruby 2.5

rvm install 2.5.1 && rvm use 2.5.1 --default

Install jets

gem install jets

Install yarn

npm install -g yarn

Install mysql client

sudo yum install mysql-devel

Start new project

jets new demo

Start the server on port 8080 as required by Cloud9

jets server --port 8080

Code Sample
NoMethodError at /
undefined method `request' for nil:NilClass

Ruby | /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/jets-1.8.5/lib/jets/controller/middleware/main.rb: in setup, line 35

Traceback (innermost first)

    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/jets-1.8.5/lib/jets/controller/middleware/main.rb: in setup
              @controller.request.set_env!(@env)...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/jets-1.8.5/lib/jets/controller/middleware/main.rb: in call!
              setup...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/jets-1.8.5/lib/jets/controller/middleware/main.rb: in call
              dup.call!...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/jets-1.8.5/lib/jets/controller/middleware/main.rb: in call
              instance.call...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/bundler/gems/webpacker-3651cc317358/lib/webpacker/dev_server_proxy.rb: in perform_request
              @app.call(env)...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/rack-proxy-0.6.5/lib/rack/proxy.rb: in call
              rewrite_response(perform_request(rewrite_env(env)))...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/rack-2.0.6/lib/rack/etag.rb: in call
              status, headers, body = @app.call(env)...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/rack-2.0.6/lib/rack/conditional_get.rb: in call
                status, headers, body = @app.call(env)...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/rack-2.0.6/lib/rack/head.rb: in call
              status, headers, body = @app.call(env)...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/rack-2.0.6/lib/rack/session/abstract/id.rb: in context
                  status, headers, body = app.call(req.env)...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/rack-2.0.6/lib/rack/session/abstract/id.rb: in call
                  context(env)...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/jets-1.8.5/lib/jets/controller/middleware/local.rb: in call
                @app.call(env)...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/rack-2.0.6/lib/rack/method_override.rb: in call
              @app.call(env)...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/rack-2.0.6/lib/rack/runtime.rb: in call
              status, headers, body = @app.call(env)...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/jets-1.8.5/lib/jets/middleware.rb: in call
              stack.call(env)...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/rack-2.0.6/lib/rack/lint.rb: in _call
              status, headers, @body = @app.call(env)...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/rack-2.0.6/lib/rack/lint.rb: in call
              dup._call(env)...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/rack-2.0.6/lib/rack/show_exceptions.rb: in call
              @app.call(env)...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/rack-2.0.6/lib/rack/common_logger.rb: in call
              status, header, body = @app.call(env)...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/shotgun-0.9.2/lib/shotgun/loader.rb: in proceed_as_child
              status, headers, body = assemble_app.call(@env)...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/shotgun-0.9.2/lib/shotgun/loader.rb: in call!
                proceed_as_child...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/shotgun-0.9.2/lib/shotgun/loader.rb: in call
              dup.call!(env)...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/shotgun-0.9.2/lib/shotgun/favicon.rb: in call
                app.call(env)...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/shotgun-0.9.2/lib/shotgun/static.rb: in call
                @app.call(env)...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/rack-2.0.6/lib/rack/urlmap.rb: in block in call
                return app.call(env)...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/rack-2.0.6/lib/rack/urlmap.rb: in each
              @mapping.each do |host, location, match, app|...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/rack-2.0.6/lib/rack/urlmap.rb: in call
              @mapping.each do |host, location, match, app|...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/rack-2.0.6/lib/rack/builder.rb: in call
              to_app.call(env)...
    /home/ec2-user/.rvm/gems/ruby-2.5.1/gems/rack-2.0.6/lib/rack/handler/webrick.rb: in service
                status, headers, body = @app.call(env)...
    /home/ec2-user/.rvm/rubies/ruby-2.5.1/lib/ruby/2.5.0/webrick/httpserver.rb: in service
              si.service(req, res)...
    /home/ec2-user/.rvm/rubies/ruby-2.5.1/lib/ruby/2.5.0/webrick/httpserver.rb: in run
                  server.service(req, res)...
    /home/ec2-user/.rvm/rubies/ruby-2.5.1/lib/ruby/2.5.0/webrick/server.rb: in block in start_thread
                  block ? block.call(sock) : run(sock)

@lefthand Thanks for the details in the bug report. Addressed in #202.  Released in v1.8.6


Jax-ws client requires WSDL file at runtime ?

Has the following problem been solved which I copy from Jax-ws client requires WSDL file at runtime ? and bothering me a lot.

I generated client and server code for my wsdl using wsimport.
However the client code on startup is looking for the WSDL file in the location it existed in on the machine where I ran wsimport. I would imagine that with all the annotations in the generated code the client should not require the .wsdl at runtime ?
The problem is, when I take the standalone client code and distribute it to other machines, the client fails with the "Failed to access WSDL file exception".
Exception in thread "AWT-EventQueue-0" javax.xml.ws.WebServiceException: Failed to access the WSDL at: file:/C:/BackendManagerForGUI.wsdl. It failed with:
/C:/BackendManagerForGUI.wsdl (No such file or directory).
at com.sun.xml.ws.wsdl.parser.RuntimeWSDLParser.tryWithMex(RuntimeWSDLParser.java:162)
at com.sun.xml.ws.wsdl.parser.RuntimeWSDLParser.parse(RuntimeWSDLParser.java:144)
at com.sun.xml.ws.client.WSServiceDelegate.parseWSDL(WSServiceDelegate.java:254)
at com.sun.xml.ws.client.WSServiceDelegate.(WSServiceDelegate.java:217)
at com.sun.xml.ws.client.WSServiceDelegate.(WSServiceDelegate.java:174)
at com.sun.xml.ws.spi.ProviderImpl.createServiceDelegate(ProviderImpl.java:104)
at javax.xml.ws.Service.(Service.java:56)
I am creating a proxy using code that looks like:
            BackendManagerService bm = new BackendManagerService();

            port = bm.getBackendManagerPort();

            // Use Proxy Instance as BindingProvider
            BindingProvider bp = (BindingProvider) port;

            Map<String, Object> rc = bp.getRequestContext();
            rc.put (BindingProvider.ENDPOINT_ADDRESS_PROPERTY,
            "http://localhost:9077/BackManager");

Is there some other way for me to create the client service so that it does not require the .wsdl file at runtime ?

This mojo is just a wrapper around the JAX-WS toolset, so what you're asking about is about the classes generated by that tool (wsimport).
Having said that I solved this by including the wsdl in the jar and make the generated ws client use it during runtime by providing this mojo execution configuration:
    <configuration>
        <packageName>com.acme.foo.wsclient</packageName>
        <wsdlDirectory>${project.basedir}/src/main/resources</wsdlDirectory>
        <wsdlFiles>
            <wsdlFile>com/acme/foo/wsclient/BarService.wsdl</wsdlFile>
        </wsdlFiles>
        <wsdlLocation>BarService.wsdl</wsdlLocation>
        <xuseBaseResourceAndURLToLoadWSDL>true</xuseBaseResourceAndURLToLoadWSDL>
    </configuration>

The trick here is the combination of xuseBaseResourceAndURLToLoadWSDL and wsdlLocation. Do note that the wsdl file needs to be in the package path of the generated client.


Add Derek Featherstone's course "UX Foundations: Accessibility"

I guess this would go under "Private (paid)" but note that lynda.com has a 30-day free trial feature.
https://www.lynda.com/Accessibility-tutorials/Foundations-UX-Accessibility/435008-2.html

Hi @carmacleod would you like to make a PR to add this to the repo?  If not, I can add it.  Thanks for the suggestion!

resolved via this commit

Oops - sorry. Have been on vacation for a month - just got back today. Thanks for adding this!


HDF5 1.10 Compatibility w/ netCDF4

With the release of HDF5 1.10, we are faced with a couple issues.  First and foremost, netcdf4 files written using the 1.10 library (compiled with or without 1.10 API, it makes no matter) cannot be read by a libnetcdf compiled against HDF5 1.8.x.  This has serious implications for the Unidata datastreams, so long as there are nodes using 1.8 trying to read data generated by nodes using 1.10.
The full changes from HDF5 1.8 to 1.10 can be found here, at the time of this writing:

https://www.hdfgroup.org/HDF5/doc/ADGuide/Changes.html

The API compatibility report may be found here:

https://www.hdfgroup.org/HDF5/doc/ADGuide/Compatibility_Report/CR_1.10.0.html

49.2% incompatibility!
It appears that there may be command line tools (5hformat_convert) for re-writing the index data so that 1.10 generated files can be read by 1.8, but lit review is required to determine if this is true.  There is also a new function that is listed as "yet to be documented", e.g. H5[D/F]format_convert.
This issue is opened to track progress on how we mitigate these changes for our user community.

From some initial testing, it looks like the ninth byte of the Superblock (the version number of the superblock) has been changed with 1.10:
With version 1.8.15-patch1, the int value is 2.
With version 1.10.0, the int value is 3.
The first 8 bytes have stayed the same.
What does a superblock of 3 mean? Well, so far it's not documented, but at least it got incremented.
https://www.hdfgroup.org/HDF5/doc/H5.format.html#Superblock

It appears there is a compatibility mode in HDF5 1.10 for writing downward compatible netcdf-4 files.  Would someone please confirm my interpretation of the release documentation?  Please read under "Obtaining, Testing, and Experimenting with This Software" on this page:
https://www.hdfgroup.org/HDF5/docNewFeatures/

Files will be written in the older format if these two conditions are met:

Set write compatibility:
H5Pset_libver_bounds (file_id, low=H5F_LIBVER_EARLIEST_F, high=H5F_LIBVER_LATEST_F)

Avoid using the so-called "new features " in HDF5 1.10.  In other words, do not change current HDF5 function sequences for writing Netcdf-4 files.

--Dave

@Dave-Allured That is a good catch; I was hoping there would be something like that available once I'd had time for proper lit review.  I'll need to do some testing to determine the exact behavior of H5Pset_libver_bounds, but I believe you are reading it correctly.  We will need to add the explicit call to H5Pset_libver_bounds, assuming it works as expected.

I've created a new branch, gh250, and have roughed in a test that will see if netcdf can open a set of files which will have been generated by a netcdf library linked against hdf5 1.10.  These currently fail as expected when using libnetcdf+1.8.  This will help us in determining if/how/when we've fixed the problem.

I've got a tentative fix in place that anecdotally appears to work. Will have to do further tests but will try to have this out and in a proper release very quickly.

The base issue is fixed and merged into master.  @DennisHeimbigner is working on a patch which inserts metadata into the file (as a special attribute) which records the version of libhdf5 used to write the file.  Once that is done, we'll roll out the 4.4.1-rc1 release, and I'll close out this issue.  Leaving it open for now as a discussion hub.

Please close this issue #250.  This was resolved by release 4.4.1 on 2016 June 28, "quite a while ago".  Heh heh.  This does not need to show up on Milestone:Future in 2019 February.  ;-)


404 from - https://owner-api.teslamotors.com/oauth/token

Am I doing something obvious wrong?  Using a basic REST post I'm getting a 404

Make sure you're doing a POST.


question about disabling gradient bp

https://github.com/facebookresearch/fairseq/blob/main/fairseq/modules/ema_module.py#L41
Hi. It seems you forget this line, which may also disable gradient bp of the student encoder? Thx!

你好，这是石晓宇的邮箱。我会尽快处理该邮件

Hello Xiaoyu, thanks for your suggestion. I fixed the issue!


fix(installer): Allow --deploy-installer to run

What does this PR do?
https://github.com/DataDog/datadog-agent/pull/27740 forgot to add the if_deploy_installer rule to the workflow rules, preventing deploy when using --deploy-installer. This PR fixes it.
Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


[FEATURE REQUEST]

Please add some kind of blocking distracting sites!

Can you please explain this?

Closing this. Blocking distracting sites is not part of the scope of this project.


Error: Can't resolve all parameters for *: (?).

constructor(private heroService: HeroService) {} // NOT WORK

http://stackoverflow.com/questions/37997824/angular-2-di-error-exception-cant-resolve-all-parameters
commit: 79bdfdca

constructor(
  @Inject(HeroService)
  private heroService: HeroService // WORKS
) {}

http://stackoverflow.com/a/35371225/2760730
Use @Inject(HeroService) to inject Component and Service.


Replace eijebong's key with a smaller one

Apparently the builders don't like 16k keys

This change is

@bors-servo r+
Thanks!

:pushpin: Commit d527fb6 has been approved by edunham

:hourglass: Testing commit d527fb6dce51b540dffa2809f98c7664b096603d with merge 539e79ee4713e6ea1173854a0321d41bdaf70357...

:broken_heart: Test failed - status-travis

@bors-servo retry

:hourglass: Testing commit d527fb6dce51b540dffa2809f98c7664b096603d with merge ead05cf74383d57ac773949b499a1a3d5a2cbfda...

:sunny: Test successful - status-travis
Approved by: edunham
Pushing ead05cf74383d57ac773949b499a1a3d5a2cbfda to master...


update(OWNERS): add Luca Guerra (LucaGuerra) to owners

Signed-off-by: Luca Guerra<EMAIL_ADDRESS>What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind documentation
Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area proposals
What this PR does / why we need it:
As we discussed in yesterday's (Aug 3rd 2022) Falco community call and given the positive feedback from other current maintainers present, since I worked over the course of the past few months on new features, bugfixes, releases, security content, reviews and refactoring on the Falco libraries I would like to bring my contributions to this project to the next level by proposing myself as a maintainer.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: LucaGuerra
Once this PR has been reviewed and has the lgtm label, please assign molter73 for approval by writing /assign @molter73 in a comment. For more information see:The Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

The following users are mentioned in OWNERS file(s) but are untrusted for the following reasons. One way to make the user trusted is to add them as members of the falcosecurity org. You can then trigger verification by writing /verify-owners in a comment.

lucaguerra

User is not a member of the org. Satisfy at least one of these conditions to make the user trusted.

I wasn't present yesterday at the community meeting therefore i could not state my incredible joy in seeing this PR!
Luca is a great guy and an even better security oriented developer :closed_lock_with_key:  I am glad to see you joining us, and am sure that you will help us improve the libs with both PRs and reviews! Welcome on board :)
This is a big :+1: from me!

Also for me a huge +1 :heart_eyes:

+1 on my end as well, Luca makes really great contributions to the repo, we would benefit greatly from having him onboard!

LGTM label has been added.  Git tree hash: e32761d15bc557de8248c95ede1f08818326f4b5

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Big +1 me too. I know Luca and his skill. After his recent contribution, I'm totally convinced he deserves to be a maintainer.
PS
@LucaGuerra
Could you open a PR in test-infra to add yourself as a member of the org and to the libs-maintainers team, please?
https://github.com/falcosecurity/test-infra/blob/master/config/org.yaml#L19

/verify-owners

The following users are mentioned in OWNERS file(s) but are untrusted for the following reasons. One way to make the user trusted is to add them as members of the falcosecurity org. You can then trigger verification by writing /verify-owners in a comment.

lucaguerra

User is not a member of the org. Satisfy at least one of these conditions to make the user trusted.

/verify-owners


npz files

Hi, could you please share hdr_100_short_resolution_256.npz,hdr_100_medium_resolution_256.npz,hdr_100_long_resolution_256.npz and LOL_low_resolution_256.npz? Thanks! @Fayeben

Hi, you can modify imagenet_dataloader/save_random_images_hdr.py to obtain the .npz files.
And the .npz will be uploaded soon.

I have uploaded the .npz files. Please have a try!
https://drive.google.com/drive/folders/1Lvp6vjovXOa3vYv98evVSxsmGX_C9sgy?usp=sharing


[Bug] After oauth, redirect to web UI

Not sure if this is fixed here or in Auth0 settings, but after these steps:

Visit https://web.loren.temporal-dev.tmprl.cloud/signin
Click "Continue to SSO"
Select which Google account<EMAIL_ADDRESS>
I'm taken to https://myaccount.google.com/

this was addressed in recent versions, i'll keep the ticket open until the most recent version is published to cloud

https://github.com/temporalio/ui-server/pull/122

@feedmeapples We might want to double check. It was definitely present when @lorensr and I were pairing the other day and I don't think Loren was using a particularly old version, but maybe it shipped in v0.9.0, which shipped the morning before?

fixed in versions v0.10.1+


Move all.js to single .js files for schema/table/row

Move all.js to single .js files for schema/table/row to improve speed and avoid rendering useless chunks of js

The same for css


fix clippy errors for rust 1.74

If we are going to do this we should bump the clippy version checked in CI to make sure we don't reintroduce any new warnings..

Obviously if you are only going to fix the lints for the GUI, only bump the version in the CI for the GUI, not the daemon

Self-ACK 3196a8025ea9686cf76f407043b3ef2f85e0b4fa


Proc block for normalizing RGB images

In today's stand-up, @SamLeroux mentioned that this is a pretty common operation when you are working with images.
The current normalize proc block finds the overall min/max and normalizes each element individually instead of aggregating across a dimension, so we probably won't want to reuse that.

Quoting @SamLeroux from Slack:

It depends on how you trained the model:

The easiest option is to just divide every RGB value by 255 to obtain values between 0 and 1 but then you don't take the data distribution into account. If for your data, all the values are in a limited range, this might not be the best option
For images, we most often calculate the mean and standard deviation of the R, G and B channel independently for the training dataset. We then preprocess everything by subtracting the mean and dividing by the standard deviation.
Sometimes people also calculate a mean image. So they take the mean of all the training images and subtract this mean image from all images during training and testing

The input also doesn't have to be between 0 and 1, for the second option it will be between -3 and 3 more or less. Other people normalize it to [-1,1]. We just have to make sure that the data is preprocessed in exactly the same way during testing as it was during training


README.md: Update Executor section code

The code in the Executor section of README.md refers to the class DefaultLimiter which does not seem exist anymore.

REAME.md has been brought up to date.

Thanks! 👍


HBASE-23561 Look up of Region in Master by encoded region name is O(n)

This is my first PR using github.
If something is wrong. please let me know.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
2m  4s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-0 :warning:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
5m 37s
master passed

+1 :green_heart:
compile
0m 57s
master passed

+1 :green_heart:
checkstyle
1m  7s
master passed

+1 :green_heart:
shadedjars
4m 40s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 37s
master passed

+0 :ok:
spotbugs
4m 15s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
4m 14s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 59s
the patch passed

+1 :green_heart:
compile
0m 59s
the patch passed

+1 :green_heart:
javac
0m 59s
the patch passed

+1 :green_heart:
checkstyle
1m  8s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
4m 42s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
16m 10s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1 :green_heart:
javadoc
0m 38s
the patch passed

+1 :green_heart:
findbugs
4m 14s
the patch passed

_ Other Tests _

-1 :x:
unit
65m  8s
hbase-server in the patch failed.

+1 :green_heart:
asflicense
0m 35s
The patch does not generate ASF License warnings.

124m 29s

Subsystem
Report/Notes

Docker
Client=19.03.6 Server=19.03.6 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1193/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1193

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 4718f2be10ea 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1193/out/precommit/personality/provided.sh

git revision
master / fe0d49bc52

Default Java
1.8.0_181

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1193/1/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1193/1/testReport/

Max. process+thread count
9750 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1193/1/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 33s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-0 :warning:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
5m 35s
master passed

+1 :green_heart:
compile
0m 57s
master passed

+1 :green_heart:
checkstyle
1m  9s
master passed

+1 :green_heart:
shadedjars
4m 41s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 39s
master passed

+0 :ok:
spotbugs
4m 18s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
4m 15s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 58s
the patch passed

+1 :green_heart:
compile
0m 57s
the patch passed

+1 :green_heart:
javac
0m 57s
the patch passed

+1 :green_heart:
checkstyle
1m  4s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
4m 43s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
16m  2s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1 :green_heart:
javadoc
0m 37s
the patch passed

+1 :green_heart:
findbugs
4m 26s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
64m 58s
hbase-server in the patch passed.

+1 :green_heart:
asflicense
0m 38s
The patch does not generate ASF License warnings.

122m 46s

Subsystem
Report/Notes

Docker
Client=19.03.6 Server=19.03.6 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1193/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1193

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 15416d7d2cb0 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1193/out/precommit/personality/provided.sh

git revision
master / fe0d49bc52

Default Java
1.8.0_181

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1193/2/testReport/

Max. process+thread count
9839 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1193/2/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

Got a green, let me merge this PR. Thanks for the contribution, @mwkang !

@saintstack I think it looks okay. However, I am not sure only a single thread accessing them. And as you have already mentioned, If I want to synchronize two maps, I should use synchronize or locks. It was my mistake. I didn't think about it then. Should do I use synchronization or locks?

I think the current code may have different values for the two maps.
What do you think about changing it like below?
RegionStateNode node = regionsMap.computeIfAbsent(regionInfo.getRegionName(),key -> new RegionStateNode(regionInfo, regionInTransition));
encodedRegionsMap.putIfAbsent(node.getRegionInfo().getEncodedName(), node);
return node;


Iterable equality within object

Summary
Fixes #8280  iterableEquality ignores other properties within inequal objects.

This PR solves an issue in iterableEquality for iterators within objects with other properties.
Test plan

Added test for iterable within enequal objects
Added test for iterable within equal objects

  iterableEquality
    ✓ returns true when given circular iterators (3ms)
    ✓ returns true when given circular Set
    ✓ returns true when given nested Sets (1ms)
    ✓ returns false when given inequal set within a set
    ✓ returns false when given inequal map within a set (1ms)
    ✓ returns false when given inequal set within a map
    ✓ returns true when given iterator within equal objects (1ms)
    ✓ returns false when given iterator within inequal objects
    ✓ returns false when given iterator within inequal nested objects
    ✓ returns true when given circular Set shape (1ms)
    ✓ returns true when given circular key in Map (3ms)
    ✓ returns true when given nested Maps
    ✓ returns true when given circular key and value in Map
    ✓ returns true when given circular value in Map

By the way, don't forget to add a changelog entry for this fix :)

@jeysal I used Object.entries which is not supported by node@6. should i change that or should i wait for #8346 ?

@jeysal I used Object.entries which is not supported by node@6

Oh right, damn Node 6 is old 😁 Yeah even if Node 6 is EOL soon, we can only drop support for it in a major release and we might want some minors / patches first, so I guess it's better to not break it already.

Codecov Report

Merging #8359 into master will increase coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #8359      +/-   ##
==========================================
+ Coverage   62.22%   62.25%   +0.03%
==========================================
  Files         266      266
  Lines       10720    10722       +2
  Branches     2608     2609       +1
==========================================
+ Hits         6670     6675       +5
+ Misses       3463     3460       -3
  Partials      587      587

Impacted Files
Coverage Δ

packages/expect/src/utils.ts
97.35% <100%> (+0.1%)
:arrow_up:

packages/expect/src/matchers.ts
95.5% <0%> (-0.34%)
:arrow_down:

packages/jest-util/src/createProcessObject.ts
80% <0%> (+6.66%)
:arrow_up:

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 0f77332...3cd2b4e. Read the comment docs.

@d7my11 Thank you for clear solution to this problem. The questions that I asked say more about current excess complication in jasmineUtils.ts and utils.ts than about your work.

@d7my11 ping 🙂

anything i can do to get this over the line, the current behavior is surprising (and led to lots of bad tests i didn't know about) and hard to work around.


consider creating a C/C++ SDK

To keep things simpler we can compile the golang SDK as an embeddable module that a C/C++ environment can use, and create some wrappers around it maybe.

Could you assign to me, I want to try it @v0lkan

Assigned.
I’ll add more details sometime this week, hopefully.

reassigned :)

The idea is to reutilize the go code by using a module in the c code.
But we can also explore using https://github.com/HewlettPackard/c-spiffe


Remove kimchi premium and add Kucoin quoter

https://github.com/terra-money/oracle-feeder/blob/db8b36ef55bb0a0f8e3c23e78d620737eeb3a921/price-server/src/prices.ts#L31-L35
The filter is enforcing to select USD/USDT, but the calculation expects USDT/USD.
Maybe L31 should use getBaseCurrency?

https://github.com/terra-money/oracle-feeder/blob/db8b36ef55bb0a0f8e3c23e78d620737eeb3a921/price-server/src/prices.ts#L31
Seems current cryptoProvider config does not contains any some/USDT

https://github.com/terra-money/oracle-feeder/blob/db8b36ef55bb0a0f8e3c23e78d620737eeb3a921/price-server/src/prices.ts#L29-L31
Seems current cryptoProvider config does not contain any some/USDT.

It is not required by default. This feature is made for special cases.


release/3.16.9-quad.1.0.0

This version should integrate all the code produce for the QUAD pipeline up to version 3.15.12-quad.3.14.1.
Softwares that need tests are:

After Effects (Wizz stuffs)
Maya (Symlink, Action Builder, Machine Limits, Limit Groups)
Nuke (OpenRV)
OpenRV
Photoshop (Wizz stuffs)
TVPaint (Wizz stuffs)

Also:

builder plugin instances (collect_builder_action_paths)
RVActionReview
ExtractThumbnail fixes for ftrack

@Guilhemz please regenerate in the right folder the extension.zxp for After effects

@Guilhemz please regenerate in the right folder the extension.zxp for After effects

It's done !


Add  to all pom.xml files

Is your feature request related to a problem? Please describe.
Most pom.xml files do not have a <description> tag.
This can be a problem e.g. when Maven Central is restrictive. For example, with strict checking JReleaser would fail:
[ERROR]     [mavenCentral] PomChecker detected errors in operaton-cmmn-model-1.0.0-beta-2.pom
[ERROR] <description> is not defined in POM. Will use value from parent:
        Provides general dependencies and profiles for database testing
<url> is not defined in POM. Will use computed value from parent:

Describe the solution you'd like
Add the description tag for each pom.xml file. Ideally with meaningful content, but could be also ${project.name} by default.

Hey, I have added the  tag to all pom.xml files. This is my Pull-Request. https://github.com/operaton/operaton/pull/298


login to snapcraft when releasing

Codecov Report

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

@@           Coverage Diff           @@
##           master      #99   +/-   ##
=======================================
  Coverage   23.68%   23.68%
=======================================
  Files          13       13
  Lines        1081     1081
=======================================
  Hits          256      256
  Misses        813      813
  Partials       12       12

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 dc87228...a2abab0. Read the comment docs.


Update raven and other minor version updates for libraries.

Coverage remained the same at 65.228% when pulling 0afb432e7d92a599c8e7e2bf9ce9e810d1b9b649 on do/OCW-28 into bdd5df332f79b37b83e869c745da479fb80a16f0 on master.


CPU, Memory, Disk unusual occupied on Win10.

What OS are you using (uname -a, or Windows version)?

What programming language are you using (C/C++/Go/Rust)?

I tried the demo follow the Getting started, but each time I startup the output program, I just got a white page, and then my PC would  frozen. I found the occupied of CPU, Memory and Disk has risen obviously.
Once startup the program:

After killed the program:

Here the source file, the .dll I download, and the output binary:
webview_example.zip

I tried the recommend tdm-gcc, still not works.

Same problem here.
OS: Microsoft Windows [Version 10.0.19041.572]
Compiler: gcc (tdm64-1) 9.2.0
go version:  go1.15.3 windows/amd64
After I launch the .exe the ram usage spikes at 15gb, sdd usage 100% and gpu usage 60%. After closing the window is impossible to launch again the program for a couple of minutes until the disk usage goes to 0.
I suspect that the disk activity is caused by the usage of the paging file (seems like 15gb are not enough).
Tried the example in a virtual machine and I'm getting the same behaviour

I seem to get the same CPU/RAM problem with other Hints as well. HintMax for sure and HintFixed I think as well. The only one that works is HintMin. And there is another strange behavior. The window initially comes up with a certain size, but as soon as I click the lower right corner to resize it, it "jumps" in size to something larger in width and height by a few hundred pixels...

Same thing happened to me. See #487.


Starships: Cheyenne-class is missing

So, yeah. as the title says, I'm not seeing the Cheyenne-class in the stock spaceframes on the Starfleet side, and it is in the Utopia Planitia book, just before its cousin, the Springfield-class.

Yup, that's missing all right.


Update build-container.sh and run-container.sh to fix errors which occur if path contains spaces

Fixed an issue where user would get an error if the WORKDIR contained spaces.

Thank you for this!


log_sklearn_plot: update templates

Fixes #424

Codecov Report
Base: 96.13% // Head: 96.19% // Increases project coverage by +0.05% :tada:

Coverage data is based on head (1c423a5) compared to base (ac51460).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #442      +/-   ##
==========================================
+ Coverage   96.13%   96.19%   +0.05%
==========================================
  Files          37       37
  Lines        2201     2232      +31
  Branches      184      187       +3
==========================================
+ Hits         2116     2147      +31
  Misses         49       49
  Partials       36       36

Impacted Files
Coverage Δ

src/dvclive/plots/sklearn.py
92.18% <ø> (ø)

tests/test_dvc.py
100.00% <ø> (ø)

src/dvclive/live.py
90.62% <100.00%> (+0.18%)
:arrow_up:

src/dvclive/studio.py
94.28% <100.00%> (+0.34%)
:arrow_up:

tests/test_main.py
100.00% <100.00%> (ø)

tests/test_studio.py
100.00% <100.00%> (ø)

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Windows Services Processes

I would like to see a button or something i can click to stop or remove the services on my computer that i don't need. for example someone who doesn't know what is ok to stop and leave on and running to get those process down without breaking windows completely but to get down to a minimal process and still keep the security aspect.

This is added and have been tweaking it for a normal user.