Bchua alexa app

Hi anyone! ^__^  @andkhong @GrandSchtroumpf pls. merge the pull request when you get the chance.

TYVM @andkhong !!!!! ^____^

I didn’t change anything since the last one
De : Bernice Anne W. Chua<EMAIL_ADDRESS>Envoyé : dimanche 29 mai 2016 20:22
À : BerniceChua/alexa-choose-your-own-adventure<EMAIL_ADDRESS>Cc : François<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Objet : Re: [BerniceChua/alexa-choose-your-own-adventure] Bchua alexa app (#4)
Hi anyone! ^__^ @andkhong https://github.com/andkhong  @GrandSchtroumpf https://github.com/GrandSchtroumpf  pls. merge the pull request when you get the chance.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/BerniceChua/alexa-choose-your-own-adventure/pull/4#issuecomment-222404444 , or mute the thread https://github.com/notifications/unsubscribe/AHxCaETSV_ANLF5YnU4do9DUoCSpuk-Cks5qGlfAgaJpZM4Ipc_X .  https://github.com/notifications/beacon/AHxCaMVZmEP8IgKJ-G7uOEyDUjOBBmeoks5qGlfAgaJpZM4Ipc_X.gif


Bug: Currently, we can access the /login route even in a logged-in situation.

Bug
Currently, we can access the /login route even in a logged-in situation.
Expected behavior
The /login route should not be accessible when I have a valid JWT and I am also in a logged situation.
Grab the issue

to grab the issue comment below I want to contribute in this issue under JWoC. Assign me this issue.

Contributor Guideline

Read the README.md file before contributing

Hey @sayan404 Sir I Would Like to Work on This issue Under JWOC 2024

okay, assigning this issue to you @JavidSumra


Use clang-format-18

Summary

Only clang-format-18 will be used now

Will do. Thanks for feedback

LGTM


Custom metadata and reporting

🛑  PLEASE DOT NOT MERGE this till https://github.com/ethyca/fidesplus/pull/581 has been merged into main
Closes #524 #525 #526 #528
Code Changes

All of my code changes are behind the customFields feature flag. Currently, this feature flag is only turned on for DEV environment. You can update this directly in the fides > clients > admin-ui > src > flags.json file.

Only a FidesPlus user can access the custom fields functionality
Allow a user to create a custom field and assign it to an edited resource such as a Data Category, Data Subject, Data Use, and System.
Clicking on the Add a custom field button will trigger the Add custom field modal
Within the Add custom field modal, user can create one or more custom list(s) and assign this list to a newly created custom field.
Custom field definition supports Single select and Multiple select functionality
When a user clicks the Save button within the modal, a successful toast notification is displayed. The tab form is always in a create mode.
In the event the user clicks the Close or Cancel button or clicks outside the modal with unsaved data changes, a prompt will be displayed asking if the user would like to continue editing or discard the data changes.

Steps to Confirm

Execute the make server command via FidesPlus terminal window
Run the Fides Admin UI via npm run dev command via Fides Admin UI terminal window
Open a browser tab and navigate to http://localhost:3000
Login as root_user
Click the Management top level nav link
Edit a Data Category, Data Uses, or Data Subjects record by clicking on the Edit button
Click the Add a custom field button
Create a custom list
Create a custom field
Close the modal
Select one or more custom field values
Click the Update entity button to assign the custom field to a given resource

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation:

[ ] documentation complete, PR opened in fidesdocs
[ ] documentation issue created in fidesdocs

[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[x] Update CHANGELOG.md

Screen Shots

@adamsachs @sanders41 When invoking the API endpoint http://localhost:3000/api/v1/plus/custom-metadata/custom-field PUT, I am receiving the following exception:
Invalid resource type
NOTE: This is happening on the create custom field for a given resource.

@chriscalhoun1974 i think the issue there may be that you're passing a resource_type field at all in this call.
the PUT /custom-field endpoint just needs to have the following fields: resource_id, custom_field_definition_id and value. the resource type is inferred on the backend based on the associated custom_field_definition_id and so it's not needed in the request payload here. are we able to try without that field and see if it resolves things?

Ok, here's my testing results:
Server commit: https://github.com/ethyca/fidesplus/commits/c48ac26
UI commit: https://github.com/ethyca/fides/commits/23fd3e2d9 (Chris's PR#2536 squashed)
Main checks

[x] No errors seemed to be the API's fault. (No 500; validation errors were correct.)
[x] With the feature flag (customFields) disabled, editing taxonomy entities still works.

Functionality status

Defining custom lists

[x] Create
[x] Use for a field definition

Defining custom fields

[x] Create single select
[x] Create multi select
[x] Cancel button warns about discarding changes

[ ] "Show in forms and reports"

How do we verify this?

Using single select

[x] Initial selection
[ ] Removing selection
[ ] Changing selection
[ ] Initial load when data has been saved

Using multi select

[x] Initial selection
[ ] Removing selection
[ ] Changing selection
[ ] Initial load when data has been saved

[ ] When there are multiple metadata fields
[ ] Switching between taxonomy entities
[x] Switching between resource types

Does seem to reset the form correctly

Not good

On initial load the spinners don't go away
The requests 404 because no fields have been added to a category

The API is correct to return 404
The UI needs to understand the "nothing exists yet" case
Probably causes the spinners problem

Clicking "edit" on another category doesn't update the custom fields form

Inconsistent
Might only happen if you made change in the form
Wait, unless custom fields are supposed to show up for all data categories? In which case the problem is that the form sometimes forgets which fields to show.

Removing a item from the field select list doesn't remove the item

But if you "Update entity" and refresh the page it will have been saved

Clicking "save" doesn't close the add field modal
Clicking "save" doesn't refresh the custom field form
Having two fields backed by the same select list causes the form fields to get mixed up

This should be fine. They should have the same list of choices, but the values should be independent.
Actually, it might not matter which custom list the fields use. Values seem to attach to the wrong fields sometimes regardless.
Can result in API error response because a single select was passed an array or a multi was passed a string

http://localhost:3000/api/v1/plus/custom-metadata/custom-field

{"custom_field_definition_id":"plu_a5f493f8-cbb7-435d-a94c-f28ec281f4c1","id":"plu_5ee31d1d-7ed2-44fe-9cfb-6970295f0d39","resource_id":"system.operations","value":["a"]}

{"detail":"The value for the specified custom field type must be a string"}

Fields reorder after updating their values
"Update" button stays disabled after request completed
Form fields flicker after update.


Failed to download: 1 due to The requested range is not satisfiable

Hi,
I have problems downloading files from S3 to my android app using a merged Cognito/S3TransferUtility app. I can copy the whole stack trace but the central issue seems to be
Failed to download: 1 due to The requested range is not satisfiable (Service: Amazon S3; Status Code: 416; Error Code: InvalidRange; Request ID: BBA7FCE8A5FA9C4B)
com.amazonaws.services.s3.model.AmazonS3Exception: The requested range is not satisfiable (Service: Amazon S3; Status Code: 416; Error Code: InvalidRange; Request ID: BBA7FCE8A5FA9C4B), S3 Extended Request ID: iSBDXNaXQnZeodu0rum5l8ZRLXFWaJOswLjm+ZY1jVHHD3iEDhlwEds/TLol96yAT8CaCdF7iEc=
Any ideas?
cheers

Hi @jonasze
Are you providing the range field in your S3 request? If so, you may have requested a range larger then your file size.
If you have further questions, please provide the necessary code and/or steps to reproduce error.

I came across with the same problem. However, I am not understanding about providing a range field in the S3 request or setting up a new AVd. How do I do that within my application? I searched the internet for more information related, but haven't found a proper documentation yet.

Hi,
I remember solving it by clearing cashes, updating the API or similar. The
error message should actually state: Something in the background fucked up
and we don't know what.
On 19 December 2017 at 16:27, Jair Avilés<EMAIL_ADDRESS>wrote:

I came across with the same problem. However, I am not understanding about
providing a range field in the S3 request or setting up a new AVd. How do I
do that within my application? I searched the internet for more information
related, but haven't found a proper documentation yet.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/awslabs/aws-sdk-android-samples/issues/85#issuecomment-352791677,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASfbLurfMyPCkRDTWi3PWRx5jyVGpAV4ks5tB9XFgaJpZM4KV7uw
.


Binary haskell backend mode segfaults on macOS/x86

This is a known issue on macOS/arm64, and we have the --no-haskell-binary flag to disable the behaviour. On a recent run of the K test suite I noticed that this is now happening on my Intel mac as well, when it previously didn't.
I'm not sure if this is down to an OS upgrade, or something changing in the K dependencies, but I wanted to report it anyway.
Steps to reproduce:
$ cd $k_root
$ mvn clean && mvn package -DskipTests
...
$ ./k-distribution/bin/kompile --version
K version:    v5.5.53-0-gad7444a626
Build date:   Mon Dec 19 10:33:03 GMT 2022
$ make -C k-distribution/tests/regression-new/array-haskell
set -o pipefail; (cat 1.test.in 2>/dev/null || true) | /Users/brucecollie/code/k/k-distribution/bin/krun 1.test --no-exc-wrap  -d . | diff - 1.test.out
/Users/brucecollie/code/k/k-distribution/bin/../lib/kframework/k-util.sh: line 100: 30104 Segmentation fault: 11  "$@"
[Error] krun: kore-exec ./test-kompiled/haskellDefinition.bin --module TEST --pattern .krun-2022-12-19-10-45-50-975lPqA9Gu/tmp.in.KdbSXzWI9O --output .krun-2022-12-19-10-45-50-975lPqA9Gu/result.kore
[Error] krun: Backend crashed during rewriting with exit code 139
0a1,3
> <k>
>   0 ~> .
> </k>
make: *** [1.test] Error 1
$ sed -i '' 's/ TEST/ TEST --no-haskell-binary/' k-distribution/tests/regression-new/array-haskell/Makefile
$ make -C k-distribution/tests/regression-new/array-haskell clean
rm -rf ./test-kompiled .depend-tmp .depend .kompile-* .krun-* .kprove-* .kbmc-* kore-exec.tar.gz
$ make -C k-distribution/tests/regression-new/array-haskell
/Users/brucecollie/code/k/k-distribution/bin/kompile --syntax-module TEST --no-haskell-binary --no-exc-wrap --backend haskell  test.k -d .
set -o pipefail; (cat 1.test.in 2>/dev/null || true) | /Users/brucecollie/code/k/k-distribution/bin/krun 1.test --no-exc-wrap  -d . | diff - 1.test.out
set -o pipefail; (cat 2.test.in 2>/dev/null || true) | /Users/brucecollie/code/k/k-distribution/bin/krun 2.test --no-exc-wrap  -d . | diff - 2.test.out
set -o pipefail; (cat 3.test.in 2>/dev/null || true) | /Users/brucecollie/code/k/k-distribution/bin/krun 3.test --no-exc-wrap  -d . | diff - 3.test.out
set -o pipefail; (cat 4.test.in 2>/dev/null || true) | /Users/brucecollie/code/k/k-distribution/bin/krun 4.test --no-exc-wrap  -d . | diff - 4.test.out
set -o pipefail; (cat 5.test.in 2>/dev/null || true) | /Users/brucecollie/code/k/k-distribution/bin/krun 5.test --no-exc-wrap  -d . | diff - 5.test.out

Happens on my Intel mac as well, was never able to use it without --no-haskell-binary.

Ah, feel free to close in that case - for some reason I thought this was new!


Update to Julia 1.3

Hey!

I updated the package to be compatible with Julia 1.3. Note that I don't have lot of experience with Julia, but the example reproduces.

I didn't have the time to work on unittests. But at least the package is working now :-)

I was so free to change some of the fields. E.g. "EEG.Marker" => "EEG.marker", to loosely follow the Uppercase => Type syntax I saw everywhere and to be consistent with EEG.data (lowercase).
I also renamed EEG.SamplingRate to EEG.srate for my convenience.

I added some more fields to EEG.marker

I changed the folder structure so that this package can easily be installed using Pkd.add instead of include

Happy to improve small things, but I think e.g. for an EEG-data structure, a lot is dependent on EEG.jl being ported, or somebody else writing a nice julia EEG dataformat.
Best,
Bene

Holla, i hadn't expected anyone actually being interested in this little experiment of mine from what feels years ago.  Oh - it is actually years ago. :D Haven't really used julia since, so i am more than thankful for your contribution!


Automatically release upon created release

#179

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

Needs the private NPM_TOKEN to be set


[doctest] Updated to 2.3.1

Looks like the option to not build the tests has been removed in this release 🤷♂
edit: nvm, it's been re-added in 2.3.1.

Thanks for the PR!


Recruits Steve Models are glitched

The models are initially fine. However, after combat when it comes back to rest it rests in a weird position. Here the screenshots:
After killing a spider

A recruit that spawned in this position

The same recruit after I restarted my game

Update your Mob Player Animator version to 1.3.1. This issue was fixed for all mobs.

See here: https://github.com/Thelnfamous1/Mob-Player-Animator/issues/9

Oh yeah that did it, thank you!


Why new Deferred is used internally instead of defer()

I wanted to add something to every promise created in the app, so I've overriden $q.defer() in a decorator which adds a new property into promise created by the method. I also needed to override the then method, so the property is also on the promise created by that method.
However, this doesn't cover $q.all(), $q.when() and so on, as I would need to override these also. I was just wondering why Angular internally uses new Deferred way, which is not available in the public API, instead of the $q.defer() method which can be easily overriden by a decorator.

I suspect the primary reason is to avoid the need for the injector to be used, when the function is readily available. Let's discuss how we could help here...

There has been no activity on this for a long long time!


docs: Add monitoring/metrics doc

This doc adds some information about how to set up a monitoring stack using Prometheus and Grafana.
A list of exposed metrics is also provided.
Co-authored-by: Sri Venkateshwara Kishore Kumar Goutham<EMAIL_ADDRESS>Co-authored-by: Nick Hill<EMAIL_ADDRESS>Co-authored-by: Angel Luu<EMAIL_ADDRESS>

/lgtm


Neural Style Transfer using VGG19

i added notebook show casing Neural Style Transfer , Please review and let me know what do you think !

@googlebot I signed it!


Scheduler improvements

Qlue currently uses a cron-like scheduler, which could be more useful. Some possible improvements include:

Jobs can be marked "important". That means if the job doesn't run at the required
time for whatever reason (e.g., the application is down), it is run at the next
possible opportunity.

Jobs can be assigned tags and there is per-tag concurrency control. For example,
you might want to mark database jobs with a "db" tag and state that only one such
job can be running at one time.

If a job is running, another instance of the same job is not started.

Job failures are an application event, cause admin notification, etc.

The scheduler code has been removed from Qlue in order to simplify the framework.


mybatis-3.4.2 needs to be complied by Java 6 not Java 8

MyBatis version
3.4.2
Database vendor and version
Test case or example project
Steps to reproduce
The mybatis-3.4.2 released at Jan 2017 was compiled by Java 8. I think it should be complied by Java 6 unless the java that mybatis supports is upgraded to java 8.

Make a sample code for my batis
Run it with mybatis-3.4.2.jar by java 6

Expected result
These must be no error used in java 6
Actual result
java.lang.UnsupportedClassVersionError happens.
java.lang.UnsupportedClassVersionError: org/apache/ibatis/io/Resources : Unsupported major.minor version 52.0 (unable to load class org.apache.ibatis.io.Resources)
at org.apache.catalina.loader.WebappClassLoaderBase.findClassInternal(WebappClassLoaderBase.java:2271)
at org.apache.catalina.loader.WebappClassLoaderBase.findClass(WebappClassLoaderBase.java:811)
at org.apache.catalina.loader.WebappClassLoaderBase.loadClass(WebappClassLoaderBase.java:1254)
at org.apache.catalina.loader.WebappClassLoaderBase.loadClass(WebappClassLoaderBase.java:1119)

mybatis-3.4.2 needs to be complied by Java 6

MyBatis 3.4.2+ requires Java 8 to build, but runs fine on Java 6.
It is to support some features that requires Java 7+ API.

java.lang.UnsupportedClassVersionError happens.

It should not happen if you are using the JAR in the official release.
When building MyBatis, you need to specify the right maven profile.
$ mvn clean package -Pjava16

Closing as it's not a bug.

The manifest says it all...it's designed from the parent to log in the manifest what jdk version it is targetting.  Plus as others noted, we are only using various jdks for testing purposes.
From manifest off maven central....
X-Compile-Target-JDK: 1.6
X-Compile-Source-JDK: 1.6
So it is definitely compiled with java 6.  So I'm curious as to how that error was even encountered.  For one that would mean adding at least property values to switch the compiler to use java 8.  The profiles don't matter at all in this case as that is for test only.  Can you provide any more info on what you did specifically to result in this?

Hi,
Here are the steps I did.

I downloadedmybatis-3.4.2.zip https://github.com/mybatis/mybatis-3/releases/download/mybatis-3.4.2/mybatis-3.4.2.zip from https://github.com/mybatis/mybatis-3/releases https://github.com/mybatis/mybatis-3/releases.
I unzipped the zip.
I extracted mybatis-3.4.2.jar to jar folder.
I ran the following command and it said it was a Java 8 class.

Z:\Downloads\mybatis-3.4.2\jar\org\apache\ibatis\io>javap -v Resources.class | findstr version
minor version: 0
major version: 52
But when I did same thing with mybatis-3.4.3-20170117.233125-7.jar https://oss.sonatype.org/content/repositories/snapshots/org/mybatis/mybatis/3.4.3-SNAPSHOT/mybatis-3.4.3-20170117.233125-7.jar from https://oss.sonatype.org/content/repositories/snapshots/org/mybatis/mybatis/3.4.3-SNAPSHOT/ https://oss.sonatype.org/content/repositories/snapshots/org/mybatis/mybatis/3.4.3-SNAPSHOT/
It said it was a Java 6 class.
Z:\Downloads\mybatis-3.4.3\jar\org\apache\ibatis\io>javap -v Resources.class | findstr version
minor version: 0
major version: 50
The both have same target and source JDK 1.6
X-Compile-Target-JDK: 1.6
X-Compile-Source-JDK: 1.6

But the build Jdk is different.
mybatis-3.4.2.jar was built by JDK 1.8.0_111
X-Compile-Target-JDK: 1.6
X-Compile-Source-JDK: 1.6
Created-By: Apache Maven Bundle Plugin
Build-Jdk: 1.8.0_111

and mybatis-3.4.3-20170117.233125-7.jar was built by JDK 1.8.0_31
X-Compile-Target-JDK: 1.6
X-Compile-Source-JDK: 1.6
Created-By: Apache Maven Bundle Plugin
Build-Jdk: 1.8.0_31

I guess there are some changes on JDK 1.8.0_111 and it caused the issue.
Thank you,
JB.

On Jan 18, 2017, at 10:20 PM, Jeremy Landis<EMAIL_ADDRESS>wrote:
The manifest says it all...it's designed from the parent to log in the manifest what jdk version it is targetting. Plus as others noted, we are only using various jdks for testing purposes.
From manifest off maven central....
X-Compile-Target-JDK: 1.6
X-Compile-Source-JDK: 1.6
So it is definitely compiled with java 6. So I'm curious as to how that error was even encountered. For one that would mean adding at least property values to switch the compiler to use java 8. The profiles don't matter at all in this case as that is for test only. Can you provide any more info on what you did specifically to result in this?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/mybatis/mybatis-3/issues/900#issuecomment-273671684, or mute the thread https://github.com/notifications/unsubscribe-auth/ARGsUrbd1dF3y_gLc-vbgiMyP3Se8ax0ks5rTtZ5gaJpZM4LnZ7o.

Actually, the .zip archive on the release page contained classes with wrong version (my bad!).
I have already replaced the .zip archive with the valid one, so please download it again.
@hazendaz ,
Is it possible to specify the compiler version explicitly in bundle profile?
Or I might make the same mistake again in the future ;D

Thank you so much,
JB

On Jan 19, 2017, at 8:54 AM, Iwao AVE<EMAIL_ADDRESS>wrote:
Actually, the .zip archive on the release page https://github.com/mybatis/mybatis-3/releases/tag/mybatis-3.4.2 contained classes with wrong version (my bad!).
I have already replaced the .zip archive with the valid one, so please download it again.
@hazendaz https://github.com/hazendaz ,
Is it possible to specify the compiler version explicitly in bundle profile?
Or I might make the same mistake again in the future ;D
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/mybatis/mybatis-3/issues/900#issuecomment-273781497, or mute the thread https://github.com/notifications/unsubscribe-auth/ARGsUlBrN9aGQTCv3DZIygUdMWMQLvKMks5rT2sBgaJpZM4LnZ7o.

Thank you for taking the time to report! =)


API cleanups

This is for closing ensime/ensime-node#22

Oh, sorry, travis didn't really work on the last PR either, did it? Could you look at that? Seems something with param to jasmine is messed up?

I'm going to take a look. It's strange, because I don't have any problems in my local enviroment and also in the logs there is this message:
Error: spawn sbt ENOENT
    at exports._errnoException (util.js:1018:11)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:193:32)
    at onErrorNT (internal/child_process.js:367:16)
    at _combinedTickCallback (internal/process/next_tick.js:80:11)
    at process._tickCallback (internal/process/next_tick.js:104:9)

