Documentation

Done

[x] Adds Sphinx
[x] Sphinx Autodoc for Python API docstring
[x] TypeDoc for JavaScript API docstring
[x] Configure Read The Docs

TODO:

[ ] Write an overview
[ ] Document schemas
[ ] Explain how to create a custom model
[ ] Write docstring for the python package
[ ] Check docstring for the javascript package

Screencast
Screencast from 11-24-2022 10:45:24 AM.webm

Wow, that's great, thanks a lot Carlos!

@davidbrochart, Do you know if I can ignore the docs folder for the link_check workflow?

Maybe do the same as https://github.com/jupyter-server/jupyter_server/commit/05913dd7a35662f48907fddefa0edf6a5a674e07 ?

Maybe do the same as jupyter-server/jupyter_server@05913dd ?

Ah, it's already there.

Thanks, @davidbrochart!
Yes, it is already there. Anyway, I'm reorganizing the index to remove the failing links, and I can maybe build the docs before checking the links for the remaining link.

Yes, because that link points to the TypeDoc documentation for the javascript package that is generated when building the docs. That link will always fail, we need to ignore it.

Ah I see, I don't know if it's possible to exclude a single link.
Otherwise, why not have an empty file, that will be overwritten when building documentation?

Yeah, that is a solution. I can generate that file when building the documentation.

@davidbrochart. Much easier, I added a placeholder file, so the link doesn't fail.

I can see that when clicking on JavaScript API, it goes to a different layout, without the top banner. Is it expected?

Yes, it is the same JupyterLab does.
It is because I'm using TypeDoc to automatically generate the documentation of the JavaScript package from the docstring. That generates a completely different webpage that I copy to the build folder of the Sphinx documentation, see:
https://github.com/jupyter-server/jupyter_ydoc/blob/4c958a95bb48aa30de24dda438b38455f6dcb57c/javascript/package.json#L28
and:
https://github.com/jupyter-server/jupyter_ydoc/blob/4c958a95bb48aa30de24dda438b38455f6dcb57c/docs/source/conf.py#L82-L101
There is an extension for Sphinx, sphinx-js, that does everything inside sphinx, but it doesn't work yet with the jinja2>=3. See: https://github.com/mozilla/sphinx-js/pull/191
And the latest Sphinx uses jinja2>=3. In addition, jupyter-server, jupyterlab, and some other packages that we need to install in the same env all use jinja2>=3, soo it makes everything more complicated for development.

I would like to merge and do the remaining tasks in follow-up PRs. This way, we can set up the Read The Docs project and check that everything works.

Thanks!


Update cicd-graduates.md

This PR includes:

A change to the [cicd-graduates.md] page to include my git username.

Looks great! 🎉


eVolv2k_v3 recipe

First try at the eVolv2k recipe. @cisaacstern Sorry about the run-around, hopefully I have everything straightened out now.

Off that hanging synchronize task on the last commit is not good https://github.com/pangeo-forge/staged-recipes/runs/8578849527
I've also been seeing that on some other PRs lately. Fixing this bug is a priority which I plan on looking at today: https://github.com/pangeo-forge/pangeo-forge-orchestrator/issues/97

@cisaacstern Can you halt it and I'll push a small change and see if that "frees the reels" as they used to say about stuck cassette tapes.  I wonder if I pushed a follow up change while it was running checks which induced brief chaos that became insurmountable?

A totally reasonable hypothesis, but nothing you've done is causing this. This is just a good-ole server-side bug in our backend application, which I hope to fix this afternoon. 😅

pre-commit.ci autofix

wow @andersy005 I didn't know that chatops feature with pre-commit was possible. that's awesome 😄

/run eVolv2k

@andersy005 Hi there! If you have any logs I can look at, I'm happy to troubleshoot!

@jordanplanders see https://github.com/pangeo-forge/staged-recipes/pull/194#issuecomment-1261487532 for discussion of... how do we even figure out which job failed? 🤷 I have not yet figured that out the answer to that question for this PR...

