amsmath latex math environments

Thanks for the fantastic plugin. The plugin already recognizes inline and display math environments like \[ \]
However there are amsmath extensions, used by all scientific papers, most significant being:
\begin{equation}
\end {equation}
Further documented here: http://www.ams.org/arc/tex/amsmath/amsldoc.pdf. Assuming that treesitter already knows how to parse them. Which seems to be the case as far as syntax highlighting is concerned. Could these also be supported?

You'll have to help me out a bit because I don't use latex.
Give me an example of some latex code with which parts you want highlighting?
This is really just a simple case of tweaking the queries. You can install https://github.com/nvim-treesitter/playground and run :TSPlaygroundToggle to see the parse tree.

I will check out playground. How would I go about adding queries? I am a noob at this.
The basic example is
\begin{equation} \label(somelabel)
e^{\pi} = -1
\end{equation}
the main difference with [ ] is that the equation will have a label. The label itself is optional.

The above snippet is parsed by tree-sitter as:
environment [1, 0] - [3, 14]
  begin: begin [1, 0] - [1, 16]
    name: word [1, 7] - [1, 15]
  ERROR [1, 17] - [1, 23]
  child: mixed_group [1, 23] - [1, 34]
    child: text [1, 24] - [1, 33]
      word [1, 24] - [1, 33]
  child: text [2, 0] - [2, 2]
    word [2, 0] - [2, 2]
  child: brace_group [2, 2] - [2, 7]
    child: generic_command [2, 3] - [2, 6]
      name: generic_command_name [2, 3] - [2, 6]
  child: text [2, 8] - [2, 12]
    word [2, 10] - [2, 12]
  end: end [3, 0] - [3, 14]
    name: word [3, 5] - [3, 13]

Which has an error on \label(somelabel).
So I guess treesitter-latex doesn't support amsmath?

I made a syntax error: It should be curly brackets, sorry. Here is a complete snippet with document class amsart, which may matter also.
\documentclass{amsart}
\begin{document}
\begin{equation} \label{somelabel}
e ^{\pi i}  = -1.
\end{equation}
\end {document}

Ok now we get:
class_include [1, 0] - [1, 22]
  path: path [1, 15] - [1, 21]
environment [2, 0] - [6, 15]
  begin: begin [2, 0] - [2, 16]
    name: word [2, 7] - [2, 15]
  child: environment [3, 0] - [5, 14]
    begin: begin [3, 0] - [3, 16]
      name: word [3, 7] - [3, 15]
    child: label_definition [3, 17] - [3, 34]
      name: word [3, 24] - [3, 33]
    child: text [4, 0] - [4, 3]
      word [4, 0] - [4, 1]
      word [4, 2] - [4, 3]
    child: brace_group [4, 3] - [4, 10]
      child: generic_command [4, 4] - [4, 7]
        name: generic_command_name [4, 4] - [4, 7]
      child: text [4, 8] - [4, 9]
        word [4, 8] - [4, 9]
    child: text [4, 11] - [4, 16]
      word [4, 13] - [4, 16]
    end: end [5, 0] - [5, 14]
      name: word [5, 5] - [5, 13]
  end: end [6, 0] - [6, 15]
    name: word [6, 6] - [6, 14]

Which has enough information but will require an elaborate query.

I haven't noticed when the change occurred, but it is working perfectly now. At least for the basic environments I tested. So thanks a lot!

I haven't changed anything

Maybe treesiter did? Anyway I will reopen when the problem resurfaces.
Yasha
On Wed, Nov 24, 2021, 12:51 Lewis Russell @.***> wrote:

I haven't changed anything
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/lewis6991/spellsitter.nvim/issues/37#issuecomment-978139521,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQGIZUMEOWXWGQY72AXHRDUNUX2FANCNFSM5HMVYTNA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

As always, please provide testcase.


docs(dynamic-plugins): update and complete the docs

Description
Update and complete the dynamic-plugins docs.
Which issue(s) does this PR fix
No issue
PR acceptance criteria
Please make sure that the following steps are complete:

[ ] GitHub Actions are completed and successful
[ ] Unit Tests are updated and passing
[ ] E2E Tests are updated and passing
[ ] Documentation is updated if necessary (requirement for new features)
[ ] Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

/test


BUG FIX: Import command not detecting duplicate residents

Below is the error that you from importing from test.csv which contains duplicate residents.

Codecov Report

Merging #272 (d625250) into master (e29222c) will decrease coverage by 0.04%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #272      +/-   ##
============================================
- Coverage     68.76%   68.71%   -0.05%
  Complexity      892      892
============================================
  Files           139      139
  Lines          2907     2909       +2
  Branches        294      294
============================================
  Hits           1999     1999
- Misses          800      802       +2
  Partials        108      108

Impacted Files
Coverage Δ

...hdb/logic/commands/filecommands/ImportCommand.java
89.47% <0.00%> (-10.53%)
:arrow_down:

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


Error when test HTC model

I have trained HTC model on our own dataset. Train process is all right but I find error when test.
Test command is
python  tools/test.py configs/htc/htc_without_semantic_x101_64x4d_fpn_dconv_c3-c5_mstrain_400_1400_2x_Future3d.py /tools/work_dirs/htc_without_semantic_x101_64x4d_fpn_dconv_c3-c5_mstrain_400_1400_2x_Future3d/latest.pth --format-only --options "jsonfile_prefix=./mask_rcnn_test-dev_results"

Error trackback here.
File "/remote-home/my/Future3d/tools/test.py", line 149, in <module>
    main()
  File "/remote-home/my/Future3d/tools/test.py", line 127, in main
    args.show_score_thr)
  File "/remote-home/my/Future3d/mmdet/apis/test.py", line 48, in single_gpu_test
    result = model(return_loss=False, rescale=True, **data)
  File "/usr/local/miniconda3/envs/dl10/lib/python3.6/site-packages/torch/nn/modules/module.py", line 541, in __call__
    result = self.forward(*input, **kwargs)
  File "/usr/local/miniconda3/envs/dl10/lib/python3.6/site-packages/torch/nn/parallel/data_parallel.py", line 150, in forward
    return self.module(*inputs[0], **kwargs[0])
  File "/usr/local/miniconda3/envs/dl10/lib/python3.6/site-packages/torch/nn/modules/module.py", line 541, in __call__
    result = self.forward(*input, **kwargs)
  File "/remote-home/my/Future3d/mmdet/core/fp16/decorators.py", line 51, in new_func
    return old_func(*args, **kwargs)
  File "/remote-home/my/Future3d/mmdet/models/detectors/base.py", line 150, in forward
    return self.forward_test(img, img_metas, **kwargs)
  File "/remote-home/my/Future3d/mmdet/models/detectors/base.py", line 135, in forward_test
    return self.aug_test(imgs, img_metas, **kwargs)
  File "/remote-home/my/Future3d/mmdet/models/detectors/two_stage.py", line 204, in aug_test
    x, proposal_list, img_metas, rescale=rescale)
  File "/remote-home/my/Future3d/mmdet/models/roi_heads/htc_roi_head.py", line 393, in aug_test
    proposal_list = self.aug_test_rpn(img_feats, img_metas,
  File "/usr/local/miniconda3/envs/dl10/lib/python3.6/site-packages/torch/nn/modules/module.py", line 585, in __getattr__
    type(self).__name__, name))
AttributeError: 'HybridTaskCascadeRoIHead' object has no attribute 'aug_test_rpn'

Actually, i can't find 'aug_test_rpn' in the class.

Thanks for your bug report. This is indeed a bug and we will fix it ASAP.


feat(autopilot): use new routing flag

This PR adds the new experimental routing flag that will improve the efficiency of autopilot and improve the overall on-boarding UX as autopilot will now start searching for channels before the headers are synced
Description:
Added the new routing flag
Motivation and Context:
Autopilot will open channels quicker and users will be on-boarded more seamlessly
How Has This Been Tested?
Use the new routing flag with autopilot
Types of changes:
Push a new flag into the neutrino args
Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[x] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage increased (+0.02%) to 16.167% when pulling 376cfa6d9e5150309dfe0cc821972f6a8fff3d0a on feat/assume-chan-valid into e6a9ce0c448cae01f55ac88875f95c4f286706da on master.

@JimmyMow the '--routing.assumechanvalid' switch is only available after building lnd with the experimental build tag, like:
make install tags=experimental

I'm tempted to say that we should leave this out until it is included in an official lnd release. Otherwise, the only way that we can have it included in a release of our own is to bundle zap with one of our own custom lnd builds. I would rather see us include an official signed lnd binary in our official releases.

Can you squash these commits plz @JimmyMow


Test results percentage colors are misleading.

Summary

Test results colors shown in CLI are misleading as only 100% is green.
Steps To Reproduce
Run tests for classes with different coverage.
Expected result
Colors are assigned gradually.
75% and less is red as it does not meet the required coverage
76%-90% is orange as this coverage is acceptable but could be improved
91%-100% is green as it is hard to get full 100% coverage in most cases.
Actual result

Command: sfdx project deploy start --ignore-conflicts --verbose --test-level RunLocalTests --source-dir . --target-org sandbox88
Currently, everything below 100% is red.

System Information

PowerShell 7.3.6
sf version --verbose --json
{
  "cliVersion"<EMAIL_ADDRESS>  "architecture": "win32-x64",
  "nodeVersion": "node-v18.15.0",
  "osVersion": "Windows_NT 10.0.22621",
  "shell": "cmd.exe",
  "rootPath": "C:\\Users\\u1\\AppData\\Local\\sf\\client\\2.4.10-703551c",
  "pluginVersions": [
    "@oclif/plugin-autocomplete 2.3.6 (core)",
    "@oclif/plugin-commands 2.2.21 (core)",
    "@oclif/plugin-help 5.2.16 (core)",
    "@oclif/plugin-not-found 2.3.36 (core)",
    "@oclif/plugin-plugins 3.2.0 (core)",
    "@oclif/plugin-search 0.0.22 (core)",
    "@oclif/plugin-update 3.1.30 (core)",
    "@oclif/plugin-version 1.3.8 (core)",
    "@oclif/plugin-warn-if-update-available 2.0.46 (core)",
    "@oclif/plugin-which 2.2.29 (core)",
    "@salesforce/cli 2.4.10 (core)",
    "apex 2.3.10 (core)",
    "auth 2.8.12 (core)",
    "data 2.5.5 (core)",
    "deploy-retrieve 1.17.2 (core)",
    "info 2.6.38 (core)",
    "limits 2.3.29 (core)",
    "login 1.2.25 (core)",
    "org 2.9.30 (core)",
    "schema 2.3.22 (core)",
    "settings 1.4.24 (core)",
    "sobject 0.2.2 (core)",
    "source 2.10.30 (core)",
    "telemetry 2.2.5 (core)",
    "templates 55.5.9 (core)",
    "trust 2.5.4 (core)",
    "user 2.3.28 (core)"
  ]
}

