Implement simple source

Hello,
The PR

removes description from Error since the method is deprecated now,
adds a trivial ``source`

Thanks for taking your time to make a PR!
The changes you suggest while technically correct aren't substantive enough to include.
Removing description is technically a breaking change and it doesn't hurt anyone by being there. Just implementing it doesn't appear to cause any deprecation warnings so I don't think it's a good idea to remove at this time.
Adding a source method isn't necessary as the default implementation already returns None.


ComputeCenterOfMass always returns false

While trying out version 4.0 of this library, I found that there was no implemented version of ComputeCenterOfMass which returns true, or sets the value of the center of mass to other than 0.
Is it suggested to implement this function in an inherited class, or is this intentional?

Sounds like it just got missed in the refactor of the code. I'll see about hooking it up.


Cannot compile with a new trantor

In file included from /fantasy-peak/drogon/lib/inc/drogon/utils/HttpConstraint.h:17,
                 from /fantasy-peak/drogon/lib/inc/drogon/HttpAppFramework.h:20,
                 from /fantasy-peak/drogon/lib/src/HttpAppFrameworkImpl.h:17,
                 from /fantasy-peak/drogon/lib/src/HttpAppFrameworkImpl.cc:15:
/fantasy-peak/drogon/lib/inc/drogon/HttpTypes.h: In function ‘trantor::LogStream& drogon::operator<<(trantor::LogStream&, drogon::ReqResult)’:
/fantasy-peak/drogon/lib/inc/drogon/HttpTypes.h:215:16: error: ambiguous overload for ‘operator<<’ (operand types are ‘trantor::LogStream’ and ‘std::string_view’ {aka ‘std::basic_string_view<char>’})
  215 |     return out << to_string_view(result);
      |            ~~~ ^~ ~~~~~~~~~~~~~~~~~~~~~~
      |            |                    |
      |            trantor::LogStream   std::string_view {aka std::basic_string_view<char>}
In file included from /fantasy-peak/drogon/lib/inc/drogon/utils/string_view.h:24,
                 from /fantasy-peak/drogon/lib/inc/drogon/HttpTypes.h:18,
                 from /fantasy-peak/drogon/lib/inc/drogon/utils/HttpConstraint.h:17,
                 from /fantasy-peak/drogon/lib/inc/drogon/HttpAppFramework.h:20,
                 from /fantasy-peak/drogon/lib/src/HttpAppFrameworkImpl.h:17,
                 from /fantasy-peak/drogon/lib/src/HttpAppFrameworkImpl.cc:15:
/fantasy-peak/drogon/trantor/trantor/utils/LogStream.h:208:11: note: candidate: ‘trantor::LogStream::self& trantor::LogStream::operator<<(std::string_view)’
  208 |     self &operator<<(const std::string_view v)
      |           ^~~~~~~~
In file included from /fantasy-peak/drogon/lib/inc/drogon/HttpTypes.h:18,
                 from /fantasy-peak/drogon/lib/inc/drogon/utils/HttpConstraint.h:17,
                 from /fantasy-peak/drogon/lib/inc/drogon/HttpAppFramework.h:20,
                 from /fantasy-peak/drogon/lib/src/HttpAppFrameworkImpl.h:17,
                 from /fantasy-peak/drogon/lib/src/HttpAppFrameworkImpl.cc:15:
/fantasy-peak/drogon/lib/inc/drogon/utils/string_view.h:36:19: note: candidate: ‘trantor::LogStream& trantor::operator<<(trantor::LogStream&, const string_view&)’
   36 | inline LogStream &operator<<(LogStream &ls, const drogon::string_view &v)

6:19: note: candidate: ‘trantor::LogStream& trantor::operator<<(trantor::LogStream&, const string_view&)’
36 | inline LogStream &operator<<(LogStream &ls, const drogon::string_view &v)

Try to move the trantor::LogStream& trantor::operator<<(trantor::LogStream&, const string_view&) to the boost::string_view section.

Now it's okay
namespace drogon
{
#if __cplusplus >= 201703L || (defined _MSC_VER && _MSC_VER > 1900)
using std::string_view;
#else
using boost::string_view;

namespace trantor
{
inline LogStream &operator<<(LogStream &ls, const drogon::string_view &v)
{
    ls.append(v.data(), v.length());
    return ls;
}
}  // namespace trantor

#endif
}  // namespace drogon


Consume consignment needs to update the state of all contracts.

I issued two contracts in the same UTXO and generated an invoice to pay one of both.
After making the transfer, publishing the Bitcoin tx, and accepting the consignment file, only the contract referenced in the invoice changed its state, even though it had a bundle referencing the other contract in the consignment file.
I think all contracts related to the consignment file need your state changed to avoid the state keep anchored to spent UTXO.

Can you please clarify: there is a payment side and beneficiary side.

and accepting the consignment file

This happens on the beneficiary side. The rest is related to the paying side. In your case was you self-paying?

This happens on the beneficiary side. The rest is related to the paying side. In your case was you self-paying?

No, I test with two environments: One for the payer and the other for the beneficiary.
The error occurs in the payer side, because only he is aware of the existence of the two contracts. After publish bitcoin TX, the payer needs consume the consignment to update your contract(s) state(s). But, only the contract related with invoice changed your state, while the other contract remains anchored in the UTXO that was spent.
Does make sense?

The consignment in the payee side doesn't need to be consumed since it is already part of the stash.
But if it is consumed nothing should change (it should be a no-op), so this still a bug

The consignment on the payee side doesn't need to be consumed since it is already part of the stash.
But even if it is consumed nothing should change (it should be a no-op), so this still a bug

Ok, sure.
Today I execute the accept transfer on the payer's side because even after the transfer operation, the state of the stock/inventory remains the same.


OpenAlClientError - statuscode: 404, UndocumentedPayload()

Getting this as a response

I'm getting the same issue too. API token is correct when testing on Postman. Something must be wrong with the URLs are generated in the package


implement "autocomplete_suggest_imports"

It looks like this functionality was never implemented, even though the option exists.

I've now pushed a new release of margo that should support this by default. CHANGELOG.md has details about it.


Relax the build target checks in windows-targets sub-crates

I've done this change for Rust9x initially, but realized that the recently-added tier3 win7 targets also fail to link with the strict check in the target crates:
> cargo +nightly build --target x86_64-win7-windows-msvc -Zbuild-std
[...]

error: linking with `link.exe` failed: exit code: 1181
  |
  = note: "C:\\Program Files\\Microsoft Visual Studio\\2022\\Preview\\VC\\Tools\\MSVC\\14.39.33321\\bin\\HostX64\\x64\\link.exe" "/NOLOGO" [...]
  = note: LINK : fatal error LNK1181: cannot open input file 'windows.0.52.0.lib'

This change aligns the checks for the msvc targets with the checks in crates\libs\targets\Cargo.toml.
I didn't touch the gnu/gnullvm ones since I'm unclear about target_abi actually being stable/checkable yet. LMK if I should add the respective checks.
Also please feel free to close if this doesn't align with windows-rs. For example, technically it could be interpreted as support for unsupported target vendors.
CC @roblabla

My only other comment is that if this is worth doing then it's worth doing for all the target crates, not just the msvc ones.

Agreed, just wasn't sure about the target_abi checks. If someone could chime in there I'll update it asap!

Ah, unfortunately cfg_target_abi is not stable yet https://github.com/rust-lang/rust/issues/80970
So it'd need to be parsed from the target string.

That's what I thought as well, but wasn't sure anymore after seeing https://github.com/microsoft/windows-rs/blob/a6bdfc873e9e9b7a3018830e1c243ec399555853/crates/libs/targets/Cargo.toml#L28
Anyways, would a check for .ends_width("-gnullvm") be good enough for now, then?

I went ahead and added those changes as a separate commit 👍

It's quite a mess since (a) cfg_target_abi isn't stable and (b) gnu/llvm in general seems to be quite a mess. (#2515).

gnu/llvm is potentially getting promoted to a tier 2 target soon so I'll try to push for stabilization of target_abi.

Yikes, these conditionals are hard to review without loading up a bunch of toolchain context. If the tests pass, that's great. But it would be helpful to see table of supported targets and the attributes being used to identify them to understand the composition. (Not a blocker here.)

Documented here: https://doc.rust-lang.org/cargo/reference/environment-variables.html
Be good to have some targeted yml workflows that verify each target.

Another positive consequence of this change is that you can now use windows-sys with i586-pc-windows-msvc, where before it would filter specifically on i686.
(I discovered this PR because I was trying to figure out why the released windows-sys didn't work for me even though the code at Git head did...)


Removed URLStringConvertible conformance on URLRequest

This PR is an alternate approach to solving the issue identified in #1488. Overall, I think this is a better approach to solving the problem. This allows us to keep the same public APIs, but allow the compiler to catch all the places where the problem wasn't previously identified.
Overall, I don't think it is necessary to keep the URLRequest API conforming to URLStringConvertible. We don't actually use it for anything anymore and it's misleading and problematic. I actually almost removed it before but didn't have a strong reason to because I hadn't realized what @tremblay did.

cc @jshier and @kcharwood for review.

ð Looks good to me.

Awesome, thanks @jshier.

This has broken AlamofireImage

See https://github.com/Alamofire/AlamofireImage/issues/174

For what it's worth, I definitely agree that this is a better solution than my approach in #1488

Thanks for the heads up @adamwaite...we'll get AlamofireImage fixed up as well.

@cnoon @tremblay I just noticed the same issue, strangely enough the Carthage dependency compiles and the app crashes at runtime with a dyld: lazy symbol binding failed: Symbol not found error. I would have expected that Carthage would have failed to build AlamofireImage. Any idea why?

AFI is a bit out-of-date with AF at the moment. We're wrapping the AF 4 release right now. Once that is out, AFI will be next.


Dependencies

Changed the dependency structure as discussed in https://github.com/biojs/biojs3/issues/4

thanks - I will now update the dependencies to point to biojs ;-)


Check code style during Travis CI build

This will currently break the build as intended

@floriankramer can you approve the other PR fixing the syntax issues too? Then we can merge that one before and don't break the build on master

Arghh, versions. So with the current master my local Arch Linux clang-format (6.0.1) runs fine but the Travis (5.0) complains on Operations.h. My guess would be it differs in C++17 features or so. Then again 6.0 is not available on Travis even with the extra LLVM repo.
@floriankramer how do you think we should proceed?

@floriankramer maybe we could combine this with doing just a docker build on Travis CI as suggested in #122. Then we could settle on using the clang-format in the same Ubuntu version as used by the docker images and I could simply do my formatting with a container.


[FLINK-25222][tests] Remove NetworkFailureProxy used in Kafka tests

Unchanged backport of #18057

@flinkbot run azure

@flinkbot run azure


IndexError in pycocotools

Thanks for your error report and we appreciate it a lot.
Describe the bug
When running the image_demo.py, I get an error about numpy index.
Reproduction
python demo/image_demo.py demo/demo.jpg configs/mask_rcnn/mask_rcnn_r50_fpn_1x_coco.py     checkpoints/mask_rcnn_r50_fpn_1x_coco_20200205-d4b0c5d6.pth --device cpu

Environment
`ys.platform: linux
Python: 3.7.4 (default, Aug 13 2019, 20:35:49) [GCC 7.3.0]
CUDA available: True
CUDA_HOME: /usr/local/cuda
NVCC: Cuda compilation tools, release 10.0, V10.0.130
GPU 0: GeForce GTX 1080 Ti
GCC: gcc (Ubuntu 5.4.0-6ubuntu1~16.04.12) 5.4.0 20160609
PyTorch: 1.4.0
PyTorch compiling details: PyTorch built with:

GCC 7.3
Intel(R) Math Kernel Library Version 2019.0.4 Product Build 20190411 for Intel(R) 64 architecture applications
Intel(R) MKL-DNN v0.21.1 (Git Hash 7d2fd500bc78936d1d648ca713b901012f470dbc)
OpenMP 201511 (a.k.a. OpenMP 4.5)
NNPACK is enabled
CUDA Runtime 10.1
NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_61,code=sm_61;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_37,code=compute_37
CuDNN 7.6.3
Magma 2.5.1
Build settings: BLAS=MKL, BUILD_NAMEDTENSOR=OFF, BUILD_TYPE=Release, CXX_FLAGS= -Wno-deprecated -fvisibility-inlines-hidden -fopenmp -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -O2 -fPIC -Wno-narrowing -Wall -Wextra -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-sign-compare -Wno-unused-parameter -Wno-unused-variable -Wno-unused-function -Wno-unused-result -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Wno-stringop-overflow, DISABLE_NUMA=1, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, USE_CUDA=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON, USE_STATIC_DISPATCH=OFF,

TorchVision: 0.5.0
OpenCV: 4.2.0
MMCV: 0.5.4
MMDetection: 2.0.0+unknown
MMDetection Compiler: GCC 5.4
MMDetection CUDA Compiler: 10.0
Error traceback
Traceback (most recent call last):
  File "demo/image_demo.py", line 26, in <module>
    main()
  File "demo/image_demo.py", line 22, in main
    show_result_pyplot(model, args.img, result, score_thr=args.score_thr)
  File "/root/mmdetection-master/mmdet/apis/inference.py", line 146, in show_result_pyplot
    img = model.show_result(img, result, score_thr=score_thr, show=False)
  File "/root/mmdetection-master/mmdet/models/detectors/base.py", line 211, in show_result
    mask = maskUtils.decode(segms[i]).astype(np.bool)
  File "/root/anaconda3/lib/python3.7/site-packages/pycocotools-2.0-py3.7-linux-x86_64.egg/pycocotools/mask.py", line 91, in decode
    return _mask.decode([rleObjs])[:,:,0]
  File "pycocotools/_mask.pyx", line 146, in pycocotools._mask.decode
  File "pycocotools/_mask.pyx", line 128, in pycocotools._mask._frString
IndexError: only integers, slices (`:`), ellipsis (`...`), numpy.newaxis (`None`) and integer or boolean arrays are valid indices

Please try to install pycocotools through pip pip install "git+https://github.com/cocodataset/cocoapi.git#subdirectory=PythonAPI".
Sometimes the numpy version can also cause some problem. If the error still exist when you have installed the right pycocotools, you may try numpy=1.17

@Johnson-Wang I did install pycocotools from github, and the numpy version is 1.17, but the problem is not solved.

@Johnson-Wang and I also tried lowering the numpy version, it just doesn't work

Maybe try numpy>=1.18 ?

@ZwwWayne  Yes, I also try numpy 1.18, but it still doesn't work

Did you install pycocotools from pip before you install from github using pip install "git+https://github.com/cocodataset/cocoapi.git#subdirectory=PythonAPI", it might because the environment already have one and the new installation does not work. In this case you need to uninstall all pycocotools in your environment and re-install a new one.

@ZwwWayne Before install from github, I uninstalled pycocotools and make sure that there is no pycocotools libarary files in the system, but it just can't work.

Then how about numpy? Did you do the same thing?

Yes, I did everythingk, I even uninstall conda and clean all python environment, it doesn't work

try to rewrite the def show_result(self, data, result, **kwargs) in your detector.py and add self.CLASSES = tuple('A','B', ... , 'Z') before super(detector_name, self).show_result(data, result, **kwargs)

@aimhabo Actually I am doing something about instance segmentation, so I have to call the pycocotools , otherwise I will delete the segmentation part. But I don't understand how could similar labels cause an error in numpy?

@mangdian I mean the similar ground-truth information like labels.
In the dataset settings, it will load MS-COCO's information automatically while you use other dataset with dataset_type = 'CocoDataset'.
In my similar situation, I force the detector's self.CLASSES in show_result().

The problem is that the segmentation returned by the models is numpy boolean arrays instead of undecoded bytes data, so there is no need to decode the result. I finally solve this problem by modifying mmdet/apis/inference.py (delete the decode part). However, it is very strange that the code works well on another linux server of mine, no need to modify inference.py. Anyway, modifying inference.py can be a temporal solution to solve this problem.

Hi @mangdian ,
Thanks for your bug report. Do you use the newest version when you meet the bug? It seems that we did not change inference.py after changing the test logic. Would you like to create a PR to fix that?

@ZwwWayne Yes, I am using the newest version of mmdetection. What confuses me is that I tried the demo in two linux server. On one machine the demo code works, while on the other it doesn't work. I am not sure if it is a bug, or it is something related the numpy version or Cython version (if it is related to version, modifying the code may not be a good choice). But the reason that leads to the error is clear. The segmentation result returned by the model is numpy boolean array, insead of encoded bytes, so the cocotools failed to decode the results. It seems that some other people also meet this problem. Should I create a PR?

Met the same problem here.
I install the newest version mmdetection following the guide,  it very annoying to met this error while testingmask rcnn with the demo code:
IndexError: only integers, slices (`:`), ellipsis (`...`), numpy.newaxis (`None`) and integer or boolean arrays are valid indices

What's more, the model download urls for aliyun was not work

@FantasyJXF Please try my solution:
replace line 213 in mmdet/models/detectors/base.py:
               mask = maskUtils.decode(segms[i]).astype(np.bool)
with
mask = segms[i]

@FantasyJXF Please try my solution:
replace line 213 in mmdet/models/detectors/base.py:
 mask = maskUtils.decode(segms[i]).astype(np.bool)
with
mask = segms[i]

Very useful, thanks for your contribution.
So there is no need to do the extra decode to get boolean array, the model already do that, maybe that's because the developer didn't check the segmentation model while release.
First day using mmdetection, almost  turn to dedectron2.
Thank you again.

@mangdian Thanks for the fix, it will be appreciated if you could create a PR for it.

@hellock  OK， I will create a PR for it. Glad to do it.

Two things to fix:

Change https://github.com/open-mmlab/mmdetection/blob/master/mmdet/models/detectors/base.py#L213 to mask = segms[i].
Change https://github.com/open-mmlab/mmdetection/blob/master/mmdet/models/detectors/cascade_rcnn.py#L34 to return xxxx

#2697

@hellock  Hi, anthor not related problem in model zoo
It seems the mirror sites for ALIYUN was not work, and the AWS mirror url potfix open-mmlab seems changed to mmdetection.

Yes, the aliyun mirror site has not been updated to host v2.0 models. It will be updated later.


Storage keeps disconnecting

Several times a day Luckperms loses connection to my MariaDB storage. I have changed databases several times but it still happens.  None of the other plugins I use have had a connection problem only luck perms do. I have even tried using one of the databases one of my other plugins is on which works correctly but luck perms still lose connection to it. It keeps spamming this every second in my console for hours when it starts.
23:04:09] [luckperms-scheduler-worker-21/WARN]: java.sql.SQLTransientConnectionException: luckperms-hikari - Connection is not available, request timed out after 5000ms.
> [23:04:09] [luckperms-scheduler-worker-21/WARN]: at me.lucko.luckperms.lib.hikari.pool.HikariPool.createTimeoutException(HikariPool.java:697)
> [23:04:09] [luckperms-scheduler-worker-21/WARN]: at me.lucko.luckperms.lib.hikari.pool.HikariPool.getConnection(HikariPool.java:196)
> [23:04:09] [luckperms-scheduler-worker-21/WARN]: at me.lucko.luckperms.lib.hikari.pool.HikariPool.getConnection(HikariPool.java:161)
> [23:04:09] [luckperms-scheduler-worker-21/WARN]: at me.lucko.luckperms.lib.hikari.HikariDataSource.getConnection(HikariDataSource.java:100)
> [23:04:09] [luckperms-scheduler-worker-21/WARN]: at me.lucko.luckperms.common.storage.implementation.sql.connection.hikari.HikariConnectionFactory.getConnection(HikariConnectionFactory.java:131)
> [23:04:09] [luckperms-scheduler-worker-21/WARN]: at me.lucko.luckperms.common.messaging.sql.SqlMessenger.getConnection(SqlMessenger.java:85)
> [23:04:09] [luckperms-scheduler-worker-21/WARN]: at me.lucko.luckperms.common.messaging.sql.AbstractSqlMessenger.pollMessages(AbstractSqlMessenger.java:96)
> [23:04:09] [luckperms-scheduler-worker-21/WARN]: at me.lucko.luckperms.common.plugin.scheduler.AbstractJavaScheduler$ErrorReportingRunnable.run(AbstractJavaScheduler.java:118)
> [23:04:09] [luckperms-scheduler-worker-21/WARN]: at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
> [23:04:09] [luckperms-scheduler-worker-21/WARN]: at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
> [23:04:09] [luckperms-scheduler-worker-21/WARN]: at java.lang.Thread.run(Thread.java:748)
> [23:04:09] [luckperms-scheduler-worker-21/WARN]: Caused by: java.sql.SQLException: Connection.setNetworkTimeout cannot be called on a closed connection
> [23:04:09] [luckperms-scheduler-worker-21/WARN]: at me.lucko.luckperms.lib.mariadb.internal.util.exceptions.ExceptionMapper.getSqlException(ExceptionMapper.java:278)
> [23:04:09] [luckperms-scheduler-worker-21/WARN]: at me.lucko.luckperms.lib.mariadb.MariaDbConnection.setNetworkTimeout(MariaDbConnection.java:1624)
> [23:04:09] [luckperms-scheduler-worker-21/WARN]: at me.lucko.luckperms.lib.hikari.pool.PoolBase.setNetworkTimeout(PoolBase.java:552)
> [23:04:09] [luckperms-scheduler-worker-21/WARN]: at me.lucko.luckperms.lib.hikari.pool.PoolBase.isConnectionAlive(PoolBase.java:168)
> [23:04:09] [luckperms-scheduler-worker-21/WARN]: at me.lucko.luckperms.lib.hikari.pool.HikariPool.getConnection(HikariPool.java:185)

Please see here for more info: https://github.com/lucko/LuckPerms/wiki/Storage-system-errors


Going forward with Bevy 0.12

Bevy 0.12 adds native support for remote asset sources, making most code here redundant.
Will bevy_web_asset become a legacy crate, or will it provide boilerplate implementations for http/https?
app.register_asset_source("http", ..);
app.register_asset_source("https", ..);

0.7.0 is out! Thanks for reviving this crate :D


