RuntimeError: multiple tensor pointing at the same memory location

Hey there! I loved your paper and I'm finding your code quite interesting.
I'm running a training job locally on the Tetrominoes config and dataset and, while first iterations run smoothly, at the end of the epoch I'm finding this error, which seems to stem from the optimization.step() line.
Traceback (most recent call last):
  File "src/trainer.py", line 1091, in <module>
    trainer.run(seed=seed)
  File "/home/ubuntu/education/src/dti-sprites/src/utils/__init__.py", line 102, in wrapper
    return f(*args, **kw)
  File "src/trainer.py", line 292, in run
    self.single_train_batch_run(images)
  File "src/trainer.py", line 337, in single_train_batch_run
    self.optimizer.step()
  File "/home/ubuntu/anaconda3/envs/dti-sprites/lib/python3.7/site-packages/torch/optim/lr_scheduler.py", line 67, in wrapper
    return wrapped(*args, **kwargs)
  File "/home/ubuntu/anaconda3/envs/dti-sprites/lib/python3.7/site-packages/torch/autograd/grad_mode.py", line 15, in decorate_context
    return func(*args, **kwargs)
  File "/home/ubuntu/anaconda3/envs/dti-sprites/lib/python3.7/site-packages/torch/optim/adam.py", line 131, in step
    p.addcdiv_(exp_avg, denom, value=-step_size)
RuntimeError: unsupported operation: more than one element of the written-to tensor refers to a single memory location.
Please clone() the tensor before performing the operation.

Have you had this same issue? Do you know how to solve it? I believe it's caused by a tensor not being called the clone() or expand() operation.

Hi @JHevia23, thanks for reaching out!
I have not encountered this error, and it is weird it is only happening at the end of the epoch and not every iteration... Have you built the conda env following the readme.md? If not, could you check that your python packages have the same version as the ones I provided in the environment.yml?
Yes the error often means there is an expand operation before an in-place operation (here addcdiv_); could you get the tensor name or size of p that raises the issue so that we identify it?

closing the issue since there is no reply, feel free to re-open it!


Tag people in issue/PR for Mibudin

When you add code to the repository, please try to use PR and tag me like when I said in https://github.com/yxuan0329/R-tree-Search-Engine/issues/1 . But just merge immediately without waiting for the review (I very likely cannot do it very quickly). But the PR still gives me an opportunity to review your code.

@Mibudin there is no checkin or PR in the repository.  Please let me know if I can help you.


The TCP problem

The reason why smoltcp isn't working is because the GMAC is doing some frame filtering.
Setting the CAF bit in GMAC_NCFGR fixes this, alternatively there should be a way of actually setting the addresses you want to copy, but I haven't gotten that far yet.

Hey!
Not sure where you saw my struggles, but the actual problem I had was that I was loading the MAC address into the hardware wrong, likely causing all kinds of problems.
You can see the relevant delta here: https://github.com/jamesmunns/same70-experiments/commit/9e0c358f77075bb772c742564f3d3f173c9e7913#diff-e9fd362657139078d4a5146e7f2e5009bf82862042adbf6ec1e846b58780e9b0L480-L494


Hope the developer releases soon to compete with Everything

It's been two weeks, and I finally finished and released WindFind Birth. You can give it a try, and feel free to share any feedback or suggestions you have.
Everything is a really great piece of software. Right now, Everything focuses on graphical users, while WindFind caters to command line users, and I believe both can develop very well.

Will there be a Linux or macOS version of this app?

@zzugyl
Hello, the Linux version WindFind_1.1.0_Linux_Portable has been released. Please give it a try. The macOS version is expected to be released early next month. Stay tuned!

Windows 11 ，The 'wf' command can easily trigger the firewall to open in Windows
wf -d 2 D:\software\

@zzugyl
Thank you. I know about this issue, but the abbreviation of the product name is like this, so it has to be this way. However, this is an easy issue to fix. There are two solutions: one is to change it to WindFind.exe or another unique name, and the other is to run the following command with administrator privileges to add the directory containing 'wf' to the system PATH:
setx PATH "C:\PATH\TO\wf.exe;%PATH%" /M

The release of a GUI version will be considered in the future to make it more convenient to use. And let me know if you need any further adjustments!

What does this symbol mean?

@zzugyl
These are the icons for the files. You need to set your terminal font to Nerdfonts, such as RobotoMonoNerdFont-Regular.ttf. The specific method for setting the terminal font may vary, so you should refer to your terminal's user manual for instructions. If set correctly, the display should look like this:


Design catalog editing page

Catalog editing page
There should be functionality to:

[ ] Add new lists
[ ] Remove lists (only if they a list is not used in any questionnaire atm)
[ ] Add new text entries to a list
[ ] Delete entries from a list
[ ] Edit entries
[ ] Important: both lists' names and entries in the lists should have translations functionality

Storypoints: (2 + 1) * 1.5 = 3 * 1.5 = 4.5


Enhance compile fix way before or after test migrate, and search test case,or other test way

(1) We can divide the time when compilation errors occur into before and after test migration, and specify corresponding strategies for this.
Before testing the migration:
1b. Search for JDK version,（now we have a version）
2b. Repair the pom or gradle file that cannot access the snapshot,
3b. Invalid dependencies, find a suitable version to replace.
4b. Others
The objects to be repaired for compilation failure before migration include BFC. If successful, the partial strategy should be propagated to BFC-1.
After testing the migration:
1a. Detect refactoring and revert the refactoring
2a. Check whether the pom contains interface upgrades. Start with a simple junit and use rules to downgrade the api under test.
3a. If the JDK versions in BFC and BFC-1 are inconsistent, some codes may need to be converted, such as changing "lamda iteration" in jdk8 to "for loop" in 1.7, refer to the rules in the paper.
4a. Others

For BFC who couldn't find the test, but had huge suspicions. We need to adjust the testing method:

If the modified test method is private, we should look for a public one.
Directly test all tests of the entire project and retrieve BFC PASS & BFC-1 FAIL tests in the test report.
Search for the test in the commit after the BFC.
The code snippets existing in the bug report are converted into tests.
For multi-module testing, detect the test execution command of the module

I would likes to try (1)1b and (1)1a. (2)3 and(2)2


Cannot build or run iOS after flutter upgrade to 0.11.9

Steps to Reproduce

Run flutter build ios -t lib/main.dart
Build fails and you get something like this:

[   +7 ms] Could not build the application for the simulator.
[   +2 ms] Error launching application on iPhone XR.

Logs

flutter run logs are attached as separate file run.log because they are too long to put in body.

run.log

flutter analyze
Analyzing kratos...

   info • Name non-constant identifiers using lowerCamelCase • lib/commons/constants.dart:15:14 • non_constant_identifier_names
   info • Name non-constant identifiers using lowerCamelCase • lib/commons/constants.dart:21:14 • non_constant_identifier_names
   info • Name non-constant identifiers using lowerCamelCase • lib/commons/constants.dart:25:14 • non_constant_identifier_names
   info • Name non-constant identifiers using lowerCamelCase • lib/commons/constants.dart:29:14 • non_constant_identifier_names
   info • Name non-constant identifiers using lowerCamelCase • lib/commons/constants.dart:33:14 • non_constant_identifier_names
   info • Name non-constant identifiers using lowerCamelCase • lib/commons/constants.dart:37:14 • non_constant_identifier_names
   info • Name non-constant identifiers using lowerCamelCase • lib/commons/constants.dart:41:14 • non_constant_identifier_names
   info • Name non-constant identifiers using lowerCamelCase • lib/commons/constants.dart:45:14 • non_constant_identifier_names
   info • Name non-constant identifiers using lowerCamelCase • lib/commons/location.dart:49:9 • non_constant_identifier_names
   info • Name non-constant identifiers using lowerCamelCase • lib/commons/location.dart:50:18 • non_constant_identifier_names
   info • Name non-constant identifiers using lowerCamelCase • lib/commons/location.dart:75:9 • non_constant_identifier_names
   info • Name non-constant identifiers using lowerCamelCase • lib/commons/location.dart:76:18 • non_constant_identifier_names
   info • This class inherits from a class marked as @immutable, and therefore should be immutable (all instance fields must be final) • lib/components/auto_time.dart:14:7 • must_be_immutable
   info • This class inherits from a class marked as @immutable, and therefore should be immutable (all instance fields must be final) • lib/components/onboarding_cta_button.dart:3:7 • must_be_immutable
   info • This class inherits from a class marked as @immutable, and therefore should be immutable (all instance fields must be final) • lib/components/outline_button.dart:3:7 • must_be_immutable
   info • Avoid using braces in interpolation when not needed • lib/model/google/protobuf/any.pb.dart:73:22 • unnecessary_brace_in_string_interps
   info • Name types using UpperCamelCase • lib/model/google/protobuf/descriptor.pb.dart:109:7 • camel_case_types
   info • Name types using UpperCamelCase • lib/model/google/protobuf/descriptor.pb.dart:146:7 • camel_case_types
   info • Name types using UpperCamelCase • lib/model/google/protobuf/descriptor.pb.dart:366:7 • camel_case_types
   info • Name types using UpperCamelCase • lib/model/google/protobuf/descriptor.pb.dart:964:7 • camel_case_types
   info • Name types using UpperCamelCase • lib/model/google/protobuf/descriptor.pb.dart:1053:7 • camel_case_types
   info • Name types using UpperCamelCase • lib/model/google/protobuf/descriptor.pb.dart:1117:7 • camel_case_types
   info • Name types using UpperCamelCase • lib/model/google/protobuf/descriptor.pbenum.dart:11:7 • camel_case_types
   info • Name types using UpperCamelCase • lib/model/google/protobuf/descriptor.pbenum.dart:61:7 • camel_case_types
   info • Name types using UpperCamelCase • lib/model/google/protobuf/descriptor.pbenum.dart:81:7 • camel_case_types
   info • Name types using UpperCamelCase • lib/model/google/protobuf/descriptor.pbenum.dart:101:7 • camel_case_types
   info • Name types using UpperCamelCase • lib/model/google/protobuf/descriptor.pbenum.dart:121:7 • camel_case_types
   info • Name types using UpperCamelCase • lib/model/google/protobuf/descriptor.pbenum.dart:141:7 • camel_case_types
   info • Name types using UpperCamelCase • lib/model/google/protobuf/struct.pb.dart:16:7 • camel_case_types
   info • Name types using UpperCamelCase • lib/model/google/protobuf/type.pbenum.dart:29:7 • camel_case_types
   info • Name types using UpperCamelCase • lib/model/google/protobuf/type.pbenum.dart:81:7 • camel_case_types
   info • Name non-constant identifiers using lowerCamelCase • lib/res/themes.dart:5:11 • non_constant_identifier_names
   info • Name non-constant identifiers using lowerCamelCase • lib/res/themes.dart:25:11 • non_constant_identifier_names
   info • Name non-constant identifiers using lowerCamelCase • lib/views/comments_view.dart:161:20 • non_constant_identifier_names
   info • The value of the field '_longPressActive' isn't used • lib/views/create_event.dart:35:8 • unused_field
   info • The value of the local variable 'res' isn't used • lib/views/create_event.dart:301:13 • unused_local_variable
   info • Unused import: 'package:kratos/components/outline_button.dart' • lib/views/create_profile.dart:9:8 • unused_import
   info • Avoid empty statements • lib/views/discover.dart:25:21 • empty_statements
   info • The value of the local variable 'bs' isn't used • lib/views/discover.dart:279:9 • unused_local_variable
   info • This class inherits from a class marked as @immutable, and therefore should be immutable (all instance fields must be final) • lib/views/event_info.dart:29:7 • must_be_immutable
   info • The getter '_messagesAction' isn't used • lib/views/home.dart:222:14 • unused_element
   info • This class inherits from a class marked as @immutable, and therefore should be immutable (all instance fields must be final) • lib/views/profile.dart:19:7 • must_be_immutable
   info • Unused import: 'package:kratos/components/outline_button.dart' • lib/views/signup.dart:8:8 • unused_import
   info • Unused import: 'package:flutter/services.dart' • lib/views/splash.dart:2:8 • unused_import

