Error Encrypting sensitive data (encrypt , decrypt)

Description
Using the encrypt and decrypt methods generates an error.
Code example to reproduce the issue:
import flet as ft
from flet.security import encrypt

secret_key = "lW1iwuRlHeJjvBhavZfv8vnvdVFajh8e"

def index_page(page: ft.Page):
    text = ft.TextField()

    def encrypt_text(e):
        text.value = encrypt(text.value, secret_key)
        text.update()

    view_one = ft.View(
        controls=[
            ft.Text("Home page", size=30),
            text,
            ft.FilledButton("encript", on_click=encrypt_text),
        ],
        vertical_alignment=ft.MainAxisAlignment.CENTER,
        horizontal_alignment=ft.CrossAxisAlignment.CENTER,
    )

    page.views.append(view_one)
    page.go(page.route)

ft.app(index_page)

Describe the results you received:
Traceback (most recent call last):
File "C:\Users\dxs\Desktop\xd.venv\Lib\site-packages\flet_runtime\security_init_.py", line 6, in
from cryptography.fernet import Fernet
ModuleNotFoundError: No module named 'cryptography'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "c:\dxs\Desktop\xd\app\views\index.py", line 2, in
from flet.security import encrypt
File "C:\Users\dxs\Desktop\xd.venv\Lib\site-packages\flet\security_init_.py", line 1, in
from flet_runtime.security import *
File "C:\Users\dxs\Desktop\xd.venv\Lib\site-packages\flet_runtime\security_init_.py", line 11, in
raise Exception('Install "cryptography" Python package to use Flet security utils.')
Exception: Install "cryptography" Python package to use Flet security utils.
Describe the results you expected:
Additional information you deem important (e.g. issue happens only occasionally):
Flet version (pip show flet):
ame: flet
Version: 0.21.1
Summary: Flet for Python - easily build interactive multi-platform apps in Python
Home-page:
Author: Appveyor Systems Inc.
Author-email<EMAIL_ADDRESS>License: Apache-2.0
Location: C:\Users\dxs\AppData\Local\Programs\Python\Python311\Lib\site-packages
Requires: cookiecutter, fastapi, flet-runtime, packaging, qrcode, uvicorn, watchdog
Required-by:

Give your requirements.txt file (don't pip freeze, instead give direct packages):
flet

Operating system:
Windows

As exception suggests "Install "cryptography" Python package to use Flet security utils."


🤬 Censor blog

Removed every swear word at the request of Nina and Michael! "Definitely my personality ... but sounds better said than on paper"

LMFAO


Kyma Dashboard - Pod Logs not available

Description
I am not able to see logs for any Pods in the Kyma UI on Google Chrome. This affects all clusters(prod/non-prod)/namespaces(test, dev, pilot, prod).
Used versions: Kubernetes: v1.23.12; Kyma: 2.7.3
My assigned role is cluster-admin
A collegue with the same cluster-admin role is able to see the logs correctly, others have troubles as well.
Error message of the pod's logs:

Additionally I inspected the page with the browser's dev tools and discovered that the fetch request for the logs cancel after a while of being caught in pending state:

printing the logs to the console works as intended.
Expected result
Correct Display of incoming logs for a specific time period
Actual result
see error image above

We cannot reproduce this scenario. Logs seems to work as expected


[firefox] Add unmaintained releaseCycles > 102

From Discussion in #2828

Preview: https://deploy-preview-2839--endoflife-date.netlify.app/firefox
@marcwrobel what do you think?

Lgtm


Better support for republish update_type

This band aids a couple of the holes that we have in the system for using an update_type of republish - as evidenced from running the specialist publisher republish task.
I think we should aim to remove this update_type as it adds junk to the history and all feels complicated and prone to bugs, but until then it's probably best to keep it going.

@steventux Just addressed that comment.


V0.1.37 rebase master

At first I was going to say this doesn't look right, but it's ok. https://github.com/openshift/service-catalog/releases/tag/v3.11.0-v0.1.35 looks to have come from https://github.com/openshift/service-catalog/pull/23, but that's for 0.1.31. I think the wrong tag should probably be deleted after this goes in.

At first I was going to say this doesn't look right, but it's ok. https://github.com/openshift/service-catalog/releases/tag/v3.11.0-v0.1.35 looks to have come from #23, but that's for 0.1.31. I think the wrong tag should probably be deleted after this goes in.

Thanks for catching that bad tag Jeff.  v3.11-0.1.35 should have been tagging https://github.com/openshift/service-catalog/pull/29.  I'll catch up with you on slack tomorrow to verify corrective action.


Stream component - doesn't work with H265 / HEVC streams?

I have few hikvision cameras, which, before stream component was added, I was using via ffmpeg camera and this ffmpeg options:
input: -loglevel quiet -rtsp_transport tcp -i rtsp://USER:PASS@IP/Streaming/Channels/101
and it worked for both H264 and H265 / H265+ streams. With stream component, setting camera to record in H265 results in just "spinning the wheel" when I click to initiate streaming, but starts working if I switch the camera to h264.
HomeAssistant 0.90.1 - official docker image
p.s. I manually applied Prefer TCP for RTSP streams patch from dev inside container, it didn't help.

Please make sure to fill out the issue template entirely, never delete it wehn you create an issue.  Also H265 is not supported, only H264 is supported.

Sorry about the template, Daniel.
Regarding H265 support - same time the ffmpeg binary bundled with the HA docker image decodes H265 just fine. Does not stream component uses ffmpeg too?

For the first release H265 is not supported, so just since it works in FFMPEG or elsewhere does not necessarily mean it should work for stream right now.  This is more of a feature request than a bug I think.

This is not a bug, please go to our forum or discord channel for support.


⚠ Minecraft has degraded performance

In 58d40a4, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1123 ms

Resolved: Minecraft performance has improved in 98ae38e.


Adds an option to massage kwargs before they go to the Step's run()…

@epwalsh, I added another fix. Can you give it another look?

@dirkgr am I missing a commit? I this changelog update https://github.com/allenai/tango/pull/211/commits/374f28e9710769fdd9704679665e9746fe59cfc7, but I don't see changes that actually address that.

🤦🏻


Fix C++ compatibility

stdio.h uses new as a parameter name for rename(), which conflicts with the C++ keyword, so I renamed the parameter.
stddef.h tries to redefine wchar_t, which seems to be a builtin type in C++, which makes redefining it forbidden.
/edit:
The issues in this PR have been reported (and partially fixed) upstream. As soon as the issues are fixed upstream, we can rebase and drop this PR.

For the record: This was merged accidentally; I've rebased onto DevSolars upstream and force-pushed.
The respective changes are in https://github.com/XboxDev/nxdk-pdclib/commit/66369399af0819f75454a8fa0f81e21bf4e8024e and https://github.com/XboxDev/nxdk-pdclib/commit/b328a759a06c39e0bdc532c85197e14bb297e3b9 (both from upstream).


Use ansible_local provisioner

Vagrant's latest 1.8 release adds an ansible_local provisioner which should be able to replace the functionality of the JJG-Ansible-Windows script.

Most likely. That will save some maintenance effort for sure! I'm still trying to get some VMware licensing issues sorted out so I can try out Vagrant 1.8.0 without it blowing up my environments.

Working on this now.

Works great! Only issue—which is only sort of an issue, because it has plusses and minuses—is that every time you run vagrant provision, all the ansible roles are updated via ansible-galaxy install -r [roles file] --force.
Meaning every provision (if you don't have Ansible installed locally) will take 1-2 extra minutes. But I guess it's even more motivation to install Ansible on your host if you're on Mac or Linux, or switch to Mac or Linux from Windows if not :P


Fix require circle between extractText.js and TSpan.js issue

close #773

When will this be merged?

I think this breaks handling of text no? Should use inversion of control instead to inject the tspan reference. The cycle shouldn't even be an issue as it is. Or?

Closing in favor of https://github.com/react-native-community/react-native-svg/pull/813


Fix module name capitalisation

Change module name capitalization so that only the first letter is
affected. Without this, ‘module QtIviVehicleFunctions’ gets capitalised
as Qtivivehiclefunctions which looks awful.
This can’t easily be worked around in the generator as all the symbol
name lookup for type names is based on this.

Fixed in d4a76a6333d13231fecb938474e583130929119a


Fixing Grammatical Errors

OS
Linux
VERSION
v1.6.1
ISSUE DESCRIPTION
Fixing the grammatical errors.

you can open a pr

Please verify

you can open a pr

Any updates on this?


Starting point can be chosen multiple times by _get_random_starting_points

In the func _get_random_starting_points in the map generator, the resulting array can have multiples of the same starting point.
e.g. from my print: [4, 0, 0, 4, 2, 5]
Because of this 4 starting points can branch into 5 nodes as seen in the map generator video at 1:47:18
probably enough to pull y_coordinates.append(starting_point) into the "if not y_coordinates"

Hm actually just noticed that this is the same in SlayTheSpire, so maybe not a bug, but a feature? :)

Hey, yeah it works the same way.
The number of starting points does not equal to the number of branching paths. I'll close this for now if you don't mind. :)


[4/?] - input+lnwallet: prepare input package for funding logic, add new MusigSession abstraction

Change Description
Depends on https://github.com/lightningnetwork/lnd/pull/7333 and https://github.com/lightningnetwork/lnd/pull/7331.
Only the last 3 commits are new. We'll also want to synchronize with #6974 as both the PRs update the deps to use the latest version of musig2.
In this PR, we first do a small refactoring to extract the musig2 session logic into a new package + struct. This allows use to re-use it later in tests w/o creating the entire wallet system.
Building on this, we then do some small prep to update the chanfunding package to be ready for the new musig2 channels.
The final change in this PR adds a new struct to encapsulate the new musig2 session logic for channels. A session can be created as soon as the verification nonce for the local party is known. At any time, we'll have two sessions: one for the remote commitment and one for the local commitment. A session can be finalized once the signing nonce (sent by the remote party) is known. When generating a signature, we'll use the remote party's verification nonce (the remote session), then generate a fresh nonce to complete the session. Similarly, when accepting a new state, we'll use the remote party's signing nonce to finalize the session, then verify the incoming signature.
For our local session, we'll use a counter-based system to generate the nonces we send to the remote party. The underlying counter here is the commitment height, which is then used by the existing shachain producer to generate fresh, but deterministic randomness. This lets us not have to store the secret nonce to disk, instead we'll just re-generate the nonce and use the sig we have on disk to combine, then create the final witness to broadcast.
The next PR in this series uses this PR, and its dependents to implement the funding logic within the wallet itself (reservations).
Steps to Test
Pull Request Checklist
Testing

[ ] Your PR passes all CI checks.
[ ] Tests covering the positive and negative (error paths) are included.
[ ] Bug fixes contain tests triggering the bug to prevent regressions.

Code Style and Documentation

[ ] The change obeys the Code Documentation and Commenting guidelines, and lines wrap at 80.
[ ] Commits follow the Ideal Git Commit Structure.
[ ] Any new logging statements use an appropriate subsystem and logging level.
[ ]  There is a change description in the release notes, or [skip ci] in the commit message for small changes.

