[Label] Data attributes for disabled, checked

Feature request
Overview
Given a LabelPrimitive.Root and an associated Radio or Checkbox, e.g.
const Label = styled(LabelPrimitive.Root)({})

  <CheckboxPrimitive.Root />

Some Text

Append data-disabled and data-checked (or data-state) to the Label, so that styles can be applied to the outer label element depending on whether the inner input is disabled or checked.
const Label = styled(LabelPrimitive.Root)({
  '&[data-state="checked"]' : {borderColor: 'black'},
  '&[data-disabled="true"]' : {opacity: 0.3 },
})

  <CheckboxPrimitive.Root />

Some Text

if you are adding disabled prop to the radio/checkbox, why not just add it manually to the label yourself also? the comps can be controlled for the checked case.

if you are adding disabled prop to the radio/checkbox, why not just add it manually to the label yourself also?

I was trying to avoid exposing an API from our DS that would require users to do this, but it's pretty minor so fair enough.

the comps can be controlled for the checked case

At the moment our DS is in storybook and uncontrolled. Yes when they are used outside storybook they will likely be controlled I could add a checked property to the label.
Both of the above are minor, but something I figured would result in a cleaner API and something that others could benefit from too - but feel free to close if you disagree or it would be too hard or hacky to implement.

i'm not convinced this should belong in radix because a label cannot be checked. it's not really a functional label concern. also, if you are passing props up from within a child, you'd need to use React.useEffect / React.useLayoutEffect which means it would not have the necessary styling when server-side rendered.
the request sounds specific to your product's design requirements which is why radix provides controllable apis, so you can wrap the radix parts and provide the api your product needs.
the extra steps don't need to be pushed onto the consumer like you describe if you do something like the following, but again, you'll have the SSR issues:
const DesignSystemLabelContext = React.createContext();

const DesignSystemLabel = (props) => {
  const [attrs, setAttrs] = React.useState({});
  return (
    <DesignSystemLabelContext.Provider value={setAttrs}>
      <RadixLabel {...props} {...attrs} />
    </DesignSystemLabelContext.Provider>
  );
}

const DesignSystemCheckbox = (props) => {
  const setLabelAttrs = React.useContext(DesignSystemLabelContext);
  const { checked: checkedProp, defaultChecked, onCheckedChange, ...rest } = props;
  const [checked = checkedProp, setChecked] = React.useState(defaultChecked);
  const isControlled = checkedProp !== undefined;

  React.useEffect(() => {
    setLabelAttrs({
      'data-state': checked ? 'checked' : 'unchecked',
      'data-disabled': disabled ? "" : undefined,
    });
  }, [props.disabled, checked]);

  return (
    <RadixCheckbox
      {...rest}
      checked={checked}
      onCheckedChange={checked => {
        if (!isControlled) setChecked(checked);
        props.onCheckedChange?.(checked);
      }}
    />
  );
}

I would have thought that would be invalid markup because the radio item renders a button. you can probably get rid of the label tho and just put the text there because a button is automatically labelled by the text within it.


Fix whitespace column datetime coercion for pandas < 0.17

This patches pandas.to_datetime() for pandas < 0.17.  Does not affect things for Pandas == 0.17.
Resolves issue #340.

@jreback @cpcloud I think this is ready to go -- thoughts?

@kwmsmith pd.to_datetime is has always been vectorized by definition. It is ONLY applicable to Series.
you are applying to values!
e.g. doing something like df[[.....]].apply(pd.to_datetime) is ok as you are applying column by column
by df[column].apply(pd.to_datetime) is a no-no

@jreback @kwmsmith isn't apply to values we're doing something similar to your df[[...]].apply example

oh ok then!

right ... sorry about that

merge when ready


[RDK-40623] Unit Test for System Plugin API-Set(6)

Unit Test for
SystemServices::onSystemModeChanged

@sethulakshmisk - Pls fix conflict.

@sethulakshmisk - Pls fix conflict.

@anand-ky - Conflict resolved. Kindly review .


delete other twitter

No need for social media to move to virtual reality communication only.

@shelly-mippr can you take a look.


Unsafe redirect to URL with protocol 'https'

Possibly affected class of: HttpResponseRedirect or a function that uses it under the hood (e.g. render)
Sentry errors:
https://saleor.sentry.io/issues/3853339977/?project=6417854&referrer=project-issue-stream&statsPeriod=7d
Example trace: https://app.datadoghq.com/apm/traces?query=env%3A"production" service%3A"core" resource_name%3A"tenants.middleware.SaleorTenantMiddleware.call%22%20host%3A%22i-07a9430cb955ae8c0%22%20%40version%3A%223.9.29-60f9135%22&env=production&graphType=flamegraph&historicalData=true&shouldShowLegend=true&trace=a50cb3ed5652c718&traceID=17467183125630341282&start=1678286090486&end=1678286150486&paused=true

I found the culprit, the Adyen plugin is accidentally modifying the HttpResponseRedirect class:
https://github.com/saleor/saleor/blob/9d4df5e2ae6926ff95fd347e5fb8309263d8f069/saleor/payment/gateways/adyen/webhooks.py#L1074-L1075


[RESOLVED] High ICMP ping response time

Problem started at 12:53:29 on 2023.05.10
Problem name: High ICMP ping response time
Host: HP_iLO
Severity: Warning
Operational data: Value: 396ms
Original problem ID: 20487
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23268&eventid=20487

Problem has been resolved in 7m 0s at 13:00:29 on 2023.05.10
Problem name: High ICMP ping response time
Host: HP_iLO
Severity: Warning
Original problem ID: 20487


Add an underlay for a dev synpuf omop dataset.

The mapping is copied form the existing omop_test dataset, but pointed at a different backing datset.
More sophisticated omop templating and copying will come later.

For future reference, how did you generate this? Manually? Or via BigQuery schema? Any CLIs you can include in the PR description might be useful later.

