Development: Increase client test coverage for team config and submission components

Checklist
General

[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] I followed the coding and design guidelines.
[x] I added multiple integration tests (Jest) related to the features (with a high test coverage).

Motivation and Context
We would like to have a reasonably high test coverage percentage aimed to catch potential bugs before deployment.
Description
Increase client test coverage of team config and submission components to over 80%.
Review Progress
Code Review

[ ] Review 1
[ ] Review 2

Test Coverage

File
Branches %
Lines %

team-config-form-group.component.ts
0% → 100%
29.41% → 100%

team-submission-sync.component.ts
64.28% → 71.42%
30% → 86.66%

The code looks good, I'm just not sure if the checks at the end of team-submission-sync.component.ts really work. The expect calls inside the subscribe blocks do not seem to run at all.

I changed this by invoking done() at the end of the function passed into the subscription. I verified that the test can fail inside this function

I have no idea why this passes; calling done twice should not be allowed or possible, and doesn't make sense ( https://github.com/facebook/jest/issues/7052 ). done should be called once as soon as you're done with everything if you're testing one async function.
To check, try this: If you remove done(); from the second subscription block, the test fails due to failed assertions in the first block.
This leads to the conclusion that test only passed because the second subscription called done() before the assertions in the first block were ran at all.
Luckily, you don't need done() in your test case at all. All observables here are created using of() and do not come with any delay or similar stuff, so you can simply try it like below.
Instead of asserting inside subscriptions, try to get the result from the subscriptions and use these results in expect calls afterwards.
  // Get the results
  let submissionPayload: SubmissionSyncPayload | undefined;
  submissionSyncObservable.subscribe((result) => (submissionPayload = result));

  let submissionRes: Submission | undefined;
  submissionObservableWithParticipation.subscribe((result) => (submissionRes = result));

  // Asserts that results have been set
  expect(submissionPayload).not.toBe(undefined);
  expect(submissionRes).not.toBe(undefined);

  // Perform checks
  expect(receiveSubmissionEventEmitter).toHaveBeenCalledTimes(1);
  ...


reset focus context key when the webview is disposed

Otherwise, commands that depend on the the focus may still be active after it is disposed.

Thanks! Should be in the VSCode 1.20 insiders builds once those start up


accordion; text realignment when opening

For the default accordion and the Free - Can open multiple items simultaneously accordion the text realigns when opening.

Accordions will be replaced by a feature for List tiles (former list items) in v0.7 soon, so not worth it to fix anything for Collapsibles.
The text realigning is an effect of changing the padding for the content. In my perspective I thought it was an interesting effect, but will drop it in v0.7. Anyway, thanks for addressing this!
Closing the bug and please wait for v0.7 aka "Polished" :)


Nanobox doesn't warn of "no boxfile" if there is a boxfile a level up

/path/to/code
  boxfile.yml # <- this is either a note or is for another app
  /app
    notaboxfile.yml # <- this is where we want to deploy from

From /path/to/code/app a nanobox build won't complain about a missing boxfile because there is one at /path/to/code
I tracked it to this line of the LocalDir function. It seems like we would want to only use the current boxfile and not assume users are deploying from a directory deep into their project.

It seems we do want this git-like functionality, we may just need to determine a better location to implement it.


Error in running subctl diagnose from a pod - self pod appears to be 'pending'

What happened:
During subctl diagnose from inside a pod,
the test: checking sumariner pods test failed
✗ Pod ["submariner-diagnose-f4lcq" "Pending"] is not running. (current state is %!v(MISSING))
What you expected to happen:
The test was expected to succeed
How to reproduce it (as minimally and precisely as possible):
Run subctl diagnose from a pod according to this manual:
https://submariner.io/operations/deployment/subctl/#running-subctl-diagnose-from-a-pod-in-cluster
Anything else we need to know?:
Environment:

Diagnose information (use subctl diagnose all):
Gather information (use subctl gather)
Cloud provider or hardware configuration:
Install tools:
Others:

@mkolesnik @vthapar

subctl diagnose checks status of all pods in submariner-operator namespace. Sometimes diagnose pod's status is not updated by the time we check for pod status, resulting in this error. This is mostly harmless, but can confuse the users. Simple fix would be to ignore diagnose pod when checking status in diagnose.

can we only check pods with a certain known label?
or have a label to ignore pods by this check?
this would be much more explicit, and we can utilize k8s filtering instead of relying on pod names (which can obviously change)


⚠ Jenkins Plugins Artifactory has degraded performance

In 6bc75fa, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1271 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 7e6d20f after 3 hours, 41 minutes.


WIP: Fix Feast 0.18.1 Universal Integration Tests 51/66.

Summary

Fix 51/66 integration tests against feast 0.18.1.
For some of the changes to work, must make this patch to Feast 0.18.1 repo detailed in this branch.
https://github.com/kevjumba/feast/tree/feast_offline_store
Tested scenarios

In order to run,

Copy feast into the top level directory under feast_spark_offline_store/
Run *** export FULL_REPO_CONFIGS_MODULE='feast_spark_offline_store.feast_tests' ***

Fixed issue:

@jbvaningen Hey Joost! Could you take a look at this and give me some feedback?

Hi Kevin!
Thanks for your PR, very cool to see you managed to get the feast integration tests running! Happy to add this to the plugin.
My thoughts when going over the changes:

I'm working on this from my work laptop and our pypi mirror does not have all packages available to install feast[dev] so I could not run tests myself. I'm going to work on requesting the required packages!
While you were working on this, I've also been busy adding extra tests, and adding the option to read spark dataframes from file. See the PR here: https://github.com/Adyen/feast-spark-offline-store/pull/18. I think these changes will combine nicely with yours. I'm trying to merge my changes soon, so perhaps you could pull them in when ready and merge?
RegardingSparkDataSourceCreator

I would recommend moving into its own module to keep spark.py clean
We have a method for creating spark sessions get_spark_session_or_start_new_with_repoconfig, I think you can re-use that

Why did you rename our test dir from tests to offline_store_tests?
Why did you place the integration test inside the main feast_spark_offline_store module? IMO this is wrong because we want to avoid mixing test and application code
I understand this is a WIP but code is quite hard to read due to commented lines/imports/formatting Most IDEs offer option to optimize imports, and you can run black formatting to clean it up automatically.


Enable using functions partially

If we have a function like:
    function poseidon_permutation<0> input_in0, input_in1, input_cap -> in0, in1, cap {}

The instruction calling the function currently has to have the same signature:
    instr poseidon X0, X1, X2 -> X3, X4, X5 = poseidon.poseidon_permutation

which compiles to the following lookup:
instr_poseidon { 0, X0, X1, X2, X3, X4, X5 } in main_poseidon.LATCH
    { main_poseidon.function_id, main_poseidon.input_in0, main_poseidon.input_in1,
      main_poseidon.input_cap, main_poseidon.in0, main_poseidon.in1, main_poseidon.cap };

We should add a way to set some arguments to constants and ignoring some outputs, without having to need assignment registers.
For example, setting the third input to 0 and keeping only the first output should yield an instruction instr poseidon X0, X1 -> X2 which compiles to:
instr_poseidon { 0, X0, X1, 0, X2 } in main_poseidon.LATCH
    { main_poseidon.function_id, main_poseidon.input_in0, main_poseidon.input_in1,
      main_poseidon.input_cap, main_poseidon.in0 };

cc @Schaeff so he can pitch closures 😄

Yes my proposal is
instr poseidon X0, X1 -> X2 = |X, Y| poseidon.poseidon_permutation(X, Y, 0).0;

As a former Python / Numpy programmer, I'll throw this into the ring:
instr poseidon X0, X1 -> X2 = |X, Y| poseidon.poseidon_permutation(X, Y, 0)[[0, 2]];

instr poseidon X0, X1 -> X2, X3 = poseidon.poseidon_permutation(X, Y, 0) -> X2, _, X3;

@leonardoalt tuples is just the only complex type we have now, I don't like .INDEX either but it could be solved with structs, or with destructuring like below.
Closures are flexible enough to set inputs and ignore outputs:
instr poseidon X0, X1 -> X2, X3 = |X, Y| {
   let (Z, _, U) = poseidon.poseidon_permutation(X, Y, 0);
   (Z, U)
}

The only problem I see with them is that they are potentially too powerful as you're restricted to a single call. But I'd take that over ad-hoc syntax.

yea I think I agree


Remove builtin commands

Remove builtins from runtimejs, since I added more commands in an optional module called runtime-shell. I've already opened this PR before, but, I accidentally deleted the branch, so... I recreated it and opened it again.

@ArielAbreu I think it's a good idea to move commands out of core to runtime-shell, except maybe reboot/poweroff because they're very system dependent.

Like echo and clear are part of shell for sure.

I'll redo the PR because it I accidentally deleted the repository.

except maybe reboot/poweroff because they're very system dependent

I would move them too, and only allow them as an API.

I agree with @piranna, anything that is added for convenience can (not must) be moved out into a module.

@piranna hm, I use them very frequently when debugging runtime in KVM in curses mode, however I don't need full shell for that

(I can remotely ssh to linux machine and run runtime in KVM without graphical interface)

I've been working on a telnet server for runtime which listens on port 9000 (the forwarded port) which can be used for more direct remote control. That aside,  if shutdown and poweroff are going to be the only commands, maybe they could be moved into index.js with command 1.

@ArielAbreu sounds good, let's do this

Opened #69.


Hardcoded LoginView

https://github.com/pydanny/django-admin2/blob/develop/djadmin2/viewmixins.py#L104-L113

Thanks @pahaz for spotting that. Could you contribute fix for that in #414 ?


⚠ Samsung has degraded performance

In 5f7297f, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2360 ms

Resolved: Samsung performance has improved in 750e36d.


unable to run the example on my local machine via go run main.go

Hello,
I am trying an example at this below location and wonder any help if you can help me understand why I couldn't run the program using go run main.go but was able to run the binary which is the outcome of go build
https://github.com/charmbracelet/bubbletea/tree/master/examples/list-fancy

list-fancy consists of multiple source files (ending in .go) and your trying to run only a single one of them.


