Controls don't work for LegacyRpiCam

v4l2 controls don't take effect on rpicamsrc streams.

Some of the Legacy Raspberry Pi camera controls are not supported yet:
ERROR 14:36:39.406 src/video/video_source_local.rs:380: Failed to get control 'Horizontal Flip (9963796)' from device /dev/video0: Custom {                                                 [10/9515]
    kind: Other,
    error: "Control type is not supported: Boolean(\n    false,\n)",
}
ERROR 14:36:39.406 src/video/video_source_local.rs:380: Failed to get control 'Vertical Flip (9963797)' from device /dev/video0: Custom {
    kind: Other,
    error: "Control type is not supported: Boolean(\n    false,\n)",
}
ERROR 14:36:39.407 src/video/video_source_local.rs:380: Failed to get control 'Power Line Frequency (9963800)' from device /dev/video0: Custom {
    kind: Other,
    error: "cannot handle control type",
}
ERROR 14:36:39.407 src/video/video_source_local.rs:380: Failed to get control 'Color Effects (9963807)' from device /dev/video0: Custom {
    kind: Other,
    error: "cannot handle control type",
}
ERROR 14:36:39.407 src/video/video_source_local.rs:380: Failed to get control 'Video Bitrate Mode (10029518)' from device /dev/video0: Custom {
    kind: Other,
    error: "cannot handle control type",
}
ERROR 14:36:39.407 src/video/video_source_local.rs:380: Failed to get control 'Repeat Sequence Header (10029538)' from device /dev/video0: Custom {
    kind: Other,
    error: "Control type is not supported: Boolean(\n    false,\n)",
}
ERROR 14:36:39.408 src/video/video_source_local.rs:380: Failed to get control 'Force Key Frame (10029541)' from device /dev/video0: Os {
    code: 13,
    kind: PermissionDenied,
    message: "Permission denied",
}
ERROR 14:36:39.408 src/video/video_source_local.rs:380: Failed to get control 'H264 Level (10029671)' from device /dev/video0: Custom {
    kind: Other,
    error: "cannot handle control type",
}
ERROR 14:36:39.408 src/video/video_source_local.rs:380: Failed to get control 'H264 Profile (10029675)' from device /dev/video0: Custom {
    kind: Other,
    error: "cannot handle control type",
}
ERROR 14:36:39.408 src/video/video_source_local.rs:380: Failed to get control 'Auto Exposure (10094849)' from device /dev/video0: Custom {
    kind: Other,
    error: "cannot handle control type",
}
ERROR 14:36:39.409 src/video/video_source_local.rs:380: Failed to get control 'Exposure, Dynamic Framerate (10094851)' from device /dev/video0: Custom {
    kind: Other,
    error: "Control type is not supported: Boolean(\n    false,\n)",
}
ERROR 14:36:39.409 src/video/video_source_local.rs:380: Failed to get control 'Auto Exposure, Bias (10094867)' from device /dev/video0: Custom {
    kind: Other,
    error: "cannot handle control type",
}
ERROR 14:36:39.409 src/video/video_source_local.rs:380: Failed to get control 'White Balance, Auto & Preset (10094868)' from device /dev/video0: Custom {
    kind: Other,
    error: "cannot handle control type",
}
ERROR 14:36:39.409 src/video/video_source_local.rs:380: Failed to get control 'Image Stabilization (10094870)' from device /dev/video0: Custom {
    kind: Other,
    error: "Control type is not supported: Boolean(\n    false,\n)",
}
ERROR 14:36:39.409 src/video/video_source_local.rs:380: Failed to get control 'ISO Sensitivity (10094871)' from device /dev/video0: Custom {
    kind: Other,
    error: "cannot handle control type",
}
ERROR 14:36:39.410 src/video/video_source_local.rs:380: Failed to get control 'ISO Sensitivity, Auto (10094872)' from device /dev/video0: Custom {
    kind: Other,
    error: "cannot handle control type",
}
ERROR 14:36:39.410 src/video/video_source_local.rs:380: Failed to get control 'Exposure, Metering Mode (10094873)' from device /dev/video0: Custom {
    kind: Other,
    error: "cannot handle control type",
}
ERROR 14:36:39.410 src/video/video_source_local.rs:380: Failed to get control 'Scene Mode (10094874)' from device /dev/video0: Custom {
    kind: Other,
    error: "cannot handle control type",
}

@joaoantoniocardoso some or all ?

Some, and almost certaintly it is a bug caused by #102. I'll confirm it, but we might revert that for now.


Consider introducing greenkeeper.io & snyk

The library currently has 17 devDependencies.
Greenkeeper.io would allow or easier version bumping as it automates the process when new versions are added to npm.
Snyk does vulnerability checking, by introducing this you can check for dependencies that could be hazardous at the version implemented.

I've just started investigating this. Vulnerability scanning is handled by Github and @AlexLakatos and I are the recipients of any vulns.
Greenkeeper looks nice, but there are some concerns around giving it write access to the repo so we're going to hold off now and stick with running npm outdated periodically. Stability is our main focus so as long as the library has no security issues, we'll likely only update when required for new features.


Notification Reminder Bug

Describe the bug
There is an issue with the notification system in version v0.22.2. When the user admin posts a memo, user1 comments on it, and then admin replies to user1's comment, the notification received by user1 leads to a blank page upon clicking.
Steps to reproduce

Log in as admin.
Post a memo.
Log in as user1.
Comment on the memo posted by admin.
Log in as admin.
Reply to the comment made by user1.
Log in as user1.
Click on the notification received from admin's reply.

The version of Memos you're using.
v0.22.2
Screenshots or additional context

Might be fixed.

Might be fixed.

fixed now ? Still have issue in child memos' comment notification


Fixes for Manual Sync in response to MMS4 performance issues

MDK-72 and MDK-73

@ivan-gomes please re-review


Add Smacker Video detection

Some reference to the SDK, such as the developer site or a wikipedia article.
Smacker Video: http://www.radgametools.com/smkmain.htm
SteamDB links
https://steamdb.info/app/215930
https://steamdb.info/app/549170
https://steamdb.info/app/410980
Possible way to detect it
Contain any of the file: smackw32.dll

Proposal to merge into one group RAD, as Nvidia..
RAD.Bink_Video, RAD.Smacker_Video, RAD.Oodle, RAD.Miles, RAD.Granny

i think nvidia's naming is like that because of the official name of the technology

RAD's technology is just Granny or Miles Sound System, but not RAD Miles Sound System i think

otherwise Oodle and Bink would get named like that already

RAD, or Epic Games Tools (formerly RAD Game Tools), is an Epic Games company based in Bellevue, Washington.

http://www.radgametools.com/aboutrad.htm

but if you go on the bink page it just says bink

and if you go on the nvidia dlss page it says nvidia dlss

As you see fit. At your own discretion
I'm more interested in this (?:^|/)bink2?w(?:64|32)?.dll$
there is also binkw32.dll and binkw64.dll

i can try making an update for it after i make a pr for this smacker video thing

As you see fit. At your own discretion
I am not the one deciding things though, im just a contributor who makes regexes in my free time, so if xPaw of Collaborators choose to name it "RAD Technology" it'll be their choice

sorry, i'll probs go to bed now, but you can make more issues overnight and me/other people will make regexes for them

actually, according to tests file binkw files are already getting checked, and if bink tech is on website regex probably works

also, you are Ukrainian according to your github page, so if you want to discuss something in Ukrainian you can dm me on discord (diedyesterday#5745), since im pretty sure english is the only language available here, and using Ukrainian will be easier for both of us


Opening ports for Security Group based locations

Where config keys define the explicit ports, these will be opened automatically. It's worth referencing other parts of the docs for this - see https://github.com/apache/brooklyn-docs/blob/master/guide/yaml/custom-entities.md#port-inferencing
In fact, do you think these new docs should be added to that section? Or that section extracted to its own page? I'm not sure, as this new page does talk specifically about configuring the location (though maybe it shouldn't do that!).

I think having separated docs for customising locations is fine. But I would definitely add the port-opened explicitly if we add configkeys, as mentioned by @aledsage.
Otherwise, LGTM

@bostko good stuff here, are you able to incorporate the comments ?

Hi, I rebased PR and addressed Aleds' comments for SharedLocationSecurityGroupCustomizer.
@aledsage , regarding the section talking about configuring the location, if using provisioning.properties
as you suggested then makes it location non specific.
My driver for splitting into two sections: location customizers is to have better visibility about what both do and document easily classes like SudoTtyFixingCustomizer, AbstractEbsVolumeCustomizer in jclouds location customizers section.


[0.8.9] Spool history - Put file name in Jobinfo column.

That's already there - or do you refer to the complete filename including path?

And sometimes the name is a bit wishywashy since you loaded it and changed stuff in there. Which is why the name is only the filename sometimes but usually Spooler: x items

I would like to propose the following to make things less wishy-washy.

There needs to be a "current" filename based on the following rules:

If the last load/save action was a save, then the save filename.
If the last action was a load and there is one file node and no other children of the elements super-node, the file in the one file node.
If the last action was a load and rule 2 does not apply - the filename should be empty.

