Adding the pod logs

(yet another follow-up to the conversations that triggered #44 + #47 + #48 - last one for now)
Our customer would like to correlate pod logs, this is potentially easy done in OpenShift as you can take the pod info from the dump and search in the logging system. However, because our customer is a vendor the cluster logging system might not be accessed by the respective tenants very easily.
Another "easy" solution would be to include the pod log (up until the crash) into the zip file. My naive thought would include a simply cp from the configured log directory on the host. Alternatively this could be done with kubectl log going through the API server.
What's your take on the story?

For the cp approach there are lot of different logging variations that would have to be catered for in order for it to be generalised:

Node logging agent
Streaming sidecar container
Sidecar container with a logging agent
Exposing logs directly from the application
https://kubernetes.io/docs/concepts/cluster-administration/logging/#cluster-level-logging-architectures
I'm not convinced this is the right approach.

I would also be a bit hesitant on querying the container through the API as the pod may be an unknown state given that the process has SIG'ed. The permissions aspect will also need to be worked out as the kubeapi will be from the core composer on the host. But it may be worth investigating this as it would be more generally applicable if it works.
This is coupled with https://github.com/IBM/core-dump-handler/issues/47#issuecomment-1012293670 makes me think they may be better with a post processor but I'm open to keep discussing options.

Moved the crictl code into a specific library for reuse in another project and added a log method to it.
https://docs.rs/libcrio/latest/libcrio/struct.Cli.html#method.logs
I'll integrate that in on the next release

Initial implementation here https://github.com/IBM/core-dump-handler/commit/de95956082a7638b4cd630598fe156f22612619f
On reflection shipping the whole log is not optimal and could potentially timeout the dump.
I'm going to switch this to be a tail command where the default size of the tail will be 500 lines (Subject to validation)
See https://github.com/No9/libcrio/issues/1

Moving this to done but keeping this open to confirm discussion on log tail approach.

I think the tailing is good enough already. Thanks so much for your work @No9 :)

Great - I have an initial cut of the tail feature on this branch - https://github.com/IBM/core-dump-handler/tree/pod-logs
It has your openshift.yaml work and points to an specific release candidate so you should be able to run it using the usual steps.
I've also updated the segfaulter container to generate a 1000 log lines before crashing.
Can you give it a spin when you have time and confirm you are getting 500 lines in the log file that is in the zip that would be great.
Also any feedback on the docs updates would be great.
If we are good on those bits I just need to add more hardening to the agent CI tests and I'll cut a release.

just tested this again with OpenShift 4.9 on AWS. It works like a charm :) Thanks a lot once more, @No9.

Also any feedback on the docs updates would be great.

do you want to create a PR maybe? happy to review it through there

Thanks for looking at it @tjungblu
I've PRd the whole change set here https://github.com/IBM/core-dump-handler/pull/53
Closing this as we've validated the log collection.


UTF-8 characters

Hello :)
I was trying this package out and I noticed that if I use any of the accentued characters from UTF-8 the files do not render correctly on the Filemanager view but are uploaded correctly.
Here's an example:
File: chouriço.jpg
File uploads fine, but when the view is updated, a 404 gets thrown.
GET http://localhost/storage/chouri%C3%A7o.jpg 404 (Not Found)
All these characters behave the same:
À, Á, Â, Ã, Ä, Å, Æ, Ç, È, É, Ê, Ë, Ì, Í, Î, Ï, Ð, Ñ, Ò, Ó, Ô, Õ, Ö, ×, Ø, Ù, Ú, Û, Ü, Ý, Þ, ß, à, á, â, ã, ä, å, æ, ç, è, é, ê, ë, ì, í, î, ï, ð, ñ, ò, ó, ô, õ, ö, ÷, ø, ù, ú, û, ü, ý, þ, ÿ

Hi. Thanks for your issue. This will be fixed in the new release.
The upload method has been rewrited and now no more Dropzone dependency. Now it will work with drag&drop on the file manager. Also all file names will be sanitized before upload.
Expected to be released this week.

New version released. Please update to version 1.1.0, and check it.


Added warning for setupkind script

setupkind script can only successfully setup kind
with metallb in Linux environment. This PR will warn
a user if this script gets run in an envrionment other
than Linux
Signed-off-by: Tong Li<EMAIL_ADDRESS>Please provide a description of this PR:

@litong01: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
9d97afc281c82944dfb476f4c92a0ebd241d2430
link
true
/test integ-security-multicluster_istio

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

Also in the README change:
This bash script sets up a k8s cluster using kind and metallb.

to:
This bash script sets up a k8s cluster using kind and metallb on Linux.

WARNING: It does not work on macOS or Windows.

@litong01: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
58c175d1c97e4524f22d96d9d2ef5581e574d3d6
link
unknown
/test integ-security-multicluster_istio

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

@litong01: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
58c175d1c97e4524f22d96d9d2ef5581e574d3d6
link
true
/test integ-security-multicluster_istio

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

/retest


VKBridge.instance.init() call never returns a result

I'm facing an issue with initial call of VKBridge.instance.init() in my project when this call never returns a result and app never goes past this call. Turns out it's also reproducible with the example app from this repo.
Steps to reproduce:

Clone the repo
Run the sample app from vk_bridge/example folder:

flutter run -d web-server

Observe never ending loading progress bar.

Browser console output:
developer_patch.dart:75 registerExtension() from dart:developer is only supported in build/run/test environments where the developer event method hooks have been set.
_registerExtension @ developer_patch.dart:75
js_primitives.dart:30 vk_bridge.d: vk_bridge: _sendInternal(VKWebAppInit)
js_primitives.dart:30 vk_bridge.d: send(VKWebAppInit, {})
js_primitives.dart:30 vk_bridge.d: _eventHandler: {"type":"vk-connect"}

According to console output above, print('VKBridge.init: $result'); line is never executed.

To start VK Bridge, you need to open your app through the VK Mini App. Create a new application, there set up a debug address to your local machine and everything will work

Thanks, I'll give it a shot.
Is there a way to tell if Flutter app is running as a regular web app or inside Vk? For instance, I have a Flutter app that works both web app on my own web site and I want to make it work on Vk with extra features using Vk Mini Apps API. The only idea I've got at the moment is to use different flavors for each of these web apps.

In my project, we thought of the following solution:
We build the application with a flag by which we can determine that this application is built for VK flutter build web --release --dart-define=IS_VK_MINI_APPS=true
Then in the code we can get the value as follows:
static const isVKMiniApps = bool.fromEnvironment(
        'IS_VK_MINI_APPS',
        defaultValue: false,
    );

Looks like the root cause of the initial issue was not running the app inside Vk Mini Apps. Using Vk Tunnel to run the app confirms that it works as expected.


Insane public_suffix version requirement