"unset" doesnt work as it expected on event saving when the object is updated

When i'm going to save a object (update or create) I remove some properties, example
this.on('saving', Promise.method(function(model, attrs, options) {
   if (mycondition) {
     model.unset(key);
   }
}));

the previous code must be run when the object is created or updated, but for some reason when the object going to be updated i have to an extra delete
detele attrs[key];

This is a bug?

What happens if you don't delete the attribute in this case?

I get a error: property doesn't exist on the table

What's the SQL output? And does the property actually exist in the table?

I get
update "resources" set "id" = $1, "is_private" = $2, "tags" = $3, "title" = $4, "topics" = $5, "type" = $6, "updated_at" = $7, "url" = $8 where "id" = $9 - no existe la columna «tags» en la relación «resources»

I property i need remove is tags I did before save I get if I dump model.attributes I dont see it, on create the object works fine but on update I need remove also from attrs

I didn't know you can use promises mixed in with events like that. What happens if you remove the Promise.method?
Also, what is the value of tags that is being passed to that query?

the value of tags is an array [ {..}, {..}, {..}], I remove it before save because I handle it in another place
I use Promise.method to bubble up all throw new Error(...) I have my .catch() in only one place

You don't need Promise.method because "saving" accepts a promise to be returned. It is redundant, but not wrong. However it should not be causing an issue.

i have to an extra delete

Are you saying that the attribute is not unset?

Yes,  I means it's removed from the model.attributes but if you don't remove from attrs then it's passed to the update query but this only happens on updated creating don't present this problem

So, Model.forge({id: 5, value: 'hello'}).unset('value').save() would generate update model where id = 5 set value = 'hello';?

var Model = bookshelf.Model.extend({
  initialize: function() {
    this.on('saving', function(model) {
       model.unset('value');
    });
  }
})

Model.forge({id: 5}).fetch().then(function(model) {
   return model.save();
});

The previous is generating the update model where id = 5 set value = 'hello';

Thanks @rkmax. I'll look into it.

@rhys-vdw I think maybe has some relation with multiples  saving events attach to the model, because My models extends from Another and this model his own saving event listener, my extend call
Parent.prototype.initialize.apply(this, arguments);

Recently I test with another model even the unset is not enough I have to do
delete model.attributes.stepHrefImage;
delete attrs.stepHrefImage;

Is the "saving" callback being called?

Yes

Hey @rkmax. Just looking through Model#save. There is a bit of dodginess here I intend to clean up while addressing #814.
So, this will always happen on patch, but that makes sense and is by design.
However, it seems this could be a problem when defaults are set on model.
If you have defaults on your model, that would explain the problem.

Actually I just realized something... Your example here doesn't actually set value to "hello". Can you show me where you're setting the field?

@rhys-vdw I've updated the example

Oh right, you're passing the value through in the save attributes. Yep, that's just how it works. You're right, you'll have to unset it from the attributes argument too. Planning to deprecate the first argument to save soon, I'd recommend just using model.set(attrs).save() instead.

my bad, @rhys-vdw Looking my code I found I'm using .set().save()

Well I think it's probably safest for now to handle the attrs argument as you were. This is what I've done in the past. I'll revisit this when we go over #814.


Disable Beta Popup Dialog

Disabled Beta Popup Dialog:

The component is not deleted in case we need it back. So far its import is commented.
Removed the unused imports in the component.

The Playwright tests are actually testing the pop up as well. Now that it is gone, they fail. Try to fix them. It should be as easy as removing the lines that actually interact with the popup. 👀


grid-list problem

Document demo page here
Hi:
When mouse hover at grid list, should the window scroll bar be disabled?

@EcutDavid,  grid is there in a overflow auto container that has its own scroll. This is why while scrolling over grid window scrollbar is "disabled". Did I understood correctly your issue?

@igorbt
Thank you!
But I mean the window scrollbar should still "disabled" after scroll to the grid bottom.

After  reaching the end of the grid scrolling will continue on window. That's just how normally scrolling behaves. Grid list don't alter any scrolling behavior. Or you think it should be changed? In what way?

@igorbt
Hi, yes, I think it should change, may be I'm wrong :)

I guess it's very normal, closing this one.


"Be sure to close your comment by deleting all trailing comment tags, i.e. -->." should read something like "Be sure to close your comment by adding -->."

Challenge Waypoint: Specify How Fonts Should Degrade has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/46.0.2490.86 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.

I agree with your proposed verbiage:
Be sure to close your comment by adding <code>--></code>.

I modified the text as suggested. Can you take a look when you get a chance?

It on my list :smile: - just need to get to my dev box.

Oops. I am sorry. I should have asked for it before I started working on it. Anyways this motivated me to set up the workspace. Just close it when you get a chance.

Closed by #4697


Webpack stops with no error - html/Uglify issue

It seems like there could be a problem with new Vue.js syntax and how html is uglified by Webpack.
When I run webpack with no uglify plugin it works.
When I run webpack with uglify or just wepack -p than it stops (see screenshot)
When I remove new syntax ( @click="dismiss" ) and replace it with (v-bind:click="dismiss") in my html, it works in both cases.
To replicate the issue just require this file in your test on win10 64 with standard webpack config.
It could be related to https://github.com/webpack/webpack/issues/522 and https://github.com/webpack/webpack/issues/537

To quote @sparty02:

I am having this same issue and I think I've narrowed it down to something related (either directly or indirectly) to a combination of using UglifyJsPlugin in conjunction with html-loader. Looks like there is a related issue going on with #1573.

For triaging, I commented out the minification logic in the html-loader here and, sure enough, my bundle finished fine.

i'm doing something with vue.js(yeah, ^1.0 version) and webpack, and i found a similar problem...
i cannot compress my project now and this problem really annoys me.
:sob:

@zcoding Just use full syntax like so v-bind:click="dismiss" and you should be fine. It worked for me.

I am having the same problem, and I am not using Vue.js or @ syntax in any html templates.  All of the my templates are just standard html.  I tried commenting out the code @sparty02 suggested and it fixed the issue for me as well.

html?minimize=false also made it work for me

For those Vue users that are stumbling upon this, there's a vue-html-loader: https://github.com/vuejs/vue-html-loader/, which allows you to minimize and use the @ syntax.

@blacksonic Have you been able to minify html?

haven't tried

Is this relevant still?


cam plugins variables

I think there's a bug in the camera plugins - there's not enough variable query in them:
header_auth=$(uci -q get telegramopenwrt.ipcam.header_auth)
ipaddr=$(uci -q get telegramopenwrt.ipcam.ipaddr)
user=$(uci -q get telegramopenwrt.ipcam.user)
passwd=$(uci -q get telegramopenwrt.ipcam.passwd)

Hi, @nikkymen
Depends on the camera that you use. If the authentication is a Basic, is just the "user: password" in base64.

What type of authentication your camera has ?
The code to call the camera is this:
curl -s -k -H "Authorization:Basic ${header_auth}" "http://${ipaddr}/videostream.asf?user=${user}&pwd=${passwd}" -m 25 -o /tmp/movie.asf -D /tmp/header

What type of authentication your camera has ?
The code to call the camera is this:
curl -s -k -H "Authorization:Basic ${header_auth}" "http://${ipaddr}/videostream.asf?user=${user}&pwd=${passwd}" -m 25 -o /tmp/movie.asf -D /tmp/header

You use variables like {ipaddr}, but not define them in the camera plugins examples


Improving the code indentation in the html course and fix some syntax error in some codes

🛠 Fixes Issue #430

👨💻 Changes proposed

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

Please review @arpit456jain @singhmansi25 @KKhushhalR2405 @raghavTinker


Restart Cube

Hi,
thank you very much for this amazing project.
I would ask you, if it is possible to add restart cube functionality.
Thank you very much.
Ales

I am really alos interested with this feature

It is in my roadmap. I hope to get it ready in HASS before end of February.

It is in my roadmap. I hope to get it ready in HASS before end of February.

Okay, contributing code to HA is much harder than anticipated. Maxcube component in HA was not maintained and there is a lot of work before I can commit to this piece of work. I hope to get it done before end of year.


support for multiple classes in a string

