Missing scope on server side when using OAuthClient from Authlib

When I use any of the standard clients, according to the documentation, I pass the scope key in the client_kwargs argument.
# Example config.py for Flask application
HAMSTER_API_BASE_URL = 'http://api:5000/'
HAMSTER_ACCESS_TOKEN_URL = 'http://api:5000/oauth/token'
HAMSTER_CLIENT_ID = 'CLIENT'
HAMSTER_CLIENT_SECRET = 'SECRET'
HAMSTER_CLIENT_KWARGS = {
    'grant_type': 'client_credentials',
    'scope': 'api'
}

The server side is also implemented using Authlib. When trying to get a token using the ClientCredentialsGrant grant, the scope on the server side is lost. As a result, the token is broken, the server gives it, but without scope.
from authlib.flask.client import OAuth

def fetch_hamster_token():
    name = 'hamster'
    token = OAuth2Token.get(name=name)

    if token:
        delta = token.expires_at - datetime.now().timestamp()

        if delta > 60:
            return token.to_token()

    token = getattr(oauth, name).fetch_access_token(timeout=30)
    save_hamster_token(token)
    return token

oauth = OAuth()
oauth.register('hamster', fetch_token=fetch_hamster_token)

If you use curl, then this problem is not observed.
What am I doing wrong? And sorry for Google translator

I don't understand your question. You can provide a minimal project to reproduce your problem, describe with:

what are you doing?
what result are you expecting?
instead, what your code gives you?

Ok, I'll do it later, please do not close the question yet.

Fixed with https://github.com/lepture/authlib/commit/ade352bc9cb6ce2be332d964bc6c2725c65d6922

currently, you can use fetch_access_token(scope='profile')

BTW, your config:
HAMSTER_CLIENT_KWARGS = {
    'grant_type': 'client_credentials',
}

The config is invalid. It will not read grant_type from CLIENT_KWARGS.


[Feature Request]: Lack of User Testimonials or Social Proo

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
There's no visible social proof (like testimonials or user reviews), which might reduce trust in the effectiveness of the plans.
Use Case
this feature will help the app to standout from the crowd and also show the benefits and success stories of purchasing the product
Benefits
Add testimonials, reviews, or success stories to build credibility and encourage conversions.
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC-EXT contributor
[ ] I'm a HACKTOBERFEST contributor
[X] I have starred the repository

@ROHAN7051 updates?

i am working on it i will ping u as soon as i done with it sorry for delay ;)

@jeevan10017  i am done with the feature

@jeevan10017 i am done with the feature

Raise PR


Remove lazy hashCode and hashCode calculation on initialization

This will reduce the memory footprint (slightly?) and reduce the initialization time on object construction.
Especially collections are rarely used as HashMap/HashSet keys. I think it will be more balanced to remove lazy hashCode instance variables of collection nodes.
Caution: Take a look at the equals methods. There the laziness of hashCode was used sometimes. Hashcode should not be used in equals impls.

It seems HashArrayMappedTrie hashCode and equals are not symmetric. I noticed it when simplifying the equals/hashCode impls of collections.
It makes sense to define that two HAMT's are equal, if their structure and their contained values are equal. That means that I will update the equals method of HAMT and take care that no collection relies on comparing their backends by HAMT.equals.
The same applies for RedBlackTrees (needs to be checked).

HAMT equals and hashCode should be expressed independent of the order of elements. In other words, it does not matter how HAMT(and RedBlackTree) structures store values. Only the values matter.

HAMT hashCode should be expressed by applying XOR to the hashCode of all entries. XOR is commutative and equally distributed (50% 0, 50% 1). See https://stackoverflow.com/questions/5889238/why-is-xor-the-default-way-to-combine-hashes
HAMT equals should be expressed by checking key sets for equality and then check if values are congruent. size()-check may be also used for optimization, it is O(1)

This strategy allows us to delegate equals and hashCode to an underlying HAMT backend

Fixed


Gradle fails with cryptic error when plugins {} contains unexpected code

Can't run JUnit tests with GradleRunner (tested on 3.5 and 4.10.2)
Minimal test-case from gradle-node-plugin: https://gist.github.com/deepy/b53c1bcc1c4f92c21ac6f182db413b71
(also happening on travis)
https://travis-ci.org/node-gradle/gradle-node-plugin/jobs/462819188#L656
    A problem occurred configuring root project 'junit2647550049405801039'.
    > Could not open cp_proj remapped class cache for aymfmvqz7m4rx0xrpgrjhp46f (C:\Users\deepy\AppData\Local\Temp\.gradle-test-kit-deepy\caches\3.5\scripts-remapped\build_3hr5fzwun61jnjy0zqxoam0ig\aymfmvqz7m4rx0xrpgrjhp46f\cp_proj0341ca279e16587311ec2a53cd62afc0).
       > Could not open cp_proj generic class cache for build file 'C:\Users\deepy\AppData\Local\Temp\junit2647550049405801039\build.gradle' (C:\Users\deepy\AppData\Local\Temp\.gradle-test-kit-deepy\caches\3.5\scripts\aymfmvqz7m4rx0xrpgrjhp46f\cp_proj\cp_proj0341ca279e16587311ec2a53cd62afc0).
          > BUG! exception in phase 'conversion' in source unit '_BuildScript_' org.codehaus.groovy.ast.expr.TupleExpression cannot be cast to org.codehaus.groovy.ast.expr.ArgumentListExpression

Steps to Reproduce (for bugs)
Run any of the _integTest classes (e.g. Node_integTest.groovy)
./gradlew :test --tests "com.moowork.gradle.node.task.Node_integTest.exec simple node program"

Your Environment
https://github.com/node-gradle/gradle-node-plugin
https://travis-ci.org/node-gradle/gradle-node-plugin/jobs/462819188#L656

@melix Do you know what the BUG! here from Groovy is about?

This is not a bug in the Groovy compiler, but rather a bug in the Gradle AST transformation that extracts plugin usages from the plugins block. The underlying error is:
Caused by: java.lang.ClassCastException: org.codehaus.groovy.ast.expr.TupleExpression cannot be cast to org.codehaus.groovy.ast.expr.ArgumentListExpression
        at org.gradle.groovy.scripts.internal.AstUtils.hasSingleConstantArgOfType(AstUtils.java:221)
        at org.gradle.plugin.use.internal.PluginUseScriptBlockMetadataExtractor$1.visitMethodCallExpression(PluginUseScriptBlockMetadataExtractor.java:118)
        at org.codehaus.groovy.ast.expr.MethodCallExpression.visit(MethodCallExpression.java:70)
        at org.gradle.plugin.use.internal.PluginUseScriptBlockMetadataExtractor$1.visitExpressionStatement(PluginUseScriptBlockMetadataExtractor.java:155)
        at org.codehaus.groovy.ast.stmt.ExpressionStatement.visit(ExpressionStatement.java:42)
        at org.gradle.plugin.use.internal.PluginUseScriptBlockMetadataExtractor$1.visitBlockStatement(PluginUseScriptBlockMetadataExtractor.java:63)
        at org.codehaus.groovy.ast.stmt.BlockStatement.visit(BlockStatement.java:71)
        at org.gradle.plugin.use.internal.PluginUseScriptBlockMetadataExtractor.extract(PluginUseScriptBlockMetadataExtractor.java:58)
        at org.gradle.groovy.scripts.internal.InitialPassStatementTransformer.transform(InitialPassStatementTransformer.java:73)
        at org.gradle.groovy.scripts.internal.AstUtils.filterAndTransformStatements(AstUtils.java:90)
        at org.gradle.groovy.scripts.internal.SubsetScriptTransformer.call(SubsetScriptTransformer.java:51)
        at org.codehaus.groovy.control.CompilationUnit.applyToSourceUnits(CompilationUnit.java:976)

This is caused by unexpected code in the plugins block:
plugins {
   apply plugin: 'com.github.node-gradle.node'
}

Which should be:
plugins {
   id "com.github.node-gradle.node" version "1.2.0"
}

Thanks @melix.  I've renamed the issue to better reflect what's going on here. We could make the error a bit better.
@deepy the "fix" for you is to use the syntax that Cedric suggests above.

@melix Thanx for the solution

@melix Thanx for the solution

@melix thanks man, i have been searching for this issue for the last six hours and yours has helped me finally.
Thanks alot

@melix thanks man, i have been searching for this issue for the last six hours and yours has helped me finally.
Thanks alot

@wolfs can you please reopen this issue?
It still strucks people, for example at https://discuss.gradle.org/t/installing-error/45452.
And I'm also about to send a pull request to fix it. :-)

Thank you Björn for your contribution!

https://github.com/gradle/gradle/pull/24937

This has been merged and will ship in 8.3 RC1.


Changed the order of the argumets for output to appear last

Hi,
The output arguments are defaulted to appear before the global arguments - This could cause issues with ffmepg command line as it expects all the stream arguments to be between the -i and -f.
See this example:
This would work: ffmpeg -i ~/Movies/sample.ts -map data -c copy -f data -
But this would return a "Stream not found" error: ffmpeg -i ~/Movies/oparea4.ts -f data - -map data -c copy
This PR replaces the order so that output arguments would appear last

This needs unit-test changes. CI is failing.


Add real "Last Lecture" date to courses under "My Courses" and rename "Last Lecture" to "Next lecture"

Is your feature request related to a problem? Please describe.
When logging into the website, the section "My Courses" always displays "Next lecture" under the courses, but with the date of the last lecture.
Describe the solution you'd like
It would be nice to keep the "Next lecture" line, but display the actual date of the next lecture planned in the system. Also, adding a line showing the "Last Lecture" information already present.
Describe alternatives you've considered
Alternatively, the "Most recent lecture" link could be integrated into "Last Lecture".
Additionally, a monospace font could be used to align "Next Lecture" and "Last lecture" vertically and increase readability (fixed 2 characters for day and month would also be great : ))
Additional context
Example picture of the desired functionality:

Wow thanks for the issue, I actually have never noticed that the "next lecture" is displayed with incorrect date.
Of these two I prefer the former one. Or, not adding the date of the last lecture. I as a user would like this list to be concise. (Just personal thought.)

Describe alternatives you've considered
Alternatively, the "Most recent lecture" link could be integrated into "Last Lecture".
Additionally, a monospace font could be used to align "Next Lecture" and "Last lecture" vertically and increase readability (fixed 2 characters for day and month would also be great : ))


Update password with wrong current password gives an incorrect error message

