Live mode returns nil when regular SnipRun returns correct output

Describe the bug
I use https://github.com/akavel/up a lot for quick iteration on awk/sed/grep/regex's on standard output. So when I read about live mode, which would allow me to have this AND all neovim goodies right in the editor  I was delighted.
However, it behaves weird in two situations:

reading files (yes, I read     - disable the live mode whenever your code modifies files or invoke system commands. in sniprun/ressources/live_mode.md, but that's what I need for a similar workflow to up)
working with scripts which contain pre-existing code

I've tried this in python and lua, both same issue
For issue 1)
Here's using regular SnipRun

Here's when starting with an EMTPY file and typing the same code with live mode enabled

(on the right you can see file exists)
For issue 2)
Here, the red text was in the script before opening, when I then start live mode, that part appears to be ignored.

Note: even if I e.g. redefine the variable after starting live mode, it still shows nil
Environment:

Neovim version NVIM v0.8.0-dev+1468-gd3121f1e6
OS/distribution OSX 12.3
if possible, GLIBC version (output of ldd --version)
Sniprun

- -n Version :  v1.2.2

Can't tell you how excited I am about live mode to very quickly iterate on python/lua/... regexes and other data parsing (e.g. API request results) et al - would be a game changer!

reading files

Sniprun has (documented, i believe) trouble with relative paths. That probably...won't change, but maybe I can work on a configuration switch to enable sniprun to work in 'the current directory'

working with scripts which contain pre-existing code

Here's when starting with an EMTPY file and typing the same code with live mode enabled

Well, live mode works by running the "current-line-you're-modifying" only. You should probably enable "REPL" (sniprun has several repl-capable interpreters for python and one for lua) so that sniprun remembers the previous lines. That variable shouldn't be nil anymore after that

Sorry - finally had time to test this again - it works now without typos in config and the right setup, and using Python3_original an full paths (Python3_fifo crashed after every second line).
Few things would've made it a bit easier:

SnipInfo displaying info like LspInfo
the logs are very verbose - it was very difficult to realize where what happens
Help with relaying a bit more errors - I feel like (pure UX, probably not config'd right) often just nothing happens and I have no idea why - other live mode repls like codi, up will just show syntax errors etc when your code is incomplete yet. As it is, I feel like iteration speed (which is my big selling point for live mode) is still faster to just :!chmod +x % and vsplit  | te % via keymaps, or use something like https://github.com/jghauser/kitty-runner.nvim since some errors will happen anyways and my shell is at least vocal about it.

Feel free to ignore all, probably just rambling. This can be closed from my end :)

Feel free to ignore all, probably just rambling.

One's man rambling is another man's precious feedback, they say

(Python3_fifo crashed after every second line)

that one is definitely a bug. I'll test

Nice idea :-)
Yeah, but at the same time, it's not possible to debug every possible bug if they aren't. Those are supposed to be more dev(me)-facing than user-facing, and I can make sense of that unstructured mess easily enough for now.
There are several issues here:

Live-mode is (for now) limited to a line-by-line basis: I can't make sense of def f_test(): because it doesn't mean anything on its own. You can, however, make a visual selection and send it to sniprun the usual way, it'll be taken into account by subsequent live mode runs.
Errors are way too verbose for virtual-line display, and sumarizing them an impossible task. I could make the thing display a red 'x' or '?' though *
" is still faster to just :!chmod +x % and vsplit | te % ". :%SnipRun is basically equivalent then

* I'll keep that in mind for a future release, maybe not the next one but sometime...


Add support for Preconditions.checkArgument through custom CFG translation

See #47 for discussion.
This PR adds support for Preconditions.checkArgument(...) and Preconditions.checkState(...)
throw a new PreconditionsHandler handler.
In order to implement said handler, we need a custom CFG translation pipeline in NullAway
(NullAwayCFGBuilder) which subclasses the Checker Framework's CFGBuilder
(which we were using directly before this change). This pipeline allows us to add handler
callbacks during the AST to CFG translation process. At the moment, we add a single
such callback, right after visiting a MethodInvocationTree. We also add a more or less
generic method to insert a conditional jump and a throw based on a given boolean expression
node.
Abstracting some details about our AST and CFG structures, we translate:
Preconditions.checkArgument($someBoolExpr[, $someString]);

Into something resembling:
Preconditions.checkArgument($someBoolExpr[, $someString]);
if ($someBoolExpr) {
   throw new IllegalArgumentException();
}

