Auth_request module not behaving as expected

Describe the bug
When using Nginx's auth_request module in Nginx v2.9.6, the authentication behaves as expected. When updating to v2.9.7, the module does not redirect to the given subdirectory. Subdomains seem to always work fine. Authentication is using Organizr and tested by @causefx
Nginx Proxy Manager Version
Working: 2.9.6
Not Working: 2.9.7
To Reproduce

Set up an auth_request under a subdirectory to authenticate a page in NPM 2.9.6 (NPM JSON.txt)
Visiting the webpage should work normally if authenticated, and fail with a 401 error if not authenticated.
Update to 2.9.7
Make any change to the proxy host configuration for the page that is to be protected to force NPM to reload the file.
Visit the page with subdirectory again and it fails to load properly with the error:
Page Error.txt

Expected behavior
subdirectory page should load the same in 2.9.6 and 2.9.7
Operating System
Docker Container via Unraid

Clicked the wrong button and accidentally closed the issue.

Had same issue. When using 2.9.9 was working, when updated to 2.9.11, experienced the exact issue you were.

Same issue, can provide logs if needed

Not sure if I would even be able to help, but I don't really understand the issue, to be specific what the Page Error.txt should mean. What exactly is the error that occurs? Where does it occur? In the browser? Is it returned by some network request? Is it an error in the npm logs? Host logs? Could any of you describe what exactly the error is? If you have any helpful additional information like logs, that's probably very useful, too.

So essentially, we all utilize organizr for auth services. Prior to the update, the setup was similar to the NPM JSON in OP, and it worked in requireing any proxy host to authenticate with organizr before providing access. After the update, it appears after providing said credentials, NPM would return the JSON in the Page Error.txt file instead of redirecting to the appropriate proxy host.

Possibly unrelated, if I try and edit one of the services that I use this auth with and then change it back, I now get 500 Internal server errors with the console saying:
Failed to load resource: the server responded with a status of 500 () service.domain.org/:1
For the time being to get past both issues, I have had to resort to setting basic auth in NPM, which is not ideal.

I have managed to work around the issue by moving the auth block for organizr from the location tab to the advanced tab in NPM
location ~ ^/auth-(.*) {
        ## Has to be local ip or local DNS name or if Proxied use proxy address
        proxy_pass http://organizr:80/api/v2/auth?group=$1;
        proxy_pass_request_body off;
        proxy_set_header Content-Length "";
        }

Like a lot of people I was following the IBRACORP instructions on getting Organizrs auth block setup in NPM. This worked until a recent update in NPM and I have since had to apply the workaround.
Essentially I have followed the IBRACORP guide exactly the same but have moved all the location tab settings to the advanced tab.

Me trying the above workaround is how I found I am now getting the 500 error.  Happens with it set or after reverting.

Me trying the above workaround is how I found I am now getting the 500 error. Happens with it set or after reverting.

Not sure if this is anything to do with it but did you remember to also add the auth block for organizr itself but using the block as per the instructions. The block is slightly different.
location ~ ^/auth-(.*) {
proxy_pass http://organizr:80/api/v2/auth?group=$1;
proxy_set_header Content-Length "";
internal;
 }

This is just a wild guess though and may not be related to your issue.

Me trying the above workaround is how I found I am now getting the 500 error. Happens with it set or after reverting.

Not sure if this is anything to do with it but did you remember to also add the auth block for organizr itself but using the block as per the instructions. The block is slightly different.
location ~ ^/auth-(.*) {
proxy_pass http://organizr:80/api/v2/auth?group=$1;
proxy_set_header Content-Length "";
internal;
 }

This is just a wild guess though and may not be related to your issue.

Yep, its been like that (proxy_pass address is the difference) since prior to version 2.9.6 of NPM.

My setup has been slightly different but essentially the same as I followed organizr's instructions from their documentation.

For S's and G's I have updated my settings to the current organizr documentation for a test and I still get a 500 error.

Okay, it might have to do with the move to a variable with the proxy pass directive. Try the following, edit the /app/templates/_location.conf file inside the container. You can do this however you wish, but you could for example connect a shell to the container and run apt update && apt install nano, and then use nano to edit the file.
Replace line 2 with the following very similar line:
    set              $upstream {{ forward_scheme }}://{{ forward_host }}:{{ forward_port }}{{ forward_path }}{% unless path contains "(" %}$request_uri{% endunless %};

Okay, it might have to do with the move to a variable with the proxy pass directive. Try the following, edit the /app/templates/_location.conf file inside the container. You can do this however you wish, but you could for example connect a shell to the container and run apt update && apt install nano, and then use nano to edit the file.
Replace line 2 with the following very similar line:
    set              $upstream {{ forward_scheme }}://{{ forward_host }}:{{ forward_port }}{{ forward_path }}{% unless path contains "(" %}$request_uri{% endunless %};

I would assume after the change this file, they need to edit the proxy (add a comment or something) to trigger NPM to rebuild the proxy file..?

Yes, that's correct, sorry I forgot to mention that

I've attempted this but still get the 500, I will restore an older config backup (one that I had the actual stated issue in op) later in case the changes I have made today have broken something.

I dont have the error 500 that Chrono is getting, but none-the-less, it did not work for me.

After restoring my config and making that change, and then making a change to a host Im back to the original error.

