Incorrect sample command az network vhub route-table route add

Following sample below:
az network vhub route-table route add -n MyRouteTable -g MyResourceGroup --vhub-name MyVhub --destination-type CIDR --destinations "<IP_ADDRESS>/16" "<IP_ADDRESS>/16" --next-hop-type ResourceId --next-hop /subscriptions/MySub/resourceGroups/MyResourceGroup/providers/Microsoft.Network/azureFirewalls/MyFirewall
Outputs below error:
Usage error: --next-hop and --route-name must be provided as you are adding route to v3 route table.
Therefore, command is missing the --route-name parameter/input.
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 7c572a40-459c-4dc0-0664-226f3ce37eaa
Version Independent ID: b823f80d-9831-bcb3-7baf-766f7c95a0e5
Content: az network vhub route-table route
Content Source: latest/docs-ref-autogen/network/vhub/route-table/route.yml
Service: virtual-network
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

Thank you for opening this issue, we will look into it.


URI encoding of '+' on re-callback

The current callback uses encodeURI to encode the content of the editor.
When typing a string that includes the '+' character the onTextChanged callback returns an empty space.
Using encodeUriComponent seems to fix the issue, however I haven't done extensive testing to see if it affects other elements.

I'm also facing the same issue where all '+' signs get converted to spaces from the editor.
Did you include richeditor-android source code as sub-module in your project to update encodeUri() ? I'm hesitant to include richeditor-android src code in my project for this bug fix.


Added Web Design Playground, Second Edition

Hi,
Stjepan here from Manning. I thought this title might be a good match for your list. Thank you for considering it.
Best,

Thank you.
Best,
On Thu, Mar 23, 2023 at 4:30 PM Ritik Patni @.***>
wrote:

Merged #188
https://github.com/RitikPatni/Front-End-Web-Development-Resources/pull/188
into master.
—
Reply to this email directly, view it on GitHub
https://github.com/RitikPatni/Front-End-Web-Development-Resources/pull/188#event-8828968763,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANDIEMNRXPHG6JJWMXXAQ3TW5RUC5ANCNFSM6AAAAAAVFPE5MY
.
You are receiving this because you authored the thread.Message ID:
<RitikPatni/Front-End-Web-Development-Resources/pull/188/issue_event/8828968763
@github.com>

--
Stjepan Jureković
Manning Publications - Marketing
@.***


Gobblin launcher standard implementations

Add the following to :

Builder for JobSpec
A helper class DefaultConfigurableImpl for implementations of Configurable
A helper class JobCatalogListenersList to manage callbacks to JobCatalogListener instances
A basic implementation of JobCatalog which stores JobSpecs in memory
Unit tests for the above

@autumnust @ibuenros Can you review?

I will merge that since it contains a fix for the broken build. This should not affect anything. I will apply any review comments additionally.


json: cannot unmarshal array into Go value of type database.FollowOperation

The title is rather clear.

https://steemd.com/b/4703885

Somehow, I bet you also had a run-in with the pesky de-referencing asterisk..... *
;)


arraysum Not Found

Description
Aim
Sum elements of an array using std::array::sum(_input).
Expected behavior
Successfully proving a Noir program as follows:
use dep::std;

fn main() {
    let arr = [1, 2];
    let arr_sum = std::array::sum(arr);
    constrain arr_sum == 3;
}

Bug
Running nargo prove p on the above fails with:
thread 'main' panicked at 'internal error: entered unreachable code: cannot find a low level opcode with the name arraysum in the IR', crates/noirc_evaluator/src/ssa/code_gen.rs:613:17

Environment

OS: macOS
Proving backend

[ ] default
[x] wasm-base

aztec_backend = { optional = true, package = "barretenberg_wasm", git = "https://github.com/noir-lang/aztec_backend", rev = "fb42a54a04f3952f422e8fa4eff4f8eb2ca4e51b" }

Additional context
Not sure if it's simply unimplemented or a bug. Feel free to re-label as appropriate.

arraysum and arrayprod are deprecated functions, they can easily be implemented by the user, for instance arraysum is:
fn sum(a:[Field]) -> Field {
  let mut s = 0;
  for i in a {
    s = s + i;
  };
  s
}

Like in rust, we want to keep the stdlib lean so that we minimize backward compatibility issues. You can refer to https://users.rust-lang.org/t/rust-should-have-a-big-standard-library-and-heres-why/37449/25 for discussion about what should be in stdlib.
As a result, we are going to remove completely arraysum and arrayprod.

great


Fix #2488

BUGS:

This PR will fix the ErrorException when editing product in different language

Fixes #2488

Already Fixed in Master


Authz does not work as expected

Description:
User {"user":"paas", "ns":"paas"} can write replication controller with namespace "test" which is not expected.
My Config:
# kubectl version
Client Version: version.Info{Major:"0", Minor:"13+", GitVersion:"v0.13.2-1-g58b98aca84becd", GitCommit:"58b98aca84becdabbfc8d719c47b375a6a1bc855", GitTreeState:"clean"}
Server Version: version.Info{Major:"0", Minor:"13+", GitVersion:"v0.13.2-1-g58b98aca84becd", GitCommit:"58b98aca84becdabbfc8d719c47b375a6a1bc855", GitTreeState:"clean"}

# cat known_tokens.csv
myToken0,admin,8888,
myToken1,fb,1003,
myToken2,test,1004,
myToken3,paas,1005,
myToken4,read-only,1006,

# cat authz_policy.json
{"user":"admin"}
{"user":"test", "ns":"test"}
{"user":"fb", "ns":"facebook-pmd"}
{"user":"paas", "ns":"paas"}
{"user":"read-only", "readonly":true}