I copied the existing [omop_test.yaml] (https://github.com/DataBiosphere/tanagra/blob/main/api/src/main/resources/underlays/omop_test.yaml) file and changed the database BQ project & dataset id. That file I wrote manually.


Javascript error in gridstack.js when specifying height option

The error occurs when you try to drag a widget beyond the left,rigth and top edge of the grid if the grid has the height option specified.
The error : 'gridstack.js:404 Uncaught TypeError: Cannot read property 'width' of undefined'
GridStackEngine.prototype.moveNode function is called with argument node undefined, because the
GridStackEngine.prototype.canMoveNode can't provide a clonedNode because it seems that the engine does no reference any nodes anymore.
test_case.zip

Thanks!
2016-03-02 10:15 GMT+01:00 Pavel Reznikov<EMAIL_ADDRESS>
Closed #373 https://github.com/troolee/gridstack.js/issues/373 via
92d7e51
https://github.com/troolee/gridstack.js/commit/92d7e51d687d8a7ef228255ed6532ed2ff49ae5f
.
—
Reply to this email directly or view it on GitHub
https://github.com/troolee/gridstack.js/issues/373#event-574362157.

--
Pierre MORADEI


[Feature] Add support for Keyfile as attachment

Hello,
it would be good if you could not only specify a path to a KeyFile, but simply add the KeyFile as an attachment to the entry and use it instead of the path. Some other plugins offer also this way.... e.g.: KeeAutoExcec

I'm sorry for the radio silence, gonna check about this soon enough! Thanks for your entry!


Pinging tunneled server failing

Is your feature request related to a problem? Please describe.
Pinging a server in the Multiplayer Screen (getting ping, player count, server motd) doesn't work (probably a feature request and not a bug)
Describe the solution you'd like
Add support for this
Describe alternatives you've considered
PR myself if I have the time
Additional context
latest version from modrinth

Support for pinging a tunneled server was added in this commit. This should be available in versions over 1.0.0.

cool, works. thank you very much!


uncollapse by double click

Crude method to uncollapse clades by double clicking on them. The main goal of this pr was to allow users to uncollapse clades so that figure 2a can be recreated. This PR does NOT support the ability to recollapse the clade. The only way to recollapse is by toggling the "collapse clade" check box. However, this pr does set up the basic frame work and can be extended to support more complex operations.

The following artifacts were built for this PR: empire-biplot.qzv, empire.qzv, empress-tree.qzv, just-fm.qzv, plain.qzv

Super cool, thanks @kwcantrell


Collect data on each Vision 0 city

Start collecting data related to crash statistics for each city in the Vision 0 Action Network. Crash statistics include but not limited to injuries/fatalities of drivers/passengers, injuries/fatalities of pedestrians, injuries/fatalities of bicyclists

https://docs.google.com/spreadsheets/d/1_NtzL4gpi_JaPgO0PwGO87RV7Ex_6HZw7PwTuMMzp9c/edit#gid=0


Allow Datepicker to be localized

Adds ToStringLocalized, which allows lenses which target Localized. This allows the months and days used by the datepicker to be localized.
To localize the datepicker, simply add the appropriate key/values to an ftl file. See examples/resources/translations/fr/hello.ftl for example.

Fixes #353

Great! I'm admittedly not that great at understanding other peoples Rust code yet, but sifting through the diff view it seemed t me that this enables other widgets/views to be localized as well, doesn't it?

Great! I'm admittedly not that great at understanding other peoples Rust code yet, but sifting through the diff view it seemed to me that this enables other widgets/views to be localized as well, doesn't it?

Yep that's right. It was already possible before to pass a Localized to views in order to have localized (translated) text, but it's now possible to pass a lens to a Localized, which was necessary for the datepicker which internally used a picklist which expects a lens to a vec of data. Now this data can be a vec of Localized, which is now what it does. In case you wanted to know the details.
I think the datepicker was the only view using English text but let me know if you find anything else. Ty.

That's good to know, always helpful to have some details. Haven't found anything else with English text either, glad that opening the issue was of help. Impressive time to fix btw!


Unable to filter signing identities by expire date

The README suggests that passing the expire date via -c or -i will cause the signing ids to be filtered by the expire date.  The code, in https://github.com/KrauseFx/sigh/blob/master/lib/sigh/spaceship/runner.rb doesn't seem to actually do this, and only seems to support filtering by cert_id or cert_owner_name.
      # Filter them
      certificates = certificates.find_all do |c|
        if Sigh.config[:cert_id]
          next unless (c.id == Sigh.config[:cert_id].strip)
        end

        if Sigh.config[:cert_owner_name]
          next unless (c.owner_name.strip == Sigh.config[:cert_owner_name].strip)
        end

        true
      end

We could really use this ability since we can reliably detect the expire date based on the signing identity p12.  Is this something we could add?

I made the quick change to move forward, but would appreciate thoughts on this:
@@ -112,7 +106,7 @@
       # Filter them
       certificates = certificates.find_all do |c|
         if Sigh.config[:cert_id]
-          next unless (c.id == Sigh.config[:cert_id].strip)
+          next unless (c.id == Sigh.config[:cert_id].strip) or (c.expires.strftime("%d/%m/%Y") == Sigh.config[:cert_id])
         end

I don't like to identity a certificate using the expiry date. Where does it say -i in the README? I can't find it.
Why don't you pass the certificate ID instead?

Ok, I get it now.  The README says
... or pass the name or expiry date of the certificate as argument: ...
which I mistook to be the signing id given a quick read.
The issue we are running into is that the id value that is retrieved from iTunes is not derived from the p12 file.  The expire time can be derived from the certificate using the openssl so, when the signing id's p12 is provided to us, we can inspect it and find the expire time and use that time to identify the signing certificate from multiple options during the automated provisioning profile creation.  The problem with using the 'certificateId' field is that we cannot determine how that field is generated by iTunes.  We could leverage Spaceship to go find it out when we bring a new client on, but, since we have to acquire the p12 file for the build process anyway, we'd prefer to use the expire time which can be explicitly and clearly determined instead of the iTunes specific id which we'd have to do additional work to retrieve.
All this is based on my inability to find a way to access the certificate ID without using Spaceship to access the account.  If you have other suggestions I'd be happy to investigate.
As far as your concerns, I agree that -i probably isn't the best way.  Would you be open to a broader idea of a 'filter' where we could take a few fields associated with the certificate to generate a description much like you do here:
Helper.log.info ("\t- Name: " + c.owner_name + " - ID: " + c.id + " - Expires: " + c.expires.strftime("%d/%m/%Y")).green

The we could then do a simple check of does the description contain the filter string.  This would allow the filter to be the id or the expire date.  It's less precise but would solve this use case without overloading the use of -i

Going to close this issue since it has been open for some time without any additional comments and doesn't seem to be a universal desired feature request. We'd be happy to review a PR that made certificate selection easier!


Constrain unbounded Tasks growth in rollover protocol

Instead of appending tasks indefinitely, constrain the rollover actor to only
allow one task per order ID at a time.
We implicitly rely on the invariants that we cannot get two decisions for the same rollover
at the same time.
Note: This is not a general solution, but it removes unbounded growth of the vector.

bors r+


Integrate ThemeProvider

@iamstarkov has been working on a unified styling solution https://github.com/iamstarkov/theming and react-jss needs to integrate it
Main issue https://github.com/cssinjs/jss/issues/488

I need feedback, pls take a look https://github.com/iamstarkov/theming/pull/2

Are you planning on adding support for the "statically themed" components as mentioned in #72 or will the theme only be available through props? iamstarkov/theming does not seem to force a specific approach.
Example below:
import React from 'react';
import injectSheet from 'react-jss';

const styles = theme => ({
    button: {
        padding: (props) => props.compact ? 5 : 20, // this will be dynamic ofcoure
        backgroundColor: theme.color.primary
    }
});

const Button = ({ classes, children }) => (
  <button className={classes.button}>
    { children }
  </button>
)

export default injectSheet(styles)(Button);

Statically themed component is basically taking a theme manually and putting it into props. Thats easy it seems.

Maybe I did not explain well enough, or maybe I did not understand correctly what you are trying to say. I think calling them statically themed components is where I went wrong.
What I would like to know is: when you add support for iamstarkov/theming, will the theme provided to its <ThemeProvider />:
a) be available only through JSS' function values (which well get the theme through props, I suppose), like so:
const styles = {
    button: {
        color: props => props.theme.color.primary
    }
};

injectSheet(styles)(Component);

or b) are you open to also allow passing a function to injectStyles that will receive the theme as its argument, like so:
const styles = theme => ({
    button: {
        color: props => theme.color.primary
    }
});

injectSheet(styles)(Component);

My apologies if this is what you were describing above 🙂. As a side-note, I'm happy to help out if you're open to option B but not planning on implementing it yourself.

both!

It is already implemented https://github.com/cssinjs/react-jss/tree/feat/theming and waiting for theming package consolidation with other cssinjs lib maintainers. We are building a unifying solution.

Great, thanks! I will wait a bit before I start hacking together something custom then. 😀

submitted initial theming support in #95, just ThemeProvider and withTheme for now

Done in next branch


Fix Compose Text

Improve vertical sizing
Fix 'can I send to this narrow' logic

Automated message from Dropbox CLA bot
@borisyankov, it looks like you've already signed the Dropbox CLA. Thanks!


[MXNET-857] Add initial NVTX profiler implementation

Description
This PR builds on-top of current profiler support to allow profiling via NVIDIA's NVTX APIs.  These extensions mark readable ranges in the NVIDIA Visual Profiler which helps show correlations between kernel launches and graph node executions.
Example shown here: https://user-images.githubusercontent.com/7443219/33946110-34296d18-e021-11e7-8d18-6d40b797405c.png  (The additional information enabled is in the 'Markers and Ranges' row.)
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this changs

Notes:

TODO: Integration test

One thing that would make the profile more readable is different colors for different ranges. Here is an example how to do this: https://github.com/NVIDIA/DALI/blob/master/dali/common.h#L149
You could for example use the first letter of the range as a very simple hash to the list of predetermined colors (don't do anything more fancy like actual hashing of the name since that would only introduce overhead which you don't want during profiling).

Great idea.  Totally agree.
On Fri, Aug 24, 2018, 8:46 PM Przemyslaw Tredak<EMAIL_ADDRESS>wrote:

One thing that would make the profile more readable is different colors
for different ranges. Here is an example how to do this:
https://github.com/NVIDIA/DALI/blob/master/dali/common.h#L149
You could for example use the first letter of the range as a very simple
hash to the list of predetermined colors (don't do anything more fancy like
actual hashing of the name since that would only introduce overhead which
you don't want during profiling).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-mxnet/pull/12328#issuecomment-415848462,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHGTE8BdwlbGFLcacaCC2pyFUX6eu1FOks5uUEobgaJpZM4WLDxa
.

@mxnet-label-bot [pr-work-in-progress]

@KellenSunderland Is the PR still WIP?

@vandanavk Yes, I'll loop around to this once I finish some other tasks.  Does it cause problems for your team to leave it in a WIP state?  For me it's quite handy to be able to have WIP PRs that I can put aside and resume later.

@KellenSunderland It doesn't cause problems for our team. But we do try to go through all the stale PRs in the repo to make sure someone is working on it and PRs are getting timely reviewed.

@KellenSunderland thanks for the contribution! Any updates on progress/ETA?

Hey @lupesko.  Sorry still in my backlog.  I've reviewed a few suggestions so no longer blocked on anyone.  I just need to find the time to finish this off.

@KellenSunderland pinging for update.

@KellenSunderland ping
feel free to close the PR and reopen it if you have time :)

@mxnet-label-bot add [pr-awaiting-testing]

@KellenSunderland ping for update and trigger ci. thanks!

This will be very helpful addition to profiler utility. @KellenSunderland - looking forward for the update.

@mxnet-label-bot add[pr-awaiting-review]

@mxnet-label-bot remove[pr-work-in-progress]

@rahulhuilgol Please review

@anirudh2290 @szha Could you please help review?

@anirudh2290 Ping again.

@KellenSunderland is this still WIP? name says it is, label says otherwise

Still making progress on this.  I believe the last TODO is either use static linking or copy make sure the nvExt library is copied to the lib path for testing on windows.  It's probably best for the customer to use static linking, so I'll try to implement that.

@sandeep-krishnamurthy ready for review if you've got a second.

LGTM. Will this be enabled by default (e.g. via pip)?

Yes, it'll be on by default for CUDA builds.

@eric-haibin-lin Any other questions about this PR?  I've done a bunch of perf tests offline and don't see any regressions, but don't mind doing some additional investigations if you'd like.  If you're ok with the change would you be able to set this PR to approved, I'm hesitant to merge without green checkboxes :-).

@KellenSunderland any update? look forward to this feature

Thanks @eric-haibin-lin.  Updated with a Makefile build.  Sorry it took a while, just wanted to make sure I tested it well with that build method.


System.InvalidOperationException: Dispatcher processing has been suspended, but messages are still being processed.

Microsoft PowerToys version
<IP_ADDRESS>
Installation method
Microsoft Store
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce
Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.22000.0
IntPtr Length: 8
x64: True
Date: 2023/3/7 16:22:51
Exception:
System.InvalidOperationException: Dispatcher processing has been suspended, but messages are still being processed.
at System.Windows.Threading.Dispatcher.WndProcHook(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at MS.Win32.HwndWrapper.WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at MS.Win32.HwndSubclass.DispatcherCallbackOperation(Object o)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)

✔ Expected Behavior
.
❌ Actual Behavior
.
Other Software
.

