Null Pointer response in discover()

This line occurs twice in the discover() method:
CoapResponse response = client.get();

The first one (for the devices) always gives me Null Pointers on my Raspberry Pi 2.
It always works on my Laptop, where I have my IDE for debugging...
Communication with the gateway from the Pi also works (tested with coap-client from libcoap).
Documentation says that get() "... blocks until the response is available".
I don't get why this error only occurs only on my Pi 2, will also test on my Pi 3 now...

OK, on my Pi 3, this problem does not occur, very strange.
On a side note I like to point out that installation of maven in Raspbian causes a JDK7 to get installed and prioritized higher than your out-of-the-box JDK8, then you will get errors because the TRADFRI2MQTT pom.xml wants a Java 1.8 target.
Fix it with:
sudo update-alternatives --config java

found here: http://www.savagehomeautomation.com/projects/raspberry-pi-change-default-java-virtual-machine-jvm.html

I'm seeing this as well on a pi, both Oracle and OpenJDK 8. I'll tweak the build to generate 7.0 compatible bytecode so it can run on both

It's running fine on OpenJDK 7 on my Pi 2. I'll check in the pom.xml changes (plus the test for colour temp as my bulb [TRADFRI bulb E27 opal 1000lm] isn't reporting that value)

This commit didn't change the line that gives me Null Pointers. I only have full white spectrum bulbs, I guess your commit was to fix issues with the bulbs that only have one temperature.
Currently, it is in line 182 in Main.java, this is where I get a NullPointerException, because CoapResponse response = client.get(); is null.

Did you rebuild with the Java 7.0 bytecode and run it with OpenJDK 7?
My best guess is that this is a JIT issue on ARM so you could try running with the -Xint command line arg.
It will be REALLY slow but it will confirm if it's the JIT

Actually it's running for me with Oracle Java 7.0 it's working with, but I've just tried the 8.0 and -Xint and it's still throwing a NPE

OK, fixed it with Java 8 as well. The doc is not great, but client.get() can return null when a response times out. The timeout defaults to 2 seconds, you can increase it by editing the Californium.properties file.
Edit the ACK_TIMEOUT value. 4000 seams to work for me
ACK_TIMEOUT=4000

That helps indeed, thanks a lot


Direct import of zips

In the description for the plugin, you write "Ever got a zip with drawables for your Android project by your designer with the following structure?". I do, all the time :-)
But the current version of the plugin seems to require unzipping the archive and then dragging each file to the multi-source drawable import window. Wouldn't it be nice if I could just drag the zip and it would populate all the files, or manually unzip, but when I drag the first file, the plugin automatically finds the other resolutions?

cc @xalep

Good idea. The problem is that probably every designer uses different namings. So how should I detect the suitable files? I could do simple contains "-xhdpi" checks for example.

Yes, - and _ seem like reasonable separators. Great plugin btw :-)

Thanks :+1: Will think about a suitable solution.

You rock, thanks for the latest update!


Qa

Coverage increased (+0.8%) to 83.968% when pulling 97318bae166be71aa5149366d2509b41214062b3 on qa into 1d3453d0fa7be225b4886fa294389eccdc4803f1 on master.


refactor: Use shims instead of direct operations on PATH.

fix #253
fix #84
fix  #235
fix version-fox/vfox-golang#9
Instead of modifying the PATH directly, the current change uses the fixed path $VFOX_HOME/shims and $VFOX_SHELL_TMEP/shims instead.
Currently, the shim's generation is very violent, later we need to ask plugins to tell vfox what bins need to be generated shim. see https://github.com/version-fox/vfox/discussions/191#discussioncomment-9064948

There's a problem on Windows, so change this to a draft.

There's a problem on Windows, so change this to a draft.

As I expected the directory to be the link instead of the executable file, many tools' executable programs will depend on the directory they are in.
Just like https://github.com/Schniz/fnm

As I expected the directory to be the link instead of the executable file, many tools' executable programs will depend on the directory they are in.

Yes.  It's not that big of a problem, we can just implement the program exactly as mentioned in #84.

Yes. It's not that big of a problem, we can just implement the program exactly as mentioned in #84.

I like the soft link directory solution very much. I have used several version management tools using this method and it works well. I look forward to this implementation.

The problem on Windows has been solved, and the original plan is still used.


[multistage][feature] FunctionRegistry unification attempt2

this is #12229 improvement to address the recent modification to #12278

Codecov Report
Attention: 142 lines in your changes are missing coverage. Please review.

Comparison is base (7b2a82d) 61.54% compared to head (108a583) 27.68%.
Report is 4 commits behind head on master.

Files
Patch %
Lines

...apache/pinot/common/function/FunctionRegistry.java
0.00%
86 Missing :warning:

.../pinot/common/function/sql/PinotOperatorTable.java
0.00%
25 Missing :warning:

...ot/common/function/schema/PinotScalarFunction.java
0.00%
15 Missing :warning:

...ery/runtime/operator/operands/FunctionOperand.java
0.00%
8 Missing :warning:

...ime/operator/operands/TransformOperandFactory.java
0.00%
7 Missing :warning:

.../java/org/apache/pinot/query/QueryEnvironment.java
0.00%
1 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #12302       +/-   ##
=============================================
- Coverage     61.54%   27.68%   -33.87%
+ Complexity     1153      207      -946
=============================================
  Files          2415     2422        +7
  Lines        131482   131880      +398
  Branches      20303    20359       +56
=============================================
- Hits          80926    36514    -44412
- Misses        44636    92599    +47963
+ Partials       5920     2767     -3153

Flag
Coverage Δ

custom-integration1
?

integration
<0.01% <0.00%> (-0.01%)
:arrow_down:

integration1
<0.01% <0.00%> (-0.01%)
:arrow_down:

integration2
0.00% <0.00%> (ø)

java-11
27.67% <0.00%> (-33.85%)
:arrow_down:

java-21
27.68% <0.00%> (-33.75%)
:arrow_down:

skip-bytebuffers-false
27.68% <0.00%> (-33.86%)
:arrow_down:

skip-bytebuffers-true
?

temurin
27.68% <0.00%> (-33.87%)
:arrow_down:

unittests
27.68% <0.00%> (-33.87%)
:arrow_down:

unittests1
?

