usability: Add aggregated ClusterRoles for admin, view, and edit.

Changes proposed by this PR

closes #450

Adds ClusterRoles for end-users which aggregate into the Kubernetes default user-facing roles.
Release Note

Users with the Kubernetes default admin, edit, and view roles can now access Cartographer resources.

PR Checklist
Note: Please do not remove items. Mark items as done [x] or use ~strikethrough~ if you believe they are not relevant

[x] Linked to a relevant issue. Eg: Fixes #123 or Updates #123
[x] Removed non-atomic or wip commits
[x] Filled in the Release Note section above
[x] Modified the docs to match changes

@evankanderson If you could rebase on main the validation tests should start passing (fixed here: #447)

Merged; I can rebase / squash if you want.

While not necessary the focus of this PR, this raises an interesting question as to how developers are expected to get access to the cluster scoped resources (ClusterSupplyChain in particular) to know what workload types are supported in the cluster, or what params are available on a particular supply chain.

@scothis I believe that is what RFC 12 addresses.

@martyspiewak regardless of how the contract in expressed on the resource, if the user doesn't have access to view that resource, the contract isn't going to be helpful.

@scothis Makes sense. So do we expect most developers to have this view role? If we do, I think I'd change my mind and say we should leave the cluster scoped resources on that role.

So do we expect most developers to have this view role?

Not sure, but probably.

If we do, I think I'd change my mind and say we should leave the cluster scoped resources on that role.

The view ClusterRole is typically bound with a namespaced RoleBinding. Based on Evan's findings above the cluster scoped resources won't be visible. If a user was granted access to the view role for the cluster, then they would be able to see all Workloads (among many other resources) in all namespaces.
I'd suggested not including any cluster scoped resources here and figuring out a different way to grant access to them.

Sorry for dropping this for so long over the holidays!
So @scothis just to recap, from

I'd suggested not depending on the aggregated roles for any cluster scoped resources and figuring out a different way to grant access to them for regular users.

it sounds like we should drop the cluster scoped resources from the view role as well. Am I understanding that correctly?

@evankanderson per the discussion above, could you drop the cluster scoped resources from the view role? I think it should be good to merge once that's done.

Just committed that small change myself instead so we can merge this. Thanks for the PR @evankanderson and for helping with the review @scothis!


Citation for research paper

I would like to cite psd-tools in a research paper. Is there a .bib file? something like:
@Misc{psd-tools
author =   {Kota Yamaguchi and Mikhail Korobov and others},
title =     {{psd-tools}: Open source psd tools for {Python}},
year =      {2019--},
url = "https://psd-tools.readthedocs.io"
}

You can use whatever format to cite. Perhaps authors should be 'psd-tools developers' or entirely omitted, because there are many contributors

Thank you for the quick response.
Will do


Also recognize github's new  (#1234) syntax

Argh, needs prettier --fix '**/*.js'

@Munter this seems great, can you fix the code formatting :-)

I think I wrote this on Peter's computer :see_no_evil:
I'll fix it up, one sec.

Released in 1.7.0


Use thumbnail as full image.

Closes #147.

Coverage increased (+0.04%) to 97.921% when pulling 10bdaa132e7f819a844d6723aad86e0f4a11ac29 on thumbnail_as_full_image into 8ae930f6312aba917849651e1dff8d460dd3124c on master.


Code Bug for llava/model/llava_arch.py

