[BUG] How to turn off loading if the video load fails

Describe the bug
I want to show a network error when there is a network problem, It's not always loading ，How should I judge the status
To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Technical Details:

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

How fix?


[$250] All report submissions display "submitted $0.00," regardless of the actual report amount.

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number:
Reproducible in staging?: Needs Reproduction
Reproducible in production?: Needs Reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @puneetlath
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1724678229911119
Action Performed:
Prerequisite: Control workspace with scheduled submit enabled and set to anything but Instant/Manual

Create expense as a submitter, ensure it doesn't have any violations
Either wait for scheduled submit to run or trigger with this SO

Expected Result:
The comment for the submitted amount should match the actual report amount.
Actual Result:
All report submissions display "submitted $0.00," regardless of the actual report amount.
Workaround:
None
Platforms:
All
Screenshots/Videos

Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01b5aba96981a28449
Upwork Job ID:<PHONE_NUMBER>097820564
Last Price Increase: 2024-08-30

Issue OwnerCurrent Issue Owner: @suneox

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

FYI @garrettmknight this seems to only happen when the report is auto-submitted by harvesting. More context in the slack thread.

Testing with https://staging.expensify.com/report?param={"pageReportID":"1365542954725485","keepCollection":"true"} on my<EMAIL_ADDRESS>account. Will be able to confirm tomo.

Confirmed

The reportAction from auto-submission flow is missing amount field in originalMessage response, so I think we need BE check the response for reportAction.actionName = SUBMITTED & automatic: true

cc @marcochavezf - i wonder if this came from https://github.com/Expensify/App/pull/45020/files#r1750911757?

Because it looks like BEFORE that PR, we didn't look for a submitted report action to have originalAmount.amount, instead i believe we went off of report.total

So i think we CAN fix this externally, OR if we WANT there to be a amount in the report action, then ya this should probably fixed internally

I think there isn't much we can do for old submitted actions where we never stored the amount. We can display the report's total instead, but what if the user added expenses to the report after it was submitted? another option is to just display "submitted this report" if we don't have the amount stored in the action

Very fair - I like the another option is to just display "submitted this report" if we don't have the amount stored in the action option best probably

I kind of question the value of putting the amount in this message. The report total is already at the top of the report, why duplicate the info?

My understanding of why "using the report total" isn't perfect is this situation, for example:

User A creates a report in OldDot & adds an expense for $1.00 & submits
User A clicks "Undo Submit"
User A adds another expense for $2.00, then submits again

In NewDot, would we prefer:

BOTH report actions showing submitted $3.00? Or
The first report action showing submitted $1.00, and the second report action showing submitted $3.00?

I think I'd prefer seeing 2⃣

I don't think the report total is that useful in either scenario. I feel like it just makes things more complicated. I think in both cases just seeing Submitted the report would be plenty.

aah i think that's 1000% fair 👍

Doesn't that break our pattern though? e.g.

Submitted [total]
Approved [total]
Paid [total]

It could work still in order to remove this issue, as I agree that we keep getting hit by $0.00 system messages. Maybe though it we went this direction we don't used report and simply say Submitted if we want to simplify the logic as a starting point.

Doesn't that break our pattern though? e.g.
Submitted [total]
Approved [total]
Paid [total]

Yeah, I'm questioning whether the [total] pattern has any value, given that the report total is already its own field on the report. Just thinking that if we remove this [total] pattern then that's one less feature to maintain.

Expense amounts can be changed though after submission prior to approval, can't they? So the total acts as a snapshot at submission for the history.
Equally though, I think in cases like paid or approved, a notification that reads "paid $X with Expensify" is much better to include the amount you were paid back as validation, over a generic "paid this report" notification.
Then I guess the other consideration is that one-expense reports aren't "reports" in the UI since we did away with them, so using "this report" terminology doesn't quite fit when it looks and feels like just an expense for them.

Ok, fair enough. It feels to me like a feature we built over and above what we had in OldDot, even though I don't recall customers bringing this up as a problem with OldDot. But if people feel strongly that this is a good and important feature, then I'm fine fixing it and maintaining it instead of removing it.

Yeah, OldDot had a totally independent notification system though with standalone copy and what not to notify for these actions. In NewDot, we don't have that, so the system message is how you get told you were paid X.

With the Auth PR on prod, are we good to close this out @youssef-lr?

@youssef-lr bump on the question above!

Ah yes, we can close this.


Secure Boot: Run signtool with keepcache=true

I have changed sb-signing-utls to remove the cache by default but it's
important to preserve the current behavior under op-build, as features
that rely on cache re-use depend on it.
Signed-off-by: Dave Heller<EMAIL_ADDRESS>

It requires the environment variable to be set.  If there's a better way to set it, go ahead and do so please.


Tooltip is displaced

First of all, thanks for this great directive! I really like it.
Now here's my problem: I have a couple of sliders that should show a different tooltip for the min and max values. I created a function that returns an initialization object for the sliders. I have two issues:
1.) The tooltip on the max value is displaced to the left. It will show when the cursor is a little but under the value and when I move the tooltip sometimes is displayed correctly on top of it but most of the time it is left to it. Here's a gif to illustrate the problem:

2.) Every slider should get a different tooltip text. I create the slider options with this function:
createSliderObject(sliderName) {
    return {
      options: {
        floor: 1,
        ceil: 5,
        stepsArray: '1   5'.split(''),
        showTicksValues: true,
        ticksValuesTooltip: (val) => {
          if (val === 0) return this.sliderLabels[sliderName];
          else if (val === 4) return this.sliderLabels[sliderName];
        }
      },
      value: 2
    }
  }

sliderLabels is an array that contains different tooltips for every slider.
...and create the sliders here:
this.sliderNames = [
  'slider1',
  'slider2'
  ...
]
for (let f of this.sliderNames) {
      this.sliders[f] = this.createSliderObject(f);
}

But every slider somehow has the same tooltip text for the max value. Any hint in the right direction greatly appreciated!

First of all, you are not using the stepsArray option correctly, in your example you don't really seem to need it, just set floor=1, ceil=5 and step=1. Then for the wrong position of the tooltip, this is weird, I would need a demo to see what's going on.

The thing is, I just want to display the first and the last value. I don't want 1,2,3,4,5 to be displayed, just 1 and 5 and have a different tooltip for both. I couldn't figure out a way to do it with the given options.

OK so put stepsArray: ['1', '','','','5']

Could you please take a look at this? http://jsfiddle.net/ko9vk9jL/6/

I am sorry but I am on holiday for 10 days and I'm only on mobile phone so I cant debug anything.

You forgot to load angular-bootstrap... Here's a working version: http://jsfiddle.net/3ts4w58y/


KWinFT: a kwin fork focused on performance, maintance, wayland/wlroots

Project description

KWinFT (KWin Fast Track) is a robust, fast and versatile yet
easy to use composited window manager for the
Wayland and
X11
windowing systems on Linux.

KWinFT is intended to be used as part of a
KDE Plasma Desktop.
The KWinFT project is a reboot of KDE's
KWin.
KWinFT differentates itself from KWin in some important aspects:

KWinFT values stability and robustness.
This is achieved through upholding strict development standards
and deploying modern development methods to prevent regressions and code smell.
KWinFT values collaboration with competitors and and upstream partners.
We want to overcome antiquated notions on community divisions
and work together on the best possible Linux graphics platform.
KWinFT values the knowledge of experts but also the curiosity of beginners.
Well defined and transparent decision processes enable expert knowledge to proliferate
and new contributors to easily find help on their first steps.

In the last year our development focus for KWinFT was on the integration with the wlroots library. In a single sentence our goal with this is to share underlying architecture, patterns and ideas across the Wayland ecosystem, instead of reimplementing things again and again for each and every big project while leaving smaller ones behind.

Metadata
recent release blog post with lots of info: https://subdiff.org/blog/2022/more-wlroots-with-kwinft-524/

homepage URL: https://gitlab.com/kwinft/kwinft
source URL: https://gitlab.com/kwinft/kwinft
license: gpl2
platforms: linux

@cidkidnix was working on an overlay for this and might have some input.

I made some progress packaging it in https://github.com/InternetUnexplorer/kwinft-overlay, but it's not working yet unfortunately. At a minimum, I still need to copy this patch and figure out why this is needed (maybe a bug in upstream?). The overlay itself might also be done incorrectly. Anyway, while I don't have the time/energy to get it working right now, feel free to use any of the stuff there if you want to try!


Rotation PID Loop values seem to not be respected

We are having an issue as we now have the offsets to be correct, but it seems like the rotation of the robot isnt working. We have tried all sorts of different values and it still seems to not work properly. It will keep overshooting its setpoint regardless of what PID values are entered. If we set a really high D value, it will overshoot worse, but I assume that is normal for a PD loop that is destructively resonating (I cant think of the right term).
I also tried turning the invert IMU both ways and no change seems to happen.

It does overshoot consistently for the PID values given. The gyro is as centered as basically possible. I'll have to record it on Wednesday.

Sorry for the delay. We did some work on it and honestly I dont know what changed, but it seems to be working with the config. But it acts weird: https://youtube.com/shorts/5C02hNgNkg
I hope the video does a good job of showing what is happening, but it only turns 180. But seems to handle the turning well. It does overshoot due to a lot of slip of no carpet and PID isnt tuned. But its just weird

https://youtube.com/shorts/5C02hNgNkgs?feature=share Not sure what happened but try this one

We are just using the example YAGSL so I assume that is not what the intention is?

This is fixed at least for the most part. The default control closedFieldAbsoluteDrive Where side to side of the joystick does not turn the robot it sets the angle of the robot based on the x and y components, which does work when we tried it, but it doesnt really explain the randomness of the original experience, but we haven't been able to recreate it.


Automatic PR from RestAPI-PR640

Generated from RestAPI PR: https://github.com/Azure/azure-rest-api-specs/pull/640

Coverage decreased (-0.005%) to 56.473% when pulling 372a9828f0b2faba106b7e0fafcd4c0913609e7e on AutorestCI:RestAPI-PR640 into c640bb1b9faae1a73ec49058f2ad5eb8ac912208 on Azure:dev.

