Garbage collect old page versions

Design and implement rules for which page versions need to be retained in the page server.
Currently, the Page Server keeps all page versions. That obviously doesn't scale, you will run out of disk space eventually.
Timeline, newer LSNs on the right:

------+-----#####-------------+-------+-----++++++++############>
     (4)     (5)             (3)     (3)       (2)      (1)

Need to keep X amount of recent page versions, for read replicas.

To support point-in-time recovery to an arbitrary point, need to
keep all page versions back to X hours or days. No need to keep
intermediary states between transaction commits for PITR,
though. User-visible changes only happen at commits.

Need to keep any explicitly tagged points in time ("zenith tag"), aka
snapshots.

When a new timeline is branched, need to keep the page versions at
the branch point for as long as the new timeline needs them.

Admin can request to keep everything in any additional time
range. Maybe? Do we need this? I think we will have the
capability anyway, to support (1).

Need to implement UI to specify these things. and the logic to implement these policies in the storage.

Hi @hlinnaka
When you have a chance can you:

assign this issue to someone (I'm doing a general pass through the issues in the v0.1. milestone and noticed this issue did not have an assignee)
provide a general update on current status of this issue

Hi @hlinnaka this item was put into the done category during the milestone v0.2 planning session on 2021-07-30. I'm closing out. Please re-open if I am mistaken.


Add 1.27 template env for e2e

Issue #, if available:
Description of changes:
Add 1.27 template envs for e2e tests
Testing (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/test eks-anywhere-presubmit

/lgtm


Travis CI on Linux is currently broken

Linux images update:

https://blog.travis-ci.com/2017-08-29-trusty-image-updates

Current issue:

https://github.com/travis-ci/travis-ci/issues/8363#issuecomment-327909904

Cancelled builds (to restart later):

https://travis-ci.org/ingenue/hunter/builds/272937751
https://travis-ci.org/ingenue/hunter/builds/272381382

What is the blocker on this right now?

Mess with the PATH.

Workaround https://github.com/ruslo/hunter/commit/8a950294b6574d8973af4cd1a33f955609f3e5c7


Backported TimeDrift check

Up-ped Planet version to backport TimeDrift check to 5.5.x

Upgrades are failing with:
{"checker":"ping-checker","status":"running"},{"checker":"time-drift","detail":"failed to check time drift","status":"failed","error":"rpc error: code = Unimplemented desc = unknown method Time"}

it seems that upgrades are failing when upgrading from 5.2.x cause those checks are non-existent in there yet... I'll investigate why.

@eldios I remember when working on this check, I accounted for the rolling update use-case and made sure that it tolerates a "not implemented" error (which is what gRPC API returns for unknown methods): https://github.com/gravitational/satellite/blob/7.0.1/monitoring/timedrift.go#L205. Maybe in 5.2 the error is somehow not wrapped correctly (I think trace.NotImplemented was introduced not too long ago)?

closing the PR for now to fix it properly.


Add replica-level Service Fabric Events for release 6.5

@srrengar - Would you review the proposed changes? Thanks

#sign-off. Thanks @derekeom !


feat: add fake secret store provider for test

What type of PR is this?
/kind feature

What this PR does / why we need it:
add fake secret store provider for testing
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Additional documentation e.g., design docs, usage docs, etc.:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.04%) to 76.926%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
8638

Relevant Lines:
11229

💛  - Coveralls


Add last-resort/backup system for drawing save events

Had a few reports of folks loosing drawings after spending a lot of time editing (and not saving) either due to session timeouts or others.
Would be best to store image data to the browser before pushing to the server, for potential access/restore in the event something goes wrong.
Not yet sure what the UI would look like, or where it even fits in.
Maybe check and prompt to restore on diagrams.net boot, if old content found in store?

Merged in https://github.com/BookStackApp/BookStack/pull/4457


Release29

This is a PR for Release 29.
Status:

[x] In development
[ ] In testing at NRK
[ ] Finalize and merge

Release 29 overall status: https://github.com/nrkno/Sofie-TV-automation/issues/26

Codecov Report

Merging #413 (95f01fa) into master (b11ca6a) will decrease coverage by 0.22%.
The diff coverage is 56.04%.

@@            Coverage Diff             @@
##           master     #413      +/-   ##
==========================================
- Coverage   68.89%   68.67%   -0.23%
==========================================
  Files         198      213      +15
  Lines       13202    14275    +1073
  Branches     3129     3357     +228
==========================================
+ Hits         9096     9803     +707
- Misses       4027     4298     +271
- Partials       79      174      +95

Impacted Files
Coverage Δ

meteor/client/lib/invalidatingTime.ts
18.18% <0.00%> (ø)

meteor/lib/Settings.ts
100.00% <ø> (ø)

meteor/lib/api/migration.ts
100.00% <ø> (ø)

meteor/lib/api/peripheralDevice.ts
97.50% <ø> (ø)

meteor/lib/api/playout.ts
100.00% <ø> (ø)

meteor/lib/collections/AdLibActions.ts
100.00% <ø> (ø)

meteor/lib/collections/AsRunLog.ts
100.00% <ø> (ø)

meteor/lib/collections/Blueprints.ts
100.00% <ø> (ø)

meteor/lib/collections/BucketAdlibs.ts
100.00% <ø> (ø)

meteor/lib/collections/ExpectedMediaItems.ts
100.00% <ø> (ø)

... and 100 more

Continue to review full report at Codecov.

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


Hi @DeveloperAspire, kindly take a look at my change.
What is the default command that will make people able to install the Next.js template with swifty-config. I also noticed an index.js file in the root folder. What do I need to do in it, so that the command is extended from it.

@kaf-lamed-beyt Hey Chief! Great work! Amazing stuff you have done.
I will review this as soon as I get home and make comments when needed.
However, for the command, I think you will understand it better when you try and run the Swifty Config locally and you will see how it works.
run yarn create swifty-config to see how it all comes together.
if it still not clear, we can have a call for me to explain better
Great job so far!

Thank you for the opportunity, chief!

I will review this as soon as I get home and make comments when needed.

I'll be on the lookout for this, thanks! 😃

I'll also look at the config, by running the command, when I'm chanced. And please accept my apologies for being slow about this 🙏🏼

Made a mistake with my last commit message 🤣
@DeveloperAspire, kindly review

@kaf-lamed-beyt I apologize that I haven’t reviewed this or given you feedback, please bear with me. I have a lot of activity taking up my time. When I settle I will review and drop comments where necessary.
Thank you for the good work. I am grateful!

Ah! It's nothing! I understand 👍
Do take a look at the new issue I created BTW #3

Hi @DeveloperAspire, kindly take a look at my change.
What is the default command that will make people able to install the Next.js template with swifty-config. I also noticed an index.js file in the root folder. What do I need to do in it, so that the command is extended from it.

Hey @kaf-lamed-beyt  So for the default command. What are you thinking it should be? You know that NextJS already has it's own command. Personally I don't think people will trust a random package command more than the command given by the package itself.
If you have an idea how this can be done, I will really apperciate if you do.

Ah! It's nothing! I understand +1
Do take a look at the new issue I created BTW #3

Thanks for for understanding. Yes I have seen the issue. Thank you for the good work.

Made a mistake with my last commit message rofl
@DeveloperAspire, kindly review

Hey, @kaf-lamed-beyt  Thank you for all your hard work on this. I am delighted!. Your PR is neat.
Here are some reviews I think will make your work better so we can merge it.
First I will like to explain what Swifty Config actually those, NextJS and React, and Vue all have their default commands developers can use to get started. But here is the thing. Let's say you are in a hackathon and you need to set up a project fast, you know stuff like Vite and CRA will just give you a bare-bone project folder, You might need SCSS, Tailwind CSS, Chakra UI, and Redux.
The stress of setting up all those stuff can be pretty cumbersome. My goal for Swifty Config (Which has become your goal too) is to project a command that lets you spin up the template that has these basic configurations. In the future, we would have templates for Redux, Chakra UI, TailwindCSS, Redux etc. So that people can forget the hard part of configurations and start building immediately.
Note: If you have more ideas on how this can be done, do share.
That being said. The NextJS template PR is lacking a few things I would be pleased you added.

Eslint config (Check the existing templates and copy and paste them)
SCSS ( This is the default styling, we can create another styling template with Tailwind CSS)
Stylint (Check the existing templates and copy and paste them)

Then the home page style. It will be cool if the home page styles have the same look.
You can copy and paste the styling for this.
Here is how it should look:

I believe this is all that is needed and we are good to go.!
Thank you for lending a hand to work on this project. I am so glad you are!
Looking forward to seeing your next comment!

Hey @kaf-lamed-beyt So for the default command. What are you thinking it should be? You know that NextJS already has it's own command. Personally I don't think people will trust a random package command more than the command given by the package itself.

I think the default command is create-swifty-config [app-name], yeah? Or do you mean the default config for each template, such that people can be able to do something like this? 👇🏼
csc next-ts || react-ts || vanilla-ts

Where csc is short for create-swifty-config?

And overall, this is just too awesome of a feedback! Thanks for this mehnn! 😄

The stress of setting up all those stuff can be pretty cumbersome. My goal for Swifty Config (Which has become your goal too) is to project a command that lets you spin up the template that has these basic configurations. In the future, we would have templates for Redux, Chakra UI, TailwindCSS, Redux etc. So that people can forget the hard part of configurations and start building immediately.

This idea sounds incredible, tbh!

That being said. The NextJS template PR is lacking a few things I would be pleased you added.
Eslint config (Check the existing templates and copy and paste them)
Stylint (Check the existing templates and copy and paste them)

Okay... while this actually sounds good. I think we should add an option for people to decide if they want to have these linters in their projects. I personally do not like them, and I think there may be a few other folks who are like me too, who wouldn't like them.
So I'm suggesting that we make it optional. If accepted, I can create a PR for this, as we progress.
For now, I'll go and modify the homepage to conform with the styles you've already written, and I'll also modify the README file.
cc: @DeveloperAspire

I'm having an issue with replacing the styles though. I tried copying all the content in the styles folder from the react template, and I placed it inside the next template. One of the issues I faced was the fact that the font was not being applied, and the background color itself isn't being applied.
I had to apply an arbitrary bg color to the component, and this is what I am seeing. 😄

The stress of setting up all those stuff can be pretty cumbersome. My goal for Swifty Config (Which has become your goal too) is to project a command that lets you spin up the template that has these basic configurations. In the future, we would have templates for Redux, Chakra UI, TailwindCSS, Redux etc. So that people can forget the hard part of configurations and start building immediately.

This idea sounds incredible, tbh!
So, all in all now, what this project encompasses is the ability to provide a ready-to-use config for people to use any day, anytime, with some of the stylings too, yeah?

Yes exactly!

Yes! This is part of the goal! To make it optional! Please do create an issue for it.

That being said. The NextJS template PR is lacking a few things I would be pleased you added.
Eslint config (Check the existing templates and copy and paste them)
Stylint (Check the existing templates and copy and paste them)

Okay... while this actually sounds good. I think we should add an option for people to decide if they want to have these linters in their projects. I personally do not like them, and I think there may be a few other folks who are like me too, who wouldn't like them.
So I'm suggesting that we make it optional. If accepted, I can create a PR for this, as we progress.
For now, I'll go and modify the homepage to conform with the styles you've already written, and I'll also modify the README file.
cc: @DeveloperAspire

Yes! This is part of the goal! To make it optional! Please do create an issue for it.

Did you install SCSS?

Oh! Yeah. I think I may have installed the wrong one... I did 👇🏽
npm install node-sass

Are they somewhat different?

Yes! This is part of the goal! To make it optional! Please do create an issue for it.

Awesome! I'll do that now and mention you!

Did you install SCSS?

Oh! Yeah. I think I may have installed the wrong one... I did 👇🏽
npm install node-sass

Are they somewhat different?

Yeah, I think they are different.
Try: npm install sass.
Also check if you imported the files properly. See how it is done in the other templates. Something must be off somewhere, you know how these things can be 😂

Sorry for circling back soo late. I'll definitely give it a go, and get back to you 😃

Sorry for circling back soo late. I'll definitely give it a go, and get back to you 😃

@kaf-lamed-beyt  Hey Chief, How is it going? Any progress? How you doing man? Sent you a message on Twitter.

Chief!!! I am so sorry. I completely forgot. I was carried away with the current yarnings in the nation and some of my backlog of work ☹

Hi @DeveloperAspire<
I tried npm install sass and I'm still getting the same thing.
import React from "react";
import styles from "../styles/app.module.scss";
export default function Home() {
return (

⚡ Swifty Config

A Frontend project configuration tool 🛠.

With one command, setup your Frontend project with additional
configuration out of the box.

);
}

Hi @DeveloperAspire
I tried npm install sass and I'm still getting the same thing.

I even tried changing the way the styles are being used in  _app.js Formerly, yours was like this 👇🏼
import React from "react";
import styles from "../styles/app.module.scss";

export default function Home() {
  return (

⚡ Swifty Config

          A Frontend project configuration tool 🛠.

          With one command, setup your Frontend project with additional
          configuration out of the box.

  );
}

So I changed it to this 👇🏼, and I'm still having this blank screen.
import React from "react";
import styles from "../styles/app.module.scss";

export default function Home() {
  return (

⚡ Swifty Config

          A Frontend project configuration tool 🛠.

          With one command, setup your Frontend project with additional
          configuration out of the box.

  );
}

@kaf-lamed-beyt Hey Chief, sorry for the stress in all this. I will see to this issue, please do me a favour by pushing your latest changes to your forked version, I want to clone and reproduce locally so I can look into the issue.
Sorry for the stress Chief. Thank you for your time. and greate work so far!

Alright chief.  I'll push both templates today.


Add Selection-sort in C by vansh-patel-8123

Pull Request Type

[X] Sample Program Documentation
[ ] Language Documentation
[ ] Project Documentation
[ ] Other (explain below)

Notes
Adding Selection-sort in c

please add hacktober fest label


Cant remove event listeners

Somehow this slipped past both the unit tests and half a year of production.
const { render } = brynja;

render(_=>_
  .child('button', _=>_
  	.text('first render')
  	.on('click', () => {
    	console.log('first render');
    })
  )
);

render(_=>_
  .child('button', _=>_
  	.text('second render')
  	.on('click', () => {
    	console.log('second render');
    })
  )
);

Clicking the button will log both first render & second render.
However it should only log second render.

Related to #22

Have you considered to create a proxy for the events instead of removing the builtin ones?


Processors are not called on LogRecords prior to calling Handler filter methods

I have an application in which we use a Processor to conditionally add information to LogRecords.  Later on in our pipeline, we filter LogRecords based on information that was added by the Processor.  This pattern no longer works as of https://github.com/mitsuhiko/logbook/commit/43a63589b318482edbc990faa079017669b0ff36, because that change moves the check on handler.filter to before the call to self.process_record.
A simple fix would be to move the call to process_record to happen prior to the filter check in RecordDispatcher.call_handlers, but it's unclear to me whether that's correct because the current implementation is explicitly short-circuiting when blackhole = True, presumably to avoid doing unncessary processing.

(bump) We just upgraded logbook to 0.9.0, not realizing that this hadn't been fixed yet, and it broke our app logging again. sigh Any chance we can get some attention to this?

I can try and get to it today or tomorrow. A PR with a failing test would
help a lot.
On יום ג׳, 31 במרץ 2015 at 19:18 Jonathan Kamens<EMAIL_ADDRESS>wrote:

(bump) We just upgraded logbook to 0.9.0, not realizing that this hadn't
been fixed yet, and it broke our app logging again. sigh Any chance we
can get some attention to this?
—
Reply to this email directly or view it on GitHub
https://github.com/mitsuhiko/logbook/issues/112#issuecomment-88152563.

Ok, I pushed a naive fix for this, which basically means:

Handlers which are black holes without filters will avoid heavy init and discard immediately
Handlers which are black holes with filters will first heavy initalize, then filter, and then discard if the filter matches
All other cases will first heavy init, then filter as requested.

The shouldHandle method remains the fast way to discard records while avoiding heavy init.
Before releasing a version I would like some more people to weigh in on this and make sure it makes sense. @brainstorm, what do you say? @grzn / @yalon , have any inputs?

PS the commit in question is ee18350754ad81eded1bcea18b94171c892c8f83

Also pushed 5934e94 to fix a typo

The shouldHandle method remains the fast way to discard records while avoiding heavy init.

I don't see "shouldHandle" documented anywhere. "git grep -i shouldHandle" doesn't match anything. Could you elaborate on this?

@jikamens it is not so publicly known and is a relatively old feature that was there before as well. It is defined here: https://github.com/mitsuhiko/logbook/blob/master/logbook/handlers.py#L198

it is not so publicly known and is a relatively old feature that was there before as well. It is defined here: https://github.com/mitsuhiko/logbook/blob/master/logbook/handlers.py#L198

Ah, so when you wrote shouldHandle, you meant to write should_handle?
Maybe this feature should be documented somewhere?

I don't think should_handle is going to work for us. Our Processor is changing the record in a way that is subsequently used by the filter to decide how to handle it. That means that should_handle will have the same problem as we're having now with Logbook 0.9.0 -- when it gets called, our Processor will not yet have been called, so the data we need to decide whether to handle the record won't be available.
It would therefore be helpful to us if this fix could be shipped in a PyPI release.

I must admit I'm confused. Are the two fixes working for you or aren't they?

I must admit I'm confused. Are the two fixes working for you or aren't they?

I had not tested them.
I was waiting for the feedback from others that you requested above ("Before releasing a version I would like some more people to weigh in on this and make sure it makes sense. @brainstorm, what do you say? @grzn / @yalon , have any inputs?") and for a release containing them, which I got the impression, from the comment I just quoted, would be forthcoming.
I just tested these changes now and it does indeed solve our problem.

Ok, lacking any other comments I will release a minor version containing this fix. Thanks!

Released 0.9.1 with the fix. Please let me know if you have any issues with it.


Fix PyInteger / float division

This was broken due to changes to type inference. It infers the type as a double, but "py_operations" for binops need to have a Python result. Therefore I've separated into a two step:

calculate Python result
coerce to known C type.

In principle Optimize.c could take this back to 1 step. But at the moment it doesn't.
Fixes #6183
Fixes #6004

Implementation seems OK (but I cannot tell for 100%). I added just one comment regarding tests.

In principle Optimize.c could take this back to 1 step.

Did you mean the transforms in Optimize.py instead of Optimize.c? There are certainly cases where we drop "to py - from py" cycles. It's just not always safe since there may be actual type coercions along that way. Doing these with a C cast might not always work or do the exact same thing.

Yes I do mean the transforms in Optimize.py.

PR changes also other operators (e.g. *). They are not tested here. Should not be added here?

Yes I think you're probably right. We've only had complaints about \ for some reason. But more tests might spot the next breakage before it happens.


[bug report] targetFound is not triggered if you call start when the phone already points at tracking

@hiukim If autostart is set to false, and you call start manually with the phone already pointing at the tracking, the targetFound event is fired the very first time. However, if you then call stop, and with the phone still pointing at the tracking, you call start again, the targetFound event is not fired. The same problem happens with pause/unpause. Currently targetFound can't be relied on to make the content visible/invisible because of this issue.

Temporary workaround: After pause or stop, make entity invisible. This will trigger targetFound after it's unpaused/started again.
[...document.querySelectorAll('[mindar-image-target]')].forEach(target => target.object3D.visible = false);

Maybe this helps? https://github.com/hiukim/mind-ar-js/pull/241


Improved the Twitter service

Just two simple improvements:

Show the branch name of the commit(s) in the tweet
Use a dash instead of a minus. This just looks better.

Example:

[steam-condenser-ruby@update-api] git.io/Drgvag Sebastian Staudt – 1 commits

Just realized that @ isn't a good prefix for the branch on Twitter. ;) I'll update the code with a more appropriate one. Stay tuned.

Is the branch necessary?  Won't it take up precious tweet characters?

For me, it is. Maybe it's only adequate for the digest mode?
But I'm open to suggestions.


Would you consider C90 support?

I need a C90 json library and this is so close.  Would you consider accepting the changes necessary to make this C90 compliant?
It seems that the biggest change would be related to the json_error macro, which would need to become a function instead.

Take a look at the "strict-ansi" branch. It's tidied up and restricted
to plain old C89/C90. Though note the API is a bit different (and
simpler).

Great!  I didn't notice that previously.  I'll try it out.


Correct section names on Windows GNU assembly

Currently this data is placed into ".rodata", which is an unknown section, therefore has read-write protections by default. After this change, it would be in .rdata, which is for read only data, and therefore has such protections set.

Submitter is correct. Typically .rdata is the section name on Windows for constants, and that is the case on the .asms, but I though that the GNU toolchain also recognized .rodata. It doesn't.

Thanks @namazso !


empty string is not a valid Metastore URI

and should therefore be rejected.
Same for default_index_root_uri

@fmassot @fulmicoton : while I'm in the quickwit-config area, I could close this one if it's still relevant, and add tests for different combinations of config entries.
Is there more detail to add to this issue ? How does it tie-in with #1137  ?

@ddelemeny sorry for the super late reply.
Unfortunately my brain is not super fresh on these subjects.
I think I created this issue after getting a paper cut while testing quickwit.
Please do as you see fit.


Add technology descriptions to FERC small generators table

It looks like all the issues in this Epic are closed. Can/should the Epic be closed?


should it be catch (e) { ...} ?

https://github.com/facebook/jest/blob/7e5e27a6aa5f07037c6b8b23a3e49afc30008937/packages/jest-worker/src/WorkerPool.ts#L24
It will cause a SyntaxError when I run it on node V8.11.3.

no, https://github.com/tc39/proposal-optional-catch-binding

Node 8 is EOL and we don't support it: https://github.com/facebook/jest/blob/7e5e27a6aa5f07037c6b8b23a3e49afc30008937/packages/jest-worker/package.json#L23-L25
Use Jest 25 if you need to use node 8

Node 8 is EOL and we don't support it:
https://github.com/facebook/jest/blob/7e5e27a6aa5f07037c6b8b23a3e49afc30008937/packages/jest-worker/package.json#L23-L25
Use Jest 25 if you need to use node 8

Got it, thank you.


Feature request: Faster refresh feed intervals (less than 10 minutes)

One of the features I really prefer in other readers/services is this option which feeds updates quicker than NNW since always (as I can remember).
It would be nice to see quicker options rather than "Every 10 minutes" at minimum.

Asking for new features became irrelevant due to a stupid decision to get rid of development for proper macOS versions

I don’t understand — what do you mean by “get rid of development for proper macOS versions”?

Never mind — you made your point in a separate bug.


chore: add logo for mog-weth pool

📑 Description

add logo for new MOG-WETH aerodrome factory vault pool

✅ Checks

[ ] I created a new folder with the token address, all in lowercase
[ ] I added the token's logo as a 32x32 PNG file, named logo-32.png
[ ] I added the token's logo as a 128x128 PNG file, named logo-128.png
[ ] I added the token's logo as a SVG file, named logo.svg
[ ] My SVG logo is a proper SVG and not some base64 stuff
[ ] My documentation/website clearly display the token address somewhere

oof, seems like this image is base64...reached out to mog team to try and get a proper SVG version

oof, seems like this image is base64...reached out to mog team to try and get a proper SVG version

if you get a proper one, can you submit it for their token as well in this PR

our designer came through and made an SVG, see #443

amazing, ty good ser @abmisx0


OSError: All residues have to be unique in polyply.

Hi Fabian,
Sorry to disturb you again !
I try to do a very simple thing :  add a chain of amino acid to a lysozyme (My json file who describes the combination ) .
But when I try to generate the coordinates, I have this issue :
OSError: Residue THR with resid 130 in moleculetype polymol is
different from residue THR with resid 40 in moleculetype
polymol. All residues have to be unique in polyply.

I can't find my mistake ! Please how can I fix this problem?
(And just in case, my itp)
Thank you !
Romuald

Apparently there as been a bug in the code that deals with making edges between atoms/beads in a molecule, which causes it to ignore constraints in making edges. In your case you are actually having some THR that only have constraints but no bonds. I think those come from martinize - which might have a bug there as well. In one part made with plolyply the THR have both bonds and constraints. Hence the edges are made correctly.
I will fix the bug on the polyply side but there is also something wrong with the martinize version you are using for making the base lysozyme itp in my opinion.

@Romumrn from the polyply side this should be fine now.

Thanks for your answer!
Indeed, with the new version of polyply and the new martinize2 version everything works!
Cheers


Improve project organization

Proposed improvements to project organization.
This organization change has 3 goals:

Make the project more open and accessible
Make the project more scalable
Do this in an incremental way, without unnecessary refactoring

The actual organization change is covered in 6a99b8dea935be9228a47c84bcbfb7491e6e5584. Cosmetic changes are discussed in #9177 and #9178

@shykes next steps? and DCO marker? ;-)

