How to get id and value on event trigger?

My json:
[
    {id:1, name:'paul'},
    {id:2, name:'jim'},
    {id:3, name:'tom'},
    {id:4, name:'medor'},
    {id:5, name:'janzy'}
]

instance:
var bh = new Bloodhound({
    datumTokenizer: Bloodhound.tokenizers.obj.whitespace('name'),
    queryTokenizer: Bloodhound.tokenizers.whitespace,
    prefetch: '//localhost/data/names.json',
    identify: function(datum){
        return datum.id;
    }
});

$('#name').typeahead({"highlight":true}, {
    "name":"name",
    "source":bh,
    "display":"value",
    "limit":10
})

event triggers:
$('#name').bind('typeahead:select', function(ev, suggestion){
    console.log(suggestion);
})

suggestion is an object which contain only the text value (i.e. name). How do i obtain the id?

I'd say:
console.log(suggestion.id);
You'll probably need to define a template or set the display to make things work properly.
"display":"name",

I'd say:
console.log(suggestion.id);

You'll probably need to define a template or set the display to make things work properly.
 "display":"name",

Actually there was something weird going with the browser cache. it cached my first json which did not contain name and id, just the values. i had to clear those content and then i could do console.log(suggestion.id);.
Thanks anyways.

there are some issues with event. i don't know why.

Yes there are issue with the last version of the plugin. I went back to version 0.10.5 which was working for me without issues.
@Locustv2 : you're welcome. Maybe set TTL:0 while testing to prevent such issues.


ppc64le: //tensorflow/python/kernel_tests:atrous_conv2d_test test fails

Please assign this issue to me and add the tag: stat:community support
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes, Have written a Dockerfile.gpu.ppc64le file and modified ci_parameterized_build.sh to allow for build/test runs on ppc64le. (Will submit as a PR)

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): ppc64le Ubuntu 16.04.4

TensorFlow installed from (source or binary): source

TensorFlow version (use command below): master from may 30th

Python version:  2.7.12

Bazel version (if compiling from source): 0.11.0

GCC/Compiler version (if compiling from source): 5.4.0

CUDA/cuDNN version: 9.2.88, 7

GPU model and memory: 2 P100 GPUs with 16 GB of memory each

Exact command to reproduce:
bazel test --config=cuda -c opt --local_test_jobs=2 --cache_test_results=no --run_under=//tensorflow/tools/ci_build/gpu_build:parallel_gpu_execute //tensorflow/python/kernel_tests:atrous_conv2d_test

Describe the problem
----------------------------------------------------------------------
Ran 4 tests in 14.613s

FAILED (failures=1)
atrous_conv2d gradient err = 0.000759065
atrous_conv2d gradient err = 0.000683963
atrous_conv2d gradient err = 0.000596046
not close where =  (array([0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1]), array([0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 0, 0, 1, 1]), array([0, 1, 1, 2, 2, 3, 4, 5, 7, 8, 9, 9, 0, 0, 0, 1, 0, 1]), array([1, 0, 1, 0, 1, 0, 1, 0, 0, 0, 0, 1, 0, 1, 0, 0, 0, 0]))
not close lhs =  [   3.    3.   13.   10.   30.   30.  113.  113.  221.  275.  310.  410.
   19.   93.  163.  165.  181.  183.]
not close rhs =  [   2.38105154    2.80899453   12.87652969   10.06372643   29.96479416
   30.14238167<PHONE_NUMBER>7<PHONE_NUMBER>6<PHONE_NUMBER>9<PHONE_NUMBER>5
<PHONE_NUMBER>8<PHONE_NUMBER>    19.19488144   93.29485321<PHONE_NUMBER>2
<PHONE_NUMBER>2<PHONE_NUMBER>1<PHONE_NUMBER>5]
not close dif =  [ 0.61894846  0.19100547  0.12347031  0.06372643  0.03520584  0.14238167
  0.12899017  0.32045746  0.52029419  0.45703125  0.39041138  0.5213623
  0.19488144  0.29485321  1.18998718  0.20986938  0.79176331  0.26371765]
not close tol =  [ 0.00338105  0.00380899  0.01387653  0.01106373  0.0309648   0.03114238
  0.114129    0.11432046  0.22252031  0.27554297  0.3113904   0.41152138
  0.02019488  0.09429486  0.16281003  0.16579014  0.18279177  0.18426372]
dtype = float32, shape = (2, 13, 13, 2)
================================================================================
Target //tensorflow/python/kernel_tests:atrous_conv2d_test up-to-date:
  bazel-bin/tensorflow/python/kernel_tests/atrous_conv2d_test
INFO: Elapsed time: 67.037s, Critical Path: 46.28s
INFO: Build completed, 1 test FAILED, 3 total actions
//tensorflow/python/kernel_tests:atrous_conv2d_test                      FAILED in 1 out of 2 in 16.1s
  Stats over 2 runs: max = 16.1s, min = 4.3s, avg = 10.2s, dev = 5.9s
  WARNING: //tensorflow/python/kernel_tests:atrous_conv2d_test: Test execution time (4.3s excluding execution overhead) outside of range for MODERATE tests. Consider setting timeout="short" or size="small".
  /root/.cache/bazel/_bazel_root/eab0d61a99b6696edb3d2aff87b585e8/execroot/org_tensorflow/bazel-out/ppc-opt/testlogs/tensorflow/python/kernel_tests/atrous_conv2d_test/shard_1_of_2/test.log

Executed 1 out of 1 test: 1 fails locally.

Source code / logs
atrous_conv2d_test.log

@sandipmgiri any idea?

Currently I'm working on CPU support only for TF-master, and this test is passing successfully without any error on Ubuntu:16.04(ppc64le).  Need to check for GPU support.

Nagging Assignee @drpngx: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

@wdirons looks like we cannot repro. Closing.


Not respecting Astro's excluding routing rule

Routing 🚀 Astro Documentation

You can exclude pages or directories from being built by prefixing their names with an underscore (_). Files with the _ prefix won’t be recognized by the router and won’t be placed into the dist/ directory.

Actual builds:
├── cn
│   ├── _about
│   │   └── index.html
│   ├── _blog
│   │   └── index.html
│   └── index.html
├── en
│   ├── _about
│   │   └── index.html
│   ├── _blog
│   │   └── index.html
│   └── index.html

Follow https://github.com/jlarmstrongiv/astro-i18n-aut/pull/30 for updates


Update nvALT.app to support macOS Sierra

This is an update to the nvALT cask version 2.2120.
nvALT was updated to support macOS Sierra and this update is compatible
with versions of OS X >=10.9. The previous version os nvALT was
preserved for those users with OS X < 10.9.

@vitorgalvao the requested changes have been made. Thanks for the review.

Thank you.


Add download status progress bar

Adding a status bar to show the progress the stream would be great.

As of 411d34e9ba3986038f551f5b5206cdb522a59361, stream progress is now shown along with the ongoing transfer speed.


Add response rewrite to dns-search-path-optimization.

How to categorize this PR?

/area networking
/kind feature
What this PR does / why we need it:
As some clients/DNS resolver validate the hostname in a DNS response, the hostname in the answer should be rewritten.
As the namespace is not known in all cases, we do not rewrite the following requests:

app.namespace.svc.namespace.svc.cluster.local. In this case the 3rd entry in the seach-path will match anyway.
custom.example.com.default.svc.cluster.local. This request will not result in answer. However the next case in the search path (custom.example.svc.cluster.local) can be rewritten.

Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

Add response rewrite to dns-search-path-optimization.

/cc @ScheererJ


feat(contracts): add implementation deploy script

Purpose or design rationale of this PR
This PR add deployment script for bridge optimization PR.
PR title
Your PR title must follow conventional commits (as we are doing squash merge for each PR), so it must start with one of the following types:

[x] feat: A new feature

Deployment tag versioning
Has tag in common/version.go been updated or have you added bump-version label to this PR?

[x] No, this PR doesn't involve a new deployment, git tag, docker image tag
[ ] Yes

Breaking change label
Does this PR have the breaking-change label?

[x] No, this PR is not a breaking change
[ ] Yes

Hi @zimpha, do we need to merge this PR?

Hi @zimpha, do we need to merge this PR?

I think we can close this PR.


Question websockets try2

connected question and answers from backend to question page, voting is still missing and connection to ranking as well

works with API-Game-setup or marlens-copy-of-game-api

closes #41 closes #42


Feature/license and dependency

Why? - Currently, we download a 2 line file from the buildah git repo when we build the runner image. It's a generic setting. It's better to just check that file into our repo so we don't need to mirror the buildah repo when we build disconnected.
The upstream file is covered under the Apache 2.0 license, and this repo is currently unlicensed :scream: so I also added the apache license.
Here's a green pipeline showing this doesn't break anything - https://github.com/ploigos/spring-petclinic/actions/runs/2804171836

Merging this myself because it's trivial and reviewers are either on PTO or about to be on PTO in a few hours and doing something more important meantime :cowboy_hat_face: .


Add metric yace_cloudwatch_getmetricdata_metrics_total

https://github.com/nerdswords/yet-another-cloudwatch-exporter/issues/339
I saw the issue above but seems that the conclusion is incorrect.
GetMetricData of CloudWatch API costs by the number of its returned metrics.
So it would be worth having how much data is fetched via GetMetricData.

Is there something missing here? Would be nice to get this merged as currently, there is no way to properly track costs

+1 to this feature.
The description in the README.md to track costs is wrong, which we can fix after landing this PR. I'm happy to test and update the docs myself.


The name 'input:0' refers to a Tensor which does not exist. The operation, 'input', does not exist in the graph.

Hi,
thanks for this project.
I used tensorflow 1.7.0 with GPU and get this error, it seems something wrong with the graph.
File "/home/ubuntu/.local/lib/python3.5/site-packages/tensorflow/python/framework/ops.py", line 3478, in as_graph_element
return self._as_graph_element_locked(obj, allow_tensor, allow_operation)
File "/home/ubuntu/.local/lib/python3.5/site-packages/tensorflow/python/framework/ops.py", line 3520, in _as_graph_element_locked
"graph." % (repr(name), repr(op_name)))
KeyError: "The name 'input:0' refers to a Tensor which does not exist. The operation, 'input', does not exist in the graph."
During handling of the above exception, another exception occurred:
When i use the docker container it works. I would like to implement and share a save function from the trained new model.
Thanks for any help.

During handling of the above exception, another exception occurred:

Dear btwardow,
thank you very much for your help. I spent some more time and got the native version (without docker) of face detection running.
I had to change in embedding.py - in function
def embedding(face_np):
global sess
#images_placeholder = tf.get_default_graph().get_tensor_by_name("input:0")
#embeddings = tf.get_default_graph().get_tensor_by_name("embeddings:0")
#phase_train_placeholder = tf.get_default_graph().get_tensor_by_name("phase_train:0")
images_placeholder = sess.graph.get_tensor_by_name("input:0")
embeddings = sess.graph.get_tensor_by_name("embeddings:0")
phase_train_placeholder = sess.graph.get_tensor_by_name("phase_train:0")

