Change security classname properties to boolean

Make the instance.security properties boolean instead of a classname
This will remove security interfaces from the properties without
losing the functionality provided.
The instance.security.authenticator, instance.security.authorizor
and instance.security.permissionHandler properties used interfaces that
contain internal types and were not friendly for public use.

Until that comprehensive work is done, I think these should remain non-public API, and left as an advanced / marginally supported feature through the "internal" APIs. If it's "easy" to make the existing interfaces themselves public API / SPI, without exposing additional internal types, I'd be okay with that change also.

I made attempts at trying to modify them to fit our API or SPI.  The SPI may be more feasible but the main problem is their implementation relies heavily on access to internal ZooKeeper objects.  Most of the work the classes are doing is just interacting with zookeeper.  The Kerberos impl still uses the ZK impl but with added checks.  So I don't think there is an "easy" approach.

@milleruntime These interfaces were added in ACCUMULO-259 (I still have the JIRA issue memorized, because it was quite an ordeal at the time) to be pluggable. It was not merely added to support Kerberos/SASL authentication as an alternative. Rather, it was intended to be completely pluggable... so you could swap in an LDAP implementation, for example.

Do you remember if there was an actual customer need to have pluggable LDAP or something other than Kerberos?  Reading the JIRA tickets, it seems this was just a developer idea.  https://issues.apache.org/jira/browse/ACCUMULO-259.

Do you remember if there was an actual customer need to have pluggable LDAP or something other than Kerberos? Reading the JIRA tickets, it seems this was just a developer idea. https://issues.apache.org/jira/browse/ACCUMULO-259.

I have been informed in the past that people were doing this. However, I do not know if they still are. Either way, it is a capability I'd like to preserve, and improve upon, even if the API isn't stable. If we want to mark it as @Experimental in the interim, due to its limited support as non-public API, I think that might be our best option.

If we want to mark it as @Experimental in the interim, due to its limited support as non-public API, I think that might be our best option.

I agree.  Changed the title and pushed this change.  I added a comment next to the @Experimental tag to try and explain why the tag was added.


[Triton-MLIR][ROCM] Added cuda2gcn wrapper for devicelib.

Thereis cuda2gcn wrapper library in ROCm-Device-Libs' sources: https://github.com/RadeonOpenCompute/ROCm-Device-Libs/tree/amd-stg-open/cuda2gcn
I've just slightly fixed it (going to create appropriate PRs in its repo). And then have built from the source (the default ROCm-Device-Libs build doesn't create it).
It contains mappings and/or implementations for __nv* library functions. This step allowed to remove ifs in libdevice.py since they are not neeeded there anymore.
Also added some tests (that started to pass) to test_core_amd.py.

@dfukalov Could you help:

document how cuda2gcn.bc is built?
submit a PR to ROCm-Device-Libs on changes made to the source of cuda2gcn.bc?

Pls add a perf test for ops, e.g. exp, sin etc.
Just want to know the perf is on par too.

document how cuda2gcn.bc is built?

Added example script that builds the library.

submit a PR to ROCm-Device-Libs on changes made to the source of cuda2gcn.bc?

Submitted PR to fix cuda2gcn build


fixes to a few bugs/UI issues we saw today

Closes #93, #94, and #95.
-You can now enter household member SSNs with dashes.
-Patient name appears correctly on edit history and sharing pages.
-Document image links on local, non-heroku version work again.
In the household members section:
-The "Relation" column is now labeled "Relationship" as user suggested.
-The relationship in the patient row is now "Self" instead of "Patient" as user suggested.
-When the user hasn't entered a patient name yet, it appears as "Patient" rather than "None".
-When the user hasn't entered a patient SSN/DOB yet, they appear blank rather than "None".

adding the word "completion" to tag on landing page
adding new patient history records that state "share" and "receive" statuses between clinics


When overriding files at generation test.bal is created at root, not in the tests directory.

Description:
$subject
Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

This issue is duplicated with this https://github.com/ballerina-platform/ballerina-openapi/issues/631


Refactor git.HomeDir to strict mode (report fatal when it's not initialized correctly)

At the moment, it's impossible to make the git.HomeDir work in strict mode, for unknown reasons (ref: #19930)
If git.HomeDir calls log.Fatal when it's not initialized, some integration tests fails.
https://drone.gitea.io/go-gitea/gitea/55939/3/7
This problem should be fixed in new PRs, to make sure git module always be initialized correctly.
=== TestRepoLanguages (integrations/git_helper_for_declarative_test.go:95)
--- FAIL: TestRepoLanguages (1.65s)
    testlogger.go:78: 2022/06/11 04:50:14 ...eb/routing/logger.go:99:func1() [I] [62a41f06-15] router: completed GET /user2/repo1/_new/master/ for , 200 OK in 46.9ms @ repo/editor.go:184(repo.NewFile)
    testlogger.go:78: 2022/06/11 04:50:14 ...eb/routing/logger.go:99:func1() [I] [62a41f06-27] router: completed POST /api/internal/hook/pre-receive/user2/repo1 for <IP_ADDRESS>:57948, 200 OK in 7.8ms @ private/hook_pre_receive.go:106(private.HookPreReceive)
    testlogger.go:78: 2022/06/11 04:50:14 ...eb/routing/logger.go:99:func1() [I] [62a41f06-17] router: completed POST /user2/repo1/_new/master/ for , 303 See Other in 401.9ms @ repo/editor.go:344(repo.NewFilePost)
    testlogger.go:78: 2022/06/11 04:50:15 ...eb/routing/logger.go:99:func1() [I] [62a41f07] router: completed GET /api/v1/repos/user2/repo1/languages for , 200 OK in 6.5ms @ repo/language.go:48(repo.GetLanguages)
    api_repo_languages_test.go:48:
        	Error Trace:	api_repo_languages_test.go:48
        	            				git_helper_for_declarative_test.go:96
        	            				git_helper_for_declarative_test.go:91
        	            				git_helper_for_declarative_test.go:95
        	            				api_repo_languages_test.go:17
        	Error:      	Arguments must have the same number of keys
        	Test:       	TestRepoLanguages

        --- FAIL: TestGit/HTTP/PushCreate (1.29s)
            testlogger.go:78: 2022/06/11 04:54:05 ...eb/routing/logger.go:99:func1() [I] [62a41fed-8] router: completed GET /user2/repo-tmp-push-create-http.git/info/refs?service=git-receive-pack for <IP_ADDRESS>:58168, 401 Unauthorized in 3.9ms @ repo/http.go:531(repo.GetInfoRefs)
            testlogger.go:78: 2022/06/11 04:54:05 ...eb/routing/logger.go:99:func1() [I] [62a41fed-9] router: completed GET /user2/repo-tmp-push-create-http.git/info/refs?service=git-receive-pack for <IP_ADDRESS>:58168, 403 Forbidden in 200.4ms @ repo/http.go:531(repo.GetInfoRefs)
            testlogger.go:78: 2022/06/11 04:54:05 ...eb/routing/logger.go:99:func1() [I] [62a41fed-11] router: completed GET /user2/repo-tmp-push-create-http.git/info/refs?service=git-upload-pack for <IP_ADDRESS>:58170, 401 Unauthorized in 3.5ms @ repo/http.go:531(repo.GetInfoRefs)
            testlogger.go:78: 2022/06/11 04:54:06 ...eb/routing/logger.go:99:func1() [I] [62a41fed-12] router: completed GET /user2/repo-tmp-push-create-http.git/info/refs?service=git-upload-pack for <IP_ADDRESS>:58170, 404 Not Found in 200.4ms @ repo/http.go:531(repo.GetInfoRefs)
            testlogger.go:78: 2022/06/11 04:54:06 ...eb/routing/logger.go:99:func1() [I] [62a41fee-2] router: completed GET /user2/repo-tmp-push-create-http.git/info/refs?service=git-receive-pack for <IP_ADDRESS>:58172, 401 Unauthorized in 3.8ms @ repo/http.go:531(repo.GetInfoRefs)
            testlogger.go:78: 2022/06/11 04:54:06 ...eb/routing/logger.go:99:func1() [I] [62a41fee-3] router: completed GET /user2/repo-tmp-push-create-http.git/info/refs?service=git-receive-pack for <IP_ADDRESS>:58172, 200 OK in 209.2ms @ repo/http.go:531(repo.GetInfoRefs)
            testlogger.go:78: 2022/06/11 04:54:06 ...eb/routing/logger.go:99:func1() [I] [62a41fee-11] router: completed POST /api/internal/hook/pre-receive/user2/repo-tmp-push-create-http for <IP_ADDRESS>:58174, 200 OK in 8.0ms @ private/hook_pre_receive.go:106(private.HookPreReceive)
            testlogger.go:78: 2022/06/11 04:54:06 ...eb/routing/logger.go:99:func1() [I] [62a41fee-6] router: completed POST /user2/repo-tmp-push-create-http.git/git-receive-pack for <IP_ADDRESS>:58172, 200 OK in 573.4ms @ repo/http.go:499(repo.ServiceReceivePack)
            git_test.go:595:
                	Error Trace:	git_test.go:595
                	Error:      	Should be false
                	Test:       	TestGit/HTTP/PushCreate

--- FAIL: TestPullMerge (2.69s)
    testlogger.go:78: 2022/06/11 04:57:31 ...eb/routing/logger.go:99:func1() [I] router: completed POST /user2/repo1/pulls/6/merge for , 303 See Other in 613.0ms @ repo/pull.go:885(repo.MergePullRequest)
    pull_merge_test.go:86:
        	Error Trace:	pull_merge_test.go:86
        	            				git_helper_for_declarative_test.go:96
        	            				git_helper_for_declarative_test.go:91
        	            				git_helper_for_declarative_test.go:95
        	            				pull_merge_test.go:69

        	Error:      	"[%!s(*webhook.HookTask=&{1 1 1 uuid1 <nil>   true 0 1970-01-01 00:00:00 UTC false  <nil>  <nil>})]" should have 2 item(s), but has 1
        	Test:       	TestPullMerge

--- FAIL: TestPullRebase (2.70s)

    testlogger.go:78: 2022/06/11 04:57:31 ...es/pull/temp_repo.go:117:createTemporaryRepo() [E] [62a420bb-47] Unable to fetch origin base branch [user2/repo1:master -> base, original_base in /drone/src/integrations/gitea-integration-sqlite/data/integrations/gitea-integration-sqlite/tmp/local-repo/pull.git169687650]: exit status 128:

        	fatal: '/tmp/tmp.zvcOK53rAu/integrations/gitea-integration-sqlite/gitea-repositories/user2/repo1.git' does not appear to be a git repository
        	fatal: Could not read from remote repository.

        	Please make sure you have the correct access rights
        	and the repository exists.

    testlogger.go:78: 2022/06/11 04:57:31 ...rvices/pull/patch.go:64:TestPatch() [E] [62a420bb-47] CreateTemporaryPath: Unable to fetch origin base branch [user2/repo1:master -> base, original_base in tmpBasePath]: exit status 128

        	fatal: '/tmp/tmp.zvcOK53rAu/integrations/gitea-integration-sqlite/gitea-repositories/user2/repo1.git' does not appear to be a git repository
        	fatal: Could not read from remote repository.

        	Please make sure you have the correct access rights
        	and the repository exists.

    testlogger.go:78: 2022/06/11 04:57:31 ...rvices/pull/check.go:341:testPR() [E] testPatch[2]: Unable to fetch origin base branch [user2/repo1:master -> base, original_base in tmpBasePath]: exit status 128

        	fatal: '/tmp/tmp.zvcOK53rAu/integrations/gitea-integration-sqlite/gitea-repositories/user2/repo1.git' does not appear to be a git repository
        	fatal: Could not read from remote repository.

        	Please make sure you have the correct access rights
        	and the repository exists.

--- FAIL: TestPushDeployKeyOnEmptyRepo (8.13s)
   testlogger.go:78: 2022/06/11 04:58:50 ...eb/routing/logger.go:99:func1() [I] [62a4210a-11] router: completed GET /api/v1/repos/user2/deploy-key-empty-repo-1?token=037da467132d3d8120015afa8fad0f5d505cfcf1 for , 200 OK in 209.3ms @ repo/repo.go:502(repo.Get)
    --- FAIL: TestPushDeployKeyOnEmptyRepo/CheckIsNotEmpty (0.21s)
        ssh_key_test.go:25:
            	Error Trace:	ssh_key_test.go:25
            	            				api_helper_for_declarative_test.go:146
            	Error:      	Not equal:
            	            	expected: false
            	            	actual  : true
            	Test:       	TestPushDeployKeyOnEmptyRepo/CheckIsNotEmpty

FIXED!
The problem is in gitea post-receive hook, which calls git.NewCommand before git.Init. The failure message is not shown in CI logs, it's really difficult to debug ......


[FEATURE] Password Strength UI

Similar to other websites that have password strength UI when creating a user, the security dashboards plugin should have something similar.

A UI component that shows whether a given password is weak to very strong
Shows red/green progress bar as password gets stronger

First Step: - Don't allow users to create users with passwords that violate the regex rule specified in the config file
Second step: - Try to use https://github.com/dropbox/zxcvbn - or similar to match UI with what backend would return for the strength of a given password

[Triage] @derek-ho thank you for filing this issue. Going to tag UX/UI, and assign this issue to you ;)

