make guetzli failed

When I run make,it failed and show some error message.
==== Building guetzli (release) ====
butteraugli_comparator.cc
In file included from ./guetzli/butteraugli_comparator.h:22:0,
                 from guetzli/butteraugli_comparator.cc:17:
third_party/butteraugli/butteraugli/butteraugli.h:129:76: warning: ‘assume_aligned’ attribute directive ignored [-Wattributes]
 #define BUTTERAUGLI_CACHE_ALIGNED_RETURN __attribute__((assume_aligned(64)))
                                                                            ^
third_party/butteraugli/butteraugli/butteraugli.h:146:45: note: in expansion of macro ‘BUTTERAUGLI_CACHE_ALIGNED_RETURN’
   static void *Allocate(const size_t bytes) BUTTERAUGLI_CACHE_ALIGNED_RETURN;
                                             ^
third_party/butteraugli/butteraugli/butteraugli.h:129:76: warning: ‘assume_aligned’ attribute directive ignored [-Wattributes]
 #define BUTTERAUGLI_CACHE_ALIGNED_RETURN __attribute__((assume_aligned(64)))
                                                                            ^
third_party/butteraugli/butteraugli/butteraugli.h:257:42: note: in expansion of macro ‘BUTTERAUGLI_CACHE_ALIGNED_RETURN’
   ConstRestrict<T *> Row(const size_t y) BUTTERAUGLI_CACHE_ALIGNED_RETURN {
                                          ^
third_party/butteraugli/butteraugli/butteraugli.h:129:76: warning: ‘assume_aligned’ attribute directive ignored [-Wattributes]
 #define BUTTERAUGLI_CACHE_ALIGNED_RETURN __attribute__((assume_aligned(64)))
                                                                            ^
third_party/butteraugli/butteraugli/butteraugli.h:268:7: note: in expansion of macro ‘BUTTERAUGLI_CACHE_ALIGNED_RETURN’
       BUTTERAUGLI_CACHE_ALIGNED_RETURN {
       ^
dct_double.cc
debug_print.cc
entropy_encode.cc
fdct.cc
gamma_correct.cc
guetzli.cc
idct.cc
jpeg_data.cc
jpeg_data_decoder.cc
jpeg_data_encoder.cc
jpeg_data_reader.cc
jpeg_data_writer.cc
jpeg_huffman_decode.cc
output_image.cc
preprocess_downsample.cc
processor.cc
In file included from ./guetzli/butteraugli_comparator.h:22:0,
                 from guetzli/processor.cc:24:
third_party/butteraugli/butteraugli/butteraugli.h:129:76: warning: ‘assume_aligned’ attribute directive ignored [-Wattributes]
 #define BUTTERAUGLI_CACHE_ALIGNED_RETURN __attribute__((assume_aligned(64)))
                                                                            ^
third_party/butteraugli/butteraugli/butteraugli.h:146:45: note: in expansion of macro ‘BUTTERAUGLI_CACHE_ALIGNED_RETURN’
   static void *Allocate(const size_t bytes) BUTTERAUGLI_CACHE_ALIGNED_RETURN;
                                             ^
third_party/butteraugli/butteraugli/butteraugli.h:129:76: warning: ‘assume_aligned’ attribute directive ignored [-Wattributes]
 #define BUTTERAUGLI_CACHE_ALIGNED_RETURN __attribute__((assume_aligned(64)))
                                                                            ^
third_party/butteraugli/butteraugli/butteraugli.h:257:42: note: in expansion of macro ‘BUTTERAUGLI_CACHE_ALIGNED_RETURN’
   ConstRestrict<T *> Row(const size_t y) BUTTERAUGLI_CACHE_ALIGNED_RETURN {
                                          ^
third_party/butteraugli/butteraugli/butteraugli.h:129:76: warning: ‘assume_aligned’ attribute directive ignored [-Wattributes]
 #define BUTTERAUGLI_CACHE_ALIGNED_RETURN __attribute__((assume_aligned(64)))
                                                                            ^
third_party/butteraugli/butteraugli/butteraugli.h:268:7: note: in expansion of macro ‘BUTTERAUGLI_CACHE_ALIGNED_RETURN’
       BUTTERAUGLI_CACHE_ALIGNED_RETURN {
       ^
quality.cc
quantize.cc
score.cc
butteraugli.cc
In file included from third_party/butteraugli/butteraugli/butteraugli.cc:32:0:
third_party/butteraugli/butteraugli/butteraugli.h:129:76: warning: ‘assume_aligned’ attribute directive ignored [-Wattributes]
 #define BUTTERAUGLI_CACHE_ALIGNED_RETURN __attribute__((assume_aligned(64)))
                                                                            ^
third_party/butteraugli/butteraugli/butteraugli.h:146:45: note: in expansion of macro ‘BUTTERAUGLI_CACHE_ALIGNED_RETURN’
   static void *Allocate(const size_t bytes) BUTTERAUGLI_CACHE_ALIGNED_RETURN;
                                             ^
third_party/butteraugli/butteraugli/butteraugli.h:129:76: warning: ‘assume_aligned’ attribute directive ignored [-Wattributes]
 #define BUTTERAUGLI_CACHE_ALIGNED_RETURN __attribute__((assume_aligned(64)))
                                                                            ^
third_party/butteraugli/butteraugli/butteraugli.h:257:42: note: in expansion of macro ‘BUTTERAUGLI_CACHE_ALIGNED_RETURN’
   ConstRestrict<T *> Row(const size_t y) BUTTERAUGLI_CACHE_ALIGNED_RETURN {
                                          ^
third_party/butteraugli/butteraugli/butteraugli.h:129:76: warning: ‘assume_aligned’ attribute directive ignored [-Wattributes]
 #define BUTTERAUGLI_CACHE_ALIGNED_RETURN __attribute__((assume_aligned(64)))
                                                                            ^
third_party/butteraugli/butteraugli/butteraugli.h:268:7: note: in expansion of macro ‘BUTTERAUGLI_CACHE_ALIGNED_RETURN’
       BUTTERAUGLI_CACHE_ALIGNED_RETURN {
       ^
Linking guetzli
obj/Release/guetzli.o: In function `_ZN12_GLOBAL__N_17ReadPNGEP8_IO_FILEPiS2_PSt6vectorIhSaIhEE.constprop.61':
/opt/guetzli-master/guetzli/guetzli.cc:46: undefined reference to `png_create_read_struct'
/opt/guetzli-master/guetzli/guetzli.cc:51: undefined reference to `png_create_info_struct'
/opt/guetzli-master/guetzli/guetzli.cc:57: undefined reference to `png_set_longjmp_fn'
/opt/guetzli-master/guetzli/guetzli.cc:64: undefined reference to `png_init_io'
/opt/guetzli-master/guetzli/guetzli.cc:74: undefined reference to `png_read_png'
/opt/guetzli-master/guetzli/guetzli.cc:76: undefined reference to `png_get_rows'
/opt/guetzli-master/guetzli/guetzli.cc:78: undefined reference to `png_get_image_width'
/opt/guetzli-master/guetzli/guetzli.cc:79: undefined reference to `png_get_image_height'
/opt/guetzli-master/guetzli/guetzli.cc:82: undefined reference to `png_get_channels'
/opt/guetzli-master/guetzli/guetzli.cc:139: undefined reference to `png_destroy_read_struct'
/opt/guetzli-master/guetzli/guetzli.cc:136: undefined reference to `png_destroy_read_struct'
/opt/guetzli-master/guetzli/guetzli.cc:53: undefined reference to `png_destroy_read_struct'
collect2: error: ld returned 1 exit status
make[1]: *** [bin/Release/guetzli] Error 1
make: *** [guetzli] Error 2

who can help me to see what is the reason?Thank you very much.

Thanks for the report.
Please tell me:

what OS are you running,
what does pkg-config --libs libpng output,
(if it's easy for you to find) what version of libpng you have installed.

what OS are you running,
CentOS release 6.8 (Final)

what does pkg-config --libs libpng output,
*-L/opt/libpng/lib -lz *

(if it's easy for you to find) what version of libpng you have installed.
my libpng version is 1.6.28

Maybe I should tell you my gcc version
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/local/libexec/gcc/x86_64-unknown-linux-gnu/4.8.2/lto-wrapper
Target: x86_64-unknown-linux-gnu
Configured with: ../configure
Thread model: posix
gcc version 4.8.2 (GCC)

I found the libpng.pc file that I wrote maybe wrong.
Then I copy the libpng16.pc in /opt/libpng/lib/pkgconfig to my PKG_CONFIG_PATH,  run make again and success.
But when I run guetzli, there was a new problem.
guetzli: error while loading shared libraries: libpng16.so.16: cannot open shared object file: No such file or directory


⚠ Bofrost.de has degraded performance

In a07d9c4, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1331 ms

Resolved: Bofrost.de performance has improved in fdd705e.


convert secman desktop from electron to pwa

I think converting from electron to pwa is a good thing for secman

done, and it's looks great and awesome


Sign up and VOMS

Summary
Try to get extract common stuff

Sign up section
Move VOMS info to HTC getting started

Related issue :

The Getting Started section of HTC seems redundant with the changes so I completely removed it.
The workload manager docs should include the documentation that is missing from there

I'm missing some stuff, fixing it

we should use the same way on the documentation to refer to X509 certificates, cause sometimes we use X509 others X.509 or x509.

we should use the same way on the documentation to refer to X509 certificates, cause sometimes we use X509 others X.509 or x509.

I have moved everything to X.509

Further insights must be entered here to explain the use of VO access, the relationship between a VO and a COU group, the role of Perun between VOMS and Check-in, finally missing the link to proc24 (how to register a VO) in the paragraph of the VO / VOMS.

where should this be added?
PROC24 is linked in the VOMS page.
Relationship between COU group and VO and Perun/VOMS is relevant for Check-in documentation, but probably out of scope for a Getting Started page

I also brought the whole https://wiki.egi.eu/wiki/USG_Getting_Certificate#Requesting_the_Certificate to the VOMS page

thanks!

@andrea-manzi I have detected some duplication in your bits... you will need to re-approve soon ;)

I'm a bit struggling being sure who GitHub is willing a review from...
@vardizzo-lab it seems you may be the one with the power to validate this, can you have a look?

I'm a bit struggling being sure who GitHub is willing a review from...
@vardizzo-lab it seems you may be the one with the power to validate this, can you have a look?

GitHub says: Waiting on code owner review from @vardizzo-lab and/or @CatalinCondurache.

GitHub says: Waiting on code owner review from @vardizzo-lab and/or @CatalinCondurache.

Where do you see this? from GitHub CLI?

GitHub says: Waiting on code owner review from @vardizzo-lab and/or @CatalinCondurache.

Where do you see this? from GitHub CLI?

So it wasn't my eyes but a GitHub bug, it's clear now :)

Reviewed and approved.

Thanks, but please don't close it so that we can merge it ;)

@CatalinCondurache it seems we also need your approval, probably as it's touching parts of the doc that are under your responsibility, can you please have a look?


fix: @genql/runtime is required

The @genql/runtime package is a runtime requirement, however the instructions in the current README does not install it when --no-dev is specified.

I was in the process of publishing v3 but i wanted to get more comments in the RFC, that's why the README is not correct now


Make Default num_traces 0, update .traces upon calling generate_traces() again

Had a merge issue with the fd sampling, but after that fix, rest looks good!


Purge cloudflare cache when deploying a new version

Purging only the sw.js file should be enough since that's what is checked by the browser for updates.
Purging cache for / and /internalDiscovery are not necessary since html doesn't get cached by cloudflare anyway.
Depends on #866

This was fixed in https://github.com/rendajs/domain-server/commit/fb52ee6e64c61863bc9a2983dc37cdcb306b5570


Add the option to include an extension in a phone number

We have a need to include phone extensions when adding a number to
contact details but the current field does not allow for the extra
characters. To avoid differences in formatting, we've added a new field
for the extension digits that are then automatically formatted on the
front end.
Additions

New extension option to the wagtail contact phone molecule
New ext formatting on the front-end of the contact phone molecule

Testing

tox -e fast v1.tests.atomic_elements.organisms.test_organisms
Build a sub-landing page and include an expandable.
Add contact phone molecule and include a phone number and extension.
Verify the extension is formatted on the page.

Screenshots

Checklist

[x] PR has an informative and human-readable title
[x] Changes are limited to a single goal (no scope creep)
[x] Code can be automatically merged (no conflicts)
[x] Code follows the standards laid out in the development playbook
[x] Passes all existing automated tests
[x] Any change in functionality is tested
[x] New functions are documented (with a description, list of inputs, and expected output)
[x] Placeholder code is flagged
[x] Visually tested in supported browsers and devices
[x] Project documentation has been updated
[x] Reviewers requested with the Reviewer tool :arrow_right:

@jimmynotjim this looks great, thanks! One thing I'm curious about--will the extension show up next to all versions of the phone number (vanity, tty) when entered? Or just once with the main phone?

will the extension show up next to all versions of the phone number (vanity, tty) when entered

@sarahsimpson09 Oh, good point, I didn't think to add it to those. Is it safe to assume the TTY extension might not match the primary extension and would need it's own field?

Is it safe to assume the TTY extension might not match the primary extension and would need it's own field?

@jimmynotjim yeah, there may likely be times when the only difference between the main line and TTY is the extension, so it would help for TTY to have its own extension field.
As for the vanity number, since it sits on the same line right next to the main number, I don't think the extension needs to be repeated twice, but open to being wrong if it looks strange without it!

@sarahsimpson09 Added the TTY Extension option to the admin

Not sure this is what is supposed to be on the front-end though

Do we have any place we've used TTY on the site?

Pushed each number to it's own line and added the . to Ext text.

Also added helper text to the vanity input

@sarahsimpson09 @anselmbradford want to sign off and I'll merge it?

@jimmynotjim this looks good to me--thanks!

Pushed a commit to update the migration. Will merge if Travis passes.


crash when using selection for find

Using cmd+E on xi-mac I get a crash when the selection is a caret.
There's a related issue where the search bar isn't correctly populated when the selection isn't a caret.
@scholtzan maybe this is one for you?

I ignored CMD + E for now, but it is the next thing on my list.

This is fixed in https://github.com/google/xi-editor/pull/701 and can be closed.


Additional ideas:

Good evening (relative to my country's time :) ). There's something I don't like. Often, and especially now, when I use your program to dub Russian videos into Uzbek, the voice doesn't sync properly with the video. That is, sometimes the voices are slow, and as a result, the subsequent voices surpass the actual lip movements. If possible, try to fix this.
How do I think this can be resolved (my opinion):
You could identify the timing of the voices according to the subtitles and then adjust the speech rate of the TTS-generated voice. For instance, if the TTS-generated voice's duration is shorter than the time the person in the video is speaking, the speech rate should be reduced; if it's longer, it should be increased.
Adjusting the speech rate does help, but if the videos are long, it makes it harder for people to determine the exact speech rate.

Thank you sincerely for your suggestion. This problem is something we have been contemplating for a long time as well.
The crux of the issue is that even with the same expression, the length varies by language. Generally, translations into Chinese and Korean tend to be shorter, while English and Spanish are longer. In particular, Russian seems to be very long.
Therefore, when dubbing Korean into Russian, we might have to read at double speed to match the time sync. As you pointed out, this is very unnatural and difficult to understand. We have tried to detect sentence lengths and automatically adjust the speaking speed, but the results were not good.
The best method we propose for translation and dubbing is to use LLMs. In other words, we should entrust the translation to models like ChatGPT. We can simply add a request in the prompt to keep the lengths of the original and translated texts as similar as possible. However, running an LLM on a local PC is still challenging.
The realistic approach we can support is to provide the translations in an editable format.
Starting from v1.6.7, both Whisper subtitles and translations can be uploaded and edited externally.
Additionally, we have separated the translation and dubbing processes for this purpose. Now you can modify the translated sentences before starting the dubbing.

One more thing~~
We have also added dubbing functionality using F5-TTS in the Studio tab, in addition to Edge-TTS. Try dubbing with Celeb Voice. However, please note that F5-TTS only supports English and Chinese.

Hello, I thought it wasn't that good, but I found a solution that needs improvement using the Chilab program. Of course, I worked with ChatGPT because I am a beginner programmer :). In short, after chatting with it, I realized that there is a small model, and it can remove filler words and make the text more concise. You should try this: Link . After removing the filler words, I think speeding up the voice will be more effective.

Hello, I thought it's not that good, but I found a solution that should be improved by using developers. Of course, I worked with ChatGPT because I am a beginner programmer :) In short, after chatting with it, I realized that there's a small model, and it can remove filler words and compact the text. You should try it: https://chatgpt.com/share/67435db3-0bcc-800c-bfe6-0fe2b8f6901c
After removing the filler words, I think speeding up the voice will be more effective.


Modifying errors returned from middleware

Hi there,
I'm using this library in an API I'm building and was wondering if there is a way of changing the default error object that gets returned from routes protected by the oauth middleware:
{
  "error": "invalid_request",
  "error_description": "The request is missing a required parameter, includes an invalid parameter value, includes a parameter more than once, or is otherwise malformed. Check the \"access token\" parameter."
}

I'm trying to keep all responses from the api to a consistent format, similar to how the Instagram API behaves. For example:
{
	"meta": {
		"error_type":"OAuthParameterException",
		"code":400,
		"error_message":"Missing client_id or access_token URL parameter."
	}
}

I suggest you looking at @GrahamCampbell's exceptions package. There you can add an transformer for the oauth exceptions.


Type of device.

Hello, sorry, I would like to know if the library provides me with the option to configure the type of device shown in the Alexa App, such as Switch, plug, etc., and if it is possible to add more than 1 esp82 66 connected on the same network. Thank you very much in advance.

Roger, as far as i know the Espalexa only emulate Hue Lights (OnOff, Dimmable, Color, etc.).
To emulate a plug, you can use the Wemo Emulator (you can find here in github).
About the number of devices (ESP8266) connected on the same network... yes, you can have more than 1 connected. I'm doing it with ESP32 and so far I haven't had any problems.

Roger, que yo sepa, el Espalexa solo emula Hue Lights (OnOff, Dimmable, Color, etc.).
Para emular un complemento, puede usar el emulador de Wemo (puede encontrarlo aquí en github).
Acerca de la cantidad de dispositivos (ESP8266) conectados en la misma red ... sí, puede tener más de 1 conectado. Lo estoy haciendo con ESP32 y hasta ahora no he tenido ningún problema.

Thank you, you have resolved my question.


adding kuwarkapur

Hey @kuwarkapur there are some conflicts. You may try

Updating the fork
As it is on your main. it will automatically reflect here. Let us know if you need any help

Only updating the fork is missing / resolving the merge conflict

i forked the new repo.


Cannot Delete Triggers retrieving more records than necessary

There exist triggers on Account, Contact, Program Plan, and Plan Requirement that block deletion of records if one or more records are present on key child objects.
Currently, the child queries are not bounded.  This means the more children that are present, the more they will retrieve.  This could impact a bulk deletion operation and/or have minor performance impact.
Recommend adding limit 1 to all subqueries in these triggers (see below for list of triggers) so that it never retrieves more than one child record per subquery, as that is enough to confirm the deletion cannot be allowed to proceed.
Trigger List:

ACCT_CannotDelete_TDTM
CON_CannotDelete_TDTM
PPlan_CannotDelete_TDTM
PReq_CannotDelete_TDTM

**lurch: add

Tracking W-027957

Included in beta release 1.64 (Beta 2)

Included in production release 1.64


add mineires dialects

add mineires dialects (most important expression "Logali")

Hi, @YagoAzevedo, first, thanks for contributing!!
Could you adjust the tests and update the readme?

Hi, @YagoAzevedo I'm not answering for a long time, if you answer, I'll open the PR again.


[MNT] convert black extend-exclude parameter to single string

In https://github.com/sktime/skbase/pull/205, pre-commit CI seems to complain that the extend-exclude parameter is not followed by a string (but a list of strings).
Attempting to fix this by changing the list of strings into a single string tha should hopefully comply with the expected syntax.

Codecov Report

Merging #207 (a65e80e) into main (a33be08) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main     #207   +/-   ##
=======================================
  Coverage   82.41%   82.41%
=======================================
  Files          43       43
  Lines        2832     2832
=======================================
  Hits         2334     2334
  Misses        498      498


Fix multicall balances issue

An invalid response was not parsed when a revert calling balanceOf
happened

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 90.252%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

gnosis/eth/ethereum_client.py
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
2398

Relevant Lines:
2657

💛  - Coveralls


vllm deploy the same query always same response. I cannot find where to config random seed

是否已有关于该错误的issue或讨论？ | Is there an existing issue / discussion for this?

[X] 我已经搜索过已有的issues和讨论 | I have searched the existing issues / discussions

该问题是否在FAQ中有解答？ | Is there an existing answer for this in FAQ?

[X] 我已经搜索过FAQ | I have searched FAQ

当前行为 | Current Behavior
vllm 0.2.2 with pytorch 2.1, cuda 12.1, qwen/Qwen-7B-Chat
I successfully run the from
from examples.vllm_wrapper import vLLMWrapper
the same query always same response.
I cannot find where to config random seed
I also fastly check vllm, there is no mention this random seed issue.
Do you encounter similar issue?
thanks!
My query(prompt) are
response, history = model.chat(query="你好",
history=None)
print('1,', response, flush=True)
response, history = model.chat(query="给我讲一个年轻人奋斗创业最终取得成功的故事。",
history=history)
print('2,', response, flush=True)
response, history = model.chat(query="给这个故事起一个标题",
history=history)
print('3,', response, flush=True)
response, history = model.chat(query="再讲一个不同的创业故事,情节复杂和具体的长一点的故事，逆袭成功",
history=history)
print('4,', response, flush=True)
期望行为 | Expected Behavior
there are different response for the same prompt.
复现方法 | Steps To Reproduce
No response
运行环境 | Environment
- OS: ubuntu 20.04
- Python: 3.10.13
- Transformers: 4.36.1
- PyTorch: 2.1
- CUDA (`python -c 'import torch; print(torch.version.cuda)'`): 12.1

备注 | Anything else?
No response

Currently, vllm_wrapper initializes vLLM with seed=0. You can add the seed argument on this line:
https://github.com/QwenLM/Qwen/blob/0a5b3b6b9aec3a3e7b1c2bd93580f35cfe6546fb/examples/vllm_wrapper.py#L135-L141

thanks a lot.


Unexpected end of input after update from Node 12 to Node 14

I updated from Node 12 to Node 14 and now flow gives me this error:

$ repo/node_modules/.bin/flow check --max-warnings 0
  | Skipping /repo/client/<PROJECT_ROOT>/flow-typed/: No such file or directory
  |
  | Error ┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈
  | ../../../libs/tools-common/node_modules/@babel/helper-define-polyfill-provider/node_modules/resolve/test/resolver/malformed_package_json/package.json:2:1
  |
  | Unexpected end of input, expected the token }
  |
  | 1│ {
  | 2│
  |
  |

My app is setup in a workspace and imports a package called /tools-common im not sure how that's causing a failure in my main app though

@mroch Any ideas on what could be happening here?

I added:
[ignore]
.*/**/malformed_package_json/package.json

same here with flow 0.176.3 / Node 17.8.0 / CRA 5.0.1
The fix mentioned by @simkessy right above does the trick but as pointed out by @mvitousek I can't understand why flow even cares about a deep node_modules file like that 🤷


✨ [REQUEST] Firefly: ingress support

Checked

[X] If a new addon, I have checked on  Google that such as addon doesn't already exists
[X] If a new feature, I have searched in the discussions & issues that it was not already covered

Which addon?
fireflyiii
Is your feature request related to a problem? Please describe
Not a problem, new feature
Describe the solution you'd like
Hi!
Will it be possible to add ingress support?
I would love to add it to my sidebar, Im using SSL so right now its a bit of a problem :(

I'd also love if that AddOn would use the ssl certificate provided by the Let's Encrypt AddOn so I could connect to it through https and not have it's unencrypted http port exposed...

And I guess this is a duplicate to https://github.com/alexbelgium/hassio-addons/issues/425 ?

I'd also love if that AddOn would use the ssl certificate provided by the Let's Encrypt AddOn so I could connect to it through https and not have it's unencrypted http port exposed...

Hi, this is a different request but actually the recommended way to use ssl is through an addon named nginx proxy manager that manages it in a global way and not addon per addon. Here is a very short non tested wiki on that, don't hesitate to complete it if it works ;-) https://github.com/alexbelgium/hassio-addons/wiki/Add‐ons-feature-:-ssl

Indeed, thanks I'll close it. Indeed there is low probability that firefly supports ingress due to embedded security features that are not compatible with the way the ingress proxy works


Follow-up commits for recent API changes

The recent changes lead to some minor bugs on mac and windows, which were fixed with these commits. See the two build errors here:
https://github.com/bastibl/FutureSDR/actions/runs/3513017436/

Sorry, I didn't see the other PRs. Looking at #10, I noticed that the braces I added are not necessary. I updated the PR.

The macos fix, i.e., the first commit in this PR is still relevant. There is also another PR for that. Not sure why all PRs are ignored.

How do these changes compare with the new 0.7.5? I just merged a pull request that should fix the OSX issue. Sorry, I cannot test on Windows and OSX anymore now that TravisCI has axed its free tiers.


std.traits: Adapt unittest to pass with -preview=in

I'm using https://github.com/dlang/dmd/pull/11632 as a way to adapt projects.
With this PR, it should pass for Phobos and the documentation tester.
I should probably add it to Buildkite like what Steven did for auto-decoding 🤔

CC @wilzbach , this is what motivated dlang/dmd#11635 . As you can see the workaround isn't terrible either.

I should probably add it to Buildkite like what Steven did for auto-decoding 🤔

Good idea, except implementing it for stable first ^^

this is what motivated dlang/dmd#11635 .

Yeah, I'm well aware of the problem. We unfortunately did similar workarounds in the past, e.g.
https://github.com/dlang/phobos/blob/6194b172955571ea3eb87bc8380d413dd7694598/std/typecons.d#L7356-L7362


Add an http header to the requests we send to mediaflux

Add an http header to the requests we send to mediaflux which indicates whether the user has authenticated via CAS.
Acceptance criteria

[ ] If a user has authenticated via CAS, there is a header in the http requests sent to mediaflux

[ ] the header includes the netid TIGERDATA_NETID - the user netid that is logged in
[ ] the header includes domain:princeton TIGERDATA_DOMAIN - Hardcoded to princeton
[ ] the header includes the CAS timeout data TIGERDATA_TIMEOUT - Follow the format from CAS

[ ] No call sto mediaflux should be made if we are not logged in, but if this happens then

[ ] If a user has not authenticated via CAS that header does not exist or is empty
[ ] If a user's CAS session has expired, that header does not exist or is empty

Hey team! Please add your planning poker estimate with Zenhub @hectorcorrea @JaymeeH @jrgriffiniii @kelynch @leefaisonr @bess


Replace localhost with <IP_ADDRESS> to fix Heroku issue

See https://stackoverflow.com/a/25418895.
Also see my discussion: https://github.com/danny-avila/chatgpt-clone/discussions/339

@javon27 It makes more sense to set the env variable to <IP_ADDRESS> in your heroku setup.
This seems like a simple revision but could potentially mess up other configurations.
I haven't tried the heroku setup truthfully, so if you could send a PR to revise it, that would be really appreciated!


Discussion about "duplicate" fields in log messages

Good day! On both my Mac and Yocto Linux, log messages have a timestamp and computer name prepended by the syslog process, as well as the long format timestamp and computer name added by ain2. The duplication is a bit noisy and I'm seeking to reduce it.
Current
Apr  7 10:53:21 matthews-imac 2016-04-07T15: 53:21.130Z matthews-imac.--.--.---.com MyAppname[12345]: And finally, the actual log message!
Preferred
Apr  7 10:53:21 matthews-imac MyAppname[12345]: And finally, the actual log message!
This is one option I've come up with: https://github.com/matttrawicki/log4js-syslog-appender/commit/c9d392f0658e3fcfab2d49194a47e4a9204e683a, though I'm not sure it fits the log4js mindset. Seeking your advice. Thanks and best regards, Matthew

@m-reiniger Please respond

If I am not mistaken, this is what layouts are used for. Please take a look at log4js Layouts first.
And sorry, for the late reply. The notification from github must have collided with my spam filter.

No worries, thanks for responding! Yes, I would agree that a custom layout should allow control over the formatting. The challenge in this case is that the text from the default formatter or even a custom log4js formatter is given to ain, but then ain further formats the message, adding the computer name, process id and timestamp. Ain does offer the option to pass its constructor a "message composer" function. But because ain is your dependency, there isn't a way for a consumer to configure its message composer. That's what support I added in this branch. But, that doesn't feel very log4js-ish. It's also very technical, exposing the concepts of the facility and severity.

I'll take a closer look at it when I find the time. Maybe we find a nice way to do this. But I have to read back into ain, since I haven't really used it for quite a while.
Nevertheless, thanks for the feedback.

I'll take a close look at it when I find the time. Maybe there is a nice way to do this. But I have to read back into ain and log4js, since I haven't used them in quite a while.
The problem I see, since this is just an appender, it would have to be configured in the log4js configuration. But as of now, I am not really sure if this is the way to go. I don't really want to pollute the log4js config with an ain message composer config.
Nevertheless, thanks for the feedback.

As you might have seen, I didn't have the time to implement your requested changes. And I will not have the time to do so in the future. Please forgive me and thank you for using this package. It's outdated.


Ensure tag colours are consistent

If generating multiple diagrams from the same model, it would be expected for the tag colours to remain consistent. However currently they are not.

I was thinking about this a bit when I did my last PR. Essentially what you want want is a 𝑓(text) -> pretty color. This is fundamentally a hashing problem that might be quite fun to solce 😆


APIRequestArgument doesn't support Arrays or Dictionaries

Describe the bug?
I didn't know whether to log this here, or on the OktaIdx github. But I'm considering it a bug, since OktaIdx is able to use APIRequestArgument for types it isn't compatible with, and this is something that can be fixed by a change within this SDK, so here's what happens:
I've got a Remediation.Form.Field object that has an option and that option has a nested structure within its value field that I can't get through in the OktaIdx SDK, due to the fact that the value property is returned via its custom get(): _value as? APIRequestArgument, which works just fine, but only when working with primitive types.
I'm assuming just appending
`extension [Any]: APIRequestArgument {
public var stringValue: String { "(self)" }
}
extension [AnyHashable: Any]: APIRequestArgument {
public var stringValue: String { "(self)" }
}`
to the APIRequestArgument file would fix the issue, but I have no idea whether this classifies as an Obvious Bug so I don't know whether to create a PR with it. Not to mention that we'd still need OktaIdx to be updated to require whatever new version of OktaAuthFoundation would result out of this change.
What is expected to happen?
Remediation.Form.Field's value object should be able to contain and return arrays and dictionaries.
What is the actual behavior?
Arrays and Dictionaries are returned as null due to the fact that the value property is "converted" to an APIRequestArgument object, which in a protocol not supported by arrays and dictionaries.
Reproduction Steps?

Additional Information?
No response
SDK Version(s)
OktaIdx 3.0.7, OktaAuthFoundation 1.2.0
Build Information
No response

Hi @RobertManeaSST, since options within the API behave differently than string or numeric primitive values, selecting an option uses a different property. To select an option, you assign the value to the parent field's selectedOption property.

Hi @mikenachbaur-okta. Actually, my problem was that our structure was attempting to use options, but it really only showed me one option (and I actually wanted to view said options so that I could show them to the user). So I don't know whether it's a problem with the SDK not parsing the JSON structure properly or whether our API guys improperly implemented the response here.
Regardless, I've rushed into creating this issue. I ended up just creating extensions for Array and Dictionary within our app's code so that I may parse the response and get the things I needed.
extension Array: APIRequestArgument {
public var stringValue: String { "(self)" }
}

extension Dictionary: APIRequestArgument {
public var stringValue: String { "(self)" }
}

I'm happy to close this ticket if there's nothing else you think would be worth investigating.

I'm not sure if that will work out for you in the long term, because under some circumstances the options may themselves have values that require user input (for example, options are used within Security Question authenticators, some of which may require a user to submit their own question and answer).  This is the reason why we implemented the "selectedOption" approach, since as a developer, you are choosing one sub-tree of fields to be submitted to the server for a given parent field.
The README has a few examples of how to use options, such as enrolling in a security question, which might help in the patterns you can use when interacting with options.


Delete icon Fixes #2

Fixes #2 - delete functionality complete

Added minus icon
Used Tooltip with  text  on  top  setting
modified  board_list css to align-item flex-end
added css .board_delete to position tooltip and minus icon to the  appropriate location
created  deleteBoard method  to delete board on user click, board to be deleted is based on boardId

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


some missing methods

This should probably also add some other methods. In particular, similar would be nice (otherwise it falls back to Array), and convert would enable conversions.

PR welcome.


Not an issue at all...

Just a thank you, but also a question: why not use Ansible as the provisioner, writing a simple Role for each of the languages, then just apply the relevant Role/Platbook to the Vagrantfile? Just a thought.
(I had to use an issue as I can't find an email for you, and your website just loops back to your Github profile.)

Hey, @mrcrilly,
I didn't want to use Ansible (or Puppet/Chef etc) so installation instructions were there in plain text.
As well as using this repo as a lower barrier to playing with languages, I think it's also useful to see how to install it too without needing extra infrastructure/systems.
Re website - I'm pretty lazy and haven't got around to making one yet :see_no_evil:

That's a great reason. I knew there would be a good one, so thanks for clarifying.
Just grab a Bootstrap3 theme and fill in the boxes! ;-)


[semantic-sil] Enable the mandatory sil ownership optimization on all…

… stdlib overlays.
We can do this b/c all overlays pass the ownership verifier, so it is safe.

Killing the smoke test jobs.

Part of what is going on here is that when I put in the code to only run on stdlibCore, I forgot that I had already put in an option to turn off/on the mandatory semantic arc opts. I removed the code that checked the stdlib and instead changed to rely on that code.
This will let me via cmake enable this in the appropriate places by putting in the enable-... instead of disabling it in selective places.

Build failed
Swift Test OS X Platform
Git Sha - b324761505246b3dc845d91d2dd907f2c87edc8e

Build comment file:
Code size: -swiftlibs

TEST
OLD
NEW
DELTA
RATIO

Improvement

libswiftCryptoTokenKit.dylib
28672
24576
-14.3%
1.17x

libswiftMetal.dylib
40960
36864
-10.0%
1.11x

How to read the data
The tables contain differences in performance which are larger than 8% and
differences in code size which are larger than 1%.
If you see any unexpected regressions, you should consider fixing the
regressions before you merge the PR.
Noise: Sometimes the performance results (not code size!) contain false
alarms. Unexpected regressions which are marked with '(?)' are probably noise.
If you see regressions which you cannot explain you can try to run the
benchmarks again. If regressions still show up, please consult with the
performance team (@eeckstein).

Hardware Overview
  Model Name: Mac Pro
  Model Identifier: MacPro6,1
  Processor Name: 12-Core Intel Xeon E5
  Processor Speed: 2.7 GHz
  Number of Processors: 1
  Total Number of Cores: 12
  L2 Cache (per Core): 256 KB
  L3 Cache: 30 MB
  Memory: 64 GB

--------------

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test

Build failed
Swift Test Linux Platform
Git Sha - b324761505246b3dc845d91d2dd907f2c87edc8e

Build failed
Swift Test Linux Platform
Git Sha - b324761505246b3dc845d91d2dd907f2c87edc8e

Build failed
Swift Test OS X Platform
Git Sha - b324761505246b3dc845d91d2dd907f2c87edc8e

@swift-ci test

@swift-ci test

Build failed
Swift Test OS X Platform
Git Sha - 34f036eb9c3ae4076f3c12f8f8a7ab7767e77744

Build failed
Swift Test Linux Platform
Git Sha - 34f036eb9c3ae4076f3c12f8f8a7ab7767e77744


Does this work in build too?

According to this https://github.com/posva/unplugin-vue-router/issues/2/ it says it works in build too but refreshing on pages does not work correctly in build.
Reproduction: https://stackblitz.com/github/jossephus/vue-router-build-test
Steps to reproduce:

build the project: pnpm build
Open the built project from dist.. Routing works fine... we can go to other pages and go back... but refreshing when the url path is not "/" gives cannot get url error: shown below

That’s because you’re not serving the folder just opening the file. It’s not related to the plugin


run the otel-daemon alongside load tests

Description
With this in place, we just need client details to hook up to xray in
order to receive open telemetry info from our load test suite.
Type of change

[x] :robot: Test

Issue(s)

#902

Test Plan

[x] :green_heart: E2E

I'm looking more into how xray works, but looks good from the CI side of things.

@meain Are there any blockers preventing this from moving forward?


HAWKULAR-1035 : Fix mismatch in DS used percentage

Merging #845 despite the iTest failures that have nothing to do with UI code.


Write something to report from Trusted Advisor to Kibana

We should report Trusted Advisor findings straight to Kibana.

we are no longer using trusted advisor


Entity Integrity

We need to perform checks, in the repository?, that ensures you can only save an entity if it's version matches that of the latest entry in the database.

Can we infer the version through COUNT(identifier)?
Should we explicitly store the version and use for a composite key?
Any other suggestions?

Discussion ... open

I'm thinking that using a composite key (key, serialNumber) would be the best way forward - as we could then provide event hooks to augment the event log storage (integrity hash, meta data, etc)

Had to remove Repository type checks ( #40 ) as we don't store the entity class in the event store. Now considering if this should change also!


Where to find the APK?

The app description (and its screenshots) look promising – but I found no hint on where to get the .apk. Would you mind attaching it to the corresponding releases/? Or would you mind having it included with F-Droid?

Good idea. I'll make a habit of adding the release APK to each tag, starting with this one: https://github.com/nitroshare/nitroshare-android/releases/tag/<IP_ADDRESS>

Cool! That was fast! I hope you don't mind a little promotion? Also, as initially pointed out, if you wish I could rrequest to have your app included with the official F-Droid repository. As far as I can see, it should meet the "inclusion criteria" (not being a dev, I might have missed something though ;)

Sure, I'd love to have it included in F-Droid.

OK then, I've initiated that. Feel free to "chime in" – and it might be a good idea to keep an eye on it, in case the maintainers need some input :wink:

@relan Thanks! I've pushed a fix for that and bumped the version to <IP_ADDRESS>.

Added in this commit. Thanks!


refactor: Remove experimental Anchore Enterprise upload functionality

This PR removes the experimental Anchore Enterprise upload functionality, as this is now covered by the commercial CLI tool.
Fixes #1252

-diff FTW!


Update miniz_oxide to 0.4

Duplicate of #235. Same reason as that for CI failure, it's not 1.34.2 compatible. (See https://github.com/Frommi/miniz_oxide/pull/84).

Oh sorry, I didn't check currently open PRs


[terraform/docker] Config cleanup

Remove unused trusted_peers file. Fix docker run script to publish port
and show logs of last validator. Remove prod network_peers which isn't
needed by the client, and sync consensus_peers with generated prod
config.

Good call showing last validator logs
@bors-libra r+

:pushpin: Commit baceb56 has been approved by opsguy

:hourglass: Testing commit baceb5637433cdc7a064e908681c29d7979ce5f6 with merge 344391b12250d4c49cf31c71d70cf3b64050d38f...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: opsguy
Pushing 344391b12250d4c49cf31c71d70cf3b64050d38f to master...

@mgorven : Thanks for fixing!


Language on UIMenuController

Hi,
I'm having a problem with setting the language on UIMenuController. I want to set another language on "Copy"/"Define" options that appears on log press in the cell. How can I set the language of the UIMenuController items? I'm trying without success...
Thank You.
Best regards.

@patriciasousacarbon Most of the strings in JSQMessagesViewController is localized and that is handled by the system.  When the user updates the iOS system locale, it should update the Copy/Define strings as well.  Can you attach a screenshot of the incorrect language being shown?

Hey @patriciasousacarbon :smile:
Copy/Define/etc are provided by iOS. If you have custom MenuItems, you can initialize them with localized strings.


[Bug]: rspack serve compiles bundle multiple times

System Info
System:
OS: Windows 10 10.0.22621
CPU: (12) x64 Intel(R) Xeon(R) W-2135 CPU @ 3.70GHz
Memory: 32.35 GB / 63.73 GB
Binaries:
Node: 20.10.0 - C:\Program Files\nodejs\node.EXE
Yarn: 1.22.4 - ~\AppData\Roaming\npm\yarn.CMD
npm: 10.2.3 - C:\Program Files\nodejs\npm.CMD
Browsers:
Chrome: 121.0.6167.140
Edge: Chromium (120.0.2210.144)
Internet Explorer: 11.0.22621.1
npmPackages:
@rspack/cli: latest => 0.5.4
@rspack/core: latest => 0.5.4
@rspack/plugin-html: latest => 0.5.4
@rspack/plugin-react-refresh: latest => 0.5.4
Details
rspack serve compiles multiple times.

Reproduce link
https://github.com/rspack-contrib/rspack-examples/tree/main/rsdoctor/arco-pro
Reproduce Steps
Repro:
cd rsdoctor/arco-pro
npm install
npm run dev
Expected: "Rspack compiled successfully" should happen just once
Actual: It should compile just once

Seems to be regression in @rspack/core 0.5.1. It is working fine with 0.5.0

Screenshot above is the reason that watchpack emits change event.
@Boshen could you please help on this. seems that some packages with scoped name in file dependencies led watchpack emitting change event.
also windows only.
i've tried oxc_resolver 1.4.0 with no luck.
not sure which component of rspack and oxc_resolver should fix this.

I'll get a windows laptop at work ...

I made some updates regarding windows path, can someone confirm whether it's fixed with
version 0.5.4-canary-bf8f473-20240220125622?

i'll try out it late tonight

I made some updates regarding windows path, can someone confirm whether it's fixed with version 0.5.4-canary-bf8f473-20240220125622?

thanks @Boshen 0.5.4-canary-bf8f473-20240220125622 works fine
env
Windows 11 and all the same as the reply i left in this thread before - except the rspack version.
have checked

initial build: didn't emit any rebuild without source change
source change: emitted once rebuild


Override default bootstrap CSS properties

Hi everyone,
thanks for making such a useful project. I'm wondering if it could be possible to allow overriding of the default bootstrap CSS properties. For instance, I would like to make a wider offcanvas inside a specific svelte component. Something like:
<Offcanvas placement="end">...</Offcanvas>

<style>
.offcanvas-end {
    width: 500px;
}
</style>

This code does work as is, because the CSS selector gets renamed by the svelte compiler to svelte-xxxx and thus does not override the default attribute anymore.
Maybe another way of doing this would be:
<Offcanvas placement="end" class="wider">...</Offcanvas>

<style>
.wider {
    width: 500px;
}
</style>

If this is already doable and I didn't find the right way, I would be glad to know :)

Hi, svelte lets you do global styles such as:
<style>
:global(.offcanvas-end) {
    width: 500px;
}
</style>

Otherwise it scopes as you mentioned

Thanks for your answer, I didn't know about this. However, the rule declared with :global affects every components, so all the offcanvas in my app are now wider; whilst I would like only the offcanvas inside a specific svelte component to be changed. Any way to do this?


Feature/mm number dashboard project rows

This branch will add a number column to number the projects in the dashboard

Pretty high impact change you got here! Merging


ICE_MEM is too small, needs to use 24M02 instead of 24M01

HX8K bitstream is about 32 Kbit too large for the 1 Mbit device, unfortunately.
I think this (+ possible DFM issues with ESD diodes?) pushes us to revC2.
Thanks to @smunaut for discovering this.

just changing to a 24M02 will not be enough as this will create an address clash with the fx2 eeprom.
See irc discussion for details and ideas to fix it.

I looked into this further, let me recap.

The FX2 can boot from 1010000 and 1010001. My understanding is that it first tries to boot from 1010000 and treats it as a 1 address byte EEPROM, and then from 1010001 and treats it as a 2 address byte EEPROM. However, this needs to be verified on actual hardware.
Currently, we have FX2_MEM at 1010001 and ICE_MEM at 1010010/1010011. We also have ADCs at 1010100 and 1010101.
Using the E2 strap, 24M02 can take either addresses 1010000..1010011 (which will not work because FX2 will interpret 1010000 as a 1 address byte EEPROM and likely fail to boot from it), or 1010100..1010111 (which clashes with ADCs).
The ADCs can be easily remapped to 1011000/1011001 by changing the ADR1 strap to V_A. This does not clash with the 24M02-xR variant, which has an identification page at 1011100 when E2 is strapped high.
Strapping ADCs with ADR1 to V_A deprives us of the ability to use four ADCs on the main I2C bus on revD.
When the FX2 is booting from an EEPROM, it can never directly load more than 16 KB since it only has that much RAM; the largest FX2 package that exposes the external memory bus lets it have up to 64 KB of memory, but we don't use that package, so we have a 16 KB hard limit. I have used a 32 KB EEPROM just in case it will come in handy.
CAT24M01 costs 0.78€@100. CAT24C256 costs ~0.18€@100. M24M02 costs 1.98€@100.

My conclusion is that, even if consolidating ICE and FX2 flashes into a single package would work (i.e. the FX2 can actually boot from a M24M02 strapped with E2 to low), it would be more expensive. I think the simplest solution is to remap accesses to ICE flash above certain addresses back to the top addresses of FX2 flash.


proposal view

Scope
create a view for each of the ~50 proposals that will exist to vote on during the congress. the proposals can be read as JSON from https://www.npoint.io/docs/217ecb17f01746799a3b. each porposal contains a description property, which holds text. the text should be displayed according to the formatting and colors.
There are mockups here: https://xd.adobe.com/spec/2bc5e6f1-589f-4724-6275-73449bd3ddaa-5df4/
Deliverables

a screen for each proposal
vote button on the bottom
pop-up of vote controls when vote button clicked
progress screen after submission
receipt screen after submission
rendered version of the BBcode description

Roles
bounty gardener: @johannbarbie   / 10%
bounty worker: name / 80%
bounty reviewer: name / 15%

@sunify please, check this branch:
https://github.com/deora-earth/voting-frontend/tree/feature/basic-vote-screen
There's a lot of stuff already implemented.
Would be cool if we could sync on how to deliver this.


Warm the internal config cache to avoid sending an empty snapshot.

Add a WaitForSync method in caches that exposes a channel that will receive an event once the cache has been warmed.

The cache pkg can get initialized with a list of Ingresses that will be used to determine if the cache has been warmed (all the ingresses in that list should be in the cache).

The xdsServer waits for the cache to be warmed before starting to listen. This is also controlled by a timeout of 12 minutes (~1000 ksvcs)

@davidor @markusthoemmes can you give a look at this PR? thanks! :)

Looks good :+1: but maybe we should merge #89 first. The integration tests did not run in this PR.

@davidor thanks! But I will make it more based on events instead of polling as per @markusthoemmes suggestion!
/hold

/unhold

A test for the cache behavior would be great.

added some basic tests that should cover the cache warming.

@markusthoemmes @davidor can I get a review? thanks!

/lgtm


CI builds for Python 3.7 / 3.8

CI builds for latest python versions.

ping @jcrist

Any news on this @jcrist?


Prevent jerry_create_string from copying string?

Is there a way to make jerry_create_string not duplicate the string literal instead use the same reference?

No, it is not trivial because of the inner representation of strings.

I am not sure I fully understand your question, but do you want to "pass a char* pointer" to jerryscript? Currently that is only possible for ES6 buffers.

Yes! How do I pass a pointer to JS?


[STORM-902] Simple Log Search

This feature is aimed for greatly improving the debugging of Storm.
Simple Log Search supports following important functionalities:

String search:
In UI, user can search certain string, e.g., "Exception", in for a certain worker log.
This search can happen for both normal text log or rolled zip log files.
Deep search:
User can search a string for a certain topology, which means the UI will try to search on all the supervisor nodes in a distributed way to find the matched string in all logs for this topology.
Then the matched results can be shown on the UI with url links, directing user to the certain logs on each supervisor node. This powerful feature is very helpful for users to find certain problematic supervisor nodes running this topology.

Also, the topology history REST API is added to give typeahead hints for the previously launched topology IDs for a certain user; another REST API (topology-workers) is added for managing the info (host:port) of workers in a topology.
And the html files, images files, js file, test resource files and unit tests are added for log search and its tests.
Snapshots for "search a worker log" and "search a topology's logs":

Thanks @revans2 , commits squashed and nimbus-test fixed

typeahead.jquery.min.js needs to be added to LICENSE.txt as it is part of an external project.

@zhuoliu for the most part things look really good, just a few comments that I see need to be addressed.

Sure, @revans2 . Comments addressed.

REST API docs added for topology/history and topology-workers/:id

Upmerged and rebased since the profiling stuff has been merged in.

+1 looks good to me.

+1 OK

Hello @harshach @Parth-Brahmbhatt @HeartSaVioR , since this a quite a big feature, more eyes would be very helpful. Could you guys have a look at it? Thanks in advance.

@zhuoliu Thank you for this feature. This is really great and going to help lot of users in debugging topologies. I'll add my reviews.

Thanks @harshach for the review , your comments related to the history API are addressed.

+1. @Parth-Brahmbhatt  has some reviews.

Hi @Parth-Brahmbhatt, could I have you some time to check back the fixes? Appreciate that.

@zhuoliu Sorry I did partial review and then got distracted, give me today's day and I will add more comments.

Sure, that would be great.

Hi @Parth-Brahmbhatt , really appreciate that if you have some more comments.

Hi, I am +1 on this patch. Sorry took me longer to review than I expected. Thank a lot for your contributions.

Thank you all sincerely for the thorough reviews!

+1


Add barcode detection model

added barcode scanning activity to get EANs from product barcodes.

@googlebot I signed it!


Revisit error messages to limit the instructions to "visit url for more details"

Description
Revisit error messages to limit the instructions to "visit url for more details"
Related Issues
None
Testing
Manual tests
Tasks That Must Be Done

[ ] Did you update CHANGELOG.md?
[ ] Did you add or update tests?
[ ] Did you update documentation?
[ ] Were there any special dev tricks you had to use to work on this code efficiently?

[ ] Did you add them to DEVELOPING.md?

The "broken" link can be ignored for now (already being looked into elsewhere).

Some screenshots.
If already installed:

Full install:

Error with installed version:


zoom puts people in waiting room

Setting up the Server-to-Server Oauth was working about a week ago, now when the meeting is created, people are thrown in a waiting room and it won't start.
I have reached out to Zoom but haven't heard anything. My first assumption was something in the API changed?
We have default zoom waiting rooms off, and allow people to join meetings without the host.

I was able to reproduce this. I only have a user account, not a paid account, so I can't troubleshoot further. I was required to change these settings before I got it to work:

This could be because the zoom Api v1 seems to not support
 settings: {
          join_before_host: true,
          waiting_room: false,
        },


Fix log statement causing UnicodeDecodeError in certain circumstances

In Python 2, when writing bytes data to a binary field, UnicodeDecodeError is possible if the field name is provided as a Unicode string, due to string formatting in log.debug().
The fix doesn't solve problems with non-ASCII field names in Python 2, just allows to pass field name consisting of ASCII symbols as a Unicode string (which is convenient in case when the calling code uses from __future__ import unicode_literals).
The changes in test code are to make sure that all possible byte values are written correctly.

Coverage remained the same at 84.363% when pulling 72c589542b97bbfbdcee1d80a38772ed40e8e110 on Ariki:gpkgblob into f7fc2bc9381c88ca8328af84cde196f23776e475 on Toblerity:master.

Thanks @Ariki !


[CI] Disable CodeQL for changes only on YAML

Describe the bug
CodeQL takes a long time to run, and has no meaningful output when changes are only done on YAML or MD.
Using paths-ignore doesn't quite work, because this will result in a stalemate situation where the check is Required to run, but never does. It doesn't seem to be "skipped",
One solution is maybe to create a job as a pre-step that exits early when the diff context (somewhat dificult to grab) is only YAML or MD
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/defining-the-mergeability-of-pull-requests/troubleshooting-required-status-checks#handling-skipped-but-required-checks

https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/defining-the-mergeability-of-pull-requests/troubleshooting-required-status-checks#handling-skipped-but-required-checks

So the solution is ...

"You can fix this by creating a generic workflow, with the same name,"

I suppose that both workflows with the same name will always run but if one of them fails then the required check fails? Feels like a strange way to solve this issue.

We should also ignore CodeQL changes when we are only changing markdown docs as well.

Since the no-op one always succeeds, the only one that can fail is the original, but it shouldn't run due to paths-ignore. Weirdly enough the PR test I have still hangs on expected/waiting, despite the names of the workflows and jobs to be identical.

I believe this is happening because the protected branch expects "checks" to be posted by the workflow. In the case of our no-op the checks aren't posted. I think we need to require the "CodeQL / Analyze" job to run successfully rather than require checks to be posted.
required status checks on protected branches is really strange UX

Thanks Ian! That was helpful. I just re-enabled it after the PR merge to allow for "any source" rather than the github scanning tool. I think that addresses that issue. Check me on it if needed

Unfortunately it looks like that's not working... I think it still expects scanning results, just from "any" source but our no-op doesn't post any.
We can add the "Analyze" job to the required checks and I think that's the right solution, but the current Analyze job step is a matrix strategy which translates to a "Analyze ([language])" job name like "Analyze (go)". It's so annoying that GitHub uses the job names for this.


Prepare release 4.0.1

Codecov Report

Merging #5421 (c941a42) into master (1ea0c74) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #5421   +/-   ##
=======================================
  Coverage   96.08%   96.08%
=======================================
  Files         717      717
  Lines       71366    71366
=======================================
  Hits        68571    68571
  Misses       2795     2795

Flag
Coverage Δ

servertests
35.77% <50.00%> (ø)

unittests
96.02% <100.00%> (ø)

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

Impacted Files
Coverage Δ

moto/s3/models.py
96.15% <100.00%> (ø)

moto/s3/responses.py
95.60% <100.00%> (ø)

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


Support multiple WiFi networks for an interface

Nerves.Network doesn't currently support configuring multiple ssids for an interface.
WPA supplicant supports multiple network entries with priority (where higher number is higher priority), where the interface scans for ssids and connects to the highest priority network, and Nerves.Network can be modified to accept and configure wpa_supplicant to do that.
POC code to add a configuration to wpa supplicant without removing other network configs (thanks to @GregMefford for working through this with me):
alias Nerves.WpaSupplicant
wlan_if = :"Nerves.WpaSupplicant.wlan0"

x = WpaSupplicant.request(wlan_if, :ADD_NETWORK)
WpaSupplicant.request(wlan_if, {:SET_NETWORK, x, :ssid, "My Network"})
WpaSupplicant.request(wlan_if, {:SET_NETWORK, x, :key_mgmt, :"WPA-PSK"})
WpaSupplicant.request(wlan_if, {:SET_NETWORK, x, :psk, "letmein"})
WpaSupplicant.request(wlan_if, {:SET_NETWORK, x, :priority, 10})
WpaSupplicant.request(wlan_if, {:ENABLE_NETWORK, x})

Although the wpa_cli documents that this causes a reassociation to the current network, in practice it causes the interface to rescan networks and connect to the highest priority available ssid:
WpaSupplicant.request(wlan_if, :REASSOCIATE)

Together these pieces can be used to add support to the Nerves.Network.setup function and the config.exs support for default network config so that multiple ssids can be supported at configuration or runtime.

I pushed a branch with working support for multiple networks to https://github.com/nerves-project/nerves_network/tree/multiple-wifi-networks. It maintains backwards compatibility with:
      Nerves.Network.setup "wlan0", ssid: "myssid", key_mgmt: :"WPA-PSK", psk: "secretsecret"

while adding support for multiple network config via:
      Nerves.Network.setup "wlan0", networks: [
        [ssid: "myssid", key_mgmt: :"WPA-PSK", psk: "secretsecret"],
        [ssid: "myotherssid", key_mgmt: :"WPA-PSK", psk: "othersecret", priority: 10]]

I'm not particularly attached to the :networks key or the nested list of keyword lists. I just wanted to push something forward and see where the discussion goes.

Testing changes from the multiple-wifi-networks branch out for a bit since I saw some crashes during development that gave me pause. They seemed to be unrelated but I want to be more certain before I open a PR.

Has there been upates to this issue?

I think we can close this with version 0.4.0 being released, right?

Also, thanks again @tmecklem ❤

Whoops yep. Thought the PR would close it. Thanks @tmecklem!!


Add additional FOR_ALL_ macros and use it consistently

Codecov Report

Merging #5339 into master will increase coverage by <.01%.
The diff coverage is 93.23%.

@@            Coverage Diff             @@
##           master    #5339      +/-   ##
==========================================
+ Coverage   82.25%   82.26%   +<.01%
==========================================
  Files         134      134
  Lines      147578   147572       -6
==========================================
+ Hits       121389   121396       +7
+ Misses      26189    26176      -13

Impacted Files
Coverage Δ

src/netbeans.c
27.53% <0%> (+0.29%)
:arrow_up:

src/spellsuggest.c
79.37% <0%> (ø)
:arrow_up:

src/buffer.c
85.1% <100%> (ø)
:arrow_up:

src/filepath.c
83.92% <100%> (ø)
:arrow_up:

src/userfunc.c
88.45% <100%> (-0.01%)
:arrow_down:

src/window.c
88.3% <100%> (ø)
:arrow_up:

src/autocmd.c
87.74% <100%> (ø)
:arrow_up:

src/evalvars.c
85.38% <100%> (ø)
:arrow_up:

src/insexpand.c
88.17% <100%> (ø)
:arrow_up:

src/if_ruby.c
91.23% <100%> (ø)
:arrow_up:

... and 32 more

Continue to review full report at Codecov.

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


[Horizon] Modeline active and inactive foreground should be swapped

In doom-horizon (thx for the port btw!) the foreground face of modeline under active state and inactive state should be swapped like so:
    (modeline-fg     (doom-darken fg 0.2))
    (modeline-fg-alt (doom-lighten bg 0.2))

I just created a PR: https://github.com/hlissner/emacs-doom-themes/pull/402

Fixed on #402!


reevaluate remote write queue config defaults

We'll want to be able to maintain the same max samples/s output with the new defaults, but bumping the max samples per send would lower the shard requirements, and upsharding can lead to DDOS like issues since we have no back pressure mechanism (related issues: https://github.com/prometheus/prometheus/issues/5166)

Uploading Transactional based remote-write _ Design doc.docx…

The main thing I would change is to lower max-shards from 1,000 - maybe 100.  Even that is too big for most installations.

The defaults have been changed: https://github.com/prometheus/prometheus/blob/main/config/config.go#L155-L172

And it was me that changed them 🤦♂
Current MaxShards: 200 is still too high, but not quite as bad.

Hello from the bug scrub:  seeing that there has been no PR opened in the last 3 years, this seems to be low priority or went away. Please reopen if there is new information, work to be done.


Ability to use WebC components as client-side custom elements?

Hey @zachleat ,
This is super, super cool! I'm excited to play with it.
I have a project at work that I think this may be a good match for, but I'm not 100% sure if I understand correctly.
For the project, we're going to be making a handful of components that will be used in a variety of different server environments (PHP, Ruby, Node)
In a perfect world I'd like to provide component consumers with a couple of options for using these components:

Hit an API endpoint that would return the HTML, JS, and CSS necessary for rendering the component. They can use these to build their page server-side.
Alternately, allow them to include a script and then use the web components as native client-side custom elements (e.g. add my-component in their markup and have the component render client-side without SSR.)

From reading the README it seems like WebC would be an excellent fit for the first use case, but it's not clear to me whether it would work for the second use case.
Is WebC meant to support both of those use cases or is it just focused on the first?
Thanks!

I don’t think WebC will likely support the use case where you don’t use a build to compile the optimized markup. I don’t think it’d be a good investment of time to try to do “isomorphic” components that can render on the client or the server (though that would be possible, WebC is simple enough that it could run in a browser with minimal repackaging 👀). I just don’t think that will produce output that I would want Eleventy to produce. I’d rather see Eleventy operate in Serverless or Edge rendering where you wouldn’t have to do clientside stuff. (My opinion on this may change if WebC gets more popular 😅, I can definitely see some use cases in CMS-land where it’d be nice to have WebC rendering for client previews? Not sure yet!)
All that said, I do want to plug the latest tutorial that shows how to use clientside interactive web components with Eleventy, which hopefully is of some interest to you! https://www.youtube.com/watch?v=p0wDUK0Z5Nw
Let me know if you have more questions!

Thanks @zachleat !
That makes sense. I totally understand focusing on the Serverless/Edge use case.
I appreciate you taking the time to answer, and I'm excited to get a chance to play with this. That's a great tutorial. WebC seems like a great authoring experience, especially when coupled with 11ty. (Makes me want to start redesigning my personal site again 😅 )


Very rare ReferenceError

We are running python 3.8 on AWS lambda. We use boto3. The code is patched by aws-xray-sdk and lumigo tracer. Very rarely (every few months) we encounter a ReferenceError. This will happen again and again as long as the same instance of lambda is reused.
We could not find a way to reproduce it. All we have is the stack trace.
Here is one stack trace:
[ERROR] ReferenceError: weakly-referenced object no longer exists
*** application part of stack trace ***
table_handler.update_item(
File "/var/runtime/boto3/resources/factory.py", line 580, in do_action
response = action(self, *args, **kwargs)
File "/var/runtime/boto3/resources/action.py", line 88, in call
response = getattr(parent.meta.client, operation_name)(*args, **params)
File "/opt/python/botocore/client.py", line 530, in _api_call
return self._make_api_call(operation_name, kwargs)
File "/opt/python/wrapt/wrappers.py", line 644, in call
return self._self_wrapper(self.wrapped, self._self_instance,
File "/opt/python/aws_xray_sdk/ext/botocore/patch.py", line 38, in _xray_traced_botocore
return xray_recorder.record_subsegment(
File "/opt/python/aws_xray_sdk/core/recorder.py", line 462, in record_subsegment
six.raise_from(exc, exc)
File "", line 3, in raise_from
File "/opt/python/aws_xray_sdk/core/recorder.py", line 457, in record_subsegment
return_value = wrapped(*args, **kwargs)
File "/opt/python/botocore/client.py", line 943, in _make_api_call
http, parsed_response = self._make_request(
File "/opt/python/botocore/client.py", line 966, in _make_request
return self._endpoint.make_request(operation_model, request_dict)
File "/opt/python/botocore/endpoint.py", line 119, in make_request
return self._send_request(request_dict, operation_model)
File "/opt/python/botocore/endpoint.py", line 198, in _send_request
request = self.create_request(request_dict, operation_model)
File "/opt/python/botocore/endpoint.py", line 134, in create_request
self._event_emitter.emit(
File "/opt/python/botocore/hooks.py", line 412, in emit
return self._emitter.emit(aliased_event_name, **kwargs)
File "/opt/python/botocore/hooks.py", line 256, in emit
return self._emit(event_name, kwargs)
File "/opt/python/botocore/hooks.py", line 239, in _emit
response = handler(**kwargs)
File "/opt/python/botocore/signers.py", line 105, in handler
return self.sign(operation_name, request)
File "/opt/python/botocore/signers.py", line 149, in sign
signature_version = self._choose_signer(
File "/opt/python/botocore/signers.py", line 219, in _choose_signer
handler, response = self._event_emitter.emit_until_response(

Hi @tsah-alike, in reference to your recent comment in issue linked above, were you able to resolve this issue by replacing the lambda instance? Has this error reoccurred since?

Hi @carolabadeer, I'm closing this as I managed to reproduce it without the xray tracing.


Showig incorrect number of streaks

i have continues streak and my streak status is showing till jul 21 only , i didnt break any streaks

I am having the same issue.
👉🏽 https://github.com/rica213
The issue started in the 20th of July
My contributions: never missed a day 👇🏽

Yet my streak is restarting  👇🏽

The results displayed represent whatever the GitHub API returns, it can give different results for different people possibly due to timezones.
It's possible to manually add commits to fill in the dates that are missing if you want, there's more info in the FAQ.


missing TE header

Getting this on the server:  NettyServerHandler [WARNING] (grpc-nio-worker) - Expected header TE: trailers, but null is received. This means some intermediate proxy may not support trailers
This results in the client getting:  Error:  One or more errors occurred.  (Status(StatusCode="Cancelled", Details="No grpc-status found on response."))

It's already got fixed in Best HTTP/2, but it's not released yet. I plan to release it in one or two weeks, but if you drop me a mail at<EMAIL_ADDRESS>or ping me at the plugin's discord channel i can give a download link to an updated package.

resolved by an update to Best HTTP/2.  Closing.


Adding transparency option to Lualine

Lualine transparent TRUE

Lualine transparent FALSE or DEFAULT

All done!

Amazing! Really appreciated.

@cuencapm FYI, I forgot we turned this on by default in the config. I've turned it off in the latest commit so you'll need to turn it on manually


Implements StylePropertyMapReadOnly.entries

refs #15

Current dependencies on/for this PR:

main

PR #18   👈

PR #19

This comment was auto-generated by Graphite.

Merge Activity

Sep 24, 8:28 PM: @tylergaw started a stack merge that includes this pull request via Graphite.


Improving support for django-suit and the likes

Since django-cms has grabbed the opportunity to depend on the django admin backend for frontend editors, we'll have to deal with a lot of crappy fixes and work-arounds -- sorry, just being provocative ;)
But really, you're not getting me to leave django-suit, so I'm opening this ticket to document and refer my pull requests.
For instance, javascript quirks are looking for <input type="button"> in the admin HTML output in order to put that in the parent fra. SInce django-suit is using a <button> element, it breaks like so:

(Danish: "Aflys" is Cancel, but there's no "Gem" / Save button)
I'm going to make a pull request in a moment for the most urgent things like getting the Save button back.

All specific admin styles within the CMS have been removed in favour of supporting different admin themes. Themes can now control the content display of the modal or sideframe using:

.cms-admin for global injection (wherever the cms renders an iframe)
.cms-admin-modal only for the modal
.cms-admin-sideframe only for the sideframe

This allows the theme developers to have more control and shifts the responsibility to their repos. Examples of the implementation can be reviewed via https://github.com/divio/djangocms-admin-style

@darklow :)


pyMSVC

Describe your issue
I am the author of pyMSVC and I just noticed the link in the Cython documentation. I went and checked the insights for that repository and over the last 30 days there were 59 visitors that were referred by the Cython documentation. I know it's not a huge number and hopefully it has helped some of those people.
I have not had a need to make any changes with the library at all. Kind of a bad thing because it looks like it has been abandoned. So I wanted to make sure that you guys know it is still being maintained there hasn't been any need to update anything. no bug reports or anything like that, it just works.
I did a little bit of research to see if setuptools has been updated to sole the issues. I found that it has not been updated. over the last year there has been 2 changes to the MSVC compiler detection, one had to do with numpy and the other removed a deprecation warning. Even with all of the complaints about it not working correctly it has not been fixed.
I did however notice some comments at the top of the msvc.py file in setuptools and I actually read them and didn't just glance at them. simplified version of what it says is that it does not support Visual Studio... It only supports Visual C++ Build Tools (Visual Studio Build Tools). Those are actually 2 different things and locations of the components used to compile an extension will be different for both the registry and also the file system. This is something that is not documented for setuptools and is only found in the comments of the msvc.py file located within setuptools
so it is known, pyMSVC supports both VS and Build Tools.
I am hoping that people that are looking at the pyMSVC repo are actually using it and solving their issues with it. Has there been a decline in the number of people having failed builds on Windows?

I'm going to close this because I don't know that there's anything for us to actually do here. But:

Glad it's still maintained and we can continue to link to it!

Presumably if it's working right then people should visit once from the Cython documentation and then never think about it again. So perhaps a huge rate of visitors isn't expected.

If you think there's anything we can improve on the Cython documentation then please feel free to suggest it

Has there been a decline in the number of people having failed builds on Windows?

I don't know to be honest - I don't remember seeing too many recently, but there was never a huge rate of issues. Possibly it is helping, but it's hard to be sure. I think using Mingw on Windows is the current poorly-documented semi-supported feature that people like to ask about.

no worries about closing this. It was more for the purposes of keeping you guys informed that I am still maintaining the project even tho it appears to be stale. I also wanted to let you know what I had discovered about the MSVC portion of setuptools so if anyone does come asking you have some answers to give them. I did want to let you know that people have found the link in the documentation as well so it's doing it's job for the people that need it.


CORS

Hi Ant how can we deal with CORS. I have a React application using Axios but when I do so there is a message saying that CORS is not allowed.

Hi I just now figure out we will need to add flask-cors to the requirement.txt firstly and then run make server.install to update the dependencies and then add it to the server.py. If you don't mind will you add that to your current code?

hi @clifflo !
I do not think flask-cors is required, since it won't be needed for every API this boilerplate is used for.
Your issue is a good documentation for addressing this problem though, thank you for it and for your response!
For those coming here with the same problem, you can find the documentation for flask-cors here: https://flask-cors.readthedocs.io/en/latest/
Closing this now


Gdm/display group names

Looks good.
Did not look at the code or tests, can someone from the devs do that.


Map isn't updating on region prop change

I have this issue on Android. onRegionChange function is called, component render() function is also called but the map doesn't load new data. I used code from example. First image is my initial map, and second one is what happens when i zoom it few times. Did anyone have this problem or has idea how to fix it ?
Thanks!!

It worked when i used browser key for google maps, instead of using android on!


[Backport 2.x] Fix error handling while reading analyzer mapping rules

backport 6d20423 to 2.x
Signed-off-by: Rabi Panda<EMAIL_ADDRESS>
Description
Add new parseWordList method that takes a parser as a parameter. It reads custom rules from settings or a file, parses and handles errors. Make error messages less verbose for rules files outside config directory.
Issues Resolved
N/A
Check List

[x] New functionality includes testing.

[x] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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

This change is also going to 2.4, should we update the CHANGELOG for 2.x ?

This change is also going to 2.4, should we update the CHANGELOG for 2.x ?

You did the right thing here. I have a PR out now to provide some updated instructions here. Soon the skip-changelog label will work when we don't need to add a changelog entry. (for now I used autocut which has the same effect)

Build is failing on something new, https://github.com/opensearch-project/OpenSearch/issues/5121, @mch2 is on it.

Rebase this and make a 2.4 backport as well, please? Let’s make sure this passes all CI.


Scheduler "live demo" sample code broken

Copying and pasting the "live demo" example code chunk from the documentation results in a non-working scheduler.
With the "live demo" code from the docs:
https://jsfiddle.net/cmcculloh/h6gfhwnk/
With the "markup" code from the docs (works):
https://jsfiddle.net/cmcculloh/31omqduv/

This is probably related to #1599 and #1511.

@chriscorwin  please confirm the "live demo" code matches the "markup" code

@cmcculloh please tell me exactly what URL you are seeing this issue at.
We have a lot of "live demo" stuff. :D

http://getfuelux.com/javascript.html#scheduler-examples-live-demo
vs
http://getfuelux.com/javascript.html#scheduler-usage-markup
would have to confirm with @cormacmccarthy to determine where he got the code for the two fiddles :guitar:


feature request: support pnpm and yarn projects

pnpm uses a shrinkwrap.yaml file and yarn uses yarn.lock.
Both are different than the lockfile of npm, so it might be good to support them.

The issue is that in a lot of ways - the security analysis of npm is just a wrapper around nsp. If one was strictly using node, I would recommend just using nsp or npm audit. ODC is useful in the case of multi-language projects or for organizations that wanted to have a single format that could be imported more easily into dependency-track or a vulnerability management platform.

The issue is that in a lot of ways - the security analysis of npm is just a wrapper around nsp

Not anymore as nsp is deprecated and they use https://github.com/nodejs/security-wg and other resources now.

See https://www.npmjs.com/package/nsp
npmjs bought nsp.

We know - and yet today, the dependency-check analysis is using nsp which will be around through Sept, 2018.

https://github.com/yarnpkg/yarn/issues/5808#issuecomment-409468136
https://github.com/yarnpkg/yarn/issues/5808#issuecomment-410171673
https://github.com/npm/npm-audit-report/

We know - and yet today, the dependency-check analysis is using nsp which will be around through Sept, 2018.

Which is not so far away ;-)
Is there a PR for the migration to the audit API or other endpoints?

@stevespringett any progress on the changing the endpoints for the NspAnalyzer?

@jeremylong I'll be creating a branch for this soon as the commit is going to be decent size (20+ files modified already). I should have something posted later this week. After we replace NSP with NPM Audit however, we'll have to continue to make changes since NPM Audit, to this day, still has a requirement on NSP. So its a bit of a moving target that we'll have to stay on top of.
@DanielRuf Previously, NSP simply used package.json in a slightly modified payload to analyze the project for vulns. NSP would perform all the resolution, etc. There wasn't a requirement to specify every single dependency and since package.json was fairly universal, it wasn't specific to NPM. However, NPM Audit has dramatically changed that. The service is more closely tied to NPM and requires package-lock.json or npm shrinkwrap. A yarn project will need to convert yarn.lock to package-lock.json in order for it to be analyzed. Check out synp for a possible solution. File conversion is out-of-scope for the Dependency-Check project. Conversion to package-lock.json will need to be done on each of the non-NPM ecosystems...... That said, analyzing Node projects doesn't necessarily need to rely on NPM Audit in the future. Once #1276 and #1391 are complete (this requires back-end changes on Sonatypes part), it will be possible to create a Dependency-Check analyzer for Yarn lock files and use OSSIndex to perform the analysis instead of NPM Audit. This will likely require additional setup however - i.e. registering for OSSIndex and obtaining an API Token since registered users have a higher request limit than non-registered users. Having a registered account would be especially important for Node projects because of their micro-module approach.

It would be awesome with automatic conversion. I tried adding a synp step to convert in my build and then analyse, but still get "Skipping CPE Analysis for npm"
My cmd line:
synp --source-file yarn.lock && mkdir -p dependency-check && /Users/et2448/Downloads/dependency-check/bin/dependency-check.sh --enableExperimental --format ALL -o dependency-check --project TACJFR --scan package.json

At least we have yarn audit now.

I'm guessing this request will never be implemented?

hey.  It's been a few years, is this still in consideration?

I have not had time to support yarn. We accept PRs - that is actually where most of the support for other package managers originated.

I was hoping for pnpm support :)
Is there a doc for adding support?  I don't really know java, but if there's a doc and decent model to follow, I can probably figure out enough to get a PR up for review.

Honestly, the current node analysis is doing exactly the same thing as npm audit (or in your case pnpm audit). We are looking at bringing in the Node.js information from Retire.js - but no work has been started. What would be needed is a way to convert the package files from pnpm into the information that would get submitted to the npm audit service. The start of this can be found here:
https://github.com/jeremylong/DependencyCheck/blob/c33513933bcddfb2efebd97384be9f39820bd62a/core/src/main/java/org/owasp/dependencycheck/analyzer/NodeAuditAnalyzer.java#L175-L189

Yarn audit is now wrapped by ODC.


The sparse matrix that "scipy.io.savemat" function save cannot be properly loaded in Matlab R2014a

My matlab version: R2014a (<IP_ADDRESS>2) 64-bit (glnxa64)
My scipy version: 0.15.1 (in python3)
My operating system: CentOS release 6.6 (Final)
My python3 code to save a csr_matrix:
import numpy as np
from scipy.io import savemat
from scipy.sparse import csr_matrix

a = np.array([[1, 2, 3, 4], [5, 6, 7, 8]])
b = csr_matrix(a)
savemat('test.mat', {'b': b})

Later I load test.mat in my matlab:
load test.mat
b

The result is:
b =

  9.9e-323 *

   (1,1)       0.0500
   (2,1)       0.2500
   (1,2)       0.1000
   (2,2)       0.3000
   (1,3)       0.1500
   (2,3)       0.3500
   (1,4)       0.2000
   (2,4)       0.4000

The result is very strange, and the reason seems to be the default .mat file format changed to v7.3 in newer version of matlab. I just think maybe it's better to show warning message in scipy when user use savemat function.  Thank you.

After testing the code below:
import numpy as np
from scipy.io import savemat
from scipy.sparse import csr_matrix

a = np.array([[1, 2, 3, 4], [5, 6, 7, 8]], dtype='double')
b = csr_matrix(a)
savemat('test.mat', {'b': b})

only convert a into a float type, the matlab works well:
>> load test.mat
>> b

b =

   (1,1)        1
   (2,1)        5
   (1,2)        2
   (2,2)        6
   (1,3)        3
   (2,3)        7
   (1,4)        4
   (2,4)        8

It seems that object MatFile5Writer in function savemat could not correctly handle integer type, cause a format bug in matlab
function savemat: https://github.com/scipy/scipy/blob/v0.15.1/scipy/io/matlab/mio.py#L203

Related issue: https://github.com/scipy/scipy/issues/4826


member_subscription :: json payload returned from pusher and from slanger differs

One thing I noticed is that the presence channel subscription succeeded events which Slanger sends, differ somewhat from Pusher.
I had to add code like this for example to get the member subscriptions (the ruby example uses my fork of the pusher-client gem)
 # for subscription events
      if USE_SLANGER
        @member_ids = pusher_response['presence']['ids'].compact.uniq
      else
        @member_ids = JSON.parse(pusher_response).map {|member| member['user_id'].to_i}
      end

can you paste what json both slanger and pusher sends. this is going to be the most helpful for me. it be logged in the browser console if you define Pusher.log e.g.
Pusher.log = function(message) { console.log('\t\t', message); };


new version of asde deployment extension

the vsix file has already been uploaded to the storage account.

@kburtram , I'll leave this to you to merge and publish.


Create an issue template for GitHub

The repository already has a CONTRIBUTING.md file that mentions calling youtube-dl with the --verbose flag when reporting issues. This file is linked from the new issue page above the new issue form and is noticeable. I think it has helped reduce somewhat the number of bad bug reports. I don't know if you share that opinion.
Recenly, GitHub has introduced the concept of issue templates. Perhaps you could create an ISSUE_TEMPLATE.md file with an unmissable request to include the --verbose flag and the full output in a proper code block, and to make sure they're running the latest version of youtube-dl.
Feel free to close if you don't agree.

Solved. Thanks!


feat: add nasa apod project

Type of change

[x] Added new project

Changes proposed in this pull request

Project name : NASA Astronomy Picture of the Day

Short description : The simple project downloads the NASA APOD with the help of NASA API key gotten from NASA Open APIs

Checklist

[x] The code is Python 3 compatible
[x] I have added relevant comments in my file to make code understandable
[x] I have added my project in a folder which includes all the necessary files

Looks good!
Happy Hacktoberfest 👍

Thanks, @Mrinank-Bhowmick
I'm so glad to contribute as a first-timer.


chore(readme): Fixed mistaken error description

Removing mistaken comment from "could not be processed" error description.
It mistakenly states that this indicates a virus has been found. When it means it could not scan the file.

Thanks @NikoRoberts!

You are very welcome Antti!


✨ added FileName to logger

Added FileName parameter to logger for better understanding where the function was called from.

looks nice!
Can we put line number also?

line number is always same, it is the line where .log(isDebug: ) function is located ㅠ

The file name is always same as well..


Update NodeJS and project packages due to security warnings

📝 Issue
Resolves #91  .
📖 Notes
Upgrade to the new NodeJS LTS version.
Upgrade packages due to security vulnerabilities

Ok right now the only one issue is the well know web3. I will complete the review performing some functional testing.

All the functional tests passed correctly:

keystore creation
trading wallet creation
deposit and withdraw of ETH and Token
create and cancel exchange orders


need to avoid to specify access key and secret key

For accessing to cloudwatch best practice to use ec2 metadata endpoint for getting creds.
Usually for EC2 specify instance profile with required permissions

I believe the boto3 client takes care of this. Take a look at this for more details - https://boto3.amazonaws.com/v1/documentation/api/latest/guide/configuration.html#configuring-credentials
In the readme, we can be more explicit about setting up credentials.

thanks


Bug: Class Property is missing at child level

The documentation mentions as follows

However, Schema misses out on adding class property within conditions

HI @koustov
I went through CONTRIBUTING.MD file. It has been mentioned to run npm test
However, I observe that test has not been setup on package.json
Could you please guide me on how to run this locally. I would like to attempt on fixing this issue.

HI @koustov I went through CONTRIBUTING.MD file. It has been mentioned to run npm test However, I observe that test has not been setup on package.json
Could you please guide me on how to run this locally. I would like to attempt on fixing this issue.

Hello @supminn ,
Thanks for the issue. I am working on improving the readme and expected to complete by EOD today. Will update you on the same and then you can take up this issue.
I will also be tagging this issue with hactoberfest

HI @koustov I went through CONTRIBUTING.MD file. It has been mentioned to run npm test However, I observe that test has not been setup on package.json
Could you please guide me on how to run this locally. I would like to attempt on fixing this issue.

Hello @supminn , Thanks for the issue. I am working on improving the readme and expected to complete by EOD today. Will update you on the same and then you can take up this issue. I will also be tagging this issue with hactoberfest ANd regarding npm test. Its not there as of today, will create an issue for it.

Sure @koustov, Thanks.

I have got #4 raised which covers both #3 and #5.
I have tested them out locally with example payloads added to the repo.
Please review it and let me know your feedback.

PR merged


Parameterized builds

Problem
When triggering a build, optional parameters can be passed to override or add environment variables to that job:
https://circleci.com/docs/parameterized-builds/
Currently the library doesn't support doing this (Unless I completely missed it 😳)
Solution
I though about modifying the Build method to taken an optional (Can be nil) body but it changes the interface that some people may be relying on and as there's currently no versioning in the app could end up breaking peoples apps if they pull down the latest version.
Opted for a separate method ParameterizedBuild that essentially has an extra parameter for a struct that's Marshalled into a JSON string and posted to the trigger build endpoint.

@stevenjack Thanks for putting this together! What do you think about doing map[string]string for the build parameters to provide a more structured interface?

Also, I think you made the right call with the separate method; even if the library was versioned I think this is still a better trade-off than changing the existing method signature (to avoid having to break the interface).

Opened #15 based on this branch to make this change, thanks for getting this started @stevenjack !


Android amalgamation mxnet_predict-all.cc: cblas error when execute forward function

When execute the line "p->exec->Forward(false);":
int MXPredForward(PredictorHandle handle) {
  MXAPIPredictor* p = static_cast<MXAPIPredictor*>(handle);
  API_BEGIN();
  p->exec->Forward(false);
  API_END();
}

There will generate cblas error:
 ** On entry to SGEMM  parameter number 13 had an illegal value
 ** On entry to SGEMM  parameter number 13 had an illegal value
 ** On entry to SGEMM  parameter number 13 had an illegal value
 ** On entry to SGEMM  parameter number 13 had an illegal value
 ** On entry to SGEMM  parameter number 13 had an illegal value

Because i use five convolution layer in network, so i guess perhaps this error relate to the cblas function cblas_sgemm which do the matrix multiplication. Anyone can help me? Thanks!

I have the same problem.

@nihilityworld are you using OpenBLAS or some other implementation ? On the OpenBLAS side https://github.com/xianyi/OpenBLAS/issues/1014 was created but we did not receive further details.

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


Feature/13296 carousel image section

I want to merge this change because...

PR intended to be tested with API branch:
Screenshots

Pull Request Checklist

[ ] This code contains UI changes
[ ] All visible strings are translated with proper context including data-formatting
[ ] Attributes [data-test-id] are added for new elements
[ ] Changes are mentioned in the changelog
[ ] The changes are tested in different browsers and in light/dark mode

Test environment config

API_URI=https://qa.staging.saleor.cloud/graphql/

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: trungle88:x: Nguyen Ngoc DuyNguyen Ngoc Duy seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Make sure all SDKs subscribe to the correct topics

Each SDK has a different approach to subscriptions, we need to standardize the subscriptions to these well defined list of topics:

Consumer properties control path (e.g. <realm>/<device_id>/control/consumer/properties)
Server owned interfaces subpaths (e.g. <realm>/<device_id>/<interface_name>/#)

Right now many SDKs subscribe also to this topics, which must be removed:

Device owned interfaces
Server owned interface root path (e.g. <realm>/<device_id>/interface_name>)

The latter was inherited from the first Astarte SDKs, it was there for future usage but it isn't currently used for anything so it can be removed and possibly added again when it's needed.
SDK checklist to close this issue:

[x]  Qt5: https://github.com/astarte-platform/astarte-device-sdk-qt5/pull/19
[x] Java
[x] Elixir: https://github.com/astarte-platform/astarte-device-sdk-elixir/pull/31
[x] ESP32: https://github.com/astarte-platform/astarte-device-sdk-esp32/pull/66
[x] Go: https://github.com/astarte-platform/astarte-device-sdk-go/pull/14
[x] Python: https://github.com/astarte-platform/astarte-device-sdk-python/pull/7

Closing this since it's implemented in all SDKs


Create 0000-01-02-xanderlav.md

My first contributions to this branch! how cool is that!

Let's go with this!

Let's go with this!


Missing Python Api

Hi,
i'm using kicad 5.1.4 on Linuxmint 19.2 Cinnamon.
If i try to use your plugin it says
Missing required Python API:
pcbnew.ExportSpecctraDSN
pcbnew.ImportSpecctraSES
But under File -Export (and import) both options are there and working.

This plugin only works in nightly versions. In 5.1.x the export/import functionality is there (in menus) but not exposed through python api so plugin can not function.


Rebol formatting fails after a multiline comment

(Original issue 985 created by drgchiu on 2014-05-05T08:46:25.742859+00:00)
See https://github.com/gchiu/RSOChat/blob/master/pygments.reb
for an example of how "comment" itself is not recognised, and how syntax highting fails completely after this

(Original issue 985 last updated on 2014-05-05T17:36:38.933659+00:00)
(Issue automaticaly closed due to status in Bitbucket: resolved)

(Original comment by Oldes on 2014-05-05T08:57:28.802041+00:00)
comment is recognized correctly, but the end of it is not recognized. I will try to fix it.

(Original comment by Oldes on 2014-05-05T09:32:56.474393+00:00)
Here is pull request with fix.
https://bitbucket.org/birkenfeld/pygments-main/pull-request/345/rebol-comments-handling-fixed/diff

(Original comment by tshatch on 2014-05-05T17:36:38.926945+00:00)
FIX: Issue #985 - REBOL comments
→ <>