Bug while playing music files in background in iOS (RELEASE Version)

There is a weird bug that occurs when I try to play an audio file(.mp3) from assets, in iOS. My task is to play an audio file (either from local storage or assets), after some delay, while the app is not in foreground, that is while the screen is locked or app in in recent tray. The problem is, when I try to play the audio file after any timer function when iPhone is locked (ex. after timer.delay, Future.delay, etc) the app doesn't plays it. The app plays it after unlocking the iPhone. Also a point is to be noted that EVERYTHING WORKS as intended in DEBUG version of the app, and NOT IN RELEASE version. One more thing, I have added background audio permission in info.plist, which is necessary for playing audio in background. In Android, everything is working fine.
My dummy code for app is as follows:
class AudioTesting extends StatefulWidget {
  @override
  _AudioTestingState createState() => _AudioTestingState();
}

class _AudioTestingState extends State<AudioTesting> {

  @override
  void initState() {
  AudioPlayer _audioPlayer = AudioPlayer();
  AudioCache _audioCache = AudioCache();
    super.initState();
  }

  @override
  void dispose() {
    super.dispose();
  }

  @override
  Widget build(BuildContext context) {
    return AudioServiceWidget(
      child: Scaffold(
        appBar: AppBar(
          title: Text('Testing Screen'),
        ),
        body: Center(
          child: Column(
            mainAxisAlignment: MainAxisAlignment.center,
            children: <Widget>[
              RaisedButton(onPressed: (){
                  Future.delayed(Duration(seconds: 5),(){
                    _audioCache.load('$test.mp3');
                    _audioPlayer = await _audioCache.play('$test.mp3');
                    //This doesnt works in release mode, but works in debug mode, when played in Locked state
                  });
              },
                child: Text('Future.delay func'),
              ),
              RaisedButton(onPressed: (){
                sleep(Duration(seconds: 5));
                _audioCache.load('$test.mp3');
                _audioPlayer = await _audioCache.play('$test.mp3');
                // This works completely fine in both version
              },
                child: Text('sleep func'),
              ),
            ],
          ),
        ),
      ),
    );
  }
}

You can reproduce this bug as follows:

Copy paste the above code snippet.
build a release version and deploy it on any physical iOS device.
Click on the buttons and lock your iPhone.

In my case, I used iPhone 8 with iOS 14.4
Are there any possible alternative solutions that would let me play music in background, when triggered by some functions in iOS?

Hi! Have the same issue! Have you found solution?

@rekonvald No solution till now. Not even any alternative. I guess we will have to either move to native side or wait for Author to check it out.

Sadly, other than the suggestions I already added to the FAQ, I don't know why this happens... If anyone has any idea please let us know.

What suggestion did you mention?

FYI we have discussed this here:
https://github.com/ryanheise/audio_service/issues/458
In short, iOS has some restrictions on what you're allowed to do in the background in order to conserve battery, but you can work around this restriction by playing a silent audio file of the desired duration instead of the timer.

@ryanheise But it does plays in debug mode, while using a timer. It just doesn't plays in Release mode. Why is this possible?

Many things are different in debug and release mode but as to why in this case, only Apple knows.

Have the same issue! Have you found solution?

To correctly play things in the backgroung, the audio_service plugins needs to be used alongside audioplayers.
cc @luanpotter we should add this to the our FAQ as well.

I'm facing a similar problem today. The player failed to play the next track in background if the playback is initiated on dart side(onPlayerComplete). But it does works if it's triggered on system side using controllers provided by iOS itself, such
as notification.
I'm not familiar with iOS developing. But after searching for hours, I think the cause should be: play function called by resume is not allowed to start playback in background when no audio is being played by the same application.
https://github.com/bluefireteam/audioplayers/blob/5dcc38201a32e0bd1000962dda5c8829c0d8452a/packages/audioplayers_darwin/darwin/Classes/WrappedMediaPlayer.swift#L86-L90
I'm using the following workaround for now:
class PlaybackServiceHackForIOS {
  static PlaybackServiceHackForIOS? _instance;

  final AudioPlayer player = AudioPlayer(playerId: 'hack-for-ios');
  PlaybackServiceHackForIOS._() {
    init();
  }

  factory PlaybackServiceHackForIOS() {
    _instance ??= PlaybackServiceHackForIOS._();
    return _instance!;
  }

  Future<void> init() async {
    await player.setReleaseMode(ReleaseMode.loop);
    await player.play(AssetSource('silent.wav'));
  }
}

silent.wav was generated by the following command:
ffmpeg -f lavfi -i anullsrc=r=11025:cl=mono -t 1 silent.wav

and should be put in assets folder.
But the workaround is really hack, and it break randomly. Hope this issue to be resolved soon.


improve abstractions for the mint rounding solution

Closes: #XXX
What is the purpose of the change

create common methods for handling truncations

used by mint distributions
used by developer rewards distributions

make distribution functions deeper
remove obsolete methods

Merged into private branch


Bugfix/customproxy

Description (required)
Added a CustomProxy class, the instance of which overrides the invocationHandler() method which returns appropriate values for different datatypes instead of null for all of them. See https://stackoverflow.com/questions/52083338/expected-to-unbox-a-string-primitive-type-but-was-returned-null
Fixes #2737 App crashed with NPE in UploadPresenter due to null object returned by Proxy InvocationHandler
Tests performed (required)
Tested {ProdDebug} on {Pixel} with API level {27}.

Codecov Report

Merging #2738 into master will increase coverage by 0.02%.
The diff coverage is 31.57%.

@@            Coverage Diff            @@
##           master   #2738      +/-   ##
=========================================
+ Coverage    2.72%   2.75%   +0.02%
=========================================
  Files         267     268       +1
  Lines       12801   12816      +15
  Branches     1137    1141       +4
=========================================
+ Hits          349     353       +4
- Misses      12426   12437      +11
  Partials       26      26

Impacted Files
Coverage Δ

...va/fr/free/nrw/commons/upload/UploadPresenter.java
12.63% <100%> (+1.58%)
:arrow_up:

...in/java/fr/free/nrw/commons/utils/CustomProxy.java
7.14% <7.14%> (ø)

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 cbab7dd...f40f692. Read the comment docs.


AdMob undefined

I am using Cordova 6.3.1 and Android SDK 24.4.1. I have tried my best to follow the guidelines given by the plugin, but it is not showing banner at all.
So, I decided to use a DOM element as console. and I used try and catch, where in try I added AdMob code. Now I am getting error: AdMob is undefined, while compiling the app it shows no error, but when APK is installed, it shows the error.
I have also added location of AdMob.js file to plugin folder in script tag in my index.html file.
I have also checked that all other Javascript code is running perfectly, even ajax. I have double checked about Banner ID etc...
Help me on these, where I have made mistake???
Here is my code,
`var admobid = {};
logmessage('initializing the advertisement...');
// TODO: replace the following ad units with your own
if( /(android)/i.test(navigator.userAgent) ) {
admobid = { // for Android
banner: 'ca-app-pub-xxxxxxxxxxxxxxxx/zzzzzzzzzzzz',
interstitial: 'ca-app-pub-xxxxxxxxxxxxxxxx/zzzzzzzzzzzz'
};
}
function initApp() {
if (! AdMob ) { logmessage( 'admob plugin not ready' ); return; }
logmessage('creating banner...');
// this will create a banner on startup
AdMob.createBanner( {
adId: admobid.banner,
position: AdMob.AD_POSITION.BOTTOM_CENTER,
isTesting: true, // TODO: remove this line when release
overlap: false,
offsetTopBar: false,
bgColor: 'black',
autoShow: true
} );
logmessage('preparing interstitial...');
// this will load a full screen ad on startup
AdMob.prepareInterstitial({
adId: admobid.interstitial,
isTesting: true, // TODO: remove this line when release
autoShow: true
});
}
//console.clear();
try {
initApp();
}
catch(e) {
logmessage(e.name + ': ' + e.message);
}
function logmessage(message){
document.getElementById('logmessage').innerHTML = message;
}`

I think that the option adId is not supported.
You should either use publisherId (for banners) or interstitialAdId

Thanks that was the thing, solved and running


Try to open Pillar file clicking name in file Reference not working

Trying to open a Pillar file does not work on version 0.8.1 Windows 10, as shown in the following gif:

Reviewed in version 0.8.95. Fixed.


Dead empty space at the end of a note

Issue
When I scroll all the way down in Hover window, it allows me to scroll way pass the bottom of the note, creating dead empty space.
I would like it to stop right where it should be.
Hover Mode

Normal Preview

I'm not able to reproduce this.  More specifically, I can't reproduce the normal preview not having a half-screen of blank space at the end of every note.  As far as I'm aware, Obsidian always has extra space at the bottom of notes, in editing or preview mode - I have never known it not to, on any version that I can recall.
Can you upload the exact note text with the issue?  Specifically, a note that does NOT have extra space shown in the preview, when used with the stock theme in the Obsidian sandbox with no plugins or snippets active?

So this is the default theme from Obsidian

The preview window

The text content from the note

I actually can reproduce what you said that Obsidian always has extra space at the bottom of notes, especially when I'm using "Hover Editor" plugin.

Preview window when Hover Editor is enable

Please upload the file that produces the problem, not a picture of the file.  Thanks.

Reading minds through body language.md

So, I'm not able to get any reading or preview view that doesn't have whitespace below that, on Obsidian 0.15.9 or 1.1.10, with or without plugins, stock theme, etc.  Even without Hover Editor.

It's weird. Anyway, thank you for checking that out.


[Question] Bot c#: How can I get user location and time zone on the device from the bot

I need to get user location from bot without asking the user to writes address and also how can I get time zone of the users device to get the local time. My idea is when user says hi or hello bot to respond with good morning or good evening according to the time of the day. And for location, I want to use GPS from users phone to get me the location so I can suggest nearby places to user.

@msBot1111 what channel are you using with your Bot? Eg. Slack, Telegram, Facebook Messenger, Kik?
Depending on the channel, the capabilities and corresponding instructions will be different.  Most likely you will find the answer in the API docs for your specific channel.

@nwhitmont I am using Skype and I've integrated my bot in xamarin app

@nwhitmont Is it possible in any platform to get user location and time zone?

Is there a way I can get gps location from users device in bot? My bot is integrated in skype channel and in the xamarin app. When I try to get the location from skype it returns null.  I saw this link for messenger, but in skype it doesn't work. https://github.com/Microsoft/BotBuilder/blob/master/CSharp/Samples/EchoBot/EchoLocationDialog.cs#L110

@msBot1111 It's all platform specific and hence its hard. you can ask the user to give his location and resolve it using google places.

@msBot1111 can you try looking at these and see if they help for location. The data is not stored by the framework itself though.  I am going to close this because it is not exactly a bot framework issue, but a question on geolocation and time zones per device.  Feel free to continue the discussion here if you feel so inclined, we can always reopen.
https://stackoverflow.com/questions/38054783/get-user-location-for-bot-framework
https://stackoverflow.com/questions/42904014/reverse-geocoding-with-botframework-location/42913084#42913084

