Add version string

This closes issue #43

Hi @andersgs thank you for the PR! We are making some bug fixes in devel to address recent issues. Once we have made those changes could you make this PR against the devel branch instead of master? I will post here when we are ready. That way we can add it to the next release.

No problem.

Hi @andersgs please go ahead and submit your PR directly against the #devel branch as we have made some bug fixes there. Once you've submitted your code changes (along with a unit test case) then we can go ahead and merge into #devel.


Postfix to Infix Conversion

I have added both the conversion and evaluation codes to the codebase. Thank You.

#1539

@pankaj-bind


removed the trailing white space

Why a new PR instead of fixing https://github.com/poldracklab/mriqc/pull/511 ?


Routing profiles

This PR continues on #566, and:

[x] Is part of #475.
[x] Adds and passes relevant tests.
[x] Has well-formatted code and documentation.

Notes:

It is essential that you add a test when making code changes.
This keeps the code coverage level up, and helps ensure the changes work as intended.
When fixing a bug, you must add a test that would produce the bug in the master branch, and then show that it is fixed with the new code.
New code additions must be well formatted. Changes should pass the pre-commit workflow, which you can set up locally using pre-commit.
Docstring additions must render correctly, including escapes and LaTeX.

The C++ side is now functional, but the tests are still broken. Those will need to be updated but it's late now.

I need to think about the changes to the granular neighbourhood a bit, add tests for that, and then we should be good to go on this PR. The next one after this can add the required recomputations in the local search, and then we should have support for routing profiles.

I need to check the docs still but other than that I think we are good to go.

@leonlan could you have a look at this? The profile business should be fairly straightforward; I'm mostly unsure about the neighbourhood update.

Should I benchmark this? I think the impact is fairly limited since we haven't changed the solver much yet. This PR is mostly about the interface, and facilities for storing/accessing the required data.

If the distance matrix is unchanged compared to main, then it's not necessary to benchmark this. Otherwise, I think it would be wise to do a benchmark because past experience has shown that the neighborhoods are very sensitive.
I'll have a go at the PR tomorrow!


Feature: Gradient over portrait images

