Would serde support be possible?

Offering serde support by adding a serde feature seems to be quite common for data-type crates and it makes them more useful downstream as one can skip conversions just to gain serde support.
Would that be possible here as well?

On another note, I switched gix-attributes from kstring to yarn, primarily because yarn offers both byte-string and UTF8-string support. Even though it took a moment to understand that string-inlining is explicit, I came to like this differentiation (and control) that comes with it, and the reward was unexpected even: an important data structure that carries a lot of (hopefully inlined) strings shrunk by ~25%, and the performance of a git-attribute heavy workload increased by 2%. I will definitely look around and try to use it in more crates.
Thank you! (see PR)

Hi!
That's really awesome work! I'm glad my work here has provided a performance improvement.
As for serde support, I will take a pass on that. The serde author has shown themselves to be both an irresponsible programmer and a shitty person, as recorded here: https://dragon.style/@pyrex/111003848107863932. I do not want to endorse their work or support their ecosystem.
I also think cfgs of this type are evil, which I have been meaning to write about at some point.
However, serde does provide a way for threading custom serialization functions, so closing this request does not actually block anyone's work. Does it mean some people won't use my library? Sure, but that's not what I'm optimizing for. :)

I looked at your change and wanted to point out some tips (which maybe suggest documentation improvements on my end).'
Where you did
Yarn::inlined(name).unwrap_or_else(|| name.to_string().into_boxed_str().into()),

it is sufficient to do
Yarn::new(name).immoralize()

Internally, yarn makes its best effort to inline everything, because:

Copying two words' worth of memory is free; your libc's memcpy has special cases for this.
Freeing memory sooner is better, because it makes copies free.
It simplifies internal book-keeping and is necessary to enable certain layout optimizations.

It may be worth having a Yarn::new_copied() factory as a shorthand, since I've found this to be a common pattern.

