Support upgrading Gradle itself

Is there an existing issue for this?

[X] I have searched the existing issues

Feature description
Gradle projects store which version of Gradle to use in the /gradle/wrapper/gradle-wrapper.properties file, and frequently forget to upgrade it.
A gradle-wrapper.properties may have various entries in any order but only the distributionUrl entry is relevant to the versioning and looks something like this:
distributionUrl=https\://services.gradle.org/distributions/gradle-8.1-bin.zip

To upgrade the gradle version, you only need to update the string in between gradle- and -bin. For example to upgrade from Gradle 8.1 to 9.0.1 as above, the line needs to be changed to:
distributionUrl=https\://services.gradle.org/distributions/gradle-9.0.1-bin.zip

It may end in either "-bin" or "-all" depending on which kind of distribution the user picked, but doesn't make a difference for upgrading the version.
The latest release of Gradle can be obtained programmatically from Gradle's GitHub releases.

Thanks for the detailed description.
Does native Gradle have a feature to update this line itself? Ie, if we leveraged more native Gradle tooling, would it make it easier to solve this as a byproduct?

https://github.com/dependabot/dependabot-core/issues/1164

An alternative approach that'd make it easy to solve this is probably:

https://github.com/dependabot/dependabot-core/issues/2483

There is also a command to upgrade which does the same thing:
./gradlew wrapper --gradle-version=9.0.1

I think that this would be a "new ecosystem" as we would be entering the business of upgrading a different type of files (wrapper files)

See also: https://github.com/dependabot/dependabot-core/issues/2223

Sometimes newer versions of a Gradle wrapper also make changes to associated files:

./gradlew (shell script)
./gradlew.bat
./gradle/wrapper/gradle-wrapper.jar

We could naively edit the value in ./gradle/wrapper/gradle-wrapper.properties, but then these other files wouldn't be updated.
If we could run ./gradlew wrapper --gradle-version=9.0.1, that would be ideal since it would update all four files.

That would indeed be great! Note that the proposed command is not complete. It's often desirable to use the new version's checksum as well, which means providing the --gradle-distribution-sha256-sum option in addition to --gradle-version.
For more info about the possibilities for this, please have a look at this GitHub Action:
https://github.com/gradle-update/update-gradle-wrapper-action

Sometimes newer versions of a Gradle wrapper also make changes to the following files

Because of this, it's usually useful to run the ./gradlew wrapper command twice: the first time sets the version in the properties file but is still running under the old gradle verison. The second time, the actual new gradle version runs, so it updates the related wrapper scripts and jar.

Also as a personal thing, I would love it if it would also ensure that ./gradlew has chmod +x so that Mac and Linux users can use it even if a Windows user generated it, but that might be out of scope

That should just be done the first time you generate the files. Or if the first person committing gradlew forgot to do it, it's easily fixable by just running git update-index --chmod=+x gradlew and then committing and pushing the change (it also has to be done only once). I don't think there is any point in dependabot to handle one-time setups (the point is to maintain things up-to-date, not to preconfigure things). AFAIK, subsequently updating the wrapper normally should keep the executable flag so I don't think Dependabot would mess this up.

There is also a command to upgrade which does the same thing:
./gradlew wrapper --gradle-version=9.0.1

I'm not sure if this is also available with the tooling API, but that may complicate things more compared to just editing a string in the properties file.

And since Gradle 8.1, it's possible to use a label, instead of a specific version, for example:
./gradlew wrapper --gradle-version latest

https://docs.gradle.org/8.1/release-notes.html#gradle-wrapper-introduces-labels-for-selecting-the-version

IS there any update on this?


Thank you!

Just wanted to leave a thank you note that I've been meaning to learn about codecs & related vidoe technologies for so long, but always felt intimidated by the terminology and complexity.
Your project on Github made it really approachable and to the point! Waiting for the sections on Online Streaming - That's what i really want to focus on as my background is in Web Development.
Thank you once again for putting together such great material!

Indeed thanks a lot, really well done, can't wait for the online streaming part! :D


Remove deprecated columns in ResourceFile_Consumables_Items_and_Packages.csv

We should remove the columns in this file: ResourceFile_Consumables_Items_and_Packages.csv as they are not longer being used and could cause confusion.
This file is generated by an analysis script, so the change needs to happen at the level of script.
We should be careful that the final script produces a ResourceFile that is identical to that currently in master, except for the removal of these columns.
This links to PR #1290
This is the context:  https://github.com/UCL/TLOmodel/issues/1298#issuecomment-2017747431