Note that this causes $someBoolExpr to be evaluated twice. This is necessary based on how
NullAway evaluates branch conditionals, since we currently do not track boolean values through
our dataflow (e.g. boolean b = (o == null); if (b) { throw [...] }; o.toString(); produces a dereference
according to NullAway, independent on whether that code was added by rewriting or directly on the
source.
Finally, our implementation of NullAwayCFGBuilder and particularly NullAwayCFGTranslationPhaseOne
in this PR, depend on a Checker Framework APIs that are currently private. We are simultaneously
submitting a PR to change the visibility of these APIs (see [ToDo])

Important: This will not build (and shouldn't land) until the corresponding CF PR is landed and a new CF version cut.

FWIW, overall this approach looks fine to me!

I pulled in today's CF release.  Did not do all the other relevant cleanup (like removing mavenLocal())

Pull Request Test Coverage Report for Build #870

0 of 0   changed or added relevant lines in 0 files are covered.
7 unchanged lines in 4 files lost coverage.
Overall coverage increased (+0.05%) to 92.15%

Files with Coverage Reduction
New Missed Lines
%

../nullaway/src/main/java/com/uber/nullaway/dataflow/AccessPath.java
1
96.28%

../nullaway/src/main/java/com/uber/nullaway/handlers/Handler.java
1
91.67%

../nullaway/src/main/java/com/uber/nullaway/handlers/Handlers.java
1
96.55%

../nullaway/src/main/java/com/uber/nullaway/dataflow/DataFlow.java
4
95.45%

Totals

Change from base Build #869:
0.05%

Covered Lines:
4825

Relevant Lines:
5236

💛  - Coveralls

I pulled in today's CF release. Did not do all the other relevant cleanup (like removing mavenLocal())

Let's do this the slightly slower but more CHANGELOG friendly way: https://github.com/uber/NullAway/pull/610 😉

I ran our benchmarks before and after this PR and saw no increase in NullAway overhead; so this change seems to be safe from that standpoint.

@lazaroclapp ping me when this is ready for a full review


Remove Posters on Show Tracks

it's excessive to have the same poster used over and over again.

Makes sense to remove this if we weren't planning to have playlists, but if we can create playlists, mixing different shows together it makes sense to keep this and add the show title to the tracks. Some fancier work could be done in order to only have this and show title displayed for playlists, but I'll wait until playlists are ready to make that decision.


Shared Preferences does not work for android although it is working for iOS

I have shared preferences installed and working fine in the app for iOS. But the same app has issues persisting data running on android.
shared_preferences 0.5.12
pubspec.yaml
name: timetracker
description: A new Flutter project.

# The following defines the version and build number for your application.
# A version number is three numbers separated by dots, like 1.2.43
# followed by an optional build number separated by a +.
# Both the version and the builder number may be overridden in flutter
# build by specifying --build-name and --build-number, respectively.
# In Android, build-name is used as versionName while build-number used as versionCode.
# Read more about Android versioning at https://developer.android.com/studio/publish/versioning
# In iOS, build-name is used as CFBundleShortVersionString while build-number used as CFBundleVersion.
# Read more about iOS versioning at
# https://developer.apple.com/library/archive/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html
version: 1.0.0+1

environment:
  sdk: ">=2.3.0 <3.0.0"

dependencies:
  flutter:
    sdk: flutter
  image_picker: ^0.6.3+4
  camera: ^0.5.8+6
  flutter_map: ^0.8.2
  user_location: any
  provider: ^4.3.2+2
  tuple: ^1.0.3
  date_utils: any
  sqflite: ^1.3.1+1
  postgres: ^2.2.0
  flutter_simple_dependency_injection: ^1.0.3
  load: ^0.1.6
  shared_preferences: ^0.5.12
  get_it: ^5.0.0
  flutter_localizations:
    sdk: flutter

  # The following adds the Cupertino Icons font to your application.
  # Use with the CupertinoIcons class for iOS style icons.
  cupertino_icons: ^0.1.2

dev_dependencies:
  flutter_test:
    sdk: flutter

# For information on the generic Dart part of this file, see the
# following page: https://dart.dev/tools/pub/pubspec

# The following section is specific to Flutter.
flutter:

  # The following line ensures that the Material Icons font is
  # included with your application, so that you can use the icons in
  # the material Icons class.
  uses-material-design: true

  # To add assets to your application, add an assets section, like this:
  assets:
    - assets/bg/
    - assets/images/

  # An image asset can refer to one or more resolution-specific "variants", see
  # https://flutter.dev/assets-and-images/#resolution-aware.

  # For details regarding adding assets from package dependencies, see
  # https://flutter.dev/assets-and-images/#from-packages

  # To add custom fonts to your application, add a fonts section here,
  # in this "flutter" section. Each entry in this list should have a
  # "family" key with the font family name, and a "fonts" key with a
  # list giving the asset and other descriptors for the font. For
  # example:
  fonts:
    - family: AvenirNext
      fonts:
        - asset: assets/fonts/AvenirNextCyr-UltraLight.ttf
          weight: 100
        - asset: assets/fonts/AvenirNextCyr-Regular.ttf
          weight: 400
        - asset: assets/fonts/AvenirNextCyr-Demi.ttf
          weight: 500
        - asset: assets/fonts/AvenirNextCyr-Medium.ttf
          weight: 600
        - asset: assets/fonts/AvenirNextCyr-Bold.ttf
          weight: 700

    # - family: Schyler
    #   fonts:
    #     - asset: fonts/Schyler-Regular.ttf
    #     - asset: fonts/Schyler-Italic.ttf
    #       style: italic
    # - family: Trajan Pro
    #   fonts:
    #     - asset: fonts/TrajanPro.ttf
    #     - asset: fonts/TrajanPro_Bold.ttf
    #       weight: 700
  #
  # For details regarding fonts from package dependencies,
  # see https://flutter.dev/custom-fonts/#from-packages

code
Future<void> main()  async {
    // WidgetsFlutterBinding.ensureInitialized();
    // SystemChrome.setEnabledSystemUIOverlays([]);
    // SystemChrome.setPreferredOrientations([DeviceOrientation.portraitUp]);
    WidgetsFlutterBinding.ensureInitialized();
    await setupLocator();
    var sharedPrefService = locator.get<SharedPreferences>();
    sharedPrefService.reload();
    print(sharedPrefService.getKeys());
    print(sharedPrefService.getBool("remember_me"));

    if(sharedPrefService.getBool("remember_me") == false){
      sharedPrefService.clear();
    }
    await Injection.initInjection();
    List<CameraDescription> cameras = [];
  //  Obtain a list of the available cameras on the device.
  var firstCamera;
    try {
      cameras = await availableCameras();
      firstCamera = cameras.first;
    } on CameraException catch (e) {
      print(e);
      print("Error occured in camera exception");
    }
    // Get a specific camera from the list of available cameras.

    runApp(MyApp());

}

class MyApp extends StatelessWidget {

  var sharedPrefService = locator.get<SharedPreferences>();
  @override
  Widget build(BuildContext context) {
    print(sharedPrefService.getBool("remember_me"));
     return MultiProvider(
       providers: [
        ChangeNotifierProvider.value(
          value: AuthenticationProvider(),
        ),
      ],
            child: new MaterialApp(
        onGenerateRoute: RouteGenerator.generateRoute,
        initialRoute: (sharedPrefService.getBool("remember_me") == true) ? homeRoute : introductionRoute,
    ),
     );
  }
}

import 'package:get_it/get_it.dart';
import 'package:shared_preferences/shared_preferences.dart';
import 'package:timetracker/core/services/shared_prefs.dart';

final locator = GetIt.instance;

Future setupLocator() async {
  var instance = await SharedPreferences.getInstance();
  locator.registerSingleton<SharedPreferences>(instance);
}

flutter doctor -v
```
[✓] Flutter (Channel dev, 1.23.0-4.0.pre, on Mac OS X 10.15.2 19C57 x86_64, locale en-SA)
    • Flutter version 1.23.0-4.0.pre at /Users/faheemuddinkhan/Development/flutter
    • Framework revision 83dd176777 (17 hours ago), 2020-09-22 12:04:44 -0700
    • Engine revision 2abe69c608
    • Dart version 2.10.0 (build 2.10.0-136.0.dev)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
• Android SDK at /Users/faheemuddinkhan/Library/Android/sdk
• Platform android-29, build-tools 29.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_202-release-1483-b49-5587405)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 11.3.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.3.1, Build version 11C504
• CocoaPods version 1.8.4
[✓] Android Studio (version 3.5)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 43.0.1
• Dart plugin version 191.8593
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
[✓] VS Code (version 1.49.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.14.1
[✓] Connected device (1 available)
• Android SDK built for x86 (mobile) • emulator-5554 • android-x86 • Android 10 (API 29) (emulator)
</details>

Hi @FaheemuddinKhan
Please provide clear steps to reproduce the issue, the espected vs actual results and a minimal reproducible code sample that does not use 3rd party plugins.
Thank you

`name: timetracker
description: A new Flutter project.
https://developer.apple.com/library/archive/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html
version: 1.0.0+1
environment:
sdk: ">=2.3.0 <3.0.0"
dependencies:
flutter:
sdk: flutter
image_picker: ^0.6.3+4
camera: ^0.5.8+6
flutter_map: ^0.8.2
user_location: any
provider: ^4.3.2+2
tuple: ^1.0.3
date_utils: any
sqflite: ^1.3.1+1
postgres: ^2.2.0
flutter_simple_dependency_injection: ^1.0.3
load: ^0.1.6
shared_preferences: ^0.5.12
get_it: ^5.0.0
flutter_localizations:
sdk: flutter
The following adds the Cupertino Icons font to your application.
Use with the CupertinoIcons class for iOS style icons.
cupertino_icons: ^0.1.2
dev_dependencies:
flutter_test:
sdk: flutter
`

Hi @FaheemuddinKhan
Please provide a code that does not contains third-party plugins.
Thank you

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


fix wrong mutation name

Good catch @s4|1h! Can you also change the string to 'HELLO'? I'm away from my computer.

done!

Thanks!


🔧 fix ruff action to include formatting

lgtm!

@philippemiron feel free to merge this in :smile:


HttpRequest: add feature to avoid url encoding of path

@JuergenGleiss Thanks so much for your patch, please update the swap() method of the HttpRequestImpl class.

Sorry, I've missed the methode


reduced mul-ops for input indexing in loops.

I see it on Gerrit now, thanks! I will close this PR.
James


How to install Apex on Windows system

I have tried various methods to compile and install, but all have encountered errors. Does Apex not support Torch 2.5.1 under Windows

Thanks for your interest. You don't need to install apex to run consisid. All thhe required packages are included in requirements.txt. If you don't need training, deepspeed doesn't need to be installed.


Update links to GCP and Azure for STR downloads on browser

For Friday's release, we displayed only AWS links as we had control over the timing of the data sync to AWS. The files should now all be synced to GCP and Azure and available to the public on those platforms.

See: https://gnomad.broadinstitute.org/downloads#v3-short-tandem-repeats
Notice how the STR datasets only have the Amazon link. Need to add GCP/Azure links, similar to the other datasets on the page.


Make methods static.

Description
This makes a few methods which can be static, static.
Checklist:

[x] I have run ./ci/run_stable_checks.sh
[x] I have reviewed my own code
[x] I have added tests – if this is a bug fix, these tests will fail if the bug is present (to stop it from cropping up again). If this is a feature, my tests prove that the feature works.

@hgzimmerman Sorry about that; it was the result of a mess up over a rebase which meant that some commits which triggered you as a code owner were added on to this pull request – I've now removed them.

@teymour-aldridge The same thing could be done for ResizeService, right?

I imagine so.


[Bug] Running CaptainHookPhp installer in Docker container ignores run-mode

This might be an edge-case no one so far thought of, sorry for that.
Situation is: I want to use the Captain in a frontend project where not all team members have a PHP/Composer setup running. Since the Captain supports Docker this should work fine.
This is what I tried:

Install the Captain via a Docker container

docker run --rm -v $(pwd):/app -w /app prooph/composer:7.2 require --dev captainhook/captainhook

Run Captain install

docker run --rm -v $(pwd):/app -w /app php:7.2-cli /app/vendor/bin/captainhook install --run-mode=docker --run-exec="docker exec php"

As a result the git hooks are stored as PHP scripts not as shell scripts, which is what I have expected. If I run the commands locally without docker, things are working fine, the git hooks are stored as shell scripts as expected.
Are there any checks if the run mode is valid which might not work in this Docker setup?

Not a bug, my mistake. Parameters like --run-mode and --run-exec  are passed to Docker itself not the command running inside the container. Could fix it by defining the commands in the captainhook.json file.


Add Developer Guide

[x] describe tasks. pipeline and base model
[x] how to add tasks
[x] describe tests

maybe add a link in the readme to the developer guide?


First page doesn't have padding and other UI tweaks applied to it, but all others do

I have a OnboardingViewController with six OnboardingContentViewControllers. When I apply layout tweaks such as:
ovc.bodyFontSize = 26
ovc.titleFontSize = 36
ovc.iconSize = 350
ovc.underIconPadding = 100
ovc.underTitlePadding = 100
ovc.topPadding = 150

they are applied to the 2nd -> 6th page, but not the first. Screenshot from an iPad attached below.
My complete code for the OnboardingViewController is below: any ideas? Is this perhaps a known bug?
    var onboardingVC = OnboardingViewController(backgroundImage: UIImage(named: "onboardBackground"), contents: [firstPage, secondPage, thirdPage, fourthPage, fifthPage, sixthPage])

    // Configuration properties of the VC
    onboardingVC.shouldFadeTransitions = true
    onboardingVC.fadePageControlOnLastPage = false
    onboardingVC.shouldBlurBackground = true
    onboardingVC.shouldMaskBackground = false
    onboardingVC.allowSkipping = false
    onboardingVC.hidePageControl = false
    onboardingVC.swipingEnabled = false
    onboardingVC.bodyFontSize = 22
    onboardingVC.iconSize = 250
    onboardingVC.view.clipsToBounds = true // Stop bits been shown on pop

    if (iPad) {
        onboardingVC.bodyFontSize = 26
        onboardingVC.titleFontSize = 36
        onboardingVC.iconSize = 350
        onboardingVC.underIconPadding = 100
        onboardingVC.underTitlePadding = 100
        onboardingVC.topPadding = 150
    }

    onboardingVC = adjustOnboardingForScreenSizes(onboardingVC)

    // Push the Onboarding View Controller and flip the Status Bar Colour
    self.navigationController?.pushViewController(onboardingVC, animated: false)

Any ideas anybody?

Is this still an issue with the most recent version?


Add support to includecode for pulling from GitHub repo

{% includecode github_path="github-user/project/src/api/sample.py" %}

Taking this.


Alternative cmake build system

@timkpaine FYI

Just FYI @alaindargelas we will not merge without a review, nothing should be merged without a proper review. We will most likely review in the period of the next few days.

I think we're almost ready to merge this just 2 small requests:

can you run a cmake-format on the files? Indentation seems to be a little inconsistent. I suppose it makes sense to even include cmake format checks in CI so any future updates will keep correct formatting

please rebase the PR on top of main (where the OpenTitan issue is fixed), so we can have green CI here

Rebased and formatted.


ios/Pods/Headers/Public/React-Core/React/RCTBridgeModule.h:132:1: Property with 'retain (or strong)' attribute must be of object type

I am using this RCTBridgeModule to create my own custom bridge module for one of my application. Recently i migrated the app to latest react-native version 0.61.5. Now the iOS xcode build and react-native run ios command build gives me the error "ios/Pods/Headers/Public/React-Core/React/RCTBridgeModule.h:132:1: Property with 'retain (or strong)' attribute must be of object type".  The build works fine when i removed the strong keyword from this line below. The strong or retain keyword is not required here as i see the variable methodQueue  here is not an object. Could you please fix this issue ASAP.
https://github.com/facebook/react-native/blob/75a617827963fd53f5f36987677a992e137eaebf/React/Base/RCTBridgeModule.h#L146
Thanks
Jencir C J

Is this issue going to be fixed in upcoming versions? See this link for more details on how to declare dispatch_queue as a property after ARC - https://stackoverflow.com/questions/8904206/what-property-should-i-use-for-a-dispatch-queue-after-arc
I look forward to take this issue as priority as it cause build error for those developers who try to build their own native iOS components using react-native's custom bridge method


noImplicitAny is detecting an error in lib.d.ts

Yesterday I was compiling a new version of ionic-minify module and added the noImplicitAny flag. It showed an error in lib.d.ts

tsc --version
Version 1.7.5

Could you specify what declaration it errored on? We haven't seen this at all.

I'm closing this issue since it seems that my TypeScript installation was corrupted, I reinstalled the typescript module and it doesn't complain anymore.
I'm using tsc with Ubuntu 14.04.4 LTS with Visual Studio Code (latest)

Awesome, glad it's working correctly.


changing default variable to match used variable in "remaining_nodes.yml"

Defaults result in errors due to default "rke_type" vs "rke2_type".
"remaining_nodes.yml" errors out as it is expecting "rke2_type"

Hi @ProjectInitiative, nice catch. Thanks :)


Tick rate back down to 60% after updating to Sequoia

Title pretty much describes it.
Unsure if it's the plugin, the update, or what, but my ticks are back down to 60% after updating to Sequoia, and the plugin is enabled.

Thanks for filing an issue. Have you tried changing the target IP (to <IP_ADDRESS>, for example) or reducing the ping interval? The minimum interval is 100ms.

Hey yeah I did both of those.
The only thing that fixes it is watching a live broadcast a la twitch. I'm not sure if something happened to change how things are pinged but it's weird.


Make lookup of names easier for users

I think its not very easy for the user to figure out what name / id to use in the HA config.
@Guzz-T made that point in the discussion in #187 , that we should write the code in a way that makes it easy for the user to look the name up.
I propose that we add a GitHub Action that parses the code instead and create a static web page with a search field that we then host on GitHub pages for the users to look the right parameters up.
That is in my eyes the better way to help the user than to write the code in a way that users can look up the needed nmes directly in the code.

I think we should also add some small piece of documentation for each parameter / calculation for which we know the meaning.

@gerw I agree, my key point here is that the user should not have to look up the needed info in the source.


where is CLAP_weights_2022.pth

Where can I find  CLAP_weights_2022.pth mentioned in wav_evaluation/cal_clap_score.py

+1

https://huggingface.co/microsoft/msclap/tree/main


TIme zone

could you please release new version as 1.1.0 itself does not support timezones

same problem here why does time zone not work anymore?

Install the none release version: pip install git+https://github.com/dbader/schedule

Sorry for the delay! Just released 1.2.0 that includes the timezone.


Fix set partial query

Scope of changes
This fixes an issue where the partial flag on all API keys was getting overwritten every time a new API key was created. This doesn't affect the actual key permissions but does impact whether "Partial" or "Full" is displayed on the frontend.
Fixes SC-16081
Type of change

[ ] new feature
[x] bug fix
[ ] documentation
[ ] testing
[ ] technical debt
[ ] other (describe)

Acceptance criteria
Describe how reviewers can test this change to be sure that it works correctly. Add a checklist if possible.
Author checklist

[x] I have manually tested the change and/or added automation in the form of unit tests or integration tests
[ ]  I have updated the dependencies list
[ ]  I have recompiled and included new protocol buffers to reflect changes I made
[ ]  I have added new test fixtures as needed to support added tests
[x]  Check this box if a reviewer can merge this pull request after approval (leave it unchecked if you want to do it yourself)
[x]  I have moved the associated Shortcut story to "Ready for Review"

Reviewer(s) checklist

[ ] Any new user-facing content that has been added for this PR has been QA'ed to ensure correct grammar, spelling, and understandability.
[ ] Are there any TODOs in this PR that should be turned into stories?

Good fix, nice work catching that. Before was it setting partial on all API Keys?

Yes, and not even in the same project I believe. You can thank @daniellemaxwell for catching it in her QA.


Can we use CompletionItem#additionalTextEdits to update content after the cursor in the same line?

We have a language server which provides completions for tags (look like XML). We have the capability to update orphan end tag with  the completion apply.
For instance given this XML (| is used to show where completion is triggered):
<|></>

When completion is opened, it show some element like foo, bar. When foo completion item is applied, it update the end tag too to have the following result:
<foo></foo>

To do that we use additionalTextEdits  to update the end tag on the completionItem/resolve. It works perfectly except if the user type quickly when completion is opened. We could have this bad result:
<foo>foo</>

I wonder for this usecase (update content after the cursor in the same line) if the solution is to use a command because it seems command is executed on after completion insert according the documentation An optional command that is executed *after* inserting this completion.. But the following comment says Note that additional modifications to the current document should be described with the additionalTextEdits-property.
My question is it a bug of vscode or should I use command for this usecase.
Many thanks for your answer.

According to https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#completionItem
        /**
	 * An optional array of additional text edits that are applied when
	 * selecting this completion. Edits must not overlap (including the same
	 * insert position) with the main edit nor with themselves.
	 *
	 * Additional text edits should be used to change text unrelated to the
	 * current cursor position (for example adding an import statement at the
	 * top of the file if the completion item will insert an unqualified type).
	 */

I would think from the example, there would be overlap when the main text edit is longer than the distance between the tag element start/end.

According to the spec neither additional text edits nor commands can be used for this. All I can think of right now is to use a snippet that replaces <|></> with <foo></foo> and uses the cursor end position to put the cursor at the right position.

Would that work for you?

Would that work for you?

Badly no -(
If we use snippet, the only solution to ashow completion is to use choice which is limitated just with string, so we loose:

the completion documentation.
the capability to generate XML attributes when a completion item is applied.

@aeschli do you have an idea. Is this something you solved in HTML?

@aeschli do you have an idea. Is this something you solved in HTML?

In HTML there is no this feature https://github.com/redhat-developer/vscode-xml/blob/main/docs/Refactor.md#surround-with-tags-wrap

For HTML, we use the linked editor provider to do that.
It's also there in LSP: https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#textDocument_linkedEditingRange

For HTML, we use the linked editor provider to do that.

We use it too, and it works great, but not when you try to open completion, apply it and generate elements with attributes.

I am actually out of ideas. You could directly see if there is a suggestions in VS Code itself and then we could start discussing whether we could add something to LSP itself.

Ok, thanks so much @dbaeumer @aeschli for your feedback.

Treating the issue as a question and closing it.


Update readme roles groups

Update the README.md to clarify how to work with appRoles in the manifest and Azure SSO in order to be able to pull roles into authorization.

Minor release on this?

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Local planner gets stuck because of driver constraints

I'm implementing hybrid planning to the UR10e robot and the local planner gets stuck often, asking for a re-plan from the global planner causing this action:
https://user-images.githubusercontent.com/31903354/201063046-b55b7f32-2abc-4a6d-b02f-1e19ced651c7.mp4
After a lot of debugging I found that it's caused by these lines in ur_controllers.yaml:
shoulder_pan_joint: { trajectory: 0.2, goal: 0.1 }
shoulder_lift_joint: { trajectory: 0.2, goal: 0.1 }
elbow_joint: { trajectory: 0.2, goal: 0.1 }
wrist_1_joint: { trajectory: 0.2, goal: 0.1 }
wrist_2_joint: { trajectory: 0.2, goal: 0.1 }
wrist_3_joint: { trajectory: 0.2, goal: 0.1 }

If I increase this threshold or remove these lines, the problem goes away. Is it possible that these values are too low? Increasing them to 0.3 and 0.15 mostly solves this issue:
https://user-images.githubusercontent.com/31903354/201064915-2e6db542-3924-413c-b0e4-b219c1ab1fce.mp4
Removing these lines completely fixes this issue:
https://user-images.githubusercontent.com/31903354/201065216-7f3df691-10a2-4a69-8a0d-e4edde5783fa.mp4

Making the joint limits stricter also improves the performance, even with 0.2 and 0.1 as the tolerances

I'll close this issue since this seems to be solved by limiting joints and keeping the tolerances the same

Hi @skaeringur97, can u post here a video with the final result pls.
Thanks in advance

@marioesteban11
https://user-images.githubusercontent.com/31903354/203079095-7e12c6c5-72f1-445d-9d20-1474dd5712bf.mp4

May I know, which joint_limits and ur_controllers parameters are you using? @skaeringur97

@marioesteban11
Joint limits:
joint_limits:
  shoulder_pan_joint:
    # acceleration limits are not publicly available
    has_acceleration_limits: false
    has_effort_limits: true
    has_position_limits: true
    has_velocity_limits: true
    max_effort: 330.0
    max_position: !degrees  -50.0
    max_velocity: !degrees  120.0
    min_position: !degrees -200.0
  shoulder_lift_joint:
    # acceleration limits are not publicly available
    has_acceleration_limits: false
    has_effort_limits: true
    has_position_limits: true
    has_velocity_limits: true
    max_effort: 330.0
    max_position: !degrees  5.0
    max_velocity: !degrees  -90.0
    min_position: !degrees -200.0
  elbow_joint:
    # acceleration limits are not publicly available
    has_acceleration_limits: false
    has_effort_limits: true
    has_position_limits: true
    has_velocity_limits: true
    max_effort: 150.0
    # we artificially limit this joint to half its actual joint position limit
    # to avoid (MoveIt/OMPL) planning problems, as due to the physical
    # construction of the robot, it's impossible to rotate the 'elbow_joint'
    # over more than approx +- 1 pi (the shoulder lift joint gets in the way).
    #
    # This leads to planning problems as the search space will be divided into
    # two sections, with no connections from one to the other.
    #
    # Refer to https://github.com/ros-industrial/universal_robot/issues/265 for
    # more information.
    max_position: !degrees  -5.0
    max_velocity: !degrees  180.0
    min_position: !degrees -150.0
  wrist_1_joint:
    # acceleration limits are not publicly available
    has_acceleration_limits: false
    has_effort_limits: true
    has_position_limits: true
    has_velocity_limits: true
    max_effort: 56.0
    max_position: !degrees  220.0
    max_velocity: !degrees  180.0
    min_position: !degrees -220.0
  wrist_2_joint:
    # acceleration limits are not publicly available
    has_acceleration_limits: false
    has_effort_limits: true
    has_position_limits: true
    has_velocity_limits: true
    max_effort: 56.0
    max_position: !degrees  220.0
    max_velocity: !degrees  180.0
    min_position: !degrees -220.0
  wrist_3_joint:
    # acceleration limits are not publicly available
    has_acceleration_limits: false
    has_effort_limits: true
    has_position_limits: true
    has_velocity_limits: true
    max_effort: 56.0
    max_position: !degrees  220.0
    max_velocity: !degrees  180.0
    min_position: !degrees -220.0

Parameters I use:
Driver - robot_ip:=aaa.bbb.cc.d use_fake_hardware:=true initial_joint_controller:=joint_trajectory_controller
Moveit -  use_fake_hardware:=true ur_type:=ur10e use_sim_time:=true launch_servo:=false

Reasons explained in #503 except for launch_servo:=false, I added that because for some reason psi.getKnownObjectNames() doesn't work for objects published to a move_group node before the current node is started. If I have launch_servo:=false, it works as expected

Hi @skaeringur97 , one more question, are u using geometryPose or something different to move the UR?
I have several movement problems and I want to solve it. I`m using UR3e if it gives you more info

@marioesteban11 Yes, I make a geometry_msgs::msg::PoseStamped object and send it to the hybrid planner using this:
moveit_msgs::msg::Constraints goal_constraints =
kinematic_constraints::constructGoalConstraints(end_effector, target_goal, tolerance_pose, tolerance_angle);
goal_motion_request.goal_constraints.push_back(goal_constraints);

If you explain your problem better and maybe show a video I can maybe help more. Maybe it's even better to make a new issue

I'm gonna post how I launch all my system, a video, and my code to move the robot.
First of all I start controller:

ros2 launch ur_robot_driver ur_control.launch.py ur_type:=ur3e robot_ip:=<IP_ADDRESS> use_fake_hardware:=true launch_rviz:=false initial_joint_controller:=joint_trajectory_controller

After that I launch moveit files:

ros2 launch ur_robot_driver prueba_grande.launch.py robot_ip:=<IP_ADDRESS> use_fake_hardware:=true  use_sim_time:=true

This is not the original moveit file, but it just launch directly the ur3e robot.
 What problems do you have?

@skaeringur97 exactly that, I want that all the moves are smooth, more or less like yours

@marioesteban11 You could try using different planners. If you play around in Rviz you will see that some planners take much more reasonable paths. I don't know how to change the planners myself tho. I think limiting the joint constraints is also good if that's possible. You don't necessarily lose any reachable space by doing so.


Feature request: Make contrib.receptive_field extensible

System information

TensorFlow version (use command below): 1.8.0

Describe the problem
For the problem I am working on, I need to compute a cross-correlation where the filters are a function of some inputs, i.e. c_i = conv2d(f(a_i), g(b_i)) for the i-th example (a_i, b_i) in the batch. After doing this, the receptive field of c with respect to a is still well-defined.
I can think of several ways to implement this, but none of them are compatible with receptive_field!
For example, I usually use reshape-depthwise_conv2d-reshape-sum to transform a batch of B convolutions with C channels into a batch of 1 depthwise-convolution with B*C channels, then sum over the C channels at the end. Another way to do it would be using tf.map_fn. However, neither of these will work with compute_receptive_field_from_graph_def.
I understand that it would be very difficult to support edge cases like this. However, it breaks the workflow a bit to keep track of every place where the graph is incompatible with receptive_field and perform the composition manually here.
Perhaps it would be possible to add a manual override parameter to compute_receptive_field_from_graph_def that allows a user to manually specify the receptive field from one node to another, instead of using the graph?
Another way could be to support custom layers, either by checking if the custom layer has a receptive field method, or allowing users to register custom layers with the receptive_field module. However, this would require me to write my own layer for the "dynamic" convolution described above, which is not currently necessary.
I'm open to suggestions!

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): no
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): n/a
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: n/a
TensorFlow installed from (source or binary): binary
Python version: 2.7
Bazel version (if compiling from source): n/a
GCC/Compiler version (if compiling from source): n/a
CUDA/cuDNN version: n/a
GPU model and memory: n/a
Exact command to reproduce: n/a