@JasonSowers  Any plan to integrate this into the framework itself in near future?

So are we able to get the latitude and longitude of the user in Skype Channel by asking whenever we need it?


.Now will be deprecated with Hugo v0.20

.Now will be deprecated replaced with the now template function in Hugo v0.20 which eventually will be released on Monday. It's likely that the deprecation will cause builds to fail.

Thanks so much for letting me know about this.
I can see how I should update it - but I can't see how I could make some kind of branch in the template for older versions which don't support now yet. I see the Ubuntu 16.04 package is still at 0.16.
I searched the docs but couldn't find any sort of template function that returns the version of Hugo the user is at?

I can see how I should update it - but I can't see how I could make some kind of branch in the template for older versions which don't support now yet. I see the Ubuntu 16.04 package is still at 0.16.

Such packages aren't maintained by us officially. So we (the maintainers of Hugo) don't have direct influence on the packages.

I searched the docs but couldn't find any sort of template function that returns the version of Hugo the user is at?

In other words you want to keep your theme backwards compatible. I haven't thought about that :+1:
Perhaps you can use the :Hugo.Version variable to check whether the current version is smaller than v0.20.

Sorry for not closing this issue - I did update to now a while ago.


add option to override parse function

What's your use case?
This package doesn't requiring injection of spdx-expression-parse because that's the only implementation that it was intended to work with.
Allowing users to override spdx-expression-parse doesn't save them having to install it as an indirect dependency.

Yep. It's to be able to configure spdx-expression-parse for things like the relaxed option or licenseVisitor that are in flight

If @motet-a is still available and interested in spdx-expression-parse, I'd like his input on the changes there. If we end up shipping -parse with an options argument, we should definitely consider publishing a new -satisfies that takes either a parser argument, or an options object to pass to -parse.

@dabutvin If you have pressing needs that require packages in the public registry, feel free to publish forks as new packages. We can always re-merge and deprecate.

@kemitchell What's your thinking here? We are working out of a fork (OK) so this is not super critical but if would be great to just wrap up. This is a pretty modest change that would be still be relevant even if parse ends up having options.

@jeffmcaffer I'd support a new package that implements a parser for the SPDX grammar with invalid identifiers, and a patch to spdx-expression-parse that runs that new parser, then validates that all identifiers are valid SPDX identifiers. Would be happy to host the new, generic parser here in jslicense.

@kemitchell my thinking here was that even with a more parser choices it would be good to allow satisfies to be configured with the parser to use (this PR) rather than an option to determine which parser to create. That is more future-proof and enables people to innovate on parsing as they see fit.

@jeffmcaffer not ignoring you. Just very, very busy.

@jeffmcaffer, by chance, is the root cause of your problem here a bottleneck getting new licenses onto the SPDX list?

In a sense. IIRC, we need to handle expressions that have NOASSERTION as a valid element. We are not using LicenseRef to date but that would be a thing. In the case of this module it is really separating the concerns of parsing the expression and checking satisfaction. The other proposed changes to spdx-parse are certainly related

Another aspect to this is enabling flexibility within the parser, such as being case insensitive.
I think it's valuable to be able to have this work in certain situations
satisfies('mit', 'MIT OR Apache-2.0')

@dabutvin
satisfies(correct('mit'), '(MIT OR Apache-2.0)')

@jeffmcaffer I don't recall offhand. Is NOASSERTION a valid identifier in SPDX license expressions?

@kemitchell spdx-correct is pretty aggressive and resulting in false corrections. In any event, the scenario remains, it would be great to have parsing separate from satisfaction (or any other) computations. An alternate design approach is to allow calling either with a string (in which case the default parse is used) or an already parsed structure.
So, for example, satisfies(parse('mit'), 'MIT') where parse is a whatever parsing tech the caller wants. That would be fine as well for our scenarios.

@jeffmcaffer: How would you feel about a major version that accepts an array of acceptable identifiers as rule and an AST in spdx-expression-parse shape?

I did not answer @kemitchell's question about NOASSERTION. You are likely right but, most/all of the computations (e.g., satisfies) should not care, just parsing. the computations will just do string equality etc. Having NOASSERTION be in an expression is very important otherwise data is lost. For example, in trying to merge together the licenses from 3 files with license tool output of MIT, ISC, NOASSERTION, ideally we would get MIT AND ISC AND NOASSERTION. That helps readers know that there is something funky going on and they need to look deeper.
Of course, it would be great if we could always have a id for every license but given the 000s of licenses out there and the 00s of SPDX ids, that's not practical. Even with the ongoing thinking about dynamic license identification and registration we will likely still have NOASSERTION cases.
So, I would lobby to have NOASSERTION be a valid part of SPDX expressions.

@jeffmcaffer: How would you feel about a major version that accepts an array of acceptable identifiers as rule and an AST in spdx-expression-parse shape?

That could be OK with some tweaks

the array of ids should not run through a parse. If they are then we have the same issue as this PR is trying to avoid.
The ids would have to all be normalized, canonicalized, ... satisfies could do this but in the current code at least, that requires the ids to actually be expression nodes.
Given the potential complexity of SPDX expressions, we would need a helper function that took an expression object and returned the list of all licenses mentioned. Effectively flatten in the current code. Essentially we'd just be making callers do  flatten(normalizeGPLIdentifiers(parse(second))) (see https://github.com/jslicense/spdx-satisfies.js/blob/master/index.js#L134).

Perhaps we can roll back to your basic issues with the proposed change. Might be that we are just not aligned on a fundamental API design philosophy or something. More than happy to try and fit into the approach but am fuzzy on how.

Might be that we are just not aligned on a fundamental API design philosophy or something.

Don't go there yet!
I'm mostly just really, really, really busy. But I have some time for this tonight.


url attributes not generated in plantuml export

running 'export' using CLI with  "-format plantuml"  with the test.dsl workspace from the examples folder does not generate any url/links/annotations in the generated puml files.
The url tags are being parsed - as invalid urls do generate errors .
e.g.
java -jar structurizr-cli-1.6.0.jar export -workspace test.dsl --output out
I was expecting something in either the skinparam the participant definition:
Generated structurizr-Dynamic-sequence.puml snippet:
skinparam sequenceParticipant<<1>> {
  BackgroundColor #ffffff
  FontColor #000000
  BorderColor #777777
  roundCorner 20
}
...
participant "User\n<size:10>[Person]</size>" as 1 <<1>> #ffffff

It's the StructurizrPlantUMLWriter that needs to be modified if anybody wants to add this feature.

It's the StructurizrPlantUMLWriter that needs to be modified if anybody wants to add this feature.