Additional information

fixed
https://github.com/salesforcecli/plugin-source/pull/934/files

@AllanOricil Thank you for such a quick fix.
I just have second thoughts. After reading this article https://help.salesforce.com/s/articleView?id=000385650&type=1 my understanding is that 75% is an acceptable value so it should be then in the yellow range.
Then <= 74 should be red.

@PawelWozniak fixed

by the way, I'ill keep doing this until Salesforce cant ignore me anymore and send me an offer so that I can refuse and post on linkedin

:/

🧓

👴🏻

@AllanOricil Thanks for the PR, but we're not doing enhancements to deprecated commands in plugin-source.
This issue references project deploy start from https://github.com/salesforcecli/plugin-deploy-retrieve.  You might want to look in https://github.com/salesforcecli/plugin-deploy-retrieve/blob/26769ca9d2c8fe13dbec98df496220813598f6e7/src/utils/coverage.ts#L97

Sorry for that 😅


Better intg test exit codes

this way buildkite has more meaningful exit codes than simply just "1" for everything.  mostly in accordance to Matthew's suggestions

Another exit code possibly worth tracking (from CI on this PR):
Some pods either were not assigned to nodes or did not deploy properly

Should be as simple as deciding on an error code and adding it here.


Merge dev to main

Description
Merge dev to main
Details
Changes include:

ruff
mypy >1 version
regex v2022

Looks like the Create Release job is failing because the git tag is not the same of the project version. Using code from release.yml, the git tag is,
$ git describe HEAD --tags --abbrev=0
v0.3.5

Whereas the poetry project version is,
$ poetry version --short
0.3.7

If we upgrade the tag to v0.3.7 am I correct in thinking this should sort the pipeline?
I see that there is a for version v0.3.7-alpha here https://github.com/stfc/pre-commit-hooks/tags so I am surprised that I didn't get this version from git describe HEAD --tags --abbrev=0?

versions of python and poetry - agree which we should use

In the CI/CD we have switched 1.8.2 which is much more recent version of poetry. I think the only thing that could cause backwards compatibility issues is (in the pyproject.toml) switching from the old group syntax,
[tool.poetry.dev-dependencies]

to the new group syntax
[tool.poetry.group.dev.dependencies]

According to the documentation, the new syntax was introduced in 1.2. This is an old enough version that I think we can expect users to upgrade to at least 1.2.
For the python version, I think we should stick with 3.9 as the default for the time being. When we upgrade to 3.10 on repo-templates perhaps we can think about testing both 3.9 and 3.10. But, I think that is a job for another PR.

I've used git-changelog previously here to have good control over the inclusions in the CHANGELOG.md for repo-templates


Thread killed on uncaught exception Sys_Error

Haven't had a chance to investigate properly, but opening and closing the menus repeatedly eventually leads to a crash with the error above.

This actually seems to come from the quick open attempts to open files work but print a Thread killed exception, and eventually the app crashes.
Might relate to #211

Thanks for logging the issue @Akin909 ! Ya, indeed, it looks like what happens is the Unix.kill call will throw that exception if the process is already closed. Just added a fix in rench here: https://github.com/bryphe/rench/pull/25


Ionic 3 - can't find module "rxjs/operators"

Getting the above error when running ionic serve. I have checked node_modules/rjxs/operators and the module is there, I'm also calling specific operators for other parts of the project.
I have deleted node_modules and reinstalled for good measure. Any ideas?

Did you check whether you're using rxjs 5.5.x in your package.json and you're imports are correct (like import { map, pairwise, tap } from 'rxjs/operators';)?

Thanks Jonaz,
I had thought the rjxs Ionic had installed was doing the job but I needed to update it and add it manually.


chore: fix typos

@ccoVeille thanking you 🙏 I'm terrible with typos. I'll allow the American English.

@ccoVeille thanking you 🙏 I'm terrible with typos. I'll allow the American English.

#47 may help you then @savedra1


Cannot change hostname after CT creation

Hi,
After I create a CT, I cannot change it's hostname on the edit page. The change is discarded.

I fix this issues. Now the new hostname is saved in config file but, anyway, the hostname is not updated inside the container after restart. I think this form field is really useless...


arch: handle more boot paths for installs

Fixes https://github.com/anatol/booster/issues/246

Thank you! The change looks reasonable.

The Arch Linux user actually needs to choose whether ESP will be mounted to /efi or /boot, ie. to decide whether initramfs will be installed on the ESP or the /boot directory (root partition)
For example, amd-ucode and intel-ucode are always installed in the /boot directory...
https://gitlab.archlinux.org/archlinux/packaging/packages/linux-firmware/-/blob/main/PKGBUILD?ref_type=heads#L111
https://gitlab.archlinux.org/archlinux/packaging/packages/intel-ucode/-/blob/main/PKGBUILD?ref_type=heads#L28
mkinitcpio also installs initramfs there by default, right?
So why change?

The Arch Linux user actually needs to choose whether ESP will be mounted to /boot or /efi , ie. to decide whether initramfs will be installed on the ESP or the /boot directory (root partition)
For example, amd-ucode and intel-ucode are always installed in the /boot directory... https://gitlab.archlinux.org/archlinux/packaging/packages/linux-firmware/-/blob/main/PKGBUILD?ref_type=heads#L111 https://gitlab.archlinux.org/archlinux/packaging/packages/intel-ucode/-/blob/main/PKGBUILD?ref_type=heads#L28 mkinitcpio also installs initramfs there by default, right?
So why change?

I'm closing this PR in favor of aligning to the distribution's current default paths. Going against the distro's standards just makes life harder and not having /boot doesn't appear to be common.


[FEATURE] Add scroll bar on view console output in jobs panel

Alternatively, ability to launch log text in a regular text editor.


Indexed Entry Table

Is anyway to save each entry index so app not need to iterate all of archive each time need one of entries?
now I do this:
while (reader.MoveToNextEntry() && !reader.Entry.Key.Equals(name, StringComparison.OrdinalIgnoreCase))
{
}
reader.WriteEntryTo(memStream);

I like something like this:
IIndexedEntryTable entryTable = reader.CreateIndexedEntryTable();
entryTable[name].WriteEntryTo(memStream);

Readers are useful for forward only streams.  If you're iterating over the stream multiple times, then it's probably not a forward-only stream.
Use the Archive interface which indexes entries for random access.

Tank you. this lib is really great and helpful.


Add more loaders

I would like to work on this. Can you assign this to me?

I would like to work on this. Can you assign this to me?

You can contribute to this project by adding a new CSS-Loader, to be sure to follow the rules.

@rohit8020 I have finished this task. Here is the screenshot. The last two loaders were added by me. Can you assign it to me so that I can do a pull request?

@rohit8020 Can you see my pull request #77?


Add Aboutus Page , attractive design.

Please assign me this issue under hacktoberfest 2024.

hey @newcodergit assigned go star the repo and follow me.


Build a Product Card

Design a product card using HTML and CSS for an e-commerce website.

Plz assign this issue to me.


Update typescript 2.7

Fix https://github.com/karen-irc/option-t/issues/237

This change is

bors r+


add example variant

The example variant is currently not working. Vervet doesn't have any variant ids in the database which can be used for search. We need to add the location with vf= to the url in order to navigate to the variant summary page. Is there a way of encoding this in the ini file without ending up with a mangled up URL following the link from the Example variant? Thanks.

I think it will need a tweak in the main ensembl-webcode to get rid of the ampersand. Could you submit a JIRA ticket?

Thank you Anne. The ticket is here.


chore: dockerize development environment

Ticket: DI-14
In the sake of harmonisation and to ease development, this PR adds a docker file in which you can run the test or lint the code, while using your normal IDE.
They are a few caveats, the first one is that because we mount the project to the /app folder, the node_modules are shared and slow down the execution by a lot. To avoid that, the node_modules are installed in the parent folder (the root) and an empty volumes is mounted to avoid copying the pacakges from the host.
This works fine but prevent the modification of the packages, and the image must be rebuilt when some are added.
I used Docker Desktop in the readme to install docker because docker machine is deprecated and the code is archived.
To test the Dockerfile, follow the DOCKER_README.MD instructions and then run yarn test:unit for example.

Thanks @Haroenv and @DevinCodes for the review !

What do you think of harmonising docker image in CI and this docker image in some later step?


[krampfkn] Spack exercise

Implemented additional task 2: Adding build options to spack and CMake. To test the build, you would need to change the repo + branch of the "main" version. I included a patch-file for doing that.
I chose a very simple/brute option in the code for adhering to the build options (would be even better to change the available interfaces, so we can detect at build time missing build options), but since that is not the main scope, I hope it is ok I did it in a simpler fashion..


ctx.getParent is not a function

Description:
When I update from 1.0.0-build-66 to 1.0.0-build-9, I get the following error when I click a button in my application:
ctx.getParent is not a function

Unfortunately, I'm not able to reproduce the same problem via https://github.com/ractivejs/ractive/issues/3067#issuecomment-328033011

According to the build dates, 1.0.0-build-66 is newer than 1.0.0-build-9, but npm update ractive updates 1.0.0-build-66 to 1.0.0-build-9. Is there any way to fix that problem?

npm is confused because it's doing string rather than number comparisons for the build versions. I don't know that there's a good way to get automatic updates from npm before a full release.

left-pad... by 3 digits. 😁

We could release 0.10 and start over the count. This time, pad the numbers sufficiently.

Could you publish the current release with 1.0.0-build-900 again and increment by one for each next?


Import issue tensorflow 2.1.0 using Windows 10 AMD Radeon processor with GPU

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): bin
TensorFlow version: 2.1.0
Python version: 3.5.2
Installed using virtualenv? pip? conda?: pip
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: None
GPU model and memory: AMD Radeon 1 GB