@jvlmdr Thanks for sharing the use case. Customizing your own layer would be a quicker route but adding knobs to compute_receptive_field_from_graph_def is a viable approach especially when your use case may change frequently in the future. You can modify TensorFlow graph in place using Graph Editor via tf.contrib.graph_editor

Glad to help :)


ff.options() is not working as expected & also not giving error

I am trying to run this command
from pyffmpeg import FFmpeg
ff= FFmpeg()
ff.overwrite=True
command =""" -i apirs.mp4 -i flicks.png -filter_complex "[0:v][1:v] overlay=x=(main_w-overlay_w-40):y=80,drawtext=fontfile=Poppins-SemiBold.ttf:fontcolor=white:text=@suman:fontsize=24:y=main_h-30:x=10" -b:v 2200k -c:a copy  output.mp4"""
ff.options(command)
print("completed")
the output is
2022-11-27 05:25:52,638 - pyffmpeg.FFmpeg - INFO - FFmpeg Initialising
2022-11-27 05:25:52,638 - pyffmpeg.FFmpeg - INFO - Save directory: .
2022-11-27 05:25:52,638 - pyffmpeg.misc.Paths - INFO - bin folder: /home/suman/.pyffmpeg/bin
2022-11-27 05:25:52,638 - pyffmpeg.misc.Paths - INFO - Inside load_ffmpeg_bin
2022-11-27 05:25:52,638 - pyffmpeg.FFmpeg - INFO - FFmpeg file: /home/suman/.pyffmpeg/bin/ffmpeg
2022-11-27 05:25:52,638 - pyffmpeg.FFmpeg - INFO - inside options
2022-11-27 05:25:52,638 - pyffmpeg.FFmpeg - INFO - Options is a String
2022-11-27 05:25:52,638 - pyffmpeg.FFmpeg - INFO - Shell: False
completed
The problem is output.mp4 file is not being created.
I am trying to run this code in gcloud Ec2 linux instance. any help

