Return proper LogoutResponse to the IdP in case of failure in SAML IdP-initated logout

This is a follow-up to soon-to-be-merged #20038. SAMLAuthBackend.process_logout doesn't return LogoutResponse in case of failure like demanded by the spec:
https://docs.oasis-open.org/security/saml/v2.0/saml-core-2.0-os.pdf

The recipient of a  message MUST respond with a  message, of
type StatusResponseType, with no additional content specified.

That's due to python3-saml OneLogin_Saml2_Logout_Response only supporting building a success response:
    def build(self, in_response_to):
        """
        Creates a Logout Response object.
        :param in_response_to: InResponseTo value for the Logout Response.
        :type in_response_to: string
        """
        sp_data = self._settings.get_sp_data()

        self.id = self._generate_request_id()

        issue_instant = OneLogin_Saml2_Utils.parse_time_to_SAML(OneLogin_Saml2_Utils.now())

        logout_response = OneLogin_Saml2_Templates.LOGOUT_RESPONSE % \
            {
                'id': self.id,
                'issue_instant': issue_instant,
                'destination': self._settings.get_idp_slo_response_url(),
                'in_response_to': in_response_to,
                'entity_id': sp_data['entityId'],
                'status': "urn:oasis:names:tc:SAML:2.0:status:Success"
            }

        self._logout_response = logout_response

This can be reasonably easily implemented (though may benefit from some refactoring of associated SAMLReponse/LogoutRequest-handling code first) by subclassing and implementing our own build method supporting failure responses.
Then, one can look at OneLogin_Saml2_Auth.process_slo to see how a full, valid response should be built and returned:
                in_response_to = logout_request.id
                self._last_message_id = logout_request.id
                response_builder = self.logout_response_class(self._settings)
                response_builder.build(in_response_to)
                self._last_response = response_builder.get_xml()
                logout_response = response_builder.get_response()

                parameters = {'SAMLResponse': logout_response}
                if 'RelayState' in self._request_data['get_data']:
                    parameters['RelayState'] = self._request_data['get_data']['RelayState']

                security = self._settings.get_security_data()
                if security['logoutResponseSigned']:
                    self.add_response_signature(parameters, security['signatureAlgorithm'])

                return self.redirect_to(self.get_slo_response_url(), parameters)

Section <IP_ADDRESS>  Element <StatusCode> of https://docs.oasis-open.org/security/saml/v2.0/saml-core-2.0-os.pdf has details on the available status codes which should be used to communicate the failure.

https://github.com/onelogin/python3-saml/pull/288 if merged, will let us do this without needing to subclass OneLogin_Saml2_Logout_Response

@mateuszmandera is this issue something we can fix easily? The linked upstream PR does look to have been merged.


Configure pytest to use all processors by default

I modified the pytest configuration to run the unit tests in parallel by default, using as many processors as available.  This significantly speeds up running the unit tests on a local multi-core machine.
I'm opening this PR to see if CI (Azure) is ok with this change!

Interesting!  Some of the CI jobs fail.  I wonder if there is a concurrency problem which needs fixing inside Lightkurve.


x/tools/gopls: add unit tests for snapshot.clone

It could also probably benefit from being refactored -- it is getting very long and complicated.
See https://golang.org/cl/258121.

I factored out some of the cloning logic into cache/workspace.go, which has unit tests.  We could probably carve it up a bit more though.

This is unfeasible without a refactoring, so let's not keep this issue around.


first login event

In the event "on player first login" or "on player logs in for the first time", the "first time" part is not working

How are you testing that?
You realize you basically just about need to get a new user onto your server for each and every testing of it?
(Or resetting all the server data)

I can confirm, that the event works correctly since I have new players on a daily basis and log data.

Is this related to file saving or something ? If a file is not save when I shutdown the test server then the server won't know that it is not a new player

Had done multiple testing and it work fine


Invalid warnings for section titles out of sequence

I have an Asciidoctor setup producing HTML, EPUB3 and PDF output via Maven. Both HTML and PDF work flawlessly. However, the EPUB3 output spits out warnings about allegedly out of sequence section titles:
[INFO] --- asciidoctor-maven-plugin:1.5.3:process-asciidoc (html) @ spring-data-commons ---
[INFO] Rendered /Users/olivergierke/Documents/workspace/spring-data-commons/src/main/asciidoc/index.adoc
[INFO]
[INFO] --- asciidoctor-maven-plugin:1.5.3:process-asciidoc (epub) @ spring-data-commons ---
asciidoctor: WARNING: dependencies.adoc: line 49: section title out of sequence: expected level 1, got level 2
asciidoctor: WARNING: dependencies.adoc: line 54: section title out of sequence: expected level 1, got level 2
asciidoctor: WARNING: repositories.adoc: line 16: section title out of sequence: expected level 1, got level 2
asciidoctor: WARNING: repositories.adoc: line 104: section title out of sequence: expected level 1, got level 2
asciidoctor: WARNING: repositories.adoc: line 181: section title out of sequence: expected level 1, got level 2
asciidoctor: WARNING: repositories.adoc: line 215: section title out of sequence: expected level 1, got level 2
asciidoctor: WARNING: repositories.adoc: line 398: section title out of sequence: expected level 1, got level 2
asciidoctor: WARNING: repositories.adoc: line 481: section title out of sequence: expected level 1, got level 2
asciidoctor: WARNING: repositories.adoc: line 631: section title out of sequence: expected level 1, got level 2
asciidoctor: WARNING: auditing.adoc: line 5: section title out of sequence: expected level 1, got level 2
asciidoctor: ERROR: index.adoc: line 26: invalid part, must have at least one section (e.g., chapter, appendix, etc.)
asciidoctor: WARNING: repository-namespace-reference.adoc: line 6: section title out of sequence: expected level 1, got level 2
asciidoctor: WARNING: repository-populator-namespace-reference.adoc: line 6: section title out of sequence: expected level 1, got level 2
asciidoctor: WARNING: repository-query-keywords-reference.adoc: line 5: section title out of sequence: expected level 1, got level 2
asciidoctor: WARNING: repository-query-return-types-reference.adoc: line 5: section title out of sequence: expected level 1, got level 2
asciidoctor: ERROR: index.adoc: line 35: invalid part, must have at least one section (e.g., chapter, appendix, etc.)
[INFO] Rendered /Users/olivergierke/Documents/workspace/spring-data-commons/src/main/asciidoc/index.adoc
[INFO]
[INFO] --- asciidoctor-maven-plugin:1.5.3:process-asciidoc (pdf) @ spring-data-commons ---
[INFO] Rendered /Users/olivergierke/Documents/workspace/spring-data-commons/src/main/asciidoc/index.adoc

Steps to reproduce:

git clone https://github.com/spring-projects/spring-data-commons
cd spring-data-commons
mvn clean generate-resources -Pdistribute

Versions in use:

Asciidoctor Maven plugin 1.5.3
AsciidoctorJ PDF 1.5.0-alpha.11
AsciidoctorJ EPUB3 1.5.0-alpha.6

This is very closely related to #47. At the moment, Asciidoctor EPUB3 enforces a very strict structure described here: https://github.com/asciidoctor/asciidoctor-epub3#declaring-the-spine. We want to lift this requirement, which is the focus of #47.

But the document is well structured, it just uses level offsets on imports?

The spine document cannot contain any content other than include directives. I think that is the problem. Again, I want to get rid of this restriction...that's just how it works currently.

I'm double checking that assessment.

Okay, thanks for clarifying.

First, for the Preface to show correctly, you need a blank line after the = Preface line in the preface.adoc.
Second, the content in the spine document outside of includes is simply ignored. It doesn't break the converter in anyway.
Finally, the problem is that the includes must start with a level-0 section. The problem right now is that you are using leveloffset: +1, which pushes it down to level-1 section. Take this away as well as the section titles in index.adoc and it will work.
The other converters need the chapter sections to be a level-1, so this is the setting I recommend in the header of the spine document.
ifndef::ebook-format[:leveloffset: 1]

I'd probably then exclude the parts from the EPUB3 as well using:
ifndef::ebook-format[]
[[reference-documentation]]
= Reference documentation
endif::[]

or as a one-liner:
ifndef::ebook-format[= Reference documentation [[reference-documentation]]]

Here's how it looks when I finished editing it:
= Spring Data Commons - Reference Documentation
Oliver Gierke; Thomas Darimont; Christoph Strobl; Mark Pollack; Thomas Risberg
:revnumber: {version}
:revdate: {localdate}
:toc: macro
:doctype: book
ifndef::ebook-format[:leveloffset: +1]

(C) 2008-2015 The original authors.

NOTE: Copies of this document may be made for your own use and for distribution to others, provided that you do not charge any fee for such copies and further provided that each copy contains this Copyright Notice, whether distributed in print or electronically.

ifdef::basebackend-html[toc::[]]

include::preface.adoc[]

ifndef::ebook-format[= Reference documentation [[reference-documentation]]]

include::dependencies.adoc[]
include::repositories.adoc[]
include::auditing.adoc[]

ifndef::ebook-format[= Appendix [[appendix]]]

:numbered!:
include::repository-namespace-reference.adoc[]
include::repository-populator-namespace-reference.adoc[]
include::repository-query-keywords-reference.adoc[]
include::repository-query-return-types-reference.adoc[]

With that new index, you can generate both a PDF and EPUB3 with no warnings.

If you still want the parts "Reference documentation" and "Appendix" in the EPUB3, you'll need to create include files for them and use includes in those files to include the chapters. Except the chapters will then become sections.
This turns out to be a fairly helpful use case because it allows us to see all the dimensions that need to be considered when creating the EPUB3 divisions.

Also, imagesdir must be set to images, which is this issue: #22. Otherwise, you get a broken image in the chapter header.

any update on this?

I think I explained how to solve the problem. Is there a question you have about my explanation?

@mojavelinux , oh, I thought your explanation was more of a workaround, hence this issue is still open. (should this issue be closed?)

Yes. Since this is addressed in the documentation, I will close this issue. If there is something that needs clarification, please reopen with a new request.


intellij hangs with playfw + scala + morphia

I have a new project with modules crud, docviewer, grizzly, scalal-0.9.1, secure, testrunner
When I add morphia-1.2.1beta6 and set the log level to TRACE, the startup hangs after:
22:47:47,562 INFO  ~ Precompiling ...
Compiling:
[...]\modules\secure\app\controllers\Check.java
[...]\modules\docviewer\app\controllers\PlayDocumentation.java
[...]\modules\testrunner\app\controllers\TestRunner.java
[...]\modules\secure\app\controllers\Secure.java
[...]\modules\docviewer\app\DocViewerPlugin.java
[...]\modules\morphia-1.2.1beta6\app\morphia\ObjectIdBinder.java
[...]\modules\docviewer\app\helpers\CheatSheetHelper.java
[...]\modules\crud\app\controllers\CRUD.java
When I remove morphia, the Server launches as expected. This is the very first try with playfw and it's kind of frustrating, unfortunately...

Well, we switched to Eclipse and dropped Scala, so it's working now, besides some unwelcome and therefore hard to understand Blob-Getter-Mangling. We really would prefer to manually add annotations instead of guessing what kind of wrappers morphia is misteriously applying to our methods, but that seems to be a matter of taste. Overall, it's fun working with playfw + morphia!

which PlayMorphia version are you using? The latest 1.2.4 will not enhance
your blob getter methods, instead it loads blob field from gridfs along
with the entity
On Tue, Nov 15, 2011 at 8:14 AM, Searle <
<EMAIL_ADDRESS>
wrote:

Well, we switched to Eclipse and dropped Scala, so it's working now,
besides some unwelcome and therefore hard to understand
Blob-Getter-Mangling. We really would prefer to manually add annotations
instead of guessing what kind of wrappers morphia is misteriously applying
to our methods, but that seems to be a matter of taste. Overall, it's fun
working with playfw + morphia!

Reply to this email directly or view it on GitHub:
https://github.com/greenlaw110/play-morphia/issues/31#issuecomment-2736658


Add method to clear persistent storage

Possible solution to an issue we discussed briefly in Discord — our application registers an extension, and displays a button allowing users to save their changes to a file. We enable persistent storage in case the page crashes or reloads before changes have been saved. In some cases the user may decide not to save the changes they've made since the last save, in which case we need a way to clear persistent storage.
I've tried just clearing all localstorage entries with the right prefix and then reloading, but then theatre.js is doing one last commit to localstorage before the page reloads and the cache persists. This PR would expose a new method, studio.clearPersistentStorage(), which could be used by userland code or extensions.
This contribution is funded by The New York Times.

Thanks @donmccurdy! Just note that I've renamed the API to: studio.__experimental.__experimental_clearPersistentStorage(persistenceKey)

I wonder if an alternative to this would be to add a config param when initializing studio to disable persistent storage? there are some cases, I imagine, where storing and restoring state is managed in userland.


Button improvements

Add a class name to the buttons, so that they can be styled. This helps users customize the buttons for their app (In my case, I had to adjust the CSS layout due to Bootstrap's CSS reset). Also helps mitigate #13.
Apply the drop shadow without a prefix as well, for Firefox.

Thanks!


[Fix #12989] Fix an error for the inherit_gem config when the Gemfile contains an uninstalled git gem

I also move the bundler isolation into the shared context and add a test for it
Replace this text with a summary of the changes in your PR.
The more detailed you are, the better.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[ ] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Let's wait for feedback from OP in the issue (or here). I don't know how to test this in the context of that tool

Seems fine: https://github.com/rubocop/rubocop/issues/12989#issuecomment-2166064469

Thanks!


SUGGESTION: allow 3rd level to be snippets?

Hi
im loving using yankpad after the complexity of yasnippet :) and also since im an orgmode fanboy this is great for me ;-)
one thing i notice while transitioning to yanpad from yasnippet is that i find myslef wanting sub levles in a category. for example in org mode i have > 50 snippets but some are for uni some for cooking some for home etc.
will it be possible perhaps with a user defined option to make yankpad consider lv1 as category, igonre lv2 and use lv3 headers as snippets?
best
Z

Fair suggestion. I've made that configurable now. For now it is global though, so all snippets have to be at the same level (you can not have level 2 snippets in one category and level 3 snippets in another).
Add (setq yankpad-snippet-heading-level 3) to your init-file to try it out. You can also try to change yankpad-category-heading-level if you like.


Should try babel-core/register first, then babel/register

From Setting up Babel 6:

The babel package is no more. Previously, it was the entire compiler and all the transforms plus a bunch of CLI tools, but this lead to unnecessarily large downloads and was a bit confusing. Now we’ve split it up into two separate packages: babel-cli and babel-core.

Where is it recommended to use babel?

We do that already. Update your installation.

Both local and global versions are latest.
[11:43:24] Failed to load external module babel-register
[11:43:24] Requiring external module babel-core/register
[11:43:24] CLI version 1.0.0
[11:43:24] Local version 3.9.0

But now that I look at it, I see that you're first loading babel-register (not babel/register), I have no idea what that is, so I'll shut up.

It's babel-core/register: https://github.com/babel/babel/blob/master/packages/babel-core/register.js

babel-register should be supported once #1456 has been released.

babel-core/register is already out of date. Babel is recommending you use babel-register package moving forward.

@duereg: Read the PR linked in the comment above yours and check out interpret: https://github.com/js-cli/js-interpret


Add useful macros for merging request parsers

As per https://github.com/Metaswitch/swagger-rs/pull/74 but for v3 / master.

bors r+


Enable / disable selection of multiple items

Function to enable / disable selection of multiple items in the is-row-checkable property.
Can you validate that it is correct?

Hi,
Thank you for your contribution!
Enable/disable selection is a great addition, it would be even better if we can remove checkboxes completely so that the user is less confused. Maybe something like this:
:is-row-checkable="(row) => row.type != 'back'"
:checkable="can('batchDownload') || can('write') || can('zip')"

However, design changes are not consistent and somehow duplicated:

Yes! Removing the checkbox was our first attempt. Unfortunately the component does not work. The other change, to show the title, we submitted incorrectly.


X Ray Image Query produces NaN data

The query is producing NaN data for me in a toy problem I have developed. This is the intensities image from the blueprint output:

This is the corresponding blueprint yaml:
fields:
  intensities:
    topology: "image_topo"
    association: "element"
    units: "no units provided"
    values: [0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, "-nan", "-nan", "-nan", "-nan", 0.0, 0.0, 0.0, 0.0, "-nan", "-nan", "-nan", "-nan", 0.0, 0.0, 0.0, 0.0, "-nan", "-nan", "-nan", "-nan", 0.0, 0.0, 0.0, 0.0, "-nan", "-nan", "-nan", "-nan", 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0]
    strides: [1, 8, 48]

I will continue to investigate and see if this issue is caused upstream or is specific to the blueprint output. Here is a reproducer:
OpenDatabase("./testdata/silo_hdf5_test_data/globe.silo")

DefineScalarExpression("u1", 'recenter(((u+10.)*0.01), "zonal")')
DefineScalarExpression("v1", 'recenter(((v+10.)*0.01*matvf(mat1,1)), "zonal")')
DefineScalarExpression("v2", 'recenter(((v+10.)*0.01*matvf(mat1,2)), "zonal")')
DefineScalarExpression("v3", 'recenter(((v+10.)*0.01*matvf(mat1,3)), "zonal")')
DefineScalarExpression("v4", 'recenter(((v+10.)*0.01*matvf(mat1,4)), "zonal")')
DefineScalarExpression("w1", 'recenter(((w+10.)*0.01), "zonal")')

DefineScalarExpression("tets",     'matvf(<mat1>, "3")')
DefineScalarExpression("pyramids", 'matvf(<mat1>, "4")')

AddPlot("Pseudocolor", "tets")
DrawPlots()

HideActivePlots()

AddPlot("Pseudocolor", "pyramids")
DrawPlots()

AddOperator("Threshold", 1)
ThresholdAtts = ThresholdAttributes()
ThresholdAtts.outputMeshType = 0
ThresholdAtts.boundsInputType = 0
ThresholdAtts.listedVarNames = ("default")
ThresholdAtts.zonePortions = (1)
ThresholdAtts.lowerBounds = (0.001)
ThresholdAtts.upperBounds = (1e+37)
ThresholdAtts.defaultVarName = "mat1"
ThresholdAtts.defaultVarIsScalar = 0
ThresholdAtts.boundsRange = ("0.001:1e+37")
SetOperatorOptions(ThresholdAtts, 0, 1)
DrawPlots()

params = dict()
params['output_type'] = "hdf5"
params['image_size'] = (8, 6)
params['divide_emis_by_absorb'] = 1
params['focus'] = (0., 0., 0.)
params['view_up'] = (-0.0651, 0.775, 0.628)
params['normal'] = (-0.840, -0.383, 0.385)
params['view_angle'] = 30.
params['parallel_scale'] = 12.5
params['near_plane'] = -25
params['far_plane'] = 25
params['image_pan'] = (0., 0.)
# params['image_zoom'] = 8
params['perspective'] = 1
params['vars'] = ("v2", "v1")
Query("XRay Image", params)

import conduit
n = conduit.Node()
conduit.relay.io.blueprint.load_mesh(n, "output.root")
print(n["domain_000000"].to_yaml())

DeleteAllPlots()

OpenDatabase("output.root")
AddPlot("Pseudocolor", "mesh_image_topo/intensities")
DrawPlots()

FWIW, as a numerical tool, I think VisIt could do a better job handling FPEs (floating point exceptions) of all sorts (see #1680) and providing users way of controlling behavior in the presence of FPEs better.

I have done some sleuthing and arrived at the problem. I didn't realize I had divide_emis_by_absorb turned on; it took me getting all the way into the code to realize. However, let's say a user turns that on and they have a zone or zones where the absorbtivity and emissivity are both 0.0. They are divided in this calculation: intensityBins[j] = intensityBins[j] * tmp + (e[j] / a[j]) * (1.0 - tmp);. Is producing NaN in the query output acceptable? Should there be an error? Or just set the result to 0? Something else?

@JustinPrivitera if divide_emis_by_absorb is an expression function, I think it should be adjusted to use the divide() expression function...
divide(<val_numerator>, <val_denominator>, [<div_by_zero_value>, <tolerance>])

If instead, its C++ code, then thats harder. I did a search on divide by zero in our repo and see a lot of cases where we had to go adust C++ code to avoid a divide-by-zero. It would be better if we handled that it a more general way perhaps by overloading operator/() for various types and then having a preference for what to do with the result for the bad cases.

Yes this is C++ code. I wonder if @cyrush would have anything to say about knowingly dumping NaNs into Blueprint data?

Spoke to @cyrush about this and we determined that the best answer for now is to put a note in the documentation that cautions against this case.


ERROR in training

i just have 1 gpu
&
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2016 NVIDIA Corporation
Built on Tue_Jan_10_13:22:03_CST_2017
Cuda compilation tools, release 8.0, V8.0.61
the problem ONE is :
(1)i change the N_GPU in scripts/train_tsn.sh to be 1 ,and change the batch_size in models/ucf101/tsn_bn_inception_rgb_train_val.protoxt. to be 128
(2) run
bash scripts/train_tsn.sh ucf101 rgb
then i get the error:
logging to logs/ucf101_rgb_split1.log
It looks like MPI_INIT failed for some reason; your parallel process is
likely to abort.  There are many reasons that a parallel process can
fail during MPI_INIT; some of which are due to configuration or environment
problems.  This failure appears to be an internal failure; here's some
additional information (which may only be relevant to an Open MPI
developer):
ompi_mpi_init: ompi_rte_init failed
--> Returned "(null)" (-43) instead of "Success" (0)
*** An error occurred in MPI_Init_thread
*** on a NULL communicator
*** MPI_ERRORS_ARE_FATAL (processes in this communicator will now abort,
***    and potentially your MPI job)
[sandy:30438] Local abort before MPI_INIT completed completed successfully, but am not able to aggregate error messages, and not able to guarantee that all other processes were killed!
mpirun noticed that the job aborted, but has no info as to the process
that caused that situation.
TRY AGAIN: i make the try as:
i change MPI_BIN_DIR=/home/lalala/.openmpi/bin/  in  scripts/train_tsn.sh ,the N_GPU=1 & batch_size=128, but i also get  error:
行 10: /home/sandy/.openmpi/bin/: 是一个目录
logging to logs/ucf101_rgb_split1.log
It looks like MPI_INIT failed for some reason; your parallel process is
likely to abort.  There are many reasons that a parallel process can
fail during MPI_INIT; some of which are due to configuration or environment
problems.  This failure appears to be an internal failure; here's some
additional information (which may only be relevant to an Open MPI
developer):
ompi_mpi_init: ompi_rte_init failed
--> Returned "(null)" (-43) instead of "Success" (0)
*** An error occurred in MPI_Init_thread
*** on a NULL communicator
*** MPI_ERRORS_ARE_FATAL (processes in this communicator will now abort,
***    and potentially your MPI job)
[sandy:30451] Local abort before MPI_INIT completed completed successfully, but am not able to aggregate error messages, and not able to guarantee that all other processes were killed!
mpirun noticed that the job aborted, but has no info as to the process
that caused that situation.
TRY THREE TIMES:because i just have one gpu so i guess maybe need'nt mpi to parallel training ,so i change ：
${MPI_BIN_DIR}mpirun -np $N_GPU
$TOOLS/caffe train --solver=models/${DATASET}/tsn_bn_inception_${MODALITY}solver.prototxt
--weights=models/bn_inception${MODALITY}_init.caffemodel 2>&1 | tee ${LOG_FILE}
to
$TOOLS/caffe train --solver=models/${DATASET}/tsn_bn_inception_${MODALITY}solver.prototxt
--weights=models/bn_inception${MODALITY}_init.caffemodel 2>&1 | tee ${LOG_FILE}
in scripts/train_tsn.sh
but i get the error:
I0705 20:57:13.839275 29401 solver.cpp:490]     Test net output #1: loss = 4.61802 (* 1 = 4.61802 loss)
F0705 20:57:13.889241 29401 syncedmem.cpp:51] Check failed: error == cudaSuccess (2 vs. 0)  out of memory
*** Check failure stack trace: ***
@     0x7ff24f4c2daa  (unknown)
@     0x7ff24f4c2ce4  (unknown)
@     0x7ff24f4c26e6  (unknown)
@     0x7ff24f4c5687  (unknown)
@     0x7ff24f995978  caffe::SyncedMemory::mutable_gpu_data()
@     0x7ff24f978171  caffe::Blob<>::mutable_gpu_data()
@     0x7ff24f9c4a4a  caffe::CuDNNConvolutionLayer<>::Forward_gpu()
@     0x7ff24f82302c  caffe::Net<>::ForwardFromTo()
@     0x7ff24f8232bf  caffe::Net<>::ForwardPrefilled()
@     0x7ff24f84a637  caffe::Solver<>::Step()
@     0x7ff24f84b84f  caffe::Solver<>::Solve()
@           0x40b02c  train()
@           0x408e3d  main
@     0x7ff24e6f1f45  (unknown)
@           0x409212  (unknown)
@              (nil)  (unknown)
Setting device 0
do you know how to  fix it? what cause the error?
looking forward to your reply.
last but not least, all command before " bash scripts/train_tsn.sh ucf101 rgb" i had run and get no error.

