I'm trying to install in rails + angularjs

I'm trying to install md-date-time, but there are two things that I don't understand. My project does not use angular-material, use materialcss, I'm using in a rails project. There are two things that I don't understand:

Where put the shim of the md-button?.
Step 2 of usage. Where add the dependency to mdDateTime?
Sorry but i'm noob.
Thanks

There are a handful of different ways you can use angularJS with rails, but both the dependency and shim can be added where you define your angularJS app:
angular.module('yourApp', [
'mdDateTime'
])

.directive('mdButton', [function(){
    return {
        replace: true,
        restricte: 'E',
        transclude: true,
        template: '<button class="btn btn-link md-button" ng-transclude></button>'
    };
}]);

Alternatively you can add the directive to a separate file:
angular.module('yourApp')
  .directive('mdButton', [function(){
    return {
        replace: true,
        restricte: 'E',
        transclude: true,
        template: '<button class="btn btn-link md-button" ng-transclude></button>'
    };
}]);
``````
This will keep things organized as your project grows, just make sure to include it in your application.js or index.html file!


Clicking Override All in the Theme Editor with no Type selected crashes the editor

Godot version
4.1 beta 3
System information
Godot v4.1.beta3 - Windows 10.0.19045 - Vulkan (Compatibility) - NVIDIA GeForce GTX 1060 6GB (NVIDIA; <IP_ADDRESS>24) - AMD Ryzen 7 2700X Eight-Core Processor (16 Threads)
Issue description
This is a pretty specific reproduction but one that can accidentally happen (as happened to me). Clicking Override All on Theme Editor with no category selected will cause the editor to crash. Opening the project back up shows no Types in the dropdown (despite the theme resource looking fine upon cursory inspection) and attempting to add any new types will crash the editor again. Also important to note that in the first project I had this on the theme was applied as the default theme to my project and it's still visible on elements in the project (fonts and panels and such), but attempting to edit the theme will crash the editor. This is actually because the theme editor is showing my theme but does not have a reference to it somehow. Selecting the theme again fixes this
This shouldn't normally happen under normal operation but it is still something that does, as it requires the Override All button to be available but no Type to be selected - as far as I know, the only condition which causes this is if you open up a project after having last edited a Theme. Then the button should be available, but no Type is selected by default.
Steps to reproduce

Create a new Theme
Add some Type, doesn't matter what
Close project
Reopen Project, Theme editor should be open
Click Override All
Enjoy

Minimal reproduction project
N/A

This may be the same as https://github.com/godotengine/godot/issues/78281, which should be fixed in 4.1 RC1. Though the case that you describe in the reproduction steps may still be a problem.

👀 lookin forward to that RC1 - though I also wanted to correct myself and say that the theme isn't corrupted, I'm just a silly goose who did not realize that the currently edited theme was not actually pointing to anything, and the subsequent crashes were solved by actually selecting the theme. I'll try building from the latest master to see what comes up

Looks like [2930fbeec] will simply not open the Theme Editor by default when reopening a project, which solves that issue. The bottom tray is still expanded but the only way I can tell to be able to edit any theme is to have one selected, so I think we can call this closed!

Thanks, confirming as duplicate of https://github.com/godotengine/godot/issues/78281.


Add E2E tests to all endpoints

This includes:

/drip
/drip/bot
/drip/web

Hey @mordamax & @rzadp I just realized I don't really know if this can be achieved.
We need a valid captcha which can not be obtained without some kind of UI automation.
There is some tools to do automated testing but I think that's intended for UI testing, not API testing.

In https://github.com/paritytech/substrate-matrix-faucet/pull/219 I have an E2E test which calls the drip/web endpoint, and the backend uses a testing captcha secret which makes any captcha token valid.
Maybe it will be enough for an E2E smoke test.

In #219 I have an E2E test which calls the drip/web endpoint, and the backend uses a testing captcha secret which makes any captcha token valid.
Maybe it will be enough for an E2E smoke test.

Oh amazing. Will use that knowledge then. Thank you!

the api also has programmatic way to get a token, so may be that could be alternative option with "special" testing keys
https://web.archive.org/web/20230127150134/https://developers.google.com/recaptcha/docs/invisible
grecaptcha.execute('key', {action: 'homepage'}).then((token) => {
               ...
});


Fix bug: requests with a 'file' argument work now

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've been mindful about doing atomic commits, adding documentation to my changes, not refactoring too much.
[x] I've a descriptive title and added any useful information for the reviewer. Where appropriate, I've attached a screenshot and/or screencast (gif preferrably).
[x] I've written tests to cover the new code and functionality included in this PR.
[x] I've read, agree to, and signed the Contributor License Agreement (CLA).

PR Summary

e.g. Fix bug for requests with a 'file' argument. The current code assumes 'file' is always a file's content (upload), whereas there are many requests that take a fileID with the argument name 'file'.

Related Issues

N/A (that I have found)

Test strategy

e.g. Test files.info in addition to the files.upload test that already exists.

Coverage increased (+0.06%) to 61.328% when pulling ea6f490785d42a17ffd391385d2f72757699ad59 on artoonie:bugfix/file-info into 7dfa7f3fdacdd5f935611147830e03cb32b9ffb8 on slackhq:master.

Coverage increased (+0.06%) to 61.328% when pulling ea6f490785d42a17ffd391385d2f72757699ad59 on artoonie:bugfix/file-info into 7dfa7f3fdacdd5f935611147830e03cb32b9ffb8 on slackhq:master.

Not sure why coveralls is failing on the py35 build...is this normal? Locally, I get the same issue on master.

This looks good. Thanks!
I just retriggered the test run and it passed (I don't know why that happens, but it does sometimes).


Add SheSharp to the repository list

📢 Only PRs from existing Virtual Coffee members will be accepted.

Type of Contribution
Check the item(s) that applies.

[ ] 🌱 Practice Open Source
[x] 📃 Repositories List

If your type of contribution is Repositories List, check items in the repository checklist below that apply.
⚠ We only accept PRs where most of the items are checked

[x] Relevant name
[x] Description
[x] Relevant tags
[x] README.md file
[x] CONTRIBUTIONS.md/CONTRIBUTING.md file
[x] Open source license
[x] Code of Conduct (COC)
[x] Issue and Pull Request templates

Description
This PR adds SheSharp's Knowledge Hub to the Repositories List.

@dominicduffin1 this PR is ready foor review! ;)
Thank you :D

Thanks, @dominicduffin1!


Semantic release support

Add semantic release support to patternfly-next to enable automatic releases

Semantic release support added - 1.0.0 release automatically handled and requires merge to release branch manually now (until we want to enable it on master, probably for beta1)


Can't remove RKE2 cluster if Harvester Cloud Credential is deleted

Rancher Server Setup

Rancher version: v2.6.3
Installation option (Docker install/Helm Chart): Docker

Information about the Cluster

Kubernetes version: v1.21.8+rke2r2
Cluster Type (Local/Downstream): Harvester

User Information

What is the role of the user logged in? Admin

Describe the bug
Users can remove cloud credentials if there is an RKE2 cluster based on it. After that, we will get an error message in the machine-provision-remove handler.
Also, if users only delete the Harvester cluster CR but remain the cloud credential, we can remove the RKE2 cluster without error. However, associated VMs in the Harvester cluster can't be removed together.
To Reproduce

Import Harvester Cluster.
Create a Cloud Credential.
Create an RKE2 cluster based on Harvester.
Remove the Harvester cloud credential.
Remove the RKE2 cluster.

Result
RKE2 cluster status is stuck in Removing and there is an error message in the machine-provision-remove handler.
2022/01/27 03:15:33 [ERROR] error syncing 'fleet-default/rke2-in-aws-pool1-7c09ec84-8jv8n': handler machine-provision-remove: secrets "cc-k4t8n" not found, requeuing

Expected Result

Prevent users from deleting Cloud Credentials or Harvester cluster CR if there is an RKE2 cluster based on it.
Remove cluster-agent in Harvester if users remove Harvester cluster CR in Rancher.

Related issue
https://github.com/harvester/harvester/issues/1799

@thedadams, I try this case for AWS. There are also error logs like the following.
[ERROR] error syncing 'fleet-default/rke2-in-aws-pool1-7c09ec84-msbq6': handler machine-provision-remove: secrets "cc-dbsgw" not found, requeuing

Reproduce steps:

Create AWS credential.
Create an RKE2 cluster on AWS.
After the provisioning job is done, delete the credential.
Remove the RKE2 cluster.

https://user-images.githubusercontent.com/4927361/154038942-85d70c67-d88f-4f94-a80c-78518c1cbf87.mov

Root cause
The root cause was that cloud credentials could be deleted even when there was a provisioning cluster that was using the cloud credential.
What was fixed, or what changes have occurred
Now, a user cannot delete a cloud credential if it is being used by a provisioning cluster.
Areas or cases that should be tested

Create a cloud credential
Provision an RKE2/K3S cluster
Try to delete the cloud credential. An error should be returned.
Delete the RKE2/K3S cluster
Try to delete the cloud credential. This should be successful now.

What areas could experience regressions?
Deleting cloud credentials
Are the repro steps accurate/minimal?
Yes

Tested in Harvester master-bfd6a8a9-head and Rancher 948355c. You can still delete the cloud credential without an error when there is a cluster created with it.
Verification steps

Import Harvester into Rancher
Create a cloud credential for Harvester
Create a new RKE2 cluster
Delete the cloud credential being used for the RKE2 cluster

If you go into the cluster config now you get an error that the secret is missing

@noahgildersleeve how are you deleting the cloud credential? Through the UI?

I'm re-testing this with v2.6.4-rc4 because my image for the deploy might have been old.

Tested in rancher v2.6.4-rc4 with Harvester master-bfd6a8a9-head. Verified as fixed with previous steps
Verification steps

Import Harvester into Rancher
Create a cloud credential for Harvester
Create a new RKE2 cluster
Delete the cloud credential being used for the RKE2 cluster
Validate that you get an error


dependencies clean up

replace akka with MIT-library
remove JAXB
replace Logback with java.util.logging

all good. theoretically netty can be removed but profit is questionable.


Revert "Redirect 404 to search page"

Reverts statsig-io/docs#377

🚀 Deployed on https://62eb0b79e4ebee10b38ad9d1--cozy-fox-0defec.netlify.app


Immunity improvements

Description of change
Immunity improvements:

Fix heading tags
Add missing word Review
Consistency between routes: The pattern we follow is that usually on the assessment we have only the route and on the review we are adding the review_ACTION, thus I've renamed it and thought of adding assess_ACTION just to have a better division of routes
Rearrange text

@cdccollins Yeah I totally agree, is something we can definitely improve most probably once we finish the reviewer journey 😄


Classic UI Limited Parsys only works if URL contains "/content"

The "if" statement here is not always valid...
https://github.com/Adobe-Consulting-Services/acs-aem-commons/blob/master/content/src/main/content/jcr_root/apps/acs-commons/widgets/source/js/classicui-limit-parsys.js#L30
if( ( pathName !== "/cf" ) && ( pathName.indexOf("/content") !== 0)){         return;     }
Many customers use Sling Mapping to avoid having "/content/" in their URLs.  With this logic, the Limited Parsys would only work if the user is using Content Finder or if the URL contains "/content".

Also, it would be nice to change the wording of the alert to remove the dash.
Instead of:
"Limit exceeded, allowed - 3"
It would be better to say:
"Limit exceeded, allowed: 3"
The first could be interpreted as "-3" which does not make sense.

