protojson: unmarshal does not properly support null input values

What version of protobuf and what language are you using?
v1.26.0
(v1.5.0 of APIv1 module)
What did you do?
I tried to unmarshal JSON into a protobuf message, with various scenarios where the input data contains null values.
You can see the code here.
In particular, note TestJSONPb and TestProtoJSON functions.
What did you expect to see?
I expected the protojson package to support the test data containing null values, just as the jsonpb package does. This is based on the following snippet from the spec for the JSON format of protocol buffers:

If a value is missing in the JSON-encoded data or if its value is null, it will be interpreted as the appropriate default value when parsed into a protocol buffer.

What did you see instead?
The protojson package only supports null values as direct field values, not if they are elements in JSON arrays or values in JSON maps. So only 4 of the 12 test cases in the example code pass. (All of them pass for the jsonpb package.)
I can understand there being some confusion about how to handle this case for message types -- for normal fields, the default/zero value of a message type is nil. But an element in a repeated field that is nil cannot be serialized: a repeated message value can be an empty message but not nil.
However, for symmetry, I would expect a null value in a JSON array to result in an empty message type, just as the older jsonpb did. I say "symmetry" because this is basically what happens if I construct a Go struct literal for a protobuf type, where a repeated message field (e.g. a slice of pointers to some other Go struct) contains a nil pointer: it gets serialized as if it were an empty message. (See the TestMarshal function in the gist linked above.)
Anything else we should know about your project / environment?
Nope.

There's a internal document created by the protobuf team that's generally considered more authoritative than whats posted on the developer site. In that document, it says:

“null” is only allowed for a whole field. It cannot be used as an element in a repeated field or as key/value for a map entry.
// Invalid: “null” not allowed as entries in a repeated field.
{
  “repeatedValue”: [1, null, 2]
}

// Invalid: “null” is not allowed to be map key.
{
  “mapValue”: {
    null: 0
  }
}

// Invalid: “null” is not allowed to be map value.
{
  “mapValue”: {
    0: null
  }
}

It's hard for us to change the behavior here since it seems explicitly specified how we're supposed to operate. I think this would need to be an issue filed with the main protobuf project.

This is one of those intentional changes we made in V2, but seems like we missed out on mentioning this in the JSON serialization section.
V2 does not allow for a JSON null in a repeated message item or a map message value except for types google.protobuf.NullValue or google.protobuf.Value.
I wish this is documented in the Protobuf guide.  I've only found a similar issue raised for the Java implementation at https://github.com/protocolbuffers/protobuf/issues/5113.

Given that:

the "official" documentation for protobuf-to-JSON specifies that we reject null in lists and maps,
the Java implementation behaves the same way (I didn't check C++) per protocolbuffers/protobuf#5113, and
the former tech lead of protobufs affirmed such behavior per https://github.com/protocolbuffers/protobuf/issues/5113#issuecomment-419532565,

I'm inclined to close this as working as intended. Any changes to this behavior would have to be proposed to the main protobuf project for all language implementations to support.

the "official" documentation for protobuf-to-JSON specifies that we reject null in lists and maps,

What documentation are you referring to here? I could not find this anywhere on the official doc site. Or did you mean Go doc for the protojson package?

This is an internal document written by the person who designed the protobuf to JSON mapping. I really wish it would be made open, but that's out of my hands.

Closing as "working as intended".

What if I do need null in a list. Any workaround here?

What if I do need null in a list. Any workaround here?

There is unfortunately no analogous concept of NULL or nil in protobuf semantics.
If you absolutely must be able to use a null value in a protobuf JSON, then you’ll need to find someway to integrate with google.protobuf.NullValue likely through the google.protobuf.Value type. Though, I think that will end up a little too generic, and would permit all values to be used, so you would have to do type checking, and for protobuf messages even a struct field mapping transfer.
It is important to recognize the limitations of protobuf’s JSON mapping. It is not intended to permit all possible JSON, it’s solely designed to describe a standard way to map protobuf semantics into JSON safely, so there is a strict subset of all possible JSON  that can be used with protobuf.


Files .ts don't appear

Files .ts don't appear, only .mp4, how i to do for appear .ts on elementum?
On qbittorrent appear 35 folders and +100 files .mp4 and .ts

On Elementum KODI appear only 27 itens

@RafaelGodoyEbert Elementum filters files, based on size.

so you can try to adjust that size in settings.

@RafaelGodoyEbert did it help?

I had found it, just didn't know how to delete my question. Thanks for commenting anyway!


Cross Origin Resource Sharing CORS

Does it support CORS? I don't see any such annotations as found in nginx ingress controller. Thanks.

also see #1005

CORS works. I use ASP.Net Core and all configurations are done in the application and make use of the platform middleware. Test is done using curl. Make sure to set the header when testing. Otherwise there won't be CORS headers in the response.


Won't compile... :(

Doesn't want to compile, all dependencies are in place and up to date.
No errors or warnings before this.
Utils.cpp:57:73: error: cannot convert 'GPU_TEXCOLOR' to 'sf2d_texfmt' for argument '3' to 'sf2d_texture* sf2d_create_texture(int, int, sf2d_texfmt, sf2d_place)'
sf2d_texture* tex = sf2d_create_texture(w, h, GPU_RGBA8, SF2D_PLACE_RAM);
make[1]: *** [Utils.o] Error 1
make: *** [build] Error 2

Fixed :)

Not quite. :P Ran into another issue.
uiFont.cpp:22:29: fatal error: 3DS_UI/ttf/Font.h: No such file or directory
Compiles if i comment that out obviously, but i guess it's there for a reason. :) Or is it?...


[SpotBugs] Set threshold to middle with exceptions

What changes were proposed in this pull request?

Enable middle priority checks in spotbugs.
Fix some violations.
Exclude other violations to make CI pass.

Why are the changes needed?
Enforce stronger code quality check.
See violations fixed in this PR for example.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Run mvn test-compile spotbugs:check manually.

Codecov Report

Merging #517 (3d1f305) into master (6076226) will decrease coverage by 0.49%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #517      +/-   ##
============================================
- Coverage     59.79%   59.31%   -0.49%
+ Complexity     1770     1551     -219
============================================
  Files           205      183      -22
  Lines         11531     9698    -1833
  Branches       1033      874     -159
============================================
- Hits           6895     5752    -1143
+ Misses         4231     3583     -648
+ Partials        405      363      -42

Impacted Files
Coverage Δ

...rg/apache/uniffle/storage/common/LocalStorage.java
48.36% <0.00%> (-1.97%)
:arrow_down:

...mapreduce/task/reduce/RssInMemoryRemoteMerger.java

...org/apache/spark/shuffle/writer/AddBlockEvent.java

.../java/org/apache/spark/shuffle/RssSparkConfig.java

...pache/hadoop/mapreduce/task/reduce/RssFetcher.java

...pache/hadoop/mapreduce/task/reduce/RssShuffle.java

...org/apache/spark/shuffle/RssSparkShuffleUtils.java

.../org/apache/spark/shuffle/writer/WriterBuffer.java

...n/java/org/apache/hadoop/mapreduce/RssMRUtils.java

...pache/spark/shuffle/writer/WriteBufferManager.java

... and 13 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Ref #532


Lots of issues trying LaTeX

Describe the bug
(EDIT: edited to show standard/reproducible JAES template, rather than my specific document.)
context
Hi guys. Saw Steve's post about "Writing a Scientific Paper in MyST Markdown" and Rowan's seminar next week. Since I'm starting two new scientific papers, I decided to try MyST.
Followed the instructions https://curvenote.com/blog/how-to-use-latex-with-myst-markdown, ...downloaded a ZIP of my stuff from Overleaf, using the template files from the Journal of the Audio Engineering society.
Installed myst via npm, ran it in the directory where my .tex file is and....
expectation
I expected to get a wonderful rendering of my scientific document.
bug
But instead lots of errors occur .
/tmp/aes-template$ cp ~/Downloads/JAES_Latex_template.zip .
/tmp/aes-template$ unzip JAES_Latex_template.zip
Archive:  JAES_Latex_template.zip
  inflating: jaes-sample1-bibtex.tex
  inflating: jaes.bib
  inflating: jaes.bst
  inflating: jaes.cls
  inflating: cuted.sty
  inflating: readme.txt
  inflating: jaes-sample2-bibitem.tex
  inflating: jaes-guide.pdf
  inflating: headshot2.jpg
  inflating: headshot1.jpg
  inflating: fig1_example.eps
  inflating: fig2_example.eps
  inflating: jaes-sample1-bibtex.pdf
  inflating: jaes-sample2-bibitem.pdf