Describe the bug
I'm trying to update the password using Amplify iOS library.
As we know, the method accepts a current password and a new password. When we try to send an incorrect current password it gives us an error message "Incorrect username or password".
We are unable to override an error message as we are getting the exception type as nil in this scenario. Please find the error message below.
▿ notAuthorized : 3 elements
- .0 : "Incorrect username or password."
- .1 : "Check whether the given values are correct and the user is authorized to perform the operation."
- .2 : nil
Steps To Reproduce
Try to provide an incorrect current password.

Expected behavior
Error message to be relevant to the use case or Exception type should not be nil.
Amplify Framework Version
1.9.1
Amplify Categories
Auth
Dependency manager
Swift PM
Swift version
5.0
CLI version
4.50.0
Xcode version
12.4
Relevant log output
No response
Is this a regression? (i.e. was this working before a version upgrade)
No response
Device
iPhone 11
iOS Version
iOS 14.4
Specific to simulators
No response
Additional context
No response

@lawmicha @royjit Can we get an update on this?

The error message is the one returned by Cognito User Pool service and Amplify is passing the same error back in the api call. I tried to reproduce the issue and the error I got from server is:
{
    "Content-Length" = 79;
    "Content-Type" = "application/x-amz-json-1.1";
    Date = "Fri, 10 Sep 2021 03:56:38 GMT";
    "x-amzn-errormessage" = "Incorrect username or password.";
    "x-amzn-errortype" = "NotAuthorizedException:";
    "x-amzn-requestid" = "2axxx";
}

I do not think there is a fix for this since we are passing on the service error here.


[Bug]: 404 error at URL: /txns/105067447252

What happened?
A bug happened!
URL
https://explorer.helium.com/accounts/13cEmaogbLM6zdwpd2J93JnKjMADh5KzDU9W7YppCJcujNAP7uQ
What browsers are you seeing the problem on?
Chrome
Do you see the issue on Mobile or Desktop?
No response
Relevant log output
No response

that is not a valid transaction on the helium blockchain. please double check.


When a protocol is cancelled from a pause, the robot makes a sudden movement

overview
When a protocol is cancelled from a pause, the robot suddenly starts moving towards its next command, but then stops and pauses.

current behavior
When a protocol is cancelled from a pause, the robot starts performing its next command, but stops after a fraction of a second.  This behavior is counterintuitive and could be dangerous, as a user might pause a protocol and then cancel it if something goes wrong.
Having the robot suddenly move on a cancel would be unexpected.
It appears this issue is connected with the fix done in #1443.

steps to reproduce

Upload and run the following protocol:

from opentrons import robot, labware, instruments

source_rack = labware.load('96-PCR-flat', '5')
dest_rack = labware.load('96-PCR-flat', '2')

tiprack = labware.load('opentrons-tiprack-300ul', '4')

pipette_left = instruments.P300_Single(mount ='right', tip_racks=[tiprack])

pipette_left.transfer(25, source_rack.wells('A2'), dest_rack.wells('A2'))
robot.pause()
pipette_left.transfer(25, source_rack.wells('A3'), dest_rack.wells('A3'))

When the robot pauses between the 2 transfers, cancel the protocol.
The robot will suddenly start moving in the direction of its next command.

expected behavior

When a protocol is cancelled (whether from a pause or not), all movement of the robot should stop immediately (or remain stopped if the robot was already paused).
┆Issue is synchronized with this Wrike Task by Unito

Thanks for the report @Cin316. You're right, this is a known behavior caused by #1443. That change was introduced due to fundamental architecture problems with the protocol run API. Our options were basically:

Protocol cancel doesn't always work if paused
Protocol cancel works if paused, but we need to allow a "resume" first that may move the robot

This behavior will remain in place until we are able to rewrite this part of the robot software. This project is in its planning stages, and I'll be sure to include / update this ticket in those plans

I'm pretty sure that ExecutionManager resolved this issue.

I'm pretty sure that ExecutionManager resolved this issue.


Highlight label should not create extra span nodes

I noticed that the HighlightedLabel class creates extra span elements for text ranges. These should not be needed. Using text children directly should be faster for creation and also reduce the number of nodes in the document
I also replaced the conditional spread with a longer version that uses a simple call to push. This is worth doing since HighlightedLabel is so widely used in the editor

Looks good to me, but @jrieken probably knows more why the span was added initially


Checkbox within a cell not being updated when paginating

Thanks for an amazing library! Am I missing something with my current issue?
Describe the bug (required)
When using pagination from server side, checkbox within a cell is not being populated correctly despite current "true" / "false" are being retrieved successfully
Provide an example via Codesandbox! (required)
I forked current example of server side pagination
https://codesandbox.io/s/pagination-error-checkbox-nzf8p?file=/src/App.js
Steps To Reproduce (required)
Steps to reproduce the behavior:

Go to codesandbox
Page 1, whatever row, all are marked to true in the last column, uncheck it
Go to second page
checkbox for page 2 in the same row position remains the same state, instead of being rerendered with its current true/false state

Expected behavior (Recommended)
I think that when moving along pages, all checkboxes should be populated with its own specific data, as does text inputs
Screenshots
Think the codesandbox is enought, no console errors
Desktop (please complete the following information):

OS: [Mac last version]
Browser [Chrome]
Version [last]

Also asked in this discussion, but not sure if its the same problem

To add further info, I dont want to use the select row feature that is shown in the examples, I dont want to select multiple rows or anything similar.
I just want to show a "read/unread" state for a certaint type of notifications, using a input checkbox, dont know if this is any help or can be any other way.
Funny stuff: The checkbox when debugging shows in html as "selected" but is not being rendered like that

Hi @sauldeleon,
You want the checkbox input to be a controlled component rather than an uncontrolled component.
If you change the defaultChecked prop here:
{
  Header: "Will assist",
  accessor: "assist",
  Cell: ({ value }) => (
    <input type="checkbox" defaultChecked={value} />
  )
}

... to the checked prop:
{
  Header: "Will assist",
  accessor: "assist",
  Cell: ({ value }) => (
    <input type="checkbox" checked={value} />
  )
}

... then your checkboxes will always reflect the current value that is passed to them.
Hope that helps :)