I gotta believe most people write their classes as a string instead of an array.  This little change just adds a check to see if someone put in an alternative way to name classes.  eg.  ["params" => ["class" => "alert alert danger alert-dismissible"]], where as now (without this change you have to put each class as a separate array element. eg.  ["params" => ["class" => ["alert", "alert-danger", "alert-dismissible"]]]

Forget this, I found an issue with it which causes a php warning.


GDScript 2.0: Compound primitives with setters cannot be modified without replacing the whole value

Godot version
4.0dev, 8c162f4a7ba0f542cd7e9fbbe30f004504cc0809
System information
Windows 10
Issue description
If you have a property with properties (Vector2, Rect2, etc) that has a setter, I.E.
var rect: Rect2:
	get:
		return rect
	set(value):
		rect = value

and you try to modify its contents by calling its properties, I.E.:
rect.position.x = 50
rect.size.y = 50

the properties aren't actually changed and the setter is not called unless you go and change the whole property, I.E.:
var new_rect = rect
new_rect.position.x = 50
new_rect.size.y = 50
rect = new_rect

Steps to reproduce
extends Node

var rect: Rect2:
	get:
		return rect
	set(value):
		rect = value

func _ready():
	rect.position.x = 50
	rect.size.y = 50
	print(rect) # [P: (0, 0), S: (0, 0)]

	var new_rect = rect
	new_rect.position.x = 50
	new_rect.size.y = 50
	rect = new_rect
	print(rect) # [P: (50, 0), S: (0, 50)]

Minimal reproduction project
No response

Currently, the MRP works if you remove the getter, even if the setter remains. The remaining issue is probably related to this: #54829


python3Packages.jaxlib: fix darwin build

Description of changes
Fix the darwin build for jaxlib.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@ofborg build python39Packages.jaxlib

@ofborg build python3Packages.jaxlib

Build timed out: https://logs.nix.ci/?attempt_id=96c8179f-ec2f-44ea-b029-ef19bdc1255b&key=nixos%2Fnixpkgs.184395

I don't have x86_64-darwin to test, but I can try on arch64-darwin

On ofborg it failed to build aarch64-darwin, and not due to timeout, some table gen issue.

Ah no I remembered it wrong, it was because it can't find a darwin_arm64 toolchain: https://logs.nix.ci/?attempt_id=4a181b56-e474-4456-8ac7-47daf859ad27&key=nixos%2Fnixpkgs.184395, probably something to do with the different ways M1 identifies itself, I think I've hit it before. Sorry I didn't want to look more into aarch64-darwin because mine is a 13", so the builds take forever.

Addressed review.

Fixed it on aarch64-darwin too, so you can test it locally now.

Sadly it seems that darwin builds may be re-broken until we can get macOS SDK 10.14 into nixpkgs (https://github.com/NixOS/nixpkgs/pull/183051) 😞

That should be ok at least from an SDK point of view, we're using the macOS 11.0 SDK here, that's what the pkgs.darwin.apple_sdk_11_0.stdenv is about. Hopefully it will just work or require only small changes.

Does anyone know why ofborg won't pick up python3Packages.jaxlib from the PR title automatically?

@ofborg build python3Packages.jaxlib python39Packages.jaxlib

Would also be good to figure out why the linux derivations fail to evaluate on ofborg, they build properly in hydra.

https://github.com/ofborg build python3Packages.jaxlib

@ofborg build python3Packages.jaxlib

Result of nixpkgs-review pr 184395 run on aarch64-darwin 1

34 packages marked as broken and skipped:

python310Packages.aeppl
python310Packages.aesara
python310Packages.arviz
python310Packages.augmax
python310Packages.chex
python310Packages.dalle-mini
python310Packages.distrax
python310Packages.dm-sonnet
python310Packages.elegy
python310Packages.flax
python310Packages.objax
python310Packages.optax
python310Packages.pymc
python310Packages.rlax
python310Packages.tensorflow-datasets
python310Packages.treex
python310Packages.vqgan-jax
python39Packages.aeppl
python39Packages.aesara
python39Packages.arviz
python39Packages.augmax
python39Packages.chex
python39Packages.dalle-mini
python39Packages.distrax
python39Packages.dm-sonnet
python39Packages.elegy
python39Packages.flax
python39Packages.objax
python39Packages.optax
python39Packages.pymc
python39Packages.rlax
python39Packages.tensorflow-datasets
python39Packages.treex
python39Packages.vqgan-jax

10 packages failed to build:

python310Packages.dm-haiku
python310Packages.jax
python310Packages.jmp
python310Packages.numpyro
python310Packages.treeo
python39Packages.dm-haiku
python39Packages.jax
python39Packages.jmp
python39Packages.numpyro
python39Packages.treeo

Fixed it on aarch64-darwin too, so you can test it locally now.

Hmm I'm having trouble building on aarch64-darwin: https://gist.github.com/samuela/471a67182e4c7dded61675fba72f18cf. Looks like it may be a 404

That's a red herring, it's strange that it's not available on the bazel mirror, but it will get it from github: https://github.com/bazelbuild/rules_cc/commit/081771d4a0e9d7d3aa0eed2ef389fa4700dfb23e.
The error is:
ERROR: /private/tmp/nix-build-bazel-build-jaxlib-0.3.0-deps.tar.gz.drv-0/output/external/local_config_cc/BUILD:48:19: in cc_toolchain_suite rule @local_config_cc//:toolchain: cc_toolchain_suite '@local_config_cc//:toolchain' does not contain a toolchain for cpu 'darwin_arm64'

Are you sure you checked out the latest commit? That's what the patch file is supposed to fix.

I'm seeing the same error on ofborg, will try to repro, there might be something wrong with my own setup.

But maybe let's merge the working x86_64-darwin build in the meantime.

@ofborg build python3Packages.jaxlib

The failure on ofborg is:
ERROR: /private/tmp/nix-build-bazel-build-jaxlib-0.3.0.drv-0/output/external/org_tensorflow/tensorflow/compiler/mlir/hlo/BUILD:400:18: TdGenerate external/org_tensorflow/tensorflow/compiler/mlir/hlo/lib/Dialect/mhlo/IR/mhlo_canonicalize.inc failed: (Illegal instruction): mlir-tblgen failed: error executing command bazel-out/darwin-opt-exec-50AE0418/bin/external/llvm-project/mlir/mlir-tblgen -gen-rewriters external/org_tensorflow/tensorflow/compiler/mlir/hlo/lib/Dialect/mhlo/IR/mhlo_canonicalize.td -I ... (remaining 17 arguments skipped)

I've had that locally when nix was confused on what the current system architecture was, because it was running on an M1 with rosetta, but I'm not sure why that would happen on an x86_64 mac. I've seen in it some of the past builds too, but not all, so it seems to only affect specific machines.

I've had that locally when nix was confused on what the current system architecture was, because it was running on an M1 with rosetta, but I'm not sure why that would happen on an x86_64 mac. I've seen in it some of the past builds too, but not all, so it seems to only affect specific machines.

Huh, very odd... doesn't look like your fault in any case. I would be curious to figure out the underlying issue, but doesn't need to be in scope for this particular PR.

Thanks for all your hard work on this @uri-canva! These changes LGTM and are a big leap forward in terms of our ability to support more platforms.


Optimization: constant propagation

Lots of things can fall under this:
say 1 + 2
var x = 10
say x + 20
say ((1 + 2) * 3) - 4
say {1, 2, 3} * 4
say {{1, 2, 3}, 2, 3}[0][1] + 4
x = 1
say {1, 2, 3}[x]

etc, etc,
Not sure how far down this rabbit hole I want to go.

Would be nice if str.hash also could be calculated at compile-time.

str.hash can be static now: https://github.com/voidqk/sink/commit/d5c64587faa3c6307c139e6af92511806d4a6559

A lot of these work, I think we're far enough along... closing.


Remove check on intSlice type from config map validation

PR #20304 fixed the validation of the config map options at startup. The validation is successful if each option value, represented as a string, can be converted to a value of the specific option type, using the related type conversion function.
Due to some limitations related to the intSlice option type in the spf13/cast package, PR #20282 forbade its use. Therefore, no validation related to an intSlice option should take place either be successful.
This PR removes the branch related to intSlice in the options validation code.

/test


⚠ Minecraft has degraded performance

In 3fb81e3, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1115 ms

Resolved: Minecraft performance has improved in 561a188.


Autocomplete doesn't work in OS X Safari

Autocomplete has some trouble I don't understand with Safari, while in Chrome it works perfectly well. I spent a few hours on that without seeing any solution. Help will be appreciated, thanks.
This is my webpage https://rodighiero.github.io/DH2020/
This is library implementation https://github.com/rodighiero/DH2020/blob/master/src/elements/search.js

Hello @rodighiero,
I've checked your nice webpage on safari, and I believe that there another javascript code that's preventing the user from typing inside the input field more than one character for some unknown reason, which I'm trying to figure out.
Something else I've noticed that's worth mentioning that when you change window focus and get back to your page, you get to write another single character.
I'm currently investigating your issue, and I'll let you know as soon as I find out any clue.
Have a nice day! :)

Thank you @TarekRaafat,
I took a look at the bundle, but it seems okay. I use just two libraries that should not affect any input listener. They are PIXI.js and d3.js. Please tell me if I can do further checks and keep me updated!
Have a nice day you too!

I've found the problem and it was not in the library. Safari needs to add this code in the CSS:
input, input:before, input:after {
	-webkit-user-select: initial;
	-khtml-user-select: initial;
	-moz-user-select: initial;
	-ms-user-select: initial;
	user-select: initial;
}

Thanks for helping me,
Dario

Interesting finding I've never came across it before.
I'm glad that you've found the solution at the end of the day.
Anytime, cheers! :)


User Secrets Not Compatible with Visual Studio Docker Support

Summary: User secrets cannot (easily, portably) be used with Visual Studio's "Docker Support" feature for ASP.NET Core projects.
Hi! I've been using User Secrets for a while in several ASP.NET Core projects. My company is evaluating a move to Docker for deployments, and – as part of that evaluation – I activated the Docker Support feature in Visual Studio. After this, the application completely failed to reference my secrets.
I've worked around this for my own support by bind-mounting (in docker-compose.override.yml) the secrets.json file on my development box to the correct location in the Linux container, but this seems to violate Microsoft's advice:

You should not write code that depends on the location or format of the data saved
with the Secret Manager tool, as these implementation details might change.

...in addition to not being portable between our Windows and Linux developers.
Versions of things

Microsoft.Extensions.Configuration.UserSecrets: 1.1.1
Microsoft.Extensions.SecretManager.Tools: 1.0.0
Visual Studio: 15.1 (26403.7)

ping @natemcmaster

The advice was written with authors of third-party class libraries and tools in mind. The location of the files is not "under contract" so your docker-compose file might break in future versions. However, until that happens, volume mounting the user secrets location for development environments is fine, and seems like something Docker Tooling could consider doing by default. cc @dazhao-msft
That said, mounting UserSecrets for production is not recommended. For production secrets, checkout https://github.com/aspnet/Configuration/tree/dev/src/Microsoft.Extensions.Configuration.DockerSecrets

[...] mounting UserSecrets for production is not recommended.

Right on, no worries there.

[...] volume mounting the user secrets location for development environments is fine [...]

To say things that I'm sure you know, only to get them on the written record:
This solution moves a little backwards from (what I perceive to be) the design of User Secrets, as this means we have a file in source control (docker-compose.override.yml) that some of our developers will have to "forget" to commit their change to. Though, thanks to User Secrets, that change is only a Windows vs. Linux path. Additionally, this requires us to copy and paste the magic value of the user secrets ID – which is otherwise system-managed – into this file, twice.
That said, operating like that will certainly be OK for now. It doesn't not work. 😛 And the feature is valuable enough to get over the inconvenient ergonomics.
Thanks for the assist, @natemcmaster. Would it be better for me to close this now, as the workaround still works, or wait for @dazhao-msft to chime in?

I know ASP.NET team is working on something to resolve this issue for runtime. Check this out:
https://github.com/aspnet/Configuration/tree/dev/src/Microsoft.Extensions.Configuration.DockerSecrets
@glennc has more context about it.