Describe the problem
Cannot import tensorflow after installation
Provide the exact sequence of commands / steps that you executed before running into the problem
pip install tensorflow
Any other info / logs
import tensorflow
Traceback (most recent call last):
File "C:\Users\kumar.shivam\AppData\Local\Programs\Python\Python35\lib\site-packages\tensorflow_core\python\pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "C:\Users\kumar.shivam\AppData\Local\Programs\Python\Python35\lib\site-packages\tensorflow_core\python\pywrap_tensorflow_internal.py", line 28, in
_pywrap_tensorflow_internal = swig_import_helper()
File "C:\Users\kumar.shivam\AppData\Local\Programs\Python\Python35\lib\site-packages\tensorflow_core\python\pywrap_tensorflow_internal.py", line 24, in swig_import_helper
_mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
File "C:\Users\kumar.shivam\AppData\Local\Programs\Python\Python35\lib\imp.py", line 242, in load_module
return load_dynamic(name, filename, file)
File "C:\Users\kumar.shivam\AppData\Local\Programs\Python\Python35\lib\imp.py", line 342, in load_dynamic
return _load(spec)
ImportError: DLL load failed: The specified module could not be found.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "", line 1, in
File "C:\Users\kumar.shivam\AppData\Local\Programs\Python\Python35\lib\site-packages\tensorflow_init_.py", line 101, in
from tensorflow_core import *
File "C:\Users\kumar.shivam\AppData\Local\Programs\Python\Python35\lib\site-packages\tensorflow_core_init_.py", line 40, in
from tensorflow.python.tools import module_util as module_util
File "C:\Users\kumar.shivam\AppData\Local\Programs\Python\Python35\lib\site-packages\tensorflow_init.py", line 50, in getattr
module = self.load()
File "C:\Users\kumar.shivam\AppData\Local\Programs\Python\Python35\lib\site-packages\tensorflow_init.py", line 44, in _load
module = importlib.import_module(self.name)
File "C:\Users\kumar.shivam\AppData\Local\Programs\Python\Python35\lib\importlib_init.py", line 126, in import_module
return _bootstrap.gcd_import(name[level:], package, level)
File "C:\Users\kumar.shivam\AppData\Local\Programs\Python\Python35\lib\site-packages\tensorflow_core\python_init.py", line 49, in
from tensorflow.python import pywrap_tensorflow
File "C:\Users\kumar.shivam\AppData\Local\Programs\Python\Python35\lib\site-packages\tensorflow_core\python\pywrap_tensorflow.py", line 74, in
raise ImportError(msg)
ImportError: Traceback (most recent call last):
File "C:\Users\kumar.shivam\AppData\Local\Programs\Python\Python35\lib\site-packages\tensorflow_core\python\pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "C:\Users\kumar.shivam\AppData\Local\Programs\Python\Python35\lib\site-packages\tensorflow_core\python\pywrap_tensorflow_internal.py", line 28, in
_pywrap_tensorflow_internal = swig_import_helper()
File "C:\Users\kumar.shivam\AppData\Local\Programs\Python\Python35\lib\site-packages\tensorflow_core\python\pywrap_tensorflow_internal.py", line 24, in swig_import_helper
_mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
File "C:\Users\kumar.shivam\AppData\Local\Programs\Python\Python35\lib\imp.py", line 242, in load_module
return load_dynamic(name, filename, file)
File "C:\Users\kumar.shivam\AppData\Local\Programs\Python\Python35\lib\imp.py", line 342, in load_dynamic
return _load(spec)
ImportError: DLL load failed: The specified module could not be found.
Failed to load the native TensorFlow runtime.
See https://www.tensorflow.org/install/errors
for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.

@shivamspj,
Could you please check this comment from a similar issue and let us know if it helps. Thanks!

@amahendrakar
Thanks for the prompt reply. I could solve the issue by reinstalling python version 3.5. MSVC reinstallation. And corresponding libraries(tensorflow).


Need to update build.gradle

When using current version of apkx extension it overrides android dependencies as the build.gradle file is empty. It causes the impossibility to bundle android build when using some other extensions (for example push-extension).
I believe that proper dependencies list in build.gradle should be next:
dependencies {
compile 'com.android.support:support-v4:26.+'
}
Could you please make it asap? Or grant me rights to make a pull request please

It's a public project under the Apache 2.0 license. You have the rights to fork it, fix the issue, then submit a pull request back to this repo at your leisure.
As I said on defold/extension-push#23, once I get the time to catch up with maintenance, if you submit a PR, I'll happily test it and merge it. Until then, you can use your fixed fork as a dependency URL for your game.

@ShreiderWalter How to fork a GitHub repository: https://docs.github.com/en/github/getting-started-with-github/fork-a-repo


504 Gateway timeout and cloudfront.net blocking even if my node is ON?

OS: Windows 10 x64
Version of IPFS Desktop: 0.30.1

Describe the bug
Shareable link gives 504 error even if my node is up and running.
To Reproduce
Steps to reproduce the behavior:
Share link.
Expected behavior
The download should start.
Screenshots
https://github.com/ipfs/ipfs-desktop/assets/13344308/0413ce22-2c17-46d4-ae13-fcef94a1d231

PD: I need to open some ports or..?

Alternatively, if your node has the content, and you'd like that to be served right away, did you try using ipfs-companion? it should be able to redirect your request directly to the local node instead of waiting for the node running at ipfs.io retriece the content.

So other ppl must install ipfs-companion addon to download from me?


[r2.16-rocm-enhanced] Include various test script updates

SWDEV-462029
SWDEV-436947

retest gpu-non-pip-multi please

retest cpu-non-pip please

retest gpu-non-pip-multi please


iOS: Green indicator of camera don't close

Describe the bug
When I'm using library I can see with green dot near front camera that is showing that I use Camera that is fine
however it does not turn off when I turn off camera/screen with camera component
To Reproduce
Steps to reproduce the behavior:

Use this library
Try to scan code
Use sample setup camera

       <Camera
          ref={cameraRef}
          style={styles.cameraContainer}
          scanBarcode
          cameraType={CameraType.Back}
          onReadCode={onCodeFound}
          onError={() => setHasPermission(false)}
        />

Tun on screen
Showing green indicator even after screen with camera is closed
Only putting app into background shut down this green idicator

Expected behavior
On other apps like Instagram, when I stop using camera then this green indicator is stop showing
Smartphone (please complete the following information):

Device: iPhone 13 Pro
OS: 17.1.1

this can be problem, since potential users can think that my app is recording them

if you're using https://reactnavigation.org/ for navigation. Just implement disabling camera with useFocusEffect or something like this:
1. const [isActive, setIsActive] = useState(false)
....
2. useEffect(() => {
  const focusUnsubscribe = navigation.addListener('focus', () => {
      setIsActive(true)
  })

  const blurUnsubscribe = navigation.addListener('blur', () => {
      setIsActive(false)
    })
  return () => {
      blurUnsubscribe()
    }
}, [navigation])
.....
3. return (
{isActive ? <Camera  .../>: null}
)

@acherkanov I was trying it yesterday and it was not working, does it works for you?

if you're using https://reactnavigation.org/ for navigation. Just implement disabling camera with useFocusEffect or something like this:
1. const [isActive, setIsActive] = useState(false)
....
2. useEffect(() => {
  const focusUnsubscribe = navigation.addListener('focus', () => {
      setIsActive(true)
  })

  const blurUnsubscribe = navigation.addListener('blur', () => {
      setIsActive(false)
    })
  return () => {
      blurUnsubscribe()
    }
}, [navigation])
.....
3. return (
{isActive ? <Camera  .../>: null}
)

@acherkanov I was trying it yesterday and it was not working, does it works for you?

Correct, that works for me

  const [isActive, setIsActive] = useState(true);
...

  const goBackWithCameraUnmount = () => {
    // handle problem with camera not unmounting
    // cousing problem with green camera indicator
    // that show up during working with app
    // that can potentially cause battery drain and security consents
    // from end users
    setIsActive(false);
    setTimeout(() => {
      navigation.goBack();
    }, 10);
  };

I did wrap my goBack method to this callback, and does not render camera if isActive is set to false


Create Ore Generation Incompatibility

I apologize for the length of this, but I couldn't upload config files directly into here.
It appears that at OreTweaker and Create ore generation are incompatible with one another. I kept Create's ore generation as is and only edited the vanilla generation using OreTweaker. Here are copies of the Create and OreTweaker configs:
Create
[worldgen]

	#
	#Modify Create's impact on your terrain
	[worldgen.v2]
		#
		#Prevents all worldgen added by Create from taking effect
		disableWorldGen = false
		#
		#Forward caught TileEntityExceptions to the log at debug level.
		logTeErrors = false

		[worldgen.v2.copper_ore]
			#
			#Range: > 0
			clusterSize = 18
			#
			#Amount of clusters generated per Chunk.
			#  >1 to spawn multiple.
			#  <1 to make it a chance.
			#  0 to disable.
			#Range: 0.0 ~ 512.0
			frequency = 2.0
			#
			#Range: > 0
			minHeight = 40
			#
			#Range: > 0
			maxHeight = 85

		[worldgen.v2.weathered_limestone]
			#
			#Range: > 0
			clusterSize = 128
			#
			#Amount of clusters generated per Chunk.
			#  >1 to spawn multiple.
			#  <1 to make it a chance.
			#  0 to disable.
			#Range: 0.0 ~ 512.0
			frequency = 0.015625
			#
			#Range: > 0
			minHeight = 10
			#
			#Range: > 0
			maxHeight = 30

		[worldgen.v2.zinc_ore]
			#
			#Range: > 0
			clusterSize = 14
			#
			#Amount of clusters generated per Chunk.
			#  >1 to spawn multiple.
			#  <1 to make it a chance.
			#  0 to disable.
			#Range: 0.0 ~ 512.0
			frequency = 4.0
			#
			#Range: > 0
			minHeight = 15
			#
			#Range: > 0
			maxHeight = 70

		[worldgen.v2.limestone]
			#
			#Range: > 0
			clusterSize = 128
			#
			#Amount of clusters generated per Chunk.
			#  >1 to spawn multiple.
			#  <1 to make it a chance.
			#  0 to disable.
			#Range: 0.0 ~ 512.0
			frequency = 0.015625
			#
			#Range: > 0
			minHeight = 30
			#
			#Range: > 0
			maxHeight = 70

		[worldgen.v2.dolomite]
			#
			#Range: > 0
			clusterSize = 128
			#
			#Amount of clusters generated per Chunk.
			#  >1 to spawn multiple.
			#  <1 to make it a chance.
			#  0 to disable.
			#Range: 0.0 ~ 512.0
			frequency = 0.015625
			#
			#Range: > 0
			minHeight = 20
			#
			#Range: > 0
			maxHeight = 70

		[worldgen.v2.gabbro]
			#
			#Range: > 0
			clusterSize = 128
			#
			#Amount of clusters generated per Chunk.
			#  >1 to spawn multiple.
			#  <1 to make it a chance.
			#  0 to disable.
			#Range: 0.0 ~ 512.0
			frequency = 0.015625
			#
			#Range: > 0
			minHeight = 20
			#
			#Range: > 0
			maxHeight = 70

		[worldgen.v2.scoria]
			#
			#Range: > 0
			clusterSize = 128
			#
			#Amount of clusters generated per Chunk.
			#  >1 to spawn multiple.
			#  <1 to make it a chance.
			#  0 to disable.
			#Range: 0.0 ~ 512.0
			frequency = 0.03125
			#
			#Range: > 0
			minHeight = 0
			#
			#Range: > 0
			maxHeight = 10

