Add 5 hours of hop playtime requirement to captain

About the PR

Captain needs five hours of HoP playtime to unlock.
Why / Balance

Captains sometimes do not know the basics of the HoP's duties, or how to use the ID card computer. This will prevent that.
Technical details

Add five hours to the role playtime requirements for the role of Head of Personnel in the captain's prototype.
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

tweak: Added five hours of HoP playtime to the Captain's unlock requirements.

it doesnt take 5 hours to figure out the id console, and every head has one so why hop specifically

Not every and especially not on every map, but they should...
That said its not rocket science you insert one ID and then another, no need to demote anyone to HoP for 5 hours to figure that one out, you can just tell the cap two simple steps.

It's not just the ID card computer, but about not knowing the HoP's duties in general. A Captain should be familiar with that, and I think that five hours is a fine amount of time.

it doesnt take 5 hours to figure out the id console, and every head has one so why hop specifically

Most heads do not have to interact with it ever and usually delegate it to HoP when they do, also knowing what HoP actually does seems fairly important for a Captain

usually delegate it to HoP when they do

unless someone gives hop aa they cant promote randoms to your department its been like that for a while

I feel like playing HoP for 3 to 4 rounds is pretty reasonable to play the most powerful job on station

usually delegate it to HoP when they do

unless someone gives hop aa they cant promote randoms to your department its been like that for a while

Heads always just give you a stamped paper and tell you to talk to HoP, which they very much can do...?

If a person has 60 hours ingame and still doesn't know what a HoP does I fear for that person.

rather hop cant give head's accesses and armoury so hos promoting a warden has to know how to do that and youd expect cap to figure that out

it takes like one look at the id console to figure it out, not 5 hours, make it like 1 hour or something

I've played cap over 15hrs and HOP for only 15min due to late joining. I have figured out the ID console perfectly fine at this rate and I'd rather not be locked behind a role I dont want to play when I already have knowledge of how things operate.

I've played cap over 15hrs and HOP for only 15min due to late joining. I have figured out the ID console perfectly fine at this rate and I'd rather not be locked behind a role I dont want to play when I already have knowledge of how things operate.

this, ive played hop ONCE and its never come up

I think only thing that might come up is that captain does not have access configurator but HoP does, but if you check their locker once then you pretty much know all you need to.

rather hop cant give head's accesses and armoury so hos promoting a warden has to know how to do that and youd expect cap to figure that out

You should start a github discussion on this. It has nothing to do with the pr.

Get some chadmins to rise up in support but I don't think HoP is particularly hard to figure out.


JSON Decode Error

I ran into this issue this week, I saw that :mount command was not running on ranger so I decided to output into a file using sterr and got the following error
  1 Traceback (most recent call last):
  2   File "/home/user/.config/ranger/ranger_udisk_menu/menu.py", line 229, in <module>
  3     cp = ChoosePartition()
  4   File "/home/user/.config/ranger/ranger_udisk_menu/menu.py", line 34, in __init__
  5     self._read_partitions()
  6   File "/home/user/.config/ranger/ranger_udisk_menu/menu.py", line 38, in _read_partitions
  7     self.blkinfo = json.loads(r)
  8   File "/usr/lib/python3.10/json/__init__.py", line 346, in loads
  9     return _default_decoder.decode(s)
 10   File "/usr/lib/python3.10/json/decoder.py", line 337, in decode
 11     obj, end = self.raw_decode(s, idx=_w(s, 0).end())
 12   File "/usr/lib/python3.10/json/decoder.py", line 353, in raw_decode
 13     obj, end = self.scan_once(s, idx)
 14 json.decoder.JSONDecodeError: Expecting ',' delimiter: line 66 column 22 (char 1579)
~

So I decided to check the dependencies listed and I have it installed all of them also I tried to run lsblk --all --json -O  and get a good json format. I was wondering if this can be from json.load() function?

having the same issue, any luck on this?

Can confirm.  Same error, except Line 14 is slightly different.  Please lmk if you need more info from me.
json.decoder.JSONDecodeError: Expecting ',' delimiter: line 66 column 22 (char 1687)

I ran into this issue this week, I saw that :mount command was not running on ranger so I decided to output into a file using sterr and got the following error
  1 Traceback (most recent call last):
  2   File "/home/user/.config/ranger/ranger_udisk_menu/menu.py", line 229, in <module>
  3     cp = ChoosePartition()
  4   File "/home/user/.config/ranger/ranger_udisk_menu/menu.py", line 34, in __init__
  5     self._read_partitions()
  6   File "/home/user/.config/ranger/ranger_udisk_menu/menu.py", line 38, in _read_partitions
  7     self.blkinfo = json.loads(r)
  8   File "/usr/lib/python3.10/json/__init__.py", line 346, in loads
  9     return _default_decoder.decode(s)
 10   File "/usr/lib/python3.10/json/decoder.py", line 337, in decode
 11     obj, end = self.raw_decode(s, idx=_w(s, 0).end())
 12   File "/usr/lib/python3.10/json/decoder.py", line 353, in raw_decode
 13     obj, end = self.scan_once(s, idx)
 14 json.decoder.JSONDecodeError: Expecting ',' delimiter: line 66 column 22 (char 1579)
~

So I decided to check the dependencies listed and I have it installed all of them also I tried to run lsblk --all --json -O  and get a good json format. I was wondering if this can be from json.load() function?

same error here but (char 1686)

Traceback (most recent call last):
  File "/home/user/.config/ranger/ranger_udisk_menu/menu.py", line 229, in <module>
    cp = ChoosePartition()
  File "/home/user/.config/ranger/ranger_udisk_menu/menu.py", line 34, in __init__
    self._read_partitions()
  File "/home/user/.config/ranger/ranger_udisk_menu/menu.py", line 38, in _read_partitions
    self.blkinfo = json.loads(r)
  File "/usr/lib/python3.10/json/__init__.py", line 346, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python3.10/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python3.10/json/decoder.py", line 353, in raw_decode
    obj, end = self.scan_once(s, idx)
json.decoder.JSONDecodeError: Expecting ',' delimiter: line 66 column 22 (char 1633)

Same error, slightly different character (1633). The error persists on different Python versions (tested 3.7, 3.8, 3.10). Is the character referring to the output of lsblk --all --json -O? I went to char 1633 in the output of this command, but it looks like normal json to me.
If I edit out the -O in the menu.py call to lsblk, the two commands in question run fine.
import json; import subprocess
r = subprocess.check_output(['lsblk', '--all', '--json'])
json.loads(r)

However, ranger_udisk_menu relies on some of the information from -O which I've yet to investigate.

I also had this issue recently and found a quick and dirty fix.
The JSON output from lsblk seems to have some issues that interfere with the json.loads() function. In my case, also on line 66, column 22, it had to do with 0B not being a string.
 "zone-sz": 0B,
By doing a direct replace on the lsblk output for 0B on menu.py, the JSON is parsed and the menu appears.
def _read_partitions(self):
    r = subprocess.check_output(['lsblk', '--all', '--json', '-O'])
    r = r.decode().replace('0B,', '\"0B\",')
    self.blkinfo = json.loads(r.encode())

I've tested it with an external disk and usb drive and it works as expected.

@Shinkirou Good find! I think this is the fix. Do you want to PR it?

But can that value be something else then 0B, for example 5B?


[REST] Logging in with an existing auth token

When opening the Pandora Web client, after having signed in already, it seems to log In with a request like the following, with an empty password string and a populated existingAuthToken field:
POST https://www.pandora.com/api/v1/auth/login

{
  "existingAuthToken": "<token>",
  "username"<EMAIL_ADDRESS>  "password": "",
  "keepLoggedIn":true
}

The existing auth token is always the one that was received when logging in previously.
Upon replicating this request, however, I get the standard invalid credential error:
{
  "message": "Invalid username and/or password",
  "errorCode": 0,
  "errorString": "AUTH_INVALID_USERNAME_PASSWORD"
}

Does anyone know what's going on here?

Closing as this is no longer relevant (#45)

{
"message": "The request could not be validated",
"errorCode": 0,
"errorString": "INVALID_REQUEST"
}
how can i get existing auth token?

{
"existingAuthToken": null,
"keepLoggedIn": true,
"password": "dfssdf345",
"username"<EMAIL_ADDRESS>}
{
"message": "The request could not be validated",
"errorCode": 0,
"errorString": "INVALID_REQUEST"
}
how can i get existing auth token?

The only way now is through the JSON API

Ok, do you have skype?

@hacker1024 is there a basic example for User Login via the JSON API using something like CURL? I just want to make some simple requests to workaround this issue (migrating from REST API) and the docs aren't very clear to me.


refactor(vala): refine vala project root detection

take https://github.com/phw/peek or https://github.com/benwaffle/vala-language-server for example,
git clone https://github.com/benwaffle/vala-language-server
cd vala-language-server
nvim

we opened src/server.vala then the lsp will run meson build under the wrong dir src/
because the src dir has a file named meson.build which actully is not a real meson.build,
it is included by the root meson.build, and the lsp server will complain the "ERROR: First statement must be a call to project"
this PR:

detect the valid meson.build file by detection first line include project( detective or not
2. prefer cwd if root is a descendant, otherwise fallback to git repo

I'm afraid there's no tool to use to report the workspace root.
according to https://wiki.gnome.org/Projects/Vala/Tools

Meson - a front end to the Ninja build system with Vala support (The main build system for Vala)

and ref to https://mesonbuild.com/Syntax.html#comments

A comment starts with the # character and extends until the end of the line.

and also the meson.build file "First statement must be a call to project"

split the logic part into meson_matcher()

Thanks!


Fixed typo in string formatting for listmembership creation.

In listmembership creation url string was the email parameter, meanwhile in the format arguments no email parameter was specified but list_id was. According to docs http://developer.pardot.com/kb/api-version-4/list-memberships/ (see create operation) list_id is the right one.

Thank you @bernardito-luis !


Honeywell alarm panel/TCC

Home Assistant release with the issue:

0.86.2
Last working Home Assistant release (if known):
/
Operating environment (Hass.io/Docker/Windows/etc.):

Debian 8
Component/platform:

ReadyNAS OS 6.9.5
Honeywell EvoHome Security system (International Total Connect Comfort)
Description of problem:
Can't get the alarm panel/security component to work. Heathing works perfectly though.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
alarm_control_panel:
  - platform: totalconnect
    username: xxx
    password: xxx

Traceback (if applicable):
Traceback (most recent call last):
  File "/apps/homeassistant/lib/python3.5/site-packages/homeassistant/helpers/entity_platform.py", line 128, in _async_setup_platform
    SLOW_SETUP_MAX_WAIT, loop=hass.loop)
  File "/usr/local/lib/python3.5/asyncio/tasks.py", line 406, in wait_for
    return fut.result()
  File "/usr/local/lib/python3.5/asyncio/futures.py", line 294, in result
    raise self._exception
  File "/usr/local/lib/python3.5/concurrent/futures/thread.py", line 55, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/apps/homeassistant/lib/python3.5/site-packages/homeassistant/components/alarm_control_panel/totalconnect.py", line 40, in setup_platform
    total_connect = TotalConnect(name, username, password)
  File "/apps/homeassistant/lib/python3.5/site-packages/homeassistant/components/alarm_control_panel/totalconnect.py", line 57, in __init__
    username, password)
  File "/apps/homeassistant/lib/python3.5/site-packages/total_connect_client/TotalConnectClient.py", line 56, in __init__
    self.authenticate()
  File "/apps/homeassistant/lib/python3.5/site-packages/total_connect_client/TotalConnectClient.py", line 68, in authenticate
    raise AuthenticationError('Unable to authenticate with Total Connect')