@dazhao-msft the DockerSecrets config provider is about loading configuration from "docker secrets" - a Docker feature for swarm mode.  UserSecrets is a separate feature, not related to Docker.
The request here is to flow the UserSecrets data into containers when running in development mode (e.g. launching container debugging in VS). Docker Tools for VS already volume mount some folders -- such as the NuGet cache -- to preserve the context of the host machine inside the container. Adding the user secrets folder to that list seems reasonable to me.

@natemcmaster does this issue belong in a tooling repo/issue, can you route this appropriately/close if there isn't anything for us to do in Configuration

This issue belongs to the tooling team. There are no actionable items from the runtime.
@mlorbetske or @dazhao-msft  - where should we move issues for Web / Docker Tooling in VS? TFS, https://developercommunity.visualstudio.com/, or some other GitHub repo?

@natemcmaster - Thanks for bringing this to our attention.  David has moved on from the Docker Tools team, but @yanchenw and I are good contacts going forward.  I'd suggest that GitHub issues for Docker Tools be moved here: https://github.com/microsoft/dockertools/issues

Thanks @sgreenmsft. Let's move this to https://github.com/Microsoft/DockerTools/issues/24


Request for a stop-circle icon

Hi, I know this might be trivial but it would be great if we can have a stop-circle icon like this one https://iconify.design/icon-sets/mdi/stop-circle-outline.html since there are already official play-circle and pause-circle icons.

That icon already exists: import StopCircleOutlinedIcon from '@material-ui/icons/StopCircleOutlined';

topCircleOutlinedIcon

Hi, thanks for pointing out. It's weird that this icon does not appear with either keyword 'circle' or 'stop' (https://material-ui.com/components/material-icons/)


Traffic history and last 10 received payload on App and device level

It would help if I can see the the last 10 received payloads and a traffic history on App and device level.

It would be good if the LoraWan frame logs keep something like the last 10 messages. Right now the list is cleared everytime you change the manu on the lora app server. Keepping the last messages helps to debug issues on the nodes.
Sigofx backend and another Lorawan services provides this kind of log.

Currently I don't have any plans to support this, as it would also impact the performance (which happened for some people when logging all frames the the database, which is why this moved to live data using web sockets recently).


Create "Warfighter" on spark demo environment

"Warfighter" needs to be created on spark demo server.

Create these on YouTrack, please.


Adding Project

This PR fixes #54

Very simple and has no functionality
Make projects with creative ideas and add some functionalities


Try to debug TestService

Closing PR, as I can no longer reproduce the test failure I was trying to debug


FIX: Handle split line breaks in metadata chunks

@ironman-machine check it

Hello @LaurenSpiegel
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "SCALITY_S3_BRANCH": "fix/v4chunks", "DEFAULT_BRANCH": "rel/6.3"}
Please follow http://ci.ironmann.io/gh/scality/Integration/7895 for CI status.


support deny import operations for incompatible components

Development Task
cc https://github.com/pingcap/tidb/issues/46850

/affect 6.5


Refactor logging config

Description
Refactor logging config.
Type of change

[x] Refactor

/lgtm

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

Project coverage is 95.43%. Comparing base (ab8cb14) to head (1f4b585).
Report is 10 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1130      +/-   ##
==========================================
- Coverage   95.45%   95.43%   -0.02%
==========================================
  Files          56       56
  Lines        2374     2364      -10
==========================================
- Hits         2266     2256      -10
  Misses        108      108

Files
Coverage Δ

ols/app/models/config.py
95.70% <100.00%> (-0.08%)
:arrow_down:

/approve

/retest

/unhold

/override "Red Hat Konflux / lightspeed-service-on-pull-request"


Truncate user name in dashboard to avoid line break.

Uses the truncatechars template tag to avoid line breaks in the dashboard sidebar.

Current coverage is 49.97%

Merging #498 into master will not affect coverage as of ac11b02

@@            master    #498   diff @@
======================================
  Files           90      90
  Stmts         4140    4140
  Branches       427     427
  Methods          0       0
======================================
  Hit           2069    2069
  Partial         51      51
  Missed        2020    2020

Review entire Coverage Diff as of ac11b02

Powered by Codecov. Updated on successful CI builds.

Wouldn't class="truncate" work better here?

The styles of the menu expect inline, so we would need to add an additional class to set it to inline-block as well as the custom width. This seemed the simpler solution.

Agreed it's simpler this way but with certain letters it may look weird (22 letters "w" are not the same width as 22 letters "i" etc.).

That's fair. 21 'W's will definitely not fit. I'll update my branch.

We are redesigning side navigation and header areas. Changes preventing user name overflowing will be merged in other pull request.


Update proposed new versioning rules for easier preview build usage

Summary of the PR
Updates the proposal for the software development plan going forward such that preview/development builds are considered newer than the release they follow by standard semver/nuget conventions.
Related issues, Discord discussions, or proposals

@ThomasMiz @Beyley @HurricanKai PTAL - SDP proposal changes.
Added to the Next Working Group Meeting milestone.

Approved provided we change it so that we just upversion to a higher version number (even if it isn't the final version number) after shipping an update


IOS Only Vibrations

Can you explain what exactly is pop, peek and nope vibrations in readme?

Done ;)

Hello, how long is the duration for weak boom and strong boom?
Thank you


Fixes Issue 2 Sponsor Interest Form

Created a form and an event handler function. We'll want to send this data somewhere I'm assuming, but for now it's being saved as variables.
Also, we'll likely want to import this component into a Sponsor's page

I'd rebase: squash the comment into the first commit
and submit package-lock.json in a different PR

Hi @lucasvocos.  package-lock.json has been merged in a different PR, would you mind removing it from this one, to resolve the conflict?


Support for vault kv version 2

In vault 0.10 just released there are two versions of the kv engine with different apis ( path, request format etc)

Duplicate of #239


GD-503: Rework plugin exit to avoid system crash and memory leaks

Why
The plugin has some issues to release resources at Godot exit and result sometimes in a segmentation fault under Linux
What

fix remove_child on free_instance to be called deferred now

@AnidemDex please check out this branch, and check if you run in errors or crashes.
I tested on Windows and macOS, the tests runs and when exit Godot no real errors I see, also no orphans on Windows, but leaked images on macOS only

Segfault is no more


remove unreachable statement

Is "input.dim() == 0" unreachable as it could only be 2 or 3 in assertion of Line 329?

According to "transcribe_streaming" interface, length should be with shape (B,).
So is the original if input.dim() == 0 statement a typo of if length.dim() == 0?

According to "transcribe_streaming" interface, length should be with shape (B,). So is the original if input.dim() == 0 statement a typo of if length.dim() == 0?

Hi @glynpu, yes, you're right — this is a typo. Thank you for the fix!


Fix example not working without a page reload on first open.

Summary
Resolves #782.
Set of changes
The crucial change is adding self.clients.claim();. Without this, overriding fetch in the service worker does not work until the page is reloaded.

@amyboyd  I just tried this out but it looks like the problem still persists. Did you manage to unregister the service worker before reloading?


Support Conditional Logic in Telemetry Configuration for Dynamic Metric Labels EnvoyFilter

(This is used to request new product features, please visit https://discuss.istio.io for questions on using Istio)
Describe the feature request
In our k8s cluster, we have enabled the labels "request.url_path" and "request.method" and dont include "workloadSelector" in Envoyfilter to specify the apps, so that every API hosted on the the cluster would have those two labels reported in Premetheus server.
However, there is a scenario that if any API has path parameters (e.g., /v1/message/{conversationId}), Istio's telemetry captures the entire URL in metrics, leading a high cardinality in Prometheus.
The use of Lua and WASM filters can modify request/response metadata, the modification only affects the Enovy dynamic metadata space. However, the dynamic metadata is not directly used by the default istio telemetry set up (i.e, 'istio.stats' filter) when reporting metrics to Prometheus.
Describe alternatives you've considered
I come cross this post https://stackoverflow.com/questions/30967822/when-do-i-use-path-params-vs-query-params-in-a-restful-api. and this example use case is modifying the request.url_path metric label based on the presence of a custom header (like x-prometheus-url-path). If the header is present, the metric label would use the header's value; otherwise, it would default to the standard `request.url_path.'. And this solution wont wont work unless that EnvoyFitler could support conditional logic in Telemetry configuration for dynamic metric labels.
I propose adding a feature to Istio's telemetry configuration that allows for conditional logic in defining metric labels. For instance, a syntax resembling a ternary operator (e.g., condition ? value_if_true : value_if_false) could be used to dynamically set a metric label based on the presence of a specific header.
Benefits:

Reduced Cardinality: Allows for more control over metric labels, reducing potential high cardinality in Prometheus, especially in APIs with path parameters.
Enhanced Flexibility: Provides users with more flexibility in defining how metrics are labeled based on the dynamic context of requests and responses.
Improved Telemetry Customization: Enhances Istio's telemetry customization capabilities, aligning with diverse user needs and use cases.

Affected product area (please put an X in all that apply)
[ ] Ambient
[ ] Docs
[ ] Dual Stack
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ x] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Affected features (please put an X in all that apply)
[ x] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane
Additional context

is this what you want?


Missing TPIC in response header if mvc-interceptor chain is interrupted

hello,
currently, the logic for writing back the TPIC id is implemented in "postHandle" and not "completion".
If one of the mvc-interceptors throws an exception or returns "false" for "postHandle", the "postHandle" is never invoked of following interceptors in the chain, e.g. request with invalid model (@Valid on mvc-controller):
i can debug miy code (spring 4.1) that the tracee-filter gets invoked for "preHandle" only, but not "postHandle", as the mvcValidator interrupts the mvc-interceptor chain.Therefore no tracee-id is written back in response-header.
solution: move the logic for writing TPIC in response header from "postHandle" to "completion". If so, you must also check for already committed responses, as headers can only be written, if the response is not committed yet.

Hey @rfelgent ,
you're right. Spring skips over the method postHandle in case of exceptions. This wasn't clear to me because this behavior isn't mentioned in the javadoc of the org.springframework.web.servlet.HandlerInterceptor interface.
I've checked the HttpServletResponse in the afterCompletion method and it's still open. So I agree to your solution: We move the logic of writing the tracee response header to the method.