unittests2
27.68% <0.00%> (-0.09%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


1.3.0 input[date] model is a Date object in local time zone instead of UTC

Please check this plunker for a demonstration.
Assuming the computer time zone is not UTC and the browser supports input[date], select a date from the input[date] dropdown.
The input field will display a date in the localized format, for example "01.08.2014" in my case. My current timezone is GMT+4.
value is
"2014-07-31T20:00:00.000Z"

but expected is either
"2014-07-31T20:00:00.000-0400"

or
"2014-08-01T00:00:00.000Z"

It seems that the code that converts input[date].value into a Date model object first obtains the correct value "2014-07-31T20:00:00.000-0400" but then replaces the correct "-0400" suffix with incorrect "Z", or it is a matter of formatting when the date is formatted as localized but the timezone is not part of the format and instead the "Z" suffix is hardcoded..
The code that converts the string value of the input[date] into a Date model should follow the  HTML5 input[type=date] guidelines and convert the value string as if it is the start of the day in the UTC regardless of the current timezone.

I'm not sure if I don't understand or if it doesn't work right for me on Galaxy S3 but for me the string and also Android's input dialogs ar incorrect, even though the browser's locale is correct (alert(new Date()) shows "...GMT+0100 (CET)").
But the unformatted date output still says ".....000Z" and the HTML input shows 2015-03-04 instead of 04.03.2015 with ng-model-options="{timezone: 'UTC'}" (tried GMT+0100 too). In the Android's date selector dialog the date is correctly formatted but in both date and time selector dialogs, the "Clear" button is in english instead of german and in the time selector dialog, the time selection goes from 1 to 12 with AM/PM instead of 0 to 23 without AM/PM.
thx

I think it's totally crazy that date & time don't have the same format (correct Date Object.) I made a codepen to illustrate.
If I set it to today, at 1AM I get different timezones for the 2 inputs, both of which are not UTC.
Wed Jun 01 2016 00:00:00 GMT-0700 (Pacific Daylight Time)
Thu Jan 01 1970 01:00:00 GMT-0800 (Pacific Standard Time)

I can adjust for this by subtracting the timezone offset of each, then adding the local timezone offset when I add the 2 together, but this is incorrect behavior. They should both agree on the timezone.

@konsumer, in both it is your local timezone with and without daylight saving. This is because the dates are different. We can't really fix this.

These are just 2 elements in the same page.  As you can see in the valFixed code, if I strip the zone offset from both, then add it from new Date() it works fine, but I shouldn't have to do this. The date objects attached to the ng-model of date and time inputs shouldn't default to different timezones.

As you can see, outFixed is the correct value, but only after I strip the timzone from both, then add the local timezone:

Time input defaults to PST, when everything else defaults to PDT. This seems like an obvious bug.

@konsumer if you feel this is still a problem. Please open up a new issue. If you could provide a broken unit test that needs fixing that would be even better.

Upon more thought, I'm not sure what it should do instead. I think I initially misunderstood what @gkalpak was saying. The reason the timezone is different is that pacific changes based on the time of the year (not just location) so, January 1st (1970) is a different timezone. If I want to use it as a simple addition of milliseconds, I can set the timezone of the time-input to UTC, or adjust it after (as I did in codepen,) I guess I expected the default to be UTC for time, so it would add correctly, but I think I understand why it's not. I will just remember this gotcha, in the future, and set my time-input zones to UTC or process them after.


Add NumberRangeArgumentParser

Resolve #53

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


unconsistent behaviour when overriding transition duration

UnoCSS version
0.61.0
Describe the bug
strange things happen when i put duration-2000 directly after duration-300 it does not work but when i switched duration-300 to duration-100 it works just fine idon't know why
example 1 (not working):
<template>
	<div bg-blue flex items-center justify-center h-screen>
		<button
			py-4
			px-8
			rounded-md
			cursor-pointer
			@click="togglebgColor"
			transition-colors
			duration-300
			duration-2000
			:class="active ? 'bg-black' : 'bg-amber'"
		>
			Click me
		</button>
	</div>
</template>
<script setup lang="ts">
const active = ref(false);

const togglebgColor = () => {
	active.value = !active.value;
};
</script>

example 2 (working):
```vue
<template>
	<div bg-blue flex items-center justify-center h-screen>
		<button
			py-4
			px-8
			rounded-md
			cursor-pointer
			@click="togglebgColor"
			transition-colors
			duration-100
			duration-2000
			:class="active ? 'bg-black' : 'bg-amber'"
		>
			Click me
		</button>
	</div>
</template>
<script setup lang="ts">
const active = ref(false);

const togglebgColor = () => {
	active.value = !active.value;
};
</script>

Reproduction
https://stackblitz.com/edit/unocss-unocss-7565vhg?file=app.vue
System Info
system: windows 10
browser : chrome , firefox,edge,brave
framework : nuxt
Validations

[X] Read the Contributing Guidelines.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

The reason is that in the sorting after CSS compilation, duration-200 and duration-300 using localeCompare result in -1, while duration-200 and duration-100 result in 1.
This causes the final order to be different.
https://github.com/unocss/unocss/blob/e3d3b2ed21f02c213351d2d2c1d46ae57cae808b/packages/core/src/generator/index.ts#L298

@CofCat456 yeah and this behaviour is not just for the duration i tested in colors as well some colors over the previous ones while  others doesn't

I don't think relying on the alphabetical order works great, you can either add ! to add important or a new layer. play

@henrikvilhelmberglund  isn't that a bad practice ? i'm looking for something more convenient without adding important everytime . i want to override styles for various buttons . it's seems like the only way to do it is using props but again i find it impractical to do so . is there any way to merge styles like tailwindcss does?

@henrikvilhelmberglund isn't that a bad practice ? i'm looking for something more convenient without adding important everytime . i want to override styles for various buttons . it's seems like the only way to do it is using props but again i find it impractical to do so . is there any way to merge styles like tailwindcss does?

If you only have a base state with classes and want to override a certain class imo ! is the ideal way to do it. Important is bad practice when writing normal non utility style CSS when you (ab)use the cascading part but should be fine for utility classes.
If you've got some different buttons that have different styles you may want to use shortcuts instead. And then you could have base classes with the default classes you want, using layers to give them a lower specificity, that way you don't need !. Or just switch the whole  shortcut for every button.

Unocss is a runtime framework. I don't want it to check if users write conflicting override rules on the same attribute. It will add extra burden to the runtime. This problem should be paid by the users themselves.


fix: remove clunky sentence

Checklist:

[X] I have read freeCodeCamp's contribution guidelines.
[X] My pull request has a descriptive title (not a vague title like Update index.md)
[X] My pull request targets the master branch of freeCodeCamp.
[X] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #XXXXX

This sentence is fairly unclear as it is in English, and does not come
across as a translation-friendly string. Also does not seem to add
value to the challenge information.
Signed-off-by: nhcarrigan<EMAIL_ADDRESS>cc/ @RandellDawson

I went ahead and added a note to clarify that.

I went ahead and added a note to clarify that.


#77 - Add support for multiple Zoom instances

Add multiple instances to the config
Iterate through both instances
Add a extra column to the database for the new instance
Support JWT authentication (formerly #78)
Fixes #77.

Fixes #77.

Thanks, guess I didn't notice it didn't do that! Saw this issue.
https://github.com/isaacs/github/issues/568
Also cool browser extension mentioned on there that does take care of that
https://github.com/sindresorhus/refined-github

Thanks for the feedback, upon further research I notice requests can take a parameter for a custom auth. I created one (since JWT wasn't an option) and it wasn't much work as it was what I already had. This gets the request really close to the call. This seems like a really cool idea and should incorporate it other places.
I also enabled mypy in .vscode/settings.json when I was going through this and found a number of errors in the code I was working on. There might be issues in other code that wasn't addressed here.

Ok I think I got this all fixed up now. I also changed the ZOOM_CONFIG to a list rather than an object/dict as Chris is okay with just using an indexed variable. We'll have to in a future task (when we do the database) have a mapping somehow for this list item into a type in the database.

Ok I think I got this all fixed up now. I also changed the ZOOM_CONFIG to a list rather than an object/dict as Chris is okay with just using an indexed variable. We'll have to in a future task (when we do the database) have a mapping somehow for this list item into a type in the database.

Sounds fine. There are a couple docstring related Codacy issues, but not a bit deal to fix right now. Please feel free to merge.

Yeah, I fixed those two, I'm just giving it one more run to make sure the csv looks good after this config change and I'll commit. Thanks for all the feedback. This one looked simple but it actually did a lot to get this code quality better for the next step.


Can you share the pre-trained weights of waterbirds?

It's already in the ./pretrained folder. Thx for your interest!


fix(tests): broken PDF reader test

Fix the broken test in the main branch plus some tidying

Fixed it in main.


ARROW-15776: [Python] Expose IpcReadOptions

This PR intends to expose IpcReadOptions to pyarrow.

@jorisvandenbossche As discussed I think this can start review process. I was going to assign you as reviewer but I don't have permission.

Actually, we should also add it to the docs here: https://github.com/apache/arrow/blob/b8299436c8b1a2d7cd3d6e019a2b750893a3af87/docs/source/python/api/ipc.rst?plain=1#L41

Actually, we should also add it to the docs here:
https://github.com/apache/arrow/blob/b8299436c8b1a2d7cd3d6e019a2b750893a3af87/docs/source/python/api/ipc.rst?plain=1#L41

Done and validated documentation generated:


Mitigate hidden opencv dependency conflicts in pixi

What
I think it should fix most of

https://github.com/rerun-io/rerun/issues/6901

In short, the opencv packages are an aboslute dumpster fire. This is a good summary of relevant discussions on the matter:

https://github.com/opencv/opencv-python/issues/987

In short: opencv-python and opencv-contrib-python are two distinct, conflicting packages.  But pypi doesn't have a way of specifying package conflicts (same as our rerun vs rerun-sdk issue).  If you install both, one will shadow the other, non-determinically causing some features to appear to be missing.
Conda fortunately somewhat side-steps this by only providing one.  Unfortunately, the pixi  pypi-conda-mapping only maps opencv-python -> conda: py-opencv but doesn't handle the mapping for opencv-contrib-python.
To solve this we now: force opencv to be installed via conda for both base and examples.  This means any package that depends on opencv-python will pick up the conda version (which actually includes contrib). We still have both direct and indirect (via mediapipe) packages that depend on opencv-contrib-python that lead to some dangerous package-shadowing. But at least they're mostly the same package at this point. (I added a comment to the tracking issue with with pix as well: https://github.com/prefix-dev/pixi/issues/771#issuecomment-2237638133)
Also I hit some weird issues with numpy since rerun-sdk has a dependency on numpy<2. Certain permutations of packages seem to trick the solver into thinking the constraints (when pulled from pypy) are non-satisfiable. By pulling the numpy dep into conda dep list everything seems happy.
Checklist

[x] I have read and agree to Contributor Guide and the Code of Conduct
[x] I've included a screenshot or gif (if applicable)
[ ] I have tested the web demo (if applicable):

Using examples from latest main build: rerun.io/viewer
Using full set of examples from nightly build: rerun.io/viewer

[x] The PR title and labels are set such as to maximize their usefulness for the next release's CHANGELOG
[x] If applicable, add a new check to the release checklist!
[ ] If have noted any breaking changes to the log API in CHANGELOG.md and the migration guide

PR Build Summary
Recent benchmark results
Wasm size tracking

To run all checks from main, comment on the PR with @rerun-bot full-check.

@rerun-bot full-check


Update MobilePose Docs

Adds mobile pose specific documentation on how to use it.

Docs look great! The video was a great addition. LGTM.


[BUG] SageFormInput prefix & suffix don't support strings with spaces

Detailed description

Describe in detail the issue you're having.

When adding a prefix or suffix where the string contains a space (e.g. two words), the form input component misplaces the string for the data attribute and the label does not display any additional words after a space.

Is this issue related to a specific component?

SageFormInput

What did you expect to happen? What happened instead? What would you like to see changed?

Expect prefix or suffix to display full string.

What browser are you working in?

Chrome / Firefox
Steps to reproduce
Describe the steps to reproduce the behavior:

Spin up docs site
Visit http://localhost:4000/pages/component/form_input
Update prefix or suffix for an input with two words in docs/app/views/examples/components/form_input/_preview.html.erb
Notice that the prefix or suffix ignores words after space.

Screenshots
For the provided screenshot the prefix and suffix strings were set to "two words"

Moved to Jira: https://kajabi.atlassian.net/browse/UXD-72


Please update dependencices to resolve vulnerabilities found

In braces, prettier-stylelint > stylelint > micromatch > braces
Regular Expression Denial of Service, https://nodesecurity.io/advisories/786

@shonoru 👍 braces fixed this issue long back and the change got through micromatch and stylelint. Upgrading stylelint should fix this issue. I will create a PR for this.


[Bug]: The Mongo Mock datasource query fails with Json error

Is there an existing issue for this?

[X] I have searched the existing issues

Description
The Mongo Mock datasource query fails with Json error

Steps To Reproduce

go to release.app.appsmith.com
Go to edit mode go to datasource section
now add the new datasource i.e Mongo sample datasource
Now before the table loads click on new query button on the right top corner
The query is auto filled
Just run the query
The json error is displayed
Note:if the "New query" button is clicked before the datasource structure is ready the query is displayed empty

Public Sample App
No response
Environment
Production
Severity
High (Blocker to building or releasing)
Issue video log
No response
Version
v1.19

couldn't consistently repro the issue would like more debugging on this so reducing the priority here

Closing this as couldn't reproduce the issue.


Refactor/2462 Deprecate store pckg

Related issues
closes https://github.com/DivanteLtd/vue-storefront/issues/2462
Short description and why it's useful
It's redundant at this point so it's a simplification. Also we will need to to this anyway so it's better to change it now than when we will have tons of modules
Upgrade Notes and Changelog

[ ] No upgrade steps required (100% backward compatibility)
[x] I've updated the Upgrade notes and Changelog on how to port existing VS sites with this new feature

@pkarw @patzick please review as it's a blocker for few OTHER PRs for me

Yeah i guess this dep was missing. I thought we have it somewhere else but not so ill add it to the pckg json


Discovery info to discover the application that can be applicable to a given stack/devfile

In a devfile that is being used to define a stack, there are certain files that the stack may be looking for to decide if a given project can be run by a given stack or not, e.g. a maven stack may look for the pom.xml and a node.js stack may look for package.json.
If we can provide a way to allow a given stack to define those required resources, we can use it in tools like Che or odo to do quick check on project to suggest the stacks on a devfile repo that may support the a given application.  An example can be:
parent.yaml
---
schemaVersion: 2.0.0
metadata:
  publisher: my-org
  name: maven-stack
  version: x.y.z
  matchingRules:
    existingFiles: [pom.xml, aaa.xml]

devfile.yaml
---
schemaVersion: 2.0.0
metadata:
  name: my-maven-project
parent:
  id: organization/maven-stack/x.y.z
...

There may be other types other than existingFiles that we can support in the future.

Reviewed and agreed

We will have to think about how we do the same for a parent existing as a Kubernetes resource (DevWorkspaceTemplate). We might want to use K8S metadata int his case, but then let's keep in mind that K8S metadata are simply a map of strings, while here the matching rules define a structured sub-schema. I assume we don't want to split the related syntax and have 2 different ways to express that between yaml parent devfiles and K8S CRD parents devfiles.

If that's a structured information that is valid for both the devfile Yaml and the devworkspace template CRD, I'd rather have a true field for it.


Unable to rotate screen for PC handheld One X Player

Hi, I’m trying to rotate the screen left to accommodate the portrait display in my one x player gaming handheld with intel xe i7 1165g7. Using the shell I was unable to xrandr —rotate left for my output, error crtc 0 failed. I’m also unable to get to the desktop after enabling in the settings.

Thanks in advance

How is the screen rotated by default?
You can customize the rotation by editing
~/.config/steamos-compositor-plus
You can uncomment and change ROTATION
to left, right, or normal.
There is no desktop mode.
On Wed., Jul. 21, 2021, 16:27 project-sbc, @.***> wrote:

Thanks in advance
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ChimeraOS/chimeraos/issues/251#issuecomment-884476957,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIWMWCPWR2GVLCQEDIVTLTTY4US3ANCNFSM5AYVUSPQ
.

Thanks for the feedback! I modified the last line to ROTATION=“LEFT”
and there wasn’t any change
I then tried adding the native display resolution 2560x1600 display so I uncommented that and added 2560x1600
Still no change, did I enter something incorrect?
Get Outlook for iOShttps://aka.ms/o0ukef

From: alkazar @.>
Sent: Wednesday, July 21, 2021 5:54:35 PM
To: ChimeraOS/chimeraos @.>
Cc: project-sbc @.>; Author @.>
Subject: Re: [ChimeraOS/chimeraos] Unable to rotate screen for PC handheld One X Player (#251)
How is the screen rotated by default?
You can customize the rotation by editing
~/.config/steamos-compositor-plus
You can uncomment and change ROTATION
to left, right, or normal.
There is no desktop mode.
On Wed., Jul. 21, 2021, 16:27 project-sbc, @.***> wrote:

Thanks in advance
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ChimeraOS/chimeraos/issues/251#issuecomment-884476957,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIWMWCPWR2GVLCQEDIVTLTTY4US3ANCNFSM5AYVUSPQ
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/ChimeraOS/chimeraos/issues/251#issuecomment-884524509, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASZ4DJPKZWE2SOYPASJ5FHLTY46ZXANCNFSM5AYVUSPQ.

Did you restarted your PC/session after the change?

Yes I have rebooted after every change
Get Outlook for iOShttps://aka.ms/o0ukef

From: Joaquín Ignacio Aramendía @.>
Sent: Wednesday, July 21, 2021 6:21:19 PM
To: ChimeraOS/chimeraos @.>
Cc: project-sbc @.>; Author @.>
Subject: Re: [ChimeraOS/chimeraos] Unable to rotate screen for PC handheld One X Player (#251)
Did you restarted your PC/session after the change?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/ChimeraOS/chimeraos/issues/251#issuecomment-884536647, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASZ4DJJEW7Y3OZJEVWICLS3TY5B57ANCNFSM5AYVUSPQ.

I made a video if diagnosing it makes it easier
https://youtu.be/_I8TYF735GY
Get Outlook for iOShttps://aka.ms/o0ukef

From: Chris Moore @.>
Sent: Wednesday, July 21, 2021 6:22:39 PM
To: ChimeraOS/chimeraos @.>; ChimeraOS/chimeraos @.>
Cc: Author @.>
Subject: Re: [ChimeraOS/chimeraos] Unable to rotate screen for PC handheld One X Player (#251)
Yes I have rebooted after every change
Get Outlook for iOShttps://aka.ms/o0ukef

From: Joaquín Ignacio Aramendía @.>
Sent: Wednesday, July 21, 2021 6:21:19 PM
To: ChimeraOS/chimeraos @.>
Cc: project-sbc @.>; Author @.>
Subject: Re: [ChimeraOS/chimeraos] Unable to rotate screen for PC handheld One X Player (#251)
Did you restarted your PC/session after the change?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/ChimeraOS/chimeraos/issues/251#issuecomment-884536647, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASZ4DJJEW7Y3OZJEVWICLS3TY5B57ANCNFSM5AYVUSPQ.

You mentioned you tried using xrandr. For it to work from a Terminal you need to provide the appropriate DISPLAY environment variable like so:
$ DISPLAY=:0 xrandr -o left
Could you test if it works?

You need to put:
ROTATION=left

Display=:0 xrandr -o left yields an error of failed to changed screen configuration
changing config file doesn’t change it either

Display=:0 xrandr -o left yields an error of failed to changed screen configuration
changing config file doesn’t change it either

Be aware that capitalization is important. Linux (and most OS apart from DOS-based ones) are case-sensitive.

Thanks for the feedback, incorrect capitalization yields a “can’t open display” error, which is different than the correct syntax

Does it run with root privileges (i.e. sudo)?
$ DISPLAY=:0 sudo xrandr -o left

Fixed it, I had to uncomment the good resolution and good rates and add the screen resolution to the list in the steamos compositor plus. The touchscreen did not rotate, is there a way to rotate that in that config file?
Get Outlook for iOShttps://aka.ms/o0ukef

From: Joaquín Ignacio Aramendía @.>
Sent: Tuesday, July 27, 2021 11:27:49 AM
To: ChimeraOS/chimeraos @.>
Cc: project-sbc @.>; Author @.>
Subject: Re: [ChimeraOS/chimeraos] Unable to rotate screen for PC handheld One X Player (#251)
Display=:0 xrandr -o left yields an error of failed to changed screen configuration
changing config file doesn’t change it either
Be aware that capitalization is important. Linux (and most OS apart from DOS-based ones) are case-sensitive.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/ChimeraOS/chimeraos/issues/251#issuecomment-887609372, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASZ4DJJX4WIDB6I7LJV3OXTTZ3F7LANCNFSM5AYVUSPQ.

Great that worked for you. You might need to fiddle around with xorg.conf file specific for that device. Both screen and touchscreen rotation can be set up by it.
For reference: https://wiki.archlinux.org/title/Tablet_PC#Rotation

Thanks for your help, will close this


docs: linked hydration term to glossary

Glossary link for hydration was just at the end, added it also to the other occurrences.
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[ ] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[ ] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

Thank you!


Is it possible to fill the bar with one color at each fill percentage?

I have a bar with 180 in size and would like to fill it with 4 colors, one being every 25% fill. I tried to do it this way, but some colors take up more space than others and in addition they repeat themselves.
$('.progress-bar').circleProgress({
  value: .60,
  size: 180.0,
  startAngle: Math.PI*1.5,
  fill: {
    //color: 'rgb(250,250,250)'
    gradient: ['#e74c3c', '#e67e22', '#f1c40f', '#27ae60'],
    gradientAngle: Math.PI*1.5
  },
  emptyFill: 'rgba(200,200,200, 0.2)',
  animation: {
    duration: 3000
  },
  lineCap: 'round'
}).on('circle-animation-progress', function(event, progress) {
  $(this).find('strong').html(
    Math.round((100 * progress)*$(this).circleProgress('value')) + '<span>%</span>'
  );
});

Is #56 what you're looking for?

Almost this, but with a gradient color change. See, in the figure below, that the desired effect is produced up to 50%. But after that, the colors repeat themselves.
https://imgur.com/a/rFDg4

The only way to do this is via image fill, because currently <canvas> doesn't support conical gradient fill.
See #73, especially this comment with an example.

I got an effect similar to what I wanted using image. I had to use lineCap: 'butt', because 'round' did not work as I would have liked. Thanks!


Point to JS build

fixes #4

Great solution, but tests not pass, can you check?

It's a weid approach, but if works, nice 👌
Could be nice, if you are interested on this project, refactor the code and remove coffeescript.


FIX: defining _DEFAULT_SOURCE on linux platforms to support strsep()

GCC complains about undefined functions without the correct feature macro present. This fixes running into an error on linux systems. Currently, this only affects strsep().
Tested on Debian 11 (Bullseye) amd64 - No virtualization.

Hey, thanks a lot for testing!
Actually, I don't think it should be hidden in #ifdef __linux__. As I understand it, strsep() is not a standard C function, so _DEFAULT_SOURCE (enabling POSIX, BSD, etc APIs) should always be defined to use it. Probably a glitch in FreeBSD's headers that it's visible without it!

Addendum, I'd also leave out the comment. Nothing against comments, but that way, I'd have to list every single POSIX function used in the files that already define _DEFAULT_SOURCE, I think that would be overkill ;)

Thanks, pulled. That's what you get testing only on one single system ;)

My pleasure :)


Add translations support to json_error and JsonableError messages

Most Zulip error messages go through the json_error code path; it'd be great to add support for sending those in users' native languages; I think this shouldn't be too hard since we already have the Django translation system integrated in Zulip.

@timabbott working on this now


Problems when exporting from "Select Stars from dataset" form.

When you check the "export to excel" box and click "run query" a new dialog box asks you for export type, but the ONLY option is CSV.
I'm ok with that if it's what we want, but we should change the parent dialog's caption on the "export to excel" checkbox to just "Export Selected Stars"
and then remove the csv option from the child dialog since it's ALWAYS going to be a CSV.
The ADVANTAGE of excel files is that they're compressed, essentially as small as zipping a csv, but that's probably not worth the effort. Let's just use csv and change the checkbox to "Export Selected Stars"

ok, it turns out that exporting to excel has a huge performance impact. Its mostly the library I use.
I'll make this change

ok, fixed


Generator misses offsets

I have successfully generated an sdk, but on most classes this happens:
uint8_t           TickGroup;        // 0x00D5 (0x0001) [0x0000000000000002] (CPF_Const)
class AActor*       Owner;        // 0x00D8 (0x0008) [0x0000000000000022] (CPF_Const | CPF_Net)

If tickgroup ends in offset D6,  there should be a padding of 0x2 before owner... Is there any way to automate this instead of fixing all classes by hand?

It's your job to determine what kind of padding/alignment your game is using and configure it properly in Configuration.cpp. Looks like you're using AActor for for that offset and testing myself using a forced class padding 4 bytes those offsets are correct. I suggest you look into more how compilers do memory alignment and padding for classes/structs.

The offset comments printed are from the direct value from UProperty->Offset in your games memory, taking your compilers settings out of the question and only using the ones determined by the game/unreal engine. That's partly the reason there's a "FinalAlignment" option in Configuration.cpp if a custom one is needed, this determines the pack pragma of your final generated sdk.

Thanks for the quick response, as you said, the game was using a different alignment from the one I specified in configuration.cpp, I changed it and now the sdk works perfectly :)


URL Cleanup

This commit updates URLs to prefer the https protocol. Redirects are not followed to avoid accidentally expanding intentionally shortened URLs (i.e. if using a URL shortener).
Fixed URLs
Fixed But Review Recommended
These URLs were fixed, but the https status was not OK. However, the https status was the same as the http request or http redirected to an https URL, so they were migrated. Your review is recommended.

[ ] http://uuuuuuuu (UnknownHostException) with 1 occurrences migrated to:
https://uuuuuuuu (https result UnknownHostException).

Fixed Success
These URLs were switched to an https URL with a 2xx status. While the status was successful, your review is still recommended.

[ ] http://creativecommons.org/licenses/by-nd/3.0/ with 1 occurrences migrated to:
https://creativecommons.org/licenses/by-nd/3.0/ (https result 200).
[ ] http://docs.microsoft.com/cli/azure/overview with 1 occurrences migrated to:
https://docs.microsoft.com/cli/azure/overview (https result 301).
[ ] http://www.oracle.com/technetwork/java/javase/downloads/ with 1 occurrences migrated to:
https://www.oracle.com/technetwork/java/javase/downloads/ (https result 301).

Ignored
These URLs were intentionally ignored.

http://localhost:8080 with 2 occurrences

Closing as this PR is now outdated.


Allow export/import of pipeline processor filters

We are building up more content packs that include pipelines.  As it stands once these have been imported you have to go into each pipeline and add a processor filter, typically setting a feed and a type.
It would be great if we could export the proc filters with the pipeline so that we can make content packs that have filters in place and ready to run.
@stroomdev66 and I had a discussion about this, some thoughts:

A proc filter probably ought to be an entity in its own right, with its own UUID
The export of a pipeline should also export all the proc filters associated with it.
The export dialog maybe ought to display the proc filter entities as children of the pipeline, so there is the option to include/exclude some/all as required.
Debate as to whether the explorer tree should display the proc filters if they become entities, but this would require different screens for managing them in isolation to the pipeline, e.g. its editor would need a selector for the pipeline.  This is probably an optional piece of work.
There should be a global prop to control whether proc filters are enabled on import or not.  In our docker stack we would want them to be enabled on import so data can start flowing.
Thought needs to be given to whether to export filters that involve things like stream ID as this is not a portable concept.  Maybe we don't worry and just export everything, or let the user in the export dialog decide (if that is implemented).

Delivered by resolution to issue gh-1417


App crash on large image

What did you do?
Run this in a UICollectionViewCell
if let url = URL(string: group.mainImage) {             imageView.af_setImage(withURL: url)         }
What did you expect to happen?
Load image in the cell.
What happened instead?
All images load properly and run but if I use an image of size 20MB, the app memory usages increases to 600MB from 50 MB. The image url = http://lc.seenit.in/batch_images/Pizigani_1367_Chart_10MB.jpg

Large images require a lot of memory to display, so I suggest you ether download it and resize it for display on device, or request it in a size that can be more easily displayed without your app being killed by the system watchdog.


NameError on DeliveryReceipts

@mattwr18 and I have seen NameErrors in production when our mobiles would send delivery receipts. This is caused by not implemeted message types and unsafe Object.const_get.
Especially problematic: Threema API Gateway would retry to send the messages, probably because of 5xx HTTP error, and this has burned our credits.

This was closed in https://github.com/threemarb/threema/pull/28


Dashmate local setup error after reinstalling OS

Expected Behavior

As dash dapp maintainer, maintainer should be able to run dashmate setup locally only after reinstalling the OS and following dashmate instructions precisely.
Current Behavior

Error received when running the reset scripts.
Error message:
Error: Default group is not set. Please use --group option or set default group
Possible Solution

https://github.com/dashameter/jembe/pull/6
Deleting corrupted dashmate docker image on local would be a possible solution.
Steps to Reproduce (for bugs)

Reinstall Ubuntu OS 20.04 using Virtual Machine.
Install docker
Install node
https://github.com/dashameter/jembe - clone this repo.

cd jembe  npm run reset:dashmate
https://github.com/dashameter/jembe/blob/master/scripts/resetDashmate.sh - this is the reset script.
Context

I'm a maintainer of jembe Dapp(https://github.com/dashameter/jembe).
And I need to run dashmate on my local, so that I can test the jembe dapp.

https://github.com/dashameter/jembe/pull/6 - I changed dashmate reset script used in jembe dapp. This is the PR for it.
Your Environment

Version used:
Environment name and version (e.g. Chrome 39, node.js 5.4):
dashmate --version @dashevo/dashmate/0.20.0-dev.3 linux-x64 node-v14.16.1 node --version v14.16.1
Operating System and version (desktop, server, or mobile):
NAME="Ubuntu" VERSION="20.04 LTS (Focal Fossa)" ID=ubuntu ID_LIKE=debian PRETTY_NAME="Ubuntu 20.04 LTS" VERSION_ID="20.04" HOME_URL="https://www.ubuntu.com/" SUPPORT_URL="https://help.ubuntu.com/" BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/" PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy" VERSION_CODENAME=focal UBUNTU_CODENAME=focal
Link to your project: https://github.com/dashameter/jembe

Discussed in triage and agreed to close.


Azure AD LDAPS Logging

What is the best way to check for the logs of Azure AD LDAPS for user access.  It does not show within the user login blade of AAD of a successul ldaps authentication.

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

ID: 325a9114-36b2-a6b3-d0a4-19c522b7abf0
Version Independent ID: cbd84d9f-fbfd-d97d-9826-26947f64d18a
Content: Tutorial - Configure LDAPS for Azure Active Directory Domain Services
Content Source: articles/active-directory-domain-services/tutorial-configure-ldaps.md
Service: active-directory
Sub-service: domain-services
GitHub Login: @iainfoulds
Microsoft Alias: iainfou

@gate502 Thanks for your feedback! We will investigate and update as appropriate.

@gate502 you should be able to check the security audit logs for these types of events. There's nothing LDAP specific but it should give you what you're looking for. https://docs.microsoft.com/en-us/azure/active-directory-domain-services/security-audit-events
If you don't see these there is likely an issue with your LDAP connection and I would suggest opening a support ticket.
Hope this helps! If you have further questions feel free to tag me in the comments and I will gladly continue the discussion.


Add sum-if sample demonstrating lambdas.

Thanks. I'll be catching up on updating the site this coming weekend, so I'll take a look then.

I feel like a better sample would use std::accumulate with a conditional predicate. However, perhaps even better would just be a simple sample demonstrating how to use std::accumulate! Will come back to this later.

I've created issue #62 for creating a simpler std::accumulate sample.


Illegal Invocation from createClock

Just updated lolex and started getting a bunch of test failures with this error:
TypeError: Illegal invocation
  at webpack:///node_modules/lolex/lolex.js:746:0 <- webpack.tests.js:77738:52
  at Array.forEach (<anonymous>)
  at createClock (webpack:///node_modules/lolex/lolex.js:745:0 <- webpack.tests.js:77737:18)
  at Object.install (webpack:///node_modules/lolex/lolex.js:796:0 <- webpack.tests.js:77788:21)
  at ./node_modules/sinon/lib/sinon/util/fake_timers.js.exports.useFakeTimers (webpack:///node_modules/sinon/lib/sinon/util/fake_timers.js:29:0 <- webpack.tests.js:125808:21)
  at Sandbox.useFakeTimers (webpack:///node_modules/sinon/lib/sinon/sandbox.js:310:0 <- webpack.tests.js:123807:46)

Tests are being run in Chrome 68 headless.

Not good. Thanks for reporting! Which version worked? Any reproducible code? Hard to dig into without a test case.

It looks like both 2.4.0 and 2.4.1 are broken, but 2.3.2 works.
It seems to happen as soon as a test calls sinon.useFakeTimers()

This also looks like the issue I addressed here: https://github.com/sinonjs/lolex/pull/162/files
I also ran into this when trying to run the lolex tests in headless chrome.

Yeah, I think it's the same as #162 and #167

This started happening to me yesterday as well. I get TypeError: Illegal invocation in headless chrome (and normal chrome as well) as soon as we try to use fake Timers

Versions 2.4.0 and 2.4.1 also don't work on Firefox.
You can see it in build result: https://travis-ci.org/sinonjs/lolex/jobs/376426623#L2348

Fixed by #162. Released as 2.4.2.

I'm worrying I may have done something wrong as other people say it is fixed, but it still doesn't work for me. I ran a minimal code on console to reproduce the issue to make sure my NPM installation does not affect the result:
navigator.userAgent
// Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/67.0.3396.87 Safari/537.36
const value = Object.create(performance);
Object.getOwnPropertyNames(window.Performance.prototype)
// (17) ["timeOrigin", "onresourcetimingbufferfull", "timing", "navigation", "memory", "now", "getEntries", "getEntriesByType", "getEntriesByName", "clearResourceTimings", "setResourceTimingBufferSize", "mark", "clearMarks", "measure", "clearMeasures", "toJSON", "constructor"]
Object.getOwnPropertyDescriptor(Performance.prototype, 'mark').writable
// true
value.mark = Performance.prototype.mark;
Object.defineProperty(window, 'performance', { writable: false, value });
performance.mark('foo');
// VM130:1 Uncaught TypeError: 'Illegal invocation'

Can anyone confirm this, or point out a mistake?

Confirmed, this is still broken on my build as of Sinon release 6.1.5, which is running lolex 2.7.1.

Just to clarify my confirmation as well, this is occurring in Chrome and Headless Chrome.

The comments on this still being a thing probably point to #200, which a PR I sent in is about to close (hopefully). Closing this while keeping #200 open, as I suspect some comments are basically duplicates.


container registry proxy change for azure china cloud

What this PR does / why we need it:
We have set up a new container registry proxy(gcr.akscn.io) in azure china, crproxy.trafficmanager.net is located in west us, it's too slow. As comparision, I deployed a k8s cluster with 100 nodes by acs-engine in azure china, it would cost about 2 hours, while if using gcr.akscn.io, it only costs 15 min to complete, that's a magnificant change.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
@jackfrancis
FYI @karataliu @feiskyer
If applicable:

[ ] documentation
[ ] unit tests
[ ] tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:

container registry proxy change for azure china cloud

/lgtm


Tutorial

It would be great if there was some tutorial, documentation or example somewhere. Right now I have no clue how to use this. There used to be some documentation, but that seems all gone. The official documentation seems to have an installation section (useful), but no "how do you use this". If there were some simple examples (i.e. IJulia notebooks that show a plot or just anything) that would also work.

The original documentation of RCall is removed as it is outdated and no longer relevant after series of updates.
The documentation is now generated by Documenter.jl.
Examples could be written by utilizing the example feature of Documenter.jl.

Fixed by #105. Thanks, this is great!


Clear Telephone number error

When I am using this command to clear the user's telephone number.
Set-CsOnlineVoiceUser -Identity<EMAIL_ADDRESS>-TelephoneNumber $null
I have an error this error
Microsoft.Teams.ConfigAPI.Cmdlets.internal\Set-CsUserGenerated : Value cannot be null.
Parameter name: Value cannot be null or empty.

Thanks for feedback!
I don't know which version of Teams PS Module you are using, but I'm not able to repro the issue on the latest GA module 3.0.0. In that version Set-CsOnlineVoiceUser -Identity<EMAIL_ADDRESS>-TelephoneNumber $null works for me.
You could also try using the new Teams PS cmdlet Remove-CsPhoneNumberAssignment to remove the phone number.


Failed: To display results of demo.py with Pycharm connecting to a remote ubuntu server

Environment: Pycharm with remote ssh interpreter
Error happens when trying to run demo.py to display the results:
Got keys from plugin meta data ("xcb")
QFactoryLoader::QFactoryLoader() checking directory path "/home/anaconda3/envs/dev/bin/platforms" ...
loaded library "/home/anaconda3/envs/dev/lib/python3.6/site-packages/PyQt5/Qt/plugins/platforms/libqxcb.so"
qt.qpa.xcb: could not connect to display
qt.qpa.plugin: Could not load the Qt platform plugin "xcb" in "" even though it was found.
This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.

Available platform plugins are: eglfs, linuxfb, minimal, minimalegl, offscreen, vnc, wayland-egl, wayland, wayland-xcomposite-egl, wayland-xcomposite-glx, webgl, xcb.

Does it exists some other ways to display the results, or just saving the results to jpg, png or any other ways?

You can save the point cloud and predicted boxes, and then use mayavi to visualize the result on your local pc or laptop.

You can save the point cloud and predicted boxes, and then use mayavi to visualize the result on your local pc or laptop.

Thanks

@jihanyang hi, thank you for sharing. This is my first contact with mayavi. I also encountered visualization errors when running demo under docker. Can you explain in detail how to modify the code to save the point cloud and predicted boxes. If you can, please explain how to use mayavi on your local PC to load and display. Thanks!

@jihanyang I save the predicted data and try to visualize it on my macbook, but I the mayavi window shows nothing.

@bluestyle97 Hello, please provide more details about your problem. I have tried to visualize with mayavi on macbook before, but I haven't meet questions.


Support for metric wildcards

I have noticed that stackdriver exporter requests the lists of metrics at some point; it'd be great to be able to specify the list of metrics via wildcard instead of each of them explicity, e.g. compute.googleapis.com/*.

Id say @elsbrock that if you just specify a prefix, yeah, it lists and fetches everything. Is that not the case in your example?

It does? I never tried it, just used the above expression and that did not work. I will try it out later. Thanks!

I can confirm this works. Should the README maybe enhanced with this info?

You are right: maybe another example here: https://github.com/frodenas/stackdriver_exporter#example , maybe pointing out how to import all k8s/nodes related metrics ("kubernetes.io/node/", https://cloud.google.com/monitoring/api/metrics_kubernetes), could facilitate the understanding of that functionality. I think @frodenas is not very active now in this project, but feel free to submit a PR for that!


Update index.js

minigrid not working.

@pulse-studio hey folks! The PR looks alright but there was a reason to set display: inline which I can't remember now 😄 I'll have a look at it again and come back to you 👍

Good!
Waiting for news;) :+1:

hi @pulse-studio it is working already. The issue on that jsbin is because you can't import scripts from GitHub directly so this src="https://github.com/henriquea/minigrid/blob/master/src/index.js" won't work. I replaced the src using npmcdn and you can see the result here.

On the local project I have does not work.

Not working!
last version 3.0.4.
demo - http://pulse.studio/minigrid/
Version 3.0.3 works


Feat/2423 align admin with section 2d3d qualifications changes

What's included?
closes #2423
Who should test?
✅ Product owner
✅ Developers
✅ UTG
How to test?

Go to the preview link: https://jac-admin-develop--pr2457-feat-2423-align-admi-o5n9agb4.web.app/exercise/X2tDg5AKt18xvzBRJFYW/applications/applied/application/g6e7d5gxp7FTEDnrwAPv
Click Edit
Change and save the answer for Are you applying under Schedule 2(3)?

If the answer is Yes, it should appear None of above option for Qualification
If the answer is No, it should not appear None of above option for Qualification
If the answer is changes, the Qualification(s) should be reset

Risk - how likely is this to impact other areas?
🟢 No risk - this is a self-contained piece of work
Additional context
Include screen grabs, video demo, notes etc.
Related permissions
Have permissions been considered for this functionality?

No permission changes required
Permissions have been added / updated. Details:

PREVIEW:DEVELOP
can be OFF, DEVELOP or STAGING

@KoWeiJAC This is not working for me. First of all, there was no option set under eligibility in set-up (you have to choose 2d or 2(3). I've now set it to 2d and at that point the Are you applying under section 2d appeared. However, irrespective of my answer, None of the above appears in the dropdown. Also, it doesn't appear to reset when the initial answer to Are you applying under...  changes.

@KoWeiJAC This is not working for me. First of all, there was no option set under eligibility in set-up (you have to choose 2d or 2(3). I've now set it to 2d and at that point the Are you applying under section 2d appeared. However, irrespective of my answer, None of the above appears in the dropdown. Also, it doesn't appear to reset when the initial answer to Are you applying under... changes.

@KoWeiJAC This is not working for me. First of all, there was no option set under eligibility in set-up (you have to choose 2d or 2(3). I've now set it to 2d and at that point the Are you applying under section 2d appeared. However, irrespective of my answer, None of the above appears in the dropdown. Also, it doesn't appear to reset when the initial answer to Are you applying under... changes.

@nickaddy
Not sure if the exercise being reset, I set it to schedule 2(3) originally, thank you for fixing it.
I've had following changes:

'None of above' appears according to schedule question, it will change when schedule being selected not saving.
Reset all the qualifications after the schedule question change saved. Because it will be to safer to reset the qualification after schedule being saved.
Could you help me to confirm if it is ok ? Thanks
https://github.com/jac-uk/admin/assets/156695133/1c600910-f141-4e1e-aeeb-264daa57fca3


[Error] Part of speech: null

When a word has no "part of a speech" information, Part of speech: null is shown. For example:
❯ got -o -f es -t en "ah"
Translated text: ah

Part of speech: null

Definition:
- Indica que el que habla se ha dado cuenta de algo o lo ha comprendido.
Use in sentence:
- ¡ah!, entonces tú no estabas

Definition:
- Indica admiración, sorpresa o pena.
Use in sentence:
- ¡ah!, ¿sí?

Definition:
- Indica satisfacción o alegría.
Use in sentence:
- ¡ah, qué bien lo pasábamos entonces!

Definition:
- Se utiliza para advertir de algo o para decir algo que se había olvidado
Use in sentence:
- ah, y no se lo digas a nadie que todavía no es seguro

Part of speech: interjection

- Ha!: ¡Ah!
- Ahoy!: ¡Ah!

Engine: Google
Version: 2a5b336c (got -v don't show a thing. since I just installed I could assume I have the last current commit)

This comment applies also in this case.
However here I can substitute the null part with something else (or just leave it empty).
If you have any suggestion in this regard I'll happily implement it.

Thank you for given a look to this. Maybe the Part of speech heading could be completely omitted when null is provided by the engine. What do you think?

I too was thinking about something like this, but I noticed that it could potentially be misleading since one can interpret a translation as some other part of speech.
E.g. if a word has multiple definition/translation based on different context, by omitting one header it could led to think that all translations belongs to the same part of speech, which is incorrect since it is instead undefined.
I hope it is clear enough.
IMO there are 2 viable options, both of which leave the header:

omit part of speech
use undefined or similar concepts to let the user know that the part of speech is not specified

I see. undefined sounds like a good approach then.


Add Ability for User to Customize Email Notification Frequency

UI:

This commit adds the backend logic for
allowing users to enable or disable email
notifications. The implementation ensures
that only issue notification emails get disabled
and important emails are still sent regardless
of the setting.
The UI to toggle this setting has not yet been
implemented.

This commit completes the functionality to allow
users to disable their own email notifications.

Codecov Report

Merging #7813 into master will decrease coverage by <.01%.
The diff coverage is 28.57%.

@@            Coverage Diff             @@
##           master    #7813      +/-   ##
==========================================
- Coverage   41.41%   41.41%   -0.01%
==========================================
  Files         474      475       +1
  Lines       63876    63892      +16
==========================================
+ Hits        26453    26458       +5
- Misses      33978    33990      +12
+ Partials     3445     3444       -1

Impacted Files
Coverage Δ

models/migrations/migrations.go
1.3% <ø> (ø)
:arrow_up:

routers/user/setting/account.go
25.8% <0%> (-0.87%)
:arrow_down:

models/migrations/v93.go
0% <0%> (ø)

models/user.go
50.68% <60%> (-0.01%)
:arrow_down:

models/issue_mail.go
51.54% <75%> (ø)
:arrow_up:

models/gpg_key.go
56.66% <0%> (+0.83%)
:arrow_up:

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 cde95f9...fd51209. Read the comment docs.

I would like the third option send mail only mentioned me.

I would like the third option send mail only mentioned me.

That is a good idea. I'll add it here for now but I think in the long run, that should be an option that is repository specific.

@lunny Could you take another look?

Could you add some tests for this?

@guillep2k Done.

@guillep2k Done.

Sorry, I was expecting something for checking if the actions (e-mails) are generated in the correct cases, i.e., create an issue, comment on an existing issue  and check which e-mails were generated and which weren't. Like this:
https://github.com/go-gitea/gitea/blob/ca6fb004ac50fc924861112403895d637c6a2d1d/models/repo_watch_test.go#L71-L87
Checking the e-mail generation is useful for the future, as many changes in the code made by others could inadvertently break this functionality. A test will help on that case.

I'm not sure how that would work for the email system in its current state. The function that has to be run to check will send emails on its own. Maybe it could be done as part of integrations but I'm a bit short on time at the moment.

Can change the selected option for every user by Adminatrator?

Can change the selected option for every user by Adminatrator?

Not currently, I'll add a default option that admins can choose.

@lunny @lafriks
Could you take a look?

And how onmention work? It seems there is no related code changes.

The check for watch and participants is "do not send if notifications are not enabled"  so onmention will not be sent, and the check for mentions are "send emails if not disabled" so onmention will be sent.

As for the build error, please check the conversation here:
https://github.com/go-gitea/gitea/pull/8021#issuecomment-525940166

nit in migration comment but otherwise LGTM

modules.txt needs to updated with value golang.org/x/crypto v0.0.0-20190829043050-9756ffdc2472 for golang.org/x/crypto line

It seems that onmention does not trigger sending a notifications for when a user is assigned to an issue. Does it make sense to include this scenario?

@mario-neuhold AFAICT assigments should send an e-mail notification to the assigned user (if they have notifications enabled) as of version 1.11.1. If you're having problem with that, could you please open an issue and provide details?


My style does not work

I use previewTemplate, and want to use some styles.
Inline CSS is work normally, but CSS in

This problem was due to scoped style.
It worked well without scope

@tatane616 This saved my day.

@tatane616 This saved my day.

You can also use /deep/ before css class in scoped, like this:
<style lang="scss" scoped>
/deep/.dz-message {
  font-size: 20px;
}
</style>

You can also use /deep/ before css class in scoped, like this:
<style lang="scss" scoped>
/deep/.dz-message {
  font-size: 20px;
}
</style>


Bot stop respond after sending two zero values.

/random command
Bot stop respond after sending values -0 and 0.
And...well...sorry :)

I suppose, reason of this bug is that random.org api don't support min equal to max. I fix this bug later. Thanks for feedback

Last commit fixed this bug. Closing the issue.


add some error handling

Thanks for the suggestion, I did add a lot more error handling and safety check in PR #3.


fix ranchervm-backend service type

change nodeport to ClusterIP

Merged this as part of https://github.com/rancher/vm/pull/97 before I noticed this PR. Thanks

okey!


CodeceptJS v3.0.6 doesn't show number of passed/failed tests  in combination with mocha-junit-reporter

What are you trying to achieve?
To get number of passed/failed test running in console output report when I use CodeceptJS version: v3.0.6 version and mocha-junit-reporter
I'm using next command - npx codeceptjs run-workers 5 --plugins allure --reporter mocha-junit-reporter
What do you get instead?

Provide console output if related. Use --verbose mode for more details.

OK  | 0 passed   // 2m

Provide test source code if related

// paste test

Details

CodeceptJS version: v3.0.6
NodeJS Version: v12.16.3
Operating System: any
mocha-junit-reporter: 2.0.0
playwright

It's only stop working with CodeceptJS version: v3.0.6 and continue working with CodeceptJS version: v3.0.5

same issue. passing the flags --reporter eats up all the verbose logs from codeceptjs


Icon Column Alignment & Breadcrumbs - Our Culture

The icons two columns section on the Our Culture does not render properly - https://main--shredit--stericycle.aem.live/en-us/about/careers/our-culture
The icons, headings, and descriptions, in the left and right columns should be aligned vertically.

@MarquiseRosier a new issues just appeared on this page - https://main--shredit--stericycle.aem.live/en-us/about/careers/our-culture
There's so weird breadcrumbs being added. Can you see that this entire white breadcrumb section is removed. I don't see it in the document to delete it.


#12 Make ChipCombobox Vaadin conform, add Setters

Fixes #12
This PR implements the changes as requested in issue #12. I've added three new Setters (including the ones required from the HasItems interface), which call the matching methods that already existed previously.
setLabel(String) calls this.withLabel(String)
setPlaceholder(String) calls this.withPlaceholder(String)
setItems(Collection<T>) calls this.withAllAvailableItems(ArrayList<T>, true)

Changes should be corrected now

LGTM;
Added missing setter for setFullComboBoxWidth


KafkaBinderProperties does not override with the KafkaBinderConfigurationProperties

the Kafka Streams Binder topic does not seem to take the overrides provided in application.yml (or application.properties)
I have added the property - (default port for Kafka is different)
spring.cloud.stream.kafka.streams.binder.brokers: localhost:29092
The KafkaBinderConfiguration seems to read from KafkaBinderConfigurationProperties by initializing a new new KafkaBinderConfigurationProperties() and does not read the binder properties in application.yml
 @Configuration
@ConditionalOnMissingBean({Binder.class})
@Import({KafkaAutoConfiguration.class, PropertyPlaceholderAutoConfiguration.class, KafkaBinderHealthIndicatorConfiguration.class})
@EnableConfigurationProperties({KafkaExtendedBindingProperties.class})
public class KafkaBinderConfiguration {
    @Autowired
    private KafkaExtendedBindingProperties kafkaExtendedBindingProperties;
    @Autowired
    private ProducerListener producerListener;
    @Autowired
    private KafkaProperties kafkaProperties;

    public KafkaBinderConfiguration() {
    }

    @Bean
    KafkaBinderConfigurationProperties configurationProperties() {
        return new KafkaBinderConfigurationProperties();
    }

    @Bean
    KafkaTopicProvisioner provisioningProvider(KafkaBinderConfigurationProperties configurationProperties) {
        return new KafkaTopicProvisioner(configurationProperties, this.kafkaProperties);
    }

The KTableBinderConfiguration is doing the right thing with resolved KafkaBinderConfigurationProperties
public class KTableBinderConfiguration {
    @Autowired
    private KafkaProperties kafkaProperties;
    @Autowired
    private KafkaStreamsExtendedBindingProperties kafkaStreamsExtendedBindingProperties;

    public KTableBinderConfiguration() {
    }

    @Bean
    public KafkaTopicProvisioner provisioningProvider(KafkaBinderConfigurationProperties binderConfigurationProperties) {
        return new KafkaTopicProvisioner(binderConfigurationProperties, this.kafkaProperties);
    }

    @Bean
    public KTableBinder kTableBinder(KafkaStreamsBinderConfigurationProperties binderConfigurationProperties, KafkaTopicProvisioner kafkaTopicProvisioner, KafkaStreamsBindingInformationCatalogue KafkaStreamsBindingInformationCatalogue) {
        KTableBinder kStreamBinder = new KTableBinder(binderConfigurationProperties, kafkaTopicProvisioner, KafkaStreamsBindingInformationCatalogue);
        return kStreamBinder;
    }
}

the workaround for the override is to use the Kafka Spring Boot properties.
spring.kafka.producer.bootstrap-servers: localhost:29092

Creating a new bean like that is fine because of...
@ConfigurationProperties(prefix = "spring.cloud.stream.kafka.binder")
public class KafkaBinderConfigurationProperties {

That's what triggers the property population.
However, note that this class is shared between the kafka message channel binder and the KStream binder. I don't see the .streams element, but I am not familiar with the KStream binder so I don't know if there is some other magic involved - I do see that the documentation says to add the .streams element to the property.
Can you try spring.cloud.stream.kafka.binder.brokers: localhost:29092 (without the .streams part).
cc/ @sobychacko

I just checked the KStreams binder config and it has
	@Bean
	@ConfigurationProperties(prefix = "spring.cloud.stream.kafka.streams.binder")
	public KafkaStreamsBinderConfigurationProperties binderConfigurationProperties() {
		return new KafkaStreamsBinderConfigurationProperties();
	}

So it looks like you are using the message channel binder, not the KStream binder so you must not have that .streams element.

Thanks @garyrussell . Yes it works with -
spring.cloud.stream.kafka.binder.brokers: localhost:29092
thanks. the documentation talks about the using the spring.cloud.stream.kafka.binder.brokers
what through me off, i guess, is the configuration (Map with a key/value pair containing properties pertaining to Apache Kafka Streams API.) is expected under the spring.cloud.stream.kafka.streams.binder.
thanks. this solves for what i am looking for. Thanks Gary for your help.

spring.cloud.stream.kafka.stream.binder.brokers:  should have worked. It didn't?
For example: spring.cloud.stream.kafka.streams.binder.brokers: 192.168.99

It sounds like he was incorrectly using the KStream binder configuration properties for a message channel binder,

Ah - thanks @garyrussell.
@prashantulavapalli  Can we close this issue then?

@sobychacko - Yes we can close this. thanks for the quick closure on this.


/usr/bin/env: 'node\r': No such file or directory

I've just installed s3rver on Ubuntu 18.04 with the following:
sudo npm install s3rver -g

But when I try to launch it I get the following error:
$ s3rver
/usr/bin/env: «node\r»: Aucun fichier ou dossier de ce type
$ LANG=C s3rver
/usr/bin/env: 'node\r': No such file or directory

It seems /usr/local/bin/s3rver uses Windows newlines (\r\n) and /usr/bin/env doesn't like this.

Can you check if this is still a problem in v3.4.0?

Seems to work, thank you!


ColorScheme - Make selections in mini map more visible

I like the selections in the minimap to stand out more.
Consider the following screenshot. Can you recognize which part of the text is currently selected? I can not (and there are definitely selections). Hence, this is a point of improvement.

It would be also great to more visibly color other instances of the currently hovered word.

Hi @pykong I can't change mini map, only the bacground of currently section selected.

@mauroreisvieira thanks for the fast feedback on this issue.
It is unfortunate that Sublime Text does not support a more fine-grained control for allowing us to address the minimap coloring separately.
Hence I am closing this for now.

I asked on the discord server about how to achieve this.
Indeed the response was negative. There currently does not seem a way to style the minimap differently from the text area.


Add 'ALPNProtocols' option to TLS Node

Hi all,
In some branches network, where the security policy is strict (eg. only allowing http/https), running mqtt using port 8883 is not working. My options is using only port 443, but it's not working. After few dyas, i learned that we need to specified Application Layer Protocol Nego (ALPN) options in TLS handshaking.
I'm not into a programming, but i try editing one of the file below, is working.
file: @node-red/nodes/core/io/05-tls.js
Excerpt:
TLSConfig.prototype.addTLSOptions = function(opts) {
...
...
opts.ALPNProtocols = ["x-amzn-mqtt-ca"];
return opts;
}
My question is, can we include this options in the node-red GUI on mqtt tls options? At least user can specified what is the ALPN strings that they want.
Thanks you

I would like to take this up, if not already taken. @knolleary

Hi @tarunbatra - sure. Before you dive into the code, would be good to agree what exactly the scope would be.
There are lots of possible options on the TLS node - I had wanted to do a bit of a review to see if there were any other 'obvious' ones to ought to be exposed in some way.

Yes, that makes sense.

Hello all, is there any update regarding this feature ?

Same question here. I have the same use case, but this issue seems to be stagnating.
@knolleary are there any news?
@tarunbatra are still interested in taking this?

Sorry for not replying earlier. I have tested and it works, just specify the mqtt server port to 443 (from the usual 8883) and ALPN protocols to "x-amzn-mqtt-ca" (for AWS IOT service)
Thanks to you all


Tracker: Updating the configuration automatically without sigHUP with --watch-config

If it's okay with you I'd like to add a new parameter --watch-config that uses fsnotify to watch the configuration file for changes.
The default will false (which means the config is only reloaded on a HUP signal). When set to true, the config will still reload on HUP but also when fsnotify detects a change.
I'll create a PR if this is okay to add. I really do not see the point in running sidecars and all that for this functionality.

Perhaps something opionated: I will implement this to watch the actual certificate files as well. Please confirm that this is okay or whether a second parameter would be more appropriate (--watch-certificates or something)


DOC: Add build from source guidance

As there are some C++ code in this project, I hope there is a "build from source" part  in the doc or readme. Thanks!

I mean, what dependency or compiler version developers should prepare.

Hi, @luweizheng . I make a PR about this issue. The information about how to build xoscar from source will be shown in the README.md.


Add support for elixir-ls

Name: elixir-ls
URL: https://github.com/JakeBecker/elixir-ls
A language server for Elixir code.
With my previous setup, using neovim language server client, it was working really well. The only bad thing was that elixir-ls wasn't super easy to install

This PR adds elixir-ls support and works well for me: https://github.com/w0rp/ale/pull/1956

Oh sorry.
I searched (I pretty sure I did) but didn't find it. Thanks for pointing me in the right direction :smile:


什么时候支持集成到 HarmonyOS，期待！！！

I guess you want: Fxcit5 for Android. It can now run on Mobile Harmony (and other forks of Android), but it doesn't work on OpenHarmony


Update dev 0917

Codecov Report

Merging #2314 into dev will increase coverage by <.01%.
The diff coverage is 90.68%.

@@            Coverage Diff            @@
##              dev   #2314      +/-   ##
=========================================
+ Coverage   93.79%   93.8%   +<.01%
=========================================
  Files         163     163
  Lines       18579   18581       +2
=========================================
+ Hits        17427   17430       +3
+ Misses       1152    1151       -1

Impacted Files
Coverage Δ

qiita_db/test/test_artifact.py
99.84% <ø> (ø)
:arrow_up:

qiita_db/test/test_commands.py
99.6% <ø> (ø)
:arrow_up:

...ita_pet/handlers/study_handlers/tests/test_base.py
98.14% <ø> (ø)
:arrow_up:

qiita_db/__init__.py
100% <ø> (ø)
:arrow_up:

qiita_db/commands.py
96.55% <ø> (ø)
:arrow_up:

qiita_db/metadata_template/constants.py
100% <ø> (ø)
:arrow_up:

qiita_pet/test/test_prep_template.py
97.87% <ø> (ø)
:arrow_up:

qiita_pet/handlers/api_proxy/tests/test_studies.py
99.4% <100%> (ø)
:arrow_up:

...ta_pet/handlers/analysis_handlers/base_handlers.py
98.63% <100%> (ø)
:arrow_up:

...ta_pet/handlers/study_handlers/listing_handlers.py
90% <100%> (ø)
:arrow_up:

... and 95 more

Continue to review full report at Codecov.

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


Feature/1859/creating hosted entry should return 201

#1859
The abstract method that creates the hosted workflow or tool now returns
a Response created object instead of the Workflow, which results in a
201 returned for the Restful call
Please make sure that you've checked the following before submitting your pull request. Thanks!

[x] Check that you pass the basic style checks and unit tests by running mvn clean install

Coverage decreased (-8.3%) to 68.254% when pulling 09dc37228f505cf337212bef11ea74d3769fe085 on feature/1859/creating-hosted-entry-should-return-201 into 66c2d77912257293a5bf9432da6ddd0272588ec6 on develop.

I am closing this PR based on feedback from the Dockstore team.

A different way to solve this is to inject HttpServletResponse and explicitly set the status:
    @POST
    @Path("/hostedEntry")
    @Timed
    @UnitOfWork
    public T createHosted(@ApiParam(hidden = true) @Auth User user,
        @ApiParam(value = "For tools, the Docker registry") @QueryParam("registry") String registry,
        @ApiParam(value = "name", required = true) @QueryParam("name") String name,
        @ApiParam(value = "Descriptor type", required = true) @QueryParam("descriptorType") String descriptorType,
        @ApiParam(value = "For tools, the Docker namespace") @QueryParam("namespace") String namespace,
            @Context HttpServletResponse httpServletResponse) {

        // check if the user has hit a limit yet
        final long currentCount = getEntryDAO().countAllHosted(user.getId());
        if (currentCount >= calculatedEntryLimit) {
            throw new CustomWebApplicationException("You have " + currentCount + " workflows which is at the current limit of " + calculatedEntryLimit, HttpStatus.SC_PAYMENT_REQUIRED);
        }

        descriptorType = checkType(descriptorType);
        T entry = getEntry(user, registry, name, descriptorType, namespace);
        checkForDuplicatePath(entry);
        long l = getEntryDAO().create(entry);
        T byId = getEntryDAO().findById(l);
        elasticManager.handleIndexUpdate(byId, ElasticMode.UPDATE);
        httpServletResponse.setStatus(HttpServletResponse.SC_CREATED);
        try {
            httpServletResponse.flushBuffer();
         } catch (IOException e) {
            e.printStackTrace();
        }
        return byId;
    }

Which almost seems OK, except for the ugly flushBuffer... See this StackOverflow page.

It's better, but still not quite sold


RecordBatchWriter trait can't be used in dynamic context due to close(self)

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
I'm trying to use RecordBatchWriter trait to introduce dynamism in the type of output - e.g. depending on the command line options, I'd like to write parquet, csv, json or ipc. I know that corresponding writers all implement this trait, thanks to #4206 and #4228 by @alexandreyc.
Specifically, I was planning to use Box<dyn RecordBatchWriter> and pass it to the writing code, something like this:
let writer: Box<dyn RecordBatchWriter> = match args.format {
    "parquet" => Box::new(parquet::arrow::ArrowWriter::try_new(file, schema, None)),
    "csv" => Box::new(arrow::csv::Writer::new(file)),
    ...
}

writer.write(my_batch);
writer.close();  // this is required to write the formats correctly

Unfortunately, due to close method in that trait consuming self, we can't really use this approach, as now close() needs to know the exact type of the writer. Calling close() on the trait object gives compilation error "the size of dyn RecordBatchWriter cannot be statically determined". See example in Rust playground.
Describe the solution you'd like
Ideally, switch to close(&mut self). This is not backwards-compatible. On the other hand, I don't see how this trait can reasonably be used in client code in the current state, so maybe it's worth it? Let me know if I'm missing anything.
Describe alternatives you've considered
Manually downcasting to specific types and then calling close() on them, but that doesn't seem to work, as RecordBatchWriter needs to explicitly have as_any method or similar.

Ok trying to reimplement it myself, looks like the problem is that parquet::arrow::ArrowWriter has close(self) too. Any advice on how to deal with this would be appreciated.

Well good news is that now we have finish(&mut self), added in March in #5471, so maybe we can switch to that?

On the other hand, I don't see how this trait can reasonably be used in client code in the current state

It is designed to be used with a generic type constraint. IIRC we switched to taking self to make it easier when working with non-static Write, and to prevent calling close multiple times

That works in simple cases, yes. I've tried this in my slightly more complex code and then spent unreasonable amount of time trying to make it work. Specifically, the writer in my case is stateful, so I created a struct with generic W: RecordBatchWriter and needed to specialize it for every format manually, like this
struct MyWriter<W: RecordBatchWriter> {
    writer: W,
    ...
}

let writer: Box<dyn MyWriter> = match args.format {
    "parquet" => Box::new(MyWriter::new(parquet::arrow::ArrowWriter::try_new(file, schema, None), ...)?),
    ...
}

use_my_writer_as_part_of_a_pipeline(writer);

Now we can't really do that ^^ because MyWriter is not a trait, so we need to introduce another higher-level trait.
Here's alternative proposal - can we add a finish(&mut self) to the trait to enable dynamic dispatch? We have a similar semantics in parquet where we have consuming close and non-consuming finish. That should be backwards compatible and would resolve the difficulties.

One other consideration is the fn write_batches above can only write the whole file at a time, which means we have to keep all the file contents in memory. If we want to do any kind of streaming, we're back to the same difficulties.

Here's alternative proposal - can we add a finish(&mut self) to the trait to enable dynamic dispatch

I believe the trait would still not be object safe, due to the presence of the close method

If we want to do any kind of streaming, we're back to the same difficulties.

An iterator / stream can compute the next batch on demand

Here's alternative proposal - can we add a finish(&mut self) to the trait to enable dynamic dispatch

I believe the trait would still not be object safe, due to the presence of the close method

It won't be object-safe, but we can use both write and finish methods dynamically - see playground.

If we want to do any kind of streaming, we're back to the same difficulties.

An iterator / stream can compute the next batch on demand

Uffh I'd need to restructure the whole program to support that :)

so we need to introduce another higher-level trait.

Or you can just define your own trait over the writers...

That's what I'm doing now and it's sad that my own trait is so close to existing one and I have to reimplement basically the same things.


Black actions

🛠 Changes

Add black workflow for actions
Delete flake config and actions

📝 Associated issues
#2
🤔 Considerations

Remove the other unused branches

Remove the other unused branches

Done!


(user profiles) Search for Users

Create a search bar and search results page with the necessary backend components such that users can search for other users in the navigation bar by looking up emails (May edit to include search by username if we decide to edit the User model to include usernames)

This issue depends on #29 and #28 to be able to send request and direct to profile pages from the search results page

This issue concerns the whole app so we are collaborating with the games team to handle it.

Closing statement: we met with the games team and decided to work together on this feature. We will create two issues (for not) to handle the searching functionality in the app. Each issue will have members from both teams working on it. The games team documented their progress in #89
@omarelkashef met with the games team to devise an initial plan for collaboration and draft for what work needs to be done
@24raniwalar implemented a search bar that redirects to a profile given the email of a user.

Issue Score: Excellent
Comments:
Good work communicating with game management and also updating them on matches' need for a search bar too. It looks like most of the documentation and research is on the other team's issue. Also since you guys are breaking this down into more issues, I'm labeling this one as minor.


release:archive is allegedly including VCS history

Some of our friends in the open source community have reported that the downloadable archive for 4.3.1 is around 300mb, which includes VCS history.
Cow should automatically exclude this - we may need to investigate why this has happened.
It was also reported that 4.3.1's archive is tied to PHP 7.1. We fixed this in #65 recently, so perhaps the releaser's cow version was not up-to-date.

We no longer do archive releases, hooray!


Disable intermittent test-cases

Purpose

This is an aggregation of #22991 #22993 #22995

Codecov Report

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

@@           Coverage Diff           @@
##           master   #23004   +/-   ##
=======================================
  Coverage   14.59%   14.59%
=======================================
  Files          51       51
  Lines        1411     1411
  Branches      219      219
=======================================
  Hits          206      206
  Misses       1189     1189
  Partials       16       16

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 efcb688...8e9faf1. Read the comment docs.


wtf config?

I would like to understand what are the differences? why is there a hyphen somewhere and not somewhere...

Copy paste syndrome


Fixes and optimization

Hi,
I did some optimization and fixes since my last PR (about lazy loading).

Take last meta informations (instead of first), in order to make schema inheritance easier
Use dedicated function to retrieve meta informations
Forward the isInput parameter to descToField method (prevent input sub schema to be interpreted as output)

Regards,
Jérémy

Sorry for the incredibly long delay in seeing this. Looks like you might need to rebase. Thanks for the PR!


[SPARK-31073][WEBUI] Add "shuffle write time" to task metrics summary in StagePage.

What changes were proposed in this pull request?

I've applied following changed to StagePage.

Added Shuffle Write Time to task metrics summary.
Added checkbox for Shuffle Write Time as an additional metrics.
Renamed Write Time column in task table to Shuffle Write Time and let it as an additional column.

Why are the changes needed?

Task metrics summary doesn't show Shuffle Write Time even though it shows Shuffle Read Blocked Time.
Shuffle Read Blocked Time is let as an additional metrics so I also let Shuffle Write Time as an other additional metrics.
Does this PR introduce any user-facing change?

Yes. After this change, task metrics summary can show Shuffle Write Time and its visibility is controlled by a checkbox.

Write Time column is already shown in task table but the title is ambiguous so I've renamed it as Shuffle Write Time.
After this change, this column is also additional column like Shuffle Read Blocked Time.

How was this patch tested?

I've tested manually using following code and confirm the UI.
sc.parallelize(1 to 1000).map(x => (x,x)).reduceByKey(_+_).collect

Test build #119472 has started for PR 27837 at commit 33c298e.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24203/
Test PASSed.

Test build #119472 has finished for PR 27837 at commit 33c298e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119472/
Test PASSed.

I'll update it. Thanks!


Qubit Rotation in JAX

Since the idea of qgrad is to extend autodiff capabilities using the existing autodiff libraries (like JAX) to a more general set of functions (quantum hamiltonians as Qobj in this case), a good first step is to produce a working tutorial of something in JAX that uses autograd. Pennylane qubit rotation tutorial seems like a good first step. Pennylane also uses autograd wrapper btw.
I am working on a notebook to cover this.

This may also be related, a notebook in Pennylane on VQE with quantum natural gradient, also showing it on the QuTiP Bloch sphere cc @quantshah.

Closing this on PR https://github.com/qgrad/qgrad/pull/10


Omitting tsconfig option tsconfig-object fails

grunt-ts at 5.1.0
When I omit the tsconfig option within the tsconfig object:
ts: {
      default: {
          tsconfig: {
            //tsconfig: 'tsconfig.json',
            ignoreFiles: false,
            updateFiles: true,
            overwriteFilesGlob: false
          },
          //src: ['<%= yeoman.app %>/scripts/**/*.ts', 'typings/tsd.d.ts']
      }
}

The result when running grunt ts:
Error reading "undefined": {}

If I don't omit the tsconfig option, then everything is fine. According to the documentation using tsconfig inside the tsconfig object is optional and should default to tsconfig.json when omitted.

Hi - sorry for the delay.  I've been very busy with a new job and baby and a few other projects.  I will try to look into this as soon as possible.

Hi - I should get this into the next point release - thanks for the report.  Sorry I wanted to get 5.3.0 out the door and I got the other issue in.

Thanks for the update and all the great work you are doing!

You're quite welcome.  I love feedback!  This should be fixed in 5.3.1 which I just published to npm.  Please re-open if there is an issue.  You're in the patch notes.  Thanks!


format configuration file

Fix typo and format code and json related parts.

Thank you for your support. This MR looks good to me


download_pic(): HTTP error code 500. [retrying; skip with ^C]

Describe the bug
HTTP error code 500
To Reproduce
Highlights from Profile
Expected Behavior
There should be download all hightlights without error
Error messages and tracebacks

download_pic(): HTTP error code 500. [retrying; skip with ^C]

This is my command

instaloader --login=USERNAME +args.txt --highlights --igtv PROFILE --no-iphone --quiet

very minimal report. for bug reporting do not use --quiet parameter.
additionally error 500 is server side, not a bug in instaloader.

I use --quiet because highlights title contain emoji.


Change name of Arrays that does not conform to AiiDA naming convention

The names generated by lammps, might result in array names that are not suitable for aiida. Hence one needs to change the names of the arrays so that they can be stored in the database without problem.
This tends to happen when one is trying to store arrays that have [ in the name.

Closed via #43


Fix column tooltip error when side menu is opened

Fixes #626
We could modify the hover handler to know where the button is, but this fix seems easier/more robust to future issues

Nice, this originally had an onLeave and I removed it to handle the case when the popper is above the grid, this is better.


Error with multi-currency

I am trying to create a rollup summary to count the # of child records on the parent.  The org has multi-currency enabled and I'm getting the following error when trying to save the rollup summary record:
Invalid field 'CurrencyIsoCode' for object 'ApexTrigger'
Error is in expression '{!save}' in component apex:commandButton in page dlrs:rollupsummaryenhanced: (dlrs)
An unexpected error has occurred. Your solution provider has been notified. (dlrs
is there a fix to this?

I am have the same issue - is there any way to fix this? Thanks!

Sorry about this, this slipped in as a regression from the last release. I'm doing a new release now!

Fixed in v1.22.

Thank you so much!  The fix worked...you rock!!


Update File.store_var description to mention which properties of an object are included.

This update is in response to discussion at https://github.com/godotengine/godot/issues/49430.

Thanks! And congrats for your first merged Godot contribution :tada:

Cherry-picked for 3.4.

Cherry-picked for 3.3.3.


Integrate KeyboardEvent.key code from upstream blink

Currently, in Atom, there is an issue with non-US keyboard layouts  caused by Blink. The issue, [Some non-US keyboard layouts not working #37] (https://github.com/atom/atom-keymap/issues/37),  along with other similar issues have a common source. KeyboardEvent.keyidentifier is not working correctly, does not support Alt Graph and has the wrong values in some cases.
In example, on a German keyboard the # is read as  \
There is a fix for this in Chrome. A part of it is already in the trunk while the other is still waiting to be reviewed.
 In order to get .key working and solve this issue, you have to follow these steps:

ideally use a new/upstream Blink checkout
apply  issue 929053004
apply    issue<PHONE_NUMBER> .  ( manually, if it doesn't auto apply)

This enables the .key and .code KeyboardEvent members, fixing the layout issues.
Tested on :
Ubuntu 14.04 ( 3.16.0-30-generic # 40~14.04.1-Ubuntu SMP )
Chromium version : Version 46.0.2455.0 (64-bit)
Screenshot:

Please feel free to ask for any other additional information, if needed!
@nathansobo  @zcbenz

Thanks @pandrei. @zcbenz this was the patch I mentioned to you at Summit. I'm not sure about the effort involved in back-porting it to the current version of Electron, but if it's not too painful it would be really great to fix this issue for international users sooner rather than later. Thanks!

ideally use a new/upstream Blink checkout

The blink commit is very tightly bound to the corresponding Chromium commit, there is no way to update the blink version in an old Chromium to the one of a newer Chromium.
If we really want this feature we should back port it into Chrome 43 and submit the patches to libchromiumcontent, you can find examples at https://github.com/atom/libchromiumcontent/tree/atom/patches.

I don't know if backporting is a solution, but KeyboardEvent.key is the solution for all of our shortcut woes, it will make life much, much easier for app developers once it lands

It is in Chrome 48, let's just wait for next Chrome upgrade.

@zcbenz Am I correct in my understanding that Chrome 48 only includes support for KeyboardEvent.code, meaning we'll still have to manually resolve this to keys in JS?

@nathansobo The KeyboardEvent.key is behind a runtime feature flag, we can even enable them in Chrome 47. I'll add options for them.

@zcbenz I talked to Rick Byers a couple months back, who heads up Chrome's input team, and he said there were still some issues resolving the .key property on Windows. I assume that's why it's behind a flag. I could be misinformed though.

Can you enable this and ship it on the beta so at least non windows systems have atom-keymap#37 fixed?

@asantos3 Thanks for the heads-up. We're still on Electron 0.36.8. Once we upgrade to 0.37.x we'll try flipping the flags. I may do some experimentation next week.


stanchion job of riak is crashing

After deploying riak (v6 release) to cf the broker returns the catalog. But while creating a service instance the broker returns with 500 and the message below. I did dig deeper into the issue and curled the riak-cs instance with /cf-riak-cs-service-broker-bindings which returned 503. Then Google lead to an issue with stanchion. I remember that bosh deployment had a timeout but instance was running afterwards. Actually it seems that stanchion dumps and is not running. So, stanchion seems to be the root cause of all issues.
Is this a known issue?
{
description: "Excon::Errors::Forbidden: Expected(200) <=> Actual(403 Forbidden) response => #<Excon::Response:0x007f80d03fc430 @data={:body=>"InvalidAccessKeyIdThe AWS Access Key Id you provided does not exist in our records./cf-riak-cs-service-broker-bindings", :headers=>{"Connection"=>"close", "Content-Length"=>"247", "Content-Type"=>"application/xml", "Date"=>"Fri, 14 Nov 2014 14:36:15 GMT", "Server"=>"Riak CS", "X-Cf-Requestid"=>"0e1ff3e8-81b6-460c-7db2-9249247108b0"}, :status=>403, :remote_ip=>"<IP_ADDRESS>"}, @body="InvalidAccessKeyIdThe AWS Access Key Id you provided does not exist in our records./cf-riak-cs-service-broker-bindings", @headers={"Connection"=>"close", "Content-Length"=>"247", "Content-Type"=>"application/xml", "Date"=>"Fri, 14 Nov 2014 14:36:15 GMT", "Server"=>"Riak CS", "X-Cf-Requestid"=>"0e1ff3e8-81b6-460c-7db2-9249247108b0"}, @status=403, @remote_ip="<IP_ADDRESS>">"
}

Looking at this issue (a bit delayed) and I'm not sure we have enough information to reproduce. Did you get this resolved on your own?
If not, do you have the info from googling that lead you to believe it was stanchion?
We'll probably also need a sanitized version of your deployment manifest if we're going to debug this.
You might also try v7, but I don't know of any stanchion related changes recently.

Closing due to inactivity.
Feel free to create a new ticket if you have new information.


Placeholder doesn't appear in mat-select, select box

Bug, feature request, or proposal:
Bug
What is the expected behavior?
Placeholder to appear
What is the current behavior?
Placeholder doesn't appear
What are the steps to reproduce?
I've just added this code and the placeholder doesn't ppear after the mat prefix update
<mat-select placeholder="Favorite food">
            <mat-option [value]="0">
              TOMATO
            </mat-option>
</mat-select>

What is the use-case or motivation for changing an existing behavior?
rolling back to the old md prefox
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
LATEST
Is there anything else we should know?
No

mat-select must appear inside a mat-form-field since beta12

@benb7760 oh ok, actually that works !, thx man
do you know where can I find this documentation? the website material.angular.io isn't updated
thx again

@JosephSKh A lot of people are having issues with caching - myself included. I've been browsing it in incognito mode

@benb7760 yess, it was caching =S thx man