Why not relax the version requirement a little bit? All you're using is PublicSuffix.parse which is the most core possible thing in the library. I don't see any way the author would change the API of this method without bumping the major version (to be honest I don't even see him changing it at all, ever). And this method has existed in public_suffix since forever. So why not make it "< 2.0"?

I'll accept a pull request that changes it to '~> 1.0'

lol fine https://github.com/blatyo/page_rankr/pull/49

Version 4.6.1 has your change. By the way, the reason I said '~> 1.0' is because PublicSuffix < 1.0.0 was called PublicSuffixService, which would break this gem.

:+1:


Empty call path

I installed<EMAIL_ADDRESS>with spack on a Ubuntu 22.04 workstation. For example, I wrote the following program
program prog

   implicit none

   integer, parameter :: dp  = selected_real_kind(15, 307)
   integer, parameter :: n = 1000
   real(dp) :: A(n,n), B(n,n), C(n,n)
   integer :: i, j, k

   call random_number(A)
   call random_number(B)

   C = 0.0_dp
   do i=1, n
      do j=1, n
         do k=1, n
            C(i,j) = C(i,j) + A(i,k) * B(k,j)
         end do
      end do
   end do

end program prog

and I execute hpctoolkit with the following commands
rm -rf hpctoolkit-*-measurements
hpcrun ./prog
hpcstruct prog.f90
hpcprof -S prog.hpcstruct -I $PWD hpctoolkit-prog-measurements

This works fine on several computers, but on the said workstation hpcrun doesn't seem to find anything for the call path, and  hpcviewer shows

Any idea what could be causing this?

Got it, solved it. Thanks.


macos-14 github actions debugging

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 52.62%. Comparing base (6a2a3fe) to head (6457380).

Additional details and impacted files
@@            Coverage Diff             @@
##           master       #6      +/-   ##
==========================================
- Coverage   52.65%   52.62%   -0.04%
==========================================
  Files          42       42
  Lines        5762     5762
==========================================
- Hits         3034     3032       -2
- Misses       2728     2730       +2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Debugging complete, new PR opened for the upstream fork. Closing this since it is no longer needed.


[sample] Update CognitiveServices.TextAnalytics to non preview

Description
Trying to use this sample against the live service will cause strange errors, which has bitten me twice. Easy enough to just update.
Checklist
To ensure a quick review and merge, please ensure:

[x] The PR has a understandable title and description explaining the why and what.
[x] The PR is opened in draft if not ready for review yet.

If opened in draft, please allocate sufficient time (24 hours) after moving out of draft for review

[x] The branch is recent enough to not have merge conflicts upon creation.

Ready to Land?

[ ] Build is completely green

Submissions with test failures require tracking issue and approval of a CODEOWNER

[ ] At least one +1 review by a CODEOWNER
[ ] All -1 reviews are confirmed resolved by the reviewer

Override/Marking reviews stale must be discussed with CODEOWNERS first

The diff without generate code is trivial - https://gist.github.com/chamons/d2c67fda973b45d8b6e2d51fc3fe51f3


Keep top menu bar visible

The top bar disappears when scrolling down. Keep it visible.

@mariobehling Can I work on this? 😄

Please ensure top bar is semi-transparent as soon as scrolling down - also on red top area.

@mariobehling But, It is already semi-transparent

Please compare on the FOSSASIA main site. As soon as you start scrolling down the transparency of the top bar changes. On the PSlab site the menu bar always appears completely transparent for me on Chrome.

Ah ok. I see it is changing as soon as the white area is appearing. Could you adopt it, so the top bar changes when the scrolling starts, please?

@mariobehling I would like tow ork on the same pls


Removed 'archived' property and added 'in_trash' property.

Description
The Archived properties defined for Block, Page, and Database objects have been deprecated and replaced with new InTrash properties. The API reference has not yet been edited to the new InTrash in some places (block update page), but all API responses seem to have been adapted to InTrash.Migration from Archived to InTrash properties has no functional differences other than the name change, so simply replacing them will cause all existing APIs to work properly.
I have also modified the test code for the areas applicable to the change.
Fixes #410
Type of change

[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?
Unit testing was successfully completed and the various APIs (query, update, search) affected by the change were tested in Notion.
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

Comment
If you need to maintain backward compatibility and support both Archived and InTrash properties, please contact us again!

Sorry. I forgot to add the renamed TrashPageResponse.json and had to push it back.


Refactor failed messages graph

Current failed messages x -axis is per month rather than per day like the rest of the graphs. The title too is /hr rather than /10 Mins / Per Day.

continues at #173


fix: wrong serializer for worker nodes

This isn't needed yet,  I accidentally update rq to a newer version.


FAQ collections

don't merge -- use @goodcodeguy's PR instead and just leave this for reference on tagging


feature request: free form pose movement

like the work so far, but setting a pose using radials to rotate each bone is cumbersome - great for fine tuning, but not that great to set the general pose.
suggestion is to have free-move mode where click & drag moves a bone on two prominent axes and lock the third axis
for example:

if using front view, it mouse moves on x/y and z is locked
if using side view mouse moves on y/z and x is locked

and determining which two axes are prominent can be determined from camera angle
second part of feature request is to actually utilize right click to add a popup menu - delete/duplicate/move-mode - they can all be embedded controls so main ui can be vastly simplified.
and entire camera params section can be removed - the full functionality is via mouse movement anyhow.

I will try to improve it, your advice is useful.

thanks!
fyi, i did something like that in an unrelated project (my older work, nothing to do with stable diffusion):
https://motioniq-ai.github.io/demo/prototype.html
load any image in inputs and you can freeform click-and-move any anchor points.

@vladmandic We update a lot recently. You can open "Move Mode" and "Free Mode" to move joints.

@ZhUyU1997 thanks for the heads-up and all the work - overall feels much better. the axis selectors are still a bit cumbersome - i think in most cases you shouldn't even need them?
right now you need to click on joint, release the button, then select axis to perform move on (free move or rotation-based), hold the buttom and perform the move.
but axis can be determined from camera angle, so entire workflow can be simplified to click and drag joint (without needing to declick, select axis and then click-and-drag). and in that case, you can even hide orbit controls (show them as optional in the menu) as they are not required to perform the move.

Move mode doesn't really address the original request, since it simply stretches the bone to a new location. It doesn't preserve length or rotate everything else.
The suggestion is that the behavior would be like free mode, rotating the joint and all the nodes under it in the hierarchy, but the way to get the rotation in the UI is not to rotate a circle on the joint, but to grab and move the next joint down (like move mode does) and cause that movement to be matched with a corresponding joint rotation on the next node up. This is how lots of posing software works, as it gives the closest to direct manipulation of joint rotation as possible, and feels nice and simple and intuitive.


Ambiguous example in registry_mirror.md

The syntactic coloration is a little bit ambiguous, I suggest this as a fix: new line for registry parameter.

Can you please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "patch-1" git@github.com:enguerran/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

This will update the existing PR, so you do not need to open a new one.

I like it. LGTM (not a maintainer)

LGTM

LGTM

Thanks, my first contribution to docker ^^


Fix loop issue with signing out

These changes fix the issue documented at #168 by changing session to sessionData inside of the useEffect dependency array.

Would love if I could get some eyes on this! We have a pending auth overhaul project using connectkit's SIWEProvider but this is currently preventing us from going all the way.


how do you set color for entire row?

@kunalprabhu21
Changing the current row color is possible through a callback like the link below.
https://weblaze.dev/pluto_grid/build/web/#feature/row-color
Changing the default row color without a callback should be added.
I will consider changing the default row color in the next version.

thanks for the reply

PlutoGrid 5.0.0 has been deployed.
There are other changes from version 4, so please check the ChangeLog.
https://pub.dev/packages/pluto_grid/changelog#500---2022-7-11
The demo page has also been updated.
https://weblaze.dev/pluto_grid/build/web/#/


Strange bonuses/penalties being applied when alternating (1/2/1) diagonal mode is enabled

Describe the bug
When alternating (1/2/1) diagonal mode is enabled, sometime strange bonuses/penalties are applied for some diagonal movement calculations
To Reproduce
Steps to reproduce the behavior:

Enable Alternating (1/2/1) diagonal mode in core foundry settings
Enable "Token Ruler" in Elevation Drag Ruler settings
Drag a token 2 spaces up, and 5 spaces to the left
See that movement cost is 25ft, because a -5ft bonus has been applied.
Drag a token 4 spaces up, and 3 spaces to the left
See that movement cost is 30ft, because a +5ft penalty has been applied

Expected behavior

If a token moves 2 spaces up, and 5 spaces to the left, the movement cost should be 30ft
If a token moves 4 spaces up, and 3 spaces to the left, the movement cost should be 25ft

Screenshots

Additional context

Foundry v2.331
dnd5e v3.3.1
Elevation Drag Ruler v1.10.1

Just discovered this is fixed by disabling the "Use Grid for Movement Penalties" setting.  I don't understand what this setting is supposed to be doing, but it doesn't seem correct to me.


Understanding How Masking Affects BCE Loss Function

Hi, I'm experimenting with masking and have come across an interesting observation. I've created a simple GRU-based model with zeroed-out weights and biases along with a simple X and Y tensor. Theoretically, because my weights and biases are set to zero I should get the same loss value for the same dataset regardless if masking is on or not. However, I find that masking affects my results and I'd like more clarification on why this is happening. Below is my code to reproduce the difference. Additionally, I am using version 2.4.1 and 2.4.0 for tensorflow and keras, respectively
import tensorflow as tf
from tensorflow.keras import layers
from tensorflow.keras import losses
from tensorflow.keras import regularizers
from tensorflow.keras import initializers
from tensorflow.keras import Model
from tensorflow.keras import backend as K
from tensorflow.keras.optimizers import Adam

GRU_SIZE = 50
DENSE_DIM = 200

# generate data
data = tf.random.uniform([2, 4, 4], maxval=2, dtype=tf.int32, seed=1)
data = tf.cast(data, tf.float32)
data = K.eval(tf.identity(data))

# zero-out specific timesteps in each patient
data[0, 2, :] = 0.0
data[0, 3, :] = 0.0
data[1, 2, :] = 0.0
data[1, 3, :] = 0.0

# use the 1st n-1 timesteps in X to predict the last n-1 timesteps in Y
x_dummy_data = data[:, 0:-1, :]
y_dummy_data = data[:, 1:, :]

# create the masked model
input_shape = (x_dummy_data.shape[1], x_dummy_data.shape[2])
inputs = layers.Input(shape=input_shape)
masking = layers.Masking(mask_value=0, input_shape=input_shape)
input_masked = masking.compute_mask(inputs)
output1 = layers.Dense(DENSE_DIM)(inputs)
output3 = layers.GRU(GRU_SIZE, return_sequences=True)(output1, mask=input_masked)
output4_code = layers.Dense(input_shape[1],
                                                   activation='sigmoid',
                                                   name="output_code")(output3)
model_mask = Model(inputs=inputs, outputs = output4_code)
model_mask.summary()
model_mask.compile(loss=tf.keras.losses.BinaryCrossentropy(reduction=tf.keras.losses.Reduction.SUM_OVER_BATCH_SIZE, from_logits=False),
                                     optimizer=Adam(learning_rate=1e-4))

# set weights and bias to zero
for ix, layer in enumerate(model_mask.layers):
     weights = layer.get_weights()
     for arr in weights:
        arr[(arr > 0) | (arr < 0)] = 0
     layer.set_weights(weights)

# create the unmasked model
input_shape = (x_dummy_data.shape[1], x_dummy_data.shape[2])
inputs = layers.Input(shape=input_shape)
output1 = layers.Dense(DENSE_DIM)(inputs)
output3 = layers.GRU(GRU_SIZE, return_sequences=True)(output1)
output4_code = layers.Dense(input_shape[1],
                                                   activation='sigmoid',
                                                   name="output_code")(output3)
model_unmask = Model(inputs=inputs, outputs = output4_code)
model_unmask.summary()
model_unmask.compile(loss=tf.keras.losses.BinaryCrossentropy(reduction=tf.keras.losses.Reduction.SUM_OVER_BATCH_SIZE, from_logits=False),
                                     optimizer=Adam(learning_rate=1e-4))

# set weights and bias to zero
for ix, layer in enumerate(model_unmask.layers):
     weights = layer.get_weights()
     for arr in weights:
        arr[(arr > 0) | (arr < 0)] = 0
     layer.set_weights(weights)

# evaluate on fake data
model_unmask.evaluate(x_dummy_data, y_dummy_data)
model_mask.evaluate(x_dummy_data, y_dummy_data)

The model_unmask.evaluate leads to a loss of 0.6931471824645996, while the masked model is 0.4620981216430664. Can you explain how the latter is calculated? If weights and biases are set to 0 and the last layer is sigmoid activated then the only input into the loss function is a tensor of 0.5's. Shouldn't these both be the same when you take an average over the batch?

@chethanjjj ,
I ran the code shared and face a different error, please find the gist here and share all dependencies to replicate the issue or share a colab gist with the reported error.Also, please let us know the tensorflow version you are using.Thanks!

Got you. I fixed it by adding the dependencies, addressing any remaining errors, and indicating the dependency versions in the first paragraph.

@ymodak  ,
I was able to reproduce the issue in tf v2.4,v2.5 and nightly.Please find the gist here.

This question is better asked on TensorFlow Forum since it is not a bug or feature request. There is also a larger community that reads questions there.
Thanks!

Hmmm that's a valid place to ask too, but how is this not a bug? Theoretically, both should give the same value if all weights and biases are zeroed out and you were able to reproduce it.

I have cross posted your issue to keras repo for further review from keras team. Please follow the tagged thread for updates. Thanks!


Amplify allows non alphanumeric lambda function name and then fails on amplify push

I used amplify add api to create an api for my project
I use amplify add api
I wanted to create a RESTful api
During this process I created a new Lambda function
There was no warning when I named a lambda function using a non alphanumeric name
When I try to use amplify push I get the following error:
Template format error: Resource name functionfind-buddies is non alphanumeric.
An error occurred during the push operation: Template format error: Resource name functionfind-buddies is non alphanumeric.
When I try to use amplify remove api and remove my api resource and try amplify push again I get the same error.
How can I delete the api and start over ?
This is a bug as I should get a warning if I use a non alpha numeric name for a Lambda function.

@donnrri it looks like this is a bug on the CLI side for now you would have to remove the function and replace it without a hyphen

Took me a long time to figure this out. I added an amplify function, where the name had dashes.
On running amplify push, I got the following output: "Updating resources in the cloud. This may take a few minutes...Error updating cloudformation stack" followed by some nice green messages about my graphql schema compiled successfully, and finally "√ All resources are updated in the cloud"
No error information at all. Cloudformation didn't show any error. My function just didn't appear anywhere.
The only way I figured it out was by trial and error multiple times, until I realized the functions without dashes in their names worked fine.

I think there was an error before when adding function with non alphanumeric lambda function name as the pic shows. I confirmed this with<EMAIL_ADDRESS>
I suggest validating function name in adding function because you need to remove and add function again after adding function now and this is very difficult process to think of for Amplify beginners.

The fix for this has been released in the latest version of the CLI. Please upgrade to the latest version to try it out.


feat(transformer): add TransformOptions::module option

@Boshen I have replaced None with ESM in Module, which is compatible with esbuild, BTW do you know how to set Module in testing by options.json?

#7188 : 2  dependent PRs (#7206 , #7218 ) 👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Boshen and the rest of your teammates on  Graphite


Issue with the background of both timers

What happens is that at the end of all the picks and bans, when there are 60 seconds left for the game to start, "Picking Phase 2" continues to appear in the middle of both timers with the color red in the background, and both timers still have the red color in the background.
The correct thing is that the background of both timers should turn black and the text "Picking Phase 2" should disappear along with the red background of the same.
I am going to put 2 images of what happens and how it should be correctly.

Thanks for pointing this out. It should have been fixed in the last release, but I'll look into it.

Thanks for pointing this out. It should have been fixed in the last release, but I'll look into it.

Ok thanks!

I wasn't able to replicate this bug with on my side. Could you please send me the log file of the champion select, which caused this issue.  It can be found in the logs folder and is named replay_TIMESTAMP.json.

Yes you can add me on Discord: Litzuck#5760.


"TypeError: colorset1 is undefined" when using the from_hex() method

Hi,
I am trying to get working your amazing lib in my project, but when I try to generate colors using the from_hex() method, I get the following error in the browser: TypeError: colorset1 is undefined (color-scheme.js:522:10)
I noticed that if I replaced from_hex() by the from_hue() method, it was working, so it sounds like a bug, that's why I'm posting it here.
In the debugger, at line 511, there is colorset1 = ColorScheme.COLOR_WHEEL[derivative1];, but derivative1 is set to -15, so that's why colorset1 is undefined... Have you any idea of the source of the problem?
Here is a sample of the code I used:
var theme = this.colorScheme.from_hex("AD1457")
                            .scheme('contrast')
                            .variation('light');

UPDATE: I just noticed that the problem occurs in the demo too when I try to put "AD1457" in the Hex field of http://c0bra.github.io/color-scheme-js/.
Thank you in advance & keep up the good work!

@Banasura thanks for the heads-up.
Looks like the hues weren't being allowed to go around the color wheel. Somehow your hex color was ending up with a negative hue: -26, and that was causing things to bomb.
I've added a change to v0.0.6 that will rotate the hues around so -26 becomes 334. Let me know if you have any issues with the change!


[BUGFIX] add missing alias to dojoConfig.js

Why don't you use bower for this external library?

@wcho I didn't know the history of this library. I just reverted the line as quick as possible because there are errors to run some functions on the webida.net site.


chore: Update APK generation and follow naming convention

Fixes #1764

Wait till #1770

@iamareebjamal Since #1770 is merged, any changes required in this?

https://github.com/fossasia/open-event-organizer-android/blob/fe47981075f71b2fb285af9cbb976e7b338d58bc/scripts/update-apk.sh#L67
I think eventyay-organizer-master-app.aab should replace with eventyay-organizer-master.aab.

@iamareebjamal Please confirm.

yes

Updated.


Fixed breakpad compilation errors on Linux

This PR should fix linker issues for Ubuntu and other Linux distros.

@nnian this PR can be merged once CI builds are green 🙂


XSLT Saxon native support

I hit this same issue when trying to build a native executable with camel-quarkus-xslt-saxon, though my use case is slightly different in that I exclude the home edition and import the personal edition.
Doing this I hit the same error as above. I then ran Quarkus along with the native agent to generate the required reflect-config.json file and can report that Saxon is now working fine for me in native mode.

@CynanX it would be very useful if you can share with reflect-ocnfig.json here. And we can take a look to improve in the build steps. Thanks!

I'm currently using Quarkus 2.7.5.Final and building on GraalVM 21.3.1 Java 17 CE (Java Version 17.0.2+8-jvmci-21.3-b09).
The relevant sections of my pom are:
<dependency>
    <groupId>org.apache.camel.quarkus</groupId>
    <artifactId>camel-quarkus-xslt-saxon</artifactId>
    <exclusions>
        <exclusion>
            <groupId>net.sf.saxon</groupId>
            <artifactId>Saxon-HE</artifactId>
        </exclusion>
    </exclusions>
</dependency>

<dependency>
    <groupId>net.sf.saxon</groupId>
    <artifactId>saxon-pe</artifactId>
    <version>11.3</version>
</dependency>

<dependency>
    <groupId>com.sun.mail</groupId>
    <artifactId>javax.mail</artifactId>
    <version>1.6.2</version>
</dependency>

Here's is my resource-config.json file:
{
  "resources": {
    "includes": [
      {
        "pattern": "\\Qsaxon-license.lic\\E"
      }
    ]
  }
}

And finally here is my reflect-config.json file:
[
  {
    "name": "com.saxonica.config.ProfessionalConfiguration",
    "methods": [
      {
        "name": "<init>",
        "parameterTypes": []
      }
    ]
  },
  {
    "name": "com.saxonica.functions.extfn.Parse",
    "methods": [
      {
        "name": "<init>",
        "parameterTypes": []
      }
    ]
  },
  {
    "name": "net.sf.saxon.functions.StringJoin",
    "methods": [
      {
        "name": "<init>",
        "parameterTypes": []
      }
    ]
  },
  {
    "name": "net.sf.saxon.functions.String_1",
    "methods": [
      {
        "name": "<init>",
        "parameterTypes": []
      }
    ]
  },
  {
    "name": "net.sf.saxon.functions.Tokenize_1",
    "methods": [
      {
        "name": "<init>",
        "parameterTypes": []
      }
    ]
  },
  {
    "name": "org.apache.camel.component.xslt.XsltBuilder",
    "allDeclaredFields": true,
    "queryAllDeclaredMethods": true
  },
  {
    "name": "org.apache.camel.component.xslt.XsltComponent"
  },
  {
    "name": "org.apache.camel.component.xslt.XsltComponentConfigurer"
  },
  {
    "name": "org.apache.camel.component.xslt.XsltEndpointConfigurer"
  },
  {
    "name": "org.apache.camel.component.xslt.XsltEndpointUriFactory"
  },
  {
    "name": "org.apache.camel.component.xslt.saxon.XsltSaxonBuilder",
    "allDeclaredFields": true,
    "allDeclaredClasses": true,
    "queryAllDeclaredMethods": true,
    "methods": [
      {
        "name": "<init>",
        "parameterTypes": []
      }
    ]
  },
  {
    "name": "org.apache.camel.component.xslt.saxon.XsltSaxonComponent"
  },
  {
    "name": "org.apache.camel.component.xslt.saxon.XsltSaxonComponentConfigurer",
    "methods": [
      {
        "name": "<init>",
        "parameterTypes": []
      }
    ]
  },
  {
    "name": "org.apache.camel.component.xslt.saxon.XsltSaxonEndpointConfigurer",
    "methods": [
      {
        "name": "<init>",
        "parameterTypes": []
      }
    ]
  },
  {
    "name": "org.apache.camel.component.xslt.saxon.XsltSaxonEndpointUriFactory"
  },
  {
    "name": "org.xmlresolver.loaders.XmlLoader",
    "methods": [
      {
        "name": "<init>",
        "parameterTypes": [
          "org.xmlresolver.ResolverConfiguration"
        ]
      }
    ]
  },
  {
    "name": "org.xmlresolver.logging.DefaultLogger",
    "methods": [
      {
        "name": "<init>",
        "parameterTypes": [
          "org.xmlresolver.ResolverConfiguration"
        ]
      }
    ]
  }
]

Note. There are several more Saxon functions not in the file above but will be required it they are utilised by your XSLT files.
I saw a thread on StackOverFlow not long ago where by @michaelhkay mentioned they were looking at producing a native friendly version of Saxon, so he may be willing to assist here.

Yes, we (more specifically my colleague O'Neil Delpratt) are actively working on getting Saxon to work under GraalVM; and as you've found, dynamic loading and reflection is the main challenge.
I'm not sure if we have anything more we can share with you at the moment. We're probably at about the same stage of development. But we're happy to share information if this helps us reach the goal more quickly.
Michael Kay
Saxonic

On 16 Apr 2022, at 16:46, CynanX @.***> wrote:
I'm currently using Quarkus 2.7.5.Final and building on GraalVM 21.3.1 Java 17 CE (Java Version 17.0.2+8-jvmci-21.3-b09).
The relevant sections of my pom are:

org.apache.camel.quarkus
camel-quarkus-xslt-saxon

net.sf.saxon
Saxon-HE

net.sf.saxon
saxon-pe
11.3

com.sun.mail
javax.mail
1.6.2

Here's is my resource-config.json file:
{
"resources": {
"includes": [
{
"pattern": "\Qsaxon-license.lic\E"
}
]
}
}
And finally here is my reflect-config.json file:
[
{
"name": "com.saxonica.config.ProfessionalConfiguration",
"methods": [
{
"name": "",
"parameterTypes": []
}
]
},
{
"name": "com.saxonica.functions.extfn.Parse",
"methods": [
{
"name": "",
"parameterTypes": []
}
]
},
{
"name": "net.sf.saxon.functions.StringJoin",
"methods": [
{
"name": "",
"parameterTypes": []
}
]
},
{
"name": "net.sf.saxon.functions.String_1",
"methods": [
{
"name": "",
"parameterTypes": []
}
]
},
{
"name": "net.sf.saxon.functions.Tokenize_1",
"methods": [
{
"name": "",
"parameterTypes": []
}
]
},
{
"name": "org.apache.camel.component.xslt.XsltBuilder",
"allDeclaredFields": true,
"queryAllDeclaredMethods": true
},
{
"name": "org.apache.camel.component.xslt.XsltComponent"
},
{
"name": "org.apache.camel.component.xslt.XsltComponentConfigurer"
},
{
"name": "org.apache.camel.component.xslt.XsltEndpointConfigurer"
},
{
"name": "org.apache.camel.component.xslt.XsltEndpointUriFactory"
},
{
"name": "org.apache.camel.component.xslt.saxon.XsltSaxonBuilder",
"allDeclaredFields": true,
"allDeclaredClasses": true,
"queryAllDeclaredMethods": true,
"methods": [
{
"name": "",
"parameterTypes": []
}
]
},
{
"name": "org.apache.camel.component.xslt.saxon.XsltSaxonComponent"
},
{
"name": "org.apache.camel.component.xslt.saxon.XsltSaxonComponentConfigurer",
"methods": [
{
"name": "",
"parameterTypes": []
}
]
},
{
"name": "org.apache.camel.component.xslt.saxon.XsltSaxonEndpointConfigurer",
"methods": [
{
"name": "",
"parameterTypes": []
}
]
},
{
"name": "org.apache.camel.component.xslt.saxon.XsltSaxonEndpointUriFactory"
},
{
"name": "org.xmlresolver.loaders.XmlLoader",
"methods": [
{
"name": "",
"parameterTypes": [
"org.xmlresolver.ResolverConfiguration"
]
}
]
},
{
"name": "org.xmlresolver.logging.DefaultLogger",
"methods": [
{
"name": "",
"parameterTypes": [
"org.xmlresolver.ResolverConfiguration"
]
}
]
}
]
Note. There are several more Saxon functions not in the file above but will be required it they are utilised by your XSLT files.
I saw a thread on StackOverFlow not long ago where by @michaelhkay https://github.com/michaelhkay mentioned they were looking at producing a native friendly version of Saxon, so he may be willing to assist here.
—
Reply to this email directly, view it on GitHub https://github.com/apache/camel-quarkus/issues/1615#issuecomment-1100693491, or unsubscribe https://github.com/notifications/unsubscribe-auth/AASIQIXRSMJ3WAVWJM6J3NTVFLOF5ANCNFSM4QLTNXCA.
You are receiving this because you were mentioned.

Thanks for the input, @CynanX. I am afraid, we won't be able to incorporate any Saxon PE and com.saxonica.* config because of licensing reasons. Anyway, to add a native support to an extension, the integration test is usually crucial. So if you have some camel-quarkus-xslt test to share a PR would be highly welcome.

Native support for XSLT Saxon was actually implemented via 66565dd0ea7ffd65184a3574fc20fd216fac5fdc so this issue might be seen as a duplicate of https://github.com/apache/camel-quarkus/issues/1710


Update parent pom and fix startup errors

Fixes 2 issues:

Project doesn't build with newer maven issues (parent pom update fixes)
hpi:run has lots of errors due to missing dependencies for the build failure analyzer

@jan-molak this fixes the build, there's more I would like to do here, but would like to do it in a follow up after the master build is fixed,
could you take a look please


Add limit kwarg to repo functions

Allow users to specify a limit that will limit how many roles are
repoed at one time. This is an optional kwarg that defaults to a value
of zero, which results in not limiting the repo action.
Also add AFTER_REPO_ROLES hook and add errors to the
AFTER_REPO hooks context.

Coverage increased (+0.09%) to 61.748% when pulling 5d612219cc8b5abef9c486060a99e74f262c54b5 on batch-limit into 3771bfb6ace475626d75c4312fba18d336ed4b77 on master.


Unable to run cypress tests in Ubuntu 20.04 LTS ( Linux ) server machine

I have setup the tests to run in a ubuntu server machine. But for some reason, I am not able to run the tests in my ubuntu machine . The same test can be successfully run in a windows machine as well as in a mac machine but not in an Ubuntu Machine.
When I run the tests, what  I see is it is getting identified as a cucumber tests but running from the test runner gives me the following output
**Error: Can't walk dependency graph: Cannot find module 'fsevents' from '/home/timothyrajan/Documents/cypress/smokebdd/node_modules/chokidar/lib'
    required by /home/timothyrajan/Documents/cypress/smokebdd/node_modules/chokidar/lib/fsevents-handler.js
    at /home/timothyrajan/Documents/cypress/smokebdd/node_modules/browser-resolve/node_modules/resolve/lib/async.js:46:17
    at process (/home/timothyrajan/Documents/cypress/smokebdd/node_modules/browser-resolve/node_modules/resolve/lib/async.js:173:43)
    at ondir (/home/timothyrajan/Documents/cypress/smokebdd/node_modules/browser-resolve/node_modules/resolve/lib/async.js:188:17)
    at load (/home/timothyrajan/Documents/cypress/smokebdd/node_modules/browser-resolve/node_modules/resolve/lib/async.js:69:43)
    at onex (/home/timothyrajan/Documents/cypress/smokebdd/node_modules/browser-resolve/node_modules/resolve/lib/async.js:92:31)
    at /home/timothyrajan/Documents/cypress/smokebdd/node_modules/browser-resolve/node_modules/resolve/lib/async.js:22:47
    at callback (/home/timothyrajan/.cache/Cypress/7.4.0/Cypress/resources/app/packages/server/node_modules/graceful-fs/polyfills.js:289:20)
    at FSReqCallback.oncomplete (fs.js:183:21)**

This seems to be the same issue as the one mentioned in the below link
https://github.com/TheBrainFamily/cypress-cucumber-preprocessor/issues/10
But I tried the solution mentioned in the above page
"With "cypress-cucumber-preprocessor" :- Everything is working fine , if we do not use below line
const { given } = require("cypress-cucumber-preprocessor"); instead of this use global variable given , then etc"
but still facing the issue.
The same project works well in a mac machine without any code change. Is there any reservations on ubuntu server machines ( 20.04 Server machine LTS ) ?
The following are the versions.

Cypress version:      "cypress": "^7.4.0",
Preprocessor version:   "cypress-cucumber-preprocessor": "^4.1.2",
Node version:  v10.19.0

Thanks very much for your time on this.

Hello,
Could you provide a repository with a minimal reproduction example?
I do all my development on manjaro and didn't encounter this problem. Could you try out a different version of cypress-cucumber-preprocessor?

@MateuszNowosad  - The git repo is below
https://github.com/timrajan/mybdd.git
Also, I have asked the question in Stackoverflow . Let me know your thoughts.
https://stackoverflow.com/questions/67920638/unable-to-run-cypress-tests-in-ubuntu-20-04-lts-linux-server-machine
Comment 1 is not working for me and still facing the same issue
Not sure how to implement Comment 2
Thanks a lot  @MateuszNowosad  for your help on this.
Again to reiterate, the same code works well in a mac machine. Its only Linux where I am facing this issue.

@MateuszNowosad  - Any updates on this ? Would be very helpful for me as I am dependent on my mac for all executions.
Thanks again for your time.

Your project runs fine on Arch Linux with the following changes and I'm not able to reproduce the error you've mentioned in the original post. I'm afraid nobody is going to actively work on an issue that they can't reproduce and you're likely not going to see any more updates until you can provide a reproducible example.
diff --git a/cypress/integration/tests/step_definitions/test/test.steps.js b/cypress/integration/tests/step_definitions/test/test.steps.js
index a824afe..093b044 100644
--- a/cypress/integration/tests/step_definitions/test/test.steps.js
+++ b/cypress/integration/tests/step_definitions/test/test.steps.js
@@ -1,16 +1,14 @@
-//import {And,Given,When,Then} from 'cypress-cucumber-preprocessor/steps'
+import {And,Given,When,Then} from 'cypress-cucumber-preprocessor/steps'

-//const { Given,When,Then,And } = require("@cypress/browserify-preprocessor");

-
-given("that the website is loaded and user registration is possible",()=>{
+Given("that the website is loaded and user registration is possible",()=>{
         cy.log('I am in Given1')
 })

-when('a user registers by filling out only the mandatory fields',()=>{
+When('a user registers by filling out only the mandatory fields',()=>{
         cy.log('I am in Given2')
 })

-then('the newly registered user is able to log back in',()=>{
+Then('the newly registered user is able to log back in',()=>{
         cy.log('I am in Given5')
 })
\ No newline at end of file
diff --git a/cypress/plugins/index.js b/cypress/plugins/index.js
index 84b6e34..e483f14 100644
--- a/cypress/plugins/index.js
+++ b/cypress/plugins/index.js
@@ -21,20 +21,11 @@ const path = require('path');
 const fs = require('fs-extra');
 const cucumber = require('cypress-cucumber-preprocessor').default

-const optionalPlugins = [];
-const { IgnorePlugin } = require('webpack');
-
-if (process.platform !== "darwin") { // don't ignore on OSX
-  optionalPlugins.push(new IgnorePlugin({ resourceRegExp: /^fsevents$/ }));
-}
-
-

 module.exports = (on, config) => {
   // `on` is used to hook into various events Cypress emits
   // `config` is the resolved Cypress config
   on('file:preprocessor', cucumber())
-  plugins: [ ...optionalPlugins ]
 }

 // const browserify = require("@cypress/browserify-preprocessor");
diff --git a/cypress/support/index.js b/cypress/support/index.js
index 9bcf94b..e2df5b6 100644
--- a/cypress/support/index.js
+++ b/cypress/support/index.js
@@ -16,7 +16,6 @@

 // Alternatively you can use CommonJS syntax:
 require('./commands')
-require('cypress-cucumber-preprocessor')
 require('cypress-file-upload')
 require('cypress-xpath')
 require('faker')

Thanks @badeball . I will give it a try . Thanks once again for your time @badeball . if everything fails, I need to rebuild my ubuntu machine.

Due to personal reasons, the previous maintainers of this package are stepping down and handing the reigns over to me, a long-time contributor to the project and a user of it myself. This is a responsibility I'm very excited about. Furthermore, I'd like to thank @lgandecki ++ for all the work that they've done so far.
Read more about the transfer of ownership here.
The repository has however moved and all outstanding issues are being closed. This is not a reflection of the perceived importance of your reported issue. However, if after upgrading to the new version, you still find there to be an issue, feel free to open up another ticket or comment below. Please make sure to read CONTRIBUTING.md before doing so.


chore: Merge changelogs to match previous behavior

Closes SAP/cloud-sdk-backlog#699.
Main change:

Add a script (merge-changelogs.md) which can merge the generated changelogs (and uses fancy Regex, better ideas are welcome)

Minor changes:

Update existing changesets to fit our new best practices
Renames version to more speaking currentSdkVersion (only used in scripts, therefore no compatibility note)

Let me know if this should be added to any GH Actions or other scripts.

[q] I tested changeset with merge-changelogs branch locally. Somehow, changeset bumps the version directly to 3.0.0-... in each package/**/CHANGELOG.md file after running:

yarn changeset pre enter ${date}
yarn changeset version

I did not replace the .md with canary-release-changeset.md just for testing. All .md have actually only minor or patch changes. Could this be a problem or did I miss anything?

I don't fully understand why this happens but it does not happen when there are no minor changesets. It is probably due to a weird interaction with our peerDependencies... Should not be a problem as far as I know.

ok already merged :-)


Set default flutter source directory for gradle builds

See #142498
See this discussion: https://discord.com/channels/608014603317936148/1186378330178601000
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Cc @reidbaker @bartekpacia


Model coupling issues

Couple the FTT models with the following four aspects

Electriciy demand; heat, transport and steel should feed back to electricity
Electricity price: add the elasticity from the X-databank (elasticity is econometrically estimated). Feed electricity price into the other models.
Cross-learning: specifically in terms of electricity storage
RLDC: add some uncertainty from literature on additional strain on grid

This sounds like we need to iterate between the electricity users and suppliers within each year until prices and demand have converged. Not impossible and I wouldn't think it would take many interations to find convergence.
If we are to implement this in the master version, then it would be great to have a switch available that allows us to undo the coupling. The reason being that during update cycles, we'd want to compare E3ME output to the standalone output (using inputs from E3ME) to check the FTT results align.
On learning: Perhaps we need to run all FTTs in a single timeloop. It would be pose a couple of design issues, but it does allow us to combine the cross-sectoral learning effects wherever they may apply.


Change prop source string to array

The prop source must be array no a string for url. The data must be set and not the same component to get it
Reference to #36

Can you tell me what exactly do you want for this. I was unable to understand properly from the description provided.

@Rud156 Now, vue datasource gets the data from a url (api), this is a bad idea as this should be done by the user for better control.
What do you have to do? Change the way in which the data is processed by the component.

Ok. Thanks I'll look into it

@Rud156 Thanks man 👍

I'm sorry I won't be able to help you because of some issue


[BUG] Recent point releases are not creating new releases in github

Describe the bug
The Github release is still 1.4.0 while it looks like both 1.4.1 and 1.4.2 have been updated. This means my application says it is version 1.4.2 but the latest version is 1.4.0.

should be resolved in the latest release


add ci build workflows in github-hosted runner

Signed-off-by: Xiao, Ziyang<EMAIL_ADDRESS>Please provide a description of this PR:
Build step without push except proxyv2,just test workflow
To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Please check any characteristics that apply to this pull request.

[ ] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

https://github.com/intel/istio/blob/a73921dc1685bc5525d7e7b6d880601472191d4a/pilot/pkg/xds/sds.go#L351
Seems that which caused build failed. Need to be fixed. CC @rveerama1 @ipuustin

https://github.com/intel/istio/blob/a73921dc1685bc5525d7e7b6d880601472191d4a/pilot/pkg/xds/sds.go#L351
Seems that which caused build failed. Need to be fixed. CC @rveerama1 @ipuustin

Thanks for the heads up. sent PR: https://github.com/intel/istio/pull/5

Can you rebase? #5 is merged now.

Retrigger CI by reopening pr.

Reopen it seems CI missed.

I verified CI can works right in repo. So I'll create a new pr for self-hosted runner, just close this one.

Re-used this pr is ok.

re-trigger build


Added Option to delete all tasks #44

Related Issue
Closes #44
Type of Change
Put x inside the square bracket to specify what type of change your PR is:

[x] New Feature

Description of Change
Added actions menu in app bar which contains option to delete all the tasks and saves a empty task list.
Implementation Details
      void _onOptionSelected(String option) {       setState(() {         if (option == "Delete all tasks") {           tasks = [];           TaskStorage.saveTasks(tasks);         }       });     }
          actions: [             PopupMenuButton<String>(               onSelected: _onOptionSelected,               itemBuilder: (context) {                 return [                     const PopupMenuItem(                       value: "Delete all tasks",                       child: Text("Delete all tasks"),                     ),                 ];               },             ),           ],
Demo

@Sanat-Jha  While merging main branch into your branch, you missed the Task of the day and TaskList Screen component. Please resolve it first and restore these components.

@lakshita10341 I guess its fixed now, please confirm.


[feat] Blobstream Contracts Integration Tests

Create integration tests (scripts) for the Blobstream contract(s) by doing the following :

Launch a Katana node
Declare & Deploy the BlobstreamX contract ( & dependencies such as the gateway ) to the Katana node
Make calls to public functions on BlobstreamX contract & check state updates are correct

Possibly use mock implementations where needed.
References
https://github.com/keep-starknet-strange/blobstream-starknet/blob/main/src/lib.cairo

This is currently blocked. Waiting for Katana to upgrade the supported JSON-RPC spec to 0.6.0
It was posted that this should be soon https://t.me/dojoengine/5922

The decision to switch from Katana to starknet-devnet-rs has unblocked this. Katana does now support 0.6.0, but this decision was made prior to that. I think this is still the best way to go because forking starknet-devnet-rs doesn't require forking the entire dojo engine. ( Forking will be required to unblock Groth EC pairings with new syscalls for now )

Update
We have decided to integrate with Herodotus storage proof's / API. So, instead of using a devnet for integration tests, we will need to do tests on Sepolia for the integration.
This would include setting up deployment and upgrading scripts for the BlobstreamX contracts on Sepolia, script for sending a blob to Celestia, using the request data commitments Herodotus script, and verifying DA for the blob on Starknet through BlobstreamX.
References

Herodotus integration
Herodotus data commitments request script
Posting blobs to Celestia

So remaining scripts to do are :

updating Starknet BlobstreamX data commitments from the Herodotus Fact registry
verifying DA for the blob on Starknet through BlobstreamX

Can I work on these ? If so, is starkli the way to go ?

@thomas192 yes this is a good summary of the remaining scripts. Can you take #138 and I'll take #139 ?

duplicate of https://github.com/keep-starknet-strange/blobstream-starknet/issues/136 ?


Bug: Reset button returns error in Console

Describe the bug
Trying to reset GooseMod to its defaults results in the error shown below.
Reproduction Steps
1.) Go to Settings -> Goosemod
2.) Open up developer tools (usually Ctrl-Shift-I)
3.) Click on the reset button
Expected behavior
Prompt the user to reset all GooseMod data.
Screenshots
Uncaught TypeError: e.onclick is not a function
    at HTMLDivElement.M.onclick (eval at <anonymous> (eval at <anonymous> (@me:1)), <anonymous>:40:12146)
M.onclick @ VM360:40

GooseMod info:

Update Channel: release
Version: 8.0.0 (1e5f588)
Modules: None; removed all for testing purposes

Discord info:

Update Channel: Stable
OS: Linux 64-Bit (5.11.13-200.Fc33.X86_64)
Host Version: Host 0.0.14
Web Version: Stable 82117 (58a452e)

Fixing now.

Done in last commit, will be released in v8.1.0 very soon.


0.1.0

Codecov Report

Merging #1 into master will decrease coverage by 5.2%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master      #1      +/-   ##
=========================================
- Coverage   11.77%   6.57%   -5.21%
=========================================
  Files           2       2
  Lines        1401    2586    +1185
=========================================
+ Hits          165     170       +5
- Misses       1236    2416    +1180

Impacted Files
Coverage Δ

test/spec/core.spec.js
6.84% <ø> (ø)

test/spec/matrix.spec.js
6.27% <ø> (ø)

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 3956a3d...8ff31cd. Read the comment docs.


Update renovatebot/github-action action from v32.85.0 to v32.86.0

This PR contains the following updates:

Package
Type
Update
Change

renovatebot/github-action
action
minor
v32.85.0 -> v32.86.0

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Renovate will not automatically rebase this PR, because other commits have been found.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, click this checkbox. ⚠ Warning: custom changes will be lost.

This PR has been generated by Renovate Bot.

@balena-ci retest


[OB3] CIBA auth weblink based authenticator implementation.

Description:
This issue is to track the multi authorisation CIBA requirement.
Suggested Labels:
CIBA, Accelerator, OB3
Suggested Assignees:
Affected Product Version:
OB3
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:
[Any related issues such as sub tasks, issues reported in other repositories (e.g component repositories), similar problems, etc. ]

Closing as updates are released


Revise How to: Use Selected Value... topic to display all XAML

Revise How to: Use Selected Value... topic to display all XAML
General
See Issue #4526
The How to: Use Selected Value, SelectedValuePath, and SelectedItem topic needs to be prevised to present the entire contents of the window1.xaml file.  Highlighting should then be used to indicate the individual parts that are discussed in the topic.
The code also needs to be modified, as PR #4526 suggests, and the text modified to more clearly explain the code.
@ManGupt @BillWagner

I agree with @BillWagner on updating the XAML code. Here is my suggestion:
changing <HierarchicalDataTemplate DataType="EmployeeInfo"
to <HierarchicalDataTemplate x:Key="SampleTemplate" DataType="EmployeeInfo"
and
changing
to
What do you think?

@rpetrusha I would love to do the modification. Just guide me as I am new.

@ManGupt I just noticed your previous comment. Most of our team was involved with the MVP summit last week, and missed some GitHub notifications.
What can we do to help you get started?

How do I download the code "samples/snippets/csharp/VS_Snippets_Wpf/TreeViewSelectedValue/CS" to check the changes that I have made?

How can I add reviewers to my pull request #4662 ?

@ManGupt

How do I download the code "samples/snippets/csharp/VS_Snippets_Wpf/TreeViewSelectedValue/CS" to check the changes that I have made?

You have to download the whole repository and then find the directory there. To download the repository, you have two choices:

Use git. This is the common approach that lets you keep an up-to-date local copy of the repository and also perform other operations.
To do this, go to the main page of the repository, click Clone or download and either select Open in … if you want to use one of the offered choices of git client (like GitHub Desktop) or copy the URL and use it in your git client. If you're using git command line interface, the command to use would be git clone https://github.com/dotnet/docs.git.

Download a ZIP archive of the repository. This is not commonly used and makes sense only if you need access to a copy of the repository rarely.
To do this, go to the main page of the repository, click Clone or download and select Download ZIP.

How can I add reviewers to my pull request #4662 ?

I believe you can't actually add reviewers in a way GitHub would recognize (you would need "write access" to do that). GitHub already automatically added @BillWagner as a reviewer, because he's the "code owner" for C# samples.
If you want to notify someone else, you can add a comment to your Pull Request, mentioning the user with a @ before their user name. It's also common to indicate that you're notifying someone of the PR by using cc: (which is a convention coming from email). So, if I wanted to notify you, I would write cc: @ManGupt.


About the tokenizer

Why the tokenizer class specified in tokenizer config is BloomTokenizer? There is only BloomTokenizerFast in transformers implementation.

Hi @yuyq96,
Thanks for your attention!
Could you upgrade the transformers version: pip install git+https://github.com/huggingface/transformers and try again?
Best,
Zhihong

There is no BloomTokenizer even in the latest transformers.

Hi yuyq96,
I got the same issue. Any solution for this issue.
Thanks in advance.

There is no BloomTokenizer even in the latest transformers. https://github.com/huggingface/transformers/tree/main/src/transformers/models/bloom


Added quotes around strings in json response for huemulator

When adding the HABridge to Home Assistant the Home Assistant kept crashing on the json response. The json response for state changes containing alert / effect was returning none instead of "none".

Thanks, totally missed those. This will be in the next release


Guidelines for contributing out of date

A meta-issue.. If I open a ticket here, it asks me to see the guidelines for contributing.  They say to open a ticket in Jira, but Jira tells me to open a ticket here.
I'm pretty sure here is the right location?

I've updated the contributing docs to the current version.  In the future I'm going to see if we can just get the cookbook contributing and testing docs hosted on the docs site so we can avoid this.  If someone has an older version of the cookbook in their environment they shouldn't be reading out of data process docs.


Allow users to list pods and get metrics cluster-wide

Is your feature request related to a problem? Please describe.
To allow users to easily determine resource allocation on nodes they need to be able to list pods cluster-wide in order to run k describe node.
Listing pods should not reveal any information that a user does not already have access to via Kibana and Grafana.
Being able to get metrics cluster-wide enables users to run k top nodes to quickly be able to determine resource usage exposed via the metrics-server.
Describe the solution you'd like
Users should be assigned the following cluster-wide permissions
- apiGroups:
  - metrics.k8s.io
  resources:
  - pods
  - nodes
  verbs:
  - get
  - list
  - watch
- apiGroups:
  - ""
  resources:
  - pods
  verbs:
  - list

Note that I'm not 100% sure if they need list and watch for the metrics or if it is enough with get.
Describe alternatives you've considered
Not giving users more privileges.
Definition of done:
A user  can run k describe nodes and see pods and allocated resources, and they can run k top nodes and see usage information.

With list pods, they can also actually retrieve the entire object, so go through and make sure that there are no secrets specified directly in the pod spec.


Bug 1955761: Set reason/message for Available condition in packageserver co

Upstream-commit: c21dcfc95b6e434938f9edc07550492c76cebb2b
Upstream-repo: https://github.com/operator-framework/operator-lifecycle-manager
Signed-off-by: Anik Bhattacharjee<EMAIL_ADDRESS>

/bugzilla refresh

/retest

/retest


Add support for newline characters in d2l-dialog-confirm text.

This PR adds support for newline characters in d2l-dialog-confirm text.  One paragraph will be rendered for each line of text provided.  Consumers will need to construct their text in HTML using
 or in JavaScript using \n.
In the future we may pull this out and combine it with other simple formatting if we ever support some type of SML in our components.

Nice! I like where this ended up.

Thanks for the suggestion!


Populate Document

Task done✅
https://docs.google.com/document/d/1oZM_O1LnQczyQJWQ9631_ASo68Cuy4ay4cE3YXDq7qY/edit?usp=drivesdk

Great work 👍

Thank yooooouuuu


Explorer OUTLINE view will look messy if are too many imports in file(seen on python)

OUTLINE imports in Explorer tab, if are too many, will look messy. Will be a lot of scrolling if you want to reach to functions.
Pls group them behind a "Imports" or "_imports_" (for python specific) label entry category for easy grouping.
VS Code version: Code 1.86.2 (https://github.com/microsoft/vscode/commit/903b1e9d8990623e3d7da1df3d33db3e42d80eda, 2024-02-13T19:40:56.878Z)
OS version: Windows_NT x64 10.0.19045
Example:

Group behind Imports(or other name....) all Outline imports seen in Explorer Outline.

I use Jedi. You have right on pylance imports not appear.
Yahoo Mail: Căutare, organizare, reușită
Pe vin., mar. 8, 2024 la 0:01, Erik De @.***> a scris:
I don't see imports in the Outline when using Pylance. Are you perhaps using Jedi instead? Check the python.languageServer setting.
This is what I see with Pylance:
image.png (view on web)
And this is what I see with Jedi:
image.png (view on web)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

I use Jedi. You have right on pylance imports not appear.

Ok. Please file an issue on Jedi at https://github.com/davidhalter/jedi/issues.


feat(rest): trailing slash option

readme update still missing

This change is

@RWOverdijk that seems quite a bit more convoluted. just keeping trailing slashes seems easier. anyways, other than user/1 all requests have kept them previously i think. hence the other option might get simplified.
as for accepting trailing slashes. isn't it more likely the other way round. all servers will accept that trailing slash most likely

@doktordirk I agree with you. But if we keep the trailing slash it'll be a BC break. We could do that still, I suppose, but it's magic. Magic scares people (and makes for odd issues). I personally think making it configurable is a better idea, but I'd love your input :)

your right about magic isn't a good design choice. but this is rather about consistent behaviour since you are wrong, it's not (very) breaking. it was only inconsistent before.
as you can see maybe in the other commit, the only place where the trailing slash wasn't maintained previously is with criteria as id. (path/id) in all other cases, trailing slashes had been maintained (eg is path/?id=1 already). hence, i thing in this case, the better option is too keep it simple and use the other option

we'll use the other option for now


khronos: 1.0.8 -> 3.5.9

Motivation for this change
https://github.com/lainsce/khronos/releases
https://github.com/lainsce/khronos/compare/1.0.8...3.5.9
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

It would be cool of you to add a khronos: clarify license commit that sets meta.license = licenses.gpl3Plus.

Result of nixpkgs-review pr 137928 run on x86_64-linux 1

1 package built:

khronos


Profile for Esperanto

I've created a profile for Esperanto: http://danielnaber.de/tmp/eo.zip. It's based on the data from http://tatoeba.org, which is about 15 MB of plain Esperanto text.

I've updated the file, it now also contains the training data. I suggest to also store that in git so everybody can re-build the profile. I used a minimum frequency of 20.

I've updated the file, it now also contains the training data. I suggest to also store that in git so everybody can re-build the profile. I used a minimum frequency of 20.


Typo in type definition (v3.0.0)

The extra semicolon in the type definition (https://github.com/webroo/dummy-json/blob/master/index.d.ts#L54) throws the following error:
node_modules/dummy-json/index.d.ts:54:2 - error TS1036: Statements are not allowed in ambient
contexts.

54 };

Thanks for letting me know! Fixed in 3.0.1


[README] Mujoco Requirement

Hi folks, first of all, thanks for the amazing material and work you open sourced, outstanding and generous gift to the community.
I wondered if would be interesting to add a side note regarding Mujoco 1.3 on recent Macs.
This version didn't support NVMe disks and still an open issue where there are more details regarding it.

Sure. Feel free to make a pull request for the README.

As soon as https://github.com/openai/gym/pull/767 is merged we could remove the note for Mac users and update the dependencies to MuJoCo v1.5, if it's of your interest.
I tested those changes on an NVMe Mac and it works.


fix the operation name of spring mvc span is incorrect

Coverage remained the same at 17.441% when pulling 7c279d7da7c68c98bf8724cc50a8a6686a0a319d on ascrutae:fix/spring-annotation-issue into 25036048aeb6e8132d5127c218ec0c3830f7fed1 on wu-sheng:master.


Startup speedup

Findings
It takes about 4s for thermald to start and around 15s for locationd to start. Thermald might be slightly fixable

Please remove ui change, that has nothing to do with this PR.


feat(conf): Add migrations.ttlSecondsAfterFinished to chart template

What this PR does / why we need it:
Adds migrations.ttlSecondsAfterFinished as an option in the chart template for allowing removal of migrations containers after a specified time.
The use of ttlSecondsAfterFinished was first discussed here, however the TTL mechanism was in beta at the time.
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[] PR is based off the current tip of the main branch.
[ ] Changes are documented under the "Unreleased" header in CHANGELOG.md
[ ] New or modified sections of values.yaml are documented in the README.md
[ ] Commits follow the Kong commit message guidelines

 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.

 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.

Please also make sure to sign the CLA so that the PR can merge.

@pmalek With regards to the CLA, it's telling me that I've signed it and that it's in a 'pending' state. Are you able to check the issue?

Thanks for the contribution @ashman1984 🙇


Curve2D:tessellate produces a slanted curve

Godot version: v3.2.3.stable.official

OS/device including version: PC, Linux Mint, kernel 4.15.0-118

Issue description:
calling Curve2D.tessellate with f.i. (20,0), (40,60), (-40,60), (140,0)
produces a curve that is not symmetric, as it should be, but instead slated towards one of the endpoints
(given that "in" and "out" in this example are mirror images, equal but with opposite sign in x).
My guess is that the issue is at
https://github.com/godotengine/godot/blob/master/scene/resources/curve.cpp, line 941,
_bake_segment2d(midpoints.write[i], 0, 1, points[i].pos, points[i].out, points[i + 1].pos, points[i + 1].in, 0, p_max_stages, p_tolerance);
which should be
_bake_segment2d(midpoints.write[i], 0, 1, points[i].pos, points[i].out, points[i + 1].pos, points[i].in, 0, p_max_stages, p_tolerance);
(.in and .out both belong to points[i] for any given Bezier segment)
Thanks!
Minimal reproduction project:

Could you please provide a minimal reproduction project that shows the resulting curve? It would help other contributors confirm the issue and review a potential fix.

42727.zip
Hi,
sorry for the incomplete post yesterday. Attached is a minimal example.
After downloading the sources in the 3.2 branch, I can confirm that the change mentioned in the first post (points[i + 1].in -> points[i].in) fixes the issue. I used the 3.2 branch because there are differences in the master branch that I didn't want to work around for the example project (in the master branch PoolArrayVector2D becomes PackedArrayVector2D, and Vector2D.linear_interpolate does not appear to exist). The relevant line in scene/resources/curve.cpp (line 941 in master, 953 in the 3.2 branch) seems to be the same, though.
Thanks again!

It seems some control points might be inverted in your script. I'm getting the proper result by setting an out control on the first point, and in control on the second point:
func tessellated():
	var c = Curve2D.new()
	c.add_point(begin, Vector2.ZERO, offset_out)
	c.add_point(end, offset_in, Vector2.ZERO)
	return c.tessellate()

If there were 3 points, the middle point would probably get in + out controls.
Does that make sense to you?

Ah, I see! Thanks, then I believe we can close the issue.


Ad-Hoc support?

Curious if this could support creating ad-hoc networks?
I have a particular use case where I need the computer to host a WiFi network with a specified SSID and password. 2.4Ghz if it matters. DHCP addressing would be great too! :)

Hi,
Sorry for the delay. This is funny because at the beginning node-wifi was meant to be used in a project to create self-hosted wifi networks and we also had a dhcp server. Hosting wifi might be a feature to implement if we found a solution to make it worked on all OS which is easy. Please propose an implementation if you want these feature.
However DHCP is not planned since it is totally different technology. I think that you can find nice DHCP servers in node.js

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.


[Expose Functions] 1218 "Map Incoming Doc to Purch Doc"

Following #12944, the requested function ProcessField, GetValueFromIntermediate, IsFieldProcessed have not been exposed.
Can you please make the functions global, so we can process our own fields (in OnAfterProcessLines).

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Retract votes

Will close #81

I'd be completely fine if we need to drop 9de09d1 and seek for a better solution. The memory usage is just, not good at all.

Hmmm. Upsetting. It's so sad to have to balance memory vs script size, too.


Release read-write Tx after close

Overview
This pull request removes references to the last write transaction and and its dirty pages after the transaction has been closed. This bug did not affect the safety of the data, however, it would cause dirty pages to linger in-memory until the next write transaction began.

/cc @pauldix


Completion popup shouldn't be moved when scrolling

It should stay at that same line / not be affected by scrolling.

I have this already covered. I'll commit the PR once this PR is merged https://github.com/fdorg/flashdevelop/pull/732

There's a similar issue with zooming in and out.

Yeah, those are covered as well IIRC.


Anyone know the list of models for all the features ?

when i test fill_anything i get :
1、vae\diffusion_pytorch_model.safetensors not found
2、text_encoder\model.safetensors not found
i want to know all the models download URL for the features:
[Remove Anything]
[Fill Anything]
[Replace Anything]
[Remove Anything 3D]
[Remove Anything Video]
thank you !

Hi, I assume you've installed the environment for Remove Anything, then you should further install some dependencies regarding Fill Anything as below.
python -m pip install diffusers transformers accelerate scipy safetensors
Similar to this, you should install feature-specific packages if you wanna run a feature. Kindly refer to installation in each feature introduction in README.md.

Hi, I assume you've installed the environment for Remove Anything, then you should further install some dependencies regarding Fill Anything as below. python -m pip install diffusers transformers accelerate scipy safetensors Similar to this, you should install feature-specific packages if you wanna run a feature. Kindly refer to installation in each feature introduction in README.md.

OK , Thank you


Refactrefact literal get field and value

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

Add some test case will be more better.

We could add some sql based literal SQL test like cast.groovy

Add some test case will be more better.

done!

run buildall


Support Filter in tuple sketches

This PR adds support for filtering tuple sketches values based on summary

There's an issue here in that the result isn't really a valid sketch. The estimated count from the full sketch scaled by the % that match your predicate is the best estimate of the total number from the full stream matching the predicate. But removing items like this will make errors in subsequent uses worse in ways that mean the error bounds are unknown.
It'd be fine to return a cardinality estimate based on applying the predicate.

Hi Jon,
Thanks for your comment. As I understand, the error bounds on the Tuple sketches depend on the theta value and the number of retained entries. While it's true the theta value depends directly by the number of elements added to the sketch, I still don't see a reason why those are still unknown.
I am wondering as well, how this actually would work for the intersection. If you think a bit about it, a filter is just an intersection between 2 sets, one of it declared by enumeration and the other one declared by comprehension. Indeed, the approach followed by the Intersection class looks quite similar to this one.

We debated this internally since we all had the same initial reaction, and we decided that yes, it is a valid thing to do. It will indeed function as an intersection, meaning that there is some accuracy degradation but with known bounds. I modified my initial comment to note the retraction.

The other thing we discussed is whether we want this as a stand-alone class or if it makes more sense to have something an applyPredicate(Predicate p) method on the sketch itself. So we'd appreciate hearing why you went with this route? The are pros and cons to both approaches, of course, so it's not like there's a single correct way.

Thanks for the clarification.
The reasons why I choose this approach are 2:

The main one is it follows a similar pattern to the Filter and Intersection  classes, so it feels consistent with the rest of the code.
The second one is the semantics are not very clear. i.e: for the UpdatableSketch whether this operation is immutable and if the predicate is also applied to the original sketch.

As well, implementing this as a method in the Sketch class would require a circular dependency on the QuickSelectSketch and introduce a dependency of the Factory method (or rather have several implementations, as CompactSketch and QuickSelectSketch only share Sketch as parent)
All in all, I'm open to changes in the code

@jmalkin Is it possible to move this forward?

Sorry, I thought I'd submitted the comments already. Apparently they've been sitting there for a while now.

Sorry, had only phone-based network access recently. Will try to get to this shortly.


Ping does Not work with the OCE DC

Hi There, Ping does not detect ping for the OCE Datacentre. It does detect the IP just shows a latency of 0. Screen Shot attached. Not a big issue, Just thought I would point it out. Thanks for your hard work.

Out of curiosity, are you using WINE or any gaming VPN?

Nope. Just Straight windows 11

I also am encountering this issue. And I am using BattlePing, but it used to work fine with BattlePing until the OCE update. I just restarted the game w/o BattlePing and the plug-in does work.
Just wanted to add that little bit of information.

Does it work for you on the OCE server tho? It works for me on all the other DCs, Just not Materia

Actually not completely, It keeps going to 0 for a few seconds

I have been playing around with it this morning and it looks to be working perfect. Thank you for your hard work!


Windows containerd endpoints and source-vip are causing errors in RKE2.

The containerd socket information has a typo and being generated incorrectly. Those issues have been corrected and unit tests have been added to verify the fix for generating the correct runtime endpoint.
In addition to the above, the crictl template was incorrect due to upstream changes that altered the expected format resulting in the incorrect one being generated.
Lastly, the source-vip had the HNS network hard coded to the flannel value which doesn't allow it to be overridden when other CNIs are used.

Nothing to test; only used in RKE2


Chrome Dev Tools new CSS key is almost invisible

When typing out a new CSS property key, the already-typed letters are rendered as black against the dark background. This makes them very difficult to see.
See example image.

You have to set your theme to "Dark" on the settings panel to ensure default text is light instead of dark.


feat: TextToSpeech Transformer

/azp run

Codecov Report

Merging #1320 (e784b25) into master (46cd375) will decrease coverage by 11.98%.
The diff coverage is 43.24%.

@@             Coverage Diff             @@
##           master    #1320       +/-   ##
===========================================
- Coverage   84.66%   72.68%   -11.99%
===========================================
  Files         283      264       -19
  Lines       13844    12921      -923
  Branches      675      682        +7
===========================================
- Hits        11721     9391     -2330
- Misses       2123     3530     +1407

Impacted Files
Coverage Δ

...oft/azure/synapse/ml/cognitive/SpeechSchemas.scala
80.00% <0.00%> (-20.00%)
:arrow_down:

...t/azure/synapse/ml/cognitive/SpeechToTextSDK.scala
16.47% <0.00%> (-74.12%)
:arrow_down:

...soft/azure/synapse/ml/cognitive/TextToSpeech.scala
48.48% <48.48%> (ø)

...la/org/apache/spark/ml/param/TypedArrayParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...e/synapse/ml/recommendation/RankingEvaluator.scala
0.00% <0.00%> (-88.47%)
:arrow_down:

...soft/azure/synapse/ml/cognitive/AudioStreams.scala
0.00% <0.00%> (-87.88%)
:arrow_down:

...ft/azure/synapse/ml/cognitive/AzureSearchAPI.scala
2.59% <0.00%> (-87.02%)
:arrow_down:

...osoft/azure/synapse/ml/cognitive/AzureSearch.scala
7.19% <0.00%> (-80.58%)
:arrow_down:

...icrosoft/azure/synapse/ml/recommendation/SAR.scala
17.70% <0.00%> (-78.13%)
:arrow_down:

...e/synapse/ml/cognitive/FormRecognizerSchemas.scala
7.69% <0.00%> (-76.93%)
:arrow_down:

... and 60 more

Continue to review full report at Codecov.

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

/azp run


Selected Area polygons aren't being shown with midpoints

Need to restore these!

Fixed in 36ba800e8. Polygons now draw midpoints when selected, and those midpoints can be dragged to add new nodes to the area.
https://user-images.githubusercontent.com/1887955/207729585-b00a5ce0-0aac-4048-8aee-584f1243cfa2.mov


make 'pods' the initial view, when available

@jml pointed out that when running k8s, the container view is rarely what one wants to look at. So perhaps we should make the 'pods' view the default initial view, if available.

TBH, I'd start at "Services", which is the highest level view.
On Wed, 17 Aug 2016 at 11:11 Matthias Radestock<EMAIL_ADDRESS>wrote:

@jml https://github.com/jml pointed out that when running k8s, the
container view is rarely what one wants to look at. So perhaps we should
make the 'pods' view the default initial view, if available.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/weaveworks/scope/issues/1809, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHq6kFh0rxvNJlbhAejld5prM5li8NKks5qgt46gaJpZM4JmS0U
.

Implemented on client w/ 'services' as default if available.

Implemented

where? PR?

To quick for me


No CSS file in dist directory

I'm trying to use this library after installing, and under the example, it shows a css file we should be using under the dist directory.
This is the example:
import React, { Component } from 'react';

// component and styles
import BillboardChart from 'react-billboardjs';
import 'react-billboardjs/dist/billboard.css';

However, when installing react-billboardjs, it's not listed, and importing it will cause errors:

Is there a way to get this css file?

Woops, closing as NPM is not up to date unlike this README.


Flash: enable support for flash on nvram

Alternative to #67687, supersedes #70578.
The PR proposed a method to add support for flash devices defined on top of nvram to enable taking advantage of the fact that nvram does not need to be erased before writing data.
The PR extends the flash_ex_op() function with a code (FLASH_EX_OP_GET_NVRAM_PROPERTIES) to receive information about the nvram backend that is used. At the moment limited to the information that it is a nvram backend.
The result of the flash_ex_op() call can be used to determine if the erase function can be skipped.
Support for the functionality is added for the flash_simulator and flash_ambiq.
The flash_disk driver is modified to use this functionality and a configuration for native_sim in tests/subsys/fs/fat_fs_api is provided that allows modifying the simulator in a simulator on nvram or a simulator on "classic" flash.
The proposed solution does not make any change to existing flash devices that do not have the property that they are on nvram, and does not change the flash api.
When a flash user can take advantage of the property it can skip erases. The flash_write(), flash_read() and flash_erase() will continue to work as expected. If a user would need to "clean" a specific area without erasing a call to flash_write() can be made with data set to the erase-value.
As a extention of the PR the existence of the FLASH_EX_OP_GET_NVRAM_PROPERTIES code could be triggered by a dts compatible that is enabled for flash on nvram devices. The presence of the compatible could be used to retrieve the property without doing a flash_ex_op() call and would allow compile time optimisation.

NACK. This not only forgets that there is already flash_get_parameters, that allows to provide information on device parameters, but also forces more work and more code for devices that actually require less work and less code. Flash Disk is not ENHANCED by the need to call erase on Flash devices, it works better with devices that do not require it. It is EXTRA step step for Flash Disk to erase, it is not-EXTRA step to do no erase. Flash Disk gets parameters anyway and this forces it to do extra SYSCALL on devices it should do no extra work at all. This is complete reverse to what should be done.

The property that a flash device can skip an erase call is actually not a flash_parameter at all. I have never seen a datasheet of a flash device that states this. As it is not a flash_parameter it is better not to make it one.
FLASH_disk is enhanced by this functionality, NVRAM_disk would not require this. A addition of nvram_disk.c (or a expansion of ram_disk.c) that is written directly on top of a nvram driver (or a nvram memory region) is still possible and would be a major simplification over flash_disk.c.

Once more. We are slowly transitioning Flash API to MTD, without single huge rework of everything. We are trying to generalize the Flash API class. We are not setting parameters, we are reporting capabilities of hardware, without taking type of device into account, just it capabilities! We are deconstructing, at driver level, devices to capabilities.

And once more, this shouldn't be done by modifying (some would say bolting it on) the flash API, it is confusing and might be a completely wrong approach.
NVRAM (and EEPROM) devices do not have any property like pages or erase-blocks. A solution that covers these devices should make things like pages or erase-blocks part of its capabilities, they should not be default.
By the way mtd on linux is on a flash device (http://www.linux-mtd.infradead.org/doc/general.html).

Once more. We are slowly transitioning Flash API to MTD, without single huge rework of everything. We are trying to generalize the Flash API class. We are not setting parameters, we are reporting capabilities of hardware, without taking type of device into account, just it capabilities! We are deconstructing, at driver level, devices to capabilities.

And once more, this shouldn't be done by modifying (some would say bolting it on) the flash API, it is confusing and might be a completely wrong approach.

How many times do I have to explain what ROM means in EEPROM? Nobody is confused that device that is READ ONLY DEVICE can be written. People get use to it. People were used to, before introduction of erase_value that flash erases to 0xFF, and everything was hardcoded that way. Now everyone checks erase_value.
This is not a valid argument.

NVRAM (and EEPROM) devices do not have any property like pages or erase-blocks. A solution that covers these devices should make things like pages or erase-blocks part of its capabilities, they should not be default.

If code checks that there is no erase required it can select ANY range to erase. It does not have to check page layout at all!. This may be completely optimized out. And again, this is for common code only. Users may choose to optimize out stuff by hardcoding their operations into values they pick from datasheet.
I have left that for convenience, because slow move from Flash API to generic API requires changes in code that was written only for paged devices to be altered with new set of parameters configurable by user, at a subsystem level if no-device is able to provide such info.

By the way mtd on linux is on a flash device (http://www.linux-mtd.infradead.org/doc/general.html).

And FreeBSD as geom devices: https://svnweb.freebsd.org/base/head/sys/dev/flash/mx25l.c?view=markup.
So, systems use them differently. And in Zephyr Flash == MTD, block devices are handled by Disk Access.

NVRAM (and EEPROM) devices do not have any property like pages or erase-blocks. A solution that covers these devices should make things like pages or erase-blocks part of its capabilities, they should not be default.

As you wish, I have removed these two from API when there is not device that supports it.

This proposal is to complex, it should be much simpler to add support for these type of devices.


Add an input_datetime

Description:
This is a Work-in-Progress proposal. It's not ready for merging yet, but I'd appreciate some feedback.
This adds an input_datetime component allowing to set a date, a time, or both via the frontend. This date / time can then be used by automations etc. to trigger certain events at certain times, e.g. facilitating alarm clocks, time-setpoints for the opening / closing of covers, etc. pp.
Some people have requested this in the forum [0], and there are some hacks around mimicking a 'set time' functionality via a combination of input sliders or input selects (e.g., [1], [2]). Thus, I think there is a demand for such an input (I'd like to use it myself for an alarm clock).
There are some design decision that I'm not sure about. The most important one being: Depending on whether the input is configured to have time, date or both, the status of the input is a date, time or datetime object. I'm not sure whether to default to datetime in all cases. The advantage would be that the class of the state object does not depend on the configuration, the disadvantage would be that I'd need some dummy values for date / time when it's not used.
This is the accompanying polymer frontend PR: https://github.com/home-assistant/home-assistant-polymer/pull/418
[0] https://community.home-assistant.io/t/date-time-picker/9062
[1] https://community.home-assistant.io/t/creating-a-alarm-clock/410
[2] https://community.home-assistant.io/t/alarm-clock-with-sonos-philips-hue/23421
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#3390
Example entry for configuration.yaml (if applicable):
input_datetime:
  test:
    name: 'Test'
    has_time: true
    has_date: true
  test_date:
    name: 'Test Date'
    has_time: false
    has_date: true
  test_time:
    name: 'Test Time'
    has_time: true
    has_date: false

Checklist:
If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] Tests have been added to verify that the new code works.

Whoop, sorry, should really have linted before creating the PR.

Here's some screenshots, of the 'More Info' dialog including the two pickers for date and time (which definitely needs some styling done…), of the date picker, and of a card containing a datetime, date and time input:

At this point, before moving further with other input_xxxxxxx components, would it be a good idea to break this out into a single component (input) with multiple platforms (boolean, select, text, number, datetime)?

@dale3h I actually also thought about that. However, the different inputs behave very differently, I think. For example have a look at the services provided by input select (select next / previous…) and the input boolean (turn on / turn off). While for the input boolean, this could probably be covered by a generic 'set_value', turn on / off is more in accordance with light / switches, I think.
Also, as far as I see it, the UI elements are determined per domain, i.e., would be the same for all inputs. I can hardly imagine some UI that's fitting for a 'select' input as well as for a date / time input or a text input.
As far as I see this, a 'component' describes a certain way of behavior - basically, an interface to the user, while a platform describes the technical way to achieve this. With the inputs, it's somewhat the other way round: The technical way of achieving things is kind of similar (store some state, make it editable), but the interaction with the user is very different.

Yeah the UI work with component part to deside the correct card. I think a component/platform solution is hard for the frontent. Maybe we can convert it later if the UI is ready for that.

Regarding the validation that at least one of CONF_HAS_DATE and CONF_HAS_TIME must be set to True: I just created a has_at_least_one_key_value method in config_validation.py (which basically works the same as has_at_least_one_key) and use it to validate instead of having a helper method in input_datetime.py. Does that look like a feasible approach?

I've fixed the places with FIXMEs (I hope the way I validate the service data is what you had in mind?) and also merged the time / date validators and added tests.

I just created the documentation PR. I think this is ready for merging from my side now.

@pvizeli Did I address your change requests? Or is something left?