📝 Please see our  Contribution Guidelines for further guidance.

@roasbeef, remember to re-request review from reviewers when ready

@crypt-iq: review reminder
@roasbeef, remember to re-request review from reviewers when ready

@crypt-iq: review reminder
@roasbeef, remember to re-request review from reviewers when ready

@roasbeef, remember to re-request review from reviewers when ready

@crypt-iq: review reminder
@roasbeef, remember to re-request review from reviewers when ready

@crypt-iq: review reminder
@roasbeef, remember to re-request review from reviewers when ready

@crypt-iq: review reminder
@roasbeef, remember to re-request review from reviewers when ready

@crypt-iq: review reminder
@roasbeef, remember to re-request review from reviewers when ready

@roasbeef, remember to re-request review from reviewers when ready

@roasbeef, remember to re-request review from reviewers when ready

@roasbeef, remember to re-request review from reviewers when ready

Initial set of review comment addressed. Will request review again after I put up the unit tests.

Unit tests added, PTAL!

(some linter things remain that I assume will be addressed in a final clean up commit once all the others are also in

Really been fighting the linter on this one...should be ✅ now with the latest push.


Allow association of work item and iteration (#714 and #716)

Allow association of work item and iteration (#714 and #716)

@nimishamukherjee We need to handle long iteration name in the dropdown and also in detail page.

@vikram-raj created issue #909 for handling long iteration names.


Allow multiple 'local' installs of cuda to be specified

Exposes the local_cuda repository_rule to allow additional installs of cuda to be installed as toolchains.
e.g.
local_cuda(name="cuda_11_4", toolkit_path = "/usr/local/cuda-11.4")

Is is still draft. I think this should good to go?

It is weird that the CI failure cannot be reproduced on local machine with cuda 11.*

@jsharpe Finally got sometime to figure this one out. The reason a mixture of CI and you impl. On CI we only installed cuda-nvcc-10-1 cuda-cudart-dev-10-1, then /usr/local/cuda is not configured so only /usr/local/cuda-10.1 exists. And in you change cuda.local_toolchain(name = "local_cuda", toolkit_path = "/usr/local/cuda") cause bazel creating a dangling reference. Can you change it default to toolkit_path = ""? So that the environment CUDA_PATH will be picked up.


Release trivor-nlp 0.0.1-alpha.0

0.0.1 of trivor-sdk
URL pattern bug fixed
Pace insight updated to RateOfSpeech #121
Lemmatizer #123
Java docs #99
Chunker
Sentiment Analysis
Sentence-based insights apart from frequency insights
Keywords ingisht

0.0.1-alpha.0 released on https://bintray.com/kalnee/maven/trivor-nlp


Everything plugin fail to load and would be disabled

Microsoft PowerToys version
0.53.3
Running as admin

[X] Yes

Area(s) with issue?
PowerToys Run
Steps to reproduce
Just update PowerToys and the plugin stopped working.
✔ Expected Behavior
The plugin to work
❌ Actual Behavior
A message box is displayed: plugins everything fail to load and would be disabled.
Rest of powertoys works as expected.
Other Software
Everything version <IP_ADDRESS>5 (x64)

I have just notices that I installed this plugin as an external tool :( However it would be nice to include it somehow in the main stream as it is really a great tool to search files.
If the plugin developer reads this, please try to update.... Thanks a lot!!!

Hi @jlanza,
Recently, we did some refactoring across modules for PowerToys, including PowerToys run. This is what caused that plugin not functionally.
As you said, the plugin is not the official plugin. You'll have to contact the plugin directly to get this fixed.
You may find the author of the plugin in #3278. (There are serval versions of the Everything plugin out there)
Thanks!

I would also like to mention  Pull Request 10937 where I found the solution on another github, that seems to be tracking updates.

We don't officially support non-PowerToy plugins as we knew we'd be breaking stuff from time to time in the short term.
/dup #3200 you'll have to reach out to them.


Building android with gradle and remote keystore

Question Subject
Fastlane gradle
Question Description
I am following your steps to bundle a android project and sign it with keystore like;
gradle(
  task: "bundle",
  build_type: "Release",
  properties: {
    "android.injected.signing.store.file" => "keystore.jks",
    "android.injected.signing.store.password" => "store_password",
    "android.injected.signing.key.alias" => "key_alias",
    "android.injected.signing.key.password" => "key_password",
  }
)

I am using a cd server (bitrise) to store the keystore.jks.
My question is, is there a way to use the remote url for "android.injected.signing.store.file" => "some http url",
Would I maybe have to copy it somewhere local, then use a path?
Thanks

If you can provide a remote URL I don't know, as this part is handled by gradle.
But yes, downloading the file and using that path sounds like a reasonable solution. Many CI providers also provide a mechanism to handle secret files (upload in a web UI, then have a env var with its URL to use during the build for example).

Ahhh, this is more of a gradle question than a fastlane question, sorry.  I can ask in a gradle forum.
I did manage to work around it by using gradle('bundle') -> pass the .aab path to Bitrise:sign_apk:step -> pass signed .aad path to supply
Tiny trick, to get fastlane .aab path(s), add these lines after gradle(bundle)
gradle(task: 'bundle')....

Please feel free to close.

Where does Bitrise:sign_apk:step get the signing credentials from? Can you upload them for it on Bitrise?

Yesir, they have code signing storage and vars

Ok, then that sounds like probably the simpler workaround.

puts "AABs: #{Actions.lane_context[SharedValues::GRADLE_ALL_AAB_OUTPUT_PATHS]}"
puts "AAB: #{Actions.lane_context[SharedValues::GRADLE_AAB_OUTPUT_PATH]}"

Yay, we just added those to the documentation: https://docs.fastlane.tools/actions/gradle/#lane-variables @mollyIV <3


Prediction with best weights get error

After saving best weights with save_best_weights=True (TinyYoloV2 model) option I try to use this weights and predict.
model.set_weights('model_best_weights_uvSn8E')
model.predict(data=dettbl) #dettbl is training table
I get error message:
NOTE: Model weights attached successfully!
ERROR: A floating point exception or access violation occurred that prevents progress of the analysis. This is usually caused by wrong scales of inputs, large learning rate, or exploding gradients.
ERROR: The action stopped due to errors.

what happens if you do not use the bestweights?  are you able to get anything?

If I don’t use best weights in “set_weight” prediction work normally, without errors.
From: sasxianhu<EMAIL_ADDRESS>Sent: Thursday, December 13, 2018 4:31 PM
To: sassoftware/python-dlpy<EMAIL_ADDRESS>Cc: Mariusz Dzieciatko<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [sassoftware/python-dlpy] Prediction with best weights get error (#91)
what happens if you do not use the bestweights? are you able to get anything?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/sassoftware/python-dlpy/issues/91#issuecomment-447010041, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ArjsKhD6XAHuEjXxUJHdmCBJKOeUF63Qks5u4nK9gaJpZM4ZRzjG.

Hi, I cannot repeat the error. Could you send me your script? So that I can debug it. Thanks.

detekcja_obiektow_test.zip

I use MPP but without GPU. I will try to do a test on SMP.

this should have been fixed with the new vdmml release. I am closing this issue and please re-open it if it happens with the newest vdmml release. thanks


HBASE-26542 Apply a package to test protobuf files (addendum)

RpcServer identifies the services it hosts by unqualified service name. Thus, use getName()
instead of getFullName(). See also HBASE-26589.

Your recommended solution in HBASE-26589 makes sense.

Maybe there's an argument to be made in favor of sending smallest messages possible? Perhaps the service should accept the name so long as it is unambiguous amongst the registered services. I don't think it likely that we'd host multiple services from different packages. RpcServer startup should probably make some assertions about the services it is hosting, that their unqualified names are unambiguous. As of now, it looks like we have a first-registered-wins scenario.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 29s
Docker mode activated.

Test failures are unrelated, and the previously broken test passes.


Howto: Add Bosnian translation

What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo
For resources
Description
Translated HOWTO.md to Bosnian.
Checklist:

[x] Read our contributing guidelines
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction)

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

I would separate the translation and the links to it into two separate PRs. The links generate the conflicts, and we can do a cleanup PR after things have settled a bit.


Rapid clicking on sort order can get pills and table headers out of sync

I thought we had fixed this, but:
If you very rapidly and repeatedly click on either the sort order pills or the table headers, sometimes  you can get the pill up/down indicator the the table header up/down indicator to be opposite each other.

The directions no longer get permanently out of sync. However, when you change the direction on one field, the table header changes immediately and the pill waits until the data returns. They should both happen at the same time.


Automator: update istio/api@master dependency in istio/istio@master

Generated by Automator - 2020-03-10T08:17:17+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
48b0cbd50a0caa68f24ee4eb26d8fbf194544f15
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
9777360b62f2561c7bd702fc7f5c270b47ab814f
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
68e24662cff118d95bed2a9b30bd68ced05faca3
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
9ea8cce209ba8004136546e839359e0cbed6abf5
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Windows compatibility and CommonJS interop change back to Babel 5 functionality

Hi, I have two changes.
Windows Compatibility
Use rimraf package which mirrors rm -rf behaviour. This is for compatibility with Windows on the
clean script.
CommonJS interop issue with exports.default
Because you're using Babel 6, exports default gets transformed to exports.default as ES6 module. For CommonJS interoperability it is needed to now import this module with .default which is not desirable as evident by #116. babel-plugin-add-module-exports reverts to Babel 5 functionality, which does the same as above, but also adds a module.exports line for CommonJS.

Thanks @khell! 😄


Added ReferenceBidirectionalRange to std.internal.test.dummyrange

This was separated out of an upcoming PR of mine.

:white_check_mark:

LGTM.

Auto-merge toggled on

Thanks!


NIFI-12239 - Add OS/Java details at startup in bootstrap log

Summary
NIFI-12239 - Add OS/Java details at startup in bootstrap log
Example of the added log:
2023-10-17 17:26:33,501 INFO [main] org.apache.nifi.bootstrap.RunNiFi {"javaVersion":"21","cores":16,"totalPhysicalMemoryMB":16384,"maxOpenFileDescriptors":10240,"xms":"1g","xmx":"1g"}

Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[ ] Apache NiFi Jira issue created

Pull Request Tracking

[ ] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[ ] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[ ] Pull Request based on current revision of the main branch
[ ] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[ ] Build completed using mvn clean install -P contrib-check

[ ] JDK 21

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Yeah JSON was kind of a default choice to ease automatic parsing of this data but a KV approach is fine with me too, will update

New format:
2023-10-17 18:24:24,923 INFO [main] org.apache.nifi.bootstrap.RunNiFi {totalPhysicalMemoryMB=16384, xmx=1g, cores=16, javaVersion=21, maxOpenFileDescriptors=10240, xms=1g}

Switched to Runtime for available processors. Open to suggestions for the two other values unless we believe we should just drop those values because not important enough to justify the use of MXBean

Updated with your suggestion


Documentation Tooltip for Rules

If anyone wants to post any ideas about how to implement, would be glad to take a look.

Hello @aryeh-looker, can you give us an example of a project where you see this issue? I can see the documentation when I try with this example kotlin project

Closing the issue, if you can still reproduce it please reopen.


404 Page Not Found

I followed the exact steps mentioned in the README

Set up everything
Added a 'dev' branch (not master)
Ran bin/deploy successfully

But I still see the following message in the settings tab
Your site is having problems building: The tag asset_path on line 1 in _includes/image.html, is not a recognized Liquid tag.
Pls help!

Can you re-run bin/setup?

I ran bin/setup and bin/deploy again. I didn't get any errors while doing so, but I still get 404 Page Not Found when I load my page and the same error message in the settings tab. Since mine is a user page, it is directly being sourced from the master branch. Could that be the reason? I am new to Jekyll so any help would be appreciated!

I use another service to host the static files, so I'm not the best person to help you out with this one. @nielsenramon, is more knowledgeable than me on this pattern.
However, on the the README.md there's this paragraph (emphasis mine):

Important note: Chalk does not support the standard way of Jekyll hosting on GitHub Pages. You need to deploy your working branch (can be any branch, for xxx.github.io users: use another branch than master) with the bin/deploy script. This is because Chalk uses Jekyll plugins that aren't supported by GitHub pages. The bin/deploy script will automatically build the entire project, then push it to the gh-pages branch of your repo. The script creates that branch for you so no need to create it yourself.

Are you using xxx.github.io? Can you try using another branch with bin/deploy?

I am using xxx.github.io. But I have created a new branch (dev) that is separate from my master branch, and am running all the commands in dev. Am I missing something here?

@sidv1993 Can you remove the master branch eventually and see if that helps? You could also switch the target branch in the repo settings!
Let me know if that helps.

@nielsenramon I tried that. Didn't work. What do you mean by switching the target branch? I have set my default branch to (dev) and not master.

@sidv1993 Can you set it to the gh-pages branch and see if that worked (in the settings)?

@sidv1993, long shot, but did you change the base url?

@nielsenramon I tried that but it didn't work. Infact, when I removed my master branch the settings page indicated that "Github pages isn't set up". So it seems that I need to keep my master branch in there. And the error that I get when doing so is " Your site is having problems building: The tag asset_path on line 1 in _includes/image.html, is not a recognized Liquid tag"
@migueldemoura  I had added a baseurl:xxx.github.io in my _config.yml, but I have removed it. I just have url: http://<IP_ADDRESS>:4000 currently.
It seems to me that the master branch is having an issue with the {% asset_path } liquid tag. Is there a way add it correctly?

@sidv1993, the baseurl option is to change your blog home from xxx.github.io/ to xxx.github.io/$baseurl, and requires an ajustment to the deploy script. Since you removed that option and its "invalid" value, could you try and redo the deploy? There's a similar issue here https://github.com/nielsenramon/chalk/issues/108. This may be the reason for the "404", with the other error coming as a result of the branch modifications.

I tried removing baseurl, commiting & pushing the change, running bin/deploy from my (dev) branch. But didn't work. Here is a screenshot of the issue.

It seems to me that the master branch is required for hosting username.github.pages. But while doing bin/deploy, it is advised to use a branch other than master for xxx.github.io.

Do I have to do something similar with bower install --allow-root as mentioned in https://github.com/nielsenramon/chalk/issues/18

Fixed! I was able to get it to work using @cheny1ran comments in #18 about merging gh-pages and the master branch. Here is what I did-

Ran bin/deploy on (dev) branch, which created a new gh-pages branch
git checkout master and git merge gh-pages
Started up xxx.github.io and it showed up.
Thanks, @migueldemoura  and @nielsenramon  for the help!

@sidv1993 Glad you got it working! I really need to come up with a better way of supporting .github.io sites 😃


Training on multi classes

Hi, first of all. Thank you for sharing your excellent work. I tried to modify a code to train a multi classes detection.
In this line. Does it allow to change to my own multi classes??? thank you.
bboxes_labels_original = ['Glue tube' for _ in bboxes_original]

Hi, first of all. Thank you for sharing your excellent work. I tried to modify a code to train a multi classes detection. In this line. Does it allow to change to my own multi classes??? thank you. bboxes_labels_original = ['Glue tube' for _ in bboxes_original]

Hi, did you find the way to solve this problem? I have an idea to solve two problems (object classification and keypoint detection) by two different algorithms,. But of course one would be better ;)

if you use two algorithms, the time would be influenced.

Of course, but it's my task to use neural network for classification and localization with orientation (I want to use keypoint detection for this). Even if it cause longer time.


CSV Import no longer requires 'email' to be case sensitive, can be written with a dash e.g. E-mail

It seems the reason it could not accept variations of the word email is due to client side validation on the ImportCSV component. I just added some simple conditional logic that allows all reasonable variations of E-mail to be valid.

Hey so I did my best to undo those changes, but I'm not sure if I did so properly. I've never had to rewrite a branch's history before. If I made a mistake please let me know so I can do it properly.

I'm not exactly sure of what you did.
Squashing your PR before merging would be clean enough. Should I do it or should I leave you the opportunity to try your hand at reseting and make this PR all shiny and squeaky clean?
Here is what I'm proposing you to try:
Before anything, make a copy of your mail-for-good directory, just in case. You can go with git reset HEAD~X where 'X' is the number the commits you want to erase.
After that, commit only "client/containers/lists/ImportCSV.js" and here you have the most meaningful and concise PR.
I'm most likely a maniac and I should squash then merge. I'm just convinced that if the history is clean enough it leads to everyone understanding who did what, how it was done and why.

I did what you recommended and it seems the old commits are still on file. I think it's looking like I'm too much of a git noob to pull this off cleanly. Sorry, I tried my best to fix it! And also sorry for not going for it for 8 days haha.

Sorry, I tried my best to fix it!

Alright, you tried your best. I'll merge it thank you for your time and effort toward solving this issue.

sorry for not going for it for 8 days haha.

<Sarcasm>See, I'm pretty sure that everyone who did interact with this repo knows by know that I'm fast to answer, so yeah, you better be sorry.</Sarcasm>


PROOF-OF-CONCEPT: Externalized React, React-DOM to support React Addons

This is not intended for merge!
This is the result of my experiments with React-MD (#196), Material-UI (#119), and others relying on react-addons-* (#204).
To make React Addons usable in Next.js out of the box, Next.js has to accomplish the following:

Allow custom Webpack configs - #40, #174
Allow custom Babel config - #26 (to allow aliasing using babel-plugin-module-resolver)
Provide a way to configure external scripts (I had to hardcode React-MD into Next.js to make it a properly working external dependency)

Here are the steps to get a working project with React-MD:

package.json:
{
  "name": "my-app",
  "dependencies": {
    "next": "^1.1.1",
    "react-addons-css-transition-group": "^15.3.2",
    "react-addons-pure-render-mixin": "^15.3.2",
    "react-addons-transition-group": "^15.3.2",
    "react-md": "^0.3.7"
  },
  "scripts": {
    "test": "jest",
    "dev": "next",
    "build": "next build",
    "start": "next start"
  },
  "devDependencies": {
    "babel-jest": "^16.0.0",
    "enzyme": "^2.5.1",
    "jest": "^16.0.2",
    "react-addons-test-utils": "^15.3.2"
  },
  "babel": {
    "presets": [
      "es2015",
      "react"
    ]
  }
}

pages/index.js:
import React from 'react'
import Head from 'next/head'
import { RaisedButton } from 'react-md/lib/Buttons'

export default class App extends React.Component {
  render() {
    return <div>
      <Head>
        <link rel="stylesheet" href="https://npmcdn.com/react-md/dist/react-md.css" />
        <link rel="stylesheet" href="https://fonts.googleapis.com/css?family=Roboto:300,400,500" />
        <link rel="stylesheet" href="https://fonts.googleapis.com/css?family=Material+Icons" />
      </Head>

      <RaisedButton label="I am a button!" />
    </div>
  }
}

npm install

Fetch these changes and compile Next.js yourself, or extract this dist into ./node_modules/next/dist: dist.tar.gz

npm run dev

/cc @nkzawa

Fixed by @arunoda


Add "Content-MD5" header for S3.put_object/3

I was trying to follow the Amazon S3's guide on the ObjectPUT API. Since I had some integrity requirements that I need to fulfill while uploading files to AWS, I decided to use the Content-MD5 field in header to make sure that the upload was not corrupted.
However, when I checked through the list of allowed headers in ExAws.S3.Utils. I figured out that :content_MD5 was not included in the list.  It would be great if that's added to the list of allowed headers.

This does indeed appear to be an oversight. On it.

Fixed in Master, apologies on the delay.

@benwilson512 Got it. Thank you so much!


Please downgrade chalk to V4, new V5 is causing compatibility issues

The new chalk V5 is has compatibility issues, mentioned in their NPM page:

IMPORTANT: Chalk 5 is ESM. If you want to use Chalk with TypeScript or a build tool, you will probably want to use Chalk 4 for now. Read more.

I've reported it with example project here:
https://github.com/Lusito/mockzilla-webextension/issues/14
But this package is the source of the issues:
https://github.com/Lusito/mockzilla/blob/bafad35fb7dc7ddd7053ac3359b97de5354c3bc3/package.json#L38
The release notes for V5 says:

It's totally fine to stay on Chalk v4. It's been stable for years.

So, could you downgrade it to v4? :)

Hi @Juraj-Masiar, It seems jest still uses v4 as well, so I'll go back for now. You should consider updating your project to be ESM compatible though, as sooner or later all libraries will change to this.

New versions of mockzilla and mockzilla-webextension have been released.

Thank you @Lusito for the fix! It works now!
I've tried to migrate to ESM some (long) time ago but something didn't worked and I had to rollback.
Did you converted your addons / projects to ESM? How did it go? Any tips? :)
I have a lot of code shared between addons / frontend Angular projects / backend NestJS so big changes like this are pretty challenging for me :(.
Especially when I have so many other feature requests to implement...
But I'm sure you are super busy as well, so thank you again for taking time fixing my issue :), have a great week!


Missing idle state for heater

Is your feature request related to a problem? Please describe.
With current implementation you don't see how long the heater runs in order to keep target temperature. It is shown as it heat all time when it is on.
Describe the solution you'd like
When target temp is achieved, thermostat gets to idle state.
Describe alternatives you've considered
Template switch and climate can get this functionality:
switch
  - platform: template
    switches:
      spa_heat:
        value_template: "{{ is_state('climate.spa', 'heat') }}"
        turn_on:
          service: climate.turn_on
          target:
            entity_id: climate.spa
        turn_off:
          service: climate.turn_off
          target:
            entity_id: climate.spa
climate:
  - platform: generic_thermostat
    name: SPA_Custom
    heater: switch.spa_heat
    target_sensor: sensor.spa_current_temperature
    max_temp: 40
    min_temp: 20
    target_temp: 37
    precision: 1

Hi @jaroslav-fedor-swi,
This integration aims at connecting the spa and provide home assistant with data retrieved from the spa. However, this heat/idle effective state is not provided by the data retrieved from the spa.
We might optimistically assume the real state, but there are many parameters missing: the spa internal thermostat algorithm and parameters are unknown, and the provided water temperature precision level is certainly lower than the one the spa internal thermostat has access to. Hence, assuming the real state would lead to a significant error in the value provided, and this is why I will not implement this feature.
Nevertheless, as you suggested, you could use a climate template to get a general idea of the spa heat/idle state, just remember it will not provide a reliable status.
A reliable status could be retrieved by measuring the spa consumed power: each component of the spa unit has its own consumption and we can deduct what is currently used. Heater is around 2kW.


MOVR liquidity provisioning

refactor to break apart large re-usable MOVR methods.
nearly complete support of scraping liquidity provisioning (but no tax analysis yet)
export the MOVR transactions to an .xlsx spreadsheet (using pandas)

great PR. I didn't realize how far you have come so far.
I would like to offer some general feedback. In order for me to be able to properly review future PRs, we need to find ways how I can easily test it, either by unit tests or samples in the samples directory.
Right now I can only read through the code and assume it does what it says.

Thanks for merging. So to-do list when I return to subscrape is:

proceed with deleting the old JSON output file and recreating it.
Finish interpreting LP, specifically the Solarbeam stable pairs with different interface.
Add to documentation
Add at least one unit test to demonstrate the code.


manual_backward logical bug

Actually, this problem has been raised in issue #7698 , but it has been closed.
This problem still exists in V1.3.7. My current solution is to comment track_and_norm_grad function. I would suggest adding a conditional judgment of grad clipping.

    def backward(self, result, optimizer, opt_idx, *args, **kwargs):
        self.trainer.dev_debugger.track_event("backward_call")

        should_accumulate = self.should_accumulate()

        # backward can be called manually in the training loop
        if isinstance(result, torch.Tensor):
            self.trainer.accelerator.backward(result, optimizer, opt_idx, should_accumulate, *args, **kwargs)
        else:
            result.closure_loss = self.trainer.accelerator.backward(
                result.closure_loss, optimizer, opt_idx, should_accumulate, *args, **kwargs
            )

        if not self.should_accumulate() and self.grad_clip:
            # track gradients
            self.track_and_norm_grad(optimizer=optimizer)

I recently also encounter this problem. I think there is indeed some logic error in manual_loss.
    ....
    self.manual_backward(loss)

lightning.py: manual_backward

    def manual_backward(self, loss: Tensor, optimizer: Optional[Optimizer] = None, *args, **kwargs) -> None:
        ...
        # backward
        self._running_manual_backward = True
        self.trainer.train_loop.backward(loss, optimizer=None, opt_idx=None, *args, **kwargs)
        self._running_manual_backward = False

trainer.train_loop.backward(loss, optimizer=None,...)

the argument optimizer is None, but the last line requires a optimizer
    def backward(self, result, optimizer, opt_idx, *args, **kwargs):
        self.trainer.dev_debugger.track_event("backward_call")

        should_accumulate = self.should_accumulate()

        # backward can be called manually in the training loop
        if isinstance(result, torch.Tensor):
            self.trainer.accelerator.backward(result, optimizer, opt_idx, should_accumulate, *args, **kwargs)
        else:
            result.closure_loss = self.trainer.accelerator.backward(
                result.closure_loss, optimizer, opt_idx, should_accumulate, *args, **kwargs
            )

        if not self.should_accumulate():
            # track gradients
            self.track_and_norm_grad(optimizer=optimizer)   # raise error here

Hi!
Setting gradient_clip_val or accumulate_grad_batches on your Trainer with automatic_optimization=False (aka using manual_backward) is not allowed. #7788 (the PR that closed the issue you linked) implements a better error message for that case but still stops the program.
You are not seeing this in the 1.3.7 release because the changes are in master and will be released with 1.4.0.
But in your case, you just need to remove the aforementioned flags in your Trainer definition.


Posting notifcation

When trying to post (with images), I get an error message that says images are uploading.
It can't be read though because it only lasts a split second.

This has been reported again in issue 50. https://github.com/guardianproject/securereader_ios/issues/50


Publish with Ghost on Github Pages

Ghost change a lot . after 1.0.0 . Can anyone help me to figure out how can i publish ghost in github pages

Ghost is not a static site generator and GitHub is not a suitable place for questions. If you need help please use slack.


Anchored headings

Trello: https://trello.com/c/MdleLLSX/726-add-a-link-to-here-link-to-headers-in-the-design-system-to-make-it-easier-for-support-staff-to-send-links-to-users

You can preview this change here:
Built with commit b88b27cc92895eb6c56503685dffd84964491b86
https://deploy-preview-295--govuk-design-system-preview.netlify.com

Spoke to Joe, we're not happy with a) That the icon will be understood by users b) That it's discoverable on mobile only when clicking the header.
With this in mind we want to try a 'contents list' approach as we believe it'll be more well understood.
https://govuk-publishing-components.herokuapp.com/component-guide/contents_list


Dev

named tuple support added.

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

Project coverage is 91.86%. Comparing base (0e2c209) to head (e928d4b).
Report is 33 commits behind head on master.

Files
Patch %
Lines

pydantic_xml/serializers/factories/named_tuple.py
87.80%
5 Missing :warning:

pydantic_xml/serializers/factories/call.py
90.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #193      +/-   ##
==========================================
- Coverage   91.96%   91.86%   -0.10%
==========================================
  Files          27       29       +2
  Lines        1555     1610      +55
==========================================
+ Hits         1430     1479      +49
- Misses        125      131       +6

Flag
Coverage Δ

unittests
91.86% <89.47%> (-0.10%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


eq not implemented on MidiFile

All in the title

What do you need this for? What is "equality" between MIDI files? Exact same bytes? Same song?

This is a great suggestion! I'll look at your implementation and we'll get this into Mido.


QKTM on 1.20.1

This was an experience I think

Whelp nobody has said anything so I'll merge in a couple minutes, I just want to keep a branch for the currently configured version


Add uefi to workspace members

I'm confused as to how this hasn't caused problems to be missing -- like why does cargo fmt --all work for the uefi crate even though it was missing from this list?
Checklist

[ ] Sensible git history (for example, squash "typo" or "fix" commits). See the Rewriting History guide for help.
[ ] Update the changelog (if necessary)

All path dependencies residing in the workspace directory automatically become members.

https://doc.rust-lang.org/cargo/reference/workspaces.html#:~:text=The exclude key can be,want to remove a directory.

That's probably what happened here

Thanks, makes sense. I was scratching my head over that :)


stdin?

it would be awesome to be able to use this with stdin: npx awesome-npx | npx markdown-chalk --input -- or something.

@zkat Sorry for missing this for so long. Yes, I think that'd be awesome. It's currently very hard coded to read from a file, but I'd be happy for PRs related to this, and failing that I'll get to it at some point soon-ish and explore how to handle stdin input :) :+1:
Thanks for creating this :)


Does not support special characters (ä,ö,å, etc.)

This is not a problem but it's worth to mention. Had an ID called myymälä. Smoothscroll didn't react to it as it should.

Hey, so sorry about this. This was a bug that had been fixed many versions ago but got reintroduced in a rewrite. v16.1.1 fixes it. Just pushed.
Thanks for letting me know!


Issues with RTD and CommonMark > 0.5.6

Recently RTD just stopped building my docs with the following error:
python /home/docs/checkouts/readthedocs.org/user_builds/fyrd/envs/dev/bin/sphinx-build -T -E -b readthedocs -d _build/doctrees-readthedocs -D language=en . _build/html
Running Sphinx v1.5.3
making output directory...

Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/fyrd/envs/dev/lib/python3.5/site-packages/sphinx/application.py", line 512, in setup_extension
    mod = __import__(extension, None, None, ['setup'])
  File "/home/docs/checkouts/readthedocs.org/user_builds/fyrd/envs/dev/lib/python3.5/site-packages/sphinxarg/ext.py", line 13, in <module>
    from sphinxarg.markdown import parseMarkDownBlock
  File "/home/docs/checkouts/readthedocs.org/user_builds/fyrd/envs/dev/lib/python3.5/site-packages/sphinxarg/markdown.py", line 1, in <module>
    from CommonMark import Parser  # >= 0.5.6
ImportError: cannot import name 'Parser'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/fyrd/envs/dev/lib/python3.5/site-packages/sphinx/cmdline.py", line 295, in main
    opts.warningiserror, opts.tags, opts.verbosity, opts.jobs)
  File "/home/docs/checkouts/readthedocs.org/user_builds/fyrd/envs/dev/lib/python3.5/site-packages/sphinx/application.py", line 199, in __init__
    self.setup_extension(extension)
  File "/home/docs/checkouts/readthedocs.org/user_builds/fyrd/envs/dev/lib/python3.5/site-packages/sphinx/application.py", line 516, in setup_extension
    err)