I changed the error message, but I'm not sure how to address the larger concern. Do people really implement those mapping rules on author? I usually see that only on publish.

Hi Justin.  We do hide "/content" on Author.  I would like to offer this suggest to replace the current "if" statement...
var pathName = "";
if (CQ.WCM && CQ.WCM.getPagePath){
pathName = CQ.WCM.getPagePath();
}else{
pathName = window.location.pathname;
}

Hi Justin. We do hide "/content" on Author. I would like to offer the following changes. Please see file attached.
A couple of highlights.

Instead of getting path from "window.location.pathname" use "CQ.WCM.getPagePath()"

The value of "CQ.WCM.getPagePath()" is not set correctly until much later in the page load.  So, instead of executing the Limited Parsys code on page load, execute it on CQ.Ext.onReady

Preventing items from being dropped is good but can easily be worked around by double-clicking the drop zone and selecting a component.  So, completely hide the drop zone if the max-limit is reached.  I could not find a perfect approach so I came up with somewhat of a hack.  Maybe you'll have a better approach.  What I did was look at the resource type of each editComponent and if it is a "new" component I hide that edit
classicui-limit-parsys.txt
Component.

if(dropTargets[0].editComponent.params["./sling:resourceType"]==="foundation/components/parsys/new"){

@chrsbrq do you still need this? Im w @justinedelson in that i havent seen this mapping on AEM Author.
/cc @schoudry

Deprecated feature (ClassicUI)


Make services icons highlight to green (or make them green and stay highlight to blue) ?

made the change - was very tiny:
You can easily change background color in style.css by going here and changing "color":
.service > .fa-stack > .fa-circle {
  color: #CADC56;
  transition: .3s;
}

and here to change icon  color by going here and changing "color":
.service {
  margin-top: 30px;
  padding: 45px 20px 20px 20px;
  border: 1px solid #ececec;
  border-radius: 4px;
  text-align: center;
  transition: .3s;
  position: relative;
  color: #fff;
}


Rename symbols for iOS compatibility

Apple forbids some symbols that Numpy is linked against (ccopy, dcopy, scopy, zcopy and xerbla). Compilation and tests work fine but the App Store complains and prevents uploading such builds.
This patch aliases the culprits to the public, allowed method names, allowing the linking to proceed as before and the resulting binary to be accepted by the App Store.

+1 on merging this change. Just ran into this problem myself.


Option to trigger copy setting action via keyboard

Testing #16530

While using Enter to navigate the filtered settings, there is no way to trigger the "copy" action using just keyboard. Can we use arrow keys to navigate instead and use Enter to trigger the "copy" action?

Marking this as a dup of #35581 since they are related


pub run error when $dart2js is in pubspec.yaml

This issue was originally filed by<EMAIL_ADDRESS>
What steps will reproduce the problem?
$ mkdir myapp
$ cd myapp/
$ stagehand consoleapp
$ pub get
Add the following to pubspec.yaml:
transformers:

$dart2js

$ pub run main
What is the expected output? What do you see instead?
Expected:
Hello world: 42
Actual:
'file:///tmp/pub_iklQDe/runInIsolate.dart': error: line 2 pos 17: string interpolation not allowed in this context
import 'package:$dart2js/$dart2js.dart';
                ^
dart:isolate-patch/isolate_patch.dart 286                                                                       Isolate.spawnUri
/mnt/data/b/build/slave/dart-editor-linux-dev/build/dart/sdk/lib/_internal/pub_generated/lib/src/dart.dart 311  _isolateBuffer
dart:isolate-patch/isolate_patch.dart 242                                                                       _startIsolate
What version of the product are you using?
Pub 1.8.0-dev.2.0
Dart VM version: 1.8.0-dev.2.0 (Wed Oct 29 00:34:12 2014) on "linux_ia32"
On what operating system?
Ubuntu 14.04 LTS

This issue has been moved to dart-lang/pub#1185.


Stateless in Peril

Fixes #457
Pretty well fleshed out, worthy of review now anyway

@ashfurrow any guesses on this?
Configuration file: /home/travis/build/artsy/artsy.github.io/_config.yml
            Source: /home/travis/build/artsy/artsy.github.io
       Destination: _gh-pages
      Generating...
              Lunr: Creating search index...
jekyll 2.5.3 | Error:  Max nesting depth of 256 exceeded
rake aborted!
Command failed with status (1): [PRODUCTION="YES" jekyll build --destinatio...]
/home/travis/build/artsy/artsy.github.io/Rakefile:12:in `block in <top (required)>'

@orta not sure, I'll take a look.

Nicceee, we now get PR previews - https://deploy-preview-460--agitated-allen-f5b4e7.netlify.com/blog/2018/06/18/on-obsessive-statelessness/

Pretty cool! I really like the CloudyConway background.

OK, this is happening 🚶 🚶♀


Expose ClassDB::is_enum_bitfield as ClassDB::is_class_enum_bitfield

Currently, there isn't a way for plug-in authors to resolve whether a given class-defined enum is a bitfield.  Having this knowledge exposed allows plug-in authors to render UIs and handle this specific case differently from normal enum types.

Thanks!


Authenticate HTTP requests and WebSocket connections

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

69 of 80 (86.25%)  changed or added relevant lines in 9 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.7%) to 95.286%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/GraphQL.AspNetCore3/GraphQLHttpMiddlewareOptions.cs
3
4
75.0%

src/GraphQL.AspNetCore3/WebSockets/WebSocketConnection.cs
2
3
66.67%

src/GraphQL.AspNetCore3/GraphQLHttpMiddleware.cs
38
40
95.0%

src/GraphQL.AspNetCore3/WebSockets/WebSocketHandler.cs
10
13
76.92%

src/GraphQL.AspNetCore3/WebSockets/BaseSubscriptionServer.cs
5
9
55.56%

Totals

Change from base Build<PHONE_NUMBER>:
-0.7%

Covered Lines:
1533

Relevant Lines:
1577

💛  - Coveralls


Once I run torch code on a multi-gpu machine. It slows down over 100% !

@ngimel
Hi, ngimel.
I am faced with a very big performance problem in torch code and need help from you.
My code can only run on a single GPU.
When the I run my torch code on a machine which has single GPU, the speed is 0.5s per image. However,
when try running it on a machine which has multiple GPUs, it slows down to 1.2s per image.
More than 100% speed decrease！ I find that other gpus all have ~300Mb memory usages occpied by
the same thread. I think whether it is the bottleneck of the decrease. How to solve it?

Update:
'export CUDA_DEVICES_VISIBLE=5' makes no differences.


MsgGrant fix

Hello,
You have a bad implementation for MsgGrant  (https://github.com/InjectiveLabs/injective-ts/blob/master/packages/sdk-ts/src/core/modules/authz/msgs/MsgGrant.ts)
Change:
authorization.setValue(genericAuthorization.getMsg());
To
authorization.setValue(Buffer.from(genericAuthorization.serializeBinary()).toString('base64'));
Also, to solve this problem, you need to fix the implementation in your Fee Delegation Service
To fix it I wrote the patch for myself.

Why don't you submit a PR to resolve the issue?


Add the flag of Norway next to EU and Switzerland for Transnational Exposure Logging

Avoid duplicates

[x] This enhancement request has not already been raised before
[ ] Enhancement request is specific for iOS only, for general issues / questions that apply to iOS and Android please raise them in CWA-Wishlist
[ ] If you are proposing a new feature, please do so in CWA-Wishlist

Current Implementation

In one of the recent updates, Switzerland was added to the list of countries supported by transnational exposure logging and the flag was added next to the EU flag where the latter is displayed. However, the list of countries also contains Norway, which, like Switzerland, is not part of the EU.

Suggested Enhancement

Add the flag of Norway next to those of the EU and Switzerland
Expected Benefits

This is not a technical improvement, just a more accurate representation of the facts.
Also, I didn't check if this also affects the Android app. If that is the case, please transfer this issue to a more appropriate place like the wishlist.

This has already been declined once, please see https://github.com/corona-warn-app/cwa-documentation/issues/566

Thanks for the link, I didn't search the documentation repo


Insert selection in admonition [Feature request]

I use the Insert Admonition command a lot, and I've noticed that many times, I want to insert text that I've already written in the note into a new admonition rather than create the admonition and then type what I want in it (or copy and paste text I've already written into the new admonition).
In other words, I want to be able to select some text in the note, use the Insert Admonition command and have the selected text inside the new admonition. Currently, if I do this, it just overwrites what I've selected.

Hi, on 6.3.1 the insert command will insert around a selection.


Evolution custom name (text as optional part)

Resolve #209

@igorkamyshev
Good idea. Actually, I would add:

Exceptions (do not proceed with execution) for cases of any conflict of numbering is found (existing any of 2: 1-abba.sql, 1-baab.sql, 1.sql, 1-.sql)
Warning (with promt "Files ... will be ignored because of incorrect name format. Proceed?" in case if there are "not in scope" files (e.g. test.sql, test.txt, *1-incorrect.sql or -1-worng.sql).
Let me know what do you think.

Will add. Meanwhile, any plans to check test coverage?


Added basic (throw) support for anonymous/dynamic types.

Closes #388

:white_check_mark: Build ExtendedXmlSerializer <IP_ADDRESS>-eqdkiuul completed (commit https://github.com/ExtendedXmlSerializer/home/commit/66d8d99e71 by @Mike-E-wins)
Install it by running the following command in Package Manager Console:
Install-Package ExtendedXmlSerializer -Version <IP_ADDRESS>-eqdkiuul -Source https://ci.appveyor.com/nuget/extendedxmlserializer-preview


How to open drawer onPress in v4?

Actions.drawerOpen
On Sun, Jul 30, 2017 at 9:51 PM, harmeetv<EMAIL_ADDRESS>wrote:

Version
Tell us which versions you are using:

react-native-router-flux v4.?.? (v3 is not supported)
react-native v0.?.?

Expected behaviour Actual behaviour Steps to reproduce
For non-obvious bugs, please fork this component, modify Example project
to reproduce your issue and include link here.
1.
2.
3.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aksonov/react-native-router-flux/issues/2143, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABQpcYdPSnaXl0be4oKTCDYa-8NBoYBWks5sTN6ugaJpZM4Onup9
.


Cycle Private Keys

Your private key leaked:
https://github.com/covid-be-app/cwa-server/blob/54b6e17f018b1fcdb10b1da3aba1cf572623fe5b/services/distribution/src/test/resources/application.yaml#L17

this is a key used during unit testing (test/resources) ... not used in an actual deployment / environment.

Allright, a non-issue thus.


Selecting the blockchain storage directory

Hi
I would like to be able to change the blockchain directory. Perhaps through some parameter in zen.conf or by selecting a directory during the installation process.

Accepted as an enhancement request.

@zennermac is this not already integrated through the datadir flag passed to zend?

Swing is in pure maintenance mode, no new features will be implemented anymore. Only critical bug fixes and compatibility releases for zend deprecations are being done.


feat: Response view switcher for all action editors

Response view switcher for actions, currently supported formats are Table, Json and Raw.

Fixes #3630

Please delete options that are not relevant.

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

Manual tests employed.

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

Test coverage results :test_tube:
:red_circle: Total coverage has decreased
// Code coverage diff between base branch:release and head branch: feat/response-view-switcher

Status
File
% Stmts
% Branch
% Funcs
% Lines

:red_circle:
total
56.3 (-0.05)
37.71 (-0.09)
35.89 (-0.07)
56.52 (-0.05)

:red_circle:
app/client/src/actions/pluginActionActions.ts
67.62 (-0.03)
0 (0)
6.06 (-0.19)
54.41 (-0.14)

:red_circle:
app/client/src/components/ads/MultiSwitch.tsx
44.44 (-2.23)
57.14 (-2.86)
0 (0)
44.44 (-2.23)

:red_circle:
app/client/src/components/ads/Tabs.tsx
22.41 (-3.08)
12.12 (-4.55)
0 (0)
23.21 (-3.32)

:red_circle:
app/client/src/components/editorComponents/ApiResponseView.tsx
49.04 (-0.96)
29.13 (-4.2)
0 (0)
50.51 (-1.21)

:red_circle:
app/client/src/components/editorComponents/EntityBottomTabs.tsx
27.59 (-0.98)
0 (0)
0 (0)
30.77 (-1.23)

:red_circle:
app/client/src/components/editorComponents/CodeEditor/styledComponents.ts
88.14 (0)
65.17 (-4.71)
100 (0)
89.47 (0)

:red_circle:
app/client/src/components/editorComponents/form/fields/EmbeddedDatasourcePathField.tsx
28.89 (0)
2.76 (-0.08)
3.23 (0)
30.72 (0)

:red_circle:
app/client/src/pages/Editor/APIEditor/Form.tsx
35.98 (-1.38)
36.21 (-1.29)
0 (0)
37.57 (-1.51)

:red_circle:
app/client/src/pages/Editor/APIEditor/RapidApiEditorForm.tsx
55.56 (-7.48)
41.86 (-4.29)
0 (0)
55.56 (-7.48)

:red_circle:
app/client/src/pages/Editor/QueryEditor/EditorJSONtoForm.tsx
34.55 (-1.26)
25.77 (-2.32)
0 (0)
35.85 (-1.35)

:red_circle:
app/client/src/pages/Editor/QueryEditor/Form.tsx
33.33 (-14.04)
0 (0)
0 (0)
33.33 (-14.04)

:red_circle:
app/client/src/pages/Editor/QueryEditor/index.tsx
27.61 (-1.07)
5.8 (0)
5.26 (-0.3)
29.27 (-1.24)

:red_circle:
app/client/src/reducers/entityReducers/actionsReducer.tsx
6.67 (-0.23)
0 (0)
1.69 (-0.06)
6.84 (-0.24)

:red_circle:
app/client/src/sagas/ActionExecution/PluginActionSaga.ts
17.33 (-0.36)
2.11 (-0.13)
9.52 (0)
19.92 (-0.32)

:red_circle:
app/client/src/selectors/entitiesSelector.ts
48.85 (-0.1)
13.25 (-0.17)
25.32 (-0.32)
45.21 (-0.11)

/ok-to-test sha=2bf492c

/ok-to-test sha=87352ed

/ok-to-test sha=e94ad94

/ok-to-test sha=f6930f5

/ok-to-test sha=f3344dc

/ok-to-test sha=c94a481

/ok-to-test sha=cfad173


Parsing regression (?)

https://github.com/zzazzdzz/fools2018 compiles correctly with RGBDS 0.3.3, but errors out with 0.3.7:
ERROR: main.asm(7) -> main_sram2.asm(6) -> engine/text_engine.asm(369) -> includes/charset.asm(499):
    syntax error
error: Assembly aborted in pass 1 (1 errors)!

(Although the setup surrounding RGBDS is kinda hacky, this invocation should be completely vanilla.)

Am I right in saying that that error corresponds to these lines?
496     db $ff,$05
497
498 Char38: ; "
499     db $00,$01,$02
450     db $20,$21,$22

It could be getting confused by the " in the comment on the line before, though I've no idea what parsing changes could've introduced this.

Would it be easy for you to find out which rgbds version is the first that fails to compile?

It's probably because of https://github.com/rednex/rgbds/commit/efdd42c6a80945c78b396054ebfdbd4fd9f1f1a6

@Ben10do Well, I tried a minimal repro with the same intuition, and it didn't work either. Not sure what it could be besides the quote, though?
Also, I could try compiling it with other versions, if needed.

Have you tried git bisect to see where the problem started?

Well, since it's not my repo, I didn't try that :stuck_out_tongue:

Ok, I thought that maybe you had done more tests than just checking 0.3.3 and 0.3.7. Do you have a minimal example that triggers the bug? @Ben10do, could you post what you used?

@AntonioND This is the quick-and-dirty test that I tried, but it compiles without error:
SECTION "x", ROM0

Test: ; "
    db 0, $1F

That is what I tried as well. I then wondered if it could be caused by an INCLUDE or smth, but I couldn't compile the file alone (missing macros, IIRC). I didn't look any further, though.

Bumping this - @DevEd2 got a similar error with RGBDS 0.3.7, declared on the line after a ld [ScrollPos+1], a ; ". Apparently this is an actual bug, but why can't it be tripped with a minimal example?

This appears to reproduce the bug (also gives a spurious "expression must be 8-bit"):
test:
	db "hello, world"
	db 123
test2:
	db "test" ; "
	db "test2"
.test3
	db	"asdasdasd" ;"
	dw	456```

Maybe the double quotes are, not the label?

As noted above by both I and Ben10do, the double quotes are not enough to trigger the error. Removing the last two lines seems to make the thing compile fine?

I can't reproduce the error with that example (rgbasm -o bug.o bug.asm).

Huh, same. @DevEd2?

I can't reproduce this with the case linked in the first message.
@ISSOtm Are you compiling rgbds yourself or using a prebuilt version?

I'm using it on Linux, thus I have to build it myself

I'm also testing it on Linux, but I don't get the error when I do the following.
cd fools2018/savefile
rgbasm -o main.o main.asm

It assembles without any errors. Is there anything special you have to do?

rgbasm v0.3.7-34-g5cb6c4a

Well, maybe it has been fixed since? I'm using rgbasm v0.3.7.

Might be related to #326

I agree that this was caused by #326. The confusion over reproducing this makes sense given that it only happens when you use Windows line endings. It should be fixed now that #370 has been merged.

The following does not generate an error, I'll assume this is fixed then.
issotm@link-panda tmp.jXWovMS7IM% printf 'SECTION "test",ROM0\r\ntest:\r\n\tdb 1 ; "\r\n.test2\r\n\tdb 2 ; "' > test.asm
issotm@link-panda tmp.jXWovMS7IM% cat test.asm
SECTION "test",ROM0
test:
	db 1 ; "
.test2
	db 2 ; "
issotm@link-panda tmp.jXWovMS7IM% rgbasm test.asm
issotm@link-panda tmp.jXWovMS7IM%


Add license

Cc: @SreeArikatla

@jcfr Thanks, this project is indeed open-source and should be Apache 2.


Can't add "msg" metadata to transaction

Describe the bug
I'm trying to add some metadata to an ADA transaction, just like this:
metadata = {674: {'msg': ['some text here']}}
builder = TransactionBuilder(chain_context)

builder.auxiliary_data = AuxiliaryData(
    data=AlonzoMetadata(metadata=Metadata(metadata))
)
# also tried like this:
# builder.auxiliary_data = AuxiliaryData(data=Metadata(metadata))

# (...)

tx_body = builder.build(change_address=change_address)
tx = Transaction(tx_body, TransactionWitnessSet())

Logs
When I try to sign the tx using the nami web wallet, this is the error I'm getting on their log:
code: 2
info: "Wallet could not send the tx."
message: "transaction submit error ShelleyTxValidationError ShelleyBasedEraBabbage (ApplyTxError [UtxowFailure (FromAlonzoUtxowFail (WrappedShelleyEraFailure (MissingTxMetadata (AuxiliaryDataHash {unsafeAuxiliaryDataHash = SafeHash \"ef64ed78c9efa5555caaec0f3987bc03b71de2890763b3a55108f19fd4d191a9\"}))))])"

Figured it out! Just needed to add this:
# (...)
tx = Transaction(tx_body, TransactionWitnessSet(), auxiliary_data=auxiliary_data)

Closing this issue, thx.


fix integration matrix exclude

exclude was not correctly handling arrays, so it has been updated to explicitly define all excluded combinations
list of jobs that will be executed now (courtesy of act):

[blockchain-provider:besu database-type:postgres test-suite:TestEthereumMultipartyE2ESuite]
[blockchain-provider:geth database-type:sqlite3 test-suite:TestEthereumMultipartyE2ESuite]
[blockchain-provider:geth database-type:postgres test-suite:TestEthereumMultipartyE2ESuite]
[blockchain-provider:geth database-type:sqlite3 test-suite:TestEthereumGatewayE2ESuite]
[blockchain-provider:geth database-type:postgres test-suite:TestEthereumGatewayE2ESuite]
[blockchain-provider:besu database-type:sqlite3 test-suite:TestEthereumGatewayE2ESuite]
[blockchain-provider:besu database-type:postgres test-suite:TestEthereumGatewayE2ESuite]
[bockchain-provider:geth database-type:sqlite3 test-suite:TestEthereumMultipartyTokensRemoteNameE2ESuite]
[blockchain-provider:geth database-type:postgres test-suite:TestEthereumMultipartyTokensRemoteNameE2ESuite]
[blockchain-provider:besu database-type:sqlite3 test-suite:TestEthereumMultipartyTokensRemoteNameE2ESuite]
[blockchain-provider:besu database-type:postgres test-suite:TestEthereumMultipartyTokensRemoteNameE2ESuite]
[blockchain-provider:fabric database-type:sqlite3 test-suite:TestFabricGatewayE2ESuite]
[blockchain-provider:fabric database-type:postgres test-suite:TestFabricGatewayE2ESuite]
[blockchain-provider:fabric database-type:sqlite3 test-suite:TestFabricMultipartyE2ESuite]
[blockchain-provider:fabric database-type:postgres test-suite:TestFabricMultipartyE2ESuite]

Codecov Report

Merging #973 (c9aba4b) into main (f0392bb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #973   +/-   ##
=======================================
  Coverage   99.99%   99.99%
=======================================
  Files         299      299
  Lines       19498    19498
=======================================
  Hits        19497    19497
  Misses          1        1

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


The WindowManager is blocking any text input and even Back button on the background Acitivty.

To fix this, please change in 'LGSnackbarPresenter#createDefaultLayoutParams()'  line 117
layoutParams.flags = WindowManager.LayoutParams.FLAG_NOT_TOUCH_MODAL | WindowManager.LayoutParams.FLAG_NOT_FOCUSABLE;

Fixed here: https://github.com/loregr/LGSnackbar/commit/af3516bfd246fa921f46f44b0c9b23396bf71111
Thanks


fix webhook request

right now it's commented and the stub is returned instead
https://github.com/cappuccinotm/dastracker/blob/95e313cecd88c478ca497dc65dd555197e90da0a/app/tracker/github.go#L122-L158

seems like fixed


Update for generator resume arguments

Blocked on a new nightly being released with the required tooling.
Still needs all the comments updated to match the new implementation.

Moved to #28 cause of github issues.


WIP: Get the bones in place (flesh comes later)

@cg @bdero @pdpinch - I'm trying to get the basic pieces in place so I can begin work.  I'd like to write the tests before the implementations, but I need help in laying out the code.  I consider myself a relative Python newbie so I'll accept all suggestions and criticisms.

Mostly a question, but I'm wondering how we want to organize everything.  I would think we would probably want (if this makes sense), separate classes for each of the API's that inherit the common stuff.  i.e. a base class that handles auth and making the actual http requests, and then a Gradebook class, a Membership class and so on.  If that doesn't make sense with the backing API, then just the monolithic client is fine, but with all the methods already defined here, I think we probably want to break it up so it is easier to work on in bite sized chunks.

I like what @carsongee is saying above about having a base class. Some of the methods in the existing StellarGradeBook class are good candidates as methods in the base class (most of init, rest_action, get, post, delete).
We can do that in the next version though. I suggest the initial goal here should be to simply move the existing StellarGradeBook class to a separate package.  That will make a great 0.1 version.

Can you push your changes @pwilkins, so we can review your fixes?  Thanks.
Also, I think if we want to split out classes, which would change the current API, we should do that now since that will break anything using this lib. It shouldn't be difficult to do.  While we aren't claiming a stable API, it is generally best to keep as stable as we can.


Save - JSON output does not the same order with the form build.

I saw the output of JSON on save event does not the same order with the form i build. Ex, i build the form with: 1. Checkbox, 2. TextBox, 3. Dropdown, but the output of json is not that order. Could you please help to resolve this one? Thanks.

I also noticed this anomaly!

same here...

could you please solve same


Don't place original .ts files in dist

Is there a reason you're including the source .ts files in dist? This seriously confuses Webpack, as it'll try to include and build the .ts files rather than just using the .js files.
Edit: I fixed it by telling Webpack to prefer the js files (resolve.extensions: [".js", ".ts"]), but I still find this an odd decision.

I think it was to get source mapping to work in jspm. Probably it would be easy to rearrange; I expect I was just exasperated at the time trying to get the typescript compiler to do what I wanted in any way shape or form.
Thanks for posting the workaround, since I don't use webpack. Yet.

Ahh, I guess that makes sense.
Now that the Aurelia webpack plugin is barreling towards RTM, I'd strongly recommend it! 100% simpler to use and configure than gulp/jspm setups.

what do you use to get set up with webpack? just tried using the webpack skeletons, and they don't seem to do dynamic requires right. That or I haven't figured out the right place to configure them..

The short story is there isn't "dynamic" requires now. You have to declare dependencies statically using PLATFORM.moduleName("<dependencyName>") or by using the ModuleDependencies webpack plugin. See https://github.com/jods4/aurelia-webpack-build/wiki/Managing-dependencies

@ariovistus I actually like having the source included with the distribution. That lets me consume this directly from source if I wish. I did notice one issue.

We are having the same issue, webpack in combination with the aurelia-webpack-plugin uses the typescript files from the dist folder instead of the already transpiled javascript files. I guess this can go wrong if the typescript settings from this project differs a lot from the typescript settings in a consuming project.
(Actually it does go wrong, fortunately this PR https://github.com/ariovistus/aurelia-mask/pull/13 fixes this issue for now for us)
Would it be an option to put the ts files into an src folder or similar in the dist map for people that likes to have the typescript files around?
PS: Thanks for this cool project, once it works, it works really cool!

I honestly have no idea how to configure source map generation. It looks like recent versions of typescript have an option to inline the source in the map. It seems to work fine for requirejs cli projects, have yet to try it with webpack and jspm.
doesn't fix #10 though, grrr

@ariovistus If you don't figure it out, I can take a swing at it. Wouldn't be my first time fighting with ts sourcemaps :)

heads up to anyone who's interested: I have a build packaging without source files and with inline source maps that I am going to be pushing to npm probably tomorrow unless things aren't as well baked as I remember. If you depend on the sources being there, you should still be able to link directly to the github repo, but if you expect them to be in dist you should change that to src, since this is mostly me wanting to get build artifacts out of source control. Not sure if I'll actually remove them immediately, but they will stop being updated.

@ariovistus thanks for the heads up but I am disappointed by this decision.

there is really no good option here. source maps are just busted in requirejs cli. webpack cli ignores them in the default configuration,
with devtool: 'eval' it uses my distributed js files as source and ignores my source maps. Don't know if there's a plugin that does handle input source maps

Why not just inline the source maps? Uglify will remove them anyway

@ariovistus Thank you, this seems to be the correct way!


Permission +x and fix qt-5.9 archive address

Run of cd depends; make gives error 404
  curl: (22) The requested URL returned error: 404
  Fetching qtbase-opensource-src-5.9.6.tar.xz from https://download.qt.io/archive/qt/5.9/5.9.6/submodules
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0   196    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  curl: (22) The requested URL returned error: 404
  Fetching qtbase-opensource-src-5.9.6.tar.xz from https://bitcoincore.org/depends-sources
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0  264k    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  curl: (22) The requested URL returned error: 404
  make: *** [funcs.mk:247: /maxeter-<IP_ADDRESS>/depends/sources/download-stamps/.stamp_fetched-qt-qtbase-opensource-src-5.9.6.tar.xz.hash] Error 22

So my commit fiz qt-5.9 archive address. And also change permissions to autogen.sh and depends/config.{guesss,sub} for build on linux.

Also had this problem on Linux 6, gcc 12:
  In file included from ./script/script.h:11:
  ./serialize.h: In instantiation of ‘constexpr CheckVarIntMode<Mode, I>::CheckVarIntMode() [with VarIntMode Mode = VarIntMode::NONNEGATIVE_SIGNED; I = unsigned int]’:
  ./serialize.h:393:5:   required from ‘void WriteVarInt(Stream&, I) [with Stream = CAutoFile; VarIntMode Mode = VarIntMode::NONNEGATIVE_SIGNED; I = unsigned int]’
  ./serialize.h:604:35:   required from ‘void CVarInt<Mode, I>::Serialize(Stream&) const [with Stream = CAutoFile; VarIntMode Mode = VarIntMode::NONNEGATIVE_SIGNED; I = unsigned int]’
  ./serialize.h:784:16:   required from ‘void Serialize(Stream&, const T&) [with Stream = CAutoFile; T = CVarInt<VarIntMode::NONNEGATIVE_SIGNED, unsigned int>; typename std::enable_if<std::is_class<T>::value>::type* <anonymous> = 0]’
  ./undo.h:29:20:   required from ‘void TxInUndoSerializer::Serialize(Stream&) const [with Stream = CAutoFile]’
  ./serialize.h:784:16:   required from ‘void Serialize(Stream&, const T&) [with Stream = CAutoFile; T = TxInUndoSerializer; typename std::enable_if<std::is_class<T>::value>::type* <anonymous> = 0]’
  ./undo.h:79:24:   [ skipping 5 instantiation contexts, use -ftemplate-backtrace-limit=0 to disable ]
  ./serialize.h:1305:20:   required from ‘void SerReadWriteMany(Stream&, CSerActionSerialize, const Args& ...) [with Stream = CAutoFile; Args = {std::vector<CTxUndo, std::allocator<CTxUndo> >}]’
  ./undo.h:108:9:   required from ‘void CBlockUndo::SerializationOp(Stream&, Operation) [with Stream = CAutoFile; Operation = CSerActionSerialize]’
  ./undo.h:104:5:   required from ‘void CBlockUndo::Serialize(Stream&) const [with Stream = CAutoFile]’
  ./serialize.h:784:16:   required from ‘void Serialize(Stream&, const T&) [with Stream = CAutoFile; T = CBlockUndo; typename std::enable_if<std::is_class<T>::value>::type* <anonymous> = 0]’
  ./streams.h:665:20:   required from ‘CAutoFile& CAutoFile::operator<<(const T&) [with T = CBlockUndo]’
  validation.cpp:1594:16:   required from here
  ./serialize.h:369:62: error: static assertion failed: Signed type required with mode NONNEGATIVE_SIGNED.
    369 |         static_assert(Mode != VarIntMode::NONNEGATIVE_SIGNED || std::is_signed<I>::value, "Signed type required with mode NONNEGATIVE_SIGNED.");
        |                       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~
  ./serialize.h:369:62: note: ‘((VarIntMode::NONNEGATIVE_SIGNED != VarIntMode::NONNEGATIVE_SIGNED) || ((bool)std::integral_constant<bool, false>::value))’ evaluates to false

I looked at line 29 in src/undo.h
      //::Serialize(s, VARINT(txout->nHeight * 2 + (txout->fCoinBase ? 1 : 0), VarIntMode::NONNEGATIVE_SIGNED));
And this call is not right, because variable is unsigned int and with this need to use Mode=DEFAULT.
This is why I edited:
vim src/undo.h
  29:      //::Serialize(s, VARINT(txout->nHeight * 2 + (txout->fCoinBase ? 1 : 0), VarIntMode::NONNEGATIVE_SIGNED));
           ::Serialize(s, VARINT(txout->nHeight * 2 + (txout->fCoinBase ? 1 : 0), VarIntMode::DEFAULT));

With VarIntMode::DEFAULT compiled fine!
Also added chmod +x share/genbuild.sh
Could you accept this commit?


THRIFT-5429: build: autotools: add foreign to AM_INIT_AUTOMAKE

Add foreign to AM_INIT_AUTOMAKE() in configure.ac to prevent autoreconf complaining about "missing" files. Autotools projects have historically required these to be present, but their absense is are less critical these days.
$ autoreconf -vfi
...
Makefile.am: error: required file './NEWS' not found
Makefile.am: error: required file './README' not found
Makefile.am: error: required file './AUTHORS' not found
Makefile.am: error: required file './ChangeLog' not found
...

Updated bootstrap.sh as well to remove unnecessary options from the automake invocation.
Fixes THRIFT-5429

[x] Did you create an Apache Jira ticket?  (not required for trivial changes)
[x] If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
[x] Did you squash your changes to a single commit?  (not required, but preferred)
[x] Did you do your best to avoid breaking changes?  If one was needed, did you label the Jira ticket with "Breaking-Change"?
[ ] If your change does not involve any code, include [skip ci] anywhere in the commit message to free up build resources.

LGTM.

Honestly, I would have removed the --add-missing as well from bootstrap.sh but it seems that that option still cannot be added to AM_INIT_AUTOMAKE(). It was brought up with the autotools maintainers a long time ago here, but for the wrong reason (libtool reasons).


Added SigningJDAO

Added SigningJDAO
Added signature output support to JSON Outputter

Fix merge conflict so I can merge. Thanks

@kgrgreer Done


System freezes

On Ubuntu 14.04 x64 with 4 GB of RAM and a 16 GB swap I try running
`cd ~/neural-style
`th neural_style.lua -gpu -1 -style_image ./examples/inputs/starry_night.jpg,./examples/inputs/the_scream.jpg -content_image Creacion.jpg -image_size 512 -output_image weird_creation.png -model_file ./models/VGG_ILSVRC_19_layers.caffemodel

I don't have a videocard, except a built-in Intel® G41, and the CPU is  Pentium Dual-Core CPU E5700 @ 3.00GHz.
It gets as far as loading layer 2 out of 30, and then the whole system freezes.
It should probably be noted that the same works for -image_size 256, although it takes several hours to calculate and doesn't look too impressive (that is, nice, but too small to see the details).

Try adding -print_iter 1 to make it print progress every iteration; it might just be very slow.
Also it's possible that you have filled up your RAM and are swapping to disk, which could be extremely slow; you should be able check that in your system monitor.

Ok, this is not an issue with neuralstyle per se, it's an issue with Ubuntu and how it handles "normal" swaps versus encrypted swaps.

By my current estimation, each iteration takes about 15 seconds. Thanks for suggesting the -print_iter.

Wow that's very slow - usually you start getting decent results only after a few hundred iterations.
To reduce memory usage you could try using the ADAM optimizer instead of LBFGS, but then you'll have to play a bit with the learning rate.

I'm getting decent results already, I just wish to go through all iterations.
As for speed and memory usage, I'm saving up for some g2xlarge time :-)


[BUG] - Error Retrieving Vault Token on Recent Tests

Describe the bug
We are experiencing an issue where retrieving the vault token fails, impacting all tests that require credentials from Vault.
Error: failed to retrieve vault token. code: ERR_NON_2XX_3XX_RESPONSE,
message: Response code 400 (Bad Request),
vaultResponse: {"errors":["audience claim found in JWT but no audiences bound to the role"]}

Expected behavior
tests should be passing
OS and architecture in which you are running Nebari
Linux (CI)
How to Reproduce the problem?
https://github.com/nebari-dev/nebari/actions/runs/11192855913/job/31117510633?pr=2756
Command output
No response
Versions and dependencies used.
No response
Compute environment
None
Integrations
No response
Anything else?
No response

@dcmcand fixed the issue by updating the bound_audiences scope that was missing from the tokens.


Add non-zero exit code when errors occur

Add a non-zero error code when errors occur in order to combine the node-cipher CLI with other bash scripts.

Thanks! I'll merge when I'm next on my personal computer. :)