it seems to solved the last problem by set the bach_size to be a small number,such as 8.
and i get the result as:
I0706 09:42:08.366730  4402 solver.cpp:433] Iteration 3500, Testing net (#0)
I0706 09:42:36.688565  4402 solver.cpp:490]     Test net output #0: accuracy = 0.691579
I0706 09:42:36.688733  4402 solver.cpp:490]     Test net output #1: loss = 1.08233 (* 1 = 1.08233 loss)
I0706 09:42:36.688741  4402 solver.cpp:420] Optimization Done.
I0706 09:42:36.688745  4402 caffe.cpp:203] Optimization Done.
Setting device 0

I would like to notice that by reducing batch_size, you will need to increase iter_size or number of iterations trained to compensate. Otherwise, you will get inferior performance.


Laravel 10 is released

package needs update

I just published 0.1.4, which has laravel 10 support.


Add support for namespaces in aliases

I would like to be able to switch to an alias and have it also switch to a namespace in that cluster.

This is something that would be very helpful to us as well.


Render the form object as a trimmed down dictionary

Added a dict property which transforms the form into a dictionary of inputs (which retain the _fields order), and errors.  The output of the dict property can be easily serialized to JSON or XML.  The idea is to be able to use the same WTForms objects in your markup as you do in your APIs.
e.g.
new_form = SomeWTForm()
    if new_form.is_submitted():
        if new_form.validate():
            # do some stuff
        else:
            return Response(json.dumps(new_form.dict), 400)
    else:
        return Response(json.dumps(new_form.dict), 200)

What's wrong with form.data ?

This seems more appropriate for an extension.


Fix for Issue #50 Python3 Fix in _build_command()

Running code:
zigbee = ZigBee(serial)
zigbee.send('at', command='MY')

Running on python3 causes error:
  File "../pyalertme/base.py", line 119, in read_addresses
    self._xbee.send('at', command='MY')
  File "/usr/local/lib/python3.5/site-packages/xbee/base.py", line 394, in send
    self._write(self._build_command(cmd, **kwargs))
  File "/usr/local/lib/python3.5/site-packages/xbee/base.py", line 212, in _build_command
    packet += data
TypeError: can't concat bytes to str

Caused because _build_command() tries to concat bytes with at command 'MY' of type string.
This solution proposes checking data type and converting to bytes.
Also written a small unit test to make sure now works. I am not totally sure if test_zigbee.py is the most appropriate place for it.
Fixes Issue #50

Actually, putting this PR on hold for a bit as I am not 100% sure if its impact on Python 2. Will return to this once I have checked it out.

@hansmosh @mattdodge Please could one of you give this a check over? I didn't want to merge this in until at least one of you had given it a look. I have written a test (albeit not sure in the right place?)


SmartArt | Easily create structured diagram with minimal typing

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Is your feature request related to a problem? Please describe.
I'm always frustrated when I want to create quick visual diagrams by just typing. It takes a lot of time and effort to create nice structured diagrams.
Describe the solution you'd like
As a user I want to easily create attractive / structured visual diagrams with minimal typing, allowing me to quickly and easily organize my thoughts.
Describe alternatives you've considered

I used the "layout" features like vertical tree's
I used "advanced" feature like mermaid

Additional context
Microsoft SmartArt is a feature in Microsoft Office applications that allows users to easily create professional-looking visuals. It includes hundreds of pre-made templates, or users can create their own from scratch. SmartArt allows users to quickly create diagrams and visuals from existing data or to just create an organized graphic based on their own ideas. With SmartArt, users can easily add text, shapes, colors, and other formatting to create visually appealing visuals.
First the user chooses and selects from the SmartArt categories to find the style he/she wants.

Then the user can type the text in the text box to the left / right of the SmartArt graphic.

Any changes will be realtime updated.

The other features of SmartArt not needed, if you guys only can add this it will be more than enough.

Thank you for sharing your feature request. We understand that creating quick, visually appealing diagrams is important for you and appreciate your feedback.
However, it's important to note that implementing a feature like Microsoft SmartArt would be a significant undertaking, as it involves a great deal of development and resources. While we strive to improve the functionality of diagrams.net, we must carefully balance the needs and requests of all our users and prioritize features that are feasible and most valuable to the majority of our users.
As an alternative, you may want to consider exploring the various layout and advanced features that are available in diagrams.net, such as vertical trees and Mermaid. These features can help you quickly create structured diagrams that meet your needs.
Thank you for using diagrams.net and for sharing your feedback. If you have any further questions or concerns, please don't hesitate to let us know.

We've implemented https://github.com/jgraph/drawio/discussions/3414 . I will close this as it meets the requirement of "Easily create structured diagram with minimal typing"


Small fixes and updates

Thanks for making this list available!
I suggest a couple of small fixes and updates. These are mainly updated links.
Cheers,
Patrick

Many thanks @rnckp - fellow Awesome list maintainer - for your contribution :bow:


is there something that can make configuring exponea throw an expection?

the example code to configure exponea:
async function configureExponea(configuration: Configuration) {
  try {
    if (!await Exponea.isConfigured()) {
      Exponea.configure(configuration)
    } else {
      console.log("Exponea SDK already configured.")
    }
  } catch (error) {
    console.log(error)
  }
}

ive been playing around with this, but i cant seem to "force" an exception. i want to enter the catch block, but it never does. so i was curious to if there is something which can trigger an exception so we enter the catch block. ive tried removing some of the required parameters but it does not trigger an exception.

It seems complex and more paranoid way rather than reasonable, but still recommended. Our mantra is to create stable library but we may make a mistake. Therefore this try-catch wrap is recommended way to log error message rather then cause crash of your app.


Unable to use API-client

Hello, after giving authorisation I get the following error:
Fatal error: Uncaught exception 'RuntimeException' with message 'Puli Factory is not available' in /Users/Dirk/Develop/www/pebbler/sites/rezelman/dashboard/vendor/php-http/discovery/src/ClassDiscovery.php:32 Stack trace: #0 /Users/Dirk/Develop/www/pebbler/sites/rezelman/dashboard/vendor/php-http/discovery/src/ClassDiscovery.php(79): Http\Discovery\ClassDiscovery::getPuliFactory() #1 /Users/Dirk/Develop/www/pebbler/sites/rezelman/dashboard/vendor/php-http/discovery/src/ClassDiscovery.php(99): Http\Discovery...
Since I didn't use composer.phar before I have no clue how to solve this. I followed the install-instruction and used the demo-code for authentication.
Can someone point me out how to solve this?
Thanks in advance.
Dirk Schaap

Hey.
I would recommend you to use composer. It is a great tool that has revolutionized PHP.
I will (within the coming weeks) review the documentation and make sure it is easy to understand and follow. So please give me feedback. But composer is a hard requirement. =)

Hey,
I installed composer and did use it since it is recommended. However, I didn't use it before and now I run into this fatal error. Maybe the last part of the error message gives a clue...
happyr/linkedin-api-client/src/Http/RequestManager.php(34 in /Users/Dirk/Develop/www/pebbler/sites/rezelman/dashboard/vendor/php-http/discovery/src/ClassDiscovery.php on line 32
Regarding the manual:
Since I didn't have any experience with composer it took me a lot of time to get it running with only the line: composer require happyr/linkedin-api-client. With only this line it gave some errors...

It looks like you are on the right way. Read https://github.com/Happyr/LinkedIn-API-client#installation and https://github.com/Happyr/LinkedIn-API-client#finding-the-http-client and you will understand that you need to tell the LinkedIn client what library you use for sending HTTP messages. Ie you need to
$linkedIn=new Happyr\LinkedIn\LinkedIn('app_id', 'app_secret');
$linkedIn->setHttpClient(new \Http\Adapter\Guzzle6\Client());
$linkedIn->setHttpMessageFactory(new Http\Message\MessageFactory\GuzzleMessageFactory());

Hi Nyholm,
Thanks for your help!! I got it working!


riscv: linker: provide symbol of _flash_used

The symbol provides the image size.
Signed-off-by: Dino Li<EMAIL_ADDRESS>

This change is

This seems to be pretty generic functionality. Shouldn't it be added to common linker script for all the platforms?

Done. (Only apply to platforms have defined symbol of _image_rom_start)

This _flash_used symbol is not really making much sense to me.
This feature was initially added in d85efe0.
From that commit message:

Using '_image_rom_end' will not work, as there are
symbols loaded after its value.

The point of specifying GROUP_START and GROUP_END for the ROMABLE_REGION and having separate linker script (header) files for ROM and RAM sections in the first place is to organise the sections for different memory regions separately -- so, we should not be loading anything into the ROMABLE_REGION after GROUP_END(ROMABLE_REGION), and therefore _image_rom_end - _image_rom_start should indeed be the flash (aka. ROM) size.
In general, our linker scripts are a mess and in need of clean-up/refactoring. The _flash_used symbol was a hack added in order to work around the mess, and I do not think we should be expanding the scope of that hack to all architectures.
cc @andyross @ioannisg @tejlmand

Change to modify riscv's linker script only.

In general, our linker scripts are a mess and in need of clean-up/refactoring. The _flash_used symbol was a hack added in order to work around the mess, and I do not think we should be expanding the scope of that hack to all architectures.

Agree to that.

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @abrodkin, @andyross, @carlocaione, @dcpleung, @Dino-Li, @evgeniy-paltsev, @hakonfam, @ioannisg, @jackrosenthal, @jhedberg, @kgugala, @MaureenHelm, @nashif, @pgielda, @ruuddw, and @sjg20)