Hi @BenJenkinson
Thanks for your response. I have updated the codesandbox with your last advice which I think is good.
I created the CheckboxCell function, which is a single checkbox with its own state for handling the onChange event, as there is a warning in React stating that if there is a checked property without the onChange handler, it will become a readonly component, so when I click on it, it does not change without the onChange.
Said so, the same problem occurs, so I obviously Im missing something important :(. The codesandbox shows the same problem with the controlled component. I can change it correctly, but after paginating, it shows the same state as the previous page
In my sceneario, clicking in the checkbox triggers an api call to switch that value in backend, so when paginating im retrieving the updated state of the checkbox (true or false), but this does not matter as the checkbox shows the same state as the previous page, it does not get updated.
I have updated the codesandbox with a update function that changes the assist value in the original Data set, and put a series of console logs to play with first and second page with the first value.
As you can see, when changing the state of the checkbox, data is updated correctly, but the checkbox still remains the same.
Here is the codesandbox link again: link
I think Im missing something really stupid here! but the only way I can make this work is making the "loading" property to re-render the whole table, despite the values are correct :(
Many thanks again for your help

Well, I think we can close this.
I have finally read again React docs about what @BenJenkinson said in his first comment, and what im doing know is refetching table info when updating the checkbox, that specific page.
It seems that you have to keep external controll of everything to get this work, nothing related with react-table.
Thanks!


Cannot fetch the values of the User while creating multiple users

I am creating the 10 users using this IAM-user module, but the problem is the output.tf file is not designed to provide the user's related values for multiple user. The only option is to define the output values for every user separately which is not viable as no. of users can be unlimited.

This module is not suitable for what you want to do. You can use a suitable data-source to get information about existing user(s). See this, for eg, https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/iam_users


PurpleExplorer only show the first 100 queues

I'm connected to a SB with more than 100 queues and the app only shows the first 100.

This is interesting, I was aware there is a default setting for that.
I'm currently working on a settings page that we can configure this and some other options for the application.


Enhancement: remove approval embeds if progger removes signup

Is your feature request related to a problem? Please describe.
if a progger removes their signup via /remove-signup the corresponding approval embeds in the channel should also be deleted. Reacting to them would cause an error that no corresponding signup could be found
Describe the solution you'd like
Check for any existing messages for that progger and remove them. This relates to #264 and that should be done first.

Resolved


Adding more bots (and ignoring some bundler stuff)

Looked through the user agent for many days of high traffic for my site (mysteryscience.com) scanning for the word "bot." Added my findings to bots.yml.

bump!

@fnando: Would love to have this merged in if you've got the time.

bump :+1:

Merged in the latest and resolved conflicts so this is good to go! Would love a merge. Thanks!

@avand Maybe @fnando will merge this in sooner if you update the bots test like this for each new bot - https://github.com/fnando/browser/commit/901a7087f51d0449a3a18a30b013d4d44a23f20f
@fnando Any input on this branch?

Will do! Out of curiosity, what's the value of these test? Haven't dug deep here but looks like we're just duplicating the user agent in the test and that the tests are just a confirmation of what's in the bots.yml.

@avand Yea, the tests appear to just be confirming that the user agents in bots.yml are marked as bots. So I guess they're just to ensure that things are working expected. I'm not sure if @fnando is requiring new bots to have these tests, but I noticed the last 3 pull requests he merged all updated these tests for the bots they added. So I figured it's worth a shot. ¯_(ツ)_/¯

Ah, that makes good sense. So bots.yml is more of the pattern to match on and ua.yml is the full user agent. That's actually a problem for me because I didn't save the full user agents of those bots. I'll see what I can find.

I started Googling for user agents for each of these bots and got through about 5 of them and realized that this was incredibly tedious and I'm not sure it really provides much value. Notably, most of the bots in bots.yml are not tested. So I'm inclined to wait until @fnando at least chimes in before doing all this copy/paste work.

@avand Good call, and thanks for the speedy response!

This is pretty useful and contains a lot of bots that we're seeing as well. I was just about to open a PR for detecting some of the bots listed here (e.g. UptimeRobo), so it'd be great to see this merged soon.

These tests aren't that important; is more of a sanity check for the most important robots like Google Bot. You don't have to add other robots to the list.
Since you're just updating the bots list, there's no problem in merging this… which I just did.

Thanks @fnando! Any idea when you'll cut a new version of the gem with these changes? I'd love to point our Gemfile back.


Add Loader in the WebSite.

Is this a unique feature?

[X] I have checked "open" AND "closed" issues and this is not a duplicate

Is your feature request related to a problem/unavailable functionality? Please describe.
currently the website directly open and without attract the user too much.
Proposed Solution
increase the user interest toward the website we implement the loader, which gives best aesthetic looks and also looks like user-friendly and modern website
Screenshots
screen-capture (13).webm
Do you want to work on this issue?
Yes
If "yes" to above, please explain how you would technically implement this (issue will not be assigned if this is skipped)
firslty i make the jsx file for the Loader and give its styling which cover the whole screen and after some seconds the loader will be invisible and the main page of website will be shown,this can be done with help of useState() and conditional rendering & setTimeOut() & useEffect()

sorry @AnkitLuhar  upto till now my system went down now it is fixed so that's why late reply you can do your work @AnkitLuhar


v1 release plan

Question
Do we have a plan of v1 release (todo before v1), or date of release?

Yeah, @pedronauck if we could get a roadmap we could potentially help out! :)

Now what's missing it's just few bugs I think, I'm working hard to release until next Monday 🙏


Add MacOSTCC

Adds an artifact definition for Mojave's Transparency, Consent, and Control framework logs.
References:

https://blog.fleetsmith.com/tcc-a-quick-primer/
https://carlashley.com/2018/09/06/reading-tcc-logs-in-macos/

Also add /Library/Application Support/com.apple.TCC/MDMOverrides.plist to fetch MDM overrides

@jipegit that sounds like that should be a separate artifact, could send a PR for that?


How to fit the extracted face features in a full head mesh?

Hi, Thanks for sharing this awesome work. I wanted to make a full face 3D model, in which I can later fit some hair features. Also can this output mesh be attached to a separate 3D mesh?
Thanks in advance!.

FaceProfilling does it by fitting anchor points on face mesh to full head, you may give it a try.

Great, Thanks for the reply. I found a link for the face profiling matlab code in the repository, but its unavailable. Suggest any alternate sources even if available in python?

As far as I know, there is no public python repo. I re-upload the Matlab version here: FaceProfiling.

Thanks for sharing. I will start proceeding with this.

Hi, I dont have much knowledge on Matlab and just started with it. Can the matlab based face profiling integrated along with the existing project? The end output must be the merging of the frontal face mesh from 3DDFA on the full head mesh. Please help me to solve this. Thanks

3DDFA only outputs the 3DMM parameters of  face mesh. If combined with the head mesh, you should fit it or use a full-head face model.

Hi, I dont have much knowledge on Matlab and just started with it. Can the matlab based face profiling integrated along with the existing project? The end output must be the merging of the frontal face mesh from 3DDFA on the full head mesh. Please help me to solve this. Thanks

you can refer to this link, this script can fit bfm face model to flame head model. I think this is what you want.


Platform settlement expense has strange figures

https://opencollective.com/ocnz/expenses/39597

Why is there a line item for something that's $0? Just don't show it.
Why is the total showing as -- instead of $214.96?

There's something weird going on with the attached CSV as well. What are all these $0 transactions that say they are contributions to Open Collective?

@kewitz could you look in the things related "Reimburse: Payment Processor Fee for Platform Tips", you're the most suited for this.


⚠ Bofrost.fr has degraded performance

In 10942fb, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1911 ms

Resolved: Bofrost.fr performance has improved in 6eca351 after 9 minutes.


Fix table formatting in README.md

Fix table formatting

Yuck, has that been broken the whole time? Maybe GitHub got more strict with their table parsing? Either way, thanks for the fix!
I'll merge once TravisCI passes. Thanks for the CLA.


idea2019.3 not install

I'm submitting a ...

[ ] bug report
[ ] feature request
[ ] puppy => You're not submitting a puppy. I already have one and he's adorable

What is the current behavior?

Is this a bug? Sorry about that. If so give me explicit details how to reproduce:

Open IntelliJ
???
Profit

What is the expected behavior?

What is the motivation / use case for changing the behavior?

Please tell me about your environment:

Gitflow4idea version: 0.7

Gitflow version: 1.12.3

IntelliJ Help -> about > click copy icon and paste here. Should look like this:

IntelliJ IDEA 2019.3 (Ultimate Edition)
Build #IU-193.5233.57, built on November 20, 2019
Licensed to Vladislav Kovalenko
Subscription is active until January 9, 2020
Runtime version: 11.0.4+10-b520.11 x86_64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
macOS 10.14.6
GC: ParNew, ConcurrentMarkSweep
Memory: 4029M
Cores: 8
Registry: compiler.automake.allow.when.app.running=true
Non-Bundled Plugins: GrepConsole, JProfiler, JRebelPlugin, Lombook Plugin, MavenRunHelper, SequenceDiagram, String Manipulation, cn.yiiguxing.plugin.translate, com.alibaba.p3c.smartfox, com.alibabacloud.intellij.toolkit-intellij, com.ccnode.codegenerator.MyBatisCodeHelperPro, com.dubreuia, com.github.novotnyr.gitlab-quickmr-idea-plugin, com.ppolivka.gitlabprojects, com.sjhy.plugin.easycode, de.netnexus.camelcaseplugin, gr.jchrist.gitextender, zielu.gittoolbox

* **Other information** (e.g. detailed explanation, stacktrace, related issues, suggestions how to fix, links for me to have context words of praises, pictures of puppies (again with the puppy??) )

Yeah, doesn't work with 2019.3

Have a look at an earlier post: https://github.com/OpherV/gitflow4idea/issues/241#issuecomment-515444744
This explains what you can do until the new release is out.

@roland-d latest release is 7.0, that doesn't support 2019.3 yet, so we have to manage it from cli.

@webcoderkz I have 0.7.0 (this is the latest release) running just fine on 2019.3

@webcoderkz I have 0.7.0 (this is the latest release) running just fine on 2019.3

doesn't work for me (

Have a look at an earlier post: #241 (comment)
This explains what you can do until the new release is out.

Thanks. It's working!

Hello,Thank you very much for your project. do you know whether the project will continue to maintain and support idea2019.3?

New version will be in the jetbrains repo soon

thank you @opher for your work and support :-)

+1

Thx @OpherV for your support and reactivity ;)

@OpherV thank you for your support. Is the version released?

@halilsafakkilic not yet, still ironing out a few issues (for example #255, #232, #249) once I'm done and properly test it will be released

Why can't the plugin be found on the marketplace?

Why can't the plugin be found on the marketplace?

Click on Installed. Maybe this plugin will be there and has been installed earlier.

@RamallahS There is no plugin on both tabs!

@khusamov It does not appear because it's incompatible with the current IDE version. You'll have to wait for the next plugin release

Version 0.7.1 now in the repo

Thank you for your work! This plugin is one of the most important for me!


Ability to upload request body as iOS Upload Task

Adds the ability to specify IOSUploadTask as a config parameter to RNFetchBlobConfig. Specifying this parameter will create an NSURLSessionUploadTask with the request's body. If the IOSBackgroundTask flag is also set, the body is first written to a temporary file. See commit log for more details.

FYI
react-native-fetch-blob has a new maintained location. Check out the README.
Please consider submitting your PR there, this repository is unmaintained.


Add a containsInOrder for collections

Feature summary
Adding a way to test if some elements in a ordered collection like an ArrayList are present in a particular order.
Example
assertThat(Arrays.asList("one", "two", "three", "four", "five")).containsInOrder("one", "three", "four");

Oh, I just finally found that it is under the notion of "sequence" in AssertJ with containsSubsequence().


Patch react-native-math-view to resolve inconsistent inline latex height between android and ios

Summary

Unlike iOS, height of MathView Component is set on Android.
Ticket Link

https://github.com/mattermost/mattermost-mobile/pull/7724#issuecomment-1877842151
Checklist

[ ] Added or updated unit tests (required for all new features)
[x] Has UI changes
[ ] Includes text changes and localization file updates
[ ] Have tested against the 5 core themes to ensure consistency between them.

Device Information
This PR was tested on:
Screenshots

Android (Before)
Android (After)

Example message text:
hello $\frac{\sqrt{x^n}}{x^2}$ world world world world world world world world world world world world world world $\frac{\sqrt{x^n}}{x^2}$  world world world world world world $test$

Text with formula $\hbar \omega$ inline

$\sum_{n=0}^{2N-1} x_n \cos \left[\frac{\pi}{N} \left(n+\frac{1}{2}+\frac{N}{2}\right) \left(k+\frac{1}{2}\right) \right]$

test $test$

world $\frac{\sqrt{x^n}}{x^2}$

Release Note

NONE

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

/update-branch

/update-branch

/update-branch

/update-branch


Not compatible with 2.3.5

Hi,
Latest version 0.3.1 of DBAL doesn't seem to be compatible with latest Crate.io 2.3.5. The reason being:
CrateSchemaManager::listTableColumns is broken as getListTableConstraintsSQL returns empty string because this line should read:
"WHERE table_name = '$t[1]' AND table_schema = '$t[0]' AND constraint_type = 'PRIMARY KEY'";
note: PRIMARY_KEY -> PRIMARY KEY, however even if we fix that, result of the above query differs between 2.0.4 (where DBAL works OK) and 2.3.5.
v. 2.0.4:
{"cols":["constraint_name","constraint_type"],"col_types":[[100,4],4],"rows":[[["id"],"PRIMARY_KEY"]],"rowcount":1,"duration":0.911611}
note: constraint_name returns array of PK column names
v. 2.3.5:
{"cols":["constraint_name","constraint_type"],"col_types":[4,4],"rows":[["cdr_pk","PRIMARY KEY"]],"rowcount":1,"duration":0.915925}
note: constraint_name returns string "table_name_pk" ?
Regards,
Chris

hi @l7s,
we are currently overhauling the crate-pdo. there will be a new release this week.
will keep you up to date.
thanks,
joe


close() of read only workbook doesn't allow me to delete the file

When I run this code:
import openpyxl
import os
import shutil
import tempfile

old_path = "C:\\Users\\x\\spectral_work\\2019-GRESB-Asset-Spreadsheet.xlsx"
excel_path = "C:\\Users\\x\\spectral_work\\my_book.xlsx"

try:
    shutil.copy2(old_path, excel_path)
    gresb_book = openpyxl.load_workbook(excel_path, read_only=True)

    building_characteristics_sheet = gresb_book.worksheets[3]
    row = 8
    while building_characteristics_sheet.cell(row, 3).value is not None:
        row += 1

    gresb_book.close()

finally:
    os.remove(excel_path)

I get this error:
PermissionError: [WinError 32] The process cannot access the file because it is being used by another process: 'C:\\Users\\x\\spectral_work\\my_book.xlsx'

Do you have any ideas of how I could fix this?
If I remove the read_only=True argument and the .close() statement it works. Furthermore if I remove the for loop it runs fine without errors. However after accessing the workbook it doesn't allow me to delete the copy.
Cheers,
Dylan.

@dylanmorroll As a heads up, Openpyxl development was moved here.


Replace ray[default] with ray[tune]

Motivation
To install dependencies for Ray Tune, we need to use pip install -U 'ray[tune]' script instead of pip install 'ray[default]'. Refer to this docs for details.
Modification
Please briefly describe what modification is made in this PR.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects, like MMDet or MMCls.
The documentation has been modified accordingly, like docstring or example tutorials.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@10965e3). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #112   +/-   ##
=======================================
  Coverage        ?   72.79%