Looked at _location.conf of 2.9.9 and replaced that line 2 with what was in there (set              $upstream {{ forward_scheme }}://{{ forward_host }}:{{ forward_port }}{{ forward_path }};) but didnt work either

I'm gna set my test server back up and will test this.

Okay, it might have to do with the move to a variable with the proxy pass directive. Try the following, edit the /app/templates/_location.conf file inside the container. You can do this however you wish, but you could for example connect a shell to the container and run apt update && apt install nano, and then use nano to edit the file.
Replace line 2 with the following very similar line:
    set              $upstream {{ forward_scheme }}://{{ forward_host }}:{{ forward_port }}{{ forward_path }}{% unless path contains "(" %}$request_uri{% endunless %};

Then go ahead and edit the proxy host and save, to regenerate the config file.

This workaround worked for my testing on version v2.9.11 © 2019 jc21.com

@chaptergy i did some more testing...  While your edit does help set the upstream to not include the request_uri variable, the problem is 100% due to using the upstream variable in the proxy_pass directive.  Why? I'm not sure...  If I manually comment out the upstream variable and replace with what the variable was assigned with minus the request_uri variable all will work as expected.  I will do some more testing.

@chaptergy I see exactly what the issue is now.  We need to use a different variable name instead of upstream
I renamed it to : upstreamHost
location ~ /organizr-auth/(.*) {
    set              $upstreamHost http://<IP_ADDRESS>:8000/api/v2/auth/$1;
    proxy_set_header Host $host;
    proxy_set_header X-Forwarded-Scheme $scheme;
    proxy_set_header X-Forwarded-Proto  $scheme;
    proxy_set_header X-Forwarded-For    $remote_addr;
    proxy_set_header X-Real-IP          $remote_addr;
    proxy_pass       $upstreamHost;

and everything worked correctly.  So renaming it from upstream to anything else - such as upstreamHost and setting your work around to remove the request_uri variable will fix this.
I edited /app/templates/_location.conf to this:
  location {{ path }} {
    set              $upstreamHost {{ forward_scheme }}://{{ forward_host }}:{{ forward_port }}{{ forward_path }}{% unless path contains "(" %}$request_uri{% endunless %};
    proxy_set_header Host $host;
    proxy_set_header X-Forwarded-Scheme $scheme;
    proxy_set_header X-Forwarded-Proto  $scheme;
    proxy_set_header X-Forwarded-For    $remote_addr;
    proxy_set_header X-Real-IP          $remote_addr;
    proxy_pass       $upstreamHost;

    {% if access_list_id > 0 %}
    {% if access_list.items.length > 0 %}
    # Authorization
    auth_basic            "Authorization required";
    auth_basic_user_file  /data/access/{{ access_list_id }};

    {{ access_list.passauth }}
    {% endif %}

    # Access Rules
    {% for client in access_list.clients %}
    {{- client.rule -}};
    {% endfor %}deny all;

    # Access checks must...
    {% if access_list.satisfy %}
    {{ access_list.satisfy }};
    {% endif %}

    {% endif %}

    {% include "_assets.conf" %}
    {% include "_exploits.conf" %}

    {% include "_forced_ssl.conf" %}
    {% include "_hsts.conf" %}

    {% if allow_websocket_upgrade == 1 or allow_websocket_upgrade == true %}
    proxy_set_header Upgrade $http_upgrade;
    proxy_set_header Connection $http_connection;
    proxy_http_version 1.1;
    {% endif %}

    {{ advanced_config }}
  }

@causefx's solution worked for me

Can confirm that worked for me as well

@chaptergy I see exactly what the issue is now. We need to use a different variable name instead of upstream
I renamed it to : upstreamHost
location ~ /organizr-auth/(.*) {
    set              $upstreamHost http://<IP_ADDRESS>:8000/api/v2/auth/$1;
    proxy_set_header Host $host;
    proxy_set_header X-Forwarded-Scheme $scheme;
    proxy_set_header X-Forwarded-Proto  $scheme;
    proxy_set_header X-Forwarded-For    $remote_addr;
    proxy_set_header X-Real-IP          $remote_addr;
    proxy_pass       $upstreamHost;

and everything worked correctly. So renaming it from upstream to anything else - such as upstreamHost and setting your work around to remove the request_uri variable will fix this.
I edited /app/templates/_location.conf to this:
  location {{ path }} {
    set              $upstreamHost {{ forward_scheme }}://{{ forward_host }}:{{ forward_port }}{{ forward_path }}{% unless path contains "(" %}$request_uri{% endunless %};
    proxy_set_header Host $host;
    proxy_set_header X-Forwarded-Scheme $scheme;
    proxy_set_header X-Forwarded-Proto  $scheme;
    proxy_set_header X-Forwarded-For    $remote_addr;
    proxy_set_header X-Real-IP          $remote_addr;
    proxy_pass       $upstreamHost;

    {% if access_list_id > 0 %}
    {% if access_list.items.length > 0 %}
    # Authorization
    auth_basic            "Authorization required";
    auth_basic_user_file  /data/access/{{ access_list_id }};

    {{ access_list.passauth }}
    {% endif %}

    # Access Rules
    {% for client in access_list.clients %}
    {{- client.rule -}};
    {% endfor %}deny all;

    # Access checks must...
    {% if access_list.satisfy %}
    {{ access_list.satisfy }};
    {% endif %}

    {% endif %}

    {% include "_assets.conf" %}
    {% include "_exploits.conf" %}

    {% include "_forced_ssl.conf" %}
    {% include "_hsts.conf" %}

    {% if allow_websocket_upgrade == 1 or allow_websocket_upgrade == true %}
    proxy_set_header Upgrade $http_upgrade;
    proxy_set_header Connection $http_connection;
    proxy_http_version 1.1;
    {% endif %}

    {{ advanced_config }}
  }

How would someone who uses/installs dockers via unRaid implement this solution? Are we able to edit the container as stated above as well?

Yep same issue, hope fix comes in unraid docker image soon!

@chaptergy I see exactly what the issue is now. We need to use a different variable name instead of upstream
I renamed it to : upstreamHost
location ~ /organizr-auth/(.*) {
    set              $upstreamHost http://<IP_ADDRESS>:8000/api/v2/auth/$1;
    proxy_set_header Host $host;
    proxy_set_header X-Forwarded-Scheme $scheme;
    proxy_set_header X-Forwarded-Proto  $scheme;
    proxy_set_header X-Forwarded-For    $remote_addr;
    proxy_set_header X-Real-IP          $remote_addr;
    proxy_pass       $upstreamHost;

and everything worked correctly. So renaming it from upstream to anything else - such as upstreamHost and setting your work around to remove the request_uri variable will fix this.
I edited /app/templates/_location.conf to this:
  location {{ path }} {
    set              $upstreamHost {{ forward_scheme }}://{{ forward_host }}:{{ forward_port }}{{ forward_path }}{% unless path contains "(" %}$request_uri{% endunless %};
    proxy_set_header Host $host;
    proxy_set_header X-Forwarded-Scheme $scheme;
    proxy_set_header X-Forwarded-Proto  $scheme;
    proxy_set_header X-Forwarded-For    $remote_addr;
    proxy_set_header X-Real-IP          $remote_addr;
    proxy_pass       $upstreamHost;

    {% if access_list_id > 0 %}
    {% if access_list.items.length > 0 %}
    # Authorization
    auth_basic            "Authorization required";
    auth_basic_user_file  /data/access/{{ access_list_id }};

    {{ access_list.passauth }}
    {% endif %}

    # Access Rules
    {% for client in access_list.clients %}
    {{- client.rule -}};
    {% endfor %}deny all;

    # Access checks must...
    {% if access_list.satisfy %}
    {{ access_list.satisfy }};
    {% endif %}

    {% endif %}

    {% include "_assets.conf" %}
    {% include "_exploits.conf" %}

    {% include "_forced_ssl.conf" %}
    {% include "_hsts.conf" %}

    {% if allow_websocket_upgrade == 1 or allow_websocket_upgrade == true %}
    proxy_set_header Upgrade $http_upgrade;
    proxy_set_header Connection $http_connection;
    proxy_http_version 1.1;
    {% endif %}

    {{ advanced_config }}
  }

Tried this but nothing, I hope for a fix in the next updates

Any news on this, seems to still be a problem

Any news on this, seems to still be a problem

They pushed out a new release and everything works fine for me now. Are you sure you're having the same problem as others on this thread?

I think so, I followed the IBRACORP guide to setup NPM with Organizr and it just stopped working after a update.
I have updated to the latest version but I still get this when trying to access the webpage through Organizr and the NMP setup:
{
"response": {
"result": "success",
"message": "User is authorized",
"data": {
"user": "MY-SUERNAME",
"group": 0,
"email": "MY-EMAIL",
"user_ip": "IP",
"requested_group": 0
(One thing I wdont know why its says group 0 the setup provides group 1)

make sure to make a change to the proxy so it re-saves.  It needs to compile the conf file again.

I'm still having an issue using this with unraid. I get the same message as casperse.

i am also having this issues after following along with the docs here: https://docs.organizr.app/features/server-authentication/nginx-server-authentication
i just get the json payload saying user is authorized and i am using a fresh install of the latest version  (2.9.12)
i looked at the _location.conf file and it isnt using the "$upstream" variable already, it uses "$targetUri" and has additional logic to what is displayed in previous posts here.

I'm not sure if they have merged the reversal yet

This seems to be happening again with 2.9.12. Think this needs to be reopened.


Unable to retrieve job result. QiskitError: "The QPY format version being read, 10, isn't supported by this Qiskit version. Please upgrade your version of Qiskit to load this QPY payload

Environment

0.45.2:
3.10.13:
Windows 11:

What is happening?
I am running through Q-CTRLs Get Started guide here: https://docs.q-ctrl.com/q-ctrl-embedded/ibm/get-started. When I run the code below:
from qiskit_ibm_runtime import Sampler, Options

shot_count = 2048

backend = service.backend(backend_name)
options = Options()
options.execution.shots = shot_count
sampler = Sampler(backend=backend, options=options)

q_ctrl_result = sampler.run(circuit_qis).result()
q_ctrl_probabilities = (
    q_ctrl_result.quasi_dists[0]
    .nearest_probability_distribution()
    .binary_probabilities(num_bits=11)
)

I receive this error: 'Unable to retrieve job result. QiskitError: "The QPY format version being read, 10, isn't supported by this Qiskit version. Please upgrade your version of Qiskit to load this QPY payload'
How can we reproduce the issue?
To reproduce the errors follow the guide here for the Get Started documentation: https://docs.q-ctrl.com/q-ctrl-embedded/ibm/get-started. Enter your IBM Cloud API keys and IBM Cloud CRN and follow the steps up the this section of code:
from qiskit_ibm_runtime import Sampler, Options

shot_count = 2048

backend = service.backend(backend_name)
options = Options()
options.execution.shots = shot_count
sampler = Sampler(backend=backend, options=options)

q_ctrl_result = sampler.run(circuit_qis).result()
q_ctrl_probabilities = (
    q_ctrl_result.quasi_dists[0]
    .nearest_probability_distribution()
    .binary_probabilities(num_bits=11)
)

What should happen?
The results should be returned to the device, and then you can plot the results and calculate the success probability, which estimates the probability that a single execution (shot) of the circuit will yield a solution.
Any suggestions?
No response

The error indicates a qpy version mismatch between your local environment and the remote execution environment. I encourage you to raise this issue in https://github.com/Qiskit/qiskit-ibm-runtime to point out the need for updating qpy. While the fix is being rolled out, I suggest downgrading your local version to qiskit 0.44 if you can, to match the remote version of qpy (qpy 10 was introduced in qiskit 0.45).


Feature/add macd indicators

Closes https://github.com/zackurben/alphavantage/issues/21 and adds both MACD and MACDEXT technical indicators with test.
Note: I ran into a data issue, but sent an email to customer support to attempt to get it fixed.

Coverage decreased (-1.4%) to 98.649% when pulling e80d79f0446954a6cf57563093e44dc76f16141d on feature/add-macd-indicators into 4fd1d4d7b4ef70afbd3b91d6f42d54ba40e0a1bb on master.

Coverage decreased (-1.4%) to 98.649% when pulling e80d79f0446954a6cf57563093e44dc76f16141d on feature/add-macd-indicators into 4fd1d4d7b4ef70afbd3b91d6f42d54ba40e0a1bb on master.

Coverage remained the same at 100.0% when pulling e99ba5b8dd1c7c3fc52e70c780bd454612c15084 on feature/add-macd-indicators into 4fd1d4d7b4ef70afbd3b91d6f42d54ba40e0a1bb on master.


Validate token in current_user

We figured that current_user(line 220) function when receiving a token, doesn't validate the token like lib does in admin_token(line 141) method.
Is it by design or can I create a fix for that?

Hey @valeriiduz,
I can not find the flaw you’re mentioning. The current_user method utilizes the _decode_token method which in turn verifies the token via public key. Could you provide the mentioned code snippet?

In lib, we have token_is_valid method which can check token is valid or not, but in current_user we don't use them. For example, in admin_token we have a condition for that
    @property
    def admin_token(self):
        """ Holds an AccessToken for the `admin-cli` client

        Returns:
            KeycloakToken: A token, valid to perform admin actions

        Notes:
            - This might result in an infinite recursion if something unforeseen goes wrong
        """
        if self.token_is_valid(token=self._admin_token):
            return self._admin_token
        self._get_admin_token()
        return self.admin_token

and I want to add the similar condition to current_user method. Something like that
        def current_user(token: OAuth2PasswordBearer = Depends(self.user_auth_scheme)) -> OIDCUser:
            """ Decodes and verifies a JWT to get the current user

            Args:
                token OAuth2PasswordBearer: Access token in `Authorization` HTTP-header

            Returns:
                OIDCUser: Decoded JWT content

            Raises:
                ExpiredSignatureError: If the token is expired (exp > datetime.now())
                JWTError: If decoding fails or the signature is invalid
                JWTClaimsError: If any claim is invalid
                HTTPException: If any role required is not contained within the roles of the users
            """
            if not self.token_is_valid(token=token, audience="account"):
                return None
            decoded_token = self._decode_token(token=token, audience="account")
            user = OIDCUser.parse_obj(decoded_token)
            if required_roles:
                for role in required_roles:
                    if role not in user.roles:
                        raise HTTPException(status_code=403, detail=f'Role "{role}" is required to perform this action')
            return user

        return current_user

Ah okay! So, the admin token is managed by this library, the user tokens are usually your custom implementation. Both the admin token and the user tokens are validated. The validation you're referring to here determines whether the admin token is still valid, it checks if the exp (expiry) is reached yet and if the token has expired, a new one is requested. There is not much sense in providing the same functionality for the user tokens, as they're not managed by this library. You could either wrap the functions and implement a custom handler for the exception that is thrown for expired user tokens or check the exp upfront.
I hope I got you right?

Yep, got it. You are right, better to leave this approach upfront, cause different systems can have different flows to regenerate token for users. Thanks! I'll close the issue.


⚠ ClamAV has degraded performance

In 7aa4df2, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 67 ms

Resolved: ClamAV performance has improved in a983686.


Dev tools not themed properly under Firefox 69

Dev tools has not been themed properly under Firefox 69.
Screenshot

I installed shadowfox using the shadowfox-updater and I did not encounter any issues while running it. Every other part of the UI seems to have been themed properly except for this one.

Firefox 69.0.1 (64-bit) stable
Linux (Fedora KDE 30)
Just open dev tools ctrl+shift+I
Installed via shadowfox updater.

Open developer tools settings (F1 or triple-dot-menu -> Settings) and choose Dark theme in the settings. Works wonders.

Open developer tools settings (F1 or triple-dot-menu -> Settings) and choose Dark theme in the settings. Works wonders.

Thank you! I hadn't done so, never made sense to do it with light firefox because I mostly inverted the color of the window with a desktop extension. If it's at all possible to do via userChrome.css, maybe a future version of the scripts can perform that change for the user?


Added Gradient Waves Background

Fixes Issue🛠
Closes #578
Description👨💻
I added the Gradient Waves Background in Backgrounds section.
Type of change📄

[✅] New feature (non-breaking change which adds functionality)

How this has been tested✅
in my laptop.
Checklist✅

[✅] My code follows the style guidelines of this project
[✅] I have performed a self-review of my own code
[✅] I have commented my code, particularly in hard-to-understand areas
[✅] I have added demonstration in the form of GIF/video file
[✅] I am an Open Source Contributor

Screenshots/GIF📷

Merged 🥳✨ @mansi1309


WIP: motor ramp test app

WARNING: do not use this command unless you know exactly what you are doing, it will spin up all connected motors!
This adds a new system command to do motor ramp tests. Based on bad experience with low KV quad setups we felt it is necessary to analyze this more closely. The issue with low KV setups is that you can easily over tune them and get props to stall. This app should accomplish 2 things:

clarify and quantify the problem
measure the limits of a specific system so we can tune it accordingly

If we understand the problem well enough we could then add protective measures, for example a PWM output slew rate limit.
TODO:

[ ] safety: what kind of protective measures should still be built in here to prevent this from accidentally going off? (at least I'll take the same protection over from the esc_calib routine)
[ ] sinus ramp: @tumbili

Results of initial tests with a TBS endurance setup doing a linear ramp with different timings and min PWM to start with. The TBS endurance setup is one that we were easily able to get to flip on aggressive corrections when tuned tightly. Now on the bench it reacted actually better than I expected, but it clearly shows the problem. Also it shows that minimum PWM is important.
Setup: 6S 400kV max ~400W 15x5 inch prop
1100 min, ramp to full in 0.2 sec, all good:

1100 min, ramp to full in 0.1 sec, motor out of sync:

1200 min, ramp to full in 0.1 sec, all good:

1200 min, ramp to full in 0.05 sec, motor our of sync:

Also, the following is an example of a real flight (tuning test) with a large quad plane where it actually flipped, the ramp of the motor/prop that stalled was actually 0.4 seconds long! (but with very low PWM before the ramp). Setup: 6S 440kV max ~900W 18x5.5 inch prop

@kd0aij My thinking was to create the closed loops transfer function from your script for one of these bigger systems. We could read the bandwidth from that and based on this come up with an initial guess for a slew limiter on the outputs, e.g.
say you excite your system with
u(t) = sin(omega * t)
where omega is the bandwidth of the closed loop system
the derivative of that signal is
u'(t) = omega * cos(omega * t)
so the largest possible magnitude of this function is omega.
So choosing the max slew rate to be at least a magnitude larger than then controller bandwidth might be a good initial choice.

@tumbili @AndreasAntener Since I don't have any heavy-lift quads on hand, perhaps one of you could generate a log for me by flying the newLogger branch. It would then be straightforward to do the analysis that Roman suggests.

@kd0aij As soon as we have the new logger branch in a state to go into master we can do that.

Jenkins: test this please

Here's a short video from one of the tests where the ramp failed: https://youtu.be/9BRfw0Jipu4

@AndreasAntener Looks good, I'll add the sinus mode to it.

Reviewed, looks good, only minor comments. Nice analysis and plots!

@AndreasAntener @julianoes I've added an option to output a sine wave. The app will run through 3 periods of the sine. Will test this asap.

Where is this standing?

It's in the test queue..

@AndreasAntener Do you have an update on this?

Updated, sine ramp tested (thanks @tumbili), and cleaned-up

Now from here to https://github.com/PX4/Firmware/issues/3080 :)


libcalico yields misleading error from bad ETCD_AUTHORITY env var

If ETCD_AUTHORITY environment is missing a colon, pycalico will fail to initialize the IPAMClient with  a misleading stack trace.
$ ETCD_AUTHORITY=erroneous python -c "import pycalico.ipam; pycalico.ipam.IPAMClient()"
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/usr/local/lib/python2.7/dist-packages/pycalico/datastore.py", line 137, in __init__
    (host, port) = etcd_authority.split(":", 1)
ValueError: need more than 1 value to unpack

Link to line 137
I propose that we should catch the value error and reword it to state "malformed ETCD_AUTHORITY", possibly in the form of a new MalformedEtcdAuthority exception?

SGTM


fix(metrics): fix Prometheus env variable in the MCE consumer deployment

When .Values.global.datahub.monitoring.enablePrometheus is set to true, GMS and MAE deployments get a JMX port opened and ENABLE_PROMETHEUS set, but MCE deployment only gets the port. I assume it's a mistake and I'm adding this variable to MCE.

DataHub main chart version should get a bump to pass the linting, but otherwise looks okay to me. Not sure if @jjoyce0510 has any context on if this was intentionally excluded or just missed.


[FEATURE REQUEST] gRPC internal implementation

Is your feature request related to a problem? Please describe.
Currently gRPC server implementation is outside of this repository (located here: https://github.com/LdDl/license_plate_recognition). It's hard to figure it out how to use gRPC properly (may be no clear enough).
Describe the solution you'd like
Add folder 'server_example' inside of parent folder 'cmd' and add main.go file with simple code example which can be used as boilerplate.
Describe alternatives you've considered
Another repository with boilerplated gRPC server
Additional context
nope

https://github.com/LdDl/odam/commit/4fddb6a287d38e761708c4a24b7ea9609a9e98c4

If we'll make #18 to be done that we can close this issue.


[Feature Request]: Added parameters to the timezone validation rule

Feature Description
Added parameters to the timezone validation rule in v10.12.0
laravel/framework#47171

fixed in 1.24.2


shell: add a directory for backends

This change organizes the file structure for the shell.
Files implementing backends are moved to a separate folder.
Signed-off-by: Jakub Rzeszutko<EMAIL_ADDRESS>

@jakub-uC can you rebase please?

@carlescufi  Some tests do not pass while they are not related to my change. What's more, locally the same tests pass correctly.


how can I generate apk file for android project

[x] Review the documentation: https://github.com/oblador/react-native-vector-icons
[x] Search for existing issues (including closed ones): https://github.com/oblador/react-native-vector-icons/issues

Environment

Description
Describe your issue in detail. Include screenshots if needed.
Demo
You can use https://snack.expo.io/ to create a demo that can help users to better understand your problem.

Not RNVI related


20293

Issue #: bcgov/entity#20293
Description of changes:

updated NAICS URL in example env file
app version = 5.0.1
updated NAICS API reference in vaults file

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).

/gcbrun

Temporary Url for review: https://business-edit-dev--pr-565-i4dx0lm6.web.app


missing forms do not generate error's if years pulled are less

Hi Jack
Thanks a lot for your help in advance, i took the latest package of secedgar 0.1.7 today(previously i was on 0.1.6. I was trying to pull 10-k forms for various companies and things were working good. I was able to pull a few companies but when i pull the symbol 'dis' for 10 years of records i see only 2019 file. Then i check secedgar page and it only show results for 2019. The question i have is that can some error handling or reporting be added to the API's to say that n number if years were not fetched and only x years were fetched ?
eg
my_filings = Filing(cik_lookup='dis', filing_type=FilingType.FILING_10K, count=10)
my_filings.save('/Users/folder-to-save-on-my-local-disk')
In case you have more questions please let me know. I have subscribed this repo
Thanks
~Rahul

@rahul18cracker thanks for sharing this. That sounds like a good idea and should not be too difficult to implement. I’ll work on adding a warning for v0.2.0.


number-names crate deprioritized in build queue

The two published versions of my crate number-names has been assigned a -1 priority in the build queue.  Rust Discord user jynelson requested I open this issue for investigation.

This was an unintended consequence of our rule to deprioritize big monorepos in the build queue. I fixed the pattern and now it should work properly!


Sometimes throwing error "Not an undo file" when opening certain files, then later deleting the contents of the file

Sometimes, when opening a file via Harpoon (or toggling to it) for the first time in an nvim session, I will get an error of this form:
E51308: Error executing lua ...e/nvim/site/pack/packer/start/harpoon/lua/harpoon/ui.lua:195: Vim:E823: Not an undo file: /home/<user>/.vim/undodir/%home%<user>%<filepath>
stack traceback:
   [C]: in function 'nvim_set_current_buf'
   ...e/nvim/site/pack/packer/start/harpoon/lua/harpoon/ui.lua:195: in function 'nav_file'
   ...e/nvim/site/pack/packer/start/harpoon/lua/harpoon/uilua:163: in function select_menu_item'
   [string ":lua"]:1:" in main chunk

At a certain point later on, the contents of my file will be entirely deleted. The file will exist in the undodir, but it will be entirely empty. It will remain empty, even if I save edits to the file, remove the file from harpoon, etc..  An undotree does grow during a given nvim session, but when starting a new session, it seems to revert back to an empty file.
I'm not sure that this is harpoon causing the issue, because harpoon might just be finding the undofile empty, and there might be some other cause.  But I only ever get this issue with files that I harpoon, so I feel that harpoon must be involved somehow.
Using native vim "marks" might contribute to the issue. Not sure.

@haydenstephan maybe is a good opportunity to migrate to Harpoon 2.
https://github.com/ThePrimeagen/harpoon/tree/harpoon2
If the error continues, we can get better logs to fix it, and you'll use the latest version with a lot of improvements.

are you using harpoon1 or 2?

are you using harpoon1 or 2?

I only found ui.lua:195 at harpoon 1


Fixed bug from #152.

Problem with overriden Enum serialization to Map.
We added also tests confirming the problem.
Kamil Krol & Dreando

@KamilKrol ; can you rebase to force re-test please

@KamilKrol ; any update?

@garethahealy , done. Let me know if you need something additional before merging this pull request.
Kamil

@orange-buffalo ; can you review please

@KamilKrol ; instead of using "expected=Exception", can you use the JUnit rule checker instead.

https://github.com/DozerMapper/dozer/blob/master/core/src/test/java/org/dozer/functional_tests/GranularDozerBeanMapper_WithExceptionLoggedTest.java#L36

As currently, MappingException is for everything, so we wouldn't know if this test was actually broken in the future.

@KamilKrol ; are you able to work on the suggested changes?

@garethahealy , I fixed all comments from code review and committed latest version.
But it seems that some OSGI tests are failing.

@KamilKrol ; the OSGi failure was a test flake.

LGTM
@orange-buffalo ?

https://github.com/DozerMapper/dozer/issues/152

@garethahealy , perfect that this OSGI failure is understood.
Let me know if I can do anything more for this pull requests.


GC+: Optimized Run Discovery

Breakdown of the optimized run efforts into manageable tasks

Can be closed for now


Implement Stop Command and Button in UI

At the moment you are only able to start paper traders and market watchers, but you cannot stop them.
Please implement such a function.

This is a very important feature that is still missing in Gekko. See also #2059

This has since been implmented in 0.6!


Release v3.2.4

Summary
Diff: https://github.com/mattermost/mattermost-plugin-jira/compare/v3.2.3...release_v3.2.4
Open Tickets

[x] https://github.com/mattermost/mattermost-plugin-jira/pull/915
[x] https://github.com/mattermost/mattermost-plugin-jira/pull/919

Codecov Report
Patch coverage has no change and project coverage change: -0.04 :warning:

Comparison is base (e38c741) 31.15% compared to head (f55f14c) 31.12%.

Additional details and impacted files
@@               Coverage Diff               @@
##           release-3.2     #917      +/-   ##
===============================================
- Coverage        31.15%   31.12%   -0.04%
===============================================
  Files               49       49
  Lines             6040     6040
===============================================
- Hits              1882     1880       -2
- Misses            3969     3971       +2
  Partials           189      189

Impacted Files
Coverage Δ

server/subscribe.go
67.61% <0.00%> (-0.38%)
:arrow_down:

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

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

@mickmister and @hanzei I believe we should be ready to merge this and cut a release.


Sequences with wait(60000) can't be executed from the API

Expected Behavior
Sequence with wait(60000) command (or longer) can be executed from the API
Actual Behavior
node = {'kind': 'execute', 'args': {'sequence_id': sequence['id']}}
response = requests.post(self.farmware_url + 'api/v1/celery_script', data=json.dumps(node),
                        headers=self.headers)

second line generates exception
('Connection aborted.', BadStatusLine("''",))

Steps to Reproduce

create a sequence open watering, wait 60+ sec, stop watering
run the script above - you get exception
change the wait duration to 59s, run script - no exception

note: any sequence can be executed from the web page without any problems.
Version 6.4.8

This appears to be a bug in python?

https://bugs.python.org/issue8450
https://bugs.python.org/issue8450

I'm updating underlying Linux system deps today. I'm not sure if it has the fix of not, but i'll update this issue when it's out and test it out.

Turns out there is no fix for this in python2 from what i can tell. I can probably increase the http keep-alive header, but i'm not sure that is a solution, more of a hack/patch.
This is sort of a problem with HTTP itself, and us overloading its intended functionality, although i'm not sure of many other solutions that are simple to implement.

Also for what it's worth, anything celeryscript node that takes longer than 60 seconds will be broken. (this includes long movements, compound short movements etc)

I propose to increase timeout as a short term remedy. This feature worked before and got broken few weeks ago. Some people were affected as you can see

Fixed in #620


textual update samples from v3 to v4

All the linked examples actually use bot framework 4 , not v3 as was mentioned.
additionally updated the sentence structure .

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/concepts/authentication/authentication.md
85
12
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit f921947:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/concepts/authentication/authentication.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


how about chinese

I am a native Chinese speaker. May I ask if you have any inclination to sinicize? Or need any help?

I don't believe there's any active work on supporting more languages. Some people have worked on it in the past, but either stopped due to lack of time/motivation or haven't responded to feedback on what their solution is missing before it can be accepted.
For clarity, the prices used by WFInfo correspond to the global version of Warframe. If you're playing the china-specific version and not the global version with language set to chinese, the displayed prices would not be accurate.


Model builder generates non-partial classes in Dll mode.

Hi.
I used dotPeek to see the model generated by ModelBuilder. And I realized that the model has been generated as non-partial classes. Is that ok? Is there any way to force ModelBuidler to generate partial classes in "Dll" mode.
Thanks
Sebastian

All models should be generated as partial classes. That is, the source code indicates that the classes are partial classes. When compiling, all partial source code bits are assembled into one class, and "partial" does not make much sense at binary level. So... it would be normal that dotPeek cannot expose that the class is partial. What you want to look at is the source.

 I forgot that I can't have partial across assemblies. As you said - partial does not make sense at binary level.
I'm sorry for that confusion. And of course many thanks for quick response.


[JOSS Review] Package Description Missing from PyPI

The PyPI page does not show the package description.

https://github.com/openjournals/joss-reviews/issues/5703


Update command wallet/address to use V2 libraries

This PR updates the wallet/address command so it uses V2 libraries.

@faustbrian Travis complaining it misses write access:
npm WARN checkPermissions Missing write access to /home/travis/build/ArkEcosystem/javascript-cli/node_modules/arkjs
How to proceed?

Merging solves that :)


Define data source plugin type that wraps/refers to extractor, transform, etc

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/1973
Originally created by @DouweM on 2020-04-21 23:00:52

View 10 previous comments from the original issue on GitLab


Noob Guide

Hi All, this looks to be a great tool, however, i've never used Python, so any other programming language. Is there a Noob version of this where it would say:

Install Python 3 (select C:/ folder to install, etc.)
Install Chrome Driver
Run Python.exe (i don't know if this the right way to run)
Etc.

Or any good videos I can view to learn some basics and incorporate this script.
Sorry, i know this is not an issue, however, I wasn't sure how to add comments here.
Thanks again.

You can install Python  2.7 for Windows: https://datascience.com.co/how-to-install-python-2-7-and-3-6-in-windows-10-add-python-path-281e7eae62a
Once you do that and have cloned the repository: git clone <repo-name>
You can run the script of choice by python <script-name>.py from the terminal

https://www.notion.so/using-pcomputo-s-script-to-find-whole-foods-delivery-slots-acbb6d71ef934da7b6822b1df451a11c


Add no_totals argument to claim search

Does passing --no_totals do anything different? If I recall correctly the feature was dropped to return totals in the first place so --no_totals is redundant I think.

Code exists throughout the SDK to support totals (and use of --no_totals flag), including in jsonrpc_claim_search.  Do you mean that the feature of --no_totals throughout the code was dropped, or specific to claim_search?  Neither is true, as far as I see.
The result structure is appended with totals unless the flag is used, with the submitted PR.  See below:
(lbry-venv) patrick@illuminati:~/Development/lbry/lbry-sdk$ lbrynet claim search | tail -10
        "title": "Life in the Iron-Mills; Or, The Korl Woman"
      },
      "value_type": "stream"
    }
  ],
  "page": 1,
  "page_size": 20,
  "total_items": 9910688,
  "total_pages": 495535
}
(lbry-venv) patrick@illuminati:~/Development/lbry/lbry-sdk$ lbrynet claim search --no_totals | tail -10
        },
        "stream_type": "binary",
        "title": "Life in the Iron-Mills; Or, The Korl Woman"
      },
      "value_type": "stream"
    }
  ],
  "page": 1,
  "page_size": 20
}
(lbry-venv) patrick@illuminati:~/Development/lbry/lbry-sdk$

