Python 3.8 support

Hello!
With pytorch now supporting python 3.8, I was wondering if there is a plan to support py 3.8 for lightning too? I was curious about upgrading to 3.8 for the type hints to better organize my own code. Is there something that prevents 3.8 support in lightning inherently?
Cheers!

Hye, good point, I guess that we do not anything which would limit using py3.8 or have you found anything which fails? :robot:

None that I could find from first glance, but I'm trying to run the tests in a py 3.8.1 environment. Currently on my Macbook so I don't have access to GPUs, but thought it's still worth a preliminary attempt before I can get access to a machine with GPUs. Down to one error on collection:
 ImportError: cannot import name 'MLFlowLogger' from 'pytorch_lightning.loggers'
Any idea off the top of your head on how to fix this one? If not, I'll dig into how to fix this one and run the tests and report if I find something breaking. Might have to wait till Monday to run the full coverage since I won't have access to my work machine with the GPUs though.

try again from master

Pulled the latest commits from master, testing against v 0.6.1.dev0, but the problem still persists while following the instructions in the tests module README. Here's the full error:
=================================================== test session starts ===================================================
platform darwin -- Python 3.8.1, pytest-5.3.5, py-1.8.1, pluggy-0.13.1
rootdir: /Users/chiragraman/Projects/foss/pytorch-lightning, inifile: setup.cfg
collected 113 items / 1 error / 112 selected