Coverage decreased (-0.005%) to 56.473% when pulling 372a9828f0b2faba106b7e0fafcd4c0913609e7e on AutorestCI:RestAPI-PR640 into c640bb1b9faae1a73ec49058f2ad5eb8ac912208 on Azure:dev.


Arbitrary call return sizes

In EVM the caller must define the available space for return values.
In the current EVM2 design this is carried over, however it could be improved:

call doesn't defines the return value space
return doesn't writes the value to the caller's memory space (I understand this today can depend on the VM anyway)
return places the values into an intermediate, in-memory storage (callstore) and the contract is charged according to the size of this
a new opcode, callResultCopy is introduced for copying between callstore and the caller's memory space (and callResultSize to retrieve the total size)
the callstore is erased when a call is executed

It is a rough design, but it could be ironed out.

I think this is related.


Add percentile metrics to WHN metrics page

🚀 Deployed on https://66048d371c4a375d79e3ac94--cozy-fox-0defec.netlify.app


TypeError: virtual is not a function when importing package using ESM

Hi! 👋
Firstly, thanks for your work on this project! 🙂
Today I used patch-package to patch<EMAIL_ADDRESS>for the project I'm working on.
I was seeing the following type error when running vite build:  TypeError: virtual is not a function
The vite.config.ts file was interpreting the imported file as a CommonJS module. I checked the package.json and looks like the exports are not properly configured to be compatible with ES Modules.
I patched the package.json on my end to return ESM when importing and CommonJS when requiring as a fix. Can we get this change shipped into main as well? Thanks!
Here is the diff that solved my problem:
diff --git a/node_modules/vite-plugin-virtual/package.json b/node_modules/vite-plugin-virtual/package.json
index 487ec58..2d0a40b 100644
--- a/node_modules/vite-plugin-virtual/package.json
+++ b/node_modules/vite-plugin-virtual/package.json
@@ -2,9 +2,16 @@
   "name": "vite-plugin-virtual",
   "description": "Virtual modules with HMR invalidation plugin for Vite",
   "version": "0.2.0",
-  "main": "dist/index.js",
-  "module": "dist/index.mjs",
-  "types": "index.d.ts",
+  "type": "module",
+  "main": "./dist/index.js",
+  "types": "./dist/index.d.ts",
+  "exports": {
+    ".": {
+      "types": "./dist/index.d.ts",
+      "import": "./dist/index.mjs",
+      "require": "./dist/index.js"
+    }
+  },
   "license": "MIT",
   "author": "patak-js<EMAIL_ADDRESS>   "repository": {

This issue body was partially generated by patch-package.

Thanks! This should be fixed in 0.3.0


Support for MRT

It would be nice to do deffered rendering with oryol.
MRT is only supported via extension on WebGL 1.0 and OpenGL ES 2.0 but I think it would be still worth to add support for it.

Agreed, that's an important feature to have. I need to figure out a few details about how to best fit MRT setup into the existing Gfx API (mainly TextureSetup object and ApplyOffscreenRenderTarget).

I think it should look like this:
auto rtSetup = TextureSetup::RenderTarget(640, 480);
// main surface and depth buffer
rtSetup.ColorFormat = PixelFormat::RGBA8;
rtSetup.DepthFormat = PixelFormat::D24S8;
// add additional surfaces with resource locators (for lookup later)
rtSetup.AddMRT("MRTSurface1", PixelFormat::RGBA8);
rtSetup.AddMRT("MRTSurface2", PixelFormat::RGBA8);

// create the MRT, returned handle can be used as texture handle for first surface
Id rt = Gfx::CreateResource(rtSetup);

// lookup texture handles for the 'attached' surfaces
Id mrt1 = Gfx::LookupResource("MRTSurface1");
Id mrt2 = Gfx::LookupResource("MRTSurface2");

lgtm.
Maybe also add a way to lookup how many color/depth your current hardware supports.

MRT is now supported via render pass objects.


Feature: Remembering scroll position when switching Home and Retweets.

The option to separate retweets from home posts is enabled. On the Home tab, scroll down a bit and go to Retweets tab, then go back to the Home tab. The page refreshes, causing the scroll bar to go up and making it harder to track your previous page position.
My solution is to make the page not refresh on single click on the "Retweets" button from the Home tab, and the "Home" button from the Retweets tab. Refresh only when pressing "Home" button on Home tab, and "Retweets" button on Retweets tab.

This is an artifact of the very unsmart way the Retweets tab is implemented - when you're viewing Latest Tweets, Tweak New Twitter is just hiding all retweets, and Twitter's infinite scrolling implementation keeps loadings more tweets to fill the window. When you click on the Retweets tab, Tweak New Twitter just switches to hiding everything but retweets. When you switch back and forward, whatever Twitter is doing to track your position in the timeline seems to get confused and brings you back to the top.
Your solution sounds like the right thing to do, but to make it work we'd have to figure out how Twitter keeps track of and restores your position in the timeline when you navigate back to it and trigger that functionality when switching between Latest Tweets and Retweets.
I know that built-in functionality already works with the way the Retweets tab is implemented, as I set it up to take you back to the Retweets tab if you navigate away to another page and then come back to the main timeline without clicking "Home", and Twitter still successfully restores your position in the Retweets timeline even though it doesn't really exist:

I don't think this is doable as Folllwing and Retweets are the same timeline, but with different things hidden. This means Twitter's implementation for restoring scroll position is only going to be good for whatever the last one you looked at is


Scroll performance on OSX

After installing the current version 1.4.2 on OSX 10.10.2 the scroll performance, even on small files with a few 100 lines of code, slows down significantly. Scrolling is no longer smooth, instead it skips every other line.
PHPStorm 8.0.2 on a Macbook Pro Retina Mid 2014 with GT 750M

Update: this is still present on the lastest PHPStorm 10.x version

I have a bad feeling that this is probably going to be related to natural scrolling or retina, both are really difficult to debug without running OSX.
I am going to be doing a general performance pass in the next release so it might go away, but if its every other line then maybe not :-/

might be related to https://github.com/Vektah/CodeGlance/issues/93

Huge overhaul to give pixel perfect scrolling in 1.5.0. Should be fixed


API error in Swagger Editor

Loading the API into Swagger Editor, having an error :
https://editor.swagger.io/?url=https://raw.githubusercontent.com/camaraproject/QualityOnDemand/main/code/API_definitions/qos-stable-latency.yaml

Thank you for noticing. I created a PR #10 correcting the security requirements.

No feedback for the last ten days, since this is a minor bug correction (2 lines in total), I am merging the PR.
I am keeping this issue open this for the next few days should there be any additional comments.

PR with correction has been merged, there are no further comments. I am closing the issue.


bench in api-maps

We should bench this lib in api-maps and compare the performance against our current compositing code.
cc @artemp @mapsam

:+1:
We also need a way to visually test over-zoomed tiles are properly aligned and don't have undesired artifacts.  Can we setup simple local VT server and point a native GL client or there's a better way to do this?
/cc @millzpaugh @mapsam


feat: rsyslog audit logging (WIP)

Going to take a step back on this, and see if this can be reworked simpler to no longer require rsyslog.
Adding rsyslog audit logging for EDA features.
This PR is still a work in progress.
Please feel free to review/add comments though in the meantime :)
Still need to ensure this works when containerized
Jira: 28245

Closing PR


Modify eventScraper to pull in DocumentHistoryActions instead of Hearings

Modify scrapeEvents function to pull in DocumentHistoryActions instead of Hearings.
Currently, the scrapeEvents function is making use of the listEvents function from the malegislature file.
See:
https://github.com/codeforboston/maple/blob/main/functions/src/malegislature.ts
https://github.com/codeforboston/maple/blob/main/functions/src/events/scrapeEvents.ts

The backbone for publishing notifications currently already exists in the publishNotifications cloud function. However, no notifications are published because the events it looks for are in the form bill-example-1 or org-example-1.
Right now, the events scraper (scrapeEvents) only scrapes hearings (scrapeHearings) and sessions (scrapeSessions) which are saved in the form hearing-1 and session-1-1, respectively. This needs to be expanded to scrape all DocumentHistoryActions and store them in a form that can be understood by publishNotifications.

Create a new scraper class (DocumentHistoryActionsScraper) that is a child of the EventsScraper abstract class that all of the other scrapers inherit from (see: scrapeEvents.ts). Use the HearingScraper as reference.
Add the scraper to the existing list of scrapers (see: runScrapers.ts)

the documenthistoryactions are called the "bill history" and it is scraped and saved as part of the bill info, used on the bill page, right now in the bill history modal. hopefully this is helpful--
https://github.com/codeforboston/maple/blob/main/functions/src/bills/bills.ts


Consider Decoupling EXO/Teams from Defender

💡 Summary
Currently, for EXO/Teams controls that are implemented in Defender (or optionally by third party software), ScubaGear simply informs the user to check the Defender report and doesn't perform the appropriate check in-place. In the interest of reducing the number of grayed out results, we could consider performing the appropriate checks in-place, with a config option that switches off the tests if they are using a third party.

Motivation and context
Pros of doing the checks in-place:

Less gray results
User wouldn't need to switch back and forth between the reports
Now that the Defender baseline recommends using the preset policies, there is no longer a clean 1:1 mapping between the EXO/Teams "third-party" policies.

Cons of doing the checks in-place:

Extra login needed for EXO/Teams for Connect-IPPSSession
Increased code duplication

Acceptance criteria

[ ] Reach a consensus on the path we want to take
[ ] Create an issue to refactor the code if appropriate

@adhilto Are there any updates needed for this to be workable? Things like policy omission are now configurable and may meet some of the requirements, for example.

Policy omission definitely helps with this. Now that we have that, I'd consider any further enhancements nice-to-haves, rather than essential.
For example, say I know I'm using Defender and I want to get rid of all the manual checks in EXO. I could do that by omitting all the Defender-related controls from EXO, with the rationale being something like, "We're using Defender for this, see the Defender report, no manual check needed." So at least in that use case, the users now have a path forward.
Now say I'm not using Defender and I want to get rid of both the "manual checks" from EXO and "failures" from Defender. I can do that, by omitting both the Defender-related controls from EXO and the Defender baseline checks. Again, I have a path forward.
What I don't like is that in both cases you have to omit controls in order get a clean report. In an ideal world I think ScubaGear would be optimized for one or the other.
Possible further enhancements I can think of:

Map the EXO (and Teams) Defender-related controls to the appropriate Defender controls. For example, if a user is passing MS.DEFENDER.4.1v1 and MS.DEFENDER.4.2v1, we know that they're passing MS.EXO.8.4v1. Essentially, we could modify the Defender code into shared functions available to multiple baselines to both remove the need for manual checks and prevent code duplication. Doable, but high-level of effort.
A slightly lower lift, add config options like "DefenderLeveraged" and "ThirdPartyInsteadOfDefender". If both are false, ScubaGear behaves as normal. If DefenderLeveraged, the manual checks are all automatically omitted. If ThirdPartyInsteadOfDefender, both the manual checks and Defender baseline checks are omitted.

Or we could do nothing. As I said, with the omit config option, the users at least have a path forward.


Fix Typo The => Then

@blowdart @Rick-Anderson @rachelappel

Hi @HaoK, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;


Add transaction UUID to log & debug entry object to ease debugging

In order to ease log grouping add a transaction unique identifier (GUID version 4)

@Szandor72 any chance to review this one ? :)


version 4.0.1: oracle 8 databases

Now, I can see table views in oracle 8 databases ( not in former versions ), but it is still an important problem: I can´t edit rows. Previous to that, that error is received when opening the table:
SQL Error [942] [42000]: ORA-00942: la tabla o vista no existe
java.sql.SQLException: ORA-00942: la tabla o vista no existe

dbeaver-debug.zip

Please post debug log (https://github.com/serge-rider/dbeaver/wiki/Log-files) with related errors/warnings.

Updated with debug info
Thanks

Now problem is in table columns reading.
DBeaver executes query
SELECT * FROM SYS.ALL_TAB_COLS c WHERE c.OWNER=? AND c.TABLE_NAME=?

I believe it should work on 8.x. Could you check it?

SQL Error [942] [42000]: ORA-00942: la tabla o vista no existe
java.sql.SQLException: ORA-00942: la tabla o vista no exist
It works this:
select * from all_tab_columns where owner='owner' and table_name ='table_name'

Ok, thanks for info.


IDEA-169757 : support charset for html report

As last step for IDEA-169757: Code coverage report: html file encoding  and https://github.com/JetBrains/coverage-report/pull/3,  https://github.com/JetBrains/intellij-coverage/pull/57, we need to modify interface to inject charset from configuration.
Reviews will be of great help to resolve issue i'm working on 😄 thanks, bro :)

Could you run :apiDump Gradle task to update .api file?

Could you run :apiDump Gradle task to update .api file?

thank you for reviewing 👍 i updated .api file

Thanks :) When are you planning the next release?


msbot clone: Add new qna maker kb to existing qna service

Tool
Name: Msbot
Describe the problem
I'm using msbot clone in an unconventional way to enable multiple language configurations for Dispatch, LUIS, and QnA. I run msbot clone against the same resource group for each language I want to add to my bot. Each time I run the command, it creates a new QnA Maker service with Azure Search, etc. in the resource group. This results in a lot of azure search services and quickly exhausts the quota for a single subscription.
Describe the solution you'd like
I'd like to be able to run msbot clone services against an existing resource group, and if there is an existing QnA Maker service in the resource group, add the kb to that service instead of making a new one. This might also be a good idea for the LUIS services, but less critical.
Describe alternatives you've considered
Having multiple QnA services doesn't break anything, it's just inconvenient. I'm aware there could be situations when you might want multiple QnA services in a single resource group, so maybe it is a flag that could be added or something.
[enhancement]

msbot is deprecated.  we won't add any new features and will on fix Pri0 bugs


Best way to display multiple data in tooltip

I would like to show a title and a description in each of my tooltips.
Something like below:
        {badges.map(badge => (
          <FlexCol xs={4} sm={3} className="badge" key={badge.pk}>

            <ReactTooltip
              id="badge-tooltip"
              getContent={({name, description}) => (
                <React.Fragment>

                    {name}

{description}

                </React.Fragment>
              )}
            />
          </FlexCol>
        }

The problem is, html5 data attributes only seem to handle strings, so we cannot send arrays or objects to it.
I currently cannot see any other solution than concatenating my 2 data items in data-tip, when split it within getContent, which is far from optimal.
Would it be possible to handle any data-tip-* attribute for me to set data-tip-name and data-tip-description attributes, and easily reuse them from getContent ?

I know there was a recent PR to pass the value of data-tip into getContent. If you use the most recent, then 'badge' should get passed in, and you could reference badge.name and badge.description inside getContent().
Please try that, thanks!

yes it is working ! thank you

broken with 3.8.0, worked from v3.8.1 . Thanks, closing

@aronhelser sorry, I closed without verifying due to @oladhari's comment, but this doesn't work. I had raised the issue using v3.8.0, and it's still broken with v3.8.1.
As explained in the issue description, only a string can be given to getContent. We would like arrays or objects to be supported as well, to have a smaller/cleaner code footprint using the react-tooltip plugin.

it is my mistake, I did not verify passing the object, while trying to pass an object i got this error:
Uncaught TypeError: Cannot read property 'name' of null

If I'll pass an object, it's automatically converted to string "[object Object]"...

@ketysek I experience the same thing. Did you find a solution?

@mtsalenc
here the solution that we came with
we created a function that handle the content in a customized component for the tooltip:

  handleContent = dataTip => {
    ReactTooltip.rebuild();
    if (!dataTip) {
      return "";
    }
    const [name, description] = dataTip.split("|");

    return description ? (

        {name}

        {description}

    ) : null;
  };

in the component using the tooltip you just pass the data-tip like this:
data-tip={`${name}|${description}`}

hope it is helpful

I did something very similar but using JSON.parse and JSON.stringify so I can manage the whole element and have more flexibility.
So instead of using
const [name, description] = dataTip.split("|");
I did something like
const obj =  JSON.parse(dataTip)
Here I can access obj.name and obj.description
And when passing the values, Instead of
data-tip={${name}|${description}}
I did:
data-tip={JSON.stringify(the_object)}
Where 'the_object' is the element that contains a name and description attributes

@oladhari Where did you call the customized handleContent function to pass an object to dataTip?

@capstonednc
in the react-tooltip getContent
            <ReactTooltip id="badge-tooltip" getContent={this.handleContent} />

@oladhari got it. I was still getting a [object Object] response, until I parsed the dataTip in the function and stringified it in the getContent prop. Thanks for the help!

I kept getting [object Object] and then I removed html={true} from my props and the problem went away.


Feature/871/oauthbearer

fixes https://github.com/confluentinc/confluent-kafka-dotnet/issues/871

It looks like @thtp hasn't signed our Contributor License Agreement, yet.

The purpose of a CLA is to ensure that the guardian of a project's outputs has the necessary ownership or grants of rights over all contributions to allow them to distribute under the chosen licence.
Wikipedia

You can read and sign our full Contributor License Agreement here.
Once you've signed reply with [clabot:check] to prove it.
Appreciation of efforts,
clabot

[clabot:check]

@confluentinc It looks like @thtp just signed our Contributor License Agreement. :+1:
Always at your service,
clabot

Thanks for this PR @thtp! Waiting for this feature.
@mhowlett could we include it in the next release please?

Missing tests (trivup can be used to set up an oauthbearer cluster)
@edenhill, is it correct that integration tests are not run CI builds, and supposed to be run manually on local environment? I mean, should I do something to ensure test environment is configured properly (some scripts or whatever), or I can just 'git push' test assuming there already is a configured oauthbearer cluster on localhost?

@mhowlett can you explain the integration test setup?

@mhowlett can you explain the integration test setup?

Well, I already have a cluster with configured oauth to play with. The question is, do I have to do something to set up a test cluster for CI?

Oh, found docker-compose. It must be the answer )

@edenhill Regarding integration tests, I have to ask again, because I’m not sure that I understood correctly.
Should I provide any sort of docker-compose files or scripts for trivup, or will the tests themselves be enough? If i must provide something beside tests, what exactly should it be?

@mhowlett Can you chime in on how integration test setups are done with the dotnet client?

tests: we have a docker compose file for the schema registry tests. we don't currently have an equivalent for the Confluent.Kafka integration tests (note: I just run them locally against a locally running kafka, not in CI). I think making a docker compose file modeled after the schema registry one is what we want here.

btw: thanks a lot for the PR @thtp - definitely something we've been needing to get done.

tests: we have a docker compose file for the schema registry tests. we don't currently have an equivalent for the Confluent.Kafka integration tests (note: I just run them locally against a locally running kafka, not in CI). I think making a docker compose file modeled after the schema registry one is what we want here.

My bet was on the "default, builtin AuthenticateCallbackHandler implementation" promised as default token validation mechanism for OAuthBearerLoginModule. I beleave, unsecured JWT is exactly what we need here. But I have no success in getting it work. I mean, all my attempts to start a broker with unsecured JWT are failng. Maybe you know someone who can help with creating a docker-compose for a Kafka broker that will use unsecured JWT? @mhowlett ?
P.S.: I Can provide a docker-compose with keycloak as an identity server ) That will work, but I'm not sure that it is a good solution. I mean, we well have to have a code to get an actual token just for couple of tests. And those tests will also test a keycloak which is completely out of a scope.

Highly recommend using trivup to create test clusters, for oauthbearer it should be as simple as:
$ pip install trivup
..

$ python -m trivup.clusters.KafkaCluster --sasl OAUTHBEARER --version 2.5.0
# Interactive mode
# - Waiting for cluster to go operational in /home/maglun/old/kafka/KafkaCluster/90659419
# - Client configuration in /home/maglun/old/kafka/KafkaCluster/90659419/rdkafka.conf
# - Connect to cluster with bootstrap.servers SASL_PLAINTEXT://localhost:39809,SASL_PLAINTEXT://localhost:38125,SASL_PLAINTEXT://localhost:35643
# - You're now in an interactive sub-shell, type 'exit' to stop the cluster and exit back to your shell.

To run a command as administrator (user "root"), use "sudo <command>".
See "man sudo_root" for details.