@patrickkeane thank you for this PR, please follow these instructions so that we can show you some appreciation: https://lbry.com/faq/appreciation

Perhaps issue #2788 can be closed.

Perhaps issue #2788 can be closed.

In the future if you put "fixes #2788" in the description of your PR the issue will automatically be closed when your PR is merged.


Option to hide noise

Dear team,
could you please add an option to hide all the noise of "Not vulnerable" URLs.

Maybe just display success URL and progress how many payloads were executed like Gobuster and ffuf.

not user friendly bcz if some sites are not vulnerable then its show nothing and people think its not working so thats why we used verbose so they know its testing the paylaods


chore(main): release 1.1.2

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

update tabela IRPF e Aliquota 2024 e 2025. (7c2aae7)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/diillson/calculador-de-plr/releases/tag/v1.1.2 :sunflower:


How do I configure a public ip address for the StarRocks BE node for external network connection test

General Question
How do I configure a public ip address for the StarRocks BE node for external network connection test
I looked at the source code for starrocks-connector-for-apache-flink. which gets the ip address of the BE node by calling http://<fe_ip>:8030/api//<table_name>/_query_plan.
However, it can only get the Intranet ip of the BE node, causing my local starrocks-flink project to fail.

Our Starrocks cluster is on a cloud server

It's over by VPN


