in new version 1.10.2 , culture picker does not work in front end!

in new version 1.10.2 , culture picker does not work in front end!
I would like to add Multilanguage.
I follow this tutorial https://youtu.be/t6Tt8vjlOVw
In version 1.10.2 Culture Picker (Sergey Ermakovich version) seem not available,
is replaced with one already in 1.10.2 version,
this show a language picker, in admin page, but is not available to add in Header Widget (like video)

It's a gallery module, we can't fix it. But I think there is something out of  the box in Orchard for that as someone asked how to hide it not later than this week. @MatteoPiovanelli-Laser do you know?

I think that out of the box you only get to have links to the other localizations of the ContentItem being displayed (if any). That is what they were asking to hide recently on gitter. It's the default "Detail" display type for LocalizationPart.

Just to add +1 to this issue.
I am setting up a localized site for the first time but there is no means of allowing a user to set their locale for the site. I can translate content items which shows the language options for just that content item however it doesn't set the locale rather just switches to the alternate content item.
If I have a layer rule set up to show me French culturecode("fr-FR") the only way I can get the front-end of the website to actually display this layer is to set the default culture for the entire site to French. There is no module or widget that I can place on the front-end to allow the user to set the culture.
Unless I'm missing something?

As @mtugnoli mentioned, there used to be the CulturePicker module off the gallery to do just that. We use a (heavily) customized version of it.
Note that there is ICultureSelector: roughly speaking, implementations of this interface are used to determine the WorkContext.CurrentCulture.
If I remember correctly, once you navigate to a ContentItem whose culture is "fr-FR", CurrentCulture should be set to "fr-FR" by the default ICultureSelector implementations. At that point, in theory, you should see the correct layers.
The main thing the CulturePicker did was provide a Controller with a ChangeCulture/{culture} Action that would look for localizations of the current content in the given culture, and redirect to that if found. A widget would add for each existing culture a link to the same action, with a different parameter. The ICultureSelectors should be taking care of pretty much everything after that.
You can see all that in action on http://www.anfiteatromorenicoivrea.it/

On ther other hand, you can use an alternate for the DIsplay view of LocalizationPart. I think that shows llinks to the localized versions. As long as those are all in place, it will work like a less automated version of CulturePicker.
Depending on how you are organizing contents, you could play with the alternates there to get the links sort of working also for Contents that are not translated (e.g. by linking to their section).

Thanks for the response @MatteoPiovanelli-Laser

If I remember correctly, once you navigate to a ContentItem whose culture is "fr-FR", CurrentCulture should be set to "fr-FR" by the default ICultureSelector implementations. At that point, in theory, you should see the correct layers.

I think this is the problem area, in my example (1.10.2) clicking the fr-FR translation shows me the content item translation but does not reveal the layer with the rule culturecode("fr-FR"). The only way I can get that to show is by setting the site default culture to fr-FR in the admin which is obviously not the behaviour I'm expecting.
To be clear, the ideal is a solution like in your example where the user can just set their culture and we essentially show them different menus based on the layer rule which all link to the translated versions of the content. I looked at the CulturePicker module off the gallery but seemingly that's no longer supported. I was hoping for an admin UI driven solution but will take a deeper look.

@DannyT try this for a layer rule:
Lang('fr-FR')

@MatteoPiovanelli-Laser interestingly, this defaults it back to culturecode("fr-FR"). Weirdly, however, it now seems to be working as I would expect...
Would a website restart be required after adding a new culture? Alternatively I wonder if caching was making me think this wasn't working...
Seems to be okay now though so thanks for your help.

I would blame caching. I never had to restart a website after adding cultures.
You are welcome.


Option to use a spinner without the white circle

I'd like to use this, but I don't really want that white circle for my gallery. It would be cool if a normal CircleProgress component could be used.

I also looked at alternatives beside that. Didn't liked that white background much either. But without it CircleProgress fades into the background to much.

I also Increased the size to fill out the space. Also not very satisfying.

Maybe we can achieve something with linear progress at the top or bottom.

I want to have a light-gray background (that's why I opened #2) and have a loader like this on it. :smile:


Move to new client version which uses Doubles instead of Floats, fix Monitor tests

What does this PR do?
Moves to new version of the client which moved all Float fields to Double fields. Fixes rounding errors for high Monitor threshold values.
Additionally, the tests for monitor handler didn't really work, so I fixed them to actually pass.
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?

Thanks for the review, merging.


RSE Page: Loading error

Description
On the integration cluster, the RSE page crashes.
The client side errors are:

This is very likely due to the page not being correctly hooked up with the NextJS endpoints
Steps to reproduce

Go to List RSE page
Search for *
Click on any RSE on the list
Open the browser console to see the errors

Rucio WebUI Version
35.0.0
Additional Information
No response

Re-opening. The queries should be moved into the component as discussed in #438


Bad animation export (a channel is written twice in gltf file)

Exporting following file,
light_anim.zip
glTF file is not valid, a channel is written twice (rotation on node 0

Both rotation_euler and rotation_quaternion are animated in the file.
This is now a good idea, in Blender side.
Maybe we should add a check (and a warning) to avoid duplicate rotation animation export

@julienduroure just my 2 cent on this: wouldn't it be better to just skip/drop/don't export rotations channels that are not the current object's rotation mode here ?
Both cube and spot are in 'QUATERNION' mode so they are linked to the quaternion channels, the euler one being just the result of a misclick on "LocRot" when generating the animation and are just not attached rotations ?

@AurL You're right, we should keep only the current object rotation mode, and skipping other(s) one(s), Instead of keeping the first one retrieve and ignore other(s) one(s).
Great idea, thanks :)
PS :
But it will not work with delta transforms (I really don't know if this feature is often used)

I don't have a lot of experience with delta transforms but imo it's something that could trigger a baking operation, same for everything that is added on top of raw/base T/R/S keyframe animation like limits/offsets and IKs.
(Our usecase might be different but for our Blender processings in Sketchfab, we usually bake everything to be sure to get the exact visual result for the animations)


urgent Site is broken

Describe the bug
website is looking completely broken and unformatted
To Reproduce
Open website in chrome or Firefox

Go to the website

Expected behavior
Clear and formatted website
Screenshots

Yeah, same for me
The site appears completely broken

Issue is with the CSS file.

@zigmint Yes it seems main CSS is 404

A long term solution could be to put the CSS and JS files over to a CDN, seeing the number of visitors you have, you might wanna implement that. @JunaidBabu  @jeremyphilemon

A long term solution could be to put the CSS and JS files over to a CDN, seeing the number of visitors you have, you might wanna implement that. @JunaidBabu  @jeremyphilemon

Working on a fix :)

It's working now, for me at-least. 👍

Yes now it's working, I too created another bug request for this, wondering how to close it


New CLI options for facebook bot to automatically get localtunnel.me URL

In addition to CLI-related changes, I fixed a typo in storage_test.js

Such a good PR! Good job @rocketsummer

You're welcome!

using botkit 0.2.2
when I use the cli options  like --lt --ltsubdomain mydomain
I'll see this:
Serving webhook endpoints for Messenger Platform at: http://MY_HOST:5000/facebook/receive
so I'm not sure if a localtunnel.me tunnel is actually setup.
i assume it would be mydomain.localtunnel.me just as if i was using the standalone lt command.


Next.js v15-canary: access to cookies() should be asynchronously

The following code:
https://github.com/moshest/next-client-cookies/blob/68439a612ab339307ff833cadd2ac8ff6c730ed5/src/server.tsx#L9
Triggers a warning on next@canary:
In route / a cookie property was accessed directly with `cookies().getAll()`. `cookies()` now returns a Promise and the return value should be awaited before accessing properties of the underlying cookies instance. In this version of Next.js direct access to `cookies().getAll()` is still supported to facilitate migration but in a future version you will be required to await the result. If this `cookies()` use is inside an async function await the return value before accessing attempting iteration. If this use is inside a synchronous function then convert the function to async or await the call from outside this function and pass the result in.

Rational

The fix should be simple, since we are in a server component, mark the function with async and call the function with await

Released under 2.0.0-alpha.0.
Can you check?

No warnings now, useCookies() works perfectly.
Although I never use it in my app, I did test getCookies() as well (what's the purpose, we can just use cookies()?). It now properly returns a promise, but Typescript doesn't detect it as such:
'await' has no effect on the type of this expression

Thank you. You're right. I've update the types on version 2.0.0-alpha.1.
The getCookies() is just a wrapper, but it provide the same interface as you have on the client side so it's useful if you need to share cookies logic.

This warning is gone now, works perfectly

Great!
I will keep this issue open until Next.js canary is officially released and we can release it as well.

Thanks for the fix, Next 15 is now stable

Available now under v2.0.0


[NVIDIA XLA] Add new pattern for horizontal fusing instructions whose output tensor has same output shape into kLoop fusion kernel.

This PR add another horizontal fusion choice which fuses operands with the same output shape into kLoop fused computation, which should be faster in most cases than current kInput fusion pattern.
The below Case1 is the newly added pattern, because the kernel generated from kLoop instruction is much more efficient.
Detailed pattern are like:
// The following illustrates the mechanism of the horizontal fusion. Before
// fusion, there are two trivial kernels in the illustrating example. One has
// only a Mul op, while the other consists of only an Add op.  Can you run the perf benchmark with it?

@sergeykozub Could you take a look for benchmarks?

@Sergey Kozub @.***> do you think you could take a look?
On Fri, Dec 2, 2022 at 3:09 PM Frédéric Bastien @.***>
wrote:

@cheshire https://github.com/cheshire This is the update to h-fusion
that I discussed. Can you run the perf benchmark with it?
—
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/58763#issuecomment-1335281851,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACVGH2WEZQYRUM6VJDHO2LWLH7JNANCNFSM6AAAAAASRUMRSM
.
You are receiving this because you were mentioned.Message ID:
@.***>

@shawnwang18 Can you please check @sergeykozub's comments and keep us posted ? Thank you!

@sergeykozub I submitted the fix based on your comments, could you help proceed the PR?

Hi @sergeykozub,

There is no test coverage for the added code path, could you add a test verifying the basic functionality?

Can you run Google internal performance tests first? We do not have a wide internal performance test yet and we have already been hit by late regression detected in other models than the one we work on. They requested big code change or even made us drop the change completely. So, it is more time efficient for all of us to have them detected earlier than at the last step.
I'm asking this as this change can have a wide performance impact that we can't verify. If the impact wouldn't be wide, I wouldn't ask for this.
If the performance tests go well, we will add tests.

Can you run Google internal performance tests first?

I've run the internal benchmarks, and there are no regressions (and no improvements though, probably the new code path is not triggered in the existing benchmarks).

I've run the internal benchmarks, and there are no regressions (and no improvements though, probably the new code path is not triggered in the existing benchmarks).

Ok. Not what we expected. Thanks for running it.

@sergeykozub Thanks for your careful review, I submit the fix, and also adding the unittest for the new pattern,  please check.

Hi @sergeykozub Can you please review this PR ? Thank you!

@sergeykozub can we merge this PR now?

Sorry for the delays in merging. This is failing internal tests and we are still trying to figure out why. We hope to have it merged soon.

By modifying the internal test, I created a relatively simple HLO that this PR segfaults on:
HloModule f

ENTRY f {
  param.1 = ((f32[], f32[]), (f32[], f32[])) parameter(0)
  get-tuple-element.2 = (f32[], f32[]) get-tuple-element(param.1), index=0
  get-tuple-element.3 = f32[] get-tuple-element(get-tuple-element.2), index=0
  constant.4 = f32[] constant(1)
  add.5 = f32[] add(get-tuple-element.3, constant.4)
  get-tuple-element.6 = f32[] get-tuple-element(get-tuple-element.2), index=1
  constant.7 = f32[] constant(2)
  add.8 = f32[] add(get-tuple-element.6, constant.7)
  tuple.9 = (f32[], f32[]) tuple(add.5, add.8)
  get-tuple-element.10 = (f32[], f32[]) get-tuple-element(param.1), index=1
  get-tuple-element.11 = f32[] get-tuple-element(get-tuple-element.10), index=0
  constant.12 = f32[] constant(3)
  add.13 = f32[] add(get-tuple-element.11, constant.12)
  get-tuple-element.14 = f32[] get-tuple-element(get-tuple-element.10), index=1
  constant.15 = f32[] constant(4)
  add.16 = f32[] add(get-tuple-element.14, constant.15)
  tuple.17 = (f32[], f32[]) tuple(add.13, add.16)
  ROOT tuple.18 = ((f32[], f32[]), (f32[], f32[])) tuple(tuple.9, tuple.17)
}

You can run using the //tensorflow/compiler/xla/tools:replay_computation_gpu, but unfortunately it is currently broken. It will be fixed soon, but for now you can manually patch in this change to run it: https://github.com/reedwm/tensorflow/commit/b110bdd2e4b3bbec61a806623a2b0c9f1c000f7e. Then run:
bazel build --config=cuda //tensorflow/compiler/xla/tools:replay_computation_gpu
bazel-bin/tensorflow/compiler/xla/tools/replay_computation_gpu --use_fake_data ~/module.hlo

The internal failing test is actually //tensorflow/compiler/xla/tests:local_client_execute_test_gpu, in particular, the LocalClientExecuteTest.LargeNestedTuple test. But for some reason, I cannot reproduce the failure in open source TensorFlow by running that test. The HLO above is a simplified version of the HLO run in local_client_execute_test_gpu. I can reproduce the failure when running the above HLO directly with replay_computation_gpu.
The issue seems to be that the horizontal_loop_fusion pass is introducing a dead instruction (an instruction that no other instruction has as an operand). The TuplePointsToAnalysis cannot handle dead instructions and segfaults. Normally the HloDCE pass removes dead instructions but horizontal_loop_fusion is run after HloDCE
@shawnwang18, can you fix this issue? Also line 305 in horizontal_loop_fusion.cc ends in whitespace, so can you fix that as well?

Hi @shawnwang18 Can you please check @reedwm's comments and keep us posted ? Thank you!


Path with Maximum Gold

Problem Link
https://leetcode.com/problems/path-with-maximum-gold/
Coding platform
Leetcode
Programming language

[ ] C
[x] C++
[ ] Java
[ ] Python

Additional context (OPTIONAL)
Add any other context or screenshots about the feature request here.

Please review this problem and assign me this in C++

I want to contribute in java language, please assign to me

@weirdrag08 @ItsmeSauravSonu Working on this ?

Yes

@weirdrag08 @ItsmeSauravSonu Working on this ?

Yeah, I am working on it. Please give me some time :-)

@ItsmeSauravSonu @weirdrag08 ??

@ItsmeSauravSonu @weirdrag08 ??

Just give me 2 more days, I will solve it for sure :-)

