[BUG] LocalPen not loading in Safari

Currently, visiting "https://localpen.io" with the latest version of Safari, both on Big Sur as well as on the latest iOS and iPadOS, yields a never-ending loading screen:

Calling up the console on Safari MacOS, we see only 1 error - not sure whether it's helpful or not:

Visiting the site with Firefox or Chrome loads the LocalPen interface as expected - problem only appears on Safari, on all platforms.

Hi @iocouto
Thank you for reporting.
This has already been fixed.
You can test it on the development build:
https://dev.localpen.io/
The fix will be published (with lots of other features and fixes) on the next release.
LocalPen is being actively developed. The next release will be published once it is relatively stable.
Please let me know how you find it.
Thank you 😊

Can confirm: https://dev.localmente.io loading as expected on all platforms. 👍🎉


Installation may fail even with sudo due to lack of admin privileges

While following the example to create a program documented here:
https://os-doc-builder.test.mbed.com/docs/development/tools/working-with-mbed-cli.html
I ran into some warnings:
bash-3.2$ python mbed.py new mbed-os-program
[mbed] Creating new program "mbed-os-program" (git)
[mbed] Adding library "mbed-os" from "https://github.com/ARMmbed/mbed-os" at branch/tag "latest"
[mbed] Updating reference "mbed-os" -> "https://github.com/ARMmbed/mbed-os/#16bac101a6b7b4724023dcf86ece1548e3a23cbf"
[mbed] Auto-installing missing Python modules...
[mbed] WARNING: Unable to auto-install required Python modules.
—
[mbed] WARNING: -----------------------------------------------------------------
[mbed] WARNING: The mbed OS tools in this program require the following Python modules: mbed_host_tests, mbed_greentea, beautifulsoup4, fuzzywuzzy, pyelftools
[mbed] WARNING: You can install all missing modules by running "pip install -r requirements.txt" in "/Users/lingao01/mbed-cli/mbed/mbed-os-program/mbed-os"
[mbed] WARNING: On Posix systems (Linux, Mac, etc) you might have to switch to superuser account or use "sudo"
I then used sudo to execute "pip install -r requirements.txt", the following error occurred:
Collecting setuptools>=19.0 (from hidapi->pyOCD>=0.8.1a1->mbed-host-tests>=1.1.2->-r requirements.txt (line 10))
Using cached setuptools-39.0.1-py2.py3-none-any.whl
Installing collected packages: setuptools, hidapi, pyOCD, mbed-host-tests, lockfile, funcsigs, pbr, mock, mbed-greentea, beautifulsoup4, fuzzywuzzy, pyelftools
Found existing installation: setuptools 18.5
Uninstalling setuptools-18.5:
Exception:
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/pip/basecommand.py", line 211, in main
status = self.run(options, args)
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/pip/commands/install.py", line 311, in run
root=options.root_path,
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/pip/req/req_set.py", line 640, in install
requirement.uninstall(auto_confirm=True)
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/pip/req/req_install.py", line 716, in uninstall
paths_to_remove.remove(auto_confirm)
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/pip/req/req_uninstall.py", line 125, in remove
renames(path, new_path)
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/pip/utils/init.py", line 315, in renames
shutil.move(old, new)
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/shutil.py", line 299, in move
copytree(src, real_dst, symlinks=True)
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/shutil.py", line 208, in copytree
raise Error, errors
It doesn't seem "sudo" works. I finally ran "Make Me Local Admin" app and succeeded. This should be documented.

@linlingao Thanks for the detailed investigation. Do you think you could send a PR to the documentation so this is mentioned?

ARM Internal Ref: MBOTRIAGE-447


Error: "Uncaught TypeError: Illegal invocation"

When scrolling in a List component, do I get the exception Uncaught TypeError: Illegal invocation in Chrome. The error is thrown at https://github.com/bvaughn/react-virtualized/blob/9.18.0/source/utils/requestAnimationTimeout.js#L33.
My setup:
  "dependencies": {
    "babel-core": "^6.25.0",
    "babel-loader": "7.x",
    "babel-plugin-add-module-exports": "^0.2.1",
    "babel-plugin-syntax-dynamic-import": "^6.18.0",
    "babel-plugin-transform-class-properties": "^6.24.1",
    "babel-polyfill": "^6.23.0",
    "babel-preset-env": "^1.5.2",
    "babel-preset-react": "^6.24.1",
    "babel-preset-stage-0": "^6.22.0",
    "react": "^15.6.1",
    "react-dom": "^15.6.1",
    "webpack": "^3.0.0",
    "webpack-dev-server": "^2.5.0"
    ...
  }

This issue seems to be related: https://github.com/fritz-c/react-sortable-tree/issues/230

@bvaughn I got the same error. I think the reason is the fact that raf and caf from utils/animationFrame are not bound to window and lose context.

I've never seen this happen, and aside from this report, I've never heard of others seeing it either. Can you provide a repro? Better yet, can you provide a fix if you're able to repro?

I tried to recreate the issue in a repo, without any luck so far… but what i found out was that the error comes from here: https://github.com/bvaughn/react-virtualized/blob/9.18.0/source/utils/requestAnimationTimeout.js#L33, and that, when debugging in chrome, if I inspect this I get { _esmodule: false }. In my test repo where the error don't happen (yet) do this instead return undefined. So one guess it, that it tries to assign a native function (requestAnimationFrame) from window to a object which is not legal. But that is about how far I get in debugging. Might be something related to the setup of babel and/or webpack.

Yeah, that should be raf.call(window, timeout).
Reference: https://stackoverflow.com/questions/9677985/uncaught-typeerror-illegal-invocation-in-chrome

1 on this will try to code up a js fiddle of the issue.

After more investigation this is a webpack issue.  Global functions get called in the wrong context in Webpack 3. If you upgrade to Webpack 4 (you will also need to update any webpack plugins) this is fixed.
https://github.com/webpack/webpack/issues/5111
Hope this helps people and thanks for the library! @bvaughn

For those who cannot upgrade to Webpack 4, downgrading to react-virtualized 9.9.0 fixed the issue.

This also occurs in more 'seasoned' build systems like gulp (when using gulp-uglify et al). Thanks @thomaspaillot for the tip!

raf.call(window, timeout)
why is this not yet implemented in requestAnimationTimeout.js ?

Facing the same error using turbopack with next.js. It's working fine with webpack.


Change command names and redo grouping and ordering

Fixes #1848
See some screenshots here: https://github.com/Azure/azure-dev/issues/1848#issuecomment-1546014906

VSCode Extension Installation Instructions

Download the extension at https://azuresdkreleasepreview.blob.core.windows.net/azd/vscode/pr/2190/azure-dev-0.6.0-alpha.1.vsix
Extract the extension from the compressed file
In vscode
a. Open "Extensions" (Ctrl+Shift+X)
b. Click the ...\ menu at top of Extensions sidebar
c. Click "Install from VSIX"
d. Select location of downloaded file


Setting cookies query

Heya, simple questions relating to cookies.

I see in the .conf it's undef, but when I try putting the path to my cookies it brings up a regexp error on startup.
Do cookies work like they do for other apps, giving me the ability to download/play videos without an API key or logging in?

The cookie file must be a # Netscape HTTP Cookie File. Same format as youtube-dl requires.
Here's a Firefox extension that can extract the cookies in the required format: https://addons.mozilla.org/en-US/firefox/addon/cookies-txt/
Passing a cookie is only useful to overcome the 429: Too Many Requests issue. And maybe also in viewing age-restricted videos in Europe. But, other than that, there is no benefit in passing a cookie. An API key will still be needed.
See also: https://github.com/ytdl-org/youtube-dl#how-do-i-pass-cookies-to-youtube-dl
Alternatively, you can try pipe-viewer, which is a fork of youtube-viewer that does not require an API key. Passing a cookie to pipe-viewer will behave a little bit like being logged in, allowing you to see unlisted playlists from your account.


How to track the people in the video？

Greate jobs！ The jobs work fine when video just have one people. If there multiple people in the video，how can I track the one I need？ Great thx.

Are you saying given multiple people in a video, how to extract the inference results of a specific person? If so, you would have to look into object tracking for that. I believe YoloV5 has many open-source implementations so you can modify the detection portion to include tracking.  For example, if you have a video with a subject and people in the background, you could filter detections by bbox size essentially mocking "tracking" because you know the subject will be the largest in frame. Another example is if you know the subject is the first to appear in frame, you could mask out areas around the first bbox to avoid background detections in later frames. But if your videos are more complex, I would look into object tracking algorithms. Again, I am fairly certain YoloV5 already has support for that so hopefully pretty straightforward for you to implement.


[No Ticket] Disable editable install for aiohttpretty

Ticket
No ticket or ticket to be created
Purpose
Fix requirement installation by disabling editable install for aiohttpretty. This is the WB counterpart for MFR: https://github.com/CenterForOpenScience/modular-file-renderer/pull/312.
Changes

Remove -e from dev requirements, similar to the MFR PR, we don't need such editable install
Editable install creates a temporary folder which may cause future installation to hang when
docker-compose up wb_requirements finds out that the folder already exists and decide to wait for user input. Using docker-compose up, no instructions is provided to inform the user what's happening and installation just hangs
Error message by installing directly inside the container using invoke install -dObtaining aiohttpretty from
<EMAIL_ADDRESS>(from -r dev-
requirements.txt (line 3))
  Directory /code/src/aiohttpretty already exists, and is not a git clone.
  The plan is to install the git repository https://github.com/centerforopenscience/aiohttpretty.git
What to do?  (i)gnore, (w)ipe, (b)ackup

Side effects
No. If you need to modify the code on the fly, enable -e locally.
QA Notes
No. This is an update for local docker development.
Deployment Notes
No. Make sure that travis pass.

Coverage remained the same at 90.012% when pulling 701f3543d3ece232f446431051b442647b6f0be3 on cslzchen:fix/dev-requirements-install into c541ffb2ad628123544c148b792216356c5b323a on CenterForOpenScience:develop.


typo: powerbi-docs/desktop-bookmarks.md

elipses -> ellipses

@nschonni : Thanks for your contribution! The author, @davidiseminger, has been notified to review your proposed change.

Merging typo fix.

@ktoliver : Thanks for your contribution! The author, @davidiseminger, has been notified to review your proposed change.


StartStream on EventStore does not throw exception if stream already exists

I am trying to ensure that a stream does not exist yet by creating it with the StartStream method. I found that I do not get an exception even if the stream exists.
The following code snippet works without any exceptions although I would have expected one to be thrown in line 5.
public void StartStream(Guid streamId)
        {
            DocumentSession.Events.StartStream(streamId);
            DocumentSession.SaveChanges();
            DocumentSession.Events.StartStream(streamId);
            DocumentSession.SaveChanges();
        }