Overall LGTM :)

Same here, I don't see any remaining issues: LGTM :-1:

LGTM

LGTM

Ok, after much discussion on irc, I think we're ready to merge.

Added missing people
Added missing subsystems
Officialized de-facto core maintainers
Removed the vague notion of "lead"
Clarified the roles of Chief Architect, Chief Maintainer, Chief Operator

Even merged, this doesn't end the conversation. The goal is to formalize a structure which is already mostly in place informally. This is a necessary starting point for attacking more ambitious upgrades to the organization.
The big topic which remains to be addressed: how do we transform from a tool-centric organization to a platform-centric organization? Docker is no longer a single tool, but a collection of tools unified under a platform. We want this platform to be composable (each tool should be useful standalone, and replaceable by another), but also well integrated: the user experience should feel like more than the sum of the parts. To achieve this, we need to address integration at many levels. More on this topic soon!

@cyphar it's mostly formalizing what is already in place, but also incorporating de-facto rules that have appeared organically but weren't documented. For example, the concept of "core maintainers" which are the de facto owners of everything which isn't a clearly separated subsystem - that was already the case but not clearly documented. This file also deprecates the more fine-grained MAINTAINERS files - because in practice we were ignoring them often.
The other important change is that I am reducing the role of the bdfl, and creating 3 new leadership roles: Chief Architect, Chief Maintainer, Chief Operator. That should help reduce the dependency on bdfl for running the project, while keeping it as a guarantee for the long-term direction and governance of the project.