$ cat $RDKAFKA_TEST_CONF
bootstrap.servers=SASL_PLAINTEXT://localhost:39809,SASL_PLAINTEXT://localhost:38125,SASL_PLAINTEXT://localhost:35643
broker.address.family=v4
security.protocol=SASL_PLAINTEXT
sasl.mechanism=OAUTHBEARER
enable.sasl.oauthbearer.unsecure.jwt=True
sasl.oauthbearer.config=scope=requiredScope principal=admin

$ run-your-tests
...

# terminate cluster
$ exit

There should not be a default OAUTHBEARER handler in any of the clients, it does not make sense since all oauthbearer deployments will be different, and we don't want to use the unsecure default of JWT.

Default handlers are documented here. Since it is an official feature designed for testing and development, I don't see any contraindications in using it for testing and development ) Furthermore, using that feature will keep us from testing anything but what should be tested. The problem is - i haven't yet figured out how to set up broker to use it properly. Or, may be, I just misunderstood what are all those default handlers about?

I misremembered the default stance, you're right.

we only need basic testing of this stuff i think in the .net client. trivup may indeed be what we ultimately want to use for integration tests, but we don't currently and if we can avoid locking that in now that would be good imo.

@thtp - unfortunately i've never set this up before, so can't give any wisdom ...

Okay, now I can confirm that my simple produce-consume test is working with a trivup setup provided above. So, if docker-compose is not mandatory, I may say that tests status might be considered as 'ready for review' )
Really, how badly do you want me to prepare docker-compose? I've already spent a plenty of time trying, and almost lost my willing to live ) I afraid, one more evening is all I can stand )

@thtp - thanks very much for trying :-). what you've done is fine by me, but leave the oauthbearer bootstrap servers config blank (it changes every time).
i actually feel that regression testing of this feature is of fairly low value (compared to other tests that don't exist, and would be easier to create so have roi, so we should be doing those instead) because regressions are unlikely to occur in the C# layer. but it's good to have the test committed, so we can enable it when the testing framework gets better.

Is there anything else I should change? Or can I expect  these changes in some of the nearest releases? )

i would like to get this in v1.5, which is coming pretty soon. the API is still causing me to pause though. First, I feel sure we shouldn't have the extension method on the client classes and not on the builder classes (the current state). Using extension methods on internal properties/methods actually feels like a bit of a mis-use of them to me in general, but i may be wrong, and it is quite good UX if we can make this work on the builder classes as well. Are you sure it's not possible (am i wrong below?):

the builders could have an internal OAuthBearerTokenRefreshHandler property, but there's no way to set it without the extension method is what i was thinking

So that's the first option, if possible. The second option is how you originally had it (which may be the most pragmatic, but i'm still not very fond of it). And the third option is using return values from the refresh method.
Something I feel i want to know is how java does it, and if anyone finds this deficient.
Finally, we could always mark the API as experimental (giving us the option to change our minds later, but get the feature out now). Good to avoid it, but we can do it if we must.

the builders could have an internal OAuthBearerTokenRefreshHandler property, but there's no way to set it without the extension method is what i was thinking

I don't think it's actually a good idea. It looks like such an implementation violates encapsulation and does not add any value. There might be some reason if we will be able to remove handler from the builder completely. But without complete rewriting, it will only add unnecessary complexity (imo).

Something I feel i want to know is how java does it, and if anyone finds this deficient.

As far as I understand, java client uses the same approach, as a server configuration. Setting a handler implementation class name to the corresponding client configuration property. Like sasl.whatever.handler=some.java.namespace.WhatevrHandlerImplemetation in the client config file. But that's not for sure.

Any news? Something? Anything?

the use of extension methods on the client classes violates encapsulation in the same way as the builder classes (which is why i say using extension methods seems like misusing them) and doing the extension method trick on the client and not the builder classes seems inconsistent to me.
the java client doesn't have any additional methods on the client classes. my current status on this pr is i need to find time to get to the bottom of that.

believe it or not (!), LGTM, rebasing to 1.5.x and merging.
A few minor formatting things I'll change in a follow up PR. Release forthcoming.


init scripts should have status output

Author Name: Adam Heath (Adam Heath)
Original Redmine Issue: 2234, https://dev.opennebula.org/issues/2234
Original Date: 2013-07-28

In debian, the opennebula and opennebula-sunstone init scripts should display messages saying what they are doing, similar to other init scripts.  Messages like "Starting Opennebula: oned schedule."
Actually, upon reading that script, they are not following debian policy.  While reading /lib/init/vars.sh is good, $VERBOSE is not meant to be used in that fashion.  Just call log_daemon_msg directly, and all the rest of the log_foo functions.

Original Redmine Comment
Author Name: Ruben S. Montero (@rsmontero)
Original Date: 2013-07-29T22:23:41Z

Agreed, moving to backlog
THANKS for your feedback :)


Fix UI Issues on Mobile and Desktop

In the current user interface, there's no padding on the control row, keys row, and header. Moreover, the page contains a horizontal scrollbar when it shouldn't. Here's how it currently looks on Desktop:

and mobile:

It would be good to add padding and fix this horizontal scrolling issue.

This has been resolved in the switch to the reactronica library.


Issue #278: YCSB does not throw error if there is no table created

Check that the table exists during DB binding initialization.  If this is tested for here, the workload will continue running until it finishes.

nit: please start the commit message with "[hbase]" to indicate what's impacted.

Having an issue amending the comment.  Closing this pull request and creating a new one.


Update golang version and build parameters

This PR fixes an issue with running the Docker image build with an older version of the go builder, adding go.mod and go.sum to ensure correct module versions.

LGTM


CitationID

Rather a mess in DB as this column can theoretically be required for any table. Currently there is inconsistency, some tables have it, others dont. Worse still, a single row (of any table) can require multiple citationIDs, such as the PhaseCitation table.
I suggest we instead we revise the phaseCitation table to have an additional column, an enumerated list of all tables. I.e, a phase can have multiple citationIDs, each of which can be relevant for multiple tables.
Can be populated automatically from existing PhaseID locations, but users have requested a trial period before CitationIDs are removed from other tables.
The only exception to this is the C14 table, since many C14 samples are not Phased, therefore still require a citationID

I now think a hyrid system might be best...
We maintain citationIDs in each row (potentially for any table), PLUS we extend the PhaseCitation table as I have proposed, such that we have the PhaseID, the CitationID, and the source table (data type) that it is for.
Therefore where there is a 1-to-1 mapping for a data row and a phase (such as a monument), multiple citationIDs can be stored in the phaseCitation table, specifying they refer to monuments.
Could probably build a trigger ensure the citationIDs in lower tables are also instantly stored in the phaseCitation table, and daily checks to ensure no duplicates.


docs: fix small typo in README

Contributing
If you want to add a project to this list, please make sure that:

[x] You read the contributing guide
[x] The repository has hacktoberfest as topic and counts for hacktoberfest event
[x] The project can't be just an "add your name" or simply + 1 contribution to hacktoberfest
[x] Place them in alphabetical order
[x] One repository per PR/contribution - avoid conflicts and simplify review
[x] Your pull request is not spammy just adding an empty line, otherwise it will be marked as invalid and not count to hacktoberfest.

If they meet the requirements above, feel free to create a PR. Happy Hacking! :sparkles:

Noticed a small typo while going through this incredibly helpful repo, fixed it


Continued fix of SLM training (see https://github.com/yl4579/StyleTTS2/pull/74)

Move generator related optimizer.steps up, so they won't get zeroed out when 'd_loss_slm != 0'. See https://github.com/yl4579/StyleTTS2/pull/74#issuecomment-1825174074.
Tuck all logic related to SLM under 'if slm_out is not None'. @yl4579 Is it normal for slm_out to be occasionally None? Is this because my OOD strings are too short? Without this fix, the logging logic after SLM training will occasionally be skipped.

 Out of 15 lines you printed it seems only two are not None. But as long as it runs I think it should be fine.

I have fixed some other issues. I think I have to test this code now on my end to make sure it produces the correct output.

I guess it's skipped too often. Out of 15 lines you printed it seems only two are not None. But as long as it runs I think it should be fine.

What's the likely cause of this? I can't figure out the meaning of https://github.com/yl4579/StyleTTS2/blob/main/Modules/slmadv.py#L129...

The run cannot be completed if the batch size is less than 1 because of some squeeze operation.

The slmadv_params.min_len(=400) slmadv_params.max_len(=500) range is too tight. Decreasing slmadv_params.min_len to 200 reduces the frequency of slm_out is None to almost 0 (I'm just using the standard Data/OOD_texts.txt).

These were default on 4 A100 GPUs.


put cpu in user mode

Hi, recently I'm working on a project that also need to do the same thing.
Currently, in x86, I have to setup the GDT, gdtr and MSR properly, and use sysexit to force the cpu goto ring3.

using ljmp can only jump to the same ring level or a General Protection Fault will be thrown.

Not quite sure whether there is a better way to do this.

Hey, thanks for commenting. This is how I'm doing it for ARM right now: https://github.com/lunixbochs/usercorn/blob/e88f46c2d7c61b9620d81a7d210d4186cd21619f/go/arch/arm/arm.go#L9
I think this is basically what I need to do for each arch until Unicorn allows directly setting the CPU mode.

By the way, the following is the code that I use in x86_32 to jump from ring 0 to ring 3.
https://gist.github.com/mike820324/2b7ea9c282de05e39d2c
As for x86_64, I may need to read more intel manual about the 64 bits architecture.


Differentiated control scheme: [ Keyboard | Mouse] controls [ individual | int | ext | all ] displays) + Differentiated menu item visibility

Before opening the issue, have you...?

[X] Searched for existing issues

Is your feature request related to a problem? Please describe
I'd like to achieve this:
1) Controlling displays individually by…
a) Keyboard: ✅ Works 100% satisfying by combining:
Brightness and contrast:  [ Standard keyboard brightness keys                    ▾ ]

      Screen to control:  [ Depends on mouse pointer position                    ▾ ]

b) Mouse: Via menu item: 💡 Okay but could be more differentiated
in https://github.com/MonitorControl/MonitorControl/pull/957#issuecomment-1047854270 @waydabber describes this possibility

Settings  →  App menu

              Menu Icon:  [ Only if at least one slider is present               ▾ ]

     Brightness control:   ☐ Enable for Apple branded and built-in displays as well