https://github.com/LLaVA-VL/LLaVA-NeXT/blob/b3a46be22d5aa818fa1a23542ae3a28f8e2ed421/llava/model/llava_arch.py#L230
Not every model config has the attribute "add_faster_video" (e.g. https://huggingface.co/lmms-lab/llava-onevision-qwen2-0.5b-ov/blob/main/config.json). People have to add an overwrite config parameter to their codes manually. I do not see this as intended (?)
Maybe should add hasattr() for further judgment.

https://github.com/LLaVA-VL/LLaVA-NeXT/blob/b3a46be22d5aa818fa1a23542ae3a28f8e2ed421/llava/model/llava_arch.py#L230
Not every model config has the attribute "add_faster_video" (e.g. https://huggingface.co/lmms-lab/llava-onevision-qwen2-0.5b-ov/blob/main/config.json). People have to add an overwrite config parameter to their codes manually. I do not see this as intended (?)
Maybe should add hasattr() for further judgment.

Agreed. But even in the Video-7B-Qwen2, the add_faster_video is set to False, which makes me wonder whether the slow-fast implementation is already supported?


CM scripts for Micro23AE, #38 G10

Thank you! Checking!

It seems to be fine! I just did a few minor changes (change python3 to our CM environment to be able to work with virtual environments or different python versions prepared by CM) ... Thank you very much for participating in our pilot project to test CM interface for AE ...


Avoid multiple state mutations when calculating amount and total

This should fix side effects that can happen when calling setState multiple times calculating the total and amount when making an order. Given that setState is async, it can result in the wrong values being used.
Fixes #268.

Perfect. 👌


Update to ArchUnit 1.0.0-rc1

For JDK 19 compatibility.
Fixes #22189

@bot-gradle test and merge

OK, I've already triggered a build for you.


twitter-cldr automatically sentence-casing elements (TimespanFormatter, patterns)

The values for TimespanFormatter:patterns are all sentence cases, with the letter of the first word capitalized, e.g.

https://github.com/twitter/twitter-cldr-js/blob/master/lib/assets/javascripts/twitter_cldr/es-CO.js#L36
https://www.evernote.com/l/AKvaqZp2bMdIiKl8oef6lRDy7j27aqyW1jAB/image.png
"Hace {0} …"

The values in the CLDR code charts are all lower cased, e.g.:

http://www.unicode.org/cldr/charts/29/summary/es.html#1426
https://www.evernote.com/l/AKvOfiSJuOdGPrpwTuB2xeXnP_bjZ2l7rwgB/image.png
"hace {0} …"

This causes problems when the value to be inserted is not in sentence-initial position, viz. in the middle of the sentence, e.g.:

https://www.evernote.com/l/AKvMY_Qp-RhM8rTEeJ0bsaLUGsiEWTWyp9kB/image.png
INCORRECT : "Desde el lanzamiento … Hace 3 días, …" (Since we started … 3 days ago, …)
correct: "Desde el lanzamiento … hace 3 días, …" (Since we started … 3 days ago, …)

Please advise.

Hey @pchew-change, unfortunately the fields/units data built into this gem are from CLDR v23, not v29 as you linked. In v23, time spans were uppercased. The ruby version of twitter_cldr also suffers from this issue. As I mentioned in #80, I'm working on supporting the new units data introduced in CLDR v24, which should let us move to v29 without too many problems. Then I should be able to focus on improving the Javascript story.


Upgrade to Preview 10

Work in progress... Waiting for quickstart image to be fixed

[ ] Upgrade quickstart image
[ ] Upgrade sdk
[ ] Build and create issues for every error


Platform exception

Hello,
can i get help??? i have tried to test Quasar for 2 weeks now after downloading and building it using
visual studio 2017, but i keep getting an error and been trying to debug

thanks

Please build in debug mode and then debug Quasar, the exception should appear there too. You can the just copy it here.

Hello,
thank you very much for your reply.
should i build from screenshot selecting Mixed platform and Quasar.client or  All CPU and Quasar.server
thanks

Keep it at "Mixed Platforms", just change "Release" to "Debug" and then do a right click on "Quasar.Server" on the right list and click "Debug".

Hello,
am using a windows 10 i have also disabled my antivirus. please see error below after debugging
System.TypeInitializationException
HResult=0x80131534
Message=The type initializer for 'Quasar.Common.Helpers.PlatformHelper' threw an exception.
Inner Exception 1:
COMException: The specified service does not exist as an installed service. (Exception from HRESULT: 0x80070424)

thanks

Okay good, on the exception you showed me in the previous screenshot, can you click on Copy Details and paste it here?

Hello,
i could not copy the error details but i found out the reason why i had that error and it was because of my anti-virus, my antivirus has auto-containment. but after disabling all features of the anti-virus. i got these errors
Severity	Code	Description	Project	File	Line	Suppression State
Error		The command ""C:\Users\agent47\QuasarRAT\packages\ILMerge.2.14.1208\tools\ILMerge.exe" /out:"C:\Users\agent47\QuasarRAT\bin\Debug\Client.all.exe" "C:\Users\agent47\QuasarRAT\bin\Debug\Client.exe" "C:\Users\agent47\QuasarRAT\bin\Debug\protobuf-net.dll" "C:\Users\agent47\QuasarRAT\bin\Debug\Gma.System.MouseKeyHook.dll" "C:\Users\agent47\QuasarRAT\bin\Debug\Quasar.Common.dll" /target:WinExe /wildcards /internalize
copy "C:\Users\agent47\QuasarRAT\bin\Debug\Client.all.exe" "C:\Users\agent47\QuasarRAT\bin\Debug\client.bin" /Y" exited with code 1.	Quasar.Client
Severity	Code	Description	Project	File	Line	Suppression State
Warning	CS0169	The field 'Client._typeIndex' is never used	Quasar.Client	C:\Users\agent47\QuasarRAT\Quasar.Client\Networking\Client.cs	197	Active
Severity	Code	Description	Project	File	Line	Suppression State
Warning		The referenced component 'protobuf-net' could not be found.	Quasar.Client
Severity	Code	Description	Project	File	Line	Suppression State
Warning		The referenced component 'Gma.System.MouseKeyHook' could not be found.	Quasar.Client

Oh, okay... you need to restore the NuGet packages first..

please whats the NuGet packages because it was just a new cloned project from git

Check out this on how to restore NuGet packages in Visual Studio: https://docs.microsoft.com/en-us/nuget/consume-packages/package-restore

ok thanks will check it out

hello,
did several tries on package restore and update and package cache clearing but still same result. then i noticed that when i switch off my comodo anti virus thats when that error occurs while building but when my antivirus is one is shows success, then when i opened Quaser.exe in the release folder and tried to build an .exe/build client file and using my IP as localhost. there was nowhere on my PC it stored the build
1>------ Rebuild All started: Project: Quasar.Common, Configuration: Release Any CPU ------
1>  Quasar.Common -> C:\Users\agent47\Desktop\QuasarRAT-master\QuasarRAT-master\bin\Release\Quasar.Common.dll
2>------ Rebuild All started: Project: Quasar.Server, Configuration: Release Any CPU ------
3>------ Rebuild All started: Project: Quasar.Client, Configuration: Release x86 ------
4>------ Rebuild All started: Project: Quasar.Common.Tests, Configuration: Release Any CPU ------
2>  Quasar.Server -> C:\Users\agent47\Desktop\QuasarRAT-master\QuasarRAT-master\bin\Release\Quasar.exe
5>------ Rebuild All started: Project: Quasar.Server.Tests, Configuration: Release Any CPU ------
5>  Quasar.Server.Tests -> C:\Users\agent47\Desktop\QuasarRAT-master\QuasarRAT-master\bin\Release\Quasar.Server.Tests.dll
4>  Quasar.Common.Tests -> C:\Users\agent47\Desktop\QuasarRAT-master\QuasarRAT-master\bin\Release\Quasar.Common.Tests.dll
3>  Quasar.Client -> C:\Users\agent47\Desktop\QuasarRAT-master\QuasarRAT-master\bin\Release\Client.exe
3>          1 file(s) copied.
6>------ Rebuild All started: Project: Quasar.Client.Tests, Configuration: Release x86 ------
6>  Quasar.Client.Tests -> C:\Users\agent47\Desktop\QuasarRAT-master\QuasarRAT-master\bin\Release\Quasar.Client.Tests.dll
========== Rebuild All: 6 succeeded, 0 failed, 0 skipped ==========

how fix Severity	Code	Description	Project	File	Line	Suppression State
Warning	CS0169	The field 'Client._typeIndex' is never used	Quasar.Client	C:\Users\darkh\Desktop\Quasar-1.4.1\Quasar.Client\Networking\Client.cs	188	Active

Clint Builder is not available in DEBUG configuration
Please build the project using RELEASE  configuration

Clint Builder is not available i

n DEBUG configuration
Please build the project using RELEASE  configuration


Small enhancement to the animated doc

Basically, in the Animated doc is not super clear that you have a callback option for start(); the only section mentioning it is here as far as I can tell.
I think it would be super helpful/clarifying to have a clearer code sample. The closest thing to it is in a reply on a GH issue of the main repo:
this.animateValue.spring({}).start(function onComplete() {})
(src)
Which I think is also outdated, because according to the link above in the docs should be something like
this.animateValue.spring({}).start((finished) => {})
(at least, I think - ideally it would be better to go and check source code)

BTW I only found out about this via a StackOverflow issue so yeah it seems that it's quite relevant.
I'll assign to myself and try to tackle asap, just can't now but don't want to lose track of this.
(also, it's probably a really good first issue for someone trying to do their first contribution so I'll add that tag too)

hi @kelset , are you still planing to tackle this? if not, I would love to help.

Hey @gedeagas - no I haven't had time yet, feel free to do a PR for it if you want :)

It would be great if start was listed under Animated's methods, too! Please tag this with #1579 to get credit for completing the API docs, @gedeagas!

@kelset Awesome, i will be taking this then.
@rachelnabors thank you for the information, will do.

The issue reported here has been fixed by the merge of PR created by @gedeagas.


Tabs current line in wrong position when using scroll

Hi guys, I'm using the Tabs with a scroll for my mobile layout.
They are part of the Google Material Design:
https://www.google.com/design/spec/components/tabs.html#tabs-types-of-tabs
The second type is the "Scrollable tabs".
The problem is that the position of the line doesn't take care of the left scroll.
You can check it in the image attached.

You can easily achieve the same result going on the React Toolbox website:
http://react-toolbox.com/#/components/tabs
and adding the "overlfow-x: scroll" rule to the element with the attribute data-react-toolbox="tabs"

Thanks for reporting!

I encountered the same problem with Tabs when I used a fixed width to contain the navigation part, it looks like the active line will be added before I moved to the full display of active tab.
Image attached:

Could we have some options to let the tab horizontal scrollable and customization? Currently, I just add the overflow-x: auto to enable the default scrollbar of browser, can we use other React component for this scrollbar like [react-custom-scrollbars](https://github.com/malte-wessel/react-custom-scrollbars)

Can we have some update of this bug?

Can you try the updates made with https://github.com/react-toolbox/react-toolbox/pull/706? I'll try to give a look soon. That project looks very nice!

I am attempted to push tab pagination through here: https://github.com/react-toolbox/react-toolbox/pull/975

This can be closed I guess :)


Add LICENSE

Thanks for developing this great plugin!
Can you add LICENSE file to clarify the OSS license? (e.g. MIT License, Public Domain)

@rot1024
Thank you very much for asking. We chose the MIT License. 😉
Please find the LICENSE file at:
https://github.com/archival-archetyping/about-widget-reearth-plugin/blob/main/LICENSE

Thank you!


new package: Mendeley

This request is to add the Mendeley Desktop reference manager for 32 and 64 bit Linux.

Built locally, thanks.

$ nix-env -f . -qa '*' --meta --xml --drv-path --show-trace
error: while querying the derivation named ‘mendeley-1.13.1-stable_amd64’:
while evaluating the attribute ‘installPhase’ of the derivation ‘mendeley-1.13.1-stable_amd64’ at "/home/bfo/nixpkgs/pkgs/applications/office/mendeley/default.nix":34:3:
while evaluating ‘makeSearchPath’ at "/home/bfo/nixpkgs/lib/strings.nix":44:28, called from "/home/bfo/nixpkgs/pkgs/applications/office/mendeley/default.nix":52:20:
while evaluating ‘concatStringsSep’ at "/home/bfo/nixpkgs/lib/strings.nix":34:33, called from "/home/bfo/nixpkgs/lib/strings.nix":45:5:
while evaluating ‘fold’ at "/home/bfo/nixpkgs/lib/lists.nix":20:19, called from "/home/bfo/nixpkgs/lib/strings.nix":35:5:
while evaluating ‘fold'’ at "/home/bfo/nixpkgs/lib/lists.nix":23:15, called from "/home/bfo/nixpkgs/lib/lists.nix":27:8:
while evaluating anonymous function at "/home/bfo/nixpkgs/lib/strings.nix":15:32, called from "/home/bfo/nixpkgs/lib/lists.nix":26:14:
while evaluating anonymous function at "/home/bfo/nixpkgs/lib/strings.nix":45:32, called from undefined position:
attribute ‘gcc’ missing, at "/home/bfo/nixpkgs/pkgs/applications/office/mendeley/default.nix":25:5

gcc -> cc?

Fixed in master.


Hover underline wrapping to new column in AnchorLinks

In Chrome (did not test other browsers), hovering last anchor link has underline in next column.

If you add transform: translate3d(0,0,0) to .bx--anchor-links .page-ul > li a declaration block seems to fix it.

NOT STALE, see https://github.com/carbon-design-system/carbon-website/issues/1702 for a great gif detailed description.


Cleanup crossref code

Motivation
Code hygiene makes for healthy engineering. By DRY. Delete dead code.
Definition of Done

All CrossRef code is in encapsulated in a single service

Tasks

[ ] Refactor this code, moving it here
[ ] Delete this dead code implementation of CrossrefProvider and its tests.

We'll soon be making crossref calls from the frontend, too. If this is a service with an API, that'd be fantastic

Currently, it's just a library-level API (in the "business layer") that is used privately by the backend code, so it's not exposed via an HTTP API. But do add a new issue for that if needed! (maybe as part of upcoming epic that requires it)

CC: @metakuni ^^^

Handling the second task in #6414

Moved back to Disorder after completing subtask named above. Refactor for Cellguide (first item) still remains to be completed.

I'm adding to a larger crossref epic so this doesn't get lost.


kubernetes field names are de-dotted - dot replaced with underscore

The diff is pretty hard to scan - basically, I replaced . with _ in the kubernetes_labels_* field names.

@ewolinetz ptal

:+1:  LGTM

[merge]


Updated the code to the new swagger_spec_validator version - FIX BUILD

Changes proposed in this pull request:

removed the validate_defaults method from Operation because the new swagger_spec_validation version do this validation
fixed swagger_spec_validator array items validation
fixed swagger_spec_validator defaults validation

@hjacobs
the new pypi version of swagger-spec-validation is breaking the build

Closing as we merged #611 (which is the same), sorry for overlooking your PR.


New MAKES documentation (KESM, schema, synonym, data)

Docs Build status updates of commit 53e4ba2:
:warning: Validation status: warnings

File
Status
Preview URL
Details

academic-services/knowledge-exploration-service/how-to-index-build.md
:warning:Warning

Details

academic-services/knowledge-exploration-service/TOC.yml
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/how-to-index-data.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/how-to-index-schema.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/how-to-index-synonym.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/how-to-integrate-with-existing-search.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/reference-makes-command-line-tool.md
:white_check_mark:Succeeded

academic-services/project-academic-knowledge/reference-extended-removal.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/how-to-index-build.md

Line 0, Column 0: [Warning-h1-missing] H1 is required. Use a single hash (#) followed by a space to create your top-level heading.

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


Сonway primitives fix

CDDL changes:

NewCommittee was renamed to UpdateCommittee
Committee was deleted

Update plutus v3 related structs
I also added a test with manually constructed block with a tx which contains all conway certs, voting proposals and voting procedures

Where are you finding the new RedeemerTag:
    #[n(5)]
    VotingProposal,

@Ryun1 could you confirm that will be a new redeemer tag for VotingProposal ? If now I'll delete it from CSL too

Response from Ledger team (@lehins) on this:
Yes, it is still not implemented, since PlutusV3 integration is not done yet.
Expect there to be two extra tags 4 and 5.
We are currently working on this, but it will take time.

Thanks @Ryun1 ! @jmhrpr could you update your PR on txpipe repo ?


Fails with 'arch: i386'

Thanks for making this action.
It seems like ports.ubuntu.com doesn't supply i386 binaries, which causes arch: i386 to fail with "Failed to fetch" errors, but I imagine should be in the normal (non-ports) repositories.
Do you think it'd make sense to skip adding the ports repositories if arch=i386?

Amd64 and i386 packages are in archives.ubuntu.com
Arm is on ports.ubuntu.com

hey, yeah seems reasonable to me! archives.ubuntu.com already gets constrained to amd64/i386 so just skipping the ports.ubuntu.com step for both of those cases should do the job


How can I play txt files in version 4.1?

I've tried everything in the docs, but it doesn't work. I would be very grateful if you could give me an example.

i figure out


No hover type for for char in string

When iterating over a slice or a dynamic array using for in syntax, the item we get has a correct type on hover. But then iterating a string nothing shows up.
some_str: string
for char in some_str {
	char // should have type `rune` on hover
}

Fixed in rename.


Creating Pop up window

In this milestone,  We have implemented the following features:

When the user clicks (or taps) the button to check project details, the popup with details about the project appears, both for mobile and desktop versions.
When the user clicks (or taps) the close (X) button, the popup disappears.
use a JavaScript array to store all of the information for all projects.

I just deleted the comments from my VS code because it makes the comments window pop Ups each time I visit the files.

Ohh I see that the comments are deleted too from GitHub @BenMukebo, What should we do???

Hello @Christianib003,
Both you and your partner are to implement this popup windows feature in both of your repositories and submit the repositories individually. This is your partner's repo which has been approved, so now you guys have to implement the feature in your own repo and submit it for approval 👍

We created a pop up window which would be displayed by JavaScript dynamically.


chore: release main

:robot: I have created a release beep boop
stdlib: 1.1.2
1.1.2 (2023-12-24)
Bug Fixes

components: refs, tests (4190f1d)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/pavophilip/react-shader-fiber/releases/tag/stdlib-v1.1.2 :sunflower:


Broken code examples in date-components storybook

⚠ This issue moved from uikit repo authored by @ldrv565 (Original issue: https://github.com/gravity-ui/uikit/issues/903)
https://preview.gravity-ui.com/date-components/?path=/docs/components-calendar--docs

Fixed


[BUG] The value of Date type column is not correct

Describe the bug
The value of Date type column is not correct

What did you do

make the tidb and tispark have same time zone

execute in tidb
create table t(id varchar(11),b date)
insert into values(null,'2020-10-10')

execute in tispark
select * from t

What do you expect
null, '2020-10-10'

What happens instead
null, '2020-10-09'

Spark and TiSpark version info
Git Commit Hash: 9687b70202584883fe5b88d39fb62d64867ebde2
Git Branch: release-2.3
UTC Build Time: 2020-10-16 02:40:15
Supported Spark Version: 2.3 2.4
Current Spark Version: 2.4.3
Current Spark Major Version: 2.4
TimeZone: Asia/Shanghai
TIDB: release-4.0

Related to #1703

Closed by #1703


reject should support refinements

Like ordinary filtering does.

Can't - TS doesn't currently support negative type guards like : x is not A.


Synexpand may be too sensitive

The current ^*# macro will recursively walk the tree looking for anything that looks like the mini-language syntax. Currently, that's anything in the function position that would compile to a string and contain at least one of the magic characters /&@<>*:^, unless it's the starting character (to avoid detecting normal control words as syntax). However, symbols do sometimes contain these characters, so it may try expanding things you wouldn't want it to. let*from, any*map, set@, and && come to mind. (:* and :** also qualify, but wouldn't normally be found in the function position.) The * seems especially problematic. These characters also have to be escaped when used in the syntax (e.g. let*from`).
This was less of a concern before I added the recursion, as one had to explicitly mark the desired usage. It is possible to escape things you don't want affected (if you notice) by using syntax that would expand to the thing you want e.g. (:: let*from) would just expand to let*from.
^*# could require some explicit mark, like a :: prefix, but that's already two characters of overhead and shortening ^*# to (say) *# could accomplish that much without the recursion, although there are some cases where a *#: prefix would still be necessary. I suppose *# is an option.
Another option might be to require two (or three) magic characters before it counts. I really don't want to overcomplicate the detection rules (which would make the macro hard to use), but this wouldn't be too bad. There may be legitimate usage of the syntax that only requires one (or two) magic character(s), but one could always add NOPs : anywhere to bring the count up. In most cases, it wouldn't be required, so there would be no overhead. Two would exclude everything but &&, which may be a bad name anyway. Three would exclude everything, but might add the NOP overhead a lot more often.
Another option would be to require that certain magic characters be present, but not count others. Because : (not at the start) and ^ seem especially common in the syntax, but rare in ordinary symbol usage, incorrect detection requiring an escape and overhead forcing detection would be pretty rare, even if we only need one of the characters to count.

I think I'll try requiring either a ^ or :, not at the start.


[WIP] fix RebootCodes option for cinst wrapper

This should "re-enable" the possibility to pass -RebootCodes to Chocolatey commands in Boxstarter packages.
I stumbled across the parameter in the tests (i.e. here) and wanted to use this feature in my generic Boxstarter bootstrapper files, however there are several small bugs/inconsistencies that basically render this unusable in the current state.
inside a Boxstarter package:
samplePackage.ps1
cinst far
cinst foobar -RebootCodes @(8, 3010, 42)
cinst barz

when installing samplePackage.ps1 via Boxstarter wrapper (BoxStarter.bat), Invoke-Reboot will be triggered immediately after a package (far, foobar or barz) returned a "valid reboot code".

hey @mwrock - I'm having trouble figuring out what you're doing with $global:Error.Count vs $currentErrorCount here ...
I see that the Choco exception details are resolved via $global:Error - why not just reset that before Boxstarter invokes Chocolatey and ditch that $currentErrorCount?

It's been a couple years since I have looked at this (and wow what a mess) but I believe there is some examination elsewhere of the global error table that expects it to remain intact.

@mwallner Is this ready for review? If so can you remove [WIP] from the title?

@mwallner Is this ready for review? If so can you remove [WIP] from the title?

not quite there yet.. I'm still trying to figure out how that $global:Error magic is supposed to work - I'm not sure if bb3776a98318b14c9c7737b556a60f7949a9a86d breaks something or not.

That's great. I've removed the other reviewers just now. Just add us back in when you're ready to go. If you need any help let me know.

@mwallner Is there an issue for this we can link to?

@mwallner Is there an issue for this we can link to?

nope, not yet - I'm still trying to figure out what effects my changes will introduce on the different types of invoking boxstarter. (from boxstarter-shell, from inside a boxstarter package, from boxstarter.bat bootstrapper...)
as soon as I'm confident I've tackled everything, I'll remove the [WIP] and re-request a review :smile:
@pauby do we need a separate issue for each PR?

do we need a separate issue for each PR?

Not a separate issue. But we do need an issue.

do we need a separate issue for each PR?

Not a separate issue. But we do need an issue.

Ref #389

@mwallner Before I go ahead and hit the big green button you're happy with this now?

yup, please do :-)


Implement deferred binding support in WebJobs Tables Extension

Contributing to the Azure SDK
Issue: https://github.com/Azure/azure-functions-dotnet-worker/issues/1473
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
API changes are not detected in this pull request.


Bump version even on minor updates.

Could you please not re-release using the same version number? :)
Even if the changes are minor, they could be relevant enough for the user to update.
Thanks!

Had to be done for testing an auto update feature for AppImages.

So, a one time thing?


Native Javascript Support

Goal:  Add ability to execute Javascript programs in Source Academy frontend

This is a bare-bones implementation, features such as SICP function integration and security concerns will be addressed in future PRs. Unit testing will also be added once SICP functions are fully integrated in the upcoming PR

Reference:

Sprint #1
Sprint #2

Implementation Details
Identifying nativeJS

In order to identify nativeJS, we reused the SourceLanguage type as implementing a separate interface to handle "non-source" language might cause too much unnecessary complexity as of now.

Setting the chapter === -1 will be a indicator for nativeJS

https://github.com/source-academy/frontend/blob/059a6d3e7953764fcb2bca442128adf4f6abba48/src/commons/application/ApplicationTypes.ts#L128-L132

nativeJS Evaluation
We will be using the eval() function without any additional parsing or instrumentation to evaluate nativeJS program strings

Without additional instrumentation, it will be hard for us to catch non-terminating programs (e.g. infinite loops), therefore we have added new UI features and recovery information to guide users.

When the program is still under execution, the "Run" button will be in the color green (more on this below)
Recovery information is added to the introduction tab

https://user-images.githubusercontent.com/47914880/148048824-cdf0641b-8da2-450c-9596-84a2abce3b6c.mov

EvalErrorLocator.ts In order to provide rich error messages, we have a ErrorLocator to help extract line & column numbers by using the stack trace of Error() objects (Error.stack)

https://user-images.githubusercontent.com/47914880/148048970-f4f29288-2b93-44f9-8d52-210498c64f6d.mov
Syntax Highlighting

Rules are extended from source syntax highlighting rules.

Main changes would include allowing javascript keywords and operators
Color scheme is inherited from the source mode

Unblocking the UI before running program
We included a generic wrapper to convert the current microtask of running a program (be it source or nativeJS) to a task by using setTimeout. This will allow the UI to have the opp. to render and display the indicator that a program is currently running.
https://github.com/source-academy/frontend/blob/d8d917138a9352c8e67d4f591ef2fad0119bd941/src/commons/utils/RunHelper.ts#L13-L18
// Example usage
const nativeJSResult: NativeJSEvalResult = yield call(runWrapper, evalNativeJSProgram, program);
const sourceResult = call(runWrapper, runInContext, code, context, {
                                            scheduler: 'preemptive',
                                            originalMaxExecTime: execTime,
                                            stepLimit: stepLimit,
                                            throwInfiniteLoops: throwInfiniteLoops,
                                            useSubst: substActiveAndCorrectChapter})

We only used this wrapper for nativeJS evaluation and have not integrated it with the execution of source programs

However, this still does not allow UI to be updated or users to interact with UI in the process of running a program.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

17 of 71 (23.94%)  changed or added relevant lines in 10 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.06%) to 30.872%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/commons/application/reducers/SessionsReducer.ts
0
1
0.0%

src/commons/controlBar/ControlBarChapterSelect.tsx
0
1
0.0%

src/commons/sagas/WorkspaceSaga.ts
3
4
75.0%

src/features/nativeJS/NativeJSActions.ts
2
3
66.67%

src/commons/nativeJS/nativeJSWorker.js
2
7
28.57%

src/commons/nativeJS/nativeJSEval.ts
0
21
0.0%

src/commons/sagas/NativeJsSaga.ts
3
27
11.11%

Totals

Change from base Build<PHONE_NUMBER>:
-0.06%

Covered Lines:
3484

Relevant Lines:
10522

💛  - Coveralls

Is making commits to this draft the recommended approach for me to make (possibly) any contributions? Or shall I open up a separate pull request 😂

Is making commits to this draft the recommended approach for me to make (possibly) any contributions? Or shall I open up a separate pull request 😂

I think pushing to the current branch would be better? If not we will have 2 pull requests implementing the same feature 😅

Yes, that's right. You both should push to the same branch.

Got it :)