Warn/deprecate continuing on empty lines in Dockerfile

This fix is related to #29005 and #24693. Currently in Dockerfile empty lines will continue as long as there is line escape before. This may cause some issues. The issue in 24693 is an example. A non-empty line after an empty line might be considered to be a separate instruction by many users. However, it is actually part of the last instruction under the current Dockerfile parsing rule.
This fix is an effort to reduce the confusion around the parsing of Dockerfile. Even though this fix does not change the behavior of the Dockerfile parsing, it tries to deprecate the empty line continuation and present a warning for the user. In this case, at least it prompt users to check for the Dockerfile and avoid the confusion if possible.
Here are some of the examples:

No warning for comments continuation

ubuntu@ubuntu:~/tmp$ cat Dockerfile
FROM busybox
RUN echo \
# comment
hi
ubuntu@ubuntu:~/tmp$ sudo docker build -t test .
Sending build context to Docker daemon  11.2 MB
Step 1/2 : FROM busybox
 ---> e02e811dd08f
Step 2/2 : RUN echo hi
 ---> Running in 982812e845b2
hi
 ---> 58c355115a35
Removing intermediate container 982812e845b2
Successfully built 58c355115a35
ubuntu@ubuntu:~/tmp$

Warning for empty line continuation:

ubuntu@ubuntu:~/tmp$ cat Dockerfile
FROM busybox
RUN echo "foo" \
#&& echo "bar";

# Do something else
RUN echo "something else"

RUN echo "end"
ubuntu@ubuntu:~/tmp$ sudo docker build -t test .
Sending build context to Docker daemon  11.2 MB
[WARNING]: Empty lines detected in the following instruction:
[WARNING]:   RUN echo "foo" \
[WARNING]:   #&& echo "bar";
[WARNING]:
[WARNING]:
[WARNING]:
[WARNING]:   # Do something else
[WARNING]:   RUN echo "something else"
[WARNING]: Empty lines inside a RUN instruction is deprecated, and will be removed in Docker 1.16.
Step 1/3 : FROM busybox
 ---> e02e811dd08f
Step 2/3 : RUN echo "foo" RUN echo "something else"
 ---> Running in 0c95c73c0e6b
foo RUN echo something else
 ---> e94a89843d4b
Removing intermediate container 0c95c73c0e6b
Step 3/3 : RUN echo "end"
 ---> Running in 22d749190d5d
end
 ---> 82e486951d62
Removing intermediate container 22d749190d5d
Successfully built 82e486951d62

No warning as long as there are escaping symbols for each line:

ubuntu@ubuntu:~/tmp$ cat Dockerfile
FROM busybox
RUN echo "foo"; \
\
# This is a comment
\
    # So is this
    \
    echo "and this is part of the same RUN"

RUN echo "this is the next RUN"
ubuntu@ubuntu:~/tmp$ sudo docker build -t test .
[sudo] password for ubuntu:
Sending build context to Docker daemon  11.2 MB
Step 1/3 : FROM busybox
 ---> e02e811dd08f
Step 2/3 : RUN echo "foo";         echo "and this is part of the same RUN"
 ---> Running in cf785c61ed05
foo
and this is part of the same RUN
 ---> e375992a8a57
Removing intermediate container cf785c61ed05
Step 3/3 : RUN echo "this is the next RUN"
 ---> Running in cba814794d62
this is the next RUN
 ---> 712c017c8ef1
Removing intermediate container cba814794d62
Successfully built 712c017c8ef1
ubuntu@ubuntu:~/tmp$

Signed-off-by: Yong Tang<EMAIL_ADDRESS>

How does this affect the #escape= directive? The recommendation was to ensure there WAS a blank line between the #escape= and the FROM statement? (Sorry, not looked in detail)

/cc @thaJeztah @duglin
I did put 1.14 milestone on it 👼

How does this affect the #escape= directive? The recommendation was to ensure there WAS a blank line between the #escape= and the FROM statement? (Sorry, not looked in detail)

@jhowardmsft this is only for blank lines inside a RUN statement, I.e., of someone uses a blank line without using \ (or backtic in PowerShell) as a line continuation marker

SGTM
I can't seem to find the spot where we agreed on the plan... what was the decision for comments in continuation lines?  Deprecate those too or just leave them?

I'm not sure the current implementation is correct. Should have a look if we can do it otherwise, but just printing a list of "Skipping empty line ...." before the docker build may not provide enough information to the user to resolve the issue.
Haven't had time to look into it further, but thought to just leave a comment for now :blush:

I think one issue is that, currently the output of build always shows the instruction in one line (after processing). For example,
FROM busybox
RUN echo "foo" \
#&& echo "bar";

# Do something else
RUN echo "something else"
ubuntu@ubuntu:~/tmp$ sudo docker build -t test .
Sending build context to Docker daemon  11.2 MB
Step 1/2 : FROM busybox
 ---> e02e811dd08f