=======================================
  Files           ?       59
  Lines           ?     1632
  Branches        ?      235
=======================================
  Hits            ?     1188
  Misses          ?      345
  Partials        ?       99

Flag
Coverage Δ

unittests
72.79% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


YouTube searching not working

I'm trying to get this music bot set up and for some reason when I type "!play" and then put something I want to search for, it just says "Unable to queue up song at _____ to be played."
In the command prompt is says there's an error that ____ is not a valid URL. Set --default-search "ytsearch". I'm especially concerned because my friend told me that my options file is not the same as his. Anyone know what I need to do to fix it?

You're using the develop version, which doesn't have that feature.  Use RC5, under Releases.

Thanks!


Merge branch View to mainline

All the code changes have already been reviewed. I'll delete the view branch after this PR is merged.

Codecov Report

Merging #508 into master will increase coverage by 0.87%.
The diff coverage is 99.54%.

@@             Coverage Diff              @@
##             master     #508      +/-   ##
============================================
+ Coverage     90.51%   91.38%   +0.87%
- Complexity      614      647      +33
============================================
  Files           117      118       +1
  Lines          2277     2519     +242
  Branches        210      220      +10
============================================
+ Hits           2061     2302     +241
  Misses          160      160
- Partials         56       57       +1

Impacted Files
Coverage Δ
Complexity Δ

...ain/java/io/opencensus/stats/BucketBoundaries.java
100% <ø> (ø)
5 <0> (ø)
:arrow_down:

...src/main/java/io/opencensus/stats/Aggregation.java
100% <ø> (ø)
1 <0> (ø)
:arrow_down:

...re/src/main/java/io/opencensus/stats/ViewData.java
100% <100%> (ø)
3 <3> (+2)
:arrow_up:

...ain/java/io/opencensus/stats/MeasureToViewMap.java
91.11% <100%> (+0.37%)
12 <0> (ø)
:arrow_down:

...rc/main/java/io/opencensus/stats/StatsManager.java
100% <100%> (ø)
6 <2> (ø)
:arrow_down:

...ain/java/io/opencensus/stats/RpcViewConstants.java
100% <100%> (ø)
2 <0> (ø)
:arrow_down:

core/src/main/java/io/opencensus/stats/View.java
100% <100%> (ø)
4 <4> (+3)
:arrow_up:

...n/java/io/opencensus/stats/MutableAggregation.java
100% <100%> (ø)
1 <1> (?)

...main/java/io/opencensus/stats/MutableViewData.java
97.87% <97.84%> (+9.94%)
17 <16> (+15)
:arrow_up:

...src/main/java/io/opencensus/internal/Provider.java
58.33% <0%> (-21.67%)
3% <0%> (+2%)

... and 6 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 199be03...a6e1c7d. Read the comment docs.


Parent "x" button to delete

The x button will not work but deleting the permission node for the parent rule itself will work as an alternative.

Is that happening for players or for groups?

Both players and groups.

Ah yeah I changed some stuff recently with the parent groups and forgot
about the delete button, thanks!
On Sun, 17 May 2020, 7:15 am Zane<EMAIL_ADDRESS>wrote:

Both players and groups.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/lucko/LuckPermsWeb/issues/237#issuecomment-629706573,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACSYFB2LFQYKLQDFGCNFBBDRR3657ANCNFSM4NDCMBDA
.

Fixed in https://github.com/lucko/LuckPermsWeb/commit/a1d4e5b89e60bfe506feb3603b950c54bb0d742a


control-C does not exit app on iOS device from flutter run

Application running.
Type "h" or F1 for help and "q", F10, or ctrl-c to quit.
Type "w" to print the widget hierarchy of the app, and "t" for the render tree.
^CApplication finished.

Seems we should either fix the instructions or fix the behavior.  I don't really care which. :)

Instructions no longer include ctrl-c to quit.

Why doesn't ^C exit the process on Mac?

Just tested on Android and as you note, even though we don't make the promise anymore, there's a definite difference in behaviour. Re-opening.
Here's what appears to be going on. When the terminal receives ctrl-c, it sends SIGINT to the processgroup. On iOS, one of the child processes of flutter tool is iproxy, which manages all the port forwarding we do -- in this case, that includes the observatory port forwarding. iproxy gets SIGINT and dies, which kills our observatory connection.
The stopApp call that we do in the SIGINT handler gets called and tries to issue an ext.flutter.exit call to the VM service to shutdown the running Flutter app on the device... but the VM service connection is dead so it never gets the shutdown call.
As a test, instead of starting iproxy directly, I patched the tool to start it through a wrapper script that traps and ignores SIGINT and everything ran nicely as expected.
Why doesn't this happen for Android? adb doesn't stay running as a child process, it starts its own daemon off to the side.

It's also possible to catch the SIGINT from the dart script and have some sort of orderly shutdown I would imagine?  In any case, whatever you decide is fine. :)  I recall we had to catch SIGINT in Webkit's python testing script (as KeyboardInterrupt) to do a bunch of orderly shutdown stuff.

We do catch it in the Dart script, but for some reason it also hits iproxy.

Yep; that appears to be the case. I'll play around a bit with it and do some digging over the weekend to see what we can do. I've got a workaround, but it's a hack and there's almost certainly a simple, proper way to do this.


Errors when launching pyportify

Hi!
After finally being able to install everything without errors i get an error when launching pyportify. I get a similar error when trying to launch pyportify-copyall aswell. This is the error message from simply launching "pyportify":
Traceback (most recent call last):
File "/usr/local/bin/pyportify", line 5, in
from pkg_resources import load_entry_point
File "/Library/Python/2.7/site-packages/distribute-0.6.14-py2.7.egg/pkg_resources.py", line 2671, in
working_set.require(requires)
File "/Library/Python/2.7/site-packages/distribute-0.6.14-py2.7.egg/pkg_resources.py", line 654, in require
needed = self.resolve(parse_requirements(requirements))
File "/Library/Python/2.7/site-packages/distribute-0.6.14-py2.7.egg/pkg_resources.py", line 552, in resolve
raise DistributionNotFound(req)
pkg_resources.DistributionNotFound: six>=1.6.1
Mac OSX 10.10.3
Help is greatly appreciated!

Try installing the brew python distribution
brew install python

See https://github.com/rckclmbr/pyportify/issues/20 and try that

Thanks for responding!
I already installed python through brew, though I had a previous installation of Python that I removed (and reinstalled brew verison). Now I get another error:
$ pyportify
-bash: /usr/local/bin/pyportify: /usr/bin/python: bad interpreter: No such file or directory
Sorry if I'm asking blatant questions but I'm not used to doing these kind of things. I only have basic Unix/terminal knowledge. Installed both brew and pip for the first time when following your install instructions.

I solved it using this command:
sudo ln -s /usr/local/bin/python /usr/bin/python
Thanks for a great program! Now I can finally give google music a fair try!


[Question] Does this work for a server I've been banned from?