include/arch/riscv/common/linker.ld, line 272 at r2 (raw file):
Previously, brockus-zephyr wrote…
Dino, can this PR be dropped?

Yes. I am going to close this PR.


basic video creation

Basic video creation with cv2

Aggregate frames together
Pass to function that combines them to form a video


enhancement: allows using multiple web listen addresses

these variants should work now:
alertmanager_web_listen_address: '<IP_ADDRESS>:9093'

alertmanager_web_listen_address:
  - '<IP_ADDRESS>:9093'
  - '<IP_ADDRESS>:9093'

fixes #115

Please add a version check in the template so that we won't attempt to use the unsupported flag on older versions.

Do you know which version this was introduced? The changelog doesn't say (searched for both listen-address and toolkit) and I really don't wanna browse git log -p :sweat_smile:

Please add a version check in the template so that we won't attempt to use the unsupported flag on older versions.

Do you know which version this was introduced? The changelog doesn't say (searched for both listen-address and toolkit) and I really don't wanna browse git log -p 😅
Edit: Okay, so I did browse the git log -p of prometheus/alertmanager a bit. I went with 0.25.0 for now, although I'm not sure at all if that's really the first version supporting multiple web listen addresses.

I think the change was introduced in this PR https://github.com/prometheus/exporter-toolkit/pull/95 so any version which uses exporter-toolkit v0.8.0 or later (https://github.com/prometheus/exporter-toolkit/commit/6b1221e7939a58e8aa767b7dc6d02140b34d31e3)
You can see that in alertmanager v0.25.0 the version of exporter-toolkit was bumped to v0.8.1 https://github.com/prometheus/alertmanager/commit/ce7b4756974d6896f95b9d26466c9a6447b9435f
Can you add multiple web listen address support to the other roles as well?

I think the change was introduced in this PR prometheus/exporter-toolkit#95 so any version which uses exporter-toolkit v0.8.0 or later (prometheus/exporter-toolkit@6b1221e) You can see that in alertmanager v0.25.0 the version of exporter-toolkit was bumped to v0.8.1 prometheus/alertmanager@ce7b475

@gardar thanks, I was looking for exporter-toolkit in this collection :sweat_smile:

Can you add multiple web listen address support to the other roles as well?

Will do.

I will need to bump all the versions in the */alternative/molecule.yml for the tests to succeed.

Note: systemd_exporter doesn't have a release since exporter-toolkit 0.8+, so I'm not touching that one.

Also gonna use @pytest.mark.parameterize("sockets", ...) in all the roles I touch, the inconsistency annoys me :sweat_smile:

Can you add multiple web listen address support to the other roles as well?

Will do.

Done, searched for _web_listen_address: and added the multiple web listen address to all the roles, except for the systemd_exporter which doesn't have a stable release since the relevant toolkit addition.

Just rebased. Is there anything standing in the way of this getting merged and released?

I don't know what to do about the failing CI cases, don't even know, if/how this is related to this PR. Please advise or feel free to take over.

Looks like a few assert tasks need to be adjusted, but otherwise LGTM

I'm thinking about this for the asserts:

- name: Naive assertion of proper listen address
  ansible.builtin.assert:
    that:
      - "':' in node_exporter_web_listen_address"
  when:
    - node_exporter_web_listen_address is defined
    - node_exporter_web_listen_address is string

- name: Naive assertion of proper listen address
  ansible.builtin.assert:
    that:
      - "':' in item"
  when:
    - node_exporter_web_listen_address is defined
    - node_exporter_web_listen_address is iterable
    - node_exporter_web_listen_address is not mapping
    - node_exporter_web_listen_address is not string
  with_items: "{{ node_exporter_web_listen_address }}"

What do you think @gardar @SuperQ? Dunno if this could be simplified :thinking:

What do you think @gardar @SuperQ? Dunno if this could be simplified 🤔

How about something like:
- name: Naive assertion of proper listen address
 ansible.builtin.assert:
   that: "[node_exporter_web_listen_address] | flatten | reject('match', '.+:\\d+$') | length == 0"

To explain:

[node_exporter_web_listen_address] turns the var into a list, no matter if it's a single string or a list already.
Then flatten makes sure we don't end up with a nested list, in case the variable was a list already.
Then we reject any items from the list that match the regex .+:\d+$
Then we assert that the resulting list doesn't contain any items.

What do you think @gardar @SuperQ? Dunno if this could be simplified 🤔

How about something like:
- name: Naive assertion of proper listen address
  ansible.builtin.assert:
    that: "[node_exporter_web_listen_address] | flatten | reject('match', '.+:\\d+$') | length == 0"

Thanks @gardar that's the simplification I was looking for :sweat_smile: I just pushed the updated asserts and also implemented them for the roles that don't yet themselves support multiple web listen addresses. Let's hope that the CI works out now :crossed_fingers:

Looks like the new commits are missing the DCO.

fatal: [ubuntu-20.04]: FAILED! => {"msg": "The conditional check '[alertmanager_web_listen_address] | flatten | reject('match', '.+:\d+$') | length == 0' failed. The error was: Unexpected templating type error occurred on ({% if [alertmanager_web_listen_address] | flatten | reject('match', '.+:\d+$') | length == 0 %} True {% else %} False {% endif %}): object of type 'generator' has no len()"}

I suppose this could be fixed with reject() | list | length as it occurs only with Ansible 2.9?

Don't know what to do about these:

fatal: [fedora-37]: FAILED! => {"changed": false, "msg": "Unable to start service prometheus: Job for prometheus.service failed because the control process exited with error code.\nSee "systemctl status prometheus.service" and "journalctl -xeu prometheus.service" for details.\n"}

fatal: [ubuntu-20.04]: FAILED! => {"msg": "The conditional check '[alertmanager_web_listen_address] | flatten | reject('match', '.+:\d+$') | length == 0' failed. The error was: Unexpected templating type error occurred on ({% if [alertmanager_web_listen_address] | flatten | reject('match', '.+:\d+$') | length == 0 %} True {% else %} False {% endif %}): object of type 'generator' has no len()"}

I suppose this could be fixed with reject() | list | length as it occurs only with Ansible 2.9?

Yeah type casting to a list should fix the issue.

Don't know what to do about these:

fatal: [fedora-37]: FAILED! => {"changed": false, "msg": "Unable to start service prometheus: Job for prometheus.service failed because the control process exited with error code.\nSee "systemctl status prometheus.service" and "journalctl -xeu prometheus.service" for details.\n"}

fatal: [debian-11]: FAILED! => {"changed": false, "msg": "Unable to reload service snmp_exporter: snmp_exporter.service is not active, cannot reload.\n"}

That one is a bit tricky to diagnose from the current logs as by default ansible doesn't print the service failure log output.
It does suggest that there is something wrong with the service template that's preventing the service from running, it could be either a syntax error or a versioning issue where the generated service config has some unsupported options in it.
Could you run the molecule test locally with debug output? As we currently don't have a easy way of catching the error in the CI.

Looks like it's because the molecule alternative test is using the variable version and not prometheus_version
That test is configured for prometheus_binary_local_dir install where the role does not download the binary but instead uses a locally provided one, so that it actually never needs the prometheus_version variable, until now.
I think you could safely change the version variable to prometheus_version in molecule/alternative/prepare.yml and molecule/alternative/molecule.yml
With that being said, perhaps we also want to verify that there are not multiple listeners configured when using a old version, either by doing it in a assert task or we could also just grab the first item from the listener list in the template when using a old version (probably not ideal to do it in the template - might confuse users).

I think you could safely change the version variable to prometheus_version in molecule/alternative/prepare.yml and molecule/alternative/molecule.yml

Done. Did the same for for the alertmanager role, it was also using a plain version.

With that being said, perhaps we also want to verify that there are not multiple listeners configured when using a old version, either by doing it in a assert task or we could also just grab the first item from the listener list in the template when using a old version (probably not ideal to do it in the template - might confuse users).

Yes, I'm gonna do that next. As a separate assert or can/should I add another that: expression to the existing naive assertion of proper listen address?

@gardar That seems to work for the new assert:
- name: Assert that used version supports listen address type
  ansible.builtin.assert:
    that:
      - >-
        prometheus_web_listen_address is string
        or
        (
          prometheus_version is version('2.40.0', '>=') and
          prometheus_web_listen_address | type_debug == "list"
        )

- name: Naive assertion of proper listen address
  ansible.builtin.assert:
    that:
      - >-
        [prometheus_web_listen_address] |
          flatten |
          reject('match', '.+:\\d+$') |
          list |
          length == 0

Looks good?

Yes, that looks clean, I like it :)
But you could also merge it into a single task if you want to have it even more compact, since the assert module accepts a list of expressions.

I think I fixed the smokeping prober alternative issue, was missig a \ in the service file for the multiple web listen addresses.

For snmp exporter alternative, I've added these tasks to the end of main.yml:
- command: systemctl status snmp_exporter.service
  failed_when: no
  register: status

- debug: var=status.stdout_lines

- command: journalctl -xe
  failed_when: no
  register: journal

- debug: var=journal.stdout_lines

