opae.io: add support for dfh v1

Add dfh1 register struct and change the dfh_walk routine to:

default header parameter to None
parse the header as a dfhv1, then change it to:

header class passed in as an argument
dfh0 if version is 0

Signed-off-by: Rodrigo Rojo<EMAIL_ADDRESS>

Coverage remained the same at 64.938% when pulling 2616812da91487a4086b7a9a706e160fd6fc73fe on rrojo/opae.io-dfh1 into 5fdb142df0425b7d452cde460cea2d507b6d2a46 on master.


textarea overflows when "border-radius:1rem"

Summary
When i scroll down a page past a textarea, it's contents breaks out and spills over downwards.
This seems to happen exactly when the textarea leaves my screen.
Operating system
Linux
Steps to reproduce

Use reduced example
Scroll down.
See text overflow

Expected behavior
Text shouldn't escape it's textarea
Actual behavior
Text escapes and draws over the remaining content.
URL for a reduced test case
https://www.useragents.me/
HTML/SVG/etc. source for a reduced test case
<!DOCTYPE html>

<textarea style="border-radius:1rem;">
01) Lorem impsum.
02) Lorem impsum.
03) Lorem impsum.
04) Lorem impsum.
05) Lorem impsum.
06) Lorem impsum.
07) Lorem impsum.
08) Lorem impsum.
09) Lorem impsum.
10) Lorem impsum.
</textarea>

bottom