fix: anchor link issue with hash navigation

closes #10234
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[X] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[X] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[X] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

I think the issue was caused by load_cache. Inside the click listener there is check:
if (current.url.hash === url.hash) {
	event.preventDefault();
	a.ownerDocument.getElementById(hash)?.scrollIntoView();
	return;
}

which was true and the function returned so no navigation occured. The current.url contained the url with the hash while we are on the '/' route probably because load_route returns the previously set promise.  I am not 100% sure if setting load_cache = null after hash navigation and update_url is the best solution (trying to get familiar with the codebase).
If it's ok, let me know if I need to add a changeset. Thanks!

Hi @geodask , thanks for submitting a PR for this. I've added some additional findings here https://github.com/sveltejs/kit/issues/10234#issuecomment-1752044976 I think we may have to do more than clear the load_cache at the point in time of clicking the hash link. Ideally, we would have a changeset and test for the solution too.

Hi @s3812497, thanks for your feedback! Due to some commitments during this period of time, I won't be able to tackle the additional work required here. If anyone's up for it, please feel free to take over this PR.

Closing due to inactivity and because it seems the issue is stemming from a different source.


Version of Gobblin dependencies are missing from gobblin-mapreduce.sh

Version of Gobblin dependencies are missing from gobblin-mapreduce.sh
During the creation of the Gobblin tarball the current project version is applied to the jars
however, the dependencies in the gobblin-mapreduce launcher script have no version.
This patch introduces a placeholer GOBBLIN_VERSION in the script which is substituted with the actual version during :gobblin-distribution:build.

LGTM

Thanks for contributing! Merging.

@sahilTakiar @chavdar Thanks for the review!


Graph zoom-in/out moves objects in an offset manner

Describe the bug
In the global graph view (see To Reproduce screenshot), any zoom except the default one will lead to theses behaviors:

Default zoom: When dragging an object everything is fine.
zooming-in: When dragging an object, the dragged object will go further than your mouse in the dragging if we take the starting point of the sliding as a reference.
zooming-out: The dragged object will go less far than your mouse in the dragging if we take the starting point of the sliding as a reference.

To Reproduce

Use a web browser (tested on chrome/firefox/Edge, same issue)
Go to any quartz vault
Open Global graph view on the graph preview at the right (You can quick test on https://quartz.jzhao.xyz/)

Try slowly dragging an object from a point to another far away: this should work fine
Now zoom-in with mouse wheel (not browser zoom) only once*: The dragged object will go further than your mouse in the dragging if we take the starting point of the sliding as a reference.
Now zoom-out 2 times (1 to go back normal zoom, another to zoom-out 1 time) and test again. The dragged object will go less far than your mouse in the dragging if we take the starting point of the sliding as a reference.

The best way to see that this is a real annoying bug is to:

zoom-in a lot, then drag something, like this:

Expected behavior
When zooming in and out, the object dragged should always be at the mouse position.
Screenshots and Source

You can try yourself because you can really feel the bug more than see it (You can't see if my clic is still maintained on a video or not for example).
Help to speed up fixing the problem
Providing a simple reproduction
I've none, sorry, i just know for sure that the bug in on quartz side, not browser side.
Quartz repository of the bug
Any repository as long as it is published and you can see that graph. (works on quartz.jzhao.xyz graph)
Additional context
I never used Quartz, i was just chilling in the issue list on the github, then clicked on someone's quartz vault and found this bug.
I then tested on quartz official one (since it should be the most up to date one) and same.

Hi there, can you record a video showcasing this "bug"?

https://github.com/jackyzha0/quartz/assets/29749331/714dcb28-f708-4841-aa79-3e05d89040df
The mouse behaviour after scrolling in

I think this is there but im not sure
https://github.com/jackyzha0/quartz/blame/bd05950c2db1335b68c8e1701385219e681a4241/quartz/components/scripts/graph.inline.ts#L168

yeah probably need to account for scaling factor

closed via https://github.com/jackyzha0/quartz/pull/1328


[WIP] Remote inference

WIP; DO NOT MERGE;
This PR adds remote inference abilities to DL4J/SameDiff models

Built-in interface adapters (common classes users might want to use) - List, INDArray, DataSet, float[], double[], double, etc

INDArray/DataSet interface would assume that nd4j apis + impls will be introduced to client scope, which kind of kills idea of separate json serving. But will have great sense for grpc-based serving. Maybe don't touch those until we provide grpc (or any other rpc) support for dl4j/samediff?
as for float[] etc - absolutely agreed. Implementations added for String, Integer, Boolean, Float, Double, float[], double[].

version properties in pom

Unirest version change was rolled back, so there’s no changes anymore.

I'd probably even consider a "JSON via java serialization and base64" option for quick-and-dirty (or complex object) serialization

I’d prefer to avoid this option as long as possible...

it’s non-efficient
it makes json serving concept java-locked.

So, maybe we leave that option to (g)rpc as well? Avro, protobuf etc...

Oh, and I forgot to mention in the review - if I was doing this, I'd add a simple helper class for UX for the built-in serializers: So users can do Serializers.doubleArray() etc, rather than having to look through classes. i.e., static methods help with code suggestions/completion.


Optimize: move experimental cmd to alpha stage

Describe what this PR does / why we need it
move below experimental cmd to alpha stage to incubate.

sealer prune
sealer gen
sealer debug
sealer exec
sealer cert
sealer merge
sealer upgrade

Does this pull request fix one issue?

fix #1522
Describe how you did it
Describe how to verify it
Special notes for reviews

cmd/sealer/cmd/alpha/merge.go:38: File is not gofmt-ed with -s (gofmt) @kakaZhou719

Codecov Report

Merging #1577 (0b6608a) into main (a9919aa) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1577   +/-   ##
=======================================
  Coverage   11.19%   11.19%
=======================================
  Files          88       88
  Lines        8067     8067
=======================================
  Hits          903      903
  Misses       7052     7052
  Partials      112      112

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 a9919aa...0b6608a. Read the comment docs.


Update migrate-az-3.0.0.md fix cmdlet names

Some cmdlet names are wrong -- they don't exist and don't match the examples.

Docs Build status updates of commit 7680ae8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azps-8.1.0/migrate-az-3.0.0.md
:white_check_mark:Succeeded
View (azps-8.1.0)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


(actions): bump go version for go e2e tests

Description of the change:
Make Go e2e tests run with Go version 1.18
Motivation for the change:
As part of the Kubebuilder  bump to support k8s 1.24, Go 1.18 is now used when scaffolding a project with the go/v3 plugin. We need to adjust our e2e tests/actions to accommodate this change in the scaffolding.
Checklist
If the pull request includes user-facing changes, extra documentation is required:

[x] Add a new changelog fragment in changelog/fragments (see changelog/fragments/00-template.yaml)
[ ] Add or update relevant sections of the docs website in website/content/en/docs

/hold

Closing as obsolete. Tests and testdata will be updated as a part of https://github.com/operator-framework/operator-sdk/pull/5843


[sival,pattgen] Port pattgen_ios test to master

This is a port of #23164 to master. The big difference between the ES and master is the inactive level feature was added in #23198 . I elected to cherry-pick the commits of #23164 (+some rebase fixes) and then implement the inactive feature on top of it to make the review easier.

Fixes #23339


disallow overbroad bans?

See #972. @mogad0n reported a channel operator who did /kb some_nick, resulting in a ban on<EMAIL_ADDRESS>where example.com is the server name --- i.e., a ban on all always-on clients. This should possibly be disallowed.

I've also thought about a BAN command in this regard: you do /ban add some_nick or /quote ban add some_nick and Oragono generates the appropriate ban (either *!*@host for anonymous or account!*@* for authed).

Maybe for always-on clients we could just set their hostname to the accountname or some variation of it, similar to how cloaks work on freenode/etc (e.g. unaffiliated/dan). We might be able to use alwayson/dan or something else? That'd probably let the bans work as intended without any changes from clients.
If we do create a BAN command I'd mostly be wondering how much it'd be used, since /kb and similar commands (even via the gui) already get provided by the client themselves and it seems like people are used to 'em.

#1312


Steps to Add library to a project

Please explain steps to add your library to a project(Swift).

Add cge.framework to your project.
#import <cge/cge.h>
Then you can use it.

Thanks


Setting SnackBar's autoHideDuration to null doesn't cancel timer

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
When a SnackBar component is set to have an autoHideDuration and that duration is set to null on a subsequent re-render, the timer still fires and the onClose handler is still called.
Expected Behavior 🤔
If a re-render sets the autoHideDuration to null and there is a timer in place, I believe that timer should be cancelled.
Steps to Reproduce 🕹
Here is the code I'm using:
    render() {
      return (
        <Snackbar
          message={this.state.message}
          open={this.state.isOpen}
          autoHideDuration={this.state.duration}
          onClose={this.hide.bind(this)}
        />
      )
    }

    /* Public methods */

    show(message, duration=null) {
      // Show the given notification message.  If 'duration' is specified, the
      // notifier will auto-hide after the given number of milliseconds.
      console.log({message, duration})
      this.setState({
        message: message,
        isOpen: true,
        duration: duration
      })
    }

    hide() {
      // Hide the notifier.
      this.setState({isOpen: false})
    }

As you can see, the component uses state variables to set the message, open state, and auto hide duration.  I control the component by calling the public methods show() and hide().
The above code generally works -- I can call show('...', 2000) to show the SnackBar for two seconds, and then have it auto-hide.  Similarly, I can call show('...') and it will show without auto-hiding, and I can then call hide() to programmatically hide it.
The problem is that if the user does something that triggers a show('...', 2000) and then, within two seconds, calls show('...'), the timer isn't reset, and so the SnackBar disappears even though in theory the timer has been cancelled
Context 🔦
I'm attempting to create a generic "notification" component that lets me show messages to the user as various actions take place.  It generally works, unless the user happens to perform the sequence of operations that cause show('...') to be called before the timer has expired.
Your Environment 🌎

Tech
Version

Material-UI
v4.8.0

React
16.12.0

Browser
Chrome 79.0.3945.88

Do you have a reproduction?

I have the exact same issue with Material-UI v4.9.2.


Are the setup instructions the same as auth0-ember-simple-auth ?

I tried this addon after failing to make things work with auth0-ember-simple-auth which seems as official as unmaintained and currently doesn't work with Ember2 (not using the latest version of simple-auth).
However I couldn't manage to get things to work with this one either and i was wondering if the setup was very different from the one described here for example.
My main issue is that I can get to the point of a callback actually being sent to my app with a url like http://localhost:4200/sign_in#access_token=<XXX>&id_token=<XXX> but nothing happens next, the session is not updated with the new identity

For me, the missing part was to register the Jwt authorizer as well as the Authenticator that this addon provides. So in a custom initializer:
import Authenticator from 'my-app/authenticators/auth0-lock';
import Jwt from 'my-app/authorizers/jwt';

export default {
  name:   'custom-lock',
  before: 'ember-simple-auth',
  initialize: function(registry) {
    registry.register('simple-auth-authenticator:lock', Authenticator);
    registry.register('simple-auth-authorizer:jwt', Jwt);
  }
};

This appears to be done automatically for you in the "official" plugin, but not in this one.

Hey, thanks for your reply,
On my side, I identified, that I just needed to change a few things from the doc I previsouly linked

ember-simple-auth need to be installed with ember install ember-simple-auth
there is no ember generate scaffold-auth0
imports must be changed from simple-auth/... to ember-simple-auth/...
authorizers are now looked up by ember simple auth as authorizer:<name> instead of simple-auth-authorizer:<name>

I isolated the problem to ember-simple-auth-auth0 never getting the callback from Lock when authentication is finished. This is due to the fact that a full refresh of the page happens and the context is therefore lost.
Maybe an initializer could be added which parses the hash of the URL with Lock.js in order to seek for an access_token and an id_token ?

Hey, thanks for the feedback. This addon should be feature parody and more than the official one. Sorry I have not updated the docs. We are currently using this in production at our company so I'll try to get the docs up to date sometime this week.

Great! Looking forward to seeing it working :)
In the meantime, I have also poked at auth0 on both Twitter and customer support about them not maintaining the "official" addon. Let's what this does...
One useful thing to have would be an example app that demonstrate how things are done. Bu

Great! Looking forward to seeing it working :)
In the meantime, I have also poked at auth0 on both Twitter and customer
support about them not maintaining the "official" addon. But I'll choose
whatever works first.
Le 24 oct. 2016 20:00, "Chris Watts"<EMAIL_ADDRESS>a écrit :