I changed the language of travis to cpp to scala because sbt is required for the full-stack-smoke-test.spec.ts

It looks like that Travis does't support Scala for osx travis-ci/travis-ci#2316

Well, the problem was that travis was trying to run all the tests including integration tests. I tried to make it run but the test was failing, so I removed the integration tests.

I really want travis to run the integration tests too :( But maybe it's to slow. I had problems with that before…

Look here: https://travis-ci.org/ensime/ensime-node/builds

Now that ITs are excluded from the tests, this PR going to fix the master broken build.

@Jarlakxen Sorry, I'm not sure I follow. What is the reason you have to disable integration tests? I really would like to have them :)

Oh! Sorry for not been clear! =P
Well, one the problems is that we need SBT for running the test. I can switch language to scala in the travis file, but scala is not available for osx. In this commit https://github.com/ensime/ensime-node/pull/32/commits/07a87699da2c798412c8071018906f8382cd3095 I switched to scala and removed osx but the integration test failed. ( You can check the build here https://travis-ci.org/ensime/ensime-node/builds/234659506 ). I run the test locally and It works without any problem =S.
Just as a side note, I have another branch ready to make a PR in which I rewrote the integration tests, to tests the API, because I added a new command, and also improve a lot of other things =). This is the branch https://github.com/Jarlakxen/ensime-node/tree/new-commands and this is the new integration test I coded https://github.com/Jarlakxen/ensime-node/blob/new-commands/src/spec-integration/server-api.spec.ts

