total_vi.train()  ValueError: Expected parameter loc (Tensor of shape (256, 20)) of distribution Normal(loc: torch.Size([256, 20]), scale: torch.Size([256, 20])) to satisfy the constraint Real()

[TEXT HERE]

 scvi.model.TOTALVI.setup_mudata(
            mdata,
            rna_layer="counts" if rna_use_raw else None,
            protein_layer="counts" if protein_use_raw else None,
            modalities={
                "rna_layer": "multiomics" if self._use_hvg else "rna",
                "protein_layer": "protein",
            })

  total_vi = scvi.model.TOTALVI(mdata, **kwags)
  total_vi.train()

Traceback (most recent call last):
  File "../../saw_multianalysis/multiAnalysis.py", line 410, in <module>
    run()
  File "../../saw_multianalysis/multiAnalysis.py", line 405, in run
    main(rna_path, protein_path, bin_size, protein_list, out_dir, convert_py_bool(use_gpu), gpu, num_threads,
  File "../../saw_multianalysis/multiAnalysis.py", line 309, in main
    totalVI(rna_data, protein_data, prefix, proteins, out_dir, use_gpu, gpu, num_threads, report)
  File "../../saw_multianalysis/multiAnalysis.py", line 176, in totalVI
    total_vi = ms_data.tl.total_vi(
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/stereo/algorithm/total_vi.py", line 142, in main
    total_vi.train(use_gpu=use_gpu, **train_kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/scvi/model/_totalvi.py", line 299, in train
    return runner()
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/scvi/train/_trainrunner.py", line 82, in __call__
    self.trainer.fit(self.training_plan, self.data_splitter)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/scvi/train/_trainer.py", line 188, in fit
    super().fit(*args, **kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 696, in fit
    self._call_and_handle_interrupt(
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 650, in _call_and_handle_interrupt
    return trainer_fn(*args, **kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 735, in _fit_impl
    results = self._run(model, ckpt_path=self.ckpt_path)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 1166, in _run
    results = self._run_stage()
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 1252, in _run_stage
    return self._run_train()
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 1283, in _run_train
    self.fit_loop.run()
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/loops/loop.py", line 200, in run
    self.advance(*args, **kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/loops/fit_loop.py", line 271, in advance
    self._outputs = self.epoch_loop.run(self._data_fetcher)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/loops/loop.py", line 200, in run
    self.advance(*args, **kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/loops/epoch/training_epoch_loop.py", line 203, in advance
    batch_output = self.batch_loop.run(kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/loops/loop.py", line 200, in run
    self.advance(*args, **kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/loops/batch/training_batch_loop.py", line 87, in advance
    outputs = self.optimizer_loop.run(optimizers, kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/loops/loop.py", line 200, in run
    self.advance(*args, **kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/loops/optimization/optimizer_loop.py", line 201, in advance
    result = self._run_optimization(kwargs, self._optimizers[self.optim_progress.optimizer_position])
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/loops/optimization/optimizer_loop.py", line 248, in _run_optimization
    self._optimizer_step(optimizer, opt_idx, kwargs.get("batch_idx", 0), closure)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/loops/optimization/optimizer_loop.py", line 358, in _optimizer_step
    self.trainer._call_lightning_module_hook(
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 1550, in _call_lightning_module_hook
    output = fn(*args, **kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/core/module.py", line 1705, in optimizer_step
    optimizer.step(closure=optimizer_closure)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/core/optimizer.py", line 168, in step
    step_output = self._strategy.optimizer_step(self._optimizer, self._optimizer_idx, closure, **kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/strategies/strategy.py", line 216, in optimizer_step
    return self.precision_plugin.optimizer_step(model, optimizer, opt_idx, closure, **kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/plugins/precision/precision_plugin.py", line 153, in optimizer_step
    return optimizer.step(closure=closure, **kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/torch/optim/optimizer.py", line 391, in wrapper
    out = func(*args, **kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/torch/optim/optimizer.py", line 76, in _use_grad
    ret = func(self, *args, **kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/torch/optim/adam.py", line 148, in step
    loss = closure()
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/plugins/precision/precision_plugin.py", line 138, in _wrap_closure
    closure_result = closure()
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/loops/optimization/optimizer_loop.py", line 146, in __call__
    self._result = self.closure(*args, **kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/loops/optimization/optimizer_loop.py", line 132, in closure
    step_output = self._step_fn()
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/loops/optimization/optimizer_loop.py", line 407, in _training_step
    training_step_output = self.trainer._call_strategy_hook("training_step", *kwargs.values())
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 1704, in _call_strategy_hook
    output = fn(*args, **kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/pytorch_lightning/strategies/strategy.py", line 358, in training_step
    return self.model.training_step(*args, **kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/scvi/train/_trainingplans.py", line 559, in training_step
    inference_outputs, _, scvi_loss = self.forward(
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/scvi/train/_trainingplans.py", line 282, in forward
    return self.module(*args, **kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
    return forward_call(*args, **kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/scvi/module/base/_decorators.py", line 33, in auto_transfer_args
    return fn(self, *args, **kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/scvi/module/base/_base_module.py", line 276, in forward
    return _generic_forward(
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/scvi/module/base/_base_module.py", line 837, in _generic_forward
    inference_outputs = module.inference(**inference_inputs, **inference_kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/scvi/module/base/_decorators.py", line 33, in auto_transfer_args
    return fn(self, *args, **kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/scvi/module/_totalvae.py", line 491, in inference
    qz, ql, latent, untran_latent = self.encoder(
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
    return forward_call(*args, **kwargs)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/scvi/nn/_base_components.py", line 1029, in forward
    q_z = Normal(qz_m, qz_v.sqrt())
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/torch/distributions/normal.py", line 56, in __init__
    super().__init__(batch_shape, validate_args=validate_args)
  File ".conda/envs/multiAnalysis/lib/python3.8/site-packages/torch/distributions/distribution.py", line 80, in __init__
    raise ValueError(
ValueError: Expected parameter loc (Tensor of shape (256, 20)) of distribution Normal(loc: torch.Size([256, 20]), scale: torch.Size([256, 20])) to satisfy the constraint Real(), but found invalid values:
tensor([[nan, nan, nan,  ..., nan, nan, nan],
        [nan, nan, nan,  ..., nan, nan, nan],
        [nan, nan, nan,  ..., nan, nan, nan],
        ...,
        [nan, nan, nan,  ..., nan, nan, nan],
        [nan, nan, nan,  ..., nan, nan, nan],
        [nan, nan, nan,  ..., nan, nan, nan]], grad_fn=<AddmmBackward0>)
Epoch 1/400:   0%|

Versions:

python 3.8
scvi-tools 0.19.0

VERSION

Please try out our recent changes by installing scvi-tools from the main branch. I hope this is fixed there. Those will be released with scvi-tools 1.2

@canergen Thank you for your reply. Since scvi-tools 1.2 requires python > 3.8, I cannot upgrade the python version. Secondly, I tried to manually update the commit you fixed (https://github.com/scverse/scvi-tools/pull/2632/files) to _trainingplans.py, and the problem still occurs.

I am using python 3.12 and have the same problem when training a scvi.model.SCVI model. I tried to install the main branch or the 1.2.x branch and the same error was met.
cmd:
model.train(early_stopping=True, accelerator='cpu')    # same error with "mps"

error:
File [~/miniforge3/envs/sc/lib/python3.12/site-packages/torch/distributions/distribution.py:70](http://localhost:8890/lab/tree/~/miniforge3/envs/sc/lib/python3.12/site-packages/torch/distributions/distribution.py#line=69), in Distribution.__init__(self, batch_shape, event_shape, validate_args)
     68         valid = constraint.check(value)
     69         if not valid.all():
---> 70             raise ValueError(
     71                 f"Expected parameter {param} "
     72                 f"({type(value).__name__} of shape {tuple(value.shape)}) "
     73                 f"of distribution {repr(self)} "
     74                 f"to satisfy the constraint {repr(constraint)}, "
     75                 f"but found invalid values:\n{value}"
     76             )
     77 super().__init__()

ValueError: Expected parameter loc (Tensor of shape (128, 10)) of distribution Normal(loc: torch.Size([128, 10]), scale: torch.Size([128, 10])) to satisfy the constraint Real(), but found invalid values:
tensor([[nan, nan, nan,  ..., nan, nan, nan],
        [nan, nan, nan,  ..., nan, nan, nan],
        [nan, nan, nan,  ..., nan, nan, nan],
        ...,
        [nan, nan, nan,  ..., nan, nan, nan],
        [nan, nan, nan,  ..., nan, nan, nan],
        [nan, nan, nan,  ..., nan, nan, nan]], grad_fn=<AddmmBackward0>)

session info
-----
anndata     0.10.8
scanpy      1.10.3
-----
CoreFoundation              NA
Foundation                  NA
PIL                         10.4.0
PyObjCTools                 NA
absl                        NA
anyio                       NA
appnope                     0.1.4
arrow                       1.3.0
asttokens                   NA
attr                        24.2.0
attrs                       24.2.0
babel                       2.14.0
brotli                      1.1.0
certifi                     2024.08.30
cffi                        1.17.1
charset_normalizer          3.3.2
chex                        0.1.86
colorama                    0.4.6
comm                        0.2.2
contextlib2                 NA
cycler                      0.12.1
cython_runtime              NA
dateutil                    2.9.0
debugpy                     1.8.5
decorator                   5.1.1
defusedxml                  0.7.1
docrep                      0.3.2
etils                       1.9.4
executing                   2.1.0
fastjsonschema              NA
filelock                    3.16.1
flax                        0.9.0
fqdn                        NA
fsspec                      2024.9.0
gmpy2                       2.1.5
google                      NA
h5py                        3.11.0
idna                        3.10
importlib_resources         NA
ipykernel                   6.29.5
isoduration                 NA
jax                         0.4.31
jaxlib                      0.4.31
jedi                        0.19.1
jinja2                      3.1.4
joblib                      1.4.2
json5                       0.9.25
jsonpointer                 3.0.0
jsonschema                  4.23.0
jsonschema_specifications   NA
jupyter_events              0.10.0
jupyter_server              2.14.2
jupyterlab_server           2.27.3
kiwisolver                  1.4.7
legacy_api_wrap             NA
lightning                   2.4.0
lightning_utilities         0.11.7
llvmlite                    0.43.0
markupsafe                  2.1.5
matplotlib                  3.9.2
ml_collections              NA
ml_dtypes                   0.5.0
mpl_toolkits                NA
mpmath                      1.3.0
msgpack                     1.1.0
mudata                      0.3.1
multipledispatch            0.6.0
natsort                     8.4.0
nbformat                    5.10.4
numba                       0.60.0
numpy                       1.26.4
numpyro                     0.15.3
objc                        10.3.1
opt_einsum                  v3.3.0
optax                       0.2.2
overrides                   NA
packaging                   24.1
pandas                      2.2.2
parso                       0.8.4
patsy                       0.5.6
pickleshare                 0.7.5
platformdirs                4.3.6
prometheus_client           NA
prompt_toolkit              3.0.47
psutil                      6.0.0
pure_eval                   0.2.3
pycparser                   2.22
pydev_ipython               NA
pydevconsole                NA
pydevd                      2.9.5
pydevd_file_utils           NA
pydevd_plugins              NA
pydevd_tracing              NA
pygments                    2.18.0
pynndescent                 0.5.13
pyparsing                   3.1.4
pyro                        1.9.1+0a67ddc
pythonjsonlogger            NA
pytz                        2024.2
referencing                 NA
requests                    2.32.3
rfc3339_validator           0.1.4
rfc3986_validator           0.1.1
rich                        NA
rpds                        NA
scipy                       1.14.1
scvi                        1.1.6
send2trash                  NA
session_info                1.0.0
six                         1.16.0
sklearn                     1.5.2
sniffio                     1.3.1
socks                       1.7.1
sparse                      0.15.4
stack_data                  0.6.2
statsmodels                 0.14.3
sympy                       1.13.2
threadpoolctl               3.5.0
toolz                       0.12.1
torch                       2.4.0
torchgen                    NA
torchmetrics                1.4.2
tornado                     6.4.1
tqdm                        4.66.5
traitlets                   5.14.3
typing_extensions           NA
umap                        0.5.6
uri_template                NA
urllib3                     2.2.3
wcwidth                     0.2.13
webcolors                   24.8.0
websocket                   1.8.0
xarray                      2024.9.0
yaml                        6.0.2
zmq                         26.2.0
zstandard                   0.23.0
-----
IPython             8.27.0
jupyter_client      8.6.3
jupyter_core        5.7.2
jupyterlab          4.2.5
-----
Python 3.12.6 | packaged by conda-forge | (main, Sep 11 2024, 04:55:15) [Clang 17.0.6 ]
macOS-14.6.1-arm64-arm-64bit
-----
Session information updated at 2024-09-19 22:09

@raozuming We don’t support the use of outdated Python versions. There was an unfortunate stack of activation functions. It is safe to try several runs to get one that succeeds (like 10). If the issue persists, please set up a new environment with a supported Python version 3.10-3.12.
@mt1022 It’s a different issue as it’s a different model. Please check out: https://docs.scvi-tools.org/en/latest/faq.html. It is usually a problem with low count cells that give bad gradients. Please share your dataset size, all parts of your code and verify that your AnnData contains counts.

@canergen Thanks for your suggestion, I tried to use python 3.10 version + scvi-tools main branch code, and the related problems were fixed.

@canergen Thanks for your suggestion, I tried to use python 3.10 version + scvi-tools main branch code, and the related problems were fixed.

I  reinstalled latest release of scvi-tools and rerun the notebook. The model was successfully trained when setting "accelerator" to cpu. Therefore, the previous error is not related to my data. The same error will occur if mps was used. When I rerun only the train line with cpu after mps error, I got the same error too.
Anyway, I can keep using the cpu mode. My dataset has only 40k cells and it took about 20min to train with cpu.

Unfortunately MPS is not correctly implemented in pytorch and broadcasting is broken: https://github.com/pytorch/pytorch/issues/132605. Training doesn't fail if you use dispersion='gene_label' or similar. However, I don't trust MPS in its current state as we use broadcasting (repeating/reshaping without creating a new object) at several locations in the code. Replacing this would increase runtime and memory consumption. A longer description and the not recommended fix are described at: https://discourse.scverse.org/t/macbook-m1-m2-mps-acceleration-with-scvi/2075/7.

@canergen Hello, regarding issue #2981, since it occasionally occurs in the same batch of data, could you please describe the specific reason? Is it caused by the fact that the data of the shape (256, 20) in the protein data is all 0? If it is a data problem, could you provide the filtering conditions to filter the abnormal data externally? Thank you!


From where can i get the Dataset

kindly share the link of the dataset used

http://<IP_ADDRESS>:8181/dataset/


don't embed commit SHA in as many files

This PR is intended to solve problems we've had with the mathlib_docs repo. Previously, every update of the docs led to tens of thousands of new blobs being created, because the commit SHA was embedded in every doc page and in every src redirect page.
I made the following changes:

[x] source links in the HTML files of the doc pages now link to master, and nav.js rewrites the URLs to point to the precise commit when the page loads. To test: visit any doc page and click a "source" link; it should take you to the right place on GitHub. It may be worth testing both pages in mathlib and in core Lean since GitHub links for the latter aren't rewritten.
[x] the src redirect pages now do not contain the commit SHA; a separate add_commit.js file performs the redirect. To test: try this src link for dvd_sub_pow_of_dvd_sub (in mathlib, requires a rewrite) and this src link for nat (in core Lean, no rewrite required).

For the src redirect pages, I'm thinking of trying out @digama0's idea:

the find/polynomial/src page can have the form:

<script href="add_commit.js"></script>
<script>redirectTo("src/data/polynomial/basic.lean#L24")</script>
<noscript>
<meta http-equiv="refresh" content="0;url=https://github.com/leanprover-community/mathlib/blob/master/src/data/polynomial/basic.lean#L24">
</noscript>

add_commit.js:

function redirectTo(tgt) {
  window.href = "https://github.com/leanprover-community/mathlib/blob/<commit>/" + tgt;
}

#deploy

#deploy

#deploy

Nice! As far as I can tell this works as expected.
With this change, the update diffs should be pretty small. Should we change the push logic (in another commit) to do non-destructive updates?

#deploy

With this change, the update diffs should be pretty small. Should we change the push logic (in another commit) to do non-destructive updates?

We're force pushing because cloning the history was taking too long, right? I think we can try out going back to normal pushes.

We're force pushing because cloning the history was taking too long, right? I think we can try out going back to normal pushes.

The other reason was space: the generated documentation is several gigabytes large, and most of the files are changing every time (due to the git revisions).

#deploy


Fix app_metadata and user_metadata

 Issue by landonreed
Thursday Aug 30, 2018 at 19:04 GMT
Originally opened as https://github.com/catalogueglobal/datatools-ui/pull/240

This (in conjunction with changes in datatools-server) allows new Auth0 tenants to work with Data
Tools. Instead of fetching the user profile from Auth0, we fetch it directly from the
datatools-server API. The server decodes the JWT token into the user profile and re-maps certain
fields (e.g., the scoped app_metadata and the user_id fields) to keys the front end expects.
fixes #207

landonreed included the following code: https://github.com/catalogueglobal/datatools-ui/pull/240/commits

 Comment by landonreed
Thursday Aug 30, 2018 at 19:30 GMT

@evansiroky, could you verify that this works with a new Auth0 tenant and the existing dev tenant?  You'll need to checkout the fix-auth0-new-tenants  branch for both the server and the UI. I've tested with both, but I would like to confirm it's all working.  One thing I haven't tested (and don't necessarily expect to work) is the Data Tools user management UI under a new Auth0 tenant.

 Comment by landonreed
Friday Oct 12, 2018 at 16:49 GMT

Closing in favor of #250


Rebuild iced-x86 to pick up rustdoc fix

Crate name
iced-x86
Build failure link
N/A
Additional details
Technically the build succeeded, but there is a page (CodeAssembler) that crashes Chrome tabs. We've updated rustdoc to improve the situation so it no longer does that, so it would be great to rebuild the crate.

Rebuild was queued


Breadcrumb implement exceptions

Kapil Dhiman
5 hours ago
Hi
@Paolo Mottadelli
this is regarding breadcrumbs
We had a discussion on breadcrumbs and here is the feedback.
Breadcrumb will follow the URL structure but the last slash in pathname will be replaced with the h1 of the page.
Need to exclude some path from breadcrumb e.g ['en', 'assets', 'br', 'img'], etc. We'll provide the final array that should be excluded
Some paths in Breadcrumb don't have URLs so they will have a custom URL e.g.
[
'app-note' => 'https://www.moleculardevices.com/search-results#t=All&sort=relevancy&f:@md_contenttype=[Application Note]',
'ebook' => 'https://www.moleculardevices.com/search-results#t=All&sort=relevancy&f:@md_contenttype=[eBook]'
]
we'll provide the final list.
Please let me know if you've any concerns.
Kapil Dhiman
1 hour ago
to exclude the path I am taking an e.g. of this url
https://www.moleculardevices.com/en/assets/customer-breakthrough/br/university-of-california-davis
en, assets & br wont be part of the breadcrumb
Molecular DevicesMolecular Devices
University of California, Davis uses the MultiWash washer to engineer replacement and temporal bridge tissue
Leach Laboratory at UC Davis focuses on cellular and tissue engineering. Kaitlin Murphy helps engineer functional replacement and temporary bridge tissues.
Kapil Dhiman
1 hour ago
in the same way, there will be another path to exclude
Kapil Dhiman
1 hour ago
For above url breadcrumb should be
Home -> Customer Breakthrough -> University of California, Davis uses the MultiWash washer to engineer replacement and temporal bridge tissue
Kapil Dhiman
1 hour ago
In 2nd case e.g.
https://www.moleculardevices.com/en/assets/app-note/bpd/easy-picking-with-qpix-400
In this url need to exclude 'en', 'assets', 'abd' and need to add custom  link for app-note
so breadcrub should be
Home -> App Note -> Easy picking with the QPix 400: multiple selection modalities, wide range of microorganisms
and App Note should point to https://www.moleculardevices.com/search-results#t=All&sort=relevancy&f:@md_contenttype=[Application Note]
Molecular DevicesMolecular Devices
Easy picking with the QPix 400: multiple selection modalities, wide range of microorganisms
Explore QPix colony pickers that support a wide variety of microorganisms & multiple selection modalities, including fluorescence intensity, blue/white selection, size & proximity.

@kapildhiman please post the full list of exceptions in this issue


process_file() not working

I'm trying to set up code coverage with my MatpowerCases.jl package. I can't get Coveralls to report code coverage; it says it cannot find coverage data. I suspect this problem is related to an error I get when I try to use Coverage locally.
The README instructions worked well for me until I ran covered_lines, total_lines = coverage_folder(). This was the output:
src/MatpowerCases.jl
Skipped src/MatpowerCases.jl
src/MatpowerCases.jl.cov
Skipped src/MatpowerCases.jl.cov
0-element Array{Any,1}

So Coverage skipped the only .cov file in src/. But when I use covered_lines,total_lines = coverage_file("src/MatpowerCases.jl"), I get this:
(11,13)

which matches visual inspection of the MatpowerCases.jl.cov file in src/.
I went digging in the code and found that my .cov file is being skipped because this line throws an error, so I ran it in a REPL and got this:
julia> new_sf = Coveralls.process_file("src/MatpowerCases.jl.cov","src")
ERROR: `convert` has no method matching convert(::Type{Int64}, ::Nothing)
 in setindex! at array.jl:307
 in merge_coverage_counts at /home/jkersulis/.julia/v0.3/Coverage/src/Coverage.jl:22
 in process_cov_with_pids at /home/jkersulis/.julia/v0.3/Coverage/src/Coverage.jl:83
 in process_file at /home/jkersulis/.julia/v0.3/Coverage/src/Coverage.jl:194

So it traces back to this function. Why is my package having this problem?

The wrong "skip" is actually the first one, the messages are silly. Can you try
Coveralls.process_file("src/MatpowerCases.jl","src")

Whoops, sorry about that.
Looks like the error is the same:
julia> Coveralls.process_file("src/MatpowerCases.jl","src")
ERROR: `convert` has no method matching convert(::Type{Int64}, ::Nothing)
 in setindex! at array.jl:307
 in merge_coverage_counts at /home/jkersulis/.julia/v0.3/Coverage/src/Coverage.jl:22
 in process_cov_with_pids at /home/jkersulis/.julia/v0.3/Coverage/src/Coverage.jl:83
 in process_file at /home/jkersulis/.julia/v0.3/Coverage/src/Coverage.jl:194

Here is my coverage file.
Update: I found that
process_cov("src/MatpowerCases.jl.cov")

returns an array with one entry per line of code, but
process_cov_with_pids("src/MatpowerCases.jl.cov","src")

or
process_cov_with_pids("src/MatpowerCases.jl","src")

both throw this error:
ERROR: `convert` has no method matching convert(::Type{Int64}, ::Nothing)
 in setindex! at array.jl:307
 in merge_coverage_counts at /home/jkersulis/.julia/v0.3/Coverage/src/Coverage.jl:22
 in process_cov_with_pids at /home/jkersulis/.julia/v0.3/Coverage/src/Coverage.jl:83

I think your problem is that you are using 0.3. The change to use PIDs was made only on 0.4. But I might have made an implicit assumption when I updated the code that requires 0.4. Let me take a look - Coverage.jl obviously still needs to work with 0.3!

Fixed now

Everything works locally, and Coveralls matches what I see on my machine. Thanks!


Error: no step configured

Guys am I missing something here? Following your readme I get the following error:
fly -t concourse-poc sp -p terraform-pipeline -c pipeline.yml -v aws-access-key=redacted -v aws-secret-key=redacted
error: error unmarshaling JSON: while decoding JSON: no step configured

pipeline.yml:
---
resources:
  - name: master-branch
    type: git
    source:
      uri: https://github.com/rossrollin/terraform-poc
      branch: master
  - name: common-tasks
    type: git
    source:
      uri: https://github.com/telia-oss/concourse-tasks.git
      branch: master

jobs:
  - name: terraform-pipeline
    serial: true
    plan:
      - aggregate:
        - get: master-branch
          trigger: true
        - get: common-tasks
          params: { submodules: [ terraform ] }
        - task: terraform-plan
          file: common-tasks/terraform/0.12.29.yml
          input_mapping: { source: master-branch }
          params:
            command: plan
            cache: true
            access_key: ((aws-access-key))
            secret_key: ((aws-secret-key))
            directory: terraform-poc/dev

Very new to concourse though so could be doing something wrong, I just cant figure out what

Removing the aggregate bit fixes it.


Update or remove Vector skin's "VectorGeneratedSkinData" patch

Canasta contains a patch to the Vector skin, by @pastakhov, that adds the hook "VectorGeneratedSkinData" to the Vector skin's VectorTemplate.php file:
https://github.com/CanastaWiki/Canasta/blob/master/_sources/patches/skin-vector-addVectorGeneratedSkinDataHook.patch
This patch will not work any more with the MW 1.39 version of Canasta, because the file VectorTemplate.php no longer even exists, as far as I can tell. So the patch needs to be either updated or removed - depending, I assume, on whether that hook is still being used.

it is used in https://github.com/WikiWorks/FsHeader extension by FamilySearch.    But right now it seems the extension is not used. We'll clarify about the extension, most likely the patch should be removed.

Yes - I can confirm that this extension (and this hook) are no longer used on FamilySearch, and thus that, as far as I know, no one is using  this hook.


Large Sized Disassembly in Watch View overflows the next Watch Row.

If you try to view a large sized disassembly (large byte size) in the watch view it will be displayed like this - visual overflowing into the next watch rows.

Yeah, needless to say really, the disassembly visualizer (and basically every complex visualizer) is still heavily "under construction". I'll update this thread when this is further along.

This is now fixed on dev, where the disassembly visualizer is identical to the regular disassembly tab UI. This will be in 0.9.12. I don't recommend pulling from dev just yet, since there are a few broken things I am still fixing before release, but nonetheless this fix will be in the next build.


Return proper CORS headers in all REST API responses

The following middleware might be useful:
https://www.gorillatoolkit.org/pkg/handlers#CORS

I'm cloning this issue to the smart-proxy repository and closing this one due to issue #47


Reels views

When I retrieve viewcount of a REEL post, I get a number that is lower of the real one that I see on Instagram. This doesn't happen with the normal VIDEO posts, only with REELS. How do you get this number?
Thanks

Got it.

what was the solution?


fix(genQA): show no title in citation when we have empty title

SVCC-3635
Show No title when citation have empty title.
Before:

After:

I suppose the fix will be implemented in Quantic separately, right?

Yes, I don't think they have a ticket for that, I will inform Etienne


Recursive glob

It would be great to have this action support recursive globs so you can check files at any arbitrary level. By default bash has them not enabled. You can enable them by just running shopt -s globstar. But given the way all this is packaged up, not something that can be done as far as I can tell outside the action.
Adding that option to the script is technically a breaking change, so wanted to file an issue rather than a pr for it. Maybe this is a good option to add?

Is what you're asking for the ability to run woke **/*.txt which would only run against all the text files recursively?

Running this action with **/*.txt only causes it to check files up to one folder deep. So it does not check my/very/nested/folder/file.txt. Ideally that would be possible.

Got it, just making sure I'm understanding what you're asking. In older versions, this functionality was available (it still says so in the docs) but it clearly no longer works as expected.
I'm going to move this issue to https://github.com/get-woke/woke which is the repo where this should be handled. Thanks for reporting!


Fix longstanding crash when prefs.yaml is corrupted

Fix longstanding crash when prefs.yaml is corrupted
Instead, warn the user with a popup dialog, and reset settings to default.
Every few months, a user would report corrscope failing to open due to a corrupted prefs.yaml. This commit should stop all occurrences of that from now on. The next commit will help ensure that we never save corrupted files in the first place.
Atomically save prefs.yaml to prevent file corruption
This adds a dependency on the atomicwrites package. Note that it was already a transitive dev-dependency, but is now a direct dependency as well.
On Linux, atomicwrites fsyncs the directory after saving. This is unnecessary for consistency after a system crash, and slows down program shutdown slightly. But I'm too lazy to vendor or rewrite the library to not fsync the directory.

[ ] If you are a maintainer (only @nyanpasu64 at the moment), make sure to edit the changelog if needed. Otherwise, a maintainer will edit the changelog.

Codecov Report

Merging #377 (76499d8) into master (f80730f) will decrease coverage by 0.08%.
The diff coverage is 42.85%.

@@            Coverage Diff             @@
##           master     #377      +/-   ##
==========================================
- Coverage   77.80%   77.72%   -0.09%
==========================================
  Files          26       26
  Lines        3685     3694       +9
==========================================
+ Hits         2867     2871       +4
- Misses        818      823       +5

Impacted Files
Coverage Δ

corrscope/config.py
91.90% <0.00%> (+0.47%)
:arrow_up:

corrscope/settings/global_prefs.py
70.58% <33.33%> (-1.29%)
:arrow_down:

corrscope/gui/__init__.py
45.71% <50.00%> (-0.02%)
:arrow_down:

Continue to review full report at Codecov.

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


Sequence writer

Using this PR to experiment with different options for SequenceWriter
Not intended to be merged before ModelSerialization goes live
Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Core


bot threads

added thread functionality
threads can be named
threads can be deleted
if someone tries to destroy a thread that doesn't exist or that they don't own, bot reacts with 😡
added void return types to be more explicit
added some documentation
added some basic arg parsing

@hunter-graves Looks like there is a package.json in the root of the file hierarchy and also a node_modules folder and package.lock.json please delete those files


Error: Unknown switch -f in 8.0.0

Describe the problem
Running copier with copier -f doesn't work in 8.0.0
Template
None
To Reproduce

Run copier -f

Logs
Error: Unknown switch -f
------
copier 8.0.0

Create a new project from a template.

Docs in https://copier.readthedocs.io/

WARNING! Use only trusted project templates, as they might execute code with the
same level of access as your user.

Usage:
    copier [SWITCHES] [SUBCOMMAND [SWITCHES]] args...

Hidden-switches:
    -h, --help         Prints this help message and quits
    --help-all         Prints help messages of all sub-commands and quits
    -v, --version      Prints the program's version and quits

Sub-commands:
    copy               Copy from a template source to a destination.; see
                       'copier copy --help' for more info
    recopy             Recopy a subproject from its original template; see
                       'copier recopy --help' for more info
    update             Update a subproject from its original template; see
                       'copier update --help' for more info

Expected behavior
Copier --help-all states -f still exists:
 -l, -f, --defaults              Use default answers to questions, which
                                    might be null if not specified.

so it should work
Screenshots/screencasts/logs
No response
Operating system
Linux
Operating system distribution and version
Ubuntu 22.10
Copier version
copier 8.0.0
Python version
Python 3.7.3
Installation method
pipx+pypi
Additional context
No response

-l and --defaults give the same error message

Figured it out. I missed the introduction of sub commands. I need to run copier update -f.
The documentation here should also be updated:
https://pypi.org/project/copier/8.0.0/

That is expected and was listed in the breaking changes: flags are now passed to subcommands, like copier update -f. See the v8 remease notes :)


Light not adapting as expected

Version information: v1.4.1
Description:
I just installed the integration and set it up. It appears to be working, however sun set was at 5:55pm it’s currently 7:51pm and the lights are at 38% I would expect the lights to be at my min brightness by now. Not unless I’m misunderstanding how it all works.
I don’t have any errors messages.
Is it possible that the time zone or location is incorrect?

Hello, I have the same. I bought a GLEDOPTO 12W RGB CCT Zigbee 3.0 bulb E26 E27 Pro. The bulb changes color and brightness, but never goes down to 0% brightness and does not start from that. I don't know what settings to make.

hi, as far as i can see this is the expected behavior. i created some sensors to track the sun, brighness and color temperature:

this is the brightness:

my sleep_mode is active from 22:00 to 06:00, this explains the bigger changes at this times.
as you can see, the brightness will still be at 100% at sunset and slowly decrease to your minimum brightness value over the next hours.

I came here with the same question and I think it was a misunderstanding about when the minimum brightness is reached. I expected it to reach min_brightness right when it was fully dark outside but it actually dims gradually throughout whole the night.

Hello. So this is not a behavior consistent with the observed from the point of my residence. It is a sine wave. If so, how, in addition to manual control, to force working hours? Setting the sunrise time and sunset time manually?
My next question is about the brightness of the light bulb. In RGB mode it is much darker. In temperature control mode, it shines three times brighter. However, there is no gradual change in this mode.

I came here with the same question and I think it was a misunderstanding about when the minimum brightness is reached. I expected it to reach min_brightness right when it was fully dark outside but it actually dims gradually throughout whole the night.

That was my assumption as well, but sounds like that isn't the case.  Maybe this is more of a feature request than an issue - not sure.
I would like to be able to configure min-max brightness to work together with sunrise/sunset if possible.  Obviously there would be around a 30/60 minute buffer since sunset doesn't mean it is pitch dark outside instantly.

My expectation is similar, I want to be able to configure min and max brightness to work together with sunrise/sunset if possible. Of course, a buffer of around 30/60 minutes would be necessary, as sunset doesn't mean it's immediately dark outside. If you can ask for such functionality, I'm also in it.

+1 to this and following this thread. I'd love to see some way to customise the curve involved in blending "day" and "night" presets for temp/brightness!

The problem is that the current implementation assumes you want to reach minimum brightness at midnight and maximum brightness at sunrise.
The offset parameters are a poor option for adjusting this as they also impact the times that midnight & noon fall.
My suggestions:

Add new, or switch the existing offsets to work without affecting midnight/midday calculations. Clamp them within "event boundaries" so you don't break the math.
Add "dimming duration", "finish dimming X time after sunset", "brightening duration", and "finish dimming X time after sunrise" options.

This should let us get the exact behaviour we're looking for - controlling the start/end of the window around sunrise/sunset for brightness transition times.
PS. I believe the current behaviour WRT colour temp is fine; I don't need to customise this personally. Brightness, however... Yeah, I want it darker faster, please 😅

Is this issue not fixed with the addition of max_sunrise_time, min_sunset_time, sunset_time, and sunrise_time variables?


Issues while trying to connect Presence/Private Channels with authModule (Sanctum & Soketi)

I am using Soketi as the server to receive requests. It seems that public channels work fine, but when I enable auth for presence/private channels, I end up running into the issue below. I am using Sanctum for authentication, I've enabled the required configuration for Laravel for Sanctum to work with the authEndpoint below.
nuxt.config.js
buildModules: [
  ...
  '@nuxtjs/laravel-echo',
],

echo: {
    broadcaster: 'pusher',
    authEndpoint : 'http://<IP_ADDRESS>:8000/api/broadcasting/auth',
    authModule: true,
    wsHost: '<IP_ADDRESS>',
    wsPort: 6001,
    key: 'my_key',
    encrypted: true,
    enableLogging: true,
    connectOnLogin: true,
    disconnectOnLogout: true,
    disableStats: false,
  }

Error:
[x] pusher-with-encryption.js?8024:6425 WebSocket connection to 'wss://<IP_ADDRESS>/app/my_key?protocol=7&client=js&version=7.0.6&flash=false' failed:

[x] GET https://stats.pusher.com/timeline/v2/jsonp/1?session=Mjg3NTYxNjU2&bundle=MQ%3D%3D&key=ekRZaGZrMnVrMkdmUlBJckxkaFZuRkZiR1pGUjl6bUs%3D&lib=anM%3D&version=Ny4wLjY%3D&cluster=bXQx& ...

Managed to get past the issue by adding in forceTLS: false to nuxt.js.config
However, now I get the following error,
POST http://<IP_ADDRESS>:8000/api/broadcasting/auth 401 (Unauthorized)

Any ideas?

If I do something like:
nuxt.config.js
echo: {
    ...
   connectOnLogin: true,
}

Then I get:
Error:
[x] Refused to set unsafe header "referer" - pusher.js?782e:863

@flyingspoon - did you figure out an answer to this?

@flyingspoon - did you figure out an answer to this?

Yes, I solved it by making my own version of laravel-echo.
I believe if I remember correctly, I ended up utilizing a specific version of Echo (might be wrong), then re-coded parts of this plugin so it works with Laravel 9x.
If you would like to see it I can post it later today, it gets the job done! :)

If you would like to see it I can post it later today, it gets the job done! :)

@flyingspoon - yes! that would be extremely helpful! :)

If you would like to see it I can post it later today, it gets the job done! :)

@flyingspoon - yes! that would be extremely helpful! :)

You can find the code below, I made it work for Soketi & Laravel Echo within Nuxt 2x:
https://github.com/flyingspoon/nuxt-echo
I last touched it when I opened this issue which was in May. So I have written down what I could remember, sorry if I have missed out anything! Good luck, and happy coding! 👍


226-yolov7 on Windows (errors or just warning)

It seems the code will still run even due to the deps error.

yes, it is just pip warning about potential dependency conflict

Ok. Would this installation perform self update? I see it connects to the git. How is it tracking versioning?


Handle IP and port conflict errors more gracefully

this will produce things like
130 doanac@andy-wg-server:~$ sudo ./factory-wireguard.py -f foo -t foo -k foo enable
External Endpoint: <IP_ADDRESS>:5555
VPN Address: <IP_ADDRESS>
ERROR: A UDP socket is already opened on <IP_ADDRESS>:5555

1 doanac@andy-wg-server:~$ sudo ./factory-wireguard.py -f foo -t foo -k foo enable --port 5558
External Endpoint: <IP_ADDRESS>:5558
VPN Address: <IP_ADDRESS>
ERROR: IP Address <IP_ADDRESS> is already in use


New "on completion" option: Keep X items

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.
[X] I am only requesting a single feature. Multiple changes should be split into individual requests, with links between them.
[X] I believe my requested feature will be generally applicable to other users, not just me: it is not uniquely tied to my personal workflow.

🔖 Feature description
Right now, I have two choices for completed recurring tasks:

🏁 keep gives me an ever-growing list that makes my tasks file more difficult to manage.
🏁 delete gives me no way to see a task's history, or what items I managed to complete for the day.

I would like a third option: 🏁 keep X, where X is an integer.
✔ Solution

Day 1: I define a daily recurring task, and include 🏁 keep 1 in the definition..
Day 1: I mark the task complete. A new task for Day 2 is created, with Day 1 retained underneath.
Day 2: I mark the Day 2 task complete. A new task for Day 3 is created, with Day 2 retained underneath. The task for Day 1 is deleted.

Because 1 previous item is retained, I can see the completed task in my query view for the day.
Similarly: I can set the task to 🏁 keep 7. Now I can see how consistently I've completed the task during a weekly review.
❓ Alternatives
Archiving tasks on completion to another note file, or a tool for cleaning them up (say, "run this to delete any completed items older than X days"), would help with many of these use cases. I do think this is distinct enough to be useful in its own right, though, even if it's only for seeing your task progress during the day.
📝 Additional Context
No response

Do you have any ideas about how this could be implemented, given that task lines are independent entities?

Because 1 previous item is retained, I can see the completed task in my query view for the day.

A new feature isn't needed for this particular case, though.
I just phrase my daily note to search for any tasks that were created today and have onCompletion is delete...

Do you have any ideas about how this could be implemented, given that task lines are independent entities?

I'll leave this open a day or so more, in the hope that you have time to reply...

Do you have any ideas about how this could be implemented, given that task lines are independent entities?

I'll leave this open a day or so more, in the hope that you have time to reply...

Closing. Will re-open if you have time to reply.


Event handlers

It may make sense to have events where the HIL admin can cause arbitrary logic to run.
For example, before giving a node to a user out of the free pool, it could implement:

a quota check
billing info
node operations (power on, assign to a provisioning network)

Places we could hook:

Node alloc/free
Net alloc/free

Tricky to watch out for is synchronization/race conditions.

+1.
This could be as simple as doing a POST on some endpoint for notifications. Re: alloc/free race conditions, we could have an option that makes it so when a node is "freed," it goes into a sort of "limbo," the external hook is notified, and that hook makes a separate call to actually return it to the free pool.
We could also, rather than having an explicit option, just have this happen when you have a hook set up for that.


Example/mp4sample supports fragmented mp4

Example/mp4sample supports fragmented mp4.
Add some fields into trun box :

first_sample_flags
sample_durations
sample_flags
sample_cts

And set correct flag in trun box.

Hi @alfg
Thanks for your comments, and I try to fix it :

Field data_offset in TrunBox is optional. The fragmented.mp4 does not have data_offset in TrunBox.
Extract duplicated codes as a  new function.

Now mp4sample works well with fragmented.mp4.

Hi @alfg
Thanks for your comments, and I try to fix it :

Field data_offset in TrunBox is optional. The fragmented.mp4 does not have data_offset in TrunBox.
Extract duplicated codes as a  new function.

Now mp4sample works well with fragmented.mp4.

@nintha Thanks for fixing!

@nintha Thanks for fixing!


@mention a user in Ms teams bot

I read this document https://docs.microsoft.com/en-us/microsoftteams/platform/concepts/bots/bot-conversations/bots-conv-channel
where its mentioned on the top that 'You can also access Teams-specific functionality within your bot like querying team information and @mentioning users'
I am interested to know how can I achieve @mentioning users via BOT that I have in MS teams as 1:1/personal scope.
I am aware of the process where I need to build a .zip with manifest.json and icons using App store and sideload that as a App in Ms teams channel. But, this will allow me to do @ mention {Bot name} so and I can parse user input in bot code. but, I am trying to see if same BOT can fetch users so I can @ mention users as well.
Desired state that I would like to achieve
For BOT in personal / 1:1 scope where we add a BOT by searching Bot ID in teams
1 . User navigates to bot in personal chat
2. user types 'Can you assign this ticket to @yinaa @Stevenic for quick fix ?'
3. Bot's business logic , (nodejs source code) gets above to users info. / user object like name etc..
4. Bot processes the request
For BOT in teams scope where we created .zip APP and sideloaded to a teams channel
1 . User @ {botName} in teams channel
2. user types '@ {botName} Can you assign this ticket to @yinaa @Stevenic for quick fix ?'
3. Bot's business logic , (nodejs source code) gets above to users info. / user object like name etc..
4. Bot processes the request

@rjgmail88 - This can be achieved in a channel. Please have a look at Node.js example code: Check for and strip @bot mention.
You can not @mention users in 1:1 conversation with the bot. You can also consider adding tabs and bots to chats which is in Public Developer Preview. In group chat you can at @mention the users which are part of it.

@Wajeed-msft , thank you for your suggestion. Let me reiterate what I understood from your comment.
A Bot which is added to a channel (scope=team) can be @ mentioned by user and same user can @ mention other users from same channel.
so user can type this '@ {botName} Can you assign this ticket to @yinaa @Stevenic for quick fix ?''
but, same user won't be able to @ mention anyone else from outside of the channel ? Let's say we 20K plus people in our o365 tenant and our bot should be able to @ mention anybody.
Should this be a feature request ?
Our use case:- We have a support ticket BOT which can be used to assign support tickets to other user. ex: User A can talk to a Bot and @ mention other people so they get notified. and @ mentioning should happen global level meaning anybody can @ mention anybody.

Yes this is a feature request - and an expensive one actually because it's a spam vector and a performance challenge (AutoComplete). I see the use case though - it's a good one.

Just want to add that we want to implement very similar functionality in our company 20k+ users (and we actually have it in our own custom web implementation right at the moment and want to mirror it in Teams)
So for example, we want to use messages like 'Bot, book 1 on 1 meeting with @someone' (in private conversation with bot) and we expect it to search over all users in our tenant and suggest the results alongside with user input; something like autocomplete input.

+1,
our requirement is kind of same where 1:1 conversation with bot and the user can select another users (multiple) in 0365 tenant in msteams using autocomplete and the bot in backend will get their id to process further.

For the booking a meeting scenario, you could have your bot return a Task Module and embed your existing web service as a web view in the task module. Essentially the flow could be:

User says: @bot book a 1-1 meeting
Bot responds with a task module (modal popup)
User uses the task module to add the meeting details
Bot creates the meeting (using either Graph or a deeplink)
bot sends the confirmation response message to the user

The missing piece in this flow would be the bot wouldn't necessarily be able to message the other party, unless they already had the bot installed. You could solve this problem by pinning/installing the bot at the tenant level.
Alternatively (or additionally), if you already have a web experience you could create a tab with a personal scope and embed that into Teams.

Hi @clearab. Would appreciate if you can help detailing the above more where in we call a deep link inside the Bot. Can you please help on the below ?

Are these task modules supported in other channels of the Bot. If a web chat channel is enabled for bot, can we reuse the task modules built. How the user experience is going to be ?
How do we secure the deep links for calling inside Bot ? Do we need build any additional security ?
Are these supported on mobile version of MS teams Bot ?
What are the limitations of task modules ?

Hi Team,
Could you please clarify above to further decide on the approach ?

Hi All,
Any update on above issue ?


FeatureFlagservice is broken

Bug Report
Which version of the demo you are using? 1.4-latest
Symptom
When I try to edit featureflags, the POST request path goes to feature/featureflags/edit when it should go to /featureflags/edit
What is the expected behavior?
Editing the flag should POST to /featureflags/edit

I ran into this issue as well, and it was caused by me accessing the featureflagservice directly. Apparently it should be accessed through the frontendproxy service, which is where the feature/ prefix comes into play. For example, hitting:
POST http://otel-demo-frontendproxy:8080/feature/featureflags/1

(where otel-demo-frontendproxy is the service name in k8s, or the hostname of the frontendproxy)
I haven't been able to find this documented anywhere outside of Github issues, so maybe this issue should be repurposed for updating the docs on how to update FF's since this doc doesn't mention it at all.

This line is adding /feature prefix to all urls:
https://github.com/open-telemetry/opentelemetry-demo/blob/main/src/featureflagservice/config/config.exs#L19
changing:
url: [host: "localhost", path: "/feature"]

to
  url: [host: "localhost"],

frontend rendering works now.
@austinlparker : is there a reason why this /feature/ prefix to the urls is needed?

The demo is meant to be accessed through the frontend-proxy service only on the /feature/ route. This is so the demo only needs to expose a single port for all UIs/services that need to be exposed.
I will close this issue since it refers to an older demo version, and the feature flag service has received updates that specifically changed the edit page to introduce new functionality.


[Feature]

Version information

Laravel: v5.8
Voyager: v1.2
PHP: 7.2
Database: MySQL 8.0

Description of problem

We are using Voyager for our Admin dashboard. Also we need to translate the admin dashboard with Armenian language. We added the translations files in /resources/lang/vendor/voyager/am.Some features we can translate but we can not change the logged in user locale from Admin dashboard.All locales read from lang folder of vendor voyager( getLocales() function )
Proposed solution
Can we add new translation languages files and commit it.
Alternatives considered
Can we change the getLocales function of Voyager.php file and use it.
Additional context
Are you tried to add Armenian translation language in voyager.

Why don't you submit your translations as a PR instead?
That way you will also be able to select AM

Do you approve it then?

Of course

Ok.
I will translate all files in Armenian and after finishing it will create new branch(I cloned the voyager in my local side already), commit it and create Pull Request.
Thanks

Hello.
I finished the translation of Armenian language but can not push my codes with seperate branch in your repository.I need accesses.How can I add my changes then?
Thanks

In your own personal fork. From there you can open a pull request

Closing as #4028 was merged.


Flash mode check fix

Flash mode check function had only two states. It has three now ( SBL , EDL and NORMAL)

Abandoning this change.


bug: the component on_complete_notify behaves inconsistenly

Neovim version (nvim -v)
0.10.0
Operating system/version
arch linux
Describe the bug
What works
assuming the next config
component_aliases = {
  default = {
    "on_exit_set_status",
    "on_complete_notify",
    "open_output"
  },
},

The tasklist will is updated every time a notification is sent to the user (which is, every time a task start).
What doesn't
If we remove "on_complete_notify", the tasklist will update every n ticks (time based, unless I'm wrong).
What is the severity of this bug?
minor (annoyance)
Steps To Reproduce

On unix and having compiler.nvim installed run: :luafile ~/.local/share/nvim/lazy/compiler.nvim/tests/tests.lua

the tasklist will take a long time to update (10-20 seconds) unless the component on_complete_notify is present. I assume the time ticks are actually shorter (1s?), but because the tests use vim.wait() that is interfering. Which may also happen to users.
Expected Behavior
The tasklist should be updated every time a task starts (no matter if the component on_complete_notify is present). The only difference must be we don't send notifications, other behaviors shouldn't change.
Minimal example file
No response
Minimal init.lua
No response
Additional context
No response

The UI is not updating because you're blocking the main thread with vim.wait. If you want the UI to update, you'll have to use callbacks with vim.defer_fn, or an async library like nvim-nio or plenary, or you could do the same thing yourself with raw coroutines, like so:
local language = require("compiler.languages.cpp")
local example = vim.fn.stdpath("data")
  .. "/lazy/compiler.nvim/tests/code samples/languages/cpp/"

coroutine.resume(coroutine.create(function()
  local co = coroutine.running()
  local function sleep()
    vim.defer_fn(function() coroutine.resume(co) end, 1000)
    coroutine.yield()
  end

  -- Build and run
  vim.api.nvim_set_current_dir(example .. "build-and-run/")
  language.action("option1")
  sleep()

  -- Build
  vim.api.nvim_set_current_dir(example .. "build/")
  language.action("option2")
  sleep()

  -- Run
  language.action("option3")
  sleep()

  -- Build solution (without .solution file)
  vim.api.nvim_set_current_dir(example .. "solution-nofile/")
  language.action("option4")
  sleep()

  -- Build solution
  vim.api.nvim_set_current_dir(example .. "solution/")
  language.action("option4")
  sleep()
end))

I can't even start to tell you how grateful I am for that snippet @stevearc. Thank you, that fixes it indeed.


do you have plan support  godot 4.0?

do you have plan support  godot 4.0?

I planned to bring the plugin to Godot 4, but I wanted to wait until Godot 4 C# projects can be exported to iOS, because I mainly use C# in Godot.
Additionally the official iOS Godot Plugins are currently not supported in Godot 4, because the maintainer is no longer active. So I am not sure the plugin can be compiled for Godot 4 (I‘m using the official iOS plugins as a template).
I might have a look at compiling the plugin for Godot 4 late October when I have some time to spare, but no guarantees.

hello，I have build your plugin for godot 4, it is work. the official iOS Godot Plugins are currently supported in Godot 4 with gdscript in master line,(you need yourself build it)

Close issue

I don't know how you did that.
I tired to build for Godot 4 but I need the Godot 4 extracted headers. I tried to compile Godot 4.2 to generate the headers, but the compilation had an error.
I used what was there and tried to compile the Plugin with the Godot 4.2 (incomplete?) files/headers and only got error like "error: no member named 'REAL' in 'Variant' [2]
ADD_SIGNAL(MethodInfo("log", PropertyInfo(Variant::STRING, "message"), PropertyInfo(Variant::REAL, "number")));"
So... I don't know if it is possible to compile the plugin for Godot 4, at least with my limited level of knowledge and patience...


While Scaning the qr the session string not returning its show no multi device features in whatsapp-web.js its shows like that

Yes, leave from beta.

how ?

Leave From Beta Is Dumb Idea ..
Let Me Check It And Create A PR ..

See https://github.com/tuhinpal/WhatsBot/pull/63


How can I modify the system time of create3?

How can I modify the system time of create3?

The Create 3 gets its system time using NTP when it first connects to a network on boot. You might try rebooting the robot if it's not rebooted since its first network connection.


Badges don't disappear when clicking on the chat

Hi,
the badges in the Mac OS X dock (didn't try windows) don't disappear when i click on the chat which has new messages, only when i leave this chat (switch to another room)

Same goes for windows as my colleague has tested it just now.

It's a side effect of solution for #2. As you said, badges disappear by switching to another channel.
I think v1.0.1 behavior is better than '<=v1.0.0'. Because it has been important that "Are there new messages while I 'm away from the PC?" in my daily work.
If you have any idea for this, please let me know. Of course PR is welcome.

Same issue here. This 'fake unread message' makes my teammates throw out lots of complaints. Hope it will be rolled back in next version.

The fix is WIP at #22. For now, I think you can overwrite src/browser/webview/mattermost.js of v1.0.2 with v1.0.0 version (I can't test it now).

v1.0.3 clears badges for active channel when the tab(or window) is focused.

@yuya-oc this fix works great on OS X.


delayed scaling: stop syncing weight amax values across ranks

Stack from ghstack (oldest at bottom):

#278
-> #277
#276

Summary:
FSDP already ensures that each rank receives the same weight, so the
amaxes of weights are the same on each rank.
I checked performance before/after on the multi GPU benchmark and
didn't see a significant impact on the toy model, but less comms value is better.
Test Plan:
./test_everything.sh passes
Reviewers:
Subscribers:
Tasks:
Tags:
Differential Revision: D58396925

@vkuzo has imported this pull request.  If you are a Meta employee, you can view this diff on Phabricator.


THRIFT-5537: Remove python2

Remove python2

[x] Did you create an Apache Jira ticket?  (not required for trivial changes)
[x] If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
[x] Did you squash your changes to a single commit?  (not required, but preferred)
[x] Did you do your best to avoid breaking changes?  If one was needed, did you label the Jira ticket with "Breaking-Change"?
[x] If your change does not involve any code, include [skip ci] anywhere in the commit message to free up build resources.

I think you still need to sort out some CI issues, but please ping me when you are ready and I'll be happy to review it. thank you for doing it!

Thanks for doing this!
Those seem to be new errors related to python:

https://app.travis-ci.com/github/apache/thrift/jobs/582437473
https://app.travis-ci.com/github/apache/thrift/jobs/582437474
https://app.travis-ci.com/github/apache/thrift/jobs/582437475
https://app.travis-ci.com/github/apache/thrift/jobs/582437476

@fishy turns out cmake was picking up python2 instead of python3 for these test runs. however i cannot get cmake version > 3.10 because by default it's the one shipped in bionic. i'm trying to upgrade to focal in #2528 but before that i guess this PR is a bit stuck unless we want to drop python tests in cmake for the moment


Remove deprication warrnings for iOS 8

iOS 8 depricated interfaceOrientation property of UIViewController.
CTAssetsViewController was using interfaceOrientation and documentation
suggests using [[UIApplication sharedApplication] statusBarOrientation]
instead, so I've replaced it.
Definitions on NSCalendarUnit enum were renamed, and old names were
depricated. I've replaced old names with new.

@indiche Thank you.


Algorithms references

@rushter , thanks for your library!
Can you please add references (articles, etc.) describes which (exactly) algorithms does heamy realize?
Thanks.

You can use this article as a reference:
http://mlwave.com/kaggle-ensembling-guide/
I'm not aware of good papers describing ensemble learning.


How can I apply a different layout (CSS) to this package?

I love this package and have developed at least four systems with it. However, they end up looking very similar, and some clients who use more than one of these systems have complained about this. Does anyone know of an external package that changes the CSS of all components, pages, etc., providing a different look?

I know I can write the CSS as needed, but I was wondering if anyone knows of any external packages that already provide a distinct look and feel out of the box.

Hello @aeusteixeira , I can't help you here, this package is based on the AdminLTE v3 template, and most of the Admin templates looks similar, there's always a sidebar, navbar, footer, etc.
However, you are able to customize some colors of the template among other things, play with the right sidebar on the demo site I linked previously. Most of those options are supported on this package using the configuration.
Read the Wiki pages to know about the available layout options, specially the section Layout and Styling Config.


Add remaining components to 1.3.1 manifest

Signed-off-by: Sayali Gaikawad<EMAIL_ADDRESS>Description
Add remaining components to 1.3.1 manifest. Still waiting for few PRs to get merged.
Issues Resolved
Part of https://github.com/opensearch-project/opensearch-build/issues/1805
Check List

[x] Commits are signed per the DCO using --signoff

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

Codecov Report

Merging #1840 (53d8e10) into main (440c3c0) will increase coverage by 5.46%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #1840      +/-   ##
=============================================
+ Coverage     94.53%   100.00%   +5.46%
=============================================
  Files           176         6     -170
  Lines          3622       105    -3517
  Branches         27        19       -8
=============================================
- Hits           3424       105    -3319
+ Misses          194         0     -194
+ Partials          4         0       -4

Impacted Files
Coverage Δ

src/jenkins/Messages.groovy

src/ci_workflow/ci_target.py

src/test_workflow/integ_test/service_opensearch.py

src/run_assemble.py

...PromoteArtifacts_OpenSearch_Dashboards_Jenkinsfile

src/test_workflow/integ_test/integ_test_runner.py

src/paths/assemble_output_dir.py

...RunBwcTestScript_OpenSearch_Dashboards_Jenkinsfile

...workflow/integ_test/integ_test_start_properties.py

.../jenkins/jobs/PromoteArtifacts_actions_Jenkinsfile

... and 160 more

Continue to review full report at Codecov.

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


fix: updates solution to use AMAZON_COGNITO_USER_POOLS to authenticate to AppSync

This PR updates the UI to use AMAZON_COGNITO_USER_POOLS authentication to access the AppSync GraphQL endpoint. Previously the API_KEY method was being used. When built and minified, this could result in the API_KEY being exposed in the frontend web app source.
To prevent this issue from occurring, we have changed the AppSync auth method to use COGNITO_USER_POOL authentication as detailed here
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

LGTM


upgrade: Display an error when a false parameter gets passed

there is no response from the server so we need to issue a handcrafted
error message

 crowbatctl repocheck  does not fail


850 modflow model instance

@pkdash could you please review this? Ideally we could get this pulled and on www by the CUAHSI bienniel.

I will continue with review after test failures are fixed.

@pkdash all tests should pass now

@jsadler2 The web page hs_core/templates/pages/metadata_terms.html needs to be updated for this new resource type.

@jsadler2 One unit test is still failing. There are also pylint related check failures.

@pkdash @jsadler2 I updated the terms page to include the information about the MODFLOW Model Instance resource.

@pkdash all tests now pass. The problem was with the test for the api which we had not been running locally.

@pkdash now it looks like the problem is with pylint. The default line length that pycharm says is too long is 120 characters but it looks like pylint is checking for 100 characters. Do you know why they are different and which one we should follow?

@jsadler2 pylint has been set to 100 characters. You can change the pycharm settings to 100 characters.

@pkdash it's ready (finally :))

@zhiyuli Can you review the front-end changes in this PR?

@jsadler2 Is this ready for another round of review?

@pkdash  not quite. should be in the next couple hours

@jsadler2 Sounds good.

@jsadler2 I see the pylint issue with 'receivers' import in models.py. That one is bit tricky to resolve. Let me know if you need help on that.

@pkdash yeah I was wondering about that because when I remove it, it stops working on the interface. How do you fix it?

@jsadler2 Refer to hs_collection_resource app. First add a new file apps.py and copy the code from the apps.py in hs_collection_resource. Change the code as necessary. Then copy the code from the  init.py of hs_collection_resource to your app's same file. Then change the code appropriately. Then you can remove the receivers import.

@pkdash okay thanks. I did that. It is now ready for another round.

@jsadler2 A unit test seems to be failing and there are still some pylint issues to be resolved.

@pkdash I am a little at a loss here. I changed the 'import receivers' problem to the way you suggested and that is the pylint problem. I don't know what to do about that. As far as the unit tests go, when I run all the unit tests in my environment, they all work, so again I don't know what the problem is or how to resolve it.

@pkdash now there is only one problem: the pylint one concerning the "import receivers" statement

@jsadler2 add the 'apps.py' file to the 'setup.cfg' file to prevent pylint checking that file.

@pkdash It is now ready.

@jsadler2 I have finished 2nd round of review with minor comments.

@jsadler2 I am reviewing your frontend code. It looks good so far. I would like to have a modflow sample file to test with. Could you send one to me? thanks

@zhiyuli there are sample files in the hs_modflow_modelinstance/test directory

+1 goes to front-end once unused code gets removed

+1
@jsadler2 you may need to get another +1 from @pkdash

+1

@pkdash, @zhiyuli, thanks for your help with the review. Now that you have +1'd it, can I just merge it or do Hong or Michael do that?

@jsadler2 You can merge it.


front end release January

I've added two imgs at the admin-ui changes, otherwise it is 🆗

Thank you 🙇🏻♂


Non accessible paths by url

Hi there,
Drupal's tmp and private folders must be placed before the drupal root (web) once the site has been deployed with Ansistrano.
They shouldn't be accessible by url.
Thanks in advance ;-)

Thank you for your suggestion!
Done!


pathPrefix not working

I have some backend services (Java Spring boot). We using one endpoint for access to api
For example -
https://test-api.example.com/service1
https://test-api.example.com/service2
https://test-api.example.com/service**

We have HTTPRoute:
apiVersion: gateway.networking.k8s.io/v1beta1
kind: HTTPRoute
metadata:
  name: api-prod-other-1
  namespace: master
spec:
  parentRefs:
  - name: k8s-gateway
    namespace: gateways-list
  hostnames:
    - "test-api.example.com"
  rules:
    - matches:
      - path:
          value: /agent-polis
      backendRefs:
      - name: agent-polis-master
        port: 80
    - matches:
      - path:
          type: PathPrefix
          value: /telegram-bot
      backendRefs:
      - name: alert-bot-master
        port: 80

Each service has health check (for java Spring Boot - /actuator/health)
When we trying running endpoint https://test-api.example.com/telegram-bot/actuator/health getting HTTP/2.0 404 Not Found
Logs:
[2023-06-22T09:15:54.775Z] "GET /telegram-bot/actuator/health/ HTTP/2" 404 - via_upstream - "-" 0 118 5 4 "<IP_ADDRESS>" "xh/0.17.0" "0b6076b5-cbea-9370-a3c5-398b69129648" "test-api.example.com" "<IP_ADDRESS>:8080" outbound|80||alert-bot-master.master.svc.cluster.local <IP_ADDRESS>:58382 <IP_ADDRESS>:443 <IP_ADDRESS>:49643 test-api.example.com master.api-prod-other-1.1
Because java service provides only /actuator/health
I try added:
filters:
        - type: RequestRedirect
          requestRedirect:
            path:
              type: ReplacePrefixMatch
              replacePrefixMatch: /

not working for me. Please help!

Fixed. We used URLRewrite filter

Glad you were able to figure this out, closing this one.


voting bug fixed

well done sir
On Sat, May 20, 2023 at 2:01 PM Dihan141 @.***> wrote:

Merged #27 https://github.com/51AN/Markooz-IUT/pull/27 into main.
—
Reply to this email directly, view it on GitHub
https://github.com/51AN/Markooz-IUT/pull/27#event-9294004583, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AWYAEKA6U7H5VEL4X7EROB3XHB24LANCNFSM6AAAAAAYISU5P4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


71 - Add Swift Package Manager Support

fixes #71

Current coverage is 97.11% (diff: 100%)

Merging #105 into master will not change coverage

@@             master       #105   diff @@
==========================================
  Files            27         27
  Lines          1039       1039
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1009       1009
  Misses           30         30
  Partials          0          0

Powered by Codecov. Last update 53b4f17...d871024


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 1634 ms

Resolved: Bofrost.de performance has improved in b66ab2c.


Board Review: Python Event Hubs Preview 1

The Basics

Service team responsible for the client library:  Azure Messaging
Link to documentation describing the service: Product Documentation
Contacts:

Javier Fernandez (Service PM),
SeongJoon Kwak (Service Dev Lead)
Ramya Rao (ADP team)

Languages for this review: Python

Review Resources
To be added
Champion Scenarios

send multiple events to no particular partition
send multiple events to the same partition identified by its id
send multiple events to the same partition, but don't care which partition
find metadata of the event hub instance
get ids of all partitions in the event hub instance
receive events from a particular partition in a consumer group
receive events from a particular partition in a consumer group starting from given position

Agenda for the review

Revisit some of the board suggestions that were made in prior meetings for the same service but other languages for clarification.
Review client creations via constructors and static helpers in Python
Review champion scenarios in Python

We have had reviews for .Net, Java and Javascript before this point. Below are some board suggestions that were made then that we want to re-open for discussion

Renaming of GetEventHubRuntimeInformation to GetProperties on the client

client.GetProperties would imply one is getting properties of the client. In this case, the information we are getting are the properties of the event hub instance not the client.
We want to propose using the name getHubProperties which would make it clear that the properties are of the Event Hub instance that this client represents
For more context, the info returned are the date the event hub instance was created on, number of partitions and the ids of each partition.

Renaming the option that specifies the position in the stream from where to receive events from EventPosition to BeginReceivingAt

This is the only option that sounds like it is telling (literally) the receiver what to do. Other options dont. Example: ConsumerGroup is not named GroupToReceiveFrom. ExclusiveReceiverPriority is not named SetExclusiveReceiverPriorityTo.
I (personal opinion here from Ramya) would suggest to either not change the name here and keep EventPosition or rename to just Position.

createSender() doesnt take partitionId, the send() methods on the sender takes partitionId

We recently discovered that the only way we can send to a partition specified by the id is by creating a dedicated AMQP sender link for the partition. Therefore, this design has been updated such that createSender() takes optional partitionId and the send() methods don't
- If no paritionId is passed, service decides which partition to use
- No questions here. Just FYI as this was a deviation from what was discussed in prior meetings

Name of the sender and receiver class

Board has suggested EventSender and PartitionReceiver. The reason being that the client allows sending without specifying partition, but only allows receiving from given partition
But, this difference is clear from the arguments that createSender() and createReceiver() take. We want to suggest using simpler Sender and Receiver to avoid confusion when
- one looks at list of classes in API references documentation
- or looks at EventSender and wonder if they need to work with another class to send to a partition

Board meeting notes:

Unify the constructor parameters across languages
Remove partition_count from the result of get_properties()
Remove all the factory methods except for from_connection_string.
Hide auto_reconnect and keep_alive_interval from kwargs
In receiver, consumer_group should be optional and default to $default
Consider removing open() in the receiver and just manage the connection in .receive().  If you opt to keep .open(), then also add it to other languages.
Implement __repr() and __str() on the events that are received
Support custom JSON decoders
Remove the receiver.subscribe() for preview - discuss with @johanste about design before inclusion
Consider hiding queue_messages() and send_pending_messages() - they are covered by .send() already

API approved for preview, pending these changes.

Can we re-open this to get a resolution on the open items listed in the issue originally? Mainly on the objections to the below renames suggested by the board. For details, please see the original issue descriptions above

EventPosition -> BeginReceivingAt
Sender -> EventSender
Receiver -> PartitionReceiver

cc @bterlson, @KrzysztofCwalina , @JonathanGiles , @johanste

I think we are closed on all issues now.  If not, please open new issues with the outstanding issues.


shared workflows

Current dependencies on/for this PR:

main

PR #20   👈

This comment was auto-generated by Graphite.


Bazel problem when using Tensorflow as an external dependency.

When using the HEAD version of Tensorflow as an external Bazel dependency (like tensorflow_serving does), we run into an issue in the line:
load("@//third_party:common.bzl", "template_rule")
inside
tensorflow/third_party/jpeg/jpeg.BUILD
That line assumes that Tensorflow is the main repository. But when Tensorflow is included in another project the main repository is the main project so that common.bzl file is not found. I think that line needs to read:
load("@org_tensorflow//third_party:common.bzl", "template_rule")

So a straightforward fix fails with
INFO: Loading package: @png_archive//
ERROR: /workspace/tensorflow/core/platform/default/build_config/BUILD:108:1: error loading package '@jpeg//': Encountered error while reading extension file 'third_party/common.bzl': no such package '@org_tensorflow//third_party': error loading package 'external': The repository named 'org_tensorflow' could not be resolved and referenced by '//tensorflow/core/platform/default/build_config:jpeg'.

It's not possible to use TensorFlow as an external dependency yet due to https://github.com/bazelbuild/bazel/issues/1248

Well, tensorflow_serving certainly uses it right now, but synced to a slightly older version that didn't have this issue. We also do it for an internal project at Snapchat.
I think there should be workaround in Tensorflow while Bazel doesn't fix that issue. The problem was introduced recently in Tensorflow.

Do you know what change caused the issue?

Your problem most likely originated in https://github.com/tensorflow/tensorflow/pull/6462. I wasn't aware that other Bazel projects were referencing TensorFlow as an external dependency. So it seems fixing this should be much higher on our list of priorities.
As a workaround, what we can do is add an omit_jpeg parameter to tf_workspace similar to what Closure Rules does. Then you could copy jpeg.BUILD into your codebase, replace @// with @org_tensorflow// and then declare the new_http_archive for jpeg repository yourself. That's the best workaround that I can think of at the moment that doesn't involve patching Bazel.
If you're willing to use a patched Bazel, I can take a peak into the codebase and see if I can fix the issue referenced above in the next few days.

This is a bug that's actually been bugging me for the better part of a year. So I'd really love to see it solved.

We can also do what @gunan is proposing (https://github.com/tensorflow/tensorflow/pull/6710#issuecomment-271129858) and rollback the change.

Can you please rollback the breaking change until Bazel is fixed? Today the nasm library URL broke and there is no commit that both has all the working urls and a working BUILD rule for jpeg.

@jart let's roll this back so we stop breaking tensorflow/serving in particular. @damienmg told me the fix for this should be in bazel 0.4.4, so hopefully we can re-enable this soon.

https://github.com/tensorflow/tensorflow/pull/6958
This PR pushed out our workarounds we implemented internally with @damienmg's help.
@kirilg confirmed that the workarounds fixed servo's problems.
@raraujosc could you try syncing to head and retry?

I've verified that using the HEAD version of tensorflow and passing extra arguments to tf_workspace now allows me to add tensorflow as an external dependency. Thanks!

Then i think we can close this issue for now.

Are we sure this is fixed? As far as I can tell, running ./configure is still mandatory, so I'm not sure how TensorFlow could be used as an external dependency.

What this issue refers to should be fixed.
But you are right, I am not sure how the options set by configure are handled.
@kirilg @raraujosc @nealwu how do you handle configure script when depending on tensorflow as an external dependency?
Do you simply default to GPU? Or create your own configure script?

Yes, this issue is fixed but is not in 1.0 release. Can you please cherrypick it into 1.0? Thanks.

TensorFlow serving requires you to run ./configure from within the TensorFlow directory. It would be great if we could somehow depend on TF as a workspace target and configure it with some defaults, but I agree it's a different issue and this one is resolved (though may need to be cherrypicked into 1.0 as Robson mentioned).

FWIW a bit update on Bazel side:
We released 0.4.4 that should remove the need for the @%s// trick (to be verified).
The change for removing need from ./configure as a separate step are still in-flight: see https://cr.bazel.build/8218 and its parent, should be in next Bazel release.
Once those change are in the Bazel release, we can update TF configure script to be only an user interface that set the good environment variable and just call bazel fetch //... at the end.

I believe this is fixed now. @girving, you have a project depending on TF using workspace... does everything work now?

Yes, everything should work now (the repo is https://github.com/tensorflow/deepmath).  I haven't synced in the past little while, so it's possible other bugs have surfaced since then (there are no continuous tests for this stuff).

Thanks. I'll close this issue then. An integration test would indeed be nice.

For posterity, this was the commit that fixed my issue: https://github.com/tensorflow/tensorflow/commit/c699e5fa03b1f01913636de620aae5e550e58d40.

Is there a bug to track the being able to include tensorflow in your project without having to run ./configure?

I don't think we have an issue for that, but we're working on it. You can
see that we've moved some code in configure to just modifying --action_env
already. Once that's all it does, you don't have to run it any more. Still
WIP.

Any update on this?

At the moment, you can depend on TF the same way tensorflow/serving github project is depending.
We are working on making things better, but it is taking time.

Thanks. Regarding this I have a few questions:

If you have a new_http_archive which can be Tensorflow git .zip, rather than using a local_archive how do you run ./configure ? Should I maybe write a gen_rule?

What is io_bazel_rules_closure within the top WORKSPACE file in serving?

I also tried pre-compiling separately tensorflow_cc and creating a include and lib folder, but I some issues including Eigen, which has a BUILD file which calls some other Tensorflow stuff...

I haven't tried, but I think the easiest way to use tensorflow without ./configure right now would be:
1 - fork it
2 - run ./configure
3 - add the 3 generated files to your repo
4 - depend on the forked repo

@martinwicke Didn't you say configure wasn't necessary, or was that rerunning configure?

@gunan, it shouldn't be, is it still?

Unfortunately it is, still.
@case540 has volunteered and working towards making configure script completely optional.
He can report on his progress.


fix exportshow

fix exportshow
Signed-off-by: Huaqiao Zhang<EMAIL_ADDRESS>

@badboy-huaqiao you can choose "rebase and merge" option which will not produce “Merge remote xxx” commit messages in commit flow.


Changed the color of text inside of chip tag

Signed-off-by: Savio629<EMAIL_ADDRESS>Description
This PR fixes #1011
Notes for Reviewers
Changed the color of text inside of chip tag to black for better visibility.
Signed commits

[x] Yes, I signed my commits.

@Savio629  Can you please replace the black(color for text) with the variables that are defined here link for Black color.

@UtkarshMishra12 There isn't exact black color available in the file. But I have used a similar shade of black present in the file.
Also does it look fine or do I need to make it a little bold.

Regarding the variables, there are lot of places where black and white color are directly used instead of using variables...

@leecalcote using this variable is good or we can use black(text) for color directly? 🤔

@leecalcote using this variable is good or we can use black(text) for color directly? 🤔

Good question. The variable is ideal here. 👍


Search

As a user,
I want a search feature that can search by company name
So that I can find relevant companies easily and have the results sorted alphabetically.
Acceptance Criteria:
Scenario 1: Searching by Company Name with Exact Match
Given I am on the main page
When I enter a valid company name "Сільпо" in the search bar
And I click on the search button
Then the system should display a list of companies with an exact match for "Сільпо"
And the results should be sorted in alphabetical order by company name
And the list should only include companies with an exact match to the search term "Сільпо"
Scenario 2: Searching by Partial Company Name
Given I am on the main page
When I enter a partial company name "Сіль" in the search bar
And I click on the search buttonт
Then the system should display a list of companies that contain the partial term "Сіль" in their name
And the results should be sorted in alphabetical order by company name
And the list should only include companies with names containing the partial term "Сіль"
Scenario 3: Empty Search Input
Given I am on the main page
When I leave the search bar empty
And I click on the search button
Then the system should display an error message indicating that a valid company name must be provided for searching.
OR the system does nothing
Scenario 4: No Search Results
Given I am on the main page
When I enter a valid company name "Samsung" in the search bar
And I click on the search button
Then the system should display a message indicating that there are no companies matching the search term "Samsung."
Scenario 5: Case Insensitive Search
Given I am on the main page
When I enter a company name "Сільпо" in the search bar, with different letter cases (e.g., "сільпо" or "СІЛЬПО")
And I click on the search button
Then the system should treat the search as case-insensitive and display results for "Сільпо" regardless of letter case
And the results should be sorted in alphabetical order by company name
And the list should only include companies with names matching "Сільпо" in any letter case.

Scenario 3: Empty Search Input
Given I am on the main page
When I leave the search bar empty
And I click on the search button
Then the system should display an error message indicating that a valid company name must be provided for searching.
OR the system does nothing
define the behavior
@AnastasiiaLutsykiv please take a look at the user story


[WIP] Autodoc typehints

Description
This tests sphinx autodoc based on Python 3 typehints.
Impact: Auto-generate type annotations in the documentation based on machine-reaadable typehints that we may test with mypy in the future.
Comments
Advantages of using typehints in our codebase:

With your IDE set-up correctly (https://github.com/ambv/flake8-mypy), you will receive warnings when not handling parameters correctly. For example, if a parameter is allowed to be None but you assume it to always be non-None in some statement, your editor will warn you. (And such warnings could be enforced on CI)

Advantages of using typehints for autodoc:

Type annotations in our documentation are taken from machine-readable and (potentially) CI tested source-code annotations according to PEP484.

Disadvantages of using typehints for autodoc:

While the semantics of type annotations are now well-defined (and not just arbitrary free-text strings), not everyone may be familiar with PEP484 typehints and some people may be confused.

I suggest to gradually introduce typehints in our codebase when refactoring or adding new code. We may or may not want to enable the use for autodoc. Please review the example output at http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-830/4/api/modules/vocab.html#gluonnlp.Vocab and compare to our current doc http://gluon-nlp.mxnet.io/api/modules/vocab.html#gluonnlp.Vocab and comment below.

Found link check problems in job PR-830/4:
(line   19) broken    https://www.salesforce.com/products/einstein/ai-research/the-wikitext-dependency-language-modeling-dataset/ - 404 Client Error: Not Found for url: https://www.salesforce.com/products/einstein/ai-research/the-wikitext-dependency-language-modeling-dataset/
(line    5) broken    https://www.salesforce.com/products/einstein/ai-research/the-wikitext-dependency-language-modeling-dataset/ - 404 Client Error: Not Found for url: https://www.salesforce.com/products/einstein/ai-research/the-wikitext-dependency-language-modeling-dataset/
(line    5) broken    https://www.salesforce.com/products/einstein/ai-research/the-wikitext-dependency-language-modeling-dataset/ - 404 Client Error: Not Found for url: https://www.salesforce.com/products/einstein/ai-research/the-wikitext-dependency-language-modeling-dataset/
(line   21) broken    https://www.salesforce.com/products/einstein/ai-research/the-wikitext-dependency-language-modeling-dataset/ - 404 Client Error: Not Found for url: https://www.salesforce.com/products/einstein/ai-research/the-wikitext-dependency-language-modeling-dataset/
(line  442) broken    https://www.aclweb.org/anthology/P02-1040.pdf)[1 - 404 Client Error: Not Found for url: https://www.aclweb.org/anthology/P02-1040.pdf)%5B1
(line   92) broken    https://nlp.stanford.edu/pubs/glove.pdf)[2 - 404 Client Error: NOT FOUND for url: https://nlp.stanford.edu/pubs/glove.pdf)%5B2
(line  208) broken    https://www.bioinf.jku.at/publications/older/2604.pdf)[3 - 404 Client Error: Not Found for url: https://www.bioinf.jku.at/publications/older/2604.pdf)%5B3

Job PR-830/4 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-830/4/index.html

@dmlc/gluon-nlp-team please take a look at above example and make your opinion known.

I think typing will add some automatic robustness and reduce unit test size. Some thoughts:

Can we make from typing import * the convention and implicitly reserve its subclass names (List, Optional, etc.)? If that's too permissive we can do from typing import Optional, List, .... Both would be preferable to wordy type hints param: typing.Optional[typing.List[int, typing...]]
How do we document tensor-shape constraints not in the type signature? We'd need a convention.
Idea: Could typing / NDArray be extended to support tensor shape checking? e.g., def f(...) ->  NDArray.dim(3)

Found link check problems in job PR-830/5:
(line   19) broken    https://www.salesforce.com/products/einstein/ai-research/the-wikitext-dependency-language-modeling-dataset/ - 404 Client Error: Not Found for url: https://www.salesforce.com/products/einstein/ai-research/the-wikitext-dependency-language-modeling-dataset/
(line    5) broken    https://www.salesforce.com/products/einstein/ai-research/the-wikitext-dependency-language-modeling-dataset/ - 404 Client Error: Not Found for url: https://www.salesforce.com/products/einstein/ai-research/the-wikitext-dependency-language-modeling-dataset/
(line    5) broken    https://www.salesforce.com/products/einstein/ai-research/the-wikitext-dependency-language-modeling-dataset/ - 404 Client Error: Not Found for url: https://www.salesforce.com/products/einstein/ai-research/the-wikitext-dependency-language-modeling-dataset/
(line   21) broken    https://www.salesforce.com/products/einstein/ai-research/the-wikitext-dependency-language-modeling-dataset/ - 404 Client Error: Not Found for url: https://www.salesforce.com/products/einstein/ai-research/the-wikitext-dependency-language-modeling-dataset/
(line  442) broken    https://www.aclweb.org/anthology/P02-1040.pdf)[1 - 404 Client Error: Not Found for url: https://www.aclweb.org/anthology/P02-1040.pdf)%5B1
(line   92) broken    https://nlp.stanford.edu/pubs/glove.pdf)[2 - 404 Client Error: NOT FOUND for url: https://nlp.stanford.edu/pubs/glove.pdf)%5B2
(line  208) broken    https://www.bioinf.jku.at/publications/older/2604.pdf)[3 - 404 Client Error: Not Found for url: https://www.bioinf.jku.at/publications/older/2604.pdf)%5B3

Job PR-830/5 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-830/5/index.html

Job PR-830/6 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-830/6/index.html

Job PR-830/7 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-830/7/index.html


New feature: test-output-verbosity switch

Some author testing may require interaction with the distribution's code on the command line (getting input from the user, for example). This becomes difficult with Test::Output because output is suppressed.
This PR allows the tester to echo output to the screen during the test so the tester can interact with the program by setting a $verbosity flag in the Test::Output module.
The flag is activated with: test-output-verbosity(:on) and deactivated with test-output-verbosity(:off).

What has to happen for this to take effect? More reviewers?

if you also bump the version in the META6.json, we're good to merge  :-)


handleStoreError not always called with next

Hey 👋
We are using express-brute, but we experienced that handleStoreError is not always called with a passed callback.
See https://github.com/AdamPflug/express-brute/blob/master/index.js#L156
So when calling reset and we pass a callback and a handleStoreError is present, then it would be great forwarding the callback. (next: callback)
Thank you!

So for most of the store errors we're setting next to the next middleware in the chain, but in the case of reset that doesn't make sense (because reset can be called outside the context of a request). Are you wanting just to set a separate callback property in that case? That seems reasonable, but I don't think setting next to the callback really makes sense because it's not really continuing the a request (or at least not always).
Pull requests always appreciated :)

Pull requests always appreciated :)

Will create a PR 👍 But before i do, i have one last question.
Why is handleStoreError called for reset at all?Thanks for your help :)

handleStoreError can be called from reset because reset needs to clear the key in the store (which could potentially error out).


Still some 404 errors on ElasticSearch: https://travis-ci.org/UKGovernmentBEIS/beis-opss-psd/jobs/596707749

It's great to see the tests passing reliably now.


Add link to configuration example.

Adding a link to the entitlements-config repo. Adding this to the app
README as I only happened to find this example repo by having a browse
around and it would be very useful for people looking to adopt
entitlements.

Thanks for the PR @lazzurs

Thank you!