Hey, thanks for the feedback. This addon should be feature parody and more
than the official one. Sorry I have not updated the docs. We are currently
using this in production at our company so I'll try to get the docs up to
date sometime this week.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/seawatts/ember-simple-auth-auth0/issues/4#issuecomment-255816931,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOYMrGVNr3KIFBVE_B60dcHTREvu7y7ks5q3PIsgaJpZM4KdWFS
.

I'm running into the same issue where the authenticated event doesnt fire the first time and reloads the page with the

http://localhost:4200/sign_in#access_token=&id_token=

If I press the login button again, the authenticated event will fire and it will load the user and bring them into the app as authenticated.  It would be great to see a working example of this of this library.  And thanks for making this addon and bringing it up to speed!

@mrship this addon is setup a little differently. You don't actually have to inject the authenticator or authorizer.
You should just be able to use it by default with the way that ember-simple-auth tries to find the authenticators.
get(this, 'session').authenticate('authenticator:auth0-lock', lockOptions);

@Floby I am now including ember-simple-auth during the installation stage of this addon now.
@reedyrm I can't seem to replicate this. Can you create a new issue with repo steps. Thanks.
I have added more docs to the README and I'll release a new version soon. It will be a major version bump seeing as there are breaking changes.

@seawatts that's a plus =) great
@reedyrm I solved the problem by using an option I found in this file for a dummy app used for tests. The lock options used there are:
      {
        autoclose: true,
        auth: {
          redirect: false,
          params: {
            scope: 'openid'
          }
        }
      }

I think the redirect: false is what was missing in my initial attempts. Now lock opens a (mostly blank) popup window which is used for the redirect instead of the main one, locks is able to extract the token and close the popup pretty quickly.
@seawatts Maybe this option should be the default, since it doesn't seem to work otherwise? Or setup an initializer which could extract the token from the URL upon redirect before the app is initialized.

@Floby interesting, I think I'd rather go with including redirect: false by default instead of trying to extract it out of the url myself, even though that is what I have to do when impersonating.

I haven't tried impersonation yet, but I know I'll need it soon. Is there
some trick to it?
Le 3 nov. 2016 18:28, "Chris Watts"<EMAIL_ADDRESS>a écrit :

@Floby https://github.com/Floby interesting, I think I'd rather go with
including redirect: false by default instead of trying to extract it out
of the url myself, even though that is what I have to do when
impersonating.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/seawatts/ember-simple-auth-auth0/issues/4#issuecomment-258214353,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOYMoMlYTrniKqmBaa6x-SaIkn-3iBzks5q6hnHgaJpZM4KdWFS
.

@Floby I updated the readme with how to do impersonation


README example two simplifying

Simplified example two: removed the closure from updateDom and splitted it to two abstract functions so now theres two pure functions and an impure one.
Relates #1

@hasansaad1 also perhaps you can either turn off 'fix on save' or disable linting of markdown files to prevent prettier from auto-formatting the file 😄


Installation problem

 Problem 1
    - Installation request for saxulum/saxulum-doctrine-mongodb-provider 2.0.0 -> satisfiable by saxulum/saxulum-doctrine-mongodb-provider[2.0.0].
    - Conclusion: remove pimple/pimple v1.1.1
    - saxulum/saxulum-doctrine-mongodb-provider 2.0.0 requires pimple/pimple >=2.1,<4 -> satisfiable by pimple/pimple[v2.1.0, v2.1.1, v3.0.0].
    - Can only install one of: pimple/pimple[v2.1.0, v1.1.1].
    - Can only install one of: pimple/pimple[v2.1.1, v1.1.1].
    - Can only install one of: pimple/pimple[v3.0.0, v1.1.1].
    - Installation request for pimple/pimple == <IP_ADDRESS> -> satisfiable by pimple/pimple[v1.1.1].

Hi
Thanks for that, it did solve the problem for me.

@masitko your welcome


Publishing Services from Script fails on ArcGIS for Server

Service publishing fails on ArcGIS for Server (all versions) in all cases using the release candidate and the currently released versions of the scripts.
Issue does not occur on AGOL.
@allisonmuise Can you please provide more information on this issue?

My experience was the reverse of what is listed above. I had no issues as is with publishing to AGS. However, I would get an error when publishing to AGOL. I made a minor change to the AGOL side and they are both completing without error for me now. If you are still seeing an issue I will need some more details I guess.

Also I only looked at this on 10.3.1

@allisonmuise could I see an example .cfg that was used for testing the AGS side please?

This is now non-repro, so closing

Note: the changes made for this issue were for the AGOL part of publishing the script which wasn't the original  and issue. This change has not been included in the July release. But is a positive change and should be included in the next release per discussion with John and Allison.
(AGOL publishes fine without this issue as verified by LT and WP)


Remove: Google.Chrome.Canary version 124.0.6326.0

404 Not Found
Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

URL: https://dl.google.com/release2/chrome/jrk77q3dnb2rw5xs5sfqsk5zx4_124.0.6326.0/124.0.6326.0_chrome_installer.exe
Response status code does not indicate success: 404 (Not Found).
(Automated message - build 872)


Debug Java Application in IntelliJ leads to "Cannot load JVM TI agent twice" error

Bug Description
IntelliJ mirrord plugin version: 3.56.2
When trying to start a Java application in IntelliJ with the Mirrord IntelliJ plugin active, leads to the problem that the application fails to start with the following error message:
.../java -agentlib:jdwp=transport=dt_socket,address=<IP_ADDRESS>:46177,suspend=y,server=n -XX:TieredStopAtLevel=1 -Dspring.output.ansi.enabled=always -Dcom.sun.management.jmxremote -Dspring.jmx.enabled=true -Dspring.liveBeansView.mbeanDomain -Dspring.application.admin.enabled=true -Dmanagement.endpoints.jmx.exposure.include=* -javaagent:/home/.../.local/share/JetBrains/Toolbox/apps/intellij-idea-ultimate/plugins/java/lib/rt/debugger-agent.jar -Dfile.encoding=UTF-8 -classpath ... Error occurred during initialization of VM agent library failed to init: jdwp Picked up JAVA_TOOL_OPTIONS: -Xdebug -agentlib:jdwp=transport=dt_socket,address=<IP_ADDRESS>:5005,server=y,suspend=n ERROR: Cannot load this JVM TI agent twice, check your java command line for duplicate jdwp options.
Steps to Reproduce

Create a simple Java project in IntelliJ with Spring Initializr (Language: Java, Type: Gradle - Groovy)
Activate mirrord in IntelliJ
Start the Application.java in debug mode
-> Applications fails to start
Deactivate mirrord in IntelliJ: Applications starts in debug mode

Backtrace
No response
Relevant Logs
No response
Your operating system and version
Windows 11 WSL2 21H2
Local process
IntelliJ
Local process version
No response
Additional Info
No response

Hey! Thanks for reporting this. I suspect this happens due to duplicate env being fetched from the remote pod - you can probably remove the local env or exclude the remote one using

"env": {"exclude": ["JAVA_TOOL_OPTIONS"]}

Hello,
thanks for the fast response.
This fixed my problem.

 If you like mirrord we'd appreciate a review in the marketplace :)


Aging report is horrible to set-up !! It's unnecessarily confusing to set up parameters !

[Enter feedback here]

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

ID: 2905850a-1484-f4a7-b156-94fa0ddc3570
Version Independent ID: 6bb2f10c-f60b-a2c8-45ca-b4d9a7992bb6
Content: Set up and generate accounts receivable aging information - Finance | Dynamics 365
Content Source: articles/finance/accounts-receivable/tasks/set-up-accounts-receivable-aging-information.md
Service: dynamics-ax-applications
Product: ****
Technology: ****
GitHub Login: @mikefalkner
Microsoft Alias: mfalkner