Step 2/2 : RUN echo "foo" RUN echo "something else"
 ---> Running in 93c62c013a56
foo RUN echo something else
 ---> 03b1c6850531
Removing intermediate container 93c62c013a56
Successfully built 03b1c6850531
ubuntu@ubuntu:~/tmp$

In the above example, we only see:
Step 2/2 : RUN echo "foo" RUN echo "something else"

which might be confusing initially.
I think for the generated warning, it might make sense to output the instruction in the original format, e.g.,
[WARNING]: Empty lines detected in:
[WARNING]: [INSTRUCTION] RUN echo "foo" \
[WARNING]: [INSTRUCTION] #&& echo "bar";
[WARNING]: [INSTRUCTION]
[WARNING]: [INSTRUCTION]
[WARNING]: [INSTRUCTION] # Do something else
[WARNING]: [INSTRUCTION] RUN echo "something else"
[WARNING]: Empty lines inside an instruction is deprecated, and will be removed in Docker 1.16.

The PR has been updated and the warning message has been updated.
Below is the new output to give user a better idea of which instruction (multiple lines) could be ambiguous:
ubuntu@ubuntu:~/tmp$ cat Dockerfile
FROM busybox
RUN echo "foo" \
#&& echo "bar";

# Do something else
RUN echo "something else"

RUN echo "end"
ubuntu@ubuntu:~/tmp$ sudo docker build -t test .
Sending build context to Docker daemon  11.2 MB
[WARNING]: Empty lines detected in the following instruction:
[WARNING]:   RUN echo "foo" \
[WARNING]:   #&& echo "bar";
[WARNING]:
[WARNING]:
[WARNING]:
[WARNING]:   # Do something else
[WARNING]:   RUN echo "something else"
[WARNING]: Empty lines inside a RUN instruction is deprecated, and will be removed in Docker 1.16.
Step 1/3 : FROM busybox
 ---> e02e811dd08f
Step 2/3 : RUN echo "foo" RUN echo "something else"
 ---> Running in 0c95c73c0e6b
foo RUN echo something else
 ---> e94a89843d4b
Removing intermediate container 0c95c73c0e6b
Step 3/3 : RUN echo "end"
 ---> Running in 22d749190d5d
end
 ---> 82e486951d62
Removing intermediate container 22d749190d5d
Successfully built 82e486951d62

Ain't it enough for those corner cases to escape it like \#&& echo "bar" and let the current behavior as it is?

@Yajo The issue is fairly complicated. One issue is that, under current behavior, a combination of escape with comment will result in the continuation beyond the last comment.
Below is the example:
FROM busybox
RUN echo "foo" \
#&& echo "bar";

# Do something else
RUN echo "something else"

The above Dockerfile resulting in:
Step 1 : FROM busybox
 ---> 47bcc53f74dc
Step 2 : RUN echo "foo" RUN echo "something else"
 ---> Running in 991b76486899
foo RUN echo something else
 ---> d488a4c21b22

See issue #24693 for reference.

Yeah, but that's what I'd expect given that Dockerfile. I mean that you should fix it like this:
FROM busybox
RUN echo "foo" \
\#&& echo "bar";

# Do something else
RUN echo "something else"

... isn't it?

@duglin @thaJeztah did we decide on design here?

We discussed this in the maintainers meeting and think the warning is a great thing to have, but we're still a bit worried about actually deprecating it. We should, but not sure how long to take before removing it.
Adding a version to the dockerfile format would help 😇  https://github.com/docker/docker/issues/4907

We also should have some tests for these :smile:

Would be great to see a unit test for either scenario - a test with a Dockerfile that should and shouldn't error.

Would love to see tests for both scenarios as well. They would be of great help to those of us writing Dockerfile editors and/or linters.

ping @yongtang 👼

Design LGTM

@yongtang needs a rebase 👼

Rebased to fix merge conflict.

I think we're going to be changing the result value of Parse() in another PR to be a struct. That will allow us to return Warnings as part of the response, and avoid having to pass in stderr here.
I'll ping you when that PR opens

I see https://github.com/docker/docker/issues/32580 was opened

This requires a pretty major rebase.

@yongtang let me close this for now, but let me know if you want to work on this again

I'll look at carrying this in a new PR.

Opened #33719 to carry


Bump to 0.53.0, update changelog

r? @Eh2406
(rust-highfive has picked a reviewer for you, use r? to override)

@bors r+

:pushpin: Commit d78b873a442f23dd96f63eccfd5aae4479fcfaee has been approved by Eh2406

:hourglass: Testing commit d78b873a442f23dd96f63eccfd5aae4479fcfaee with merge b842849732f89df8675eb2d933c384d6338e4466...

:sunny: Test successful - checks-actions
Approved by: Eh2406
Pushing b842849732f89df8675eb2d933c384d6338e4466 to master...


Use SemVer 2.0 for NPM packages

This has been around and supported for at least six years in NPM.
Fixes #236

Sorry. This was still on my radar to do, but great to see you've done it. Thanks!

No problem.


fix: custom form implementation

Details:

Fetch custom form when ordering ticket. A custom field will contain information like this:

"attributes":{
            "deleted-at":null,
            "field-identifier":"address",
            "type":"text",
            "is-required":false,
            "is-included":true,
            "form":"attendee",
            "is-fixed":false
         },

Showing fields that is included in the custom forms
Set required fields when form indicate required
Apply data binding to Attendee View

Fixes: #1966
Screenshots
Example of applying all fields with firstname, lastname, email, github required

Resolved conflicts

Database has changed to use 7.json now

Updated


[sui-framework] Remove "delegate" and replace with stake

This does a large rename of "delegation/delegate" (etc) to the equivalent "stake" name, e.g., request_add_delegation becomes request_add_stake.
Move
Entry functions renamed (all within sui_system.move):
request_add_delegation ~> request_add_stake
request_add_delegation_mul_coin ~> request_add_stake_mul_coin
request_add_delegation_with_locked_coin ~> request_add_stake_with_locked_coin
request_add_delegation_with_mul_locked_coin ~> request_add_stake_with_mul_locked_coin
request_withdraw_delegation ~> request_withdraw_stake

Move Object fields
StakingPool::pending_delegation ~> pending_stake
StakedSui::delegation_activation_epoch ~> stake_activation_epoch

Sui types
This renames a couple sui types since they no longer made sense to be called what they were:
DelegatedStake ~> Stake
DelegationStatus ~> StakeStatus

This then performs the core set of renamings to make the system happy with this. So there are still a number of functions in Rust and Typescript that will contain their old delegation name.
Note that on the typescript side we keep DelegatedStake since (1) there was already a type called Stake (which meant something else) and in the context that I could see it being used it appeared as though DelegatedStake really did mean "this is my stake that I have delegated to this validator." If we want to also rename this I'm more than happy to, but will need some help coming up with a good name for it :)
Test Plan
Make sure existing tests still pass.

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[X] user-visible impact
[X] breaking change for a client SDKs
[X] breaking change for FNs (FN binary must upgrade)
[X] breaking change for validators or node operators (must upgrade binaries)
[X] breaking change for on-chain data layout
[X] necessitate either a data wipe or data migration

Release notes

the governance api PR https://github.com/MystenLabs/sui/pull/8848 has some of the same renaming changes on the RPC layer, SDK and the FE apps, this will create quite a bit of conflicts, can we put this on hold 🙏?

The plan is to wait for #9058 to land, and once it does this PR will be rebased on that and then landed.
In the meantime please review and let me know if there is anywhere I have missed or that you think an error was made.


feat(purchase orders): show the preferred location for each PO Line

Adds the ability for the Purchaser to specify where the item is
intentended to go when received.  If the Purchaser does not set a
preferred location, then the default location for the part is displayed.
If the item is received them where it was actually placed is shown.
NOTE: if an item is split when received only one of the resulting
StockItem location is used.
Fixes #1467
Addresses some of the requests in #551

Hey @nwns can you please address the failing tests here

Thanks @nwns this seems to work very cleanly, nice feature!


Nuke: Build template from workfile

Brief description
Following discussion https://github.com/pypeclub/OpenPype/discussions/2013
Similar to PR https://github.com/pypeclub/OpenPype/pull/3544, but allows making template from workfile in one go, and works with creators.
There is already a mechanism for loading Nuke template scripts, even automatically on a first open. Nuke workfile Builder settings allow to create first workfile and run Builder on first launch. This PR allows to create template from workfile with one click. It connects loaders and creators, and applies (some) knob values from the template, for example  publishing and Deadline settings.
Description
For propagating master shots, prepare the master shot, and when happy, convert it to the template Nuke script.
Save the template Nuke script to the location set up in the workfile builder settings, for example {root[work]}/{project[name]}/templates/{task[name]}.nk. When artists launches the task with matching task name, your master script will autoload as version 1, complete with correct loaders and creators.
Simple Template
Showing two Loaders and one Write Render

Shot with Template Applied
from above, at first open

Tested Workfile Builder settings
Settings in json:
WorkfileBuilder.json.txt

Testing notes:

Set up the Workfile Builder settings
Work on first Nuke task as usual. Save
Run OpenPype - Workfile to Template. Note that if you select some nodes, only selection will be templated
Use Save as to save the template to the location specified by settings workfile_builder - curstom_templates - path
Launch Nuke in context of other task. If no version exists, template will auto apply
Alternatively, you can run OpenPype - Builde Workfile and Apply Template from the top menu

I will check PR https://github.com/pypeclub/OpenPype/pull/3578. Thx.
I aimed on ease of use and self building first version. I have similar stuff in production (OP2) for long time, can't be without it...

But in overall I really like the approach, great work @jrsndl ! I cannot wait to introduce this workflow in productions ;)

I fixed the type bug for Nuke 13, and updated the top description with settings in json

@jrsndl What features are you missing in #3544?

@jrsndl What features are you missing in #3544?

Ease of use
I do miss an easy way to create template from working Nuke script. For artists or io guys prepping the shots, create placeholder dialog is maybe too technical.
More Loaders
The template for Nuke should be able to handle not only Loadclip and Nukenode loaders, but also at least the LoadEffectsInputProcess often used for look.
Creators
Having working template for creators is also important, especially for shots with multiple outputs. At least WriteRender and WritePrerender.
Joining template with WorkfileBuilder
WorkfileBuilder already present in OpenPype allows for nice template filtering for task type and task name. So you can have a template for roto, cleanup and comp task names. Another advantage of joining template with WorkfileBuilder is already present feature of auto creating first version of the work file on opening.
The end goal is to have first version of the workfile generated by template and published for editorial check, all automatically.

This template is generated by user or code?

@ClementHector sorry for the late reply.

Is this template node user generated or code generated?

This template is code generated from Nuke script. Basically, OP loaders are "converted" to the template nodes.
I see now that your approach with regex filters is much more flexible. Maybe, to keep gest from both ways, the solution might be to make a Nuke script that "converts" from already existing Nuke loaders to your template nodes. This would keep the great flexibility, and get the easy start for artists as well?
Creators
For Nuke, creators are "asset specific", so "generic" creators do not work for Nuke. Having creators (for example the main output) ready from the template is much appreciated by junior artists. It would be awesome to have it.