Course in between 1-360

APRS spec sais, course should be in between 1 to 360.
I suppose gps.course.deg() returns 0 for north -> it should be assured, that 0 is changed to 360.
Since the course and speed fields are optional, a special case could be interesting to be dealt with:
if speed is 0 and course is zero, we are not moving. -> We can omit "000/000" in the aprs comment.
I would go this way:
                String course_and_speed = String("");
                int speed_int = (int)gps.speed.knots();
                if (speed_int > 0 && speed_int <= 999) {
                        String speed = padding(speed_int, 3);
                        int course_int = gps.course.deg();
                        /* course in between 1..360 due to aprs spec */
                        if (course_int == 0)
                                course_int = 360;
                        String course = padding(course_int, 3);
                        course_and_speed.concat(course + "/" + speed);
                } /* else: if speed is 0, course does not make sense, and is changing, even if we stand still */

Btw, I assure for the correct length (exactly, not mor. than 3 characters).
Speeds above 999 knods should be considered as bug, I think. But I'm not sure ;)
An assurance like above I added to the altitude field: better no altitude than non-sense ;)
                int speed_int = (int)gps.speed.knots();
                if (speed_int > 0 && speed_int <= 999) {
                        String speed = padding(speed_int, 3);

fixed with https://github.com/lora-aprs/LoRa_APRS_Tracker/commit/727c90eeae65a3d0668974cb6f507c4e9ecf07f3


kent/src/hg/utils/hubCheck/makefile

I think the included mk file in the makefile is wrong, because there is no "install" target in the common.mk. The correct mk file to be included is userApp.mk.
The modified top part of the makefile is:
kentSrc = ../../..
# include ../../../inc/common.mk # comment-out

L += ${HALLIBS} ${MYSQLLIBS} -lm
MYLIBDIR = ../../../lib/${MACHTYPE}
MYLIBS =  ${MYLIBDIR}/jkhgap.a ${MYLIBDIR}/jkweb.a

A = hubCheck
O = hubCheck.o

include ${kentSrc}/inc/userApp.mk # added

I'm sorry, but this doesn't look like GitList code?

Hi, maybe I don't follow the community rule. I think I should not open an issue.
I just wanted to point out that the if I modified the makefile as I did, the "make" process went on successfully.
Now I am going to close the issue:)


Import GPG signing key for @cjihrig

This PR adds @cjihrig GPG key to the list of keys being imported before verifying io.js binary signatures.
Related: nodejs/io.js#2217
Signed-off-by: Hans Kristian Flaatten<EMAIL_ADDRESS>

@Starefossen @cjihrig The upgrade from 2.4 to 2.5 broke this PR.
Let me redo this.

Fixed here: https://github.com/nodejs/docker-iojs/commit/0a5fc1175cdb1704b172c72f3d34d53f8310da31

Thanks @hmalphettes 👍🏼

@Starefossen enjoy the summit in SF and salutations to everyone!

Thanks, I'm really looking forward to it :smile:


Add integrated statistics to FreeCEN footer

The FreeCEN footer only contains vld information

Using the new site statistics collection display the number of searches, the number of records and additions


build multi-arch binaries and multi-arch container

Does:
Build multi-arch binaries and containers.
Does not:
Does not automate the publishing of all binaries or pushing container.
How To:
make build-all-arch
make container
make container-push

or
make all
make container-push

Result:
# ls -lah ./bin
45M client-darwin-amd64
44M client-darwin-arm64
58M client-linux-amd64
44M client-linux-arm64
44M client-linux-ppc64le
44M client-linux-riscv64
47M client-linux-s390x
45M client-windows-amd64

# buildah images
REPOSITORY                             TAG      IMAGE ID       CREATED          SIZE
ghcr.io/uor-framework/client-ppc64le   v0.2.0   df172325cdb4   25 minutes ago   93.1 MB
ghcr.io/uor-framework/client-s390x     v0.2.0   2139b9e75493   27 minutes ago   87.4 MB
ghcr.io/uor-framework/client-amd64     v0.2.0   98bf336eeef3   28 minutes ago   101 MB
ghcr.io/uor-framework/client-arm64     v0.2.0   44f36559c04e   29 minutes ago   87.1 MB
ghcr.io/uor-framework/client           v0.2.0   b232ac903061   30 minutes ago   110 B

# podman run --rm -it --pull never 44f36559c04e
UOR Client:
 Version:       v0.2.0+dev
 Go Version:    go1.18.3
 Git Commit:    3ba03d3
 Build Date:    2022-07-09T23:50:21Z
 Platform:      linux/arm64

Other:
closes #38

Any reason to not automate pushing the containers to ghcr.io?

@vsoch It's on the week. See this PR

Ah perfecto! I'll follow instructions from there, or if the PR will be reviewed/merged soon I can test the resulting containers. Thank you @sabre1041 !

@vsoch PR will be reviewed today/tomorrow. will let you know when assets are ready for a review!


Fix wizard in python 3

Coverage remained the same at 89.574% when pulling 6bb1562125f51d2afe5c9b435167449ff76c59a9 on yakky:feature/fix_wizard_py3 into c6c24c6bf3beb8bb4a06ca90940c05cc156d1ac9 on divio:develop.

Coverage remained the same at 89.574% when pulling 6bb1562125f51d2afe5c9b435167449ff76c59a9 on yakky:feature/fix_wizard_py3 into c6c24c6bf3beb8bb4a06ca90940c05cc156d1ac9 on divio:develop.