Bug reproduce steps:
# ls
test.ns.json  test-rc.json
#  kubectl create -f test-rc.json --server=https://k8s.paas.ndp.com:6443 --namespace=test --token=myToken4
F0413 09:26:14.913556   25736 create.go:50] Client error processing command: request [&{Method:POST URL:https://k8s.paas.ndp.com:6443/api/v1beta3/namespaces/test/replicationcontrollers Proto:HTTP/1.1 ProtoMajor:1 ProtoMinor:1 Header:map[] Body:{Reader:} ContentLength:647 TransferEncoding:[] Close:false Host:k8s.paas.ndp.com:6443 Form:map[] PostForm:map[] MultipartForm:<nil> Trailer:map[] RemoteAddr: RequestURI: TLS:<nil>}] failed (403) 403 Forbidden: Forbidden: "/api/v1beta3/namespaces/test/replicationcontrollers"
[it is OK since user "read-only" can not write.]

root@AMZ-IAD-WallE-35-50:~/kubernetes/clusters/aws-us-east-1/projects/test#  kubectl create -f test-rc.json --server=https://k8s.paas.ndp.com:6443 --namespace=test --token=myToken3
hello-world
[Error: user "paas" should not write in namespace "test"]

root@AMZ-IAD-WallE-35-50:~/kubernetes/clusters/aws-us-east-1/projects/test#  kubectl delete -f test-rc.json --server=https://k8s.paas.ndp.com:6443 --namespace=test --token=myToken1
hello-world

[Error: user "fb" should not write in namespace "test"]

Resolved by reading code, it works when change "ns" -> "namespace". However we should change related doc in file https://github.com/GoogleCloudPlatform/kubernetes/blob/master/pkg/auth/authorizer/abac/example_policy_file.jsonl

@deads2k
Sent from my iPhone

On Apr 13, 2015, at 9:59 PM, Gao Zheng<EMAIL_ADDRESS>wrote:
Resolved by reading code, it works when change "ns" -> "namespace". However we should change related doc in file https://github.com/GoogleCloudPlatform/kubernetes/blob/master/pkg/auth/authorizer/abac/example_policy_file.jsonl
—
Reply to this email directly or view it on GitHub.

@derekwaynecarr  This caused a problem because an unknown json tag doesn't produce an error during parsing, it's just silently ignored.  Is there an issue open about unknown json tag handling?  I expect it will get more confusing as we keep moving up API versions.

Not to my knowledge.
Sent from my iPhone

On Apr 14, 2015, at 8:12 AM, David Eads<EMAIL_ADDRESS>wrote:
@derekwaynecarr This caused a problem because an unknown json tag doesn't produce an error during parsing, it's just silently ignored. Is there an issue open about unknown json tag handling? I expect it will get more confusing as we keep moving up API versions.
—
Reply to this email directly or view it on GitHub.

I expect this will go from being a config file to being a REST resource
pretty soon.  At that point, I think our client code will ensure that the
POSTed resource matches the response?
On Tue, Apr 14, 2015 at 7:49 AM, Derek Carr<EMAIL_ADDRESS>wrote:

Not to my knowledge.
Sent from my iPhone

On Apr 14, 2015, at 8:12 AM, David Eads<EMAIL_ADDRESS>wrote:
@derekwaynecarr This caused a problem because an unknown json tag
doesn't produce an error during parsing, it's just silently ignored. Is
there an issue open about unknown json tag handling? I expect it will get
more confusing as we keep moving up API versions.
—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/6752#issuecomment-92888334
.

Server-side validation: #5889

Client-side validation should be possible with --validate


ci: finish release please fix

Many commits and test fixes later, I finally managed to fix the release please bot to use the workspace and publish it to npm.

I also made release-please a bit more resilient so it installs/builds/tests, just as npm-publish.yaml does, before publishing. Since --filter uses that package.json, it actually doesn't run the roots "prepare" script, it uses the package.json prepare script, which doesn't exist. What I noticed in my testing was, that it will still publish, even if ./dist doesn't exist. So it would publish without the types. The additions will safeguard that, since there is no prepare, but just something to note.
(I thought about including a prepare script for that package, but husky is a root dependency and not one for that workspace, so rather than duplicate the dependency and bloat it a bit, I chose to just run all the typical ci suites... Technically, this would be after a persons PR got merged, therefore, already running the test suites, therefore running the test twice. Once before a person's merged, and once after the bots version change gets merged)

Lastly, it should be noted that post 0.2, the README for the ./packages/bootstrap-vue-3 is slimmed down a lot. This will affect the README on npm once a 0.2.x is released. Should we duplicate it to make it look nice for npm, make it different, or just leave it the way it is?

Lastly, it should be noted that post 0.2, the README for the ./packages/bootstrap-vue-3 is slimmed down a lot. This will affect the README on npm once a 0.2.x is released. Should we duplicate it to make it look nice for npm, make it different, or just leave it the way it is?

Yes would be great

Lastly, it should be noted that post 0.2, the README for the ./packages/bootstrap-vue-3 is slimmed down a lot. This will affect the README on npm once a 0.2.x is released. Should we duplicate it to make it look nice for npm, make it different, or just leave it the way it is?

Yes would be great

Which one? 😆

I think it's fine the way it is. People can always click on github repo.


Where is my link ?

after setting up everything shark couldn't generate link and directly took me to the info tab.

just go for new shark


intellisence for node js modules

VSCode Version:the latest
OS Version:ubunto 16.04

assume that I want to harness 'fs' module features.naturally I have a code like:
var fs=require('fs');
then when I want to read a file or define a stream to a file for writing I expect when I type fs a variety of different options would be displayed.but it seems modules are unknown for VSCode and ,if  so,this is an extremely significant failure .
please guide me if there is a solution what's that.this case occurs for child_process and other modules too.

@masoud-moghini : I hope you already installed node module and added reference in your project like below:

Besides using /// references you can define a JS project using a jsconfig.json file and add typings into the project. A doc describing how to do this is available here: https://code.visualstudio.com/docs/languages/javascript


MSID: 91684 Version: 1 DOI: 10.1101/2<IP_ADDRESS>4028

MSID: 91684
Version: 1
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>4028
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=91684
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>4028

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/91684v1
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>4028
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/91684/v1/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

@acollings Looks like another assessment in need of tweaking (I think the important needs to be replaced with another adjective):

Thanks!

Tweaked: thanks @JGilbert-eLife!


Add check that effect value (beta/OR) falls within CI

A user reported an instance where an effect value did not fall within the quoted confidence interval (CI). This is an error as the effect (beta/OR) must fall within the CI quoted.
Implement a check of association data with an error produced if effect does not fall within CI.
Association data is entered into Catalog in 3 ways:

Upload templates in deposition
Manual entry in curation interface
Summary statistics file upload
This QC will need adding in each of these places.

This check should be carried out on all data currently in Catalog but the importance of doing this should be discussed as would need curation resources to follow up on any errors identified.

Add this to future deposition app functionality

Add this to future deposition app functionality


The latest version on Red Hat Enterprise Linux (RHEL)

Thanks for the great article on migration. Are these version mentioned as compatible the latest? The latest recommended AWS RHEL version is 7.6 with general support for 6.7 ending in 2018. Could we still migrate AWS VMs if they are of version RHEL 7.6? Thanks for the support.

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

ID: 4d92f050-73a2-c519-2046-2aba7caeecb0
Version Independent ID: e0fe7de0-ce70-de87-32c7-fe498552f9be
Content: Migrate AWS VMs to Azure with the Azure Site Recovery service
Content Source: articles/site-recovery/migrate-tutorial-aws-azure.md
Service: site-recovery
GitHub Login: @rayne-wiselman
Microsoft Alias: raynew

The article is up-to-date. We'll update with more versions as they're available.
Thanks
#please-close


Manage dependencies

Added conditional imports to manage import errors due potential package absence.
Updated pyproject.toml to move packages to optional packages. Choose Postgres as default db for time being till we have support for sqlite.

Resolves - https://app.zenhub.com/workspaces/dag-authoring-610d4c4494b6c40010873404/issues/astro-projects/astro/82

@dimberman @tatiana  - Do you guys have any suggestion to test this with UT?

@utkarsharma2 it's great to see progress in this stream!

Docs: we probably want to update the docs on how the installation changed, explaining how users installing Astro with pip can benefit from this feature. It's straightforward if we are installing locally, having the pyproject.toml, but it may be worth documenting to users downloading from PyPI using pip install astro-projects[snowflake] etc.

Tests: I agree we need to find a way of testing things before merging this feature.
A strategy could be to mock the value of some_module.__import__ so it raises ModuleNotFoundError, have you tried this?


[Lint] Add a rule to detect and transform [<Type>]() into literal …

…array init
Instead of using an archaic initializer call syntax, let's replace it with
an empty array literal with a type annotation (if there isn't one).

We can do this in expression context as well i.e. reduce([Int]()) { ... } but I'm not sure what to use as a replacement it could be either Array<Int>() or [] as [Int].

This syntax is supported because that's how this operation looked in Objective-C which made sense at the beginning. The language doesn't support uses of type names without .self in expression context elsewhere though, this is a very narrow special case we have to support, so it might be good to nudge people to switch to Swift spelling.

cc @airspeedswift

Sure, if there are compelling reasons to try to nudge folks toward a specific spelling, then I'm happy to have that discussion—I just want to make sure I have all the context first. (That's why making the rule opt-in at first is totally fine.)
I might just be hesitant because it's the syntax I've been using for ages 🙂  But this does introduce an explicit inconsistency for initialization of empty collections, so I want to make sure we're doing the right thing style-wise. For example, using explicit type annotations for type members is probably always a good thing (for readability and for type checker performance), but it feels harder to tell users who usually do var someVar = someExpression for simple local variables that now there's an exception for empty collections where they're expected to use a type annotation or as cast.
So I guess what I'm looking for is a good rationale for that inconsistency, one that isn't just about a special case in the compiler. (Although if you look at the complexity of the implementation of UseShorthandTypeNames, I feel that pain too.)

I think the main motivation here is that it’s inconsistent special case in the surface language which is not supported in any other constructs but this bracket syntax, the spelling of type references in general cannot appear without other an operation or .self.

I am fine making this opt-in though:

@allevato I think I addressed all of the feedback and I've also extended the rule to parameters with default values.

+1 for this PR, var birds: [Bird] = [] is how Apple's code samples initialize empty collections. See: https://developer.apple.com/documentation/swiftui/backyard-birds-sample

@allevato Added tests are requested although arguably we should suggest to replace them with leading-dot syntax, I've made a note of that in the tests.


Pro controller button resets to white on write.

Whenever I try to change a pro controllers button color, it always resets to white. Even if I use one of the stock colors.

Is it possible to use the following (I don't have a Pro controller):
http://desowin.org/usbpcap/
and capture some packets so I can check what's happening?
The procedure is the following:

Install USBPCap and restart
After the restart you will see the usb drivers re-installing them selves. Wait a bit for all of them.
After this is done open JC Toolkit and change the colors and wait. Don't hit "Write Colors" yet!.
Run USBPcapCMD.exe from C:\Program Files\USBPcap
Write the number of the usb hub that includes the port that your BT adapter is and hit enter.
write a name for the file. e.g. test.pcap and hit enter
It will ask you if you want to allow it, select yes.
The usb packet capturing will start. Hit "Write Colors" in the app
After it's done, go to the USBPcap console and press CTRL+C

Send me the .pcap file to my email. You can find it in my GitHub profile.

I sent the email

Thank you for the time you invested!
It helps me a lot to understand the Pro controller behavior.
The colors you tried were
Body:    a6 28 ff
Buttons: 56 cd 46

Correct?
So what I see is a strange behavior.
Your controller replies back that it got the write commands, but the address and size are zeroed..
Also another thing I saw is that it constantly sends button state data. But as I can see, you don't press any button or play with the sticks. So maybe this is normal for Pro controller. Anyway, it's not a big issue though.
Anyway, I'll fully examine your file and compare with one from my  Joy-Con later.

I'm glad I was able to help, yes those were the colors I was setting. If you need any more info, feel free to ask.

Yes, the grey buttons in v2.5.1 for the FF FF FF values was by design.
The algorithm I was using to paint the controller, was producing many unwanted white spots.
You can check the improvement in quality, by opening v2.5.1 and v3.0.1 side by side.
Choose, 254,254,254 for v2.5.1 to bypass my code and check how it looks versus v3.0.1.

Nice to hear! Seams that the mitigation works.
One last request to you, is it possible to capture one more USBPcap file?
Now I need to check the whole procedure:
Open USBPcap before opening the app and start capturing.
Open JC toolkit, choose colors, write them, close USBPcap.

Fixed!


Power Query error on Azure Data Factory in Azure Government (DoD) environment

Are any DoD users getting this error when trying to use PowerQuery on Azure Data Factory in the Azure Government (DoD) environment?
I'm using a csv file in an Azure Blob. I can't validate or publish because of the error.
Error:
UserQuery: The InlineProvider provider cannot be used with this identity.
Also getting this warning:
Currently not all Power Query M functions are supported for data wrangling despite being available during authoring.

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

ID: d5b463b1-36a5-baa8-1085-d72fea129f9c
Version Independent ID: 144bf021-a61e-ee70-4f10-3a9da430b069
Content: Power Query activity in Azure Data Factory - Azure Data Factory
Content Source: articles/data-factory/control-flow-power-query-activity.md
Service: data-factory
Sub-service: data-flows
GitHub Login: @kromerm
Microsoft Alias: makromer

@carlosandrews
Thanks for your feedback! We will investigate and update as appropriate.

Hello @carlosandrews
Please refer this below link for your reference. All transformations should be done on the UserQuery as changes to dataset queries are not supported nor will they be persisted.I have reproduced this issue and successfully validated and published the Power Query.

Currently not all Power Query M functions are supported for data wrangling despite being available during authoring. This is because some functions may not be compatible with the Azure Data Factory environment or may not be necessary for the specific use case.
If there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

@carlosandrews
We are going to close this thread, if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.
#Please-close


Local_auth plugin with MissingPluginException(No implementation found for method authenticateWithBiometrics on channel plugins.flutter.io/local_auth)

Steps to Reproduce

Add local_auth: ^0.3.0 to a working project
Copy the example code from the package. I attach it here, not sure if it helps https://gist.github.com/function1983/42c1c47413c570d08edc5e50ffa98d60
Add <uses-permission android:name="android.permission.USE_FINGERPRINT"/> to Android manifests as instructed
flutter runreturns with error
flutter clean doesn't help
library is imported as shown

Logs

E/flutter (15593): [ERROR:flutter/shell/common/shell.cc(182)] Dart Error: Unhandled exception:
E/flutter (15593): MissingPluginException(No implementation found for method authenticateWithBiometrics on channel plugins.flutter.io/local_auth)
E/flutter (15593): #0      MethodChannel.invokeMethod (package:flutter/src/services/platform_channel.dart:278:7)
E/flutter (15593): <asynchronous suspension>
E/flutter (15593): #1      LocalAuthentication.authenticateWithBiometrics (package:local_auth/local_auth.dart:75:27)
E/flutter (15593): <asynchronous suspension>
E/flutter (15593): #2      _MyAppState._authenticate (file:///media/func/SSD/NewAndroid/myapp/lib/main.dart:65:34)
E/flutter (15593): <asynchronous suspension>
E/flutter (15593): #3      _InkResponseState._handleTap (package:flutter/src/material/ink_well.dart:507:14)
E/flutter (15593): #4      _InkResponseState.build.<anonymous closure> (package:flutter/src/material/ink_well.dart:562:30)
E/flutter (15593): #5      GestureRecognizer.invokeCallback (package:flutter/src/gestures/recognizer.dart:102:24)
E/flutter (15593): #6      TapGestureRecognizer._checkUp (package:flutter/src/gestures/tap.dart:242:9)
E/flutter (15593): #7      TapGestureRecognizer.acceptGesture (package:flutter/src/gestures/tap.dart:204:7)
E/flutter (15593): #8      GestureArenaManager.sweep (package:flutter/src/gestures/arena.dart:156:27)
E/flutter (15593): #9      _WidgetsFlutterBinding&BindingBase&GestureBinding.handleEvent (package:flutter/src/gestures/binding.dart:143:20)
E/flutter (15593): #10     _WidgetsFlutterBinding&BindingBase&GestureBinding.dispatchEvent (package:flutter/src/gestures/binding.dart:117:22)
E/flutter (15593): #11     _WidgetsFlutterBinding&BindingBase&GestureBinding._handlePointerEvent (package:flutter/src/gestures/binding.dart:97:7)
E/flutter (15593): #12     _WidgetsFlutterBinding&BindingBase&GestureBinding._flushPointerEventQueue (package:flutter/src/gestures/binding.dart:60:7)
E/flutter (15593): #13     _WidgetsFlutterBinding&BindingBase&GestureBinding._handlePointerDataPacket (package:flutter/src/gestures/binding.dart:44:7)
E/flutter (15593): #14     _invoke1 (dart:ui/hooks.dart:159:13)
E/flutter (15593): #15     _dispatchPointerDataPacket (dart:ui/hooks.dart:113:5)

flutter doctor -v
[✓] Flutter (Channel master, v0.10.2-pre.25, on Linux, locale en_US.UTF-8)
    • Flutter version 0.10.2-pre.25 at /media/func/SSD/NewAndroid/flutter
    • Framework revision 0edbe726a1 (2 hours ago), 2018-10-22 10:29:00 -0700
    • Engine revision 58cdd53f90
    • Dart version 2.1.0-dev.7.1.flutter-b99bcfd309

[!] Android toolchain - develop for Android devices (Android SDK 28.0.2)
    • Android SDK at /media/func/SSD/AndroidSDK
    • Android NDK at /media/func/SSD/AndroidSDK/ndk-bundle
    • Platform android-28, build-tools 28.0.2
    • ANDROID_HOME = /media/func/SSD/AndroidSDK
    • Java binary at: /media/func/SSD/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
    ✗ Android license status unknown.

[✓] Android Studio (version 3.2)
    • Android Studio at /media/func/SSD/android-studio
    • Flutter plugin version 29.1.1
    • Dart plugin version 181.5656
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)

[!] VS Code (version 1.28.2)
    • VS Code at /usr/share/code
    • Flutter extension not installed; install from
      https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (1 available)
    • Mi A1 • <IP_ADDRESS>:5555 • android-arm64 • Android 8.1.0 (API 27)

! Doctor found issues in 2 categories.

Somewhat similar to #16819

Thanks @zoechi , it was an honest mistake. In my project there are TWO GeneratedPluginRegistrant.class

com.mycompany.GeneratedPluginRegistrant; and
io.flutter.plugins.GeneratedPluginRegistrant

After specifically calling io.flutter.plugins.GeneratedPluginRegistrant.registerWith(this); in MainActivity extends FlutterActivity all is working well now.
Hope it helps other who faces the same problem as I did.
Thank you.

i don't seem to have a .java file and i have a .kt file... what would i do in that case?
Please do help


Editable tables

Feature Request
I'd like to have Editable tables in SUIR. Is that on the roadmap? Or since vanilla SUI doesn't have that feature, SUIR will not either?

Nope, there are no plans for it 😿 It's not supported by SUI core and even in Fomantic, so I can't be done on our side. I tried to use react-table multiple times and was satisfied with results:
https://github.com/tannerlinsley/react-table/tree/v6#codesandbox-examples
But it required a custom styling 💣

Ah, sad.
I know of react-table, but was hoping for out of the box solution with SUI styles..


How difficult would it be to add other sites? Dailymotion, Vimeo, etc.

Hello,
I recently launched a site which uses a modification of your code - here (incidentally, how should I give attribution? Right now I just have a mention at the bottom of the DIY page - is that okay, or would something else be better?)
My question is, how easily could this code be adapted to work for Youtube competitors such as Dailymotion or Vimeo? Dailymotion seems to be the 2nd-biggest video site overall, and I've noticed that there has been a particular trend for some years now among some directors and studios to upload to Vimeo instead of Youtube. I've always wanted to have an option besides Youtube.
I haven't looked much into the particulars of how Youtube External Subtitle works, so... I'd appreciate hearing your perspective.

Hello @Esn024 ,
At first, sorry for the late answer.
For me it is not necessary to mention the original script on your site. But of course if you do, I appreciate it :) I think you site looks really cool!
Well, I think it wouldn't be that hard to support other sites as well.
We only have Youtube specific logic by getting the id of the video from the url, and also by checking the current time position of the playback. These are the only things needed to display the subtitles.
So I would say, that it is relatively easy to implement it, although I'm not planning to support other video sites in this script.

Thanks. :)
"We only have Youtube specific logic by getting the id of the video from the url, and also by checking the current time position of the playback. These are the only things needed to display the subtitles."
Could you please point out the lines in the code where those things are done? I'm trying to understand enough of how it works to make the necessary changes myself. There's a lot here I don't get yet, but every little bit would help.
There seem to be quite a lot of lines in the code that have to do with Youtube player API functions and for which I'd need to write alternative logic, so I'm just trying to identify all of them...
Also, does "getVideoData()" work? It's not mentioned in the Youtube player API reference, and a search online says that it has been removed...

Also, I was wondering, what was the reason that the file structure was changed in Mar 30 of last year, and the one .js file separated out into multiple .ts files?


Error when changing time range dropdown using Internet Explorer

Is ok in Chrome and Safai.

This occurs in IE 11 when changing the dropdown or selecting the tickbox.  Chrome, Safari and Edge all unaffected.

Looks to be a bug in Google Chart API v 42: https://github.com/google/google-visualization-issues/issues/2084
Using a frozen version V43, it is resolved.  Estimated release date Oct 29.


feat: update credentials.yml files with unipipe-cli

We can extend the feature a little bit more before the merge (maybe also adding browse cmd support as well). I opened this PR for the discussions and another contributions. It updates the credentials.yml file and changes the status file with a single command. we can also separate credential commands all together and add a new sub-command for it.
e.g adding multiple credentials: unipipe update -i INSTANCE_ID -b BINDING_ID -c "username: doruk" -c "accesskey: test" -c "secretkey: ***" --status succeeded  REPO_PATH

I have added the browse command support as well. I have extended browse -> binding -> update command. There is no delete credentials option and probably we don't need it but it is possible to add one.


Use page cache for image and delta layers

Image and delta layers don't use the page cache, every read operation results in a read() system call. That makes compaction pretty slow as it reads all the existing versions from a delta layer and writes it back out. That is currently slow and shows up in 'perf' profile. (The code in 'main' is missing that too, but it's not as obviously bad there because we don't do anything like the "compaction" there)
Also, they both have an 'index' HashMap that is fully loaded into memory on first access. Should refactor that into a page-oriented data structure and use the page cache for that too.

I do not think that we really need cache here, because delta and image layers are not expected to be accessed frequently. So probability of cache hit will be close to zero.
What we need here is buffered IO. Right now delta_layer is using BufWriter for ChapterWriter. I wonder if we also should use BufReader for Book:
        let book = Book::new(BufReader::new(file))?;
        let chapter = book.chapter_reader(VALUES_CHAPTER)?;

Right now it is not possible because of unsatisfied trait bounds:
Book<std::io::BufReader<std::fs::File>>
But you have said that we can change Eric's bookfile. Can we just change
impl Book<File> to impl<R> Book<R> where R: Read,Seek
and use BufReader ? Does it solve the problem with performance?
It seems to be necessry only for DeltaLayer, not for ImageLayers, because 8k pages is better to read without extra copies.

Looking more at chapter_reader I realized that it can't be used with any buffered reader because it returns constant reference and any buffered reader by definition has to store some state.
So I have implemented different approach: see https://github.com/zenithdb/zenith/pull/1369

I do not think that we really need cache here, because delta and image layers are not expected to be accessed frequently. So probability of cache hit will be close to zero.

Why do you think they would not be accessed frequently? Only the last 1 GB of updated pages sit in the in-memory layer, all other accesses are to image and delta layers. Did you perhaps mean that the accesses would be very random, so that the page cache won't help much? That may be true. But even then, I doubt it would be completely random. And the index pages with the offsets to within the file are surely more frequently accessed than any actual data page, so those should be cached. Currently they are cached by slurping he index completely into memory, but that's very inflexible, and we don't have any good scheme for releasing memory. So at least that should be accessed through the page cache.
Another idea would be to mmmap() the files and let the OS handle the caching. That could be very fast. However, I'm a bit scared of mmapping. A single read error would crash the whole process. And we might run into limitations with the number of mmaps etc.

Why do you think they would not be accessed frequently?

It is just based on the assumption that size of shared-buffers in compute node is (should?) comparable with page cache size. It means that if working set fits in compute memory, then this page will not be requested from page server at all. And if it doesn't fit in compute memory and we have random access pattern, then probability of page cache hit is very small. If page was evicted from shared buffers at compute node, then most likely it was also evicted from page cache in pageserver which is shared by multiple tenants.


Please add my1login.

We are a cloud based password manager and we support 2FA

Closing this issue since it's been months and there's no update. @ljackman feel free to reopen if you'd like.


update(logs): restricting the duplicate logs

Signed-off-by: shubhamchaudhary<EMAIL_ADDRESS>

##Logs
INFO[2021-01-29T18:18:16+05:30] Beginning to serve on port :8081
INFO[2021-01-29T18:18:16+05:30] Started creating Metrics
INFO[2021-01-29T18:18:16+05:30] Registering Fixed Metrics
WARNING[2021-01-29T18:18:17+05:30] No chaosengine found with monitoring enabled
INFO[2021-01-29T18:18:17+05:30] [Wait]: Waiting for the chaosengine with monitoring enabled ...
INFO[2021-01-29T18:18:37+05:30] [Wait]: Cheers! Wait is over, found desired chaosengine
WARNING[2021-01-29T18:18:37+05:30] No chaosresult found!
INFO[2021-01-29T18:18:37+05:30] [Wait]: Waiting for the chaosresult ...
INFO[2021-01-29T18:18:42+05:30] [Wait]: Cheers! Wait is over, found desired chaosresult
INFO[2021-01-29T18:18:43+05:30] The chaos metrics are as follows              TotalDuration=0 ResultNamespace=default EndTime=0 ChaosInjectTime=0 PassedExperiments=0 ResultName=helloservice-pod-delete-pod-delete FailedExperiments=0 AwaitedExperiments=1 ProbeSuccessPercentage=0 StartTime=1.611924518e+09
INFO[2021-01-29T18:18:46+05:30] The chaos metrics are as follows              ResultNamespace=default PassedExperiments=0 StartTime=1.611924518e+09 ProbeSuccessPercentage=0 EndTime=0 ChaosInjectTime=0 ResultName=helloservice-pod-delete-pod-delete TotalDuration=0 FailedExperiments=0 AwaitedExperiments=1

##Logs
INFO[2021-01-29T18:18:16+05:30] Beginning to serve on port :8081
INFO[2021-01-29T18:18:16+05:30] Started creating Metrics
INFO[2021-01-29T18:18:16+05:30] Registering Fixed Metrics
WARNING[2021-01-29T18:18:17+05:30] No chaosengine found with monitoring enabled
INFO[2021-01-29T18:18:17+05:30] [Wait]: Waiting for the chaosengine with monitoring enabled ...
INFO[2021-01-29T18:18:37+05:30] [Wait]: Cheers! Wait is over, found desired chaosengine
WARNING[2021-01-29T18:18:37+05:30] No chaosresult found!
INFO[2021-01-29T18:18:37+05:30] [Wait]: Waiting for the chaosresult ...
INFO[2021-01-29T18:18:42+05:30] [Wait]: Cheers! Wait is over, found desired chaosresult
INFO[2021-01-29T18:18:43+05:30] The chaos metrics are as follows              TotalDuration=0 ResultNamespace=default EndTime=0 ChaosInjectTime=0 PassedExperiments=0 ResultName=helloservice-pod-delete-pod-delete FailedExperiments=0 AwaitedExperiments=1 ProbeSuccessPercentage=0 StartTime=1.611924518e+09
INFO[2021-01-29T18:18:46+05:30] The chaos metrics are as follows              ResultNamespace=default PassedExperiments=0 StartTime=1.611924518e+09 ProbeSuccessPercentage=0 EndTime=0 ChaosInjectTime=0 ResultName=helloservice-pod-delete-pod-delete TotalDuration=0 FailedExperiments=0 AwaitedExperiments=1


a11y: change searchBox to ComboBox in filter dialog

Description
test in narrator and nvda
Previously, the reader would say 'Filter dialog. Filter dialog'
Now, the reader says 'Filter dialog, Select or type a dialog'
change searchBox to Combobox

Task Item
refs #2054
refs #2055

Screenshots

@DesignPolice could you take a look?

Hi @corinagum and @lei9444
Ya looks like this one slipped into the production path - @mareekuh didn't want to swap to a ComboBox on this one as it does not match expected interaction pattern for either combo or filter control.
We have an alternative in mind to use the Global Search with some limitations to search only in the app.
The new figma design flow is here:
https://www.figma.com/file/LlTlh5vXwq91zjGnFtrUUR6l/Composer-master-design-spec-UI-library?node-id=1123%3A20058
Happy to start a conversation on how best or when best to implement this, in regard to the a11y timeframes

Thanks a bunch

Thanks DesignPolice. it looks like this will need further discussion.
@lei9444, I think your previous PR was very close to a working intermediary fix that we can use until the decision on dialog filter has been completed. If I recall correctly, I believe you changed the ariaLabel from 'Filter dialog' to 'Search box', but since there were repeats, maybe you can remove the ariaLabel and see if that produces 'Search box. Filter dialog' instead. What do you think? Let me know if you have any questions.
@mareekuh and @cwhitten I think the above paragraph should be sufficient to fix this a11y bug without changing surface area until the UX change (if any) is finalized. During design, I recommend involving the a11y audit team so they can chime in before the spec is finalized.

@cwhitten - Apologies for the misunderstanding. The design to address the accessibility issues and keep the desired behavior is located here: https://www.figma.com/file/LlTlh5vXwq91zjGnFtrUUR6l/Composer-master-design-spec-%2F-UI-library?node-id=1123%3A19432. It's been up there for a while but we have had areas of higher priority to review in our meetings (publish, skills manifest etc).
Expected look & feel + behavior:

the search box is located at the top of the frame, centered in the header/chrome.
the scope of the search/filter is set to the current view, in this case the design view/dialog list. Since we do not yet have a larger scale search enabled, we can use the 'search box with custom icon' and use the filter instead of the search icon + include the following as placeholder text "Filter dialog and trigger list"

More information about the Fluent control can be found here:
(https://developer.microsoft.com/en-us/fluentui#/controls/web/searchbox).
We cannot use a combo-box for this functionality. I won't get us the behavior intended and sets the wrong expectations to our users about how to use this.
Sooner rather than later we'll want to enable search in Composer, especially when the Bot responses and User input pages will get more content which quickly is the case for more substantial bots (vs samples). We don't want to have 2 search boxes in different locations so we'll have to replace the current filter box with the regular Fluent Search control in the near future.
Please let me know if you have any questions.
Marieke
Regarding the combo box, this is the prescribed functionality/behavior:
Combo box :
A ComboBox is a list in which the selected item is always visible, and the others are visible on demand by clicking a drop-down button or by typing in the input (unless allowFreeform and autoComplete are both false). They are used to simplify the design and make a choice within the UI. When closed, only the selected item is visible. When users click the drop-down button, all the options become visible. To change the value, users open the list and click another value or use the arrow keys (up and down) to select a new value. When collapsed if autoComplete and/or allowFreeform are true, the user can select a new value by typing.

@mareekuh we're not going to implement global search to address the filter box accessibility issues. @corinagum 's guidance is right. global search needs more than some static screens and placeholder data before we build it.

I am not asking for a global search box.
I am asking for placing a search box with filter icon and control in the place prescribed by Fluent.
Same behavior, same scope, same look and feel different location (see below). But a standard control that is accessible and meets fluent guidelines. A combo box does not filter.

Oh, sorry I'm confused. We use  today in this control.
<SearchBox
placeholder={formatMessage('Filter Dialogs')}
styles={searchBox}
onChange={onFilter}
iconProps={{ iconName: 'Filter' }}
/>
On Wed, Apr 15, 2020 at 2:01 PM mareekuh<EMAIL_ADDRESS>wrote:

I am not asking for a global search box.
I am asking for placing a search box with filter icon and control in the
place prescribed by Fluent.
Same behavior, same scope, same look and feel different location (see
below). But a standard control that is accessible and meets fluent
guidelines. A combo box does not filter.
[image: Screen Shot 2020-04-15 at 1 39 55 PM]
https://user-images.githubusercontent.com/48503434/79388058-8b8ad000-7f21-11ea-8a41-3524a5bbb417.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/BotFramework-Composer/pull/2653#issuecomment-614277944,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAI2MYG2PDFXL3A4W7SIARDRMYODLANCNFSM4MIIY73A
.

Hi, @cwhitten @mareekuh @sangwoohaan @corinagum, I have removed the combobox and reverted the code. I have add a aria-label for reader.
I use narrator and NVDA to test. It works and says 'Type dialog name, Filter Dialog' only once now

Thanks @lei9444 - appreciate all the help.

Hi @corinagum #2055  can't be fixed in this PR, could you help tp move it to Todo again. I will fix it in next PR


Minimum width and height in pixels

In my app, I'm not supposed to crop into too small a picture, because the cropped image will then be rejected by the server I upload to.
In my particular case, we have a minimum width and a minimum height. It's even more complicated, the dimensions are ok if width >= minimumWidth OR height >= minimumHeight.
How could we implement those limits in crop_image?
Perhaps we should add a method to the controller, null by default, that checks if the dimensions are OK and recomputes the new dimensions if needed.
A method to be implemented by the app developer if needed in each specific case, for more flexibility.
cf. https://github.com/openfoodfacts/smooth-app/issues/4219

Yes, an additional field (maybe not a method, just a field) could be good.


Prepare for glib::gstring::GString

Part of https://github.com/gtk-rs/glib/pull/389.
cc @GuillaumeGomez, @sdroege, @philn

This not ready for merge despite CI passed,
as also need fix for https://github.com/gtk-rs/glib/pull/389#issuecomment-445122868

Trampoline problem fixed, no need changes in examples.
This PR IMHO ready for merge

Then I merge. Thanks!


Refactor parallel for DC

Signed-off-by: shiyu22<EMAIL_ADDRESS>

Codecov Report

Merging #837 (a264c43) into main (5090e08) will increase coverage by 0.02%.
The diff coverage is 97.77%.

@@            Coverage Diff             @@
##             main     #837      +/-   ##
==========================================
+ Coverage   70.94%   70.97%   +0.02%
==========================================
  Files         185      185
  Lines        8182     8186       +4
  Branches     1339     1338       -1
==========================================
+ Hits         5805     5810       +5
- Misses       1979     1980       +1
+ Partials      398      396       -2

Impacted Files
Coverage Δ

towhee/functional/mixins/parallel.py
96.19% <97.05%> (+1.85%)
:arrow_up:

towhee/functional/data_collection.py
94.05% <100.00%> (+0.16%)
:arrow_up:

towhee/functional/option.py
77.77% <100.00%> (ø)

towhee/hub/repo_manager.py
80.18% <0.00%> (-0.91%)
:arrow_down:

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 5090e08...a264c43. Read the comment docs.

/lgtm
/approve


As a Project Coordinator, I want quarterly and annual report to be automatically generated, so that I don't have to go through the effort or make a mistake when inputting these due dates.

Description:
Quarterly & annual reports are always due at the same time, regardless of project, and could be calculated based upon contract date & end of Target Emissions Intensity measurement period.
Note: this end date comes from usability testing sessions with Bailey and Amanda.
Acceptance Criteria:
Given that a CIF admin has created a project
When they enter start & end dates
Then quarterly reports will automatically be created on the following January 5, April 5, July 5, and October 5 until the end of the project.
Given that a CIF admin has created a project
When the contract date is within 44 days of the next quarterly report date
Then the first quarterly report will not be the most immediate quarterly date, it will be the following.
Development Checklist:

[ ] Checklist item
[ ] Checklist item
[ ] Checklist item
[ ] Meets the DOD

Definition of Ready (Note: If any of these points are not applicable, mark N/A)

[ ] User story is included
[ ] User role and type are identified
[ ] Acceptance criteria are included
[ ] Wireframes are included (if required)
[ ] Design / Solution is accepted by Product Owner
[ ] Dependencies are identified (technical, business, regulatory/policy)
[ ] Story has been estimated (under 13 pts)

·Definition of Done (Note: If any of these points are not applicable, mark N/A)

[ ] Acceptance criteria are tested by the CI pipeline
[ ] UI meets accessibility requirements
[ ] Configuration changes are documented, documentation and designs are updated
[ ] Passes code peer-review
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Ticket is ready to be merged to main branch
[ ] Can be demoed in Sprint Review
[ ] Bugs or future work cards are identified and created
[ ] Reviewed and approved by Product Owner

Notes:

3 Amigos meeting with Taylor and Amanda: The due date of the 1st annual report is tied to the TEIMP report, and can be pushed back.

@suhafa @nanyangpro - I've slightly updated the ticket to reflect research and design.

Design covered in wireframe (with all possible cases such as edit, missing information, deletion etc)

Thank you so much for all your dedication and effort on this one @suhafa!  This was so much more complicated than I expected, but I know this will significantly help Amanda and the CIF team!


Add support for python magic

Now that we have the rust backend, lets make magic happen for ipython/jupyter!

The challenge here will be handling connections to different database types. fugue is probably the best bet for operating on in memory data frames or csv files, as we can route out to dask or spark and get cracking fast. But it can't connect to existing database files (someone call me on this if I'm wrong). DuckDB is brilliant, but only natively supports duckdb instances. There are extensions to convert other types (sqlite, mysql, etc) to duckdb, but none are supported in the Python API. There is an sqlalchemy driver for duckdb though.
This is all a really long way of saying that I think we should probably check the connection input. If it's an in-memory dataframe or a csv file, use fugue[spark,sql] to run the compiled sql query. If it's anything else, including duckdb, just try connecting with sqlalchemy and using their methods. What are everyone's thoughts?

Great idea!
sqlalchemy sounds like a good plan to allow flexibility.
How do we envisage passing things like connection parameters?
Are there any popular magics out there for using SQL? Potentially we could piggyback on those. IIRC bigquery had a bigquery-specific one (but their connection model is a bit different from traditional DBs).
What would you see the return type being? A pandas dataframe?

Wthl the ipython/jupyter cell magic, everything following %%prql but on the same line as it is passed as a string. Conceivably, additional arguments beyond connections strings could be parsed out that way. There is a whole setup that basically mimics argparse for use with magics, but I'd lean away from that where possible. These magics work best the simpler they are.
There are definitely several magics existing. There's an %%sql magic from the python-sql package. If we were to build on one, that's probably the best. You pass a connection string, then an sql query, and hey presto, results. It's already got expansive functionality and, since it uses sqlalchemy under the hood, its supports any format supported by them, including duckdb. They natively support everything we'd want, except reading from csv's. Would just need to see about adding our parsing into it...
fugue also have a magic %%fsql which is brilliant for handling sql with in memory pandas data frames. What makes this particularly challenging is namespace limitations. The fsql query expects the table name to be the name of the in-memory dataframe. But it actually takes a lot of work to safely move that between scopes in python.
My inclination is to prefer pandas data frames as the output (supported by both plugins above). Then, people can either panda's their data, or do more queries.

Those look great! Having the equivalent of %%sql as %%prql would be really good — definitely the best way for people to use prql so far.
And possibly this isn't too difficult™ if we take python-sql as a dependency and pass most args along, though it's rarely that easy :)