Ok, cool. I added a private drone before, but can't really run it for PR's
because security. That's too bad. I want to revisit that stuff because I
really want good test coverage and integration tests on PR's. travis just
doesn't seem to cut it.
On Mon, May 22, 2017 at 2:43 PM Facundo Viale<EMAIL_ADDRESS>wrote:

Oh! Sorry for not been clear! =P
Well, one the problems is that we need SBT for running the test. I can
switch language to scala in the travis file, but scala is not available for
osx. In this commit 07a8769
https://github.com/ensime/ensime-node/commit/07a87699da2c798412c8071018906f8382cd3095
I switched to scala and removed osx but the integration test failed. ( You
can check the build here
https://travis-ci.org/ensime/ensime-node/builds/234659506 ). I run the
test locally and It works without any problem =S.
Just as a side note, I have another branch ready to make a PR in which I
rewrote the integration tests, to tests the API, because I added a new
command, and also improve a lot of other things =). This is the branch
https://github.com/Jarlakxen/ensime-node/tree/new-commands and this is
the new integration test I coded
https://github.com/Jarlakxen/ensime-node/blob/new-commands/src/spec-integration/server-api.spec.ts
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ensime/ensime-node/pull/32#issuecomment-303088989,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHHKD7TT2axvkq5SD9J8DmJl-MWl-8kks5r8YLggaJpZM4NhsHC
.

I hope you can make it work =)
Is there anything else you want me to do, to close this PR?

I'll merge. It's not this PR that broke integration tests anyway :)
Let's see what we can do in the future to get good test coverage. Thanks!


RE2. This is a bug that makes it practically impossible to progress.

Whenever you enter certain events and areas, the HMD screen will look like this.
The screen of the monitor will go black.

Immediately after entering the police station. The event will occur when you look at the surveillance camera that is placed there.
2・It will happen when you first encounter Ada in the underground parking lot.
3・It happens when you encounter Ada for the second time in the jail.
In the waiting room on the second floor of the police station, go out the door on the right and down the stairs (where the zombies are banging on the door).

4 in particular is a bug that makes it practically impossible to progress.
I have
win10
3080
10900KF
Q2-VD
I'm playing on

You can always progress. Black screens happened to me several times during RE2. Most of the time, restarting the game fixed it, other times I had to restart VR for it to work.
Only one time nothing seemed to work (first Ada encounter in the garage). What I did was to disable VR (removing the  VR files), watch the cutscene flat, save the game and continue again in VR (after pasting back the removed files). It's a very easy and quick fix that will allow you to continue.

That's what I do in cut scenes.
The problem is that even outside of cutscenes, when I get to a particular place, I get the same screen.
It's a passage you have to go through many times.

It's happened to me maybe 5 times in the whole game, counting both scenarios. I only had to see the cutscene flat once in the whole game but I don't know what is causing the black screen so I guess it might be different for others.
Most of the time, reloading a save point or exiting the game worked fine for me, give that a try before changing to flat just in case.

In my case, reloading the save does not cure the problem.
It goes away when I quit the game, but it always goes to a black screen in the same place.


add doc example for implementing BinRead

Adds a slightly non trivial example for how to implement BinRead. I found that implementing BinRead for a generic type with the recent args changes was straightforward but not very obvious without examples.

Hi, sorry about the delay. Wondering if it would be possible to do this example not using the FilePtrArgs type? It might be a silly thing to request since these are technically part of the public API but at the moment my gut says it isn’t a good idea to put examples in the docs that use these associated types that are only intended to be used with their parent class.

I can change it to use a custom args type like CustomPtr32Args with the same fields.

Do the latest changes look ok? @csnover

Any updates on this? @csnover

Does this look ok? @csnover


HV-2020 Use OSGi framework utils to get a classloader

https://hibernate.atlassian.net/browse/HV-2020
Use OSGi framework utils to get a classloader, and do not rely on a dependency to expressly.
Based on the change from https://github.com/hibernate/hibernate-validator/pull/1493 that targets 7.0 where we still could run OSGi tests

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on licensing, please check here.

well... we won't get the coverage here since we do not have the OSGi tests that this change is for ...


Make Immutable DB independent from EpochSize

At the moment, the immutable DB stores blocks per epoch. Unfortunately, this introduces a dependency between the immutable DB and the ledger state, because the ledger state tells us when epoch sizes change. Instead we should give the immutable DB a fixed "chunk" parameter, with a precondition that if there are EBBs present, they must line up with the first slot in each chunk. Unfortunately, this means that the size of the files stored on disk will go down a factor of 10 or even 20 (depending on the choice of the Praos f parameter), but after discussing this with @dcoutts , we considered this an acceptable compromise.
This is blocking the work on the hard fork combinator; indeed, it is blocking the work at #1698 on #1637 / #1205 .

WIP PR at https://github.com/input-output-hk/ouroboros-network/pull/1717.

Closed in https://github.com/input-output-hk/ouroboros-network/pull/1750.


Refactor duplicate code

Closes #647
I did not really mark the code, as I didnt find it really helpful, so I rather did a small refactoring to reuse the functions from the other Jenkinsfile. The downside from reusing the functions right now is, that test mdbook and test frontend is executed in the release pipeline now, which wasnt before.
Maybe we can try this new release pipeline for #879, but I'm not 100% sure yet. Might be better for the release after, to get v3.3 out today.
Basics

[ ] I added a line to changelog.md
[ ] The PR is rebased with current master.
[ ] Details of what you changed are in commit messages.
[ ] References to issues, e.g. close #X, are in the commit messages and changelog.
[ ] The buildserver is happy.

Checklist

[ ] I have installed and I am using pre-commit hooks
[ ] I fully described what my PR does in the documentation
[ ] I fixed all affected documentation
[ ] I fixed the introduction tour
[ ] I wrote migrations in a way that they are compatible with already present data
[ ] I fixed all affected decisions
[ ] I added automated tests or a manual test protocol
[ ] I added code comments, logging, and assertions as appropriate
[ ] I translated all strings visible to the user
[ ] I mentioned every code or binary not directly written or done by me in reuse syntax
[ ] I created left-over issues for things that are still to be done
[ ] Code is conforming to our Architecture
[ ] Code is conforming to our Guidelines
[ ] Code is consistent to our Design Decisions
[ ] Exceptions to any guidelines are documented

Review

[ ] I've tested the code
[ ] I've read through the whole code
[ ] I've read through the whole documentation
[ ] I've checked conformity to guidelines

What is left open here? Did this work?

@markus2330 ready to be merged, lets see if it works later.


Add QWANT to search engines

Please add https://www.qwant.com/ to the list of search engines.
It's a privacy concerned search engine made in France
Probably more interesting for french-speaking/reading users, but definitely worth his place on the search engine list imho.

On the next update there will be a Premium Access feature for enabling a custom search engine. This should resolve this request.


Inline query's in channel post

I want to sent a message from bot to channel (no via) , all the things go correct But when i add some buttons in inline keyboard (switch inline query or call back) the buttons don't take any action when i click on them although when i use robot inline mode in private chats and sent message the buttons work correctly .
How can i fix it ...?

hi
check sign messages in channel settings

@FaRzad-845 API v3 Documentation states:

Inline keyboards with switch_inline_query and switch_inline_query_current_chat can no longer be sent to channels because they are useless there.
@GochoMugo This issue can be closed.


Some questions about model

hello!
In other issues,you said your publised model is trained for 1000 epochs  using all 2015 KITTI traning dataset.In the process of training, do you choose all 200 sets of data in the KITTI2015 training set for training, and then use the 40 sets of data in the training set as the validation set. The selected model is the model with the smallest verification error?

@huangcimu
The published model is adopted from the final epoch.

Thank you for your answer. So do you mean that you directly train the 200 data sets without the validation set, and then select the model produced by the last epoch is your model

Thank you for your answer. So do you mean that you directly train the 200 data sets without the validation set, and then select the model produced by the last epoch is your published model

@huangcimu
Yes.


Update limits-specifications-teams.md

Updated "Number of teams a global admin can create" from Unlimited to 500,000 as that is the absolute cap on the number of Teams in a tenant.

@islubin - OK to merge? cap on Teams is 500,000?

@tseward - does this relate to the other SharePoint max where the product team wanted it to say unlimited?

@kenwith Not sure what other you're referring to.

@tseward - I thought I remembered something similar.. maybe not.

@LolaJacobsen - I noticed you removed @ninadara as   ms.reviewer  in metadata..   any idea who can review this on behalf of the Teams product group now? @islubin just FYI in case you know. I suspect you are at Ignite though this week. :)

@kenwith  is the question whether the change is okay by me (switching to 500k as the max, it is) or who from Teams PG should be added?  You can add @karuanag


added alert

😍 Add Hacktoberfest label to this PR 👍

[x] Modify feature
[x]  Improve UI
[Issue no : ] - #30

resolve branch conflicts


Data grid handled events

What does the pull request do?
There was a breaking change in Avalonia 11.0 where some events can't be canceled (https://github.com/AvaloniaUI/Avalonia/pull/9918). This PR tries to fix DataGridCell PointerPressed.
What is the current behavior?
You can't handle the event in a way that would prevent the bubbling. It was handled before on the framework level. Now this needs to be handled by each component.
What is the updated/expected behavior with this PR?
If the event has been handled the bubbling should be canceled.
Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/avalonia-docs with user documentation