Zlib Subtree dependency has been updated to v1.2.12

This PR will address CG Security Bugs existed part of Zlib v1.2.11 dependency
[What is changed:]

Removed Old Git Subtree for Zlib v1.2.11
Added a new Git Subtree for Zlib v1.2.12 which includes fixes for CG Security Bugs related to

trees.c
deflate.c

Updated the README.md with correct version, commit & release link for Zlib v1.2.12

[How Verified:]

Compiled the code with updated Zlib v1.2.12 version for windows environment

makewin.cmd x64
makewin.cmd x64 -px

Executed tests above compiled cases ensured all the tests are passing as before

.vs\msixtest\msixtest.exe

Just make sure that when you complete this PR you don't squash and merge


fix all, dart publish --dry-run warnings and errors

Codecov Report

Merging #45 (e76fb04) into main (0976a0e) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #45   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           11        11
  Lines           66        66
=========================================
  Hits            66        66

Impacted Files
Coverage Δ

lib/src/hooked_bloc.dart
100.00% <ø> (ø)

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 0976a0e...e76fb04. Read the comment docs.


Add ARIA label field to Hyperlink atom

One of the things we learned in the course of the accessibility mini-project is that we can use the aria-label attribute on <a> elements to provide additional context to screen reader users in cases where the cleared link text was something like "Learn more" or when including the context would significantly harm the experience for sighted users (e.g., repetitive links in a long table).
This PR adds an ARIA label field to the Hyperlink atom that's found in many modules so that Wagtail editors can add a value for the aria-label attribute when appropriate.
Additions

New ARIA label field on Hyperlinks and the template update to output it

Testing

Pull branch
Migrate
Add/edit any module that includes a Hyperlink
Preview it without adding anything to the ARIA label field and see that it has no aria-label attribute
Enter something in the ARIA label field and preview it
Observe that it now has an aria-label attribute

Screenshots

Notes

Unfortunately, this doesn't do anything for links added within a rich text field. After the move to Wagtail 2.0, we should consider a Draftail plugin that would enable that.

Checklist

[ ] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the development playbook
[ ] Passes all existing automated tests
[ ] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future todos are captured in comments
[ ] Visually tested in supported browsers and devices (see checklist below :point_down:)
[ ] Project documentation has been updated
[ ] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist
Browsers

[ ] Chrome
[ ] Firefox
[ ] Safari
[ ] Internet Explorer 8, 9, 10, and 11
[ ] Edge
[ ] iOS Safari
[ ] Chrome for Android

Accessibility

[ ] Keyboard friendly
[ ] Screen reader friendly

Other

[ ] Is useable without CSS
[ ] Is useable without JS
[ ] Flexible from small to large screens
[ ] No linting errors or warnings
[ ] JavaScript tests are passing

As mentioned in chat, I have the same issue as @jimmynotjim

Hmm... I'll look into it.

Seems like I overestimated how much the Hyperlink template was being used. I'll update all the components I can find to use it.

@Scotchester what's the status of this PR?

Covered by https://github.com/cfpb/consumerfinance.gov/pull/6497


ci: bump node version in release workflow

semantic-release now requires at least version 20. Missed that when merging the bump PR with all the red from the invalid commit messages.

:tada: This PR is included in version 4.4.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fira Code on Google fonts does not have enough characters

This is content when exporting text with Fira code from Google Fonts

And this is when I use the local Fira code font

Some how the font in Google Fonts misses some characters for drawing boxes

Fixed with #376 and #375


Add hostalises to opentelemetry-collector helm chart

I am using opentelemetry-collector helm chart to deploy my instances of collector that serves as a batch processor of jaeger tracing and exports traces to remote jaeger collector. During this process, I need to update the hostalies field of otel collector deployment to add a DNS entry for routing purpose. However this field is not available in the helm chart and I hence have to customize the helm chart to achieve this goal.
I wonder if it is possible to include this field in otel collector helm chart? I am happy to submit a PR for this.

duplicate of #81

