Failed to initialize keyboard hotkeys

The cumulative update preview for Windows 11 Version 23H2 (KB5041587) appears to have encountered an issue that failed to initialize keyboard.

Check if offsets are outdated then, and also make sure it's not paged out.


Release/2022 03 24 grey wolf

Try out this version of the Hiro Wallet - download extension builds.

Content:

https://github.com/hirosystems/stacks-wallet-web/pull/2312
https://github.com/hirosystems/stacks-wallet-web/pull/2277
https://github.com/hirosystems/stacks-wallet-web/pull/2313
https://github.com/hirosystems/stacks-wallet-web/pull/2304

cc/ @kyranjamie @fbwoolf @beguene @He1DAr

We need to make sure to switch the hiro-config file here to point to 'main' before merging.

This was found while testing the new home page and onboarding steps PR. It seems unrelated to those changes, but we should confirm this exists in production before shipping this release: https://github.com/hirosystems/stacks-wallet-web/issues/2328

Note: The GITHUB_PRIMARY_BRANCH on the latest build is set to main which is a breaking change. If we need to test this again we will have to test a previous build or change it back to dev temporarily.

@beguene on Monday, can you look at this issue and confirm if the buy options are working as you expect: https://github.com/hirosystems/stacks-wallet-web/issues/2328
*https://github.com/hirosystems/stacks-wallet-web/pull/2332#issuecomment-1079231383

FYI This https://github.com/hirosystems/stacks-wallet-web/pull/2313 should have been released before merging
https://github.com/hirosystems/stacks-wallet-web/pull/2312 as I mentioned here https://github.com/hirosystems/stacks-wallet-web/pull/2313#issue-1173464120
This is causing this issue https://github.com/hirosystems/stacks-wallet-web/issues/2328
If we release this grey-wolf PR it will fix https://github.com/hirosystems/stacks-wallet-web/issues/2328

@beguene on Monday, can you look at this issue and confirm if the buy options are working as you expect: #2328
*#2332 (comment)

Thanks @fbwoolf. We can simply release everything in this grey wolf PR and the issue https://github.com/hirosystems/stacks-wallet-web/issues/2328 will be solved.

FYI This #2313 should have been released before merging #2312 as I mentioned here #2313 (comment) This is causing this issue #2328 If we release this grey-wolf PR it will fix #2328

Ok, thanks, I did see that comment but it was already merged into dev, so at that point we should have just merged it into main separately? I did think that was the issue but wanted to confirm with you. 👍  Seems like we are good to release this today! Sorry for the confusion.


Add  Some New Quote

Hi, I want to add some more quotes under hacktoberfast.
Can you assign me this with hacktoberfast level.

kindly review my PR @Shariar-Hasan

Since this issue is not assigned to anyone.Can you assign me this issue?
I can add some more quotes to Quotes.js file

✅🆗

hello can you also assign me this issue i want to add some more quotes in this repo

🆗assigned

Hi, I want to add some quotes
Can you assign me under hacktoberfest?

🆗assigned

I have made the necessary changes and opened a pull request #15 .Can you check and merge it?

I have made the necessary changes and opened a pull request #15 .Can you check and merge it?

check comment in your pull request

Can i contribute into adding some quotes?

sure, give a start in the repo ⭐
fork the repo ✅
contribute & create PR

Assign this issue to me under Hacktoberfest, I will Add some of the quotes. Assign this issue to me.

sure,
assigned to you. but make sure to read the rules first

ad the rules first

okay


Dark mode w/ toggle switch

Added a dark mode theme, along wth some minor font tweaks.

Thank you so much! It's been merged, but not through github so I guess github doesn't understand it.

I've changed the default font to sans-serif and made it slightly smaller, just not as small as you suggested.


Supabase Does Not Provide Access Token To Storage

Describe the bug
Hey All,
Thanks for the communities work on the Python SDK!!
I recently tried using Supabase storage in the python backend of our app. After a day and a half of debugging, I think I have pegged the issue. It appears that the access token is not being provided to the storage client correctly.
I have checked that the RLS is correct. I compared this to JS and found this problem.
See below
To Reproduce
supabase: Client = create_client(
    supabase_url=self.supabaseUrl, supabase_key=self.supabaseAnonKey
)
response = supabase.auth.sign_in_with_password(
    {"email": self.supabaseStandEmail, "password": self.supabaseStandPassword}
)
session = response.session

# Due to a bug detailed here https://github.com/supabase-community/supabase-py/issues/185
# The postgrest client needs to be reinitialized with the auth token
postgrest_client = supabase.postgrest
postgrest_client.auth(session.access_token)

# I had to do this in order to get the storage to make the request correctly.
storageSessionDict = supabase.storage.session.__dict__
storageSessionDict["_headers"]["authorization"] = (
    "Bearer " + supabase.auth.get_session().__dict__["access_token"]
)

Expected behavior
I expect supabase.storage to just work like the JS SDK.
Thanks for the help! Let me know if i'm missing something.
Jack

Hi @jackeverydayzero, this is indeed a known issue. See #185 for a workaround until this is fully functional in python.

@tobias-scheepers I'll be honest mate, that didn't actually fully solve it for me. You can see the code I ended up using to solve it for the moment.
Definitely not ideal, probably doesn't refresh the token probably etc.
        print("Signing in to Supabase...")
        response = None
        try:
            response = supabase.auth.sign_in_with_password(
                {
                    "email": self.supabaseStandEmail,
                    "password": self.supabaseStandPassword,
                }
            )
            session = response.session
        except Exception as e:
            raise Exception(
                "Failed to sign in to Supabase. Please check your Supabase credentials.",
                e,
            ) from e

        # Due to a bug detailed here https://github.com/supabase-community/supabase-py/issues/185
        # The postgrest client needs to be reinitialized with the auth token
        postgrest_client = supabase.postgrest
        postgrest_client.auth(session.access_token)

        # It appears that the authorization token is not correctly provided to storage. This of course means that on refresh, this will not be updated.
        storageSessionDict = supabase.storage.session.__dict__
        storageSessionDict["_headers"]["authorization"] = (
            "Bearer " + supabase.auth.get_session().__dict__["access_token"]
        )

Fixed by #560


Cpp: Document Windows limitations and workarounds

This is to document that OmeroCpp on Windows is not going to work by default on Windows for 5.3, unless work is put in to build all the ice 3.5 bits by hand (same as 5.2 with ice 3.5).

@jburel I hope this is sufficient to document the limitations for 5.3 and retire the old jenkins jobs.

This renders fine on https://www.openmicroscopy.org/site/support/omero5.3-staging/developers/Cpp.html, can it be merged @jburel ?