@ItsmeSauravSonu @weirdrag08 ??

Please give me one more day, I will solve it for sure :-)

I will do today, please give me some time
On Mon, Mar 8, 2021, 2:12 AM Aditya Anand<EMAIL_ADDRESS>wrote:

@ItsmeSauravSonu https://github.com/ItsmeSauravSonu @weirdrag08
https://github.com/weirdrag08 ??
Please give me one more day, I will solve it for sure :-)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Nivedita967/Data-Structure-and-Algorithms/issues/309#issuecomment-792349091,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANZ5XWCVWXDSO2KBTJLGOGDTCPQMVANCNFSM4X2WLCHA
.

Hi @Nivedita967
I would like to solve this problem in Python. Could you please assign it to me?

Sure @abhinav5481 Go for it.


Add author attribution for Unsplash when a media element is focused or hovered

As per Unsplash requirements. See PRD.

Looks like this was already merged, so checked in Main Branch
Uploaded a few Unsplash image. Not seeing author on hover.
https://drive.google.com/file/d/1_6kPGENevgZLDlnZA1tki0OxWEMVR04Y/view
The 4 images in the video were selected from Unsplash tab

Heya, I a don't think that's what's need to be tested here. Actually, I don't think this is for QA at all as the whole Unsplash integration is behind a feature flag.
@winterDroid Please confirm that Unsplash integration is still behind a feature flag. As a reminder, Feature-flagged items are not going through QA until the flag is enabled.

@swissspidy yes it is still behind a feature flag. This feature is very implicitly behind one as otherwise no attributions will be present. That's why I moved it to QA to ensure that no existing media element logic is broken.

Testing instructions would have been very helpful in that case


"View" and "Edit" links in documentation don't work

Expected Behavior
On a page such as https://gf180mcu-pdk.readthedocs.io/en/latest/IPs/SRAM/gf180mcu_fd_ip_sram/cells/gf180mcu_fd_ip_sram__sram512x8m8wm1/gf180mcu_fd_ip_sram__sram512x8m8wm1.html there is a box in the lower-right corner that says  v: latest. Clicking on this produces a window with options including View and Edit.
Presumably these links should allow one to view or edit the documentation, however they do not.

Actual Behavior
Clicking on the link goes to a nonexistent page. For example: https://github.com/google/gf180mcu-pdk/blob/main/docs/IPs/SRAM/gf180mcu_fd_ip_sram/cells/gf180mcu_fd_ip_sram__sram512x8m8wm1/gf180mcu_fd_ip_sram__sram512x8m8wm1.rst
Steps to Reproduce the Problem

Go to a cell documentation page such as https://gf180mcu-pdk.readthedocs.io/en/latest/IPs/SRAM/gf180mcu_fd_ip_sram/cells/gf180mcu_fd_ip_sram__sram512x8m8wm1/gf180mcu_fd_ip_sram__sram512x8m8wm1.html
Click View or Edit

Specifications

Version: v:latest
Platform: Readthedocs

PTAL - @kgugala / @umarcor


[SPARK-8425][Core][WIP] Add blacklist mechanism in task scheduling

This is a proposal to add blacklist mechanism in Spark for better scheduling task to avoid running on the bad enough executors, this blacklist mechanism is based on the finished status of task with heuristic algorithm to track the bad enough executors to add to blacklist, the idea is brought from MR. The reason and detailed design doc please refer to SPARK-8424.

Merged build triggered.

Merged build started.

Test build #35093 has started for   PR 6870 at commit b8066f3.

Test build #35093 has finished for   PR 6870 at commit b8066f3.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
 final class ExecutorFailureStatus

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #35094 has started for   PR 6870 at commit fb3821f.

Test build #35094 has finished for   PR 6870 at commit fb3821f.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
 final class ExecutorFailureStatus

Merged build finished. Test PASSed.

I believe @davies implemented something similar before

My PR (#3541) is a little different,

Thanks @andrewor14 and @davies , I will take a look at #3541 , from my cursory look, I think both the target and implementation is a little different.

I think the original blacklist mechanism and what @davies did is to avoid the resubmitted tasks running on the same executors/hosts, which means tasks should be failed beforehand, then the blacklist mechanism could take effect. But about the first submitted tasks, current implementation cannot avoid scheduling tasks on such "bad" executors. This is what this PR addresses about. The basic idea and algorithm is brought from MR, I think for Spark it also requires such feature.

Hi @jerryshao thanks for working on this.  I did have one larger design question, about what to do when there are too many black listed executors ... though I'm not sure I have a better suggestion at the moment.
I also think you need to add a test to TaskSetManagerSuite.
But overall this is looking promising.

Hi @squito , thanks a lot for your comments, greatly appreciate it. I think the total blacklist executor number is controlled by spark.scheduler.blacklist.maxBlacklistFraction, which is 0.5 by default, means the number of blacklisted executors should not be more than half of the whole cluster.

I think the original blacklist mechanism and what @davies did is to avoid the resubmitted tasks running on the same executors/hosts, which means tasks should be failed beforehand, then the blacklist mechanism could take effect. But about the first submitted tasks, current implementation cannot avoid scheduling tasks on such "bad" executors. This is what this PR addresses about. The basic idea and algorithm is brought from MR, I think for Spark it also requires such feature.

@jerryshao I see, thanks for the explanation. Are you still actively working on this? (Is it still "WIP") If so could you rebase to master?

Thanks @andrewor14 , one of my colleague will continue to work on this :).

@andrewor14 I am working on it now, here is the latest design doc based on Saisai's previous work.
https://docs.google.com/document/d/1EqdocdbOH0eZ0Vp1RAHsE-8gKv9yPez1Xt8W5xgXn3I/edit?usp=sharing
And I have synced with latest master and will implement on it.

Is this superseded by #8760 then? Can we close this?

Yeah, sure.


Add predict example for C++

This is a simple predictor which shows how to use c api for image classfication
It uses opencv for image reading

Thanks!, I was working on the same this morning, here is the alternative code just in case that you find it useful: https://github.com/pertusa/mxnet_predict_cc. My results are very similar to yours (although I wasn't cropping the image), but after some tests I've found that there is a problem on both implementations. The output is not even similar to that obtained with the corresponding python "mxnet_predict_example.py" for some images, like the one attached. Could you please check it?

Thanks! This helps a lot!
A minor suggestion: could you add your build command at the top?

@pertusa I looked at your code. It pretty good, especially the make file. After we merge this you are welcome to make improvements to it. Thanks for you interest!

I have add a Makefile and README file for help, and move every thing to an independent folder.
Thanks to @pertusa & @piiswrong

Thanks @piiswrong , I'll contribute once merged. I've tested the @skylook code, and I get a similar output than with my version, but still very different from the python results. Classifying the vulture image (from my last message) with the ImageNet1K model, I get only a clear peak at the class 904 (window). However, the python script returns a kite as first option, and then some birds. I'm not sure if it a problem of the Makefile (some missing library?), but I've also tried with static linking (updated Makefile in my repo) and this issue remains. Could you try to classify this image and check that the output is consistent?

Yes, I met the same problem as @piiswrong , the predict result of c_api seems not very stable.
Even most of the time it shows the same result with the same image, but sometimes it shows different.
I do not know if it is a bug or just my miss unstanding with the c api.

Usually non-deterministic results (different outputs from the same input) are due to linkage issues, typically missing libraries. For instance, when using MKL in C++ it is necessary to add many extra flags and libs. So maybe the Makefile is wrong. I can't check it this week, but I'll try to do it ASAP.

When I used Theano, some layers with random noise, such as the dropout layer, will make the result non-deterministic. So make sure it is not the reason of your problem.

Good point @zhangchen-qinyinghua, I'll also check this