To achieve this:

Situation
Outcome

Internal display only
Menu Icon hidden

Internal display + external display
Menu Icon shown

c) Menu item visibility: Current semantically indirect/unclean option "Only if at least one slider is present " does not allow a differentiated control scheme

If Menu Icon gets 1-2 extra options:

Only if at least two sliders are present regardless of how many of those are internal or external
Only if at least 1 external display is connected as an even more concise option

Then one could show the Menu Icon also with "1 external display only" when the MacBook is:

in clamshell mode
or has its internal display off so that the GPU can drive one more external display (possible e.g. with BetterDisplay according to this testimony) )

The settings would then look like this:

              Menu Icon:  [ Only if at least 1 external display is connected     ▾ ]

     Brightness control:   ☐ Enable for Apple branded and built-in displays as well

2) Controlling all displays at once by…
a) … keyboard
✅ Works very well already with:
Brightness and contrast:  [ Standard keyboard brightness keys                    ▾ ]

… many modifiers options exists:

Use the brightness keys of your Apple keyboard to control brightness. Additionally you can hold:

  ⌃      to adjust the built-in display.
  ⌃   ⌘  to adjust external displays.
  ⇧ ⌥   for fine control.
  ⌃ ⌥ ⌘ adjusts contrast on DDC compatible displays.

❗  My particular desired modifier is missing, I'd love the following to be added:

⇧ ⌃ ⌥ ⌘ to adjust all displays.

… b) mouse: Via menu item
✅ Works perfect with

Settings  →  App menu

      Multiple displays:  [ Use combined slider for all displays                 ▾ ]

❌ Except that this option is effectively unavailable when Enable for Apple branded and built-in displays as well is OFF which is needed for 1c. If 1bc do not have that quirky unclean/indirect option Only if at least one slider is present but instead the semantically cleaner options Only if at least 1 external display is connected  and Only if at least two sliders are present then you can have Enable for Apple branded and built-in displays as well is ON independently.
Describe the solution you'd like

2a) ⇧ ⌃ ⌥ ⌘ to adjust all displays would be the minimal fix.

Describe alternatives you've considered

Fixing 1c) and subsequently thus decoupling 2b) would make the full 1abc 2ab suite possible! And would be the perfection.

Anything else?
No response

Maybe you can try switching to BetterDisplay as it has keyboard shortcut to control the built-in display only (Shift+Command+brightness keys) and all external displays only (Control+Command+brightness keys). The latter combo should be working with MonitorControl as well.


Restore onEnd event

Addresses https://github.com/chatterbugapp/chatterstreams/issues/796

:shipit:


⚠ Bofrost.at has degraded performance

In cc1ac68, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 902 ms

Resolved: Bofrost.at performance has improved in ec15e4b.


Add new package - opensc

It brings in following dependent packages as well

gd
graphviz
pcsc-lite
pcsc-lite-ccid
perl-Fedora-VSP
perl-generators
softhsm

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/tools/cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

What does the PR accomplish, why was it needed?
Adding new package opensc for security hardening of Mariner. It enables use of 2FA hardware devices.
Change Log

New packages - opensc, gd, graphviz, pcsc-lite, pcsc-lite-ccid, perl-Fedora-VSP, perl-generators, softhsm
Updated python3 package to provide pathfix.py script

Does this affect the toolchain?

NO
Associated issues

Links to CVEs
Test Methodology

Built and tested manually

Linter errors in gd.spec is not fixable. And openjdk8.spec is because of bug in linter script.

I'm not agree , & i'm subcribed oky


Update README.md

Fix Plugin Documentation link

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


Remove unavailable resources from more section

What does this PR do?
Remove resource(s)
For resources
Description
Removed unavailable links (resources) from the more section of the repo. Status: 404 Not Found
Why is this valuable (or not)?
This will display only available resources and improve the quality of repo.
How do we know it's really free?
NA
For book lists, is it a book? For course lists, is it a course? etc.
NA
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

removing the pt_BR file requires that the link be removed from README; that's in another PR, though. I think closing and opening will make the diff go away

ok that didn't work maybe update from main?

I'll remove it from main then, and will report back.

@eshellman I've made the changes and removed it from main index, Can you please review it?

more/free-programming-interactive-tutorials-pt_BR.md is still being deleted even though it already has been. will look more closely in the morning


Adding a node to a kubernetes cluster gives failed to load Kubelet config file /var/lib/kubelet/config.yaml and no networks found in /etc/cni/net.d

I have a two node k8s cluster working. I added another node to the cluster and the sudo kubeadm join ... command reported that the node had joined the cluster. The new node is stuck in the NotReady state:
kubectl get nodes
NAME               STATUS     ROLES    AGE    VERSION
msi-ubuntu18       NotReady   <none>   29m    v1.19.0
tv                 Ready      master   131d   v1.18.6
ubuntu-18-extssd   Ready      <none>   131d   v1.17.4

The journalctl -u kubelet shows this error:
Started kubelet: The Kubernetes Node Agent.
  22039 server.go:198] failed to load Kubelet config file /var/lib/kubelet/config.yaml, error failed to read kubelet config file "/var/l...

But the file /var/lib/kubelet/config.yaml exists and looks OK.
The sudo systemctl status kubelet shows a different error on the new node:
kubelet.go:2103] Container runtime network not ready: NetworkReady=false reason:NetworkPluginNotReady message:docker: network plu
cni.go:239] Unable to update cni config: no networks found in /etc/cni/net.d

And there is no /etc/cni/ directory on the new node. (The existing node has /etc/cni/net.d/ with calico files in it.) If I run
kubectl apply -f https://docs.projectcalico.org/v3.11/manifests/calico.yaml
on the master again it doesn't solve the problem. There is still no /etc/cni/ dir on the new node.
At what point does the /etc/cni/ directory get created on the new node? It's also puzzling that the kubeadm join ... command indicates success when the new node is stuck in NotReady.

What does kubectl get pods -n kube-system -o wide show, is there a calico-node pod starting on the new node? What is it's status? The daemonset of calico-node should cause a calico-node pod on the new node and an init container should install a file in /etc/cni/net.d that would cause the node to go Ready.

NAME                                       READY   STATUS    RESTARTS   AGE   IP               NODE               NOMINATED NODE   READINESS GATES
calico-kube-controllers-59877c7fb4-hbkhs   1/1     Running   0          14h   <IP_ADDRESS>   tv                 <none>           <none>
calico-node-27k9m                          1/1     Running   0          14h   <IP_ADDRESS>      tv                 <none>           <none>
calico-node-5vj8r                          1/1     Running   0          14h   <IP_ADDRESS>     ubuntu-18-extssd   <none>           <none>
coredns-66bff467f8-bfmgm                   1/1     Running   0          14h   <IP_ADDRESS>   tv                 <none>           <none>
coredns-66bff467f8-td29p                   1/1     Running   0          14h   <IP_ADDRESS>   tv                 <none>           <none>
etcd-tv                                    1/1     Running   0          14h   <IP_ADDRESS>      tv                 <none>           <none>
kube-apiserver-tv                          1/1     Running   0          14h   <IP_ADDRESS>      tv                 <none>           <none>
kube-controller-manager-tv                 1/1     Running   0          14h   <IP_ADDRESS>      tv                 <none>           <none>
kube-proxy-4rs5x                           1/1     Running   0          14h   <IP_ADDRESS>     msi-ubuntu18       <none>           <none>
kube-proxy-8q98b                           1/1     Running   0          14h   <IP_ADDRESS>     ubuntu-18-extssd   <none>           <none>
kube-proxy-cbkgz                           1/1     Running   0          14h   <IP_ADDRESS>      tv                 <none>           <none>
kube-scheduler-tv                          1/1     Running   0          14h   <IP_ADDRESS>      tv                 <none>           <none>

There is no calico node running on the failed node msi-ubuntu18.  My master and the nodes are all different versions of kubernetes.  Could that be the problem?

The issue might be that the os label has changed (or they dropped support for the beta label) in K8s v1.19. The manifest version you're using has beta.kubernetes.io/os: linux but I believe that has been deprecated in favor of kubernetes.io/os=linux.
You should do a kubectl describe node msi-ubuntu18 and see what labels it has. If it doesn't have that beta label, then that is the reason it isn't getting calico-node deployed. Assuming you'll only have linux nodes you can do a kubectl edit daemonset -n kube-system calico-node and remove the nodeSelector. Or you could also add the label to your new node by doing a kubectl edit node msi-ubuntu18.
If the above isn't the issue then look again at the describe node and see if there is anything there suggesting why calico-node isn't be deployed. Maybe check events if there are any. Check the taints, though I think the tolerations on calico-node should allow it to run anywhere.

I was finally able to solve this by doing
kubectl delete -f https://docs.projectcalico.org/v3.11/manifests/calico.yaml
followed by
kubectl apply -f https://docs.projectcalico.org/manifests/calico.yaml
There must have been some version incompatibility between version 3.11, which I had installed a few months ago and the new node.

I'm glad you were able to solve your issue.
FYI for updates in the future, I wouldn't have suggested doing the delete 1st because that would remove the CRDs and you could have lost some config/data for Calico. One specific example of what could be lost would be Calico Network Policies (policies created using calicoctl, Kubernetes NetworkPolicies should still be present).
I think it should have worked to just apply the new manifest.
You might want to restart your pods that were running before you did the delete as their IP Addresses may get handed out to new pods so there might be multiple pods with the same IP address which would be problematic. This would be due to the IPPools and pool data being lost when the CRDs were deleted.


home manager wont install on any device

i can't get home manager to install on any devices, i have tried it on 6 devices so far
hauwei mate SE
google pixel
samsung tab S
samsung s6, s7
hauwei media pad
samsung note 4
its all the same error, i will attach an image because select all doesn't work on this lovely little app and it files i write in nix-on-droid are for some reason impossible to find on android (thanks for that)
I've attached images of it

There will be no further complaints from me here, this is broken garbage and i wish you the best of luck fixing it
uninstalled thanks

Whatever. Work on your tone, read the error messages next time, and good luck.


Honoring Base Href in plasmic build

Is it possible to get plasmic to generate relative paths that honor the base href setting?  currently i can't host plasmic rooted on say the /home path of our site.