========================================================= ERRORS ==========================================================
_________________________________________ ERROR collecting tests/test_logging.py __________________________________________
ImportError while importing test module '/Users/chiragraman/Projects/foss/pytorch-lightning/tests/test_logging.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
tests/test_logging.py:9: in <module>
    from pytorch_lightning.loggers import (
E   ImportError: cannot import name 'MLFlowLogger' from 'pytorch_lightning.loggers' (/Users/chiragraman/Projects/foss/pytorch-lightning/pytorch_lightning/loggers/__init__.py)
==================================================== warnings summary =====================================================
pytorch_lightning/loggers/test_tube_logger.py:8
  /Users/chiragraman/Projects/foss/pytorch-lightning/pytorch_lightning/loggers/test_tube_logger.py:8: DeprecationWarning: `test_tube_logger` module has been renamed to `test_tube` since v0.6.0. The deprecated module name will be removed in v0.8.0.
    warnings.warn("`test_tube_logger` module has been renamed to `test_tube` since v0.6.0."

pytorch_lightning/logging/__init__.py:8
  /Users/chiragraman/Projects/foss/pytorch-lightning/pytorch_lightning/logging/__init__.py:8: DeprecationWarning: `logging` package has been renamed to `loggers` since v0.6.1 The deprecated package name will be removed in v0.8.0.
    warnings.warn("`logging` package has been renamed to `loggers` since v0.6.1"

/Users/chiragraman/Projects/foss/miniconda3/envs/lightning/lib/python3.8/site-packages/torchvision/io/video.py:2
  /Users/chiragraman/Projects/foss/miniconda3/envs/lightning/lib/python3.8/site-packages/torchvision/io/video.py:2: DeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
    import imp

-- Docs: https://docs.pytest.org/en/latest/warnings.html
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! Interrupted: 1 error during collection !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
============================================== 3 warnings, 1 error in 1.48s ===============================================

I'll dig deeper in a while, but thought of leaving this here in case someone else is also looking at it in the meanwhile.

Do you have installed all logging libraries? #792

@Borda thanks for the info! I haven't checked my full setup, it may well be the case that I didn't have all the logging libraries installed. I shall report back shortly.
I do think torchvision 0.5 is indeed ported to py 3.8 though. I had an issue with it on Mac that I reported here (https://github.com/pytorch/vision/issues/1780), but it has since been fixed. Not sure if this affects #859

So I managed to get all tests to pass on MacOS Catalina 10.15.1:
❯ py.test -v                                                                                <lightning>
=================================================================================== test session starts ====================================================================================
platform darwin -- Python 3.8.1, pytest-5.3.5, py-1.8.1, pluggy-0.13.1 -- /Users/chiragraman/Projects/foss/miniconda3/envs/lightning/bin/python
cachedir: .pytest_cache
rootdir: /Users/chiragraman/Projects/foss/pytorch-lightning, inifile: setup.cfg
collected 133 items
.
.
.
======================================================================= 133 passed, 147 warnings in 223.19s (0:03:43) =======================================================================

The issue I faced earlier was indeed because I hadn't installed the extra dependencies for the loggers under requirements-extra.txt. @Borda, should the tests/README.md be updated to make a note of perhaps requiring the extra dependencies for testing under the Running tests subsection?

The issue I faced earlier was indeed because I hadn't installed the extra dependencies for the loggers under requirements-extra.txt. @Borda, should the tests/README.md be updated to make a note of perhaps requiring the extra dependencies for testing under the Running tests subsection?

The extras are part of the tests/requirements.txt

Yes, I did notice that part of the discussion you linked to in #792, I meant that the instructions in tests/README.md mention the following:
cd pytorch-lightning
...
pip install -r requirements.txt

I imagined this perhaps ought to be:
cd pytorch-lightning
....
pip install -r tests/requirements.txt

or maybe alternatively the working directory should be updated to pytorch-lightning/tests? Following the README naively didn't work out of the box, so I think there is a minor update required  to the doc that either changes the working directory or points to tests/requirements.txt

Well I would expect that for running the test a user/developer shall install all dependencies required for running full test coverage, otherwise it is kind of lottery to run and see if already has everything and correct version (if you are an active developer you usually have all dependencies)
But I agree that we shall revisit test README, could have a look and send a PR with your suggestions? :]

I agree I agree,  I didn't mean to justify my negligence of not opening and reading the tests/requirements.txt, I just thought since there is a readme with steps, it could be updated to be accurate. I'll send a PR with the update :)

we will add 3.8 testings by #915 otherwise I assume that this is solved...
Btw, feel  free to re-open if needed :robot:


Create a GET API for the root uri

Using Actix-web a endpoint to handle the root uri with a response of "Hello World"

#4 , Refer this


join bug with int 0 join key

This join doesn't work for the 0 value:
cat -n a.zson b.zson
     1	{a:0}
     2	{a:1}
     1	{b:0}
     2	{b:1}

e.g.,
% zq "join on a=b" a.zson b.zson
{a:1}

But it does work for this input
cat -n a2.zson b2.zson
     1	{a:1}
     2	{a:2}
     1	{b:1}
     2	{b:2}

e.g.,
% zq "join on a=b" a2.zson b2.zson
{a:1}
{a:2}

as well as this input...
cat -n a3.zson b3.zson
     1	{a:"0"}
     2	{a:"1"}
     1	{b:"0"}
     2	{b:"1"}

e.g.,
% zq "join on a=b" a3.zson b3.zson
{a:"0"}
{a:"1"}

Fixed in #3334 and verified at 9aa6ed2388009374482568b3e55cef55f42421ec.


🐛 FlutterFire_UI/Auth Email validation fails on '_' (underscore)

Attempt to register an email with an '_' fails<EMAIL_ADDRESS>Not sure if this is in the firebase_auth package or _ui package.
flutterfire_ui: ^0.3.0

this happen when email contain underscore and dash character
hope this issue will fixed soon

@KelvinConnor
Can you provide flutter doctor -v , pubspec.yaml and a complete minimal code sample that shows this behavior along with any error log / message that you get while trying to register with it ?
Thanks.

I don't really have any good code sample since the issue is raised in the UI aspect of the package. No debug info is being provided from the package to help id the exact location of the problem.
I believe this source of the problem is located: https://github.com/FirebaseExtended/flutterfire/blob/eaf0af6c4b0d418aa36d7dd2cd8cd60e40b06ebf/packages/flutterfire_ui/lib/src/auth/validators.dart  LN45 final _emailRegexp = RegExp(r'^[a-zA-Z0-9.]+@[a-zA-Z0-9]+.[a-zA-Z]+');
Perhaps just adding the '_' (and perhaps the -) will fix the issue and the other issue mentioned above.
final emailRegexp = RegExp(r'^[a-zA-Z0-9.-]+@[a-zA-Z0-9-]+.[a-zA-Z]+');

Thanks for looking at this!
---------------CODE----------------------------------------------------------------
I am using the code example from 'https://firebase.flutter.dev/docs/ui/auth/integrating-your-first-screen'
@override
Widget build(BuildContext context) {
return StreamBuilder<User?>(
stream: FirebaseAuth.instance.authStateChanges(),
builder: (context, snapshot) {
// User is not signed in
if (!snapshot.hasData) {
// ..
logger.d('User is NOT signed in');
//return const LoginPage();
return const SignInScreen(
providerConfigs: [
EmailProviderConfiguration(),
],
);
}
logger.d('User IS signed in');
// Render your application if authenticated
return const HomePage();
},
);
}
---------------FLUTTER DOCTOR -------------------------------------------------------
[√] Flutter (Channel stable, 2.8.1, on Microsoft Windows [Version 10.0.19044.1415], locale en-US)
• Flutter version 2.8.1 at C:\dev\flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 77d935af4d (7 days ago), 2021-12-16 08:37:33 -0800
• Engine revision 890a5fca2e
• Dart version 2.15.1
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
• Android SDK at C:\Users\kelvi\AppData\Local\Android\sdk
• Platform android-31, build-tools 30.0.2
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
• All Android licenses accepted.
[√] Chrome - develop for the web
• Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe
[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.9.1)
• Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
• Visual Studio Community 2019 version 16.9.31105.61
• Windows 10 SDK version 10.0.19041.0
[√] Android Studio (version 4.1)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin can be installed from:
https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
[√] VS Code (version 1.62.3)
• VS Code at C:\Users\kelvi\AppData\Local\Programs\Microsoft VS Code
• Flutter extension can be installed from:
https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter
[√] Connected device (5 available)
• KFFOWI (mobile)              • G000H4046223024Q • android-arm    • Android 5.1.1 (API 22)
• AOSP on IA Emulator (mobile) • emulator-5554    • android-x86    • Android 9 (API 28) (emulator)
• Windows (desktop)            • windows          • windows-x64    • Microsoft Windows [Version 10.0.19044.1415]
• Chrome (web)                 • chrome           • web-javascript • Google Chrome 96.0.4664.110
• Edge (web)                   • edge             • web-javascript • Microsoft Edge 96.0.1054.43
• No issues found!
----------------------PUBSPEC ----------------------------------------------------------------------
name: flytyersworkbench
description: A new Flutter application.
The following line prevents the package from being accidentally published to
pub.dev using flutter pub publish. This is preferred for private packages.
publish_to: 'none' # Remove this line if you wish to publish to pub.dev
The following defines the version and build number for your application.
A version number is three numbers separated by dots, like 1.2.43
followed by an optional build number separated by a +.
Both the version and the builder number may be overridden in flutter
build by specifying --build-name and --build-number, respectively.
In Android, build-name is used as versionName while build-number used as versionCode.
Read more about Android versioning at https://developer.android.com/studio/publish/versioning
In iOS, build-name is used as CFBundleShortVersionString while build-number used as CFBundleVersion.
Read more about iOS versioning at
https://developer.apple.com/library/archive/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html
version: 1.0.0+1
environment:
sdk: ">=2.15.1 <3.0.0"
Dependencies specify other packages that your package needs in order to work.
To automatically upgrade your package dependencies to the latest versions
consider running flutter pub upgrade --major-versions. Alternatively,
dependencies can be manually updated by changing the version numbers below to
the latest version available on pub.dev. To see which dependencies have newer
versions available, run flutter pub outdated.
dependencies:
flutter:
sdk: flutter
The following adds the Cupertino Icons font to your application.
Use with the CupertinoIcons class for iOS style icons.
cupertino_icons: ^1.0.2
firebase_core: ^1.10.6
flutterfire_ui: ^0.3.0
logger: ^1.1.0
dev_dependencies:
flutter_test:
sdk: flutter
The "flutter_lints" package below contains a set of recommended lints to
encourage good coding practices. The lint set provided by the package is
activated in the analysis_options.yaml file located at the root of your
package. See that file for information about deactivating specific lint
rules and activating additional ones.
flutter_lints: ^1.0.0
For information on the generic Dart part of this file, see the
following page: https://dart.dev/tools/pub/pubspec
The following section is specific to Flutter.
flutter:
The following line ensures that the Material Icons font is
included with your application, so that you can use the icons in
the material Icons class.
uses-material-design: true
To add assets to your application, add an assets section, like this:
assets:
- images/a_dot_burr.jpeg
- images/a_dot_ham.jpeg
assets:
- assets/images/fly_box_16_9_small.png
- assets/images/fly_box_16_9_med.png
- assets/images/fly_box_16_9_large.png
- assets/images/fly_box_9_16_small.png
- assets/images/fly_box_9_16_med.png
- assets/images/fly_box_9_16_large.png
- assets/images/ftw_logo_300.png
An image asset can refer to one or more resolution-specific "variants", see
https://flutter.dev/assets-and-images/#resolution-aware.
For details regarding adding assets from package dependencies, see
https://flutter.dev/assets-and-images/#from-packages
To add custom fonts to your application, add a fonts section here,
in this "flutter" section. Each entry in this list should have a
"family" key with the font family name, and a "fonts" key with a
list giving the asset and other descriptors for the font. For
example:
fonts:
- family: Schyler
fonts:
- asset: fonts/Schyler-Regular.ttf
- asset: fonts/Schyler-Italic.ttf
style: italic
- family: Trajan Pro
fonts:
- asset: fonts/TrajanPro.ttf
- asset: fonts/TrajanPro_Bold.ttf
weight: 700

For details regarding fonts from package dependencies,
see https://flutter.dev/custom-fonts/#from-packages

https://github.com/FirebaseExtended/flutterfire/blob/efdcc16aa2f3d45a3835f0eb73801ba88e808e07/packages/flutterfire_ui/lib/src/auth/validators.dart#L45
underscore is just not included in the validator.
The validator should be customizable I think.  #7638

Thanks for the update. Labeling for further investigation and fix.

Actally this does not work with email containing dots tooo.
For eg if my mail is<EMAIL_ADDRESS>then UserCrediantial.user.email would return null. Please fix this too. @darshankawar


Unable to Install via Addons Page - Install Button

I have tried installing this plugin 4 or 5 separate times. and each time nothing happens. I saw last year you mentioned to someone with a similar issue that "there was no working plugin yet, this was a test version I started on recently" (paraphrasing). I am trying to get Fallout 4 animations into Blender and this tool would be a massive help. But i cant tell if its me doing something wrong, or if its the tool. I'm on Blender's most up-to-date build. Thankyou for your time. Any help or insight would be greatly appreciated!

to add more detail. When i hit Install i get the little blue text at the bottom of the screen where it says "installing addon : Users\Addon\File_Path" and thats it. I go search through my Addons and this plugins no where to be found. I go to my File dropdown tab, and go to import or export and theres no option for HKx. I checked everywhere in the Addons page. And its just not Installed. I hope this is just an issue with me and that the plugin is working and available. I'd appreciate any insight into this issue. Thankyou for your time.

What you describe is probably a Blender version incompatibility. I made this addon for 3.2 and haven't updated it since.
However, you will still have issue #2.


Added roman numerals test generator

Shortened the branch name and resubmitted pull to remove long line SHA1 issue.

Sorry, messed this one up. Closing and will submit pull to the CORRECT branch.


feat: Added show_instances to hide and show instances quickly without updating buffers

This allows users to change the number of instances which are displayed quickly without modifying buffers. Useful if you need to quickly "slice" or truncate the instances.
Example Usage: 3D Printer GCode Renderer
Uses a InstancedModel to display 3D printer movements as Cylinders, a Slider input on the right allows the user to truncate the InstancedModels to step through each layer of the 3D print:
Before: Full 3D Print

After: User has selected to see only up to a certain layer

Makes sense but I would rather have it as a method that only draws a subset of the instances. So something like pub fun draw_subset(&self, ...., start: u32, end: u32). Would that be ok for you? The only difference is that you need to provide the number each render call.

Where does draw get called? I have code that looks like this and I'm not sure where I'd need to modify this to call draw_subset instead of draw:
                    Screen::write(
                        &context,
                        ClearState::color_and_depth(1.0, 1.0, 1.0, 1.0, 1.0),
                        || pipeline.render_pass(
                            &camera_control.camera,
                            scene_objects.as_slice(),
                            &lights,
                        ),
                    )
                    .unwrap();

Separately I'm not sure how to implement the starting index. I agree it would be more ergonomic to be able to choose a subslice with an arbitrary starting index as long as it can maintain similar performance (that's a question, I'm quite new to Open/WebGL and not entirely clear on how these things get passed into it yet).

draw_subset would be a bit deeper down since it is a special case (that just means you need to remove the instanced model from the scene_objects and call draw_subset directly on that before or after you call the pipeline.render_pass) or we need to go with your suggestion to add a variable in instanced model. What do you think?
It requires a bit of changes to get a start index, but nothing major, it is already supported, just not exposed. So I could just add that support if necessary, no problem 👍 Also, the performance is exactly the same.
A completely different option would be to split the instanced model into layers, so one instanced model per layer instead of one containing all layers. Then you can remove and add instanced models to the scene_objects slice as the user drags the slider. Would that work? I don't think you should worry about performance loss with this change unless you have hundreds, maybe thousands of layers.

Unfortunately I have hundreds or thousands of layers.
Let me look into how I would use_subset in my own code next week and get back to you once I understand it better.

Oh, hmm, so I have many objects in this scene. I'm looking at these line in render_pass:
    let mut culled_objects = objects
        .iter()
        .filter(|o| camera.in_frustum(&o.aabb()))
        .collect::<Vec<_>>();
    culled_objects.sort_by(|a, b| cmp_render_order(camera, a, b));
    for object in culled_objects {
        object.render(camera, lights)?;
    }

I think I need to copy and paste that object ordering code in render_pass but with a conditional for the draw subset InstancedModel that then also needs a modified version of render_with_materials to switch to draw_subset. Something like this:
let mut culled_objects = objects
    .iter()
    .filter(|o| camera.in_frustum(&o.aabb()))
    .collect::<Vec<_>>();
culled_objects.sort_by(|a, b| cmp_render_order(camera, a, b));

for object in culled_objects {
    if object == my_gcode_slice_object {
        let fragment_shader_source =
            material.fragment_shader_source(self.mesh.color_buffer.is_some(), lights);
        self.context.program(
            &Self::vertex_shader_source(&fragment_shader_source)?,
            &fragment_shader_source,
            |program| {
                material.use_uniforms(program, camera, lights)?;
                self.draw_subset(
                    program,
                    material.render_states(),
                    camera.uniform_buffer(),
                    camera.viewport(),
                    start,
                    end
                )
            },
        )
    } else {
        object.render(camera, lights)?;
    }
}

This seems like a larger change to my code then I was expecting and there's a worry about maintaining the copy & pasted code.
I'm wondering would adding a set_draw_subset(start: usize, end: usize) method to InstancedModel work instead? ie essentially the original proposal but with the start offset you suggested?
It adds usize to the memory footprint of instanced models which I'm imagining will be a negligible addition in most use case but if there aren't any other major downsides it could help keep this ergonomic for devs that aren't super ready to drop down into the lower level APIs.

I think you are right, that is a big change and there is no real advantage of my suggestion, so let's go with your suggestion. I will change it slightly and then merge it in 👍
Also, I was wrong about the start index into the instanced buffers, that is not possible. It is only possible to use a start index into the vertex buffers, so it is possible to render a subset of triangles of the instanced model, but not a subset of the instances.


datetime example

Initial version of a datetime example which validates that a string is in ISO local date and time format (2017-05-02T15:30:00) and will result in an Java 8 LocalDateTime.
Single elements in the pattern can be substituted by Int values, see tests.scala for an example.

I'm keen to merge this (along with #7), pending the discussions over in #13.

I'm not sure if this should be part of the base artifacts. I just got it working for Scala.Js and Native, but there this would either not work (no JVM), or would require an extra dependency: https://github.com/scala-js/scala-js-java-time so I think it should stay in a separate module.

That's a good point, @mkotsbak. This issue #10 is discussing a separate project, but I think it's important that whatever goes into contextual-representations should cross-compile. I'll comment more when I've got a bit more free time!

And I'll leave the PR open for now.


enhancement: rebrand MVG to Pontus-X

Proposed Changes

replace MVG and minimal viable Gaia-X instances with the new Pontus-X - Gaia-X Web3 Ecosystem branding

Should we also think about renaming the repository?


Add support to delete the rule from Simplivity policy

Signed-off-by: Hemant Chaudhari<EMAIL_ADDRESS>Description
Add support to delete the rule from Simplivity policy
Issues Resolved
None
Check List

[x] New functionality includes testing.

[x] All tests pass($ tox).

[x] New functionality has been documented in the README if applicable.

[x] New functionality has been thoroughly documented in the examples (please include helpful comments).
[x] New endpoints supported are updated in the endpoints-support.md file.

[x] Changes are documented in the CHANGELOG.

Added the example for delete

@svt-hchaudhari , resolve the conflicts to merge this PR.


Draft of web identity+OIDC docs for p-aws

Add some setup instructions for web identity+OIDC authentication to resolve pulumi/pulumi-aws#2425.

Direct link to edited instructions


Offscreen rendering doesn't work on server & workarounds & fixes

Hi Davis,
I found the offscreen rendering doesn't work with the required installation. My workaround is to 1) re-installing pyrender with mesa and 2) adding os.environ["PYOPENGL_PLATFORM"] = "osmesa" to viz_fitting_rgb.py.
Then there is a related issue. Even if it is set as "RGBA" by default, my renderer only generates the color_image with three channels, which leads to an error of computing the valid_mask here and then the composition here.
My fix in mesh_viewer.py is as follows:
input_img = self.cur_bg_img
if color_img.shape[2] == 4:
    output_img = (color_img[:, :, :-1] * color_img[:,:,3:] +(1.0 - color_img[:,:,3:])*input_img)
else:
    valid_mask = (depth_img > 0)[:, :, np.newaxis]
     output_img = (color_img * valid_mask + (1 - valid_mask) * input_img)

-Jimei

When use offscreen render mode in Pyrender with platform='osmesa', it will produce 3 channel image, which means the alpha channel is lossed. I don't konw why, but it is. Instead you can do like follow:
            #valid_mask = (color_img[:, :, -1] > 0)[:, :, np.newaxis]             valid_mask = (np.sum(color_img, axis=-1) > 0)[:, :, np.newaxis]


Tracking daily tasks

Thanks a lot for checking! My second job is still a bit busy and will be the same for the next few months, but I'm doing okay. :) Have a great weekend!

 Hangout with me tooo

Likewise! You can't die from working too much ;o;


Impl. of auto select of the last partial watched or next episode to be watched

Check if this PR fulfills these requirements:

[x] My changes respect the Kodi add-on development rules
[x] I have read the CONTRIBUTING document.
[x] I made sure there wasn't another one Pull Requests opened for the same update/change
[x] I have successfully tested my changes locally

Types of changes

[x] New feature (non-breaking change which adds functionality)
[ ] Feature change (non-breaking change which change behaviour of an existing functionality)
[ ] Improvement (non-breaking change which improve functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

A hacky implementation to auto select the next episode to watch without scroll manually every time
Kodi not offer this feature... this is better than nothing...
In case of Feature change / Breaking change:
Describe the current behavior

Describe the new behavior

Screenshots (if appropriate):

Will this fix the problem that if you watch Netflix with Up Next, that after you watch 3 episodes the virtual directory listing has the cursor on the first episode, and all episodes are marked as unseen?
Because that is what we see today in Leia after watching multiple episodes (or IIRC even after watching one episode).

i do not think, I had investigated and tried different ways to circumvent the problem
but the problem remains,
it seems to be a concurrence issue of gui update in kodi core
Also the "resume" a video does not work well, and i think it's another similar problem
resume time data is sent correctly, but at some point in Kodi source code the data is inexplicably deleted by a function intersected with the database (I already opened an issue but at today ignored)

We have had similar issues which we worked around in two ways IIRC

Ensure all code paths that update metadata are very fast (make slow stuff like online updates asynchronous)
Force our own container refresh when it is needed

This way we managed to get resuming a video working directly after you stopped playback.

Ensure all code paths that update metadata are very fast

I don't think i know exactly what you mean, but the resumeTime of each listitem are already updated in realtime (in database, then when kodi require to update the episode list in GUI the values are already ready)

Force our own container refresh when it is needed

there is a way to know when kodi refresh the list?
would make it ugly to see double refreshed

This way we managed to get resuming a video working directly after you stopped playback.

in my case the problem of resume happens not only (in the exact moment) when you make a stop
but also if you close/reboot kodi and after reopen it you try to resume the same item,
kodi ask you to resume the video from previous time, but after dialog confermation it starts from beginning...
and the problem is not always consistent, sometimes in a listitem the resume works, another times happen that same listitem fails to resume, that's why i opened up the issue: https://github.com/xbmc/xbmc/issues/17426
time ago i have tried to debug kodi core, and i found where in source code the resumeTime value is lost (see "Possible Fix")
Unfortunately, without more help, I'm not able to go more deeper in the code

@CastagnaIT

Ensure all code paths that update metadata are very fast

I don't think i know exactly what you mean, but the resumeTime of each listitem are already updated in realtime (in nf database, then when kodi require to update the episode list in GUI the values are already ready)

I don't know if this is technically correct, but if playback stops it takes an amount of time for Kodi to return back to the menu. If your metadata is updated before this has happened, the listing has all the updates you have made. Since setting resumetime is done in a Player thread (recommended), it means you have to do this as fast as possible, that is why we opted for asynchronous update. So we optimized that path.

Force our own container refresh when it is needed

there is a way to know when kodi refresh the list?
would make it ugly to see double refreshed

Well, if you refresh before returning to the listing, your refresh will not be acted on. So if you have a fast path to update resumetime, you can always order a refresh. If the listing was already shown, there will be a noticable refresh, but in our opinion it is better to make it functionally work right always, than avoid a visible refresh.

This way we managed to get resuming a video working directly after you stopped playback.

in my case the problem of resume happens not only (in the exact moment) when you make a stop
but also if you close/reboot kodi and after reopen it you try to resume the same item,
kodi ask you to resume the video from previous time, but after dialog confermation it starts from beginning...

IIRC We update resumetime on start, pause, stop, playbackerror. We do not update resumetime (online) every X seconds, but I guess we could do this asynchronously as well.
Our biggest problem is keeping the local metadata in sync with the online metadata. If that would be possible we would not have to force the (cached) remote resumetime, but we could update the resumetime in the local metadata. @mediaminister fixed this in xbmc/xbmc#17202 , but this was not backported to Leia, unfortunately.
BTW We track all our Kodi/IA issues and feature requests at:

https://github.com/orgs/add-ons/projects/3
https://github.com/orgs/add-ons/projects/4

oooh i think we are talking about two different things!!!... LOL

Updating the resumetime for us means: updating our own local cache and updating the remote VOD service.

I try to explain myself better in my shitting english, i hope now you understand better
Due to particularity of netflix i preferred to use a different way, so exclude the standard kodi functionality of the watched system, by set to each ListItem these properties e.g.:
ListItem.TotalTime = the full time
ListItem.ResumeTime = some time elapsed (partial watched)
ListItem.setInfo('video', {'PlayCount': '1'}) << this only when a video is full watched
when i talk about resume time i mean the ListItem.ResumeTime
using this everything works well without use the jsonrpc override, because i set these properties,
but kodi has a bug in ListItem.ResumeTime property that lose the value when it elaborate internally the value...

In nf addon the resumetime value update works similary of you explaing:
when a video is played:
-in local the resumetime value is saved in realtime in a temporary cache
-in real nf server the resume time value will be send the update in async way, every minute (netflix design not an personal choice)

Ok, we came to an almost identical implementation :-)


Stack references for functions using rsp to track stack frame

I noticed that if a function only uses rsp to track stack frame (the rbp register is used as a regular register), stack references and offsets are incorrect.
How can I get correct offsets?
Is there any rsp based stack analysis in ghidra?

I have the same question (decompile eBPF Processor for Ghidra)!!! eBPF uses only frameponter register, it’s unchangeable, and we use this register and offset to access the stack (such as index in array)
Check out my issue...
https://github.com/Nalen98/eBPF-for-Ghidra/issues/2

The stack analysis in Ghidra is only RSP based.  All stack variables are relative to RSP at entry to the function.  The function can use any register for the frame, or none at all.  If the RSP is moved into any register and a reference is seen to the stack, a stack reference is created relative to RSP.  This is different than other disassemblers or decompilers that use offsets from the EBP register after the frame is established.
There have been thoughts of providing an option to show references/offsets relative to some a frame register as an option to show the same offsets as say IDA or a debugger that may rely on an established frame to show values.
We can take a look, at the example to see whats going on.  It is also possible that an instruction with incorrect semantics is being used and needs to be fixed, or a call to a function is messing with the stack in a hidden way.
That said, it doesn't mean there isn't some issue...

@HQ1995, can you provide an example set of bytes for a function that has this issue?

@emteere, In my case, there is a function directly uses rsp register to reference the stack variable.
At first, ghidra analyzed the stack(newStackAnalysis) automatically, the variable in offset 0x78 has no appeared.

Then I manually committed local variables (right-click the function name) for this function and redo the stack analysis.
Although the stack array appeared, the reference has not appeared yet.

Thanks for your help!

Hadn't seen this had been updated, until I went back through.
The stack references created on the listing are based purely on a read/write access to the stack.  In the case above a pointer on the stack was put into a parameter that was then passed to a function.  We currently don't detect this access.
The code does look at parameters to functions for constants passed that are references.  It just doesn't consider stack offsets passed in parameters.  So it should be easy to implement.


Enable azure RBAC deployments on all services

Context
Enable azure RBAC deployments on all services
Changes proposed in this pull request

Replace get-credentials with make get-cluster-credentials

Add get-cluster-credentials: kubelogin convert-kubeconfig -l $(if ${GITHUB_ACTIONS},spn,azurecli) to get-cluster-credentials

Update kubernetes Provider

Guidance to review
Access https://dev.teacher-qualifications-api.education.gov.uk/health, ensure status returns "OK"
Checklist

[x] Attach to Trello card
[ ] Rebased master
[ ] Cleaned commit history
[ ] Tested by running locally

 Enable Azure RBAC deployment on all services


Error in link for "download the full list"

There is a trailing underscore in the Excel file link that seems to prevent download.  https://download.microsoft.com/download/1/5/4/15454442-CF17-47B9-A65D-DF84EF88511B/Products_by_Benefit_Level.xlsx_

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

ID: 10ab4ecc-3cca-8224-8b30-7bccd1ebae31
Version Independent ID: 2e89b277-021e-b185-ea1a-bc153481b0e1
Content: Using Subscriber Downloads on Visuals Studio Subscriptions
Content Source: subscriptions/subscriber-downloads.md
Service: vs-subscription
GitHub Login: @evanwindom
Microsoft Alias: jaunger

@mhyon This has been fixed. Thanks for reporting it! And @evanwindom, thanks for fixing it. :)


Add support for tracking devices on Netgear access points

Description:
It also allows the use of SSL parameter. Autodetection of the URL has also been added.
Cf doc PR for more infos.
This is waiting on balloob/pynetgear#30, balloob/pynetgear#32, balloob/pynetgear#33 and balloob/pynetgear#35 to be merged.
In the meantime you can test with pip install git+https://github.com/MatMaul/pynetgear.git@hass-orbi.
Tested with Orbi.
Pull request in home-assistant.github.io: home-assistant/home-assistant.github.io#4958
Example entry for configuration.yaml:
device_tracker:
  - platform: netgear
    password: YOUR_ADMIN_PASSWORD
    devices:
      - MY-LAPTOP
      - 00:1A:2B:3C:44:55
    exclude:
      - MY-SERVER
    accesspoints:
      - 3C:44:55:00:1A:2B

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] New dependencies have been added to the REQUIREMENTS variable.
[x] New dependencies are only imported inside functions that use them.
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc: nothing since omitted file