Oh sorry I did not recall I have raised a duplicate issue before :(


Rate Limit Warning Somewhat Noisy

Im guessing GitLab doesn't send these rate limit headers, and seeing this warning (https://github.com/JuliaWeb/GitForge.jl/blob/master/src/ratelimits.jl#L31) for every request seems somewhat misleading and noisy.
Is there a better solution?

Self-hosted GitLab has no rate limits (or they can be disabled at least), the community version does. GitLabAPI has a has_rate_limits keyword to disable the checks and subsequent warnings.

oh interesting, thanks for the info! Guess I can probably close this then


Function deployment fails in multi node kind clusters

Setup:
1 master node
2 worker nodes
Image used: vhiveease/vhive_dev_env
Error:
Knative function containers fail to boot. Status loops between [Error, CrashLoopBackOff, Terminating].
Error log:
root@kind-control-plane:/vhive# kubectl logs helloworld-0-00001-deployment-7b8fcdb447-knnhf queue-proxy
time="2021-02-18T13:27:16Z" level=info msg="This is the vHive QP code"
required key GUEST_ADDR missing value

I experience the same error when trying to run vHive on a single-node minikube cluster. However, I don't think the cluster setup has anything to do with the error. It seems like the image docker.io/vhiveease/queue-39be6f1d08a095bd076a71d288d295b6 doesn't run correctly unless GUEST_ADDR is set (perhaps as an environment variable).
The following lines (introduced on 27.11.2020 and 30.11.2020) inside the file cri/container_create.go seem to require a value for GUEST_ADDR inside the function createQueueProxy(...):
guestIPEnv = "GUEST_ADDR"
...
guestIPKeyVal := &criapi.KeyValue{Key: guestIPEnv, Value: vmConfig.guestIP}

Note: I have tried both versions (i.e., latest and  c3d2e0ad00457c972b642a1b6cf7c67ac1a58a5dcb83207e244c3cf6e19befbe) of the image docker.io/vhiveease/queue-39be6f1d08a095bd076a71d288d295b6, but neither works for me. Since both versions are younger than the code changes described above, this isn't a surprise to me.
If I decide to work with vHive in the future, I will take a closer look at this issue.

@vfoehn thank you for your comment.
vHive starts each VM with a tap and its corresponding IP address (this is assigned by the vHive code not the CNI plugin), using the environment variable that you mentioned. This is the intended behavior.
This issue is about a multi-node setup in a kind container (i.e., a kubernetes cluster running in Docker containers instead of VMs or bare metal). The multi-node setup should work on bare metal or VMs without any problem.
If you use vHive in a different scenario. Please raise a separate issue and we could try to investigate it together.

@ustiugov thanks for the quick reply and the information.
I didn't follow the instructions from docs/quickstart_guide.md to a T since I wanted to run vHive on a minikube cluster (as opposed to creating a new cluster using kubeadm) I had already installed on my machine. Consequently, it is very possible that the mistake was caused by me or my setup. This is why I'm hesitant to raise a separate issue just yet. However, if I start using vHive for my project, I will investigate it more closely.

Hi @ustiugov ,
I am running into a similar issue using a local single node cluster setup using the single node steps described in the quick-start guide.
Since I used the create_one_node_cluster and did not see any errors, I am assuming all the deployment have been done correctly. Any insight into it is appreciated.

Hi @prateeksahu,
This issue was open for kind containers, are you running in kind or on a bare metal / VM? If your issue is not related to the kind setup, please open a new issue.
Then, could you please specify the exact steps (which you execute on a clean Ubuntu 18 node or a vm) and supply relevant snippets of the logs, attaching full logs?


Cant select "MP" In Settings

I downloaded RC7 fron the github updated the cydia impactor, Put it on my iphone 7 running 11.3.1  I was able to select MP the first time and the phone restarted half way through it (didnt jailbreak) then I was unable to select MP again.

“This isn’t a bug, it’s a feature”
You need to have an Apple developer certificate to use multipath. You can purchase one for $99 USD/year from https://developer.apple.com
If the app detects that you don’t have a dev cert, it won’t let you use MP because it would never work.


ConstantContact Provider

I currently use ConstantContact integration in my project and I would like to contibute the provider since it is not currently included.  Is this out of scope for the current project?

I'm not familiar with them. I don't mind pulling them in as a provider. I may need help testing it later.
Please keep the scopes to the smallest subset they expose to get basic user logins.
Why don't you go ahead and submit the pr.

I will do that.  They follow a standard oauth 2.0 process flow as described here, https://developer.constantcontact.com/docs/authentication/oauth-2.0-server-flow.html.  I have based most of the code on several of the other providers as it creates follows the same process.

I added PR #193


fix: typo environment

typo.

Thanks!

:tada: This PR is included in version 4.1.14 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Blog: Provide anchor on article headings

What's your idea?
The headings featured on the individual blog post pages currently do not include a slug ID which would allow visitors to directly link and land on specific blog post headings for convenience.
Traditionally, a special symbol is shown adjacent to the heading to let the visitor copy the link to that heading.

Follow-up idea: Replace shorthand (possibly [TOC]) in blog post HTML with a table of contents containing all of the headings of the post.

TODO: Instead of using the HTML that GitHub automatically renders from our blog markdown, we can set up custom Markdown rendering. Looking into using a library, such as https://markdown-it.github.io/markdown-it/#MarkdownIt.

cc @jjoeldaniel, @jaasonw.


Add support for UnleashContext #29

The idea is to simplify common strategies via a shared unelash context. The context should be possible to provide just in time, or via a request bound probider.
Just in time example:
UnleashContext context = UnleashContext.builder().userId("user@mail.com").build();
unleash.isEnabled("someToggle", unleashContext);

The main benefit of just in time is that it is more explicit. It is also makes it easier when ThreadLocal is not bound to the current request.
Provider example:
UnleashContextProvider provider = new CustomUnleashContextProvider();
UnleashConfig config = UnleashConfig.builder().contextProvider(provider);
Unleash unleash = new DefaultUnleash(config):

// context will be provided to the strategy impl via the provided contextProvider.
unleash.isEnabled("someToggle");

this is still WIP!
todo:

[x] Implement UserContext
[x] Implement common strategies:

[x] UserWithIdStrategy
[x] GradualRolloutRandom Strategy
[x] GradualRolloutUserWithId Strategy
[x] GradualRolloutSessionId Strategy
[x] RemoteAddress Strategy

[ ] Add support for providing unleashContext (via config and inside UnleashDefault)
[ ] Do some testing before merging.

Coverage decreased (-1.9%) to 94.023% when pulling 0bf9da9fed674ace0b99e7be03ce08424236aa53 on unleash_context into 1c0bb2967861f9eea0eb82e3c6c8ff184f0136ec on master.

Coverage decreased (-1.7%) to 94.131% when pulling 98d2bfd4c4e2cc460abd44e83c51be37f46bcdf8 on unleash_context into 1c0bb2967861f9eea0eb82e3c6c8ff184f0136ec on master.

Coverage decreased (-2.5%) to 93.407% when pulling ab021c2b8e47d224ff9394c01464a4e303cd7e90 on unleash_context into 1c0bb2967861f9eea0eb82e3c6c8ff184f0136ec on master.

Coverage decreased (-1.7%) to 94.177% when pulling 87141d96cb2a4c2c00cb34442e41f5f423df7611 on unleash_context into 1c0bb2967861f9eea0eb82e3c6c8ff184f0136ec on master.

Coverage decreased (-1.3%) to 94.556% when pulling 0afd2fd8b3b2ce3ebd181aed2252cd1836c7b5b5 on unleash_context into 1c0bb2967861f9eea0eb82e3c6c8ff184f0136ec on master.

Coverage decreased (-1.3%) to 94.556% when pulling 66c21284c270ee0ba78675143134b5831d2c3e67 on unleash_context into 1c0bb2967861f9eea0eb82e3c6c8ff184f0136ec on master.

Question:
Should we override the DefaultUnleash constructor to also accept a "UnleashContextProvide" or should we add it as an option to the generic UnleashConfig object?

Coverage decreased (-1.3%) to 94.556% when pulling f7a5cadab07f986d0600970b2e380887d391d763 on unleash_context into 1c0bb2967861f9eea0eb82e3c6c8ff184f0136ec on master.

Coverage decreased (-1.2%) to 94.703% when pulling ef075f3bcbd8e5367c44e29fac88b4d5fb96c5d4 on unleash_context into 1c0bb2967861f9eea0eb82e3c6c8ff184f0136ec on master.

Coverage decreased (-1.2%) to 94.703% when pulling d153bb5420079e1e96547eaf73b190c278f99d08 on unleash_context into 1c0bb2967861f9eea0eb82e3c6c8ff184f0136ec on master.

Coverage decreased (-1.6%) to 94.262% when pulling ac9388bc0df63529b9629eb1e433ebc072e61c81 on unleash_context into 1c0bb2967861f9eea0eb82e3c6c8ff184f0136ec on master.

Coverage decreased (-0.6%) to 95.267% when pulling 5814a2d2baedebabddd56c6ec8e8198240d77376 on unleash_context into 1c0bb2967861f9eea0eb82e3c6c8ff184f0136ec on master.

Coverage decreased (-0.6%) to 95.267% when pulling 1f22237b7e867eac99eacef034d4425b2d6fb380 on unleash_context into 1c0bb2967861f9eea0eb82e3c6c8ff184f0136ec on master.

Coverage decreased (-0.6%) to 95.277% when pulling 1ba466f7d34d76a6ba3da65e466d86507799a573 on unleash_context into 1c0bb2967861f9eea0eb82e3c6c8ff184f0136ec on master.

Coverage decreased (-0.6%) to 95.277% when pulling fa2c824cfa06029289cfd16c1fda09805814985b on unleash_context into 1c0bb2967861f9eea0eb82e3c6c8ff184f0136ec on master.


After 5 minutes running, colab shuts down...

Everything's okay, but when i start using RDP, after 5-10 minutes my google colab notebook shuts down therefore  my Remote Desktop shuts down too, what can i do?
sorry for my english

Avoid disabling the Colab menu


rabitMQ secret engine configuration issue

Describe the bug
When trying to configure rabbitMQ secret engine using externalConfig values vault-configurer container crashes with following error message:
panic: interface conversion: interface {} is map[interface {}]interface {}, not []interface {}

goroutine 1 [running]:
github.com/banzaicloud/bank-vaults/pkg/vault.(*vault).configureSecretEngines(0xc4204c0d80, 0x0, 0x0)
	/go/src/github.com/banzaicloud/bank-vaults/pkg/vault/vault.go:509 +0x129d

Here's the corresponding section of config:
externalConfig:
  secrets:
    - type: rabbitmq
        description: internal-Rabbit
        configuration:
          config:
             connection:
                connection_url: "hostname:15672"
                username: user
                password: password
          roles:
            - name: rolename
               vhosts:
                 - /web:
                      write: "production_.*"
                      read: "production_.*"

To Reproduce
Deploy bank-vault using vault helm-chart and use described portion of config to setup rabbitMQ secret engine.
Version
vault-configurer:
    Image:         banzaicloud/bank-vaults:latest
    Image ID:      docker-pullable://banzaicloud/bank-vaults@sha256:d76d992a02708d326c1bb117a0f3067c0c96bb2d892557032969100152dd4d2e

@osnagovskyi
There are some mistakes in your configuration. According to the official RabbitMQ Secret engine docs the example configuration should looks like something similar to this:
  secrets:
    - type: rabbitmq
      description: internal-Rabbit
      configuration:
        config:
          - name: connection
            connection_uri: "http://hostname:15672"
            username: user
            password: password
        roles:
          - name: prod_role
            vhosts: '{"/web":{"write": "production_.*", "read": "production_.*"}}'

Please note that other than structural changes connection_url should be connection_uri with the proper http scheme.
If you still have troubles let us know!

@tarokkk thanks so much, worked just fine. Really impressed with such a quick turnaround!


Installing Recovery Provider on Server Core

Hi, I've found several sites discussing how to install the provider on Server 2012 R2 Hyper-V host running Server Core, but nothing seems to work for me (setupdr.exe crashes immediately when running). Are there certain system requirements other than OS version (e.g., .NET version)? Is there official documentation from Microsoft for how to install on Server Core?

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

ID: f09250c3-f338-5ed0-9b83-fbf66e1ed48d
Version Independent ID: 159b7261-c371-4d3d-dfb2-4f43727d8211
Content: Set up disaster recovery for on-premises Hyper-V VMs (without VMM) to Azure with Azure Site Recovery
Content Source: articles/site-recovery/hyper-v-azure-tutorial.md
Service: site-recovery
GitHub Login: @rayne-wiselman
Microsoft Alias: raynew

@shaunm001 Here is a the prerequisite requirements for Hyper-V Servers.
Are you getting any specific errors in your Administrator logs when the install fails?

@shaunm001 . Hi, support matrix is here - https://docs.microsoft.com/en-us/azure/site-recovery/hyper-v-azure-support-matrix#on-premises-servers

I’m good, it was crashing in our test environment, but tried it in production and installed successfully.
From: TravisCragg-MSFT<EMAIL_ADDRESS>Sent: Thursday, July 05, 2018 4:31 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Shaun Michelson<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Installing Recovery Provider on Server Core (#11277)
@shaunm001https://github.com/shaunm001 Here is a the prerequisite requirements for Hyper-V Servers.https://docs.microsoft.com/en-us/azure/site-recovery/hyper-v-prepare-on-premises-tutorial#review-requirements-and-prerequisites
Are you getting any specific errors in your Administrator logs when the install fails?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/11277#issuecomment-402843732, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Aacwq2JYrFZpqqmAjQcS1cksiUh0Q8WAks5uDnd6gaJpZM4VEXYj.

Good to hear. Feedback always welcome :-) #please-close

@shaunm001 We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


refactor(components): Rename Memory Component for Better Clarity and Consistency

Changes in Helper Components:
Component Renames:

Memory -> MessageRetriever:

Renamed class from MemoryComponent to MessageRetrieverComponent
Updated display name to "Message Retriever"
Changed icon to "message-square-text"
Updated component name to "MessageRetriever"

Import Updates:

Updated import statements in init.py
Renamed imported components:

MemoryComponent -> MessageRetrieverComponent

File Renames:

memory.py -> message_retriever.py

No Functional Changes:

Core functionality remains the same
Only naming and cosmetic changes applied
Component descriptions maintained

I believe that the changes are no longer necessary, due to the implementations that are already being made in the memory component, that said I will be closing this PR


If the connectionProfile was undefined, then return undefined

Fixes https://github.com/microsoft/azuredatastudio/issues/10773

Coverage decreased (-0.1%) to 34.464% when pulling 1729882b76f5490c0f1667feab20b29e3f4345e4 on amir/azureAuth/fix/10773 into a7110d89160e25cf914030dd467f3d85c0292692 on master.


Mismatch on the contacts count vs contacts shown on the page

In the index action of the contacts controller we have this to find the contacts:
@contacts =
  Contact.sent_by(current_subject).
          joins(:receiver).merge(Actor.alphabetic).
          merge(Actor.letter(params[:letter])).
          merge(Actor.name_search(params[:search])).
          related_by_param(params[:relation]).
          active
# and then ...
@contacts = @contacts.page(params[:page]).per(10)

that is we find the current page of all the contacts sent by the current_subject (filtered)
on the page to set the number of records found we have this:
current_subject.contact_subjects(:direction => :sent).count

which calls Actor.contact_actors which by default filters only the actors having Relation::Custom  or Relation::Public ties.
The method used in the controller still finds all the contacts which the subject may have refused (i.e. those with Relation::Reject) and thus they are shown on the page, moreover the difference in how the count is calculated may be a problem in the future (the two methods can diverge for good reasons).

Here: luca@c4a78de737f6688d03475023032efaad8db54fe0 I have fixed it by refactoring a bit the code:

added a not_rejected scope to Contact which skips all the reject relations,
extracted the global contacts scope in an @unfiltered_contacts variable (also excluding the rejected contacts),
used @unfiltered_contacts both on the page (to get its count) and in the controller (applying the filters to find the correct set)

this works with the only caveat that it isn't possible anymore to view the rejected contacts (not even explicitly filtering for them), but I'm not 100% sure I've got the right semantics for what should go on the page :)

Hi Luca, you are right. The rejected contacts should not be shown in that screen.
I like your approach of the not_rejected scope. I have modified it a bit to a positive scope, in the case that more "negative" relations show up in a future.
In the other hand, I think that the count should came from all positive contacts as well.
Please, tell us if commit ddd1cc3  fit your needs.
Thank you very much for your contributions.

I like your version a lot, and it makes a lot more sense: imagine an 'its a spam' relation :)
I'm missing a couple of things, related to the count on the page, which now is:
@contacts.count

doing a count overwrites the select("DISTINCT contacts.*") that is part of the positive scope, in my case it shows the wrong count as I have contacts with multiple ties (e.g. both a friend and an acquaintance). The only way I can think of to solve it is by adding the distinct clause to the count call like: @contacts.count("DISTINCT contacts.id"). It's awful but it works (at least on MySQL, I've not been able to test it on postgres)
counting on the @contacts will give the total contacts with the params filters applied, e.g. If I have 10 contacts and only 2 whose name starts with "A" and I'm filtering the contacts to see only those whose name starts with the letter "A" @contacts.count will return 2. I think the number on the page was meant ot give the total number of contacts (removed all filters). This is the reason I was splitting the scope in an unfiltered version (used for the count) and a filtered version.

Absolutely right, I fixed both issues in 53406c2466c7104eab6dd21569975e9442c6802a
Please, close the issue if you feel it Is ok?

cool, I'm closing it


Make builds of the examples

We should make at least web builds of the example scenes to show to potential users who want to try the demos without downloading unity or the full package.

Done:
https://etra-etra-games.itch.io/etras-starter-assets

https://etra-etra-games.itch.io/etras-starter-assets


cargo install does not honour alternate cargo installation directory

Hopefully I'm not duplicating an issue here, did a quick search. Jist is cargo install should put binaries next to cargo instead of a hard-coded directory.
This problem comes up because I'm using Bash for Windows with Rust for some Linux cross-compiling work and also happen to have my $HOME set to "/mnt/c/Users//" because that makes me happy. This config means that "~/.cargo" is filled with Linux binaries at the moment, and the Windows bins live elsewheres.
I installed using the standalone installer to "C:\Program Files\Rust stable MSVC 1.18\bin" and that's working fine. I managed to get rust install racer to build and be happy. However, it didn't place it in "Program Files" and instead went into "~/.cargo/bin".