I found the solution
not working ::
command =""" -i apirs.mp4 -i flicks.png -filter_complex "[0:v][1:v] overlay=x=(main_w-overlay_w-40):y=80,drawtext=fontfile=Poppins-SemiBold.ttf:fontcolor=white:text=@suman:fontsize=24:y=main_h-30:x=10" -b:v 2200k -c:a copy output.mp4"""
working ::
command =""" -i apirs.mp4 -i flicks.png -filter_complex [0:v][1:v]overlay=x=(main_w-overlay_w-40):y=80,drawtext=fontfile=Poppins-SemiBold.ttf:fontcolor=white:text=@suman:fontsize=24:y=main_h-30:x=10 -b:v 2200k -c:a copy output.mp4"""
this is the difference
but still if you guys can add error logs, will help much more

@legend1998 glad you got it fixed. Based on your request. I am reopening this issue.
Thanks!

FFmpeg throws exceptions and also adds errors to the logs


Add license explanation to close button label

Fixes

Fixes #987 by @zackkrida
Description

I tried to fix this as best I can, but it needs to be tested in VO and JAWS. I'm working on JAWS, but someone with a Mac will have to help with VO.
This adds the license explanation to the close button. That way when the SR opens the popover and the close button is focused, it reads the explanation and then also declares that the close button is focused (so that the user knows they can close the popover).
Testing Instructions

Checkout this branch and go to the search results page, /search?q=birds then, navigating with a screen reader, open the filter sidebar and tab into it (note the experience here will improve with #1394). Focus the license explanation popover button. Open the popover and verify that the SR reads the license explanation and informs the user that they are on the close button for the popover and that you can close the popover successfully.
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[N/A] I added or updated tests for the changes I made (if applicable).
[N/A] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

It's working in that you do have more context to the Close button, but you get the license description too, as part of the label:

I would expect it to simply say "Close license explanation button", but maybe I\m doing something wrong?


SolidEdgeConstants (C#)

When I try to build the DemoAddIn, I get an error stating that 'ApplicationCommandConstants' in the namespace 'SolidEdgeConstants' is missing.

The SDK\Samples will always be built with the latest version of the software. If you are using an older version, adapt the code as needed to meet your needs.

Jason, thank you for replying to this issue so quickly.  I did have an older version but I completely deleted the folder before I downloaded the new copy.  I tried again today and got the same results.  When I download the repository, I verified that there wasn't a Interop.SolidEdgeConstants.dll under the DemoAddIn folder.  As expected, the obj/Debug folder doesn't yet exists.  When I open DemoAddIn.sln in Visual Studio 2017, the Interop.SolidEdgeConstants.dll is generated (see attach and notice the date and time for the file) and it still doesn't contain the ApplicationCommandConstants.
I don't understand what I'm doing wrong.  Can you please explain.
Thank you.

The samples that I provide will always be using the latest version of Solid Edge. In this case, Solid Edge 2021. If you have a need to run the samples that are provided in an older version of Solid Edge, you will need to make the necessary code changes.

I have upgraded to version 2021 and SDK\Samples is working correctly.
Thanks for your help.


Exception in features view of module Orchard.Modules

Hi,
i was playing around with some modules today and i came to a point where Orchard was not running anymore. I tracked it down to the file
src\Orchard.Web\Modules\Orchard.Modules\Views\Admin\Features.cshtml

and the following line that looks like this
var dependencies = (from d in feature.Descriptor.Dependencies
                    select (from f in Model.Features where f.Descriptor.Id.Equals(d, StringComparison.OrdinalIgnoreCase) select f).SingleOrDefault()).Where(f => f != null).OrderBy(f => f.Descriptor.Name);

The call to SingleOrDefault() throws an exception when a user copies a module in the Modules folder and therefore two Module.txt files with the same content exist. I think it would be better to call FirstOrDefault() as it
a) does not crash
b) shows features correctly (in my case one feature multiple times) in the admin features view
So lonG

Agreed, let's use FirstOrDefault, but we also need a check for this case, and display an error notification.

Can't reproduce this anymore, therefore no need to fix something.


There should be a configurable option to prevent user from setting high and low model values as equal!

Currently a user can set high and low as equal but what if there is a use case where such a thing is invalid?
Also high and low pins shouldn't cross over in this use case.

Now it could be done by using slideEnded event and validating model.


jsonwebtoken Version has high vulnerabilities in docusign-click

Hi Docusign,
I currently have docusign-click version 1.0.0. Currently, I'm getting vulnerabilities after running an audit (yarn audit) on the packages installed on my project.

The solution is to upgrade jsonwebtoken to version 9.0.0.
My temporary solution is to manually update the docusign-click dependency versions directly in the yarn.lock file and then run yarn to reinstall packages.

Ideally, docusign-click should update jsonwebtoken and release a new version.
Thanks in advance!

+1 on this :)

Thanks for your note. We have opened up an Engineering fix for this.
For your reference it is DCM-8728.

@kamran-DS Is there a good place where I can keep track of the progress made on DCM-8728?
On the other hand, I appreciate the quick response and action toward getting this issue resolved.

Hi folks, this patch is being applied to our public node SDKs this week. It is a breaking change and will require you to upgrade to the next major version.
For esign this will be version 6.0.0

https://github.com/docusign/docusign-esign-node-client/releases/tag/v6.0.0