total_connect_client.TotalConnectClient.AuthenticationError: Unable to authenticate with Total Connect

Additional information:

@MaartenProvo from your logs:
total_connect_client.TotalConnectClient.AuthenticationError: Unable to authenticate with Total Connect

You may be using wrong credentials, since you mentioned heating, I just want to clarify that TCC - Total Connect Comfort is different than TC2 - Total Connect 2.0.
ATM, the total_connect alarm_control_panel can only control an alarm connected to TC2.
Please make sure you can login to https://totalconnect2.com with the credentials you are using in your config.

this is outside the scope of home assistant at this time.  Either the total-connect-client needs to be updated as @kkr16 mentioned or a new dependency needs to be added for this platform to work.

@robbiet480 i think this can be closed.  it's not an issue with home assistant and needs dependency work (or a new dependency altogether).


Auto completion doesn't work as expected

I found Pylance when writing python, it is very pretty. However Auto-completion doesn't work as expected, Take an example of numpy, here's Jedi performs

Jedi tells me there is a function called sin(), but Pylance doesn't.

I have below extensions related to python: Pylance, pylint, python.
And I found a similar issue  https://github.com/microsoft/pylance-release/issues/787#issue-775813882

This should be fixed in the next release with numpy 1.20 installed:

This issue has been fixed in version 2021.4.1, which we've just released. You can find the changelog here: https://github.com/microsoft/pylance-release/blob/main/CHANGELOG.md#202141-14-april-2021


How to use LAVIS to video retrieval?

I want to use this code base to test the performancc ALPRO on MSRVTTT, but I did not find the tutorial of how to process video and do a text to video retrieval.

Have you found the answer?


Removed toLowerCase from naughtyHref

Removed toLowerCase from naughtyHref to enable CAPITAL , lower and MiXed allowedSchemes are allowed.
This was one of the blocker for me.

How was this a blocker for you? Chrome treats the scheme as case insensitive. We would miss that and reject it if we did not do this.

How was this a blocker for you? Chrome treats the scheme as case insensitive. We would miss that and reject it if we did not do this.

The problem is for our group code we have several custom protocols in a href tag like
CUST1://
CuSt2://
company1://
ComPany2://
instead of http or https:// for mobile applications so that users when clicked on links will redirect to their own respective APPS.
I hope i made sense.
Regards

@boutell Do you think there's reason to carry this forward?

It's not pressing, but if the OP is interested in devising a solution where we test case insensitively for all protocols not explicitly marked as case sensitive, we could look at that.


hg19 fasta

I was wondering which hg19 fasta is need for mutational plots
Thanks a lot!
Zayni

Hi,
All hg19 are same, except Ensemble doesn't use 'chr' prefix whereas ucsc does. See argument prefix while running  trinucleotideMatrix to add or remove prefixes.

Thanks a lot ! got it
Also, prepareMutsig functions is not working . Am I doing anything wrong ?
"prepareMutSig(maf = laml)
Error: could not find function "prepareMutSig"""

Hi,
Which version of maftools are you using ? Its on 1.2.0 on Bioconductor. Try updating it.


[FEATURE] Automatically skip some slide

Discussed in https://github.com/jeertmans/manim-slides/discussions/301
The suggestion is to add some parameter, e.g., auto_skip=True, to Slide.next_slide to that some slide may be skipped automatically.
TODOs

[ ] Check compatibility with loop=True, and maybe prevent that both are equal to True
[ ] Add a new option to the SlideConfig and document it
[ ] Watch for EndOfMedia with present and skip if needed
[ ] Check if this can be supported by RevealJS and/or PowerPoint?

Originally posted by Adcombrink October 25, 2023
Loving manim-slides so far, great work. However, I've been trying to do something and am starting to think it's not possible, maybe a new feature. I want to click to the next slide, have an animation play, and then loop another animation, all with only clicking the first time. Right now, if i write
self.next_slide()

self.play(... initial animations ...)
self.start_loop()
self.play(... looping animations ...)
self.end_loop()

then it instantly jumps to the looping part, as if the initial animations have already run.

Nice! Thanks for the good work!


Build for 2.13

When running tests against 2.13, I noticed an exception was thrown in ConsumerRecords when someone tries to build one from an empty collection. This should never happen in real life, but it may in tests. So for that we replace with maxByOption.

Codecov Report

Merging #4 (3ce719c) into master (63b7cfe) will increase coverage by 0.15%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
+ Coverage   66.66%   66.82%   +0.15%
==========================================
  Files          16       16
  Lines         633      627       -6
  Branches       33       29       -4
==========================================
- Hits          422      419       -3
+ Misses        211      208       -3

Impacted Files
Coverage Δ

...ala/cakesolutions/kafka/akka/TrackPartitions.scala
85.71% <ø> (ø)

...main/scala/cakesolutions/kafka/KafkaConsumer.scala
77.77% <ø> (ø)

...main/scala/cakesolutions/kafka/KafkaProducer.scala
80.48% <ø> (ø)

...scala/cakesolutions/kafka/KafkaSerialization.scala
100.00% <ø> (ø)

...cakesolutions/kafka/TypesafeConfigExtensions.scala
55.55% <ø> (ø)

...cala/cakesolutions.kafka/testkit/KafkaServer.scala
90.00% <ø> (+1.26%)
:arrow_up:

...ala/cakesolutions/kafka/akka/ConsumerRecords.scala
62.96% <100.00%> (+3.70%)
:arrow_up:

.../cakesolutions/kafka/akka/KafkaConsumerActor.scala
56.98% <100.00%> (-0.63%)
:arrow_down:

.../main/scala/cakesolutions/kafka/akka/Offsets.scala
77.77% <100.00%> (ø)

... and 3 more

Continue to review full report at Codecov.

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

Looks good to me