Am I missing something?
I'm using Marten 3.7.1, PostgreSQL 10.10 for Ubuntu, .Net Framework 4.62

@Crown0815 since you are not storing any events, in this case, it is essentially an empty stream hence does not get stored in DB even though you attempt to save changes. If you had added any events then you would see an exception ExistingStreamIdCollisionException.

@mysticmind There is a actually a stream in the database after calling the first SaveChanges(). It seems that the StartStream method does not actually check whether the steam exists but whether the stream is at a version greater than zero.
Is this intentional?

Yes, you are right, it looks like there isn't a check for Stream Id exists. We would need to add a check. Marking this as a bug. We will discuss and take it up.

@Crown0815 Quick follow on note, even though there is an entry in the db with zero events, only when a stream contains events, there would be conflict and hence checking for stream exists for StartStream is essential to throw an exception. The current implementation satisfies this requirement. So I think we can go ahead and close this unless you think a check is required. Thoughts>

@mysticmind @Crown0815 We almost wiped out StartStream() several times in the early days and just saying that AppendEvent is a de facto upsert. I think we should do the check that @Crown0815 is suggesting, but it would be breaking behavior and shouldn't be done until 4.0.

From ongoing conversations, this will be enforced in Marten v4

Duplicate with #1648


Splash effect with theme

I have been seeing an annoying splash effect when the default mode (currently night) does not match the user setting mode (most users have set this to light (as part of sign up defaults))
This is going to be the same if we defaulted to light theme and user had night theme.

Just a quick explanation: Gatsby builds a static (html) site and then 'hydrates' it with JavaScript.  The problem here is that the static site doesn't know what the user's settings are and one of them is dark/light mode.  What's happening on .dev is the default theme appears initially and then is almost immediately replaced.  If the user's settings aren't the default, they see a flash of the other colour.
I think any solution is going to have to either hide the site for a moment (spinner or whatever) before the hydration happens or use css to smooth the transition.

This is a workaround and something that'll need to be done later but what if we write a line of JS separately in the head of each page. This would fix it for now, but as I said, it's a workaround, not a fix

@raisedadead wouldn't the cookies still need js to work? If the problem is hydration, then I don't think cookies will fix it - it would still need hiding. The loader discussed in #36757 will address this, I guess, though I'm not 100% on how that's going to work.
Presumably it would go something like:
Loader (render spinner only) -> cookie parsed (render full site) -> server response (change accordingly)
Finally, as mentioned in #36734 there's already a partial solution using local storage. That should be taken out if we're using cookies.

As a side thought - night mode(or whatever is default) isn't available for people without a username or not logged in. Perhaps it could or should be taken out of the db completely and stored in browser storage with a toggle switch in the nav or something.

@raisedadead I thought a bout was hat I  wrote but it won’t wo work sonthats that
About what’s @moT01 said, I second that, I’ve been doing the same for my portfolio website too.

We are removing the support for storing theme on the user object and instead moving to store theme per device, like most sites Youtube, Twitter etc.
This would mean some users using multiple devices are affected. But I find that to be a much less of a pain than having to manage the quirks of hydration and all other nuances.
Thanks everyone for your involvement in these issues. Closing in favor of https://github.com/freeCodeCamp/freeCodeCamp/issues/36819


Add night/dark theme

Add a setting for switching between the normal (light) theme and a dark theme. I think the best way to do this is to make a night layout with name night_ for each existing layout.

AlertDialogs still look bad on night mode because I haven't figured out how to change the text color for the message (keeps being black).


Add parser for manifest.yml and integrate with  servicedeployer.

We want a parser to support the elements in the manifest files of testcase1.
I'm going to make the following changes:

utils/manifest.go will be added for manifest_full.yaml parser.
tests/manifest_test.go will be added accordingly.
remove deployment.go and test code according.
Integrate manifest.go with deployerService.
Add corresponding test cases for the integration.
Refactor and change existing code if necessary.

Done.


The service worker navigation preload request was cancelled before 'preloadResponse' settled. If you intend to use 'preloadResponse', use waitUntil() or respondWith() to wait for the promise to settle.

Background
I am trying to use a navigationPreload and a cache data by StaleWhileRevalidate strategy for a navigation request.
Problem
I am getting an error message:
The service worker navigation preload request was cancelled before 'preloadResponse' settled. If you intend to use 'preloadResponse', use waitUntil() or respondWith() to wait for the promise to settle.

and cache is not updated
Finding
I think for this case this.waitUntil is missing to wait for a response
const response = await this.waitUntil(this.fetch(input));

https://github.com/GoogleChrome/workbox/blob/0b5d9d816ab828d32021d05a808c63d17a28dbf1/packages/workbox-strategies/src/StrategyHandler.ts#L240
Question
I am doing something wrong in my code or this finding is a needed fix?
navigationPreload.enable();
self.addEventListener('install', () => {
    self.skipWaiting();
});
clientsClaim();

const routes = [
    new RegExp(`/route1`),
    new RegExp(`/route2`)
];

registerRoute(new NavigationRoute(new NetworkOnly(), {
    denylist: routes,
}));

registerRoute(new NavigationRoute(new StaleWhileRevalidate({
    cacheName: NAVIGATION_CACHE_NAME,
}), {
    allowlist: routes,
}));

Thank you so much for creating this great library.

Hello!
From reading your code, my guess is that you're seeing this with the second navigation route, the one using the StaleWhileRevalidate handler. Is that correct?
I believe that the you're seeing is logged by Chrome whenever event.respondWith() is called before the navigation preload promise settles, as this typically represents an error in your service worker's logic. In the case of using a StaleWhileRevalidate handler, it's not actually a logic error, though—the stale response is passed to event.respondWith() right away, and the preload response does end up being used afterwards to refresh the cache.
Adding a waitUntil() around that fetch() call in the StrategyHandler shouldn't be necessary, as the StrategyHandler has its own mechanism of coordinating all the promises that need to complete, and it orchestrates that via https://github.com/GoogleChrome/workbox/blob/1ec268901d6d8ac5286ec3b473d5b535e88260a3/packages/workbox-strategies/src/StrategyHandler.ts#L134
So basically, unless you're actually seeing unexpected behavior, I think the explanation for the logged message is that it's effectively a false positive, motivated by Chrome attempting to log something helpful. We can revisit this if that assessment seems incorrect.

I am getting canceled request for new cache and not updated cache on second navigation (best visible on refresh)

Okay, sorry for initially assuming that this was just logging noise. I was able to reproduce failures to update the cache during the revalidate step of StaleWhileRevalidate, when that strategy is being used for navigation requests, and navigation preload is enabled.
The fix is adding in a waitUntil() around the entire promise that performs the revalidation. It should make it into the next release of Workbox.

@jeffposnick Thanks a lot 🙂

This fix is now in the https://github.com/GoogleChrome/workbox/releases/tag/v6.5.0 release.


[docs] adding dependent tool: json-schema-faker

Hi, I'm a co-author of json-schema-faker (github, online demo). It's a tool that reads JSON schema and generates fake structures using fake generators. One of the generators is chance.js.
We would like to add such information to your README.md. Would you accept a pull request that extends your readme? If so, please tell me how would you like us to do it - I haven't seen any dependent tools section in chance.js.

Yes, I would gladly accept that pull request!
And yes, a dependent tools section in the README would be perfect!

I'd also like to add it to the main docs site, but you go ahead and add it to the README and I'll add it to the docs site.

perfect, thank you... I'm working on it...


Can I use npm instead of yarn?

I'm curious if this project works with npm as well? The package.json looks compatible and I tried to run with npm install && npm run start:dev, it seems to work fine. Is there any particular feature that only works on yarn?
I'm not asking you to change yarn to npm. I'm asking because I'm building a tutorial on Rust and WebAssembly. I've already introduced npm earlier sections (while introducing wasm-pack-template), and I'm trying to see if I can ask the student to use npm on this project as well. The tutorial is targeting complete beginners so I want to avoid introducing both npm and yarn and confuse the students.

@shinglyu yes, npm is actually probably better than using yarn but both work!


Error when setting Envelope on AMSynth

When trying to set the envelope on a AMSynth I get an error message.
This works:
var synth = new Tone.AMSynth().toMaster();

synth.set("envelope", {
	"attack" : 2,
	"decay" : 0.2,
	"sustain" : 1,
	"release" : 0.8,
});

synth.triggerAttackRelease("C4", "1n");

The following does not:
var synth = new Tone.AMSynth().toMaster();
var env = new Tone.Envelope({
	"attack" : 2,
	"decay" : 0.2,
	"sustain" : 1,
	"release" : 0.8,
});

synth.set("envelope", env);

synth.triggerAttackRelease("C4", "1n");

I get the following error:
Uncaught TypeError: Failed to set the 'value' property on 'AudioParam': The provided float value is non-finite.
    at Tone.TimelineSignal.set (Tone.js:3315)
    at Tone.AmplitudeEnvelope.Tone.set (Tone.js:197)
    at Tone.AMSynth.Tone.set (Tone.js:207)
    at app.js:...

In the docs the type seems to be wrong (Tone.Oscillator).

Thanks for the note on the docs. I've updated that. Feel free to send PRs about those kinds of issues on the dev branch. I'll gladly merge doc fixes.
As far as your first question, set is to change the settings, not to replace the envelope. This is consistent with the way set is used throughout the library. Replacing the envelope would require disconnecting the old envelope and reconnecting the new one, which is unnecessary in most situations and would probably produce unwanted side effects in many places.

Thanks, I will consider a PR next time.
What’s the preferred way to access the nodes of a Tone.js object? In case not all parameters are accessible via set?


Is it accurate for BMP to populate HarPageTimings.onLoad?

Hi,
The comments in BrowserMobProxy.java describe how the HarPageTimings.onLoad field will be populated on endHar() and newPage().