OreTweaker
"Enable Debug Output" = false
"Disable Ores" = ["minecraft:coal_ore", "minecraft:iron_ore", "minecraft:gold_ore", "minecraft:diamond_ore", "minecraft:lapis_ore", "minecraft:redstone_ore", "minecraft:emerald_ore"]

[["Custom Ore"]]
	"Ore Name" = "minecraft:lapis_ore"
	"Max Vein Size" = 7
	"Filler Name" = "minecraft:stone"
	"Min Vein Level" = 1
	"Max Vein Level" = 48
	"Spawn Rate" = 8

[["Custom Ore"]]
	"Ore Name" = "minecraft:redstone_ore"
	"Max Vein Size" = 7
	"Filler Name" = "minecraft:stone"
	"Min Vein Level" = 1
	"Max Vein Level" = 48
	"Spawn Rate" = 8

[["Custom Ore"]]
	"Ore Name" = "minecraft:emerald_ore"
	"Max Vein Size" = 1
	"Filler Name" = "minecraft:stone"
	"Min Vein Level" = 1
	"Max Vein Level" = 48
	"Spawn Rate" = 1

With these configs I used World Stripper to uncover the terrain in new chunks and this is what I got:

You may notice that EvilCraft is also in this mod. OreTweaker does not interfere with its ore generation. I did a similar thing in The Nether as I have Netherrocks installed as well (the only other mod that affects ore generation) and it appears to work as intended:

So this seems to be specific to Create. As I have no way of telling whether this is a Create or an OreTweaker problem, I am simply reporting this to both. And for the sake of completion, here is my log file for this session:
latest.log

Note that I do have a workaround I will implement to fix this: Namely disable Create's ore generation entirely and add the "ores" as custom ores in OreTweaker. So, while this isn't a major problem, it is definitely something to look into.

My version of Ore Tweaker does not currently support anything beyond Minecraft 1.12, but some total retard thought it was a good idea to create a mod with the exact same behavior and name so I am getting flodded with issue reports I have nothing to do with. I am releasing my version of Ore Tweaker for 1.16 later this week.
You prob want to contact this guy here: https://github.com/noahc3/oretweaker/issues
I am gonna make sure this works out of the box once I release my 1.16 version ;-)

Alright. In this case I will be removing this guy's mod and replacing it with yours as soon as it is out. In addition, I will be adding to the description of my modpack that there was once an illegitimate version of OreTweaker present and link the site.
Thank you for your time. You can close this issue as this isn't an issue with your mod, specifically.

I want to state that this person has done nothing wrong and his version is not illegitimate. All my mods are open source under the MIT License and everyone can create their own version of my mods. This person has written his mod from scratch but the main problem I have is that he is distributing it under the exact same name as my mod but without a space between the two words.
This causes a lot of confusion for the end users and a ton of people let to believe they are using my mod that haven't been updated since 1.12. The result is me getting a ton of bugs and issues that I cannot fix and needs to be redirected. It is common in the modded Minecraft community to change the name of the mod once you create a continuation / successor of someone product.
Like "Too Many Items" (TMI) became "Not Enough Item" (NEI) and then "Just Enough Item" (JEI).

Hey, I just released Ore Tweaker 2.0.0 on CurseForge.
I have "fixed" the bug you where experiencing but I am still trying to figure it properly out.
There seem to be a issue where the game thinks creates ores all are "minecraft:lapis" for some strange reason.
So when you tell the mod to disable lapis it also disables all Create ores..
I have created a temp workaround to fix this by checking that if create is present it will not tweak lapis. This also results in that you cannot tweak lapis but is a temporary compromise until I can properly figure out what going on. I might have to check in with the creators of Create and check if they inherit some properties from Lapis in their mod.
Link to new 1.16 version of Ore Tweaker :: https://www.curseforge.com/minecraft/mc-mods/ore-tweaker

This is now properly fixed due to the bug uncovered in #11 and #9
Patch can be found here: https://www.curseforge.com/minecraft/mc-mods/ore-tweaker/files/3307868


docs: add basic speed test

close #13

Docs or analysis?

@korikuzma "analysis" is not an official conv. commit (? maybe ?) so I went with docs even though it felt sorta ill-fitting


latest nightly in CI and rust-toolchain

@dlubarov @unzvfu @Nashtare @nbgl Do you mind taking a quick look at the Clippy-suggested changes I made? I think it's all just replacing &mut ... with &... in function signatures and replacing vec![...] with [...], so should be harmless, but just wanted a to make sure since it touches all of your code


CSS Emmet shortcuts for pre-built font families dont work

Some useful Emmet CSS shortcuts as according to its cheatsheet doesn't work, e.g. :-

ff:a -> expands into font-family:auto; instead of font-family: Arial, "Helvetica Neue", Helvetica, sans-serif;

ff:t expands into font-family:t;instead of font-family: "Times New Roman", Times, Baskerville, Georgia, serif;

ff:v expands into font-family: v; instead of font-family: Verdana, Geneva, sans-serif;

The above shortcuts work perfectly in sublime text :)

VS Code uses the newer modular Emmet libraries  where these snippets are not supported.
The snippet library doesnt have entries for font families that can support the above.
@sergeche Your thoughts on this?

These snippets are missing, you can PR them to CSS snippets

I can go ahead and give this a shot. I would only need to add the suggested snippets to CSS snippets file ?

Yes. Note that : isn’t allowed in snippet names anymore, it’s used to explicitly separate property name and property value. Thus, you should either add pipe-separated values to ff snippet so they will be picked by automatic snippet resolved for passed property name (e.g. in ff:a abbreviation a is property value, which can be matched with Arial, "Helvetica Neue", Helvetica, sans-serif value). Or create new snippets without : in name.

Got it, just to make sure I understood correctly for a snippet like ff:a since the : are no longer allowed I can add Arial, "Helvetica Neue", Helvetica, sans-serif as value to the ff property.

Exactly. It’s a good idea to add it as a custom snippet into VS Code and ensure everything works as expected

Published as<EMAIL_ADDRESS>

Thanks @ely-alamillo and @sergeche
I'll include the changes in VS Code in the next week or so.

I'm still seeing ff:a expand to font-family: auto; in c86cab2211d6dce686e5c89934d915b1e020ba9e

Try 'ffa'

@mjbvz, @sergeche  mentioned that : are not allowed in snipped names anymore so ff:a does not work. To get that snippet to work you should use ffa instead.

Yep looks good without the :


v2: allow overriding default remark plugins in content-docs

🚀 Feature
I would like to override the default remark plugins in the content-docs plugin: for example, to have control over the markup generated by remark-admonitions. Perhaps we could have an option to turn the default ones on/off?
Or alternatively: the content-blog plugin allows the options beforeDefaultRemarkPlugins and beforeDefaultRehypePlugins, which if they were supported in docs-content (the docs say they are, but they are not) would allow me to insert my own plugin to process remark before the default one can be run

Make sense, we should have an option like beforeDefaultRemarkPlugins for all plugins supporting remark to make all this uniform (blog + docs + pages)

Can I work on it @slorber ?

Yes thanks 😉
Le sam. 19 sept. 2020 à 11:22, Rémi Doreau<EMAIL_ADDRESS>a
écrit :

Can I work on it @slorber https://github.com/slorber ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/facebook/docusaurus/issues/3437#issuecomment-695195612,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFW6PQTDOXIP2FHZFOVZLLSGSA45ANCNFSM4RHOZLIQ
.


Consider switching the no results message to use v-if

When the component renders it renders the message telling the user there are no results with v-show, this means the markup is always there, this can be confusing for users of screen readers.
Consider using v-if instead, so the markup is only in the DOM when there are really no suggestions found, and also make the message something that can be overridden.
I'm more than happy to do it and submit it as a PR.

Hey! v-if caused some issues back in the days but I think this shouldn’t be a problem anymore. I was experimenting with it just yesterday and haven’t seen any regressions.
Feel free to make a PR! :)

I'm not sure what that really gets you. display:none (which is what v-show toggles, I think) hides content from all users, including screenreader users. If you do need to add something to the DOM, then you'll need to alert screenreader users with an ARIA live region -- that's something you might want to minimize. Regarding other content, if you use aria attributes that refer to other elements (e.g., aria-controls, aria-labelledby, aria-describedby, etc.), you're better off showing and hiding those elements with v-show. Otherwise the code will be invalid and the aria will be useless.


[Android] myLocationButton = false or myLocation = false and the plugin ask for geolocation permission

I'm submitting a ...  (check one with "x")

[ ] question
[x] any problem or bug report

OS:  (check one with "x")

[x] Android
[ ] iOS
[ ] Browser

cordova information:  (run $> cordova plugin list)
It is not neccesary
Current behavior:
Creating a Map with myLocation = false or myLocationButton = false. The app ask for geolocation permission.
Expected behavior:
myLocation = false or myLocationButton = false are the same like if I don't pass these params
Screen capture or video record:
It is not neccesary
Related code, data or error log (please format your code or data):
var options = {
	camera: {
	target: {lat: ..., lng: ...},
	zoom: 19
	},
	controls: {
	myLocation: false
	}
};
var map = plugin.google.maps.Map.getMap(mapDiv, options)
var opts = {
	port :      1947,
	baseDir :   __dirname + '/../../'
};