I used to be in a really toxic Discord server that banned me for asking them to be more respectful towards me. I decided I don't want any of my messages in that server any more and was hoping I could use this to delete them all. However, it's giving me a 403 access error when I fill in the details (I got the server/channel IDs from Discord's data download option). Is there anything that can be done?

You can't delete a message unless you know the guildId, the only exception is for DMs which doesn't have a Guild.
About the servers you are banned I don't know, I'd assume you will get a non authorized error, but you can test this by creating a server, grab the ids, leave it and click start

I have the same problem, helped a server for months and they ditched and all the info I had in the server is still there, I would love to delete it.

Sadly I don't think it can be done due to authorisation. I retrieved the guildID, as well as each separate channelID, by downloading my Discord data but it responses with an error message, I think because when you are banned from a server you lose authorisation to do anything in that server. But you can still try it with your server if you go to Discord settings and choose to download your data, which will allow you to retrieve the IDs.

I’m not an expert on the subject, but I think if you terminate your account you can request to have all your data wiped, and I think discord is compliant with the new laws regarding data protection and the right to be forbidden. If you really want that I think it worth trying to get an answer from discord support. Let us know if you they get back to you

Thanks I will try that I've been contacting Discord about this too. Next time they reply I'm gonna ask if terminating my account can wipe all my info. However in some servers I've seen messages that say 'Deleted User'.

AFAIK deleting your Discord data with the big shiny delete account button doesn't delete your messages with others or in guilds -- it just changes your username to "Deleted user XXXXXXX" and removes some data such as your email and phone number, profile picture, connected accounts etc. Their help pages aren't very helpful, but this is what I've seen in practice.

So how do you terminate the account then?

I think you need to delete it with the delete account button and contact Discord Support with your account details and ask them to remove the data. Otherwise if you don't request it your messages stay with the label "Deleted User".
In my case, I don't want to lose my account and all my messages, I just wanted the ones sent to the nasty server deleted. I did try contacting Discord Support about this with my details and they told me they won't interfere and delete any messages unless there is a breach in their TOS which in my case there is not. They said I'd just have to ask the server owner to unban me which is never gonna happen.

@ZeldaPlayer29 If Discord Support refused to remove your messages without a TOS breach, why would you suggest that @Bruk749 contact Discord Support asking that same thing? Are you trying to say that it is more of an "all or nothing" deal where they'll only remove selective messages with a TOS breach, otherwise you can only ask for all of your messages in all of your guilds to be deleted?

My assumption was based on this https://en.m.wikipedia.org/wiki/Right_to_be_forgotten
But again I don’t understand the legal nuances, but I think this would only be valid for a full account wipe, bc from a practical point they will probably not selectively delete stuff.

Based on the assumption that they don't delete messages when an account is deleted I'm willing to bet their interpretation of RtbF considers sent messages to be exempt, because there is wiggle room in current GDPR/Privacy Shield compliang regs for certain types of data to be exempt when RtbF is invoked. I believe the wording is along the lines of "necessary for the continued function of the service" and there is a case to be made that preserving messages people have made in guilds fits that description. Can't hurt to send them an email if you're curious.
You are correct in saying that you can't selectively invoke RtbF. If there is some specific piece of personal data that you can't otherwise delete that invoking RtbF would get rid of, that's your only option.

One thought I've just had is that you could ask the server owner (or a server admin) to use this tool to delete all your messages for you. It's not really ideal but, in my case, I'm not sure what else to do.

Yea, the admin is more or less my friend is letting me delete my comments, however now I found another problem some of my messages that include photos don't go away, while I was deleting my comments I saw this also some messages don't get deleted as well.

Well, I've emailed the guy and asked him to do it for me, although I'm not sure he will. I kinda didn't want to do it because they were making fun of me for deleting some previous messages I had sent, but I don't want to be associated with the server at all now so I don't really have much of choice.

Sigh, knew it wouldn't work

I contacted Discord support to ask about their policy on removing messages when an account is deleted. I'll update as soon as I get a response.
@ZeldaPlayer29 If, when Discord responds to me, it turns out that Discord does allow users who delete their accounts some option to have their entire message history removed, even across guilds which they may no longer have access to, I'd advise you to take this option, as it appears to be your only one to get those messages removed. It's not hard to transfer guild ownership, friends lists, and if you want your PM message history saved, you can request a data download. If you have Nitro, you might have some sentimental value attached to the date on the badge on your profile, but if that's worth more to you than having those messages deleted then you probably wouldn't have commented here. Beyond, say, creating an alternate account, joining that Discord server and over a period of time attaining their trust enough for them to give you the "manage messages" permission, you don't have any other options.
@Bruk749 I'm not sure what you mean. Any picture sent on Discord is the same object as a message and can be deleted the same way. The only messages which can't be deleted w/o "manage messages" are system messages, i.e. joining/boosting a server. Are you referring to reactions, or messages sent by bots which contain pictures, or the server picture, or something else?


Use linuxbrew/brew not linuxbrew/linuxbrew Docker image on CircleCI

Requested in #49.

Azure needs fixing too:
https://github.com/Linuxbrew/homebrew-extra/blob/0a5fc2e9c2a56843841308e3cfad2dc876e05b1f/azure-pipelines.yml#L17
Change https://github.com/Linuxbrew/brew to https://github.com/Homebrew/brew
In this PR or a new PR is fine.

The ultimate plan is to migrate from CircleCI to Azure for building bottles. We're not there yet, but making progress. See https://github.com/Homebrew/brew/issues/6255

Shall I add it to the Xorg tap?

Sure, if you like. Go for it. I mean need to enable it for you on Azure. I can take a look this afternoon. Let me know if you want to do this.


stacks: pre/post script hooks

Is your feature request related to a problem? Please describe.
In general, there isnt any way, how to handle secrets in Portainer, sops seems like an lightweigh option ( at render time ), how to keep versioned secrets in git near docker-compose files. Then, there isnt any way how handle external notifications, nothing is worse than automatic system without observability. Of couse, We can use exporters like node-exporter, cadvisor, etc, but We will still miss business smetrics like portainer_stacks_updated, portainer_stack_upgradable, etc. For now it should be possible to just trigger some webhook, for example with post hook. And at the same time prepare deployment with pre hook.
Describe the solution you'd like
Would be superb to be able to specify pre-hook and post-hook ( shell script, where can be internal variables referenced, like $PORTAINER_STACK_NAME, $PORTAINER_STACK_COMPOSE_PATH, $PORTAINER_STACK_STATUS,  etc..  ) before and after stack deployment, especially with git automatically syncing stack from git repositry. This would allow us to:

prepopulate env variables ( decrypt secrets sops -d secrets.enc.yaml --output-type dotenv > .env , not sure about injecting them into env in portainer )
send notifications ( curl -X POST -H 'Content-type: application/json' --data '{"text":"Stack deployed.."}' https://slack...
do additional stuff

Describe alternatives you've considered

its already possible without Portainer with some CI

Additional context
Thanks

Thanks for that suggestion, we'll take it into consideration.

As I'm in deep need of this solution my-self, I've been thinking about this issue - and I've realized it poses a security threat to allow blind execution of scripts pre/post-hook.
Note that to my needs I need to be able to decrypt some files on Git clone and that's what my thoughts are based around.
I first thought of a solution using Git Hooks, but the go-git library does not execute hooks nor is the global-config available for reading hooks defined globally in the system.
Then I was looking into a per repo yaml-language based file for defining hooks but that is just similar to a shell-script placed the git-repo, although a yaml-basd language file allows us to use pre-defined keys that Portainer can look for and validate to some extent. Using file defined in a repo limits the use-case to git-repositories only.
In the end I think a hook-tab in the stack page would be the solution as that would not only limit to Git-repos and this will allow us for controlling the user-input to some extend.
Regards to security, maybe sandbox the execution of arbitrary shell scripts in a docker container using Docker-in-Docker (DnD)?

Attaching my workaround on this. I've created a patch for the 2.14 release supporting Git hooks by directly calling "git checkout" after the repository is cloned.
This hook is strictly limited to a static shell-script running on the Portainer filesystem, but that is good enough for me.
You can find the patch here: https://gist.github.com/dot-mike/d38427fd3776ca4a31de59e3e2fbf4e1
To make this hook work we use Git hooks as documented here: https://git-scm.com/docs/githooks
Add a shell script named post-checkout to the filesystem running Portainer and update the Git configuration pointing to the directory where post-checkout lives: git config --system core.hooksPath /etc/git/hooks


A few instances where modules are referenced globally even though they were loaded locally

This is different than https://github.com/SAP/openui5/issues/2584 because, in this case, the dependencies are being properly required/imported, but then the module/object is referenced from the global namespace instead of the AMD-style function parameter result.
OpenUI5 version: 1.60 (but checked that issue is still present in master branch)
Browser/version (+device/version): N/A
Any other tested browsers/devices(OK/FAIL): N/A
URL (minimal example if possible): N/A
User/password (if required and possible - do not post any confidential information here): N/A
Steps to reproduce the problem:

Open one of the referenced files.  The file is the line with the complete path and js extension in the list below.
Search for one of the modules under the file name in the list below.
See that the API dependency IS being required/imported properly with the AMD-style sap.ui.define, but then the module is referenced in the global namespace (eg. sap.ui.Device.browser.msie) as opposed to the local reference from the sap.ui.define function (eg. Device.browser.msie).

What is the expected result?
Use of the global namespace is kept to a minimum.
What happens instead?
There are some instances where the global namespace is used, even though the same object is available from the sap.ui.define API.
Any other information? (attach screenshot if possible)
src/sap.m/src/sap/m/MultiComboBox.js
sap.m.Tokenizer.TokenChangeType
src/sap.m/src/sap/m/PDFViewer.js
jquery.sap.global (It provides jQuery.sap.log.warning)
src/sap.m/src/sap/m/PlanningCalendar.js
sap.m.Sticky
src/sap.m/src/sap/m/StepInput.js
sap.m.StepInputValidationMode
src/sap.m/src/sap/m/Table.js
sap.m.ListBaseRenderer
src/sap.ui.core/src/sap/ui/core/routing/async/Target.js
sap.ui.core.ComponentLifecycle
src/sap.ui.core/src/sap/ui/model/FilterProcessor.js
sap.ui.Device
src/sap.ui.unified/src/sap/ui/unified/calendar/Month.js
sap.ui.unified.DateRange
src/sap.ui.ux3/src/sap/ui/ux3/DataSet.js
sap.ui.model.SelectionModel

These should all be fixed with the cleanup mentioned in https://github.com/SAP/openui5/issues/2584#issuecomment-511097339 . I'll update this ticket once the cleanup has been submitted.

Yep, all fixed by a run of the ui5-migraton tooling.


How sensitive is this model to different batch size?

Will small batch size like 512 work? I only have 8 GPUs.

The smallest batch size I tested is 1024, which gives a similar performance. Since we have a learning rate scaling w.r.t. the batch size, I guess the performance will not degrade much with bsz=512, but I'm not very certain.

Will small batch size like 512 work? I only have 8 GPUs.

Hello， could you tell me how to reconstruct an image  work with MAGE? I get the output image almost the same as the input image with the released checkpoint? could you help me with that?


Deleted vacuum_metrics module from docs and src

Deleted vacuum_metrics module from docs/source/symbolic/ , src/einsteinpy/symbolic and src/einstienpy/tests .
Fixes #468

I redid the commits and changed user.name.

As long as you don’t set up you user.email same as what you used to set up GitHub, your commits wouldn’t link to your profile. That’s what Shreyas was trying to say.

Ok so I'll have to change the mail and do the commits again ryt.

Yes. That’s pretty much it.
Regards,
Ritwik

On 01-Apr-2020, at 2:13 PM, Gagan-Shenoy<EMAIL_ADDRESS>wrote:

Ok so I'll have to change the mail and do the commits again ryt.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Superseded by #478


changing the grammar to allow scientific notation numbers

For issue #7 (unable to parse scientific notation)
I can add more tests if needed.  Also, I wasn't sure if I should include the build artifacts (terraformer-wkt-parser.js and terraformer-wkt-parser.min.js).

thanks @tfordon.  this is very cool!  i'll take a closer look soon.

:+1: no need to include built files.  i should be able to tag a new release today.


query optimizer does not use index on linked attribute filter

OrientDB Version: 3.0.22
Java Version:
openjdk version "1.8.0_191"
OpenJDK Runtime Environment (build 1.8.0_191-b12)
OpenJDK 64-Bit Server VM (build 25.191-b12, mixed mode)
OS: centos07
Expected behavior
Use the index in the query plan
Actual behavior
Does not use the index in the query plan
Steps to reproduce
Set up two classes and index the link property
create class Evidence extends V
create class Source extends V
create property Source.name string
create property Evidence.source link source
create index Evidence.source on Evidence(source) NOTUNIQUE

If I use a subquery, the index is picked up and used
select * from evidence where source in (select * from source where name = 'bob')

+ LET (once)
  $$$SUBQUERY$$_0 =
    +-------------------------
    | + FETCH FROM CLASS source
    |   + FETCH FROM CLUSTER 17 ASC
    |   + FETCH FROM CLUSTER 18 ASC
    |   + FETCH NEW RECORDS FROM CURRENT TRANSACTION SCOPE (if any)
    | + FILTER ITEMS WHERE
    |   name = 'bob'
    | + CALCULATE PROJECTIONS
    |   *
    +-------------------------
+ FETCH FROM INDEX Evidence.source
  source IN $$$SUBQUERY$$_0
+ EXTRACT VALUE FROM INDEX ENTRY
  filtering clusters [15,16]
+ DISTINCT
+ FILTER ITEMS BY CLASS
  evidence
+ CALCULATE PROJECTIONS
  *

but if I do the same thing using the link it is not
select * from evidence where source.name = 'bob'

+ FETCH FROM CLASS evidence
  + FETCH FROM CLUSTER 15 ASC
  + FETCH FROM CLUSTER 16 ASC
  + FETCH NEW RECORDS FROM CURRENT TRANSACTION SCOPE (if any)
+ FILTER ITEMS WHERE
  source.name = 'bob'
+ CALCULATE PROJECTIONS
  *

This is possibly related to https://github.com/orientechnologies/orientdb/issues/5546 tho it doesn't use a linkset, just a single link

Hi @creisle
That's strange, it's supposed to work...
I'll check it ASAP and let you know
Thanks
Luigi

Oh, ok, I got it, to use an index chain you also need an index on Source.name...
This is because the subquery is not always the best strategy, eg. if you only have a few Evidence entries and millions of Source entries, traversing the link directly (without doing a subquery) is much more efficient
Thanks
Luigi

@luigidellaquila ok thanks! that makes sense. Does the query optimizer take into account the class size? in this example (a larger version) Evidence would be more than 100X the size of source

Hi @creisle
The query execution planner takes into consideration the class size in some contexts, but not in this case (yet).
My advice in the short term is to just define an index on Source.name, if you need the index chain to be used
Thanks
Luigi

Ok, do you have a ticket for a feature request for this already? or should I leave this one open for that?

Hi @creisle
I'd prefer to have a specific ticket for that
Thanks
Luigi


DirectLine Conversation Formatted Text only?

Is there a way to receive message response metadata (e.g. action buttons etc..) with the DirectLine API?  When I use the DirectLine interface the messages only contain formatted text and no other metadata even though the Bot Service is providing action button information.

I'm having issues with this as well. The 'Attachments' specifically, seem to lack a lot of the data that I put into it with my bot. For example, the FallbackText, Title, TitleLink, etc. don't seem to come in through direct line. Also, the URLs that come in through attachments seem to be formatted differently than what I set them to. Do I need to call Direct Line again to get these images?

Direct Line is currently simplifying some of these fields, and it recreates the attachment on its own service as they may not pass through from the bot. We're going to add more fidelity to a future version of the API, but I hadn't thought about changing the hosting URLs for attachments. Is it a problem that we copy the attachment to our own service before making it available through the API?

No, I realized later that I needed to prepend the 'directline.botframework.com' host to the beginning of the attachment ID that was given by the API


fix: Avoid panic when inventory update errors

Fix two bugs in the inventory provider:

Avoid null pointer panic when inventory update errors (appliedObj was nil)
Avoid losing inventory modifications caused to mutating webhooks (e.g. annotations)

/hold
Blocked on https://github.com/kptdev/kpt/pull/4055 and need to push gke-e2e


Can't read the first column by the name

var entry struct {
Product string name:"Product"
}
if the "Product" is in the first column, it will fail.
debug output:
Product did not appear in the first line

When name:"ColumnName" is set in the field, easycsv treats the first line as the header row.
Thus, the input CSV must be something like
Product,Price
Apple,100

What kind of data did you try to parse?


Support ONVIF Cameras

This would allow HA to support virtually every major security camera.
It also supports a pub/sub motion detection ability, which allows HA to use cameras such as most of the newer Foscam cameras as motion detection sensors as well.
Some notes on how zoneminder does onvif notifications is useful.
There is a python onvif package on pypi which would allow investigating this.
This is a placeholder issue which I very well might take a stab at myself to get some Foscam cameras also doubling as a motion detectors since they have built-in PIR sensors.

Blocking on this issue as I can't test their library without it being resolved.

For a workaround for Motion / Sound sensors.
I done the following: (pending on #https://github.com/quatanium/foscam-python-lib/pull/8).

Get foscam-python-lib and set foscmd as executable.
git clone https://github.com/texnofobix/foscam-python-lib.git
chmod +x ./foscam-python-lib/foscmd

Then add the following:
- platform: command_sensor
    name: foscam_motion
    command: './foscam-python-lib/foscmd -H <IP_ADDRESS> -P 88 -u texnofobix -p mypass'
    value_template: '{% if value_json.motionDetectAlarm == "0" %} disarm {% elif  value_json.motionDetectAlarm == "1" %} off {% else %} on {% endif %}'
- platform: command_sensor
    name: foscam_sound
    command: './foscam-python-lib/foscmd -H <IP_ADDRESS> -P 88 -u texnofobix -p mypass'
    value_template: '{% if value_json.soundAlarm == "0" %} disarm {% elif value_json.soundAlarm == "1" %} off {% else %} on {% endif %}'

Closing this issue as feature requests should be posted in the forum.


Feat/docs

Github Pages docs in MD format
Swagger UI

Seems that you have included the grafana PR inside this PR. Can you remove the grafana dashboard changes so that we isolate the 2 changes one from another


Figure out how to incorporate egeria-ui into demo scenario - coco or other

Currently the 'odpi-egeria-lab' contains

kafka
zookeeper
egeria platforms (*4)
react UI (node based app)
egeria-ui (static content for polymer UI)
ui-chassis
nginx (proxy for UI)

However

We don't currently have any 'lab' oriented content for the UI
The sample data loaded by our 'data catalog' notebook doesn't typically work with searching in the UI (it doesn't work in 3.0 for example)
There is no clear documentation on use of rex/tex (which do work in the react UI)
We don't have any open lineage functionality integrated in the lab demo

In part this is because the drive for this UI has focussed on different use cases, and has been more oriented to initial production use where it offers a Business level interface supporting functionality such as Open Lineage
For this reason it doesn't fit well in the current lab demo.
Therefore I propose to remove

egeria-ui
ui-chassis
nginx

From the 'odpi-egeria-lab'
Then, we can offer an 'odpi-egeria-lab-withui' or similar chart, which will have the odpi-egeria-lab chart as a dependency (which will include the other core elements), and then add in the ui static/api content & proxy
Furthermore since the release cycle of the UI & team ownership differs this allows for appropriate delegation and asynchronous development of the two pieces. The egeria release focussing on the core, and a UI release where the additional features can be developed & tested, notebooks updated & docs clarified.
Finally this tightens up a scenario where the operator can be dropped in more cleanly when ready.
This should add more flexibility, clarity & still retain the container-based demo deployment approach.
cc: @lpalashevski

The UI/lineage lab was included in 3.3 so closing


How to force an action dependency?

What is the issue?
I'm trying to wrap my head around inputs, outputs, and dependencies. From my understanding, one action is dependent on another if it uses the output of that action. But let's say I have this:
actions: {
    source: core.#Source & {
        path: "."
    }
    pull: docker.#Pull & {
        source: "node:current"
    }
    copy: docker.#Copy & {
        input: pull.output
        contents: source.output
    }
    install: bash.#Run & {
        input: copy.output
        script: contents: """
            npm ci
            """
    }
    build: bash.#Run & {
        input: copy.output
        script: contents: """
            npm run build
            """
    }
    deploy: bash.#Run & {
        input: copy.output
        script: contents: """
            npm run deploy
            """
    }
}