I did find that i was able to get the JS and CSS from next.js to compile using basePath: /home.... However non of the images are honoring.
Honoring base href would be the most ideal solution

Hello, we found a bug on our "Button" interactive component where it is not honoring the base href setting, do you also have this problem when using directly a Link/ element on studio? either way we are looking into it to provide a fix.

Yes with the button, though how it honors base href should be dependent on how a link is setup.  We still need to be able to use a link like '/docs' to go to a root site location and a link to 'docs' to honor base href.
My specific issue has more to do with the generated index.html file and how it references the js/css files.  When i looked at it more, it seemed a bit of an issue with next.js itself.
When we set the basePath to '/home' so that we could host it on a '/home' path on our site, that worked for the JS/CSS, but did not work for images (images were still trying to pull from root /plasmic instead of /home/plasmic).
I think, in an ideal world, image paths would not be /plasmic but rather just 'plasmic' and js/css would be '_next' instead of '/_next'.  Then, with the use of  i could achieve the local context of plasmic, allowing me to host a plasmic site on any path on my domain, instead of theoretically using plasmic for everything (bit of a micro-frontends approach we have going on).

After writing that, maybe the image issue and the button issue are similar?  And if those were fixed and honored base href, then at least the basePath option in next.config would be back on the table.

Thanks for the detailed report, we will work on this.

Hi @mcgear, are you using codegen with next.js?  If you look at your plasmic.json, do you see that it's using images.scheme: "public-files"?  If so, you can try updating your plasmic.json file to set something like...
"images": {
  "scheme": "public-files",
  "publicDir": "../public",
  "publicUrlPrefix": "/home"
}

The publicUrlPrefix will be prepended to the image src url, which should point to the right place?


Redis get or set inside express route

I am having some severe memory leak issues, I have two endpoints, one does a set and the other one does a get from redis. Each request coming from the client goes to either one route, and there a lot of requests coming in. Would having a get or set inside the express route function cause a memory leak? If it does, is there a way around this?
Example code:
            function (req, res) {
                var id = req.get('id');
                redisClientMaster.get(id, function (err, reply) {
                    // do something with id
                    res.send({code: 100, message: 'SUCCESS', data: returnObject});
                });
            }

Generally speaking, no. Nothing about doing a Redis operation inside a route should cause problems (I do it all the time). Looking at your code, I'm assuming you're handling errors somehow but just haven't shown it, otherwise it looks fairly normal.
I would guess that your memory leak is in your app logic somewhere. I'm going to go ahead and close this as we can't really re-create any conditions with your example code. If you can show us a reproducible example that is causing a memory leak, we'll be glad to take a look.

@AliUz do you have more details? How did you detect the leak and do you have a heap dump that indicates a redis leak?


Patch 1

Add references to stk and chamo:
https://discuss.ocaml.org/t/ann-stk-0-1-0-sdl-based-gui-toolkit-and-chamo-4-0/11770

Thanks!


Add a step to create a new web hook

I'm a new user to Mattermost and just installed this Nagios integration. It took me a while to figure out that you have to use the URL from the incoming web hook as the URL in the command. I'd been trying to use my main Mattermost URL without success.
I think adding this new step will help clarify to new Mattermost users what URL they should be using in the command.

Hi Josh,
Although this plugin assumes that the person already understands how Mattermost works, it can be helpful having extra instructions for newcomers to do a quick setup.
I am accepting your PR, thanks.


Add Litecoin again ?

I really like your wallet. Thanks for building it ! I recall back in mid 2017 that Litecoin was still supported in the settings and then was removed due to a block explorer shutting down. Can you enable Litecoin support again please ? Also is it correct to say I can still use the 2017 version that had Litecoin support to generate offline Litecoin transactions ? (non Segwit addresses only of course). Thanks again.

I enabled Litecoin in https://github.com/Wowee0/coinbin/tree/litecoin, pull request https://github.com/OutCast3k/coinbin/pull/182.  Announced in Reddit (r/litecoin/comments/bf0sfp/coinbin_light_wallet_and_litecoin).  In reference to your 2nd question, the 2017 version uses a block explorer API that no longer exists.

Litecoin support is back as of today.

Litecoins been re-added


isolated usage of logistic model

Hi
I really liked the implementation, especially two way likelyhood estimation for theta and question parameters as well.
I am wondering If I want to do something custom with the 2PL probabilities. Is it good idea to use your  2PL or 4PL model. since you have tweaked the formulae a bit for optimisations.

The 2PL likelihood function is given here by
LOGISTIC(a * x + b).

I understand it won't have any effect on final theta estimation, but in my use case I want to explicitly use probability of solving a question then update theta heuristically.

Hi! I'm so sorry for not answering sooner...
Sure, it's possible to use this in isolation.
Please note, though, that my use of a,b is different from the usual parameterization of he 2PL model, in order to allow for better preformance. Usually the parametrization is such that the likelihood is given by LOGISTIC(a * (x - b)), so that b is the "difficulty" of the question.
To convert from my parameters to the usual ones:
a_usual = a_my
b_usual = -b_my / a_my
To convert back from the usual parameterization to mine:
a_my = a_usual
b_my = -a_usual * b_usual
Again, so sorry for only answering now... Please let me know if this answers your question!
Best regards.


Cache base docker image

Prebuild image with all the build and test dependencies to avoid building it on every test run.
The base image can be updated by running:
make build_and_push_base_docker
The name of the image is: tigrisdata/tigris-build-base

Codecov Report

Merging #426 (6bcc2d7) into main (c7e9b22) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #426   +/-   ##
=======================================
  Coverage   29.74%   29.74%
=======================================
  Files          75       75
  Lines        7887     7887
=======================================
  Hits         2346     2346
  Misses       5302     5302
  Partials      239      239

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


static in array parameters

Thanks to Tachyon.
$ cargo run
void f(int v[static 5]);
<stdin>:1:21: error: expected ')', got '5'

Should parse as an array known to have at least length 5.