@advent77, thank you for providing feedback.
@roschlom, please review and assign to the appropriate PM.

Bug 509455 has been logged to track this issue.

The Customer aging report has been updated to address this feedback.


Correctly handling sampler calls for low == high cases

Source: https://twitter.com/kutachu/status/1095218870250463232
Example: https://colab.research.google.com/drive/1yRvYCPT0nTc0tEDBljkoGupfOBabgTwZ#scrollTo=pm4BsrN33kph

I found that, when low == high, calling suggest_uniform raises an internal exception. I think it would be useful if Optuna can correctly handle such calls.

Thank you for substitute.
My indication in accordance with example.
I think, this problem in optuna.samplers.tpe.sampler._sample_uniform function.
It use float in 'low' and 'high', but after that optuna.samplers.tpe.sampler._sample_from_gmm function use 'if low >= high'.
Optuna should be rounding, or multiplied by coefficient like optuna.samplers.tpe.sampler._sample_int function.
I'm not experts, I'm sorry If I wrong.
Thank you.

Current version of Optuna apparently does not support the same value for low and high for trial.suggest_uniform(). I often want to do this to include the constant float value in the log.
As a workaround, it is possible to use trial.suggest_categorical like this:
float(trial.suggest_categorical('foo', ['0.123']))

Resolved: https://github.com/pfnet/optuna/pull/384


Codecov button

When I was trying to create a pull request, I noticed someone have already link the codecov! That is awesome! Thanks!
How about we add the codecov button on the README file. It should be on the website of the codecov related to our repository, just one line url but so cool!

Sounds good to me

But who did the codecov job? cuz the link should be on the account of the codecov and only the person have access can have the button link.


Dynamic tank and OpenComputers

I have spent the last half an hour trying to figure out how to get OC to read the fluid level on a dynamic tank. I am by now pretty sure, that it is not a general issue as other Mek things (like Induction Matrix or Laser Amplifier) show up just fine as components and read back values.
It seems like the dynamic tank is the only thing that is not responding.
Mek version is 9.2.1 on MC 1.10.2

Push

For whoever stumbles upon this:
I found a "solution".
You can place the adapter next to a tank valve and install a "Tank Controller Update". This allows you to read out the fluid levels. I didn't test it very thoroughly (yet), but in a short experiment it correctly detected a bucket of water in a small dynamic tank.


ibax wallet adress change bug

Describe the bug
i need change ibax wallet adress. i wrote wrong adress before. i can't change wallet adress. Can you help me?
Receive notification email
<EMAIL_ADDRESS>Modify
Address of the wallet that has been bound to receive IBXC
1800-1137-3846-3390-3961 i need change this wallet
my new wallet adress 0820-5816-7102-5938-6564
Reproduction
i need change ibax wallet adress. i wrote wrong adress before. Can you help me?
Receive notification email
<EMAIL_ADDRESS>Modify
Address of the wallet that has been bound to receive IBXC
1800-1137-3846-3390-3961 i need change this wallet
my new wallet adress 0820-5816-7102-5938-6564
System Info
i need change ibax wallet adress. i wrote wrong adress before. Can you help me?

Receive notification email

<EMAIL_ADDRESS>Modify

Address of the wallet that has been bound to receive IBXC

1800-1137-3846-3390-3961 i need change this wallet

my new wallet adress 0820-5816-7102-5938-6564

please help me

Logs
i need change ibax wallet adress. i wrote wrong adress before. Can you help me?

Receive notification email

<EMAIL_ADDRESS>Modify

Address of the wallet that has been bound to receive IBXC

1800-1137-3846-3390-3961 i need change this wallet

my new wallet adress 0820-5816-7102-5938-6564

please help me

Validations

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Hello, I have the same problem, did you solve it?

what's your problem
I can not understand
http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
Virüs
yok.www.avg.com
http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
muhammetgokce @.***>, 16 Mar 2023 Per, 14:06 tarihinde
şunu yazdı:

Hello, I have the same problem, did you solve it?
—
Reply to this email directly, view it on GitHub
https://github.com/IBAX-io/go-ibax/issues/2097#issuecomment-1471744588,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXKIOKGJIBXWTPPZ2DTNBF3W4LX4TANCNFSM6AAAAAATTDR2JY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Sama problem for me


added ockam:// scheme association in app for linux and mac

This pr is based on alpha11 upgrade branch.
Added support for both linux and mac for the ockam:// custom scheme.
Since tauri doesn't handle linux url passing I've recovered work from previous pr.
Both .desktop file and pList respectively for linux and mac, are now directly generated by tauri.
Note that to apply a patch, tauri has been overridden to https://github.com/build-trust/tauri.git
In order to start cargo tauri build the tauri-cli must be compiled and installed from the fork.
Merge this PR first

solves https://github.com/build-trust/ockam/issues/5579 and https://github.com/build-trust/ockam/issues/5580


Support to double click in thumb-slider

Support to double click in thumb-slider and rename event click for a name most semantical.
In my case I add touch-based padlock background:

Thank you!


doc: Add translation for Simplified Chinese(zh-CN)

Summary/ Motivation (TLDR;)
Translation for zh-CN.
Use the dropdown menu on the top to switch languages.

As I didn't find how to customize the navbar for each page, the navbar items have not to be translated.

Use pnpm instead of yarn

All addressed. Thanks! @jsun969

@RiverTwilight Thank you very much for your time invested in vitest-preview 🥰
@jsun969 Thanks for your help in reviewing this PR.

@all-contributors please add @RiverTwilight for translation

Hi @RiverTwilight @jsun969. I just found out that the merge commit hasn't been deployed to production. I just cleared cache and manually deploy the Chinese version of the docs. You can check it out at https://www.vitest-preview.com/zh/
Again. Thank you for your help 🎉.


Fix typo

python -> Python

Learn Build status updates of commit b5c5d5f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/machine-learning/azure-machine-learning-release-notes.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Looks good; thanks, @changeworld


New post on preference behaviour and the role of modules on inheritance.

PR Summary

PR Checklist

Do you want this post to be published on a specific date?: N (I have put it a folder for December 2021):
[X] I have read the contributors guide and followed the style and process guidelines
[X] PR has a meaningful title
[ ] Includes content related an open issue

see Closing issues using keywords.

[X] This PR is ready to merge and is not Work in Progress

 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.

@jhoneill Can you please sign the CLA?

 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.

@jhoneill Can you please sign the CLA?

I thought I had, but it was telling me I had not, so I have just signed [again], if you want to take another look.

@jhoneill Thanks for signing the CLA. You should only have to do this once per repository. Maybe you signed it for another? Either way, it is signed now. Thanks.


Sanity check primers

One issue we should check is whether the primers used in the amplification match the 5' ends of the sequence.  PI mentioned that some custom amplicons may not match up

So far a spot check seems fine for the majority of data, but there are a few sequences where the sequence is a clear mismatch (likely misamps).

Project-specific bug (low concentration sequence) and low-priority.


Allow user to change Device Info settings

Consider using:
https://www.npmjs.com/package/electron-settings

Another interesting option: https://github.com/tkambler/electron-preferences


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-master" workflow

Codecov Report
Patch and project coverage have no change.

Comparison is base (25904ad) 100.00% compared to head (b8050e5) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #189   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           52        52
  Branches        15        15
=========================================
  Hits            52        52

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


Is this library thread safe, not sure what to make of this error?

java.lang.IllegalStateException: java.lang.IllegalStateException: Connection [Not bound] is still allocated - code works fine when using .stream() instead of .parallelStream() - I've also tried doing this with virtual threads and I get a different error java.lang.IllegalStateException: Connection <IP_ADDRESS>:58156<-><IP_ADDRESS>:443 is still allocated - I've also tried the inbuilt executeAsync() and I get the same error as I do with virtual threads.
I'll share a code example -
 Map<String, Track> uriToTrack = extractAndPartitionUris(streamData).parallelStream()
                .map(spotifyRepository::getTrackDataByIds)
                .flatMap(List::stream)
                .collect(Collectors.toMap(Track::uri, Function.identity()));

spotifyRepository.getTrackDataByIds() looks like
 public List<Track> getTrackDataByIds(List<String> ids) {
        List<Track> tracks = new ArrayList<>();
        try {
            LOG.info("Requesting track data for ids: {}", ids);
            se.michaelthelin.spotify.model_objects.specification.Track[] response =
                    spotifyApi.getSeveralTracks(ids.toArray(new String[]{})).build().execute();
            Arrays.stream(response).map(spotifyResponseMapper::mapTrack).forEach(tracks::add);
        } catch (IOException | SpotifyWebApiException | ParseException e) {
            LOG.error("Error retrieving track data: {}", ids, e);
        }
        return tracks;
    }

nevermind - looks like this was fixed in 9.0.0-RC1 - I was using an older version.


Cloudformation issue

I want to deploy to version of MIE in the same region. But I can't, because of this error
StageExecutionDLQ already exists in stack arn:aws:cloudformation:us-east-1:862440218923:stack/mediainsightengine202003/4e55b620-5f94-11ea-8492-0e1cfc6fd327

Currently you can only deploy one MIE stack per region. If you need to deploy more than one, checkout this pull request:
https://github.com/awslabs/aws-media-insights-engine/pull/131
I haven't had a chance to test that PR yet, so it would be useful to hear how it works for you.

Duplicate of https://github.com/awslabs/aws-media-insights-engine/issues/61


Offer a single point of entry for browsing Parser/Writer settings