I'm not sure if we have precedence for this kind of scenario, but approach 1 seems like a good candidate.
If you need another alternative, consider adding a parameter to the password setting API such as a 'dry-run' flag.  Then you can reuse whatever error comes from the backend system to populate the front end ❌, or a ✔

On a high-level, strong/weak indicators alone is rather ambiguous. We should provide upfront guidance on how to make their password strong, eliminating any guesses on users.
As users are typing in their passwords, I would consider specifically listing out which password criteria users are missing, so that users have clear actionable path.

@derek-ho

Two possible ways we can solve this, as @cwperks pointed out, rule/regex based regex seems to be at odds, or at least not 1:1 with library based password scoring.
Is there an issue that gives full context behind why we need a password strength UI? What did @cwperks pointed out? Thanks.

@kamingleung I think there is two things here -

Does the password meet the minimum criteria to be accepted by OpenSearch - should be addressed in another PR: #1581
Is the password considered strong enough to be accepted by OpenSearch - related to zxcvbn library. In order to implement this on the UI I am calling the same library the backend is calling to validate the strength of a password - however this doesn't have some specific set of standards as far as I can tell - it is based on how long the library thinks it would take to crack the given password, thus showing it as a progress bar. For example test is a very weak password, test12 is weak (better than test), but test123 is considered very weak again, even though test123 is more complex than test12. Outside of a strength bar I think this will be hard to communicate to users.


Changed agility plugin functionality to correctly account for xp on i…

Notes

Changed agility plugin functionality to correctly account for xp on intermediate obstacles.

This resolves a glitch where leveling up on intermediate obstacles does not recalculate laps left.

Changed List.transform function into JDK native stream function for HotColdClue.java.

This resolves bug outlined in issue runelite#6317.

Testing

Tested agility changes on my RS account by intentionally getting an agility level through intermediate obstacles. Confirmed that lap count is now updated.

The cause of the bug was that lapsTillNextLevel in AgilitySession.java was being updated only when experience gained was equal to the lap completion bonus xp: https://tinyurl.com/y4n8k72h. If the amount of xp gained was less than the last obstacle XP, the method would return and lapsTillNextLevel in AgilitySession.java would never be recalculated.
The changes in AgilitySession.java were minor style tweaks. The IncrementLapCount method was actually calculated laps till next level and incrementing lap count, so I separated them out for more clarity.

Looks like 2 checkstyle errors were thrown due to curly bracket placement and an unused import. Working on making those tweaks.


DatabaseFileLockedException

When rerunning the program, a DatabaseFileLockedException occurs. The IDE must be restarted in order for the program to function correctly again. Please find the cause of this and resolve the issue.

No longer needed because not implementing db4o anymore.


Format all

This PR adds make format and changes the check code style script to instead list the modules to ignore.

Jenkins test this please.

Closing in favour of new code_style branch.


Development

I fixed all of the issues with c2man using another Debian docker container and added a main script to build everything. I added some arguments to help with debugging.

Since this line was removed, it will fail the building of c2man on Fedora as well.
If this PR were merged, it will break the building process on Fedora, and it still might have problem on Debian.

Yes, you are correct, I will add a condition to fix this

I added the condition, for this PR I am fixing the c2man issue on docker, I can start working on fixing the native builds, and create a build script for Debian if you would like.

No, sir.
Adding an variable $USINGDOCKER shouldn't be count as a "fix".
I started to wonder if I was talking to an AI chatbot. Correct me if I were mistaken.

I suggest that any further development be fully discussed beforehand to avoid wasting your time.

No, sir.
Adding an variable $USINGDOCKER shouldn't be count as a "fix".
I started to wonder if I was talking to an AI chatbot. Correct me if I were mistaken.

You are being corrected, please explain what exactly you would like to do, I attempted to run a docker build in the current build, due to the fact that docker does not permit the use of systemd or dockerd, it is impossible to run both docker scripts at once.

You are being corrected

I'm sorry for this misunderstanding, thanks for your clarification.

please explain what exactly you would like to do

This script should work well on Debian using Docker, and should work well natively on Fedora.
So, if we are trying to modify anything, we should firstly ensure that 1) we don't break anything, 2) we don't solve anything which was already solved by using docker.
And I would like to keep this script as it is if the improvements break old things and add little value for new features.

This script should work well on Debian using Docker, and should work well natively on Fedora.

I'm sorry, my fault, it seems the current code failed to build on current Fedora as well, due to the problem caused by c2man.
https://github.com/VitoVan/pango-cairo-wasm/actions/runs/11756336008/job/32752377759#step:7:9800

Due to the old age of c2man, I have to fix the system version to Fedora 38.
The problem was fixed by 7e1372dd85ec55bc2a14e8ff38dc0ed55b8f772c

I'm closing this PR, because it doesn't fit the newly added contributing guide:
https://github.com/VitoVan/pango-cairo-wasm?tab=readme-ov-file#contributing
I'm sorry if you think I'm being a bad person.
But, anyway, thanks for your work.


Migrate Exchange Powershell to card based landing page.

Finish DEV TASK 249321
Following this guide: Create or update a landing page

Docs Build status updates of commit cc5ddc7:
:clock10: Full build: the average full build time is 17 min(s) 37 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit cc5ddc7:
:warning: Validation status: warnings

File
Status
Preview URL
Details

exchange/docs-conceptual/toc.yml
:warning:Warning
View (exchange-ps)
Details

exchange/docs-conceptual/index.md
:white_check_mark:Succeeded

exchange/docs-conceptual/index.yml
:white_check_mark:Succeeded

exchange/docs-conceptual/toc.yml

Line 0: [Warning-InvalidFileLink] Invalid file link:(~/docs-conceptual/index.yml).

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit cc5ddc7:
:clock10: Full build: the average full build time is 17 min(s) 37 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit cc5ddc7:
:warning: Validation status: warnings

File
Status
Preview URL
Details

exchange/docs-conceptual/toc.yml
:warning:Warning
View (exchange-ps)
Details

exchange/docs-conceptual/index.md
:white_check_mark:Succeeded

exchange/docs-conceptual/index.yml
:white_check_mark:Succeeded

exchange/docs-conceptual/toc.yml

Line 0: [Warning-InvalidFileLink] Invalid file link:(~/docs-conceptual/index.yml).

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 1d993a2:
:clock10: Full build: the average full build time is 17 min(s) 37 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 1d993a2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docfx.json
:white_check_mark:Succeeded

exchange/docs-conceptual/index.md
:white_check_mark:Succeeded

exchange/docs-conceptual/index.yml
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/toc.yml
:white_check_mark:Succeeded
View (exchange-ps)

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

Docs Build status updates of commit 06f2c06:
:clock10: Full build: the average full build time is 17 min(s) 40 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 06f2c06:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docfx.json
:white_check_mark:Succeeded

exchange/docs-conceptual/index.md
:white_check_mark:Succeeded

exchange/docs-conceptual/index.yml
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/toc.yml
:white_check_mark:Succeeded
View (exchange-ps)

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

Docs Build status updates of commit 40dc1d0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docfx.json
:white_check_mark:Succeeded

exchange/docs-conceptual/index.md
:white_check_mark:Succeeded

exchange/docs-conceptual/index.yml
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/toc.yml
:white_check_mark:Succeeded
View (exchange-ps)

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


Limit directory navigation

Hello,
I would like to suggest a feature that would help me a lot in a project that I am developing with Code Server and I think it could help some other people too.
Basically, I am running Code Server in my laptop and accessing it through Chromium in some Raspberry Pi's.
I am already specifying a directory path when running the code server command, but in the Raspberry Pi's I can access some previous directories by clicking in "Open Folder" and typing ".." in the navigation bar of the VSCode, as you can see in the pictures below .
When I click in "Open Folder":

When I type ".." in the navigation bar:

I would like to limit this navigation so the Raspberry Pi's can access only the specified directory or the posterior ones (the directories inside the specified directory). This specification could be through command line or the way you find it easier to implement.
Some infos that maybe will help:
I am running code server in Ubuntu 18.04 with this command:
console code-server / home / teacher / SOMAS / --auth none
I am accessing code server via Chromium in a Raspbian OS, but the same happens if I access it through Google Chrome in another laptop with Ubuntu 18.04.
If you need any further information, please let me know.
Thank you!

One way I think you could accomplish this is to use chroot.
Related: https://github.com/cdr/code-server/issues/1251. We opted not to do anything since from a security perspective it doesn't help since users still have access to the terminal, however maybe this make sense purely from a UX perspective.
Related: https://github.com/cdr/code-server/issues/1251


Where is the interaction point between OnModelCreating and the migrations?

I need to add custom SQL when migrating the database, but I don't want to mess with the migrations generated code, I rather doing it using the ModelBuilder in the OnModelCreating routine.
How can I interact between the ModelBuilder and the MigrationBuilder?
Related

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c13eb107-79c2-7824-12c1-e637d2a92add
Version Independent ID: aafc9673-c635-3cf2-6235-8577ff15d19c
Content: Custom Migrations Operations - EF Core
Content Source: entity-framework/core/managing-schemas/migrations/operations.md
Product: entity-framework
GitHub Login: @bricelam
Microsoft Alias: bricelam

I've tried overriding SqlServerMigrationsSqlGenerator.Generate but I can't find my custom annotations in the provided parameters.