[Bug]: Terms and conditions URL hard-coded in settings.yml

Installation Method
None
The Problem
The settings.yml contains this line:
legal:
Automatically Generated Settings (Do Not Edit Directly)
termsAndConditions: https://www.stirlingpdf.com/terms-and-conditions # URL to the terms and conditions of your application (e.g. https://example.com/terms) Empty string to disable or filename to load from local file in static folder
When using through docker, this leads to the Terms and Conditions button being shown (and linking to stirlingpdfs' T&C), even though this should not be the case.
Even when setting LEGAL_TERMSANDCONDITONS: "" in the compose file, the link will not disappear.
Behaviour should be similar to privacyPolicy / Impressum etc, where there is '' in the settings.yml
Version of Stirling-PDF
latest
Last Working Version of Stirling-PDF
No response
Page Where the Problem Occurred
Start page
Docker Configuration
No response
Relevant Log Output
No response
Additional Information
No response
Browsers Affected
No response
No Duplicate of the Issue

[X] I have verified that there are no existing issues raised related to my problem.

Hey, can I work on this issue?

Sorry, I accidentally created a PR for this issue ahead of time 😅 but I already closed it and plan on waiting for this issue to be assigned to me before reopening it.

If no one is working on this issue, can I pick it up?

Thinking about this more...
I actually want to always have terms and conditions for legal reasons
You can replace with your own if required but by default showing Stirling pdf seems like good functionality


RenumberAtoms fails for systems with explicit Hs

In particular, Chem.RenumberAtoms is unable to reproduce atom numbering corresponding to those after Chem.AddHs:
m1 = Chem.AddHs(Chem.MolFromSmiles('NC(C)C(=O)O'))
for a in m1.GetAtoms():
    a.SetAtomMapNum(a.GetIdx())
ps = Chem.SmilesParserParams()
ps.removeHs = False
m2 = Chem.MolFromSmiles(Chem.MolToSmiles(m1), ps)
new_order = [a.GetAtomMapNum() for a in m2.GetAtoms()]
m3 = Chem.RenumberAtoms(m2, new_order)
print([a.GetAtomMapNum() for a in m3.GetAtoms()]) # 0,1,2,3..12 is expected
> [0, 1, 2, 12, 8, 6, 9, 10, 11, 7, 5, 3, 4]

@IvanChernyshov In your code, new_order is rather the old order.
To actually make it the new order, you need to change
new_order = [a.GetAtomMapNum() for a in m2.GetAtoms()]

into
new_order = tuple(zip(*sorted([(j, i) for i, j in enumerate([a.GetAtomMapNum() for a in m2.GetAtoms()])])))[1]

See https://gist.github.com/ptosco/a9b9bc5309e3114b89fcc4aefdef9f57 for a better explanation.

@ptosco thanks a lot!


Cannot compile for chip stm32g070cb Error: p.TIM3 error[E0609]: no field TIM3 on type Peripherals

I try to use p.TIM1 and p.TIM3 for pwm generation.
My board has chip STM32G070CBTx which is for embassy feature "stm32g070cb"
Setting the feature features =  "stm32g070cb"   on [dependencies.embassy-stm32] gives compile error  p.TIM3 error[E0609]: no field TIM3 on type Peripherals
Setting the feature to "STM32g071cb" does let me compile the code. The code is operational: I can control the motors connected to TIM3, The motors connected to TIM1 do not react. Of course I do understand the chip is incorrect now
Diving into the pac definitions of both chips in stm32-data-generated I see that for both chips TIM1 and TIM3 are defined, even with the same chip register addresses.
So my request: What is the problem here?

Probably TIM3 is being used for the embassy-time driver. Remove feature time-driver-any, add another time-driver-timX to make it use another timer.

(and the issue with TIM1 is probably #2053)

Selecting feature time-driver-tim15 did let me compile for stm32g070cb.
After updating the main branch  of embassy the motors on TIM1 and TIM3 are functional now. Thanks!
Request: can there be a few comment lines in the cargo.toml of embassy-stm32 about what features are possible (and resolved dynamically?
Or even better: a link in the toml file to a markdown file with all possible features (from the user point of view, statically and dynamical) with a short description?
Embassy is really great, I really like it. Switched from rtic to embassy. But the low level stuff is hard, very hard. For example: If you want to give a module the ownership of adc, what is the typing of the adc parameter in this case. Up to today I was not able to figure this out. No example to help me out.
Documentation can make life a bit more easy!
You can close this issue.


add tag:TagResources for newest builds to iam policy

Closes #661

Cancelling and will submit another PR with proper email (and signed the CLA)

Coverage remained the same at 35.635% when pulling 32157604fdcbe57aa1a05695fed03dc13b749f01 on natefox:add-tag.TagResources into 13b731a5ee5835560ddfcd2a51a6a9a1d15655cb on kubernetes-sigs:master.


settings for cloud9 ide added to gitignore

settings for cloud9 ide added to gitignore

Seems fine to me, @lirantal @NeverOddOrEven any objections?

:+1: lgtm

merged.


AQL Based Search

points asf-search to cmr's /search/concepts/search endpoint
allows us to dodge subquery system, improving variety of returned results (this will be useful if burst products wind up using the same platform as regular Sentinel-1 data)

Codecov Report

Merging #162 (a192d4d) into master (66cb1c9) will decrease coverage by 3.30%.
The diff coverage is 88.97%.

@@            Coverage Diff             @@
##           master     #162      +/-   ##
==========================================
- Coverage   89.16%   85.86%   -3.31%
==========================================
  Files          48       49       +1
  Lines        1634     1705      +71
==========================================
+ Hits         1457     1464       +7
- Misses        177      241      +64

Flag
Coverage Δ

unittests
85.86% <88.97%> (-3.31%)
:arrow_down:

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

Impacted Files
Coverage Δ

asf_search/CMR/field_map.py
100.00% <ø> (ø)

asf_search/CMR/aql_map.py
82.66% <82.66%> (ø)

asf_search/search/search.py
94.95% <97.05%> (-1.41%)
:arrow_down:

asf_search/CMR/__init__.py
100.00% <100.00%> (ø)

asf_search/CMR/translate.py
91.77% <100.00%> (-4.09%)
:arrow_down:

asf_search/constants/INTERNAL.py
100.00% <100.00%> (ø)

asf_search/search/baseline_search.py
95.74% <100.00%> (ø)

asf_search/search/search_count.py
100.00% <100.00%> (ø)

asf_search/CMR/subquery.py
18.51% <0.00%> (-81.49%)
:arrow_down:

... and 2 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Closing aql branch due to frequent timeouts, will look into using json queries for search if feature becomes available in CMR


Different behavior observed for 1way vs 2way SSL when deploying pulsar connector NAR File to pulsar cluster

I am using Cassandra 4 and cdc-apache-cassandra 2.2.3.
I am seeing different behaviour for two way vs one way ssl when trying to deploy pulsar connector NAR file in the Pulsar cluster.
I have my certificates created using JDK (JAVA Key Tool). If am setting value of ssl.provider as "OpenSSL", I am getting exception while deploying pulsar connector NAR File
But for 1way SSL it's getting deployed successfully without any exception.
For 2way SSL, I am using below configuration:
ssl.provider="OpenSSL"
ssl.hostnameValidation="false"
ssl.keystore.password=
ssl.keystore.path=
ssl.truststore.password=
ssl.truststore.path=
For 1way SSL, I am using below configuration:
ssl.provider="OpenSSL"
ssl.hostnameValidation="false"
ssl.truststore.password=
ssl.truststore.path=
Could you please confirm if this is expected?

would you be able to try your 2-way TLS setup with tlsRequireTrustedClientCertOnConnect=true on your brokers/proxies and see if it goes through?
Reference: https://pulsar.apache.org/docs/2.11.x/security-tls-transport/#configure-brokers-1

I tried it out but still am able to see different behavior

@praagraw-tibco looking closer into this, it seems we are configure the mTLS between the connector and Cassandra - not between the connector and the brokers. For the latter:

Use Pulsar Admin CLI settings for experimenting if you are running the connector in local mode. Relevant settings can be found if you run but also see here:

./bin/pulsar-admin source localrun

Use the functions_worker.yml if when you deploy the connector in cluster mode:

Hope that helps. Please note that those settings are pulsar connector related and not CDC specific.


Bug report: [summarise the issue here]

Description of issue
Trying to install cask/sqlitestudio on OSX el Capitan
Output of `brew cask install sqlitestudio --verbose`
brew cask install sqlitestudio
```
==> Downloading http://sqlitestudio.pl/files/sqlitestudio3/complete/macosx/sqlitestudio-3.1.0.dmg
Already downloaded: /Users/michael.gracy/Library/Caches/Homebrew/Cask/sqlitestudio--3.1.0.dmg
==> Verifying checksum for Cask sqlitestudio
==> Moving App 'SQLiteStudio.app' to '/Applications/SQLiteStudio.app'
Error: Permission denied - (/usr/local/Caskroom/sqlitestudio/3.1.0/SQLiteStudio.app, /Applications/SQLiteStudio.app)Most likely, this means you have an outdated version of Homebrew-Cask. Please run:
  brew uninstall --force brew-cask; brew untap phinze/cask; brew untap caskroom/cask; brew update; brew cleanup; brew cask cleanup
If this doesn’t fix the problem, please report this bug:
https://github.com/caskroom/homebrew-cask#reporting-bugs
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/fileutils.rb:519:in rename' /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/fileutils.rb:519:in block in mv'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/fileutils.rb:1558:in block in fu_each_src_dest' /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/fileutils.rb:1574:in fu_each_src_dest0'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/fileutils.rb:1556:in fu_each_src_dest' /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/fileutils.rb:508:in mv'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/moved.rb:39:in move' /usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/moved.rb:15:in block in install_phase'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/moved.rb:32:in block in each_artifact' /usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/moved.rb:31:in each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/moved.rb:31:in each_artifact' /usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/moved.rb:11:in install_phase'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:135:in block in install_artifacts' /usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:132:in each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:132:in install_artifacts' /usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:75:in install'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:25:in block in install_casks' /usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:19:in each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:19:in install_casks' /usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:10:in run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:107:in run_command' /usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:146:in process'
/usr/local/Homebrew/Library/Homebrew/cask/cmd/brew-cask.rb:8:in <top (required)>' /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in require'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in require' /usr/local/Homebrew/Library/Homebrew/brew.rb:23:in require?'
/usr/local/Homebrew/Library/Homebrew/brew.rb:99:in `'
Error: Kernel.exit
</details>

<details><summary>Output of `brew doctor`</summary>

brew doctor
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry and just ignore them. Thanks!
Warning: Unbrewed dylibs were found in /usr/local/lib.
If you didn't put them there on purpose they could cause problems when
building Homebrew formulae, and may need to be deleted.
Unexpected dylibs:
/usr/local/lib/libecomlodr.dylib
/usr/local/lib/libMonoPosixHelper.dylib
/usr/local/lib/libSFFileMonitor.32.dylib
/usr/local/lib/libSFIPC.32.dylib
/usr/local/lib/libSFIPC.I.dylib
/usr/local/lib/libSFsqlite3.7.4.dylib
/usr/local/lib/libSFSyncEngine.I.dylib
Warning: Unbrewed header files were found in /usr/local/include.
If you didn't put them there on purpose they could cause problems when
building Homebrew formulae, and may need to be deleted.
 When you opened this issue, the template in place urged you to do so and gave you the relevant links.
The template also stated clearly that

if it is apparent you ignored the instructions for reporting issues, your issue may be closed without review.

That seems very much to be the case, here, which is why this was closed.
Again, as the template states (emphasis added)

When the guide isn‘t followed we get the same issues over and over. Having to repeatedly deal with the same solved and documented problems leads to maintainer burnout and a lot of wasted hours that could instead have been spent improving Homebrew-Cask itself and fixing real bugs.

I’ll add it is exhausting, and when maintainers are exhausted, everyone loses.
A lot of work was put into those guides precisely so we can keep things running smoothly.
I ask that you respect our time and efforts and follow the guides.
Specifically: https://github.com/caskroom/homebrew-cask/blob/master/doc/reporting_bugs/a_cask_fails_to_install.md#permission-denied-error


Model - Identifier - potential issue with containerization/missplacement

We discovered a single record with a bad identifier_object_id.  That the bad value was allowed/persisted is the concern.

Recheck validation
Recheck containerization and movement of identifiers

@mjy can you share the single-sample here? Why is it a bad value?

It was a large int to a id of that type that (likely) never existed.  I destoyed, the record, sorry (might be trace of it in error logs).  Sorry this is vague.  It's likely related to #1367 we think, which can be used to experiment to see if it can be repeated.

INHS Insect Collection 862422 and INHS Insect Collection 862424 are two such examples (Production)

Steps to replicate:
Create a collection object. Add values to various fields. Lock all fields and click "increment". Click "Add to container". Save. Click "Save and Next". Attempt to add identifier next in sequence. Get error (unable to save, bad identifier has been created somewhere).

SIMPLER steps to replicate:
Steps to replicate:
Create a collection object. Add values to various fields. Lock all fields ~and click "increment"~. Click "Add to container". Save. Click "Save and Next". Attempt to add identifier next in sequence. Get error (unable to save, bad identifier has been created somewhere).

Just to be clear - it is apparently NOT an issue with the "increment" button. I can create phantom identifiers even without it. Is it that the system thinks it needs to add both the current AND a subsequent identifier to the containerized specimen?

INHS Insect Collection 862422, 862424, 862436, 862439
 as well.

Check INHS Insect Collection 861446- 861450 as well.


Kubeflow appliance deletion

When deleting a Kubeflow appliance, the roles need to be cleaned up in order to recreate them when launching a new appliance.
This is not the expected behaviour. Please bear in mind that you cannot even create a new one on a different region.
I tried to create the IAM role from the CLI and the error below appeared:

username & password used: admin

Just to close the loop on this publicly after a lot of internal back and forth. We aren't 100% sure what got Andreea's account into this state (maybe some concurrent deploys that failed because of name collisions?), but the CF template does allow you to inject an AppName that allows creating stacks in new (or the same) region. It seems like the Kubeflow team will end up updating the CF template to not provide a default AppName and make it a required parameter to avoid this for future users.


Add ci_image folder back

The folder is required to build the docker container that is used to build images:
https://github.com/aws/meta-aws-ci/blob/master/buildspec/ci_image.yml#L13
Also, the customised Dockerfile has Yocto-specific modifications:
https://github.com/aws/meta-aws-ci/blob/51436380ede729b62d68c966a4acff3f9c3a05e5/ci_image/Dockerfile#L183

@franccan I need to deprecate this directory.  the directories are under core/ now. The main reason I will be building additional container images required for other BSPs that need different toolchains.  For Cloudformation, I think it's all set.  Is the issue in the CDK?

There are two issues:

GithubBaseImageRepositorySpecLocation in prerequisites-stack.ts would need to be updated to match the location of the buildspec file. (but this may not be needed, look at issue No.2)

CodeDeploy seems to be picky about the location and name of the buildspec file. It seems that the file cannot be in an arbitrary location with arbitrary name.  The docs do not mention this specifically, it may be an issue with CodeDeploy https://docs.aws.amazon.com/codebuild/latest/userguide/build-spec-ref.html

Error when the file sits in core/buildspec/ci_image.yml (and GithubBaseImageRepositorySpecLocation pointing to it):
[Container] 2021/08/13 09:43:14 Waiting for agent ping
[Container] 2021/08/13 09:43:17 Waiting for DOWNLOAD_SOURCE
[Container] 2021/08/13 09:43:18 Phase is DOWNLOAD_SOURCE
[Container] 2021/08/13 09:43:18 CODEBUILD_SRC_DIR=/codebuild/output/src395347404/src
[Container] 2021/08/13 09:43:18 YAML location is /codebuild/output/src395347404/src/core/buildspec/ci_image.yml
[Container] 2021/08/13 09:43:18 Phase complete: DOWNLOAD_SOURCE State: FAILED
[Container] 2021/08/13 09:43:18 Phase context status code: YAML_FILE_ERROR Message: wrong number of container tags, expected 1

Moving the file back to buildspec/ci_image.yml  works without issues:
[Container] 2021/08/13 10:15:52 Waiting for agent ping
[Container] 2021/08/13 10:15:55 Waiting for DOWNLOAD_SOURCE
[Container] 2021/08/13 10:15:56 Phase is DOWNLOAD_SOURCE
[Container] 2021/08/13 10:15:56 CODEBUILD_SRC_DIR=/codebuild/output/src587018348/src
[Container] 2021/08/13 10:15:56 YAML location is /codebuild/output/src587018348/src/buildspec/ci_image.yml
[Container] 2021/08/13 10:15:56 Processing environment variables

1/ This is codebuild, not codedeploy
2/ I don't know why we are facing this buildspec issue because the yocto image build process has an arbitrary location

CDK project has been deprecated


bump critools into ca1571e6edd116b2c95f52e3dfa0b4779b74223a

It is to fix #7264 with kubernetes-sigs/cri-tools#1026. Currently, it is not release tag but it can save our time to prevent from rerun CI manually. There will be follow-up when kubernetes v1.26 releases.
REF: https://github.com/kubernetes/sig-release/tree/master/releases/release-1.26#summary
Signed-off-by: Wei Fu<EMAIL_ADDRESS>

https://github.com/containerd/containerd/actions/runs/3466444274/jobs/5790356738
reruning


Add documentation to connect with etcd cluster with host tuple

Add the example to connect with etcd cluster with IPs and ports.
Refer to https://github.com/jplana/python-etcd/issues/223

Coverage remained the same at 88.212% when pulling 7d245b447c6236837786fa41ceba4d2fadf28309 on tobegit3hub:add_doc_about_host_tuple into 1399cc749358e31d814f1aad180e7933d8760045 on jplana:master.


Create hs-kempten.txt

All students of this university get an email with<EMAIL_ADDRESS>teachers have access with<EMAIL_ADDRESS>

push

Sorry for the delay. hs-kempten.de is already added so both<EMAIL_ADDRESS>and<EMAIL_ADDRESS>are covered. You may apply already. Thanks!


Missing opacity attribute on LineStyle

Hi,
I'd like to request the opacity feature for the LineStyle (not just the ItemStyle).
Would be very useful to have this for area (confidence bands etc) charts.
Thank you,
Alex

你好，能否PR一个?

Hi Abel,
Thank you for your reply.
I was just about to but it seems like you have included it in the new version.
Much appreciated!
Alex

Hi Abel,
Reply you for your Thank.
The I IT WAS But to the About the Just Seems like you have have included in IT at The new new Version.
Much appreciated!
Alex

Created the pull request

PR有问题。


Compressing symbols over 2GiB corrupts them

https://randomascii.wordpress.com/2013/03/09/symbols-the-microsoft-way/
https://bugs.chromium.org/p/chromium/issues/detail?id=1019811

Apparently makecab, compress.exe, and symstore all have the limitation that their input files have to be less than 2 GiB. This is a problem for Chrome’s unified chrome.dll.pdb which is currently (October 2019) about 2.5 GB, and therefore cannot be compressed. Oops. pigz can handle 4 GiB input files but a version that generates CAB files is not yet open source. We ultimately fixed this by foregoing Microsoft’s compression and instead using HTTP compression – we added -Z to our gsutil command line so that the PDBs would be gzip compressed on upload.

The simple solution is to just not run compression when the binary reaches that size.

Thanks for raising the issue @nasisakk
This symstore package is using the libgcab package (from msitools project), so that's different compression implementation compared to microsoft's symstore.exe.
However, surprisingly  the libgcab also seem to have an issues with silent corruption when compressing large files! It seems that the problem is event worse, my initial test shows that you get corrupted CAB archives event when compression source files in megabyte range.
I'll dig around the dependency chain and see if I can figure out in more details what the problems are.

cab.py should check the filesize and simply copy if it's larger than 2GB
Most of the tools to create CAB files are broken because one of the early examples was wrong :(


Increase allowed line length

This PR addresses  #161. I appreciate that it has not been discussed at length and agreed, but this shows how it could be done (and will check that CircleCI honours the config).
If people decide they are happy this can be pulled. If it gets objections, we can close this without pulling.

Hmm, the failed checks are not due to flake8 errors, so CircleCI is working as I expected. They are due to a stochastic test failing (which it has not done before!). If re-run they will probably pass, but we will try and fix them so it doesn't re-occur. That will be a separate pull request.


[5.5] MethodNotAllowedHttpException displays symfony red Whoops error

Laravel Version: 5.5.19
PHP Version: 7.1.10

Description:
I believe Laravel should handle this exception, not symfony.
The @return says this should be \Illuminate\Http\Response
/**
 * Render an exception into an HTTP response.
 *
 * @param  \Illuminate\Http\Request  $request
 * @param  \Exception  $exception
 * @return \Illuminate\Http\Response
 */
public function render($request, Exception $exception)
{
    return parent::render($request, $exception);
}

A 405.blade.php will display if it exists.
Steps To Reproduce:

Create a new 5.5 project laravel new laravel55
Change Route::get to Route::post in routes/web.php
Set APP_DEBUG=false in .env
Visit site in browser.

This will prevent the actual issue from appearing to a developer even in development mode, you'll look at your routes and see that the route is defined but still it gives a Page not found response.
For suggestions and feature requests please use the https://github.com/laravel/internals repo.


Allow viewing of decision notices on complete applications

 Planners and managers should be able to view a decision notice (but not make any changes to it) after it has been determined.

 Planners and managers should be able to view a decision notice (but not make any changes to it) after it has been determined.


Body is empty on page load

Decided to try out the app to see what I can contribute but on page load there was no data showing up. The body was completely blank
Assign me this issue under hacktoberfest and allow me to figure it out

We are using live API and occasionally, there might be a delay in getting the data, which is why you'll see a shimmer effect for a brief moment.

For me nothing showed up @Anandsg . I checked the console and there was an issue

can you share the screenshot


Could not determine the dependencies of task ':app:compileDebugJavaWithJavac'. > Could not resolve all task dependencies for configuration ':app:debugCompileClasspath'.

Hi @salihstack
When opening an issue, please try to provide all of the information indicated in the template.
Closing this issue, since no information has been provided.
Thank you.


Analyzer Feedback from IntelliJ

Just crashed. The analysis pane is frozen. I can scroll and select things, but eg double-clicking does nothing. It is showing some warnings I just fixed. Hitting Restart Dart Analysis Server 🔄 doesn't help.
Dart analyze from the command line works fine.
Huh. I was going to look for a log to send you, so I tried to open Preferences, and that doesn't work either.
Save All looks like it isn't working either. That's a bummer, because I'm a have to restart…
Version information

IDEA AI-203.7717.56.2031.7784292
2.14.4
AI-203.7717.56.2031.7784292, JRE 11.0.10+0-b96-7249189x64 JetBrains s.r.o., OS Mac OS X(aarch64) v12.0.1, screens 3360.0x2100.0  Retina

Apologies that we did not get to this issue; closing as stale but please re-open if you see a crash like this on a recent version of Dart.


Add domaintools, domainIQ, tcpiputils, viewdns

Feel free to make description changes as you see fit.

Good call, @DanielRuf.
Also, not sure if he intends to add wildcard DNS, but if so:

http://nip.io/
http://xip.io/

I doubt that xip and nip are checker sevices.

Fair point

@ryanmaynard thanks for this!
@DanielRuf thanks for the input too :)


MINOR: add metadataQuorumCommandTest for controller

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@chia7712 Could you take a look if you have time?


sync main branch with the upstream opendatahub main branch

sync main branch with the upstream opendatahub main branch
Related-to: https://github.com/opendatahub-io/notebooks/tree/main
Fixes: #250

Merging to get the main branch ready.


Cynerio add workbooks

Required items, please complete
Change(s):

Add workbook and analytic rules

Reason for Change(s):

Added content to solution

Version Updated:

Yes

Testing Completed:

Yes

Checked that the validations are passing and have addressed any issues that are present:

No

Hi @meravcy, please repackage the solution using this tool https://github.com/Azure/Azure-Sentinel/tree/0cc258c900726e89aaf8b39984be98d9a7a3cc1a/Tools/Create-Azure-Sentinel-Solution/V3
Please go through the readme.md file and follow the instructions mentioned there.
And only add version 3.0.0 zip in this PR.

Hi @v-rbajaj , created v3 package and added release notes

Hi @v-rbajaj , can you please help with the arm-ttk validation test that fails? I couldn't understand what's wrong with the workbook, it's a very simple one, exported json and looks like other workbooks exist in the repository

Hi @meravcy, to fix the failing ARM TTK validation we would need to add the details of the new workbook in WorkbooksMetadata --> https://github.com/Azure/Azure-Sentinel/blob/master/Tools/Create-Azure-Sentinel-Solution/V2/WorkbookMetadata/WorkbooksMetadata.json) and then repackage the solution (we don't have to increment the version of the solution)

Hi @meravcy,
Below are few things which I have noted about the analytical rules (applies to all the rules)
They are few properties like "tags" etc which are missing and also sequencing isn't proper, please check this analytical rule for the same - https://github.com/Azure/Azure-Sentinel/blob/master/Solutions/DNS Essentials/Analytic Rules/ExcessiveNXDOMAINDNSQueriesAnomalyBased.yaml

Hey @v-rbajaj , resolved all discussions


SpotMenu popout vertically too high when computer menu bar is set to show-hide

Description
When the computer is set to Automatically hide and show the menu bar in (System Preferences -> General) clicking on SpotMenu does not reveal the usual popout/modal.
Steps to Reproduce

In System Preferences > General set computer to Automatically hide and show the menu bar
Fire up SpotMenu
Fire up Spotify
Play any song in Spotify
Reveal Mac menu bar and click on SpotMenu

Expected behavior: SpotMenu reveals usual popout/modal of album art, details, and track controls
Actual behavior: SpotMenu modal is only seen once mouse away allowing the menu bar to hide. It looks like the top position of the modal is firing offscreen/too high with show-hide menu bar setting
Reproduces how often: 100%
Versions
SpotMenu version: Dec 24, 2018 date modified (can't find version number, sry)
You can get this information by highlighting SpotMenu.app (most likely in your Applications folder) in finder and pressing space.
MacOS version: 10.14.3
Additional Information

Capturing the bug behavior:
Menu bar view mouse IN and before click:

Menu bar view mouse OUT and AFTER click:

(that greenish box is the popout/modal from SpotMenu)

Thanks for the feedback, the latest commit should fix this. Here's a fresh build:
SpotMenu.zip

Amazing! So fast. Thanks for this great little app.

@kmikiy Can we have this release as 1.10?


Write a use agreement

this should be read at the time a new user is created.

add a placeholder for the EULA (filled with some riker ipsum!). Still need a EULA or agreement of some sort.

Our use agreement should cover three categories of things: (1) we don't take any responsibility or provide any warranties. If our tool loses your data or gives bad results, we are not to blame. (2) You agree to let us see, use, and publish (anonymously) your data to improve our tool. (3) Finally, the copyright. I just gave us a Creative Commons 4.0 International License that allows others to share and improve upon our code given that they share their improvements. The use agreement draft I wrote is here.
Here is a little Creative Commons badge to put on the license page:
SUMsarizer by Daniel Wilson, Jeremy Coyle, and Ajay Pillarisetti is licensed under a Creative Commons Attribution-ShareAlike 4.0 International License.Based on a work at https://github.com/SUMSarizer/.

Can't seem to access the draft user agreement. Added the CC badge / license link. Commit a803807


error in transcoding audio from mp3 to aac

I tried 3_transcoding.c  to transcode audio.
  StreamingParams sp = {0};
  sp.copy_audio = 0;
  sp.copy_video = 0;
  sp.video_codec = "libx264";
  sp.codec_priv_key = "x264-params";
  sp.codec_priv_value = "keyint=60:min-keyint=60:scenecut=0:force-cfr=1";
  sp.audio_codec = "aac";

The format of the input video file is as below.
  Stream #0:0[0x1](und): Video: h264 (High) (avc1 / 0x31637661), yuv420p(progressive), 1280x720, 1718 kb/s, 30.20 fps, 30 tbr, 15360 tbn (default)
    Metadata:
      handler_name    : VideoHandler
      vendor_id       : [0][0][0][0]
  Stream #0:1[0x2](und): Audio: mp3 (mp4a / 0x6134706D), 44100 Hz, stereo, fltp, 255 kb/s (default)
    Metadata:
      handler_name    : SoundHandler
      vendor_id       : [0][0][0][0]

when transcoding from mp3 to aac, I got the error msg
[aac @ 0x55e811fab600] nb_samples (1152) > frame_size (1024)

@EdsionYeh the msg you're posting as an error message seems to be a warn only, what was the output?

with
  sp.copy_video = 1;

I got a bunch of  such msgs and then Segmentation fault
[aac @ 0x5633867f8a80] nb_samples (1152) > frame_size (1024)
[aac @ 0x5633867f8a80] nb_samples (1152) > frame_size (1024)
[aac @ 0x5633867f8a80] nb_samples (1152) > frame_size (1024)
[aac @ 0x5633867f8a80] nb_samples (1152) > frame_size (1024)
[aac @ 0x5633867f8a80] nb_samples (1152) > frame_size (1024)
[aac @ 0x5633867f8a80] nb_samples (1152) > frame_size (1024)
[aac @ 0x5633867f8a80] nb_samples (1152) > frame_size (1024)
[aac @ 0x5633867f8a80] nb_samples (1152) > frame_size (1024)
Segmentation fault

The output file size looks reasonable, but it is not playable, It is very likely that av_write_trailer  didn't get a chance to be called.
with
  sp.copy_video = 0;

I got
[aac @ 0x561f16315600] nb_samples (1152) > frame_size (1024)
[aac @ 0x561f16315600] nb_samples (1152) > frame_size (1024)
[aac @ 0x561f16315600] nb_samples (1152) > frame_size (1024)
[mp4 @ 0x561f15f32280] Application provided invalid, non monotonically increasing dts to muxer in stream 0: 512 >= 0
LOG: Error -22 while receiving packet from decoder: Invalid argument

The output video file is only 48 bytes in size


"Too many decal surfaces"

Sometimes the game crashes with an error "Too many decal surfaces" - so far I only encountered this in Half-Life: Echoes. After uninstalling HLFixes, the error disappeared.

Is there a specific map this occurs in? Does it still happen if you put --no-overbright-fix in the launch options for Echoes?

This happened in echoes22 and echoes23. Putting --no-overbright-fix in the launch options didn't fix it.

I've played through echoes22 and echoes23 on both the current Steam build of Half-Life and the final pre-anniversary build and have been unable to reproduce. If you can send me a save file where this consistently happens, that would be very useful.


Feature/receptor pool addons

Same as pull request involving master bug merging into nobunaga

We can't merge this since we have a dendrite rewrite in asyncio already being built


48kHz training

Hi, thanks for such contribution!
Is it possible to train RVAE on 48 kHz data? If yes, what modifications of hyperparameters should I make?

Hi Ivan, I didn't try our method on 48kHz data. This repo is only for 16kHz data.
Theoretically, for 48kHz audio a longer CTF filter, a longer STFT window, and a larger STFT hop size should be used.

Hi Ivan, I didn't try our method on 48kHz data. This repo is only for 16kHz data.
Theoretically, for 48kHz audio a longer CTF filter, a longer STFT window, and a larger STFT hop size should be used.

Should I make some modifications in the architecture? For example, convolution kernels size or smth else?

Sorry, I have to say that I haven't tried it yet. Therefore, I can't answer your question now.
I recommend you to try it. I will also inform you and update the results if I do the experiments in the future.


[SPIKE] Compare K8s options

Is your feature request related to a problem? Please describe.
We want to have additional comparison of our solution and tools analyzed in #1818
Describe the solution you'd like
In this spike, we should compare all of tools from task #1818 and our k8s code in one table. We should consider at least what was researched in previous task, and also:

Installation
Maintenance
Troubleshooting
Upgrades
Migration of existing products

Describe alternatives you've considered
None.
Additional context
In task #1818 there was performed research on what external tools there are and what options do we have.

Small update: First version of the comparison done here. Generally we focused on comparing RKE with our code but in order to have clear score of the spike we just need to add to the table other tools such as: kubeone and kubespray.

Small update: First version of the comparison done here. Generally we focused on comparing RKE with our code but in order to have clear score of the spike we just need to add to the table other tools such as: kubeone and kubespray.

Is this research focused on RKE or RKE2?

Is this research focused on RKE or RKE2?

@mkyc - RKE :)

Update: RKE2 also added to the table :) PR #2017
SPIKE done :)

https://github.com/epiphany-platform/epiphany/blob/develop/docs/design-docs/k8s-engines-on-prem/k8s-engines-on-prem-table.md

Moving to DoD since there is nothing more what what we can add right now.

Documentation temporarily moved to confluence :
https://pgga-es.atlassian.net/wiki/spaces/EPI/pages/1415448563/ON-PREM+KUBERNETES+AUTOMATION+RESEARCH+RESULTS


Updated vep to version 95; plugins remain unchanged

I've been a bit out of the loop here but would it make sense to switch the versioning from 1.1.0 to the actual VEP version so that it's easier to keep track of what is being run? (this is obviously something that would need to change in the appropriate docker repo)


How to specify command-line arguments for running application via dub

For example, I must to provide command-line arguments for application. I can do it like this:
dub build
./myapp --myargs

Is it possible to specify "--myargs" arguments via dub, like this:
dub -- --myargs

Or this:
dub --params="--myargs"

Online documentation has been implemented by https://github.com/D-Programming-Language/dub-registry/commit/f5751e37f1254249dd217edb33d13fdb0fcdb027 and this particular piece of information is now found in the first paragraph.


Update measures due to BNF code changes

amending for BNF changes - makes classification easier

@richiecroker Looking at gluten free measure - I see GF products have mostly moved to new BNF codes. However looking on OpenCodelists it seems however there are a number of GF items that haven't moved. For example:
Barkat gluten free multigrain rice bread(0904010H0BYAEAA)
Looking up some of these codes it seems they are marked - Not available - so perhaps not of interest for BSA to change - e.g. https://openprescribing.net/dmd/amp/3666011000001102/
I wonder how this might affect the measure though if these were available and prescribed in the past few years? Not huge numbers for this example but combined it might add up:
https://openprescribing.net/analyse/#org=CCG&numIds=0904010H0BYAEAA&denom=nothing&selectedTab=chart


Bot causes connection trouble in firefox

Using firefox we are notified that we cannot connect to the server.  This message is not displayed when the bot is off.  Looking at the network tab we see many more status code 101s when prior there had been one and one additional 304 when there had only been one.
The 304s aslo constently refresh when the bot is on where as the single 304 remains in that state when the bot is off.

You can make a pull request and see if the connection issue is fixed.

the same for chrome

Same here. ERROR AND STUCK ON CONNECTING WITH CHROME AND FIREFOX.

Hoped this would work.

This issue is due to the Client -> Server Protocol and happens on all browsers. The client sends three messages with the opcodes 254, 255 and 80 when it connects. Opcode 255 sends an unsigned integer "initialise key" to the server which is like a handshake, to verify you are a valid user. This "initialise key" is hard-coded in launcher.user.js - line 394:
a.setUint32(1,<PHONE_NUMBER>, !0);
The "initialise key" used now is<PHONE_NUMBER> but due to some unknown reason, it doesn't let you connect to the server. This can be replaced with 154669603 which lets you connect to the server but then it doesn't display correctly or it may only display partially and then disappear.
Line 394 in launcher.user.js should be changed to :
a.setUint32(1, 154669603, !0);
I have also sent a pull request so this can be reviewed and merged with the main branch.
The UI not rendering is a separate issue and should be resolved separately but lets do one thing at a time :)
#565