That's what I'm investigating today. The first two arguments of every python cell magic are line and cell. The first is a string of everything on the line, the second is a string of everything else. If we can write a decorator that just wraps their function and compiles prql to sql where appropriate, it'd be rather straightforward. (Famous last words, I know!)

It was, dare I say it, more or less that simple. Branch fix043 should have a minimal working magic that you can activate by calling import pyprql.magic in either Python or Jupiter. It's still super rough - currently, additional args can't be passed as line magic, which breaks some of the functionality (example below)
Just a word on features as well - their set is all we need, I think. If you have a dataframe in memory and an established connection to a database, then you can add that dataframe to the database with %prql --persist df. Since they wrap sqlalchemy, if we ship with the sqlalchemy duckdb extension, then behold in memory querying of data frames! ...Once I get arguments on line magics sorted :p

That sounds great!
How are you getting it working? I tried in Jupyter & IPython but it didn't immediately work. I can look more if helpful but figure I'm just doing something wrong:

In [1]: import pyprql.magic

In [2]: %prql
UsageError: Line magic function `%prql` not found.

In [3]: %%prql
   ...: derive x: 3
   ...:
   ...:
UsageError: Cell magic `%%prql` not found.

No prob at all. I just pulled the latest.
I'm confident it's me doing something wrong — I seem to recall issues installing jupyter extensions in the past.
What are your install steps? I'm doing:
 ~/w/PyPrq