Design for all software
The code was planned for a simpler integration in all software, we even made a template builder for photoshop on the same code abstraction as maya and nuke.

This is the reason why your PR is so much better then my attempt. Thank you for the awesome work!

Closing as it is replaced by https://github.com/pypeclub/OpenPype/pull/3838


[FEAT] CBH - Schedule Page

🎨 Fat-marker Sketch
🧑🏫 Places and Affordances

Not sure what a fat-marker sketch and places & affordances drawing is? Please see this page for more info!

This is the Schedule Page where CBH can schedule and publish all the contents. For the contents that are schduled, CBH can change the published date.

🤔 What problems does this solve?

Help CBH schedule and publish a content to organizations.

🤖 Technologies focused on in this feature

React
Tailwind
Shadcn
NextJS
Supabase/PostGreSQL

🍕 Related Pieces

[x] #98
[ ] #99
[ ] #100
[ ] #101

I want to work on this

Can you assign me this @hzz4343


Add getAttribute method to DOMHelper

Adds a method to getAttribute as a compliment to setAttribute. This helper is needed for element actions since they'll need to get/set the data-ember-action attribute.

LGTM

Why is reading the data-ember-action attribute needed for {{action}}

@wycats it is to coalesce multiple action modifiers on one element via the first one that sets the attribute
https://github.com/emberjs/ember.js/blob/783a656e105a7121fc0dc38374931880d14b6bc3/packages/ember-routing-htmlbars/lib/keywords/element-action.js#L47

which is kind of weird because in that case more than one cleanup for the same id would happen in the old code https://github.com/emberjs/ember.js/blob/783a656e105a7121fc0dc38374931880d14b6bc3/packages/ember-routing-htmlbars/lib/keywords/element-action.js#L60

Yeah, for actions in ember-glimmer I will unregister using the actionID as well as the actionState object, so it will only remove the actionState it's supposed to, one at a time.
Regardless, it needs to check for and grab the data-ember-action attribute in order to facilitate the binding as Kris mentioned.


Video list empty

Source information
Multimovies 14.17
Source language
English
Steps to reproduce

Open the last of us
Try watching episode 1
An error occurs 'video list is empty'

Expected behavior
The episode should play
Actual behavior
The episode stops playing and an error occurs saying 'video list is empty'
Aniyomi version
<IP_ADDRESS>
Android version
Android 13
Other details
This also happens to some movies and other shows
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

temporary solution would be to retry the operation as for me retrying helps. You might have to retry few times.


Support for restore operations on DeltaTable

DBR has restore to version and restore to timestamp utility commands.
https://docs.databricks.com/delta/delta-utility.html#restore-a-delta-table-to-an-earlier-state
Would be great to have these in the OSS version of Delta to take advantage of time travel for ops and quickly resolving data issues.
Technically, it's probably something that we could implement outside of Delta, but I would expect it to be a generally useful feature to include.

Hi,
Just a thought. Is this equivalent in OSS Delta Lake to read with the
versionAsOf option and overwrite?
Pozdrawiam,
Jacek Laskowski
https://about.me/JacekLaskowski
"The Internals Of" Online Books https://books.japila.pl/
Follow me on https://twitter.com/jaceklaskowski
https://twitter.com/jaceklaskowski
On Fri, Mar 26, 2021 at 7:54 PM mrichards-scottlogic <
@.***> wrote:

DBR has restore to version and restore to timestamp utility commands.
https://docs.databricks.com/delta/delta-utility.html#restore-a-delta-table-to-an-earlier-state
Would be great to have these in the OSS version of Delta to take advantage
of time travel for ops and quickly resolving data issues.
Technically, it's probably something that we could implement outside of
Delta, but I would expect it to be a generally useful feature to include.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/delta-io/delta/issues/632, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAPG2KB7JOFQ47W6WGA24DTFTJ53ANCNFSM4Z35MRKA
.

Not quite.
If you do a rollback, the current version is equal to versionOf.  So all parties use the correct version (which was rollbacked). We do not have to make sure each process/pipeline receive the correct versionOf. It would be very difficult in multi tenant, multi cluster environment

Makes sense. Stretching out my delta muscles with another possible OSS-based solution.
In 0.8.0 they added DeltaTable.upgradeTableProtocol operator. That could "shut down" all read and write access for maintenance "rollback" window. You could then remove the versions up to the one for rollback using mere rm command.
WDYT?

In ideal world it should work as:
All readers when starting to read should have an indicator in sparkSession which version of this table they are using.
When operator do a command something like "ROLLBACK TO VERSION OF ", what happens
a) all writers get "shutdown". We either gracefully wait or do a hard stop (there probably should be a new type of Transaction exception)
b) we generate a new json manifest file which is copy of the rollback.
c) all readers still are reading the version from which they started (this needs to be handled by the process). We can have additional parameter if it is a batch read the complete if streaming then do a hard stop

@wtfzambo I do not know that. However we can think of not to generate a new json log. Instead of we can create a "technical" transaction on top which would point to the version which we rolled back.

@jles01 damn I had missed the notification of your reply.
How would you create a "technical" transaction without generating a new json log?
We need a way to let delta know to ignore latest version and instead read from whatever version specified in the rollback operation.
Second, parallel question: you mention a multi-tenant environment and I don't understand how a RESTORE operation would be different than reading versionAsOf and then overwriting the full table.

Hi,
As I know for table restore Databricks generates a new transactions (json). This transaction contains all files from "restore" version as an Add operation and all files missed in "restore" version but existent in current as a Delete operation. To generate such transaction we don't need to read\write parquet files and reading _delta_log files is enought.
As for me this has sense, because we should not break transaction history and each transaction should be in delta log.