Obviously, I want to run install before build and build before deploy. But how do I make these actions dependent on the previous action if there isn't an output?
I tried something like this:
install: bash.#Run & {
    input: copy.output
    script: contents: """
        npm ci
        """
    output: copy.output // <--
}
build: bash.#Run & {
    input: install.output // <--
    script: contents: """
        npm run build
        """
}

but that causes errors.

Your approach is correct, install should have input: copy.output and build should have input: install.output.
If you're having errors it must be something else, so you need to look into the error message more closely. If you need help with that I suggest you fill a support request issue.

Right, just do like @helderco mentions above.
What's not obvious is that since each bash.#Run contains a docker.#Run
https://github.com/dagger/dagger/blob/main/pkg/universe.dagger.io/bash/bash.cue#L73
which itself has an output field with a docker.#Image,
https://github.com/dagger/dagger/blob/main/pkg/universe.dagger.io/docker/run.cue#L173-L176
you get that output field already in every bash.#Run.

Thanks @jpadams! I was missing this part of the equation:

What's not obvious is that since each bash.#Run contains a docker.#Run
https://github.com/dagger/dagger/blob/main/pkg/universe.dagger.io/bash/bash.cue#L73
which itself has an output field with a docker.#Image,


Readme and Manual Update

Update of the MegaMol manual:

Static build
LUA project files
New frontend megamol
ImGui
Built-in configurator
Usage of Cinematic plugin with cinematic_editor.lua project

Links to 'rendered' markdown files:
readme
manual
devguide

PR fixes issue  #689

Closing because git is confused ... better safe than sorry :-P


Fan Speed

Hi, first of all, thank you very much and excellent Plug in!
I would like to ask if it is possible to control the fan speed. Currently in my AC only the temperature can be moved, but there is no way to change the fan speed.
Additionally, if the app could read the current speed and not move it when adjusting the temperature, that would be great.
Currently when I change the temperature it changes the fan speed to Auto, it even changes the speed to Auto the moment I enter the AC in the Home application. (I usually use my AC on low speed or Windfree).
One last request, do you think that later on, you can put a switch to turn the Windfree mode on and off?
Thank you!

Hi @agamboagg! I'd love to see fan speed controls as well. I already started with an implementation but there is one small logical challenge: Homebridge uses a percentage value for the fan speed whereas SmartThings uses discrete values off, low, medium,  high, max and auto. While I can somehow map most values to fractions and vice versa, it is not clear how to handle auto... The key question is what value to show if the fan speed is set to auto and how to set the speed to auto using homebridge controls. Unfortunately, you cannot add arbitrary controls to a heater cooler device. This is all what is supported: https://developers.homebridge.io/#/service/HeaterCooler
If you have a good idea, I'd be happy to implement it!

Hi, thank you very much for the reply!
The way I would put it, although it may seem a bit counterintuitive, is:
0% -> off
20% -> low
40% -> mid
60% -> high
80% -> max
100% -> Auto
However, could it be made to not change the fan speed when the temperature is changed? I mean that even if you don't allow the fan speed to be changed, read the speed at which it is set, and when the temperature is changed, keep that speed.
Example: the AC is at 23c Mid speed, and I want to set it to 22c, which reads that it is at Mid speed and change it to 22c Mid, without having to give the option to adjust the fan.
One question, can you create a switch external to the AC to modify some parameter? in this case it would be to make an independent switch in Homebridge to the Heater / Cooler to activate the Auto speed of the AC ...
If I don't explain myself, please tell me, since my native language is Spanish.
Greetings!