Goal:  Add ability to execute Javascript programs in Source Academy frontend
Refrence: Sprint #2
TODOs

[ ] Unit test

Implementation Details

[Frontend] Improve error message: achieved by parsing Error.stack.

adding a error locator
https://github.com/source-academy/frontend/blob/d80af35d2e01664ad5d49f74a0b42837a7c4d4b0/src/commons/sagas/NativeJsSaga.ts#L44-L86
pass the full error object
https://github.com/source-academy/frontend/blob/d80af35d2e01664ad5d49f74a0b42837a7c4d4b0/src/commons/nativeJS/NativeJSEval.ts#L8

[UX] Recovery info display
https://github.com/source-academy/frontend/blob/d80af35d2e01664ad5d49f74a0b42837a7c4d4b0/src/commons/controlBar/ControlBarAutorunButtons.tsx#L60-L67
https://github.com/source-academy/frontend/blob/d80af35d2e01664ad5d49f74a0b42837a7c4d4b0/src/commons/controlBar/ControlBarAutorunButtons.tsx#L77-L80
[UI] Rename and relocate menu items
[UI] Change the phrasing of the text on the right to cater to full JavaScript mode
[UI] Syntax highlighting: achieved by first copying and then modifying source.js from js-slang.

Demo

this is just a basic demo, this is not intended to show that it works (more testing is definitely needed)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

19 of 40 (47.5%)  changed or added relevant lines in 13 files are covered.
140 unchanged lines in 7 files lost coverage.
Overall coverage decreased (-0.002%) to 30.933%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/commons/controlBar/ControlBarAutorunButtons.tsx
0
1
0.0%

src/commons/controlBar/ControlBarChapterSelect.tsx
0
1
0.0%

src/commons/utils/AceHelper.ts
1
2
50.0%

src/commons/sagas/WorkspaceSaga.ts
4
6
66.67%

src/commons/nativeJS/NativeJSEval.ts
0
3
0.0%

src/pages/playground/Playground.tsx
0
3
0.0%