Also interested in this feature.
One usecase we have is if a bad commit makes its way into a table, we would like a way to roll back to the previous state without having to rewrite the entire table.
Real example: Back filling some partitioned data into a table. Was about 5TB of data across ~50k files. One job accidentally got triggered twice. Having to read in the entire table (about 20TB or so) from a previous version and rewriting it would be very costly and time consuming. Being able to rollback by just modifying the commit log would save a lot of time and effort.
The workaround we found was to manually go and remove transaction log files (since they hadn't been compacted yet) and then do manual cleanup on the files that were "lost" to delta by parsing the transaction log

Hi
i have a PR ready for This just need to add testing. Generały I create additional log file with a Column restoreToVersion. And If reader sees it it will read from that version skipping everything in between. Also when writer Wants to save transaction and the restore appeared it will throw an exception

@jles01 I think that the logic proposed by @Maks-D is more appropriate, given how Delta logs are structured.
It would also work better for people integrating delta table with Athena / Presto for what regards generating manifest symlink files.

@mrichards-scottlogic I've create PR to add Scala API for DeltaTable using logic described here. SQL and python API will be added in a separate PR if Scala API PR be merged

Thanks @Maks-D for impementing this!


Custom Training Help

I'm training on a 1 class dataset that has relatively small objects.  When training I noticed that the loss on Output 3 is huge compared to the others but eventually seemingly converges.  However, when running the trained model output, the bounding boxes are just wrong.  Both the xmin,xmax and ymin,ymax coordinates are the same in some cases, or they define almost the entire image.
Has anyone encountered something like this?

我也遇到了这个问题，我正在训练安全帽预测问题，训练后运行效果很差，边界框很多错误。


[Tabs] - Arrow keys navigation in Safari doesn't work

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
In Safari Version 14.1.2, the Tabs component doesn't navigate user to next/previous Tab when using Arrow Keys. This behaviour works fine in Chrome.

By pressing arrow keys, nothing happens in Safari.
Expected Behavior 🤔
By pressing arrow keys, focus should be shifted to next/previous tab and new item should be set to selected.

Steps to Reproduce 🕹
Open https://material-ui.com/components/tabs/#keyboard-navigation in Safari and press arrow keys. Do the same in Chrome and you'll see that the Tabs get navigated.
take a look at this https://codesandbox.io/s/14hsc in Safari.

Steps:

Open https://material-ui.com/components/tabs/#keyboard-navigation in Safari
press arrow keys (left or right)

Context 🔦

Your Environment 🌎
React 17.0.2
Typescript 4.3.5
Material Ui core 4.12.3

`npx @mui/envinfo`
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Thanks for the feedback.
I cannot reproduce this on Safari 14.1.2 in BrowserStack using https://p25uo.csb.app/.
@abdrah1 Can you still reproduce this problem? If so, please attach a screen recording and include the operating system and version you're using.

Thanks for the feedback.
I cannot reproduce this on Safari 14.1.2 in BrowserStack using https://p25uo.csb.app/.
@abdrah1 Can you still reproduce this problem? If so, please attach a screen recording and include the operating system and version you're using.

Yes, I can still reproduce the problem. Open https://codesandbox.io/s/accessibletabs2-material-demo-forked-kz3ye in Safari and use the keyboard nav.
https://user-images.githubusercontent.com/86606646/135454531-0c52eb13-3b0c-4824-be54-209ce4fa5e48.mp4

I don't understand the screen recording. Between which codesandobxes are you switching? Looks like the first one is working.
Seems like in the second one the tab isn't focused on click which means keyboard navigation cannot work. Might be intended by Safari to not focus on click in this instance. This would not be actionable for us.

I am not switching between codesandboxes, I am switching between Chrome (first one) and Safari (second one). The video shows that keyboard navigation works in Chrome which is shown first and then it shows that the keyboard navigation does not work when I switch to Safari.
The tab is focused in the second one because I click the tab before hitting any keyboard keys.

I don't understand the screen recording. Between which codesandobxes are you switching? Looks like the first one is working.
Seems like in the second one the tab isn't focused on click which means keyboard navigation cannot work. Might be intended by Safari to not focus on click in this instance. This would not be actionable for us.

I am not switching between codesandboxes, I am switching between Chrome (first one) and Safari (second one). The video shows that keyboard navigation works in Chrome which is shown first and then it shows that the keyboard navigation does not work when I switch to Safari.
The tab is focused in the second one because I click the tab before hitting any keyboard keys.

The tab is focused in the second one because I click the tab before hitting any keyboard keys.

That is not necessarily given. Clicking does not always trigger focus depending on platform settings. Could you check via devtools if the clicked tab actually has focus?

I'm closing as it seems a question. Sebastian highlighted the behavior, they Arrow works if the element is focused. Clicking a button doesn't focus on Safari. See https://developer.mozilla.org/en-US/docs/Web/HTML/Element/button#clicking_and_focus


term: don't mix quoted text (notes) with bold "Note!" remarks

That's kind of contradictory. Notes in md quotes > are not very visible so they should be meant for side-notes that could be completely omitted without affecting the document – same function as parentheses in a paragraph, for example.
Bold notes with excalmation point (**Note!**) should not be in md quotes.

Context: https://github.com/iterative/dvc.org/pull/833#pullrequestreview-328534908


URLs with : in them are not "clickable" in terminal

Issue Type: Bug
echo any url that has a semicolon in in the path (not the querystring, that appears to work):
echo https://www.example.com/this:is:a:test?whatever=go

Then try CTRL+click on the echo'ed link.
It'll send you to https://www.example.com/this, excluding the rest of the URL.
At google we have a lot of debugging URLs with semicolons in them, and they don't work from VS Code terminal.
VS Code version: Code 1.27.2 (f46c4c469d6e6d8c46f268d1553c5dc4b475840f, 2018-09-12T07:11:15.873Z)
OS version: Linux x64 4.17.0-3rodete2-amd64

System Info

Item
Value

CPUs
Intel(R) Xeon(R) CPU E5-1650 v3 @ 3.50GHz (12 x 2661)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: disabled_softwarevideo_decode: unavailable_softwarevideo_encode: unavailable_softwarewebgl: enabledwebgl2: enabled

Load (avg)
1, 1, 1

Memory (System)
31.39GB (4.31GB free)

Process Argv
/usr/share/code/code .

Screen Reader
no

VM
0%

Extensions (8)

Extension
Author (truncated)
Version

bazel-code
Dev
0.1.9

mustache
daw
1.1.1

tslint
eg2
1.0.39

g3
eva
0.0.8

python
ms-
2018.9.0

vsliveshare
ms-
0.3.735

vscode-icons
rob
7.27.0

vscode-import-cost
wix
2.9.0

(2 theme extensions excluded)

@Tyriar I was looking into this issue and I believe changing https://github.com/xtermjs/xterm.js/blob/5bc7fc4256edac39eeb340ec21f6ba0c4d877e00/src/addons/webLinks/webLinks.ts#L17 to
(\/[\/\w\.\-%~:]*)* should fix it. Would there be any side effects to me doing so?


Installation on Windows

I'm trying to install on Windows 10 and am running into an error I can't figure out. It appears to be related to hiredis, which as far as I can tell should install on Windows. Has anyone run into this?

C:\> pip3 install aioredis
Collecting aioredis
Using cached aioredis-0.2.9-py3-none-any.whl
Collecting hiredis (from aioredis)
Using cached hiredis-0.2.0.tar.gz
Building wheels for collected packages: hiredis
Running setup.py bdist_wheel for hiredis ... error
Complete output from command c:\code\torus\venv\scripts\python.exe -u -c "import setuptools, tokenize;file='C:\Users\Dave\AppData\Local\Temp\pip-build-a8cyfjue\hiredis\setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" bdist_wheel -d C:\Users\Dave\AppData\Local\Temp\tmpwtzhpc1xpip-wheel- --python-tag cp34:
running bdist_wheel
running build
running build_py
creating build
creating build\lib.win-amd64-3.4
creating build\lib.win-amd64-3.4\hiredis
copying hiredis\version.py -> build\lib.win-amd64-3.4\hiredis
copying hiredis_init_.py -> build\lib.win-amd64-3.4\hiredis
running build_clib
building 'hiredis_for_hiredis_py' library
error: INCLUDE environment variable is empty

Failed building wheel for hiredis
Running setup.py clean for hiredis
Failed to build hiredis
Installing collected packages: hiredis, aioredis
Running setup.py install for hiredis ... error
Complete output from command c:\code\torus\venv\scripts\python.exe -u -c "import setuptools, tokenize;file='C:\Users\Dave\AppData\Local\Temp\pip-build-a8cyfjue\hiredis\setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record C:\Users\Dave\AppData\Local\Temp\pip-0keqj3wf-record\install-record.txt --single-version-externally-managed --compile --install-headers c:\code\torus\venv\include\site\python3.4\hiredis:
running install
running build
running build_py
creating build
creating build\lib.win-amd64-3.4
creating build\lib.win-amd64-3.4\hiredis
copying hiredis\version.py -> build\lib.win-amd64-3.4\hiredis
copying hiredis_init_.py -> build\lib.win-amd64-3.4\hiredis
running build_clib
building 'hiredis_for_hiredis_py' library
error: INCLUDE environment variable is empty
----------------------------------------

Command "c:\code\torus\venv\scripts\python.exe -u -c "import setuptools, tokenize;file='C:\Users\Dave\AppData\Local\Temp\pip-build-a8cyfjue\hiredis\setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record C:\Users\Dave\AppData\Local\Temp\pip-0keqj3wf-record\install-record.txt --single-version-externally-managed --compile --install-headers c:\code\torus\venv\include\site\python3.4\hiredis" failed with error code 1 in C:\Users\Dave\AppData\Local\Temp\pip-build-a8cyfjue\hiredis\

Hi, @dland512
I think you need msvc compiler to build hiredis

Okay, I'll look into that. Thank you.

how did you repair this poblem ? i have this problem too. but i don't now how repair this.


Update ni-tcpioctl-ioctl_tcp_query_information_ex.md

@justanotheranonymoususer : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks for fixing these issues!


Status bar foreground not changing

VSCode Version: 1.37.1
OS Version: Windows 10

Every time I install VSCode from scratch I pull my settings from Github Gist, which works fine. One setting, however, never gets applied and that is
"statusBar.background" : "#ffffff"
It happens every time and usually I end up restarting the vscode, trying to remove the setting and add it again.

Maybe you have 0 folders opened. In that case "statusBar.noFolderBackground" is used.

Oh wow.. that's new :D Thank you!


Walking receiver and operation state chains

Fact 1: All operation states store the receiver of the enclosing (parent) async scope.
Fact 2: The vast majority of receivers store little more than a pointer back to the operation state for the current async scope.
Implication: If we follow these chains -- operation state -> parent receiver -> parent's operation state -> grandparent's receiver, etc -- we are essentially performing an async stack walk.
The ability to traverse the async stack can be useful in a number of scenarios. Libunifex has a dedicated "visit_continuations" mechanism for doing just this.
Q: Should we require all operation states to expose their parent's receiver, and require that all receivers can return pointers to their operation states?

Note: this would be a way to explore from inner-to-outer async scopes. When we have previously discussed "operation state queries", the hope for those was to explore from outer-to-inner scopes, which is problematic because there isn't a simple 1-to-1 parent-to-child relationship with async scopes as there is with child-to-parent relationships.


Button is Fixed

Button is Fixed now , Please take a look .

Raise up the new PR for this as conflicts are now in package-lock.json


Label leakage

https://github.com/GraphCL/PyGCL/blob/33c7d1c53c6e0f5458122ecad59cd69f9e1d4af6/GCL/model/sampler.py#L120

https://github.com/GraphCL/PyGCL/blob/33c7d1c53c6e0f5458122ecad59cd69f9e1d4af6/GCL/model/sampler.py#L124
https://github.com/GraphCL/PyGCL/blob/33c7d1c53c6e0f5458122ecad59cd69f9e1d4af6/GCL/model/sampler.py#L132
It seems that extra positive/negative will only be added for nodes in the training set.


Update the YNH version to update the package?

From what I understand this pulls from git but if you do not change the versioning of the YNH package it means no update push right? The package is almost a year old in that regards. Should I try to submit a pull request for the YNH versioning change? Let me know if that's all it needs.

From what I see, this app doesnt rely on a git clone / git pull, it pins a specific commit : https://github.com/YunoHost-Apps/epicyon_ynh/blob/master/conf/app.src
So to update this app, you should edit this file + corresponding version number in the manifest
Feel free to make a PR and then we can trigger out automatic test suite ;)

Thanks. So it is pulling from a certain commit that archive? Because I do not see any releases/archives on their github repo. Should we just use the latest commit from the main? I can try to keep this one up to date from time to time if so.

I did I hope that's ok. I am not familiar with github that much, how do I change multiple files and push under 1 single pull request?


Backing up to outlook.com or yahoo.com

I use my own VPS server for backup (except calendar), and have specified just my username, rather than full email address, in addition to port 993 on the server, and TLS security. This works well, and has avoided the gmail problem for me.
I had forgotten about this problem, and I tried to set up SMS Backup+ for a friend, using her Gmail account. Of course, it failed. I then tried to use Custom server, specifying first yahoo.com:993 and then outlook.com:993 with her respective yahoo and outlook accounts. Neither worked.
Has anyone else tried these servers? Have they been known to work? If so, with what particulars?
Thanks.

Outllook supports IMAP access, as described in https://support.office.com/en-us/article/POP-IMAP-and-SMTP-settings-for-Outlook-com-d088b986-291d-42b8-9564-9c414e2aa040
Yahoo has only partial support for IMAP (their servers use a non-standard version) so it may or may not work, see http://www.yahooimapsettings.com/

@p-b-west Did that help?

I haven't seen her recently, so i haven't had the chance to try it.  I will when I see her.