Hi @MatMaul,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Updated with ssl parameter instead of url, and default username is now handled by pynetgear.

A new release of pynetgear has been released, this should be ready for inclusion now.

This PR is ok to merge when linting issues addressed: https://travis-ci.org/home-assistant/home-assistant/jobs/368807694#L569

Congrats on your first PR and the new release of pynetgear 🐬 🎉

Can you please add a paragraph to the PR description explaining the breaking change.

What do you mean ? I don't break anything here, everything is backward compatible.

Oh, in that case the "breaking-change" label was incorrectly added. Removing it.


applying simp? gives a different result from simp

Prerequisites
Please put an X between the brackets as you perform the following steps:

[X] Check that your issue is not already filed:
https://github.com/leanprover/lean4/issues
[X] Reduce the issue to a minimal, self-contained, reproducible test case.
Avoid dependencies to Mathlib or Batteries.
[X] Test your test case against the latest nightly release, for example on
https://live.lean-lang.org/#project=lean-nightly
(You can also use the settings there to switch to “Lean nightly”)

Description
Applying the code action from simp? gives a different result than simp or simp? itself. Example:
def nothing (T : Nat → Prop) : Prop := sorry
def bar (T : Nat → Prop) : Nat := sorry

def foo (x : Nat) : Prop :=
  ∃ (S : Nat → Prop) (_ : nothing S) (_ : ∀ T, (∀ x, T x → S x) → nothing T), x = bar S