Hm, shouldn't this be handled by cargo install --root option?
I think that installing alongside cargo binary by default won't work really well: Cargo can be installed to a system partition, so to put files there, cargo will need root/administrator privileges.

This '--root' magic will work nicely. RTFM strikes again!


misc(v2): inject theme in navbar classname

@yangshun Got it, I can work with that. I had noticed that navbar--light was a hard-coded classname in https://github.com/facebook/docusaurus/blob/master/packages/docusaurus-theme-classic/src/theme/Navbar/index.js where I seemed to make sense that the light vs dark flag would reflect the state of the current navbar being rendered. A CSS override from the html selector also does the trick.


Add information on force-ordering values to TSV output ref

Docs Build status updates of commit 9041c08:
:clock10: Full build: the average full build time is 6 min(s) 47 sec(s), this is based on the last 209 build(s) for this repository.

Docs Build status updates of commit 9041c08:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/format-output-azure-cli.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

docs-ref-conceptual/query-azure-cli.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

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.


merge trichome morphogenesis

Hi,
I think trichome morphogenesis should be merged with
trichome differentiation. According to the development
document, cells can have differentiation terms and not
development and morphogenesis terms. And since
trichome is a epidermal cell differentiation, I think the
morphogenesis should be merged with this.
http://www.ebi.ac.uk/~jclark/GOwebsite/development\_fo
lder/development.html
Cheers,
Suparna
Reported by: smundodi
Original Ticket: "geneontology/ontology-requests/1713":https://sourceforge.net/p/geneontology/ontology-requests/1713

thanks www.movied.org


-[JSONModel __importDictionary:withKeyMapper:validation:error:]

Please help me! I can't find why? Who can tell me possible reason. Thank you!

https://github.com/jsonmodel/jsonmodel/blob/master/CONTRIBUTING.md#general-helpsupport

This is a online bug. I find bug from Fabric.This bug crash sometimes.
json.txt
model.txt


index.js missing in package.json 'files' property

Because of that, jspm does not install the 'index.js' resulting in a broken 'angular-ui-bootstrap' package.
See: https://github.com/jspm/npm/issues/119

@wesleycho - any idea on when this one might be released?
/CC @eestein


using numpy for LED matrix

I am hoping to use the sense hat to display part of a full size image in response to pupil movement. I have modified the sense_hat to use numpy arrays. This is an order of magnitude faster than the python loop methods (alternatively it leaves a lot more processing capacity for the CPU to do other things). It also allows simpler 8x8 arrays of pixels which can be reasonably easily manipulated (i.e. rotation by arbitrary angles etc)

This change should be rebased, is project still maintained?

Hi. My sense hat is still happily running as part of radio alarm so not looked at this code for years. But I'm happy to bring it up to date.


upgraded dependencies

this is superseded by a new PR: https://github.com/ministryofjustice/hmpps-electronic-monitoring-datastore-api/pull/40


Question: Is the latest feature-hub (and history-service v3) compatible with react-router 6.4

Hi there,
in the demos react-router v6.3 ist used.
But we like to use v6.4 or greater because of the possibility to use scroll-restoration component.
Are we able to use React Router 6.4?
Thanks, Michael

Hi @mischah,
I can't see why it shouldn't. The history package, which should really be the only touchpoint between react-router and the history-service, is still at 5.3.0 in each. Could you try and report back? 😉


Changed error code with v1.2.0

Hello, we have a simple build test and it seems that the error code is changed from 1 to 139.
Can you check this is intended and have to change our test?
Thanks,
  ==> /opt/homebrew/opt/socket_vmnet/bin/socket_vmnet /dev/null 2>&1
  sh: line 1:  7219 Segmentation fault: 11  /opt/homebrew/opt/socket_vmnet/bin/socket_vmnet /dev/null 2>&1
  WARNING: --vmnet-gateway=IP should be explicitly specified to avoid conflicting with other applications
  ERROR| WARNING: Running without root. This is very unlikely to work: See README.md
  ERROR| bind: Address already in use
  ERROR| socket_bindlisten: Address already in use
  Error: socket_vmnet: failed
  An exception occurred within a child process:
    Minitest::Assertion: Expected: 1
    Actual: 139

This is not intentional, but I think you still want to modify your test because /dev/null is not a valid socket:
- /opt/homebrew/opt/socket_vmnet/bin/socket_vmnet /dev/null 2>&1
+ /opt/homebrew/opt/socket_vmnet/bin/socket_vmnet >/dev/null 2>&1

Please leave this issue open though; the program should not crash, even with invalid input.
Old version:
$ socket_vmnet /dev/null
WARNING: --vmnet-gateway=IP should be explicitly specified to avoid conflicting with other applications
WARNING: Running without root. This is very unlikely to work. See README.md .
bind: Address already in use
socket_bindlisten: Address already in use

New version:
$ ./socket_vmnet /dev/null
WARNING: --vmnet-gateway=IP should be explicitly specified to avoid conflicting with other applications
ERROR| WARNING: Running without root. This is very unlikely to work: See README.md
ERROR| bind: Address already in use
ERROR| socket_bindlisten: Address already in use
Segmentation fault: 11

There are other ways to get the program to crash:
$ ./socket_vmnet -p /dev/null
Segmentation fault: 11

This crashes with both the old and the new version. It too should produce a proper error message instead.

WARNING: --vmnet-gateway=IP should be explicitly specified to avoid conflicting with other applications
ERROR| WARNING: Running without root. This is very unlikely to work: See README.md

Also some feedback on the prefixes:

Why do warnings have a "WARNING: " prefix but errors use "ERROR| "?
And why does the first line have both prefixes: "ERROR| WARNING:"? What is it?

WARNING: --vmnet-gateway=IP should be explicitly specified to avoid conflicting with other applications
ERROR| WARNING: Running without root. This is very unlikely to work: See README.md

Also some feedback on the prefixes:

Why do warnings have a "WARNING: " prefix but errors use "ERROR| "?
And why does the second line have both prefixes: "ERROR| WARNING:"? What is it?

The first warning come from cli.c, which does not use the logger.
The second one is an error, and the error test contains "Warning:". We need to add a WARNF() macro.

@jandubois is this fixed by #100?

@jandubois is this fixed by #100?

I believe the crashes are fixed with #100, so I'm going to close it. Will you create a new issue for the confusing warning/error messages?

Will you create a new issue for the confusing warning/error messages?
Added #106 and #107


[pulsar-broker] Dispatch batch messages according consumer permits

Motivation
Right now, dispatching of batch messages is not predictable for a shared-subscription. Dispatcher doesn't consider number of batch messages in an entry which can cause higher number of message delivery than consumer receiver-queue.
For example:
If a topic has message-entries with 100 batched message in an entry.
If consumer has receiver queue=40 and has available-permits=20. then broker dispatches 20 entries with total 2000 (20*100) messages in it. because of that consumer's will high number of messages than expected and we can see available-permits in negative.
Modification

Find out average number of batch messages in an entry
Dispatch messages considering total batch-messages in an entry

Result
It will avoid dispatching higher number of messages than consumer's available-permit and will fix -ve available-permits.

@rdhabalia Looks like the same purpose as #6719

@codelipenghui
I saw  #6719 but it doesn't solve fundamental problem of dispatching.  #6719 tries to predict how many messages should be read by considering total-available permits and total batch messages. but it's still not controlling while dispatching messages.
for example: there are 10 consumers connected with 100 permits each and one batch message contains 100 messages. Now, broker reads 10 messages based on both factors but broker doesn't know number of batch-messages in each message so, it will dispatch all 10 messages to the consumer which has 100 permits which is the actual issue because broker dispatched 10*100=1K message to the consumer.

@rdhabalia Thanks, I now know this problem that you want to fix.

I. think it should open a doc issue to update docs for this PR. @Jennifer88huang , could you help open an doc issue for this PR?

Thanks @rdhabalia. overall lgtm. Would you please help on @codelipenghui 's last comments?

I. think it should open a doc issue to update docs for this PR. @Jennifer88huang , could you help open an doc issue for this PR?

@Huanli-Meng Thanks for informing me on this. We can add a label "doc-required" in the Labels list, and everyone can create issue for this.

@rdhabalia thanks for your work.

Any progress for this PR?
Could you please help add docs accordingly? Thanks

I will rebase this soon.

@rdhabalia could you please help add docs accordingly? Then I would like to help review, thanks.

@rdhabalia Needs to be rebased.

Sorry, I thought this was the other PR. Could you please edit the title of this one or https://github.com/apache/pulsar/pull/10417 to distinguish them a little more? Their titles are almost identical.

@Anonymitaet I believe that this is only a bugfix, so need to add particular documentation.

@eolivelli OK, many thanks for your guidance!

@merlimat do you want to take a look as well?

Closing and reopening this PR to get the fix for flaky test.

@rdhabalia Nice work!


Windows compatibility

Hi,
How would the dependencies be installed on a Windows machine (for example: apt-get install libsuitesparse-dev)?

I don't know. Perhaps Windows Subsystem for Linux or NuGet or conda or Cygwin. If you find out, we can update the instructions.

Marking as closed. Please re-open if this is still an issue.


Kubernetes-Anywhere Release

We don't have any release yet, given that the Kubernetes-Anywhere repo is stable. Can we tag a release now?
// cc @kubernetes/kubernetes-anywhere-admins
// cc @kubernetes/kubernetes-anywhere-maintainers
// cc @pdhamdhere @tusharnt @BaluDontu @divyenpatel @kerneltime

I don't think is kube-anywhere is stable. @divyenpatel has identified few issues w.r.t to new add on weave-net. I will ask divyen, to post more information on this.

I plan to tag a release tomorrow EOD. Let me know if any important issue needs to be addressed before release.

Done


Catching Exception from performRequest method

I'm trying to catch an exception that is thrown during the performRequest (the if statement with the conditional: (($error = curl_error($feed)) !== '')) but can't seem to get it to function correctly. Any ideas?
Here's what I'm trying to do:
$settings = array(
    'oauth_access_token' => OAUTH_ACCESS_TOKEN,
    'oauth_access_token_secret' => OAUTH_ACCESS_TOKEN_SECRET,
    'consumer_key' => CONSUMER_KEY,
    'consumer_secret' => CONSUMER_SECRET
);
for($abc = 0; $abc <= 2; $abc++){
	try{
		$twitterGet = new TwitterAPIExchange($settings);
		$twitterPost = new TwitterAPIExchange($settings);
	}catch(Exception $e){
		echo 'Exception Caught';
		if($abc === 2){
			exit("Failed to Connect to Twitter.");
		}
	}

	if(!isset($e) || $e==='' || $e===null){
		break;
	}
}

Doing the above I still get an exception coming through and giving me a Fatal Error and Exception notice when I lose an internet connection from the system running the script.

Have you tried CURL debug?
A nice explanation can be found here https://blog.kettle.io/debugging-curl-requests-in-php/ and that may narrow it down for you. I find that CURLOPT_SSL_VERIFYPEER is normally the biggest culprit.
Best of luck