❯ git rev-parse HEAD
2401f0190b136b6ddb77c48159c4064b3edbe377

 ~/w/PyPrql
❯ pip install -e .

and then
❯ ipython

In [1]: %%prql
   ...: derive x = 3
   ...:
UsageError: Cell magic `%%prql` not found.

In [2]: %prql derive x = 3
UsageError: Line magic function `%prql` not found.

Sorry the absence! Back at work on this now. There should now be (roughly) functional. Pair of caveats: 1) the line magic %prql can currently only be used for arguments and connection strings. 2) The cell magic only returns results if you store to a local variable. So, assuming you have an in-memory dataframe named data, then the following will let use the magic:
import pyprql.magic

%prql duckdb:///:memory:

%prql --persist data

%%prql results <<
from data
filter freg > 100
select [ food_name ]

prints(results)

Few more comments on in-progress bits...we shouldn't need to pass the cell magic to a local variable to see the results (that's what results << is doing in the cell magic). But for reasons I haven't figured out, leaving the local variable out returns nothing, regardless of the query. Also, the original %sql line magic supports single line queries. However, the parsing to determine whether the line is a connection string, options, or a query is extensive, and we'd need to implement most of it as we'd need to identify where the prql query is in order to parse it to SQL.

What's confusing me about the cell magic is that manually running the query with %%sql returns results without local assignment, just as I'd expect. Not sure why that disappears in our wrapper...