Breaking changes
I don't think there should be any raised handled event that should be still taken into account for this.
Obsoletions / Deprecations

Fixed issues

Another solution would be to remove the handledEventsToo: true flag.

I forgot to add a motivation for this. Avalonia has a weird behavior where when I select multiple items in the DataGrid and then I would like to drag all items, they are deselected first. This deselection is caused by mouse down even before the app realizes I am dragging. It should happen on mouse up. It was possible to workaround this, but after this change, it can't be done.


--tsv doesn't honor --nostarted fully

Here's my example:
Just agenda --nostarted
# gcalcli agenda --nostarted

Fri Jan 15   2:30pm  Discussion

Just --tsv
# gcalcli agenda --tsv
2016-01-15      10:00   2016-01-15      11:00   Call
2016-01-15      11:00   2016-01-15      11:30   Check-point
2016-01-15      11:45   2016-01-15      13:15   Lunch
2016-01-15      12:00   2016-01-15      13:00   Call
2016-01-15      14:30   2016-01-15      15:30   Discussion

Now --tsv & --nostarted
$ gcalcli agenda --tsv --nostarted
2016-01-15      11:45   2016-01-15      13:15   Lunch
2016-01-15      14:30   2016-01-15      15:30   Discussion

Note --tsv is still showing the current event which I'm currently in. The agenda --nostarted behavior is what I would expect from --tsv --nostarted

Made https://github.com/insanum/gcalcli/pull/227 to fix this.


[bug] docker-compose.yaml starts postgres but does not use it.

Describe the bug
docker-compose.yaml starts postgres but does not use it.
To Reproduce
Steps to reproduce the behavior:

Download guac-compose.tar.gz
compose up

Expected behavior
Only start containers needed
GUAC version
v0.2.0
Additional context
We should have an easy way to compose-up with postgres/arango when those are ready. For now don't start it with the released compose yaml.
ALSO: The image name used is postgres, and should be docker.io/library/postgres for podman compatability.

I can take this issue.

we can create a follow-on issue for better/ more configurable way to bring up the proper backends once ready for use.


Migrate system boot test to Slim branch

Make sure the test that boots 21 producers via the voting -> unlock -> staking process runs on Slim branch

I did this.


Select and deselect All Options in mat-select

Hi,
I am using Angular Material 5 and want to make this costume SelectAll functionality.