@cisaacstern Interesting, so a fraction of the battle is pulling the id info from the submitted (not to be confused with queued) jobs? It looks like there is a url in the line just the one with the dictionary that you pointed to that also points to information about the submitted job?  (I tried visiting, but very appropriately it's behind authentication.) Is there useful information here once you know to go there?
I import a couple of packages that aren't xarray or pangeo-forge at for this recipe (cftime and numpy). Is that problematic?

pre-commit.ci autofix

so a fraction of the battle is pulling the id info from the submitted (not to be confused with queued) jobs

At this early stage of the platform development, this is a bit of a struggle for us maintainers, yes. But I've gotten a better handle on it even in the last hour or so, and was able to dig up logs for your failed run.

I import a couple of packages that aren't xarray or pangeo-forge at for this recipe (cftime and numpy). Is that problematic?

Actually yes this appears to be the problem. Not because those packages are unavailable, but because of a quirk of the distributed computing model of Apache Beam (which runs these recipes on the cloud), wherein all names used within the function body need to be imported within the function body. This would be considered, if anything, poor style in other Python contexts, but here it is actually necessary. I've made that fix and will re-run now.

/run eVolv2k

@cisaacstern Looks good!! Thanks! And that was a great tidbit about import statements. I wonder if there might be other recipes blocked by that issue (I went crawling through the failed runs for references to other packages to see if that might be the issue, but wasn't comprehensive.)

Certainly a good thing to keep an eye out for, and if you suspect it on any other PRs you see, certainly chime in. 🙏


Greeting_Start_Delay only accepts 0 or whole seconds, fractional seconds in decimal seems to introduce breaking error

https://github.com/nickpourazima/rotary-phone-audio-guestbook/blob/4c8bbd84d1a8fc324ebc9d6e4963807a303974e4/config.yaml#L19C1-L19C24
Tried to input a 1.5 second Greeting Start Delay and caused an error.
Could not find documentation from aplay about what is or isn't supported as duration flag parameter.
Just note this with in-line documentation for the meantime?

Thanks @Bonneykins for pointing that out! I'll look into it and keep you posted.

Hi @Bonneykins, I'm working on an update for this along with some other fixes, just need a bit more time after work to validate. Should be able to get it out this weekend.


Infinite load issue when trying to add more beds in facility when all bed types are already selected

Describe the bug
A modal with infinite loading shows up if we try to add more beds in a facility in which all bed types were already selected
To Reproduce
Steps to reproduce the behavior:

Go to '/facility'
Click on View Facility on any Facility
If all bed types are not selected for that facility then try adding them
Now again click on Add More Beds
You can see that a modal with infinite load shows up

Expected behavior
We shouldn't allow users to click on Add More Beds when all bed types are already fulfilled
Screen Recording
https://github.com/user-attachments/assets/eb74d906-e9d4-4446-81db-1dd0225088b7
Suggested Behavior
https://github.com/user-attachments/assets/f190f2b7-b04e-4f05-b8f8-c683ceaa34be
Desktop (please complete the following information):

OS: Windows
Browser: Chrome
Version: Latest

I'd like to work on this issue

Would be solved along with #8918


Maintainer

@deweyjose Is this possible for me to ask for me to be granted as maintainer (or the right status) to be able to produce new releases, through GitHub, in order to follow dgs-codegen release rythm.

hi @zorglube appreciate the PRs. apologies for the release delays. In the future I'll definitely be a bit more responsive.
Let me look into this. One the of the issues will be around securely distributing the plugin in maven central - this requires secure credentials and keys.

Hey @deweyjose ,
Normally, in the project configuration, you can set-up some GitHub action -- like : publish release -- and you have a "secret" section that allows you to give some credentials to an action without show them to anyone.

Ok I'll look into it this weekend.

@zorglube did some work this weekend and added two new github action workflows: build and publish.
The v1.16 release was managed by the the publish workflow - so much easier than what I was doing prior.

I saw that, it seems very nice.

Hey @deweyjose !
How it is going about the CI/CD  ?

Seems to be working just fine.


file_put_contents(./composer.lock): failed to open stream: Permission denied

When run docker-compose run --rm composer update

I have the same error

vi docker-compose.yml

maybe change laravel -> root

No new updates, so I'm closing this issue. If that changes, feel free to open this back up!


I got an apache error when sending http request in the virtual box

$ curl http://localhost/ricochet/ricochet?url=http://news.ycombinator.com
configuration error:  couldn't perform authentication. AuthType not set!: /ricochet/ricochet

If you deployed it using the included vagrant scripts, it's exposed via port 8080. Try something like this:
curl http://<IP_ADDRESS>:8080/ricochet/ricochet?url=http://news.ycombinator.com&ct=text/plain

I'll leave this issue open until you let me know if that worked.

Here is what I did to fix it.
$ sudo vim /etc/apache2/sites-enabled/ricochet-apache.conf
<Directory /var/www/>
Require all granted

$ sudo vim /etc/apache2/sites-available/default
change to 8080
$ ln -s /etc/apache2/sites-available/default /etc/apache2/sites-enabled/default


Transaction list is sorted by Status

Transaction list is sorted by status, would make more sense to be sorted by date

Transaction list is now sorted by date descending as default


Checking the location settings

How I can check location setting after user give permission ?

@MohamedHamada12
You can simply do this by the following two lines :
LocationManager locationManager = (LocationManager) getSystemService(LOCATION_SERVICE);
if (locationManager.isProviderEnabled(LocationManager.GPS_PROVIDER)) {
// GPS settings enabled
} else {
// GPS settings not enabled
}

@yehiahd
Yes I know we can check using this way , but it should provided by library
because Lib considered location setting enabled and try get location.

@MohamedHamada12
You are right but in the meanwhile we have to do it ourself (manually) till this feature is added to the library


Added breaking change documentation for behavior change in Core.

@vinjiang @Jinming-Hu
This is the behavior breaking change that's Storage team did not catch, causing Storage's beta.3 release having a critical bug that needs to be hotfixed.

The breaking change was documented on this PR: https://github.com/Azure/azure-sdk-for-cpp/pull/598
The main reason of that PR was to make URL class to get all values as url encoded.
Storage team fixed the AppendQueryParamenter() because the name of the function changed.
But ApendPath() was ignored.
Note how in that PR we even change the arg name for AppendPath to be
void AppendPath(const std::string& encodedPath) so it was very clear that the value is expected to be encoded.
Core also exposed Encode/Decode APIs for Storage to call before adding values to a URL, however, Storage team has created their own Encoding functions.

This PR had a couple other API/behavioral changes: https://github.com/Azure/azure-sdk-for-cpp/pull/730
@vhvb1989, @RickWinter - should we enumerate all of them in the CL?

The breaking change was documented on this PR: #598
The main reason of that PR was to make URL class to get all values as url encoded.
Storage team fixed the AppendQueryParamenter() because the name of the function changed.
But ApendPath() was ignored.
Note how in that PR we even change the arg name for AppendPath to be
void AppendPath(const std::string& encodedPath) so it was very clear that the value is expected to be encoded.
Core also exposed Encode/Decode APIs for Storage to call before adding values to a URL, however, Storage team has created their own Encoding functions.

Well, I don't think you should rely on storage team reading every line of code in your PR. For these kind of breaking changes, it'll be better if you can explicitly notify us in some way.


Update Spec.json

Hello Davide,
Thank you again for all your great Vega creations. I found a possible improvement for the gantt chart : if several phases start on the same date, the sorting of phases and tasks gets mixed up, and you basically end with phases on top, and tasks below, which is inconvenient. It is related to the Vega "Window" transform. During sorting, if two phases have the same start date, they will also get the same rank (as "peers"). To overcome this, I have created this workaround. There is probably a better solution with the Window transform itself, but I could not find it. Cheers

Thanks @Ed-Ross but I believe this was already fixed a while ago. The spec was updated but the link wasn't. Can you try the following and see if it works as you expect?
Design and Initiation both start on the same day.

Link

Also - Spec.json is old and deprecated. You should be using Gantt 2.0 Spec.json

Thank you, sorry for the false alert.. I did use the old spec and missed the new one.
Your example uses the same test data as I used and shows it is solved !
Cheers

Glad to hear and thanks for pointing it out. 😊


fix font_theme for 1950s and coffee

This will fix using themes 1950s and coffee to avoid error
The `<theme>` font set was not found! Check that the `font` option in `config/_default/params.toml` matches the name of an installed font set.

You're welcome @gcushen.


Disable log4j logging for unirest?

I realise this isn't specifically about Unirest, but how do I configure log4j to prevent debug output from the library?
Thanks

Yes, please, it is VERY noise and flooding the logs during all of our test runs

There are various ways to do this depending on what logging it ends up using. For me it was logback, so I just added this logback.xml:

%-4relative [%thread] %-5level %logger{35} - %msg %n

<root level="ERROR">
    <!-- appender referenced after it is defined -->
    <appender-ref ref="STDOUT"/>
</root>

https://stackoverflow.com/questions/2621701/setting-log-level-of-message-at-runtime-in-slf4j


pulll catalog?

I've been experimenting with creating catalogs and I was wondering if it might be a good idea to have a pull catalog button?   Most of the time I can just hit reload, and the new changes come in, but many times I'll hit reload a couple of times and it refuses to trigger, and I'll go in settings remove my catalog and hit save, and then re-add the catalog.  It might be nice to have a rebuild catalog button, or an api call that I curl in?

@joshuacox - After thinking about it, if you click on the reload and it doesn't update with the changes you made, then that is a bug.  Next time can you make a note of what didn't update properly and add it to this issue.  If you can give me an example I can reproduce, then I might be able to get a dev to fix it :).

Something to note is that I am incrementing the version everytime now.  Another behavior I've adopted is hitting a hard reload on the catalog page and not the catalog item itself.   Most of the time this works, but every now and then I do have to "go in settings remove my catalog and hit save, and then re-add the catalog" as above.  I have waited a few times (gone to bed, and wait for morning), and it will still be on the old version and I'll have to do the above procedure to 'kick' it.  But for the most part it is working as intended, and I consider this an edge case.  I'm going to close as my workaround does indeed force it to update, feel free to reopen.

this is still a major issue.  I was going through some of the behaviour I go through to comply with Rancher not wanting to pull my catalogs and @jamessmoore was astounded at how many times you must reload a page before a catalog will update.

there needs to be a "pull catalogs" button somewhere for catalog developing sanity

I have also notice that this sentence "Alternatively, you could update the templates in the 0 folder and just re-deploy the entry." on this page is not true.  For example,
create a template
deploy in rancher
edit yml files in place inside 0
git commit and push
redeploy
changes are almost never carried over unless I reload stacks/user an insane amount of times (insane because I have no clue what this number is, sometimes once, sometimes 500 times and still will not update)

You're missing wait ≥ 5 min for polling or click refresh part...

'click refresh'  thank you for the pic!  I would've assumed you were talking about ctrl-shift-R or similar.  Hopefully that eliminates my issues.


Support for Windows?

I really like this ap, but my OS is Windows
please develop the Windows version too

Technically there is no reason that this shouldn't work on Windows, and I know there are users who have it running on Windows machines. You just need to have Python, Selenium and all of the Python module dependencies. I'm going to edit this issue to make the title "Add documentation for running in Windows"


splitTextToSize method crashes if cell value is not a string

`API.splitTextToSize = function(text, maxlen, options) {
"use strict";
 .......

// first we split on end-of-line chars
var paragraphs;
if (Array.isArray(text)) {
  paragraphs = text;
} else {
  paragraphs = text.split(/\r?\n/);
}

.....
})(jsPDF.API)`
this method crashes at this line paragraphs = text.split(/\r?\n/)
if "text" is not a string.
in my case cell value/text can be number also
so 34.split() method causes error.
TypeError: t.split is not a function     at Object.xt.splitTextToSize (split_text_to_size.js:360)     at Object.<anonymous> (cell.js:567)     at Array.map (<anonymous>)     at Object.s (cell.js:564)     ....
this case should be handled by converting this number to string I think

The table function accepts only strings as cell values. You could preprocess your data and convert the numbers to a string beforehand.

yes. for now I am writing a  utility function to convert to string


[SPARK-6887][SQL] ColumnBuilder misses FloatType

https://issues.apache.org/jira/browse/SPARK-6887

Test build #30199 has started for   PR 5499 at commit d169d33.

@liancheng Where should I add the test? At first, I thought NullableColumnBuilderSuite is the place But, why NullableColumnBuilderSuite does not really use those real column builders?

Test build #30199 has finished for   PR 5499 at commit d169d33.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

I will add a test in InMemoryColumnarQuerySuite.

Good catch! The change LGTM.
NullableColumnBuilderSuite uses a mock builder because the real builders also mix in CompressibleColumnBuilder, while NullableColumnBuilderSuite is used to test nulls.
This wasn't detected before because we were using a column type ID in the columnar buffer, and column accessors are constructed according to the type ID. However, while adding column type for fixed-decimal, decimals with different precisions and scales are actually different types, a single type ID is not enough to represent complete type information. Thus column type ID is invalidated and now column accessors are built according to the data types in the schema.

Test build #30357 has started for   PR 5499 at commit 84cba38.

Test build #30357 has finished for   PR 5499 at commit 84cba38.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

Thanks!  Merged to master.


RuntimeError: failed to create XNNPACK runtimeNode number 2977 (TfLiteXNNPackDelegate) failed to prepare.

1. System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Google Colab with Python 3.10.12
TensorFlow installation (pip package or built from source):
pip package
TensorFlow library (version, if pip package or github SHA, if built from source):
v2.17.0

2. Code
import tensorflow as tf

saved_model_dir = '/content/saved_model'

num_calibration_steps = 100

input = tf.cast(tf.random.normal((1, 640, 640, 3)), tf.float32)
dummy_input = tf.cast(tf.random.normal((1, 2)), tf.int64)

def representative_dataset_gen():
    for _ in range(num_calibration_steps):
        yield [dummy_input, input] #model has 2 input tensors

converter = tf.lite.TFLiteConverter.from_saved_model(saved_model_dir)
converter.optimizations = [tf.lite.Optimize.DEFAULT]
converter.representative_dataset = representative_dataset_gen
converter.target_spec.supported_ops = [
  tf.lite.OpsSet.TFLITE_BUILTINS_INT8,
  tf.lite.OpsSet.SELECT_TF_OPS
]
converter.inference_input_type = tf.int8
converter.inference_output_type = tf.int8

tflite_quant_model = converter.convert()

# Save the quantized model to a local file
with open('quantized_model.tflite', 'wb') as f:
    f.write(tflite_quant_model)

3. Failure after conversion
After converting the model from ONNX using onnx2tf, I got saved_model, which I tried to convert to int8 quantized model using the code above. When trying to inference the model, after reading the model by the interpreter and calling the function allocate_tensors()
interpreter = tf.lite.Interpreter(model_path="/content/quantized_model.tflite")
interpreter.allocate_tensors()

I get the following error:
RuntimeError                              Traceback (most recent call last)
[<ipython-input-7-b6b80a3bdf94>](https://localhost:8080/#) in <cell line: 6>()
      4 interpreter = tf.lite.Interpreter(model_path="/content/quantized_model.tflite")
      5 print(interpreter.get_input_details())
----> 6 interpreter.allocate_tensors()

[/usr/local/lib/python3.10/dist-packages/tensorflow/lite/python/interpreter.py](https://localhost:8080/#) in allocate_tensors(self)
    535   def allocate_tensors(self):
    536     self._ensure_safe()
--> 537     return self._interpreter.AllocateTensors()
    538
    539   def _safe_to_run(self):

RuntimeError: failed to create XNNPACK runtimeNode number 2977 (TfLiteXNNPackDelegate) failed to prepare.

Could someone give me some advice, suggestions on how to solve this error? I couldn't even find that anyone has solved the same problem.
The closest to this error is this issue, but the workaround is to convert the ONNX model to Keras and for my complex model it is not possible to fix.

Hi, @isuchy
Thank you for bringing this issue to our attention, if possible could you please help us with your saved_model or Google colab notebook which you used to get saved_model so I'll try to replicate the same behavior from my end and will help you further ?
Thank you for your cooperation and patience.

Hello @gaikwadrahul8,
Thank you for your fast response.
I've uploaded the saved_model, please try in your environment to see if you can reproduce the error.

Hi, @isuchy
Thank you for providing the saved_model access, I'm able to reproduce the same error RuntimeError: failed to create XNNPACK runtimeNode number 2977 (TfLiteXNNPackDelegate) failed to prepare. from my end and will need to dig more into this issue for reference here is gist-file
Thank you for your cooperation and patience.

Hi, @pkgoogle
Please take look into this issue. Thank you.

Hi, @gaikwadrahul8, @pkgoogle
have you had a chance to check it out? Does it look like a more serious problem with the XNNPACK runtime implementation?

Hi @isuchy, can you convert to a pytorch model and use ai-edge-torch? Let us know if that works better for you.

I'm running into something very similar with a simple MobileNetV3 based model:
Exception has occurred: RuntimeError
failed to create XNNPACK runtimeNode number 230 (TfLiteXNNPackDelegate) failed to prepare.
  File "tflite_test.py", line 20, in <module>
    interpreter.allocate_tensors()
RuntimeError: failed to create XNNPACK runtimeNode number 230 (TfLiteXNNPackDelegate) failed to prepare.

Going to give ai-edge-torch a shot in the meantime..

Hi @isuchy, I ran your script and I noticed this model needs flexops... in what runtime environment are you running inference? Can you ensure your have followed direction here? https://ai.google.dev/edge/litert/models/ops_select

Hi @pkgoogle,

I ran your script and I noticed this model needs flexops... in what runtime environment are you running inference? Can you ensure your have followed direction here? https://ai.google.dev/edge/litert/models/ops_select

I ran the model inference in Google Colab and as it says here https://ai.google.dev/edge/litert/models/ops_select#python, for LiteRT runtime it is enough to have the TensorFlow package installed. But later I would like to deploy the model on a Raspberry Pi 5, running on Python as well.

Hi @isuchy, can you convert to a pytorch model and use ai-edge-torch? Let us know if that works better for you.

I also tried ai_edge_torch as you recommended, it worked well for converting from the Pytorch model, but its output is directly the TFLite format (.tflite), with no way to set parameters to perform post-training int8 quantization. If I wanted to use tf.lite.TFLiteConverter, it does not support creating a TFLiteConverter object from a TFLite format model, only the following methods are supported

experimental_from_jax
from_concrete_functions
from_keras_model
from_saved_model

So I haven't actually figured out how to quantize a float32 .tflite model to be able to perform inference on that model.

I was also able to replicate on tf-nightly here.
@rascani, can you please take a look? Thanks.
@isuchy For the ai-edge-torch workflow, you would need to do TFL quantization here.

Hi @pkgoogle,
I tried the quantization as you advised in your comment and I was also inspired by this article

@isuchy For the ai-edge-torch workflow, you would need to do TFL quantization here.

import torch
import torchvision
import ai_edge_torch

from ai_edge_torch.quantize import pt2e_quantizer
from ai_edge_torch.quantize import quant_config
from torch.ao.quantization import quantize_pt2e

pretrained_weights = torch.load('/content/model.pth', weights_only=True)
resnet50 = torchvision.models.resnet50(weights=pretrained_weights)
sample_inputs = (torch.randn(1, 3, 640, 640),)

quantizer = pt2e_quantizer.PT2EQuantizer().set_global(
    pt2e_quantizer.get_symmetric_quantization_config()
)
graphModule = torch.export.export(resnet50.eval(), sample_inputs).module()
model = quantize_pt2e.prepare_pt2e(graphModule, quantizer)

torch.ao.quantization.move_exported_model_to_eval(model)

model = quantize_pt2e.convert_pt2e(model, fold_quantize=False)

with_quantizer = ai_edge_torch.convert(
    model,
    sample_inputs,
    quant_config=quant_config.QuantConfig(pt2e_quantizer=quantizer),
)
with_quantizer.export("resnet50_int8.tflite")

The model is saved as a tflite file after conversion and quantization, but several warnings are raised during the run.
/usr/local/lib/python3.10/dist-packages/torchvision/models/_utils.py:223: UserWarning: Arguments other than a weight enum or `None` for 'weights' are deprecated since 0.13 and may be removed in the future. The current behavior is equivalent to passing `weights=ResNet50_Weights.IMAGENET1K_V1`. You can also use `weights=ResNet50_Weights.DEFAULT` to get the most up-to-date weights.
  warnings.warn(msg)
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_1) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_2) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_3) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_4) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_5) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_6) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_7) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_8) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_9) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_10) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_11) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_12) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_13) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_14) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_15) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_16) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_17) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_18) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_19) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_20) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_21) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_22) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_23) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_24) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_25) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_26) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_27) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_28) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_29) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_30) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_31) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_32) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_33) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_34) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_35) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_36) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_37) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_38) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_39) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_40) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_41) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_42) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_43) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_44) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_45) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_46) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_47) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_48) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_49) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_50) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_51) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/fx/graph.py:1062: UserWarning: erase_node(_native_batch_norm_legit_no_training_52) on an already erased node
  warnings.warn(f"erase_node({to_erase}) on an already erased node")