/tmp/aes-template$ myst init
💾 Writing new project and site config file: myst.yml
/tmp/aes-template$ myst start
🐕 Fetching template metadata from https://api.myst.tools/templates/site/myst/book-theme
🌎 Cloning template from https://github.com/curvenote/book-theme.git
💾 Saved template to path _build/templates/site/myst/book-theme
⤵  Installing web libraries (can take up to 60 s)
📦 Installed web libraries in 13 s
⛔ jaes-sample1-bibtex.tex:21:1 Unhandled TEX conversion for node of "macro_markboth"
⚠  jaes-sample1-bibtex.tex:21:15 Unknown character
⚠  jaes-sample1-bibtex.tex:21:19 Unknown character
⛔ jaes-sample1-bibtex.tex:28:1 Unhandled TEX conversion for node of "macro_authorgroup"
⚠  jaes-sample1-bibtex.tex:29:19 Unknown character
⚠  jaes-sample1-bibtex.tex:29:23 Unknown character
⛔ jaes-sample1-bibtex.tex:30:1 Unhandled TEX conversion for node of "macro_role"
⛔ jaes-sample1-bibtex.tex:32:1 Unhandled TEX conversion for node of "macro_role"
⛔ jaes-sample1-bibtex.tex:33:35 Unhandled TEX conversion for node of "macro_quad"
⛔ jaes-sample1-bibtex.tex:33:40 Unhandled TEX conversion for node of "macro_quad"
⛔ jaes-sample1-bibtex.tex:33:45 Unhandled TEX conversion for node of "macro_quad"
⛔ jaes-sample1-bibtex.tex:33:50 Unhandled TEX conversion for node of "macro_quad"
⛔ jaes-sample1-bibtex.tex:33:55 Unhandled TEX conversion for node of "macro_quad"
⛔ jaes-sample1-bibtex.tex:33:60 Unhandled TEX conversion for node of "macro_quad"
⛔ jaes-sample1-bibtex.tex:33:65 Unhandled TEX conversion for node of "macro_quad"
⛔ jaes-sample1-bibtex.tex:33:70 Unhandled TEX conversion for node of "macro_quad"
⛔ jaes-sample1-bibtex.tex:33:75 Unhandled TEX conversion for node of "macro_quad"
⛔ jaes-sample1-bibtex.tex:33:80 Unhandled TEX conversion for node of "macro_quad"
⛔ jaes-sample1-bibtex.tex:33:85 Unhandled TEX conversion for node of "macro_quad"
⛔ jaes-sample1-bibtex.tex:39:1 Unhandled TEX conversion for node of "macro_abstract"
⛔ jaes-sample1-bibtex.tex:125:1 Unhandled TEX conversion for node of "macro_tabcolsep"
⛔ jaes-sample1-bibtex.tex:130:50 Unhandled TEX conversion for node of "macro_colrule"
⛔ jaes-sample1-bibtex.tex:140:31 Unhandled TEX conversion for node of "macro_botrule"
⛔ jaes-sample1-bibtex.tex:142:1 Unhandled TEX conversion for node of "env_tabnote"
⚠  jaes-sample1-bibtex.tex:196:160 Unknown character
⚠  jaes-sample1-bibtex.tex:196:166 Unknown character
⛔ jaes-sample1-bibtex.tex:263:2 Unhandled TEX conversion for node of "macro_biography"
⚠  jaes-sample1-bibtex.tex:263:23 Unknown character
⚠  jaes-sample1-bibtex.tex:263:27 Unknown character
⚠  jaes-sample1-bibtex.tex:263:61 Unknown character
⚠  jaes-sample1-bibtex.tex:263:65 Unknown character
⚠  jaes-sample1-bibtex.tex:263:771 Unknown character
⚠  jaes-sample1-bibtex.tex:263:775 Unknown character
⛔ jaes-sample1-bibtex.tex:265:2 Unhandled TEX conversion for node of "macro_biography"
⛔ jaes-sample2-bibitem.tex:21:1 Unhandled TEX conversion for node of "macro_markboth"
⚠  jaes-sample2-bibitem.tex:21:15 Unknown character
⚠  jaes-sample2-bibitem.tex:21:19 Unknown character
⛔ jaes-sample2-bibitem.tex:28:1 Unhandled TEX conversion for node of "macro_authorgroup"
⚠  jaes-sample2-bibitem.tex:29:19 Unknown character
⚠  jaes-sample2-bibitem.tex:29:23 Unknown character
⛔ jaes-sample2-bibitem.tex:30:1 Unhandled TEX conversion for node of "macro_role"
⛔ jaes-sample2-bibitem.tex:32:1 Unhandled TEX conversion for node of "macro_role"
⛔ jaes-sample2-bibitem.tex:33:35 Unhandled TEX conversion for node of "macro_quad"
⛔ jaes-sample2-bibitem.tex:33:40 Unhandled TEX conversion for node of "macro_quad"
⛔ jaes-sample2-bibitem.tex:33:45 Unhandled TEX conversion for node of "macro_quad"
⛔ jaes-sample2-bibitem.tex:33:50 Unhandled TEX conversion for node of "macro_quad"
⛔ jaes-sample2-bibitem.tex:33:55 Unhandled TEX conversion for node of "macro_quad"
⛔ jaes-sample2-bibitem.tex:33:60 Unhandled TEX conversion for node of "macro_quad"
⛔ jaes-sample2-bibitem.tex:33:65 Unhandled TEX conversion for node of "macro_quad"
⛔ jaes-sample2-bibitem.tex:33:70 Unhandled TEX conversion for node of "macro_quad"
⛔ jaes-sample2-bibitem.tex:33:75 Unhandled TEX conversion for node of "macro_quad"
⛔ jaes-sample2-bibitem.tex:33:80 Unhandled TEX conversion for node of "macro_quad"
⛔ jaes-sample2-bibitem.tex:33:85 Unhandled TEX conversion for node of "macro_quad"
⛔ jaes-sample2-bibitem.tex:39:1 Unhandled TEX conversion for node of "macro_abstract"
⛔ jaes-sample2-bibitem.tex:125:1 Unhandled TEX conversion for node of "macro_tabcolsep"
⛔ jaes-sample2-bibitem.tex:130:50 Unhandled TEX conversion for node of "macro_colrule"
⛔ jaes-sample2-bibitem.tex:140:31 Unhandled TEX conversion for node of "macro_botrule"
⛔ jaes-sample2-bibitem.tex:142:1 Unhandled TEX conversion for node of "env_tabnote"
⚠  jaes-sample2-bibitem.tex:196:160 Unknown character
⚠  jaes-sample2-bibitem.tex:196:166 Unknown character
⛔ jaes-sample2-bibitem.tex:205:1 Unhandled TEX conversion for node of "env_thebibliography"
⛔ jaes-sample2-bibitem.tex:263:2 Unhandled TEX conversion for node of "macro_biography"
⚠  jaes-sample2-bibitem.tex:263:23 Unknown character
⚠  jaes-sample2-bibitem.tex:263:27 Unknown character
⚠  jaes-sample2-bibitem.tex:263:61 Unknown character
⚠  jaes-sample2-bibitem.tex:263:65 Unknown character
⚠  jaes-sample2-bibitem.tex:263:771 Unknown character
⚠  jaes-sample2-bibitem.tex:263:775 Unknown character
⛔ jaes-sample2-bibitem.tex:265:2 Unhandled TEX conversion for node of "macro_biography"
⛔ jaes-sample1-bibtex.tex:101:1 Unsupported image extension ".eps" may not correctly render.
⛔ jaes-sample1-bibtex.tex:114:1 Unsupported image extension ".eps" may not correctly render.
⛔ jaes-sample2-bibitem.tex:101:1 Unsupported image extension ".eps" may not correctly render.
⛔ jaes-sample2-bibitem.tex:114:1 Unsupported image extension ".eps" may not correctly render.
📖 Built jaes-sample1-bibtex.tex in 179 ms.
📖 Built jaes-sample2-bibitem.tex in 163 ms.
📚 Built 2 pages for my-myst-site in 368 ms.

	✨✨✨  Starting Curvenote Book Theme  ✨✨✨

⚡ Compiled in 442ms.
The path "myst-transforms" is imported in ../theme-base/packages/myst-to-react/src/myst.tsx but "myst-transforms" was not found in your node_modules. Did you forget to install it?

The path "myst-to-tex" is imported in ../theme-base/packages/myst-to-react/src/myst.tsx but "myst-to-tex" was not found in your node_modules. Did you forget to install it?

The path "myst-to-jats" is imported in ../theme-base/packages/myst-to-react/src/myst.tsx but "myst-to-jats" was not found in your node_modules. Did you forget to install it?

🔌 Server started on port 3000!  🥳 🎉

	👉  http://localhost:3000  👈

💌 GET  / 302 - - 435.368 ms
💌 GET  /my-myst-site 500 - - 39.363 ms

The web server shows this image:

problem
This is a problem for _people doing any kind of scientific publishing because we will need to be producing results in accordance with an approved template from a publisher or conference.    If we have to write two different versions of our document, one for true publishing, and one in Markdown just for... IDK I'm not trying to make a website, I'm trying to write a journal article, ... then user adoption of MyST will be limited.  (i.e. if I should just write the LaTeX version if MyST can't help me, then...I have no reason to use MyST)
Clarification
I did not try to write a document stariting in MyST-markdown, and then try to import the LaTeX templates for the purposes of rendering a PDF.   Such was my original expectation for authoring a scientific paper using MyST markdown.
But then when I read the page about using LaTeX and MyST I got the impression that I was just supposed to read in my .tex files.  Rowan will probably clarify this at the seminar, and perhaps this whole Issue is just resulting from a big misunderstanding on my part!
Reproduce the bug

Make sure you have a version of Latex installed, e.g.

$ which pdflatex
/Library/TeX/texbin/pdflatex

Install myst
Download the JAES Author Latex Template
Create a new directory
cd into the new directory created
copy the JAES template into that directory
unzip the JAES zip file
run myst init
run myst start

List your environment
$ jupyter-book --version
-bash: jupyter-book: command not found

There was no mention of jupyter-book when I followed the MyST installation instructions.  If we need jupyter-book, please update the installation instructions.

I've heavily edited this Issue from its initial version. The initial version showed my specific document, and issues such as myst not finding any of the references defined in my BibTex file.
To make things more reproducible for developers, I've removed much of that and replaced it with what you get if you just try the template documents provided by the journal.
If you want to see my original version, then presumably GitHub provides a way for you to view the edit-history of this Issue.

I went back and installed jupyter-book.  It didn't make any difference to errors reported.  But here's the version info:
$ jupyter-book --version
Jupyter Book      : 0.13.1
External ToC      : 0.2.4
MyST-Parser       : 0.15.2
MyST-NB           : 0.13.2
Sphinx Book Theme : 0.3.3
Jupyter-Cache     : 0.4.3
NbClient          : 0.5.13

Thanks for the feedback, and taking the time to detail and make this reproducible!! There is a ton more to do on the latex parsing side to make this fully usable in all situations, the hard error in the rendering is something that we should fix, and at least you should have seen a partially rendered document that would have been pretty good I think.
As you suspected the currently supported workflow is in the other direction (MyST + Template --> TeX --> PDF):

write a document starting in MyST-markdown, and then use a LaTeX templates

We are madly writing docs and onboard guides!! I have already fixed up those two spurious "was not found in your node_modules" which are a problem with a library upgrade we did and they are not real errors. We also need to do a better job on the differences between JupyterBook and myst.tools, they use the same syntax but completely different engines.
This is really helpful feedback as I start to create the intro slides for next Wednesday. Hope to see you there. :)

MyST + Template --> TeX --> PDF  will serve my needs very well.
Looking forward to Wednesday!

Feel free to close this issue if you like.

I will fix some of the easy things in that template and then close this off. Looking to make progress on the LaTeX parsing side as well, so that at the very least the initial experience doesn't end in an error screen!!

I'm going to close this issue, as much of it was based on misunderstanding.


expect.addSnapshotSerializer() not working as expected

Seems the "serializer" is added globally, maybe because we are not running test files isolatedly?
I haven't dug into the problem, I'll post what I found.

Ah, found this https://github.com/facebook/jest/blob/main/packages/jest-snapshot/src/plugins.ts#L39, so I can backup it and restore after test?

Yeah, just module state. Which works fine in Jest, but caching probably messes up for this runner

Is #47 good? Or is there a better solution?

I think that's good. It might make sense for Jest to expose some sort of "reset serializers" API which reset the array to the default (happy to take a PR)? But with the current API I think your approach is good


Version 0.5.2

<!-- Make sure you have read http://napalm.readthedocs.io/en/latest/contributing/index.html --!>

Coverage increased (+0.1%) to 86.207% when pulling b0ffdcd75f47766086b46c299c20d9613bbc30f5 on rel-0.5.2 into c40cc819d8de6695468869ac45a64a18045930ce on develop.

Did anyone run the unit tests against a real device? I think we should do this, given all the Python3 changes.
I can see if I can get this done, but it probably won't be until Friday.

I did and seemed to work fine, great job : )

Yes, I have also tested it yesterday - forgot to mention.


How to define width of a col?

I've read some solutions in this post here https://github.com/SheetJS/sheetjs/issues/1473 but the !cols seems to not exist here on worksheet, how can I define width for a col?

/** Column Properties Object */
export interface ColInfo {
    /* --- visibility --- */

    /** if true, the column is hidden */
    hidden?: boolean;

    /* --- column width --- */

    /** width in Excel's "Max Digit Width", width*256 is integral */
    width?: number;

    /** width in screen pixels */
    wpx?: number;