Thanks for the good job, can you please make a pass on code style to make it consistent with rest of the project ?  i.e. Google's C style.
Check the  lint job in main Makefile, which can help auto checks the code style
Thanks!

I've found the problem of the inconsistent output. OpenCV data array is stored as BGR,BGR,BGR, whereas MXPredSetInput expects the array as RRRRRR...,GGGGGGG...,BBBBBBB.... I've updated my code in https://github.com/pertusa/mxnet_predict_cc accordingly and now it works. @tqchen, could you please change your loop in getMeanFile?
About the non-deterministic output, dropout layers give some random values as @zhangchen-qinyinghua commented. This is not a serious issue as the expected results are usually quite similar but some frameworks like Caffe have some layers that can be discarded int the TEST phase (usually dropouts). I wonder if there is something similar in MXNet, but if not it could be a good feature.

@pertusa Are you sure it's not BBBBBB...,GGGGGGGG....,RRRRRRRR......

Yes, I've checked it printing the data array received in MXPredSetInput() of c_predict_api.cc. With the python call from (mxnet_predict_example.py) and a red image, the first elements in the array are the ones with the highest values, so it's RRR...GGG...BBB...

Some small modifications:

Updated the code according to Google C Style and @tqchen 's suggestions.
Fix the image loading error and add synset result showing as @pertusa .

Good work Skylook, thanks!

Thanks for being patient on this and great contribution. Can you please do a lint on your file? like
dmlc-core/script/lint.py cpp mxnet cpp example/cpp

You can add this path to https://github.com/dmlc/mxnet/blob/master/Makefile#L145 so it is automatically checked by travis This should detect most of common things linter complained on, I commented a few. This is just to make sure the project have consistent programming style :)
After this is done, please

Rename the suffix from .cpp to .cc
update the PR to add yourself to https://github.com/dmlc/mxnet/blob/master/CONTRIBUTORS.md
Squash the commits to meaningful ones via http://mxnet.readthedocs.org/en/latest/contribute.html#how-to-combine-multiple-commits-into-one

So each commit means meaningful progress.

Then it is good to be merged.

@skylook can you update the code accordingly?

@skylook Hi, Thanks for improving my code in #531.
May I add my name after you to the header of image-classification-predict.cpp?

I am working on it, sorry for being too busy :)

I got some sample code from my workmates and didnot notice your github, I will sure to add your name when I finished :) @caprice-j

Hi, I finally have time to do these following changes:

Change code style to pass cpplint check.
Update README file to add more details of building and usage of the code.
Combine multiple commits into one.

Hopefully this patch is ready to merge :) @tqchen

Looks good to me. @tqchen

Great Job! Thanks for the contribution and make mxnet better for everyone!

@skylook Thanks for your kindness.


Zulip recipe

Q
A

Issue Type
Feature Request

Deployer Version
6.x

Description
At company we use Zulip and starting to think that we can make an integration of Deployer in our chats. Unfortunately didn't found an Zulip recipe.
If accepted, I will start to work on it.

Do it :) The more the merrier!


Categories Slice

[x] Slice
[x] Schemas
[x] Reducers
[x] Generators
[x] Actions
[x] Selectors
[x] Constants

Closed by #125


Mat/Pat - Add info in mail to supervisors and applicants on submission

AC

[ ] Add the following line to the email sent to supervisors informing a new form submission pending for approval

NOTE:  If the employee is taking only parental leave, a leave can be approved with a future date of birth, however, it is the responsibility of the supervisor and the employee to ensure parental leave is started only after the child is born. If there is any change in the date of the birth or leave start date, the employee or supervisor can edit them on Digital Journeys and resubmit. Not doing so can lead to payroll errors.

[ ] Add the following line to the email sent to applicants taking only parental leave confirming submission of a new form (NOT the email sent after approval)

NOTE:  Your supervisor can approve your parental leave with a future date of birth, however, it is both your and your supervisor's responsibility to ensure parental leave is started only after the child is born. If there is any change in the date of the birth or leave start date, the employee or supervisor can edit them on Digital Journeys and resubmit. Not doing so can lead to payroll errors.

There is another very small email fix in #1670 about adding pensions.
Could we do that at the while we have the emails open?

This email content are before approval. #1670 is after approval. I can check it with Andrew and add it along with this ticket.

@ayushdamani @fazil-ey
It is ready to test on TEST environment.

@ayushdamani yes needs to be bolded

Thank you, Fazil!
Moving back to in-progress!

@ayushdamani Updated content, selected text are bold now. Moving again for testing.

Tested in both TEST and DEV .Looking great - moving to PO
Supervisor email -

Employee email -

Dont see the note in Employee email. Application ID# 5329
Took only parental leave

@fazil-ey
I did check, I saw an email with notes. Same value selected.
birthing parent: No
Taking Parental leave: yesHoweverOnlyIWillBeTakingParentalLeave

Please hard refresh before launching mat-pat form or try in incognito.

It works, i was checking the wrong email!


Saga action retry semantics

Add retry semantics to saga actions. Quite a few possibilities here, but as a start support for periodic retries, up to a fixed number, or with some form of backoff

Some work done to remove the Command Ids from the Saga definition. #44 . See the PR comment for motivation.

Done in PR #56


F3/hev

Adds PHEV model that is tested in rust
TODO: write a demo script in python for the hev and compare to fastsim-2 hev results

@calbaker I'm seeing the tests aren't passing; I just re-requested they run again in case something was stale but I'm getting the same result.

@calbaker Should the python unit test section of the workflow be removed or commented out, then?
See bottom of the CI tests workflow file


Add ability to exclude more resources via the cloud-nuke-excuded tag

Currently, you can only exclude S3 buckets that are tagged with key cloud-nuke-excluded and value set to true. This PR adds more resources that are protected via this tag. This functionality is used in case you have a test environment that you do not want to get wiped out, like an integration setup with an EKS cluster, ELB, workers in an ASG etc. - you can propagate the tag across your setup for more fine-grained control (in comparison to specifying a creation time). The current setup is backwards compatible and has been tested with every resource.

Nice work @merkata! What is preventing this from getting merged in @brikis98 @yorinasub17

It would  be great if this feature can be reviewed / merged 💚

Bump.
This PR is related to this issue 166

Hi 👋
Any updates on this ?

@merkata this change set looks great to me, could you resolve the conflicts in the PR and add test cases for the new tagging feature?

@merkata this change set looks great to me, could you resolve the conflicts in the PR and add test cases for the new tagging feature?

Thanks a lot @ellisonc  - I've fixed the conflicts and I'll be writing the tests to cover the new functionality. Appreciate your input!

@merkata this change set looks great to me, could you resolve the conflicts in the PR and add test cases for the new tagging feature?

Thanks a lot @ellisonc - I've fixed the conflicts and I'll be writing the tests to cover the new functionality. Appreciate your input!

@merkata This looks like a great change! Could you also please update the README to reflect this expanded functionality as well?

@merkata this change set looks great to me, could you resolve the conflicts in the PR and add test cases for the new tagging feature?

Thanks a lot @ellisonc - I've fixed the conflicts and I'll be writing the tests to cover the new functionality. Appreciate your input!

@merkata This looks like a great change! Could you also please update the README to reflect this expanded functionality as well?

Thanks @zackproser , I'll update the README as well!

@merkata apologies for the delay, I attempted to fix the conflicts and get this merged but our tooling doesn't allow pushes to any branch named master and I don't want to steal your credit by making the changes on a new branch of my own.  Could you resolve the conflicts on a feature branch and update the PR?

@merkata apologies for the delay, I attempted to fix the conflicts and get this merged but our tooling doesn't allow pushes to any branch named master and I don't want to steal your credit by making the changes on a new branch of my own. Could you resolve the conflicts on a feature branch and update the PR?

Hey @ellisonc , thanks for the feedback! I fixed the conflicts, two things stand out:

the current README.md doesn't seem to cover exclusion of resources based on tags, the previous list of resources described which resources can be filtered out based on tag and I extended the descriptions of the resources I added.

please have a look at aws/snapshot.go, I believe the logic is correct, still there is the addition of not deleting a snapshot with the AWS backup tag and I want to make sure I got the logic right.

Thanks!

@merkata I also don't see the readme update you mentioned, did that not get staged?

@merkata I also don't see the readme update you mentioned, did that not get staged?

I didn't explain it accurately - I updated the old version of the README.md, it looked like this:
Inspecting and deleting all ACM Private CA in an AWS account
Inspecting and deleting all Auto scaling groups in an AWS account - except for ASGs tagged with Key=cloud-nuke-excluded Value=true
Inspecting and deleting all Elastic Load Balancers (v1 and v2) in an AWS account - except for ELBs tagged with Key=cloud-nuke-excluded Value=true
Inspecting and deleting all Transit Gateways in an AWS account
Inspecting and deleting all EBS Volumes in an AWS account - except for volumes tagged with Key=cloud-nuke-excluded Value=true
...

and I specifically added the except for <resource name> tagged with... section for each resource.
Now the section looks like this:
Cloud-nuke suppports 🔎 inspecting and 🔥💀 deleting the following AWS resources:

| Resource Family | Resource type
| --------------- | ----------
| EC2 | Auto scaling groups |
| EC2 | Elastic Load Balancers (v1 and v2) |
| EC2 | EBS Volumes |
| EC2 | Unprotected EC2 instances |
...

So I'm not sure where to make a note about excluding a resource based on an exclusion tag.

@merkata I see what you're saying. How about adding a new section header under the Excluding Resources by Age header (or maybe right above it?)

@zackproser sorry I wasn't able to response earlier, I've added a short paragraph with the list of resources that would be supported, let me know what you think of it. Thanks!