I have tried to add select and deselect all mat-option but i am getting below error.
ERROR Error: Uncaught (in promise): Error: Template parse errors:
There is no directive with "exportAs" set to "ngModel" ("er="Analysis Type" disableRipple [formControl]="documentTypesFormCtrl" [(ngModel)]="selectedValues" [ERROR ->]#analysisType="ngModel" multiple (change)="updateDocumentTypes();">
This is my current code.
<mat-select class="analysis-drop" placeholder="Analysis Type" disableRipple
[formControl]="documentTypesFormCtrl"
[(ngModel)]="selectedValues"
#analysisType="ngModel" multiple (change)="updateDocumentTypes();">
<mat-option
class="selectAllVals"
[value]="year"
(click)="selectAll(analysisType, documentTypesList)">
Select All

<mat-option *ngFor="let documentType of documentTypesList " [value]="documentType.code">{{documentType.desc}}

Please let me know what I should change in it.
Thanks in advance.

Adding an extra "action" option like this ins't something we support, since it doesn't fit in well with the accessibility interaction pattern for the select. #11799 added a select-all keyboard shortcut, which does fit into the a11y pattern; you can add an additional button next t the select that would do the same thing.

Hi @jelbourn ,
Thanks for the quick response. I have tried it but still getting error. I am using [formControl] in html binding. When I remove it then it works otherwise not. I need this [formControl] to get the values of form. Can you please look at this URL ?
https://stackblitz.com/edit/angular-material-select-multi-qkyxzj
I Would be very thankful to you.
Thanks in advance.

Does anyone know how in the heck to make a div with a form disappear upon deselect? onSelectionChange is supposed to account for deselect, but idk how to remove a div that's already appeared without explicitly adding a "remove" button. I want deselect to take care of this somehow.


Validate the example data

Finish validating the docs/example-data.json with the docs/models.yml. This is the first try by Norman: https://github.com/OpenSlides/OpenSlides/blob/openslides4-dev/docs/modelsvalidator/check_json.py

I have tried to improve the check_json.py and get a lot of missing fields in the initial_data.json.
Check for ../../docker/initial-data.json failed:
Collection meeting 1. Missing fields: speaker_ids, poll_default_group_ids, poll_default_100_percent_base, assignment_poll_ballot_paper_selection, poll_default_method, poll_ballot_paper_number, poll_default_type, poll_sort_poll_result_by_votes, poll_ballot_paper_selection, assignment_poll_ballot_paper_number, conference_stream_poster_url, poll_default_majority_method.
Collection group 1. Missing fields: used_as_poll_default_id.
Collection group 2. Missing fields: used_as_poll_default_id.
Collection group 3. Missing fields: used_as_poll_default_id.
Collection group 5. Missing fields: used_as_poll_default_id.
Collection group 6. Missing fields: used_as_poll_default_id.

I have tried to improve the check_json.py and get a lot of missing fields in the initial_data.json.
Check for ../../docker/initial-data.json failed:
Collection meeting 1. Missing fields: speaker_ids, poll_default_group_ids, poll_default_100_percent_base, assignment_poll_ballot_paper_selection, poll_default_method, poll_ballot_paper_number, poll_default_type, poll_sort_poll_result_by_votes, poll_ballot_paper_selection, assignment_poll_ballot_paper_number, conference_stream_poster_url, poll_default_majority_method.
Collection group 1. Missing fields: used_as_poll_default_id.
Collection group 2. Missing fields: used_as_poll_default_id.
Collection group 3. Missing fields: used_as_poll_default_id.
Collection group 5. Missing fields: used_as_poll_default_id.
Collection group 6. Missing fields: used_as_poll_default_id.

Ok, these are indeed issues. Please check again with the changes in #5831 included. Please check the docker/initial-data.json as well as docs/example-data.json.

Ok, these are indeed issues. Please check again with the changes in #5831 included. Please check the docker/initial-data.json as well as docs/example-data.json.

First checks are done. Still a TODO:

Check template fields
More checking of fields and error handling: What happens if an id is not an int, ...


”Berezeni” is listed in the drop-down menu

Description:
"Berezeni" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Berezeni" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Berezeni, VASLUI".


Update issue templates

@jacobwhall This could be a good way to continually update what we need to do for each dataset (and remind us of best practices / key items we might forget)

Here are a few more suggestions off the top of my head:

[ ] use tmp_to_dst_file() context manager to wrap download, processing steps
[ ] test locally
[ ] test using a deployment
[ ] check if there are newer data available (years or versions)

@sgoodm this is a great idea. I added some more content, and an issue template chooser configuration (info here). What do you think?


DSD-1228: TextInput accessibility updates

Fixes JIRA ticket DSD-1228.
This PR does the following:

Adds autocomplete attribute to TextInputs with type "tel", "email", and "url".
Fixes issue where aria-describedby attribute was not being added to some TextInputs.

How has this been tested?

Locally in Storybook, and with a new unit test.

Accessibility concerns or updates

None.

Checklist:

[x] I have updated the Storybook documentation accordingly.
[x] I have added relevant accessibility documentation for this pull request.
[x] All new and existing tests passed.

Front End Review:

[ ] Review the Vercel preview deployment once it is ready.

I do see the attribute now so all is good on my end.


Tab opening on each step

In the gif below, for some reason, a new tab is opened at the beginning of debug session and then each time step_over() is called

My dap config:
local dap = require('dap')
local dapui = require("dapui")

dap.listeners.after.event_initialized["dapui_config"] = function()
    dapui.open()
end
dap.listeners.before.event_terminated["dapui_config"] = function()
    dapui.close()
end
dap.listeners.before.event_exited["dapui_config"] = function()
    dapui.close()
end

dapui.setup({
    icons = { expanded = "-", collapsed = "+", current_frame = "●" },
    mappings = {
        expand = { "<CR>", "<2-LeftMouse>" },
        open = "o",
        remove = "d",
        edit = "e",
        repl = "r",
        toggle = "t",
    },
    layouts = {
        {
            elements = {
                { id = "scopes", size = 0.25 },
                "stacks",
                "watches",
            },
            size = 40,
            position = "left",
        },
        {
            elements = {
                "repl",
                "console",
            },
            size = 0.25,
            position = "bottom",
        },
    },
    controls = {
        enabled = false,
    },
})

Any idea what could cause this?

Nevermind, found out this was dap plugin itself

@sharklasers996 Do you remember what in the dap plugin was causing this?

Nevermind, someone helped me solve the issue here.


Enhancement: Writable sensors

Enhancement related to
Sunsynk Home Assistant Add-On
Description
Hi @kellerza, I see you recently introduced the concept of a RWSensor so I can see that you are planning to support this in the future. Have you got a branch or any in-progress work for this? If not, I'd like to take a stab at it to introduce a missing sensor and make it writable based on a Select Entity. The sensor is number 243 which allows you to toggle between Battery or Load priority for PV production.
Happy to hear your thoughts!

Writing registers are supported by profiles today. So if you create a new profile and add your sensor to the array it should work.
Actual values being written is stored in yaml files under /share/addon_name/. Select UPDATE in the frontend and the current value (or multiple values in case of many sensors) will be written
See here

Yeah I have been using profiles for a while, they are extremely useful!

What I was thinking was that having certain entities editable can be complementary to profiles and actually make creating them easier from Home Assistant. For example, whilst on vacation (therefore no access to the inverter), any time I want to modify the battery percentage of one of the programs (Prog1 Capacity-Prog6 Capacity) I need to access the /share/addon_name, copy and paste one of the profile files, edit the values in it, rename it and then reload. If however I could edit them via the Home Assistant UI, that would be so much easier. Same for the above sensor 243 - currently a profile would need to exist already for the register to be writable.
Here is where the inspiration came from, having device config entities for some/all writable inverter registers:
https://developers.home-assistant.io/blog/2021/10/26/config-entity/#entity-categories

Ok, that is an interesting idea. entity_category seems to be part of the MQTT autodiscovery options, so it should be possible.
If you decide to add all RWSesnors / descendants they will also have to be read as well, and for that the round robin filter should be ok

I'll do a proof of concept first to see if the idea is viable and report here.

Ok it is starting to take shape. As an experiment I converted the Prog 1 to 6 Battery Capacity sensors to NumberEntity instead of SensorEntity. The default min, max and step values (0, 100, 1) for this entity type map perfectly to the battery SOC settings. I also set the entity_category for these sensors to config.
Here is the result on the Device that this add-on creates:

Here is a link explaining the extra "Configuration" card and the implications.
The profile picker is still a primary entity and is therefore in the topmost "Controls" card:

Now going into the NumberSelect entity in detail, it worked out of the box for loading up the existing register values.
I had to create it in mqtt.py:
@attr.define
class NumberEntity(Entity):
    """A HomeAssistant Number entity."""

    command_topic: str = attr.field(default=None, validator=required)

    on_change: Callable = attr.field(default=None)

    _path = "number"

For now the logic of when to create the NumberEntity in run.py is very rudimentary and needs to change, but here it is for discussion:
if isinstance(sensor, RWSensor) and not isinstance(sensor, TimeRWSensor):
    ents.append(
        NumberEntity(
            name=f"{OPT.sensor_prefix} {sensor.name}".strip(),
            entity_category="config",
            state_topic=f"{SS_TOPIC}/{OPT.sunsynk_id}/{sensor.id}",
            command_topic=f"{SS_TOPIC}/{OPT.sunsynk_id}/{sensor.id}_set",
            unit_of_measurement=sensor.unit,
            unique_id=f"{OPT.sunsynk_id}_{sensor.id}",
            device=dev,
            on_change=create_on_change_handler(filt, lambda val: int(val))
        )
    )
else:
    ents.append(
        SensorEntity(
            name=f"{OPT.sensor_prefix} {sensor.name}".strip(),
            state_topic=f"{SS_TOPIC}/{OPT.sunsynk_id}/{sensor.id}",
            unit_of_measurement=sensor.unit,
            unique_id=f"{OPT.sunsynk_id}_{sensor.id}",
            device=dev,
        )
    )

The interesting parts of the NumberEntity are that the entity_category is set, there is a command_topic and there is an on_change handler. There are other attributes that I'd like to set based for NumberEntity, namely min_value, max_value, step and mode, depending on the sensor. These parameters could be defined per sensor in definitions.py of the sunsynk module.
When the entity value changes, the register is written to the inverter, read back again and then the result is published. Error handling could be improved.
async def write_sensor(
    filter: Filter, value
) -> bool:
    """Write sensor with the Modbus interface."""

    sensor = filter.sensor
    newv = ensure_tuple(value)
    if newv == sensor.reg_value:
        return False

    _LOGGER.info(
        "Writing sensor %s: %s=%s  [old %s]", sensor.name, sensor.id, newv, sensor.reg_value
    )
    sensor.reg_value = newv
    await SUNSYNK.write_sensor(sensor)
    await read_sensors([sensor])
    await publish_sensors([filter], force=True)

    return True

Some quirks with the NumberEntity include:

The icon changed to a slider, whereas it was previously a battery for the Prog sensors - see my screenshot in my previous message under "Time of use"
When you click on the entity name and the modal pops up, both the slider and a text box which is squashed are displayed:We'd probably want to pick either slider or box and not have the mode set to auto

Now because I had some time I took things a bit further.
I had previously created a card using the multiple-entity-row HACS add-on. I simply adjusted it to display the new number.* entities for the Prog registers (which were previously sensor.* entities. It looks the same, however tapping on the battery percentages allows you to edit them in a dialog like in the screenshot above - cool!

Lastly - and this is probably going to be controversial - now that some entities are writable in Home Assistant, one can potentially use Scenes to achieve the same functionality that Profiles do. For example, you can create a Scene which sets certain entities to certain values, just like Profiles do, the difference being that you would configure them via the Home Assistant UI. This does need some thinking / discussion.
A further thought about other registers - there are a few other entity types that could be - for example, a SwitchEntity for on/off-type registers and a SelectEntity for registers which allow for selecting one of several fixed values. For the Prog1-6 Time registers I found not find a suitable "time input" entity, so perhaps a SelectEntity with hours or 15 minute steps in a 24 hour day might be appropriate.
Thoughts?

It looks very promising @Ivan-L !
Like all changes, lets do it in incremental steps. Start with the NumberEntity, but without customizing min/max/changing definitions.
The only change to definitions might be to introduce a a new class NumberRWSensor (from RWSensor) to make your test more natural (and ensure it doesnt break when a new class is introduced
In future PR we could even include a text based input for RWSensor
If you can change the settings with scenes, that is simply an additional way to do this. The main reason I made the profiles was that settings like the bit values for programming becomes complex (multiple switch sensors per register) and the current profiles is still the most flexible&safest way to change this.

Hi @Ivan-L you have been very quiet,- are you still working on this?

Hey @kellerza apologies, a few things happened and I have been spending a lot of time sorting out my inverter + battery, so unfortunately I have not had a chance to work on this. I will pick it up this week.
Last I worked on it there was some weirdness where the mode of the number entities kept on resetting back to auto despite me specifying otherwise in the python code. I did the inheritance thing where I selectively opt-in certain registers to use NumberRWSensor. The Prog1..n_Capacity ones were easy since their min/max values are in line with the Home Assistant defaults and I can PR that, however another issue I ran into, which could be tackled later, was with the maximum values for the Prog1..n_power entities - there I cannot hardcode a value of 5000 watts (for 5kW inverters) because that would limit inverters with higher capabilities. There is a specific register that I think gives us the max power to use for the program power registers, but then that register would likely need to be treated the same as the serial number sensor - read on startup and use for the maximum values of other sensors. Furthermore, the UI on the inverter does not allow you to set a Prog1..n_Capacity lower than the low battery capacity register, so that is also something to maybe consider, i.e the minimum is not actually a hard-coded 1 or 0.
This is all in the name of not allowing the user to set values outside of the allowable ranges that the inverter would allow through Home Assistant.
A final spanner is that the Prog1..n_Capacity registers should not be editable if the battery is configured to use voltages rather than % or lithium mode, though that is probably not the end of the world if they are, the user would just not be performing a useful action if editing these entities.
I'd love to hear your thoughts on the above.
I can also push what I have on my fork for you to look at / use.

Hi, register 16&17 seems to be rated power (0.1W) so this might be an option for capacity. Else you can add the min/max to the config of the dev addon while we're testing it

Hi @kellerza, I put up a PR for the above. It ended up being bigger than I thought - let me know if you'd like me to split it up into smaller PRs, I'd be happy to do so.

Hi @kellerza I have implemented some more writable sensors. No PR yet as I just want to make sure you're happy with #48 first before proceeding.
Here is a screen grab:

@Ivan-L so far its running smoothly. A couple of entities changed from sensor.x to number.x and had to be renamed in the frontend
Updating System_mode seems a bit less responsive, but might have to rethink the concept (especially creating new profiles) now that we can directly change the values.

Hi @kellerza, great to hear it's running smoothly.
Yeah the sensors that become writable do migrate from sensor.x to number.x or (in the next PR) select.x and switch.x entities. What we can do is introduce an OPT flag which when set to true makes all sensors writable but if not then the sensors end up as the regular sensor.x entities - I made that suggestion in the initial PR, I guess it just needs to be decided whether to push a breaking change like this out or introduce the option flag so that users can elect to opt into writable sensors and when doing so they are warned that the entity IDs will change.
Updating system mode is slow yeah, I believe this is due to the round robin filter for all of the prog sensors. This can be changed to use a similar approach as the write_sensors function that I implemented where you read the written sensors and publish them again, and you'd get near instant feedback when switching system modes. The previous PR did not affect switching profiles. And yes, HA scenes can be used as an alternative to profiles, but I found profiles extremely useful whilst developing writable entities because I could restore my sensor values easily. With regards to creating new profiles - one enhancement I'd suggest with the current implementation is to get rid of the UPDATE option in the system mode select and replace it either with a MQTT button which triggers a profile update - perhaps that is the only way to create new profiles as currently I have many profiles automatically being created if I tweak some writable entities.
I will put up some PRs over the weekend for SelectRWSensor, TimeRWSensor (updated) and SwitchRWSensor.

Hi @kellerza just checking if you saw #52 is up? I have two more PRs after that one lined up.

Hi @kellerza, PR #53 is up. Please note the following important comment:
When the value of a TimeRWSensor changes, the other TimeRWSensors which depend on it through their min or max fields will not have their options updated. This is a larger issue that we started discussing in a previous PR, where it was debated whether we should subscribe to sensor updates so that we can update HASS discovery info when that happens. In my view this would be desirable not only for solving this issue but also reacting to sensor changes if they are done on the inverter itself. I can put up a separate PR with a proposed solution if you'd like, or I can update this PR with such functionality.

PR #54 was added to elicit some feedback on updating HASS auto discovery info when sensor values change.

Hi @kellerza, just a gentle reminder about #53 and #54 if you can have a look when you get a gap.

How dynamic are these updates in the HASS UI?

With #54 the updates to the affected entities are instant.
For example, assume the following Prog1 Time and Prog 2 Time configuration (ignore the icons, I was playing around with them):

Because Prog 1 Time is 1:00, the min value of Prog 2 Time is 1:00.
This is reflected here:

If I open Prog 1 Time and change it from 1:00 to 2:00, then the min value of Prog 2 Time is updated without the screen even refreshing.

It works really well.
This is true if I walk to the inverter and change the values there as well, although it takes a few seconds to take effect due to the round robin filter.

Hi @Ivan-L I guess we can close this?

Hi @kellerza , I have one last PR that is almost ready. Will put it up tonight.

🎉 thanks @Ivan-L !!


Cluster-autoscaler: handle unready nodes

Introduction
Currently ClusterAutoscaler stops working when the number of nodes observed on the cloud provider side doesn’t match to the number of ready nodes on the Kubernetes side. This behavior was introduced in the first version of CA in order to prevent CA from breaking more the already broken cluster. However the client reported feedback indicated that this behavior is oftentimes suboptimal and leads to confusion. This document sketches how the problem of unready nodes will be solved in the next release of K8S.
Use cases
The number of ready nodes can be different than on the mig side in the following situations:

[UC1] A new node is being added to the cluster. The node group has been increased but the node has not been created/started/registered in K8S yet. On GCP this usually takes couple minutes.
Indicating factors:
-- There was a scale up in the last couple minutes.
-- The number of missing node is at most the size of executed scale-up.
Suggested action: Continue operations, however include all yet-to-arrive nodes in all scale-up considerations.

[UC2] A new node is being added to the cluster. The node has registered on the cluster but has not yet switched its state to ready. This should be fixed in couple seconds. Indicating factors:
-- The unready node is new. CreateTime in the last couple minutes.
Suggested action: Continue operations, however include all yet-to-arrive nodes in all scale-up considerations.

[UC3] A new node was added to the cluster but failed to start within the reasonable time. There is little chance that it will start anytime soon. Indicating factors:
-- Node is unready
-- CreateTime == unready NodeCondition.LastTransitionTime
Suggested action: Continue operations, however do not expand this node pool. The probable scenario is that the node will be picked by scale down soon.

[UC4] A new node is being added to the cluster. However the cloud provider cannot provision the node within the reasonable time due to either no quota or technical problems. Indicating factors:
-- The target number of nodes on the cloud provider side is greater than the number of nodes in K8S for the prolonged time (more than couple minutes) and the difference doesn’t change.
Suggested action: Reduce the target size of the problematic node group to the current size.

[UC5] A node is in an unready state for quite a while (+20min) and the total number of unready/not-present nodes is low (less than XX%). It could either not switched from unready to ready on node registration or something crashed on the node and could not be recovered. Indicating factors:
-- Node condition is unready and last transition time is >= 20 min.
-- The number of TOTAL nodes in K8S is equal to the target number of nodes on the cloud provider side.
Suggested action: Include the node in scale down, although with greater (configurable) unneeded time.

[UC6] Some nodes are being removed by cluster autoscaler. Indicating factor:
-- Node is unready and has ToBeRemoved taint.
Suggested action: Continue operations. Nodes should be removed soon.

[UC7] The number of unjustified (not related to scale-up and scale-down) unready nodes is greater than XX%. Something is broken, possibly due to network partition or generic failure. Indicating factors:
-- >XX% of nodes are unready
Suggested action: halt operations.

cc: @fgrzadkowski @jszczepkowski @piosz @mumoshu @andrewsykim @kubernetes/sig-autoscaling-misc

UC3 appears two times, for two different cases

@mwielgus can you shoot me a message when the main code for this is ready? I don't mind beta testing it on some AWS clusters I'm running.

@andrewsykim Sure, probably early next week.

I was testing out these changes (which totally fix an issue we've been having with spot instances in a group, so thank you!) and found a bug but not sure what the best way to fix it will be.
I was getting the log messages:
I0124 02:36:50.508141       1 cluster_autoscaler.go:313] 8 unregistered nodes present

Did some digging and found getNotRegisteredNodes added some logging:
W0124 02:36:49.432285       1 clusterstate.go:451] A ProviderID is aws:///us-west-2c/i-0887029c6d9c57ea0
W0124 02:36:49.471646       1 clusterstate.go:455] A Node is i-016c0fc5668d72f0c

So the !registered.Has(node) on L#458 is comparing
aws:///us-west-2c/i-0887029c6d9c57ea0 == i-016c0fc5668d72f0c
which explains the log message. I didn't wait to find out but I assume they'd all get deleted when the unregistered node timeout hits.
I would just submit a PR for this but I'm not sure (design-wise) where this should live.

Changing AwsCloudProvider.Nodes() to return provider id's seems wrong.
I could use node.Spec.externalID (which is just the instance ID) but I don't know if that's true for the other providers and externalID is deprecated.
Adding special case handling for the provider ID for AWS seems wrong. And again, I don't know if the other providers follow the pattern <ProviderName>://<ProviderRegion>/<ProviderSpecificNodeID> though with that said, I feel like the other providers are failing since they should at least be in the format <ProviderName>://<ProviderSpecificNodeID> (unless they're returning that from their respective Nodes() calls.

@mwielgus any progress on this? Any way I can help move this forward?

The feature seems to be completed. Closing the issue.


Invalid Ion Timestamps

Issue #44
Fix malformed Ion timestamps.
Fixes include:

Year, month, day must be larger than 0 (minimum date is 0001-01-01)
Months cannot be larger than 12.
Days, depending on which month, should be either 28,29,30 or 31.
All different valid formats should be parsed correctly.
Timestamps with Hour, must have minute too.
Parse leap years properly.

This fix only aims invalid timestamps. There are still issues with timezone and fractional seconds which are not part of this fix (marked in the code as TODO)
The only outstanding invalid timestamp test in malformedIonsSkipList is timestampLenTooLarge.10n

Codecov Report

Merging #50 into master will increase coverage by 0.21%.
The diff coverage is 87.03%.

@@            Coverage Diff             @@
##           master      #50      +/-   ##
==========================================
+ Coverage   75.83%   76.05%   +0.21%
==========================================
  Files          22       22
  Lines        4573     4610      +37
==========================================
+ Hits         3468     3506      +38
+ Misses        681      680       -1
  Partials      424      424

Impacted Files
Coverage Δ

ion/textutils.go
81.32% <84.44%> (+3.12%)
:arrow_up:

ion/bitstream.go
78.35% <100.00%> (+0.31%)
:arrow_up:

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 c6d91a2...70c21c6. Read the comment docs.

Thanks Zack and David for the review.
Updated the pull request.


Add angular loading bar

As a user
I want a visual indication when the system is loading/sending Ajax data
So that I know something is happening in the background

I got this idea from JHipster.  Interesting note, on JHipster it fires all the time, but you don't see it with a node backend...demonstrates the difference between a node backend and a Java backend.


add plot util for getting voltage angle

so the functions should be named get_voltage_magnitude_series and get_voltage_angle_series?

so the functions should be named get_voltage_magnitude_series and get_voltage_angle_series?

Yes please


On-Premises suppport

Reading this article (maybe on a to high level for me) I don't understand if its possible to use this read-scale-out feature from an on-premises SQL DB to an Azure SQL DB. The application I'm thinking of is to set up an DB in azure that is optimized for Direct Query from Power BI to do real-time queries for a ticketing system

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

ID: 7a2961d1-4448-11b7-2dc2-3b738bd70939
Version Independent ID: 206eb022-25c4-66d7-7285-0d53a84956e3
Content: Azure SQL Database - read queries on replicas
Content Source: articles/sql-database/sql-database-read-scale-out.md
Service: sql-database
GitHub Login: @anosov1960
Microsoft Alias: sashan

@erapade Thanks for the feedback! We are currently investigating and will update you shortly.

@erapade Please note that when you enable Read Scale-Out for an Azure SQL database, the ApplicationIntent option in the connection string provided by the client dictates whether the connection is routed to the write replica or to a read-only replica. Here the client can be on premise database or PowerBI and if the client you are choosing has an option to modify the connection string to include ApplicationIntent option it should work.

@erapade I am afraid that it is not possible. This is Azure SQL Database feature, so when Azure database is created with Premium tier, AlwaysON replicas are created. And this Read Scale-Out feature allows you to load balance SQL Database read-only workloads using the capacity of one of the read-only replicas instead of sharing the read-write replica.

@erapade We will now proceed to close this thread. If there are further questions regarding this matter, please open a new issue and we will gladly continue the discussion.


Upgrade to Tika 1.9

This will solve the problem found in issue #31 by upgrading to the most recent version of Tika.

@Erol it would be awesome to see this merged in :)