flutter doctor -v
[✓] Flutter (Channel beta, v0.11.9, on Mac OS X 10.14.1 18B75, locale en-IN)
    • Flutter version 0.11.9 at /Users/ishaanbahal/dev/flutter
    • Framework revision d48e6e433c (29 hours ago), 2018-11-20 22:05:23 -0500
    • Engine revision 5c8147450d
    • Dart version 2.1.0 (build 2.1.0-dev.9.4 f9ebf21297)

[✓] Android toolchain - develop for Android devices (Android SDK 28.0.3)
    • Android SDK at /Users/ishaanbahal/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.1, Build version 10B61
    • ios-deploy 1.9.4
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 30.0.1
    • Dart plugin version 181.5656
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)

[✓] VS Code (version 1.29.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 2.20.0

[✓] Connected device (1 available)
    • iPhone XR • 165E306C-DF04-4C2F-BA3F-65E092E0156B • ios • iOS 12.1 (simulator)

• No issues found!

pubspec.yaml
name: kratos
description: Some description

version: 1.0.4+12

environment:
  sdk: ">=2.0.0-dev.68.0 <3.0.0"

dependencies:
  flutter:
    sdk: flutter
  cupertino_icons: '^0.1.2'
  grpc: '^0.6.5'
  protobuf: '^0.10.4'
  intl: '^0.15.7'
  geolocator: '^2.1.0'
  shared_preferences: '^0.4.2'
  image_picker: '^0.4.10'
  image: '^2.0.4'
  image_cropper: '^0.0.6'
  cached_network_image: '^0.4.2'
  sqflite: '^0.12.1'
  google_maps_flutter:
     git:
        url: git://github.com/flutter/plugins
        path: packages/google_maps_flutter
  firebase_messaging: '^2.1.0'
  firebase_dynamic_links: '^0.1.0'
  share: '^0.5.3'
  http: 'any'
  url_launcher: '^4.0.1'
#  quick_actions: '^0.2.1'
  package_info: '^0.3.2'
  flutter_google_places_autocomplete: '^0.1.3'

dev_dependencies:
  flutter_test:
    sdk: flutter

flutter:
  uses-material-design: true
  assets:
    - lib/assets/
  fonts:
      - family: Sacramento
        fonts:
          - asset: lib/fonts/Sacramento-Regular.ttf

Prior to this version, 0.11.8 was working fine, only issue was hot reload. Now I cannot seem to be able to build iOS at all. Have done flutter upgrade today and stuck here. Android build is working fine. I suspect there might be an issue with the Swift 4.2 that is being used with permission-handler plugin that geolocator uses Implicitly, but I'm not sure. XCode does show errors

I'm not sure how to get these errors in some log file from XCode as I'm pretty new to iOS in general, hence the screenshot.
Any help would be appreciated. :)