(I think, that's what we should do for all services in CI to get better error messages.)
[Unit]
Description=Prometheus SNMP Exporter
After=network-online.target
StartLimitInterval=0

[Service]
Type=simple
User=nobody
Group=nogroup
ExecReload=/bin/kill -HUP $MAINPID
ExecStart=/usr/local/bin/snmp_exporter \
  --web.listen-address=<IP_ADDRESS>:9116 \
  --web.listen-address=<IP_ADDRESS>:9116 \
  --log.level=info \
  --config.file=/etc/snmp_exporter/snmp.yml

KillMode=process

SyslogIdentifier=snmp_exporter
Restart=always
RestartSec=1

[Install]
WantedBy=multi-user.target

$ systemctl status snmp_exporter.service
● snmp_exporter.service - Prometheus SNMP Exporter",
     Loaded: loaded (/etc/systemd/system/snmp_exporter.service; enabled; vendor preset: enabled)
     Active: activating (auto-restart) (Result: exit-code) since Tue 2023-11-28 10:13:06 UTC; 669ms ago
    Process: 730 ExecStart=/usr/local/bin/snmp_exporter --web.listen-address=<IP_ADDRESS>:9116 --web.listen-address=<IP_ADDRESS>:9116 --log.level=info --config.file=/etc/snmp_exporter/snmp.yml (code=exited, status=1/FAILURE)
   Main PID: 730 (code=exited, status=1/FAILURE)
        CPU: 3ms

$ journalctl -xe
Nov 28 10:24:49 ubuntu-22.04 snmp_exporter[736]: ts=2023-11-28T10:24:49.500Z caller=main.go:148 level=info msg=\"Starting snmp_exporter\" version=\"(version=0.21.0, branch=HEAD, revision=0d8c3527cac0c26f1d6005b84b74413d14264c37)\"
Nov 28 10:24:49 ubuntu-22.04 snmp_exporter[736]: ts=2023-11-28T10:24:49.500Z caller=main.go:149 level=info build_context=\"(go=go1.19.3, user=root@51dfd4b1f59b, date=20221122-15:14:18)\"
Nov 28 10:24:49 ubuntu-22.04 snmp_exporter[736]: ts=2023-11-28T10:24:49.500Z caller=main.go:157 level=error msg=\"Error parsing config file\" err=\"yaml: unmarshal errors:\\n  line 3: field if_mib not found in type config.plain\"
Nov 28 10:24:49 ubuntu-22.04 systemd[1]: snmp_exporter.service: Main process exited, code=exited, status=1/FAILURE

yaml: unmarshal errors:
line 3: field if_mib not found in type config.plain

This might be due to snmp_exporter_version: 0.21.0 which I've added to allow for multiple web addresses. The default from defaults/main.yml is snmp_exporter_version: 0.24.1. Has configuration considerably changed between these two versions? I don't know/use snmp exporter, help appreciated.

The last one I don't get working is smokeping_prober, succeeds locally, but fails in CI:
$ ANSIBLE_DIFF_ALWAYS=True molecule test -s alternative -p ubuntu-22.04

...

--- before                                                                                [373/9543]
+++ after: /home/umcdev/.ansible/tmp/ansible-local-245143asywggq4/tmp4d3313t_/smokeping_prober.service.j2
@@ -0,0 +1,41 @@
+#
+# Ansible managed: Do NOT edit this file manually!
+#
+[Unit]
+Description=Smokeping Prober
+After=network-online.target
+StartLimitInterval=0
+StartLimitIntervalSec=0
+
+[Service]
+Type=simple
+User=smokeping
+Group=smokeping
+PermissionsStartOnly=true
+ExecReload=/bin/kill -HUP $MAINPID
+ExecStart=/usr/local/bin/smokeping_prober \
+  --config.file=/etc/smokeping_prober//probes.yml \
+  --web.listen-address=<IP_ADDRESS>:8080 \
+  --web.listen-address=<IP_ADDRESS>:8080 \
+
+SyslogIdentifier=smokeping_prober
+KillMode=process
+Restart=always
+RestartSec=5
+
+LockPersonality=true
+NoNewPrivileges=true
+MemoryDenyWriteExecute=true
+PrivateTmp=true
+ProtectHome=true
+RemoveIPC=true
+RestrictSUIDSGID=true
+
+AmbientCapabilities=CAP_NET_RAW
+ProtectControlGroups=true
+ProtectKernelModules=true
+ProtectKernelTunables=yes
+ProtectSystem=strict
+
+[Install]
+WantedBy=multi-user.target

...

INFO     Running alternative > verify
INFO     Executing Testinfra tests found in /home/umcdev/src/ansible/prometheus/roles/smokeping_prober/molecule/alternative/tests/...
============================= test session starts ==============================
platform linux -- Python 3.11.6, pytest-7.4.3, pluggy-1.3.0
rootdir: /home/umcdev
plugins: testinfra-10.0.0
collected 5 items

tests/test_alternative.py .....                                          [100%]

============================== 5 passed in 1.79s ===============================
INFO     Verifier completed successfully.

Yes, I recently did a major rewrite of the snmp_exporter config, so configs need to match the version.
If you want, I can open a separate PR to make that test cleaner.

blackbox exporter alternative fails, but only on debian 10:
[Unit]
Description=Blackbox Exporter
After=network-online.target
StartLimitInterval=0
StartLimitIntervalSec=0

[Service]
Type=simple
User=blackbox-exp
Group=blackbox-exp
PermissionsStartOnly=true
ExecReload=/bin/kill -HUP $MAINPID
ExecStart=/usr/local/bin/blackbox_exporter \
  --config.file=/etc/blackbox_exporter.yml \
  --log.level=warn \
    --web.listen-address=<IP_ADDRESS>:9000 \
  --web.listen-address=<IP_ADDRESS>:9000 \

SyslogIdentifier=blackbox_exporter
KillMode=process
Restart=always
RestartSec=5

LockPersonality=true
NoNewPrivileges=true
MemoryDenyWriteExecute=true
PrivateTmp=true
ProtectHome=true
RemoveIPC=true
RestrictSUIDSGID=true

AmbientCapabilities=CAP_NET_RAW
ProtectControlGroups=true
ProtectKernelModules=true
ProtectKernelTunables=yes
ProtectSystem=strict

[Install]
WantedBy=multi-user.target

    "-- Logs begin at Thu 2023-11-30 09:06:49 UTC, end at Thu 2023-11-30 09:07:35 UTC. --",
    "Nov 30 09:07:32 debian-10 systemd[1]: /etc/systemd/system/blackbox_exporter.service:22: Unknown lvalue 'RestrictSUIDSGID' in section 'Service', ignoring",
    "Nov 30 09:07:33 debian-10 systemd[1]: /etc/systemd/system/blackbox_exporter.service:22: Unknown lvalue 'RestrictSUIDSGID' in section 'Service', ignoring",
    "Nov 30 09:07:33 debian-10 systemd[1]: Started Blackbox Exporter.",
    "-- Subject: A start job for unit blackbox_exporter.service has finished successfully",
    "-- Defined-By: systemd",
    "-- Support: https://www.debian.org/support",
    "-- ",
    "-- A start job for unit blackbox_exporter.service has finished successfully.",
    "-- ",
    "-- The job identifier is 98.",
    "Nov 30 09:07:33 debian-10 blackbox_exporter[1597]: blackbox_exporter: error: unexpected SyslogIdentifier=blackbox_exporter, try --help",
    "Nov 30 09:07:33 debian-10 systemd[1]: blackbox_exporter.service: Main process exited, code=exited, status=1/FAILURE",
    "-- Subject: Unit process exited",
    "-- Defined-By: systemd",
    "-- Support: https://www.debian.org/support",
    "-- ",
    "-- An ExecStart= process belonging to unit blackbox_exporter.service has exited.",
    "-- ",
    "-- The process' exit code is 'exited' and its exit status is 1.",
    "Nov 30 09:07:33 debian-10 systemd[1]: blackbox_exporter.service: Failed with result 'exit-code'.",
    "-- Subject: Unit failed",
    "-- Defined-By: systemd",
    "-- Support: https://www.debian.org/support",
    "-- ",
    "-- The unit blackbox_exporter.service has entered the 'failed' state with result 'exit-code'.",
    "Nov 30 09:07:33 debian-10 systemd[1]: blackbox_exporter.service: Consumed 27ms CPU time.",
    "-- Subject: Resources consumed by unit runtime",
    "-- Defined-By: systemd",
    "-- Support: https://www.debian.org/support",
    "-- ",
    "-- The unit blackbox_exporter.service completed and consumed the indicated resources."

I think the problem is this:

Nov 30 09:07:33 debian-10 blackbox_exporter[1597]: blackbox_exporter: error: unexpected SyslogIdentifier=blackbox_exporter, try --help

I think systemd on debian 10 is too old that it can't deal with the trailing backslash like the newer versions:
  --web.listen-address=<IP_ADDRESS>:9000 \

SyslogIdentifier=blackbox_exporter

It turns it into
ExecStart=/usr/local/bin/blackbox_exporter ... --web.listen-address=<IP_ADDRESS>:9000 SyslogIdentifier=blackbox_exporter

I'm gonna try nukeing the trailing backslash with loop last hacks.

Awesome!

Thanks for the merge and the guidance, this helped a lot.


Sticky navbar

Description

Added the sticky navbar in script.html . No depandancies needed

Fixes #1601
Type of change
Please delete options that are not relevant.

[x ] Bug fix (non-breaking change which fixes an issue)

Checklist:

[ x] My code follows the style guidelines(Clean Code) of this project
[ x] I have performed a self-review of my own code
[x ] I have commented my code, particularly in hard-to-understand areas
[ ] I have created a helpful and easy to understand README.md
[ ] My documentation follows Template for README.md
[ ] I have added the project meta data in the PR template.
[ ] I have created the requirements.txt file if needed.

Project Metadata
If there is no-file/nothing to fill the below fields with, then type: none
Example: If no requirements.txt needed/present, then type none in Requirments.
Category:

[ ] Calculators
[ ] AI/ML
[ ] Scrappers
[ ] Social_Media
[ ] PDF
[ ] Image_Processing
[ ] Video_Processing
[ ] Games
[ ] Networking
[ ] OS_Utilities
[ ] Automation
[ ] Cryptography
[ ] Computer_Vision
[ ] Fun
[x ] Others

Title: <>
Folder: <gh-pages branch>
Requirements: <None>
Script: <None>
Arguments: <None>
Contributor: <parasss19>
Description: <Enter a one line description that describes your script. Also, explain the arguments usage here>

Hi @avinashkranjan I made the navbar sticky in script folder
Screenshot
https://github.com/avinashkranjan/Amazing-Python-Scripts/assets/101205226/a2d0f2ef-6179-4a01-8d58-1c3a3f36fd98
I changed the branch to gh-pages and then made a new branch Stick-Navbar and changed the script.css file only and add the commit 7a5ed22 But i dont know why there are so many commits present

Make the changes on gh-pages branch


style: format using prettier

CI will pass after #26 is merged.

FWIW, prettier is what's configured in the format script in package.json.
I just ran npm run format ¯_(ツ)_/¯


Help me!!!

Hi all:
My system is Raspberry+Domoticz+Homebridge+homebridge-edomoticz
If not't user ~/.homebridge/config.json file ,the homebridge setup is OK!
Use the file homebridge setup is always error!!
*** WARNING *** The program 'node' uses the Apple Bonjour compatibility layer of Avahi.
*** WARNING *** Please fix your application to use the native API of Avahi!
*** WARNING *** For more information see http://0pointer.de/avahi-compat?s=libdns_sd&e=node
*** WARNING *** The program 'node' called 'DNSServiceRegister()' which is not supported (or only supported partially) in the Apple Bonjour compatibility layer of Avahi.
*** WARNING *** Please fix your application to use the native API of Avahi!
*** WARNING *** For more information see http://0pointer.de/avahi-compat?s=libdns_sd&e=node&f=DNSServiceRegister
[2017-7-19 15:45:17] Loaded plugin: homebridge-edomoticz
[2017-7-19 15:45:17] Registering accessory 'homebridge-edomoticz.eDomoticz'
[2017-7-19 15:45:17] Registering platform 'homebridge-edomoticz.eDomoticz'
[2017-7-19 15:45:17] ---
[2017-7-19 15:45:17] There was a problem reading your config.json file.
[2017-7-19 15:45:17] Please try pasting your config.json file here to validate it: http://jsonlint.com
[2017-7-19 15:45:17]
/usr/nodejs422/lib/node_modules/homebridge/lib/server.js:197
throw err;
^
SyntaxError: Unexpected token   in JSON at position 483
at Object.parse (native)
at Server._loadConfig (/usr/nodejs422/lib/node_modules/homebridge/lib/server.js:191:19)
at new Server (/usr/nodejs422/lib/node_modules/homebridge/lib/server.js:52:38)
at module.exports (/usr/nodejs422/lib/node_modules/homebridge/lib/cli.js:26:16)
at Object. (/usr/nodejs422/lib/node_modules/homebridge/bin/homebridge:17:22)
at Module._compile (module.js:570:32)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
~/.homebridge/config.json is as follows:
{
"bridge": {
"name": "Homebridge",
"username": "b8:27:eb:35:f1:20",
"port": 51826,
"pin": "031-45-154"
},
"description": "This is an example configuration file with one fake accessory and one fake platform. You can use this as a template for creating your own configuration file containing devices you actually own.",
"platforms": [{
"platform": "eDomoticz",
"name": "eDomoticz",
"server": "<IP_ADDRESS>",
"port": "8080",
"ssl": 0,
"roomid": 2,
"mqtt": 1
}],
"accessories": []
}
Hoping to get help!!!

Something is wrong with your config file, you probably has a misplaced tab somewhere near the end of it.

I punched that into jsonlint and it gives me the valid message, are you sure that is the correct config.json that is being loaded when homebridge is run?

What editor are you using to make your config.json file? Some editors use non standard double quotes, or put double byte characters in odd places. I only use nano to edit my config files.
I don't know if this fixes your issue, but it is worth a shot.


Add support for virtualenv in installer

Fixes #1641
Add support for virtualenv in installer

@JeanChristopheMorinPerso Ready!

Thank you @Pixel-Minions!


Add HDR display support in DJV

Hello,
The possibility to view animation sequence which are HDR by nature (Typically exr sequences in ACEScg or ACES2065-1 colorspaces) on a HDR compatible display in HDR mode would be very neat.
The only way I'm able to review my work in HDR right now is I encode the sequences in HEVC 10bits HDR10 using FFmpeg and I view them on a HDR TV or recently with PotPlayer which is the only player I found yet under Windows to be able to watch HDR movies in HDR (without converting the display in SDR) on a HDR compatible display.

The GLFW toolkit that DJV uses has an open issue for adding HDR support:
https://github.com/glfw/glfw/issues/890

Thanks for the answer.
By the way, viewing HDR content in HDR using PotPlayer is indeed only possible by setting the display surface to Direct3D11 under Windows 10 as confirmed by your link.

Thanks, I was wondering about that. At some point I would like to add native rendering (Direct3D11 and Metal) but it's a huge amount of work...

recently with PotPlayer which is the only player I found yet under Windows to be able to watch HDR movies in HDR

VLC, mpv, chrome... LOL, what?


Total value in treemap title does not reflect .time solo selection

If you have a treemap that uses the .time and .title methods, the "Total" value shown in the title does not reflect solo changes.  For example, if I have a timeline with 2 years of data: 2014 and 2015, the treemap title always shows the sum of ALL time periods, not just the selected one (using solo).
I think this should be considered a bug because the total value is not consistent with what is being displayed on the page.
Here is a JS Fiddle illustrating my point: https://jsfiddle.net/mac471/j0prj9f9/
The treemap opens with just 2014 data selected - the chart title total shows 450 but if you mouse over the two boxes, you'll see that the total is just 150.
Is there a workaround (or the proper way to ensure that the "total" shown by .title) is correct?
Thanks and keep up the good work,
Dan

I'm having the exact same issue, thanks for sharing a detailed example.
Any workaround you found so far?


Pm view changes

Made changes in the layout as the background image has changed

The relevance of this pull request expired


Kanagawa watchface update to v0.3

Fixes a bug where the watch still showed seconds when locked screen is on, after a message from the messageui was displayed.

Thanks!


Embedded Ansible Loop on activation.

Embedded ansible not activating with Gaps-4.

Duplicate of https://github.com/ManageIQ/manageiq/issues/17738


[FLINK-5701] [kafka] FlinkKafkaProducer should check asyncException on checkpoints

Prior to this change, at-least-once is violated for the FlinkKafkaProducer because we were not failing checkpoints if there were async exceptions from the Kafka producer.
With this PR, on snapshotState(), we fail if there previously were async exceptions. We also fail if the flushed records on checkpoint resulted in exceptions.
This PR also improves the tests in FlinkKafkaProducerBaseTest to use one-shot latches instead of sleeping for more stable tests. It also removes the test testAtLeastOnceProducerFailsIfFlushingDisabled(). My reasoning is that essentially, you might still have at-least-once even if flushing is disabled (i.e. always no pending records to flush on checkpoint), so I don't see the necessity in having that test. I'm open to discussing the removal of that test and adding it back if others think it's necessary.

I don't think that you have at least once guarantees if you disable flushing. Assume the following: You have the input event1, checkpoint barrier, event2. Now you write event1 to Kafka but it is not yet committed. Now you process the checkpoint barrier and complete the checkpoint without waiting for event1 to be written. Now writing event1 to Kafka fails for some reason and triggers a recovery. Then you will start at event2 without having ever written event1 to Kafka.

@tillrohrmann
Yes, in the condition that you described, then at-least-once doesn't hold. I said might mainly considering there is chance that for every checkpoint barrier, the previous events (i.e. event1 in your example) has been committed. But yes, essentially this indeterminate behaviour means that it is not at-least-once, so I'm incorrect in saying that it might.
I was trying to point out that this indeterminate behaviour made it hard to have a stable test for testAtLeastOnceProducerFailsIfFlushingDisabled() without relying on sleeping, which ultimately leads to flaky tests.

Ah now I understand what the problem of testAtLeastOnceProdcuerFailsIfFlushingDisabled was. Can't we mock the KafkaProducer to control when the record's callbacks are triggered? That way we have full control over the execution and can write a proper test.

@tillrohrmann thanks for the comment. I'll try again and see if I can come with up a proper test for testAtLeastOnceProdcuerFailsIfFlushingDisabled().

Instead of directly re-adding testAtLeastOnceProducerFailsIfFlushingDisabled , I instead added a test testDoesNotWaitForPendingRecordsIfFlushingDisabled to simply assure that the snapshot method returns even if there are pending records.
The purpose of the testDoesNotWaitForPendingRecordsIfFlushingDisabled test is to assure that the actual testAtLeastOnceProducer test is valid.
I think this was what the original testAtLeastOnceProducerFailsIfFlushingDisabled was actually meant for anyway.
@tillrohrmann please let me know if this makes sense to you :-)