...
Face Detection is now working in the browser. When I capture the faces, I can see the thumbnail, but in my terminal console i saw:
) -> physical GPU (device: 0, name: Tesla V100-PCIE-16GB, pci bus id: 0000:00:06.0, compute capability: 7.0)
Traceback (most recent call last):
File "server.py", line 49, in detect
faces = recognize(detection.get_faces(image, threshold))
File "/opt/tf-face-recognition/tensorface/recognition.py", line 21, in recognize
result = classifier.predict(X)
File "/opt/tf-face-recognition/tensorface/classifier.py", line 118, in predict
return model.predict(X)
File "/opt/tf-face-recognition/tensorface/classifier.py", line 88, in predict
result.append((UNKNOWN_CLASS, None, None, None))
File "/home/ubuntu/.local/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 1545, in exit
self._default_graph_context_manager.exit(exec_type, exec_value, exec_tb)
File "/usr/lib/python3.5/contextlib.py", line 77, in exit
self.gen.throw(type, value, traceback)
File "/home/ubuntu/.local/lib/python3.5/site-packages/tensorflow/python/framework/ops.py", line 5086, in get_controller
yield g
File "/usr/lib/python3.5/contextlib.py", line 77, in exit
self.gen.throw(type, value, traceback)
File "/home/ubuntu/.local/lib/python3.5/site-packages/tensorflow/python/framework/ops.py", line 4896, in get_controller
yield default
File "/home/ubuntu/.local/lib/python3.5/site-packages/tensorflow/python/framework/ops.py", line 5086, in get_controller
yield g
File "/opt/tf-face-recognition/tensorface/classifier.py", line 68, in predict
self.dist_threshold: self.dist_threshold_value})
File "/home/ubuntu/.local/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 905, in run
run_metadata_ptr)
File "/home/ubuntu/.local/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 1065, in _run
raise RuntimeError('The Session graph is empty.  Add operations to the '
RuntimeError: The Session graph is empty.  Add operations to the graph before calling run().
[2018-09-22 21:53:54,353] ERROR in app: Exception on /detect [POST]
Traceback (most recent call last):
File "server.py", line 49, in detect
faces = recognize(detection.get_faces(image, threshold))
File "/opt/tf-face-recognition/tensorface/recognition.py", line 21, in recognize
result = classifier.predict(X)
File "/opt/tf-face-recognition/tensorface/classifier.py", line 118, in predict
return model.predict(X)
File "/opt/tf-face-recognition/tensorface/classifier.py", line 88, in predict
result.append((UNKNOWN_CLASS, None, None, None))
File "/home/ubuntu/.local/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 1545, in exit
self._default_graph_context_manager.exit(exec_type, exec_value, exec_tb)
File "/usr/lib/python3.5/contextlib.py", line 77, in exit
self.gen.throw(type, value, traceback)
File "/home/ubuntu/.local/lib/python3.5/site-packages/tensorflow/python/framework/ops.py", line 5086, in get_controller
yield g
File "/usr/lib/python3.5/contextlib.py", line 77, in exit
self.gen.throw(type, value, traceback)
File "/home/ubuntu/.local/lib/python3.5/site-packages/tensorflow/python/framework/ops.py", line 4896, in get_controller
yield default
File "/home/ubuntu/.local/lib/python3.5/site-packages/tensorflow/python/framework/ops.py", line 5086, in get_controller
yield g
File "/opt/tf-face-recognition/tensorface/classifier.py", line 68, in predict
self.dist_threshold: self.dist_threshold_value})
File "/home/ubuntu/.local/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 905, in run
run_metadata_ptr)
File "/home/ubuntu/.local/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 1065, in _run
raise RuntimeError('The Session graph is empty.  Add operations to the '
RuntimeError: The Session graph is empty.  Add operations to the graph before calling run().
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/ubuntu/.local/lib/python3.5/site-packages/flask/app.py", line 2292, in wsgi_app
response = self.full_dispatch_request()
File "/home/ubuntu/.local/lib/python3.5/site-packages/flask/app.py", line 1815, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/home/ubuntu/.local/lib/python3.5/site-packages/flask/app.py", line 1718, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "/home/ubuntu/.local/lib/python3.5/site-packages/flask/_compat.py", line 35, in reraise
raise value
File "/home/ubuntu/.local/lib/python3.5/site-packages/flask/app.py", line 1813, in full_dispatch_request
rv = self.dispatch_request()
File "/home/ubuntu/.local/lib/python3.5/site-packages/flask/app.py", line 1799, in dispatch_request
return self.view_functionsrule.endpoint
File "server.py", line 69, in detect
print('POST /detect error: %e' % e)
TypeError: a float is required

Please help me out with the error.. Even am facing the same..

I am having now same issue:
tensorboard==1.13.1
tensorflow==1.13.1
tensorflow-estimator==1.13.0
tensorflow-gpu==1.13.1

Related to #10

Related to #9 - all these are having same root cause seems to me.

To reproduce such issue I did "pipreqs ." to get just project requirments and this is result:
six==1.11.0
requests==2.21.0
Flask==1.0.2
numpy==1.16.1
tensorflow_gpu==1.13.1
Pillow==6.0.0
scikit_learn==0.21.1
tensorflow==1.13.1
typing==3.6.6

Can someone provide versions where this project is stable and runnable?

So I run docker now and examine the environment, so we have there following:
Python 3.5.2 vs local 3.6.5
Packages;
numpy==1.14.2
Flask==0.12.2
opencv_python==<IP_ADDRESS>
tensorflow==1.7.0
requests==2.18.4
six==1.11.0
Pillow==6.0.0
scikit_learn==0.21.1
typing==3.6.6

So when I reverted locally all versions via pip to what I find in docker container loaded in docker hub:
I get error on startup, but the application starts ok:
angetsu@andromeda ~/proj/prometheus-core/tf-face-recognition/server $ PYTHONPATH=".." python server.py
ModuleNotFoundError: No module named 'numpy.core._multiarray_umath'
ModuleNotFoundError: No module named 'numpy.core._multiarray_umath'
ModuleNotFoundError: No module named 'numpy.core._multiarray_umath'
ModuleNotFoundError: No module named 'numpy.core._multiarray_umath'
ModuleNotFoundError: No module named 'numpy.core._multiarray_umath'
ModuleNotFoundError: No module named 'numpy.core._multiarray_umath'
Model filename: /home/zangetsu/pretrained_models/20180402-114759/20180402-114759.pb
 * Running on https://<IP_ADDRESS>:5000/ (Press CTRL+C to quit)

When I open GUI it can capture 10 samples, but when I do update model I get:
raceback (most recent call last):
  File "server.py", line 49, in detect
    faces = recognize(detection.get_faces(image, threshold))
  File "/home/zangetsu/proj/prometheus-core/tf-face-recognition/tensorface/recognition.py", line 21, in recognize
    result = classifier.predict(X)
  File "/home/zangetsu/proj/prometheus-core/tf-face-recognition/tensorface/classifier.py", line 119, in predict
    return model.predict(X)
  File "/home/zangetsu/proj/prometheus-core/tf-face-recognition/tensorface/classifier.py", line 71, in predict
    if _valid_nn_num == self.K:
NameError: name '_valid_nn_num' is not defined
[2019-05-18 13:57:36,965] ERROR in app: Exception on /detect [POST]
Traceback (most recent call last):
  File "server.py", line 49, in detect
    faces = recognize(detection.get_faces(image, threshold))
  File "/home/zangetsu/proj/prometheus-core/tf-face-recognition/tensorface/recognition.py", line 21, in recognize
    result = classifier.predict(X)
  File "/home/zangetsu/proj/prometheus-core/tf-face-recognition/tensorface/classifier.py", line 119, in predict
    return model.predict(X)
  File "/home/zangetsu/proj/prometheus-core/tf-face-recognition/tensorface/classifier.py", line 71, in predict
    if _valid_nn_num == self.K:
NameError: name '_valid_nn_num' is not defined

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/zangetsu/.local/lib64/python3.6/site-packages/flask/app.py", line 1982, in wsgi_app
    response = self.full_dispatch_request()
  File "/home/zangetsu/.local/lib64/python3.6/site-packages/flask/app.py", line 1614, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/home/zangetsu/.local/lib64/python3.6/site-packages/flask/app.py", line 1517, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/home/zangetsu/.local/lib64/python3.6/site-packages/flask/_compat.py", line 33, in reraise
    raise value
  File "/home/zangetsu/.local/lib64/python3.6/site-packages/flask/app.py", line 1612, in full_dispatch_request
    rv = self.dispatch_request()
  File "/home/zangetsu/.local/lib64/python3.6/site-packages/flask/app.py", line 1598, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "server.py", line 69, in detect
    print('POST /detect error: %e' % e)
TypeError: must be real number, not NameError
<IP_ADDRESS> - - [18/May/2019 13:57:36] "POST /detect HTTP/1.1" 500 -

I used this docker image in dockerhub which works just fine, but the code is probably different against what is in the repository:
docker run -it --rm -p 5000:5000 btwardow/tf-face-recognition:1.0.0


ENH:Uncompressed .nii files not supported?

Thank you for your feedback.
It seems that only .nii.gz files can be loaded into reorient, and not uncompressed .nii files.
This is a admittedly a slightly unfair criticism, as the load page explicitly says Choose a .nii.gz file.., but the first file I tried to load into the tool was a .nii file, and I found it strange that nothing happened.
It is also true that compresesd .nii.gz files are more common than .nii files, so feel free to ignore and close this issue if you don't think it is a big deal :)
Current behaviour

If this is a bug, please explain how to reproduce the problem

Which tools did you use in which sequence?
Does the error concern the online display/ functionality or the downloaded result?
What is the type and resolution of your data? (Can you provide a link to your data?)
Do not include sensitive information.
Upload the browser error message if you are familiar with web developer tools.

Expected behaviour
Version information (for bug reports)

Browser + version: Firefox 81.0 / Google Chrome 85.0.4183.121
Your OS + version: Ubuntu 18.04

now uploading .nii files is supported as well : )


Add a link to Flow usage stats collection cases

This change is

Test failing because "Missing Sauce credentials", which means that someone with appropriate permissions would have to commit the change to a branch in this repo and create a PR from that (or just ninja submit).

Will merge as is, in the future please add [skip ci] to the commit message to skip the Travis at all for this kind of changes.

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable


feat: generate lock-ckb-tx and unlock-ckb-tx

Codecov Report

Merging #266 (2c2373c) into feature-nervos-to-ethereum (5cb250e) will decrease coverage by 3.11%.
The diff coverage is n/a.

:exclamation: Current head 2c2373c differs from pull request most recent head 3f7612f. Consider uploading reports for the commit 3f7612f to get more accurate results

@@                      Coverage Diff                       @@
##           feature-nervos-to-ethereum     #266      +/-   ##
==============================================================
- Coverage                       42.07%   38.96%   -3.12%
==============================================================
  Files                              22       16       -6
  Lines                            1925     1699     -226
  Branches                           48        0      -48
==============================================================
- Hits                              810      662     -148
+ Misses                           1115     1037      -78