sphinx.errors.ExtensionError: Could not import extension sphinxarg.ext (exception: cannot import name 'Parser')

Extension error:
Could not import extension sphinxarg.ext (exception: cannot import name 'Parser')

I also reported the on the RTD issue tracker here: https://github.com/rtfd/readthedocs.org/issues/3032

Would you have a wordkaround for this problem on rtd ? Does it need a new sphinx-argparse version relaxing the dependency on CommonMark ?
Thanks


[AIRFLOW-1267] [AIRFLOW-1874] Add dialect parameter to BigQueryHook

Allows a default BigQuery dialect to be specified at the hook level, which is threaded through to the
underlying cursors.
This allows standard SQL dialect to be used, while maintaining compatibility with the
DbApiHook interface.
Addresses AIRFLOW-1267 and AIRFLOW-1874
JIRA

Addresses https://issues.apache.org/jira/browse/AIRFLOW-1267 and https://issues.apache.org/jira/browse/AIRFLOW-1874

Description

Adds a use_legacy_sql parameter to the BigQueryHook constructor. This parameter determines whether queries executed via this hook will be issued using legacy or standard dialect by default. To maintain compatibility, this can still be overridden at the cursor level, and in the get_pandas_df method.

Tests

Adds TestBigQueryHookLegacySql, which tests that this parameter is threaded through from hook instantiation down to the underlying cursor. No regressions on existing BigQueryHook tests

