[Feature Request] Ability to Remove Games

Currently (as far as I am aware) there is no way to remove downloaded games through steam-tui. It would be a nice addition and be really convenient.
PS: I think the aur package is a bit behind the repo so whoever maintains it might want to update it.

Hmm, this makes sense but is also dangerous?
steamcmd does have a app_uninstall : <appId>, but casually hitting a button and uninstalling a game seems like a pain?
There's also no logging of the spawned threads, so no way of getting immediate feedback if an uninstall did occur (see #20).
The best way of doing this might be this might be running steam-tui with a flag, e.g. steam-tui --dangerous which would let you press x to uninstall a game.
I am also unsure whether an 'uninstall' removes files/saves/cache, or just purges whatever local database.
What were you thinking? Thoughts?
RE: PS: Yes it is. Cargo is also behind main (we now have auto-downloaded and cached images!), I'll sync everything up next big change I make.

Hmm, this makes sense but is also dangerous?
steamcmd does have a app_uninstall : <appId>, but casually hitting a button and uninstalling a game seems like a pain?
There's also no logging of the spawned threads, so no way of getting immediate feedback if an uninstall did occur (see #20).
The best way of doing this might be this might be running steam-tui with a flag, e.g. steam-tui --dangerous which would let you press x to uninstall a game.
I am also unsure whether an 'uninstall' removes files/saves/cache, or just purges whatever local database.
What were you thinking? Thoughts?
RE: PS: Yes it is. Cargo is also behind main (we now have auto-downloaded and cached images!), I'll sync everything up next big change I make.

I can see how just pressing a button to delete every file might be dangerous. A confirm prompt or as you said a flag to start a "dangerous" mode might be the best option.I think steamcmd just removes every folder it creates for a game but I might have to test it out. Also steamcmd has a forceinstalldir function which might be simple to implement into steam-tui if you are interested.