Coverage remained the same at 89.574% when pulling 6bb1562125f51d2afe5c9b435167449ff76c59a9 on yakky:feature/fix_wizard_py3 into c6c24c6bf3beb8bb4a06ca90940c05cc156d1ac9 on divio:develop.


Added float2x2 cast to float4 (and vise versa)

fixes: #4415
Note: not adding matrix<1,N> and matrix<N,1> translations since this will bloat stdlib with many expensive to process extension operations. matrix<N,1> is already planned to be treated as vectors (which should solve this problem by proxy).

I really don't think we should allow implicit cast from float2x2 to float4 because they are fundamentally different types. This is not the same as 1-vector and scalar.

Why do we need this?

Why do we need this?

https://github.com/shader-slang/slang/issues/4415

I don't think we should provide everything HLSL has. And I think this is one of the bad things we shouldn't be adding to our language.

We can add an explicit conversion between float2x2 and float4, but we shouldn't make them implicit.


not working

C:\Users\Peter\Desktop\My Office Add-in>npm run configure-sso
><EMAIL_ADDRESS>configure-sso C:\Users\Peter\Desktop\My Office Add-in
> office-addin-sso configure manifest.xml
(node:14720) UnhandledPromiseRejectionWarning: SyntaxError: Unexpected token  in JSON at position 980
at JSON.parse (<anonymous>)
at C:\Users\Peter\Desktop\My Office Add-in\node_modules\office-addin-sso\lib\configure.js:188:40
at Generator.next (<anonymous>)
at C:\Users\Peter\Desktop\My Office Add-in\node_modules\office-addin-sso\lib\configure.js:6:71
at new Promise (<anonymous>)
at __awaiter (C:\Users\Peter\Desktop\My Office Add-in\node_modules\office-addin-sso\lib\configure.js:2:12)
at childProcess.exec (C:\Users\Peter\Desktop\My Office Add-in\node_modules\office-addin-sso\lib\configure.js:182:95)
at ChildProcess.exithandler (child_process.js:285:7)
at ChildProcess.emit (events.js:198:13)
at maybeClose (internal/child_process.js:982:16)
(node:14720) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:14720) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

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

ID: 3020ff5d-ee3b-976b-f1ad-0ed378fdb47b
Version Independent ID: 5de0010a-6435-154b-a7fd-fd2ea05ebc0b
Content: Use the Yeoman generator to create an Office Add-in that uses SSO (preview) - Office Add-ins
Content Source: docs/quickstarts/sso-quickstart.md
Product: non-product-specific
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

@quantrpeter Sorry that you're hitting this issue and thanks for letting us know.
@TCourtneyOwen Can you take a look?
Thanks.

@TCourtneyOwen  no space in my tenant name, thx

@quantrpeter Did you make any modifications to the base template after creating it and before running "configure-sso"?  Also, could you please check the version of the office-addin-sso and see if it's running the latest version of the package - 1.0.12.  If it's not, please update the version of the package to the latest

One more thing, can you please provide all the output of the configure-sso process so I can determine at what stage it's failing for you? For example, here's my output that I just generated:
Opening browser for authentication to Azure. Enter valid Azure credentials
Login was successful!
Registering new application in Azure
Application was successfully registered with Azure
Setting identifierUri
Setting signin audience
Granting admin consent
Setting application secret
App secret is Bqs8H2I<EMAIL_ADDRESS>Adding application secret for Office-Addin-NodeJS-SSO to Windows Credential Store
Updating source files with application ID and port
Logging out of Azure now
Application with id 1d90426e-5543-4b04-b4a2-22c1bdfc1b8c successfully registered in Azure.  Go to https://ms.portal.azure.com/#home and search for 'App Registrations' to see your application


hideStack option

Coverage remained the same at 100.0% when pulling d8901b8a87c06fb36e72ef21373f29de57ad5a61 on tanhauhau:master into df765a50c0c1de4fdbe4ce9cef2d910df4d8ba3e on expressjs:master.


The first character is always the same in the same session

It is caused by #random_letter method that is memoized

Maybe this helps. I reimplemented from scratch with help from ChatGPT https://gist.github.com/narze/985712e46d33be3613e395388b549c2d

Thanks for the report, this is fixed on #2


Allow custom number of runs

Currently the operations benchmarked will be run 500 (5*100) which might be too long in some cases. It would be nice to have a localOption-like thing to limit the number of runs of a particular branch of the hyperion tree.

Addressed in #19.


Normalize loudness with ytdl-core relative loudness

https://github.com/fent/node-ytdl-core/blob/e14c9da91fbb383e8cc84fc6dab2febd13ee277d/typings/index.d.ts#L250

resolved with https://github.com/vphoebe/beatnik/commit/a756875d628170140c8529e82d7010b0995de4f2


Responsive images (derivatives) generation on image upload or via option in the admin backend

Currently, in the latest version of GRAV the image derivatives are generated on the first page request, which can take a lot of processing time. Are you working on a feature to generate the images on upload (f.e. according to admin settings) or through a button "Generate responsive images" in the backend? This would be a very important feature.
I tried some plugins which should solve this problem, but none of them worked for me.

@HoloBack if you hit preview in admin, would that technically do first page request? essentially generating the images?

@ricardo118 Thanks for the fast reply! Yes it does the first page request, but it doesn't solve the problem - you just click through all the pages manually in preview mode. I think it would save a lot of time, if images would be generated on upload and additionally with a button "Generate images", where a background process is started which iterates through every page and generates the images. That's how we all know it from Wordpress and other CMS.

I would love this feature too. Generating derivatives for each image on upload would be less intensive and hopefully avoid server timeout in the case of a lot of images with derivatives on one page.

How does it behave in this first page request scenario, if there are concurrent first page requests? Would it trigger a race condition?
That would be a much more important reason to have the generation run in the background on upload rather than on page display


Remove targetSdk declaration

Setting the targetSdkVersion is the domain of apps, not libraries. Given the project's minSdkVersion is 21, apps that target 21 should be able use it, so adding this is probably too restrictive as the manifest merging process will use the highest one, which would be 23, if we left this in, which would override what the app's setting stipuates.

I think I understand better now after reading the comment in the other (separate demo-app) PR. We should specify it explicitly in the demo-app build and not here.

If this is possible without causing any hidden/strange behaviors then I'm all for it, IIRC the stuff that's added to the conventions was the bare minimum requirements for libs to compile, so for some reason I was under the impression that the compiler would complain if this value was not set. Although, if that's not the case and it still works without it (and it doesn't add any value by default that's too high either) then by all means I'd like to remove it. To me, the least amount of versions to keep track of, the better! 😁

Btw I just checked and it seems like we didn't even use this for libs so most likely it wouldn't cause any issues to remove it. Maybe it was there just as a convenience for tests that involve creating a full app, such as the okhttp one for example. So based on that, if we removed this from the app conventions, we'd have to manually add this value for all of those kinds of tests, @bidetofevil?

Ugh, I messed up my branch. If I can't fix the PR I'll just blow it up and start it again.  I set up my repo once and forget all my settings. I'm using Graphite to simplify my workflow but it's not great at coming in the middle of PRs. Anyway, rebased. Feel free to merge when ready.

Btw I just checked and it seems like we didn't even use this for libs so most likely it wouldn't cause any issues to remove it. Maybe it was there just as a convenience for tests that involve creating a full app, such as the okhttp one for example. So based on that, if we removed this from the app conventions, we'd have to manually add this value for all of those kinds of tests, @bidetofevil?

By default, the minSdkVersion set on the library will be used when a test requires an app to be built - or whatever the highest min or target set on the other libraries included in the test will be. Of course if the test is testing specific logic in a particular Android version, you can always override that on a test by test level, or set it on a global basis.
If we want a higher targetSdk to be used for tests (which we probably should?), I can find the global setting and do it there.


Migrate "Clans, Allies & Followers" from old wiki

Breakdown of #5
Describe concepts of Clans, Allies & Followers, possibly reusing content from old wiki.

@kazk Do you think it's ok w.r.t. placement and organization? I am not sure if it's worth having separate HOWTOs on joining or leaving a clan, but maybe you think it is?
I wanted to add screenshots of allies/followers/following boards, I am just not sure if it's ok to put screenshots from my board (potentially anonymized, blurred, etc), or should I somehow create some "neutral" boards with some generic/fake data?

@kazk Do you think it's ok w.r.t. placement and organization? I am not sure if it's worth having separate HOWTOs on joining or leaving a clan, but maybe you think it is?
I wanted to add screenshots of allies/followers/following boards, I am just not sure if it's ok to put screenshots from my board (potentially anonymized, blurred, etc), or should I somehow create some "neutral" boards with some generic/fake data?

I think it's fine.

I am not sure if it's worth having separate HOWTOs on joining or leaving a clan, but maybe you think it is?

What you already have looks enough to me.
 There's no such thing. It's just a free text field that makes users with the same text follow each other. Nothing more.
This is why you get more and more allies as you change your clan field. Previous clan members are not removed automatically because there's no difference from users you had followed manually.
I've been wanting to deprecate it and support proper team/org, but it's not a high priority.

I wanted to add screenshots of allies/followers/following boards, I am just not sure if it's ok to put screenshots from my board (potentially anonymized, blurred, etc),

I think using yours is fine without anonymizing. It's public information.

I think it's fine.

I am not sure if it's worth having separate HOWTOs on joining or leaving a clan, but maybe you think it is?

What you already have looks enough to me.
 There's no such thing. It's just a free text field that makes users with the same text follow each other. Nothing more.
This is why you get more and more allies as you change your clan field. Previous clan members are not removed automatically because there's no difference from users you had followed manually.
I've been wanting to deprecate it and support proper team/org, but it's not a high priority.

I wanted to add screenshots of allies/followers/following boards, I am just not sure if it's ok to put screenshots from my board (potentially anonymized, blurred, etc),

I think using yours is fine without anonymizing. It's public information.

Yeah I decided to migrate this part because I considered it somewhat of a low hanging fruit, and questions about clans pop up from time to time.
I will try to get some screenshots then and see how they turn out.

Yeah I decided to migrate this part because I considered it somewhat of a low hanging fruit, and questions about clans pop up from time to time.
I will try to get some screenshots then and see how they turn out.


"Flint" walletStringName should be "flint"

Not sure how recent the change was but the correct walletStringName for flint should all be lowercase. This is for the most up-to-date Flint Chrome extension