@Erol, so, can we get this pull request merged?

@jipis do you think it's worth upgrading to tika 1.10? @Erol what do you think about merging this in?

Did @Erol  pass away or something? How can we get Tika upgraded?

@jasonperrone I'm very much alive, thank you. Just been very busy. My apologies if this took too long to get merged.

LOL. Glad you're alive, Erol :)
So, I actually ended up forking your project and upgrading Tika all the way to 1.11. And guess what? It didn't help the PDF parsing at all. So, I switched to PDF Reader.


Portals not starting anymore

07.09 22:33:06 [Server] ERROR Error occurred while enabling vane-portals v1.2.1 (Is it up to date?)
07.09 22:33:06 [Server] INFO org.oddlama.vane.external.json.JSONException: A JSONObject text must begin with '{' at 0 [character 1 line 1]
07.09 22:33:06 [Server] INFO at org.oddlama.vane.external.json.JSONTokener.syntaxError(JSONTokener.java:507) ~[vane-core-1.2.2.jar:?]
07.09 22:33:06 [Server] INFO at org.oddlama.vane.external.json.JSONObject.<init>(JSONObject.java:222) ~[vane-core-1.2.2.jar:?]
07.09 22:33:06 [Server] INFO at org.oddlama.vane.external.json.JSONObject.<init>(JSONObject.java:406) ~[vane-core-1.2.2.jar:?]
07.09 22:33:06 [Server] INFO at org.oddlama.vane.core.persistent.PersistentStorageManager.load(PersistentStorageManager.java:111) ~[vane-core-1.2.2.jar:?]
07.09 22:33:06 [Server] INFO at org.oddlama.vane.core.module.Module.load_persistent_storage(Module.java:346) ~[vane-core-1.2.2.jar:?]
07.09 22:33:06 [Server] INFO at org.oddlama.vane.core.module.Module.onEnable(Module.java:176) ~[vane-core-1.2.2.jar:?]
07.09 22:33:06 [Server] INFO at org.bukkit.plugin.java.JavaPlugin.setEnabled(JavaPlugin.java:264) ~[patched_1.17.1.jar:git-Paper-"6fa1983"]
07.09 22:33:06 [Server] INFO at org.bukkit.plugin.java.JavaPluginLoader.enablePlugin(JavaPluginLoader.java:370) ~[patched_1.17.1.jar:git-Paper-"6fa1983"]
07.09 22:33:06 [Server] INFO at org.bukkit.plugin.SimplePluginManager.enablePlugin(SimplePluginManager.java:500) ~[patched_1.17.1.jar:git-Paper-"6fa1983"]
07.09 22:33:06 [Server] INFO at org.bukkit.craftbukkit.v1_17_R1.CraftServer.enablePlugin(CraftServer.java:535) ~[patched_1.17.1.jar:git-Paper-"6fa1983"]
07.09 22:33:06 [Server] INFO at org.bukkit.craftbukkit.v1_17_R1.CraftServer.enablePlugins(CraftServer.java:449) ~[patched_1.17.1.jar:git-Paper-"6fa1983"]
07.09 22:33:06 [Server] INFO at net.minecraft.server.dedicated.DedicatedServer.init(DedicatedServer.java:277) ~[patched_1.17.1.jar:git-Paper-"6fa1983"]
07.09 22:33:06 [Server] INFO at net.minecraft.server.MinecraftServer.x(MinecraftServer.java:1212) ~[patched_1.17.1.jar:git-Paper-"6fa1983"]
07.09 22:33:06 [Server] INFO at net.minecraft.server.MinecraftServer.lambda$spin$0(MinecraftServer.java:319) ~[patched_1.17.1.jar:git-Paper-"6fa1983"]
07.09 22:33:06 [Server] INFO at java.lang.Thread.run(Thread.java:831) [?:?]

At one point I also had an issue where I entered a portal and I came out in a completely different location but I'm assuming this is unrelated. Thank you!