Okay, so I've managed to solve it. Seems like I needed to upgrade my swift version to 4.2 in Runner > Target > Build Settings > Swift Language Version using XCode 10.1
Also had to change UIApplicationLaunchOptionsKey to UIApplication.LaunchOptionsKey in AppDelegate.swift and delete ~/Library/Developer/Xcode/DerivedData/* to be able to build and run the project. So if anyone else runs into such an issue, this solution might be helpful.
Anyone who uses Geolocator, permission-handler was updated to use Swift 4.2 and that's probably why my build didn't work. Relevant commit: https://github.com/BaseflowIT/flutter-permission-handler/commit/e07ce94b46039b86661422c38ea084c3ecad481f
It works fine now, but I'm not sure if I have done the right thing. Leaving the issue open for a day if someone has a better solution to this, else will close it.

After reading @ishaanbahal 's comment, I've managed to solve it by changing a single line in the Podfile without touching anything in the Build Settings tab on the XCode. Just looking for the config.build_settings['SWIFT_VERSION'] = '4.1' in Podfile and change the version number to '4.2'. Don't forget to delete Podfile.lock and Pods folder. After that, run 'flutter clean' and 'flutter build ios'. Hope this way help you out too 😃

tried above mentioned solutions but no luck


Livechat page fails to load

Livechat screen displays blank screen and there is no way to dismiss the blank screen therefore user has to restart the app .
For sdk to render the chat screen, a script message should be received from the webpage (messageType : uiReady). Which is not been received when using our license id and is working fine for the example license id(1520).
attached screenshot for reference.
Could you please look into this issue?
email<EMAIL_ADDRESS>

Hi LiveChat Team,
This issue is not app specific. I'm unsure if there is issue either with license id validation or sdk.
Blank screen is also seen in the sample app provided, when we plugin customer license id there.
As previously mentioned, I have noticed a discrepancy, a message from webView is not been received when using customer license id.
WebViewBridge unknown messageType: {
messageType = uiReady;
}
Could you please prioritise this issue and investigate from the sdk perspective as what could be the cause for it?
Thanks

@VinayRajK95 thanks for your report. Could you please provide a licenseId you are using so we will be able to investigate the issue?

@VinayRajK95 thanks for your report. Could you please provide a licenseId you are using so we will be able to investigate the issue?

Hi @kamil-szostakowski ,
Its loading now. thanks

Hi @kamil-szostakowski ,
Its loading now. thanks


Reimplement ignored Groups dependency integration test

The Groups with dependant Applications get upgraded in the correct order with maintained upgrade strategy test in src/test/scala/mesosphere/marathon/integration/GroupDeployIntegrationTest.scala is being  ignored, because it was flaky.
We need to reimplement it, so that it isn't flaky anymore.

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-2927. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


Group electrostatic potential handlers

Description
This PR aims to group all of the available SMIRNOFF electrostatic parameter handlers under a single potential handler.
This proposal is driven by:

The systems' potential handlers have a 'slot' based view of the world, whereby each potential handler maps to a specific valence type or pairwise interaction. The bond potential handler handles terms applied to valence bond terms, the angle handler to valence angles and so on.
The current implementation however introduces handlers which don't necessarily map to corresponding pairwise interaction types, namely library charge slots, charge increment slots and (in a sense) am1bcc slots. What these different handlers really do is provide parameters which occupy specific electrostatic / charge slots.
Hence it would seem to make sense to have a single potential handler that describes which parameter have been applied to which electrostatic slots, even if those parameters came from multiple parameter handlers.

In the current implementation validating which slots have been filled and by what is challenging and leads to more complex design patterns being needed. What happens, for example, if a system that has both a SMIRNOFFLibraryChargeHandler and a SMIRNOFFChargeIncrementHandler have assigned a parameter to the same TopologyKey(atom_indices=(X,) slot?
Should they be added? Or one override the other? and if so what is responsible for ensuring that this logic is enforced?
Moving all of the matching logic under a single handler unifies such validation and choices, and does not require a parent object to orchestrate and enforce such logic.

Having a single slot map for electrostatics makes it easier to track which original parameter resulted in which partial charge. Currently one would need to look into the AM1, charge increment and library charge handlers to track this information down. This makes bookkeeping and aggregation during parameter optimisation significantly cleaner and easier. This is also true when trying to compute the energy of a topology based on its system object due to all of the related terms being cleanly grouped together.

Having a potential handler be able to store parameters from multiple handlers should make adding things like v-site support down the road easier. I.e. rather than have a VSiteVdWHandler and a VSiteElectrostaticsHandler one have the existing vdW and Electrostatics potential handlers consume the v-site parameter handler and themselves handle the required charge distribution and parameter matching logic.

@mattwthompson it would be good to get your input here.
Note: the 'tests' added here are to show the proof of concept and should be replaced with proper unit tests if this proposal moves ahead.
Questions

Currently a potential key does not track the handler that assigned a particular parameter, only the 'id' (i.e. SMIRKS pattern). Maybe PotentialKey should allow a tuple as an id such that the id could be PotentialKey(id=(handler_type, smirks_pattern))?
How should a potential handler communicate via its API which parameter handlers it can consume?

Checklist

[ ] Add tests
[ ] Lint
[ ] Update docstrings

Thanks for putting all of this together - I'll need to think about it more deeply, but the motivations seems sound. In particular, pointing out that electrostatics potentially involve a "many parameters -> one slot" mapping makes a single handler sound like a good approach.

Maybe PotentialKey should allow a tuple as an id such that the id could be PotentialKey(id=(handler_type, smirks_pattern))?

Would it be better or worse to group these together into a single tuple vs. a separate field? I.e. what about PotentialKey(id=smirks_pattern, handler_type=handler_type)? Currently this question is avoided altogether because potential keys are only allowed to exist within a handler, bypassing the need for a key to know where it should go. handler_type could maybe an optional field in order to cover electrostatics without carrying around redundant information for keys in valence handlers.

How should a potential handler communicate via its API which parameter handlers it can consume?

We could store a sequence of accepted handler types in the class and expose them with a method ElectrostaticsHandler.digestable_parameter_handlers (named something better). I can see this being handled with a one-off solution to make an electrostatics handler work, but maybe it's worth considering how it looks for the other handlers, like the valence parameters that we implicitly assume will only ever digest one type of parameter handler?
One thing that comes to mind is if/how parameters should be applied after initial construction. This is probably a moot point if the construction delegates to other calls intelligently. Splicing some of this test code together, the idea is that these two snippets should produce the same result (I think), both the same charges and similar provenance
        parameter_handlers = [
            ElectrostaticsHandler(version=0.3),
            charge_increment_handler,
            library_charge_handler,
        ]

        potential_handler = SMIRNOFFElectrostaticsHandler.from_toolkit(
            parameter_handlers, top
        )
        potential_handler.partial_charges

        parameter_handlers = [
            ElectrostaticsHandler(version=0.3),
            charge_increment_handler,
        ]

        potential_handler = SMIRNOFFElectrostaticsHandler.from_toolkit(
            parameter_handlers, top
        )
        potential_handler.somehow_update_with_another_handler(
            library_charge_handler,
        )
        potential_handler.partial_charges

Would it be better or worse to group these together into a single tuple vs. a separate field? I.e. what about PotentialKey(id=smirks_pattern, handler_type=handler_type)? Currently this question is avoided altogether because potential keys are only allowed to exist within a handler, bypassing the need for a key to know where it should go. handler_type could maybe an optional field in order to cover electrostatics without carrying around redundant information for keys in valence handlers.

This is along the lines that I was thinking, but I wasn't sure if it would make sense for the other potential handlers that you support (e.g. Foyer handlers).

We could store a sequence of accepted handler types in the class and expose them with a method ElectrostaticsHandler.digestable_parameter_handlers (named something better). I can see this being handled with a one-off solution to make an electrostatics handler work, but maybe it's worth considering how it looks for the other handlers, like the valence parameters that we implicitly assume will only ever digest one type of parameter handler?

This is also what I was thinking (I think I went with associated_handlers but wasn't a huge fan of that...) but didn't want to make the initial PR too convoluted 🙂  It would ultimately work almost like dependency injection.

One thing that comes to mind is if/how parameters should be applied after initial construction. This is probably a moot point if the construction delegates to other calls intelligently.

I think one could just have a function which internally calls from_toolkit, and then updates the slot / potential dicts as you suggest.

Notes from a chat with @SimonBoothroyd

If one allows a PotentialHandler to store multiple ParameterHandler objects, a PotentialKey would need to know which handler it's actually associated with

This is a feature we need

Some use cases in which different handlers need different metadata (like an AM1BCCHandler knowing the AM1 method, number of conformers, etc. - how would an ElectrostaticsHandler access this information?)
The simple handlers, like bonds, are "one-to-one" in the sense that one parameter from a force field applies to one "slot." In general, we need to be concerned with "many-to-one," "one-to-many," and "many-to-many," each of which can be demonstrated with partial charge methods and/or virtual sites.

For potential keys in charge models, mult carries a meaning more like "the atom index of the original attribute (minus 1)," where the charge model could be charge increments, library charges, or something similar. This should be renamed, but it's hard to come up with a suitable name.
For topology keys in charge models, mult carries a meaning more like "this is the nth layering of a charge model into this atom index," which is fairly consistent with its meaning in topology keys associated with torsions. This should also be renamed to something.

We think that prerequisite PR needs to be done before this. It needs to

Allow PotentialHandlers to be created from multiple ParameterHandler. The missing behaviors are

A PotentialHandler needs to know which types of ParameterHandlers they're allowed to digest
PotentialHandler.from_toolkit needs to allow the input of multiple parameter handlers, mostly as a bookkeeping check. This will probably only be used by electrostatics handlers at first.

Then, this can be fleshed out with unit tests - I will handle this and ask @SimonBoothroyd for review. Integration tests will not be a part of this PR, but may come later.

General status update:

If one allows a PotentialHandler to store multiple ParameterHandler objects, a PotentialKey would need to know which handler it's actually associated with

Implemented in #207

Some use cases in which different handlers need different metadata

Unresolved

A PotentialHandler needs to know which types of ParameterHandlers it's allowed to digest

Implemented in #204

PotentialHandler.from_toolkit needs to allow the input of multiple parameter handlers, mostly as a bookkeeping check. This will probably only be used by electrostatics handlers at first.

Implemented in 9a9bd31, but might be iterated on futher.
The simplest tests cases are implemented, but there's plenty more to do before this will feel useful:

Handling charge method precedence
In general missing a few dozen tests for non-trivially-simple behavior
Updating the different uses of mult
Would TopologyKey.associated_handler be useful? Given that it's a key, and cannot perform one-to-many mapping while only storing the atom indices.

@SimonBoothroyd I could some feedback on the general direction I've taken this, if that's possible to do with a brief look at the changes. I have the three simple unit tests passing locally (won't pass on CI until the API breaks are fixed elsewhere, and I don't want to do that until this is working in its own scope).

@mattwthompson thanks for your work on this, things are coming along nicely!

Implemented in 9a9bd31, but might be iterated on futher.

I think the _from_toolkit method added in 9a9bd31 duplicates most of the logic already implemented in SMIRNOFFElectrostaticsHandler.store_matches. It looks like L581-L606 can be replaced with just handler.store_matches(parameter_handlers, topology) (note the use of parameter_handlers and not parameter_handler).

Handling charge method precedence

This should already be handled I think if the above change is made.

Would TopologyKey.associated_handler be useful? Given that it's a key, and cannot perform one-to-many mapping while only storing the atom indices.

Hmm I'm not sure I understand this point - could you perhaps give an example of where it would come in handy?

I think a good next step (separate PR which should be merged before this) would be to try and replace the System.from_smirnoff handler mapping code with a more dependency injection type approach now that allowed_parameter_handlers is exposed. In particular, it would seem that whole block could be replaced with something more like:
parameter_handlers_by_type = {
    force_field[parameter_handler_name].__class__
    for parameter_handler_name in force_field.registered_parameter_handlers
}

for potential_handler_type in [
    SMIRNOFFAngleHandler,
    SMIRNOFFBondHandler,
    SMIRNOFFConstraintHandler,
    SMIRNOFFImproperTorsionHandler,
    SMIRNOFFProperTorsionHandler,
    SMIRNOFFvdWHandler,
    SMIRNOFFElectrostaticsHandler,
]:

    parameter_handlers = [
        parameter_handlers_by_type[allowed_type]
        for allowed_type in potential_handler_type.allowed_parameter_handlers()
    ]

    potential_handler = potential_handler_type._from_toolkit(parameter_handlers, topology)

    sys_out.handlers[potential_handler.type] = potential_handler

@mattwthompson I think 37a125b exposed a bug in from_openmm - namely the vdW parameters are all imported as 0.0 as the conversion only considers standard NonbondedForce and not the new split CustomNonbondedForce introduced in #201.
For now I've xfailed the test as it is unrelated to this PR but this should probably be fixed once #200 is in!

Ok @mattwthompson this is now coming back all green and seems to be working well and efficiently. You may want to consider merging this soon and then handling the remaining issues (mult, individual charge handler provenance etc) in more targeted PRs as this one is now getting quite large!

Hmm, yeah that behavior was probably not implemented in #201. That's a lower priority feature so I'm fine breaking it between alpha releases.
All tests passing and 93% diff coverage is plenty good for a huge change like this one - hard to really review this so I do think the path forward is to merge this now/soon and build out the other fixes and tests in smaller PRs. Sound good?

Sure that sounds good to me!

Thanks for all your help on this PR 🙂


Support of partial derivative

Based on this page (https://leanprover-community.github.io/undergrad_todo.html), it seems the partial derivative (up to an arbitrary order) is not supported yet in mathlib4.
Can anyone brief the core bottleneck to implement the partial derivative into mathlib4/Lean4? If I want to do this, how challenging it would be?
Thank you!

Hey, I found this zulip stream with infos that could help you
https://leanprover.zulipchat.com/#narrow/channel/287929-mathlib4/topic/Partial.20derivatives

@herostrat Thank you very much!!

Hi @herostrat! Based on the zulip chat, I think lineDeriv is the most appropriate way for partial derivatives.
However, I'm not sure if mathlib4 supports integration over multi-variable functions.

To reflect what was said on zulip: yes, mathlib has integration over multi-variable functions.


Support resetting a value to it's default by adding Default::to_default()

Motivation
Sometimes it's useful to be able to mutate a value to it's Default implementation, and it would be useful to have that in the standard library (ideally as a part of Default, but possibly as another trait).
The code that would be generated would be of this form:
impl Default for T {
    fn to_default(&mut self) {
        self.x = Default::default();
        // ... other fields ...
    }
}

This is distinct from just swapping (via mem::swap) an existing instance because I want to alter the existing instance specifically, effectively hollowing it out (see example existing use case).
Alternatives
Users could continue to write this code themselves fairly simply themselves.
Alternatively, someone could create a small crate that reflects this functionality -- in writing an unrelated library it was fairly simple to write a macro that did this, so I could release that as an independent crate, and call the trait ResetDefault/ToDefault or something like that.
Existing use-cases

A crate I wrote, async-dropper, achieves this via a derive macro, and it's useful in particular because the code that runs on Drop branches on equality to the Default instance. If you used a simple mem::swap here you'd swap the full instance with empty instances forever.

The default implementation in the trait can be written as fn to_default(&mut self) { *self = Self::default() }, right? And the only motivation to derive this with the fields instead of using the default implementation is to avoid calling the Drop impl?
But the problem is, is it really desirable to be able to reset all fields to the default value without calling Drop? Automatically doing so for all existing std Default derive macro calls may break some assumptions in traits that explicitly rely on running Drop but still derived Default. For example consider a simplified Vec implementation:
#[derive(Default)]
struct ByteVec {
    ptr: Option<NonNull<u8>>,
}
impl Drop for ByteVec {
    fn drop(&mut self) {
        if let Some(ptr) = self.ptr {
            drop_items(ptr);
            free(ptr);
        }
    }
}

Now, adding a to_default() method that simply sets self.ptr = None; would leak the allocated buffer.
Since mem::forget() is safe, addition of to_default should not expose new possibilities of UB, but it may lead to unintended leaking.

By the way, referring to https://rust-lang.github.io/api-guidelines/naming.html, to_* should be used for &T -> U methods, not &mut T -> (). A more appropriate name would be set_default. And to avoid polluting the prelude namespace, this should not be part of the Default trait since Default is in the prelude.

Hey @SOF3 Thanks for the detailed consideration -- I think you've captured the gist & details accurately.

But the problem is, is it really desirable to be able to reset all fields to the default value without calling Drop? Automatically doing so for all existing std Default derive macro calls may break some assumptions in traits that explicitly rely on running Drop but still derived Default. For example consider a simplified Vec implementation:

Thanks for noting, this is a great point and it is indeed troublesome -- with your example it's pretty clear that automatically doing it for all Default derive macros is too risky given this approach.
Users that write code which must make an assumption of a custom Drop function should probably not be using this, and making that clear in set_default() seems a bit onerous, but not completely unheard of.
I'm unaware if there's any way to ensure that using set_default() could be disabled for custom implementations of Default, but if there isn't, then it could definitely become a tripping point for people writing code with custom Drop implementations that they expect to be called.
That said, I think this is something users would specifically look to do (when looking to re-use/repopulate some object) rather than at most situations.
Maybe the somewhat niche aspect of this makes it a much better fit for a standalone derive macro crate.

By the way, referring to https://rust-lang.github.io/api-guidelines/naming.html, to_* should be used for &T -> U methods, not &mut T -> (). A more appropriate name would be set_default. And to avoid polluting the prelude namespace, this should not be part of the Default trait since Default is in the prelude.

I definitely agree here, since the name I used in my code is reset_to_default. I checked the API guidelines but was torn between to_ and into_ rather than considering set.
It looks like if this shouldn't be in the prelude namespace and Default isn't the right place, then this is another sign (to me) that it belongs in a crate -- adding a new Trait to the stdlib with only this functionality feels wrong to me.

Very open to hearing more comments, but closing this for now so we can have one less issue on rust-lang/rfcs!


build: add CMAKE_INSTALL_PREFIX length warning

Linux mandates a limit on the length of a command that may appear in
the "shebang" line of a script. In our internal environments, this
limit has been exceeded, causing ill-behaved builds.
Show a warning if the length of CMAKE_INSTALL_PREFIX exceeds the
127 character limit.

Sure, what did you have in mind?

Coverage remained the same at 66.067% when pulling 65ec9023e620f34e7c300f1d0492e470efaf460c on tswhison/prefix_check into 30bbaf6c511458eb58025645351422c0eff02203 on master.


bugs of integer division and subplot for one class

Thanks for your work! It's really handy, saving me a lot of time when learning YOLO in keras :)
I understand this program is for python 3. I run it with python 2.7 though, and a small modification will get it work.
In File "mean_average_precision/mean_average_precision/utils/bbox.py", function jaccard returns 0 when the coordinates of bounding boxes are all integers, because python 2.7 yields an integer for integer division. Modifying line 49 from
return inter / union
to
return inter.astype('float32') / union
works well.
Second, when calculating mAP for just one class, in File "mean_average_precision/mean_average_precision/detection_map.py", function plot raises an error
Traceback (most recent call last):
  File "mean_average_precision/detection_map.py", line 229, in plot
    for i, ax in enumerate(axes.flat):
AttributeError: 'AxesSubplot' object has no attribute 'flat'

because when grid == 1, fig, axes = plt.subplots(nrows=grid, ncols=grid) return axes as type of matplotlib.axes._subplots.AxesSubplot.
I modify it from
        grid = int(math.ceil(math.sqrt(self.n_class)))
        fig, axes = plt.subplots(nrows=grid, ncols=grid)
        mean_average_precision = []
        # TODO: data structure not optimal for this operation...
        for i, ax in enumerate(axes.flat):
            if i > self.n_class - 1:
                break
            precisions, recalls = self.compute_precision_recall_(i, interpolated)
            average_precision = self.compute_ap(precisions, recalls)
            self.plot_pr(ax, i, precisions, recalls, average_precision)
            mean_average_precision.append(average_precision)

to
        grid = int(math.ceil(math.sqrt(self.n_class)))
        if grid == 1:
            fig, ax = plt.subplots()
            mean_average_precision = []
            precisions, recalls = self.compute_precision_recall_(0, interpolated)
            average_precision = self.compute_ap(precisions, recalls)
            self.plot_pr(ax, 0, precisions, recalls, average_precision)
            mean_average_precision.append(average_precision)
        else:
            fig, axes = plt.subplots(nrows=grid, ncols=grid)
            mean_average_precision = []
            # TODO: data structure not optimal for this operation...
            for i, ax in enumerate(axes.flat):
                if i > self.n_class - 1:
                    break
                precisions, recalls = self.compute_precision_recall_(i, interpolated)
                average_precision = self.compute_ap(precisions, recalls)
                self.plot_pr(ax, i, precisions, recalls, average_precision)
                mean_average_precision.append(average_precision)

to get it work, but I think it can be more precise .

Hi, thanks for the bug report! I will fix this as soon as possible!
I am quite busy right now but it should be pushed somewhere around next week.

Hi YuCosine,
The source code that you fixed it run, but the result still wrong when we only predict one class.
For example:
pred_bb1 = np.array([])
pred_cls1 = np.array([],dtype=int)
pred_conf1 = np.array([])
gt_bb1 = np.array([[0.86132812, 0.48242188, 0.97460938, 0.6171875],
[0.18554688, 0.234375, 0.36132812, 0.41601562],
[0., 0.47265625, 0.0703125, 0.62109375],
[0.47070312, 0.3125, 0.77929688, 0.78125],
[0.8, 0.1, 0.9, 0.2]])
gt_cls1 = np.array([1, 1, 1, 1, 1,1 ])
Because the model didn't predict object so Mean Average Precision = 0, but we get Mean Average Precision = 1

@TrungHieu-Le
With only one class, the gt of class label should be 0.
I modify the last line to gt_cls1 = np.array([0, 0, 0, 0, 0,0 ])  and get MAP=0

@MathGaron
Thank for your help.
Have a nice weekend!

Actually @TrungHieu-Le  did all the work for now! Thanks again!


Fix GBM color format

I fixed the color format value because the value specified in gbm_surface_create was incorrect.  I don't know why but it worked on RPI4 etc.
See: https://www.khronos.org/registry/EGL/extensions/MESA/EGL_MESA_platform_gbm.txt
Related issues

https://github.com/sony/flutter-embedded-linux/issues/4

@taka-okayama Will you review this PR?

@HidenoriMatsubayashi
I have a question about pixel format.
Why do you choose GBM_FORMAT_ARGB8888 format? This format has ALPHA component, so this effects alpha blending in every rendering frames. I think  GBM_FORMAT_XRGB8888 is a better choice than ARGB such as EGL_MESA_platform_gbm examples.

Because Flutter Embedder uses Alpha. See: https://github.com/sony/flutter-embedded-linux/blob/master/src/flutter/shell/platform/linux_embedded/surface/context_egl.cc#L23. Also, Flutter UI planes can overlap with other planes such as Video.

Because Flutter Embedder uses Alpha. See: https://github.com/sony/flutter-embedded-linux/blob/master/src/flutter/shell/platform/linux_embedded/surface/context_egl.cc#L23. Also, Flutter UI planes can overlap with other planes such as Video.

I got. thank you.


How to resolve a require string within a custom webpack loader?

I'm writing a custom loader for webpack.
Within my loader I've got a require string like this:
"css!less!./myFile.less"

Is there a way to get the output of the resolved request?
I'm looking for the output of the module after all loaders have been applied. Or in other words:
How do I get the compiled css from the string above?
I tried to use this.resolve on the loader context:
this.resolve(this.context, "css!less!./myFile.less", function(err, result){
     // Best case scenario so far:
     // result == "./myFile.less"

     // How do I get the css from myFile.less here?
     // Is that even possible/the right way to get this?
});

But I can't seem to get the resolved output. I'm probably doing something wrong, the only documentation I've found about this function is here: http://webpack.github.io/docs/loaders.html#resolve
I also asked this question on StackOverflow but didn't get any views:
http://stackoverflow.com/questions/32383195/how-to-resolve-a-require-string-within-a-custom-webpack-loader

this.loaderModule(context, request, callback: function(err, source))

@greenish Did the tip work for you?

@sokra and when the loader is in sync mode, will be the loadModule call sync too?

Closing. Free free to re-open at Stack Overflow if still relevant.

Recently needed this and still can't figure out how to use this.loadModule to get the raw CSS in my custom loader. Here's the code:
var loaderUtils = require("loader-utils");

module.exports = function (content) {
  this.cacheable();
  var cb = this.async();

  // Produces the following URL:
  // !!/Users/marek/Projects/displays/node_modules/css-loader/index.js!/Users/marek/Projects/displays/node_modules/postcss-loader/index.js??ref--3-3!/Users/marek/Projects/displays/node_modules/sass-loader/lib/loader.js??ref--3-4!/Users/marek/Projects/display-wall/src/scss/main.scss
  var url = "!!" + loaderUtils.getRemainingRequest(this);

  var self = this

  this.loadModule(url, function (err, source, map, module) {
    if (err) { return cb(err); }

    // How to access the evaluated module here to read CSS loader's output?

    // This throws "module not found" errors with relative paths in it. I'm npm linking shared dependency, may that be the problem?
    // console.log(self.exec(source, url))

    // Need to return raw CSS here from the source
    cb(null, "body {background: black;}")
  });
}

@mareksuscak Did you ever figure out how to eval the module?
Your code works to load it (or at least it doesn't throw an error), but I still can't figure out how to execute it either :-(

@mnpenner Whoa, it's a 3 years old comment but as far as I remember, I did although I did not write my own loader in the end. I used string-replace-loader  instead which was all I needed.


Disable aws on ci

AWS CI is slowing us down. For now we're going to disable it, until we can make the cleanup work better ... and ideally get the builds to work faster (we went from ~10m to ~25m build times).
More info is here:
https://github.com/pachyderm/pachyderm/issues/2109

LGTM


Route Constraint not working

This example:
@page "/Users/{Id:int}"

The user Id is @Id!

@code {
[Parameter]
public int Id { get; set; }
}
doesn't work on /Users?id=1
if I remove parameter and use just /Users and configure some default value for Id like public int Id { get; set; } = 2; , then the route works. Just whenever use the parameter, it fails

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

ID: 79da574a-570f-04eb-86bd-c8065a016f44
Version Independent ID: ac771374-6102-b838-57d2-e99bcf3f7d3e
Content: ASP.NET Core Blazor routing
Content Source: aspnetcore/blazor/routing.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

That's a query string. Do this .......
/Users/1

I found an issue in engineering that says they'll probably work on making working with query strings easier ...
https://github.com/aspnet/AspNetCore/issues/11977
Feel free to comment on that issue if you want to give them additional feedback.
I also see some devs talking about it here ...
https://stackoverflow.com/questions/53786347/multiple-query-string-parameters-in-blazor-routing

Oops, you are correct. I was porting a razor page to blazor and I forgot the routing convention change from ?id=1 to /1
By the way, I think few others online also missed this. Their solution is using Microsoft.AspNet.Core.WebUtilities for querystring. Some references: https://www.mikesdotnetting.com/article/340/working-with-query-strings-in-blazor
Probably good to mention the routing convention change somewhere. I recall it was mentioned in one of the preview updates before 3.0
Thanks again!


How to add an object to cache

Hi I want to change this proxy in a way that I can add an object from outside, for example I have a complete object [headers and content] how I can add to the cache that it appears as if it was cached in normal way and all subsequent requests for this object would be served from cache?

That's a support request, not a bug.  Please use the mailing list.


RN2903 Stuck on SF10

Been using your library for a while now and the transmissions will not change from SF10, no matter the distance I am from a gateway or the gateway which I use. I don't know the exact cause of this, but I can't seem to find anything in my code which indicates this. I am located in Australia and use the AU915 frequency plan.
My hardware setup consists of an Adafruit WICED Feather Developer Edition with a LoRaWAN FeatherWing add-on board
For reference, my code is below:
/*
 * Author: JP Meijers
 * Date: 2016-09-07
 *
 * This program is meant to be used on a The Things Uno board.
 *
 * Transmit a one byte packet via TTN. This happens as fast as possible,
 * while still keeping to the 1% duty cycle rules enforced by the RN2483's built
 * in LoRaWAN stack. Even though this is allowed by the radio regulations of the
 * 868MHz band, the fair use policy of TTN may prohibit this.
 *
 * CHECK THE RULES BEFORE USING THIS PROGRAM!
 *
 * CHANGE ADDRESS!
 * Change the device address, network (session) key, and app (session) key to
 * the values that are registered via the TTN dashboard.
 * The appropriate line is "myLora.initABP(XXX);" or "myLora.initOTAA(XXX);"
 * When using ABP, it is advised to enable "relax frame count" on the dashboard.
 *
 */