See solution here.
Please document this scenario @bricelam.

Found answer. Posted here.
Please update docs tho.


iplot raises AttributeError: module 'plotly.offline' has no attribute '__PLOTLY_OFFLINE_INITIALIZED'

Minimal code example:
import cufflinks
import pandas as pd
pd.DataFrame([0]).iplot()

This raises AttributeError: module 'plotly.offline' has no attribute '__PLOTLY_OFFLINE_INITIALIZED'.
Using cufflinks 0.14.5 and plotly 3.3.0 (on Windows 10).
Can be solved (in a hacky way) by changing plotly.offline.__init__ to
"""
offline
======
This module provides offline functionality.
"""
from . offline import (
    download_plotlyjs,
    enable_mpl_offline,
    init_notebook_mode,
    iplot,
    iplot_mpl,
    plot,
    plot_mpl,
    __PLOTLY_OFFLINE_INITIALIZED  # <-- this line is added
)

Is there a reason that nobody responds to this issue? Am I the only one having it?

It's quite possible, I have not encountered this. Feel free to submit a fix, let me know if you need any help.

@mmrahn I can't reproduce with plotly 3.4.2 and 0.14.1

I had this

I also had this, used the same hacky solution.
Cufflinks 0.14.6, plotly 3.7.0, python 3.6.6

I had this too but its quite random.

i had this...

+1. Non-random. Consistently reproducible.

hello, thank you very much
I do it, but it does not work.
Do you konw why?

If you're still seeing this issue, please post the versions of Cufflinks and Plotly that you have installed :)

(and note that as of right now, the latest versions are Cufflinks==0.17 and Plotly==4.2.1 :)

thank you.
 My  versions are Cufflinks==0.17 and Plotly==4.2.1