have had a brief look into this. Would need to use the url (when present) to generate "[[url]]" at the end of description.
PlantUML does support this hyperlink having an alias e.g. [[url VIEW]], which would help with the diagram sizing,
Presume this would require a wider change to the DSL url tag which only allows a valid URL as parameter 1 to allow a url alias.
e.g.
DSL
user = person "User" "Description" "Tag" {
            url "https://structurizr.com/otherdiagram.svg" "navigate"

generated PlantUML :
participant "User\n<size:10>[Person]</size>\n\n[[https://structurizr.com/otherdiagram.svg navigate]]" as 1 <<1>> #ffffff
Workaround is to manually add \n[[url alias]] into the description of the elements themselves in the DSL and not use the url property.
e.g.
user = person "User" "Description\n[[https://structurizr.com/otherdiagram.svg navigate]]" "Tag" {
generates
participant "User\n<size:10>[Person]</size>Description\n\n[[https://structurizr.com/otherdiagram.svg navigate]]" as 1 <<1>> #ffffff
so maybe this change is not required?

have had a brief look into this. Would need to use the url (when present) to generate "[[url]]" at the end of description.
PlantUML does support this hyperlink having an alias e.g. [[url VIEW]], which would help with the diagram sizing,
Presume this would require a wider change to the DSL url tag which only allows a valid URL as parameter 1 to allow a url alias.
e.g.
DSL
user = person "User" "Description" "Tag" {
            url "https://structurizr.com/otherdiagram.svg" "navigate"

generated PlantUML :
participant "User\n<size:10>[Person]</size>\n\n[[https://structurizr.com/otherdiagram.svg navigate]]" as 1 <<1>> #ffffff
Workaround is to manually add \n[[url alias]] into the description of the elements themselves in the DSL and not use the url property.
e.g.
user = person "User" "Description\n[[https://structurizr.com/otherdiagram.svg navigate]]" "Tag" {
generates
participant "User\n<size:10>[Person]</size>Description\n\n[[https://structurizr.com/otherdiagram.svg navigate]]" as 1 <<1>> #ffffff
so maybe this change is not required?

Adding a URL alias would also require changes to the core data structure (and every other library/tool will need to be changed to add support), so I'm not keen on that.
The PlantUML writer could output a (more) type link if a URL is present on the element, but styling the hyperlink colour doesn't seem to work as expected ... here's an example.

Adding a URL alias would also require changes to the core data structure (and every other library/tool will need to be changed to add support), so I'm not keen on that.
The PlantUML writer could output a (more) type link if a URL is present on the element, but styling the hyperlink colour doesn't seem to work as expected ... here's an example.

This works better: (the hyperlinkIUrl needs to be in the top level skinparam, and not per rectangle..
@startuml
title Software System - System Context
caption An example of a System Context diagram.
skinparam {
hyperlinkColor #ffffff
shadowing false
arrowFontSize 10
defaultTextAlignment center
wrapWidth 200
maxMessageSize 100
}
hide stereotype
top to bottom direction
skinparam rectangle<<1>> {
BackgroundColor #1168bd
FontColor #ffffff
BorderColor #0B4884
' HyperlinkColor #ffffff
}
rectangle "==Software System\nsize:10[Software System]\n\nMy software system.\n\nsize:10[[https://structurizr.com (more)]]" <<1>> as 1
@enduml

This works better: (the hyperlinkIUrl needs to be in the top level skinparam, and not per rectangle..
@startuml
title Software System - System Context
caption An example of a System Context diagram.
skinparam {
hyperlinkColor #ffffff
shadowing false
arrowFontSize 10
defaultTextAlignment center
wrapWidth 200
maxMessageSize 100
}
hide stereotype
top to bottom direction
skinparam rectangle<<1>> {
BackgroundColor #1168bd
FontColor #ffffff
BorderColor #0B4884
' HyperlinkColor #ffffff
}
rectangle "==Software System\nsize:10[Software System]\n\nMy software system.\n\nsize:10[[https://structurizr.com (more)]]" <<1>> as 1
@enduml

I have also used this technique to make the main description of the item clickable (without using the url tag)
softwaresystem "[[structurizr-myContext.svg MySystem]]" "My Service for providing info" {
...
}
but i have to inject the skinparam hyperlink color into the .puml before generating the .svg

I have also used this technique to make the main description of the item clickable (without using the url tag)
softwaresystem "[[structurizr-myContext.svg MySystem]]" "My Service for providing info" {
...
}
but i have to inject the skinparam hyperlink color into the .puml before generating the .svg

The ideal solution would be to replicate the Structurizr web renderer (example) where the entire element is double-clickable, but that doesn't seem straightforward with PlantUML.

The ideal solution would be to replicate the Structurizr web renderer (example) where the entire element is double-clickable, but that doesn't seem straightforward with PlantUML.

PlantUML lets a whole element on an activity diagram be clickable, but I don't think 'rectangle' supports this as far as I can tell. Otherwise it is simply links in text rather than links on elements.
e.g.
@startuml
start
[[http://plantuml.com]]:Some activity;
:Some [[http://plantuml.com link]];
end
@enduml

PlantUML lets a whole element on an activity diagram be clickable, but I don't think 'rectangle' supports this as far as I can tell. Otherwise it is simply links in text rather than links on elements.
e.g.
@startuml
start
[[http://plantuml.com]]:Some activity;
:Some [[http://plantuml.com link]];
end
@enduml

Since it doesn't seem possible with PlantUML, shall we close this issue?

Since it doesn't seem possible with PlantUML, shall we close this issue?

If we can't auto generate these links perhaps we can allow an option somewhere to allow config to feed into the skinparam hyperLinkColor , so we can at least control the colours in generated .puml files. Rather than them always defaulting to a blue hyperlink that planuml has.
If you would rather not do this, happy to close the issue.

If we can't auto generate these links perhaps we can allow an option somewhere to allow config to feed into the skinparam hyperLinkColor , so we can at least control the colours in generated .puml files. Rather than them always defaulting to a blue hyperlink that planuml has.
If you would rather not do this, happy to close the issue.

Perhaps you could raise an issue with the PlantUML team, and ask for them to support hyperlinkColor on overridden skin params?

Perhaps you could raise an issue with the PlantUML team, and ask for them to support hyperlinkColor on overridden skin params?

ah, yes - good idea.
and/or another thought would be to allow a structurizer plantuml include option in strcturizr DSL so all the plantuml settings do not also have to be structurizr DSL settings... still does not solve the hyperlink colour issue, but could reduce the number of plantuml settings you have for no value other than passing to plantuml

ah, yes - good idea.
and/or another thought would be to allow a structurizer plantuml include option in strcturizr DSL so all the plantuml settings do not also have to be structurizr DSL settings... still does not solve the hyperlink colour issue, but could reduce the number of plantuml settings you have for no value other than passing to plantuml

If the PlantUML team adds support for hyperlinkColor, I'll certainly be happy to add that for you. But I'd rather keep the PlantUML export as simple as possible, so if you need more control, you may be better off forking one of the PlantUML writers in the java-extensions project (there are several), and creating your own custom PlantUML renderer (the CLI export command is just a thin wrapper around this anyway).

If the PlantUML team adds support for hyperlinkColor, I'll certainly be happy to add that for you. But I'd rather keep the PlantUML export as simple as possible, so if you need more control, you may be better off forking one of the PlantUML writers in the java-extensions project (there are several), and creating your own custom PlantUML renderer (the CLI export command is just a thin wrapper around this anyway).

keeping plantuml export simple as possible makes sense.
Issue raised PlantUML #436

keeping plantuml export simple as possible makes sense.
Issue raised PlantUML #436


Object parameter for options

Thanks for publishing this! 😄 Hoping to give it a try soon in 11ty; my current pagination is linear.
I noticed that the current API makes it a little difficult to read examples like this since all of the arguments share the same type:
generate(4, 9, 1, 1)

There may also be situations where a user wants to customize just one option but not touch any of the others. e.g., if I want to specify a custom string for glue, I'll need to pass along all of the intermediate arguments as well.
What are your thoughts on changing the API to accept an object instead? Maybe something like this:
const generate = (options) => {
  const {
    curPage,
    numPages,
    numPagesAtEdges= 2,
    numPagesAroundCurrent = 2,
    glue = '…'
  } = options;
}

(This also proposes standardizing number => num, or vice versa, since the arguments would be named.)

Hi @AleksandrHovhannisyan, thank you for this suggestion. It's also something I've been thinking about.
As version 1.x already got tagged, altering the function signature would be a breaking change and require a 2.x release. To not introduce such a breaking change, I thought about extending the package to export an additional function that accepts a configuration object and wraps around the existing generate. However, that would extend the API surface this module exposes, bringing me back to making it a 2.x release. As you can tell: I'm conflicted between both options there.
Either way, good call to standardize the variable names.

This has been released in version 1.2.0


bugfix: sample: disconnect when all the messages are sent

Description of the problem
Current implementation of the sync version of provision_symmetric_key.py calls disconnect API repeatedly, it seems weird.
Description of the solution
I think it would be correct just as the async version samples/async-hub-scenarios/provision_symmetric_key.py does.

I'm not sure why the CI fails...
@olivakar Could you take a look at it ?

I'm not sure why the CI fails... Mentioning @olivakar , who is checked in the latest update. Could you take a look at it ?

Apologies for the late reply - there was an infrastructure issue that broke CI. It has now been addressed, we will take in this fix.

/azp run Python E2E


Add SWIFT_ENABLE_TENSORFLOW markers to docs/ABI/Mangling.rst.

Follow-up to https://github.com/apple/swift/pull/25804 and https://github.com/apple/swift/pull/26595.

@swift-ci Please test tensorflow


No Stop Method?

I'm trying to make it so that if the user scrolls while smoothScroll.animateScroll is running, the smoothScroll is cancelled, otherwise the page seems to jank if you try to scroll as the method is being called.
Is there any way to cancel the smoothScroll effect? Something like smoothScroll.stop() would be nice.

Duplicate: https://github.com/cferdinandi/smooth-scroll/issues/309


Extension does not start

Before submitting your bug report

[X] I believe this is a bug. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that reports the same bug
[X] I've seen the troubleshooting guide on the Continue Docs

Relevant environment info
- OS: Manjaro 6.4.3
- Continue: 0.0.409
- Python: 3.11.5 (and 3.10.12)
- IDE: Code OSS 1.82.0

Description
Extension prompts the "Continue Server Starting" loading screen and this loading screen keeps on forever.

It seems like the server is starting as expected (tried running it manually and the server starts without issues). It worked for me on a previous version (I think in August but I am not really sure, I reinstalled my OS in between and I am not sure if I used Continue after) but I tried and older version earlier and I have the same issue.
I noticed that the server keeps running after VSCode is closed until I reopen VSCode. Then the process is killed and a new server is started.
After deleting the .continue folder I also noticed that the config.py and other files in this folder are missing.
To reproduce

Install Continue extension
Swap to Continue extension in the side bar

Log output
Server Logs (Restarted VSCode multiple times):
[2023-10-03 10:41:47,318] [DEBUG] ------ Begin Logs ------
[2023-10-03 10:41:47,643] [DEBUG] Downloading MeiliSearch...
[2023-10-03 10:42:02,762] [DEBUG] Starting MeiliSearch...
[2023-10-03 10:42:02,762] [DEBUG] Meilisearch started
[2023-10-03 10:50:41,299] [DEBUG] ------ Begin Logs ------
[2023-10-03 10:50:41,616] [DEBUG] Meilisearch started
[2023-10-03 10:56:47,354] [DEBUG] ------ Begin Logs ------
[2023-10-03 10:56:47,687] [DEBUG] Meilisearch started

VS Code console logs:
[Extension Host] Using Continue version:  0.0.409
workbench.desktop.main.js:sourcemap:132 [Extension Host] In workspace:  /run/media/lukas/linux_data/Projects/Homepage
workbench.desktop.main.js:sourcemap:132 [Extension Host] Killing server from old version of Continue
workbench.desktop.main.js:sourcemap:132 [Extension Host] Downloaded server executable at  /home/lukas/.continue/server/exe/run
workbench.desktop.main.js:sourcemap:132 [Extension Host] Looking for file at  /home/lukas/.continue/server/exe/run
workbench.desktop.main.js:sourcemap:132 [Extension Host] ---- Starting Continue server ----
workbench.desktop.main.js:sourcemap:132 [Extension Host] Error caught: Error: spawn ETXTBSY.

Retrying attempt 1...
workbench.desktop.main.js:sourcemap:132 [Extension Host] Continue server started

Would you mind trying a few things to help characterize the situation? It's not immediately clear what the problem is but I have a few ideas

When the "Continue Server Starting" wheel is spinning, does lsof -i :65432 show that the server is running?
At any point in the lifecycle of opening/reloading VS Code does ~/.continue/server/server_version.txt exist?
When you reload the window, do you each time see the bottom right notification of "Installing Continue server..."?

And then I'm not super familiar with Manjaro, would expect that a binary built on Ubuntu would work there, but is there any reason it might not, or anything potentially relevant about the OS?

When the "Continue Server Starting" wheel is spinning, does lsof -i :65432 show that the server is running?

Yes it does:
COMMAND   PID  USER   FD   TYPE DEVICE SIZE/OFF NODE NAME
run     12305 lukas    8u  IPv4 142919      0t0  TCP localhost:65432 (LISTEN)

At any point in the lifecycle of opening/reloading VS Code does ~/.continue/server/server_version.txt exist?

Yes it does exist

When you reload the window, do you each time see the bottom right notification of "Installing Continue server..."?

Yes it pops up each time

And then I'm not super familiar with Manjaro, would expect that a binary built on Ubuntu would work there, but is there any reason it might not, or anything potentially relevant about the OS?

I am myself not sure if this would work or not. As Manjaro is based on Arch instead of Debian maybe this causes problems. However it did work before without any problem
Complete system information (in case that helps):
Operating System: Manjaro Linux
KDE Plasma Version: 5.27.8
KDE Frameworks Version: 5.110.0
Qt Version: 5.15.10
Kernel Version: 6.5.3-1-MANJARO (64-bit)
Graphics Platform: X11
Processors: 20 × 13th Gen Intel® Core™ i5-13500
Memory: 31.1 GiB of RAM
Graphics Processor: AMD Radeon RX 6750 XT
Manufacturer: Micro-Star International Co., Ltd.
Product Name: MS-7D98
System Version: 1.0

Thanks for checking all of this. I know we've seen a problem on Arch linux before, so this could be related - the binary has a dependency on glibc, so perhaps differences could cause a problem (though I lean less toward this because the server is listening).
What would probably expose the problem is entering the ~/.continue/server/exe directory and double-clicking (or running from terminal) the run binary. My guess is that there will be some useful output, perhaps that didn't show up in the logs

Out of the box I didn't had the permission to execute the binary. I changed that (chmod 755 ./run) and tried executing that and I get the following error.
$ ./run
[31961] Cannot open PyInstaller archive from executable (/home/lukas/.continue/server/exe/run) or external archive (/home/lukas/.continue/server/exe/run.pkg)

The binary exits after the error.

Ok, this seems like most likely a build incompatibility (unless somehow the binary is corrupted). What I can try is building a binary on a Manjaro EC2 myself and then sharing to see if it works - though I don't think that GitHub runners support Manjaro out of the box, so staying up to date might be easier by manually running the server

Are the steps for building the binary those, described in the documentation for manually running continue?
I could try building the binary on my machine to see if it works/changes anything.

Yeah, you can follow the steps here: https://continue.dev/docs/walkthroughs/manually-run-continue#build-the-server-binary-from-source
I believe this will do everything necessary to create the build - this would be amazing

I now had time to build the binary myself. It starts up just fine but the UI still shows "Continue Server Starting" (I enabled "Manually Running Server" in the settings and reloaded VSCode).
I even deleted the .continue folder. Here is the complete console output I got when I started the binary.
[2023-10-05 12:52:30,871] [DEBUG] ------ Begin Logs ------
Certificates at:  /tmp/_MEIpaxSKJ/ca_bundle/cacert.pem
Running Continue server version 0.0.350
INFO:     Started server process [6879]
INFO:     Waiting for application startup.
[2023-10-05 12:52:31,217] [DEBUG] Downloading MeiliSearch...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100  4352  100  4352    0     0  13139      0 --:--:-- --:--:-- --:--:-- 13139
Downloading Meilisearch binary v1.4.0 for linux, architecture amd64...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:--  0:00:05 --:--:--     0
100  109M  100  109M    0     0  5205k      0  0:00:21  0:00:21 --:--:-- 7824k
Meilisearch v1.4.0 binary successfully downloaded as 'meilisearch' file.

Run it:
    $ ./meilisearch
Usage:
    $ ./meilisearch --help
INFO:     Application startup complete.
INFO:     Uvicorn running on http://<IP_ADDRESS>:65432 (Press CTRL+C to quit)
[2023-10-05 12:53:57,318] [DEBUG] Starting MeiliSearch...
[2023-10-05 12:53:57,319] [DEBUG] Meilisearch started

The only difference I noticed is that the dropdown in the left bottom corner of the UI now has a default value but I'm not sure if this is related.

I just clicked on the "history" button in the UI just to see what happens and it seems like the Extension sent an request to the server successfully even though it still says "Continue Server Starting"
Returning sessions list:  []
INFO:     <IP_ADDRESS>:47648 - "GET /sessions/list HTTP/1.1" 200 OK

Thanks for sharing this! At this point it seems the problem has been narrowed down to an inability of the extension (outside of the webview) to connect to the server.
I've been working on a pre-release version that is now published, which ships with the binary built-in - this might be worth a try.
Otherwise, I'm going to add a few logging points throughout the server startup lifecycle that might help narrow down the problem since it's still not clear exactly how to solve this.

Today I had time to try out the pre-release. I still have the same Problem
The output in continue.log did not change but I now get the following output in the VSCode developer tools:
[Extension Host] Using Continue version:  0.1.8
[Extension Host] In workspace:  /run/media/lukas/linux_data/Projects/Homepage
[Extension Host] Killing server from old version of Continue
[Extension Host] ---- Starting Continue server ----
[Extension Host] Continue server started

I also noticed, that the bottom right notification of "Installing Continue server..." disappeared.

Out of curiosity I tried installing the official Visual Studio Code instead of Code - OSS and installed the regular Version of Continue and that seems to work.
So the problem seems to be either with my installation/setup of Code - OSS or with Code - OSS in general.

@lufixSch "Installing Continue server..." disappearing is ok, since in the pre-release, and moving forward, we will be shipping with OS-specific binaries instead of downloading after-the-fact.
The Code - OSS detail is really good to know! This gives me something to test against - I'll try this myself and see if I can get to the bottom of it. Thanks again for being willing to try so much! Hopefully by the end of this there's a whole new group of users who have compatibility : )

@lufixSch Unfortunately I just tried with Code - OSS and setup went smoothly. So seems to be at the intersection of Code - OSS and Manjaro Linux. I've just published a version that should be ready in ~20min (v0.1.10, a pre-release) that adds logging to the websockets setup:
newWebsocket.addEventListener("open", () => console.log("Websocket connection opened"));
    newWebsocket.addEventListener("error", (error: any) => {
      console.error("Websocket error occurred: ", error);
    });
    newWebsocket.addEventListener("close", () => {
      console.log("Websocket connection closed");
    });

This seems like the best starting point I can think of to further figure out what's going on. These will show up in developer tools console

Tried the pre-release and got the following output:
ERR [Extension Host] Websocket error occurred:  {}
[Extension Host] Websocket error occurred:  {}
    m @ console.ts:137
    $logExtensionHostMessage @ mainThreadConsole.ts:39
    N @ rpcProtocol.ts:455
    M @ rpcProtocol.ts:440
    H @ rpcProtocol.ts:370
    G @ rpcProtocol.ts:296
    (anonymous) @ rpcProtocol.ts:161
    w @ event.ts:1107
    fire @ event.ts:1138
    fire @ ipc.net.ts:650
    X.onmessage @ localProcessExtensionHost.ts:585
[Extension Host] Websocket connection closed

Incredibly unfortunate that the error message is empty...apparently the error is only included in the onclose event, so I'm uploading a new version that will print this. At least we know that it happened after the websocket was opened - I would be surprised if there weren't any error message on the server side to go along with this

I'm uploading a new version that will print this.

I am not sure, if you already uploaded this version. I'm now running v0.1.14 but the error did not change.

@lufixSch the "Websocket error occurred" would still be empty, but the object would show up with "Websocket connection closed" like here https://github.com/continuedev/continue/blob/c94053baedfad4adb5a5739e842f089c4c514552/extension/src/util/messenger.ts#L64

My bad. Did not see that.
But the output still doesn't look really useful to me. It's still an empty object

@lufixSch I promise I haven't forgotten about this, it's just been unable to take priority as we've been fixing a large handful of critical bugs in the new JetBrains extension.
Someone else today came in with what appears to be a similar error on CentOS, so I'll keep you updated if any new information shows up there.
Thanks again for helping so much with debugging this, and hopefully we can find our way to a solution!

No problem. Until this is fixed I will simply use Visual Studio Code instead of Code - OSS which is fine for me.

Hey @sestinj I just used Code - OSS for something and tried Continue out of curiosity and it seems to work now.
I'm running v0.1.33 but I have no Idea since when it is working again.

@lufixSch Very odd, but not complaining.
I'm working on some changes this weekend that I expect to solve the original problem, so even in case it comes back I hope to squash the bug for good

@lufixSch The changes I mentioned I was making are now released. If things work there as well, then this issue might be considered finally solved

@sestinj I tried the newest version and it still works. So I think the issue is fixed.
Thanks for your work :)

Nice!! Thanks for your patience


Pages Plugins

This experimental feature allows people to compile Plugins for Pages Functions. A compiled plugin is a factory for something incredibly similar to a compiled Pages Functions worker, but with just a couple of minor tweaks. I've kept the implementations separate here until we finalize what we want to keep common.
Pairs nicely with @threepointone 's recent PR to hide the wrangler pages functions command (https://github.com/cloudflare/wrangler2/pull/347).
I've also bumped esbuild to 0.14.14 from 0.14.1 (needed some of the newer esbuild plugin features).

Is this urgent? Could we please table this till I'm back? Thank you!
cc @geelen


LTK output when Bluetooth LE connected (IDFGH-7701)

Is your feature request related to a problem? Please describe.
Cannot get current link key in event params in gap_event_cb() when handling ESP_GAP_BLE_AUTH_CMPL_EVT event.
Describe the solution you'd like
Some way to get the link key of current connection
Describe alternatives you've considered
I'm afraid there is no alternative.

you can print LTK in btsnd_hcic_ble_ltk_req_reply，
code:
esp_log_buffer_hex("LTK",ltk,HCIC_BLE_ENCRYT_KEY_SIZE);

note that:
#include "esp_log.h"

Thanks for reporting, sorry for slow turnaround, feel free to reopen.


Doesn't work with Satisfactory

Using the command /home/jonah/Games/Steam/wine-discord-ipc-bridge/winediscordipcbridge-steam.sh %command% works for Deep Rock Galactic, but when playing Satisfactory, Discord detects nothing.
My system monitor shows that winediscordipcbridge.exe is running, and it does not stop when I close the game, indicating that no connection has been made.
I unfortunately can't figure out how to get the logs from the bridge.
Using Proton Experimental and wine-discord-ipc version 0.0.2.

Are you sure it supports rich presence? Rich presence is not just when discord detects the game you're playing, it's when the game actually provides custom status information to discord.
Rich presence:

Not rich presence:

Ah.  You're absolutely right, the game does not support rich presence. Sorry for wasting your time!


test: scene loading from Assets within Test Runner

This is just a test to see if this passes Yamato build

This was used as an experimental branch to test scene loading while running under Yamato.
This is no longer needed.


Can you make a new release on npm please?

I wanted to use the new PHP + Guzzle converter but I can't.

Recently released 3.0.1


Ensure expressions in list expressions are lowered

What was wrong?
We missed to lower expressions inside list expressions.
How was it fixed?
Call to expr
I didn't create a release notes entry for this because I don't think that in previous releases there was a way for an user to hit this bug. But they can hit it now with the const support.

:+1:


Improve codecov setup

🔧 Changes
Stabilizes the codecov settings to make them more reliable.
📚 References

🔬 Testing

📝 Checklist

[x] All new/changed/fixed functionality is covered by tests (or N/A)
[x] I have added documentation for all new/changed functionality (or N/A)

Codecov Report
Patch coverage has no change and project coverage change: +1.01 :tada:

Comparison is base (23869cf) 50.44% compared to head (d87a704) 51.46%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #678      +/-   ##
==========================================
+ Coverage   50.44%   51.46%   +1.01%
==========================================
  Files          93       93
  Lines       11771    11771
==========================================
+ Hits         5938     6058     +120
+ Misses       5405     5278     -127
- Partials      428      435       +7

see 4 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


built-in functions not highlighted?

I don't get any highlighting for basic functions/constructors like print or list. See screenshot here. I am running VIM version 8.1.3741.

It seems like the colorscheme I am using is causing this issue. Closing.


Websocket Error connecting to No route to host (Socket::Connect Error)

Hi, I have a problem with Websocket, I have a project about cryptocurrency, I am using Websocket to connect a cryptocurrency exchange(Binance), with code below:
require "http/web_socket"

p "connecting ws"
ws = HTTP::WebSocket.new(host: "stream.binance.com", path: "ws", port: 9443, tls: true)
p "connected"
ws.send %({"method":"SUBSCRIBE","params":["dogeusdt@trade"],"id":1})
ws.on_message do |msg|
  p msg
end

ws.on_close do |msg|
  p msg
end

ws.run

However, the error pop up:
./binance_ws
"connecting ws"
...
(wait about 2 minutes)
...
Unhandled exception: Error connecting to 'stream.binance.com:9443': No route to host (Socket::ConnectError)
  from /usr/local/Cellar/crystal/1.0.0/src/socket/addrinfo.cr:73:15 in 'initialize'
  from /usr/local/Cellar/crystal/1.0.0/src/socket/tcp_socket.cr:27:3 in 'initialize'
  from /usr/local/Cellar/crystal/1.0.0/src/socket/tcp_socket.cr:27:3 in 'new'
  from /usr/local/Cellar/crystal/1.0.0/src/http/web_socket/protocol.cr:275:5 in 'new'
  from /usr/local/Cellar/crystal/1.0.0/src/http/web_socket.cr:48:5 in 'new:host:path:port:tls'
  from binance_ws.cr:4:1 in '__crystal_main'
  from /usr/local/Cellar/crystal/1.0.0/src/crystal/main.cr:110:5 in 'main_user_code'
  from /usr/local/Cellar/crystal/1.0.0/src/crystal/main.cr:96:7 in 'main'
  from /usr/local/Cellar/crystal/1.0.0/src/crystal/main.cr:119:3 in 'main'

I can connect the websocket server using Golang but cannot connect using Crystal, So I think that a problem about crystal, not the bianace server

If its worth anything, it works fine for mewith this change:
- ws = HTTP::WebSocket.new(host: "stream.binance.com", path: "ws", port: 9443, tls: true)
+ ws = HTTP::WebSocket.new(host: "stream.binance.com", path: "/ws", port: 9443, tls: true)

although I got a different error (400 on handshake) and not the socket error.

If its worth anything, it works fine for me with this change:
- ws = HTTP::WebSocket.new(host: "stream.binance.com", path: "ws", port: 9443, tls: true)
+ ws = HTTP::WebSocket.new(host: "stream.binance.com", path: "/ws", port: 9443, tls: true)

although I got a different error (400 on handshake) and not the socket error.

I change to path: "/ws", but still socket error.

You can test websocket with a nice tool ws, screencast below:
https://user-images.githubusercontent.com/3498786/117101099-08485080-ada8-11eb-95cf-6c47682ba215.mov

I can't reproduce the connection error. So this looks like a network issue on your system.
Running the OP code results in Handshake got denied. Status code was 400.. That's a server error I can't know the reason of.

@straight-shoota That is fixed by my edit here: https://github.com/crystal-lang/crystal/issues/10670#issuecomment-831634561
@TangMonk's confusion stems from only Crystal having issues connecting. Golang, and that other tool they posted works fine.
It would seem to be due to some difference in the way Crystal issues DNS resolution on MacOS, that these other things don't? Maybe someone else on MacOS could try.

@TangMonk Could you try resolving the domain name and connecting to each IP address?
Something like this:
require "socket"

addresses = Socket::Addrinfo.tcp("stream.binance.com", 9443).map(&.ip_address)
addresses.each do |address|
  puts "Connecting to #{address}"
  TCPSocket.open(address.address, address.port) { puts "connected"; }
rescue IO::Error
  puts "failed"
end


When making a folder using Electron-forge, after installing the exe file, a green boot page will pop up. How to close it?

Pre-flight checklist

[X] I have read the contribution documentation for this project.
[X] I agree to follow the code of conduct that this project uses.
[X] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Electron Forge version
7.3.0
Electron version
29.1.4
Operating system
windows 11
Last known working Electron Forge version
No response
Expected behavior
How to remove this boot page? And I found that after closing the boot page, the program will exit and then reopen.
Actual behavior

Steps to reproduce
Package through make and then install the exe file.
Additional information
No response

/assign @Ayush9026 i will solve this.

in preload.js
function useLoading()
It's it.

@Akemi-Madoka Thanks for your reply, but can you provide more details? I didn't find anything about the useLoading function in the forge and electron documents.

You can at least swap out this loading gif in the installer via the https://js.electronforge.io/interfaces/_electron_forge_maker_squirrel.InternalOptions.SquirrelWindowsOptions.html#loadingGif option. I don't know if there's an additional API option to remove it at the moment.

@erickzhao Thank you for your reply. Yes, I saw the documentation for the loadingGif option, but as you said, this option is required and cannot be deleted.

@Akemi-Madoka Thank you for your help, but when I added the useLoading function, nothing changed and it still appeared. I tried to add log in the useLoading function and found that it was not executed.

I also encountered this problem and this topic helped me solve many problems. Thanks
But I still have a question,
The template generated by the official command we used,
npm init electron-app@latest my-new-app -- --template=vite
Nothing is added. Terminal execution npm run make
Packaged Windows installer, click Execute, why does it open my application directly instead of executing the installer?
Have you encountered this problem?


Disappearing Client Secret

Howdy,
Stumbled across this hub while trying to find a solution to a weird problem I'm having. In short, when I enter a client secret for an OAuth client that I've created, it disappears when I hit Save Changes. I've tried to check and see if maybe I didn't configure something properly but to no avail.
Am I missing something perhaps? Thanks!

Greetings to the 90ies ;)
That's by design - the secrets gets stored hashed. You can't have it back in plain text.

Oh, I feel silly. So then I suppose that means we have to remember what we
set it to before it goes into the void. Thank you for your reply!
On Tue, Aug 22, 2017 at 11:50 PM, Dominick Baier<EMAIL_ADDRESS>wrote:

Greetings to the 90ies ;)
That's by design - the secrets gets stored hashed. You can't have it back
in plain text.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/IdentityServer/IdentityServer2/issues/890#issuecomment-324220321,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Adz9MpqOfbA4o4tClCmLwFrHDnqG6iZIks5sa6-igaJpZM4O_QrL
.


Authentication stopped working

Simplenote#authenticate started raising an exception when I log in. I'm guessing Simplenote changed their API because simplenote.vim hasn't changed in a while, and I've verified my user/pass still works via the Simplenote Mac app and their website.
The only info I could find from HTTPError is 400 BAD REQUEST.

Also getting 'Login Failed' with correct login details.

If that's still happening then your only option is obtain a token from elsewhere (i.e the web app) and use it with that branch I've pushed. See the commit comment for how to get that token.
Still working on merging that in and caching a token (which I can do so we can manually edit that "cache" and add a token from elsewhere).

Once I've bumped simplenote.py to handle errors from invalid tokens ^ and then made changes here I think we can close this out.

Closing this out now with: https://github.com/mrtazz/simplenote.vim/commit/c79ca3f982b3cfe547d93e42907f1b1f527ebd82


A11y: Map icons do not show in WHCM

Describe the task

When in Windows High Contrast Mode / dark the map icons are hard to see
Note: I'm not sure how customizable the map icons are. If it's not working out, please let me know.

Acceptance Criteria
Open the page in WHCM dark or simulate it using Devtools(see link below)

[ ] The map icons will be easier to see / not gray. Please use ButtonText for the color. See: https://www.smashingmagazine.com/2022/03/windows-high-contrast-colors-mode-css-custom-properties/

Additional context

https://www.smashingmagazine.com/2022/06/guide-windows-high-contrast-mode/
How to simulate WHCM in Chrome Devtools - https://devtoolstips.org/tips/en/emulate-forced-colors/

@bacitracin does this still need to be worked on? if so, we can bump it in the slack channel

@nlebovits Yeah this one is still an issue. It's pretty low priority though, so no need to bump


customBodyRender won't render new state when changing table page

When using a custom component (MyCustomComponent) its local state doesn't seem to be rendered when moving between pages. The latest_version column from the example below is not updated/rendered, retaining the values from the first page, however, when changing the amount of rows per page, the customBodyRender component gets properly rendered.
Here's an example:
https://codesandbox.io/s/friendly-chebyshev-1k89e
Has anyone ever ran into this issue before?
Closest I found so far:
https://github.com/gregnb/mui-datatables/issues/822
https://github.com/gregnb/mui-datatables/issues/643

@romerojunior I did some digging and I think I know what's going on. Check out this modified version of your example:
https://codesandbox.io/s/twilight-cdn-2fo03
I updated it so that the Latest component would reflect the value of the input prop (in this case package_name). As you can see, the component correctly displays the prop value for each row. What's happening is that the component isn't unmounted when the page changes, only its input props are changing. Since your component relies on setting state in componentDidMount, you see this issue.
I'd update your row components not to rely on the componentDidMount function and to instead rely on the props.

Would you mind reuploading the modified example? It's currently down and I would like to check it to solve an issue with a customBodyRender getting stuck on the page 1 state for all items on page 2 and above.


setDate is not working properly

App crashes when you set a date programmatically and tries to change value.
lazyDatePicker.setDate(new Date());
java.lang.NullPointerException: Attempt to invoke interface method 'void android.text.TextWatcher.beforeTextChanged(java.lang.CharSequence, int, int, int)' on a null object reference

Do you use the latest version of the library? This has been corrected in this one:
implementation 'com.mikhaellopez:lazydatepicker:1.0.4'

Yes. Using latest version 1.0.4

Can you send me your XML and/or your implementation to try to helped you ? (maybe a sample project with your problem)
Because everything works well in the sample project by adding:
// Init LazyDatePicker
LazyDatePicker lazyDatePicker = findViewById(R.id.lazyDatePicker);
lazyDatePicker.setDate(new Date());

and
<com.mikhaellopez.lazydatepicker.LazyDatePicker
        android:id="@+id/lazyDatePicker"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        app:ldp_date_format="mm-dd-yyyy" />

XML
<com.mikhaellopez.lazydatepicker.LazyDatePicker                                 android:id="@+id/lazyDatePicker"                                 android:layout_gravity="center"                                 android:layout_width="match_parent"                                 android:layout_height="wrap_content"                                 app:ldp_date_format="mm-dd-yyyy"                                 android:padding="@dimen/size_5dp"                                 app:ldp_hint_color="@color/primaryGray"                                 app:ldp_text_color="@color/colorPrimary" />
LazyDatePicker lazyDatePicker = findViewById(R.id.lazyDatePicker); lazyDatePicker.setDate(new Date());
Now just try to pick a new date and it will give an error.

It's now fix on the last version 1.0.5:
implementation 'com.mikhaellopez:lazydatepicker:1.0.5'

Thx for you feeback
Can you confirm it's now ok for you ?

Fixed now. Excellent. :)


ed25519 does not work with node 12

ed25519 does not work with node 12.
https://github.com/gaoxiangxyz/ed25519/issues/1

More info:
Window 10 Pro. 1909- OS build: 18386.657
Node v 12.14.0
Npm 6.13.4

Added Node 12 PR to the upstream: https://github.com/gaoxiangxyz/ed25519/pull/2

Published package version 0.3.0 with Node 12 updates


use nginx proxy in Kubernetes

Broken out from #101.

Resources:

http://kubernetes.io/docs/user-guide/ingress/
https://github.com/kubernetes/kubernetes/issues/2856
https://github.com/nginxinc/kubernetes-ingress

No longer relevant.


Light2d ignores mask, occluders on all layers cast shadows

Godot Engine v3.2.1.stable.official (via Steam)
Windows 10
x64
Appears in both OpenGL 3.0 and 2.0 setup
Intel Core i7-770K CPU 4.20GHz
GeForce GTX 1060 6GB
Occluder2D, set on Light Mask layer 2 casts shadow, but Light2D is set on layer 1. When the Layer 1 on Light2D's Shadow Item Cull Mask is uncheched and checked again, everything looks good in editor, but reverts when project is played/exported or Light2D is saved as Scene.
Steps to reproduce:

In new project add 2 Light Occluders 2D, 1 Light2D (enable shadow) and background to see shadows (in my example it's Polygon2D)
Set one of the occluders' Light Mask to layer 2 only, it still casts shadow
Uncheck and check Light2D's shadow's Item Cull Mask Layer, shadow of second occluder shoudn't appear
Play Project, the shadow shows again, but only in game window

Minimal reproduction project:
Test.zip

Duplicate of #29612?

Oh, I didn't find this issue while searching, I'm not sure if it is exactly the same bug, but might have the same origin.

I've just been looking at this. It seems to be a bug that appears in the editor sporadically. I.e. I think the run version is correct, but the editor will sometimes show one shadow and sometimes both.
When I step through canvas_light_shadow_buffer_update I can see the occluder instances, in some cases in the editor there are 2 (correct) and in some cases only 1 (wrong).
This seems to be triggered when I change the shadow item cull mask for the light. If it starts at layer 1 they both show, if I click to only layer 2 in the editor neither show, when I click back on layer 1 both show, and when i turn off layer 2 (to the same situation as at the start) only the left shadow shows.

Ok I think the weird behaviour might be down to this.

On startup, the shadow_cull_mask doesn't appear to being set from the main code to visual server, thus it is using the default -1 (which is setting on all occluders)
When you do set the shadow_cull_mask in the editor, it is being set, so it then starts doing its thing
The shadow_cull_mask behaviour appears to be different from the layer masks used in the rest of godot (as I understand them):

In other parts of godot layer 1 has a special significance, showing everything (I guess being equivalent to all bits set). This doesn't appear to happen with the shadow mask.
The first thing to do will be to ensure that the shadow_mask is set properly on startup in the editor and in game, then decide what is the desired behaviour for the layers.

Ok the initialization problem may be quite simple. I figured out that when your light shadow_mask is set to layer 1 only (the default) nothing is written in the tscn file, therefore there is no call to canvas_light_set_item_shadow_cull_mask when starting the scene.
The problem is that the default value for shadow_mask in the constructor is -1, rather than 1.
I can change the constructor to 1. However we need to decide what is the intended behaviour for layers here. If the desire is to have the same behaviour of a master layer, then -1 may in fact be correct, and the problem is in how the layers are applied. We just need a consistent rule.

Fixed by #39162 (3.2) and 61b3eb3b28671a397ed873d47db925f662382e74 (master).


Use hyperfine for benchmarking

We should use hyperfine (or some other aggregation) tool for benchmarking the rewriters, and write the results to a JSON file to be able to easily merge the results into a single table.

hyperfine is undesirable since it measures total time instead of only rewriting time. However, the benchmark task should be adapted to aggregate several runs, but it should stop when a process fails.

Made my own repeat and compute the average, not as sophisticated but should be fine for simple benchmarking. The benchmark setup could be further improved by also allowing the REC input files instead of mCRL2, but this is a separate issue.


Error when change speed  within two ports from two different Novus 100G cards

Use this script to reproduced this issue
import snappi
ixn = snappi.api(host='https://<IP_ADDRESS>', ext='ixnetwork')
config=ixn.config()
config.ports.port( name='P1',location='<IP_ADDRESS>;01;1')
config.ports.port( name='P2',location='<IP_ADDRESS>;02;1')
layer1 = config.layer1.layer1()[-1]
layer1.name = '100G'
layer1.port_names =  [port.name for port in config.ports]
layer1.speed = 'speed_100_gbps'
layer1.ieee_media_defaults = False
layer1.auto_negotiate = False
layer1.auto_negotiation.rs_fec = True
config.options.port_options.location_preemption = True
response = ixn.set_config(config)

Looks like entire validation done in snappi[ixnetwork]==0.3.15 build

Ashutosh  5:24 PM
@here snappi[ixnetwork]==0.3.15 is out. Need your help in validating layer1 test script.
UHD - Ranga
Novus100G - Anish
Scenario raised by Dianna - Alakendu
Please provide update by 6:30 PM today. @Alakendu Jana please provide the version above to Dianna in that thread once we get an update from everyone here.
Anish  5:59 PM
Tested, works fine
Alakendu Jana  6:01 PM
Scenario raised by Dianna - Alakendu => Looks good
Rangababu R  6:50 PM
uhd works fine


Bugfix/home page typo

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: MartinCalderon963 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Failure to set AfterHostStarted working directory stops RHost

On my laptop I have (for some unknown reason) two user directories: alsher and alsher.REDMOND. The first one is not accessible without admin permissions and isn't really used. When I switch to local, RTVS peeks alsher as a user directory and since R has no access to it, setwd fails:
Error in setwd("C:/Users/alsher/Documents") :
  cannot change working directory

   at Microsoft.R.Host.Client.RExpressionEvaluatorExtensions.ThrowOnError(String expression, REvaluationResult res) in D:\GitHub\RTVS\src\Host\Client\Impl\Definitions\IRExpressionEvaluator.cs:line 146
   at Microsoft.R.Host.Client.RExpressionEvaluatorExtensions.<ExecuteAsync>d__3.MoveNext() in D:\GitHub\RTVS\src\Host\Client\Impl\Definitions\IRExpressionEvaluator.cs:line 117
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
   at Microsoft.R.Host.Client.Session.RSession.<AfterHostStarted>d__112.MoveNext() in D:\GitHub\RTVS\src\Host\Client\Impl\Session\RSession.cs:line 412
R Host process is stopped. Click "Reset" to start a new one.

I thought we added check for the directory existence. But I guess it also needs to handle access denied.

setwd should be fixed.


Sentinel example does not contain bicep implementation

Description
The Terraform example for deploying Sentinel does not make it clear that the purpose of the example is to show how to deploy a Terraform template after MLZ has been deployed using the Bicep template.
Expected Behavior
The first paragraph of the README should tell the reader that the template demonstrates how to deploy a Terraform template into a Mission Landing Zone instance that has been deployed using Bicep.
Actual Behavior
The README says that it's a Terraform template, but does not say why the example is useful.

@lisamurphy-msft I updated the description of the bug to add expected and actual behavior. We don't need an additional Bicep implementation of Sentinel (the core MLZ deployment already deploys Sentinel)--we need to tell readers why this example is here as Terraform deploying into an MLZ environment that was provisioned using Bicep.


Add DB Docker Setup, Review Readme, Review Auth Function

Easy DB setup with Docker
Revised Auth
Commented out Mailgun to make contributions easier (no need for paid API keys)
Change NPM for Yarn

Just a suggestion. The goal of this PR is to make it easier for new contributors to write the first few tests which I think is the most important next step.
interesting architecture, congrats

Thanks, looks good.

If you don't have MongoDB installed and running, install Docker and docker-compose and run:

But not everybody uses docker and is easier to just install Mongo.
Also changing npm to yarn adds another dependency, because you'll need to have yarn first installed to run the project.
I think those things should be optional, but I'll let the community decide whats best.

Added a comment at a line that confused me. Not sure if I didn't understand the flow or it is a valid concern.
The docker-compose file personally will help me but as Santiq said, it is not widely used and will add an additional learning curve to this repository. Small sized teams looking for a suitable architecture to start off with may skip using Docker. Is there a way to make running the project with Docker optional?
Also in the Roadmap, can you add a tick to "Add agenda dashboard" bullet point as that is available.

Is there a way to make running the project with Docker optional?
Because you can pass in a .env file to docker (and it gets put in env. variables), it would be possible to check if the env variables are set. If they arent already set, load from .env file. If they are, dont do anything.

@ent3r I believe it's optional the way it is and the docker solution just simplifies things for some people. You either:

run docker-compose up (this assumes you have docker installed)
OR
run mongoDB & run a DB mgmt tool as services in your machine (this assumes you have mongoDB & a DB mgmt tool installed)


Web page not found when inserting video url

Web page not found is showing in editor whenever I am trying to insert the video url from insert Video Url.
but when was checking Code view I got this :

In above I append https: in src then it's working fine.

It's issue is coming only in mobile platform. on web it's working.  please add this changes in iframe of  lib/summernote/summernote-lite.min.js

What is the URL you type into the "insert video" field? Does it start with "www" or directly "youtube.com"?

Inserting youtube video url start with simple https:
url is https://www.youtube.com/watch?v=W1pNjxmNHNQ

Thanks - this is apparently a known issue in Summernote: https://github.com/summernote/summernote/issues/3252
Let me see if I can modify the source to fix this.

Welcome, You can do it  you just need to go in lib/assets/summernote-lite.min.js as I mentioned above and search the ifram code then append https: in that. Thanks

Thanks - this is apparently a known issue in Summernote: summernote/summernote#3252
Let me see if I can modify the source to fix this.

Done, thanks for reporting!

I am waiting for it.

Done, thanks for reporting!
I am releasing 1.7.0+1 with the various fixes for the recent issues.

It is uploaded and should be available now :)


v1.12: k8s: Restrict configuring reserved:init policy via CNP

[ ] #28007 (@joestringer)

Once this PR is merged, you can update the PR labels via:
for pr in 28007; do contrib/backporting/set-labels.py $pr done 1.12; done

or with
make add-labels BRANCH=v1.12 ISSUES=28007

/test-backport-1.12

k8s-1.17-kernel-4.9 (test-1.17-4.9): this one is similar to https://github.com/cilium/cilium/issues/27995 although it's a different test case. the error message is "Cannot retrieve \"cilium-l8z4b\"'s policy revision: cannot get policy revision: \"

/test-1.17-4.9


Recipe: serve different content according to device type

@arcturus wrote a proof of concept, maybe we can add it as a recipe: https://github.com/arcturus/sw-device-resources

Yes, it was in my radar for another more than offline recipe.


bufio.(*Reader).Read panic: runtime error: slice bounds out of range

What version of Go are you using (go version)?

$ go version
go version go1.17.3 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?

I'm using gomavlib: https://github.com/aler9/gomavlib to build a server process , but find out some server crash like this:
 uavm[212516]: panic: runtime error: slice bounds out of range [:987] with capacity 512
 uavm[212516]: goroutine 845 [running]:
 uavm[212516]: bufio.(*Reader).Read(0xc0005f7860, {0xc00060d72f, 0x1, 0x1})
 uavm[212516]:         /usr/local/go/src/bufio/bufio.go:238 +0x7f9
 uavm[212516]: io.ReadAtLeast({0x1199040, 0xc0005f7860}, {0xc00060d72f, 0x1, 0x1}, 0x1)
 uavm[212516]:         /usr/local/go/src/io/io.go:328 +0xde
 uavm[212516]: io.ReadFull(...)
 uavm[212516]:         /usr/local/go/src/io/io.go:347
 uavm[212516]: github.com/aler9/gomavlib/pkg/frame.(*V2Frame).Decode(0xc00043e660, 0x7fd4bbec5420)
 uavm[212516]:         /home/XXX/go/pkg/mod/github.com/aler9/gomavlib@v0.0.0-20211028051547-c91b43b554d6/pkg/frame/v2.go:141 +0x3af
 uavm[212516]: github.com/aler9/gomavlib/pkg/transceiver.(*Transceiver).Read(0xc0005b4100)
 uavm[212516]:         /home/XXX/go/pkg/mod/github.com/aler9/gomavlib@v0.0.0-20211028051547-c91b43b554d6/pkg/transceiver/transceiver.go:130 +0xcb
 uavm[212516]: github.com/aler9/gomavlib.(*Channel).run.func1()
 uavm[212516]:         /home/XXX/go/pkg/mod/github.com/aler9/gomavlib@v0.0.0-20211028051547-c91b43b554d6/channel.go:93 +0x18f
 uavm[212516]: created by github.com/aler9/gomavlib.(*Channel).run
 uavm[212516]:         /home/XXX/go/pkg/mod/github.com/aler9/gomavlib@v0.0.0-20211028051547-c91b43b554d6/channel.go:85 +0x199
 systemd[1]: uavm.service: Main process exited, code=exited, status=2/INVALIDARGUMENT

The code in gomavlib causing the problem is :
func (f *V2Frame) Decode(br *bufio.Reader) error {
	// header
	buf, err := br.Peek(9)
	if err != nil {
		return err
	}
	br.Discard(9)
	msgLen := buf[0]
	f.IncompatibilityFlag = buf[1]
	f.CompatibilityFlag = buf[2]
	f.SequenceID = buf[3]
	f.SystemID = buf[4]
	f.ComponentID = buf[5]
	msgID := uint24Decode(buf[6:])

	// discard frame if incompatibility flag is not understood, as in recommendations
	if f.IncompatibilityFlag != 0 && f.IncompatibilityFlag != V2FlagSigned {
		return fmt.Errorf("unknown incompatibility flag (%d)", f.IncompatibilityFlag)
	}

	// message
	var msgEncoded []byte
	if msgLen > 0 {
		msgEncoded = make([]byte, msgLen)
                // THIS LINE IS CAUSING THE CRASH!!!!!!!!!!!!!!!!
		_, err = io.ReadFull(br, msgEncoded)
		if err != nil {
			return err
		}
	}

The bufio.Reader is created with a size of 512, how can we copy with a byte array index value 987?
I have already built the process using -race and found no data race report.  From my point of view, io.ReadFull is not supposed to crash on this scenario, that's why I fire an issue here.
But I can't reproduce it right now, I'm still trying to figure out what might cause this, any clues and suggestions are welcome.
What did you expect to see?
Expect the process not to crash.
What did you see instead?
Process crash on some situation.

Could you build a minimal reproducer with the function in question?

@andig Still trying, I will update here ASAP.

In case it helps, in similar issues in the past it seems like the panic might have been due to concurrent reads of the bufio.Reader.

@nishanths Thanks, that's the first thing came into my mind, I will double-check that.

Finally, I figured it out, it was caused by an invalid implementation of  io.Read:
https://github.com/aler9/gomavlib/blob/bb2bb0de607f0b2d782de88087cf82045a4ef1fc/pkg/udplistener/udplistener.go#L97
// Read implements the net.Conn interface.
func (c *conn) Read(byt []byte) (int, error) {
	var buf []byte
	var ok bool

	if !c.readDeadline.IsZero() {
		readTimer := time.NewTimer(time.Until(c.readDeadline))
		defer readTimer.Stop()

		select {
		case <-readTimer.C:
			return 0, errTimeout
		case buf, ok = <-c.read:
		}
	} else {
		buf, ok = <-c.read
	}

	if !ok {
		return 0, errTerminated
	}

	copy(byt, buf)
	c.listener.readDone <- struct{}{}
	return len(buf), nil
}

This function returns the length of buf instead of how many bytes has been copied into byt.
On the other hand, bufio use what have returned without any check to update the write position, thus causing an invalid w that out of range of inside buf, which in my case is something more than 512.
func (b *Reader) Read(p []byte) (n int, err error) {
	n = len(p)
	if n == 0 {
		if b.Buffered() > 0 {
			return 0, nil
		}
		return 0, b.readErr()
	}
	if b.r == b.w {
		if b.err != nil {
			return 0, b.readErr()
		}
		if len(p) >= len(b.buf) {
			// Large read, empty buffer.
			// Read directly into p to avoid copy.
			n, b.err = b.rd.Read(p)
			if n < 0 {
				panic(errNegativeRead)
			}
			if n > 0 {
				b.lastByte = int(p[n-1])
				b.lastRuneSize = -1
			}
			return n, b.readErr()
		}
		// One read.
		// Do not use b.fill, which will loop.
		b.r = 0
		b.w = 0
		n, b.err = b.rd.Read(b.buf)
		if n < 0 {
			panic(errNegativeRead)
		}
		if n == 0 {
			return 0, b.readErr()
		}
		b.w += n
	}

	// copy as much as we can
	n = copy(p, b.buf[b.r:b.w])
	b.r += n
	b.lastByte = int(b.buf[b.r-1])
	b.lastRuneSize = -1
	return n, nil
}

For my case, I will fix the invalid implementation of io.Read, but I think it will be helpful if we add some extra sanity check here, maybe a panic with an error, because your problem will panic anyway later on and it's much harder to trace.

We don't generally try to defend against misbehaving io.Readers. There are lots of places readers and writers are used, and do we really want to defend against bad behavior everywhere?
In addition, it's only a partial defense. It would only catch the error if the return value was wrong enough to overflow the buffer. A wrong answer that was still within the bounds of the buffer would generate bad behavior (junk data?) without triggering the check.
The error here isn't really all that much "later". It's immediately after the call to the underlying badly-behaved reader. It's not an unbounded time later.
All that said, I think we could do something here. Maybe just a comment on the line with [b.r:b.w] saying that if it out-of-bounds panics, it must be because of a misbehaving reader. That should be enough to get a hint about the cause from the line number of the backtrace.


Add Data Prepper Overview

Signed-off-by: Naarcha-AWS<EMAIL_ADDRESS>Description
Adds Data Prepper overview to main vs Data Prepper overview to the data-prepper-2.0 repo. Brings over content and feedback from Data Prepper 2.0 #1304.
Cannot merge before the release of Data Prepper 2.0.
Issues Resolved
Part of issue #1267
Checklist

[x] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@Naarcha-AWS  Did you mean to close this PR? Does it need more reviews?

Overview has been published on this page: https://opensearch.org/docs/latest/data-prepper/index/


TypeError: fn is not a function

Please describe the problem you are having in as much detail as possible:
TypeError: fn is not a function
    at Map.filter (/app/node_modules/discord.js/src/util/Collection.js:268:11)
    at message.channel.fetchMessages.then.messages (/app/Bots/Self/commands/clear.js:7:18)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:188:7)

https://github.com/hydrabolt/discord.js/blob/master/src/util/Collection.js#L268
It functions as supposed to, but also outputs this error.
Include a reproducible code sample here, if possible:
message.channel.fetchMessages({ limit: Math.min(args[1], 100), before: message.id }).then(messages => {
  messages.filter(() => message.author.id === client.user.id).map(m => m.delete());
  message.edit(`✅ Cleared ${messages.filter(m => m.author.id === client.user.id).size} messages`).then(message.delete({ timeout: 10000 }));
});

Further details:

discord.js version: 12.0.0-dev
node.js version: 8.4.0
Operating system: Windows 10 & Ubuntu 16.04
Priority this issue should have – please be realistic and elaborate if possible: Low

[x] I found this issue while running code on a user account
[x] I have also tested the issue on latest master, commit hash: 243ff48a67446fba96da91503670cea0aba3a968

Thanks for the better code 👍 but the point is it worked before I updated to the latest commit and now it doesn't 👎

The error is that you didn't give a function to .then

Yes thank you, #general just confirmed that.
Thanks for support 👍

next time you should ask on one of the support channels 🤔  (if you didn't do that already)


animated_plots

Addresses issue #299
currently implemented for one orbit at a time

Hello @dark00infinity, If this PR is a Work in Progress effort ... It is better to convert it to a Draft PR ... You can always mark it Ready for Review whenever you feel this is ready :)

Hello @dark00infinity, If this PR is a Work in Progress effort ... It is better to convert it to a Draft PR ... You can always mark it Ready for Review whenever you feel this is ready :)

ok @DhruvSondhi

For testing the animations, here is a collection of resources:

This issue from the official pytetst-mpl plugin about providing support for animations: https://github.com/matplotlib/pytest-mpl/issues/74

Its original author offers the possibility of using the following testing utilities: https://github.com/t-makaro/animatplot/blob/master/tests/tools.py. However, it would be interesting to implement this under the pytest-mpl.

For the moment, let us try first to implement the animation class. We will carry about tests once everything will be ready merge 👍🏽


修改license文件头

统一替换为

/*
 * Copyright (c) Baidu Inc. All rights reserved.
 *
 * Licensed to the Apache Software Foundation (ASF) under one or more
 * contributor license agreements.  See the NOTICE file distributed with
 * this work for additional information regarding copyright ownership.
 * The ASF licenses this file to You under the Apache License, Version 2.0
 * (the "License"); you may not use this file except in compliance with
 * the License.  You may obtain a copy of the License at
 *
 *      http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

统一替换为
/*

Copyright (c) Baidu Inc. All rights reserved.

Licensed to the Apache Software Foundation (ASF) under one or more
contributor license agreements.  See the NOTICE file distributed with
this work for additional information regarding copyright ownership.
The ASF licenses this file to You under the Apache License, Version 2.0
(the "License"); you may not use this file except in compliance with
the License.  You may obtain a copy of the License at

 http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
*/

已修改上传


Caret symbol not included in stored URL when using non-encode plugin

I found a similar Issue that is marked solved so I suspect I've caused this by making use of a plugin that was found inside another Issue solution..  there for not a bug
My problem URL comes from a Splunk search:
https://hidden/search?earliest=-15m&latest=now&q=search tag%3DSpeechIVR-Production tag%3DSpeechIVR-Log   | rex ".*TSAnswerPhone%3ASessionId%3A(%3F<SessionId>[^%3A]%2B)" &display.page.search.mode=fast&dispatch.sample_ratio=1&display.general.type=events&display.page.search.tab=events&sid=1495425928.166882_668608EB-CEE5-4FA2-A108-FC3E8A28D65F
The stored URL using YOURLS results in:
https://hidden/search?earliest=-15m&latest=now&q=search tag%3DSpeechIVR-Production tag%3DSpeechIVR-Log   | rex ".*TSAnswerPhone%3ASessionId%3A(%3F<SessionId>[%3A]%2B)" &display.page.search.mode=fast&dispatch.sample_ratio=1&display.general.type=events&display.page.search.tab=events&sid=1495425928.166882_668608EB-CEE5-4FA2-A108-FC3E8A28D65F )
The problem for me is that [^%3A] in original gets stored as [%3A].  the caret is completely dropped so I don't think it's an encoding issue.  Suspect it's special handling inside the code base (?) and my reason for logging as an issue is that it's probably unintended and caused by the plugin.  I wouldn't call it a bug since plugins surely are buyer beware...
The plugin I am using is the following:
/*
Plugin Name: Don't encode/decode
Plugin URI: http://yourls.org/
Description: Don't encode/decode submitted URLs
Version: 1.0
Author: Ozh
Author URI: http://ozh.org/
*/

yourls_add_filter( 'encodeURI', 'stewx_no_decode' );
function stewx_no_decode( $decoded, $original ) {
    return $original;
}

This plugin made the Splunk URL's work perfectly (until this example).
If I deactivate the plugin the URL stored is as follows:
https://hidden/search?earliest=-15m&latest=now&q=search tag=SpeechIVR-Production tag=SpeechIVR-Log   | rex ".*TSAnswerPhone:SessionId:(?<SessionId>[^:]+)" &display.page.search.mode=fast&dispatch.sample_ratio=1&display.general.type=events&display.page.search.tab=events&sid=1495425928.166882_668608EB-CEE5-4FA2-A108-FC3E8A28D65F
This URL doesn't work for us because at the first ? in this version, it's probably being interpreted as a parameter.  In the versions using the plugin it's being stored "as is" which is great.  Conversely the Caret in the non plugin version gets encoded.
I am using your tool on RHEL running lampp
MediaWiki 	1.24.1
PHP 	5.6.3 (apache2handler)
MySQL 	5.6.21
I think if someone was willing to help a non-coder here - and write me a plugin that encoded only a list of characters (e.g. caret "^") but nothing else - could call it Splunk URL plugin because this tool is extremely useful for users sharing splunk searches (which can be thousands of characters long)..

Any updates on this? As I'm now running into the exact same issue. My ^ are being converted to %5E but not converted back properly. Which breaks the purpose.

Any updates on this?

Not yet, we are aware of the issue but no solution has been submitted so far.
Feel free to submit your PR if you have one! 🙂

Any updates on this?

Not yet, we are aware of the issue but no solution has been submitted so far.
Feel free to submit your PR if you have one! 🙂

I don't have a new PR to submit... But, I were able to circumvent it with this plugin I wrote,
<?php
/*
Plugin Name: Do Not Encode
Description:
Version: 1.0
Author: Norway174
Author URI:
*/

yourls_add_filter( 'sanitize_url', 'decode' );

function decode( $url, $unsafe_url ){
	return rawurldecode($url);
}

Based off this one.
But that one didn't work for me, so I ended up writing my own.

Closing: 2691 and associated PR will supersede


HDDS-8389. [Snapshot] Added integration test for SnapDiff when OM leader failover happens

What changes were proposed in this pull request?
Added integration test for SnapDiff when OM leader failover happens
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-8389
How was this patch tested?
Ran newly added tests locally and verified they pass.

@swamirishi Could you take a look at this?

There are some conflicts in the test due to HDDS-8645.

Fixed conflicts and also ran test 100 times to check flakiness.
It passed all the time (Workflow failed because of timeout) : https://github.com/hemantk-12/ozone/actions/runs/5049594952/jobs/9059656428

Thanks @hemantk-12 for the test addition, and @swamirishi for the review.


Add courseRun

Feature Request
Is your feature request related to a problem or unsupported use case? Please describe.
We need to add the courseRun to complete course session information(startDate, endDate, subscriptionDate,etc.)
Describe the solution you'd like
Add a page model that we can attach by plugin to course page.
Describe alternatives you've considered
Have the courseRun created as plugin of course. What do you think @sveetch ?
Discovery, Documentation, Adoption, Migration Strategy
schemaRichie.pdf
Do you want to work on it through a Pull Request?
Proof of concept within the next 2 weeks.

I had lengthy talks with Robert and we believe that having the courseRun only exist within the course is a mistake. One that openEdx has done. It now wants to seperate the runs from the courses according to the following doc: https://github.com/edx/open-edx-proposals/blob/mroytman/oep-0027/oeps/oep-0027-course-metadata-authoring.rst
We should avoid the initial mistake and separate our runs from the course right now.
Associating the course run to a course in the same way a course is associated to it's main organization would allow us to integrate it to the course page and thus control which one are shown. The added benefit being that each and all course run page can be consulted by the pedagogical engineer's. Whereas having the complete history only in the course admin would not allow that role to consult them since it is likely we won't allow acces to the admin interface.
I'll rework Samuel's prototype on another branch.

I'm proposing #338 as base for this.


Update Travis and Ruby dependencies

This PR updates Travis from trusty to bionic, thereby updating Ruby so we can update the Ruby dependencies, especially nokogiri.

Danger has errored
[!] Invalid Dangerfile file: No jacoco xml report found at sealedenum/build/reports/jacoco/test/jacocoTestReport.xml
 #  from Dangerfile:1
 #  -------------------------------------------
 >  shroud.report 'sealedenum/build/reports/jacoco/test/jacocoTestReport.xml', 80, 80, true
 #  shroud.report 'sealedenumprocessor/build/reports/jacoco/test/jacocoTestReport.xml', 80, 80, true
 #  -------------------------------------------

  Generated by :no_entry_sign: Danger

Danger has errored
[!] Invalid Dangerfile file: No jacoco xml report found at sealedenum/build/reports/jacoco/test/jacocoTestReport.xml
 #  from Dangerfile:1
 #  -------------------------------------------
 >  shroud.report 'sealedenum/build/reports/jacoco/test/jacocoTestReport.xml', 80, 80, true
 #  shroud.report 'sealedenumprocessor/build/reports/jacoco/test/jacocoTestReport.xml', 80, 80, true
 #  -------------------------------------------

  Generated by :no_entry_sign: Danger

Thanks @shadyoak !

Thanks @shadyoak !


Billing api extra labels

Codecov Report
Attention: 50 lines in your changes are missing coverage. Please review.

Comparison is base (bdb637d) 69.66% compared to head (e93bef8) 69.58%.

Files
Patch %
Lines

db/python/layers/billing_layer.py
40.00%
30 Missing :warning:

api/routes/billing.py
50.00%
12 Missing :warning:

db/python/layers/billing_db.py
46.66%
8 Missing :warning:

Additional details and impacted files
@@                   Coverage Diff                   @@
##           billing-release-002     #619      +/-   ##
=======================================================
- Coverage                69.66%   69.58%   -0.08%
=======================================================
  Files                      116      117       +1
  Lines                     9517     9565      +48
=======================================================
+ Hits                      6630     6656      +26
- Misses                    2887     2909      +22

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


haskellPackages.{github,nix-thunk}: Fix

Motivation for this change
hackage-packages.nix should be regenerated instead of hand-updated, but I don't know the proper way to do that.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Good to merge after https://github.com/obsidiansystems/nix-thunk/pull/22 too so the commit doesn't go away.

@cdepillabout @sternenseemann @maralorn @expipiplus1 Know what I should do this this? This is draft only because I am not sure how to regenerated, whether this should go to haskell-updates, etc.

This should go to haskell-updates
hackage-packages.nix can be regenerated with ./maintainers/scripts/haskell/regenerate-hackage-packages.sh --do-commit
Thanks!

How should I backport this to 21.11? I cherry-picked the first commit and went to regenerate, but some other things got downgraded. I guess the 21.11 packages are out of sync with the yaml :/

@Ericson2314 I am sorry. We currently haven‘t optimized the infrastructure for backports. We currently don‘t regenerate the file on stable. If possible you need to manually only cherry-pick the changes you want for the file, if you used the regenerate feature.

I'll just do a regular backport then, and punt on fixing the existing issues making it cleanly regenerate. Thanks for letting me know.


Unable to create qemu vm's after proxmox ve 7.3-3 update

I'm unable to apply my plan after upgrading to proxmox ve 7.3-3.
It starts out fine, but for some reason is unable to 'finish' the vm it has just deployed. The logs say something about the vm is stopped, but in proxmox ui it is running.
I attached logs, can you make out the problem with these?
terraform-plugin-proxmox.zip

I'm facing the same issue, not sure how to go about this, since TRACE logs don't contain much of use.

I'm also having this similar issue. The VM is created and running, but in the console I'm getting this message : Error: VM XXX already running

Also seeing this more often now.
Error: VM XXX already running

Also, come across this issue, I have proxmox ve 7.3-3 installed. I found this old issue:
https://github.com/Telmate/terraform-provider-proxmox/issues/460
set my iothread to 0 and all was working again

I have similar issue. But in my case, there is a slightly different problem, with the help of the terraform debug, I see that there are calls to proxmox HA, I did not force these options, but it went to poll them, and fell out into the error TASK ERROR: VM 105 already running. But after I added the hastate, hagroup. Vm was able to rise to normal

After I changed the iothread option to 0, the problem also went away, there were calls to proxmox HA, but no additional errors popped up, and the vm was able to create correctly


HHH-9374 EntityGraph applied to subquery when using collection function

Replaces https://github.com/hibernate/hibernate-orm/pull/826
We would be grateful if you merged those changes with branch 4.3 :)

Added a test case demonstrating the issue existence

Hi,
Can I do something to help you merge this correction?
Otherwise, what are we waiting for?

Sometimes it just takes pinging us.  I scheduled it for inclusion in 5.0.3

Ok thanks ;)
Do you plane to backmerge the correction on current stable branch?

5.0 is the stable branch
On Fri, Oct 2, 2015, 9:55 AM Réda Housni Alaoui<EMAIL_ADDRESS>wrote:

Ok thanks ;)
Do you plane to backmerge the correction on current stable branch?
—
Reply to this email directly or view it on GitHub
https://github.com/hibernate/hibernate-orm/pull/947#issuecomment-145050504
.

Applied upstream.  Thanks!

Can we please have that fix in an upcoming 4.3 release?


Update "acceleration/TensorRT_inference_acceleration.ipynb"

Fixes #1667
Checks

[ ] Avoid including large-size files in the PR.
[ ] Clean up long text outputs from code cells in the notebook.
[ ] For security purposes, please check the contents and remove any sensitive info such as user names and private key.
[ ] Ensure (1) hyperlinks and markdown anchors are working (2) use relative paths for tutorial repo files (3) put figure and graphs in the ./figure folder
[ ] Notebook runs automatically ./runner.sh -t <path to .ipynb file>

To function properly, this PR must also update the bundle.

To function properly, this PR must also update the bundle.

Hi @KumoLiu , could to post a link of the

To function properly, this PR must also update the bundle.

Do you mean the change for load_pretrain support?
I suggest to update the endoscopic_tool_segmentation bundle first and update the cell output of this notebook