I would like to help myself and everyone and get this bot back up and running.** I could take over Apos' work in order to keep it alive, using my knowledge of programming to its full extent.**

Please reinstall the scripts - working now


assesment.py file added for nlp task

Description
Thanks for contributing this Pull Request. Make sure that you submit this Pull Request against the master branch of this repository, add a brief description, and tag the relevant issue(s) and PR(s) below.

Relevant Issues: A program to add a new feature for users to upload a csv file and conduct nlp assesment on it was missing. It's an essential step to secure RDS instance, without working upon the client downtime.
Relevant PRs: (optional)
Type of change :

[x] New feature
[ ] Bug fix for existing feature
[x] Code quality improvement
[x] Addition or Improvement of tests
[ ] Addition or Improvement of documentation

Hello @stationeros, I have added the NLP file in the PR, have a look and do let me know what changes further I need to do in this.

@markandey1414  The ask was not this. Do take a look at the video, https://github.com/intuit/Trapheus/tree/master/labs/TrapheusAI we need to add an option in the UI to take in CSV input.


Remove link to CSS-in-JS comparsion site

Screen: https://reactjs.org/docs/faq-styling.html
The link to a CSS-in-JS comparison site should be removed as it 1) hasn't been maintained for years, 2) doesn't provide much relevant information, 3) we don't actively audit these 3rd party efforts for accuracy
https://github.com/reactjs/reactjs.org/blob/ddbd064d41d719f9ec0c2f6a4227f797a5828310/content/docs/faq-styling.md#what-is-css-in-js-what-is-css-in-js