Also noting that in PR #1288, I introduced a file that classifies the item_code into diagnostics/medicine/other.
resources/healthsystem/consumables/ResourceFile_Consumables_Item_Designations.csv
We could merge this information into ResourceFile_Consumables_Items_and_Packages.csv so that the definitions are all in one place. (I didn't do this, knowing that this work is on-going).


Partitions number not equals as expected error in 2.7.1

Describe the bug
Related to https://github.com/apache/pulsar/issues/8000
We have a PR https://github.com/apache/pulsar/pull/9446 before to fix some related issues, seems #9446 does not cover all cases, so it needs to investigate more details about this issue.
org.apache.pulsar.client.api.PulsarClientException: java.util.concurrent.ExecutionException: java.lang.IllegalStateException: allTopicPartitionsNumber 180 not equals expected: 179

Additional context
Pulsar client version 2.7.1

The issue had no activity for 30 days, mark with Stale label.

Closed as stale. Please create a new issue if it's still relevant to the maintained versions.


Idle Time Calculation Incorrect

Upon reviewing the implementation, I noticed that the IdleDuration property in the rate limiters appears to be incorrectly set up. In the built-in rate limiters, idle time only begins once all tokens are replenished. Until then, IdleDuration should return null. However, in your implementation, idle time starts immediately when a token is acquired.
This distinction is crucial because Microsoft's internal DefaultPartitionedRateLimiter uses IdleDuration to determine whether a rate limiter should be discarded. If a rate limiter has been idle for over 10 seconds, the DefaultPartitionedRateLimiter will remove it from its cache, prompting a new rate limiter to be created upon subsequent requests.
Here’s an example to illustrate: Imagine a rate limiter is set with a maximum of 10 tokens per hour, and requests are made every 20 seconds. With the current setup, the rate limiter would be discarded 10 seconds after the last request. This results in no client blocking, effectively allowing 180 requests per hour (one request every 20 seconds), far exceeding the intended limit.

Well, the data is stored in Redis, disposing the rate limiters doesn't affect the current stats (unlike the in-memory implementation). Right?

 I apologize. Thank you for the fast response, I'll close the issue.


a man from moon need some help of yours

hy i hope ur doing well i am using your ulauncher yt extension and its awsome but i just wanna play the video directly by mpv or  vlc instead of opening the yt page in browser can u plz guid me how can i achive this.
thanks in advance

Not sure if you still need these but in case anyone else lands here looking for the same this is what you need to do:
Edit src/items.py as follows
# At the top of the script import RunScriptAction
from ulauncher.api.shared.action.RunScriptAction import RunScriptAction

# On generate_search_item change "on_enter" action
def generate_search_item(search, description_template):
     ...
     on_enter=RunScriptAction(f"mpv --no-terminal -- {search['url']}")

That should be all you need. For personal use just fork the repo, add he changes I mentioned and use that url to install the extension.
Not sure if this project is still in development. I could open a PR and maybe add an option to select if you want to open with mpv or in the browser similar to the option to load thumbnails
.


Make wheels available on PyPI?

Would it be possible to upload Raysect wheels to PyPI? They seem to build OK using python setup.py bdist_wheel, and aren't significantly larger than the tarball already on PyPI. It would significantly speed up installation for most users, and I believe it would also significantly speed up the CI builds on Cherab.

I've built and uploaded manylinux1 wheels for python 3.6 and 3.7 for the latest release. Could you ry using them with a basic pip install raysect?
Took a bit of faffing around to get these built, but it seems to be working here. We should be able to use the same approach for Cherab if this works. I'll need to built a script to automate it, I did it manually this time.

The install seemed to go OK, in a clean conda environment on Python 3.7:
[jovell@cmodws79 ~]$ pip install raysect
Collecting raysect
  Downloading https://files.pythonhosted.org/packages/19/61/f250819d184ea34ad6545847190c438edc314eff85f9318e33ee2617f665/raysect-0.6.0-cp37-cp37m-manylinux1_x86_64.whl (38.9MB)
     |████████████████████████████████| 38.9MB 1.2MB/s
Collecting numpy
  Downloading https://files.pythonhosted.org/packages/9b/af/4fc72f9d38e43b092e91e5b8cb9956d25b2e3ff8c75aed95df5569e4734e/numpy-1.17.4-cp37-cp37m-manylinux1_x86_64.whl (20.0MB)
     |████████████████████████████████| 20.0MB 20.4MB/s
Collecting cython>=0.28
  Downloading https://files.pythonhosted.org/packages/d8/58/2deb24de3c10cc4c0f09639b46f4f4b50059f0fdc785128a57dd9fdce026/Cython-0.29.14-cp37-cp37m-manylinux1_x86_64.whl (2.1MB)
     |████████████████████████████████| 2.1MB 18.7MB/s
Installing collected packages: numpy, cython, raysect
Successfully installed cython-0.29.14 numpy-1.17.4 raysect-0.6.0

I'm not sure Cython is necessary if the package is already built. Also, matplotlib wasn't pulled in, which should be a dependency of raysect. I noticed this when I tried to run one of the demos, and the pipeline rendering failed.
Finally, I tried runing the unit tests, and it seems one of them errors out:
[jovell@cmodws79 ~]$ python -m unittest discover -s ~/scratch/anaconda3/envs/raysect-wheel-test/lib/python3.7/site-packages/raysect
.............................................................................................................................................................................................................................................................................................................................................................E
======================================================================
ERROR: optical.observer (unittest.loader._FailedTest)
----------------------------------------------------------------------
ImportError: Failed to import test module: optical.observer
Traceback (most recent call last):
  File "/home/jovell/scratch/anaconda3/envs/raysect-wheel-test/lib/python3.7/unittest/loader.py", line 470, in _find_test_path
    package = self._get_module_from_name(name)
  File "/home/jovell/scratch/anaconda3/envs/raysect-wheel-test/lib/python3.7/unittest/loader.py", line 377, in _get_module_from_name
    __import__(name)
  File "/home/jovell/scratch/anaconda3/envs/raysect-wheel-test/lib/python3.7/site-packages/raysect/optical/observer/__init__.py", line 32, in <module>
    from .imaging import *
  File "/home/jovell/scratch/anaconda3/envs/raysect-wheel-test/lib/python3.7/site-packages/raysect/optical/observer/imaging/__init__.py", line 39, in <module>
    from .opencv import OpenCVCamera
  File "raysect/optical/observer/imaging/opencv.pyx", line 41, in init raysect.optical.observer.imaging.opencv
AttributeError: type object 'raysect.optical.observer.imaging.opencv.OpenCVCame' has no attribute '__reduce_cython__'

----------------------------------------------------------------------
Ran 350 tests in 0.043s

FAILED (errors=1)

I haven't seen this error before. Also, it's rather curiously running 350 tests, despite the last build on Github running 349. Perhaps there is something wrong in the way I'm getting unittest to discover and run the tests...

Thanks for testing that, glad it worked.
I suspect you have opencv installed, so it is running an additional test. I'll have to modify the CI to include opencv to trigger that test.
The cython dependency is necessary for the source distribution at the moment, so it gets installed as a side effect. You are right about the matplotlib dependency. I'm not sure why that is missing. I'll fix that in the next point release.

It doesn't look like I have opencv installed:
[jovell@cmodws79 ~]$ pip list
Package         Version
--------------- -------------------
certifi         2019.9.11
cycler          0.10.0
Cython          0.29.14
idlbridge       1.1.1
kiwisolver      1.1.0
matplotlib      3.1.1
MDSplus         7.91.0
numpy           1.17.4
pip             19.3.1
pyparsing       2.4.5
python-dateutil 2.8.1
raysect         0.6.0
setuptools      41.6.0.post20191030
six             1.13.0
wheel           0.33.6

(Ignore idlbridge, that came from a pip install --user). And I installed matplotlib manually.

By the way, using the raysect wheels speeds up the Cherab core CI checks by about 9 minutes, a factor of 3.

This has been done by @CnlPepper.


Fix typos in Docker Swarm guide

Fix two minor typos in the new Docker Swarm guide

LGTM.  Thanks for the PR.

Hi @makuk66,
I forgot to ask, would it be possible for you to sign a contributors agreement?  It's detailed here:
http://docs.projectcalico.org/en/latest/contribute.html
Thanks,
Rob

Done


fromBeginning belongs to subscribe() and not consumer

when reading the document
https://kafka.js.org/docs/consuming#a-name-from-beginning-a-frombeginning
The consumer group will use the latest committed offset when fetching messages. If the offset is invalid or not defined, fromBeginning defines the behavior of the consumer group.
it doesn't say which function takes this parameter fromBeginning
it would be nice to update the document that this parameter is used as
consumer.subscribe({ topic: 'test', fromBeginning: true })

I agree, thanks for the suggestion


Tokens on Rarible are not showing after contract deploy

Discussed in https://github.com/hashlips-lab/nft-erc721-collection/discussions/210

Originally posted by ecerroni April  4, 2022
There seems to be a recurring issue in getting the tokens being displayed on OpenSea after successfully deploying the contract.
Unfortunately, I've been of those that experienced this before hand. I tried to investigate the issue in many ways as I wanted to check if it was something related to the contract or because of OpenSea.
My conclusions are that the contract in its current implementation is totally fine. The issue seems to be with OpenSea and how it handles new contracts deployed on the block chain.
Generally the issues the user may facing are:

error messages while trying importing the contract: "We couldn't find this contract. Please ensure that this is a valid ERC721 or ERC1155 contract deployed on Ethereum/Polygon and that you have already minted items on the contract."
Contract has been imported, but none or less tokens are listed

This is mainly on OpenSea side. I found an interesting article that sums it up quite well:

Based on observed behavior this is how OpenSea ERC721 validation works:

OpenSea automatically monitors all new contracts that are deployed. The moment OpenSea detects a new contract has been deployed it calls contract.supportsInterface(type(IERC721).interfaceId) on that contract. If the function call returns true then OpenSea adds it to its list or database of potential ERC721 contracts. If the function call returns false then OpenSea will not detect it as an ERC721 contract and any attempts at validation fails.

OpenSea checks during deployment and after deployment (for any contract that passes step 1 above) if any ERC721 Transfer events have been emitted from a contract. If not then any validation fails.

If a contract passes step 1 and 2 then it will pass ERC721 validation on

ref: https://dev.to/mudgen/how-to-solve-openseas-we-couldnt-find-this-contract-during-erc721-validation-4lp2
There are many issues that people opened on this repo that claim having these kind of problem. Both Ethereum and Polygon may be affected with Polygon being significantly more vulnerable to this. Personally I deployed on Polygon when I had this issue.
The fact that successfully deploying the contract to testnets first does not prevent this kind of issues on the mainet.
As @liarco himself pointed out there is no rule here, but a single mint operation with thousands of tokens can have negative impact on the expected outcome.
My 0.02 is that:

OpenSea db update cannot catch up with thousand of events firing at the same time for a single contract, it throws an error and the minting events are gone forever from their point of view
OpenSea does not like thousands of concurrent mints at the same time in the first place and it just ignore them. Maybe it sees is as spam.
OpenSea/Blockchain integration is far from perfect and things get lost with this kind of excessive load, especially on Polygon

The above assumptions are based on my personal circumstantial evidence:

I deployed the contract
I minted 4947 items
I cannot import the contract (I tried twice a day)
In the meantime contract is imported on Rarible (after 6 days of failed attempts), Refinable (although with the help of their devs) and NFTrade (quite quickly) with all 4947 items showing properly.
after 2 weeks I mint an additional item
I was able to finally import the contract on OpenSea with one item showing (the last mint only)
nothing changed for 3 days
I did transfer one of the initial minted items from me to myself again. This item showed up on OpenSea under the collection after just a few minutes.

So the main takeaway is to not mint all at once. I am also keen to believe that with the standard process of minting through the dapp will not result in any issue.
Any contribution to this discussion is very welcome.

Hi, thank you for providing so many details about this topic. Anyway, issues are intended for bug report only so I kindly ask you to use #210 for any follow up on this.
Thank you for your understanding.


[Issue] domain add fails due to incorrect domain formatting

I was running cat domains.txt |bbrf domain add - -s subfinder  --show-new and got this error:
[ERROR] string indices must be integers
I found some domains (from subfinder) starting with '_', character that is not allowed.
It would be cool/nice to have a warning/error message when these type of formatting errors are found.

Hi @pdelteil - the underscore is unlikely to be the issue here, as they should be allowed since v1.1.2 or otherwise ignored; I am planning to add more verbosity to error messages in debug mode to help troubleshoot this. It looks like the error is a result of an unexpected response from the BBRF server, rather than malformed input. I'm going to assume this is the same root cause as #44 unless this persists after that is resolved.
Closing for now!

I found some domains (from subfinder) starting with '_', character that is not allowed.

Turns out this was exactly the problem 😅


Explain "dwc:" in preamble to Quick Reference Guide

We have implemented the addition of namespaces wherever specific terms are referenced in term definitions, comments, and examples. The introductory text to the Quick Reference Guide would benefit greatly by including an explanation of this usage.

Done.


Issue 12

This PR is a follow up of #15

It updates the handlers processing the AggregationSourceDiscovered and ResourceCreated events. At the reception of the AggregationSourceDiscovered event the handler adds a UUID as Context to the Subscription. The Agent will have to add such Context to the following Events issued to the Sunfish server.
It adds a Makefile simplifying the steps to build the whl

@mgazz this needs to be rebased on the current master. Also, all commits must be signed. See the DCO check failing.

@christian-pinto thank you for the review. I rebased the branch and added verified commits. I'll be available in case on any question.

@dledford thank you for the feedback. I agree with your request for changes. I will update as requested.
Your review bring up something that we have yet to discuss: styling. We have yet to tackle this problem, we can adopt a linter that will help us styling the code automatically (example: pylint,flake8,etc..). (We can discuss this in another issue).


HTML 06: Create html file for day page

e.g fonts and dayjs

Day js and font awesome custom links added to html file


Print password prompts to stderr

This makes rclone with encrypted config better suited for use in
pipelines. E.g.:
$ rclone lsl mydrive:Some/Dir | sort -k 4

If the password prompt ("Enter configuration password") is printed to
stdout, it will be swallowed by sort. By printing it to stderr, you
still see the prompt, without sacrificing compatibility with the unix
pipeline.
This patch only focuses on password prompts, which may seem a bit haphazard and inconsistent. However, I didn't want to embark on a full code reorg just to change this in your project :P feel free to adapt, of course.

In a typical program, y password prompts and reading of password input are done via /dev/tty.  This can get annoying; e.g. it prevents entering a password response as part of a pipeline, so some programs also allow for optional "use stdin for password reading".
If you go down the /dev/tty path then beware the case where you can't open it (e.g. from a cron job) because there is no terminal associated with it; in this case it may make sense to abort because if there's no terminal then the user can't enter a password!
Also I'm not sure what the windows equivalent is; in the old DOS days we'd use CON: but I don't know if that carries over in the modern windows world.

A check with passwd on linux indicates it writes its prompt to stderr, unless I did something stupid.
$ passwd USER
Changing password for USER.
(current) UNIX password:
passwd: Authentication token manipulation error
passwd: password unchanged

$ passwd USER >/dev/null
(current) UNIX password:
passwd: Authentication token manipulation error
passwd: password unchanged

$ passwd USER 2>/dev/null
Changing password for USER.

$

The library rclone uses for reading the terminal prompt reads from stdin at the moment.  That could easily be /dev/tty instead.  However I'm really struggling to work out how this could be done in a cross platform way!
There is already quite a good work-around for having to enter the password for rclone commands - set the environment variable.  I have this in a little shell script
#!/bin/echo Source this file don't run it

read -s RCLONE_CONFIG_PASS
export RCLONE_CONFIG_PASS

I'm verging on thinking that this PR is a net improvement.  I think using /dev/tty is probably too difficult to do cross platform, and is bound to cause problems to existing users.
Any futher thoughts @sweharris ?

passwd is complicated, due to PAM interaction and it may change depending on configuration and platform.  Equally interesting is whether it uses stdin for reading the password :-)
I agree that sending it to stderr will help in pipelines so it can be removed from filters is an improvement over current.