shell-gpt tests/*.py get installed to site-packages/tests - intentional or bug?

The current setup.py uses find_packages().  In the current git repo, this finds tests as a package, and so includes tests as part of the bdist.
This winds up installing tests/*.py into /lib/pythonX.Y/site-packages/tests.  Is this intentional that it installs tests specific to shell-gpt into a generically named 'tests' directory in site-packages?
Perhaps the bdist should not include the tests/*.py files?

See also discussion in a downstream packager - https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=270542
Note that installing to site-packages/tests/ creates a real life conflict with another package - https://github.com/greenbone/notus-scanner
It is possible the notus-scanner package is also mistaken in installing its test files to site-packages/tests as well.

@jhgit Should be fixed in the lates release 0.8.3. Could you please confirm it?

I think using MANIFEST.in would do the job to get tests in the sdist (and not the bdist):
$ cat MANIFEST.in
graft tests

@jhgit Thank you, shoud be fixed.


gnuplot lexer: put comment tokens into a single token

Changed comment token to use regex in gnuplot lexer.
This will fix #2825.

This might fix the latex math issue, but it will stop recognizing comments with line continuations (backslash before the newline).
PS. What do you mean by "use regex"?

You are right. I've fixed the line continuation problem.
PS. Please ignore my comment above because I was lacking in understanding of the codebase...


UnicodeEncodeError: 'utf-8' codec can't encode characters

Problem Description
A clear and concise description of what the bug is.
Addon error: Traceback (most recent call last):
File "/usr/local/lib/python3.6/site-packages/mitmproxy/addons/dumper.py", line 250, in error
self.echo_flow(f)
File "/usr/local/lib/python3.6/site-packages/mitmproxy/addons/dumper.py", line 218, in echo_flow
self._echo_request_line(f)
File "/usr/local/lib/python3.6/site-packages/mitmproxy/addons/dumper.py", line 166, in _echo_request_line
self.echo(line)
File "/usr/local/lib/python3.6/site-packages/mitmproxy/addons/dumper.py", line 70, in echo
click.secho(text, file=self.outfp, **style)
File "/usr/local/lib/python3.6/site-packages/click/termui.py", line 548, in secho
return echo(message, file=file, nl=nl, err=err, color=color)
File "/usr/local/lib/python3.6/site-packages/click/utils.py", line 272, in echo
file.write(message)
UnicodeEncodeError: 'utf-8' codec can't encode characters in position 111-112: surrogates not allowed
Steps to reproduce the behavior:

mitmproxy crashed and exit app

System Information
Paste the output of "mitmproxy --version" here.
Mitmproxy: 5.1.1
Python:    3.6.8
OpenSSL:   OpenSSL 1.1.1g  21 Apr 2020
Platform:  Linux-3.10.0-1127.13.1.el7.x86_64-x86_64-with-centos-7.8.2003-Core
App started:
mitmdump --ssl-insecure --mode transparent --showhost

Does this happen with the latest version as well (5.2)?
How can we reproduce this? Does  mitmdump --ssl-insecure --mode transparent --showhost crash immediately for you or are there more steps involved? Do we have to visit a specific website?

Upgrade version mitm:
Mitmproxy: 5.2
Python:    3.6.9
OpenSSL:   OpenSSL 1.1.1g  21 Apr 2020
Platform:  Linux-3.10.0-1127.13.1.el7.x86_64-x86_64-with-centos-7.8.2003-Core
Error:
Addon error: Traceback (most recent call last):
File "/opt/rh/rh-python36/root/usr/lib/python3.6/site-packages/mitmproxy/addons/dumper.py", line 250, in error
self.echo_flow(f)
File "/opt/rh/rh-python36/root/usr/lib/python3.6/site-packages/mitmproxy/addons/dumper.py", line 218, in echo_flow
self._echo_request_line(f)
File "/opt/rh/rh-python36/root/usr/lib/python3.6/site-packages/mitmproxy/addons/dumper.py", line 166, in _echo_request_line
self.echo(line)
File "/opt/rh/rh-python36/root/usr/lib/python3.6/site-packages/mitmproxy/addons/dumper.py", line 70, in echo
click.secho(text, file=self.outfp, **style)
File "/opt/rh/rh-python36/root/usr/lib/python3.6/site-packages/click/termui.py", line 548, in secho
return echo(message, file=file, nl=nl, err=err, color=color)
File "/opt/rh/rh-python36/root/usr/lib/python3.6/site-packages/click/utils.py", line 272, in echo
file.write(message)
UnicodeEncodeError: 'utf-8' codec can't encode characters in position 111-112: surrogates not allowed
I'm send custom request contains cyrilic symbol
Example:
"https://test.com/test/download.php?s=ппп"

I'm send custom request contains cyrilic symbol

What HTTP client are you using? At least in regular mode I cannot reproduce it. Even with ncat and literally using the cyrilic symbols (explicitly without URL encoding). I haven't set up transparent mode, so maybe it only happens there (I don't think so).
But it looks like this is a duplicate of #4070
echo -e 'GET http://www.example.com/test/download.php?s=ппп HTTP/1.1\r\nHost: www.example.com\r\n\r\n' | ncat localhost 8080

I'm using windows JScript code
var ie2 = new ActiveXObject("InternetExplorer.Application");
ie2.Navigate(""https://test.com/test/download.php?s=ппп"");

I'm closing this in favor of #4070, I think this was a duplicate


No default profile available when k8s 1.15

On 2.5-head - commit id: e2295c6  using cis v2 app v1.0.3
cisoperator:
    repository: rancher/cis-operator
    tag: v1.0.3-rc4
  securityScan:
    repository: rancher/security-scan
    tag: v0.2.2-rc4

Deploy a 1.15.12-rancher2-7 cluster (through cluster.yml) as a standard user.
deploy CIS v2
create scan
No default profile is selected in the dropdown.

Two ways this dropdown ends up empty:

the current user doesn't have permission to view the default profile configmap(e.g. a user with the cis-admin role)
The profile specified in the above configmap isn't valid for this cluster (wrong kubernetes version)

This PR rancher/dashboard#2109 addresses #2 by making the dropdown fallback to cis-1.5 or cis-1.6 profiles. #1 can't be addressed on the UI side.

Logged issue - For #1 - https://github.com/rancher/cis-operator/issues/83

On 2.5-head - commit id: bb69b03
cisoperator:
    repository: rancher/cis-operator
    tag: v1.0.3-rc5
  securityScan:
    repository: rancher/security-scan
    tag: v0.2.2-rc5

Deploy CIS v2 on a cluster.
Add a user user1 with cis-admin role and as cluster-member of the cluster
Login as user1
Create a scan CR
User with cis-admin role has no scan profile listed as default in dropdown while creating a scan

This doesn't work. Reopening the issue.

@mantis-toboggan-md The user with cis-admin role is able to get the default-clusterscanprofiles configmap in cis-operator-system namespace:

On 2.5-head - commit id: 48254f944  and ui-dashbaord-index pointing to: releases.rancher.com/dashboard/release-2.5.4
cisoperator:
    repository: rancher/cis-operator
    tag: v1.0.3-rc5
  securityScan:
    repository: rancher/security-scan
    tag: v0.2.2-rc5

Deploy CIS v2 on a cluster.
Add a user user1 with cis-admin role and as cluster-member of the cluster
Login as user1
Create a scan CR
User with cis-admin role does have a default value in the dropdown


maxDataSize:0 crashes the app when using koa and ctx is captured

I am not sure if this is due to memory maxing out or something else. I also don't have a proposal/solution.
I think something weird is happening with ctx circular references and the way data is serialized.
Setting maxDataSize to 1MB (1048576) works perfectly but blocks for about 3s

Thanks for opening this issue.  I agree this could potentially be caused by circular references in ctx, but I'll have to look into it in more detail.  I'll keep this thread updated with what I find.

https://github.com/GoogleCloudPlatform/cloud-debug-nodejs/blob/a136e143530aa14bb36d0aaa93c830159b8bceae/src/agent/config.ts#L211-L219
maxDataSize: 0 specifies no limit in the amount of data returned, so @kilianc's theory that getting an expression with a circular reference causes an issue seems to be correct. We can avoid this by either setting a hard upper limit even if the user asks for none, or detecting circular references. @DominicKramer thoughts?

@DominicKramer @ofrobots Assigning this to @DominicKramer since P1s should be assigned and active :)

Pending a new release, this should be fixed for Node <10. The same bug for Node 10 is being tracked in #516.

I am closing this issue since it has been resolved for Node < 10 with the release of version 3.0.0.  Further conversation about the solution for Node 10 should occur on issue #516.  Any issues anyone finds with Node < 10 should be reported in this thread.

FYI, the feature needed to fix this bug was removed in Node 12, so this bug now appears once again in all versions of Node 12+. To work around it, use a non-zero maxDataSize value.
This bug will likely not be fixable unless Node adds this feature back.


Document the data type compatibility matrix

Dyndao/Go's data type <-> matching data type and limitations for each individual database.

goal is to document and fix this all in the elysium branch.


No license

Hi. Could you possibly add a license to this example?

Done, sorry, I had missed this issue for a year :-(


Nested custom vjps do not correctly propagate cotangents

Description
The use case is nesting two custom_vjp operators to perform reverse mode AD over reverse mode AD (to calculate an HVP over a function that uses custom_vjp to call into external, untraceable code). I have not used custom_vjp before and the documentation didn't seem to cover anything related to this problem, so please forgive me if I'm missing something obvious here!
When nesting two custom_vjp operations, the cotangents computed by the second custom_vjp do not seem to be propagated properly through the compute graph. A minimal example is below, using the function y=x**3; the computed gradient is 0, which implies that the ddy computed in the second custom_vjp (in op_bck_bck) is never further propagated.
import jax

@jax.custom_vjp
def op(x):
    return x**3

def op_fwd(x):
    # return the output and the saved values
    return op(x), (x,)

@jax.custom_vjp
def op_bck(saved, dy):
    x, = saved
    return 3 * x**2 * dy, # gradient of y=x^3

def op_bck_fwd(saved, dy):
    return op_bck(saved, dy), saved

def op_bck_bck(saved, ddx):
    x, = saved
    ddx, = ddx
    ddy = 3 * x**2 * ddx
    # return:
    # - dSaved (None as we don't want to diff through saved values)
    # - ddy (use chain rule)
    return (None,), ddy

op_bck.defvjp(op_bck_fwd, op_bck_bck)
op.defvjp(op_fwd, op_bck)

# test it
def jop(x):
    return x**3

xx = jax.random.normal(jax.random.PRNGKey(0), (5,))
ww = jax.random.normal(jax.random.PRNGKey(1), (5,))
ww2 = jax.random.normal(jax.random.PRNGKey(2), (5,))

def fn(op, x):
    def l(x):
        # first vjp
        return jax.grad(lambda z: op(z) @ ww)(x) @ ww2

    # should call second vjp
    return jax.grad(l)(x)

gt_jac = fn(jop, xx)
my_jac = fn(op, xx)

print(gt_jac) # [-0.16732852 11.739656   -0.06793058  0.4734605  -0.15350063]
print(my_jac) # [0. 0. 0. 0. 0.]

System info (python version, jaxlib version, accelerator, etc.)
jax:    0.4.26
jaxlib: 0.4.26
numpy:  1.26.4
python: 3.10.14 (main, Mar 21 2024, 16:24:04) [GCC 11.2.0]

Here is an edit to your nested custom_vjp snippet that causes the second-order derivative values to match the reference values:
import jax

@jax.custom_vjp
def op(x):
    return x ** 3

def op_fwd(x):
    return op(x), (x,)

@jax.custom_vjp
def op_bck(xs, dy):
    x, = xs
    return 3 * x ** 2 * dy,

def op_bck_fwd(xs, dy):
    return op_bck(xs, dy), (xs, dy)

def op_bck_bck(xs_dy, dzs):
    xs, dy = xs_dy
    x, = xs
    dz, = dzs
    dx = 6 * x * dy * dz
    ddy = 3 * x ** 2 * dz
    return (dx,), ddy

op_bck.defvjp(op_bck_fwd, op_bck_bck)
op.defvjp(op_fwd, op_bck)

A couple of observations behind this edit:

Returning None from the second backward function you had doesn't seem quite right. Even though this was invoked with a saved residual, we still want to differentiate. You can see the VJP with respect to this argument as the expression dx = 6 * ... in my op_bck_bck.
In op_bck's custom derivative, it's useful to save the input value dy as a residual, since we need to use it in computing the VJP expression from the previous bullet (dx = 6 * ...).

What do you think? Take a look and see if this makes sense to you. If something remains confusing then we can think on how we'd improve docs somehow, or consider adding an example somewhere (maybe in a future version of the AD cookbook).

This makes much more sense now, thank you for the help!


Update SQL-DS-Cache dependency to arrow 4.0.0

For further release, we will not modify Plasma-ds-cache arrow-plasma dependency in pom.xml file. Unless we public a new binary on maven central repo.


How can I register my own dataset?

Hello, thanks for your excellent work! Here I have a code problem when I fine-tune the model on my own dataset. I just follow the organization manner  in CoOp/datasets to write my dataset code, but failed to register the dataset. I also add xxx.yaml in CoOp/configs/datasets, and still failed to register, could you give me some advice if I need to add extra code? Thanks!

https://github.com/KaiyangZhou/Dassl.pytorch#add-a-dataset

https://github.com/KaiyangZhou/Dassl.pytorch#add-a-dataset

I have resolved it, thanks a lot!

Just so that future people know, you also need to update train.py to add your own dataset.
Specifically, I added the following line to the top of train.py.
import datasets.YOUR_DATASET

Thank you @zanedurante . This error has been bothering me for two days.

Just so that future people know, you also need to update train.py to add your own dataset.
Specifically, I added the following line to the top of train.py:
import datasets.YOUR_DATASET

Thank you @zanedurante . This error has been bothering me for one days.

@buproof This file is used to instruct the dataset constructor on how to split it on train and test sub groups. Check out datasets/oxford_pets.py for more info.