Unable to extract player params: VKontakte video download bug

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.05.07. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2017.05.07

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
> $ python youtube-dl --version
2017.05.07
> $ python youtube-dl --username <username> --password <password> --verbose "https://vk.com/video-33209565_456239032"
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--username', 'PRIVATE', '--password', 'PRIVATE', '--verbose', 'https://vk.com/video-33209565_456239032']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.05.07
[debug] Python version 3.6.1 - Linux-4.10.13-1-ARCH-x86_64-with-arch
[debug] exe versions: ffmpeg 3.3, ffprobe 3.3, rtmpdump 2.4
[debug] Proxy map: {}
[vk] Downloading login page
[vk] Logging in as<EMAIL_ADDRESS>[vk] -33209565_456239032: Downloading webpage
ERROR: Unable to extract player params; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "youtube-dl/youtube_dl/YoutubeDL.py", line 760, in extract_info
    ie_result = ie.extract(url)
  File "youtube-dl/youtube_dl/extractor/common.py", line 433, in extract
    ie_result = self._real_extract(url)
  File "youtube-dl/youtube_dl/extractor/vk.py", line 401, in _real_extract
    'player params'),
  File "youtube-dl/youtube_dl/extractor/common.py", line 782, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract player params; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

You need to be a member of this group to view its video files.

@dstftw reopen please.
I am a member of this group and I can download some other videos from it.


Resizing shapes distorts them

Sorry, I don't know how to describe this – please update the issue title/description as appropriate.

I have also noticed this.
It seems to affect library and user created icons/groupings.

It was happening before, though not with a lot of elements — but it may may have gotten worse. I'll try to take a look soon.
Closing in favor of https://github.com/excalidraw/excalidraw/issues/4001


TNTPrimeEvent does not always fire in every situation

The TNTPrime event fires if you ignite it with a lighter 99.7% of the time, increase the radius. It also does not fire at all with arrows, or any other way of igniting.
Current Ways to Fire it

Flint and Steel
Other TNT

Needed ways to fire it

Fire Catch
Flaming Arrow
Redstone Current

Fixed in https://github.com/cyberpwnn/Phantom/releases/tag/W4


Implement method of exchanging base hand animations

This PR does some cleanup on the hand models and adds a new feature where you can change the hand open and hand close animations that drive the hand system.
Cleanup
The asset folder in the plugin was meant to just hold assets (model files) and we're about to add a bunch more in there for other features. Having the player body, poke and hand scenes in there makes it messy so those have all been moved into a player folder. For now I've added a few inherited scenes here to prevent breakage but I'm thinking about removing those.
Hand model
Blender was being very annoying since the original blend file was made in a much older version of Blender. I've updated the blend file to Blender 3.1 and re-exported the GLB. For some reason animations created with the newer blender were incompatible with the older files. I also have a suspicion that whatever position the hands last ended up in in Blender, becomes the rest pose, which is disastrous.
Bring on Godot 4 and the ability to pose skeletons within the editor and create the animation files in Godot itself.
Finally note the hidden feature nodes/use_legacy_names=true in the .import file, without that the names are used as they are in the blend file.
Seeing I'm doing big changes already I'm tempted to remove this from the import file and create all anim files with the new names, and rename the names in the blend file to what OpenXR expects. That said, this would require a duplication of everything to create a right hand model instead of our current solution of reusing the left hand and mirroring it.
Open and Closed hand animations
So the trick we've always been using is to blend between an open and closed hand animation depending on trigger and grip inputs. There is now an enhancement to this where you can select different animations as your open and closed hand.
Our defaults in our plugin are:

Then in our demo we have a few extra animations:

The XRToolsFunctionPickup knows the controller, and could probably find the XRToolsHand under it.
The XRToolsFuncrionPickup could be expanded to look for open/closed animations attached to the packable (or its PickupCenter) and apply them to the hand when it becomes the closest.

Pretty much what @Malcolmnixon says, though I'm not sure how much we should do automatically and how much we need to leave up to the game developer as we can't really predict the many ways this could be used.
In the end, when an object is picked up you can react to that and assign the correct animation, just need a nice way to reset it when you let go