This is looking very close @merkata, thanks for the great work! Left a couple NITs (one typo that was present in a couple of places, and some thoughts on the use of aws.StringValue and related methods for dereferencing pointers. Otherwise we should be good to get this merged shortly :+1:

Hey @merkata, we haven't heard back from you for quite some time. Do you plan to continue making progress on this PR?
We've recently made some refactoring changes in the code and it may require a bit of conflict resolution.

Moved this change to this PR - https://github.com/gruntwork-io/cloud-nuke/pull/573.


false merge query parameter order

in Merge.ts line 93:
return `${mergeStr}${setCypher}${onCreateCypher}${removeCypher}${deleteCypher}${nextClause}`;

should be:
return `${mergeStr}${onCreateCypher}${setCypher}${removeCypher}${deleteCypher}${nextClause}`;

example merge query with set and on create: https://neo4j.com/docs/cypher-manual/current/clauses/merge/#merge-merge-with-on-create-and-on-match

Hi @FCQCarryOut
What are you trying to achieve?
Your comment is correct, I'll fix it soon, however, keep in mind that the examples you are providing are not affected by this:
For example:
MERGE (person:Person)
ON CREATE
  SET person.found = true
RETURN person.name, person.found

Would be done as:
new Cypher.Match(person).onCreate([person.property("found"), Cypher.true]).return(person.property("name"))

In this case, .onCreate is also adding the SET (maybe it could be worth renaming the method to onCreateSet)
The fix you point out would be used to generate queries like this:
MERGE (this0:MyLabel)
ON CREATE SET
    this0.age = 20
SET
    this0.age = 10

@angrykoala in your example, did you mean new Cypher.Merge instead of new Cypher.Match ?


using pipenv in container fails for first run...

Moin all, I have been using pipenv in a CentOS7 container, and it fails on the first pipenv install. I have attached a log, let me know what else you want to have a look at...
$ docker pull centos/python-35-centos7
[...]
$ docker run --rm -ti  centos/python-35-centos7 /bin/bash
(app-root)pip install pipenv
Collecting pipenv
  Downloading pipenv-7.7.9.tar.gz (3.6MB)
    100% |████████████████████████████████| 3.6MB 14.0MB/s
Collecting virtualenv (from pipenv)
  Downloading virtualenv-15.1.0-py2.py3-none-any.whl (1.8MB)
    100% |████████████████████████████████| 1.8MB 30.1MB/s
Collecting pew>=0.1.26 (from pipenv)
  Downloading pew-0.1.26-py2.py3-none-any.whl
Collecting pip>=9.0.1 (from pipenv)
  Downloading pip-9.0.1-py2.py3-none-any.whl (1.3MB)
    100% |████████████████████████████████| 1.3MB 16.0MB/s
Collecting pip-tools>=1.9.0 (from pipenv)
  Downloading pip_tools-1.9.0-py2.py3-none-any.whl
Collecting requests>2.0.0 (from pipenv)
  Downloading requests-2.18.4-py2.py3-none-any.whl (88kB)
    100% |████████████████████████████████| 90kB 9.8MB/s
Requirement already satisfied (use --upgrade to upgrade): setuptools>=17.1 in /opt/app-root/lib/python3.5/site-packages (from pew>=0.1.26->pipenv)
Collecting pythonz-bd>=1.10.2 (from pew>=0.1.26->pipenv)
  Downloading pythonz_bd-1.11.4-py3-none-any.whl (117kB)
    100% |████████████████████████████████| 118kB 16.2MB/s
Collecting virtualenv-clone>=0.2.5 (from pew>=0.1.26->pipenv)
  Downloading virtualenv-clone-0.2.6.tar.gz
Collecting click>=6 (from pip-tools>=1.9.0->pipenv)
  Downloading click-6.7-py2.py3-none-any.whl (71kB)
    100% |████████████████████████████████| 73kB 7.4MB/s
Collecting six (from pip-tools>=1.9.0->pipenv)
  Downloading six-1.11.0-py2.py3-none-any.whl
Collecting first (from pip-tools>=1.9.0->pipenv)
  Downloading first-2.0.1-py2.py3-none-any.whl
Collecting idna<2.7,>=2.5 (from requests>2.0.0->pipenv)
  Downloading idna-2.6-py2.py3-none-any.whl (56kB)
    100% |████████████████████████████████| 57kB 10.6MB/s
Collecting chardet<3.1.0,>=3.0.2 (from requests>2.0.0->pipenv)
  Downloading chardet-3.0.4-py2.py3-none-any.whl (133kB)
    100% |████████████████████████████████| 135kB 16.6MB/s
Collecting certifi>=2017.4.17 (from requests>2.0.0->pipenv)
  Downloading certifi-2<IP_ADDRESS>-py2.py3-none-any.whl (349kB)
    100% |████████████████████████████████| 352kB 10.6MB/s
Collecting urllib3<1.23,>=1.21.1 (from requests>2.0.0->pipenv)
  Downloading urllib3-1.22-py2.py3-none-any.whl (132kB)
    100% |████████████████████████████████| 135kB 30.6MB/s
Collecting resumable-urlretrieve (from pythonz-bd>=1.10.2->pew>=0.1.26->pipenv)
  Downloading resumable_urlretrieve-0.1.5-py2.py3-none-any.whl
Installing collected packages: virtualenv, idna, chardet, certifi, urllib3, requests, resumable-urlretrieve, pythonz-bd, virtualenv-clone, pew, pip, click, six, first, pip-tools, pipenv
  Running setup.py install for virtualenv-clone
  Found existing installation: pip 7.1.2
    Uninstalling pip-7.1.2:
      Successfully uninstalled pip-7.1.2
  Running setup.py install for pipenv
Successfully installed certifi-2<IP_ADDRESS> chardet-3.0.4 click-6.7 first-2.0.1 idna-2.6 pew-0.1.26 pip-9.0.1 pip-tools-1.9.0 pipenv-7.7.9 pythonz-bd-1.11.4 requests-2.18.4 resumable-urlretrieve-0.1.5 six-1.11.0 urllib3-1.22 virtualenv-15.1.0 virtualenv-clone-0.2.6
(app-root)pipenv install requests
Courtesy Notice: Pipenv found itself running within a virtual environment, so it will automatically use that environment, instead of creating its own for any project.
Creating a Pipfile for this project…
Discovering imports from local codebase…
Traceback (most recent call last):
  File "/opt/app-root/bin/pipenv", line 9, in <module>
    load_entry_point('pipenv==7.7.9', 'console_scripts', 'pipenv')()
  File "/opt/app-root/lib/python3.5/site-packages/pipenv/vendor/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/opt/app-root/lib/python3.5/site-packages/pipenv/vendor/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/opt/app-root/lib/python3.5/site-packages/pipenv/vendor/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/opt/app-root/lib/python3.5/site-packages/pipenv/vendor/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/opt/app-root/lib/python3.5/site-packages/pipenv/vendor/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/opt/app-root/lib/python3.5/site-packages/pipenv/cli.py", line 1610, in install
    ensure_project(three=three, python=python, system=system, warn=True)
  File "/opt/app-root/lib/python3.5/site-packages/pipenv/cli.py", line 593, in ensure_project
    ensure_pipfile(validate=validate)
  File "/opt/app-root/lib/python3.5/site-packages/pipenv/cli.py", line 324, in ensure_pipfile
    for req in import_from_code('.'):
  File "/opt/app-root/lib/python3.5/site-packages/pipenv/cli.py", line 278, in import_from_code
    for r in pipreqs.get_all_imports(path):
  File "/opt/app-root/lib/python3.5/site-packages/pipenv/vendor/pipreqs/pipreqs.py", line 106, in get_all_imports
    with open(join("stdlib"), "r") as f:
FileNotFoundError: [Errno 2] No such file or directory: '/opt/app-root/lib/python3.5/site-packages/pipenv/vendor/pipreqs/stdlib'
(app-root)pipenv install requests
Courtesy Notice: Pipenv found itself running within a virtual environment, so it will automatically use that environment, instead of creating its own for any project.
Installing requests…
Requirement already satisfied: requests in /opt/app-root/lib/python3.5/site-packages
Requirement already satisfied: idna<2.7,>=2.5 in /opt/app-root/lib/python3.5/site-packages (from requests)
Requirement already satisfied: chardet<3.1.0,>=3.0.2 in /opt/app-root/lib/python3.5/site-packages (from requests)
Requirement already satisfied: certifi>=2017.4.17 in /opt/app-root/lib/python3.5/site-packages (from requests)
Requirement already satisfied: urllib3<1.23,>=1.21.1 in /opt/app-root/lib/python3.5/site-packages (from requests)

Adding requests to Pipfile's [packages]…
P.S. You have excellent taste! ✨ 🍰 ✨
Locking [dev-packages] dependencies…
Locking [packages] dependencies…
Updated Pipfile.lock!

I found the same behaviour. It looks like during pipenv installation both stdlib and mappings are not copied in the vendor/pipreqs directory.

this should now be fixed.


Ensure that Cloud Storage bucket is not anonymously or publicly accessible

Possible check to target the following resource

[ ] google_storage_bucket_iam_binding

closing, tracking in spreadsheet


Joining a singleplayer world with forgified fabric and supplementaries causes a Network Protocol Error (Neoforge 1.21.1)

Upon joining a singleplayer world with Supplementaries and forgified fabric api, the game will disconnect you from the server with a server crash log.
Only occurs when Supplementaries and Forgified are running together
latest.log
disconnect-2024-11-06_17.02.07-client.txt

According to the Supplementaries dev, FFAPI is altering the data jsons of the mod.
(His example) Content of one becomes "minecraft:" when it was "minecraft:bed"


is there a way to override css?

I couldn't found any prop or way to pass classNames in the component, I wanna update the width of the month input

@manish-basargekar I think it might be possible to update the width of the element by change the --month-input-width variable on the html element. Not entirely sure if it does not get overwritten.

Sure, you can inspect the component with your browser's developer tools to find the relevant CSS classes. Then, you can apply any custom styles you need.
Here's an example of how I customized the styles for react-lite-month-picker:
/* react-lite-month-picker styles */ ._monthInputField_1ny07_1 {     padding: 0.5rem !important;     width: 100%;     min-width: 0px;     font-size: 20px;     margin-bottom: 0rem !important; }


Multiple PySoundIo instances crash the program

>>> import pysoundio
>>> sio1 = pysoundio.PySoundIo(backend=pysoundio.SoundIoBackendAlsa)
>>> sio2 = pysoundio.PySoundIo(backend=pysoundio.SoundIoBackendJack)
>>> sio1._soundio
140182975315920
>>> sio2._soundio
140182972132512
>>> sio1.close()
>>> sio2.close()
'python3' terminated by signal SIGABRT (Abort)

The main reason of this problem is caused by the use of global variables in https://github.com/joextodd/pysoundio/blob/aca6ce52828885f620e259a530f6e63445854941/pysoundio/_soundiox.c#L347-L349
In the function pysoundio__destroy, the code calls soundio_destroy against the global variable rc.soundio instead of a handle passed from the Python layer, which is overwritten by consecutive pysoundio__create calls.
https://github.com/joextodd/pysoundio/blob/aca6ce52828885f620e259a530f6e63445854941/pysoundio/_soundiox.c#L396-L404
Similarly, PySoundIo will also crash if the user creates multiple streams, because the handles to the streams are also stored in global variables.

Thanks for reporting this! This was a terrible design but I backed myself into a corner with the C-API a bit. #33 includes a migration to CFFI which will fix this issue as well as make it more efficient!

Thanks for reporting this! This was a terrible design but I backed myself into a corner with the C-API a bit. #33 includes a migration to CFFI which will fix this issue as well as make it more efficient!

Indeed, global variables are the source of many bugs. And I understand fixing this takes a great effort. So, Thank you for the fix!
I am looking forward to try the new version! 👍


How can you manage users on the interface?

Issue submitter TODO list

[X] I've searched for an already existing issues here
[X] I'm running a supported version of the application which is listed here and the feature is not present there

Is your proposal related to a problem?
Now I have to create several users and restrict access for them in the UI of other services! Now I can not such a feature has not found yet!
Describe the feature you're interested in
Like if there could be such a thing at the entrance asked login and password about the user and then skipped and they could be attached to the role to so see was like say the company has 3 admins they have access for all services but other users only view the topics have access!
Describe alternatives you've considered
The service confluent-cc has such a thing user management at login asks for username and password, and can we implement this thing for this project kafka-ui?
Version you're running
latest in docker
Additional context
No response

You can manage users with oauth2. We are mapping user roles to OKTA groups. Works very well

Hi,
editing RBAC within UI is planned within #3073.
Creating several users within the app is not possible, we support login form with a single user, otherwise you have to use OAuth/LDAP.
Let me know if you have any other questions.


sqitch doesn't seem to work with remote databases

I have a MySQL database running on Amazon RDS. I'm able to connect to it using mysql command line and also MySQLWorkbench. But when I try to use it it fails with an error 'Access denied for user ']username]'@'[ip]' (using password: YES)'. Here is the command that I'm using
sqitch status db:mysql://[user]:[pwd]@[host]:[port]/[databasename]
Is sqitch supposed to work with remote hosts?
-Srini

Is that literally the connection URI you're using? I think you want to set the username, password, host, port, and database name.

No, I masked the connection details. I do not want to past the connection details on this thread. Is there a way to check the log for connection attempt? Thank you.

Well your error output, 'Access denied for user ']username]'@'[ip]' (using password: YES)'., looked weird. I guess you pasted in the details? Do you have any non-URL-safe characters in the URI?
You can get a trace from the mysql interface from Sqitch by setting the DBI_TRACE environment variable.
DBI_TRACE=5 sqitch status $URI