Thanks for the detailed review @tillrohrmann, I'll follow-up and address your comments.
Regarding removing the setFlushOnCheckpoint:
I think it was added at first to provide flexibility for users who know what they are doing, and making sure that the producer will be able to work in all environments (see comments in #2108).
However, recently I've also gathered opinions (from you and others) about the settings over complicating at-least-once guarantees for the producer, and I have the feeling we can remove it starting from the next release.
There is FLINK-5728 to enable flushing by default (currently the default is no flushing). I'll incorporate your opinion on this to that JIRA, and decide there if we only want to disable if by default or remove it completely.

I think we should allow users to disable the wait on flush, because it can substantially delay the confirmation of a checkpoint.
If a user favors fast checkpoints over complete data in Kafka (for example when a particular producer instance is used mostly for debugging purposes only), we should allow them to do that. The overhead for us making this configurable is very low, but the benefit for some users might be huge.

@tillrohrmann thanks for your second-pass review, the tips you mentioned were helpful. I've incoporated all of your comments except:

Maybe you could override the DummyFlinkKafkaProducer#flush method to insert some latches to see when you enter and when the method is done. Then you could wait on the first latch and check with the latter whether the method has completed.

I think this is over-complicating things. I don't think it makes sense to add ways to explicitly wait for the flush method to complete - it's called in the snapshotState method, so isn't it identical to waiting for the snapshot thread to complete?
Instead, I override the snapshotState method and added a flag inside the to make sure that when the base snapshotState implementation returns, it returned after it finished calling flush. Whether or not flush blocks correctly is out-of-scope of these tests, because flush is actually an abstract method for Kafka version-specific concrete subclasses to implement.

Addressed waiting for flushing to start with a latch instead of sleeping.
Will merge this to master and release-1.2 once Travis turns green (running also locally due to the timeouts).

Tests have passed, failed ones are Travis timeouts. Merging this now ...


1.19.3 Support

Looks like the current version doesn't support 1.19.3 - the game crashes when attempting to use any printer functionality. Running on 1.19.3 with the latest version of litematica and fabric for 1.19.3 as of this post.

Crash report:
java.lang.NoSuchMethodError: 'boolean net.minecraft.class_1799.method_7987(net.minecraft.class_1799, net.minecraft.class_1799)'
at io.github.eatmyvenom.litematicin.utils.InventoryUtils.areItemsExact(InventoryUtils.java:111)
at io.github.eatmyvenom.litematicin.utils.InventoryUtils.requiresSwap(InventoryUtils.java:143)
at io.github.eatmyvenom.litematicin.utils.InventoryUtils.swapToItem(InventoryUtils.java:168)
at io.github.eatmyvenom.litematicin.utils.Printer.doSchematicWorldPickBlock(Printer.java:147)
at io.github.eatmyvenom.litematicin.utils.Printer.doPrinterAction(Printer.java:1187)
at fi.dy.masa.litematica.util.WorldUtils.handler$eka000$onDoEasyPlaceAction(WorldUtils.java:2717)
at fi.dy.masa.litematica.util.WorldUtils.doEasyPlaceAction(WorldUtils.java)
at fi.dy.masa.litematica.util.WorldUtils.handleEasyPlace(WorldUtils.java:411)
at fi.dy.masa.litematica.event.KeyCallbacks$KeyCallbackHotkeys.onKeyAction(KeyCallbacks.java:268)
at fi.dy.masa.malilib.hotkeys.KeybindMulti.triggerKeyAction(KeybindMulti.java:185)
at fi.dy.masa.malilib.hotkeys.KeybindMulti.updateIsPressed(KeybindMulti.java:145)
at fi.dy.masa.malilib.event.InputEventHandler.checkKeyBindsForChanges(InputEventHandler.java:270)
at fi.dy.masa.malilib.event.InputEventHandler.onMouseClick(InputEventHandler.java:172)
at net.minecraft.class_312.handler$fck000$hookOnMouseClick(class_312.java:3556)
at net.minecraft.class_312.method_1601(class_312.java:54)
at net.minecraft.class_312.method_22686(class_312.java:165)
at net.minecraft.class_1255.execute(class_1255.java:102)
at net.minecraft.class_312.method_22684(class_312.java:165)
at org.lwjgl.glfw.GLFWMouseButtonCallbackI.callback(GLFWMouseButtonCallbackI.java:43)
at org.lwjgl.system.JNI.invokeV(Native Method)
at org.lwjgl.glfw.GLFW.glfwPollEvents(GLFW.java:3403)
at com.mojang.blaze3d.systems.RenderSystem.flipFrame(RenderSystem.java:190)
at net.minecraft.class_1041.method_15998(class_1041.java:310)
at net.minecraft.class_310.method_1523(class_310.java:1222)
at net.minecraft.class_310.method_1514(class_310.java:781)
at net.minecraft.client.main.Main.method_44604(Main.java:244)
at net.minecraft.client.main.Main.main(Main.java:51)
at net.fabricmc.loader.impl.game.minecraft.MinecraftGameProvider.launch(MinecraftGameProvider.java:461)
at net.fabricmc.loader.impl.launch.knot.Knot.launch(Knot.java:74)
at net.fabricmc.loader.impl.launch.knot.KnotClient.main(KnotClient.java:23)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.base/java.lang.reflect.Method.invoke(Method.java:568)
at org.prismlauncher.impl.OneSixLauncher.invokeMain(OneSixLauncher.java:104)
at org.prismlauncher.impl.OneSixLauncher.launchWithMainClass(OneSixLauncher.java:176)
at org.prismlauncher.impl.OneSixLauncher.launch(OneSixLauncher.java:186)
at org.prismlauncher.EntryPoint.listen(EntryPoint.java:144)
at org.prismlauncher.EntryPoint.main(EntryPoint.java:74)

Doesn't make it crash, but now you take damage while using the printer (both in survival and creative). all default settings used

....what??? I don't use any damage related function or code in printer itself, so could it be some kind of... anticheat?

I'm not sure. I tested it in singleplayer with only malilib, litematica, printer and fabricapi enabled to make sure it's not another mod causing it.

does it happen in creative mode too? and is printerBreakBlock option enabled? and is it instant kill?

Yes i tested it first in creative. It's like falling in the void, it takes 5 hits to kill but it remembers previous damage taken. PrinterBreakBlocks is disabled (false)

that's really weird... can you try with fakeRotation off? mojang maybe added some weird feature related to it

Without fakeRotation the issue is not there

I also can confirm fake rotation causes self damage - in singleplayer. Otherwise this appears to be working reasonably well.
In multiplayer, fake rotation doesn't cause self damage, but it doesn't really work and can cause the user to glitch around.

it casues self damage in multiplayer for me aswell. fake rotation works for me in multiplayer but at some angles (limited testing) will cause self damage, and sometimes it does not and places everything correctly.

Well I'm not sure but mojang definitely added some weird stuff.... huh....

@luisccl  @FelineAstronomer @Notoriousi Fixed in https://github.com/aria1th/litematica-printer/commit/39a377fc6f3112043c2abc54a3d0658d3d0e5516 (should)
can you try with automatic build artifact? just to make sure before I make release

Seems to have fixed it for me in both survival and creative. Muntiplayer untested


unittest: predicate matcher should have more precise type

http://api.dartlang.org/docs/continuous/unittest.html#predicate
it's not clear what arguments the function will be called with. I had to pull up core_matchers.dart to double check this :)

This issue has been moved to dart-lang/test#159.


Add better error messages for Image Upload fail states

Tested on an older version of master before I realised it wasn't the latest code, can't test on latest version of master because #1061 😢
Fairly simple logic but if we get an error from Imgur we show it straight up with no magic, if it's an error enforced by us we do the rate limit error. If it's an unexpected error we just use a generic one. Atleast we know which path it went down before it failed!
"Invalid client_id" is the current one, seems fair!
fixes #1047

Tested on master, works :)