Impacted Files
Coverage Δ

...es/eth-contracts/contracts/libraries/SafeERC20.sol

...ain-modules/eth-contracts/contracts/test/ERC20.sol

...ules/eth-contracts/contracts/libraries/Address.sol

...les/eth-contracts/contracts/libraries/SafeMath.sol

...in-modules/eth-contracts/contracts/ForceBridge.sol

...th-contracts/contracts/libraries/MultisigUtils.sol

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 5cb250e...3f7612f. Read the comment docs.


Broken link

The link for Windows Exploit Guard under "Set device protection settings for Windows 10 devices" section give a 404 error.

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

ID: 1d156f82-2ba0-2d66-a639-1df86554ac83
Version Independent ID: 4e158c5b-09fd-7214-474f-e7ff44b86d39
Content: Microsoft 365 Business security and compliance features
Content Source: microsoft-365/business/security-features.md
Service: o365-administration
GitHub Login: @Sirkkuw
Microsoft Alias: sirkkuw

@mamidu3  - Thank you for submitting feedback.
I will get this issue over to the Microsoft 365 writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

@mamidu3 - From our understanding, the issue has been resolved. If you feel it hasn't been resolved, please re-open this issue.
Thank you for your contribution to make the docs better! Much appreciated!


What is the plan after clicking get started and clicking services?

I see the website as completely dead as of now
@MonalikaPatnaik what are the plans?

Heyy...currently working on the services page!

@MonalikaPatnaik you can give these pages to me but then you need to provide me figma design.
And the idea of what we are trying to achieve

Hello @gmmkmtgk  I want to make a valuable contribution to this issue. Can you assign it to me ??

@MonalikaPatnaik you can give these pages to me but then you need to provide me figma design.
And the idea of what we are trying to achieve

Hey you are free to make any page for services section..pick any one services page and start working on it..It's not mandatory that you create the page as a whole in one go..you can even create separate issues !
for eg--let's suppose you are creating connect with friends page, then at first you just create a dummy page having basic functionalities, then after the PR is merged, you will raise a separate issue for enchancing it's features...for eg--adding dark theme to that page..hover effect, animations etc..
Hope it clears!

Hey Everyone now It's been done so I'm closing it.


Refresh test json templates due to removal of input data

Impact of the bug
WMCore validation in general
Describe the bug
As we get started with the HG2401 / WMAgent 2.2.6 validation, there are many workflows getting stuck in assigned status. Checking MSTransferor logs, one can see that many calls to Rucio are not yielding any results, meaning that data has been completely removed from the grid [1].
How to reproduce it
Inject the relevant test json templates
Expected behavior
Matching those datasets against our test json templates, suggest that the following templates need to be remade/refactored because the RelVal data is no longer available:
test/data/ReqMgr/requests/Integration/SC_ReDigi_Harvest_Prod.json
test/data/ReqMgr/requests/Integration/SC_PY3_PURecyc.json
test/data/ReqMgr/requests/Integration/TaskChain_PUMCRecyc.json
and for the non-relval data that has been removed (e.g. DQMIO), the following needs to be remade:
test/data/ReqMgr/requests/DMWM/DQMHarvest_RunWhitelist.json
test/data/ReqMgr/requests/Integration/DQMHarvesting_MultiRun.json
test/data/ReqMgr/requests/Integration/DQMHarvesting.json
test/data/ReqMgr/requests/Integration/DQMHarvesting_LumiMask.json
Additional context and error message
[1] Relevant log from MStransferor
2024-01-14 00:01:55,786:ERROR:PycurlRucio: Failure in getBlocksAndSizeRucio function for container /RelValTTbar_14TeV/CMSSW_11_2_0_pre8-112X_mcRun3_2024_realistic_v10_forTrk-v1/GEN-SIM. Response: {'url': 'http://cms-rucio.cern.ch/dids/cms/dids/search?type=dataset&long=True&name=/RelValTTbar_14TeV/CMSSW_11_2_0_pre8-112X_mcRun3_2024_realistic_v10_forTrk-v1/GEN-SIM%23%2A', 'data': '', 'headers': 'HTTP/1.1 200 OK\r\nDate: Sun, 14 Jan 2024 00:01:55 GMT\r\nContent-Type: application/x-json-stream\r\nContent-Length: 0\r\nConnection: keep-alive\r\nAccess-Control-Allow-Origin: None\r\nAccess-Control-Allow-Headers: None\r\nAccess-Control-Allow-Methods: *\r\nAccess-Control-Allow-Credentials: true\r\nCache-Control: post-check=0, pre-check=0\r\nPragma: no-cache\r\nX-Rucio-Host: cms-rucio.cern.ch\r\n\r\n'}
2024-01-14 00:01:55,786:ERROR:PycurlRucio: Failure in getBlocksAndSizeRucio function for container /RelValZMM_14/CMSSW_12_0_0_pre6-120X_mcRun3_2021_realistic_v4-v1/GEN-SIM. Response: {'url': 'http://cms-rucio.cern.ch/dids/cms/dids/search?type=dataset&long=True&name=/RelValZMM_14/CMSSW_12_0_0_pre6-120X_mcRun3_2021_realistic_v4-v1/GEN-SIM%23%2A', 'data': '', 'headers': 'HTTP/1.1 200 OK\r\nDate: Sun, 14 Jan 2024 00:01:55 GMT\r\nContent-Type: application/x-json-stream\r\nContent-Length: 0\r\nConnection: keep-alive\r\nAccess-Control-Allow-Origin: None\r\nAccess-Control-Allow-Headers: None\r\nAccess-Control-Allow-Methods: *\r\nAccess-Control-Allow-Credentials: true\r\nCache-Control: post-check=0, pre-check=0\r\nPragma: no-cache\r\nX-Rucio-Host: cms-rucio.cern.ch\r\n\r\n'}
2024-01-14 00:01:55,789:ERROR:PycurlRucio: Failure in getBlocksAndSizeRucio function for container /RelValQCD_Pt_600_800_14/CMSSW_11_2_0_pre8-112X_mcRun3_2024_realistic_v10_forTrk-v1/GEN-SIM. Response: {'url': 'http://cms-rucio.cern.ch/dids/cms/dids/search?type=dataset&long=True&name=/RelValQCD_Pt_600_800_14/CMSSW_11_2_0_pre8-112X_mcRun3_2024_realistic_v10_forTrk-v1/GEN-SIM%23%2A', 'data': '', 'headers': 'HTTP/1.1 200 OK\r\nDate: Sun, 14 Jan 2024 00:01:55 GMT\r\nContent-Type: application/x-json-stream\r\nContent-Length: 0\r\nConnection: keep-alive\r\nAccess-Control-Allow-Origin: None\r\nAccess-Control-Allow-Headers: None\r\nAccess-Control-Allow-Methods: *\r\nAccess-Control-Allow-Credentials: true\r\nCache-Control: post-check=0, pre-check=0\r\nPragma: no-cache\r\nX-Rucio-Host: cms-rucio.cern.ch\r\n\r\n'}
2024-01-14 00:01:55,798:ERROR:PycurlRucio: Failure in getBlocksAndSizeRucio function for container /NoBPTX/Run2016F-23Sep2016-v1/DQMIO. Response: {'url': 'http://cms-rucio.cern.ch/dids/cms/dids/search?type=dataset&long=True&name=/NoBPTX/Run2016F-23Sep2016-v1/DQMIO%23%2A', 'data': '', 'headers': 'HTTP/1.1 200 OK\r\nDate: Sun, 14 Jan 2024 00:01:55 GMT\r\nContent-Type: application/x-json-stream\r\nContent-Length: 0\r\nConnection: keep-alive\r\nAccess-Control-Allow-Origin: None\r\nAccess-Control-Allow-Headers: None\r\nAccess-Control-Allow-Methods: *\r\nAccess-Control-Allow-Credentials: true\r\nCache-Control: post-check=0, pre-check=0\r\nPragma: no-cache\r\nX-Rucio-Host: cms-rucio.cern.ch\r\n\r\n'}
2024-01-14 00:01:55,799:ERROR:PycurlRucio: Failure in getBlocksAndSizeRucio function for container /BTagMu/Run2022D-10Dec2022-v1/DQMIO. Response: {'url': 'http://cms-rucio.cern.ch/dids/cms/dids/search?type=dataset&long=True&name=/BTagMu/Run2022D-10Dec2022-v1/DQMIO%23%2A', 'data': '', 'headers': 'HTTP/1.1 200 OK\r\nDate: Sun, 14 Jan 2024 00:01:55 GMT\r\nContent-Type: application/x-json-stream\r\nContent-Length: 0\r\nConnection: keep-alive\r\nAccess-Control-Allow-Origin: None\r\nAccess-Control-Allow-Headers: None\r\nAccess-Control-Allow-Methods: *\r\nAccess-Control-Allow-Credentials: true\r\nCache-Control: post-check=0, pre-check=0\r\nPragma: no-cache\r\nX-Rucio-Host: cms-rucio.cern.ch\r\n\r\n'}
2024-01-14 00:01:55,801:ERROR:PycurlRucio: Failure in getBlocksAndSizeRucio function for container /ZeroBias/Run2016B-UL16_ver2_forHarvestOnly-v1/DQMIO. Response: {'url': 'http://cms-rucio.cern.ch/dids/cms/dids/search?type=dataset&long=True&name=/ZeroBias/Run2016B-UL16_ver2_forHarvestOnly-v1/DQMIO%23%2A', 'data': '', 'headers': 'HTTP/1.1 200 OK\r\nDate: Sun, 14 Jan 2024 00:01:55 GMT\r\nContent-Type: application/x-json-stream\r\nContent-Length: 0\r\nConnection: keep-alive\r\nAccess-Control-Allow-Origin: None\r\nAccess-Control-Allow-Headers: None\r\nAccess-Control-Allow-Methods: *\r\nAccess-Control-Allow-Credentials: true\r\nCache-Control: post-check=0, pre-check=0\r\nPragma: no-cache\r\nX-Rucio-Host: cms-rucio.cern.ch\r\n\r\n'}

As I haven't made any progress on this for the last month, I am setting it back to the ToDo queue.

Our templates have degraded even further and perhaps half of them are now broken. Most common issues are:

pileup not available at the expected location
SL6 workflows failing to find condor_chirp; etc

Here is a short summary of workflows (templates) and the problems found during Agent 2.3.7 validation:
amaltaro_SC_6Steps_PU_Agent237_Val_241017_144446_265
RootEmbeddedFileSequence no input files specified for secondary input source.

amaltaro_TC_6Tasks_PU_Agent237_Val_241017_144428_6763
RootEmbeddedFileSequence no input files specified for secondary input source.