src/commons/sagas/NativeJsSaga.ts
2
12
16.67%

Files with Coverage Reduction
New Missed Lines
%

src/commons/controlBar/ControlBarAutorunButtons.tsx
1
0.0%

src/commons/controlBar/ControlBarRunButton.tsx
1
0.0%

src/commons/utils/IntroductionHelper.ts
1
91.67%

src/commons/controlBar/ControlBarChapterSelect.tsx
3
0.0%

src/commons/utils/AceHelper.ts
4
22.22%

src/commons/sagas/WorkspaceSaga.ts
56
73.87%

src/pages/playground/Playground.tsx
74
1.64%

Totals

Change from base Build<PHONE_NUMBER>:
-0.002%

Covered Lines:
3480

Relevant Lines:
10485

💛  - Coveralls


Create local authority model and importer

Changes
This creates a Local Authority model and importer which imports the local authority data into the database.
I have included the local_authorities.csv and saved it within a fixtures folder, to allow easy future access to the local authorities data.
Checklist

[x] Attach this pull request to the appropriate card in Trello.
[x] Update the CHANGELOG.md if needed.

@beeLorna it's fine in this case as local authority addresses do not contain personally identifiable information, but let's be super careful when commiting data like in this work! 👍


Ability to pass a form object argument into a content block and use it?

Hello,
I'm not sure if I'm missing something basic but I was trying to build a really simple component for searching. I'm unable to build forms using the default builder with Rails.
Here's a simplified example:
<%= form_with(url: @url, method: :get, id: "search", data: { controller: "form", action: "input->form#debouncedSubmit", turbo_frame: @turbo_frame, turbo_action: "advance" }) do |form| %>

    <%= search_field_tag :query, @query, placeholder: @placeholder, autofocus: true %>

      <%= content %>

<% end %>

<%= render SearchComponent.new(url: admin_users_path, turbo_frame: "admin_users", placeholder: "Search users", query: @filter_query) do |form| %>
  <%= form.label :deactivated, class: "inline-flex items-center ml-6" do %>
    <%= check_box_tag :deactivated, "yes", @filter_deactivated, data: { action: "change->search#submit" } %>
    Deactivated
  <% end %>
<% end %>

When it comes to displaying the label, it produces this HTML output:

    <span class="ml-2 text-slate-600">Deactivated</span>

