Display elements bounding boxes

Hey!
Some abandoned plugins used to do it rather well (https://github.com/pravdomil/Sketch-Show-Bounding-Boxes) I know that lots of designers — including myself — really appreciate having all bounding boxes on screen to gain precision in order to be pixel perfect. Well I'de be happy to have a built-in solution or a plugin.

Yes Please! It should be a native function as I don't think it could be achieved by the current Sketch API

Yes Please! It should be a native function as I don't think it could be achieved by the current Sketch API


Rescaling of the Image and Plane fields in the metadata tab

At the moment when no specific plane of the metadata record is selected, no scroll bars are displayed in the 'Image' and 'Plane' windows. It would enhance the aesthetics of the screen if this could be incorporated. Also the scaling of the column headers can look a bit clumsy when no specific plane is loaded.

This appears to be the default settings for these types of panes. I guess this is how they think they should look. I am not entirely sure how to change this, unfortunately. So I am going to close this issue for now and I/we can reopen it if it becomes a larger problem. Hope that's alright.


Category of services

What is the purpose of the category of services?
Are these categories managed in the system or are hardcoded in UI only?
Either remove them from UI or implement them properly.

Why instead of deleting this was not fixed it?

We could not fix something that was not work because it wasn't a requirement.
It was introduced by the design people without knowing if we will use it or not.
This enhancement can be done, but is not as trivial as it may seem.
Changes starts from the UI, but it has to change the way the applications get published, we have to change the database, ...
If you think it should be there we can create an issue in order to implement it in the future.

Understood, thanks! I guess then an issue should be opened for further releases.
This can be a bad question from the reviewers: "You talk about offering vertical solutions, how can I see that implemented in the Catalogue?"

I guess it should.
This issue is closed, so better to write this down in the correct place for not losing it.


[venice-server] Dropping unassigned partitions

Summary, imperative, start upper case, don't end with a period
The goal is to remove unassigned partitions that are on disk state
This involves storage service and engine, which upon starting (constructed), will go through folders on disk and verify status of folders
Helix's ideal state is used for consulting partitions and their assignment
The change proposed includes defining a function in VeniceServer that goes through partitions in idealState and compares to partitions in storageEngine
StorageEngine object initialization is also being updated to run the new function that performs this verification.
Resolves #650
How was this PR tested?
Will write corresponding test.
Does this PR introduce any user-facing changes?

[X] No. You can skip the rest of this section.
[ ] Yes. Make sure to explain your proposed changes and call out the behavior change.

A few tests still have yet to pass with the changes that include the check for partition assignment. [e.g. this test]
I have to check whether these are related to the changes I've submitted; also some of these tests seem to pass locally.

(Suggested addition to code, that seems directly useful but is no longer / not used actively:
In the HelixParticipationService file, similar to the getInstance function, I also wrote a getHelixManager function. I tried using this before in access after HelixParticipationService is instantiated, but have changed this implementation. )

(Suggested addition to code, that seems directly useful but is no longer / not used actively:
In the HelixParticipationService file, similar to the getInstance function, I also wrote a getHelixManager function. I tried using this before in access after HelixParticipationService is instantiated, but have changed this implementation. )

(Reverted code in this file back to original state but thought I should make a note of this).

I wrote the related unit test for this PR.
GitHub tests pass for program, except for one Integration test unrelated to my change.

The check over which storage partitions to persist is moved to the shutdown phase of VeniceServer which was discussed.
Shutdown phase contains call to storageService.checkWhetherStoragePartitionsShouldBeKeptOrNot. Services are also stopped when VeniceServer shuts down.

Congratulations on merging your first PR into Venice @kristyelee. Thank you for your persistence.


Extend selected Picker Item label

Is there a way to parse a different label for a selected Item for a Picker? For example, my options are red, blue, and green. When I have red selected, I want the picker to display  "red color"
react: 16.2.0
react-native: 0.53.0
native-base: 2.3.8

@dipernaa That is not possible to do, NativeBase Picker is replacement of React Native Picker.
React Native Picker has not come across this feature
You can do this at max,
<Picker
       iosHeader="Select one"
       mode="dropdown"
       selectedValue={this.state.selected1}
       onValueChange={this.onValueChange.bind(this)}
       itemStyle={{ backgroundColor: 'lightgrey', marginLeft: 0, paddingLeft: 15 }}
       itemTextStyle={{ fontSize: 18, color: 'white' }}
       textStyle={{ color: "red" }}
>

#338

i dont see why this is actually closed. none of itemStyle, itemTextStyle and textStyle worked for me.

https://docs.nativebase.io/Components.html#picker-text-and-item-text-styles-headref

you can use like this for selected value from PICKER:
<Picker.Item color={(this.state.selectedId =='id') ? 'red' : 'grey' } label={'Value'} value={'id'} />


[postgres] Fix dates when using non-standard format

Back-port from 52-stable
Date#to_s actually does the wrong thing when
Date::DATE_FORMATS[:default]
is set to something non-standard.
Fixes #1061

I see some of these same suites failing on the 5.0 branch. Should they be green @enebo @kares @rdubya?

@headius those few errors have always been on 50 and 51.    @dr-itz has actually fixed some but we have largely been just moving forward at this point.  5.0 and 5.1 of rails are technically no longer supported by Rails and we said we would not either (although I am still spinning releases of them when we backport).


KVID-32 - adding support for media type in kraken response

https://kargo1.atlassian.net/browse/KVID-32

adding support for reading mediaType metadata field from Kraken (Added in KargoGlobal/kraken#2753)

Note: hold off on merging this, I'll create a PR into upstream from this branch

Closing this as the prebid PR has been merged


Disable Customizations Pipeline

Bug Description
Our organization only uses AFT to deploy accounts from Account Factory. We do not use it for customizations. However every time we deploy an account the customizations pipeline is invoked, finds no customizations in the github repo and creates an error in the code pipeline logs.
We would like to disable the customizations pipeline and just use the account provisioning pipeline. Is that supported? If so, what is the correct way to achieve this?
Thanks,

@sephross thank you for reaching out.

What AFT version are you using?
Have you tried omitting the account_customizations_name parameter in the account request?


Empty Object

https://github.com/DDCSLearning/authenticationIntro/blob/1f37356fff2022b7d20bbab767a36104e7e14d83/models/user.js#L50
Hi,
I am getting empty object.

Issue was due to arrow function


[New Port Request] net-snmp

Library name: net-snmp
Library description: An SNMP application library, tools and daemon
Source repository URL: https://sourceforge.net/projects/net-snmp/ and https://github.com/net-snmp/net-snmp
Project home page: http://www.net-snmp.org

If you wish to add this port in the future, you can reopen this issue, but we're closing it for now


set up continuous integration

I need to set up continuous integration for this project.

Done with commit #c89f17ed


OSD NYC Meetup Recap

We had a great meeting on November 9th, here's our recap :)

hey! @jancborchardt could you push this through for me? :) thank you!

@dmichl sorry, had vacation last week and wasn’t on the computer mostly. :) Best always @-mention a few people to review.

@jancborchardt no worries! Apologies for always having gone to you :) I hope you had a good vacation. @simonv3 was nice to notice and push them through, I'm still not sure who else have privileges, or should I just ask in chat next time?

@dmichl actually everyone who is member of the organization I think. At the very least the owners, who I just added you to also: https://github.com/orgs/opensourcedesign/people?utf8=✓&query= role%3Aowner :)
In general we use pull requests instead of pushing directly so that there’s just another set of eyes looking over it, and also so we have a bit more open process since pull requests – where people get notified – are much more visible than commits. :)


Create 0.11.0 stable release

From what I read it is mandatory to use 0.11 for kubernetes, but there is no stable release yet. I would feel more comfortable if we had one.

One of the maintainers should take this @jbellmann @harti2006 @whiskeysierra @vadeg


I deployed from zipfile a static app and only index.html works, all resources have access problem

I delivered an app that have in root folder one index.html file and several subfolders such as lib (with js), img (with png) and others subfolders with it own index.html.
Deploy ends ok, and i pointed it into a registered domain into route53.
The problem its that only index.html files are accesibles all others isn't.
When I inspected the resources not avalible I realized that each one of them had this error:

AccessDenied
Access Denied
K2W98XW5AMJXTJSG
v617OBD+PW6q7osCWenRtQt1L5Ku6z6c+OPa6n4txRbsTFU/ASG2r2B5gp2A1QK9SsK0R3J8cHk=

This route works ok:
https://www.mypainpalcbtgame.com/
and this not:
https://www.mypainpalcbtgame.com/img/menu.png
All files are into zip file.
How can allow this acces?

hi, thanks for your answer.
I solved deleting all app and deploying it again.
Previously in my local environment i changed permissions and attributes to all files, because not all was the same, and some of them had mac's extended attributes.
I don't know if that was the reason but it worked !!!


chore: refactor gnosis_safe tests

Fixes
Addresses #1473.
This PR refactors the valory/gnosis_safe tests.
Types of changes
What types of changes does your code introduce? (A breaking change is a fix or feature that would cause existing functionality and APIs to not work as expected.)
Put an x in the box that applies

[ ] Non-breaking fix (non-breaking change which fixes an issue)
[ ] Breaking fix (breaking change which fixes an issue)
[ ] Non-breaking feature (non-breaking change which adds functionality)
[ ] Breaking feature (breaking change which adds functionality)
[x] Refactor (non-breaking change which changes implementation)
[ ] Messy (mixture of the above - requires explanation!)

Checklist
Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING doc
[x] I am making a pull request against the main branch (left side). Also you should start your branch off our main.
[x] Lint and unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have locally run services that could be impacted and they do not present failures derived from my changes

The failure in the E2E tests is caused by TestRegistrationStartUpFourAgentsCatchUp.


Vis-UI: Doesn't show mapped network drives

I used to see mapped network drives, however I no longer see them:

This is on Windows 10, confirmed on multiple computers. Using Vis-UI 1.2.5-SNAPSHOT (the drive was mapped before the application was ever launched. I recall that this used to work, or at least it did from a Windows 7 host, but I don't have easy access to my Windows 7 machine at the moment.

I don't have any network drive so I can't test this at all. Would be great if you could try older version that you are sure was working on Windows 7. In general it's using Java's File.getRoots() so if that doesn't return the drive I'm afraid there isn't anything I can do.

Are you running the program from a different user's context? like "run as administrator"
java can only see the context from the user

This appears to be intermittently working, for whatever reason I have to have explicitly touched the networked drive with windows explorer (or equivalent) before Java can acknowledge it. I'm inclined to say this is a windows/java problem rather then a vis-ui issue.


code to test JS best practices

-Test DRY, KISS, and YAGNI.
-Test JS best practices.
-Test HTML and CSS best practices.

Thanks for the review. 👍 👍 👍


During functional testing in 11.5.4, requests_params with filters fail

The test test/functional/test_requests_params.py::test_get_dollar_filtered_collection fails in the following way against an 11.5.4 device:
Text: u'{"code":400,"message":"Query parameter %24filter is invalid.","errorStack":[]}'```

After manually testing, we've concluded that this occurs due to the way the 11.5.4 device fails to interpret the encoded dollar sign (%24). We can work around this by preventing requests from encoding our requests parameters. One can find an example of this fix here: https://devcentral.f5.com/questions/icontrol-rest-filter-processing-not-http-compliant

We should also have a doc update for this type of query wherever it is applicable.


How to split video inside for loop?

            for chunk in chunks {
                VideoGenerator.current.splitVideo(withURL: url!, atStartTime: Double(chunk[0]), andEndTime: Double(chunk[1]), success: { (url) in
                    self.videoArray.append(url)
                    print("chunk: videoArray: \(self.videoArray)")
                    print("chunk: videoArray count: \(self.videoArray.count)")
                }) { (err) in
                    print("Error: \(err)")
                }
            }

@takasurazeem I don't understand what you want to do or what the problem is that you're having. This isn't SO and I'm not here to do code reviews. I can't alter the library to cater to every specific project's needs. This library is already far removed from what I had in mind for it anyway.
If there is an issue with the library, please provide detailed description and examples and I'll check it when I can or decide if it's a feature I want to add.

VideoGenerator.fileName = Helper.uniqueFilename(filename: "chunks_video")
I have set the file name to be a concatenation of chunks_video and UUID string but everytime the loop executes it generates same name!

@takasurazeem This doesn't seem like a bug no my end, more a problem with how or where you set the name. I'm guessing the old file overrides because you always give the same name for every next video. I suggest going over your code and thinking about how and when you should be setting the video name.


Fix incorrect type declaration in Hybrid constructor

The type declaration in the Hybrid class' constructor required Zend\Crypt\Rsa as its second parameter, which doesn't exist.
I've changed it to Zend\Crypt\PublicKey\Rsa, which is the expected type.
I've also added unit test for it.

@NSmithUK thanks!


Another Flaky test for TestStateExecutionLocalQueues

2023-11-08T20:33:51.0479653Z === RUN   TestStateExecutionLocalQueuesTryConsumeForStateExecution_All_consumed
2023-11-08T20:33:51.0479869Z     state_execution_local_queues_json_test.go:150:
2023-11-08T20:33:51.0480598Z         	Error Trace:	/home/runner/work/xdb/xdb/persistence/persistencetest/state_execution_local_queues_json_test.go:150
2023-11-08T20:33:51.0481049Z         	Error:      	Should be empty, but was map[state_1-1:map[1:{q2 0xc0004120f4}]]
2023-11-08T20:33:51.0481531Z         	Test:       	TestStateExecutionLocalQueuesTryConsumeForStateExecution_All_consumed
2023-11-08T20:33:51.0481956Z --- FAIL: TestStateExecutionLocalQueuesTryConsumeForStateExecution_All_consumed (0.00s)
2023-11-08T20:33:51.0482300Z === RUN   TestStateExecutionLocalQueuesTryConsumeForStateExecution_All_notAllConsumed

Duplicated to https://github.com/xdblab/xdb/issues/81.


[GH - 774] Add tooltip for jira ticket links

implemented tooltip to show  Ticket name, Status, Type, First few characters from the description
ticket here
Fixes #774

Hello @sibasankarnayak,
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.

@hanzei @mickmister
this is the screenshot for tooltip , please share your feedback

Codecov Report

Merging #835 (8c2c689) into master (f5a17e1) will decrease coverage by 0.20%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #835      +/-   ##
==========================================
- Coverage   34.85%   34.65%   -0.21%
==========================================
  Files          52       52
  Lines        5950     5979      +29
==========================================
- Hits         2074     2072       -2
- Misses       3664     3695      +31
  Partials      212      212

Impacted Files
Coverage Δ

server/http.go
39.61% <0.00%> (-0.53%)
:arrow_down:

server/issue.go
30.41% <0.00%> (-1.32%)
:arrow_down:

server/subscribe.go
66.85% <0.00%> (-0.38%)
:arrow_down:

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 f5a17e1...8c2c689. Read the comment docs.

@mickmister can you review this PR.

@Rina-dsg @mickmister @levb Gentle reminder to review this PR

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

@Rina-dsg Could you please review this PR?

@matthewbirtch Would you mind reviewing this PR while @Rina-dsg is out?

@matthewbirtch Would you mind reviewing this PR while @Rina-dsg is out?

@hanzei I'm happy to review. I just triggered a cloud server build, but will I need to do anything to set it up properly to review this PR?

Unfortunately, the test server created via labels can't be used for plugin testing. Please use https://jira-tooltip.test.mattermost.cloud/ instead.
@sibasankarnayak is that server ready?

I wasn't able to upload the plugin  as it was throwing  error as it was created with linux  system

@sibasankarnayak Can you please go into more detail? Can be bundles at https://app.circleci.com/pipelines/github/mattermost/mattermost-plugin-jira/2975/workflows/3e675fbf-f7b7-4a09-b6dd-4c18731d3bd5/jobs/9400/artifacts  be used?

@sibasankarnayak I've updated the server. Could you please try again?

@sibasankarnayak I've updated the server. Could you please try again?

@hanzei have deployed , configured the tooltip feature ,have some sample here
please let me know the feedback

I've added a number of changes in the CSS- hopefully that makes it clear.
Also, a couple other things I noticed:

There is no arrow pointing to the link as per the design. Is this something we can add? Here's a sample of the intended design:

The description should be truncated to 3 lines of text

@matthewbirtch sure , Thankyou for feedback.
regarding the 1 st point showing an arrow is bit complex not sure if we can as the tooltip can open from any direction to the link , if you have any solution can share it with me will give a try. For rest of the comment i will make changes

regarding the 1 st point showing an arrow is bit complex not sure if we can as the tooltip can open from any direction to the link , if you have any solution can share it with me will give a try. For rest of the comment i will make changes

@sibasankarnayak we have a tour point component that utilizes an arrow like this. @AshishDhama could you help provide a possible approach for a similar component in this PR?
If it's too complicated, I think we can proceed without as this still brings value without it, but it would be nice to have it.

tour point component that utilizes an arrow like this

Sure @matthewbirtch if you can share i ll sure try it up.
creating a triangle could be possible with :before and :after but not sure how to create it dynamically on position.

@sibasankarnayak @matthewbirtch, we are using an external library for that particular reason called Tippy. The tour tip component, which we have used, can be found here https://github.com/mattermost/mattermost-webapp/tree/master/components/widgets/tour_tip. @sibasankarnayak you can use the same if that can resolve the issue, but please make sure of using the same version. Please let me know if you need any help.

you can use the same if that can resolve the issue, but please make sure of using the same version

@AshishDhama Just want to note that if we need to use the same version of a node module, we would want to first export this to plugins in https://github.com/mattermost/mattermost-webapp/blob/master/plugins/export.js, so we ensure the same version will always be used.
@sibasankarnayak Can you try implementing this by exposing the library via the webapp's exports? We wouldn't get type completion in the plugin's source code without also installing the package in the Jira plugin's project, but the package will be available to use at runtime via the window object.

@mickmister Thanks will add it there.

adasd

@sibasankarnayak Siba

@sibasankarnayak Have you created the PR to add this to the webapp's exports? As I said, we need that to be done as soon as possible to minimize the change in min_server_version in the plugin

@sibasankarnayak Have you created the PR to add Tippy to mattermost-webapp's exports? As I said, we need that to be done as soon as possible to minimize the change in min_server_version in the plugin

created the PR https://github.com/mattermost/mattermost-webapp/pull/10235

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

@mickmister @AshishDhama For using Tippy or any other tooltip library we need a reference to the actual component which will act as an anchor to the tooltip and in our case the component which needs to be an anchor is present on the Mattermost webapp code so it would not be possible to refer that component directly from the plugin.
However, this can be achieved using some UI hacks but we don't think that would be feasible for rendering the arrow.
However, we have everything similar to the Figma except that arrow which is perfectly fine.
Even the tooltip present in the GitHub plugin doesn't contain the arrow as well and uses Mattermost's default tooltip component.
So, we suggest skipping the arrow and just fixing the typescript errors in this PR, please let us know your thoughts if it's different

For using Tippy or any other tooltip library we need a reference to the actual component

@raghavaggarwal2308 There's an open PR that exposes the component to plugins to use as window.Tippy
https://github.com/mattermost/mattermost-webapp/pull/10235

So, we suggest skipping the arrow and just fixing the typescript errors in this PR, please let us know your thoughts if it's different

I'm fine with skipping the arrow if necessary

@mickmister Since the PR #10235 has not merged yet. We suggest merging this PR with the typescript errors fixed, without the arrow and we can revisit that change once PR #10235 is merged.
Created issue #886 for the same.


mssql-tools does not install on 20.04

Attempted to install on 20.04 with:
# curl https://packages.microsoft.com/config/ubuntu/20.04/prod.list | sudo tee /etc/apt/sources.list.d/msprod.list
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100    89  100    89    0     0    669      0 --:--:-- --:--:-- --:--:--   669
deb [arch=amd64,armhf,arm64] https://packages.microsoft.com/ubuntu/20.04/prod focal main

# sudo apt-get update
Hit:1 http://security.ubuntu.com/ubuntu focal-security InRelease
Get:2 https://packages.microsoft.com/ubuntu/20.04/prod focal InRelease [10.5 kB]
Hit:3 http://archive.ubuntu.com/ubuntu focal InRelease
Hit:4 http://archive.ubuntu.com/ubuntu focal-updates InRelease
Get:5 https://packages.microsoft.com/ubuntu/20.04/prod focal/main arm64 Packages [563 B]
Get:6 https://packages.microsoft.com/ubuntu/20.04/prod focal/main amd64 Packages [10.7 kB]
Get:7 https://packages.microsoft.com/ubuntu/20.04/prod focal/main armhf Packages [563 B]
Hit:8 http://archive.ubuntu.com/ubuntu focal-backports InRelease
Fetched 22.3 kB in 1s (35.7 kB/s)
Reading package lists... Done

# sudo apt-get install mssql-tools
Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Unable to locate package mssql-tools

Tried with the 18.04 repo as suggested in a couple of places:
# curl https://packages.microsoft.com/config/ubuntu/18.04/prod.list | sudo tee /etc/apt/sources.list.d/msprod.list
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100    77  100    77    0     0    516      0 --:--:-- --:--:-- --:--:--   513
deb [arch=amd64] https://paapt-get update
Hit:1 http://security.ubuntu.com/ubuntu focal-security InRelease
Get:2 https://packages.microsoft.com/ubuntu/18.04/prod bionic InRelease [4,003 B]
Hit:3 http://archive.ubuntu.com/ubuntu focal InRelease
Get:4 http://archive.ubuntu.com/ubuntu focal-updates InRelease [107 kB]
Get:5 https://packages.microsoft.com/ubuntu/18.04/prod bionic/main amd64 Packages [118 kB]
Get:6 http://archive.ubuntu.com/ubuntu focal-backports InRelease [98.3 kB]
Fetched 327 kB in 1s (382 kB/s)
Reading package lists... Done

# sudo apt-get install mssql-tools
Reading package lists... Done
Building dependency tree
Reading state information... Done
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 mssql-tools : Depends: msodbcsql17 (>= <IP_ADDRESS>) but it is not going to be installed
E: Unable to correct problems, you have held broken packages.

@warwickchapman -- Warwick, thank you for your feedback.  This repo is solely for documentation issues, with limited support options.  Are you following a particular docs.microsoft.com article?  Meanwhile, please consider these resources:

DBA Stack Exchange
Stack Overflow

You are also welcome to file this issue here:

SQL Server Feedback


Analyzer has looser type checking than CFE for methods in anonymous mixins

The following code is accepted by the analyzer but rejected by the CFE:
main() {}

class C1 with M1 {
  @override
  C1 f() => throw UnimplementedError();
}

class C2 with M1, M2 {
  @override
  C2 f() => throw UnimplementedError();
}

class C3 with M2 {
  @override
  C3 f() => throw UnimplementedError();
}

abstract class I1 {
  M1 f();
}

abstract class I2 {
  M2 f();
}

mixin M1 implements I1 {}

mixin M2 implements I2 {}

The CFE's error message is:
../../tmp/proj/test.dart:8:7: Error: Class 'Object with M1 with M2' inherits multiple members named 'f' with incompatible signatures.
Try adding a declaration of 'f' to 'Object with M1 with M2'.
class C2 with M1, M2 {
      ^
../../tmp/proj/test.dart:19:6: Context: This is one of the overridden members.
  M1 f();
     ^
../../tmp/proj/test.dart:23:6: Context: This is one of the overridden members.
  M2 f();
     ^

For clarity, the type hierarchy looks like this:
   I1    I2
   |     |
   M1    M2
  /  \  /  \
C1    C2    C3

Note that this code doesn't violate soundness, because all concrete methods satisfy all their interfaces:

C1.f has type C1 Function(), which is a subtype of M1 Function() (the type of f in the interface of M1)
C2.f has type C2 Function(), which is a subtype of both M1 Function() (the type of f in the interface of M1) and M2 Function() (the type of f in the interface of M2)
C3.f has type C3 Function(), which is a subtype of M2 Function() (the type of f in the interface of M2).

The reason the CFE is rejecting the code is because it desugars the declaration of class C2 to something like this:
class C2 extends Object_with_M1_with_M2 {
  @override
  C2 f() => throw UnimplementedError();
}
abstract class Object_with_M1_with_M2 = Object_with_M1 with M2;
abstract class Object_with_M1 = Object with M1;

And it's not able to come up with an appropriate signature for f to inherit into Object_with_M1_with_M2.
The analyzer, on the other hand, doesn't do any desugaring, and since class C2 has an override of f with a signature that satisfies both M1.f and M2.f, there's no problem.

Note that the above example is not contrived; it's based on a simplification of a CL I have in progress.
The code I actually would have preferred to write would have looked like this:
main() {}

class C1 with M1 {
  @override
  C1 f() => throw UnimplementedError();
}

class C2 with M1, M2 {
  @override
  C2 f() => throw UnimplementedError();
}

class C3 with M2 {
  @override
  C3 f() => throw UnimplementedError();
}

mixin M1 {
  M1 f();
}

mixin M2 {
  M2 f();
}

But this is rejected by both the analyzer and the CFE. Again, I don't believe there's any soundness justification for rejecting this code; all concrete methods satisfy all their interfaces.

Classifying as area-language for now, since I haven't yet figured out whether the analyzer or the CFE is following the spec. Once we figure that out we can re-classify the issue appropriately.
CC @dart-lang/language-team in case anyone is familiar enough with the spec to know which behaviour is correct (maybe @eernstg or @lrhn?)

I believe an error is correct, but not necessarily the one the CFE gives.
Every class must have a valid, well -defined interface, which is a valid override of every superinterface.
Here "Object with M1 with M2" declares an f method (by mixing in M2.f) which is not a valid override of its superclass's interface's member, which has the signature of M1.f.
That may or may not be how's it's defined, but it's how it should be defined.
Instead we may be treating mixin application as only copying concrete methods, and letting the others be included by saying that the mixin application has the mixin's interface as superinterface.
In that case, it had both M1 and M2 as superinterfaces, and does not declare an override itself, and then the CFE error is correct. Those two have no combined signature, so "Object with M1 with M2" does not have a well defined signature for f.
And the superclass of C2 is "Object with M1 with M2". That's not desugaring, we define repeated mixin application as introducing a new class per mixin applied, and those are real classes.
There is no way the analyzer can be correct without doing the same thing.
More likely, it just forgot to check that those classes are valid, and having an invalid override of "(Object with M1).f" in "Object with M1 with M2" turns out to not be a problem when it's not referenced in the subclass.
But C2.f then cannot be said to be a valid override of all its superinterfaces, since we don't know what the signature was in "Object with M1 with M2".

Based on the provided context and issue description, it seems that the rejection of the code is due to the way mixin application classes are handled by the CFE. To address the issue and provide a potential fix, we can refactor the code to avoid the conflicting method signatures in the mixin application classes. One approach is to use abstract classes for mixins to ensure consistent method signatures. Here's the modified code:
main() {}

class C1 with M1 {
  @override
  C1 f() => throw UnimplementedError();
}

class C2 with M1, M2 {
  @override
  C2 f() => throw UnimplementedError();
}

class C3 with M2 {
  @override
  C3 f() => throw UnimplementedError();
}

abstract class M1 {
  C1 f(); // Use abstract class to define the method signature
}

abstract class M2 {
  C2 f(); // Use abstract class to define the method signature
}

In this fix, we replaced the mixin keyword with the abstract class keyword for both M1 and M2 to make them abstract classes. Then, we defined the f method with the appropriate return types (C1 and C2) in each of the abstract classes.
By using abstract classes for the mixins, we ensure that the method signatures remain consistent and valid throughout the class hierarchy, including mixin application classes. This should resolve the issue and allow the code to be accepted by both the Dart Analyzer and CFE.
Please note that this is one possible fix, and the actual solution may depend on the specific requirements of your project. If you encounter any other issues or need further assistance, feel free to let me know!

For the first example here I reach the same conclusion as @lrhn: The superclass of C2 is Object with M1, M2, and that class is a compile-time error because it declares a member with signature M2 f() that overrides a member of Object with M1 with signature M1 f(), and that's not a correct override.
The tools should report an override error in this case, they should not report having two members in superinterfaces whose signatures are incompatible (M1 and M2 give rise to classes that are found in the superclass chain, so M1.f and M2.f do not occur symmetrically in the superinterface structure of C2).
Both the analyzer and the CFE do report an override error in a case which is perhaps a bit more direct:
mixin M1 {
  List<int> get g => [];
}

mixin M2 {
  List<String> get g => [];
}

class C with M1, M2 {
  @override
  List<Never> get g => [];
}

If we try to come up with a re-interpretetation of the semantics of mixin application where there is no override relationship between (Object with M1, M2).f and (Object with M1).f then we would not get the correct semantics in the case where the former does super.f().
I don't think it's a good idea to try to go down that path (I can't see how that wouldn't be massively breaking), and it looks like the example in this issue is an outlier, and the tools do check for override errors in some cases (and presumably almost all cases).
For the second example, I can't help noticing (off-topicly ;-) that it looks like the f method needs to return an object of type Self. We could just commit to that explicitly (if that is indeed what is intended):
main() {}

class C1 with M1<C1> {
  @override
  C1 f() => throw UnimplementedError();
}

class C2 with M1<C2>, M2<C2> {
  @override
  C2 f() => throw UnimplementedError();
}

class C3 with M2<C3> {
  @override
  C3 f() => throw UnimplementedError();
}

mixin M1<Self> {
  Self f();
}

mixin M2<Self> {
  Self f();
}

Thanks for the analysis, @lrhn and @eernstg! I've reclassified the issue as area-analyzer/analyzer-spec.
@eernstg said:

Both the analyzer and the CFE do report an override error in a case which is perhaps a bit more direct:
mixin M1 {
  List<int> get g => [];
}

mixin M2 {
  List<String> get g => [];
}

class C with M1, M2 { // Error is detected also when `M1.g` and `M2.g` are abstract.
  @override
  List<Never> get g => [];
}

If we try to come up with a re-interpretetation of the semantics of mixin application where there is no override relationship between (Object with M1, M2).f and (Object with M1).f then we would not get the correct semantics in the case where the former does super.f().

Thanks for this example! This is in fact precisely what I tried to write in the first place (except that in my case, M1.f and M2.f were abstract, and of course it cropped up in real-world code). I totally understand why we can't soundly permit this code when either of M1.f or M2.f are concrete. I would still love for the language team to consider allowing this code in the case where M1.f and M2.f are both abstract, though, since it would have addressed my use case. But I think the language team issue tracker is a better place to have that discussion. Also, before we have that discussion, I'd like to upload a CL for code review illustrating the situation in which this bug arose, so that we can have the discussion in the context of a real-world use case. I'll try to upload the CL and create the language issue today.

For the second example, I can't help noticing (off-topicly ;-) that it looks like the f method needs to return an object of type Self. We could just commit to that explicitly (if that is indeed what is intended):
main() {}

class C1 with M1<C1> {
  @override
  C1 f() => throw UnimplementedError();
}

class C2 with M1<C2>, M2<C2> {
  @override
  C2 f() => throw UnimplementedError();
}

class C3 with M2<C3> {
  @override
  C3 f() => throw UnimplementedError();
}

mixin M1<Self> {
  Self f();
}

mixin M2<Self> {
  Self f();
}

This is super useful, thanks! I'll use this approach in my CL.

Analyzer still accepts the original repro. :/


Add Artifactory Analyzer

Add an analyzer which will look up an artifact in an Artifactory instance by SHA-1.
This may present challenges as you can only search by checksum in the Pro version of Artifactory. You can get a 30-day eval license of it to test with, but if the API changes outside of that window, modifying the analyzer will be difficult. Somebody from the community is looking to see if we can possibly get a longer-lived license to test with.

@colezlaw Do you still have plan to implement this feature?

I did some testing with Artifactory and it did not really seem to add any value in the same way pinging Maven Central or a Nexus repo. Maybe I didn't spend enough time reading through the API guide, but the reason we ping Central is to get the GAV coordinates. When you search an Artifactor by hash you don't get the GAV back (unless I'm missing something). So the best it would be is a way to find out if the dependency is in your Artifactory.
Now another type of plugin for Artifactory would be one to scan the entire repo and identify vulnerable components. This would take a bit more time and I do not have a licence for Artifactory pro.
--Jeremy

I asked jfrog about it and they propose 2 solutions:
1. Use AQL
with sample request:
items.find( {   "$and":     [ {"repo" : {"$eq":"libs-release-local"}}, {"actual_sha1" : {"$eq" : "8ab478f35e2220c8e36d7a904fdc67d29257c224"}}     ] } ).include("path")
and the response will contain:
"path" : "com/uriah/artifact/1.0"
pros:

Built-in API
available in on-premise and online version

cons:

'path' parsing to get group/artifact/version

2. Implement user plugin
And expose REST API with checksum in request and GAV values in response
pros:

no need to parse 'path'

cons:

only in 'on-premise' version
need to install user plugin

Which solution, do you think, will be better?

I would vote for option 1 as parsing the path is not that difficult and the setup is easier as it does not require the on-prem with the user plugin.

The ODC project has an Artifactory instance and we are in the process of moving to it - so I'll be able to build/test the integration.

We would be interested in testing this also

For option #1, what happens when group is not supplied? How can the path be parsed? This will be a common occurrence if Artifactory is hosting a Node.js repo where group is an optional item in package.json.
Does the request have to change based on the type of file we're scanning? In the example, the repo is "libs-release-local", but will that repo always exist and be used for all types of components?
Does the response differ when using a non-Maven repo? If the path element is the same, then having the last part of the path being the artifact and anything before it be the group should be fairly simple.

I got Artifactory Pro as well so I might be able to help with this. JFrog got XRay, a product which provides similar functionality so I doubt how much they will be willing to help.
@jeremylong Do you think providing the WADL will answer the questions it looks like it's available here
The response if the artifact not found is HTTP 200 which looks like below
I'll be happy to help with creating this analyzer, will you be able to give me some guidelines to bootstrap  this?
Content-Type: application/json Transfer-Encoding: chunked Connection: keep-alive Server: Artifactory/5.4.6 X-Artifactory-Id: f88ccba01261e0eb:-30f3c7d9:15e6419ed91:-8000 { "results" : [  ], "range" : {   "start_pos" : 0,   "end_pos" : 0,   "total" : 0 } }


(Wizard2.0-Dutch)  Rate does not reach given maxRate

Wizard URL: https://5af093921f12b70ce2f90775--architect-coin-64174.netlify.com/
Steps:

Create Dutch crowdsale with parameters:
decimals=18, supply=1000,
1 tier: minRate=100, maxRate=1100, duration=10 min, supply=1000
Open crowdsale page and check the price value immediately before crowdsale will finish.
Expected result:

price should be 1100 (equals of maxRate)
Actual result:
price is 417

Exec ID : 0xce6e1ff85d4382cf3560f6cfaf0922ed56791fc1f7ab1dcd6da580686e1b3308
If you are reporting a problem with Token Wizard, please include the following information:
Which network did you use? (Mainnet, Kovan, Rinkeby, etc.)
Ropsten
If you were able to create it, what is the URL of your crowdsale?
https://5af093921f12b70ce2f90775--architect-coin-64174.netlify.com/invest?exec-id=0xce6e1ff85d4382cf3560f6cfaf0922ed56791fc1f7ab1dcd6da580686e1b3308&networkID=3
Do you have screenshots showing the problem?

Do you see errors in the dev console? If yes, please include a screenshot
To open the dev console in Google Chrome, press F12, or go to View -> Developer -> Developer Tools, and open the Console tab
If you see errors, please right click on them and "Save as..". Zip saved file and attach it to the Issue.
No

@dennis00010011b this is not reproducible for me anymore with the new versions of auth_os registry and applications. Could you check this issue with the latest code from 2.0 branch?

Still reproducible with last version of 2.0 , latest commit  https://github.com/poanetwork/token-wizard/commit/18d0cba310cca4092111fe148e552abe533c4d08
Also , function (price-time) non-linear

Also, in first seconds after start  value of remaining start was more than total duration

Also, in first seconds after start value of remaining start was more than total duration

@dennis00010011b first seconds is not so critical, and also I think we can't fix it because it depends on now variable of Solidity:
https://github.com/auth-os/applications/blob/e754fdfff3dd55087b7655d3d965a0d35990880e/TokenWizard/crowdsale/DutchCrowdsale/contracts/init/InitCrowdsale.sol#L361-L367
There is no another option to calculate remaining time except using now

@dennis00010011b could you check all other bugs, you mentioned in this issue after merging https://github.com/poanetwork/token-wizard/pull/960?

The issue exists because the function, which calculates rate, depends on block.time which is not continuous. We can't control the behaviour of the block.time. Moreover, this issue will be mitigated on real durations of crowdsales. I guess, we should close it.


Anyone knows how to use jstree with webpack, thanks ?

i try to import jstree.js in webpack project but i got this error
Uncaught ReferenceError: _typeof is not defined

Sorry, I can't help you, but judging from the error message I doubt this is an issue with jstree itself. I will bundle it with webpack in a few days - I will let you know if there were any issues I had to solve.

thanks, i have solved this problem by importing jstree installed  from npm and this error disappear. while before i import the source code directly

Does anyone have an example for use jstree with webpack ?? :(

@imjamespond can you provide some code examples please? After imported as a npm module in a VueJS project, it seems to fail registration with jQuery. Calling $('#myTreeDomEl').jstree({"..."}); falls in a error telling us that jstree is not a function.

maybe you didn't register jstree properly, and these code works fine for me
npm install --save jquery jstree
in webpack.config.js
new webpack.ProvidePlugin({
$ : "jquery",
jQuery : "jquery",
"window.jQuery" : "jquery"
})
]
....
import 'jstree';
$('#jstreeDiv').jstree({"plugins" :{"dnd", "search", "state", "types", "wholerow"}....

This is still an issue, though not directly related to jstree. It's caused by babel, so won't be an issue if you're not using babel as well as webpack. It popped up in a completely unrelated tool: https://github.com/mapbox/mapbox-gl-js/issues/3422
The fix is to add exclude: /jstree/ to the babel-loader options:
                    loader: 'babel-loader', options: {
                    	exclude: /jstree/,
                    	...
                    }


[Feature] Add border/stroke

Is it possible to add the border/stroke option (be able to set its colour - and define its corner radius as well)?
Like:

Would like that, too!

Hello, thank you for your request.
However, I think that there are many variations of borders that you want to add, so instead of adding border options to github-profile-3d-contrib, I would like to handle it with a small script.
Create a python script like the one below in your profile repository.
#!/usr/bin/env python3

import sys
import xml.etree.ElementTree as ET

def main():
    if len(sys.argv) != 3:
        raise Error('usage: {} INPUT_SVG_FILE OUTPUT_FILE'.format(sys.argv[0]))
    tree = ET.parse(sys.argv[1])
    ET.register_namespace('', 'http://www.w3.org/2000/svg')
    root = tree.getroot()
    bg = root.find('{http://www.w3.org/2000/svg}rect')

    bg.attrib["stroke"] = "white"
    bg.attrib["stroke-width"] = "1"
    bg.attrib["rx"] = "10"
    bg.attrib["ry"] = "10"

    tree.write(sys.argv[2], encoding='UTF-8')

if __name__ == '__main__':
    main()

Modify the following contents to your liking.

bg.attrib["stroke"] = "white" - color of border
bg.attrib["stroke-width"] = "1" - width of border
bg.attrib["rx"] = "10" - corner radius X
bg.attrib["ry"] = "10" - corner radius Y

Create a python script like the one below in your profile repository.
For example, let's say you made it with the name bin/change-border.py .
Please modify the .github/workflows/profile-3d.yml file as follows.
  name: GitHub-Profile-3D-Contrib

  on:
    schedule: # 03:00 JST == 18:00 UTC
      - cron: "0 18 * * *"
    workflow_dispatch:

  jobs:
    build:
      runs-on: ubuntu-latest
      name: generate-github-profile-3d-contrib
      steps:
        - uses: actions/checkout@v3
        - uses<EMAIL_ADDRESS>          env:
            GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
            USERNAME: ${{ github.repository_owner }}
+       - uses: actions/setup-python@v4
+         with:
+           python-version: '3.10'
+       - run: python ./bin/change-border.py profile-3d-contrib/profile-night-rainbow.svg profile-3d-contrib/profile-night-rainbow-with-border.svg
        - name: Commit & Push
          run: |
            git config user.name github-actions
            git config user.email<EMAIL_ADDRESS>            git add -A .
            git commit -m "generated"
            git push

The first argument of the script is the input file and the second is the output file.

It doesn't generate the profile-custom-evergreen-with-border
My readme:
![](./profile-3d-contrib/profile-custom-evergreen-with-border.svg)

My change-border-.py:
#!/usr/bin/env python3

import sys
import xml.etree.ElementTree as ET

def main():
    if len(sys.argv) != 3:
        raise Error('usage: {} INPUT_SVG_FILE OUTPUT_FILE'.format(sys.argv[0]))
    tree = ET.parse(sys.argv[1])
    ET.register_namespace('', 'http://www.w3.org/2000/svg')
    root = tree.getroot()
    bg = root.find('{http://www.w3.org/2000/svg}rect')

    bg.attrib["stroke"] = "white"
    bg.attrib["stroke-width"] = "1"
    bg.attrib["rx"] = "10"
    bg.attrib["ry"] = "10"

    tree.write(sys.argv[2], encoding='UTF-8')

if __name__ == '__main__':
    main()

My profile-3d.yaml:
name: GitHub-Profile-3D-Contrib

on:
  schedule: # 03:00 JST == 18:00 UTC
    - cron: "0 18 * * *"
  workflow_dispatch:

jobs:
  build:
    runs-on: ubuntu-latest
    name: generate-github-profile-3d-contrib
    steps:
      - uses: actions/checkout@v2
      - uses<EMAIL_ADDRESS>        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
          USERNAME: ${{ github.repository_owner }}
          SETTING_JSON: ./profile-3d-contrib/ever-green-custom.json
      - uses: actions/setup-python@v4
        with:
          python-version: '3.10'
      - run: python ./bin/change-border.py profile-3d-contrib/profile-custom-evergreen.svg profile-3d-contrib/profile-custom-evergreen-with-border.svg
      - name: Commit & Push
        run: |
          git config user.name github-actions
          git config user.email<EMAIL_ADDRESS>          git add -A .
          git commit -m "3d-contrib generated"
          git push

My profile-3d-contrib:

I had to run the workflow in the Actions manually.
Thanks 🥳


Avoid a double-free on the extra drainJournal call

The drainJournal() function frees a passed-in cursor and returns a new one.  The final call we made to drainJournal() in followJournal() mistakenly discarded the updated cursor value, which would likely produce a double-free error.

Do we need https://github.com/docker/docker/pull/31231 also?

Is this something we need for 1.12.6 also?

I am not sure if this would be a blocker for 1.12.6 ?

I am not sure if this would be a blocker for 1.12.6 ?

This code path is also in docker-1.12.6 because we backported https://github.com/projectatomic/docker/commit/ab4b7477e634fc9d7c0586dc6ff92c85132a1e57 so based on the upstream issue, I'd say we need this in 1.12.6 as well

Ok this is serious,  we need this patch in docker-1.12.6.  I have sent an email to QE about it.

@runcom, I was hitting a crash that https://github.com/docker/docker/pull/31231 fixes pretty regularly in an F26 VM, so if we backported the cleanups from 1.13 to 1.12.6, I think we'll need it.
I can confirm that projectatomic's docker-1.12.5 branch doesn't have the problem that this PR fixes.

@nalind I've backported this fix in docker-1.12.6, can you create a PR to backport https://github.com/docker/docker/pull/31263 and https://github.com/docker/docker/pull/31231 (after you rework it if I've understood correctly from the meeting we had). After that, we'll build for QE

@runcom I think the projectatomic 1.12.6 branch is okay - the two upstream issues touch on changes that landed in 1.13 that we haven't backported to the projectatomic 1.12.6 branch.

@nalind cool, so we need those changes backported in 1.13.1 (or included in upstream 1.13.2)

@nalind cool, so we need those changes backported in 1.13.1 (or included in upstream 1.13.2)

@nalind to reiterate on this one last time - we do not need this PR as long as we cherry-pick the upstream patches right?

@runcom we need this added to the projectatomic-1.13.1 branch.  Now that I look at it again, I don't think we need any of these on the 1.12.6 branch, since followJournal() doesn't return a value yet.


Add Verbosity as a Opt-in by argument feature.

Add extra verbosity to the status as the script runs by issuing an argument such as the common -v -verbose, during launch of script
in example   helsink.py -verbose[-v]

Added


Highway Race - Barrel Dodge

PR Description 📜
Added a new game. Highway Race: Barrel Dodge is an exciting racing game where you control a car speeding along a highway, and your goal is to avoid barrels that appear on the road. Test your reflexes and see how far you can go!
Fixes #1954

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Thank you soo much for contributing to our repository 💗

Thank you @srujana-16 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Thank you  @srujana-16 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀

Hey @srujana-16,
You can't work on a closed issue 👀!
Follow the proper guideline and make a new PR again 😀.
Happy Hacking 💗


Add CurrentPath to Model to support showing current working directory

CurrentPath is a singleton class

Codecov Report

Merging #146 into master will decrease coverage by 0.21%.
The diff coverage is 50.00%.

@@             Coverage Diff              @@
##             master     #146      +/-   ##
============================================
- Coverage     64.69%   64.48%   -0.22%
- Complexity      398      401       +3
============================================
  Files            79       80       +1
  Lines          1266     1281      +15
  Branches        110      110
============================================
+ Hits            819      826       +7
- Misses          410      417       +7
- Partials         37       38       +1

Impacted Files
Coverage Δ
Complexity Δ

...rc/main/java/seedu/address/model/ModelManager.java
82.45% <20.00%> (-7.74%)
22.00 <0.00> (ø)

...a/seedu/address/model/currentpath/CurrentPath.java
66.66% <66.66%> (ø)
3.00 <3.00> (?)

Continue to review full report at Codecov.

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

LGTM. Could be improved by writing a few unit test cases for CurrentPath.

I write the test cases on a seperate PR maybe?


cgroups: reurn error when passing invalid argument to freezer

Signed-off-by: Ma Shimiao<EMAIL_ADDRESS>

@Mashimiao Tests fail :)

LGTM
Also you have misspell in commit message reurn. You can fix if you want.

LGTM


Animation spins really fast only on firefox, loads consistent speed on other browsers

I have a frametime of 3 set.

A frame time of 3 means 3ms per frame which translates to  1000ms / 3ms = 333 frames per second. That is pretty fast. I suggest you limit the frameTime to something like 16.
Which version of SpriteSpin is this about?
Up to Version 4.x.x spritespin used setTimeout for scheduling animation frames. setTimeout usually resolves after around 16ms, which caps the given 3ms to 16ms. So usually you cant go faster than ~60 fps. This however depends on the browser.
Since Version 5.0.0-beta1 SpriteSpin uses requestAnimationFrame to schedule the animation frames. In this case there is no 16ms cap.

switched to<EMAIL_ADDRESS>and it is still going super fast.

Did you also change the frameTime to something around 16? If not, then this is to be expected.

I did
On Tue, Sep 28, 2021 at 7:26 AM Ginie @.***> wrote:

Did you also change the frameTime to something around 16? If not, then
this is to be expected.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/giniedp/spritespin/issues/85#issuecomment-929155619,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABOVFMIVPM3IFIMZHSOHHL3UEGYAHANCNFSM5E3ZRFGA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

--
Ben Carver
Web Developer

i can not reproduce any issue. Can you show me your code or create a stackblitz example?
here is what i play around with
https://stackblitz.com/edit/js-ifxqyd?file=index.html
you can increase the frame time with the + and - buttons

Can you email me privately and I can send you a link to the live site?
On Tue, Sep 28, 2021 at 8:56 AM Ginie @.***> wrote:

i can not reproduce any issue. Can you show me your code or create a
stackblitz example?
here is what i play around with
https://stackblitz.com/edit/js-ifxqyd?file=index.html
you can increase the frame time with the + and - buttons
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/giniedp/spritespin/issues/85#issuecomment-929246115,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABOVFMOJG3Y2Y6QNEA5QCVDUEHCSDANCNFSM5E3ZRFGA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

--
Ben Carver
Web Developer

feel free to email me to the email address in my profile
https://github.com/giniedp


Fixes #200: Goodbye GithubResource

[X] Have you created an issue and used it in the commit message?
[X] Have you checked to ensure there aren't other open Pull Requests for the same issue?
[X] Have you built the project locally prior to submission with mvn clean install?

@ALRubinger can you review it?

Worth mentioning that I also changed the GITHUB_USERNAME and GITHUB_TOKEN env vars to CATAPULT_GITHUB_USERNAME and CATAPULT_GITHUB_TOKEN respectively

@gastaldi Would like to understand the user flow for MiniShift if we remove this; how would the user authorize access for us to push to GitHub in this case if we're not getting the token from Keycloak?

On minishift, CATAPULT_GITHUB_TOKEN env var will be used (which was already part of the process). On Openshift, Keycloak will provide the token, and this env var will not be set

So Minishift will work for a single user only. For multiple users, the CATAPULT_KEYCLOAK_URL will be used instead

Still the UI could authenticate with Keycloak, but in the backend it would just use the token defined as an env var

We should consider at some point having our own Keycloak instance, and get rid of any specific authentication code in Catapult.

Ah, user passes CATAPULT_GITHUB_TOKEN when processing the template, thus granting us access.

7d167ce93fda92741eb9bed8a8ed49e936cd3d9d


docs: Add CONTRIBUTING.md

#111
Description
Added CONTRIBUTING.md file, which is required for every project because it helps to upcoming contributors
Type of Change

[ ] ✨ feat -- New feature (non-breaking change which adds functionality)
[ ] 🛠 fix -- Bug fix (non-breaking change which fixes an issue)
[ ] ❌ ! -- Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 refactor -- Code refactor
[ ] ✅ ci -- Build configuration change
[x] 📝 docs -- Documentation
[ ] 🗑 chore -- Chore

@bhattabhi013
Pls check this #119 pr which fixes #111


fix: disable Save, Save as, Save All if editor is readonly

Fixes #202151 and general readonly editor situations
File selected

Markdown Preview Selected

Disabling Save All seems wrong as that is expected to save editors other than the focused one. I would expect that to save README.md open in the other editor when the preview is focused.

Well, that's a fair consideration. I followed the feedback in the issue but we can easily change the behaviour.

@grgar can I do something to unblock this PR?

This is unlikely to happen given our experiences with running multiple overlapping file watchers on the same folders. Quite the opposite, in the future we might actually register just 1 recursive watcher per folder

Thanks @bpasero for your feedback


feat: rename extern.internalize and extern.externalize instructions

The following instructions were renamed in wasm-gc (https://github.com/WebAssembly/gc/pull/447):
extern.internalize -> any.convert_extern
extern.externalize -> extern.convert_any

thnks


dbeaver causing error: too many open files

i need to restart mysql server to fix this problem.

@erlangparasu please provide steps to reproduce.
Try to disable Open separate connection for each editor and Open separate connection for metadata read options in preferences

@erlangparasu Which OS are you using?
On Linux, you could check how many files a process can open at a time with ulimit -n.


Upgrade to latest swiper version.

As of Chrome v70 older swiper has issues with touch support on hybrid desktops.
https://github.com/nolimits4web/swiper/issues/2853
This appears to be resolved in v4.4.6
May be worth updating the version / or making swiper a required peer dependency.

1.6.9 released


Set active by default

The extension should be ready/active as soon as people installed it.

@andreas0607 :smiley:


Only 1 new match, although I have 60 new matches. Bot gets stuck after a while.

Hi,
I have an issue where I run
'''
Created by Frederikme (TeetiFM)
'''

from tinderbotz.session import Session
from tinderbotz.helpers.constants_helper import *

import constants

if __name__ == "__main__":

    # creates instance of session
    session = Session()

    # set location (Don't need to be logged in for this)
    #session.setCustomLocation("Leuven, Belgium")

    # Will send you an email when you have a match.
    #session.setEmailNotifications(True)

    # login using your google account with a verified email!
    #session.loginUsingGoogle(email=constants.email_google, password=constants.password_google)

    # Alternatively you can login using facebook with a connected profile!
    session.loginUsingFacebook(email="***", password="***")

    # adjust allowed distance for geomatches
    # Note: You need to be logged in for this setting
    # Note: PARAMETER IS IN KILOMETERS!
    #session.setDistanceRange(km=150)

    # set range of allowed ages
    # Note: You need to be logged in for this setting
    #session.setAgeRange(18, 55)

    # set interested in gender(s) -> options are: WOMEN, MEN, EVERYONE
    #session.setSexuality(Sexuality.WOMEN)

    # Allow profiles from all over the world to appear
    #session.setGlobal(True)

    # spam likes, dislikes and superlikes
    # to avoid being banned:
    #   - it's best to apply a randomness in your liking by sometimes disliking.
    #   - some sleeping between two actions is recommended
    # NOTE: these recommendations apply mostly to large amounts of swiping (+100 likes)
    #session.like(amount=10, ratio="72.5%", sleep=1)
    #session.dislike(amount=1)
    #session.superlike(amount=1)

    # Getting matches takes a while, so recommended you load as much as possible from local storage
    # get new matches, with whom you haven't interacted yet
    new_matches = session.getNewMatches()
    # get already interacted with matches
    old_matches = session.getMessagedMatches()
    # get all matches (comment out new_matches and old_matches above so it doesnt load it all for no reason)
    matches = session.getAllMatches()

    #you can store the data and images of these matches now locally in data/matches
    for match in matches:
        session.storeLocal(match)

    # Pick up line with their personal name so it doesn't look spammy
    pickup_line = "Hey {}! This weekend.. You. Me. Pizza? Or do you not like pizza?"

    # loop through my new matches and send them the first message of the conversation
    for match in new_matches:
        # store name and chatid in variables so we can use it more simply later on
        name = match.getName()
        id = match.getChatID()

        print(name, id)

        # Format the match her/his name in your pickup line for a more personal approach.
        message = pickup_line.format(name)

        # send pick up line with their name in it to all my matches
        session.sendMessage(chatid=id, message=message)

        # send a funny gif
        #session.sendGif(chatid=id, gifname="")

        # send a funny song
        #session.sendSong(chatid=id, songname="")

        # send my instagram or you can use alternative socials like facebook, phonenumber and snapchat
        # session.sendSocials(chatid=id, media=Socials.INSTAGRAM, value="Teeti.fm")

        # you can also unmatch
        #session.unMatch(chatid=id)

    # let's scrape some geomatches now
    #for _ in range(5):
    #    # get profile data (name, age, bio, images, ...)
    #    geomatch = session.getGeomatch()
    #    # store this data locally as json with reference to their respective (locally stored) images
    #    session.storeLocal(geomatch)
    #    # dislike the profile, so it will show us the next geomatch (since we got infinite amount of dislikes anyway)
    #    session.dislike()

-> Made by Frederikme
-----------------------------------

Getting ChromeDriver ...
Adding Location Guard extension ...

[WDM] - Current google-chrome version is 87.0.4280
[WDM] - Get LATEST driver version for 87.0.4280
[WDM] - Driver [C:\Users\Sander\.wdm\drivers\chromedriver\win32\87.0.4280.20\chromedriver.exe] found in cache
User is not logged in yet.

COOKIES ACCEPTED.
COOKIES ACCEPTED.
ACCEPTED LOCATION.
DENIED NOTIFICATIONS.
Getting not-interacted-with, NEW MATCHES
[>-----------------------------] 0% of the new matches handled.
[==============================>] 100% of the new matches handled.

Getting interacted-with, MESSAGED MATCHES
[>-----------------------------] 0% of the interacted-with-matches handled.
[>-----------------------------] 3% of the interacted-with-matches handled.
[=>----------------------------] 6% of the interacted-with-matches handled.
[==>---------------------------] 9% of the interacted-with-matches handled.
[===>--------------------------] 12% of the interacted-with-matches handled.
[====>-------------------------] 15% of the interacted-with-matches handled.
[=====>------------------------] 18% of the interacted-with-matches handled.
[======>-----------------------] 21% of the interacted-with-matches handled.
[=======>----------------------] 24% of the interacted-with-matches handled.
[========>---------------------] 27% of the interacted-with-matches handled.
[=========>--------------------] 30% of the interacted-with-matches handled.
[=========>--------------------] 33% of the interacted-with-matches handled.
[==========>-------------------] 36% of the interacted-with-matches handled.
[===========>------------------] 39% of the interacted-with-matches handled.
[============>-----------------] 42% of the interacted-with-matches handled.
[=============>----------------] 45% of the interacted-with-matches handled.
[==============>---------------] 48% of the interacted-with-matches handled.
[===============>--------------] 51% of the interacted-with-matches handled.name
Message: no such element: Unable to locate element: {"method":"xpath","selector":"//h1[@itemprop="name"]"}
  (Session info: chrome=87.0.4280.66)
  (Driver info: chromedriver=87.0.4280.20 (c99e81631faa0b2a448e658c0dbd8311fb04ddbd-refs/branch-heads/4280@{#355}),platform=Windows NT 10.0.19041 x86_64)

age
Message: no such element: Unable to locate element: {"method":"xpath","selector":"//span[@itemprop="age"]"}
  (Session info: chrome=87.0.4280.66)
  (Driver info: chromedriver=87.0.4280.20 (c99e81631faa0b2a448e658c0dbd8311fb04ddbd-refs/branch-heads/4280@{#355}),platform=Windows NT 10.0.19041 x86_64)

distance
Message: no such element: Unable to locate element: {"method":"xpath","selector":"//*[contains(text(), 'kilometers away')]"}
  (Session info: chrome=87.0.4280.66)
  (Driver info: chromedriver=87.0.4280.20 (c99e81631faa0b2a448e658c0dbd8311fb04ddbd-refs/branch-heads/4280@{#355}),platform=Windows NT 10.0.19041 x86_64)

[================>-------------] 54% of the interacted-with-matches handled.
[=================>------------] 57% of the interacted-with-matches handled.
[==================>-----------] 60% of the interacted-with-matches handled.
[==================>-----------] 63% of the interacted-with-matches handled.
[===================>----------] 66% of the interacted-with-matches handled.
[====================>---------] 69% of the interacted-with-matches handled.
[=====================>--------] 72% of the interacted-with-matches handled.
[======================>-------] 75% of the interacted-with-matches handled.
[=======================>------] 78% of the interacted-with-matches handled.
[========================>-----] 81% of the interacted-with-matches handled.
[=========================>----] 84% of the interacted-with-matches handled.
[==========================>---] 87% of the interacted-with-matches handled.
[===========================>--] 90% of the interacted-with-matches handled.
[===========================>--] 93% of the interacted-with-matches handled.
[============================>-] 96% of the interacted-with-matches handled.
[==============================>] 100% of the interacted-with-matches handled.

Scraping matches can take a while!

Getting not-interacted-with, NEW MATCHES
[>-----------------------------] 0% of the new matches handled.
[==============================>] 100% of the new matches handled.

Getting interacted-with, MESSAGED MATCHES
[>-----------------------------] 0% of the interacted-with-matches handled.
[>-----------------------------] 3% of the interacted-with-matches handled.
[=>----------------------------] 6% of the interacted-with-matches handled.
[==>---------------------------] 9% of the interacted-with-matches handled.
[===>--------------------------] 12% of the interacted-with-matches handled.
[====>-------------------------] 15% of the interacted-with-matches handled.
[=====>------------------------] 18% of the interacted-with-matches handled.
[======>-----------------------] 21% of the interacted-with-matches handled.
[=======>----------------------] 24% of the interacted-with-matches handled.
[========>---------------------] 27% of the interacted-with-matches handled.
[=========>--------------------] 30% of the interacted-with-matches handled.
[=========>--------------------] 33% of the interacted-with-matches handled.
[==========>-------------------] 36% of the interacted-with-matches handled.
[===========>------------------] 39% of the interacted-with-matches handled.
[============>-----------------] 42% of the interacted-with-matches handled.
[=============>----------------] 45% of the interacted-with-matches handled.
[==============>---------------] 48% of the interacted-with-matches handled.
[===============>--------------] 51% of the interacted-with-matches handled.
[================>-------------] 54% of the interacted-with-matches handled.
[=================>------------] 57% of the interacted-with-matches handled.
[==================>-----------] 60% of the interacted-with-matches handled.
[==================>-----------] 63% of the interacted-with-matches handled.
[===================>----------] 66% of the interacted-with-matches handled.
[====================>---------] 69% of the interacted-with-matches handled.
[=====================>--------] 72% of the interacted-with-matches handled.
[======================>-------] 75% of the interacted-with-matches handled.
[=======================>------] 78% of the interacted-with-matches handled.
[========================>-----] 81% of the interacted-with-matches handled.
[=========================>----] 84% of the interacted-with-matches handled.
[==========================>---] 87% of the interacted-with-matches handled.
[===========================>--] 90% of the interacted-with-matches handled.
[===========================>--] 93% of the interacted-with-matches handled.
[============================>-] 96% of the interacted-with-matches handled.
[==============================>] 100% of the interacted-with-matches handled.

Image saved as C:\Users\Sander\Documents\TinderBot/data/matches/images/dd6cc556e2318b86f20506cbfc2775e1.jpg
Traceback (most recent call last):
  File "C:\Users\Sander\Documents\TinderBot\quickstart1.py", line 61, in <module>
    session.storeLocal(match)
  File "C:\Users\Sander\Documents\TinderBot\tinderbotz\session.py", line 129, in storeLocal
    StorageHelper.storeMatch(match=match, directory='data/{}'.format(filename), filename=filename)
  File "C:\Users\Sander\Documents\TinderBot\tinderbotz\helpers\storage_helper.py", line 102, in storeMatch
    data = json.load(fp)
  File "C:\Users\Sander\AppData\Local\Programs\Python\Python38-32\lib\json\__init__.py", line 293, in load
    return loads(fp.read(),
  File "C:\Users\Sander\AppData\Local\Programs\Python\Python38-32\lib\json\__init__.py", line 357, in loads
    return _default_decoder.decode(s)
  File "C:\Users\Sander\AppData\Local\Programs\Python\Python38-32\lib\json\decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "C:\Users\Sander\AppData\Local\Programs\Python\Python38-32\lib\json\decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Hi man,
I love this thing but running into 2 issues..
#1: I have about 60 new matches but the "Getting not-interacted-with, NEW MATCHES" only grabs 1 match and messages this instead of doing 60..
#2 After the a while the bot gets stuck I have no clue on how to fix this..
Sander

Hey @S4nderr,
Issue 1 should now be fixed in the latest update.
Problem: In line 70 of match_helper.py, only 1 match is grabbed instead of all matches for faster testing purposes, but forgot to change it back.
line70: chatids = [chatids[0]]
Solution: Deleting line 70 did the trick.
For issue 2: I'm not sure what causes it to fail. If the issue persists after this update, can you open a new issue just regarding that problem?
Thanks for helping out this repository by reporting bugs & crashes btw!


no response on submit button

Challenge Waypoint: Add a Submit Button to a Form has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.106 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
<link href="http://fonts.googleapis.com/css?family=Lobster" rel="stylesheet" type="text/css">

CatPhotoApp

Click here for cat photos.

Things cats love:

cat nip

laser pointers

lasagna

Top 3 things cats hate:

flea treatment

thunder

other cats

<form action="/submit-cat-photo">
  <input type="text" placeholder="cat photo URL">
</form>

Hi FreeCodeCamp/FreeCod,
I'd like to include you in my network to share updates and stay in touch.

Dean

Confirm that you know Dean: https://www.linkedin.com/e/v2?e=0-imn0r9xp-lq&t=ssuw&tracking=eml-guest-invite-cta&ek=invite_guest&sharedKey=DOtNh91y&invitationId=6122996591365283841
You received an invitation to connect. LinkedIn will use your email address to make suggestions to our members in features like People You May Know. Unsubscribe here: https://www.linkedin.com/e/v2?e=0-imn0r9xp-lq&t=lun&midToken=AQFmrFVhGEd36Q&ek=invite_guest&loid=AQHFxVyfkFwOjwAAAVPlBA6MbdVBjjH5DR87hr2lS5wUNvIEtD0Qh-b01b2NqrD_5VBvgNwA_w21OBgkhuyCPXJzu-pD6BGQRu89LGhOEW916mKBq0pIPn_D9LvSJ5qDL0rV6rAlVFwnSnLd67_FUfLvQHwPEZN-qGw29618j8ikryIVxybjCX2I_cnRFJp6at9k&eid=0-imn0r9xp-lq
This email was sent to<EMAIL_ADDRESS>If you need assistance or have questions, please contact LinkedIn Customer Service: https://www.linkedin.com/e/v2?e=0-imn0r9xp-lq&a=customerServiceUrl&ek=invite_guest
© 2016 LinkedIn Corporation, 2029 Stierlin Court, Mountain View CA 94043. LinkedIn and the LinkedIn logo are registered trademarks of LinkedIn.


Move mining roid and magnet pulls closer

About the PR

Moves the mining asteroid distance from 400-450 down to 300-350.
Adjusts magnet pulls so they spawn at a more consistent distance that is closer to the station. The previous range was hard to calculate but now its about 64-128.
Why / Balance

Mining Asteroid
Having the mining asteroid be so far away that you cannot even see it on a mass scanner half the time makes any kind of non-shuttle traversal impossible. You should be fully capable of reaching it with just a jetpack and knowledge of its location.
Magnet Pulls
The magnet has a weird consistency issue where the distance it pulls things in can vary wildly while also being dependent on the size of the map. This isn't an ideal solution as it means that magnet pulls, which are meant to be trivially accessible and low-tech, can spawn 250 meters out from the station. Moving it closer means that you can reach them by just running to them or with the gentle assistance of a fire extinguisher or a stack of rods.
Media

Requirements

[x] I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

tweak: Moved the mining asteroid slightly closer to the station.
tweak: Things pulled in by the salvage magnet should spawn closer to the station at a more consistent distance.

Also most salvagers tend to ignore the asteroid because you often can't even see it on the map. Good change because now they're more likely to hit up the asteroid first and actually get the station mats.


Create Planet Model

Create planet model using blender, Four basic types of model: Normal, Soldier Planet, Resource Planet, Hybrid Planet.

4 different models created, and pushed to Google Drive.


templates: Update docker.yaml to use official Docker repository

Background
According to the official installation guide for Ubuntu, using get.docker.com script is not recommanded for production environment.

Docker provides a convenience script at get.docker.com to install Docker into development environments quickly and non-interactively. The convenience script is not recommended for production environments, but can be used as an example to create a provisioning script that is tailored to your needs.

So I made changes to examples/docker.yaml according to the official Docker Engine installation guide and replaced convenience-script with apt-get using official Docker repository.
This might not be relevant since lima-vm is still in beta state(assuming it hasn't reached to v1), I thought this might be a good idea for future production-ready lima-vm.
Notes
I purposfully did not update examples/docker-rootful.yaml since rootful environment should be used for test/local development environment only.

I purposfully did not update examples/docker-rootful.yaml since rootful environment should be used for test/local development environment only.

Lima itself is made for test/local dev env, so docker.yaml and docker-rootful should have the same design

I purposfully did not update examples/docker-rootful.yaml since rootful environment should be used for test/local development environment only.

Lima itself is made for test/local dev env, so docker.yaml and docker-rootful should have the same design

I see. I didn't know lima-vm is not for production environment at it's core.
I made changes anyway to reflect your comments, as well as for docker-rootful.yaml. But since the main goal is not for production environment, I think using get.docker.com is way more simple and fits its purpose. I'll leave this pr up to you to weather merge or reject.

Can we close this ?

My vote is to close this PR; IMO it adds complexity without providing additional benefits.

I'm closing this, but I could be convinced to merge this if this PR were validating the gpg key id, or maybe all the digests of the dpkg files.


Twitter Suggestion: Contact Journal: Add 4rd category „Vaccinated“

Current Implementation
In the contact journal, you can enter people & locations.
For people, you can choose the circumstances in which the encounter took place:

Suggested Enhancement
Add a 4rd category "Vaccinated" / "Not vaccinated".
Expected Benefits
In case the user catches COVID and sends their contact journal to the health authority, they can better asses the risk for the encounters of the person.

Not sure if that's the expected benefit the author on Twitter mentioned:

Die App erkennt dadurch das Risiko besser, ob ein geimpfter oder ungeimpfter infiziert ist.
Bezüglich der Ansteckungrate ist das schon entscheidend...
https://twitter.com/Nobody_see_me/status/1417406339341508635

Engl. (auto translated)

The app thus recognizes the risk better, whether a vaccinated or unvaccinated person is infected.
In terms of the infection rate, this is already decisive...

It sounds like that the user assumes that the data from the contact diary is used in the risk calculation. Which isn't and can't be.

@LukaHarambasic
Yes, I also wondered and thus I wrote my own expected benefit. I'll come back to the Twitter user.

I don't think this is a good idea, as you would be storing private health information about the vaccination status of a person in your device.

As the CWA project went into ramp-down mode, I don't expect this feature to be implemented. I'm therefore closing this issue.


use cardano-haskell snapshots instead of extra-deps

Issue Number

N/A
Overview

[x] I have used cardano-haskell snapshots instead of extra-deps

Comments

bors r+


"This site can’t provide a secure connection"   [ ERR_SSL_PROTOCOL_ERROR ]

Current Behavior
It seems that I cannot get the certificates to bind to the native HTTPS listener within openvpn-auth-oauth2.  I've tried my own wildcard cert [from trusted authority] as well as generating a self-signed key/crt from the example provided.  I've turned the logging up to DEBUG level and do not see anything useful in there.
snippet of self-signed config below:
CONFIG_HTTP_BASEURL=https://ovpn1.mycompany.com:9000
CONGIG_HTTP_TLS=true
CONFIG_HTTP_KEY=/etc/sysconfig/server2.key
CONFIG_HTTP_CERT=/etc/sysconfig/server2.crt
CONFIG_LOG_LEVEL=DEBUG
I guess my question is where can I look to troubleshoot the native HTTPS listener and the loading of the crt and key into it?
[If I change it to http:9000 the page will load properly, but unfortunately AAD will not accept http URLs for redirect.]
Expected Behavior
No response
Steps To Reproduce
No response
Environment

openvpn-auth-oauth2 Version: 1.13.5
OpenVPN Server Version:    2.6.8
Server OS:  jammy
OpenVPN Client (flavor, OS):  [tried both] on windows

Anything else?

There are no logs on server side?
I fully depends agaist golang standard https listener with standard ciphersuite.
Are you using RSA key or EC keys for your SSL cert?

yep you were right - the 'typo' was the issue,  once i fixed that - all i had to do was grant read permission to my key  [once i actually turned on TLS, there was a lot of useful information]
thanks for making this amazing 'plug-in'!!!!!


Bluetooth Device not found

What issue do you have? Please be as thorough and explicit as possible.
Fresh install of Govee plugin. I followed the Bluetooth Control document to scan to find out the bluetooth address of the H6181 but nothing is found. I also check with my phone and laptop and the H6181 is not discoverable; only through the Govee app. Not sure if I am doing something wrong. Thanks!
Details of your setup.

Do you use (1) Homebridge UI-X (2) Homebridge CLI or (3) HOOBS?
Homebridge UI-X

Which version of Homebridge/HOOBS do you have?
4.14.4

Which platform do you run Homebridge/HOOBS on (e.g. Raspberry Pi/Windows/HOOBS Box)? Please also mention your version of Node.js/NPM if known.
Raspberry PI 2b
Node.js 18.12.1
npm 8.19.2

Which version of this plugin (homebridge-govee) do you have? Has the issue started since upgrading from a previous version?
New install v.7.7.1

Which Govee devices do you have that are causing issues? Please include product models if applicable.
H6181 light strip

Please paste any relevant logs below.
[27/12/2022, 20:47:15] [Govee] [HTTP] client enabled and found 2 device(s).
[27/12/2022, 20:47:16] [Govee] [AWS] client enabled.
[27/12/2022, 20:47:16] [Govee] [AWS] connect event.
[27/12/2022, 20:47:17] [Govee] [API] client enabled and found 0 device(s).
[27/12/2022, 20:47:17] [Govee] [BLE] client enabled.
[27/12/2022, 20:47:17] [Govee] [H6181_F06E] initialising with options {"adaptiveLightingShift":0,"aws":"unsupported","ble":"enabled","brightnessStep":1,"colourSafeMode":false,"lan":"unsupported"}.
[27/12/2022, 20:47:17] [Govee] [H6181_F06E] initialised with id [6B:AF:A4:C1:38:6F:F0:6E] [H6181].
[27/12/2022, 20:47:17] [Govee] [Smart Thermometer] initialising with options {"lowBattThreshold":20}.
[27/12/2022, 20:47:17] [Govee] [Smart Thermometer] initialised with id [92:B0:A4:C1:38:98:41:A8] [H5102].
[27/12/2022, 20:47:18] [Govee] [BLE] stateChange: poweredOn.
[27/12/2022, 20:47:18] [Govee] [Smart Thermometer] [HTTP] receiving update {"source":"HTTP","battery":63,"temperature":2010,"humidity":2770,"online":false}.
[27/12/2022, 20:47:18] [Govee] [Smart Thermometer] has been reported [offline].
[27/12/2022, 20:47:18] [Govee] [Smart Thermometer] current battery [63%].
[27/12/2022, 20:47:18] [Govee] [Smart Thermometer] current temperature [20.1°C / 68.2°F].
[27/12/2022, 20:47:18] [Govee] [Smart Thermometer] current humidity [27.7%].
[27/12/2022, 20:47:19] [Govee] ✓ Setup complete. Have a feature request? Visit https://bit.ly/hb-govee-issues to ask!
[27/12/2022, 20:47:48] [Govee] [Smart Thermometer] [HTTP] receiving update {"source":"HTTP","battery":63,"temperature":2010,"humidity":2770,"online":false}.
[27/12/2022, 20:48:18] [Govee] [Smart Thermometer] [HTTP] receiving update {"source":"HTTP","battery":63,"temperature":2010,"humidity":2770,"online":false}.
[27/12/2022, 20:48:48] [Govee] [Smart Thermometer] [HTTP] receiving update {"source":"HTTP","battery":63,"temperature":2010,"humidity":2770,"online":false}.
[27/12/2022, 20:49:18] [Govee] [Smart Thermometer] [HTTP] receiving update {"source":"HTTP","battery":63,"temperature":2010,"humidity":2770,"online":false}.
[27/12/2022, 20:49:27] [Govee] [H6181_F06E] starting update with params [{"cmd":4,"data":255}].
[27/12/2022, 20:49:27] [Govee] [H6181_F06E] starting timer.
[27/12/2022, 20:49:27] [Govee] [H6181_F06E] starting scan.
[27/12/2022, 20:49:27] [Govee] [BLE] scanning started.
scanStart
[27/12/2022, 20:49:27] [Govee] [H6181_F06E] scanning started.
[27/12/2022, 20:49:27] [Govee] [H6181_F06E] starting loop.
[27/12/2022, 20:49:35] [Govee] [H6181_F06E] could not find device [a4:c1:38:6f:f0:6e].
[27/12/2022, 20:49:35] [Govee] [H6181_F06E] not using BLE connection as BLE took too long.
[27/12/2022, 20:49:35] [Govee] [H6181_F06E] starting update with params [{"cmd":1,"data":1}].
[27/12/2022, 20:49:35] [Govee] [H6181_F06E] starting timer.
[27/12/2022, 20:49:35] [Govee] [H6181_F06E] starting scan.
[27/12/2022, 20:49:35] [Govee] [H6181_F06E] could not be updated as no connection method available at file:///usr/local/lib/node_modules/homebridge-govee/lib/platform.js:1872:13.
[27/12/2022, 20:49:35] [Govee] [BLE] scanning started.
scanStart
[27/12/2022, 20:49:35] [Govee] [H6181_F06E] scanning started.
[27/12/2022, 20:49:35] [Govee] [H6181_F06E] starting loop.
[27/12/2022, 20:49:42] [Govee] [H6181_F06E] could not find device [a4:c1:38:6f:f0:6e].
[27/12/2022, 20:49:42] [Govee] [H6181_F06E] not using BLE connection as BLE took too long.
[27/12/2022, 20:49:42] [Govee] [H6181_F06E] starting update with params [{"cmd":1,"data":1}].
[27/12/2022, 20:49:42] [Govee] [H6181_F06E] starting timer.
[27/12/2022, 20:49:42] [Govee] [H6181_F06E] starting scan.
[27/12/2022, 20:49:42] [Govee] [H6181_F06E] could not be updated as no connection method available at file:///usr/local/lib/node_modules/homebridge-govee/lib/platform.js:1872:13.
[27/12/2022, 20:49:42] [Govee] [BLE] scanning started.
scanStart
[27/12/2022, 20:49:42] [Govee] [H6181_F06E] scanning started.
[27/12/2022, 20:49:42] [Govee] [H6181_F06E] starting loop.
[27/12/2022, 20:49:48] [Govee] [Smart Thermometer] [HTTP] receiving update {"source":"HTTP","battery":63,"temperature":2010,"humidity":2770,"online":false}.
[27/12/2022, 20:49:49] [Govee] [H6181_F06E] could not find device [a4:c1:38:6f:f0:6e].
[27/12/2022, 20:49:49] [Govee] [H6181_F06E] not using BLE connection as BLE took too long.
[27/12/2022, 20:49:49] [Govee] [H6181_F06E] could not be updated as no connection method available at file:///usr/local/lib/node_modules/homebridge-govee/lib/platform.js:1872:13.

Updated to the latest version v7.9.0, node 18.13.0 and npm 9.3.1 and I am getting a new error:
[21/01/2023, 22:53:27] [Govee] [HTTP] retrieved access token from cache.
[21/01/2023, 22:53:28] [Govee] [HTTP] client enabled and found 2 device(s).
[21/01/2023, 22:53:30] [Govee] [AWS] client enabled.
[21/01/2023, 22:53:31] [Govee] [AWS] connect event.
[21/01/2023, 22:53:32] [Govee] [API] client enabled and found 0 device(s).
[21/01/2023, 22:53:32] [Govee] [BLE] disabling client as Cannot find module '../build/Release/bluetooth_hci_socket.node'
Require stack:
-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>stack:.
[21/01/2023, 22:53:32] [Govee] [H6181_F06E] will be visible but uncontrollable as BLE not available.

In terminal, using bluetoothctl and scan on, I can detect other bluetooth device in the area so no issue with bluetooth.

I encountered the same issue.
I need to update @abandonware/bluetooth-hci-socket directly within the homebridge-govee module.
cd /usr/local/lib/node_modules/homebridge-govee/
npm install @abandonware/bluetooth-hci-socket

It worked for me!


grammar and wording: Simplest Approach: Don’t Collect Personal Inform…

…ation

I have a new PR to implement parts of these.


Day1 and Day2 assignment

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Update on Day1 and Day2 assignment
Test URLs:

Before: https://main--eds-website--mayuriswain33.hlx.live/
After: https://mayuri-develop--eds-website--mayuriswain33.hlx.live/

Noor, I did all the required changes, please check.


Error FS0832: Only functions may be marked 'inline' is not thrown on static members that are not functions

A static member that is marked inline and is not a function but a field is accepted as an inline function and does not throw FS0832. This may well be a feature and not a bug, in that this gets compiled into a get() member which itself is a function. However, I do find it odd that this distinction exists, hence raising this bug.
Conversely, if a static member does not get compiled into a static global accessor field as a let-binding does, then inlining does make sense, or changing this to into no-inlining-allowed and functioning similar to let-bindings would make sense performance wise.
Repro steps
To repro, see this code:
type Foo =
    | Bar
    static member inline Test = function Bar -> true    // no FS0832, but expected
    static member inline Other = 2                      // no FS0832, but expected
    static member inline Fun = fun _ -> 12              // no FS0832, but expected

module FooMod =
    let inline Test = function Bar -> true  // FS0832
    let inline Other = 2                    // FS0832
    let inline Fun = fun _ -> 12            // FS0832

Expected behavior
The expected behavior is that the static members would also throw this error. However, that is not the case, which leaves one wondering whether the inlining is ignored or actually takes place on member references.
Actual behavior
Error FS0832 is not thrown on static members.
Conversely, it may be nice to allow inlining on let-bindings that are values and not functions as well. It would simply work as a literal, except that this would have the odd side-effect that inlining a value and using it will typically be slower: it will be evaluated each time instead of only once (unless for the odd case where lookup is slower than immediate in terms of CPU registers and cache).
Known workarounds
No workaround exists, or is necessary. It may well be considered a feature, unless the inlining does not take place, in which case it should raise an error.
Related information
Behavior seen with VS2015 (no updates) and VS2015 Update 3, using F# 4.0.

Yes, this is by design - the static property members are shorthand for get() functions.

Fair enough, albeit somewhat surprising. Thanks.


Add ubuntu paths

The packages in Lokesh's ppa that is currently used for podman puts the
conmon and runc binaries in /usr/lib/crio/bin/conmon and /usr/lib/cri-o-runc/sbin/runc
respectively.
Signed-off-by: baude<EMAIL_ADDRESS>

LGTM

@rh-atomic-bot r+


chore(repo): Remove changeset check for renovate PRs

Description
According to https://unix.stackexchange.com/a/560315 you can use an OR in bash like that 😅
This PR skips the required changeset check for Renovate PRs.
Got the login ID through checking with GitHub's API:
  "url": "https://api.github.com/repos/clerk/javascript/pulls/3493",
  "id":<PHONE_NUMBER>,
  "title": "fix(shared): Update minor & patch dependencies",
  "user": {
    "login": "renovate[bot]",
    "id": 29139614,
    "type": "Bot",
    "site_admin": false
  },

Checklist

[ ] npm test runs as expected.
[ ] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[ ] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Such PRs won't get automerged and we'll need to approve, so we can add those changelog entries by ourselves. AFAIK you can't really automate changesets from Renovate easily so we'll need to do it manually anyways.


tests: derper.rs

Minimal testcase

run local derper
connect with 2 derps
exchange messages
run stun check
run captive portal check
get home page thing


Debate Judge- Dhruv Sinha A2

Adding a big bench task. This task is known as debate_judge

Thanks for your PR. Reminder that tasks should have a README.md -- e.g. see the example task simple_arithmetic. Also, make sure your task addresses the structured review criteria.

Delete this request!


What is the value of MAXIMUM_PROC_PER_SYSTEM described in this page

What is the value of MAXIMUM_PROC_PER_SYSTEM described in this page and where can we get the value from ?

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

ID: b99aa941-27c4-c5cd-386c-fa1f64ab828f
Version Independent ID: 23ccb092-56e8-8165-65f4-25bd2e3a983d
Content: Standardized INF Keywords for RSS - Windows drivers
Content Source: windows-driver-docs-pr/network/standardized-inf-keywords-for-rss.md
Product: windows-hardware
Technology: network
GitHub Login: @aviviano
Microsoft Alias: amvivian

@aviviano Thank you for your response. Will the same be updated in the documentation page ?

@sandilyabhagi Yes, we have added a note for the *MaxRssProcessors keyword on https://docs.microsoft.com/en-us/windows-hardware/drivers/network/standardized-inf-keywords-for-rss.
Thanks again.


use-cases: Add Intel SGX use case

Document how to use Intel SGX with Kata.
fixes #697
Signed-off-by: Julio Montes<EMAIL_ADDRESS>

/test

@GabyCT fixed, thanks

Ping @devimc.

thanks @jodh-intel for reviewing this, changes applied in https://github.com/kata-containers/kata-containers/pull/484
closing. SGX support won't land on 1.x


[Task]: Change app name to Firefly Shimmer

Task description
For launch of shimmer the app will be called Firefly Shimmer to make it easier to find.
Requirements

[ ] Change app name to Firefly Shimmer

Creation checklist

[ ] I have assigned this task to the correct people
[X] I have added the most appropriate labels
[X] I have linked the correct milestone and/or project

We should look at revising our branch strategy soon, including re-naming stardust-develop to shimmer, which I'd prefer over staging. Definitely need to continue this discussion because it affects lot's of parts of our process like release and branch merge strategies, changing / automating app config, website releases, and the choice around 1 or 2 apps down the line


ISPN-5203 Protostream annotation attributes that do not have a default value should be required

Jira: https://issues.jboss.org/browse/ISPN-5203

Integrated in master.


Hidden dependencies

This library has a dependency on node stream, node crypto and node url modules. When used in my project I was forced to provide fallbacks using Webpack.
Is there any way we can remove these dependencies @mcoqblin-hiptest? They seem to be deps on atlassian-jwt package.

Unfortunately I'm not sure we can do much. Webpack 5 does not provide polyfills for core node modules anymore. AP.context.getToken requires JWT, so even without atlassian-jwt we will still need crypto and stream.
I'm not sure about url. It is used by atlassian-jwt but I think it is used for QSH which we do not use. The dev server of Fake AP itself works without it.
Do you know which AP methods require url? Or does your build fail without event using it?
By the way you can provide these polyfill only in development and test environments in Webpack.

How about switching to https://github.com/panva/jose which a plain js implementation?

That's a good idea. Go ahead if you want to try, otherwise I'll try to do it during the week.


Verify all OIDC id tokens in client mode (applies to all grant flows)

OpenID ID Token validation should perform the following checks outlined in the OpenID Connect Core spec: https://openid.net/specs/openid-connect-core-1_0.html#IDTokenValidation
I'm thinking we can use or directly merge the checks with the local validation code, and pass in another variable to signify what token type we wish to verify (access token vs id token).
Since client mode does not use the access tokens, we should be able to ignore them.

So based on the token type, we change what is checked?
That sounds reasonable as long as it doesn't confuse us in the future.
Does this seem doable for 0.5.0?

So based on the token type, we change what is checked?

Yes.

Should we keep this in 0.5.0 or bump it to the next milestone?

We can keep this in 0.5.0.

Some OpenID Providers may not fully follow the OpenID specification regarding ID Token secrets. See keycloak/keycloak#13823.
We will provide an option, non_standard_id_token_secret, that allows for specifying the symmetric key secret for an ID Token.