Cannot Start a Win11 Vm in cloudstack

ISSUE TYPE

Bug Report

COMPONENT NAME

UI,API

CLOUDSTACK VERSION

<IP_ADDRESS>

CONFIGURATION

OS / ENVIRONMENT

Ubuntu 22.04lts, KVM hypervisor
SUMMARY

I am able to create and deploy a vm with a win11 iso. it gets stuck on the system requiments screen. the message says the system does not meet the minium system requirements. I have install TPM emulator swtpm-tools (TPM emulator version 0.6.3, Copyright (c) 2014-2021 IBM Corp.)ii  ovmf  2022.02-3ubuntu<IP_ADDRESS>  all UEFI firmware for 64-bit x86 virtual machines
STEPS TO REPRODUCE

![image](https://github.com/apache/cloudstack/assets/148808916/52099eb9-a835-49eb-97f4-657449b3f254)

EXPECTED RESULTS

The Vm to boot and run normally

ACTUAL RESULTS

the vm will not install win11

Hi @Tbaugus44  have you tried the steps in this article: https://lab.piszki.pl/cloudstack-vm-with-vtpm-and-secure-boot-uefi/

We have tried that article to no avail.

We have tried that article to no avail.

Does the CPU being used fall in thw Windows 11 supported CPU  list?

Hey guys, we are working wit a Architect over at shapeblue to resolve this issue. we will post the solution once its been resolved

Hi,
Do you have a solution for Ubuntu 22?
Will you share? :-)

I do not, we were working with Shapeblue, and our project got defunded. I haven't had the time or the resources to continue trying to fix this issue. we were working with @rajujith and @alexandremattioli  to come up with a solution. maybe they will have more to add.

Hi, Do you have a solution for Ubuntu 22? Will you share? :-)

@PPisz
please refer to https://github.com/apache/cloudstack/issues/8784#issuecomment-1996873239


Update checkbox_select_cell.html

Fixes #11134
As in this issue USE_THOUSAND_SEPARATOR has value already set to true so as the values are printed with thousand seperator I set the values with unlocalize which fixes this issue
Please check the following:

[x] Do the tests still pass?[^1]
[x] Does the code comply with the style guide?

[x] Run make lint from the Wagtail root.

[x] For Python changes: Have you added tests to cover the new/fixed behaviour?
[x] For front-end changes: Did you test on all of Wagtail’s supported environments?[^2]

[ ] Please list the exact browser and operating system versions you tested:
[ ] Please list which assistive technologies [^3] you tested:

[ ] For new features: Has the documentation been updated accordingly?

Please describe additional details for testing this change.
[^1]: Development Testing
[^2]: Browser and device support
[^3]: Accessibility Target

Closing as a duplicate of #11137.


Do not use defer in hot path in daprd

Defer statements add a performance overhead that should be eliminated in hot path daprd calls.
More information here: https://github.com/golang/proposal/blob/master/design/34481-opencoded-defers.md#abstract

After additional investigation, Go 1.14 introduced considerable performance optimizations for defer making it almost zero cost. This doesn't hold true for WASM compiled code but as this is currently not a concern, this issue can be closed.


Setting up MARL Benchmark with SMARTS 0.4.18

Restructured baselines/marl_benchmark folder to allow setting up with smarts==0.4.18 and updated package requirements in setup.py and README.md to resolve training procedure errors.

The changes mentioned by @zyk516 in  Issue #1131 have been made here. Now one can follow the instructions of README.md and get the training and evaluate procedures to run for MARL benchmark.

This should now be unblocked since #1138 is merged.

It looks like https://github.com/huawei-noah/SMARTS/issues/1230 will possibly block this.

https://github.com/huawei-noah/SMARTS/pull/1235 has merged in to address a secondary exception can block training.

It looks like #1230 will possibly block this
I ran the training with social agent scenarios and saw that it is not crashing. So maybe it won't block this pr.

#1235 has merged in to address a secondary exception can block training.

Would need a new release to allow using it for this pr. Converting the PR to draft for now.

The PR is blocked by the unresolved ray issue: https://github.com/ray-project/ray/issues/5339.
Need to resolve the above issue to support multi-agent training using several policies with checkpoint learning (or export models). Or need to remove the support to export checkpoint and model. More discussion is required.

Checkpoints are magically working for some reason on my environment...

@saulfield

Changes look fine to me. I'm just wondering why the directory restructuring was needed. Would relative imports accomplish the same thing?

I am not entirely certain why the directory was restructured. However, it does make the benchmark obviously packageable.


auro-form: get auro-input working as an integrated form component

Is your feature request related to a problem? Please describe.
Auro form components, such as auro-input, auro-dropdown, etc., are currently self-contained individual repositories.
Due to this component independence, it has become challenging to maintain separate repositories with numerous interconnected dependencies, complicating the ability to validate a release across related parent components.
Describe the solution you'd like
Consolidate all Auro form components into a new auro-form component, reducing the distributed complexity of the Auro form library.
Acceptance Criteria

[x] Migrate auro-input into auro-form, beginning integration
[x] Update postCss.mjs to search directories recursively & process CSS files with PostCSS
[x] Update package.json scripts (such as sass:render) to search directories recursively
[x] Disable SCSS Stylelint rules (Components will be updated with latest markup upon release)
[x] Repo builds without error
[x] Bundler compiles as expected
[x] Confirm auro-form__bundled is generated, along with respective files for individual form components (auro-input__bundled, etc)
[x] Create Doc page that renders auro-dropdown and auro-input
[x] Migrate test files
[x] Migrate postscripts (postCss.mjs, etc) back to Auro Library for future component generation
[x] Investigate adding individual tests for components to package.json(exp: test:input)
[x] Investigate whether all nested components should be built using dev / build at once, individually or optionally.
[x] Update folder structure to incorporate new monorepo structure.
[x] #10
[ ] #12
[ ] #19
[ ] #14
[ ] #21
[ ] #20
[ ] #11

Additional context
No response
Exit criteria
No response

@sourcery-ai review


Update Fybrik CI workflow to run with OpenMetadata

We are replacing Katalog by OpenMetadata as our primary catalog, therefore we need to update some of CI workflows to work with OpenOetadata.

done by https://github.com/fybrik/fybrik/pull/1796


Log random-based outcome

The built in log is great but only logs the moves made. Some moves have random consequences (following a dice roll for instance) but there's now way to know that as it's calculated by the game and not sent from the client.
Is there a way to modify the log associated to a move and add whether it's been successful or not, or add arbitrary data to it?
Right now, I have to build in my own log to display to the players what's happened but would love to leverage the existing one.

There used to be a mechanism to add arbitrary payloads to log messages but I think it got lost after we migrated to a new plugin architecture.  Do you think something like this would be helpful?
move: (G, ctx) => {
  G.roll = ctx.random.D6();
  ctx.log.custom({ roll: G.roll });  // gets added to the log message for this move
}

Yes! That would be amazing. Arbitrary payloads feel like the right approach. Is this something I could help with in terms of PR? (not sure when I'll have the time but if you have a pointer for where I could start, I'd happily take a look)

Done in #865


Use serde_repr crate!

The https://github.com/dtolnay/serde-repr has existed for a bit now! It replaced the custom macro on https://serde.rs/enum-number.html, and this replaces usages of enum_number in this crate.
I think the main benefit should be better IDE support, since they generally read derives better than macros.

I think we technically could, but it'd probably be more efficient to keep both? Since serde_repr still needs a serde deserializer running and num_derive::FromPrimitive only requires taking in a number.

Force pushing a rebased version to fix merge conflicts.

Merging this.


file not available on dataverse

In notebooks 06 and 07, velocity fields are loaded using dataverse. However, the files no longer seem to be there, or perhaps the path has changed:
IOError: could not spawn wget --content-disposition https://dataverse.harvard.edu/api/access/datafile/2845329: no such file or directory (ENOENT)

Hi @renskegelderloos
Sorry for that. Not sure what happened but my guess is that the issue was due to the notebooks being out of date vs packages where I implemented breaking changes since then.
Should work now based on yesterday's updates -- they did both on my laptop and in cloud using https://mybinder.org/v2/gh/JuliaClimate/GlobalOceanNotebooks/master using Julia v1.5
Could you give this another try? Can we close the issue?

Yes, all good. Thanks!


Dockerfile: use stable cosa

Recent cosa:latest image seems to set invalid selinux labels so kubelet
can't access pull secret on bootstrap.
We should be using stable cosa version until the issue is sorted out.

Container test in pod e2e-aws-ipi-install-install completed successfully

/cherrypick release-4.6


feature: add CI for our fork

This adds an additional GitHub workflow file to the Mellanox fork.
The main features in this CI are:

Trigger on push to/of network-operator-* branches and tags.
Calculates Docker tag based on trigger event type.
Selects Docker registry based on tag type (pure release version vs. RC/beta).
Build and push Docker images in parallel (controller, config-daemon and webhook).
Checks out Network Operator and updates the appropriate SR-IOV version tags in the values.yaml file.
Creates a PR with above change to Network Operator.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
8 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.05%) to 39.902%

Files with Coverage Reduction
New Missed Lines
%

controllers/drain_controller.go
3
67.42%

controllers/generic_network_controller.go
5
74.53%

Totals

Change from base Build<PHONE_NUMBER>:
-0.05%

Covered Lines:
5107

Relevant Lines:
12799

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 39.933%

Files with Coverage Reduction
New Missed Lines
%

controllers/drain_controller.go
3
67.1%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
5111

Relevant Lines:
12799

💛  - Coveralls


[master>WebAssembly] Merge fix up

regen bytecodes
updating baselines for match Jsrt changes

Hi @commonlisp, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

lgtm


Question: Programmatically Custom Remote Filter