It still does not work.o(╥﹏╥)o
------------------ 原始邮件 ------------------
发件人: "Nicolas<EMAIL_ADDRESS>发送时间: 2019年11月3日(星期天) 凌晨2:27
收件人<EMAIL_ADDRESS>抄送: "1016550222"<1016550222<EMAIL_ADDRESS>主题: Re: [santosjorge/cufflinks] iplot raises AttributeError: module 'plotly.offline' has no attribute '__PLOTLY_OFFLINE_INITIALIZED' (#151)
(and note that as of right now, the latest versions are Cufflinks==0.17 and Plotly==4.2.1 :)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Can you share some specific code that you're running that triggers this bug?

import pandas as pd
import cufflinks as cf
import numpy as np
cf.set_config_file(offline=True)
cf.datagen.lines(1,500).ta_plot(study='sma',periods=[13,21,55])
D:\Anaconda3\python.exe D:/pythonproject/untitled/1111.py Traceback (most recent call last):   File "D:/pythonproject/untitled/1111.py", line 7, in <module>     cf.datagen.lines(1,500).ta_plot(study='sma',periods=[13,21,55])   File "D:\Anaconda3\lib\site-packages\cufflinks\plotlytools.py", line 1656, in _ta_plot     return iplot(figure,sharing=sharing,filename=filename)   File "D:\Anaconda3\lib\site-packages\cufflinks\plotlytools.py", line 1460, in iplot     if offline.is_offline() and not online:	   File "D:\Anaconda3\lib\site-packages\cufflinks\offline.py", line 38, in is_offline     return py_offline.__PLOTLY_OFFLINE_INITIALIZED AttributeError: module 'plotly.offline' has no attribute '__PLOTLY_OFFLINE_INITIALIZED' 进程已结束，退出代码 1
------------------ 原始邮件 ------------------
发件人: "Nicolas<EMAIL_ADDRESS>发送时间: 2019年11月3日(星期天) 上午9:57
收件人<EMAIL_ADDRESS>抄送: "1016550222"<1016550222<EMAIL_ADDRESS>主题: Re: [santosjorge/cufflinks] iplot raises AttributeError: module 'plotly.offline' has no attribute '__PLOTLY_OFFLINE_INITIALIZED' (#151)
Can you share some specific code that you're running that triggers this bug?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hmm, that's interesting. If you add cf.go_offline() instead of cf.set_config_file(offline=True) does this still happen?

In fact, it still does not work.o(╥﹏╥)o
------------------ 原始邮件 ------------------
发件人: "Nicolas<EMAIL_ADDRESS>发送时间: 2019年11月3日(星期天) 晚上10:59
收件人<EMAIL_ADDRESS>抄送: "刘铭基"<1016550222<EMAIL_ADDRESS>主题: Re: [santosjorge/cufflinks] iplot raises AttributeError: module 'plotly.offline' has no attribute '__PLOTLY_OFFLINE_INITIALIZED' (#151)
Hmm, that's interesting. If you add cf.go_offline() instead of cf.set_config_file(offline=True) does this still happen?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

You get the same error or a different one? "it does not work" isn't sufficient information for me to help you :)

yes, the same error
import pandas as pd
import cufflinks as cf
import numpy as np
cf.go_offline()
cf.datagen.lines(1,500).ta_plot(study='sma',periods=[13,21,55])
D:\Anaconda3\python.exe D:/pythonproject/untitled/1111.py Traceback (most recent call last):   File "D:/pythonproject/untitled/1111.py", line 6, in <module>     cf.datagen.lines(1,500).ta_plot(study='sma',periods=[13,21,55])   File "D:\Anaconda3\lib\site-packages\cufflinks\plotlytools.py", line 1656, in _ta_plot     return iplot(figure,sharing=sharing,filename=filename)   File "D:\Anaconda3\lib\site-packages\cufflinks\plotlytools.py", line 1460, in iplot     if offline.is_offline() and not online:	   File "D:\Anaconda3\lib\site-packages\cufflinks\offline.py", line 38, in is_offline     return py_offline.__PLOTLY_OFFLINE_INITIALIZED AttributeError: module 'plotly.offline' has no attribute '__PLOTLY_OFFLINE_INITIALIZED' 进程已结束，退出代码 1
------------------ 原始邮件 ------------------
发件人: "Nicolas<EMAIL_ADDRESS>发送时间: 2019年11月3日(星期天) 晚上11:05
收件人<EMAIL_ADDRESS>抄送: "刘铭基"<1016550222<EMAIL_ADDRESS>主题: Re: [santosjorge/cufflinks] iplot raises AttributeError: module 'plotly.offline' has no attribute '__PLOTLY_OFFLINE_INITIALIZED' (#151)
You get the same error or a different one? "it does not work" isn't sufficient information for me to help you :)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hmm. I wonder if you don't have multiple versions of plotly installed, one with conda and one with pip perhaps? Could you try uninstalling with both tools and reinstalling with just one?

Thank you very much!
I will have a try and apply to you.
------------------ 原始邮件 ------------------
发件人: "Nicolas<EMAIL_ADDRESS>发送时间: 2019年11月3日(星期天) 晚上11:10
收件人<EMAIL_ADDRESS>抄送: "刘铭基"<1016550222<EMAIL_ADDRESS>主题: Re: [santosjorge/cufflinks] iplot raises AttributeError: module 'plotly.offline' has no attribute '__PLOTLY_OFFLINE_INITIALIZED' (#151)
Hmm. I wonder if you don't have multiple versions of plotly installed, one with conda and one with pip perhaps? Could you try uninstalling with both tools and reinstalling with just one?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

if you run with ipython, the error will gone.  i mean run script in comand "ipython script.py"

Thank you! But, it still does not work. o(╥﹏╥)o
------------------ 原始邮件 ------------------
发件人<EMAIL_ADDRESS>发送时间: 2019年11月18日(星期一) 下午2:54
收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS><EMAIL_ADDRESS>主题: Re: [santosjorge/cufflinks] iplot raises AttributeError: module 'plotly.offline' has no attribute '__PLOTLY_OFFLINE_INITIALIZED' (#151)
if you run with ipython, the error will gone.  i mean run script in comand "ipython script.py"
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

I also getting the same problem as @DesmondLMJ

Still the same problem with:
cufflinks          0.17.3       Productivity Tools for Plotly + Pandas plotly             4.6.0        An open-source, interactive graphing library for Python
You can get away with it by hacking the missing attribute in:
setattr(plotly.offline, "__PLOTLY_OFFLINE_INITIALIZED", True)
I did it at module scope after the cufflinks and plotly imports.

Thank you very much!
发自我的iPhone
------------------ Original ------------------
From: Damien Levac<EMAIL_ADDRESS>Date: Thu,Apr 9,2020 11:30 PM
To: santosjorge/cufflinks<EMAIL_ADDRESS>Cc: DesmondLiu<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [santosjorge/cufflinks] iplot raises AttributeError: module 'plotly.offline' has no attribute '__PLOTLY_OFFLINE_INITIALIZED' (#151)
Still the same problem with:
cufflinks          0.17.3       Productivity Tools for Plotly + Pandas plotly             4.6.0        An open-source, interactive graphing library for Python
You can get away with it by hacking the missing attribute in:
setattr(plotly.offline, "__PLOTLY_OFFLINE_INITIALIZED", True)
I did it at module scope after the cufflinks and plotly imports.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Eventually, I gave up using cufflinks on the PyCharm.
All is well on the Jupter notebook.
I decided to write the code on PyCharm and look at the diagram on the Jupter notebook.
Thank you all for your help and efforts!

Eventually, I gave up using cufflinks on the PyCharm.
All is well on the Jupter notebook.
I decided to write the code on PyCharm and look at the diagram on the Jupter notebook.
Thank you all for your help and efforts!

I'm having the same issue on Visual Studio Code 1.44.2

I feel sorry about that.
发自我的iPhone
------------------ Original ------------------
From: Orlando<EMAIL_ADDRESS>Date: Thu,May 7,2020 4:41 AM
To: santosjorge/cufflinks<EMAIL_ADDRESS>Cc: DesmondLiu<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [santosjorge/cufflinks] iplot raises AttributeError: module 'plotly.offline' has no attribute '__PLOTLY_OFFLINE_INITIALIZED' (#151)

Still the same problem with:
cufflinks 0.17.3 Productivity Tools for Plotly + Pandas
plotly 4.6.0 An open-source, interactive graphing library for Python
You can get away with it by hacking the missing attribute in:
setattr(plotly.offline, "__PLOTLY_OFFLINE_INITIALIZED", True)
I did it at module scope after the cufflinks and plotly imports.

Thanks, this workaround worked for me, using VS code.
>>> plotly.__version__
'4.9.0'
>>> cf.__version__
'0.17.3'

Without the line above, both running cf.set_config_file(offline=True) & cf.go_offline() gave me AttributeError: module 'plotly.offline' has no attribute '__PLOTLY_OFFLINE_INITIALIZED'
Complete code as follows
import pandas as pd
import cufflinks as cf
import numpy as np
import plotly

# setattr(plotly.offline, "__PLOTLY_OFFLINE_INITIALIZED", True)
cf.set_config_file(offline=True)
# cf.go_offline()
cf.datagen.lines(1, 500).ta_plot(study="sma", periods=[13, 21, 55])

I've also run into this issue now:
$ python --version
Python 3.9.0
$ pip list pandas
Package             Version
------------------- -------
...
cufflinks           0.17.3
ipython             7.19.0
ipython-genutils    0.2.0
ipywidgets          7.6.3
numpy               1.19.4
pandas              1.1.5
plotly              4.14.1
...

import pandas as pd
import psycopg2
from plotly.subplots import make_subplots
from plotly import graph_objects as go
from plotly import express as px
import cufflinks as cf

cf.set_config_file(offline=True, sharing='public')

pd.set_option("display.max_rows", None)
pd.set_option("display.max_columns", None)
pd.set_option("display.width", None)
pd.set_option("display.max_colwidth", None)

# ...

Error:
Traceback (most recent call last):
  File "/Users/jcope/Workspace/github.com/redhat-et/caliper/python/plotter/main.py", line 44, in <module>
    df_mem.iplot()
  File "/Users/jcope/Workspace/github.com/redhat-et/caliper/venv/lib/python3.9/site-packages/cufflinks/plotlytools.py", line 1216, in _iplot
    return iplot(figure,validate=validate,sharing=sharing,filename=filename,
  File "/Users/jcope/Workspace/github.com/redhat-et/caliper/venv/lib/python3.9/site-packages/cufflinks/plotlytools.py", line 1429, in iplot
    if not offline.is_offline() or online:
  File "/Users/jcope/Workspace/github.com/redhat-et/caliper/venv/lib/python3.9/site-packages/cufflinks/offline.py", line 38, in is_offline
    return py_offline.__PLOTLY_OFFLINE_INITIALIZED
AttributeError: module 'plotly.offline' has no attribute '__PLOTLY_OFFLINE_INITIALIZED'

I've also run into this issue now:
$ python --version
Python 3.9.0
$ pip list pandas
Package             Version
------------------- -------
...
cufflinks           0.17.3
ipython             7.19.0
ipython-genutils    0.2.0
ipywidgets          7.6.3
numpy               1.19.4
pandas              1.1.5
plotly              4.14.1
...

import pandas as pd
import psycopg2
from plotly.subplots import make_subplots
from plotly import graph_objects as go
from plotly import express as px
import cufflinks as cf

cf.set_config_file(offline=True, sharing='public')

pd.set_option("display.max_rows", None)
pd.set_option("display.max_columns", None)
pd.set_option("display.width", None)
pd.set_option("display.max_colwidth", None)

# ...

Error:
Traceback (most recent call last):
  File "/Users/jcope/Workspace/github.com/redhat-et/caliper/python/plotter/main.py", line 44, in <module>
    df_mem.iplot()
  File "/Users/jcope/Workspace/github.com/redhat-et/caliper/venv/lib/python3.9/site-packages/cufflinks/plotlytools.py", line 1216, in _iplot
    return iplot(figure,validate=validate,sharing=sharing,filename=filename,
  File "/Users/jcope/Workspace/github.com/redhat-et/caliper/venv/lib/python3.9/site-packages/cufflinks/plotlytools.py", line 1429, in iplot
    if not offline.is_offline() or online:
  File "/Users/jcope/Workspace/github.com/redhat-et/caliper/venv/lib/python3.9/site-packages/cufflinks/offline.py", line 38, in is_offline
    return py_offline.__PLOTLY_OFFLINE_INITIALIZED
AttributeError: module 'plotly.offline' has no attribute '__PLOTLY_OFFLINE_INITIALIZED'

Getting same error
AttributeError: module 'plotly.offline' has no attribute '__PLOTLY_OFFLINE_INITIALIZED'

Getting same error
AttributeError: module 'plotly.offline' has no attribute '__PLOTLY_OFFLINE_INITIALIZED'

Hi,
Hit this first time going through the tutorial today.
(cufflinks-venv) root@smet52eldom9:/export/cufflinks# python
Python 3.7.9 (default, Oct  6 2020, 03:10:59)
[GCC 9.3.0] on sunos5
Type "help", "copyright", "credits" or "license" for more information.
>>>
>>> import cufflinks as cf
>>> import numpy as np
>>> import pandas as pd
>>> cf.set_config_file(world_readable=True,theme='pearl')
>>> df=pd.DataFrame(np.random.randn(100,5),index=pd.date_range('1/1/15',periods=100),
...    columns=['IBM','MSFT','GOOG','VERZ','APPL'])
>>> df=df.cumsum()
>>> df.iplot(filename='Tutorial 1')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/export/cufflinks/cufflinks-venv/lib/python3.7/site-packages/cufflinks/plotlytools.py", line 1218, in _iplot
    dimensions=dimensions,display_image=kwargs.get('display_image',True))
  File "/export/cufflinks/cufflinks-venv/lib/python3.7/site-packages/cufflinks/plotlytools.py", line 1429, in iplot
    if not offline.is_offline() or online:
  File "/export/cufflinks/cufflinks-venv/lib/python3.7/site-packages/cufflinks/offline.py", line 40, in is_offline
    return py_offline.__PLOTLY_OFFLINE_INITIALIZED
AttributeError: module 'plotly.offline' has no attribute '__PLOTLY_OFFLINE_INITIALIZED'
>>> import plotly
>>> plotly.__version__
'4.14.3'
>>> cf.__version__
'0.17.3'
>>>

I'm not familiar with cufflinks or the intention of ofline mode but the indentation in go_offline() looks wrong to me. We only set py_offline.__PLOTLY_OFFLINE_INITIALIZED if we are ran from ipython and get a TypeError. It seems like we should do this regardless so I've taken a different approach in fixing this and changed the indentation so go_offline() will always set py_offline.__PLOTLY_OFFLINE_INITIALIZED to True:
index 54c57bc..7ffd727 100644
--- a/cufflinks/offline.py
+++ b/cufflinks/offline.py
@@ -29,7 +29,7 @@ def go_offline(connected=None):
         except TypeError:
             #For older versions of plotly
             py_offline.init_notebook_mode()
-        py_offline.__PLOTLY_OFFLINE_INITIALIZED=True
+    py_offline.__PLOTLY_OFFLINE_INITIALIZED=True

 def go_online():
        py_offline.__PLOTLY_OFFLINE_INITIALIZED=False

If that seems reasonable I'll do the pull request.

AttributeError: module 'plotly.offline' has no attribute '__PLOTLY_OFFLINE_INITIALIZED'
Same in PyCharm and there's no solution for the moment... I have a look at many forums everywhere but nobody knows how to solve it, yet

I think I found a solution to this. There is a potential error in ./cufflinks/offline.py. Shorten line 32 "py_offline.__PLOTLY_OFFLINE_INITIALIZED=True"'s indentation by one level fixes this problem.

Hi, Hit this first time going through the tutorial today.
(cufflinks-venv) root@smet52eldom9:/export/cufflinks# python
Python 3.7.9 (default, Oct  6 2020, 03:10:59)
[GCC 9.3.0] on sunos5
Type "help", "copyright", "credits" or "license" for more information.
>>>
>>> import cufflinks as cf
>>> import numpy as np
>>> import pandas as pd
>>> cf.set_config_file(world_readable=True,theme='pearl')
>>> df=pd.DataFrame(np.random.randn(100,5),index=pd.date_range('1/1/15',periods=100),
...    columns=['IBM','MSFT','GOOG','VERZ','APPL'])
>>> df=df.cumsum()
>>> df.iplot(filename='Tutorial 1')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/export/cufflinks/cufflinks-venv/lib/python3.7/site-packages/cufflinks/plotlytools.py", line 1218, in _iplot
    dimensions=dimensions,display_image=kwargs.get('display_image',True))
  File "/export/cufflinks/cufflinks-venv/lib/python3.7/site-packages/cufflinks/plotlytools.py", line 1429, in iplot
    if not offline.is_offline() or online:
  File "/export/cufflinks/cufflinks-venv/lib/python3.7/site-packages/cufflinks/offline.py", line 40, in is_offline
    return py_offline.__PLOTLY_OFFLINE_INITIALIZED
AttributeError: module 'plotly.offline' has no attribute '__PLOTLY_OFFLINE_INITIALIZED'
>>> import plotly
>>> plotly.__version__
'4.14.3'
>>> cf.__version__
'0.17.3'
>>>

I'm not familiar with cufflinks or the intention of ofline mode but the indentation in go_offline() looks wrong to me. We only set py_offline.__PLOTLY_OFFLINE_INITIALIZED if we are ran from ipython. It seems like we should do this regardless so I've taken a different approach in fixing this and changed the indentation so go_offline() will always set py_offline.__PLOTLY_OFFLINE_INITIALIZED to True:
index 54c57bc..7ffd727 100644
--- a/cufflinks/offline.py
+++ b/cufflinks/offline.py
@@ -29,7 +29,7 @@ def go_offline(connected=None):
         except TypeError:
             #For older versions of plotly
             py_offline.init_notebook_mode()
-        py_offline.__PLOTLY_OFFLINE_INITIALIZED=True
+    py_offline.__PLOTLY_OFFLINE_INITIALIZED=True

 def go_online():
        py_offline.__PLOTLY_OFFLINE_INITIALIZED=False

If that seems reasonable I'll do the pull request.

Yes this fixed the said problem for me. The newest cufflinks still has this error in it.

I think you should do a pull request, this worked for me.

Try it:
from plotly.offline import download_plotlyjs,init_notebook_mode,plot,iplot
init_notebook_mode(connected=True)
cf.go_offline()


Config Flow could not be loaded: undefined

Version of the custom_component
v0.1.0

Log
 Logger: homeassistant.util.package
Source: util/package.py:99
First occurred: 2:08:28 PM (3 occurrences)
Last logged: 2:08:45 PM
Unable to install package vienna-smartmeter[async]==0.3.2: ERROR: Cannot install vienna-smartmeter[async]==0.3.2 because these package versions have conflicting dependencies. ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts WARNING: You are using pip version 22.0.3; however, version 22.0.4 is available. You should consider upgrading via the '/usr/local/bin/python3 -m pip install --upgrade pip' command.

homeassistant.requirements.RequirementsNotFound: Requirements for vienna_smartmeter not found: ['vienna-smartmeter[async]==0.3.2'].

Describe the bug
I added the integration via HACS and to see the above errors in the log.
If I try to add the integration to the system it raises "Config Flow could not be loaded: undefined"
This message was actually the reason to look into the logs :)
HA Version
Home Assistant 2022.3.7 running on a RPi4 installed as HA image

Hm... just ran into the same...

Me too

Same Issue!

Same error so the packages are outdated and the requirements not adjusted accordingly
homeassistant.requirements.RequirementsNotFound: Requirements for vienna_smartmeter not found: ['vienna-smartmeter[async]==0.3.2'].
and:
Logger: homeassistant.util.package
Source: util/package.py:99
First occurred: 16:21:28 (3 occurrences)
Last logged: 16:21:50
Unable to install package vienna-smartmeter[async]==0.3.2: ERROR: Cannot install vienna-smartmeter[async]==0.3.2 because these package versions have conflicting dependencies. ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts WARNING: You are using pip version 22.0.3; however, version 22.1.2 is available. You should consider upgrading via the '/usr/local/bin/python3 -m pip install --upgrade pip' command.

Ok, I could make this work by changing the requirements to the correct line:
In manifest.json use:
"requirements": ["vienna-smartmeter@git+https://github.com/tommy-hartmann/vienna-smartmeter.git"],
I guess it would also work by pointing it to the platysma repo

Also this comment did the same hihi:
https://github.com/platysma/hass-vienna-smartmeter/issues/26#issuecomment-1024989602


Update URL of help window #217

Closes #217
This points the URL of the help window to a more aesthetically pleasing document rather than the current one pointing directly to the markdown in github

Not exactly related to this but this means it just shows a blank screen if there's no internet connection?

@damitch No there are no tests for checking that it loads properly. I will create a separate issue in the issue tracker.

v1
@MightyCupcakes submitted v1 for review.

[v1 1/1] Modify HelpWindow.java to point to the HTML user guide

(:books: Archive)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/259/1/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you want to fetch this PR to.

v2
@MightyCupcakes submitted v2 for review.

[v2 1/1] Modify HelpWindow.java to point to the HTML user guide

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v1 and v2)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/259/2/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you want to fetch this PR to.


Error with external $ref pointers in open api 3

Hi guys.
Recently update from 'swagger 2.0' to 'open api 3'
I used external $ref pointers in my swagger file like this:
paths:
  /countries:
    $ref: './validator/countries.yaml#/routes/base'

For 'swagger 2.0' it's working perfectly fine but for 'open api 3' it throws an error
TypeError: Cannot read property 'countries.yaml#' of undefined

It began working if we change $ref to this style:
    $ref: './validator/countries.yaml/routes/base'

But this is nonconventional use of the $ref that doesn't support by the other libraries.
Is there any solutions for this Issue?

Hi @Fillosof,
Thanks for reporting the problem.
Can you please provide the stack of the error and also if possible  definition files for reproducing the problem.

Thank you @kobik for the quick answer.
Here Error stack:
TypeError: Cannot read property 'properties.yaml#' of undefined
    at /home/api/node_modules/json-schema-deref-sync/lib/utils.js:84:17
    at Array.reduce (<anonymous>)
    at Object.getRefPathValue (/home/api/node_modules/json-schema-deref-sync/lib/utils.js:83:15)
    at getRefSchema (/home/api/node_modules/json-schema-deref-sync/lib/index.js:99:18)
    at Object.<anonymous> (/home/api/node_modules/json-schema-deref-sync/lib/index.js:236:24)
    at walker (/home/api/node_modules/traverse/index.js:190:22)
    at /home/api/node_modules/traverse/index.js:208:29
    at Array.forEach (<anonymous>)
    at forEach (/home/api/node_modules/traverse/index.js:298:31)
    at walker (/home/api/node_modules/traverse/index.js:203:13)
    at /home/api/node_modules/traverse/index.js:208:29
    at Array.forEach (<anonymous>)
    at forEach (/home/api/node_modules/traverse/index.js:298:31)
    at walker (/home/api/node_modules/traverse/index.js:203:13)
    at walk (/home/api/node_modules/traverse/index.js:226:7)
    at Traverse.forEach (/home/api/node_modules/traverse/index.js:50:18)
    at derefSchema (/home/api/node_modules/json-schema-deref-sync/lib/index.js:223:27)
    at deref (/home/api/node_modules/json-schema-deref-sync/lib/index.js:310:13)
    at Object.buildSchemaSync (/home/api/node_modules/api-schema-builder/src/index.js:35:26)
    at Object.init (/home/api/node_modules/express-ajv-swagger-validation/src/middleware.js:25:32)
    at Object.<anonymous> (/home/api/dist/server.js:21:28)
    at Generator.next (<anonymous>)

and swagger files: link

Thanks @Fillosof, the error is coming from json-schema-deref-sync.
I'll investigate this further.

@Fillosof are you sure it's working with OIA2?
Do you mean, that you've being using the same document, just in OIA2 format?
From what I see the problem is even bigger as json-schema-deref-sync supports loading only refs to json files and not yaml ones.
This would require us to rethink of how to handle dereferencing properly.

I think issue somewhere in the parser.
Here old swagger 2.0 files

It is indeed json-schema-deref-sync not handling yaml files.
You can see the issue I've opened here and the PR for fixing this issue here, although I'm not sure the maintainer would like to add this functionality.

Thank you @kobik
Then all its left is to pray and believe in a bright future.
Because in another case I need precompile swagger file, and its make it 4x bigger, thats slow down startup time by 2-3 sec.((((

Hi @kobik @Fillosof, it appears you have found the root cause and thanks for the effort. Just want to let you know I had the same issue after upgrading to 1.0.8 but it worked well with 0.9.0.

@superliuwr Thanks man, you save my day))))

@Fillosof, until my fix gets merged, you can achieve nullable like behavior by adding null to the supported types of the property.
timezone:
  type:
    - 'string'
    - 'null'

Quick update,
Waiting for json-schema-deref-sync devs to publish a new version with my fix.
In the meanwhile I've prepared a PR in api-schema-builder.

@kobik Thank You, now I had a hope to clear a mess, that I created to make it work

If you'd like a better json ref resolver, we've got a fantastic one over at stoplightio/json-ref-resolver.
It's powering everything throughout Studio, Prism, Spectral, etc., and is as performant as it is powerful.

Fixed https://github.com/Zooz/express-ajv-swagger-validation/pull/103


Halloween Scenes

Would you consider adding the currently available Halloween scenes from the Hue app?
Unfortunately, as I don't own enough RGB lights, I can't reliably get all colors to extract their values; extracting them from palette sreenshots seems to be a valid substitute, though. Do you need them in x,y, or would rgb values work as well?

Done, have fun :)
https://github.com/Hypfer/hass-scene_presets/commit/067c7814fd7f81789f5b8537fef4cf3958491046

Nice! Thanks! :-D


Assignment is incorrectly reported as "late" when submitting

For 61A, we have an assignment hw01 whose due date is Thursday and lock date is 6/30. However, when running ok-client with --submit, the client gets a 403 response claiming the submission was "late."
Possibly related; the lock date used to be set to Monday (today) mistakenly, but it was changed yesterday to 6/30.

I think it was a caching issue then. I cleared the cache. Try again?

Awesome, it works now. Thanks!


Add yamllint job

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mkumatag
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [mkumatag]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mkumatag: Updated the job-config configmap in namespace prow at cluster default using the following files:

key .yamllint.yaml using file config/jobs/.yamllint.yaml
key test-infra-presubmits.yaml using file config/jobs/ppc64le-cloud/test-infra/test-infra-presubmits.yaml

In response to this:

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


fix transaction system test

Overview / Motivation
add the needed validation for the state change
The Task
TODO: Clearly describe the issue requirements here...
Implementation Notes
TODO: Add links to relevant resources, specs, related issues, etc...
Contribution Guidelines
Important: Issue assignment to developers will be by the order of their application and proficiency level according to the tasks complexity. We will not assign tasks to developers who have'nt introduced themselves on our Gitter dev channel

Introduce yourself on go-spacemesh dev chat channel - ask our team any question you may have about this task
Fork branch develop to your own repo and work in your repo
You must document all methods, enums and types with godoc comments
You must write go unit tests for all types and methods when submitting a component, and integration tests if you submit a feature
When ready for code review, submit a PR from your repo back to branch develop
Attach relevant issue to PR

sorry this is a duplicate issue to #819


Change google analytics token, digital ocean referral code

These still point to blockstack; let's change them so we can know if people are using it and get a little referral credit.

Here's a referral link for us, I'm assuming the code is the thing at the end: https://m.do.co/c/a688ee6191f9

sweet, I updated this and deployed to github pages.

It looks like I already ripped out the old googl analytics token btw.  We can add our own back if needed


N3779 User-defined Literals for std::complex

https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2013/n3779.pdf
__cpp_lib_complex_udls : 201309L

実装済み


prepare to update to next hyperdrive version (v0.0.3)

changes in preparation to moving to next hyperdrive release

Codecov Report
Patch and project coverage have no change.

Comparison is base (61e08bf) 75.07% compared to head (0802455) 75.07%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #525   +/-   ##
=======================================
  Coverage   75.07%   75.07%
=======================================
  Files          55       55
  Lines        3362     3362
=======================================
  Hits         2524     2524
  Misses        838      838

Flag
Coverage Δ

unittests
75.07% <ø> (ø)

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

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

closed in favor of #566 because commits aren't showing up on github


sha256 hash changed for googleapis archive

Does this issue affect the google-cloud-cpp project?
Yes
What component of google-cloud-cpp is this related to?
Cannot build
Describe the bug
The sha256 hash value of the archived googleapis ".tar.gz" file from GitHub has changed. The build is expecting
f77d852d538f96a1c10b447a36c807c9ce0f8716d028ba47c3dd5051d84c518b,
but as of today the value has changed to
c5a3c56ed39aca0dea8b840db32fa44c44df2230f1779a6c78c0d6027a60dfcb.
To Reproduce
Steps to reproduce the behavior:

Build the project, make sure it involves re-downloading GitHub archived dependency(s), such as  https://github.com/googleapis/googleapis/archive/c81bb701eb53991d6faf74b2656eaf539261a122.tar.gz.
Check for CMake error, for example:

-- Using src='https://github.com/googleapis/googleapis/archive/c81bb701eb53991d6faf74b2656eaf539261a122.tar.gz'
-- [download 0% complete]
-- [download 1% complete]
-- [download 2% complete]
-- [download 4% complete]
-- [download 5% complete]
-- [download 7% complete]
-- [download 9% complete]
-- [download 10% complete]
-- [download 13% complete]
-- [download 15% complete]
-- [download 17% complete]
-- [download 19% complete]
-- [download 21% complete]
-- [download 23% complete]
-- [download 26% complete]
-- [download 28% complete]
-- [download 30% complete]
-- [download 32% complete]
-- [download 34% complete]
-- [download 37% complete]
-- [download 38% complete]
-- [download 40% complete]
-- [download 43% complete]
-- [download 45% complete]
-- [download 47% complete]
-- [download 49% complete]
-- [download 51% complete]
-- [download 54% complete]
-- [download 56% complete]
-- [download 58% complete]
-- [download 60% complete]
-- [download 62% complete]
-- [download 64% complete]
-- [download 67% complete]
-- [download 69% complete]
-- [download 71% complete]
-- [download 73% complete]
-- [download 75% complete]
-- [download 78% complete]
-- [download 80% complete]
-- [download 82% complete]
-- [download 84% complete]
-- [download 86% complete]
-- [download 88% complete]
-- [download 91% complete]
-- [download 93% complete]
-- [download 95% complete]
-- [download 97% complete]
-- [download 99% complete]
-- [download 100% complete]
-- verifying file...
       file='/tmp/tritonbuild/tritonserver/build/_deps/repo-third-party-build/google-cloud-cpp/src/google-cloud-cpp-build/external/googleapis/src/c81bb701eb53991d6faf74b2656eaf539261a122.tar.gz'
-- SHA256 hash of
    /tmp/tritonbuild/tritonserver/build/_deps/repo-third-party-build/google-cloud-cpp/src/google-cloud-cpp-build/external/googleapis/src/c81bb701eb53991d6faf74b2656eaf539261a122.tar.gz
  does not match expected value
    expected: 'f77d852d538f96a1c10b447a36c807c9ce0f8716d028ba47c3dd5051d84c518b'
      actual: 'c5a3c56ed39aca0dea8b840db32fa44c44df2230f1779a6c78c0d6027a60dfcb'
-- Hash mismatch, removing...
CMake Error at googleapis_download-stamp/download-googleapis_download.cmake:170 (message):
  Each download failed!

Expected behavior
The hash value should match, and the build should succeed.
Operating system:
Ubuntu 20.04
What compiler and version are you using?
CMake, g++
What version of google-cloud-cpp are you using?
1.42.0
Additional context

This is an ongoing incident at GitHub: https://github.com/bazel-contrib/SIG-rules-authors/issues/11#issuecomment-1409438954
Closing as we just need to wait.

Thanks!


Ensure unique units are used when creating a request [#4579]

Work toward #4579
This updates the code that merges duplicate request lines together also checks that the units are the same.

No, I'll fix it as part of this. Need to move the check/merge up earlier in the process I think.


Getter Setter

New
Following are the changes in this PR,

Symmetrise the getter and setter such that we can retrieve parts of array using slices, but also set array at specific slices. Following is an example,

    var A = NDArray[i16](3, 3, 3, fill=Scalar[i16](1))
    var D = nm.random.rand[i16](3, 3, min=0, max=100)
    A[1] = D # sets the elements of A[1:2, :, :] with the array `D`
    A[:, 0:1, :] = D # sets the elements of A[:, 0:1, :] with the array `D`

Introduce a new idx object to unify and handle all indices related computations and removes the use of multiple List[Int], VariadicList[Int]. This is intended to be similar to Loc in pandas. Following is an example,

var A = NDArray[i16](3, 3, 3, fill=Scalar[i16](2))
print(A[idx(1, 2, 3)]) # prints the element at the location (1,1,2)
A[idx(1, 2, 3)] = 10 # sets the element at the location (1, 1, 2) to be 10.

Add NDArray initialisation from numpy arrays.

var np_arr = np.array([2, 4, 5, 6])
var A = NDArray[i16](data=np_arr)

Added alias for all main containers to make it simple for users to write. Following are the new aliases,

alias ndarray = NDArray
alias ndshape = NDArrayShape
alias ndstride = NDArrayStride
alias idx = Idx

Note: These aliases are meant only for users and therefore the original tags should be used for internal purposes to avoid confusions.
CHANGES

Removed all instances of getters and setters with List[Int], VariadicList[Int]. The following are not present anymore, therefore use idx object to set and get elements at specific indices.

fn __setitem__(inout self, index: List[Int], val: SIMD[dtype, 1]):
fn __setitem__(inout self, index: VariadicList[Int],  val: SIMD[dtype, 1]):
fn __getitem__(inout self, index: List[Int]) raises -> SIMD[dtype, 1]:
fn __getitem__(inout self, index: VariadicList[Int]) raises -> SIMD[dtype, 1]:

Replaced the unsafe recursively calling _traverse_iterative function with a simpler for loop variant.

@shivasankarka, would you make this a PR so that we can merge that into Experimental？


Chat Log plugin

A plugin that stores your chat data in a comma-separated text file.

It looks like you've included your screenshot fix in this branch by accident.
Try doing another git reset head~2, and then unchecking ScreenshotPlugin.java in the commit window. That'll create one clean commit with only the chatlog changes.

@SoyChai Oops.. look like I messed up something else too. Bit too eager. I'll fix it - ignore this PR for now


img2img - CUDA kernel error

I get this error when trying to do an img2img:
RuntimeError: CUDA error: no kernel image is available for execution on the device
CUDA kernel errors might be asynchronously reported at some other API call,so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1.

does txt2img work ?

Good question. I think I tried that after and it didn’t work either. Not sure what I am doing wrong (I ran everything like usual)

remove the folder "sd" from your GDrive or "stable-diffusion-webui" if it's the hlky version, then rerun the colab, it might be a problem with the AUTOMATIC webui


update container names to match the ones expected by cortex-mixin

What this PR does:
All Cortex container names take the Chart.Name value, making it impossible to query metrics based on container metric label. On top of that, the cortex-mixin configuration looks for specific names. The change aligns the container names with their functionality.
Which issue(s) this PR fixes:
Fixes #233
Checklist

[X] CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

LGTM

Hey @danfromtitan,
I use the mixin as well. I can share my config in order to get it working. I use some relabel config to the cluster part working and slighty adjusted jobnames. I did not have to rename containers, that really doesn't make super much sense too me

I can share my config in order to get it working

Thanks for sharing, I was looking for this relabeling detail earlier, I think it would reduce the amount of changes required in cortex-mixin fork but it would still need container to have the component in the name for the reasons mentioned above.

I can share my config in order to get it working

Thanks for sharing, I was looking for this relabeling detail earlier, I think it would reduce the amount of changes required in cortex-mixin fork but it would still need container to have the component in the name for the reasons mentioned above.

Yep I agree. I also didn't check every dashboard, so there still might be some problems. I feel like this is a bit out of scope but maybe we can figure something out. Thanks for raising the Issue

I did, the relabelling helps whenever a namespace/job value comes into play, but container labels are hard coded as I mentioned above.
Besides the relabelling and the container names change in the chart, you can see here the additional changes required to get all dashboards fully functional.

I did, the relabelling helps whenever a namespace/job value comes into play, but container labels are hard coded as I mentioned above.
Besides the relabelling and the container names change in the chart, you can see here the additional changes required to get all dashboards fully functional.

Have you created a PR over at cortex-jsonnet to be more dynamic with the mixin?

Have you created a PR over at cortex-jsonnet to be more dynamic with the mixin?

That was my next target, but I cannot test the changes there unless this PR is merged first.

Have you created a PR over at cortex-jsonnet to be more dynamic with the mixin?

That was my next target, but I cannot test the changes there unless this PR is merged first.

Excuse my ignorance but why? Which dashboards require a change here? I don't really get it

Excuse my ignorance but why? Which dashboards require a change here? I don't really get it

Please see in my comment above.

Excuse my ignorance but why? Which dashboards require a change here? I don't really get it

Please see in my comment above.

Gotcha! I will talk to @kd7lxl

I'm a little puzzled by the conversation here.

I cannot test the changes there unless this PR is merged first.

This doesn't make any sense to me. You can helm install from your branch just as easily as you can helm install from the master branch or a release.
Please test the desired dashboard behavior against this branch to prove that it solves the problem. Please share your results.
If everything works as expected, we can target this for v1.0.0. It will need to be in a major version release because it's a breaking change -- everyone who has dashboards targeting the current container names will need to rewrite their dashboards.

everyone who has dashboards targeting the current container names will need to rewrite their dashboards

In case someone's using cortex-mixin to create dashboards, they will need the same change I requested in this PR.
Anyone else using container label in their dashboards would look at misleading results, because container label "cortex" would query the metrics from all components instead of the specific component dashboard is intended for.
I have made changes to the cortex-mixin project that are waiting to be merged, dahboards work perfectly, we need this PR merged first before I can submit the other.

@danfromtitan have you opened up a PR with the folks over at grafana?

Not yet, it's on my list. Thanks for merging this.

@danfromtitan have you opened up a PR with the folks over at grafana?

Thanks for accepting this PR, I opened https://github.com/grafana/cortex-jsonnet/pull/427 to fix the outstanding issues with Grafana dashboards when Cortex is deployed from the Helm chart.


Counter-intuitive "Copy Link" button when inviting people to a document

Hey there 👋🏼
A couple of our users have been tripped up by the Copy link button when trying to invite people to a document.

For a few of our users, this design hints that the Copy link will create an invite link to people that they can follow to accept their invitation to a document, when in fact it seems to only be a link to the document that is of little use if the person clicking the link has not been added to the document first.
My suggestion is to either

get rid of this button
rename this button to something like Copy link to document
rename this button to something like Copy link to document and add a little disclaimer under it that says "You will still need to invite users using the panel below"
maybe implement the invite link functionality (that could be very useful in some situations)

What do you think ?

Hi Vincent!
Thanks for this user feedback. Makes sense that it's confusing!
What do you think of this?

Get rid of this button
Make actual link sharing more visible. Example below.
Implement a "request access" feature. If someone lands on a document they do not have access to they see the screenshot below. There's a few ways to end up here, and addressing a quick way to request access could help. I'd change the copy and make the primary button "Request Access".

Here's a quick stab stab making public access more visible. Currently it's link text that's hard for users to find. With this design there is always a "copy link" button visible, and what it does is tied to public access. Currently public access can be set to editor or viewer. Here, "no access" is also visible (and the default for new documents).

Some other products break off public access settings under its own header, or a line separator. We could explore that if adding it in-line (as shown above) seems too busy.

Hello,
Ideas:
Remove the Anyone with link text and add a contextual text when hovering over the Copy link button : Only those invited below have access.
or
When the choice is no access, replace "Anyone with link" by "Only those invited below have access."
Your suggestion to differentiate the Public access block from the rest seems to me to be a good idea, either by moving it or by giving it another visual form.

What do you think of a smaller change like this: change "[] Copy Link" link to either "🔒 Copy Link" or "👥 Copy Link" (with an icon to distinguish private vs link-shared docs). And on hover of that link, it would offer a tooltip with a disclaimer as @vviers suggested, either "Only people with access can open with the link" or "Anyone on the internet with the link can open".
It's a small dialog, so if we add a "Public access" block that's always visible even for private docs, we might want to update the design to make the listing generally more compact.

Paging @kbizien in case he has ideas


Add server host configuration option

With Vert.x when creating an http server, there's a host option.
It's not configurable in Redpipe (the port is). I'm a bit afraid it might cause some trouble in some corner cases like cloud deployment.
I think the only times I've been using it, was with Heroku / OpenShift deployment, where you're given a host to listen on, and just can't listen on "localhost" or "<IP_ADDRESS>".
Maybe make this a configurable option and expose it through AppGlobals

Yeah, you're right it should be configurable. Probably via a config option like the port.

Fixed.


how to fix  'NoneType' object has no attribute 'image_encoder'?

Traceback (most recent call last):
File "D:\SAM\Layer-Divider-WebUI-master\venv\lib\site-packages\gradio\routes.py", line 401, in run_predict
output = await app.get_blocks().process_api(
File "D:\SAM\Layer-Divider-WebUI-master\venv\lib\site-packages\gradio\blocks.py", line 1302, in process_api
result = await self.call_function(
File "D:\SAM\Layer-Divider-WebUI-master\venv\lib\site-packages\gradio\blocks.py", line 1025, in call_function
prediction = await anyio.to_thread.run_sync(
File "D:\SAM\Layer-Divider-WebUI-master\venv\lib\site-packages\anyio\to_thread.py", line 31, in run_sync
return await get_asynclib().run_sync_in_worker_thread(
File "D:\SAM\Layer-Divider-WebUI-master\venv\lib\site-packages\anyio_backends_asyncio.py", line 937, in run_sync_in_worker_thread
return await future
File "D:\SAM\Layer-Divider-WebUI-master\venv\lib\site-packages\anyio_backends_asyncio.py", line 867, in run
result = context.run(func, *args)
File "D:\SAM\Layer-Divider-WebUI-master\modules\sam.py", line 67, in generate_mask_app
self.set_mask_generator()
File "D:\SAM\Layer-Divider-WebUI-master\modules\sam.py", line 40, in set_mask_generator
self.mask_generator = SamAutomaticMaskGenerator(
File "D:\SAM\Layer-Divider-WebUI-master\venv\lib\site-packages\segment_anything\automatic_mask_generator.py", line 123, in init
self.predictor = SamPredictor(model)
File "D:\SAM\Layer-Divider-WebUI-master\venv\lib\site-packages\segment_anything\predictor.py", line 31, in init
self.transform = ResizeLongestSide(sam_model.image_encoder.img_size)
AttributeError: 'NoneType' object has no attribute 'image_encoder'

it seems like Sam is not properly installed.
I've just tested with latestest version and it works fine.
If anyone still encounter this issue please let me know.

@RobetZyarde
That would be torch version incompatibility issues, fixed in #6.
But this project doesn't support prompt to the image and uses sam instead of sam-v2, so it will be archived soon.
Can you try the new sam2-playground repository instead?
There's also a feature to segment a video.


Special-case CSS for pydata library outputs and style

Context
Outputs like matplotlib, bokeh, etc are particularly important given that this theme is "for" the pydata community. We should make sure that this theme behaves well in the ways that those communities are using it. One example of this is to make sure that this theme "looks good" for demonstrating outputs etc generated by sub-projects like MNE, Bokeh, Jupyter, Matplotlib, etc.
Two examples of this:

Dark mode: some auto-generated images, plots, etc have their ticklabels lost in the darkness when dark mode is enabled
Interactive plots: Some libraries that generate HTML may need special casing (e.g. #284)

Proposal
We should create a dedicated page in our Demo docs for "libraries we explicitly support" and show off demos there that can be used as a benchmark to determine if we need to add any special-cased CSS. (similar to this demo section but w/ a dedicated page)
I think we should scope this to any library in the PyData ecosystem that has a representative that wants to special-case some CSS. Ideally other libraries could make their own upstream contributions here as well.

I cleaned up the top comment a bit to try and make it more actionable and have a clear next step for discussion/implementation

https://github.com/scipy/scipy/pull/16379

This now exists here! https://pydata-sphinx-theme.readthedocs.io/en/latest/demo/pydata.html
Let's close this and address other issues as they pop up


use strict is required

I appreciate the pull request but use strict isn't be needed if Node 6.5 or higher is installed as mentioned in the readme. Since I want people to use the latest versions of node (LTS or latest) I'm going to go ahead and leave it as is.

Thanks


Wrong version check

The plugin prompts me that gradle is out of date, but I am using the latest version of gradle. My idea version is 2021.2 .And the plugin version is 2.0.4.

That's because Gradle released a new backport release. There's already a PR which fixes that issue. (#14)
But we have to wait a little, @DRSchlaubi had a very bad night with the Deutsche Bahn.

Why is it bad? Is it about COVID-19? I read the news in Germany now that the epidemic is more serious. How about you? Are you okay?

His train got cancelled, and then he had to travel home much longer. In addition, he was already awake for a long time and then found nothing to drink. But about Covid-19, we are both fine. @DRSchlaubi is already fully vaccinated, and I have my 2nd vaccination in 2 weeks.

I actually forgot to close this, this should be fixed


Add support for Sender Actions

Facebook recently added "Sender Actions" to the platform, which let you toggle a typing state for the bot. https://developers.facebook.com/docs/messenger-platform/send-api-reference/sender-actions
Since they're not part of the message object in a payload, this library currently doesn't support them. We should add them somehow, likely as something separate from sendMessage / reply.
The first solution I can think of is add toggling sender actions as a third param in the event handler. Something like (event, reply, actions) => {}, with actions having functions like actions.setTyping(state: boolean, callback: function) and actions.markRead(callback: function). This would let adding actions not be a breaking change to the module, which would be nice.

I have taken I slightly different approach but like the idea. Maybe some of the code in my current pull request will be useful.


Asset Pipeline dependency installation fails on Ubuntu 23.04

Following the quickstart guide, I complete all of the steps up until the call to "npm install" to install the asset pipeline dependencies. It fails at that point.
Below is my full terminal output as I attempt to follow all of the steps of the guide. You will notice that after the call to npm install, I run ls on the folder that it attempts to rename, and that folder does not exist.
I am not a web developer, and I am not familiar with node.js beyond the very basics, and so I'm not sure how to get you more useful information.
tim@tim-System-Product-Name:~$ git init mygame
hint: Using 'master' as the name for the initial branch. This default branch name
hint: is subject to change. To configure the initial branch name to use in all
hint: of your new repositories, which will suppress this warning, call:
hint:
hint: 	git config --global init.defaultBranch <name>
hint:
hint: Names commonly chosen instead of 'master' are 'main', 'trunk' and
hint: 'development'. The just-created branch can be renamed via this command:
hint:
hint: 	git branch -m <name>
Initialized empty Git repository in /home/tim/mygame/.git/
tim@tim-System-Product-Name:~$ cd mygame
tim@tim-System-Product-Name:~/mygame$ sudo npm install @matthewcpp/framework64-cli

added 126 packages in 11s

7 packages are looking for funding
  run `npm fund` for details
tim@tim-System-Product-Name:~/mygame$ npx framework64-cli create
Creating framework64 game: mygame
framework64 project directory: /home/tim/mygame
Check for git repository in /home/tim/mygame: OK
Add framework64 submodule
Add framework64 submodule: OK
Initialize framework64 submodule
Initialize framework64 submodule: OK
Create initial commit
Create initial commit: OK
Downloading and configuring starter project from: https://github.com/matthewcpp/framework64-starter/archive/refs/heads/main.zip
Downloading and configuring starter project from: https://github.com/matthewcpp/framework64-starter/archive/refs/heads/main.zip: OK
tim@tim-System-Product-Name:~/mygame$ npm run configure-environment-n64-modernsdk

><EMAIL_ADDRESS>configure-environment-n64-modernsdk
> node lib/framework64/scripts/ConfigureEnvironment.js n64-modernsdk

Configuring for: N64 Modern SDK
Copy: /home/tim/mygame/lib/framework64/config/n64.modernsdk/n64.modernsdk.devcontainer.json --> /home/tim/mygame/.devcontainer.json
tim@tim-System-Product-Name:~/mygame$ npm install
npm ERR! code EACCES
npm ERR! syscall rename
npm ERR! path /home/tim/mygame/node_modules/@matthewcpp/framework64-cli
npm ERR! dest<EMAIL_ADDRESS>npm ERR! errno -13
npm ERR! Error: EACCES: permission denied, rename '/home/tim/mygame/node_modules/@matthewcpp/framework64-cli' -><EMAIL_ADDRESS>npm ERR!  [Error: EACCES: permission denied, rename '/home/tim/mygame/node_modules/@matthewcpp/framework64-cli' -><EMAIL_ADDRESS>{
npm ERR!   errno: -13,
npm ERR!   code: 'EACCES',
npm ERR!   syscall: 'rename',
npm ERR!   path: '/home/tim/mygame/node_modules/@matthewcpp/framework64-cli',
npm ERR!   dest<EMAIL_ADDRESS>npm ERR! }
npm ERR!
npm ERR! The operation was rejected by your operating system.
npm ERR! It is likely you do not have the permissions to access this file as the current user
npm ERR!
npm ERR! If you believe this might be a permissions issue, please double-check the
npm ERR! permissions of the file and its containing directories, or try running
npm ERR! the command again as root/Administrator.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/tim/.npm/_logs/2023-07-01T05_33_35_454Z-debug-0.log
tim@tim-System-Product-Name:~/mygame$ ls /home/tim/mygame/node_modules/@mathewcpp
ls: cannot access '/home/tim/mygame/node_modules/@mathewcpp': No such file or directory

The quickstart guide appears to work perfectly on Windows with docker and nodejs installed, so this appears to be a linux-specific issue.

Hi there!  Thanks for reporting this.  I will have to spin up a linux box and investigate...Glad it worked for you on windows!


Mutating a non reactive object after mutating a reactive one, applies the mutation to the reactive one as well

Vue version
3.2.47
Link to minimal reproduction
https://play.vuejs.org/#eNqdksFOwzAMhl/F6qWbJlbOU4ZAnDjBgWMva8hExupEaVqBqr47Trq02daC4BbH9vf/st0mD1qvm1okm4RV3EhtoRK21nc5ylIrY6EFI3bcykZAB3ujSkipPs0RIEeusLKgigNsqZCrGu0GbrscQ0ob9flFycBYUO3Spfc10odCkMiNKAXa5+Lw6ACLJaGobO1xqxUQzmldd7w4+NjjtX7vIp3Xd4G+2fW5ukjOhWekHH824GDEDKiL3gs2oVjWD5pGTIEVpT7urKAIgL3Jxj/o6VbgHgBZBqjwJoyQFWZIDewNtNHQoPOWiZIFzDlvihUZ97QonuCd4qK2liZyz4+Sf2zTq2WmA/4ppEabgdlDgtE55LjtKWjk9m/Y+Bxm3IKlgl7hH5ZP9zFruqeTzASbZf1FsCy6k6T7BidrREw=
Steps to reproduce
Click the Increment obj.count button, then click the Increment proxy.count button.
Then click Increment obj then proxy, Increment proxy then obj.
What is expected?
Mutating a non reactive object shouldn't affect the reactive proxy.
What is actually happening?
Mutating a non-reactive object, stores the mutation and applies it when the proxy is mutated.
System Info
No response
Any additional comments?
No response

The reactive proxy reads from and writes to the nonreactive original. That's how it works.


Updated reload to work with Express 4 (Closes Issue #16)

Upgraded reload to use Express 4. Modified the example in README to reflect the changes in Express 4. This pull request closes issue #16.

Had to close this to make some tweaks and reopened it here https://github.com/jprichardson/reload/pull/18


Unified Push (Matrix / Element) not working correctly

:lady_beetle: Describe the bug

I selfhost ntfy and a Matrix Synapse and i want to recive notifications via ntfy.
Inside the Element app i added the notification service (working just fine)
When trying to check for Problems it doesnt find any.
The ntfy app does show the im.vector.app (unified push)
But when clicking on it i only can change the name of the topic or copy the theme url.
Matrix lists an 200 success Message & my ntfy server does recive something!
When following the up* topic for the Matrix notifications i get an unformated message.
:computer: Components impacted

Matrix Synapse (Docker - Synapse / 1.105.1)
ntfy Server (Docker  - ntfy / v2.10.0)
Android (ntfy 1.16.0 - fdroid) - not recieving anything for unified push
Desktop (Firefox 125.0.3) - showing the unformated message

❓ What i think should happen
A well formated message should be recived by all subscibed devices
:bulb: Screenshots and/or logs

combined_logs.log

A few screenshots i made
1 How the Message shows inside the Webbrowser

2 "Startscreen" of ntfy

3 What it shows when clicking on the up* tab

3 The Matrix Test

:crystal_ball: Additional context

Other Pushers work just fine on any platform: skripts/uptime kuma/... - Web/Android
I have whitelisted the up* for any user
Did I do something wrong?

It looks like everything is hooked up correctly. The unformatted messages seen in the web app are to be expected (you're not supposed to look at that topic manually). ntfy sends the notifications to Element Android behind the scenes, and Element Android will format them into a proper notification.
The Element Android notification troubleshooter shows that the "Push Test" passed. Did any notification pop up during the test? Do you have the notifications enabled for the Element app in Android settings?

Thank you for the response.
A notification did show up from Element. (it does even recive the callback from the click on it)
I did expect a ntfy notification to pop up.


Allow support users to cancel applications

Context
This is part of the feature to allow support users to withdraw an application on the candidate's behalf.
Raising this now because we could go with a few different approaches.
The big question is whether to introduce a new state for "cancelled" applications. The reason I think it might be good is that we can keep track of applications that have been modified since submission. The alternative is to just destroy! the ApplicationChoice. This will cause weirdness once we implement the feature to withdraw from all of the courses, we would have forms without any choices in the system, which still have been submitted.
The downside to adding a new state is that we add a bunch of overhead.
Changes proposed in this pull request
Todo:

[ ] Only allow withdrawal when there are 2 or 3 choices
[ ] Update the logic to add courses to take into account removed courses
[ ] Update documentation & statistics

Guidance to review

Link to Trello card
https://trello.com/c/fmTuhZ0y
Things to check

[x] This code doesn't rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] New environment variables have been added to the Azure config

Should we introduce a new state for "cancelled" applications

I say we should!


Nc issue#119

*** THIS CODE SHOULD BE TESTED AND DEVELOPED BEFORE MERGING ***
This is a proof of concept for a redesigned search area. Still buggy.
Please branch, test, continue the code, or make suggestions.
*** THIS CODE SHOULD BE TESTED AND DEVELOPED BEFORE MERGING ***

Closed unresolved.
Redesign post-MVP.

Closed unresolved.
Redesign post-MVP.


Update according to TODO

copy option
accessor option
dotfiles
Makefile
README
remove preamble
license info
validate modules
dev deps
tests for code pathways

Resolves compute-io/todo#41
NOTE: breaking change for reasons as documented in compute-io/add#1

Coverage remained the same at 100.0% when pulling a66b19f0377af9e7b54aab8af913bdce786f5f86 on develop into 7a59b98ff1135fec2ee5fa49df0254aa64bada77 on master.


Fixes a couple things in the display of test output

A couple things I noticed in trying out this test runner:

The "passed" count is also including failed examples
The failures are only including examples that all fail. I think an example should fail if any assertion fails.

Excellent, thanks @robertjlooby! 💯


Fix emscripten build.

Due to this musl change included in emsdk 3.1.44.

Can you verify this change still works with previous version of emscripten?

Compiles and runs fine with at least emsdk 3.1.40 through 3.1.44. Tested with bgfx 04-mesh example.


Why no CPI button on Personal Refundable Credit amounts?

The Tax-Calculator current_law_policy.json file has the _II_credit amount (which is indexed by MARS) with this specification:  "cpi_inflated": true.
Is there a reason TaxBrain does not have a CPI button on this parameter?

Same question for the _II_credit_ps amounts.  Why no CPI button in TaxBrain?

Hmm.. these are good questions. The models have a boolean field to hold this info. See, respectively, here:
https://github.com/OpenSourcePolicyCenter/webapp-public/blob/master/webapp/apps/taxbrain/models.py#L147
and here:
https://github.com/OpenSourcePolicyCenter/webapp-public/blob/master/webapp/apps/taxbrain/models.py#L152
So, for some reason, the flag is not rendering. I will investigate.


[BEAM-2740] introduce the new InnerBeamSqlEnv/BeamSqlEnv

Summary:

Simplified BeamSqlEnv: now you can only register UDF/UDAF in BeamSqlEnv, and BeamSqlEnv just record the udfName -> udfClass mapping, actually work will be done by InnerBeamSqlEnv.

BeamSqlEnv is only used by the surface API classes.

InnerBeamSqlEnv is actually the old BeamSqlEnv, it holds schema and planner.

InnerBeamSqlEnv is only used by the implementation classes.

BeamSqlEnv is converted to InnerBeamSqlEnv automatically in BeamSql and BeamSqlCli.
BeamSqlEnv and InnerBeamSqlEnv are quite different, so can not share the same interface.

R: @takidau @XuMingmin

Changes Unknown when pulling 8bf14d93c078e55d49faf6281c49940bcbc0e902 on xumingming:BEAM-2740 into ** on apache:DSL_SQL**.

I like that suggestion.

BeamSqlEnv can be removed if we only consider BeamSql, but it is needed very soon for the BeamSqlCli(I already have a pending PR: #3481 ), I'd prefer to make it ready for the comming BeamSqlCli PR.

I guess BeamSqlEnv can be hidden(I suppose any class in package *.impl is treated as hidden) for both BeamSql and BeamSqlCli.
Each BeamSqlCli will have a shared BeamSqlEnv, and public void execute(String sqlString) would be the only exposed method, --TBD in future regarding to details of BeamSqlCli.

Rebased.
Summary:

Only keep one BeamSqlEnv in impl package.
Re-organized the surfacing apis to only remain:

BeamSql
BeamSqlUdf
BeamRecordSqlType
BeamSqlRecordHelper

Refactored QueryTransform and SimpleQueryTransform to enable more code reuse.

Changes Unknown when pulling 824b14635fe26bcb1b809f5df943923774c6d70d on xumingming:BEAM-2740 into ** on apache:DSL_SQL**.

Changes Unknown when pulling 5bbae5bf9ad6a8ee680a09d00e592abdfdada78a on xumingming:BEAM-2740 into ** on apache:DSL_SQL**.

Merged, feel free to close.


No docstrings shown for stdlib symbols

Language Server version: 2023.12.104 (regression from 2023.12.103)
OS and version: Windows 11
Python version (& distribution if applicable, e.g. Anaconda): 3.12.1

Code Snippet
dict()

Repro Steps

Hover over dict

Expected behavior
Hover output contains dict docstring:

Actual behavior

done

This issue has been fixed in prerelease version 2024.1.100, which we've just released. You can find the changelog here: CHANGELOG.md


Front end - configure an Email type connector as a datastore

Comment added by Michael Brown in Aha! - View
Users can get the information about mail opens, clicks, etc from the mail provider.

Comment added by Michael Brown in Aha! - View
Works from the API side. This will get moved to a different release.


In 'lab chat', tell the user if you are using a default model, please validate --model or -m before running

After running lab init, lab check, lab generate, lab train --device cuda (on Linux), I end up with a new .gguf file:
$ ls -l models/
total 14145392
-rw-rw-r-- 1 dperique dperique<PHONE_NUMBER>0 Mar 25 16:43 ggml-model-f16.gguf
lrwxrwxrwx 1 dperique dperique         148 Mar 15 20:56 merlinite-7b-Q4_K_M.gguf -> ../../../../../../../../.cache/huggingface/hub/models--ibm--merlinite-7b-GGUF/blobs/94f3a16321c9604ca22e970f3b89931ae5b4bbfd4c5d996e2bb606c506590666

This is the merged model (i.e., original model plus adapter).  If I try to chat with it via lab chat -m ggml-model-f16.gguf, it appears to load:
$ lab chat -m ggml-model-f16.gguf
╭──────────────────────────────────────────────────────────── system ────────────────────────────────────────────────────────────╮
│ Welcome to Chat CLI w/ GGML-MODEL-F16.GGUF (type /h for help)                                                                  │
╰────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
>>>                                                                                [S][default]

But that is NOT the correct model -- even though it confidently implies it in the "Welcome ..." message.
You can observe this more clearly using a non existent file:
$ lab chat -m something_non_existent.lala
╭──────────────────────────────────────────────────────────── system ────────────────────────────────────────────────────────────╮
│ Welcome to Chat CLI w/ SOMETHING_NON_EXISTENT.LALA (type /h for help)                                                          │
╰────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
>>>

Clearly, we're defaulting to something (probably in the config.yaml) but the user should be informed explicitly -- silence is not acceptable.
With silence, the user may try to test the model manually only to find it does not appear to be fine-tuned.  This in fact was my case as I spent a lot of time trying to understand why my resultant gguf file didn't seem to incorporate my dataset.
Using the correct file loads the expected model:
$ lab chat -m ./models/ggml-model-f16.gguf
╭──────────────────────────────────────────────────────────── system ────────────────────────────────────────────────────────────╮
│ Welcome to Chat CLI w/ ./MODELS/GGML-MODEL-F16.GGUF (type /h for help)                                                         │
╰────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
>>>

And my tests show the model was fine-tuned as expected.
We should validate the existence of the value of the --model or -m parameter to avoid the user being confused about what model is being used and/or invalidating long hours of testing.

@cdoern suggesting for 5/30, when user specifies --model-path, we funnel that into the next lab chat unless they specify --model, and we wire --model up to ab chat to do something meaningful. when they do --model-path in lab serve, changes it in the config. so config.serv.modelpath is used for source of truth for chatting. Something in there needs to be changed for 5.2.
user experience - if you serve model A, then do chat... it doesn't necessarily use model A (it uses the model configured as default in the config.)
Default should probably be that chat always uses the model being served... or the model most recently served...


Run keyword and ignore error changes the type of variable

Hello,
I have a problem related to the changing of the variable type when using Run Keyword and Ignore Error.
I'm getting the epoch time type from DateTime:

${start_time}=    DateTime.Get Current Date    result_format=epoch

which is float (due to epoch time from specs regarding DateTime: Epoch time is returned as a floating point number.)
But when I'm executing:

Run Keyword And Ignore Error    Get and Check Startup Time '${start_time}'

I've got unicode type.
The python script executed inside "Get and Check Startup Time '${start_time}'" keyword is demanding float numbers, so:

if I'm using above KW without Run Keyword And Ignore Error I got passing keyword because ${start_time} is floating type,
if I'm using above KW with Run Keyword And Ignore Error I got an error because ${start_time} is a unicode type.

I think using Run Keyword And Ignore Error should just hide the result, not change it so it cannot be executed in similar situations like mentioned above. Thanks!

Although ${start_time} would be a float, '${start_time}' is a Unicode string.
Also, if this Get and Check Startup Time is your own keyword, it would be a good idea to either make it configurable like ignore_errors=False or possibly adding a separate keyword. Run Keyword And Ignore Error hides all errors, even bugs in your library, and should be used only with external libraries.

No, '${x}' is always a string in Robot's data regardless what ${x} is. There's nothing about changing argument types in Run Keyword And Ignore Error documentation simply because it doesn't do anything like that.

Well, my proposition is to execute the below code.
Execute the code, please:
At least please check it
	${start_time}=    get something
	check something ${start_time}

*** Keywords ***
get something
    ${start_time}=    DateTime.Get Current Date    result_format=epoch
    Log    epoch: ${start_time}
    [return]    ${start_time}

check something ${start_time}
	<execute python script which needs float variable here and use ${start_time}>

and then execute:

At least please check it
	${start_time}=    get something
	Run Keyword And Ignore Error    check something ${start_time}

*** Keywords ***
get something
    ${start_time}=    DateTime.Get Current Date    result_format=epoch
    Log    epoch: ${start_time}
    [return]    ${start_time}

check something ${start_time}
	<execute python script which needs float variable here and use ${start_time}>

See the difference, please.
My code returns below when I use the Run Keyword And Ignore Error (the <type 'unicode'> is printed by python):

13:01:08.881
INFO
<type 'unicode'>

13:01:08.885
FAIL
TypeError: unsupported operand type(s) for -: 'float' and 'unicode'

In the example without using Run Keyword and Ignore Error:

13:22:13.504
INFO
<type 'float'>

Well, I see the problem and I want to know why is this change happening. Execute this, please.

Oh, you are using embedded arguments. That wasn't clear from the original description. This is a known problem that is unfortunately complicated to fix. See #1595 for details.

Thank you for pointing right issue :).