Wallet.stringNames elements (and respectively Wallet.Names.Wallet) try to emulate the window.cardano.walletExtension.name value
in this case Wallet.Names.Flint value is currently "Flint", whereas the correct value should now be "Flint Wallet" as we can see from your screen.
If in your app you only use Wallet.Names properties or Wallet.stringNames elements you should have no problem.
Anyway thank you for signaling the issue, ofthen these small changes are hard to track, will change the value ASAP

To clarify, the correct value should now be 'flint' and not 'flint wallet'. This bug prevents users from using flint API at all using Wallet.stringNames or Wallet.stringNames
Wallet.stringNames doesn't work because the cardano object is case sensitive and attempts to look for window.cardano.Flint instead of window.cardano.flint. To further explain,Wallet.get(Wallet.Names.Flint) will always be undefined since it passes Wallet.get('Flint') instead of Wallet.get('flint')

I see where your concern is
actually Wallet.Names and Wallet.stringNames elements are mapped to the respective browser API as you can se, as an example from the enable method implementation, flint API case
so calling Wallet.enable('Flint') will internally call window.cardano.flint.enable
I would strongly suggest using the Wallet.Names object to access various APIs, since, as in this case, changes made in the API itself might not be immediately implemented, therefore calls like Wallet.enable( window.cardano.flint.name ) will result in an error because the name value has been modified between APIs version

I'm clòosing the issue, I hope any doubt has been clarified now!


Invalid settings for silence

When I use alertmanager, I find that the set Silence is invalid. Specifically: when I set matchers, such as team = mysql, and set the first one, I will still receive the related alert, but when I set the second one, it will immediately have an effect, and no alert will be sent. And it shouldn't be a data synchronization issue. If it's a problem of data synchronization, I set the first one. After a while, it should work. But it's not true. I have to set the second matcher exactly the same for it to work.

Here is my version information

This might be exactly the bug fixed in 0.22.2. Could you try that release and see if your problem persists?


Feature request:fastcgi timeout options

In nginx, there are various fastcgi  timeout options, e.g.:
http://nginx.org/en/docs/http/ngx_http_fastcgi_module.html#fastcgi_connect_timeout
http://nginx.org/en/docs/http/ngx_http_fastcgi_module.html#fastcgi_read_timeout
It would be nice to also have these options (or at least know the current values) in caddy.

I'll give this a shot.

Hello, just wanted to touch base on this request. I'm getting pretty close to a fix and hope to soon have a PR to review!

I've noticed an issue with the read_timeout directive. I'm working on the changes. Also, as requested, I'll set default values for both read_timeout and connect_timeout.

Work continues. I'll be investigating a send_timeout property next.

Hello, @Echsecutor!
We've added the following timeout properties to the fastcgi directive:

connect_timeout
read_timeout
send_timeout

Each of these have a default value of 60 seconds.
Are these updates sufficient? If so then I'll close this issue.

Closing.


Create PR for FPS monitor and Toggle Switch

Created PRs:
https://github.com/gskinnerTeam/flutter-stats-fl/pull/5
https://github.com/boringdeveloper/FlutterSwitch/pull/9
They are now merged.


Machine mac-4 lost external connection from external iPhone device.

Machine mac-4 lost external connection from external iPhone device.

The external connection has been restored by rebooting machine mac-4.


the vendor profile does not appear

I selected vendor for product.But As in the link, the vendor profile does not appear in product page.Also dont have route named "/vendors/vendor-name"
https://marketplace-demo.spreecommerce.org/vendors/long-pants-vendor

I am also interested in that functionality. Is it currently developed or not compatible with Spree 4?


Multi-frustum artifact for Polygons on the surface

Polygons on the surface are tessellated differently than the ellipsoid. To avoid z-fighting artifacts, we write depth for opaque polygons, but do not for the ellipsoid and instead use the depth plane. Clearing depth before rendering the next frustum causes artifacts at the boundaries:

Everything rendered is correct after the furthest frustum is rendered. When the next frustum is rendered and if the far plane of the frustum intersects the polygon and ellipsoid where the ellipsoid surface is above the polygon, the ellipsoid surface will be rendered to the fragment that has already been writen to by the polygon in the previous frustum.
One solution is to draw each frustum in a separate FBO with color and depth attachments, then resolve the depth in a final pass. We would need depth texture or MRTs support.

@bagnell do you think polygons on terrain will fix this?

@bagnell is this still an issue with the ground primitive?  If not, close this.


A Grasp of Linear Regression

Proposed Title: A Grasp of Linear Regression
Introduction paragraph(s):
Linear regression is one of the most fundamental algorithms in Machine Learning. It is a model used to show or predict the relationship between two variables, One dependent, also called the Target and the other being independent, which is called Feature(s) or Predictor(s). The two types of linear regression are Simple Linear Regression and Multiple Linear Regression.
Regression is an important tool for analyzing and modeling data. It operates by fitting a curve or a line to the data in order to differentiate with the minimum distance of the actual data points from the plotted curve or line.
Some of the usefulness of regression are:

It depicts a clear relationship between the target and the feature(s).
It shows the rate of influence of the features on the target.
It measures effect regardless of the scale of the variables.

Key Takeaways:
By the end of this tutorial, the reader will be able to:

understand fully the concept of linear regression.
create a simple dataset using pandas dataframe.
build a linear regression model from scratch.
calculate model coefficients and the application.

References:

What A Simple Linear Regression Model Is and How It Works
Multiple Linear Regression (MLR)

to help avoid a backlog queue we are only accepting one article topic/pr approved at a time 👍 @wizardcalidad


Add a Gitter chat badge to README.md

NeilRen/GeCMS now has a Chat Room on Gitter
@NeilRen has just created a chat room. You can visit it here: https://gitter.im/NeilRen/GeCMS.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Manual addition


quickly-changing-between-layouts is wrong

https://docs.microsoft.com/en-us/windows/powertoys/fancyzones#quickly-changing-between-layouts
This paragraph is wrong, at least with my version of the tool ; there is no control to select a hotkey :

[Enter feedback here]

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

ID: 6a11237d-cd08-35d0-ae4f-4b7d57df93f1
Version Independent ID: d7eaa04f-a054-4ea7-2eb8-e7a5979b0a59
Content: PowerToys FancyZones utility for Windows 10
Content Source: hub/powertoys/fancyzones.md
Product: dev-environment
Technology: powertoys
GitHub Login: @mattwojo
Microsoft Alias: mattwoj

My bad, I had a very old version


Cleanup KeyPackage.

This PR cleans-up the extensions in key packages. Basically, it changes the extensions: Vec<Extension> to extensions: Vec<KeyPackageExtension> and cleans-up afterwards. If we don't want to use a dedicated KeyPackageExtension type, this change could be easily reverted. It gave me a bit of guidance, though.

I see much more added here now. Let's not do that and only clean up here first. This is already way too big (as the labeller tells you).

Apologies for the force-push. It's best to start clean since I isolated all extension-related changes and removed anything that can be handled separately. The PR is still large, but primarily due to required test changes. Not sure what the best review strategy is, but I recommend spending less time on repeated KeyPackageBundle::new changes as it's just a refactoring. Reviewing each commit individually may help. Note, however, that in this case Lifetime changes will need to be inspected twice due to consecutive KeyPackageBundle::new changes.

We discussed this offline and I'll close this PR as it is too difficult to review and will try to make smaller PRs.


0223 - Rectangle Area

Description of the Problem
Find the total area covered by two rectilinear rectangles in a 2D plane.
Each rectangle is defined by its bottom left corner and top right corner as shown in the figure.
Image here
Example:
Input: A = -3, B = 0, C = 3, D = 4, E = 0, F = -1, G = 9, H = 2
Output: 45
Note:
Assume that the total area is never beyond the maximum possible value of int.
Link To The LeetCode Problem
LeetCode
@vJechsmayr Please assign me this.

@vJechsmayr Please assign me this.

I would like to contribute my solution for this problem.
Please assign me if it has not been contributed before.
Hope to here you from soon :)

@poojan-dalal Waiting for your PR 😸


Upgrade to Python 3

Boring first issue, but I'm happy to contribute to this cause. Python 2 is way deprecated now (there's even an app for the countdown), so might as well upgrade before EOL.

Try to upgrade to python 3 by changing , but encountered an issue in uwsgi
web_1  | 2019-03-26 10:32:13,811 INFO spawned: 'uwsgi' with pid 637 web_1  | [uWSGI] getting INI configuration from /app/uwsgi.ini web_1  | [uWSGI] getting INI configuration from /etc/uwsgi/uwsgi.ini web_1  | 2019-03-26 10:32:14,859 INFO success: uwsgi entered RUNNING state, process has stayed up for > than 1 seconds (startsecs) web_1  | 2019-03-26 10:32:16,822 INFO exited: uwsgi (exit status 22; not expected)


chore: update test snapshots

Package com.fasterxml.jackson.core:jackson-core has a new version 2.18.0-rc1 https://central.sonatype.com/artifact/com.fasterxml.jackson.core/jackson-core/2.18.0-rc1
This PR updates the snapshot of update subcommand to this latest version.

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

Project coverage is 66.53%. Comparing base (d7a0cf9) to head (6adf5b9).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1218      +/-   ##
==========================================
+ Coverage   65.97%   66.53%   +0.55%
==========================================
  Files         168      168
  Lines       14152    10970    -3182
==========================================
- Hits         9337     7299    -2038
+ Misses       4298     3152    -1146
- Partials      517      519       +2

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


Support code formatting for OCaml

I believe there is support of code formatting for reason using refmt. I would like to have the support of ocp-indent for the OCaml code. With the possibility to format all the document, only my selection or only one line.
I wonder if this should be done in this extension or if it's better to have a new extension for ocp-indent.

There is at least preliminary support for ocp-indent implemented in the server. It should show up when you have an OCaml document open if you search the command panel for "format document". I haven't tested it recently but I think it should still be working. You can see how it is integrated here.

Indeed it works for the whole document. Thank you.
The reason I would like to be able to launch it only on a line or a block or it use it to manage the indentation locally is that I can't always format a whole file. For example when I contribute to an open source project that doesn't follow a specific indentation convention, I want my new code to have a correct indentation but I don't want to change the rest of the file.

As I recall, the server does support indenting ranges too, but it looks like I may not have exposed that to the client since it was not released yet. You can see it here.
Anyway, it should be possible to support different approaches to indenting. If you just want to indent a specific line, it might need to be set up as a hotkey though because I don't think the language server conventions provide a built-in interface for that style.
Eventually there should be an option to enable automatic/electric indentation as you type as well. There's a capability in the server for that but I am not using it yet.

As Ocamlformat is adapted in Infer, I think it is the time to put Ocamlformat in the list of supporting formatters. Any ideas?

Both ocp-indent and refmt are now supported. There is an issue in the server repo for ocamlformat.