I think "dangerous" is the easiest work around for now. We can always make this default in the config later and add in a confirmation prompt. I might have some time later this week to test and try out app_uninstall
Yep! I am aware of the forceinstalldir. If not default, the steam directory has to be properly set as well (see #7)

How does steam-tui handle install progress right now?

Shows that's it's downloading, and will show when it's done. It should show a % for install too, but I haven't seen it yet....
Relevant code:
https://github.com/dmadisetti/steam-tui/blob/a27a4ea408740d48e5c21fcb2f3899447f8fe985/src/client.rs#L134

Hey, I was wondering if I could pick up this issue. It would be my first open source contribution, I would also appreciate any help with getting started.

So you can hide games now. I think to remove games we'd still want a yes/no prompt.
So steps would be,

Add keybinding to pick up delete key, and put into a confirmation mode
Add keybinding for yes, no on confirmation mode
Run a app uninstall script

At least for a first pass

There's H to hide games, but I'm going to mark this as won't fix (unless someone adds it), cause I don't want to accidently delete my games


Update nebula info from 8.2.0 to 11.4.1

Description
This change is intended to resolve CVE-2020-15522 which states:

origin/1.x
org.bouncycastle/bcprov-jdk15on 1.64
buildSrc/build.gradle runtimeClasspath<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>buildSrc/build.gradle runtimeClasspath<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>buildSrc/build.gradle runtimeClasspath<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>buildSrc/build.gradle testFixturesRuntimeClasspath<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>buildSrc/build.gradle testFixturesRuntimeClasspath<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>buildSrc/build.gradle testFixturesRuntimeClasspath<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>buildSrc/build.gradle testRuntimeClasspath<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>buildSrc/build.gradle testRuntimeClasspath<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>buildSrc/build.gradle testRuntimeClasspath<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
Check List

[ ] ~New functionality includes testing.~

[ ] ~All tests pass~

[ ] ~New functionality has been documented.~

[ ] ~New functionality has javadoc added~

[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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

This should be backported to 1.3


Maps not Orion v1.0 compartible

Will this be ported over to Orion V1.0?

Yes it will, Was actually in the middle of working on it, want to ensure 1.1 doesn't introduce any breaking changes. I will be updating within the next couple weeks. Possibly sooner.

That will be nice
On 5/27/15, Ryan Watts<EMAIL_ADDRESS>wrote:

Yes it will, Was actually in the middle of working on it, want to ensure 1.1
doesn't introduce any breaking changes. I will be updating within the next
couple weeks. Possibly sooner.

Reply to this email directly or view it on GitHub:
https://github.com/rwatts3/orionjs-contrib/issues/5#issuecomment-106152459

Hi, I'm also really interested in maps! ;) Any way to help you?

Thank you.  I was supposed to have fm this done a while back and got side
tracked.  I'll work on it this weekend.
On Friday, August 28, 2015, Ignacy<EMAIL_ADDRESS>wrote:

Hi, I'm also really interested in maps! ;) Any way to help you?
—
Reply to this email directly or view it on GitHub
https://github.com/rwatts3/orionjs-contrib/issues/5#issuecomment-135726164
.

--
Ryan Watts
<EMAIL_ADDRESS>C.520-442-8962
F.520-437-0914
IN. https://www.linkedin.com/in/ryandwatts

Hey. @rwatts3, what happened with the project?


ethereum.sendAsync doesn't work

Open alphawallet
Go to https://multisender.app

I use ethereum.enable to get eth address
Another issue is that I don't allow multiple tabs for the same url, and alphawallet doesn't close a session of opened window, hence I can't use my dapp anymore

https://rstormsf.github.io/js-eth-personal-sign-examples/
click check web3 methods

https://github.com/rstormsf/js-eth-personal-sign-examples/blob/master/index.js#L19

@all-contributors please add @rstormsf for bug

Closing as this is abstraction / architecture.

__


Version number wrong in latest tag (v0.1.14-alpha)

The verion number is wrong in the latest tag (v0.1.14-alpha, https://github.com/driftyco/ng-cordova/blob/v0.1.14-alpha/dist/ng-cordova.js#L3)

Since we're on 0.1.17; this seems like not something to worry about any more.


altair-express-middleware - support typescript

altair-express-middleware does not contain typescript definitions at the moment.
To embed it currently, I use something like that in my app:
    serverApp.use(serverconfig.options.prefix + '/altair', require('altair-express-middleware').altairExpress({
        endpointURL: serverconfig.options.prefix + '/graphql'
    }));

I would look into this. Does the middleware not work at all at the moment? Or you are just concerned about the autocomplete options?

Fixed.


Incorrect fr-fr translation (text shown in Chinese)

The French version of this page is shown in Chinese.
https://docs.microsoft.com/fr-fr/azure/active-directory/develop/active-directory-v2-protocols

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

ID: e50110dd-3c4d-6e42-8f71-332062279f30
Version Independent ID: 0ae0df0d-9fbb-861a-a50a-9e2f3792d6b1
Content: OAuth 2.0 and OpenID Connect protocols on the Microsoft identity platform - Microsoft identity platform
Content Source: articles/active-directory/develop/active-directory-v2-protocols.md
Service: active-directory
Sub-service: develop
GitHub Login: @nickludwig
Microsoft Alias: ludwignick

I could not find info on how to contact localization team.

@X00169650 This isn't something we can fix at the article editing level, so I'll close this issue and open a dialog internally with the localization team via the email thread.
#please-close
Cc: @SaurabhSharma-MSFT @nickludwig


Nightly restore bench: Make buildkite red if it fails

Issue Number
ADP-804
Overview
If the nightly restore bench fails to produce a result, then make sure Buildkite has a red status.

bors r+

bors r+

bors r+


update k8sgtp dockerfile

Address this failure: https://qe-private-deck-ci.apps.ci.l2s4.p1.openshiftapps.com/view/gs/qe-private-deck/pr-logs/pull/openshift_release/42096/rehearse-42096-periodic-ci-openshift-openshift-tests-private-release-4.14-amd64-nightly-alibaba-ipi-private-fips-f28/1689506642434985984
INFO[2023-08-10T06:17:13Z] /bin/bash: line 11: k8sgpt: command not found

Related to https://github.com/openshift/release/pull/42096

/approve
/lgtm

/lgtm


ps.sub("sort", ...) or folder-specific rules

yazi --debug output
Yazi
    Version: 0.3.3 (Arch Linux 2024-09-05)
    Debug  : false
    OS     : linux-x86_64 (unix)

Ya
    Version: 0.3.3 (Arch Linux 2024-09-05)

Emulator
    Emulator.via_env: ("xterm-kitty", "")
    Emulator.via_csi: Ok(Kitty)
    Emulator.detect : Kitty

Adapter
    Adapter.matches: Kitty

Desktop
    XDG_SESSION_TYPE           : Some("wayland")
    WAYLAND_DISPLAY            : Some("wayland-1")
    DISPLAY                    : Some(":0")
    SWAYSOCK                   : None
    HYPRLAND_INSTANCE_SIGNATURE: Some("a425fbebe4cf4238e48a42f724ef2208959d66cf_1732736821_412414895")
    WAYFIRE_SOCKET             : None

SSH
    shared.in_ssh_connection: false

WSL
    WSL: false

Variables
    SHELL              : Some("/usr/bin/zsh")
    EDITOR             : Some("nvim")
    VISUAL             : Some("nvim")
    YAZI_FILE_ONE      : None
    YAZI_CONFIG_HOME   : None

Text Opener
    default: Some(Opener { run: "${EDITOR:-vi} \"$@\"", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })
    block  : Some(Opener { run: "${EDITOR:-vi} \"$@\"", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })

Multiplexers
    TMUX               : false
    tmux version       : tmux 3.5a
    ZELLIJ_SESSION_NAME: None
    Zellij version     : No such file or directory (os error 2)

Dependencies
    file             : 5.45
    ueberzugpp       : 2.9.6
    ffmpegthumbnailer: 2.2.3
    magick           : 7.1.1-40
    fzf              : 0.55.0
    fd               : 10.2.0
    rg               : 14.1.1
    chafa            : 1.14.2
    zoxide           : 0.9.6
    7z               : 17.05
    7zz              : No such file or directory (os error 2)
    jq               : 1.7.1

--------------------------------------------------
When reporting a bug, please also upload the `yazi.log` log file - only upload the most recent content by time.
You can find it in the "$HOME/.local/state/yazi" directory.

Please describe the problem you're trying to solve
I'm trying to use yazi's size sorting as a replacement for ncdu and similar programs.

When using Yazi normally, I want the ~/Downloads directory to be sorted by modification time. (https://yazi-rs.github.io/docs/tips/#folder-rules achieves this).
When cleaning up files, all directories (including ~/Downloads) should be sorted by size (partially achieved with some modifications to https://yazi-rs.github.io/docs/tips/#folder-rules).

The best solution I got:
local function setup()
    ps.sub("cd", function()
        local cwd = cx.active.current.cwd
        -- Assuming "alphabetical" is the default
        if cwd:ends_with("Downloads") and cx.active.conf.sort_by == "alphabetical" then
            p_sort = cx.active.conf.sort_by
            p_reverse = cx.active.conf.sort_reverse
            p_dir = cx.active.conf.sort_dir_first
            ya.manager_emit("sort", { "modified", reverse = true, dir_first = false })
        else
            ya.manager_emit("sort", { p_sort, reverse = p_reverse, dir_first = p_dir })
        end
    end)
end

return { setup = setup }

This works for most cases except when the sort type is changed within ~/Downloads. However I realized that any changes I make to this code won't solve the problem because there is no event I can listen for to save the previous sort type. The cd  event only triggers when entering a directory, so i can't track if the sort type was changed while inside ~/Downloads (Also adding code in the else part to check against the previous sort type will fail in other cases which I think is a lot to write but can be deduced).
Would you be willing to contribute this feature?

[X] Yes, I'll give it a shot

Describe the solution you'd like
Add a sort kind to the Builtin kinds or a configuration option that does the same. Assuming the first solution, the problem will be solved by saving the sort type inside the callback of ps.sub("sort", ...) and restoring it in the else part of the ps.sub("cd", ...) callback.
Additional context
No response
Validations

[X] I have searched the existing issues/discussions
[X] The latest nightly build doesn't already have this feature

Nice to have!
To implement it, we need to add a new kind Sort under Load here:
https://github.com/sxyazi/yazi/blob/2771aef677305b9bbaab16dda78ab666744770c1/yazi-dds/src/body/body.rs#L16
Then, when setting the sorter, publish it:
https://github.com/sxyazi/yazi/blob/2771aef677305b9bbaab16dda78ab666744770c1/yazi-core/src/tab/tab.rs#L129
Would you like to raise a PR for it?

Hi @sxyazi, I actually implement the Sort kind to at least print the value of sort_by when it's changed by the user, but it had several problems. I realized that I only needed to listen when the user changed the sort type, so I overrode the default sort keymaps (the ones starting with ,) and it worked exactly as I wanted. I leave my solution in case someone needs it.
keymap.toml, I use ,, to "reactivate" the directory rules.
[manager]
prepend_keymap = [
    # ...
    { on = [ ",", "m" ], run = [ "sort modified --reverse=no", "linemode mtime", "plugin folder-rules --sync --args='hi'" ], desc = "Sort by modified time" },
    { on = [ ",", "M" ], run = [ "sort modified --reverse", "linemode mtime", "plugin folder-rules --sync --args='hi'" ], desc = "Sort by modified time (reverse)" },
    { on = [ ",", "c" ], run = [ "sort created --reverse=no", "linemode ctime", "plugin folder-rules --sync --args='hi'" ], desc = "Sort by created time" },
    { on = [ ",", "C" ], run = [ "sort created --reverse", "linemode ctime", "plugin folder-rules --sync --args='hi'" ], desc = "Sort by created time (reverse)" },
    { on = [ ",", "e" ], run = [ "sort extension --reverse=no", "plugin folder-rules --sync --args='hi'" ], desc = "Sort by extension" },
    { on = [ ",", "E" ], run = [ "sort extension --reverse", "plugin folder-rules --sync --args='hi'" ], desc = "Sort by extension (reverse)" },
    { on = [ ",", "a" ], run = [ "sort alphabetical --reverse=no --dir-first", "plugin folder-rules --sync --args='hi'" ], desc = "Sort alphabetically" },
    { on = [ ",", "A" ], run = [ "sort alphabetical --reverse", "plugin folder-rules --sync --args='hi'" ], desc = "Sort alphabetically (reverse)" },
    { on = [ ",", "n" ], run = [ "sort natural --reverse=no", "plugin folder-rules --sync --args='hi'" ], desc = "Sort naturally" },
    { on = [ ",", "N" ], run = [ "sort natural --reverse", "plugin folder-rules --sync --args='hi'" ], desc = "Sort naturally (reverse)" },
    { on = [ ",", "s" ], run = [ "sort size --reverse=no", "linemode size", "plugin folder-rules --sync --args='hi'" ], desc = "Sort by size" },
    { on = [ ",", "S" ], run = [ "sort size --reverse", "linemode size", "plugin folder-rules --sync --args='hi'" ], desc = "Sort by size (reverse)" },
    { on = [ ",", "r" ], run = [ "sort random --reverse=no", "plugin folder-rules --sync --args='hi'" ], desc = "Sort randomly" },
    { on = [ ",", "," ], run = [ "plugin folder-rules --sync" ], desc = "Return to normality" },
]

folder-rules.yazi/init.lua
local M = {}

local function execute_rules(cx)
    local cwd = cx.active.current.cwd
    if cwd:ends_with("Downloads") then
        ya.manager_emit("sort", { "modified", reverse = true, dir_first = false })
    else
        ya.manager_emit("sort", { "alphabetical", reverse = false, dir_first = true })
    end
end

function M.setup()
    ps.sub("cd", function()
        if not user then
            execute_rules(cx)
        end
    end)
end

function M:entry(args)
    user = args[1] and true or false
    if not user then -- refresh
        execute_rules(cx)
    end
end

return M

Nice to see you've found a solution!

but it had several problems

Could you tell me what problems have you had? Is this feature request still valuable to you - can I close it?

@sxyazi you can close it, I don't need the feature request anymore.

Could you tell me what problems have you had?

For example ya.manager_emit(“sort”, { “modified”, ...}) changed the sort type to “none”, publish a SortBy object below f.files.set_sorter() made the callback of ps.sub(“sort”, ...) to be called many more times than I expected and that I couldn't 100% know if the sort was changed by the user anyway (that's why overriding the default keymaps works perfectly). But as I said, it doesn't matter because the problem is solved.

Ah I see. Thanks for the explanation!


CommandsNext command parsing problem

Currently, CNext parses first command element per-space (instead of per-whitespace).
This is so I don't forget.
😒

#Blame_moonPtr

Resolved, forgot to close.


feat: connector client: switch PolicyService

What this PR changes/adds
Use EDC-connector-client for `ContractDefinition
Why it does that
Briefly state why the change was necessary.
Further notes
List other areas of code that have changed but are not necessarily linked to the main feature. This could be method
signature changes, package declarations, bugs that were encountered and were fixed inline, etc.
Linked Issue(s)
Closes #58 and Closese #65

At the current state on creating a new Policy the values are not used given in the Dialog.
Also i have the question about the assignee and assigner of a policy. Does the still exist? Because i don't see this fields in the management-Api Documentation. Or i just did not see it? @ndr-brt

@janpmeyer they are part of the ODRL model, but they are not used by the EDC, so they can be avoided

@ndr-brt fixed now the id of an policyDefiniton and also the permission field of the policy. But the fields prohibition and obligation are not working yet

With the new EDC-Connector-Client Beta-6 obligations and prohibitions are also working.


fix: test file name

Description
Renamed test file
Additional Information

[x] I have read the contributing docs (if this is your first contribution)
[x] This is not a duplicate of any existing pull request

Your ENS/address:
0xrinat

Ohh lol tysm Rinat !


Begin integration of new fasm2bels with interchange support

This adds CMake logic to invoke the interchange and create DCPs with RapidWright, but leaves that logic dormant unless RAPIDWRIGHT_PATH is supplied pointing to an version of RapidWright with the interchange support present.

FYI @mithro


Tags in SiyuanBlock

New data field in Siyuan v2.0.4, see https://github.com/siyuan-note/siyuan/issues/4740

Closed via 3ce46480aef13e524c7905ef17ae60ac64cbdd49


Drupal 7 compatibility

Does this project have compatibility with Drupal 7?

It does not. I think for Drupal 7 you could easily configure PHPStan and not need some of the autoloading magic, since it's all files and no PSR autoloading.
I have no plan on adding Drupal 7 support.


Better errors for indexing gettable/settable values

Gives suggestions where a type has no index signature but has get or set properties.
Fixes #26098

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

Thanks for being patient! This is a good start! I left some feedback to iterate on.

Hi @DanielRosenwasser, I've made the requested changes. Feel free to take another look at this, thanks!

@collin5 would be good to merge this after the last round of comments + merge conflicts are addressed. Thanks!

Updated. Thank you.


Enha: Exceptions not handled by user code should be unhandled: true

:scroll: Description

Closes #456
Always set to unhandled, as there are no handlers provided:

IsolateErrorIntegration
RunZonedGuardedIntegration

Set to unhandled if there are no user provided handlers:

FlutterErrorIntegration
OnErrorIntegration

Verified in tests that handled is null on the mechanism. Technically those should also be unhandled, as users are not manually calling the Sentry SDK, but failed requests are something so common and should probably not be unhandled?

IoExceptionEventProcessor
FailedRequestClient
FailedRequestInterceptor

:green_heart: How did you test it?
Added unit tests.
:pencil: Checklist

[ ] I reviewed submitted code
[x] I added tests to verify changes
[ ] I updated the docs if needed
[x] All tests passing
[ ] No breaking changes

:crystal_ball: Next steps
Review every individually created mechanism if the handled flag makes sense.

Codecov Report

:exclamation: No coverage uploaded for pull request base (v7.0.0@3e6109b). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             v7.0.0    #1186   +/-   ##
=========================================
  Coverage          ?   95.83%
=========================================
  Files             ?        2
  Lines             ?       48
  Branches          ?        0
=========================================
  Hits              ?       46
  Misses            ?        2
  Partials          ?        0

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

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

@denrase please check if the _eventWithRemovedBreadcrumbsIfHandled is not causing duplicated breadcrumbs as well because it depends on the handled flag and now its flipped.

@marandaneto Also removing them from unhandled but not chrashing event mechanisms that we create in the SDK.

@marandaneto Also removing them from unhandled but not crashing event mechanisms that we create in the SDK.

Not sure I understand this comment, mind elaborating?
Another pending point if not done yet, we have to test the integration with the Native SDKs and the release health feature.

@marandaneto Sorry, hope this is a better explanation.

Not sure I understand this comment, mind elaborating?

Ah sure, i was referring to the documentation of why we introduced the breadcrumb removal.

We then remove the breadcrumbs in all cases but if it is handled == false, this is a signal that the app would crash and android would lose the breadcrumbs by the time the app is restarted to read the envelope.

My assumption is that we still need to remove the breadcrumbs in those new instances where we have set the value to handled == false, as the app is not crashing on flutter and we would still have duplicated breadcrumbs.
I have added unit tests that cover this new behaviour.

@marandaneto Raised an issue on the cocoa side.

Closing in favor of https://github.com/getsentry/sentry-dart/pull/1535


Fix for --unauthenticated-client-action V2 issue

Previously, using az webapp auth --unauthenticated-client-action argument did not correctly update Unauthenticated Client Action configuration setting if the argument was set to RejectWith401, RejectWith404, or RejectWith403 due a naming mismatch between the CLI and the API surface.
Previous issues raised for this: https://github.com/Azure/azure-cli/issues/24086, https://github.com/Azure/azure-cli/issues/21618
The fix was to rename some of the allowed values for --unauthenticated-client-action. Also added the Return403 value. Allowed Values: RedirectToLoginPage, AllowAnonymous, Return401, Return404, Return403

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
The precondition is to put your code inside this repository and upgrade the version in the pull request but do not modify src/index.json.

webapp

Rerecorded the live tests, CI is now passing. @zhoxing-ms, could you help with approving?

@annzho Please add the description of PR into the HISTORY.rst. If you want to release a new version for this PR, please also upgrade the version defined in the setup.py.

@zhoxing-ms Have added the description of the new version in HISTORY.rst and upgraded version in setup.py. Thanks!

@zhoxing-ms, moved the version notes to top of history file, could you approve?


Remove filter for "RemovedInDjango40Warning"

In the latest release we included a fix to remove deprecation warnings generated from Wagtail due to their gettext implementation (https://github.com/wagtail/wagtail/issues/5842).
This will be solved in Wagtail 2.9, so I file this issue as a reminder to remove this filter. Ref: https://github.com/Frojd/Wagtail-Pipit/blob/develop/{{cookiecutter.project_name}}/src/pytest.ini#L7

Closed in https://github.com/Frojd/Wagtail-Pipit/commit/abd3ddc4188a110dfa38bad7e6833daa56ca87a0 since Wagtail is at 2.10rc2 right now.


How do we achieve nesting of nodes?

Is it possible to achieve nesting of nodes. Similar to react-dnd.
EX:

Boxes - Droppable / draggable

VeggieBox - Droppable / draggable

Carrots - Draggable

DairyBox - Droppable / draggable

Milk - Draggable

How do we achieve this?
Also would it be possible to integrate it with react-flow (https://github.com/wbkd/react-flow) Droppable & draggable as types of nodes.
Any help in this regard would be appreciated. Thanks

I'd be interested in this as well. I've tried using nested DnDContext to handle the different draggable types - The problem I found was that draggables from one context cannot be dropped into droppables in a different context. From the docs useDroppable and useDraggable hooks will only have access to the other draggable and droppable nodes within that context.
The event handlers only expose the droppable's id property meaning it makes it difficult to sort the nested nodes after a drop. So for example, if I drag the Carrots over the Boxes, the event handler only has id of the carrot which is not enough to know that it's not the type of droppable I want to allow into the Boxes Droppable.
I'm very new to this library so could have overlooked something obvious.

@salali independent of this issue, id is a string - you can prefix each item with a type e.g. 'carrot-1', 'milk-1', to do checks.


fix(amplify-nodejs-function-runtime-provider): change the exit status appropriately

I get an error even though Lambda is running correctly.
I tried to test the Lambda resolver using GraphiQL, but I got an error.
However, looking at the output, the Lambda resolver appears to be handling it correctly.
# Output in case of an error.
Lambda Data source failed with the following error
Command failed with exit code 1: /Users/nika/.anyenv/envs/nodenv/versions/12.18.3/bin/node /Users/nika/Documents/workspace/project/node_modules/amplify-nodejs-function-runtime-provider/lib/utils/execute.js

{ username: 'test' }
{"data":{"listUsers":{"items":[{"id":"8d7105ff-fd2a-0d77-d817-cb032e7626b0","username":"test","email":"test@test.com","type":"Admin","channel_ids":["eedc2c40-f714-cb81-29a6-6c8abae660c3"],"created_at":"12345","updated_at":"12345","__typename":"User"}],"nextToken":null,"__typename":"UserConnection"}},"loading":false,"networkStatus":7,"stale":false}

{"result":{"username":"test","id":"8d7105ff-fd2a-0d77-d817-cb032e7626b0","email":"test@test.com","type":"Admin","channel_ids":["eedc2c40-f714-cb81-29a6-6c8abae660c3"],"created_at":"12345","updated_at":"12345","__typename":"User"},"error":null}
{ username: 'test' }
{"data":{"listUsers":{"items":[{"id":"8d7105ff-fd2a-0d77-d817-cb032e7626b0","username":"test","email":"test@test.com","type":"Admin","channel_ids":["eedc2c40-f714-cb81-29a6-6c8abae660c3"],"created_at":"12345","updated_at":"12345","__typename":"User"}],"nextToken":null,"__typename":"UserConnection"}},"loading":false,"networkStatus":7,"stale":false}

When I deployed to AWS, I found that all queries, mutations, and subscriptions worked correctly.
And I've confirmed that someone else is encountering a similar Issue as follows.
https://github.com/aws-amplify/amplify-cli/issues/5378
I used the link above to modify the amplify-nodejs-function-runtage-provider/lib/utils/utils/execute.js in node_modules to make sure the Lambda resolver continues to work correctly.
For more information, see the following issue.
https://github.com/bboure/serverless-appsync-simulator/issues/51#issuecomment-709808350
Issue #, if available:
It was a minor renovation and I hope to be able to discuss it on this PR, if necessary.
Description of changes:
I changed from exit(1) to exit(0) in amplify-cli/packages/amplify-nodejs-function-runtime-provider/src/utils/execute.ts.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Thank you for submitting this fix! After digging into the execute.ts we also just discovered this, see my comment about it here: https://github.com/bboure/serverless-appsync-simulator/issues/51#issuecomment-716637182
For us it prevents us from chaining Lambdas as GQL Query resolvers.
Looking forward to you guys merging and shipping it 🎊


118-The Pascal's triangle in Python 3

Given a non-negative integer numRows, generate the first numRows of Pascal's triangle.In Pascal's triangle, each number is the sum of the two numbers directly above it.
Code
class Solution:
    def generate(self, numRows: int) -> List[List[int]]:
        if numRows==0:
            return []
        triangle=[[1]]
        for row in range(numRows-1):
            triangle.append([1]+[triangle[-1][i]+triangle[-1][i+1] for i in range(row)]+[1])
        return triangle

Link To The LeetCode Problem
LeetCode

I wanto to try this

Sure @ovasgus ..Go Ahead


Tests won't run on OTP 24

Maybe it's intended for the lib. to not be tested/testable, at that time, but I guess we can try to preemptively fix these warnings.
Warning: erlang:get_stacktrace/0 is deprecated and will be removed in OTP 24; use use the new try/catch syntax for retrieving the stack backtrace

(obtained with rebar3 ct on OTP 23).

The tests should run without issues - those warnings have no effect (the actual test code invokes the compiler on its own, outside of rebar3, in order to ensure that the parse transform does what it's supposed to do.)
But I think we can avoid the warnings by moving the test module away from rebar3's source paths. I'll give it a try.

I think we can avoid the warnings by moving the test module away from rebar3's source paths.

Easier said than done - rebar3 looks recursively for .erl files within test/, so a subdirectory is not an option. Changing the extension of test_module.erl doesn't cut it, either, since it breaks the compiler invocation within the test suite.
I think I'll just apply some ifdefs and duct tape and be done with it; that should be enough to make rebar3 ignore the code that produces the warnings.

I think there's an option for rebar3 to not go recursive. Let me find that for you.

Fixed in 4de68f9 and released under 1.2.1.

Found it. You can disable the warnings with nowarn_deprecated_function is profile test's erl_opts.

You can disable the warnings with nowarn_deprecated_function is profile test's erl_opts.

Thanks! (Perhaps my twin in a parallel universe where this issue still needs fixing will make good use of your solution.)


feat(marksman): enable for markdown.mdx

marksman works for MDX files. We don't want to set ft=markdown, since there exist tools that don't work for MDX, so we want to keep .md and .mdx as separate filetypes.

closing since there is an actual mdx language server: https://github.com/mdx-js/mdx-analyzer/pull/226


Add Derive(Deserialize) to AST structs

Description of changes
Added Derive(Deserialize) to the AST structs used within AuthorizationRequest.
This allows us to test roundtrip equality which catches issues like #1042
Note that this assumes that PR #1047 is merged first.
Issue #, if available
N/A
Checklist for requesting a review
The change in this PR is (choose one, and delete the other options):

[x] A change "invisible" to users (e.g., documentation, changes to "internal" crates like cedar-policy-core, cedar-validator, etc.)

I confirm that this PR (choose one, and delete the other options):

[x] Does not update the CHANGELOG because my change does not significantly impact released code.

I confirm that cedar-spec (choose one, and delete the other options):

[x] Does not require updates because my change does not impact the Cedar formal model or DRT infrastructure.
This PR by itself does not affect DRT, but #1047 which this is based on does.

Ah, it's interesting to see that we didn't have or need this deserialization code before.
From a testing point of view, it would be nice to retain the code so we can catch bugs (if any) in the de/serialization on the Rust side.  Though I understand John's concerns about maintaining more code just for the purpose of testing.
Curious to see what others think.

I don't think the de-serialization make this so much worse. Really, the fact that DRT depends on the serialization (and hence our internal representation) of these structures is the more annoying part.

Closed


Fix MonMan configuration

It is now in HTTP

Closed by PR #706

Revert this configuration

Closed by PR #707


ROX-14066 Fix sorting of client side attached collection cache

Description
When moving collections between the "attached" and "available" lists, sort the resulting lists by name, case insensitive, to match the expected order from the BE API.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Manual testing:
View the available collections list and verify that the collections are sorted by name, case insensitive.

Verify that clicking "view more" loads new collections that are alphabetically after the ones that currently appear in the list:

Add multiple collections to the "attached" list. Verify that the sort order remains consistent regardless of the order that the items are added.

Current dependencies on/for this PR:

master

PR #4493   👈

This comment was auto-generated by Graphite.

/retest


Drop redundant OpenGL files and add metaclass support for Surface

Changelog / Overview

OpenGL<x> classes from opengl_geometry.py, opengl_text_mobject.py, opengl_tex_mobject.py, opengl_svg_path.py, opengl_svg_mobject.py and most of opengl_three_dimensions.py have been removed.
ParametricSurface has been renamed to Surface

Motivation

Less bloat and easier pathway to OpenGL transition
Explanation for Changes

Removed:

opengl_geometry.py
opengl_tex_mobject.py
opengl_text_mobject.py
opengl_svg_path.py
opengl_svg_mobject.py
most of opengl_three_dimensions.py

OpenGLSurfaceMesh was left behind because it's opengl specific (relies on OpenGLSurface)

if config.renderer == opengl checks in mobject.py and vmobject.py

three_dimensions.py

Renamed ParametricSurface to Surface
Added metaclass capability for Surface/OpenGLSurface, now 3D mobs will rely on OpenGLSurface/Surface. depending on the renderer.
Resolution adjusts depending on the renderer for Torus and Sphere
Sphere was reworked and had some of its defaults adjusted since it was producing wrong results with the previous implementation in the OpenGLRenderer. essentially ported from OpenGLSphere.

example_scenes/opengl.py

Adjusted the examples to accomodate for the changes
Ensured that the scenes look the same.

Documentation Reference

N/A
Testing Status

Some quick code:
class Test(Scene):
    def construct(self):
        sph = Sphere()
        tor = Torus()
        vgr = Group(sph, tor).arrange()
        self.add(vgr)
        self.interactive_embed()

cairo:

opengl:

Further Comments

Renamed ParametricSurface since it was probably going to have to happen eventually.
Checklist

[x] I have read the Contributing Guidelines
[x] I have written a descriptive PR title (see top of PR template for examples)
[x] I have written a changelog entry for the PR or deem it unnecessary
[x] My new functions/classes either have a docstring or are private
[ ] My new functions/classes have tests added and (optional) examples in the docs
[x] My new documentation builds, looks correctly formatted, and adds no additional build warnings

Reviewer Checklist

[ ] The PR title is descriptive enough
[ ] The PR is labeled correctly
[ ] The changelog entry is completed if necessary
[ ] Newly added functions/classes either have a docstring or are private
[ ] Newly added functions/classes have tests added and (optional) examples in the docs
[ ] Newly added documentation builds, looks correctly formatted, and adds no additional build warnings

There are some issues stemming from the fact that OpenGLSurface is a Mobject, whereas the old surfaces were VMobjects, which breaks certain classes (such as the polyhedra) in opengl. Also, I'm not too sure about changing ParametricSurface's name. I've made this PR a draft for now 😅

Changes:

Added a fix to make BraceLabel appear (use .add instead of setting submobjects directly, because assemble_family needs to be called)
Rewrote tests (unnoticeable visual differences)
Deprecate ParametricSurface instead of breaking it
Removed if config.renderer==opengl checks in mobject.py

polyhedra.py still won't work in opengl since it's mixing the Mobject from OpenGLSurface with the VMobject from Polyhedra + Graph. Rendering each submobject individually this commit allows it to be created, but still doesn't let certain methods like .rotate work, since they have different implementations in opengl_vmobject.py / opengl_mobject.py. I reverted the original fix so that the issue won't be hidden.

I'm not sure how to resolve the polyhedra problem, since perfectly valid mobjects have some functionality stripped simply because of how they're grouped 🤷. It should probably be taken care of in a separate PR.

I've updated the changelog to reflect the fact that this introduces breaking changes to OpenGL<x> classes.

This PR is dependent on #1899 now to make test_attr.py work.  I've rewritten the test_opengl.py graphical test since OpenGLArc used 8 components, whereas regular Arc uses 9 components.
class Arc(TipapleVMmobject)
    def __init__(
        self,
        radius: float = 1.0,
        start_angle=0,
        angle=TAU / 4,
        num_components=9, # was 8 for opengl
        arc_center=ORIGIN,
        **kwargs,
    ):

Also I had to add a config.renderer="cairo" at the end of test_set_attr.py or else the following tests failed. But now that tests work, it's no longer a draft 🎉.

@hydrobeam the error I get, along with the manim command I am running is:
alexlembcke@system76-pc:~/ManimDev/manim$ poetry run manim ../surface_gradient.py SurfaceTest --renderer=opengl -p
Manim Community v0.8.0

╭─────────────────────────────── Traceback (most recent call last) ────────────────────────────────╮
│ /home/alexlembcke/ManimDev/manim/manim/cli/render/commands.py:109 in render                      │
│                                                                                │
│   106 │   │   │   │   │   scene_classes = scene_classes_from_file(file)                          │
│   107 │   │   │   │   │   SceneClass = scene_classes[0]                                          │
│   108 │   │   │   │   │   scene = SceneClass(renderer)                                           │
│ ❱ 109 │   │   │   │   │   status = scene.render()                                                │
│   110 │   │   │   │   │   if status:                                                             │
│   111 │   │   │   │   │   │   continue                                                           │
│   112 │   │   │   │   │   else:                                                                  │
│                                                                                │
│ /home/alexlembcke/ManimDev/manim/manim/scene/scene.py:215 in render                              │
│                                                                                │
│    212 │   │   """                                                                               │
│    213 │   │   self.setup()                                                                      │
│    214 │   │   try:                                                                              │
│ ❱  215 │   │   │   self.construct()                                                              │
│    216 │   │   except EndSceneEarlyException:                                                    │
│    217 │   │   │   pass                                                                          │
│    218 │   │   except RerunSceneException as e:                                                  │
│                                                                                │
│ /home/alexlembcke/ManimDev/manim/../surface_gradient.py:27 in construct                          │
│                                                                                │
│   24 │   │   │   v_range=[-3, 3]                                                                 │
│   25 │   │   )                                                                                │
│   26 │   │                                                                                │
│ ❱ 27 │   │   saddle_plane.set_fill_by_value(axes=axes, colors=[BLUE, GREEN, YELLOW, ORANGE, R    │
│   28 │   │   self.add(axes, saddle_plane)                                                        │
│   29 │   │                                                                                │
│   30 │   │   self.interactive_embed()                                                            │
│                                                                                │
│ /home/alexlembcke/ManimDev/manim/manim/mobject/three_dimensions.py:233 in set_fill_by_value      │
│                                                                                │
│   230 │   │   │   )                                                                              │
│   231 │   │                                                                                │
│   232 │   │   for mob in self.family_members_with_points():                                      │
│ ❱ 233 │   │   │   z_value = axes.point_to_coords(mob.get_midpoint())[2]                          │
│   234 │   │   │   if z_value <= pivots[0]:                                                       │
│   235 │   │   │   │   mob.set_color(new_colors[0])                                               │
│   236 │   │   │   elif z_value >= pivots[-1]:                                                    │
╰──────────────────────────────────────────────────────────────────────────────────────────────────╯
AttributeError: 'ThreeDVMobject' object has no attribute 'get_midpoint'

And if I copy ~.get_midpoint() to ThreeDVMobject and run poetry install then this error goes away.  If others are not experiencing the same thing, then perhaps it is due to something on my side.


Service Account user

@chrisism thank you for your great work on this task. We're experiencing a minor issue.
We do local deployments by installing the VSTS Agent locally on the target machine. The agent runs under the built-in Windows service account "Network Service".
It seems that we need to grant the "Network Service" user Modify rights on the target deployment folder for Robocopy task to work? Otherwise we get an Access Denied error.
Although this is a fix, we don't like this security-wise.
And we'd like to avoid doing a "net use" command because this requires converting the target deployment folder to a network share which requires extra configuration work and slows down the deployment.
Perhaps you could include 2 parameters in the task so the credentials to run robocopy under can be chosen independent of the credentials the VSTS Agent runs under?
Or if you have a better idea?

Thanks for your input. Why don't you let the agent run under a local user or domain user that has access rights? At my current company we have a custom domain user that is allowed just enough and we have configured the agent to run under that context. I would advise against using "Network service" or any of the default user contexts for the service. It is easier to set the rights on a folder to a specific user.
That being said, I will have a look in how I can achieve this.

I have considered that, but I fear that other tasks may need extra priviliges I'm not aware of that Network Service has. For example connecting to an external database to run SQL scripts.

As far as I am aware and what we do in practice is that you can use the domain user for everything. It is a special service user created in our domain. When running SQL Server with AD you can simply give that specific user all the SQL privileges it needs. Same goes with read/write access to folders. Actually with a dedicated user it is easier to exactly check what this user can do or access.
Only a suggestion though and my own prefered way of solving this.


Some menu fixes and integration tests.

What does the pull request do?
Fixes #7906. While investigating/fixing it I noticed a couple of other problems:

Clicking on a top-level menu item and then pressing the down arrow didn't select the first child menu item
If a top-level menu has a single item and the left/right arrow keys were pressed, the item closes and then re-opens causing a flicker

Added integration tests for all of these issues and fixed them.

Don't focus top-level menu items when selected unless the menu itself is open in MenuItem (fixes #7906)
In DefaultMenuInteractionHandler, select the first item when down is pressed if the submenu is open
In DefaultMenuInteractionHandler, when moving left/right from a top-level menu item, check if the newly selected item is the same as the previously selected item before closing/opening

Fixed issues
Fixes #7906

You can test this PR using the following package version. 0.10.999-cibuild0019767-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

You can test this PR using the following package version. 0.10.999-cibuild0019769-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

@MarchingCube that should have been fixed by https://github.com/AvaloniaUI/Avalonia/pull/7954 I think?

You can test this PR using the following package version. 0.10.999-cibuild0019859-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

lots of conflicts on backport. if this is needed for stable please provide an implementation for stable.


Added fuzzytoken matcher performance test

Fuzzy-token matcher performance test matches the queries against all indices in ./index/standard/.
Test results includes minimum runtime, maximum runtime, average runtime, the standard deviation and the average results for each index, each threshold and each test cycle. They are written in a csv file that is named by current time and located at ./data-files/results/fuzzytoken/.

LGTM.


Produced Dirty Html

After some stick and unstick events, this is produced html. Goes too deep, too many div inside itself.
Is this a bug? Funny question.

You need to share the code, are you disabling the correct element?

`$(window).resize(function () {
restoreSticky();
maxWidth = Math.max($(window).width(), window.innerWidth);
});
function restoreSticky() {
if (maxWidth > 991) {
$('.navCont').css('height', ['90px']);
$(".navCont").sticky();
    } else {
        $(".navCont").unstick();
        $('.navCont').css('height', ['100%']);
    }
}`

I had the same issue for what it's worth. As @atilkan said, it occurs on window resize.

I have found a very easy fix. I just need to call unstick BEFORE I call stick.
var $el = $('.element');
window.onresize = function() {
  if ($(window).width() >= 768) {
	  $el.unstick();
	  $el.sticky({topSpacing: 0, bottomSpacing: 0});
  } else {
	  $el.unstick();
  }
};


Feature request library(aggregate)

Wanted to run Numbrix Puzzle. Banged my head here:
?- time(aggregate_all(count, (length(_L, 16), line(_L)), C)).
   throw(error(existence_error(procedure,aggregate_all/3),aggregate_all/3)).

I know that I can use countall/2 instead:
?- time(countall((length(_L, 16), line(_L)), C)).
% Time elapsed 0.163s, 1138078 Inferences, 6.989 MLips
   C = 552.

But the countall/2 looks a little limited to me. Its
not future proof. What if we want i.e. failure driven loop
summing or failure driven multiplication,
does this mean for every new aggregate there will
be a new predicate sumall/2, predall/2, etc... Not
sure whether this is a  way ahead,
since library(aggregate) usually offers more. In
the world of aggregates there are equivalents of
aggregates to findall/3 and bagof/3 which can
reuse the same aggregates. So what the library
offers is only 2-3 predicates such as aggregate_all/3
and aggregate/3, plus a set of aggregates.

Try copying aggregate.pl over on your system. I won't include it in Trealla as it seems to be the sole work of Jan.
I couldn't access your numbrix attachment.

Not sure whether you can run SWI-Prolog library(aggregate)
on your system. To realize failure driven versions of count, sum, etc..
it uses setarg/3, which is an implementation specific feature of SWI-Prolog.
BTW: Second try to upload Numbrix:
numbrix.p.log

With nb_setarg/3 you can also easily program call_nth/2
in Prolog itself. Or even bootstrap another useful library,
which is llibrary(solution_sequences).
I introduced nb_setarg/3 in the form of change_arg/3 in
Dogelog Player.  But I used it to bootstrap
library(sequence) which has call_nth/2. So nb_setarg/3 is quite handy!

I won't be adding setarg/3 or nb_setarg/3.

Since you have findall/3 and bagof/3:
?- findall(X, between(1,3,X), L).
   L = [1,2,3].
?- bagof(X, (between(1,3,Y), between(1,Y,X)), L).
   Y = 1, L = [1]
;  Y = 2, L = [1,2]
;  Y = 3, L = [1,2,3].

You could use this one, it works without nb_setarg/3:
aggregate.p.log

The question is, does it work?
On Mon, 7 Oct 2024, 22:38 Jean-Luc-Picard-2021, @.***>
wrote:

Since you have findall/3 and bagof/3:
?- findall(X, between(1,3,X), L).
L = [1,2,3].
?- bagof(X, (between(1,3,Y), between(1,Y,X)), L).
Y = 1, L = [1]
;  Y = 2, L = [1,2]
;  Y = 3, L = [1,2,3].
You could use this one, it works without nb_setarg/3:
aggregate.p.log
https://github.com/user-attachments/files/17278552/aggregate.p.log
—
Reply to this email directly, view it on GitHub
https://github.com/trealla-prolog/trealla/issues/603#issuecomment-2396813765,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFNKSEUPRTPDGR237JV7H7TZ2J6DVAVCNFSM6AAAAABPOEF3ZSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJWHAYTGNZWGU
.
You are receiving this because you commented.Message ID:
@.***>

Of course:
~/trealla$ ./tpl -v
Trealla Prolog (c) Infradig 2020-2024, v2.57.1

~/trealla$ ./tpl
?- ['aggregate.p'].
   true.
?- aggregate_all(sum(X), between(1,3,X), S).
   S = 6.
?- aggregate(sum(X), (between(1,3,Y), between(1,Y,X)), S).
   Y = 1, S = 1
;  Y = 2, S = 3
;  Y = 3, S = 6.

And what's the provenance?
On Mon, 7 Oct 2024, 22:56 Jean-Luc-Picard-2021, @.***>
wrote:

Of course:
~/trealla$ ./tpl -v
Trealla Prolog (c) Infradig 2020-2024, v2.57.1
~/trealla$ ./tpl
?- ['aggregate.p'].
true.
?- aggregate_all(sum(X), between(1,3,X), S).
S = 6.
?- aggregate(sum(X), (between(1,3,Y), between(1,Y,X)), S).
Y = 1, S = 1
;  Y = 2, S = 3
;  Y = 3, S = 6.
—
Reply to this email directly, view it on GitHub
https://github.com/trealla-prolog/trealla/issues/603#issuecomment-2396854162,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFNKSETNI6VN2LE4RZCYIMTZ2KAGHAVCNFSM6AAAAABPOEF3ZSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJWHA2TIMJWGI
.
You are receiving this because you commented.Message ID:
@.***>

You can also bootstrap setof/3:
bagof(X, G, L) :-
    aggregate(set(X), G, L)

I think this was all already present in DEC-10 Prolog.

I'll look at it tomorrow.
On Mon, 7 Oct 2024, 23:12 Jean-Luc-Picard-2021, @.***>
wrote:

Closed #603 https://github.com/trealla-prolog/trealla/issues/603 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/trealla-prolog/trealla/issues/603#event-14540438479,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFNKSEWVTK72YGLF6LNPIT3Z2KCDRAVCNFSM6AAAAABPOEF3ZSVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJUGU2DANBTHA2DOOI
.
You are receiving this because you commented.Message ID:
@.***>

You put label "wontfix", but you can also Close with wont-fix:

I think the label "wontfix" is not needed anymore in GitHub issues.
Thats a new feature that I see the recent weeks.

More earlier reference than SICStus Prolog:
Copyright 1986, Fernando C.N. Pereira and David H.D. Warren
https://www.swi-prolog.org/pack/file_details/prologmud_I7/prolog/chat80/aggreg.pl?show=src
But wasn't Chat80 recently donated to the community?

Can we close this now?
$ tpl numbrix.pl
?- use_module(library(aggregate)).
   true.
?- time(aggregate_all(count, (length(_L, 16), line(_L)), C)).
% Time elapsed 0.241s, 1139737 Inferences, 4.733 MLips
   C = 552.
?-

Ok, it landed here. That was quick:
https://github.com/trealla-prolog/trealla/blob/main/library/aggregate.pl
Do you need test cases? I got some here, awful file name:
https://www.novacuor.ch/srctab/doclet/docs/13_comply/frequent/extend/struct.html

It has a small discontiguous predicate error:
sys_aggr_init(A, _) :-
   throw(error(type_error(aggregate,A),_)).

The above should read:
sys_aggr_init(A, _, _) :-
   throw(error(type_error(aggregate,A),_)).

Also I am currently preparing a new version with fast path.

Here is a version with fast path:
aggregate2.p.log
You can run it in GNU Prolog using this bridge:
change_arg(N, X, T) :- setarg(N, X, T, false).

But GNU Prolog only supports atomic via the above approach,
so bag/1 and set/1 aggregate will not work. But they still work,
even in the fast path:
?- aggregate_all((bag(X),set(X)), (between(1,3,X);between(1,3,X)), L).
L = ([1, 2, 3, 1, 2, 3], [1, 2, 3]).

The bag/1 and set/1 aggregate use some routines from my findall/3.

The test cases are the same. But you see a performance difference:

Is that speed up linear with the number of items?

In SWI-Prolog you can try:
aggregate_all2(Template, Goal0, Result) :-
    aggregate:template_to_pattern(all, Template, Pattern, Goal0, Goal, Aggregate),
    findall(Pattern, Goal, List),
    aggregate:aggregate_list(Aggregate, List, Result).

Without fast path:
?- aggregate_all2(sum(X), between(1,100000000,X), Y).
ERROR: Stack limit (1.0Gb) exceeded

With fast path:
?- aggregate_all(sum(X), between(1,100000000,X), Y).
Y =<PHONE_NUMBER>000000.

Here is a test case where I get an Killed because
aggregate_all/3 without fast path consumes too much memory:
/* Trealla Prolog */
?- test.
% Time elapsed 0.043s, 464977 Inferences, 10.807 MLips
552
% Time elapsed 3.068s, 41680528 Inferences, 13.588 MLips
8648
Killed

SWI-Prolog doesn't crash and recent version of Dogelog Player
with fast path doesn't crash either. Its a version of Numbrix:
numbrix.p.log
Disclaimer: I didn't really check whether the Trealla abort
is inside aggregate_all/3 or somewhere else. It could be
somewhere else, but still cause by aggregate_all/3.

In case you want to add meta_predicate/1 declarations,
you could use the same as from SWI-Prolog:
:- meta_predicate
    foreach(0,0),
    aggregate(?,^,-),
    aggregate(?,?,^,-),
    aggregate_all(?,0,-),
    aggregate_all(?,?,0,-).

Only my library I provided doesn't implement foreach/2,
aggregate/4 and aggregate_all/4.
But you could adopted the implementation from
SWI-Prolog as well.


Add param group support

It works pretty much the same as regular parameter changes.
Regular parameter message:
Parameter Changed: new_test_key
Group parameter message:
Parameter Changed in group "test group": test

@jamesrwu I am closing this PR. We've added handling of groups via a different PR that we just merged. We appreciate you putting this PR in place a while back.


Feature/apvs 1372/updates to advance claim content

Using DB call to get isAdvanceClaim on pages to change content displayed
Made content changes to pages based on whether advance claim or not based on https://docs.google.com/presentation/d/1w81JdBr6udjucn7l5YMVcV4Ed0bYqUx4EtlzLBkBPdg/edit

Adding IsAdvanceClaim calls
Updating views
Updating tests

Checklist

[x] Unit tests

refactor isAdvanceClaim to getIsAdvanceClaim and return true/false


Plane: front face culled ?

I'm trying to render a Plane, using a fragment shader that sets the fragment color (R, G) = texture coordinates (U, V).
Here's what it looks like from the back:

So far, so good: the color is black at (0, 0) and yellow at (1, 1).
But the same Plane refuses to render its front side:

What am I missing ?
EDIT: I'm coming from OpenGL, so I'm interpreting the first screenshot as "front" and the second as "back".

Hey! I apologize for the slow reply.

To turn it off, add a "Render State" object above your Plane object (right click on shader pass -> Add -> Render state). Uncheck the "Cull" checkbox:

Thank you, that will certainly be useful. That wasn't really the question though: what I don't understand is why the horizontal coordinates appear to be reversed. That would be expected if I were rendering the back side, but I'm not.
Is something wrong with the Plane I'm using ? It's set to TriangleList, in fact all parameters are at their default values.

Thanks, that would explain it.
On Tue, Sep 28, 2021 at 4:44 AM naota @.***> wrote:

I believe the actual issue is that normal of the created plane is
incorrectly reversed. GeometryFactory::CreatePlane appears to be using
the opposite winding order to the other geometries.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dfranx/SHADERed/issues/244#issuecomment-928661456,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGCFCGNHLP5D5BTZ5N7N5TUEETXRANCNFSM5DEH47RQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I see the same issue (flipped coordinates) happening here. Why is this thread closed? It doesn't seems to be solved.


Fixed all build ordering issues (MLDB-1411)

This improves the documentation and fixes all build ordering issues.
It was verified that the following command will successfully complete the build:
make

ie, one job at a time, no restarting the build, no continuing on failure.

I'll trust you on that one. +1

Thanks for the documentation of the build process.


[Question]: What implications does sharing your UserManager config have?

I'm currently writing an example app for my library. For that I've registered the app at Google's OpenID provider. Of course the settings used for the UserManager, including the client_id will be accessible via GitHub. What implications does this have? Should I hide the client_id to make sure no one can hijack my app?

In addition to that, I've tried registering my sample app at Google. I've passed the client_id, generated by Google into the UserManager settings. However when I run the app and try to authenticate I get the following error:

XMLHttpRequest cannot load https://accounts.google.com/.well-known/openid-configuration. No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'https://localhost:8080' is therefore not allowed access.

I've used the following settings:
{
  client_id: 'secret_for_now',
  redirect_uri: `${window.location.protocol}//${window.location.hostname}:${window.location.port}/callback`,
  response_type: 'token id_token',
  scope: 'openid profile',
  authority: 'https://accounts.google.com/',
  silent_redirect_uri: `${window.location.protocol}//${window.location.hostname}:${window.location.port}/silent_renew.html`,
  automaticSilentRenew: true,
  filterProtocolClaims: true,
  loadUserInfo: true
};

XMLHttpRequest cannot load https://accounts.google.com/.well-known/openid-configuration. No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'https://localhost:8080' is therefore not allowed access.
I did tell Google about my localhost being a valid host and I've also registered the redirect URL as well as silent redirect URL. What could be causing this?

You need to configure your client's origin as being allowed CORS origins in google's dev console.

client_id is something public since it's in JS that an end user would be running, so it's not confidential.

It seems like Google's discovery document doesn't have any CORS support. Is there a way for me to set the endpoint url's manually and circumvent the call to the discovery document?

Yep, you can bypass the authority and set all the other URLs manually on the settings. Also, tho, you should set an authority since that's the "scoping" identifier used to keep state for the signin.

The interesting one will be the signing keys...

Along these lines, I need to design the id_token validation around an abstraction to allow this to be done somewhere else. The default will use the keys, but there might be other ways.

So this should work?
// copy & paste google's discovery doc
const settings = {
 "issuer": "https://accounts.google.com",
 "authorization_endpoint": "https://accounts.google.com/o/oauth2/v2/auth",
 "token_endpoint": "https://www.googleapis.com/oauth2/v4/token",
 "userinfo_endpoint": "https://www.googleapis.com/oauth2/v3/userinfo",
 "revocation_endpoint": "https://accounts.google.com/o/oauth2/revoke",
 "jwks_uri": "https://www.googleapis.com/oauth2/v3/certs",
 "response_types_supported": [
  "code",
  "token",
  "id_token",
  "code token",
  "code id_token",
  "token id_token",
  "code token id_token",
  "none"
 ],
 "subject_types_supported": [
  "public"
 ],
 "id_token_signing_alg_values_supported": [
  "RS256"
 ],
 "scopes_supported": [
  "openid",
  "email",
  "profile"
 ],
 "token_endpoint_auth_methods_supported": [
  "client_secret_post",
  "client_secret_basic"
 ],
 "claims_supported": [
  "aud",
  "email",
  "email_verified",
  "exp",
  "family_name",
  "given_name",
  "iat",
  "iss",
  "locale",
  "name",
  "picture",
  "sub"
 ],
 "code_challenge_methods_supported": [
  "plain",
  "S256"
 ],

  // my settings here
client_id: 'no_secret',
redirect_uri: 'https://myApp.com',
...
};

const userManager = new UserManager(settings);

I updated the wiki with these additional settings: https://github.com/IdentityModel/oidc-client-js/wiki

Actually... still working on it...

Ok, that should be good now (forgot to include the metadata setting)

Got it to work with these settings:
{
  client_id: 'not_secret',
  redirect_uri: `${window.location.protocol}//${window.location.hostname}:${window.location.port}/callback`,
  response_type: 'token id_token',
  scope: 'openid profile',
  authority: 'https://accounts.google.com',
  silent_redirect_uri: `${window.location.protocol}//${window.location.hostname}:${window.location.port}/silent_renew.html`,
  automaticSilentRenew: true,
  filterProtocolClaims: true,
  loadUserInfo: true,
  metadata: {
    issuer: "https://accounts.google.com",
    authorization_endpoint: "https://accounts.google.com/o/oauth2/v2/auth",
    token_endpoint: "https://www.googleapis.com/oauth2/v4/token",
    userinfo_endpoint: "https://www.googleapis.com/oauth2/v3/userinfo",
    jwks_uri: "https://www.googleapis.com/oauth2/v3/certs",
    signingKeys: [
      {
        "kty": "RSA",
        "alg": "RS256",
        "use": "sig",
        "kid": "ba1b3ecbf5f87c50b6ef15f1bb50a8c13f17a497",
        "n": "oyMZAGpqaNdfSdWlUb8zDQab8Z99rt01UK5A4g6eMWrV7f-o2HopdZZtQebMkQ5UYMxFnjBoowwiVKaW_EWc3DCs6wwiYposR18kLZht2NKxRd9VykfsQLoxuC8fZVWQQ5c3Fznd-cRvPfbdZkdRj1Y_vkgzZQnK_Q5gcUAYsLJZodL_jTc9hRjys2F5wwt-npPpa1pMxSLY_IfCnrW7ucnvQAgCdXedI2D_s2Y-Wrks9kj0zwe31GTFI2ZUOQwdPafY982oGiO15l5fwrq-xjNxFo8JIfHBzPr3JVkWsPETmS74DJeKo172d6SccNsyNnBIUBlqxYt6IeNpktIwCw",
        "e": "AQAB"
      },
      {
        "kty": "RSA",
        "alg": "RS256",
        "use": "sig",
        "kid": "93d45f7c2d58c6eaba22373bf1adcdbe9ab49af8",
        "n": "wUSUDL3sRKAkDh9LwfGVdi_shYDB0bD23YQnVqUaKlLk0YHOuo7iS5CfQAdCnS_w-fZKTui2yGlCu4oVhVHgnTkAU3cWRhyQNjJxG_tfFQg2sID2UaCNCq7jMHTIbAsf4MmDgZ6HOpiKsafMGUfWLsJc0Loe0QJnIWywSPnHmcXjs9kWExp29Xu1ka7rGK5rsDUH39sfi_Px2IeqDPdOFqUDRLCO9goaKE-IlCbZlgv2lnAoLRkmSVu7IXyNRjQPawIfBCLw91PxYMDqBcnTMf1fHAlnXzPy68B_aodv6PgPg5li0Y7aTDiiRmbyQVl-w7MCqAD1dxxqapbIqhmmhQ",
        "e": "AQAB"
      },
      {
        "kty": "RSA",
        "alg": "RS256",
        "use": "sig",
        "kid": "df6dbd2fe4ef7f280fd258f4f9ba4d0c47084e45",
        "n": "vyM4hMZhtQM6hegDxcW1WMETR1d4ddtO8KZYrP86ax75L6TKKKfFSkIA0xGAtmnyxjGlfWi02DAj_giqSv2LSK5Ufmo79hOyQUgVbEyYgVkSFohYvBTAdY9lbIWJuzAxjplX7g3psrpv4sNdBdp1R3xvcNOiGBBUsuQJ7pZbez_wm3an07Gc2wfWNZpDYJFlhzusqRVCX4DY4OR_I1Fc9a97PRfKlKyjHzBT208YRap_IypSZrN5cQZ2ikAmPZDjX17ozaAhlXbPW-_NV31R6FmdhtH4BYz-uHSBvWvZWnUCQlk4PbfAJgf1pCNsP-qZoGtVXenyBF6nEjXnIlR7Hw",
        "e": "AQAB"
      }
    ]
  }
};

Many thanks again @Brock!

Reopening this because when I run signoutRedirect() with these settings, the logout occurs locally, but I get the following error:

es6.promise.js:119 Unhandled promise rejection Error: Metadata does not contain property end_session_endpoint(…)

Any ideas what causes this? There is no end_session_endpoint property on Google's discovery document.

Ah, ok. Yes, google does not support it. I should change the code to handle the fact that it's optional.

So I started looking at this and now I wonder why you'd call signoutRedirect if the OP doesn't support signout? Maybe your app/library doesn't know ahead of time?

In this case I think it is sufficient to call removeUser() on the user manager. signoutRedirect() in my project works fine except for the error message being logged - the user still gets removed from storage.

Yes, that's what the removeUser API is for. If the OP does support signout, then the browser needs to redirect there thus the different signoutRedirect API.


Select baseImage version for applications

You will be able to select for example node version to use.

Closes #380


Included TensorFlow CPU supported only?

After configuring sonnet, I found out tensorflow 1.01 is installed along with my original tensorflow-gpu 1.01. However only tensorflow 1.01 is recognized as the imported one... Plz share a instruction

Hi,
Can you provide a few more details? What platform are you on, and how did you (originally) install TF?

Thx for replying. I install TensorFlow-gpu
with virtualenv on Ubuntu 16.04 LTS. The installation is followed exactly
as same as the link on Sonnet. After configure Sonnet, surprisingly
tensorflow is installed along with the original tensorflow-gpu. When I try
import tensorflow/ Sonnet to run several examples, only CPU model is used.
I am totally new to tensorflow and python, thx to your help in advance.
On Mon, 10 Apr 2017 at 21:41, Malcolm Reynolds<EMAIL_ADDRESS>wrote:

Hi,
Can you provide a few more details? What platform are you on, and how did
you (originally) install TF?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/deepmind/sonnet/issues/14#issuecomment-293057033, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGVYgSZjHwZyy4djej-e6tbXPxL4U91jks5ruoXsgaJpZM4M46MY
.

--
Yu-ke LI
*PhD Candidate *
State Key Lab of LIESMARS, Wuhan Univ., Wuhan, China;
*Email<EMAIL_ADDRESS>*| Mobile: +(00)86 132 1279 9655
(China)

Did you try the fix mentioned in #3? I just installed with this and it didn't happen to me.

Yes now it runs smoothly. Plz close the issue


Improved search

We really want to be able to (a) search for anyone on the platform, not
just people in the same top-level room as me, (b) search for anybody
registered for the conference, not just people signed on to the platform
right now, and (c) send a chat message to anybody I can search for…
(JB: This is not hard. The ContextualActiveUsers already gets all of the presences and filters them to the current social space. So, the search bar could be set to search across all of the users.)

@anniejingchen This implementation is not acceptable.

The behavior of the left user list should be: show the users in the current room. Now, it shows all users, regardless of what room they are in
The code is extremely inefficient. On my 2019 MacBook Pro, loading the ICSE site now requires over 1GB of RAM, and expanding the total list of users in the sidebar takes 30-40 seconds. Please do not make an extra query to retrieve all UserProfiles: the correct way to do this is to call getUsers in withAuthentication. However, the sidebar really needs to be showing the UserPresences - I think that a better fix will be to search on-demand as the user types in the box.

Please let me know if you can resolve this today, if not, I will revert the commit so that master continues to have the correct behavior for the sidebar.
Thanks

Just to clarify, here is my understanding of how the left user list should behave:

The list displayed on the left should be just as it always has been -- i.e., it should show just users in the current room.
When the user starts a search by typing in the search bar, the search results should be displayed incrementally (i.e., at any moment, the list of results under the search bar is showing all matching names, and the list is updated as the user types)
The list of results should include (in this order)
- all rooms whose names match
- all matching users that are in the current room
- all matching users that are not in the current room

I just reversed the commit

@anniejingchen, will we have this for ICFP?

@bcpierce00 Hi, Benjamin, I have a question about where to display the found user. After we find a user who we would like to send a DM and click the name, where will it be displayed in order to send her/him a DM? Should they be displayed on the left and replace the original list for the current room?  (The pictures are about how the searching work for a room now). However, based on what you mentioned above, the list displayed on the left should show just users in the current room. If this is the case, I am a little confusing. Could you give me more explanation? Thanks.

What about: Show the user in the drop down for the search (as shown on the lefT), and pressing “enter” opens the tooltip to display the user’s profile, but does not filter the list of users in the current room.

I just implemented this in the new chat search. All that is left is to clean it up to show the user's profile in the popover when it's searching.

This is looking great!  One issue is that the search bar is hard to find at the bottom of the screen and its placement makes it hard to guess that it can be used for things besides chats.  Could we move it to the top-level menu bar or some other more global place?


[Fix #3746] Lint/NonLocalExitFromIterator doesn't warn about block passed to Object#define_singleton_method

returns don't cause problems in blocks passed to define_singleton_method.

👍


The logger middleware outputs into stderr

The logger middleware outputs into stderr what makes really difficult to read real error messages in the log file

@NikosEfthias can you explain some more please?
however, I have just committed 46d56496c539ad41168671a6b6503f0bacb30a78 which will let you set your own middleware.DefaultLogger = etc... and you can change the logger that you pass to it, so you can specify a file or whatever else you like.
cheers, happy NY

@pkieltyka have you added documentation on how to change that ?  because i have no idea what you mean by middleware.DefaultLogger should i edit the source code? middleware is unexported keyword

@pkieltyka have you added documentation on how to change that ?  because i have no idea what you mean by middleware.DefaultLogger should i edit the source code? middleware is unexported keyword

@NikosEfthias thats not something to really document, and middleware is a package, not an unexported keyword.
something like so is what you want..
// from within your program

// adjust the customLogger as you want, to a file or however you want
customLogger := log.New(os.Stdout, "", log.LstdFlags)

middleware.DefaultLogger = middleware.RequestLogger(&middleware.DefaultLogFormatter{logger: customLogger})

just one thing that i don't understand is that i checked the source code theres nothing about stderr how come the messages from logger middleware ends up in the stderr stream

the logger in defaultLogger struct is unexported so its impossible to set it should be exported value i guess

Sorry, I am not a help centre for all inquires. I wrote chi and I released it for public so others could benefit from it and as well contribute to build it together - but unfortunately I don't have the bandwidth to answer every question, especially this when it's clearly exported: https://github.com/pressly/chi/blob/master/middleware/logger.go#L15

@NikosEfthias indeed you are right.

i just capitalized the first letter of logger key in the struct and it works otherwise it gives error because naturally it cannot find such a property of the struct DefaultLogFormatter because logger is not an exported keyword.
However after the last update i dont need this anymore the problem that was the reason that i made this bug report about was solved by this update .

this will fix it: 076deb80de4f344b761681f2afadf708430cfd41

next time for such small things ill open a pr sorry :)

all good :) thanks for bringing it to my attention, but yes PRs are definitely welcome too!


RC test: Coulomb's Law 1.0.0-rc.6

Round 6!!!
@KatieWoe, @arouinfar, @jessegreenberg, @ariel-phet  coulombs-law/1.0.0-rc.6 is ready for a final spot RC check. Document issues in https://github.com/phetsims/coulombs-law/issues and link to this issue.
The latest round of issues were pretty minor, so I think just briefly checking major browsers/platforms should be sufficient.
Assigning @ariel-phet for prioritization.

General RC Test

What to Test

Click every single button.
Play with the sim normally.
Try to break the sim.
Test all query parameters on all platforms. (See QA Book for a list of query parameters.)
Complete the test matrix.
Check this LoL spreadsheet and notify AR if it not there.

Issues to Verify

[x] Boxes overlab
[x] Text goes off screen
[x] Figure won't move past a point
[x] Update color of dashed lines
[x] Dilate touch areas on charge NumberControls
[x] Charges jump when keyboard nav is used
[x] Ruler pointer area gets left behind when ruler is moved

Note that Legends of Learning page freeze hasn't been addressed, but we may publish this sim without a fix— @ariel-phet will make the call.
These issues should have the "status:fixed-pending-testing" label. Check these issues off and close them if they are fixed. Otherwise, post a comment in the issue saying that it wasn't fixed.

Link(s)

Simulation
iFrame
XHTML
Test Matrix
Legends of Learning Harness

Accessibility (a11y) RC Test

What to Test

Specific instructions can be found below.
Make sure the a11y feature doesn't negatively affect the sim in any way.
Test all possible forms of input.

Test all mouse/trackpad inputs.
Test all touchscreen inputs.
Test all keyboard navigation inputs (if applicable).

Link(s)

a11y View
Simulation
Test Matrix

Keyboard Navigation
This sim supports keyboard navigation. Please make sure it works as intended.

FAQs for QA Members

There are multiple tests in this issue... Which test should I do first?
Test in order! Test the first thing first, the second thing second, and so on.

How should I format my issue?

We typically assign the developer who opened the issue in the QA repository.

My question isn't in here... What should I do?
You should:

Consult the QA Book.
Google it.
Ask Katie.
Ask a developer.
Google it again.
Cry.

@mbarlow12 how does this list look for platforms?

[ ] Win 10 chrome Full
[ ] Win 10 firefox
[ ] Win 10 edge Full

[ ] Mac 10.14 chrome
[ ] Mac 10.14 firefox Full
[ ] Mac 10.14 safari Full

[ ] iOS 12 Full
[ ] iOS  10

[ ] Chrome OS

[ ] Win 7 IE Full
[ ] Win 7 chrome
[ ] Mac 10.11 safari
[ ] Mac 10.11 firefox

@KatieWoe Graphing Quadratics and this sim are on similar priority level

On iOS 10 the sim froze and crashed fairly quickly.  This also seemed to happen in graphing quadratics, but it was determined to be an issue with the device, which may be the case here.  I reset the device and will see if that improves the situation. @mbarlow12

Reset device and it seemed better, but crash still occurred after 10 minutes.

QA is done @mbarlow12

New RC issue in https://github.com/phetsims/QA/issues/242

Closing for new RC https://github.com/phetsims/QA/issues/243


API returns error code 503 (Service Unavailable) but APOD site running fine.

When I try to connect to API and get the JSON dictionary data, it throws error 503 (Service Unavailable) It's been 17 hours since the issue started. There is a stack trace of the error:

Error occurred at 05.06.2023 14:11:46:
Response status code does not indicate success: 503 (Service Unavailable).
at System.Net.Http.HttpResponseMessage.EnsureSuccessStatusCode()
at System.Net.Http.HttpClient.GetStringAsyncCore(HttpRequestMessage request, CancellationToken cancellationToken)
at AstronomyPictureOfTheDayWallpaperApp.WallpaperAPODloader.LoadPicture() in C:\Users\FrosterDune\source\repos\AstronomyPictureOfTheDayWallpaperApp\AstronomyPictureOfTheDayWallpaperApp\WallpaperAPODloader.cs:line 56
at AstronomyPictureOfTheDayWallpaperApp.MainForm.ActivateBT_Click(Object sender, EventArgs e) in C:\Users\FrosterDune\source\repos\AstronomyPictureOfTheDayWallpaperApp\AstronomyPictureOfTheDayWallpaperApp\MainForm.cs:line 68

I've reported this to the NASA CIO Operations - they're looking into it.

Very long :(

When I try to connect to API and get the JSON dictionary data, it throws error 503 (Service Unavailable) It's been 17 hours since the issue started. There is a stack trace of the error:

Error occurred at 05.06.2023 14:11:46:
Response status code does not indicate success: 503 (Service Unavailable).
at System.Net.Http.HttpResponseMessage.EnsureSuccessStatusCode()
at System.Net.Http.HttpClient.GetStringAsyncCore(HttpRequestMessage request, CancellationToken cancellationToken)
at AstronomyPictureOfTheDayWallpaperApp.WallpaperAPODloader.LoadPicture() in C:\Users\FrosterDune\source\repos\AstronomyPictureOfTheDayWallpaperApp\AstronomyPictureOfTheDayWallpaperApp\WallpaperAPODloader.cs:line 56
at AstronomyPictureOfTheDayWallpaperApp.MainForm.ActivateBT_Click(Object sender, EventArgs e) in C:\Users\FrosterDune\source\repos\AstronomyPictureOfTheDayWallpaperApp\AstronomyPictureOfTheDayWallpaperApp\MainForm.cs:line 68

The apod api is down for now, I hope they fix it soon.

Hurry up and fix it, you aliens. I'm using this API for my assignment. Please fix it

You guys are funny for not having a fallback for when a third-party API is not working 👀
No issues on our website when the APOD API doesn't return a daily pic… 😎
https://github.com/zorgch/zorg-code/blob/f10957c28c7f3402936b50f85e1614d40444191a/www/includes/apod.inc.php#L73

Вы, ребята, забавны тем, что у вас нет запасного варианта, когда сторонний API не работает.👀
На нашем веб-сайте нет проблем, когда API APOD не возвращает ежедневное изображение…😎 https://github.com/zorgch/zorg-code/blob/f10957c28c7f3402936b50f85e1614d40444191a/www/includes/apod.inc.php#L73

parsing apod.nasa.gov?

You guys are funny for not having a fallback for when a third-party API is not working 👀
No issues on our website when the APOD API doesn't return a daily pic… 😎 https://github.com/zorgch/zorg-code/blob/f10957c28c7f3402936b50f85e1614d40444191a/www/includes/apod.inc.php#L73

It doesn't work on your site :)

same problem here. sadface.jpg hope they fix it soon.

Service temporarily unavailable. Please try again later. (Status Code: 503)  I'll just post manually to my bot for now :)

Service temporarily unavailable. Please try again later. (Status Code: 503)
I'll just post manually for the time being :)

I'm getting a 503 when trying to access the NASA APOD API.  I've never used any of the other NASA APIs.  Can anyone confirm if those are working?

I'm getting a 503 when trying to access the NASA APOD API. I've never used any of the other NASA APIs. Can anyone confirm if those are working?

Others work

Now it's returning:
502 Bad Gateway

Please fix it. My app doesn’t work

please please please please please

It's now a 502 Bad Gateway. Progress? Negative progress? Unclear.

Some time this morning it said i've used my key a bit too much. Right now the 502 bad gateway error is back

"no healthy upstream", wow, that’s progress

Same problem here, getting more and more reports from users of my app that they are not getting new pictures.

The issue persists... 503 here

When I try to connect to API and get the JSON dictionary data, it throws error 503 (Service Unavailable) It's been 17 hours since the issue started. There is a stack trace of the error:

Error occurred at 05.06.2023 14:11:46:
Response status code does not indicate success: 503 (Service Unavailable).
at System.Net.Http.HttpResponseMessage.EnsureSuccessStatusCode()
at System.Net.Http.HttpClient.GetStringAsyncCore(HttpRequestMessage request, CancellationToken cancellationToken)
at AstronomyPictureOfTheDayWallpaperApp.WallpaperAPODloader.LoadPicture() in C:\Users\FrosterDune\source\repos\AstronomyPictureOfTheDayWallpaperApp\AstronomyPictureOfTheDayWallpaperApp\WallpaperAPODloader.cs:line 56
at AstronomyPictureOfTheDayWallpaperApp.MainForm.ActivateBT_Click(Object sender, EventArgs e) in C:\Users\FrosterDune\source\repos\AstronomyPictureOfTheDayWallpaperApp\AstronomyPictureOfTheDayWallpaperApp\MainForm.cs:line 68

Yup. It's down for everyone. The API returns "no healthy upstream." I have a .NET MAUI mobile app written in C# called APOD which uses this data, and it fails to load it now.

Cool. It's working. Now let's not give it the hug of death you guys. Plz I like my pretty space images.

Confirmed. Bots working again 👏

Ok, The problem is still not solved yet, again I got 504 gateway timeout error code and now it will hold the 60s before throwing this error code

Was working for a few hours. Getting a 504 Gateway Time-out now.
My Error Page is finally getting some love :')

Same here 😞

Was working for a few hours. Getting a 504 Gateway Time-out now.
My Error Page is finally getting some love :')

Same same :(

I just return a picture of a black hole when the API does not hand me anything useful.

504 Gateway Time-out

It worked for like a day again, they when looking again today I just doesn't load anymore and returns a 504 error. Hopefully it can be fixed soon as I used the API for a school assignment!

Same problem here, I developed an app that changes my wallpaper with the daily APOD, but it hasn't been able to reach the endpoint for a few days now. It keeps returning a 503 error. It worked for a brief moment and then stopped again. Such a shame because I don't want to miss out on any of them :(

504 Gateway Time-out
I sat down yesterday to write a new web service using this API and it worked a couple of times (yesterday), and then stopped. I was wondering if I had done something wrong and was getting throttled. Happy to know it's not me. Unhappy to know this has been going on so long.

504 Gateway Time-out
I sat down yesterday to write a new web service using this API and it worked a couple of times (yesterday), and then stopped. I was wondering if I had done something wrong and was getting throttled. Happy to know it's not me. Unhappy to know this has been going on so long.

The problem is on the api side

504 Gateway Time-out

It's working now!

It works again, I will reopen this issue if the problem reappears, but I hope it won't be needed

Working great on my website, thanks for fixing it gang! 🚀

Also, if anyone's interested, I'm starting an unofficial NASA OpenAPI specification. It's part of another project I'm working on that will hopefully capture all endpoints available at api.nasa.gov. I only have three endpoints so far, but that's because I'm upgrading my local software to handle some unexpected edge cases.


adjust log levels

Adjust logger configuration (same as for 3.x: https://github.com/fheng/fhcap/commit/d8f51961572bac2756443c2286e9651a77a3ddb3)
ping @matzew

@austincunningham this are only changes to some log levels i think we can still test the previous PR that you already deployed to a 4.x cluster.

1000 on hibernate's ERROR logging (was a recommendation by EAP PERF. team)

and the pushy/netty is also fine at ERROR level ... or at least WARN ... especially w/ the fix in place, having WARN is fine   (ERROR is OK too)

I think for pushy/netty - with the fix in place - I'd go w/ WARN ;-)


Links on the main page are unavailable

If I go to https://www.shogun-toolbox.org/showroom I get "Internal Server Error".
If I go to https://cloud.shogun.ml/ I get "This site can’t be reached"
Sounds like something exciting should be there. Would be nice to make those links accessible again.

cloud is killed i'll look into the showroom thnx for reportint

If cloud is killed I would suggest to remove the link not to confuse anyone. Please see the picture below


binaryninja\python\binaryninja is empty after an update

(INSTALLPATH)binaryninja\python\binaryninja is empty after an update (at least on Windows). No python API for me.

It is the same for me, but I have python in the path so I am not experiencing any issues currently? I think... :palm_tree:

This is a bug in the Windows build, Python is broken due to a problem in the build. Additionally, armv7 isn't working (again, just on Windows) as well. New update should be out soon with that fix.

This was a problem with 278 that was fixed in 279.


Code and docs for CVPR 15' paper "DNNs are easily fooled" - with scripts to download Caffe

Code and documentation for the CVPR '15 paper "DNNs are Easily Fooled" are placed in the modules/dnns_easily_fooled directory.
This PR is for the Vision Challenge. http://code.opencv.org/projects/opencv/wiki/VisionChallenge
@vpisarev : Hi Vadim, following https://github.com/Itseez/opencv_contrib/pull/420, I have so created scripts for users to download proper patched versions of Caffe for two different experiments in this paper. Please kindly review and let me know if anything else needs to be improved.
Best,
Anh

@anguyen8, thanks! I have a couple of questions/requests regarding this PR:

please, correct the complains from our buildbot: pullrequest.opencv.org
the patch includes quite big (tough, it's smaller than caffe) external package sferes. Is it possible to document instead on how to install it separately?

@vpisarev : Thanks for your comments. I will update accordingly.

@vpisarev : Hi Vadim, I have made the requested changes:

The PR build-bot now passed
The patched Sferes is removed and instead replaced by a script for users to download it remotely. The included Installation_Guide.pdf has instructions on how to get Sferes running.

Thanks. Anh

cool! thanks! :+1:


Fluorine should use black instead of white font color

white on yellow is not really readable

I'll try to make it a bit more darker

Fixed in 3d39217


Initial start.

Not working...not complete, but it's how I'd start.
I can work on it a bit more this afternoon if you want, or you can see where I'm going with in and run.

Whoops
Didn't upstream pull

Well, have a look at:
https://github.com/ldecicco-USGS/mda.streams/blob/master/R/init_nwis_sites.R

we'll see what #judgementbot thinks

oh my, that is clean

Doesn't do the date check yet. Or multiple data descriptors


Fix openai function calling parameter format parsing error

additional_kwargs.function_call.arguments will sometimes be in a non-JSON format and should be compatible with the string format.
I added a test case (code-interpreter), when the parameter returned by OpenAI function calling is in non-JSON format, the following problems will occur:

This PR fixes this problem and supports parameter parsing in non-JSON format, which can be tested by yarn test:single ./src/agents/tests/openai.int.test.ts -t "OpenAIAgent code interpreter".
https://app.warp.dev/block/uopAv2WHtht4SB5lRT2xOI

Oh fascinating, I didn't know that. Will dig in!

Looks like it's a bug on OpenAI as it doesn't occur on gpt-4:
Python tool:
https://smith.langchain.com/public/88423f6d-11b6-4fbe-9946-e5752a853a35/r
JS tool:
https://smith.langchain.com/public/697be3a5-ff74-46d9-9aa6-8feb3c9266bd/r
I don't think it's necessarily the best fix to pass args directly through - could you try using GPT-4?

https:app.warp.dev/block/ooIj0isjJ7mwXcf7NJOnf4
I tried GPT4 and still get the problem... Maybe it's because of my prompt?

I don't think we should update the parser just for this since OpenAI is actually returning something in the incorrect format... Going to close this for now.
This person said few-shotting the prompt seemed to help:
https://twitter.com/bennetkrause/status/1691361566912880641?s=20


fix: able to run webpack install in second dependencies`s dir

What kind of change does this PR introduce?

Did you add tests for your changes?

Does this PR introduce a breaking change?

What needs to be documented once your changes are merged?

pull details in https://github.com/webpack/webpack/issues/7461

webpack 2 is deprecated and we no longer accept PRs to it. Please upgrade.


TensorFlow Object Detection API with Keras model

Please make sure that this is an issue related to performance of TensorFlow.
As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:performance_template
System information

Have I written custom code (as opposed to using a stock
example script provided in TensorFlow): partially
OS Platform and Distribution (e.g.,
Linux Ubuntu 16.04): Linux Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if
the issue happens on mobile device:  Desktop
TensorFlow installed from (source or
binary): - TensorFlow version (use command below): 1.15
Python version: - Bazel
version (if compiling from source): 3.6.9
GCC/Compiler version (if compiling from
source):
CUDA/cuDNN version: 10.0/7.6.5
GPU model and memory: GTX970 4GB

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with: 1. TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)" 2. TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
Describe the current behavior
I feed custom trained model with a video stream. I would like to use output's bounding boxes to feed 6 different models trained in for example Keras. I use webcam tutorial from Object Detection API. Only way it works for now is to set session by Keras and load model from the beggining. It ruins the performance very much.
Describe the expected behavior
I would like to have more than 1 fps with passing the detected objects to CNN Keras model.
Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
Google Colab code
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.

@SMatusik
I am not able to access the colab code. Request  you to provide access to colab link to reproduce the issue. Thanks!

It's accessable right now, sorry for mistake

@SMatusik
I tried reproducing the issue and i am seeing the below error message (ImportError: cannot import name 'string_int_label_map_pb2').Request you to help me with the reproducible code which helps me in localizing the issue. Thanks!

@SMatusik
Any update on this issue please. Thanks!


LinkOutsideDestinationError when installing PrivateGPT

Prerequisites
Please answer the following questions for yourself before submitting an issue.

[X] I am running the latest code. Development is very rapid so there are no tagged versions as of now.
[X] I carefully followed the README.md.
[X] I searched using keywords relevant to my issue to make sure that I am creating a new issue that is not already open (or closed).
[X] I reviewed the Discussions, and have a new bug or useful enhancement to share.

Expected Behavior
Please provide a detailed written description of what you were trying to do, and what you expected llama-cpp-python to do.
Expecting it to install properly when poetry is used to install it.
Current Behavior
Please provide a detailed written description of what llama-cpp-python did, instead.
Package operations: 1 install, 0 updates, 0 removals
Fails to install properly with the below command:
poetry install --with local
• Installing llama-cpp-python (0.2.17): Failed
LinkOutsideDestinationError
'llama_cpp_python-0.2.17/vendor/llama.cpp/spm-headers/ggml.h' would link to '/tmp/ggml.h', which is outside the destination
at /usr/lib/python3.11/tarfile.py:806 in _get_filtered_attrs
802│             if os.path.isabs(member.linkname):
803│                 raise AbsoluteLinkError(member)
804│             target_path = os.path.realpath(os.path.join(dest_path, member.linkname))
805│             if os.path.commonpath([target_path, dest_path]) != dest_path:
→  806│                 raise LinkOutsideDestinationError(member, target_path)
807│     return new_attrs
808│
809│ def fully_trusted_filter(member, dest_path):
810│     return member
Cannot install llama-cpp-python.
Environment and Context
Please provide detailed information about your computer setup. This is important in case the issue is not reproducible except for under certain specific conditions.
cat /etc/issue
Ubuntu 23.04 \n \l
uname -a
Linux hostname 6.2.0-36-generic #37-Ubuntu SMP PREEMPT_DYNAMIC Wed Oct  4 10:14:28 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
poetry --version
Poetry (version 1.7.0)
python3 -V
Python 3.11.4
lscpu
Architecture:            x86_64
CPU op-mode(s):        32-bit, 64-bit
Address sizes:         40 bits physical, 48 bits virtual
Byte Order:            Little Endian
CPU(s):                  8
On-line CPU(s) list:   0-7
Vendor ID:               GenuineIntel
BIOS Vendor ID:        QEMU
Model name:            Common KVM processor
BIOS Model name:     pc-i440fx-7.2  CPU @ 2.0GHz
BIOS CPU family:     1
CPU family:          15
Model:               6
Thread(s) per core:  1
Core(s) per socket:  4
Socket(s):           2
Stepping:            1
BogoMIPS:            4788.46
Flags:               fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx lm constant_tsc nopl xtopol
ogy cpuid tsc_known_freq pni cx16 x2apic hypervisor lahf_lm cpuid_fault pti
Virtualization features:
Hypervisor vendor:     KVM
Virtualization type:   full
Caches (sum of all):
L1d:                   256 KiB (8 instances)
L1i:                   256 KiB (8 instances)
L2:                    32 MiB (8 instances)
L3:                    32 MiB (2 instances)
NUMA:
NUMA node(s):          1
NUMA node0 CPU(s):     0-7

Physical (or virtual) hardware you are using, e.g. for Linux:

Proxmox 7.4-16 KVM virtual machine
$ lscpu

Operating System, e.g. for Linux:

make --version
GNU Make 4.3
Built for x86_64-pc-linux-gnu
Copyright (C) 1988-2020 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later http://gnu.org/licenses/gpl.html
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
g++ --version
g++ (Ubuntu 12.3.0-1ubuntu1~23.04) 12.3.0
Copyright (C) 2022 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
Failure Information (for bugs)
Poetry asked 4 days ago for this to be reported to you for fixing:
https://github.com/python-poetry/poetry/issues/8645
Please help provide information about the failure if this is a bug. If it is not a bug, please remove the rest of this template.
Steps to Reproduce
Please provide detailed steps for reproducing the issue. We are not sitting in front of your screen, so the more detail the better.

Download privategpt
Install poetry
Attempt to install privategpt with poetry

Failure Logs
Installing dependencies from lock file
Package operations: 1 install, 0 updates, 0 removals
• Installing llama-cpp-python (0.2.17): Failed
LinkOutsideDestinationError
'llama_cpp_python-0.2.17/vendor/llama.cpp/spm-headers/ggml.h' would link to '/tmp/ggml.h', which is outside the destination
at /usr/lib/python3.11/tarfile.py:806 in _get_filtered_attrs
802│             if os.path.isabs(member.linkname):
803│                 raise AbsoluteLinkError(member)
804│             target_path = os.path.realpath(os.path.join(dest_path, member.linkname))
805│             if os.path.commonpath([target_path, dest_path]) != dest_path:
→  806│                 raise LinkOutsideDestinationError(member, target_path)
807│     return new_attrs
808│
809│ def fully_trusted_filter(member, dest_path):
810│     return member
Cannot install llama-cpp-python.

Having this same issue

Also seeing this

I'm actually thinking this might be a Poetry bug: https://github.com/python-poetry/poetry/issues/8645
Notably, downgrading llama-cpp-python does not fix build for me.

Bumping my Python patch release from 3.11.4 -> 3.11.5 fixed the issue for me.
It looks like Poetry may have fixed this via a (as yet unreleased) backport: https://github.com/python-poetry/poetry/pull/8649/files
So, best options seem like

Upgrade Python
Upgrade to new Poetry patch release when it comes out

Had a same issue with Poetry 1.7, fallback to 1.6 worked as as a WA for me.

I'm seeing this error using Python 3.11.7, Poetry 1.8.2, and llama-cpp-python 0.2.62.