The video_player plugin fails to play videos (sometimes)

There is a race condition in the video_player plugin code that prevents videos from displaying on a Widget.
Steps to Reproduce

Download a video on your phone: http://www.sample-videos.com/video/mp4/720/big_buck_bunny_720p_20mb.mp4
Edit the image_picker sample app (https://github.com/flutter/plugins/blob/master/packages/image_picker/example/lib/main.dart#L50) and add the following in line 50 just after if (file != null && mounted) {:

print(file);

Run the image_picker sample app, enable read permissions from external storage and choose the video from the gallery. Keep note of the video path. It will look something like /storage/emulated/0/Download/big_buck_bunny_720p_20mb.mp4 (remove the file:// prefix if needed)
Replace the main.dart content in image_picker by the code below (Based on the video_player sample app found here.
Open and close the app until the video no longer shows. You should see a blank screen.

import 'dart:io';

import 'package:video_player/video_player.dart';
import 'package:flutter/material.dart';

void main() => runApp(VideoApp());

class VideoApp extends StatefulWidget {
  @override
  _VideoAppState createState() => _VideoAppState();
}

class _VideoAppState extends State<VideoApp> {
  VideoPlayerController _controller;

  @override
  void initState() {
    try {
      super.initState();
      // _controller = VideoPlayerController.network('http://www.sample-videos.com/video/mp4/720/big_buck_bunny_720p_20mb.mp4',)
      _controller = VideoPlayerController.file(
          new File('/storage/emulated/0/Download/big_buck_bunny_720p_20mb.mp4'))
        ..addListener(() {
          print('controller listener: $_controller');
          // Ensure the first frame is shown after the video is initialized, even before the play button has been pressed.
          setState(() {});
        })
        ..initialize().then((_) {
          print('initialized listener: $_controller');
          setState(() {});
        }).catchError((error) {
          print('Unexpected error1: $error');
        });
      print('controller is set');
    } catch (e) {
      print('Unexpected error: $e');
    }
  }

  @override
  Widget build(BuildContext context) {
    try {
      print('entering build. Needs _controller $_controller');
      return MaterialApp(
        title: 'Video Demo',
        home: Scaffold(
          body: Center(
            child: _controller.value.initialized
                ? AspectRatio(
                    aspectRatio: _controller.value.aspectRatio,
                    child: VideoPlayer(_controller),
                  )
                : Container(),
          ),
          floatingActionButton: FloatingActionButton(
            onPressed: _controller.value.isPlaying
                ? _controller.pause
                : _controller.play,
            child: Icon(
              _controller.value.isPlaying ? Icons.pause : Icons.play_arrow,
            ),
          ),
        ),
      );
    } catch (e) {
      print('Unexpected error: $e');
    }
  }
}

Buggy app:

Expected:

Logs
When the video works. You'll see in the logs:
I/flutter ( 8147): controller listener: VideoPlayerController#fff7e(VideoPlayerValue(duration: 0:01:57.312000, size: Size(1280.0, 720.0), position: 0:00:00.000000, buffered: [], isPlaying: false, isLooping: false, isBuffering: falsevolume: 1.0, errorDescription: null,rotationDegrees: 0))
I/flutter ( 8147): initialized listener: VideoPlayerController#fff7e(VideoPlayerValue(duration: 0:01:57.312000, size: Size(1280.0, 720.0), position: 0:00:00.000000, buff

Otherwise you will see something like (the _controller.value is an empty/default video instance). Notice the initialized listener is never called:
I/flutter (10928): controller listener: VideoPlayerController#0235e(VideoPlayerValue(duration: null, size: null, position: 0:00:00.000000, buffered: [], isPlaying: false, isLooping: false, isBuffering: falsevolume: 1.0, errorDescription: null,rotationDegrees: null))

Root Cause
The initialized event is not always received in the dart code. This event is sent from the ExoPlayer to the Java code, from there to the Dart world through an Event Channel. However it is possible that the init event from the ExoPlayer is fired before the dart code had a chance to listen to any broadcasts from the Java code (eventSink is null in https://github.com/flutter/plugins/blob/master/packages/video_player/android/src/main/java/io/flutter/plugins/videoplayer/VideoPlayerPlugin.java#L213). In such a case, the init event is not fired and the video cannot be played nor displayed.

flutter doctor -v
[✓] Flutter (Channel master, v0.5.8-pre.152, on Mac OS X 10.12.6 16G1510, locale fr-FR)
    • Flutter version 0.5.8-pre.152 at /Users/rodrigo/Libs/flutter
    • Framework revision 8b05629658 (3 days ago), 2018-07-26 19:12:04 -0700
    • Engine revision 6f7bdc9fab
    • Dart version 2.0.0-dev.69.3.flutter-937ee2e8ca

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

[✓] iOS toolchain - develop for iOS devices (Xcode 9.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.2, Build version 9C40b
    • ios-deploy 1.9.2
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 26.0.1
    • Dart plugin version 173.4700
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)

[!] IntelliJ IDEA Community Edition (version 2018.1.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • For information about installing plugins, see
      https://flutter.io/intellij-setup/#installing-the-plugins

[!] VS Code (version 1.25.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension not installed; install from
      https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected devices (1 available)
    • SM G920F • 02157df2bc245b2a • android-arm64 • Android 7.0 (API 24)

! Doctor found issues in 2 categories.

This is possibly the same issue experienced by https://github.com/flutter/flutter/issues/19197, https://github.com/flutter/flutter/issues/14198 and https://github.com/flutter/flutter/issues/18836
Note: I have yet to check the iOS plugin code. So some of those issues could be a different bug.

Closing as duplicate of https://github.com/flutter/flutter/issues/21483

Hi!
I have just a question can i play a video offline in flutter ?does it work for both ios and android ?

Hi!
I mean without halving network on my device so the wi-fi is off!

@zhen1991 yes, when it's stored as a file on your device.

Hi,
i also have one question.
sorry may be this is not the right to ask different question,
Q. i'm using VideoPalyer plugin,this plugin works perfectly,but...video player displaying green overlay on screen.

@sujitpasalkar there was a bug fixed recently. Ensure you have the latest Flutter and video_player plugin.

i have same problem on ios 14 beta.
it play online video perfectly but when i download that file and try to play it, nothing happens.
it would not throw error neither.
show nothing in logs.
i'm losing my mind :|

@P-B1101 , I am not sure if I have the same problem as you.
One of my iOS 14 GM can play video with ONLY sound.
However, my another iOS 14 GM is totally fine.
I am losing my mind also.

@leochiu-work  You may wanna refer to this Issue cause i've been suffering from same issue over a week where as issue was https

Initialize the initial value of controller as empty url ("") in initState and then place your build function below it.
_controller = VideoPlayerController.network("")
..addListener(() => setState(() {
videoPosition = controller.value.position;
}))
..initialize().then(() => setState(() {
videoLength = _controller.value.duration;
}));

I am getting an error
E/flutter (22049): [ERROR:flutter/lib/ui/ui_dart_state.cc(177)] Unhandled Exception: PlatformException(VideoError, Video player had error com.google.android.exoplayer2.ExoPlaybackException: com.google.android.exoplayer2.audio.AudioSink$InitializationException: AudioTrack init failed: 0, Config(48000, 12, 61504), null, null)
Any suggestion will be appreciated. Thanks.

@kamleshwebtech
Were you able to fix the Platform Exception? If yes, can you please share how?
Thanks.

@kamleshwebtech
Were you able to fix the Platform Exception? If yes, can you please share how?
Thanks.

I am getting an error
E/flutter (22049): [ERROR:flutter/lib/ui/ui_dart_state.cc(177)] Unhandled Exception: PlatformException(VideoError, Video player had error com.google.android.exoplayer2.ExoPlaybackException: com.google.android.exoplayer2.audio.AudioSink$InitializationException: AudioTrack init failed: 0, Config(48000, 12, 61504), null, null)
Any suggestion will be appreciated. Thanks.

I am facing the same.
If anyone can help please!


Stuck in Queue

Woke up this morning to see that I have mined over 400k IOTA. However, the withdrawal button was grayed out and every time I hit update it says "Your withdrawal request is already in queue: 5". After some time there is still no sign of the withdrawal being made. I don't know what I should do. Also, every time I hit update my IOTA points go up slightly, but I am not currently mining.

Hi. whats is your public address?

JQKJPNRX9RCYESZVWPJUJGUSUZJLJYHXCLYIYWGLIQZUKXXHEQFFBMEKEZRGLEGIBLGSUCQEUXRS9KPQAM9CQVCTSC

This is because you win jackpot while mining, you can have only one withdrawal request at time. From jackpot it is auto withdrawal. If you check your history, you have some wins in past hours. And because IOTA have a lot of problems with confirmation, and can take hours.

So I just have to wait a while until It is all withdrawn?

No all, only withdrawal to your address. Sorry for this, but in past this wasnt so much problem, because was a lot more miners. So you dont win so much.

Ok. Thanks for the help.

On homepage you can see how much long take actual batch of transaction to get confirmated. Now it is for example 82 minutes, sometimes it is only 3 minutes. Some time couple hours. Old iota protocol is mystery. I hope new will do better.

Now you can


Added predictive model tutorial to Other Resources

Added a link to a tutorial on building a predictive model in python under Other Resources.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@akwhite Thanks for this pull request (PR).
Unfortunately, we are marking this PR invalid for hacktoberfest. We believe it does not follow the recommended guidelines:

Quality Standards
In line with Hacktoberfest value #2 (Quantity is fun, Quality is key), we have provided examples of the quality >standards we encourage. This applies mainly to beginners.

PRs that are automated e.g. scripted opening PRs to remove whitespace / optimize images.
PRs that are disruptive e.g. taking someone else's branch/commits and making a PR.
PRs that are regarded by a project maintainer as a hindrance vs. helping.
Something that's clearly an attempt to simply +1 your PR count for October.

Learn more at https://hacktoberfest.digitalocean.com/details


Correct condition in Parser::isConfig

I've corrected condition for ASCII letters. Before it was always true for any char.

invalid branch. should goto develop


added a test with overlapping subscription

cc @gavindmello @yutter @wuhkuh
This is a failing test.

The latest commit should fix it, thanks to @GavinDmello ^^.

Coverage increased (+0.2%) to 97.416% when pulling 7a716dd84e8a0007832e27e9371f17a00d8e78cf on overlapping into cf4e7ca028b9ea7228cfd01b845bfe64f546d608 on master.


add tests for server channel

More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

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

@k-wall , my draft tests. FYI


Missing Unused Function Parameters

When introducing partial variable access optimizations, we started to treat unused function parameters as invalid, despite being very real parameters.
This has the consequence of producing incorrect function prototypes that we otherwise could be reproducing correctly, and making HLIL look worse.
See attached binary for this program:
// functions.c

#include <stdint.h>
#include <stdbool.h>

void func1()
{
}

int func2(int a, int b)
{
  return a + b;
}

void *func3(int a, void *b, char *c, char d)
{
  return (void *)func3;
}

void *(*func4())(int a, void *b, char *c, char d)
{
  return func3;
}

bool func5(uint8_t a, char b, bool c, int d)
{
  return false;
}

int main()
{

  return 0;
}

functions.zip

This appears to have been fixed some time after 3.5. 3.6-dev and greater builds should correctly see function arguments in functions where those arguments aren't being used.


Added initial size for HashMaps

To avoid it being resized when Redis Hash has too many keys.

LGTM. Great finding.

Yeah I forgot we might release 2.8.x. :)

LGTM after addressing @mp911de comments.

Added load factor 1 as suggested by @mp911de

Amazing! Good catch! LGTM!

Merged. Applied to 2.8 and 2.9 respectively.


Membership validation

As of v9, i don't see a way to validate a struct field that can only take values from a restricted set of possible values. This is meant to act as a replacement for sum types (enums).
Example use case: Making sure that User.Gender can take values within []string{"male", "female", "n/a"}
Any suggestions?

Hey @alexyans
there are 2 ways, I recommend the 2nd

validate:"eq=male|eq=female|eq=n/a"

Use a function eg.

const (
  Val1 string = "value1"
  Val2 string = "value2"
   ...
)

m := map[string]struct{}{
  Val1:struct{}{},
  Val2:struct{}{},
}

validate := validator.New()
validate.RegisterValidation("mytag", func(fl validator.FieldLevel) bool {
  _,ok := m[fl.Field().String()]
  return ok
})

please let me know if that makes sense.

Hey @joeybloggs,
can you give another example? I don't understand how you compare the const struct with the map object? I would expect your algorithm is checking if the actual object matches the model?
thanks in advance!

Hi @joeybloggs Yes, this approach helped me. Thank you for your message. But, I found that we could do the validation without custom validation function. My suggestion is:
const (
Val1 string = "value1"
Val2 string = "value2"
...
)
m := map[string]struct{}{
Val1:struct{}{},
Val2:struct{}{},
}
_,ok := m[string]
if ok == true
return ok

There's a new validation added which could help.
Gender string `validator="oneof=male female n/a"`

I'm actually not sure if the slash will work, not sure if that's a reserved character.  But may be useful anyway.  Works with strings or whole numbers.


no shareable URL for a specific inspector query state

i would like to share a URL for this view:

however, the URL in the browser and the one accessible from share in the right side panel both produce this result:

@rwelty1889 - is this still a problem? When I paste this URL into a browser (e.g. copy from Brave, paste into Firefox), I see the search results.
https://www.openhistoricalmap.org/search?query=madame damnable#map=20/47.59899/-122.33457&layers=O&date=1874-06-21&daterange=1850-01-01,2023-12-31


Add support for object type spread

Flow has added support for spreading flow object types into other object types. See https://github.com/facebook/flow/issues/1326.
These should be converted into a tcomb intersection. One way to do this could be to look for members that are ObjectTypeSpreadProperty here: https://github.com/gcanti/babel-plugin-tcomb/blob/master/src/index.js#L393.
Then we could extract those out, create an interface of the remaining ones and then creating an intersection.
For our use case we also make the objects that we spread $Exact<> in order to make the types be non-mabye. See https://github.com/facebook/flow/issues/3534. tcomb spread doesn't seem to have the same limitation, so we could/should probably just discard the $Exact<> when spreading.

Gave it a try :point_up:

These should be converted into a tcomb intersection

Based on this comment they are pretty different from intersections

Based on this comment they are pretty different from intersections

Ah, that's true. I was a bit quick on the trigger.
Seems like doing t.interface.extend() will be the right thing to do?

Updated PR to do interface.extend instead.

I'm afraid is not well suited: from the documentation of tcomb you linked

Note. Repeated props are not allowed (unless they are strictly equal)

Also, in absence of official documentaton (!), the following rules seem to apply

every property from spread becomes optional
every property before non-exact spread becomes mixed
properties after spread stay the same
$Exact changes the rules above

so the behavior is way different from t.interface.extend

Thank you so much for the feedback.
Aha. I understood

Alternatively you can use the t.interface.extend(mixins: Array, options?: string | Options) => TcombType function:

To mean that I could use t.interface.extend to allow repeated props.

every property from spread becomes optional

Would an approach be to iterate over the properties in the interface being spread and make them into maybe by default, but not when the interface is $Exact<>?

every property before non-exact spread becomes mixed

We could do this by splitting properties into two groups, before and after spread and making those before spread mixed ?
And then using extend to join these properties together, given that t.interface.extend allows repeated props. If not we could maybe do some hand-wrangling to create a new interface directly?

Would a way to simplify this be to add an option to .extend() that allow overwrites that gets passed to mixin(). Or would you rather not allow this in plain tcomb ?

IMO the better option would be to define an helper function (actually a new custom tcomb combinator) and inject when needed, like assert or extend (see here https://github.com/gcanti/babel-plugin-tcomb/blob/master/src/index.js#L64). Since tcomb types are fully introspectable at runtime, you should be able to implement the required complex logic.

I've updated the pr and I think it takes in to account all of the strange rules surrounding the object spread. How do you feel about the code, and the separation between what is done at compile time vs runtime?

How do you feel about the code, and the separation between what is done at compile time vs runtime?

Seems a good balance. Let me know when you consider the PR mergeable. If you agree I'll publish this feature tagged as Experimental for now

From the CHANGELOG file: A feature tagged as Experimental is in a high state of flux, you're at risk of it changing without notice.

because

the upstream feature seems yet unstable
the feature must be tested in real world scenarios

:+1:
I've bumped the version and updated the changelog. Is there anything I've missed? If not I feel like its mergable.


Bug: multiple deploys of the same exported backend fails

When trying to deploy more than once the same exported backend, deployment fails due to name collisions, for example S3 buckets.
I've been trying to use stackName parameter in AmplifyExportedBackend but looks like it doesn't work.

Hey @PatrykMilewski :wave: thanks for raising this and apologies for the delay on this one! I was able to successfully reproduce the issue where stackName does not appear to rename the stack, however I was not encountering an issue during deployment. Instead, CDK detected the corresponding app and stack name

Marking as a bug 🙂

Digging into this a bit further it appears this prop is appearing from the extended StackProps https://github.com/aws-amplify/amplify-cli-export-construct/blob/main/src/amplify-exported-backend-props.ts#L3, but is not consumed in the base construct https://github.com/aws-amplify/amplify-cli-export-construct/blob/main/src/base-exported-backend.ts#L46-L48
Instead, it is reading the stackName from the export manifest


Fix installs for debian 11.

Not sure what the implications of the archive are tough ?

This should probably trickle upstream eventually.


Precision of range and total driving range has too many digits.

Hyundai / Kia Connect version: 2.3.1
Python version: 3.10
Operating System:
** Home Assistant 2022.12.8
** Supervisor 2022.11.2
** Operating System 9.4
** Frontend 20221213.1 - latest

Description
Precision of range and total driving range has too many digits.
For example:
IONIQ EV Total Driving Range
32 minutes ago
106.2524272 mi
What I Did
Viewed the values in Home Assistant.
Precision is too great.
Preferably something like two decimal places (certainly no more).

Could you post the contents of your data sensor attribute? I assume the api provides this many digits?

Yes, this many digits are in the sensor data. I have removed the location from the data below, otherwise unedited.
Vehicle data
alt: 0
type: 0
head: 262
speed:
value: 0
unit: 0
accuracy:
hdop: 0
pdop: 0
time: '20221227184211'
vehicleStatus:
airCtrlOn: false
engine: false
doorLock: true
doorOpen:
frontLeft: 0
frontRight: 0
backLeft: 0
backRight: 0
trunkOpen: false
airTemp:
value: 02H
unit: 0
hvacTempType: 0
defrost: false
acc: false
evStatus:
batteryCharge: false
batteryStatus: 61
batteryPlugin: 0
remainTime2:
etc1:
value: 34
unit: 1
etc2:
value: 560
unit: 1
etc3:
value: 170
unit: 1
atc:
value: 170
unit: 1
drvDistance:

rangeByFuel:
evModeRange:
value:<PHONE_NUMBER>
unit: 3
totalAvailableRange:
value:<PHONE_NUMBER>
unit: 3
type: 2
reservChargeInfos:
reservChargeInfo:
reservChargeInfoDetail:
reservInfo:
day:
1
time:
time: '0540'
timeSection: 0
reservChargeSet: false
reservFatcSet:
defrost: true
airTemp:
value: 0AH
unit: 0
hvacTempType: 0
airCtrl: 0
heating1: 0
offpeakPowerInfo:
offPeakPowerTime1:
starttime:
time: '1200'
timeSection: 0
endtime:
time: '1200'
timeSection: 0
offPeakPowerFlag: 0
reserveChargeInfo2:
reservChargeInfoDetail:
reservInfo:
day:
2
3
4
5
time:
time: '0740'
timeSection: 0
reservChargeSet: false
reservFatcSet:
defrost: true
airTemp:
value: 0AH
unit: 0
hvacTempType: 0
airCtrl: 0
heating1: 0
reservFlag: 0
ect:
start:
day: 9
time:
time: '1200'
timeSection: 0
end:
day: 9
time:
time: '1200'
timeSection: 0
targetSOClist:
targetSOClevel: 80
plugType: 0
targetSOClevel: 100
plugType: 1
ign3: false
hoodOpen: false
transCond: true
steerWheelHeat: 0
sideBackWindowHeat: 0
tirePressureLamp:
tirePressureLampAll: 0
tirePressureLampFL: 0
tirePressureLampFR: 0
tirePressureLampRL: 0
tirePressureLampRR: 0
battery:
batSoc: 81
batState: 0
sleepModeCheck: true
time: '20221227184213'
remoteWaitingTimeAlert:
remoteControlAvailable: 1
remoteControlWaitingTime: 168
elapsedTime: '00:03:39'
systemCutOffAlert: 0
tailLampStatus: 0
hazardStatus: 0
odometer:
value: 0
unit: 1
Vehicle name
IONIQ EV

I have changed the units to miles, from km, but even before that there were too many digits precision.

@ZuinigeRijder if I change this in the API would this impact you?  It would be easier than doing the home assistant side and not sold this level of precision is useful to anyone else who may consume the api. I wouldn't think so.

@ZuinigeRijder if I change this in the API would this impact you? It would be easier than doing the home assistant side and not sold this level of precision is useful to anyone else who may consume the api. I wouldn't think so.

My tool will not be impacted when the precision in km is only 1 digit. However, changing from km to miles would impact my tool.

@ZuinigeRijder if I change this in the API would this impact you? It would be easier than doing the home assistant side and not sold this level of precision is useful to anyone else who may consume the api. I wouldn't think so.

My tool will not be impacted when the precision in km is only 1 digit. However, changing from km to miles would impact my tool.

Thanks for the quick reply.   I will start rounding these values to 2 decimal places.
The km / miles item is a home assistant thing.   As a note for your tool I don't control units the car does.   The API however does have a _unit attribute which shows if km or miles.  In home assistant we injest that and can auto convert to a different unit based on settings.

https://github.com/Hyundai-Kia-Connect/hyundai_kia_connect_api/pull/221 has been merged for this.
@grahamrjuk are you able to update in HACS by clicking redownload and select version of "master".  Let me know if everything looks okay on the rounding afterward and I can close this out.

Works nicely for me. Two decimal places is probably more than necessary, but much more sensible than what I had before.
I can template in Home Assistant if I want to do anything different.

What would be your ideal count? Trying to find the balance on detail. I could see has taking this down to less or no decimals.

What would be your ideal count? Trying to find the balance on detail. I could see has taking this down to less or no decimals.

For km the odometer is sending it in 100 meters, so 1 digit. So at least 1 digit after the decimal point is needed for km.
Examples:

_odometer_value=20795.4
'odometer': {'value': 20795.4, 'unit': 1}

Not sure there is an ideal. For these measurements no decimals are needed. But two decimal places is quite reasonable for distances in general. I would be happy to leave it as it is now - or go to 1 decimal place as commented above, to be consistent with other measurements.

I dropped to one considering that makes it match odometer.    Thanks for the feedback.


In Panel having data as a scalar in the constructor does not work

When you want to initialise a DF to a particular value you can use data = 0.. But this does not work in Panel

dupe of #8285
thanks for reporting - pull-requests to fix are welcome


Our neat russian projects

https://github.com/yandex/yandex-tank
https://github.com/yandex/ClickHouse
https://github.com/yandex/pandora

Cool! Feel free to open a pull request and add them! ✨

This list seem like an auto-generated. Isn't it?

Correct, but you can contribute too.
Sent from my iPhone

On 21 Sep 2017, at 15:39, Alexander Artemenko<EMAIL_ADDRESS>wrote:
This list seem like an auto-generated. Isn't it?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

https://github.com/pyos/re2jit - a jit compiler for re2, adding backrefs and speeding up the unicode matching. C++
https://github.com/pyos/libcno - a neat io-abstract http/2 implementation. C

Amazing, but please open pull requests! 😄

I merged #9. Feel free to open pull requests with new projects! 👍


Site Request for Bigo.tv

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.11.23. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.11.23

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[x] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
youtube-dl.exe http://www.bigo.tv/91604999
[generic] 91604999: Requesting header
WARNING: Falling back on generic information extractor.
[generic] 91604999: Downloading webpage
[generic] 91604999: Extracting information
[download] Downloading playlist: BIGO LIVE
[generic] playlist BIGO LIVE: Collected 1 video ids (downloading 1 of them)
[download] Downloading video 1 of 1
[download] Got server HTTP error: HTTP Error 500: Internal Server Error. Retrying (attempt 1 of 10)...
[download] Got server HTTP error: HTTP Error 500: Internal Server Error. Retrying (attempt 2 of 10)...
[download] Got server HTTP error: HTTP Error 500: Internal Server Error. Retrying (attempt 3 of 10)...
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
Just recording video if the person has online, youtube dl would support this? Example on twitch.tv I can record the livestream.
Thank you!

I can't actually manage to get the website to work for me, every video looks like this:

I can't actually manage to get the website to work for me, every video looks like this:

Strange...here it's normal web page.


Allow ignoring some branches via action inputs

@rheaditi what about that weird space issue? are we not solving it?

what about that weird space issue? are we not solving it?

@rheaditi this issues is still there


Update README.md

Fix image file location (from changes in https://github.com/slochower/pAPRika/pull/140)
Update installation instructions (AmberTools 20 is now in conda-forge)
Added license badge

Codecov Report

Merging #144 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

Looks good to me. Merging...


Moran_Local's EI is returned as an array instead of a float

I am getting an array from this line instead of a float, as the docs suggest:
self.Is = self.calc(self.w, self.z)

After I get the returned value from lmr = Moran_Local_Rate(...), the length of lmr.EI is equal to lmr.n. Are these just the local values instead of the global one?

Yes, these are the local values so we need to correct the docs to indicate array of floats.

Closing as this is fixed in PR #741 by @sjsrey


docs: Remove call to groupadd

Adding the user to the docker group using groupadd(8) may not be a good
idea from a security perspective, so continue to use sudo(8) in all docs
to access docker.
Fixes #513.
Signed-off-by: James O. D. Hunt<EMAIL_ADDRESS>

Hi @iphutch - please could you review this one-liner? ;)

Coverage remained the same at 89.402% when pulling c252ad90d2abe3a80867c2f1a9d2625aff2b5816 on jodh-intel:docs-remove-groupadd-call into 06023ea1b0961e66801bc9f51bced9777f53a00f on clearcontainers:master.

Thanks @jodh-intel 👍

lgtm


Improve user facing error message when saving a non-binary image.

Also changed the comment on the isBinary method, because it's only checking if the image is binary, not if it can be converted to binary.

Tadeas, thank you for fixing the detect problem !


Update to 1.0.7 and use cos7 host

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Closes https://github.com/conda-forge/r-sf-feedstock/pull/71

https://github.com/conda-forge/r-sf-feedstock/pull/71 fails because it looks for things not available in cos6, e.g.
$BUILD_PREFIX/bin/../lib/gcc/x86_64-conda-linux-gnu/10.3.0/../../../../x86_64-conda-linux-gnu/bin/ld: $PREFIX/lib/libgdal.so: undefined reference to<EMAIL_ADDRESS>configure: error: GDALAllRegister not found in libgdal.
$BUILD_PREFIX/bin/../lib/gcc/x86_64-conda-linux-gnu/10.3.0/../../../../x86_64-conda-linux-gnu/bin/ld: $PREFIX/lib/libgdal.so: undefined reference to<EMAIL_ADDRESS>collect2: error: ld returned 1 exit status
$BUILD_PREFIX/bin/../lib/gcc/x86_64-conda-linux-gnu/10.3.0/../../../../x86_64-conda-linux-gnu/bin/ld: $PREFIX/lib/libgdal.so: undefined reference to<EMAIL_ADDRESS>$BUILD_PREFIX/bin/../lib/gcc/x86_64-conda-linux-gnu/10.3.0/../../../../x86_64-conda-linux-gnu/bin/ld: $PREFIX/lib/libgdal.so: undefined reference to<EMAIL_ADDRESS>
This PR just takes the bot-issued update PR and sets os_version to  linux_64: cos7 in conda-forge.yaml

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Ready for review @conda-forge/r
This is just a fixed-up #71


Crafting rescheduler

if it has no started tasks and the rest can't start then the task should be recalculated and rescheduled.


fix for #312 added checksum annotation to console deployment

Added the config checksum annotation ... it works for my use cases, changing the brokers and schema registry url(s) trigger pod restart.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

done

Can you rebase this, please, so the tests will pass? I would do it for you but that checkbox wasn't checked.

Well rats... It worked locally. Thanks!


Added border to status bar to match design spec

@winston-de sorry about all the commits! This is ready for review now.


Use nested data problem

Is it possible to process my keys in moment? Need this for process nested data.
I have template vars like this:
{user}
{user.name}
{user.age}

And data:
user: {
 name: 'Alex',
 age: 23,
 country: 'Ukraine'
}

And want result:
Alex, 23, Ukraine
Alex
23

I see three interesting features to resolve issue:
first: add .toString method to object (user), that call when  use {user} key in template (if object has toString, than ObjectCommandResultError not appear)
second: add functions aliases, that calls additionalJsContext functions. like if i put {{{user}}}, than i get result of additionalJsContext.getByKey(key)
third: if typeof data === 'function', that function receive 'key' parameter, and than can process it async and return result
At this moment i use this construction, but its to scared :DD.
Template:
{v(`user`)}
{v(`user.name`)}
{v(`user.age`)}

Code:
additionalJsContext: {
      v: async key => {
          let fieldValue = calculateFieldValue(key, documentData)
          return fieldValue
      },
  },

And again as conclusion of 2 global problems:

i cant use object string representation and object nested fields at one time.
i must to know about all fields in template before render it. But i want to get needed data in moment, where i get key, and then put it into template

To see if I understand your question correctly, you have this data
user: {
 name: 'Alex',
 age: 23,
 country: 'Ukraine'
}

And you want this command in your template
{user}

To result in
Alex, 23, Ukraine

Is that correct?
Assuming I understood correctly:
That isn't supported, because there isn't a single agreed-upon way to serialise an Object to a string.
You want it to look like result of Object.values(user).join(', '), so I suggest you just use this.
Let me know if I misunderstood anything.

No, join() is not allowed. I use {user} as example for reproduce similiar task. My real problem - is ability to process each template keys directly before inserting.

You can use the listCommands() function to list all commands in a template. See https://github.com/guigrpa/docx-templates#inspecting-templates
Is that what you mean?
Note that template commands can contain a JS code snippet of which the result will be inserted into the rendered report.
Putting the below inside your template
{Object.values(user).join(', ')}

will produce
Alex, 23, Ukraine

You can use the listCommands() function to list all commands in a template. See https://github.com/guigrpa/docx-templates#inspecting-templates
Is that what you mean?
Note that template commands can contain a JS code snippet of which the result will be inserted into the rendered report.
Putting the below inside your template
{Object.values(user).join(', ')}

will produce
Alex, 23, Ukraine

Yes, when i use listCommands(), i get it, but i cant process it step by step inserting data.. :)

Template commands can contain arbitrary JS code, meaning they are very flexible. I still don't understand what exactly it is that you want to do that isn't possible yet.

I'm also not shure what you want to do but with a Proxy you might be able to accive what you want - as you want to transform your data on the fly. But I would think transforming it before calling createReport is much simpler.

Hope this helped. If not, feel free to reopen.


[BEAM-1619] Write to the existing View in Dataflow Streaming Side Inputs

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[x] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

This updates Streaming View Overrides to use the original view when they override a transform
by causing the override to only apply to the View.CreatePCollectionView, inserting a global combine
and view write prior to the primitive which creates the view.
There are also updates to how the Dataflow Runner signifies that a Map or Multimap view has a nondeterminstic key coder, as well as an additional test for CombineGloballyAsSingletonView.

R: @kennknowles
CC: @aaltay @reuvenlax @dpmills

Run Dataflow RunnableOnService

Actually, streaming doesn't have coverage :-(

Coverage increased (+0.2%) to 70.059% when pulling 31928af28bde6c5212c508372c18b05803962e22 on tgroh:streaming_single_override into<PHONE_NUMBER>bda5aaf34a7c176e11c2441d0a7346 on apache:master.

Coverage increased (+0.2%) to 70.053% when pulling bd38ad20ad21fc66f70aa6d25d2a71f8b122499d on tgroh:streaming_single_override into<PHONE_NUMBER>bda5aaf34a7c176e11c2441d0a7346 on apache:master.

LGTM. Spark test failures seem to be a common flake right now.

Confirmed via chat that streaming tests have been run manually since we don't have them connected to Jenkins.


Updated Fibonacci.py using recursion

Description
Solved the Fibonacci Sequence Generator Code using recursion. The code also contains the documentation for the same.
Fixes #1508
Type of change

[x] New feature
[x] Documentation Update

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My documentation follows Template for README.md

Project Metadata
Category:

[x] Fun
[x] Others

Title: Fibonacci
Folder: Fibonacci_Sequence_Generator
Requirements: None
Script: python Fibonacci.py
Contributor: DevBhojani116
Description: Generates the n terms of Fibonacci Series where n is input by the user.

@avinashkranjan can you please review the pr


Credential file override flag support

fixes #41

LGTM

Want to merge in the builder first and I'll update this?

Sure, let me do that.

Updated to use builder.

:ship:


Uploading error

handler.php not check for $_FILES['qqfile']['error']
If error occurrence, FineUuploader continue with next chunk.
Something like this bellow, need near line # 138 in handler.php
if($file['error']) {
return array('error' => 'Upload Error #'.$file['error']);
}

When will Fine Uploader send such a parameter with the request? I am not aware of one.

PHP set erorr if file comes wrong. During upload if network drops some packet and file come to server wrong . Anyway, i have from time to time error =3
(from PHP documentation:  UPLOAD_ERR_PARTIAL Value: 3; The uploaded file was only partially uploaded.)

Ah, I see. Can you open up a pull request with the proposed change? I will take a look, and it all looks well, I'll publish an updated version to composer/packagist.

Fixed in #12.


Pages that aren't in the menu in version 2.0.6 don't get transferred when updating to 2.1.0

There's an issue where someone updates WonderCMS under the following scenario:

user has pages that are not in the menu in version 2.0.6
when updating to 2.1.0 (which we're working on), those pages are gone - the pages that weren't in the menu in users version 2.0.6 were not transferred to version 2.1.0
other pages (those that were in the menu) were transferred successfully

Would a change in your update script that would foreach through pages instead of menu items work?

Yes of course! It could be something like this (UNTESTED):
foreach ($olddb->pages as $pages) {
                $newMenu->{$i} = new stdClass;
                $newMenu->{$i}->name = $pages->title;
                $newMenu->{$i}->slug = wCMS::_slugify($pages->title);
                $newMenu->{$i}->visibility = "show";
                $i++;
            }

Be aware, that all pages are visible in the menu after this update procedure. So the admin has to hide the unwanted menu items after updating wonderCMS.

Completely understood and thank you. Putting this up for a test and reporting back. Just to check, unsetting doesn't need changing, correct? unset($olddb->config->menuItems);$olddb->config->menuItems = $newMenu;$olddb->config->dbVersion = '2.1.0';wCMS::save($olddb);

Yes, the unset and the other parts are essential to perform the new menu update

Was just checking if anything needed changing there, thanks :+1:.
Coming back with some short feedback about the update functionality: the pages don't seem to have transferred (pages that weren't in the menu).
Pages from database.js before updating to 2.1.0 (only home, example and test were in the menu)
    "pages": {
        "home": {
            "title": "Home",
            "keywords": "Keywords, are, good, for, search, engines",
            "description": "A short description is also good.",
            "content": "
It's alive!<\/h2>
Your website is now powered by WonderCMS.<\/p>
Click here to login, the password is admin<\/b>.<\/a><\/p>
Simply click on content to edit and click outside to save it.<\/p>"
        },
        "example": {
            "title": "Example",
            "keywords": "Keywords, are, good, for, search, engines",
            "description": "A short description is also good.",
            "content": "
Example page<\/h2>
How to add new pages<\/p>
Open the settings panel, click on the existing pages in the menu and enter a new page.<\/p>"
        },
        "1234t": {
            "title": "1234T",
            "keywords": "Keywords, are, good, for, search, engines",
            "description": "A short description is also good.",
            "content": "
Click here to create some content<\/h2>
Once you do that, this page will be eventually visited by search engines.<\/p>"
        },
        "šok": {
            "title": "šok",
            "keywords": "Keywords, are, good, for, search, engines",
            "description": "A short description is also good.",
            "content": "
Click here to create some content<\/h2>
Once you do that, this page will be eventually visited by search engines.<\/p>"
        },
        "pok": {
            "title": "Pok",
            "keywords": "Keywords, are, good, for, search, engines",
            "description": "A short description is also good.",
            "content": "
Click here to create some content<\/h2>
Once you do that, this page will be eventually visited by search engines.<\/p>"
        },
        "test": {
            "title": "Test",
            "keywords": "Keywords, are, good, for, search, engines",
            "description": "A short description is also good.",
            "content": "
Click here to create some content<\/h2>
Once you do that, this page will be eventually visited by search engines.<\/p>"
        }

Menu items and pages after updating. The pages are still there, but not transferred to the menu.
"menuItems": {
            "0": {
                "name": "home",
                "slug": "home",
                "visibility": "show"
            },
            "1": {
                "name": "example",
                "slug": "example",
                "visibility": "show"
            },
            "2": {
                "name": "test",
                "slug": "test",
                "visibility": "show"
            }
        }
    },
    "pages": {
        "home": {
            "title": "Home",
            "keywords": "Keywords, are, good, for, search, engines",
            "description": "A short description is also good.",
            "content": "
It's alive!<\/h2>
Your website is now powered by WonderCMS.<\/p>
Click here to login, the password is admin<\/b>.<\/a><\/p>
Simply click on content to edit and click outside to save it.<\/p>"
        },
        "example": {
            "title": "Example",
            "keywords": "Keywords, are, good, for, search, engines",
            "description": "A short description is also good.",
            "content": "
Example page<\/h2>
How to add new pages<\/p>
Open the settings panel, click on the existing pages in the menu and enter a new page.<\/p>"
        },
        "1234t": {
            "title": "1234T",
            "keywords": "Keywords, are, good, for, search, engines",
            "description": "A short description is also good.",
            "content": "
Click here to create some content<\/h2>
Once you do that, this page will be eventually visited by search engines.<\/p>"
        },
        "šok": {
            "title": "šok",
            "keywords": "Keywords, are, good, for, search, engines",
            "description": "A short description is also good.",
            "content": "
Click here to create some content<\/h2>
Once you do that, this page will be eventually visited by search engines.<\/p>"
        },
        "pok": {
            "title": "Pok",
            "keywords": "Keywords, are, good, for, search, engines",
            "description": "A short description is also good.",
            "content": "
Click here to create some content<\/h2>
Once you do that, this page will be eventually visited by search engines.<\/p>"
        },
        "test": {
            "title": "Test",
            "keywords": "Keywords, are, good, for, search, engines",
            "description": "A short description is also good.",
            "content": "
Click here to create some content<\/h2>
Once you do that, this page will be eventually visited by search engines.<\/p>"
        }

(we're back to ghost haunting pages :grin: )
I'll try to figure out if there's anything I can do to make this work.

Attaching the latest index.php below with added JSON_UNESCAPED_UNICODE to prevent converting of some characters ({"a":"\u00e1"} {"a":"á"}
index.php.zip
(without extended slugifier, without the search engines part).
This seems to be ready for production (2.1.0). Before we push it out and, I'm testing out additional scenarios and trying to make some of the code shorter - I have a question regarding this:
(line 229 - is encoding and decoding necessary here?
if ($menu) {$menuItems = json_decode(json_encode(wCMS::get('config', 'menuItems')), TRUE);
(and line 230 - same question about enconding and decoding)
$db->config->menuItems = json_decode(json_encode($newMenu))

Another update to the index.php, this version has multiple encoding fixes (some of them were also ghost pages with characters like &scaron instead of š.
This was fix by using html_entities_decode after calling json_decode. This now correctly retains all characters as they where, without any loss.
Will take this for a couple of more spins and tests, I will also send it to turboblack since he's an awesome tester.
@wdj-ac I'd like to take this chance to thank you once again. Rest assure you'll be properly publicly credited and once we get some more donations you and Robbie and first on the list to get rewarded.
Latest index.php // all special characters retaining solved + also perfectly retained without any loss (characters were converted to &scaron or /u190)
index.php.zip
What do you think @wdj-ac?


Keyboard manager stops working

Microsoft PowerToys version
0.70.1
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
Keyboard Manager
Steps to reproduce
Some shortcut keys stop working randomly, then continues working back at another random time.
Shortcut:

Atl Space to Win S (All apps)
Shift Backspace to Ctrl Backspace (All apps)
Note: When powerToys is restarted, it works again for about a minute or two then stops working, while other shortcuts continue working.

✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
No response

I have this issue too, please someone fix it.

I have this issue too, please someone fix it.

can you verify if this is still an issue with v0.73.0 /needinfo

I can confirm it's still an issue on the latest version v0.73.0

can you verify if this is still an issue with v0.73.0 /needinfo

I can confirm it's still an issue on the latest version v0.73.0

Combining similar keyboard issues
/dup #17344


Text doesn't render at all.

Neither text nor textflow elements are rendered. The latter renders as a blackbox.
drawing-3.txt

Seems to be a problem with the viewBox in combination with the width/height in mm - the rendering is done outside the viewbox.

As far as I was confirmed, it seems to be rendered correctly.
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
<svg
   xmlns:dc="http://purl.org/dc/elements/1.1/"
   xmlns:cc="http://creativecommons.org/ns#"
   xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#"
   xmlns:svg="http://www.w3.org/2000/svg"
   xmlns="http://www.w3.org/2000/svg"
   id="svg8"
   version="1.1"
   viewBox="0 0 210 297"
   height="297mm"
   width="210mm">
  <defs
     id="defs2" />
  <metadata
     id="metadata5">
    <rdf:RDF>
      <cc:Work
         rdf:about="">
        <dc:format>image/svg+xml</dc:format>
        <dc:type
           rdf:resource="http://purl.org/dc/dcmitype/StillImage" />
        <dc:title></dc:title>
      </cc:Work>
    </rdf:RDF>
  </metadata>
  <!-- add for check. -->
  <rect x="0mm" y="0mm" width="210mm" height="297mm" fill="lightgray" />
  <g
     id="layer1">
    <text
       id="text4536"
       y="27.301525"
       x="19.886719"
       style="font-style:normal;font-weight:normal;font-size:10.58333302px;line-height:1.25;font-family:sans-serif;letter-spacing:0px;word-spacing:0px;fill:#000000;fill-opacity:1;stroke:none"
       xml:space="preserve"><tspan
         y="27.301525"
         x="19.886719"
         id="tspan4534">Why does this text not render?</tspan></text>
  </g>
</svg>

@cmoya212b
Please tell me issue occurs runtime of details(environment).

@H1Gdev Sorry I posted the wrong file... that I got working by manually replacing flowPara with text element. Here is the Inkscape generated one that does not work.
drawing-4.txt
(Note this was saved as "Plain SVG" not Inkscape SVG)

I suppose this isnt exactly a bug since many utilities don't even support flowRoot/flowText etc.

I suppose this isnt exactly a bug since many utilities don't even support flowRoot/flowText etc.

This library does not support flowed text.
Use SVG text as described here.

Thank you @H1Gdev for working this through and pointing out the docs.


Update service-premium-per-user-faq.yml

Paginated Report is now available for Power BI Pro, so removed here because here mentioned as PPU or Premium capacity only feature

@kesemsharabi
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Add functions to bucket via api

I would like to automate the creation of a bucket with the association of one or more "Restless Functions" of noobaa. How can I do? Are there any noobaa api that can be used?
Thanks

Hi @sberequek
NooBaa serves both S3 and Lambda API's so you can create a bucket, and create a lambda function from the AWS.S3/AWS.Lambda from your SDK of choice.
The only gap is to link between the bucket and a function.
This is an API that currently is not implemented in NooBaa called BucketNotification.
See https://docs.aws.amazon.com/AmazonS3/latest/API/RESTBucketPUTnotification.html
We certainly see this API being added to NooBaa and would love to cooperate on that.
In the meanwhile, you can use our internal API (feeling brave?) to achieve the same goal
This Wiki will give you the basic instructions on how to start using the API:
https://github.com/noobaa/noobaa-core/wiki/RPC
To set the bucket to trigger functions you can use the bucket_api add_bucket_lambda_trigger() defined in a json schema here:
https://github.com/noobaa/noobaa-core/blob/b878c9607359ada87fc32c275da2bf9308692bcc/src/api/bucket_api.js#L646-L650
https://github.com/noobaa/noobaa-core/blob/b878c9607359ada87fc32c275da2bf9308692bcc/src/api/bucket_api.js#L1043-L1053
Calling it from cURL will be something like this
(Note you will need to take the NOOBAA_MGMT http/s service address and also authenticate and get a jwt token as NOOBAA_TOKEN as explained in the RPC Wiki)
curl $NOOBAA_MGMT/rpc/ -sd '{
  "api": "bucket_api",
  "method": "add_bucket_lambda_trigger",
  "params": { "bucket_name": "XXX", "func_name": "YYY", "event_name": "ObjectCreated" },
  "auth_token": "'$NOOBAA_TOKEN'"
}'

Let me know if that makes sense.
Guy

BTW what did you think of our geeky name "Restless Functions"?

Hi @guymguym
I'm trying to use the internal API https://github.com/noobaa/noobaa-core/wiki/RPC but I can't get over the "Create System" or the next call "Auth Token". In detail the answer message is "RPC VERSION MAGIC MISMATCH". The call I am using for "Create System" is as follows:
 curl https://{MY_INSTANCE_NOOBAA_MGMT}:8443/rpc/ -sd '{
  "api": "system_api",
  "method": "create_system",
  "params": {
    "name": "demo",
    "email": "{MY_USERNAME}",
    "password": "{MY_PASSWORD}",
    "activation_code": "123"
  }
}

I honestly don't know how to set the "activation_code" field, suggestions? If it can be useful I am using NooBaa version 2.10.4 - Deployment ESX.
Thanks
Ps The name "Restless Functions" is really cool :)

Hi @sberequek !
OK so I am sorry for the confusion, I assumed you were using the 3.0.0 release for Kubernetes, but in the 2.x versions, we did not expose the internal RPC API through http yet, so the error you get is expected though unfortunate (also the activation_code was dropped since then).
Going forward we are focusing on Kubernetes as a platform.
Does that make sense for you to use?
If so, you can follow the deployment guide on the README
I added a terminal recording that might give you a preview of how the deployment process goes now -

Let me know what you think

Thanks @guymguym
I will follow your advice.

Great @sberequek. Let me know how it goes.


Test project set creation in Forge after the CI/CD GitHub Pipelines implementation

Describe the Issue
As a DevOps user I want the creation of new project set will proceed without any hiccup.
This ensure this seamless behaviour,  the new procedure needs to be tested and documented
This ticket is to modify the existing project set creation and, if appropriate, develops new test cases that will ensure the new procedure works as expected.
Additional Context
Currently there is already a test case for the project set creation, however will need to be modified to reflect both the OIDC authentication and the use of the CI/CD github actions pipeline
Acceptance Criteria

Test the creation of new project set in Forge and Assembly
Modify the existing test cases to reflect the changes
Update the documentation (How to) to reflect the changes
If appropriate create new test cases to verify other aspects of the procedure
Create a new ticket for testing the project set creation in Live

After the bug was resolved, I was able to successfully create the project set tk72yo with the alb (it was the layer that was failing)
This projet set has a single account: tk72yo-dev


Add support for numpy numbers to be json serializable

The problem was discovered by running
https://github.com/combust/mleap-demo/blob/master/notebooks/airbnb-price-regression-scikit.ipynb
During a bundle serialization an object was constructed
to describe a model. This object for some reason contained numpy numbers
instead of python native integers.
Therefore it was not json serializable cause the type was not recognized
This patch adds support for numpy numbers for json serializer

@trams could you please give a bit more details on which transformer or part of the pipeline was having this issue for the airbnb example? thank you!

@ancasarb @trams Is there an update on this? Should we merge this in?

@hollinwilkins I was just trying to understand what the issue that's being fixed is. as far as i am aware, we can serialize and score the airbnb without any issues.

Closing this, due to inactivity, please re-open if it's still an issue.


Fix link in README.md

not sure if it was on purpose or not

Oh this was actually intended to be the hyperlink for the link in the main description line, but maybe the newline there is throwing it off?

I didn't know those kind of links existed in github's markdown.
It seems it was caused by a newline but because of a missing one. I updated the PR.

Thanks!


Parseint fails with radix values 33 to 36

ParseInt is specced to work with radix values 2 through 36, Line 53 of jsbuiltin.c allows only 2 through 32.

Oops. Fixed.


Check whether CI works or not

DO NOT MERGE.
I just want to check if the newly added CI works

:tada:  Awesome !!


add OWNERS for resource-usage-collect-addon

/assign @qiujian16
/cc @z1ens

@qiujian16 seems @z1ens is not the member of ocm yet, any process to follow to join?

invitation sent.

invitation sent.

Thank you!

/verify-owners

/approve
/lgtm


Get own value in "default" method

Hello I would like to make a constrain for an attribute, e.g. #role can be  either :admin, :manager or :user only, and it must be set to the :user in any other case.
attribute :role,  Symbol,  default: ->(user, role) { [:admin, :manager].include? role ? role : :user }

I think this is beyond what it means to have a "default value", this is really a kind of data validation, which I think is beyond the scope of Virtus. I'm not the maintainer of this project but I'd say you should either just override the setter method or find/build another library to do this kind of thing.


pulumi: 3.55.0 -> 3.56.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/pulumi/pulumi/releases.
meta.description for pulumi is: Pulumi is a cloud development platform that makes creating cloud programs easy and productive
meta.homepage for pulumi is: https://pulumi.io/
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.56.0 with grep in /nix/store/ggwlnrgiaa08rd407syxknpk0s9mhbvx-pulumi-3.56.0
found 3.56.0 in filename of file in /nix/store/ggwlnrgiaa08rd407syxknpk0s9mhbvx-pulumi-3.56.0

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

11 total rebuild path(s)

11 package rebuild(s)

First fifty rebuilds by attrpath
pulumi
pulumiPackages.pulumi-language-nodejs
pulumiPackages.pulumi-language-python
python310Packages.pulumi
python310Packages.pulumi-aws
python310Packages.pulumi-aws-native
python310Packages.pulumi-azure-native
python310Packages.pulumi-command
python310Packages.pulumi-random
python311Packages.pulumi
python311Packages.pulumi-aws

Instructions to test this update (click to expand)

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

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

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

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

10 packages failed to build:

pulumiPackages.pulumi-language-nodejs
pulumiPackages.pulumi-language-python
python310Packages.pulumi
python310Packages.pulumi-aws
python310Packages.pulumi-aws-native
python310Packages.pulumi-azure-native
python310Packages.pulumi-command
python310Packages.pulumi-random
python311Packages.pulumi
python311Packages.pulumi-aws

1 package built:

pulumi

Maintainer pings
cc @Trundle @veehaitch for testing.

See #219943


[wip] initial commit to implement payment simulation on LN

Template to include the simulation of payments on the ChannelGraph, extending the previous single period model to a multi period model.
Not included is the Channel Graph "listchannels20220412.json", residing in repo root.

closed, would prefer to open new branch.


Make bpftrace work correctly inside containers with PID namespacing

This has been a long time coming - I presented it at eBPF Summit 2023 - but it's finally ready!
This fixes pid, tid and ustack when running bpftrace inside a PID namespace. Previously, these builtins would return PIDs as-seen from the init (global/root) namespace, but this meant that they did not correspond to the correct processes from the bpftrace runtime's point-of-view, as it would see namespaced PIDs.
The core of the PR is covered by this commit:
Codegen: Get correct PIDs/TIDs relative to bpftrace's PID namespace

The bpf_get_current_pid_tgid helper only returns PIDs in the root namespace.

The bpf_get_ns_current_pid_tgid helper only returns PIDs in a given
namespace for processes running in that same namespace.

We must switch between the two helpers based on where bpftrace is
running in order to cover the most scenarios.

The following tables show whether we are able to get the correct PID for
a target process using each helper, when bpftrace and the target are and
are-not in PID namespaces.

Always using bpf_get_current_pid_tgid (old behaviour):
        \ target |                |
bpftrace \       | root namespace | child namespace
-----------------+----------------+-----------------
root namespace   | YES            | YES
child namespace  | N/A            | NO

Always using bpf_get_ns_current_pid_tgid:
        \ target |                |
bpftrace \       | root namespace | child namespace
-----------------+----------------+-----------------
root namespace   | YES            | NO
child namespace  | N/A            | YES

Switching between the two helpers (new behaviour):
        \ target |                |
bpftrace \       | root namespace | child namespace
-----------------+----------------+-----------------
root namespace   | YES            | YES
child namespace  | N/A            | YES

This approach is still not complete, however. If the target is running
in a namespace, and bpftrace is running in a different namespace which
still has visibility over the target's namespace, we will not get the
correct PID for the target. This would require kernel changes to work.

Checklist

~[ ]~ Language changes are updated in man/adoc/bpftrace.adoc
[x] User-visible and non-trivial changes updated in CHANGELOG.md
[x] The new behaviour is covered by tests

Bump - review please :)


@cucumber/react: Reporting for retries

Is your feature request related to a problem? Please describe.
Cucumber report(html, json formatter) doesn't include retried details when using --retry.
When we use --retry, report will mark the flaky test pass if there is one retry get pass. That is working as expected. However, if we can also add some details of retry in the report, e.g show the number of failed attempt, that will help us to find potential intermittent bugs, instead of hiding bugs.
The idea is coming from https://automationpanda.com/2021/06/14/are-automated-test-retries-good-or-bad/
Describe the solution you'd like
Something on the report to indicate there are retry attempts happened. Like the number of failed attempt. A example:

Noting here that @cucumber/query will likely need some new smarts to handle this. Currently it always gives you data from the last attempt of any given pickle or test case, but it'll need to be able to list attempts and get data for a given attempt.


App/Service: Implement an empty foreground service

This patch adds an empty service that runs in the foreground to the ML Inference Offloading App. The Service is launched by the
startForegroundService method in the onCreate callback of the Activity.
Signed-off-by: Wook Song<EMAIL_ADDRESS>

LGTM :)

After verifying this PR on your machine, please merge it. THX:)


Add more SDR record types

Creating as a draft PR as there is still some work to do.
Questions:

'event-only' sensors are missing some of the fields in SensorRecordCommon. There are a few options for handling this.

remove those fields from common
make them optional in common
break common into two types, one for Full+Compact, and one for all three

Still todo:

tidy up interfaces to be consistent
make parse methods return Result<> instead of Option<>
refactor out duplicated common parsing code

'event-only' sensors are missing some of the fields in SensorRecordCommon

I think removing the fields from SensorRecordCommon makes the most sense. Since the idea is really that it can identify/be used for any 'ole sensor, it shouldn't include fields that aren't actually common to all sensor records.
We could put some of the common-to-full-and-compact-sensors things in another struct, but since that's only two or three fields I propose that we just add those fields to the structs for now.

Figured I would check in to see if you are making progress on this/would like to finish this (eventually), or if you're OK with me making some of the changes and merging it in in some form?
This functionality is super nice, it adds a super nice "completeness" feeling to the get-info example :P

We’re actually running these commits in our fork pretty-much as is.
I will definitely get back to this eventually, but at this stage I’m not sure when that will be, so if you’d like to run with this that’s cool.


Missing buttons

The buttons "Unpack file" and "Repack file" are missing but are present in the tutorial video.

This happens because the monitor resolution does not match,
im use full hd monitor,
can use this for a while
https://drive.google.com/file/d/1bQskgElvqmvM5Fp0bLWH0bMm81ClTuSl/view?usp=sharing
On Mon, 29 Nov 2021, 01:00 luckymouse0, @.***> wrote:

The buttons "Unpack file" and "Repack file" are missing but are present in
the tutorial video.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/gil-unx/FRONTIER-GATE-BOOST-PLUS/issues/3, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ANY47SFOVECSEMB3PGX53T3UOJU2DANCNFSM5I5RNHAA
.

Thanks that works! Can you tell me where in the source code are these buttons? I don't see them in the forms designer:

Thanks that works! Can you tell me where in the source code are these buttons? I don't see them in the forms designer:

130,185


UI Issue: Text Unreadable in Dark Mode.

The Stream name is often unreadable in the dark mode due to its lack of contrast from the stream color.
The dark grey text is very hard to see when the stream has say yellow color.

@zulipbot claim

@zulipbot claim --force

@zulipbot claim

@zulipbot claim --force

This one is fixed already. Closing.


Dart SDK docker image 3.4 stable unavailable

https://hub.docker.com/_/dart

https://github.com/docker-library/official-images/pull/16769#event-12806918700


improve error message for failed import

Description:
This error message  should be reworded for clarity and grammar:your ballerina version does not support for package: wso2/twitter:*. following versions are supported: 0.9.13,0.9.14
Steps to reproduce:
I have a ballerina program with a import wso2/twitter..
Affected Versions:
Ballerina build 0.975.
OS, DB, other environment details and versions:
Mac OS.
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Central does not know the users ballerina version when pulling a package. Central only knows the balo range of the user's ballerina version. Hence we could reword as following unless the message can be modified from the CLI:
package 'wso2/twitter:*' does not support your ballerina version. following are the 'wso2/twitter' package versions compatible with your ballerina version: '0.9.13, 0.9.14'

maybe: The package 'wso2/twitter' <version number?> is not compatible with your Ballerina compiler. Try one of these package versions instead: '<versions>'.

+1. Thanks for the suggestion @rabbah


BAL Hookathon - MEV Tax Hook

The MEV Tax Hook allows Balancer pools to capture their own MEV by charging a dynamic swap fee based on the priority fee of transactions interacting with them.
The hook calculates the priority fee, then multiplies this by a configurable tax multiplier to determine the tax owed. This dynamic fee is combined with the swap's static base fee, enabling the pool to capture a portion of the MEV generated by high-priority transactions.

this is very cool! I thought about implementing it but you front-ran 😆


Regex isn't working properly

Steps to reproduce
Example1: https://vim.fandom.com/wiki/Regex_lookahead_and_lookbehind

Put: The quick fox jumps over the lazy brown dog. in an empty buffer.
Search with /brown\( fox\)\e=

Brings up an error message instead of the word brown.
You can try the rest on that page. None work. Syntax files using /zs also have issues not selecting.
Expected behaviour
Should select the word brown in provided example.
Version of Vim and architecture
VIM - Vi IMproved 9.0 (2022 Jun 28, compiled Sep 15 2022 18:56:17) macOS version - x86_64 Included patches: 1-472 Compiled by Homebrew
Environment
macOS Monterey
GUI
How MacVim was installed
Homebrew
Logs and stack traces
No response
Vim configuration where issue is reproducable
No response
Issue has been tested with given configuration

[X] by running MacVim.app from GUI macOS interface
[ ] by running vim/gvim/etc installed by MacVim
[ ] by running other versions of vim (e.g. /usr/bin/vim)

Issue has been tested with no configuration

[ ] by running mvim --clean (or gvim, supplied by MacVim distribution)
[ ] by running vim --clean (in terminal, supplied by MacVim distribution)
[ ] by running vim --clean (in terminal, other suppliers, e.g. /usr/bin/vim)

Other conditions

[ ] The both Homebrew packages "vim" and "macvim" are installed

Seems to work just fine for me? You didn't copy the example verbatim. It should be "quick brown fox" in the text, and you need to use \@=, or \ze, not \e or /ze. These details matter a lot.

You can close it. I've been looking at it too long to see the issue. Turns out it wasn't Vim but some override that blocked the /zs /ze selections I needed. Took me some hours to find the lead.

Ah ok. Yeah we have all done that 😅.


Splat referencing monotouch.dll

I've downloaded the latest release as a .zip from https://github.com/akavache/Akavache/releases (Which should support iOS Unified).
Sample Project, 100% repro
I've then created an incredibly simple sample (barebones Xamarin forms project, with striped Packages, you'll have to restore them) here : https://dl.dropboxusercontent.com/u/21635985/IncrediblySimpleExample.zip
This project simply has one PCL and one iOS project. The PCL is referencing the Portable (Portable-Net45+Win8+WP8+Wpa81) binaries and the iOS project is referencing the unified binaries (Xamarin.iOS10).
Build fail due to : MTOUCH: error MT0034: Cannot include both 'monotouch.dll' and 'Xamarin.iOS.dll' in the same Xamarin.iOS project - 'Xamarin.iOS.dll' is referenced explicitly, while 'monotouch.dll' is referenced by 'Splat, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'.
Questions

How would I go about resolving this issue.
How did you not encounter this issue when testing (Have I done something horribly wrong)
As an interesting aside, the iOS Binaries don't appear to have any form of sqlite. (Maybe this is intentional).
Why can I not compile your project when cloning from github, out of the box, without additional steps in Xamarin Studio. I'd love to be able to resolve this problem myself, but it seems like I'd have to donate an entire day to getting the Xamarin studio solutions compiling.

Aside from this issue, I had been using Akavache well, until quite recently and it was working very well indeed! I've written myself a parse.com PCL using the REST API that I'd be more than happy to share when complete.

So, I managed to resolve this by pulling down the latest splat and recompiling it. I guess my next suggestion is that you update the latest akavache release from https://github.com/akavache/Akavache/releases to include the latest (working) splat for iOS and portable, since as it stands, it doesn't work out of the box.


broken email validation in 3.19.0

In 3.19.0 emails with an extra extension on the end cause the validation to incorrectly fail.
Example of what still Works:
const rules={ email: 'required|email' }

<EMAIL_ADDRESS>
Example of what now incorrectly Fails:
const rules={ email: 'required|email' }

<EMAIL_ADDRESS><EMAIL_ADDRESS>
Those last to emails will correctly pass validation in 3.18.1

I am working on a quick update 3.19.1 and I will include the fix for this in that update.  Should be ready by end of weekend.

+1

+1

@JoueBien I just published a new release 3.19.1 to address this issue.


[sum-of-multiples] Implementation

Everybody stand back ! I'm going to try ~~CVS, SVN, ~~ GIT !

Harmony.

@habere-et-dispertire needs conflict resolution.

I haven't done a conflict resolution before. I could use some guidance.
:-)


Merge CHEBI:77125 and CHEBI:87229 ?

Hi Adnan,
Can you please check if CHEBI:77125 and CHEBI:87229 represent the same entity and merge if needed?
Thanks!

Now merged - The primary ID is CHEBI:77125


Change the storage mapping for OVA

Ref: https://github.com/kubev2v/forklift/pull/492
Issue:
Currently in OVA provider as source we need to map each source disk to a target storage class
FIx:
Map a group of OVA disks as a storage type on a targed storage class
Requiers:
https://github.com/kubev2v/forklift/pull/492

cc:// @bkhizgiy

https://github.com/kubev2v/forklift/pull/492 - merged on the backend


Dynamic data-title with ng-repeat

I would like to assign the title dynamically on creation of the html element. It seem the title parser work before the assignment. If i inspect the elements in chrome the title is set but the thead element is missing.

            {{value}}

+1


Reference the Mutiny bindings

Looks good to me. There's only one issue: it seems Node 15 has updated the package-lock.json file to lockfileVersion 2. Since the project uses Node.js 14 (LTS) we should stick with v1. Can you please remove the updated package-lock.json from the commit? Thanks.
Also, I'm wondering if we should look for hrefs that start with a slash or if we should do it the other way round (i.e. look for /^https?:\/\//). But that might be a matter of taste...

Sure let me edit/rebase

@michel-kraemer Done

@jponge Thanks! Do you want all reviewers to approve the PR or should I merge now?

You can merge now 😉


[Machine Request] Add a Quarry

big question is what type is it?
mekanism like? very configurable but smalish max acting area ? ( 32 blocks circle)
BC quarry where it digs out? ( please no)
ender quarry where it replaces with dirt?
I would say mekanism like but with a option to place dirt ( IE you can use it to just dig a area out or flatten a area) Server setting for forced place dirt but higher cost to not place.. or higher cost to place dirt lower without placing ( so that it updates less/better for server but still a option for digging a area out)


Downtime upper limit

Hi,
I recently needed to downtime something for longer than a week and I came across https://github.com/zorkian/nagios-api/blob/master/nagios-api#L283
I was wondering if there was a reason for it being a week or if it's just an arbitrary check?
Thankyou

This issue should be addressed by my pull request https://github.com/zorkian/nagios-api/pull/92


Feat/Add Project Scanner

Motivation
Firstly, I want to say thank you all for adding this awesome extension to telescope! Switching seemlessly through projects has become such a game changer in my development workflow. So I wanted to contribute by tackling the issue of automatically adding  git projects into the telescope-projects.txt. However, as I was working through the problem, I began to start refactoring some of the code base. I will attempt to explain the core changes in this PR, but of course comment below if you have an questions or concerns.
Issues
Project Scanning
Base Directory
Instead of scanning the entire file system, I wanted the user to be able to set a base_dir option, from where .git directories can be recursively searched.
Max Depth
I also wanted the user to be able to specify a max_depth option as a way of speeding search or avoid nested submodules in repositories.
Support Telescope Setup
Instead of supplying these options at project picker, I wanted to supply them in the telescope setup function call.
Example Setup:
require('telescope').setup {
  extensions = {
    project = {
      base_dir = '~/projects',
      max_depth = 3
    }
  }
}

Telescope Projects File
In order to continously scan for projects, we need a way of flagging which ones were deleted (or more precisely - deactivated) by the user - or else, the scanner will continuously dump deleted projects back into projects.
I accomplished this by adding a flag to the project line data structure, instantiating the activated flag to 1. I made sure the code is backward-compatible with the legacy data structure telescope-projects.txt, by treating a missing activated field as 1.
# legacy: title=path
telescope-project.nvim=/home/user/projects/telescope.nvim

# new: title=path=activated (activated field can be 1 or 0)
telescope-project.nvim=/home/user/projects/telescope.nvim=1

Project Structure
I quickly realized that the project.lua was getting increasingly large, so I had a look at the telescope.nvim docs, and adapted the file structure accordingly.
This also forced me to have a look at which functionality we could factor out into own functions. So now the project is structured like so:
lua
└── telescope
    └── _extensions
        ├── project
        │   ├── actions.lua  # custom telescope actions
        │   ├── finders.lua  # custom telescope finder
        │   ├── git.lua      # git project scanning
        │   ├── main.lua     # main setup of telescope extension
        │   └── utils.lua    # general utilities
        └── project.lua      # register extension

Conclusion
I have tested the changes manually, and everything thing seems to work great. I would be really happy if you could also check it out and give me some feedback.
I will also look into writing some unittests for the extension as well to make pull requests easier.
Edits:

added linting and testing for some modules
changed when the projects will be scanned (during setup). This has two advantages:

the  user will see the projects immediately on first call to projects picker (before you needed to close and open again)
it runs only once per session instead of each time you call the projects pickers, so better performance.

😍 😍 😍 😍 😍 😍 😍 😍 😍 😍
@logan-connolly, Honestly great work. I will review tom and give some feedback. I've been meaning to integrate the extension to my workflow for the longest time but I guess this is the time to adopt it.
Thanks ❤

This is very cool @logan-connolly thank you! Great work. I'll have a proper look into this later today. I've been thinking about doing a tidy up for a while. It started off okay, but grew into a bigger project so was needing this. Automatically checking for deleted projects is a really good idea also. Much appreciated

@claidler thanks for the feedback! Once again, thanks a lot for laying the ground work.

It started off okay, but grew into a bigger project so was needing this.

In the end, it was rather easy to refactor because all of the hard stuff was already done, particularly with the the actions and pickers 😄  Anyway, I am really excited to see where this project goes; I think there is a ton of potential.


[BUG] Regression: Not able to assign space role if user does not have organization_user role

Is there an existing issue for this?

[X] I have searched the existing issues

What version of the Terraform provider are you using?
0.2.1-beta
What version of the Terraform CLI are you using?
OpenTofu v1.6.2
What CF API version are you using?
3.166.0
What type of issue are you facing
bug report
Describe the bug
It seems the issue https://github.com/SAP/terraform-provider-cloudfoundry/issues/53 is not fixed yet. A fix has been released with https://github.com/SAP/terraform-provider-cloudfoundry/pull/54 but seems not to work as expected:
In my main.tf I have:
resource "cloudfoundry_space_role" "deploy_users" {
  provider   = cloudfoundry-sap
  for_each   = toset(var.deploy_users)
  space      = cloudfoundry_space.space.id
  username   = each.key
  origin     = "sap.ids"
  type       = "space_developer"
}

But without assigning an org_role first it (still) fails with:
│ Error: API Error Registering Role
│
│   with<EMAIL_ADDRESS>│   on main.tf line 159, in resource "cloudfoundry_space_role" "deploy_users":
│  159: resource "cloudfoundry_space_role" "deploy_users" {
│
│ Could not register Role with user ID  : executing POST request for /v3/roles failed: cfclient error (CF-UnprocessableEntity|10008):
    Users cannot be assigned roles in a space if they do not have a role in that space's organization.

Expected Behavior
Assigning a space_role, without assigning an org_role first, works without an error.
Steps To Reproduce
No response
Your Terraform Configuration
No response
Roles
No response
Add screenshots to help explain your problem
No response
Additional context
No response

Hi @alxsap,
Creation of an org role is necessary before a space role. #54 was to make this message as an error more clearer to the user and automatic creation of org roles was not performed as it would lead to unwanted dangling CF resources which can be better managed from the user side with an additional terraform resource. Therefore request you to kindly create  cloudfoundry_org_role resources with type organization_user before assigning them space_developer roles.


Update git branching model

What does this pull request aim to achieve? 💯
Update Github workflows based on new branching and deployment model, and update Readme. Also update PR template.
I also did the following (which is not visible in code):

Added staging and production branches, with merge/push restrictions to lead engineers only
Restricted merge method in this repo to Squash and Merge

Resolves #1130, #1127

TO DO after this PR is merged:

[ ] Update all open PRs to point to staging
[ ] Merge rc/v1.1 into staging and delete rc/v1.1 branch
[ ] Delete master branch


unresolved import hyper::header::SetCookie

I am trying to readout cookies sent by the server:
match res.headers().get::<SetCookie>() {
    Some(&SetCookie(ref content)) => println!("Cookie: {:?}", content),
    _ => println!("No cookie found"),
}

However, the import for SetCookies, use hyper::header::SetCookie does not work. I'd greatly appreciate any hints and updated documentation on this.

If trying to use 0.12, the reason the import doesn't work is because it is no longer there.


Check command canexecute before execute for button/menu command

What does the pull request do?

call canexecute before execute for button/menu command
What is the current behavior?

button executes the command even it is in inappropriate state e.g. command.CanExecute is false
What is the updated/expected behavior with this PR?

Fix Button/MenuItem to check command.CanExecute before executing the commmand

Shouldn't command.CanExecute just evaluated at the moment CommanParameter is set and when CanExecuteChanged is raised? This would always run command.CanExecute and would make that event redundant and also would eliminate the assignment of IsEnabled = CanExecute. In addition to that CanExecute can be quite expensive to compute.

in wpf Command.CanExecute(...) is called before Commnd.Execute(...), and that's what has been used in other places like: https://github.com/AvaloniaUI/Avalonia/blob/1f16de477e4317a125e85f16430457911df7f5b6/src/Avalonia.Input/KeyBinding.cs#L39-L40
don't think it's a problem or expensive to call CanExecute, altough check for IsEnabled should work the same way tbh (as IsEnabled is expected to be false when command is disabled


Full width banner

Steps to Reproduce

Make sure smart banner are enabled Appodeal.setSmartBanners(true);
Add a banner inside a container AppodealBanner(                  adSize: AppodealBannerSize.BANNER,                  placement: "default"                )

Expected results: Banner to fill the full width if is possible
Actual results: Baner has a fixed size

Image

Hello @atelog .
Thanks for the issue. We started working on changing the banner behavior for the plugin.
Have a nice day.

@da2gl
Also, sometimes, the banner is cut and the content is not fully visible

@atelog Hello. Right now we don't support smart banners for Ad View case. But we fixed banner clipping in the latest update.


modified code so that the tag duplicate alert messages display a "close' button

and remain visible until the user clicks on this button

Thanks @azurebrd


Interactive command line interface.

Overview
Build the basic app framework using the Python Cement framework for command line interfaces (CLI). When complete, should be able to run a basic command like this and be dropped into an interactive console:
python main.py interactive

Acceptance Criteria

[x] Will configure project directory to be isolated Python dev environment.
[x] Will run latest stable version of Python (v 3.12.6)
[ ] Will install Python Cement framework (using pip).
[ ] Running command python main.py interactive will load live console.

References

Set up pyenv / virtualenv: https://chatgpt.com/share/66e88077-5b90-800e-bd43-5f6da6ead736

Python Install
I already have pyenv installed. But I didn't have latest stable version of Python available in it. So I needed to update then install it:
$ pyenv update
$ pyenv install 3.12.6

Cement Install
$ pip install cement
$ cement --help
usage: cement [-h] [-d] [-q] [-v] {generate} ...

Generate Cement App
Cement expects you to generate a project from scratch and to create the project directory itself. Since I already had a directory set up, I ran the command a little differently:
# generate the app in my existing dir and use -f to overwrite any existing files (otherwise will throw an error on conflict)
$ cement generate project -f .

$ pip install -r requirements.txt

# setuptools no longer comes prepacked with Python: https://stackoverflow.com/a/16511140/1093087
$ pip install -U pip setuptools
$ python setup.py install

I'm still having some issues running my new app. I'll address those in their own comment.

Cement Errors
When I try to run the command line command from Cement docs, I get this:
$ dr_george_app --help
dr_george_app: command not found

When trying to run the file directly like I've done in other Cement apps, I get this error:
~/projects/dr-george$ python dr_george_app/main.py
Traceback (most recent call last):
  File "/home/klenwell/projects/dr-george/dr_george_app/main.py", line 4, in <module>
    from .core.exc import DrGeorgeAppError
ImportError: attempted relative import with no known parent package


⚠ TROJAN 🇺🇸 United States USO 1 has degraded performance

In bafb79a, TROJAN 🇺🇸 United States USO 1 (https://ust-2.optnl.com/) experienced degraded performance:

HTTP code: 200
Response time: 23958 ms

Resolved: TROJAN 🇺🇸 United States USO 1 performance has improved in 2a0066b.


Fix Cache table formatting.

The formatting of the cache table was broken. This should fix it.

Thank you! But the empty slot before redis and memcached are intended. I have pushed correct fix, should be good now: https://gogs.io/docs/advanced/configuration_cheat_sheet

No problem! :)


Post release automated changes for automation releases

Post release automated changes for azure-arm-automation

API change check
API changes are not detected in this pull request.


Force rerender

Is there a way to force items to rerender? I have a button on my page that changes the template used in the render function and nothing seems to make the items rerender, even forceRefresh. I'm using the dataset api and i'm using load for the initial values if that helps.

Hi there,
When using dataset, renders are only triggered with dirtyChecking is enabled, a target's data has changed, and the .dataset() method is called.
I think your use case is valid though, and perhaps I need to add an additional public API .render() method could be added to trigger arbitrary re-rendering of all targets on demand. Does this sound like it would work for you?
In the mean time, you could enable dirtyChecking trigger a re-render by changing some arbitrary property on each item in your data and then calling .dataset().
For example, add a template prop to all your items.
Initialise items with:
{
  id: 1,
  template: 'TEMPLATE_A',
}

Then change to:
{
  id: 1,
  template: 'TEMPLATE_B'
}

At the same time as updating your render function, and then call:
mixer.dataset([...])
MixItUp will see that the data of each item has changed, and call your render function.
Rather than manually changing the render function, you could have some logic inside it to check which template should be used, and render different markup for each condition. This way you would be driving the whole operation from the data, rather than just using the data to fake trigger the re-rendering.
I hope that makes sense!

Yeah a render method would be prefect. For now I ended up doing this:
this.grid.dataset([]).then(function() {
    that.grid.destroy();
    var gridElementMix = document.querySelector('[data-ref="container"]');
    that.grid = mixitup(gridElementMix, {
        data: {
            uidKey: 'id',
            dirtyCheck: true
        },
        render: {
            target: function(item) {
                return that.render(item);
            }
        },
        selectors: {
            target: '.grid-item'
        }
    });
    that.grid.dataset(that.items);
});

I am using the code inside an object and storing the items myself since I have to handle pagination (the plugin doesn't keep track of previous items when you set the dataset), so I ended up just using the code I have above. It works for now. If/when there is a render method i'll update it.

I've decided not to add this feature in the end because I feel it breaks the dataset pattern, which is that re-rendering should be caused only in response to data changes, and everything goes through dataset() method.
So, as I suggested above, rather than the button on your page changing the template, it should change the underlying data of your targets, and your template should contain the necessary conditions to produce the two desired HTML outputs.
If you have a strong reason why you think this isn't sufficient, I'd be happy to hear it but I'm going to close for now.

I understand where you are coming from, but at the same time, data is data and templates are templates. I think it is counterintuitive to have to change the template in the code and then have to modify each element in the dataset just to tell the script to re-render. Nothing in the data has changed at all, just the template used to show it, so why should I have to modify the template?

Yeah I do think that's a totally reasonable argument, and I can see the redundancy/overhead of having to do that additional mapping across a large dataset. Let me give it some additional thought to see if there's a solution that's more inkeeping with the functional pattern.

Sounds good. Also, you have to think that if I did what you said and modified all my data to have a template variable so they would be re-rendered and then the user sorted or something of that nature, now I have data that has the template variable and some that doesn't. This quickly becomes messy and it's also changing data for the sake of the template/scripting, which are two different things and goes against what an API is. You even mention yourself in the docs for preloaded data  should be identical to what's rendered so why should it then be modified and different from what comes back from the server in future requests?

Have now added a forceRender() method in https://github.com/patrickkunka/mixitup/releases/tag/v3.2.1
Please let me know if this is working as intended.
Enjoy!

Perfect!! Thanks!! That got rid of about 25 lines of code. :)


Finalize requirements for online safety

Plan needs to be emailed to DSO and other stakeholders with links to issues,


Add two no-loc words globally

See https://github.com/dotnet/docs.pl-pl/pull/211.

Thanks Tom. Seemed a bit too good to be true.


PUT schemas violate the right hand border

If I have a complex schema in a GET, it renders inside some sort of container with a horizontal scrollbar.  Good!
But the same schema as the input body of a PUT doesn't render with the horizontal scrollbar.  The text is unbounded to the right, going beyond the (in this case blue) border for the whole transaction, and even beyond my browser window size.
This is both in Chrome and Edge.
Other than that, great tool!

If you need an example, I can create one.  I just have to strip out anything that might identify the purpose of the application :-)

I'll try it with generated data first. If I cannot reproduce it, I'll ask for your test data.

Rolled out. The bug is actually on Parameter's schema . The Respond's schema  does not overflow out of div.
Can you check and let me know if it works for your data?

Woo hoo!  Nice.  Looks good.  Thanks!


Tiles that should be exposed to space don't always seem to realize.

Description
This causes areas that should've been spaced to simply Not, the space gods showing mercy upon the crew and simply providing invisible space barriers.
Reproduction
This can be reproduced by shooting out windows that are ONLY adjacent to space. If it's next to a lattice it works fine. Easiest to reproduce by exploding the window, deleting it seems to work correctly.

Can't reproduce locally, needs more testing