Can auto be set as a separate switch? when on you can't change the speed settings, when off you can change the fan speed slider

Sorry for being a bit unresponsive during the last days but I was on vacation with limited internet access 😉 So first of all, @agamboagg i thing what your suggesting is perfectly clear. I had a look at other plugins and it seems they are using a similar approach: https://www.npmjs.com/package/homebridge-gree-heatercooler-v2 uses 100% as auto, https://www.npmjs.com/package/homebridge-tado-ac does it for 0%.
I'm still haven't completely made up my mind as one the one hand, I'd like to keep as much functionality as possible within one HomeKit accessory. On the other hand, using 0% or 100% as auto is not very intuitive.
Currently, I have a tendency to an additional switch as @beagleboys suggested and remove the fan speed control from the AC as long as auto is enabled. I guess I'll just give it a try and see how it feels.

Any update? 😅

Any update? 😅

x2 😜, me too

Yeah, thanks for reminding me! it's freaking cold in Germany right now, so I'm currently not using my AC at all... so yeah, this somehow disappeared from my radar 😬 I'll see what I can do within the next weeks!

Yeah, thanks for reminding me! it's freaking cold in Germany right now, so I'm currently not using my AC at all... so yeah, this somehow disappeared from my radar 😬 I'll see what I can do within the next weeks!

Thank you from 35*C Mexico!

I gave it a try here https://github.com/0x4a616e/homebridge-smartthings-ac/tree/set_fan_speed but for some reason, it doesn't work. It looks like samsung is ignoring the fan speed command... The command completes with status success but nothing happens... Any ideas maybe? This may be unrelated, but i also cannot find the fan speed in the smartthings app anymore.

I gave it a try here https://github.com/0x4a616e/homebridge-smartthings-ac/tree/set_fan_speed but for some reason, it doesn't work. It looks like samsung is ignoring the set fan speed command... The command completes with status success but nothing happens... Any ideas maybe? This may be unrelated, but i also cannot find the fan speed in the smartthings app anymore.

Hi, I'm using HOOBS, so it won't let me install it from the branch, I could only install it from the main one, could you upload it to the main branch momentarily for me to install it?


Update prow to use repo tensorflow/k8s

The repository has moved to tensorflow/k8s so we need to update prow.

All jobs are running successfully using the new repo.


Markdown source doesn't match help.sap.com

Issue description
Hi team,
I was looking into submitting a PR to change (improve) a few sentences this part of the SAP BTP Developers Guide, when I noticed that the markdown source didn't match what was shown on help.sap.com. See the screen shot below, GitHub on the left, help.sap.com on the right (both opened in private windows):

The position of the image is different, there are text differences, and there is more content on the right. So now I'm wondering - do I not see the latest version on the left or right? What would happen to my PR if it conflicts with other changes that might have been made to the source already.
Thanks!
Best, Nico
Feedback Type (Optional)
other
Page Title on SAP Help Portal (prefilled)
Develop the User Interface
Page URL on SAP Help Portal (prefilled)
https://help.sap.com/docs/btp/btp-developers-guide/develop-user-interface?sprinklrid=11757247470

Hi @nicoschoenteich,
thank you for making us aware of this.
The version you see on help.sap.com is always the most current one. To keep the Markdown version on github.com in sync with what you see on SAP Help Portal, a manual action is required. Usually that's done together with a new release. So maybe wait until @ekaterina-mitova has done this before opening your PR. The markdown version on github.com is a week older.
Sorry for the inconvenience.

Hi @je-hal,
Thanks for clarifying! I didn't know there was a separate process involved. I thought the markdown source was the actual single point of truth. Nevermind then.
How do I know when the GitHub has been updated?

Hi @je-hal, the GitHub source is now updated.

How do I know when the GitHub source has been updated?

Hi @nicoschoenteich, in github you can see it here:

However, I don't think there is a way for externals to see when the latest update happened on SAP Help Portal. Anyway, if the last push to GitHub was done a long time ago, it's likely that someone forgot to push the button. However, other guides also have slower release cycles, so it can happen that the content on GitHub - despite being two months old - is actually up-to-date.


Settings window stretches off screen

First if you want to change the window where the saves are that is NOT going to happen as when you click the window to change the directory it stretches so far off the screen you can't see the save button or any buttons to save your settings causing it to revert back.

@vampiregoat699 consider renaming the issue to a more helpful title; may i suggest "Settings window unusable: stretches far off-screen"
Here's where the bug is kinda awful 😆

it's impossible to reduce the height of the settings window by dragging from the top
it's impossible to drag the "Save" button into view by moving the settings window: you can get right up to the window section which contains the buttons, but that window part is locked outside the visible area of Rusty-PSN, gahh! Torture 😂!

Here is a video showcase
https://github.com/RainbowCookie32/rusty-psn/assets/32672927/d39108b9-3e0f-4d75-8626-2a2631bea127

Alright, seems like there's a bug somewhere with resizable windows. Can't really go tracking it down right now, so I'll workaround it by making the window a fixed size, see 304a6d5e8c336fcf053c944a6c0f829eafacf56e. Not like the window really needs a lot of space to contain the settings.
It behaves as expected on my size, but you can grab the artifacts here when it finishes building.

It's definitely fixed 🎉thanks a lot. And yeah it's not crucial for the settings window to be resizable, although i don't know how different monitor resolutions are handled

Well, I don't know either 😎
I don't really do anything for different resolutions, just hope egui handles it. So if it ends up being an issue, someone can report it I guess. Will tag a new release with the fix and close the issue

Here we go, closing as fixed.


Errors when trying to publish: "No space left on device"

I just tried to publish aws_sdk v<IP_ADDRESS>, and got:
https://deno.land/status/6040bc900015b3d40011fae8
Status: Error: No space left on device (os error 28)

How big is the module you are trying to publish?

$ du -sh deno
159M	deno

Can you try to republish the webhook?

I have published v<IP_ADDRESS> shortly before, could it be that both runs were competing for my 200Mb quota?

The webhook status reports:
Status: Error: Failed to clone git repository https://github.com/christophgysin/aws-sdk-js-v3 at tag v<IP_ADDRESS>

But the module seems to have been published<EMAIL_ADDRESS>

@christophgysin Can you try publish <IP_ADDRESS> ? I think due to the OOM error <IP_ADDRESS> was not published in its entirety.

Sure, I will publish <IP_ADDRESS> later today.

Published <IP_ADDRESS>, same result:
https://deno.land/status/6040dad90060a69400416472
Status: Error: Failed to clone git repository https://github.com/christophgysin/aws-sdk-js-v3 at tag v<IP_ADDRESS>

Are these two separate issues? Should I open a new issue for "Failed to clone git repository"?

FYI: I just pushed <IP_ADDRESS>, and got:
Status: Success: Published module. Failed to run dependency analysis v2.

My module react_icons also suffers from 'Failed to run dependency analysis v2.'
That should be fixed By #304
In my case, the project size is 21M, and I had to drop some icon-set and use some tips to reduce the module size.
And now I need to find a place to add documentation or to split the project to allow single atomic import.
,
so instead of writing
import { FaBeer } from 'react-icons/fa';

That will import all Font Awesome icons to get the Beer SVG.
I will be able to write
import FaBeer from 'react-icons/fa/FaBeer';

To only cache the Beer SVG.
Please lucacasonato give me your point, I use fresh because... you write it... so It's your fault :smile:
That the way we should use SVG icon with fresh ?


Fix typo on the README.md

added missing 's' for 'uses'

Thanks a lot! I am not a English native so this kind of fix are more than welcome!


WebApplicationFactory not calling appropriate ConfigureServices

Is this a Bug or Feature request?:
Bug
Steps to reproduce (preferrably a link to a GitHub repo with a repro project):

Create an a web application project, add a ConfigureServicesDevelopment function and add a service to the ServiceCollection
Create an xunit project and follow the guide from today's Asp.Net Community Standup for utilizing the new WebApplicationFactory from Microsoft.AspNetCore.Mvc.Testing

Description of the problem:
I have a project that depends on Hangfire. I have been experimenting with splitting my ConfigureServices into Development vs Production by following the Asp.Net Core Docs. During normal run of the program ConfigureServicesDevelopment(IServiceCollection services) is called and my             services.AddHangfire(h => h.UseMemoryStorage()); is called and setup.
However, when running a basic xunit test I receive an exception (pasted below). If I move the calls into the normal ConfigureServices method, my tests pass.
`public RoutesTests(WebApplicationFactory fixture)
{
Client = fixture.CreateClient();
}
    public HttpClient Client { get; }

    [Theory]
    [InlineData("/api/phones")]
    public async Task AuthenticationRequired(string url)
    {
        var response = await Client.GetAsync(url);
        response.StatusCode.Should().Be(HttpStatusCode.Unauthorized);
    }`