It doesn't happen everytime and restart generally fixes it, but that shouldn't be necessary in the first place, right?

Hmm thats indeed wierd, as your error message suggests a corrupt storage.json file, and therefore should persist over restarts. It is possible that vanilla minecraft teleported you somewhere near 0 0 0 when the plugin wasn't loaded. Did you experience any unclean system shutdowns, which could have caused the corruption?

Support said there was a DDOS attack at one point and server did freeze up once where I had to kill it - that might be why ;; That was the previous host though, this right here is new - I've deleted the entire config since I haven't changed any parameters and for now it's working again.

Ah that's quite unlucky, and I'm glad it works again. Although the storage.json contains the locations of all actual portals and all persistent state similar to that, so my expectation was that you should see the error until you delete / fix that file.
Anyhow, there is a small timeframe where a server crash is critical, when these files are saved. I will make the saving more robust in the future so this doesn't happen again, even when the server crashes in that unfortunate timeframe.


Regarding the "Go to current challenge" button on the homepage

Affected page
The Freecodecamp homepage
Description
I had started the JavaScript Algorithms and Data Structures course on the website and completed the module of Basic Javascript in it but even when I press the Go to current challenge button

It redirects me to the already finished challenge and not the current one which is the start of es6
This is not a major issue but just wanted to inform you!

Thank you for reporting this issue.
This is a standard message notifying you that this issue appears to be very similar to issue #43453, so we are closing it as a duplicate.
If you think we're wrong in closing this issue, please request for it to be reopened and add further clarification. Thank you and happy coding.


xADRecycleBin: Deleted Unneeded Example and Resource Designer files

Pull Request (PR) description
This PR removes the unneeded Example and Resource Designer files from the xADRecycleBin DSCResource folder.
There is already an xADRecycleBin example in the Examples folder.
This Pull Request (PR) fixes the following issues

Fixes #334

Task list

[ ] Added an entry under the Unreleased section in the CHANGELOG.md.
Entry should say what was changed, and how that affects users (if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[ ] Unit tests added/updated. See DSC Resource Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Resource Testing Guidelines.
[ ] New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

This change is

Codecov Report

Merging #366 into dev will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff         @@
##            dev   #366   +/-   ##
===================================
  Coverage    91%    91%
===================================
  Files        19     19
  Lines      2310   2310
  Branches     10     10
===================================
  Hits       2115   2115
  Misses      185    185
  Partials     10     10


Update cdn-map-content-to-custom-domain.md

This FQDN contoso.azureedge.net resolves to an IP address that is not owned by Microsoft.

@duongau
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Not an isssue: How can I catch error codes?

I know this is not a support forum, so sorry for this:
How can I catch error codes with this library?

NVM I figured it out, using error.status


prepress email links

Hello
When the media is set as prepress, email addresses appear as:
<EMAIL_ADDRESS><EMAIL_ADDRESS>or
my email<EMAIL_ADDRESS>The<EMAIL_ADDRESS>or 'my email' are clickable links in the PDF which is good. Then the [brackets] appear because it is prepress, and that is good too. However the mailto: part is not really something that makes sense in the context of emails. Just having the email address - without the mailto part - I think, would be better .
thank you!
a

The reason this was done is to emulate the behavior of the DocBook toolchain (specifically Apache FO to PDF). The mailto: prefix indicates that the link was added to the document as a mailto: URI to distinguish it from a URL.
We could consider an attribute such as hide-mailto-scheme to force it to not be shown.


testing keyframe internals

Keyframes.Transition doesn't exist any longer, Transition itself now uses Keyframes internally. The mount status looks unusual to me, wouldn't doing it with setState cause sideeffects since setState is async? Calling it on componentWillUnmount for instance will potentially let callbacks that fire in this moment still allow to work because the mounted=false flag will either come never (not sure if React fires setState after cWU or later.

Haha, you're so right. I'll nuke this one. This was a PR for 5.x


Fix dubbo select local server error

Fix dubbo select local server error

Pull Request Test Coverage Report for Build 100

0 of 0   changed or added relevant lines in 0 files are covered.
6 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.2%) to 38.388%

Files with Coverage Reduction
New Missed Lines
%

protocol/dubbo/dubbo/codec.go
2
4.36%

pkg/metrics/prometheus_exporter.go
4
95.06%

Totals

Change from base Build 94:
-0.2%

Covered Lines:
924

Relevant Lines:
2407

💛  - Coveralls


Active Auras - dae - Libwrapper conflict

Using latest active auras and latest dae.  Getting a conflict from libwrapper.

+1
Seeing the same warning


Error: console.warning is not a function

Seems like when a buffer isn't found, this error gets thrown and can cause js to stop execution.
I've opened a PR here: https://github.com/processing/p5.js-sound/pull/378
It would be nice to get this merged in as we are using the CDNs and have had to replace them with local js files for now =)

@therewasaguy
i think this issue mus be closed now !


 Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:

/needsMoreInfo but probably relates to an extension, in which case it should be reported there.


Plans for 2.1.0?

Thanks a lot for developing such a great piece of software! :)
I've noticed you have lot's of tasty stuff in master, however, I'm a bit scared to use the master branch for building our docker images and deploying in PROD.
Thus, I'm curious if you have any plans for releasing 2.1.0 or whatever the next version number would be.

pghoard 2.1.0 has been released May 27, 2019.


⚠ Bofrost.at has degraded performance

In ea50fcc, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1662 ms

Resolved: Bofrost.at performance has improved in cb4c95e.


Unable to get new versions

Are new releases expected to appear in the Rider plugin menu? My Rider is stuck on 1.4 and doesn't get the new updates.

Unfortunately, the only way to update is to do it manually. I can't do anything on the plugin side. This should be fixed in the next 2025.1 release when the plugin is properly bundled with Rider.


fix: ensure that CQADupstackRetrieval is included in results if possible

Example from running mteb create_meta with new code:
[...]
  - dataset:
      config: default
      name: MTEB CQADupstackRetrieval (default)
      revision: CQADupstackRetrieval is a combined dataset
      split: test
      type: CQADupstackRetrieval is a combined dataset
    metrics:
    - type: main_score
      value: 46.40058333333334
    - type: ndcg_at_10
      value: 46.40058333333334
    task:
      type: Retrieval
[...]

Checklist

[x] Run tests locally to make sure nothing is broken using make test.
[x] Run the formatter to format the code using make lint.

Oh and one more question, did we decide to keep the (default) in the metadata? Do we still need to fix this: https://huggingface.co/spaces/mteb/leaderboard/discussions/132#668efce065639f1fd9cc3fee ?

Oh and one more question, did we decide to keep the (default) in the metadata? Do we still need to fix this: https://huggingface.co/spaces/mteb/leaderboard/discussions/132#668efce065639f1fd9cc3fee ?