Thanks for letting me know and for sharing the link for extra context - I hadn't read that yet and found it very interesting. Indeed I also thought for a moment what it would mean to get rid of serde, thiserror, anyhow and syn and… I had to give up :/.
But I also have hope that capable people will step up and provide alternatives eventually.
With that said, thanks a lot for your suggestions! I tried it but was surprised by the error produced by this patch:
diff --git a/gix-attributes/src/state.rs b/gix-attributes/src/state.rs
index 585e05adc..9e01d0950 100644
--- a/gix-attributes/src/state.rs
+++ b/gix-attributes/src/state.rs
@@ -49,10 +49,7 @@ impl<'a> From<ValueRef<'a>> for Value {

 impl From<&str> for Value {
     fn from(v: &str) -> Self {
-        Value(
-            ByteYarn::inlined(v.as_bytes())
-                .unwrap_or_else(|| ByteYarn::from_boxed_bytes(v.as_bytes().to_vec().into_boxed_slice())),
-        )
+        Value(ByteYarn::new(v.as_bytes()).immortalize())
     }
 }

51 |     fn from(v: &str) -> Self {
   |             -  - let's call the lifetime of this reference `'1`
   |             |
   |             `v` is a reference that is only valid in the associated function body
52 |         Value(ByteYarn::new(v.as_bytes()).immortalize())
   |               ^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |               |
   |               `v` escapes the associated function body here
   |               argument requires that `'1` must outlive `'static`

It took a moment to realize that ByteYarn = YarnBox<'static, [u8]>. But from there it worked. I wonder if this kind of issue can be prevented by documentation as well, but am not sure.
Here is the PR with the change and some performance data: https://github.com/Byron/gitoxide/pull/1045

If playing the pointless benchmark game amuses you, you should consider trying 0.3, where I've changed the core of Yarn::inlined() to reduce memory traffic (by avoiding a memcpy@plt call). Requires a newer toolchain though (1.71).

Thanks for the head's up! It's unfortunate that v0.3 will be out of reach for a while as gix MSRV is bound helix-editor which is bound to firefox 😁.

If playing the pointless benchmark game amuses you [..]

I doubt that this change will be measurable, but I will at least run the hyperfine test to assure there isn't any unforseen regression. hyperfine benchmarks with real-world workloads on real repositories plus occasional profiling have been very useful to me (and it's the reason I know byteyarn is great), but if there is anything less pointless I am all ear for sure :).

It may be worth having a Yarn::new_copied() factory as a shorthand, since I've found this to be a common pattern.

+1
I use yarn mostly this way


Attestation Service Opt-in Metrics using Prometheus

Use the common opt-in pipeline that Conner has created to get better visibility into the 3rd party AS.

Sync with Conner to get an understanding of the pipeline readiness
Use Prometheus endpoints to collect metrics
Build dashboards from these metrics

I'd love for us to get to this, especially since it sounds like on the cLabs side we are making progress on Grafana

De-prioritizing in favor of ASv2


docs: add mill-scip to external plugins

It's a bit unclear, how to use this plugin, as the added documentation only speaks about thirdparty tools like scip-java and cs.
Is there also a way to run this plugin via mill? If not, you may want to add some info, that it is some kind of integration tooling, but not directly usable by Mill users.

Sure I added the instructions to also use it directly via mill. I didn't include it originally because I assumed that most user will use this in CI and doing:
scip-java index

Will get you what you want (using Mill under the hood) without needing to worry about the version or the syntax since it takes care of it all for you. Even locally it's easier to use this way, but I get your point. I added the section in on manual usage.


Anagram checker algorithm in Kotlin

I would like to contribute a kotlin solution to the Anagram Checker algorithm!

Thank you, working on it!

I think someone already added the solution to it, so I'll add solution to some other issue

So I have added a solution to the Generate Parentheses problem!


Replaces minilp with microlp

Could you enable issues in the microlp repo ?

If you are motivated to maintain and improve microlp in the long run, a good first step could be to add support for MILP (add integer variables). The original author already made an implementation using branch & bound, but they only put in in the exemples folder instead of integrating it to the library itself: https://github.com/Specy/microlp/blob/4dc2acc643689730a8451ba9bb303c6437bb16f3/examples/tsp.rs
Integer problems are very common and a tiny implementation like the one in minilp would be very welcome for embedded use cases, such as the wasm.

Sure, also watchout, I hadn't yet added the possibility to specify both minilp and microlp feature flags in the PR

I am adding it

If you are motivated to maintain and improve microlp in the long run

I would work on it, my only issue is lack on knowledge in OR, I only have university level knowledge of it, I don't feel like I'm knowledgeable enough in the field to work on it properly, but branch and bound could be doable and I'd love to have MILP in the library. Do make an issue on the repo to keep track of it


Uncaught Error: Cannot call function 'getPosition' on missing remote object 382Error: Cannot cal...

Had an existing project
Opened new file.

Not always reproducable, only occurs very occasionally.
Atom: 1.33.0 x64
Electron: 2.0.11
OS: KDE neon
Thrown From: Atom Core
Stack Trace
Uncaught Error: Cannot call function 'getPosition' on missing remote object 382
Error: Cannot call function 'getPosition' on missing remote object 382
at throwRPCError (/snap/atom/207/usr/share/atom/resources/electron.asar/browser/rpc-server.js:143:17)
at EventEmitter. (/snap/atom/207/usr/share/atom/resources/electron.asar/browser/rpc-server.js:354:7)
at emitMany (events.js:147:13)
at EventEmitter.emit (events.js:224:7)
at WebContents. (/snap/atom/207/usr/share/atom/resources/electron.asar/browser/api/web-contents.js:295:13)
at emitTwo (events.js:126:13)
at WebContents.emit (events.js:214:7)
At /snap/atom/207/usr/share/atom/resources/electron.asar/renderer/api/remote.js:221

Error: Cannot call function 'getPosition' on missing remote object 382
Error: Cannot call function 'getPosition' on missing remote object 382
    at throwRPCError (/snap/atom/207/usr/share/atom/resources/electron.asar/browser/rpc-server.js:143:17)
    at EventEmitter.<anonymous> (/snap/atom/207/usr/share/atom/resources/electron.asar/browser/rpc-server.js:354:7)
    at emitMany (events.js:147:13)
    at EventEmitter.emit (events.js:224:7)
    at WebContents.<anonymous> (/snap/atom/207/usr/share/atom/resources/electron.asar/browser/api/web-contents.js:295:13)
    at emitTwo (events.js:126:13)
    at WebContents.emit (events.js:214:7)

    at Object.exception (/snap/atom/207/usr/share/atom/resources/electron.asar/renderer/api/remote.js:221:30)
    at metaToValue (/snap/atom/207/usr/share/atom/resources/electron.asar/renderer/api/remote.js:225:28)
    at Object.remoteMemberFunction (/snap/atom/207/usr/share/atom/resources/electron.asar/renderer/api/remote.js:125:16)
    at AtomEnvironment.getWindowDimensions (/snap/atom/207/usr/share/atom/resources/app/static/<embedded>:1:642179)
    at AtomEnvironment.storeWindowDimensions (/snap/atom/207/usr/share/atom/resources/app/static/<embedded>:1:642594)
    at WindowEventHandler.handleWindowBlur (/snap/atom/207/usr/share/atom/resources/app/static/<embedded>:11:286062)

Commands
     -0:49.5.0 find-and-replace:show (input.hidden-input)
     -0:49.3.0 core:paste (input.hidden-input)
     -0:49 core:confirm (input.hidden-input)
     -0:47.6.0 pane:show-previous-item (input.hidden-input)
  2x -0:47.1.0 pane:show-next-item (input.hidden-input)
     -0:46.6.0 find-and-replace:show (input.hidden-input)
     -0:46.5.0 core:paste (input.hidden-input)
     -0:46.3.0 core:confirm (input.hidden-input)
  2x -0:40.7.0 pane:show-previous-item (input.hidden-input)

Non-Core Packages

Thanks for contributing!
We noticed that this looks like a duplicate of https://github.com/atom/atom/issues/16630 so you can subscribe there for updates if you'd like (we believe that this will be fixed in a future Electron upgrade).

Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Align center on figure does not respect width setting

Describe the bug
I have the following rst (I can create a standalone example if that helps):
.. figure:: some-image.svg
    :width: 400px
    :align: center

I would expect that the resulting HTML renders the images centred in the main column, but with the requested width. Instead, the image takes half of the main column width (352px in my case), regardless of the requested width.
Desktop (please complete the following information):

OS: macOS 14.6
Browser: Chrome & Firefox & Safari all render the same

Thanks for reporting this!
A standalone example would be super helpful!

Here is a standalone reproduction of this bug:
furo-bug.zip
Using furo, the image is quite small:

While using alabaster, it takes the full width:

Okay, I love this reproducer. You definitely made my day a lot better with this. ^>^

1ff0ae277d3a337ac4b28b9995dcc16827403d96 should fix this! I'll cut a release over the weekend. :)

thanks a lot!


#122 Added 5 New Algorithms under String Algorithms

📥 Pull Request
Description

Summary of Change: Implemented 5 algorithms under the String Algorithms section:

Apostolico–Giancarlo Algorithm
BNDM (Backward Nondeterministic Dawg Matching) Algorithm
Bitap Algorithm (Approximate String Matching)
Commentz-Walter Algorithm
Shift-Or Algorithm

Issue Fixed: Resolved issue #122  by adding the necessary algorithms to the string algorithms section.

Motivation: The changes enhance the algorithm coverage and provide detailed implementations for each string matching algorithm.

Context: Detailed comments were added where necessary to improve clarity and code understanding.

Dependencies:  No new dependencies introduced.

Fixes #122
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update

Checklist:

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

@ajay-dhangar  Please add the required labels GSSOC Ext and Hacktoberfest . Thanks....

@ajay-dhangar Can you please review it and add GSSOC Ext and Hacktoberfest labels to it . Thanks......

@ajay-dhangar I have made the changes that you requested . Please check once...

@pratheekv39, you need to sync our latest code. I've resolved the error, so first, sync with git pull origin main, push the code to your branch, and then resolve any remaining errors.

@ajay-dhangar I have resolved and updated (synced) my branch and pushed the code. Can you now approve the PR???


Music Assistant doesn't show a song is paused on the frontend

What version of Music Assistant has the issue?
2.0.4
What version of the Home Assistant Integration have you got installed?
2024.5.1
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
Music Assistant's frontend is slow when it comes to updating information from a player. For example, if I pause a track, it pauses straight away on the player but on the frontend UI it still shows that the song is playing. I can't press play again either after this happens. I've also experienced behaviour where I will play an album and the first song from the album will start playing but the frontend UI shows that the second song is playing and that the first song has already been played. Sometimes this fixes after about 1 minute to show the correct playing with the timestamp.
How to reproduce

Play any song or radio
Press pause
Music then pauses but doesn't show that its paused on the frontend.

Music Providers

Spotify
Local and Remote shares
RadioBrowser

Player Providers

Home Assistant Media Player

Full log output
log.txt
Additional information
All my media_player entites that are coming from home assistant are Sonos speakers that have been added to HA through the Sonos integration. Maybe this is the issue, show I remove the Sonos integration from HA and use the Music Assistant one instead?
Also I'm using flow mode because speakers would only play 1 track and then stop if trying to play a queue of songs.
What version of Home Assistant Core are your running
2024.5.1
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Raspberry Pi

Please only report one issue per report.
It is not clear how you are pausing the player if not through the MA UI

Please only report one issue per report.
It is not clear how you are pausing the player if not through the MA UI

I am pausing it both through the MA UI and through home assistant media player card and they both give the same behaviour

You should always prefer the MA native player providers over the HA media players. The HA mediaplayers are a last resort if no native player is available in MA but they lack features and often have bugs regarding playback.

I will try the Sonos providor then and see if it resolves any issues.

Using MA Sonos providor has fixed both play/pause and the skipping track issue.


fix CVE for tdp branch

What is the current behavior? (You can also link to an open issue here)
fix CVE for:
Jettison 1.1
Protobuf 3.21.2
Jackson 2.13.3
Woodstox 5.3.0
netty-codec-haproxy:4.1.79.Final]
What is the new behavior?
CVE disappear
Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[x] Other... Please describe: CVE fix

Does this PR introduce a breaking change?

[ ] Yes
[x] No

https://jenkins-connectors.datapwn.com/job/components/job/build/job/wwang-talend%252FCVE-tdp/


JAMES-2793

Hi,
This pull request has just been merged.
Thank you again for your dedication toward the James project. If you need help, want to contribute documentation, new features, tests or bugfix, please don't hesitate to reach us.
One very last thing: I don't have write access to this repository, hence I can not close this issue. Wouldn't you mind doing it for me? It would avoid me annoying the Apache INFRA team with such simple concerns.
Thanks again,
Cheers,
Benoit

Benoit,
My pleasure. It is my anticipation that I will do more to contribute to this project, as I push through the learning curve.
As far as "close the issue", do you mean in GitHub? Do I just hit the "Close Pull Request"?
Thanks,
Brent

I mean 'close the pull request', sorry...

Closing the pull request per Benoit's direction.


Year value always starts at 1900 if 2 digits for the year is used

Bug, feature request, or proposal:
Year value always starts at 1900
What is the expected behavior?
Set a certain threshold like -/+ 50 of the current year to be the value when 2 digits for the year is used
What is the current behavior?
It always starts at 19+{2 digit value}
What are the steps to reproduce?

Open a date picker
Type in 1/1/1
Value is Jan 01 1901 13:15:14 GMT-0500 (EST)

What is the use-case or motivation for changing an existing behavior?
Angular Material Date Picker does this.
Which versions of Angular, MD2, OS, browsers are affected?
Latest version
Is there anything else we should know?

Same here. Only an issue if you amend the date manually by typing. If you use the graphical picker this bug doesnt happen

same here,
is there any update till now ?


Deleting cell should have confirmation before removing cell

Having just accidentally deleted a cell that had been running for over 4 hours, I'd like to suggest that there be a confirmation popup for running Jupyter cells to be deleted. Ideally there should be an option to confirm for all cells, but should be a default for cells that are currently executing.

Would take a PR to add a confirmation dialog to delete running cells, with a setting to disable this confirmation step

Would take a PR to add a confirmation dialog to delete running cells, with a setting to disable this confirmation step

Should this check happen in the action runWithContext or in the action itself?

Looks like that's just a function that's called in one place, and I think changing either is fine

i will go and deal this


Ruby 2.2.0

With ruby 2.2.0, bundle install failed because of nio4r, kgio, and raindrops so updated them.

:+1:


allow overriding the default grace period of 45 seconds

This allows users to configure a faster restart of their
webhook if desired while retaining the current behavior.

Changes

:gift: allow users to change the webhook drain timeout

/kind enhancement
Release Note
webhook.Options now contains a GracePeriod parameter to allow modifying the http server drain time from the default of 45 seconds.

/ok-to-test

/approve


remove experimental_string_processing

that flag is no longer support…ed in black > 24.1
fixes #30

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

Comparison is base (ac80923) 100.00% compared to head (b25139b) 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #31   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines           38        38
  Branches         5         4    -1
=========================================
  Hits            38        38

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


DEV: add Cython generated files to gitignore

To make sure they do not accidentally get re-committed.
Dropped the checklist because this is not a code change and none of them were applicable.
Also 🎉 on removing the generated Cython output from source control!

Oh, I see this is done as part of #375 , sorry for the noise.

Thanks Tom! 🙏
No worries. Should be addressed now. Though if you spot anything else, please let us know 🙂

I only noticed this because a git merge failed in https://github.com/tacaswell/build_the_world and I was poking around to see why.


Support EditorConfig and document it in CONTRIBUTING.md

Finally merged in master and 2.x, with minor changes. Thanks a lot!


Add a way to expose Bootstrap as only mixins/placeholders, no classes

In order to use bootstrap, all the code of a given module has to be present in the final CSS. In current versions, it generates a lot of collisions and the need to make up confusing names for my own classes.
An example
If I want to use .card as a base for a class of my own (@extending it), the whole CSS for .cards would end up in the final CSS together with my implementation.
It also brings concerns regarding existing classes. eg: .card could be a semantically significant element in my page.
An idea on how to solve it
Sass enables us to create silent classes, that are not expressed in the final CSS. This could be used (as an opt-in; just like the current flex-grid) to use Bootstrap as a mixin library.
Bootstrap:
// bootstrap/scss/_card.scss
%card {
  // all the declarations for .card elements (current code)
}
@if not($silent-classes) {
  .card { @extend %card }
}

Custom SCSS
@import "bootstrap/scss/card";

.projects {
  // class definition for the wrapper
  &__item { @extend %card }
}

This is not a perfect solution as it would still generate child elements with the original names (although this already happens when using mixins like card-inverse) and would force the user to @extend a lot of classes (like .card-block by doing a .projects__item-content { @extend %card-block }).
That said, as far as I can think, this would not change anything when using bootstrap as is, but it would give a lot more power for experienced developers who want to use Bootstrap together with their existing codebases.
What do you guys think about this?
Any better ideas on how to achieve this?
Would this change be welcome?

@cvrebert thank you very much for rephrasing the ticket! it's much clearer now! :smile:
Btw, any ideas on how to move forward with this?
I'm afraid it gets buried before it gets even looked at.

Not going to go this route, sorry. It'd be a pretty big departure from the current approach. Sorry!

Too bad @mdo 😔
I love your approche @felds! Great idea!
My components now are looking  way. Unfortunately not with bootstrap anymore  😔
I just set silent-classes to true in every component.
@import "variables";
$silent-classes: true;
@import "atomic";

#header {
  @extend %display-flex;
  @extend %align-items-stretch;
  &__logo {}
  &__title {
    @extend %align-items-end;
    @extend %flex-1-0;
    &--headline {
      @include font(20px,20px);
    }
    &--subline {
      @include font(16px,16px);
    }
  }
  &__navigation {
    @extend %display-flex;
    @extend %align-items-end;
    ul {
      @extend %display-flex;
      @include listreset();
      li {
        span {
          @include font(14px,14px);
        }
      }
    }
  }
}

@janwagner I'm glad it was useful to someone! <3


refactor: rename debug executor to wrap executor

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
As title. Because "debug" sounds misleading - it not only works in debug build.
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Documentation
None
Refer to a related PR or issue link (optional)
None

"Wrap" sounds more confusing...  ExecutorWrapper, IntrumentedExecutor, WrappedExecutor, etc

"Wrap" sounds more confusing...  ExecutorWrapper, IntrumentedExecutor, WrappedExecutor, etc.
(Nevermind, I dont have better name either.

How about WrappingExecutor? Would it be better?

Or simply CheckExecutor? All LGTM. 🤔

Renamed to "WrapperExecutor", which is the best name I could think of.

@mergifyio requeue


RDKPI-330: Include ControlService, RemoteActionMapping in RPi Builds

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

Will resubmit


[T2A4][W11-A2]Ho Yi Hang

Ready for review

Good job.. Keep up the good work.

I don't think it's good to change the code like below
     * ==============NOTE TO
     * STUDENTS=================================================================
      * =====


Props Proposal: Destructure aria- & data- props on Card component article element

Regarding the Card component, it doesn't de-structure aria- (or data-) props (like the way Button does for example).
Since the Card renders an article which shows up in the Landmarks menu (on screen reader on MacOS at least) it would be great to be able to give the element a more accessible label.
I'm happy to implement & submit a PR for this!
I think it would be sufficient to do something like this, but let me know:
// ...
import getAriaProps from '../../utilities/get-aria-props'; // <-- added line
import getDataProps from '../../utilities/get-data-props'; // <-- added line
// ...
const Card = (props) => {
	const ariaProps = getAriaProps(props); // <-- added line
	const dataProps = getDataProps(props); // <-- added line
	// ...

	return (
		<article
			id={props.id}
			className={classnames('slds-card', props.className)}
			style={props.style}
			{...ariaProps} // <-- added line
			{...dataProps} //  <-- added line
		>
			{!props.hasNoHeader && (
				<Header
					header={props.header}
					headingId={headingId}
					icon={props.icon}
					filter={props.filter}
					filterId={filterId}
					heading={props.heading}
					headerActions={props.headerActions}
					headerActionsId={headerActionsId}
				/>
			)}
			{!empty ? (
				<Body id={bodyId} className={props.bodyClassName}>
					{props.children}
				</Body>
			) : (
				<Body id={bodyId} className={props.bodyClassName}>
					{empty}
				</Body>
			)}
			{props.footer ? <Footer>{props.footer}</Footer> : null}
		</article>
	);
// ...

cc @interactivellama

Looks good! Please proceed. Be sure to mention the props in the class comments, since there are no propTypes.
We have some release pipeline blockers right now for NPM publishing, so I can't promise a date, but if we can't hit it, self publish on a Git tag is an option until the pipeline issue can be mitigated.

@interactivellama Added https://github.com/salesforce/design-system-react/pull/3071 to address this.


Downgrade android gradle plugin to version 7.2.1.

Version 7.3.0 changes the build layout which causes updates to the generated shared libraries to not be picked up.

I feel like AGP breaks compat every minor version, is that expected?
How could we better test for this to prevent future breakage? I remember we also had issues upgrading to 7.2.1 for a long time.

This partially reverts #75203 Is #74646 affected?

@KoBeWi No it's not affected; the gradle version remains the same, it's the Android gradle plugin version that is updated.

I feel like AGP breaks compat every minor version, is that expected?
How could we better test for this to prevent future breakage? I remember we also had issues upgrading to 7.2.1 for a long time.

Given our tooling doesn't fully match with their expected tooling (i.e: scons instead of cmake), yes there are some expectations of minor breakages when they change their underlying infra.

Thanks!


AI Projects JS SDK

AI Projects with Agent capabilities more details here  https://learn.microsoft.com/en-us/azure/ai-studio/how-to/create-projects?tabs=ai-studio
Packages impacted by this PR
New package ai-projects
Issues associated with this PR
Describe the problem that is addressed by this PR
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[ ] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

API change check
APIView has identified API level changes in this PR and created following API reviews.
@azure/ai-projects

Also looks like we have some 404s from README links if you check the Build Analyze results

Ya, I think it's because the README is written with windows-style separators which are invalid URLs:

I would expect sampes/v1-beta/javascript/README.md generated by dev-tool if publishing from samples-dev workflow is used.  Is this not the case?


How to hover over node and change text background

I am trying to set a :hover style for my nodes to change the fill color.  But when my mouse hovers over the node's text, the rectangle background goes back to its non-hover state.  My style is defined as:
svg { border: 1px solid #ccc; overflow: hidden; margin: 0 auto; }
.node rect { stroke: black; fill: none; }
.node rect:hover { fill: lightgray; stroke-width: 3px; }
.node text:hover { fill: darkblue; stroke: darkblue; }
When the mouse is over the rectangle's border, the entire rectangle background turns lightgray, but the text remains the same color.  When I move the mouse over the node's text, the color changes to darkblue, but the rectangle's lightgray background color goes away.  And there seems to be a "dead zone" between inside the rectangle border and the start of the text area, where the entire style goes to its non-hover state.
All I want to do is change the "overall" style of the node when I hover over it:  make the the rectangle border and text "fatter" and a different color, and change the background color of the node.  What am I missing?

I think you may want to add pointer-events: none to the node label.


Phoebemirman/ch11992/claim reporting eth needs to be formatted

https://app.clubhouse.io/augur/story/11992/claim-reporting-eth-needs-to-be-formatted
https://app.clubhouse.io/augur/story/14383/core-stats-bar-should-show-four-decimals-instead-of-scientific-notation

Pull Request Test Coverage Report for Build 956

7 of 7 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 65.9%

Totals

Change from base Build 946:
0.03%

Covered Lines:
3587

Relevant Lines:
4934

💛  - Coveralls

Pull Request Test Coverage Report for Build 956

7 of 7 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 65.9%

Totals

Change from base Build 946:
0.03%

Covered Lines:
3587

Relevant Lines:
4934

💛  - Coveralls


how to set ignore_eos when benchmark TensorRT LLM

System Info
as titled general question
Who can help?
No response
Information

[ ] The official example scripts
[ ] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
N/A
Expected behavior
N/A
actual behavior
N/A
additional notes
N/A

Due to the lack of response for a long time, this issue will be closed for now. It can be reopened later if necessary. Thanks.


Layout problem

I found that what ever how you setup Susy, even in youtube/web tutorial, when open debug image, there is always a little gap there in the right-hand side, but when you slowly resize the browser, it will disappeared, and come again like a loop, is it caused by the gutter setting? because the ratio cannot be divided into integer

Yes, this is a known browser bug in handling sub-pixel rounding on background-images as noted in the documentation. This is why we are considering an svg alternative in #440.

In chrome it's kind of unpleasant to debug. I used the following code to draw the grid for the whole 100% and then it looks perfect:
$susy: (
  columns: 7,
  gutters: 1/6,
  math: fluid,
  output: float,
);

@mixin grid-column-gradient($values...) {
  background-image: linear-gradient(to right, #{$values});
}

@function gradient-stops($grid-columns) {
  $transparent: rgba(0,0,0,0);
  $color: rgba(0,0,0,0.07);

  $column-offset: 0;

  $values: ($color 0);

  @for $i from 1 through $grid-columns {

    @if $i != $grid-columns  {
      $column-offset: $column-offset + span(1);
      $values: append($values, $color $column-offset, comma);
      $values: append($values, $transparent $column-offset, comma);
      $column-offset: $column-offset + gutter();
      $values: append($values, $transparent $column-offset, comma);
      $values: append($values, $color $column-offset, comma);
    } @else {
      $column-offset: $column-offset + span(1);
      $values: append($values, $color $column-offset, comma);
      $values: append($values, $transparent $column-offset, comma);
    }
  }

  @return $values;
}

.container {
	/* mark */
	@include grid-column-gradient(gradient-stops(7));
}

Based on that code:
_visual-grid.scss

@signalwerk That's how we used to do it, but we had exactly the same problem with that technique as our current one (and a lot more code). Chrome must have fixed the one bug, and not the other? Very strange. I'll re-open this for a patch.


Parse mix of input files and directories

Is your feature request related to a problem? Please describe.
While using the compiler, I came across the need to input both files and directories as inputs.
Describe the solution you'd like
Need to modify input list to support both

Added to v0.0.9, changed behavior of parse-files to parse files and folders. Removed parse-directories.


Use Different Rickroll link.

I found a flaw in the redirect. The flaw is that it uses the official music video, which contains ads. These ADs can allow the person realize that they are about to be rickrolled and close the video. I found this HD rickroll with no ads that you should use instead.
https://www.youtube.com/watch?v=xMHJGd3wwZk

This is no longer an issue as the new website no longer uses a YouTube video to Rick Roll people. Instead, a gif is used on a webpage.
I'm happy to see people are interested in the project though :)


Upgrade dioxus to 0.5

@marc2332 My apologies here. I had only intended to use this internally and quickly scraped something that was working for my intended purposes. I should have cleaned it up a bit before opening this PR. I am a bit of a rust noob and am running into an error that I cannot figure out how to resolve when attempting to use the custom HelmetProps with the Drop trait. Maybe this will seem obvious to you?
cannot move out of type `HelmetProps`, which implements the `Drop` trait
move occurs because `new_clone.children` has type `std::option::Option<dioxus::prelude::VNode>`, which does not implement the

Hmmm, I see, it actually doesn't make sense to implement Drop in the Props anyway. I will try to update it according the new dioxus 0.5 apis asap 👍🏻  No worries


New version: 360.360SE version 15.1.1366.64

This pull request is automatically generated by 🥟 Dumplings in #2918
Microsoft Reviewers: Open in CodeFlow

/AzurePipelines run

@wingetbot run

@wingetbot run


Bring in latest prost changes, fix build with nightly Rust

   error: failed to load source for dependency `prost`

    Caused by:
      Unable to update https://github.com/def-/prost#1dfee4ba

    Caused by:
      failed to update submodule `prost-build/third-party/protobuf`

    Caused by:
      failed to parse url for submodule `prost-build/third-party/protobuf`: `git@github.com:protocolbuffers/protobuf`

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 5 committers have signed the CLA.:white_check_mark: def-:x: tottoto:x: 3541:x: BKDaugherty:x: adpaco-awsYou have signed the CLA already but the status is still pending? Let us recheck it.


[GLUTEN-6724][CH] Shuffle writer supports compression level configuration for CompressionCodecFactory

What changes were proposed in this pull request?
ShuffleWriter and PartitionWriter do not support compression level configuration in Clickhouse. Meanwhile, CompressionCodecFactory supports setting compression level. Therefore, shuffle writer supports compression level configuration for CompressionCodecFactory in Clickhouse backend.
(Fixes: #6724)
How was this patch tested?
CI.

Ping @taiyang-li, @baibaichen.

@zhouyuan, PTAL.

CC @marin-ma

@marin-ma, PTAL.


Excessive GPU memory usage with dynamic shape input using Gluon interface

Description
When using threaded engine with dynamic shape input. MXNet uses excessive amount of GPU memory. In the following script, MXNet is only able to reach a batch size of 16 while TensorFlow, PyTorch and MXNet w/ NaiveEngine can reach a batch size of 256. Such behavior prevents me from training resnet101 fpn bn mask rcnn model even on 32GB of GPU memory. The same behavior is observed on hybridized model as well
To Reproduce
The following script create a dynamically shaped dataset, dataloader, padding batchify, and a simple neural network. It train the network with exponentially increasing batch size. For every epoch, the batch size is multiply by 2.
https://gist.github.com/Jerryzcn/bc300b431f4c2868158f3a309dc44e78
Environment
AMI
Deep Learning AMI (Ubuntu 18.04) Version 26.0 - ami-010a96c958f9ee5cf
Instance Type
p3.16xlarge
Storage
GP2 volume with 2000GB capacity
Numpy Environment
None
MXNet Environment
MXNet(+Keras2) with Python3 (CUDA 10.1 and Intel MKL-DNN)
source activate mxnet_p36
Pytorch Environment
PyTorch with Python3 (CUDA 10.1 and Intel MKL)
source activate pytorch_p36
Tensorflow Environment
for TensorFlow 2(+Keras2) with Python3 (CUDA 10.0 and Intel MKL-DNN)
My Benchmark:
GPU Stress Test (higher is better)

Test
Max Batchsize Reached
Max Throughput Reached (samples/sec)

MXNet Random
8
335.412

MXNet Sorted
16
659.268

PyTorch Random
256
354.547

PyTorch Sorted
256
564.02

TensorFlow 2.0 Random
128
267.031

TensorFlow 2.0 Sorted
128
471.632

MXNet Optimized Random
8
357.711

MXNet Optimized Sorted
16
685.75

MXNet NaiveEngine Random
512
295.208

MXNet NaiveEngine Sorted
256
409.581

Assigning @drivanov totake a look at this issue.

I created a special MXNet tool for the collection of memory consumption statistics. Using this tool, I got the following memory allocation snapshot for this stress test.

As we could see here, the memory actually used during the stress test for each batch size looks OK (see red and very tiny blue parts of the graph). But memory, which reserved in the cache (shown in yellow), grows exponentially with the batch size and for some reason, it is not released.
I am not sure if it is the only problem with this stress test. When the program crashes, I could see some error messages regarding some multithread issue in Python.
I will continue to work on that problem.

Adding nd.waitall() resolves the issue, but with the penalty of reduced throughout.

This is a problem of having memory cache outside of memory manager and memory fragmentation. You can try mx.gpu(0).empty_cache() to alleviate the problem.

@Jerryzcn, @szha. Thanks so much for your suggestions! I tried to use them, and this is what I saw in my experiments.

I did not notice any advantage of using mx.gpu(0).empty_cache(). On the contrary, sometimes I saw new crashes with the error message associated with this call.
Yes, I saw about 25-30% slowdown associated with using mx.nd.waitall(). Unfortunately, it does not solve the initial GPU memory problem.
What makes this script approx. 30% faster is the usage of del data_loader after internal loop in

for epoch in range(10):
        logger.info('Current batch size is: %d' % batchsize)
        os.system('free >> freeMemory.txt')
        data_loader = DataLoader(dataset, batch_size=batchsize,
                                batchify_fn=Tuple([Pad(pad_val=0), Pad(pad_val=0)]), num_workers=8)
        mx.gpu(0).empty_cache()
        btic = time.time()
        etic = time.time()
        for i, (src_data, dst_data) in enumerate(data_loader):
             . . .
            # Begin to update the parameter
            trainer.step(batchsize)
            . . .
        del data_loader
        batchsize *=2

Here is some evidence for this statement.
Initial version launched on datasetSize = 1000:
[Epoch 0], Speed: 95.212 samples/sec
[Epoch 1], Speed: 186.922 samples/sec
[Epoch 2], Speed: 312.916 samples/sec
[Epoch 3], Speed: 434.118 samples/sec
[Epoch 4], Speed: 405.835 samples/sec
[Epoch 5], Speed: 222.373 samples/sec
[Epoch 6], Speed: 18.799 samples/sec
[Epoch 7], Speed: 88.688 samples/sec
[Epoch 8], Speed: 371.118 samples/sec
[Epoch 9], Speed: 75.738 samples/sec
 1.993E+02 GPU stress test elapsed time

With del data_loader on the same dataset:
[Epoch 0], Speed: 93.743 samples/sec
[Epoch 1], Speed: 180.014 samples/sec
[Epoch 2], Speed: 298.007 samples/sec
[Epoch 3], Speed: 405.286 samples/sec
[Epoch 4], Speed: 527.154 samples/sec
[Epoch 5], Speed: 633.825 samples/sec
[Epoch 6], Speed: 30.708 samples/sec
[Epoch 7], Speed: 64.030 samples/sec
[Epoch 8], Speed: 207.408 samples/sec
[Epoch 9], Speed: 140.913 samples/sec
 1.368E+02 GPU stress test elapsed time

@andrei5055 thanks for reporting back your finding. Sorry that I missed the part where you recreate the dataloader multiple times. It seems evident now that the problem is starvation and thrashing of the many processes that the dataloader construction creates. However, it's unclear why explicit deletion of dataloader helped, since the object is supposed to be reference counted and freed in the next iteration.

There are reference leaks in the Gluon data API. See the tests marked with garbage_expected in https://github.com/apache/incubator-mxnet/blob/c3aff732371d6177e5d522c052fb7258978d8ce4/tests/python/unittest/test_gluon_data.py
These tests were marked as garbage_expected as I didn't have time looking into the root cause of their reference leaks yet. Given the impact noted by @andrei5055 , this warrants further investigation.

@andrei5055 as I fixed a few other reference leaks in the Gluon API, could you help verify if the del still helps improve performance on latest master?

cc @zhreshold

@leezu: I tested the changes you made for PR 18328 by launching the same script 3 times with and without del data_loader. I still see that the program runs faster when del data_loader is used. The average run times for 1 test are
(a) with del:        (77.0 + 75.8 + 80.6) / 3 = 77.8 sec
(b) without del:  (83.2 + 84.7 + 82.5) / 3 = 83.46 sec.
So, with your changes, the relative difference has been reduced from 45.7% to 7.3%.

Thank you for testing @andrei5055. That's encouraging. If you like, it would be great to test if https://github.com/apache/incubator-mxnet/commit/0210ce2c136afaa0f57666e5e1c659cab353f5f3 and https://github.com/apache/incubator-mxnet/commit/c3aff732371d6177e5d522c052fb7258978d8ce4 respectively further reduce the overhead


Add hiscore lookup for chat, clan chat, private messages, friends list and ignore list

Will add right click hiscore lookup option to messages and players in chat, clan chat, private messages, friends list and ignore list.

The Travis build failed, and I don't understand how to solve it

Patiently waiting :D

Done in #1256


Adding Unit Test lint exceptions

Adding Unit Test lint exceptions
Changes

Changed  test/unit_tests/.eslintrc to add nested-callbacks and max-statements lint exceptions

Review
@anselmbradford
@jimmynotjim
@KimberlyMunoz

Adding cross-reference note to see if nested-callbacks exception can be removed in the future through refactoring of https://github.com/cfpb/cfgov-refresh/issues/1009

:+1:


Support access_token based OAuth2 providers

This is more for mobile but some OAuth providers have client side SDKs so that you can get the access_token securely. You can then send that to the server without having to do all the redirects. The server then verifies the access_token, authenticates the user and returns a Feathers JWT.
This is already in place on the server side but not supported client side.

This is now supported via feathers-authentication-oauth2.
Simply register your passport token strategy for the provider (ie. FacebookTokenStrategy) instead of the regular strategy (as outlined in the example on that repo) and then make your call from the client like so:
app.authenticate({
  strategy: 'facebook-token',
  access_token: yourFacebookAccessToken
})
.then(response => {
  // you are now authenticated

  console.log('Your Feathers access token', response.accessToken);
})
.catch(error => {
  // handle error
});


CrossChainSwaps: Add a middleware to handle IBC metadata for the swap

Design a middleware that allows us to extend IBC transfers with the extra metadata needed for crosschain swaps:

Tokens to swap into
Callback?

On receiving the metadata, the middleware would then call a function to make the swap and transfer the seapped tokens back to the caller (or alternatively another address or contract if a callback is specified)
If possible it should be designed in generic way so that it can be upstreamed to ibc-go.

@nicolaslara when you get the time, do you mind linking all relevant PRs one could follow about this feature?

@p0mvn I don't have a PR ready yet, but I'm working on nicolas/crosschain-swaps
But I've extracted a few of those changes into these IBC-go PRs:
https://github.com/cosmos/ibc-go/pull/2296
https://github.com/cosmos/ibc-go/pull/2306
https://github.com/cosmos/ibc-go/pull/2305
Depending on what changes in ibc-go we may need more or less on our end

Done!


Reorganize DynamicCasting spec

This only rearranges the existing sections; it does not change any technical points.
This attempts to make the spec a little clearer by grouping the individual type discussions under four main headings:

Classes and Foreign Types.  This includes Swift classes, Obj-C classes, and CF types.
Other Concrete Types.  Struct, Enum, Tuple, Function, Optional, and Set/Dict/Array
Existential Types.  Any, AnyObject, Error, Protocol types, and AnyHashable(*)
Metatypes & Existential Metatypes (*)

This organization seems to flow a little better.
In particular, it gives me a place to discuss issues common to all Existential types, which I'll work through in a subsequent PR.
Footnotes: This organization isn't perfect, of course:

AnyHashable isn't really an Existential type, but it behaves as such for casting purposes, so it makes the most sense to discuss it in the Existentials section.
Metatypes are technically concrete types, but it seems to make more sense to discuss them after discussing Existential types.

@swift-ci Please smoke test and merge


How to get current month?

I need to request events from server by month. How I can get current month AND year? Also, I need to get current month and year in moment, when calendar was rendered. Please, help me

`
currentDate = $('#evoCalendar').evoCalendar('getActiveDate');
let dateParts = currentDate.split('/');
let data = {
    Month: dateParts[0],
    Day: dateParts[1],
    Year: dateParts[2],
}

`
Thanks )


Broken Link

The Link for: Public Key Infrastructure Cmdlets. is broken.

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

ID: 8e95ba48-8d07-d390-d49b-801d70730bae
Version Independent ID: 047d2736-f14e-610c-7396-89d2eae0f9cc
Content: Create a certificate for package signing - MSIX
Content Source: msix-src/package/create-certificate-package-signing.md
Product: msix
GitHub Login: @dianmsft
Microsoft Alias: diahar

@AhmadKelany Are you sure? I can't reproduce that error, the link is working correctly for me. Due to my settings are English US, i'm redirected to https://docs.microsoft.com/en-us/powershell/module/pkiclient/?view=win10-ps.
Could there be an issue with your redirect?

@AhmadKelany Are you sure? I can't reproduce that error, the link is working correctly for me. Due to my settings are English US, i'm redirected to https://docs.microsoft.com/en-us/powershell/module/pkiclient/?view=win10-ps.
Could there be an issue with your redirect?

It was fixed and closed 6 days ago..


Update CI to use a more modern iPhone

We're still running on a 6S
https://github.com/SwifterSwift/SwifterSwift/blob/0ba61d67deb60d10a2ed97f27aa7e9112bfd0cf9/.travis.yml#L11

Resolved on #498


Difficulty with enumeration

I am trying to enumerate a reaction that looks like this:

The carbons I have circle in blue are the ones that are changing for the enumeration. However, they may or may not be part of an aromatic cycle. I've tried as many different combinations of reaction SMARTS but I still can't get it to work. I usually run into a problem where it doesn't recognize the carbons as being in the aromatic ring or a problem with kekulization (the reaction not running correctly)
For example the side chain could be something like this:

The closest I've gotten is this:
   rxn7 = AllChem.ReactionFromSmarts("O=C(CCC)O.N[a:1]>>O=C(CCC)O[a:1](N)")
But I get "Can't kekulize mol" for each enumerated reaction. Any help would be appreciated
Thanks

The first reaction you show does not correspond to the specific example in the second image, and I'm not sure the specific example is what you actually intend, but I think I understand what you're looking for.
It looks like you are trying to do this general reaction:

And then your more specific example:

The difficulty here is that it's hard to "destroy" aromaticity in a reaction. Unless you include the entire aromatic ring in your reaction definition (which I doubt you want to do), you're almost guaranteed to end up with a sanitization problem. A trick to solve that is to kekulize your reactants before you run the reaction:
In [28]: rxn = AllChem.ReactionFromSmarts('[NH2:1]-[C:2](-[C:3])=[C:4]>>[N:1]-[C:2](-Cl)(-[C:3])-[C:4]')

In [29]: m = Chem.MolFromSmiles('Nc1ccccc1')

In [31]: Chem.Kekulize(m,clearAromaticFlags=True)

In [32]: ps = rxn.RunReactants((m,))

In [33]: Chem.SanitizeMol(ps[0][0])
Out[33]: rdkit.Chem.rdmolops.SanitizeFlags.SANITIZE_NONE

In [34]: Chem.MolToSmiles(ps[0][0])
Out[34]: 'NC1(Cl)C=CC=CC1'

The specific reaction  you show actually does something different since it rearranges the bonds in aromatic ring itself. For that you'd need to include the whole ring. I will post an example of that after the weekend.

Thanks so much Greg. I will try this out.
And yes, you are right my second example is incorrect I just wanted to give an idea of what I was trying to do (my orgo is a bit rusty). Basically the gist of the issue is that the most of the reactants in the reactant library has aromatic rings and I want to enumerate reactions on alpha, beta, and gamma positions relative to each functional group (amide and carboxylic acid)

And here's a simple example of using the RDKit reaction machinery to go from a aromatic 6-ring to a 1-4 diene:
In [45]: rxn = AllChem.ReactionFromSmarts('[NH2:1][c:2]:1:[a:3]:[a:4]:[a:5]:[a:6]:[a:7]:1>>[N:1][C:2]1(-Cl)-[A:3]=[A:4]-[A:5]-[A:6]=[A:7]-1')

In [46]: m = Chem.MolFromSmiles('Nc1ccccc1')

In [48]: Chem.SanitizeMol(ps[0][0])
Out[48]: rdkit.Chem.rdmolops.SanitizeFlags.SANITIZE_NONE

In [49]: Chem.MolToSmiles(ps[0][0])
Out[49]: 'NC1(Cl)C=CCC=C1'

@b-mahjour: I'm closing this because it looks like the question is answered. Please reopen if necessary.


Readme and requirements clarification

Hey, you've done a great job! I think at least two things should be added to the readme.
What is required/expected lua version? I've checked that your codegen does not work with 5.1 (accidentially installed on my machine).
Also, I think that it should be in the readme that not only Idris 2 should be installed but that Idris2 API library (i.e. those named idris2) should be installed also (e.g. using make install-api of Idris 2).

Clarified this in the readme 39a71cd30ebcb0dced4a423382a61fd62237ddc9

Great, thanks!


Livy performance improvement by using sparklyr JARs

This change speeds up Livy operations by 6X for spark_connect(), 3X for copy_to() and other operations around 5X. Improves: https://github.com/rstudio/sparklyr/issues/1865 and https://github.com/rstudio/sparklyr/issues/741.
Using Sources
livy_service_start()
system.time(sc <- spark_connect(master = "http://localhost:8998", method = "livy"))

   user  system elapsed
  4.531   0.536 141.961

system.time(iris_tbl <- sdf_copy_to(sc, iris))

   user  system elapsed
  2.697   0.302  59.991

system.time(m <- ml_linear_regression(iris_tbl, Petal_Width ~ Petal_Length))

   user  system elapsed
 11.036   0.962 230.123

Using JARs
config <- livy_config(spark_version = "2.3.0")
system.time(sc <- spark_connect(master = "http://localhost:8998", method = "livy", config = config))

   user  system elapsed
  1.351   0.080  21.977

system.time(iris_tbl <- sdf_copy_to(sc, iris))

   user  system elapsed
  2.390   0.116  21.486

system.time(m <- ml_linear_regression(iris_tbl, Petal_Width ~ Petal_Length))
livy_service_stop()

   user  system elapsed
  7.935   0.316  43.098

Notice the connection code is equivalent to the one bellow since the spark_version parameter finds and sets the jars parameter for you; however, others might want to copy the sparklyr into HDFS and list this JAR explicitly.
config <- livy_config(
  jars = "https://github.com/rstudio/sparklyr/blob/bugfix/livy-jars/inst/java/sparklyr-2.3-2.11.jar?raw=true",
  sources = FALSE
)
sc <- spark_connect(master = "http://localhost:8998", method = "livy", config = config)

time saver!

@javierluraschi  -  Looks like the commit to the master were overwritten.
Like the changes here - https://github.com/rstudio/sparklyr/pull/1868/commits/ec68830576a03873f64d47927b1c0d9268778cbc are missing in the master
https://github.com/rstudio/sparklyr/blob/master/R/livy_connection.R
Cannot specify - config <- livy_config(spark_version = "2.3.0")
Throws error - spark_version not a valid parameter

Right, this API got improved... the way to use Livy with Jars and version is as described in this post:
https://blog.rstudio.com/2019/03/15/sparklyr-1-0/
sc <- spark_connect(master = "local", method = "livy", version = "2.4.0")

We made this change since it's ore natural for users to specify the version while connecting, rather than in the configuration object.

Thanks @javierluraschi !!


Count set when deploying Stateful set is not taking effect.

Rancher versions: Build from master Mar 2
Steps to reproduce the problem:
Create a Workload of type Stateful set and set  the pod count to 2.
Workload gets created successfully , but there is only 1 pod that gets deployed.

@sangeethah seems to work now, could you retest

Tested with latest build from master.
Count set when deploying Stateful set  takes effect.


error in eval-tracker : 'Namespace' object has no attribute 'push_results_to_hub'

Running the following command:
lm_eval --model vllm --model_args pretrained=mistralai/Mixtral-8x7B-Instruct-v0.1,tensor_parallel_size=2 --tasks hellaswag --num_fewshot 5
leads to error:

likely caused by this : https://github.com/EleutherAI/lm-evaluation-harness/pull/1766

Should be fixed by #1777


Mongodb-replicaset chart vs Mongodb chart differences

Is this a request for help?:
Yes
Is this a BUG REPORT or FEATURE REQUEST? (choose one):
General Question

Version of Helm and Kubernetes:
1.11.5 K8s
2.12.3 Helm
Which chart:
Mongodb and mongodb-replicaset
What happened:
Looking between the mongodb chart and the mongodb-replicaset chart I am confused as to what the major differences between the two charts are. The mongodb chart seems to provide more support than the mongodb-replicaset chart.
What you expected to happen:
I expected the mongodb chart to not support replicasets or statefulsets seeing as how the mongodb-replicaset chart does, but it seems like both support the same
How to reproduce it (as minimally and precisely as possible):
for the mongodb-replicaset chart: helm install --name my-release stable/mongodb-replicaset
for the mongodb chart: helm install --name my-release stable/mongodb
Anything else we need to know:
no

Hi @kwill4026
The MongoDB chart is more generic and supports different configurations (replica set is one of them). However, most of them are disabled by default (e.g. you need to set replicaSet.enabled=true to enable replica set configuration).
That said, it's true that they were much more different in the past, and the stable/mongodb chart has evolved becoming more flexible.

Thanks for the response @juan131, does the MongoDB chart also provide statefulset? And what about full TLS encryption with certs or is this something I will have to configure into files/docker-entrypoint-initdb.d to be consumed as a configmap?

does the MongoDB chart also provide statefulset?

stable/mongodb/ chart use 3 statefulsets when the parameter replicaSet.enabled is set to true:

One for Primary nodes
One for Secondary nodes
One for Arbitrary nodes

And what about full TLS encryption with certs or is this something I will have to configure into files/docker-entrypoint-initdb.d to be consumed as a configmap?

By default, there are no parameters provided to configure TLS encryption.
You can modify the chart to include the corresponding secrets to configure the TLS encryption and consume it on your MongoDB containers by using you custom configuration file (see configmap parameter).

Thanks! @juan131

One last thing, I am assuming the NGINX Ingress Controller helps solve external accessing of a mongo replicaset via URI issue .Which is when you need to access a replica set in mongodb to determine the master you can't connect through a NodePort exposed service because once you connect through the connection gets switch over to using internal kubernetes namespaces. Also when connecting via node port to a replicaset, it doesnt always land on the primary and instead sometimes selects the secondary.

Hi @kwill4026
The NGINX Ingress Controller allows you accessing MongoDB though a web port (which is a little awkward) but you can use a custom external domain to access your MongoDB replicaset. It's not very common to use this feature.
As you mentioned, when configuring MongoDB with replicaSet, you can access to the primary nodes by selecting a NodePort or LoadBalancer service type. See:
https://github.com/helm/charts/blob/master/stable/mongodb/templates/svc-primary-rs.yaml


CNG key providers to Mage and Mageui SDK tools fails on XBAP compile

In v4.6.2 Framework under the Windows Forms changes, the following change is not working

Added support for CNG key providers to Mage and Mageui SDK tools [194373]

When compiling a WPF Solution as an XBAP application where the Project file contains a PropertyGroup with the HostInBrowser value set to True and using an RSA SHA2 CNG Certificate for Code Signing, the compilation fails the SignFile Task with error MSB3482.
This is the same error message that would be put out from the mage.exe utility prior to including support for the CNG Certificates.

True

1>Target "ResignApplicationManifest" in file "C:\Windows\Microsoft.NET\Framework\v4.0.30319\Microsoft.WinFx.targets" from project "C:\Test\CNGTest.csproj" (target "GenerateManifests" depends on it):
1>Task "Message" skipped, due to false condition; ('$(MSBuildTargetsVerbose)'=='true') was evaluated as (''=='true').
1>Task "Message" skipped, due to false condition; ('$(MSBuildTargetsVerbose)'=='true') was evaluated as (''=='true').
1>Task "Message" skipped, due to false condition; ('$(MSBuildTargetsVerbose)'=='true') was evaluated as (''=='true').
1>Task "SignFile"
1>C:\Windows\Microsoft.NET\Framework\v4.0.30319\Microsoft.WinFx.targets(614,9): error MSB3482: An error occurred while signing: Invalid provider type specified.

/cc @staceyhaffner

Hi TechnoDex,
It seems that you are signing your manifest in the VIsualStudio, or at least from the msbuild process? This had not been updated. The supported scenario is command line manifest signing using mge.exe or mageui.exe SDK tools. Here is a walkthrough:
https://msdn.microsoft.com/en-us/library/xc3tc5xx(v=vs.110).aspx
SDK can be downloaded from here:
https://www.microsoft.com/en-us/download/details.aspx?id=53321
Hope this helps,
Tanya


Error on Mac M2 24GB RAM

While the model t5-vicuna loads, with the first prompt I get the following error:
determined model type: t5-vicuna
...
...
...

loc("mps_clamp"("(mpsFileLoc): /AppleInternal/Library/BuildRoots/c2cb9645-dafc-11ed-aa26-6ec1e3b3f7b3/Library/Caches/com.apple.xbs/Sources/MetalPerformanceShadersGraph/mpsgraph/MetalPerformanceShadersGraph/Core/Files/MPSGraphUtilities.mm":268:0)): error: input types 'tensor<1x78x2048xf16>' and 'tensor<1x1x1xf32>' are not broadcast compatible
LLVM ERROR: Failed to infer result type(s).
[1]    76583 abort      python3 app.py
/opt/homebrew/Caskroom/miniconda/base/envs/llm-serve/lib/python3.9/multiprocessing/resource_tracker.py:216: UserWarning: resource_tracker: There appear to be 1 leaked semaphore objects to clean up at shutdown
  warnings.warn('resource_tracker: There appear to be %d '

I found a simillar report here (stable-diffusion-webui) but not sure how to set the no-half flag if thats the case.

Oh will fix it by setting no half in case of t5 vicuna. Thanks for reporting this

Hi all, i have the same issue when using SD.Next with Controlnet. (mac Studio, M2 ultra, 64gb).  SD.Next does not allow --no-half but --no-half-controlnet. Unfortunately this does not solve this issue. Any suggestions on that ?


[Feature] - Implement Error Handling and Develop Error Pages

Description
Implement global error handling and create custom error pages (e.g., 404 Not Found, 500 Internal Server Error). The default Razor pages project template comes with a default Error Page.
Motivation
To improve user experience by providing informative feedback when errors occur.
Proposed Solution

Write tests to simulate errors and ensure correct handling.
Create custom Razor Pages for common HTTP errors.
Implement middleware for global exception handling.

Acceptance Criteria

[ ] Feature is clearly defined.
[ ] All tests pass for error handling functionalities.
[x] Users are presented with friendly error pages.
[ ] Code is reviewed and approved by at least one other team member.

Additional Info

Update the project report with error handling strategies.
Ensure no sensitive information is exposed in error messages.

Asp Projects have a premade error page which we are using.


recover metablk in subtype order

In HO , when metaservice restart,  we actually want to recover repl-dev before PGinfo  and recover PGinfo before Shardinfo.
but metaservice will recover all the metablk only according to blkid and we have to use some pending method to solve this problem.
https://github.com/eBay/HomeObject/blob/dbde1bb6b2eb705fc55f7539be08d9c9fdbbd508/src/lib/homestore_backend/hs_pg_manager.cpp#L126
https://github.com/eBay/HomeObject/blob/dbde1bb6b2eb705fc55f7539be08d9c9fdbbd508/src/lib/homestore_backend/hs_homeobject.cpp#L105
this PR do some change in metaservice recovery logic to make sure all the metablk will be recovered by subtype, and the subtype order is that appear order in the metablk chain.

after this PR is merge , we can change the recovery logic  in this patch to a simple way

Codecov Report
Attention: 23 lines in your changes are missing coverage. Please review.

Comparison is base (27e7601) 69.11% compared to head (ca7b8ff) 68.93%.
Report is 1 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #200      +/-   ##
==========================================
- Coverage   69.11%   68.93%   -0.18%
==========================================
  Files          93       93
  Lines        7556     7571      +15
  Branches      965      973       +8
==========================================
- Hits         5222     5219       -3
- Misses       1868     1890      +22
+ Partials      466      462       -4

Files
Coverage Δ

src/include/homestore/meta_service.hpp
100.00% <ø> (ø)

src/lib/replication/service/repl_service_impl.cpp
59.75% <0.00%> (-0.74%)
:arrow_down:

src/lib/meta/meta_blk_service.cpp
47.81% <31.25%> (+<0.01%)
:arrow_up:

... and 9 files with indirect coverage changes

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

I think it is better to clarify this PR again.
1 what is the current metaservice recovery process?
the current recovery process of metaservice is recovering all the metablks according to BlkID, and know nothing about the upper layer logic. so from the perspective of upper layer , no recovery order guaranteeed.
for the upper layer user, they should adapt the out-of-order of metaservice recovery and implement the upper layer recovery logic carefully by themselves. that is it.
2 what does this PR do?
this PR changes the default recovery order of metaservice from out-of-order to the order of  the presence of  subsystem in the metablk chain. the presence of subsystem means the presence of the first metablk of the subsystem in the metablk chain.  for example , suppose we have 3  subsystems A, B and C , and in each subsystem we have 3 metablk( A1, A2 ,A3 ),(B1,B2,B3),(C1,C2,C3). the matablk chain is as follows:
A1->C1->C2->A2->B1->C3->B2->A3->B3
current recovery order will be ordered by the blkid among all the subsystems, which is not definite.
since A1 comes before C1 and C1 comes before B1, so subsystem A comes before C and C come before B. after the PR, the recovery order will be
(A1,A2,A3) -> (C1,C2,C3) ->(B1,B2,B3)
in each subsystem, it will be ordered by blkid. But among subsystems, it will be ordered by the presence of subsystem
this gives the upper layer a chance to untilize this recovery order to simplify the upper layer recovery logic. well , the upper lay can also ignore this change and do like before. note that , here, the upper layer can not control the order of metaservice recovery, and we also do not want to give the chance to upper layer to control until we really need.
so , in fact , what this PR does is to give a default natural order of metaservice recovery, and no extra performance issue will be involved.
3 is it necessary for the upper layer to control the recover order of metaservice in fine granularity
i don`t think it is necessary for now at least. the reason is :
1 upper layer user  will take the responsibility to pass in the correct order.
2 inside homestore, there are already some metaservice user, such as CPmanager. how do we handle the recovery order of these subsystem with those passed in subsystem. although this will not be difficult to choose a policy and implement, but it seems not uniformed that the upper layer can only partially control the recovery order. they can not control the order of those existed subsystem in the homestore.
so my suggestion is we can delay to implement fine-granularity control of the metaservice recovery order until we have a clear scenario . for now, we can just use the newly added default recovery order to simplify HO Implementation

another example of ordering requirement
https://github.com/eBay/HomeObject/pull/77/files#diff-3a2d9832e51b506b2866103eae7a445b4f67e23b3c8d215e6012d33466060fb9R149-R162

I think the logic will be much more easier to follow if it is one-to-one dependency. But it could also be subjective. Bottom line is I am okay with any solution (algorithm) to solve the current problem (which is a one-to-one dependency), but we should not be solving problems that there is no real use case.

As the discussion in https://github.com/eBay/HomeStore/pull/200#discussion_r1370545309,  we all agree that even with one-to-one dependency we need to do some sort of topology sort.   The topology sort takes 38 lines of code,  with complexity of O(V+E).  I cannot find a solution,  if limit to one-to-one dependency , can significant reduce the #LOC or complexity.
Not to say topology sort is well-known to every developer that is super easy to review, without worrying the correctness.
So the point here comes to "we should not support one to N dependency even we could",   it is simply just add one line of code in register_handler that checks the size of deps.   I dont think such check bring in business values,  but if that line is the turn-key to let us move forward,  lets add that line.

@yamingk
no matter multi or single dependency we face now , essentially we need to do topological sort to get a correct partial order relation.  the comments you mentioned here is also essentially a kind of topological sort(suitable for only single dependency ) ,  right?
based on the actual strong requirement of topological sort, the point here is not whether we need a topological sort, but what kind of topological sort we need.
coming to this point , Obviously， a common thorough solution with extremely low complexity of algorithm and implementation is better than a specific one for only one case yea?

@JacksonYao287 Can we do one thing:

only do order sort on those clients who participant the dependency (either being depended or depending on others), given that most of the clients doesn't have dependency with others.
during recovery, write a new function to do recover on order based clients and have current MetaBlkService::recover to call this new function, and leave non-order based clients to recover with existing flow?

@yamingk

only do order sort on those clients who participant the dependency (either being depended or depending on others), given that most of the clients doesn't have dependency with others.

we use a std::optional as a dependency parameter now, so actually it has given the client a change to indicate whether it has any dependecy. meanwhile,  it gives us a uniform function declaration.
topological sort is fully in memory , so it is fast. and it is only be called once when recovery, so it is one-time shot. we can basically ignore the performance cost it brings. actually , if no dependency passed in, topological sort will do nothing, since no edge exists, right?
if we want do concurrent recovery with no dependency, actually the current logic can also support this . what topological sort does is generating a recovery order. if any dependency exists,  it will generate a partial relation order. if no dependency exist, in fact a random order returned, just like the current random recovery order.
the benefit we keep this code style is it gives us a clean and uniform function, which will be better for the code reader and UT.

@yamingk
i follow you suggestion to  devided the subsystem into two part: dependent and independent. if we want to speed up recovery time in the future, we can concurrent recover the independent part.


fix(fscomponents): Move Multicarousel opacity and initialized into state

These values were not being updated properly when the properties changed, such as changing the images, so opacity was getting stuck set to zero.

@bweissbart @skyeckstrom any chance of getting this merged? Appears to be a requirement for https://github.com/brandingbrand/dsg.rn/pull/48

@bweissbart can you look at this?

@Cauldrath - commit linter is erroring due to "Merge from upstream" commit message.

Squashed commits to fix history.


move urlmatcher to own module

create github project and npm module
please add me as collaborator and owner

I thought on that. :(
I think it should be loaded automatically by seneca-auth module as default urlmatcher.

Done. Created a new plugin seneca-auth-urlmatcher


Emission

plotted the G1 and G2 phase lengths for all concentrations in figure 11.

And here is for each concentration separately.

I can make them violin plots if that has more benefits.
2. Also edited the range of states in figure 10, it was wrong before. And replaced the colors with ones that are more distinguished. Now if you take a look at figure 10, it makes a little bit more sense.
These two are actually from the same fit, as I exported the concentration-wise results from figure 11 to figure 12.

Codecov Report

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

@@            Coverage Diff             @@
##           master     #596      +/-   ##
==========================================
- Coverage   92.01%   91.81%   -0.20%
==========================================
  Files          21       21
  Lines        1528     1528
==========================================
- Hits         1406     1403       -3
- Misses        122      125       +3

Flag
Coverage Δ

#unittests
91.81% <ø> (-0.20%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

lineage/Analyze.py
83.33% <0.00%> (-3.13%)
:arrow_down:

Continue to review full report at Codecov.

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

Here is for each concentration separately.

I can make a table to show the distribution parameters, as well.

Try a stripplot: https://seaborn.pydata.org/generated/seaborn.stripplot.html

I think it totally makes sense. In lapatinib, only G1 is separating the states and G2 is almost the same across the states, and in Gemcitabine both phases are improtant.

We don’t even need figure12 (separated concentrations)
I will make that table of parameters first thing in the morning.

You could try adding in multiple concentrations this way. Making the points smaller would also help to see all the points.

This looks much better!

Looks good, but I think all four concentrations are the same data?

Should be fixed now. I used lists instead of dataframes (they were easier than I thought).
I removed the plot that was showing the concentrations altogether and only kept separate concentrations. Let me know if you think it is helpful to have that-- although it doesn't seem to add any new information when we have separated ones.

Oh, also, I'm almost certain the drug concentrations are nM, not μM.

I think it is ready. I'll merge if you approve. I am preparing a file including the parameters and will send it to you soon. @aarmey


Chrome lags on cursor changes

FPAF and other sims show this.

@jbphet says: tested this in Arithmetic recently, and it may be worse with scenery 0.2.

Timeline of FPAF shows that the cursor update triggers recalculation of styles (more expensive) and layout (less expensive). Compositing looks like a significant (dominating?) factor unless I'm reading the plot wrong (I'll read up on the tools a bit):

We could consider some creative (hacky) ways of not triggering style recalculation but still changing the cursor (e.g. maybe creating divs for each cursor type and change their stacking order might work).
Will probably need to bring back the original cursor toggling test page.

https://github.com/phetsims/scenery/issues/337 should help with this.

Unable to reproduce the cursor issue on Chrome 45 (Win 7), anyone else (@pixelzoom, @samreid, @jbphet, @aaronsamuel137) noticed this in sims recently?

I can't seem to reproduce it either. I remember a slight stutter in color-vision when the cursor changed and the photon view was present. Doesn't seem to happen anymore.

I have never seen this issue.

Can't reproduce on other platforms. Closing for now.


UCI Exhaust - Env Variable

Also - Please add the ticket number and all else the Jira description action will fail.


Feature owen dashboard support session key

Summary
Summary about this PR

Closes #issue

@jolestar @wow-sven
Panic occurs when create_session_key_with_multi_scope_entry is called via MetaMask

The suggested solution to this issue is to modify the gas of the transaction.
About metamask will cache the sequence number after we restart the service. The transaction triggers the 1002 error, the solution is to start the service by adding --chain-id test


 gone in .7?

Hey there, just updated from .4 to .7 and it looks like <Card/> is gone now?
Not sure which release it was removed in after .4 as it's not mentioned at all in any of the releases, so I'm wondering if there's somewhere else to import it, or if we need to replace it, what the best approach to replace it is? Thanks!

You can see all elements here https://github.com/wobsoriano/solid-stripe/blob/main/src/index.tsx

Perfect! Easy to update... Thanks for the library, it works great :)


Enhancement: Enhanced Placeholder Configuration in LibreChat Prompt Library

What features would you like to see added?
What features would you like to see added?
Current Behavior: Currently, placeholders in the Prompt Library are displayed as one-line input fields when using a prompt. The configuration options for these placeholders are limited, and their order on the prompt screen is fixed.
Proposed Behavior: Enhance the placeholder configuration in the Prompt Library to support:

Multiline input fields
Dropdown lists
Ability to change the order of placeholder fields on the prompt screen

Use Case:

Users can create more complex and versatile prompts with different types of input fields (single-line, multiline, dropdown).
Prompt creators can specify the most appropriate input type for each placeholder, improving user experience and prompt accuracy.
Users can customize the order of placeholder fields to create a more logical or user-friendly flow when filling out prompts.

More details
This enhancement would require changes to both the prompt creation interface and the prompt usage interface. The prompt creation process would need to be updated to allow for more detailed placeholder configuration. The prompt usage interface would need to be modified to render different types of input fields based on the placeholder settings.
Which components are impacted by your request?
General, UI
Pictures

For example, the {{original_email}} placeholder would benefit from being a multiline field, and the {{tone}} as a dropdown list.
Code of Conduct

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

Thank you for the excellent write up. Thinking out loud here, it's definitely possible even with just text alone, for example, if I wanted a dropdown:
<tone:formal|casual|business casual>

or for multi-line inputs:
{{textarea:email}}

While convenient from a dev perspective, this would be somewhat advanced usage and it may be easier to have interface elements to help create "special" variables.
Any help, even landing on a format for variable, would be really helpful.

<tone:formal|casual|business casual>

or for multi-line inputs:
{{multi-line:email}}

I think it's a great suggestion. In this case, the backend part will not need to be changed.
I don't know what type of input would be a sensible default, though.

I have created a PR for this feature: https://github.com/danny-avila/LibreChat/pull/3618

Closing, resolved by https://github.com/danny-avila/LibreChat/pull/3681
I'm sure we can add more like this, but this is a good start and any others should be scoped to their own issues.


Kotlin services

Despite the lousy commit names, this PR is functional!  It implements generation of async RPC service clients for Kotlin.
This is good enough to get the next release out the door, but is not the end of the story for services; I still want to investigate a coroutine-based client implementation, but that's a larger project potentially involving a kotlin-specific version of thrifty-runtime to support it.

Codecov Report

Merging #197 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #197      +/-   ##
============================================
+ Coverage     76.18%   76.21%   +0.02%
- Complexity      373      374       +1
============================================
  Files            29       29
  Lines          1646     1648       +2
  Branches        147      147
============================================
+ Hits           1254     1256       +2
  Misses          294      294
  Partials         98       98

Impacted Files
Coverage Δ
Complexity Δ

...java/com/microsoft/thrifty/testing/TestServer.java
76.92% <100%> (+0.6%)
19 <1> (+1)
:arrow_up:

Continue to review full report at Codecov.

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


[Visual Studio] Notebook item template is not available in F# project

Describe the bug
File\Add\New item... does not contain Notebook for F# projects
Please complete the following:

OS

Windows 11

Frontend

Visual Studio

Screenshots

Thanks for reporting! Good point - we've only added it to C# so far but we should add it to others.

Closing as support for notebooks in VS has been discontinued


Validate tunable datasource name

validate tunables-datasource-name for blank, invalid, multiple-sections, no-tunable-datasource-name, no-tunable-datasource-name-value, null, numerical values
blank-tunable-datasouce-name
apiVersion: "recommender.com/v1"
kind: "AutotuneConfig"
metadata:
  name: "blank-tunable-datasource-name"
layer_name: container
layer_level: 0
details: generic container tunables
layerPresence:
  # This layer is assumed to be available by default for all applications.
  presence: always
tunables:
  - name: memoryRequest
    value_type: double
    upper_bound: '300M'
    lower_bound: '150M'
    queries:
      datasource:
        - name: ' '
          query: 'container_memory_working_set_bytes{$CONTAINER_LABEL$="", $POD_LABEL$="$POD$"}'
    sla_class:
      - response_time
      - throughput
      - resource_usage

  - name: cpuRequest
    value_type: double
    upper_bound: '3.0'
    lower_bound: '1.0'
    queries:
      datasource:
        - name: 'prometheus'
          query: '(container_cpu_usage_seconds_total{$CONTAINER_LABEL$!="POD", $POD_LABEL$="$POD$"}[1m])'
    sla_class:
      - response_time
      - throughput
      - resource_usage

@shruacha1234 - What is not working here? If all issues are resolved close this one.

Blank-tunable-datasource-name isn't working


Get/Run Brakeman Pro

We have already added Brakeman and are addressing the vulnerabilities (#283, #301). However, @mccabe615 mentioned Brakeman Pro in his presentation to DCRUG, stating that it has some deeper testing and analysis and is cost effective. Plus, there is a free trial if that helps. More info on the differences between the paid version and open source version here.

I reached out to Justin Collins at Brakeman to see if he can give DCAF a discount or free license. I'll update here once I hear back.

@mccabe615 that'd be dreamy. Let me know if there's anything I can do to assist on that front (pulling nonprofit paperwork, etc).

Hey @colinxfleming @charleshuang80 I heard back from Justin at Brakeman. They said they could give the project a free license! What email and name should be associated with it? I'll send over the details.

That RULES. Thanks for reaching out, @mccabe615 !
Can we associate with DCAF as an org? Name can be DC Abortion Fund, email can be info at dcabortionfund dot org.

Justin sent an email with the info to that email. Let me know if you need anything else.

@mccabe615 thanks a TON for hooking this up, this is really fantastic. Email is received with info, I'm going to get it all set up over the weekend sometime.

High five, everyone --

Remaining task to do here is to see whether or not we should adjust our CI (I suspect not, but I'll do some googling).

Closing out -- I think we're in the clear! I'll rerun against master a few times as we get closer to release day, but I think we're good here.
Thanks again to @mccabe615 for hooking this up!


How to Replace Variables in Error Messages

I have a custom validation directive which checks for unique name in the given list.
 var validator = function(value){
          //Add the logic to check the uniqueness
          if(true){
            ctrl.$setValidity('uniqueInTheList', true);
            return value;
          }else{
            ctrl.$setValidity('uniqueInTheList', false);
            return undefined;
          }
        };

I am able to validate the input element, but I want to replace the message variables in the message key.
My error Message Key is {0} should be unique
When ever my field gets validated, I get message like should be unique.
Is there a way to say that Key Pair should be unique,Host Name should be unique

one way to do it is to do custom messages via http://jonsamwell.github.io/angular-auto-validate/#overriding_error_message like so
uniqueInTheList-err-type="uniqueInTheListMessageForKeyPair"

I think that is currently the best approach

@icornejo Thanks for the reply. I am already using it, I am looking for a better approach. Thanks for the time.
@jonsamwell  Can we figure out a better way to fix it. Let me know if you have any design. I can work on it and create a pull request

@jagadeshpaladugula I'm planning to look into adding this functionality this week.

Any ideas on how to make this work with values that are passed into the error message? In the exact same scenario above, I want to show the value that is invalid.
For example, instead of saying "Username already exists" I want to say "Username {typed_username} already exists" where what the user typed (and we validated with server query) is shown so that there is no doubt.

+1

Sorry all been really really really busy at work but I will get around to this next week

@dhiraj would you just want to use the value from the ng-model that is on the control?

@jonsamwell Well yeah, that's what I came to the issue with. Just that would solve my particular problem.

+1!

Has this functionality been added yet? There is no mention of it in the latest released version.


[BUG] Incorrect alignment of github stats & social media handles

Description
Comment on this issue, if anyone wants to work on this issue.
Screenshots

Any additional information?
You will have to achieve this structure:

What browser are you seeing the problem on?
Chrome
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[ ] I am willing to work on this issue (optional)

Good day @Anmol-Baranwal and @rupali-codes
I would like to work on this bug, can you please assign this issue to me?

Good day @Anmol-Baranwal and @rupali-codes
I would like to work on this bug, can you please assign this issue to me?

Go right ahead @sohaibsalman! 😊


Workload Zone Deployment Access Issue

We have SdafdeployerKv keyvault and private endpoint created in control plane deployment subscription.
While deploying the workload zone we are getting Access Issue.

Though we give full permission to keyvault , after re-running the pipeline the permission are vanishing.

Hi @nehavijapur-1  - This has been fixed by our recent release 3.6. Could you please check and confirm?


New Status: Burning

Burning N: Deals N damage. Reduce N to floor(N/2). While Burning, reduce healing by 50%
Fireball+ and Firebolt+ apply burning.

Would be nice for this to use an StSLib HealthBarRenderPower to render the damage in the health bar like poison does

Being worked on by Bobertg in discord

Hey, that's me! Also, See PR #84

#84 merged. Closing.


Integration Test - Release - 2022/06/17 10:52:51

Auto create for integration test. IssueCommandFlow

@yanyunhuang-microsoft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Docs Build status updates of commit d3ca6f5:
:warning: Validation status: warnings

File
Status
Preview URL
Details

csidevdocs-public/IntegrationTest/BuildIntegrationTestFiles/ChangeSentToAuthor.md
:warning:Warning

Details

csidevdocs-public/IntegrationTest/BuildIntegrationTestFiles/ChangeSentToAuthor.md

Line 4, Column 1: [Warning: ms-prod-and-service - See documentation] Only one of the following attributes can exist: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.
Line 4, Column 8: [Warning: duplicate-titles - See documentation] Title 'VS Code Authoring Extension' is duplicated in: 'Demo/demo.md(4,8), Demo/v-caxian/AuthorNotification.md(4,8), Demo/v-caxian/auto-merge.md(4,8), Demo/v-caxian/auto-merger.md(4,8), IntegrationTest/BuildIntegrationTestFiles/ChangeSentToAuthor.md(4,8)'. Titles must be unique within a docset.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Return nicer error if trying to pull from non-existent user

Gitea serv will currently return an 500 if we try to pull from a repository where
the owner does not exist.
This PR checks for the UserNotExist Error when checking for the user and will
return a NotFound error instead.
Fix #18225
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

The rest of unauth'd log messages in that file are error but you're right I'll move them to warn


CDC Aggregator usage stats

Original report on BitBucket by John Shepherdson (GitHub: john-shepherdson).

What is possible to count?
Base URL:

Endpoint is being called

Use of the verbs:

GetRecord – Used to retrieve an individual metadata record.
Identify – Used to retrieve repository information (ex. name, version).
ListIdentifiers – Used to retrieve only headers.
ListMetadataFormats – Used to retrieve the available metadata formats.
ListRecords – Used to retrieve actual item metadata records.
ListSets – Used to retrieve the set structure of a repository

Arguments used with verbs:

identifier
metadataPrefix
from
until
set
resumptionToken

Anything else?

Added issue to aggregator oai-pmh repo handler issue tracker at https://github.com/cessda/cessda.cdc.aggregator.oai-pmh-repo-handler/issues/43

@Joshua-cessda-admin Are you getting the info you need? Please add feedback to this issue

This has been implemented, further changes should be filed as a new issue