Error upon importing RIPE database: index row size 4120 exceeds btree version 4 maximum 2704 for index "ix_block_description"

Got this with the fresh RIPE data and latest git master. The message repeats several times.
2023-08-18 10:18:37,545 - create_db - INFO     - MainProcess - ripe.db.inetnum.gz - parsing blocks
Process Process-21:
Traceback (most recent call last):
  File "/usr/local/lib/python3.11/site-packages/sqlalchemy/engine/base.py", line 2108, in _exec_insertmany_context
    dialect.do_execute(
  File "/usr/local/lib/python3.11/site-packages/sqlalchemy/engine/default.py", line 921, in do_execute
    cursor.execute(statement, parameters)
  File "/usr/local/lib/python3.11/site-packages/psycopg/cursor.py", line 737, in execute
    raise ex.with_traceback(None)
psycopg.errors.ProgramLimitExceeded: index row size 4120 exceeds btree version 4 maximum 2704 for index "ix_block_description"
DETAIL:  Index row references tuple (72907,28) in relation "block".
HINT:  Values larger than 1/3 of a buffer page cannot be indexed.
Consider a function index of an MD5 hash of the value, or use full text indexing.

Thanks this should be fixed now. Switched the column to a full text search index


Remove unavailable commands

See #43

Did you make sure that this list is compatible with https://github.com/hashbang/shell-etc/issues/43#issuecomment-168362794 ?

I re-checked available binaries after he made that comment, and tried not to remove those that got installed meanwhile

Tell me when you believe you are finished installing and I'll update the pull request :)

Or I'll just close it.

@schierlm I think I fixed today the install issues @daurnimator had, so this should finally get handled.

Going towards solving this definitely: I solved the “disappearing packages” issue, and I (re)installed the following packages:
aria2
bcrypt
bitlbee
cloc
clojure1.6
cowsay
dos2unix
finch
ghc
golang-go
html2text
imagemagick
ipcalc
pass
pv
rtorrent
socat

Ok, went over the diff again and updated it

Added back what got installed
If the binary and the program name differ, add the binary in square brackets

I hope I did not miss anything.

Ok, installed the following additional packages which seem universally useful:
encfs remind wyrd tudu dc qalc duplicity ranger atool p7zip zpaq nmap mtr iperf lftp zangband
 slashem frotz byobu tsung silversearcher-ag{,-el}

silversearcher-ag{,-el} are ag and it's Emacs integration.

Aside from that, I checked your diff and it seems ok.
I will happily merge this once you add the needed packages.

Added everything except iperf which hasn't been on the previous manpage either.


Where can I find the previous version documentation ?

We're still under very dynamic development so we don't have legacy docs yet.


Cannot read DHT sensor when pigpiod is running

I have a Raspberry Pi 1B+ with a AM2302/DHT22 sensor connected to pin (BCM)2. Pins 17, 22, and 27 drive a LED strip using PWM, controlled by Home Assistant via pigpiod.
I have a Python script which periodically reads the DHT sensor using the Adafruit_DHT library.
My issue is that when pigpiod is running, I can no longer read the DHT sensor. When I stop the pigpiod service, I can read it again without problems.
Any ideas? Pigpiod shouldn't affect unrelated pins, right?

when pigpiod is running, I can no longer read the DHT sensor.

It is not clear what you mean by this because the Python pigpio client requires pigpiod to be running.  Perhaps you meant to say: 'When the HA service (which, presumably, also communicates with pigpiod) is running, I can no longer read the DHT sensor.'
Please clarify.  Thanks.

I'm not using the Python pigpio client. I'm using the Adafruit_DHT Python library independently of pigpiod to read the DHT sensor.
When pigpiod is running (no matter if HA is connected to it or not), reading the DHT sensor fails.

Ah, thanks for the clarification.  I don't know the Adafruit lib but it is unlikely compatible with pigpio.  There are examples of using pigpio to read DHT22 sensor that would avoid a clash between different libraries.  However, I don't know exactly what the HA client is doing so there may still be an underlying issue.

Ah, so this is normal (while pigpiod is running, it's not possible to access GPIO pins via other means)? That wasn't clear to me. I thought that still would be possible, as long as you don't try to manipulate the same pins at the same time.

Yes.  This is because pigpio does direct register level access to the peripheral hardware.  As long as all IO is done through pigpio then different pins can be independently manipulated.

I see. Thanks for clarifying. The idea of that being the case crossed my mind, but I couldn't find any info on it.
I modified my Python script to use pigpio-dht instead of Adafruit_DHT, and that appears to be working!


License header should not use JavaDoc comment style

The license header uses JavaDoc style (/** */).
/*******************************************************************************
 * Copyright (c) 2015 Eclipse RDF4J contributors, Aduna, and others.
 * All rights reserved. This program and the accompanying materials
 * are made available under the terms of the Eclipse Distribution License v1.0
 * which accompanies this distribution, and is available at
 * http://www.eclipse.org/org/documents/edl-v10.php.
 *******************************************************************************/

This can cause some JavaDoc analyzers to wrongly recognize the license header as the JavaDoc for the class, in particular if there is no other class-level JavaDoc comment.
This can be fixed by changing the comment style to a normal block comment, e.g.:
/*
 * Copyright (c) 2015 Eclipse RDF4J contributors, Aduna, and others.
 * All rights reserved. This program and the accompanying materials
 * are made available under the terms of the Eclipse Distribution License v1.0
 * which accompanies this distribution, and is available at
 * http://www.eclipse.org/org/documents/edl-v10.php.
 */

or
/* *****************************************************************************
 * Copyright (c) 2015 Eclipse RDF4J contributors, Aduna, and others.
 * All rights reserved. This program and the accompanying materials
 * are made available under the terms of the Eclipse Distribution License v1.0
 * which accompanies this distribution, and is available at
 * http://www.eclipse.org/org/documents/edl-v10.php.
 * *****************************************************************************/

Also, the copyright year should probably either be removed or updated.

We use the copyright year to indicate the original contribution. As such, it should not be updated but reflect the year the file was first added to the project.
As an aside, note that the license header you quoted here is actually a legacy header that is only used for code that was originally part of OpenRDF Sesame. For any new contributions, we use this license header:
/*******************************************************************************
 * Copyright (c) ${year} Eclipse RDF4J contributors
 * All rights reserved. This program and the accompanying materials
 * are made available under the terms of the Eclipse Distribution License v1.0
 * which accompanies this distribution, and is available at
 * http://www.eclipse.org/org/documents/edl-v10.php.
 *******************************************************************************/

As for the formatting: our license header follows the prescribed format in general use within the Eclipse Foundation. See https://www.eclipse.org/projects/handbook/#ip-copyright-headers . I must admit I have never encountered any IDE or tool that had issues with this format in the way that you describe.

I do notice that apparently we have not always been good at enforcing this format, there's a couple of places where the license header is not formatted as it should be....


[DRAFT] quickfix of daggered MCRX/MCRY/MCRZ/CRX/CRY/CRZ

fixes #215 by implementing the dagger method properly, after saving the control qubit indices as a new attribute.
TBD:

[ ] unit tests to verify this
[ ] also fix other controlled and multi controlled operations, for their dagger methods.

thanks @vincentelfving! lets add some tests, do you want me to take that over?

@vincentelfving @dominikandreasseitz What is the status on this ?

i will finish this. sorry i didnt get to it last week


DRILL-8402: Add REGEXP_EXTRACT Function

DRILL-8402: Add REGEXP_EXTRACT Function
Description
Adds regexp_extract functions to Drill.
Documentation
This PR adds support for regexp_extract(<text>, <pattern>) which returns an array of text corresponding with the capturing groups in the regex.  It also includes regexp_extract(<text>, <pattern>, <index>) which returns the text of a specific capturing group.
SELECT regexp_extract('123-456-789', '([0-9]{3})-([0-9]{3})-([0-9]{3})');
+---------------------+
|       EXPR$0        |
+---------------------+
| ["123","456","789"] |
+---------------------+

SELECT regexp_extract('123-456-789', '([0-9]{3})-([0-9]{3})-([0-9]{3})', 0);
+-------------+
|   EXPR$0    |
+-------------+
| 123-456-789 |
+-------------+

SELECT regexp_extract('123-456-789', '([0-9]{3})-([0-9]{3})-([0-9]{3})', 3);
+--------+
| EXPR$0 |
+--------+
| 789    |
+--------+

Testing
Added unit tests.

Wouldn't this change introduce ReDoS vulnerability?

Wouldn't this change introduce ReDoS vulnerability?

Potentially, but we already allow REGEXP_REPLACE and REGEX_MATCHES, so I don't know that this actually makes anything worse.  I did try adding a validator with this saferegex1 but that library is not suitable for inclusion in Drill. (It prints all kinds of stuff to STDOUT.)

Ok, in this case, we can add this UDF.

@vvysotskyi  Should we proceed with this?  Is that a LGTM +1?

@vvysotskyi Thanks for the review.  I refactored the functions so that they are not creating extra String objects.


Add more rigorous style checks

Coverage remained the same at 100.0% when pulling 6e11966a68fc4fd2615923e735c9b10f0d78d3b4 on pytest-mypy into a079e139ddaf4ea4f2f1ca967d0641c2931d6d61 on master.


 If there’s a way to get the flags that I’m not aware of, please let me know

This hidden in the default output, but you can request it if you'd like:
{
  "class": "IN",
  "data": {
    "additionals": [
      {
        "flags": "",
        "type": "EDNS0",
        "udpsize": 512,
        "version": 0
      }
    ],
    "answers": [
      {
        "answer": "<IP_ADDRESS>",
        "class": "IN",
        "name": "google.com",
        "ttl": 163,
        "type": "A"
      }
    ],
    "flags": {
      "authenticated": false,
      "authoritative": false,
      "checking_disabled": false,
      "error_code": 0,
      "opcode": 0,
      "recursion_available": true,
      "recursion_desired": true,
      "response": true,
      "truncated": false
    },
    "protocol": "udp",
    "resolver": "<IP_ADDRESS>:53"
  },
  "name": "google.com",
  "status": "NOERROR",
  "timestamp": "2024-02-17T14:49:42-08:00"
}```


Brand image is missing

Currently the checks are failing for adaptive lighting, since there's no brand image stored here for it.
adaptive_lighting is not added to https://github.com/home-assistant/brands (more-info: https://hacs.xyz/docs/publish/include#check-brands )

I added a PRs for this:

https://github.com/home-assistant/brands/pull/2727
https://github.com/home-assistant/wheels-custom-integrations/pull/466

Should be fixed now. :)


feat: Add 0G plugin for file storage

This PR adds a new plugin for 0G.
Risks
Low. This change involves adding a new plugin, which is isolated and should not affect existing functionality.
Background
What does this PR do?
This PR adds a new plugin for 0G, which includes an action that allows users to upload local files to 0G Storage.
Future work will include extending 0G support for model serving, state persistence via KV store, and enhanced file management capabilities.
What kind of change is this?
Features (non-breaking change which adds functionality).
Why are we doing this? Any context or related work?
To extend the functionality of Eliza.
Documentation changes needed?
Require a change to the project documentation.
Testing
Where should a reviewer start?
The review can start with the new 0G plugin code, focusing on the action that uploads files to ensure it handles files correctly and integrates well with 0G Storage.
Detailed testing steps

Set Environment Variables

Configure the following variables:

ZEROG_INDEXER_RPC
ZEROG_EVM_RPC
ZEROG_PRIVATE_KEY
ZEROG_FLOW_ADDRESS
More details

Add 0g-plugin into character file
Ensure the @ai16z/plugin-0g is properly added and configured in the character file.

Interact with the Agent
Send a message to the agent: Upload the file at /path/to/your/file.txt to zg storage.

Screenshots
Before
After
Database changes
Deployment instructions

Thanks for your notification. Sure, I'll fix it ASAP.
Shakker Nerd @.***> 于2024年11月19日周二 23:09写道：

Hey @Wilbert957 https://github.com/Wilbert957 Great work!
There are conflicts that needs fixing, could you fix that?
In the package.json for 0g-plugin, kind add dev command ("dev": "tsup
--watch").
—
Reply to this email directly, view it on GitHub
https://github.com/ai16z/eliza/pull/416#issuecomment-2485981520, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BGT4B6HYQLKT3KSBOW3CFJT2BNICXAVCNFSM6AAAAABSBTEVX6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOBVHE4DCNJSGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey @Wilbert957 Great work!
There are conflicts that needs to be resolved before a merge could happen, could you fix that?
In the package.json for 0g-plugin, kind add dev command ("dev": "tsup --watch").

I've fixed it. Could you please check?

There are conflicts that needs to be resolved.
Kindly attach a screengrab of the upload/file storage interaction.
Could you write tests for this plugin?

Please resolve conflicts and add a test and / or screen grab of working plugin functionality thanks so much! Amazing work with this plugin you are the GOAT!

Please resolve conflicts and add a test and / or screen grab of working plugin functionality thanks so much! Amazing work with this plugin you are the GOAT!

Sure, thanks. I've attached a screengrab.

Hey @Wilbert957 Great work!
There are conflicts that needs to be resolved before a merge could happen, could you fix that?
In the package.json for 0g-plugin, kind add dev command ("dev": "tsup --watch").

Hi @shakkernerd
I hope you're doing well! I wanted to kindly remind you that my pull request (#PR-416) is awaiting your review and approval. Your feedback would be greatly appreciated.

Hi, @monilpat. Conflicts resolved. Would know if can merge? Your feedback would be greatly appreciated.


Fix bug #114.

Ensure that MethodChannel is initialized before invoke the loadAd method.

is that fix this error MissingPluginException(No implementation found for method loadAd on channel ?

I've tried with your fork, but the error still persists.

@bdlukaa Your plugin is better than googlers'. Please approve this PR and deploy new version!

@bdlukaa Your plugin is better than googlers'. Please approve this PR and deploy new version!


TensorRT conversion error for faster_rcnn_r50

Pretty much duplicate of this issue, but the offered solutions do not work I think (and the problem still persists).
I installed mmcv with plugins as described in docs. assert is_tensorrt_plugin_loaded() returns True. I am trying to convert faster_rcnn_r50_fpn_1x_coco which should be supported according to docs. I the other thread it is suggested that this may be due to recent changes but I tried all older commits that have the conversion and the same issue still persists.
Can somebody offer a setting where this error does not occur for faster_rcnn_r50_fpn_1x_coco?
Traceback:
root@a:/my_mmdetection# python tools/deployment/onnx2tensorrt.py tmp.onnx
Traceback (most recent call last):
  File "tools/deployment/onnx2tensorrt.py", line 179, in <module>
    workspace_size=args.workspace_size)
  File "tools/deployment/onnx2tensorrt.py", line 38, in onnx2tensorrt
    max_workspace_size=max_workspace_size)
  File "/my_mmdetection/mmcv/mmcv/tensorrt/tensorrt_utils.py", line 148, in onnx2trt
    raise RuntimeError(f'parse onnx failed:\n{error_msgs}')
RuntimeError: parse onnx failed:
In node 497 (importTopK): UNSUPPORTED_NODE: Assertion failed: inputs.at(1).is_weights()

ran in docker:
Cuda version 11.0
TensorRT version <IP_ADDRESS>
mmcv-full 1.3.1
Installed as described in the docs.

The following error is due to K in TopK should be int in TensorRT, not a Tensor, but it could be a Tensor in ONNXRuntime. You could comment out those codes of creating dynamic K for TopK to workaround.
In node -1 (importTopK): UNSUPPORTED_NODE: Assertion failed: inputs.at(1).is_weights()

If it is urgent, you could try with ONNX branch and refer to the pytorch2onnx and onnx2tensorrt

Hi thanks for your help. Which occuring of top_k do you mean exactly? The top_k proposals of rpn? Where in the code would I find this?
Also do you maybe have an environment where this conversion works?

@RunningLeon I understand that this is a beta feature. Do you know what commit of mmdet and mmcv you used for working conversion of fasterrcnn? I am trying to reproduce this setting.

onnx branch certainly could export fasterrcnn. Did you follow the instructions? Can you provide detailed error message?

@christian-holland Do you still have the issue?

yes, I am trying to find the difference at the moment if it works for you. Are you still using torch version 1.6 cuda 10.2 and mmcv built from source at current master?

You can refer to the doc.

The following error is due to K in TopK should be int in TensorRT, not a Tensor, but it could be a Tensor in ONNXRuntime. You could comment out those codes of creating dynamic K for TopK to workaround.
In node -1 (importTopK): UNSUPPORTED_NODE: Assertion failed: inputs.at(1).is_weights()

If it is urgent, you could try with ONNX branch and refer to the pytorch2onnx and onnx2tensorrt

Important info from the doc for pytorch->onnx->tensorrt:

All models above are tested with Pytorch==1.6.0 and onnxruntime==1.5.1

If the deployed backend platform is TensorRT, please add environment variables before running the file:

export ONNX_BACKEND=MMCVTensorRT

If you want to use the --dynamic-export parameter in the TensorRT backend to export ONNX, please remove the --simplify parameter, and vice versa.

Feel free to reopen it if you still have the issue.

@RunningLeon,
Hi, i met similiar issue for exporting trt model. i tried the the onnx branch as you mentioned and export the onnx model sucessfully but still failed to export trt model.

below are my software versions:
pytorch1.6.0
cuda10.1
cudnn7.6
onnxruntime1.5.1
tensorRT<IP_ADDRESS>

I compiled the mmcv with onnxruntime1.5.1 with MMCV_WITH_OPS=1,MMCV_WITH_ORT=1,MMCV_WITH_TRT=1, and generated 3 so files in mmcv root path.

pth file export to onnx file succeed with pytorch2onnx().

onnx file export to trt file failed with below traceback:
......(leaved out some trt verbose logs here)
[TensorRT] VERBOSE: ModelImporter.cpp:125: Where_492 [Where] inputs: [896 -> ()], [874 -> ()], [894 -> ()],
[TensorRT] VERBOSE: ImporterContext.hpp:150: Registering constant layer: 874 for ONNX initializer: 874
[TensorRT] VERBOSE: ImporterContext.hpp:154: Registering layer: Where_492 for ONNX node: Where_492
[TensorRT] VERBOSE: ImporterContext.hpp:120: Registering tensor: 897 for ONNX tensor: 897
[TensorRT] VERBOSE: ModelImporter.cpp:179: Where_492 [Where] outputs: [897 -> ()],
[TensorRT] VERBOSE: ModelImporter.cpp:103: Parsing node: Constant_493 [Constant]
[TensorRT] VERBOSE: ModelImporter.cpp:125: Constant_493 [Constant] inputs:
[TensorRT] VERBOSE: ModelImporter.cpp:179: Constant_493 [Constant] outputs: [898 -> (1)],
[TensorRT] VERBOSE: ModelImporter.cpp:103: Parsing node: Reshape_494 [Reshape]
[TensorRT] VERBOSE: ModelImporter.cpp:119: Searching for input: 897
[TensorRT] VERBOSE: ModelImporter.cpp:119: Searching for input: 898
[TensorRT] VERBOSE: ModelImporter.cpp:125: Reshape_494 [Reshape] inputs: [897 -> ()], [898 -> (1)],
[TensorRT] VERBOSE: ImporterContext.hpp:154: Registering layer: Reshape_494 for ONNX node: Reshape_494
[TensorRT] VERBOSE: ImporterContext.hpp:120: Registering tensor: 899 for ONNX tensor: 899
[TensorRT] VERBOSE: ModelImporter.cpp:179: Reshape_494 [Reshape] outputs: [899 -> (1)],
[TensorRT] VERBOSE: ModelImporter.cpp:103: Parsing node: TopK_495 [TopK]
[TensorRT] VERBOSE: ModelImporter.cpp:119: Searching for input: 881
[TensorRT] VERBOSE: ModelImporter.cpp:119: Searching for input: 899
[TensorRT] VERBOSE: ModelImporter.cpp:125: TopK_495 [TopK] inputs: [881 -> (-1, -1)], [899 -> (1)],
Traceback (most recent call last):
File "/home/suliang/suliang_git/mmdetection-onnx/sl_BIG/onnx2tensorrt.py", line 239, in
verbose=args.verbose)
File "/home/suliang/suliang_git/mmdetection-onnx/sl_BIG/onnx2tensorrt.py", line 42, in onnx2tensorrt
max_workspace_size=max_workspace_size)
File "/home/suliang/suliang_git/mmcv/mmcv/tensorrt/tensorrt_utils.py", line 52, in onnx2trt
onnx_model.SerializeToString()), 'parse onnx failed.'
AssertionError: parse onnx failed.

Process finished with exit code 1
i am not very sure anything i missing or if you can give any more suggestion?
You mentioned we can comment the topK  code area to workaround, can you give more detail on how to to do that or on which position i can do?
thanks in advance.

Currently, we do not have this plan.
If you set export ONNX_BACKEND=MMCVTensorRT before run pytorch2onnx.py, then the K of TopK would be set to constant in the ONNX model and there would be no error of Assertion failed: inputs.at(1).is_weights().

@RunningLeon If you say that the conversion works for you, can you maybe provide an installation script that works from your provided docker container? Then I can investigate the difference to my build.


Pure native android support

Is there any plan to build a port of this for native android? We were hoping to render Katex directly on an android customview, without any kind of webview in between
Has any thought gone onto this? Is this feasible?

In order to do this we really need to hammer out an intermediate representation and then migrate existing code to use this representation.  Once we have that we can work on separate backends.  Echoing what @edemaine said, this is a rather large effort.  I wonder if it would be easier to adapt an existing Java-based TeX rendered for use on Android.

We are doing somewhat similar thing. (though not exactly the same as this issue).
https://github.com/karino2/kotlitex/
This is basically

port KaTeX to kotlin
port canvas-latex to android native canvas

Of course kotlin is different language and sometime it's impossible to map 1 by 1 from node code to kotlin code.
But we try to keep code as equal as possible.
We are not yet even in alpha stage, but basic structure of code is determined and current code base already show some feasibility of our idea.

Closing in favor of existing issues.


Add support for SameSite cookies

First version for review
Description
Fixes #3442
Added support for setting SameSite on cookies. https://tools.ietf.org/html/draft-ietf-httpbis-rfc6265bis-05
Allows setting the SameSite parameter for cookies and supports both PHP 7.2 and 7.3+ version.
Includes hacky support for PHP 7.2 for setting SameSite in the path - see SO https://stackoverflow.com/questions/39750906/php-setcookie-samesite-strict (for example)
Code contains version checks if less than 7.3.0 and the tests also differs.
I cannot get the unit tests to pass locally, so this is to check if travis is able to run them through.
Checklist:

[X] Securely signed commits
[X] Component(s) with PHPdocs
[X] Unit testing, with >80% coverage
[X] User guide updated
[X] Conforms to style guide

Seems like session mock wasn't correct. Now all tests pass.

I have done some initial testing in both 7.2 and 7.3 and see that Session is working and the cookie's SameSite value is set correctly. Also the CSRF cookie is set with the correct SameSite. Implications of setting SameSite on CSRF is beyond my understanding though.

@michalsn
Lax is the new default, makes sense. PHP 7.2 will still be supported for a couple of months....

@michalsn
I have also spent the last days thinking about this. Since I am not 100% OK with having a feature added without any way to turn it off, I initially defaulted to $cookieSameSite = ''; to indicate that the SameSite attribute should not be set at all. After som back and forth with @paulbalandan we decided with 'Lax' as the best default value. This makes my initial idea with '' as way to avoid setting the attribute at all more complicated to implement.
I wonder if it would be ok to allow explicitly setting $cookieSameSite =null; (or possibly false) to completely disable setting the SameSite attribute in case this is required for some reason?

@tangix Honestly, I see no point giving the ability to not set SameSite policy. All modern browsers go with some default value for it, if not Lax then other. Also if SameSite policy is not specified explicitly then we got warnings.
We should promote building a secure code and from what I see (looking at how browsers support it), it's already a "new standard".
In general, I would like to go with the default value Lax for every cookie and developers can change it if they want.

In general, I would like to go with the default value Lax for every cookie and developers can change it if they want.

Lax it is then. I will update the PR with this, add a throw for bad setting and fix the tests.

Based on https://github.com/codeigniter4/CodeIgniter4/issues/3442#issuecomment-671283004 :
May we should let this functionally be handled by core PHP and skip support for 7.2 (that is EOL in 4 months)? I've put this RP as Draft again so it won't get merged by accident...

Let's hope that next week someone else will have enough time to look at this PR from a different perspective and give us feedback.

@tangix Seems like you were right about the option to not set the SameSite at all. I missed that PHP docs state that. I think it should be handled as you proposed earlier - so that null value means SameSite shouldn't be set.
Personally I still think it has not much sense and may cause problems in some cases but since this is the supported option in the native function then I guess we have no choice but to follow :)

@tangix Seems like you were right about the option to not set the SameSite at all. I missed that PHP docs state that. I think it should be handled as you proposed earlier - so that null value means SameSite shouldn't be set.

null is not be the easiest setting to deal with throughout the code. To keep consistency with the other settings, an empty string '' may be OK?

I'm not sure if null can cause any problems in implementation but you convinced me with a statement about consistency. An empty string seems to be the right choice 👍

@tangix Thank you for taking your time and working on this! I think this is a really important feature. Thanks!

@tangix Thank you for taking your time and working on this! I think this is a really important feature. Thanks!

 CodeIgniter is an integral part of my business and I spend all of my PHP-development in either 3.0 or 4.0. I absolutely hate having to tinker with the core-files so giving back to the community what I fix gives a warm cozy feeling.


Testing unexpected crashes

Check what data is lost when app unexpectedly crashes during drill.

In all tested cases, all results data is lost. However, also in all tested cases future results are not corrupted due to lack of persistence. Only possible corruption of data should be fixed with #109


POST Error from  /stott.security.optimizely/api/cspreporting/reporttoviolation/ in Azure Application Insights

We have noticed several POST errors originating from "/stott.security.optimizely/api/cspreporting/reporttoviolation/" in Application Insights. These errors have been occurring since our website went live on March 28th, with a response code of 405.

Hello @Sabah-Tahir ,
I have had another person reach out with a similar issue. The exception information revealed that this was caused by the information being too large for the database index.
Can you confirm if your exception information says the same cause?
Ideally to reproduce and resolve this issue I will need the payload sent to the report-to endpoint. Can you provide this or share a URL where this is being raised so I can try and capture this?
Please note that error handling exists and this merely means you won't have the data for this violation saved. This will not prevent the Add-on from protecting your website.
Regards
Mark

Thank you, @GeekInTheNorth, for your prompt response. Are you considering adding the functionality to truncate the reported URI if it exceeds the threshold? I'm not sure what the next steps are for us in this situation.

Hello @Sabah-Tahir
Yes I will be implementing a fix to truncate a reported URI if it exceeds the threshold.
You do not need to worry or take any actions.  The fact a browser has sent this report means the browser has already blocked the URL in question.
When a fix is released, I'll tag you here on this issue to let you know that the updated version is ready to install.
Regards,
Mark

Great, Thanks @GeekInTheNorth

I've updated the CSP Violation Reporting as such:
When cleaning the report data to save the report:

Check the reported URL

Clean the URL to just the Host and Path
If the Host and Path exceeds 255 characters, then clean the URL to just the Host
If the Host alone is still greater than 255 characters, then don't save the violation report

Check the violated directive

If the violated directive is not in the list approved by the module, then don't save the violation report

Hello @Sabah-Tahir
This is now live in version 2.6.0, you can read more here: https://github.com/GeekInTheNorth/Stott.Security.Optimizely/discussions/207

Thanks @GeekInTheNorth. I will try this version and will let you know how it goes.


Ignore Jackson updates in dependabot config

since we align the version with the one used in Wildfly

since we align the version with the one used in Wildfly

FWIW you might want to give that decision some more thought, since Quarkus likely won't follow the same upgrade policy as WildFly, and we need to be compatible with both.

😔 hmmm, maybe add an upgrade job that bumps Jackson to the version that Quarkus uses? while the main branch is using the version from Wildfly?


Renamed EvictFunc to DeallocateFunc.

Release note:
None

Hey @k82cn,
Something went wrong with the build.
TravisCI finished with status errored, which means the build failed because of something unrelated to the tests, such as a problem with a dependency or the build process itself.
View build log
TravisBuddy Request Identifier: 700ce810-da6b-11e8-8964-ad637cdcaaf8

Hey @k82cn,
Something went wrong with the build.
TravisCI finished with status errored, which means the build failed because of something unrelated to the tests, such as a problem with a dependency or the build process itself.
View build log
TravisBuddy Request Identifier: c2ce98d0-da6d-11e8-8964-ad637cdcaaf8

/lgtm

/lgtm


Samsung backend update for sub v3.1

will fix the formatting issue shortly and update

@AhmedTElthakeb I uploaded the new Samsung libs to our CI storage in https://github.com/mlcommons/mobile_app_open/pull/763. Please review and merge it, then after the CI build is completed, please also download and test the unified APK generated by the CI. Thank you.


Update US English artist regex for more common phrases

Expands the artist regex to cover a few new combinations. For instance:
"Play music by <artist>"
"Play something from <artist>"
"Play stuff by <artist>"

It looks good. will do a proper test and then go ahead and merge.
Thanks!

Sorry for the delay. Tested and working fine! Merging now.


Carousel - is it possible to turn off keyboard navigation?

I am finding that I cannot turn off keyboard navigation within the carousel. The carousel panes are sliding/changing when I use the arrow keys and I can't find a way to turn this behavior off.
I can hook into .on('keydown.bs.carousel' event, but unfortunately, this fires after the carousel has executed both 'slide; and 'slid' events.
Is there another event I can hook into to determine how the carousel is being advanced?

Allright, using data attr or via js, it works, but there is an strange issue in my case.
I have disabled the key reaction in caraousel which is working fine, but i have a popup in the lets's say 5th slide, and used the bootstrap modal for popup. Now the problem is when bootstrap modal is showing, the keyboard false does not prevent the carousel from sliding? Any help is appreciated. Also tried the solutions to prevent action using event listener and others. By the way my pop modal is not in the carousel.  It is just triggered with in the carousel slide.


Clean up markdown files

Sorry in advance, I didn't intend to touch so many files. The changes should be pretty simple though.
This PR does the following to markdown files:

Add introduction sections to networking specs.
Remove "This document is a work-in-progress" from stable specs.
Add "This document is a work-in-progress" to unstable specs.
Delete double blank lines (not in Python code chunks).
Add blank line after headers.
Remove trailing whitespace.

thank you very for help. my phone my computer have the hacker. I try my way don't   work. English in the phone is gone for long time ^^
thank you Dev.  😊


child components not showing

hello! thankfully I found this library, I have two small questions.

How do I set the background color(I've tried configuring it in the params object just that doesn't seem to have an effect)
2.How would you suggestI render the rest of my app's components?

as of now, the particles render to the screen but not anything else.

Here's what I am working with:
import Main from './components/Main';
import React from 'react';
import ReactDOM from 'react-dom';

import Particle from './container/Particle'

ReactDOM.render(
  <Particle>
    <Main />
  </Particle>,
  document.getElementById('app')
)

I've abstracted Partcle into it's own file becasue of it's length

Quick Update: I modified my code to look like this:
import React from 'react';
import ReactDOM from 'react-dom';

import Particle from './container/Particle'

ReactDOM.render(
  <Particle/>,
  document.getElementById('app')
)

and the Particle component looks like this:
`` import React, {Component} from 'react'
import Particles from 'react-particles-js';
import Main from '../components/Main';
const Particle = () => {
return (
<Particles
params={{ alll th params}}>

)
}
export default Particle

But still no luck

Hi, you can take look at the demo source to see how to solve both of your problems:

The background-color can be set in the element containing the <Particles /> component; in the example I set the background-color in the body element, but it can be done in the element that contains the component i.e. the .wrapper div.

Currently the component does not support children; you can set however a fixed/absolute/relative position or a float for your particle component, then put other components beside the particle component, and style them to show over the particle component. This is because canvas cannot have children tags.

That's all great information!
I did end up looking at the demo's source code and gave it a fixed
position. That code was very helpful and solved the issue 👍🏼
I would have closed the issue earlier but I got lost in coding.
Thank you for the response and tips!
Best,
Noel
On Fri, Jun 30, 2017 at 11:56 PM Simone Bembi<EMAIL_ADDRESS>wrote:

Hi, you can take look at the demo source
https://github.com/Wufe/react-particles-js/blob/master/demo/index.html
to see how to solve both of your problems:

The background-color can be set in the element containing the  component; in the example I set the background-color in the body
element, but it can be done in the element that contains the component i.e.
the .wrapper div.
Currently the component does not support children; you can set however
a fixed/absolute/relative position or a float for your particle component,
then put other components beside the particle component, and style them to
show over the particle component. This is because canvas cannot have
children tags.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Wufe/react-particles-js/issues/8#issuecomment-312415324,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AUMvu2TvsLra_i-HVoJveUVaghNeGbMmks5sJe2kgaJpZM4OLLZv
.

You are welcome.
Feel free to open another issue if you have suggestions for improvement

Thanks!
Last quick observation: I notice that my site only recognizes my mouse
after having has clicked in the page.
Is there a way to edit that in the params prop?
Best,
Noel
On Sat, Jul 1, 2017 at 1:10 AM Simone Bembi<EMAIL_ADDRESS>wrote:

You are welcome.
Feel free to open another issue if you have suggestions for improvement
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Wufe/react-particles-js/issues/8#issuecomment-312418351,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AUMvu-6YTorFp62zHTZaMAJ8vbhQFkwvks5sJf75gaJpZM4OLLZv
.

I do have onhover enabled.
On Sat, Jul 1, 2017 at 12:12 PM Noel Parisi<EMAIL_ADDRESS>wrote:

Thanks!
Last quick observation: I notice that my site only recognizes my mouse
after having has clicked in the page.
Is there a way to edit that in the params prop?
Best,
Noel
On Sat, Jul 1, 2017 at 1:10 AM Simone Bembi<EMAIL_ADDRESS>wrote:

You are welcome.
Feel free to open another issue if you have suggestions for improvement
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Wufe/react-particles-js/issues/8#issuecomment-312418351,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AUMvu-6YTorFp62zHTZaMAJ8vbhQFkwvks5sJf75gaJpZM4OLLZv
.

What browser are you using? In chrome the canvas recognizes the mouse without a click

No comments for 3 months. Closing the issue.


GabutDM it looks half backed but still GTK

Gabut Download Manager - https://github.com/gabutakut/gabutdm

Closed with #115.


Default :max_breadcrumbs config not picked up after updating to v10.1

Hi sentry team!
I'm running into a configuration issue after bumping from v9.x to v10.x -> the :max_breadcrumbs default config does not seem be picked up, which might be related to how the config is now stored as a persistent term.
Environment
macOs Ventura 13.0 (M2)
erlang <IP_ADDRESS>
elixir 1.15.4-otp-25
sentry 10.1.0 -> :max_breadcrumbs was not previously configured
Steps to Reproduce

Update versions from 9.x -> 10.x
Attempt any mix task (mix test, mix compile, etc.)
Receive compilation error message

Expected Result
Successful compilation
Actual Result
All mix tasks now fail with the following compilation error:
== Compilation error in file <file_name>.ex ==
** (ArgumentError) errors were found at the given arguments:

  * 1st argument: no persistent term stored with this key

    :persistent_term.get({:sentry_config, :max_breadcrumbs})
    (sentry 10.1.0) lib/sentry/config.ex:504: Sentry.Config.max_breadcrumbs/0
    (sentry 10.1.0) lib/sentry/context.ex:421: anonymous fn/2 in Sentry.Context.add_breadcrumb/1
    (elixir 1.15.4) lib/map.ex:676: Map.update/4
    (sentry 10.1.0) lib/sentry/context.ex:419: Sentry.Context.add_breadcrumb/1
    (stdlib <IP_ADDRESS>) erl_eval.erl:748: :erl_eval.do_apply/7
    (stdlib <IP_ADDRESS>) erl_eval.erl:136: :erl_eval.exprs/6
    <file_name>.ex:1: (file)

I have tried the following:

Add :max_breadcrumbs to the :sentry configuration -> does not resolve compilation issue
Attempt to set the persistent term with :persistent_term.put({:sentry_config, :max_breadcrumbs}, 100) in Application.run -> does not resolve compilation issue
Add Sentry.put_config(:max_breadcrumbs, 100) before Sentry.Context.add_breadcrumb/1 is invoked -> this remedies the issue however is not recommended by the documentation

@kwardynski hiya, thanks for the bug report! 🙏
My best guess from the info you gave us is that the Sentry application is not starting. So, the first question I have is: do you by any chance have runtime: false or something like that alongside the :sentry dependency?
If that's not the case, we'll need a bit more info to try and debug this:

The Sentry config you're using
How and where you're calling Sentry.Config.max_breadcrumbs/0

Hey @whatyouhide, thanks for helping dig into this!

:runtime is not set to false for :sentry
We do not call Sentry.Config.max_breadcrumbs, my understanding is this defaults to 100. We call Sentry.Context.add_breadcrumb in a few places, this function call leads to the compile time error I posted above.

Here are the :sentry configs:
# config/prod.exs
config :sentry,
  release: version_name,
  enable_source_code_context: true,
  root_source_code_paths: [File.cwd!()]

# config/runtime.exs
if config_env() == :prod do
  config :sentry,
    dsn: env!("SENTRY_DSN", :string?, nil),
    environment_name: infra_env,
    before_send: {MyApp.Sentry, :before_send}
end

# config/test.exs
config :sentry,
  environment_name: "test"

@kwardynski awesome, thanks for the additional detail. Are you calling Sentry.Context.add_breacrumb/1 at compile time?

Yes we are... That might be the cause of this I'm guessing?
Could you confirm at which version the :max_breadcrumbs config moved to :persistent_term? We haven't seen this issue? We just did the bump from 9.0 to 10.1 and this "problem" is new

Yeah the bump is in 10.0. The cause is that you're calling this at compile time, that is correct. If I may ask, what's the use case?

We have a macro which defines an add_breadcrumbs/x function, which we then invoke with a decorator (@decorate_all). Is :sentry still able to support breadcrumbs at compile time or do you have a workaround we could use? Right now we're calling Sentry.put_config(:max_breadcrumbs, 100) above the call to Sentry.Context.add_breadcrumb in the macro as a workaround, but the docs discourage that.
I would guess compile time support would make sense for breadcrumbs so that functionality could be used in decorators/macros but I could be wrong.

I don't think that's how it works. Even if you write a macro that injects Sentry.Context.add_breadcrumb/1 calls at compile time into decorated functions, those calls should still happen at runtime for things to be working correctly. add_breadcrumb/1 uses the process dictionary to store the breadcrumbs, which means that you're not adding those breadcrumbs to the actual caller, but to the process compiling your code.
If you are able to, can you share the implementation of the relevant pieces (the macro, the decorator)? If yes, I could try and help to figure out if there's a way to do what you want to do. 🙃

Hey @whatyouhide
Sorry for the delayed response.
Just to clarify, it's not the storing of the breadcrumbs that's causing the compile time issue, it's accessing the :max_breadcrumbs parameter which is stored in :persistent_term (at least that's what I gather from the call stack)
Here is some anonymized and condensed code for the implementation. The Sentry.Context.add._breadcrumb function is accessed in two ways

Inherited directly from the MyApp.SentryDecorator macro

defmodule MyApp.SentryDecorator do
  def add_breadcrumbs(body, _context) do
    quote do
      {function, arity} = __ENV__.function

      # This was not needed before v10
      # -----
      Sentry.put_config(:max_breadcrumbs, 100)
      # -----

      Sentry.Context.add_breadcrumb(
        type: :function,
        message: "#{__MODULE__}.#{function}/#{arity}"
      )

      unquote(body)
    end
  end
end

defmodule MyApp.SomeContext do
  use MyApp.SentryDecorator

  @decorate_all add_breadcrumbs()

  def foo, do: bar()
end

Inherited indirectly through another macro...

defmodule MyApp.SentryDecorator do
  def add_breadcrumbs(body, _context) do
    quote do
      {function, arity} = __ENV__.function

      # This was not needed before v10
      # -----
      Sentry.put_config(:max_breadcrumbs, 100)
      # -----

      Sentry.Context.add_breadcrumb(
        type: :function,
        message: "#{__MODULE__}.#{function}/#{arity}"
      )

      unquote(body)
    end
  end
end

defmodule MyApp.Service do
  defmacro __using__(_) do
    use MyApp.SentryDecorator
    @decorate_all add_breadcrumbs()
  end
end

defmodule MyApp.SomeOtherContext do
  use MyApp.Service
  def bar, do: baz()
end

@kwardynski no need to apologize 😊

Where does the @decorate_all attribute come from, another library?
Where are you calling MyApp.SomeOtherContext.bar/0?

@whatyouhide

yes - :decorator v1.4
sort of a non-answer, but lots of places.

To elaborate on 2 - no module which uses the MyApp.Service module will compile without the call to Sentry.put_config in MyApp.SentryDecorator.  When running any mix task (i.e. compile, test, etc.) there will be an error trace:
== Compilation error in file lib/my_app/some_other_context.ex ==
** (ArgumentError) errors were found at the given arguments:

  * 1st argument: no persistent term stored with this key

    :persistent_term.get({:sentry_config, :max_breadcrumbs})
    (sentry 10.1.0) lib/sentry/config.ex:504: Sentry.Config.max_breadcrumbs/0
    (sentry 10.1.0) lib/sentry/context.ex:421: anonymous fn/2 in Sentry.Context.add_breadcrumb/1
    (elixir 1.15.4) lib/map.ex:676: Map.update/4
    (sentry 10.1.0) lib/sentry/context.ex:419: Sentry.Context.add_breadcrumb/1
    (stdlib <IP_ADDRESS>) erl_eval.erl:748: :erl_eval.do_apply/7
    (stdlib <IP_ADDRESS>) erl_eval.erl:136: :erl_eval.exprs/6
    /Users/kacperwardynski/git/server/lib/my_app/some_other_context.ex:1: (file)

@kwardynski yeah, I understand that no modules compile without put_config/2. I’m pretty sure that the reason that happens is just that the :sentry application hasn't started when your add_breadcrumbs/2 function executes. You can confirm this by inserting this line before the call to put_config/2:
dbg(List.keyfind(Application.started_applications(), :sentry, 0))

If that prints out nil, then that's the issue: the :sentry application was not started yet.
If that's the case, then the issue here is that the add_breadcrumbs/2 function is called before the :sentry app has been started. You said that this is called in lots of places, so I’m not sure where that would happen. 😕

@whatyouhide
Confirmed that :sentry is not started when we're compiling the add_breadcrumbs function in the SentryDecorator module.
Confident to say this is an "us" issue than a "you" issue so I'll close this. Thank you for your help!

Awesome! Glad we figured it out. I also tried to improve the error message here in 05c4c22. The message you got was pretty horrible, my bad 🙃 Thoughts?

@whatyouhide That's way more helpful 👍


fix: orphan mcs-syncer role

What type of PR is this?
bug
What this PR does / why we need it:
fix orphan mcs-syncer role
Which issue(s) this PR fixes:

Fixes #814
Special notes for your reviewer:

@zyue110026 thanks for you PR.
There is also a place: additionalRoles in approver.go
No need to add role "mcs-syncer" to child cluster anymore, remove the role "mcs-syncer" or you will meet error.

@lmxia could you point out the location of additionalRoles?

@lmxia could you point out the location of additionalRoles?
Sure, you have  found it precisely! This PR looks good to me, thanks for you commit.


Add clodinary destroyer to delete image in cloud space

Description
I add a clodinary destroyer to delete that images which was delete from mongodb database in edit.ejs file.

This PR does the following:

Adds ... Cloudinary destroyer to delete list image from cloud

Related Issues
Link any related issues using the format Fixes #issue_number.
This helps to automatically close related issues when the PR is merged.

Placeholder: "Fixes #334 "

Screenshots (if applicable)
Add any screenshots that help explain or visualize the changes.
https://github.com/user-attachments/assets/beed76d3-e352-46bf-be05-606c33503ce0
Checklist
Make sure to check off all the items before submitting. Mark with [x] if done.

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] I am working on this issue under GSSOC

@Soujanya2004 Please review this PR. After assign it proper label, marge it.

https://github.com/user-attachments/assets/fce037c3-787f-4672-8095-27fb3110f300


openbsd: constantify some arguments of openpty() and forkpty()

OpenBSD recently made termp and winp arguments of openpty() and forkpty() const.
to match the prototypes in glibc and musl libc.
It was done in following commit, which will be part of OpenBSD 7.2 .

Some changes occurred in OpenBSD module
cc @semarie
Some changes occurred in OpenBSD module
cc @semarie

r? @Amanieu
(rust-highfive has picked a reviewer for you, use r? to override)

@bors r+

:pushpin: Commit 90fb41c2be8fbce92a82bf43a47aa194d58ac741 has been approved by Amanieu

:hourglass: Testing commit 90fb41c2be8fbce92a82bf43a47aa194d58ac741 with merge a2addeee278f6f716b9fd020a14b4c595faaf787...

:broken_heart: Test failed - checks-cirrus-freebsd-11

@bors retry

:hourglass: Testing commit 90fb41c2be8fbce92a82bf43a47aa194d58ac741 with merge 9a6c13a328b9b5c3ca5582e6cb536c70d9f5bbf6...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: Amanieu
Pushing 9a6c13a328b9b5c3ca5582e6cb536c70d9f5bbf6 to master...


zcash: 4.4.1 -> 4.5.0

Motivation for this change
Update to latest version
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

@centromere FYI there will be a new 4.5.1 release of zcash soon:
https://github.com/zcash/zcash/pull/5330/files#diff-e04cb1e9f29f76897c6b84334238f621ff45ba13cf180ad1d0d442d50ebe51f7R7
But it only impacts testnet nodes:

These bugs do not affect mainnet, as zcashd v4.5.0 only set the activation
height for NU5 on testnet for testing purposes.


Test fails, times-roman.pm seems to be missing

moritz@hack:~/p6/perl6-Font-AFM$ perl6-m -Ilib t/font-afm-metrics-times.t
1..8
Could not find Font::Metrics::times-roman in any of:
  file#lib
  file#/home/moritz/.perl6/2015.09-219-gc52b0b3/lib
  inst#/home/moritz/.perl6/2015.09-219-gc52b0b3
  file#/home/moritz/p6/rakudo/install/share/perl6/lib
  file#/home/moritz/p6/rakudo/install/share/perl6/vendor/lib
  file#/home/moritz/p6/rakudo/install/share/perl6/site/lib
  inst#/home/moritz/p6/rakudo/install/share/perl6
  inst#/home/moritz/p6/rakudo/install/share/perl6/vendor
  inst#/home/moritz/p6/rakudo/install/share/perl6/site
  in block <unit> at t/font-afm-metrics-times.t:5

# Looks like you planned 8 tests, but ran 0

times-roman.pm is also referenced in META.info, but not included in the
repository. Please add it :-)

Have checked in generated AFM modules.
This module had a Build.pm Panda builder script, which was overkill.
Have replaced with make_metrics.pl, designed to be run occasionally to regenerate the AFM modules, which can then be checked in.
This is how the Perl 5 module does it.


implement display of @rend="check"

we recently decided to allow this value of @rend in the schema. but it seems that the display is not yet implemented for the editions of inscriptions. see e.g. DHARMA_INSIDENKKanjuruhan.xml.

The issue is than the code from Epidoc hasn't been made to allow such implementation. So a generic rule won't work, I will have to implement it manually to all elements.

is it possible to obtain from the EpiDoc community that the common code is changed so as to allow you to do what needs to be done mor easily?

Not for such a feature, the stylesheets epidoc are for final rendering in website. I will add manually the condition when necessary. I have just pushed the code for seg so that it can be rendered in DHARMA_INSIDENKKanjuruhan.xml. Since it is using css it can take some times to be displayed.

I have the feeling that <seg rend="check"> is not working. See e.g. https://dharmalekha.info/texts/INSIDENKSilamanikundala and its sources code.

Fixed in https://github.com/michaelnmmeyer/dharma/commit/2a30111d14809f63630ff0a3b14c0351a0a05bd5.


Add constant folding and propagation

Iteratively fold and propagate constants
Implement methods for emulating YOLOL ops

Use Yoloxide's YololNumber for ops
https://github.com/Jerald/yoloxide/blob/master/src/types/yolol_number.rs

Done for now, until Matrix finishes his yolol-number crate


STM32 counter driver error in estimating alarm time

Describe the bug
The RTC alarm of the counter driver for STM32 devices fails if the RTC time and date have been initialized with the current date. The initialization process is done using the low-level stm32 libraries. Then, if the alarm is set for a few seconds, it will not be triggered. Here you can find a sample.
After investigating the stm32 counter driver, I found that the estimated alarm time has the year 2000 as a base and is used as an input to the gmtime_r function, to produce the alarm date and time. The gmtime_r function needs as input, time from 1970. I believe the problem is here. Although this approach works because the output year will not be used in the process, 2000 is a leap year while 1970 is not. As RTC progresses, the time that the alarm is calculating will have one day difference due to the leap year.
To test my theory, I run the same example with a date before the end of February 2000 (10/1/2000) and it works. Then I run it one more time with 28/2/2000 23:59:00 as date and time. The code runs successfully for almost one minute and then stalls.
The issue can be solved by adding the 2000 timestamp to the estimated time and can be found here.
All the above test run faultlessly.
To Reproduce

from here, build the sample/counter/stm32_counter sample. There are three configuration files for the different dates mentioned above.

Flash it to nucleo_l496zg using west flash.

Open a serial console.

Expected behavior
The application should print the date and time every few seconds (ex: “Date is 2021:01:12:10:00:00”)
Impact
High impact for applications with wakeup alarm.
Environment:

OS: Linux Mint

Toolchain  Zephyr SDK

Commit SHA 88725260e622ab4d0a71a267f0a67b99f1ec6d0e

@Antonis510 Many thanks for reporting the bug, solution and way to reproduce, this really helps!


getRobotCommand(...) in poscontrollers should have avoidBallDistance as a parameter

to avoid obscurity in poscontrollers we should just always indicate what kind of poscontrol features we want in the request: when we want a command that drives to our position we should specify whether to avoid the ball or not etc. instead of setting them as class variables. (because class variables are remembered and can thus influence other skills)

So are you also making defense area's and others explicit here?

hmm. thinking about this. The defence area usually as no edge cases except what we can do from the referee, so it's probably only about the referee. the setCanMoveInDefenseArea could maybe be removed then.  The only feature that really changes a lot is the ball avoidance distance. However, I can imagine that we sometimes also want to set a min-distance to defense area. I don't wanna make this complex so I'm still thinking of a neat solution

related: #528

So this issue should be closed, because it is related to #528 and #528 is closed?

As far as I can tell, the PR that closed #528 did not do anything related to this issue. I'm not sure what this issue even is about anymore though, so I can't really tell you if it's still relevant.

This issue is related to rulesets.
getRobotCommand takes some parameters as to what path it needs to take. Sometimes we actively need to avoid the ball. Currently I believe this is a boolean which either avoids the ball or not. I think it would be nice to have a variable distance to avoid the ball. With 0 being no avoidance. This is something I'd like because there are multiple ways we could avoid a ball.
During play you sometimes want to avoid it because it is passed to another robot, but you don't need to avoid it for a full meter then. Avoiding it with 30cm would be more than enough, while for ball placement you need to have a strict minimal distance of 0.7m I believe.


feat: add kubeletVersion to the K8sNodeSample data

Added kubeletVersion to the node data
updated readme.md and skaffold.yaml as these values seemed necessary to build and deploy locally

Please let me know if there is anything else that needs to be updated.

This looks great, thanks for your efforts!
Overall seems good to me, I would only ask to add this new metric to the jsonschemas we use for e2e testing, likely here and here. Otherwise we won't be able to identify regressions or incompatbilities.

Nice!


turn auto follow into a command that can take a character name as an argument

Currently the implementation of auto follow only allows the party lead
Also, if not a command, the ability to auto disable when not in an instance would be nice (and reenable when back in) since that's what I'd automate with the command anyway

Outdated.


fix(guides): show guides for first child and adjust start index

Fixes #106
Before:

After:

Thanks for this! Looking much more consistent :)


S4cmd not working for DreamObjects or Other object storage services.

I just switched from s3cmd to s4cmd. It's working great for EC2 to Amazon S3 data transfer. But when I'm trying DreamObjects or CenturyLink Object storage in it's place, it shows the below error:
ClientError: An error occurred (InvalidAccessKeyId) when calling the ListBuckets operation: The AWS Access Key Id you provided does not exist in our records.
I properly configured .s3cfg for DreamObjects, I know it because when I type s3cmd ls it lists the buckets. But when I type s4cmd ls it gives the above error.
So is s4cmd not compatible with other object storage services currently?

had the same issue and it looks like s4cmd does not support the endpoint_url parameter.
There is a merge request though: #82

Dupe of #107

A pullrequest doing just this was merged into master a couple days back (#82 ). Should be fine to work with any endpoint using --endpoint-url. We're in the process of updating our test-suite to use minio to ensure this works E2E, but you can play around with it in the meantime! (You'll have to build from the latest code until we update pip et al)
Closing this issue for now, but let me know if you face any issues and I'll reopen it! :D

Hi,
while "ls" operation seems to be working, "put" doesn't and end up with:
[Exception] Parameter validation failed:
Invalid length for parameter Key, value: 0, valid range: 1-inf
[Thread Failure] Parameter validation failed:
Invalid length for parameter Key, value: 0, valid range: 1-inf

@Xp0is0nX I had that problem too but adding a / to the end of my bucket seemed to fix it. Ie s3://backups/ instead of s3://backups. You may have moved on but thought this may be worth mentioning.


Benchmarking some alternatives

Using the same data and setup that this prefix trie uses I compared some other alternatives we were considering for contact suggestions.

A plain list - faster than we were expecting, but right on the cusp of what we considered to be okay performance.
Naive suffix trie - Just adding all suffixes of the keys you'd normally add to the prefix trie just to get an estimate of what it would look like.  The populating performance should increase by a fair amount if we actually wrote a real implementation.
Full text search - Both of the popular engines we tried seemed pretty reasonable, main issue is that we'd be doing a fair amount of post filtering and sorting afterwards.

See this Notion doc for our notes during brainstorming.

We can always resurrect this later if needed, but this PR is really just a dump of the data that we collected.


SemanticUI config issue in v2.0.1

In version 2.0.1 semantic-ui.css is being added to vendor.css even if css import is set to false in config. Though in 2.0.0 it works as expected.

Thanks for reporting this. Looks like something must have changed when I updated the ember dependencies. I'll get a fix out in the next week for it. In the meantime if you put your settings in environment instead of ember-cli-build it will work

Yep it works fine in 2.0.1 if the SemanticUI config is placed environment.js. Is the plan to switch it back to be in ember-cli-build.js in the next release?

Actually I've since noticed that since this change Semantic UI's fonts and images aren't being included in a build. This was the commit that changed the behaviour: eb2a94207df2a5ceb698f87f9604921d22e05d65.
I'm unsure exactly what that commit was addressing, anyone know what "nested addons" are in this context?
Any chance of a 2.0.2 with this issue fixed?

Using an addon inside of an addon.

I'm running into the same issue. It would be great if the documentation would match the current implementation. So currently the settings need to go in environment.js instead of ember-cli-build.js.

please reopen if unresolved


Include stack trace when reporting loading chunk error

Feature request

What is the expected behavior?
Loading chunk error includes stack trace with code that triggered chunk loading.
What is motivation or use case for adding/changing the behavior?
In my production site, with Sentry error logging enabled, "loading chunk" errors are the most frequent ones and given that the errors just say which chunk failed to load (chunk id and URL), it's hard to figure out steps to reproduce or in which situation error has triggered. Seing which code triggred loading of the chunk would help a lot.
How should this be implemented in your opinion?
I've hacked it locally like this: https://github.com/rchl/webpack/commit/fc64322f5da7726efbd546deb7af5ad0d0ff1966
The idea there is that I create Error object when stack hasn't unwinded yet and if loading fails, I use that error object later and pass it to reject promise.
Error object's stacktrace is created on creating the error object so doing it from load/error event of the script element (as is before my change) only includes current frame as stack has unwinded at that point.
I'm not sure that is an acceptable solution because creating an error object is potentially an expensive operation so doing that on loading every chunk could have some overhead. I don't really see any other solution though...
Are you willing to work on this yourself?
no (I'm very inexpirienced with webpack and would really struggle with making tests, for example)

Send a PR


Added showcase for Aware Meditation App Dashboard

The entire site is entirely done using Material UI

@princealikhan Thanks for sharing this great project! Unfortunately, we only accept v1 project on the v1 documentation.


Added juspay/hyperswitch

Adding juspay/hyperswitch - An Open Source Financial Switch to make Payments fast, reliable and affordable

Can you describe me the issue clearer ? @manojradhakrishnan

@MehmetTerc Adding an open source project for which I am a maintainer. The project is actively maintained and had good first issue for new contributors.
Please add it to the collection.

Link to pull request #581


Query Builder Parameter Required for App

Contact App on = True/False,
Parameter required for contact which app service is ON.
and please sort contact on top which app is ON

Need parameter for-
Invitation acceptance Pending from Client side
Invited and installed app.

ContactAppStatus new query parameter


As a developer, I want to integrate the code for candidate viewing and management with GitHub for version control and collaboration with the team.

Integration is in progress.

code integration has been done


/compute transparent pixel, np.nan instead of zero

Looking over the expression parser I'm not sure if requesting nan squares is possible.
v1 * (v1 >= {filter_slider1[0]}) * (v1 <= {filter_slider1[1]})

This works great in general but I'm wondering if it would be possible to show a transparent pixel if the expression evaluates to 0.
in pseudo-code:
f"v1 if (v1 >= {filter_slider1[0]}) * (v1 <= {filter_slider1[1]}) else np.nan"

Nice case, @chapmanjacobd.
Maybe we could make this possible by adding a parameter that lets you specify a NaN / transparent value and mask with that after the expression got evaluated. By default, this could be the raster NaN value.

Now that #222 is merged, both invalid values (nan / inf) and masked values will be transparent. Most of the np.ma.masked_* functions are now allowed in expressions.
So your expression should reduce to a one-liner:
masked_outside(v1, lower, upper)

Could you try this and report back?

works great!!


Random crash: Cannot access memory at address XXX

Hi @mrfaptastic , @vortigont
i have to add back the delay for now on my end, error happens in random location mentioning unable to access memory at address as below:
error: Cannot access memory at address XXX
below are the example errors that was happens without the delay():
example error 1:
======================== THREADS INFO =========================
  Id   Target Id         Frame
* 1    process<PHONE_NUMBER> 0x4000c59a in ?? ()
  2    process<PHONE_NUMBER> 0x400dd3f0 in Adafruit_GFX::drawChar (this=0x3ffc0a94, x=-527, y=19, c=<optimized out>, color=63509, bg=<optimized out>, size_x=1 '\\001', size_y=1 '\\001') at ..libraries/Adafruit_GFX_Library/Adafruit_GFX.cpp:1225
  3    process<PHONE_NUMBER> 0x4000bff0 in ?? ()
  4    process<PHONE_NUMBER> 0x40084b00 in esp_crosscore_int_send_yield (core_id=0) at esp32/crosscore_int.c:112
  5    process<PHONE_NUMBER> 0x40084b00 in esp_crosscore_int_send_yield (core_id=0) at esp32/crosscore_int.c:112
  6    process<PHONE_NUMBER> 0x401c9aea in phy_rfcal_data_check (check_flag=0 '\\000', rf_cal_data=0x1 <error: Cannot access memory at address 0x1>, init_param=0x80089228 <error: Cannot access memory at address 0x80089228>) at phy_chip_v7.c:3305
  7    process<PHONE_NUMBER> 0x401c9aea in phy_rfcal_data_check (check_flag=0 '\\000', rf_cal_data=0x0, init_param=0x80084a4a <error: Cannot access memory at address 0x80084a4a>) at phy_chip_v7.c:3305
  8    process<PHONE_NUMBER> 0x40089956 in xQueueReceive (xQueue=0x3ffdc4cc, pvBuffer=0x3ffdd0dc, xTicksToWait=<optimized out>) athal/esp32/include/hal/cpu_ll.h:39
  9    process<PHONE_NUMBER> 0x401cd688 in cpu_ll_get_core_id () at hal/esp32/include/hal/cpu_ll.h:35
  10   process<PHONE_NUMBER> 0x401cd688 in cpu_ll_get_core_id () athal/esp32/include/hal/cpu_ll.h:35
  11   process<PHONE_NUMBER> 0x401cd688 in cpu_ll_get_core_id () at hal/esp32/include/hal/cpu_ll.h:35
  12   process<PHONE_NUMBER> 0x40084b00 in esp_crosscore_int_send_yield (core_id=0) atesp32/crosscore_int.c:112
  13   process<PHONE_NUMBER> 0x40084b00 in esp_crosscore_int_send_yield (core_id=0) at esp32/crosscore_int.c:112
  14   process<PHONE_NUMBER> 0x40084b00 in esp_crosscore_int_send_yield (core_id=0) at esp32/crosscore_int.c:112
  15   process<PHONE_NUMBER> 0x40084b00 in esp_crosscore_int_send_yield (core_id=0) at esp32/crosscore_int.c:112

example error 2:
==================== THREAD 1 (TCB: 0x3ffc1c44, name: 'main') =====================
#0  0x4000c46c in ?? ()
#1  0x400e2b5e in GifDecoder<192, 64, 12, true>::freeMalloc (this=0x3ffb6dbc <decoder>) at ..libraries/GifDecoder/src/GifDecoder_Impl.h:327

==================== CURRENT THREAD STACK =====================
#0  0x4000c46c in ?? ()
#1  0x400e2b5e in GifDecoder<192, 64, 12, true>::freeMalloc (this=0x3ffb6dbc <decoder>) at libraries/GifDecoder/src/GifDecoder_Impl.h:327
#2  0x400e3070 in loadGIF (name=0x1 <error: Cannot access memory at address 0x1>, duration=120) at ../src/animation.cpp:102

i guess memory allocation on same address are being use by the display, not sure, but this is what i have to report.
edit: also probably due to IRAM_ATTR as i disabled the isr to ram in config to preserve memory.
thanks!
TS

delay() issues is absolutely unrelated to I2S lib code since is does not allocate/release dynamic memory after begin() finished. You should dig through the rest of your/other lib code and check for congestion/use-after-free places.

IRAM_ATTR as i disabled the isr to ram in config to preserve memory

bad idea also. Doing this you must be 100% sure your code never triggers ISR during SPI access routines, otherwise - crash. And I doubt you do since you are working with GIFs. My advise - never play with ram-saving features in the areas you have no confidence, it may lead you to the very dark areas.

noted,
i guess disabling the ISR in ram will impacting performance and not crash. the call will bypass ram and take from the ISR directly.

`
By default, all functions are compiled into flash, which means that the cache may kick in for that code. However, the cache currently can’t be used during hardware interrupts. That means that, if you use a hardware ISR, such as attachInterrupt(gpio, myISR, CHANGE) for a GPIO change, the ISR must have the IRAM_ATTR attribute declared. Not only that, but the entire function tree called from the ISR must also have the IRAM_ATTR declared. Be aware that every function that has this attribute reduces available memory.
In addition, it is not possible to execute delay() or yield() from an ISR, or do blocking operations, or operations that disable the interrupts, e.g.: read a DHT.
Finally, an ISR has very high restrictions on timing for the executed code, meaning that executed code should not take longer than a very few microseconds. It is considered best practice to set a flag within the ISR, and then from within the loop() check and clear that flag, and execute code.
`
from: here
its mentioned ISR time critical, since the lib run at main thread. any delay will cause an issue. i also encountered crash at delay(100) on my other codes as well when the lib is using IRAM_ATTR


Fix missed mock of safe_mkdir.

Fix #117 cc @Yasumoto

Uh, python 3.4 you are very weird.

Thx for picking this up @wickman !

If I roll pytest back to 2.5.2, things work fine.  This seems to be some sort of an odd python 3.4 / mock / pytest interaction.


Enable hotspot buttons scaling

Hello
is there any way to enable hotspot buttons scaling? I want the hotspot buttons to become larger/smaller when I zoom in/out.
Regards,
Rafiq

you probably mean embedded hotspots, see more here https://www.marzipano.net/reference/Hotspot.html

I tried the embedded hotspots. They become larger/smaller when I zoom in/out, but they gets 'stretched' depending how we rotate image.

ok, now I understand. This is beyond standard Marzipano functionality. You have to write a listener for viewChange of the Viewer and then update scale of the hotspot manually as the FOV changes.

how can i get the current FOV?

_viewer._currentScene._view._fov

Thanks


Yocto build fail

Please use the following bug reporting template to help produce actionable and reproducible issues. Please try to ensure that the reproduction is minimal so that the team can go through more bugs!

A brief description
I tried to use WSL to build yocto and it failed right away
Expected results
I optimistically think that Yocto build should work in WSL
Actual results (with terminal output if applicable)
Faile immediately after start
Your Windows build number
14393.351
Steps / All commands required to reproduce the error from a brand new installation
Install all package required for Yocto build.
git clone the yocto project
Source oe-init-build_env to setup the build syste,
bitbake core-image-minimal
Strace of the failing command
derekyu@DEREKYU-PC:~/yocto/poky/build$ bitbake core-image-minimal
Traceback (most recent call last):
File "/home/derekyu/yocto/poky/bitbake/bin/bitbake", line 48, in
cookerdata.CookerConfiguration()))
File "/home/derekyu/yocto/poky/bitbake/lib/bb/main.py", line 468, in bitbake_main
server = start_server(servermodule, configParams, configuration, featureset)
File "/home/derekyu/yocto/poky/bitbake/lib/bb/main.py", line 354, in start_server
cooker = bb.cooker.BBCooker(configuration, features)
File "/home/derekyu/yocto/poky/bitbake/lib/bb/cooker.py", line 195, in init
self.initConfigurationData()
File "/home/derekyu/yocto/poky/bitbake/lib/bb/cooker.py", line 369, in initConfigurationData
self.add_filewatch(self.data.getVar("_base_depends", False), self.configwatcher)
File "/home/derekyu/yocto/poky/bitbake/lib/bb/cooker.py", line 281, in add_filewatch
watcher.add_watch(f, self.watchmask, quiet=False)
File "/home/derekyu/yocto/poky/bitbake/lib/pyinotify.py", line 1924, in add_watch
raise WatchManagerError(err, ret)
pyinotify.WatchManagerError: add_watch: cannot watch /home/derekyu/yocto/poky/build/conf WD=-1, Errno=Invalid argument (EINVAL)
D
Required packages and commands to install
sudo apt-get install gawk wget git-core diffstat unzip texinfo gcc-multilib
build-essential chrpath socat libsdl1.2-dev xterm
See our contributing instructions for assistance.

Hi @derekyu56 -- thanks for posting!  The error that you posted appears to be happening in pyinotify.  It also sounds like you're running the Anniversary Edition of Windows?  That version did not support the inotify kernel functionality.  Recent Windows Insider builds do support inotify; you can either join the Insider program and run pre-release versions of Windows to get that functionality right away, or you can wait until the next Windows release which will contain all of these improvements.

Inotify has been available in the Insider builds for a few months now, and will be available in the Creators Update. Closing this issue.

Ran into this problem just now... Thanks for the solution @JasonLinMS ... I hope my company upgrades to Creators Update soon!!


fix: allow getShorthandAttrValue to parse borders

Fixes #2688
This fixes the column component rendering style="width:NaNInfinity;" by improving getShorthandAttrValue to use the borderParser for border and inner-border properties.

We have a dedicated getShorthandBorderValue for that can you use it for the fix instead ?
Thanks 👍

@iRyusa I would need to modify getShorthandBorderValue to be able to parse a property (inner-border in this case) in addition to a direction https://github.com/mjmlio/mjml/blob/master/packages/mjml-core/src/createComponent.js#L131-L137
Are you ok with a breaking change and modifying the getShorthandBorderValue function to accept two parameters? Or would you want a function that breaks convention, and has the direction before an optional property name getShorthandBorderValue(direction, attribute = 'border')?

Oh. I didn't think about inner-border you're right 😬
As the project isn't typed, may be you can add an optional prefixAttribute as the second option yep to ensure retro compatibility

@iRyusa Updated. Let me know if you need anything

👍


How to disable logging to console?

By default, logger_tt logs to file and console same time.
How to disable logging to console and leave only logging to file?

Just open or create your own log_config.json file.
Then in the root logger's handlers, remove the console handler.

Just open or create your own log_config.json file. Then in the root logger's handlers, remove the console handler.

Is it possible to do it viscount custom config? Just in disable handler in python code?

Yes, you can do it the hard way.
Fork this repo, add the parameter to setup_logging.
This parameter tells logger_tt to remove any handler that outputs a log to the console.
In this case, this is the console handler of the root logger.
But in general, you have to search every logger for every handler and check that its stream is stdout before removing it.
Or the easy way.
The best practice for config loggers is to do it from a config file, not in the code.
logger_tt was designed following that practice so that minimal code is needed to config the logging.
What you are trying to do is equivalent to creating a new logger from scratch with its own handler that logs only to file.
The log_config file already does 99.99% of that job for you.
That means all you need to do is, copy/create your own version of the log_config file with minor changes to fit your need, and nothing changed in the code.


Prevent ephemerals consumers creation

I want to use both CRD and ephemeral consumers at the same time so that I can use this.


Is there a way to highlight URLs ?

Could you be more specific?

@kaushik94 feel free to reopen this issue with some examples.


(FACT-950) Un-vendor leatherman

submodules--
This should be paired with the related updates to our puppet-agent
build script and to Leatherman itself to make everything build
properly in CI. We should also discuss the workflow for
building/installing Leatherman locally for Facter to find.

CLA signed by all contributors.

It looks like the Travis situation can be handled with travis Github releases - it will only push up on a tag, which is perfect for us. I'm not sure what the appveyor step would be.
So the updates to this PR will be to the travis build script to unpack the latest release from leatherman.
Leatherman travis will need to be updated to generate a zip of an installed leatherman PREFIX

I'm closing this out for now until we're ready to actually deal with this properly


Consider adding Mugen to SeaORM showcase!

Hey @koopa1338, feel free to submit a PR to showcase your project!

https://github.com/SeaQL/sea-orm/issues/403

@billy1624 thanks for pointing me to this, created a PR


Transfers integration to the sequencer

@MartinOndejka

@MartinOndejka

Will work on this when I finish the DA layer.


GPL-694 BUG - SS Sample view page not saving data

Users unable to add sample data using the edit function as the new data is not saved.
This is specific to the N/A - Value boxes other fields are ok

raised in RT701951 (ncb)
To Reproduce
http://sequencescape.psd.sanger.ac.uk/samples/1049088
Edit RHS
go to phenotype OR donor id
click on down arrow select value
enter value
Save sample
new data is not displayed in view
(not a caching issue as data not in db)
Expected behaviour
That the data should be saved!
True for:

Windows and Mac
Browsers Chrome and Firefox

Hmm, I think I fixed a similar bug on the studies page. The problem was fields which were flagged as 'required' but were then getting hidden. There was some code in place to handle the issue, but it wasn't firing correctly if you had several fields dependent on each other. Will track down then issue.

Hmm, can't find the issue, or commits. It was quite a while ago.

@JamesGlover
I fixed these a while ago - might be relevant?

https://github.com/sanger/sequencescape/pull/2454
https://github.com/sanger/sequencescape/pull/2456


Unable to create an Instance on us-east-2

Hi guys,
I'm not able to converge an EC2 instance in US-EAST-2.
The instance is created but chef is throws the following error:
undefined method `name' for nil:NilClass on
[2017-03-20T14:26:06+02:00] DEBUG: Re-raising exception: NoMethodError - machine[Ubuntu_Server_14.04_x64_v1.2.4_2017.3.20_EBS_HVM] (my_ubuntu_instance::provisioning_machine line 37) had an error: NoMethodError: undefined method name' for nil:NilClass /opt/chefdk/embedded/lib/ruby/gems/2.3.0/gems/chef-provisioning-aws-2.1.0/lib/chef/provisioning/aws_driver/driver.rb:1031:in default_ami_for_criteria'
/opt/chefdk/embedded/lib/ruby/gems/2.3.0/gems/chef-provisioning-aws-2.1.0/lib/chef/provisioning/aws_driver/driver.rb:1042:in default_ami_for_region'   /opt/chefdk/embedded/lib/ruby/gems/2.3.0/gems/chef-provisioning-aws-2.1.0/lib/chef/provisioning/aws_driver/driver.rb:856:in bootstrap_options_for'
Here you can find the logfile: http://pastebin.com/k2NLJvfC

Check your cheffish version, we were also seeing this issue, updating to cheffish 4.1.1 resolved the issue with chef-provisioning-aws 2.1.0.

I had the same issue before, please check your driver configuration in knife.rb. The driver option should look like
driver 'aws:<profile name>'
driver_options :compute_options => {
                                     :aws_access_key_id => '<key_id>',
                                     :aws_secret_access_key => '<key>',
                                     :region => 'us-east-2'
                                    }

Your recipe dependency declaration. should look like
require 'chef/provisioning'
require 'chef/provisioning/aws_driver'


Fix K8S Default Env Bug

#1424  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Merge activity

Nov 14, 4:05 PM EST: A user started a stack merge that includes this pull request via Graphite.


Restart grant wait for synchronous branch changes

In case the signals instr_gnt_i and branch_i are asserted at the
same time, the branch request invalidates the current grant. Wait for
the next grant in order to be in sync with the data output of the fifo.

ping, any news about this? This was a quite painful bug for us and others will most probably face it too..

hello stefan, now I see what you were referring to at orconf, I'll synchronize with the team and check the pull request. It would be also good to open an issue in case of bugs like this.
Cheers,
Davide

Hi sorry, I missed this pull request somehow.
Can you describe me with an example (if possible) what problem you see? I would like to reproduce it before merging it to the master :)
Thanks a lot for your precious contribution,
Best
Davide

Thanks, @davideschiavone, unfortunately we cannot share the original code as we were migrating from another core and it was already pretty large. We can try to re-create a test case that re-produces the error, as @towoe wrote it came down to the core using the wrong instruction on a very basic branch. @towoe, can you describe a bit what led to the situation, thanks.

Hi, it is also not easy to reproduce this error, but I still have the waveform where one can see how the PC is set to an incorrect value.

And the objdump of the section:

After the PC is at 0x370 (lw), it should increase to 0x374, but the instruction from address 0x2f4 is fetched. If one decodes the data from address 0x370, 0xfff78793, as it would be a branch instruction, an offset of -124 would be the change of the PC, resulting in 0x2f4. So I would assume the branch instruction from 0x34c is still in the decoder, but the calculation of the PC is done with the data from 0x370.
This was the only time I observed the signals instr_gnt_i and branch_i to be active at the same time.
I hope this helps to understand what is happening.

We observe this behavior with both the zeroriscy and the ri5cy core and therefore opened an issue here: https://github.com/pulp-platform/riscv/issues/50
Best regards
Felix

Thanks a lot, I am trying to reproduce it

This can be closed in favor of changing core2axi as discussed in https://github.com/pulp-platform/riscv/issues/50

It is also in a PR for better documentation now: https://github.com/pulp-platform/zero-riscy/pull/12/commits/52ffb372a30ed3ba63625475bb8888171433e37b
See here: https://zero-riscy.readthedocs.io/en/latest/instruction_fetch.html#protocol


⚠ Bofrost.be has degraded performance

In aeced91, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2532 ms

Resolved: Bofrost.be performance has improved in 7b2f75d.


Stop supporting IE 8 and 9

Those versions of IE are not used enough for the amount of work needed to support them

:+1:
This would also enable removing some old vendor prefixes, e.g. [for flex](https://github.com/Daemonite/material/blob/4c3de9ade6f5e6c67b1bf4d8f0e0d073fbee8387/css/base.css#L5226.

:+1:  I upvote for it too, because of the future of Windows 10 who's trying to establish a new use of Microsoft Edge.

See also http://github.hubspot.com/tether/overview/why_we_dont_support_ie_8/

+1

support for old browsers has been removed in this commit: https://github.com/Daemonite/material/commit/94917803afdcab902789ff3dfacda469aca4e800


Parallelize isotropize

I implemented @shoyer 's amazing numpy_groupies hack to parallelize the isotropization of spectra. This addresses issues #9 and #87 .

This is cool Takaya! We could also potentially use this in xhistogram
I will review this after the Thanksgiving holiday.
p.s. @shoyer is not the creator of numpy_groupies.

p.s. Isn't this going into xarray itself? Is it necessary to put it in xrft?

p.s. Isn't this going into xarray itself? Is it necessary to put it in xrft?

I think it eventually is going to but I checked the xarray issue log and it seems that there's still some issues for it to be implemented upstream. So I figured I'd just add it to xrft in the mean time.

@rabernat Do you think we should keep this PR out until numpy_groupies gets implemented more upstream?

This looks great! Do we have tests for this feature already? If so, I approve.

The test_isotropize tests isotropization for both chunked and non-chunked data so I think the tests cover this feature.


Admiral BT support

I would say this issue is another profile request but a distinct issue.
The Hyperkin Admiral Premium BT Controller is a smaller style n64 controller available on amazon and in some brick and mortar stores.  When I couldn't find either retrobit or big N's first party 64 controller i bought this one.   It has some wacky features (a dongle that does mempack dumps to SD) and works with 64/Switch/PC.
The controller appears to be pairing

light goes solid
You cannot connect to BR webgui
Controllers turned on after get "2nd player colour"  IE PS4 pairs as 2nd controller and the bar lights red.

Alas, No buttons appear to function in any BR system I've tried it with.
It's recognized in windows/android/switch without the dongle so it is standard BT.
If there's a way to get a debug log let me know.
Windows reads as Analog:Analog, Dpad:POV, Start:10, b:1,a:2,l:5,r:6 z:15, C-Left:9, C-up:13, C-Right:12, C-Down:14
there are several overlaps with for example Dualshock4 button assignments. But no function for this contrroller.
Thanks,
Darien

Added to my amazon list:
https://www.amazon.ca/hz/wishlist/ls/2RMFK6DHX29NQ?ref_=wl_share
If someone buy me one I can investigate to add support.

Support added in v1.8.2
https://github.com/darthcloud/BlueRetro/releases/tag/v1.8.2


New Actor API

This is a complete rewrite of most of the actors (especially SSH), transfer mechanism, and the Installer/Actor internal API...  Passing the installers to the actors seems so logical to me... you could say separation of concern, blah... but I think it's the actors concern to install (and verify) things, so it makes sense to me to just pass them the installer or Verifyier and say "install [Apt]", "install [Transfer]", etc... and it's the only way (or the easy way?) to get to my next big feature of tuning the installers on a host-by-host basis to allow commands (and templates) to setup each host differently based on variables defined in the roles.
The actors know about the roles and hosts... so they need to be able to regenerate the installers (maybe later packages) dynamically per host (subbing variables, potentially changing logic, etc)... that's the big feature I'm missing that I think will make me mostly happy.
I'm submitting this for review and testing (not merging!)... I need to test it a bit more myself (likely beef up the tests) and I haven't done any testing of the Vlad code... it's just what looks correct based on what was there before.
Would love any thoughts and feedback.  Anyone who can test that'd be great.

That is a failure... it should stop.  Any deploy should stop if it hits a failure... add user should be written so that user existence isn't an issue.. though really your recipe should possibly create users in a separate step that can be validated.

But this breaks the current behaviour. And the level of granurality
would very extreme. I mean:
creating a directory that is already there (failure -> own package)
creating a link that is already there (failure -> own package)
....
I like the freedom of adding a runner to a package and a verifier so the
verifier fails and the whole package is installed again. Some things
fail (as things are already there) but this doesn't break the package if
the final verification passes.
Am 13.12.2011 23:08, schrieb Josh Goebel:

That is a failure... it should stop.  Any deploy should stop if it hits a failure... add user should be written so that user existence isn't an issue.. though really your recipe should possibly create users in a separate step that can be validated.

Reply to this email directly or view it on GitHub:
https://github.com/crafterm/sprinkle/pull/41#issuecomment-3131281

The current behavior with SSH was terrible (running sequential commands concurrently)... are you saying the current behavior (in master) with ALL other actors is as you describe?  Or are you only speaking from your past experience with SSH?
Most commands have switches to avoid throwing an error.  An error is an error.  mkdir -p for example will silently ignore a directory if it already exists.  Most commands do not throw errors for no reason.  I don't believe errors should be ignored... recipes should be written to send proper commands.  Since you can't tell what the type of error is unless you plan for the most boring of errors... a "directory already exists" error is entirely different than a "disk full" error.

To create a link you would just use ln -f to force... which would ignore existing (or replace).

I just checked master for Capistrano and it does abort on any error and I would say that is the correct behavior.  Errors on only ignored during verification (so they can be re-raised as verification errors).

The only logical behavior for deployment is to bail on an error and let the user examine the error and decide how to proceed.  It's probably possible to hard code around this... runner "mkdir blah || true"... so even if mkdir failed true would still insure the result of the command was non-error.

But again the best way is to use commands that don't throw errors and to write your packages in such a way that they can be reapplied without throwing errors.

Rebases on latest master.

I think @yyyc514 has a good point about failing out on exit codes other than 0.  But, of course, leniency of error handling is largely a matter of taste.  This would be a breaking change in that you would have to change your code to "mkdir -p" and "ln -s -f", but it seems like the correct behavior to my taste.
If it is not to your taste, @crafterm, I still think this branch has some very worthy changes to bring sprinkle to the next level.  Just change the error handling.


Add 'keto get config' functionality

Added 'keto get config' command, which generates the kube admin config for a given cluster.
Example use:
keto get config --cloud aws --cluster testcluster01 --assets-dir /home/kash/ketoassets/ --output-file kube.cfg

Optional flags:
--assets-dir: Defaults to current working directory
--output-file: Defaults to stdout
It checks for ca.crt & ca.key (specifically required by kubeadm), and will create symlinks from your kube_ca.crt & kube_ca.key if they exist to save you the hassle.
Will check for presence of the kubeadm binary and error with a curl command to retrieve it (the url is built, but if the binary doesn't exist for your distro then the link won't work).

Interesting, I'll have a look when I get a minute.


Nuke: Move nuke implementation to OpenPype

Changes
Removed nuke implementation.
Related to https://github.com/pypeclub/OpenPype/pull/2514

Task linked: OP-2358 Move Nuke to OpenPype


mwc-select options cannot be scrolled into view if inside overflow:hidden

Describe the bug
mwc-select options are curtailed by overflow:hidden, unlike native <select> options, with no apparent recourse to select them.
To Reproduce
Steps to reproduce the behavior:

Have markup like:

will cover your options

  <mwc-select label="thingies">

  </mwc-select>

Open the menu.

Expected behavior
It allows options to be displayed anyway, overriding overflow: hidden if necessary. This is what native <select> does.
Actual behavior
The options extend upward (because it's more than halfway down the screen), are covered by the orange div, and cannot be scrolled into view. Safari seems to think the options are in view because they would be if not for the overflow:hidden, and so it won't let you scroll any farther, making it impossible to select the items.
Browser Version:

OS: iOS
Browser: Safari
Version: 14.2

Thanks for the bug!
Our recommendation for MDC (https://github.com/material-components/material-components-web/issues/5590) is to set select's menu to fixed and provide an explicit width for both the anchor and menu. We can update <mwc-select> to expose this capability for MWC as well.

Thanks for the bug!
Our recommendation for MDC (https://github.com/material-components/material-components-web/issues/5590) is to set select's menu to fixed and provide an explicit width for both the anchor and menu. We can update <mwc-select> to expose this capability for MWC as well.

Could you please explain what I should add to mwc-select to get this working?
This is quite vague and I have no idea what to do here, thanks!

Have you tried using the fixedMenuPosition attribute documented in the select's readme?

    <mwc-select fixedMenuPosition>
        ...
    </mwc-select>

Demo: https://jsbin.com/pavileceya/edit?html,output
Keep in mind two things:

We recommend setting a fixed width for both the select and the menu (using width, --mdc-menu-min-width, and --mdc-menu-max-width on the <mwc-select> element)
We don't recommend scrolling while the menu is open, since the menu will remain fixed but the text field element will not. This property is intended for overflow elements such as dialog, which do not scroll the entire page. For the purpose of the demo, I've included page scrolling so you can see what this looks like.

Thanks for your quick response.
Your demo works but unfortunately it still doesn't work for me.
I use the components as npm dependencies and have updated all to 0.21.0-canary.fe73953b.0 but unfortunately that still doesn't work.
Could it have to do with the version I use?


JWL merge AUTO-SYNC

Is your feature request related to a problem? Please describe.
The problem is, that there is still NO AUTO-SYNC between all the android and windows devices
Describe the solution you'd like
php server script and maybe client app apk to auto-sync between all devices every 30 mins (or maybe fewer)
Describe alternatives you've considered
Theres no alternative at the web found by me

Thanks for the suggestion, but this will never be a feature of JwlMerge.


alert_generator: add vmalert config

Add testing configuration for VictoriaMetrics vmalert
component whcih can be used as alerts generator.
vmalert can be configured to use Prometheus as remote
storage for querying alerts and writing back results.

Thanks! Can you please fix the DCO?

Thanks! Didn't notice the warn msg at first.


Resharper support

Hi all,
sorry I add this nice to have feature here, but I didn't see it requested anywhere. Do you guys plan to make the two tools play nice with one another? Are any plans whatsoever in this direction? I follow the project from some time, and I plan to use it as soon it hits the v1 level. I do use however Resharper as my primary dev tool and my only concern is to make both of them compatible. The current Stylecop plugin for Resharper works pretty well, even though in beta, with minor issues that I can ignore. I would be grateful if you could share your plans in this direction.
Btw, your work is awesome, congrats!

I don't use ReSharper myself, but I have a coworker that uses it daily and hasn't had problems so far using it with StyleCop Analyzers.
Note that ReSharper by default disables the preview which appears in the light bulb menu in Visual Studio 2015, which means it also disables the ability to fix all instances of a violation in a document, project, or solution. For the best experience, you may need to disable ReSharper's "merged light bulb" feature (I think this is what it was called).

What I meant was that Resharper and Stylecop have in some cases different standards, for example like using or not using the _ (underscore) in private fields' names. I don't know exactly which component does it, but either resharper or stylecop addin warns that there are differences in these standards/settings and makes resharper use the stylecop standards. If this will not be automated in the first phase, could I at least see which are these differences, and maybe manually adjust them?

There are currently no plans to do this. StyleCop Analyzers and its code fixes operate independently of the settings a user has chosen within the IDE. If a Roslyn API to configure formatting settings is available in the future, it would probably make sense for this project to integrate with it, but manipulation of settings using IDE- or extension-specific APIs is likely beyond the scope of this project.

Thanks for your reply!

Note to any future visitors to this page: the closest way I've found so far to make the Resharper and StyleCop Analyzers to play nice together is to have Resharper configured with the old non-Roslyn based stylecop tool, so that you have a a "StyleCop" profile when using code cleanup in Resharper.
After that, you can use "Cleanup Code" w/ the "StyleCop" profile in Resharper which generally auto-fixes a lot of the issues that StyleCop (the old one) would report. However, at this point, this is likely to introduce a lot of white spaces at the end of certain lines (e.g.: after every line in a multi-line method call with named parameters) that trigger warnings with some of the new rules in StyleCop analyzers and other minor issues.
Immediately after that, a Visual Studio "Format Document" is enough to fix the remaining issues (as I believe it now uses the Roslyn code analyzers in some way for the code fixes).
(Ctrl + E,C   ;  Ctrl E, D).
If anyone has found a cleaner way, please let me know :).


Add domain for symmetric cipher registration

@bfussell Hmm looks like maybe the server doesn't support this right now. I'll keep around for now but hold off on merge I guess

dear all
donat


Add CHANGELOG-v0.1.0-alpha.md

What type of PR is this?
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Release note:

@andyzhangx Please review.