,...just need a nice way to reset it when you let go
From my trys previously with using blendvalues it resets just fine, no fuzz there but then again i was using animationtree blendvalues.
If we could save put the current blendvalues from a animationtree as .anim files that would give us rly Something, as so far lookin at the above u still need to make the animation pose, probably in blender and am gonna lean out the window here and say
Yes we don't need to do evrything for the potential dev User but why shouldnt we make the Best choice when godot xr could benefit from this as a xr dev platform. Making things easier = makin godot xr more attractive and competetive

@DigitalN8m4r3 indeed, the reset is just about determining the condition that you're now no longer holding something and thus need to return to the original pose. When every possible pose is in the blend tree  like in your solution, that is easy. In my solution it's easy to detect that you're no longer holding something but you've lost the pose that was used before you changed it to the pose relating to the object you are holding.
So some more thought needs to go into this and I suspect that different games will require different solutions. Right now this PR limits itself to making it possible to swapping the poses out.
As for creating the poses, fully agree. Blender last night was a nightmare, while creating the poses themselves is straight forward, the UI around dope sheets is very confusing and it is very easy to make mistakes there doesn't seem to be an easy way to come back from.
Also I noticed that Blender seems to save GLTF using whatever current pose the hand is in as the rest pose, and if the rest pose doesn't match up with the one used to create the original hand we're using, the new animations are not compatible.
Finally there is the issue with legacy names. Godot used to convert the names of the bones to camel case and that is what we're using.
Turning the legacy conversion off all of a sudden we have the correct bone names, but it seems to break the blend tree logic. I couldn't figure this out so I left them on legacy.
In Godot 3 there is no viable way to create the animations by posing the hand, but in Godot 4 you can!

i dont feel different games/different needs.
From my pov it is limiting yourself and the enduser to use animations and havin to go thru loops such as makin those with Blender or savin those out using handtracking.
At the end this pr is about what the title says and so there is no need to downtalk this, thx for your work and it is a good start.
On a sidenote, your code is already a start when it comes to using a animationtree instead...
all that needs to be done is
Change from animation pose to animation tree, force the openfield to use a tree that got no blend values changed and the closed to use the animationtree from the pickable if present and if not fallback
Now this should happen on the fly as in loading in the resource and replacing with the current animationtree
am sure thats possible, just cant wrap my head around connecting this and loading it dynamicly

I'm part-way through a hand-pose helper tool which attempts to build a pose-animation by recording the angles from the Quest 2 hand-tracker.
Currently it gives severely broken fingers, but I think that's just a rotation by 90 degrees issue. If it works ill polish the tool and put it up, but I'll also assemble a moderate collection of pre-made poses.

I used the Godot 3 animation editor (https://shmolyneaux.itch.io/godot-3-skeletal-animation-add-on) recommended by @DigitalN8m4r3 to create a set of animation files and tried to set them; however it looks like there some serious cross-talk between hands and scenes - its like the script is editing the same global AnimationPlayer and AnimationTree. I think there's shared resources involved here.
The following is some of the hand poses to play with: hand poses.zip

I used the Godot 3 animation editor (https://shmolyneaux.itch.io/godot-3-skeletal-animation-add-on) recommended by @DigitalN8m4r3 to create a set of animation files and tried to set them; however it looks like there some serious cross-talk between hands and scenes - its like the script is editing the same global AnimationPlayer and AnimationTree. I think there's shared resources involved here.

Sounds like we're missing a "resource is local to scene" tickbox somewhere. Godot indeed has a habit to share resources amongst instances of scripts meaning that if you modify one, it modifies for all instances (as it's the same thing used).
Good behavior most of the times, a PITA when you don't :)

@Malcolmnixon looks like that local to scene ensures the resource is made unique, but when the resource contains subresources, those are not made unique.
I've added a deep copy of the blend tree, its only a little bit of data, and it seems to fix the issue.

Changed this back to draft as I have some rework to do after the new hand models were introduced. Will get to that probably later in the week.

Rebased this so the logic is now based on the new hand models. As the old hand models aren't compatible with the new ones I figured it's best to remove them and prevent confusion around why anim files can't be mixed and matched.
The open/close logic has now been adapted for the new hand logic.

Works well, gonna submit a pr to the 4.0dev branch with the new hands and then this can be ported to 4.0 as well

I think you might want to make the XRToolsHand a tool script - everything looks compatible and I checked it out. It nicely allows the hand poses (and material changes) to be shown live. That will also help my next pull request with the grab-point tool script to show the hand in the correct pose.

Yeah when you extend a tool script, the new script must be a tool script as well for things to keep working.

@Malcolmnixon indeed making it a tool script worked well, just needed to disable the process so we're not trying to handle controller input. So back to review :)


Fix log removal

The line for log removal quoted the paths, which disabled the wildcard. Turns out no file will be removed. Also it may be better to make the removal verbose to let user know what are being removed.

:1: LGTM

Thanks :D


Feature Request: Embedded artwork

Just a feature request for the future. Could embedded artwork support be added.
Absolutely no rush on this. Just figured someone will want it eventually. I have no need for it.

If someone wants to do this I'm open for a PR!


GithubPages

Hi,
Your theme is awesome,and I want to use this theme to github pages or netlify platform.

:)


VSCode Hover suggestions on Langlib method invocation needs to be improved with bound type

Description:
Right now it shows actual type.  e.g:
string[] a = [];
a.push();   // <- Suggestions say any|error, but actually it should be a string.

Had an offline chat regarding this and decided to go ahead and try to implement this for at least for the straight forward cases such as the above.


Fix aliases using MODULE

This fixes an error that occurs when formatting a module with an alias on its submodules:
defmodule Test do
  alias __MODULE_.{A, B}
end

Sure, recreated as #33 to make the check pass.


Properly fix UI freezing when comparing files