amaltaro_SC_LHE_Ext_Agent237_Val_241017_144920_9028amaltaro_TaskChain_LumiMask_multiRun_Agent237_Val_241017_144914_4026
{'arguments': ['/bin/bash', '/srv/job/WMTaskSpace/cmsRun1/cmsRun1-main.sh', '', 'slc6_amd64_gcc481', 'scramv1', 'CMSSW', 'CMSSW_7_2_0', 'FrameworkJobReport.xml', 'cmsRun', 'PSet.py', '']}
CMSSW Return code: 7002
locale::facet::_S_create_c_locale name not valid
[Errno 2] No such file or directory: '/srv/.gwms.d/bin/condor_chirp': '/srv/.gwms.d/bin/condor_chirp'
WARNING: There already exists /srv/job/WMTaskSpace/cmsRun1/CMSSW_9_3_7 area for SCRAM_ARCH slc6_amd64_gcc630.

amaltaro_TaskChain_MC_Agent237_Val_241017_144942_8735
SL6 broken workflow

amaltaro_TaskChain_Prod_Agent237_Val_241017_144944_6207
An exception of category 'NoSecondaryFiles' occurred while
RootEmbeddedFileSequence no input files specified for secondary input source.

amaltaro_TC_Drop_Rules_Ext_Agent237_Val_241017_144918_1727
SL6 broken workflow

amaltaro_TC_PY3_Data_LumiList_Agent237_Val_241017_144926_546
An exception of category 'PluginLibraryLoadError' occurred while

amaltaro_TC_PY3_TTbarPU_Agent237_Val_241017_144950_8787
An exception of category 'NoSecondaryFiles' occurred while
RootEmbeddedFileSequence no input files specified for secondary input source.

We will have to work on it ASAP.

Quick update on the SL6 workflows. They are indeed broken at the moment, but still supported in central production. It will be fixed with a new version of glideinWMS (not yet released). Further details in:  https://its.cern.ch/jira/browse/CMSPROD-223


🐛 [firebase_auth] How to update phone number on web

Bug report
Describe the bug
How can I get PhoneAuthCredentials from WebConfirmationResult?
Steps to reproduce
Steps to reproduce the behavior:

I want to update the users phone number on web
To update the phone number PhoneAuthCredentials are needed
I can not access the PhoneAuthCredentials from WebConfirmationResult

Expected behavior
Is there a way to use .updatePhoneNumber on web?
Sample project
if (webConfirmationResult != null) {
  try {
    UserCredential _credential = await webConfirmationResult!.confirm(otp);
    await user!.updatePhoneNumber(_credential); // error: The argument type 'UserCredential' can't be assigned to the parameter type 'PhoneAuthCredential'.
    // Finish
  } catch (e) {
   print(e.toString());
  }
} else {
  // ...
}

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
[✓] Flutter (Channel stable, 2.10.5, on macOS 12.3.1 21E258 darwin-arm, locale de-DE)
[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 13.3.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.1)
[✓] VS Code (version 1.66.2)
[✓] Connected device (4 available)
[✓] HTTP Host Availability

• No issues found!

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
- cloud_firestore 3.1.14 [cloud_firestore_platform_interface cloud_firestore_web collection firebase_core firebase_core_platform_interface flutter meta]
- firebase_auth 3.3.17 [firebase_auth_platform_interface firebase_auth_web firebase_core firebase_core_platform_interface flutter meta]
- firebase_core 1.16.0 [firebase_core_platform_interface firebase_core_web flutter meta]
- firebase_storage 10.2.15 [firebase_core firebase_core_platform_interface firebase_storage_platform_interface firebase_storage_web flutter]
- flutter 0.0.0 [characters collection material_color_utilities meta typed_data vector_math sky_engine]

@BugsOverBugs
Check below screenshot and see if it helps.

@BugsOverBugs Check below screenshot and see if it helps.

This is not the answer I asked for. This is another topic, not my question!

Hey @BugsOverBugs, you need to return this credential.
Here is the e2e test for it (commented out because it is flakey with the emulator).
Here it is used but to test the returned exception. You should be able to get it working from these snippets.
Thanks.


No ore generation in Josharias Survival

Problem: With no ores being generated in the world of JS big parts of the tech tree are unavailble, reducing the scope of the game play module drastically. This is a regression compared to previouls (preview) releases.
The ore gen was working in preview 1123 (20210103), so it's likely related to our JOML migration effort. Based on that date and the involved modules, this (most likely) boils down to:

https://github.com/Terasology/CustomOreGen/compare/93210238ccfd50dde2a6d22776a072db2ac47c11..develop
https://github.com/Terasology/OreGeneration/compare/1144f6197cc36eed928e02d0ebe38e41f886eb0a..develop

Potential fix by @pollend in https://github.com/Terasology/CustomOreGen/pull/13

Potential fix by @pollend in https://github.com/Terasology/CustomOreGen/pull/13

Not really. Min and max positions are suppose to be local to the transform just not sure what the translation did in the old code. Maybe /@4Denthusiast would have a clue.


Change LED Color from UI