https://github.com/reactjs/reactjs.org/commit/1cc8aa3f033939e15323ff32e8fa1248706ddf92


Disallow triggering action abilities when no valid opponents

Fixes a bug where Seen in Flames could be played when no opponents had cards in hand to trigger Melisandre.
Fixes #1497.

Responding to the comment in the issue: TriggeredAbility already has this check so reactions/interrupts should be fine.


App crashes on android 9.0

Issue description
My app crashes on android 9.0(My phone is Xiaomi 8), it works well on earlier platform(Android 6.0, 7.0, 8.0).
It crashes when the screen turns off or I switch to another app.
Error log from Android Studio
Similar to https://github.com/katzer/cordova-plugin-background-mode/issues/393#issue-346518462
2019-02-21 21:08:28.938 6320-6320/cc.cs_tao.whu_library_seat_mobile E/AndroidRuntime: FATAL EXCEPTION: main
    Process: cc.cs_tao.whu_library_seat_mobile, PID: 6320
    android.app.RemoteServiceException: Bad notification for startForeground: java.lang.RuntimeException: invalid channel for service notification: Notification(channel=null pri=-2 contentView=null vibrate=null sound=null defaults=0x0 flags=0x42 color=0x00000000 vis=PRIVATE)
        at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1760)
        at android.os.Handler.dispatchMessage(Handler.java:106)
        at android.os.Looper.loop(Looper.java:201)
        at android.app.ActivityThread.main(ActivityThread.java:6806)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:547)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:873)