[TW#13568] Calling mbedtls_x509_crt_free twice sometimes causes crash.

I have seen this issue on multiple occasions, especially in some arduino libs it may happen that mbedtls_x509_crt_free gets called twice in a row. Since there are multiple null checks I don't get why that crash is happening. Also it only happens on really rarer occasions. All my HTTPS-related tasks run pinned to core 1.
Crash log:
Guru Meditation Error of type LoadProhibited occurred on core  1. Exception was unhandled.
Register dump:
PC      : 0x40159c5c  PS      : 0x00060b30  A0      : 0x8013b41d  A1      : 0x3ffd6710
A2      : 0x3ffbe85c  A3      : 0x00000001  A4      : 0x3ffbe85c  A5      : 0x3f4108ec
A6      : 0x00000001  A7      : 0x3ffb1df8  A8      : 0x80159c16  A9      : 0x3ffd66f0
A10     : 0x00000000  A11     : 0x40082660  A12     : 0x3f418508  A13     : 0x00001bb3
A14     : 0x3f418d80  A15     : 0xffffffaf  SAR     : 0x00000004  EXCCAUSE: 0x0000001c
EXCVADDR: 0x0000000d  LBEG    : 0x4000c46c  LEND    : 0x4000c477  LCOUNT  : 0x00000000

Backtrace: 0x40159c5c:0x3ffd6710 0x4013b41d:0x3ffd6730 0x4013b13c:0x3ffd6750 0x4013b14e:0x3ffd6770 0x4013b165:0x3ffd6790 0x40138d49:0x3ffd67b0 0x401394e1:0x3ffd67e0 0x40139586:0x3ffd6820 0x4011f33f:0x3ffd6840 0x4011f450:0x3ffd68b0 0x4011f4c8:0x3ffd68e0

that translates to:
mbedtls_x509_crt_free
/home/ubuntu/workspace/esp/esp-idf/components/mbedtls/library/x509_crt.c:2350
_Z15stop_ssl_socketP17sslclient_contextPKcS2_S2_
/home/ubuntu/workspace/esp/logger-firmware/components/arduino/libraries/WiFiClientSecure/src/ssl_client.cpp:208
_ZN16WiFiClientSecure4stopEv
/home/ubuntu/workspace/esp/logger-firmware/components/arduino/libraries/WiFiClientSecure/src/WiFiClientSecure.cpp:196
_ZN16WiFiClientSecureD2Ev
/home/ubuntu/workspace/esp/logger-firmware/components/arduino/libraries/WiFiClientSecure/src/WiFiClientSecure.cpp:196
_ZN16WiFiClientSecureD0Ev
/home/ubuntu/workspace/esp/logger-firmware/components/arduino/libraries/WiFiClientSecure/src/WiFiClientSecure.cpp:196
_ZNSt10unique_ptrI10WiFiClientSt14default_deleteIS0_EED4Ev
/home/ubuntu/workspace/esp/logger-firmware/components/arduino/libraries/HTTPClient/src/HTTPClient.cpp:308
_ZN10HTTPClient11sendRequestEPKcPhj
/home/ubuntu/workspace/esp/logger-firmware/components/arduino/libraries/HTTPClient/src/HTTPClient.cpp:308
_ZN10HTTPClient3GETEv
/home/ubuntu/workspace/esp/logger-firmware/components/arduino/libraries/HTTPClient/src/HTTPClient.cpp:308

This is the particular line of code that seams to be failing (the last one):
        while( seq_cur != NULL )
        {
            seq_prv = seq_cur;
            seq_cur = seq_cur->next;

Surely this can be fixed "upstream" in the HTTPS-libs but I don't think that calling this function twice is supposed to cause a crash.

Thanks for reporting this with so much detail, @Curclamas . Sorry it's taken a while to get back to you.
I've spent some time investigating, and I don't think this is a bug in IDF or mbedTLS. I'm not able to reproduce (I've attached a modified version of the IDF HTTPS example to this comment), and I can't see how this situation could happen during correct usage of the library:
The crash in seq_cur = seq_cur->next happens trying to read from memory address 0x0000000d (the EXCVADDR in the dump). The offset of the next field in seq_cur is 0xc, so this implies the value of the seq_cur pointer is 1 (which explains why the null check passes).
The only way this pointer can have (invalid) value 1 is if the cert_cur->ext_key_usage.next field value is 1, or if one of the previous certificate's next pointer has value 1. Both of these structured are zeroed out by mbedtls_x509_crt_free.
The most likely underlying reason for this is memory corruption.
It could happen if an mbedtls_x509_crt structure (or similar) is associated with the ssl context via mbedtls_ssl_conf_ca_chain() or mbedtls_ssl_conf_own_cert(), but then the structure is either freed or goes out of scope without calling mbedtls_ssl_conf_ca_chain(ssl, NULL). Now the ssl context has a pointer to freed memory. However this shouldn't happen in Arduino ssl_client, as the ssl_client and all of the cert structures are part of the same class' fields.
Another possible explanation is regular memory corruption due to some other function writing into the certificate's memory space between the first call to mbedtls_crt_free() and the second one.
A final possible explanation is if the ssl structure or any CA cert structures is shared between multiple tasks, as mbedTLS is not thread-safe without additional lock protections. This would include using the same Arduino WifiClientSecure object from multiple tasks.
I'm going to close this issue, but please reopen if you're certain there's a bug in IDF. A reproducible test case of any kind (Arduino or IDF app) would be very helpful to track this down.
Angus


Session Persistence does not maintain with Scatter

As I am not sure if the problem is with the ual-plainjs-renderer (https://github.com/EOSIO/ual-plainjs-renderer/issues/31) or the ual-scatter package, I'll also post this issue here:
When I login with the MockAuthenticator and then refresh the page it automatically logs me back in with the ual-plainjs-renderer. When I try to do the same with scatter, it gives the following error after refreshing the page and I have to manually login again with scatter:
Scatter.js:103 Uncaught (in promise) UALScatterError: Unable to login
    at Scatter.eval (webpack:///./node_modules/ual-scatter/dist/Scatter.js?:103:23)
    at Generator.next (<anonymous>)
    at eval (webpack:///./node_modules/ual-scatter/dist/Scatter.js?:7:71)
    at new Promise (<anonymous>)
    at __awaiter (webpack:///./node_modules/ual-scatter/dist/Scatter.js?:3:12)
    at Scatter.login (webpack:///./node_modules/ual-scatter/dist/Scatter.js?:93:16)
    at UALJs.eval (webpack:///../dist/UALJs.js?:98:45)
    at Generator.next (<anonymous>)
    at eval (webpack:///../dist/UALJs.js?:7:71)
    at new Promise (<anonymous>)

What is the best way to remain logged in after refreshing the page with scatter?

Closing this issue as it is not an authenticator issue, but rather a renderer one.
This issue is covered in https://github.com/EOSIO/ual-plainjs-renderer/issues/31


Don't allow bot users to be assigned to issue/pull

Feature Description
Bot users are hidden in gitea, so why would we see them in issues "assignees" filter and be able to assign issues to them?
Screenshots

I see nothing wrong with assigning a bot an issue, if for example the bot is a merger bot, this may be desirable to get the bot to merge it.

Furthermore, as @silverwind mentioned, there might be bots that explicitly listen for issues they are assigned for.
That is an absolutely permitted usecase.
So, this would even be breaking.

Bot users are hidden in gitea, so ...

wonder what's the meaning of Bot users ? ...

https://github.com/go-gitea/gitea/pull/24026

Furthermore, as @silverwind mentioned, there might be bots that explicitly listen for issues they are assigned for. That is an absolutely permitted usecase. So, this would even be breaking.

I thought that'll be done using labels

Assign action could ask the bot user do something, why not?

Assign action could ask the bot user do something, why not?

I thought assignee means human to resolve an issue (code something, etc.)

Yes, too.
But that's the funny thing about software:
You can only guide the user on how to use it, you cannot forbid unforeseen uses.


docs updated

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1335/


Vickel: Watch ROOTKIT[\W_]+HACKER?(?:[\W_]+FIRM)?

Vickel requests the watch of the watch_keyword ROOTKIT[\W_]*+HACKER?(?:[\W_]*+FIRM)?. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

ROOTKIT[\W_]*+HACKER?(?:[\W_]*+FIRM)? has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Rejected by Spevacus in Charcoal HQ. Reason: 'By request'


Cluster Greenhouse map generation tweeks

There are some bugs in the map generation.

@bauerbp can you ist out some of the bugs so we know what to review against?

A known bug includes clusters crawling up the edge of the graph (fix is pushed to i105).  Other bugs are waiting to be found.

@bauerbp
Does chooseClusterToAddTo need to be public?
can we give that a better name? getClusterToGrow? chooseClusterToGrow?
Can we add these changes as part of this issue?

@bauerbp
Cluster and ClusterGreenhouse should follow the rule of 5.
As part of this issue can you add the necessary constructors / assignment operators to satisfy this please?

Were not going to touch this since were refactoring the system


Support crossjoin in substrait.

Which issue does this PR close?

ref: #8149
Closes #.
Rationale for this change

part of #8149.
What changes are included in this PR?

Translate cross product  to LogicalPlan::CrossJoin.
Are these changes tested?

Are there any user-facing changes?

🎉


Updates the billing plan view to match the pricing page on www

Closes https://github.com/aptible/dashboard.aptible.com/issues/389
There are some interesting things to consider with displaying custom allowances in the “upgrade to a PHI-ready plan!” scenario from a dev plan. The BillingDetail model’s *InPlan computed properties will
return zero if the current plan is Development (https://github.com/aptible/ember-cli-aptible-shared/blob/master/app/models/billing-detail.js#L67). That’s not what we want to display in this context. The *Allowance properties should always be non-zero (https://github.com/aptible/billing.aptible.com/blob/master/app/models/billing_detail.rb#L38) and set to the proper defaults, but just in case, I added a ternary helper to avoid null and 0 values showing up in
that upgrade plan.
Overkill? Possibly. Poorly named? Also likely… I tried if-else and if-null, but they felt too specific.
¯\_(ツ)_/¯
Added a helper to display GBs in TBs—to match the pricing page.

Nice work @gib!  One minor complication -- this implementation works because we only have 3 plans. However, if we add a 4th Pilot plan, this implementation could get sticky, especially if we have different allowances for Pilot as we do Platform.  The fix would be to only show the billingDetail's allowances if the plan is currently selected. For each plan that isn't selected, allowances can be hard coded to their defaults for that plan.

We also have those on BillingDetail.

But... we can't access them. Do we expose any globals like
https://github.com/aptible/ember-cli-aptible-shared/blob/master/app/models/billing-detail.js#L16-L35
Or how would we do that.
I'll add a current plan check for now.

@gib -- Correct, because all of those const are exported, you can import them into your controller with:
import { CONTAINER_ALLOWANCES, DISK_ALLOWANCES, DOMAIN_ALLOWANCES } from 'diesel/models/billing-detail';

Shows how the allowances are used for non-dev plans.

:+1:


MINOR: Remove unnecessary log4j-appender dependency and tweak explicit log4j dependency

Remove log4j-appender dependency from tools, trogdor and shell
Remove explicit log4j dependency from trogdor and tools.
Add compileOnly dependency from tools to log4j (same approach as
core).

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@cmccabe is there any reason why shell and trogdor need log4j-appender?

@omkreddy maybe you can help review this.

@ijuma  sorry missed this. Can you rebase the PR.

@omkreddy Rebased.

Remove log4j-appender dependency from tools, trogdor and shell

After addressing that, the distribution does not include log4j-appender jar anymore.
chia7712@ubuntu00:~/project/kafka/core/build/distributions$ ls kafka_2.13-3.9.0-SNAPSHOT/libs | grep kafka
kafka_2.13-3.9.0-SNAPSHOT.jar
kafka-clients-3.9.0-SNAPSHOT.jar
kafka-group-coordinator-3.9.0-SNAPSHOT.jar
kafka-group-coordinator-api-3.9.0-SNAPSHOT.jar
kafka-metadata-3.9.0-SNAPSHOT.jar
kafka-raft-3.9.0-SNAPSHOT.jar
kafka-server-3.9.0-SNAPSHOT.jar
kafka-server-common-3.9.0-SNAPSHOT.jar
kafka-shell-3.9.0-SNAPSHOT.jar
kafka-storage-3.9.0-SNAPSHOT.jar
kafka-storage-api-3.9.0-SNAPSHOT.jar
kafka-streams-3.9.0-SNAPSHOT.jar
kafka-streams-examples-3.9.0-SNAPSHOT.jar
kafka-streams-scala_2.13-3.9.0-SNAPSHOT.jar
kafka-streams-test-utils-3.9.0-SNAPSHOT.jar
kafka-tools-3.9.0-SNAPSHOT.jar
kafka-tools-api-3.9.0-SNAPSHOT.jar
kafka-transaction-coordinator-3.9.0-SNAPSHOT.jar

not sure whether it is expected? KafkaLog4jAppender is deprecated now, but we should keep it in release?
For another, the system test kafka_log4j_appender.py gets failed due to same reason.

Hence, we have following approaches to fix above issues.

add log4j-appender dependency to core module. this can fix issue_1 and issue_2
add log4j-appender dependency to copyDependantTestLibs, and then make kafka_log4j_appender.py expose new classpath to include to log4j-appender jar. this can fix issue_2.

@ijuma WDYT? I prefer approach_1 as it keep the compatibility.


[Bug]: Missing class properties transform

Contact Details
<EMAIL_ADDRESS>What happened?
I am not able to build since the last update.

tsParticles Version
2.1.3
Which library are you using?
React.js (react-particles, react-tsparticles, react-particles-js)
tsParticles Configuration
<Particles
          height={"calc(100vh - 142px)"}
          width="100%"
          params={{
            particles: {
              number: {
                value: 355,
                density: {
                  enable: true,
                  value_area:<PHONE_NUMBER>322727,
                },
              },
              color: {
                value: "ffffff",
              },
              shape: {
                type: "circle",
                stroke: {
                  width: 0,
                  color: "#000000",
                },
                polygon: {
                  nb_sides: 5,
                },
              },
              opacity: {
                value: 0.48927153781200905,
                random: false,
                anim: {
                  enable: true,
                  speed: 0.2,
                  opacity_min: 0,
                  sync: false,
                },
              },
              size: {
                value: 2,
                random: true,
                anim: {
                  enable: true,
                  speed: 2,
                  size_min: 0,
                  sync: false,
                },
              },
              line_linked: {
                enable: false,
                distance: 150,
                color: "#ffffff",
                opacity: 0.4,
                width: 1,
              },
              move: {
                enable: true,
                speed: 0.2,
                direction: "none",
                random: true,
                straight: false,
                out_mode: "out",
                bounce: false,
                attract: {
                  enable: false,
                  rotateX: 600,
                  rotateY: 1200,
                },
              },
            },
            interactivity: {
              detect_on: "canvas",
              events: {
                onhover: {
                  enable: true,
                  mode: "bubble",
                },
                onclick: {
                  enable: true,
                  mode: "push",
                },
                resize: true,
              },
              modes: {
                grab: {
                  distance: 400,
                  line_linked: {
                    opacity: 1,
                  },
                },
                bubble: {
                  distance: 83.91608391608392,
                  size: 1,
                  duration: 3,
                  opacity: 1,
                  speed: 3,
                },
                repulse: {
                  distance: 200,
                  duration: 0.4,
                },
                push: {
                  particles_nb: 4,
                },
                remove: {
                  particles_nb: 2,
                },
              },
            },
            retina_detect: true,
          }}
        />

What browsers are you seeing the problem on?
No response
Relevant log output
Creating an optimized production build...
Failed to compile.

./node_modules/tsparticles-engine/esm/Core/Utils/Circle.js
SyntaxError: /Users/dndeveloper/Projects/Professional/PuStack/pustack-web-platform/node_modules/tsparticles-engine/esm/Core/Utils/Circle.js: Missing class properties transform.
  2 | import { getDistance } from "../../Utils/NumberUtils";
  3 | export class Circle extends Range {
> 4 |     radius;
    |     ^^^^^^^
  5 |     constructor(x, y, radius) {
  6 |         super(x, y);
  7 |         this.radius = radius;

Code of Conduct

[X] I agree to follow this project's Code of Conduct

You typed version 2.1.3, but the latest version is 2.4.0. Do you get the error even with the latest version? How do I reproduce the error? transform is not even something coming from tsParticles classes

We are using 2.1.3 version of react-tsparticles in our project since last year and it was working like a charm till yesterday. But since last evening, it is showing me this error. Can you help me with which kind of babel-loader I need?

@dndeveloper-saurabh I need a way to reproduce this issue. My demo project is building like always.

@matteobruni This is the sample repository, the project code and related configs are removed but the package.json is the same.
As soon as I comment out the import statement of react-tsparticles in App.js, the app works.
https://github.com/dndeveloper-saurabh/tsparticles-bug-demo

@dndeveloper-saurabh I think this should help: https://stackoverflow.com/questions/35517245/error-missing-class-properties-transform
I'm going to fix this on my side in 2.5.0. I try to release a fix later, since it's breaking builds. I'm going to change the ESM output to ES2017, I thought that ESNext would be fine, but it seems it wasn't a good idea.


Created CancerSpatialStatics v4

Fixed the KS Test, added back in AD test, added in Generalized Kullback-Leibler, Chebyshev, Hamming, Jensen-Shannon divergences/distances. There are some issues though and I can detail those on slack. :) Lots of progress though.
Cherry Picked from @camcas-PhDstudent (c68cbc5dfc29c2b3b6a85914aa4907ac45803c50) on PR #4

Closed in favor of #6


Layout

[x] Show Methods and Notes sections if available
[ ] Picture size and layout
[x] If ingredients are in multiple lists - for example "base" and "filling"
[x] Add more sections
[x] Make quantity/size/temp/time conditional too

Moving pictures size and layout to Issue #5.


Add route configuration options for apollo server path

This was the entire reason of my original PR against the apollo server, but need to get this passed through on the new AS integration. Should be simple pass through like the path.

Fixed in 1.0.1


Proper SvgTextBase.Text property setter behaviour

SvgTextBase.Text property setter resets Node (and Children) collection to have single content node with provided text.
Text elements must have at least one node to be serialized. Serializer is ignoring Content property of text elements, so Text property resets any previous text (by removing content Nodes) and creates a new node containing provided text.
fixes #259
fixes #254
fixes #151

that's quite some magic going on. thanks!

@tebjan This is tricky thing. It is a temporary solution, we have to do something with Content property, which is not suitable for all classes we have there. And Nodes and Children collections are kind of redundant.


Bring down https://datacommons.readthedocs.io/en/stable-1.x/started.html

The old documentation still shows up on google search -- it's confusing and obsolete. We should delete the source and bring down the site.

The old documentation link mentioned is not accessible now.


Allow non-square surfaces

It would be nice if plot_surface would accept arbitrary rectangular matrices X, Y and Z. Currently it seems that plot_surface behaves reasonably only when X, Y and Z are square matrices. Indeed, slightly tweaking the Surface example
# f(u, v) -> (u, v, u*v**2)
a = np.arange(-5, 5)
b = np.arange(-10, 10)
U, V = np.meshgrid(a, b)
X = U
Y = V
Z = X*Y**2
p3.figure()
p3.plot_surface(X, Z, Y, color="orange")
p3.plot_wireframe(X, Z, Y, color="black")
p3.show()

yields the following error:
---------------------------------------------------------------------------
IndexError                                Traceback (most recent call last)
<ipython-input-235-3b8e6be7aa92> in <module>()
      7 Z = X*Y**2
      8 p3.figure()
----> 9 p3.plot_surface(X, Z, Y, color="orange")
     10 p3.plot_wireframe(X, Z, Y, color="black")
     11 p3.show()

~/anaconda/envs/pcp-env/lib/python3.5/site-packages/ipyvolume/pylab.py in plot_surface(x, y, z, color, wrapx, wrapy)
    231     :return: :any:`Mesh`
    232     """
--> 233     return plot_mesh(x, y, z, color=color, wrapx=wrapx, wrapy=wrapy, wireframe=False)
    234
    235

~/anaconda/envs/pcp-env/lib/python3.5/site-packages/ipyvolume/pylab.py in plot_mesh(x, y, z, color, wireframe, surface, wrapx, wrapy, u, v, texture)
    343             p3 = index_from2d(i + 1, j + 1)
    344             triangle_index = (i * mx) + j
--> 345             triangles[triangle_index * 2 + 0, :] = [p0, p1, p3]
    346             triangles[triangle_index * 2 + 1, :] = [p0, p3, p2]
    347             lines[triangle_index * 4 + 0, :] = [p0, p1]

IndexError: index 342 is out of bounds for axis 0 with size 342

If instead we set
b = np.arange(-3, 3)

then no error is thrown but a seemingly incorrect triangulation is produced:

Ok, this should work now, thanks for reporting this!

Hi,
I was bitten by exactly the same problem and opened a bug report that basically states the same thing than the above (here #84). However, it seems this fix has not been uploaded to Pypi and the conda channel (since I installed from there).
Would it be possible to make a new release with commit 5943d6a included?
Thanks in advance,
Florian


Login and signup forms

Done with the signup and login forms, Please review

Good Job Boss. You could create a branch for your commit if you are pulling from your own repo so that it can easily be compared to main.


text selector bug

https://github.com/flutter/flutter/assets/96408557/1183ad33-f796-4d1a-8e01-19dd330e7f00
When the line length of the text in the textfield is greater than 1, the text selector is displayed on the appbar when scrolling. how to fix this

Closing this as a duplicate of https://github.com/flutter/flutter/issues/13182. Please follow up on that issue for further updates.


Steps: Infinite loop when I wait was first step

Just ran into an infinite loop when I put the And I wait _ seconds before loading the interview. I think it's because the page hadn't been created yet, but I'm not sure. Will it happen with other Steps? How can we protect people from that?

I think this was solved in a merge a week or so back. It skips setting page timeout if needed so it wouldn't get stuck before a page was loaded, then it sets timeout (again sometimes) on the page loading step so that we make sure it really is set for the page.


xsettingsd: 1.0.0 -> 1.0.2

Motivation for this change
Also use cmake as it means we have to patch less and get more things done for us automatically.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 112609 at d6a82ff0 run on x86_64-linux 1

2 packages marked as broken and skipped:

libsForQt512.kde-gtk-config
libsForQt514.kde-gtk-config

2 packages built:

libsForQt5.kde-gtk-config (libsForQt515.kde-gtk-config ,plasma5Packages.kde-gtk-config)
xsettingsd

Result of nixpkgs-review pr 112609 at d6a82ff0 run on aarch64-linux 1

2 packages marked as broken and skipped:

libsForQt512.kde-gtk-config
libsForQt514.kde-gtk-config

2 packages built:

libsForQt5.kde-gtk-config (libsForQt515.kde-gtk-config ,plasma5Packages.kde-gtk-config)
xsettingsd


How about an embed for Rumble?

Rumble is the biggest competitor for Youtube.

I haven’t heard of Rumble, but if there’s a good way to build an embed component a PR would be welcome.
We have a guide for how to add a new component that should be helpful. Happy to answer questions if things aren’t clear.

Programmatic Rumble embedding is unfortunately not as straight-forward as Youtube embedding. I've started on this though, I can't imagine it will take more than a few more days to have something to push.

I have issues with a package 'rumble-core' which needs to be imported which is clashing with the strict typescript settings and objecting to 'any' type. I now remember why I always hated typescript so much.

I have a working example but can't get it working in astro-embed because of the typescript restrictions clashing with a 'rumble-core' package. I now remember one of the reasons why I always hated typescript so much.

I have a working example but can't get it working in astro-embed

Feel free to open a draft PR and I’ll try to see if I can help out when I have time — thanks!


Fixed example

Replaced assignment operator with the equality operator.
Fixed example structure.
Improved language.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.


Ability to pass more information to callbacks.

Key changes:

Breaking change: int requestCode replaced by BaseDialogFragment dialog.
Added BaseDialogBuilder.setExtras(Bundle).
Added BaseDialogFragment.getRequestCode().

You can now handle more information about dialog in callbacks. For example you ask user to confirm delete item from list. Now you can pass item id in extras and then get it from dialog.getArguments().getBundle(BaseDialogBuilder.ARG_EXTRAS).

I believe, there is better solution than creating Breaking change. For instance new interface with extras method argument.


Port indices notation in validation errors is not consistent

In some cases Marathon will use parentheses to index validation failures, in other cases it will use square brackets.
Paths sometimes have a leading slash, but they sometimes don't.
Paths are sometimes separated by slashes, sometimes by periods.

Some examples:
{
    "details": [
        {
            "errors": [
                "Constraint definition must be an array of strings in format: <key>, <operator>[, value]"
            ],
            "path": "/constraints(0)"
        }
    ],
    "message": "Invalid JSON"
}

{
    "details": [
        {
            "errors": [
                "URI has invalid syntax."
            ],
            "path": "fetch[0]"
        }
    ],
    "message": "Object is not valid"
}

{
    "details": [
        {
            "errors": [
                "error.path.missing"
            ],
            "path": "/fetch(0)/uri"
        }
    ],
    "message": "Invalid JSON"
}

{
    "details": [
        {
            "errors": [
                "got -1, expected 0 or more"
            ],
            "path": "portDefinitions[0].port"
        }
    ],
    "message": "Object is not valid"
}

The schema should be consistent. Until v0.15.0 we were always using parentheses, I think we should go back to that.
To find the affected validations, look for usages of the every validator.
cc/ @aldipower @alexanderweber

Thanx for describing the issue.
In the meantime I will write something workarounds in the UI to handle this. :)

Also path: "apps[2].container.columes[1].value" is confusing if I only have one app defined.

@aldipower this problem you posted here only exists during development. This will not a problem for released versions.

@aquamatthias OK, I see, easy going.

@aldipower regarding your example - the object being validated here is probably a group, being the root object of validation. We decided not to prepend this information as described in #3255. Here the explicit error path would be:
/apps(2)/container/volumes(1)

That is why the index information 2 might be used in order to find the affected app in first place, namely being the third application in the group.
@aquamatthias I am about to change the scheme as following:

replace [ ] with ( ) respectively
replace . by /
prepend / at the root of each path. This would also replace self with / in root-only errors

cc /@pierlo-upitup point 3 is sort of conflicting with what we agreed on in #3255. Let me know if you have different thoughts on that one.

@alexanderweber  btw, as I mentioned in the original PR  do you think these changes should be documented in the official docs?

@pierlo-upitup Thanks for pointing that out. I did adjust documentation as mentioned #3387.


TypeError: spread non-iterable instance

Current version of wallet doesn't have such problems. Feel free to reopen this issue, if it will pop up again.


Allow base-4.10

GHC 8.2.1 with base 4.10 works fine.

Thanks for the reminder, as you can see at http://hackage.haskell.org/package/cryptohash-sha1/dependencies this has been already on Hackage for some time now.


Windows support

Describe the feature you’d like to be implemented
Would it be possible to support Windows? Currently compilation fails with:
$ cargo install ghciwatch
...
   Compiling ghciwatch v1.0.1
error[E0433]: failed to resolve: could not find `sys` in `nix`
 --> C:\Users\Javier\.cargo\registry\src\index.crates.io-6f17d22bba15001f\ghciwatch-1.0.1\src\ghci\mod.rs:5:10
  |
5 | use nix::sys::signal::Signal;
  |          ^^^ could not find `sys` in `nix`

error[E0432]: unresolved import `nix::sys`
 --> C:\Users\Javier\.cargo\registry\src\index.crates.io-6f17d22bba15001f\ghciwatch-1.0.1\src\ghci\mod.rs:4:10
  |
4 | use nix::sys::signal;
  |          ^^^ could not find `sys` in `nix`

error[E0432]: unresolved import `nix::unistd`
  --> C:\Users\Javier\.cargo\registry\src\index.crates.io-6f17d22bba15001f\ghciwatch-1.0.1\src\ghci\mod.rs:27:10
   |
27 | use nix::unistd::Pid;
   |          ^^^^^^ could not find `unistd` in `nix`

error[E0433]: failed to resolve: could not find `sys` in `nix`
 --> C:\Users\Javier\.cargo\registry\src\index.crates.io-6f17d22bba15001f\ghciwatch-1.0.1\src\ghci\process.rs:9:10
  |
9 | use nix::sys::signal::Signal;
  |          ^^^ could not find `sys` in `nix`

error[E0432]: unresolved import `nix::sys`
 --> C:\Users\Javier\.cargo\registry\src\index.crates.io-6f17d22bba15001f\ghciwatch-1.0.1\src\ghci\process.rs:8:10
  |
8 | use nix::sys::signal;
  |          ^^^ could not find `sys` in `nix`

error[E0432]: unresolved import `nix::unistd`
  --> C:\Users\Javier\.cargo\registry\src\index.crates.io-6f17d22bba15001f\ghciwatch-1.0.1\src\ghci\process.rs:10:10
   |
10 | use nix::unistd::Pid;
   |          ^^^^^^ could not find `unistd` in `nix`

Some errors have detailed explanations: E0432, E0433.
For more information about an error, try `rustc --explain E0432`.
error: could not compile `ghciwatch` (lib) due to 6 previous errors
error: failed to compile `ghciwatch v1.0.1`, intermediate artifacts can be found at `C:\msys64\tmp\cargo-installd82Fgu`.
To reuse those artifacts with a future compilation, set the environment variable `CARGO_TARGET_DIR` to that path.

ghcid is perfectly usable on windows, with the same caveats it has on Linux but still usable.
List alternatives to the feature and their pros and cons
No response
Additional context
No response

Windows support has been a non-goal so far. I would recommend building with Windows Subsystem for Linux. We build ghciwatch with Nix in CI via Garnix, neither of which supports Windows, so it would mean (at a minimum) adding a new build system without any of the guarantees that Nix offers us. ghcid also has a surprising amount of code to support Windows.
If you can get the test suite passing on Windows I'd take a look at it, but unfortunately I don't want to commit to getting it merged as I am unenthusiastic to overhaul the CI/release process.

I got a working version in https://github.com/jasagredo/ghciwatch/commit/a9a45d0ba14b825d4f43702004ac8b7fcc50f202, essentially just needing to modify how the Pid is passed around and how the process is killed.
It has been ~4 years since I last did some rust, so maybe my changes are not very idiomatic.
Anyways, I didn't fix the test-suite so it is probably not worth for consideration yet, but maybe someone that knows more rust than me can do a better job. That commit is just a proof that with very little changes it can work on Windows.


broken links in documentation to plugin data

Via @CarolynBuck-Gengler. Thanks!

In at least three of the plugin pages I have looked at (and I’m guessing this would be true for most or all), in the section where the data generated are described, there is first a paragraph “In addition to the default data collected by all plugins, this plugin collects the following data for each trial.” that has a link, and the link is broken (404 - Not found).

For example, when I click it in the html-keyboard-response page, it goes to https://www.jspsych.org/plugins/jspsych-html-keyboard-response/overview#datacollectedbyplugins
Similarly, when clicked from the instructions plugin page, it goes to https://www.jspsych.org/plugins/jspsych-instructions/overview#datacollectedbyplugins

However, I’m guessing (based on poking around) that all of them should actually be going to the middle of the page https://www.jspsych.org/plugins/overview/

Thanks @CarolynBuck-Gengler! The docs files have been updated with the correct link now and the changes will be updated on the live site when v6.2 is released (if not sooner).


Python wrapper crashes while retrieving nearest neighbors when M>100

Hi,
I am working on a problem where I need to retrieve ~500 nearest neighbors out of a million points. I am using the python wrapper for HNSW method. The code works perfectly well if I set the value of parameter M <=100 but setting it greater than 100, the code crashes during retrieving nearest neighbors (no issues while building the model) with an "invalid next size" error. Any idea why this might be happening?
Thanks
Himanshu

Well, ideally I would need a reproducible example. Is it possible to reproduce using some existing, perhaps, random data?

sure, I'll try and create an example

hi.. I have created an example. The data is slightly big (~800mb), my apologies for that. You can download the data from here - https://drive.google.com/open?id=1xq67SodKBZe_FMzMs1XfQCxBOpJ7v1KB
Following is the code that I am using to build the kNN model and test it. For the time being, I am finding the nearest neighbors of the points already present in the graph. The code crashes while querying -
import numpy
import sys
import nmslib
import time
import math
data_file = 'cent_mat_dense_bin'
M = 200
efC = 400
efS = 400
num_nbrs = 400
num_thread = 4
fp = open(data_file,'rb')
fp.seek(8)
data = numpy.fromfile(fp,dtype=numpy.float32,count=-1,sep='')
data = numpy.reshape(data,(len(data)/300,300))
end = time.time()
print 'num_inst = %d'%int(data.shape[0])
start = time.time()
index = nmslib.init(method='hnsw', space='cosinesimil')
index.addDataPointBatch(data)
index.createIndex({'M': M, 'indexThreadQty': num_thread, 'efConstruction': efC})
end = time.time()
print 'Training time = %f'%(end-start)
index.setQueryTimeParams({'efSearch': efS})
print 'Testing....'
start = time.time()
nbrs = index.knnQueryBatch(data, k=num_nbrs, num_threads = num_thread)
end = time.time()

Thank you, 800 MB is totally fine.

Just wanted to know if you were able to reproduce the error!!

Hi @hjain689 I am playing with it, thanks!

Hi @hjain689 I nearly nailed, but fixing could take time (it is a nasty thing). As a workaround you can use the following query parameters (add algoType = old):
index.setQueryTimeParams({'efSearch': efS, 'algoType': 'old'})

Hey @searchivarius thanks a lot! Does using algoType = old affect the performance(in terms of accuracy or time taken) in any way?
Do let me know when you fix the bug.

@hjain689 for small efSearch  speed can decrease a bit, but not much, say 10-20%. If vectors are long say dim=300, the diff is rather small already.

A few more questions, would be really helpful if you could please answer them -

Since HNSW starts with a random initialization, I was wondering if its possible to specify random number seed while training the hnsw graph? Basically I want to create multiple graphs with different random initializations
Which method do you think would be the best if my data is high dimensional sparse say million dimensional with ~300 non zeros? I tried hnsw but the performance wasn't very good and it was very slow too

@hjain689

Is in process
Regarding performance: could I try your data? In some cases NAPP with generated pivots could be better. In any case, I would love to see the data where HNSW doesn't perform well

Thank you!

sure i'll send u the data


Add Read Number From File in C

Happy Hacktoberfest! @ravya1108 I recommend adding a program to read a file.
I've written in c, with a quick example program to read a file and output a number.

I'll be closing the PR. I've been notified the repo has been excluded from Hacktoberfest. Better luck next year.


r.row() doesn't accept arguments

I wanted to get a time range in my table
 r.table('analytic').filter(r.row("date").le(b).and(r.row("date").ge(a))).run();
and it thrown

{ ReqlDriverError
at Function.Term._arity (C:\Users\theo\WebstormProjects\Arys\node_modules\rethinkdbdash\lib\term.js:3016:11)
at Function.Term.row (C:\Users\theo\WebstormProjects\Arys\node_modules\rethinkdbdash\lib\term.js:1320:10)
at Object.db.countAnalyticByDate (C:\Users\theo\WebstormProjects\Arys\util\rethinkdb.js:147:50)
at Object.func (C:\Users\theo\WebstormProjects\Arys\modules\analytic.js:35:20)
at Client.on.message (C:\Users\theo\WebstormProjects\Arys\Arys.js:101:42)
at emitOne (events.js:96:13)
at Client.emit (events.js:191:7)
at MessageCreateHandler.handle (C:\Users\theo\WebstormProjects\Arys\node_modules\discord.js\src\client\websocket\packets\handlers\MessageCreate.js:9:34)
at WebSocketPacketManager.handle (C:\Users\theo\WebstormProjects\Arys\node_modules\discord.js\src\client\websocket\packets\WebSocketPacketManager.js:120:65)
at WebSocketManager.eventMessage (C:\Users\theo\WebstormProjects\Arys\node_modules\discord.js\src\client\websocket\WebSocketManager.js:273:31)
at WebSocket.onMessage (C:\Users\theo\WebstormProjects\Arys\node_modules\ws\lib\WebSocket.js:442:14)
at emitTwo (events.js:106:13)
at WebSocket.emit (events.js:194:7)
at Receiver.ontext (C:\Users\theo\WebstormProjects\Arys\node_modules\ws\lib\WebSocket.js:841:10)
at C:\Users\theo\WebstormProjects\Arys\node_modules\ws\lib\Receiver.js:536:18
at Receiver.applyExtensions (C:\Users\theo\WebstormProjects\Arys\node_modules\ws\lib\Receiver.js:371:5)
at C:\Users\theo\WebstormProjects\Arys\node_modules\ws\lib\Receiver.js:508:14
at Receiver.flush (C:\Users\theo\WebstormProjects\Arys\node_modules\ws\lib\Receiver.js:347:3)
at Receiver.finish (C:\Users\theo\WebstormProjects\Arys\node_modules\ws\lib\Receiver.js:541:12)
at Receiver.expectHandler (C:\Users\theo\WebstormProjects\Arys\node_modules\ws\lib\Receiver.js:499:31)
at Receiver.add (C:\Users\theo\WebstormProjects\Arys\node_modules\ws\lib\Receiver.js:103:24)
at TLSSocket.realHandler (C:\Users\theo\WebstormProjects\Arys\node_modules\ws\lib\WebSocket.js:825:20)
msg: 'r.row takes 0 argument, 1 provided',
message: 'r.row takes 0 argument, 1 provided.' }

this at me, even if it's the same as the example from the official driver(couldn't find a proper doc for rethinkdbdash)
possible bug, or me completly failing ?
rethinkdb server : 2.3.5
rethinkdbdash version : 2.3.29

Hum, it should work - are you sure the query you linked is the right one?
There are multiple tests for r.row taking arguments
https://github.com/neumino/rethinkdbdash/blob/master/test/document-manipulation.js#L179

just checked it, it's the right one
this is what a full entry look like

{ channel: '244146816797376512',
date:<PHONE_NUMBER>989,
user: '306418399242747906'
guild: '242655328410402816',
id: '001513d4-c2d8-492d-ba8a-f8b38876db1e',
item: '<:feelsluckyman:247432630541221888>' }

hello, is there any reason why r.row is a function but not have a function prototype methods? i want to run this:
r.row.apply(null, "name")
but r.row.apply is undefined
i are working in a client side wrapper of rethinkdbdash and need can call it dinamically, current i doing
Function.prototype.apply.call(r.row, r.row, ['name']);
i feeling dirty

Hum, that's because r.row is an object.
Typically this is a valid query: r.table('foo').filter(r.row)


refactor: notification component

Currently notifications to users and developers are mixed together. Improve presentation to clarify which user a notification message are meant for.
Notifications to end-users must not appear more critical than they are.

This is done


Important! Template update for nf-core/tools v2.11.dev0

Version 2.11.dev0 of nf-core/tools has just been released with updates to the nf-core template. This automated pull-request attempts to apply the relevant updates to this pipeline.
Please make sure to merge this pull-request as soon as possible, resolving any merge conflicts in the nf-core-template-merge-2.11.dev0 branch (or your own fork, if you prefer). Once complete, make a new minor release of your pipeline.
For instructions on how to merge this PR, please see https://nf-co.re/docs/contributing/sync/.
For more information about this release of nf-core/tools, please see the v2.11.dev0 release page.

Version 2.11.dev0 of the nf-core/tools pipeline template has just been released. This pull-request is now outdated and has been closed in favour of https://github.com/nf-core/testpipeline/pull/72
Please use https://github.com/nf-core/testpipeline/pull/72 to merge in the new changes from the nf-core template as soon as possible.


Updaing state to use react hooks

Hello, I would love to update the the plant component from RCC to RFC using react hooks.

Hey!
Sure! You can do it :)

Ok, great


Error running docker swarm

[+] Building 294.2s (11/11) FINISHED                                                                                docker:desktop-linux
 => [app internal] load build definition from Dockerfile                                                                                0.1s
 => => transferring dockerfile: 466B                                                                                0.0s
 => [app internal] load metadata for docker.io/library/python:3.8.13-slim-bullseye                                                                                3.5s
 => [app internal] load .dockerignore                                                                                0.0s
 => => transferring context: 80B                                                                                0.0s
 => [app 1/7] FROM docker.io/library/python:3.8.13-slim-bullseye@sha256:dd46e1cb12432c17040638acb05508c35dc828db04f4f33c692d3894b7bc76cf                                                                      7.4s
 => => resolve docker.io/library/python:3.8.13-slim-bullseye@sha256:dd46e1cb12432c17040638acb05508c35dc828db04f4f33c692d3894b7bc76cf                                                                          0.0s
 => => sha256:e238bceb29579b6804c25c4e8c81612003af698ef4ca42c46fa87d4ef371653c 1.08MB / 1.08MB                                                                                1.0s
 => => sha256:9c37eda4ff12eb9bd68ee8feacb78034f4524716d57673ff0a0ad82f88df487c 11.33MB / 11.33MB                                                                                2.5s
 => => sha256:dd46e1cb12432c17040638acb05508c35dc828db04f4f33c692d3894b7bc76cf 1.86kB / 1.86kB                                                                                0.0s
 => => sha256:db831ae8c7e88ca6f69465a7c3f255c03678e830a1ad13c2a340bec9fc232d3e 1.37kB / 1.37kB                                                                                0.0s
 => => sha256:bdd3315885d481d5b81d7bd1af8b04dffc67decf793d6716708750b673e46b6d 7.53kB / 7.53kB                                                                                0.0s
 => => sha256:7a6db449b51b92eac5c81cdbd82917785343f1664b2be57b22337b0a40c5b29d 31.38MB / 31.38MB                                                                                3.4s
 => => sha256:120078faa76053c068cf75960a6db2a9473ef941fdf2971b724769de253f534f 233B / 233B                                                                                1.4s
 => => sha256:2d3523ccd6f419b6f4643fe4d472243848803763261afa6947386fcb161bcd16 3.18MB / 3.18MB                                                                                2.4s
 => => extracting sha256:7a6db449b51b92eac5c81cdbd82917785343f1664b2be57b22337b0a40c5b29d                                                                                2.3s
 => => extracting sha256:e238bceb29579b6804c25c4e8c81612003af698ef4ca42c46fa87d4ef371653c                                                                                0.1s
 => => extracting sha256:9c37eda4ff12eb9bd68ee8feacb78034f4524716d57673ff0a0ad82f88df487c                                                                                0.7s
 => => extracting sha256:120078faa76053c068cf75960a6db2a9473ef941fdf2971b724769de253f534f                                                                                0.0s
 => => extracting sha256:2d3523ccd6f419b6f4643fe4d472243848803763261afa6947386fcb161bcd16                                                                                0.4s
 => [app internal] load build context                                                                                0.7s
 => => transferring context: 8.53MB                                                                                0.7s
 => [app 2/7] WORKDIR /app                                                                                1.0s
 => [app 3/7] RUN apt-get -y update && apt-get install -y   wget   ffmpeg   libsm6   libxext6                                                                                58.3s
 => [app 4/7] RUN pip install --upgrade setuptools                                                                                6.6s
 => [app 5/7] COPY . .                                                                                0.1s
 => [app 6/7] RUN pip install torch torchvision torchaudio --extra-index-url https://download.pytorch.org/whl/cpu   && pip install -r requirements.txt                                                      207.7s
 => ERROR [app 7/7] RUN python download.py                                                                                9.3s
------
 > [app 7/7] RUN python download.py:
0.356 --2024-08-17 12:16:14--  https://archive.org/download/anpr_weights/weights.pt
0.367 Resolving archive.org (archive.org)... <IP_ADDRESS>
0.407 Connecting to archive.org (archive.org)|<IP_ADDRESS>|:443... connected.
0.955 HTTP request sent, awaiting response... 302 Found
1.234 Location: https://ia801509.us.archive.org/18/items/anpr_weights/weights.pt [following]
1.234 --2024-08-17 12:16:14--  https://ia801509.us.archive.org/18/items/anpr_weights/weights.pt
1.234 Resolving ia801509.us.archive.org (ia801509.us.archive.org)... <IP_ADDRESS>
1.256 Connecting to ia801509.us.archive.org (ia801509.us.archive.org)|<IP_ADDRESS>|:443... connected.
1.828 HTTP request sent, awaiting response... 404 Not Found
2.057 2024-08-17 12:16:15 ERROR 404: Not Found.
2.057
8.507 /bin/sh: 1: git: not found
8.508 /usr/local/lib/python3.8/site-packages/setuptools/sandbox.py:15: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
8.508   import pkg_resources
8.508 /usr/local/lib/python3.8/site-packages/pkg_resources/__init__.py:3144: DeprecationWarning: Deprecated call to `pkg_resources.declare_namespace('mpl_toolkits')`.
8.508 Implementing implicit namespace packages (as specified in PEP 420) is preferred to `pkg_resources.declare_namespace`. See https://setuptools.pypa.io/en/latest/references/keywords.html#keyword-namespace-packages
8.508   declare_namespace(pkg)
8.508 /usr/local/lib/python3.8/site-packages/pkg_resources/__init__.py:3144: DeprecationWarning: Deprecated call to `pkg_resources.declare_namespace('google')`.
8.508 Implementing implicit namespace packages (as specified in PEP 420) is preferred to `pkg_resources.declare_namespace`. See https://setuptools.pypa.io/en/latest/references/keywords.html#keyword-namespace-packages
8.508   declare_namespace(pkg)
8.508 Downloaded files!
8.509 Traceback (most recent call last):
8.509   File "/app/utils/google_utils.py", line 26, in attempt_download
8.509     assets = [x['name'] for x in response['assets']]  # release assets
8.509 KeyError: 'assets'
8.509
8.509 During handling of the above exception, another exception occurred:
8.509
8.509 Traceback (most recent call last):
8.509   File "download.py", line 8, in <module>
8.509     from detections import get_plates_from_image
8.509   File "/app/detections.py", line 44, in <module>
8.509     model = attempt_load(weights, map_location=device)  # load FP32 model
8.509   File "/app/models/experimental.py", line 243, in attempt_load
8.509     attempt_download(w)
8.509   File "/app/utils/google_utils.py", line 31, in attempt_download
8.509     tag = subprocess.check_output('git tag', shell=True).decode().split()[-1]
8.509   File "/usr/local/lib/python3.8/subprocess.py", line 415, in check_output
8.509     return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
8.509   File "/usr/local/lib/python3.8/subprocess.py", line 516, in run
8.509     raise CalledProcessError(retcode, process.args,
8.509 subprocess.CalledProcessError: Command 'git tag' returned non-zero exit status 127.
------
failed to solve: process "/bin/sh -c python download.py" did not complete successfully: exit code: 1

The archive where the model was stored has been deleted by the owner. That's why that error occurs.
Unfortunately I won't be able to resolve it.


Update debugger

Update debugger gem dependencies so it's possible to install gems locally. Old dependencies are yanked.

Hm, have you tried the Rails 4 branch? https://github.com/inaturalist/inaturalist/tree/rails4. I bet that resolves this. If that solves your problem I'll just close this.

Yes - rails4 branch installs gems cleanly. Is that the branch a potential contributor should start with?

Just merged Rails 4 into master. If this issue is now solved, can you close this pr?

Gems install cleanly now. I closed the issue, but it was reopened.


Adding more than one product without choosing its value leaves a blank value

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Start from scratch
Add NO2, NO, O as gas species
Add a photolysis reaction
Click the plus button on the products twice, but don't try to choose their value until AFTER you have clicked the button twice
Assign one of the products
You should see three products, two are empty, one is filled, the third was added after assigning one product

Expected behavior
Choosing a product when a list of empty products exists shouldn't add another empty product
Screenshots

Actually, the bug seems to be anytime you add something towards the bottom of the card, and then try to change something at the top, your changes at the top are not saved. Do this same thing, but set a musica name first and then try to add species and the press close


Separate META-INF.services.

Fixes #ISSUSE_ID.
Changes proposed in this pull request:

Separate META-INF.services.

/run ci


API has convert("BGR") when loading a jpg image?

i am more familiar with java and c# but while debugging an error in the api endpoint,
/represent or http://localhost:5000/represent
it seems that if you supply a regular image, it errors out with,
{
    "error": "Exception while representing: conversion from RGB to BGR not supported"
}

looking into it, it seems the file 'preprocessing.py', function 'def load_image(img: Union[str, np.ndarray]) -> Tuple[np.ndarray, str]:'
if img.startswith("http"):
        return (
            np.array(Image.open(requests.get(img, stream=True, timeout=60).raw).convert("BGR")),
            # return url as image name
            img,
        )

instead of
if img.startswith("http"):
        return (
            np.array(Image.open(requests.get(img, stream=True, timeout=60).raw)),
            # return url as image name
            img,
        )

as if it's expecting an opencv compatible image or mat? even if the detector_backend is set to 'retinaface'
http request body tried,
{
    "model_name": "Facenet",
    "img": "http://<IP_ADDRESS>/sharedDirectory/face.jpg",
    "detector_backend": "retinaface",
    "distance_metric": "cosine"
}

{
    "img": "http://<IP_ADDRESS>/sharedDirectory/face.jpg"
}

url is valid and accessible.

Would you please share the image? Could it be gray scale?

it's from celebrity faces dataset

Oh I see the root cause of the bug. Will fix this today.

Closed with PR - https://github.com/serengil/deepface/pull/1026


contract: Implicitly freeze on recovery withdrawals

Closes #57

Forgot to regenerate bindings :joy: Will only remove the freeze call though as the remaining API is unchanged.
I am also surprised that the tests work, don't we test freezing somewhere?


Adding icon

New Update -
Adding icon
Expected output-

You mean favicon !!
@MaruthiSingh , go for it.

@Jagroop2001  Created a pull request can you merge so I can work on issue#13

@MaruthiSingh , please check comments on the PR

https://github.com/Jagroop2001/AcmeRockets/pull/14


Content Creator: Premium Pricing Plugin

Premium pricing. Options are yes or no. If no, do nothing. If yes, show Contact content type and give user option to choose which one to show.
Feature: Premium Pricing
  As a Content Creator,
  I want an input plugin for Premium Pricing for the API content type
  So that user can have the option of showing Premium Pricing option for certain APIs

  Scenario: Premium Pricing (conditional)
    Given that Premium Pricing is available on some APIs
     When I enter content about Premium Pricing
     Then I would need to choose if Premium Pricing is available
      And my options would be `yes` or `no`

  Scenario: Contact (reference)
    Given that Premium Pricing is available on some APIs
     When Premium Pricing is available and I enter `yes`
     Then I will be given the ability to choose content from the `Contact` content type
      And will choose the `Contact` content that I want to display

Premium Pricing

[ ] Premium Pricing (conditional)
[ ] Contact (reference)

Blocked by https://github.com/punchcard-cms/punchcard/issues/45

Blocker for https://github.ibm.com/Watson/wox-cms/issues/404

WOX specific, so I moved to WOX-CMS: https://github.ibm.com/Watson/wox-cms/issues/411


The repo has at least one admin who is ideally not part of the Governing Board.

oops, that was an accidental issue.


Integration test tasks

Background
Get everything done for a successful test of upcoming op upgrading
Tasks

[ ] Deploy dev-net chains (1 hub and 2 spoke chains) => At least, we should be able to get the rpc urls/chainIds for dev-chains.
[ ] Add/Update the relevant smart contracts (create AdminSpokeConnector, etc), deploy them to dev-net, run the deploy script against dev-net
[ ] Deploy the subgraphs (1 hub and 2 spoke chains) with the RPC endpoint generated by the first step
[ ] Setup a separate database for dev-net. To decrease the sync time, we can basically copy/paste the data from Mainnet prod database.
[ ] Setup cartographer agents and store the subgraph data in the dev-net database
[ ] Run offchain agents (router, sequencer, lighthouse, relayer, watcher, sdk server, etc)

init

making progress on subgraph deployment
discussing the strategy for contracts and state management

Moving back into monorepo, set up tenderly devnet with CLI. Working on deploying contracts including admin connectors.

closing this issue and moving tasks under epic #4713


FB_SingleSolenoid.Reset() never return true

The Reset method never returns true, and does not call Reset() of the component base.

Hello Mr Kirkegarrd - This issue has been fixed and will be rolled out in next version.
Thank you for helping us make it better.


In async example fetch request is disallowed

When I ran the 'async' example, I got following error in chrome.
https://github.com/rackt/redux/tree/master/examples/async
Error - "XMLHttpRequest cannot load http://www.reddit.com/r/reactjs.json. The request was redirected to 'https://www.reddit.com/r/reactjs.json', which is disallowed for cross-origin requests that require preflight."

I restarted my browser with following command, and it accepts the request now.
"C:\Program Files (x86)\Google\Chrome\Application\chrome.exe" --user-data-dir="C:/Chrome Insecure Session" --disable-web-security

It used to work :-/
Is there anything we can do on our side?

Nothing :). As you figured out.. I am new to JavaScript.

Should be fixed now.

Still doesn't work:

Fetch API cannot load https://www.reddit.com/r/reactjs.json. The request was redirected to a URL
('about:blank') which has a disallowed scheme for cross-origin requests.

@pke Try disabling browser extensions: https://bugs.chromium.org/p/chromium/issues/detail?id=360583#c9

Ups! Disconnect was interfering! Thanks!


Agree on and define URL structure

And whether we use pushState "real" URLs or # parameters.
cc @kepta @ajithranka

The url structure was finalised.
Closing this ticket.


Add CRC-16 checksums without XOR in or out

I recently decided to read out my 'smart' meter for electricity, gas, and water, which complies to the DSMR v4.2.2 specification.
However, the so-called 'telegram' uses a CRC-16 checksum without XOR in or XOR out (See § 5.2 in the document mentioned above). This functionality was not present in this package or any other CRC-16 packages I could find, so I decided to add it here.
I also changed the README a little, and I removed the empty doc.go file.

This looks fine, although I'd like to propose handling it a different way. Add a "noXOR" field to the table and have the Update() function call the correct corresponding updateNoXOR() function if it is set. This way the normal hash.Hash interface can still be used. The table just needs to be created with the right fields set.


Add option for copying files instead of moving them

Adds a new composer.json config flag which allows devs to decide if the dropin files should be copied instead of moved from the source to the destination.

This PR implements #4.

@onnimonni could I get an update on this, is this OK, unwanted, or should I make changes? :)

Totally forgot this! Thanks for pinging.
What I would want you before merging this to do still is to implement a test for this feature.

Sure thing, I'll try to get one up ASAP.

I altered the test setup quite a bit as the copy flag is project specific, now there are two test Composer packages in tests/. Worked locally but we have to see if the Travis config still works. I also added a composer test command to streamline local testing a little.

Excellent! Travis is currently failing because of the github api limit. I need to fix that a bit later.

Ah OK. You can test it locally then before merging. :+1:

Thanks for adding this awesome feature and pinging me!
I have been too tired/busy to maintain this and I feel so terribly ashamed of myself.
I love the test helper you added:
$ composer test

No worries, personal stuff always comes first. :+1:
Let me know if any issues come up with the new feature so I can jump in to help if needed.
Thanks!


Add ReplacementProductList to Resource Monitoring Cluster Definition

📖 Description
The discussion around the replacement product list caused spec changes. It added 2 new attributes. This issue should only add the ReplacementProductList to the cluster definition since LastChangedTime is being handled in #27570

This is no straight forward dup. It's by design split up into more than one issue (see #27570) to track the work and help make the PRs the right scope and size.


HydratorFactory.php blows up when running in Docker

Multiple developers are experiencing the same issue when using this framework in Docker - it blows up application with an error:
ContextErrorException: Warning: rename(/var/www/trackingfirst/app/cache/dev/doctrine/odm/mongodb/Hydrators/TrackingFirstMainBundleDocumentUserHydrator.php.5901663c46c199.48165620,/var/www/trackingfirst/app/cache/dev/doctrine/odm/mongodb/Hydrators/TrackingFirstMainBundleDocumentUserHydrator.php): Operation not permitted in /var/www/trackingfirst/vendor/doctrine/mongodb-odm/lib/Doctrine/ODM/MongoDB/Hydrator/HydratorFactory.php line 415
Fix it already!

Here is the fix for you:

       rename($tmpFileName, $fileName);

       copy($tmpFileName, $fileName);

       unlink($tmpFileName);

Operation not permitted

That looks like a permission issue on your end. Could you share a little bit of information about your docker container? Of interest would be whether the cache folder used for doctrine is added to the Docker image while copying the sources into the container.
As for the suggested "fix", as I've previously indicated in #1495 that rename has the advantage of being an atomic operation, while copy can lead to processes reading a partially written file, blowing the application up with a parse error.

Maybe jumping the gun a bit, but @eluzgin

Fix it already!

Is not the best attitude towards something you probably make money with and don't pay yourself for. "Could you please fix this?" sounds way better and is friendlier.

@malarzm sorry guys, no hard feelings.
I'm just a fellow developer here and was getting frustrated with that error, which kept preventing me from getting my work done on a project.
It doesn't seem to be permission related, just Docker related.
And this is not an isolated issue since it was reported by other developers as well.
The local workaround is to replace single replace operation by copy, unlink operations.
It fixes this problem.
Now I understand your concern for atomic operations and single transaction
but it's better that it works then doesn't.

@eluzgin As I mentioned, please share some information about your docker container. I'm running a couple of ODM applications in with Docker as well and I don't have the issue at all. It's quite simple: if the system can touch('/some/path/to/file.tmp') but not rename('/some/path/to/file.tmp', '/some/path/to/file'), the culprit is somewhere else. This is not something we should fix by replacing rename with copy.

Closed due to lack of feedback. @eluzgin please reopen with the info I asked for if you still need help.

I don't have time for this. The decision have been made to move away from
this php framework any way.
However you might still see this issue reported by other users since Docker
environment is gaining popularity.
-Eugene
On Wed, May 17, 2017 at 2:01 AM, Andreas<EMAIL_ADDRESS>wrote:

Closed due to lack of feedback. @eluzgin https://github.com/eluzgin
please reopen with the info I asked for if you still need help.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/doctrine/mongodb-odm/issues/1583#issuecomment-302016015,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAiR0k6nCgzE0A8bm6uA7bvjHzOej3y6ks5r6qlDgaJpZM4NJsL0
.

Hi, I'm experiencing the same issue, but without Docker.
We're using Vagrant on Virtualbox with a PHP Lumen API as back-end and React front-end. This is totally random, and happens once in a while.
This is really annoying, since the folder is writable, and every hydrator class can be written to it, so it's not a permission issue on the server.
Using doctrine/mongodb-odm: 1.1.5 and alcaeus/mongo-php-adapter: 1.0.9 with laravel/lumen-framework: v5.4.5
The filename seems to be some kind of cached file. The original hydrator file exists already, so I think that might have something to do with it, but I'm not sure.
Here's the stack trace:
[2017-08-31 04:57:11] lumen.ERROR: ErrorException: rename(/path/to/api/storage/doctrine/hydrators/NordLumenOAuth2DoctrineODMDocumentsAccessTokenHydrator.php.59a797279da171.87882666,/path/to/api/storage/doctrine/hydrators/NordLumenOAut
h2DoctrineODMDocumentsAccessTokenHydrator.php): Operation not permitted in /path/to/api/vendor/doctrine/mongodb-odm/lib/Doctrine/ODM/MongoDB/Hydrator/HydratorFactory.php:422
Stack trace:
#0 [internal function]: Laravel\Lumen\Application->Laravel\Lumen\Concerns\{closure}(2, 'rename(/path/to/api...', '/path/to/api/vendor...', 422, Array)
#1 /path/to/api/vendor/doctrine/mongodb-odm/lib/Doctrine/ODM/MongoDB/Hydrator/HydratorFactory.php(422): rename('/path/to/api/storag...', '/path/to/api/storag...')
#2 /path/to/api/vendor/doctrine/mongodb-odm/lib/Doctrine/ODM/MongoDB/Hydrator/HydratorFactory.php(146): Doctrine\ODM\MongoDB\Hydrator\HydratorFactory->generateHydratorClass(Object(Doctrine\ODM\MongoDB\Mapping\ClassMetadata), 'Nord
LumenOAuth2...', '/path/to/api/storag...')
#3 /path/to/api/vendor/doctrine/mongodb-odm/lib/Doctrine/ODM/MongoDB/Hydrator/HydratorFactory.php(460): Doctrine\ODM\MongoDB\Hydrator\HydratorFactory->getHydratorFor('Nord\\Lumen\\OAut...')
#4 /path/to/api/vendor/doctrine/mongodb-odm/lib/Doctrine/ODM/MongoDB/UnitOfWork.php(2692): Doctrine\ODM\MongoDB\Hydrator\HydratorFactory->hydrate(Object(Nord\Lumen\OAuth2\Doctrine\ODM\Documents\AccessToken), Array, Array)
#5 /path/to/api/vendor/doctrine/mongodb-odm/lib/Doctrine/ODM/MongoDB/Persisters/DocumentPersister.php(649): Doctrine\ODM\MongoDB\UnitOfWork->getOrCreateDocument('Nord\\Lumen\\OAut...', Array, Array, Object(Nord\Lumen\OAuth2\Doctri
ne\ODM\Documents\AccessToken))
#6 /path/to/api/vendor/doctrine/mongodb-odm/lib/Doctrine/ODM/MongoDB/Persisters/DocumentPersister.php(514): Doctrine\ODM\MongoDB\Persisters\DocumentPersister->createDocument(Array, NULL, Array)
#7 /path/to/api/vendor/doctrine/mongodb-odm/lib/Doctrine/ODM/MongoDB/DocumentRepository.php(186): Doctrine\ODM\MongoDB\Persisters\DocumentPersister->load(Array)
#8 /path/to/api/vendor/nordsoftware/lumen-oauth2-doctrine-odm/src/Repositories/AccessTokenRepository.php(18): Doctrine\ODM\MongoDB\DocumentRepository->findOneBy(Array)
#9 /path/to/api/vendor/nordsoftware/lumen-oauth2-doctrine-odm/src/Storages/AccessTokenStorage.php(49): Nord\Lumen\OAuth2\Doctrine\ODM\Repositories\AccessTokenRepository->findByToken('IodHVZRlX0vwr8y...')
#10 /path/to/api/vendor/league/oauth2-server/src/ResourceServer.php(114): Nord\Lumen\OAuth2\Doctrine\ODM\Storages\AccessTokenStorage->get('IodHVZRlX0vwr8y...')
#11 /path/to/api/vendor/nordsoftware/lumen-oauth2/src/OAuth2Service.php(55): League\OAuth2\Server\ResourceServer->isValidRequest(true, NULL)
#12 /path/to/api/vendor/nordsoftware/lumen-oauth2/src/OAuth2Service.php(65): Nord\Lumen\OAuth2\OAuth2Service->validateAccessToken()
#13 /path/to/api/Access/Services/UserService.php(73): Nord\Lumen\OAuth2\OAuth2Service->getResourceOwnerId()
#14 /path/to/api/Access/Guards/OAuth2Guard.php(37): App\Access\Services\UserService->getCurrentUser()
#15 /path/to/api/vendor/illuminate/auth/GuardHelpers.php(49): App\Access\Guards\OAuth2Guard->user()
#16 /path/to/api/vendor/illuminate/auth/GuardHelpers.php(59): App\Access\Guards\OAuth2Guard->check()
#17 /path/to/api/Access/Http/Middleware/Authenticate.php(44): App\Access\Guards\OAuth2Guard->guest()
#18 /path/to/api/vendor/illuminate/pipeline/Pipeline.php(148): App\Access\Http\Middleware\Authenticate->handle(Object(Illuminate\Http\Request), Object(Closure))
#19 [internal function]: Illuminate\Pipeline\Pipeline->Illuminate\Pipeline\{closure}(Object(Illuminate\Http\Request))
#20 /path/to/api/vendor/laravel/lumen-framework/src/Routing/Pipeline.php(32): call_user_func(Object(Closure), Object(Illuminate\Http\Request))
#21 /path/to/api/vendor/illuminate/pipeline/Pipeline.php(102): Laravel\Lumen\Routing\Pipeline->Laravel\Lumen\Routing\{closure}(Object(Illuminate\Http\Request))
#22 /path/to/api/vendor/laravel/lumen-framework/src/Concerns/RoutesRequests.php(778): Illuminate\Pipeline\Pipeline->then(Object(Closure))
#23 /path/to/api/vendor/laravel/lumen-framework/src/Concerns/RoutesRequests.php(624): Laravel\Lumen\Application->sendThroughPipeline(Array, Object(Closure))
#24 /path/to/api/vendor/laravel/lumen-framework/src/Concerns/RoutesRequests.php(598): Laravel\Lumen\Application->handleFoundRoute(Array)
#25 /path/to/api/vendor/laravel/lumen-framework/src/Concerns/RoutesRequests.php(532): Laravel\Lumen\Application->handleDispatcherResponse(Array)
#26 [internal function]: Laravel\Lumen\Application->Laravel\Lumen\Concerns\{closure}(Object(Illuminate\Http\Request))
#27 /path/to/api/vendor/laravel/lumen-framework/src/Routing/Pipeline.php(52): call_user_func(Object(Closure), Object(Illuminate\Http\Request))
#28 /path/to/api/vendor/nordsoftware/lumen-fractal/src/FractalMiddleware.php(42): Laravel\Lumen\Routing\Pipeline->Laravel\Lumen\Routing\{closure}(Object(Illuminate\Http\Request))
#29 /path/to/api/vendor/illuminate/pipeline/Pipeline.php(148): Nord\Lumen\Fractal\FractalMiddleware->handle(Object(Illuminate\Http\Request), Object(Closure))
#30 [internal function]: Illuminate\Pipeline\Pipeline->Illuminate\Pipeline\{closure}(Object(Illuminate\Http\Request))
#31 /path/to/api/vendor/laravel/lumen-framework/src/Routing/Pipeline.php(32): call_user_func(Object(Closure), Object(Illuminate\Http\Request))
#32 /path/to/api/vendor/nordsoftware/lumen-cors/src/CorsService.php(114): Laravel\Lumen\Routing\Pipeline->Laravel\Lumen\Routing\{closure}(Object(Illuminate\Http\Request))
#33 /path/to/api/vendor/nordsoftware/lumen-cors/src/CorsMiddleware.php(45): Nord\Lumen\Cors\CorsService->handleRequest(Object(Illuminate\Http\Request), Object(Closure))
#34 /path/to/api/vendor/illuminate/pipeline/Pipeline.php(148): Nord\Lumen\Cors\CorsMiddleware->handle(Object(Illuminate\Http\Request), Object(Closure))
#35 [internal function]: Illuminate\Pipeline\Pipeline->Illuminate\Pipeline\{closure}(Object(Illuminate\Http\Request))
#36 /path/to/api/vendor/laravel/lumen-framework/src/Routing/Pipeline.php(32): call_user_func(Object(Closure), Object(Illuminate\Http\Request))
#37 /path/to/api/vendor/illuminate/pipeline/Pipeline.php(102): Laravel\Lumen\Routing\Pipeline->Laravel\Lumen\Routing\{closure}(Object(Illuminate\Http\Request))
#38 /path/to/api/vendor/laravel/lumen-framework/src/Concerns/RoutesRequests.php(778): Illuminate\Pipeline\Pipeline->then(Object(Closure))
#39 /path/to/api/vendor/laravel/lumen-framework/src/Concerns/RoutesRequests.php(534): Laravel\Lumen\Application->sendThroughPipeline(Array, Object(Closure))
#40 /path/to/api/vendor/laravel/lumen-framework/src/Concerns/RoutesRequests.php(475): Laravel\Lumen\Application->dispatch(NULL)
#41 /path/to/api/public/index.php(28): Laravel\Lumen\Application->run()
#42 {main}

Reopening due to @soderluk's update

This may be related to https://github.com/doctrine/doctrine2/issues/6713#issuecomment-333325870 - duplicate generation of these files would definitely cause the error described by @soderluk. There's nothing we can do, except recommend a different generation strategy (which might be inconvenient in dev, but really the only solution for an operating system issue).

OK, good to know. I thought as much, that there's probably not much that can be done to this issue. The files are all there, the directories/files are writable, and 99.99999999% of the time everything just works. I just wanted to report the issue, if anyone else has the same problem without Docker.

Facing a similar problem using mongodb-odm in Symfony through docker. Also fails on the rename, but in my case he's giving a "No such file or directory" instead of the "Operation not permitted". It seems like some sort of locking issue with docker, caused by thing that rename does under the hood.
What I did to overcome this problem, was to set the cache folder of Symfony to a folder that was not part of the volume shared with the docker container (so instead of /var/cache, I've set it /dev/shm in kernel.php)
I've previously ran into a similar problem with docker when trying to run MongoDB through docker-compose with the mongodb data folder set to a folder inside my project root. This caused MongoDB to crash as it could not create the files (although it actually did create the files). My best guess is that it has got something to do with the way docker exposes the volume/filesystem to the container.

@TomCan Thanks! Set Hydrators directory to /tmp (outside of a shared folder) on Virtualbox VM and the error was gone!


py jupyter-telemetry /  jhsingle-native-proxy # ERROR: Could not find a version that satisfies the requirement ruamel.yaml.clib>=0.1.2; platform_python_implementation == "CPython" and python_version < "3.10" (from ruamel-yaml)

nix-shell -p nixFlakes --run "nix run mach-nix#gen.python.ipython.jhsingle-native-proxy --show-trace "
regarding:

jupyter-telemetry ->  ruamel.yaml

adding
ruamel.yaml.clib>=0.1.2 has no effect
warning: unknown setting 'experimental-features'
warning: --- Large path --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- nix
dumping very large path (> 256 MiB); this may run out of memory
warning: unknown setting 'experimental-features'
trace:
applying fix 'no-rust-build' (nativeBuildInputs) for cryptography:3.3.2

warning: unknown setting 'experimental-features'
builder for '/nix/store/5nbmqydwypdmmfqwl4djhympvbvwnlyb-python3.8-ruamel.yaml-0.17.4.drv' failed with exit code 1; last 10 log lines:
  configuring
  no configure script, doing nothing
  building
  no Makefile, doing nothing
  installing
  Executing pipInstallPhase
  /build/dist /build
  Processing ./ruamel.yaml-0.17.4-py3-none-any.whl
  ERROR: Could not find a version that satisfies the requirement ruamel.yaml.clib>=0.1.2; platform_python_implementation == "CPython" and python_version < "3.10" (from ruamel-yaml)
  ERROR: No matching distribution found for ruamel.yaml.clib>=0.1.2; platform_python_implementation == "CPython" and python_version < "3.10"
cannot build derivation '/nix/store/faaq6zpy13dq0vpz3qxpx6g7jpg6vh4y-python3-3.8.8-env.drv': 1 dependencies couldn't be built
error: --- Error ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- nix
build of '/nix/store/faaq6zpy13dq0vpz3qxpx6g7jpg6vh4y-python3-3.8.8-env.drv' failed

works for
ref="refs/tags/3.1.1" / different pypi

works with
ruamel.yaml<=0.16.12

I just noticed https://github.com/DavHau/mach-nix/issues/287 is a duplicate of this. Closing the current issue as I already commented on the other one.


merge 0.4.x branch to master

Ⅰ. Describe what this PR did
merge 0.4.x branch to master
Ⅱ. Does this pull request fix one issue?
Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

LGTM


content/2018-munich/sponsor.md: Remove taken sponsorship packages and…

… fix pricing
The prospectus listed $3.250 for diversity and Grafana just signed that; so that's
now the canonical price.
Signed-off-by: Richard Hartmann<EMAIL_ADDRESS>

:+1: Thanks


Use custom collectors for worker and queue metrics

The best practice for exporters is to collect as many metrics as
possible at the time Prometheus does the scrape. This fixes issues where
threads or connections may die in the background and cause stale metrics
to be shown forever. Instead, the scrape will fail and we can alert if
celery exporter is down.

Sadly I don't think so, the main other collector is constantly receiving events so doesn't fit as well into collecting the metrics at scrape time.


Lint avoid_classes_with_only_static_members is not working as intended

Steps to Reproduce
The lint avoid_classes_with_only_static_members does not work if a class has only static methods.
If I run flutter analyze on a file with the example from https://dart-lang.github.io/linter/lints/avoid_classes_with_only_static_members.html, the lint avoid_classes_with_only_static_members is not triggered:
class DateUtils {
  static DateTime mostRecent(List<DateTime> dates) {
    return dates.reduce((a, b) => a.isAfter(b) ? a : b);
  }
}

However, if I add a static property to the class, the lint triggers as intended.
class DateUtils {
  static int i = 123;
  static DateTime mostRecent(List<DateTime> dates) {
    return dates.reduce((a, b) => a.isAfter(b) ? a : b);
  }
}

Hi @Giuspepe
The issues seem to be related to the dart analyzer
import 'package:flutter/material.dart';

void main() => runApp(const MyApp());

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Container();
  }
}

class DateUtils {
  // static int i = 123;
  static DateTime mostRecent(List<DateTime> dates) {
    return dates.reduce((a, b) => a.isAfter(b) ? a : b);
  }
}

PS C:\Users\Taha\StudioProjects\master_flutter> dart analyze Analyzing master_flutter... No issues found! PS C:\Users\Taha\StudioProjects\master_flutter>
Please open the issue in the dedicated repository for dart
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


The application is not cleaning the fields when the a “Keep in touch” request is made.

Description:
The application is not cleaning the fields when the a “Keep in touch” request is made.
-Env:
Device: MacBook Pro
Google Chrome Version 54.0.2840.98 (64-bit)
STEPS:
Steps

Open the Crafter studio with the Event site.
Go to the “Contact” section.
Fill out all the fields
Click on the “Send” Button.

CURRENT RESULT:
The registration was done correctly, but the fields were not cleared after clicking the “Send” button.
https://www.screencast.com/t/bT4xEMac
EXPECTED RESULT:
The registration was done correctly, but the fields should be cleared after clicking the “Send” button.

Fixed

Bug verified and closed as fixed in build f2ef5e6.


No more hacky tricks in vmtest/run

Fixes https://github.com/k8sp/auto-install/issues/92
The problem reported in above issue is due to a bug in Vagrant 1.8.5.  I used to think that we cannot downgrade Vagrant from 1.8.5 to 1.8.4 to escape from this bug, but I was wrong -- the uninstallation of Vagrant on Mac OS X is described here as:
rm -rf /opt/vagrant
rm -f /usr/bin/vagrant

Once we do that, we can download the right version of Vagrant https://releases.hashicorp.com/vagrant/1.8.4/vagrant_1.8.4.dmg and install it.
So this PR just revert the previous change https://github.com/k8sp/auto-install/pull/96

我让vmtest/run只用CentOS/7了。

测了nginx PASS，LGTM

1.8.4版本测试tftp PASS ,LGTM

@sunxs1101 谢谢兴帅！