Any plans for testing this with Pytorch/Tensorflow/mxnet?

This project would be a life saver (if it works) for so many researchers and enthusiasts who need GPU capabilities to train even basic models.  Intel GPUs are cheap and abundant and having this project make these ML tools work would be immensely helpful.
Not expecting performance to be on par, I understand NVIDIA has better hardware, but even Intel GPUs can open up so many possibilities.
Also wondering what help might be needed to make this a reality.

To give some perspective, there are tons of Intel GPUs on Windows laptops and Mac.  If ZLUDA can enable their owners to train models using the GPU it would be a huge productivity boost for a lot of people.

Excellent question.
Firstly, if you simply want those frameworks to run on an Intel GPUs, then ZLUDA might not be the best way. Deep Learning is quite high on the Intel priority list. Every major framework (certainly PyTorch, Tensorflow, MXNet, PaddlePaddle) has a dedicated team working on it to bring the best performance on CPU and GPU. I'm not tracking this all that closely, but AFAIK with the release of oneAPI 1.0 in December all the frameworks should be natively supported with no fiddling.
More info here: https://techdecoded.intel.io/resources/intel-oneapi-beta-releases/ and in this video: https://www.youtube.com/watch?v=9y3xpi-yPyA
I recommend that you use the official support made by an army of full-time engineers and funded by a massive corporation. Thought if you (or anybody else) want to hack it into Intel GPUs through ZLUDA I'm keen to support it 😁
The question really boils down to running cuDNN. That's what every frameworks uses to have high performance on NVIDIA.
One way to support it to keep adding missing functionality in both major translation libraries: PTX->SPIR-V kernel compiler and in the CUDA -> Level 0 host code translation layer until you can support enough of cuDNN to run those frameworks.
This, somewhat counter-intuitively, is the wrong approach to the problem. As you might be less or more aware NV GPU architecture and Intel GPU architecture are slightly different. This is not a big deal for majority of kernels. This is a big deal for first-party kernels. First-party kernels and especially kernels in such a major library are optimized to an abnormal degree. GPU Code is written with particular exact hardware capabilities in mind: register space, instruction set, bandwidth, compute. It's simply not possible to run it on different hardware without incurring a fairly major performance hit. Furthermore, to squeeze a little bit more performance, I fully expect NVIDIA to use internal CUDA driver API. Debugging and re-implementing this is a lot of work. Additionally, we rely on the applications to ship architecture-independent PTX code. Pretty much all third-party libraries and applications do this in order to support future hardware out of the box. First-party libraries shipped with the drivers (like cuDNN) have very little reason to do this.
But there's a different, more performant solution! The approach is to create a ZLUDADNN. Basically, a library with the same API as cuDNN that is injected alongside ZLUDA and which contains kernels optimized for Intel GPUs. You would wrap oneDNN and forward all the calls to it, so e.g. cudnnSetConvolution2dDescriptor(...) uses mkldnn::convolution_forward::primitive_desc(...). This would be much more performant solution, but there are several issues one should be aware of:

You would need to unpack CUDA-ZLUDA stream to Level 0 command queue etc. This can be fairly easily done by having ZLUDA expose a special function or functions to peek behind the covers. E.g. something like zluda_cuda_stream_to_ze_queue(...). I would be highly interested in merging and shipping whatever is useful for this purpose in ZLUDA
oneDNN is a bit behind the curve and is OpenCL-only with a DPC++ branch that is AFAIK just for show. One would need to either rewrite oneDNN host code to Level 0, convince the oneDNN team to do so or awkwardly interop between Level 0 and OpenCL (at performance loss).
cuDNN API and oneDNN API are different, some hacks might be involved and worst case scenario one might be forced to fork oneDNN
cuDNN API is really massive, it is a lot of work

I hope that makes sense, feel free to ask if it does not.

I think I was being naive to expect this.  Even oneDNN works only on Xeon.  I was thinking of taking advantage of all that cheap Intel hardware sitting on people's laptops doing not much.  Might not be comparable to NVIDIA, but it would definitely beat CPUs.   When it comes to performance in DNN, all you need is to beat the CPU, not compare to NVIDIA.
I am quote miffed that game developers are able to squeeze performance out of most consumer GPUs, but DNN can't.   Also looking at DNN frameworks, they have very little interest in running on consumer GPUs, so I guess, I give up.
Feel free to close.

Hi,
I am a deep learning researcher heavily relying on pytorch / tensorflow. I have limited knowledge in hardware, especially when it's low-level.
The most annoying thing using NVIDIA GPUs for me is the limited amount of RAM,  unless you have an RTX 3090. If I can get to run your project on an Intel integrated GPU (e.g. UHD630), does this mean that I can train / test big neural network models on the CPU RAM, since it's shared with the integrated GPU RAM?
Thanks!
Tae

I think this is still very relevant. Perhaps even more than before.
It seems this issue got closed by an automated script. Could we reopen @vosen ?
Thanks!


V9.0

Hi @scottdurow,
As I promised, I push the webresource dependency support changes. Hopefully I am doing it right, I am a GitHub newbie.
Here is an example of spkl.json:
{
"webresources": [
{
"profile": "default,debug",
"root": "webresources/new_/",
"solution": "SpklTest",
"files": [
{
"uniquename": "new_/js/account.js",
"displayname": "new_/js/account.js",
"file": "js\account.js",
"description": "desc",
"dependencies": {
"libraries": [
{
"uniquename": "new_/js/dg.xrmquery.web.min.js",
"file": "js\dg.xrmquery.web.min.js",
"description": "desc",
"displayname": "new_/js/dg.xrmquery.web.min.js"
}
],
"attributes": [
{
"attributename": "accountid",
"entityname": "account"
}
]
}
}
]
}
]
}
I think the json itself it is self explanatory, but if you will have questions, do not hesitate to contact me, please.
Basically I made changes to 'deploy' and to 'download' webresources functionality.
Regarding deploy, I only add dependency libraries that are store in CRM, I do not compare what is included in spkl.json.
Regarding download, I do not updatae existing files in spkl.json, if there is for example new dependency library added manually in CRM for a particular webresource. This could be also added, it is your decision.
Please, have a look at the code and you can also share your comments, concerns, whatever with me. I also included few comments.
Thanks.

Hello all,
Just want to ask what happend to this branch?  Is it still in dev?
Thx


[BUG] virtio-container-disk: "Name" cannot be more than 20 characters

Describe the bug
Saving a VM on 1.1.2-rc1 I see virtio-container-disk: "Name" cannot be more than 20 characters
To Reproduce
Edit a VM that has a container disk
Expected behavior
The container disk name size should be restricted before or not limited.

Environment

Harvester ISO version: 1.1.2-rc1
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630):

Additional context
Add any other context about the problem here.

According to the k8s specification, I now set the maximum length of the name to 63 characters.

@WuJun2016 ,
Validated that on Harvester Version: master-f93d67e1-head - creating/editing of the name length to be more than 20 characters does work:

I'll go ahead and close this out :smile:


KNL-1547 Connection leak in ClusterEventTracking

Fixed a bug to stop the ClusterEventTracking to leak connections.

I opened #4796


Add string to the Literals table and provide clarity on use of single or double quotes

See feedback in https://github.com/OAI/sig-workflows/issues/49#issuecomment-1834052133

@darrelmiller in my editing haste, I made this change directly on main.
Here's what the table contains now:

Type
Literal value

boolean
true or false

null
null

number
Any number format supported in Data Types