Have you tried CURL debug?
A nice explanation can be found here https://blog.kettle.io/debugging-curl-requests-in-php/ and that may narrow it down for you. I find that CURLOPT_SSL_VERIFYPEER is normally the biggest culprit.
Best of luck

I don't actually have a connection issue.... Things work fine. What I'm trying to do is catch the thrown exception, to output my own error message when in the rare circumstance that my system temporarily loses an internet connection. Right now I'm purposefully disabling my connection.
Here in this code the API throws an exception:
 if (($error = curl_error($feed)) !== '')
        {
            curl_close($feed);
            throw new \Exception($error);
        }

What I'm trying to do is customize the output that the exception displays by catching it with this:
for($abc = 0; $abc <= 2; $abc++){
	try{
		$twitterGet = new TwitterAPIExchange($settings);
		$twitterPost = new TwitterAPIExchange($settings);
	}catch(Exception $e){
		echo 'Exception Caught';
		if($abc === 2){
			exit("Failed to Connect to Twitter.");
		}
	}

	if(!isset($e) || $e==='' || $e===null){
		break;
	}
}

The issue is the above is NOT catching the exception.

@AndrewMcCarrick ,
This is a bit outside my comfort zone but hopefully I can't lead you astray twice! You may want to look at curl_getinfo() and checking the header as this should tell give you a 404 or another timeout error...
e.g.
curl_exec($ch);
$result = curl_getinfo($ch, CURLINFO_HTTP_CODE);
if ($result != 200) {
die('Request failed: HTTP status code: ' . $result);
}


catalyst is not in supported platform list of upload_symbols_to_bugsnag

Describe the bug
When running upload_symbols_to_bugsnag for catalyst build, I got this error:
Action 'upload_symbols_to_bugsnag' isn't known to support operating system 'catalyst'.

Upon checking this file, I see that it only support [:ios, :mac, :tvos].
Please add catalyst to this as well.
Thanks
Steps to reproduce
Run upload_symbols_to_bugsnag for catalyst

Hi @son-nguyen-goodnotes
Thank you for reaching out and raising this issue.
I've added this to our backlog for further investigation. We will be sure to fix this when priorities allow.

Hi @son-nguyen-goodnotes
This has now been fixed in the v2.3.1 release of bugsnag-dsym-upload: https://github.com/bugsnag/bugsnag-dsym-upload/releases/tag/v2.3.1
Thanks again for raising this with us. Please let us know if you continue to have any issues after upgrading to the latest version.


CON-4911: fixed problems in auto-category-resolver

remote categories are now stored before categories
are resolved, because auto category resolver needs
the remote categories
all categories are resolved in autocategoryresolver
not just the ones under the last main node

Coverage increased (+0.006%) to 37.626% when pulling d6350e8dfec080d457992947c8d63b4dba226505 on CON-4911 into 52744bd8b2f3ae998f5e869021f4ab36b40b3873 on master.

Coverage increased (+0.006%) to 37.626% when pulling 3e0cf1cf8df4ca7f2c93960dc971c34ec7de4272 on CON-4911 into 52744bd8b2f3ae998f5e869021f4ab36b40b3873 on master.


"Build a Windows Presentation Foundation (WPF) Blazor app" does not work

Can't follow the tutorial when it comes to "MainWindow.xaml" replace part.
Severity	Code	Description	Project	File	Line	Suppression State
Error	XLS0418	Assembly 'Microsoft.AspNetCore.Components.WebView.Wpf' was not found. Verify that you are not missing an assembly reference. Also, verify that your project and all referenced assemblies have been built.	WpfBlazor	C:\Users\xxxxx\source\repos\WpfBlazor\WpfBlazor\MainWindow.xaml	6
Severity	Code	Description	Project	File	Line	Suppression State
Error	XDG0008	The name "BlazorWebView" does not exist in the namespace "clr-namespace:Microsoft.AspNetCore.Components.WebView.Wpf;assembly=Microsoft.AspNetCore.Components.WebView.Wpf".	WpfBlazor	C:\Users\xxxxx\source\repos\WpfBlazor\WpfBlazor\MainWindow.xaml	11

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

ID: 94872f14-0dce-0cf3-0d25-88f23ca8d8ef
Version Independent ID: 94872f14-0dce-0cf3-0d25-88f23ca8d8ef
Content: Build a Windows Presentation Foundation (WPF) Blazor app
Content Source: aspnetcore/blazor/hybrid/tutorials/wpf.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @WinterIsComming861047 ... Did you add the Microsoft.AspNetCore.Components.WebView.Wpf package to the app?

I'm going to close as answered, but you can leave additional information if you do have the package referenced in the app and are still hitting that error.

Hi, thanks for your reply.
I thought you were talking about the part of installing Nuget package.
I just re do the tutorial again and post it to the github, maybe you can
point out which part I was doing wrong.
WinterIsComming861047/WPFBlazor (github.com)
https://github.com/WinterIsComming861047/WPFBlazor
Luke Latham @.***> 於 2023年7月7日 週五 下午7:47寫道：

Closed #29725 https://github.com/dotnet/AspNetCore.Docs/issues/29725 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/dotnet/AspNetCore.Docs/issues/29725#event-9754219795,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHEXCUQGBDYVVMFKCVU4WTLXO7ZL7ANCNFSM6AAAAAA2APZGWE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hello @WinterIsComming861047 ... I'm off for the weekend, so I'll take a look on Monday.

Sure, any help is appreciated. Thanks
Luke Latham @.***> 於 2023年7月14日 週五 下午9:20寫道：

Hello @WinterIsComming861047 https://github.com/WinterIsComming861047
... I'm off for the weekend, so I'll take a look on Monday.
—
Reply to this email directly, view it on GitHub
https://github.com/dotnet/AspNetCore.Docs/issues/29725#issuecomment-1635858106,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHEXCUUAO3ZNDA7FF4CKPDDXQFBTXANCNFSM6AAAAAA2APZGWE
.
You are receiving this because you were mentioned.Message ID:
@.***>

I had a chance to look before taking off for the weekend:

You (probably) didn't install the .NET desktop development workload.
The app isn't named correctly. It should be WpfBlazor, not WPFBlazor. It (probably) caused a namespace problem.
The app.css file goes into a wwwroot folder.
You didn't follow the step for updating the MainWindow.xaml code.

I just walked the steps, and the guidance still works ...

For some reason it does not work for me when using .net6 although everything is correct including the namespaces etc. When switching to .net7 everything works fine. When trying to install the Microsoft.AspNetCore.Components.WebView.Wpf nuGet package, I also see that it installs more other packages than when installing it with .net6. Is that explaiable?

@FlorentVllasa ... Open an issue for the product unit here ...
https://github.com/dotnet/aspnetcore/issues


WIP Replace kubeservice

Fixes #490 and an item of #139 by replacing the use of kubeservice with a bit of Go code. This relies on code from #486 at the minute, so will need to be rebased later.

Taking this off the radar.


add CoqJumpToError command

Thanks, this looks great. I want to test it out a little and might make a couple minor edits, but I should be able to merge tomorrow.

Do you have a reason for jumping to the end of the error region instead of the start? If not, I think I'd prefer using the start since that seems more intuitive to me.

I thought the end of the error region would be closer to the part that I should edit in order to fix the error. What do you think?

I guess it depends on the error. In most cases I can think of the error region is just a couple words long at most anyway so it should be pretty quick to navigate to the right spot. So let's switch to the start of the region and we can always add an option or another command later if necessary.

all right


adding dynamic meta tags

Before v0.6 I was using "yasinuslu:blaze-meta" package for adding meta tags. But unfortunately I can not after this release.Do you know how can I deal with this problem?

Can you create a repo that demonstrate that so I will be able to clone and test this?
Thank you

Just tested it and it works fine for me..
@catsoft Looks like your problem is not angular-meteor related


Enable pagination on all sensu API list endpoints

as sensu clusters keep increasing in size, the sensu API is struggling in handling un-paginated queries.
Unfortunately, not all endpoint support paginations. At the moment /silenced, /stashes and /clients support the "limit" and "offset" querystrings. However /results does not.
all the listing endpoints should support pagination

Thanks for the issue @davideauee 👍

Closed by #1853, Sensu Core 1.4.0 will have this 👍


Waiting for Elasticsearch to start...

Hi sir,
 then i install the server by  "sudo python setup.py".  it always blocking in "Waiting for Elasticsearch to start..." as below.
 i do not know how to do the next? how to start Elasticsearch? it has been installed?
 thanks

Elasticsearch already installed
Creating elasticsearch credentials
Kibana already installed
Importing Kibana Index Patterns
Importing /root/Documents/github/SweetSecurity/kibana/patterns/logstash.json
Importing /root/Documents/github/SweetSecurity/kibana/patterns/sweet_security_alerts.json
Importing /root/Documents/github/SweetSecurity/kibana/patterns/sweet_security.json
Importing /root/Documents/github/SweetSecurity/kibana/patterns/tardis.json
Importing Dashboards
Importing /root/Documents/github/SweetSecurity/kibana/dashboards/geoip.json
Importing /root/Documents/github/SweetSecurity/kibana/dashboards/SweetSecurityAlerts.json
Importing /root/Documents/github/SweetSecurity/kibana/dashboards/logstash.json
Importing /root/Documents/github/SweetSecurity/kibana/dashboards/sweet_security.json
Importing /root/Documents/github/SweetSecurity/kibana/dashboards/baselines.json
Waiting for Elasticsearch to start...
Waiting for Elasticsearch to start...

There appears to be an error in communicating with Elasticsearch to update the kibana instance. If there is a specific exception being thrown, the current installer does not log it.  You can modify the installer to print out the exception by performing the following.

Edit install/kibana.py
Modify line 99 from
 except:
to
except Exception,e:
Add a line in the exception block below on line 100:
print(e)

Paste the results here.

Are you getting issues regarding the curl of localhost:9200? If so, you can kill the install, run sudo service elasticsearch start, check the status and review the error code. I was having like issues and it was complaining java-v8 was not installed. Purged all old java and installed 8. Re-ran the install and everything is working fine now.

I am having the same issue and the exception for me is:
TransportError(404, u'document_missing_exception', u'[config][5.5.1]: document missing')
the elasticsearch service is running but the installer does not see it.

Same issue, my errors just show unable to connect over and over saying connection refused.
Restarting Apache
Installing Logstash
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0curl: (7) Failed to connect to localhost port 9201: Connection refused
Error: Waiting for Elasticsearch to start...try 1 of 10
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0curl: (7) Failed to connect to localhost port 9201: Connection refused
Error: Waiting for Elasticsearch to start...try 2 of 10
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0curl: (7) Failed to connect to localhost port 9201: Connection refused
Error: Waiting for Elasticsearch to start...try 3 of 10

I had a similar problem, but discovered it was the password I used on my elastic stack.  I built the web half in a VM and used a non-alphanumeric in the password, but the Raspberry Pi was set to En-GB keyboard, so when I typed in the password on the Raspberry Pi it was submitting the wrong character.