I've merged this - thank you for your contribution :-)

Thanks, and thank you for your careful and detailed consideration.
On Sat, 22 Apr 2017 at 09:38, Nick Craig-Wood<EMAIL_ADDRESS>wrote:

I've merged this - thank you for your contribution :-)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ncw/rclone/pull/1340#issuecomment-296358114, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAIafAZ0h8YSMdw4c12vF4CT76o8wgSIks5rybyfgaJpZM4M-aGi
.


Python 3.11

Hey there, I'd like to use you projects but am currently struggling building it on my python 3.11 running machine.
I hope I did not miss anything, but I installed python 3.10 in parallel, setup a venv running on 3.10 accordingly, installed pyinstaller in there and ran the steps from the Makefile in that order.
The application does work? when I then install the built wheel in the venv and run agua_amiga.
At least the UI does.
Whether the bluetooth part works, I cannot tell yet, as it does not find my bottle.
Before I start debugging that, are there any technical reasons, why this project is limited to 3.10 and not 3.11?
I'd really like to use the system python instead of the venv.

That was fast, thanks :D
I'm just digging into trouble-shooting my bottle.
It's a hidratespark pro, and as far as I can tell (just getting started with gatttool) at least the sip-uuid is still present; so maybe this tool works just fine, after I got past setting the bottle up initially.