Remove unsupported topolvm manifests

Which issue(s) this PR addresses:

Closes #USHIFT-907

/pj-rehearse ack
/pj-rehearse

/pj-rehearse

/pj-rehearse ack

/pj-rehearse

/retest-required

/retest

/retest

/retest

Could you add a little detail to the description and/or commit message explaining why we don't want these files?
There are 7 commits in the PR. Are the all independent, or can they be squashed into fewer?

Hey @dhellmann, np.  I've updated the description.

@pmtk or @pacevedom , we could set the merge method to squash this as it lands, if we want to get it in this week. What do you think?

/lgtm


Try to build a wechaty-TIKTOK-puppet : Regular technical report（01/11/2021-05/11/2021）

The work completed last week includes:
1，Solve the problem of fixed domain name: specify the subdomain name with local tunnel to achieve the purpose of Intranet penetration
2，Apply to the power of the tiktok development platform, get the necessary client secret and client key.
3，Use postman to successfully monitor the event and return the random number required by the platform
Current problems:
1，Tiktok can quickly resolve and return to the random number required by the platform, but it can not deploy URL successfully to webhook in the development platform.
Current solutions:
1，Tiktok development platform negotiations
2，Temporarily skip the event subscription and use the sandbox environment for interface call and function debugging
3， Change an intranet penetration method and try to deploy webhook again
Next week's plan:
1，Continue to look at the source code and learn to call the interface
2，Obtain access in the sandbox environment access_Token and client_token
3，Try to call an interface successfully

This issue is nothing related to the Wechaty Official Website.
Closed.


Transformer missing exception

Reported issue

The combination was something like

https://github.com/alnutile/larachain/actions/runs/5248223724 this might help. I could not replicate the issue but setup it up and saw two other issues that could lead to something.

when scraping a root site page it named the file .html so just to make that cleaner it now is index.html
I remove unicode from the html string just to make sure it does not mess up any thing.

There are some fixes here but since I could not replicate the code I will just see if it happens again


mod_header: add client cert related variables (177)

Codecov Report

Merging #224 into develop will decrease coverage by 0.06%.
The diff coverage is 7.14%.

@@             Coverage Diff             @@
##           develop     #224      +/-   ##
===========================================
- Coverage    53.05%   52.98%   -0.07%
===========================================
  Files          219      219
  Lines        21721    21727       +6
===========================================
- Hits         11524    11512      -12
- Misses        8891     8910      +19
+ Partials      1306     1305       -1

Impacted Files
Coverage Δ

bfe_modules/mod_header/action_header_var.go
0% <0%> (ø)
:arrow_up:

bfe_util/get_l4lb_info.go
0% <0%> (ø)
:arrow_up:

bfe_config/bfe_conf/conf_basic.go
70.43% <100%> (ø)
:arrow_up:

bfe_util/fixed_pool.go
83.33% <0%> (-16.67%)
:arrow_down:

bfe_spdy/server_conn.go
78.53% <0%> (-0.45%)
:arrow_down:

bfe_http2/server.go
71.5% <0%> (-0.33%)
:arrow_down:

bfe_server/web_server.go
0% <0%> (ø)
:arrow_up:

bfe_spdy/frame_write.go
50.5% <0%> (ø)
:arrow_up:

bfe_server/server_status.go
0% <0%> (ø)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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


refactor!: remove email module

It is not complete and is not used in Delta Chat

is not used in Delta Chat

That is not the only reason things exist in rpgp. But I agree that it should be removed from this repo


Enable using script and nonextended keys

[x] Support for key hashes and non-extended public key implementation using proper bits from cardano-addresses (rather than hack)
[x] ApiCredential extension, also in swagger, golden regenerated in core unit testig
[x] Added comprehensive instruction and more testing in integration suite

Comments

Issue Number
adp-2114

this is the commit from https://github.com/input-output-hk/cardano-addresses/pull/197
Of course #197 in cardano-addresses need to be merged first, and then cardano-addresses' commit bumped once again in this PR

bors r+


Update documentation at https://github.com/ufal/lindat-dspace/wiki/Connecting-Tomcat-with-Apache

simplified the docs a bit https://github.com/ufal/lindat-dspace/wiki/Connecting-Tomcat-with-Apache


Getting device info... this may take a second

macOS 12.6
iphone7 plus ios 15.3.1
in dfu mode, the iphone stuck on "Getting device info... this may take a second"

sudo ./sshrd.sh 15.6.1

I should merge the device info getter into 1 command so it doenst do it many times

How do I solve this problem?


[CG-47] Change references from segmentio to SafetyCulture

Looks fine, though don't merge with a broken .buildkite/pipeline.yaml. Name it something .TOFIX or something if you have to, else rm it.

Oops pushed on this branch apparently -- will revert -_-


Setting chord duration?

If I set a duration in the root note of a chord and also as an argument to the chord constructor, like this:
require('teoria').note('c4',{value:16}).chord('M',{value:8}).notes()

then none of these are used, and instead duration is set to 4:
[ { duration: { value: 4, dots: 0 }, coord: [ 1, -3 ] },
  { duration: { value: 4, dots: 0 }, coord: [ -1, 1 ] },
  { duration: { value: 4, dots: 0 }, coord: [ 1, -2 ] } ]
 [ { duration: { value: 4, dots: 0 }, coord: [ 1, -3 ] },
 { duration: { value: 4, dots: 0 }, coord: [ -1, 1 ] },
 { duration: { value: 4, dots: 0 }, coord: [ 1, -2 ] } ]
[ { duration: { value: 4, dots: 0 }, coord: [ 1, -3 ] },
  { duration: { value: 4, dots: 0 }, coord: [ -1, 1 ] },
  { duration: { value: 4, dots: 0 }, coord: [ 1, -2 ] } ]

Wouldn't it be better to be able to set duration of a chord either in the chord constructor (like note), or just set it to be the duration of the root note?

It can sort of be worked around by taking the duration of the root (or tonica for scales). It would still be slightly more intuitive if either there was a method for setting it or it just left the 4 out.
//Assuming note.play() plays a note

teoria.Chord.prototype.play = function(){
  var that = this
  that.notes().map(
    function(note){
      //set duration to that of the root
      if (that.root){
        note.duration = that.root.duration
      }
      note.play()
    }
  )
}

teoria.Chord.prototype.play = function(){
  var that = this
  that.notes().map(
    function(note){
      //set duration to that of the root
      if (that.root){
        note.duration = that.root.duration
      }
      note.play()
    }
  )
}

Hi Pelle. I personally think that the duration stuff should be removed. This library could focus on the theory, and then another library could extend the theory with features important for playback and sheet music.
Do you use the duration features?

If they work well, I want to use them. The harmonics are really well worked out with loads of scales and chord,s and it works to the point where you can just give the midi pitch to some MIDI API; whereas the time/duration stuff maybe has some way to go before being able to do that, such as for example knowing when a note is being played; maybe both in bar/beat and (milli)seconds.
I'd suggest either cutting it or improving it.
I pushed my tiny >100 line library for rhythm here, because I couldn't find any here. If that kind of stuff is within the scope of teoria, fell free to incorporate it..

Should be fixed by #110


Renamed response object to twiml.

@jarodreyes came up with this change to try to avoid naming the TwimL response variable "response" - we thought "twiml" might be more descriptive.
But we both forgot that the original import was named twiml, which Travis reminded us about. We'll keep noodling on a better naming convention for building TwimL responses in Python.


Enhancement: [Prompt Library] Let users create their own categories

What features would you like to see added?
Current Behavior:
Currently, users can only use predefined categories to organize their prompts in the Prompt Library. This limitation restricts users from customizing their prompt organization based on their specific needs and preferences.
Proposed Behavior:
Introduce a feature that enables users to create, edit, and delete custom categories for their prompts. This would allow for greater flexibility and personalization in organizing the Prompt Library.
Use Case:

Users with specific projects or themes can create relevant categories to keep their prompts organized.
Users can categorize prompts based on personal or team workflows, enhancing productivity and ease of use.
Custom categories can help in quickly locating prompts, especially when dealing with a large number of entries.

If this is something the community needs, I would like to contribute to it.
More details
This can mean couple of changes in the filter part of the prompt library since there could be unlimited numbers of different categories.
Which components are impacted by your request?
UI, Endpoints
Pictures
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Yes highly needed, also the ability to share entire categories ("prompt bundles?") with certain users

Definately need this


Refactor app for delivery

Codecov Report

Merging #3 into master will increase coverage by 1.94%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master     #3      +/-   ##
========================================
+ Coverage   98.05%   100%   +1.94%
========================================
  Files          11     12       +1
  Lines         103    102       -1
  Branches       16     15       -1
========================================
+ Hits          101    102       +1
+ Misses          2      0       -2

Impacted Files
Coverage Δ

src/Components/NotepadView.jsx
100% <ø> (+12.5%)
:arrow_up:

src/Components/AnimatedNote.jsx
100% <ø> (ø)
:arrow_up:

src/Components/ToggleButton.jsx
100% <100%> (ø)

src/Components/Note.jsx
100% <100%> (ø)
:arrow_up:

src/Components/NewNote.jsx
100% <100%> (+7.14%)
:arrow_up:

src/Containers/NoteContainer.jsx
100% <100%> (ø)
:arrow_up:

src/Components/ModalToggle.jsx
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


running time on OAG_CS dataset