Sure. I masked the output to remove username and IP, sorry for the confusion. The password does have some special characters and I think that is causing the issue. I changed the password. Now it seems to be working. I'll have to figure out how to use passwords with special characters. I also found out that I need to create a database with name 'sqitch' on the mysql instance. Thank you for the help.

I think wrapping the password with single quotes did the trick. I got it all working. Thanks agian.

It's a URI, so you need to URI-encode any special characters. Here's a one-liner you can use to encode things properly:
perl -MURI -E 'say URI->new("db:mysql:foo:bîø@somehost/dbname")'

db:mysql:foo:b%C3%AE%C3%B8/dbname


Cannot use a scalar value as an array

Inside the focus/config.php it seems that something is breaking the whole thing. I have pretty much a basic setup with latest k-next focus as well as latest beta and made sure all files have templates.

temporary fix is to declare inside the function first
$options = array();

Unfortunately this is no practical fix, because focusCrop() accepts options as third parameter, which need to be passed as an array.
In K2 there was a (deprecated) Option, that the 3rd Parameter could also be an integer, which was then used as quality. This is no longer supported.
The error ”Cannot use skalar value …“ indicates, that you probably use the old syntax focusCrop(200,300,80) instead of focusCrop(200,300,['quality' => 80]). Could you please check that?

you are correct. i was using the old syntax


Improve test coverage to about 80% or greater

This needs to be broken down. This task cant be done in 400 changes or less and requires tests across the whole system, aiming for portions of the system. This is more of a milestone

Yes agree that the task needs to be broken down.. not exactly sure how to splice it into different issues. Currently PR #https://github.com/walletd/walletd/pull/63 adds more test coverage but not yet at 80% yet


Support for 10.0.19041.1021

Running SuperRDP <IP_ADDRESS> and synced the latest config.

Download of my config+dll from "share to github" link: 10.0.19041.1021.zip

select checkbox AutoSupport, and then install

select checkbox AutoSupport, and then install

It tells me to use wechat (I don't have an account on there), and clicking "How to get code?" seems to not open anything.

type the code 1869E4 when your localtime is 2022-05-13 10:00.
And you can get the new version <IP_ADDRESS>.


refactor: Session.

Please add tests for the new warnings.

Please add tests for the new warnings.

Will do, actually other refactoring for session level will also be done with this.

Please add tests for the new warnings.

Will do, actually other refactoring for session level will also be done with this.

@mkundu1, No new warnings have been added as part of this PR. We have decided to directly make these changes without adding deprecation warnings.

Can you please verify PyConsole with this restructuring? These changes will be packaged in Fluent's PyFluent sometime during 25.1 development but we can verify them before merging this PR. You can set FLUENT_PYFLUENT_ROOT env var to pickup your local PyFluent in PyConsole.

Few minor issues needs fixing for 25.1. I have them locally and checked it's working as well. I'll push them as soon as the develop branch for 25.1 is available.


No-constructors classes

Congratulations on a releasing the well structured book! Neat use of Jupyter book!
I thought of use of no-constructor classes in your code - good for combining methods together,
but not really a great pattern for starting programming - not to replicate when a learner just starts writing Python code.
class LinearRegression:

    def fit(self, X, y, intercept = False):

        # record data and dimensions
        if intercept == False: # add intercept (if not already included)
            ones = np.ones(len(X)).reshape(len(X), 1) # column of ones
            X = np.concatenate((ones, X), axis = 1)
        self.X = np.array(X)
        self.y = np.array(y)
        self.N, self.D = self.X.shape

        # estimate parameters
        XtX = np.dot(self.X.T, self.X)
        XtX_inverse = np.linalg.inv(XtX)
        Xty = np.dot(self.X.T, self.y)
        self.beta_hats = np.dot(XtX_inverse, Xty)

        # make in-sample predictions
        self.y_hat = np.dot(self.X, self.beta_hats)

        # calculate loss
        self.L = .5*np.sum((self.y - self.y_hat)**2)

I understand this is an idiom throughout the book, but if you consider restructuring the code at some point,
there can be a more functional representation:

        # estimate parameters
        XtX = np.dot(self.X.T, self.X)
        XtX_inverse = np.linalg.inv(XtX)
        Xty = np.dot(self.X.T, self.y)
        self.beta_hats = np.dot(XtX_inverse, Xty)

def estimate_betas(X,Y):
        XtX = np.dot(X.T, X)
        XtX_inverse = np.linalg.inv(XtX)
        Xty = np.dot(X.T, Y)
        return np.dot(XtX_inverse, Xty)

The estimate_betas() function you can probably mount back to LinearRegression() for similar interface across fucntions.
Another idea is probably write a small comment on why classes with .fit() and .predict() would are needed (similar to sciket-learn API, uniform across many models, etc)


Story #1: Image Processing

User Story
As a user, I want an image uploading section, so that I can easily input an image of a food.
Task

Frontend- create a image upload section to easily input an image of a food
Backend- create api to receive and process the image

https://github.com/daduuu/CS130/commit/844effb3fbe46b66847fc79e61c32d0246b0d01b
PR #10


Crash when there are assembly loading errors

I get this error, but only on my CI it on;y works with the Resharper runner and is now crashing in VS runner and the CI
Any idea why?
[xUnit.net 00:00:02.21]   Starting:    Namespace1.Application1.AcceptanceTests
-> Loading plugin /LivingDoc.SpecFlowPlugin.dll
-> Loading plugin /TechTalk.SpecFlow.xUnit.SpecFlowPlugin.dll
-> Loading plugin /AdCodicem.SpecFlow.MicrosoftDependencyInjection.SpecFlowPlugin.dll
-> Loading plugin /Namespace1.Application1.AcceptanceTests.dll
-> Using specflow.json
-> #16: warning: The previous ScenarioContext was already disposed.
[xUnit.net 00:00:02.68]       System.AggregateException : One or more errors occurred. (Unable to load one or more of the requested types.
[xUnit.net 00:00:02.68]       A ByRef-like type cannot be used as the type for an instance field in a non-ByRef-like type.) (Object reference not set to an instance of an object.)[xUnit.net 00:00:02.67]     My test [FAIL]

[xUnit.net 00:00:02.68]       ---- System.Reflection.ReflectionTypeLoadException : Unable to load one or more of the requested types.
[xUnit.net 00:00:02.68]       A ByRef-like type cannot be used as the type for an instance field in a non-ByRef-like type.
[xUnit.net 00:00:02.68]       ---- System.NullReferenceException : Object reference not set to an instance of an object.
[xUnit.net 00:00:02.68]       Stack Trace:
[xUnit.net 00:00:02.68]
[xUnit.net 00:00:02.68]         ----- Inner Stack Trace #1 (System.Reflection.ReflectionTypeLoadException) -----
[xUnit.net 00:00:02.68]            at System.Reflection.RuntimeModule.GetTypes(RuntimeModule module)
[xUnit.net 00:00:02.68]            at System.Reflection.RuntimeModule.GetTypes()
[xUnit.net 00:00:02.68]            at System.Reflection.Assembly.GetTypes()
[xUnit.net 00:00:02.68]            at AdCodicem.SpecFlow.MicrosoftDependencyInjection.DependencyInjectionPlugin.<>c.<ConfigureBindings>b__1_0(Assembly assembly)
[xUnit.net 00:00:02.68]            at System.Linq.Enumerable.SelectManySingleSelectorIterator`2.MoveNext()
[xUnit.net 00:00:02.68]            at System.Linq.Enumerable.WhereEnumerableIterator`1.MoveNext()
[xUnit.net 00:00:02.68]            at AdCodicem.SpecFlow.MicrosoftDependencyInjection.DependencyInjectionPlugin.ConfigureBindings(IServiceCollection services)
[xUnit.net 00:00:02.68]            at AdCodicem.SpecFlow.MicrosoftDependencyInjection.DependencyInjectionPlugin.ConfigureServices(IObjectContainer delegateContainer)
[xUnit.net 00:00:02.68]            at AdCodicem.SpecFlow.MicrosoftDependencyInjection.DependencyInjectionPlugin.<Initialize>b__3_1(Object _, CustomizeScenarioDependenciesEventArgs args)
[xUnit.net 00:00:02.68]            at TechTalk.SpecFlow.Plugins.RuntimePluginEvents.RaiseCustomizeScenarioDependencies(ObjectContainer scenarioContainer)
[xUnit.net 00:00:02.68]            at TechTalk.SpecFlow.Infrastructure.ContainerBuilder.CreateScenarioContainer(IObjectContainer testThreadContainer, ScenarioInfo scenarioInfo)
[xUnit.net 00:00:02.68]            at TechTalk.SpecFlow.Infrastructure.ContextManager.InitializeScenarioContext(ScenarioInfo scenarioInfo)
[xUnit.net 00:00:02.68]            at TechTalk.SpecFlow.Infrastructure.TestExecutionEngine.OnScenarioInitialize(ScenarioInfo scenarioInfo)
[xUnit.net 00:00:02.68]            at TechTalk.SpecFlow.TestRunner.OnScenarioInitialize(ScenarioInfo scenarioInfo)
[xUnit.net 00:00:02.68]            at Namespace1.Application1.AcceptanceTests.End2End.Feature1.ScenarioInitialize(ScenarioInfo scenarioInfo)
[xUnit.net 00:00:02.68]         /agent/_work/1/s/Backend/Namespace1.Application1.AcceptanceTests/End2End/My test.feature(4,0): at Namespace1.Application1.AcceptanceTests.End2End.Feature1.MyTest()
[xUnit.net 00:00:02.68]         ----- Inner Stack Trace #2 (System.NullReferenceException) -----
[xUnit.net 00:00:02.68]            at TechTalk.SpecFlow.Infrastructure.TestExecutionEngine.OnScenarioEnd()
[xUnit.net 00:00:02.68]            at TechTalk.SpecFlow.TestRunner.OnScenarioEnd()
[xUnit.net 00:00:02.68]            at Namespace1.Application1.AcceptanceTests.End2End.Feature1.TestTearDown()
[xUnit.net 00:00:02.68]            at Namespace1.Application1.AcceptanceTests.End2End.Feature1.System.IDisposable.Dispose()
[xUnit.net 00:00:02.68]         C:\Dev\xunit\xunit\src\xunit.execution\Extensions\ReflectionAbstractionExtensions.cs(79,0): at ReflectionAbstractionExtensions.DisposeTestClass(ITest test, Object testClass, IMessageBus messageBus, ExecutionTimer timer, CancellationTokenSource cancellationTokenSource)

Funny, it's because of IronPDF. I cannot remove that dependency

From the screenshot above it looks like a VS or xUnit problem.
But if I remove the package, then the test is green and won't crash

Hello,
From what I see in your screenshot, I'm not sure that the DI plugin caused the issue.
Can you provide me a link to a repo presenting the issue, so I can investigate ? Thanks !

The issue is here:
[xUnit.net 00:00:02.68]            at System.Reflection.RuntimeModule.GetTypes()
[xUnit.net 00:00:02.68]            at System.Reflection.Assembly.GetTypes()
[xUnit.net 00:00:02.68]            at AdCodicem.SpecFlow.MicrosoftDependencyInjection.DependencyInjectionPlugin.<>c.<ConfigureBindings>b__1_0(Assembly assembly)

The issue is that  System.Reflection.Assembly.GetTypes() failed, and this is triggered by MicrosoftDependencyInjection.DependencyInjectionPlugin.
I don't think this is easy to reproduce in a small application.

So this is
https://github.com/AdCodicem/SpecFlowMicrosoftDependencyInjection/blob/0c18d04280683e8231ed367524e3a33f1257539f/AdCodicem.SpecFlow.MicrosoftDependencyInjection/DependencyInjectionPlugin.cs#L23-L28
Is there a reason we walk over all the current assemblies in the appdomain? I think this should only be the executing assembly

Thanks for providing more info.
I think this is related to this issue : #6
That's why I checked all the assemblies.
I think I can solve this issue by filtering the types.
I'll try to reproduce this issue.

Hi, since I don't have access to your code, can you test this for me ?

Download the DI plugin project and register it directly as a dependency instead of using the nuget.
In the code snippet you identified earlier, replace assembly.GetTypes() with assembly.GetLoadableTypes()
Run your tests again and tell me if the issue is fixed
Thanks !

Yes I could test, but maybe a pre-release package would be easier / better?

Hello, as you suggested, I published a pre-release package :
https://www.nuget.org/packages/AdCodicem.SpecFlow.MicrosoftDependencyInjection/1.1.6-cmarty-fix-by-ref-type-registration.1

thanks!

Unfortunately this won't fix it.
GetLoaderTypes just rethrows this error:

But if we check the code of the VSTest runner, it just ignores these errors, see the continue
            foreach (var assemblyName in depsAssms.Concat(dllsInSources))
            {
                try
                {
                    var assembly = Assembly.Load(assemblyName);
                    foreach (var type in assembly.DefinedTypes)
                    {
#pragma warning disable CS0618
                        if (type == null || type.IsAbstract || type == typeof(DefaultRunnerReporter).GetTypeInfo() || type == typeof(DefaultRunnerReporterWithTypes).GetTypeInfo() || type.ImplementedInterfaces.All(i => i != typeof(IRunnerReporter)))
                            continue;
#pragma warning restore CS0618

                        var ctor = type.DeclaredConstructors.FirstOrDefault(c => c.GetParameters().Length == 0);
                        if (ctor == null)
                        {
                            ConsoleHelper.SetForegroundColor(ConsoleColor.Yellow);
                            Console.WriteLine($"Type {type.FullName} in assembly {assembly} appears to be a runner reporter, but does not have an empty constructor.");
                            ConsoleHelper.ResetColor();
                            continue;
                        }

                        result.Add((IRunnerReporter)ctor.Invoke(new object[0]));
                    }
                }
                catch
                {
                    continue;
                }

So proposal: also ignore (and maybe log them to Debug), these loaders errors (of course move to the next assemby)

related in NLog ;)
https://github.com/NLog/NLog/blob/0b952cde2a2bda471663695788b5d3a626fb8b41/src/NLog/Internal/Reflection/ReflectionHelpers.cs#L48-L85

See also
https://stackoverflow.com/questions/7889228/how-to-prevent-reflectiontypeloadexception-when-calling-assembly-gettypes
TL;DR catch it;
I like this simple approach: https://stackoverflow.com/a/65023201/201303

I don't think it'll solve the issue, because this is exactly what it's done in the GetLoadableTypes extension method
https://github.com/AdCodicem/SpecFlowMicrosoftDependencyInjection/blob/0c18d04280683e8231ed367524e3a33f1257539f/AdCodicem.SpecFlow.MicrosoftDependencyInjection/Internal/AssemblyExtensions.cs#L17-L24
I'm still investigating

So I tried to exclude the "By-Ref" types from being loaded, here is another pre-release package:
https://www.nuget.org/packages/AdCodicem.SpecFlow.MicrosoftDependencyInjection/1.1.6-cmarty-fix-by-ref-type-registration.2
Can you test it and tell me if it solved your error ?

I don't think it'll solve the issue, because this is exactly what it's done in the GetLoadableTypes extension method
https://github.com/AdCodicem/SpecFlowMicrosoftDependencyInjection/blob/0c18d04280683e8231ed367524e3a33f1257539f/AdCodicem.SpecFlow.MicrosoftDependencyInjection/Internal/AssemblyExtensions.cs#L17-L24
I'm still investigating

ow yes indeed. The decompiled source was a bit hard to read haha

I'm busy testing. First signs are good :)

PS if you merge #11 and #12, it's easier to debug :)

So I tried to exclude the "By-Ref" types from being loaded, here is another pre-release package: https://www.nuget.org/packages/AdCodicem.SpecFlow.MicrosoftDependencyInjection/1.1.6-cmarty-fix-by-ref-type-registration.2
Can you test it and tell me if it solved your error ?

Confirm, this works!

Fixed by PR #13

Thanks!


Spinners animations

Description

Added animation helpers to spinners
Motivation & Context

Type of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Refactoring (non-breaking change)
[ ] Breaking change (fix or feature that would change existing functionality)

Checklist

[x] I have read the contributing guidelines
[x] My code follows the code style of the project (using npm run lint)
[x] My change introduces changes to the documentation
[x] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[x] All new and existing tests passed

Live previews

https://deploy-preview-38160--twbs-bootstrap.netlify.app/docs/5.3/components/spinners/#animations---reverse

Related issues

Issue #38150

Thanks for this PR @oraliahdz but unfortunately, this is what we decided not to do in https://github.com/twbs/bootstrap/pull/38081.


Bluetooth: Audio: Move channel allocation strings to a public header

In https://github.com/zephyrproject-rtos/zephyr/blob/2396b01b1ea2b0b6a61591155508cf19cd9d367c/subsys/bluetooth/audio/shell/audio.h#L714
There is a nice function, returning strings for all the channel allocations.
We would like to have this moved to a public location, and have a guard in place so only one bit can be set in the location bitfield sent to the function. If more than one bit is set it could return "Invalid" or something similar.

As per Nick Hunns  'Introducing Bluetooth LE Audio', chapter 3.6:

A single channel speaker would normally set both the Front Left and Front Right audio locations

So more than one bit can be set. @Thalley do you remember the corresponding spec, and if this is still true?

Alternatively, and perhaps better, the function could take a pointer to a string buffer and that buffer size. Then it can iterate and populate up to the given size. If it goes over, it could write "..." on the last indexes before the NUL terminator.

If we want to support multiple bits, then we need a buffer in RAM, whereas just handling a single bit per call is effectively only in flash.
Different applications may also log or print them differently (e.g. the shell prints them indented with one value per line).
I suggest we keep it simple and just returning a string for a specific bit, and then the application will be in a much better condition to use this as they see fit.

I would like to do this ticket. My plan to is to move the function to subsys/bluetooth/audio/audio.c and rename it to something like
char *bt_audio_chan_location_bit_to_str(enum bt_audio_location chan_allocation)

?
Should I add a test case for when someone attempts to set more then one bit?

That would be great :)
While this issue only requests the channel allocation, there is actually quite a few others that could benefit from the same treatment: https://github.com/zephyrproject-rtos/zephyr/blob/main/subsys/bluetooth/audio/shell/audio.h#L270-L834
I suggest to add the public API string function for all of these (some of them are easier than location, as they are not bitfields).