In case the setup script wait on elastic search run the following command:
/etic/init.d/elasticsearch status
it will most probably say that the service did not start correctly, and most probably it will be due to the Java Runtime Env.
run the following commands and it will fix your issue in case its the java:
apt-get update
apt-get install default-jre

Having same problem with error
NotFoundError(404, u'document_missing_exception', u'[config][5.5.1]: document missing')


updated Contributores.md

I updated my name along with the github id in the contributors.md file .

Hey there,
Thanks for your PR & contribution :wave:!
Don't forget to join follow me, join our organization, & follow our organization to stay updated with the latest developments 💻
Thank you, and have a good day ahead 😊

Before you go, I have a little surprise for you :)
@holopin-bot @# do your thing 🤖


Separating set/get of update state.

@aaustin @derrickstaten @qinweigong @Sarkar
Previously, this was being updated when a get was called, which feels somewhat unnatural.
Additionally, in the case of an install or open "failing," the update state shouldn't be updated.
UpdateState should only be set when the open/install call completes successfully.

So far so good!
One comment: I think looking at an entire day is super unnecessary. I changed the code on my end so we're dealing with increments of ten seconds instead of a day.
int appCreationDay = (int)([[documentsDirAttributes fileCreationDate] timeIntervalSince1970]/(10));
int appModificationDay = (int)([[bundleAttributes fileModificationDate] timeIntervalSince1970]/(10));

These should probably be renamed if we go this route. Thoughts?

:+1:

Regarding the whole day comparison, three things:

If we were going to make that change, I would prefer it be a separate PR, as it isn't really related to the changes being made here.

You're actually making additional operations here that the device doesn't care about. While as a human, comparing small numbers is simpler, the device doesn't care -- int comparisons are the same between any size ints, so the actual / 10 operation is an extra step that isn't necessary.

Dividing by 10 would actually break the functionality here. Say the first value is 10000001 and the second is 10000000. Those values divided by 10 (as ints) are equal. Instead, if you were to do modulus comparison, you'd only be comparing the last 10 seconds, but then 10000001 and 20000001 would be equal.

So, I propose we not make that change :wink:

Actually, that does bring up an interesting point, though. Why aren't we just comparing the date equality here? There's no need to change from NSDate to int.

so for a future PR, we should move to a new system.


OPENAI_API_KEY needs to be updated. However should we put this on github? or just locally?

https://github.com/TRAILab/TRAILBot/blob/dda6b701e4512046de2bb0ab08a2c4c9d7fd0187/alias/README.md?plain=1#L39

Replace this line with a generic <open_api_key>

Solved.


Demo for YoloV8

Would be good if this repo had a demo for how to implement YOLOv8.
I've started with modifying the code from the YOLOv7 demo, but am hitting some issues.
from ultralytics import YOLO
...
 # self.model = torch.hub.load("WongKinYiu/yolov7", "custom", model_path)
self.model = YOLO(model_path)
Here's how the model can be loaded with the new ultralytics repo compared to the YOLOv7 one.
However, at the step of the code that converts the bounding boxes to norfair_detections, particularly here:
detections_as_xyxy = yolo_detections.xyxy[0]
I'm having some issues extracting the xyxy from the YOLOv8 results object to a valid format.
Any help would be greatly appreciated!

Hello @kenifxyz, a YOLOv8 demo would indeed be beneficial. For now, I'm providing you with code to make Norfair works for your model.
As you pointed out, it's necessary to modify the function that transforms YOLO detections into Norfair detections. The code below adjusts this function to accommodate the new model.
import logging
from typing import List

import numpy as np
import torch
from ultralytics import YOLO

from norfair import Detection, Tracker, Video, draw_boxes

# Disable logging from the ultralytics package
logger_name = "ultralytics"
logging.getLogger(logger_name).setLevel(logging.CRITICAL)

def yolov8_detections_to_norfair_detections(
    yolo_detections: torch.tensor, track_points: str = "bbox"  # bbox or centroid
) -> List[Detection]:
    """convert detections_as_xywh to norfair detections"""
    norfair_detections: List[Detection] = []

    if track_points == "centroid":
        for object in yolo_detections:
            detections_as_xywh = object.boxes.xywh
            scores = object.boxes.conf
            classes = object.boxes.cls
            for detection_as_xywh, score, class_id in zip(detections_as_xywh, scores, classes):
                centroid = np.array([detection_as_xywh[0].item(), detection_as_xywh[1].item()])
                scores = np.array([score.item()])
                norfair_detections.append(
                    Detection(points=centroid, scores=scores, label=int(class_id))
                )
    elif track_points == "bbox":
        for object in yolo_detections:
            detections_as_xyxy = object.boxes.xyxy
            scores = object.boxes.conf
            classes = object.boxes.cls
            for detection_as_xyxy, score, class_id in zip(detections_as_xyxy, scores, classes):
                bbox = np.array(
                    [
                        [detection_as_xyxy[0].item(), detection_as_xyxy[1].item()],
                        [detection_as_xyxy[2].item(), detection_as_xyxy[3].item()],
                    ]
                )
                scores = np.array([score, score])
                norfair_detections.append(
                    Detection(points=bbox, scores=scores, label=int(class_id))
                )

    return norfair_detections

video = Video(input_path=<path-to-your-video>)

tracker = Tracker(distance_function="iou", distance_threshold=0.7, past_detections_length=0)

model = YOLO("models/yolov8n.pt")

for frame in video:
    yolo_detections = model(frame)
    detections = yolov8_detections_to_norfair_detections(yolo_detections)
    tracked_objects = tracker.update(detections=detections)
    frame = draw_boxes(frame, tracked_objects)
    video.write(frame)

If you have any further questions, please don't hesitate to ask.

Hey @DiegoFernandezC, that works perfectly! Thanks so much!

If I may add something useful to this discussion, I found this snippet of code super handy but kind of sluggish. Therefore, I optimized a bit the function yolov8_detections_to_norfair_detections with a few (tiny) changes. Just for reference, it went from around 2fps rate to more than 5fps. Again thanks for your support @DiegoFernandezC and hope someone else will find this code helpful.
`
def yolov8_detections_to_norfair_detections(
yolo_detections: torch.tensor, track_points: str = "bbox"  # bbox or centroid
) -> List[Detection]:
"""convert yolo_detections to Norfair's detections"""
norfair_detections: np.ndarray = np.array([])
if track_points == "centroid":
    for object in yolo_detections:
        detections_as_xywh = object.boxes.xywh
        scores = object.boxes.conf
        classes = object.boxes.cls
        for detection_as_xywh, score, class_id in zip(detections_as_xywh, scores, classes):
            centroid = np.array([detection_as_xywh[0].item(), detection_as_xywh[1].item()])
            scores = np.array([score.item()])
            np.append(norfair_detections, Detection(points=centroid, scores=scores, label=int(class_id))
            )

elif track_points == "bbox":
    for object in yolo_detections:
        detections_as_xyxy = object.boxes.xyxy
        scores = object.boxes.conf
        classes = object.boxes.cls
        for detection_as_xyxy, score, class_id in zip(detections_as_xyxy, scores, classes):
            bbox = np.array(
                [
                    [detection_as_xyxy[0].item(), detection_as_xyxy[1].item()],
                    [detection_as_xyxy[2].item(), detection_as_xyxy[3].item()],
                ]
            )
            scores = np.array([score.item()])
            np.append(norfair_detections, Detection(points=bbox, scores=scores, label=int(class_id)))

return list(norfair_detections)

`


Rafael vanat

Commit changes

Hey brow. How you doing?
Hope you doing fine!!
Just to clarify: didn't you wanted to pull request this? Did you mistakenly close this pull request...
I didn't understand whats happened here! :P

yes i did, but then i realized that it was a mistake to do that. So a I closed this pull. No worries =p


New package: FFTResampling v0.2.0

Registering package: FFTResampling
Repository: https://github.com/roflmaostc/FFTResampling.jl
Created by: @roflmaostc
Version: v0.2.0
Commit: 4a992fe8dc423824edbfd4577032552805661ce6
Reviewed by: @roflmaostc
Reference: https://github.com/roflmaostc/FFTResampling.jl/commit/4a992fe8dc423824edbfd4577032552805661ce6#commitcomment-46042670
Release notes:

Rename package from FFTInterpolations.jl to FFTResampling.jl

[noblock]
I renamed the package. Initially it was called FFTInterpolations.jl but this was misleading since we also do downsampling.
Therefore the better name FFTResampling.jl.

[noblock]
I renamed the package. Initially it was called FFTInterpolations.jl but this was misleading since we also do downsampling.
Therefore the better name FFTResampling.jl.


Reworking range processors to short-circuit when a suitable parsed range is found

latest and latest.integration processing have been moved into IvyProcessor such that all range string processors should now be self-contained
This fixes #257

Codecov Report
Attention: Patch coverage is 93.33333% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 92.89%. Comparing base (7640bcf) to head (3fd9870).
Report is 20 commits behind head on main.

Files
Patch %
Lines

...g/semver4j/internal/range/processor/Processor.java
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##               main     #258      +/-   ##
============================================
+ Coverage     92.17%   92.89%   +0.72%
- Complexity      268      272       +4
============================================
  Files            21       22       +1
  Lines           639      676      +37
  Branches        103      105       +2
============================================
+ Hits            589      628      +39
  Misses           17       17
+ Partials         33       31       -2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


IPv6 Static over parent v4 PPPoE link

Certain ISP's allow static v6 address assignments even though dhcp6 works too. The only issue with using statics is that the setup for apinger gateway attempts to use the real interface, i.e. igb0, this does not work as the traffic is going via the pppoe link.
This patch addresses that and sets the gateway to correctly use the pppoe link.
This is an optional setting and an additional option has been added to the interfaces as a checkbox to enable/disable this option.

@marjohn56 @fichtner wouldn't it be better to move the current dhcp6usev4iface setting (and change its help text)? It kind of suggests doing the same and there are only a few cases where $realv6iface is set to true.
All in interface_bring_down()
https://github.com/opnsense/core/blob/17.7.10/src/etc/inc/interfaces.inc#L1131
https://github.com/opnsense/core/blob/17.7.10/src/etc/inc/interfaces.inc#L1140
https://github.com/opnsense/core/blob/17.7.10/src/etc/inc/interfaces.inc#L1150

As a new opnsense user whatever you think is best. But as the pppoe link only appears when static IPv6 is selected and use v4 link only appears when dhcp is selected.
I'm not confused, but then I know what it's doing. :)

@marjohn56 I understand, no problem, our aim is to simplify the codebase, hence we try to look a step further every time. Let me propose a fix for you to test.

What is not wanted is to break a working system. It's an oddity where a user can use static addresses but is using PPPoE to sign in. In our case, IPv6 works perfectly well when set to static, the only thing that does not work is the gateway pinger, it gets set to the physical address where it needs to be routed over pppe0.
As I said, it's an oddity which probably only comes to light when you have an unusual scenario of the type we can use with our ISP.

I understand, but the other setting likely was there for a similar reason.... Just trying to prevent adding more and more features that do (almost) the same.

@marjohn56 this https://github.com/opnsense/core/commit/f722081b8779ac8d183f5b39efd8dd7c80b22ea0 should do the trick, we still have to do some work in the interface area to ditch unneeded code, but that's for 2018