(Migrated from https://openrdf.atlassian.net/browse/SES-2023)
It is not easy to browse through the various settings available for different parsers and writers right now. There needs to be a single point of entry where users can browse javadoc for the various settings available for different parsers.
This is in addition to the RDFParser/RDFWriter.getSupportedSettings methods that are available at runtime.

Added to documentation, see https://rdf4j.eclipse.org/documentation/programming/rio/#configuring-the-parser-writer


Server Store API and TCK

Thinking a bit about the data structure and the kind of operations we will want to do on it, I am wondering if we do not want something like a peek next on the Element at client side. It would look for the next Element having an equal key.
One example:

I reach an expired value in the chain
It is effectively expired unless the next element for this key is an expiry duration update, coming from a flush.


avg or count in junction table - odd behaviour

I'm not exactly sure if it's perhaps something wrong in my understanding of how things work or is there maybe a bug.
So here's the situation: I have many-to-many relationship, with some values in junction table. Models work ok, I can insert and get the data perfectly, everything works fine.
However, when I try to do a "count" or "avg" operation upon the values in junction table, I get an odd behaviour. I've described what I wanted, what SQL it generated and what I actually got in the example below:
/*
  this returns grades as an empty array
*/
new this( { id: id } )
	.fetch({
		withRelated: ['participants', {
			'grades': function(qb){
				//qb.select('affair_id');
				qb.avg('severity as average').count('severity as votes');
			}
		}]
	});
//creates this, which works when I execute it in DB:
//bindings: [ 1 ], sql: 'select avg("severity") as "average", count("severity") as "votes" from " grades" where "grades"."affair_id" in (?)' }

/*
  this returns grades correctly!!
*/
new this( { id: id } )
	.fetch({
		withRelated: ['participants', {
			'grades': function(qb){
				qb.select('affair_id');
				qb.avg('severity as average').count('severity as votes');
			}
		}]
	});
//creates this, which of course also works in DB:
//bindings: [ 1 ], sql: 'select "affair_id", avg("severity") as "average", count("severity") as " votes" from "grades" where "grades"."affair_id" in (?)' }

Any kind of feedback is much appreciated!

You can't do arbitrary queries right now with relations. You have to get that data separately and assemble your response. Relations assume that the return value should be as the appropriate model/collection.

thanks for your response. I'm just curious, how come it works in the example below? If I select another attribute besides avg and count, I get the desired result?

Might be because you include the foreign key so the relation can be associated.

You are correct; if I select anything else than the reference ID of the other table, it does not work. So, are such queries anomalies, considering that you said that 'You can't do arbitrary queries right now with relations. '?

As long as you have the required relational columns you can add whatever you'd like.

thanks for your time and answers!

No problem! The oddities here generally come back to the recurring issue of joins vs. multiple queries for relations. You can only use joins if you explicitly alias all your columns during queries since you're almost guaranteed to have conflicts (e.g. every table has an id column). In order to alias all your columns you need to list out every column in the select, (no *). And in order to do that, Bookshelf would need a way for you to tell it about every column or go looking for that info itself. We're planning to support both paths, with the latter preferred and the former available as a fallback.


线上raft切换时间过长

线上几千万级别的文件数目，并且持续有元数据访问，master切换时间比较长有时候达分钟级别，看了下代码发现如下地方

有一个默认20s的等待，请问下这个的用处是？去掉的话会有什么影响吗

@jenoudet @ggezer @yuzhu

@jenoudet I think @jayzhenghan is curious why there is a delay in this process.  so feel free to explain here. because they are noticing long raft failovers.

This waiting timeout is to ensure a smoother leader transition. By waiting this amount, we ensure that every member of the quorum has taken into account who the new leader is, and we can start processing new entries or catching up older ones from others in the quorum with this certainty in mind.

In practical, this wait is not necessary, if there are network split, wait this longer may not help anyway.
We have a private patch for this, as long as the following condition satisfied, leader can serve the requests.
@@ -706,9 +706,15 @@ public class RaftJournalSystem extends AbstractJournalSystem {

       // Wait election timeout so that this master and other masters have time to realize they
       // are not leader.
-      CommonUtils.sleepMs(mConf.getMaxElectionTimeoutMs());
-      if (stateMachine.getLastAppliedSequenceNumber() != lastAppliedSN
-          || stateMachine.getLastPrimaryStartSequenceNumber() != gainPrimacySN) {
+      try {
+        CommonUtils.waitFor("check primacySN " + gainPrimacySN + " and lastAppliedSN "
+            + lastAppliedSN + " to be applied to leader", () ->
+            stateMachine.getLastAppliedSequenceNumber() == lastAppliedSN
+                && stateMachine.getLastPrimaryStartSequenceNumber() == gainPrimacySN,
+            WaitForOptions.defaults()
+                .setInterval(Constants.SECOND_MS)
+                .setTimeoutMs((int) mConf.getMaxElectionTimeoutMs()));
+      } catch (TimeoutException e) {
         // Someone has committed a journal entry since we started trying to catch up.
         // Restart the catchup process.
         continue;

@kevincai I agree with you，thanks～

I have also paid attention to this code before, and I'd like to explain my point of view. Imagine that there are three raft nodes A, B, C. At beginning A is the leader node, then A and the other two nodes are disconnected from each other because of network split, then B becomes the new leader, but at this time A does't realize that it is not the leader any more, and A and B RPC port will be opened and accept the service at the same time. At this time, A will still accept requests from the client, and if it encounters a modification request, it will first successfully modify the UFS but fail to write the raft log, thus causing inconsistencies between the metadata in aluxio and ufs. According to my understanding of ratis, Ratis implements leader step down, A will become a follower node because it does not successfully heartbeat most of the nodes in its cluster in an election, thus close it's rpc server. So in my opinion, waiting for an election timeout seems to be necessary.
For how to reducing the raft switching leader time, in this version you are using there should be a long time consumed in starting the WEB server, I suggest to merge this PR firstly, in my previous tests I found that I can reduce the switching time by about 15s.
Then I suggested to reduce the election timeout of raft according to the deployment conditions of masters, and we referred to the advice given by etcd when setting the parameters.

@yuzhu @jenoudet
@kevincai 's code suggestions looks good to me, how about making that change?
Or we can have another propertykey to provide optional waiting time when leader transfers

@qian0817
I agree with your point on causing UFS and raft log inconsistency. But seems the situation you described could happen in any stage during the system running, why particularly needs a time wait during the new leader start? The sleep during startup can prevent the new leader from losing its primacy too soon, but can't prevent a long running leader suddenly lost its primacy, and can still causes data inconsistency.
Alluxio can follow etcd's timeout principal, but can't follow its timeout value, JVM STW FGC will be a big trouble when elect timeout is set to < 5s.

@yuzhu @jenoudet @kevincai 's code suggestions looks good to me, how about making that change? Or we can have another propertykey to provide optional waiting time when leader transfers

I prefer the patch to be private for now, until the impact to data integrity is fully discussed get to a conclusion. In our case, we use UFS mostly as READ-ONLY source, so underlying UFS modification is not a concern, the risk is acceptable and taken.

To be honest, I think the current implementation should have inconsistent issue between UFS and alluxio metadata when the leader is down, but I haven't tried to reproduce it yet.
@jenoudet Can you explain in more detail the reasons for the wait and whether it is because there are issues that I have not considered.

@kevincai @qian0817 we had an internal discussion and think that @kevincai 's private patch should  at least be no worse than the current delay. As for the inconsistent issue, @qian0817 i don't see how a longer delay would help in the case of network partition? could you explain more?  If no objections, we will welcome a patch from @kevincai or we can make a PR similar to kevin's suggestion, whichever you prefer.

@yuzhu
If network partitioning happens, then the previous leader node will definitely become a follower node after one election timeout, and will not cause two nodes to regard itself as a leader node at the same time.
At the same time, I do not oppose this modification, because the current implementation when the leader is down is also inconsistent situation.

106 is leader，Why would 106 vote for 105?, Make 105 successful

To answer your first question: 105 would start a new election if it does not receive a heartbeat from the leader for a period of time. When a member starts a new election, it votes for itself and requests votes from all the other members of the quorum. The other members accept this request vote by default (there are some rejection conditions, such as if another member is itself running another election).

To answer your first question: 105 would start a new election if it does not receive a heartbeat from the leader for a period of time. When a member starts a new election, it votes for itself and requests votes from all the other members of the quorum. The other members accept this request vote by default (there are some rejection conditions, such as if another member is itself running another election).

What is the reason for this switch? I found out that in the ping-pong selection

To answer your first question: 105 would start a new election if it does not receive a heartbeat from the leader for a period of time. When a member starts a new election, it votes for itself and requests votes from all the other members of the quorum. The other members accept this request vote by default (there are some rejection conditions, such as if another member is itself running another election).

Error sending snapshot to leader
@jenoudet

This wait was most probably added because we were using copycat, where leadership stability was not managed as comprehensively as in ratis. So I can't provide any counter argument to make the wait before checking the state.
I think we can go ahead with the fix provided; https://github.com/Alluxio/alluxio/issues/14563#issuecomment-977701877

PR #14612 implements these changes @ggezer.


build(deps): get rid of jsc-android

This PR resolves PHIRE-208
Description
Gets rid of jsc-android since we use hermes and this is already bundled with react-native nowadays if we decide to go back to it (that I don't think that will happen)

PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

remove jsc-android engine - gkartalis

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Android user-facing changes (remove jsc-android engine - gkartalis)

  Generated by :no_entry_sign: dangerJS against 57f3415ed6789bf68df86e2c920d1d23c40ae384


Thomas Cardin introduction

Closes #66

Note that we actually want to change the process to add our profiles to the website ( repo ai-cfia-ia-acia.github.io) instead of the dev-rel-docs. Can you both update the process and move your and other team profiles there?

Good, thanks for the review. I have created this pull request. Let me know if you want me to remove the team introduction, including mine, from this repository!

Good, thanks for the review. I have created this pull request. Let me know if you want me to remove the team introduction, including mine, from this repository.

Yes I do want as we're moving it to the site.

@ThomasCardin change this PR to clean out the introductions you've moved to the site.


Add filtering for browse page derived topics

While migrating Mainstream Browse pages to topics we need to prevent
the migrated clones from being visible to the user. Here we're adding
filtering to prevent items being show if the relevant field doesn't
exist or is true.
https://trello.com/c/m13ffUVc/292-update-publishing-apps-that-know-of-specialist-topics-to-hide-new-synced-ones
⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠

I'm assuming this isn't needed anymore. Can always be reopened if so.