/usr/local/lib/python3.10/dist-packages/torch/ao/quantization/utils.py:407: UserWarning: must run observer before calling calculate_qparams. Returning default values.
  warnings.warn(
/usr/local/lib/python3.10/dist-packages/torch/ao/quantization/observer.py:1315: UserWarning: must run observer before calling calculate_qparams.                                    Returning default scale and zero point
  warnings.warn(
WARNING:root:Your model is converted in training mode. Please set the module in evaluation mode with `module.eval()` for better on-device performance and compatibility.

I don't understand the warning message that the model is converted in training mode. I think the model is set in evaluation mode by calling resnet50.eval(). I know this is out of the question, but could you please give me some hints, if you have any experience with that, why is it like it happens? Thank you.

Hi @isuchy, as far as I can tell you did it correctly, I think this is just a bug for now.


Crash when merge dictionary. dictionary is Empty.

Version: Parquet.Net v3.3.4
Runtime Version: .Net Framework v4.5
OS: Windows
Expected behavior
Please describe how you expect Parquet.Net to work.
Actual behavior

Parquet.dll!Parquet.Data.BasicDataTypeHandler.MergeDictionary(System.Array untypedDictionary, int[] indexes, System.Array data, int offset, int length) 行 112	C#	已加载符号。
Parquet.dll!Parquet.File.DataColumnReader.ReadColumn(System.IO.BinaryReader reader, Parquet.Thrift.Encoding encoding, long totalValues, int maxReadCount, Parquet.File.DataColumnReader.ColumnRawData cd) 行 237	C#	已加载符号。
Parquet.dll!Parquet.File.DataColumnReader.ReadDataPage(Parquet.Thrift.PageHeader ph, Parquet.File.DataColumnReader.ColumnRawData cd, long maxValues) 行 199	C#	已加载符号。
Parquet.dll!Parquet.File.DataColumnReader.Read() 行 88	C#	已加载符号。
Parquet.dll!Parquet.ParquetRowGroupReader.ReadColumn(Parquet.Data.DataField field) 行 64	C#	已加载符号。

Attach a parquet file. python parquet load it from file ok. but parquet.net can't load.
mkt.parquet.zip

To me it looks like this file is not quite correct. The peratio2 column starts with a dictionary page that has zero entries and then it has a data page with a PLAIN_DICTIONARY encoding all of whose index values are zero. As there is nothing in the dictionary page, there's no reasonable value to return for the data.
I note that pandas.read_parquet is returning a columns of NaN for this value; I would guess that it's simply swallowing the error and returning a column of invalid values.

Oh, I see -- the definition levels aren't being taken into account. That shouldn't be too hard to fix.

(FYI, the fix hasn't been approved yet.)


Global Styles

This issue is to discuss how we define global style, what would be contained in global styles (e.g. fonts, reset), to agree on a standard and to implement any code or documentation updates needed for that standard.
We could distribute some common settings or define how a parent application would implement this manually in the README.
Things to consider:

gov.uk domains vs. non gov.uk domains
CSSinJS (injectGlobal) or static CSS file
server rendered, client rendered or universal
Use a published CSS reset or use a basic/custom one
perhaps some sites want to fall back to 100% system fonts e.g. for 100% unicode support
non govuk domains - official fallback vs. custom fonts
how to preview in storybook

There are 2 ways to do this:
Declaring your global styles (reset, normalize, etc) in a separate file.
This escape hatch: (injectGlobal)
I suggest using a file. injectGlobal works best when it is called once on the server (at the right moment) and is not called on the client.

https://medium.com/@wonderboymusic/a-field-guide-to-css-in-js-f7cbd9ed79a7

@Loque- had done the following in a CSS file manually in a prototype:
@import url('https://rsms.me/inter/inter-ui.css');

*,
*::before,
*::after {
  -webkit-box-sizing: border-box;
  box-sizing: border-box;
}

html {
  -webkit-text-size-adjust: 100%;
}

img,
svg {
  vertical-align: top;
}

html,
body {
  padding: 0;
  margin: 0;
  background: #f3f3f3;
  font-family: 'Inter UI', Arial, sans-serif;
  -webkit-font-smoothing: antialiased;
  -moz-osx-font-smoothing: grayscale;
}

planning to use what I have in the example app

Fixed in #834


Error in logs

Hi! I new in play and deadbolt and I have simple question.
I have controler with action
public class Application extends Controller {

    @SubjectPresent
    public Result index() {
        return ok(index.render("Your new application is ready."));
    }

}

I get "403 Authorization failed" in browser and it is perfectly clear beacause I'm not logged in. But I also have error in logs
[error] - be.objectify.deadbolt.java.actions.AbstractDeadboltAction - Access to [/] requires a subject, but no subject is present.

It is normal or I implemented something wrong?

That's absolutely normal.  You can find the relevant code here: https://github.com/schaloner/deadbolt-2-java/blob/master/code/app/be/objectify/deadbolt/java/actions/AbstractDeadboltAction.java#L193

Yes, I saw it.
And you think this kind of action (restricting access) is application error?

Sorry for the delay, I've been on vacation.
If you're using Play 2.4, you can update your Deadbolt dependency to "be.objectify" %% "deadbolt-java" % "2.4.1" - this changes the logging level on this line to ´info´.

Thanks!


Fix relevance picker bugs

Codecov Report

Merging #76 into master will decrease coverage by 0.00%.
The diff coverage is 44.00%.

@@            Coverage Diff             @@
##           master      #76      +/-   ##
==========================================
- Coverage   58.18%   58.17%   -0.01%
==========================================
  Files          76       76
  Lines        5921     5927       +6
==========================================
+ Hits         3445     3448       +3
- Misses       2476     2479       +3

Impacted Files
Coverage Δ

lib/pages/filter/view/relevance_picker.dart
60.81% <44.00%> (-0.40%)
:arrow_down:

Continue to review full report at Codecov.

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


Overwriting a secure file

I wish to overwrite a secure file with an updated version.. how can i do this?

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

ID: 2a5f8376-3654-0ed4-d9f8-d15f28350783
Version Independent ID: f393daba-6515-6c26-71ac-b557e8c8dad1
Content: Download Secure File
Content Source: docs/build-release/tasks/utility/download-secure-file.md
Service: devops
GitHub Login: @andyjlewis
Microsoft Alias: alewis

Can i get an update on this?

Hi @padmick, I apologize for not seeing this until now.  Currently, VSTS doesn't support updating a secure file.  Instead, the existing secure file must be deleted and another uploaded in its place.  We plan to support direct updates as soon as we can get to them.

Thanks
How can one delete the file? A delete option isn't advailable
Sent from BlueMail
On May 21, 2018, 3:10 PM, at 3:10 PM, David Staheli<EMAIL_ADDRESS>wrote:

Hi @padmick, I apologize for not seeing this until now.  Currently,
VSTS doesn't support updating a secure file.  Instead, the existing
secure file must be deleted and another uploaded in its place.  We plan
to support direct updates as soon as we can get to them.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/MicrosoftDocs/vsts-docs/issues/791#issuecomment-390665076

My latest UI may look a bit different than yours, but does yours show a delete option on this context menu?  It's a little hidden.

Ah got it thanks. I'll check now if I can delete the file and replace it with a file with the same name
Sent from BlueMail
On May 21, 2018, 3:21 PM, at 3:21 PM, David Staheli<EMAIL_ADDRESS>wrote:

My latest UI may look a bit different than yours, but does yours show a
delete option on this context menu?  It's a little hidden.

--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/MicrosoftDocs/vsts-docs/issues/791#issuecomment-390668342

works! thanks ill close this

Hello @DavidStaheli
I see that the DownloadSecureFile task should be also updated manually everywhere in each Build/Release with a new secure file, in case if the used secure file deleted and uploaded again, even if the file name is the same.
Is the support of direct updates you prognoses soon will have the same behavior?..
I mean when the secure file update will be supported - will it be necessary to update the Build/Release DownloadSecureFile tasks manually?..

Is there an issue or User Voice item that tracks the actual feature of updating secure files in place?

It would be really handy if we could update these files. Especially for iOS developers the provisioning profiles & certificates need updating from time to time and its a bit of a pain to delete everything and have to re-enabled all the permissions in our pipelines 🥺
Any ETA on the feature being implemented?

@All -- thank you for your follow-up.  This repo is solely for documentation issues, and this issue seems product-related. To make sure it gets in front of the right people, please add it here.

A proper link to the issue still ther from 2019:
Unable-to-edit-a-secure-file


[fix](index compaction)Fix core when having multiple dest segments (#40254)

bp #40254

run buildall


Feature/271 secret settings

#271
Goes with projectjellyfish/ux#215

 Code Climate has analyzed this pull request.

looks good.


🐛 BUG: GitHub action failing with environment

What version of Wrangler are you using?
2.0.23
What operating system are you using?
Mac
Describe the Bug
I'm using GitHub actions to deploy workers to different environments. But I'm getting an error as follows (though the worker is deployed correctly).
Using API Token authentication
$ Running: wrangler publish --env staging
 ⛅ wrangler 2.0.23
--------------------
▲ [WARNING] Processing wrangler.toml configuration:

    - 😶 Ignored: "type":
      Most common features now work out of the box with wrangler, including modules, jsx, typescript, etc. If you need anything more, use a custom build.
    - Deprecation: "zone_id":
      This is unnecessary since we can deduce this from routes directly.
    - Unexpected fields found in top-level field: "format"

Total Upload: 8.24 KiB / gzip: 2.46 KiB
Worker ID:  optimizer-staging
Worker ETag:  3a3a8f22a65093324d091b67d22dad172e5fd555289bf604b8b226290fcd9fe3
Uploaded optimizer-staging (0.61 sec)

✘ [ERROR] A request to the Cloudflare API (/accounts/xyz/workers/scripts/optimizer-staging/routes) failed.

 workers.api.error.duplicate_route [code: 10020]

wrangler.toml
name = "optimizer"
type = "javascript"
format = "service-worker"
main = "./src/index.js"

account_id = "abc"
zone_id = "xyz"
compatibility_date = "2022-04-13"
usage_model = "bundled"

[env.production]
name = "optimizer-production"
route = { pattern = "*/*", zone_id = "xyz" }

[env.staging]
name = "optimizer-staging"
route = { pattern = "*/*", zone_id = "xyz" }

deploy.yml
name: Deploy

on:
  push:
    branches:
      - main

jobs:
  deploy:
    runs-on: ubuntu-latest
    name: Deploy
    steps:
      - uses: actions/checkout@v3
      - name: Publish
        uses<EMAIL_ADDRESS>        with:
          apiToken: ${{ secrets.CF_API_TOKEN }}
          command: publish --env staging

The error code specifies what's going on here, you're trying to deploy the worker to the same route in both production and staging environments, which is not what you probably want.

@threepointone tried different route, but same error.
wrangler.toml
name = "optimizer"
type = "javascript"
format = "service-worker"
main = "./src/index.js"

account_id = "abc"
zone_id = "xyz"
compatibility_date = "2022-04-13"
usage_model = "bundled"

[env.production]
name = "optimizer-production"
route = { pattern = "*/*", zone_id = "xyz" }

[env.staging]
name = "optimizer-staging"
route = { pattern = "*example.com/*", zone_id = "xyz" }

It's possible that you've already deployed another worker to those routes. You'll have to remove them from the dashboard before publishing with wrangler again.

@threepointone tried deleting, but still no luck. I'm using the same .toml file with buddy.works, and it deploys fine!

You'll have to check, that's what the error means. There's a worker already deployed to the route.

@threepointone, what if I want to overwrite/update? The worker in production cannot be deleted.

You can deploy a worker with the same name to that route. I'm suggesting you change the staging one to another route that doesn't overlap with the production one.

I deleted the entire staging worker and its routes. Still not working.

I can't help you debug a usage problem here I'm afraid. This is definitely not a wrangler issue though. I'd recommend asking in discord, with more details of the wrangler.toml without substitutions.

Also I would check that you don't happen to have another completely separate worker wired up to the same route.

Let us know if you were able to get support in the Discord.
https://discord.gg/cloudflaredev


Setup Cert Manager

Startup message:
error: resource mapping not found for name: "cloudflare-cluster-issuer" namespace: "" from "STDIN": no matches for kind "ClusterIssuer" in version "cert-manager.io/v1"
ensure CRDs are installed first

Startup message: error: resource mapping not found for name: "cloudflare-cluster-issuer" namespace: "" from "STDIN": no matches for kind "ClusterIssuer" in version "cert-manager.io/v1" ensure CRDs are installed first

will be handled in #141


Write access to this repo

Hi,
I would like to be a contributor to this repo. Could any one of the maintainers grant me access?
@pcross616 @ev0ldave
Thanks & Regards
Nilanjan

@nilroy we handed off management of this cookbook to sous-chefs because we could not give it the time that was needed to keep this up to date.  If you would like to become a maintainer please join sous-chefs and add yourself.
http://sous-chefs.org/becoming-a-member/

Closing this issue as instructions for contributing were presented.


Patch instead of update Module and ManagedClusterModule status

This change includes the following:

mark the DaemonSetStatus fields are optional
replace update with patch for the {Module,ManagedClusterModule} status subresource

It resolves the issue of the reconciliation returning an StatusConflict error when trying to update the status of a {Module,ManagedClusterModule} CR, which in turn leads to multiple reconciliation requests of a CR just to update its status subresource successfully.
Context
When updating the {Module,ManagedClusterModule} status subresource, the following error usually comes up:
"the object has been modified; please apply your changes to the latest version and try again"
This error is caused by the controller-runtime cache being stale at the time of the status update, i.e. the resourceVersion of the {Module,ManagedClusterModule} in the cache is not the latest one found in the kubeapi server.
This reconcilation errors leads to additional reconciliation requests being queued, with nothing more to reconcile other than actually update the status subresource.
When using patch instead of update, the resourceVersion is not included in the payload and the above error is resolved. In order to be able to use patch instead of update, we need to mark the DaemonSetStatus fields as optional. Otherwise, when trying to patch with one or more of the DaemonSetStatus fields the validation of the patch call will fail with the error that one or more required fields are missing.
Signed-off-by: Michail Resvanis<EMAIL_ADDRESS>

/approve

/assign @yevgeny-shnaidman

/lgtm


Missing Icons

The About Section is missing an icon
TL;DR also seems incomplete

Can you provide the issue in detail ?

Hello, actually that's a bug that make user see no icon in the about section. No need to worry it will be fixed automatically

ok thanks for informing. @divyanshudhruv
I am a beginner in open source. So, is there any other issue which I can work upon and start gaining my first open source experience. Can u guide me ?

Ok, you can start open-source by contributing to this repository
If I have any issue in progress, I will contact you. Good luck

Assign me i will work on it


Cabify does not appear in the Sao Paulo city results on production

On production, when I do a search for Sao Paulo (or São Paulo), only show me easytaxi as an option. I've thought this is a bug and I forked the project to debug it. But, when I've ran the application locally there is cabify in the results.
I did the print screen that show the difference:
local

production

I don't know, but I think it seem need to deploy that update the last version on production.
PS.: congrats for project. ;)

Thanks ! I already updated: https://isthereuber.in/sao-paulo.
Thanks for letting me know :)


Update gaze-in-directx.md

Docs Build status updates of commit 564edbf:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/gaze-in-directx.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


CB-26272 - Cluster side: Ensure that /var/log/user-data.log does not reveal any secrets

I don't see any history of test images burned and validated with this branch. How was this tested?


Reload logs on SIGHUP, in case the LogDir option was modified

Here's the issue: if ever the LogDir option is modified in the
configuration file, sending SIGHUP is not enough to make the change
effective, and mirrorbits keep using the old LogDir value. In other
words: the LogDir config option is not reloaded on SIGHUP.
How to reproduce: say you have mirrorbits running, and LogDir is
commented out. It means mirrorbits doesn't log downloads under the file
$LogDir/downloads.log. Now, edit the configuration, uncomment LogDir,
send SIGHUP to mirrorbits, and observe that it has no effect: there is
still no file $LogDir/downloads.log.
I consider it a bug, even though a minor one, just a papercut.
This commit fixes it by always reloading the logs on SIGHUP, thus making
sure that, if ever LogDir was modified, the change is taken into
account.

@jbkempf Is this MR Ok with you? It's pretty trivial but I didn't receive any feedback so far.
I reworded the commit message, trying to make it clearer. If there's something unclear or obviously wrong with this commit, please say. Thanks!

Thanks!


feat: Show status message based on component analysis status

When analysis in progress:(animated rolling progress)

When there is no vulnerability:

When there is a vulnerability:

When there are vulnerabilities with exploits:

Fixes https://issues.redhat.com/browse/APPAI-1629, https://github.com/fabric8-analytics/fabric8-analytics-vscode-extension/issues/434
Changes:

Updated engine to ^1.47.0 to make use of command with title type
Fixed tests due to update to new engine version (unified mock context and shared across all tests)

Codecov Report

Merging #459 (a0ed418) into master (5c73a80) will decrease coverage by 1.93%.
The diff coverage is 16.12%.

@@            Coverage Diff             @@
##           master     #459      +/-   ##
==========================================
- Coverage   65.80%   63.86%   -1.94%
==========================================
  Files          14       16       +2
  Lines         772      797      +25
  Branches      171      168       -3
==========================================
+ Hits          508      509       +1
- Misses        264      288      +24

Impacted Files
Coverage Δ

src/caNotification.ts
4.16% <4.16%> (ø)

src/extension.ts
24.65% <9.52%> (+5.55%)
:arrow_up:

src/caStatusBarProvider.ts
37.50% <37.50%> (ø)

src/statusMessages.ts
100.00% <100.00%> (ø)

src/multimanifestmodule.ts
46.66% <0.00%> (-5.93%)
:arrow_down:

Continue to review full report at Codecov.

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

@arajkumar
Any specific reason why we are not including advisory counts in status bar?

@dvandra , vulnerability and advisory counts are added together to keep the status compact.


minecraft:warped_nylium does not work

2023-08-18 [06:34:48.395] [Server thread/EMERGENCY]: Crash occurred while handling a packet from session: XXXXXXX
2023-08-18 [06:34:48.396] [Server thread/CRITICAL]: pocketmine\data\SavedDataLoadingException: "No deserializer found for ID minecraft:warped_nylium" (EXCEPTION) in "pmsrc/src/item/Item" at line 723

To Reproduce
Steps to reproduce the behavior:

have an inventory with warped_nylium
join
crash

Version information
PocketMine: 5.4.2
Plugin: 1.0.1
Minecraft: latest

Please check that you have it set in config.yml
https://github.com/diamond-gold/DummyItemsBlocks/blob/main/resources/config.yml#L107

Yes, it is in the config.yml file.
I used INether before (in pm4) which implemented this block. Idk if it has something to do with it.

Oh and can you add crying obsidian?
pocketmine\data\bedrock\item\UnsupportedItemTypeException: "No deserializer found for ID minecraft:crying_obsidian" (EXCEPTION) in "pmsrc/src/data/bedrock/item/ItemDeserializer" at line 86
This error appears right below

I added it manually but I think it needs state support...

Are you using a fork of pmmp?
Crying obsidian is already implemented by pmmp, adding it into config.yml should display the following warning:
[Server thread/WARNING]: [DummyItemsBlocks] Block minecraft:crying_obsidian is already registered!

It does not seem like an issue with this plugin, reported to pmmp/PocketMine-MP#6008

Are you using a fork of pmmp?
Crying obsidian is already implemented by pmmp, adding it into config.yml should display the following warning:
[Server thread/WARNING]: [DummyItemsBlocks] Block minecraft:crying_obsidian is already registered!

No it is not a fork. Oh yeah and I see that message.

It does not seem like an issue with this plugin, reported to pmmp/PocketMine-MP#6008

Alright, thank you :)


Collapsible Sections

Over the past month, I have been using Hombrewery a lot. I think it would be nice to collapse sections of my brew that I don't use so that it is easier to scroll to the 25 page or find the section I want to edit. 25 pages seem like a wall of text to scroll through to find anything. I have had to split up my brew into manageable chapters so that it was easier to edit it.
Thank you for the homebrewery, this is my feedback.

Old issue/feature request, but would like to +1 this.  I was thinking that in the editor pane it would be nice to set this up like:
\page [Page Title]
A user could collapse portions of the markup in the editor pane by maybe double clicking the \page line, and the \page line would also have a user-entered title so that it is easy to find which page they need.

Old issue/feature request, but would like to +1 this.  I was thinking that in the editor pane it would be nice to set this up like:
\page [Page Title]
A user could collapse portions of the markup in the editor pane by maybe double clicking the \page line, and the \page line would also have a user-entered title so that it is easy to find which page they need.

CodeMirror has collapsing built in by the looks of things, so this shouldn't be too complicated.
I would limit it to \page boundaries, i.e. not objects within pages.
https://codemirror.net/demo/folding.html

CodeMirror has collapsing built in by the looks of things, so this shouldn't be too complicated.
I would limit it to \page boundaries, i.e. not objects within pages.
https://codemirror.net/demo/folding.html

Looks like we would need to edit shared/Naturalcrit/codeEditor/codeEditor.jsx and add:
foldGutter: {
                rangeFinder: function(startPos) //given a starting line of code, return null if this doesn't start a foldable block, or return line number where this foldable block ends. startPos.line is the starting line number.
            },
gutters: ["CodeMirror-linenumbers", "CodeMirror-foldgutter"]

Looks like we would need to edit shared/Naturalcrit/codeEditor/codeEditor.jsx and add:
foldGutter: {
                rangeFinder: function(startPos) //given a starting line of code, return null if this doesn't start a foldable block, or return line number where this foldable block ends. startPos.line is the starting line number.
            },
gutters: ["CodeMirror-linenumbers", "CodeMirror-foldgutter"]

Oh .. integration with #1147 might prove weird - that is currently deferred to #1093 V3 Renderer because it might break brews where \page does not occur at line start.
Suggestion: implement #629 but only fold \page if at line-start .. rendering would unaffected (yay), folding is available (yay), and the fact a given \page doesn't fold because it isn't at line-start might encourage editorial clean-up.

Oh .. integration with #1147 might prove weird - that is currently deferred to #1093 V3 Renderer because it might break brews where \page does not occur at line start.
Suggestion: implement #629 but only fold \page if at line-start .. rendering would unaffected (yay), folding is available (yay), and the fact a given \page doesn't fold because it isn't at line-start might encourage editorial clean-up.

I've got some pretty big brews as well and definitely agree it would be a helpful addition -- I'm going to start working on this!

@jeddai Great! New PRs are always welcome!

I would limit it to \page boundaries and maybe # Headings down to some depth, but not other objects.

@ericscheid could it make sense to include xml folding for things like <style> blocks?

I agree with @ericscheid that we should (at least to start) limit this to \page separators. Especially since with v3 we are actively trying to discourage literal XML/HTML blocks in favor of our custom Markdown syntax. Similarly, styles should all be contained within the Style tab rather than using <style> tags.

Agree. First step collapsing on \page markers.
Collapsing on headers would be problematic, as they might occur within {{curly-blocks}}.
If anything, being able to collapse {{curly-blocks}} would be the second step.

Fixed by #1792


shellcheck SC1117

shellcheck warning SC1117 triggers on harmless printf format strings. The warning was retired after shellcheck version 0.5. Debian Buster currently distributes shellcheck version 0.5.0. The warning will be suppressed until a newer version of shellcheck is available, at which point the suppression statements will be removed.

See #89.


Question on How the OV-dataset is Created

Could you confirm if the OV-SUN RGB-D and OV-ScanNet datasets provided via the link are updated from the existing base class datasets using the 3D-NOD strategy mentioned in the paper?

Hi, the 3D-NOD strategy is used to identify annotations for novel categories during training. The OV-SUN RGB-D and OV-ScanNet datasets are constructed using annotations from base categories in the original datasets. Therefore, the OV-SUN RGB-D and OV-ScanNet available through the link have not been updated yet. You'll need to update them through training.

Thank you for the quick and kind response!


[Suggestion] Shift + Scrollwheel to run through Configurator/Tool settings

This would be an awesome addition to the controls for the mod. Doesn't specifically need to be Shift+ Scrolling; that was just an example. If there were also a way to specify pipe color on the configurator prior to setting it rather than having to cycle them all every single time, that would just be a bonus.

i am wholeheartedly supporting this! 👍 it would make Life SO much better. Ender IO uses this for their wrench and its simply amazing. even better for your configurator since it has like 3 times more functions.. lol

might be added in the rewrite, added it to my notes

Done!


List of topics in channel

In the legacy app, if you're looking at the messages in a topic, we have a button in the app bar (an "up" icon) that gets you a list of all the topics in that channel. It's sorted by recency, and shows the number of unreads, whether you have an @-mention, and whether the topic is muted etc.
Basically it's similar to what you get in the web left sidebar after hitting "Show all topics" under a given channel.
We should offer the same thing in the new app. I wasn't sure people really use that feature; but it came up in user feedback after yesterday's beta announcement, so that's a sign that people do.
Related issue:

A different feature serving some of the same use cases: #412

Implementation
The list of topics will come from making a request to the server, after opening the page.
As a follow-up task, after getting the list for a given channel we might store it (on ChannelStore), and keep it up to date when handling events, so that coming back a second time to this topic-list page for a given channel is fast. For this PR, though, that's out of scope. Instead the list of topics can just live on a StatefulWidget for the page.
We'll need a design for the page. Until that's ready, this issue won't be ready to implement.

Tim and others propose featuring this page more prominently than the interleaved channel view:
Tim said:

I've never been happy with the legacy app's approach of having the channel feed be what clicking on a channel takes you to by default. We've been moving towards it being a secondary action on channels in the web app, and I think it'd be wise to just make the default click be the list of topics, and channel feed be the thing that's a smaller tap target button.

And another long-time Zulip user said:

In the old app, opening a channel from the channel view shows a feed of all the messages in the channel, but it has a button in the upper right that looks like a list, which opens a view of all topics with a single line for each topic.

In the new app, there doesn't seem to be any way to show all the topics in a channel without showing the messages within those topics.

(In an ideal world, that would be the default view I get when opening a channel.)

And another:

Highest priority for this feature as it feels unnatural to always browse all messages in all topics. Topics can be really long like folks talking about Haiku operating system.
And browsing all of them to cherry pick something is also a pain.


Serve existing app or api

By default, the emulator will serve api and app from the local folder.
This feature will allow developers to skip serving app and api from the local folder, and serve them from existing endpoints (useful if you are working on a dev version of either app or api).

[x] serve from existing app (https://github.com/manekinekko/swa-emulator/commit/2b8b4f8b4a0f7f3b4fe6755d4fd4ddee2e02ef15)
[x] serve from existing api

Added in https://github.com/manekinekko/swa-emulator/commit/398bc596db390c13b3ad324b4b2172be8ecc979f


[dashboard] Fix JS error when position_json data is empty

position_json data used to be array or null. after recent smaller_grid migration it became array or {}.
@mistercrunch @michellethomas

lgtm


Use String interpolation in SQLiteConnection data source name

Motivation and Context
Fix #430
SQLite database is not created with the expected name.
Instead of the expected file name, we get a file name "{filename}" created in the current directory.
This issue occurs because the argumement is not passed correctly to the format function.
Description
The change only modify the SQLiteConnection instance by using the good string interpolation with the argument.
Contribution Checklist

[x] The code builds clean without any errors or warnings
[ ] The PR follows SK Contribution Guidelines (https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
[ ] The code follows the .NET coding conventions (https://learn.microsoft.com/dotnet/csharp/fundamentals/coding-style/coding-conventions) verified with dotnet format
[ ] All unit tests pass, and I have added new tests where possible
[ ] I didn't break anyone :smile:

@adrianwyatt this issue will be resolved by https://github.com/microsoft/semantic-kernel/pull/425

Dupe with https://github.com/microsoft/semantic-kernel/pull/425


Tekton Triggers

Checklist:

[x] Read the Code of Conduct and Contributing Guidelines.

[x] Followed the commit message guidelines.

[x] Stack adheres to Appsody stack structure.

Modifying an existing stack:

[ ] Updated the stack version in stack.yaml

Adding Tekton Triggers artifacts required by Tekton Dashboard created webhooks.
Related Issues:

https://github.com/kabanero-io/kabanero-pipelines/issues/145

shouldn't have directly pushed branch to upstream.


Use exception stack in Py3.7

The exception handling of generators changed significantly in Py3.7. Cython should follow.
See the new CPython implementation in https://github.com/python/cpython/commit/ae3087c6382011c47db82fea4d05f8bbf514265d#diff-6b7f39a70f95725eaff0aa0640cc17f6R127
and the corresponding tracker issue in https://bugs.python.org/issue25612
The previous Cython changes in #1956 and the problem description in #1955 are incomplete.

Duplicate of #1958.

Closing as duplicate of #1958.


No anonymous functions for scheduleOnce or once

In this PR I'm adding a rule to tackle the issue brought up in #57. In this case I'm checking for function uniqueness by looking into the name property of the function and seeing if it is defined.
This rule will only be checked when you call Ember.run.scheduleOnce, run.scheduleOnce, Ember.run.once and run.once. We could potentially also check for just calling scheduleOnce or once but wasn't to sure about which route we wanted to take this.

@trentmwillis Thanks for taking a look and sorry for abandoning this PR for some time after you left the review. I've addressed your comment and also added code to cover for the case sin which  scheduleOnce and once are invoked with call and apply. Let me know what you think whenever you have some time. Thanks!


Dashboard docker shows no data

Describe the bug
Some user reported that Dashboard docker  shows no data by following instruction of dashboard
To Reproduce
Steps to reproduce the behavior:

following instruction of dashboard to start docker.

Expected behavior
dashboard could show data
Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS]

Closed it since it is related to advertisedAddress.


fix: stop automatic sorting for --help

What type of PR is this?

bug

What this PR does / why we need it:

Wrongly added catecory "" to each flag automatically, causing --help to sort automatically
Which issue(s) this PR fixes:

Fixes #1420
Special notes for your reviewer:

Testing

Release Notes

Add category only for flags that declare "category"

@FGYFFFF Thank you! Are you able to add some tests around this, too? If not, I'm happy to pick up the work.
@meatballhat  I will add unit tests later.

@meatballhat  I will add unit tests later.

Did you mean you will be adding tests as part of this PR?

Tested this PR with my project. Unfortunately, it does not fix the original issue. Disabling sort.Strings here and (https://github.com/urfave/cli/blob/0c9527f4b7f5618832c90d8b6c1ead4b7b1e2a80/category.go#L126) worked.

Tested this PR with my project. Unfortunately, it does not fix the original issue. Disabling
https://github.com/urfave/cli/blob/0c9527f4b7f5618832c90d8b6c1ead4b7b1e2a80/category.go#L163
and
https://github.com/urfave/cli/blob/0c9527f4b7f5618832c90d8b6c1ead4b7b1e2a80/category.go#L126
worked.

If your flag does not define "category", then the displayed flags will be in the order you define them. If your flag defines "category" then the flags will be grouped and sorted according to "category".

Tested this PR with my project. Unfortunately, it does not fix the original issue. Disabling
https://github.com/urfave/cli/blob/0c9527f4b7f5618832c90d8b6c1ead4b7b1e2a80/category.go#L163
and
https://github.com/urfave/cli/blob/0c9527f4b7f5618832c90d8b6c1ead4b7b1e2a80/category.go#L126
worked.

If your flag does not define "category", then the displayed flags will be in the order you define them. If your flag defines "category" then the flags will be grouped and sorted according to "category".

I've tried to reproduce the original issue using the following steps:
go.mod points to this commit:
replace github.com/urfave/cli/v2 => github.com/FGYFFFF/cli/v2 v2.10.4-0.20220702104559-e27277fa75de

Flags are defined in the following order: host, port, path, health-path.
Here's what I see on -h:
   --health-path value  HTTP health endpoint path (default: /health) [$ANYCABLE_HEALTH_PATH]
   --help, -h           show help (default: false)
   --host value         Server host (default: localhost) [$ANYCABLE_HOST]
   --path value         WebSocket endpoint path (you can specify multiple paths using comma as separator) (default: /cable) [$ANYCABLE_PATH]
   --port value         Server port (default: 8080) [$PORT, $ANYCABLE_PORT]
   --version, -v        print the version (default: false)

Here's what I see when I comment line 179 in category.go (sort.Strings in *defaultVisibleFlagCategory Flags()):
   --host value         Server host (default: localhost) [$ANYCABLE_HOST]
   --port value         Server port (default: 8080) [$PORT, $ANYCABLE_PORT]
   --path value         WebSocket endpoint path (you can specify multiple paths using comma as separator) (default: /cable) [$ANYCABLE_PATH]
   --health-path value  HTTP health endpoint path (default: /health) [$ANYCABLE_HEALTH_PATH]
   --help, -h           show help (default: false)
   --version, -v        print the version (default: false)

Works the same with either category defined or blank.

I don't think we're talking about a problem. If you declare a flag that defines a "category", then the flags under that "category" grouping need to be sorted. For example:
idlFlag := cli.StringSliceFlag{Name: "idl", Usage: "Specify the IDL file path. (.thrift or .proto)"}
moduleFlag := cli.StringFlag{Name: "module", Aliases: []string{"mod"}, Usage: "Specify the Go module name to generate go.mod.", Destination: &globalArgs.Gomod, **Category**:"fgy"}
serviceNameFlag := cli.StringFlag{Name: "service", Usage: "Specify the service name.", Destination: &globalArgs.ServiceName}

When "moduleFlag" declares "category", all flags will be sorted for below
OPTIONS:
   --idl value                       Specify the IDL file path. (.thrift or .proto)  (accepts multiple inputs)
   --service value                   Specify the service name.

   fgy
   --module value, --mod value  Specify the Go module name to generate go.mod.

But if you don't specify "category", then all flags should not be sorted：
idlFlag := cli.StringSliceFlag{Name: "idl", Usage: "Specify the IDL file path. (.thrift or .proto)"}
moduleFlag := cli.StringFlag{Name: "module", Aliases: []string{"mod"}, Usage: "Specify the Go module name to generate go.mod.", Destination: &globalArgs.Gomod}
serviceNameFlag := cli.StringFlag{Name: "service", Usage: "Specify the service name.", Destination: &globalArgs.ServiceName}

The flags will be displayed in the defined order：
OPTIONS:
   --idl value                       Specify the IDL file path. (.thrift or .proto)  (accepts multiple inputs)
   --service value                   Specify the service name.
   --module value, --mod value  Specify the Go module name to generate go.mod.

But if you don't specify "category", then all flags should not be sorted：

That is correct. I have defined my flags in the following order: host, port, path, health-path. Category is empty. I see them in the help output sorted alphabetically: health-path, host, path, port.
Besides that, I am not so sure that it's a good idea to sort flags within a category. I'd like to control that order somehow in my app. Having a flag which controls this behaviour would be great.

But if you don't specify "category", then all flags should not be sorted：
idlFlag := cli.StringSliceFlag{Name: "idl", Usage: "Specify the IDL file path. (.thrift or .proto)"}
moduleFlag := cli.StringFlag{Name: "module", Aliases: []string{"mod"}, Usage: "Specify the Go module name to generate go.mod.", Destination: &globalArgs.Gomod}
serviceNameFlag := cli.StringFlag{Name: "service", Usage: "Specify the service name.", Destination: &globalArgs.ServiceName}

The flags will be displayed in the defined order：
OPTIONS:
   --idl value                       Specify the IDL file path. (.thrift or .proto)  (accepts multiple inputs)
   --service value                   Specify the service name.
   --module value, --mod value  Specify the Go module name to generate go.mod.

In my understanding, this example is not exactly correct: flags are defined in the following order: idl, module, service, but the help order is idl, service, module.

But if you don't specify "category", then all flags should not be sorted：

That is correct. I have defined my flags in the following order: host, port, path, health-path. Category is empty. I see them in the help output sorted alphabetically: health-path, host, path, port.
Besides that, I am not so sure that it's a good idea to sort flags within a category. I'd like to control that order somehow in my app. Having a flag which controls this behaviour would be great.

But if you don't specify "category", then all flags should not be sorted：
idlFlag := cli.StringSliceFlag{Name: "idl", Usage: "Specify the IDL file path. (.thrift or .proto)"}
moduleFlag := cli.StringFlag{Name: "module", Aliases: []string{"mod"}, Usage: "Specify the Go module name to generate go.mod.", Destination: &globalArgs.Gomod}
serviceNameFlag := cli.StringFlag{Name: "service", Usage: "Specify the service name.", Destination: &globalArgs.ServiceName}

The flags will be displayed in the defined order：
OPTIONS:
   --idl value                       Specify the IDL file path. (.thrift or .proto)  (accepts multiple inputs)
   --service value                   Specify the service name.
   --module value, --mod value  Specify the Go module name to generate go.mod.

In my understanding, this example is not exactly correct: flags are defined in the following order: idl, module, service, but the help order is idl, service, module.

Yes, I agree with you:  "category" should not be sorted automatically. But you can look at the implementation of category. The implementation of category uses "map", which results in a different order each time the map is traversed, so category is sorted once by default

@meatballhat Hey, friend！！！ I've thought about this piece of code and I don't think I need to write unit tests, can I merge it in? My code currently relies on this feature.

@FGYFFFF Without unit tests we cannot approve this PR since this is a behavioral change. If this is expected behaviour this needs to be in a unit test so that we can confirm that some other change doesnt break this down the road.

Interested in this as well

@meatballhat Can you take a look at test-docs ?

@FGYFFFF @dearchap From what I can tell, the failure in the test-docs step is legitimate. This is the diff of what I have locally to fix it:
diff --git a/docs/v2/examples/flags.md b/docs/v2/examples/flags.md
index dbb41ea..44c643a 100644
--- a/docs/v2/examples/flags.md
+++ b/docs/v2/examples/flags.md
@@ -239,7 +239,7 @@ For example this:

 <!-- {
   "args": ["--help"],
-  "output": ".*Load configuration from FILE\n.*\n.*Language for the greeting.*"
+  "output": ".*Load configuration from FILE\n.*Language for the greeting.*"
 } -->
 ```go
 package main

@meatballhat Update the flags.md. All tests passed.


Fixed Issue 870, 871

Summary of issue
#870 and #871
Summary of change
Fix all issues

Error labels is doubled

 Error labels is doubled

Done

Modal form isn't centred vertically
Somehow I can submit form without a message and/or email (it just closes)

Fix comments.
And please don't combine several bugs in 1 pull request because #871 is fixed but we can't merge if because of #870 is present in this pull request

This has been fixed by BohdanBybliv. And then I've merged his branch to this.


ThorVG: update from v0.12.1 to v0.12.3

https://github.com/thorvg/thorvg/releases/tag/v0.12.3

Full Changelog: https://github.com/thorvg/thorvg/compare/v0.12.1...v0.12.3

Godot-related SVG bug fixes:

svg_loader: Add missing transform functions skewX and skewY.
thorvg/thorvg#1928
sw_engine: Rectified dash line drawing issue.
thorvg/thorvg#1932

Thanks!

Cherry-picked for 4.2.2.


Service to get list of semantic instances

Added a service to query the list of semantic instances in the map.
Namely, for all recognized instances in the scene the service returns an array of strings where each string is a human-readable label of the corresponding semantic class.
Also alphabetically reordered packages in package.xml

Test FAILed.

Pardon my ignorance on the matter as this is the first time I am looking at this project.
The service returns a list of strings which correspond to labels of the objects. Should there be a way to query the map for a bounding volume/pointcloud/other information based on the returned list?
The use case I have in mind is that a client node first requests a list of instances in the scene and then decides it wants more information about a particular returned instance. Maybe the service could return a list of instances associated with ids. The id could, for example, be used to query the map for the location of the instance.
How would you go about supporting such a use case? Are there other apis which enable something like this?

@kekeblom Your point is a very valid one :)
In fact, as soon as I started writing the service to return a desired object's pointcloud, I realized that passing in an instance id (rather than a semantic category) would simplify and disambiguate things immensely.
Give me some time to update the code in this PR by sending out both a list of instance IDs, and their corresponding semantic categories, I'll ping you when it's done.

Test PASSed.

Closing as it is contained in https://github.com/ethz-asl/voxblox-plusplus/pull/59.


ci: add clippy job to check for linting issues

Can you please revert the gratuitous style changes?

I'd rather not have this as a blocking CI check, to be honest.

Okay, it's up to you.


Fix vulnerability in Server Rendering recipe

https://twitter.com/ericf/status/636752569025622017
@timdorr
@hedgerh

Looks like the aforementioned library has some security issues too: yahoo/serialize-javascript#5
Does anyone know any other possible sanitization methods/libraries we could use in this case? There's gotta be something in npm...

@timdorr You should look at the full issue, it might cause an error on the server, but unlikely to result in an XSS to the browser. Most Yahoo sites have been using this code for years.

Yeah, I'm trying to grok where the security issue lives. We're serializing an internal state variable, not raw input from the user. The only input is the counter variable, which I parseInt on anyways.
I'm probably missing something. What should I be looking for?

The example is fine how it is today, the problem is that people will use this pattern in their apps that do have UGC and open an XSS attack vector. Since the examples serve as docs, my suggestion on Twitter was to use something that will help people holster this footgun.

That's certainly true, but the use of JSON.stringify here isn't on anything user generated that hasn't already been sanitized. Suggesting we use a generalized function to sanitize input, rather than looking at the input directly, is dangerously hand wave-y.
I can certainly add some text to the docs warning of the potential pitfalls of accepting user input. In particular, emphasizing the importance of the parseInt call and other sanitization methods on input.
Another good idea would be to not just pass in initial state, but instead dispatch some actions to an empty store to populate it. Then you could make sure your reducers are hardened, which applies to both the server and the client.

Defense in depths. Surely sanitize input values, but the context here is outputting inside a <script> and that requires special sanitation and escaping so browsers don't prematurely interpret a closing </script> — do both.
Outputting inside a <script> is the exact use case for why this package was written and has helped to patch XSS vectors exploited in production.

That is true. I probably wasn't getting the <script> output in JSON taking priority over the JS itself.
I think a security section towards the end is warranted. Especially since universal rendering opens up the attack surface area. I'll see about adding that in tonight.

Please note that the issues I found are very very unlikely to be exploitable. Just some edge cases that could be better handled

Just added #641 to (hopefully) address this.


Cannot pair

I've got everything to run up to the point where my ESP8622 with ESP8622-HomeKit-Demo is discoverable in the Home app. The app then asks me for the pairing code, which for me still is the default one.
However, the procss doesn't finish after 25 seconds for me as the tutorial states. It instead times out after 120 seconds, when the app tells me it cannot connect to the device. I tried to increase the clock speed to 80MHz in the build process, thinking that would help speeding up the crypto but to no avail.
Is there anything I can do to debug this process? How do I compile the code with debugging on?
I am, by the way on the current iOS 11 beta with all my devices, is anyone else experiencing problems on these?

You must manage to run a hello-world on your device and get serial reporting in place
My code delivers tons of debug info and without it no-one can help you.
You actually must change something to switch it off.
Meanwhile, the other day I could not pair anymore, and found that the only way to get it working again was to delete my ‘home’ in the Home-app on the phone.
Maybe this applies to you as well, but you MUST get serial working, else there is no point in working with this software...

Now thats weird. I've just tried to connect again after leaving the controller powered up yesterday. It worked like a charm. It just worked as if I didn't try and failed at doing the same thing, a dozen times from two different devices yesterday.

Good to hear that you got success.
please open a new issue with this new question, to keep things clean

Actually, put that as an issue on the Demo page. Myself I have not too much time to test out this kind of stuff (alas) but others might share their experiences.

Hi,
I managed to pair the demo accessory only once, since then no matter what I do, I always get ''.
I tried erasing the flash, resetting HomeKit configuration on my iPhone but nothing is working.
I am attaching the logs.... please help.
logs.txt

Hi, I think I had the same issue myself (and wondering if I went crazy).
My 'solution' in the end was to delete the 'Home' and start over. Then it worked.
Did you sync recently? I have added the fix to have a random Username (looks like a MAC) which is supposed to be according to spec. That could also help...
Good luck and report back?

I cloned you repository today itself and the random username generation is working fine every time I erase the flash... also I delete 'home' from home app but still getting the 'client <IP_ADDRESS>:49242 disconnected with status 0' issue... don't know why only I am facing it... is there a way to know exact reason of disconnection from the SDK API calls ??

Since it is the iPhone disconnecting we have to ask Apple… I did not see an error in the log
Sometimes life is hard, fighting your way through will learn you a lot. But no guarantees I’m afraid.

On 26 Aug 2017, at 22:26, abhinavjha86<EMAIL_ADDRESS>wrote:
I cloned you repository today itself and the random username generation is working fine every time I erase the flash... also I delete 'home' from home app but still getting the 'client <IP_ADDRESS>:49242 disconnected with status 0' issue... don't know why only I am facing it... is there a way to know exact reason of disconnection from the SDK API calls ??
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/HomeACcessoryKid/ESP8266-HomeKit/issues/28#issuecomment-325159925, or mute the thread https://github.com/notifications/unsubscribe-auth/AV0HQhaV727E8-ifeyv0t3cmv-yFyjpiks5scH9ZgaJpZM4O8bMk.

true... I'll keep trying,, anyways thanks for this great effort of yours,, looking forward for the Arduino library... wish I could also contribute but I am limited by my knowledge ;-)

@abhinavjha86 it seems like you might be losing packets, but i am not so sure about it.
Could you try again with my branch and then attach the output with DEBUG0,1,2,3,4 defined?

@abhinavjha86 it might indeed be packets that are lost. My home wifi is also not OK and I see disconnects all the time. This is not something I consider a problem since it will be part of life of many people. The thing is that even though this happens, the long term stability is not jeopardised. Actually, the last few months of the original repository were spend on making everything robust, explaining some of the more obscure parts of the code.
Can you confirm that in itself your device works well (part of the time) or you have use cases that NEVER work? My testcase is setting the 'light' to 50% and see if a week later it still is there. Like that you can know if it has reset, bringing the 'light' back to 0%.

IMPORTANT
When working on the RGBW lamp I have found "THE BUG" that must have made pairing a matter of luck for many. The change I have pushed one hour ago solves this issue where packets were ignored, and especially in the case of pairing. It has to do with timing and the cause was a delay that I introduced myself in crypto_tasks... I recommend everyone to sync up to at least this version.
At the same time there is some dark magic that whenever syncing was not clean, deleting the 'home' seems to be the only remedy...
PS. I will investigate how to fix the WPA2 vulnerability... ESP8266_RTOS has changed some stuff in a not backward compatible way so I will go to a separate branch I guess.


Adding some helpful information

[Enter feedback here]
Hi, We didn't find the correct TenantName because we thought it was together with the Tenant Status information in Azure. We wrongly used the Primary Domain name... Perhaps it would be better in this documentation to include where to find the TenantName (link to here https://portal.azure.com/#blade/Microsoft_AAD_IAM/ActiveDirectoryMenuBlade/Domains)
Just a suggestion, thanks
-Tor

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

ID: 2928e23e-3210-9e74-9cdb-1a19b703eac7
Version Independent ID: 44e303ca-e4bc-ba30-5aae-311aa9318278
Content: Controlled validation of hybrid Azure AD join - Azure AD
Content Source: articles/active-directory/devices/hybrid-azuread-join-control.md
Service: active-directory
Sub-service: devices
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@TorATB
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Thanks for the suggestion will try and add in a future revision. #please-close


New functions ?

This plugin has very useful functions, but there can be added even more, every thing has its own usage.
native SetTimerInterval(timerid, interval);
native SetTimerCount(timerid, count);
I know we can recreate the timer, but with these would be easier.

A new idea: what about SetTimerIntervalLeft ? (we already have a Get function)

Hi, @IstuntmanI,
I have no plans of continuing the development of this plugin or anything related to SA-MP. If you'd like, you can fork it and implement it by yourself. It shouldn't be hard and you can peak at other methods that do the same thing.
It should look like this:
cell AMX_NATIVE_CALL Natives::SetTimerIntervalLeft(AMX *amx, cell *params)
{
    if (params[0] < 2 * CELL_SIZE)
    {
        return 0;
    }
    int id = params[1], interval = params[2];
    if (TimerExists(id))
    {
        timers[id]->next = GetMsTime() + interval;
    }
    return 1;
}

Ugh, sucks to hear that. I will see what I can do. (I can't really compile for linux)

I am sure you can find a way to compile for Linux if you really want to.
If you don't want to install it on your computer or run a virtual machine, you can rent a VPS or use one of those online IDEs (e.g. Cloud9 or Koding) that provide a free virtual machine with enough resources to compile this plugin.
Though, I am not sure about testing (if you can run a SA-MP server on any of these servers).


Feature Request: Multiple Image Display - Enabling image caching

Enable image caching across the images. Currently this is pulling from API with each display. Caching images in the tmpFS would allow us to disable timeouts if using only cached images. #359

on quick perusal, I don't see the timeout implementation and I'm assuming code to cache images in tmpFS would be implemented here?
https://github.com/chtaylo2/Roku-GooglePhotos/blob/master/components/Screensaver/Screensaver.brs


Support localization of non-english numbers

Currently, English numbers will be localized as something like "You've cleared five levels!" or "You've cleared 8,120 levels!". But Brazilian Portuguese numbers will also be localized as something like "você limpou five níveis!" or equally problematically, "você limpou 8,120 níveis!"
Our current 'number localization' algorithm is English-centric, and has reserved words for the numbers 1-20, and converts other numbers by adding commas between them. But languages like Brazilian Portuguese use periods instead of commas. For other languages like French or Japanese, it might not make sense to treat the numbers 1-20 as special numbers -- it might make more sense to only have special words for the numbers 1-12, or to extend it to cover the numbers 1-40.
This is a problem other people have certainly solved for Godot games, so we should investigate their solutions and apply them to Turbo Fat.

This problem applies to many different utility functions in StringUtils, including english_number, format_duration, comma_sep and more.

The numbers one through twenty are now localized.


Docker error on clipper_conn.start_clipper()

I received the "Docker not running error" even when docker was running, looked as follows:
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.5/dist-packages/clipper_admin/clipper_admin.py", line 130, in start_clipper
    raise e
  File "/usr/local/lib/python3.5/dist-packages/clipper_admin/clipper_admin.py", line 114, in start_clipper
    cache_size, num_frontend_replicas)
  File "/usr/local/lib/python3.5/dist-packages/clipper_admin/docker/docker_container_manager.py", line 116, in start_clipper
    raise ClipperException(msg)
clipper_admin.exceptions.ClipperException: Unable to Connect to Docker. Please Check if Docker is running.```

Can someone help?

To resolve this, I had to sudo into the python console.
sudo python3

@sentient-soellingeraj :
I think that you must permit to manage Docker as a non-root user.
https://docs.docker.com/install/linux/linux-postinstall/#manage-docker-as-a-non-root-user

@sentient-soellingeraj  i have run python3  from root  but  still have that problem

Closed.


[docs] Adding table of contents on multiple pages

@Naros, can you review and merge. I don't think an issue is needed, it's still "typo fix" level :) Thanks!

LGTM