Cheers, I'll check it out shortly, Christmas day and all that. :)

nu rush, Christmas indeed :)

OK, that appears to work. Threw me a couple of times as I had to disable and re-enable the wan and re-save the gateway info, but it's all up and running now.
Nice one... now onto the next problem. :)

@AdSchellevis @marjohn56 looks good too and thanks for testing. closing this one. :)

Tested here and working well. Nice job


Fully Integration of diffusive CNX to v02 framework

A new diffusive model kernel called CNX is developed and integrated to v02 code base. It is tested on Falls Lake domain with reasonable outputs.
Additions
diffusive_cnx.f90
pydiffusive_cnx.f90
make
pydiffusive_cnx.h
diffusive_cnx.pyx
Changes
setup.py
compiler.sh
fortran_wrappers.pxd
compute.py
Testing
> python3 -m nwm_routing -V2 -f ../../test/input/yaml/Florence_Benchmark_diff.yaml

In the yaml file, compute kernel is set to be diffusive_cnx.

Awesome work! Before we merge can you please run four simulations on the same domain and plot the results: MC, TU, cnt, cnx. It is useful to maintain a record of comparative model behavior as we continue to develop and refine the diffusive kernels.

No problem. I will plot them on Florence domain.


Spinner inside input

v4.2 features spinner. They can be embedded inside a button: https://getbootstrap.com/docs/4.3/components/spinners/#buttons
Would be nice to be able to also embed a spinner inside an input.
Sometimes you have fields that perform async actions, a spinner inside the input would give feedback to the user (something is pending).
Example: https://github.com/tkrotoff/bootstrap-spinner

Inputs cannot have elements within them, so we're outta luck on that. For now, no plans to implement this or a variation of it. We'll keep it in mind though!

@mdo but maybe it's possible to emulate this feature using technique similar to input groups? If text appending is possible, why can't we append a spinner?

Lemme see if I can come up with something...

any progress on this one?

Bulma allows adding a spinner to form controls (input, textarea, and select) by adding an is-loading class to the control: https://bulma.io/documentation/form/input/#states. It would be nice if Bootstrap supported something similar.

Came across this issue as we are also working with async validators.
It's basically 3rd stage of a lifecycle where the form field is neither valid or invalid but "pending" (Using Angular form field terminology) / validating. Could even be good enough to have a non-spinning spinner in text-secondary color.
The Angular way would have to be to create a custom directive that appends a spinner within an input group everywhere where there this async validation.

Yes, giving feedback to the user based on certain async functions would be great.

There's a draft PR from @mdo for this: https://github.com/twbs/bootstrap/pull/31667
Would love to see it added too :-)

Probably this is the most easiest way to create spinner:


Url Helper Service should not discard question marks when parsing hash fragment

In a sense, this might be a duplicate of #415. I've created this issue because a lone PR popped up: #603.
Looking at the PR I see that indeed, for reasons unknown to me, the url helper very deliberately removes question marks from the hash fragment.

Question marks are allowed so I fail to see why we'd strip them (plus #415 is caused by this)
It was put there deliberately, so perhaps the commit logs can tell us why it's there
Removing the code is a breaking change, so even if it's for the better, it might still break users

Perhaps @danyo1399 can elaborate what triggered them to write the PR in the first place?

Hi there.
The problem i am facing that triggered the PR is that silent refresh errors get swallowed and not raised as errors because the error response is chopped to after the ?. The silent refresh will eventually timeout which generates a different error which i then use to trigger a redirect login but that happens only after 20 seconds.
When i manually manipulate the hash removing the questionmark, the silent refresh error is returned immediately to the caller and i can call the login implicit flow straight away.
My web app is setup that on navigation, check if we have a valid token and if not attempt silent login, and if that fails perform implicit flow login.
When the user first loads the site, it tries the do a silent login. If the silent login fails, the error is swallowed due to the bug and after 20 seconds it attempts the implicit flow login. This unnecessarily causes a min 20 second delay from when the user opens the website to the point they see the login screen.
Example error below. Hope it helps
Regards Dan
#error=login_required&error_description=AADSTS50058:+A+silent+sign-in+request+was+sent+but+no+user+is+signed+in.%0D%0ATrace+ID:+acd8ea14-86d6-4adf-b293-hiddenvalue%0D%0ACorrelation+ID:+e8273357-2e1c-462f-baee-hiddenvalue%0D%0ATimestamp:+2019-08-13+23:00:48Z&error_uri=https://login.microsoftonline.com/error?code=50058&state=hiddenvalue

Is there anything more you would like me to do regarding this?

should work with version 9.


Search in Ascope Docs

It would be nice if there was a search bar when visiting https://docs.advantagescope.org.

Already implemented and coming with the next release!


Switching cameras during recording produces video delay and sync issue

When you recording video and switching camera source during recording, audio is starting to go ahead from video

I'm experiencing this as well. Switching cameras (front camera to back camera or vice versa) causes the video and audio to go out-of-sync (and sometimes produces popping artifacts in the audio), with the audio getting ahead of the video. I haven't been able to find a solution for this.

Did anyone ever find a solution to this?

I'm experiencing this as well.
Also i found a solution:
before you switch the capturedevice, just pause the recording.
Like this:

(IBAction)changeCameraClicked:(UIButton *)sender
{
if ([self.recorder isRecording])
{
[self.recorder pause];
[self.recorder switchCaptureDevices];
[self.recorder record];
}else
{
[self.recorder switchCaptureDevices];
}
}

Thank you!


Events not properly filtered when using multiple exclusive filters that compare a custom attribute

Expected Behavior

When using multiple exclusive filters that compare a custom attributes for client and checks the even should be filtered and not sent to the handler for processing.  The first being a filter that check a client attribute and the second being a filter that checks a check attribute.
Current Behavior

The check is still handled even though it has the custom attribute the filter is looking for.
Possible Solution

Steps to Reproduce (for bugs)

Create a filter that will handle events if the custom attribute is set to false ( page all the time )

// Note that negate is true, making this an exclusive filter;
// if the evaluation returns false, the event will be handled.
// AKA: If business_hour_paging == false then handle the event
{
  "filters": {
    "all_hours": {
      "negate": true,
      "attributes": {
        "client": {
          "business_hour_paging":"true"
        }
      }
    }
  }
}

Create a filter that will not handle events from a node during business hours

// Note that negate is true, making this an exclusive filter;
// if the evaluation returns false, the event will be handled.
// AKA: If business_hour_paging == true _AND_
// the hours are between 9AM - 6PM then handle the event
{
  "filters": {
    "business_hours_client": {
      "negate": true,
      "attributes": {
        "client": {
          "business_hour_paging": "false"
        }
      },
      "when": {
        "days": {
          "monday": [
            {
              "begin": "06:00 PM",
              "end": "09:00 AM"
            }
          ],
          "tuesday": [
            {
              "begin": "06:00 PM",
              "end": "09:00 AM"
            }
          ]
        }
      }
    }
  }
}

Create a filter that will not handle from a check during business hours

{
  "filters": {
    "business_hours_check": {
      "negate": true,
      "attributes": {
        "check": {
          "business_hour_paging": "false"
        }
      },
      "when": {
        "days": {
          "monday": [
            {
              "begin": "06:00 PM",
              "end": "09:00 AM"
            }
          ],
          "tuesday": [
            {
              "begin": "06:00 PM",
              "end": "09:00 AM"
            }
          ]
        }
      }
    }
  }
}

Add the list of filters to a handler

{
  "handlers": {
    "pagerduty": {
      "command": "/opt/sensu/embedded/bin/handler-pagerduty.rb -j pagerduty",
      "type": "pipe",
      "filters": [
        "all_hours",
        "business_hours_client",
        "business_hours_check"
      ],
      "severities": [
        "ok",
        "critical",
        "unknown"
      ],
      "mutator": "ttl_warn_to_crit"
    }
  },
  "pagerduty": {
    "api_key": "FOOBARRRRR"

  }
}

Trigger check event failure, watch event being handled

Context

Trying to provide an easy method or setting a node or check to not alert during business hours.
Currently we create subdues at the check level to stop them from alarming, but would like a cleaner method that matches how you would stop an entire node from alarming
Your Environment

Sensu version used: sensu-1.1.2-1.el7.x86_64
Active Sensu extensions (e.g. WizardVan 0.1.0): N/A ??
Operating System and version (e.g. Ubuntu 14.04): CentOS Linux release 7.4.1708 (Core)
Transport and runtime versions (e.g. RabbitMQ 3.6.2 on Erlang 18.2): rabbitmq-server-3.6.14-1.el7.noarch

It looks like negate does not play w/ filter time windows: https://github.com/sensu/sensu/blob/44f58ebdc6881161b34c67feaff572650d0b0ded/lib/sensu/server/filter.rb#L92-L96 This could be the cause 🤔

Odd thing is if I only use the first two filters events are filtered and not alerted on as expected.  But when I add the third filter the filtered events looking for the custom check attribute do not get filtered


[Preheat] create preheat job failed

Bug report:
mediaType in manifest should be 'application/vnd.docker.distribution.manifest.v2+json' not 'application/vnd.oci.image.index.v1+json'

Expected behavior:

How to reproduce it:

Environment:

Dragonfly version:
OS:
Kernel (e.g. uname -a):
Others:

do you have the same problem right now ?
WTF, i got it;


docs: add documentation on fixed content

https://github.com/driftyco/ionic/issues/5987

The feature always exist? I can't find it.

@movrack Yes, ion-fixed was converted into an attribute. See the following test for an example of it being used: https://github.com/driftyco/ionic/blob/master/src/components/nav/test/basic/app-module.ts#L390-L391
And a screenshot of that test (the fixed button and large grey box are fixed):

It party worked. The content stays in .scroll-content and fixed content remains empty and I am using ion-segment and the segment where I don't have google map but an ion-card does not show. The content is hidden.

I can't use ion-fixed right now, is it possible to put content into container with class .fixed-content ? how ?


chore(main): release veto 0.3.32

:robot: I have created a release beep boop
0.3.32 (2024-05-23)
Bug Fixes

veto: refactor exports (e079b27)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/fuf-stack/uniform/releases/tag/veto-v0.3.32 :sunflower:


implement FiniteFirehoseFactory in InlineFirehose

Fixes #8673.

Description
Implementation of FiniteFirehoseFactory to support index_parallel tasks with inline data ingestion.
The implementation always returns 1 split.

This PR has:

[x] been self-reviewed.

Key changed/added classes in this PR

InlineFirehoseFactory.java

@ccaominh @jihoonson the approach mentioned on the ticket is implemented here.
I have tested it with basic inline ingestion tasks & will double check the behavior in a few more scenarios.

@aditya-r-m thank you for your contribution! I have a couple of comments.

InlineFirehoseFactory should override boolean isSplittable() and return false because it's not splittable.
Would you please add some unit tests? Those test should verify InlineFirehoseFactory implements FiniteFirehoseFactory and isSplittable returns false. InlineFirehoseFactory is a good place for them.

@jihoonson thank you for your guidance.
I will update the PR with the both of the points asap.

@jihoonson implemented the suggested updates.