    /** width in "characters" */
    wch?: number;

    /** Excel's "Max Digit Width" unit, always integral */
    MDW?: number;
}


Conditions on constant expr not working in stencil

Hello, i just noticed a weird behaviour with condition in stencil:
from numba import stencil
import numpy as np

@stencil
def kernel1(a):
    b = 0
    if(2==0): b=2
    return a[0,0]+b

in_arr = np.arange(100).reshape((10, 10))
print(in_arr)
output_arr = kernel1(in_arr)
print(output_arr)

the output_arr should be identical to in_arr, but it is not, output:
[[ 0  1  2  3  4  5  6  7  8  9]
 [10 11 12 13 14 15 16 17 18 19]
 [20 21 22 23 24 25 26 27 28 29]
 [30 31 32 33 34 35 36 37 38 39]
 [40 41 42 43 44 45 46 47 48 49]
 [50 51 52 53 54 55 56 57 58 59]
 [60 61 62 63 64 65 66 67 68 69]
 [70 71 72 73 74 75 76 77 78 79]
 [80 81 82 83 84 85 86 87 88 89]
 [90 91 92 93 94 95 96 97 98 99]]
[[  2   3   4   5   6   7   8   9  10  11]
 [ 12  13  14  15  16  17  18  19  20  21]
 [ 22  23  24  25  26  27  28  29  30  31]
 [ 32  33  34  35  36  37  38  39  40  41]
 [ 42  43  44  45  46  47  48  49  50  51]
 [ 52  53  54  55  56  57  58  59  60  61]
 [ 62  63  64  65  66  67  68  69  70  71]
 [ 72  73  74  75  76  77  78  79  80  81]
 [ 82  83  84  85  86  87  88  89  90  91]
 [ 92  93  94  95  96  97  98  99 100 101]]

Python:
> python --version
> Python 3.6.5 :: Anaconda, Inc.

Numba:
> numba.__version__
> '0.41.0dev0+94.g361809189'

Also, if you try to put print(2==0) in the stencil it prints True

@ehsantn, @DrTodd13: Any ideas?

operators > < >= <= work fine, the only problem is with == and !=, i tried few combinations and apparently == returns always True, no matter what, while != return always False. Something is definitively wrong in my build.

After a quick look, I think this line is the problem:
https://github.com/numba/numba/blob/c24195c89e200fc36354ef0dd50d762886c5947c/numba/lowering.py#L533
the args are supposedly both ir.UNDEFINED but for some reason they do not evaluate as True under is, however the args are equal by reference to each other. As a guess, might be some re-import based issue from re-entering the pipeline/generating code leading to more than one ir.UNDEFINED instance. Changing the test there to check whether the args are an instance of ir.UndefinedType may well fix.

ir.UndefinedType should be a singleton class but it is not enforced.


[TASK] Support variables and suffix in routes via settings

Adds support for the variables and suffix sub route configuration options
in routes via settings.
Related: FLOW-411

Nice addition! I'll check asap


Genres

we should format the gifs to be centered in their iframes or collapse the iframes onto the borders of the gifs.

LGTM


Fix incoming notification channel id

I'm really sorry.
I made a mistake in my changes. 🙇

@hiennguyen92 please, do something or grant access to another contributor if you don't work on this package

thank you.


:construction_worker: Add CI for Docker build

Adding a Github workflow for automatically building and publishing the docker image to the Github Container Registry.

@tinydjp CC

Hi @koenmtb1 thanks for your contribution, we're considering broaden our container deployment support. We're evaluation the container registry, once decided will review your PR. It should take about one or two days.


Suggestion: mention the new SetLimit function found in the golang.org/x/sync errgroup package in README.md