Could you please give example how to use all this concepts together?
I have custom filter controls for 3 columns (the custom controls are select inputs that in background load data from server), also in a distinct column I have a button, that should be the only way to call filtering on the server, no filtering should happen when user choose values in all 3 filters. Also I've already used remote paging and sorting.

@rchaser sorry I still not very clear about your question, I think in the online storybook, we had showed the ppl how to implement a remote filter or a custom filter. Maybe be you can just point me which part you are not clear.
if you can give me an online example and explain more detail that will be great: https://codesandbox.io/s/k5750vzzvv you can take this and remember to change the version.
I will reopen this issue if there's any bug and please tag me if you provide more information, thanks

@AllenFang for now in my application I've done something similar to this example, but I have concerns whether it is good approach or not.
https://codesandbox.io/s/pjr0242820
In this example I haven't implemented filter with custom component for short, but in my app one of the column filters using custom component


DNS Server doesn't accept requests over TCP

Descrpition
TCP DNS Server doesn't accept requests over TCP from client pod in Istio and Istio Booking example. It also rejects TCP DNS requests from other pods on the same node of cluster.
Error from neighbour pod
dig @<IP_ADDRESS> productpage.default.svc.cluster.local +tcp
;; communications error to <IP_ADDRESS>#53: connection reset

Steps to reproduce

Run Istio-booking example
Go to cluster2
Install dig on autoscale-endpoint using apk add bind-tools
Run dig @NSE_POD_IP productpage.default.svc.cluster.local +tcp
Look at result of request

NSE_POD_IP means pod ip of cmd-nse-l7-proxy on cluster2

PRs

https://github.com/networkservicemesh/sdk/pull/1381
https://github.com/networkservicemesh/cmd-nse-l7-proxy/pull/56


Authentication Can use all password

i activate authentication, but when i tried to login using random password it still can be connected, and i don't see any users.db on my directory

have you enabled an additional authorization script on the openvpn server?
https://github.com/flant/ovpn-admin/blob/master/setup/configure.sh#L44-L52

@stupidoes-day any updates?

Hi , Very sorry for very late response, it works now after i update to latest version thank you very much


Issue with "Contributing from Cortex XSOAR UI" in @site/docs/contributing/marketplace.md

Describe the problem

Page: Contributing from Cortex XSOAR UI
Source: https://github.com/demisto/content-docs/tree/master/docs/contributing/marketplace.md

Suggested fix
There is a typo => explainingi. it needs to fix with right word.

@mycacao thanks for reporting this, we will handle it


Feature/php post type and taxonomy scaffold

Description of the Change
This adds a PHP Scaffold for adding Post Types and Taxonomies to the scaffold.
Whilst this isn't needed, it means that implementation would be very similar across projects, reducing cognitive load when context switching. It would also allow engineers to pick up newer projects much quicker.
Below is a screencast of me using this to create a new post type and taxonomy.
https://capture.dropbox.com/L7fP0jGYEvLobArN

Closes #175
How to test the Change
Check out the branch and try it out
Changelog Entry

Added - New PHP scaffolding for post types and taxonomies

Credits

Props @darylldoyle
Checklist:

[x] I agree to follow this project's Code of Conduct.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my change.
[ ] All new and existing tests pass.

@fabiankaegy I've updated this PR and added a first-draft of the docs for adding post types. Can you please review?

@darylldoyle Could we add a more structured way for registering custom fields? I'd love to use register_setting/register_meta (unsure what the official api is) but registering custom fields in this way where you also declare the "schema" makes things easier for headless projects. I'm thinking mostly from a REST API standpoint and not really thinking about us creating a wrapper around rendering the custom fields on the front-end.
The end goal here is to make sure we're consistent in how we are exposing custom fields through the REST API. There are lots of ways to do it and it doesn't always get added to the REST API schema.

@dsawardekar I'd love your eyes on the latest update I've pushed. It introduces an AbstractCorePostType class that extends AbstractPostType and kills any methods not needed by the core post types. That allows us to maintain the same interface for taxonomy registration and the after_register() method.
@nicholasio I've spoken about this topic at length with @ivanlopez on our 1:1's. There are a bunch of ways to do it, we just need to find a way to handle it in a way that's both flexible and as simple as possible for smaller projects. Can we continue this discussion in https://github.com/10up/wp-scaffold/issues/227, please? I want to keep this PR solely to the registration of post types and taxonomies 🙂

@darylldoyle I like the after_register, it will make things a lot more predictable across projects. Looks good to me.
This might be on your radar. This PR opens the door for integrating scaffolding commands that can rapidly bootstrap new Post Types and Taxonomies. :)

I'm here after reading the internal blog post and I was wondering if you were aware of the "Extended CPTs" library created by the core committer John Blackbourn.
Basically it extends the default WP functions to register CPTs and Taxonomies to include additional parameters to make things like adding custom admin columns, filters, etc.. much easier. I've used it in a few projects in the past and it works great!
I thought I would share it here in case you'd be interested in using it in these abstract classes.


Selection based on regions

It would be good for efficiency to provide a mechanism to preselect a given region at the preprocessor step prior to xarray concatenation.
cc @vanroekel

One possible approach from @vanroekel is to pull out the region info via a function that returns a list of cellIDs, which we can then pass to selvals (e.g., #12) .  It is unclear whether this should go in the mpas_xarray wrapper or in other analysis scripts.

Starting idea on how to select via regions from @vanroekel's code at https://github.com/vanroekel/mpas_xarray/tree/generalizedSelection:
    if regionbounds is not None:
        lonmin = regionbounds[0]
        lonmax = regionbounds[1]
        latmin = regionbounds[2]
        latmax = regionbounds[3]

        if max(regionbounds > 2*np.pi):
            lonmin *= np.pi/180.0
            lonmax *= np.pi/180.0
            latmin *= np.pi/180.0
            latmax *= np.pi/180.0
        inds = np.where((ds.latCell.values < latmax) & (ds.latCell.values > latmin) & \
                               (ds.lonCell.values < lonmax) & (ds.lonCell.values > lonmin))[0]
        ds = ds.sel(nCells = inds)

This may also be helpful: https://github.com/mathause/regionmask

Will be addressed at https://github.com/MPAS-Dev/MPAS-Analysis/issues/34, not here.


Update port_lists.rb

Added Redis and webmin and mongodb

Hi there, thanks for the PR! I really like the Webmin addition, but Redis and Mongodb don't expose an HTTP interface, do they?

ah they don't for HTTP.
Ill close this PR off and if you add the 10000 port for webmin and could you add port 2375/2376 for docker as those ports expose a HTTP API that would be sweet!
If i find any more good HTTP ports i'll do a PR.


[vcpkg] http_proxy and https_proxy should be lowercase

Pacman of Msys understands only lowercase environment variables
http_proxy and https_proxy.

Thanks!


window Function - Row_Number not working

Hi I am truing to use window functions but it seems they are not supported ? on checking i found out that sqllite recently enabled them. Do you think you can include them as well.

in theory nothing is impossible

still running into Execution failed on sql 'select row_number() over
(partition by asin)': near "(": syntax error while doing this
import pandas as pd
from pandasql import sqldf
from baresql import  baresql
df = pd.read_csv(r'C:\Automation\Simulator\file.csv', index_col=0)
bsql = baresql.baresql(keep_log=True)
bsqldf = lambda q: bsql.df(q, dict(globals(),**locals()))
print('data read')
query = "select row_number() over (partition by date)"
query_result = bsqldf(query)
print(query_result.head())
Regards
Saket Dabi
On Thu, Aug 22, 2019 at 11:20 AM stonebig<EMAIL_ADDRESS>wrote:

in theory nothing is impossible
[image: image]
https://user-images.githubusercontent.com/4312421/63539380-3c1f0c00-c51a-11e9-93b4-348e866de54e.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/yhat/pandasql/issues/77?email_source=notifications&email_token=AIYKYXOII5JWWM6I23FAF73QF3KHZA5CNFSM4IOXXTHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD456U5Q#issuecomment-524020342,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIYKYXMEUMT3RFFCUQXZHRDQF3KHZANCNFSM4IOXXTHA
.

query = """select row_number() over (partition by date) from df;""" same error

query = """select row_number() over (partition by date) from df$$;"""
may be better ?

even this simple aint working .. its shocking
query = """select row_number() over ()  as rn from dff$$;"""
query_result = bsqldf(query)
q = bsqldf(query)
Regards
Saket Dabi
On Thu, Aug 22, 2019 at 1:23 PM stonebig<EMAIL_ADDRESS>wrote:

[image: image]
https://user-images.githubusercontent.com/4312421/63547023-7e9d1480-c52b-11e9-944a-84108616c930.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/yhat/pandasql/issues/77?email_source=notifications&email_token=AIYKYXPWWVFXPPPOC4WYAADQF3YVHA5CNFSM4IOXXTHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD46JCUQ#issuecomment-524063058,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIYKYXLTP6NLEVEIUI7JPBTQF3YVHANCNFSM4IOXXTHA
.

maybe try bsqldf("select sqlite_version()"

its 3.21.0
On Thu, Aug 22, 2019 at 1:46 PM stonebig<EMAIL_ADDRESS>wrote:

maybe try bsqldf("select sqlite_version()"
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/yhat/pandasql/issues/77?email_source=notifications&email_token=AIYKYXMWEHAKCQ2433AM6DTQF33MDA5CNFSM4IOXXTHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD46K7LI#issuecomment-524070829,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIYKYXPRLLJQ3BCF3E4CNQ3QF33MDANCNFSM4IOXXTHA
.

so now, you know the problem

what did i miss ? version ?which version ?
On Thu, Aug 22, 2019 at 1:50 PM stonebig<EMAIL_ADDRESS>wrote:

so now, you know the problem
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/yhat/pandasql/issues/77?email_source=notifications&email_token=AIYKYXM4LYQKSRKTGEF6G2LQF332FA5CNFSM4IOXXTHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD46LJZY#issuecomment-524072167,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIYKYXLCS7CEYEO2IJEABK3QF332FANCNFSM4IOXXTHA
.

https://www.sqlite.org/releaselog/3_25_0.html

https://bugs.python.org/issue35360

i did download the 3.25 version but how can i upgrade the package itself
for it to use 3.25 instead of 3.21
Regards
Saket Dabi
On Thu, Aug 22, 2019 at 1:56 PM stonebig<EMAIL_ADDRESS>wrote:

https://bugs.python.org/issue35360
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/yhat/pandasql/issues/77?email_source=notifications&email_token=AIYKYXNGEHLNI35NDHVWCF3QF34N7A5CNFSM4IOXXTHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD46LXHI#issuecomment-524073885,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIYKYXJLJ5EZD3IINNBTEBTQF34N7ANCNFSM4IOXXTHA
.

Replace the sqlite3.dll. see https://www.google.com/amp/s/dfir.blog/upgrading-pythons-sqlite/amp/

done - works ..much thanks
On Thu, Aug 22, 2019 at 2:32 PM stonebig<EMAIL_ADDRESS>wrote:

Replace the sqlite3.dll. see
https://www.google.com/amp/s/dfir.blog/upgrading-pythons-sqlite/amp/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/yhat/pandasql/issues/77?email_source=notifications&email_token=AIYKYXL5WE4HLBATEIQNX3DQF4AYRA5CNFSM4IOXXTHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD46OXJQ#issuecomment-524086182,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIYKYXN5U2A3OMTWJEOCBU3QF4AYRANCNFSM4IOXXTHA
.

You may close the issue, if you are satisfied.

resolved


Introduce yourbones_ppx for literal tez values

This is my first ppx and I'm still a newbie.

I guess that I have adressed all of your comments. Thanks for the review.