This is a backwards incompatible change that has broken some of our DAGs.

I'm going to revert this.

Sigh. Revert has a ton of conflicts due to the velocity of this file.
@ji-han you can't just remove parameters from public facing APIs. This will affect everyone using use_legacy_sql param with BigQuery operator, hook, cursor, etc. That's a lot of people.

I believe the correct fix is to just add the use_legacy_sql back into the run_query method and whatnot. It can override the hook default if it's set.

@criccomini my apologies - this was an oversight on my part. I’ll look into a fix to restore the parameter.

@ji-han thanks!

@criccomini I've opened a PR here: https://github.com/apache/incubator-airflow/pull/3087


persist ID as string shows NaN

Describe the bug
when id type is string, it shows NaN after persist
Steps to reproduce the bug
i have this model
import { Model as ORMModel } from '@vuex-orm/core'
import Model from './Model'

export default class Make extends ORMModel {
  static entity = 'makes'
  static eagerLoad = ['models']

  static fields() {
    return {
      id: this.string(),
      name: this.string(null).nullable(),
      active: this.boolean(true),
      models: this.hasMany(Model, 'makeId')
    }
  }
}

I use fireorm as database and everytime the model transform and inserted in vuex-orm store. it shows NaN. see console.log output:

Expected behaviour
Should allow id as string type and save correct id value.
Versions