The "current" filename is the filename used for Save - and Save As would be shown if the "current" filename is empty.
The "current" filename is what is used for the spooler - if the filename is empty then you do as shown above. I am not sure that showing a full path is necessary, but if it is then it should probably be a preference.
We need to indicate whether there have been any changes to the design i.e. that a save would be different from the load - perhaps by prepending an asterisk to the menu bar title like Inkscape does. So any design or operation changes (including creating Ops or using existing Ops when loading a file which doesn't have saved Ops) would indicate a change. This state should be put on the Undo record - so if you save, then make a change and then undo it, you get the save-needed indicator cleared (just as Inkscape does).

These changes would make the save functionality a bit more intuitive - it would distinguish between MK saved SVG files (which have Ops) and those that don't and would be more intuitive about when Save As is shown if you do a Save (i.e. if I Save As then remove the file node and import another, then if I hit Save I think it will currently overwrite the previous save even though the file contents are entirely different).

Working file name exists, this is common for save. I think it displays in taskbar that's what's believed to be the current file. But, you can load multiple files to build a single project.

This is somewhat an issue, sometimes jog commands etc are sent to the spooler and those aren't actually part of the filename as such.  Saying that a jog command was that file would be wrong.

This seems plenty doable, I think we have a flag for changed data or whatever. Though I think suggesting this would make it all more intuitive seems to be overselling that.

If we wanted to better distinguish MK and SVG files we could probably just make a different extension, part of the issue there is while meerk40t is adding more and more info there, it's still at the core a proper SVG.


ES: fix get_metric when metric does not exist

Coverage decreased (-0.07%) to 77.731% when pulling 6ad9107353b2ebd9641266e9bdb6e9afc9d03fc2 on adericbourg:bugfixes into 3ae3a415ede82c8278792e0d1926f9ec06ba6a1f on criteo:master.


Migrate from openjdk to eclipse-temurin

What and why?
The openjdk base images are deprecated and no longer maintained. This migrates to the recommended eclipse-temurin image. This also migrates from the slim Linux OS to the alpine OS images as eclipse-temurin doesn't appear to offer slim. This reduces the image from ~305MB to ~143MB
How to test
Does the image build, does the application start, and do the acceptance tests pass?

/deploy pricem

/deploy robinm3


[BUG/Question] Error calling openid_config.load_config with Httpx 0.25.1

Describe the bug
When using httpx 0.25.1 an exception is thrown when calling openid_config.load_config() from the B2CMultiTenantAuthorizationCodeBearer. The exception thrown is AsyncConnectionPool.__init__() got an unexpected keyword argument 'socket_options'
The bug can be resolved by downgrading httpx to 0.24.1. The same error is described in the stable diffusion webui project which can be found here. I am just including this in case their fix is applicable.
To Reproduce

Setup a brand new FastAPI project in a new python environment - following their documentation.
Install pydantic-settings pip install pydantic-settings
Check the correct dependencies are installed - fastapi==0.104.1, pydantic==2.5.1, pydantic-settings==2.1.0, fastapi-azure-auth==4.3.0, httpx==0.25.1
If httpx is a different version pip install -U httpx (0.25.1 is the latest version at this time)
Follow your project documentation for B2C setup, everything is successful until we arrive to this section fastaspi-azure-auth
Add the event startup code and run the application
Application will not start with the error described - The exception thrown is AsyncConnectionPool.__init__() got an unexpected keyword argument 'socket_options

Your configuration
Our settings setup is as exactly described in your tutorial / documentation, we have just renamed some variables to be a little clearer for our project. I have confirmed that the openid_config_url loads as expected in the browser.
In main.py
azure_scheme = B2CMultiTenantAuthorizationCodeBearer(
    app_client_id=Envs.AZUREB2C_CLIENTID,
    openid_config_url=Envs.OPENID_CONFIG_URL,
    openapi_authorization_url=Envs.OPENAPI_AUTHORIZATION_URL,
    openapi_token_url=Envs.OPENAPI_TOKEN_URL,
    scopes=Envs.SCOPES,
    validate_iss=False,
)

@app.on_event("startup")
async def load_config() -> None:
    """
    Load OpenID config on startup.
    """
    try:
        await azure_scheme.openid_config.load_config()
    except Exception as ex:
        print(str(ex))

Hi there,
I cannot yet reproduce after following your steps, with httpx==0.25.1. Can you add some logging as per here?
here's the log output when I tried to reproduce your example.
python main.py

main.py:103: DeprecationWarning:
        on_event is deprecated, use lifespan event handlers instead.

        Read more about it in the
        [FastAPI docs for Lifespan Events](https://fastapi.tiangolo.com/advanced/events/).

  @app.on_event('startup')

INFO:     Will watch for changes in these directories: ['/private/tmp']
INFO:     Uvicorn running on http://<IP_ADDRESS>:8000 (Press CTRL+C to quit)
INFO:     Started reloader process [1762] using StatReload
INFO:     Started server process [1764]
INFO:     Waiting for application startup.
DEBUG 2023-11-16 14:18:03,603 fastapi_azure_auth Loading Azure AD OpenID configuration.
INFO 2023-11-16 14:18:03,619 fastapi_azure_auth Fetching OpenID Connect config from https://mytenant.b2clogin.com/mytenant.onmicrosoft.com/B2C_1_signupsignin/v2.0/.well-known/openid-configuration
INFO 2023-11-16 14:18:04,035 fastapi_azure_auth Fetching jwks from https://mytenant.b2clogin.com/mytenant.onmicrosoft.com/b2c_1_mypolicy/discovery/v2.0/keys
DEBUG 2023-11-16 14:18:04,159 fastapi_azure_auth Loading public key from certificate: {'kid': 'xxx', 'nbf':<PHONE_NUMBER>, 'use': 'sig', 'kty': 'RSA', 'e': 'AQAB', 'n': 'xxx'}
INFO 2023-11-16 14:18:04,161 fastapi_azure_auth fastapi-azure-auth loaded settings from Azure AD.
INFO 2023-11-16 14:18:04,161 fastapi_azure_auth authorization endpoint: https://mytenant.b2clogin.com/mytenant.onmicrosoft.com/b2c_1_signupsignin/oauth2/v2.0/authorize
INFO 2023-11-16 14:18:04,161 fastapi_azure_auth token endpoint:         https://mytenant.b2clogin.com/mytenant.onmicrosoft.com/b2c_1_signupsignin/oauth2/v2.0/token
INFO 2023-11-16 14:18:04,162 fastapi_azure_auth issuer:                 https://mytenant.b2clogin.com/xxx/v2.0/
INFO:     Application startup complete.

You could also try to use Lifespan events for loading the config (as the startup events are officially deprecated in FastAPI now).

Hi there, thank you for the very fast response - and for the great work on the library.
Here is the output from the logging config setup you linked to when starting up using httpx 0.25.1. I haven't tried with lifespan events yet. Note the application startup complete message is because the startup code call to load_config is wrapped in a try except
INFO:     Uvicorn running on http://<IP_ADDRESS>:8000 (Press CTRL+C to quit)
INFO:     Started reloader process [67347] using WatchFiles
INFO:     Started server process [67351]
INFO:     Waiting for application startup.
DEBUG 2023-11-16 15:19:24,014 fastapi_azure_auth Loading Azure AD OpenID configuration.
ERROR 2023-11-16 15:19:24,023 fastapi_azure_auth Unable to fetch OpenID configuration from Azure AD. Error: AsyncConnectionPool.__init__() got an unexpected keyword argument 'socket_options'
Traceback (most recent call last):
  File "/Users/myuserdir/miniconda3/envs/testauthenv/lib/python3.11/site-packages/fastapi_azure_auth/openid_config.py", line 42, in load_config
    await self._load_openid_config()
  File "/Users/myuserdir/miniconda3/envs/testauthenv/lib/python3.11/site-packages/fastapi_azure_auth/openid_config.py", line 77, in _load_openid_config
    async with AsyncClient(timeout=10) as client:
               ^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/myuserdir/miniconda3/envs/testauthenv/lib/python3.11/site-packages/httpx/_client.py", line 1397, in __init__
    self._transport = self._init_transport(
                      ^^^^^^^^^^^^^^^^^^^^^
  File "/Users/myuserdir/miniconda3/envs/testauthenv/lib/python3.11/site-packages/httpx/_client.py", line 1445, in _init_transport
    return AsyncHTTPTransport(
           ^^^^^^^^^^^^^^^^^^^
  File "/Users/kevinpye/miniconda3/envs/chatbot_api/lib/python3.11/site-packages/httpx/_transports/default.py", line 275, in __init__
    self._pool = httpcore.AsyncConnectionPool(
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: AsyncConnectionPool.__init__() got an unexpected keyword argument 'socket_options'
INFO:     Application startup complete.

Hi again,
I have been doing some further investigation. I updated to lifecycle events but the issue still persisted. After some further debugging It seems like there was an issue on my machine. For some reason, an old version of httpcore - used by httpx was being installed. Once I updated httpcore to the latest version, I could use httpx 0.25.1 and this issue was resolved. I have updated my requirements txt to ensure the correct versions are installed.
You can close this issue, thank you for your help and again thank you for the work on this package and providing clear documentation.

Glad you resolved it, and thanks @davidhuser for helping out 😊
I’ll close this issue.
As a side note, we should fix startup events in favor as lifespan in the docs, though. Totally forgot about that.


Restore Zookeeper backups on start

@Randgalt Have you connected already with Netflix to take ownership over Exhibitor? If there is any results?

It's taking them forever. I'll keep pinging them about it. But, it will happen eventually.

@Randgalt - do you what the hold up is their end?

You didn't see the note? It's now out of Netflix. https://github.com/soabase/exhibitor - we're trying to get a community started. I have 2 people who've volunteered to be committers. We're looking for more. See here: https://groups.google.com/forum/#!topic/exhibitor-users/PVkcd88mk8c

@Randgalt what process do we have right now?

Closing and re-opening to trigger Travis.

Merged this in #350.


version number isn't displayed in UI in latest version

This has now been fixed with latest release.

This has now been fixed with latest release.


OTHER_LDFLAGS wrongly split

http_archive(
    name = "rules_pods",
    sha256 = "db4256e80fda350bff752f70385f3a332ac8b93d63c3ef677b11dca075dbe1ff",
    url = "https://github.com/pinterest/PodToBUILD/releases/download/0.25.2-8a5efa0/PodToBUILD.zip",
)
load("@rules_pods//BazelExtensions:workspace.bzl", "new_pod_repository")
new_pod_repository(
    name = "LibTorch",
    # More versions at https://github.com/CocoaPods/Specs/tree/master/Specs/1/3/c/LibTorch
    podspec_url = "https://raw.githubusercontent.com/CocoaPods/Specs/075e9679993ae4798982fe55acb95f99486cc963/Specs/1/3/c/LibTorch/1.4.0/LibTorch.podspec.json",
    url = "https://ossci-ios.s3.amazonaws.com/libtorch_ios_1.4.0.zip",
)

This podspec's OTHER_LDFLAGS contains quoted -force_load "...some path..." which get truncated in such a way that the generated BUILD.bazel file is invalid:
# bazel build @LibTorch//:LibTorch
ERROR: …/external/LibTorch/BUILD.bazel:85:7: syntax error at 'external': expected ',', 'for' or ']'
ERROR: …/external/LibTorch/BUILD.bazel:201:7: syntax error at 'external': expected ',', 'for' or ']'
ERROR: …/external/LibTorch/BUILD.bazel:317:7: syntax error at 'external': expected ',', 'for' or ']'
ERROR: …/external/LibTorch/BUILD.bazel:421:7: syntax error at 'external': expected ',', 'for' or ']'

Generated code looks like:
objc_library(
  # …
  copts = [
    ""external/LibTorch/install/lib/libtorch.a"",
    "-force_load",
    "-Wnon-modular-include-in-framework-module",
    "-g",
    "-stdlib=libc++",
    "-DCOCOAPODS=1",
    # …
  ],
  # …
)

I'm not sure what -force_load means nor why there is even a OTHER_LDFLAGS but I bet just splitting this string on spaces is the issue and the generated code should actually look like:
objc_library(
  # …
  copts = [
    "-force_load \"external/LibTorch/install/lib/libtorch.a\"",
    "-Wnon-modular-include-in-framework-module",
    "-g",
    "-stdlib=libc++",
    "-DCOCOAPODS=1",
    # …
  ],
  # …
)

Moreover I'd say these ldflags should be put under linkopts = [...] instead of copts = [...] but again I'm not familiar with this.
cc #111
Note: one of the example uses this flag

FYI the actual WORKSPACE + BUILD that should come from this pod:
# WORKSPACE
http_archive(
    name = "ios_libtorch",
    build_file = "@//third_party:libtorch_ios.BUILD",
    sha256 = "9806c9e49b25848d30e2036ad1e64f04f66684a64e336f6cd5cce72d559ce0dd",
    strip_prefix = "install",
    type = "zip",
    url = "https://ossci-ios.s3.amazonaws.com/libtorch_ios_1.4.0.zip",
)

# third_party/libtorch_ios.BUILD
cc_library(
    name = "libtorch",
    srcs = glob(["lib/*.a"]),
    hdrs = glob(["include/**/*.h"]),
    includes = [
        "include",
        "include/TH",
        "include/torch/csrc/api/include",
    ],
    linkopts = [
        "-force_load $(location lib/libtorch.a)",
    ],
    visibility = ["//visibility:public"],
)

Note the linkopts.


Paymetheus gets stuck during headers fetch when dcrd is updating

Not sure where the fault lies here.  It could be an issue with dcrd's getheaders RPC, the wallet's header fetching code, or the grpc api to fetch headers.
@marcopeereboom hit this while opening both PM and dcrd at the same time after dcrd was way out of sync, and I seem to recall the same issue when trying to connect to a dcrd instance in IBD, but never made a note about it earlier.

Been attempting to hit this for a while now and am failing to do so.  Might be already fixed, or could be a race.  Will reopen if experienced again.

So this is still an issue.  Steps to repro:

Have a up to date dcrd and dcrwallet+Paymetheus
Stop both
Nuke or rename the dcrd blockchain data directory so it has to do a complete resync.
Start dcrd up again
Start Paymetheus

I can reliably get wallet stuck at the "fetching headers" or "rescanning" views.  Sometimes an error message was also shown saying that some a block was not found in the main chain.
I think we either need to make the launcher wait until dcrd is reasonably up to date (block times within 2 hours of the current time) or do something in Paymetheus and/or wallet to prevent this from being an issue to begin with (second option is definitely better as it will still allow standalone Paymetheus to work correctly in all cases).

Just encountered this by using the MSI installer and opening "Launcher.exe"

I get stuck on rescanning 123 blocks when starting the app. Is that related to this ticket?

@mascondante same bug, no need for another issue.
I haven't been spending any time trying to fixing this exact issue because I don't think it is the right solution.  Instead of the wallet fetching headers during IBD, the it should wait for dcrd to finish syncing first before it catches up.  Otherwise, when seed restoring wallets, there could be missing addresses/balance because the restore was done with an out of date dcrd.


Added support for One-Shot Mode (OSM)

Just added the support for One-Shot Mode.
Simple usage is to call the "enableOSM" or "disableOSM" member function before configuring the mode of operation.
Let me know if any optimization is needed or could be done in a better way.

line 176 needs to be ==

I guess not. No change is necessary.
That line is simply storing the new value into the mode variable by the decision of osmFlag. osmFlag itself is conditional logic.

I get compiler errors the way it is:
C:\Users\andre\Documents\Arduino\libraries\arduino-mcp2515\mcp2515.cpp: In member function 'MCP2515::ERROR MCP2515::setMode(MCP2515::CANCTRL_REQOP_MODE)':
C:\Users\andre\Documents\Arduino\libraries\arduino-mcp2515\mcp2515.cpp:176:63: error: assignment of read-only parameter 'mode'
mode = osmFlag ? mode | CANCTRL_OSM : mode & ~(CANCTRL_OSM);
^
C:\Users\andre\Documents\Arduino\libraries\arduino-mcp2515\mcp2515.cpp:176:20: error: invalid conversion from 'int' to 'MCP2515::CANCTRL_REQOP_MODE' [-fpermissive]
mode = osmFlag ? mode | CANCTRL_OSM : mode & ~(CANCTRL_OSM);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
and since its basically the same as
if( mode = osmFlag)
{mode | CANCTRL_OSM}
else
{mode & ~(CANCTRL_OSM)}
I´d say it needs to be a comparison, not an assignment.
It still send 2 messages as a burst tho, not 1...

Haha thanks for showing me indirectly a mistake here.
The solution is to remove const keyword. SIMPLE. Or use a entirely new variable. I,ll make the changes tom its late here.
Plz consult how shorthand if are used.
https://www.w3schools.com/cpp/cpp_conditions_shorthand.asp

Here is the doc of MCP2515: http://ww1.microchip.com/downloads/en/DeviceDoc/80179g.pdf
It is a bug of this chip (see the page 2). If the SPI is not set to disabled (or using other solutions), the chip will keep pending. That is why the chip keep send can frames to the bus.
I think #30 solved the problem temporarily, we can direct operate SPI.
But "Use hardware pins (TXnRTS) to request transmission" may be the best solution.
@DeltaC6

This don't work: 179: modifyRegister(MCP_CANCTRL, CANCTRL_REQOP, mcpMode);
CANCTRL_REQOP mask exclude CANCTRL_OSM bit.
This work: modifyRegister(MCP_CANCTRL, CANCTRL_REQOP | CANCTRL_OSM, mcpMode);

This don't work: 179: modifyRegister(MCP_CANCTRL, CANCTRL_REQOP, mcpMode); CANCTRL_REQOP mask exclude CANCTRL_OSM bit.
This work: modifyRegister(MCP_CANCTRL, CANCTRL_REQOP | CANCTRL_OSM, mcpMode);

Very good catch. The CANCTRL_REQOP is 0xE0, which is excluding the OSM bit 0x8 on CANCTRL[3]
Here is a picture from the mcp2515 datashet;

I can also confirm this fixes it, tested with an osciloscope.
@Loverboy7000 This would be a great feature to have in the master, is there anything I could do to help?

anyone can merge this?


Paper key not accepted.

Changed distros on both of my machines a few months back, and just got around to reinstalling Keybase. I've entered my 12-word passphrase, but the app will not accept it. States that I have not entered enough words for a paper key.
Has the paper key word count changed, and if so, does this invalidate my old keys and render the account unrecoverable?

it is 13 words long. Should work on all devices. Hasn't changed, we promise!
On Sat, Jun 17, 2017 at 1:23 PM DevSlashNull<EMAIL_ADDRESS>wrote:

Changed distros on both of my machines a few months back, and just got
around to reinstalling Keybase. I've entered my 12-word passphrase, but the
app will not accept it. States that I have not entered enough words for a
paper key.
Has the paper key word count changed, and if so, does this invalidate my
old keys and render the account unrecoverable?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/issues/7448, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA05_5KBp5i_48KcMD6MN2MQLwF5YYVXks5sFAt0gaJpZM4N9SE8
.

Argh, somehow missed one in that copy. Found the backup-backup with all the words. Sorry about that! Thank goodness for backup redundancy.


Appease clippy, re-organize core_types submodules.

This is a breaking change because avoiding ambiguous globs means that the ambiguous types must be removed from the public interface. I have taken the chance to remove some other dubiously useful type from the top core_types module as well. The submodules that contained those types are made public.
The variant_array module has been renamed to array for simplicity.

bors try

What's the reason for moving the tests? Or when should integration tests be part of the implementation file, vs. inside /test?

The immediate reason is that the test crate is no longer able to access some of the test functions, that were glob re-exported before the changes, and I feel like this is the most natural way to deal with that, moving the list closer to the definition site of the tests themselves.
The general problem imo depends on the scope of the tests. Things that depend on the proc-macros or the generated API by necessity have to live inside /test. For things that test specific core_types I think it's fine for them to be placed closer to the implementations they test. You can call the former "integration" integration tests and the latter "unit" integration tests, I suppose. I don't find the current organization to be particularly problematic.

bors r+


Use JDK 11 (LTS) + 13 for AppVeyor builds

Detekt uses Gradle 6, which supports JDK 13.
Thus, we should also run a CI build for that.

Codecov Report

Merging #2141 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master   #2141   +/-   ##
========================================
  Coverage      80.5%   80.5%
  Complexity     2017    2017
========================================
  Files           336     336
  Lines          5801    5801
  Branches       1063    1063
========================================
  Hits           4670    4670
  Misses          564     564
  Partials        567     567

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 d6848af...b108f04. Read the comment docs.

blocked by #2143

I think this needs just a rebase from master and we are good to merge it :).

Can we drop JDK 11? AppVeyor is the slowest CI because they don't support parallel jobs, so if we can test only on earliest & latest supported JDK like macOS on Travis that should give sufficient coverage and cut down the build time a bit.


Fix invalid return type when an effect has aborted

Summary
Fix https://github.com/hyperion-project/hyperion.ng/commit/9475b93d9fbc192fbb33a6dac83c8163e1d46f76#commitcomment-47646390
This is a follow up to PR #1181: since, before this PR, effects would never stop, fixing this uncovered another bug: when aborting an effect, the EffectModule was written to return None on any Python method call. This means that the return type of those methods could change mid-execution, if an effect was considered to have aborted before hyperion.abort() was checked again.
By not changing the behavior depending on the abort status, the effect scripts don't crash when an abort is requested. We still have to check if this fixes launching effects from the tray, but I currently don't have access to the hardware setup to check that.
What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Docs
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing setups:
The PR fulfills these requirements:

[x] When resolving a specific issue, it's referenced in the PR's body (e.g. Fixes: #xxx[,#xxx], where "xxx" is the issue number)

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature
[ ] Related documents have been updated (docs/docs/en)
[ ] Related tests have been updated

PLEASE DON'T FORGET TO ADD YOUR CHANGES TO CHANGELOG.MD

[ ] Yes, CHANGELOG.md is also updated

To avoid wasting your time, it's best to open a feature request issue first and wait for approval before working on it.
Other information:

@vtavernier Sorry, did you test that the proposed changes will fix the problem that effects started via tray do not run?
Here the Booteffect nicely works and is stopped.
If I start an effect, it runs directly into wrapAbort with getRaimaining = -1 and the efect is not executed....
021-03-03T20:59:25.431 hyperiond EFFECTENGINE : <INFO> Run effect "Knight rider" on channel 1
2021-03-03T20:59:25.431 hyperiond EFFECTENGINE : <DEBUG> EffectEngine.cpp:183:runEffectScript() | Start the effect: name [Knight rider], smoothCfg [2]
2021-03-03T20:59:25.431 hyperiond HYPERION     : <DEBUG> PriorityMuxer.cpp:161:registerInput() | Register new input 'System/EFFECT' with priority 1 as inactive
EffectModule::wrapAbort
Effect::isInterruptionRequested(): _interupt [0] getRemaining(): [-1]
Effect::isInterruptionRequested(): _interupt [0] getRemaining(): [-1]
2021-03-03T20:59:25.444 hyperiond EFFECTENGINE : <INFO> effect finished

Would you mind check and test again?
Thank you!

OK. I have the issue. Sometime setEffect() is called without timeout parameter populated which then defaults to -1 and therefore the effect stops immediately as timeout < 0 is met.

Good catch! I also thought something was different between the tray icon launch and the boot effect but I wasn't familiar enough with the code base to diagnose it.

@vtavernier I checked in a fixe on the tray issue via PR #1199.
Would you mind testing it from you end too, please?
In regard of this PR (and I have to admit, I am new to hyperion effects, too),
my understanding is that the effect script might invoke a sequence of those "wrap" functions during execution.
If interpret it right, the intend of the  isInterruptionRequested() call at every method, is stopping the effect and not continue with the sequence of activities in the effectg. The "interrupt effect" will e.g. be triggered when hyperion is stopped.
Will this PR do any harm that hyperion will not stop in a defined fashion or reasonable timeline, as the effect script might be a long running sequence? or what should be changed to avoid the highlighted invalid return type issue?

#1199 Looks good to me, with #1195 and without.
As for the isInterruptionRequested() I think it's more of a trade-off:

Before #1195 changes: if the effect times out between the hyperion.abort() check and another method that returns "something", such as the imageWidth and imageHeight methods, the effect will crash as those will return None instead of an Int. It's unlikely but could technically happen.
After #1195 changes: all the methods keep working as they do before the timeout, but it becomes the script's responsibility to check hyperion.abort() to abort for sure.

This depends on whether we want to allow the script to run some clean-up after the timeout expires, and prevent errors that could be confusing with the 1st option (since they usually report as "cannot multiply NoneType with Float" and aren't really clear), but I don't know what are the requirements for the effect behavior.

@vtavernier Thanks for sharing your view.
While thinking further on the topic in parallel, I came to the conclusion too, that doing things right outperform any performance driven optimizations. Given that the individual functions are small units of work, having them finished rightly might be better than pulling the plug the "hard way".  So I am going to integrate #1195 into the latest PR and both of us do just some testing.... What do you think?
More concerning for me is that I saw that, if you run a background effect, it burns actively CPU cycles, even it is "hidden" by another priority in the source queue. So it does not sleep until it gets into focus again.

I also think it's not necessary to pull the plug the "hard way" as you put it. The effect module functions do a negligible amount of work: they would maybe run one time too many between the timeout and the next hyperion.abort check, which is basically nothing.
Letting the effects complete gracefully seems the most sensible option to me, and it does simplify the code (less checks for timeouts, no return type change, no need to document this effect in the Python module API, functions do behave as expected no matter the timeout).
As for the resources issue, on limited hardware, the background effects indeed put substantial strain on the CPU when enabled (on a Raspberry Pi Zero, the Rainbow Swirl consumes 20%CPU constantly). Maybe adding a hyperion.active method to notify the effect script if the current effect is active or not? I'm not familiar with all the effect scripts, but:

Some scripts may have to run continuous updates to internal state, even if not visible
This would allow skipping more resource-intensive methods that generate images, LED data, etc. which might be enough to significantly reduce the CPU load?

@vtavernier While removing the stratements...
In the various methods Py_RETURN_NONE is often returned to indicate a successful return rather than a nullptr as an indication for failure.
If you are saying that Py_RETURN_NONE is not appropriate, this stimulates the question at my end what is the appropriate alternative?
Sample:
PyObject* EffectModule::wrapImageDrawLine(PyObject *self, PyObject *args)
{
...
	if (argsOK)
	{
		QPainter * painter = getEffect()->_painter;
		QRect myQRect(startX, startY, endX, endY);
		QPen oldPen = painter->pen();
		QPen newPen(QColor(r,g,b,a));
		newPen.setWidth(thick);
		painter->setPen(newPen);
		painter->drawLine(startX, startY, endX, endY);
		painter->setPen(oldPen);

		Py_RETURN_NONE;
	}
	return nullptr;

I think it could be left that way. I only removed the "check timeout and return none" parts so the rest can be left as-is.
In the case of wrapImageDrawLine, there is indeed no need to return a particular value so it doesn't change anything. In the case of wrapImageWidth, an integer should always be returned, so returning None because the effect was aborted just creates more edge cases.

In the case of wrapImageDrawLine, there is indeed no need to return a particular value so it doesn't change anything. In the case of wrapImageWidth, an integer should always be returned, so returning None because the effect was aborted just creates more edge cases.

Thanks. I now get the point (which I might have missed before).

@vtavernier I just incorporated the changes of PR  into #1199 to just having everything at one place for testing the PR.
I will close this PR for the time being.
Thank you again very much for your time on getting this fix in place!!!
And please continue joining us addressing further issues out there.
Looking forward seeing the next PR by you ;)


README.md: add a gif showing a typical install

I suggest https://asciinema.org/
ie. https://asciinema.org/a/e5geldgvynjmf8f6uh2b8n74j is one for brew-cask-upgrade (by buo)

I dislike asciinema for this. It cannot play directly in the README (yes, we can make a gif of it, but at that point, there’s no difference to a screen recording) and we lose the Finder, which I find a really important detail, here, I really want the user to feel how quick GUI changes happen with a CLI command (those have to be imagined with ascinema, but are explicitly showed with a screen recording).

Gotcha. Haven't experimented with, but have heard good things about http://www.cockos.com/licecap/ if you want to compare gif output/filesize.
Else, the original gif LGTM. Was just looking to optimize for mobile/low-bandwidth

I normally use imageoptim on the final output image

Haven’t done so yet, but I should. Will update shortly.
Has for the gif creation itself, I’ve got that covered, and I must’ve obsessively explored all methods in existence, already. I even made two scripts for it (both in vitorgalvao/tiny-scripts) in the past: gifmaker to make a gif from a selection of images (I use processing to make an animation exported as a series of frames) and gfv (“gif from video”) to make a gif from a video file.
ffmpeg can also be used to make a gif from a sequence of images, or to export a video as frames (useful for some manual editing). Drop to GIF (brew cask install drop-to-gif) is also a simple and good-looking tool to convert videos to gifs, with good results.
I don’t recall exactly why I did not like LICEcap, but I think it was due to a lack of precision when setting up the area to record.
Currently, my preferred method is to record with ScreenFlow (brew cask edit screenflow) as it gives a a lot of editing control lacking in most screen recording methods. For example, in the gif in this PR I accelerated the parts that took longer: typing the command is about twice as fast, and the download was accelerated 1300%. I then export the video and convert to gif with something like gifify (brew install gifify). At this point it may be worth trying the conversion with different tools. If you obsess about the image quality, you’ll notice the differences between the outputs.
Note that ScreenFlow has recently gained the ability to export to GIF directly, in version 6. Can’t attest to it, as I’m still on version 5. The app is a bit pricey, but I won’t say the price is undeserved. I got it for cheap with a bundle with version 4, and then paid upgrade pricing for version 5. Will likely do it again for version 6. As far as I can tell, they don’t do many promotions (a small discount around Christmas, perhaps) and don’t participate in many bundles. They do seem to offer the same upgrading pricing regardless if you’re one version behind, or have just version 1 (this may be an advantage or a disadvantage, naturally).
Hope that wasn’t too boring, and that there’s some useful information there.

Didn't realize you had done so much research into it already, so I gladly defer to your judgement. The writeup was useful & quite comprehensive, appreciate it!
Haven't needed to make gifs too often, but will keep the above in mind, thanks!


Changes to generate a random token per Service for public dns

A cryptographically secure random token is generated during Service create and exposes in the rancher-metadata so that it can be consumed by public dns provider

We need to add some integration test for this PR ;)

@prachidamle I thought we decided the UUID is good enough for this.  With UUID we get 112 bits of random data which should be sufficient for this thing.

@ibuildthecloud no we dint decide on uuid, I guess the service uuid is exposed in the API. That wont make it secure for token exchange right?

Depending on the version, many of the bits in a UUID are not random.  I thought we decided to use the same function that generates API secret keys (40 "base62" characters, ≈240 bits).

The Java impl specifically has 112 bits of randomness.  If we want to create a random token id rather we not call this "dnsToken" then.  Just call it token.
We were going to just use the uuid because we didn't have to program anything.  Now that we've gone through the effort and did this might as well use it.  Just change the name.

@alena1108 @ibuildthecloud @vincent99 added new changes to the PR. please review...

@alena1108 any idea what is wrong for the failed test
My test has passed though
[gw4] PASSED core/test_svc_api_validation.py::test_validate_service_token

@prachidamle can you rebase?

@ibuildthecloud rebased, but again my test passed and something else fails
[gw3] PASSED core/test_svc_api_validation.py::test_validate_service_token

This test is failing "test_docker_mount_life_cycle"
@cjellick believes its a problem with the test itself, he will be fixing sometime later
@alena1108 can you kick off another tests to see if I get lucky this time for some random reason

@ibuildthecloud @alena1108 woohoo I got lucky!

LGTM


changing /graphql location

Looks like this is not quite working as its supposed to. My understanding is that we need to change the js file aswell if we specify an alternate location
in index.html
function graphQLFetcher(graphQLParams) {
        // This example expects a GraphQL server at the path /graphql.
        // Change this to point wherever you host your GraphQL server.
        return fetch('/graphql', {
          method: 'post',
          headers: {
            'Accept': 'application/json',
            'Content-Type': 'application/json',
          },

Wanted to see if I could fix it but ran into issues when I tried to reference the source as a project reference.
I pulled the source and did a build, when I referenced the project, I couldn't get the graphiql page to load. The /graphql endpoint was just not serving up the static pages.. Not too sure why. The dll in nuget definitely works so wondering if that is a different version to what is in master?

Hi @srini85,
Thanks for raising this, I'll have to take a look to confirm then get back to you.

@srini85 it seems ok when running the example project. what was your plan with this? i'm not sure how best to templatize the html, since it's just an embedded file

@simonpetty want to use an alternative location to /graphql. I was thinking of doing it at build time by setting it in the configuration json and then using that as part of the build process. However I couldn't get the project working when I built it myself. So hence wondering if the nuget version is different to the the master src.

Another option would be to use the .NET Core proxy library to capture requests to a custom endpoint and proxy them to /graphql.
@srini85 Were you working from a fresh clone of master?

@srini85 I've just tried launching Graphiql from a custom location (/hello instead of the default /graphql) and can confirm that it does work. Can you provide a repository of what you're trying to do so I can take a look?

@JosephWoodward I will see if I get time to push that repository I was working with. Maybe I am missing something, but deffinately did work of a fresh clone of master with no changes. But if you are confirming that it deffinately does work, perhaps I am missing some step or my environment is not quite right. I will continue to dig and let you know the result.
Just out of curiousity, when you said it worked, are you referring to graphiql loading or loading and making requests to the server? Seeing that the fetch promise is set to /graphql, wondering how that gets updated if your controller location is set to something like /hello (as per the original issue comment)?

@srini85 It's done here
https://github.com/JosephWoodward/graphiql-dotnet/blob/master/src/graphiql/GraphiQlExtensions.cs#L38
Allowing you to serve files from a specific location.

The graphiql page loads but the post when submitting a query doesn't work. Another idea is to use an InMemoryFileProvider for the JavaScript currently in the html page, which we can modify with the correct path. Will try to get a version of this going.

@simonpetty I'm sure I tried posting a query 😕 I'll take another look and check it out. I think there are a few options we could use to get that working, such as the proxy which I mentioned above.

Another option would be to use the .NET Core proxy library to capture requests to a custom endpoint and proxy them to /graphql.

This does not work when hosting under IIS virtual directory. For me a path relative to  graphiql would solve the issue...

I also am trying to change the routes to /foo/graphiql and /bar/graphql. The problem is client side requests are always made for /graphql path.
https://github.com/JosephWoodward/graphiql-dotnet/blob/979199d78ab4c393617416eb4e7cc3605032c5ec/src/graphiql/assets/index.html#L112

Yeah, this seems to be the biggest blocker. I'm considering adding a template string to the index.html then doing a replace on it, something like:
return fetch('{{ graphqlPath }}', {

Seems like the easiest option.

@JosephWoodward That should work. Do you think we would have a release anytime soon?

Can't wait :smile: I'll try making a PR.

Hello. Could we please get this issue addressed? We are facing the same issue, as Graphql is part of a versioned solution (so it runs as my.service.com/1.0/graphql). Actually simply removing the path from here https://github.com/JosephWoodward/graphiql-dotnet/blob/master/src/graphiql/assets/index.html#L112 should do the trick as it then makes a request to the resource itself. That should satisfy all cases, regardless if you are hosing under a completely custom URL or "just" in a subdirectory like we are.

Have an open PR for this (https://github.com/JosephWoodward/graphiql-dotnet/pull/23) which I'm currently testing an will release in the next few days.

This has now been released as a beta package, if you would mind giving it a try and reporting back if you find any issues. You can find the beta here.

Hi!
We are currently testing this. While this change allows us to explicitly set our path (eg. "/1.0/GraphQl"), I would argue this change would work better if this variable is an empty string (so that calls on the interface go to the same URL the interface is hosted on) or at least this check should be removed so we can manually do app.UseGraphiQl(""); if we want this behaviour.
Cheers

Actually, digging deeper, the real problem is that GraphiQL treats the server and JS paths the same. If you have a setup in IIS like this ...

Then IIS strips the "1.0" out of the path. So from JS side your URL is /1.0/graphql while from C# side (i.e. in GraphiQlConfig or the FileServerOptions built here) the path should just be /graphql.

So from JS side your URL is /1.0/graphql while from C# side (i.e. in GraphiQlConfig or the FileServerOptions built here) the path should just be /graphql

I don't have IIS to hand as I use a Mac/Linux, any thoughts how we could resolve this? Not sure how we could get around IIS stripping values from the path, feels like that shouldn't be the responsibility of this library. Thoughts?

Yes, I am absolutely with you, it's not your responsibility to know that. Obvious "worst case" solution is allowing setting of JS and C# paths separately, but that really isn't elegant.

@FrankyBoy Though I'm wondering if there would be any use cases for adding an overload to enable you to configure the JS and C# paths separately. I'm not completely against the idea.

Other than my use case (i.e. IIS being weird) I see also this one: Having a separate configuration would make sense if someone wants to host the UI in a different location than the actual graphql endpoint. Like /docs for the interface and /graphql for the actual graphql endpoint. That would require different paths for the FileServerOptions and the path in the delivered JS as well. You could even target another server entirely (for instance to host a sort of centralized documentation site).
Not sure if anyone would ever want to do that, it's just another potential use case I see.

Any update here? Should I just push a change that lets us configure server and client path independently or can you look into the topic?
Cheers!

1.2.0 has now been published to NuGet

In my solution works normally but when publish in IIS server show this empty string ""

In my Solution is fine, Which may be?

@areyes21 That's odd. If you perform the query via a raw HTTP Request (as opposed to using GraphiQL), do you get any results? That would isolate the issue to this library.
Also, have you looked in your browser's dev tools? GraphiQL queries your API via an ajax request so it would seem that your API is returning no data?
Is the schema viewer working?

The route of graphqlpath is /GraphQL But if I change graphqlpath for an empty string it works well.
Line of code

Is there any chance you'd be able to create a repo of this issue as I'm struggling to reproduce it?
Alternatively, what does your configuration look like? I see you're running it from localhost in your screenshots? Earlier you mentioned it was only when you published your app to IIS? (or does it also do it if you're running it via IIS locally?)

Sorry I do not speak English but I will do my best to explain

I could solve it, adding what I show in the screenshots (EN)
Lo pude resolver, agregando lo que le muestro en las capturas de pantalla  (ES)

Thanks for all the help, Greetings from Nicaragua

I'm glad you got this sorted in the end, I'm going to mark this issue as closed.


Could not find artifact org.unidal.framework:framework-bom:pom:4.0.0

Non-resolvable import POM: Could not find artifact org.unidal.framework:framework-bom:pom:4.0.0

Recommend use tag v2.0.0 for cat.
Founded here:
Dependencies: https://github.com/unidal/frameworks
Plugins: https://github.com/unidal/maven-plugins
Steps:

git clone https://github.com/unidal/frameworks.git
cd frameworks && git checkout v3.1.0
mvn -s settings.xml clean install -DskipTests
git clone https://github.com/unidal/maven-plugins.git
git checkout v2.5.8
cd manven-plugins && mvn -s settings.xml clean install -DskipTests
git checkout master
mvn -s settings.xml clean install -DskipTests

Note:

Plugin must use tag v2.5.8 for cat v2.0.0
Plugin need some 3.1.0 framework dependencies, so need install 3.1.0 first

Environment Details:
Platform: Windows
Maven: 3.3.9
JAVA:	jdk1.8.0_121(64 Bit)

git clone https://github.com/unidal/frameworks.git
cd frameworks && git checkout v4.0.0
mvn -s settings.xml clean install -DskipTests

then build cat will work

@qiangmao @boying Thanks, both works!


a small question about the loss function

In soundstream.py line 58:
def hinge_gen_loss(fake)
return -fake.mean(）
May I ask if this should return (F.relu(1 - fake)).mean()?
Thanks

so i'm taking the formulation used in stylegan2, which in turn uses this paper (paperswithcode explanation)


Server side enhancement: Project owners in create/update

I have a need to be able to manage the owners of projects via the REST API. The owner ID is missing from the query/create/update endpoints on the Server side.

Which Tableau Server version are you using? I believe owner ID is returned for all 3 of the endpoints. Though, I just noticed that it is not documented and also not in TSC yet.

Which Tableau Server version are you using? I believe owner ID is returned for all 3 of the endpoints. Though, I just noticed that it is not documented and also not in TSC yet.

Hey @shinchris, I am using 2020.3. It does look like it is returned in the response on GET and during create, but not during update.
Additionally, when I add owner as a sub element, similar to how it is returned when it is returned in a GET call, it does not seem to actually make that user the owner. Instead, it makes me the owner.

Hey @shinchris, I am using 2020.3. It does look like it is returned in the response on GET and during create, but not during update.
Additionally, when I add owner as a sub element, similar to how it is returned when it is returned in a GET call, it does not seem to actually make that user the owner. Instead, it makes me the owner.

Ah yes, you are correct that update call doesn't return the owner ID. Updating project owner is also currently not possible using the REST API. We have #269 tracking that for now.
I'm curious how the update made you the owner instead, though. Was it owned by you in the first place?

Ah yes, you are correct that update call doesn't return the owner ID. Updating project owner is also currently not possible using the REST API. We have #269 tracking that for now.
I'm curious how the update made you the owner instead, though. Was it owned by you in the first place?

Ah. It was a new project that I created then tried updating via the REST API. The create was what I was referring to as making me the owner despite being provided the owner id.
Should I close this, or do you consider returning the owner ID on update a separate issue?

Ah. It was a new project that I created then tried updating via the REST API. The create was what I was referring to as making me the owner despite being provided the owner id.
Should I close this, or do you consider returning the owner ID on update a separate issue?

Gotcha, that makes sense.
I'll go ahead and close - I would consider returning the owner ID on update as the same issue (i'll mention it also in the internal work item).

Gotcha, that makes sense.
I'll go ahead and close - I would consider returning the owner ID on update as the same issue (i'll mention it also in the internal work item).


⚠ RPC server - Arbitrum has degraded performance

In 784f7b6, RPC server - Arbitrum ($ARBITRUM_RPC_URL) experienced degraded performance:

HTTP code: 0
Response time: 0 ms

Resolved: RPC server - Arbitrum performance has improved in 0fa13e9 after 6 minutes.


Migrate content from OGM > aardvark and close the repo

@karenmajewicz I finished migrating and formatting the content from the [OGM>aardvark repo](https://github.com/OpenGeoMetadata/aardvark.
Check out the new OGM site here: https://github.com/OpenGeoMetadata/opengeometadata.github.io
I think there was a comment during the code sprint about closing the aardvark repo once this was done. Do you feel comfortable doing that?

Archived the repo and added a redirect link.


Adding in the ability to specify with env variables how to format the log lines

Adding in the ability to specify with env variables how to format the log lines

So, the default logger is not a default logger and it can be one of?
On one hand, I think it's an easy and quick solution with no code changes required. On the other hand, it changes the behavior of the "Default" logger. It can be a breaking change for tests.
We don't have a lot of places where we instantiate a logger in our projects (environment.go, and maybe some other). Maybe we can create a new method and update templater instead?

It doesn't change or break anything right now because no one uses that flag and its default was the same as before.

The templater runs with NewDefault so it can use simply the logfmt or if this is set in the environment json. Most logging frameworks have a way without code to change the logging format, we currently don't.


Docs: How to run the Todo app tutorial with sqlite?

Summary
I am currently trying to get started with litestar with the todo app with sqlalchemny tutorial.
Running the code provided by this page does not run currently because the table todo_items is missing in the database.
(sqlite3.OperationalError) no such table: todo_items [SQL: SELECT todo_items.title, todo_items.done FROM todo_items] (Background on this error at: https://sqlalche.me/e/20/e3q8)

This is triggered by result = await session.execute(query) in the function get_todo_list.
It does make sense to me, because no table is visibly created. I am not very familiar with SQLalchemy and wasn't able to figure out what to do. Not sure if this is a bug in the code or if it's just a missing hint in the docs that one has to create the table before.

[!NOTE]
While we are open for sponsoring on GitHub Sponsors and
OpenCollective, we also utilize Polar.sh to engage in pledge-based sponsorship.
Check out all issues funded or available for funding on our Polar.sh Litestar dashboard

If you would like to see an issue prioritized, make a pledge towards it!
We receive the pledge once the issue is completed & verified
This, along with engagement in the community, helps us know which features are a priority to our users.

Thanks, a contribution to add this would be very welcome!

I will add this within the next couple of days.
When I build the docs with make docs-serve I don't see different code for different python versions. Where is that configured?

Closed by merge of #2497


[BUG] Gateway Error in version 1.7.0

Describe the bug
A clear and concise description of what the bug is.
Last Frigate Beta Release 1.7.0 gives Bad Gateway Error
Version of Double Take
Current version: 1.7.0
Expected behavior
A clear and concise description of what you expected to happen.
Working Webinterface
Screenshots
If applicable, add screenshots to help explain your problem.

Hardware

OS: [e.g. Ubuntu, macOS, Windows]
Browser (if applicable) [e.g. Chrome, Safari]
HassOS Intel NUC / Chrome Browser

Additional context
Add any other context about the problem here.
Version 1.6.0 Works perfectly
my Addon Config:

Log doube-take:
info: Double Take v1.7.0-456a6ff
Log Supervisor
21-11-30 00:03:17 ERROR (MainThread) [supervisor.api.ingress] Ingress error: Cannot connect to host <IP_ADDRESS>:3000 ssl:default [Connect call failed ('<IP_ADDRESS>', 3000)]

No news here? How can i roll back to 1.6.x if no backup available? :(

Any news ?

I'm not able to reproduce this and haven't heard of anyone else having this issue at the moment. Can you post your current config file? I'm a bit baffled why this would randomly pop up.
Does the UI show up if you go to the port directly and bypass Ingress?

hello, I had zwave addon also running on that port. Now I removed zwave addon to expoe port 3000 but still the same error persists. I have absolutely no idea why it does work with version 1.6.0 and not with 1.7.x What can i do to help you with that?

having the same problem
502: bad gateway
help :)?

There is an issue with the latest hassio addon.
here is a fork:
https://github.com/rubenoost/double-take-hassio-addons
remove ORIGINAL from repo addon and replace with this one and install STABLE addon. You will see it will work again :)
SECRET_PATH somehow makes problems

I got this error recently and found this thread.  I found that after I restarted my Frigate server, the error was gone and Double Take started working again.  Good luck.

I am experiencing the same 502 Bad Gateway error.

you can try this:

https://github.com/jakowenko/double-take/issues/173#issuecomment-1003369177

#173 (comment)

I will look at the fix and apply it to the add-on repo.


Relation errors. Failed Insertion into Tables

Description:
Encountering errors during data insertion into multiple tables. The error messages indicate that the SQL statements are failing due to the absence of the corresponding relations in the target database.
Error Details:
[2024/02/26 09:44:32] [error] target/target.go:363 failed to insert into table research_report_worktime_archive: statement description failed: ERROR: relation "public.research_report_worktime_archive" does not exist (SQLSTATE 42P01)
[2024/02/26 09:45:54] [error] target/target.go:363 failed to insert into table pure_caches: statement description failed: ERROR: relation "public.pure_caches" does not exist (SQLSTATE 42P01)

Originally posted by @TristisOris in https://github.com/slotix/dbconvert-streams-public/issues/22#issuecomment-1963428603

Fixed in version https://stream.dbconvert.com/releases-2024#version-0-10-7


Asurascans Hide Premium chapters

Closes:  #6614
Checklist:

[X] Updated extVersionCode value in build.gradle for individual extensions
[X] Referenced all related issues in the PR body (e.g. "Closes #xyz")
[X] Have not changed source names
[X] Have explicitly kept the id if a source's name or language were changed
[X] Have tested the modifications by compiling and running the extension through Android Studio

preference should be enabled by default imo


unable to open .ipynb file

Type: Bug
Reproduce Problem:
1 open .ipynb file
or
1 command to create new .ipynb file
Expected Results:
Opens file
Actual Results:
"The editor could not be opened due to an unexpected error: Cannot open resource with notebook editor type 'jupyter-notebook', please check if you have the right extension installed and enabled."
or
"Command 'Create: New Jupyter Notebook' resulted in an error command 'ipynb.newUntitledIpynb' not found"
I have Jupyter and python extension installed and enabled. I have attempted uninstalling and reinstalling all these extensions and Visual studio code itself. I can include dev tools logs if that helps.
Extension version: 2023.8.1002501831
VS Code version: Code 1.82.2 (abd2f3db4bdb28f9e95536dfa84d8479f1eb312d, 2023-09-14T05:55:25.390Z)
OS version: Windows_NT x64 10.0.19045
Modes:
Remote OS version: Linux x64 4.14.322-244.539.amzn2.x86_64

System Info

Item
Value

CPUs
Intel(R) Xeon(R) Gold 6342 CPU @ 2.80GHz (4 x 2793)

GPU Status
2d_canvas: unavailable_softwarecanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onopengl: disabled_offrasterization: disabled_softwareraw_draw: disabled_off_okvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_softwarewebgpu: unavailable_software

Load (avg)
undefined

Memory (System)
8.00GB (1.46GB free)

Process Argv
--crash-reporter-id 0f70f408-1b7a-4efe-a296-313d631e5da3

Screen Reader
no

VM
100%

Item
Value

Remote
SSH: ppm-dev

OS
Linux x64 4.14.322-244.539.amzn2.x86_64

CPUs
Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz (4 x 3102)

Memory (System)
15.36GB (1.33GB free)

VM
0%

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
pythonvs932:30410667
py29gd2263:30792226
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
pynewext54:30695312
azure-dev_surveyone:30548225
3biah626:30602489
89544117:30613380
2i9eh265:30646982
showlangstatbar:30737416
0bi6i642:30841073
03d35959:30757346
pythonfmttext:30731395
fixshowwlkth:30771522
showindicator:30805244
pythongtdpath:30769146
i26e3531:30792625
pythonnosmt12:30797651
pythonidxpt:30805730
pythonnoceb:30805159
copilotsettingc:30839828
dsvsc013:30795093
dsvsc014:30804076
diffeditorv2:30821572
pythonmhint1:30842940
dsvsc015:30845448

Thank you for getting back to me.
Here is a screenshot the developer tools screen:
ERR Extension 'ms-vscode.remote-server' appears in product.json but enables LESS API proposals than the extension wants.
package.json (LOSES): resolvers, tunnelFactory
product.json (WINS): resolvers, tunnels, contribViewsWelcome
log.ts:421  INFO Invoking resolveAuthority(ssh-remote)...
log.ts:421  INFO [LocalProcess0][resolveAuthority(ssh-remote,1)][0ms] obtaining proxy...
log.ts:421  INFO [LocalProcess0][resolveAuthority(ssh-remote,1)][0ms] invoking...
log.ts:421  INFO [LocalProcess0][resolveAuthority(ssh-remote,1)][1466ms] waiting...
log.ts:421  INFO [LocalProcess0][resolveAuthority(ssh-remote,1)][2168ms] waiting...
log.ts:421  INFO [LocalProcess0][resolveAuthority(ssh-remote,1)][3002ms] waiting...
log.ts:421  INFO [LocalProcess0][resolveAuthority(ssh-remote,1)][4002ms] waiting...
log.ts:421  INFO [LocalProcess0][resolveAuthority(ssh-remote,1)][5001ms] waiting...
log.ts:441   ERR [Extension Host] (node:18792) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
(Use Code --trace-deprecation ... to show where the warning was created)
console.ts:137 [Extension Host] (node:18792) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
(Use Code --trace-deprecation ... to show where the warning was created)
m @ console.ts:137
log.ts:421  INFO [LocalProcess0][resolveAuthority(ssh-remote,1)][5208ms] returned WebSocket(<IP_ADDRESS>:58186)
log.ts:421  INFO resolveAuthority(ssh-remote) returned 'WebSocket(<IP_ADDRESS>:58186)' after 5209 ms
log.ts:421  INFO Creating a socket (renderer-Management-ce05a5bf-e961-4594-a39e-6fd8ff264939)...
log.ts:421  INFO Creating a socket (renderer-ExtensionHost-86ff2fb9-13a9-4a67-84a8-8fa448e05cfb)...
log.ts:421  INFO Creating a socket (renderer-Management-ce05a5bf-e961-4594-a39e-6fd8ff264939) was successful after 74 ms.
log.ts:421  INFO Creating a socket (renderer-ExtensionHost-86ff2fb9-13a9-4a67-84a8-8fa448e05cfb) was successful after 164 ms.
log.ts:441   ERR Extension 'github.copilot-chat' appears in product.json but enables LESS API proposals than the extension wants.
package.json (LOSES): interactive, interactiveUserActions, terminalContextMenu, terminalDataWriteEvent, terminalSelection, terminalQuickFixProvider, handleIssueUri, chatSlashCommands, readonlyMessage, chatVariables, mappedEditsProvider, aiRelatedInformation
product.json (WINS): handleIssueUri, interactive, interactiveUserActions, terminalContextMenu, terminalDataWriteEvent, terminalSelection, terminalQuickFixProvider, chatSlashCommands, readonlyMessage, mappedEditsProvider, aiRelatedInformation
log.ts:421  INFO [jithurjacob.nbpreviewer]: Command jupyter.showPreview already registered by My Jupyter Notebook Previewer (colinfang.my-nbpreviewer)
console.ts:137 [Extension Host] Initializing tunnel extension
log.ts:431  WARN No search provider registered for scheme: vscode-remote, waiting
log.ts:421  INFO [perf] Render performance baseline is 67ms
log.ts:441   ERR Error received from starting extension host (kind: Remote)
log.ts:441   ERR The remote extension host took longer than 60s to send its ready message.
log.ts:431  WARN NotebookEditorWidget#setInput failed Error: Cannot open resource with notebook editor type 'jupyter-notebook', please check if you have the right extension installed and enabled.
at V.setInput (notebookEditor.ts:251:33)
at async s.M (editorPanes.ts:422:4)
at async s.D (editorPanes.ts:252:34)
at async s.openEditor (editorPanes.ts:129:11)
at async editorGroupView.ts:1089:49
at async o.value (explorerView.ts:490:6)
log.ts:441   ERR Error: Cannot open resource with notebook editor type 'jupyter-notebook', please check if you have the right extension installed and enabled.
at V.setInput (notebookEditor.ts:251:33)
at async s.M (editorPanes.ts:422:4)
at async s.D (editorPanes.ts:252:34)
at async s.openEditor (editorPanes.ts:129:11)
at async editorGroupView.ts:1089:49
at async o.value (explorerView.ts:490:6)
When I close all tabs and run command Create: New Jupyter Notebook I get this error message:
Command 'Create: New Jupyter Notebook' resulted in an error
command 'ipynb.newUntitledIpynb' not found

can you try on an empty profile with Profiles: Create a Temporary Profile, then try the create notebook command?
(Use Profile: Switch Profile to get back to the default)
also, in the extensions tab, search for @builtin ipynb and ensure the '.ipyb support' extension is enabled.

actually, can you look in the Extension Host logs to see if it shows an activation failure - in the output panel, select "Extension Host" in the dropdown and look for errors.

Did you attempt to open an ipynb file when you collected those logs? I would expect to see an error for the ipynb extension failing to activate.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Uncaught SyntaxError when building Nuxt.js 3.11.2 with Webpack

Environment

Operating System: Linux
Node Version:     v18.18.0
Nuxt Version:     3.11.2
CLI Version:      3.11.1
Nitro Version:    2.9.6
Package Manager<EMAIL_ADDRESS>Builder:          webpack
User Config:      devtools, builder
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-x4yvog
Describe the bug
I encountered this issue when upgrading from nuxt 3.10.3 to 3.11.2 on my local environment. This error did not occur with 3.10.3.
As shown in the attached reproduction repository, the issue persists even when downgrading nuxt and @nuxt/webpack-builder to 3.10.3, so the problem may not be limited to just those packages.
I would really appreciate any help or insights you could provide on resolving this.
Additional context
No response
Logs
Uncaught SyntaxError: Invalid or unexpected token (at vendors-node_modules_webpack-hot-middleware_client_js_reload_true_timeout_30000_path_2F__webp-b4bdca.js:29:6)

I found that setting webpack-dev-middleware to 7.0.0 or lower is a workaround.
https://stackblitz.com/edit/github-x4yvog-bvn5jq

@wattanx
Thank you for providing the workaround. I really appreciate you taking the time.🥳

This is highly likely to be an upstream issue in webpack-dev-middleware. Would you be able to open an issue with reproduction in https://github.com/webpack/webpack-dev-middleware ? 🙏

I'm trying if I can reproduce this without nuxt.
Maybe it reproduces only when nitro is used.
nitro & webpack-dev-middleware
https://stackblitz.com/edit/github-yqcgdy
express & webpack-dev-middleware (not reproduced)
https://stackblitz.com/edit/github-mawhuh
Strangely, with nitro, the requested bundle is broken.

Since 3.12.1, the workaround using webpack-dev-middleware 7.0.0 no longer works 😭
https://stackblitz.com/edit/github-7zdkqz

Just curious - is there any possible way for you to move away from webpack to vite?

@martinszeltins
I'm considering migrating to Vite, but at this moment, it is difficult for me to make the switch immediately. In the short term, I would like to continue using Webpack.

Let's track in https://github.com/unjs/nitro/issues/2541. This seems to be a nitro bug. Thank you @wattanx for an excellent reproduction.


Add AdaBelief optimizer

Description
Add AdaBelief in tensorflow_addons, which adapts stepsize according to "belief" in gradient, and achieves good generalization, fast convergence and training stability.
Reference: AdaBelief optimizer: adapting stepsizes by the belief in observed gradients (Juntang Zhuang et al. NeurIPS 2020).
Results: 77.25 (AdaBelief) 77.1 (SGD)
Type of change

[ ] Bug fix
[ ] New Tutorial
[ ] Updated or additional documentation
[ ] Additional Testing
[ ] New Activation and the changes conform to the activation contribution guidelines
[ ] New Callback and the changes conform to the callback contribution guidelines
[ ] New Image addition and the changes conform to the image op contribution guidelines
[ ] New Layer and the changes conform to the layer contribution guidelines
[ ] New Loss and the changes conform to the loss contribution guidelines
[ ] New Metric and the changes conform to the metric contribution guidelines
[x] New Optimizer and the changes conform to the optimizer contribution guidelines
[ ] New RNN Cell and the changes conform to the rnn contribution guidelines
[ ] New Seq2seq addition and the changes conform to the seq2seq contribution guidelines
[ ] New Text addition and the changes conform to the text op contribution guidelines

Checklist:

[x] I've properly formatted my code according to the guidelines

[ ] By running Black + Flake8
[x] By running pre-commit hooks

[ ] This PR addresses an already submitted issue for TensorFlow Addons
[ ] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[ ] This PR contains modifications to C++ custom-ops

How Has This Been Tested?
By running a ResNet50 model from google/tensorflow_models benchmark
If you're adding a bugfix or new feature please describe the tests that you ran to verify your changes:
*

@juntang-zhuang please format the code, and make sure your flake8 and black versions are same as in tf-addons


Decorators not honoured when using classToPlain(object)

I have an object, with an x: number and y: number properties.
@Expose({ name: 'coordsX' })
 x: number = 0;
 @Expose({ name: 'coordsY' })
 y: number = 0;
Using plainToClass(Object, values), I get the properties populated correctly.
Using classToPlain(object) I get the values { x, y } exposed - no translation is done.
@Exclude also does not work - I have worked around this by prefixing all methods / properties with _ and then:
classToPlain(object, { excludePrefixes: ['_'], });
It would be nice to have the decorators working properly

+1. excludePrefixes is not working for me either

@chris-jones-pixitmedia I extended the API to allow you to pass in the type of class you want to serialize to and created a PR: #106
If you want to build the stuff up and use it, I've got it forked here: https://github.com/emlynmac/class-transformer
It's allowed me to move forward without digging into the code of class-transformer too much.

I experienced same problem. Will it be fixed in next version?

@emlynmac but hmm, does it still affects you?

Has anything changed? Yes I still am using my fork in my current projects

Hi @emlynmac and @lincanli!
I cannot reproduce your issue with 0.1.10. Here is how I tried:
import { Expose, plainToClass, classToPlain } from 'class-transformer';

class Example {
  @Expose({ name: 'coordsX' })
  x: number;

  @Expose({ name: 'coordsY' })
  y: number;
}
const plainToClassResult = plainToClass(Example, { coordsX: 5, coordsY: 3 });
const classToPlainResult = classToPlain(plainToClassResult);

console.log(plainToClassResult); // logs expected "Example {x: 5, y: 3}" class
console.log(classToPlainResult); // logs expected "{coordsX: 5, coordsY: 3}" object

I will go ahead and close this, please feel free to continue the discussion if you have any more question.


Missing mutation input fields for embedded relationships

If we supported embedded relationships on non-model GraphQL types, there should also be a way to insert that field or filter on it.
""" @model """
type Task {
  id: ID!
  version: Int
  title: String!
  description: String!
  status: String!
  """
  @db(type: 'json')
  """
  creationMetadata: CreationMetadata
}

type CreationMetadata {
  createdDate: String
  createdBy: User
}

There are no input fields for this embedded relationship.
input CreateTaskInput {
  id: ID
  version: Int
  title: String!
  description: String!
  status: String!
  creationMetadata: CreationMetadata # Let's add this
}

Awesome! Is that blocking?

Alpha release - no
0.14.0 - maybe. It depends on how much of a priority it is.
It is not a common use case in Postgres (still good to support it)
But I can see it being commonly needed in Mongo.
The data can be retrieved if it already exists, but if it cannot be inserted then it is almostr useless to have it as a feature.
Relates to https://github.com/aerogear/graphback/issues/1432

This is needed in 0.14.0 otherwise it is useless to have embedded relationships.


Ignore limit based on direction

This makes it so that panes don't expand when making the window narrower, and don't collapse when making the window wider. A potential solution to #2. I also updated the config files to the latest from the Obsidian Sample Plugin.

@mybuddymichael if you have the time I'd appreciate if you took a look at this.

There seems to be a decent number of changes that are unrelated to the problem stated in the PR text. As it is, I'm not going to accept this PR in its current form. If you want to reduce the code changes to exactly and only those that solve the problem stated in the PR, I can take a look again. If there are other problems with this repository, I encourage you to make separate PRs for those.

Hi!
I reduced the code changes.

@mybuddymichael it's been a while, would you take a look at this?

@mybuddymichael Please accept this PR. I'm gonna have to make a fork and submit it to community plugins to get this feature otherwise.

It's been a while. I made a fork that I'm in the process of adding to community plugins here: https://github.com/obsidianmd/obsidian-releases/pull/1063
Would you be open to transfering the existing plugin entry over to my repo? Or adding me as a maintainer to this repo? Or transferring this repo to me? I just want the PR to be available to the public.


Broadcast button not working

Describe the bug
I'm trying to make a multi-sig transaction, but when I have signed it with 2 devices the"broadcast" button gets enabled. But when I click it, nothing happens (I can click on it as much as I want)
Screenshot_20241028_211825_Keeper (1)
Smartphone (please complete the following information):

Device: Galaxy Z Flip 4
OS: Android 14
Browser: none (native app
Version [e.g. 22]

I believe this is already fixed in the next release, which should be out in the next few days. Please try again when it is out and let us know if the issue persists

I updated the app, now the button does something, now I get a message:
the transaction was rejected by network rules

Any suggestions what happend?
Also, something else, which worked better yesterday, was the progress of scanning the QR from my hardware wallet. Yesterday I saw progress, now nothing seems to happen while scanning. Then, after a couple of seconds, it just completes

About the Qr scanning status, can confirm the issue, we will fix it asap.
For the error you get, we've seen it once when signing with a wrong signer. Since you use multiple Jades, is it possible you may have confused between them and signed with one device a transaction of the other?

I have retested it, making sure that I do not mix the jades, but I still get the same error.
I also noticed that when I confirm with a jade, the green checkbox falls falls half off the screen

This error can also appear if for example the transaction amount is so small it falls below the network's dust limit. It's probably not the issue, but just want to confirm to be able to discard that as an option. The amount you send and the change amount are not below dust limit right?
If they aren't then I'm not sure what would cause it, but will try to recreate the scenario to see if I can replicate the issue.

Also if you could please share a screenshot of the checkbox falling off that'd be very helpful

Yep it works now when I increase the amount. So improving the error message would really help. Thanks for the support!

Thanks for confirming! I've created issue #5478 about better error message for the case to make sure we'll handle that.


Code won't exit

So after playing around with the motors, I realized that the code would never exit. I would get something like :

stepper control.Yaxis: action >enable< complete: disable.
stepper control.Xaxis: action >enable< complete: disable.
stepper control: action >wavepulsemaker< complete: all done in 1 waves.

After that though, the code would never exit. How can I solve this issue?

I hope you got this question because I am effectively stuck at this point as I cannot figure this part out. Thanks for your time.

There was already a shutdown in the motor driving thread, but nothing to call it from the command line interface class. I've added an exit() method to the clif class, so now you can cleanly shut down the threads (when the motors are idle) by clif.exit() (or equivalent in your own control class)

Ok thanks, it works now!


CAMEL-20562 - Camel-AWS-Bedrock: Support Mistral AI models - Mistral …

…7B Instruct
Description

Target

[x] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[x] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[x] I checked that each commit in the pull request has a meaningful subject line and body.

[x] I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

/component-test aws


add support for binaural features

Coverage remained the same at 98.358% when pulling 86c0f311dfde4122f4dd8b0772451c25aa1b009a on kashefy:twoears_binaural_feat into 8ce5a7013880d54161c30e88419c37e9bc31409c on nigroup:master.


Support 'thunk' from virtual-dom

It would be nice to be able to use SF to handle this somehow.
Ideally I would like to use an Eq s instance to generate appropriate calls to thunk.

Our use cases include purescript-ace, purescript-echarts, and probably others. Having a modular, clean, and robust way of using these pre-built DOM-native components is quite important.

The simplest approach seems to be something like adding a new constructor to HTML which prevents redraws using thunk:
data HTML i
  = NoRedraw (Unit -> HTML i)
  | Text ...

We could then extend that in some simple way, like adding a String identifier which could be stuck in a data- attribute and used to implement limited updates:
type Hash = String

data HTML i
  = Hashed String (Unit -> HTML i)
  | Text ...

If the hashes are equal, we assume the VTrees are equal and use the previous VTree. Otherwise, we rerender the HTML structure.

Also, @jdegoes, I'm not a vdom expert yet, but I feel like the "widget" concept might be useful for the examples you mentioned. I'll write up a separate issue for that.
https://github.com/Matt-Esch/virtual-dom/blob/master/docs/widget.md

I'd agree with that, thunk is for custom patching logic, widget is more for things like Ace.


Add flow, Windows: add downloaddir option, make installer cacheable

@frederic34
I see that you updated this project yesterday to correct the download link.
I've added cachability today.
I'll be a nice addition to speed up the action (after adding the directory to cache) and not put as much load on the download server.

@mdeweerd workflow on dolibarr get stuck on download of archive, rate limit ?

it was blocking on the install

Maybe there should be a partial mirror for dolibarr with a secret to download from it and download from it with an extra option to this action to set the head of the server url + part of the GET parameters where the value is "KEY=VALUE" to hold the token.
That with the cache option should avoid us the download issues (the installation could block because the download was partial or just indicating some rate limit was exceeded=.

the command line with msiexec was wrong https://github.com/frederic34/setup-mariadb/commit/59f279d4a8b8182433732e5ead8cb359af9909ff


feat: Invalid URL redirect 404 page

Hey @abhijeetnishal just wanted to know why is the vercel check failing

Hey @abhijeetnishal just wanted to know why is the vercel check failing

Hey @Abh-I-seK, I think the build is failing for client. Before pushing try to run command: npm run build it will show some error if builds fails.

I realized that there was an unresolved conflict in the navbar component.

I realized that there was an unresolved conflict in the navbar component.

I will fix it

Hey @Abh-I-seK, the text are not visible for light mode. I fixed it but try to see all cases.

sorry for the inconvenience, I've identified my mistake.
Thank you for letting me know.

Hey @Abh-I-seK, the text are not visible for light mode. I fixed it but try to see all cases.

sorry for the inconvenience, I've identified my mistake. Thank you for letting me know.

No problem 👍


Ingress controller crashes when kube API is unavailable

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
We experienced an issue where the (GCP / GKE 1.20) k8s API was unavailable for ~5 minutes due to some maintenance. To my surprise the Kong ingress controller crashed and stopped updating the kong proxy configuration once the k8s API returned. See the logs below. The controller stopped posting updates, so traffic started to slowly fail as pods were replaced/added/removed and getting new IP's.

Expected Behavior
I would expect the ingress controller to report the timeout/connectivity issues, but keep trying and eventually recover once the k8s API returns. Or perhaps the liveness probe should fail so the container gets restarted.
Steps To Reproduce
1. Deploy kong ingress controller
2. Make the k8s API server unavailable

Kong Ingress Controller version
Kong version 2.7, ingress controller version 2.1, installed using the helm chart (v2.6.4).

Kubernetes version
GKE 1.20

Anything else?
No response

Do you know of any means to replicate this reliably?
I attempted to create a single control+3 worker KIND cluster, deploy Kong and some Ingress to the workers, use docker pause on the control plane, unpause it after, and restart the Deployment backing the Ingress.
I did observe that KIC logging paused while the control plane was paused, which makes sense, as almost every action taken by the controller is triggered by an API server event, but after unpausing the controller began logging again and updated target IPs correctly.
If you exec into the proxy pod and check env output, what is KUBERNETES_SERVICE_HOST set to? If it's an IP, and you can trigger the same sort of maintenance event again on demand, does the cluster IP of your Kubernetes API service (usually kubernetes.default) change? That'd probably explain the issue if so, since the controller only obtains cluster configuration at startup. I'm not sure if KUBERNETES_SERVICE_HOST even updates automatically after a pod spawns.
This goes beyond my knowledge of cluster administration, but I suspect that should instead be the service hostname (kubernetes.default) if the IP of the API isn't static, though I'm unsure how where Kubernetes auto-populates that variable. It is possible to override it at the controller level by setting CONTROLLER_APISERVER_HOST.

Thanks for looking into this. It's pretty hard to reproduce unfortunately. I tried setting a NetworkPolicy that blocks outbound traffic from the kong pod, but haven't been able to reproduce in the same way as the original report. I think the key here is to trigger the timeout while kong is running. Getting a 'connection refused' or other tcp errors probably won't trigger this, but that's just a hunch. I'll keep trying.
KUBERNETES_SERVICE_HOST is set to an IP address (the address of the 'kubernetes.default' service). The IP of the service remains the same during the lifetime of the cluster (the service is not touched during maintenance). At least for our clusters this is the case (we use GKE).

What was actually done to trigger the outage? I assume it's not possible to actually repeat because GKE fully manages its control nodes, and you can't restart them unless you're upgrading their version. We also use GKE for some of our testing environments, however, and have not observed this problem when upgrading.
From some more research, it looks like that value may always be just the IP. However, if so, and the IP changing was indeed the reason the controllers did not recover, there are two questions I don't have a good answer for:

The client-go default behavior is to just use KUBERNETES_SERVICE_HOST, which seems odd if that's not a fault-tolerant choice for most environments. I didn't find any issues proposing that it needed to be changed.
The address is a Service address, so even if the API server restarts, it should remain the same. Destroying and re-creating the Service would re-assign it, but I'm not sure under what conditions that would occur.

The crash occurred during a (major) control plane upgrade of GKE. We have a single-zone cluster, so the control plane goes offline for a while during the upgrade while the master node gets replaced. I've been trying to reproduce this, also by triggering GKE upgrades, but no luck. I'm starting to think either something already got fixed, or this bug is very hard to trigger.
Regarding the API IP: I'm very sure that the API server IP never changes, at least in our cluster. I have old logs where the IP is logged, and it's still the same months later. Is seems the kubernetes.default ClusterIP service is created on cluster creation, and never changes during the lifetime of the cluster.
Since it's so hard / impossible to reproduce, let's close this for now. If it ever returns, I'll just re-open or create a new (hopefully more detailed) bug report.


[BK-1.15][FLINK-26931][Connector/pulsar] Make the producer name and consumer name unique in Pulsar

This is a backport PR for #19285.

@flinkbot run azure

@AHeise This is a backport PR. Can you review it?

Since this PR fixes an actual bug, I would expect to add a test case. Can you add one so that the issue does not arise again?

I think this is hard to test on integration level (you probably need to check on Pulsar server if a specific producer is registered) and the change is rather implicitly tested with other tests.
However, if there is an easy way to test, I'm also in favor of adding it.

Since this PR fixes an actual bug, I would expect to add a test case. Can you add one so that the issue does not arise again?

I think this could be hard to test. Because we just use the same producer name which is not allowed on the Pulsar. We just need to make sure it was unique.


Where did EventCapture go?

It's exported in src/index.js in master, but not in next.

Never mind, I see it was removed in commit 1f88499fa133f7f2d7312e2f2f10f3f8db46fac1 where the whole event subsystem was refactored.


fix(dependencies): Update dependency @gitbeaker/rest to v39.12.0

This PR contains the following updates:

Package
Type
Update
Change

@gitbeaker/rest
dependencies
minor
39.11.0 -> 39.12.0

Release Notes

jdalrymple/gitbeaker
v39.12.0
Compare Source
🐛 Bug Fix

@gitbeaker/core

Expanding typing for dynamic payload response functions #3386 (@jdalrymple)
Fixing pagination for allDiffs and allCommits methods #3388 (@jdalrymple)
Fixing statistics parameter on groups #3387 (@jdalrymple)

Authors: 1

Justin Dalrymple (@jdalrymple)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update testing docs with respect to simpler APIs

Remove the documentation's reference to catalogsourceconfigs in the install path

Create a Subscription , sourceNamespace should be "marketplace"
https://github.com/operator-framework/community-operators/blob/42475183a836473c847cacad1d207a828a5353b8/docs/testing-operators.md#8-create-a-subscription

@aneeshkp @awgreene @dmesser Thanks for the comments, updated to address everything. Please take another look.


perf: reduce memory footprint by not serializing all stats

We only serialized all stats to disk, so that users may be able to
inspect them with the webpack bundle analyzer.
But since we've introduced hops build --analyze-client-bundle the
full stats file is no longer necessary.

Bors merge bot cheat sheet
We are using bors-ng to automate merging of our pull requests. The following table provides a summary of commands that are available to reviewers (members of this repository with push access) and delegates (in case of bors delegate+ or bors delegate=[list]).

Syntax
Description

bors merge
Run the test suite and push to master if it passes. Short for "reviewed: looks good."

bors merge-
Cancel an r+, r=, merge, or merge=

bors try
Run the test suite without pushing to master.

bors try-
Cancel a try

bors delegate+
Allow the pull request author to merge their changes.

bors delegate=[list]
Allow the listed users to r+ this pull request's changes.

bors retry
Run the previous command a second time.

This is a short collection of opinionated commands. For a full list of the commands read the bors reference.

I tried this with our demo app and it fails with this error:
hops-demo:error TypeError: Cannot read property 'hops-language-en' of undefined
    at I18NServerMixin.fn (./packages/hops-i18n/mixin.server.js:15:32)
    at fn (./node_modules/hops-bootstrap/lib/utils.js:19:26)
    at reduce (./demo/src/index.js:52:12)
    at Array.reduce (<anonymous>)
    at fn (./demo/src/index.js:46:20)
    at pipe (./demo/src/index.js:111:15)
    at strategy (./demo/src/index.js:71:25)
    at o.getTemplateData (./node_modules/hops-bootstrap/lib/utils.js:16:7)
    at ReactMixin.fn (./node_modules/hops-react/render/mixin.server.js:32:17)
    at ./node_modules/hops-bootstrap/lib/utils.js:19:26

So I guess it's unfortunately not that easy

I have included the asset stats again to be backwards compatible, but in my opinion the i18n mixin should take care of registering its own webpack plugin if it needs access to certain stats.

bors merge

bors retry

bors merge