Yeah I had no other bottles to test with so I have no idea if it will work. I gathered most of my knowledge on how the protocol works from https://github.com/choonkiatlee/wban-python/blob/master/Hidrate.py#L152 which is sparse on details as well. A really great way to explore your device is with d-feet and the BlueZ dbus service. Bonus thing is that this is also how agua amiga talks to bluetooth devices so you see the same things it sees.

Hmm agua amiga uses python-dbus-next and there's an error that happens on Python 3.11 in that library. A fix was merged but a new release is not forth coming. I'm going to try building/running it from git to see if that fixes the segfault. I'm pretty sure it's in that library because I've tracked the failure to the line that connects to DBus.

Not sure if that helps, but this is an strace for the latest commit.
https://aiyionpri.me/shared/agua_amiga/ea9af4d1.strace

You mean https://github.com/altdesktop/python-dbus-next/issues/138, right?

Yeah that's the one. Using master branch from that git repo fixes the segfault on my end so I've pushed updates to the dependencies.

Confirmed; the latest commit builds without problems.

Sweet! I'll close this issue then but feel free to open another if you run into more problems.


Broadlink sensor startup failure: expected str instance, bytes found

Make sure you are running the latest version of Home Assistant before reporting an issue.
Home Assistant release (hass --version):
(homeassistant) homeassistant@pi:~/.homeassistant$ hass --version
0.60.0
Python release (python3 --version):
(homeassistant) homeassistant@pi:~/.homeassistant$ python3 --version
Python 3.4.2
Component/platform:
broadlink-switch
Description of problem:
broadlink component fails to initialize with this error
Expected:
starts up
Problem-relevant configuration.yaml entries and steps to reproduce:
switch:
  - platform: broadlink
    host: <IP_ADDRESS>
    mac: '34EA34CCCF86'
    type: 'rm2'

Traceback (if applicable):
2018-01-01 18:57:41 ERROR (MainThread) [homeassistant.components.switch] Error while setting up platform broadlink
Traceback (most recent call last):
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/helpers/entity_component.py", line 171, in _async_setup_platform
    SLOW_SETUP_MAX_WAIT, loop=self.hass.loop)
  File "/usr/lib/python3.4/asyncio/tasks.py", line 372, in wait_for
    return fut.result()
  File "/usr/lib/python3.4/asyncio/futures.py", line 277, in result
    raise self._exception
  File "/usr/lib/python3.4/concurrent/futures/thread.py", line 54, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/switch/broadlink.py", line 174, in setup_platform
    broadlink_device.auth()
  File "/srv/homeassistant/lib/python3.4/site-packages/broadlink/__init__.py", line 203, in auth
    response = self.send_packet(0x65, payload)
  File "/srv/homeassistant/lib/python3.4/site-packages/broadlink/__init__.py", line 258, in send_packet
    payload = self.encrypt(payload)
  File "/srv/homeassistant/lib/python3.4/site-packages/broadlink/__init__.py", line 162, in encrypt_pyaes
    return "".join([aes.encrypt(bytes(payload[i:i+16])) for i in range(0, len(payload), 16)])
TypeError: sequence item 0: expected str instance, bytes found

Additional info:
(homeassistant) homeassistant@pi:~/.homeassistant$ pip3 list
DEPRECATION: The default format will switch to columns in the future. You can use --format=(legacy|columns) (or define a format=(legacy|columns) in your pip.conf under the [list] section) to disable this warning.
aiohttp (2.3.6)
aiohttp-cors (0.5.3)
asn1crypto (0.24.0)
astral (1.4)
async-timeout (2.0.0)
boto3 (1.5.6)
botocore (1.8.20)
broadlink (0.5)
certifi (2017.11.5)
cffi (1.11.2)
chardet (3.0.4)
colorlog (3.0.1)
cryptography (2.1.4)
distro (1.1.0)
docutils (0.14)
ecdsa (0.13)
enum-compat (0.0.2)
envs (1.2.4)
future (0.16.0)
fuzzywuzzy (0.15.1)
gTTS-token (1.1.1)
home-assistant-frontend (20171216.0)
homeassistant (0.60.0)
idna (2.6)
Jinja2 (2.10)
jmespath (0.9.3)
MarkupSafe (1.0)
multidict (3.3.2)
mutagen (1.39)
netdisco (1.2.3)
netifaces (0.10.6)
pip (9.0.1)
protobuf (3.5.1)
py3compat (0.4)
pyaes (1.6.0)
PyChromecast (0.8.2)
pycparser (2.18)
pycrypto (2.6.1)
pycryptodome (3.3.1)
python-dateutil (2.6.1)
python-jose-cryptodome (1.3.2)
pytz (2017.3)
pywemo (0.4.20)
PyYAML (3.12)
requests (2.18.4)
rxv (0.5.1)
s3transfer (0.1.12)
setuptools (38.2.4)
six (1.11.0)
SQLAlchemy (1.1.15)
typing (3.6.2)
ua-parser (0.7.3)
urllib3 (1.22)
user-agents (1.1.0)
voluptuous (0.10.5)
warrant (0.6.1)
xmltodict (0.11.0)
yarl (0.16.0)
zeroconf (0.19.1)

Have the same issue on fresh install
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/homeassistant/helpers/entity_component.py", line 189, in _async_setup_platform
    SLOW_SETUP_MAX_WAIT, loop=self.hass.loop)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/asyncio/tasks.py", line 352, in wait_for
    return fut.result()
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/asyncio/futures.py", line 244, in result
    raise self._exception
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/concurrent/futures/thread.py", line 55, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/homeassistant/components/switch/broadlink.py", line 176, in setup_platform
    broadlink_device.auth()
  File "/Users/Shlom/.homeassistant/deps/lib/python/site-packages/broadlink/__init__.py", line 203, in auth
    response = self.send_packet(0x65, payload)
  File "/Users/Shlom/.homeassistant/deps/lib/python/site-packages/broadlink/__init__.py", line 258, in send_packet
    payload = self.encrypt(payload)
  File "/Users/Shlom/.homeassistant/deps/lib/python/site-packages/broadlink/__init__.py", line 162, in encrypt_pyaes
    return "".join([aes.encrypt(bytes(payload[i:i+16])) for i in range(0, len(payload), 16)])