Vuex ORM: 0.36.3
I am using Nuxt: 2.14.12

I think it because of toPrimaryKey method which used on id from server response.
See my issue: https://github.com/vuex-orm/plugin-graphql/issues/173

so I ended up creating a path file for plugin-graphql and remove the Guid formatting and leave it just a string.


fix: eventwatch rule limit problem

Make an eventwatch rule per lambda so that the limit of 5 is not exceeded.

Error: This repo is not allowlisted for Atlantis.


Limit exposure to ConcurrentModificationException when sys props are replaced or mutated

port of https://github.com/scala/scala/commit/f6859f28bb49193fde83e6020a6a89ce926a91e8 to fix https://github.com/sbt/sbt/issues/7873

I think the build error is unrelated
[info] [republish] Fetching org.scala-lang:scala3-interfaces:3.6.4-RC1-bin-SNAPSHOT with coursier.jar...
[info] [republish] Fetching org.scala-lang:scala3-compiler_3:3.6.4-RC1-bin-SNAPSHOT with coursier.jar...
Resolution error: Error downloading org.scala-lang.modules:scala-asm:9.7.0-scala-2
  download error: Caught java.net.ConnectException (Connection timed out) while downloading https://repo1.maven.org/maven2/org/scala-lang/modules/scala-asm/9.7.0-scala-2/scala-asm-9.7.0-scala-2.pom
  not found: /__w/scala3/scala3/dist/linux-x86_64/target/streams/_global/republishAllResolved/_global/streams/localRepo/maven2/org/scala-lang/modules/scala-asm/9.7.0-scala-2/scala-asm-9.7.0-scala-2.pom
Error:  Error running coursier.jar with args fetch --no-default --repository central --repository file:///__w/scala3/scala3/dist/linux-x86_64/target/streams/_global/republishAllResolved/_global/streams/localRepo/maven2 org.scala-lang:scala3-compiler_3:3.6.4-RC1-bin-SNAPSHOT
Error:  (dist-linux-x86_64 / republishAllResolved) Error running coursier.jar with args fetch --no-default --repository central --repository file:///__w/scala3/scala3/dist/linux-x86_64/target/streams/_global/republishAllResolved/_global/streams/localRepo/maven2 org.scala-lang:scala3-compiler_3:3.6.4-RC1-bin-SNAPSHOT
Error:  Total time: 362 s (06:02), completed Dec 10, 2024, 12:40:45 PM
Error: Process completed with exit code 1.

retrying what I hope will prove to be a transient CI failure on Windows

Thanks for merging this. You don't need nothing else from my side to backport this to other scala 3 branches, right?

Thanks for merging this. You don't need nothing else from my side to backport this to other scala 3 branches, right?

They all end up on out board and we port everything we can to the lts branch and will automatically be used for the next version. We will most likely have it in 3.3.6 and 3.6.4

(regardless, adding the "backport:nominated" label for extra assurance that it will be backported if at all possible)


[Enhancement] add missing DDO attributes

Motivation / Problem
tags and copyrightHolder are missing in the assetBuilder
TODO: check for other missing attributes
TODO2: implement functions for all missing attributes

Yes, we will need to set the tags. It would be helpful to do so through the AssetBuilder

Others that seem missing are "license" and "links"

For the moment got it working by doing:
const asset = assetBuilder.build();
asset.metadata.tags = ['tag1', 'tag2'];
asset.metadata.links = ['https://samples.org/file.csv'];
const result = await nautilus.publish(asset);


Several fixes to get test suite passing on Cygwin

See individual commit messages for more details.  I only skipped a tiny handful of tests.  In other cases test files were split up so that I could skip only those cases that depend on long double functions on Cygwin (pending any better resolution to that issue).
In other cases a few tests were slightly rewritten to work correctly, and a few fixes to the test runner itself were needed.
I don't think there is any continuous integration being done for Cython on Windows, or Cygwin in particular.  If there's any interest I would be willing to work on setting that up and maintaining it.

In any case this PR is so old it needs to be rebased, and I'm doubtful whether it still solves all test issues (it did back in May I guess...)

Thank you for you efforts, sorry it took so long to get this merged.

Did you check that #360 fixes the module_api test on Cygwin? If so, why aren't we merging #360?

That was a long time ago so I'll have to double-check, but I believe the answer at the time was "no", since the fix in #360 does not go deep enough.


Can't install on Mac 10.13.1