Should I add a test case for when someone attempts to set more then one bit?

I think that may be tricky to test. What do you think the behavior should be in that case? Ideally the function will return a const char *, so on error it could

Return NULL
Return "Invalid input" (or similar)
Return the first bit set
Assert

I think that may be tricky to test. What do you think the behavior should be in that case? Ideally the function will return a const char *, so on error it could

Return NULL
Return "Invalid input" (or similar)
Return the first bit set
Assert

I like returning NULL or "Invalid Input". IMO returning the first bit set is not the correct behaviour and Asserting would be overkill. I will go with NULL unless you want "Invalid Input". I wont do the test case.

Biggest issue with returning NULL, is that you always have the check the output before printing/logging the return value, so in that case returning an error string may be better.

While this issue only requests the channel allocation, there is actually quite a few others that could benefit from the same treatment: https://github.com/zephyrproject-rtos/zephyr/blob/main/subsys/bluetooth/audio/shell/audio.h#L270-L834

Are you okay with me doing this all in one PR?

Absolutely! :)

@Ballen7 any progress on this?


Why DSP is not enabled in arm dot prod q15.

https://github.com/ARM-software/CMSIS-DSP/blob/7236701c153c9bbdec00568a2821a39039988d8f/Source/BasicMathFunctions/arm_dot_prod_q15.c#L154

@majorlin Thanks for finding this.
I don't see any good reason for this. It is a very old change in the library.

@majorlin I correct what I have just said. I __SMLALD requires two Q15 packed in an int32.
But this line is using *pSrcA++ to read a q15 from the tail of the vector.
So the instruction __SMLALD cannot be used here.


updateAutoHeight() transition

This is a (probably):

[x] enhancement

Swiper Version: 4.1.0 via npm

What I did
I have two use cases, where I successfully call updateAutoHeight() and I'd like the new wrapper height to be transitioned with the same 'speed' parameter I supply:

Unlike swiping or clicking on navigation arrows in thumbs-gallery, clicking a specific thumb slide didn't update main gallery wrapper height
Upon lazyload (as described here): https://github.com/nolimits4web/Swiper/issues/1537

Expected Behavior
Same wrapper height transition as on swipe or navigation.
Actual Behavior
A hard jump.
P.S. I'm afraid I haven't seen any relevant answers on 'swiper updateAutoHeight transition' on SO or here.

JSFiddle or example with the issue?

Sorry it took a while. I've moved on for now by giving the lazy-preloader the desired image dimensions and placing navigation into the gallery-top.
Here is a fiddle: https://jsfiddle.net/dpfqr9u5/36/
Thank you!


Optimisation: Store VoteTracker instances in an array instead of Map #4275

PR Description
Optimisation: Store VoteTracker instances in an array instead of Map #4275
Fixed Issue(s)

Documentation

[ ] I thought about documentation and added the documentation label to this PR if updates are required.

Changelog

[ ] I thought about adding a changelog entry, and added one if I deemed necessary.

@ajsutton  are these changes sufficient for this task ?

should i abstract out the below snippets and move them to Store class
1)
    store.highestVotedValidatorIndex = getHighestVotedValidatorIndex();