My project info

Repo
whu-library-seat-mobile#83daf8357ffce4bbda762a04bc9e7005ee43784d

Apk
https://github.com/CS-Tao/whu-library-seat-mobile/releases/download/v1.6.1/whu-library-seat-mobile_v1.6.1.apk

Node and Cordova version

node  v8.15.0
cordova  8.1.2<EMAIL_ADDRESS>

Build environment in travis-ci
android:
  components:
    - tools
    - build-tools-28.0.3
    - android-27
    - extra-android-m2repository
    - extra-android-support
    - extra-google-m2repository
  licenses:
    - 'android-sdk-license.*'

cordova-plugin-background-mode versions I have tried

v0.7.2
https://github.com/katzer/cordova-plugin-background-mode
https://github.com/tushe/cordova-plugin-background-mode

Other cordova plugins in my project
{
  "cordova-android": "~7.1.1",
  "cordova-plugin-advanced-http": "^2.0.1",
  "cordova-plugin-badge": "^0.8.7",
  "cordova-plugin-device": "^2.0.2",
  "cordova-plugin-file": "^6.0.1",
  "cordova-plugin-local-notification": "^0.9.0-beta.2"
}

Can anyone help me? Many thanks.

It will not crash if I do not show the notification from this plugin.
cordova.plugins.backgroundMode.setDefaults({ silent: true })