Long press on the Colored Square to bring up a Color Picker Dialog (see https://github.com/woalk/android-colorpicker), then write updated Characteristics to BLE Service

Done


Add orijtech/structslop linter

Description:

Package structslop defines an Analyzer that checks if struct fields can be re-arranged to optimize size.
...
Example, for the first report above, the output meaning:

The current struct size is 24, the size that the Go runtime will allocate for that struct is 32.
The optimal struct size is 16, the size that the Go runtime will allocate for that struct is 16.
The layout of optimal struct.
The percentage savings with new struct layout.

refs:

https://github.com/orijtech/structslop

Duplicate of #140.


Update install prefix on Windows

MNT: Re-rendered with conda-build 3.21.8, conda-smithy 3.19.0, and conda-forge-pinning 2<IP_ADDRESS>.52.03

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


github/ci: Dont trigger command workflow for bots

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

for ref its reducing the footprint here https://github.com/envoyproxy/envoy/actions/workflows/commands.yml
there is already some contention (wait to see how long before this comment pops up ...)
im starting to move some jobs out of azp - partly to reduce contention there and partly because that is the plan (cf #27294 ) so the more we can prevent waste on the machines the better

relatedly this probably would be better to do with rk cc @itayd

@wbpcode im gonna push this through - the ci passed but is messed up here not sure why - i think unrelated
i do want to test this on postsubmit tho - so will watch/fix etc now

my first question is what happens on closed prs
/retest

i think both triggered - altho only rk posted here - probably we want to add the pr state to the condition

hmm - this didnt work exactly - its probably made it faster - it only skips the job now but still sets itself up


Webrecorder branding updates

@quinnanya Hey!  The website is open source!
Webrecorder has recently updated pretty much all of our branding!  The names and logos of our key software suite are now set in stone and I'm on a bit of a tear trying to get things unified around the web so that future new folks that run across our software in great projects like yours aren't confused.
Major Changes

Updates Webrecorder partner logo (thanks for featuring us ❤ )
WebRecorder → Webrecorder
Manual WebRecorder → ArchiveWeb.page (I kept all the internal documentation sheet naming and such references the same)
Browsertrix Cloud → Browsertrix
Browsertrix → Browsertrix Crawler

Additional Changes

Replaced download link to the ArchiveWeb.page desktop app with a link to our new site that actually has a download section!
Replaced the Browsertrix Crawler documentation link to our readme with our new actual docs site
Adds missing alt text to Browsertrix crawling screenshot
Minor copy edits (spelling mistakes)
Application copy reference changes (Some things have changed!  The terms we use will now be more solid going forward)
Adds syntax highlighting to some code blocks

Merging changes for updated webrecorder logos and language.


Add Radial Dimension + Add Angular Dimension

Hi Steve,
Can you put this on the wishlist?
it would be useful to have methods for:
•Add Radial Dimension
•Add Angular Dimension
in RhCommon and ideally also in rhinoscriptsyntax.
I could not find it here:
http://www.rhino3d.com/5/rhinocommon/html/AllMembers_T_Rhino_DocObjects_Tables_ObjectTable.htm

http://mcneel.myjetbrains.com/youtrack/issue/RH-30054


Added notes app

Have you read the Contributing Guidelines ?
(Write your answer here.)
Description
(Write your answer here.)
Fixes : #571
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
(Write your answer here.)

@prakhar3062 have a look .


请问在长按方法里如何弹出一个保存图片的小框？

func photoBrowser(_ photoBrowser: PhotoBrowser, didLongPressForIndex index: Int, image: UIImage) {
print("长按，图片size:(image.size)")
imageViewLongPressGestureRecognizer(image)
}
/// 长按图片事件
func imageViewLongPressGestureRecognizer(_ saveImage:UIImage) {
    // 暂时先用 actionSheet 吧
    let actionSheet = UIAlertController(title: nil, message: nil, preferredStyle: .actionSheet)

    let saveImageAction = UIAlertAction(title: "保存图片", style: .default) { (_) in
        print("保存\(saveImage)")
    }
    let cancelAction = UIAlertAction(title: "取消", style: .cancel, handler: nil)

    actionSheet.addAction(saveImageAction)
    actionSheet.addAction(cancelAction)
    present(actionSheet, animated: true, completion: nil)
}

上面这样子提示 下列错误，也无法弹出框
2017-08-30 17:39:46.562 PhotoBrowser[7622:1035904] Warning: Attempt to present <UIAlertController: 0x7fbe2d912cf0>  on <PhotoBrowser.MomentsViewController: 0x7fbe2b606020> which is already presenting <JXPhotoBrowser.PhotoBrowser: 0x7fbe2b50e0d0>

稍等，我在demo示例一下

用photoBrowser来present，我已更新代码。对应cocoapods 0.3.5

咦。我用photoBrowser 来present一个UIAlertController（actionSheet）的时候 会
Make a symbolic breakpoint at UIViewAlertForUnsatisfiableConstraints to catch this in the debugger.
The methods in the UIConstraintBasedLayoutDebugging category on UIView listed in <UIKitCore/UIView.h> may also be helpful.
2019-08-29 11:23:26.375017+0800 Runner[449:49445] [LayoutConstraints] Unable to simultaneously satisfy constraints.
	Probably at least one of the constraints in the following list is one you don't want.
	Try this:
		(1) look at each constraint and try to figure out which you don't expect;
		(2) find the code that added the unwanted constraint or constraints and fix it.
(
    "<NSLayoutConstraint:0x2833367b0 UIView:0x123e89870.width == - 16   (active)>"
)

Will attempt to recover by breaking constraint
<NSLayoutConstraint:0x2833367b0 UIView:0x123e89870.width == - 16   (active)>

Demo里面也有present ActionSheet，你运行那个看看有没问题，看能不能找到原因？


Reset TileMap editor drag_type when the toolbar mode is not selected

this fixes : https://github.com/godotengine/godot/issues/85077
The problem made it so that the drag_type was kept until the toolbar mode SELECT was once again pressed. This made it so tilemap was drawing automatically without having to press down the left mouse button. By changing drag type to none when select is not used, this is fixed.

For the record, your commit seems not to be linked to your GitHub account. See: Why are my commits linked to the wrong user? for more info.

For the record, your commit seems not to be linked to your GitHub account. See: Why are my commits linked to the wrong user? for more info.

Godammit. Thank you for telling me. Will I need to fix this for the pull request to be approved?

It's not absolutely mandatory, but your GitHub account will not be recognized as the author of this change. It's easy to fix by either changing your local Git identify and amending the commit (git commit --amend --reset-author), or adding your student email used to author this commit as secondary email on your GitHub account.

It's not absolutely mandatory, but your GitHub account will not be recognized as the author of this change. It's easy to fix by either changing your local Git identify and amending the commit (git commit --amend --reset-author), or adding your student email used to author this commit as secondary email on your GitHub account.

Okay it should be fixed now, I simply added the email address

It didn't seem effective yet, make sure it's the address shown in the header of https://github.com/godotengine/godot/pull/86066.patch, and you might have received an email from GH there to confirm adding it.

The last attempt at changing the author info worked, but there are now 3 commits:

When amending a commit, you should update with git push --force instead of merging the remote branch back, which keeps both the old and the new commits.
Now you can fix it with:
git fetch origin
git rebase -i origin/master

This should show:
pick 5c4b3d490a reset author
pick cc9183562d Sets drag_type of the tile_map_editor to none when the toolbar mode is not select

Change the second pick to squash, exit the text editor, and then you should have the opportunity of improving the commit message (as reset author isn't a good one). I'd suggest Reset TileMap editor drag_type when the toolbar mode is not selected.
The git log should then look like this:
commit 33aa9afd88251277d334b9b20c2ca543e96ac6bb (HEAD -> tilemapPaint)
Author: wilto938<EMAIL_ADDRESS>Date:   Tue Dec 12 12:27:26 2023 +0100

    Reset TileMap editor `drag_type` when the toolbar mode is not selected

commit 84692c6252438393999c427bf7f5d35b9c43926c (origin/master, master)
Author: Rémi Verschelde<EMAIL_ADDRESS>Date:   Tue Dec 12 11:00:06 2023 +0100

    Navigation: Fix GCC 12.3 `-Wtype-limits` warnings

And so you can force push to update your fork with git push --force.

Thank you so much for all your help, haha. I don't really understand how all those other commits showed up, but all's well that ends well

Looks good. Could you please squash the commit together ?

Looks good. Could you please squash the commit together ?

Done!

Thanks, and congrats on your first merged Godot contribution!

Cherry-picked for 4.2.2.


delete not working

unable to delete telegrams. (button not working, it just open the telegram)

yeah it doesn't work with one sync only without onesync


vosk?

how can I use VOSK on my home assistant core (docker container) installation?
i pulled your VOSK container but I did not understand how set it in home assitant... which port?

I am also interested in this. Is there a way to run VOSK in home assistant core?

I am also interested in this. Is there a way to run VOSK in home assistant core?

Yes i’m using it as docker container


Adding support for reading table into ParquetFile

[Motivation] #162
Testing
delta_result = DS.list_deltas(namespace=namespace,
...                         table_name=table_name,
...                         partition_values=ordered_partition_values,
...                         table_version=table_version,
...                         include_manifest=True,
...                         equivalent_table_types=[])

delta = delta_result.read_page()[0]

table = DS.download_delta_manifest_entry(delta, 0, TableType.PYARROW_PARQUET)
# <pyarrow.parquet.core.ParquetFile object at 0xffffa168c100>

The repartition tests are failing due to a usage of float("inf") for comparison in Expression. @valiantljk is working on fixings these tests.

Looks good overall! Just one quick question:
Could you explain why we don't have to support the input_file_init for different content_encoding as here?

@Zyiqin-Miranda Good question. The creation of ParquetFile instance does not lead to file read. Content encoding is used to decode the file bytes (i.e., after the file has been downloaded) but in our file has not yet been downloaded. The assumption is that pyarrow library should be capable of decoding appropriate encodings.

@Zyiqin-Miranda Good question. The creation of ParquetFile instance does not lead to file read. Content encoding is used to decode the file bytes (i.e., after the file has been downloaded) but in our case file has not yet been downloaded. The assumption is that pyarrow library should be capable of decoding appropriate encodings.

Thanks for the explanation. So to actually read the table, we need to use something like parquet_file.read() and assume that the file will be decoded appropriately?

@Zyiqin-Miranda Good question. The creation of ParquetFile instance does not lead to file read. Content encoding is used to decode the file bytes (i.e., after the file has been downloaded) but in our case file has not yet been downloaded. The assumption is that pyarrow library should be capable of decoding appropriate encodings.

Thanks for the explanation. So to actually read the table, we need to use something like parquet_file.read() and assume that the file will be decoded appropriately?

Yes.

Here is the performance comparison between using pyarrow.fs.S3FileSystem vs s3fs.S3FileSystem. Overall, the performance is almost the same (looking at median) however s3fs provides capability to easily pass the s3_client_kwargs which might be handy in future while pyarrow.fs.S3FileSystem does not.


Download pre-trained models

hello, thanks for sharing. I can't download pre-trained models from here. https://office365stanford-my.sharepoint.com/:f:/g/personal/tpark94_stanford_edu/EleRJUfWcTdComwfLwrrGTIBfj1xNahq06_NiNDDKUDFJg?e=s0qDch

Hi, thank you for letting me know. I've updated the link on the README.


Send code to N4SJAMK repo

Includes pull request from

api
client-react
img

All merges have been done for required repositories. This should be done on friday so we have two days to fix things before sprint review


Add optional headers to DownloadableFile messages when downloading files

Fixes #299
This PR implements the recent specification change that allows for optional headers in the DownloadableFile message, which can be used during file downloads to support authentication and other header-based configurations.
The downloadFile function in the packagesSyncer struct has been modified to include optional headers from the DownloadableFile message. This enhancement allows Agents to set custom HTTP headers when downloading files from the server. For instance, headers such as Authorization can now be included to facilitate secure downloads.
Changes Made:
-Updated the downloadFile method to check for the presence of optional headers in the DownloadableFile message.
-Added logic to incorporate these headers into the HTTP request when downloading files.
-Ensured compatibility with the updated protocol specification that outlines the use of headers for authentication and other purposes.

The build fails because we need to update to the new version of Protobufs that have the new Headers field.
This PR does it https://github.com/open-telemetry/opamp-go/pull/307

Please rebase, it should fix the build.

Rebase done @tigrannajaryan

Please fix the build: https://github.com/open-telemetry/opamp-go/actions/runs/11499942257/job/32122224014?pr=306

@RichardChukwu ICYMI, the test failure logs.

Error: client/clientimpl_test.go:1511:17: undefined: createDownloadSrv
Error: client/clientimpl_test.go:1660:17: undefined: createDownloadSrv

you can verify the changes locally by running make test

srikanthccv

Thanks, working on it already

Hi @RichardChukwu, are you still working on this?


Excuse me, could you tell me how to generate the files needed for homo_params_path and stitch_params_path

Excuse me, could you tell me how to generate the files needed for homo_params_path and stitch_params_path

Can you help me？Thank you！

This dev does not contain the generation for that. But I can provide a sample of that later.

Thank you！

您好，这两个模块加进来了吗？
Hello, have these two modules been added?

Hi, can someone update on this?


Multiple arguments for command

First of all, thanks for this really really great resource for something that is far from trivial!
I've been playing with this and getting to understand how these different pieces work together. I've bumped into issue that I can't figure out which is how do I pass multiple arguments correctly. I tried to expand the 'thermal' command and added a case like below
enum ThermalArgs: String, Codable, CaseIterable {
  case levels = "levels"
  case config = "config"
  case simulate = "simulate gpu nominal"
  var requiresAuth: Bool {
      return false
   }
}

However not that, or any other command with multiple arguments I've tried works. It errors indicating that the command's arguments are not correct. This same happens every command I've tried when there is more than one arguments to be passed.

Spaces don't have any relationship to the number of arguments; that's just a convention used by terminals (such as Terminal.app). When you're writing case simulate = "simulate gpu nominal" the spaces don't make them into separate arguments.
If you wanted to modify the sample to do this you'd need to change how the arguments property is populated, currently the rawValue is provided as the only argument: https://github.com/trilemma-dev/SwiftAuthorizationSample/blob/d5759c82e3f9e2e5f47e23c0e9969e7f5a693c39/Shared/AllowedCommand.swift#L81-L91
The argument array is then used when running the process, so this would work without modification:
https://github.com/trilemma-dev/SwiftAuthorizationSample/blob/d5759c82e3f9e2e5f47e23c0e9969e7f5a693c39/SwiftAuthorizationHelperTool/AllowedCommandRunner.swift#L43-L53


Missing version number

When using Penna the initialization message doesn't include the pennaVersion.
{"logger":"penna.api.audit.Logger","level":"INFO","message":"Penna initialized successfully","pennaVersion":"null"}
Expected: To show the Penna version
Actual: Currently showing null

As I stated in the PR, I'll set up some integration test suite. As a reminder, I'll keep this issue open until I can prove it was fixed by those tests :)

I finally got some time to push through and finished the integration tests. Build is now fixed to ensure the projects get the version and we have a test that verifies it at every PR run. Thank you so much for your help and sorry for leaving you hanging for so long 😅


[Feature Request] cgroup-support (Linux only)

Introduction
Add cgroup support as an optional feature enabled on Linux. cgroup is a Linux feature that can provide lots of great features for service supervising, like resource limit, grouped process management, etc.
Steps

[ ] [Design] cgroup manifest format
[ ] [Implement] Add cgroup support for airupd

Unresolved Questions
Nothing yet.

(initial) A general design of manifest format for resource limiting:
[resource-limit]
cpu = "150%"
memory = "1GB"


packages using ign-cmake1 aren't exporting packages using ign-cmake0 in pkg-config requires

Original report (archived issue) by Steve Peters (Bitbucket: Steven Peters, GitHub: scpeters).

This is a variant of the bug fixed in pull request #56, for which a test was added in pull request #58. In the original, a package core_child that depends on core_no_deps wasn't exporting this dependency information in the pkg-config file. This bug is a variant in which core01_child uses ign-cmake1, while the dependency core01_no_deps uses ign-cmake0. In this case, core01_child is not exporting its dependency on core01_no_deps in the .pc file. I've made an example to illustrate this in branch requires_01, though I'm not sure if we will merge it to default, since it requires ignition-cmake0. It is helpful though in illustrating the problem and testing fixes.

Closing since ign-cmake0 and ign-cmake1 are both EOL.


Feature/module manager updates

Removed loadModules from module manager constructor
Updated unit tests
Initial work on a module class classmap cache to help speed up module loading even more in production

Reviewed, merged, and pushed to master. :)


Fixed the collapse functionality

I created a function to set the default collapse property value to 'false' to every menu item. Them I did some small changes to your code, also adding the change of the css class of the menu item that indicates if the item is collapsed or not. So now, if it is collapsed it will show an fa-plus icon and if it is expanded it will show fa-minus icon.
I also added the font-awesome css that was missing in your repo. It is now complaining about some "woff2" files that could not be loaded, I don't know how to fix that though.

T.Hanks a lot!


ci/coverage: Adjust extensions/tracers

this adjustment is required for the recent security patches to land

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/assign @adisuissa

After the CVE patches PR lands, we should increase it back

i think we will have to address the issue to do that. The patches were passing CI fine until the very last rebase - we decided not to delay and address this here


readme: Tighten up default specification

TODO: edit dockerhub readme to match

I'm being a little cheeky with the pipeline commit because I didn't want my PR to go through CI. But ignore_paths: is on my mind -- it's the new feature added a few months ago to this resource that's super helpful. So thank you! and hope it's helpful to you too.

Can you get rid of the cheeky part?

:) yea