I have seen that the call to file comparison when evaluating category was changed to a Task.Run. Currently it is implemented as fire and forget which seems unwise.
I have changed the CategoryProvider interface and the implementation in Cairo Desktop (see https://github.com/kevinboss/cairoshell/tree/categoryprovider-async) to use Async/Await which allows us to not fire and forget the category detection by file comparison but still reap the benefit of a responsive UI.

Using the patches from here and your cairoshell branch, the end result seems to be running synchronously for me--the UI locks up while executing the task in GetCategoryDisplayNameByFileEqualityAsync. Am I missing something by chance?

Using the patches from here and your cairoshell branch, the end result seems to be running synchronously for me--the UI locks up while executing the task in GetCategoryDisplayNameByFileEqualityAsync. Am I missing something by chance?

No you are correct in certain scenarios it is still synchronous.
I'll have another look at it, the current solution with Thread.Run fails to correctly evaluate the category sometimes, at least for me, so a solution is needed.
The main issue I see atm is that the ShowInTaskbar getter evaluates the category which isn't ideal since it is an unexpected side effect and makes the code hard to understand and forces it to run synchronously.
Do you have a reliable way to reproduce the UI locking up? I haven't noticed so far.


Redirect for downloads

Today, package content downloads consume a thread for the duration of the download. This will not scale well. Instead, download endpoints should redirect to Azure Blob Storage/CDN.

Ideally necessary for #136


Second splice call on arrays fails with "TypeError: map is undefined"

I recently switched to 0.7.2 and when splicing arrays bound to rivet views i get "TypeError: map is undefined" exception.
Here is a jsfiddle that demonstrates the issue http://jsfiddle.net/652qtjdp/1/

I got the same issue here :(

I think this is related to https://github.com/mikeric/rivets/issues/441

Thanks guys. This issue should be fixed in 0.8.0 (just reload that JSFiddle example above).


Google Cloud Storage

I have the problem when try to connect google storage: this is the error
TooManyAuthHandlerReadyToAuthenticate at /admin/images/
2 AuthHandlers ['ComputeAuth', 'HmacAuthV1Handler'] ready to authenticate for requested_capability ['s3'], only 1 expected. This happens if you import multiple pluging.Plugin implementations that declare support for the requested_capability.
I add settings.py
DEFAULT_FILE_STORAGE = 'django_google_storage.storage.GoogleStorage'
GS_ACCESS_KEY_ID = 'key'
GS_SECRET_ACCESS_KEY = "key"
GS_STORAGE_BUCKET_NAME = 'bucketname'
I don't know what is the problem.
Thanks!

Hi @ciloeweb - please use the Google group for support requests.


Quantconnect missing references

Hi James,
I am struggling to build and would really appreciate your help.
I have cloned both lean and this project as mentioned but I am still having missing reference errors.
And when I try and add manual references to libs such as for Quantconnect.config I am unable to do so. I think its because of .Net framework incompatibility errors (not Standard).
I tried your previous version several years ago and was really happy to find this updated project.
Thanks for your thoughts.
Fig

Fixed


fix for bug-3322: error when configured with environment variables

@victordibia
Why are these changes needed?

Configuring for Azure OpenAI with the environment variable AZURE_OPENAI_API_KEY throws ValueError

"api_key is not present in llm_config or OPENAI_API_KEY env variable for agent..."

This is due to logic that checks for OPENAI_API_KEY but doesn't include a check for the AZURE_OPENAI_KEY configuration.
https://github.com/microsoft/autogen/blob/autogenstudio/samples/apps/autogen-studio/autogenstudio/workflowmanager.py#L239-L242
Completely removing the check block allows support of the AZURE_OPENAI_API_KEY configuration and improves the error message

OpenAIError: Missing credentials. Please pass one of api_key, azure_ad_token, azure_ad_token_provider, or the AZURE_OPENAI_API_KEY or AZURE_OPENAI_AD_TOKEN environment variables.

Related issue number
Closes #3322

Checks

[x] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[x] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[x] I've made sure all auto checks have passed.

Instead of removing the check, we should add validation logic for AZURE_OPENAI_API_KEY.

Isnt the logic already handled in the test_model which is catching the error that is bubbling up from OpenAIError thrown in the OpenAIWrapper? We can see this in the error message when the AZURE_OPENAI_API_KEY is not set:

When the AZURE_OPENAI_API_KEY is set but the AZURE_OPENAI_API_BASE is not set:

And when the AZURE_OPENAI_API_KEY and AZURE_OPENAI_API_BASE is set but the DEFAULT_AZURE_API_VERSION is not:


reading error?

Hi, I have tried your code and I have some problems.
the wiegand reading does not match the usb reading.
modify it to read 34 bits.
`
while(1) {
n = epoll_wait(epfd, events, 10, 15);
	for ( i = 0;  i < n; ++i) {
		value <<= 1;
		if (events[i].data.fd == ev_d1.data.fd) {
			value |= 0x01;
		}

		bit_counter += 1;
	}

	**if (bit_counter == 33){**

		printf("+++++++++++en  %d = 0x%x (%d bits)\n", value, value, bit_counter);

	}

	if (bit_counter && (n == 0)) {
		printf("got value %d = 0x%x (%d bits)\n", value, value, bit_counter);
		value = 0;
		bit_counter = 0;
	}

`
this on cards that start at 1 or 2 if it works example:
(reading usb=2126451540,  reading rfid=2126451540).
(reading usb =1378830013, reading rfis=1378830013 ).
cards that start at 3 or more do not work example:
(reading usb<PHONE_NUMBER>,   reading rfid= -1018910446)
(reading usb:<PHONE_NUMBER>,   reading rfid= -1018794206)
any ideas?

Hi, I solved it
to declare value lon long. Not int,
regards


NFR Scale test doesn't count all reloads and batches

NFR Scale test doesn't count all reloads and batches.
Example of a run:
## Test TestScale_UpstreamServers

### Reloads

- Total: 61
- Total Errors: 0
- Average Time: 125ms
- Reload distribution:
	- 500ms: 61
	- 1000ms: 61
	- 5000ms: 61
	- 10000ms: 61
	- 30000ms: 61
	- +Infms: 61

### Event Batch Processing

- Total: 64
- Average Time: 232ms
- Event Batch Processing distribution:
	- 500ms: 63
	- 1000ms: 64
	- 5000ms: 64
	- 10000ms: 64
	- 30000ms: 64
	- +Infms: 64

However, looking at the NGF logs, there are 109 reloads and the last batchID is 114:
{"level":"info","ts":"2024-05-22T16:32:46Z","logger":"eventLoop.eventHandler","msg":"Finished handling the batch","batchID":64}
. . .
{"level":"info","ts":"2024-05-22T16:33:01Z","logger":"eventLoop.eventHandler","msg":"NGINX configuration was successfully updated","batchID":114}

Full log https://github.com/nginxinc/nginx-gateway-fabric/blob/d443d6a55988eae8091d230dbbd90ceef1de2c8f/tests/results/scale/edge/TestScale_UpstreamServers/ngf.log
Acceptance criteria:

Make sure reloads and batches are counted correctly

@pleshakov did other tests run against this instance of NGF before this one?

@kate-osborn
nope. in fact, each scale tests re-installs NGF so that its logs are fresh (no carried over errors)

@kate-osborn nope. in fact, each scale tests re-installs NGF so that its logs are fresh (no carried over errors)

That's good. Well, in that case, we may be able to simplify the Prometheus queries and remove the start and end times.

@kate-osborn
the queries that migrated from test instructions to the scale_test.go don't use end time. But they do use start time, so that any NGF reconfiguration prior to the test is not counted.
https://github.com/nginxinc/nginx-gateway-fabric/blob/5f9ec2d4348eef258ffebfaa25b780e278b5557a/tests/suite/scale_test.go#L382-L390

@kate-osborn
the queries that migrated from test instructions to the scale_test.go don't use end time. But they do use start time, so that any NGF reconfiguration prior to the test is not counted.
https://github.com/nginxinc/nginx-gateway-fabric/blob/5f9ec2d4348eef258ffebfaa25b780e278b5557a/tests/suite/scale_test.go#L382-L390

Ok, so NGF is installed before each test, but the prometheus instance is shared among all the tests. I wonder if the start time is what's causing this issue.

Ok, so NGF is installed before each test, but the prometheus instance is shared among all the tests. I wonder if the start time is what's causing this issue.

I see. we do use pod name though in each query, which should be unique per installation

Ok, so NGF is installed before each test, but the prometheus instance is shared among all the tests. I wonder if the start time is what's causing this issue.

I see. we do use pod name though in each query, which should be unique per installation

I'm thinking it has more to do with when prometheus has scraped the metrics from NGF. Prometheus is configured to scrape every 10 seconds, right? So maybe it's not the start time, but the time we send the query. Is it possible that we send the query before prometheus has scraped the latest metrics from NGF?

I'm thinking it has more to do with when prometheus has scraped the metrics from NGF. Prometheus is configured to scrape every 10 seconds, right? So maybe it's not the start time, but the time we send the query. Is it possible that we send the query before prometheus has scraped the latest metrics from NGF?

quite possible. there is a 2 times scrapes interval timeout. after that, we also check that mertics exist at the end time
https://github.com/nginxinc/nginx-gateway-fabric/blob/5f9ec2d4348eef258ffebfaa25b780e278b5557a/tests/suite/scale_test.go#L270
https://github.com/nginxinc/nginx-gateway-fabric/blob/5f9ec2d4348eef258ffebfaa25b780e278b5557a/tests/suite/scale_test.go#L290-L304
that should have prevented not having recent metrics in Prometheus. but it somehow doesn't work :(


Water level overlay incorrect detection

Hello thank you for sharing your work. I run your application and it detects the top border line of the board as water level overlay and does not detect actual water level overlay. How can I fix it? Is there any parameter settings for detection algorithm?

Also calib boxes are incorrectly detected or set.

Hello,
There are parameters that can be set, but they have not yet been exposed to the interface. This is a very early version Alpha version of a GUI that runs the underlying calibration and line finding algorithms. We will get to that in due time. If you could email me one of your images, I could take a look and see if there is a way you can use the program to make this work.
Ken
Sent with ProtonMail Secure Email.
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On Monday, February 15, 2021 10:29 PM, batselem<EMAIL_ADDRESS>wrote:

Hello thank you for sharing your work. I run your application and it detects the top border line of the board as water level overlay and does not detect actual water level overlay. How can I fix it? Is there any parameter settings for detection algorithm?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Thank you for your reply. I am sorry but I couldn't find your email address on the internet.
Batselem,
<EMAIL_ADDRESS>

The email I use for this account is<EMAIL_ADDRESS>Sent with ProtonMail Secure Email.
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On Tuesday, February 16, 2021 4:37 PM, batselem<EMAIL_ADDRESS>wrote:

Thank you for your reply. I am sorry but I couldn't find your email address on the internet.
Batselem,
<EMAIL_ADDRESS>—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hello Batselem,
I just wanted to let you know that we have now linked your review on our website. Thank you so much for this good work!
It is under Citations and Reviews on this page: GaugeCam


oidc reverse proxy required for mgmt instance of console app

What is your feature idea?
any signed in user can access, but we must require auth against vault.
Why is it needed?
internet facing app capable of provisioning infrastructure and incurring cloud costs by bad actors
Is this missing feature preventing you from using kubefirst?

[X] Yes

Code of Conduct

[X] I agree to follow this project's Code of Conduct

This is fixed. We have implemented native authentication in front of Console https://github.com/kubefirst/console/pull/315


Enable combining targets for scoring

If I run a single CheckStyleGrader over multiple targets in a single call to Grader.grade(List<Target>), each target can earn (or lose) the maximum score.
For example, consider:
List<Target> targets = Target.fromPathStrings("src/main/java/student/",
        "Class1.java", "Class2.java");
CheckstyleGrader checkstyleGrader = new CheckstyleGrader(
        "config/checkstyle-rules.xml",
        1.0,
        10.0);
List<Result> results = checkstyleGrader.grade(targets);

Because there are two targets, Checkstyle is worth 20 points, not 10. This violates the
Principle of Least Astonishment.
The same thing happens with other graders.
There are workarounds, such as constructing multiple graders or postprocessing the returned List<Result>, but there should be a better way.
This applies to jacquard-examples/hw1.

I fixed this in checkstyle with commit fd3f08e.

I fixed it in PMD with commit 64da97c.


rsync should sync to copy of last release dir

If I am doing an upload/rsync - it creates a new directory in the releases dir and then starts an rsync to the empty dir.  It seems it would be better to copy the last release dir to the new one, and do an rsync to that.  Then you are only updating the files that changed, which more often than not is a small set.

Use copy_dir param or create task for the it: cp -R {{previos_release}} release

this should happen just before the upload?
I need the curly brackets on both like this?  (I'm new to deployer..)
cp -R {{previous_release}} {{release}}

Yes, like this

It was supposed to be like your first version
cp -R {{previos_release}} release because the release is the dirname - not a config option.

Try release-path


06 - Update JS

This version of the JS file needs to be completely working or just about so.

This looks good and everything currently works. Still need to add timer and score tracker.


CloudFront Use Cases

Greetings!
I was considering using this module until I ran into a problem: I need to be able to create a certificate in ap-southeast-2 AND us-east-1 using the same TLD and SAN records. That is to say:
module "my-domain-name" {
  source = "../../modules/terraform-aws-acm"

  domain_name = "my-domain-name.com"
  zone_id     = data.aws_route53_zone.my-domain-name.zone_id

  subject_alternative_names = [
    "www.my-domain-name.com",
    "example.my-domain-name.com",
  ]

  validation_allow_overwrite_records = true
}

module "my-domain-name_cfn" {
  # CloudFront requires the certificate to be in the US
  providers = {
    "aws" = aws.us
  }

  source = "../../modules/terraform-aws-acm"

  domain_name = "my-domain-name.com"
  zone_id     = data.aws_route53_zone.my-domain-name.zone_id

  subject_alternative_names = [
    "www.my-domain-name.com",
    "example.my-domain-name.com",
  ]

  validation_allow_overwrite_records = true
}

I need an ACM managed TLS certificate in Australia for our ALB. I also need the same certificate (same in the domain/SAN sense) in the US (us-east-1) because that's a CloudFront requirement. This should be a simple case of just using your module twice.
I'm worried that this will introduce race conditions, however. Also, if I create these resources and re-run, it's not idempotent for somewhat obvious reasons (the CNAME record name and value are calculated from the fixed inputs you give ACM, so they collide in the DNS Zone.)
Besides going back to my more verbose (but woking) code, can you make any suggestions or think of a way the module could be updated to accomodate this?

Sounds like a proper use-case. Route53 records are global across all regions, so I would recommend creating one ACM certificate with validate_certificate = true and another with validate_certificate = false to have DNS management idempotent.
Setting validation_allow_overwrite_records = true is not so important if DNS will be managed for just one certificate.

I actually managed to resolve this and it was obvious.
I essentially created two certificates but only one set of CNAME records in the DNS Zone (Route53). The reason for this is because the certificates are identical, so the CNAME records are identical too. Therefore, you...

Create the first certificate
Create the second certificate (in the US)
Get the DNS records required to validate it
Update the DNS Zone in Route53
Create two DNS validation resources to wait for the certificate to be issued
Done.

Because both certificates result in identical CNAME records, the validation checks both pass just fine.
Using validate_certificate = false might speed things up a bit, so I might try that, but leaving it as true did not introduce any errors, race conditions or problems for me.
Thank you for your time, @antonbabenko.


What's the segment download policy?

Excuse me, What's the segment download policy?
I know this project will cut segment to many smaller pieces,
Can this project download same segment but from difference peer(difference pieces)?
I can't find any code, It look's like download same segment from one peer, but same time can download different segments from differing peer

Segments are not split into pieces.
A segment is downloaded from one peer at a time. If the download fails the segment may be downloaded from another peer.
Few simultaneous P2P downloads are supported each from different peer.

Thanks your answer, this is very useful


Wrong Port Referenced

Your instructions say port 8080 for the webserver. The actual code uses 8008.

Ah, the Node server used 8008. http-server - the server actually serving the web content - uses 8080 (:


[array] tojagged: fixed compactness condition

An array can be compact even if self.offsetsaliased is False (because tmp._starts.base and tmp._stops.base are None).

In principle, if an array is compact, it could always be made to have aliased offsets, but it doesn't look like the function that guarantees compactness will ensure that the offsets are aliased. So you're right that this function should be looser.


heal plugin not working in v3.5.2

What are you trying to achieve?
To use the self-healing https://codecept.io/ai/#self-healing-tests with the plugin heal
Instead of I.click('Sign In') which is working, I edit it to I.click('Sign InZZZ') to see how the plugin can heal it
What do you get instead?
I got an error not about that the click on the text 'Sign InZZZ' has failed, but that the text expected after logging in has not appeared.
If I edit ./node_modules/codeceptjs/lib/plugin/heal.js, commenting out try/catch in tryToHeal function, I can see the error
I click "Sign InZZZ"
 » Running OpenAI to heal I.click("Sign InZZZ") step
 » Received 2 suggestions from OpenAI
 » <screenshotOnFail> Test failed, try to save a screenshot
 » Screenshot is saving to xxxxx.png
  × FAILED in 22336ms

-- FAILURES:

  1) Login
       User logins with unregistered email :
     I is not defined
      at eval (eval at tryToHeal (node_modules\codeceptjs\lib\plugin\heal.js:159:15), <anonymous>:1:1)
      at tryToHeal (node_modules\codeceptjs\lib\plugin\heal.js:159:15)
      at processTicksAndRejections (node:internal/process/task_queues:96:5)

Details

CodeceptJS version: 3.5.2
NodeJS Version: 18.16.0
Operating System: Windows
Playwright:
Configuration file:

{
  ...
  plugins: {
    ...
    tryTo: {
      enabled: true,
    },
    retryTo: {
      enabled: true,
    },
    heal: {
      enabled: !!process.env.OPENAI_API_KEY,
      healLimit: 2,
      healSteps: [
        'click',
        'fillField',
        'appendField',
        'selectOption',
        'attachFile',
        'checkOption',
        'uncheckOption',
        'doubleClick',
      ],
    },
  },
  ai: {
    model: 'gpt-3.5-turbo-16k',
    temperature: 0.1,
    html: {
      maxLength: 50000,
      simplify: true,
      minify: true,
      interactiveElements: [
        'a', 'input', 'button', 'select', 'textarea', 'option',
        'm-button', 'm-text-input', 'm-number-input', 'm-text-area',
        'm-form-multiselect', 'm-options-container', 'm-form-checkbox',
        'm-form-date-picker',
        'm-v2-form-field', 'm-v2-radio-button', 'm-v2-checkbox'
      ],
      textElements: ['label', 'h1', 'h2', 'm-v2-form-label'],
      allowedAttrs: [
        'id', 'for', 'class', 'name', 'type', 'value', 'tabindex', 'label', 'role',
        'data-test',
      ],
      allowedRoles: ['button', 'checkbox', 'search', 'textbox', 'tab'],
    },
  },
  ...
}

My personal quick fix
I introduce the 3rd parameter I in async function tryToHeal(failedStep, err, I) and pass helper to it in
event.dispatcher.on(event.step.before, () => {
  ...
  await tryToHeal(step, err, helper);
  ...
}

and have the test passed
  OK  | 1 passed   // 36s

===================
Self-Healing Report:
1 step(s) were healed by AI

Suggested changes:

1. To fix User logins with unregistered email
Replace the failed code with:
- I.click("Sign InZZZ")
+ I.click('button.primary.mynd-theme-external.stretched');
at Actor.loginApplicantSite (.\helpers\I.extend\login.ts:21:10)

======= Tests finished =======

It is quite fun but I know what caused that issue
Indeed I should have been defined somewhere, but because I was not used by anything except eval, I assume eslint considered this line as useless and dropped it.
It's easy fix. Should be done in 3.5.4


Replace deprecated jquery methods.

Questions
Answers

Description?
Replace deprecated jquery methods with current.

Type?
improvement

BC breaks?
no

Deprecations?
no

Fixed ticket?
I didn't find a specific issue, but it seems working out jquery deprecations is ongoing (e.g. #15943)

How to test?
In dev mode, jqmigrate no longer complains, also, verify that module work as expected (it's about click actions modifications)

As it's about click actions, I think we should push it to the QA

hello @lmeyer1
Thank you for this PR.
I check this but nothing change in console.

Could you check ,
Thank you

@sarahdib
I only fixed the display on a product page. There shouldn't display any warnings anymore.
On which page do these warnings display in console?

@sarahdib
I'm conscient, that I did not perform a in-depth review of all the code, that why I left [WIP] in the title, and there are probably remaining compatibility issues.
But I cannot understand, that on your side there are still as many notices. I can assure you, that on the product page, on my side, there is no deprecation message related to productcomments remaining anymore.
I don't have time to invest, neither to understand what the reason is behind this difference of behavior, nor to compete the work by reviewing the complete Javascript codebase.
Therefore, I prefer to close this issue, than to continue and endless revision process.


Update Composer dependencies (2024-03-19-00-18)

Visual regression test passed!
View the visual regression test report


Replace chalk with more performant ansi-colors

ansi-colors is way more performant than chalk and has no dependencies. This PR replaces chalk for ansi-colors.

Thank you for the contribution!


iOS samples does not work on Local Device

Tested MobileBlazorBindingsHelloWorld.iOS and MobileBlazorBindingsTodo.iOS. Used original code, only change is iOS Bundle Signing set to Automatic Provisioning with my developer certificate and marked projects as "Deploy" in Configuration Manager.
Tried to start with Debug and simply Deploy the release version but none of them work.
I can see the app but if I click on it the app will crash instantly.
Tested on iPhone 12 Pro Max with iOS 14.4.1.

Hi @Ogglas can you share any more info on the crash? Does the app run on the iOS Simulator?

@Eilon Got it working by specifying Supported Architectures: ARMv7 + ARMv7s + ARMv64 under iOS Build and cleaned the solution. Closing this issue.


Copy Empty Lines is not working

I have VS2022 version 17.3.6 and Tweaks 2022 1.1.139 freshly installed and the "Copy Empty Lines" and "Copy Empty Selection" features don't work.
When I have nothing selected on a line with text and press CTRL-C the whole line is copied to the clipboard.  This is the behaviour I'm trying to disable by using the plugin.  I tried uninstalling and reinstalling and rebooting, didn't help.  I tried changing the values for copy empty line/selection and copy without indentation to all possible combination of values but didn't help.
Is this feature broken?  Is there any other debug info I can provide, if so how?   Copying full lines without selecting them is annoying and has never been useful even once.

I also see this problem.
This feature was actually why I installed Tweaks 2022

I'm also experiencing this - newly upgraded to VS 2022 17.5, with Tweaks 2022 1.1.143.


MUI Link styling regression with updated @emotion/ and @mui/ packages

Context:
@mui/lab was added as a dependency and versions of @emotion/react @emotion/styled and @mui/material were bumped in order to use the MUI Lab Tabs API when implementing MetricChartTabs (PR #479).
Issue:
Presumably, updating package versions broke/changed the use of MUI createStyled() on the MUI Link component. No other components were affected:
.../MetricCompareTable.style.ts(12,14): error TS2742: The inferred type of 'StyledLink' cannot be named without a reference to '@mui/system/node_modules/csstype'. This is likely not portable. A type annotation is necessary.

VSCode is still able to infer the type:

but the build failed. Per the error, adding an explicit (though incomplete) type annotation fixed the build, but this doesn’t feel like a satisfactory solution and doesn’t get at the root cause of the issue. Removing the styled component and adding styles inline also circumvented the issue.
Package Updates:

package
old version
new version

@emotion/react
^11.9.3
^11.10.5

@emotion/styled
^11.9.3
^11.10.5

@mui/material
^5.9.1
^5.10.17

@mui/lab
--
^5.0.0-alpha.111

Maybe this will go away after solving https://github.com/covid-projections/act-now-template/issues/77 - let's hold until that's solved.

I'm not 100% sure what the issue is, but if you delete yarn.lock in packages/ui-components and run yarn again to regenerate it, the problem goes away.

See https://mui.com/material-ui/guides/typescript/#complications-with-the-component-prop - In some cases, you need to cast the type after customizing its styles (they use a different syntax to style the button, but the idea is the same)
import Button from '@mui/material/Button';
import { styled } from '@mui/material/styles';

const CustomButton = styled(Button)({
  // your custom styles go here
}) as typeof Button;


Music is too loud

The music is very loud when at 100 I found that at 20 it is better but still loud or to quiet in other games

I need to adjust the levels of all of the music to make them all the same volume, at also generally quieter than they are

Levels have been adjusted (have been for a while but i forgot to close the issue, sorry :D)


Try to fail test at pull request

Test and ruff do not fail at push, automatic workflow do not start

Lint passed (which it shouldn't), pytest failed as expected - notice that both linter and pytest are included in one GitHub Action workflow


feat: remove previous Tx history limitations for manual "Add Account"

Description
Does what it says on the box - we have no reason to keep this selector and logic in the "Add Account" screen, since it's in fact entirely decoupled from the <AppContext /> accounts/chains with activity logic for automagical account number discovery per account.
This allows users to Add (an) Account, when the current latest has no Tx history.
Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[x] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes https://github.com/shapeshift/web/issues/6670
Risk

High Risk PRs Require 2 approvals

Low - as a paranoia, even though the logic is entirely decoupled, ensure automagical accounts discovery is still happy

What protocols, transaction types or contract interactions might be affected by this PR?

Testing

It is possible to add as many account numbers as you wish for accounts supporting multi-account, regardless of whether the last added one has Tx history or not for a given chain

Engineering

☝🏽

Operations

☝🏽

Screenshots (if applicable)

Develop

This diff

Discussed this one out with @kaladinlight and agreed this could get in as-is, going to bring this up during our sync however to ensure we're all aligned with this change.