TypeError: sequence item 0: expected str instance, bytes found```

@AMSHL Have you tried the steps above to install pycryptodome before Hass self-installs the pycrypto libraries?

@jamieshaw Yes, tried that already - no result.

@benh57 thank you very much for the instruction. I implemented and got my Broadlinks working!!
this is the commands
sudo systemctl stop<EMAIL_ADDRESS>sudo su -s /bin/bash homeassistant
source /srv/homeassistant/bin/activate
pip3 uninstall pycrypto
pip3 uninstall pycryptodome
pip3 install pycryptodomepip3 install
exit
sudo systemctl restart<EMAIL_ADDRESS>

pip3 install pycryptodomepip3 install
Collecting pycryptodomepip3
Could not find a version that satisfies the requirement pycryptodomepip3 (from versions: )
No matching distribution found for pycryptodomepip3

I openned another issue since the logs wasn't the same but there's an easy fix by simply updating the broadlink library.


Reformat 'about' documentation for nextercism

Hello. I'm trying to get all of the ABOUT.md files reformatted to fit with the new look and feel of nextercism. This means removing things like line-breaks and markdown that we're not supporting.
As an example from the C++ Track, we've gone from:

to

As a general note, after these formatting changes get merged, we're hoping that track
maintainers will take some time to review the contents of the ABOUT.md file and edit
it to make it a bit shorter and more casual. We're looking for less wikipedia and more
of an enthusiastic and friendly pitch that you might make to a friend over a drink for
why they should try the language.
For some examples of tracks where I think they've done this well take a look at the
Go and R tracks:

https://github.com/exercism/go/blob/master/docs/ABOUT.md
https://github.com/exercism/r/blob/3f05539d19d7e20d2a9d5cd2cecd3dc56f4f23e8/docs/ABOUT.md

See https://github.com/exercism/meta/issues/84 for more details.
Thank you!

Which and what subset of markdown is being supported?  I notice in the change there is still markdown for the link.

Which, and what subset of, markdown is being supported?

@kotp The canonical answer for that is going to be in the meta issue (as it might update based on the discussion we're having there about the last point) but for now it is: "ABOUT.md should now only use bold, italic, links, bullet lists and number lists, and should not split paragraphs into lines."

Yep, found it, and made my objection, though Nobbz beat me to it. :)

The markdown links should go too, according to this "only use" list, or be added to that list.

Links are there after italic.

:blush: yes, I see it now.


Usability: Calibrate Single Channel Pipette Last

As a user, I would like to calibrate my single channel pipette to tipracks last, so that I can use the single channel pipette for labware calibration.
Acceptance Criteria

[ ] If there is a single channel pipette, it should appear second in the pipettes list
[ ] If there is a single channel pipette, it should be calibrated to tiprack second
[ ] If there are two single channel pipettes, calibrate the left one first (if we need a default...it doesn't really matter)

This ticket seems to also require disabling the tipracks in the list so they cannot be clicked on

Duplicate of 1128


Hide Packages & releases

Suggested to hide packages & releases if not being used in the project/repo

Okay sure I didn't knew this feature is there Thank you


feature/google-ads-api-default

Are you a current Fivetran customer?

Fivetran created PR
What change(s) does this PR introduce?

The api_source variable is now defaulted to google_ads as opposed to adwords. The Adwords API has since been depricated by Google and is now no longer the standard API for the Google Ads connector. Please ensure you are using a Google Ads API version of the Fivetran connector before upgrading this package.

Please note, the adwords version of this package will be fully removed from the package in August of 2022 (when the V2 updates roll out).

Did you update the CHANGELOG?

[X] Yes

Does this PR introduce a breaking change?

[X] Yes (please provide breaking change details below.)
[ ] No  (please provide an explanation as to how the change is non-breaking below.)

The default nature of the api_source has changed. As such, we should treat this as a breaking change.
Did you update the dbt_project.yml files with the version upgrade (please leverage standard semantic versioning)? (In both your main project and integration_tests)

[X] Yes

Is this PR in response to a previously created Bug or Feature Request

[X] Yes, Issue/Feature From The Ad Reporting Package #51
[ ] No

How did you test the PR changes?

[X] CircleCi
[X] Local (please provide additional testing details below)

Select which warehouse(s) were used to test the PR

[X] BigQuery
[X] Redshift
[X] Snowflake
[X] Postgres
[X] Databricks
[ ] Other (provide details below)

Provide an emoji that best describes your current mood

🍎
Feedback
We are so excited you decided to contribute to the Fivetran community dbt package! We continue to work to improve the packages and would greatly appreciate your feedback on our existing dbt packages or what you'd like to see next.

@fivetran-sheringuyen thanks for the review! I just pushed changes that address your inline comments. Additionally, to address your additional comments:

is it right to assume the below configs will be removed after full deprecation of adwords capability?
{{ config(enabled=var('api_source') == 'google_ads') }}

Yes, this will be removed in the next update once we remove the adwords components.

There was one warning during dbt test that I've added below. Is the below expected?

Yes this is expected. This is a result of the final_urls field possibly containing multiple urls. In a similar fashion where we select the latest historical record for some history models, in this case we select only one url to include in the final model. This warning is by design to raise awareness to users that a url is being intentionally cut off. See here for the specific test that is providing the warning.

@fivetran-joemarkiewicz Just took a peep, PR looks good! Thank you for the changes, I'm giving this a thumbs up!
Re: your warning comment, what are your thoughts on including a comment in the yml in the test section to briefly explain the expected warning?

@fivetran-joemarkiewicz Just took a peep, PR looks good! Thank you for the changes, I'm giving this a thumbs up!
Re: your warning comment, what are your thoughts on including a comment in the yml in the test section to briefly explain the expected warning?

Good call out. I actually thought I already had that in place, looks like I didn't. Will add now!


yolov7 custom tiny model:  X_LINK_ERROR | side values? | poor detection with OAK-D

Hello,
I have converted a custom trained yolov7 tiny model (13 classes and mAP=0.75, 1024x1024)  into a blob with http://tools.luxonis.com/
I have used the blob following the instruction of YoloV7_training.ipynb notebook from [depthai-ml-training] repo with a OAK-D camera (connected to USB3 with/or without additional power supply)
I have 2 issues:
1) after few seconds I get an error. Why?
Traceback (most recent call last):
File "main.py", line 51, in
pv.prepareFrames()
File "/home/mz/Projects/ObjectDetection/depthai/depthai_sdk/src/depthai_sdk/managers/preview_manager.py", line 148, in prepareFrames
packet = queue.tryGet()
RuntimeError: Communication exception - possible device error/misconfiguration. Original message 'Couldn't read data from stream: 'color' (X_LINK_ERROR)'
2) before the error I see very few and bad detections even if the trained model gave very good results on the test set for static images (mAP~0.75). Why? Please see my Obs below: is that the reason?
Obs: The JSON note in https://github.com/luxonis/depthai-experiments/tree/master/gen2-yolo/device-decoding is not clear to me:
I have not changed the Json file of my model which is 1024x1024 as I do not understand WHERE I have to change the "side" entries: I have no side32 or side16 but many of them and all concerning the anchor masks. See attached my json file from the blob conversion: shall I do change something? how and where exactly? best.zip
Note: Values must match the values set in the CFG during training. If you use a different input width, you should also change side32 to sideX and side16 to sideY, where X = width/16 and Y = width/32. If you are using a non-tiny model, those values are width/8, width/16, and width/32.
Thank you in advance
Marco

Hi @ZucchiniAI ,
We apologize for the delay, I'll check this issue tomorrow morning.
Thanks, Erik

Hi @ZucchiniAI , are you using the latest version of depthai? We have tried the same process, and it crashes with old (2.16) depthai, but not with the newest depthai (2.17.4).
Thanks, Erik

Hello  Erik,

the X_LINK_ERROR (Point1) happened with the depthai <IP_ADDRESS>.
I have now updated to 2.17.4 and indeed this do not happen at all: thank you!

What about my Point2? I see now that the objects are detected with good confidence but with much redundant bounding boxes (e.g. a couple for each object with different sizes, almost in the region of the ground truth) --> it seems to me the NMS does not work well. Could you please tell if the reason is my supposition (see Obs. in my question above) and how/where exactly  shall I change the indicated sideX and sideY for my tiny model (1024x1024)?
Thank you in advance! Marco

Hi @ZucchiniAI , could you try with different IoU threshold setting?

Hello Erik, I was trying as you suggested, but unfortunately the result do not convince me totally :-(
a) a couple of time I got still the X_LINK_ERROR even if env with depthai 2.17.4 was activated .... what is the cause? (I have the USB-C to USB 3 port and the additional power connected): RuntimeError: Communication exception - possible device error/misconfiguration. Original message 'Couldn't read data from stream: 'rgb' (X_LINK_ERROR)'
b) By changing the iou_threshold in the best.yaml in range 0.2 - 0.8 I saw some changes (0.2 being better, but not perfect) but anyhow the detections appear and disappear rapidly and in different positions: sometime is very clear that the object is detected twice even if the model before conversion detected one only. Sometime is difficult to say if is a double bounding box for the same object or intermittent small and large: is there a way to reduce the inference frequency  (with my tiny yolo7 I get some 11 fps):
My set up is to point the OAK-D camera towards the pc screen where I have the same images which yolov7 before the conversion detected correctly.
c) sorry if I ask but again your hint  in https://github.com/luxonis/depthai-experiments/tree/master/gen2-yolo/device-decoding  is not clear to me: I have not changed the Json file of my model which is 1024x1024 as I do not understand WHERE I have to change the "side" entries: I have no side32 or side16 but many of them and all concerning the anchor masks. See above attached my json file from the blob conversion: shall I do change something? how and where exactly?
Thank you!
Marco

Hi @ZucchiniAI ,
a) That's a very generic error, basically means "something went wrong". Is it sporadic error after X hours?
b) Could you share a video? That would help a lot with debugging.
c) Anchor masks are are already written in JSON (that you downloaded from the tools.luxonis.com) and will be loaded to the device, I am not sure why you would need to change them?
Thanks, Erik


Dependent pods fail linting due to very old deployment target on Xcode 15.3

:wave: hello there and thanks for the library!
I was attempting to publish this pod: https://github.com/dblock/ios-snapshot-test-case-expecta
which has ios-snapshot-test-case as a dependency.
However bundle exec pod lint fails to pass validation due to an incompatible minimum deployment target with the latest versions of Xcode:
 -> Expecta+Snapshots (3.2.0)
    - WARN  | [iOS] swift: The validator used Swift `4.0` by default because no Swift version was specified. To specify a Swift version during validation, add the `swift_versions` attribute in your podspec. Note that usage of a `.swift-version` file is now deprecated.
    - ERROR | [iOS] xcodebuild: Returned an unsuccessful exit code. You can use `--verbose` for more information.
    - NOTE  | xcodebuild:  note: Using codesigning identity override: -
    - NOTE  | [iOS] xcodebuild:  note: Building targets in dependency order
    - NOTE  | [iOS] xcodebuild:  note: Target dependency graph (6 targets)
    - NOTE  | [iOS] xcodebuild:  clang: error: SDK does not contain 'libarclite' at the path '/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/arc/libarclite_iphonesimulator.a'; try increasing the minimum deployment target
    - NOTE  | [iOS] xcodebuild:  Pods.xcodeproj: warning: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 10.0, but the range of supported deployment target versions is 12.0 to 17.2.99. (in target 'iOSSnapshotTestCase' from project 'Pods')
    - NOTE  | [iOS] xcodebuild:  Pods.xcodeproj: warning: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0, but the range of supported deployment target versions is 12.0 to 17.2.99. (in target 'Expecta' from project 'Pods')
    - NOTE  | [iOS] xcodebuild:  Pods.xcodeproj: warning: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0, but the range of supported deployment target versions is 12.0 to 17.2.99. (in target 'Specta' from project 'Pods')

[!] Expecta+Snapshots did not pass validation, due to 1 error and 1 warning.
You can use the `--no-clean` option to inspect any issue.

There is an issue in cocoapods here: https://github.com/CocoaPods/CocoaPods/issues/12033 around this but I think the short of it is that deployment targets need to be above 12 for Xcode 15+.
Any possibility of updating? I can also give it a shot.
Example update in another library: https://github.com/clemensg/sqlite3pod/pull/23/files

We no longer support CocoaPods, SPM integration is the path forward, if you see issues with that let me know.

thanks for the reply, we will probably need to drop since we are stuck on Cocoapods due to react native and don't want 2 package managers but understand it is not a priority. Have a good day!


Does the following code in losses.py contain a bug?

The following code from losses.py file from line 11 to 19.
    model.train()
    N =  batch["images"].size(0)
    assert N == 1

    blob_dict = get_blob_dict(model, batch)
    # put variables in cuda
    images = batch["images"].cuda()
    points = batch["points"].cuda()
    counts = batch["counts"].cuda()

The get_blob_dict set the model to eval(), and afterwards the model is in eval. The model is in eval during the code to calculate loss and  backward loss .
Does this correct?

It is not intentional, but it will not change the results. This is because I set the resnet backbone to .eval mode in the forward pass anyway.

The code is trained in eval mode. So the result does not change when do validation. I think the performance will improve if set the model to train during training. As the ResNet backbone use lots of BatchNorm layer.

True, let me know how well it works for you! though, it didn't work well in my experience :(

I will test it. Thanks for your replay.


nikita

Hi Nikita,
I don't think the code you've added is deployable. You need to add your image to the "images" folder in the repository. Make sure you give it a descriptive name, something like, "nikita.jpg". There might also be some changes in the CSS file that you need to make. Study the code well and understand it. Make sure you run the HTML file on your browser and see that your changes get reflected properly. If you're new to HTML/CSS programming, I suggest you take the HTML/CSS course on Codecademy first.
Also, please give descriptive pull request and commit messages. Maybe something like "Added image and link for new organiser Nikita Gandhi". Messages like "nikita" or "Update index.html" don't exactly help  in understanding the purpose behind the commit and/or pull request.
I'm closing this pull request for now without merging your changes into the code base. Please follow the above instructions and send in a new pull request accordingly.
All the best!


nimble/ll: Fix encryption start procedure

We need to update rx packet counter and direction bit after sending LL_START_ENC_RSP in central role as otherwise we will fail to decrypt LL_START_ENC_RSP from peripheral and will drop connection due to MIC failure.
This only happens if peripheral replies with LL_START_ENC_RSP in the next slot after we send LL_START_ENC_RSP. If peripheral replies with an empty PDU instead, we will update direction bit on continuation and decryption on rx will be successful.

#AutoPTS run mynewt SM/CEN/JW/BI-04-C

AutoPTS Bot results:
No failed test found.
Successful testsSM      SM/CEN/JW/BI-04-C               PASS


Next and Previous links

Michael,
Thanks for the control works like a dream.
I want to expand on it but I am new to Angular.
I want to have a next link, a current title and previous link, which reflects the selected row in the table / list paginated data.
e.g.
1a      2b    3c    <-- next / prev control
1a        <-- previous
<2b>   <-- selected row
3c        <-- next
4d
5e
6f
<<  <  1 2 3 >  >>         <-- dir-pagination-controls
I would like it to step onto the next page when relevant.
I have done this but I have had to use jQuery which is not the Angular way, and the code is not elegant.

Hi,
I don't quite understand what it is you are trying to achieve. What do the next/prev links do? Move the selected row forwards/backwards?
Maybe if you show me what you already have, I'd get a better idea (don't worry about elegance or lack of it - no judging here ;))

A picture is worth a thousand words.

What I have been trying to achieve is the same technique you used where the Angular code file contains all the controls which I can then place anywhere on the screen.

The tab grid / chart area is where the detail of the selection is shown.

Ah ha. The picture explains it perfectly.
So, the Angular way would be to do something like keeping track of the index of the selected item of your list, and then using that to bind all the controls together.
I threw together a working demo doing this, the main parts are:

    {{ getSelectedName() }}

    <button ng-click="selectPrevious()" class="btn btn-xs">
        < {{ getSelectedName(-1) }}
    </button>

    <button ng-click="selectNext()" class="btn btn-xs">
        {{ getSelectedName(+1) }} >
    </button>

        Desc
        Date

        {{ row.desc }}
        {{ row.date }}

// in your controller
$scope.currentPage = 1;
$scope.pageSize = 10;
$scope.items = items; // this is your collection from wherever you get it.

$scope.selectedIndex = 0;

$scope.getSelectedName = function(shift) {
    shift = shift || 0;
    var realIndex = ($scope.currentPage - 1) * $scope.pageSize + $scope.selectedIndex + shift;
    return $scope.items [realIndex] ? $scope.items [realIndex].desc+ ' - ' + $scope.items [realIndex].date: '';
};

$scope.selectPrevious = function() {
    if ($scope.selectedIndex === 0) {
        if (1 < $scope.currentPage) {
            $scope.currentPage--;
            $scope.selectedIndex = $scope.pageSize - 1;
        }
    } else {
        $scope.selectedIndex--;
    }
};

$scope.selectNext = function() {
    if ($scope.selectedIndex + 1 === $scope.pageSize) {
        $scope.currentPage++;
        $scope.selectedIndex = 0;
    } else {
        $scope.selectedIndex++;
    }
};

Here's a working demo (based on tasty meals, but essentially the same): http://plnkr.co/edit/uChQNw8vmznoL4vmNCO4?p=preview

But I believe I understand what you are doing.
Thank you for the prompt replies and attention very much appreciated.


Using System.env.ANDROID_NDK instead of a local.properties file which do...

...es not even exist until you run Android Studio
I;ve tested this on a newly checked out code, I already had ANDROID_NDK set up. Hopefully other people will too.

What are all the other commits for? SQLiteCpp and stuff?

The problem is that Android Studio is kind of the state of the art for Android development.  I added a new commit on master that probably fixes your issue.
https://github.com/libmx3/mx3/commit/4a516b05cb138e977d77668ea9e1dc6676ba09f9
Thanks for the report!

Sorry about other commits. I didn't realize I was on same branch as gradle env variable fix.

On Feb 20, 2015, at 1:14 AM, Steven Kabbes<EMAIL_ADDRESS>wrote:
What are all the other commits for? SQLiteCpp and stuff?
—
Reply to this email directly or view it on GitHub.


AttributeError: 'CGPrecond' object has no attribute 'use_fp16'

AttributeError: 'CGPrecond' object has no attribute 'use_fp16' happen when I run the code bash launch.sh in the diff-solvers-main about Generating 4 images to "./samples/grids/imagenet256/dpmpp_nfe6"...

Fixed in the last commit by adding self.use_fp16 = use_fp16 to class CGPrecond : )


UI is cutoff on 4:3 monitors

So with mat_letterbox_aspect_goal 1.33 and mat_letterbox_aspect_threshold 1.33 in autoexec.cfg, you can play the game in 4:3 (I still rock a 4:3 CRT, playing at 1920x1440 @ 90hz)
The games standard UI normally gets cutoff on the left and right edges. But maybe it's possible to get Northstar's UI to conform to a letterboxed 16:10 region on monitors taller than 16:10?
I'll post a screenshot below, it looks like the UI is left justified instead of centered, but I'm not sure:

Putting
mat_letterbox_aspect_goal 1.33
mat_letterbox_aspect_threshold 1.33

as described, I cannot get 4:3 to work. Am I missing anything?

Also it's been quite a while since this issue is opened, so this might no longer apply. @bazookaben would be great if you could update us with how much this is still an issue. ^^

Hey sorry I forgot to respond to this.
First off, I should mention that this really isn't a a Northfall issue, Titanfall 2 has always behaved this way with these command line arguments.
So here are the actual commands:
+mat_letterbox_aspect_goal 1.33 +mat_letterbox_aspect_threshold 1.33
And these commands won't activate until you hit alt+enter
And I found that the menu is actually displayed in whole if you hit alt+enter on the title page. If you activate later, the menu gets cropped.
And I should mention that HUD is always cropped. This is aslo a Titanfall issue, not a Northstar issue

Not game breaking, but I can't really use the minimap.
So all this to say: do you guys think it's possible to create a 3rd command line argument that preserves the HUD when using taller-than-16:10 aspect ratios? Either by locking the elements to the edges of the screen, or just preserving the 16:10 HUD in the middle section of the screen?

So all this to say: do you guys think it's possible to create a 3rd command line argument that preserves the HUD when using taller-than-16:10 aspect ratios?

I fear that might be too much work without enough payout, especially as there is the Alt+Enter workaround in menu screens as you mentioned :/
Regarding the in-game HUD, there are mods like HUDRevamp that move around HUD elements. One could probably make a mod based on it to move the position of minimap, titanmeter, killfeed, ... further inward. Maybe try contacting the author of that mod on their GitHub repo to see if they'd be interested in adding parameters that allow moving the remaining elements (some, like abilities and ordnances can already be moved). ^^

Thanks for pointing me in the right direction.


Move deps from devDeps to deps

Install in other projects via "npm install --save" fails because you put the dependencies (like mkdirp) in devDependencies instead of dependencies.

Thanks. Will fix momentarily.

try v1.0.4


[FEATURE] add anomaly detect functions in promql/function.go

[FEATURE] add anomaly detect functions in promql/function.go
Functions list:
"iad_n_sigma"
"iad_n_sigma_upper_bound"
"iad_n_sigma_lower_bound"
"iad_box_line"
How to use?
type promql like:
iad_n_sigma(metric_name[60m],3), make sure that the input should be a range vector.

Thank you.
There is some discussions on the mailing list right now: https://groups.google.com/g/prometheus-developers/c/ejm-NmoubX0
Could you join the discussion here?
thanks.

We discussed this during the bug scrub.
There is an overarching problem here: We still haven't come to terms when we want to add "special purpose" functions to PromQL. There is a topic on the dev-summit agenda to find out about it. Once we have clarity, we can decide how to proceed with this proposal.

Update from the dev-summit: As long as there aren't relatively easy ways of implementing a given function based on existing PromQL primitives, we are generally open to adding new functions and operators behind a feature flag (where people can play with them and we can later either promote them to stable or remove them again). There should be some credible claim of usefulness, of course, but as I understand the recent consensus from the dev summit, we want to keep the bar quite low here.

Hello again from the bug scrub. As said, we are willing to accept functions like this behind a feature flag. But no activity happened on this PR afterwards. We'll close this now, but whoever wants to pick this up, please re-open or re-create.


Subtitles not displayed if only English subtitles are available

Describe the bug
When only(?) English subtitles are available, the player doesn't show any subtitles.
To Reproduce
Steps to reproduce the behavior:

Create a vod
Add English subtitles to the database
Open vod -> no subtitle button

Expected behavior
The player should have a button that allows selecting the subtitles

Have a look at this stream and the network tab: https://live.rbg.tum.de/w/blutest/26421
The subtitle request for English is successful but the subtitle selector in the player is not available. Are the subtitles themselves faulty?

What browser/.. are you using?
(I use Chrome latest, linux)

Seems to be related to my firefox [102.8.0esr (64-bit)]. Weird, I'm not getting any related errors or warnings on the console.


[podspec] Update to v2.5.2

@ayanonagon after merging can you please create a tag and push it to cocoapods?
Thank you! :octocat:

:green_apple:


Adding tips section

PR Description
Please describe the changes proposed in the pull request:

This PR resolves #(put issue number here, and remove parentheses).

PR Checklist

Please ensure that you have done the following:

[ ] PR in from a fork off your branch. Do not PR from <your_username>:master, but rather from <your_username>:<branch_name>.

[ ] If you're not on the contributors list, add yourself to AUTHORS.rst.

Quick Check
To do a very quick check that everything is correct, follow these steps below:

[ ] Run the command make format from pandsa-log's top-level directory. This will automatically run:

black formatting
fix imports with isort

Hello @eyaltrabelsi! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file pandas_log/pandas_execution_stats.py:

Line 142:80: E501 line too long (92 > 79 characters)

In the file pandas_log/patched_logs_functions.py:

Line 33:80: E501 line too long (108 > 79 characters)
Line 34:80: E501 line too long (134 > 79 characters)
Line 35:80: E501 line too long (132 > 79 characters)

In the file setup.py:

Line 28:80: E501 line too long (187 > 79 characters)


fix(14): Add support for sonarr categories

Should fix #39

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

GitHub release
v1.5.6

Your semantic-release bot :package::rocket:


Disable password based registration

A registry administrator shall be able to set a site wide configuration option to disable password based registration.
When this is set, a user accessing the login page should only be provided with the 'log|register in via Google'

Added support for additional registry configuration properties (so more can be added without a registry war rebuild) and included a property to suppress the UI for password-based login and registration.


Deploy Chart to IBM Cloud

This PR includes a custom Github Action that deploys the main Alvearie Ingestion Helm Chart, with all of its services enabled, to the
IBM Cloud in order to validate that the code being submitted is correct and won't break the Chart installation.

LGTM
I see the 6 minute timeout - how long does it typically take to deploy (just curious how much we have padded it)

It takes about 1.5m here
The 6m timeout was from the kind cluster.


GenPOS crash with AC99 when using historical reports folder option

We added the ability to generate additional report files when performing Supervisor Mode Action Codes for AC99 (End of Day), AC 21 (Cashier Financial Report), and AC 23 (Register Financial Report). The original purpose was to all the generation of data files that would be put in the specified folder from which they would be automatically uploaded into a cloud based service such as Hubspot.
To enable the additional reports, there is an entry in the PARAMINI file with the historicalreportsfolder tag where the pathname to the folder where the reports are to be placed is specified.
When Crown Register was experimenting with this functionality, GenPOS would crash with an Entry Point Error when doing an AC99 report.
The ASSRTLOG was supplied along with the PEP and Layout Manager datafiles as well as the PARAMINI file.

After reviewing the PARAMINI file, we noticed that the historicalreportsfolder tag appeared to have a space between the end of the name of the deepest folder and the opening angle bracket for the closing tag.
C:\flashdisk\NCR\saratoga\database\reports
When we used this PARAMINI with GenPOS Rel <IP_ADDRESS>, in the Visual Studio 2019 debugger we saw the same behavior of GenPOS crashing after starting an AC99 report. The crash happened when a write was done to a file. Inspection of the FILE * value showed NULL indicating the file had not been opened properly.
We then modified the pathname to remove the space after the "reports" folder and retried.  After the change to the PARAMINI file, GenPOS did not crash, the file was opened properly, and the AC99 report was generated in the folder.
The investigation found the following issues:

GenPOS was not handling a file open error properly causing the crash
GenPOS was not logging into the ASSRTLOG that an error had occurred
GenPOS used the pathname as entered between the tags with no processing to eliminate blanks
All of the reports were not handling a file open error appropriately and this was not just an AC99 issue

What we decided was that it was conceivable that someone may want to use spaces in a pathname and that allowing this behavior was congruent with a general policy of allowing people as much freedom as possible. However having trailing spaces did not seem to have much utility so it would seem best to truncate any trailing spaces from the pathname.
We will make the necessary changes to properly handle a file open error. We will also add the additional ASSRTLOG logs to show file open errors. These changes will be done for all reports that use the historicalreports functionality.

This issue addressed with the following updates:

commit/7a67ce8b8f44b5879b9ebbcf5b472ca18e60b725
commit/790f9a34de92674b006c4aafab7f799c575fab01

The first commit addresses the problem of properly handling a file open error and improving logging.
The second commit is to process the historical reports folder pathname to remove leading and trailing spaces.


Is it possibile to get the Markdown title?

I'm using showdown to render Markdown but I also would like to access the standard Markdown title (#) to generate some custom table of contents I need.
I read about the "metadata" stuff but I don't want to force authors to use a redundant tag. The title is already there.
Is there a way? If not will you consider adding the feature?

@flaviotordini - I am not a contributor etc on this project. But hoping I can maybe provide you some insight for your problem. Correct me if wrong are you just looking to place a TOC in your markdown content already? Or build it in another place...?
If you are looking to build it in another place - can I suggest using an existing Table Of Contents generator, modifying it for your own needs to produce the output file somewhere else with the added Table Of Contents?
This is not what you need - but you could use the NPM package it uses to do this, again the script might need modifications assuming you are trying to get H1 values from what I am reading (if you need any help etc - feel free to make a comment here & I will try my best to guide you through)
MarketingPipeline/TOC-Generator-Action: A Github Action to generate a table of contents in your README
Hope this helps.

Thanks but I want to generate an external toc made of the titles (and paths) of many different Markdown documents.

@flaviotordini I figured this is what you are wanting to do. I will guide you through the steps to do this.
You will need this package or something similar (this is what I have setup in the action above)
markdown-toc-gen - npm
Insert in your Markdown Documents (the files you want to create a Table of Contents in) the following wherever you want the table of contents placed. There are options to place automatically at the start of each file keep in mind.
<-- toc --> <-- tocstop -->
then after installing markdown-toc-gen use the following command with [files] being your Markdown file paths.
markdown-toc-gen update [files..]
for more documentation on this again - read the following here. markdown-toc-gen - npm
Tho you mentioned you want to create something like the following example.
   [File Name](my_file_path/folder/File_Name.md)
   [File Name 2](my_file_path/folder/File_Name_2.md)

Assuming this is the case you will need to do some customization / scripting of your own to grab all file names & paths, place them inside a single markdown document with a separate header etc for each new line / file added ## [YOUR FILE NAME](YOUR FILE PATH), then add your <-- toc --> tags inside the document & generate the TOC, then remove all content after the generated TOCS / <-- toc --> tags.
Hope this helps & being this is kind of off topic from showdown I would suggest closing this issue. Tho I will be willing to help you if you have any questions - just prefer not spamming / using showdown issues section to discuss something somewhat off topic.
Again, hope this helps. :+1:
ps; in terms of getting the title inside the MD document, look for the first case of a H1 heading etc.

@flaviotordini - I did creep your profile too & seen you do mostly work in C++.
If you are not experienced  or confident with doing this, I can possibility make you a Python based script that does exactly what you need. Tho I can not guarantee an exact timeline on this.

@flaviotordini - was this not what you were needing..?