It looks like the sync/errgroup package now has a SetLimit function. You can see the documentation at https://pkg.go.dev/golang.org/x/sync/errgroup#Group.SetLimit . I believe that this is a fairly recent happening.
I feel like a mention of this new function in the original package in the README.md is probably merited.
There is some difference in the two APIs of the two packages; this one has a WithContextN(ctx, numG, qSize) function whereas that one has just WithContext(ctx) and SetLimit(n) functions. At a glance it seems like the n in the original pretty similar to the numG parameter here, and there doesn't seem to be a qSize parameter in the original package. I definitely have not compared the two packages in detail, or really even attempted to understand the implementation of either one. I am just an end-user.
I used this package some time ago at a previous job (maybe two of them, actually, I can't quite remember) and it was helpful for my purposes, so thank you for that. Ultimately, I'm wondering now if using this package in new code is necessary or recommended, given the new SetLimit function in the original package, and would appreciate hearing any thoughts from more knowledgeable sources.

Done.


Android build

Hello.
I was wondering if build for android will be added to the project?
Thank you

At the moment you can use the PWA version on android - https://iptvnator.vercel.app/


Classes Pedestrian and Cyclist much lower than paper

I uesd released weight on GitHub, KITTI DLA-34 and KITTI V2-99.
And I tried to use both weight to do evaluation on validation set. I could get similer results on Car class, but on other classes are all limited to zero.
I known the TTA and different evaluation sets are cause variance.  But the variance should not  be so much. Are there any missing settings that are causing this issue?
Below table is my evaluate results:

Car AP<EMAIL_ADDRESS>
Paper
KITTI Submit
KITTI DLA
KITTI V2

BEV AP
Easy
30.98
32.35
31.65
40.70

Med
22.56
23.41
24.43
32.04

Hard
20.03
20.42
21.72
28.54

3D AP
Easy
23.22
23.19
22.56
30.38

Med
16.34
16.87
16.98
23.73

Hard
14.20
14.36
14.93
20.88

Pedestrian AP<EMAIL_ADDRESS>
Paper
KITTI Submit
KITTI DLA
KITTI V2

BEV AP
Easy
15.90
18.58
0.04
0.538

Med
10.85
12.51
0.03
0.056

Hard
8.05
10.65
0.01
0.028

3D AP
Easy
13.91
16.64
0.007
0.017

Med
9.30
11.04
0.008
0.018

Hard
8.05
9.38
0.009
0.019

Cyclist AP<EMAIL_ADDRESS>
Paper
KITTI Submit
KITTI DLA
KITTI V2

BEV AP
Easy
3.20
9.20
0.44
0.447

Med
1.99
5.69
0.27
0.229

Hard
1.79
5.20
0.28
0.258

3D AP
Easy
2.39
7.52
0.13
0.126

Med
1.52
4.79
0.12
0.123

Hard
1.31
4.22
0.11
0.111

And I have another question about published results, why it has differece between arxiv paper and kitti submit.
Thank you so much.

Hello.
Your results for the pedestrian class and cyclist class are based on IOU=0.7. However, KITTI evaluates pedestrian and cyclist based on IOU=0.5 (While the car class is evaluated on IOU=0.7)
You can find the evaluation criterion in http://www.cvlibs.net/datasets/kitti/eval_object.php?obj_benchmark=3d

Very thank for your reply, it's my mistake.


Make navigation bar work with Python symbols

This issue has been moved from a ticket on Developer Community.

The navigation bar is invaluable for C++ and C# projects. It is very useful to be able to see the list of projects, scopes, and symbols (callable unit, class, field, etc.) available, and to be able to navigate to those symbols by using a few dropdown menus.
This would be incredibly useful for Python as well.
Curiously, GitHub (which is a Microsoft service/product, and is tied closely to Visual Studio) has a symbol navigator that works with Python source code files.
Here is a video that demonstrates the current behaviour of the navigation bar with Python source code files.

Original Comments
Feedback Bot on 6/13/2023, 10:31 PM:
(private comment, text removed)

Hi there, thanks for bringing up the issue. We have added this to our backlog as an enhancement and is tracked here: https://github.com/microsoft/PTVS/issues/6551. Feel free to please feel free to share any additional thoughts or ideas to the ticket. Thank you! Closing as duplicate of https://github.com/microsoft/PTVS/issues/6551


The "path" argument must be of type string. Received type undefined

Nuxt project produce the following error on development mode.

@nuxtjs/stylelint-module: v4.0.0

Which rule, if any, is the bug related to?

default/no rules needed

What code is needed to reproduce the bug?

can't detect from where the error originated.

What stylelint configuration is needed to reproduce the bug?

default config

Which version of stylelint are you using?

v13.5.0

How are you running stylelint: CLI, PostCSS plugin, Node.js API?

"*.{scss,css,vue}": "stylelint"
lint-staged on pre-commit along with nuxt-stylelint-module

Does the bug relate to non-standard syntax (e.g. SCSS, Less etc.)?

not sure

What did you expect to happen?

No stylelint error

What actually happened (e.g. what warnings or errors did you get)?

e.g. "The following warnings were flagged:"
Nuxt project produce the following error on development mode.
ERROR StylelintError

The "path" argument must be of type string. Received type undefined

Hi @liyasthomas
Could you provide a repository to reproduce the problem?

Mine is a private repo. Let double check whether it has an issue with any of my imports or configs or else I'll try to reproduce the issue in a bare clone.

Fresh clone of repo couldn't reproduce the issue.


node-template: remove redundant types from runtime

Removes BlockId, SignedBlock and CheckedExtrinsic as they are unused within the runtime currently and the BlockId was defined twice.

To add on this, this was flagged by the https://github.com/est31/warnalyzer tool, along with some other stuff:
node/src/cli.rs:16:6: unused Field '0'
node/src/cli.rs:18:12: unused Field '0'
node/src/cli.rs:21:13: unused Field '0'
node/src/cli.rs:24:15: unused Field '0'
node/src/cli.rs:27:14: unused Field '0'
node/src/cli.rs:30:15: unused Field '0'
node/src/cli.rs:33:13: unused Field '0'
node/src/cli.rs:36:9: unused Field '0'
node/src/cli.rs:40:12: unused Field '0'
node/src/main.rs:1:1: unused Mod ''
node/src/main.rs:11:4: unused Function 'main'
pallets/template/src/lib.rs:32:20: unused Type 'T'
pallets/template/src/lib.rs:40:21: unused Type 'T'
pallets/template/src/lib.rs:50:19: unused Field '0'
pallets/template/src/lib.rs:50:24: unused Field '1'
pallets/template/src/lib.rs:55:17: unused Type 'T'
pallets/template/src/lib.rs:73:56: unused Field '0'
runtime/src/lib.rs:57:10: unused Type 'AccountIndex'
runtime/src/lib.rs:69:10: unused Type 'DigestItem'
runtime/src/lib.rs:85:11: unused Type 'BlockId'
runtime/src/lib.rs:127:11: unused Const 'DAYS'
runtime/src/lib.rs:304:10: unused Type 'SignedBlock'
runtime/src/lib.rs:306:10: unused Type 'BlockId'
runtime/src/lib.rs:320:10: unused Type 'CheckedExtrinsic'

The AccountIndex and DigestItem are already proposed to be removed in https://github.com/paritytech/substrate/pull/9149.

bot merge
ty

@disconnect3d please merge master

ping @disconnect3d

bot merge


DAWN-394 ⁃ Port slow tests

➤ Andrianto Lie commented:
After discussed with Bart today, this unit test is obsolete. Let's make a PR to remove this file

➤ Andrianto Lie commented:
PR Made https://github.com/EOSIO/eos/pull/1664

➤ Andrianto Lie commented:
This task is now handled by  #1639

I remember this is obsolete after discussed with Bart & Andri. Can we close this?

It does appear that #1639 obsoleted this


Poor doc

"Make sure that the project refers to the correct mscorlib.dll."
How then? I am missing important information.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d345127b-8891-06b6-ac48-5aaee3181e4d
Version Independent ID: 1837c5b3-d5bb-7937-5a8d-b157df244424
Content: Compiler Error CS0518
Content Source: docs/csharp/language-reference/compiler-messages/cs0518.md
Product: dotnet-csharp
Technology: csharp-diagnostics
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks @GitHubRTim
This page should get updated to reflect the new project system. Selecting the correct runtime in the csproj file should address this issue.


Demo is not working

Demo is not working.

They all seem to be running from what I can see

Demo link

Try this -> http://mharris717.github.io/ember-drag-drop/#/horizontal

That works perfect. Thanks.
The links in the README are missing the #.

Ah, great I'll update that, thanks.


[SPARK-32180][PYTHON][DOCS][FOLLOW-UP] Fix Python lint issue

What changes were proposed in this pull request?

Seems #29640 introduced a Python lint error:
Warning, treated as error:
/home/runner/work/spark/spark/python/docs/source/getting_started/installation.rst:20:Duplicate explicit target name: "official release channel".
Makefile:20: recipe for target 'html' failed
make: *** [html] Error 2

This tries to fix it.
Does this PR introduce any user-facing change?

No
How was this patch tested?

N/A

cc @srowen

Test build #128583 has started for PR 29733 at commit e32f85f.

Merged build finished. Test PASSed.

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

Test build #128583 has finished for PR 29733 at commit e32f85f.

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

Merged build finished. Test PASSed.

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


iframe内嵌superset时，superset页面中的“api/v1/dashboard/7/filter_state?”接口抛异常“400 Bad Request: The CSRF session token is missing”

When the iframe is embedded with a superset, it can succeed. However, the "API/v1/dashboard/7/" interface in the superset page reported the following error

May I ask how to solve it? Is there a solution without closing CSRF?

Possible duplicate of https://github.com/apache/superset/issues/24717.

Is there any solution without closing CSRF?

It seems you are trying to get the embedded dashboard to work. You could use the superset API to get the CSRF token which subsequently you need to call the guest_token endpoint to authenticate.
Wrote a guide on it, hopefully, it helps someone 😄 Choose your preferred media 😝
https://huamichaelchen.substack.com/p/end-to-end-example-of-setting-up
https://medium.com/@huamichaelchen/end-to-end-example-of-setting-up-superset-embedded-dashboard-f72fc985559

Hi there, and sorry this has gone quiet for so long. We're no longer supporting Superset 2.x or prior, and it's been a while since this thread saw any activity, so I'm closing this as stale. If it is still an issue in Superset 3.x, we can re-open this, or feel free to open a new issue with updated context.


[Exception: Error: The 'responseText' property is only available if 'responseType' is set to '' or 'text', but it is 'blob'. at XMLHttpRequest.get ... at XMLHttpRequest.remoteFunction]

[x] Review the documentation: https://facebook.github.io/react-native
[x] Search for existing issues: https://github.com/facebook/react-native/issues
[ ] Use the latest React Native release: https://github.com/facebook/react-native/releases

Environment
Environment:
  OS: macOS 10.14
  Node: 10.4.1
  Yarn: 1.7.0
  npm: 6.1.0
  Watchman: 4.9.0
  Xcode: Xcode 10.0 Build version 10L201y
  Android Studio: 3.1 AI-173.4819257

Packages: (wanted => installed)
  react: 16.3.1 => 16.3.1
  react-native: 0.55.4 => 0.55.4

Description
I'm using fetch in order to call my local Rails app backend from my iPhone as the following :
    fetch('https://api-loc.myapp.co:3000/api/bootstrap').then(
      (response) => {
        console.log('[BootstrapApi] fetch("https://api-loc.myapp.co:3000/api/bootstrap") response', response)
      }
    ).catch(
      (error) => {
        console.log('[BootstrapApi] fetch("https://api-loc.myapp.co:3000/api/bootstrap") error', error)
      }
    )

but it is always failing with the following :
TypeError: Network request failed
    at XMLHttpRequest.xhr.onerror (fetch.js:442)
    at XMLHttpRequest.dispatchEvent (event-target.js:172)
    at XMLHttpRequest.setReadyState (XMLHttpRequest.js:567)
    at XMLHttpRequest.__didCompleteResponse (XMLHttpRequest.js:397)
    at XMLHttpRequest.js:503
    at RCTDeviceEventEmitter.emit (EventEmitter.js:179)
    at MessageQueue.__callFunction (MessageQueue.js:351)
    at MessageQueue.js:116
    at MessageQueue.__guardSafe (MessageQueue.js:314)
    at MessageQueue.callFunctionReturnFlushedQueue (MessageQueue.js:115)

I have added a console.log of the xhr object from the whatwg-fetch package, from fetch.js:442 , and it shows the following responseText:
[Exception: Error: The 'responseText' property is only available if 'responseType' is set to '' or 'text', but it is 'blob'. at XMLHttpRequest.get (blob:http://<IP_ADDRESS>:8081/a12ab84a-c637-4b31-883a-2b94fddce1f9:17731:17) at XMLHttpRequest.remoteFunction (<anonymous>:2:14)]

I tried exactly the same but with https://www.google.com and no error, expected 200 response received.
In order to avoid name resolution issues (I have a Dnsmasq instance on my Mac, and my iPhone is using my Mac as a DNS resolver), I tried with my IP address but same result.
Also in order to avoid SSL certificate issues (I have a self-sign for the development environment), I'm running an Nginx instance, accepting HTTP requests, redirecting to HTTPS, but still same error.

I forgot to tel that accessing my web app using Safari is working fine, the error is only with react native.

I just tried also with the iOS and Android simulators (which shouldn't have so much problems to reach the app as the real device) but exactly the same error.

I tried to remove the SSL mode of my backend app, and allow insecure connection, in the info.plist file of my app from Xcode, and I was able to reach the server.
So it looks like there is an issue with the SSL/TLS certificate.

Okay so I think I figured out what is this error The 'responseText' property is only available if 'responseType' is set to '' or 'text', but it is 'blob'. : This error didn't occur while my app was running, it occurs when I'm clicking the {...} in front of the responseText attribute of the xhr variable which I printed in Chrome's console using console.log.
So there is a real Network issue, which I don't know the real message. 😞

So updating the var support = {...} from whatwg-fetch in order to exclude the Blob allowed me to get the _response: An SSL error has occurred and a secure connection to the server cannot be made. 👍
Now I need to figure out why do I have this error as I'm using a Let's Encrypt SSL certificate, working well in Safari on the iPhone, without installation it before.

In this end, I made it so that my backend app acceptes non-HTTPS requests, and the mobile app to allow sending non-HTTPS request 🤦♂ 😢.


Ability to use previous answer in next calculation

Could be stored as a constant such as "ans". Would be hugely useful.

Thanks! I opened a PR (referenced above), all that's left is to update the documentation, which I'll get around to probably tomorrow.

Done!

Thanks alfred, great work!


Copy-paste keyboard hotkeys into and out of app UI

The Issue
Cmd-C, Cmd-V and similar do not work inside app. Right clicking to copy-paste still works.
Steps to reproduce

Expected behaviour
Tell us what should happen
Actual behaviour
Tell us what happens instead
System Configuration

LBRY Daemon version:
LBRY App version:
LBRY Installation ID:
Operating system:

Anything Else

Screenshots

I couldn't exactly reproduce it, could you be more specific as to what you couldn't copy/paste in app. I tried to copy a random snippet from a video's description and paste it in the search bar, and it worked. I tried on Linux based OS.

El Capitan 10.11.6, cmd-C and cmd-V do nothing going into or out of the app. For example, non-responsive when I want to copy-paste a URL from LBRY, or a paste a description into the app while publishing.

I believe it's Mac that's affected only. Tested it on Windows and Linux

@btzr-io any idea on this one? thought it might be related to user-select, but idk

I can't recreate this in linux, but I'll do some research...

Not related to user-select:
http://codedmaster.com/2017/01/24/getting-electron-copy-paste-shortcuts-for-mac-builds/

These are back in the menu, so this should be fixed.


Problems with issuing an invitation

I am creating a test agent, authorising my software through OAuth for sending an invitation then sending an invitation, but getting a "session record not found" 401 error which I presume is the SessionRecordNotFound exception.   What should I do?
Moreso in the sense of who should I speak to. I have emailed SDST (at their new address) and they have said they cannot recreate the error.

Hi @johnhemming can you please tells us the steps you made? so we can debug on our side

I have a test bed that creates a new agent, then authorises my software using oauth (which it does for the scope sent-invitations ) and then runs:
https://test-api.service.hmrc.gov.uk/agents/GARN0313812/invitations
With a JSON string of:
{"service": ["MTD-IT"], "clientType":"personal", "clientIdType": "ni", "clientId": "SW724953C", "knownFact": "RM85 0VL"	}
I have just run this.  I would be happy to give you the IP address from which this is running and I could allow you access to my test bed so you can run it (although it has to be switched on), but I don't want to do that via github.  Best to email me and establish a direct email contact.
I have just run it (twice actually, the first time oauth failed, the second time it worked).

I got this response:
401 Unauthorized {"statusCode":401,"message":"Session record not found"} 401 error message

Hi @johnhemming apologises for late reply. what is the best email to use to contact you? We will investigate and get back to you as soon as possible.

Also I cannot help wondering. Is this an issue for the Agent Authorization Api: https://github.com/hmrc/agent-authorisation-api? not this particular repo?

You might be right about that.   It relates to agents authorisations.   My email is<EMAIL_ADDRESS>anyway.

However, it is running the invitations rest method so It might be here as well.

I find a reference to it here:
https://github.com/hmrc/agent-subscription-frontend/blob/master/it/uk/gov/hmrc/agentsubscriptionfrontend/stubs/AuthStub.scala

Hi @johnhemming, we will continue this issue via email

This was my error and I thank Paul for assisting me and resolving it.


How to create proper authentication for kubernetes

I used helm to deploy grafana, but I cannot get it to connect to the kubernetes api. I cannot find instructions on how to create a cluserrole for grafana.

For example, where can I put the kubernetes auth token?

For example, where can I put the kubernetes token?

I'm wondering why this needs access to the cluster at all. All this data is accessible in Prometheus.

Maybe just to deploy the required kubernetes resources?

The Deploy button will deploy the following: (1) A promtheus configmap which contains the prometheus jobs that collect metrics used by the dashboards in the kubernetes app (2) a Node Exporter deployment, and (3) a Kube-State Metrics deployment

this is the solution to my issue:
https://github.com/grafana/kubernetes-app/issues/46#issuecomment-435806954


Update UoC Logo, remove from git repo

Update to default to the new University of Crete logo, and remove old from git. The current one is linked from CSD UoC's website, as the image is not licensed as the rest of the repo.

Thanks @DaKnOb


kodi: update osmc theme

Motivation for this change
With the recent v18 update merged, the nixpkgs' version of the OSMC skin plugin is no longer compatible. Since the "skin.osmc" repo doesn't do versioning, I just updated it to master's latest commit.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Forgot to mention that a commit msg that matches the contribution guidelines would be
kodiPlugins.osmc-skin: 17.0.4 -> 18.0.0

https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md#submitting-changes

I'd say this commit osmc/skin.osmc@40a6c31 would be 18.0.0

Good catch! Thank you
Sorry about commit title, I wasn't sure how that would apply to a plugin

Sorry about commit title, I wasn't sure how that would apply to a plugin

If you're ever unsure which attribute to use in the commit msg, the git history usually can reveal this pretty quick.
Just going to test that this builds now
@GrahamcOfBorg build kodiPlugins.osmc-skin

Thanks @minijackson :sparkle:


iOS 11 - HomeBridge not working remotely

Hi,
Since upgrading to iOS 11 I have been unable to operate my Fibaro devices remotely through the home app. I have deleted and re-added my devices in the home app a few times with no result. when I open the home app it shows my home hub as connected however nothing seems to update in the home app itself.
I am not sure of what to try next can anyone assist with this issue?

Guys it turned out to be resulting from my Raspberry Pi being connected to the lan by cable and Wifi. once I turned WIFI off everything worked perfectly.


Validation error when recursing subpattern in the regex pattern

I am trying to use the following pattern to match an IP address:
"pattern": "^((25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\\.){3}(?2)$"
But I get the following error.

jsen.js:200 Uncaught SyntaxError: Invalid regular expression: /^((25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?).){3}(?2)$/: Invalid group

I know the regex works as it is tested at https://regex101.com/r/yY6cW0/1 .
The following works, but I want to recurse the second subpattern to make it shorter.
"pattern": "^((25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\\.){3}(25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)$"

This regular expression is valid in PHP, but invalid in JavaScript. The statements below will fail both in the browser and in node:
var reg1 = new RegExp("^...$"); // your regex pattern
var reg2 = /^...$/              // your regex pattern


Feature Request: Support for Instapaper

If it's not totally out of scope: Please consider adding support for Instapaper. In theory only the API-Calls have to be looked at. API-Documentation can be found here: https://www.instapaper.com/api/full
P. S.: I was unable to add the enhancement label to my issue.

I never considered adding Instapaper functionality to obsidian-pocket, but after taking a quick look at the API, I'd say that the underlying API structure and authentication mechanisms are different enough from Pocket API that it would be better for there to be a separate obsidian-instapaper.
One example is that it looks like you can only get the set of unread bookmarks in Instapaper, whereas for now, we fetch all bookmarks (unread and archived) from Pocket, and will probably in the future (bandwidth and PRs from others permitting) add filtering.. so that seems different enough.
Plus auth looks pretty different too (supplying password as a parameter in Instapaper, vs getting an access token in Pocket).
OTOH, obsidian-pocket is under an MIT license, which makes it easy for whoever builds obsidian-instapaper to copy over what's common, like the data storage layer, the React-based UI, etc. Even better would be if some of those mechanisms could be abstracted into a library that is used in common between both plugins.
Anyway, I'm essentially saying that Instapaper is out of scope for obsidian-pocket, but I'd be quite happy if some of the code here was reused in an obsidian-instapaper plugin.


Update config.json to include security scheme values

Now that we can interpolate environment variables, we decided that config.json should house the security scheme values, to be outsourced to secrets.env if the user wants. @amarthadan suggested that these should go under OIS, but I disagree because it is not relevant to specifying an API or its integration at all. The alternative is putting it in config.json, which would look similar to https://docs.api3.org/next/reference/deployment-files/config-json.html#environment

This was already implemented in https://github.com/api3dao/airnode/pull/439. The credentials (an API key for example) is read from ois.credentials section. I'd say it has to be at the OIS level because we can have multiple OISes in the config.json (it's an array).
How it works now is that you can have multiple scheme definitions in ois.components.securitySchemes (following the OAS) and selecting one with ois.credentials section. The reason I designed it this way is that in the future I would like to follow the OAS example and allow one global credentials (what's currently in ois.credentials section) and allow local (per path) overrides. OAS is allowing the same thing when you want to set up for example one auth method for the whole API except for one path.
I'm open to suggestions, but this felt familiar enough to OAS so I followed thair example.

My concern is that OIS is very similar to OAS in purpose (specifies an interface), so it's not appropriate for it to include security scheme values for the same reason. Also consider the case where the security scheme values for multiple OISes are defined as ${API_KEY}.  We already support multiple OISes by having an oisTitle field (assuming unique OIS titles, which is probably not checked for).
We explicitly don't support path-specific security schemes, mostly because it has never been needed and will create confusion.

I don't think the similarity with OAS is a problem here. Also, the similar part is basically just the apiSpecifications section. We're already adding endpoints section anyway so I don't see a problem in adding credentials section as well.
As for multiple OISes and their API key variables they would have to be named separately, something like ${API_KEY_OIS_X} and ${API_KEY_OIS_Y}. But that would have to be done anyway, unless you want to use the same API key for multiple OISes (which you still can).
I don't really like the same setup as the environment field was. The point of the variable interpolation was (partly) that we can get rid of this arbitrary matching and we can just add fields where we need them. So, credentials for specific OIS should IMO go under the ois.
We can discuss it on the Airnode call.

Closing this issue to be tracked by https://api3dao.atlassian.net/jira/software/projects/AN/boards/18/backlog?selectedIssue=AN-72


Master

@007Aterix007 Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


ESP32-S2 / PiggyMeter support for UH50

The PiggyMeter (https://aquaticus.info/meter.html) kit ships with ESP32-S2 chips, but the software (https://github.com/aquaticus/piggymeter_esphome_yaml) doesn't work with the UH50 / T550 meter, and neither with this project. However I think this project is easier to adapt than adding UH50 decoding to the other project (including baud separation).
After troubleshooting, enabling serial debugging, and printing the available() bytes to read (that are actually not being read from the buffer), my conclusion is the reason is the Serial.readBytesUntil() at https://github.com/psvanstrom/esphome-uh50reader/blob/b26e0121ad7d971e74c97ef91267655b19889e18/uh50reader.h#L138
Checking Serial.available(), there is no data in the buffer as available() reports, which explains why reading from Serial does not yield data.
Instead of using Serial.read functions, just using read() from UARTDevice (https://esphome.io/api/classesphome_1_1uart_1_1_u_a_r_t_device#af6640e87989a465b2fac209d932f1c40) produces reliable results.
Most likely this is because of pins GPIO11 and GPIO12 is used for the UART on the ESP32-S2 + PiggyMeter, not mapped to Serial communications.
So I'm currently trying to replace Serial.read with UARTDevice reads, as well as implementing support for all meter sensors (ca 1 kB telegram is delivered from the meter).
I assume reading from UARTDevice will also work for the other/all ESP-chips, so it'll be a more generic implementation. Just like the sendDataCmd is using uart_out.write, not Serial write.

See PR https://github.com/psvanstrom/esphome-uh50reader/pull/6

PR accepted, thanks!


Examples needs to be updated

The method 'setWebContentsDebuggingEnabled' isn't defined for the type 'InAppWebViewController'.
Try correcting the name to the name of an existing method, or defining a method named 'setWebContentsDebuggingEnabled'.
The method 'setSettings' isn't defined for the type 'InAppWebViewController'.
Try correcting the name to the name of an existing method, or defining a method named 'setSettings'.
The method 'InAppWebViewSettings' isn't defined for the type '_MyAppState'.
Try correcting the name to the name of an existing method, or defining a method named 'InAppWebViewSettings'.

@pichillilorenzo

@paulpoyyayil make sure your pubspec.yaml used 6.0.0-beta.22 because by default it install the latest stable 5.7.2+3.
This repository use the prerelease version that's why you getting error since the one you are using is the older version.
https://inappwebview.dev/docs/migration-guide/
https://github.com/pichillilorenzo/flutter_inappwebview_examples/blob/main/third_party_oauth_user_sign_in/pubspec.yaml


HealthKit cannot initialize

after installing the library and calling AppleHealthKit.initHealthKit
I get the following error:

When I inspect the AppleHealthKit imported object, there is just "constants" and "permissions" properties, no methods.

Looks like a linking issue. Did you run react-native link rn-apple-healthkit, or use cocoapods?

I think you forgot this step:
4. In XCode, in the project navigator, select your project. Add libRCTAppleHealthkit.a to your project's Build Phases ➜ Link Binary With Libraries

seems to be resolved, since no more responses


Failed to sudo bundle install in vagrant

I managed to run vagrant ssh and bundle install but sudo bundle install will result in this following log below. Furthermore, if I try to start bundle exec rake warden:start as instructed, I get a different error saying that I need to Operation not permitted - setrlimit and if I sudo bundle exec rake warden:start... I will get an error that I need to bundle install first, which even if I had already done so.
I am able to wget https and http without any issue, git clone the respective links has no issue either. It seemed only concerning warden start.
Please advise.
Fetching https://github.com/cloudfoundry/common.git
fatal: unable to access 'https://github.com/cloudfoundry/common.git/': Failed to connect to github.com port 443: Connection timed out

Retrying git clone 'https://github.com/cloudfoundry/common.git' "/opt/rubies/ruby-2.1.6/lib/ruby/gems/2.1.0/cache/bundler/git/common-4cd969274bc7dedaa8c10ef1f1a48c5e72eb21ad" --bare --no-hardlinks --quiet due to error (2/4): Bundler::Source::Git::GitCommandError Git error: command `git clone 'https://github.com/cloudfoundry/common.git' "/opt/rubies/ruby-2.1.6/lib/ruby/gems/2.1.0/cache/bundler/git/common-4cd969274bc7dedaa8c10ef1f1a48c5e72eb21ad" --bare --no-hardlinks --quiet` in directory /warden/warden has failed.

Hi @kennetham,
Our documentation specifies that you must bundle install as root (via sudo) since the rake warden:start task needs to be executed as a privileged user. Since gems are installed per user in order to run bundle exec as root successfully you must have first bundle installed as root.
That being said I'm not sure what is causing your issue. It looks like a network or DNS error. I'd recommend destroying your vagrant box and restarting per the instructions in the readme. I'd also attempt to correlate any github outages with the timing of your attempts to run these tests.
Please keep us informed
@zaksoup and @wendorf CF Runtime Team

Hi @zaksoup @wendorf
Yes I did followed the documentation very closely and did the following but it did not work inside vagrant.
I have also destroyed the vagrant box this morning and have run it once more, I am assuming it could be likely due to company's proxy settings or so. I am going to run that on my personal machine and see if it works, it sounds like it could be an internal security block.
Thank you for the quick response.

Hi @zaksoup @wendorf
Running as root privileges on another machine and all went successfully but when I tried to bundle exec foreman start as root, this is the error log I am receiving.
10:33:25 directory_server.1 | started with pid 16102
10:33:25 nats.1             | started with pid 16104
10:33:25 directory_server.1 | src/common/logger.go:4:2: no buildable Go source files in /root/dea_ng/go/src/github.com/cloudfoundry/gosteno
10:33:25 directory_server.1 | src/directoryserver/stream_handler.go:10:2: no buildable Go source files in /root/dea_ng/go/src/github.com/howeyc/fsnotify
10:33:25 directory_server.1 | rake aborted!
10:33:25 directory_server.1 | Installation failed
10:33:25 directory_server.1 | /root/dea_ng/lib/tasks/dir_server.rb:16:in `block (2 levels) in <top (required)>'
10:33:25 directory_server.1 | Tasks: TOP => dir_server:run => dir_server:install
10:33:25 directory_server.1 | (See full trace by running task with --trace)
10:33:25 directory_server.1 | exited with code 1
10:33:25 system             | sending SIGTERM to all processes
SIGTERM received
10:33:25 nats.1             | ["Starting nats-server version 0.5.0.beta.16 on port 4222"]
10:33:25 nats.1             |
10:33:25 nats.1             | ["Server exiting.."]
10:33:25 nats.1             | exited with code 0

Hi @zaksoup @wendorf
Could you please advise me why DEA is unreachable?
12:38:58 directory_server.1 | started with pid 4304
12:38:58 nats.1             | started with pid 4306
12:39:01 directory_server.1 | {"timestamp":1433475541.769407511,"process_id":4329,"source":"directory_server","log_level":"info","message":"Starting HTTP server at host: <IP_ADDRESS> on port: 80","data":null}
12:39:35 directory_server.1 | {"timestamp":1433475575.044678211,"process_id":4329,"source":"directory_server","log_level":"info","message":"Sending HTTP request to DEA: http://<IP_ADDRESS>:1234/","data":null}
12:39:35 directory_server.1 | {"timestamp":1433475575.045011759,"process_id":4329,"source":"directory_server","log_level":"warn","message":"Error in LookupPath: DEA is unreachable","data":null}
12:39:35 directory_server.1 | {"timestamp":1433475575.062807560,"process_id":4329,"source":"directory_server","log_level":"info","message":"Sending HTTP request to DEA: http://<IP_ADDRESS>:1234/favicon.ico","data":null}
12:39:35 directory_server.1 | {"timestamp":1433475575.062947273,"process_id":4329,"source":"directory_server","log_level":"warn","message":"Error in LookupPath: DEA is unreachable","data":null}
12:41:04 directory_server.1 | {"timestamp":1433475664.435126305,"process_id":4329,"source":"directory_server","log_level":"info","message":"Sending HTTP request to DEA: http://<IP_ADDRESS>:1234/","data":null}
12:41:04 directory_server.1 | {"timestamp":1433475664.435354471,"process_id":4329,"source":"directory_server","log_level":"warn","message":"Error in LookupPath: DEA is unreachable","data":null}
12:41:04 directory_server.1 | {"timestamp":1433475664.448570013,"process_id":4329,"source":"directory_server","log_level":"info","message":"Sending HTTP request to DEA: http://<IP_ADDRESS>:1234/favicon.ico","data":null}
12:41:04 directory_server.1 | {"timestamp":1433475664.448713779,"process_id":4329,"source":"directory_server","log_level":"warn","message":"Error in LookupPath: DEA is unreachable","data":null}

Hi @kennetham
Where are you seeing these messages? It would be helpful to understand what you're trying to achieve. I'm not sure why the url .../favicon.ico is trying to be reached.
Thanks,
@zaksoup + @DanLavine CF Runtime Team

Hi @zaksoup
These messages are within DEA when I run the command for foreman to start. I tried to create a container and reach the server but this was the message that was thrown.
Running DEA itself, how would you recommend to connect and create a container to test it's integration?
Thanks!

It looks like you are trying to access the directory server from a browser? The directory server does not provide a web interface, it is an http api for getting a file from a container using a signed url.
To get a valid signed url, the normal way is to send a message over the nats message bus to ask for it.
We're not quite sure what you are trying to accomplish. There are many pieces the DEA uses to create a container for a process. Are you trying to test the full integration of sending a start command to the DEA, having it fetch a droplet and then execute that droplet in a container?

@jpalermo Ah I get you. Yes I was trying to test DEA component as it is and understand the flow. I realized that Cloud Foundry's documentation may look complete but there aren't good examples to reference to, and to understand better. For instance, deploy BOSH on OpenStack, the information are provided as is, and the challenge is figuring out what next. I have tried mailing list, but seems like I didn't get any response as well.
IRC Channels are quiet too. Any recommendations to who and how I should get help from especially with Cloud Foundry?
Please advise.

The lists are probably your best bet. Due to the move recently, we've had a period where it was not being watched as closely as it should have been, but it is better now.
It may be difficult to get a response from the dev team or the community on DEA and Warden issues though. Since both are slated to be replaced by Diego and Garden, there are no developers working on them full time. Both are also complex enough that there has never been much community involvement in either of them.


add remark field to NodeData and TaskInfo

Description
add remark field to NodeData and TaskInfo
Issues Resolved
[#6552]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[ ] 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.

just want to get the remark info in first level of data, then user can get it without analyze the key in attributes

@saratvemulapalli just want to get the remark info in first level of data, then user can get it without analyze the key in attributes

Thanks @XiaochenNan. I am not really seeing value of adding a new field but I would like to hear from other maintainers @anasalkouz @reta @andrross @dblock @nknize

@XiaochenNan Can you explain the problem this solution is trying to solve? "user may need set more additional infomation in NodeData and TaskInfo data" from the linked issue isn't nearly detailed enough. What is the problem that leads to a user needing addition information in NodeData and TaskInfo? Once the problem is clear we can start evaluating solutions.

Apologies. This PR was auto closed without reaching a resolution from the maintainers.
Re-opening to move it forward.
Thanks for your contributions to OpenSearch!

@XiaochenNan - Is this being worked upon? Please reach out to the maintainers and other active members to reach conclusion or reviews.

Hi @XiaochenNan, do we have any updates on this PR?

I'm closing this due to no response to the questions above. @XiaochenNan feel free to respond to the feedback and reopen if appropriate. Thanks!


[feature request] Support for local pdf generation from Print Friendly

Hello,
Would it be possible to support a post query from the "Print Friendly & PDF" browser plugin to generate locally the PDF version of a web page ?
This is a plugin for internet browsers, which give the possibility to review and remove part of a web page, which is useful for ads and the likes, before printing it as a pdf file.
The result is very useful for archiving, but suffer from a major problem : the review and edition is locally done in the browser, but the data is sent to print friendly to send back the pdf file.
While a local print only is available , it is using the browser integrated tools which has a lesser result (lack of compression, margins moving at random, ...)
It seems to me this would be possible, as the browser plugin sends the full page in its final state as html in a POST query.
Combined with using a local dns record to redirect the plugin to Stirling.

No browser plugin support sorry


Collapse button position

It looks ok for logged in users (after "reply") but for an anonymous reader it looks odd as we don't have "reply" at all and this will be an empty string with the icon

Button was moved to the top right corner in 5bd91f9.


add '余小磊' to Contributors list

add '余小磊' to Contributors list

@yuxiaolei1989, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?


Ability to Collect Cluster and Node info level metrics

In post analysis of results, using a metrics store, it can be useful to have reference values of the test clusters topology and configuration e.g. for calculations we sometimes need the configured indexing threads. This information is available via APIs e.g. the node info API. Whilst we could implement this in a custom runner, these statistics could be collected by a core Rally feature.
For example, we could collect as a telemetry device or as additional statistics on the rally-race documents. TBD.
@danielmitterdorfer

The existing telemetry device ClusterEnvironmentInfo already extracts data from node stats and attaches it as meta-info to metrics documents. There are two different levels: cluster and node and only cluster-level meta-data are considered for request metrics (as these are always cluster wide and not apply to a single node). So, if we need additional data we can extract it there.
If we need selected meta-data also in rally-results documents we need to amend GlobalStatsCalculator which calculates summary metrics for requests. Similarly, SystemStatsCalculator does the same per node but I don't think we need this there. One complication that I noticed is that we currently put different kinds of properties below meta and we should not copy all of them to summary metrics.

To be clear node info not node stats.

Ah, thanks for the clarification. In any case, I think this should still be implemented in ClusterEnvironmentInfo.


Jest test fails when environment is configured with webpack-hot-middleware

So I'm running a very basic Jest test to test if one of my routes returns a 200.
import request from 'supertest';
import app from './../app';

describe('GET /', () => {
  it('should render properly', async () => {
    await request(app).get('/api/testing').expect(200);
  });
});

This was working perfectly then I configured webpack-hot-middleware and it seems to break at line 2 where I import app.
  ● Test suite failed to run

    TypeError: setInterval(...).unref is not a function

      at createEventStream (node_modules/webpack-hot-middleware/middleware.js:50:17)
      at webpackHotMiddleware (node_modules/webpack-hot-middleware/middleware.js:12:21)
      at Object.<anonymous> (server/app.js:62:73)
      at Object.<anonymous> (server/test/routes.test.js:2:38)

If I remove the following code from my app.js the tests all run fine.
  app.use(middleware);
  app.use(webpackHotMiddleware(compiler));
  app.get('*', (req, res) => {
    res.write(middleware.fileSystem.readFileSync(path.join(__dirname, '../dist/index.html')));
    res.end();
  });

Has anyone configured Jest for webpack-hot-middleware? I'm kind of lost because I feel like I've tried everything.

This isn't a help forum. Please ask on stackoverflow or discord. Thank you!

@DmsChrisPena Hi, did you fix this problem? I recently ran into this same problem like you. For a long time I thought Jest was running in a node-like environment until I saw this: check this out.
So, the solution is simple, add "testEnvironment": "node" into your jest.config.json.


feat: stop vm when agent finished executing

What does this PR do ?
It sends a request to the agent to stop after they have finished executing the code
How to test ?
You need to open 3 command lines simultaneously and launch these three commands:
Api:
cargo run --bin api

Vmm
cargo build
sudo -E capsh --keep=1 --user=$USER --inh=cap_net_admin --addamb=cap_net_admin -- -c  'RUST_BACKTRACE=1 '$CARGO_PATH' run --bin vmm -- grpc'

Client:
cargo run --bin cli run --config-path <path_to_cloudlet>/src/cli/config/config.yaml

You should see a vm booted on the vmm terminal that launches the agent and stops as soon as the code is finished executing

Just this small change and it should be good :)


x/tools/gopls: 'Check for upgrades' codelens argument can be lazily computed

Gopls version: gopls v0.14.0-pre.4
Just noticed that textDocument/codeLens response for go.mod generates a long message when the project has many dependencies. Assuming the module list can be computed when gopls.check_upgrades command is received lazily, I think it's wasteful to transmit this long list for every codelen response which is more frequent.
[Trace - 8:34:25 PM] Received response 'textDocument/codeLens - (34)' in 14ms.
Result: [
   ...
    {
        "range": {
            "start": {
                "line": 8,
                "character": 0
            },
            "end": {
                "line": 171,
                "character": 1
            }
        },
        "command": {
            "title": "Check for upgrades",
            "command": "gopls.check_upgrades",
            "arguments": [
                {
                    "URI": "file:///Users/hakim/projects/vulnbench/vault/go.mod",
                    "Modules": [
                        "cloud.google.com/go",
                        "cloud.google.com/go/spanner",
                        "cloud.google.com/go/storage",
                        "github.com/Azure/azure-storage-blob-go",
                        "github.com/Azure/go-autorest/autorest",
                        "github.com/Azure/go-autorest/autorest/adal",
                        "github.com/Microsoft/hcsshim",
                        "github.com/NYTimes/gziphandler",
                        "github.com/SAP/go-hdb",
                        "github.com/Sectorbob/mlab-ns2",
                        "github.com/StackExchange/wmi",
                        "github.com/aerospike/aerospike-client-go",
                        "github.com/aliyun/alibaba-cloud-sdk-go",
                        "github.com/aliyun/aliyun-oss-go-sdk",
                        "github.com/apple/foundationdb/bindings/go",
                        "github.com/armon/go-metrics",
                        "github.com/armon/go-proxyproto",
                        "github.com/armon/go-radix",
                        "github.com/asaskevich/govalidator",
                        "github.com/aws/aws-sdk-go",
                        "github.com/bitly/go-hostpool",
                        "github.com/cenkalti/backoff/v3",
                        "github.com/chrismalek/oktasdk-go",
                        "github.com/client9/misspell",
                        "github.com/cockroachdb/cockroach-go",
                        "github.com/containerd/containerd",
                        "github.com/coreos/go-semver",
                        "github.com/denisenkom/go-mssqldb",
                        "github.com/docker/docker",
                        "github.com/docker/go-connections",
                        "github.com/dsnet/compress",
                        "github.com/duosecurity/duo_api_golang",
                        "github.com/elazarl/go-bindata-assetfs",
                        "github.com/fatih/color",
                        "github.com/fatih/structs",
                        "github.com/fullsailor/pkcs7",
                        "github.com/ghodss/yaml",
                        "github.com/go-errors/errors",
                        "github.com/go-ldap/ldap/v3",
                        "github.com/go-ole/go-ole",
                        "github.com/go-sql-driver/mysql",
                        "github.com/go-test/deep",
                        "github.com/gocql/gocql",
                        "github.com/golang/protobuf",
                        "github.com/google/go-github",
                        "github.com/google/go-metrics-stackdriver",
                        "github.com/hashicorp/consul-template",
                        "github.com/hashicorp/consul/api",
                        "github.com/hashicorp/errwrap",
                        "github.com/hashicorp/go-bindata",
                        "github.com/hashicorp/go-cleanhttp",
                        "github.com/hashicorp/go-discover",
                        "github.com/hashicorp/go-gcp-common",
                        "github.com/hashicorp/go-hclog",
                        "github.com/hashicorp/go-kms-wrapping",
                        "github.com/hashicorp/go-memdb",
                        "github.com/hashicorp/go-msgpack",
                        "github.com/hashicorp/go-multierror",
                        "github.com/hashicorp/go-raftchunking",
                        "github.com/hashicorp/go-retryablehttp",
                        "github.com/hashicorp/go-rootcerts",
                        "github.com/hashicorp/go-sockaddr",
                        "github.com/hashicorp/go-syslog",
                        "github.com/hashicorp/go-uuid",
                        "github.com/hashicorp/golang-lru",
                        "github.com/hashicorp/hcl",
                        "github.com/hashicorp/nomad/api",
                        "github.com/hashicorp/raft",
                        "github.com/hashicorp/raft-autopilot",
                        "github.com/hashicorp/raft-snapshot",
                        "github.com/hashicorp/serf",
                        "github.com/hashicorp/vault-plugin-auth-alicloud",
                        "github.com/hashicorp/vault-plugin-auth-azure",
                        "github.com/hashicorp/vault-plugin-auth-centrify",
                        "github.com/hashicorp/vault-plugin-auth-cf",
                        "github.com/hashicorp/vault-plugin-auth-gcp",
                        "github.com/hashicorp/vault-plugin-auth-jwt",
                        "github.com/hashicorp/vault-plugin-auth-kerberos",
                        "github.com/hashicorp/vault-plugin-auth-kubernetes",
                        "github.com/hashicorp/vault-plugin-auth-oci",
                        "github.com/hashicorp/vault-plugin-database-couchbase",
                        "github.com/hashicorp/vault-plugin-database-elasticsearch",
                        "github.com/hashicorp/vault-plugin-database-mongodbatlas",
                        "github.com/hashicorp/vault-plugin-database-snowflake",
                        "github.com/hashicorp/vault-plugin-mock",
                        "github.com/hashicorp/vault-plugin-secrets-ad",
                        "github.com/hashicorp/vault-plugin-secrets-alicloud",
                        "github.com/hashicorp/vault-plugin-secrets-azure",
                        "github.com/hashicorp/vault-plugin-secrets-gcp",
                        "github.com/hashicorp/vault-plugin-secrets-gcpkms",
                        "github.com/hashicorp/vault-plugin-secrets-kv",
                        "github.com/hashicorp/vault-plugin-secrets-mongodbatlas",
                        "github.com/hashicorp/vault-plugin-secrets-openldap",
                        "github.com/hashicorp/vault-plugin-secrets-terraform",
                        "github.com/hashicorp/vault/api",
                        "github.com/hashicorp/vault/sdk",
                        "github.com/influxdata/influxdb",
                        "github.com/jcmturner/gokrb5/v8",
                        "github.com/jefferai/isbadcipher",
                        "github.com/jefferai/jsonx",
                        "github.com/joyent/triton-go",
                        "github.com/keybase/go-crypto",
                        "github.com/kr/pretty",
                        "github.com/kr/text",
                        "github.com/lib/pq",
                        "github.com/mattn/go-colorable",
                        "github.com/mholt/archiver",
                        "github.com/michaelklishin/rabbit-hole",
                        "github.com/miekg/dns",
                        "github.com/mitchellh/cli",
                        "github.com/mitchellh/copystructure",
                        "github.com/mitchellh/go-homedir",
                        "github.com/mitchellh/go-testing-interface",
                        "github.com/mitchellh/gox",
                        "github.com/mitchellh/mapstructure",
                        "github.com/mitchellh/reflectwalk",
                        "github.com/mongodb/go-client-mongodb-atlas",
                        "github.com/natefinch/atomic",
                        "github.com/ncw/swift",
                        "github.com/nwaples/rardecode",
                        "github.com/oklog/run",
                        "github.com/okta/okta-sdk-golang/v2",
                        "github.com/oracle/oci-go-sdk",
                        "github.com/ory/dockertest",
                        "github.com/ory/dockertest/v3",
                        "github.com/patrickmn/go-cache",
                        "github.com/petermattis/goid",
                        "github.com/pkg/errors",
                        "github.com/posener/complete",
                        "github.com/pquerna/otp",
                        "github.com/prometheus/client_golang",
                        "github.com/prometheus/common",
                        "github.com/rboyer/safeio",
                        "github.com/ryanuber/columnize",
                        "github.com/ryanuber/go-glob",
                        "github.com/samuel/go-zookeeper",
                        "github.com/sasha-s/go-deadlock",
                        "github.com/sethvargo/go-limiter",
                        "github.com/shirou/gopsutil",
                        "github.com/stretchr/testify",
                        "github.com/tidwall/pretty",
                        "github.com/ulikunitz/xz",
                        "github.com/xdg/stringprep",
                        "github.com/xi2/xz",
                        "github.com/yuin/gopher-lua",
                        "go.etcd.io/bbolt",
                        "go.etcd.io/etcd",
                        "go.mongodb.org/mongo-driver",
                        "go.uber.org/atomic",
                        "golang.org/x/crypto",
                        "golang.org/x/net",
                        "golang.org/x/oauth2",
                        "golang.org/x/sys",
                        "golang.org/x/text",
                        "golang.org/x/tools",
                        "google.golang.org/api",
                        "google.golang.org/grpc",
                        "google.golang.org/protobuf",
                        "gopkg.in/mgo.v2",
                        "gopkg.in/ory-am/dockertest.v3",
                        "gopkg.in/square/go-jose.v2",
                        "layeh.com/radius"
                    ]
                }
            ]
        }
    },
    ...

I thinks  that the textDocument/codeLens response should be configured to only send what is needed first, deferring the module list calculation until needed.
For example, a flag or token can be placed in the response that may indicate updates, but the module list calculation will not be performed until the user takes action.


Update zenodo, DOI, etc

Codecov Report

Merging #5 (dd8525b) into main (02c1704) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main       #5   +/-   ##
=======================================
  Coverage   96.05%   96.05%
=======================================
  Files           4        4
  Lines         609      609
=======================================
  Hits          585      585
  Misses         24       24


Notebook updates for Colab, part 1

Changes

[x] Update notebooks to use absolute paths (i.e. https urls) instead of relative paths. Colab doesn't clone the entire repository by default, it only grabs the individual jupyter notebook. This means that we need to use absolute paths instead of relative paths on a filesystem to access the data files.
[x] Use a literal height=500 in the UnfoldedMap constructor. When the Colab widget starts with the default height=100% the height is very small for some reason. Providing a literal height=500 should make the output consistent across platforms.

Todo:

[ ] Get rid of jupyter-sidecar, or figure out a good way to not use it in Colab
[ ] Add colab and binder badges to the top of each notebook, so an interactive env is accessible if looking at the notebook on Github.
[ ] Add !pip install unfolded.map-sdk at the top of each notebook for colab. Colab doesn't appear to have any way to install dependencies automatically for a notebook.

Use a literal height=500 in the UnfoldedMap constructor. When the Colab widget starts with the default height=100% the height is very small for some reason. Providing a literal height=500 should make the output consistent across platforms.

Can we detect colab in some way and do this inside our SDK instead?

I think this is essentially ready to go once we publish the next version of the Map SDK


Close marble generation gap at 60-64

Looking at this, is there a reason to have a gap here? Felt weird.

I'm torn on whether to accept this.  On the one hand, yes, the gap is intentional for both marble and rock salt.  On the other, it's not for a reason I can really justify.  It's exists because the same gap exists for other similar stone types in vanilla; specifically andesite, diorite, and granite.  I wouldn't be at all surprised if there's some "here there be dragons" reason why Mojang chose to leave that gap, like plagued their chasm carvers for years (and might still!), and I have no idea how to conclusively prove to my satisfaction that there isn't.  I know it's a little weird and cowardly, but for the time being I'm leaning toward "if it ain't broke, don't fix it".  I'll think on it, though.

Mm, if this is following a model common to other media in vanilla, I'd lean towards matching them.
My guess is that vein size might stretch down or up to meet.

After thinking through this further, I've decided to reject this PR for the sake of maintaining consistency with vanilla.  I appreciate the effort to improve things, though.  Thanks!


Make atomic counter setter available

This solves issue https://github.com/andytill/oneup/issues/2
oneup:set(Counter, Value) is made available (it was tested and benchmarked with results similar to inc:
oneup:oneup_inc in 0.003652s
oneup:oneup_concurrent_inc in 0.028007s
oneup:oneup_set in 0.003875s
oneup:oneup_concurrent_set in 0.019463s

Thanks once again!

No thank YOU! :)  Very cool tool!
On Tue, Nov 7, 2017 at 3:36 PM, Andy Till<EMAIL_ADDRESS>wrote:

Thanks once again!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/andytill/oneup/pull/4#issuecomment-342630267, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ALcephVL3tuvSWECF_I6mE2Xqe8YXYbUks5s0M1LgaJpZM4QVd1l
.


add support for response refs

The swagger spec allows to define an optional root level responses object which allows to reuse response definitions for several endpoints; see: https://github.com/OAI/OpenAPI-Specification/blob/master/versions/2.0.md#fixed-fields
This PR adds support for this potential additional level of indirection.
Check the added endpoint def for /Admin/{id} in tests/apis/test.json. It contains this reference to a shared response definition:
    "responses": {
        "200": {
            "$ref": "#/responses/get_admin_200"
        }
    }

which is pointing to this response definition on the root level:
    "responses": {
        "get_admin_200": {
            "description": "Admin user details",
            "schema": {
                "$ref": "#/definitions/User"
            }
        }
    }

@Kosta-Github - Seems you you closed this - intentional?

yes, the PR didn't fix all potential combinations I am seeing in the variant swagger specs I am dealing... I will try to come up with a new proposal...

Sounds good!

this is the follow-up PR: https://github.com/mtennoe/swagger-typescript-codegen/pull/87


Fix exceptions being thrown while executing 3 tests.

This PR intends to fix the errors reported in #45.

Thanks!


chore: slice append replace loop

Description

Fixes # (issue)
Checklist

I have explained the need for this PR and the problem it solves
I have explained the changes or the new features added to this PR
I have added tests corresponding to this change
I have updated the documentation to reflect this change
I have verified that this change is backward compatible (If not, please discuss on the apinto mailing list first)

Thank you for submitting.


No password field

Hi there
good one to propose that
however seems there is an issue - there is no password field (attached screenshot)
not sure if this is because of the ADFS version (v4) of because the additional auth methods (Azure MFA and Certificate) are enabled
no modification done on your original files

thanks

@benoithamet, thanks for catching this. Would it be possible to get a little more information from you on the issue?
If possible, could you attach:

The full webpage response that gets returned (you can get this by going to 'view page source' in your browser).
Any errors you see in the developer console in your browser (most browsers open the console console on [F-12])

no, no error
attached fiddler trace and source
source.txt

I don't see anything immediately broken in the source you sent.
Would you mind deploying the debug JavaScript from the debug branch: https://github.com/Microsoft/adfsWebCustomization/blob/debugJavaScript/centeredUi/paginatedOnload.js
If you run this with the browser console open, and set to keep logs between navigation, then we should hopefully know more.
Thanks again for reporting this issue!


Low FPS  in spine objects and image objects in the same container or in the same parent container

Version

Phaser Version:3.55.2

Operating system: iPhone 6s Plus, IOS 14.4.1

Browser: Safari

Description
Low FPS  in spine objects and image objects in the same container or in the same parent container
Example:
create 15 spine objects and image objects in the same container or in the same parent container
Example Test Code

import Phaser from 'phaser'

export class DemoScene extends Phaser.Scene{

  preload ()
  {
      this.load.image('img', 'exit_btn.png');
      this.load.setPath('/gob').atlas
      this.load.spine('goblin', 'goblins.json', 'goblins.atlas')
  }

  createSpin(scene:Phaser.Scene,group:Phaser.GameObjects.Container,xS:number,y:number){
      let x = xS
      for(let i = 0 ; i < 5 ; i++){
          //create spine object
          const boy = this.add.spine(x, y, 'goblin', "walk", true)
          boy.setSkinByName('goblin')

          //create image
          let img = this.add.image(x,y,'img')

          //add spine object to container
          group.add(<any>boy);

          //add image object to container
          group.add(img)

          x = x + 150
      }
  }

  create ()
  {
      let t = this.add.text(0, 8, '0');
      t.setFontSize(40)
      this.events.on('render', ()=>{
          console.log(this.game.loop.actualFps)
          t.setText(`${this.game.loop.actualFps}`)
      });

      const group = this.add.container(0,0)
      this.add.existing(group)

      let y = 400
      this.createSpin(this,group,100,y)
      this.createSpin(this,group,100,y+330)
      this.createSpin(this,group,100,y+660)
  }
}

Additional Information

This is to be expected and is explained in the documentation. You're literally breaking the Spine batched renderer with every Image it comes across. This is why we added the SpineContainer Game Object, so you can batch together Spine Game Objects.
In short, do not mix them up like this. Every time Phaser hits a Spine object in the display list, it needs to completely halt all rendering and pass over to Spine. Spine will then take over control of rendering, draw the Spine object and, if there aren't any others in the list, return back to Phaser again. If you mix different object types together, you create a ping-pong effect between the renderers. This is the downside of Spine. Organise your display list more carefully to avoid it.


My ubuntu 22 virtual machine on cloud going asleep

After updating from ubuntu 20 to ubuntu 22 one virtual machine on a cloud provider, the sleep mode was automatically activated.
(pretty annoying the server was going asleep every six minutes of idle).

Written a short guide which seems to work : https://github.com/fugerit79/turbo-guide/blob/main/src/main/md/ubuntu/disable-sleep.md


how to upload images

hi, please tell us how to upload images by your component?

Hi,
Follow these steps -

Add image uploader plugin to your CKEditor build or make a custom build(using builder) with image upload plugin already added. You can find the plugin here https://ckeditor.com/addon/imageuploader
Now place the complete build folder on your webserver so that you can access the folder and the ckeditor.js file using a URL.  Let's say the URL for your script is _http://dummy.com/ckeditor/ckeditor.js_
Now you can simply import this package in your project and pass the custom script url to this component like this. This will load CKEditor from your custom build which will have your image upload plugin.
<CKEditor scriptUrl="http://dummy.com/ckeditor/ckeditor.js" />

If you still face problems, ping me here on my Slack group - https://join.slack.com/t/code-slayers-den/shared_invite/enQtMjQxNzM4MzQwMTk0LTVmYzcxZDQxMzliNmEyMmNjZjVmMWNhN2Q0ODkwOWZhMjBmOTA3NWFiN2JmOTY4MzhhODhlMTU4ZTM5NmMxZDI

Closing this due to no activity.

http://dummy.com/ckeditor/ckeditor.js => link die

That was just an example. The link is not a valid one. I explained you how to use CKEditor with custom build. Hosting your custom build is something you should take care of.

https://ckeditor.com/addon/imageuploader this plugin i try to import but it isn't working. Can you explain how to import this plugin to your react-ckeditor-component

I have already explained the complete process to use this plugin above. Its working perfectly fine for me. Just follow the same steps and it should work for you.
If you have already tried adding it, please attach a screenshot of the error that you are getting and your code.

i put this libary in node_modules but it said can't find it

Please read the documentation of CKEditor, you have not correctly added the imageuploader plugin in your CKEditor build. Simply installing this library won't help. You will have to create a custom CKEditor build with imageuploader plugin, upload it on your server and then use that build with this library.
Using custom build has been already explained in the docs. Please refer to the CKEditor and this library docs before asking these basic questions.

ok thanks

Oi,
Siga esses passos -

Adicione o plugin de upload de imagem à sua compilação CKEditor ou faça uma compilação personalizada (usando o construtor) com o plugin de upload de imagem já adicionado. Você pode encontrar o plugin aqui https://ckeditor.com/addon/imageuploader
Agora, coloque a pasta de compilação completa em seu servidor da Web para poder acessar a pasta e o arquivo ckeditor.js usando uma URL. Digamos que o URL do seu script seja _http://dummy.com/ckeditor/ckeditor.js_
Agora você pode simplesmente importar este pacote em seu projeto e passar o URL de script customizado para este componente como este. Isso carregará o CKEditor de sua compilação personalizada, que terá seu plugin de upload de imagem.
<CKEditor scriptUrl="http://dummy.com/ckeditor/ckeditor.js" />

Thank's :+1:

https://ckeditor.com/addon/imageuploader this plugin i try to import but it isn't working. Can you explain how to import this plugin to your react-ckeditor-component

you can use this code:
<CKEditor scriptUrl="https://cdn.ckeditor.com/4.6.2/full/ckeditor.js" />


Using Faraday instead of RequestProxies

How about using Faraday for all the connections instead of RequestProxies? It would also allow getting the tokens with EM-HTTP (or anything else that Faraday supports) instead of Net::HTTP.

Hi,
I have recently come onboard to help maintain oauth-ruby.
Thank you so much for your concerns, we need eyes everywhere, help is always welcome.
But this is a very old issue and therefore I have to assume this problem has probably already been solved by somebody else.
If anybody would like to re-open or re-create the issue.
Please, do so and I will come help you with it.
Best,
James Pinto.


Add the ability to set bindhosts per server [$5]

The server entry in config can be refactored similar to how the password was.
<Server>
    Hostname = irc.darkscience.ws
    Port = 6697
    Ssl = True
    #Password = asdgf
    Bindhost = <IP_ADDRESS>
</Server>

There is a $5 open bounty on this issue. Add to the bounty at Bountysource.

Isn't this done ages ago?

Could we close this issue it is already done and just has been sitting here for years

It's not done. Bindhost is settable per network, not per server


Dew point for temperatur sensors

Hello @ebaauw
would it be possible to have the plugin calculate a dew point value from temperature and humidity for temperature sensors like Aqara and display the dew point value in homekit on the device?
I think it would be a great function to use the values of two sensors in an automation to detect the right time for air exchange / ventilation.
King Regards
Nastras

Happy to expose dew point for devices that report it, but Homebridge Hue won’t be calculating it.  That’s a feature for a home automation system, nothing to do with exposing devices to HomeKit.

In principle, I agree with you. Unfortunately, it is not so easy to calculate this dew point reliably in Homekit. Shortcuts would be a possibility. But it is also quite unreliable.
Therefore, my idea was that you can calculate these two values Temp and humidity possibly from homebridge-hue.
Would be a great function but I can also understand your answer 😉
Thanks and have a nice weekend!


use nvim_lsp and telescope (nvim >0.5.0 and nightly build required)

https://github.com/HenryHK/wsl-nvim-config/pull/2 better version here


Fix Workflow#finished_at assignment

Closes #1571
Basically, the finished? check is called before WorkflowRetiredCountWorker can update the retired counters used by finished?.  This just moves the finished? call to WorkflowRetiredCountWorker.

LGTM - nice catch @parrish


Unable to make the fill color different from point or stroke color

Hi,
Please can anyone help me, How we can change the make two color for stroke and fill color?
Also, I am unable to add light color in bottom and darker color in top fill color in case of two lines. So, please help me in this as well.
Thanks in advance

@wisercapital you can set the stroke and fill properties per dataset for a line chart:
data = {
  datasets: [{
    backgroundColor: /* fill */
    borderColor: /* stroke */
  }]
}

Thanks @etimberg.
Also, Please can you answer me over this
I am unable to add light color in bottom and darker color in top fill color in case of two lines. So, please help me in this as well.