#include <rn2xx3.h>

//create an instance of the rn2483 library, using the given Serial port
rn2xx3 myLora(Serial1);

// the setup routine runs once when you press reset:
void setup()
{
  pinMode(PC5, OUTPUT);
  digitalWrite(PC5, HIGH);

  // Open serial communications and wait for port to open:
  Serial.begin(57600); //serial port to computer
  Serial1.begin(57600); //serial port to radio

  // make sure usb serial connection is available,
  // or after 10s go on anyway for 'headless' use of the
  // node.
  while ((!Serial) && (millis() < 10000));

  Serial.println("Startup");

  initialize_radio();

  //transmit a startup message
  myLora.tx("TTN Mapper on TTN Uno node");
}

void initialize_radio()
{
  delay(100); //wait for the RN2xx3's startup message
  Serial1.flush();

  //print out the HWEUI so that we can register it via ttnctl
  String hweui = myLora.hweui();
  while(hweui.length() != 16)
  {
    Serial.println("Communication with RN2xx3 unsuccessful. Power cycle the TTN UNO board.");
    delay(10000);
    hweui = myLora.hweui();
  }
  Serial.println("When using OTAA, register this DevEUI: ");
  Serial.println(hweui);
  Serial.println("RN2xx3 firmware version:");
  Serial.println(myLora.sysver());

  //configure your keys and join the network
  Serial.println("Trying to join TTN");
  bool join_result = false;

  //ABP: initABP(String addr, String AppSKey, String NwkSKey);
  join_result = myLora.initABP("", "", "");

  while(!join_result)
  {
    Serial.println("Unable to join. Are your keys correct, and do you have TTN coverage?");
    delay(10000); //delay a minute before retry
    join_result = myLora.init();
  }
  Serial.println("Successfully joined TTN");

}