Hi, thanks for providing the awesome code of GPT-GNN.
I am trying to run your code on OAG_CS dataset but I am not sure if I get it right. In the paper, the reported pre-training time is about 10-12 hours for 400 epochs while it took much longer on my side. I wonder if you could specify the requirements of the computational  resources. For example, how many cpus do I need for achieving a pre-training time of 10 hours? I attached the output for my run as follows.
+--------------------+-------------------------------+
| Parameter          | Value                         |
+--------------------+-------------------------------+
| attr_ratio         | 0.500                         |
+--------------------+-------------------------------+
| attr_type          | text                          |
+--------------------+-------------------------------+
| neg_samp_num       | 255                           |
+--------------------+-------------------------------+
| queue_size         | 256                           |
+--------------------+-------------------------------+
| w2v_dir            | ./data/oag_output/w2v_all     |
+--------------------+-------------------------------+
| data_dir           | ./data/oag_output/graph_CS.pk |
+--------------------+-------------------------------+
| pretrain_model_dir | ./tmp/model/gta_all_cs3       |
+--------------------+-------------------------------+
| cuda               | 0                             |
+--------------------+-------------------------------+
| sample_depth       | 6                             |
+--------------------+-------------------------------+
| sample_width       | 128                           |
+--------------------+-------------------------------+
| conv_name          | hgt                           |
+--------------------+-------------------------------+
| n_hid              | 400                           |
+--------------------+-------------------------------+
| n_heads            | 8                             |
+--------------------+-------------------------------+
| n_layers           | 3                             |
+--------------------+-------------------------------+
| prev_norm          | 0                             |
+--------------------+-------------------------------+
| last_norm          | 0                             |
+--------------------+-------------------------------+
| dropout            | 0.200                         |
+--------------------+-------------------------------+
| max_lr             | 0.001                         |
+--------------------+-------------------------------+
| scheduler          | cycle                         |
+--------------------+-------------------------------+
| n_epoch            | 20                            |
+--------------------+-------------------------------+
| n_pool             | 8                             |
+--------------------+-------------------------------+
| n_batch            | 32                            |
+--------------------+-------------------------------+
| batch_size         | 256                           |
+--------------------+-------------------------------+
| clip               | 0.500                         |
+--------------------+-------------------------------+
Start Loading Graph Data...
Finish Loading Graph Data!
paper PP_cite
paper rev_PP_cite
venue rev_PV_Conference
venue rev_PV_Journal
field rev_PF_in_L3
field rev_PF_in_L1
field rev_PF_in_L2
field rev_PF_in_L4
author AP_write_last
author AP_write_other
author AP_write_first
Start Pretraining...
Data Preparation: 80.1s
Epoch: 1, (1 / 41) 55.9s  LR: 0.00010 Train Loss: (5.129, 10.292)  Valid Loss: (5.082, 9.933)  NDCG: 0.306  Norm: 0.666  queue: 12
UPDATE!!!
Data Preparation: 23.3s
Epoch: 1, (2 / 41) 45.0s  LR: 0.00015 Train Loss: (4.877, 9.236)  Valid Loss: (4.861, 8.130)  NDCG: 0.320  Norm: 0.950  queue: 12
UPDATE!!!
Data Preparation: 34.6s
Epoch: 1, (3 / 41) 40.1s  LR: 0.00021 Train Loss: (4.776, 7.650)  Valid Loss: (4.899, 6.895)  NDCG: 0.327  Norm: 1.243  queue: 12
UPDATE!!!
Data Preparation: 37.3s
Epoch: 1, (4 / 41) 42.5s  LR: 0.00027 Train Loss: (4.716, 6.930)  Valid Loss: (4.697, 6.571)  NDCG: 0.334  Norm: 1.493  queue: 12
UPDATE!!!
Data Preparation: 33.9s
Epoch: 1, (5 / 41) 40.8s  LR: 0.00032 Train Loss: (4.635, 6.624)  Valid Loss: (4.614, 6.290)  NDCG: 0.341  Norm: 1.950  queue: 12
UPDATE!!!
Data Preparation: 38.9s
Epoch: 1, (6 / 41) 45.2s  LR: 0.00038 Train Loss: (4.572, 6.470)  Valid Loss: (4.568, 6.386)  NDCG: 0.357  Norm: 2.481  queue: 12
Data Preparation: 30.5s
Epoch: 1, (7 / 41) 42.7s  LR: 0.00044 Train Loss: (4.438, 6.391)  Valid Loss: (4.501, 6.224)  NDCG: 0.371  Norm: 2.532  queue: 12
UPDATE!!!

Hi:
From your log, it seems the bottleneck is the sampling (which is conducted on CPU). My previous setting is 8* CPU E5-2698 v4 @ 2.20GHz. (But the machine also runs other experiments so it's just a reference)
My previous implementation of the sampling is not very efficient. I'll update them to make it more efficient later.

Thank you for the quick reply and looking forward to the efficient version :)

Hi, is there any progress about the efficient version?


Separate Sequence Test from other RevEng tests

Steps to reproduce
In f873682 I put [SqlServerCondition(SqlServerCondition.SupportsSequences)] on the 2 tests SqlServerE2ETests.E2ETest_UseAttributesInsteadOfFluentApi and SqlServerE2ETests.E2ETest_AllFluentApi. To repro simply remove those attributes and run those tests on a box where the version of SQL Server is 2008.
The issue
This was a temporary measure to prevent CI failures. Long term the correct solution is to pull out the part of the test that depends on a CREATE SEQUENCE statement (see E2E.sql) into a separate test, remove that attribute from the 2 old tests and have only the new test depend on [SqlServerCondition(SqlServerCondition.SupportsSequences)].

We should discuss the correct solution for this before implementing

Had the discussion with @rowanmiller. Proceeding as planned.


[DDW-499] Enable Cardano Wallet API documentation server in development mode

This PR enables Cardano Wallet API documentation server in development mode.
While Daedalus is running you can reach the documentation server using your Browser on the following URL: https://localhost:8091/docs/v1/index/
Screenshots:

Review Checklist:
Basics

[ ] PR is updated to the most recent version of target branch (and there are no conflicts)
[ ] PR has good description that summarizes all changes and shows some screenshots or animated GIFs of important UI changes
[ ] CHANGELOG entry has been added and is linked to the correct PR on GitHub
[ ] Automated tests: All acceptance tests are passing (yarn run test)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in development build (yarn run dev)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in production build (yarn run package / CI builds)
[ ] There are no flow errors or warnings (yarn run flow:test)
[ ] There are no lint errors or warnings (yarn run lint)
[ ] Text changes are proofread and approved (Jane Wild)
[ ] There are no missing translations (running yarn run manage:translations produces no changes)
[ ] UI changes look good in all themes (Alexander Rukin)
[ ] Storybook works and no stories are broken (yarn run storybook)
[ ] In case of dependency changes yarn.lock file is updated

Code Quality

[ ] Important parts of the code are properly documented and commented
[ ] Code is properly typed with flow
[ ] React components are split-up enough to avoid unnecessary re-rendering
[ ] Any code that only works in Electron is neatly separated from components

Testing

[ ] New feature / change is covered by acceptance tests
[ ] All existing acceptance tests are still up-to-date
[ ] New feature / change is covered by Daedalus Testing scenario
[ ] All existing Daedalus Testing scenarios are still up-to-date

After Review:

[ ] Merge PR
[ ] Delete source branch
[ ] Move ticket to done on the Youtrack board

@KtorZ please drop an eye on this change - this is what I have asked you about earlier today 🙏

@nikolaglumac buildkite is failing, other than that 👍


Support related include/exclude in configuration

Something like:
class ArticleSchema(PydanticDjangoModel):

    publications: List[PublicationSchema]

    class Config:
        model = Article
        exclude = ["publications__created_at", "publications__updated_at"]

I think this can be supported by additional schemas which implement their own include/exclude behaviour - this issue probably predates that support.


Long range electrostatics

Description
This PR implements the infrastructure to calculate long-range electrostatics. In the first version, we will calculate the long-range term using a damped coulomb potential (see ref for details).
Todos

[x] Add infrastructure to support the calculation of long-range electrostatic in a postprocessing step
[x] implement damped coulomb potential
[ ] add tests

Status

[ ] Ready to go

Thanks for looking through this. I believe this PR was not ready for review; tests are also missing. I will get this prepared until Monday.


ci: backport deps by default to 0.1.x

Description
Backport by default to the release/v0.1.x branch
Makes us stay on top of our game - we can choose not to backport bigger stuff like the 0.47 migration

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[ ] included the correct type prefix in the PR title
[ ] added ! to the type prefix if API or client breaking change
[ ] added appropriate labels to the PR
[ ] targeted the correct branch
[ ] provided a link to the relevant issue or specification
[ ] added a changelog entry to CHANGELOG.md
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

I'm gonna close this and add it to #183


Event validation error with SNS Event

I'm facing currently a validation error using the Lambda Powertools Python libary in a lambda function.
The validation command is:

 from create.utils.inputSchemas import input
from aws_lambda_powertools.utilities.validation import validator, envelopes
@validator(inbound_schema=input, envelope=envelopes.SNS)
def lambda_handler(event, context):

    [...]