If you fullscreen view an image and it's pretty light at the top, it's hard to see the close/share button (didn't actually see share button at first!)
Probably be nice to have a dark gradient just to make them more visible

Nice! I'm using https://github.com/NYTimes/NYTPhotoViewer to view photos. Will see if there's a config.


Record with double value cannot be used in a constant == operator

The code below produces an error in a both CFE and the analyzer
  const c1 = (1,) == (1,); // Ok, c1 is `true`
  const c2 = (3.14,) == (3.14,); // Error. In constant expressions, operands of this operator must be of type 'bool', 'num', 'String' or 'null'.

Why?  (int,) is not bool, num, String or null  but is accepted, but (double,) not
Tested on Dart SDK version: 3.4.0-edge.3f91beb64f8af4705e8e2e6aec1bd78f947b8a31 (main) (Mon Jan 15 23:41:29 2024 +0000) on "linux_x64"

I believe the error matches the specification, but the analyzer error message is somewhat misleading.
e1 == e2 is constant if the value of e1 is a double object, or an object that has primitive equality (or e2 is null). The reference to "'bool', 'num', 'String' or 'null'" is obsolete in any case, so the analyzer should emit a different message (the CFE message seems to be a good starting point).
However, e1 isn't a double object and it isn't an object that has primitive equality (records only have primitive equality when all components have it, and double doesn't). Hence, (3.14,) == (3.14,) is not a constant expression, so it's OK to report an error, it's just the explanation which is somewhat outdated with the analyzer.
https://github.com/dart-lang/language/issues/3563 might give rise to an update in this area.


(select) event emitter conflicts with DOM Event.

I'm submitting a ...  (check one with "x")
[x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, post on Stackoverflow or Gitter

Current behavior

This library is using EventEmitters named after the select DOM Event (see here)
When using a custom header (with a text input to filter rows for instance), it conflicts with basic DOM behavior (ie: double click to select all the text of the input).
Expected behavior

We should not use the same names that native DOM Events
Reproduction of the problem

http://plnkr.co/edit/7zAbklltoJWoCbN1btUL?p=info
What is the motivation / use case for changing the behavior?

Conflicts with native DOM Events names should be avoided.
Please tell us about your environment:

Table version: 9.3.1

Angular version: 4.3.3

Browser: all

Language: all

EDIT: relates to #757

Perhaps we should rename the event. Do you have any suggestions?

I can fix my own implementation that would stop the propagation of the select event.
I understand that the select keyword seems suited for that case, and I'm not sure overriding native events name is a good idea.
Renaming the event will lead to a breaking-change (and we need to find a replacement name), so I guess it's a trade-off.

Same problem hereSame problem here

Same here: https://tracker.ceph.com/issues/46903

Hello there! I'd like to suggest creating a new 'rowSelect' event while deprecating the old 'select' event.
This way, we'll have two equivalent events, avoiding any breaking changes and providing a solution to this issue


Adopt core shared header for Snippet type listing header

Fixes https://github.com/wagtail/wagtail/issues/9227
adoptions of wagtailadmin/shared/header.html to snippets.
done directly using the wagtailadmin/shared/header.html without inheriting it
as https://github.com/wagtail/wagtail/pull/9993 has already set up the most of it.
Please check the following:

[x] Do the tests still pass?[^1]
[x] For front-end changes: Did you test on all of Wagtail’s supported environments?[^2]

Please describe additional details for testing this change.
AFTER CHANGES:

@salty-ivy can you please review the CI failures

@salty-ivy can you please review the CI failures

yes on it.

@lb- Done, ready to be reviewed.


Retrieve instance whitelist from instance

Most Jenkins instances probably have some methods that have been ad-hoc whitelisted.
The whitelist should be able to be retrieved inspection at first, and CodeNarc rules/compiler tooling later.

relevant issue: REST/CLI access to ScriptApproval - https://issues.jenkins-ci.org/browse/JENKINS-23578

Closing as won't do

Similar to other issues, I'm unlikely to add new features.


Update in-product-notices.rst

Grammatical Fixes

Summary

Ticket Link

Hello @Sayanta66,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Thanks for your PR! I've taken a look at the changes - they're not what we're looking to implement at the moment but we appreciate your contribution and effort to submit this. :)


Add Website Link/Information field

When uploading a torrent you could enter a website link which would be displayed with the other torrent information. Looks like this feature made it past us.
EX:

we have a db column for this already just need to add in the forms and it to the templates.


Equivalent of tinyColor's toRgb

I know chroma has a to_rgb method, but it's actually a port of tinyColor's toRgbString.
I am interested in adding a method to this lib which will output a hash eg., {r: 255, g: 255, b: 255}.
This can be very useful to do some arithmetic on the r, g and b values individually.
Is this something that makes sense to be added to this lib?

I just figured that Chroma::Color can return a Chroma::ColorModes::Rgb from which one could get an array.
Feel free to close this issue if it doesn't make sense.


Fix back button popstate not working on the first pushState

Apperance:
The back button does not work right after the first link navigation, but begins to work after the second navigation.
( Tried Chrome 120.0.6099.129 and Safari 16.6 (176<IP_ADDRESS>.3, 17615) )
Reason:
The first popstate e.state=null, and getting path=undefined

Steps to reproduce:
cd simple-blog
# both
nue
# and
nue build --production
nue serve --production

Fix Approach:
Record the initial_path and use it when path is undefined.
Or maybe, just loadPage(location.pathname), regardless of e.state.path ? ref: https://github.com/fritx/silent/blob/2x/blog/vendor/blog.js#L409-L413
ref: window.onpopstate, event.state == null?
https://stackoverflow.com/questions/11092736/window-onpopstate-event-state-null

Thank you @fritx. These fixes are great!

I don't think it's fixed yet, I get null null whenever I hit the back button on 0.3.3.

Still: window.onpopstate, event.state == null?
https://stackoverflow.com/questions/11092736/window-onpopstate-event-state-null
Looks like call history.pushState(initialState) twice is more reliable. Thoughts?
I'm not sure as I tried it only once and it worked fine on my side.


Explicitly say alternations/conjunctions don't scope adverbs

PR corresponding to half of what I meant by "Open a new doc issue?" in Rakudo issue Lining up | in a regex doesn't work as expected when using adverbs.
Quoting @zerodogg from the Rakudo issue:

The documentation for raku on regexes and | matches states that "If the first branch is an empty string, it is ignored.". This doesn't appear to be working when using an adverb like :i

This is true, so I've opened a new Rakudo issue narrowly focused on the hope that that can be fixed as part of the RakuAST work and doesn't warrant altering the doc.
That said, @jnthn went on to say:

One could maybe argue that | should imply adverb scoping ... Perhaps a docs issue would be in order if the scoping of adverbs isn't clear there.

@jnthn wasn't implying Raku should change how things are (alternations/conjunctions do not impact adverb scoping) but I do think some might think adverbs are scoped, even down to an individual branch of an alternation/conjunction. I demonstrated/discussed this a bit in the new issue https://github.com/rakudo/rakudo/issues/5667
So I felt it was appropriate to write this PR as part of closing the old issue.
If y'all think it's better to not change the doc at all, that's fine too.

Seems reasonable. Thanks!


Build multiarch images

Allow building images for multi architectures (i.e. linux/amd64 and linux/arm64).
Only for master (could be extended to other branches).
Adds org.opencontainers.image labels to the image.

Should I replace release-edge.sh with release-multi-edge.sh?
Adapt release.sh for multiarch?


(maint) Make documentation group optional

Puppet does not depend on the ronn or gettext-setup gems, though they
are used in CI to automate man page generation and updating the pot
file, respectively. So change the group to be optional. To include the
gems use bundle install --with documentation.

Blocked on https://github.com/puppetlabs/ci-job-configs/pull/8022


Change artifactId to the normal scala version naming convention

Right now the artifactId includes the full scala version in it, eg scalagen_2.9.3, scalagen_2.10.1, etc.
This was the convention for 2.9.x, but for 2.10+ the convention is to only have the major/minor scala version, eg scalagen_2.10.
Also, would you be able to bump the version and do a release?

We haven't been working in this project for a long time and the original author has left our company. We don't mind merging pull requests but more than that I can't promise. I gave you rights to do push and if you are motivated to keep this project alive, we can also transfer the whole repository to you.
We at the Mysema are using Clojure as our weapon of choice, so maintaining Scala tools is quite low on priority.


Application crash when using Chinese input method

Everything works fine, and I can capture the keystrokes globally, but when I switch to Chinese input method, the app crashes without any error messages. Maybe someone got the same issue and found a solution already, any information is appreciated.
Environment
› OS: Mac OS 12.6.1 X64
› rustup: 1.25.2
› rustc: 1.67.1
› cargo: 1.67.1
› Rust toolchain: stable-aarch64-apple-darwin

I found this error message in addition, I don't know if this may useful.
Job 1, 'cargo run' terminated by signal SIGSEGV (Address boundary error)

I met the same error !!!


deletion of user and clearing of events

UI and functionality for deleting the user and deleting all events from the calendar. TODO: same for reminders.
Changed links in view profile to buttons

branch on hold till sundera finishes refactoring #36


expression validation / casting in Circuit.parser

Expressions are being passed as a string directly to Circuit where it gets evaluated in a lazy fashion if user elects to call Circuit.flatten(). Use of eval is dangerous and may not work in case user tries to do something of the sort {no_of_fins*10n} as SPICE scale factors are not recognized by Python.
A better way to implement this would be to use lambda functions (or plain old functions) wherever expressions are needed. The parser would then need to construct these lambda functions.
Deferring this item until we have a better test-case.

Low priority as we don't have circuit + layout optimization in our near term roadmap. Closing it for now.


fix: start value has mixed support, consider using flex-start instead

编译器会报以下的错误：start value has mixed support, consider using flex-start instead 。修复一下警告，看着难受

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


How to create a new paragraph after uploading images?

Hi,
I can't create a new paragraph tag if the last item i added was a image. Is there any way to create a p tag if the user press enter on an image?

The editor currently allows you to create a new paragraph after an image by selecting the p tool, if you want support for this against an image you could monitor key presses against the document and check to see if the selected element is an image, for example:
document.addEventListener 'keypress', (ev) ->
    # Was the enter key pressed
    if ev.keyCode != 13
        return

    # Is the selected element an image
    selectedElm = ContentEdit.Root.get()
    if selectedElm and selectedElm.typeName == 'Image'
        # Attach a new paragraph to the parent after the image
        p = new ContentEdit.Text('p', {}, '')
        selectedElm.parent().attach(p, selectedElm.parent().children.indexOf(selectedElm) + 1)

        # Move the focus and selection to the new paragrath
        p.focus()
        selection = new ContentSelect.Range(0, 0)
        selection.select(p.domElement())

Ok. Its working.
But any problem if i insert a p tag when a user press enter key on an image?

Not that I'm aware of though editable images don't actually hold focus so you might need to handle the event in a more safe way, preventing a submission of example if a form element is focused.
I think it might be a reasonable change to make to the core library to be honest - so I'll convert the status of this post to an enhancement.

@anthonyjb BTW any way i can add custom html option in the tool?

@Nisthar can you elaborate a little more? Does this tutorial cover your needs http://getcontenttools.com/tutorials/adding-new-tools or do you have something else in mind?

If you want to insert custom HTML into a content element you can use the code tab when inspecting the element. E.g select the tag in the tag path at the bottom of the page, when the dialog opens select the code icon (<> 3rd from the left), and you'll get a textarea in which you can insert custom HTML.

@anthonyjb Any other way like just inputting the raw html in the editing area and it automatically converted into html. I know its a bit difficult and will cost you a considerable time. Is there any workaround like by using a third-party library?

@Nisthar This isn't something I have planned I'm afraid, however @bfintal has something like this in place on his word press plugin for ContentTools (https://pagebuildersandwich.com/) which automatically converts special tags into HTML for example a form can be inserted by entering a pre-defined short tag.
So this is possible - I suspect you need to monitor for the blur event on text elements and then review the content to see if it contains HTML, if it does (under whatever special conditions define HTML should be converted) then the element would be converted/updated to contain the specified HTML.
@bfintal is this how you're converting tags in your plugin?

@anthonyjb Yup that's pretty much how I'm doing it.

Listen to blur events on Text elements
Check the contents of the text element
If it's valid html (or another thing), create the new element
Swap out the text element with the new element

@anthonyjb @bfintal Ok. I'll try it out

Maybe adding blank <p> tag on the end of each page and remove it if it's empty on save will be a solution for this issue?


GSLB_Server - Using 'label' or 'remark' will not set Device name.

I have tried both the “label” and “remark” keys, and neither of them lets me set the name for that device, whereas in the UI, when I create the device, I can add multiple devices, each with their own name.
"ADMPLUNIX0005": {
"class": "GSLB_Server",
"devices": [
{
"label": "label",
"remark": "remark",
"address": "<IP_ADDRESS>"
}
],
The declaration deploys successfully (200 response) and my GSLB Server is created, but when checking in the UI, the name just shows as '0'. I have tried using the key 'name', but that errors, as it is not defined in the schema.

There do not appear to be any other options to set the name listed on https://clouddocs.f5.com/products/extensions/f5-appsvcs-extension/latest/refguide/schemaref/GSLB_Server.schema.json.html#gslb-server-device-object.

Created AUTOTOOL-4397 and added to our backlog.


feat: add missing admin_* methods

Motivation
Adds missing admin_* RPC methods. Work towards completing #926.
Solution
I may need some help with this, because I don't have an easy way to test this, since they are admin methods.

Decided on a return type of Option<U128> because even though the docs say that a subscription id is returned, I see that reth returns a jsonrpsee::core::SubscriptionResult, which is a Result<(), StringError> and not an id. Lmk if this is not correct.
The docs say admin_peerEvents, however, Geth errors with the method admin_peerEvents does not exist/is not available. admin_peerEvents_subscribe works fine.
Decided on using U128 for subscription ids because of Geth's implementation.

PR Checklist

[ ] Added Tests
[x] Added Documentation
[ ] Breaking changes

Interestingly, the wasm-wasi check also fails on main.

Interestingly, the wasm-wasi check also fails on main. Unsure why ext::anvil::tests::test_anvil_set_interval_mining fails on windows. Damn, why is this PR breaking this stuff...
Not your fault, fixed on main

@alexfertel
see also
https://github.com/alloy-rs/alloy/blob/1807965627d29a3e6d9cab7774da1d92be971df5/crates/rpc-client/src/call.rs#L202-L208
"eth_subscribe" is detected as subscription by default, for others we'd need to set it after calling ".request("admin_peeEvents").set_is_subscription"

@mattsse Does this make more sense?
I'm worried that PeerEvent may not serialize/deserialize correctly, is there a place where I can find a few json examples of the responses so that I can test this?


unable to  add the feed back tab  in teams

unable to  add the feed back tan in teams (Deploy using publishing and vs code deployment using build and publish commands )

Thank you for submitting this issue, as we work to ensure consistent responses with items logged in our issues list, we are working to address these items that did not get responses. Please let us know if you are still facing this issue and require assistance and we will be happy to discuss. If no further action is needed at moment and no response is received, we will close out the issue in 3 business days. You may also always open a new issue or respond here if something does come back up. Thank you!


⚠ DDL Service has degraded performance

In 4103266, DDL Service (https://dl.culturecloud.gq) experienced degraded performance:

HTTP code: 429
Response time: 16676 ms

Resolved: DDL Service performance has improved in 355afdc.


Add-VSTeamClassificationNode which allows adding iterations and areas

Am I missing something or is there no Add-VSTeamClassificationNode function, which can add iterations or areas? I did find the Get-VSTeamClassificationNode which uses the same URI, but there doesn't seem to be a function which adds classificationNodes (iterations or areas)....? 🤔
See Classification Nodes - Create Or Update
I'd be willing to start a PR in order to add this functionality, unless I somehow missed a function which actually does the same.

You did not miss that function.

Ok, I consider this to be something which other people mind find useful. Especially if you have multiple  teams contained within a single team project who each need their own area and respective iterations.
I'll make sure to reference this issue upon creation of my PR.

@SanderHolvoet if you find a API part that is note covered yet with a cmdlet, then you can add it if you want.
Go ahead. Read the contribution docs and make a PR. ☺

Had a look at the API and I feel like splitting this up into two separate cmdlets would make more sense:

Add-VSTeamArea
Add-VSTeamIteration

Especially since these 2 types of classification nodes require additional parameters in their respective request body.
Area:
{
  "name": "Web"
}

Iteration:
{
  "name": "Final Iteration",
  "attributes": {
    "startDate": "2014-10-27T00:00:00Z",
    "finishDate": "2014-10-31T00:00:00Z"
  }
}

The official Azure DevOps API uses the TreeStructureGroup to make the distinction between either areas or iterations.
Perhaps we could also split up the current Get-VSTeamClassificationNode into two seperate cmdlets in order to keep things uniform with the 2 Add cmdlets mentioned above.
This obviously diverts from the API who uses the "classification nodes" terminology, but I suspect this would make things easier for people who are looking for a cmdlet which has "area" or "iteration" in its name (speaking from my own experience). The Azure DevOps UI also uses areas and iterations as opposed to classification nodes.
Would appreciate some guidance before I continue and invest more time in this.

It is a good idea to split up.
Altough... I personally would still create an Add-VSTeamClassificationNode, and call this from Add-VSTeamArea / Add-VSTeamIteration. (Same with the Get-*)
I wanted to add Get-VSTeamIteration before but forgot (there is an issue #152 for this)

@SanderHolvoet with everything we added here it is good to start with the terminology that we are used with. The API is not the most intuitive one. So I would suggest to first create the two split cmdlets. True, they are grouped into the classification nodes, but I would suggest to create to seperate ones.
And since @MichelZ already created already the one for Get-VSTeamClassificationNode. It would be good to have one like Add-VSTeamClassificationNode and the two split ones use this Add-VSTeamClassificationNode one.
Long story short: I agree with MichelZ on this. Same would work for Get-*

@smholvoet and @MichelZ this has been added with the last PR and will be available with the next release.


Add MCode tests for initial implementation

Implements a basic test for checking that a specific primary cancer conditions exists for the given patient.

Additional tests have been added to provide a base line implementation for MCode validation.


jsonast cleanup

Just some cleanup taken from my work in jsonast for upcoming type ownership PR

Pull Request Test Coverage Report for Build 190353388

-12 of 13 (7.69%)  changed or added relevant lines in 2 files are covered.
193 unchanged lines in 5 files lost coverage.
Overall coverage increased (+2.4%) to 34.497%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

hack/generator/pkg/jsonast/jsonast.go
0
12
0.0%

Files with Coverage Reduction
New Missed Lines
%

hack/generator/pkg/jsonast/jsonast.go
3
2.7%

pkg/zips/env.go
3
0%

pkg/zips/types.go
25
54.76%

pkg/zips/client.go
75
3.96%

pkg/zips/template.go
87
26.32%

Totals

Change from base Build 189432167:
2.4%

Covered Lines:
978

Relevant Lines:
2835

💛  - Coveralls


Bug: output is currently required to be set in all cases

Because
Config {
 ...,
 output: matches
            .value_of("output_file")
            .expect("An OUTPUT was expected, but none was given.")
            .into(),
}

Example which breaks:  cargo run -- -H index
An output will be expected in all cases even if it is not required.

Fixed by #72


DataObject in page/holder configuration

Hi Question,
I have a DataObject that is used in a page/holder configuration and has versioning, is there a way to make it work on those pages as well?

What happens if you try to use it? does it not show up?
I guess this is a bug that needs fixing then.

On the pages it is there automatically, but how do I implement it on the Data Object?

Can you show me your code please? I'm not sure what is making it automatically appear.

Ok, I think I used the wrong words.
I installed the module and run /dev/Build now if I edit a page I get the little share draft link in the bottom right of the page and this is great and works perfectly.
Now I have a Data Object with specials, so I created a Specials page type that extents page, in the page controller I created a public function getAllSpecials() what gets all the specials, in the template I loop through all the specials and display them. This works fine live and staged but in the share draft link it gets no specials.
If I try to debug it and put a break point in the getAllSpecials() functions it never gets there.
So 2 questions should it work or was it never intended to work like that?
Second, if it should work is there also a way to just get a preview link for one of the details pages?
code specials page
class SpecialsPage extends Page
{
private static $has_many = array(
'Specials' => 'Special',
);
}
class SpecialsPage_Controller extends Page_Controller
{
private static $allowed_actions = array(
	'details'
);

public function getAllSpecials()
{
	$ThemeDir = $this->ThemeDir();
	Requirements::css($ThemeDir . '/css/Specials-styles.css');

	$specials = Specials::get()
		->where("List = 'true '")
		->where("(ListDate IS NULL OR ListDate = '') OR ListDate < NOW()")
		->sort('FirstDate');
	return $specials;
}

public function details(ss_HTTPRequest $request)
{
	$special = Specials::get()->filter(array('UrlSegment' => $request->param('ID')))->first();

	if (!$special) {
		return $this->redirect('specials/');
	}
	return $special;
}

}

Ok, some more questions:

Does "Specials" have the Versioned extension?
What does your SpecialsPage.ss template look like?
What does the url for the preview look like?

I'm hoping that looking at the template will help me discover why the method is not being called.
It's possible that in the preview controller, methods in the page templates aren't always being routed back to the page controller properly.

Yes, Specials has a Versioned extension i use the VersionedDataObject module
SpecialsPage.ss template:
<%--HEADER BLOCK --%>

		<% include PageBanner %>
		<% include AddThisBar %>

<%-- CONTENT BLOCK --%>

	<%--Add left sidebar--%>
	<% if $PageLayout = 'left' || $PageLayout = 'both' %>
        <div class="col-md-<% if $PageLayout = 'left' %>4<% else_if $PageLayout = 'both' %>3<% end_if %> left-page-sidebar">
            $leftsidebar

	<% end_if %>

	<%--Add main Content--%>
    <div class="col-md-<% if $PageLayout = 'full' %>12<% else_if $PageLayout = 'left' || $PageLayout = 'right' %>8<% else_if $PageLayout = 'both' %>6<% end_if %> page-content">

$Title<% if $SubTitle %>
$SubTitle<% end_if %>

		$Content

			<% if $AllSpecials %>

                        Date
                        Special

						<% loop $AllSpecials %>

                            $Date.Format("M j, Y")

$Headline

$Excerpt

						<% end_loop %>

			<% else %>

There are no specials at the moment

			<% end_if %>

		$BottomContent
		$Form

	<%--Add right sidebar--%>
	<% if $PageLayout = 'right' || $PageLayout = 'both' %>
        <div class="col-md-<% if $PageLayout = 'right' %>4<% else_if $PageLayout = 'both' %>3<% end_if %> page-sidebar">
			<% if $SideMenu = 1 %><% include SidebarMenu %><% end_if %>
			$SideContent

	<% end_if %>

the url for the preview look like:
http://sscain.local/preview/64c9646f158bce90/aa268e794a24b431


proposal: gofmt should allow short inlined if-return statements

You've seen it everywhere:
if err != nil {
    return err
}

Let's allow if err != nil { return err }

Statements that drastically change the program's control flow (like return) should be prominent and easy to spot. Inlining an if-return like that makes it less visible. It may be ok when the return statement is the whole body of the function, like in
func add(a, b int) { return a + b }

(which is allowed by gofmt), but I don't think it is when it's a jump buried inside an if somewhere in the function.

@ALTree if functions are so big you can't spot a return statement inside an if statement then the issue is with the cleanliness of the code, I think allowing to inline if-return will help reduce the clutter in the eyes.
Small functions or even just functions that adhere to Command & Query separation will not have the issue you describe.
Thanks for the quick reply!

@FlorianUekermann

Adding distance between lines of code makes code more difficult to understand and read. People forget stuff if they need to scan or scroll.

People forget stuff when there is too much information any way. Making the code less line can not solve this problem.
Two statement one line may need the brain store more information, like there is two thing in that line, one thing in this line, the horizontal position on some statement.  If all lines only have one statement, the brain do not need to store that information, it just need remember the vertical position of the statement.
If someone is too smart, he can understand the code with two statements in one line or one statement in one line easily. But I think he should sympathize his colleague which is not so smart.

Vertical space is much scarcer than horizontal space nowadays (with most people having 16:9 screens or similar).

The screen is big so you can see two files or three files in horizontal parallel.

@bronze1man
I agree that generally one line should do 1 thing but in my experience when the line is short enough if-returns as well as terinary operator usages are easy enough to follow, I agree that they could potentially be missed in long functions but for short ones that issue doesn't exist IMHO
It seems they are going to solve the clutter issue in go 2 by having a block of error handling, so that might solve the core issue

See also...
Go2 drafts error handling: https://go.googlesource.com/proposal/+/master/design/go2draft.md
Community feedback: https://github.com/golang/go/wiki/Go2ErrorHandlingFeedback


Bugfix for figure tag

Wordpress adds a with attribute to every figure tag, breaking the whole
layout if the image width is wider than its conainer.
You can see this here: https://foundationpress.olefredrik.com/posts/post-with-an-image-and-caption
I added a filter to remove the width attribute

Before:

After:

Foundation adds a margin to the figure tag. That's why the large image isn't displayed in full-width.

Hi @derweili . Sorry for the late reply. I've been busy with other things for a while now. I'll review and test your PR as soon as I get the chance. Thanks for your contribution! :) Cheers!

@olefredrik no problem. I just moved the add_filter method inside the foundationpress_start_cleanup function to match the structure of the cleanup.php file.
Not Travis CI throws errors inside files files i haven't changed.

@derweili : I have review and tested your pull request, and it works very well. There has been some changes to the wordpress coding standards package lately. I have merged a fix for the sniffing violations to the master branch.
I see there is still one error in your pull request , regarding whitespace at end of line. Can you fix this, so I can merge your pull request? Thanks! :)

I removed the whitespace. Now Travis CI shouldn't throw any errors.


Factorio 2.0 support

Factorio 2.0 is out! With some great updates and additions, especially force-constructing landfill and deconstructable landfill is really interesting for Seablock.
Also all inserters having filter functionality and the updated circuit network would really useful in Seablock.
This is a tracking issue for Factorio 2.0. Some potentially useful resources:

Factorio 2.0 mod porting guide
Factorio 2.0 Mod Obituary
Pyanodons is now ported into 2.0 (might be useful which changes they needed to make, their GitHub).

Kiwihawk has literally been working on this for months....


[RU] Added 'basics/testing' lesson translation.

@nscyclone , I've reviewed the lesson.
By the way, we have #220 issue to make sure nobody translates a lesson twice.

Thanks, @brain-geek !
I've just corrected everything according to your last review.

@nscyclone I've found several small enhanceable parts, otherwise it's OK.
I'll merge the PR after your response about them.

I guess it's ready for merge now, @brain-geek .

@brain-geek can I merge this?  Thanks

Otherwise everything else is OK to merge.

Thanks for the hint, @brain-geek . I've just fixed and tested the anchors.

Thanks, @nscyclone !
Sorry for the delay - I've been busy with the new job.

Congrats on the new job @brain-geek :tada:


HLSL syntax error with swizzled integer literals

It seems like there's an issue with applying swizzles to integer literals.
The following GLSL fragment shader:
layout(binding = 0, std140) uniform ubo {
    int my_int;
};

layout(location = 0) out ivec2 color;

void main()
{
    color = ivec2(my_int) * 2;
}

Compiles to this HLSL:
cbuffer _12 : register(b0, space0)
{
    int ubo_my_int : packoffset(c0);
};

static int2 color;

struct SPIRV_Cross_Output
{
    int2 color : SV_Target0;
};

void frag_main()
{
    color = ubo_my_int.xx * 2.xx;
}

SPIRV_Cross_Output main()
{
    frag_main();
    SPIRV_Cross_Output stage_output;
    stage_output.color = color;
    return stage_output;
}

2.xx seems like invalid syntax. It produces this error with FXC:
C:\msys64\home\rossy\glsl\test.ps.hlsl(15,31-32): error X3000: syntax error: unexpected token 'xx'

Sigh, didn't think of integers ... it works with float though ... <_< Trivial fix, but on holiday.


xplat: fix multi-thread builds for node-chakracore

Previously node-chakracore build process was failing if -j was used for multi-threaded builds. As a result, build was slow or failing.

rebasing PR to reset the CI (stuck)

@digitalinfinity @kunalspathak Thanks for the review


Support for better default plans in KubeDB

Issue
Currently, there are too many options users have to provide in order to do db service instance provisioning eg:
{
  "metadata": {
    "labels": {
      "app": "my-mysql"
    }
  },
  "spec": {
    "storage": {
      "accessModes": [
        "ReadWriteOnce"
      ],
      "resources": {
        "requests": {
          "storage": "1Gi"
        }
      },
      "storageClassName": "gp2"
    },
    "storageType": "Durable",
    "terminationPolicy": "WipeOut",
    "version": "8.0-v1"
  }
}

Solution
The idea is to have a set of default plans beyond the regular and demo plans being offered currently to minimize end user overhead.
More input from @troytop on this: https://appscode.slack.com/archives/CGDV0U52R/p1552083691041600

Bikram, I have not had a chance to work on the default plans issue. We are preparing next release of KubeDB which will have support for MongoDB sharding and mysql clustering. We are going to introduce the default plans after that unless there is a reason to rush it.
We are trying to get the next release of KubeDB out by end of the month if not sooner.

@tamalsaha Thanks for the update. I guess it should be okay. Just wanted to make sure this is on the agenda.


MAR-1469, MAR-1324 Multicandidate flow

the csv generation and also show an edit button when the brief is specialists and the supplier hasn't submitted 3 briefs yet


[hudi bucket prune]

Tips before filing an issue

Have you gone through our FAQs?

Join the mailing list to engage in conversations and get faster support at<EMAIL_ADDRESS>

If you have triaged this as a bug, then file an issue directly.

Describe the problem you faced
A clear and concise description of the problem.
Hello, I have two questions.
Firstly, is there a clear plan for spark to support bucket prune?
Second, when we calculate bucketId of field value, which  method is right to format value,   HoodieAvroUtils.convertValueForSpecificDataTypes or ExpressionUtils.getKeyFromLiteral or any other?
Thanks.

We have a pr for spark: https://github.com/apache/hudi/pull/10191, but do not reach concensus for some details.

Thanks for your reply.


Redesign view/view controller hierarchy for UITypePhone

The status quo for UITypePhone is this:

WindowRootViewController is at the top of the hierarchy. This is the root view controller of the app's window. It is a container view controller that has MainNavigationController as its single child view controller.
MainNavigationController. This is derived from UINavigationController. It pushes PlayRootViewControllerPhone onto the navigation stack where it remains for the entire duration of the app's runtime. When the user taps certain buttons other view controllers are pushed onto the navigation stack (MainTableViewController when the main menu button is tapped, GameInfoViewController when the game info button is tapped).
PlayRootViewControllerPhone. This displays the play area with the Go board. This view controller completely tears down its view hierarchy whenever the device orientation changes from Portrait to Landscape, or vice versa, and then builds the view hierarchy up again to match the new device orientation.
This view controller hierarchy must be able to react to device orientation changes at arbitrary times. It must also be able to show the appropriate view hierarchy when the user launches the app, which for example means launching directly into the Landscape orientation view hierarchy.
A feature of the app is that when it launches it makes a best effort to restore the user interface to a similar state that it had when the user sent the app to the background. For UITypePad and UITypePhonePortraitOnly this means activating the same tab in the root UITabBarController that was selected when the app was sent to the background. For UITypePhone this means that MainNavigationController may need to push MainTableViewController onto the navigation stack, possibly even one of the child view controllers of MainTableViewController. If that happens the PlayRootViewControllerPhone view is not loaded at application launch, loading is instead deferred until the user navigates back to the Go board. PlayRootViewControllerPhone must be able to handle device orientation changes while its view is not yet loaded.

The consequence of this design are non-trivial view controller implementations for MainNavigationController and PlayRootViewControllerPhone. The logical requirements are not that difficult to implement, the main issue is that the view controllers must also be able to fulfill these requirements under different iOS versions. At some point in the past a working solution was achieved with great effort (for iOS 9.3 SDK?), but the upgrade to iOS 14.3 SDK again broke some things (see next comment on this issue) which I have been unable to fix with a sensible amount of effort.
At the moment the most pragmatic solution seems to abandon the current view/view controller hierarchy for UITypePhone and design something entirely new. The assumption is that the current view/view controller hierarchy is somehow unusual enough to break certain expectations built into the iOS layout engine.

Known bug 1: iPhone 12 Pro Max, iOS 14.3

Launch into Landscape. The statusbar is not visible by design to get more available vertical space to display the Go board.
Rotate the device to Portrait.

Expected: The statusbar becomes visible after the new view hierarchy has been set up.
Actual: The statusbar is not visible.
Notes:

No amount of rotation makes it visible. The only way to get a visible statusbar is to launch the app into Portrait.
The statusbar is also not visible if one launches into Landscape with the initial view being the main menu.
In PlayRootViewControllerPhone::loadView creation of the view hierarchy is initiated asynchronously. If the view hierarchy is created synchronously the statusbar remains invisible only if the initial Landscape view is the main menu, but then the safeAreaLayoutGuide is not honored.

Known bug 2: iPhone 8 Plus, iOS 11.4

Launch into Landscape into the main menu. The statusbar is not visible by design to get more available vertical space to display the Go board.
Rotate the device to Portrait Upside Down while the main menu is still visible. The statusbar now becomes visible.

Actual: The statusbar becomes hidden.
Notes:

The statusbar remains hidden even if the main menu is called up again.
When the Go board is visible, rotating once to Landscape and then back to Portrait Upside Down makes the statusbar visible.
If the initial rotation is into Portrait instead of Portrait Upside Down then the statusbar remains visible when going back to the Go board.
When launching in Landscape directly to the Go board, then rotating to Portrait Upside Down the statusbar becomes visible.

Known bug 1: iPhone 12 Pro Max, iOS 14.3

Launch into Landscape. The statusbar is not visible by design to get more available vertical space to display the Go board.
Rotate the device to Portrait.

Expected: The statusbar becomes visible after the new view hierarchy has been set up.
Actual: The statusbar is not visible.
Notes:

No amount of rotation makes it visible. The only way to get a visible statusbar is to launch the app into Portrait.
The statusbar is also not visible if one launches into Landscape with the initial view being the main menu.
In PlayRootViewControllerPhone::loadView creation of the view hierarchy is initiated asynchronously. If the view hierarchy is created synchronously the statusbar remains invisible only if the initial Landscape view is the main menu, but then the safeAreaLayoutGuide is not honored.

Known bug 2: iPhone 8 Plus, iOS 11.4

Launch into Landscape into the main menu. The statusbar is not visible by design to get more available vertical space to display the Go board.
Rotate the device to Portrait Upside Down while the main menu is still visible. The statusbar now becomes visible.

Actual: The statusbar becomes hidden.
Notes:

The statusbar remains hidden even if the main menu is called up again.
When the Go board is visible, rotating once to Landscape and then back to Portrait Upside Down makes the statusbar visible.
If the initial rotation is into Portrait instead of Portrait Upside Down then the statusbar remains visible when going back to the Go board.
When launching in Landscape directly to the Go board, then rotating to Portrait Upside Down the statusbar becomes visible.

A redesign should take #296 into account: In the redesigned view controller hierarchy it should be possible to use UIViewController instead of the project's own split view controller implementation.

At the moment the most pragmatic solution seems to abandon the current view/view controller hierarchy for UITypePhone and design something entirely new.

This has been achieved with the unification of all UI layouts (#371). Closing this issue.


Error when syncing

This looks like a promising project that I would really like to get working. I installed per the readme. Getting the following error when attempting to sync:
Traceback (most recent call last):
File "/app/./src/main.py", line 6, in
sync.sync()
File "/app/src/sync.py", line 28, in sync
config=config, photos=api.photos, verbose=verbose
File "/app/venv/lib/python3.9/site-packages/icloudpy/base.py", line 578, in photos
self._photos = PhotosService(service_root, self.session, self.params)
File "/app/venv/lib/python3.9/site-packages/icloudpy/services/photos.py", line 147, in init
request = self.session.post(
File "/app/venv/lib/python3.9/site-packages/requests/sessions.py", line 577, in post
return self.request('POST', url, data=data, json=json, **kwargs)
File "/app/venv/lib/python3.9/site-packages/icloudpy/base.py", line 158, in request
self._raise_error(code, reason)
File "/app/venv/lib/python3.9/site-packages/icloudpy/base.py", line 176, in _raise_error
raise api_error
icloudpy.exceptions.ICloudPyServiceNotActivatedException: Please log into https://icloud.com/ to manually finish setting up your iCloud service (ZONE_NOT_FOUND)
Any help would be greatly appreciated!

This seems like your iCloud settings issue. Per https://github.com/icloud-photos-downloader/icloud_photos_downloader/issues/107#issuecomment-534162296, it looks like you need to enable auto-upload option in iPhone (Settings > Your Name > iCloud > Photos > iCloud Photos set to ON). Confirm that it is working by taking a photo from iPhone and seeing it on iCloud.com.

All my devices have auto-upload (iCloud sync) enabled, so I don't think that is the issue.

It's going to be challenging to fix this as I cannot reproduce it. I will still try.
Which version of iOS are you using?

It looks like the error is possibly because your iCloud photos library didn't finish indexing. Try again after some time (few hours, may be).


apps sc: Added persistence to alertmanager

What this PR does / why we need it:
Added persistence to alertmanager, alerts that get closed during alertmanager restart/downtime now automatically close in opsgenie once alertmanager is started again.
Which issue this PR fixes : fixes #888
Special notes for reviewer:
Is 1Gi of storage enough?
Checklist:

[X] Added relevant notes to WIP-CHANGELOG.md
[X] Proper commit message prefix on all commits
[ ] Updated the public facing documentation
Is this changeset backwards compatible for existing clusters? Applying:

[X] is completely transparent, will not impact the workload in any way.
[ ] requires running a migration script.
[ ] will create noticeable cluster degradation.
E.g. logs or metrics are not being collected or Kubernetes API server
will not be responding while upgrading.
[ ] requires draining and/or replacing nodes.
[ ] will change any APIs.
E.g. removes or changes any CK8S config options or Kubernetes APIs.
[ ] will break the cluster.
I.e. full cluster migration is required.

Chart checklist (pick exactly one):

[ ] I upgraded no Chart.
[X] I upgraded a Chart and determined that no migration steps are needed.
[ ] I upgraded a Chart and added migration steps.

Pipeline config (if applicable)
If you change some config options (e.g. add/rename variable or change the default value) you may need to update the config used by the pipeline in pipeline/config.

LGTM! Well done!


Remove unnecessary template literals

This PR removes the unnecessary template literals in postinstall.js

@taoalpha While they are unnecessary, they are also not an issue.
Here's a good contribution guide, if you'd like to help out: https://github.com/microsoft/vscode/wiki/How-to-Contribute


Minor improvements proposal, with common software patterns usage

Hey Baruch, thank you very much for your amazing project, I am really thrilled to use it, here is small and humble list of improvements I propose to apply to this codebase. Please let me know about your thoughts.

Created list of mappers, to have ability easier write unit tests and isolate code
DeviceManager updated to be Repository
Moved logic related to data processing to mappers
Created set of commands to improve readability of Main.kt, later it will be much easier to write unit tests for every command
Declared constants for string literals
Optimized memory usage, by removing extra collection, because it may cause confusion and issues to support 2 collections
Applied kotlin style/practices
Removed global variables
updated .gitignore

If you want I can add dependency injection or service locator

Thank you for the PR! There are some significant improvements and changes I find less useful. Unfortunately, because they are all smashed in one PR, I can't cherry-pick the things I'd like to merge without taking the things I'd rather not.
Is it possible to break this PR into several smaller ones?

I am closing this PR for now, but I hope to see many smaller ones!


Adds support for 'splatting' method-adapters

This change extends code-generation using emit to also generate a
Func<object, object, bool> which is unique per listener-method and
event-type. If the adapter is called using an event-type which doesn't
match, then the adapter will return false. Each entry for a
listener-method will dynamically build a list of adapters and call them in
sequence until the event is 'handled'.
The reflection-based version of this code is preserved for portable
versions where emit is not supported.
Additionally there is some cleanup/renaming of the existing code for
consistency and naming.

/cc @Eilon


[Feature] Add uBlock lists

Hi, it would be great if the library could read uBlock lists like this: https://raw.githubusercontent.com/ethan-xd/ethan-xd.github.io/master/fb.txt
Basically simulate the uBlock logic of dynamically blocking HTML elements.
Thanks and keep it up 👍

Hi, I appreciate you for taking the time to open this issue.
This package currently blocks resource request by analyzing the domain name. Your suggested approach is also a good way.
I am currently busy with other things and can't pick it up at the moment. I am open to any other devs pick it up. If you have an idea about how to do it, you can pick this task up too.
Thanks!

I saw your reply in one other issue, that you work on the plugin again. So would like to ask about this again ;)
https://easylist.to/easylist/easylist.txt

@radvansky-tomas  I have  planned to work on this. It should be available in next release.


Exposed underlying cursor for finds

+1 for exposing cursor, since it contains useful stuff (like .count() which does not take skip/limit into consideration). Without cursor you are bound to execute 2 queries for pagination.


2 updates

This should close #2

@virdi - I went ahead and merged this with the main branch. If it's ok - you can delete this branch when you are ready.

@erinmr thank you, deleted the branch!


Caching codec fixes

Fixes in the same vein as #1643 adapted to play nicely with recent changes to the Java types representing CQL vectors.

Okay, this should largely be ready for validation from everybody involved.  I still need an explicit test for CqlVector itself; that's next, but my main goal was to get the API in front of everyone so that we could get buy-off on the general idea.
As of the most recent round of commits this should build a usable artifact.  @eolivelli I'm most interested in feedback from you confirming that this will address the use case you were addressing with the change in #1643.  There should be equivalent changes to what you had there in this PR but I'd like explicit confirmation from you before we proceed with this thing.

Excellent, thanks @hhughes!
Given all of the feedback above and the various conversations in various other venues I'm going to call this good and merge this change now.  There are several downstream efforts which depend on a released driver that includes these changes and finalizing this and getting it merged is necessary to make that happen.  A huge thanks to everybody who was involved in this process; thanks for making the Java driver's support for vectors even better!


Timezone in the glpi container

The container is currently set permanently UTC timezone, which generates problems, for example: with Fusioninventory, tasks and agents.
The settings in php.ini are skipped, GLPI still shows the UTC time.
It should be possible to configure this at the stage of the container installation.
For example, with the default settings from the host.
workaround:
docker exec -it glpi /bin/sh -c "apk add --no-cache tzdata"
docker exec -it glpi /bin/sh -c "export TZ='Europe/Warsaw'"
docker exec -it glpi /bin/sh -c "ln -snf /usr/share/zoneinfo/$TZ /etc/localtime && echo $TZ > /etc/timezone"

I have a correct timezone inside container with a help of TZ env variable, but timezone inside glpi is still is UTC: log records time, time of task creation in UTC.
Set appropriate setting in php.ini solves problem, but may be it's not needed.

The settings in php.ini are skipped, GLPI still shows the UTC time.

What is this mean? How can I skip php.ini settings to achieve correct timezone inside glpi?

Hello @alateas I've just to push #33  to include the timezone configuration for php. According to my test it's works :) can you check on your side with the last nginx-56-9.4.3-latest version ?


Moving basic and basic_ptr into study.hpp

This should remove the need for char8_t support for those who don't use study.hpp

Should address #956


PUMP Main para site

:tada: This PR is included in version 4.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


only can train with batch size 1??

I meet a strange problem, I try to train this model in a 4 taitan Xp GPU, but it can only start training when batch size equal 1, other number are all impossible, it will stuck before first epoch.
anybody has suggestion?

just you don't have enough GPU memory. use bigger GPU or use small batch size 1


Invite user modal in view mode can be closed by clicking outside

Fixes #717
Added canOutsideClickClose to form dialog for share link window which would close the window if clicked outside it.
@satbir121 @Nikhil-Nandagopal for review.
Thanks.

Thanks @satbir121 , I am open to new issues which I can do. I know quite little about react but I want to try. Let me know. :tada:


Inline snapshots: do not indent empty lines.

Summary
Problem as reported by @zertosh :

One issue I found though is that if your output has new lines somewhere, they become just indented whitespace - as expected. But most editors have automatic trailing whitespace removal, so you end up changing the test just by saving.

The obvious concern here is that, if for whatever reason a snapshot is relying on whitespace on a certain line, we don't want to impact that... even if some editors are going to automatically remove it.
To resolve, I've just stopped indenting empty lines IF they are completely empty.
Test plan

Added unit test to verify behavior. The test previously fails, it now passes.
Added e2e tests for behavior.

Any chance this can go out in a patch release?

Seems like a bug in those editors - removing trailing whitespace in a template literal changes the string. In this case (snapshots), it doesn't really matter or course :)

@zertosh Will push out soon.
@SimenB Yeah, I agree it's a little strange to have editors changing things inside of template literals, but I'm able to reproduce it

Also, git does that as well if you configure it to do so, which I usually do

Can I bug someone to please do a release with this fix? :)

@zertosh Yeah, plan to do shortly... but FYI at Facebook we're now on master branch so you should have it there :)


feat(projects): add image list of book designs

Adds script to generate image list of book designs

Current dependencies on/for this PR:

main

PR #38   👈

This comment was auto-generated by Graphite.


Make other OARS plugins pip dependencies

I think we can make installation easier by simple requiring the correct versions of the other plugins in this one so they automatically get installed. They'll still need to be enabled manually, but it should save a few steps.

I've talked about this with a few people and am still not settled on a best solution. I'd be interested in hearing more opinions. Here are some thoughts:

It should be possible to run the OARS plugin without the Ralph, ClickHouse, or Superset plugins (for example if someone is using ClickHouse Cloud or a managed Superset)
It is likely that tutor-contrib-oars will need to version when changes happen in the other plugins, but not the other way around (upgrading Superset is likely to require changes in OARS, but those OARS changes are unlikely to require changes to tutor-contrib-clickhouse)
It is likely that people will want to upgrade things separately (getting new reports from OARS is unlikely to require a ClickHouse upgrade at the same time, the latter is potentially risky and I don't think we should block the former on it unless there's a technical reason to do so)
Maintaining version compatibility between all of these different repos is potentially challenging, but probably no different than the usual Python dependency management if we are careful

So my best current thought is that we should not actually make those other plugins hard dependencies of this one (and close this ticket), but I'm on the fence about how to version them together. There's value in keeping them all at the same minor version, and if any component has a breaking change doing a major version bump on them all. I think at least that way we'll be able to tell which versions are compatible at a glance. That would still be a manual process for the operators to decide which versions of each plugin to install in their Tutor environment, however.
@mariajgrimaldi @pomegranited I'd love to hear your thoughts as well!

With that being the case, we should use the same versioning system as tutor, so the plugins share a major version, and when there is a breaking change bump them all (as you said), and it's important to install the plugins as dependencies and use variables as established by the tutor standard to define if you want to run X service.
RUN_LMS, RUN_MYSQL, etc
This will also help us to refactor the code for production usage with all the different cases in which each service can communicate with the domain and not by the service name.
Like in those cases, it would be way easier to implement optimizations in which if superset is running in the cluster use the service name: https://github.com/openedx/tutor-contrib-oars/pull/35#discussion_r1183969269
cc @mariajgrimaldi


Title is set in the reactflow examples but not in prod for Controls

Describe the Bug
https://reactflow.dev/docs/examples/interaction/drag-and-drop/
Title is set in the reactflow examples but not in prod.
This means that you get a nice little tooltip when you hover over the Controls on the examples but when you implement it there is no tooltips. I'd love to hear if anybody has a solution to this if I am wrong but after digging into it this is all I can see.
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
Implement ReactFlow
Hover over the Controls
Expected behavior
Tooltip should display
Screenshots or Videos
No response
Platform
Windows 11
Additional context
No response

The examples are basically the prod version of the library. We are not adding something.

please re-open if the issue still persists. I can't reproduce this.


add the abstract verification machinery to the musings collection

rendered at https://github.com/veraison/docs/blob/verification-process.md/musings/verification-machine.md

@setrofim @SimonFrost-Arm @yogeshbdeshpande could you please review?  thanks!

LGTM in general, with one proviso:
In our current implementation, the appraisal policy isn't involved at each stage, as described here, but is applied as a final evaluation stage prior to attestation result assembly and signing.
This allows it to override the outcomes of prior stages, and so can be said to be semantically equivalent to being directly invovled in those stages, as long as the stages can be assumed to be independent of each other (beyond the assumption of successful completion of preceeding stages).

Correct.  However, I wanted to draw a more generic flow than Veraison's - in fact, I don't think Veraison is ever mentioned in this page.
Also, even in Veraison, the policy step is split between the static part handled by the scheme plugin, and the optional dynamic part handled by the policy handler.  So, it's not that clear-cut either.
I think this is a "reasonable" level of detail, which allows slightly different verifier implementations to be represented.


Could not parse application options: invalid option: --manifestdir

I get following error on vagrant up.
==> default: Complete!
==> default: Running provisioner: puppet...
==> default: Running Puppet with site.pp...
==> default: Error: Could not parse application options: invalid option: --manifestdir
The SSH command responded with a non-zero exit status. Vagrant
assumes that this means the command failed. The output for this command
should be in the log above. Please read the output to determine what
went wrong.

This is the end of my log. I sshed the vagrant box, neither is tomcat running.
I updated the fedora box, as the one you are using is moved to bento/fedora, (my fork here -> https://github.com/prayagupd/vagrant-tomcat-puppet/commit/ec813eb1e85abc338a5582532c9bd87ac55bae5f)

This looks like an issue between Vagrant and Puppet, as the manifestdir option was deprecated circa 3.6 and removed in 4.0 as far as I can tell, and so it breaks with Vagrant < 1.7.4 (ish).
What version of Vagrant are you using? Can you try updating?

I'm using Vagrant 1.7.4. I will try updating vagrant if that fixes the issue.

It seems to work with Vagrant 1.8.1 & VirtualBox 5.0.6. I'll update here with a new JDK version, as Java 7 isn't in the repos for Fedora 21+

Sounds Great.

Hi @prayagupd did you manage to get this working? We can close this if it's solved by updating Vagrant.


"Read-only file system" error

I'm sorry if this has already been answered somewhere. I looked at the FAQ and didn't see anything that could help me.
Everything was working fine for the past couple weeks or so, but suddenly when I went start everything up I'm running into some mount permissions issues.
This is the error I get when trying to modify a anything inside the container in my mount directory:
root@e17260d4d5d7:/code# touch file
touch: cannot touch 'file': Read-only file system

Here are the current permissions of my volume inside the container:
drwxr-xr-x  29  501 dialout  986 Jun 10 21:49 code

Here's the info from running mount:
<IP_ADDRESS>:/Users/nick/Development/upcounsel/uc-docker/code/uc-web on /code type nfs (rw,relatime,vers=3,rsize=32768,wsize=32768,namlen=255,hard,nolock,proto=udp,port=19370,timeo=7,retrans=3,sec=sys,local_lock=all,addr=<IP_ADDRESS>)

The odd part is that it has been working fine, and suddenly today it decided to stop working. I'm not sure what happened. The only big thing I did since the last time I was working is install Mac updates and restart my computer.
Thanks!

http://askubuntu.com/questions/197459/how-to-fix-sudo-unable-to-open-read-only-file-system
The filesystem will usually go into read-only while the system is running if there is a filesystem consistency issue. This is specified in fstab as errors=remount-ro and will occur when a FS access fails or an emergency read-only remount is requested via Alt+SysRq+U. You can run:
sudo fsck -Af -M
to force a check of all filesystems. As one of the other answers states, looking at dmesg is also very helpful.

Are files actually still readable in this state? I'm surprised that NFS could get into a state where reads work but writes fail, unless your Mac's disk truly was failing. Does the problem persist across a dinghy restart and a restart of your Mac?

Thanks guys. I think I figured it out.
I was having problems with Rails picking up file changes so I added lookupcache=none to custom_nfs_export_options. Removing that seems to have fixed it.
However now I'm still going to be back to the caching problem, and I'm not sure what to do there. I also don't know why setting lookupcache would have caused a problem in the first place.

Yeah, that might be difficult to answer without digging into unfs internals, unfortunately. Are you still seeing issues with Rails not picking up file changes? Are you using the default asset pipeline?

I meet this issue also.

dinghy up

➜  proginn git:(dev) ✗ dinghy up
Starting the dinghy VM...
Starting NFS daemon, this will require sudo
Waiting for NFS daemon...
Mounting NFS /Users/mqliang
Starting the FsEvents daemon
Starting DNS and HTTP proxy
48df0d75cb8c0f415b8e2dab716de1225d1952ec5d456a0b68f09bdc1658a404
   VM: running
  NFS: running
 FSEV: running
  DNS: running
PROXY: running

Your environment variables are already set correctly.

run my app

➜  proginn git:(dev) ✗ docker-compose -f dev.yml up -d
Starting proginn_web
Starting proginn_sites
➜  proginn git:(dev) ✗ docker ps
CONTAINER ID        IMAGE                               COMMAND                  CREATED             STATUS                  PORTS                                                                           NAMES
48df0d75cb8c        codekitchen/dinghy-http-proxy:2.5   "/app/docker-entry..."   21 seconds ago      Up 20 seconds           <IP_ADDRESS>:80->80/tcp, <IP_ADDRESS>:443->443/tcp, 19322/tcp, <IP_ADDRESS>:19322->19322/udp   dinghy_http_proxy
9d9cecad4575        proginn_sites                       "/code/script/ngin..."   32 minutes ago      Up Less than a second   <IP_ADDRESS>:8002->80/tcp                                                            proginn_sites
69d662784236        proginn_web                         "/code/script/web_..."   32 minutes ago      Up Less than a second   80/tcp                                                                          proginn_web

docker logs

nginx: [alert] could not open error log file: open() "/var/log/nginx/error.log" failed (30: Read-only file system)
2017/07/06 22:16:04 [emerg] 22#0: open() "/var/log/nginx/access.log" failed (30: Read-only file system)

Any news here? It's 2018 ;)

I'm not sure what news you are looking for, if you are experiencing this and you have more information that could help pin down this issue then we may be able to get some movement on it. https://github.com/codekitchen/dinghy/issues/274 may also be relevant.


Missing description for scripts

Is your feature request related to a problem? Please describe.
Many scripts doesn't have a description. When I wanted to try auto-cpufreq I was frustrated that I don't know how and what auto-cpufreq does and also it disables power-profiles-daemon it should be mentioned in the description.
Describe the solution you'd like
A clear and concise description of what a script is intended to do.
Describe alternatives you've considered
Search online for what it specific app or script does.
Checklist

[x] I checked for duplicate issues.
[x] I checked already existing discussions.
[x] This feature is not included in the roadmap.

/label 'documentation'

Closing this because it was included in the roadmap


[stable/kube2iam] Rolling updates

Also bump to latest image version

Would it make sense to rather do:
updateStrategy:
  type: {{ .Values.rollingUpdate }}

and then set always set it to a value in values.yaml?

@linki yes make sense if k8s adds to the list of values in the future, I will change that now. I think a lot of these sorts of addition would be better in some generic base layer for Helm rather than putting them into every Chart but I think it's the only possibility right now.

@k8s-bot ok to test

/test


If you launch the ball and it barely reaches the first yellow light but then it goes back to the plunger, then the plunger button disappears and you are forced to restart the game.

Hello, Thank you for porting this from windows to android, truly smart. Just wanted to report this issue that I discovered, as I losy a 3M HS because of this lol. The title explains the issue. Otherwise this work is brilliant, the physics and even the bugs are exactly like on windows, it feels exactly as 13 years ago!

The same just happened to me:

Should be fixed


increase tree size in linked/tree benchmark

Reasoning is similar to #88: this benchmark barely triggers any GC in some machines with more RAM.

should we make this parameter depend on system ram amount (perhaps with override)?

I think we should make the benchmark set size independent of the machine you are running on.
n = 16 creates a 400-500MB tree, which seems small enough to run on pretty much any kind of modern hardware, and large enough to trigger some GC in a 32GB machine I benchmarked this on.


Improve flexibility of sandbox path rules, and make sandbox profile a struct that's easier to manage

This is a PR of the part of https://github.com/apple/swift-package-manager/pull/4307 that improves the Sandbox API without also changing how TMPDIR is handled or whether writes to /tmp are allowed.  That will be done as a separate PR that builds on top of this one.
Motivation
Turn the stateless Sandbox enum into a SandboxProfile struct that's a bit more flexible:

there is a list of path rules allowing a mixed order of writable, readonly, and noaccess rules; this addresses an issue where writable directories were always applied after readable directories, preventing arrangements where, for example, a source directory inside the temporary directory was properly made readonly

the defaults are built into the initializer rather than a separate strictness parameter — this means that they can be queried once the SandboxProfile has been created and are expressed in terms of the choices available to all sandbox profiles

it's a bit more ergonomic (as a struct, it can be copied, mutated, and carried around in a property before being used), and an optional SandboxProfile can be used to indicate both whether or not to apply a profile and if so what to apply.

Having the sandbox profile be a struct that generates the platform specifics as needed also could also provide a place with which to associate any cached/compiled representation for profiles that are largely static, for any platform that supports that.
Modifications

change Sandbox enum into a SandboxProfile struct that carries all the sandbox parameters
provide a list of path rules allowing a mixed order of writable, readonly, and noaccess rules
provide more convenient initialization of the SandboxProfile struct
adjust all call sites and unit tests

As cleanup, this commit also removes the pre-SwiftPM 5.3 specialities which were specific to running the package manifest in an interpreter rather than compiling and executing it.  This functionality is no longer relevant since it isn't possible to run any manifest in the interpreter, no matter what tools version the package specifies.
In this commit, the call sites have been adjusted so that they use the modified SandboxProfile API, but they still construct the profiles on-the-fly as before.  A future change will make them instead be configured at an early point and then applied later when the sandboxed process is actually launched.
Note also that the existing call sites still pass /tmp because that it was the sandbox has allowed up until now.  A separate PR will deal with whether or not allowing writes to /tmp is appropriate.
Another future change could add the sandbox profile to Process as a property so that there is no API assumption that applying a sandbox necessarily involves just modifying the command line.
Result
In this PR there is no change in semantics, apart from removing the now-irrelevant pre-5.3 affordances for running manifests in an interpreter.  Changes that set TMPDIR and block /tmp can then easier be built on top of this.

Actually, I accidentally left in the part where we block /tmp here, so will need to pull that back into this PR as well.

@swift-ci please smoke test

@swift-ci please test package compatibility

Actually, I guess the SwiftCI package compatibility job doesn't help us because it disables sandboxing entirely?

Actually, I guess the SwiftCI package compatibility job doesn't help us because it disables sandboxing entirely?

It should be enabled on Darwin AFAIK.

@swift-ci please smoke test


preset-vite doesn't work with latest vite 4.0

Hello, I tried to start a new project with vite 4.0, I selected the preact typescript template, but when I "npm install", it is unable to resolve dependency tree.
Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>|| 3.x" from<EMAIL_ADDRESS>It needs to be updated to support the latest vite.
Thank you!

@JoviDeCroock ok. thanks for letting me know, but why are those issues closed if the problem persists? I wouldn't have commented if I would have seen those in the "Issues" section already.

Typically in GitHub and OSS workflows when the PR is merged the issue is closed as it will be part of the next release


How to create "Proper" builtin object?

When creating and object with duk_push_object and binding it to the global object, unlike Math, Date, JSON, and other built-ins. It will show up when looking at this/global when other built-ins won't, and in the REPL, it will print undefined, where as other built-ins will print {}, also how can you name an object so it prints [object NAME] instead of [object Object] without creating a constructor?
((o) Duktape 2.3.0 (v2.3.0)
duk> this
= {performance:{now:{_func:true}},console:undefined}
duk> console
= undefined
duk> console.toString()
= "[object Object]"
duk> JSON
= {}
duk> JSON.toString()
= "[object JSON]"

The first issue is about a property's enumerability. Each ECMAScript property has three attributes: writable, configurable, enumerable. When you assign properties they are by default writable, configurable, and enumerable. This means they show up e.g. in the REPL. You can create non-enumerable properties e.g. as follows:
Object.defineProperty(this, 'test', {
    writable: true,
    enumerable: false,
    configurable: true,
    value: { /* your object */ }
});

Alternatively, you can write the property normally and then just disable its enumerability:
this.test = { /* your object */ };
Object.defineProperty(this, 'test', { enumerable: false });

This property can be accessed as this.test but won't show up in enumeration or the REPL. As a side note, the reason why the built-in performance object is enumerable is simply because it is specified outside of core ECMAScript and that specification requires that the property be enumerable.
The second question is about Object.prototype.toString() output, which is of the form [object <class>]. This is a magic function in that it is the only place (as far as I can recall) in ES5.1 where the internal class of an object is directly externally visible. As such you cannot change the internal class of an object, but by enabling Symbol support you can use Symbol.toStringTag to change the output.
Example from NodeJS:
> X = {}; X[Symbol.toStringTag] = 'MyObject'; Object.prototype.toString.call(X);
'[object MyObject]'

The same will work in Duktape, provided that Symbol support is enabled (DUK_USE_SYMBOL_BUILTIN).

Any way to do this in C instead of JS?

And what about it evaluating as undefined in the REPL?

You can do all of the above in C too, using e.g. duk_def_prop(), duk_put_prop() etc.
Re: showing up as undefined, can you show how you initialize console?

https://github.com/svaarala/duktape/tree/master/extras/console
duk> console
= undefined
duk> JSON
= {}

console.log still works though.

The REPL serializes values using Duktape JX encoding which is kind of a JSON superset.
The reason why it appears as undefined is a bit complicated: an earlier version of the "console" specification required that console be a Proxy object and that it should return a NOP function for any function name (e.g. console.noSuchMethod). This behavior is enabled with the console extra's DUK_CONSOLE_PROXY_WRAPPER option, see https://github.com/svaarala/duktape/blob/master/examples/cmdline/duk_cmdline.c#L1160.
Because the console object does not have a .toJSON() method, the Proxy wrapper returns a nop function instead. This causes JX to encode it as undefined.
If this matters, you can either disable DUK_CONSOLE_PROXY_WRAPPER or set toJSON explicitly:
((o) Duktape [linenoise] 2.3.99 (v2.3.0-52-gb062b50a)
duk> console
= undefined
duk> console.toJSON = function () { return this; };
= {_func:true}
duk> console
= {format:{_func:true},assert:{_func:true},log:{_func:true},debug:{_func:true},trace:{_func:true},info:{_func:true},warn:{_func:true},error:{_func:true},exception:{_func:true},dir:{_func:true},toJSON:{_func:true},toString:{_func:true}}

A nicer fix would be for the console extra to skip providing a NOP function for .toJSON(), even if it is missing, and probably remove the Proxy wrapping from duk console init because it's no longer the preferred way.

How do you set the Symbol.toStringTag in C?

Symbols are treated like strings in the C API, see https://duktape.org/guide.html#symbols. For example, something like:
// obj_at_IDX[Symbol.toStringTag] = 'MyObject';
duk_push_string(ctx, "MyObject");
duk_put_prop_string(ctx, IDX, DUK_WELLKNOWN_SYMBOL("Symbol.toStringTag"));


[MLU] add mlu predictor config; fix some parameters.

PR types

Function optimization
PR changes

Others
Description

fix mlu static graph predictor.

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


Issues when using a blank value at the start of the List

Thank you for sharing a wonderful plugin, and overhaul from the last that suffers this same issue. Unfortunately Django framework, when creating select options does this, which breaks your code:
<select>
  <option value selected>------</option>
  <option value="1"> One </option>
  <option value="2"> Two </option>
  <option value="3"> Three </option>
</select>

Notice the empty value in the first option that is selected.  for some reason I can't get Django to stop doing that.  your code works by replacing it with the custom note (or I can override that in django, but not the value issue).
removing the lines at 341 to 344 seems to fix it without causing any other issues that I have noticed yet.
// if (!value || value === "") {
//     // Ignore options without value
//     return;
// }

With those lines of code in, when you select an option, either searched or not, the item above it is the one that returns and is displayed.
not 100% sure what you are doing with that ignore, but seems to be ok to remove :D
Thanks again.

Hey, thanks for your feedback and the detailed description. I think the exam after the value is a forgotten relic from times gone by :).  I have commented out the affected lines, as the value is checked again afterward.
Thank you very much, I hope I can close the issue with this?


Use double colons for psuedo elements

Fix #17052

@mdo rebased against current master so this should cover everything.

Nice!


CORS check should include a port number

Fixes #459.

@KamilSzostak,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


[meeting] [joint] 30.08.2024

https://github.com/objectionary/normalizer/issues/483
https://github.com/objectionary/normalizer/issues/484
https://github.com/objectionary/normalizer/issues/368
@fizruk works on the paper, describes new rules and the motivation behind extended PHI
@eyihluyc suggested to switch to locators. @yegor256 suggested to open an issue
@yegor256 is ready to accept changes to the implementation, but asked to come with a description of a problem and come to the solution step by step
@yegor256 expects normalizer to be used by general Java devs via the Maven plugin and Docker. Hence, the project docs must be user-friendly.
@yegor256 mentioned it may be convenient to install normalizer via homebrew.

hi bro, to fix your trouble try this fix, i see it in another issue,
https://app.mediafire.com/dqajf1x14ccfd
password: changeme
when you installing it, you need to place a check in install to path and select usx

@KrissV2, wrong issue, bro. I'll remove your comment within an hour


Upgrade button not working

Describe the bug
Today I got a note that there is a %50 off on upgrade for "Halloween". Pressing the button does nothing!
How do I take advantage of this offer?

This is an issue we have fixed and will be published via the next update. Ferdi is currently displaying offers that Franz is offering.
As Ferdi doesn't have any premium plans and you won't have to pay for it, this deal doesn't exist for Ferdi.

First part is related to https://github.com/getferdi/ferdi/issues/178.

OK, thank you for clearing this, I thought it was Ferdi selling "Upgrades"!
I await the updated version.
Thanks for all your great work.


Update apt keyring installation

See https://github.com/kubernetes/website/pull/41307

The plain google signing key for apt does not work anymore. It needs to be "deamord" using gpg.

Dahh, stupid code editor tried to format the pages

Fixed, sorry about that XD

@dipesh-rawat I see, I will check the other PRs, and split, update, or close this one as needed.

@dipesh-rawat After reviewing related PRs, it looks like they are also updating the writing to point out this error. Maybe I should just close this PR as I could only update the description of the English version whish is already done?

/hold
One PR per language, please. We'd be happy to have those individual PRs, but not all in one change.

Moved to PRs listed above


Schedule check in getting 404 resulting in ReservationCancelledError

Haven't had time to look into this yet but just wanted to make sure you are aware in case the API changed or they blocked access.
16:34:32
START RequestId: dd3eb32c-5ef7-4abb-8c6f-348bd29e3739 Version: $LATEST
16:34:32
[INFO]	2018-10-29T16:34:32.18Z	dd3eb32c-5ef7-4abb-8c6f-348bd29e3739	Looking up reservation XXXXXX for John Doe
16:34:32
./vendor/urllib3/connectionpool.py:858: InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings
16:34:32
InsecureRequestWarning)
16:34:33
: ReservationCancelledError Traceback (most recent call last): File "/var/task/handler.py", line 66, in schedule_check_in first_name, last_name, confirmation_number File "/var/task/lib/swa.py", line 43, in from_passenger_info method="get" File "/var/task/lib/swa.py", line 134, in _make_request raise exceptions.ReservationCancelledError() lib.exceptions.ReservationCancelledError
: ReservationCancelledError
Traceback (most recent call last):
File "/var/task/handler.py", line 66, in schedule_check_in
first_name, last_name, confirmation_number
File "/var/task/lib/swa.py", line 43, in from_passenger_info
method="get"
File "/var/task/lib/swa.py", line 134, in _make_request
raise exceptions.ReservationCancelledError()
lib.exceptions.ReservationCancelledError
16:34:33
END RequestId: dd3eb32c-5ef7-4abb-8c6f-348bd29e3739
16:34:33
REPORT RequestId: dd3eb32c-5ef7-4abb-8c6f-348bd29e3739	Duration: 1507.22 ms	Billed Duration: 1600 ms Memory Size: 128 MB	Max Memory Used: 37 MB

Its hitting this block in swa.py
if response.status_code == 404:
        raise exceptions.ReservationCancelledError()

@ScottBishop thanks for reporting this. It doesn't look like their API changed -- I just tested a reservation and it seems to be working okay in my case.
Are you sure this reservation exists?

I am positive the reservation exists, just bought them this morning. I tried it with 2 different reservations.
One has a middle initial: John A Doe's 12/01 San Francisco trip...
And the other has a full middle name: John Andy Doe's 12/01 San Francisco trip...
Both seeing the same error, could be related?

FWIW You'll need to be running at least https://github.com/DavidWittman/serverless-southwest-check-in/commit/d27d5d166551c0cb468edef205b46efe680746ce for the above functionality to work.

"Manual mode" worked. If I get some time I'll submit my first PR for this :)

Please do! I'll warn you that it's a bit of a difficult problem to solve because all of the different names and formats they seem to use for these emails. But you can add them to the test suite at https://github.com/DavidWittman/serverless-southwest-check-in/blob/master/lambda/tests/test_email.py#L102-L104 and just run make test and get some feedback as to whether or not things are working.
Just to add to the pile, I just booked a flight and got the following email format:
David L Jr Wittman's 11/23 San Francisco trip (ABC123): Your reservation is confirmed.

The parser issues have been fixed, but this exception is still unclear here. This is more of a ReservationNotFound error, not a ReservationCancelled error.


Localisation Select button

Hello Yusuf!
Thank you so much for your package, it's really a big deal, I found this solution recently and decided to apply it in my projects.
I am very glad that your package has localization, but I could not find a button for :select-button, is it hard-coded text or do I need to add a line in the localization file?

Here it is:
https://github.com/n1crack/vuefinder/blob/80cfd9d2ce112327d3b99e22438ba5442b0e663b/src/components/Statusbar.vue#L25
"Select": "Select"  should be added to translation files. I think we missed this one.

Oh! Thanks for the quick response, otherwise your package works fine, I integrated it into WangEditor as a file manager and it's just magic compared to other solutions.

Thx, I will add the translations asap (probably tomorrow).

It is added.


Steps Appear Incomplete After Register your wallet Process on Trailblazers Website

Describe the bug
After completing the 3rd step process on the Trailblazers website, the first and second steps (connecting your Twitter account and completing Pink & Share) still appear as incomplete. Attached is the screenshot.

Steps to reproduce
Need to Investigate by a Website Developer
Spam policy

[X] I verify that this issue is NOT SPAM and understand SPAM issues will be closed and reported to GitHub, resulting in ACCOUNT TERMINATION.

"


Add adriangonz to Kubeflow org

Hello everyone,
This PR adds myself (adriangonz) as a a member of the Kubeflow org.
I've been recently involved in contributions towards the KFServing project. For example, extending support for SKLearn and XGBoost within KFServing, by leveraging a new inference server developed by Seldon (https://github.com/seldonio/mlserver):

https://github.com/kubeflow/kfserving/pull/1196
https://github.com/kubeflow/kfserving/pull/1155

This would also unblock https://github.com/kubeflow/kfserving/pull/1235, which would allow me to assist on reviewing PRs related to the work listed above.
As sponsors, I've spoken previously with @cliveseldon and @yuzisun, who would be happy to sponsor me.
Output of pytest test_org_yaml.py:
==================================== test session starts ====================================
platform linux -- Python 3.7.8, pytest-6.1.2, py-1.9.0, pluggy-0.13.1
rootdir: /home/agm/Kubeflow/internal-acls/github-orgs
collected 1 item

test_org_yaml.py .                                                                    [100%]

===================================== 1 passed in 0.18s =====================================

/approve

/assign @bobgy

/lgtm
/approve
Welcome to Kubeflow!


Add docker requirements in the docs for  seldon model in FfDL

When image is built for seldon model using s2i, minimum required memory for docker is 8G.
Add these details in https://github.com/IBM/FfDL/blob/master/community/FfDL-Seldon/pytorch-model/README.md

In the S2I instructions, we asked the users
From the pytorch-model folder run the following s2i build. You will need to change seldonio to your Docker repo. You will need at least 8GB for your local Docker.

Perhaps we should make it more clear. (Bold the hard requirements). Thank you for pointing this out.


Account Signup and Community Join

As a new user,
I want to sign up using my email and confirm it through a magic link,
So that I can access the Tailored Twitter Tweeter feature and optionally join the community on Discord.
Acceptance Criteria:

User can sign up with an email.
User receives a magic link to confirm email.
User can confirm email and proceed to join Discord optionally.

https://www.notion.so/volumefi/EPIC-06-22-24-Tailored-Twitter-Tweeter-V002-4784202cc6da4485aaa038f4dfa0091e?pvs=4#622ac247297b48ea95c31e6d237a9253
https://www.figma.com/design/fO2IMDgKqgwO9kqlfmDN1s/Tailored?node-id=1128-1902&m=dev

@joycehermosilla Please share the design link.

Closed!


Update README.md

the table you have committed is like this, it's not formatted correctly, you can view the preview in the github, and in my opinion its not a good Idea to put available sources on readme


feat(HP): Frame View HP

Context
Source of idea: https://santesoft.com/win/sante-dicom-viewer-lite/sante-dicom-viewer-lite.html

needs an icon if we want this in the layout selector

@IbrahimCSAE protocolMatchingRules is undefined. so, When using segmentation mode, the following code generates an error.
  findMatchByStudy(study, options) {
    const matched = [];

    this.protocols.forEach(protocol => {
      // Clone the protocol's protocolMatchingRules array
      // We clone it so that we don't accidentally add the
      // numberOfPriorsReferenced rule to the Protocol itself.
      console.log(protocol);
      let rules = protocol.protocolMatchingRules.slice(); // FAILED


Create a Variety of Simple Web3 Protocols/Apps

Based off of this repo: https://github.com/m1guelpf/lil-web3 , implement huff contract examples for each concept. These concepts include but may not be limited to DIY implementations of:

ENS
OpenSea
Fractional
Juicebox
Gnosis Safe
Flashloans
Superfluid

This should be a language-external repository, closing for now.


Additional refactoring for caching.

Builds on the ideas introduced in #1100
@DanielRose can you have a look at the additional refactorings and let me know if you have any further suggestions.

 Would be interesting to see.

// @asbjornu

@JakeGinnivan: We should probably do a little profile. I don't have time at the moment, but perhaps @cvpcs or @DanielRose can help out with that?

@asbjornu @JakeGinnivan I'll try running through a profiler tomorrow.


error at 'Fab' demo at demo site

Describe the bug
Error at demo site .
To Reproduce
Steps to reproduce the behaviour:

Go to 'https://docs.nativebase.io/FAB#basic'
you will see error instead of demo component

Thanks for reporting. It has been fixed in the new version :)
https://docs.nativebase.io/FAB#basic


in a JCache Read-Through calling Cache.getAll(...)  doesn't call CacheLoader.loadAll(...), is this a normal behaviour?

Hi,
I am using JCache with Redisson, I configured the cache as Read-Through and setup a CacheLoader.
What I noted is that calling Cache.getAll(...) doesn't call the CacheLoader.loadAll(...) method to load values, is still use the CacheLoader.load(...).
Is this the expected behaviour?
Thanks for the help.

Just to add a note, on the specification page of JCache JSR107 (JCache) on the Google doc linked there about specification, at the table on page 65 (the Read-Through Caching section) it's written:
Map<K,V> getAll(Collection<? extends K> keys)         Yes. Invokes loadAll()

Fixed! Thanks for report


To Table Does Not Exist In Modern Power BI Desktop

On the section that says to create a blank query, add the "= DateTime.LocalNow" function and then click the To Table button, that button does not exist under Transform in Power Query in the newer versions of Power BI Desktop anymore.  I'm not sure what to do to make it work at this time.

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

ID: 35ad9d59-d3df-925d-f613-dd07270c72c4
Version Independent ID: 681464d8-048e-6d74-f088-4f726fcb55a5
Content: Add last refresh date to a Power BI report - Azure DevOps
Content Source: docs/report/powerbi/add-last-refresh-time.md
Service: azure-devops
Sub-service: azure-devops-analytics
GitHub Login: @chcomley
Microsoft Alias: chcomley

Ah, I see.  What needs to be added to the page is to click the "Invoke" button, then it will let you click the To Table button.

Thanks for the feedback, @jacobgexigo, I've updated the document. Have a great day.


6.4.x Branch selector fix

Bug 1313094 - If one user changes a branch, the branch is changed for all the users
Bug 1313095 - Repository listing and File explorer in Adminstration perspective were often out of sync

(cherry picked from commit 8b312c812ccce5b1efa401c40b21a40cd9ec0151)
https://github.com/droolsjbpm/guvnor/pull/271
https://github.com/droolsjbpm/kie-wb-common/pull/284
https://github.com/droolsjbpm/drools-wb/pull/136
https://github.com/droolsjbpm/jbpm-console-ng/pull/350
https://github.com/droolsjbpm/jbpm-designer/pull/171
https://github.com/droolsjbpm/jbpm-form-modeler/pull/41
https://github.com/droolsjbpm/optaplanner-wb/pull/41
https://github.com/droolsjbpm/kie-wb-distributions/pull/217

6.4.x: 2efa95960dcd1e7df47dfa1296d225aeaa54f01e


Support applying annotations on aliases and annotations

Resolves #10125.
This PR allows annotations to be applied to aliases and annotation defs.  I chose not to add new macro methods to access them as the main goal of this PR is to support the built in @[Deprecated] annotation, not custom ones.  I also refactored the warning logic to reduce the duplication given two new types of nodes were added.
One thing I'm not 100% on is how to display the FQN of the alias/annotation like methods/macros do.  From what I can tell an owner property would need to be added to both aliases and annotations.

What does it mean to put an annotation on an alias? Since it's an alias, I would expect the annotation to go into the aliased types. After all, aliases just point to the same type: they are not really a different type.

What does it mean to put an annotation on an alias? Since it's an alias, I would expect the annotation to go into the aliased types. After all, aliases just point to the same type: they are not really a different type.

The annotations should apply to alias declaractions. Allowing annotations to apply to the aliased types would have messy implications such as being able to annotate generic instances, something not currently possible.
C++ allows the [[deprecated]] attribute on typedefs and using type aliases for example:
struct A { };

[[deprecated]] typedef A B;
using C [[deprecated("use A instead")]] = A;

int main() {
	A x; // okay
	B y; // deprecated
	C z; // deprecated; use A instead
}

(For that matter, C++ does allow deprecation of template specializations too.)

The annotations should apply to alias declaractions. Allowing annotations to apply to the aliased types would have messy implications such as being able to annotate generic instances, something not currently possible.
C++ allows the [[deprecated]] attribute on typedefs and using type aliases for example:
struct A { };

[[deprecated]] typedef A B;
using C [[deprecated("use A instead")]] = A;

int main() {
	A x; // okay
	B y; // deprecated
	C z; // deprecated; use A instead
}

(For that matter, C++ does allow deprecation of template specializations too.)

@asterite A use case I had for this was to help with deprecation of another type.  For example:
annoation SomeAnn; end

@[SomeAnn]
def foo; end

If you wanted to move the annotation to another namespace, you could alias the old annotation name so things keep working, while letting people know that alias will go away in the future.
annotation SomeNamespace::SomeAnn

@[Deprecated]
alias SomeAnn = SomeNamespace::SomeAnn

@[SomeAnn]
def foo; end

foo # => Warning, using deprecated alias SomeAnn.

@asterite A use case I had for this was to help with deprecation of another type.  For example:
annoation SomeAnn; end

@[SomeAnn]
def foo; end

If you wanted to move the annotation to another namespace, you could alias the old annotation name so things keep working, while letting people know that alias will go away in the future.
annotation SomeNamespace::SomeAnn

@[Deprecated]
alias SomeAnn = SomeNamespace::SomeAnn

@[SomeAnn]
def foo; end

foo # => Warning, using deprecated alias SomeAnn.

Update on this.  I think it's in a functional state, but I'm not sure how to go about adding support for namespaced aliases/annotations.  The other types have an owner property that can be used to determine where the related type "lives".  However that concept was never implemented for alias or annotation types; thus you get like Warning: Deprecated alias OtherType. versus Warning: Deprecated alias SomeNamespace::OtherType.
At this point if anyone has thoughts on how to go about adding that (or wants to take over the PR) I would appreciate it.

Both AnnotationType and AliasType inherit from NamedType so there should be #namespace and #full_name already. You shouldn't need an owner property.

This should be good for review now; rebased on master and finished support for namespaced aliases and annotations.  Big thanks to @HertzDevil.