At least to start, this should be very simple to implement - just discard the static keyword using match_next. Ping me on discord if you want to implement this, but it would look something like this:
Modify postfix_expr to call self.match_next(Keyword::Static) just after the [ is seen. match_next consumes and returns the given token if it exists, or returns None otherwise.

https://stackoverflow.com/questions/3430315/what-is-the-purpose-of-static-keyword-in-array-parameter-of-function-like-char

Note that int a[static 5] is only valid in function parameter declarations, not in object declarations.


Downgrade freedom-for-firefox

uProxy becomes unusable with freedom-for-firefoxv0.6.11, this changes
the version we require to v0.6.10.

:+1:


mem-ruby: Copyback UD_RU line when evicted in CHI protocol

This is a followed up fix to #791 mem-ruby: Fix possible dirty line loss in CHI when ReadShared hit on UD line.
UD_RU line may have stale data since the upstream could have updated the line, so its local cache line data is treated as invalid (dataValid=false). But when the line is evicted, it must be written back to downstream because the upstream may have the line in clean state (UC). This change fixes it by performing copy back the UD_RU line while keeping its dataValid as false.
Example error case:

L3 was in UD_RSC and being evicted without back-invalidation. LLC (HN) was in RU state.
Because there's still upstream sharer, L3 sends WriteClean.
Because the data state was unique and dirty, L3 sends CBWrData_UD_PD.
LLC becomes UD_RU.
When the line is evicted from LLC (LocalHN_Eviction), the line is just dropped, causing the loss of the dirty copy

@tiagormk

@tiagormk, this is a friendly reminder to look at this PR when you have time. The changes are related to the CHI protocol. Thank you.


Updates to mesh command help text

Updates to mesh command descriptions
Added duration units

@adammil2000: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

operator-lint
6049e866df0f591b485adf2416cce971404f3e3f
link
/test operator-lint

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

@adammil2000: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

operator-lint
6049e866df0f591b485adf2416cce971404f3e3f
link
/test operator-lint

operator-build
6049e866df0f591b485adf2416cce971404f3e3f
link
/test operator-build

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

@adammil2000: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

operator-lint
6049e866df0f591b485adf2416cce971404f3e3f
link
/test operator-lint

operator-build
6049e866df0f591b485adf2416cce971404f3e3f
link
/test operator-build

operator-mandiff
6049e866df0f591b485adf2416cce971404f3e3f
link
/test operator-mandiff

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

@adammil2000: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

operator-lint
6049e866df0f591b485adf2416cce971404f3e3f
link
/test operator-lint

operator-build
6049e866df0f591b485adf2416cce971404f3e3f
link
/test operator-build

operator-mandiff
6049e866df0f591b485adf2416cce971404f3e3f
link
/test operator-mandiff

operator-test
6049e866df0f591b485adf2416cce971404f3e3f
link
/test operator-test

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


no-icu: make no icu option is usable and testable

no-icu CI will be enabled after this PR is merged.

@jackhorton Thanks for the review


3.4.6 isn't in the change log

Summary
Nice & simple one. Version 3.4.6 isn't in the change log.
Expected behaviour
Changes for version 3.4.6 to be detailed in CHANGELOG.txt
Actual behaviour
Details of 3.4.6 are missing from CHANGELOG.txt

Hello @marksweb,
Thank you very much for your contribution to the django CMS project. The django CMS project has recently joined forces under the banner of the newly founded django CMS Association to revive the development of django CMS. For this reason we have analyzed all pull requests and issues in detail and decided which ones fit the product vision we want to go with django CMS.
Due to the age of this issue we are unable to safely accept the contribution. If you feel this issue is still required please inform a member of our technical committee<EMAIL_ADDRESS>Nevertheless we thank you for your participation and would like to emphasize that we do not want to discourage you in your contribution to the django CMS project. On the contrary, we encourage everyone who is interested in the success of django CMS to join us!
If you want to learn more about the work of the technical committee, our product vision and how you can become a member of the dCA, you can find all information here:

The Technical Committee of the django CMS Association
Technical Roadmap and Product Vision for django CMS
How to become a member of the django CMS Association

We are looking forward to every companion who would like to shape the future of django CMS together with us!
If you have any questions, feel free to reach out to Nicolai, our Community Manager<EMAIL_ADDRESS>Sign up for the Association now!


not really an issue -

without desk I would never be using many awesome CLI tools because I'd never remember the syntax. For example, transfer.sh.  Maybe you could tell them that instead of telling people to use aliases, they could use desk !

Glad to hear, @jonpolak.


Test versions do not match for Triangle exercise

Setup file triangle.go is 2, actual test is 3.

Ouch. You are absolutely right.
How long has it been like this?
Wow, four months, since #356. That's embarrassing for us, sorry.

Thank you for reporting. We should definitely fix it.


How can I get the image size before uploading?

How can I get the image size before uploading? I need to get the size to validate that the image is at least X pixel width and height.
I know I can do it like this in a directive that I apply to the <img> element itself.
		link: function(scope, element, attributes) {
			element.on('load', function () {
				scope.cmsImageSize.originalHeight = this.height;
				scope.cmsImageSize.originalHidth = this.width;
			});
		}

But how do I get it from within my files I'm watching? I've spent now some time inspecting the objects I have there but couldn't figure out from where I could get the image dimensions.
$scope.$watch('data.files', function() {
				for (var i = 0; i < $scope.data.files.length; i++) {
					var file = $scope.data.files[i];
					$scope.upload = $upload.upload({
						url: '/api/v1/image_files/upload.json',
						method: 'POST',
						data: {
							'data[ImageFile][model]': 'CmsPageElementImage',
							'data[ImageFile][foreign_key]': $scope.element.CmsPageElement.id
						},
						file: file,
						fileFormDataName: 'data[ImageFile][file]'
					}).progress(function (evt) {
                                            /* ... */
					}).success(function (data, status, headers, config) {
                                            /* ... */
					});
				}
			});

OK, I've managed to get it to work with this code. Guess this is how it has to be done in JS, looks like an ugly work-around to me.
					var fr = new FileReader;
					fr.onload = function () { // file is loaded
						var img = new Image;
						img.onload = function () {
							alert(img.width); // image is loaded; sizes are available
						};
						img.src = fr.result; // is the data URL because called with readAsDataURL
					};
					fr.readAsDataURL(file); // I'm using a <input type="file"> for demonstrating

Thanks burzum, this worked for me.

If you have ngf-max-width or similar validation, the width and height of the image will be calculated automatically and you can get it by file.width or file.height.
You can also use Upload,imageDimensions(file).then(success, error)

Danial, even better.  I never saw the width and height properties on file, but that must be because I was not using validation.  I will just set it very high as I don't want to limit dimensions, in order to access those properties.  Thanks.

Thanks guys. @drichfield you can use ngf-min-height="1" or ngf-min-width="1" instead, I think is better. @danialfarid there is a way to get the image resolution in ppp and the deep in bits? Thanks again.

@grutta16 create a separate issue for that request. Not at the moment.

@danialfarid Hi Danial, I'm trying to use your tool with large size images but i don't se ngf-max-width with 300MB images... there something particular to setup? I'm more interested to get the image width and heigh than limit the size.

Use 'Upload.imageDimensions(file).then()`

sorry it is what i have tried but I ended in error Value: "load Error" there something else I miss?

Are you able to show the image using <img src='url to the image'> on the page?

I don't want to do it because it's too big... and i risk to hang the browser it's up to 60,000PX x 20,000PX and 250GB size
I'm looking at a way to get size without opening the image like from General informations in Operation system

I don't think that is possible, the file information you get from the browser when you select the file through popup is very limited, only type, size and name. You need to load the image in the browser to get the image dimensions.

thank's for the reply i will look somewhere else in processing the image after upload... I don't want to hand the uploader


Fix prefix and suffix, commands execution otimization.

About permissions; It should do some checks before starting call functions that will not be used in some cases.
Example:
IRocketCommand command = R.Commands.GetCommand(requestedCommand);
double cooldown = R.Commands.GetCooldown(player, command);

If the command does not exist, rocketmod should not call GetCooldown.
About prefixes and suffixes: i changed the event from Provider.onCheckValid to Provider.onCheckValidWithExplanation and some otimizations.
btw rocketmod currently listen Provider.onCheckValidWithExplanation so Provider.onCheckValid is never called

This sounds like a good change, especially fixing the usage of onCheckValid. Will examine further soon.

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


automatically subdivide large polygons

If a polygon is too large (one or more side's length is >100) then the level generator should slice it up in a way that each piece fits into a 100x100 quad.

[x] create Polygon.isTooLarge()
[x] create dummy Polygon.subdivide() with nothing inside (maybe add a // TODO: implement)
[x] in ArxMap.finalize() go through every polygon and if it .isTooLarge() then .subdivide()
[ ] implement Polygon.subdivide()

Having any of the sides longer than 100 isn't a good way to tell if a polygon's too large, here's an exception:

https://math.stackexchange.com/questions/4791032/triangle-inside-a-unit-square

what about EntityModels?


jquery null ref

I'm probably missing something fairly obvious here... but I'm trying to make a tooltip style popper and I keep getting some jquery error:

Component code:
class InfoTip extends Component {
  state = {
    visible: false,
  }

  onMouseOver = () => {
    this.setState({visible: true});
  }

  onMouseOut = () => {
    this.setState({visible: false});
  }

  render() {
    return (
      <Manager>
        <Target>
          {({targetProps}) => {
            return (
              <InfoIcon
                dark
                style={{cursor: 'help'}}
                onMouseEnter={this.onMouseOver}
                onMouseLeave={this.onMouseOut}
                {...targetProps}
              />
            );
          }}
        </Target>

        {this.state.visible && (
          <Popper placement="right" className="popper">
            {this.props.children}
            <Arrow className="popper__arrow"/>
          </Popper>
        )}
      </Manager>
    );
  }
}

Any thoughts?

Seems like react-popper can't get your target element, I think you should assign targetProps.ref to a property of your InfoIcon component that proxies the ref of its internal element.

ahhh yes, so it looks like because InfoIcon was a functional stateless comp it wasn't working.  I added a span around it and tied the targetProps to that and it seems to work now, thanks for the direction!

Hi there, I'm having a similar issue when using react-travel to render the Popper outside my component. To be clear, the code works in the browser I only get the errors when running Jest snapshot tests.
This is what the error looks like:

My code looks like this:
<Manager className='ifl-component ifl-dropdown'>       <Target>         <DropdownInput           name={name}           storeName={storeName}           className={className}           refCallbackFocusElement={localState.refCallbackFocusElement}/>         {localState.isClearable && <DropdownClearButton name={name} storeName={storeName}/>}         <DropdownArrowButton name={name} storeName={storeName}/>       </Target>       <Portal>         <Popper placement={'bottom-start'} className='ifl-dropdown-options' modifiers={popperModifiers}>           <DropdownOptions name={name} storeName={storeName}/>         </Popper>       </Portal>     </Manager>

Neverming, I figured it out. React-travel has a prop called 'useArray' which, when absent, causes the Protal to render as 'null' which caused the jQuery error from above.


feat: add js.doc

Description
This PR includes the addition of Jsdocs to the NestJs server application.
Related Issues
#74
Check list before submitting

[x] I have performed a self-review of my code (no debugs, no commented code, good naming, etc.)
[x] I updated the documentation if necessary
[x] This PR is wrote in a clear language and correctly labeled

@SubstantialCattle5 thank you for your work, it looks great !
Few things though: it seams that the CodeFactor CI detected minor style issues on your code, can you check it out ? We have also the master branch that evolved and created some conflicts.
Thank you very much

Yeah i will and do the changes asap


Fix NPE when resolving module version from including build

project will not be in projectRegistry, if the project to be resolved is in another included build,
potentially the including build.
E.g. build A includes builds B, B includes build C. C has some dependencies from B, which have been substituted to project references.
In that case, projectRegistry does not know about those projects and returns null.
OTOH, 'selected' does have a valid module version.
Contributor Checklist

[X] Review Contribution Guidelines
[X] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[X] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[X] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[X] Ensure that tests pass sanity check: ./gradlew sanityCheck
[X] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

The issue can be reproduced from a bigger OSS piece I'm working on.
The relevant parts are:

project "A" including build "B" from the PR description: https://github.com/snazy/nessie-integration-tests/blob/main/settings.gradle.kts#L135
project "B" including build "C" from the PR description: https://github.com/snazy/nessie/blob/tools-integration-tests/settings.gradle.kts#L254
project "C" is plain https://github.com/apache/iceberg
project "C"'s dependency, substituted to a project in "B": https://github.com/apache/iceberg/blob/master/build.gradle#L594

@bot-gradle test this

Thank you for the contribution!
This issue hints there could potentially be a deeper problem. In order to investigate this more efficiently it would be helpful if you could provide a minimal reproducer project.

@snazy, I don't have the bandwidth for now to investigate this. Could you provide a reproducer project using a recent release of Gradle? (e.g. 8.2.1)

I don't have a reproducer

Sorry for the late reply.
@snazy Please add tests to cover your changes.
This PR will be closed in 14 days otherwise.


Login not possible after upgrade of CherryPy to 8.1.2

On a brand-new installation the login is no longer working on dev. The upgrade (#3805) to CherryPy 8.1.2 seems to cause the issue.
Home Assistant release (hass --version):
0.31.0.dev0
Python release (python3 --version):
Python 3.5.1
Component/platform:
http
Description of problem:
Login is not possible -> Unexpected result from API and traceback.
Expected:
Access to frontend.
Problem-relevant configuration.yaml entries and steps to reproduce:
http:
  api_password: 12345

Traceback (if applicable):
16-10-11 19:55:39 INFO (CP Server Thread-13) [homeassistant.components.http] Serving /api/bootstrap to <IP_ADDRESS> (auth: True)
TypeError('unorderable types: int() < str()',)
Traceback (most recent call last):
  File "/home/fab/Documents/repos/home-assistant/lib64/python3.5/site-packages/cherrypy/wsgiserver/__init__.py", line 1408, in communicate
    req.respond()
  File "/home/fab/Documents/repos/home-assistant/lib64/python3.5/site-packages/cherrypy/wsgiserver/__init__.py", line 862, in respond
    self.server.gateway(self).respond()
  File "/home/fab/Documents/repos/home-assistant/lib64/python3.5/site-packages/cherrypy/wsgiserver/__init__.py", line 2333, in respond
    response = self.req.server.wsgi_app(self.env, self.start_response)
  File "/home/fab/Documents/repos/home-assistant/homeassistant/components/http.py", line 436, in __call__
    return app(environ, start_response)
  File "/home/fab/Documents/repos/home-assistant/lib64/python3.5/site-packages/werkzeug/wsgi.py", line 659, in __call__
    return app(environ, start_response)
  File "/home/fab/Documents/repos/home-assistant/homeassistant/components/http.py", line 410, in base_app
    response = self.dispatch_request(request)
  File "/home/fab/Documents/repos/home-assistant/homeassistant/components/http.py", line 393, in dispatch_request
    return self.views[endpoint].handle_request(request, **values)
  File "/home/fab/Documents/repos/home-assistant/homeassistant/components/http.py", line 525, in handle_request
    result = handler(request, **values)
  File "/home/fab/Documents/repos/home-assistant/homeassistant/components/frontend/__init__.py", line 171, in get
    'panels': PANELS,
  File "/home/fab/Documents/repos/home-assistant/homeassistant/components/http.py", line 541, in json
    result, sort_keys=True, cls=rem.JSONEncoder).encode('UTF-8')
  File "/usr/lib64/python3.5/json/__init__.py", line 237, in dumps
    **kw).encode(obj)
  File "/usr/lib64/python3.5/json/encoder.py", line 199, in encode
    chunks = self.iterencode(o, _one_shot=True)
  File "/usr/lib64/python3.5/json/encoder.py", line 257, in iterencode
    return _iterencode(o, 0)
TypeError: unorderable types: int() < str()

Hmmm, another installation no issue.
Guess that was a false alarm.

Works ok on Firefox (windows) and server on Centos :+1:

Chromium works fine with and without password as well. Only that single test instance has spat out the traceback.

As nobody else is complaining I will close that.


[Bug]: PrestaShop set two cookies and caused problems with session management

Actual Behavior
When PrestaShop website tries to set two cookies we will encounter issues. However, according to the documentation we must have only one cookie:
https://devdocs.prestashop-project.org/8/development/components/cookie/#limits
PrestaShop will create only one cookie, in the $_COOKIE array, with each key and value data appended to a string and finally encrypted for safety reasons.

How I encountered the issue?
When I logged into the shop and tried to do an add to cart/ or checking account info, the user will be logged out on front-end. Why? because it uses wrong cookie.

I think we should not handle this situation on front-end.

third-party modules can cause having many cookies:
https://github.com/PrestaShop/PrestaShop/issues/29868

This issue is not related to front, it must be fixed on back-end instance, so I close it.


Markdown extensions, consider coauthor package

coauthor
also allows for a number of extensions to standard markdown. Some of them might be very useful to consider (and it has the regexes). It's all under MIT license so can easily mention & inspire.

For instance the spacing could be nice (midskip etc) but really not high on the priority list


Support/docs for using date and time pickers simultaneously

I can see the need for being able to pick both the time and date. Currently, this sort of works, but is buggy.
I would like to see a demo/docs of using the mdp-date-picker and mdp-time-picker on the same form on the same model object.
I gave this a shot with the following code, but changing one picker's value would cause the other picker's required message to appear.
                    <mdp-date-picker name="startDate" mdp-placeholder="Start date" ng-model="scenario.startDate" required>

A start date is required

                    </mdp-date-picker>
                    <mdp-time-picker name="startTime" mdp-placeholder="Start time" mdp-auto-switch="true" ng-model="scenario.startDate" required>

A start time is required

                    </mdp-time-picker>

I forked the CodePen demo and added code demonstrating the issues I hinted at above: http://codepen.io/shawmanz32na/pen/yOPJZB
To see obvious problems, specifically around required validation:

Click the Both pickers time picker clock icon and cancel out of the time picker - One would expect a This is required message to appear due to failed validation, but the picker seems hunky-dory ( :disappointed: )
Click the Both pickers date picker calendar icon, pick a date (any date), and click OK - This all looks great! ( :smile: )
Click the Both pickers time picker clock icon and select a time (any time), and click OK - One would expect the validation to pass and no error messages to display, but the Both pickers date picker shows a This is required error message ( :disappointed: )


Assert fee/weight ordering of transactions

In the Transaction_pool unit tests, add another assertion on the pool, that transactions provides commands ordered in decreasing fee/weight order, subject to nonce ordering for fee payers.
In Indexed_pool.get_highest_fee, use specific Set and Map modules, rather than the Stdlib ones.
Note: get_highest_fee seems to use min_elt_exn as a tie-breaker, when multiple commands have the same fee/weight. Adding that tie-breaker to the sort breaks the assertion for some tests. Omitting the tie-breaker allows all tests to succeed. I don't understand why adding the tie-breaker breaks tests, but I think omitting it is OK, because the assertion nonetheless establishes that transactions are ordered by decreasing fee/weight.
Part of #10067.

In some of the tests, the expected list is not ordered based on transaction hash, probably why it fails. but that's fine for the sake of this test

the expected list is not ordered based on transaction hash

My understanding is that transactions, which relies on get_highest_fee, uses the transaction hashes to impose a total order when the fee/weights are tied. But adding the hash comparison to the sort breaks a few tests, not sure why.


🛠 [TASK] : User Input Component - Tag Selection

Summary
As a proposer, I need to be able to assign a category / tag to my proposal, so that it can be discovered by other users with similar interests.
Use Cases:

Horizons / Theme

Acceptance Criteria

[ ] Default value should show "no tag selected"
[ ] In edit mode, user only see category dropdown at first. Upon selection, the the tags for that category appear below as selectable chips
[ ] User may select only 1 category / tag per proposal
[ ] In filled state, category and tag to be show as a single theme in the format " : "
[ ] Both selections must be made before user can save

Designs
https://www.figma.com/design/1e1MMDXyi0oL9sG9IunWL7/UI-Kit-Figma-for-catalyst?node-id=5271-49626&t=gTh3B1j2eYONQrjf-0
Default

Selected

Editing

Filled

@mike-mcnulty Added AC item:  Both selections must be made before user can save


Svelte API 404's when hooks and SSR is used together

Describe the bug
Using hooks.ts at all will any API endpoints to return a 404 error
The strangest part is this error only occurs when ran on a windows environment when tested on a linux environment the error does not occur
Reproduction
https://github.com/ok-hackers/Production/tree/user_settings_page
the issue seems to have to do with hooks.ts as deleting it stops the problem from appearing
import { sequence } from '@sveltejs/kit/hooks';

async function SSRAll({ event, resolve }) {
	const response = await resolve(event, {
		ssr: false
	});

	return response;
}

export const handle = sequence(SSRAll);

Logs
Browser Logs

404

Not found: /LearnYourNumbers/1

_get_navigation_result@http://localhost:3000/.svelte-kit/dev/runtime/internal/start.js:940:11
update@http://localhost:3000/.svelte-kit/dev/runtime/internal/start.js:766:38
handle_navigation@http://localhost:3000/.svelte-kit/dev/runtime/internal/start.js:755:14
_navigate@http://localhost:3000/.svelte-kit/dev/runtime/internal/start.js:413:23
goto@http://localhost:3000/.svelte-kit/dev/runtime/internal/start.js:282:16
start@http://localhost:3000/.svelte-kit/dev/runtime/internal/start.js:1377:19
@http://localhost:3000/LearnYourNumbers/1:11:10

Server Console
Cannot read property 'url' of undefined
TypeError: Cannot read property 'url' of undefined
    at resolve (file:///S:/Desktop/New%20folder/node_modules/@sveltejs/kit/dist/ssr.js:1829:39)
    at SSRAll (s:/Desktop/New%20folder/src/hooks.ts:4:32)
    at apply_handle (file:///S:/Desktop/New%20folder/node_modules/@sveltejs/kit/dist/hooks.js:20:11)
    at Object.handle (file:///S:/Desktop/New%20folder/node_modules/@sveltejs/kit/dist/hooks.js:10:10)
    at respond (file:///S:/Desktop/New%20folder/node_modules/@sveltejs/kit/dist/ssr.js:1809:30)
    at file:///S:/Desktop/New%20folder/node_modules/@sveltejs/kit/dist/chunks/index.js:205:30
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

### System Info

```shell
System:
    OS: Windows 10 10.0.19042
    CPU: (12) x64 AMD Ryzen 5 2600X Six-Core Processor
    Memory: 21.29 GB / 31.93 GB
  Binaries:
    Node: 16.7.0 - S:\Utility\nodejs\node.EXE
    npm: 7.7.5 - S:\Utility\nodejs\npm.CMD
  Browsers:
    Chrome: 97.0.4692.99
    Edge: Spartan (44.19041.1266.0), Chromium (97.0.1072.69)
    Internet Explorer: 11.0.19041.1202
  npmPackages:
    @sveltejs/adapter-auto: next => 1.0.0-next.10
    @sveltejs/kit: next => 1.0.0-next.229
    svelte: ^3.44.0 => 3.46.1

Severity
blocking all usage of SvelteKit
Additional Information
PS. This is my teams senior project for our college so the blocking of 3 out of 4 of our developers is a major issue for us.
Any advice would be appreciated.

Your system info has you on SvelteKit next.229, but you're using event in your hook, which was not introduced until next.234. Try updating to the latest SvelteKit and see if that fixes it.
I suspect that the reason it works on Linux is that you have a different SvelteKit version installed in that environment.

Your system info has you on SvelteKit next.229, but you're using event in your hook, which was not introduced until next.234. Try updating to the latest SvelteKit and see if that fixes it.
I suspect that the reason it works on Linux is that you have a different SvelteKit version installed in that environment.

This seems to have resolved the issue. Thank you so much. lesson learned to always check for updates before banging my head against it for a few hours.


Sort by difference in Fields Counts

The resulting df is sorted by a field name, it makes more sense to sort it by difference instead.
                                                    Difference, %
_validation                                                   100
_validation/price_now                                         100
_validation/price_was                                         100
configurations/description                                     65
configurations/name                                            65
configurations/options                                         65
delivery_options/date_range                                    13
delivery_options/name                                          13
delivery_options/price                                         13
spec_10th_Hard_Drive                                          100

Fixed in the past