Populates the {_@_link net.lightbody.bmp.core.har.HarPageTimings#onLoad} value based on the amount of time the current page has been captured.

My question is, is it helpful that BMP sets a default value for this?
A browser can set an accurate value for this field, but I'm not sure BMP can. E.g. the onload event could fire long before endHar() or newPage() is called, giving an inaccurate value.
I suppose the consumer of the HAR could check the har.log.creator field to see if BMP created the HAR, and then choose to ignore the onLoad value. Or the HarPageTimings.comment field could be used to pass a warning to the HAR consumer.
I expected (rightly or wrongly) this field to be -1 (http://www.softwareishard.com/blog/har-12-spec/#pageTimings), but I probably haven't considered all the factors.
Any thoughts?

@gitgrimbo I think you're right about this. The HAR spec describes the onLoad field as:

Page is loaded (onLoad event fired). Number of milliseconds since page load started (page.startedDateTime)

There's no onLoad event for a proxy, but we kind of have a concept of artificial "pages", mostly because the HAR format requires it.
agree though, it would make more sense for BMP to set this to -1. I'll label this a bug, but please feel free to open a PR.


Increase time until Django times out

Would it be possible to increase the Django script timeout duration on the API server?
I am running a scripts that will take 15-30 minutes to complete. I can observe progress through an API call counter for 5-10 minutes, but they aren't completing. (The code is supposed to print results to the log when it completes.) I'd like to figure out a way to keep them running until completion.

I can't find any settings for Django to increase this timeout. I read an article that says it may work if you print some content during the process to stdout (ie to the web client.) Are you currently printing anything while this process is running?
Alternatively, is there any way to run this as a standalone script or something so we don't have to go through the Load Balancer/proxy configuration?

This should be fixed - I increased the timeout on the ElasticLoadBalancer on Amazon AWS.


Error if passed subject or session name not found during data transfer

Currently if a subject or session is passed to upload() or download() and does not exist inthe source directory it will just be skipped. It would be better to check all exist before starting transfer, in case the user made a typo.

This is actually not trivial at all in the current implementation because if the sub / ses is specifically specified we just assume it exists on the local / central. If we don't, central has to be polled just to check it exists which is fiddly and takes up time. For now, don't do this unless people report it is causing problems.


Is there a confidence threshold on proposals?

Hello,
I know there is a confidence threshold on the final detections. Is there a similar one for object proposals to separate background from objects?

If I understand correctly, the threshold you refer to is test_cfg.rcnn.score_thr
https://github.com/open-mmlab/mmdetection/blob/d3cf38d91c454b1a6881e8c36c1e4a66dc5521b8/configs/_base_/models/faster_rcnn_r50_fpn.py#L97-L108
which is used here to filter boxes with low scores https://github.com/open-mmlab/mmdetection/blob/d3cf38d91c454b1a6881e8c36c1e4a66dc5521b8/mmdet/core/post_processing/bbox_nms.py#L41
There is no similar step during proposal generation for RPN.

@yhcao6 I am referring to objects that are output of RPN and before they go to RCNN. So basically the object proposals, which are classified as objects or background from the RPN. These should have a confidence of belonging being an object (no specific class) or background. I believe you mean the same thing and it looks like the threshold of 0.05 is applied to them before they go into the RCNN. Please correct me if I am wrong.

No, they are different. test_cfg.rcnn.score_thr is exactly the confidence you mentioned I know there is a confidence threshold on the final detections. There is no such filtering step for object proposals that you mentioned objects that are output of RPN and before they go to RCNN. So basically the object proposals, which are classified as objects or background from the RPN. You can find the detail in this method:
https://github.com/open-mmlab/mmdetection/blob/d3cf38d91c454b1a6881e8c36c1e4a66dc5521b8/mmdet/models/dense_heads/rpn_head.py#L78-L86

Maybe the object detection API of Tensorflow can help you solve your problem.
I remember there is a nms_score threshold parameter of faster rcnn second stage in its config file.


Else content is ignored

if(month) Month
else Day

will compile to
{{#if month}}
   Month
{{else}}
{{/if}}

resolved


Authentication ideas and services

I've noticed notes about authentication on https://github.com/witheve/Eve/blob/master/design/io.md#authentication-and-identity and I think the project shouldn't be dependent on a proprietary service like AuthRocket. There are very good open-source options:

Keycloak (Identity and Access Management for Modern Applications, Services and APIs, login service)
JOSSO 2 Identity Management, Access Management and Single Sign-On system

In case proprietary service will be needed (hopefully not) , I can recommend Atlassian Crowd identity mgmt (it should be free for open-source, and non-profit projects and for school classrooms).

How does the authentication look like in the end? How was the issue of dependency on a proprietary service solved?

dumblob,
We currently don't have a strategy for authentication -- the core of the system has changed so much since we used AuthRocket, that we have to revisit the fundamentals of what authentication is for our system. I appreciate your sentiment about openness, though, and I believe you are right that an open source project should not depend on a proprietary service. When we revisit the issue, I will keep that in mind.
Corey

Thanks for explanation.


Introduce "buckets" to DebugGroup counting

From experience with counting our uacs implementation, I want to propose the following:
We introduce the notion of "buckets" into DebugGroups. From the user's perspective, the flow should look roughly like
group.setBucket("user");
doStuff();
group.setBucket("provider");
doOtherStuff();

group.print("user"); //prints count of operations that happened in doStuff()
group.print("provider"); //prints count of operations that happened in doOtherStuff()

This would allow the user to effectively count operations done in interactive protocols.
More specifically/additionally:

Counting becomes a static thing. I ran into the issue that I had two copies of DebugGroup (one was created during deserialization of an internal instance of PSSignatureScheme), so the count was split between two objects (I only noticed because the count seemed somewhat off). Ideally, this whole thing becomes a bit more robust w.r.t. serialization.
My quick thought: A bucket is (an object with) a map Group -> (numSquarings, numExps, numPairings, ...), i.e. a bucket counts the number of operations per Group. A bucket is stored statically in DebugGroup (probably: static Map<String, Bucket> as a name -> bucket map). This takes care of the static counting thing from the previous bullet point "for free".
To implement the two ways of counting, either we store two buckets for each bucket name (like type1bucket: Group -> Bucket and type1bucket: Group -> Bucket become static members of DebugGroup) or each bucket contains the two types of count.

Additional suggestions for improvement:

The count output is somewhat difficult to read right now. I think it would be cool if we could have a "at a glance" tree-like structure:

BucketName
    Pairings: 6
    G1
        (Costly) Operations: 400
            Mul: 300 (290 of which happened during (multi-)exp)
            Square: 100 (100 of which happened during (multi-)exp)
        Inversions: 170 (168 of which happened during (multi-)exp)
        Exponentiations: 5
        Multiexponentiations (number of terms): [2, 2, 5]
        getRepresentation() calls: 2
    G2
        ...

This also accounts for numbers people will probably want to put into their papers. For example, often, we don't have to space to differentiate op and square, so we just show the sum of them. Inversions are basically free, so we'd just disregard them usually.

Good idea! I agree with your implementation suggestions. Some additional technical considerations:
We will some kind of default bucket that is used whenever the user does not want to explicitly specify a name. Basically would be used whenever no distinction between different protocol parties or so is necessary. I would store that one separately from the hash map used for the named ones, (to avoid having to choose an internal name that might conflict with a name chosen by the user). I would then have setBucket() and print() methods (without arguments) that enable the default bucket and print its results, respectively.
We also need to consider the (multi-)exponentiation algorithm selection features. These are specific to the LazyGroup instances and are not static and do not get serialized. For nicer interaction with the serialization it would probably be better if the chosen algorithms and window size options got serialized as well. This would increase the LazyGroup serialization size, however, which is not great. We could also store the selected algorithm and window size inside the bucket; then we can adjust them whenever the bucket is changed. This latter approach interacts with the bucket stuff the best, I think.
Also, do we want all the buckets to be serialized together with the DebugGroup? In your Benchmark this is not necessary since the static variable is shared across all instances of the class in your JVM, but whenever the serialization is actually sent to another system, the static variable won't be present. I could see this coming up in some kind of integration test where you are counting operations and testing the networking aspects at the same time.
I think serializing the buckets has some advantages: Imagine you have such a protocol. Then you are sending the DebugGroup between parties, and at the end the last party has all the combined results locally present (you don't need to combine the results from each party).
You do have to consider what happens if a bucket is already present when serializing a DebugGroup instance (e.g. the local party has bucket "user" and the deserialized instance has bucket "user"). I think the most sensible approach here would be to merge the two buckets, i.e. add up all the numbers. Then we get the desired effect of adding up all the numbers at the last party.
Only problem maybe is the multi-exponentiation algorithm stuff if we store those in the bucket as well (how do you select between those? I would probably give precedence to the locally selected algorithm).

Default bucket: very good idea. I'd say that one should probably always count (even when a bucket is selected manually), so that we always have a "total" count.
Storing the window sizes in the bucket feels strange. It's kind of cheating to change window sizes between invocations of stuff if the "real" (non-counting) code doesn't do that. I agree with the general issue, though. But this may be an unrelated issue. Changing window sizes for "normal" LazyGroups has the same issues if serialization is involved. Maybe that's another issue.
Bucket serialization: I'm unsure. For example: if I serialize my group and deserialize it on the same machine, the counts would double?!. Feels dangerous and hard to convey the meaning for this. I'd say we keep the semantics to "everything that happens on this machine in DebugGroups will be counted".
Oh, one more thing: I think it would be cool, if we could have constructors for the DebugGroup that takes a BigInteger size. That would make it easier to model, for example, Mcl group's size. I had to look up how the "security parameter" is interpreted.

Default bucket: very good idea. I'd say that one should probably always count (even when a bucket is selected manually), so that we always have a "total" count.

Interesting. My initial thought would be that the "total" one should be something separate from the default one. The reason being that then you can use the default bucket just like the other ones, i.e. for your protocol benchmark you might use the default bucket for the user and a named one for the provider. In terms of functionality it doesn't really matter, but I wonder which way is more intuitive: Separating them seems maybe easier to explain? E.g. "There is a default unnamed bucket which counts operations. You can switch to separate buckets using a unique name. Oh and there are also methods that automatically sum up all the counts across the buckets for you" vs "There is a default unnamed bucket which counts operations. You can switch to separate buckets using a unique name. The default bucket also includes the counts of operations across all named buckets.". The former seems a bit more intuitive to me since it reduces the responsibility of the default bucket, i.e. it is just another bucket, just without a name. For the latter you always need to remember that it counts total operations too (additional complexity).

Bucket serialization: I'm unsure. For example: if I serialize my group and deserialize it on the same machine, the counts would double?!. Feels dangerous and hard to convey the meaning for this. I'd say we keep the semantics to "everything that happens on this machine in DebugGroups will be counted".

Right. I didn't consider that. I agree.

I just noticed that having the default bucket be unnamed provides some issues with the printing and getter methods for the results. Its best if those take in the name of the bucket which you want to get the data from. Hard to do that for an unnamed bucket (and having separate getters for the default bucket results in too many methods). So I'm thinking I'll just call that bucket "default" and have it act as a normal bucket.

I'd like a "total" bucket that is always actively counting. Maybe that's in addition to the "default" bucket.
My thinking was that if you never call setBucket(), "total" or "default" is just the same thing.
If you do call setBucket (or whatever it's called), then in the end the output would just contain an additional "total" entry (in addition to your self-defined buckets). And that may be useful.

In the "total bucket is the default bucket" world, it's probably even easier to explain (because the output explains it clearly by labelling the thing "total"). It may be slightly less natural to have a "default" bucket output?!


Add reborrow functions to Container

Add reborrow and reborrow_ref functions to Container to shorten the lifetime of the GATs.

Turns out this might actually not be needed for what I had in mind. Might still be useful, tho?


provide guideline or mechanism to resolve conflicts in DVC files on merge

When we change something (let's say a stage command) in one branch (experiment) and want to propagate this change to the derived branch it's easy to get a merge conflict in a DVC file.
Usually merge strategy theirs (git merge -X theirs) do the job.
We need to think if it's always the case. We should specify a guideline how to resolve these conflicts. If there are some ideas how to avoid them, let's discuss this. Right now it's frustrating and unclear how to manage these conflicts.

When we change something (let's say a stage command) in one branch (experiment) and want to propagate this change to the derived branch it's easy to get a merge conflict in a DVC file.

For small changes, I think that cherry-pick would be better and safer than a merge.
From what I have seen (and I have zero ML experience, I have just read some DVC tutorials), we merge a branch A onto a branch B when we want to completely overwrite B with A, for example when we merge the branch with the best metrics onto branch master. In this case it is safe to use the merge strategy theirs. Then, after making as well a dvc checkout, all the data and pipelines will be up to date (or at least most of them).

I think this is done! See https://dvc.org/doc/user-guide/merge-conflicts
Closing for now.


Is the Wiki still being used for documentation?

I was going to add some solutions to common build issues to the main readme, (mentioned here: #934), but I'm not sure if it would be better to have them in the wiki?
I noticed it hasn't been updated since April, is this because you can't make pull requests to the wiki?

Yeah that needs to get updated. I think there's a plan to generate the documentation straight from the code base via the xml.

Updated Wiki


Unused imports removed

Remove unused imports

Some tx_builder.rs tests weren't unwrapping add_mint() calls to check for Err case

Some other unused function/variable (outside codegened code) warnings fixed too

Besides that one protocol magic thing this is just removing imports and a couple fixes for warnings so we should be able to merge this quickly.

@gostkin  Besides that one protocol magic thing this is just removing imports and a couple fixes for warnings so we should be able to merge this quickly.

@gostkin I'm just going to merge this so I can put up more PRs for multi-era without any merge conflicts. Let me know if there's anything changed here that needs changes. The behavior didn't really change so it's probably fine.


ImagePicker get exif metadata

Hi, i'm using the plugin:
https://github.com/flutter/plugins/tree/master/packages/image_picker
for get images and photo. I'm using it also with maxWidth and maxHeight.
Is possible have the exif metadata before the resize?
How can i do it?
Thanks

What info do you specifically need?
There's no currently an API for this, but I'm interested in implementing it.

I need:
Make, Model, FNumber, SpectralSensitivity, ExposureTime, FocalLength, GPSLatitude, GPSLongitude
I also wrote this flutter plugin:
https://github.com/romaluca/exif_flutter
(it is a bad port of this:  )
It works but only when i not specify maxHeight or maxWidth in image picker

Ah, I see.
The EXIF data is lost since when you use maxWidth or maxHeight, a scaled version of the image is created and saved on the filesystem. This is so that we don't actually downscale the original picture in the users gallery, which might result in angry users.
So, in the case of maxWidth or maxHeight being used, the EXIF data should be copied from the original image to the scaled one. This needs a new version of the image_picker plugin.

I fixed it. #866
Please check if you are fixing what you are imagining.

I've written notes about how to add exif on iOS/Android to: https://github.com/flutter/flutter/issues/10533#issuecomment-441719073 . I suspect all of these exif bugs are the same root cause and easy to fix.

I am experiencing the same issue after resizing the image on android, exif data is lost (can confirm it atleast for orientation). Any news on this?

Still experiencing this issue with image_picker: 0.5.0+3

I'm fixing this in https://github.com/flutter/plugins/pull/1586
I will create a separate PR to copy GPS data after this PR is merged in

flutter/plugins#1586 is merged, you can use the latest version. Image picked now has the EXIF data copied from the original photo.


fix: redact sensitive req/response headers by default

fix #468
Redacts some of the well known sensitive headers and other Access token related keys that might be accidentally captured by the agent when capturing network information.
Cookies are already filtered out by Playwright, But we still get rid of Cookie and Set cookie for the request/response cycle to be on safer side.

LGTM 🎉
Testing went well:
before the PR changes screenshot

after the PR changes screenshot


More feature flags

Codecov Report

Merging #44 (5770a18) into master (b890ce7) will increase coverage by 0.32%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #44      +/-   ##
==========================================
+ Coverage   89.42%   89.74%   +0.32%
==========================================
  Files          54       56       +2
  Lines         832      858      +26
  Branches      144      144
==========================================
+ Hits          744      770      +26
  Misses         88       88

Impacted Files
Coverage Δ

...s/insights-common-typescript/src/utils/Insights.ts
100.00% <ø> (ø)

...on-typescript/src/components/Beta/BetaDetector.tsx
100.00% <100.00%> (ø)

...escript/src/components/FeatureFlag/EnvDetector.tsx
100.00% <100.00%> (ø)

...typescript/src/components/FeatureFlag/RenderIf.tsx
100.00% <100.00%> (ø)

...insights-common-typescript/src/components/index.ts
100.00% <100.00%> (ø)

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 31e3684...5770a18. Read the comment docs.


I want to distribute multiple versions.

Describe the solution you'd like
Access to each version is inconvenient if you want to distribute multiple versions in install.appcenter.ms.
There is not always one application version under development.
Describe alternatives you've considered
It would be nice if install.appcenter.ms had the ability to filter by version.

on the project im working on i think the previous devs had the same issue. what they did was that each version is a completely new app on appcenter.

That's right.
My team has created develop1, develop2, develop3 in AppCenter and assigned versions.
However, that is inconvenient.


"hash" transformer or "hash" tera filter

Hello and thank you for this tool. It works pretty neat. <3
My goal is to dump anonymized version of sensitive data in such away that the value can be verified, given the original value. It is a common usecase in our place to hmac(secret, value) or just sha(salt + value) to hide the field in a dump or log a pseudoanonymized value.
Generating random values works in many places, but we need either a hash transformer or a tera filter capable for generating such hashes, that are 1 => 1 associations.
I went ahead and implemented a sha256 transformer and then looked for tera filters to do the same for comparison. (But this is not working as simply as the transformer Trait)
I would like to open a PR and get some review done. What do you think?

Hello! Yes, of course, feel free to open the PR.

With pleasure! Would you prefer it as tera function or transformer? I have both by now.

Let's start with the function.

cant push a branch to this repo @evgeniy-r

Push to your fork and create a PR from your fork, please.


ugettext* deprecated in Django 4.2.6

Hi all
Trying to install openuds 3.6 in Ubuntu 20.04 ( python3.8 ... expect no problems )
"pip install Django" downloads and installs version 4.2.6
When running python manage.py sync get multiple errors about lack of ugettext* functions.
surfing in the web found that these functions are now deprecated:
https://code.djangoproject.com/ticket/30165
So had to manually surf across source code an replace ugettext* instances to gettext* to get no related errors.
Is this ok, or is there a better way to deal with?
Thanks in advance.
Juan Antonio
BTW: writing a tiny install_howto to avoid me getting lost :-)

Right, UDS 3.6 is compatible with Django 3.2.x only
If you look at the "requirements.txt" file on server folder, you will see that 3.6 is compatible with Django 3.2.19 (well, i have currently upgraded it to 3.2.22). 4.0 will use Django 4.x (or maybe 5.x, if it is already in stable stage when finished 4.0 :) ).
Note that currently "master" (will be 4.0) version is not usable, it's under heavy changes (refactoring, new service models, ...), and its not ready for anything right now.
Also, i have added Pillow dependency version for 3.6 (due to some constants being removed...) to 9.5.0.
Hope this helps:
Note:
Look here, should have all dependencies if i'm not wrong :).
https://github.com/VirtualCable/openuds/blob/v3.6/server/requirements.txt
Any install help would be appreciated, we can include it somewhere so it's easier for next one to install it... :)

Thanks.
Downgrade Django to 3.2.19 and problem solved


Allow passing an array of refs to useClickAway

Is your feature request related to a problem? Please describe.
For my use case I have a button that opens a modal. Once the modal is open, I want clicking away to close the modal, but if I click on the modal toggle button, I get two conflicting commands - from the modal useClickAway event and from the button's onClick event.
Describe the solution you'd like
If I could pass an array of refs then I could prevent the behavior described above by passing a ref for the modal as well as the button.
Describe alternatives you've considered
I had tried making the modal toggle button only fire when the modal is closed, however that still ran into a race-condition, even when setting the mouse events to ['mouseup', 'touchend']. Another alternate solution would be to use a setTimeout, however I'd prefer to not use that.

Oh man, I can't tell you the number of times this has happened to me outside of this library! I'll try to fix this weekend, feel the pain!

@blnk-space any luck? I've run into a second instance where this would be nice: I'm using a portal and need to ref the portal as well. May try to timebox a solution tomorrow

PR: https://github.com/streamich/react-use/pull/1125

For those who are looking for a solution to this problem
You can check if your dropdown is already open and then close it
Example:
Wrong:
const [isDropdownOpen, setIsDropdownOpen] = useState(false);
const ref = useRef();

useClickAway(ref, () => setIsDropdownOpen(false));

Correct:
const [isDropdownOpen, setIsDropdownOpen] = useState(false);
const ref = useRef();

useClickAway(ref, () => isDropdownOpen ? setIsDropdownOpen(false) : null);

I'm facing exactly this problem and it always takes like 3-4 additional internal state checking just to close a modal correctly, especially when I have to move modal to portal.
Really looking forward to the PR.
Many thanks.

I have a similar use case, and for now, I'm using a ref on the element to ignore to work around it.
If the element contains children you'll need to prevent clicks on them, for example using pointer-events: none.
const [isSidebarOpen, setIsSidebarOpen] = useState(false);
const navRef = useRef(null);
const navButtonRef = useRef(null);

useClickAway(navRef, (event) => {
  const clickedNavButton = event.target === navButtonRef.current;

  if (!clickedNavButton) {
    setIsSidebarOpen(false);
  }
});

Hi, everyone. I added this feature to useClickAway (https://github.com/streamich/react-use/pull/2593)


Very Long Chains

Hello, thank you for the amazing library!
I'm just curious why the grammar is creating such a long chains of contexts. I have a simple expression 1+1 and this is just a snapshot of the tree (I just visualise children of each context):

You can see how many contexts are built in order to reach the arithmetic expression with many redundant nodes in between. I worked with java-ast and there were no redundant nodes.

The actual python parsing is all antlr (grammar and library), so it might be better for discussion over there. https://github.com/antlr/grammars-v4/issues/2251 seems to discuss the need to optimise the grammar to avoid this sort of chaining, for example.

How did you even visualize this snapshot? I can't figure out how to use this library.

@genderev I used my own visualisation code for this using vega. This library just builds the AST, nothing else.


CB-15245:Fix NPE in create-aws-cluster when encryption type set to null

PR created to master : https://github.com/hortonworks/cloudbreak/pull/11840/files

Please see my comment https://github.com/hortonworks/cloudbreak/pull/11842#issuecomment-983944308 and close this PR.


[request]: fix awscli/awsebcli botocore conflict

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Tell us about your request
When installing the latest versions of awscli and awsebcli, I'm getting a version conflict of botocore
pip3 install --upgrade --user awscli
pip3 install --upgrade --user awsebcli
ERROR: awscli 1.17.2 has requirement botocore==1.14.2, but you'll have botocore 1.13.50 which is incompatible.

Are you currently working around this issue?
Workaround is to install an old version of awscli
pip3 install --upgrade --user 'awscli<1.17' awsebcli
Or use something like virtualenv/pipenv

Thanks for the workaround.
I had gotten all this set up back in December and right after January 1 it got all messed up and this finally got me up an running again!

Closing this issue as awsebcli 3.17.1 has been released with a fix for this conflict.
Thank you for reporting this!

If this happens again in the future, a very easy workaround is using the pipx tool, just run pipx install awsebcli

@halcarleton this issue is re-introduced with latest awseb and awsebcli versions. will you be so kind to introduce a general fix for future releases instead of hotfixing this? Maybe, talk to "the other" team and produce a consensus?
awsebcli 3.19.0 requires botocore<1.18,>=1.17, but you'll have botocore 1.18.7 which is incompatible.
awscli 1.18.14 requires botocore==1.15.14, but you'll have botocore 1.17.63 which is incompatible.

same:
ERROR: awsebcli 3.19.3 has requirement botocore<1.20.0,>=1.19.0, but you'll have botocore 1.20.5 which is incompatible.

And again:
...
[pipenv.exceptions.DependencyConflict]: The operation failed... A dependency conflict was detected and could not be resolved.
ERROR: Cannot resolve botocore (1.20.15) due to conflicting parent dependencies:
        awsebcli => botocore (>=1.19.0,<1.20.0)

Just use the pipx tool
https://pipxproject.github.io/pipx/

Thanks for the suggestion, but I was actually hoping for a proper solution from AWS. No offense, but I'm not really looking for yet another pip<something> just to install AWS stuff that should simply work, out-of-the box, with plain old pip.

... and yet again: my working Pipfile.lock has awsebcli version 3.19.3, with urllib3 version 1.26.3, but I cannot update:
ERROR: Cannot resolve urllib3 (1.26.5) due to conflicting parent dependencies:          awsebcli => urllib3 (>=1.25.4,<1.26)
How can this be? awsebcli 3.19.3 was installed with urllib3 1.26.3 before, but now, all of a sudden, urllib3  must be <1.26?
This is forcing me to freeze awsebcli, boto3, botocore, and urllib3 in the Pipfile.
The "highly recommended" EB CLI installation scripts provide an inconvenient workaround. That is not a solution. So why is this issue closed?
As (one of) the largest cloud computing service provider(s) in the world, I would expect AWS to be able to fix this properly.

Same issue again when updating...

Just adding to this thread to suggest this issue should not be closed. Ran into this dependency conflict again today when deploying my Django application.

this issue is still unresolved after 3 years....

hi guys - can someone look into that?

This should be resolved. Dependabot is updating our botocore library version in lock step with the same change being made with the awscli. However, if you install the two cli's days apart, you may run into this issue since the package is updated daily.

@jsheld if i look at aws-elastic-beanstalk-cli and aws-cli i see that they use different versions of botocore still:
latest [aws-elastic-beanstalk-cli](https://github.com/aws/aws-elastic-beanstalk-cli/tree/master) requires botocore>1.23.41,<1.29.124

https://github.com/aws/aws-cli/blob/develop/setup.cfg is on botocore==1.29.133

https://pypi.org/project/botocore/#history ... is on 1.29.133

Thanks for clarifying. I was scoping this to the awscli and ebcli versions on GitHub.

@jsheld if i only look at pypi:
https://pypi.org/project/awsebcli/ ... awsebcli 3.20.6, requires: botocore>1.23.41,<1.29.119
https://pypi.org/project/awscli/#history ... awscli 1.27.134, requires: botocore==1.29.134
https://pypi.org/project/botocore/#history ... botocore 1.29.134

As of today, the issue still persists as
awscli 1.32.7 requires botocore==1.34.7
awsebcli 3.20.10 requires botocore<1.32.0,>1.23.41
What is the correct botocore version that is compatible with both?

Still getting ERROR:
pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
awsebcli 3.20.10 requires botocore<1.32.0,>1.23.41, but you have botocore 1.35.8 which is incompatible.


Improvement: detailed circuit broken errors

This PR creates a CircuitBrokenError class which has a more detailed error message and puts detailed request stats on the returned error object.
Example CircuitBrokenError:
{
    message: 'Circuit has been broken - The percentage of failed requests (60%) is greater than the threshold specified (50%)',
    totals: {
        failed: 60,
        timedOut: 0,
        total: 100,
        successful: 40
    }
}

Coverage remained the same at 100.0% when pulling 0d769511cc0c997f5d72c64f05a6f4cbceb33f12 on improvement/detailed-errors into c8b5aa74c7fd9883f637bdb0575a97cca394720a on master.

Coverage remained the same at 100.0% when pulling 0d769511cc0c997f5d72c64f05a6f4cbceb33f12 on improvement/detailed-errors into c8b5aa74c7fd9883f637bdb0575a97cca394720a on master.


test: add custom id's to e2e tests

Signed-off-by: srivickynesh<EMAIL_ADDRESS>Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Issue ticket number and link
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] 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
Checklist:

[x] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have updated labels (if needed)

The current supported Mapping is 1 test case = 1 source file.
For having Describe or contexts as a separate case, Modifying the test case generator or putting every test case in a separate file are the options.

Hi @Dannyb48 and @srivickynesh, Sri is right - the current version of the polarion test case generator expects there is only one describe in source file - so 1 test case per file. This way are written tests for HAS.  Right now I am trying to add logic to have for example BuildSuiteDescribe and under it Describe to have multiple test cases in one source file distinguished by Describe - like it is in build.go.
In chains tests I am kind of struggling to find multiple test cases, since there is only one Describe and for example I do not know, if this Context is test step or test case: https://github.com/redhat-appstudio/e2e-tests/blob/main/tests/build/chains.go#L34 ?
In my opinion we should use Describe(ChainsSuiteDescribe and so on) to distinguish between test cases. The Context and When can be used in test steps: Context + When + It  - will be the test step. WDYT?

Hi @srivickynesh   ,
the CI configuration in main branch of e2e-tests repository was recently updated and it no longer uses pre-kcp branch of infra-deployments for bootstrapping appstudio on OCP cluster. Now it uses main branch, which leverages kcp.
Since appstudio/hacbs services are not fully integrated with kcp yet, lots of tests implemented in this repo can't be run successfully against it.
To address this, a branch "pre-kcp" was created in e2e-tests repo and can be used for adding/updating tests that can be run against "pre-kcp" version of appstudio.
Once we are able to run e2e tests for all appstudio/hacbs services successfully, the plan is to merge back all changes done in pre-kcp branch back to main.
If your PR is intended to update test used for "pre-kcp" version of appstudio, please update the base branch in your PR like this:

Thanks!

/test redhat-appstudio-e2e

Closing this PR now. Will reopen once we have a more generalized way of defining the tests in e2e-tests.


fix panic when logging a nil pointer to a Stringer

jsonFormatter.go only checks whether the val is nil, but if val is a pointer, it's been cast to interface{}, and val == nil no longer works.  See PR for failing test case.

See PR #25


BasicRealtimePlotter hangs badly if COM port not found

This is probably obvious, but the BasicRealtimePlotter hangs very badly if the COM port can't be found (or maybe it is because there was nothing connected to the port). On OS X it really locks up, the user is unable to close the window normally and it is necessary to "Force Quit" the application in order to get the Mac running smoothly again.
It would be nice if the application failed a bit more gracefully. Maybe use try{}catch{}
These are the reported errors:
RuntimeException: Error opening serial port /dev/tty.usbmodem1411: Port not found
RuntimeException: Error opening serial port /dev/tty.usbmodem1411: Port not found
RuntimeException: Error opening serial port /dev/tty.usbmodem1411: Port not found
Could not run the sketch (Target VM failed to initialize).
For more information, read revisions.txt and Help → Troubleshooting.

Thanks, sounds like a good improvement. I don't have time to fix this myself, but I generally accept pull requests.


beta: discount factor vs. discount rate

beta is in many cases referred (correctly) to as "discount factor", but sometimes referred to as "discount rate". The usage should be consistent.

Thanks @oyamad, indeed beta should be the discount factor.
@AnjuJoon , could you please change any instance of "discount rate" to "discount factor" in this lecture?

@jstac I will make the required changes


Helper streams require get

If invokeHelper is to return a stream, we must ensure that stream always has a value generated. Sometimes a stream's compute function may yield a block instead of generate a value.
This forces compute to happen once.
There is probably a better way to achieve the base goal here. For example, I should likely only link nodes that a new-style Ember.Helper, where they cannot interact with yield etc.
See also: https://github.com/mixonic/ember.js/pull/8

LGTM

Cleaned this up to only call the value hook when the value is used. Ensuring that a helper using internals (yielding blocks etc) will be run is the responsibility of invokeHelper in Ember.


History change

Description
Added support for the new version of the history package i.e - 5.x.x.
Fixes # https://github.com/quintype/quintype-node-framework/issues/220 and https://github.com/quintype/ace-planning/issues/157
Dependencies # NA
Documentation # NA
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

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] Locally with the beta version

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[x] New and existing unit tests pass locally with my changes

"history": "^4.9.0", is mentioned on react-router, this might break.

Rejecting PR


Raima Rdm support

What version of Flyway are you using?
4.1.1
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin, SBT plugin, ANT tasks)
Command-line
What database are you using (type & version)?
Raima rdm
What operating system are you using?
Windows
What did you do?
(Please include the content causing the issue, any relevant configuration settings, and the command you ran)
In our company we were using flyway to manage db migrations on Postgres database. Now we are migrating to a proprietary db: Raima Rdm http://raima.com/raima-database-manager. It supplies a properly Jdbc driver.
I’ve tried to launch flyway migrate but it can’t support raima rdm database.
Because I would like to continue to use flyway for my purpose, could be Raima supported in the future?
What did you expect to see?
I expect flyways can support this db
What did you see instead?
ERROR: Unsupported Database: Raima Database manager 14.0

Closing due to lack of interest.


Expose functions directly through the endpoint

For feature requests, please fill in the next sections:
What feature are you missing?
I saw a FR a while ago about exposing functions directly as endpoints (can't find it anymore, please link it if found). This is useful for any project that now requires an external webtask, because of a callback URL, or the file-proxy for example.
As function exexution has now moved to AWS, and the project file has matured a lot, I think the steps to achieve this now are minimal.
How could this feature look like in detail? Tradeoffs?
I would propose adding another function type to the project definition, something like endpoint or something (needs a better term). Those functions would be available directly through https://api.graph.cool/simple/v1/c......./functionName. They probably also need a HTTP method (get/post).

Closed as duplicate of #354.


Free fall take-off

Do any of you have any idea if it is possible to add a support for free fall take-off? The app has this feature, so it should be also possible to access/implement. Any kind of help would be appreciated.
Thanks!

I can help with the implementation, but I will need a bit more details about the BT-command that is being sent.

Hi,
I'm working on a fork implementing some more commands and I have autotakeoff working; at some point I plan to upload my code, but in the meantime, here's the buffer for the freefall take off:
new Buffer([0x02, ++this.steps.fa0c & 0xFF,
0x02,
0x00,
0x05, 0x00,
0xFF])

Hi, naus3a! Thanks a lot!

no problem :)

Anyone able to implement this already?

you just need to send the buffer in my last post in the same way the other BT messages are sent :)

@naus3a if you've already got this working, please submit a PR!

I have a fork with some project specific modifications; I'll extract the generic parts and make a PR out of them


eslint server shuts down

Checking the output channel reveals:
[Info  - 5:04:27 PM] ESLint server stopped.
[Info  - 5:04:28 PM] ESLint server running in node v8.9.3
[Info  - 5:04:28 PM] ESLint server is running.
[Info  - 5:04:29 PM] ESLint library loaded from: c:\Users\alexa\kapla3\node_modules\eslint\lib\api.js
"process.env.NODE_ENV" must be "development" to use this webpack config
[Error - 5:04:31 PM] Server process exited with code undefined. This usually indicates a misconfigured ESLint setup.
[Error - 5:04:32 PM] Connection to server got closed. Server will not be restarted.
[Info  - 5:04:32 PM] ESLint server stopped.

This happens every time I open the project immediately after opening the first js file.
Running cross-env NODE_ENV=development eslint --cache --format=pretty . on the command line works nicely.
What could I be doing wrong?
This could be related to https://github.com/Microsoft/vscode-eslint/issues/589
This is my package.json file: https://github.com/barbalex/kapla3/blob/f6cf167b503175f9ee8b31bff80329d347355d1c/package.json
More info:

Windows 10 (up to date) and MacOs Mojave 10.14.2 (happens on both)
project created using electron-react-boilerplate (https://github.com/electron-react-boilerplate/electron-react-boilerplate)
vscode v1.30.0

I am pretty sure that you might use a ESLint feature that requires a newer version of node. By default the eslint extension uses the node that ships with electron. Can you try to configure a different node version using the following setting: eslint.runtime. Best point it to the node version you use when running eslint on the command line.

Not sure if I set this correctly. I tried:

on Windows: "eslint.runtime": "C:\\Program Files\\nodejs\\node.exe"
on MacOS: usr/local/bin/node

But in both cases the crash still occurs.
The mode version I use was 10.13. I just now updated to 11.4. I see that VsCode uses v8.9.3.

O.k., the setting worked. I now get this error:
[Info  - 5:26:37 PM] ESLint server stopped.
[Info  - 5:26:37 PM] ESLint server running in node v10.13.0
[Info  - 5:26:37 PM] ESLint server is running.
[Info  - 5:26:38 PM] ESLint library loaded from: c:\Users\alexa\kapla3\node_modules\eslint\lib\api.js
"process.env.NODE_ENV" must be "development" to use this webpack config
[Error - 5:26:40 PM] Server process exited with code undefined. This usually indicates a misconfigured ESLint setup.
[Error - 5:26:41 PM] Connection to server got closed. Server will not be restarted.
[Info  - 5:26:41 PM] ESLint server stopped.

and after updating to node v11.4:
[Info  - 5:49:51 PM] ESLint server stopped.
[Info  - 5:49:52 PM] ESLint server running in node v11.4.0
[Info  - 5:49:52 PM] ESLint server is running.
[Info  - 5:49:53 PM] ESLint library loaded from: c:\Users\alexa\kapla3\node_modules\eslint\lib\api.js
"process.env.NODE_ENV" must be "development" to use this webpack config
[Error - 5:49:55 PM] Server process exited with code undefined. This usually indicates a misconfigured ESLint setup.
[Error - 5:49:56 PM] Connection to server got closed. Server will not be restarted.
[Info  - 5:49:56 PM] ESLint server stopped.

As the error mentions misconfigured ESLint setup as probable cause: here is my config: https://github.com/barbalex/kapla3/blob/56ab81520ad8ad6310cf5864bf277eda11d5d465/.eslintrc

@barbalex what happens if you run the eslint command directly in the terminal. Will that work?

Yes:
PS C:\Users\alexa\kapla3> yarn lint
yarn run v1.12.3
$ cross-env NODE_ENV=development eslint --cache --format=pretty .
Done in 3.42s.
PS C:\Users\alexa\kapla3>

@barbalex thanks. Can I easily reproduce this using the https://github.com/barbalex/kapla3 repository. If so can you provide steps on how to do so?

I think these should work:

clone https://github.com/barbalex/kapla3 repository
yarn (or npm i) in it's folder
open a component in vscode => watch output in eslint channel
run yarn lint on the command line to check if that works (does for me)

@barbalex this is caused by one of your scripts that are loaded into the ESLint server. It is https://github.com/barbalex/kapla3/blob/master/internals/scripts/CheckNodeEnv.js which calls process.exit(2) which then shutsdown the ESLint server. Fix should be to simply set the required env variable :-)

By the way (though I am probably being unfair to Microsoft here): After 25 years of using Microsoft products this was probably the first instance of useful help I got from Microsoft.
This may sound negative but I take it as a great sign of what has been changing at Microsoft since Nadella is in charge.

Thanks for the nice comment. I appreciate it. Have you considered using TypeScript for your Electron app. VS Code itself is written in TS and BTW half of the team sits in Zurich :-).

Yeah, I have started using TypeScript in my newest project.
After first enjoying it I stopped using it because I use function components combined wich many HOC's using compose. And typing HOC's was too much of a hassle for me :-(
Little after removing TypeScript from that project react hooks came out. I refactored the project to use them. And I now wish I hadn't removed TypeScript, as typing hooks is so much easier.
I will certainly use TypeScript in the next new project.
And BTW I can see Zürich from my kitchen window...


Error: [libs/datamodel/connectors/dml/src/model.rs:161:64] Could not find relation field Trade_OrderToTrade on model Order.

Command: prisma introspect
Version: 2.23.0
Binary Version: adf5e8cba3daf12d456d911d72b6e9418681b28b
Report: https://prisma-errors.netlify.app/report/13330
OS: x64 darwin 20.4.0
JS Stacktrace:
Error: [libs/datamodel/connectors/dml/src/model.rs:161:64] Could not find relation field Trade_OrderToTrade on model Order.
    at ChildProcess.<anonymous> (/<....>/core/node_modules/prisma/build/index.js:39885:28)
    at ChildProcess.emit (events.js:315:20)
    at ChildProcess.EventEmitter.emit (domain.js:467:12)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:277:12)

Rust Stacktrace:
   0: backtrace::backtrace::trace
   1: backtrace::capture::Backtrace::new
   2: user_facing_errors::Error::new_in_panic_hook
   3: user_facing_errors::panic_hook::set_panic_hook::{{closure}}
   4: std::panicking::rust_panic_with_hook
   5: std::panicking::begin_panic_handler::{{closure}}
   6: std::sys_common::backtrace::__rust_end_short_backtrace
   7: _rust_begin_unwind
   8: core::panicking::panic_fmt
   9: core::option::expect_failed
  10: dml::model::Model::find_relation_field_mut
  11: sql_introspection_connector::re_introspection::enrich
  12: sql_introspection_connector::calculate_datamodel::calculate_datamodel
  13: <core::future::from_generator::GenFuture<T> as core::future::future::Future>::poll
  14: <core::future::from_generator::GenFuture<T> as core::future::future::Future>::poll
  15: <futures_util::future::either::Either<A,B> as core::future::future::Future>::poll
  16: <futures_util::future::future::Then<Fut1,Fut2,F> as core::future::future::Future>::poll
  17: <futures_util::future::either::Either<A,B> as core::future::future::Future>::poll
  18: introspection_engine::main::{{closure}}
  19: <core::future::from_generator::GenFuture<T> as core::future::future::Future>::poll
  20: introspection_engine::main
  21: std::sys_common::backtrace::__rust_begin_short_backtrace
  22: std::rt::lang_start::{{closure}}
  23: std::rt::lang_start_internal
  24: std::rt::lang_start

Probably the same: https://prisma-errors.netlify.app/report/13337

New example: https://prisma-errors.netlify.app/report/13520


Implement fetching of user timeline

I have opened a PR to rettiwt-core, if you approve it, I will open a PR here to add user tweets endpoint.
https://github.com/Rishikant181/Rettiwt-Core/pull/74

@omarcinkonis I have approved the PR and the changes have been merged and published as an alpha pre-release.
Please use<EMAIL_ADDRESS>(latest alpha) for development purpose for now.
After a few more feature additions, we'll migrate from alpha to stable upon next Rettiwt-API release. Thanks for the contributions!


clicking "START NEW MODEL" leaves original url, instead of changing to reflect new model ID

This is a little disconcerting if you've been starting new models from the SAE using the START NEW MODEL button, and then want to share one of those subsequent models with another person by copying the url and sending it to them.
Here are the urls:
of the first model I was on when I opened the SAE:
http://<IP_ADDRESS>:8910/workbench/simple-annoton-editor/?model_id=gomodel:5ae3d45e00001973 &barista_token=xmib5fvbw4wrweixn8h8
after clicking START NEW MODEL
http://<IP_ADDRESS>:8910/workbench/simple-annoton-editor/?model_id=gomodel:5ae3d45e00001973 &barista_token=xmib5fvbw4wrweixn8h8
after naming new model and clicking outside of box
http://<IP_ADDRESS>:8910/workbench/simple-annoton-editor/?model_id=gomodel:5ae3d45e00001973 &barista_token=xmib5fvbw4wrweixn8h8
after saving a new annoton
http://<IP_ADDRESS>:8910/workbench/simple-annoton-editor/?model_id=gomodel:5ae3d45e00001973 &barista_token=xmib5fvbw4wrweixn8h8
As you can see, the id of the gomodel did not change at all, not even when I had saved annotations into the new model.
It would be really great if the START NEW MODEL button could trigger something to reload the page or whatever is required to change the url to match the new model.

@kltm what should I do with one? does replacing gmodel id work?

The model id is there for your workbench, to do as is pleases. Depending on how you're running, you may not have access to a new model ID. I would either deprioritize this, as some future update may hide some of that information anyways in some cases, or simply rewrite the URL.

This all appears to be working correctly.


hope support docker image to deploy

Hi,
Do we support it ?
Thanks and regards

Feel free to use https://github.com/intergalacticalvariable/reader ; )


[BUG] Fix CodeQL warnings

Description
Currently, there are many CodeQL warnings. LGTM is marking our code quality as B. We need to get it higher (link: https://lgtm.com/projects/g/TheAlgorithms/C-Plus-Plus/alerts/?mode=list).
Expected behavior
We should fix all the CodeQL/LGTM warnings.
Actual behavior
LGTM is reporting 7 alerts, which marks our quality code as B.
Possible fix
Fix all the CodeQL/LGTM warnings.
Steps to reproduce

Go to https://lgtm.com/projects/g/TheAlgorithms/C-Plus-Plus/alerts/?mode=list.
You will see all the warnings CodeQL is reporting, which makes it easier for us to fix them.
There's a help button in case you need help or are confused with that error.

Context
Having bugs in our code (even minimal bugs) can make it all go wrong or simply affect other students/people reading and understanding our algorithms.
Additional information
Nope.

Hello, I want to take this one.

Hello, I want to take this one.

Hey there! Feel free to make a PR and we'll review it! 😄

hello, i will handle this issue

i want to help you in this...


Add additional log statements and update client-go

Issue #, if available:
Description of changes:

Add logging statements where containers/pod already have required properties
Update client-go to kubernetes-1.18. Update to go1.13

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Can we test this change against a 1.15 cluster to ensure backwards-compatibility?

@jqmichael - Can confirm the container image works against k8s 1.15 server

@jaypipes @nckturner - Pushed a small update to bump up logging levels for some statements. Re-requesting review :)


Add KServeModel abstract base class.

Refactor Ray code into a subclass of KServeModel
What this PR does / why we need it:
This adds the KServeModel abstract base class which is inherited from by the Model and RayServeModel classes. This allows us to isolate the Ray code so we can easily make it an optional dependency.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Type of changes
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] 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

Feature/Issue validation/testing:
Updated the dataplane unit tests to work with this new code.

Logs

Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Checklist:

[x] Have you added unit/e2e tests that prove your fix is effective or that this feature works?
[ ] Has code been commented, particularly in hard-to-understand areas?
[ ] Have you made corresponding changes to the documentation?

Release note:
Add abstract base class for `KServeModel`. Move Ray Serve functionality into subclass.

@sivanantha321 this relates to your PR to make Ray Serve an optional dependency.


Update HarfBuzz to 7.0.0

According to the changelog, 7.0.0 should be perfectly backwards compatible with 6.0.0 and brings faster shaping speed.
That said, this was primarily done as part of #73051 (see there for full context), but spinned off for more convenient maintenance.

Pushed just to update the COPYRIGHT.txt file.

For the record, there's 7.0.1 now (but not a priority to update right now :)).

This could now be merged, but in the meantime HarfBuzz 7.1.0 was released, so it would be good to update to it.

Updated. Smoke tested locally.

Thanks!


React + Express Example

I created an example React project for myself: https://github.com/adrianmcli/meta-auth-example
It uses a barebones, non-ejected, Create-React-App boilerplate (which is the standard in the industry).
I figured you might want this in your repo as well. It certainly would make things easier for people who are already familiar with React.
App.js has the most important stuff. I will also highlight the important points here:
Step 1 — Get the challenge
Assuming that web3 and accounts has been loaded into local component state, this is how you request the challenge from the server and then save the response:
getChallenge = async () => {
  const { accounts } = this.state;
  const res = await fetch(
    `http://localhost:3001/auth/${accounts[0].toLowerCase()}`
  );
  this.setState({ challenge: await res.json() });
};

Step 2 — Ask user to sign the challenge
This will ask the user to sign the challenge in order for him to prove he is who he says he is. It should cause MetaMask to pop-up.
signChallenge = async () => {
  const { web3, challenge, accounts } = this.state;
  web3.currentProvider.sendAsync(
    {
      method: "eth_signTypedData",
      params: [challenge, accounts[0]],
      from: accounts[0]
    },
    (error, res) => {
      if (error) return console.error(error);
      this.setState({ signature: res.result });
    }
  );
};

Step 3 — Send the signed challenge to the server for verification
Depending on the response of the server, we will know whether or not the user's identity has been authenticated.
verifySignature = async () => {
  const { challenge, signature, accounts } = this.state;
  const res = await fetch(
    `http://localhost:3001/auth/${challenge[1].value}/${signature}`
  );
  const recovered = await res.text();
  if (res.status === 200 && recovered === accounts[0].toLowerCase()) {
    console.log("Signature verified");
  } else {
    console.log("Signature not verified");
  }
};

I apologize for taking so long. I've been working on another project and plan to roll this one into its organization.
We're going to be moving to Heritage-Aero and this will be part of Airbus' Heritage project (www.heritage.aero).
I will review before/after the move.

@tenthirtyone Sounds good!
I'm actually on the Truffle team now, thanks for giving the talk at our conference earlier this month! Too bad we didn't get a chance to meet.

@adrianmcli That's AWESOME. Congratulations!
I still owe Tim an email. Hopefully we can cross paths soon.

Thanks!
But speaking of which, I need to update this PR to work with EIP-1102 and the new way MetaMask injects a provider (due November 2). Maybe I'll update this on the weekend.


workflow for closing old issues

Fixes #202
@rohansx kindly review

@rohansx the gssoc labels are not added , pls add them

@Tushar98644 Added the labels


Upgrading slf4j dependency breaks junit-platform-console on hamcrest dependency

If the subject line sounds confusing, it is to me too.  I'm not even sure this is the right project to which to submit this issue, but since the error is occurring during execution of this plugin, it's at least a cry for help from someone who might have an inkling where else I can look!
Background:

My project targets support for JDK 8 and everything works fine there

I have a runtime dependency on SLF4J (1.7.30, trying to upgrade)

My tests are set up using the same package name in a test directory

I have test dependencies on junit-jupiter-api (5.7.1) and hamcrest (2.2)

I publish a separate artifact which simply adds the module-info.java class

I'm using the junit-platform-maven-plugin and the module-info.test approach outlined in your fantastic "testing in the modular world" document, which makes everything work in JDK 11+

The change causing problems:

A user requested that I upgrade SLF4J to the version which contains a module descriptor, currently 2.0.0-alpha1.  Despite the alpha name/unstable categorization, it's in wide usage.
Simply changing the version of SLF4J breaks the Junit class loading on the hamcrest test dependency

I can't find any logical reason why this would happen.  There's no dependency relationship between these two projects
There's only one guess/hunch that I have: before this change, there are non-modular projects on the classpath/unnamed module. With this change, all dependencies have either a module name, or in the case of hamcrest, an automatic module name.

I don't know enough about modulepath, classpath, and automatic module names to know whether that's relevant

The symptoms:

When running any maven command invoking tests (test, verify, install, deploy, etc.) the junit-platform-maven-plugin fails with messages like the below (middle bits redacted for brevity):

[INFO] Launching JUnit Platform 1.7.1...
[ERROR]
[ERROR] Thanks for using JUnit! Support its development at https://junit.org/sponsoring
[ERROR]
[ERROR] Usage: ConsoleLauncher [-h] [--disable-ansi-colors] [--disable-banner]
<snip>
[ERROR] org.junit.platform.commons.JUnitException: TestEngine with ID 'junit-jupiter' failed to discover tests
[ERROR] 	at org.junit.platform.launcher@1.7.1/org.junit.platform.launcher.core.EngineDiscoveryOrchestrator.discoverEngineRoot(EngineDiscoveryOrchestrator.java:111)
<snip>
[ERROR] Caused by: org.junit.platform.commons.JUnitException: ModuleSelector [moduleName = 'com.github.oshi'] resolution failed
[ERROR] 	at org.junit.platform.launcher@1.7.1/org.junit.platform.launcher.listeners.discovery.AbortOnFailureLauncherDiscoveryListener.selectorProcessed(AbortOnFailureLauncherDiscoveryListener.java:39)
<snip>
[ERROR] Caused by: java.lang.NoClassDefFoundError: org/hamcrest/Matcher
[ERROR] 	at java.base/java.lang.Class.getDeclaredMethods0(Native Method)
<snip>
[ERROR] Caused by: java.lang.ClassNotFoundException: org.hamcrest.Matcher
[ERROR] 	at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:606)

Reviewing debug logs, hamcrest is included on the module path:
[DEBUG] /usr/local/Cellar/openjdk/15.0.2/libexec/openjdk.jdk/Contents/Home/bin/java
[DEBUG] -javaagent:/Users/danielwiddis/.m2/repository/org/jacoco/org.jacoco.agent/0.8.6/org.jacoco.agent-0.8.6-runtime.jar=destfile=/Users/danielwiddis/git/oshi/oshi-core/target/jacoco.exec,excludes=**/driver/**:**/common/**:**/Abstract*:**/platform/**:**/linux/**:**/mac/**:**/windows/**:**/unix/**:**/demo/**
[DEBUG] -enableassertions
[DEBUG] -Dfile.encoding=UTF-8
[DEBUG] --module-path
[DEBUG] /Users/danielwiddis/.m2/repository/net/java/dev/jna/jna-jpms/5.8.0/jna-jpms-5.8.0.jar:/Users/danielwiddis/.m2/repository/net/java/dev/jna/jna-platform-jpms/5.8.0/jna-platform-jpms-5.8.0.jar:/Users/danielwiddis/.m2/repository/org/slf4j/slf4j-api/2.0.0-alpha1/slf4j-api-2.0.0-alpha1.jar:/Users/danielwiddis/.m2/repository/org/slf4j/slf4j-simple/2.0.0-alpha1/slf4j-simple-2.0.0-alpha1.jar:/Users/danielwiddis/.m2/repository/org/junit/jupiter/junit-jupiter-api/5.7.1/junit-jupiter-api-5.7.1.jar:/Users/danielwiddis/.m2/repository/org/apiguardian/apiguardian-api/1.1.0/apiguardian-api-1.1.0.jar:/Users/danielwiddis/.m2/repository/org/opentest4j/opentest4j/1.2.0/opentest4j-1.2.0.jar:/Users/danielwiddis/.m2/repository/org/junit/platform/junit-platform-commons/1.7.1/junit-platform-commons-1.7.1.jar:/Users/danielwiddis/.m2/repository/org/hamcrest/hamcrest/2.2/hamcrest-2.2.jar:/Users/danielwiddis/git/oshi/oshi-core/target/junit-platform/patched-test-runtime:/Users/danielwiddis/.m2/repository/org/junit/platform/junit-platform-launcher/1.7.1/junit-platform-launcher-1.7.1.jar:/Users/danielwiddis/.m2/repository/org/junit/platform/junit-platform-engine/1.7.1/junit-platform-engine-1.7.1.jar:/Users/danielwiddis/.m2/repository/org/junit/platform/junit-platform-reporting/1.7.1/junit-platform-reporting-1.7.1.jar:/Users/danielwiddis/.m2/repository/org/junit/platform/junit-platform-console/1.7.1/junit-platform-console-1.7.1.jar:/Users/danielwiddis/.m2/repository/org/junit/jupiter/junit-jupiter-engine/5.7.1/junit-jupiter-engine-5.7.1.jar

I've looked for split modules and other dependency conflicts.  Nothing abnormal:
[INFO] --- maven-dependency-plugin:3.1.2:tree (default-cli) @ oshi-core-java11 ---
[INFO] com.github.oshi:oshi-core-java11:jar:5.7.1-SNAPSHOT
[INFO] +- net.java.dev.jna:jna-jpms:jar:5.8.0:compile
[INFO] +- net.java.dev.jna:jna-platform-jpms:jar:5.8.0:compile
[INFO] +- org.slf4j:slf4j-api:jar:2.0.0-alpha1:compile
[INFO] +- org.slf4j:slf4j-simple:jar:2.0.0-alpha1:test
[INFO] +- org.junit.jupiter:junit-jupiter-api:jar:5.7.1:test
[INFO] |  +- org.apiguardian:apiguardian-api:jar:1.1.0:test
[INFO] |  +- org.opentest4j:opentest4j:jar:1.2.0:test
[INFO] |  \- org.junit.platform:junit-platform-commons:jar:1.7.1:test
[INFO] \- org.hamcrest:hamcrest:jar:2.2:test

Things I've tried with no success:

Moved hamcrest earlier in dependency list: no change
Adding requires org.hamcrest to my module-info.java: Compile error: module not found: org.hamcrest

To fix above, changing scope from test to compile:  Split module: Every test class tries to run and fails with multiple compilation problems: The package org.hamcrest is accessible from more than one module: <unnamed>, org.hamcrest

Adding --add-reads com.github.oshi=org.hamcrest to the module-info.test file:  Same symptoms as long list above, but the line just before the stack trace says "WARNING: Unknown module: org.hamcrest specified to --add-reads"

I don't know if this is a bug or user (me) error or just a need for a configuration setting in maven.
Happy to try any other suggestions and provide any other output needed.  Or you can try playing with it yourself on the java11 branch here: https://github.com/oshi/oshi/tree/java11

Hi Daniel, I'll try to take a look later this week. I skimmed the configuration and it looks good to me:
https://github.com/oshi/oshi/blob/9fbccfb3a4c71fb714e012f3d8fc33f03eaa1075/pom.xml#L261-L280

3 weeks ... and still got not time. Soon™...

Your definition of "Soon™" may need recalibrating.  😁
I was bored enough today to try to debug this myself, and after a few hours of trial and error and stack trace digesting, I managed to get it working by adding the following lines to module-info.test:
--add-modules
  org.hamcrest
--add-reads
  com.github.oshi=org.hamcrest

So I think this was just a configuration thing on my end that I needed to fix.


Dev Conference

I added the new category Dev Conference inside the types/category.js file and then add multiple conferences to the Dev Conference category.
Pull Request Template
Please ensure all items below are checked before creating a pull request:

[x] My changes were made in the resources folder, not in the auto-generated README.md file or db folder
[x] My submission is formatted according to the guidelines in the contributing guide
[x] My submission is ordered alphabetically based on the resource name
[x] My submission has a useful description
[x] I have searched the repository for any relevant issues or pull requests

Thank you so much for that, Juan!
I just renamed the category to be only "Conference" as the "Dev" is redundant due to the nature of the project.


Initial Build Erroring

Just npm installed, and cant run npm run build without this error
><EMAIL_ADDRESS>build /Applications/MAMP/htdocs/obrella/wp-content/themes/FoundationPress
> grunt build

/Applications/MAMP/htdocs/obrella/wp-content/themes/FoundationPress/node_modules/grunt/node_modules/findup-sync/lib/findup-sync.js:33
    }).flatten().uniq().value();
       ^
TypeError: undefined is not a function
    at Object.module.exports [as findup] (/Applications/MAMP/htdocs/obrella/wp-content/themes/FoundationPress/node_modules/grunt/node_modules/findup-sync/lib/findup-sync.js:33:8)
    at Task.task.init (/Applications/MAMP/htdocs/obrella/wp-content/themes/FoundationPress/node_modules/grunt/lib/grunt/task.js:427:16)
    at Object.grunt.tasks (/Applications/MAMP/htdocs/obrella/wp-content/themes/FoundationPress/node_modules/grunt/lib/grunt.js:120:8)
    at Object.module.exports [as cli] (/Applications/MAMP/htdocs/obrella/wp-content/themes/FoundationPress/node_modules/grunt/lib/grunt/cli.js:38:9)
    at Object.<anonymous> (/usr/local/lib/node_modules/grunt-cli/bin/grunt:45:20)
    at Module._compile (module.js:460:26)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Function.Module.runMain (module.js:501:10)

Here is the npm-debug.log
0 info it worked if it ends with ok
1 verbose cli [ 'node', '/usr/local/bin/npm', 'run', 'build' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'prebuild', 'build', 'postbuild' ]
5 info prebuild<EMAIL_ADDRESS>6 info build<EMAIL_ADDRESS>7 verbose unsafe-perm in lifecycle true
8 info<EMAIL_ADDRESS>Failed to exec build script
9 verbose stack Error<EMAIL_ADDRESS>build: `grunt build`
9 verbose stack Exit status 1
9 verbose stack     at EventEmitter.<anonymous> (/usr/local/lib/node_modules/npm/lib/utils/lifecycle.js:214:16)
9 verbose stack     at EventEmitter.emit (events.js:110:17)
9 verbose stack     at ChildProcess.<anonymous> (/usr/local/lib/node_modules/npm/lib/utils/spawn.js:24:14)
9 verbose stack     at ChildProcess.emit (events.js:110:17)
9 verbose stack     at maybeClose (child_process.js:1008:16)
9 verbose stack     at Process.ChildProcess._handle.onexit (child_process.js:1080:5)
10 verbose pkgid<EMAIL_ADDRESS>11 verbose cwd /Applications/MAMP/htdocs/obrella/wp-content/themes/FoundationPress
12 error Darwin 15.0.0
13 error argv "node" "/usr/local/bin/npm" "run" "build"
14 error node v0.12.0
15 error npm  v2.13.3
16 error code ELIFECYCLE
17 error<EMAIL_ADDRESS>build: `grunt build`
17 error Exit status 1
18 error Failed at the<EMAIL_ADDRESS>build script 'grunt build'.
18 error This is most likely a problem with the foundationpress package,
18 error not with npm itself.
18 error Tell the author that this fails on your system:
18 error     grunt build
18 error You can get their info via:
18 error     npm owner ls foundationpress
18 error There is likely additional logging output above.
19 verbose exit [ 1, true ]

Updated node. Fixed.


Missing Dimension Null | Can't enter the server after dying | 1.16.5

Hi.
I was playing in my world with this mod and when I died it kicked me out of the server, I tried to enter again but this error pops up:
Internal Exception: io.netty.handler.codec.Encoder.Exception:java.lang.RuntimeException: Missing Dimension Null
I've tried to delete and install again the mod but nothing seems to be working
It's a multiplayer world

Unfortunately the 1.16.5 version is not being supported now.

Unfortunately the 1.16.5 version is not being supported now.

Is there any way I can contact with someone that does help with 1.16.5?

You seem to be using Forge. The 1.16.5 versions of the Forge version is outdated and incompatible with Forge's dynamic dimension system. It's recommended to stop using immptl 1.16.5 Forge version. There is no one maintaining the 1.16.5 Forge version now.
A new Forge version for 1.18.2 is being developed.


How to save user ID to model when importing data?

Full question here: http://stackoverflow.com/questions/33847382/django-import-export-how-to-add-current-admin-user-to-model
save_model work with normal admin but not with import/export.

I managed to make it work with:
class KWResource(resources.ModelResource):
....
....
   def before_import(self, dataset, dry_run, *args, **kwargs):
       li = []
       li.append(kwargs.pop('user', 1))
       dataset.insert_col(0, li, 'user')
       return super(KWResource, self).before_import(dataset, dry_run, *args, **kwargs)

But it only works as long as my author model is ForeignKey to user. If I change it to ManyToManyField it gives this error:
Traceback (most recent call last):   File "/home/django/myproject/myprojectenv/lib/python2.7/site-packages/import_export/resources.py", line 373, in import_data     self.import_obj(instance, row, real_dry_run)   File "/home/django/myproject/myprojectenv/lib/python2.7/site-packages/import_export/resources.py", line 238, in import_obj     self.import_field(field, obj, data) TypeError: 'User' object does not support indexing

See documentation on Import data workflow. import_data  would receive user instance - you can save it as an instance variable. Then assign it to an instance in before_save_instance.

Any examples of similar things? I tried pretty much everything and can't seem to make it work.
author = fields.Field(column_name='user', attribute='author', widget=ManyToManyWidget(User, field='id'))
def after_save_instance(self, instance, dry_run, **kwargs):

    instance.user = User.objects.get(id=kwargs.pop('user', 1))
    instance.user.save()
    instance.save()

Shouldn't this work?

If you look at admin source code, your resource class can override import_data:
def import_data(*args, **kwargs):
    self._user = kwargs.get('user', None)
    return super(import_data(*args, **kwargs))

and then use self._user in after_save_instance
code is from head, I hope it'll help.

Thank you, this did it. If in future anyone needs, here is full code: Model included, so code is complete :)
author = models.ManyToManyField(User, blank=True, editable=False)

def import_data(self, *args, **kwargs):
    self._user = kwargs.get('user', None)
    return super(KWResource, self).import_data(*args, **kwargs)

def after_save_instance(self, instance, dry_run, **kwargs):

    instance.save()
    instance.author.add(self._user)


chore: Always make the built binary executable

I keep forgetting to make the binary executable after I build with make build-static and before I build my new docker image.
This PR should save me from myself.

Ooops. This is me - I just submitted it with the wrong account.