I also tried modifying my component with <%= form_with(url: @url, builder: ActionView::Helpers::FormBuilder, ... but it also produced the same output as above.
I saw the known issues around compatibility with Rails form helpers but I'm not sure how to proceed. I didn't want to invest in a custom builder gem since I wanted to use the out of the box Rails form helpers. Is that not an option?
I tried enabling the compatibility patch, but it produces this HTML, it almost worked but not quite:

  <input type="checkbox" name="deactivated" id="deactivated" value="yes" data-action="change->search#submit" />
  Deactivated

I have 6 or 7 search forms that all share this same structure. I didn't want to make a duplicate view component for each one. In this example I could technically use  directly to avoid the Rails helper but in practice other search forms will use select boxes and an assortment of other form helpers.
I thought I might be missing something basic because I'm not sure how to solve this use case. What am I missing? Thanks!

Hey @nickjj, thanks for asking about this. As the issue stands now, I'm not sure I have enough information to diagnose the problem. The first thing that jumps out at me is that you're calling #label on an instance of SearchComponent. Unless I'm mistaken, I think you meant to call #label on an instance of FormBuilder instead, i.e. the object yielded by #form_with. Moreover, I don't see you passing the builder into SearchComponent.new, so even if FormBuilder#label were being proxied to the builder via SearchComponent#label, it has no reference to the builder.
Since the form builder is created inside the component, there's no way to pass it to the parent unless you were to set an instance variable in the form_with { ... } block and expose it in SearchComponent. Here's how that might look with some of the extra stuff stripped out (although to be honest, I don't advocate for this approach):
# app/components/search_component.rb

class SearchComponent < ViewComponent::Base
  attr_reader :form_builder

  def initialize(**kwargs)
    @query = "query"
    @placeholder = "placeholder"
  end
end

<%# app/components/search_component.html.erb %>
<%= form_with(url: @url, method: :get, id: "search", data: { action: "input->form#debouncedSubmit"}) do |form| %>
  <% @form_builder = form %>
  <%# more stuff here %>
<% end %>

<%# app/views/some_controller/index.html.erb %>

<%= render SearchComponent.new(url: "/foo") do |search| %>
  <%= search.form_builder.label :deactivated do %>
    <%= check_box_tag :deactivated, "yes", false, data: { action: "change->search#submit" } %>
    Deactivated
  <% end %>
<% end %>

I feel like I have to be missing some context/code here, please advise!

Hi,
Thanks for the reply. The context is you might have a Rails form in a partial but then have 1 or more ViewComponents that you'd like to use within your Rails form block. Ideally you'd like to pass the Rails form object into the content of the ViewComponent (such as wanting to reference form.label in your ViewComponent).
This lets you mostly use partials and sprinkle in a couple of components as needed, all while sticking with the default Rails form builder and no extra dependencies.

Ok, that makes sense 👍  Can you post a more complete code example then? I'm not able to follow what you're trying to do given the code you've already posted.

Thanks, sorry about that. I ended up not making a view component for this specific snippet in the end. If another use case pops up where something similar happens I'll provide more details than what was currently provided.


extract_genres_from_file_directories returning 0x80131937 aka<PHONE_NUMBER>

Probably return a 1. But more task scheduler error messages.

Message from _log transcript:
SELECT
directory_path                      / What we are going to search for new files     /
,  useful_part
FROM
directories_ext_v
Message: Exception calling "ExecuteReader" with "0" argument(s): "ERROR [42703] ERROR: column "useful_part" does not exist;
Error while executing the query"
StackTrace:    at System.Management.Automation.ExceptionHandlingOps.CheckActionPreference(FunctionContext funcContext, Exception exception)
at System.Management.Automation.Interpreter.ActionCallInstruction`2.Run(InterpretedFrame frame)
at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)
at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)
Failed on 396
Error Record= Exception calling "ExecuteReader" with "0" argument(s): "ERROR [42703] ERROR: column "useful_part" does not exist;
Error while executing the query"

Command start time:<PHONE_NUMBER>3945

PS>TerminatingError(): "System error."

Actual error # was 42703. So nothing to do with what the Task Scheduler displayed. Simplot: Who cares. Boise: Wut? VisitPay: That's not your job!!!!!!!!!!!!!

Fixed. Will wait to run before clos-ed the case.

Ran-ed.


tf.avg_pool fails for data format "NCHW" on ppc64le

Please go to Stack Overflow for help and support:
http://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:

It must be a bug or a feature request.
The form below must be filled out.

Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Ubuntu 16.04 ppc64le
TensorFlow installed from (source or binary):
source
TensorFlow version (use command below):
'v1.0.1-0-ge895d5c-dirty'
Bazel version (if compiling from source):
Build label: 0.4.4-2017-04-13 (@80a07b5)
CUDA/cuDNN version:
No GPU
GPU model and memory:
No GPU
Exact command to reproduce:

You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
I'm trying to run tf.nn.avg_pool (https://www.tensorflow.org/api_docs/python/tf/nn/avg_pool) by passing in NCHW as the data format on a CPU (no GPU) but see the error:
" Executor failed to create kernel. Invalid argument: Default AvgPoolingOp only supports NHWC."
The API documentation mentions both NHWC and NCHW are supported. Please see logs below for complete details
Query: Does avg_pool support NCHW for CPUs?
Source code / 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. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.
Note: Below, NHWCToNCHW defined standard func to convert from one form to another (but not shown in this snippet)

x
array([[[[  1.,   2.,   3.],
[  4.,   5.,   6.],
[  7.,   8.,   9.]],

    [[ 10.,  11.,  12.],
     [ 13.,  14.,  15.],
     [ 16.,  17.,  18.]]]], dtype=float32)

t = tf.placeholder(tf.float32)
t = NHWCToNCHW(t)
t = tf.nn.avg_pool(t,ksize=[1,2,2,1],strides=[1,2,2,1],padding="SAME",data_format="NCHW")
actual = sess.run(t, {inputs: x})
E tensorflow/core/common_runtime/executor.cc:594] Executor failed to create kernel. Invalid argument: Default AvgPoolingOp only supports NHWC.
[[Node: AvgPool_1 = AvgPoolT=DT_FLOAT, data_format="NCHW", ksize=[1, 2, 2, 1], padding="SAME", strides=[1, 2, 2, 1], _device="/job:localhost/replica:0/task:0/cpu:0"]]
Traceback (most recent call last):
File "", line 1, in
File "/usr/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 767, in run
run_metadata_ptr)
File "/usr/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 965, in _run
feed_dict_string, options, run_metadata)
File "/usr/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 1015, in _do_run
target_list, options, run_metadata)
File "/usr/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 1035, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.InvalidArgumentError: Default AvgPoolingOp only supports NHWC.
[[Node: AvgPool_1 = AvgPoolT=DT_FLOAT, data_format="NCHW", ksize=[1, 2, 2, 1], padding="SAME", strides=[1, 2, 2, 1], _device="/job:localhost/replica:0/task:0/cpu:0"]]

Caused by op u'AvgPool_1', defined at:
File "", line 1, in
File "/usr/lib/python2.7/site-packages/tensorflow/python/ops/nn_ops.py", line 1765, in avg_pool
name=name)
File "/usr/lib/python2.7/site-packages/tensorflow/python/ops/gen_nn_ops.py", line 50, in _avg_pool
data_format=data_format, name=name)
File "/usr/lib/python2.7/site-packages/tensorflow/python/framework/op_def_library.py", line 763, in apply_op
op_def=op_def)
File "/usr/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 2327, in create_op
original_op=self._default_original_op, op_def=op_def)
File "/usr/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 1226, in init
self._traceback = _extract_stack()
InvalidArgumentError (see above for traceback): Default AvgPoolingOp only supports NHWC.
[[Node: AvgPool_1 = AvgPoolT=DT_FLOAT, data_format="NCHW", ksize=[1, 2, 2, 1], padding="SAME", strides=[1, 2, 2, 1], _device="/job:localhost/replica:0/task:0/cpu:0"]]

@npanpaliya any comment on this?

Please see #2660 for a discussion about the same issue in MaxPooling operation

MaxPool on GPU supports NCHW and NHWC
MaxPool on CPU only supports NHWC.


Get stats from Dig's results

Effect of depths
Number of generated inputs
Number of iterations
Compare quality over Dig's input generation vs random input generation

analysis of symbolic depths: d4be803959712e3f452c298d89d97036436447eb


Set CustomBarnKit, ModularFlightIntegrator, and GCMonitor to use AVC version

Should fix some issues with their current CKAN listings (1.1 versions
being installed on 1.0.5 etc)

@politas suggests that it might be worth keeping the x_netkan_jenkins and x_netkan_version_edit fields even if they are now redundant
@dbent mentions that if NetKAN ever starts looking at old versions, the version overrides could be useful.

LGTM.

Checked old versions for CustomBarnKit, and they all have valid .version files.
Likewise for ModularFlightIntegrator.
GCMonitor has a few old versions that don't have .versions, but it's been solid for some time.
I'm cleaning up the KSP version compatibility of all the old versions based on the .version files.


Gens rankings missing excluded players from rankings

https://github.com/lautaroangelico/WebEngine/blob/8e87ecb27b255dacc282e14a36ab5433e3307ef9/includes/classes/class.rankings.php#L497

Fixed in commit dac9069


pull request for e2e test at 01/27/2017 00:26:10

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Is it possible to set the minimum value on a DiceRoll object?

For instance, if I use the notation d3, is there a way to set the minimum value to 0 for the DiceRoll object? This would make possible values 0, 1, 2.
My use case is such that I'm using an average value to determine which types of dice to throw. The average value is a static value based upon different character attributes, but I would like to randomize the outcome to where the roll can deviate up or down some, though statistically over time would equal the average value. For instance, if the average value is 1, I would like to throw a d3 that would have an averageTotal of 1.
I know that I could construct a bunch of StandardDice to define the min, and max values, but I really like the ability to use notations to describe the dice throws. Is there a way that one can use the DiceRoll object in this way? I also know that I could just subtract 1 from the roll, but I would rather have the roll itself just return 0's if possible.

This currently isn't possible; Standard dice have a min value of 1.
As you mentioned, you could create a custom StandardDice object, and specify the min / max values manually.
Or do something like d3-1.
I am hoping to allow developers to create custom dice at some point, but it would require a fundamental re-write of how the parser works, so it's unlikely to be done any time soon.

Thinking about it, it could be useful to allow the StandardDice min value to be modified through notation, and for it to offset the sides / max value.
I think it should be fairly simple to implement, but I'd have to run some tests.
I'm not sure what the notation would be for something like that though. Any thoughts?
A couple of idea off the top of my head:
// using "db" for Dice with Base number
db0,3  // roll a 3 sides die, with a min value of `0` (0, 1, 2)
db4,2  // roll a 2 sided die, with a min value of `4` (4, 5)

// square brackets to define a range (Same rolls as above)
d[0, 2]  // roll a dice that has all values between 0, and 2 inclusive (0, 1, 2)
d[4, 5]  // roll a dice that has all values between 4, 5 inclusive (4, 5)


Made pytorch tests not be skipped in CI.

Same as #473, but for pytorch.

From the logs, pytorch does not provide a pypi installation. This means that we likely need another way to test it, e.g. through miniconda.

Yeah I tried several things, but so far to no avail.

Irrelevant now, torch is being installed in CI and pytorch tests are running.


TestController.hover() doesn't scroll elements other than the body unless the window is focused, hangs indefinitely

What is your Test Scenario?
On a page with an element that scrolls (not the body), I want to scroll to an item that's not in the element's view port. This blog suggests that .hover(element) should do the trick.
What is the Current behavior?
When the hover() is awaited, the test hangs indefinitely, until the browser window is brought to the foreground, at which time the scroll happens and the test proceeds. It would be my guess that in a headless environment, the scroll would never happen.
What is the Expected behavior?
The scrollable element should scroll to the hovered element without the window needing focus.
What is your web application and your TestCafe test code?

Sample Application:
https://jsfiddle.net/3vce7o4b/3/

TestCafe Test
NOTE: Though the JSFiddle example necessitates switching to an iframe, I have the same problem in my application without an iframe involved.
import { Selector } from "testcafe";

fixture("Test")
  .page("https://jsfiddle.net/3vce7o4b/3/")
;

test("Scroll internal element using hover", async tc => {
  await tc.switchToIframe(Selector("[name='result']"));
  await tc.expect(Selector("main").scrollTop).eql(0);
  await tc.hover(Selector("button"));
  await tc.expect(Selector("main").scrollTop).notEql(0);
});

Your complete test report:
NOTE: The test passes, but only if the test browser window is focused.
Make changes to the source files or press Ctrl+R to restart the test run.

A test run is starting...
 Running tests in:
 - Chrome 86.0.4240.183 / macOS 10.14.6

 Test
 ✓ Scroll internal element using hover

 1 passed (20s)

Screenshots:

Steps to Reproduce:

Run the above TC test from a terminal.
When the browser window appears, quickly switch back to the terminal.
If the test passes, run the test again in live mode and make sure the browser window isn't focused.

Your Environment details:

testcafe version: 1.9.4
node.js version: v13.13.0
command-line arguments: npx testcafe --live chrome <testfile>
browser name and version: Chrome 86.0.4240.183 (Official Build) (x86_64)
platform and version: MacOS 10.14.6

If I'm honest, what I really want is to just be able to do t.scrollIntoView(Selector(...)).
This workaround does what I want for the moment:
      return this._tc.eval(() => {
        document.getElementById(...)?.scrollIntoView();
      });

Thank you for your report. This looks like some kind of Chrome rendering optimization. I see this behavior only when a background window is fully covered by some other window. If even a small part of the window is visible, everything works fine for me (even if the window is still in the background). Also, everything works fine in Chrome Headless and Firefox (both GUI and Headless). I also noticed that t.takeScreenshot doesn't work under the same conditions. I tried adding the --disable-renderer-backgrounding flag, but it doesn't help. I need some more time to research this issue.

I tried to reproduce the issue in Chrome in the latest TestCafe version. It works as expected, so I'll close the issue. If the issue still persists in your case in the latest TestCafe version, please feel free to create a separate issue or add a comment here.


Running Jscep 2.3.0 with either Java 1.6 or Java 1.7

Hi @seize-the-dave,
I am working with Jscep 2.3.0 (running it with Java 1.8), and I am happy with the product. I would like to run it with an earlier version of Java, either 1.6 or 1.7; will Jscep 2.3.0 run fine with these versions? What is the minimum Java version required to run Jscep?
Thanks,
Aditya

I'm not sure.  I'm pretty sure it out to run with 6+.
Dave
On 8 Mar 2016 5:17 pm, "adityamalik"<EMAIL_ADDRESS>wrote:

Hi @seize-the-dave https://github.com/seize-the-dave,
I am working with Jscep 2.3.0 (running it with Java 1.8), and I am happy
with the product. I would like to run it with an earlier version of Java,
either 1.6 or 1.7; will Jscep 2.3.0 run fine with these versions? What is
the minimum Java version required to run Jscep?
Thanks,
Aditya
—
Reply to this email directly or view it on GitHub
https://github.com/jscep/jscep/issues/53.


[IOTDB-13]Support batched ingestion

Corresponding to JIRA issue: https://issues.apache.org/jira/browse/IOTDB-13
Add a new batched writing interface for users to write data in multiple sensors to TsFile.

I would add unit test in the next commit.

I have added the documentation for RowBatch.


feat: Gateway Mnemonic and Common Database

Fixes: https://github.com/fedimint/fedimint/issues/4446
Newly joined federations use clients that are created from a common mnemonic. This mnemonic is generated when the gateway boots. "Legacy" federations are still supported and can be upgraded by leaving, deleting the database, then re-joining. A test for this has been added and an automated endpoint for doing this can be added in a follow up.
This PR also cleans up the old standing issue of having separate databases per joined federation inside the gateway.
Note: This only provides backup-ability of the ecash of each joined federation. The recover API still needs to be added, as well as recovering lightning contracts as described  https://github.com/fedimint/fedimint/issues/5839

it's so beautiful 😍

Blocked on https://github.com/fedimint/fedimint/pull/5871

Had a quick chat with @bradleystachurski and he explained what I was doing wrong with the new mnemonic test. Now the mnemonic test runs as part of the upgrade tests. After the next release we can probably remove it, as we will have sufficient confidence that this upgrade path works.
Upgrade tests pass: https://github.com/fedimint/fedimint/actions/runs/10479242969/job/29024449941?pr=5852

can you add instructions for local testing?

Sure. Running the upgrade tests should be sufficient to verify it works (./scripts/tests/upgrade-test.sh v0.4.0). Version here doesn't matter for the mnemonic test. You can also test it in mprocs (gateway-lnd seed) to see the seed phrase for the various gateways.

also think we'll need to surface the mnemonic behind a protected path on the rpc server so I can reveal in the gateway for when someone creates it locally and they want to back it up

The /mnemonic path is already a protected path. You can verify in mprocs.

Ah shoot, newly added gatewayd-mnemonic test is failing. I will fix before trying to merge.

Ah shoot, newly added gatewayd-mnemonic test is failing. I will fix before trying to merge.

Likely related to https://github.com/fedimint/fedimint/issues/5948

Ah shoot, newly added gatewayd-mnemonic test is failing. I will fix before trying to merge.

Likely related to #5948

I still don't know the root cause, but reverting this commit fixed the upgrade tests locally. You can try a revert on your end to see if it's the same issue.
https://github.com/fedimint/fedimint/commit/11d61ad2046468acaf326dd63430e7b64906a62c

Backport failed for releases/v0.4, because it was unable to cherry-pick the commit(s).
Please cherry-pick the changes locally and resolve any conflicts.
git fetch origin releases/v0.4
git worktree add -d .worktree/backport-5852-to-releases/v0.4 origin/releases/v0.4
cd .worktree/backport-5852-to-releases/v0.4
git switch --create backport-5852-to-releases/v0.4
git cherry-pick -x 13df2f0682cdf74073561085465d17734d1b5c69 532c66e8a5c74fdf6d7eb3eea967cb82024d6bfa 81f769f5761b2b2f78565078b673d11702e236d0 128d475986762c5c547f3f01fba5b182ef6fd6bd a0ddae713dc493171ad3b27705b224268fb6aa33


Bug in greedy plan enumeration algorithm

When working on #75 to address #74, I noticed the following bug in the class that wraps the usage of the greedy plan enumeration algorithm (i.e., the class GreedyQueryOptimizer, formerly known as WrappingDPAlgorithm). The else-branch in the function called optimizePlan simply returns the given plan. This else-branch is for cases in which the root operator of the given plan is not a multiway join. While it is correct that the enumeration algorithm cannot simply be applied as it is done in the if-branch for plans that have a multiway join as their root, it is incorrect to simply return such a plan because, somewhere inside the plan, there may be more multiway joins. To fix this issue, the else-branch needs to traverse into all subplans, rewrite them recursively if necessary, and then combine the resulting, potentially changed subplans using the same type of operator that is the root of the given plan.
@chengsijin0817

After merging PR #79, the bug is now in the class called SimpleJoinOrderingQueryOptimizer.


py2app for Python 3

We want to make the standalones in Python 3.  I need to figure out if there are any differences with doing this in Python 2 vs. Python 3.  As the master branch has become relatively stable, this is the blocker for creating new OS X standalones.

Looking forward to having the standalones on the main development branch again. Let me know how I can help. I can run the Python 3 standalone pmag_gui on some data I'm familiar with before we release it to the public.

py2app should be compatible with Python 3.  However, it doesn't seem to work with 3.6.
So I downgraded to 3.5, which fixed some problems.
Then I ran into a LOT more problems.
Those are mainly documented here and here.  I don't have solutions yet.
I'm going to try using a non-Anaconda Python 3 and see if I run into fewer (or at least different) obstacles.

Using a different Python 3 (Enthought EDM) results in the same 'missing constraints' error.  So this is a more systematic issue.

Pyinstaller was able to produce a minimal working wxPython python 3 frozen binary.  So I'm trying with that for now.
This does require pulling pyinstaller from develop:
https://github.com/pyinstaller/pyinstaller/issues/2434
And I am now stuck for the real GUIs on this issue:
https://github.com/pyinstaller/pyinstaller/issues/2135

monkey patch for pyinstaller with EDM:
found:  /Users/***/anaconda/python.app/Contents/Resources/qt_menu.nib
try putting it here:  /Users/**/.edm/envs/py2app-python3/Resources/Python.app/Contents/MacOS/../../../../python.app/Contents/Resources
so it is now:   /Users/***/.edm/envs/py2app-python3/python.app/Contents/Resources/qt_menu.nib/
And it can, apparently, be found.
Now pyinstaller runs without errors.  However, no GUI shows up when I try to open it.

Looks like I need Pyside, which is only compatible with Python 3.4.  sigh.

pip install PySide.whl (downloaded)
pip install delocate: https://github.com/matthew-brett/delocate

Now let's try Anaconda with pyinstaller.
First, install develop version of pyinstaller:  pip install git+https://github.com/pyinstaller/pyinstaller.git
Minimal script non-windowed with matplotlib import --> success
Minimal script windowed with matplotlib --> success
Magic GUI --> pyinstaller runs without substantial errors, but then lots of SandboxViolations in console.  Progress.

Something in self.add_fit is breaking things in Demag GUI.  It isn't self.user_warning.  Seems this may be a matplotlib issue?

It is a matplotlib issue.  Trying with some different versions of matplotlib:
Newest version (2 something) --> fails
1.5.3 --> still fails
1.5.2 --> not available

So, with these specifications, everything works except actually showing a minimal plot:
pyinstaller # latest from develop as of 6/5/17.  must use develop branch or else breaks with wx
python # anaconda 3.5 or 3.6
matplotlib # 2.2, 1.5.3, or 1.5.1
Error output:
/Users/***/Python/PmagPy/dist/minimal_matplotlib ; exit;
 ~ /Users/***/Python/PmagPy/dist/minimal_matplotlib ; exit;
/var/folders/qb/3td4bl3s2pz__jc94hp7z2c40000gn/T/_MEIm5bXgl/matplotlib/font_manager.py:273: UserWarning: Matplotlib is building the font cache using fc-list. This may take a moment.
/var/folders/qb/3td4bl3s2pz__jc94hp7z2c40000gn/T/_MEIm5bXgl/matplotlib/font_manager.py:273: UserWarning: Matplotlib is building the font cache using fc-list. This may take a moment.
Intel MKL FATAL ERROR: Cannot load libmkl_avx2.dylib or libmkl_mc.dylib.
logout

https://github.com/scikit-learn/scikit-learn/issues/5046  # info on the Intel MKL error
Looks like there is a MKL library error.  Trying again after conda install nomkl:
Minimal matplotlib script works!!!  And so does demag_gui build.  There will likely be a performance hit for calculations (not using MKL), but still.

Install anaconda python 3.
To build:
pip install git+https://github.com/pyinstaller/pyinstaller.git
conda instal nomkl

Make the .spec file:
pyi-makespec --onefile --windowed --icon=.\programs\images\PmagPy.ico  --name=PmagGUI .\programs\pmag_gui.py

add the following to the resulting pmag_gui.spec file:
files = [
     ('/Users/***/PmagPy/pmagpy/data_model/*.json', './pmagpy/data_model/')
]

...
a = Analysis(['programs/pmag_gui.py'],
    ...
    datas=files,
...

Then run:
pyinstaller pmag_gui.spec

This will create pmag_gui.app in PmagPy/dist.

I have run the pmag_gui that you linked in your dropbox and it runs on my system. I can see data with demag_gui. When I try to create an upload file with these MagIC files:
HagstrumMagICFiles.zip
the program hangs.
I also do not see a console window for looking at the error and warnings. Does the consolw=e exist for you @moonshoes87 ?

I just replaced both GUIs on Dropbox with version that do show a console window.  Now I'll have a look at the upload problem.

I fixed the upload issue and another problem or two.  An updated Pmag GUI is synching on Dropbox now.  Let me know what else you find!

@njarboe, from your testing, do you think the Python 3 OS X standalone is ok to be released?  I technically have one more week of work before going on leave, so I'm trying to wrap up loose ends as much as possible.

@moonshoes87, I have been using the new pmag_gui stand-alone to look at some data. Looks good so far. I just opened a new issue on the gui problems in the demag_gui, but that is probably an on issue and not related to the new code. I am leaving to do some field work tomorrow and won't be back until Sunday/Monday. If you are out for a while starting next week I would say you should push the new standalones to the public site by Friday. Thanks for the hard work.

Thanks @njarboe.  I will release OSX standalones by the end of the week.  Please create new issues for anything else weird that comes up!  Enjoy your field work :)


if replacement audio file is shorter than ad, silence will fall.

Setup:

download & install adfree APK
create a local folder with contents of adfree-resources/*.mp3
configure adfree app for:

local music

reproduction:

play music in spotify

expected result:

ads are replaced with multiple "replacement audio" files until spotify music resumes

actual results:

when a single "replacement audio" track ends, there is only silence until spotify music resumes

As ads are fairly short, I am personally not planning to add such feature to play multiple tracks in sequence. But I flag it as a feature-request.


Added the CLOSE (x) button to the pop up menu to make it simple and elegant (#1148)

Related Issue
Fixes: #12027
Description
Added a close button to the popup menu that appears on the webpage. This button allows users to dismiss the popup by clicking on the "X" located at the top right corner. The button size was customized to be smaller (24x24px) to ensure it does not interfere with the overall popup layout. The close functionality was implemented using a simple JavaScript function to hide the popup when clicked.
Motivation: This enhancement was necessary to provide a better user experience, allowing users to easily exit the popup without interacting with other elements.
Dependencies: No additional dependencies are required for this change.
Type of PR
Feature enhancement
Before:

After:

video :
https://github.com/user-attachments/assets/9d9725ae-f358-4366-8362-cc5732e2d77c
Checklist:
I have made this change from my own.
My code follows the style guidelines of this project.
I have performed a self-review of my own code.
I have commented my code, particularly in hard-to-understand areas.
I have tested the changes thoroughly before submitting this pull request.
I have provided relevant issue numbers and screenshots after making the changes.
Im a GSSOC -Ext and Hacktober fest contributor !

@PriyaGhosal , able to merge with any error !

@tejasam571

no cross button is working


Migration from 2.2 to 3.0 doc mentions Authorization overload that doesn't exist

I'm using .Net Core 3.0 Preview 8
The #authorization section talks about how middleware can be authorized and

If the app uses an AuthorizeFilter as a global filter in MVC, we recommend refactoring the code to provide a policy to the UseAuthorization middleware.

In the following example, a custom policy to be applied to all requests when UseAuthorization is called, and the HomeController allows access without the user signing into the app:

Followed by suggestion the call to
// Configure method
app.UseAuthorization(new AuthorizationPolicyBuilder().Build()));
app.UseEndpoints(endpoints =>
    {
        endpoints.MapDefaultControllerRoute();
    });

But app.UseAuthorization() does not have an overload taking a policy. So my next guess was this:
// ConfigureServices method
services.AddAuthorization(options =>
{
    options.DefaultPolicy = new AuthorizationPolicyBuilder()
        .RequireAuthenticatedUser()
        .RequireScope("ScopeName")
        .Build();
});
// omitted call to services.AddAuthentication().AddIdentityServerAuthentication() ...

Assuming if the default policy was to require authentication with my scope, that all endpoints would 401 out if no Bearer token was present. Also, my understanding was such that I remove the global AuthorizeFilter in MVC or SignalR, such as
// ConfigureServices method
services.AddControllersWithViews(options =>
{
	options.Filters.Add(new AuthorizeFilter());
...

But it does not appear to be the case, requests go through. Overall, I do not know how to use the new Authorization middleware, or how to setup globally required Authorization on any endpoint, which appears to be the suggestion in the current docs (Middleware, MVC, or SignalR), and the docs appear to be for an older preview (advising to use an overload of UseAuthorization() that does not exist) or just plain missing (IE, the aspnet/core/security/authorization/policies section of the docs does not appear updated for 3.0).

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

ID: 76f39eec-9dd1-6167-56ec-47b26e0b5063
Version Independent ID: f9f2b0f3-01eb-cf58-8767-194fe0fc0e44
Content: Migrate from ASP.NET Core 2.2 to 3.0 Preview
Content Source: aspnetcore/migration/22-to-30.md
Product: aspnet-core
Technology: aspnetcore-migration
GitHub Login: @tdykstra
Microsoft Alias: tdykstra

So you're wanting to apply an auth policy to a single endpoint?
Something like the below?
app.UseEndpoints(endpoints =>
{
    endpoints.MapDefaultControllerRoute()
        .RequireAuthorization(new AuthorizeAttribute("policyName"));
});

Thanks for the fast response!
I was hoping to secure all endpoints, MVC/SignalR by default, and I have some middleware that runs before MVC/SignalR that I would like to know is authenticated or not. I didn't realize there was a RequiresAuthoriztion() chain call. But now I see that in the HealthChecks example.
So I took your example:
services.AddAuthorization(options =>
{
    options.DefaultPolicy = new AuthorizationPolicyBuilder()
        .RequireAuthenticatedUser()
        .RequireScope("ScopeName")
        .Build();
});

...

app.UseEndpoints(endpoints =>
{
    endpoints.MapHub<SessionsHub>("/Sessions").RequireAuthorization();
    endpoints.MapControllers().RequireAuthorization();
});

And that works, nice! So this is the new advice for adding authorization to MVC/SignalR then? If I'm in middleware, do I just look at HttpClient like this? if (httpContext.User?.Identity?.IsAuthenticated is true) or call httpClient.AuthenticateAsync call? Or is there a new way to look at auth in middleware with the 3.0 auth middleware in place?
IE:
app.UseAuthentication();
app.UseAuthorization();
app.UseMiddleware<SomethingCustom>(); // Wants to know if user is authorized or not.
app.UseEndpoints(endpoints =>
{
    endpoints.MapHub<SessionsHub>("/Sessions").RequireAuthorization();
    endpoints.MapControllers().RequireAuthorization();
});

@rynowak or @HaoK what is the recommended way to enforce authentication for all endpoints and middleware?

UseAuthorization middleware with the fallback policy should do that
something like https://github.com/aspnet/AspNetCore/blob/8c02467b4a218df3b1b0a69bceb50f5b64f482b1/src/Security/Authentication/Negotiate/samples/NegotiateAuthSample/Startup.cs#L20 would require an authenticated user by default.  You can set the policy to something different, Fallback is used when no policy is specified, its slightly different than default policy which is used when [Authorize] is specified

Alright, so just to make sure I'm fully understanding, if we take my code again, and use FallbackPolicy, and remove .RequireAuthorization() from the MVC/SignalR calls, then that's the right approach in 3.0 for globally requiring authentication? Do I need to worry about setting the DefaultPolicy if I take this approach?
services.AddAuthorization(options =>
{
    options.FallbackPolicy = new AuthorizationPolicyBuilder()
        .RequireAuthenticatedUser()
        .RequireScope("ScopeName")
        .Build();
});

...

app.UseAuthentication();
app.UseAuthorization();
app.UseMiddleware<SomethingCustom>(); // Wants to know if user is authorized or not.
app.UseEndpoints(endpoints =>
{
    endpoints.MapHub<SessionsHub>("/Sessions");
    endpoints.MapControllers();
});

And in middleware, just look at httpContext.User.Identity?
You guys have been very helpful and quick to respond, thank you!

@HaoK  should we update the doc or is this issue enough to doc this? This issues is discoverable from the end of the doc page:

We should update the doc, I can take a look at this next week

@AdamDotNet correct, FallbackPolicy lets you avoid specifying the default policy or worrying about adding the Authorize attribute/metadata anywhere, as the middleware will take care of applying it consistently (that's the idea behind the feature anyways, if its not, that would be a bug)

@AdamDotNet  can you write up an update for the doc (in this issue), or PR the doc. You can edit the doc in the browser and make a PR.

@Rick-Anderson -  Sure. Do you suggest changing the existing example to remove the incorrect call to app.UseAuthorization(new AuthorizationPolicyBuilder().Build())); + add modify endpoints.MapDefaultControllerRoute(); to endpoints.MapDefaultControllerRoute().RequireAuthorization();? Or to mention the FallbackPolicy? Both?

@haok can you help @AdamDotNet  get the update started?

I would suggest both, since they are slightly different, the RequireAuthorization only applies to the default controller route, while demonstrating the fallback scenario to show how to protect all endpoints.
Does that make sense?


Re-auth with Last.fm not working

tl;dr It suddenly stopped getting track names (searches for  - null; I have to add everything manually). I tried to re-auth and it won't open the dialog to do so.
(I'm using Google Music Play.)

Anyone use google music to verify? Potentially related to #843?

Okay so, I tried reinstalling the extension for the last time and also cleared some of the Chromes browsing data, cleared Java stuffs (because why not). Re-authenticated (this works, yay!) and it seemingly worked... for one song. At least, it detects the track name, but doesn't scrobble.
So ya, close, but no cigar.


Errors in console for 'troubleshooting information' admin page

When calling the 'troubleshooting information' admin page (i.e. the /admin/plugins/info URL), I see these messages in the log:
[ERROR] console - Failed to load 'ep_comments_page/static/js/index' for 'ep_comments_page/comments_page/client_hooks/postToolbarInit': ReferenceError: window is not defined
[ERROR] console - Failed to load 'ep_comments_page/static/js/index' for 'ep_comments_page/comments_page/client_hooks/postAceInit': ReferenceError: window is not defined
[ERROR] console - Failed to load 'ep_comments_page/static/js/index' for 'ep_comments_page/comments_page/client_hooks/aceAttribsToClasses': ReferenceError: window is not defined
[ERROR] console - Failed to load 'ep_comments_page/static/js/index' for 'ep_comments_page/comments_page/client_hooks/aceEditorCSS': ReferenceError: window is not defined
[ERROR] console - Failed to load 'ep_comments_page/static/js/index' for 'ep_comments_page/comments_page/client_hooks/aceEditEvent': ReferenceError: window is not defined
[ERROR] console - Failed to load 'ep_comments_page/static/js/index' for 'ep_comments_page/comments_page/client_hooks/aceInitialized': ReferenceError: window is not defined

ep_comments_page 0.1.37 (installed via npm install)
ep 1.8.6-256-g325941a

Steps to replicate:
$ git clone https://github.com/ether/etherpad-lite.git
$ cd etherpad-lite
$ bin/run.sh
$ npm install ep_comments_page
$ echo '{"users":{"admin":{"password":"changeme1","is_admin":true}}}' > credentials.json
$ bin/fastRun.sh

and then, in another terminal:
$ curl --user admin:changeme1 http://<IP_ADDRESS>:9001/admin/plugins/info

This is on Fedora 33, with
nodejs-14.14.0-2.fc33.x86_64
npm-6.14.8-<IP_ADDRESS>.2.fc33.x86_64


fix bugs of common package

fix bug of common package with two func:

CheckSlotDistributionEqual
CheckVersionChecksum

 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.Seer seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Update UI when there are no fields

For partials or files that don't have fields, the field panel is currently blank (see screenshot) and it looks broken.
When this is the case, we should add a message:
No editable fields were found in this file. [Edit source]

Where "edit source" is a button that, when clicked, it switches to the "Source" tab so that the user can edit the source directly.

Alternative approach: many of the fields that show up blank are top-level mappings, e.g.
foo: bar

In these scenarios it could be useful to have the field editor just show fields for all values in the file (i.e. a file that's completely autoguessed). This would be useful for strings files, config mappings, etc.


Class 'Walker_Nav_Menu' not found

Hi,
I have tried using the controller implementation and the direct reference to the bootstrap4 navwalker. However both approaches throw this error:
Fatal error: Class 'Walker_Nav_Menu' not found in /var/www/vendor/mwdelaney/sage-bootstrap4-navwalker/bootstrap4-navwalker.php on line 20. Which is strange because the Walker_Nav_Menu class should exsist in the WP application?
Looking further, given that all the /vendor packages are loaded before the wp stuff in the /web/wp-config.php the WP_Nav_Menu class could be called before wp was loaded?

Ah wait, ofcourse the package should be installed within the themes dir 👍 My bad!


Create Gateway API client

As discussed with Starkware, Gateway support is to be discontinued. So I'm closing this.


Expose BigQuery schema autodetect in Java SDK

Fixes #20515

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Java_GCP_IO_Direct PreCommit

All checks passed. Please take a look! @robertwb @Abacn

I'm not sure I understand this PR. The schema is required if CREATE_IF_NEEDED is set, in which case the schema is needed to create the table. We don't currently require a schema for existing tables. How does autodetect schema work if the table doesn't exist yet?

waiting on author


Chore/#4/setup ci

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@1cd5a72). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #12   +/-   ##
=======================================
  Coverage        ?   75.79%
=======================================
  Files           ?        4
  Lines           ?      157
  Branches        ?        0
=======================================
  Hits            ?      119
  Misses          ?       38
  Partials        ?        0

Flag
Coverage Δ

unittests
75.79% <0.00%> (?)

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

Continue to review full report at Codecov.

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


Switch to using CircleCI and automated testing with Karma

This is ready and has docs. I'm ready to merge it, unless @mike-thompson-day8 wants to review it first.

I just want to let you know that karma start --single-run=false does not work with larger code bases, because cljs compiler (or google closure compiler, I don't know) writes compiled files in chunks. So even though you increase the value of autoWatchBatchDelay, karma may start with incomplete output.

Hey, thanks for the heads up. That does explain some of the errors I saw on our internal projects running with cljsbuild auto. I don't think it would be an issue here though, because we're doing a full cljsbuild before we even start up karma. Is that correct?

Yes, it's correct.


CDI Integration fails with latest camel-2.17

With
    @Inject
    @ContextName("cdi-context")
    private CamelContext camelContext;

I see
[INFO] Caused by: java.lang.Exception: {"WFLYCTL0080: Failed services" => {"jboss.deployment.unit.\"camel-tests.war\".WeldStartService" => "org.jboss.msc.service.StartException in service jboss.deployment.unit.\"camel-tests.war\".WeldStartService: Failed to start service
[INFO]     Caused by: org.jboss.weld.exceptions.DeploymentException: WELD-001408: Unsatisfied dependencies for type CamelContext with qualifiers @ContextName
[INFO]   at injection point [BackedAnnotatedField] @Inject @ContextName private org.wildfly.camel.archetype.test.MyServlet.camelContext
[INFO]   at org.wildfly.camel.archetype.test.MyServlet.camelContext(MyServlet.java:0)
[INFO] WELD-001475: The following beans match by type, but none have matching qualifiers:
[INFO]   - Default CDI Camel Context
[INFO] "}}
[INFO] 	at org.jboss.as.controller.client.helpers.standalone.impl.ServerDeploymentPlanResultFuture.getActionResult(ServerDeploymentPlanResultFuture.java:134)
[INFO] 	at org.jboss.as.controller.client.helpers.standalone.impl.ServerDeploymentPlanResultFuture.getResultFromNode(ServerDeploymentPlanResultFuture.java:123)
[INFO] 	at org.jboss.as.controller.client.helpers.standalone.impl.ServerDeploymentPlanResultFuture.get(ServerDeploymentPlanResultFuture.java:85)
[INFO] 	at org.jboss.as.controller.client.helpers.standalone.impl.ServerDeploymentPlanResultFuture.get(ServerDeploymentPlanResultFuture.java:42)
[INFO] 	at org.jboss.as.controller.client.helpers.standalone.ServerDeploymentHelper.deploy(ServerDeploymentHelper.java:55)
[INFO] 	at org.jboss.as.arquillian.container.ArchiveDeployer.deployInternal(ArchiveDeployer.java:77)

That is a consequence of CAMEL-5742 that has been addressed as part of the new Camel CDI component. The issue is marked as fixed since Camel 2.16.0 though that is not actually the case :).
In your case, you just have to remove the @ContextName("cdi-context") qualifier. The new component deploys a CamelContext bean with the default qualifier if no CamelContext beans exist (with whatever qualifier) in the application. Then, the application must refer to these Camel context beans and no Camel contexts will be created automatically.

Currently we have multiple ways of wiring CamelContext in CDI. First, we use an annotated RouteBuilder and inject that into the target bean ...
source:
@ContextName("contextA")
public class RoutesContextA extends RouteBuilder {

    @Override
    public void configure() throws Exception {
        LOG.info("Adding route from " + a + " to " + b);
        from(a).to(b);
    }
}

target:
public class CDIIntegrationTest {

    @Inject
    RoutesContextA routesA;

The second approach is we @Inject a named CamelContext
source:
@Startup
@ApplicationScoped
@ContextName("cdi-context")
public class MyRouteBuilder extends RouteBuilder {

    @Override
    public void configure() throws Exception {
    	from("direct:start").beanRef("helloBean");
    }
}

target:
@WebServlet(name = "HttpServiceServlet", urlPatterns = { "/*" }, loadOnStartup = 1)
public class MyServlet extends HttpServlet {
    @Inject
    @ContextName("cdi-context")
    private CamelContext camelContext;
}

@tdiesler, the binding of Camel contexts and associated abstractions like RouteBuilder, Endpoint, and ProducerTemplate, relies on CDI qualifiers.
If an application declares two {{CamelContext}} beans:
@ContextName("first-context")
FirstCamelContext extends DefaultCamelContext {
}

@ContextName("second-context")
SecondCamelContext extends DefaultCamelContext {
}

Then the resources are bound using the declared qualifiers, e.g.:
@ContextName("first-context")
public class FirstRouteBuilder extends RouteBuilder {
}

@ContextName("first-context")
public class SecondRouteBuilder extends RouteBuilder {
}

@Inject
@Uri("direct:inbound")
@ContextName("first-context")
ProducerTemplate firstProducer;

@Inject
@Uri("direct:inbound")
@ContextName("second-context")
ProducerTemplate secondProducer;

And the contexts can be injected using standard CDI type safe resolution mechanism, e.g.:
@Inject
@ContextName("first-context")
CamelContext firstContext;

@Inject
@ContextName("second-context")
CamelContext secondContext;

@ContextName is just a CDI qualifier that Camel CDI provides for convenience to bind resources to named contexts. However, any CDI qualifiers can be used to declared Camel contexts and bind their resources and works identically than the above example.
Besides, as the multi Camel contexts application may not be the majority of use cases, Camel CDI deploys for convenience a default Camel context with the @Default built-in CDI qualifier if any Camel context is deployed by the application. So in your example, which is a single Camel context example, you can choose to use that default Camel context bean. Or the application can declare its own Camel context bean with whatever qualifiers that it'll use to bind the associated resources.
In a way, the new Camel CDI component restores the type safety of CDI, hence CAMEL-5742. Indeed, the previous version was automagically creating new Camel context instances, not even beans, for each @ContextName it was discovering in the deployment.
The problem for WildFly Camel is that it relies on the presence of the @ContextName annotation in the deployment to activate the subsystem. That logic is unfortunately not complete and should ideally be improved to cover more use cases. This relates to #925.

Done


Liquid VR 360 SDK?

Anandtech cites it in catalyst 17.7.2 so coming soon?

Was released yesterday.

Nice, but where? can’t find it.. can provide link? It’s public?

http://support.amd.com/en-us/download

Sorry I meant the Liquid VR SDK (like the title of the issue) not the crimson 17.7.2 driver..
thanks

The latest public LVR SDK is here: https://github.com/GPUOpen-LibrariesAndSDKs/LiquidVR
What is LiquidVR 360 SDK? What feature are you looking for?

thanks for offer..
no need for NDA version..
just what is not clear to me is from Anandtech:
"AMD has brought yet another asset to the LiquidVR platform: LiquidVR 360 SDK. The SDK enables end-to-end GPU-accelerated HEVC 4K x 4K video playback."
can at least speak publicily if AMF doesn't support already  4K x 4K HEVC decoding on supported GPUs?

The 360 SDK is based on AMF and AMF does support such decoding. The 360 SDK showcases high-level integration into GPU pipeline, timewarp since with 360 video it can be done more accurate than in HMD software, ambisonic audio , stereoscopic and integration with HMDs.

privet Mikhail. Exist whether video players based on LiquidVR 360 or use it?

I am not aware of a public player based on LiquidVR 360 SDK. Though, the SDK includes a sample player. What is your interest? There are plenty of 360 players around.

It's been about a year since the SDK was released and I would like to use at least one video player using this technology.

Speak russian?
2018-07-03 16:42 GMT+03:00 Mikhail_AMD<EMAIL_ADDRESS>
I am not aware of a public player based on LiquidVR 360 SDK. Though, the
SDK includes a sample player. What is your interest? There are plenty of
360 players around.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/GPUOpen-LibrariesAndSDKs/LiquidVR/issues/9#issuecomment-402161784,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Am3IOevcSB_vC-u29HVxMI8vwWXwa5zbks5uC3TQgaJpZM4OlcsE
.

Yes, Skype?
<EMAIL_ADDRESS>Mikhail
From: diarrat<EMAIL_ADDRESS>Sent: July 3, 2018 9:43 PM
To: GPUOpen-LibrariesAndSDKs/LiquidVR<EMAIL_ADDRESS>Cc: Mironov, Mikhail<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [GPUOpen-LibrariesAndSDKs/LiquidVR] Liquid VR 360 SDK? (#9)
Speak russian?
2018-07-03 16:42 GMT+03:00 Mikhail_AMD <notifications<EMAIL_ADDRESS>
I am not aware of a public player based on LiquidVR 360 SDK. Though, the
SDK includes a sample player. What is your interest? There are plenty of
360 players around.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/GPUOpen-LibrariesAndSDKs/LiquidVR/issues/9#issuecomment-402161784,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Am3IOevcSB_vC-u29HVxMI8vwWXwa5zbks5uC3TQgaJpZM4OlcsE
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/GPUOpen-LibrariesAndSDKs/LiquidVR/issues/9#issuecomment-402338424, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AOcWv0xFKXm1mwsZte5BqdTh7p8kXz0Kks5uDB2ZgaJpZM4OlcsE.


Mapbox limit not working

From @terry90
Hello Denis,
It's me again :)
I do not want to open too much issues, so I'm gonna ask you by email.
What is wrong with the mapbox provider ?
I'm getting 422 errors claiming:
{"message":"limit must be combined with a single type parameter when reverse geocoding"}%
But this is not the weird part, when I curl the exact same query by replacing "%20" with "\ " it works !
I tried to change the limit option, remove it etc.. the best I can get is a features: [], EMPTY !
It drives me mad haha.
Did you encountered this issue ?
Here are proofs:
curl 'https://api.mapbox.com/geocoding/v5/mapbox.places/9 rue Alphonse Penaud Paris, 75020 France.json?access_token=YOURTOKEN&limit=1'
will return: ..."features":[]...
curl 'https://api.mapbox.com/geocoding/v5/mapbox.places/9 rue Alphonse Penaud Paris, 75020 France.json?access_token=YOURTOKEN&limit=5'
will return:
{"message":"limit must be combined with a single type parameter when reverse geocoding"}
curl 'https://api.mapbox.com/geocoding/v5/mapbox.places/9\ rue\ Alphonse\ Penaud\ Paris,\ 75020\ France.json?access_token=YOURTOKEN&limit=1'
will return a single result in features[], and the right one !!!
Have a good day !

I've got it working using the CLI & geocoder-geojson library.
I simply dropped the postal code.
$ geocode -p mapbox "9 rue Alphonse Penaud Paris, France" --limit 1

It looks like it's a Mapbox issue, when I dropped the postal code it worked, I'm pretty sure it does the following:
Query: 9 rue Alphonse Penaud Paris, 75020 France

Find 9 (thinks it's longitude)
Finds 75020 (thinks it's latitude since it's after a comma)

Mapbox's Forward & Reverse Geocoding API is very similiar.
Reverse
http://api.tiles.mapbox.com/v3/{mapid}/geocode/{lon},{lat}.json

Forward
http://api.tiles.mapbox.com/v3/{mapid}/geocode/{query}.json

CC: @sbma44

Oh right ! Nice one.
I did not thought about that.

That one is a strange one, I'm going to close this issue, but this doesn't fix the Mapbox provider issue.

Hey, belatedly: thanks for the bug report. I can replicate this issue and we'll be having a closer look.

@sbma44 Cool stuff! This behaviour looked a bit strange to me, I don't ping you very much. (don't mind my very fast reply, I had just refreshed my email box when you submitted your comment).

Hey @DenisCarriere, Thanks a ton for finding this bug! Happy to say I've tested and deployed a fix for this as of last night. The fix itself can be seen here: https://github.com/mapbox/carmen/pull/579
Feel free to ping us any time if you have any questions or run into any bugs!

Afterthought: It can take up to 48 hours for the query to clear out of our cache. If you add something like &cachebuster=1234 to the end of the query it will miss the cache, hitting the server with the fixed result.

Thanks ! I will use it back :)

@ingalls Thanks! 👍 The query works now, it's great that you've included the Paris, France example in the Carmen test cases 😄


Proper way to whitelist/publicize a health api route with authMiddleware?

With the old middleware, i performed a "public" check on my /api/healthz route, and allowed public traffic before ever calling useAuth. With the new authMiddleware, adding the /api/healthz route to the publicRoute list does not accomplish this behavior.
Instead I'm using a negative look ahead like so:
 apiRoutes: ['/api/(\\?!healthz)(.*)', '/trpc/(.*)'],

Is this the recommended method?

Hello @dcyoung
I would suggest to either ignore the middleware for the health endpoint if you don't consider the middleware part of the health checks. Otherwise I would add ignore the health endpoint for authMiddleware specifically by using the ignoredRoutes property.

Perfect. Thanks!

@dimkl Using the ignoredRoutes works and is definitely more explicit, but I'm seeing the following log whenever I hit the health endpoint:
Clerk: The middleware was skipped for this request URL: http://<IP_ADDRESS>:3000/api/healthz. For performance reasons, it's recommended to your middleware matcher to:
export const config = {
  matcher: ["/((?!.*\\..*|_next).*)","/","/(api|trpc)(.*)"],
};

Alternatively, you can set your own ignoredRoutes. See https://clerk.com/docs/nextjs/middleware

We run health checks in our orchestration layer at frequent intervals (~15s), and this may pollute our logs. Anyway to suppress this log?
Also, the config recommended by the message is identical to the one exported in my middleware file.

🤔 We cannot determine if the request is for ignored route or not before receiving a request, so I couldn't find a way to skip this log.
We are taking a deeper look at some conditions we have to allow adding the /api/healthz endpoint to both publicRoutes and apiRoutes and make it work. Eg :
export default authMiddleware({
    apiRoutes: [/\/api/],
    publicRoutes: [/\/api\/healthz/]
});

Until we make the above or something similar available, I would suggest you to add ignore the /api/healthz in the middleware exported config.matcher.
Let me know if this works for you.

Hi there @dimkl
We've been ok with polluted logs in the short term, but we're still hoping for a resolution on this.

Until we make the above or something similar available, I would suggest you to add ignore the /api/healthz in the middleware exported config.matcher.

Can you provide a snippet/example of that matcher regex.
Our current setup looks like:
export default authMiddleware({
    publishableKey: env.XXX,
    ....,
    ignoredRoutes: ['/api/healthz(.*)'],
});

export const config = {
    matcher: ["/((?!.*\\..*|_next).*)", "/", "/(api|trpc)(.*)"],
};

@dimkl  ^ friendly bump

Can you provide a snippet/example of that matcher regex.

I have not verified it, but it is probably something like this:
matcher: ["/((?!.*\\..*|_next|api\\/healthz).*)", "/", "/(api|trpc)(?!\\/healthz)(.*)"]

This basically adds /healthz to the negative lookahead construct.

I'd like to note that we're working on an easier way to accomplish this with middleware as a first class solution in our upcoming major release. We don't have an exact release date yet but sometime within the next couple months. Hopefully that will make this way easier for you all 😁

Our current middleware is now opt-in instead of opt-out, and so this particular pattern should be much easier to accomplish: https://clerk.com/docs/references/nextjs/clerk-middleware.


🌐 Add French translation

The filestructure must be the same so the URLs when changing in between languages don't break.
Thank you Lucas :))


How do I launch the api on a graphics card other than cuda: 0

1、deepspeed --include localhost:4 api.py
2、CUDA_VISIBLE_DEVICES=4 python api.py
Neither method can specify cuda to use, so cuda: 0 is used by default
How do I set it?

same question,  CUDA_VISIBLE_DEVICES not working