I'm also receiving this error when deleting characters after searching for items.
For example I type steam and it correctly shows results. I then backspace a few times and it crashes.
Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.22621.0
IntPtr Length: 8
x64: True
Date: 3/8/2023 5:04:57 PM
Exception:
System.InvalidOperationException: Dispatcher processing has been suspended, but messages are still being processed.
at System.Windows.Threading.Dispatcher.WndProcHook(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at MS.Win32.HwndWrapper.WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at MS.Win32.HwndSubclass.DispatcherCallbackOperation(Object o)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)

/dup https://github.com/microsoft/PowerToys/issues/18166


DOCS-13049 revision

@kmendiratta-crm  this is based on SE-20921
I did a few extra revisions to split it into two different tasks depending on whether or not you want to completely delete the gravity cluster

Closing this PR due to unverified commits issue. Changes were merged on https://github.com/mulesoft/docs-runtime-fabric/pull/582


Fixed missing format specification in asprintf

A compiler warning was being generated due to a missing format specification in an 'asprintf()' call in tpm2/src/util/file_io.c. This pull request fixes that error.
Note: Although, I did not change the configuration for the indent utility in the makefile, a number of white-space edits were generated by building the code, and included in this very simple pull request.

Replaced by PR #103

Replaced by PR #103


Shiny Problem + PID problrm

You can no longer make a Pokemon shiny, atleast
on my device. Also, the PID is a number, where it is a string of letters and numbers. Please fix these issues.

I have the same issue with not being able to check the shiny box for pre-existing Pokémon. It works fine if I do it on a brand new pokemon made with the editor.


[IMPROVEMENT] SVG support

This PR adds the SVG support for the app.
Here is a document (just for reference) that shows the RC v2 app performance measurements proving that there are no major impacts when supporting this image format: https://goo.gl/y9aWPm
Closes #618

@filipedelimabrito Does the app can now render this svg -> https://thedevconf.rocket.chat/assets/logo.svg?

@leonardoaramaki Yes, the app must have the ability to open any SVG image. We need only to pay attention with the header tag of the SVG file. For instance, this image starts with <?xml header tag, while this image starts with <svg header tag. In both cases that supported header tags are declared here and here.

@filipedelimabrito Nice. Have you/Can you test with both images? I ask this because we had something similar to that on legacy and still Fresco integration with https://github.com/BigBadaboom/androidsvg didn't work as expected


2.11 Bug fix for checking Date Field presence

Description
Adding a condition to check if the date Field is included in the selected fields
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

could you add a test case for this?

Sure, i'll add it in future PR


Updates ourplan page goals section

Summary
Briefly describe the purpose of this PR.
Description
Explain in detail what this PR is all about. This should include the problem you're solving, the approach taken, and any technical details that reviewers need to know.
This PR styles the goals section current theme of our website . Each goal is presented in increasing order of box size for visual hierarchy. The section is responsive, with adjusted padding and font sizes for different screen sizes, and includes optional fade-in animations for enhanced user experience.
Images
Include any relevant images or diagrams that can help reviewers visualize the changes, if applicable
Before

After

Issue(s) Addressed
Enter the issue number of the bug(s) that this PR fixes

Template should be strictly Closes <issue_number>
Example: Closes #1382

Prerequisites

[X] Have you followed all the CONTRIBUTING GUIDELINES?

Kindly resolve conflicts  @Meetjain1

Kindly resolve conflicts @Meetjain1

@Its-Aman-Yadav Resolved sir

Kindly resolve conflicts @Meetjain1

@Its-Aman-Yadav Resolved sir

Resolved sir @Its-Aman-Yadav

@Its-Aman-Yadav sir please review prs since the deadline is 7pm today so I request you to please review as soon as possible.  Thanks

@Its-Aman-Yadav @Kalivarapubindusree please review before 7pm

@Its-Aman-Yadav @Kalivarapubindusree please review before 7pm

@Its-Aman-Yadav

I apologize @Meetjain1 bit busy today

Thank you for all of your contributions


spfx webpart deploying to site features not site collection features

Category

[x ] Question
[ ] Typo
[ ] Additional article idea
[x] Checked items have no spaces between the [] (delete this line)

Question
Hello,
I have deployed an SPFx webpart which contains site columns and content type definitions. I see the feature in site features and not the site collection features. Is this expected? I want the feature to be available to the whole site collection, including sub-sites. Is there something I need to change?

Yes, that's expected. When installing the app, it's installed int he root site in the site collection.


Added template option for Notification

Supports OpenFin's requirement for a template
Defaults to notification.html relative to current URL
Fixed bug in uri utility

Maybe, but as it's ignored in browser and electron (and assuming you want your app to work in all three), you might as well just set the setting no matter what container you're in.
I guess I could have called it openfinTemplate, but in case we decide to start using the template in Electron too...


Percentage based allocation

So, the old AdMob allowed you to use percentage based allocation of ads, especially for House Ads. The new interface has removed this, more genius from google. Anyway, is there a way to pass an option to your plugin to request an add is filled 100%? This doc goes into it ...
https://code.google.com/p/google-mobile-dev/source/browse/examples/custom-events/src/com/google/example/ads/customevents/impl/PercentageHouseAds.java?repo=examples-android
The use-case is to only show house ads, never a mediated ad. Thanks for any suggestions.

to show house ads, you can use the admob plugin pro version, and set ad unit id to your google DFP (doubleclick for publisher).

Thanks, yes, that's exactly what we have done.
On Wed, Jun 10, 2015 at 5:02 AM, Raymond Xie<EMAIL_ADDRESS>wrote:

to show house ads, you can use the admob plugin pro version
https://github.com/floatinghotpot/cordova-admob-pro, and set ad unit id
to your google DFP (doubleclick for publisher).
—
Reply to this email directly or view it on GitHub
https://github.com/floatinghotpot/cordova-plugin-admob/issues/193#issuecomment-110678691
.

--
Christopher Schaub
http://chris.schaub.com


BUG: Support sparse arrays in scipy.sparse.csgraph.laplacian

Reference issue

Fixes #19149
What does this implement/fix?