Hello,
I get the following errors when I make the latest bedtools2 (showing the tail of the output):
StrandQueue.cpp:91:15: error: no member named 'push' in
'std::__1::priority_queue<Record *, std::__1::vector<const Record *,
std::__1::allocator<const Record *> >, RecordPtrSortDescFunctor>'
_queues[2]->push(record);
~~~~~~~~~~  ^
StrandQueue.cpp:101:26: error: no member named 'size' in
'std::__1::priority_queue<Record *, std::__1::vector<const Record *,
std::__1::allocator<const Record *> >, RecordPtrSortDescFunctor>'
sumSize += _queues[i]->size();
~~~~~~~~~~  ^
fatal error: too many errors emitted, stopping now [-ferror-limit=]
20 errors generated.
make[2]: *** [../../../../obj//StrandQueue.o] Error 1
make[1]: *** [Records] Error 2
make: *** [src/utils/FileRecordTools] Error 2
What can I do?
Thanks

I'm running into a similar issue

Are you trying to install directly from a source clone, or are you using homebrew?
Not sure if this will help, but building from master/HEAD on Mac works in 10.12.6 with the following commands:
make CXX=clang++
sudo make install CXX=clang++

You should also be able compile the latest version in the repo:
git clone https://github.com/arq5x/bedtools2
cd bedtools2
make

I will be releasing a point release soon that addresses this.


[i2c, doc] Add checklist.md for i2c

[i2c, doc] Add checklist.md for i2c
Signed-off-by: Tung Hoang<EMAIL_ADDRESS>

Close this PR and submit a new one for V1


Spooky Kids Animation

Please check for duplicates, similar issues or closed issues with the same problem, as well performing simple troubleshooting steps (such as deleting the export folder if it's an issue with compiling, reinstalling Kade Engine) before submitting an issue.
If you are playing a downloaded version of the game, what operating system are you using?
Windows (x86), Windows (x86_64), Linux, or macOS? Specify below.
Windows ('x86_64')
What version of the game are you using? Look in the bottom left corner of the main menu.
Kade Engine V1.5, Vanilla game v<IP_ADDRESS>
Have you identified any steps to reproduce the bug? If so, please describe them below. Use images if possible.
Step 1: Play Week 2
Step 2: Watch the Spooky Kids' animation
Step 3: They will loop.
Please describe your issue. Provide extensive detail and images if possible.
I was making a FNF charting mod but then I found that the spooky kids' animation looped, but only on one side. It can alternate after their turns however.
The YouTube video linked below is proof.
https://www.youtube.com/watch?v=B381VQ7Msao

fixed already

Hi -  try 1.5.1 and see if it works. It appears to be fixed in master, so it might be part of 1.5.1. The below comment may help:

fixed already in master


MUMUP-2870 Removes unneeded lti-launch widget type

Removes the unneeded and unused LTI Widget Type.

Coverage increased (+0.02%) to 15.783% when pulling f6f216a565f43a59911180f46bf88b24d8ac820b on refactorWidgetLTIType into 50d77453986c478b7d86e1de1ac51ce356c84c1b on master.


Support for socks5-http-client/lib/Agent

adding
var requestArgs = ['uri','url','qs','method','headers','body','form','formData','json','multipart','followRedirect','followAllRedirects', 'maxRedirects','encoding','pool','timeout','proxy','auth','oauth','strictSSL','jar','aws','gzip','time','tunnel','proxyHeaderWhiteList','proxyHeaderExclusiveList','localAddress','forever', 'agent', 'agentOptions', 'agentClass'];

for instance:
const Agent = require('socks5-http-client/lib/Agent');

var c = new Crawler({
    // rateLimit: 2000,
    maxConnections: 20,
    agentClass: Agent,
    agentOptions: {
        socksHost: 'localhost',
        socksPort: 9050
    },...```

I've merged the pr, see #338

:+1: Thanks @mike442144

It's never going to be released?

It's never going to be released?

I think its already in master, just not added as release


Implement Signup functionality

Steps to implement:

Take the user input from the form
Check that password length should be greater than 6.
Request structure can be found here
For successful response from the server, you will get a secret in the response. Store that in localStorage with key name secret.
For each response show a message(antd component) whether signup was  successful or not.

The helper functions for sending requests are present in the components/utils/API folder and for localStorage functions are in components/utils/LocalStorage folder. Use these function only to send request.

Make sure to pull before starting your work.
For any doubts or difficulty, you can ping me on discord, I'll try to help you out as soon as possible.
All the Best!!

claim

Assigned to @TheDivineAurora

Any blockers @TheDivineAurora  ?

Give me time !

Giving you time till 2 am @TheDivineAurora

Ok

Good work @TheDivineAurora


"Alpha Animals" Pawns can't get into the door

Describe the bug
They can't get into the door but can freely pass through them.
Bug caused by Alpha Animals.
To Reproduce (VERY IMPORTANT)
Steps to reproduce the behavior:

Go to existing save file or make a new world
Turn on dev mode and build a door.
Select any pawn and right click the door

Error Log
Player.log : https://gist.github.com/GervyMaglina/53862c0d76d77cd6bcdb6e73d0e041a1
HugsLib log : https://gist.github.com/e079dd34d162723e76a96b6fdfa030c7
Mod List
Harmony
Core
HugsLib
Vanilla Expanded Framework
Alpha Animals
Mod Manager
Rimthreaded [1.2]

@GervyMaglina can you please attach a save file where the error is reproduced? that would accelerate resolution time. Thanks in advance!

Is it this?
https://www.dropbox.com/s/st8tarc69rdanhe/Error.rws?dl=0

It was only for a test since the origin of it contained a lot of mods. Even my pawn bedroom temperature was forced to stay atleast 100 C despite having no steam geyser or bad weather, I don't know what actually cause that to happen but it can happen when i load it the next time

The bug came when Vanilla Expanded Framework merged a new mod called MVCF into their mod, then Alpha Animals follow suit by overhauling all of their ranged animal system which produce the bug

issue has been fixed on 2.3.9


Add option for generic Persistent volume claims

Details
Helm chart name: radarr, sonarr, lidarr, nzbget
Describe the solution you'd like: For use of Remote Path Mappings there is another persistent volume claim needed. Currently the only option i see is to use the shared volume for it. It would be nice to add a default disabled generic volume in the commons.yaml to make it more flexible to mount other volumes into helm charts.

Did you check out our docs on the different storage options?
https://docs.k8s-at-home.com/our-helm-charts/common-library-storage
Using the custom type you should be able to mount any storage type you need

@bjw-s yes. but as far as i can understand it i can just add volumes that are already defined and for the radarr chart for example there is only config and media defined and the inherited shared volume. as far as i understand it there is no way to add more volumes.

Ok sorry i was wrong i thought i could just add volumes that are predefined. but seems like i can just give it a name and it works. Will close it :)


SubObjectPropertyOf axioms are somewhat complex

Parsing this:
SubObjectPropertyOf(:activity :influencer)

yields this axiom object:
SubObjectPropertyOf(subObjectPropertyExpression=SubObjectPropertyExpression(v=ObjectPropertyExpression(v=ObjectProperty(v=':activity'))), superObjectPropertyExpression=ObjectPropertyExpression(v=ObjectProperty(v=':influencer')), annotations=[])

This is kinda awkward to work with. All I want to do is reconstruct a basic property hierarchy, as shown here:

(this is prov)
My code is already quite complex:
if isinstance(a, SubObjectPropertyOf):
                print(f'SUB: {a}')
                if isinstance(a.subObjectPropertyExpression, ObjectProperty):
                    child = self.iri_to_name(a.subObjectPropertyExpression)
                    if isinstance(a.superObjectPropertyExpression, ObjectProperty):
                        parent = self.iri_to_name(a.superObjectPropertyExpression)
                        print(f'{child} < {parent}')
                        slot_isamap[child].add(parent)
                    else:
                        logging.error(f"cannot handle anon parent properties for {a}")
                else:
                    print(f"cannot handle anon child properties for {a}")

but this doesn't work as it seems I have to check various levels of .v objects?

I figured out how to do it:
if isinstance(a, SubObjectPropertyOf):
                sub = a.subObjectPropertyExpression.v
                if isinstance(sub, ObjectPropertyExpression) and isinstance(sub.v, ObjectProperty):
                    child = self.iri_to_name(sub.v)
                    sup = a.superObjectPropertyExpression.v
                    if isinstance(sup, ObjectPropertyExpression) and isinstance(sup.v, ObjectProperty):
                        parent = self.iri_to_name(sup.v)
                        print(f'{child} < {parent}')
                        slot_isamap[child].add(parent)
                    else:
                        logging.error(f"cannot handle anon parent properties for {a}")
                else:
                    print(f"cannot handle anon child properties for {a}")

but this seems a little complex

Lets use this as one of the first tests for issue #42


[Messaging Clients] Microsoft.Azure.Amqp Package Update

Summary
The focus of these changes is to update the version of the Microsoft.Azure.Amqp package used by the messaging clients to v2.4.6, which was recently
released.  This new version contains a fix for a hard crash on .NET v5 and no other changes.
Last Upstream Rebase
Wednesday, September 9, 10:12am (EDT)
References and Related Issues

Microsoft.Azure.Amqp Branch Commits
[BUG] Azure Service Bus Queue (RegisterMessageHandler) not working on .NET 5.0 Preview 7

/azp run net - servicebus - tests

/azp run net - eventhub - tests

/azp run net - eventhub - tests

/azp run net - servicebus - tests


Kuma Layout Change

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
Other
🔖 Feature description
Hello,
I would like to request a change in the current layout of Kuma, if possible, to improve the visualisation on panels / TV. The current layout is very strange and not practical when you have in mind to put on a TV to monitor all day, especially if there are several items to monitor.
✔ Solution
I believe that a layout that highlights the column that is currently on the left and that has cards for each monitor, with a cleaner designer focusing only on displaying whether it is on/off, if you want more information click on the card and a modal will be displayed, which would be the current right column.
❓ Alternatives
No response
📝 Additional Context
I display kuma on a 32" tv where I monitor my services and servers, but the current layout of kuma makes it very difficult and impractical to visualise the information on this screen.

@Chronos093 Please report issues in english. Nobody on our team can read portugese.
I have ran deepl over this issue.

Multi column displays for the status page are tracked in #3934, #1844 and #592
I don't see a real difference between this and the other issues
=> closing as a duplicate


Added inpage provider

This provider meets the EIP-1193 spec completely and is usable with all existing ethereum web3 libraries.

@chaitanyapotti Will we have any conflicts if someone has MetaMask installed alongside with Quill, such as one overriding the other at window.ethereum (https://github.com/MetaMask/providers#usage)?

I haven't added that part yet. We can decide on whether we should use window.ethereum or window.quill in this extension at a later date

@chaitanyapotti Will we have any conflicts if someone has MetaMask installed alongside with Quill, such as one overriding the other at window.ethereum (https://github.com/MetaMask/providers#usage)?

Yeah that's going to be an issue. Currently I think we overwrite and also set window.quillEthereumProvider so you can be explicit if you want to. We should probably look into what else we can do here like notifying the user if a conflict is detected.

@chaitanyapotti I marked this as a draft because (correct me if I'm wrong) this is currently just adding unused code. Let us know if I'm mistaken or when it becomes ready.

It looks like this design is heavily copied (from torus?). I'm concerned that we're bringing in more than we need by doing that. E.g. Why is there a BaseProvider? Why have streams been introduced? Also need to make sure the licence is being complied with if it's not public domain - getting tangled up in licences via copy+modify makes me nervous (more complicated than when you have an exact copy by using a dependency).

Also, this appears to be aimed at accurately implementing the provider spec (e.g. including deprecated APIs), which is quite different from adding support for arbitrary transactions. Wouldn't it be easier to do that first? It would more naturally build on top of the current system and ties in with the work @kautukkundan is doing to represent those transactions in the confirm dialog?
(Not to say that we don't need to implement the spec accurately, I assume we will and these contributions can still be useful.)

(Sorry if this is raining on your parade a bit... maybe we should have a chat about it?)

Addressed the comments in jitsi call.
Here's how metamask connects ui to background:
https://github.com/MetaMask/metamask-extension/blob/develop/app/scripts/ui.js
https://github.com/MetaMask/metamask-extension/blob/develop/ui/index.js

🤞


Credentials warning in dsub version 0.3.4

Since the 0.3.4 release, this warning appears when running dsub with the google-v2 provider:
"""
UserWarning: Your application has authenticated using end user credentials from Google Cloud SDK. We recommend that most server applications use service accounts instead. If your application continues to use end user credentials from Cloud SDK, you might receive a "quota exceeded" or "API not enabled" error. For more information about service accounts, see https://cloud.google.com/docs/authentication/
warnings.warn(_CLOUD_SDK_CREDENTIALS_WARNING)
"""
Because dsub is not a server application, this warning can be safely ignored. This warning will be filtered out in the next release.

This is fixed in 0.3.5


Feat/remaining lock file

Fixes #104
Has to be configurable via an override flag on the nitric up command as a conflict is a valid reason to fail if there is more than 1 person working on a stack concurrently. If we make it the default to cancel the updating stack, the other person will lose their update progress. Thus, I've added a flag -c or --cancel, to cancel the current update in progress.

Codecov Report

Merging #258 (d5ad1ab) into develop (5a3841b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #258   +/-   ##
========================================
  Coverage    34.05%   34.05%
========================================
  Files           49       49
  Lines         4513     4513
========================================
  Hits          1537     1537
  Misses        2824     2824
  Partials       152      152

Flag
Coverage Δ

unittests
34.05% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


Datagrid improvement

Issue Reference
The first and the last page action are missing in the pagination af datagrids
Description
The possibity to go directly to the first or to the last page of a datagrid has been added without to have to type the page number in the paginator. It prevents user to go back to the keyboard while navigating throw recordsets

Go to the first page
Go to the last page

Hi, I made a little correctif because of the alignment of the filter button. Now it's ok


Goutte Factory sending through wrong Guzzle config

Hi there,
We have been using Mink for a while with our self cert SSL and after an upgrade via composer it's not not been working. Were currently running the following

Mink 1.7.0
Behat 3.0.15
Mink Extension 2.1.0
Mink Goutte Driver 1.2.0
Guzzle 6

I found that I needed to change the configuration to my behat to be the following
default:
    extensions:
        Behat\Symfony2Extension: ~
        Behat\MinkExtension:
            sessions:
                default:
                    goutte:
                        guzzle_parameters:
                            verify: false

So in theory everything should be fine but our self-cert is still being checked reguardless of what we tell Mink. After some digging we found that the parameter is not being passed through to Guzzle, the problem lies in GoutteFactory line 93 where it does
return new Definition('GuzzleHttp\Client', array(array('defaults' => $parameters)));

The parameter being passed to Guzzle are being put one level deeper than they need to be. Now I find it hard to believe that the problem lies with this bit of code but it is the only real explanation. I have written a little bit of test code to try and prove it using Guzzle direct. The following fails
$url = 'https://my.url.com';
$client = new GuzzleHttp\Client(['defaults' => ['verify' => false]]);
$var = $client->request('GET', $url);

Whereas this passes
$url = 'https://my.url.com';
$client = new GuzzleHttp\Client(['verify' => false]);
$var = $client->request('GET', $url);

If I remove the extra layer in GoutteFactory e.g.
return new Definition('GuzzleHttp\Client', array($parameters));

Our SSL is skipped as expected. I'm still digging in the problem a bit but as the Mink driver and Guzzle are so widely used it feels very odd that I have to change GoutteFactory for this to work. Sorry for the wall of text but any help is appreciated

I ran into the same issue when setting up a new project using the goutte driver and library. It looks like Goutte v3 uses Guzzle 6 which changed the client config format. I worked around it in the meantime by adding Goutte v2 to my composer dependencies which uses Guzzle 5 and the old config format.
I believe this should work as a possible solution without breaking BC, but want to make sure that it looks acceptable for a pull request. Let me know what you think.
https://github.com/Behat/MinkExtension/compare/master...epfremmer:master

Good to know I'm not the only one and going mad :)

@epfremmer same issue here. guzzle6 changed arguments. i can confirm your changes are working. for now downgrading goutte is working. maybe open a merge request so @stof can extend the last commit in GoutteFactory
"fabpot/goutte": "~2.0"

  - Removing guzzlehttp/guzzle (6.1.1)
  - Installing guzzlehttp/guzzle (5.3.0)
    Downloading: 100%
    Extracting archive

  - Removing fabpot/goutte (v3.1.2)
  - Installing fabpot/goutte (v2.0.4)
    Downloading: 100%
    Extracting archive

@epfremmer I confirm the fix worked for me too.  It is almost exactly as I would have written it.  I would make one change.  Reflection is not needed as the constant from the interface is publicly available.  So, the isGuzzle6 becomes:
private function isGuzzle6()
{
    return interface_exists('GuzzleHttp\ClientInterface') &&
        version_compare(\GuzzleHttp\ClientInterface::VERSION, '6.0.0', '>=');
}

:+1: I ended up rolling back to ``"behat/mink-goutte-driver": "1.1",` in order to get guzzle to use my verify parameter.

fixed by https://github.com/Behat/MinkExtension/commit/7f0c691d24784b1ef924fb58ff0cfef709ca38de. but not implements a version check

@Haehnchen the issue is that your fix broke support for Guzzle 4 and 5 (and so for Goutte 2). Only Guzzle 6 does not need the default key


Provide example of copying directory

Clear example of how to copy a folder.

Thanks for the PR.
I think this is already clear enough though. Have you had any difficulty about the example?

Thanks for the feedback. I've updated the README about that.


Editor: Arrow key doesn't work

VSCode Version: Code 1.11.1 (d9484d12b38879b7f4cdd1150efeb2fd2c1fbf39, 2017-04-06T13:55:56.395Z)
OS Version: Linux x64 4.4.51-40.58.amzn1.x86_64
Extensions: none

Quite simple but serious problem.
Around version 1.11, editor window doesn't accept any arrow key at all!
I tried clean reinstall, however, it doesn't solve this symptom.
Steps to Reproduce:

Install Visual Studio Code
Open a file, type some characters, and press arrow keys to move around.

I'm using LXDE desktop on Ubuntu 16.
Special things are this environment is on TigerVNC virtual desktop prepared with Docker container version of Ubuntu.
$ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 16.04.2 LTS
Release:	16.04
Codename:	xenial

Duplicate of #24107 -- the latest insiders build already contains a fix


fix(builder): checkSyntax targets should get default browserlist when only set checkSyntax.exclude

Summary

🤖 Generated by Copilot at 42084a4
This pull request fixes and enhances the checkSyntax plugin for the @modern-js/builder package. It adds a type definition, a modern operator, and a test case for the plugin, and updates the CheckSyntaxOptions interface and the changeset file. It also improves the type safety and readability of the plugin code.
Details

🤖 Generated by Copilot at 42084a4

Add changeset file to document patch updates and fix for checkSyntax plugin (link)
Refine type of options parameter for CheckSyntaxPlugin constructor (link)
Make targets property optional in CheckSyntaxOptions interface (link)
Import CheckSyntaxOptions type and refactor getCheckTargets function in checkSyntax.ts (link, link)
Add test case to verify default browserlist usage when only exclude property is set in checkSyntax.test.ts (link)

Related Issue

Checklist

[ ] I have added changeset via pnpm run change.
[ ] I have updated the documentation.
[ ] I have added tests to cover my changes.

🐂🐂


[FEATURE REQUEST] Visibility toggle for tabs group

In order to avoid having a lot of tabs opened it would be great if the extension could have a toggle button to open and close all the tabs in it.
The button behavior should be like the one on vs-tab-groups extension, and only open/close the tabs of a certain group. That means leaving any other previously opened tabs untouched.

One can achieve a similar result using the restore and then close-all button (implemented in Release 1.1.13):


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 70 ms

Resolved: ClamAV performance has improved in 8931c98.


No visual indicator when a find toolbar checkbox or zoom dropdown are focused in Chrome

When using Chrome, there is no visual indicator when you tab over to the "Highlight All" and "Match Case" checkboxes within the find toolbar.
Steps to reproduce:

Open Chrome
Go to the PDF.js demo PDF viewer
Click on the Find button
Use the tab keys to navigate over to either the "Highlight All" checkbox, the "Match Case" checkbox, or the zoom dropdown
Note that there is no visual indicator that any of these elements are focused

In Firefox and Internet Explorer 11, the user can clearly see that these elements are focused because the label has a highlight around it. Note that the element is still able to be activated by using the spacebar (there is a separate issue regarding this behavior not working in Firefox or Internet Explorer 11), but there is no visual focus to indicate that the element has focus.

Is this not an upstream browser issue then, if only Chrome has the problem, or do you think that this can somehow be caused by the PDF.js codebase?

The issue seems to be an issue with viewer.css. When you look at a CSS-less dropdown menu or [checkbox[(http://www.w3schools.com/jsref/tryit.asp?filename=tryjsref_checkbox_get), there is highlighting in Chrome. In fact, if you comment out the line to import the viewer.css, you can focus those elements.

@Thunderforge is correct that it is an issue with viewer.css
However this issue of no focus states is present in more than just chrome as I have reproduced in:
Safari (9.1.1 11601.6.17)
Edge (25.10586.0.0)
IE11 (11.420.10586.0)
FF Win (47.0) - does have a 1px outline focus but is not as visible as FF Mac blue glow
FF 47.0 (Ubuntu 16.04)
Note these are all the latest versions of the browsers as of 22 Jun 2016 NZST

This should have been (indirectly) fixed by the re-styling in PR #11077.


closes #71

I don't have windows box to test this on, but the commit seems innocent and sensible to me ...

It has been confirmed to work by @oaramsda (https://gitter.im/kodeklubben/oppgaver)


pythonPackages.tensorflow: update sha256

Motivation for this change
It seems the sha256 has changed.
Things done

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

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Have you looked into why? The checksums are supposed to protect against tampering, but that is negated if we blindly update the checksum whenever it changes.

see 633387c0e8ae274767b8beb98648dde84b412477 the hash wasn't actually updated cc @jhhuh


📖 [GEP-23] Autoscaling Shoot kube-apiserver via Independently Driven HPA and VPA

How to categorize this PR?
/area auto-scaling
/kind enhancement
What this PR does / why we need it:
This GEP proposes a simple design which overcomes existing HVPA deficiencies, reduces maintenance effort by reducing Gardener's custom code base, and enables subsequent reduction in shoot kube-apiserver compute cost.
Special notes for your reviewer:
Release note:
NONE

Without having looked into the content: Can we try to find a better name, please? HPlusVAutoscaler, often shortened as HPVA is not much different from the existing HVPA...

Oliver, just to confirm:
By https://github.com/gardener/gardener/pull/7410#discussion_r1098685488, you are referring to Istvan's question about driving HPA on memory requests, and not to Alternative#1 described in the GEP, correct?
@oliver-goetz

Without having looked into the content: Can we try to find a better name, please? HPlusVAutoscaler, often shortened as HPVA is not much different from the existing HVPA...

Any update?

Without having looked into the content: Can we try to find a better name, please? HPlusVAutoscaler, often shortened as HPVA is not much different from the existing HVPA...

Any update?

Sure. I intentionally postponed responding until I had finished the first big round of updates, so I don't draw your attention back to the PR while it's in turbulent state.
The HPlusVAutoscaler wasn't an easy choice. The abbreviation similarity to HPVA-HVPA was immediately apparent. In the end, I had to grudgingly settle for it. I'm very much open to ideas for a better name, but personally I failed to come up with one. Below are the factors which contributed to the current choice:

HVPA and the HPlusVAutoscaler will coexist only briefly. The HPlusVAutoscaler is a direct replacement for HVPA, so their coexistence is only a transitional necessity.
Currently, an abbreviation for HPlusVAutoscaler is used in only two places: one package g/g source tree, and the names of the HPA/VPA resources deployed to the shoot control plane, which follow the "kube-apiserver-hpva" format.
It is more important to have an intuitive, descriptive name for the HPlusVAutoscaler over the long period of its exist existence, than to avoid the brief similarity in abbreviations.
At least for me, "H plus V" is very close to what naturally comes to the tongue whey I try to describe the thing. In my case, those words are "It's just a VPA, plus HPA acting on request rate".

The bottom line: HPlusVAutoscaler is just the best candidate so far. Ideas are welcome.

Let's avoid similarities to HVPA, even if it's just for "briefly". What about AutoscalingViaCustomMetrics or HorizontalAutoscalingViaCustomMetrics if you want to be more precise?

It's not just a name of the mode of autoscaling, but also for structural elements (e.g. package). Preferably should be in the form ...Autoscaler. Also for consistency, including in the abbreviation - HPA, VPA, HVPA, MDA, cluster autoscaler.
So, AutoscalingViaCustomMetrics becomes something like CustomMetricsAutoscaler (CustomMetricsPodAutoscaler may be better). Here's what I don't like about it:

The part "custom metrics" is an implementation detail. Not that it's likely to change, but I tried to stay as close as possible to naming based on functional purpose. HPlusV can be thought of as a strictly functional description (where e.g. H does not mean HPA but horizontal).
The "custom metrics" is likely to become an incomplete description in the future. Not a big deal, but worth mentioning. In a future step, I have an idea of supplementing it with a safety net made of OOMKill events (for memory exhaustion) and HTTP 429s (too many requests - for CPU exhaustion). Both are already available in prometheus and could save us when someone tries to do 600 "list cluster config maps" per second, a case where RequestsPerSecond won't trigger horizontal scale-out.

@rfranzke

CustomMetricsAutoscaler sounds like a dedicated autoscaler component/pod, but anyways, I could live with that. For sure, any similarity to HVPA should be avoided. HPlusV is simply too close. In contrast to HVPA, we should also strive for a reasonable timeline regarding promotion and removal of this feature gate. It should not stay longer as needed (like HVPA - we could have removed this months/years ago), and this new way of autoscaling should become the default. Hence, we don't need to find a perfect name for this feature gate now - just something that is reasonably simple and doesn't yield to even more confusion in the current code base.

we don't need to find a perfect name for this feature gate now

Wait, when speaking about finding a better name, were you referring strictly to the user-facing element - the feature gate? Not about the name of internal elements (golang package and the two resources in the control plane namespace)? If so, my objections above do not apply. For a feature gate, the direction taken by e.g. AutoscalingViaCustomMetrics absolutely makes sense to me.

I am referring to both, actually. The references in the implementation are probably even more important given the current "not-easy-to-digest" situation with the multiple autoscaling modes.

@andrerun @voelzmo you can also add me as a main reviewer for the implementation of this GEP :)

/remove-lifecycle stale
/lifecycle frozen

/remove-lifecycle stale

Given that https://github.com/gardener/gardener/pull/8012 has been approved, the GEP number should be bumped to '23' in this PR.

/remove-lifecycle stale

gardener-apiserver also has an HVPA (ref https://github.com/gardener/gardener/blob/master/charts/gardener/controlplane/charts/runtime/templates/apiserver/hvpa.yaml) - does this GEP also include it? It's also an API server built with k8s.io/apiserver, just like kube-apiserver, so probably the same principles apply? In this case, we need to deploy this new component also into the garden cluster via gardener-operator.

The document is ready for another review round.
Among other things, a list was added of quantities to be used when evaluating the fitness of the new autoscaling solution. The list was a non-trivial compromise between what we'd like to measure, and what we can measure with sufficient accuracy/feasibility.

/remove-lgtm
Please delete docs/proposals/assets/gep23-hvpa-scaling-space.png file. It is not used anymore in the document.
Otherwise lgtm 😄

/lgtm

/retest

Does anyone else plan to take a look? If so please comment here so that we can wait for the review. Otherwise, let's merge this PR by Monday.

So I guess we're good to go here? Let's uncancel the hold and get this in!

Yes, let's merge the GEP :)
/hold cancel


Reject other QASM versions in QASM 2 parser

Summary
Fixing QASM-319, qasm parser will fail with appropriate error if invalid OPENQASM version is specified. Reno release note is added.
Details and comments
Small change in qasm lexer to only accept 2.0 as a version, rather than any digit.digit. The latter caused a bug where user could provide any version, like 3.0 and believe it works while in reality their code was still parsed as 2.0.
Should not break any workflows, unless they rely on broken openqasm strings. Added reno release note. Did not see any place in the documentation per se that would require updating.

Its effectively blocked by https://github.com/Qiskit/qiskit-terra/issues/7457

Its effectively blocked by https://github.com/Qiskit/qiskit-terra/issues/7457

Makes sense, thanks! Will try to look late today or early tomorrow

@jakelishman any chance for some quick eyes here?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.005%) to 83.112%

Files with Coverage Reduction
New Missed Lines
%

qiskit/extensions/quantum_initializer/uc.py
2
88.65%

qiskit/pulse/library/waveform.py
3
89.36%

Totals

Change from base Build<PHONE_NUMBER>:
-0.005%

Covered Lines:
51941

Relevant Lines:
62495

💛  - Coveralls


Apple IOS events

Drawstop and drawpoint events are not triggered in ios.

Unfortunately I dont have any mac device to test my code on. Feel free to create a PR to fix this issue!

I do not have it either. I'm testing on appetize dot io


v0.2.1/macOS: setup fails to download files, no output, completes with "everything fine" but no toolchain installed.

"Error downloading zephyr-tools-0.1.7-aarch64-apple-darwin.tar.gz. Check output for more info."
"Error downloading zephyr-sdk-0.16.4_macos-aarch64_minimal.tar.xz. Check output for more info."
~/.zephyrtools:
~/.zephyrtools> ls -la
total 0
drwxr-xr-x    6 mikaelj  staff   192  5 Mar 14:02 .
drwxr-xr-x@ 102 mikaelj  staff  3264  5 Mar 13:57 ..
drwxr-xr-x    3 mikaelj  staff    96  5 Mar 13:58 cmake
drwxr-xr-x    6 mikaelj  staff   192  5 Mar 13:58 downloads
drwxr-xr-x    6 mikaelj  staff   192  5 Mar 13:58 env
drwxr-xr-x    3 mikaelj  staff    96  5 Mar 13:58 newtmgr
 ~/.zephyrtools> ls -lah downloads/
total 184992
drwxr-xr-x  6 mikaelj  staff   192B  5 Mar 13:58 .
drwxr-xr-x  6 mikaelj  staff   192B  5 Mar 14:02 ..
-rw-r--r--  1 mikaelj  staff    67M  5 Mar 13:58 cmake-3.22.0-macos-universal.tar.gz
-rw-r--r--  1 mikaelj  staff   8,5M  5 Mar 13:58 newtmgr.zip
-rw-r--r--  1 mikaelj  staff   4,9K  5 Mar 13:57 zephyr-sdk-0.16.4_macos-aarch64_minimal.tar.xz
-rw-r--r--  1 mikaelj  staff   295K  5 Mar 13:57 zephyr-tools-0.1.7-aarch64-apple-darwin.tar.gz
Pellefant [14:02] ~/.zephyrtools>

As you can see, no ~/.zephyrtools/toolchain directory.
VSCode Output Tab:
[SETUP] Installing zephyr-sdk-0.16.4 (latest) toolchain...
[SETUP] downloading https://github.com/zephyrproject-rtos/sdk-ng/releases/download/v0.16.4/zephyr-sdk-0.16.4_macos-aarch64_minimal.tar.xz
[SETUP] downloading https://github.com/circuitdojo/zephyr-tools-cli/releases/download/0.1.7/zephyr-tools-0.1.7-aarch64-apple-darwin.tar.gz
[SETUP] downloading https://docs.jaredwolff.com/files/newtmgr/darwin/newtmgr.zip
[SETUP] unzip /Users/mikaelj/.zephyrtools/downloads/newtmgr.zip to /Users/mikaelj/.zephyrtools/newtmgr
[SETUP] downloading https://github.com/Kitware/CMake/releases/download/v3.22.0/cmake-3.22.0-macos-universal.tar.gz
tar -xvf "/Users/mikaelj/.zephyrtools/downloads/cmake-3.22.0-macos-universal.tar.gz" -C "/Users/mikaelj/.zephyrtools/cmake"
git version 2.42.0
[SETUP] git installed
python3 --version
[SETUP] python3 found
python3 -m pip --version
pip 24.0 from /Users/mikaelj/Library/Python/3.9/lib/python/site-packages/pip (python 3.9)
[SETUP] pip installed
python3 -m venv --help
[SETUP] python3 venv OK
python3 -m venv "/Users/mikaelj/.zephyrtools/env"
[SETUP] virtual python environment created
Collecting west
  Using cached west-1.2.0-py3-none-any.whl (95 kB)
Requirement already satisfied: setuptools in /Users/mikaelj/.zephyrtools/env/lib/python3.9/site-packages (from west) (58.0.4)
Collecting pykwalify
  Using cached pykwalify-1.8.0-py2.py3-none-any.whl (24 kB)
Collecting packaging
  Using cached packaging-23.2-py3-none-any.whl (53 kB)
Collecting colorama
  Using cached colorama-0.4.6-py2.py3-none-any.whl (25 kB)
Collecting PyYAML>=5.1
  Using cached PyYAML-6.0.1-cp39-cp39-macosx_11_0_arm64.whl (174 kB)
Collecting ruamel.yaml>=0.16.0
  Using cached ruamel.yaml-0.18.6-py3-none-any.whl (117 kB)
Collecting python-dateutil>=2.8.0
  Using cached python_dateutil-2.9.0.post0-py2.py3-none-any.whl (229 kB)
Collecting docopt>=0.6.2
  Using cached docopt-0.6.2.tar.gz (25 kB)
Collecting six>=1.5
  Using cached six-1.16.0-py2.py3-none-any.whl (11 kB)
Collecting ruamel.yaml.clib>=0.2.7
  Using cached ruamel.yaml.clib-0.2.8-cp39-cp39-macosx_12_0_arm64.whl (134 kB)
Using legacy 'setup.py install' for docopt, since package 'wheel' is not installed.
Installing collected packages: six, ruamel.yaml.clib, ruamel.yaml, python-dateutil, docopt, PyYAML, pykwalify, packaging, colorama, west
    Running setup.py install for docopt: started
    Running setup.py install for docopt: finished with status 'done'
Successfully installed PyYAML-6.0.1 colorama-0.4.6 docopt-0.6.2 packaging-23.2 pykwalify-1.8.0 python-dateutil-2.9.0.post0 ruamel.yaml-0.18.6 ruamel.yaml.clib-0.2.8 six-1.16.0 west-1.2.0
[SETUP] west installed
[SETUP] Zephyr setup complete!

Doesn't give any specific info.

@mikaelj pushed version 0.2.2 which should address this. Thanks for reporting!


GH Actions CI spamming commits

Encouraging the use of conventional commits makes sense but I'm concerned pinging each commit someone pushes up against a branch is a little too aggressive :)

https://github.com/open-telemetry/opentelemetry-ruby/pull/446#issuecomment-715890873
Hey @dazuma, you are our resident CI expert is this something you could help with?

Hmm. Yeah I'll see if I can update the script not to add another comment if one has been added previously.


Error lunch web - not implemented on web

Describe the bug
It says that it is compatible for all three platforms, but I don't understand how I can make it work on the web, I have tried downloading the project itself and launching it but there is no way, it gives me this error:
core.mjs:6484
ERROR Error: Uncaught (in promise): Error: Not implemented on web.
Error: Not implemented on web.
    at BarcodeScannerWeb.unimplemented (index.js:376:16)
    at web.js:7:20
    at Generator.next (<anonymous>)
    at asyncGeneratorStep (asyncToGenerator.js:3:1)
    at _next (asyncToGenerator.js:25:1)
    at asyncToGenerator.js:32:1
    at new ZoneAwarePromise (zone.js:1349:1)
    at asyncToGenerator.js:21:1
    at BarcodeScannerWeb.hideBackground (web.js:6:28)
    at index.js:196:35
    at resolvePromise (zone.js:1262:1)
    at resolvePromise (zone.js:1216:1)
    at zone.js:1329:1
    at _ZoneDelegate.push.3279._ZoneDelegate.invokeTask (zone.js:443:1)
    at Object.onInvokeTask (core.mjs:25463:33)
    at _ZoneDelegate.push.3279._ZoneDelegate.invokeTask (zone.js:442:1)
    at Zone.push.3279.Zone.runTask (zone.js:214:1)
    at drainMicroTaskQueue (zone.js:632:1)

To Reproduce
Steps to reproduce the behavior:

Go to BarCodeScanner
Click on Scan
See error

Expected behavior
I hope that clicking the Scan button will open the camera on the desktop or laptop screen.
Screenshots
App:

Package.json:

Implementation method:

Version
v2.1.0
Desktop (please complete the following information):

OS: macOS Monterey Versión 12.3 (21E230)
Browser Chrome Versión 99.0.4844.84 (Build oficial) (x86_64)

Unfortunately no support for web has been added yet. Closing as a duplicate of #31

@tafelnl Thank you very much for the clarification. Could you point me to a scanner plugin for BarCodes, QR, etc. that has support for Web, Android and iOS?


test/utils/analytics_spec: fix flaky test

A couple of them check the tags, which are cached.

Review period skipped due to critical label.


Provisioning Out-of-Band OAuth Clients page needs to be updated

Description:
Rukshan reported as follows:
Provisioning Out-of-Band OAuth Clients page [1] needs to be updated.

New config is
[apim.devportal] enable_key_provisioning = true
UIs need to be updated
Store needs to be renamed to Developer portal

[1] https://apim.docs.wso2.com/en/latest/Learn/APISecurity/OAuth2DeepDive/provisioning-out-of-band-oauth-clients/
Suggested Labels:
Suggested Assignees:
Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

Reviewed and updated -

https://apim.docs.wso2.com/en/latest/learn/api-security/oauth2/provisioning-out-of-band-oauth-clients/
https://github.com/wso2/docs-apim/pull/802
https://github.com/wso2/docs-apim/pull/802/commits/0a9116780aab09f3d4d053f7ea1f016e04bf37ff


chore(dep): Bump gateway api to the last version

Please refer to individual  commit for more details.

/ci-gateway-api

/ci-gateway-api

/test

/test


[#103483120] Adds screenshot tutorial

uses glReadPixel, freeing the need for preserveDrawingBuffer performance impacts.

Hitting a wall on #399
because the glTrace show that we apply a default "gl.viewport(800,600)" breaking glReadPixel read.

it' just about adding a camera lastDraw callback here:
https://github.com/cedricpinson/osgjs/blob/develop/sources/osg/RenderStage.js#L129
as done in OSG, see here
https://github.com/openscenegraph/osg/blob/f1a0ff43c6bd47387a480de89f35fff2d0ed7a95/src/osgUtil/RenderStage.cpp#L1316

@Kuranes yes you can implement it using the same method as in osg