Test Name:	Test.RoutesTests.AuthenticationRequired(url: "/api/phones")
Test FullName:	.Test.RoutesTests.AuthenticationRequired
Test Source:	RoutesTests.cs : line 22
Test Outcome:	Failed
Test Duration:	0:00:00.001
Result StackTrace:
at Hangfire.HangfireApplicationBuilderExtensions.ThrowIfNotConfigured(IApplicationBuilder app)
at Hangfire.HangfireApplicationBuilderExtensions.UseHangfireDashboard(IApplicationBuilder app, String pathMatch, DashboardOptions options, JobStorage storage)
at .Web.Startup.Configure(IApplicationBuilder app, IHostingEnvironment env)
--- End of stack trace from previous location where exception was thrown ---
at Microsoft.AspNetCore.Hosting.ConventionBasedStartup.Configure(IApplicationBuilder app)
at Microsoft.AspNetCore.HostFilteringStartupFilter.<>c__DisplayClass0_0.b__0(IApplicationBuilder app)
at Microsoft.AspNetCore.Hosting.Internal.AutoRequestServicesStartupFilter.<>c__DisplayClass0_0.b__0(IApplicationBuilder builder)
at Microsoft.AspNetCore.Hosting.Internal.WebHost.BuildApplication()
at Microsoft.AspNetCore.Hosting.Internal.WebHost.StartAsync(CancellationToken cancellationToken)
at Microsoft.AspNetCore.TestHost.TestServer..ctor(IWebHostBuilder builder, IFeatureCollection featureCollection)
at Microsoft.AspNetCore.Mvc.Testing.WebApplicationFactory1.CreateServer(IWebHostBuilder builder)    at Microsoft.AspNetCore.Mvc.Testing.WebApplicationFactory1.EnsureServer()
at Microsoft.AspNetCore.Mvc.Testing.WebApplicationFactory1.CreateDefaultClient(DelegatingHandler[] handlers)    at Microsoft.AspNetCore.Mvc.Testing.WebApplicationFactory1.CreateClient(WebApplicationFactoryClientOptions options)
at .Test.RoutesTests..ctor(WebApplicationFactory`1 fixture)
Result Message:	System.InvalidOperationException : Unable to find the required services. Please add all the required services by calling 'IServiceCollection.AddHangfire' inside the call to 'ConfigureServices(...)' in the application startup code.
Version of ``Microsoft.AspNetCore.All`:
2.1.0-rc1-final

I thought this should've been fixed for RC1? https://github.com/aspnet/Mvc/commit/e0b4c13895fee575544f963b7dbc59d12f8ef7b4

Yeah I saw that line in the source and wondered, but still doesn't appear to work for me.

I thought it was supposed to be ConfigureDevelopmentServices and ConfigureProductionServices?
https://github.com/aspnet/Hosting/blob/2575ebfcfe3f7019bd157fdbab8daaa1d8c45fdb/src/Microsoft.AspNetCore.Hosting/Internal/StartupLoader.cs#L284-L285

@khellang pointed out that I had named my methods incorrectly. Closing as the error was between keyboard and chair.


Update to JupyterLite server extension

Update the cookiecutter to generate JupyterLite server extensions.

We can keep it as a fork so it's easier to sync changes from the upstream repo.


Notification in app that collected IDs will be lost and can't be transfered

Feature description
There should be a clear notification/note in the app (NOT the online documentation!) that the collected IDs will be lost if the app is removed and that you cannot transfer the collected IDs to a new smartphone.
There should be an instruction what to do if you get a new phone (have the old phone active for two weeks, ...).

Problem and motivation
Christmas is two weeks away. I guess that lots of people will get new smartphones as a present. So they will transfer their personal data (google and apple make this possible with cloud backup) and use the new phone, not the old one.
I suspect that most of them do not know that the collected IDs of CWA will NOT be transferred. From this follows that much data collected in the two weeks before Christmas is useless.
A note in the online documentation is not enough, because most people don't look there.

Is this something you're interested in working on
No

@TR0LLIP0P
There is a related discussion in https://github.com/corona-warn-app/cwa-website/issues/411 about adding instructions to the FAQs concerning what to do when moving to a new mobile phone.
Again, this is only the FAQs not the in-app documentation, which you are targeting, but

https://www.coronawarn.app/en/faq/#backup covers the fact that backup doesn't include the CWA keys
https://www.coronawarn.app/en/faq/#delete_random_ids discusses about whether keys are preserved on uninstall or not (for iOS: yes, for Android: no)

I would strongly encourage the developers to think about a one-time push notification explaining what @TR0LLIP0P pointed out and linking that notification to either the FAQ-page or an in-app explanation. I'm personally informed about two people who switched phones and haven't thought about this over the past weeks, I believe this might become a huge problem during Christmas.

I think most new phones are in use now. Christmas is over and sadly no notification appeared. So the main reason for my issue (christmas ->lots of new phones -> lots of keys useless) is gone.
I think I can close this issue?

@TR0LLIP0P
I would leave this Issue open since a notification regarding this would be important regardless of Christmas...

Hi @TR0LLIP0P, @Ein-Tim, @achisto
Thanks again for contributing. We will bring this issue again to the developers.
From what I can see though, it is not possible for iOS devices to send a push notification on an uninstall event, see
https://developer.apple.com/forums/thread/116445 (last post).
DS

@dsarkar Why do you want to send a notification while uninstalling? If you uninstall the app it's too late. The reason for this issue was that people (probably) don't know that keys can't be transferred if they use a new phone. I don't think that people delete all apps from their old phone when they use a new one.

@TR0LLIP0P
I would leave this Issue open since a notification regarding this would be important regardless of Christmas...

The problem is known since the beginning of the app and didn't seem important.
The main event where a lot of people use new phones (= lots of data is lost) is christmas.
So I think it's too late and resources can better be used for other issues.

@TR0LLIP0P
But the question is: When should the user get informed that the data is not included in the backup? At the frost start of the App? At a random time?
At the first start of the App seems best, but I think normal users will quickly forget about this...
If you want to close this, close it. We could continue the discussion in #8

Why do you want to send a notification while uninstalling? If you uninstall the app it's too late. The reason for this issue was that people (probably) don't know that keys can't be transferred if they use a new phone. I don't think that people delete all apps from their old phone when they use a new one.
The problem is known since the beginning of the app and didn't seem important.
The main event where a lot of people use new phones (= lots of data is lost) is christmas.
So I think it's too late and resources can better be used for other issues.

You are right, I believe that's what OP meant. Sending a notification on uninstall is not possible - although at least on iOS there is a notification shown that looks like this. As iOS knows which Apps are using the ENF it should be possible for Apple to add another sentence to this notification like e.g. "Deletion of this App will also delete all collected keys. They can not be recovered, not even through a cloud backup". I'm aware that the CWA-developers can't change underlying iOS-notifications but maybe there is a chance to voice this issue towards Apple.
However it was never only about the uninstallation of CWA but of the backup-and-restore process as @TR0LLIP0P pointed out in their initial post:

So they will transfer their personal data (google and apple make this possible with cloud backup) and use the new phone, not the old one.
I suspect that most of them do not know that the collected IDs of CWA will NOT be transferred.

And this is what already became a problem way before Christmas, as I pointed out above:

I would strongly encourage the developers to think about a one-time push notification explaining what @TR0LLIP0P pointed out and linking that notification to either the FAQ-page or an in-app explanation. I'm personally informed about two people who switched phones and haven't thought about this over the past weeks, I believe this might become a huge problem during Christmas.

This was my comment 20 days ago and now that Christmas eve has passed I already heard of another two people who switched phones while selling their old ones. They now have no possibility to get notified about past encounters as they both believed that the keys would be included in the iOS-backup - just like anything else is. The average user cannot be expected to know all the technical details of what's going on in the background, therefore I'm a bit upset that the chance to prevent events like this has passed.
Still I would leave the issue open because people switching phones and not knowing about how the backups work is not something that will go away anytime soon.

I'm a little bit disappointed. Yesterday I got a notification that I should open the app (or something like that. The text was too long for the notification bar and after clicking on it, the app opened, but there were no infos). But this proves that it was (easily) possible to send a notification like "If you get a new phone, please be aware ... Thank you for using this app. Have a nice holiday" to the users before christmas.
@achisto knows at least 4 people whose IDs are lost because they didn't know about this problem.
@Ein-Tim

But the question is: When should the user get informed that the data is not included in the backup? At the frost start of the App? At a random time?
At the first start of the App seems best, but I think normal users will quickly forget about this...

My idea was to send the notification a few days before christmas to all users, as stated in my post.
Because it's too late for that, it could be integrated in the screen when you open the app first:
You install apps on your new phone and open the cwa for the first time. A screen asks you "Is this the first time you downloaded the app or did you use it on another phone before?" If the user clicks "I used it before" (there has to be a choice, otherwise many user will not read the text!) there could be a text like "please be aware that your collected IDs were not transferred to this phone. For more information...."

I'm a little bit disappointed. Yesterday I got a notification that I should open the app (or something like that. The text was too long for the notification bar and after clicking on it, the app opened, but there were no infos). But this proves that it was (easily) possible to send a notification like "If you get a new phone, please be aware ... Thank you for using this app. Have a nice holiday" to the users before christmas.
@achisto knows at least 4 people whose IDs are lost because they didn't know about this problem.
@Ein-Tim

But the question is: When should the user get informed that the data is not included in the backup? At the frost start of the App? At a random time?
At the first start of the App seems best, but I think normal users will quickly forget about this...

My idea was to send the notification a few days before christmas to all users, as stated in my post.
Because it's too late for that, it could be integrated in the screen when you open the app first:
You install apps on your new phone and open the cwa for the first time. A screen asks you "Is this the first time you downloaded the app or did you use it on another phone before?" If the user clicks "I used it before" (there has to be a choice, otherwise many user will not read the text!) there could be a text like "please be aware that your collected IDs were not transferred to this phone. For more information...."


Info files should be updated with new version when pgBackRest is upgraded

The backup.info and archive.info files keep their original version when a new version of pgBackRest is installed and run.  This is not harming anything but it would be good to keep these versions up to date.

Implemented in b17bf31.


IBus module should not set GTK_IM_MODULE = "ibus" under gnome

Describe the bug
IBus module sets GTK_IM_MODULE = "ibus", which causes ibus still leave committed texts on screen in certain apps like gnome-console and nautilus under gnome.
This environment variable should not be set under gnome any longer as it seems that gnome now handles it properly, and setting this variable will break it. (According to this and this)
Steps To Reproduce
Steps to reproduce the behavior:

Launch gnome-console with GTK_IM_MODULE=ibus kgx and type some random texts, then press left shift.
Committed texts leave on screen

Expected behavior
Committed texts should not leave on screen
Screenshots

Additional context
Add any other context about the problem here.
Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.9.0-rc1, NixOS, 24.05 (Uakari), 24.05.20240329.d8fe5e6`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.2`
 - channels(root): `"nixos"`
 - nixpkgs: `/nix/store/cb1gs888vfqxawvc65q1dk6jzbayh3wz-source`

Add a :+1: reaction to issues you find important.

Or maybe we can add an option to let user choose whether to set this environment variable or not

According to this post, it seems that this variable should not be set when you're under wayland, not just gnome.

Temporary workaround.
environment.variables.GTK_IM_MODULE = lib.mkForce "";

Perhaps the following patch is needed for ibus as well. Unfortunately I do not have time to work on it.
https://github.com/NixOS/nixpkgs/pull/278765/files

Temporary workaround.
environment.variables.GTK_IM_MODULE = lib.mkForce "";

Perhaps the following patch is needed for ibus as well. Unfortunately I do not have time to work on it. https://github.com/NixOS/nixpkgs/pull/278765/files

It's not a very clean workaround, since it leaves error in log(not important though). For a cleaner workaround, you can fetch the ibus.nix and remove those variables, then disable the original module and import the modified one.


Small null check fix to abbrs inside details blocks

Docs abbr tags inside details breaks on responsive view simulated touch for me because e.changedTouches[0] is null. This null check fixes that.
Related to #510

Closed for being a Firefox RDM issue more than our usage of the API, and it's even fixed in the next FF version.