shall become
store.updateHighestVotedValidatorIndex(getHighestVotedValidatorIndex());// we will have a new method in Store class to do this

    if (store.highestVotedValidatorIndex.intValue() >= store.votes.length) {
      store.votes = Arrays.copyOf(store.votes, store.highestVotedValidatorIndex.intValue() + 1000);
    }

shall become
store.ensureSize(store.highestVotedValidatorIndex.intValue())

    votes.forEach(
        (key, value) -> {
          store.votes[key.intValue()] = value;
        });

shall become
 store.batchUpdate(votes);

the rationale behind this is we should keep the instance variables of a Store class private , and modify them via class's methods

@ajsutton should i abstract out the below snippets and move them to Store class
...
the rationale behind this is we should keep the instance variables of a Store class private , and modify them via class's methods

Store is a bit unusual in having package protected variables.  We started off with them all being private but the code to apply changes from transactions (particularly for things other than votes via StoreTransaction) is quite complex so it was better to have them in a separate class for testability and just to make Store not be thousands of lines long.
So I'm a bit in two minds about what to do with StoreVoteUpdater here - it currently follows the same pattern as StoreTransaction and directly accesses the variables but as you suggest, it doesn't have much complexity so it would be reasonable to just have those package protected methods in Store instead. But then the approach is inconsistent between the two forms of transactions.
Given that we'd still wind up reading from the variables as part of the getters in StoreVoteUpdater I think I'd probably leave it with the variables just being package protected.  It is a bit odd, but it's just complex enough that we're essentially treating the package as the boundary of encapsulation instead of the class.

@ajsutton
in the private constructor of Store class.
i have tried to initialize the votes array like the below
    this.highestVotedValidatorIndex =
        votes.keySet().stream().max(Comparator.naturalOrder()).orElse(UInt64.ZERO);

    this.votes = new VoteTracker[this.highestVotedValidatorIndex.intValue() + 1];

    votes.forEach(
        (key, value) -> {
          this.votes[key.intValue()] = value;
        });
    this.blockMetadata = blockMetadata;

but these two tests(in ForkChoiceTests) were failing onBlock_shouldHandleNonCanonicalBlockThatUpdatesJustifiedCheckpoint() and onBlock_shouldUpdateVotesBasedOnAttestationsInBlocks()  with ArrayIndexOutOfBoundsException due to my above change.
upon debugging i found that, the StoreVoteUpdater.getVote() method is not always preceeded by StoreVoteUpdator.putVote()
(basically we are calling  StoreVoteUpdater.getVote(index) even before that index is inserted/present  in Store).
so leaving the current code as it is should be fine, where we initialize the Store.votes array with size 1024.
but want to point out that this can throw  ArrayIndexOutOfBoundsException if someone calls getVote(index) with index>=1024 in the initial day itself.   would like to hear your thoughts on this.

@ajsutton
in the private constructor of Store class.
i have tried to initialize the votes array like the below
    this.highestVotedValidatorIndex =
        votes.keySet().stream().max(Comparator.naturalOrder()).orElse(UInt64.ZERO);

    this.votes = new VoteTracker[this.highestVotedValidatorIndex.intValue() + 1];

    votes.forEach(
        (key, value) -> {
          this.votes[key.intValue()] = value;
        });
    this.blockMetadata = blockMetadata;

I think this is correct - we need to be initialising the votes array with the votes from the map passed into the constructor and for existing databases those votes could be for every validator in the state.

upon debugging i found that, the StoreVoteUpdater.getVote() method is not always preceeded by StoreVoteUpdator.putVote()
(basically we are calling StoreVoteUpdater.getVote(index) even before that index is inserted/present in Store).

Yes that's expected.  It should return null if the vote doesn't currently exist - StoreVoteUpdater will then return VoteTracker.DEFAULT to provide an initial value.  The fork choice algorithm is always trying to update a vote, so it asks for the existing vote first and then calculates the weighting changes between the previous state and the new one, applies that to the protoarray and then puts the new vote into the StoreVoteUpdater.

so leaving the current code as it is should be fine, where we initialize the Store.votes array with size 1024.

As mentioned above we need the initial votes passed into the constructor so will need to size it correctly.

i can add an if condition in Store.getVote() to avoid this exception, shall i proceed in that way
  VoteTracker getVote(UInt64 validatorIndex) {
    readLock.lock();
    try {
      if(validatorIndex.intValue()>= votes.length)return null;
      return votes[validatorIndex.intValue()];
    } finally {
      readLock.unlock();
    }
  }

Yep, this is right - it then behaves the same way as the previous Map version where votes.get(validatorIndex) would return null if there wasn't a previous vote.

@ajsutton  i have refactored it, can you check current change.

@ajsutton  merge this once you get time


sample mycronjob file to schedule crnjobs in kubernetes

please review this in your free time

Proof of work PR. Not mergable.


Parsing API for non akka-http clients

fixes #84

Very nice! Please fix the two minor wording issues.

ok

done, please review


embedding referenced memo result in "Memo not found: memos"

Describe the bug
When adding an embedded reference the error Memo not found: memos is displayed.
To me the created reference already looks malformed. eg. ![[memos/memos/<id>]]
Context
this behavior was discovered with #3158
Steps to reproduce

create a memo
add a reference
check Use as Embedded Content

The version of Memos you're using
v.0.21.0
Screenshots or additional context

Fixed with https://github.com/usememos/memos/commit/200a0d3e2b7c595663626076166b4dc9f27c4435


[Bug]: AmazonCorretto string versions too long to cast to System.Version

What happened?
Get-EvergreenLibraryApp -Inventory $library -Name AmazonCorretto errors out due to the Corretto version numbers being too long to cast to a [Version] type.
Unless there's a way to coerce System.Version to work with longer version numbers, an easy fix would be just leaving the JDK version off of the version number since that data is already present in the JDK Version property.
Version
2305.792
What PowerShell edition/s are you running Evergreen on?
Windows PowerShell
Which operating system/s are you running Evergreen on?
Windows 10
Have you reviewed the documentation?

[X] Troubleshooting at: https://stealthpuppy.com/evergreen/troubleshoot/
[X] Known issues at: https://stealthpuppy.com/evergreen/issues/