theorem foo_zero (n : Nat) : foo n := by
  rw [foo]
  simp?

After the final simp or simp? call, the goal is
∃ S, (∀ (T : Nat → Prop), (∀ (x : Nat), T x → S x) → nothing T) ∧ nothing S ∧ n = bar S

However after applying the code action simp? gives me, simp is replaced with
theorem foo_zero (n : Nat) : foo n := by
  rw [foo]
  simp only [exists_prop', nonempty_prop, exists_and_left]

and after the simp only call now, the goal is
∃ S, nothing S ∧ (∀ (T : Nat → Prop), (∀ (x : Nat), T x → S x) → nothing T) ∧ n = bar S

Context
Occurred while trying to demonstrate how simp? works for teaching. Discussion here: https://leanprover.zulipchat.com/#narrow/stream/270676-lean4/topic/The.20result.20of.20simp.3F.20gives.20a.20different.20result.20to.20simp
Steps to Reproduce

Code as above.

Expected behavior:
The simp only command given by simp? should behave the same as simp itself.
Actual behavior: Squeezing the simp changes what it does.
Versions
4.12.0-rc1, MacOS Monterey. Also reproduces on Lean nightly at time of writing.
Additional Information
[Additional information, configuration or data that might be necessary to reproduce the issue]
Impact
Add :+1: to issues you consider important. If others are impacted by this issue, please ask them to add :+1: to it.

The three lemmas listed here are non-confluent, which partially explains the issue. But as long as simp and simp only apply the same strategy, it shouldn’t matter.
My assumption (not further investigated) is that the rules come out of the DiscrTree in a different order. Maybe if there are multiple rules there, they should be sorted by some predictable way – even lemma name could help to make this more reliable.

Should be a duplicate of #4615


Update(docs): docs/jsx-in-depth.md

clarify the example code with description.

The example is specifically to show conditionally rendering the one component, not all of them. Thank you for the PR!


Problem with email mask on Android devices

When you start to type, plugin adds a character and jumps to the next part of email string after @ sign
Also strange behavior when trying to delete symbols with Android backspace key

Yes, on mobile phones there are some issues with email editing. Here video example: https://disk.yandex.ru/i/rZqUvH17zDSHoA


RHIDP-2023: Added a procedure for running RHDH behing a corporate proxy

Purpose: To resolve this issue: https://issues.redhat.com/browse/RHIDP-2023
Preview link: https://redhat-developer.github.io/red-hat-developers-documentation-rhdh/pr-225/admin-rhdh/#assembly-running-rhdh-behind-a-proxy
Cherrypick version: 1.2.x
SME: @rm3l
QE: @albarbaro
Peer: @linfraze

@shipsing @albarbaro FYI, https://github.com/janus-idp/backstage-showcase/pull/1225 has been merged. You might need to wait until the downstream image is built for this feature to be available in RHDH.

/cherrypick 1.2.x


[PWA]: Install manifest is broken

Bug
Expected Functionality
An installed version of the application should function properly
Actual Functionality
Currently there is a forwarding loop that is happening, preventing the application from loading.
Steps to reproduce

Go to '...'
Click on '....'
Scroll down to '....'
See error

Screenshots

Device information

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser: [e.g. stock browser, safari]
Version: [e.g. 22]

Other remarks

fixed by making path an absolute to /index.html


Index: use a bare repository rather than a non-bare repository without checkout

Ever since https://github.com/rust-lang/cargo/issues/4015 , Cargo avoids checking out the index as a working directory; however, cargo still creates the repository as a non-bare repository with a working directory that just doesn't have any files checked out.
Ideally, I'd love to store this as a bare repository instead, where git doesn't expect to have a working directory.
This would have a few advantages:

Git tools would understand the repository better, making it easier to explore and understand and debug.
Tools that want to check .cargo into git (for caching or testing or other purposes) would be able to do so; git can track a repo.git bare repository, but cannot track a .git directory.

(This was discussed briefly in a Cargo team meeting; I'm filing this issue as a follow-up.)
If we don't worry about a one-time re-download of the index, we could solve this by just switching to a bare repository.

Just to be clear, is the only difference that .git/config would have core.bare = true in it?   And I guess the directory layout would also remove the .git directory?  That might be something that would cause a lot of disruption that may be quite tricky to handle, particularly when using older versions of cargo.  There could be a parallel directory, but that would use a very large amount of disk space.

The main difference would be having .cargo/registry/index/github.com-1ecc6299db9ec823.git rather than .cargo/registry/index/github.com-1ecc6299db9ec823/.git; both that and the configuration tell git that it's a bare repository with no working directory. (The current configuration looks to git like a working directory where all the files have been deleted.)


Dataplane

TODO:

[x] bfdSend.Mem
[x] bfdSend.String
[x] bfdSend.Send (skipped)
[x] Strengthen spec of decodeLayers
processPkt and dependencies

[x] reset
cases:

[x] interBFD
[x] intraBFD
[x] onehop (#191)
[x] scion

[x] resolveInbound
[x] connect with the rest of the proof

Run

[x] realized that the spec for BatchConn is unnecessarily restrictive, we should make it weaker - use wildcard perms instead of full perms
[x] converge in postconditions of processPkt
[x] merge #240
[x] verify calls to the closures in (*DataPlane).Run(...) (#244)
Packscmp (#126, #192) (partially done in #243, but we might have run into a bug that keeps us from verifying it further)

[x] come up with a spec for packSCMP (#243)
[x] verify it according to the given spec (#243)
[ ] remove all @TODO() that precede calls to it (blocked on a bug from scion)

In summary, we verified mem. safety for when receiving and sending SCION packets (except when we send SCMP packets as reply - ongoing and depends on packSCMP), when handling empty and when handling onehop packets.
This means that the only things missing are the epic paths and the generation of SCMP packets.
Nice to have:
On a first step:

[ ] processEpic (#196,  #261) - kinda done, but we need to debug performance problems

[x] libepic (#199, #266)

newBFDSend (and the setter methods that call this)

requires making slayers.SCION.Mem() more general (to allow nil ubufs)
setters of bfd connections in the dataplane

AddExternalInterfaceBFD
AddNextHopBFD

send and new

missing getters and setter

Later:

finish prepareSCMP (#245, #360) past the @TODO()

this may require some refactor of (*SCION).Mem and the introduction of a ghost field to describe whether a SCION struct was created manually or parsed from a slice

Split memory footprint predicates from functional properties, as started in https://github.com/viperproject/VerifiedSCION/pull/228
finish and merge the refactor (#122, #155)

adapt according to https://github.com/viperproject/gobra/pull/619

merge the latest commits in the upstream SCION (#190)
check op. for linearity to use our linear type system
allow _ perm in iterated collections with range (gobra)
bfdSend
[x] issue #2
address all files that have been specified but not verified
adapt to the new map encoding (https://github.com/viperproject/gobra/pull/606)
dataplane adapter
ghost fields to simplify Dataplane

I will close this issue. All major tasks that are still missing have open issues already, we will track progress through them.


Bulk upload applications in HR jobs

Is your feature request related to a problem? Please describe.
Currently, we have an application mechanism in the portal for the website only. We would require functionality to upload the applications from an excel sheet.
Describe the solution you'd like
We can give a bulk upload feature, where the applications can be uploaded from an excel sheet. There is already an existing implementation created (start researching with this route). We will start with tweaking the particular section.
Dev process to follow:

Figure out a way to run the current implementation, and see to which extent it is working
Once we get an idea of the code changes we need to do, create issues for fixing the uploading of applications, and what's working and what's not.  BREAKING IT DOWN IS A MUST
After the applications are created, and proper mapping is done in the database, we need to figure out the best way to upload the resume corresponding to each application. We do have an existing functionality that moves portal resumes to CC websites in WordPress. We need to check if moving the resumes to S3 would be the best option (yet to be researched)
Bulk uploaded applications will have channels specified (database structure would be completed after issue #1818  is resolved
Post bulk upload, we will also include a telephonic interview round in the evaluation
Extensive testing would be required whether the application is created as per the website flow.
Post this import functionality, we will implement a telephonic interview round for this

You are not alone

Reach out for help on the portal hangout channel, to get help in understanding the workflow, breakdown of tasks

GitHub workflow
It would be a huge functionality, hence we will do it in a feature branch

Create a feature/bulk-import-applications branch, this branch would be like the master branch of the portal.
Breakdown the bulk upload into different phases and each phase would have a PR which upon review will be merged into  feature/bulk-import-applications, the main branch would then be merged into master, once the functionality is completed.
We will create different issues for breakdown items.

The issue stated above could be broken down into the following phases:
P1: Add an import excel button beside the Add new application button.
P2: On clicking the button, a modal should appear containing mandatory fields like job and upload excel file
P3: Once the import is started, all applications should be mapped in the hr_applications table
P4: The path of the resume corresponding to each application will be saved in the resume column in the database

This issue is completed. Closing it.


fix: join actor when dropping

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
Ensure actor task is completely dropped after calling drop_all_actors, see https://github.com/risingwavelabs/risingwave/issues/7208#issuecomment-1375060881.
Also remove unused method wait_all.
Checklist
- [ ] I have written necessary rustdoc comments
- [ ] I have added necessary unit tests and integration tests

[ ] All checks passed in ./risedev check (or alias, ./risedev c)

Documentation
If your pull request contains user-facing changes, please specify the types of the changes, and create a release note. Otherwise, please feel free to remove this section.
Types of user-facing changes
Please keep the types that apply to your changes, and remove those that do not apply.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note
Please create a release note for your changes. In the release note, focus on the impact on users, and mention the environment or conditions where the impact may occur.
Refer to a related PR or issue link (optional)

Blocked by https://github.com/madsim-rs/madsim/issues/99.


Latest version on npm

Is it possible to push the latest version with the updated snake casing to npm?

v0.0.4 has been published to NPM


Implemented spike version for video content type support

refs https://linear.app/tryghost/issue/CORE-121/create-a-video-storage-adapter
refs https://github.com/TryGhost/Toolbox/issues/95

Example source files downloaded from https://filesamples.com/
This is an experimental implementation of video file upload support
The storage adapter still needs more thinking as it's almost the same as the "LocalStorgeAdapter" that stores images.
Also the output serializer skipped use of url utils in favor of inline implementatoin - this should almost certainly be it's own package

There will be follow up work with /media/ endpoint that will be done directly in main


update frequency

wondering how often the new definitions are pulled? the last version was published 6th Dec but is not detected on your service yet.

Seems they changed the structure of the downloads page since last week. Updated and redeployed so everything should work fine again now.


ARROW-8591: [Rust] Reverse lookup for a key in DictionaryArray

This PR enables reverse lookup for already built dict.

Do you have any ETA for merging this PR? Currently, I am using my local copy but I want to update to the git dep.

Was just giving other committers a chance to review.  If there is no update, I'll merge later today.


Applying LogReaderRBAC.yaml doesn't seem to work anymore?

Hi, I have followed the instructions on this page lots of times (as you can see by all the issues I created :)). I did it again today for a new cluster running 1.14.6, but when I try to apply the LogReaderRBAC.yaml file, I get an error "Error from server (NotFound): the server could not find the requested resource", nothing more. Am I doind something wrong?

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

ID: a113bacb-b999-8f0d-a66b-748e60e5c4d7
Version Independent ID: d7766575-5ef8-4f9a-49e5-2dbd7696d18e
Content: View Azure Monitor for containers logs in real time
Content Source: articles/azure-monitor/insights/container-insights-live-logs.md
Service: azure-monitor
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

Nevermind, I needed to update my kubectl version.

@mrdfuse Glad to know that you have self resolved this GitHub issue.

@mrdfuse So closing this issue for now. If there are further / any questions regarding this matter or feedback on the Azure documentation, please do let us know and we will gladly continue the discussion.


Ansible fails to run in nix-shell, when other package (yq) is also a dependency.

Issue description
When combining ansible and yq packages in a nix-shell, ansible commands will fail. This looks to be due to ansible (a python2 project) attempting to use pyyaml pulled in with yq (a python3 project).
Steps to reproduce
The following works:
nix-shell --pure -p ansible --run 'ansible --version'

The following doesn't work:
nix-shell --pure -p ansible yq --run 'ansible --version'

It fails with the following error:
Traceback (most recent call last):
  File "/nix/store/a8rxqzhrhcq4y5ck67rh3ssm9kx81yyy-python2.7-ansible-2.6.2/bin/.ansible-wrapped", line 68, in <module>
    import ansible.constants as C
  File "/nix/store/a8rxqzhrhcq4y5ck67rh3ssm9kx81yyy-python2.7-ansible-2.6.2/lib/python2.7/site-packages/ansible/constants.py", line 17, in <module>
    from ansible.config.manager import ConfigManager, ensure_type, get_ini_config_value
  File "/nix/store/a8rxqzhrhcq4y5ck67rh3ssm9kx81yyy-python2.7-ansible-2.6.2/lib/python2.7/site-packages/ansible/config/manager.py", line 14, in <module>
    from yaml import load as yaml_load
  File "/nix/store/d14wyhghrdkqr660hbcqmlvh6lkip80m-python3.6-PyYAML-3.13/lib/python3.6/site-packages/yaml/__init__.py", line 284
    class YAMLObject(metaclass=YAMLObjectMetaclass):
                              ^
SyntaxError: invalid syntax

Technical details
Please run nix-shell -p nix-info --run "nix-info -m" and paste the
results.
 - system: `"x86_64-darwin"`
 - host os: `Darwin 17.7.0, macOS 10.13.6`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.0.4`
 - channels(root): `""`
 - channels(jasper): `""`
 - nixpkgs: `/Users/jasper/dev/nixpkgs`

I'm running this against the latest master commit on nixpkgs at the time of writing (463704a62e3564b951eff84909eeffea4d728130).

Yeah this is kind of awkward, but I don't think there is really an easy fix.
As a workaround, reseting PYTHONPATH should work:
nix-shell --pure -p ansible yq --run 'PYTHONPATH= ansible --version'

nix-shell is intended for opening a shell to develop a package. In this case, you would like to simply have a shell with the listed packages on your $PATH. In that case, use nix run. Contrary to nix-shell, this won't execute any setup hooks, and thus PYTHONPATH won't be exported.
$ nix run nixpkgs.ansible nixpkgs.yq -c ansible --version


Move scala3doc branch to dotty-staging/scala3-example-project

@abgruszecki

dotty-staging/scala3-example-project is ahead in commit history


This causes a compile error in the cast

https://github.com/launchbadge/hedera-sdk-go/blob/611814074e31eaff7491878ca6add184cf0d12f7/transaction.go#L35
changing to
C.hedera_transaction_set_fee(tx.inner, C.ulonglong(fee))
fixed it for me

Hi Sam,
Sorry I missed your email earlier.
I would get this:
github.com/hashgraph/hedera-sdk-go
......\src\github.com\hashgraph\hedera-sdk-go\transaction.go:35:45: cannot use _Ctype_ulong(fee) (type _Ctype_ulong) as type _Ctype_ulonglong in assignment
So I change line 35 of transaction.go from:
            C.hedera_transaction_set_fee(tx.inner, C.ulong(fee))

to:
            C.hedera_transaction_set_fee(tx.inner, C.ulonglong(fee))

and it works!
Regards,
Michel
From: Sam Ulrich<EMAIL_ADDRESS>Sent: maandag 26 augustus 2019 21:18
To: launchbadge/hedera-sdk-go<EMAIL_ADDRESS>Cc: MichelBrinckman<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [launchbadge/hedera-sdk-go] This causes a compile error in the cast (#40)
Can you post the compiler error that you get? I can't reproduce the issue on my end
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/launchbadge/hedera-sdk-go/issues/40?email_source=notifications&email_token=ADCDNYWTXTCTKZZXMD4TAL3QGQT5LA5CNFSM4IO6GKUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5FMDKQ#issuecomment-524992938 , or mute the thread https://github.com/notifications/unsubscribe-auth/ADCDNYXABLGECP3DV7RPWKLQGQT5LANCNFSM4IO6GKUA .  https://github.com/notifications/beacon/ADCDNYULAJ43SLBT2ZAVNC3QGQT5LA5CNFSM4IO6GKUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5FMDKQ.gif


Could not find theme file error

@mrodalgaard reported:
Could not find theme file error is received from Alfred when trying to use alfred-atom, probably because it is looking for a theme file that does not exist.
I also tried to add an empty json file at the specific location and that removed the error but alfred-atom output was still empty.
Hugo version: 1.2.0 used by alfred-atom 2.3.2
Alfred version: 3.6.1
NodeJS version: v9.5.0
Exception / Stacktrace:
[2018-04-09 15:43:23][STDERR: input.scriptfilter] Could not find theme file "/Users/martin/Library/Application Support/Alfred 3/Alfred.alfredpreferences/themes/theme.bundled.default/theme.json"
Could not find theme file "/Users/martin/Library/Application Support/Alfred 3/Alfred.alfredpreferences/themes/theme.bundled.default/theme.json"

Turns out it was me using the old syntax for atom projects. I added a PR #20 with info for anyone else experiencing this issue.


LruCache returns partial record for QUERY_ROOT resulting in cache lookup failure

Summary
I am trying to load a record from the Apollo cache.  I know the record exists in the SQL cache.  However, LruNormalizedCache has an incomplete record for QUERY_ROOT that it returns from loadRecord so the entire cache lookup fails.  Steps:

Load FooQuery such that there are more than maxEntries things inserted into the lruCache
Perform a small mutation.  LruNormalizedCache will insert this new incomplete QUERY_ROOT because performMerge doesn't check nextCache before storing the result
Try to load FooQuery from the cache (apolloClient.apolloStore().read(query).execute()) and get an exception that the record couldn't be loaded.

I'm going to try to write a testcase that shows this bug but I'm having trouble importing this project into AS.
Version
1.3.2

Hi 👋 . Indeed, I don't think we have such a test case at the moment. Maybe it's working on dev-3.x, maybe the bug is still there. In all cases we should add a test case. If you feel like it, the IDE situation is way better now than it was one year ago.
You can open the apollo-android/tests composite project in a recent IntelliJ and it should work fine. Feel free to send a PR. Or if it's been way too long since last year, we'll get on it at some point.


Fix the flaxy tests (generally ComposeTestRule)

See https://github.com/google/horologist/issues/407
Identify why these are flaky in CI. Find solutions for this and fix.

Moving to Robolectric


MAT-1932

To convert Maccor protocol files to Biologic Protocol files we will need to be able to control what I'm calling "procedural steps" to sequences as opposed to the other type of Maccor step - "control steps".
Control steps are any of the following:

Advancing Cycle Number
Looping
Do statements (beginning a cycle)

All other steps are procedural steps.
Procedural steps have a "control" portion to them, in that when a <ReportEntry> limit is reached they will go to some other arbitrary step. Since the mapping of steps to sequences is not 1-1 we'll need a more global context about the correspondence between step number from the input and sequence number in the output.
This behavior will be handled separately in a PR that's pending for the actual conversion.

@patrickherring-TRI
Could I have some guidance on how test coverage works? Will I need test cases for every single error path to pass that check?

@willpowelson All of the checks should pass, with the possible exception of coverage. In terms of test cases, I think the most important thing to capture in the tests are the intended functionality, not necessarily focusing on every method having a test (although that can be nice). If there are errors that you can foresee, having some kind of assert might help to catch problems in the future, but you don't need an exhaustive list of errors.

@willpowelson Regarding the test coverage, I think the higher level philosophy is test driven development, writing the test as part of writing the code. But I think that's pretty similar to your normal workflow :)


Would there be a way for Vue.js to sense that a given slot is being rendered in a list and/or that it has multiple entries, like so?
Child.vue:
<template>

      <slot name="foo"></slot>

</template>

And elsewhere we would have:
<child>

Lorem ipsum dolor sit amet

Hello, World!

</child>

The rendered code would be exactly the same as mentioned above. I think this would greatly simplify an uncommon but useful practice.

I think the proposed syntax is quite confusing - <slot name="foo"></slot> is static inside each loop, it doesn't make use of item; and what does item represent?
I think this is a use case where a render function would make much more sense since you get full JavaScript flexibility. I really don't think adding more micro syntax to slots is a good idea.


Search Results - Sort by date - both ascending and descending

Currently, the planned functionality of the "date" sorting is only be most recent, - while this should be the default, we should also provide an option to sort by earliest. Haven't thought too far into how this should be displayed - there are a few options - so feel free to check in with me (Sam) about this!

Hey @davidascher @andreadelrio - I've attached an image to show the revised sorting box in the sidebar. First, we want to give it a title, "Sort Results by"
Second, let's change the language from "Most recent" and "Alphabetical" to "Date Published" and "Title (A–Z)" respectively.
These links should re-sort on click.. as default click - sort by most recent, or "A" first. On second click, toggle to sort by earliest, or by "Z" first.
The arrows should also give users an indication of sort order - down arrow active - correlating with "most recent" and "A first", up arrow active - correlating with "earliest" and "Z first"
Also, changing 'alphabetical' to 'title (a-z)' gives a more clear indication of what we're actually sorting.
Let me know if you have questions, thoughts, or suggestions on this!

Looks good.  Confirming we want to sort by Date Published, not Date Last Edited?
When sorting descending alphabetically, do we want to update the text to say Title (Z-A)?
I will say that Date published 'up' is a bit opaque to me. Do you agree? How about something like

Date Published [no selection]
Most recently published [down]
Least recently published [up]

?

I'm not satisfied w/ my suggestion.


SPEC: define wire-compatible changes

We should call out what modifications are allowed while retaining wire and language-level API compatibility. For instance, adding new methods does not break compatibility with the service and existing methods, but obviously servers won't support the new method. Changing a protobuf message name is safe on-the-wire but not safe for generated code compatibility, and it also isn't safe in metadata since the name is used as a key. We've also heard the question of whether it is safe to change a unary request/response into a streaming request/response, which is semi-safe on-the-wire but not safe for generated code compatibility.
@louiscryan, @a11r

@ejona86 @a11r do we have this documented somewhere? Thanks!

This should remain open.

@a11r ping. Also, is this a P1?


Add support for resize column to fit content

Originally from #1923
We should do some evaluation on data tables to determine if this makes sense in the Datagrid context. Some of the interactions described feel more akin to a DataSpreadsheet.
Desired behavior

Hover over the right side of the column header until a horizontal split cursor appears.
Double-click the column line.
The column is resized to fit the contents.

### Tasks
- [ ] Evaluate whether this interaction is appropriate for data tables (or data spreadsheets)
- [ ] Work with design to define a comparable experience for keyboard users

We've been investigating TanStack Table, a third-party, open-source offering, which provides extensive data table capabilities surpassing what our Carbon Datagrid offers. It provides much more flexibility and customization. TanStack Table is headless which means it can easily be added alongside Datagrid component in your product or application. The benefits of more flexibility for product teams and less maintenance for Carbon makes it a win win. Lastly, it is available in multiple frameworks including React and Web Component so it provides an option to non-React product teams.
For these reasons, we have decided to transition from building our own custom table component to using an example-based approach with TanStack Table. Datagrid will still exist in our library for existing teams but we are announcing the deprecation* of the Datagrid component in v2.54.0 release so teams can begin to work through the transition. Details about how to use both Datagrid and TanStack together can be found here.
*Deprecation means that no new features will be added however sev 1 and sev 2 bugs will be supported.


fix RTD

[Current coverage][cc-pull] is 57.35%

Merging [#662][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #662   diff @@
==========================================
  Files            63         63
  Lines         10280      10280
  Methods           0          0
  Messages          0          0
  Branches       1667       1667
==========================================
  Hits           5896       5896
  Misses         3824       3824
  Partials        560        560

Powered by Codecov. Last updated by [03b51a5...fa84b4c][cc-compare]
[cc-base-branch]: https://codecov.io/gh/crossbario/autobahn-python/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/crossbario/autobahn-python/compare/03b51a5e73493df8edb8b8768d81d2c28fdc4533...fa84b4c79694d4a88dd18d1381ffa12fb9c0562f
[cc-pull]: https://codecov.io/gh/crossbario/autobahn-python/pull/662?src=pr


Cachefilesd: implement a cachefilesd handler for KVC.

@anjalisood @dzungductran why should this be a handler in KVC? Isn't the daemon supposed to be installed in each nodes in the cluster as a part of installation.

/assign @anjalisood

This is no longer possible with KVC

I tried to do a POC and was not able to get the cachefilesd installed and running in a container. This is because systemctl and systemd are not available within ubuntu/centos container images. This is by design. Docker should be running a process in the foreground in the container and it will be spawned as PID 1 within the container's pid namespace. Docker is designed for process isolation, not for OS virtualization, so there are no other OS processes and daemons running inside the container (like systemd, cron, syslog, etc), only the entrypoint or command that is run. If they included systemd commands, we would find a lot of things not working since the entrypoint replaces init. Systemd also makes use to cgroups which docker restricts inside of containers since the ability to change cgroups could allow a process to escape the container's isolation. Without systemd running as init inside the container, there's no daemon to process the start and stop commands.
And without systemd and systemctl commands in the container, it is not possible to have the cachefilesd installed in the container.


Signature spoofing patch doesn't work for Android 7.1.1

Hi,
I want to build a custom ROM based on Android 7.1.1... Applying the signature spoofing patch from here (https://github.com/microg/android_packages_apps_GmsCore/tree/master/patches) worked fine for Android 6 and 7.0, however for 7.1.1 I receive the following error:
Prüfe Patch core/res/AndroidManifest.xml ...
Patch-Bereich #1 erfolgreich angewendet bei 1903 (249 Zeilen versetzt)
Prüfe Patch core/res/res/values/config.xml ...
Patch-Bereich #1 erfolgreich angewendet bei 1267 (-31 Zeilen versetzt)
Prüfe Patch core/res/res/values/strings.xml ...
Patch-Bereich #1 erfolgreich angewendet bei 664 (48 Zeilen versetzt)
Prüfe Patch services/core/java/com/android/server/pm/PackageManagerService.java ...
error: bei der Suche nach:
final Set permissions = permissionsState.getPermissions(userId);
final PackageUserState state = ps.readUserState(userId);
    return PackageParser.generatePackageInfo(p, gids, flags,
            ps.firstInstallTime, ps.lastUpdateTime, permissions, state, userId);
}

@Override

error: Anwendung des Patches fehlgeschlagen: services/core/java/com/android/server/pm/PackageManagerService.java:2816
error: services/core/java/com/android/server/pm/PackageManagerService.java: Patch konnte nicht angewendet werden
Can you give me a hint how to fix this?
Thanks in advance and best regards.

OmniROM has the patch active in their 7.1 branch (see this commit). What ROM are you trying to patch? There might be slight modifications in the ROM that cause the patch to fail.

I tried to patch CopperheadOS...

@uiopc try the patch I just uploaded (https://github.com/microg/android_packages_apps_GmsCore/blob/master/patches/android_frameworks_base-N.patch)

Thanks a lot for your quick help :) Applying the patch works fine...


Cannot open display when running with a user space switch and innamespace

When running mininet with a userspace switch and with the --innamespace argument, X forwarding stops working!
For example, wireshark returns the following error:
** (wireshark:6096): WARNING **: Could not open X display
error: XDG_RUNTIME_DIR not set in the environment.

(wireshark:6096): Gtk-WARNING **: cannot open display: localhost:11.0

xterm return a similar error.
How to reproduce:
sudo mn --topo single,3 --mac --switch user --controller remote --innamespace

Works fine for me in 2.2.1.

Here is some more info:
mininet@mn-vm:~$ sudo mn --version
2.2.1
mininet@mn-vm:~$ sudo mn --switch user --innamespace --topo linear,2,2
*** Creating network
*** Adding controller
*** Adding hosts:
h1s1 h1s2 h2s1 h2s2
*** Adding switches:
s1 s2
*** Adding links:
(h1s1, s1) (h1s2, s2) (h2s1, s1) (h2s2, s2) (s2, s1)
c0 <-> s1 s2
*** Testing control network
s1 -> c0
c0 -> s1
*** Results: 0% dropped (2/2 received)
s2 -> c0
c0 -> s2
*** Results: 0% dropped (2/2 received)

*** Configuring hosts
h1s1 h1s2 h2s1 h2s2
*** Starting controller
c0
*** Starting 2 switches
s1 s2
*** Starting CLI:
mininet> h1s1 wireshark

(process:14103): Gtk-WARNING **: Locale not supported by C library.
	Using the fallback 'C' locale.

** (wireshark:14103): WARNING **: Could not open X display
error: XDG_RUNTIME_DIR not set in the environment.

(wireshark:14103): Gtk-WARNING **: cannot open display: localhost:10.0
mininet> s1 wireshark

(process:14894): Gtk-WARNING **: Locale not supported by C library.
	Using the fallback 'C' locale.

** (wireshark:14894): WARNING **: Could not open X display
error: XDG_RUNTIME_DIR not set in the environment.

(wireshark:14894): Gtk-WARNING **: cannot open display: localhost:10.0
mininet> xterm h1s1

So with the user space switch I can not start wireshark from mininet. xterm works though, this is definetely not an X forwarding problem.
mininet@mn-vm:~$ sudo mn --switch ovsk --topo linear,2,2
*** Creating network
*** Adding controller
*** Adding hosts:
h1s1 h1s2 h2s1 h2s2
*** Adding switches:
s1 s2
*** Adding links:
(h1s1, s1) (h1s2, s2) (h2s1, s1) (h2s2, s2) (s2, s1)
*** Configuring hosts
h1s1 h1s2 h2s1 h2s2
*** Starting controller
c0
*** Starting 2 switches
s1 s2 ...
*** Starting CLI:
mininet> h1s1 wireshark

(process:14583): Gtk-WARNING **: Locale not supported by C library.
	Using the fallback 'C' locale.

** (wireshark:14583): WARNING **: Could not open X display
error: XDG_RUNTIME_DIR not set in the environment.

(wireshark:14583): Gtk-WARNING **: cannot open display: localhost:10.0
mininet> s1 wireshark

(process:14601): Gtk-WARNING **: Locale not supported by C library.
	Using the fallback 'C' locale.

** (wireshark:14601): WARNING **: Couldn't connect to accessibility bus: Failed to connect to socket /tmp/dbus-yUpoTM8Br0: Connection refused
Gtk-Message: Failed to load module "canberra-gtk-module"

With the ovs switch, I can start wireshark on the switch itself, although as you see it prints some warnings, but it then works! It is still not possible to start it on a host.
From what I understand, the userspace switch does not support starting wireshark, nor do the hosts. The ovs switch on the other hand, does!
This is not a big problem as I can still open a second SSH console and start wireshark directly on the VM, but it is less convinient!

 happened with me as well, any resolutions?

hhimanshu, the image of Mininet 2.2.1 on Ubuntu 14.04 LTS is a server version.
Maybe you need this.

mininet@mininet-vm:~$ wireshark &
[1] 1315
mininet@mininet-vm:~$
(process:1315): Gtk-WARNING **: Locale not supported by C library.
       	Using the fallback 'C' locale.

(wireshark:1315): Gtk-WARNING **: cannot open display:

[1]+  Exit 1                  wireshark

I got same error too on mininet2.2.1

Same problem with me. My version of mininet is 2.2.2.
Does anybody knows how to fix this problem?

Is this thread alive?

Verified fixed in master.
$ sudo mn --topo single,3 --mac --switch user --innamespace -v output
s1 -> c0
c0 -> s1
*** Results: 0% dropped (2/2 received)
mininet> s1 ping -c1 c0
PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from <IP_ADDRESS>: icmp_seq=1 ttl=64 time=0.050 ms

--- <IP_ADDRESS> ping statistics ---
1 packets transmitted, 1 received, 0% packet loss, time 0ms
rtt min/avg/max/mdev = 0.050/0.050/0.050/0.000 ms
mininet> xterm h1

works as expected.

i have a problem, help please, i can't open wireshark and miniedit with my virtual machine, i need your help please
mininet@mininet-vm:~$ mininet@mininet-vm:~$ wireshark -sb
mininet@mininet-vm:~$: command not found
mininet@mininet-vm:~$ QXcbConnection: Could not connect to display
QXcbConnection:: command not found
mininet@mininet-vm:~$ Aborted
Aborted: command not found


⚠ Processors has degraded performance

In 6b81a1b, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 86 ms

Resolved: Processors performance has improved in 1f0bf7e.


afl-clang*-fast toolchains built with clang from source fails test_build

Hi, I was trying to build afl-clang*-fast toolchains using clang built from source.The compiler wrappers do build successfully, however they fail at the test_build with the following error:
clang (LLVM option parsing): Unknown command line argument '-sanitizer-coverage-block-threshold=0'.  Try: 'clang (LLVM option parsing) --help'
clang (LLVM option parsing): Did you mean '  --sanitizer-coverage-pc-table=0'?

The error does indeed stop occuring if I make the mentioned fix in afl-clang-fast.c. However, from what I understand these are two completely different flags. The original flags is associated with inlining instrumentation callbacks and the pc-table flag refers to a feature where PC's corresponding to function/non-function entry blocks is made as a pre-processing step.
My clang version which I built from source is:
clang version 10.0.0 (git@github.com:llvm-mirror/clang.git 65acf43270ea2894dffa0d0b292b92402f80c8cb) (git@github.com:llvm-mirror/llvm.git 2c4ca6832fa6b306ee6a7010bfb80a3f2596f824)
Target: x86_64-unknown-linux-gnu

The command I am using to build llvm_mode is: AFL_TRACE_PC=1 CC=/path/to/built/clang CXX=/path/to/built/clang++ make -C llvm_mode clean all

For reproducibility, the commit hashes of projects I used to build my clang toolchains from source are provided below:
llvm: 2c4ca6832fa6b306ee6a7010bfb80a3f2596f824
clang: 65acf43270ea2894dffa0d0b292b92402f80c8cb
compiler-rt: 69445f095c22aac2388f939bedebf224a6efcdaf
lld: 64b024a57c56c3528d6be3d14be5e3da42614a6f

@lszekeres Laszlo, are you familiar with this by any chance?

@prashast remove all -fsanitize-coverage* options and replaced them with this single one:
-fsanitize-coverage=trace-pc-guard and you are set to go. Note that this instrumentation is slower than the native llvm_mode one

@vanhauser-thc Thanks! Yeah, I think just simply commenting out -sanitizer-coverage-block-threshold makes it work.
My initial understanding of this flag was mistaken as well. From the discussion here, this flag helps in inlining if (*guard) checks which were initially used in __sanitizer_cov_trace_pc_guard. However, the current pc-guard instrumentation does not have this if check so I think we can safely remove this flag from afl-clang-fast.c without any observable performance drop. Let me know if you would want me to make a pull request making this change.

@prashast the reason it didnt compile for you was that the llvm version you are using is not supported in this project. (only up to 4.0, however it works until 6.0 and with tricks until 8).
So IMHO if a PR starts supporting other llvm version, the "normal" instrumentation of llvm_mode should also be expanded to support that version

@vanhauser-thc I see. I noticed how AFLPlusPlus already accounts for varying LLVM versions in its default instrumentation pass here
I guess, in that case its just a matter of taking a decision whether to relay the changes in your fork to this repo as well. :)


Feature Request: EXPIRED/NOEXPIRED option on set_password and set_passphrase

Is your feature request related to a problem? Please describe.
set_password and set_passphrase options currently don't have the option to set non-expired passwords.
Describe the solution you'd like
We should add an optional argument to these functions to make the password non-expired.
Describe alternatives you've considered
N/A
Additional context
N/A

Resolved by #55