This does not fix this case no. I think we should change the leaderboard to allow for default (however that might be harder than simply changing it here? WDYT?

Also maybe we should leave in the merging script at least for some time in case someone wants to merge older results or there are scenarios where the automatic merging is not triggered?

I believe the new should work for older cases as well (due to the backward compatible loading). but I am fine with leaving it in as well

Nice! 🙌 Maybe it would be good to also automatically merge it when evaluating not via the cli but importing mteb in Python? So instead of specific to the cli, maybe it should be

Seems like this was never finished, not quite sure what you mean?

I believe the new should work for older cases as well (due to the backward compatible loading). but I am fine with leaving it in as well

Nice if it works for older cases to (without having to necessarily run an evaluation but just merge as is) then that's  fine I think! Should it be mentioned somewhere here: https://github.com/embeddings-benchmark/mteb/blob/main/docs/adding_a_model.md#adding-a-model-to-the-mteb-leaderboard ?

Nice! 🙌 Maybe it would be good to also automatically merge it when evaluating not via the cli but importing mteb in Python? So instead of specific to the cli, maybe it should be

Seems like this was never finished, not quite sure what you mean?

Sorry 😅 I meant if people are evaluating via MTEB in Python then it wouldn't be triggered right? So maybe it should check everytime a CQADupstack task is run or something rather than everytime the CLI is used?

This does not fix this case no. I think we should change the leaderboard to allow for default (however that might be harder than simply changing it here? WDYT?

Yeah it seems like changing it here is simpler I think

Sorry 😅 I meant if people are evaluating via MTEB in Python then it wouldn't be triggered right? So maybe it should check everytime a CQADupstack task is run or something rather than everytime the CLI is used?

Aah right, yes you can't run "CQADupstackRetrieval" within using mteb (cli or python). But if you run all its constituents mteb create_meta will create the aggregate score.
I have considered implementing it as a "benchmark" and then you get a score pr. benchmark similar to a task. Then we can also have that a 'benchmark' as a task within another benchmark, but that requires a lot more utility handling for benchmarks than we currently have (though in the future I would love to add it, e.g. allow benchmarks to implement their own aggregations scores)

Sorry 😅 I meant if people are evaluating via MTEB in Python then it wouldn't be triggered right? So maybe it should check everytime a CQADupstack task is run or something rather than everytime the CLI is used?

Aah right, yes you can't run "CQADupstackRetrieval" within using mteb (cli or python). But if you run all its constituents mteb create_meta will create the aggregate score.
I have considered implementing it as a "benchmark" and then you get a score pr. benchmark similar to a task. Then we can also have that a 'benchmark' as a task within another benchmark, but that requires a lot more utility handling for benchmarks than we currently have (though in the future I would love to add it, e.g. allow benchmarks to implement their own aggregations scores)

I guess if create_meta does the aggregation it is fine for now 👍 We'll see if people run into issues!


Notifying updates

Hello,
I think a "Download new version" button would be nice.

Hi, already implemented. The software checks every 8 hours for a new version, if available a link "new version available" should appear on the top menu.


Mouse not recognized upon reset

Hey,
I have two Lenovo mice of the same type (MSU1175) which are not working when I start my SPARCstation IPX.  If I unplug and re-plug them, they work.  I hear one confirmation tone when the adapter initially starts - Would I head two separate tones for keyboard and mouse if they'd both be recognized upon power-up?
I have suspected that the problem might be a startup-up timing issue, but adding a second of delay in setup() or loop() did not solve the issue.
On one occasion, I found that only the mouse was enumerated, but not the keyboard: I re-plugged the usb3sun with both USB devices connected and heard one device confirmation tone.  The keyboard did not work, however.  When I unplugged and re-plugged it, it worked.
Does enumeration of both the mouse and keyboard at startup usually work, or is this a known issue?
Thanks,
Hans

The confirmation tones can overlap, depending on USB enumeration timing, so you may only hear a longer-than-usual low tone followed by a high tone when multiple devices are connected.
The underlying USB stack has often had bugs that cause problems when enumerating some devices. For example, the Microsoft Wired Keyboard 600 (045E:0750) and 400 (045E:0752) needed a delay to work, while the WMO 1.1a (045E:0040) still fails to enumerate occasionally. Do you have the same problems with a different mouse and/or keyboard?
By the way, can you connect your keyboard and mouse to a Linux machine, and attach the output of lsusb -v?

I've verified that the problem only occurs with the Lenovo mouse, no matter what keyboard is connected.  The other mice that I tested work right away.
Here's the lsusb output for the mouse:
Bus 001 Device 028: ID 17ef:6019 Lenovo M-U0025-O Mouse
Device Descriptor:
  bLength                18
  bDescriptorType         1
  bcdUSB               2.00
  bDeviceClass            0
  bDeviceSubClass         0
  bDeviceProtocol         0
  bMaxPacketSize0         8
  idVendor           0x17ef Lenovo
  idProduct          0x6019 M-U0025-O Mouse
  bcdDevice            1.00
  iManufacturer           1 PixArt
  iProduct                2 Lenovo USB Optical Mouse
  iSerial                 0
  bNumConfigurations      1
  Configuration Descriptor:
    bLength                 9
    bDescriptorType         2
    wTotalLength       0x0022
    bNumInterfaces          1
    bConfigurationValue     1
    iConfiguration          0
    bmAttributes         0xa0
      (Bus Powered)
      Remote Wakeup
    MaxPower              100mA
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        0
      bAlternateSetting       0
      bNumEndpoints           1
      bInterfaceClass         3 Human Interface Device
      bInterfaceSubClass      1 Boot Interface Subclass
      bInterfaceProtocol      2 Mouse
      iInterface              0
        HID Device Descriptor:
          bLength                 9
          bDescriptorType        33
          bcdHID               1.11
          bCountryCode            0 Not supported
          bNumDescriptors         1
          bDescriptorType        34 Report
          wDescriptorLength      46
         Report Descriptors:
           ** UNAVAILABLE **
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x81  EP 1 IN
        bmAttributes            3
          Transfer Type            Interrupt
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0004  1x 4 bytes
        bInterval              10
Device Status:     0x0000
  (Bus Powered)

Thanks! I’ve added this to the compatibility list. To help me diagnose the problem, can you flash the firmware below and attach the output you see on the UART_TX header? Set your UART to 115200 8N1, and be sure to connect both UART_TX and GND. If flashed correctly, your display will say “5a88d79f53.01” and the Sun keyboard interface will be disabled.
https://bucket.daz.cat/usb3sun/5a88d79f53.01[115200%2C8N1%2CUHID_VERBOSE%2CCFG_TUSB_DEBUG%3D3].uf2

Here is the log file.  WIth the debug firmware, the timing seems to be different enough to make the enumeration at power-up somewhat work, although the tones don't sound quite right to me.  I recorded videos with the debug and the non-debug firmware for comparison:
ZOC2311__dev_tty.usbserial-1221200309.log
https://photos.app.goo.gl/pNuCwReeYDb4q52j6 (debug firmware)
https://photos.app.goo.gl/WJcabqi5TXRbQhb89 (non-debug firmware)

Hi Delan, I'm seeing the same issue with other keyboard/mouse combinations, so it does not seem to be a specific incompatibility.  How can I help diagnosing this further?

This smells a lot like a previous tinyusb bug with the Microsoft Wired Keyboards 600 and 400, which I found by removing TU_LOG2 and TU_LOG3 lines until I could reproduce the problem, then inserting a delay accordingly (see tinyusb2.patch).
I can try upgrading tinyusb and pico pio usb again next weekend, since they are currently held back due to regressions that I haven’t yet investigated. In the meantime, you can try binary searching the log messages until you find a place where we can try inserting a delay, like I did for those Microsoft keyboards.
If you haven’t already tried the prebuilt firmware 1.5, be sure to try that first in case that solves your problem.

I've tried the 1.5 release binary and with that, the problem does not exist.  That puzzles me because I have built the firmware from the default branch, and there are no differences in the firmware compared to 1.5.  I'm using a Mac, but I'm not sure how that'd matter.  In any case, I think the problem is solved for me now.

I wonder if there’s a gap in our build process causing a build reproducibility problem. Did you remember to apply the two patches in the firmware docs? Can you tell me:

the sha256 of the firmware.uf2 or firmware.elf you built
what commit you built it from
the output of pio pkg list

I'm more inclined to think that I have a gap in my documentation reading process.  With the two patches applied, the firmware I built works just fine.  I feel particularly stupid because of this:

Thank you for your help!


[FEATURE REQ] Support for Retrieving Secrets from Key Vault with Specific Secret Version

Library name
Azure.Extensions.AspNetCore.Configuration.Secrets
Please describe the feature.
Currently, when retrieving secrets from the Key Vault, there is no option to specify a particular version of a secret.
This limitation makes it challenging to access historical versions or specific iterations of a secret when needed for rollback scenarios, auditing, or compliance purposes.
It would be highly beneficial to have the ability to pass the secret version alongside the name to retrieve the correct data

Hi @nsztyk.  Thanks for reaching out and your suggestion.  As the extensions are meant to integrate with IConfiguration which does not have the concept of versioning, I'm not sure that this request is feasible.  Given that IConfiguration represents settings using only a string-based key, in order to include version information, we would likely need to create some custom notation that embeds a secret name and version as the configuration item name, such as "MyThing:1234", which deviates from the standard patterns that we're integrating with.
Is there a reason that you're unable to use the KeyVault client library directly where you need versioned secrets?  Was there a specific way to pass version information that you had in mind, which I'm potentially overlooking?

Hi, @jsquire, thanks for your quick response. I understand that IConfiguration doesn't support versioning natively. However, my main goal is to specify the secret's version only at the time of retrieval from Key Vault. Once the secret is fetched, I don't need the version information anymore; I just want to use the secret value under its standard key.
For example, I'd like to specify something like MyThing@old in the Appsettings file. This way, the application can download the 'old' version of the MyThing secret from Key Vault and store it in IConfiguration under the key MyThing, without any version suffix.
Additionally, my aim is to coordinate the deployment of the application so that under the same key in IConfiguration, I can have one version of a secret, and later, by changing, for example, the Appsettings file, I can run the application using a different version of the secret.

@jsquire to add to what @nsztyk wrote I'll exaplin that our team is already using a custom implementation of KeyVaultSecretManager to decide which secrets to load, which has access to full SecretProperties and can in fact filter based on the secret version (or so it seems from the signature):
public override bool Load(SecretProperties secret)
{
   return secret.Name == "mysecret" && secret.Version == "myversoin";
}

We have our own way of specifying the names and versions to load, which is based on IConfiguration coming in from the CreateDefaultBuilder, and then we configure the Key Vault provider inside of ConfigureWebHostDefaults(webBuilder => webBuilder.ConfigureAppConfiguration(...)) - but that is irrelevant, you can assume names & version are hardcoded for the purpose if this discussion, even if that seems impractical. In any case we are not looking for any new notation inside of IConfiguration.
The actual issue comes later, where an inconsistency seems to manifest between the Load method's contract and what happens next in AzureKeyVaultConfigurationProvider.AddSecretToLoader - after Load has been queried only the secret's name is passed on down the chain, and then the usual logic of getting the latest version kicks in:
private void AddSecretToLoader(SecretProperties secret, Dictionary<string, KeyVaultSecret> oldLoadedSecrets, Dictionary<string, KeyVaultSecret> newLoadedSecrets, ParallelSecretLoader secretLoader)
{
    if (!_manager.Load(secret) || secret.Enabled != true)
    {
        return;
    }

    var secretId = secret.Name;
    ...
}

So you see, bool Load(SecretProperties secret) is a bit misleading in that in the end it actually behaves like bool Load(string secretName).
What we propose is to pass the entire SecretProperties object down the chain and load by Id or pass down Name+Version.
There might be some backward compatibility issues here though - I'm not sure how to combine this with the existing version selection logic, as consumers will already depend on it. Perhaps something like bool LoadSelectsVersions { get; } would be needed in KeyVaultSecretManager to enable the new code path with Name+Version, otherwise Version would be left null for autoselection down the chain.

Thanks for the additional context, @nsztyk and @TripleEmcoder.  Given that this library is intended to be a straight-forward extension of IConfiguration which lacks any concept of versioned configuration, the additional complexity that this would introduce for callers, and the non-trivial backwards compatibility impact, I don't think that this is a feature suggestion that we'll be moving forward with.
For versioned scenarios, we would recommend using the KeyVault SDK explicitly rather than this extensions package.


Multiple tooltip styles?

Is it possible to have multiple styles for tooltips without a way to set a class on the tooltip?
For instance, if I need red tooltips in one part of the app and blue tooltips in another?

Good idea! IMO it's easiest to provide a way of setting custom classes (styles) on a specific tooltip:
..

What do you think?

Thatd would be awesome!
Ive seen other packages use this method (e.g. angular's ng-bootstrap tool
tips)
On Mar 7, 2016 2:46 PM, "Johan Brook"<EMAIL_ADDRESS>wrote:

Good idea! IMO it's easiest to provide a way of setting custom classes
(styles) on a specific tooltip:
..
What do you think?
—
Reply to this email directly or view it on GitHub
https://github.com/lookback/meteor-tooltips/issues/14#issuecomment-193419349
.

Hi!
Wanted to check in to see if there was any plans to implement this feature in the near future?

It could be: PRs are welcomed :)

I will try


React is not defined

Describe the bug
In a very simple test for a component I keep getting React is not defined
To Reproduce
Run test in the UI for a component
Expected behavior
Run test successfully. The plugin works ok with non react tests. The react tests run without any issue in the console.
Screenshots

Environment

OS: macOS
VSCode version: 1.70.2
Vitest version: 0.22.1
Vitest plugin version: 0.2.24

Probably related to #13, but with my example it'll work fine until monorepo is supported.
If you have a monorepo, the command should be started with the help of your package manager. Here is an example with npm:
"vitest.commandLine": "npm -w @monorepo/package-name test --"

Put this line in .vscode/settings.json and it should work. Can't give yarn/pnpm examples but it's the same for all.
Also the package's package.json should contain the script you are calling, in this case is test:
"scripts": {
    "test": "vitest",
}

@Pustinyak Thanks for the solution, I changed it a little bit for yarn, and works fine in one monorepo
"vitest.commandLine": "yarn workspace @monorepo/package-a test --watch"

but I also have a package-b in this monorepo, vitest.commandLine is specific with package-a already,
I tried, it failed in package-b.
If you have any idea about this, please.
Thanks in advance.
Ken

You can try using the vscode workspace to open folders that require different configs. Each folder can have its own .vscode settings.