// the loop routine runs over and over again forever:
void loop()
{
    Serial.println("TXing");
    myLora.tx("!"); //one byte, blocking function
    delay(2000);
}

ADR occurs after 20 uplinks.


Validation is failing constantly

Hi @TimothyMeadows ,
I am trying to use your package for recaptcha on my site.
Microsoft.AspNetCore- 2.2.0
Recaptcha.aspnetcore - 3.0.10
I have followed your example as is.  This is what I have in my controller:
        [ValidateRecaptcha]
        public async Task<ActionResult> Contact_Form(Feedback fb)
        {
           var recaptcha = await _recaptcha.Validate(Request);
            if (!recaptcha.success)  //  || recaptcha.score != 0 && recaptcha.score < _minimumScore)
                ModelState.AddModelError("Recaptcha", _errorMessage);

And this is in my View:

                    @(Html.Recaptcha<RecaptchaV2Checkbox>(RecaptchaSettings.Value))
                     @Html.ValidationMessage("Recaptcha")

```
`

This is in my Startup.cs file
```
            // Add recaptcha and pass recaptcha configuration section
            services.AddRecaptcha(Configuration.GetSection("RecaptchaSettings"));

```
   Whether I click on the recaptcha checkbox or not, the 'recaptcha.success' value is always false.  can you help resolve this please?  Am I doing something wrong?

Thank you

try disabling async
@(Html.Recaptcha<RecaptchaV3HiddenInput>(RecaptchaSettings?.Value, new RecaptchaV3HiddenInput { IsAsync = false }) )


Link to gamja for web chat

gamja offers a better experience than Kiwi.
Obviously I'm 100% biased. Completely fine to not make the switch if someone feels differently.

gamja dev is a cool dude, i'll accept the bias :p

ill merge after rebase

Rebased!


Clean up SVG to drawable conversion code and output format

Very honored to have other developers to participate in the construction of the project 👍
I want to refactor these terrible codes for a long time but have no time, thank you for your contribution.
BTW, have you tested the codes in your PR? if yes, I will merge it and release a new version as soon as possible

Maybe we can write more test cases, and use the CI tool like Travis to execute it after every commit 😄


Fix empty shapes layer duplication and Convert to Labels enablement logic for selected empty shapes layers

Description

Enable duplicating empty shapes layer and prevent enabling Convert to Labels option from layers context menu if an empty shapes layer is selected:

Type of change

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

References

closes #5549
How has this been tested?

[x] all tests pass with my change
[x] I check if my changes works with both PySide and PyQt backends
as there are small differences between the two Qt bindings.

Final checklist:

[x] My PR is the minimum possible work for the desired functionality

Some conflicts need solving before we can merge.


Proper build pipeline

We should use Github Actions to test, build and deploy the package to pypi instead of doing it manually
https://github.com/pypa/gh-action-pypi-publish

Build dist, think about where to set the version
Deploy any PR to test pypi with version 0.{#PR} or some other incremental counter
Deploy any release to pypi with given tag

https://medium.com/@vanflymen/blazing-fast-ci-with-github-actions-poetry-black-and-pytest-9e74299dd4a5

Requirements:

static versioning
pull requests should automatically deploy on test.pypi.org after approval
github releases should automatically deploy on pypi.org

Pull Request Opened

Run Black, Flake8, tests
Create a temp version to publish on test.pypi.org: Fork desired version: verison = version-pr.1, e.g. Pull Request 3: 1.0.4 → 1.0.4-pr.3
Run poetry version $VERSION to temporarily change pyproject.toml
Build package poetry build
Publish package poetry publish
Post link about deployed package in PR

Release/Publish

It doesn't matter what tag we specify, it could be unique but it could also correlate with the one in pyproject.toml
Run Black, Flake8, tests
Build package poetry build
Publish package poetry publish

Todo: poetry support, black, flake8, http://mypy-lang.org/

see PRs and https://github.com/filipre/signalbot/deployments


neo4j: 5.22.0 -> 5.24.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/neo4j/versions.
meta.description for neo4j is: Highly scalable, robust (fully ACID) native graph database
meta.homepage for neo4j is: https://neo4j.com/
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.24.2 with grep in /nix/store/ix1lqxpawpn0h41vywvznrg44h39j914-neo4j-5.24.2
found 5.24.2 in filename of file in /nix/store/ix1lqxpawpn0h41vywvznrg44h39j914-neo4j-5.24.2

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

neo4j

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ix1lqxpawpn0h41vywvznrg44h39j914-neo4j-5.24.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A neo4j https://github.com/r-ryantm/nixpkgs/archive/dd450db253afd0f0b7669726b0c9b967d8f39eb4.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/dd450db253afd0f0b7669726b0c9b967d8f39eb4#neo4j

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ix1lqxpawpn0h41vywvznrg44h39j914-neo4j-5.24.2
ls -la /nix/store/ix1lqxpawpn0h41vywvznrg44h39j914-neo4j-5.24.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

neo4j

Maintainer pings
cc @offlinehacker for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 349164

aarch64-darwin

:white_check_mark: 1 package built:

neo4j


[BACKLOG-2588] Introduce versioning by file that implements by content t...

...ype

Should we deprecate the original methods instead of replacing them?


Add CNAME

@carderne What will the final url be? africa-grid.energydata.info?

africagrid.energydata.info has already been forwarded to Thao, so let's stick with that.


Make actor.Stop respect the given exit reason

Requires https://github.com/gleam-lang/erlang/pull/65
The actor would stop with Normal before this.

I noticed afterwards, that there's some overlap between this and https://github.com/gleam-lang/otp/pull/86  Sorry for not spotting it earlier. I'll take a closer look at what was done there later.


fix: validate queue parameters more strictly

Updated from Kong:

max_entries and max_bytes must be integers
initial_retry_delay and max_retry_delay must be greater than zero

I see more stuff moving

yes; the metaschema.lua file also was updated in the same kong commit, to allow for number values on the between options.  but as usual, there were several other changes to that file that we didn't have, and in order to minimise the diff between kong and this, i just copied the whole file.


HD44780_i2c blue LCD not properly working on MAX2PLAY connected to seperate LMS

Hello Dhrone,
i installed latest pydPiper on latest MAx2play. Display is connected to 0x27 on port 1. LMS is connected, confirmed by log file.
After "pydPiper Starting..." the nice clock appears and thats all. When shutting down the message "Exiting.." appears.
No Information of song or other infos are displayed.
Please see last entry of log:
2020-07-22 12:49:37,897:INFO:Exiting... 2020-07-22 12:51:41,414:INFO:pydPiper starting 2020-07-22 12:51:41,559:DEBUG:Loading display controller 2020-07-22 12:51:41,559:DEBUG:Loading music controller 2020-07-22 12:51:41,573:DEBUG:LMS musicdata service starting 2020-07-22 12:51:41,573:DEBUG:Connecting to LMS service on <IP_ADDRESS>:9090 2020-07-22 12:51:42,257:DEBUG:Connected to LMS using player YamahaPi 2020-07-22 12:51:42,391:DEBUG:Connecting to LMS raw service on <IP_ADDRESS>:9090 2020-07-22 12:51:43,576:DEBUG:Music Controller Starting 2020-07-22 12:51:43,577:DEBUG:Loading pages_lcd_20x4.py as page file 2020-07-22 12:51:43,579:DEBUG:Initializing current system status update process.  System status will update every five minutes 2020-07-22 12:51:43,621:DEBUG:Weather service not enabled 2020-07-22 12:51:43,674:DEBUG:Weather service not enabled 2020-07-22 12:51:43,714:DEBUG:System status: Temp 53Â°c, Disk space remaining 74%, IP address <IP_ADDRESS> 2020-07-22 12:51:43,778:DEBUG:Loading image progbar **2020-07-22 12:51:43,806:DEBUG:v out of range with value -1.  Should have been between 0 and 100**
Hopefully you can point where my fault is..
Thanks in advance
Klaas

It looks like the LMS data service is successfully connecting.  I'd have to download the current Max2play to see if there has been some change to the way it is transmitting LMS data.  Can you confirm the version of Max2Play and the version of LMS you are running?

I installed 249 for PI 3, not Pi4.
https://www.max2play.com/max2play-image/
LMS Version 8 -<PHONE_NUMBER> @ Sat Jul 11 09:25:03 CEST 2020
·         Betriebssystem: piCore - DE - utf8
·         Plattformarchitektur: armv8l-linux
·         Perl-Version: 5.28.2 - arm-linux-gnueabihf-thread-multi-64int
·         Audio::Scan: 1.02
·         IO::Socket::SSL: 2.066
·         Datenbankversion: DBD::SQLite 1.58 (sqlite 3.22.0)
Regards
Klaas
Von: dhrone<EMAIL_ADDRESS>Gesendet: Mittwoch, 22. Juli 2020 15:18
An: dhrone/pydPiper
Cc: Klaasemann; Author
Betreff: Re: [dhrone/pydPiper] HD44780_i2c blue LCD not properly working on MAX2PLAY connected to seperate LMS (#104)
It looks like the LMS data service is successfully connecting. I'd have to download the current Max2play to see if there has been some change to the way it is transmitting LMS data. Can you confirm the version of Max2Play and the version of LMS you are running?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/dhrone/pydPiper/issues/104#issuecomment-662447461 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AQLXMYU4RKZVJCZBXUPHZXLR43RIBANCNFSM4PEV5LFQ .  https://github.com/notifications/beacon/AQLXMYQIR3RIN3MCDVU57DLR43RIBA5CNFSM4PEV5LF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOE56CKZI.gif

Hello,
last weekend i just created a new fresh installation based on latest version (249).
Then followed the instructions. Now its working. The Titles and infos are shown.
But:
After some title changes and using snchronisation, i just see the clock.
This is the main player and after a reboot it works again..
Do you have an idea ?
Regards
Klaas

Usually this type of behavior occurs when their is a problem receiving metadata from the LMS server.  The time issue is probably unrelated.

Hello,
i tried the installation prezess several times on Raspi 2 and 3b+ with hd44780 20x4 on max2play 249.
pydPiper works properly just on devices where LMS is locally installed on. Showing clock and showing text of playing tracks.
pydPiper on a LMS client does just show the clock. No text.
I just got following errorlog on ALL clients:
2020-08-19 17:22:16,595:ERROR:LMS connect failure
Traceback (most recent call last):
File "/app/sources/musicdata_lms.py", line 96, in connect
self.dataserver.connect()
File "/usr/local/lib/python2.7/dist-packages/pylms/server.py", line 58, in connect
self.telnet_connect()
File "/usr/local/lib/python2.7/dist-packages/pylms/server.py", line 66, in telnet_connect
self.telnet = telnetlib.Telnet(self.hostname, self.port)
File "/usr/lib/python2.7/telnetlib.py", line 209, in init
self.open(host, port, timeout)
File "/usr/lib/python2.7/telnetlib.py", line 225, in open
self.sock = socket.create_connection((host, port), timeout)
File "/usr/lib/python2.7/socket.py", line 571, in create_connection
raise err
Any idea hint ?
Regards
Klaas

The problem is in the pylms library but its not clear why the connection request is failing.  Unfortunately, I have not had time to load the current max2play client to see what may have changed.

Can you tell me on wich distro it is working ?`
Thanks in adcance
Von: dhrone<EMAIL_ADDRESS>Gesendet: Montag, 14. September 2020 03:33
An: dhrone/pydPiper
Cc: Klaasemann; Author
Betreff: Re: [dhrone/pydPiper] HD44780_i2c blue LCD not properly working on MAX2PLAY connected to seperate LMS (#104)
The problem is in the pylms library but its not clear why the connection request is failing. Unfortunately, I have not had time to load the current max2play client to see what may have changed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/dhrone/pydPiper/issues/104#issuecomment-691762929 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AQLXMYWBITMAUP2OAU3CPMLSFVXDJANCNFSM4PEV5LFQ .  https://github.com/notifications/beacon/AQLXMYRDLWKXV7BWBSK2IITSFVXDJA5CNFSM4PEV5LF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFE5XN4I.gif

This appears to be the same issue as #109.  Are you attempting to connect to an LMS service located on a different server?

Hi Dhrone,
Sorry for all my recent questions. I have been trying to get this to run on max2play to see if that will serve my needs, but seem to be having an issue with getting anything to display at all. Running Max2play (version 2.55) on pi4 2gb, and have LMS on a second pi (IP <IP_ADDRESS>).
I have cloned pydpiper from main branch, and run the install docker script, and then ran the configure.py script, and copied the service file to the systemd/system/ folder and enable and start pydpiper.  Both pydpiper and docker are showing as running using sudo systemctl status. Attached are my service and cfg files.
pydpiper.service.txt
pydPiper.cfg.txt
For the service file, I have tried it with both options enabled for the exec start (and ensured copied to /etc/systemd/system/). I note in the service file, in the [Unit] section, it states 'After = volumio.service' I assume this should be changed as I am not running volumio, although it didn't affect anything when I ran pydpiper on Moode.
When I run the following sudo /usr/bin/docker run --network=host --privileged -v /var/log:/var/log:rw -v /home/pi/pydPiper:/app:rw dhrone/pydpiper:latest python /app/pydPiper.py I get the following error:
python /app/pydPiper.py
pydPiper starting
Uncaught exception
Traceback (most recent call last):
File "/app/pydPiper.py", line 739, in
lcd = displays.hd44780_i2c.hd44780_i2c(rows, cols, i2c_address, i2c_port, enable)
File "/app/displays/hd44780_i2c.py", line 114, in init
self.bus = smbus.SMBus(i2c_bus)
IOError: [Errno 2] No such file or directory
Traceback (most recent call last):
File "/app/pydPiper.py", line 739, in
GPIO not installed
lcd = displays.hd44780_i2c.hd44780_i2c(rows, cols, i2c_address, i2c_port, enable)
File "/app/displays/hd44780_i2c.py", line 114, in init
self.bus = smbus.SMBus(i2c_bus)
IOError: [Errno 2] No such file or directory
After reading issue #106 I ran sudo docker container ls -a and that just shows a large list of containers which are created but keep exiting. Assuming it is to do with max2play, as it works beautifully on volumio using same install methods.
Thanks for all your effort and apologies for my many questions.


Refactor Explore page

Combine the 4 panels we have now: Files, Helm Charts, Kustomizations, Images into a single Explorer page that has all inventories as an accordion.

Thomas was always working on this.

Data structures are different, so we will need some implememtation work

Closing this in favor of #3071


feat(pm_list): [Trustpay] add bank_redirect -  blik pm type required field info for trustpay

Type of Change

[ ] Bugfix
[ ] New feature
[ ] Enhancement
[ ] Refactoring
[ ] Dependency updates
[ ] Documentation
[ ] CI/CD

Description

Add bank_redirect -  blik pm type required field info for trustpay , and made blik_code field as optional for blik bank redirect data
Additional Changes

[ ] This PR modifies the API contract
[ ] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

Checklist

[x] I formatted the code cargo +nightly fmt --all
[x] I addressed lints thrown by cargo clippy
[x] I reviewed the submitted code
[ ] I added unit tests for my changes where possible
[ ] I added a CHANGELOG entry if applicable

Regenerate openapi spec

Please create an issue for this


Can the EntryId field added to identity the Cron job be please merged back to master

Can  you please merge the entryId changes from V2 to Master ?

My concern is about breaking backwards compatibility, which is why I made that a separate branch. I'm not really sure the best long term solution

@sumeshsoman see https://github.com/robfig/cron/issues/84#issuecomment-312231489, that may solve your issue.

The v3 branch will be the last version of cron.  I'm updating it with all of the fixes and upgrades between master and v2 and to work with Modules. Please have a look and let me know if there is anything lacking there.


Scan API - support for exception rules

Overview
Even the latest version of API (V2 API on Kubescape v3.0.3) doesn't let to send scan exclusions as a part of HTTP request
Valid model arguments:
account | [...]
excludedNamespaces | [...]
failThreshold | [...]
complianceThreshold | [...]
format | [...]
hostScanner | [...]
includeNamespaces | [...]
keepLocal | [...]
submit | [...]
targetNames | [...]
targetType | [...]
useCachedArtifacts | [...]

all while CLI client has --exceptions argument
Solution
Enable a new field in the model to accept JSON with exceptions:
https://github.com/kubescape/kubescape/blob/master/examples/exceptions/exclude-allowed-hostPath-control.json

I would like to work on this issue

@dwertent I am having trouble solving this issue and don't know where to start can you give me some resources to solve this issue so that I can learn and solve these kinds of issue in the future by myself.

we need to add
Exceptions      []armotypes.PostureExceptionPolicy  // list of exceptions to apply on scan results
field in type PostScanRequest
in https://github.com/kubescape/opa-utils/edit/main/httpserver/meta/v1/datastructure.go

I would like to take on this issue please assign this to me

Hey @matthyx @RomanTheLegend @Sriramkashyapks i would like to work on this issue, is the issue still up for a grab?

Hey @matthyx @RomanTheLegend @Sriramkashyapks i would like to work on this issue, is the issue still up for a grab?

Sure propose a PR and I'll review it


AfterUpdate/AfterSave missing data?

Your Question
Is it expected that the data in the AfterUpdate/AfterSave hooks only have the changed fields? I'm looking to provide real time updates of data, should I use the hook to perform a find after save to get the full data or just always include the primary key value in the update so I have something to merge on?
I would expect that at least primary keys would be populated.

The document you expected this should be explained
https://gorm.io/docs/hooks.html

Expected answer

What data should or should not be present to the hook function.

It can only have the changed field or it requires another query SQL.

It can only have the changed field or it requires another query SQL.

Newbie question: in AfterUpdate how do get the original record?
Say I have something like

func (newUser *User) AfterUpdate(tx *gorm.DB) (err error) {
  // what should go here to get the original record?
}

I'm calling something without an ID in my update
db.Model(&model.User{}).Where("id = ?", 3).Updates(model.User{Age: 30})


Is it possible to call db.end() after every query?

For example:
db.query('SELECT blah blah here ', function(err, results){

		db.end();
	});

Except, can we remove db.end() and put it inside the main db class or something of sort so it always ends the connection after each query automatically?
Thanks!

Unfortunately there is not enough context to really say anything about this besides for the obvious: you can only ever call .end on a connection once. You may want to look into using the Pool and just calling pool.query(...)? Otherwise, please feel free to add more details to the question :)

Well, Im just sick and tired of using 'db.end()' each time inside a db.query function. Is there a way to just edit the main MYSQL files inside \apps\node_modules\mysql\lib and just make it so each time a query is run, it calls db.end() automatically? Hard to explain, thanks.

This is because if I do not call db.end I get a sleep query (memory leak) in my mysql pool, which I do not like :(

Untested, but you can make use of something like this:
var query = db.prototype.query;
db.prototype.query = function(){
    var args = Array.prototype.slice.call(arguments);
    var callback = args.pop();

    args.push(function(){
        db.end();
        callback.apply(this,arguments);
    });

    query.apply(this,args);
};

I just use the query pooling though

If you are using a framework like Hapi you could use a plugin like https://github.com/salesflare/hapi-plugin-mysql that handles the connection endng for you.

Gotcha. So the intention is that for every query, you want to open a brand new connection to your MySQL server, make the single query, and then close down the connection? You can certainly do this, but just remember that this is basically a "driver library" for Node.js, and as such, it does require that you manage the lifecycle of connections yourself.
@EnzoMartin gave a pretty simply example :) Another one could be the following:
var mysql = require('mysql');
var db = {};
db.query = function (sql, callback) {
  var _err;
  var _rows;
  var conn = mysql.createConnection();
  conn.query(sql, function (err, rows) {
    _err = err;
    _rows = rows;
  });
  conn.end(function () {
    callback(_err, _rows);
  });
};

and now you can just call db.query('SELECT something', function (err, rows) {}) anywhere and it'll create, connection, query, and end every time.

Not necessarily doug, I do not want to open a new one, but just close the current one if that makes sense.

Sure, but where are you opening the connection?

I'm just opening it whenever a user connects to my gameserver to let's just say to move an item in their inventory. (changes a slot value).
my javascript sends: 'SU 421 21'
Slot update command get's read by nodejs, goes to my function updateSlot. I run db.query and update their item ID (421) to slot position 21.  They do this each time they move an item, mysql is used and I end the connection each time. Is this bad?

It's not bad, it's just in Node.js this works very differently from, say, PHP. In PHP, you have a single script that would execute: it would start up, run your code, connect, query, and then when the script ends, it'll do things like clean up your db connections.
In Node.js, however, this is completely different. You are not writing scripts but a single long-running app that never ends. This means that if you open a new connection, it will never close automatically unless you stop your Node.js server.

I want that to happen with this module and it will be heaven. lol

The answer is that it is literally impossible to do this in Node.js, I'm sorry, I know you didn't want to hear that, but this module has no way to know when you're going to stop calling db.query() because Node.js is async and never ends.


Stop defaulting aws private haproxy external port to 6443

We want to remove the unncessary exposure of port 6443 externally, as this increases the used SG rules, which decreases our max number of HC per management aws account https://github.com/openshift/hypershift/pull/3139 We split the PR to roll out the HO first to all envs to make sure the haproxy in dataplane for newly created HCs don't point to a port 6443 that won't be exposed.

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/lgtm

{Failed  === RUN   TestUpgradeControlPlane/Main/EnsureNoCrashingPods
    util.go:422: Container olm-operator in pod olm-operator-5b7fcb87c5-694dk has a restartCount > 0 (1)
        --- FAIL: TestUpgradeControlPlane/Main/EnsureNoCrashingPods (0.02s)
}

/retest

/test e2e-kubevirt-aws-ovn

/retest

/retest

needs https://github.com/openshift/hypershift/pull/3144

/cherry-pick release-4.14


FIX-#45: Add What do these changes do section in PRs description

Signed-off-by: Igoshev, Yaroslav<EMAIL_ADDRESS>

Why your PR doesn't have this template? Is it enough to simply add the file?

Yes, the template will be available when we merge the PR into master branch.


aurelia-dialog no longer works with out-of-band v2.0.0

The body of dialogues sourced from aurelia-dialog are now empty.
I assume we need a 2.0.0 of aurelia-dialog too?

None of the plugins have been updated to the new shadow dom implementation since it's out-of-band and we don't want to break everyone on that. For now, you can fix the issue by diving into the source of the dialog's html files and changing each <content> element to a <slot> element. The dialog.html uses select which isn't part of the spec, but you could just replace those three with a single <slot> because that should work as long as you put your header, body footer in the right order, which makes sense anyway.

Issue moved to aurelia/dialog #156 via ZenHub.io

Ah, multiple <slot>s without a name!!!
I tried to change it using named slots yesterday but had no luck.
Thanks for that


Ballchasy - Bug fix & Progress

bcManager

updates score reporting to use asyncio.to_thread
adds an info.json to the cog

This is required an update to the bot's python version to be updated to a minimum of python 3.9.0

One comment on this report -- The "View Group" link for the tier does not appear in this example because of the 2 matches present, one match was previously reported, while replays were never found for the other, so the replay location traversal never occurred for the tier on this execution, and no tier group was populated at run time.
I believe this to be a minor issue worth noting, but not addressing.
If anyone believes otherwise, it will involve either refactoring the match cog saved data (injecting the group link into the tiers) or by executing the traversal before the replay search, which I believe would be a poor choice considering rate limiting limits.


Updating CW dashaboards for xaccounts

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Initiated a test event to validate the changes. I will update the issue once I finish. Thanks!

Thanks for the fixes submitted this morning. I left a couple of comments about expressions that need to be adjusted in the vpc cni dashboard. Once these fixes are committed, I am happy with the PR.
Thanks so much for this PR. Very appreciated!

Committed the required fix. The PR looks good to me.
Tagging Carlos for merging it @csantanapr.


Add PKGBIUILD and get on the AUR.

We could adapt the install_linux.sh script into a PKGBUILD and publish it on the AUR (not a hard process).
For those who don't know, the AUR stands for Arch User Repository and is an unofficial repository for Arch users to install things not in the official repositories.

I don't plan to publish it on the AUR yet.


Removing ansible-lint error from test-replica-node-affinity

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:

ansible-lint outputs
vijay@vijay-Satellite-C650:~/openebs/e2e/ansible/playbooks/hyperconverged/test-replica-node-affinity$ ansible-lint prerequisistes.yml
vijay@vijay-Satellite-C650:~/openebs/e2e/ansible/playbooks/hyperconverged/test-replica-node-affinity$ ansible-lint replica-node-affinity-cleanup.yml
vijay@vijay-Satellite-C650:~/openebs/e2e/ansible/playbooks/hyperconverged/test-replica-node-affinity$ ansible-lint replica-node-affinity-vars.yml
vijay@vijay-Satellite-C650:~/openebs/e2e/ansible/playbooks/hyperconverged/test-replica-node-affinity$ ansible-lint replica-node-affinity.yml
[ANSIBLE0013] Use shell only when shell functionality is required
/home/vijay/openebs/e2e/ansible/playbooks/hyperconverged/test-replica-node-affinity/replica-node-affinity-cleanup.yml:6
Task/Handler: Delete Percona deployment

[ANSIBLE0013] Use shell only when shell functionality is required
/home/vijay/openebs/e2e/ansible/playbooks/hyperconverged/test-replica-node-affinity/replica-node-affinity-cleanup.yml:26
Task/Handler: Delete the namespace

[ANSIBLE0013] Use shell only when shell functionality is required
/home/vijay/openebs/e2e/ansible/playbooks/hyperconverged/test-replica-node-affinity/replica-node-affinity-cleanup.yml:37
Task/Handler: Check whether the namespace is deleted

[ANSIBLE0013] Use shell only when shell functionality is required
replica-node-affinity.yml:49
Task/Handler: 2b) Create test specific namespace

[ANSIBLE0013] Use shell only when shell functionality is required
replica-node-affinity.yml:83
Task/Handler: 4) Deploying Percona Application

[ANSIBLE0013] Use shell only when shell functionality is required
replica-node-affinity.yml:180
Task/Handler: 8a) Delete the replica on one of the nodes

vijay@vijay-Satellite-C650:~/openebs/e2e/ansible/playbooks/hyperconverged/test-replica-node-affinity$ ansible-lint replica_patch.yml
vijay@vijay-Satellite-C650:~/openebs/e2e/ansible/playbooks/hyperconverged/test-replica-node-affinity$

@vijaysinghrawat1 Seems like duplicate PR.


Selecting recent files

What's the Problem?
Selecting recent files is something I'm really used to doing and I miss it
Solution/Idea
Being able to select recent files like normal files
Alternatives
Any other way of opening the context menu of more than one recent file
Files Version
<IP_ADDRESS>
Windows Version
Win 11 10.0.25179.1000
Comments
No response

+1 for this, Would like to be able to drag from recent files into software like I can in the native windows browser


Bees spawn at the bottom of the world and fall, dying

See attached video:
https://user-images.githubusercontent.com/64770632/107866835-cb646180-6e29-11eb-8877-cbc4f405e573.mp4
It was hard to tell exactly from my vantage point, but they spawn around y=0. (Using F3)

I thought I had checked all the issues regarding bees, but it seems that one introduced a commit that was their removal. I only have access to the latest release, so this wasn't obvious. Closing


Kasten K10 pattern construct added

Issue #, if available:
Description of changes: Pattern Construct added for Kasten K10.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Let's merge #26 then resolve conflicts and merge this one. The original Kasten should be closed.


Reindex operators

The concept of reindex from Jittor is very good.
Many operators are of the reindex family. Let's implement them.

[ ] Broadcast
[x] Slice
[x] Permute
[ ] Reshape
[ ] Embedding

Closed with #53


Fixed icon size

Is it possible settings fixed icon size after specific zoom level? I would like that the icons not scaling

@mike-000 I used a VectorTIleLayer

@mike-000 I used a VectorTIleLayer

Icons bigger than the tiles would not display correctly unless you use declutter.

@mike-000 ok thanks.


fix(TDI-38357):All Azure Blob components -> if no Prefix was filled in,

it will happen NPE exception.
What is the current behavior? (You can also link to an open issue here)
https://jira.talendforge.org/browse/TDI-38357
What is the new behavior?
Please check if the PR fulfills these requirements

[ ] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: success :star:

Pull request test result: success :star:

Pull request test result: success :star:


List all the partners

Wallets, Dapps, Protocols, Projects

Could you list out all the partners here? 🙌


TextMetrics.wordWrap - Don't add a newline if it's the last token/word

Fix for the issue I mentioned in https://github.com/pixijs/pixi.js/issues/4791.

Thanks @TheWyo and very helpful first commit 🎉 !

Would be cool to add a test for this

Good idea @brenwell, would welcome a PR for this.


it's possible to return an array in catchError()

Describe the bug
I have discovered that it's possible to return an array in catchError.
Expected behavior
Returning arrays shouldn't be possible.
catchError(() => [])
Reproduction code
import { tap, switchMap, throwError, catchError, of } from "rxjs";
of(1)
  .pipe(
    tap(console.log),
    switchMap(() => throwError(() => new Error())),
    catchError(() => []),
  )
  .subscribe(console.log);

Reproduction URL
https://codesandbox.io/p/devbox/catcherror-issue-ld2pd7?layout=%257B%2522sidebarPanel%2522%253A%2522EXPLORER%2522%252C%2522rootPanelGroup%2522%253A%257B%2522direction%2522%253A%2522horizontal%2522%252C%2522contentType%2522%253A%2522UNKNOWN%2522%252C%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522id%2522%253A%2522ROOT_LAYOUT%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522UNKNOWN%2522%252C%2522direction%2522%253A%2522vertical%2522%252C%2522id%2522%253A%2522cluaw6to000073b6lc47k8wmr%2522%252C%2522sizes%2522%253A%255B70%252C30%255D%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522EDITOR%2522%252C%2522direction%2522%253A%2522horizontal%2522%252C%2522id%2522%253A%2522EDITOR%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522EDITOR%2522%252C%2522id%2522%253A%2522cluaw6to000023b6lvkv3v5ag%2522%257D%255D%257D%252C%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522SHELLS%2522%252C%2522direction%2522%253A%2522horizontal%2522%252C%2522id%2522%253A%2522SHELLS%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522SHELLS%2522%252C%2522id%2522%253A%2522cluaw6to000043b6lqg5dww1y%2522%257D%255D%252C%2522sizes%2522%253A%255B100%255D%257D%255D%257D%252C%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522DEVTOOLS%2522%252C%2522direction%2522%253A%2522vertical%2522%252C%2522id%2522%253A%2522DEVTOOLS%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522DEVTOOLS%2522%252C%2522id%2522%253A%2522cluaw6to000063b6lo6ueuwrr%2522%257D%255D%252C%2522sizes%2522%253A%255B100%255D%257D%255D%252C%2522sizes%2522%253A%255B50%252C50%255D%257D%252C%2522tabbedPanels%2522%253A%257B%2522cluaw6to000023b6lvkv3v5ag%2522%253A%257B%2522id%2522%253A%2522cluaw6to000023b6lvkv3v5ag%2522%252C%2522tabs%2522%253A%255B%255D%257D%252C%2522cluaw6to000063b6lo6ueuwrr%2522%253A%257B%2522tabs%2522%253A%255B%257B%2522id%2522%253A%2522cluaw6to000053b6lq544m71b%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522type%2522%253A%2522TASK_PORT%2522%252C%2522taskId%2522%253A%2522dev%2522%252C%2522port%2522%253A5173%252C%2522path%2522%253A%2522%252F%2522%257D%255D%252C%2522id%2522%253A%2522cluaw6to000063b6lo6ueuwrr%2522%252C%2522activeTabId%2522%253A%2522cluaw6to000053b6lq544m71b%2522%257D%252C%2522cluaw6to000043b6lqg5dww1y%2522%253A%257B%2522id%2522%253A%2522cluaw6to000043b6lqg5dww1y%2522%252C%2522tabs%2522%253A%255B%257B%2522id%2522%253A%2522cluaw6to000033b6lnuuh5tgl%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522type%2522%253A%2522TASK_LOG%2522%252C%2522taskId%2522%253A%2522dev%2522%257D%255D%252C%2522activeTabId%2522%253A%2522cluaw6to000033b6lnuuh5tgl%2522%257D%257D%252C%2522showDevtools%2522%253Atrue%252C%2522showShells%2522%253Atrue%252C%2522showSidebar%2522%253Atrue%252C%2522sidebarPanelSize%2522%253A15%257D
Version
7.8.1
Environment
osx: 13.6.3 (22G436)
Additional context
No response

The issue can be closed, I recognize that this is a feature. An '[]' is ArrayLike

Indeed, it is a feature. It is designed to cover anything that can be converted to an Observable.
The interface for this operator is ObservableInput which includes ArrayLike - arrays can be converted to Observables.
Closing as requested.


Minimum cost to cut the stick

Problem Description: We are given a stick of length N and a CUTS array of size C. The stick has markings as shown, and the CUTS array depicts the marking at which the stick needs to be cut (shown in red).

Kindly please assign me this issue

Kindly please assign me this issue

@rishirohilla Done!

hi i am a second year student , paving my way into open source can please assign this issue to me @Saloni6111


Integration into androguard

Hi!
I would like to integrate the permission mappings into androguard, which currently only uses the mappings generated by PScout. We are also missing the newer mappings.
I think it should not be too hard to read the files from this repository, there are just minor adjustments for the path required, ie.
android.hardware.location.ActivityRecognitionHardware.disableActivityEvent(java.lang.String,int)boolean would be Landroid/hardware/location/ActivityRecognitionHardware;-disableActivityEvent-(Ljava/lang/String; I)B in the androguard defintion.
What do you think would be the best way to integrate axplorer into androguard? Maybe just as a git submodule? Idealy the permission mappings should be in a format which is easily readable from python, like json or pickle. In this case I would convert the text files from axplorer to this format at compile time.
For reference, there is already an issue in androguard: https://github.com/androguard/androguard/issues/252

Sorry for the delayed answer, don't have much time for programming atm. Great to hear that AndroGuard wants to adopt our mappings. I'm aware of the notation problem (JVM vs Java). For the export, I intentionally chose to normalize the JVM format to a human-readable format. In fact, WALA, the framework we're building on, uses a weird mix of notations. An option could be to provide the mappings in both notations.
Providing the mappings in an easier parsable format such as JSON is a good point.  As soon as I can spend some time on this, I'm going to implement a JSON export.  Similarly, I hope to be able to provide mappings for API level 26 soon.

Thanks for the feedback! I already integrated the mappings into androguard and wrote a short converter: https://github.com/androguard/androguard/blob/master/generators/axplorer_to_androguard.py
It is actually not that hard to switch between them, so I think you should stay with the syntax and we just convert them to json on the go.
So far I had no problems with the mappings, so thanks for your effort in generating them!

Hello, sorry for disturbing you, I see you integrated the mappings into androguard, maybe I don't know how to use the  androguard to extract these mapping informations, Could you give some suggestions?  Thank you!


Images do not load from device backup

Describe the bug
I am uploading a project from backup, and the images do not appear to transfer over from the backup into the project.
So I backed up project to device, did a clean install of Strabo (deleted the app), and then copied the backup from my computer into the strabospot2 files folder on the iPad, and then loaded project from device in Strabo2.
Nathan said it was a Redox issue that he had encountered with offline maps.

fixed with commits 00209f8704cec72505cab5a0c48caa2a9a078959 and b0a37a6b239c0c3450fc8eb13f3c32406b45c217

fixed with commits 00209f8704cec72505cab5a0c48caa2a9a078959 and b0a37a6b239c0c3450fc8eb13f3c32406b45c217


Update Marketplace Verbiage for Single Node offer for 2021-Q1 release

AB#1277116

Look at https://github.com/wls-eng/arm-oraclelinux-wls/issues/266 and replace the equivalent text in this offer with the text from that merged PR.


Compile warnings on WSL Debian 9.5

At the latest tabsidebar (ac5d4a5d8c494f6e5ff295a1e6702852d63e8447) branch, make results in the following compile warnings:
screen.c: In function ‘draw_tabsidebar’:
screen.c:10942:53: warning: passing argument 4 of ‘draw_tabsidebar_and_calc’ from incompatible pointer type [-Wincompatible-pointer-types]
     draw_tabsidebar_and_calc(1, maxwidth, fillchar, &total_row, &curtab_row);
                                                     ^
screen.c:159:13: note: expected ‘int *’ but argument is of type ‘int **’
 static void draw_tabsidebar_and_calc(int calc_mode, int maxwidth, int fillchar, int* total_row, int* curtab_row);
             ^~~~~~~~~~~~~~~~~~~~~~~~
screen.c:10942:65: warning: passing argument 5 of ‘draw_tabsidebar_and_calc’ from incompatible pointer type [-Wincompatible-pointer-types]
     draw_tabsidebar_and_calc(1, maxwidth, fillchar, &total_row, &curtab_row);
                                                                 ^
screen.c:159:13: note: expected ‘int *’ but argument is of type ‘int **’
 static void draw_tabsidebar_and_calc(int calc_mode, int maxwidth, int fillchar, int* total_row, int* curtab_row);
             ^~~~~~~~~~~~~~~~~~~~~~~~
screen.c:10943:53: warning: passing argument 4 of ‘draw_tabsidebar_and_calc’ from incompatible pointer type [-Wincompatible-pointer-types]
     draw_tabsidebar_and_calc(0, maxwidth, fillchar, &total_row, &curtab_row);
                                                     ^
screen.c:159:13: note: expected ‘int *’ but argument is of type ‘int **’
 static void draw_tabsidebar_and_calc(int calc_mode, int maxwidth, int fillchar, int* total_row, int* curtab_row);
             ^~~~~~~~~~~~~~~~~~~~~~~~
screen.c:10943:65: warning: passing argument 5 of ‘draw_tabsidebar_and_calc’ from incompatible pointer type [-Wincompatible-pointer-types]
     draw_tabsidebar_and_calc(0, maxwidth, fillchar, &total_row, &curtab_row);
                                                                 ^
screen.c:159:13: note: expected ‘int *’ but argument is of type ‘int **’
 static void draw_tabsidebar_and_calc(int calc_mode, int maxwidth, int fillchar, int* total_row, int* curtab_row);
             ^~~~~~~~~~~~~~~~~~~~~~~~
screen.c: In function ‘draw_tabsidebar_and_calc’:
screen.c:10959:10: warning: unused variable ‘i’ [-Wunused-variable]
     int  i;

Confirmed fix. Thanks 😄


[Grappler] No registered '_FusedMatmul 'OpKernel for GPU devices.

Describe the current behavior
DeepRec commit: 986498f5714
Enable oneDNN in GPU version DeepRec.  Run model would report the error as follows.

Fixed. https://github.com/alibaba/DeepRec/pull/404


Portfolio view. Tranfer button does not working for token with some networks inside

Description
Transfer button do nothing for token with a few networks inside

Example
https://github.com/novasamatech/nova-spektr/assets/40560660/3123a210-0f31-4de8-bb21-265a29baa592

Steps to Reproduce

Create wallet with Polkadot and Moonbeam accounts
Go to Portfolio page
Try to open Transfer page for the DOT

Expected Behavior
Oppened transfer page with possibility to select source network
Actual Behavior
The button do nothing
Environment
https://github.com/novasamatech/nova-spektr/commit/3bbef1f809eceab3a777694d4d086734ba7adab8

Do not reproduced


Update wiki

Update wiki to be in-line with merged https://github.com/spinalcordtoolbox/manual-correction/pull/43.

Thanks for your updates @NathanMolinier!
Just a minor comment:
Your wiki Introduction section:

was basically just repeating the section from README.md. To avoid duplications, I deleted the section from the wiki and kept it only once in the README.
I also updated the commands for the remaining workflows.


:broom: Accommodate to knative-sandbox/reconciler-test#294

Changes

:broom: Accommodate to knative-sandbox/reconciler-test#294

/cc @knative-sandbox/eventing-writers
/cc @knative-sandbox/discovery-approvers


Library name changed on npm?

It seems the library name has changed from node-google-translate-free to node-google-translate-skidz on npm. But this github project is still called -free. And packages with both names exist on npm, -free being the outdated one.
This might confuse users. It also creates problem for contributers, when you clone this repo from github it clones as -free and npm link links it with that name which the project using it isn't able to find.
Please unpublish the -free package from npm (you might need to email them to remove it entirely or have it redirected) and rename this github repo to -skidz (github handles redirects automatically I think)

Hi, library its different on NPM because someone uploads node-google-translate-free to npm first with this code. I'll change repo name to mach with NPM name. Thanks.