Verbose output
VERBOSE: Input object has the required Inventory property.
VERBOSE: Filtering library inventory for 'AmazonCorretto'
Sort-Object : Cannot convert value "<IP_ADDRESS>.1" to type "System.Version". Error: "Version string portion was too short or too long."
At C:\Program Files\WindowsPowerShell\Modules\Evergreen\2305.792\Public\Get-EvergreenLibraryApp.ps1:42 char:64
+ ... .Versions | Sort-Object -Property @{ Expression = { [System.Version]$ ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidResult: (@{Version=11.0....chitecture=x64}:PSObject) [Sort-Object], RuntimeException
    + FullyQualifiedErrorId : ExpressionEvaluation,Microsoft.PowerShell.Commands.SortObjectCommand

Sort-Object : Cannot convert value "<IP_ADDRESS>.1" to type "System.Version". Error: "Version string portion was too short or too long."
At C:\Program Files\WindowsPowerShell\Modules\Evergreen\2305.792\Public\Get-EvergreenLibraryApp.ps1:42 char:64
+ ... .Versions | Sort-Object -Property @{ Expression = { [System.Version]$ ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidResult: (@{Version=11.0....chitecture=x86}:PSObject) [Sort-Object], RuntimeException
    + FullyQualifiedErrorId : ExpressionEvaluation,Microsoft.PowerShell.Commands.SortObjectCommand

Sort-Object : Cannot convert value "<IP_ADDRESS>.1" to type "System.Version". Error: "Version string portion was too short or too long."
At C:\Program Files\WindowsPowerShell\Modules\Evergreen\2305.792\Public\Get-EvergreenLibraryApp.ps1:42 char:64
+ ... .Versions | Sort-Object -Property @{ Expression = { [System.Version]$ ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidResult: (@{Version=15.0....chitecture=x64}:PSObject) [Sort-Object], RuntimeException
    + FullyQualifiedErrorId : ExpressionEvaluation,Microsoft.PowerShell.Commands.SortObjectCommand

Sort-Object : Cannot convert value "<IP_ADDRESS>.1" to type "System.Version". Error: "Version string portion was too short or too long."
At C:\Program Files\WindowsPowerShell\Modules\Evergreen\2305.792\Public\Get-EvergreenLibraryApp.ps1:42 char:64
+ ... .Versions | Sort-Object -Property @{ Expression = { [System.Version]$ ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidResult: (@{Version=16.0....chitecture=x64}:PSObject) [Sort-Object], RuntimeException
    + FullyQualifiedErrorId : ExpressionEvaluation,Microsoft.PowerShell.Commands.SortObjectCommand

Sort-Object : Cannot convert value "<IP_ADDRESS>.1" to type "System.Version". Error: "Version string portion was too short or too long."
At C:\Program Files\WindowsPowerShell\Modules\Evergreen\2305.792\Public\Get-EvergreenLibraryApp.ps1:42 char:64
+ ... .Versions | Sort-Object -Property @{ Expression = { [System.Version]$ ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidResult: (@{Version=17.0....chitecture=x64}:PSObject) [Sort-Object], RuntimeException
    + FullyQualifiedErrorId : ExpressionEvaluation,Microsoft.PowerShell.Commands.SortObjectCommand

What's the value to Filter you're using for AmazonCorretto in your EvergreenLibrary.json?

"$_.Type -eq "Msi""

So you're downloading all major releases and architectures to the library?
I've made a change in the development branch which would be worth testing


Sweep sine get more harmonics when increasing the bitrate

@WenzhengQ has reported:
I disable ltpf unconditionally  in analyse() function of the lc3.c file; output sine signal is normal;

//    if (nn_flag)
        lc3_ltpf_disable(&side->ltpf);

But I set encode bitrate=500kbps, encode the same full scale sweep sine(20hz-24khz)  and decode the bitstream;  the ouput sine signal has more harmonic at higher sinus frequency; when i set encode bitrate=128kbps, output sine has less harmonic  than bitrate=500kbps;

1.encode the wav
./elc3 -b 500000 -m 10 48khz_16bit_2ch.wav 48khz_16bit_2ch_enc_500kbps.bin
2.decode the bitstream
./dlc3 48khz_16bit_2ch_enc_500kbps.bin 48khz_16bit_2ch_enc_500kbps_dec.wav

Originally posted by @WenzhengQ in https://github.com/google/liblc3/issues/33#issuecomment-1633739950

Thank for these observations.
Note that the bandwidth is limited to 20 KHz.
I have generated a mono linear sweep 20 Hz to 20 KHz.
And done the encoding/decoding loop :
./bin/elc3 /tmp/sweep.wav -b 64000 | ./bin/dlc3 > /tmp/sweep_64k.wav
./bin/elc3 /tmp/sweep.wav -b 128000 | ./bin/dlc3 > /tmp/sweep_128k.wav
./bin/elc3 /tmp/sweep.wav -b 256000 | ./bin/dlc3 > /tmp/sweep_256k.wav

The first track is the linear spectrogram of source sweep, next it's 64k, 128k and 256k ones.
It seems you're completely right. I will look in following days.
Thanks.

@WenzhengQ, can you double check with the fix https://github.com/google/liblc3/commit/67f2231815532bb7b34a097108b1a24ff49dd9f3.
That looks ok on my side.
For the background: there is 2 quantization steps. The second step, slightly adjust the quantizer (global gain).
This gain should be limited to avoid saturation of the spectral coefficients, fixed to 16 bits by the bitstream.
This was properly done, on the adjustment phase, and select sometimes a quantizer one step to high.
And will occurs more often, when the compression ratio is lower.

@asoulier
I check with your fixed patch,  the issue is fixed;
Thanks for your help.


w-preserve-body on widget root

<div w-bind w-preserve-body>

Currently, placing preserve variations on the root of the widget will result in "Error: No widget found." I'm assuming this is because the widget has not yet been created at that point. I could see why w-preserve or w-preserve-if should always be placed on children elements, but I think that w-preserve-body and w-preserve-body-if should be allowed on the root. It makes sense semantically since the the root widget element could be rerendered while the rest of the widget body is set to be preserved.

Looks like a bug. I agree with you that w-preserve related attributes should be allowed on the root elements. I'm going to add a test case to see what I can figure out. Thanks for reporting the problem.


Document tooling verified to work with knowledge base structure produced by generator

Describe the request
There are many different knowledge base/note taking applications. Some of these use a proprietary or alternative (e.g., XML) format for the knowledge base files and thus can't be used with this generator. However, a significant number of applications use a Markdown file-based knowledge base. Many of these can be used with the generator.
In addition, there are supplemental tools that, although not necessarily specifically designed for use with knowledge bases, are useful in knowledge base projects.
It will be useful to document the specific tools that have been found to be useful and compatible with the documents produced by the generator.
Describe the current behavior
The generator user is not given any guidance in tooling options for their knowledge base.
@per1234/generator-kb-document version version
fc9a3fd0faeb27a1b2a688acf63324bb61f9b38b
Operating system
All
Operating system version
Any
Additional context
To Do

[ ] Obsidian

Document how to work around Obsidian's poor handling of the "folder per document" KB file structure

[ ] MkDocs

link to tags view configuration docs

[ ] Foam
[ ] VS Code Markdown preview
[ ] markdown-tags VS Code extension

Keywords

"Foam"

"Obsidian"

"PKMS"

"VS Code"

"Zettelkasten"

Issue checklist

[X] I searched for previous reports in the issue tracker
[X] I verified the problem still occurs when using the latest version
[X] My report contains all necessary details

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Hi @bkballoon. I'm sure the off topic comment you made is intended to distribute malware (the ZIP file having a password is a dead giveaway, as this is used to circumvent antivirus scans).
I was preparing to report your account to GitHub, but I see that you do have what appears to be legitimate activity. I also saw this comment you made in another issue:
https://github.com/libigl/libigl/issues/2409#issuecomment-2314563700

My email has been bombarded, please do not open the contents of my emails.

So I am going to give you the benefit of the doubt in case maybe your account has been hacked?
Please take immediate measures to secure your account so that it is not further used for malicious purposes. If I see any additional malicious activity from you, anywhere on GitHub, I will proceed to report your activity to GitHub.


application version does not show in banner

Application:
Spring-Boot:  (v1.5.3.RELEASE)

This is valid for all our services


Events won't send from the console

https://github.com/getsentry/raven-js/pull/1080 updated captureMessage to check whitelistUrls before sending the event. However, events typed in from a browser console have a URL of <anonymous>, so the check fails.

This has affected all releases since 1.19:

In my experience it's helpful to be able to trigger one-off events from the console to ensure that Raven is configured correctly (particularly when setting up Raven for the first time, or deploying a new version).

One workaround is to pass a string to captureException:
Raven.captureMessage('test')              // doesn't work
Raven.captureException(new Error('test')) // doesn't work
Raven.captureException('test')            // works

Hey @seanlinsley, thanks for reporting this issue and adding a "workaround", as it exposes the issue that shouldn't be there :) https://github.com/getsentry/raven-js/pull/1304/
As for the original issue, you can either add this exception to whitelistUrls, or disable them when you want to verify the communication between your app and Sentry server.
var config = {
  whitelistUrls: []
}}

if (isDevelopment()) {
  config.whitelistUrls.push('<anonymous>')
}

// or

var config = {};
if (isProduciton()) {
  config.whitelistUrls = ['yourserver.com/something']
}

Adding something like {force: true} would be too much overhead, as we'd have to pass it around everywhere, take all the other options into account and make sure to cover all the API variants.

It seems to me that commands from the console should be properly associated with the site's domain, instead of being marked as anonymous.
I'd expect to find that a lot of Sentry users rely on the old behavior. How are you supposed to be 100% sure that Sentry is working in production (w/ its usual production configuration) without being able to trigger an event from the console? Expecting users to always embed a call in their source code & then later remove it after testing is complete is too heavy-handed, IMO.

We don't have a control over V8, SpiderMonkey, Chakra or any other browser's JS engine and there's no reliable way to evaluate what's the source of such calls.
Chrome returns <anonymous> where IE returns Unknown script code. It's not only the console calls, it's the same for eval code for example.

I'd expect to find that a lot of Sentry users rely on the old behavior.

We didn't get a single report nor support ticket related to the issue you mention.

How are you supposed to be 100% sure that Sentry is working in production (w/ its usual production configuration) without being able to trigger an event from the console?

If you really need it, you can always do this in the console:
Raven._globalOptions.whitelistUrls = [];
Raven.captureMessage('test')

Closing due to inactivity. Feel free to reopen if still relevant.


More String Utilities?

https://discuss.gohugo.io/t/more-string-utilities/5410
I encounter some problems to handle strings when working with the "shortcodes" variable {{ .Inner
}}
1.How to use a escape newline character "/n" in go template ?
{{ delimit (slice "foo" "bar" "buzz") "/n" }} dosen't work as I expected to get three lines.
2.How to remove the leading newline characters "/n" ?
"/nHello world!" -> "Hello world!"
3.How to concatenate a string constant and a string variable in one step directly ?
{{ $prefixed := delimit (slice "bar" .Title) "" }}
It looks strange to do this with the function delimit. I'd prefer a new function concat()
{{ concat "bar" .Title }}
4.How to count number of lines in a string ?
"foo/nbar/nbuzz" => three lines
is there a easier way to do these ? thank you ~

You have started a thread on the forum, please do not double post. Finish that discussion, then we can maybe create some practical issues in here.

Sorry to do that , Thanks for your advice.


Added keyboardShouldPersistTaps parameter

Added keyboardShouldPersistTaps to control when the keyboard should stay visible after a tap in ScrollView
Available values for keyboardShouldPersistTaps:
'never' tapping outside of the focused text input when the keyboard is up dismisses the keyboard. When this happens, children won't receive the tap.
'always', the keyboard will not dismiss automatically, and the scroll view will not catch taps, but children of the scroll view can catch taps.
'handled', the keyboard will not dismiss automatically when the tap was handled by children of the scroll view (or captured by an ancestor).
Official Docs link: https://reactnative.dev/docs/scrollview

@NataliaLinnik can we merge this PR?

@JairajJangleSLIOT thank you for your contribution!
Sure, sorry for this delay.
I would like to publish a new version with your changes. I think, it should be done during this week.

No worries, thanks!

@JairajJangleSLIOT the changes were released. I changed the default type to "never", because I wanted to stay consistent with react native scroll view documentation.


Pr range aid fixes

Here are some fixes that should solve the remaining issues when using range aid.

Trying to follow this, I have a couple of questions.
In one commit, you handle the case of "in range aid mode and hgt mode is range", I don't really think that makes sense. Shouldn't it just be that, when using rng for hgt, the range aid flag is ignored?

@ndepal

In one commit, you handle the case of "in range aid mode and hgt mode is range", I don't really think that makes sense. Shouldn't it just be that, when using rng for hgt, the range aid flag is ignored?

That's wrong in the commit message. If you have a look at https://github.com/PX4/ecl/pull/286/commits/c382af7a7d2214b111b3dae2c7dd78f1ce0a8997#diff-4cac31c333867233bb8a08de1e812a9aR762 you can see that it just switches to baro independent on range aid. The highlighted line could/should go away.

@tumbili can you verify?

@ChristophTobler The changes make sense to me.


KAFKA-4449: Add Serializer/Deserializer for POJO

Currently, there are only build-in serializer/deserializer for basic data type (String, Long, etc). It's better to have serializer/deserializer for POJO.
If we had this, user can serialize/deserialize all of their POJO with it. Otherwise, user may need to create e pair of serializer and deserializer for each kind of POJO, just like the implementation in the stream example PageViewTypeDemo
https://github.com/apache/kafka/blob/trunk/streams/examples/src/main/java/org/apache/kafka/streams/examples/pageview/PageViewTypedDemo.java
Let's take above streams-example as an example, Serde was created for PageView as below
    final Serializer<PageView> pageViewSerializer = new JsonPOJOSerializer<>();
    serdeProps.put("JsonPOJOClass", PageView.class);
    pageViewSerializer.configure(serdeProps, false);

    final Deserializer<PageView> pageViewDeserializer = new JsonPOJODeserializer<>();
    serdeProps.put("JsonPOJOClass", PageView.class);
    pageViewDeserializer.configure(serdeProps, false);

    final Serde<PageView> pageViewSerde = Serdes.serdeFrom(pageViewSerializer, pageViewDeserializer);

If we use this POJO serializer/deserializer, the Serde can be created with only one line
Serdes.serdeFrom(PageView.class)

Closing this PR because it seems abandoned. Also note, public API changes require a KIP (cf https://cwiki.apache.org/confluence/display/KAFKA/Kafka+Improvement+Proposals)


DirectCellEditor Support

Currently direct Celleditors are not supported

We have a rule on mobile flutter clients: If direct cell editor or preferredEditorMode 0, edit should be possible....
direct cell editors are checkbox, choice
others are editors like linked cell editor, but prefEditMode 0 (= single click) is not set automatically. It's a user-defined option.
Our flutter client opens a popup in this case, but our react UI should show the linked cell editor.
Editing a whole record is already possible direct in the table?

Editing a record directly in the table is possible via double click (text, linked etc), single click is currently only supported by checkbox and choice, preferredEditorMode and directCellEditor are currently not considered.

prefEditMode = 1 means single click, 0 is double click


Set Model position attribute to the 0th index

This fixes a bug with shadow-casting on linux where the normal attribute is the first attribute in the glTF file but is compiled out of the shadow-cast shader, and so no attributes are bound to the 0th index causing the model shadows to be broken. This fix forces the position attribute to be the 0th index. It's a little weird though, because the program node only  lists attribute names and not semantics, so I had to search through the techniques to find the attribute name that is tied to the POSITION semantic.

I'm not sure about this approach.  There's two ways I think we could make it simpler.

Instead of looking at the semantic, just look at the name with a case insensitive search for position.  In practice, it will be reliable and is much more clean.
Or, as a preprocess (at load time), walk the techniques, find the POSITION semantic and then augment the program object (then delete this after load) with the attribute index.

Updated using the first approach.

Sorry, late request: add a comment about why this is happening and update CHANGES.md if needed.

Updated.

Looks good!