SidebarNav - subcomponent: item

I'm using SidebarNav, implement a navConfig strutured as the document show us. But I'd like to use "target: '_blank'" in a item to open the link in a new tab. How could I do that?

Looking for assistance here as well. I've tried a few different approaches, but none of them seem to work out nicely. All I want to do is generate an anchor tag with the target='_blank' attribute attached to it. :-1:

hi @marciokuroki @DannyAllegrezza
please update:

@coreui/coreui to 2.1.0
@coreui/react to 2.1.0

usage example:
{
  name: 'Try CoreUI PRO',
  url: 'https://coreui.io/pro/react/',
  icon: 'cui-layers icons',
  variant: 'danger',
  attributes: { target: '_blank', rel: "noopener" },
},

@xidedix thanks!


Migrate to @bifravst/aws-cdk-lambda-helpers

The code for packing lambdas and layers has been published as a standalone library: https://github.com/bifravst/aws-cdk-lambda-helpers
Example migration: https://github.com/hello-nrfcloud/backend/pull/588

Which commit implemented the change @Lenakh97?

Still used Lambda.Function: https://github.com/hello-nrfcloud/public-parameter-registry-aws-js/blob/saga/cdk%2FRegistryStack.ts#L94

Still used Lambda.Function: https://github.com/hello-nrfcloud/public-parameter-registry-aws-js/blob/saga/cdk%2FRegistryStack.ts#L94
Should use https://github.com/bifravst/aws-cdk-lambda-helpers/blob/saga/src%2FPackedLambdaFn.ts

Oh okay. I thought this commit fixed it:
https://github.com/hello-nrfcloud/public-parameter-registry-aws-js/commit/e062aee762e6ea21c8e72c710f3c771a4fb15b34
I will look into it!


After a round is played, the loser can decide the punishment steps for the next round (BackEnd)

https://github.com/sopra-fs23-group-28/sopra-fs23-group-28-server


:sleep in InsertEnter may break cmp's InsertEnter

Problem
I use a plugin that registers an InsertEnter autocmd that polls a channel in :sleep loop.
When that plugin is active, cmp's InsertEnter autocmd is not processed correctly.
As a result, cmp's mappings are not initialized.
Minimal reproducer
mini.vim
set runtimepath^=~/.vim/plugged/nvim-cmp
lua << EOF
local cmp = require'cmp'
cmp.setup {
  mapping = {
    ['<CR>'] = function(_)
        print('123')
    end
  },
}
EOF

Run:
nvim -u mini.vim -c 'au InsertEnter * sleep'

Type:
i<CR><ESC>:mess<CR>

Expected outcome:
Typing <CR> should not insert a newline and instead print 123 to :mess.
Actual:
Typing <CR> inserts a newline and does not print 123 to :mess,
indicating that the provided mapping is not active.