Resolves AttributeError raised on sparse arrays in scipy.sparse.csgraph.laplacian (See #19149)
Additional information

This resolves/unblocks test coverage in scikit-learn/scikit-learn#27161
Tagging @perimosocordiae for review, as he reviewed the original issue

Thanks for the PR! While we're here, could you add a unit test that exercises the bug?

Thanks for the PR! While we're here, could you add a unit test that exercises the bug?

I need help with adding a test. I know where to add it - scipy/sparse/csgraph/tests/test_graph_laplacian.py
but I don't know whether to extend the existing tests or add a completely new one seeing these params
@pytest.mark.parametrize("arr_type", [np.array,
                                      sparse.csr_matrix,
                                      sparse.coo_matrix])

The arr_type parametrizing is the part of the test that should be extended. The existing _check... functions should not be affected by sparray input. And additional tests shouldn't be needed. So I think you should just add to the arr_type list with 2 more types:  csr_array and coo_array.  I think that shows up on lines 171, 247, 304. That should show that it breaks on the current system and is fixed by this PR.

What additional steps should be taken to get it merged or backport it to another version?

Merged. Thanks!


ERROR:Dataset Not Registerd

I am getting an error that DATASET IS NOT REGISTERED. PLEASE CREATE A METHOD TO READ IT

I am having the same problem as @uneverknwwhoim. Does anyone know how to fix it?

The problem disappears if we do not pass the name of the dataset to the console. The prgram will just takes the default one, which is berlin dataset.


AWS: Allow specification of a custom AMI

This is probably the least fair feature request I've ever made. About fifteen minutes ago, AWS announced support for encrypted root EBS volumes. Why don't you guys support that yet? :smiley:
I might file a separate request for that, but this ticket is a prerequisite: I need to be able to bring up this stack with a custom AMI -- one that's been encrypted using our KMS key. Seems like it'd be sufficient to add an ami parameter to the stack template that overrides the built-in CoreOS version selector if present.
As an added bonus, this would let folks specify newer versions of CoreOS if this repo hasn't merged them in quite yet.
Will probably submit a PR for this in the next couple days if the design of that feature makes sense to everyone.

You should be able to modify the AMI in the rendered cloud-formation template to include your custom image. However, I don't think this approach will work well with the core-update engine, as you will be getting OS updates from the upstream AMIs...

That seems fine -- in my case I'm not really using a "custom" image, it's just a copy of the CoreOS AMI that has its EBS encrypted with my KMS key. Unless I'm misunderstanding something about the update process, it should work just as well as it does on the upstream AMI.

You're right, the update process should just modify bits on disk rather than pull a new AMI in place. So hypothetically this might work (start with your initial encrypted AMI, then receive updates that modify those local bits).
I checked with one of the guys on our OS team, but he hadn't had a chance to look into how the encrypted root volume feature works, so couldn't really say for sure if this would work out-of-box. Feel free to open an issue at https://github.com/coreos/bugs to get more info on that specific functionality.
Still want to keep this open to track the requested feature of "specify custom AMI".
/cc @colhom any thoughts on this?

FYI, right now we're using kube-aws render then modifying the output before sending to CloudFormation. We are using encrypted root volumes... seems to be working fine.
I'm new to Go, but it sounds like you're interested in a patch so I'll look into work required to specify a custom AMI. If you aren't interested in a patch, just close this issue.

The kube-aws tool has been moved to its own top-level directory @ https://github.com/coreos/kube-aws
If this issue still needs to be addressed, please re-open the issue under the new repository.

FYI, we can now specify custom AMI ID thanks to https://github.com/coreos/kube-aws/pull/14


Config reader

https://jenkins.code.rdkcentral.com/job/github-triggered/39154/ started

https://jenkins.code.rdkcentral.com/job/build-component/77605/ started.

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/22746/

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/22747/

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/22748/

https://jenkins.code.rdkcentral.com/job/github-triggered/39206/ started

https://jenkins.code.rdkcentral.com/job/build-component/77646/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/6074/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/10118/ started.

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/22820/

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/22819/

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/22829/

https://jenkins.code.rdkcentral.com/job/github-triggered/39365/ started

https://jenkins.code.rdkcentral.com/job/build-component/77874/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/6202/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/10303/ started.

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23253/

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23252/

build-brcm-refboard Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/2280f944adb41dff95e300038a87fdf2

https://jenkins.code.rdkcentral.com/job/github-triggered/39372/ started

https://jenkins.code.rdkcentral.com/job/build-component/77891/ started.

https://jenkins.code.rdkcentral.com/job/github-triggered/39374/ started

https://jenkins.code.rdkcentral.com/job/build-component/77892/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/6207/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/10316/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/6208/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/10317/ started.

https://jenkins.code.rdkcentral.com/job/github-triggered/39376/ started

https://jenkins.code.rdkcentral.com/job/build-component/77894/ started.

https://jenkins.code.rdkcentral.com/job/github-triggered/39378/ started

https://jenkins.code.rdkcentral.com/job/build-component/77896/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/6209/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/10320/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/6210/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/10321/ started.

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23266/

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23267/

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23268/

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23269/

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23275/

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23274/

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23277/

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23276/

https://jenkins.code.rdkcentral.com/job/github-triggered/39384/ started

https://jenkins.code.rdkcentral.com/job/build-component/77903/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/10331/ started.

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23292/

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23293/

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23317/

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23318/

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/6217/ started

https://jenkins.code.rdkcentral.com/job/github-triggered/39390/ started

https://jenkins.code.rdkcentral.com/job/build-component/77922/ started.

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23357/

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23356/

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23358/

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23359/

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/6261/ started

https://jenkins.code.rdkcentral.com/job/build-component/78013/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/10436/ started.

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23605/

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23607/

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23606/

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/6282/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/10487/ started.

https://jenkins.code.rdkcentral.com/job/github-triggered/39476/ started

https://jenkins.code.rdkcentral.com/job/build-component/78075/ started.

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23756/

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23758/

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23757/

'['RPI3 Hybrid/MC']' Sanity test FAILURE, investigation pending.
Build Jobs: ['build-rpi-hybrid-wpe-dunfell'/'build-rpi-mc-wpe-dunfell']
Flash MC [FAILURE]
Flash HYB [FAILURE]
MC - Video Test skipped [FAILURE]
HYB - Video Test skipped [FAILURE]
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-rdkv-raspberrypi-flow/5713/

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/23874/

https://jenkins.code.rdkcentral.com/job/github-triggered/39624/ started

https://jenkins.code.rdkcentral.com/job/build-component/78299/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/10715/ started.

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/24210/

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/24211/

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/6408/ started

https://jenkins.code.rdkcentral.com/job/build-component/78313/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/10728/ started.

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/24220/

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/24221/

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/24222/

https://jenkins.code.rdkcentral.com/job/github-triggered/39817/ started

https://jenkins.code.rdkcentral.com/job/build-component/78620/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/6538/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/10989/ started.

build-brcm-refboard Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/f104b870b03b701b081784fe7aaaab24

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/24797/

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/24796/

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/11019/ started.

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/24873/

https://jenkins.code.rdkcentral.com/job/github-triggered/39960/ started

https://jenkins.code.rdkcentral.com/job/build-component/78774/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/11149/ started.

https://jenkins.code.rdkcentral.com/job/github-triggered/39964/ started

https://jenkins.code.rdkcentral.com/job/build-component/78776/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/6650/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/11151/ started.

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/25140/

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/25139/

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/25141/

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/25145/

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/25144/

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/25146/

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/25148/


Add basic example for ios

Fix

Need to add a new basic example for ios

Changes

Made a new example named ClickThePaintings in Examples/Debugging
Installed babel-plugin-module-resolver and add alias(@: ./src) in babel.config.js file
Installed react-router-native for routing
Implemented a function where click a painting and get brief information

How Verified
N/A
Notes
N/A
References
N/A

Good. Do not forget to use capital letter for first character of the commit message.

@alledy How about resolve requested changes?

@JeffGuKang She is traveling until next Wednesday. I think it will be resolved after next Wednesday. 😀


Terraform option gives no response

When creating a terraform template of the design i've produced gives no response. Compartment names exist in my tenancy as well.
Any info. on how I can debug this further? This is the error in the console:
POST http://localhost/okit/generate/terraform 500 (INTERNAL SERVER ERROR)

@Davidmh93 if you are running the docker version can you start it with the -it instead of -d and then execute the generate. You should then see what the server error is. If running the Vangrant you should be able to log in and check the log file in /okit/log.
Can you then post the error.

@Davidmh93 Can you test this with the new 0.7.0 release


Add option to configure rpcBus timeout #614

closes #614

Codecov Report

Merging #622 into rusk-ipc will increase coverage by 0.49%.
The diff coverage is 60.10%.

@@             Coverage Diff              @@
##           rusk-ipc     #622      +/-   ##
============================================
+ Coverage     54.96%   55.46%   +0.49%
============================================
  Files           156      159       +3
  Lines          9689    10473     +784
============================================
+ Hits           5326     5809     +483
- Misses         3723     3954     +231
- Partials        640      710      +70

Impacted Files
Coverage Δ

pkg/config/consts.go
0.00% <ø> (ø)

pkg/core/consensus/collector.go
0.00% <0.00%> (ø)

pkg/core/consensus/header/header.go
24.75% <0.00%> (ø)

pkg/core/consensus/reduction/testutil.go
46.42% <0.00%> (-1.72%)
:arrow_down:

pkg/gql/handler.go
0.00% <ø> (ø)

pkg/gql/httpserver.go
65.11% <0.00%> (ø)

pkg/gql/query/mempool.go
30.00% <0.00%> (-1.04%)
:arrow_down:

pkg/gql/query/transactions.go
63.86% <0.00%> (+2.07%)
:arrow_up:

pkg/p2p/peer/peer.go
69.93% <0.00%> (ø)

pkg/p2p/peer/responding/datarequestor.go
45.09% <0.00%> (-0.91%)
:arrow_down:

... and 60 more

Continue to review full report at Codecov.

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


Conversion Script's handling of special cases

The conversion script is unable to deal with the Microsof.Authorization namespace pipelines as they are not referenceing the main deploy.bicep file, but sub_deploy.bicep files and the like.
In the current version of the logic, these references are simply not updated. In a soon to be merged version, they are at least updated to .json. However, this won't solve the problem just yet, as templates like sub_deploy.bicep don't exist in the module's root, but its .bicep folder. As this folder is not just not converted to JSON, but actually deleted, we have to find a different solution.
The easiest might be to reference deploy.json for this namespace post conversion instead. However, as this may still cause problems (e.g. due to a lack of permissions), this must be documented.
Still better as no solution, I guess.

Resolved by the rework of the authorization namespace. #1033


Use Ubuntu Bionic for Travis CI

Update from Ubuntu 14.04 to 18.04

Thanks for updating this, @peterbecich! However, the Contributors libraries are all moving to GitHub Actions (via the setup-purescript action), and so we're going to be moving on from Travis. That update should affect this library sometime in the next month or so.


Incorrect guidance on removed feature, create a team has been removed from UI after refresh

This is now incorrect, you can not add a custom team or create and manage a team in the new refresh UI for the Viva Insights Teams app
_Manage your team
As a licensed Viva Insights user, you can create a team and view insights about them within the Viva Insights app. You’ll be able to add or confirm members on the Home page as explained below, but you can also make changes to your team on the Teamwork habits page.
If you’re not a manager, Viva Insights prompts you to Create your team._

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

ID: e76af468-1860-354e-710f-a62fe998b314
Version Independent ID: e76af468-1860-354e-710f-a62fe998b314
Content: Teamwork habits in Viva Insights
Content Source: Viva/insights/org-team-insights/reference/teamwork-habits.md
Service: viva
Sub-service: viva-insights
GitHub Login: @lilyolason
Microsoft Alias: v-lilyolason

Hi @saashby -- thank you for submitting. The Teamwork habits content is retired and redirects to our current Team insights documentation.


AuthenticatedRouteMixin

Hello,
On line 96 of the authenticated-route-mixin.js file, why is the transition to the authenticationRoute returned?
...
return this.transitionTo(authenticationRoute);
...
This made me to waste quite some time today. After I overrode the beforeModel in my app and removed the return statement, the application worked as expected.
Is there a reason for the return statement?

What kind of error were you seeing before removing the return in the overridden method?
I don't think there's a need/reason for the return and it should be fine to remove it.

No error, but when I extended the application route with the mixin the transition never happened. That was the funny part, no errors - no transition 😄

Hm, strange. We definitely shouldn't need the return though - want to submit a PR?


⚠ Processors has degraded performance

In b7de0c2, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 82 ms

Resolved: Processors performance has improved in 3de638c.


add unit tests for subagents (in multiagent branch)

We need to add unit tests for our subagents & related tools

[x] #36
[x] #35
[ ] unit tests for task critic
[x] #37
[ ] unit tests for refining-task curriculum agent
[x] unit tests for subagent-based tools (i.e. Iterator)

Communicate below whenever one of us start working on one of these tasks

added in PR #60. The curriculum agent isn't part of iterator code anymore, and task critic is now combined with code critic, so the remaining items in checklist is now irrelevant. all unit tests related to subagents are completed.


Broken Link in ReadMe

They Build status link in readme is broken
This is what it shows currently

can you specify which link you're talking about


fix: auth check in payout-methods and navbar overflow in payout-methods as well as calendar.

PR Fixes:

Issue 1: Auth check is performed on /payout-methods and only logged-in users are allowed to access this page
Issue 2: Both /calender and /payout-methods page contents are under the Navbar with proper margin and padding.

Resolves #1201
Before: (In Greater Detail)
After:
Solution Video For Issue 1:
https://github.com/user-attachments/assets/03986005-c1c4-4811-b386-c232fcabfeea
Solution Video For Issue 2:
https://github.com/user-attachments/assets/973c5767-45b4-4fbc-be9b-0073b8c74cc9
Checklist before requesting a review

☑ I have performed a self-review of my code
☑ I assure there is no similar/duplicate pull request regarding same issue

Additional Improvement Discussion
The commit I pushed in this PR moved /payout-methods and /calendar into the (main)/(pages)/ directory. I did not change any file contents except some import paths in /payout-methods as shown below:

I would suggest to remove the unnecessary auth-check present in the calendar route separately, as now all of the auth-checking is already performed by the layout.tsx in the (pages) directory:

But now the /calendar still has an unnecessary auth check as shown below:

My Proposal
I propose to remove the second unnecessary auth-check in /calendar, I have already tried and tested it locally and it works fine. It would be great if I was allowed to add this change as well. Otherwise, the original PR anyways solves the issue #1201.

@rahulbarmann Bhai its empty PR

@siinghd Sorry but may I know what you mean by empty PR? I did commit the changes in this PR but the changes are very minimal (only folder location change and import paths) to fix the issue:

On top of this, I additionally requested permission to delete the unnecessary auth-check in /calendar in the "Additional Improvements" section of the PR comments.

There's nothing changed here. @rahulbarmann can u double check again and raise another pr? Closing as of now


Create Key Pairs From Private Key

A nice little guide dumbing this down for us using cryptoJS:
http://ramkulkarni.com/blog/encrypting-data-with-crypto-js-in-javascript/

This will help especially: http://stackoverflow.com/questions/27248644/how-to-decrypt-a-cryptojs-aes-encrypted-message-at-the-java-server-side

Oops, thos use RSA, need to use PGP: https://github.com/openpgpjs/openpgpjs


Adds a brush for zooming to stats timeline

Now that I have the brush zoom in place on speed.yjit.org (thanks!) I've added text to the view mentioning it.
Also, what do you think about scaling the y axis according to what points are visible? That would make brush-zoom a possible solution to a current problem where you can't easily distinguish between two lines because of older data with much higher values...


Missing components (wifi_provisioning, spiffs, mqtt)

Someone mentioned this repo for using with esp-idf and arduino as a component. But when I try to use it as a custom platform in platformio it can't resolve multiple components that are mentioned in https://github.com/espressif/arduino-esp32/blob/master/CMakeLists.txt
I've checked the actual ESP-IDF 5.1.4 and they are present, official platformio espressif32 platform also has them.
Just manually copied those components as a workaround.

As designed and wanted.


Replace ILMBASE_THREADING_ENABLED with ILMTHREAD_THREADING_ENABLED

A remnant vestage from v2.X

Thanks! I ran into that one as part of #1473.


Switch from Update() to Patch()

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #63
Special notes for your reviewer:
Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.
Release note:

NONE

The unrelated changes in this PR are gofmt updates.

/lgtm
/assign @chuckha

/lgtm


Calico role

I have installed latest MANTL on bare-metal ( Centos 7.2), I added etcd and calico to their respective roles to install them. Even though the ansible script ran successfully, when I am passing a marathon job with CALICO_IP and CALICO_PROFILE none of them are actually getting applied to the docker container.
I am concerned about this in the calico role,
"DOCKER_HOST": "localhost:2377" is passed as --executor-variables to mesos.
The executor_environment_variables file is copied to /etc/mesos-slave/executor_environment_variables and mesos-slave service is restarted, but there is no mesos-slave service running on workers it is renamed to mesos-agent and it is not fetching any arguments from /etc/mesos-slave
Secondly, even if DOCKER_HOST is passed as localhost:2377 how would that and "-H unix:///var/run/docker.sock" interact?

"-H unix:///var/run/docker.sock" overrides DOCKER_HOST settings. There is a Mesos issue for that:
https://issues.apache.org/jira/browse/MESOS-3821
There is also a related problem with powerstrip (#1317)

@Zogg Is this relevant after #1133?

Nope.


Stop the execution when all jobs have finished

Make the program exit (gracefully) once all the jobs have finished / no jobs are running anymore. Something like that. Give it some time to think or ask use if he wants to close the program. Idk, to be discussed.

Issue addressed with commit 21406e0


Introduce the ability to start taking state snapshot

Description
This PR adds a new crate sui-snapshot for taking state snapshot.
Test Plan
Added unit tests

comment


[CAPT 1433] - Refactor/user journey db performance

Optimize user journey queries
CAPT-1433

LGTM, it might would be useful to show before/after stats as well to establish baseline

As an example by executing twice per case: bundle exec rspec ./spec/features/combined_teacher_claim_journey_with_teacher_id_check_mobile_spec.rb
I got the following results:
Before
Finished in 1 minute 10.89 seconds (files took 1.72 seconds to load) 9 examples, 0 failures
Finished in 1 minute 9.37 seconds (files took 1.71 seconds to load) 9 examples, 0 failures
After
Finished in 59.1 seconds (files took 1.74 seconds to load) 9 examples, 0 failures
Finished in 57.02 seconds (files took 1.7 seconds to load) 9 examples, 0 failures


Update remodal.css

Added the touch-action style to solve the Windows Phone 8 scroll issue

thanks


par2cmdline: 0.6.13 -> 0.7.3

Motivation for this change
update
Things done
Please check what applies. Note that these are not hard requirements but merely serve as information for reviewers.

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

n/a Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Thanks!


About repo's order

could you resort to the repositories according to their starts(or something else)?

Unfortunately this isn't allowed in the current format, maybe that's possible with a third party website.

Unfortunately this isn't allowed in the current format, maybe that's possible with a third party website.

Maybe a python crawler can be written to solve this problem.

Maybe this website can help you. https://neovimcraft.com/
Sorting the repositories according to their stars, updating data or something change frequently may not be a good idea.


remove 'friedelino' as a maintainer

Description of changes
Remove myself as maintainer because i will switch to a new github-account & i don't really do much maintaining on these packages.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

omg! did not update my branch before. excuse me please, will fix that


Upgrade node version to 12

Description
These changes make the necessary updates to be able to use node version 12 to build and run code in this repo.
Individual commits contain detailed information regarding each change.
One of the changes was to introduce a dependency on @celo/phone-number-privacy-common 1.0.26-dev I think maybe this needs to be published and the dependency adjusted to 1.0.26. It looks like this could be the reason for the google cloud build failing.
I'm also unclear as to what ci/circleci: protocol-test-release-snapshots Expected — Waiting for status to be reported is required to make this step complete.
Tested
yarn install and build work locally & circleci is passing.
Related issues

Fixes #7084

Backwards compatibility
The backwards compatibility will depend on the specific packages that were updated, I guess some of these changes will be backwards compatible but some wont.
Documentation
SETUP.md has been updated to instruct people to use node version 12, I think there is some more doc that needs updating though, so I don't consider this part finished yet.

@medhak1 I think I need to release a new version of phone-number-privacy-common in order to get the google cloud build working, would appreciate your insight here :)

Closing, this because this work was covered by #7747 #7833 #7851


[SPARK-47199][PYTHON][TESTS] Add prefix into TemporaryDirectory to avoid flakiness

What changes were proposed in this pull request?
This PR proposes to set prefix for TemporaryDirectory to deflake the tests. Sometimes the test fail because the temporary directory names are same (https://github.com/apache/spark/actions/runs/8066850485/job/22036007390).
File "/__w/spark/spark/python/pyspark/sql/dataframe.py", line ?, in pyspark.sql.dataframe.DataFrame.writeStream
Failed example:
    with tempfile.TemporaryDirectory() as d:
        # Create a table with Rate source.
        df.writeStream.toTable(
            "my_table", checkpointLocation=d)
Exception raised:
    Traceback (most recent call last):
      File "/usr/lib/python3.11/doctest.py", line 1353, in __run
        exec(compile(example.source, filename, "single",
      File "<doctest pyspark.sql.dataframe.DataFrame.writeStream[3]>", line 1, in <module>
        with tempfile.TemporaryDirectory() as d:
      File "/usr/lib/python3.11/tempfile.py", line 1043, in __exit__
        self.cleanup()
      File "/usr/lib/python3.11/tempfile.py", line 1047, in cleanup
        self._rmtree(self.name, ignore_errors=self._ignore_cleanup_errors)
      File "/usr/lib/python3.11/tempfile.py", line 1029, in _rmtree
        _rmtree(name, onerror=onerror)
      File "/usr/lib/python3.11/shutil.py", line 738, in rmtree
        onerror(os.rmdir, path, sys.exc_info())
      File "/usr/lib/python3.11/shutil.py", line 736, in rmtree
        os.rmdir(path, dir_fd=dir_fd)
    OSError: [Errno 39] Directory not empty: '/__w/spark/spark/python/target/4f062b09-213f-4ac2-a10a-2d704990141b/tmp29irqweq'

Why are the changes needed?
To make the tests more robust.
Does this PR introduce any user-facing change?
No, test-only. There's a bit of user-facing documentation change but pretty trivial.
How was this patch tested?
Manually tested. CI in this PR should test them out as well.
Was this patch authored or co-authored using generative AI tooling?
No.

Merged to master.


Somehow store whether the contents/thumbnail sidebar is open

Proposal for a small UX enhancement:
I just had the experience that I:

opened a PDF file
opened the contents sidebar
navigated away from the PDF by clicking a link in it
navigated back to the PDF by clicking the browser's Back button

Result: the contents sidebar is not open anymore.
I think this issue might be solved relatively easily by adding #sidebar-content/#sidebar-thumbs to the URL when the sidebar is open. (And then of course changing the UI depending on that.)
But there might be better ways.

Might be fixed in a similar way as #5949.

#5949 is not working now - you can try doing it in the demo link. After reload, the viewer does restore the last scroll position, but not the rotation angle. (This is the same issue as #5927)
That may be because almost all changes made in #5949 have been removed.
I redid those changes, and it started working.

Anyone working on this? If no, I would like to try to fix this issue. Thanks!

Anyone working on this?

Not that I know of, feel free to take it on!

@brendandahl, since the user moves away from the PDF in this issue, would it be good idea to store the state of sidebar in the local-storage of the browser and initialize sidebarOpen variable in viewer.js from local-storage if available. What do you think? Please let me know.

We have code already built to persistently store preferences. As Tim suggested above, this will be very similar to the fix in #6980 and I'd suggest looking at that.

Thanks. I will have a look and try to make changes locally and test them.


Can not get Access token for the Apps create in German cloud.

Hi,
this doesn't seem to work for the Apps created under German cloud.
https://developer.microsoft.com/en-us/graph/docs/concepts/rest
One of my customer uses an 'https://portal.microsoftazure.de',
he says German cloud has special data security feature, and created an Azure app.
For this app we get errors to get Authorization/AccessToken.
Are there any additional parameters to add to the request?
Thanks in advance.

turned out that I need to change the endpoint as described here.
https://docs.microsoft.com/en-us/azure/germany/germany-developer-guide


This does not work on python3

https://github.com/asterisk/ari-py/issues/17
Similar problem, with urlparseon Python3. Should fix the problem.


Feature request : Support send multiple arguments to the compute function

Use case
Often in apps that require some "computation heavy" algorithms (like computer graphics etc), we have to use the compute method here and there to avoid blocking the ui and run them in an isolate. Most of the time those algorithms are not single arguments functions and thus doesn't work with compute. A work around is to put those arguments into a Map  as explained in this SO response. Modify those algorithms / or wrap them to accept a Map instead of the type we intended at the moment of designing the API is not very convenient and error prone.

Proposal

Thus I would like the flutter team to modify compute (breaking change I think) or create an other convenient method like compute that accepts multiple arguments. This feature might or might not need this request on dart language design repository to land;

Why isn't this still a thing? It's so annoying to create and parse some boilerplate classes just to pass multiple arguments :roll_eyes:

You can just create a helper data structure that encapsulates all the arguments you want to pass in.

...and create another 3x lines of boilerplate code -_-
luckly i think now Isolate.run() solves this problem (?)


Update README.md

Update README.md to current status

I agree.  The original repo should be have reference to forks.


action set key/macro doesn't work on Logitech G102

Information

ratbagd version: 0.17 (installed via aur/libratbag-git)
Distribution: Arch
Kernel version (ex. uname -srmo): Linux 6.4.2-arch1-1 x86_64 GNU/Linux

Device info (if applicable):
$ ratbagctl <device> info
hollering-marmot - Logitech G102 LIGHTSYNC Gaming Mouse
             Model: usb:046d:c09d:0
       Device Type: Mouse
 Number of Buttons: 6
    Number of Leds: 1
Number of Profiles: 1
Profile 0: (active)
  Name: n/a
  Report Rate: 1000Hz
  Resolutions:
    0: 400dpi
    1: 800dpi (active) (default)
    2: 1600dpi
    3: 3200dpi
    4: 50dpi (disabled)
  Button: 0 is mapped to 'button 1'
  Button: 1 is mapped to 'button 2'
  Button: 2 is mapped to 'button 3'
  Button: 3 is mapped to 'button 4'
  Button: 4 is mapped to 'button 5'
  Button: 5 is mapped to 'resolution-cycle-up'
  LED: 0, depth: rgb, mode: cycle, duration: 5500, brightness: 66

Describe the bug

Run ratbagctl hollering-marmot button 4 action set key KEY_LEFTMETA. Other buttons and other keys are also affected.

 $ ratbagctl hollering-marmot button 4 action get
 Button: 4 is mapped to key 'KEY_LEFTMETA'

But the mapping doesn't work as expected.  But I cannot reproduce it again.
ratbagctl hollering-marmot button 4 action set macro KEY_A KEY_B doesn't display anything but it has no effect and the ratbag logsratbag error: Error while writing macro for button 4
ratbag error: hidpp20: failed to update button (-22)
ratbagd error: error committing device (-1000)

Same with a G502 under fedora37
Button 4 always add KEY_LEFTCTRL KEY_LEFTALT and Button 3 always add KEY_LEFTALT to the key
If you have any workarround or alternative I'm intersted


BUGFIX: Lock events table before committing

There are situations where two processes try to write data to the event
store simultaneously.
|sequence number|event                 |source        |
|1              |do first thing        |first process |
|2              |do second thing       |first process |
|3              |do something different|second process|
|4              |do third thing        |first process |
|5              |do fourth thing       |first process |

The second process starts second but will finish first.
So there is a point in time the event store looks like this:
|sequence number|event                 |source        |
|3              |do something different|second process|

If now a listener starts catching up, it will be remered as
highest applied sequence number = 3.
As soon as the first process finishes, another catch up happens starting
at sequence number 4.
So in this very situation, sequence numbers 1 and 2 are recorded and
even recorded in the right order but never played.

The description makes sense to me and I guess the implementation too (though I can't really judge for a lack of db internal knowledge)..
A test would be great of course, but I can imagine that this case is really hard to cover..
It should be a BUGFIX though, right?

And made it a BUGFIX…

But that would mean that transaction isolation is broken

good point.. But isn't that always the case since it's two separate PHP processes?

But that would mean that transaction isolation is broken

good point.. But isn't that always the case since it's two separate PHP processes?

But it's one database :) the reason you have transaction isolation exactly is that you can have multiple processes do stuff without (or with various levels of) interfering each other.
The default isolation level for mysql seems to be "repeatable read", which is exactly what we want - each transaction can only see the state of the database as of the start of the transaction:
https://dev.mysql.com/doc/refman/5.6/en/innodb-consistent-read.html#sa30444632

"Suppose that you are running in the default REPEATABLE READ isolation level. When you issue a consistent read (that is, an ordinary SELECT statement), InnoDB gives your transaction a timepoint according to which your query sees the database. If another transaction deletes a row and commits after your timepoint was assigned, you do not see the row as having been deleted. Inserts and updates are treated similarly."

https://dev.mysql.com/doc/refman/5.6/en/innodb-transaction-isolation-levels.html
Now how I understand the described effect is that the first process has not yet committed it's transaction for the four events, second process started in between and committed already. Then a projection that reads the database can only see the committed event of the second process. BUT, as stated above, the second process should likewise not see the events 1+2 and hence end up with a sequence number of 1 instead of 3, in which case the Projection would be correct (and the commit of the first process should fail). Hence, I must be missing something.
What this change does, is that it makes all other "select for update" wait for every previously started transaction to finish. This means, that in the scenario above the following happens:

first process starts, locks table, assigns itself 1 as next sequence number
second process starts, is blocked
projection runs, sees nothing and stops
first process commits, unlocks table
second process locks table, sees highest sequence number 4 and assigns itself 5 as next sequence number
projection (maybe) runs again, sees events 1-4 and starts processing
second process commits, unlocks table
projection finally runs again, sees event 5 and processes that

https://dev.mysql.com/doc/refman/5.6/en/innodb-locking-reads.html
From the projection POV this is sane, but we have effectively serialized the writes with locks (so no more concurrency errors, but waiting processes potentially piling up). As soon as some process starts updating/deleting events, this might even become a deadlock.

I guess @stephanschuler would be the one to know. Or to find out. 😎

@stephanschuler  ping

I'd suggest we close this until this pops up again with a reproducible scenario


Add E2E test for channel display modes

Summary
Add new E2E test cases for channel display mode

[x] Full width
[x] Fixed width, centered

Ticket Link
https://github.com/mattermost/mattermost-server/issues/10033
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)

To test fixed width in visual, the center channel requires width bigger than 1000 but it has 979, so I can validate only its CSS property.
Can I adjust the display size to see this spec?

Oh well, I can use cy.viewport() API to adjust in scope

There is an issue to handle cookies in cypress. jump to convo
And this will also fix other tests that have not been verified correctly.

Woot, thanks @cometkim!! Will defer reviews on this PR to devs who are more familiar with the changes.
Thanks for helping contribute! 👍

LGTM ... just had the one question above. :)


Bladebit with ramplot

Good morning Harold,

Can you please provide some sample strings in using ramplot
Everytime I down your version I have to change the name (too many versions out there)
Can you please provide the string to get help on ramplot as you can see the error.
C:\bb>bladebitv201h ramplot -h
Fatal Error:
A farmer public key must be specified.
C:\bb>
Thanks again
Roy
Just in case you wanted to know my system spec ( Dell T7610 512gb memory, C:\ Samsung 850EVO 1tb, 2 PCIE 1tb for temp drives and a 18tb end disk to collect the plots :-)

Hi Ro, you can use the ramplot command basically as you did with with BB v1. The only thing that applies to it are the global options, which are shown with bladebit -h.
You can use something like:
bladebit -f ... -p ... -t 32 ramplot D:/my/output/dir

I will mark this issue as a bug to track ramplot not showing the help string.

Thank you Harold,
Help and samples are always great...
Roy

Harold,
This worked
.\bladebitv201h -n 10 -f fffffffffffffffffffffffffff -c cccccccccccccccccccccccc ramplot p:\ram
( I rename your files to keep track)
Two questions is there a plot number max in -n ? And when the plots are finished and it go back to the prompt is there a time delay before you can run the same string again. When it gets to the prompt and you up arrow and enter it crashes after 5 seconds, up arrow enter crashes, let it sit for 20 seconds up arrow enter then it starts running. is there some buffer cleanup routine etc. running at the end?

Good morning Harold,
This morning came down stairs to see the ten plots created. I up arrowed to run another 10 plots and 4 second crash to the prompt, Task manager show no othe rapp running and more than 505 mb free, this time, up arrow crash and up arrow crash. I rebooted the node let it sit and tried to run the same string saved in a text file no go.
Any other ideas on what to look into, When  the app runs it looks for memory and allocates etc after some seconds you can see the memory in task manage go up. But this morning I can't the app to run. Open for ideas. I can make plots with plot manager so I do know that machine does work.
Roy   (Blade-bit 2.5???)

@Drhicom
I am running
Version      : 2.0.1
Git Commit   : 9fac46aff0476e829d476412de18497a3a2f7ed8

If you have 2 CPU installed, turn on "Node Interleaving" in BIOS under memory settings and add --no-cpu-affinity before -f.
Regardless of CPU count, add -w -v to your command line before ramplot. -w will touch all the allocated buffer pages before generating F1 and in my case noticably decreases overall plot time. I also suggest adding -t # to be your logical processor count minus 2. For example I have 2x12C/24T CPU == 24C/48T so I run -t 46 for best performance without starving OS for CPU time. I tested multiple plots using 40/42/44/46/48 and 46 is fastest average.
.\bladebit.exe --no-cpu-affinity -t 46 -f BLAHBLAH -c YADAYADA -w -v ramplot O:\BBOutput
I do not use -n rather I execute a batch loop and log console output to file for each plot via Powershell wrapper call. Negligible overhead for the benefit of per-plot logs I can compare and average time over many runs. My loop is infinite, but you could add a counter and EXIT when desired. After each plot completes, the allocated buffer pages are released which takes 5-10 seconds before Powershell closes, the CMD loop repeats and a new plot process starts.
@ECHO OFF
PUSHD D:\Plotter

:NewPlot
SET hr=%time:~0,2%
IF "%hr:~0,1%" equ " " set hr=0%hr:~1,1%
SET DATETIME=Log_%date:~-4,4%%date:~-10,2%%date:~-7,2%_%hr%%time:~3,2%%time:~6,2%
SET LOG_FILE=logs/%DATETIME%.log
IF NOT EXIST logs MKDIR logs

START /WAIT powershell ".\bladebit.exe --no-cpu-affinity -t 46 -f BLAHBLAH -c YADAYADA -w -v ramplot O:\BBOutput | tee '%LOG_FILE%'"
MOVE O:\BBOutput\*.plot O:\MoveToOtherMiner

GOTO NewPlot

@spleen911

I didn't see any "Node Interleaving" in the BIOS setup on this Dell T7610 machine. Even using your string, I have to execute  the command 4 to 5 time before it starts and grabs. I wait 5 seconds between up arrow and enters and this is what I'm trying to understand.

I will give it a try, but again, it runs and make plots once it gets started. You just have to keep up arrowing the command and run the same thing until it catches. Its too crazy, like typing dir c:\ and have it give you a response on the 5th try.

If OS commands don't even execute from console cache, sounds like a problem unrelated to BB. Best of luck.


Add modc method to integer types

Everything is green besides osx, because it has llvm 7 in its homebrew cellar.


User preferences

Allow users to configure their preferences, with a separate collection in the DB dedicated to this.

@TejasOS it would be cool if you could manage the DB collections and such and I'll manage the user input. Models go in the models directory and helper functions can go in helpers/db.ts.

@xlq902 if you want to take a look at this issue, maybe see if you can get multi-select menus working for selecting different preferences. The preferences themselves can be found in models/userConfig.ts. Take a look here to learn more. If you can get the Discord side working, I'll work on the DB stuff.

See https://github.com/ADawesomeguy/AwesomeSciBo/tree/feature-preferences for progress

DB time 💪🏽

FINALLY done with 48243e61f0eb3c7ae9186c67bf6e4e472a0aa00d.


PRs not getting assigned properly to reviewers

sometimes only @yangwao gets assigned to review

before it was selecting people from code review group, now its selecting from internal dev

excepted behavior?
https://github.com/kodadot/nft-gallery/commit/d791e8c242e9c176adafe221ac678145e1f1d40d

excepted behavior? d791e8c

but why some PRs gets only one person assigned

but why some PRs gets only one person assigned

@roiLeo was it solved? Because I'm manually setting reviewers for every PR
should viki and yangwao be requested for code review?


No gap in between sections on the page

Every other section has a consistent gap between them, except between what's happening and stay tuned to the commons, and the varying shades of mustard strike me as possibly not very accessible

I believe that's in the design that was agreed to. You can check the page out with a color blind simulation tool for starters. I'll leave it up to @ageeroms to confirm the design.

It was the design agreed to. Yellow is wrapped around the posts and newsletter to indicate they're all about what's happening with CC:

Also, I don't think their is an accessibility limitation here.  If you were colour blind, you could recognize the content is connected as well. See

okay, thanks


dbvis: Drop 32bit arch, update 64bit url and hash,

Closes #9313

[x] I have read the Contributing Guide.

/verify

the package without JRE is more suitable for Scoop.

Ok, it's fixed now.

Try the manifest from my personal bucket if you want to:
scoop bucket add MyScoop 'https://github.com/Zliced13/MyScoop'

scoop install dbvis_fixed

/verify

/verify

/verify


Remove IdeaFlow tool from Release CI

Description
In the recent past,  IdeaFlow Tool has only provided limited verification value since it is not used much within New Relic. Let's remove the IdeaFlow Tool and prepare for our CI to be able to include Crossroad. Please see the link for the ticket
Internal Only
Expected Behavior
Improve our CI with deployment test of Ruby Gem in Crossroad.

Jira tickets related to getting this set up for discourse plugin as well:
internal only
internal only

Jira tickets related to getting this set up for discourse plugin as well:
internal only
internal only


CodegenError should have optional context, rather than pretty-ifying it into a string

pretty_error should be replaced with methods on CodegenError that mutate the error and provide the context that pretty_error otherwise adds, and the CodegenError's Display impl should print what pretty_error creates when it has the necessary context.
These context-adding methods would be similar to how the wast crate adds file contents and filename context to errors with these methods:

https://docs.rs/wast/24.0.0/wast/struct.Error.html#method.set_text
https://docs.rs/wast/24.0.0/wast/struct.Error.html#method.set_path

Then we can delete pretty_error, and avoid using string as an error type in more places (notably in cranelift-filetests and its pretty_anyhow_error workaround).

pretty_error takes a reference to isa and func. Doing this would require CodegenError to borrow from them.

The idea is to take copies of the data gathered from those things and store them into the error, rather than borrow those things.

pretty_error needs the entire function, as the error shows the full function source. From the target isa it needs the register_info and encoding_info.

I'd like to take this issue. Is anyone looking into this?

It's all you @taegyunkim!

This issue is still relevant, but looking over the past discussion and the implementation in cranelift/codegen/src/print_errors.rs, I'm not convinced it should be labeled "easy". You're welcome to give it a shot anyway and we'll help however we can!


fix: add back projectName methods for monitoring clients

We accidentally merged an update which broke backwards compatibility with some of the Monitoring Clients by removing the projectName static method. This adds them back.
See https://github.com/googleapis/google-cloud-php/pull/4917

Looks good to me! The failures on 5.5/5.6 are due to: https://github.com/protocolbuffers/protobuf/issues/9746


Installing PHP modules

I am using Wocker to try out using a theme and plugin that my company bought for a client, that also requires the PHP Zip module to be installed. Running inside the Wocker environment, The theme is giving an error message when trying to access the Tools section (in wp-admin) that says Oops, Unyson Backup requires PHP Zip module but it is not enabled on your server. If you are not familiar with PHP Zip module, please contact your hosting provider.
I have been able to shell into the vagrant box's docker container to install Pear and try to install the zip module. I have so far been unsuccessful, but I thought I'd reach out and ask if there is a recommended way to configure php extensions through the Vagrantfile in this repo? Apologies if this question doesn't make sense due to my unfamiliarity with Vagrant.

@andrewtcymmer Thank you for using Wocker.

if there is a recommended way to configure php extensions through the Vagrantfile in this repo?

To configure PHP extensions, you have to customize the Dockerfile. It requires knowledge of Docker.
I tried adding php7.0-zip. Could you test it and tell me the result?

Make sure your VM is running. If not:$ vagrant up

SSH into the machine:$ vagrant ssh

Make sure no container is running. If not:wocker ~ $ wocker stop

Run a container (e.g. zip-test) using wocker/wocker:zip imagewocker ~ $ wocker run --name zip-test wocker/wocker:zip

This will pull the wocker/wocker:zip image from Docker Hub automatically.
Test if your theme and plugin work.

@ixkaito I have verified that the docker image wocker/wocker:zip is working correctly with my theme and plugin. Thank you for your prompt and accurate response.

@andrewtcymmer I added the PHP  zip module to the official Wocker image on v1.4. After wocker update, you can just wocker run without wocker/wocker:zip to test your plugin.


dummy commit to trigger PR check to verify WhiteSource Security Check

Signed-off-by: Tianle Huang<EMAIL_ADDRESS>Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #1754 (cb865f2) into main (faf26d1) will increase coverage by 5.20%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #1754      +/-   ##
=============================================
+ Coverage     94.79%   100.00%   +5.20%
=============================================
  Files           169         6     -163
  Lines          3536       105    -3431
  Branches         26        19       -7
=============================================
- Hits           3352       105    -3247
+ Misses          181         0     -181
+ Partials          3         0       -3

Impacted Files
Coverage Δ

tests/jenkins/jobs/BuildDockerImage_Jenkinsfile

src/run_build.py

src/build_workflow/build_artifact_checks.py

src/assemble_workflow/bundle_url_location.py

src/build_workflow/builders.py

...orkflow/integ_test/integ_test_runner_opensearch.py

...flow/dependency_installer_opensearch_dashboards.py

src/ci_workflow/ci_check_list.py

src/ci_workflow/ci_check_list_source.py

src/manifests/distribution.py

... and 153 more

Continue to review full report at Codecov.

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

This is just a dummy PR for double checking the WhiteSource security check in this PR https://github.com/opensearch-project/opensearch-build/pull/1748

revert https://github.com/opensearch-project/opensearch-build/pull/1700 and see if the security check will pass

Known issue https://github.com/opensearch-project/opensearch-build/issues/1533
Close this experimental PR


Refactor Enums to reactive approach

[ ] Breaking change? (if so, please describe the impact and migration path for existing applications:)

What changes did you make? (Give an overview)
Fixes https://github.com/opendatadiscovery/odd-platform/issues/614

Move EnumValueMapper to Mapstruct
Implement reactive repository for EnumValue
Refactor EnumValueService to include update logic

I have yet to add test coverage for the changes above, hence the draft PR.
Is there anything you'd like reviewers to focus on?
How Has This Been Tested? (put an "X" next to an item)

[ ] No need to
[X] Manually(please, describe, when necessary)
[x] Unit checks
[x] Integration checks
[ ] Covered by existing automation

Checklist (put an "X" next to an item, otherwise PR will fail)

[X] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation (e.g. ENVIRONMENT VARIABLES)
[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
[ ] Any dependent changes have been merged

Check out Contributing and Code of Conduct

@DementevNikita this is mergeable assuming there won't be any new feedback


the pre-scale knob

Hello! I've been using the iO-808 and I can't modify the pre-scale knob. Is it available?

The Pre-Scale knob hasn't been implemented and I personally don't find the complexity it brings to the implementation to be worth the functionality it adds. Of course I'm fine with someone else taking on the work tho!


fix: use eventify internally to avoid extra events

Allows users to require() the findOrAdd, and have events.

Besides the one little comment, this is good to merge

I've removed the JSDoc comment from lib/find-or-add-with-events.js

merged via 07bd9dd (had to rebase on master) :+1:


Wrong obsolete information or missing method?

Using version 3.0
Having this code:
var hasAuthorize = context.ApiDescription.ControllerAttributes().OfType<AuthorizeAttribute>().Any();
Will give you the following obsolete warning:

Deprecated: Use TryGetMethodInfo

This is however not possible since TryGetMethodInfo will return

controllerActionDescriptor.MethodInfo

and the original method would return

controllerActionDescriptor.ControllerTypeInfo

as can be seen here:
https://github.com/domaindrivendev/Swashbuckle.AspNetCore/blob/9e19afa50d70c9ef646fa41ee65290163de0e20a/src/Swashbuckle.AspNetCore.SwaggerGen/Generator/ApiDescriptionExtensions.cs#L19
Solution:
I would suggest adding the following method to the class:
        public static bool TryGetControllerTypeInfo(this ApiDescription apiDescription, out ControllerTypeInfo controllerTypeInfo)
        {
            var controllerActionDescriptor = apiDescription.ActionDescriptor as ControllerActionDescriptor;

            controllerTypeInfo = controllerActionDescriptor?.ControllerTypeInfo;

            return (controllerTypeInfo != null);
        }

and change the obsolete text to

Deprecated: Use TryGetControllerTypeInfo

To get the controller attributes you just need to dig a little deeper on the MethodInfo:
var controllerAttributes = context.MethodInfo.DeclaringType.GetTypeInfo().GetCustomAttributes(true);


Fix a very dengerous bug created by #55

Update some UI.

Resolved / Related Issues
Items resolved / related issues by this PR.

Closes #issue...
Related #issue...

Details of Changes
Add details of changes here.

Added...
Fixed...

Validation
How did you test these changes?

[ ] Built and ran the app
[ ] Tested the changes for accessibility

Screenshots (optional)
Add screenshots here.


Unable to programmatically open multiple accordion items.

Bug report
Current Behavior
Unable to programmatically open multiple accordion items.
Expected behavior
Ability to programmatically open multiple accordion items.
Reproducible example
CodeSandbox Template
Clicking on the button Open item-1 & item-2 should open both item-1 and item-2 but it is not doing it.
Suggested solution
I followed the advice in this discussion thread, but it doesn't seem to work.
https://github.com/radix-ui/primitives/discussions/824#discussioncomment-1194231
Additional context

Your environment

Software
Name(s)
Version

Radix Package(s)
@radix-ui/react-accordion
latest

React
n/a

Browser
Chrome
Version 114.0.5735.198 (Official Build) (64-bit)

Assistive tech

Node
n/a

npm/yarn
npm
v8.19.2

Operating System
Ubuntu WLS
22.04

You use the defaultValue which is for uncontrolled mode.
 Thanks for pointing that out 🙇🏼 It works 🙏🏼


Update hybrid plugin scaffolding

This commit:

Adds additional customizations to be done in config/ as
post-scaffolding step in init command.
Modify Dockerfile template.
Modify main.go to accept additional flags for watches.yaml and
leader-election-id.

Signed-off-by: varshaprasad96<EMAIL_ADDRESS>TODO:

[ ] Update the commit hash of helm-operator-plugin in main.go after the pr is merged.
[ ] Update testdata/

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 89.734%

Totals

Change from base Build 999232256:
0.0%

Covered Lines:
1381

Relevant Lines:
1539

💛  - Coveralls


Was working great yesterday... now error during start up.

Describe the bug
when I run "src/bin/server.ts --config ./config.json"
get the following error:
(node:2519) Warning: To load an ES module, set "type": "module" in the package.json or use the .mjs extension.
/opt/eufy-security-ws/src/bin/server.ts:3
import { resolve } from "path";
^^^^^^
SyntaxError: Cannot use import statement outside a module
at wrapSafe (internal/modules/cjs/loader.js:915:16)
at Module._compile (internal/modules/cjs/loader.js:963:27)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)
at Module.load (internal/modules/cjs/loader.js:863:32)
at Function.Module._load (internal/modules/cjs/loader.js:708:14)
at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:60:12)
at internal/main/run_main_module.js:17:47
To Reproduce
Steps to reproduce the behavior:

src/bin/server.ts --config ./config.json

Expected behavior
Program should start
Screenshots & Logfiles
see errors above
Versions:

Client version: latest
Node version: v12.22.7
Operating system: raspian

Additional context
Worked great yesterday.  Awesome work.  This morning... it won't startup.

Have a look at the readme example. It must be started with ts-node. You started it with node...


[red-knot] Handle invalid assignment targets

Summary
This fixes several panics related to invalid assignment targets. All of these led to some a crash, previously:
(x.y := 1)  # only name-expressions are valid targets of named expressions
([x, y] := [1, 2])  # same
(x, y): tuple[int, int] = (2, 3)  # tuples are not valid targets for annotated assignments
(x, y) += 2  # tuples are not valid targets for augmented assignments

closes #14321
closes #14322
Test Plan
I symlinked four files from crates/ruff_python_parser/resources/{invalid,inline/err} into the red knot corpus, as they seemed like ideal test files for this exact scenario. I can also copy them, if we prefer that (these relative symlinks are probably annoying if we move folders around). I think eventually, it might be a good idea to simply include all invalid-syntax examples from the parser tests into red knots corpus (I believe we're actually not too far from that goal). Or expand the scope of the corpus test to this directory. Then we can get rid of these symlinks again.

I would support just modifying the corpus test to run on all parser examples, whenever we are at a point where that doesn't panic. If we aren't at that point yet, symlinking certain files seems like a fine temporary solution (if it works cross-platform)


Too many authentication failures when defining lots of servers

[x] I have the latest version of sake
[x] I have searched through the existing issues

Info

OS

[x] Linux
[ ] Mac OS X
[x] Windows
[ ] other

Shell

[x] Bash
[ ] Zsh
[ ] Fish
[ ] Powershell
[ ] other

Version: 0.10.1

Problem / Steps to reproduce

Define lots of servers with different pubkey
Attempt any task
Some servers will fail as ALL pubkeys are tried one by one, which may exceed MaxAuthTries on sshd

ssh: handshake failed: ssh: disconnect, reason 2: Too many authentication failures

Observation
From the code here, all provided identities are collected and tried for each server with pubkey support.
https://github.com/alajmo/sake/blob/47cdac64f3ae4329a18293828a464204bf9a7fbd/core/run/exec.go#L163-L166
This should not be the case, each server should only try with (own) provided and global identity.
Also this approach is not very efficient as N servers with same pubkey will generate N identical identities. If a server using pubkey2 is declared below 100 servers using pubkey1, it will need to try 100 wrong identities before it can make connection, if even can pass the MaxAuthTries.

Thanks for the bug report and finding the inefficiencies, will check it out after vacation.

Should be fixed now with v0.10.3, please re-open if the issue persists.


Trained model as a zip-file

Is there any reason for not sharing the trained model as a zip file exported with the command nnUNet_export_model_to_zip?
It would make it very simple to import and use as there is a corresponding command to install a trained model from a zip-file.

Dear nis,
We now share the model exported using the nnUnet_export_model_to_zip command. Please kindly see the release section and let us know, if you are able to run the model.
Kind regards
Florian Putz

Thank you.
I've run the model on one of our images and it seems to work. I have little anatomical knowledge, so I don't know how accurate it is. But it looks good to me.
Are the values for the different levels sequential? Does level 1a have the value 1, 1b left the value 2, 1b right the value 3?