The problem is in this line:
if (controls.has("myLocationButton") || controls.has("myLocation")) {

The problem is the plugin only ask for if in control has both keys, but it should ask if has and the value is true.

Hi!, I am a mate of @zarelint. We had update our app 2 weeks ago. And in the way we have found several bugs or details.
We had updated our apps (Android and iOS) from version 2.1.1 forked to last version forked: 2.6.2
Thanks a lot for this maravellous project! We can grow up thanks of you.
Links of our apps, if you want test or something (please move to Spain, France or Portugal):

https://play.google.com/store/apps/details?id=com.joinuptaxi.passenger&hl=es
https://apps.apple.com/es/app/joinup-pedir-y-reservar-taxi/id554668455

Google Maps SDK for iOS needs the permission when one of flags is true. This plugin designs all code are synchronized in each platforms as much as possible, in order to maintain the code.
This is not a big. Designed behavior.

@wf9a5m75 I didn't explain good, with something like this:
var options = {
controls: {
myLocation: false,                         <-- False
myLocationButton: false                <-- False.
}
};
var map = plugin.google.maps.Map.getMap(mapDiv, options)
In Android, the plugin ask for geolocation permission.
With something like this, the plugin does not ask for geolocation permission:
var options = {
controls: {
}
};
var map = plugin.google.maps.Map.getMap(mapDiv, options)
@wf9a5m75 Please, consider to reopen this issue...


Dependency browser lacks arrow head to show dependency direction

Dependencies are shown has lines which is not very informative

Same issue as https://github.com/moosetechnology/Midas/issues/52


Stop numpy consuming >>

Looks like this won't be as hard as I first thought.

added array_ufunc to Fn and Partial in c
Can revisit later on to improve performance but for moment it appears to work


[BUG] Navigation inside app is blocked by cmp

Describe the bug
Navigation inside the app is not possible. We have the imprint and privacy links in the cmp (see screenshot "Impressum" and "Datenschutzhinweise") which are deep links inside the app. When I press them, the navigation happens in the background, but the cmp is still in foreground.
I know this is kind of tricky, as you have no user interaction, when closing the cmp, and when should it reshow. Maybe we need external web links here in favour of deeplinks?
Screenshots

usercentrics/react-native branch/commit
v2.0.3

Hello @fmacherey I'm afraid that we cannot do much in this case, because we can only dismiss the CMP if the user has interacted with the consent buttons, can you wrap your new screen in a <Modal>? It might present the new screen over the CMP.

Sadly this does not work for us. So maybe external links are our way to go here.


[api-docs] undocument system user flag

A new commit was made into the api-docs repo. This issue was created automatically. Need to review if we made any change to the library for this commit.
https://github.com/discord/discord-api-docs/commit/9293f0d490ac6acf9d627e429e5a8131b303b528

Done


Add an about page

I want to add an About page to your website. @Trisha-tech

I want to add an About page to your website. @Trisha-tech

Kindly follow the template for creating issue. @LNischala

[Bug]: Login and sign up page is not working well. #433 @Trisha-tech


Bluetooth: GAP/IDLE/NAMP/BV-01-C requires Read by UUID

GAP/IDLE/NAMP/BV-01-C:
Verify the IUT can perform the Name Discovery Procedure and retrieve the device name from a peer device.
In order to pass this test PTS expects Read by UUID to be used.

This seems to be already merged. Right @Vudentz


[mungegithub] retest munger complaint message update for clarity

@brendandburns @lavalamp
https://github.com/kubernetes/kubernetes/pull/19874#issuecomment-178345665
Is the current message.

it isn't clear what the user did wrong
the <number> is messed up because markdown eats the <>
it was showing all test flakes, not just open test flakes.

LGTM


Add golang todo rest crud example

What's this PR do?
Summary of changes in this PR or what it accomplishes.
I would like to share an example of a todo REST API, the original repository is ToDo API Golang.
Description
REST API to create, update and retrieve ToDos, including grateful shutdown, rate limiting, structured logging, unit tests, integration tests, environment variables, health check and API documentation with swagger. Technologies: Golang 1.19, MongoDB (with Docker Compose), Gorilla Mux, Go Swagger, Tollbooth (rate limiting), Zap (logging), Viper, Mockery, Makefile, Pre-commit, and DockerTest (integration tests).
@ulises-jeremias I left the workflow folder because I wasn't sure if I should include it or not. Please let me know if you consider better to remove it because this project already have it own actions defined.
<!--

Please title your PR as follows: `feature: fix foo bar`.
Always start with the thing you are fixing, then describe the fix.
Don't use past tense (e.g. "fixed foo bar").

Explain what your PR does and why.

If you are adding a new function, please document it and add tests:

```
code you added/updated
```

If you are fixing a bug, please add a test that covers it.

Before submitting a PR, please:
  - specify the command to execute or steps to follow to know that the problem was solved

We try to process PRs as soon as possible. They should be handled within 24 hours.

Applying labels to PRs is not needed.

Thanks a lot for your contribution!

->

hey @chelodoz this is a wonderful PR!! Great POC! we will definitely merge it! I'll do a code review first. It will probably take me some time but I'll try to have it done by the end of the day or tomorrow 😊


How to add array parameters type support in docstring?

parameters:
- name: images
  description: pics
  required: true
  type: array

? it seems not work

Yes I have the same problem when I sended post request with an array parameter like this: "hello", "bye" or hello,bye. Swagger always send like one parameter, it do not undestand that its a list ?
Or perhaps I sent it in bad format ?
I have a SlugRelatedField with many=True and I need to rewrite for this field in docstring view for that field be an array type... but still doesn't work

Closing, YAML is now deprecated


Change duplicated variant.getJavaCompile() to variant.getJavaCompileP…

…rovider()
Related issue #279

Closing due to inactivity, feel free to re-open if you desire to work on it


ensure all material elements are imported correctly

A slightly tedious task as likely people will consistently auto import from named imports
Bad:
import {Button} from '@material-ui/core'
Good:
import Button from '@material-ui/Button
(for tree shaking reasons, will see noticeable improvement in compiling time)
Suggest that periodically search across files for "} from '@material" to pick up first case and correct
This issue shouldn't be closed

Don't you think this informations would deserve a section in the wiki instead of an issue @chrismclarke ?
Keep issues open as informations seems quite odd to me. If it is an important information or behavior that deserve to live in the wiki to me.


Fix underflow introduced in #1007

max can be 0 and subtracting 1 causes underflow

There are still underflows here, max - diff - 1 is negative. I think we need to change the params of neg_deinterleave to be signed 32-bit integers as it was in the C code.


Support Dataset Version put and get query

Codecov Report
Base: 53.78% // Head: 14.66% // Decreases project coverage by -39.12% :warning:

Coverage data is based on head (81b9f21) compared to base (489bac4).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@              Coverage Diff              @@
##             master    #2332       +/-   ##
=============================================
- Coverage     53.78%   14.66%   -39.12%
+ Complexity     7136     2365     -4771
=============================================
  Files           759      760        +1
  Lines         58731    58805       +74
  Branches       7203     7208        +5
=============================================
- Hits          31587     8626    -22961
- Misses        24352    49664    +25312
+ Partials       2792      515     -2277

Impacted Files
Coverage Δ

...ava/com/github/ambry/account/mysql/AccountDao.java
0.00% <0.00%> (-78.68%)
:arrow_down:

.../github/ambry/account/mysql/MySqlAccountStore.java
0.00% <0.00%> (-89.48%)
:arrow_down:

...com/github/ambry/account/DatasetVersionRecord.java
0.00% <0.00%> (ø)

.../src/main/java/com/github/ambry/store/Message.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/named/PutResult.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/quota/QuotaMode.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/quota/QuotaName.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/rest/RestMethod.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...c/main/java/com/github/ambry/config/KMSConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...c/main/java/com/github/ambry/config/SSLConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 493 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

1 last comment above and also some tests failed. After these two, u may merge


Add preflight checks back

Issues
This copies the preflights from https://gitlab.com/jklabsinc/OpsLevel/-/blob/main/on-prem/manifests/kots-preflight.yaml?ref_type=heads and brings them into the chart.
When they live in the chart replicated will detect this and generate extra commands that can be run using the release to run preflight checks.
They look like this
curl https://krew.sh/preflight | bash
helm template oci://registry.replicated.com/opslevel/helm/opslevel | kubectl preflight -

I have created a helm-unstable channel and i'm going to test it out.

This was added to main in a different way.


built-in properties are ignored when get user property by property name

we return user custom property directly when get user property.
https://github.com/feature-flags-co/ffc-go-sdk/blob/1d5258c95d6165463d4a2df9527de373235569a7/model/ffcuser.go#L68-L74
while in java sdk we do consider user built-in property
https://github.com/feature-flags-co/ffc-java-sdk-commons/blob/c0fda46b61f40bd2a61c5203081c817ebd983c86/src/main/java/co/featureflags/commons/model/FFCUser.java#L141-L154

fixed,
https://github.com/feature-flags-co/ffc-go-sdk/commit/f1f7d218dc78eb1eae41b2cfa5df268ac23be5f6


As a developer, I would like a vscode snippet extension

Type of Change

[ ] New Feature Request
[ ] Documentation
[ ] Future Spec
[x] Improvement / Suggestion
[ ] Bug
[ ] Other (please clarify below)

Summary
Visual Studio Code Snippet Extensions are extremely useful in assisting developers with repetitive tasks.
Use Case
One use case would be every single time you need to create a new component you have to write the entire component boilerplate or copy it from another component. It would be easier just to type 'evc' + tab to complete the code snippet.  We should document the extension and suggest installing it to speed up development.
Code Sample
Here's a working extension I've already created and submitted to the vscode marketplace.
Pros / Cons
Pro:

faster development
add a variety of scaffolds

Whoa, this is a wicked cool idea @hutchgrant !  💡
I think this would be great for the community at large.  The main projects supporting Project Evergreen are:

lit-html
lit-element

I think you should check those repos out and open an issue in lit-element repo and propose a few snippets and see what the community thinks?  Maybe we could crowd source a couple different snippets and make it a little open source thing?  Maybe the namespace could be lit-*?

Also, for code generation, what do you think of this tool?
https://github.com/elmsln/wcfactory/


Problems while rotating/flipping an image on CPU and GPU

Hi,
Can anyone share an example for rotation of an image on CPU and GPU?
When I try the CPU version, it seems to indicate that rotate function is not available.
local cv_im = cv.imread{'demo/data/lena.jpg',  cv.IMREAD_COLOR}
local im_rotated = cv.rotate{src=cv_im, dsize={512,512}, angle=1.5708}

attempt to call field 'rotate' (a nil value)
And when I rotate on GPU, I do not seem to get the correct result.
local cv_im = cv.imread{'demo/data/lena.jpg',  cv.IMREAD_COLOR}

local im_cuda = cv_im:cuda()

local im_rotated_gpu = cv.cuda.rotate{src=im_cuda, dsize={512,512}, angle=1.5708}

cv.imwrite{'demo/data/lena_rotated_gpu.jpg', im_rotated_gpu:byte()}

The image written is all black !!
Similar error while flipping an image on GPU:
cv.cuda.flip{src=im_cuda, flipCode = 0}
attempt to call field 'flip' (a nil value)

Hi,
on CPU refer to OpenCV documentation and use cv.warpAffine. cv.cuda.flip isn't implemented unfortunately. And there are still many bugs in CUDA wrappers that I didn't have time to fix.
Overall, I strongly recommend using Python because Torch is basically dead and this library is very weakly maintained.


Unit testing deepwalk

I believe that current testing is insufficient and it would be very easy to introduce bugs that would go undetected. This PR intends to introduce (simple) unit testing of deepwalk model.
This idea of testing can be abstracted and applied to other models as well.

Many thanks!


Named args

rename the variables used in desugaring.
delete unnecessary logs
add test

Before making a formal PR(i.e., non-draft), please:

read the contribution guide
make sure your code passes all regression tests
remove all compilation warning messages and unnecessary debug information

Seems this PR doesn't yet take into account the change to Fld's definition.

Seems this PR doesn't yet take into account the change to Fld's definition.
I think it would be ok after merging optional

Seems this PR doesn't yet take into account the change to Fld's definition.

I resolved the suggested changes, and also merge this branch with the current new-definition-typing(on local), to resolve possible conflicts.

To solve the compilation failures, just use FldFlags.empty instead of FldFlags(false, false).

I noticed that your latest commits removed some test cases, for example https://github.com/hkust-taco/mlscript/pull/174/commits/84995d033b9a52c196e238b54c0e82697d13c691. There are already very few tests, so please do not make it worse by reducing the coverage further! ;^)


Each game should use single currency

Cash games as well as Tournaments should have a defined currency, removing the need to enter the currency for (almost) any action.

Hmm, strange. I thought I had seen that issue, but now have a hard time reproducing.

The creation pages for cash game and tournament contain an initial Buy-In amount. This amount contains the currency. Afterwards all entered amounts are without currency.


pull request for e2e test at 01/12/2017 18:15:12

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


RBAC with owner

🎉 Description

[ ] 🐛 This is a bug report.
[x] 📈 This is a feature request.

📝 Details
Ownable controls ownership of contract. I think RBACWithOwner is suitable to control access controll with permission than RBACWithAdmin. RBACWithAdmin has several problems.

It can't be integrated with Ownable becuase both control permissions. If we have a contract T is RBACWithAdmin, Ownable and a single admin added, and want to chagne owner, 3 transaction are required

contract.transferOwnership(newOwner, {from: owner});
contract.adminAddRole(newOwner, 'admin', {from: owner});
contract.adminRemoveRole(owner, 'admin', {from: owner});

RBACWithAdmin may have no admin at all if admin send trancation

contract.adminRemoveRole(admin, 'admin', {from: admin});

I suggest to integrate Ownable with RBACWithAdmin to handle owner and ROLE_ADMIN at the same time, making sure that owner has all permission to control ROLE_ADMIN

Thanks for the issue!
RBACWithAdmin will be moved to the examples folder; it's just too powerful 😂 see: https://github.com/OpenZeppelin/openzeppelin-solidity/pull/936#issuecomment-393050257
RBACOwnable will be introduced as part of https://github.com/OpenZeppelin/openzeppelin-solidity/pull/950 , but without a focus on backwards compatibility with Ownable.sol. We previously tested a backwards-compatible version of RBACOwnable.sol but decided that it wasn't a good enough solution.

Closing as we've removed the previous RBAC and our focus is on Roles now.


terraform_docs hook broken in v1.92.2

Describe the bug
Using the latest version of the hook (v1.92.2), pre-commit is always in "PASSED" state even if we change variables, outputs or versions constrains.
How can we reproduce it?
vim versions.tf
sed -i 's/version = "~> 3.80"/version = "~> 3.86"/g' versions.tf
pre-commit run -a

Terraform fmt...........................................................................Passed
Terraform docs..........................................................................Passed

It should failed and update README.md file.
Environment information

OS:

OS: Win11 with Ubuntu 20.04 on WSL2
OS: Ubuntu 20.04

Docker image tag/git commit:  v1.92.2

.pre-commit-config.yaml:

file content
  25   │   - repo: https://github.com/antonbabenko/pre-commit-terraform.git
  26   │     rev: v1.92.2
  27   │     hooks:
  28   │       - id: terraform_fmt
  29   │         stages: [pre-commit]
  30   │       - id: terraform_docs
  31   │         stages: [pre-commit]
  32   │         args:
  33   │           - --args=--config=.config/terraform-docs.yml

(everything is working as expected with v1.92.1)

Might probably be related to https://github.com/antonbabenko/pre-commit-terraform/pull/698 🤔
@Shr3ps Would you please also share contents of the .config/terraform-docs.yml file?

Might probably be related to #698 🤔
@Shr3ps Would you please also share contents of the .config/terraform-docs.yml file?

Yep, here you go:
---
# This file is automatically maintained within this module repository -- DO NOT EDIT
formatter: "markdown"

settings:
  anchor: false
  lockfile: false

output:
  file: "README.md"

sections:
  hide: [requirements]

content: |-
  ## Global versioning rule for Claranet Azure modules

  | Module version | Terraform version | AzureRM version |
  | -------------- | ----------------- | --------------- |
  | >= 7.x.x       | 1.3.x             | >= 3.0          |
  | >= 6.x.x       | 1.x               | >= 3.0          |
  | >= 5.x.x       | 0.15.x            | >= 2.0          |
  | >= 4.x.x       | 0.13.x / 0.14.x   | >= 2.0          |
  | >= 3.x.x       | 0.12.x            | >= 2.0          |
  | >= 2.x.x       | 0.12.x            | < 2.0           |
  | <  2.x.x       | 0.11.x            | < 2.0           |

  ## Contributing

  If you want to contribute to this repository, feel free to use our [pre-commit](https://pre-commit.com/) git hook configuration
  which will help you automatically update and format some files for you by enforcing our Terraform code module best-practices.

  More details are available in the [CONTRIBUTING.md](./CONTRIBUTING.md#pull-request-process) file.

  ## Usage

  This module is optimized to work with the [Claranet terraform-wrapper](https://github.com/claranet/terraform-wrapper) tool
  which set some terraform variables in the environment needed by this module.
  More details about variables set by the `terraform-wrapper` available in the [documentation](https://github.com/claranet/terraform-wrapper#environment).

  ```hcl
  {{ include "examples/main/modules.tf" }}

{{ .Providers }}
{{ .Modules }}
{{ .Resources }}
{{ .Inputs }}
{{ .Outputs }}
...

@Shr3ps Thank you.
@MaxymVlasov Need your brain here 🤷🏻

As you used wrong bug template, @Shr3ps please provide info about environment.
https://github.com/antonbabenko/pre-commit-terraform/blob/f809e53ddfcab9fbeb0d58adaf0c44f5dfc8e706/.github/ISSUE_TEMPLATE/bug_report_local_install.md#L74-L101
Because currently, I can't reproduce this issue.
But I found a minor issue that if output.file specified, warning always shown

Hello,
some steps to reproduce my issue:
$ git clone https://github.com/claranet/terraform-azurerm-rg/
$ cd terraform-azurerm-rg
$ pre-commit install
$ sed -i 's/>= 1.32/>= 1.33/g' versions.tf
$ sed -i 's/Resource group name/Resource group name./g' outputs.tf
$ git add .
$ pre-commit run
$ git commit -m 'chore: test pre-commit'

the bug is that pre-commit outputs:
Terraform fmt...........................................................................Passed
Terraform docs..........................................................................Passed
Terraform validate......................................................................Passed
Terraform validate with tflint..........................................................Passed

but should fail instead because of the changes done on .tf files.
Environment information

$ uname -a
Linux FR-LAP822653786 <IP_ADDRESS>-microsoft-standard-WSL2 #1 SMP Fri Mar 29 23:14:13 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Tools availability and versions:

GNU bash, version 5.1.16(1)-release (x86_64-pc-linux-gnu)
pre-commit 3.7.0
OpenTofu v1.8.1
Terraform v1.5.3
python SKIPPED
Python 3.10.12
checkov SKIPPED
infracost SKIPPED
terraform-docs version v0.18.0 228c7a7 linux/amd64
terragrunt SKIPPED
terrascan version: v1.1.0
TFLint version 0.44.1
+ ruleset.terraform (0.2.2-bundled)
tfsec v1.28.1
trivy Version: 0.54.1
tfupdate SKIPPED
hcledit SKIPPED

.pre-commit-config.yaml:
https://github.com/claranet/terraform-azurerm-rg/blob/master/.pre-commit-config.yaml

Default markers:

Your markers:

Documentation about that:

So you just need to set
      - id: terraform_docs
        args:
          - --args=--config=.config/terraform-docs.yml
          - --hook-config=--use-standard-markers=true

to work as expected.
I am not sure why it was able to work before 1.92.2, but that wasn't expected behavior.
We could release a new minor version which will include force change of markers to TF_DOCS default makers (they were introduced a few years later than in this repo, that's why they different), but that will not be a bug fix :)

Also, check https://github.com/antonbabenko/pre-commit-terraform/pull/701

Thanks @MaxymVlasov , I can confirm adding the param - --hook-config=--use-standard-markers=true fixes the issue.

@Shr3ps v1.93.0 should work without - --hook-config=--use-standard-markers=true


[BUG] Materials Quantity parsed incomplete or incorrectly

Hi!
I thought your project was very good, congratulations!
I just had some OCR issues when scanning to my seelie.me account, I set it to 200ms speed.
At the top the inventory in the seelie, and at the bottom the game inventory:

Thanks for report. Unfortunately this is a known problem as the size of the numbers and the fonts on digits (especially 1's) can sometimes skew the results from the scanner. We are working on finding ways to improve this accuracy.
In the meantime can you provide the images of some items with expected and actual values?
Something like this:
(Without the red circle)

Expected: 7
Parsed: 2

This will likely be an ongoing issue but the priority on refining this kind of scanning is low priority. Other duplicate issues will be redirected to this to continue future discussion.

i'm having this issue on max settings 16:9 1440p

V1.3.0 has improved material quantity scanning so I am closing this issue. It's still not going to be perfect but it's better.


[DOM] Add a getter for innerText

The innerText DOM property was introduced by MS Internet Explorer and adopted by the other major browsers without a lot of enthusiasm, however it has its defenders and its use cases. See The poor, misunderstood innerText by Juriy Zaytsev for background information.
The property is specified in HTML, although I did not use it to write the code:
https://html.spec.whatwg.org/multipage/dom.html#the-innertext-idl-attribute
As said in the aforementioned blog post, a typical use case is about rich text editing in a browser, however it is useful for any application where a rich-text representation of a document fragment (with all its HTML tags and styles) is used, but one also wants a plain text version of that content (to store in a database field or a plain text document).
This library focuses on non-browser use cases, and this implementation is not intended to be completely equivalent to what web browsers do. For example, I try to avoid some empty lines that often appear in innerText as given by browsers, and a white space is added before a list item —as suggested by J. Zaytsev's blog post, which has been very useful— although only if the list-style-position CSS property is set to inside.
The fe-innertext branch contains the patch intended for merging.

Two of the tests use the documents innertext.html and innertext.xhtml, which are based on a sample by @kangax (see https://kangax.github.io/jstests/innerText/), as mentioned in the comment at the top of the files. I thank him (and Aryeh Gregor) for making it available.

Commit merged.


Hotpot service dockerfile update

What this PR does / why we need it:
This patch is proposed to update some Dockerfile in Hotpot service, so as to reduce docker images size.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

Coverage remained the same at 37.653% when pulling 9934975a4f9f774e7c6c52ff7caeb0741c6eac60 on leonwanghui:dockerfile-update into d065db29156f7fae61cc74e795aa816e9af9b7b4 on opensds:development.

Coverage remained the same at 37.653% when pulling 9934975a4f9f774e7c6c52ff7caeb0741c6eac60 on leonwanghui:dockerfile-update into d065db29156f7fae61cc74e795aa816e9af9b7b4 on opensds:development.


Reduce depreciation and other warnings

Started minor updates that address some of the warnings in the PSS build. Many are deprecation warnings, but there are a few more that are also addressed. Changes so far are quite minor, though there are a few more to address as of yet.

This seems to have knocked the number of warnings down to 52 or 53 (python version pending). We may be able to address some of these others, but many seem to be due to other packages like, PINT and fitsio, as well as astropy caches.

This addresses parts of issues #157 and #158.

I am going to merge this PR so that there are no future conflicts. Future warnings will be addressed in a future PR.


Tweak async transition implementation

@taion how would this work with react-router and some routes being async and others not?

See react-router-scroll, which will wrap this up for you.


Possible regression in microbenchmarks_ios stock_animation_subsequent_frame_average

It's noisy, but could be around https://github.com/flutter/flutter/pull/53757.

Looks like it's been slowly creeping up over several months.  Rebaselined.


preserve focus when showing output channel

I would like to propose that we do not pass focus to the output channel when we open it automatically.
@dbaeumer fyi

Hi @bpasero, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Benjamin Pasero). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Hi @bpasero, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Benjamin Pasero). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Thanks @bpasero. To my defense I have to say that it is strange that a panel that doesn't allow typing takes focus by default in the first place. Wouldn't it make more sense to never take focus on show and have a separate method focus.

@dbaeumer totally, now you have to convince @jrieken to behave like that and we are all set


Modify test cases for DeleteEmployee and DeleteTask

Codecov Report

Merging #325 (369c2fb) into master (0c7bd77) will increase coverage by 0.61%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #325      +/-   ##
============================================
+ Coverage     75.72%   76.34%   +0.61%
- Complexity      851      857       +6
============================================
  Files           107      107
  Lines          2604     2604
  Branches        311      311
============================================
+ Hits           1972     1988      +16
+ Misses          520      506      -14
+ Partials        112      110       -2

Impacted Files
Coverage Δ

...ain/java/manageezpz/model/task/UniqueTaskList.java
80.24% <0.00%> (+4.93%)
:arrow_up:

...nageezpz/logic/commands/DeleteEmployeeCommand.java
100.00% <0.00%> (+15.78%)
:arrow_up:

...a/manageezpz/logic/commands/DeleteTaskCommand.java
100.00% <0.00%> (+50.00%)
:arrow_up:

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 0c7bd77...369c2fb. Read the comment docs.


Unsafe Foods week 4 blog post

Here's this week's blog post from the Unsafe Foods team.

@arokem yeah you're right, my text editor's line wrapping messed that up, will fix

@arokem fixed

Thanks, Kara! I suspect the link will work now so merging.


Update asset retargeter to account for anim files with script references

Overview
After #5868 was merged, the asset retargeter failed due to .anim files potentially have script references as script instead of m_Script. This change accounts for that and improves logging this case to determine the actual file it was hit in.
Changes

Fixes: https://dev.azure.com/aipmr/MixedRealityToolkit-Unity-CI/_build/results?buildId=5553

Verification
I ran this locally and saw that retargeting succeeded.

/azp run

DO NOT MERGE tag added: Just noticed I need to update the line replacing the script IDs as well. Update incoming...

/azp run


python310Packages.mlflow: 1.24.0 -> 1.25.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.mlflow is: "Open source platform for the machine learning lifecycle"
meta.homepage for python310Packages.mlflow is: "https://github.com/mlflow/mlflow"
meta.changelog for python310Packages.mlflow is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
Warning: no invocation of /nix/store/yl4mazcnddh6y573c3cm52mwsv2b5wwx-python3.10-mlflow-1.25.0/bin/mlflow had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/yl4mazcnddh6y573c3cm52mwsv2b5wwx-python3.10-mlflow-1.25.0/bin/.mlflow-wrapped had a zero exit code or showed the expected version
0 of 2 passed binary check by having a zero exit code.
0 of 2 passed binary check by having the new version present in output.
found 1.25.0 with grep in /nix/store/yl4mazcnddh6y573c3cm52mwsv2b5wwx-python3.10-mlflow-1.25.0
found 1.25.0 in filename of file in /nix/store/yl4mazcnddh6y573c3cm52mwsv2b5wwx-python3.10-mlflow-1.25.0

Rebuild report (if merged into master) (click to expand)

6 total rebuild path(s)

3 package rebuild(s)

3 x86_64-linux rebuild(s)
3 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
mlflow-server
python310Packages.mlflow
python39Packages.mlflow

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/yl4mazcnddh6y573c3cm52mwsv2b5wwx-python3.10-mlflow-1.25.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.mlflow https://github.com/r-ryantm/nixpkgs/archive/e968d2b34455231984fc46f8965f003448363b16.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/yl4mazcnddh6y573c3cm52mwsv2b5wwx-python3.10-mlflow-1.25.0
ls -la /nix/store/yl4mazcnddh6y573c3cm52mwsv2b5wwx-python3.10-mlflow-1.25.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

mlflow-server
python310Packages.mlflow
python39Packages.mlflow

Maintainer pings
cc @tbenst for testing.

Result of nixpkgs-review pr 168478 run on x86_64-linux 1

3 packages built:

mlflow-server
python310Packages.mlflow
python39Packages.mlflow


should not be rendered on server

https://github.com/siberiacancode/google-recaptcha/blob/60bce2b882a76059a1dcc0839540fe5f09c60091/packages/react/src/components/GoogleReCaptchaCheckbox.tsx#L46C3-L46C28
should be replaced with
useLayoutEffect
I think

Hello @srghma , thanks for your question
useIsomorphicLayoutEffect specially created for ssr
You can read more here https://usehooks-ts.com/react-hook/use-isomorphic-layout-effect
Also useEffect and useLayoutEffect don't work on the server. https://react.dev/reference/react/useEffect#caveats last point


Reconnecting subscriptions after a network disconnection

I am subscribing to an event published by several Particle devices.  It works well until there is a network issue and the subscription client is disconnected.  Please correct me if I am mistaken, but as far as I can tell, the SDK does not provide any way to detect this disconnect and it doesn't seem to reconnect on its own.
After the client has been disconnected and the network is restored, no further events are received.  I can call other methods (GetDevicesAsync) successfully, so that doesn't work as a way to determine whether the subscription is still connected.  If I log out, log back in, and then resubscribe, I start receiving events again.
Is there a generally accepted way to handle this situation?  Specifically, I mean is there a way to tell whether the client lost its subscription connection vs there just haven't been any events?

@mhirons currently no. This should detect the loss of connection and on restore, automatically restart. You could technically do this in your own code, and disconnect and reconnect.
I can look into adding this


Unable to find the tiltAPB2 dataset in the anomaly_detection tutorials

According to the tutorial, the original data is the tiltAPB2 dataset from the UCR Anomaly Archive, but I don't see any data with the word "tiltAPB2" in the entire archive at the source address.
In this tutorial, the original data in "198_UCR_Anomaly_tiltAPB2_50000_124159_124985.out" has only two columns. Because label is not involved in the import of the model and is only used for evaluation (through adjbestf1), preds is reconstructed according through the model. So what does the original data represent?

Hi,
All our datasets are taken from TSB-UAD which pre-processes multiple time series anomaly detection datasets in a unified format, including the UCR Anomaly Archive.
In this case the original data is a time series along with anomaly labels. It is taken from the UCR Anomaly Archive as you mentioned.
The goal is to identify anomalous time steps given input time series. The training part of the time series is anomaly free.
Does that answer your question?
Best,
Mononito

Hi I had to reopen the issue in order to point out some of your linking errors.
In your anomaly detection tutorial you use this address UCR Anomaly Archive as the source of the dataset but the page is UCR Time Series Classification Archive.
The correct dataset source should be: https://www.cs.ucr.edu/~eamonn/time_series_data_2018/UCR_TimeSeriesAnomalyDatasets2021.zip
For TSB-UAD,  they still use the same data structure as you, i.e. value + label (0 or 1).
My question is, how do you guys convert the raw data into e.g. . /data in an xx.out file. Or did you just simply use the UCR Anomaly Archive and the labels already defined in the different data sources in it.
Looking forward to your reply, thanks a lot!


Suggest example notebook update. Library moved.

https://github.com/pinecone-io/examples/blob/a7b13d0feec0f1080106679cf6a31449732f9ebc/learn/generation/langchain/handbook/06-langchain-agents.ipynb#L342C5-L343C59
Libraray 'langchain.chains.SQLDatabaseChain' appears to have moved to 'langchain_experimental.sql.SQLDatabaseChain'
I made the following changes to make it work:
# from langchain.chains import SQLDatabaseChain
from langchain_experimental.sql import SQLDatabaseChain

Thank you for the note @bryanbarnard!
This issue has been resolved with the following PR: https://github.com/pinecone-io/examples/pull/264.

Thanks @DosticJelena


Version T - Typescript Conversion

We will be converting all of the code in the project to typescript code.
This is a major change. All files will need to be changed and all code refactored to take full advantage of Typescript's amazing features.

NEVERMIND! WE'RE BACK ON TRACK.
VERSION 3 ON THE WAY.


Dialog auth flow

Sorry if I named the issue incorrectly. Here's the flow. Need to mention that I'm running it in emulator but it still seems to be valid case. Let's say I have 2 conversation commands "List mails" and "Write mail". These commands are handled in OnMessageActivityAsync with switch-case flow. When user type in "List mails" I'm running mails listing flow where I check auth. If user is not authorized I start auth dialog like in https://github.com/microsoft/BotBuilder-Samples/tree/master/samples/csharp_dotnetcore/46.teams-auth
In the very end of auth I'm getting validation code page where I need to copy that code and paste it into the chat.
Here's the question. Who should consume that code? I see that code is getting into OnMessageActivityAsync but how should I handle it? Because now nothing happens. Auth is just getting stuck. I suppose it should somehow go into the running dialog but how I can find out which of list mails and write mail dialog is running? And how can I define that the message is a code but not random text?

I found how it should work. Will write it here if anybody face the same problem. So the code user passed into the conversation must be consumed by the dialog flow:
var dialog = new Conversation.SignInDialog(this.configuration, null, finalStep);
await dialog.RunAsync(turnContext, this.conversationState.CreateProperty<DialogState>(nameof(DialogState)), cancellationToken);

in the code above validation code will be processed by the dialog as we pass it with turnContext variable and dialog will continue to the next step.
In order to control valid flow (like on web when you redirected back to the requested url after login) Conversation state should be used: https://docs.microsoft.com/en-us/azure/bot-service/bot-builder-howto-v4-state?view=azure-bot-service-4.0&tabs=csharp


Hash '#' in URL path

Hi,
In the getting started kit, the router adds a '#' in all URL paths (eg. http://<IP_ADDRESS>:8080/#/users ).
Is there a way to get rid of the '#'?

See the section on "Configuring Push State" here: http://aurelia.io/docs.html#/aurelia/framework/1.0.0-beta.1.1.4/doc/article/cheat-sheet/7

Thanks! I will have a look.

So to my understanding all that I should do is add
config.options.pushState = true;

The '#' is gone now, but I don't understand what's going on. As far as I can tell this makes the browser use relative URL paths. But why was the '#' there in the first place? Is there an article somewhere that explains this?
Furthermore, this seems to break the routing a bit. Without pushState, I can go to the users page and refresh the page. However, with pushState, when I refresh on the users page, I get 404. Is there something else I need to change?

@bhaeussermann What you're seeing is part of the way that URIs are implemented in web browsers - it's not an Aurelia-specific thing. The hash symbol is part of the (fragment identifier)[https://en.wikipedia.org/wiki/Fragment_identifier]. It has a lot of uses, but it's used by single-page applications for history purposes, explained here: https://en.wikipedia.org/wiki/Single-page_application#Browser_history
Pushstate simply uses a different method of storing history in the browser that doesn't use the fragment identifier. You'll have to configure your server accordingly. There's a (StackOverflow question)[http://stackoverflow.com/q/17954888/677526] on the topic, and although it deals with Ruby on Rails, the principle is the same across all server types. Essentially, your server needs to serve your application page for all routes that the application itself could receive.

@jedd-ahyoung Thanks for the explanation! It seems that the approach with the fragment identifier (with pushState = false) is pretty standard among single-page apps anyway, so we should probably stick with that.
What does bother me is that it seems Aurelia adds an unnecessary / after the #. As in http://localhost:43804/#/users instead of just http://localhost:43804/#users. The latter URL certainly looks much nicer to me.


ProtoDUNE-VD sim/reco update

Running simulation and reconstruction using CORSIKAGen for cosmics generation in ProtoDUNE-VD, using Gnocchi module and Pandora reconstruction.

@wenqiang-gu Here is the last version of the running cosmics generation. Please consider the rotated generator of Corsika using double phase version and update detectorproperties and calorimetry module.


arangosh fails when provided with complex passwords in --server.password argument.

My Environment

ArangoDB Version:  3.7.12
Storage Engine: RocksDB
Deployment Mode: Single Server
Deployment Strategy:   Manual Start in Docker (docker-compose)
Configuration: N/A
Infrastructure: localhost
Operating System: Windows 10 / Docker
Total RAM in your machine:     64Gb
Disks in use: 1TB nvme, .5 Tb SSD, 2 * 2 Tb HDD, 1 * 2 Tb SSD
Used Package: arangodb:latest on Docker

Component, Query & Data
Affected feature: arangosh
AQL query (if applicable): N/A
AQL explain and/or profile (if applicable): N/A
Dataset: N/A
Size of your Dataset on disk: N/A
Replication Factor & Number of Shards (Cluster only): N/A
Steps to reproduce

Create persistence machine: docker run -d --name problem-arangodb-persistence -v /data tianon/true

Create persistence machine: docker run -d --name witness-arangodb-persistence -v /data tianon/true

Create arango image: docker run -e ARANGO_ROOT_PASSWORD=".ca8)+QR=i*c-]eM4g^:9A#y8z~t.@^@" --name problem-arangodb -p 8529:8529 --volumes-from problem-arangodb-persistence -d arangodb

Create witness arango image: docker run -e ARANGO_ROOT_PASSWORD="testtest" --name witness-arangodb -p 8530:8529 --volumes-from witness-arangodb-persistence -d arangodb

On problem-arangodb: Run arangosh --server.username root --server.password ".ca8)+QR=i*c-]eM4g^:9A#y8z~t.@^@"

On witness-arangodb: Run arangosh --server.username root --server.password "testtest"

Problem:
On problem-arangodb:
Command-line history will be persisted when the shell is exited. You can use `--console.history false` to turn this off
Could not connect to endpoint 'tcp://<IP_ADDRESS>:8529', database: '_system', username: 'root'
Error message: 'not authorized to execute this request'

On witness-arangodb:
Command-line history will be persisted when the shell is exited. You can use `--console.history false` to turn this off
Connected to ArangoDB 'http+tcp://<IP_ADDRESS>:8529, version: 3.7.12 [SINGLE, server], database: '_system', username: 'root'

Apparently arangosh cannot handle complicated passwords with symbols
Expected result:
problem-arangodb would behave like witness-arangodb.

the password can be reduced to .@^ to reproduce this issue.

hm, its not a bug, its a feature:
# ./bin/arangosh --log.hostname 'a@abc@' --javascript.execute-string "print(require('internal').options()['log.hostname'])"
a
# abc=123 ./bin/arangosh --log.hostname 'a@abc@' --javascript.execute-string "print(require('internal').options()['log.hostname'])"
a123

whether this makes sense in passwords could be questioned.

we however do have two issues here which will be worked on with BTS-515 :

only  [A-Za-z_0-9] must be allowed between @'s, else an exception should be thrown
one should be able to escape @'s by using double-@@.

@AtomicNicos you probably now know how to work around your issue using the PASSWORD environment variable? (see last post)

I get why it happens, but for a password field that makes no sense in the context of arg parsing. Is this a "feature" of the C++ arg parser?
Because otherwise instead of making the symbols not allowed in general and requiring an escape rule, you could just take the password string as it is without throwing it through a parser.
Let's be honest, if we wanted to use an environment variable we could just use it the normal way we use env variables (ie. $abc)

Every startup option supports substitution using the @var@ syntax. This is not limited to startup options you supply via command-line arguments, but also works in configuration files.
Another possible reason for this feature is to have a consistent way across OSes and terminals to interpolate environment variable values. On Windows it's %var% in cmd.exe, but $Env:var in PowerShell for instance.
By the way: arangosh --server.username root --server.password ".ca8)+QR=i*c-]eM4g^:9A#y8z~t.@@^@@" (with at signs doubled) should already work in current versions of ArangoDB.

Warning issued in docs, closing issue.


How to install programs in the simpleSSD?

Hi,
The OC-SSD device is finally captured when I typing following command: ./nvme list.
I would like to do a git clone, but there cannot be done apt-get, and downloading a program with wget also cannot be done because of the network problem.
So, how to create a target?

Hi,
gem5 does not support internet connection with host machine (because of timing differences).
You must compile all the programs/libraries on host machine (using cross-compiler, if you use ARM), and copy it to the disk image (e.g., linux-aarch64-linux.img).
To create pblk target, use following commands inside the gem5.
# You can find total number of LUN in nvm notation
# e.g., nvm notation (old): [8/1/512/512/64/8] -> 5th number (64) is total LUN
# e.g., nvm notation (new): [8/8/512/64/8] -> 4th number (64) is total LUN
nvme lnvm create -d nvme0n1 -n lnvm0 -t pblk -b 0 -e 63
# -d : identifier of desired device
# -n : target name of the device to initialize
# -t : identifier of target type
# -b : begin of LUNs to use
# -e : end of LUNs to use (here, 63 = 64 - 1)

This will create block device named /dev/lnvm0.
Thanks.

hello，how can i solve the problem that can't find the type of pblk

hello，how can i solve the problem that can't find the type of pblk
If i use 'sudo nvme lnvm info' , there is no target type registered. So I can't create pblk target.


can https be used?

Is https supported?
it always returns an empty list {} when I use it with python-odata
(example url:  https://services.odata.org/V4/Northwind/Northwind.svc/ )

That site seems to have problems with its certificate. My tests just return odata.exceptions.ODataConnectionError: hostname 'services.odata.org' doesn't match either of '*.azurewebsites.net', '*.scm.azurewebsites.net', '*.azure-mobile.net', '*.scm.azure-mobile.net' when trying to use it.

ok. that was just an example.
Having the same issue with other https that I tested (sorry, not publicly available yet)

It should work, python-odata doesn't really care if you're using http or https. The underlying library Requests handles all connectivity. If you receive empty results (no exceptions from requests, that is) then I'd say https is working.

https is working at my end, we have been using it on Production for a while without any issue.


fix off by one error in stats display

Fix an off-by-one error in displaying timing stats.

Maybe a comment?

To convey what? The access here is just to the last element in a sorted array (which is the max value of the times collected).


Glob events

Implements glob matching for event listening #19

I like file globbing but there's no reason why we couldn't do both.

I do find globbing easier to understand what the intent is though,
*/* means from all other devices all events, whereas the regular expression .* just means match anything.

Yeah, I agree actually. If we need full regex support, (which I sort of doubt), we can just add it then.


Async unfold?

Hi,
The only unfold-like method I've found in System.Interactive.Async is AsyncEnumerable.Generate, which doesn't take any async functions.
So I looked at its source code and it seemed easy enough to write a class similar to GenerateAsyncIterator but with an async iterator func.
However AsyncIterator is internal, so it seems I shouldn't write my own iterators after all? How should I go about this (barring the obvious brute force of copying and pasting the AsyncIterator code into my own codebase)?
Thanks

Right now none of the Ix methods take async parameters. There are several open issues on this, #144, #168, #194, # 280. This will be added eventually, but we're also looking at the C# async streams features to see how the design relates.

Right now none of the Ix methods take async parameters. There are several open issues on this, #144, #168, #194, #280. This will be added eventually, but we're also looking at the C# async streams features to see how the design relates.

Could you please point to these "C# async streams features"? I haven't been able to find them.

Hi @BenjaminHolland, there are two threads. The really long original one here: https://github.com/dotnet/roslyn/issues/261 that was then closed/moved over to here: https://github.com/dotnet/csharplang/issues/43

Closing this for now, defering to C# 8.

FWIW if anyone happens to land on this issue: I gave up trying to use Ix.NET long ago and switched to https://github.com/Dasync/AsyncEnumerable where it is trivial to write an async unfold.