Analysis
At this line,
https://github.com/hrsh7th/nvim-cmp/blob/881ca573dbbf39ebef6b15d3e3b484484aec53b3/lua/cmp/init.lua#L266
add the following line:
@@ -263,6 +263,7 @@ cmp.setup = setmetatable({
 autocmd.subscribe('InsertEnter', function()
   -- Avoid unexpected mode detection (mode() function will returns `normal mode` on the InsertEnter event.)
   vim.schedule(function()
+    print('InsertEnter', vim.inspect(vim.api.nvim_get_mode()))
     if config.enabled() then
       cmp.core:prepare()
       cmp.core:on_change('InsertEnter')

Then, run the same command as described above.
Expected:
InsertEnter {
  blocking = false,
  mode = "i"
}
123
InsertEnter {
  blocking = false,
  mode = "c"
}

Actual:
InsertEnter {
  blocking = false,
  mode = "n"
}
InsertEnter {
  blocking = false,
  mode = "c"
}

The first InsertEnter is triggered by i, and the second one is probably triggered by :.
Note that the mode of the first event of the actual output is "n", not "i".
This make config.enabled() return false, and as a result cmp.core:prepare() does not run.
On the other hand, other InsertEnter autocmds don't seem to trigger this issue.
For example,
au InsertEnter * for i in range(0,1000000) | endfor

My conclusion is that there is a subtle bug in the interaction of sleep, vim.schedule, and vim.api.nvim_get_mode().

lua << EOF
function log(num, start)
    print(num,
          vim.fn.reltimestr(vim.fn.reltime(start)),
          vim.api.nvim_get_mode().mode)
end
EOF
au InsertEnter * lua start = vim.fn.reltime()
au InsertEnter * lua vim.schedule(function() log(1, start) end)
au InsertEnter * exe 'lua print("sleep")' | sleep | lua log(2, start)
au InsertEnter * lua vim.schedule(function() log(3, start) end)

Output:
sleep
1   0.000050 n
2   1.001121 n
3   1.001208 i

So it seems that the scheduled callback can be invoked while :sleep is running, at which point the mode is n.
BTW please don't be bothered with this issue too much if there are other important issues.  I'll try to think of a workaround.

@tomtomjhj I've tried to reproduce this in my local environment (latest neovim) but it can't reproduced.
I was using the following config.
if has('vim_starting')
  set encoding=utf-8
endif
scriptencoding utf-8

if &compatible
  set nocompatible
endif

let s:plug_dir = expand('/tmp/plugged/vim-plug')
if !filereadable(s:plug_dir .. '/plug.vim')
  execute printf('!curl -fLo %s/autoload/plug.vim --create-dirs https://raw.githubusercontent.com/junegunn/vim-plug/master/plug.vim', s:plug_dir)
end

execute 'set runtimepath+=' . s:plug_dir
call plug#begin(s:plug_dir)
Plug 'hrsh7th/nvim-cmp'
Plug 'hrsh7th/cmp-buffer'
call plug#end()

lua <<EOF
  local cmp = require'cmp'

  cmp.setup({
    mapping = {
      ['<CR>'] = function(_)
          print('123')
      end
    },
    sources = cmp.config.sources({
      { name = 'nvim_lsp' },
    }, {
      { name = 'buffer' },
    })
  })
EOF

autocmd InsertEnter * sleep

Probably, latest neovim or keymap.lua refactor solves this problem....

Unfortunately, I can still reproduce the issue. This issue only happens when the sleep autocmd is registered after cmp's. Cmp's autocmds are defined in plugin/, which are sourced after sourcing init.vim. So, the last line should be
autocmd VimEnter * autocmd InsertEnter * sleep

My patch is wrong. Sorry. It was reverted now.

Fixed the issue by https://github.com/hrsh7th/nvim-cmp/commit/a0053885ba2d50a62660731519aa5b6fef0859c6


Enhance RemoteDebug

...adding a method to capture the current JVM's remote debugging arguments.
Reviewer checklist

[ ] Ensure relevant issues are linked (description should include text like "Fixes #")
[ ] Ensure any appropriate documentation has been added or amended

Coverage decreased (-0.5%) to 94.966% when pulling ffc65e5fd76f11228116a56a7b7a2d084ab0e6f7 on current_remote_debug_args into 449280e37ca1f23d9714ca76cee71b213a995cf8 on main.


Bump PyYAML and unpin mpi4py version

Our previous pyyaml version constraint leads to a pyyaml version that is incompatible with Cython 3.0.0 (https://github.com/yaml/pyyaml/issues/601). Likewise, we had mpi4py pinned to a specific commit that fixes an issue with setuptools, but now that commit is incompatible with Cython 3.0.0. The latest mpi4py release (3.1.4) works just fine, though, so we demand at a minimum that version. pyyaml seems to work fine for versions >= 6.0.0, so we set that version constraint wherever pyyaml is required.

Warning
Dependent on #485.

Nice to see the upgrade off the pinned mpi4py 🎉


lots of refractoring, backtesting is out of order, but more tests are…

… put in place, but are currently out of order

Yolo deploy


Migrate visualization to top level namespace

Summary
This commit moves the visualization code out of tools into it's own top
level namespace, qiskit.visualization. The visualization was a bit out
of place in the tools directory and is really it's own top level thing
at this point. However, because we declared the public visualization
interfaces stable as part of the 0.7 release we must maintain backwards
compatibility. In order to facilitate that all the public APIs are still
accessible via qiskit.tools.visualization. However, private
api/modules are not accessible off of qiskit.tools.visualization
anymore.
Details and comments

while moving these files should we also remove the leading underscores? (#1206)

yes I think we should

@mtreinish can you also remove the leading underscore from the files here as you move them?


Deprecate configmap as default on extension refs

fixes #292

/ok-to-test

thanks for the review and tips @robscott , I think i've resolved as well as made the requested changes. One area where some uncertainty arose was WRT to the desired end state on local_object_reference_types.go . Unsure if the reduced comment strings should still include the group/resource example ... please advise if i have not guessed correctly :)

Thanks for the updates @eth0xFEED! This mostly LGTM. I think I'd prefer removing the example from the comments in this case. I'd also suggest removing the +kubebuilder:validation:Required (sorry I missed this earlier). It looks like this specific annotation is entirely unnecessary since fields are required by default.

Thanks!
/lgtm
/approve


Adding Sublime Docker Image

Status
Ready
Related Content Pull Request
No related PR
Related Issues
No related issues
Description
Need a docker container to integrate with sublime security products

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

Excellent work! Thank you for addressing my comments. Please share the PR containing the relevant content that necessitates the use of this docker image. This information is crucial for us to proceed with merging the PR.

Hey @BEAdi I'm not for sure what you mean? I haven't been able to build the content yet as I don't have a docker image that can run it.

Hi, you are right. I'm proceeding with merging this docker image, and we will need the content branch to be created within the following weeks. Once the branch is ready, please attach it here.


Object category name <-> id for mp3d

Hi,
I am looking for a map for object semantic id to semantic name in mp3d. I couldn't find this info in the doc. So I print it by checking with the simulator (checking with obj.category.name() and obj.category.index()). Below is what I get. -1 is mapping to an empty string. I wonder what does that means?  Also what is "void". And it seems there is no id number 32 and 35?
Is there an existing official doc for this information? Thank you!
-1
0 void
1 wall
2 floor
3 chair
4 door
5 table
6 picture
7 cabinet
8 cushion
9 window
10 sofa
11 bed
12 curtain
13 chest_of_drawers
14 plant
15 sink
16 stairs
17 ceiling
18 toilet
19 stool
20 towel
21 mirror
22 tv_monitor
23 shower
24 column
25 bathtub
26 counter
27 fireplace
28 lighting
29 beam
30 railing
31 shelving
33 gym_equipment
34 seating
36 furniture
37 appliances
38 clothes
39 objects
40 misc

You can see the full mapping here: https://github.com/niessner/Matterport/blob/master/metadata/mpcat40.tsv.  The ones that are missing for yours are likely because you constructed this from a single scene and not all scenes have all objects.  -1 is unlabeled.


Hide cluster info from non-admin users

Fixes #933
Work: Hide Cluster field from non-admin user in Details card on VM dashboard

ci build please

The host and cluster fields are visible to the non-admin user, the host field is empty and cluster field has correct value. Admin user can see both fields and both contain correct values.

@isaranova  I've rebased and the cluster field seems to be doing well for non-admin user. For the host field, it's empty even on master.
@sjd78 Do you know why Host's info might be empty? I believe it's a separate issue

LGTM


SAK-33732 - Error in time while modifying an event with timeslots

Tested successfully in the Atica server (S2U QA server)


Course student stats

This is another report that we need. This one is much more important so has the highest priority among reports at the moment. Whoever is doing reports needs to do this one first (or next if already working on one).
We need a report for each course. There should be a link in the course admin page (similar to "show report" in the assignments admin page) called "Student Stats". This opens up a new page similar to assign report view. In this, we need to do the following:
For all students registered in this course, we need the following fields:
No.
Student (comes from student.str)
Roll number
Assignments completed (total number -- a completed assignment is one with at least one submission)
Late days remaining
Average marks (in percentage since scores might be different for each assignment)
Average number of submissions (so, total number of submissions / assignments completed)
Average time taken (time taken for an assignment is last submission time - open date time .. then we average over the whole course)
If you can think of another field, please comment so we can discuss.
Has to be done with a separate views file and not in the main views.py file.
@soachishti Please assign forward to whoever is relevant.

@BilalZaib What do you say? I think this might be simpler... you can use existing code of assignment_report (its table) to add this report...

Yes its seems easy, I'll start work on it.
There were virus issues with my laptop and now I've installed new window. It took little time to setup all required softwares

@BilalZaib OK. No problem, take you time.

Assignments whose due date is not passed should be included in this report or not??

I think it should be included; "a completed assignment is one with at least one submission".

To calculate average marks of a student
average marks = sum_of_marks/completed_assignments or sum_of_marks/total_assignments ??
(Because there may be students who don't submit an assignment so it will be incomplete)

What will be the formula for "Average time taken"?

average marks = sum_of_marks/completed_assignments
average time taken = average(time taken for an assignment is last submission time - open date time) -- again, only for completed assignments.


Trailing forward slash was being consumed as part of the domain name.

Description
In the regex handling the parsing of the domain name from the URL, if a trailing slash existed in the okta_org_url config parameter, the trailing slash was being consumed as part of the domain.  This was causing the cookie logic to misbehave, since the domains no longer exactly matched.
Related Issue

#150
Motivation and Context

Dual MFA prompt is extremely obnoxious, and was preventing the use of SMS.
How Has This Been Tested?

Local testing.
Screenshots (if appropriate):
N/A
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Thanks to @epierce for finding this well-hidden bug!


Add a blog post for mini site generation

We've recently added a new site interface to the CLI that aims to replace the old Makefile. This blog post explains how can we use this new interface to build a mini-site.
Fixes: #369

Hello there, I've added a new blog post that explains how anyone can use the metanorma site interface to build a mini-site or customize it if necessary. I've tried to make it as less geeky as possible but could you guys please have a look at it and let me know your thoughts?
//cc: @ronaldtse


cmake complains that gcc 12.2.0 doesn't support C++17

Cmake identifies the compiler is gcc 12.2.0 but then erroneously claims the compiler doesn't support c++17.
spack-build-out.txt

You are explicitly passing C++ 11 to the CMake command line:
-DCMAKE_CXX_STANDARD:STRING=11

This is off the master branch.  24.03.0 is fine.

This is a bug in the Legion Spack package, which apparently has not been updated. Legion now requires C++ 17.
Until the Spack package gets updated you would need to build with the variant cxxstd=17. Otherwise Spack explicitly builds for C++ 11 and Legion of course objects.

Potential posted to https://github.com/spack/spack/pull/44596
@jpietarilagraham If you'd like to test, I think you can clone my Spack fork (assuming you have no other local changes you need).

Fixed by https://github.com/spack/spack/pull/44596.


Identical stat config using Telemetry API versus EnvoyFilter causes 10x config size

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
Hello, we recently migrated from configuring stats via EnvoyFilters to Telemetry v2, using the first-class Telemetry resources.
We discovered that identical config (applied to all metrics) resulted in 10x more config with Telemetry than it did using EnvoyFilters.
Take a simple Telemetry resource:
apiVersion: telemetry.istio.io/v1alpha1
kind: Telemetry
metadata:
  name: netshoot
  namespace: netshoot
spec:
  selector:
    matchLabels:
      app: netshoot
  metrics:
  - providers:
    - name: prometheus
    overrides:
    - tagOverrides:
        destination_service:
          value: "CATHAL_TEST_1"

Port-forward to any workload on 15000, and inspect a dynamic listener. It results in
"configuration": {
"@type": "type.googleapis.com/google.protobuf.StringValue",
"value": "{\"metrics\":[{\"dimensions\":{\"destination_service\":\"CATHAL_TEST_1\"},\"name\":\"request_messages_total\"},{\"dimensions\":{\"destination_service\":\"CATHAL_TEST_1\"},\"name\":\"response_messages_total\"},{\"dimensions\":{\"destination_service\":\"CATHAL_TEST_1\"},\"name\":\"requests_total\"},{\"dimensions\":{\"destination_service\":\"CATHAL_TEST_1\"},\"name\":\"request_duration_milliseconds\"},{\"dimensions\":{\"destination_service\":\"CATHAL_TEST_1\"},\"name\":\"request_bytes\"},{\"dimensions\":{\"destination_service\":\"CATHAL_TEST_1\"},\"name\":\"response_bytes\"},{\"dimensions\":{\"destination_service\":\"CATHAL_TEST_1\"},\"name\":\"tcp_connections_closed_total\"},{\"dimensions\":{\"destination_service\":\"CATHAL_TEST_1\"},\"name\":\"tcp_connections_opened_total\"},{\"dimensions\":{\"destination_service\":\"CATHAL_TEST_1\"},\"name\":\"tcp_received_bytes_total\"},{\"dimensions\":{\"destination_service\":\"CATHAL_TEST_1\"},\"name\":\"tcp_sent_bytes_total\"}]}"
}

Pretty print:
{
  "metrics": [
    {
      "dimensions": {
        "destination_service": "CATHAL_TEST_1"
      },
      "name": "request_messages_total"
    },
    {
      "dimensions": {
        "destination_service": "CATHAL_TEST_1"
      },
      "name": "response_messages_total"
    },
    {
      "dimensions": {
        "destination_service": "CATHAL_TEST_1"
      },
      "name": "requests_total"
    },
    {
      "dimensions": {
        "destination_service": "CATHAL_TEST_1"
      },
      "name": "request_duration_milliseconds"
    },
    {
      "dimensions": {
        "destination_service": "CATHAL_TEST_1"
      },
      "name": "request_bytes"
    },
    {
      "dimensions": {
        "destination_service": "CATHAL_TEST_1"
      },
      "name": "response_bytes"
    },
    {
      "dimensions": {
        "destination_service": "CATHAL_TEST_1"
      },
      "name": "tcp_connections_closed_total"
    },
    {
      "dimensions": {
        "destination_service": "CATHAL_TEST_1"
      },
      "name": "tcp_connections_opened_total"
    },
    {
      "dimensions": {
        "destination_service": "CATHAL_TEST_1"
      },
      "name": "tcp_received_bytes_total"
    },
    {
      "dimensions": {
        "destination_service": "CATHAL_TEST_1"
      },
      "name": "tcp_sent_bytes_total"
    }
  ]
}

Versus using an EnvoyFilter which does the exact same thing;
apiVersion: networking.istio.io/v1alpha3
kind: EnvoyFilter
metadata:
  name: stats-filter-1.17
  namespace: istio-system
spec:
  configPatches:
  - applyTo: HTTP_FILTER
    match:
      context: SIDECAR_OUTBOUND
      listener:
        filterChain:
          filter:
            name: envoy.filters.network.http_connection_manager
            subFilter:
              name: envoy.filters.http.router
      proxy:
        proxyVersion: ^1\.17.*
    patch:
      operation: INSERT_BEFORE
      value:
        name: istio.stats
        typed_config:
          '@type': type.googleapis.com/udpa.type.v1.TypedStruct
          type_url: type.googleapis.com/envoy.extensions.filters.http.wasm.v3.Wasm
          value:
            config:
              configuration:
                '@type': type.googleapis.com/google.protobuf.StringValue
                value: |
                  {
                    "metrics": [
                      {
                        "dimensions": {
                          "destination_service": "CATHAL_TEST_1"
                        }
                      }
                    ]
                  }
              root_id: stats_outbound
              vm_config:
                code:
                  local:
                    inline_string: envoy.wasm.stats
                runtime: envoy.wasm.runtime.null
                vm_id: stats_outbound

Yet only produces the following config:
"configuration": {
"@type": "type.googleapis.com/google.protobuf.StringValue",
"value": "{\n  \"metrics\": [\n    {\n      \"dimensions\": {\n        \"destination_service\": \"CATHAL_TEST_1\"\n      }\n    }\n  ]\n}\n"
}

Pretty print:
{
  "metrics": [
    {
      "dimensions": {
        "destination_service": "CATHAL_TEST_1"
      }
    }
  ]
}

Both appear to work identically, but the Telemetry approach repeats the config 10 times, once for each traffic stats metric.
This causes significant additional memory overhead in proxies with a large number of listeners. It also results in significantly increased pressure on Istiod, which has to calculate, store, compress, and distribute that config across the data plane.
Naturally the impact here is greater in larger clusters.
Version
istioctl version
client version: 1.15.3
control plane version: 1.17.5
data plane version: 1.17.5 (3 proxies)

kc version --short
Flag --short has been deprecated, and will be removed in the future. The --short output will become the default.
Client Version: v1.24.1
Kustomize Version: v4.5.4
Server Version: v1.24.15

/cc @whitneygriffith @zirain and @lei-tang

Thanks for reporting this, it's somehting we should improved.

Hello,
Same happened when switching from Telemetry V2 to Telemetry API and dropping labels aka tags on istio 1.18.2
Istio will generate tags_to_remove per metric name:
  "name": "istio.stats",
  "typed_config": {
    "@type": "type.googleapis.com/stats.PluginConfig",
    "metrics": [
      {
        "name": "request_messages_total",
        "tags_to_remove": [
          "destination_app",
          "destination_canonical_revision",
          "destination_canonical_service",
          "destination_cluster",
          "destination_principal",
          "destination_service",
          "destination_service_name",
          "destination_service_namespace",
          "destination_version",
          "destination_workload_namespace",
          "grpc_response_status",
          "source_app",
          "source_canonical_revision",
          "source_canonical_service",
          "source_cluster",
          "source_principal",
          "source_version",
          "source_workload_namespace"
        ]
      },
      {
        "name": "response_messages_total",
        "tags_to_remove": [
          "destination_app",
          "destination_canonical_revision",
          "destination_canonical_service",
          "destination_cluster",
          "destination_principal",
          "destination_service",
          "destination_service_name",
          "destination_service_namespace",
          "destination_version",
          "destination_workload_namespace",
          "grpc_response_status",
          "source_app",
          "source_canonical_revision",
          "source_canonical_service",
          "source_cluster",
          "source_principal",
          "source_version",
          "source_workload_namespace"
        ]
      },

In case of using Telemetry V2 name is missing and only one block of tags_to_remove present in configuration for all metrics.
 {
    "name": "istio.stats",
    "typed_config": {
      "@type": "type.googleapis.com/udpa.type.v1.TypedStruct",
      "type_url": "type.googleapis.com/stats.PluginConfig",
      "value": {
        "metrics": [
          {
            "tags_to_remove": [
              "destination_canonical_service",
              "destination_principal",
              "destination_version",
              "destination_app",
              "destination_cluster",
              "destination_service",
              "destination_service_name",
              "destination_service_namespace",
              "destination_canonical_revision",
              "destination_workload_namespace",
              "source_cluster",
              "source_app",
              "source_version",
              "source_principal",
              "source_workload_namespace",
              "source_canonical_service",
              "source_canonical_revision"
            ]
          }
        ]
      }
    }
  },

Example config to drop labels using telemetry api:
---
apiVersion: telemetry.istio.io/v1alpha1
kind: Telemetry
metadata:
  name: default-metrics
  namespace: istio-system
spec:
  metrics:
    - providers:
        - name: prometheus
      overrides:
        - match:
            metric: ALL_METRICS
            mode: CLIENT_AND_SERVER
          tagOverrides:
            source_app:
              operation: REMOVE
            source_cluster:
              operation: REMOVE
            source_version:
              operation: REMOVE
            source_principal:
              operation: REMOVE
            source_workload_namespace:
              operation: REMOVE
            source_canonical_service:
              operation: REMOVE
            source_canonical_revision:
              operation: REMOVE
            destination_app:
              operation: REMOVE
            destination_cluster:
              operation: REMOVE
            destination_version:
              operation: REMOVE
            destination_principal:
              operation: REMOVE
            destination_service:
              operation: REMOVE
            destination_workload_namespace:
              operation: REMOVE
            destination_canonical_service:
              operation: REMOVE
            destination_canonical_revision:
              operation: REMOVE
            destination_service_name:
              operation: REMOVE
            destination_service_namespace:
              operation: REMOVE
            grpc_response_status:
              operation: REMOVE

this's a known issue, PRs welcome.

Do we see this as a beta blocker for Telemetry API?

not stale


aws server migration error

vagrant up command error
hyper v not running,hyper v is in enable, virtual box running  ,but not looking 7 options

power shell not  answer, provider hyperv not run, provider virtual box runing, but not looking the 7 options

please shrare the screenshots


Create you own model

Create your own CNN model for the car.

Trainning it with the same datas.

Then competing in the most intense of competitions.

Now we need to merge every model back to main


podvm: allow the builder to checkout a refspec

The podvm builder dockerfile uses the CAA_SRC and CAA_SRC_BRANCH build arguments to, respectively, get the repository and branch from where the CAA sources should be cloned. However, it may be needed to checkout from a git refspec instead of the branch. For example, on CI it should build the sources from the pull request refspec.
This introduced the CAA_SRC_REF build argument that instructs the dockerfile to fetch and checkout on the refspec. Thus, that variable takes precedence over CAA_SRC_BRANCH when set.
Fixes #607
Signed-off-by: Wainer dos Santos Moschetta<EMAIL_ADDRESS>

For CI and development purposes, wouldn't it make more sense to allow checking out on a ref (and also allow to fetch different repos/branches) on the actual building Dockerfiles? it will allow to reuse the builder containers

For CI and development purposes, wouldn't it make more sense to allow checking out on a ref (and also allow to fetch different repos/branches) on the actual building Dockerfiles? it will allow to reuse the builder containers

Hi @snir911 !
I'm not sure I got your comment. With this changes it will be possible to checkout any repo and ref. For example:
$ docker build -t podvm_builder --build-arg CAA_SRC=https://github.com/wainersm/cc-cloud-api-adaptor --build-arg CAA_SRC_REF="refs/pull/23/merge" -f Dockerfile.podvm_builder .

I kept the CAA_SRC_BRANCH because checking a single branch will produce a smaller tree, thus a smaller image. Whereas to checkout a ref you need to clone the entire repo, fetch & then checkout the ref.
Do you think it can be done differently?

Hi, @wainersm, I meant allowing overriding/fetching/checking-out different repos/branches/refs in the building container i.e. in the Dockerfile.podvm* files instead of the Dockerfile.podvm_builder* files.

Hi, @wainersm, I meant allowing overriding/fetching/checking-out different repos/branches/refs in the building container i.e. in the Dockerfile.podvm* files instead of the Dockerfile.podvm_builder* files.

Good idea. This will allow reuse of the builder container and save time.
@wainersm wdyt ?

Hi, @wainersm, I meant allowing overriding/fetching/checking-out different repos/branches/refs in the building container i.e. in the Dockerfile.podvm* files instead of the Dockerfile.podvm_builder* files.

Good idea. This will allow reuse of the builder container and save time. @wainersm wdyt ?

ah, got it @snir911 . It is a good idea to me either, @bpradipt
One question though: it seems we don't need the sources on the builder image, rather completely move the cloning/checking-out logic to the building container?

Hi, @wainersm, I meant allowing overriding/fetching/checking-out different repos/branches/refs in the building container i.e. in the Dockerfile.podvm* files instead of the Dockerfile.podvm_builder* files.

Good idea. This will allow reuse of the builder container and save time. @wainersm wdyt ?

ah, got it @snir911 . It is a good idea to me either, @bpradipt
One question though: it seems we don't need the sources on the builder image, rather completely move the cloning/checking-out logic to the building container?

For released builds we might need it. For example builder image for release vX.Y.Z having all the pieces to reproduce the build.

@bpradipt , @snir911 ,
double checking I understood it: we want to allow developers to overwrite the caa repo and refspec on both builder and building containers? i.e. add the following change to all the Dockerfile.podvm.* files:
diff --git a/podvm/Dockerfile.podvm b/podvm/Dockerfile.podvm
index 5bca50f..4301f32 100644
--- a/podvm/Dockerfile.podvm
+++ b/podvm/Dockerfile.podvm
@@ -13,12 +13,23 @@ ARG PODVM_DISTRO=ubuntu
 ARG AA_KBC=offline_fs_kbc
 # If not provided, uses system architecture
 ARG ARCH=x86_64
+ARG CAA_SRC=""
+ARG CAA_SRC_REF=""

 ENV CLOUD_PROVIDER ${CLOUD_PROVIDER}
 ENV PODVM_DISTRO ${PODVM_DISTRO}
 ENV AA_KBC ${AA_KBC}
 ENV ARCH ${ARCH}

+RUN if [ -n "${CAA_SRC}" ]; then \
+      rm -rf cloud-api-adaptor && \
+      git clone ${CAA_SRC} cloud-api-adaptor;\
+    fi && \
+    if [ -n "${CAA_SRC_REF}" ]; then \
+      cd cloud-api-adaptor && \
+      git fetch origin ${CAA_SRC_REF} && \
+      git checkout FETCH_HEAD -b ${CAA_SRC_REF} ;\
+    fi
 # Installs add-ons for foreign target, if required
 RUN ./cloud-api-adaptor/podvm/hack/cross-build-extras.sh

@bpradipt , @snir911 ,
double checking I understood it: we want to allow developers to overwrite the caa repo and refspec on both builder and building containers? i.e. add the following change to all the Dockerfile.podvm.* files:

I got confused with builder and building containers :-). But the example that you quoted helps clear my confusion.
So we want to allow developers to overwrite CAA repo when building the Pod VM image, which is modifying all the Dockerfile.podvm.* files (and not the Dockerfile.podvm_builder.*)

diff --git a/podvm/Dockerfile.podvm b/podvm/Dockerfile.podvm
index 5bca50f..4301f32 100644
--- a/podvm/Dockerfile.podvm
+++ b/podvm/Dockerfile.podvm
@@ -13,12 +13,23 @@ ARG PODVM_DISTRO=ubuntu
 ARG AA_KBC=offline_fs_kbc
 # If not provided, uses system architecture
 ARG ARCH=x86_64
+ARG CAA_SRC=""
+ARG CAA_SRC_REF=""

 ENV CLOUD_PROVIDER ${CLOUD_PROVIDER}
 ENV PODVM_DISTRO ${PODVM_DISTRO}
 ENV AA_KBC ${AA_KBC}
 ENV ARCH ${ARCH}

+RUN if [ -n "${CAA_SRC}" ]; then \
+      rm -rf cloud-api-adaptor && \
+      git clone ${CAA_SRC} cloud-api-adaptor;\
+    fi && \
+    if [ -n "${CAA_SRC_REF}" ]; then \
+      cd cloud-api-adaptor && \
+      git fetch origin ${CAA_SRC_REF} && \
+      git checkout FETCH_HEAD -b ${CAA_SRC_REF} ;\
+    fi
 # Installs add-ons for foreign target, if required
 RUN ./cloud-api-adaptor/podvm/hack/cross-build-extras.sh

+1

@bpradipt would it make sense to completely remove the repo handling from the podvm_builder containers?
Once we'll have releases we can either modify the Dockerfiles to clone the latest release tag by default (unless CAA_REF,CAA_REPO... are specified) or to build the containers/image and tag it with the released version.

@bpradipt would it make sense to completely remove the repo handling from the podvm_builder containers? Once we'll have releases we can either modify the Dockerfiles to clone the latest release tag by default (unless CAA_REF,CAA_REPO... are specified) or to build the containers/image and tag it with the released version.

@snir911 Usually, the code will be tied to the toolchain - golang version, operator sdk version, rust version etc. The purpose of the builder container is to have the required toolchain that can build the specific code version.
If we decouple the code from the toolchain, then the purpose won't be met and we run the risk of failed or incorrect builds due to the incompatibility between the toolchain and the code.
Having said that it's not a hard problem to solve. For example we can include a toolchain version file with the code and ensure it's tracked with the code. @stevenhorsman already created an issue which should address this requirement.
Further we add a Makefile target to first check for compatibility with the toolchain when building the code. Once we have this support, we can remove the code from the builder container and only provide required toolchains in the builder container to speed up the different image creation process.
Does this make sense ?

I see now, thanks @bpradipt
BTW different problem might be there's no 1:1 mapping between the container image used for building and the podVM image used for running, as it may be slightly different (maybe irrelevant once everything is statically built)

@bpradipt @snir911 I updated this PR so that now the sources can only be overridden on the podvm build (i.e. the Dockerfile.podvm.* files). I also added a fix to CentOS, apparently they changed the base image recently.

/lgtm Thanks @wainersm Add a minor clarification and assuming this was tested. I didn't get time to test the changes.

Yep, it tested it. That's how I hit the problem I fixed for CentOS :)