For me everything seems right but I'm getting the following error:
[ERROR] JMESPathTypeError: In function powertools_json(), invalid type for value: {'version': '0', 'id': '0000000-0000-0000-0000-00000000', 'detail-type': 'EC2 Instance State-change Notification', 'source': 'aws.ec2', 'account': '0000000000', 'time': '2021-05-18T14:14:12Z', 'region': 'eu-central-1', 'resources': ['arn:aws:ec2:eu-central-1:000000000:instance/i-0000000000'], 'detail': {'instance-id': 'i-000000000', 'state': 'terminated'}}, expected one of: ['string'], received: "object"
Traceback (most recent call last):
  File "/var/task/aws_lambda_powertools/middleware_factory/factory.py", line 133, in wrapper
    response = middleware()
  File "/var/task/aws_lambda_powertools/utilities/validation/validator.py", line 119, in validator
    event = unwrap_event_from_envelope(data=event, envelope=envelope, jmespath_options=jmespath_options)
  File "/var/task/aws_lambda_powertools/utilities/validation/base.py", line 65, in unwrap_event_from_envelope
    return jmespath.search(envelope, data, options=jmespath.Options(**jmespath_options))
  File "/var/task/jmespath/__init__.py", line 23, in search
    return parser.Parser().parse(expression).search(data, options=options)
  File "/var/task/jmespath/parser.py", line 509, in search
    result = interpreter.visit(self.parsed, value)
  File "/var/task/jmespath/visitor.py", line 94, in visit
    return method(node, *args, **kwargs)
  File "/var/task/jmespath/visitor.py", line 269, in visit_pipe
    result = self.visit(node, result)
  File "/var/task/jmespath/visitor.py", line 94, in visit
    return method(node, *args, **kwargs)
  File "/var/task/jmespath/visitor.py", line 171, in visit_function_expression
    return self._functions.call_function(node['value'], resolved_args)
  File "/var/task/jmespath/functions.py", line 80, in call_function
    self._validate_arguments(resolved_args, signature, function_name)
  File "/var/task/jmespath/functions.py", line 91, in _validate_arguments
    return self._type_check(args, signature, function_name)
  File "/var/task/jmespath/functions.py", line 97, in _type_check
    self._type_check_single(actual[i], allowed_types,
  File "/var/task/jmespath/functions.py", line 111, in _type_check_single
    raise exceptions.JMESPathTypeError(END RequestId: 92c00173-eb6c-4035-b3f7-80f009e95035

That doesn't make any sense for me, because why is validate expecting a string, usually event is a dict object and I can't find any information is the documentation.
The event for testing the validation is:
{
  "Records": [
    {
      "EventSource": "aws:sns",
      "EventVersion": "1.0",
      "EventSubscriptionArn": "arn:Subscriptionname",
      "Sns": {
        "Type": "Notification",
        "MessageId": "00000000-0000-0000-0000-000000000",
        "TopicArn": "TOPICARN",
        "Subject": "None",
        "Message": {
          "version": "0",
          "id": "00000000-0000-0000-0000-000000000000",
          "detail-type": "EC2 Instance State-change Notification",
          "source": "aws.ec2",
          "account": "000000000000",
          "time": "2021-05-18T14:14:12Z",
          "region": "eu-central-1",
          "resources": [
            "arn:aws:ec2:eu-central-1:000000000000:instance/i-00000000000000000"
          ],
          "detail": {
            "instance-id": "i-00000000000000000",
            "state": "terminated"
          }
        },
        "Timestamp": "2021-05-18T15:55:35.862Z",
        "SignatureVersion": "1",
        "Signature": "000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000==",
        "SigningCertUrl": "https://sns.eu-central-1.amazonaws.com/SimpleNotificationService-00000000000000000000000000000000.pem",
        "UnsubscribeUrl": "https://sns.eu-central-1.amazonaws.com/?Action=Unsubscribe&SubscriptionArn=arn:aws:sns:eu-central-1:00000000000000000000:00000000-0000-0000-0000-000000000000",
        "MessageAttributes": {}
      }
    }
  ]
}

The validation scheme is:
   {
        "definitions": {},
        "$schema": "http://json-schema.org/draft-07/schema#",
        "$id": "https://example.com/object1622542263.json",
        "title": "Root",
        "type": "object",
        "required": [
            "version",
            "id",
            "detail-type",
            "source",
            "account",
            "time",
            "region",
            "resources",
            "detail"
        ],
        "properties": {
            "version": {
                "$id": "#root/version",
                "title": "Version",
                "type": "string",
                "default": "",
                "pattern": "^.*$"
            },
            "id": {
                "$id": "#root/id",
                "title": "Id",
                "type": "string",
                "default": "",
                "pattern": "^.*$"
            },
            "detail-type": {
                "$id": "#root/detail-type",
                "title": "Detail-type",
                "type": "string",
                "default": "",
                "pattern": "^.*$"
            },
            "source": {
                "$id": "#root/source",
                "title": "Source",
                "type": "string",
                "default": "",
                "pattern": "^.*$"
            },
            "account": {
                "$id": "#root/account",
                "title": "Account",
                "type": "string",
                "default": "",
                "pattern": "^.*$"
            },
            "time": {
                "$id": "#root/time",
                "title": "Time",
                "type": "string",
                "default": "",
                "pattern": "^.*$"
            },
            "region": {
                "$id": "#root/region",
                "title": "Region",
                "type": "string",
                "default": "",
                "pattern": "^.*$"
            },
            "resources": {
                "$id": "#root/resources",
                "title": "Resources",
                "type": "array",
                "default": [],
                "items": {
                    "$id": "#root/resources/items",
                    "title": "Items",
                    "type": "string",
                    "default": "",
                    "pattern": "^.*$"
                }
            },
            "detail": {
                "$id": "#root/detail",
                "title": "Detail",
                "type": "object",
                "required": [
                    "instance-id",
                    "state"
                ],
                "properties": {
                    "instance-id": {
                        "$id": "#root/detail/instance-id",
                        "title": "Instance-id",
                        "type": "string",
                        "default": "",
                        "pattern": "^.*$"
                    },
                    "state": {
                        "$id": "#root/detail/state",
                        "title": "State",
                        "type": "string",
                        "default": "",
                        "pattern": "^.*$"
                    }
                }
            }

        }
    }

Is it a bug or my mistake?

Hey @aquisx - Thanks for raising this!
How did you create the testing event?
By looking at the event, I'd guess you have EventBridge -> SNS -> Lambda. In that case, EventBridge will serialize the EC2 event and pass a Payload to SNS - hence why it's failing because SNS doesn't send the payload message as a dictionary but as a JSON string.
I've just re-created that to double check in case SNS changed the contract - it confirms EventBridge sends as a JSON string[1], which works as expected since Validator will deserialize the JSON string with the envelope feature before it applies schema validation:
Hope that helps!

Your test event with the payload now serialized - this works
{
    "Records": [
        {
            "EventSource": "aws:sns",
            "EventVersion": "1.0",
            "EventSubscriptionArn": "arn:Subscriptionname",
            "Sns": {
                "Type": "Notification",
                "MessageId": "00000000-0000-0000-0000-000000000",
                "TopicArn": "TOPICARN",
                "Subject": "None",
                "Message": '{"version": "0", "id": "00000000-0000-0000-0000-000000000000", "detail-type": "EC2 Instance State-change Notification", "source": "aws.ec2", "account": "000000000000", "time": "2021-05-18T14:14:12Z", "region": "eu-central-1", "resources": ["arn:aws:ec2:eu-central-1:000000000000:instance/i-00000000000000000"], "detail": {"instance-id": "i-00000000000000000", "state": "terminated"}}'
                ,
                "Timestamp": "2021-05-18T15:55:35.862Z",
                "SignatureVersion": "1",
                "Signature": "000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000==",
                "SigningCertUrl": "https://sns.eu-central-1.amazonaws.com/SimpleNotificationService-00000000000000000000000000000000.pem",
                "UnsubscribeUrl": "https://sns.eu-central-1.amazonaws.com/?Action=Unsubscribe&SubscriptionArn=arn:aws:sns:eu-central-1:00000000000000000000:00000000-0000-0000-0000-000000000000",
                "MessageAttributes": {},
            },
        }
    ]
}

[1] Raw JSON event from EventBridge->SNS->Lambda
{
    "Records": [
        {
            "EventSource": "aws:sns",
            "EventVersion": "1.0",
            "EventSubscriptionArn": "arn:aws:sns:eu-west-1:123456789012:powertools457-issue:08d71ab7-45d0-4a14-a3ba-58a7ee1d5bc0",
            "Sns": {
                "Type": "Notification",
                "MessageId": "21e66291-ea41-530e-bfd6-3ba0886e1211",
                "TopicArn": "arn:aws:sns:eu-west-1:123456789012:powertools457-issue",
                "Subject": null,
                "Message": "{\"version\":\"0\",\"id\":\"ca6432a8-89b0-0007-8e23-c57c29435065\",\"detail-type\":\"EC2 Instance State-change Notification\",\"source\":\"aws.ec2\",\"account\":\"123456789012\",\"time\":\"2021-06-02T10:39:43Z\",\"region\":\"eu-west-1\",\"resources\":[\"arn:aws:ec2:eu-west-1:123456789012:instance/i-0aef86944196d09b7\"],\"detail\":{\"instance-id\":\"i-0aef86944196d09b7\",\"state\":\"running\"}}",
                "Timestamp": "2021-06-02T10:39:43.471Z",
                "SignatureVersion": "1",
                "Signature": "cxBQDU+k/SXYezXEIfiwW4dEdFOlW3T6QLq1qEeFCqzJTWacMttXXRwRLpj3NU1bjD1dDdgOqhZZIql//EXIQALiAJkoi3mo0Cfn6kuFTyfZ1N4mlnehbHC+scOGbo8K0gafuVZJJj3nNCHqsb7WvrKAUkajCerunoa0tQcwJchQJsZf/Th8jUlARM8xc97n5n7p0rkdQGT77/dRAXG2YtBZuwpxwQCdtrUrvaD7vFiF1aVzCy8jj4PLuN8jcLZWQTyvLY1aZDKEm/9TR6H87g6aRn5M/d8Tqrdov7cgoRJlYuZWL+Kk0CBybramLguGNWpTPrxeWpQ8l8DzvH/vAQ==",
                "SigningCertUrl": "https://sns.eu-west-1.amazonaws.com/SimpleNotificationService-010a507c1833636cd94bdb98bd93083a.pem",
                "UnsubscribeUrl": "https://sns.eu-west-1.amazonaws.com/?Action=Unsubscribe&SubscriptionArn=arn:aws:sns:eu-west-1:123456789012:powertools457-issue:08d71ab7-45d0-4a14-a3ba-58a7ee1d5bc0",
                "MessageAttributes": {}
            }
        }
    ]

Closing this as we haven't heard from you - feel free to reopen otherwise.
Thank you


highlight element not showing in  side navbar

highlight element not showing in side navbar. but other element showing in outside of navbar. @kamranahmedse if you have any solution please give me.

same problem please guided me how to solve this problems

you can add class after import driver.min.css
.driver-highlighted-element { pointer-events: none; }

add
but its not work

We just released v1.0 of driver.js which has been rewritten from the ground up. This issue should no longer exist. Please have a look at the docs and do let me know if you face any issues.


Unrecognized Data

{
    "before": "ba7120f8f2fa7fc05be129a63a8d44524a9b6755",
    "after": "488f91c0e80f7e1f21a232465249d130a8183440",
    "ref": "refs/heads/develop",
    "user_id": 18,
    "user_name": "fuyi.liu",
    "project_id": 185,
    "repository": {
        "name": "*******",
        "url": "*******",
        "description": "*******",
        "homepage": "*******"
    },
    "commits": [
        {
            "id": "488f91c0e80f7e1f21a232465249d130a8183440",
            "message": "*******",
            "timestamp": "2016-08-23T13:47:59+08:00",
            "url": "*******",
            "author": {
                "name": "*******",
                "email": "*******"
            }
        }
    ],
    "total_commits_count": 1
}

When I push to gitlab slack is Unrecognized Data

That POST is missing the object_kind property. A long time ago, GitLab updated their webhooks to all include that property which is used to differentiate the types. What version of GitLab are you on?

My GitLab version is 7.6.2.

If it doesn't have the object_kind property, it's too old for this. Sorry about that.

@bingofly I'm having the same issue with gitlab 6.  I made a few changes based on @nazrhyn's comment above.
https://github.com/pdericson/gitlab-slack/tree/gitlab-6

@pdericson If you'd like to make a PR based on your changes, I'd be happy to review it!


Special weather time series types

Description of the issue
How is it possible to annotate special weather time series types. In a forecast model I have different input time series. Among others a time series of snow cover in mm and direct solar radiation in kW. For both I have chosen the entity "weather time series". For the direct solar radiation time series I have created a relationship via the property "is about" to an individual of the entity "direct solar radiation". For the snow cover time series this was not possible, because there is no entity that expresses that it is about snow cover. The question now is, if it is at all intended to connect a weather time series via the property "is about" to an entity with meta knowledge about the time series like "direct solar radiation" or if more specific SubClasses of weather time series should be defined.
Ideas of solution
If a weather time series is best defined using the "is about" property, or possibly another property, I would suggest adding the "snow" subclass below the "water" entity. However, if a weather time series should be better described with a subclass of the entity "process", as it would be the case for "direct solar radiation" or e.g. for a wind speed time series, I would suggest to create an entity "snow cover" or "snowfall rate" below the process.
However, if it would be better to create more specific weather time series, I would suggest to create e.g. an entity "snow cover time series", "wind speed time series", "direct solar radiation time series" etc.
Workflow checklist

[ ] I discussed the issue with someone else than me before working on a solution
[ ] I already read the latest version of the workflow for this repository
[ ] The goal of this ontology is clear to me

I am aware that

[ ] every entry in the ontology should have a definition
[ ] classes should arise from concepts rather than from words

ENVO has a concept for snow which could be reused. It is classified as environmental material, a class that OEO does not have. But water might be a good parent class for us.
For both, water and snow, a quality depth or level could be introduced.

I'm ok with your proposal. Since it seems to be at least inspired by ENVO, we might cite or mention it, though, e.g. by an annotation may be identical to or definition source.

I'd be ok to introduce specific weather time series, as proposed by @ap-peh , maybe as equivalent classes.
Regarding the axioms for specific weather time series, is about is probably the best that we have at the moment. But I think creating a subproperty that is more specific, in the sense that the time series represents the very parameter referred to, is a good idea for proper time series description.

Another time series needed for a forecasting model is a consumption time series. Here, too, the entity Consumption already exists and should be used to express that the time series to be described is a consumption time series or should an extra entity be introduced as in the case of the weather time series.

Another time series needed for a forecasting model is a consumption time series. Here, too, the entity Consumption already exists and should be used to express that the time series to be described is a consumption time series or should an extra entity be introduced as in the case of the weather time series.

consumption time series EquivalentTo: 'time series' and 'is about' some consumption
just like this @ap-peh?

From your comment, I figure that forecast time series should be an own type of time series, since weather time series can be forecast time series or measured or even modelled/synthetic time series.
This might require a complex parallel structure of equivalence dependencies. But probably useful.

Another time series needed for a forecasting model is a consumption time series. Here, too, the entity Consumption already exists and should be used to express that the time series to be described is a consumption time series or should an extra entity be introduced as in the case of the weather time series.

consumption time series EquivalentTo: 'time series' and 'is about' some consumption just like this @ap-peh?

I would suggest creating a specific property for the statement: The time series is of type XY. This makes it clear that if this property is used, then it is a relationship between a time series and the type assigned to it. To keep the entity time series consistent I would remove special entities like generation time series.

I would suggest creating a specific property for the statement: The time series is of type XY.

Ok, good idea. time series, being GDCs, are not intuitive for adding properties, i.e. SDCs, yet we did it already for some entities, e.g. model. There we chose role, which might be the most suitable SDC here, too. @viktorwichern @nelekoehler can you backcheck with BFO please?
I see several roles then: forecast, measured, synthetic, weather, generation, consumption
Are there more?

Can we please start with definitions first before deciding how we implement this? I have not seen any proposals for definitions yet.
For forecast time series I see two completely different possible definitions:

A forecast time series is a time series that is used to compute a forecast.

These are two completely different things: the first one is an input into some forecasting process, the other the result of a forecasting process.

For forecast time series I see two completely different possible definitions:

A forecast time series is a time series that is used to compute a forecast.

I have the second "type" in mind, i.e. output of a forecast computation.
To draft a definition and come back to my proposal above: forecast time series role (or other kind of SDC): A forecast time series role is a role of a time series that is output of a forecasting model calculation.
forecasting model calculation is obviously not (yet) part of the OEO, though.

Ok, good idea. time series, being GDCs, are not intuitive for adding properties, i.e. SDCs, yet we did it already for some entities, e.g. model. There we chose role, which might be the most suitable SDC here, too. @viktorwichern @nelekoehler can you backcheck with BFO please?

So, the definition for role in the BFO is:
Each instance of role is a realizable entity that (1) exists because the bearer is in some special physical, social, or institutional set of circumstances in which the bearer does not have to be (optionality), and (2) is not such that, if this realizable entity ceases to exist, then the physical make-up of the bearer is thereby changed (external grounding).
For GDC's, their "aboutness" defines their physical properties. So if the relation (through a SDC) we want to express would concern their "aboutness"  this would be inconsistent with the definition of role. If a PDF could have the role to be "about", say, weather, instead of budgeting plans, if it would lose this role, it would cease to be what it was. Therefore, roles cannot dictate the "aboutness", especially of information content entities. A possible role could be something like "study material". A PDF could either be study material or not, if it would lose the role, the bearer would not change.
Also, role is a realizable entity, which is defined as having at least one independent continuant entity as its bearer, so a GDC probably can't have a role anyway.
.. which is kind of unfortunate, because function feels like it would fit pretty well here (but it is a realizable entity)
A function is a special kind of disposition. It is a realizable entity whose realization is an end-directed activity of its bearer that occurs because this bearer is (a) of a specific kind and (b) in the kind or kinds of contexts that it is made or selected for. Thus a function is a disposition that exists in virtue of the bearer’s physical make-up, and this physical make-up is something the bearer possesses because of how it came into being—either through natural selection (in the case of biological entities) or through intentional design (in the case of artifacts). Roughly, the entities in question came into being in order to perform activities of a certain sort, called “functionings.”
TL;DR: I do not think SDC's can inhere in GDC's, but this is very complicated and I would like a second opinion, preferably from another philosopher. Also, roles should theoretically not concern the aboutness (of information content entities).

Also, role is a realizable entity, which is defined as having at least one independent continuant entity as its bearer, so a GDC probably can't have a role anyway.
[...]
TL;DR: I do not think SDC's can inhere in GDC's, but this is very complicated and I would like a second opinion, preferably from another philosopher. Also, roles should theoretically not concern the aboutness (of information content entities).

In general, you are right. SDC are supposed to only have IC as bearer. Yet, in "real ontology development", it showed that some GDC also require specific "properties". I discussed this with @fneuhaus several times, and it seems that in other ontology communities this constraint is weakend as well.
In the OEO, we added the model role to model (GDC), for instance.
If you're interested in this discussion, we can ask @fabianneuhaus to discuss this in a meeting.

.. which is kind of unfortunate, because function feels like it would fit pretty well here (but it is a realizable entity)

nice, function is better than role!


v0.4.0 previously worked - Assigning to 'uint32_t' (aka 'unsigned int') from incompatible type 'std::optional'

I was successfully using the v0.4.0 release, but today, just after cleaning the cache of SPM, I encountered this error:

I created a test project by adding the version 0.4.0, but I still encountered the same error. It's quite strange that after some time, the release stopped working.
But using main branch "fixes" this.

SPM probably updated CXXAudioUtilities to the newest version which has a few breaking changes. The versions of the dependencies specified in Package.resolved are the versions I tested the release with and should work. Newer minor/patch versions "should" work, but I don't strictly adhere to semantic versioning for versions less than 1.0 so there is no guarantee.

Yep, that was my idea too, but I don't understand how this is possible if for 0.4.0 which I used before.
dependencies: [
	.package(url: "https://github.com/sbooth/CXXAudioUtilities", from: "0.1.0"),

Okay, the latest CXXAudioUtilities is 0.1.2, but 0.4.0 is not using it.. so how.. it worked since I've added 0.4.0, but now if you create a new project and add it into, it wont.


[fix] Crunchyroll login

I see on web guide for login on Crunchyroll using API
Reference https://github.com/CloudMax94/crunchyroll-api/wiki
Support JSON  and XML response
{request} = API request
{version} = your version (better use value 0)
{sess_id} = Session ID as a cookie named "sess_id"
Response Type | URL
JSON | api.crunchyroll.com/{request}.{version}.json?&session_id={sess_id}
XML | api.crunchyroll.com/{request}.{version}.xml?&session_id={sess_id}

Carefully read new issue template and provide all requested information.


Missing periods.

Some comment formatting inconsistency.
Fixes #.
Added missing periods.

Much needed improvement. Thank you :+1:

Great job! 👍

Thanks for your interest in the repo.  In general, we won't be accepting comment-only fixes to our code unless it's fixing an identified, logical issue (like #104 did).  We will accept fixes to comments as part of larger pull requests.


Huobi.me not working any more

Need to update this codebase.

yes, need to update... :(

These exchange interfaces are updated frequently


Xtensor template fix

This is a workaround for newer versions of LLVM that are having some sort of interaction with xtensor that is causing issues. Notably, this will also affect the main development branch of OpenMC eventually, though is much more of a present issue for the OMP fork given our reliance on newer compilers.
The intertwined issues are a little hard to parse, but I'm gathering that the issue may be in xtensor itself rather than with LLVM, but I'm not 100% on that.
Thankfully, there is an easy workaround flag that can be passed to LLVM that fixes the error. I think we should add it in now so that our auto testing works again and I'll plan on checking manually now and then to see if the flag is still required or not (and/or if an update to the xtensor submodule fixes it).
See issues:

https://github.com/llvm/llvm-project/issues/91504
https://github.com/xtensor-stack/xtensor/issues/2783

Note: I was running into this tonight building openmc on my mac. the LLVM homebrew gives you now runs into this.


agent0: bring back hyperdrive::smart_long policy

We deleted the smart_long.py file in #909 because it required a few yieldspace functions that we no longer support. Once #910 is complete, we should be able to bring the smart_long back easily.

Gotta do the same for smart_short; that needed a variable rate, which we don't presently have.

smart long: https://github.com/delvtech/elf-simulations/pull/994/files
Not bringing back smart_short because it wasn't very smart.


Load/use feature flags with IConfiguration

Summary of pain point
I wish for unleash to hook into IConfiguration to enable offline feature management & alternative syncing of rules. This is avoid having to implement a local api for software to call which is a no-op. Note I already have a secured solution to sync the feature flags to local servers and don't want local solution to talk directly to unleash.
The solution I'd like
unleash client for dotnet includes the following:

A dotnet configuration provider for fetching feature flags from unleash including configurable refresh interval
unleash uses feature flag properties as defined in IConfiguration.

Describe alternatives you've considered
Rolling out my own api which behaves like unleash

Answered in #183

This is not going to get addressed in the SDK. As discussed in #183, the SDK already supports Bootstrapping and backup file loading on startup