string
Strings MUST use single quotes (') around the string. To use a literal single quote, escape the literal single quote using an additional single quote ('').

Make sense, or would you suggest an alternative description for string?

Reviewed during Jan 17th working group call. Group are satisfied with the string addition to the literals table


UPDATE pip install  only:  latest: v03.13 : Not Found configs/memgpt_hosted.json / AssertionError: Must provide llm_config field when creating an Agent from a Preset -

Describe the bug
AssertionError: Must provide llm_config field when creating an Agent from a Preset
looking for -->
Config file not found at /usr/local/lib/python3.11/site-packages/memgpt/cli/../../configs/memgpt_hosted.json
Please describe your setup

[ ] How did you install memgpt?

pip install pymemgpt

[ ] Describe your setup

Linux
How are you running memgpt? Terminal

MemGPT Config
error occurs befor write of config

If you're not using OpenAI, please provide additional information on your local LLM setup:
Local LLM details
memgpt free endpoint
memgpt run

also tested memgpt quickstart --backend memgpt
SCRIPT /usr/local/lib/python3.11/site-packages/memgpt/cli
FILE PATH /usr/local/lib/python3.11/site-packages/memgpt/cli/../../configs/memgpt_hosted.json
Config file not found at /usr/local/lib/python3.11/site-packages/memgpt/cli/../../configs/memgpt_

When do you get this error? When you run memgpt run?
Can you post whats in your ~/.memgpt/config file?

I've got the same error. I was inspecting the memgpt/cli/cli.py and it looks for the config file that is not found.
https://raw.githubusercontent.com/cpacker/MemGPT/main/memgpt/configs/memgpt_hosted.json

yes: memgpt run
only when 'pip install' (03.13) is used
-> error occurs before the config is written ..


Implement the additional manifests for customized yamls

Fixes #330
Proposed Changes

Add the model and implement spec.additionalManifests.

Release Note

/test pull-knative-operator-upgrade-tests

/test pull-knative-operator-upgrade-tests

/test pull-knative-operator-upgrade-tests

/retest

/test pull-knative-operator-upgrade-tests

/retest


refactor: rps logic :rocket:

Closes #159
📑 Description

✅ Checks

[ ] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[ ] All the tests have passed

ℹ Additional Information

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
9

Relevant Lines:
9

💛  - Coveralls


docs: publishing schedule and demo script

🤖 Generated by Copilot at 4dc5380
Summary
📄🎬📣

This pull request adds two new files to the docs/docs/devrel folder: dxos-demo-script.md and publishing-plan.md. These files document and outline the DXOS team's plans and best practices for demonstrating and promoting their platform to potential users and developers. These changes aim to support the DXOS team's efforts to grow their community and showcase their platform.

DXOS demos planned
publishing-plan.md guides
Autumn content blooms

Walkthrough

Add two new files to document and share the DXOS demo script and publishing plan (link, link)

@richburdon @zhenyasav is this the right place to put these docs?


scraping metrics is slow

For queries to github which take a since field, start using that field and using pickles to save old stuff.
For example, change repo->get_commits() to:
cortx_community.get_commits(ps,repo):
url = repo.get_url()
(latest,items)=ps.get_items(url)
new_items = repo.get_commits(since=latest)
for item in new_items:
latest = max(latest,item.timestamp)
ps.add_items(url,latest,new_items|items)
ps.add_items(key,ts,items):
items_structure[key]=(latest,items)
persist(items_structure)
ps.get_items(key):
return items_structure[key]
And create a new items pickle and hope it doesn't grow too big.  If it is too big, then will need to split into multiple pickles.  One for commits, one for comments, one for pull requests, etc.

@johnbent, to whom this issues needs to be assigned or direct to as I am unclear?

@johnbent, to whom this issues needs to be assigned or direct to as I am unclear?

I assigned it to myself.  Thanks.

I assigned it to myself.  Thanks.

I might just close this one.  The metrics somehow broke in a very strange way.  I honestly suspect that github changed something on their back-end.  In the pygithub api, I used to do this to check the rate:
gh.rate_limiting
And it worked!  But just yesterday, it stopped working.  It would query once and then cache the answer.  So I'd keep checking it and it would also look like the rate limiting was fine until the scripts would due to rate limiting.  So weird that it's behavior changed.  I didn't change anything in my scripts nor the version of python nor the modules.  I think that it must query a 'can-cache' value or something which changed server side.
Anyway, it took forever to debug but finally I realized it's behavior had changed and I replaced it with:
gh.get_rate_limit()
which works as expected.
So the scripts are slow but since they run only once a week in the background, I don't care that much.  When they were failing, I was thinking we needed another solution.
So let's leave this open for a few weeks and if the scripts are fine, we can close it.

I might just close this one.  The metrics somehow broke in a very strange way.  I honestly suspect that github changed something on their back-end.  In the pygithub api, I used to do this to check the rate:
gh.rate_limiting
And it worked!  But just yesterday, it stopped working.  It would query once and then cache the answer.  So I'd keep checking it and it would also look like the rate limiting was fine until the scripts would due to rate limiting.  So weird that it's behavior changed.  I didn't change anything in my scripts nor the version of python nor the modules.  I think that it must query a 'can-cache' value or something which changed server side.
Anyway, it took forever to debug but finally I realized it's behavior had changed and I replaced it with:
gh.get_rate_limit()
which works as expected.
So the scripts are slow but since they run only once a week in the background, I don't care that much.  When they were failing, I was thinking we needed another solution.
So let's leave this open for a few weeks and if the scripts are fine, we can close it.

The metric scraping is slow but it works fine.  Closing this.

The metric scraping is slow but it works fine.  Closing this.


Ability to convert today's date into Hijri

Thanks for this work. It's a great resource!
`# Current date
current_date = datetime.now()
year = current_date.year
month = current_date.month
day = current_date.day
print ("1 : ", Gregorian(year, month, day).to_hijri())
print("2 : ", Gregorian.fromdate(current_date).to_hijri)
current_date = date.today()
print("3 : ", Gregorian.fromdate(current_date).to_hijri)`
1- Works, but I get errors for the 2nd and 3rd attempts.
I have consulted  https://hijridate.readthedocs.io/en/stable/usage.html but not sure what to do next. Can you confirm if Option (2) is possible? I thought it would be after reading the Gregorian Class description.
Many thanks!
1 :  1446-01-21
2 :  <bound method Gregorian.to_hijri of Gregorian(2024, 7, 27)>
3 :  <bound method Gregorian.to_hijri of Gregorian(2024, 7, 27)>

Please close this issue. My mistake in not including the parenthesis at the end e.g. to_hijri()
Please accept my apologies and thanks for the great work.


Fix Travis build

Seems to fail during mvn corbertura:cobertura ...:
Failed to execute goal org.eluder.coveralls:coveralls-maven-plugin:3.1.0:report (default-cli) on project cas-server: Build error: Either repository token or service with job id must be defined

Generated locally, but the Travis logs are truncated and the real error is not displayed.

I started to experiment with Travis and turned on sudo: false. so far so good

We may also want to remove http://developer.jasig.org/repo/content/groups/m2-legacy/

Yes, I think this repository is used also during the release. Without any success.

Build is fixed. I will remove cobertura and the legacy repo next and experiment again

Do you intent to replace corbertura by something else or simply drop it?

Well, I dont really know of a good replacement. I'll look around to see what exists, but otherwise we may just drop it.

The cobertura plugin has issues; It cant at times parse the try-resource syntax, and it also complains about missing dependencies during instrumentation. It works to the point that it no longer crashes the build and runs, and submits test coverage data, partially. I will leave it there, and have opened up several issues for the plugin project owners.

This is working fine now. The legacy repo is needed only for spnego module because of some dead dependencies that are not in central. I'll keep the issue open and monitor travis for a bit. Looks like Hazelcast tests are failing because of travis's environment somehow.

Done.


Test MetaInfo for Java DSL and invoke JIRA API library on Citrus Test Failure

TestMetaInfo annotation which stores testcase information such as
description
requirementID
scenario
author

CSV report generation from the test output.

All the test cases can be imported into JIRA through JIRA import plugin from citrus csv file.

CSV report gets generated with the listed information and mapping is
@CitrusTest.name - Test Name
@TestMetaInfo.description- Test Description
Place Holder for JIRA import- Test Step
Place Holder for JIRA import- Test Data
Place Holder for JIRA import- Expected Result
UUID- External ID
Current time- Time Stamp
Place Holder for JIRA import- Components
Place Holder for JIRA import- Priority
@TestMetaInfo.author- Author
Place Holder for JIRA import- Labels
@TestMetaInfo.requirementID- Requirement Id
@TestMetaInfo.scenario- Scenario
Test output- Actual Result
Exception if any	Failure Reason
csv file generated control is similar as html report
citrus.csv.report.enabled : Enables/disables CSV report generation (default= true).
citrus.csv.report.directory : Output directory path (default= ${project.build.directory}/test-output/citrus-reports).
citrus.csv.report.file : File name for the report file (default= citrus-test-results-dateandtime.csv).

On Test Faillure, JIRA API will be invoked which we created for our own organization to create failure report through API calls.

One question will users that do not have the failure report API in JIRA be able to use this extension in a profitable way?

Any news here @viharnaik ?

This only works with JIRA APIs. User without API in JIRA will not be able to use it.

Hi again!
We discussed this PR internally and agreed that this would be a great extension but more as a plug-in instead of a core feature of the framework.
To achieve this, we'll have to design a SPI for citrus. This would enable anybody to add isolated implementations for citrus related features as separate projects. Unfortunately this is nothing we can provide in the near future.
So even if we will not accept this PR, your help and the idea that has been grown out of it is much appreciated! Please continue to propose PRs and ideas as you desire. We much appreciate every input from the community!


fix: README.md github app example secret

The configuration is expecting list of object, the given example has invalid JSON syntax.

Description of your changes

Fixes #
I have:

[ ] Read and followed Crossplane's contribution process.
[ ] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

@efekete thanks.


DOC update FAQ and index to better reference the different topics

Codecov Report

Merging #530 (a1db5ab) into main (9bc14de) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #530   +/-   ##
=======================================
  Coverage   54.48%   54.48%
=======================================
  Files          43       43
  Lines        2786     2786
  Branches      511      511
=======================================
  Hits         1518     1518
  Misses       1153     1153
  Partials      115      115


diagnoser's load method signature does not match ruby's causing failure on Ruby 1.9.3

This is causing a failure at least in Ruby 1.9.3 and possibly earlier versions as well when doing spork -d
Just fixed the signature.

I'm still getting an error even after I've upgraded to spork 1.0.0rc2:
$ spork -d
Using RSpec, Rails
Preloading Rails environment
Loading Spork.prefork block...
wrong number of arguments (2 for 1) (ArgumentError)
~/.rvm/gems/ruby-1.9.3-p125@mign/gems/spork-1.0.0rc2/lib/spork/diagnoser.rb:35:in `load'
~/.rvm/gems/ruby-1.9.3-p125@mign/gems/activesupport-3.2.1/lib/active_support/dependencies.rb:245:in `block in load'
~/.rvm/gems/ruby-1.9.3-p125@mign/gems/activesupport-3.2.1/lib/active_support/dependencies.rb:236:in `load_dependency'
~/.rvm/gems/ruby-1.9.3-p125@mign/gems/activesupport-3.2.1/lib/active_support/dependencies.rb:245:in `load'
~/.rvm/gems/ruby-1.9.3-p125@mign/gems/spork-1.0.0rc2/lib/spork/test_framework.rb:138:in `block (2 levels) in preload'
~/.rvm/gems/ruby-1.9.3-p125@mign/gems/spork-rails-3.2.0/lib/spork/app_framework/rails.rb:8:in `preload'
~/.rvm/gems/ruby-1.9.3-p125@mign/gems/spork-1.0.0rc2/lib/spork/test_framework.rb:134:in `block in preload'
~/.rvm/gems/ruby-1.9.3-p125@mign/gems/spork-1.0.0rc2/lib/spork.rb:62:in `exec_prefork'
~/.rvm/gems/ruby-1.9.3-p125@mign/gems/spork-1.0.0rc2/lib/spork/test_framework.rb:120:in `preload'
~/.rvm/gems/ruby-1.9.3-p125@mign/gems/spork-1.0.0rc2/lib/spork/runner.rb:69:in `run'
~/.rvm/gems/ruby-1.9.3-p125@mign/gems/spork-1.0.0rc2/lib/spork/runner.rb:10:in `run'
~/.rvm/gems/ruby-1.9.3-p125@mign/gems/spork-1.0.0rc2/bin/spork:10:in `<top (required)>'
~/.rvm/gems/ruby-1.9.3-p125@mign/bin/spork:19:in `load'
~/.rvm/gems/ruby-1.9.3-p125@mign/bin/spork:19:in `<main>'


[OPT] INode KeyedObject Pool

Why INode KeyedObject Pool?
Multiple users may access the same path at the same time, for example, /tmp/1.txt. The system must guarantee only one INode exist in NameNode. Since we removed INodeMap from NN, new INode for 1.txt will be created if new requests are coming from new users.  That's will cause the cache coherence problem because we use local cache in INode.
For single NN, we may implement INode KeyedObject Pool to force only one INode per file or directory in the system.
For multiple NNs, we may forward the same requests for 1.txt (write op) to same NN. If no write op in advance,  read op can forward to any NNs.

This is also a good way to improve the performance.

https://github.com/apache/commons-pool/blob/c67581593f1668cac625bf212c9405324c9b47d8/src/main/java/org/apache/commons/pool2/impl/GenericKeyedObjectPool.java#L1445-L1534


refactoring to allow for building and releasing of images

Changes

Updating the project structure so Dockerfile can be used by build/release process
Adding Build and Release jobs to push images to a container registry

Validation

Release from my fork: https://github.com/acornett21/preflight-trigger/releases/tag/0.0.2
Images pushed from main/release job: https://quay.io/repository/acornett/preflight-trigger?tab=tags

Note: This PR is dependent on other PR's with updated make file changes for build, vet, fmt, etc.

/approve

/approved


Add RtmClient::send_rich_message

A convenience method for sending messages with attachments.

@mthjones how does this interact with the codegen changes upstream? Will this be necessary?

On second thought, this is the wrong way to do this. I should make a patch for slack-api-schemas instead.

I'm hesitant to add (more) methods to the RtmClient that just proxy to the web API. I think these methods are best left just in slack_api as either another library that needs to be pulled in or re-exported as an api module from this library.
Regarding the actual changes here -- there are going to be few breaking changes from the new release that would break this change as well.
I'd like to improve this use case (currently the method just takes an Option<&str> for attachments), but I don't think this is the right way to do so.

Just thought I'd chime in. In my app I made a very rough message builder struct where you could assign at the bits you need before sending it. Also an attachment builder. Seemed nicer than the method with lots of Nones

Yep, I've done the same thing as well. It'd be nice if everyone didn't have
to reimplement that themselves though. :)
Regarding passing a bunch of Nones, that goes away in the next release.
Instead of passing each param, each method now takes a struct that derives
Default, so you can just toss a ..Default::default() in there to fill in
the fields you don't care about.
On Mon, Mar 6, 2017 at 2:21 AM David Hewson<EMAIL_ADDRESS>wrote:

Just thought I'd chime in. In my app I made a very rough message builder
struct where you could assign at the bits you need before sending it. Also
an attachment builder. Seemed nicer than the method with lots of Nones
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/slack-rs/slack-rs/pull/72#issuecomment-284320468, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACx4GXlmTSM9W97iWjAm44DnuyFJCCXks5ri7PngaJpZM4MTEtK
.


rubocop fix PercentLiteralDelimiters errors accross codebase

discovered by: https://github.com/fastlane/fastlane/pull/8751

I'm going to merge this right away to try to get some builds unstuck 👍  Thanks!


add a prop to autoclose the date picker after the user has selected a date range

This function solves the problem (这个功能解决的问题)
makes selecting a date range faster and less confusing
Expected API (期望的 API)
a prop to have the closing behavior

maybe ALSO add a way to open/close the date picker programmatically?

maybe ALSO add a way to open/close the date picker programmatically?

I think we should do this.

Closing behavior can be a prop. We will add it later.


added logger and manifest features

Signed-off-by: kumarabd<EMAIL_ADDRESS>Description

Added logger support for controller logging interface.
Split ApplyManifest() to fetch runtime.Object separately.
Another member function to return unstructured.Unstructured{} for a given manifest.

I think this is ready for a minor version upgrade. @mgfeller @utkarsh-pro your thoughts?

@kumarabd not sure about the purpose of the controller logger and where it is used, some documentation could help.
Minor version upgrade do you mean to 0.2.0 (as per semver?). It's not a bugfix, but adds functionality (Controller Logger) in a backward compatible way?

@mgfeller<EMAIL_ADDRESS>for your reference on the controller-runtime logger interface that it uses. This is needed for the custom controller manager (Meshery operator) that we are working on currently.
Yes 0.2.0, yes all of them backward compatible as we are not changing the interface. Rather adding an additional method. Does it make sense?

@kumarabd thanks. yes, makes sense.


Question/Feature Request: return timing information only

I'd like to compare two functions loop1 and loop2, running each of them count times, and compare the average durations.
I've raised a separate issue about capturing the output e.g. y| returned None from loop1() in 0.414763 seconds  and it's easy enough to parse that string to get just the duration 0.414763 as a float or datetime object, but I wondered if it would be an easy enough enhancement to add keyword argument e.g. x = y(loop(), duration_only=True) for users just wanting to record durations and do further analysis (calculate the mean, plot a graph comparing the two loops etc)?
Or have I just missed an easy way of achieving this from the README?
import requests
from ycecream import y

@y(show_exit=True, show_enter=False)
def loop1():
    x = page*count
    for i in range(len(x)):
        s=len(x)

@y(show_exit=True, show_enter=False)
def loop2():
    x = page*count
    lx = len(x)
    for i in range(lx):
        s = lx

if __name__ == "__main__":
    page = requests.get('https://www.bbc.co.uk/news').text
    count = 10
    for loop in [loop1, loop2]:
        results=[]
        for run in range(10):
            result = y(loop(), as_str=True, )
            results.append(result)
        print(results)

OUTPUT
y| returned None from loop1() in 0.414763 seconds
y| returned None from loop1() in 0.418484 seconds
y| returned None from loop1() in 0.408090 seconds
y| returned None from loop1() in 0.406661 seconds
y| returned None from loop1() in 0.418872 seconds
y| returned None from loop1() in 0.413751 seconds
y| returned None from loop1() in 0.418103 seconds
y| returned None from loop1() in 0.406673 seconds
y| returned None from loop1() in 0.406783 seconds
y| returned None from loop1() in 0.412694 seconds
['y| loop(): None\n', 'y| loop(): None\n', 'y| loop(): None\n', 'y| loop(): None\n', 'y| loop(): None\n', 'y| loop(): None\n', 'y| loop(): None\n', 'y| loop(): None\n', 'y| loop(): None\n', 'y| loop(): None\n']
y| returned None from loop2() in 0.157985 seconds
y| returned None from loop2() in 0.158627 seconds
y| returned None from loop2() in 0.160737 seconds
y| returned None from loop2() in 0.165181 seconds
y| returned None from loop2() in 0.171878 seconds
y| returned None from loop2() in 0.165958 seconds
y| returned None from loop2() in 0.164515 seconds
y| returned None from loop2() in 0.160872 seconds
y| returned None from loop2() in 0.160113 seconds
y| returned None from loop2() in 0.165789 seconds
['y| loop(): None\n', 'y| loop(): None\n', 'y| loop(): None\n', 'y| loop(): None\n', 'y| loop(): None\n', 'y| loop(): None\n', 'y| loop(): None\n', 'y| loop(): None\n', 'y| loop(): None\n', 'y| loop(): None\n']

I think I have answered this in issue #5

Many thanks for the quick replies and suggestsions Ruud!  Very elegant.  Using output= catches the results perfectly.  In case it's of use here's my final code based on your suggestions:
from statistics import mean, variance
from ycecream import y

def collect(s):
    if not hasattr(y, "results"):
        y.results = {}
    func = s.split("from ")[1].split("()")[0]
    if not y.results.get(func):
        y.results[func] = []
    seconds = float(s.split("in ")[1].split(" seconds")[0])
    y.results[func].append(seconds)

def results(name):
    output = f"\nMean duration for <{name}>:\t"
    output += f"{mean(y.results[name])} seconds"
    output += f"\nMin/Max duration for <{name}>:\t"
    output += f"{min(y.results[name])} to {max(y.results[name])} seconds"
    output += f"\n\nResults for <{name}>:"
    output += f"\n{y.results[name]}"
    return output

benchmark = y.fork(output=collect, show_enter=False)

@benchmark()
def func1():
    for i in range(len(x)):
        s=len(x)

@benchmark()
def func2():
    lx = len(x)
    for i in range(lx):
        s = lx

if __name__ == "__main__":
    x = "x" *6500000
    for func in [func1, func2]:
        for run in range(10):
            func()
        print(results(func.__name__))

It might be beyond the scope of what you intended for ycecream but I do think it's a common scenario for people to want to compare two (or more) different functions and get back a range of duration values while debugging, so if it were possible to internalise something like collect and results into a convenience method in ycecream itself I think it might appeal to an even wider audience, but your call!

I think it is not possible to define a generic function to realise what you would like.
But if you can specify it, I can have a look on how to implement that. For now, I think this issue can be closed.


[Auto-generated] Check & update dependencies (pyproject.toml)

Update dependencies (pyproject.toml)
Automatically created PR based on the 'CI - Check pyproject.toml dependencies' workflow in SINTEF/ci-cd.

Codecov Report

:exclamation: No coverage uploaded for pull request base (ci/dependency-updates@f75a9cf). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@                   Coverage Diff                    @@
##             ci/dependency-updates      #64   +/-   ##
========================================================
  Coverage                         ?   73.10%
========================================================
  Files                            ?       12
  Lines                            ?      751
  Branches                         ?        0
========================================================
  Hits                             ?      549
  Misses                           ?      202
  Partials                         ?        0

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

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


Doesn't seem to work with Rails 5

After following install instructions, seems to error out when trying to include assets. Maybe it needs a gemspec update?
File to import not found or unreadable: vertical-timeline.

User error


Required upgrade to angular 19

I'm facing an issue while upgrading the application to angular 19, when can i expect an update for angular 19

Release version 19 that support Angular 19


re2: fix for pkgsStatic

Not using lib.optional to avoid changing the .drv hash.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This is causing 26 rebuilds.

ofBorg disagrees.

This is causing 26 rebuilds.

ofBorg disagrees.

ofBorg disagrees.

Using lib.optional does 26 rebuilds which is totally fine to merge into master. Please only use rebuild avoiding hacks when it would cause stdenv rebuilds. This is not such change so we don't need to do it.

ofBorg disagrees.

Using lib.optional does 26 rebuilds which is totally fine to merge into master. Please only use rebuild avoiding hacks when it would cause stdenv rebuilds. This is not such change so we don't need to do it.

@SuperSandro2000 mind having another look?

@SuperSandro2000 mind having another look?

Checked it manually and it builds.

Checked it manually and it builds.


Add switch for dark mode

Allow users to switch to a dark mode inspired theme for UI components.

Fixed in 70ccb05f8255d2dab22c715b5f929abe060bd0fa


[EasyCodingStandart] Add only method to ECSConfig

Thank you :+1:
Could you also deprecate the ONLY constant like the others? So folks know the value is deprecated.

I hope I understood you correctly 😅

Thank you 😌


return a nil message to get skipped in the pipeline

rather than rely on changing the op type for a message to be skipped, setting it to nil will result in getting skipped here
fixes #260

Coverage increased (+0.08%) to 59.812% when pulling f9d120bcb36c4b128feaa6da1dc0cd1af7fadf17 on 260-drop-noops into df11d3251a92c71cfd255a8dfb4901ba2153d673 on master.

This LGTM. However I do see a bug in the skipMsg logic. When skipping a message, we don't update the LastMsg, so if we end up using that for persistence it'll be off by a bit.

good catch, I think until we start using LastMsg that issue may disappear as the state of persisting transporter state is still influx.

Yeah, it should be okay for now and when we spec out persistence, it'll get caught