Note: Notifications from cordova-plugin-local-notification still work well in my app.

Add  in AndroidManifest.xml， refer to https://stackoverflow.com/questions/52382710/permission-denial-startforeground-requires-android-permission-foreground-servic

Add in AndroidManifest.xml， refer to https://stackoverflow.com/questions/52382710/permission-denial-startforeground-requires-android-permission-foreground-servic

Hi, thank you for giving me this solution, I have tried it, but it doesn't seem to solve my problem. The problem in my app is invalid channel for service notification, but this solution is for requires android.permission.FOREGROUND_SERVICE.

Try this:
cordova plugin add https://github.com/katzer/cordova-plugin-background-mode --nofetch
ionic cordova platform remove android
ionic cordova platform add android

That fixed the problem for me. And It fixed an iOS bug too where app didn't start even.
Hope It helps

@Genarito Think you. I have tried to reinstall this plugin or reinstall android platform, but all of these did not solve my problem.
My project is not based on ionic, and this bug only occurs on Android 9.0(Maybe include Android 8.1).

Sad of hearing it :S. In mi case the --no-fetch made the difference. And yes, It affected Android >= 8.
Hope you find a solution!

@Genarito Oh I see. I'll have a try with --nofetch, thank you.

Sadly, it still can't work. 😭