Log output and (if possible) backtrace
dodo@dodo:~/ladybird/ladybird$ ./Meta/ladybird.sh run
ninja: Entering directory `/home/dodo/ladybird/ladybird/Build/ladybird'
[0/2] Re-checking globbed directories...
ninja: no work to do.
QStandardPaths: wrong permissions on runtime directory /run/user/1000/, 0755 instead of 0700
Ladybird PID file '/run/user/1000/Ladybird.pid' exists with PID 676, but process cannot be found
qt.qpa.wayland: Wayland does not support QWindow::requestActivate()

Screenshots or screen recordings

Build flags or config settings
No response
Contribute a patch?

[ ] I’ll contribute a patch for this myself.

At a guess, it sounds like we're skipping the masking operation when the mask is outside the viewport?

Playing around with this, it's not specific to <textarea>, but applies to any situation where:

Parent element has border radius and scrollable overflow
Child element has overflow: visible which is the default
eg, this modified test case shows the same issue:

<!DOCTYPE html>

01) Lorem impsum.

02) Lorem impsum.

03) Lorem impsum.

04) Lorem impsum.

05) Lorem impsum.

06) Lorem impsum.

07) Lorem impsum.

08) Lorem impsum.

09) Lorem impsum.

10) Lorem impsum.

bottom

Also, removing the continue here fixes the issue:
https://github.com/LadybirdBrowser/ladybird/blob/93712b24bf298a61d3a614ce57574739015776c2/Libraries/LibWeb/Painting/DisplayList.cpp#L61-L64
I'll see if I can fix this properly.


Cleanups

Codecov Report

Merging #26 (3ab0dd1) into main (d1b8c04) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #26   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines           38        38
  Branches         4         5    +1
=========================================
  Hits            38        38

Continue to review full report at Codecov.

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


exit / finish emits required

Hi, there is an emit for "exit". But why not for "finish"?
As example: When I click the "X" button on top right of my onboarding, it executes the @exit=".." emit. But when the last step has been reached, there is a "Finish" button which executes as well the @exit=".." emit. But it should emit a @finish=".." function.
The reason for this is, I'm trying to seperate a cancel / finish function to save the state in my database. Now I have to write a complex condition to check if it's finished or has been exited manually by user. :/
Thanks!

Hey @DominikGanic, you are right. I have some other bug fixes as well, I'll add this to my list. They should be released this weekend

I've added a comment at this commit:
https://github.com/fatihsolhan/v-onboarding/commit/ecb32258a5b826a66ceb6006438be9cbe1f2c12e
Looks like the following line there causes problems, since I've updated the package today.
index.2a3644af.js:1 DOMException: Failed to execute 'setAttribute' on 'Element': 'finish,' is not a valid attribute name.

Thanks mate! Happens to the best ;)!


[Accessibility]A11y_AzureDataStudio_CSVSelectDestinationDatbse_ScreenReader:Screen reader is announcing order of the list item incorrectly for “Schema for new external table” combo box.

Check out Keros! - Identify accessibility bugs before check-in and make bug fixing faster and easier.
GitHubTags: #A11yMAS; #A11yTCS; #MAS1.3.1; #A11y_AzureDataStudio; #Win32; #DesktopApp
Environment Details:
Application name: Azure Data Studio
Version: 1.8.0-insider (user setup)
Electron: 3.1.8
Node.js: 10.2.0
V8: 6.6.346.32
OS: Windows_NT x64 10.0.17763
NVDA Version:2019.1
JAWS version : 2019.1810.64 ILM
Additional Details:
Tool Name: NVDA
MAS Reference:MAS1.3.1
Steps to Reproduce:

Install the app Azure Data Studio.
Connect to database->navigate to the Data services folder->HDFS Folder-> navigate to CSV files folder->Right click on the files->Select option Create external table from CSV files.
Navigate till  “Schema for new external table” combo box .
Observe that screen reader is announcing the order of list item incorrectly.

Actual:
Navigate till  for “Schema for new external table” combo box , here screen reader is announcing the order of list item incorrectly. It reads  list item as “dbo 1of 3 level 1 ” but there are only two item present in the list.
Similar issue also observed with JAWS
Expected:
Screen reader should  read  list item as “ dbo 1of 2 level 1 ” for “Schema for new external table” combo box.
Recommendation:
Please refer the following snippet link for fixes:
https://microsoft.sharepoint.com/teams/msenable/mas/pages/browse-fixes.aspx
User Impact:
Screen reader users find it difficult to understand the number of item present in the list .
MAS Reference:
MAS1.3.1-
https://microsoft.sharepoint.com/:w:/r/teams/msenable/_layouts/15/WopiFrame.aspx?sourcedoc={54f28d1f-a2d1-4dcd-84e1-5c9b87e8aba4}
Attachment for Reference:
5969_[Accessibility]A11y_AzureDataStudio_CSVSelectDestinationDatbse_ScreenReader.pptx
Does this issue occur when all extensions are disabled?: Yes

Possibly related to #5971

I've noticed some issues with editable dropdowns where there will be blank entries at the bottom. This may be related.

Reasonably sure this is due to those extra blank entries, so this is a dropdown problem, rather than an accessibility one.


Access to page/item properties from the context object

Category

[x] Question

If I view the HTML source of a modern page I see page context info for properties as user and item assigned to an internal variable spClientSidePageContext which is passed into SPFx bootstrapping. This object also has a property spPageContextInfo which is made available via this.context.pageContext in SPFx.
Are there plans or any chance we can access the user and item property as well, to avoid executing an extra HTTP call to fetch this information? Seems a waste of resources to re-fetch something which is already present on the page :) #lazyprogrammer

Hey Mikael - it's unlikely that we'll expose the user, as pretty much all of the user information is available in the page context.  Let me know if there is a property that isn't.
We're thinking about the item, and how to handle it.  Getting into more detail than you probably want to hear - the spClientSidePageContext is something delivered to the app, and then the app surfaces some of that data through the API.  The spPageContextInfo is something that everything in SharePoint understands and can produce, and so we can expose it in a general way through all the scenarios (classic, modern, modern-list, workbench, etc.).  The item is trickier to handle.  In an ideal world, I would want to see a similar schema / API / etc. between the current item when you are on a page, and the current item you have selected from a doclib webpart (via the dynamic data API).  The DD json format is still being worked on though.
Please post this to uservoice though, which is where the general set of Feature Requests go.


Connection is not closed when used with oracle.

Describe the bug(Please use English)
A clear and concise description of what the bug is.
Environment :

Flink version :  1.16.0
Flink CDC version: 2.3
Database and version: Oracle 19g

To Reproduce
Steps to reproduce the behavior:

The test data : I connect flink over flink-cdc with oracle. The number of connections is more or less 180 for 760.000 records of data.
The test code : select count(1) from gv$session where machine =
The error : regardless if the job is idle or cancelled those connections are not closed at any point of time, nor can i find any config option to reduce the number

How did you solve it？

I deactivated scan.incremental.snapshot.enabled , at least for my version of oracle this caused open connections from the initial load.

Did you have a problem with the fact that after completing the task, the session to the database did not end ?

Yes, and based on the chunk size, a huge amount of connections were left open.

When deactivating this parameter 'scan.incremental.snapshot.enabled'='false', the session does not close when the job is completed ? (AFAIK, Flink does not control the closing of the session to the database)


updated next to v13

What does this PR do?
Updated next.js to v13
Type of change

[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

How should this be tested?

[ ] Existing unit tests

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code and corrected any misspellings
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Closed in favour of https://github.com/boxyhq/jackson/pull/634


Convert ddev to use mariadb image instead of mysql image

What happened (or feature request):

Change request.

What you expected to happen:

See PR here https://github.com/drud/docker.mysql-local/pull/27#issuecomment-330607039.
Essentially, given the mariadb container has both feature parity with the mysql container and it utilizes a more preferential license, we should swap.
Not urgent, but something worth addressing sooner than later if it's not a complicated lift.

Are we just using the stock mariadb image? Or should the drud/docker.mysql-local image be completely rebuilt on top of mariadb?

We've made some customization over time to the mysql image, and we'll need to be sure those changes are present in the maria replacement image. Off-hand I'm aware of custom my.cnf, healthcheck script, and I believe we've altered the entrypoint script. I'm sure there are others as well though.

So to add to Tanner's response: Just modify docker.mysql-local to use the official mariadb image as upstream, making whatever changes are required to do that.


Fabo/disable token sending on hub

Closes #2360
Description:

Disable token sending on the hub and show a tooltip
Thank you! 🚀

For contributor:

[ ] Added entries in PENDING.md with issue # and GitHub username
[ ] Reviewed Files changed in the github PR explorer
[ ] Attach screenshots of the UI components on the PR description (if applicable)
[ ] Scope of work approved for big PRs

For reviewer:

[ ] Manually tested the changes on the UI

screenshot

screenshot

don't have my ledger with me :/


Any way to get indent guides (like in VS or Atom)?

Would it be possible to show / get indentation guides like in other editors?
https://atom.io/packages/indent-guide-improved

Is the monaco-editor used in ALM taken from the Visual Studio Code (the full source) or from the Monaco Editor site (https://microsoft.github.io/monaco-editor/index.html) and would it be hard to update?

Have you done a lot of changes so it would be a big task (for someone / me) to update it to the newest

Closing as this project is going in take as is mode :rose:


use ninja --version instead of ninja -v

Describe the pull request
When mingw is on the path,  ninja accepts --version as a command-line argument, but it doesn't accept -v. This is the actual issue:
https://github.com/ninja-build/ninja/issues/1463
> ninja -v
ninja: error: loading 'build.ninja': The system cannot find the file specified.

> ninja --version
1.10.2

Based on the source code, -v means --verbose for ninja.
https://github.com/ninja-build/ninja/blob/7905dee5ac62f7a1e0dfec4d936b97d96c7566d7/src/ninja.cc#L223
I think wherever ninja -v is called should be replaced with ninja --version.

What does your PR fix?
Fixes #17110

Which triplets are supported/not supported? Have you updated the CI baseline?
all

Does your PR follow the maintainer guide?

N/A
If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

I misunderstood the intention here. When ninja is called without the target name it builds the default target. I thought you wanted to check that ninja works by spawning it.

"if targets are unspecified, builds the 'default' target (see manual).\n"


Upgrade to Laravel 8.

What's this PR do?
This pull request updates Northstar to Laravel 8 following the upgrade guide.
How should this be reviewed?
👀
Any background context you want to provide?
...
Relevant tickets
References Pivotal #178201843.
Checklist

[x] This PR has been added to the relevant Pivotal card.
[ ] Documentation added for new features/changed endpoints.
[x] Added appropriate feature/unit tests.

Fixed up the failing northstar:setup step on CircleCI builds by installing<EMAIL_ADDRESS>in 758a741 & then fixed a failing test from the tweaked language files in 703d92e. Builds are now passing! 💚
Last step – wrap up & publish next Gateway release.


Performance of beta 1.9 beta 3 much slower than previous public release

Steps to reproduce issue (Detailed enough for me to reproduce):
When using the pre-built binary on macOS High Sierra, the current 1.9.3 beta version is significantly slower at compressing and de-compressing than the previous version I was using (from many months ago).
What you expect to happen:
I expected the performance to be on-par at least.
What actually happened:
It was significantly slower.
For example, a 12.5 MB  file took less than a second to compress on the older version, but it took about 15 seconds on the latest beta version.
The newest version does write out a bunch of logging information that wasn't in the previous version:
[General] Calculated new CRC:<PHONE_NUMBER> from old CRC:<PHONE_NUMBER>
[General] Writing 1024 bytes into archive from buffer
[General] Calculated new CRC:<PHONE_NUMBER> from old CRC:<PHONE_NUMBER>
[General] Writing 1024 bytes into archive from buffer
[General] Calculated new CRC:<PHONE_NUMBER> from old CRC:<PHONE_NUMBER>
[General] Writing 1024 bytes into archive from buffer

Could it be calculating the new CRC after every write is slowing it down?

It’s not doing anything with the CRC that it wasn’t before. What method are you calling, and is it possible to include the archive (or email it to me?)

I'm using the following bit of code to write out an archive:
+ (BOOL)zipItemAtURL:(NSURL *)itemURL archiveURL:(NSURL *)archiveURL progress:(NSProgress *)compressProgress completion:(nullable void (^)(NSProgress * __nullable progress))completionHandler error:(NSError **)error {

	NSError *localError = nil;

	BOOL success = NO;

	UZKArchive *archive = [[UZKArchive alloc] initWithURL:archiveURL
													error:&localError];

	NSFileManager *fileManager = [[NSFileManager alloc] init];

	NSArray *subPaths = [fileManager subpathsOfDirectoryAtPath:itemURL.path
													  error:&localError];

	NSProgress *progress = [[NSProgress alloc] init];
	progress.totalUnitCount = subPaths.count;
	progress.completedUnitCount = 0;

	for (NSString *subPath in subPaths) {

		if (compressProgress.isCancelled) {
			if (completionHandler) {
				completionHandler(progress);
			}
			return YES;
		}

		NSString *filePath = [[itemURL path] stringByAppendingPathComponent:subPath];

		NSURL *fileURL = [NSURL fileURLWithPath:filePath];

		NSString *filePathInZip = [[itemURL lastPathComponent] stringByAppendingPathComponent:subPath];

		NSNumber *isDirectory = nil;
		[fileURL getResourceValue:&isDirectory forKey:NSURLIsDirectoryKey error:NULL];

		NSDate *modificationDate = nil;
		[fileURL getResourceValue:&modificationDate forKey:NSURLAttributeModificationDateKey error:&localError];

		// skip directories since the directory will effectively be in the archive
		// when we add a real file to it with a subPath that contains the directory path.

		if (![isDirectory boolValue]) {

			NSData *fileData = [NSData dataWithContentsOfURL:fileURL
													 options:NSDataReadingUncached | NSDataReadingMappedIfSafe
													   error:&localError];
			const void *bytes = fileData.bytes;
			unsigned int bufferSize = 1024; //Arbitrary
			NSError *writeError = nil;

			success = [archive writeIntoBuffer:filePathInZip
					fileDate:modificationDate
					error:&writeError
					block:
			^BOOL(BOOL(^writeData)(const void *bytes, unsigned int length), NSError**(actionError)) {
			     for (NSUInteger i = 0; i <= fileData.length; i += bufferSize) {
			          unsigned int length = (unsigned int)MIN(fileData.length - i, bufferSize);
			          if (!writeData(&bytes[i], length)) {
			                 return NO;
			          }
			     }

			 return YES;
			}];

			if (success) {
				progress.completedUnitCount += 1;
				progress.localizedAdditionalDescription = [filePath lastPathComponent];
				if (completionHandler) {
					completionHandler(progress);
				}
			} else {
				TFFLog(@"Error writing zip file: %@", writeError);
			}
		}
	}

	*error = localError;

	return success;
}

And for extracting it seems I'm using a deprecated API:
		success = [oldArchive extractFilesTo:NSTemporaryDirectory()
								   overwrite:YES
									progress:^(UZKFileInfo * _Nonnull currentFile, CGFloat percentArchiveDecompressed) {
										dispatch_async(dispatch_get_main_queue(), ^{
											float percent = percentArchiveDecompressed * 100;
											progress.completedUnitCount = percent;
											progress.localizedAdditionalDescription = [currentFile.filename lastPathComponent];
										});
									} error:&error];

I'll work on converting it to the recommended API.

I deprecated that method because you don't need a progress block anymore - I support NSProgress and NSProgressReporting, which is what it looks like you're already using. You can use it implicitly or explicitly, with a guide on the README for the 1.9 branch.
To help me reproduce your issue, since I don't see anything blatantly wrong with your code, can you provide an example file you're compressing or decompressing that's slower?
Out of curiosity, have you run in Instruments to profile where it's spending its time?

Sure, here's a file that I compressed:
https://www.dropbox.com/s/ddtgoajuj5iwg2k/Sample.tapforms-2017-11-09T19-12-36.zip?dl=0
It's 12.4 MB compressed.
It's 13.4 MB uncompressed.
With the older library it took 1.073878 seconds. With the new version it took 97.259650 seconds to compress the same document file. So almost a hundred times slower.
However, when I ran it in Instruments, it was about the same speed as the previous version. So maybe the issue is all the logging it's doing within Xcode. I tried an archive build of my app too and that also performed well.
So it really must just be the logging causing the performance problem from Xcode then. How do I turn off the logging? Is there an environment variable I can use?

Take a look at the v1.9 readme, and Apple's documentation

Doh! Now I feel silly. Yup. Setting the log level to default solved the problem. I'm just used to having to set arguments in the scheme settings to control logging.
I'll close this issue now. Thanks.


Starting the Hebrew translation - waiting to be transferred to the website to continue

I'm create https://hosted.weblate.org/ account, and try to transfer the content of the need to be translated to the site.
Thanks for your Github PR and suggestions


E2e tests improvements

[ ] Add phone number verification
[ ] Cover less used features (exchange, payment request, invite)

Closing since it's an epic from a previous milestone


Consider using luabidi

https://github.com/deepakjois/luabidi is a pure Lua implementation of the Unicode Bidirectional Algorithm, and should be a more conformant implementation than the one we are currently using.

It looks like the author of that Lua library @deepakjois has previously contributed to SILE, but the Bidi library is a new project and didn't exist when Simon was hacking on this. Using a standardized and conformant implementation of this  would certainly be a good thing. I know it was discussed that we're currently mostly doing the right thing, but using a full implementation of the spec instead of hacks that cover current known cases can only be a good thing.
This discussion should go along with the one about using ICU directly (#173).

Although luabidi itself is pure Lua, one of its dependencies luaucdn, is a Lua binding around ucdn, which is a C library. So now I am wondering if it is fair to claim that luabidi is a pure Lua implementation of the BiDi algorithm.
All the dependencies are installabale via LuaRocks though.

...at which point it may be better to use ICU.

ICU would be my first choice of course, but we can abstract the luacdn dependency if needed since we already have a stripped down UCD of sorts for the current bidi implementation.

We're already at the point where SILE gives differing input when you have ICU and when you don't. I'm now toying with the idea of making ICU a hard requirement, in which case, handing off bidi to ICU becomes more desirable. Any reasons why I shouldn't?

ICU should be fine, its implementation is up to date and should be conformant, LibreOffice/OpenOffice, Java use it and I think Firefox is switching to it as well, so we should be in good company.

We now use ICU for bidi - at least for getting the levels. Reordering is still done internally because ICU only works on text, and we need to maintain node information.


Chat shows message contents when screen is locked

When the MacOS Keybase client receives an incoming chat message, it sends a notification through Apple's desktop notification system, which causes a box/bubble to appear on the screen, even if the screen saver is active and the screen is locked.
The problem is, this notification contains the content of the received chat message. If I'm chatting with somebody about something which needs to remain private, and I lock the screen while I step away from my desk, anybody who happens to walk by my desk and glance at the screen while I'm away can read those incoming chat messages.
MacOS - Keybase 1.0.23-20170522181119+8a8aea0

Also, I just noticed the Copyright message at the bottom of the "About" window says "2015" rather than "2015,2016,2017" (i.e. every year during which changes were made to the code.)

The problem is that the notifications contain the actual text of the message that somebody sent to me.

Yeah that's a problem.  I can see that we can disable "mobile plaintext notifications", but can we also disable "desktop plaintext notifications"

+1 -- I've had a chat message flash up before when showing something on my screen to a coworker that I really did not want him to see and I do not see how to turn that off.
Ideally I'd love to see it the way slack does it -- Options for show notification with content, show notification  with just name of sender, and don't show notification.

Is this the official channel for leaving feedback / asking for feature requests? Honest question, first timer with Keybase, and I also want this feature to exist.

@brokenthumbs This is the official repository for the Keybase client's code, and assuming they're using Github the way it was designed, this "issue" is on their list of things to look at. It doesn't look like anybody above is one of the developers so it's possible they haven't seen it yet, but I'm sure at some point they will.
For now I've just turned notifications off on the work machine, since I also get notifications on my phone and iPad. It's not ideal, but it does keep message content from unexpectedly showing up on the screen (or in whatever logs Apple may keep for their notification system.)

Was just about to create this ticket myself. The worst part about this is that it is on by default and has no settings to disable within the app. Not just hiding message contents, but even disabling desktop notifications at all.
It looks like it's been 5 months since this issue was created -- should the maintainers be pinged somewhere else?

It looks like issues should be reported here:
https://github.com/keybase/keybase-issues

^ Other issue created at that repo, thanks @jeremysears!

closed via #17337


[More...] tags are appearing on every message rendered in the background

I've noticed as of Zulip 1.6.0 [More...]tags are appearing on every message that's beed loaded offscreen while Zulip was in the background.
Steps to reproduce:

open https://chat.zulip.org
scroll down to the bottom of Home
leave it running in a background tab for 10+ min and let new messages accumulate without interacting with the tab
come back to zulip, and scroll down
all (almost all?) new messages that were loaded below the scroll point will have [More...] tags, despite being short messages

Clicking the [Condense this message] confirms that the expanded and condensed versions are same height & same content.
Confirmed in Chrome >=v59 & Electron consistently for me, and Firefox occasionally for Reid.
The Fix:
This likely has to do with how zulip computes the height of messages by rendering them first, then getting the height off the rendered html element.
If you can find that code, just run a test using setTimeout to make sure it's reporting non-zero heights when rendering in the background in Chrome and Firefox.
If it does report 0px heights when background rendering, check to make sure it's using getBoundingClientRect and not outerWidth/innerWidth, which have issues with 0px reporting.
Related links:

further speculation in the zulip thread
semi-related ticket https://github.com/zulip/zulip/issues/3819 about the library for estimating height of zulip messages without rendering

Dropping the priority since we haven't heard other reports of this issue in a month.  I doubt the root cause has been fixed, though.

It's still happening for me, both on the bleeding-edge chat.zulip.com and my hosted 1.6 server.

(Adding conversation link
https://chat.zulip.org/#narrow/stream/6-frontend/topic/more.20option.20on.20every.20message)

I just confirmed this is still happening occasionally as of 1.7.0.

@pirate yep, we just figured out the root cause of this, and it's now fixed in master.  Should be in the 1.8 release, coming out soon (we'll definitely have a release candidate this week).


added TrackMateR

TrackMateR (R package for crunching TrackMate data) added to Cell migration and particle tracking section

Thank you @quantixed.


Get hand_tracking_cpu source code to extract detected hand metadata

By running this code below, I am able to get the demo app to run but, where can I get the source code so that I can get the hand metadata regarding finger positions and joints?
GLOG_logtostderr=1 bazel-bin/mediapipe/examples/desktop/hand_tracking/hand_tracking_cpu \
    --calculator_graph_config_file=mediapipe/graphs/hand_tracking/hand_tracking_desktop_live.pbtxt

I will like to modify: bazel-bin/mediapipe/examples/desktop/hand_tracking/hand_tracking_cpu so I can output a file with hand metadata for sign language analysis. But this seems to be a binary file. How can I use this or integrate this with a new app?

You are probably looking for

mediapipe/examples/desktop/demo_run_graph_main.cc

You are probably looking for
mediapipe/examples/desktop/demo_run_graph_main.cc

hi @MAkcanca
Thank you for your response. I am new to this, so bear with me here for a moment. So I need to put to use the multi_hand_graph (/mediapipe/blob/master/mediapipe/graphs/hand_tracking/multi_hand_tracking_desktop.pbtxt a)
so I guess to run this it will be something like...
demo_run_graph_main.cc --calculator_graph_config_file=multi_hand_tracking_desktop.pbtxt --input_video_path=myVideo.pm4 --output_video_path=something.mp4
Thank you sir

demo_run_graph_main.cc is a Code file. You can't directly run it without compiling it first.
Try checking out the documents for details on how to compile and run: https://mediapipe.readthedocs.io/en/latest/multi_hand_tracking_desktop.html

So i did:
cc mediapipeline
bazel run --define MEDIAPIPE_DISABLE_GPU=1 mediapipe/examples/desktop:demo_run_graph_main
and got:
ERROR: Cannot run target //mediapipe/examples/desktop:demo_run_graph_main: Not executable
any idea on why bazel dont like it?

I was able to build it using bazel build :demo_run_graph_main  directly from the examples/desktop folder

now i need to obtiain the hand segments and nodes from open CV

@research506
Please look at https://mediapipe.readthedocs.io/en/latest/multi_hand_tracking_desktop.html for how to build and run multi-hand tracking example on desktop.
On desktop, you can add an output stream poller to demo_run_graph_main.cc or demo_run_graph_main_gpu.cc, such as
ASSIGN_OR_RETURN(mediapipe::OutputStreamPoller multi_hand_landmarks_poller,
                   graph.AddOutputStreamPoller("multi_hand_landmarks"));

to listen for output packets on a particular output stream, such as "multi_hand_landamrks" above.
See https://mediapipe.readthedocs.io/en/latest/hello_world_desktop.html for example.
Then, on every frame, check if a packet is available in the output stream:
mediapipe::Packet multi_hand_landmarks_packet;
if (!multi_hand_landmarks_poller.Next(&multi_hand_landmarks_packet)) break;
const auto& multi_hand_landmarks = multi_hand_landmarks_packet.Get<std::vector<std::vector<mediapipe::NormalizedLandmark>>>();

Note that you need a dependency on
#include "mediapipe/framework/formats/landmark.pb.h"

in the file you are modifying (and modify the corresponding rule in BUILD file as well).
And then you should be able to do what you want with these landmarks, such as print them:
int hand_index = 0;
for (const auto& hand_landmarks : multi_hand_landmarks) {
  int landmark_index = 0;
  for (const auto& landmark : hand_landmark) {
    std::cout << "[Hand<" << hand_index << ">] Landmark<" << landmark_index++ << ">: (" << landmark.x() << ", " << landmark.y() << ", " << landmark.z() << ")\n";
  }
 std::cout << "\n";
 ++hand_index;
}

@research506
Please look at https://mediapipe.readthedocs.io/en/latest/multi_hand_tracking_desktop.html for how to build and run multi-hand tracking example on desktop.
On desktop, you can add an output stream poller to demo_run_graph_main.cc or demo_run_graph_main_gpu.cc, such as
ASSIGN_OR_RETURN(mediapipe::OutputStreamPoller multi_hand_landmarks_poller,
                   graph.AddOutputStreamPoller("multi_hand_landmarks"));

to listen for output packets on a particular output stream, such as "multi_hand_landamrks" above.
See https://mediapipe.readthedocs.io/en/latest/hello_world_desktop.html for example.
Then, on every frame, check if a packet is available in the output stream:
mediapipe::Packet multi_hand_landmarks_packet;
if (!multi_hand_landmarks_poller.Next(&multi_hand_landmarks_packet)) break;
const auto& multi_hand_landmarks = multi_hand_landmarks_packet.Get<std::vector<std::vector<mediapipe::NormalizedLandmark>>>();

Note that you need a dependency on
#include "mediapipe/framework/formats/landmark.pb.h"

in the file you are modifying (and modify the corresponding rule in BUILD file as well).
And then you should be able to do what you want with these landmarks, such as print them:
int hand_index = 0;
for (const auto& hand_landmarks : multi_hand_landmarks) {
  int landmark_index = 0;
  for (const auto& landmark : hand_landmark) {
    std::cout << "[Hand<" << hand_index << ">] Landmark<" << landmark_index++ << ">: (" << landmark.x() << ", " << landmark.y() << ", " << landmark.z() << ")\n";
  }
 std::cout << "\n";
 ++hand_index;
}

My brother, this was the droid I was looking for. I will update this thread with a link to my repo where the code can be found. I owe you a beer

@eknight7 could you please explain

in the file you are modifying (and modify the corresponding rule in BUILD file as well).

I added the line "//mediapipe/framework/formats:landmark_proto", to the "demo_run_graph_main" cc_library in the mediapipe/examples/desktop/BUILD file but I get the following error at build time:
rule '//mediapipe/framework/formats:landmark_proto' is misplaced here (expected cc_library, objc_library, cc_proto_library or cc_import) and '//mediapipe/framework/formats:landmark_proto' does not have mandatory providers: 'CcInfo'
I'm a total neophyte in c++ and I think the error comes from the fact that I also have to update the srcs parameter, but I tried every way I could think of to add the mediapipe/framework/formats/landmark.proto to srcs and nothing worked. Any help would be appreciated.

Link to updated gist for example: https://gist.github.com/eknight7/d4a57504c8f866fc80c0eb2c61ff6b4f


SAK-29931 Solve fail in pack task.

After SAK-29896 we need to fix some paths inside ant build for packaging.

LGTM


Anway to ask it what it knows?

Feature request
I think it would be neat if there was a way I could ask it what it Knows. Is there anyway I could do this, I assume i would need a way to put the data in the DB dyamically of the pages it knows?

Yeah, your best bet is probably to add this to the initial prompt/instructions given to GPT. If this info is dynamic for your use case, you would need to dynamically generate it from your DB.


Rancher Catalog stops parsing templates if yml syntax error is encountered

Rancher Version: 1.0.1
Docker Version: 1.10
OS and where are the hosts located? (cloud, bare metal, etc): AWS/Ubuntu 14.04
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB) Single Node
Environment Type: (Cattle/Kubernetes/Swarm/Mesos) Cattle
Steps to Reproduce:
Create a series of catalog template versions in the same catalog entry.  (say test/1 test/2 & test /3.
Put a tab or some kind of invalid yml in test/2
Refresh the catalog, select test, and click the drop down list
Results:
only the version in test/1 will appear.   Even if the yml in test/3 is fine, rancher will stop parsing at test/2.
Expected:
We should see all the valid catalog entries in the drop down list.

Version 1.1.1
@cwundram - I cannot reproduce this in our current build.  Please upgrade to 1.1.1


Implement sliders for the homepage

Issue details

The task is to add two sliders very similar to the udemy homepage.
The first slider will be the current courses the user is enrolled in and the progress. Each card will display the course image (mock images for now), course title and progress (as a progress bar). Again, refer to the udemy homepage for inspiration. In case th user is not enrolled add an animated/static component to tell the user to buy their first course.
The second slider will be cards of the latest updated courses, each card needs to have the course image, title, price and average rating (both stars and numbers). Use starabilitycss or shadcn for the star ratings.

Since I'm currently not doing much can I take this up ?


How to display International System of Units values?

It should have been the default. Instead, I can't even find a way to switch from GiB to GB.

Thanks for suggestion!

Excellent!
Could you make it honor $BLOCKSIZE and / or $BLOCK_SIZE ?

Good idea

Excellent!
Could you make it honor $BLOCKSIZE and / or $BLOCK_SIZE ?

Nice idea! I have created #120 for this.


Check_load against localhost is blank

When I run the check_load against the localhost the webpage is blank. I can still get metrics from Prometheus but that don't correlate to NRPE and look like basic default metrics like up and process_max_fds Any ideas to why? Thanks

Hello. The same here.
Used with CentOS 8 and NRPE v 4.0.3

Hello. The same here.
Used with CentOS 8 and NRPE v 4.0.3

If it helps I am also on CentOS 8 and using the same version of NRPE

Hello,
For information, I solved problem.
I disabled SSL. On CentOS 8, you don't have /etc/default/nrpe for boot option, so, directly in systemd nrpe file, you need add --no-ssl option.
[root@centos-8-01 ~]# more /usr/lib/systemd/system/nrpe.service
[Unit]
Description=Nagios Remote Plugin Executor
Documentation=http://www.nagios.org/documentation
After=var-run.mount nss-lookup.target network.target local-fs.target time-sync.target
<EMAIL_ADDRESS>plymouth-quit.service xdm.service
Conflicts=nrpe.socket

[Install]
WantedBy=multi-user.target

[Service]
Type=simple
Restart=on-abort
PIDFile=/usr/local/nagios/var/nrpe.pid
RuntimeDirectory=nrpe
RuntimeDirectoryMode=0755
ExecStart=/usr/local/nagios/bin/nrpe -c /usr/local/nagios/etc/nrpe.cfg -f --no-ssl
ExecReload=/bin/kill -HUP $MAINPID
ExecStopPost=/bin/rm -f /usr/local/nagios/var/nrpe.pid
TimeoutStopSec=60
User=nagios
Group=nagios
PrivateTmp=true
OOMScoreAdjust=-500


vimPlugins.todo-nvim: remove

todo.nvim is abandoned.
Description of changes
Removed todo-nvim from nixpkgs.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

I force pushed to remove the plugin from generated.nix and added an error message


peer to peer code review - 04-20

Change the color

I think it's fine if you use the same color as in the design because you colors for headings and bars are different so use the right color for each part.

thank you and did it


Paginate-Links :classes attribute

I'm trying to use font-awesome icons for the next and prev links using the following setup on the pagination-links element:
:async="true"  :limit="10"  :hide-single-page="true"  :show-step-links="true"  :step-links="{ next: '', prev: '  ' }"  :classes="{'li.left-arrow > a' : ['fa', 'fa-chevron-left'], 'li.right-arrow > a' : ['fa', 'fa-chevron-right']}"
The issues I get are:

the classes specified are getting applied to the ".ellipses" class if the ellipses link is clicked.

if the step-links have the same value, the .left-arrow>a classes gets applied to the .right-arrow>a element.

I am having the same problem.

Same problem here anyone can help?

I'm having this issue too, .left-arrow/.right-arrow classes not working properly (applying to each other, applying to ellipses). Seems like there's quite a few problems in the library and it doesn't seem to be maintained, might have to switch or make my own.

@lukeramsden I ended up just forking the repository and modifying the code so it showed the icons I wanted.

I found a work around to this issue (though it's super jank). If you label one of the buttons ' ' and the other '' (space in one no text in the other), the labels will end up having the correct classes.
I'm assuming that the connection of classes to each anchor tag depends on the text of the anchor tag. Therefore, differentiating them by having different strings solves the issue (tho this is just a guess)
Try this for the step-links binding:
:step-links="{
next: '',
prev: ' '
}"

same problem... I'm guessing this plugin is no longer being supported by the developer?
(no response since 2017??)


[Bug]: Unexpected token(1:71)

Preflight Checklist

[X] I use the latest version of YouTube Music (Application).
[X] I have searched the issue tracker for a bug report that matches the one I want to file, without success.
[X] I understand that th-ch/youtube-music has NO affiliation with Google or YouTube

YouTube Music (Application) Version
2.0.0
Checklists

[ ] I use the portable version of the YouTube Music Application.
[X] I can reproduce this issue in the official version of (WEB) YTM.

What operating system are you using?
Windows
Operating System Version
windows 11 home version 24h
What CPU architecture are you using?
x64
Last Known Working YouTube Music (Application) version
No response
Reproduction steps
no me permite descargar ,inicialmente traspase las descargas a una memoria sd, luego de expulsar,no me permitió continuar con las descargas a pesar de reiniciar y hasta desinstalar la aplicación

Expected Behavior
esperaba descargar los archivos,ya que tambien trate de cambiar la ruta de descarga a la inicial(musica)
Actual Behavior
me aparece una advertencia con el codigo (1:71) unexpected token

Enabled plugins
1.Descargador

Additional Information
No response

This application is just a fork of https://github.com/th-ch/youtube-music.
(Already fixed at https://github.com/th-ch/youtube-music/issues/2751)


Module not found: Can't resolve 'styled-jsx/style'

Thanks for making this, very cool. Quick question though. When I add it to my react project i'm getting this error:
./~/react-multi-crops/lib/components/Icons.js
Module not found: Can't resolve 'styled-jsx/style'
I see styled-jsx is a dev dependency in package.json but not in production, could that be the issue?

I noticed the source committed here doesn't have the issue. Maybe you just need to upload the latest to NPM?


Autocomplete in text editors

Any way to get CSS properties to autocomplete in text editors when loading them with css-loader? Ideally, I'd type something like styles. and then a bunch of suggestions would show up using classes from the imported file.

There currently isn't a way for this, and this would be out of the scope for css-loader. You could make a plugin for your editor.


Implementation for the Open API validation

See #125:
I want to implement the validation logic for the Open API element. Here's my design (it's not finished).
General design:
Can validation each part individually. for example, Developer create a object, he can call the validate api to validate the created object.
Validate rule can be customized. For example, developer can create their own validation rules.
Detail:
OpenApiElementValidator.cs has the public apis
.\Validations\Visitors has the visitor classes for each element.
.\Validations\Rules has the rule classes for each element.
Please have a look and give me some feedback.

Can I make a suggestion for doing these kind of design reviews?  It would be helpful to see the code changes for a couple of examples rather than a complete implementation.  Trying to review changes to 60 files in the GitHub UI is harder than it should be.

@darrelmiller Yes.  🤣
Can you use the CodeFlow for the large PR?
You can try to run the following command at your local cmd window to open the code flow:
\\codeflow\public\cf.cmd openGitHubPR -webUrl https://github.com/Microsoft/OpenAPI.NET/pull/152

@darrelmiller @PerthCharern I added the attribute for Rule classes. Would you please take a look at https://github.com/Microsoft/OpenAPI.NET/pull/152/commits/da5f90a05a0c3c931cb8ad214c2dda9b0191aee3

@darrelmiller @PerthCharern I also added the attribute for visitor classes. Would you please take a look at :
https://github.com/Microsoft/OpenAPI.NET/pull/152/commits/4a6ca43701b589a359794a9fa99e3ff6e1c0d33a

You just meant the logic to traverse all the Visitors, not a C# attribute, right? I want to make sure I'm not missing something.

In reply to: 352585589 [](ancestors = 352585589)

@PerthCharern For the visitor, Yes. I have the interface IVisitor. That's enough.

For the visitor, Yes. I have the interface IVisitor. That's enough.

In reply to: 352597979 [](ancestors = 352597979,352585589)

@PerthCharern @darrelmiller


mkgmap: 4905 -> 4906

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for mkgmap is: Create maps for Garmin GPS devices from OpenStreetMap (OSM) data
meta.homepage for mkgmap is: https://www.mkgmap.org.uk/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4906 with grep in /nix/store/rfplf7fpgavmj7nhnsnjlwfx8zl3qp57-mkgmap-4906
found 4906 in filename of file in /nix/store/rfplf7fpgavmj7nhnsnjlwfx8zl3qp57-mkgmap-4906

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
mkgmap

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/rfplf7fpgavmj7nhnsnjlwfx8zl3qp57-mkgmap-4906 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A mkgmap https://github.com/r-ryantm/nixpkgs/archive/1d503d2c17eb26ddee3d915047388600ece49ae6.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/rfplf7fpgavmj7nhnsnjlwfx8zl3qp57-mkgmap-4906
ls -la /nix/store/rfplf7fpgavmj7nhnsnjlwfx8zl3qp57-mkgmap-4906/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

mkgmap

Maintainer pings
cc @sikmir for testing.

Result of nixpkgs-review pr 219801 run on aarch64-linux 1

1 package built:

mkgmap


[release-2.6] Triggering image rebuild

Signed-off-by: dislbenn<EMAIL_ADDRESS>

/retest

/retest


DOC: "Options" is not a valid Numpydoc section header.

This does not appear to be in the numpydoc format description[1], nor is
it handled by the Numpydoc Package, as sections are  being parsed
differently [2] if they are actually named 'Parameters', 'Other Parameters',
'Attributes', or 'Methods'.
My guess is that in the functions, this modifies, 'Parameters', was
meant for mandatory, and Options for optional parameters, though both of
those should be in Parameters. They could also be put in "Other
Parameters" as well, though this other section is also rarely used.
While it does not affect sphinx rendering (much), this make it a harder
to process docs using NumpyDocs and in particular this caused issues in
one of my project [3] to improperly detect undocumented parameters. This
would also affect one of the improvement I'm planning for IPython/Jupyter
to understand and reder the NumpyDoc format in a more pleasant and
functional way.
1: https://numpydoc.readthedocs.io/en/latest/format.html
2: https://github.com/numpy/numpydoc/blob/6d7eaf70a19664aa43997c07c65bea0b7774c16a/numpydoc/docscrape.py#L401-L407
3: https://github.com/carreau/velin

Hum, I see what the problem is, I did not realize those were included in Rst docs. I only had checked for dynamic doc.  I can exclude those functions from my current work for now, though that means that my current nice terminal renderer for IPython via papyri, or auto formatting with vélin will not work.
I'm happy to implement that as an extra feature later, but I'm trying to keep the number of edge cases to a minimal, and bring changes upstream when there are only a few of them.
I'll convert that back to a draft for now.

but I'm trying to keep the number of edge cases to a minimal, and bring changes upstream when there are only a few of them.

that makes perfect sense, thanks @Carreau. I know these docstrings are very weird, they also have a custom renderer.


GMCMode=0 does not disable function

Using the information from issue 81 on the main repo, I was able to successfully disable GMCMode under Ubuntu 12.04 and using ROS Hydro.  However, under Ubuntu 14.04 and Indigo, setting GMCMode=0 does not appear to disable this function.  Also, it appears that the file PS1080.ini does not exist, and adding it does not seem to help.  I found two other .ini files (/etc/openni/GlobalDefaults.ini and /etc/openni2/OpenNI.ini) but adding this option to both files also fails to turn off this feature.  How can I turn off GMCMode?  Which .ini file is loaded by OpenNI2 under Ubuntu 14.04 and ROS Indigo?

I'm not sure what changed, but here is the solution (with some background):
In order to determine which file the driver was trying to load, I enabled verbose logging: sudo gedit /etc/openni2/OpenNI.ini.  I set the log level to verbose and enabled logging to the console as follows:
<snip>
[Log]
; 0 - Verbose; 1 - Info; 2 - Warning; 3 - Error. Default - None
Verbosity=0
LogToConsole=1
LogToFile=0
<snip>

After setting this, I brought up the openni driver in ros: roslaunch openni2_launch openni2.launch .
The ROS output should now include Openni logging output.  It should looks similar to this:
<snip>
 721146 VERBOSE    Configuring module 'Depth' from section 'Depth' in file '/usr/lib/OpenNI2/Drivers/PS1080.ini'...
   722167 INFO       Module 'Depth' configuration was loaded from file.
   722574 INFO       Setting Depth.OutputFormat to 100...
   722584 INFO       Depth.OutputFormat value did not change.
   722588 INFO       Setting Depth.FPS to 30...
   722590 INFO       Depth.FPS value did not change.
   722594 INFO       Setting Depth.XRes to 640...
   722598 INFO       Property Depth.Resolution was changed to Custom.
<snip>

As indicated above, the Openni driver is looking for a config file in /usr/lib/OpenNI2/Drivers/PS1080.ini.  On my machine, this file did not exist.  I copied an existing file, with all values commented out, to this location.  I then disable GMCMode by setting: GMCMode=0
When running the ROS driver again, the following log messages indicate that GMCMode has been disabled:
<snip>
   709396 VERBOSE    Configuring module 'Depth' from section 'Depth' in file '/usr/lib/OpenNI2/Drivers/PS1080.ini'...
   709802 INFO       Setting Depth.GMCMode to 0...
   709815 INFO       Property Depth.GMCMode was changed to 0.
   709821 INFO       Depth.GMCMode was successfully set.
   709856 INFO       Module 'Depth' configuration was loaded from file.
<snip>

@akgoins, please verify this change has the intended effect on the 3D data it self.

@shaun-edwards, yes this did turn off the GMCMode and remove the vertical lines.


IPv6 issues

Hi,
I'm having some IPv6 issues . Here's the concise version, and I'll copy/paste the logs from the IRC channel where it was discussed.
IPv6 does not work, except for local connections using the fe80:: local stateless addresses.  (IPv6 works normally on the rest of my LAN's devices.  Address prefixes and routes are supplied by radvd, and set by stateless autoconfiguration.)
If you look on NetworkManager on the title bar, and click Connection Information, IPv6 addresses are supplied (I have a wired NIC as well as the WiFi connection, both of which are configured).  If you look via ip -6 addr ls or ifconfig, however, the only IPv6 addresses visible on the NICs are the fe80:: ones.
Updating the kernel and uImage did not help.  Logs below (2017/09/09, UTC-6):
21:13 < PhotoJim> any suggestions of how to get IPv6 working on the default Ubuntu install on the Pinebook? stateless autoconfig works on other machines on the LAN. the
NetworkManager popup window shows IPv6 addresses but they don't show up in ifconfig or ip -6 addr ls, and only local traffic works on IPv6 (and it uses the
fe80:: addresses)
21:20 -!- Bill<EMAIL_ADDRESS>has quit [Quit: Konversation terminated!]
21:23 -!- kuku<EMAIL_ADDRESS>has joined #Pine64
21:28 <@marcushh777> PhotoJim:  what do you mean "default Ubuntu install" ?
21:33 < PhotoJim> marcushh777: the one that was on the device when shipped
21:34 < PhotoJim> marcushh777: the BSP image
21:34 <@marcushh777> ok
21:34 <@marcushh777> please update the image
21:35 <@marcushh777> the default image is badly broken
21:35 <@marcushh777> all of the issues have been fixed by the recent community images
21:36 < PhotoJim> as in install from scratch?  or running the kernel and uImage etc. scripts, and doing apt-get update etc.?
21:36 <@marcushh777> use the Pine Installer and load the Pinebook from scratch
21:36 < PhotoJim> ahh, darn, that sucks :)
21:36 <@marcushh777> ... you can try the images first by booting from SD
21:36 <@marcushh777> PhotoJim:  it does not suck
21:37 < PhotoJim> I've got everything nicely set up except for IPv6, so yeah, it does suck :) I have to do it all again.
21:37 <@marcushh777> just for grins, please do   uname -a for me ?
21:37 <@marcushh777> and post
21:37 < PhotoJim> Linux tanani 3.10.105-bsp-1.2-ayufan-77 #1 SMP PREEMPT Sun Jul 9 12:09:30 UTC 2017 aarch64 aarch64 aarch64 GNU/Linux
21:38 <@marcushh777> hmmm
21:38 <@marcushh777> that's the image it was shipped with ?
21:39 <@marcushh777> interesting
21:39 < PhotoJim> no, that's after running pine64_update_kernel.sh and ..._package and ..._uboot
21:39 <@marcushh777> oh ok
21:39 <@marcushh777> because that's my kernel too;
21:39 < PhotoJim> 0.6.2 I believe
21:40 <@marcushh777> right
21:40 <@marcushh777> that should be ok...
21:40 < PhotoJim> it works perfectly.  except for IPv6.
21:40 <@marcushh777> before you do anything to it, you need to talk it over with ayufan
21:40 <@marcushh777> don't screw around with it until you talk to him
21:40 < PhotoJim> ok
21:41 <@marcushh777> I mean, use the Pinebook, just don't mess with the stack
21:41 <@marcushh777> or upgrades or anything like that;
21:41 <@marcushh777> to be honest , I have not even tried IPV6 on mine yet
21:42 <@marcushh777> we need to find out if he's tested it either
21:42 < PhotoJim> it's odd that NetworkManager shows IPv6 addresses if you look in Connection Information, but ip -6 addr ls and ifconfig show nothing but the fe80:: addresses
21:42 <@marcushh777> he was having problems with ethernet ;  and got them fixed yesterday
21:43 <@marcushh777> he may still have an issue with IPV6;  although, I have not heard anyone speak of it
21:43 <@marcushh777> report it to him, and lets see what he says
21:43 <@marcushh777> ayufan
21:43 <@marcushh777> or ayufan1
21:43 < PhotoJim> where best to notify him?
21:43 < PhotoJim> it's not urgent, this is just an experimental machine
21:43 <@marcushh777> on the github site, or the forum,  or find him here
21:44 <@marcushh777> he lives in Poland so, sound asleep right now
21:44 < PhotoJim> k, I'll try to talk to him in here tomorrow (we're UTC-6 here)

Can confirm this is a thing.  IPv6 still doesn't work, even after these updates:
uname -a:
Linux pinebei 3.10.105-bsp-1.2-ayufan-77 #1 SMP PREEMPT Sun Jul 9 12:09:30 UTC 2017 aarch64 aarch64 aarch64 GNU/Linux
lsb_release -a:
Distributor ID:	Ubuntu
Description:	Ubuntu 17.04
Release:	17.04
Codename:	zesty


CHANGELOG.md for 2.0.2

Can you write the CHANGELOG.md for version 2.0.2?
Thanks

Hello,
You can find the changelog here https://github.com/BatchLabs/cordova-plugin/blob/master/CHANGELOG.md


Any chance of some backport love for the isPresented overload of navigationDestination?

From the SwiftUI interface (documentation link):
    /// Associates a destination view with a binding that can be used to push
    /// the view onto a ``NavigationStack``.
    ///
    /// In general, favor binding a path to a navigation stack for programmatic
    /// navigation. Add this view modifer to a view inside a ``NavigationStack``
    /// to programmatically push a single view onto the stack. This is useful
    /// for building components that can push an associated view. For example,
    /// you can present a `ColorDetail` view for a particular color:
    ///
    ///     @State private var showDetails = false
    ///     var favoriteColor: Color
    ///
    ///     NavigationStack {
    ///         VStack {
    ///             Circle()
    ///                 .fill(favoriteColor)
    ///             Button("Show details") {
    ///                 showDetails = true
    ///             }
    ///         }
    ///         .navigationDestination(isPresented: $showDetails) {
    ///             ColorDetail(color: favoriteColor)
    ///         }
    ///         .navigationTitle("My Favorite Color")
    ///     }
    ///
    /// Do not put a navigation destination modifier inside a "lazy" container,
    /// like ``List`` or ``LazyVStack``. These containers create child views
    /// only when needed to render on screen. Add the navigation destination
    /// modifier outside these containers so that the navigation stack can
    /// always see the destination.
    ///
    /// - Parameters:
    ///   - isPresented: A binding to a Boolean value that indicates whether
    ///     `destination` is currently presented.
    ///   - destination: A view to present.
    public func navigationDestination<V>(isPresented: Binding<Bool>, @ViewBuilder destination: () -> V) -> some View where V : View

I'd offer to do a PR but I feel I'm a bit out of my depth here.

Thanks for raising this issue @kielgillard  - this API had completely escaped my notice! When I have a chance, I'll see if it can be accommodated in NavigationBackport.

+1 for this :D

I finally spent some time on this and have made good progress, so I'll try to merge it in soon.

+1

It took a while but I've added support for this in v0.7.0. Please let me know if you find any issues with it.


[default: ] vs. (default: )

I see a mixture of [default: ...] and (default: ...) for default values of user options.

24 occurrences of [default: ...], for example:
https://github.com/pypa/distutils/blob/e5b06e144d1e57e0efebbc943bf071cd22e46da8/distutils/command/install_lib.py#L41-L46

11 occurrences of (default: ...), for example:
https://github.com/pypa/distutils/blob/e5b06e144d1e57e0efebbc943bf071cd22e46da8/distutils/command/install_data.py#L18-L23

Is it on purpose? If so, what meaning does either version convey? If not, would it be OK to standardise?

See https://github.com/pypa/setuptools/issues/4421.

I'm unaware of any intent behind the difference. Feel free to spelunk the blame history to see if the commit messages provide any insight, or don't and we can proceed with standardizing. I have no preference.

The blame history of a few examples does not provide any insight.
I suggest using [default: ]. The closest to a "standard" that defines how default values should be printed is perhaps docopt:

[...] and whether that argument has a default value ([default: 10]).


docker inspect with an image instead of a container logs errors (trivial)

From a script, I've called docker inspect ${my_image}. Everything works as expected, but docker logs a couple "error" lines when it first fails to look up my image as a container, before proceeding to look it up as an image.
It should probably only be logging an error if both lookups fail.
INFO[5635] -job container_inspect(registry.chi.shopify.io:80/shopify:0beb47b3f04357bf6a4dfb25784d591fa0358a02) = ERR (1)
ERRO[5635] Handler for GET /containers/{name:.*}/json returned error: No such container: registry.chi.shopify.io:80/shopify:0beb47b3f04357bf6a4dfb25784d591fa0358a02
ERRO[5635] HTTP Error: statusCode=404 No such container: registry.chi.shopify.io:80/shopify:0beb47b3f04357bf6a4dfb25784d591fa0358a02
INFO[5635] GET /v1.17/images/registry.chi.shopify.io:80/shopify:0beb47b3f04357bf6a4dfb25784d591fa0358a02/json
INFO[5635] +job image_inspect(registry.chi.shopify.io:80/shopify:0beb47b3f04357bf6a4dfb25784d591fa0358a02)

The docker inspect look up container first, if fail, then look up a image. So every inspect will look up container and then look up image even if you just want to inspect a image
I suggest to have a flag such as -i to specify to inspect a image, and inspect a container by default
This is useful if a container happen to have the same name of a container and I want to inspect a image.  What you guys think? If it is ok, I'll work on a PR

Maybe this should just be a debug entry instead of ERROR

@cpuguy83 But if I have a image which happened to have a same name of a container (I have really meet this situation) and I want to inspect the information of  the image, I have to rename or remove the container so I can inspect the image, otherwise it always return the container information

We're also seeing alerts come out of upstart's docker.log for what looks to be similar behavior. It seems like this shouldn't be an error, but more a handled exception that is logged as info or debug, with a message on inspect of "container or image doesn't exist".

Just chiming in here, but in my opinion, the solution is to drop these down from error level to something else, like info.
The error level is commonly used to indicate an issue within the application (https://stackoverflow.com/questions/2031163/when-to-use-log-level-warn-vs-error). These failed requests are not an issue with the docker daemon, but rather a bad request from the client. Thus they should not be logged as errors.

@phemmer Thanks for your info, I'v posted a new approach, #13537

@phemmer these are errors. As I explained in #13537 a bad call to an endpoint which throws a 404 is an error and the api/daemon have to log it. I don't see why would we rather threat these errors at the info level. I really expect to see an error in the logs cause every api/web implementation threat a resource not found as an error and log it

To put my comment another way. Clients don't decide errors that should show up in daemon logs. It's client own business to make correct calls and because inspect is a cli command it should not mess around with api logs.
It should not log an error only if both calls fails because it's just wrong not to log the first bad call to container/json

It's client own business to make correct calls and because inspect is a cli command it should not mess around with api logs.

I completely agree, that's why it's the client's job to log the error, not the daemon.
How do you expect to be able to distinguish client errors from daemon errors? If I want to monitor the docker daemon to make sure the system is healthy, it is impossible to do so if client errors are logged by the daemon as daemon errors.

I still think api 404 errors are server side errors, well not daemon error but of course backend api error. The problem here may be in how docker is structured, meaning we should log daemon errors because it's our application and api errors as well because it's daemon public layer, @phemmer wdyt?

this is not a client error but a server one

Server error means the server did something wrong and needs to be fixed. What needs to be fixed on the server if the client tried to perform an operation on a non-existent resource?
How do you propose solving the issue where I need to monitor the daemon for actual errors? Like if daemon can't launch a container (disk full, etc), or the daemon can't reach the hub, etc? I can't just look for level="error" because this contains bad requests from clients as well.

what I was saying is it's also a controlled server side error (you catch it)

Since both are related, see https://github.com/docker/docker/pull/13537

I think we should close this issue, not because the premise is wrong but rather because this is just the way docker works right now.  For better or worse, user caused errors are logged as ERRORs in the daemon's log.  I know this isn't technically a user error, but from the daemon's perspective it is.
Once we get #9509 then we can "fix" it so that the only ERRORs that appear in the daemon logs are things that are really wrong with the daemon and not user cause issues.

If we can't have a conclusion about if we should show non exist container/image as daemon side error message. At least we should fix this scenario, use a valid command docker images with a valid image id, appears error message on daemon is never right and not acceptable.
Maybe we should reopen #13505 and fix this by:
If people use docker image, then don't take the non exist container as error but just return 404, only show error when both container and image are not exist.
As I said before, #10773 just fixed another problem, which is container and image have the same name, as long as we still offer docker images xxx to show container/image without any option, we will face this problem, and I think that's mostly what will happen.

Sorry to see that, now we have no opening PR for this issue, and seems we still have no idea how to fix this problem, any thoughts?

@hqhq once #9509 is done I want to go through and see if we can remove all user-generated errors from the daemon log files - I think that would solve this issue.

You can use docker inspect --image $myimage to avoid container lookup. That's the way to make those logs disappear :smile_cat:  I'm closing this issue.. Thanks for your comments.

@runcom I know this particular issue was just closed, but I would like to reiterate a point that has been made in a few different threads as well:
The docker daemon in general emits a lot of unnecessary errors, causing monitoring and alerting to be problematic. The maintainability of a system is highly dependent on the ability to effectively monitor and alert on it. Some guidelines:

Client-side errors or request errors should be handled by the client. This is the unix way and is typical of client / server architecture. While it might be helpful to know clients are making bad requests, this should be done at the debug, info, or warning level (depending on severity and impact to the business logic).

It's a best practice to log server-side events as errorlevel "ERROR" only when business logic has failed and demands user action to take corrective measures (no disk space, unable to contact endpoints, etc.). This enables effective monitoring and alerting without excessive alerts (pagerduty calls) and prevents complicated filtering rules to decide what's actually a problem or not. Monitoring docker should be as easy as blindly capturing syslog events. That line could accurately have been logged as:

INFO[5635] Handler for GET /containers/{name:.*}/json returned error: No such container: registry.chi.shopify.io:80/shopify:0beb47b3f04357bf6a4dfb25784d591fa0358a02
INFO[5635] HTTP Error: statusCode=404 No such container: registry.chi.shopify.io:80/shopify:0beb47b3f04357bf6a4dfb25784d591fa0358a02

When retry or failover logic is in place, only the final attempt should fail. For example, if the business logic says to pull a remote image if a local image doesn't exist, then an error shouldn't be reported if the local image isn't found (it should be debug or info). This is because the request hasn't actually failed as it is able to pull a remote image. An error should only be logged if a valid user request fails after exhausting retries (info / warnings will help indicate retries are necessary however).

Industry standard naming of events should be used in place of custom ones. This allows for seamless integration with existing monitoring solutions (without having to implement a new set of rules just for docker, i.e. CRITICAL|FATAL|ERROR|WARNING|INFO|DEBUG|TRACE).


Sending two emails fails

Hi, I'm using swiftmail in my Symfony2 apllication.
I'm tying to send in the same function (sendEmaillDemande()), 2 differents messages.
The first one is sent but the second is not.
How could i make it work?
Thank you.

Not a bug, too few details, check on mailing list


Minor ui changes for Ben's client auth PR

+heartbeat for web client that was in ben's branch, it's merged to master already

closing in favor of https://github.com/CatalystCode/3dtoolkit/pull/63


Notefield zoomed out on some styles in 480p themes

Something in the new notefield positioning code is not correctly compensating for older 480-resolution themes, causing doubles to not be zoomed correctly. This has been reported by other users as well on SL.

This is likely to the new scaling code, we'll look into this,
Thanks

Fixed 4.6 Final


stop cleaning on post-install

Lots of issues stemming from cleanup. Useful, so I've left the script, but it no longer runs automatically now.

should fix https://github.com/nodegit/nodegit/issues/561

Word, thanks for addressing this quickly @maxkorp!


"too many commands" & "Ignoring command 'echo'"

I downloaded version 1.3.4 and loaded my songs, and when I went into a CS:GO deathmatch and typed 'exec slam' then 'la' it came up with 15-37 of my playlist even though I had all my songs loaded (1-37) in SLAM.exe. Also, when the songs loaded up in the console, it wasn't normal. It said "ignoring command 'echo'" and then the song number and song name. And at the very bottom after song 37, it said "too many commands"
Now, the 22 songs I was able to play still worked and I could select the song and play it, but I want my other songs that I loaded into SLAM aswell. Please help.
~ thorN

Valve made a mistake with an update. It was patched. Issue should be resolved.


Fix xpath query selector for 'Save episode' button

This PR fixes the xpath query for the button 'Save episode' that is clicked after the .mp3 audio file of the episode is uploaded.
Please review the PR and see if I should make changes in some other files, as I made the fix just so I can use it for my own purposes.
I am happy to open the first PR for this project.

Thank you for contributing to this project!

Is it possible this is still causing issues like #27

I have a bunch of changes made to my fork, the main branch of my repo https://github.com/matevskial/youtube-to-anchorfm is functional and is used by my organization extensively. Can  you please give me advice how to bring my changes to your repo if possible?
Here are the changes in summary:
Add environment variable 'IS_EXPLICIT' , configure if the episode to be uploaded has explicit content or not
Upload episode art from youtube thumbnail, upload the thumbnail straight from youtube into anchor
Fix save as draft and publish button

I like the idea of uploading episode art!

It would be nice if we can preserve the publish date.

@matevskial Why don't you create a pull request with your changes? All the issues I've had you seem to have already addressed!

@abe-101 I wanted to create several PR's in serial, feature by feature in order to make code review easier, but since the first PR alone was not properly working, that is a mistake on my end, I'll create a PR with all the features I added then.


Embed Twitter timeline on blog page

I would like to embed the Library Carpentry Twitter timeline on the blog page. Does anyone know how to do that? https://github.com/LibraryCarpentry/new-website/blob/gh-pages/blog.html
Code is:
Tweets by LibCarpentry

@richyvk or @libcce do you know how to do this? I managed to embed it but it looked awful and wasn't nice. Aiming for something like the way it looks on this page: https://carpentries.org/blog/

Awesome @evanwill - thanks !


Feature Request: Support config file for all sdk commands

Feature Request
PR #950 implemented config file support for the sdk's scorecard subcommand. It would be useful to have support for this in multiple subcommands for the sdk. Due to the way cobra and viper can integrate, it shouldn't be too difficult to implement in other subcommands, but it may be a bit time consuming as it would require quite a bit of refactoring.

/remove-lifecycle rotten

HI @joelanford, @jmccormick2001 and @estroz
Is it something that can be closed? It shows still valid to be considered with the changes that will be made on it. WDYT?

Yeah, I think this can be closed (for now at least). We can revisit after the KB integration and create a new issue if necessary.


Fix page layout of installing kubectl on linux

Changes to fix the current broken page layout at https://kubernetes.io/docs/tasks/tools/install-kubectl-linux/

Possibly, resolves https://github.com/kubernetes/website/issues/42651

Original Page | Deploy Preview

/approve


improved readme layout

Moved logo to center

 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.

Codecov Report

Merging #229 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #229   +/-   ##
=======================================
  Coverage   50.38%   50.38%
=======================================
  Files          47       47
  Lines        1175     1175
=======================================
  Hits          592      592
  Misses        583      583

Continue to review full report at Codecov.

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


docs: beta release changes

Beta release has been done, updated necessary docs changes.

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Build Cause: Pull request #263 opened

Start Time: 2021-04-23T16:47:06.565+0000

Duration: 14 min 10 sec

Commit: 71023d6adc7c2f92d7e507b004beb549f3e73d7f

Test stats :test_tube:

Test
Results

Failed
0

Passed
81

Skipped
0

Total
81

Trends :test_tube:


terraformGateway --help switch requires ATLAS_USERNAME to be set

Running terraformGateway --help throws this traceback:
Traceback (most recent call last):
  File "/Users/robin/.pyenv/versions/td-terraform-3.7.1/bin/terraformGateway", line 56, in <module>
    @click.option('--organization', default=os.environ['ATLAS_USERNAME'], help='Terraform Organization name (Uses ATLAS_USERNAME environment var if not provided)')
  File "/Users/robin/.pyenv/versions/3.7.1/lib/python3.7/os.py", line 678, in __getitem__
    raise KeyError(key) from None
KeyError: 'ATLAS_USERNAME'

It works OK if ATLAS_USERNAME is set when running the command:
ATLAS_USERNAME=robinbowes terraformGateway --help
Usage: terraformGateway [OPTIONS] COMMAND [ARGS]...

  Console utility for interfacing with Terraform Enterprise

  To see details for each command, use:
   terraformGateway [command] --help
...

I didn't mean to close this. I only said "should fix" not "does fix" !!


"ERROR: No animation found" even though it's fine for the characters name

Describe your bug here.
when i use my character, there was no image and animation, instead, uses the flixel logo which is annoying asf. like it's fine bro, why?
Crash Log or logcat logs (if existing)
Null Object Reference
flixel/graphics/tile/FlxDrawQuadsItem.hx [line 119]
flixel/FlxCamera.hx [line 772]
flixel/FlxGame.hx [line 875]
flixel/FlxGame.hx [line 563]
openfl/events/EventDispatcher.hx [line 444]
openfl/display/DisplayObject.hx [line 1400]

Are you modding a build from source or with Lua?
Lua
What is your build target?
Desktop (Linux, Windows or Mac)
Did you edit anything in this build? If so, mention or summarize your changes.
no

no im on mobile lol, also about the error, when i make a custom character with the same name and xml, it uses flixel logo

where are you placing the character image? and are you sure you're selecting your mod before entering the Character Editor ??

no im on mobile lol, also about the error, when i make a custom character with the same name and xml, it uses flixel logo

https://github.com/MobilePorting/FNF-PsychEngine-Mobile/wiki/Installing-Mods


TestRef example is missleading and pointless.

arrays are reference types, passing them with ref  changes nothing. You can remove the ref  from static void FillArray(ref int[] arr) and from FillArray(ref theArray); and the result will be the same. This is very very confusing to people trying to learn from the official docs.
With and without using ref  the following output is achieved:
Output:
Array elements are:
1111 2 3 4 5555 Press any key to exit.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: e4e7c322-b662-56f9-99af-f47106af0780
Version Independent ID: ae1843c3-6606-c1fa-50c8-f1ac99ad5d05
Content: Passing Arrays Using ref and out (C# Programming Guide)
Content Source: docs/csharp/programming-guide/arrays/passing-arrays-using-ref-and-out.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

@Alumniminium

arrays are reference types, passing them with ref changes nothing.

passing with ref changes a lot :) if you pass with ref you can change to what reference references. Consider the following example code (I've already put outputs into comments, however you might just copy it and run in try.dot.net):
using System;

public class Program
{
  public static void Main()
  {
    int[] origin = { 1, 2, 3, 4, 5 };
    int[] copy = origin;
    ClearArray(origin);
    Display(origin);  // 0 0 0 0 0
    Display(copy);    // 0 0 0 0 0

    int[] origin2 = { 1, 2, 3, 4, 5 };
    int[] copy2 = origin2;
    ClearArrayRef(ref origin2);
    Display(origin2);  // 0 0 0 0 0
    Display(copy2);    // 1 2 3 4 5  remark: copy2 and origin2 reference two different arrays

    int[] origin3 = { 1, 2, 3, 4, 5 };
    int[] copy3 = origin3;
    ClearArrayAsIfRef(origin3);
    Display(origin3);  // 1 2 3 4 5   remark: method has failed to clear array.
    Display(copy3);    // 1 2 3 4 5
  }

  private static void Display(int[] arr) => Console.WriteLine(string.Join(" ", arr));

  private static void ClearArray(int[] input)
  {
    for (var i = 0; i < input.Length; i++)
    {
      input[i] = 0;
    }
  }

  private static void ClearArrayRef(ref int[] input)
  {
    input = new int[input.Length];
    for (var i = 0; i < input.Length; i++)
    {
      input[i] = 0;
    }
  }

  private static void ClearArrayAsIfRef(int[] input)
  {
    input = new int[input.Length];
    for (var i = 0; i < input.Length; i++)
    {
      input[i] = 0;
    }
  }
}

Note that ClearArrayRef and ClearArrayAsIfRef differ only by ref, however, they behave very differently.
I agree that documentation should provide better examples that would highlight the difference.

I had some internal discussions, and here's our recommendation:
This article should be removed, and it should be redirected to the array index page.
@Alumniminium is correct that this is a misleading set of examples. @pkulikov is also correct that there are semantic differences in passing arrays (or any object) by reference. We tried to formulate a good example that would pass an array by reference. Any example we constructed could be a better API design using a combination of return values, or out arguments.
I've added this to the backlog, and added the up-for-grabs label, if anyone wants to submit a PR for the fix.


🔥 FIREBREAK: Evaluate octoDNS for Efficient DNS Management Across Providers

Background
Our team currently manages DNS entries across multiple providers with varying levels of manual intervention and inconsistency in processes. octoDNS offers a "DNS as code" approach, providing tools and patterns to manage DNS records across multiple providers from a single, version-controlled repository. This aligns with our goals of infrastructure as code, aiming to streamline DNS management, reduce errors, and improve deployment consistency.
Questions / Assumptions

Our current DNS management practices are fragmented and can benefit from more unified management.
octoDNS supports all our current DNS providers and record types as per the documentation.
We need to understand the setup complexity and the operational benefits of using octoDNS.
Assume a learning curve and initial setup time that will be offset by long-term efficiency gains.

What hypothesis do we want to test?/What do we want to learn?
We hypothesize that implementing octoDNS will lead to more streamlined, efficient, and error-free DNS management across multiple providers. We aim to learn:

The ease of setting up octoDNS with our existing DNS configurations.
The compatibility of octoDNS with all our DNS providers.
The robustness of the deployment and rollback mechanisms provided by octoDNS.

Definition of done

[x] Setup a test environment with octoDNS managing a subset of our DNS records.
[x] Document the setup process, including any obstacles encountered.
[x] Conduct a dry-run (noop) to assess the changes octoDNS would apply.
[x] Execute a live deployment on the test environment and verify the results.
[x] Evaluate the benefits and any limitations observed during the test.
[ ] Decision made on whether to adopt octoDNS across all environments.
[ ] If adopted, create detailed implementation and migration plans.
[x] Firebreak findings documented and shared with the team.

TL;DR:

Octodns is a simple and effective tool for DNS management that uses a YAML configuration file to define DNS records.

One of its key strengths is the ability to manage DNS records across multiple providers, which could be valuable for the Ministry of Justice in the future.

It offers a dry run feature and syncing capabilities for safe and consistent DNS management.

Compared to Terraform, Octodns has a simpler setup, uses YAML instead of HCL, and doesn't require a state file, but lacks Terraform's IAM Roles for Service Accounts (IRSA) support.

For the Ministry of Justice, Octodns could be a good choice if we anticipate needing to manage DNS across multiple providers in the future, but sticking with Terraform may be preferable if we plan to continue using a single provider and are already heavily invested in Terraform.

I posted a message in our Slack channel and will now mark this issue as resolved.

Here is an example of the code required to get OctoDNS working.
https://github.com/ministryofjustice/operations-engineering-dns-spike


chore(sat): rename sat struct fields so they're more descriptive

Closes #55
This PR does two things:

renames fields
tidy up some file imports

It does not change any functionality whatsoever.

Duplicate of https://github.com/suborbital/sat/pull/60


Feature request: automatically determine input for desktop.libs task

Currently, all node_modules that need to be 'copied' for the desktop need to be specified manually in desktop.libs.ts. However, from what I can determine, all information about the dependencies and their locations is already specified in the included configuration.
I'm looking for a way in which the desktop.libs task can determine automatically, based on the configuration on the dependencies, which files to copy. Would this be doable?

I updated desktop.libs.ts to automatically add the dependencies and assets that were added to project.config.ts, by adding this line, and the necessary include:
include { relative, join } from 'path';

src.push(...Config.DEPENDENCIES.map(x => relative(Config.PROJECT_ROOT, x.src)));

It's not perfect (Config.DEPENDENCIES actually includes the hardcoded NPM_DEPENDENCIES values from seed.config.ts), but at least you don't have to manually add any dependencies or assets you add to project.config.ts to desktop.libs.ts.

Amazing. You could do a PR with this!

Will do.


General: Remove Learning Goals, Prerequisites, Tutorial Groups API Call when messaging is enabled

Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
[ ] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[ ] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[ ] I chose a title conforming to the naming conventions for pull requests.

Server

[ ] I followed the coding and design guidelines.
[ ] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[ ] I added pre-authorization annotations according to the guidelines and checked the course groups for all new REST Calls (security).
[ ] I implemented the changes with a good performance and prevented too many database calls.
[ ] I documented the Java code using JavaDoc style.

Client

[ ] I followed the coding and design guidelines and ensured that the layout is responsive.
[ ] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[ ] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[ ] I added authorities to all new routes and checked the course groups for displaying navigation elements (links, buttons).
[ ] I documented the TypeScript code using JSDoc style.
[ ] I added multiple screenshots/screencasts of my UI changes.
[ ] I translated all newly inserted strings into English and German.

Changes affecting Programming Exercises

[ ] I tested all changes and their related features with all corresponding user types on Test Server 1 (Atlassian Suite).
[ ] I tested all changes and their related features with all corresponding user types on Test Server 2 (Jenkins and Gitlab).

Motivation and Context

Description

Steps for Testing

Prerequisites:

1 Instructor
2 Students
1 Programming Exercise with Complaints enabled

Log in to Artemis
Navigate to Course Administration
...

Exam Mode Testing

Prerequisites:

1 Instructor
2 Students
1 Exam with a Programming Exercise

Log in to Artemis
Participate in the exam as a student
Make sure that the UI of the programming exercise in the exam mode stays unchanged. You can use the exam mode documentation as reference.
...

Review Progress

Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Exam Mode Test

[ ] Test

Test Coverage

Screenshots

Codecov Report

Merging #6452 (df7ce06) into develop (c9707db) will increase coverage by 3.96%.
The diff coverage is 14.28%.

:exclamation: Current head df7ce06 differs from pull request most recent head f375923. Consider uploading reports for the commit f375923 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #6452      +/-   ##
=============================================
+ Coverage      76.48%   80.44%   +3.96%
- Complexity         0    13030   +13030
=============================================
  Files           1248     2323    +1075
  Lines          47096    89249   +42153
  Branches        8797    13191    +4394
=============================================
+ Hits           36020    71799   +35779
- Misses          5684     9620    +3936
- Partials        5392     7830    +2438

Impacted Files
Coverage Δ

...n/webapp/app/overview/course-overview.component.ts
72.48% <14.28%> (-3.05%)
:arrow_down:

... and 1076 files with indirect coverage changes

Flag
Coverage Δ

client
76.47% <14.28%> (-0.02%)
:arrow_down:

server
84.89% <ø> (?)

Continue to review full report in Codecov by Sentry.

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


[Feature request] HugoMods module to integrate Apache Echarts for data visualisation

Hi
Requested Feature
A HugoMods module to support the Echarts  interactive data visualization
data definition options

via data/*
as shortcode content

What'is ECharts
An Open Source JavaScript feature rich Data Visualization Library.
A Declarative Framework for Rapid Construction of Web-based Visualization
https://echarts.apache.org/en/index.html
Third party integration
from LoveIt
The echarts shortcode supports data visualization in Hugo with [ECharts](https://echarts.apache.org/) library.
https://hugoloveit.com/theme-documentation-echarts-shortcode/
Best regards,
Tshitshi

It's good to have.

Simple user guide.
1. Import module
github.com/hugomods/echarts
2. Include templates

Skip this step if you're using HB Framework themes.

It depends the theme implementation, please ask for help from theme author, here is an example.
<html>
  <head>
  </head>
  <body>
    ...
    {{/* Put following template near the end of body.  */}}
    {{ partial "echarts/assets/script" . }}
  </body>
</html>

3. Using with code block
```echarts {height=320 id="foo" class="c1 c2" data-value="foo"}
{
  "title": {
    "text": "ECharts Getting Started Example"
  },
  "tooltip": {},
  "legend": {
    "data": [
      "sales"
    ]
  },
  "xAxis": {
    "data": [
      "Shirts",
      "Cardigans",
      "Chiffons",
      "Pants",
      "Heels",
      "Socks"
    ]
  },
  "yAxis": {},
  "series": [
    {
      "name": "sales",
      "type": "bar",
      "data": [
        5,
        20,
        36,
        10,
        10,
        20
      ]
    }
  ]
}
```

It accepts YAML/JSON/TOML definitions, please note that you can't copy examples from ECharts site directly, since the source code is written JavaScript, the module can't recoginate it. There are also some online converter, you can convert JS to JSON there first.

height: height, optional, default to 400.
id optional, will assign a default ID if not set.
class, data-value are optional.
You can also specify width, which is 100% by default.

4. Using with shortcode
// data/charts/line.json
{
  "xAxis":{},
  "yAxis":{},
  "series":[
    {
      "data":[
        [
          10,
          40
        ],
        [
          50,
          100
        ],
        [
          40,
          20
        ]
      ],
      "type":"line"
    }
  ]
}

{{< echarts height=480 id="line" class="c3 c4" data-value="line" data="charts.line" />}}

Parameters are same as the code block, except the data, which is the name of data file, support page data and site data.

The dot . of data name represents the folder seperator.

5. Styling
All echarts containers have the echarts class name, you can apply styles on there.

Additonal usage for shortcode, you can omit the data parameter and put the definitions inside the shortcode as the code block way did.
{{< echarts height=480 id="line" class="c3 c4" data-value="line" >}}
PUT CHARTS DEFINITIONS HERE
{{< /echarts >}}


Install instructions are not correct

npm install apex-frontend-boost is not the correct way to install project. If you do that it installs everyting in ~/node_modules

I'm not too familiar with Unix, but ~/ is the root of what exactly? For me, npm install apex-frontend-boost creates node_modules into the active directory, and installs all the dependencies there.
Another thought: we could suggest npm install -g apex-frontend-boost which installs it globally. One caveat is it makes finding config.json more complicated and less intuitive (Example: C:\Users\vince\AppData\Roaming\npm\node_modules\apex-frontend-boost\config.json). I'll keep this thought on hold for now.
Based on https://github.com/npm/npm/issues/3597, I am not against reverting back to git clone, but I would keep npm as well as another option. I'll actually document both options, but will have git clone first.


[Feature] 5x astral key bug

Describe the bug
5 astral crate keys disappeared after opening the crate
Expected behavior
I bought 5 astral crate keys and wanted to open them. The moment I tried to open the astral crate all my keys disappeared and I didnt get a single reward.
Screenshots

**Username: Westpaint15
**Server name (all applicable): Titan
Additional context
Foxxarcher (Trialmod server titan) saw the astral crate keys disappear from my inventory

Please message me on Discord (Fatyg5#0001) the next time you are online!


add wind steering mode

Thanks for this! MAybe you can add commands for tack also? I guess it is tested?

Yes it works for me. I will try to find out the tack commands next time.


Update PolarisAuthorizerImpl to use PolarisGrantManager interface and add EntityCache based PolarisGrantManager impl

Description
Currently, the PolarisAuthorizer implementation relies on the ResolvedPolarisEntity to return a list of grants for each entity in order to determine if a Principal has authorization to perform an operation on a given target. The grant records are populated by the Resolver, utilizing the EntityCache for a given realm to avoid unnecessary roundtrips to the persistence store for, e.g., the grants held by a given PrincipalRole or CatalogRole.
Because the PolarisAuthorizer relies on the grant records in the ResolvedPolarisEntity, there's no opportunity to utilize custom grant logic (for example, to declare a target to be globally viewable or attach PrincipalRoles dynamically) without either updating the Resolver or writing custom grant records prior to the Resolver execution.
This change decouples the PolarisAuthorizer from the ResolvedPolarisEntity so that it depends entirely on the PolarisGrantManager to declare which grants exist on a securable. It implements a EntityCacheGrantManager that, under the hood, still works with the EntityCache so that the lookups from the Resolver populate the cache with both entities and grants. However, the PolarisAuthorizer doesn't need to know anything about the EntityCache or the Resolver types. For convenience, it still uses the PolarisResolvedPathWrapper, as it's still a useful container for passing around full paths for entities.
The logic in the PolarisAuthorizer is unchanged and all tests still pass. An additional test class is added to verify the EntityCacheGrantManager works directly with the cache and will delegate to the underlying PolarisGrantManager when necessary.
Note that the logic that implicitly grants the service_admin role SERVICE_MANAGE_ACCESS privileges on the implicit root container has been moved to the EntityCacheGrantManager. I ran the tests without this move and they all still passed, so it seems that, at some point, the root container is being persisted and the grants are recorded. However, to ensure backward compatibility, I ported the logic. We may want to get rid of that at some point.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[X] EntityCacheGrantManagerTest

Test Configuration:

Hardware:
Toolchain:
SDK:

Checklist:
Please delete options that are not relevant.

[X] I have performed a self-review of my code
[X] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[ ] If adding new functionality, I have discussed my implementation with the community using the linked GitHub issue

@dimas-b @adutra now that the CDI stuff is in, can we move forward with this PR? This is part of the overall move to help support the federated identities (removing the dependency on the resolver to obtain principal role grants). There are a lot of refactoring PRs in flight and I'm worried that there are too many conflicting approaches to the overall problem.


For ":Allow observation fields from" user pref - don't let other users edit one's observation fields

From: https://forum.inaturalist.org/t/disallow-others-to-edit-your-observation-fields-opt-in/13917/
Currently, one can choose to allow only site curators or oneself to add observation fields to one's observations:

There's been a request to expand that preference to not allow other users to edit the observation fields on one's own observations. So something like "Allow Adding and Editing Observation Fields from".

If other people can edit but not add, that seems like a bug to me. What if we just made it so this existing pref applied to edits in addition to new fields?

fine with me


Release the docker image of store in CI as well.

Currently we will release the docker image of graphscope and manager, we may also want to release the store as it's now in main.
And publich the helm chart as well.

Image will released along with graphscope, and helm charts are published.


orm: Improves locking behavior

Improves the locking behavior of the orm broker
Adds lock counter to the monitor for the orm broker

Current coverage is 95.20%

Merging #67 into master will increase coverage by +0.15% as of dc2e7a7

@@            master    #67   diff @@
=====================================
  Files           36     36
  Stmts         2002   2022    +20
  Branches         0      0
  Methods          0      0
=====================================
+ Hit           1903   1925    +22
  Partial          0      0
+ Missed          99     97     -2

Review entire Coverage Diff as of dc2e7a7

Powered by Codecov. Updated on successful CI builds.


CocoaPods Support

Hey guys! super cool initiative and amazing work!
I wanted to ask if you guys plan to add support for CocoaPods any time soon?
Thanks!

Hi!
I think that Swift Package Manager integrated into Xcode 11 should be enough for using OpenCombine in iOS/macOS/tvOS/watchOS projects, but if you feel like it's necessary, I can go ahead and add support for CocoaPods. Or you can help me out a bit and create a pull request :)

@broadwaylamb cocoapods would be amazing!

@broadwaylamb is anyone working on this? I'd like to pick this up.

@broadwaylamb is anyone working on this? I'd like to pick this up.

@broadwaylamb is anyone working on this? I'd like to pick this up.

@Ravi61 feel free! That would be awesome.
Sent with GitHawk

is it possible to use SPM in an existing project with Cocoapods?

@haemi I don’t see any reason for it not to be possible, but I haven’t tried though.

@gal-orlanczyk @reuvenlevitsky @Ravi61 @haemi
CocoaPods is now supported 🥳