using`Ice` 3.5 (requires building `Ice` ... end bracket is missing

Fixed now, thanks.


CORGI-726 keep products updated

Another attempt at fixing CORGI-726. I've added many tests this time and pushed the many-to-many remove calls down into the component level, which should make the changes more robust.
This is not very efficient where brew_tags are moved between streams because all of the builds related to the brew_tag will have their product taxonomy updated, and all relations deleted. The brew_tag will then be processed as if new in the new stream.
I need to add similar code for yum_repositories, errata_info, and composes. This only addresses the moving of brew_tags between streams, or removal from a stream.
I also added support from removal of a product_stream, product_version, or product from product_definitions.
@RedHatProductSecurity/corgi-devs please revew

I pushed another commit which makes use of the new remove_product_from_build task to clean up the CDN relations.

How many prod-defs models are there overall?

new_product_names = 110
new_version_names = 349
new_stream_names = 1276

How many Errata Tool variants are there overall?

cm.CollectorErrataProductVariant.objects.get_queryset().count() = 1935

How many Products / Versions / Streams are still defined in Corgi, but have been removed from prod-defs?

None, but that is on the horizon with prod-defs merge_requests/2234

Do any Streams have linked Variants (from Brew tags or ET Variant names) in Corgi, which don't match brew_tags or errata_info in prod-defs? In other words, do any Corgi streams still have a reference to some Variant, even though the Variant shouldn't exist due to updated prod-defs data?

In this commit [1] rhaos-middleware-rhel-8-container was incorrectly added to the cyrostat-2.0 ps_update_stream. I suspect that was cleaned up by the changes merged previously for CORGI-726.
The same situation would occur once prod-defs merge_requests/2234 was merged.
[1] baba3db65f560b801f1b68d8c5ff5c933cdd132c

How many streams have only brew_tags, how many have errata_info, and how many have both?

m.ProductStream.objects.exclude(et_product_versions=[]).count() = 46
m.ProductStream.objects.exclude(brew_tags={}).count() = 294
rhel-8.8.0.z is the only one which has both errata_info and brew_tags.

Do any streams in prod-defs reuse a Brew tag that's already defined on another stream? This would cause Variants (which Corgi creates from the Brew tag) to move back and forth between Streams.

This seems to be the main cause of the remaining data issues not resolved by this PR. Judging by the differences remaining between the Corgi models and prod-defs data generated by compare_variants.py script attached to CORGI-726.
example variant_differences:
{
    # Brew tag shared with another stream rhacm-2.7.z
    'rhacm-2.7': {'8Base-RHACM-2.7'},
    # Brew tag shared with another stream fdp-el7-default
    'fdp-el7': {'7Server-RH7-Fast-Datapath'},
    # Brew tag shared with multiple streams including jaeger-1.20.4
    'jaeger-1.20.3': {'8Base-JAEGER-1.20'},
    # Brew tag shared by multiple streams including openjdk-11
    'openjdk-17': {'7Server-OpenJDK-TextOnly'},
    # Brew tag shared with another stream devworkspace_operator-0
    'codeready_ws-2-default': {'8Base-DEVWORKSPACE-1.0'},
    # Brew tag shared with another stream rhacm-2.5.z
    'rhacm-2.5': {'8Base-RHACM-2.5'},
    # Brew tag shared by quarkus-2
    'quarkus-2.13': {'8Base-MW-RHBQ-2.13'},
    # Brew tag shared by pipelines-1.7.1
    'pipelines-1.7': {'8Base-PIPELINES-1.7'},
    # Brew tag shared by gitops-1.2.z
    'gitops-1.2': {'8Base-GitOps-1.2'},
    # Brew tag shared by openstack-13-els
    'openstack-13': {'7Server-RH7-RHOS-DEVTOOLS-13.0-ELS', '7Workstation-RH7-RHOS-TOOLS-13.0-ELS',
                  '7Server-RH7-RHOS-13.0-ELS', '7Workstation-RH7-RHOS-TOOLS-13.0', '7Server-RH7-RHOS-13.0',
                  '7Server-RH7-RHOS-TOOLS-13.0-ELS', '7Server-RH7-RHOS-DEVTOOLS-13.0',
                  '7Client-RH7-RHOS-TOOLS-13.0-ELS', '7Server-7-7.6.EUS-RH7-RHOS-13.0',
                  '7Server-RH7-RHOS-DEPLOYMENT-TOOLS-13.0', '7Client-RH7-RHOS-TOOLS-13.0',
                  '7Server-RH7-RHOS-DEPLOYMENT-TOOLS-13.0-ELS', '7Server-RH7-RHOS-TOOLS-13.0'},
   ....
}

In the prod-defs data, do any Brew tags for stream X match any ET variant names for stream Y? This would also cause Corgi Variants to move back and forth between Corgi Streams.

I don't think so, except for rhel-8.8.0.z which has both brew_tags and errata_info which overlap. The values in the Corgi data seem to come from the brew_tag variants, not errata_info.
There are some variants currently linked to dts-10.1.z, and dts-12.1.z which shouldn't be. Those should be skipped during parse_variants_from_brew_tags. That was also a change introduced in this PR. Which needs to be cleaned up.

The changes remaining here seems to do a good job of cleaning up the remaining CDN relations from components. I ran this last week in dev using product data and the deptopia-reconciliation script is now showing that we have less extraneous packages remaining. The ones that are remaining seem to be caused by CORGI-778

replaced by pull/567


[Event Request] Table 36 "Sales Header" - procedure UpdateSellToCust

Describe the request
procedure UpdateSellToCust(ContactNo: Code[20])
var
    ContBusinessRelation: Record "Contact Business Relation";
    Customer: Record Customer;
    Cont: Record Contact;
    CustomerTempl: Record "Customer Templ.";
    SearchContact: Record Contact;
    ContactBusinessRelationFound: Boolean;
    IsHandled: Boolean;
begin
    OnBeforeUpdateSellToCust(Rec, Cont, Customer, ContactNo);

    if not Cont.Get(ContactNo) then begin
        "Sell-to Contact" := '';
        exit;
    end;
    "Sell-to Contact No." := Cont."No.";
    OnUpdateSellToCustOnAfterSetSellToContactNo(Rec, Customer, Cont);

    if Cont.Type = Cont.Type::Person then
        ContactBusinessRelationFound := ContBusinessRelation.FindByContact(ContBusinessRelation."Link to Table"::Customer, Cont."No.");
    if not ContactBusinessRelationFound then begin
        IsHandled := false;
        OnUpdateSellToCustOnBeforeFindContactBusinessRelation(Cont, ContBusinessRelation, ContactBusinessRelationFound, IsHandled);
        IF not IsHandled THEN
            ContactBusinessRelationFound :=
                ContBusinessRelation.FindByContact(ContBusinessRelation."Link to Table"::Customer, Cont."Company No.");
    end;
    OnUpdateSellToCustOnAfterFindContactBusinessRelation(Rec, Cont, ContBusinessRelation, ContactBusinessRelationFound); //CustPub

    if ContactBusinessRelationFound then begin
        CheckCustomerContactRelation(Cont, "Sell-to Customer No.", ContBusinessRelation."No.");

[IntegrationEvent(false, false)]
local procedure OnUpdateSellToCustOnAfterFindContactBusinessRelation(SalesHeader: Record "Sales Header"; Cont: Record Contact; var ContBusinessRelation: Record "Contact Business Relation"; var ContactBusinessRelationFound: Boolean) //CustPub
begin
end;

Additional context
Publisher request marked with //CustPub tag.Internal work item: AB#494029

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 14714.


Utilize nameof operator

Minor PR, just catched a few missing nameof operators.

Hi @henkmollema, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.dotnetfoundation.org.

TTYL, DNFBOT;

@henkmollema, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, DNFBOT;

:shipit: we'll get this in later today

:shipit:

89b5f5b Thanks for the contribution!

:smile:


earnapp.py error

line 234

Errors present after these update

s

I have just carried out the test after updating the Kaspersky security profiles and on a second ISP. Same result on both machines...

@fazalfarhan01 I've found the issue. There needs to be an 'f' before the string in earnapp.py line 235, and the error, when the content is shown, is as follows.
pyEarnapp.errors.UnKnownError: Failed to get device statuses. Status code: 400 Response: b'Missing CSRF token'

+1 This is impacting me as well, any idea on when the fix will be released @fazalfarhan01 @Yariya  ?

@eraters thanks for working out the fix.
@linax101 expect a new update release in another day. (I have been busy, sry)

Having the same issue here, I have been using EarnApp for a while but haven't been checking my dashboard, so I just wanted something to remind me of how much I have instead having to refresh to see my earnings. But when trying to but it in my server it doesn't work. Hopefully this can get resolved!

Despite the "f" being added, it still does need that CSRF token.


Thanks Gopal for quick response. I tried creating Secret Key and use in the program ZP_PENGG_AZOAI_DEMO_COMPLETION. But what will be the input for other Selection Screen Parameters?

          Thanks Gopal for quick response. I tried creating Secret Key and use in the program ZP_PENGG_AZOAI_DEMO_COMPLETION. But what will be the input for other Selection Screen Parameters?

Otherwise If we have any demo program without Azure let me know.

Originally posted by @Dev4Develop in https://github.com/microsoft/aisdkforsapabap/issues/20#issuecomment-1653235325

Hi @Dev4Develop ,
For the Base URL and Key, please refer to this documentation
The Deployment ID would be the name you have given for a GPT-3 model deployment.  (eg: test-davinci-003)
You can give a token length of say, 1500
To see a demo video of this program, please see this Youtube Video.
NOTE : There are more video tutorials planned in near future.

What would be the version? I gave v1 and it was saying invalid version. ( Couldn't recollect the exact text )

As shown in screenshot.
Base URL here is azure related right? But my azure free trial has been disabled. What to give here?
Version is 2022-12-01
Key is Secret Key that I have created in Open AI directly.

Hi @Dev4Develop ,
If the azure trial is disabled, the Azure Open AI resource will not work - and you wont be able to run the application using Azure Open AI.
However, as I mentioned before, AI SDK for SAP supports open AI as well.
I will try and publish a video tutorial on the documentation site (Over the weekend) , which showcase

How to set up Open AI authorization key.
How to run the demo program using Open AI

Best Regards,
Gopal Nair.

Sure, that helps Gopal.

Hi @Dev4Develop ,
Please use this link for Open AI Initial Setup
Once set up, please see this documentation page to perform a test run.
For Open AI, the model name is the deployment ID. Also, the version is "v1", as shown in the test run.
Please let me know how it went.
Best Regards,
Gopal Nair.

Hi Gopal,
Firstly, I couldnt see ZP_MSAISDKDEMO_MODELS_OAI in my list.
I downloaded repository on Jun 08th

So I chose ZP_MSAISDKDEMO_MODELS.

I gave below input

Below is the response

Hi @Dev4Develop ,
It seems, the incorrect demo program is used here.

Please open the documentation page for Models.

Switch to "Open AI" mode in the documentation.

You will notice that, the entire documentation site has now changed to "Open AI" relevant information. This includes, the demo program name as well. The demo program name for Open AI Models is : ZP_MSAISDKDEMO_MODELS_OAI

As you can see in the documentation, the version for Open AI is "v1".

Please let me know if you run into further issues.
Best Regards,
Gopal Nair.

Hi @Dev4Develop ,
I missed the first part of your message, where you mentioned, you could not find the program "ZP_MSAISDKDEMO_MODELS_OAI" in your system.
Could you please try pulling the "main" branch again, so that the latest code is pulled and installed into your system?
Please let me know how it goes.
Best Regards,
Gopal Nair.

Hi Gopal,

I tried ZP_MSAISDKDEMO_MODELS_OAI.

It says "Version Not Supported"

Then I tried ZP_MSAISDKDEMO_COMPLETION_OAI

same message

Hi @Dev4Develop ,
This is interesting. I double checked by running the model demo program for Open AI, with version "v1" (The default that comes up), and everything worked OK.
How about we meet at some time, so that I can look in your system?
Best Regards,
Gopal Nair.

Sure. We can check. Our Demo system is available only on Tue, Wed, Thursdays.
Will be better if we can connect some time today or else we have to wait for next week.


Register Sovrin MainNet issuers with Sovrin MainNet instance of the BC Endorser service

Sovrin MainNet instance of the BC Endorser service:

aries-endorser-agent-sovrin

Register the following issuers with the Sovrin MainNet instance of the BC Endorser service:

[ ] agent-primary - BC Registries Issuer Agent
[ ] mines-permitting-agent-primary
[ ] ghg-emissions-agent-primary
[x] agent-lsbc
[ ] agent-lcrb
[ ] agent-buybc

Cleanup

[ ] Ensure the agents are demoted to the role of author once they have been registered with the endorser service.

Note: - Some agents may need to be upgraded to an ACA-Py version that supports the endorser protocols before they can be connected to the endorser service.  Refer to https://github.com/bcgov/DITP-DevOps/issues/18 for details.
Scripts to perform the agent registration can be found here; registerAuthor.

Blocked by https://github.com/bcgov/DITP-DevOps/issues/28

LSBC Agent has been connected to the endorser service and the agent's DID has been demoted to author; https://indyscan.io/tx/SOVRIN_MAINNET/domain/157681

@WadeBarnes Can we close this issue, since we are planning to migrate the agents?

Yes


Superset bug doesn't work dashboard

How to reproduce the bug
After clicking on the dashboard, I go back in to check after updating the chart, but the dashboard does not work shows that the chart that I was trying to add has been added, so it is not clicked, so I cannot modify it.

Hi @jo1013 , can you please fill out the bug template? In this case, a video might be helpful in helping us understand the issue that you're seeing. Thanks!
Template below:
A clear and concise description of what the bug is.
How to reproduce the bug

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected results
what you expected to happen.
Actual results
what actually happens.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment
(please complete the following information):

browser type and version:
superset version: superset version
python version: python --version
node.js version: node -v
any feature flags active:

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[ ] I have reproduced the issue with at least the latest released version of superset.
[ ] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Add any other context about the problem here.


Handling of more than one Heating Setups

Before submitting a new issue

[ X ] I redownloaded the latest release and restarted Home Assistant
[ X ] I've checked known issues
[ X ] I've enabled debug logs and attached the relevant logs to this ticket

Hello,
currently we have a myVaillant Account with more than one Heating Setups. (Means more than one sensoNET VR 921)
The Problem is that when i restart the mypyllant Integration (or HAOS itself) and one of those sensoNET VR 921 is Offline, i see an Error 404 in the Logs and none off my Heating Setups are shown in HA. (They are all Offline with no Data. But in Real Life only One is Offline...)
I hope you understand what i mean. :-)
Is there a way to change the Handling that this Error will be handled in another way?
Thanks and best regards.

Could you post debug logs, please? In theory, this integration should work with two controllers.

Off course, is that enough Data or should i leave Debugging on for  longer Time?
home-assistant_mypyllant_2024-05-02T06-39-05.397Z.log
Thanks and best regards

Today i tried to restart the myVaillant Integration. This Error came up:
Unexpected error fetching myVAILLANT data
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 315, in _async_refresh
    self.data = await self._async_update_data()
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/mypyllant/coordinator.py", line 169, in _async_update_data
    data = [
           ^
  File "/usr/local/lib/python3.12/site-packages/myPyllant/api.py", line 335, in get_systems
    control_identifier = await self.get_control_identifier(home.system_id)
                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/myPyllant/api.py", line 1068, in get_control_identifier
    return ControlIdentifier(control_identifier)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/enum.py", line 744, in __call__
    return cls.__new__(cls, value)
           ^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/enum.py", line 1158, in __new__
    raise ve_exc
ValueError: 'none' is not a valid ControlIdentifier

How does it look in the latest version? If the error persists, you would need to try reverse engineering to find out what API endpoints your setup uses.


Improve confirm-email-change page when user is now deactivated

I think probably we should just raise UserDeactivatedError and then open a follow-up issue for improving the error handling for deactivated accounts.  The current model is that most other things (login, etc.) display a notice inline in the page, rather than having a separate page, and it doesn't seem worth blocking this fix on making the notice for a very rare case pretty.

Originally posted by @timabbott in https://github.com/zulip/zulip/issues/20156#issuecomment-965746423

@zulipbot claim

Yes, I am still working on this.

@zulipbot abandon


Scan Page https://sara-sabr.github.io/ITStrategy/accueil.html for accessibility issues

Scan this page using axe to see if there are any accessibility issues

no issues found


docs & clear RL pkg

What does this PR do?
Cleaning reinforce and move docs
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Hello @Borda! Thanks for opening this PR.

In the file pl_bolts/models/reinforcement/double_dqn_model.py:

Line 127:121: E501 line too long (158 > 120 characters)

In the file pl_bolts/models/reinforcement/dqn_model.py:

Line 19:48: W605 invalid escape sequence '\L'
Line 19:73: W605 invalid escape sequence '\g'
Line 19:80: W605 invalid escape sequence '\m'
Line 19:89: W605 invalid escape sequence '\i'
Line 24:48: W605 invalid escape sequence '\L'
Line 24:75: W605 invalid escape sequence '\g'
Line 24:82: W605 invalid escape sequence '\m'
Line 24:91: W605 invalid escape sequence '\i'
Line 114:121: E501 line too long (160 > 120 characters)

In the file pl_bolts/models/reinforcement/noisy_dqn_model.py:

Line 19:121: E501 line too long (124 > 120 characters)

In the file pl_bolts/models/reinforcement/per_dqn_model.py:

Line 13:48: W605 invalid escape sequence '\L'
Line 13:84: W605 invalid escape sequence '\s'
Line 23:48: W605 invalid escape sequence '\L'
Line 156:121: E501 line too long (168 > 120 characters)

In the file pl_bolts/models/reinforcement/reinforce_model.py:

Line 70:121: E501 line too long (171 > 120 characters)

In the file pl_bolts/models/reinforcement/vanilla_policy_gradient_model.py:

Line 63:121: E501 line too long (164 > 120 characters)

Do see the Hitchhiker's guide to code style

Does it make sense to include the "results" section from the READMES as there are no plots ?

Codecov Report

Merging #71 into master will decrease coverage by 0.04%.
The diff coverage is 89.18%.

@@            Coverage Diff             @@
##           master      #71      +/-   ##
==========================================
- Coverage   87.87%   87.83%   -0.05%
==========================================
  Files          71       71
  Lines        3655     3642      -13
==========================================
- Hits         3212     3199      -13
  Misses        443      443

Flag
Coverage Δ

#unittests
87.83% <89.18%> (-0.05%)
:arrow_down:

Impacted Files
Coverage Δ

pl_bolts/models/rl/dqn_model.py
90.00% <ø> (ø)

pl_bolts/models/rl/reinforce_model.py
97.34% <ø> (ø)

...l_bolts/models/rl/vanilla_policy_gradient_model.py
97.69% <ø> (ø)

pl_bolts/models/rl/noisy_dqn_model.py
75.00% <75.00%> (ø)

pl_bolts/models/rl/__init__.py
100.00% <100.00%> (ø)

pl_bolts/models/rl/double_dqn_model.py
100.00% <100.00%> (ø)

pl_bolts/models/rl/dueling_dqn_model.py
100.00% <100.00%> (ø)

pl_bolts/models/rl/n_step_dqn_model.py
100.00% <100.00%> (ø)

pl_bolts/models/rl/per_dqn_model.py
85.96% <100.00%> (ø)

...bolts/models/self_supervised/amdim/ssl_datasets.py
43.10% <0.00%> (-1.44%)
:arrow_down:

... and 9 more

Continue to review full report at Codecov.

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

@djbyrne @williamFalcon I have moved all readme content to the modules so it can be included in docs

Does it make sense to include the "results" section from the READMES as there are no plots ?

it seems that the images were missing, not sure if they got lost in some past PR or were not committed at all, but it the time I started this PR there was no folder with images...
@djbyrne mind share it again? otherwise, are you fine with this change?

@Borda yes sorry I meant to ask do you guys want to store benchmark results of the models in this repo ? If so I can add the images folder

@Borda yes sorry I meant to ask do you guys want to store benchmark results of the models in this repo ? If so I can add the images folder

yes, add an image folder with results/illustrations to e.g. docs/source/_images/RL and try the images minimal so prefer jpg format and the resolution just to be fine, e.g. 600x600

@Borda it looks like this PR and #70 are on the same branch. I should have created a new one for my changes. Would it best to just merge this in as is, then create a seperate branch and PR for the results section?

@Borda it looks like this PR and #70 are on the same branch. I should have created a new one for my changes. Would it best to just merge this in as is, then create a seperate branch and PR for the results section?

I merged the other one as it is smaller and try to rebase this one... and then lets add images in separate PR?

yeah sounds good :+1:

yeah sounds good

@djbyrne done, mind check it and approve if you are fine with it :]


Add run to CI/CD

Summary
Run the main code in CI/CD to make sure it executes
Tests
https://github.com/SOMAS2021/SOMAS2021/actions/runs/1569363568

Logs too expensive to store. Will store with frontend instead where we can overwrite old logs or something similar

Thanks Jaafar.
I think the logging issue is more that we should probably utilise more logging levels rather than just info, but also determine what things we actually want to log and what should be put as a print statement

oh fuck yes


Failed to load url vitest.setup.ts

I see this every time I click Run(Vitest), am I doing something wrong? My vitest.setup.ts file exists in a different folder than the folder where the test lives, but there doesn't seem to be a config setting to change the setup file path.
FAIL  module.test.tsx [ module.test.tsx ]
Error: Failed to load url <obfuscated-path-to-project>/src/mapping/obf/vitest.setup.ts (resolved id: <obfuscated-path-to-project>/src/mapping/obf/vitest.setup.ts). Does the file exist?

I see this every time I click Run(Vitest), am I doing something wrong? My vitest.setup.ts file exists in a different folder than the folder where the test lives, but there doesn't seem to be a config setting to change the setup file path.
FAIL  module.test.tsx [ module.test.tsx ]
Error: Failed to load url <obfuscated-path-to-project>/src/mapping/obf/vitest.setup.ts (resolved id: <obfuscated-path-to-project>/src/mapping/obf/vitest.setup.ts). Does the file exist?

I get this error when there is an issue with compiling the test (wrong import / syntax etc).  Once it's cleared then the runner seems to run the test fine.


Bug: @xstate/cli shows predictableActionArguments warning despite option is set

Description
Given the following machine:
import { createMachine } from "xstate";

export const testMachine = createMachine({
  id: "testMachine",
  tsTypes: {} as import("./test-machine.typegen").Typegen0,
  context: {},
  initial: "initializing",
  states: {
    initializing: {},
  },
  predictableActionArguments: true,
  preserveActionOrder: true,
});

Running the following:
$ yarn run xstate typegen test-machine.ts

yarn run v1.22.19
warning package.json: No license field
$ /tmp/xstate-predictable/node_modules/.bin/xstate typegen test-machine.ts
It is highly recommended to set `predictableActionArguments` to `true` when using `createMachine`. https://xstate.js.org/docs/guides/actions.html
test-machine.ts - success

This shows the warning about predictableActionArguments although the option is set in the machine.
Expected result
Does not show a warning about predictableActionArguments.
Actual result
It shows the warning It is highly recommended to set predictableActionArgumentstotruewhen usingcreateMachine. https://xstate.js.org/docs/guides/actions.html.
Reproduction
https://github.com/phylor/xstate-predictableActionArguments
Additional context
No response

@Andarist Thank you very much for the quick fix! :tada:


213232 acc kind diagram

Hi @ClearlyEleanor , can you please review and approve this pull request?
I'll handle the merge.
Thanks!
Kat

CONFIDENTIAL
Thanks!
From: Eleanor Lovelock @.>
Sent: 31 August 2022 11:21
To: clearbank/clearbank.github.io @.>
Cc: Kat Kent @.>; Author @.>
Subject: Re: [clearbank/clearbank.github.io] 213232 acc kind diagram (PR #17)
⚠ CAUTION!
This email originated from outside of the organisation.
Do not click links or open attachments unless you recognise the sender and know the content is safe.
@ClearlyEleanor approved this pull request.
—
Reply to this email directly, view it on GitHubhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fclearbank%2Fclearbank.github.io%2Fpull%2F17%23pullrequestreview-1091648703&data=05|01|kat.kent%40clear.bank|eb518b9274294b2ea03d08da8b3a8973|c66e509ac81241ee9c00ae1517b0bb7e|0|0|637975380800152583|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=iN9OqAL47c1TUilbZDGuEf01%2Fj5p3wBaw4LEZ%2BC6YVM%3D&reserved=0, or unsubscribehttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAY52ELI4YNFMQR5NIV2DZFLV34WZVANCNFSM6AAAAAAQBHL5J4&data=05|01|kat.kent%40clear.bank|eb518b9274294b2ea03d08da8b3a8973|c66e509ac81241ee9c00ae1517b0bb7e|0|0|637975380800152583|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=X7deaLzYggs8VRDW3ww2giXMr%2FwOQnfHfqwhKO5n3YI%3D&reserved=0.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>Kat Kent
Technical Author Lead
[Clear.Bank]http://www.clear.bank/
Web | http://www.clear.bank/ LinkedIn | https://www.linkedin.com/company/clear.bank/ Twitterhttps://twitter.com/clear_bank
ClearBank® Ltd. Authorised by the Prudential Regulation Authority and regulated by the Financial Conduct Authority and the Prudential Regulation Authority (754568)
Disclaimer: This email (including attachments) is confidential and is solely intended for the addressee. Unless you are the addressee, you may not read, use or store this email in any way, or permit others to. If you have received it in error, please contact ClearBank® on +44 (0) 20 30 30 41 10. The views and opinions expressed within this message are unless otherwise specifically stated the views of the sender as a private person and should not be interpreted as being the views or opinions of ClearBank Ltd.
All reasonable precautions have been taken to ensure no viruses are present in this E-mail. ClearBank Ltd cannot accept responsibility for loss or damage arising from the use of this E-mail or its associated attachments. We recommend that you subject these to your own virus checking procedures prior to use.
Registered in England and Wales under Company Number 09736376. Registered office Level 4, 25 Marsh Street, Bristol, BS1 4AX


Missing config file

Hi,
I've tried to use the GUI on my conda deeplabcut environment. It launches correctly, I can select my input and output folders, but as soon as I try using the "Atlas to Brain" option, I get the following error message, even though I did change the paths on the config.yaml file to match my computer's. Do you know where this comes from, should I modify an existing file, or is something missing from my environment ?
Thanks in advance

Hi @brunoboehmm, thanks for reaching out! If you're using the default config file, it should not be necessary to change the filepath. It looks like your input images are inside your MesoNet git repository folder (MesoNet-master/mesonet) - this can potentially cause issues with filepaths. Could you try placing your input image folder outside of the MesoNet git repository (i.e. outside of MesoNet-master) and try again? Thanks!

Hi,
Thank you for your help! I tried this, but I still have the exact same error message...

Hi @brunoboehmm, could you try putting the DeepLabCut model folder (if you're using the default from OSF, it's atlas-DongshengXiao-2020-08-03 into a subfolder called dlc within the mesonet folder of the git repository? I realize the instructions in the wiki said to put it into the models folder directly; I may need to correct that. You can also do this process automatically by running get_default_models.sh within the mesonet folder.
If that doesn't help, I recommend you run test.py within the tests folder of the repository to verify that the code works properly on a sample dataset.
I hope this helps!

The test.py actually runs and is able to locate the landmarks (I get the "Landmark prediction complete!" display when I run it) but as soon as I select the Atlas to Brain or Brain to Atlas function on the GUI, I get the config error message.

I should be able to test the GUI today or tomorrow to see what is happening. In the meantime, if you have placed the default DeepLabCut model in the default location, you should not need to change the filepath to the model in the GUI - see if that works. Additionally, in the meantime the following Python code should replicate the atlas-to-brain method in the GUI:
import mesonet

config_file_atlas_brain = mesonet.config_project(input_file, output_file_atlas_brain, 'test',                                   atlas_to_brain_align=True, use_voxelmorph=False,               use_unet=True, use_dlc=True, sensory_match=False, mat_save=False, config=dlc_config, model=model)

# Identify outer edges of the cortex
  mesonet.predict_regions(config_file_atlas_brain)

# Identify and use cortical landmarks to align the atlas to the brain
    mesonet.predict_dlc(config_file_atlas_brain)

The first two arguments are your input and output folders, dlc_config is the path to your DeepLabCut config file, and model is the name of the default .hdf5 U-Net model that you selected in the list box on the right side of the GUI.
I hope this works for now! Thanks for your patience.

Hi @brunoboehmm, I believe I have resolved the issue. The behaviour of opencv's VideoWriter changed and I had to update the default codec used on Windows for writing the DeepLabCut video for landmark estimation (in df7323f4c8db10b1793d12571c0e1afe83a52f37). Please pull the latest code and install it; it should hopefully work then. I think that the DeepLabCut model folder location should also not matter with this change, so long as you select the config file in the GUI. Please let me know if you are still experiencing issues, and sorry for any inconvenience!

Hi,
Thanks a lot for all your help, I'm sorry I haven't answered earlier, I haven't had the time to test the GUI again since Friday. I uninstalled everything, reinstalled the latest version of MesoNet, selected the config file (the one from the DLC folder), but still have the same error message...

No problem! I've just tested it again and it works for me on Windows 10 when the DLC folder (e.g. atlas-DongshengXiao-2020-08-03) is placed outside of the MesoNet repo and the config file inside this folder is selected. Try copying and pasting the model folder outside of your MesoNet folder, then selecting the config file in that folder when using the MesoNet GUI..
I hope this helps!

Thank you for your quick answer and your help, unfortunately, even with this I still get the same error message. Just to make sure though, when you talk about the model folder, you talk about the atlas-DongshengXiao-2020-08-03, not the actual "models" folder containing the U-Net models? Because this folder has to stay in my MesoNet repo (otherwise the GUI can't find the model files...). Also, I have to manually change the "MesoNet git repo folder" path on the GUI as the default one is now weird (C:$Recycle.Bin\S-1-5-21-2549886845-264585227-397852783-218182$R3TOLBK\mesonet) since I deleted and reinstalled the MesoNet files. But here if I simply put the path of the latest version of the main folder (MesoNet) it should work, right?
Anyway thanks for you help! Hope you still have an idea for this...

No problem! By model folder I mean atlas-DongshengXiao-2020-08-03 - just move this folder outside of the git repo and use it from there, and don't move the models folder (the U-Net .hdf5 models can stay where they are).
Regarding the git repo path - make sure to fully delete the previous MesoNet folder in your recycle bin as it may be trying to use that folder instead. Although it should work if you manually change this git repo path, it's best to delete the old one just to be safe.
Hope this helps!

Thanks for the tips, this is indeed exactly what I did, but I just wanted to make sure. I also emptied the whole recycle bin, so the automatic paths now work correctly, but still I get the config error message.

Thanks for letting me know! If you've tried using the DeepLabCut model folder outside of the MesoNet repo, while not changing the automatic path in the DeepLabCut config file within this model folder, and it's still not working, I would recommend trying the command-line based method for now (the functionality will be the same; be sure to replace input_file, output_file, config, and model with the appropriate filepaths):
import mesonet

input_file = 'path/to/input/folder'

output_file =' path/to/output/folder'

config = 'path/to/dlc/config/file'

model = 'name of hdf5 model file for U-Net'

config_file = mesonet.config_project(input_file, output_file, 'test', atlas_to_brain_align=True, use_voxelmorph=False, use_unet=True, use_dlc=True, sensory_match=False, mat_save=False, config=dlc_config, model=model)

# Identify outer edges of the cortex
mesonet.predict_regions(config_file)

# Identify and use cortical landmarks to align the atlas to the brain
mesonet.predict_dlc(config_file)

I will continue to look into this issue on Windows. A few additional things I would recommend trying are:

Ensuring that the filepath under videos in the atlas-DongshengXiao-2020-08-03 config file looks similar to the first, automatically set filepath (this second filepath is not always set automatically; it should point to a .avi video within atlas-DongshengXiao-2020-08-03).
Deleting other versions of the folder atlas-DongshengXiao-2020-08-03 on your computer (including the one in the git repo).
Running the GUI using ipython (this the main environment in which it was tested).
Using MesoNet in a virtual environment (e.g. Anaconda or Miniconda), if you haven't already (in case the issue results from interfering software versions - which it shouldn't unless you have more than one instance of DeepLabCut on your computer, but just in case).
If you have more than one drive connected, checking that your model folder and MesoNet code are all installed on drive C: (we don't yet support other drive letters on Windows unless you manually specify the path to the git repository).

Thank you for your patience!

Closing issue due to inactivity.

Hi,
I ran into the same problem as @brunoboehmm. I've tried moving [atlas-DongshengXiao-2020-08-03] outside the MesoNet folder and setting the path to the video file correctly, but it's still not working. I'm still getting that 'config file is not found' error.
While troubleshooting, I ran test.py and got the error below. Also, based on reading the other issue threads, I suspect you might ask for the OpenCV version so I've included that here too.
In [18]: print(cv2. __version__)
4.7.0

In [13]: %run test.py
Creation of the directory /home/scott/Documents/MesoNet/mesonet/../tests/results/mesonet_output_atlas_brain failed
Creation of the directory /home/scott/Documents/MesoNet/mesonet/../tests/results/mesonet_output_brain_atlas failed
Creation of the directory /home/scott/Documents/MesoNet/mesonet/../tests/results/mesonet_output_sensory failed
Creation of the directory /home/scott/Documents/MesoNet/mesonet/../tests/results/mesonet_output_MBFM_U_Net failed
Creation of the directory /home/scott/Documents/MesoNet/mesonet/../tests/results/mesonet_output_voxelmorph failed
/home/scott/Documents/MesoNet/mesonet/models/DongshengXiao_brain_bundary.hdf5
1. Atlas-to-brain warp with U-Net and DeepLabCut
/home/scott/Documents/MesoNet/mesonet
/home/scott/Documents/MesoNet/mesonet/models/DongshengXiao_brain_bundary.hdf5
2023-02-20 15:28:58.933354: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libcufft.so.10'; dlerror: libcufft.so.10: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /home/scott/anaconda3/envs/DLC/lib/python3.8/site-packages/cv2/../../lib64:
2023-02-20 15:28:59.026245: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libcusparse.so.11'; dlerror: libcusparse.so.11: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /home/scott/anaconda3/envs/DLC/lib/python3.8/site-packages/cv2/../../lib64:
2023-02-20 15:28:59.026459: W tensorflow/core/common_runtime/gpu/gpu_device.cc:1934] Cannot dlopen some GPU libraries. Please make sure the missing libraries mentioned above are installed properly if you would like to use GPU. Follow the guide at https://www.tensorflow.org/install/gpu for how to download and setup the required libraries for your platform.
Skipping registering GPU devices...
2023-02-20 15:29:00.047118: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-02-20 15:29:00.084229: I tensorflow/compiler/mlir/mlir_graph_optimization_pass.cc:354] MLIR V1 optimization pass is not enabled
/home/scott/Documents/MesoNet/mesonet/../tests/sample_data/pipeline1_2
/home/scott/anaconda3/envs/DLC/lib/python3.8/site-packages/mesonet-<IP_ADDRESS>-py3.8.egg/mesonet/predict_regions.py:93: UserWarning: `Model.predict_generator` is deprecated and will be removed in a future version. Please use `Model.predict`, which supports generators.
/home/scott/anaconda3/envs/DLC/lib/python3.8/site-packages/keras/engine/training_v1.py:2356: UserWarning: `Model.state_updates` will be removed in a future version. This property should not be used in TensorFlow 2.0, as `updates` are applied automatically.
  updates=self.state_updates,
Lossy conversion from float32 to uint8. Range [0, 1]. Convert image to uint8 prior to saving to suppress this warning.
Lossy conversion from float32 to uint8. Range [0, 1]. Convert image to uint8 prior to saving to suppress this warning.
Output cortex masks can be found in `results/mesonet_output_atlas_brain/output_mask`
2
/home/scott/Documents/MesoNet/mesonet/../tests/sample_data/pipeline1_2/0.png
/home/scott/Documents/MesoNet/mesonet/../tests/sample_data/pipeline1_2/1.png
OpenCV: FFMPEG: tag 0x4745504d/'MPEG' is not supported with codec id 2 and format 'mp4 / MP4 (MPEG-4 Part 14)'
OpenCV: FFMPEG: fallback to use tag 0x7634706d/'mp4v'
---------------------------------------------------------------------------
FileNotFoundError                         Traceback (most recent call last)
File ~/Documents/MesoNet/tests/test.py:97
     94 print('Output cortex masks can be found in `results/mesonet_output_atlas_brain/output_mask`')
     96 # Identify and use cortical landmarks to align the atlas to the brain
---> 97 mesonet.predict_dlc(config_file_atlas_brain)
     99 print('Output aligned brain images and atlases can be found in `results/mesonet_output_atlas_brain/output_overlay`')
    101 ## 2. Brain to atlas
    102 # Brain-to-atlas warp with DeepLabCut

File ~/anaconda3/envs/DLC/lib/python3.8/site-packages/mesonet-<IP_ADDRESS>-py3.8.egg/mesonet/dlc_predict.py:373, in predict_dlc(config_file)
    371 flow_path = cfg["flow_path"]
    372 coords_input_file = cfg["coords_input_file"]
--> 373 DLCPredict(
    374     config,
    375     input_file,
    376     output,
    377     atlas,
    378     sensory_match,
    379     sensory_path,
    380     mat_save,
    381     threshold,
    382     git_repo_base,
    383     region_labels,
    384     landmark_arr,
    385     use_unet,
    386     use_dlc,
    387     atlas_to_brain_align,
    388     model,
    389     olfactory_check,
    390     plot_landmarks,
    391     align_once,
    392     original_label,
    393     use_voxelmorph,
    394     exist_transform,
    395     voxelmorph_model,
    396     template_path,
    397     flow_path,
    398     coords_input_file,
    399 )

File ~/anaconda3/envs/DLC/lib/python3.8/site-packages/mesonet-<IP_ADDRESS>-py3.8.egg/mesonet/dlc_predict.py:155, in DLCPredict(config, input_file, output, atlas, sensory_match, sensory_path, mat_save, threshold, git_repo_base, region_labels, landmark_arr, use_unet, use_dlc, atlas_to_brain_align, model, olfactory_check, plot_landmarks, align_once, original_label, use_voxelmorph, exist_transform, voxelmorph_model, template_path, flow_path, coords_input_file)
    152                 out.write(i)
    153             out.release()
--> 155             deeplabcut.analyze_videos(
    156                 config, [video_output_path], videotype=".mp4", save_as_csv=True
    157             )
    158             deeplabcut.create_labeled_video(config, [video_name], filtered=True)
    159 #             if "2." in deeplabcut.__version__:
    160 #                 scorer_name = "DeepCut"
    161 #             else:

File ~/anaconda3/envs/DLC/lib/python3.8/site-packages/deeplabcut/pose_estimation_tensorflow/predict_videos.py:487, in analyze_videos(config, videos, videotype, shuffle, trainingsetindex, gputouse, save_as_csv, in_random_order, destfolder, batchsize, cropping, TFGPUinference, dynamic, modelprefix, robust_nframes, allow_growth, use_shelve, auto_track, n_tracks, calibrate, identity_only, use_openvino)
    484 tf.compat.v1.reset_default_graph()
    485 start_path = os.getcwd()  # record cwd to return to this directory in the end
--> 487 cfg = auxiliaryfunctions.read_config(config)
    488 trainFraction = cfg["TrainingFraction"][trainingsetindex]
    489 iteration = cfg["iteration"]

File ~/anaconda3/envs/DLC/lib/python3.8/site-packages/deeplabcut/utils/auxiliaryfunctions.py:207, in read_config(configname)
    204                 raise
    206 else:
--> 207     raise FileNotFoundError(
    208         "Config file is not found. Please make sure that the file exists and/or that you passed the path of the config file correctly!"
    209     )
    210 return cfg

FileNotFoundError: Config file is not found. Please make sure that the file exists and/or that you passed the path of the config file correctly!

Hi @scottkil, thanks for reaching out and for using DeepLabCut! I've determined that this issue was caused by a default config filepath setting for the automated pipelines in the GUI. I've just merged a change that should fix this (#84ebdf2). I've also updated the manual to mention the get_default_models.sh script (#14). Let me know if this fixes the issue for you!

Looks like it worked. Thanks @bf777. I appreciate the help, and I look forward to using MesoNet!

I'm glad to hear that it worked @scottkil - happy to help!


Pico ZSeries Support

Perform the necessary to implement the APIs used by the ZSeries (see here, https://github.com/blucey/picobrew-z-api/blob/master/Picobrew Z API.md).
Implementation details: likely due to all the changes forking the support APIs similar to zymatic and pro/c/s have done is best bet.

@chiefwigms this is a placeholder issue for what I'm working on. Go ahead and assign it to me.

Had my first connected brew session for testing today. Things went really smoothly... Only step that failed was the session creation. Likely one of the documented parameters in my API capture wasn't what I coded or vice versa.
Also after building out the entire interface I assumed that recipeIds were GUIDs based on the Postman collection I made for requests made within BrewHouse (online crafter and session log). Indeed they aren't when the machine communicates to the server and instead expects an Integer value in the JSON... Super lame, but easy to modify the ZSeriesRecipe to generate/persist both instead.
Something to pay attention to when we import from BrewHouse as we cant just rely on atomicly increasing integer values for the recipe.id.
Video of brew session will be uploaded unlisted on my YouTube channel and linked here when available.

I haven't don't anything to get the "order of recipes" forced. The machine asks for 20, not sure what happens when it receives more than that. We should figure that out as if you have 21 recipes and the device only syncs 20 to be user selectable there needs to be a button added and backend support for "bumping" so the intended recipe just shows top of the list.
This is the behavior of BrewHouse and BrewPen/BrewCrafter that have a "sync to device" option. I'd imagine this was an issue in Zymatic as well, obviously there is a workaround of renaming or relocating recipe files for our server to work, but best to just have "active recipe" lists which could be symlinked files to a "recipe archive". Eventually all this session and recipe data could potentially live in external storage (database, etc) ... But yeah don't really need that ATM.

As promised https://youtu.be/SuPRbfImims

have a WIP PR to the picobrew-z-api documentation project, https://github.com/blucey/picobrew-z-api/pull/2

https://github.com/chiefwigms/picobrew_pico/pull/16 adds this capability. Will need to document the need for ssl, but will track that separately as this project not supports the Z device API functionality and I'm already 5 brews in on it.

I have 2 Zs and have setup the server and noticed it is listening on port 80. I thought that I read that the Zs communicate via TLS (I assume on port 443). I have setup dnsmasq on my router and have confirmed that if I connect with my cell phone to http://picobrew.com I'm hitting my local server. Are there other requirements to get the Z talking to the local server? Thanks!

@daytongray yes right now this server when running locally doesn't contain a certificate for allowing https traffic. Right now if you want to use this solution with the Z you will need a tool like mitmproxy in between your Z and the python server.
reference for setting up mitmproxy on a raspberrypi -> https://www.dinofizzotti.com/blog/2019-01-09-running-a-man-in-the-middle-proxy-on-a-raspberry-pi-3/

Thank you for the link Trevor. I figured this was the case but thought I ask anyhow. I didn't know if a self-signed cert has been tested and if the Z would ignore the warnings. I'll go down the MitM path.

MITM proxy is what I have setup. The benefit I see with this tool is there is a web UI used to capture traffic (called "flows") both request and responses. Thus if there is something not supported by the local server switch mitmproxy back to picobrew.com proper and capture those flows and then build out that capability in our local server.
Long term there isn't much value in having mitmproxy so would likely just setup a self signed certificate here or definitely bundled in the raspberrypi image we build out.

During a break today I pushed up the python modules I use for intercepting traffic purposed for picobrew.com to a localhost hosted server (obviously can switch those properties out with how your network topography is setup).
https://github.com/tmack8001/picobrew-mitmproxy


@JuliaRegistrator register()

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/69801
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.9.8 -m "<description of version>" f8025dcf6ac90a0c5b29b28c80ddbe855de56783
git push origin v0.9.8

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/69893
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.9.9 -m "<description of version>" f258ba4f9cc148a638a16ccf2d9ee1a4a6dc0b53
git push origin v0.9.9


React on SIGINT

Description
Changes proposed in this pull request:

React on Ctrl+C when running busola in docker.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


breaking: serve public env dynamically, prevent use of dynamic env vars during prerendering

Fixes #10008. This does two things:

prevents the use of $env/dynamic/public while prerendering (use $env/static/public instead)
when landing on a prerendered page, populates $env/dynamic/public from a new dynamically rendered /_env.js module, instead of populating it with stale build-time values

TODO:

[x] allow %sveltekit.env.PUBLIC_WHATEVER% to continue working (seems reasonable that this would be an exception to the 'you can't use dynamic public env vars during prerendering' rule)
[x] nicer code
[x] I guess it also makes sense to prevent use of $env/dynamic/private during prerendering
[x] docs
[x] preload _env.js when prerendering if we detect that $env/dynamic/public is used somewhere in the app
[x] make _env.js configurable, via config.kit.env.publicModule
[x] provide an API for adapter-static (and other adapters, if they want) to generate the _env.js module at build time
[x] some tests, i guess

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[ ] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[ ] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

I just realized that the adapter-static case is a bit weird. So far we had minors for all the adapters to tell "hey we support SvelteKit 2.0 as well". adapter-static should also get that treatment for the current version, but that is swallowed by this major bump now. We should create a minor release for adapter-static from master to circumvent that.

Is there much value in an adapter-static minor? If the only new feature is SvelteKit 2 support, why not just do a major version?

True, if someone upgrades to SvelteKit 2 they might as well upgrade to adapter-static v3. We should add that to the migration though (auto-bump the adapter-static version)

I guess it also makes sense to prevent use of $env/dynamic/private during prerendering

I think this might be incorrect. Even with prerendering disabled it now prevents any handle fn in hooks.server.ts from making use of dynamic env values, which might be needed at runtime.
My specific use-case is using the @sveltejs/adapter-node node adapter, and the env defines the 'ROOT_DOMAIN' that the site is serving, for instance .example.com, so that requests to site-one.example.com can load the appropriate site context by site ID, and redirect to the site's "proper" custom domain if it has one set. When the same request is made via the custom domain, it loads the site based on the domain name instead and doesn't redirect.
The ROOT_DOMAIN isn't known at build time - the same docker image can be deployed to environments that have different values (testing, staging, production).
The build initially compiles OK but then errors at the pre-render stage even with pre-rendering disabled:
+layout.server.ts:
export const prerender = false

Build output:
vite v5.0.9 building SSR bundle for production...
✓ 88 modules transformed.

vite v5.0.9 building for production...
✓ 62 modules transformed.
.svelte-kit/output/client/_app/version.json                                  0.03 kB │ gzip:  0.05 kB
.svelte-kit/output/client/.vite/manifest.json                                2.66 kB │ gzip:  0.53 kB
.svelte-kit/output/client/_app/immutable/chunks/stores.OXeE6BI3.js           0.24 kB │ gzip:  0.17 kB
.svelte-kit/output/client/_app/immutable/nodes/1.a7topCIA.js                 0.79 kB │ gzip:  0.50 kB
.svelte-kit/output/client/_app/immutable/nodes/0.VoE1dcHf.js                 0.89 kB │ gzip:  0.58 kB
.svelte-kit/output/client/_app/immutable/chunks/preload-helper.0HuHagjb.js   0.91 kB │ gzip:  0.56 kB
.svelte-kit/output/client/_app/immutable/chunks/scheduler.k-kUyWhY.js        2.16 kB │ gzip:  1.02 kB
.svelte-kit/output/client/_app/immutable/nodes/2.YZTI5OQK.js                 2.48 kB │ gzip:  1.24 kB
.svelte-kit/output/client/_app/immutable/chunks/singletons.i1C8PUYA.js       2.92 kB │ gzip:  1.48 kB
.svelte-kit/output/client/_app/immutable/entry/app.WtRj-Ezm.js               5.20 kB │ gzip:  1.97 kB
.svelte-kit/output/client/_app/immutable/chunks/index.wJJws_a0.js            5.24 kB │ gzip:  2.21 kB
.svelte-kit/output/client/_app/immutable/entry/start.OF9vDRZY.js            28.00 kB │ gzip: 11.17 kB
✓ built in 404ms

node:internal/event_target:1006
  process.nextTick(() => { throw err; });
                           ^
Error: Cannot read values from $env/dynamic/private while prerendering (attempted to read env.HOST_DOMAIN). Use $env/static/private instead
    at Object.get (file:///Users/simon/dev/temp/sk-envs/.svelte-kit/output/server/index.js:3664:11)
    at file:///Users/simon/dev/temp/sk-envs/.svelte-kit/output/server/chunks/hooks.server.js:17:33
    at ModuleJob.run (node:internal/modules/esm/module_job:193:25)
    at async Promise.all (index 0)
    at async ESMLoader.import (node:internal/modules/esm/loader:526:24)
    at async Server.init (file:///Users/simon/dev/temp/sk-envs/.svelte-kit/output/server/index.js:3696:24)
    at async prerender (file:///Users/simon/dev/temp/sk-envs/node_modules/.pnpm/@sveltejs+kit@2.0.0_@sveltejs+vite-plugin-svelte@3.0.1_svelte@4.2.8_vite@5.0.9/node_modules/@sveltejs/kit/src/core/postbuild/prerender.js:102:2)
    at async MessagePort.<anonymous> (file:///Users/simon/dev/temp/sk-envs/node_modules/.pnpm/@sveltejs+kit@2.0.0_@sveltejs+vite-plugin-svelte@3.0.1_svelte@4.2.8_vite@5.0.9/node_modules/@sveltejs/kit/src/utils/fork.js:22:16)
Emitted 'error' event on Worker instance at:
    at [kOnErrorMessage] (node:internal/worker:290:10)
    at [kOnMessage] (node:internal/worker:301:37)
    at MessagePort.<anonymous> (node:internal/worker:202:57)
    at [nodejs.internal.kHybridDispatch] (node:internal/event_target:731:20)
    at exports.emitMessage (node:internal/per_context/messageport:23:28)

Node.js v18.10.0

It's possible to workaround it by adding tests for building before accessing the values, hopefully that helps anyone who has the same issue and finds this.

@CaptainCodeman this does seem to be an issue, see #11341.
was there a workaround? if so, could you share it in #11341 perhaps? I did not quite understand the building stuff you referred to


Remove Liquidity Nested - Proportional

Pending a new nested router contracts delivered by SC team which will support nested remove.

Similar to RemoveLiquidity we can update RemoveLiquidityNested to branch depending on protocol version.
Router will support exiting to leaf tokens only.


Slideshows only displays a single slide

The issue seems to have been confusion with master/main. Redoing the steps cleared up the issue.


Better positioning module

Maybe support for centerVertical and centerHorizontal.

Yeah,that's a good idea, I'll implement it soon, thx :)


Add synchronization around comment/pr checking

Shared state (on the PR) is being updated during these functions.  We should synchronize on that object

Thank you for this pull request! Please check this document for how the Jenkins project handles pull requests.


Ursine Polar dark shadow over text

Hi there.
I found a bug in Settings and dialogs tab. See images below.
Typora version: 0.9.73
Ursine version: 1.6.3
Tested in Linux and Windows 10

Thank you.

Ah, that's the new preferences window. Doesn't affect Mac, but looks like Windows & Linux needs some styling. I'm gonna get right on it when I can


Powershell 7 icon & other forks

Added Powershell 7 icon and merged 2 other forks

Thanks for PR, and for fixing some existing icons to proper resolution.


Remove: Discord.Discord.Development version 1.0.660

## .\Tools\YamlCreate.ps1 -SkipPRCheck -Mode 5 -PackageIdentifier Discord.Discord.Development -PackageVersion 1.0.660

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.0 schema?

Microsoft Reviewers: Open in CodeFlow

/azp run


CT static API support

This PR enables Tessera to support a CT Static API personality.
There's broadly two things which are necessary to be able to support that personality:

Formatting of EntryBundle is different and incompatible
tlog-tiles specifies a length/value scheme, whereas CT Static API specifies just value and relies on the fact that value in this case is self-describing.
Each log entry contains its own index in the log.
The SCT returned to the submitter contains the sequence number of the entry in the log in the CtExtensions field of the SCT struct. What's not obvious from the spec is that this must also be part of the Merkle leaf (otherwise compatibility with RFC6962 would be broken).
This means that we don't actually know the LeafData or its MerkleLeafHash until we've sequenced it.

Beyond enabling a CT Static API personality to be built, other large goals for this PR are:

To make it hard/impossible to build new transparency log ecosystems which attempt to use these patterns outside of CT.
Avoid "overly invasive" changes spreading throughout Tessera, instead containing them as close as possible to the personality:
Non-CT logs shouldn't have to pay a performance cost simply because the support exists
Contain changes to the sequencing side, leaving integration concerned only with managing Merkle specifics.

Toward #41

Codecov Report
Attention: Patch coverage is 23.23944% with 109 lines in your changes missing coverage. Please review.

Please upload report for BASE (main@81ca927). Learn more about missing BASE report.

Files
Patch %
Lines

ctonly/ct.go
0.00%
66 Missing :warning:

storage/gcp/gcp.go
47.05%
16 Missing and 2 partials :warning:

ct_only.go
0.00%
12 Missing :warning:

entry.go
43.75%
7 Missing and 2 partials :warning:

storage/queue.go
75.00%
2 Missing and 1 partial :warning:

storage/integrate.go
50.00%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #73   +/-   ##
=======================================
  Coverage        ?   43.55%
=======================================
  Files           ?       14
  Lines           ?     1102
  Branches        ?        0
=======================================
  Hits            ?      480
  Misses          ?      550
  Partials        ?       72

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

Summarizing our conversation here for the record.
The logic makes sense to me, so I'll approve this PR.
We could improve naming though to make it easier to read the code:
There are 2 things called entry: tessera.Entry, and storage.entry. And storage.entry stores a tessera.Entry in a data field. Not to mention storage.SequencedEntry.
There are 3 things called index: tessera.Entry.internal.Index (an integer pointer), tessera.Entry.Index (a method interface), storage.Entry.index (a future method).


Package Jedis as an OSGi bundle

The Jedis jar isn't suitable for use in OSGi, but with the addtion of some simple manifest entries it could be e.g.
Manifest-Version: 1.0
Bundle-ManifestVersion: 2
Bundle-SymbolicName: org.jedis
Bundle-Version: 2.1.0
Export-Package: redis.clients.jedis;version="2.1.0",
 redis.clients.jedis.exceptions;version="2.1.0",
 redis.clients.util;version="2.1.0"
Import-Package: org.apache.commons.pool;version="[1.5,2)",
 org.apache.commons.pool.impl;version="[1.5,2)"

This is just an example. I would recommend using a tool like bnd to generate this metadata for you. It should be very easy to plug in to the existing gradle build.

closed by #920


commenting does not work

Making a comment does not work. Hitting 'POST' does not have any effect.

That's because your email address is not verified. Try going to your profile settings and click that button that resends the verification email.

Thanks, that did indeed resolve it.


Add labels on a per-log basis to Stackdriver upload via Log4net

Is your feature request related to a problem? Please describe.
I am using Log4net with the GoogleStackdriverAppender in a .NET framework project. I want to be able to add labels to our log entries for Cloud Monitoring.  I want to be able to group logs easily using these labels (which is what their purpose seems to be). However, the GoogleStackdriverAppender only allows static custom labels to be added that apply to all logs flowing through the Appender.
Describe the solution you'd like
Ideally there would be a way to attach labels to certain logs, and not just statically for all logs generated. I'm assuming this decision was made to try and "fit" Log4net nicely into the Stackdriver data model, but I would think there would be a way to accommodate per log labels.
Describe alternatives you've considered

Creating multiple appenders. (Seems like a terrible idea?)
Creating structured logs to create our own labels at the proto payload level. (Seems like a band aid fix)

I'll take a look at whether we can do this and how, but just to set expectations, I won't get to it for at least a couple of weeks.

Any updates? We are running into log query latency issues and using log labels would speed things up dramatically (because they are indexed).

@williecohen-google: Sorry, not yet - we'll try to have a look next week.

I'm working on this, will have an update early next week.

I believe you can already achieve this using properties, custom labels and patterns for custom labels.

Configure the GoogleStackdriverAppender to use patterns for custom labels by setting the UsePatternWithinCustomLabels property to true.
Add costume labels that use patterns to reference properties, like such:
{ "userName", "%property{userName}" }
Use any of the log4net GlobaContext, ThreadContext and LogicalThreadContext to set properties.

The only caveat is that if a property is not set, the pattern will return "(null)", so the label will always be set but with a "(null)" value in some instances. Either you filter out the ones with "(null)" values, or I we could consider adding a StripNullValuedLabels property to the appender so that we can remove such labels if they existed.
Let me know if this sounds reasonable to you.

Ok, I think this sounds reasonable. We have a default label, which we could set at GlobalContext to prevent null values, but it would probably be nice for other use cases to strip nulls out. I'll try this out and see if I run into any issues. Thanks!

Ok I was able to successfully add log labels with this! I do think for a follow up:

This should be documented somewhere (which maybe I missed).
Adding a StripNullValuedLabels property would be useful for adding labels to select logs.

Thanks for looking into this!

Thanks for confirming that this solves your issue.

I'll make sure to add some docs/samples around this.
We'll look int it.

I'll keep open until I've added the docs at least.

I've added documentation on how to translate Log4Net properties into labels. You can find it here.
As for the StripNullValuedLabels property, I've moved this issue to our backlog in #5628 to wait for more user demand.
I'll be closing this issue now since we've done as much as we are planning to do for now, but do please add another comment if you think there's something else we can do.


Add missing annotation to istio-ingressgateway service

Motivation
The Vendor = StreamNative tag was not propegating to the Istio Ingress Gateway resource through the AWS Load Balancer Controller. This PR addresses this by adding an annotation directly, rather than relying on default tags to get applied by the controller.
Also added a dependency on the AWS Load Balancer Controller during initial install.
Modifications
Describe the modifications you've done.
Verifying this change

[X] Make sure that the change passes the CI checks.

This change is a trivial rework / code cleanup without any test coverage.
Documentation
Check the box below.
Need to update docs?

[ ] doc-required
(If you need help on updating docs, create a doc issue)

[X] no-need-doc
(Please explain why)

[ ] doc
(If this PR contains doc changes)

LGTM


[Logging Improvement] Using lambda invocations instead of checking debug/trace isEnabled explicitly

Converted debug/trace/warn/info checks to lambda based logging APIs.
Present scenario:
if (logger.isTraceEnabled()) {
logger.trace("Some long-running operation returned {}", expensiveOperation());
}
Replace the existing checking of trace/debug logging before actual logging with lambda invocations within the debug/trace loggers:
logger.trace("Some long-running operation returned {}", () -> expensiveOperation());
This achieves the same lazy logging without having to check for logger levels.
Javadoc of "void debug(String message, Supplier<?>... paramSuppliers)" -> "Logs a message with parameters which are only to be constructed if the logging level is the DEBUG level."
Resolves #8646
Check List

[] ~New functionality includes testing.~
[x] All tests pass
[ ] ~New functionality has been documented.~
[] ~New functionality has javadoc added~
[ ] ~API changes companion pull request created.~
[ ] ~Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)~
[x] Commits are signed per the DCO using --signoff
[] ~Commit changes are listed out in CHANGELOG.md file (See: [Changelog]~(../blob/main/CONTRIBUTING.md#changelog))~
[ ] ~Public documentation issue/PR created~

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Why are these precommit tasks are not executing? Any issues from CI?

Why are these precommit tasks are not executing? Any issues from CI?

@akolarkunnu I think its probably because of merge conflicts. We can check CI once again once you resolve conflicts and address the code comments.

@akolarkunnu Did you get a chance to get to profiling and benchmarking?
It will be real nice to get these changes ahead of 3.0 release actually.


Follow react best practice and improve the separation of concerns

Here there are some points I would like to highlight

props in getInitialState is an Anti-pattern https://github.com/ubilabs/react-geosuggest/blob/master/src/Geosuggest.jsx#L38   -> https://facebook.github.io/react/tips/props-in-getInitialState-as-anti-pattern.html

To respect Single Responsibility Principle the input should have its own component and the same the list. This helps maintaining the code, replacing parts of the component and it could also improve component customization... For example would be easier to integrate my PR if all components were split correctly from the start. Decoupling is always better.

Google api services should not be part of the state. Instead they should be members of the component. State is meant for mutable data not for third party libraries. If you never see setState({foo:"bar"}) but you have "foo" defined  in your state you should start thinking you're doing something wrong ;)

As a convention I just prefer to have the render() method ALWAYS a the bottom, lifecycle methods at the start of the file in the order they are called, component methods that are exposed just after lifecycle methods and in the middle private methods prefixed with _. You'll notice that it's not just my convention but it's what most of the people use today. See for example how FB guys write their components https://github.com/facebook/fixed-data-table/tree/master/src

These stuff can be found in my PR https://github.com/ubilabs/react-geosuggest/pull/29

Will think about a better structure and take a look at your implementation.

I restructured the code to address this (finally).

list and input have their own components now
render is at the bottom
Google API is not part of the state
Except for the initialValue there is no prop as state. But that one doesn’t make sense to use as a prop in general, as that would mean that the user handles the essential input changes…


Make the ship image the way to attack other ships

... instead of an attack link. At Sidsnake's suggestion.
This will be an option, 'cause I kinda like it as it is. When enabled, we'll add a profile link instead of an attack one.
Note to self: this gives us a way to remove clutter - just make it possible to disable adding any visible links, but still make the image an attack link.

On second thought (years later haha), this won't be touched. It may confuse users who already know how to use the current UI; the current UI is pretty enough; and people who need speed are already using keyboard scripts.


MWPW-126724 - Support non-link text in copyright

Allow footer copyright to be authored as unordered list ul
If authored as ul, non-link text will remain visible in copyright
Maintain support for existing single line p authoring

Resolves: MWPW-126724
Test URLs:
New Authoring ul:

Before: https://main--milo--adobecom.hlx.page/drafts/methomas/test-footer-text?martech=off
After: https://methomas-footer-text--milo--adobecom.hlx.page/drafts/methomas/test-footer-text?martech=off

Existing Authoring p (for regression):

Before: https://main--milo--adobecom.hlx.page/drafts/methomas/checkmark?martech=off
After: https://methomas-footer-text--milo--adobecom.hlx.page/drafts/methomas/checkmark?martech=off

@meganthecoder Sorry, missed this in original overview. Is there a test to update?

@JasonHowellSlavin No, I couldn't find any footer tests

@fullcolorcoder I'm not sure I see the use in writing tests on code that's going to be rewritten soon. Our tests are highly dependent on the dom structure, and anything I wrote would likely be useless after the rewrite

@meganthecoder given the additional feedback, I don't think we should skip tests in the footer because it has wider impact and will remain valid through refactoring.

Failing test isn't for footer


Running --msa-mode single_sequence with --custom_template_path

I am trying to get localcolabfold (latest update) to run with custom templates
the Alphafold2.ipynb works properly with amber, custom_template, msa-mode = single_sequence and pair_mode=unpaired. I can tell that my template is working because the log file shows.
2022-07-18 21:38:39,138 Sequence 0 found templates: ['0005_L', '0005_H', '0005_H', '0005_L']
However I am not able to replicate this when using localcolabfold/colabfold_batch commands with
colabfold_batch --amber --templates --custom-template-path template/ --msa-mode single_sequence --pair-mode unpaired --use-gpu-relax --num-models 1 msas test2
No matter what I try I cannot get the "found templates" to seem to work.
The output  .a3m from the notebook looks like this
107,121	1,1
>101
DIQLTQSPSSLSASVGDRVSITCKASQDVSIAVAWYQQKPGKAPKLLIYSASYRYTGVPDRFSGSGSGTDFTLTISSLQPEDFAVYYCQQHYITPLTFGAGTKVEIK-------------------------------------------------------------------------------------------------------------------------
>102
-----------------------------------------------------------------------------------------------------------QVQLQQSGSELKKPGASVKVSCKASGYTFTNYGMNWVKQAPGQGLKWMGWINTYTGEPTYTDDFKGRFAFSLDTSVSTAYLQISSLKADDTAVYFCARGGFGSSYWYFDVWGQGSLVTVSS

But if I use this same file with colab_batch I get the error
2022-07-18 19:58:20,778 Could not generate input features 0: MSA 0 must contain at least one sequence.
I guess I dont entirely understand how this works in the notebook but doesn't seem to be working in localcolabfold. How is the MSA supposed to look for a single_sequence for proper input to custom templates? Thanks so much for your help. I have tried to find the answer on my own so far but its not clear to me. I have attached my cif file in case this is the problem, but it does seem to be working in the notebook. Thanks!
Drew
cif_files.zip

Did you manage to resolve this issue?


Flutter Library rendering requests BUG Report for list_tile

Steps to reproduce
Add the following code to simple project and run via
 body: const Center(
          child: RadioColumn(),
        ),

enum RowNumber { l, j, b }
class RadioColumn extends StatefulWidget {
  const RadioColumn({super.key});

  @override
  State<RadioColumn> createState() => _RadioColumnState();
}

class _RadioColumnState extends State<RadioColumn> {
  RowNumber? _character = RowNumber.l;

  @override
  Widget build(BuildContext context) {
    return Column(
      children: <Widget>[
        SizedBox(
          width: 100,
          height: 30,
          child: ListTile(
            //title: const Text('L'),
            leading: Radio<RowNumber>(
              value: RowNumber.l,
              groupValue: _character,
              onChanged: (RowNumber? value) {
                setState(() {
                  _character = value;
                });
              },
            ),
          ),
        ),
        SizedBox(
          width: 100,
          height: 30,
          child: ListTile(
            //title: const Text('J'),
            leading: Radio<RowNumber>(
              value: RowNumber.j,
              groupValue: _character,
              onChanged: (RowNumber? value) {
                setState(() {
                  _character = value;
                });
              },
            ),
          ),
        ),
        SizedBox(
          width: 100,
          height: 30,
          child: ListTile(
            //title: const Text('B'),
            leading: Radio<RowNumber>(
              value: RowNumber.b,
              groupValue: _character,
              onChanged: (RowNumber? value) {
                setState(() {
                  _character = value;
                });
              },
            ),
          ),
        ),
      ],
    );
  }
}

Expected results
I expected a small set of radio buttons with single letters as names. (I actually want just the radio buttons without text but controlling the layout seems to be a general problem). A number of similar issues appear on StackOverflow.
Actual results
UI appears but the following error message follows and the labels do not appear. This bug seems only to be repeatable on an intermittent basis and I am a struggling Flutter/Dart newbe.

 log

════════ Exception caught by rendering library ═════════════════════════════════
The following assertion was thrown during performLayout():
Leading widget consumes entire tile width. Please use a sized widget, or consider replacing ListTile with a custom widget (see https://api.flutter.dev/flutter/material/ListTile-class.html#material.ListTile.4)
'package:flutter/src/material/list_tile.dart':
Failed assertion: line 1318 pos 7: 'tileWidth != leadingSize.width || tileWidth == 0.0'

Either the assertion indicates an error in the framework itself, or we should provide substantially more information in this error message to help you determine and fix the underlying cause.
 Please use a sized widget, or consider replacing ListTile with a custom widget (see https://api.flutter.dev/flutter/material/ListTile-class.html#material.ListTile.4)
'package:flutter/src/material/list_tile.dart':
Failed assertion: line 1318 pos 7: 'tileWidth != leadingSize.width || tileWidth == 0.0'
The relevant error-causing widget was:
════════════════════════════════════════════════════════════════════════════════

════════ Exception caught by rendering library ═════════════════════════════════
Leading widget consumes entire tile width. Please use a sized widget, or consider replacing ListTile with a custom widget (see https://api.flutter.dev/flutter/material/ListTile-class.html#material.ListTile.4)
'package:flutter/src/material/list_tile.dart':
Failed assertion: line 1318 pos 7: 'tileWidth != leadingSize.width || tileWidth == 0.0'
The relevant error-causing widget was:
════════════════════════════════════════════════════════════════════════════════

════════ Exception caught by scheduler library ═════════════════════════════════
Updated layout information required for RenderConstrainedBox#53a52 relayoutBoundary=up1 NEEDS-LAYOUT NEEDS-PAINT to calculate semantics.
'package:flutter/src/rendering/object.dart':
Failed assertion: line 3649 pos 12: '!_needsLayout'
════════════════════════════════════════════════════════════════════════════════
Restarted application in 709ms.
Lost connection to device.

Exited.

Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

Sorry I am at the very early stages with Flutter but the ListTile Widget seems to be very fragile and subject to rendering issues, which is why it is wrapped in a SizedBox. Further, the support examples are quite limited with none showing how to recover which one is selected.

Leading widget consumes entire tile width. Please use a sized widget, or consider replacing ListTile with a custom widget (see https://api.flutter.dev/flutter/material/ListTile-class.html#material.ListTile.4)

@FreekyFrank Try to follow the hint in the log as above and also refer to the example link mentioned in it to properly implement the feature. It seems to be an issue with code implementation rather than a bug in framework.
If not, then provide us a complete runnable reproducible code sample without custom implementation that triggers the error you are seeing, along with your flutter doctor -v.


KAFKA-10515: Properly initialize nullable Serdes with default values

This is a cherry pick of PR #9338 on branch 2.6.
Also introduced the notion of WrappingNullableSerdes (aligned to the concept
of WrappingNullableSerializer and WrappingNullableDeserializer) and centralized
initialization in WrappingNullables.
The added integeration test KTableKTableForeignKeyJoinDistributedTest tests
whether all serdes are now correctly set on all stream clients.
Reviewers: John Roesler<EMAIL_ADDRESS>Committer Checklist (excluded from commit message)

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

Ah, it also looks like a bunch of tests fail due to EasyMock expectations being needed, and I also saw a couple of the null checks failling.

Hey, @mimaison , do you think we can get this into 2.6.1?

ok to test

test this please

... I'm not sure how to get Jenkins to test this PR.

@vvcephei Yes, feel free to merge in 2.6 once it's ready

Ah, it also looks like a bunch of tests fail due to EasyMock expectations being needed, and I also saw a couple of the null checks failling.

@vvcephei Actually I think that the tests are correct but the implementation incorrectly requires non null default serdes.

Also fixed this error in #9478 for trunk. Should also be cherry picked to 2.7. Otherwise, every Stream topology without a default serde will fail if it uses foreign key joins.
@vvcephei Can you review this and #9478? Thanks!

Ahh, I already commented on https://github.com/apache/kafka/pull/9478, but I think now I see the motivation for that PR.
If it was indeed just because there are internal tests that need new expectations now, we should just add the expectations instead of twisting the "main" code into handling an unexpected condition. By the way, I'm sure it's not obvious, but the fundamental reason for my taking this position is that in production code, the context serdes can never be null. There is a "default default" of ByteArraySerde even if you don't configure a "deafult" serde at all.

@vvcephei Sorry for the confusion, creating a new commit now.

Ready for review

@vvcephei feel free to rebase/merge this PR in any way that is needed. If I can be of any help, just let me know.

Finally merged that Jenkinsfile PR (seeing a lot of timeouts, but decided just to go ahead and merge it).
Merged in the 2.6 branch, and now the tests are running! We should hopefully get to merge this today.
Thanks again, @thake

Merged to 2.6 (cc @mimaison )
Thanks again for the contribution, @thake !


This is a suggestion: 'claimed' label

Would it be possible to create a 'claimed' label for those issues that have already been claimed and/or are in progress, to help keep track of what others are working?

Good idea, i'm on it !


Remove unused lockfiles

Some unused lockfiles are triggering dependabot alerts.

Codecov Report

Merging #882 (c93f582) into main (f17c05a) will decrease coverage by 40.21%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #882       +/-   ##
==========================================
- Coverage   48.88%   8.67%   -40.21%
==========================================
  Files         432     410       -22
  Lines       34337   32825     -1512
  Branches     7612    7214      -398
==========================================
- Hits        16784    2849    -13935
- Misses      12277   29175    +16898
+ Partials     5276     801     -4475

Flag
Coverage Δ

unittests-aries-vcx
8.67% <ø> (-40.17%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 323 files with indirect coverage changes


Setting program name for Oracle DB when using HikariCP with Spring

I am trying to connect to an Oracle DB with a specific v$session.program parameter.
My application.yml
spring:
  datasource:
    hikari:
      jdbc-url: jdbc:oracle:thin:@<db_host>
      driver-class-name: oracle.jdbc.OracleDriver
      username: <username>
      password: <password>
      data-source-properties:
        v$session.program: <name>

How would I configure data source properties using HikariCP?

Via Spring?  I do not know.  Pls ask on stackoverflow.com or the spring forums.

Thank you, I will ask there. How would I do it without Spring?

Try the readme

spring:
datasource:
hikari:
data-source-properties: v$session.program=AppName


Add externally visible cleanup package

Externalizes the cleaner function so that callers can cleanup everything if crashed.

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "",
    "component": "trireme-lib",
    "pr-id": "768",
    "commit-sha": "d31329d7a1491c9647b457635c0790bb277a5cbe"
  },
  {
    "project": "",
    "component": "enforcerd",
    "pr-id": "1099",
    "commit-sha": "c3293009f01188d6d931802589d3f1b10809eb37"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "",
    "component": "trireme-lib",
    "pr-id": "768",
    "commit-sha": "d31329d7a1491c9647b457635c0790bb277a5cbe"
  },
  {
    "project": "",
    "component": "enforcerd",
    "pr-id": "1099",
    "commit-sha": "61ebbb380bc9e605d3673d1d0c4750cea26a25ed"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "",
    "component": "enforcerd",
    "pr-id": "1099",
    "commit-sha": "ea64737824b1aa6c99cc350ea90b34d75c6f6031"
  },
  {
    "project": "",
    "component": "trireme-lib",
    "pr-id": "768",
    "commit-sha": "d31329d7a1491c9647b457635c0790bb277a5cbe"
  }
]


[BUG] show difference in error message projects

show a difference when you can't make a new project because of an old edition and when something went wrong when creating a project

How about not showing the create project button on an old edition?

Great idea, this isn't allowed anyways so no reason to show it.

Maybe not so simple. Even after removing the button the route is still available so users could go to the link.
Normally users can't do bad things because the backend wouldn't allow it. But maybe make something like an AdminRoute but for preventing you to go to routes of an old edition.

That's a bit broader so I moved it to a separate issue #365

yes makes sense


v22.8.41 update composer dependencies

Codecov Report

Merging #717 into master will not impact coverage.

@@           Coverage Diff            @@
##             master   #717    +/-   ##
========================================
  Coverage       100%   100%
- Complexity        0    187   +187
========================================
  Files            29     29
  Lines           439    439
========================================
  Hits            439    439

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 76ea09d...25e8cfc. Read the comment docs.


Unrelated combobox is operated when select a item of combobox

Expected Behavior
An item in the combobox is selected.
Actual Behavior
After correct behavior, an unrelated combobox is changed and expanded.
Short Example of Code to Demonstrate the Problem
I tried the following source by the NewControls in VCSamples.
VCSamples/VC2010Samples/MFC/Visual C++ 2008 Feature Pack/NewControls/

source
# launch the app
app = Application("uia").start(r"..\newcontrols.exe")
dlg = app.window(title_re = "NewControls")
# get combobox with resource ID
combobox = dlg.child_window(auto_id = '1002')
# select a combobox item
combobox.select('Image on right')

In this example, I try to change only the 'Image Location" combobox, but the "Appearance" combobox is operated and expanded without collapsing..
note

I used backend='uia', because I want to use resource ID of MFC.
When changing the source to select with number (ex.combobox.select(2)), the source cannot work.
After removing source under # do health check and apply workaround for Qt5 combo box if necessary in select() method of ComboBoxWrapper class, the program worked fine.

Specifications

Pywinauto version: 0.6.8
Python version and bitness: Python3.7(64bit)
Platform and OS: Windows 10

This is the first time I have written to the github.
So, if there is a problem in my writing, I would be glad if you could point out.

Thanks for reporting it! I'm planning to get back to combo box issues after returning to UI recorder feature. This may happen this year, but I can't guarantee exact deadline.

Thank you for your reply.
I'm happy to tell you the problem.
I will continue to use PyWinAuto.


Less-than-carets are removed when event is triggered

The less-than caret followed by text is removed when you e.g. set some text to bold in wysiwyg mode.
E.g: in the demo, write "<e", and change some other text to bold.
Is there any possibility to preserve these when typed in?

This issue comes from HTML special characters not being escaped properly in the WYSIWYG editor. Getting the textContent or innerText from the browser will give the unescaped version of the text. I've fixed the issue over at BlinkUX/woofmark and I'll try and see if I can't clean up a PR to merge upstream.


Remove MapsApi from DeleteExpiredData overload

This commit removes the MapsApiAttribute from the DeleteExpiredData overloads, so that the overload with a body are used for yaml tests

Failing tests are related to #4781. Merging this in.


Add opn module to open localURL in default browser

I'd like to keep it like it is 😊

Would you consider adding it as an option ? I'm also interested by this

I would accept a PR for piping the URL to the command: serve | open or something similar.


Overlay and CMake changes

You don't have to merge the PR, only for reference how the overlay would work and the cmake file

it does not build with -O2 but i honestly think this is a compiler bug.
If __attribute__((noinline)) is added to overlay_FLASH_ReadByAHB in sram-overlay.c it works. But i didn't push that is i don't understand the reason why it is needed


Use tradingview.com for universe creation

tradingeconomics.com provides component lists for all major indices, that can be scraped from their websites.

trading view.com provides a customizable stock screener with download option for stocks worldwide. Maybe this is a nice alternative, though it is not automatable..

Only implemented data ingestion. Universe building based on that data is still missing.

implemented with #52


Updated Project & Added NotificationFactory to Java implementation

Cleaned and updated the project. It compiles for ios and android.
Also added a NotificationFactory to the Java implementation

On first pass, this looks like a lot of good changes. Will try to fully review and merge this today or tomorrow when I have time.


How I can highlight selected text in the floating lsp

can i highlight this stuff

NormalFloat, FloatBorder may help but as you're using nvim-cmp I'd advise you look at the highlights it provides and customise them through this color scheme with custom highlight groups.