The same problem, @CS-Tao you didn't find a solution?

@weglov No, the only way I find is cordova.plugins.backgroundMode.setDefaults({ silent: true }), nothing more. 😢

+1

@CS-Tao @KinG-InFeT for me this solution it's working:
cordova plugin add https://github.com/katzer/cordova-plugin-background-mode --nofetch
because the external package contains these changes https://github.com/katzer/cordova-plugin-background-mode/pull/389

@weglov Thank you, It works!
And @Genarito ‘s solutions can solve this problem, too. I sorry that I did not delete the caches in Travis-ci before, so Travis always releases the apk with the old plugin.

I am having a problem on Android 9 as well when I pause the app.
As @CS-Tao  mentioned this works for me also:
cordova.plugins.backgroundMode.setDefaults({ silent: true });
What is the ramification for my app to setting silent true? Is it still going to work in the back ground?

I am having a problem on Android 9 as well when I pause the app.
As @CS-Tao mentioned this works for me also:
cordova.plugins.backgroundMode.setDefaults({ silent: true });
What is the ramification for my app to setting silent true? Is it still going to work in the back ground?

From the app plugin:

In silent mode the plugin will not display a notification - which is not the default. Be aware that Android recommends adding a notification otherwise the OS may pause the app.

I was looking for another solution and found that:

You can also add the permission to Ionic 3 / Cordova apps by adding this in your config.xml under android platform:
    <platform name="android">
        <config-file parent="./" target="app/src/main/AndroidManifest.xml" xmlns:android="http://schemas.android.com/apk/res/android">
            <uses-permission android:name="android.permission.FOREGROUND_SERVICE" />
        </config-file>
    </platform>

It fixed my crash issue. Now just need to test if android 9 leaves the app running in the background or it kills it after 5 minutes.

So I just found out that when the targetSdkVersion is set to 27, the plugin works perfectly well but when changing it to 28 the issue is back, and can only be solved by actively adding the FOREGROUND_SERVICE permission.
I think this issue should be reopened and the permission should be included in the plugin itself.

@Genarito: i tried ur solution but not worked for me can you tell me sir any another way?

@kapilSoni101 unfortunately I've not worked in that project since a long time ago. So I don't know if there's a better solution to your problem. But I remember that all the solutions that worked for me were installing different forks of the plug-in that solved different problems.
If you install a plug-in and nothing happen try to remove the platform, delete the plugins folder and add the platform again for a cleaning installation.
Hope you solve it!

@Genarito : ok sir I will try another solutiona and let u know it's working or not.

@Genarito : sir i tried ur another solution but its not worked for me i got same result  again same error?

Sad of hear it, hope you can find a solution, I've no idea what could be the reason of your problem :/

@Genarito:-ok sir thanks for ur valuable feedback


blog migration

Once the new flask template is loaded, I need to migrate all my blogs to their own pages on the new site

Migration complete