The link "AD FS Scenarios for Developers" in first paragraph is broken

The url https://docs.microsoft.com/en-us/windows-server/identity/ad-fs/overview/ad-fs-scenarios-for-developers doesn't exist, perhaps it should be https://docs.microsoft.com/en-us/windows-server/identity/ad-fs/ad-fs-development ??

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

ID: 2f214c1c-fe2d-6034-ddcb-27a2e398d0ee
Version Independent ID: 1957e1d2-85a9-e9cb-d067-34dc7edc4bb6
Content: AD FS support in Microsoft Authentication Library for .NET - Microsoft identity platform
Content Source: articles/active-directory/develop/msal-net-adfs-support.md
Service: active-directory
Sub-service: develop
GitHub Login: @TylerMSFT
Microsoft Alias: twhitney

@kiwiswift
Thanks for your feedback! We will investigate and update as appropriate.

@kiwiswift thanks for this! I made a pull request to fix the link. https://github.com/MicrosoftDocs/azure-docs-pr/pull/94565


Few shot forecasting

Issue #, if available:
Description of changes:
Add code of internship project "Probabilistic Few-Shot Prediction" to nursery.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Please tag this pr with at least one of these labels to make our release process faster: BREAKING, new feature, bug fix, other change, dev setup

@lostella: Not sure what happended to the previous merge since I can't find the package in the default branch. I updated the pull to pass the type and style checks.

@RingoIngo thanks for reopening! I think it just fell off our radar


Module's source code (annotation) no longer used as testAnnotationProcessor for its test?

I ran into this issue trying to upgrade gradle from 4.10.2 to 5.0.
Given a project as follows
A/src/main/MyImplAnnotation.java
A/src/test/MyObject.java // relies on `MyImplAnnotation.java` to generate `MyObject_Impl.java`
A/src/test/Test.java // does something with `MyObject_Impl`

For Gradle < 5.0, this works.
For 5.0+, there would be a ClassNotFound error on MyObject_Impl because project's source code does not seem to/cannot be used as annotation anymore for test sources.
Could you kindly clarify whether this is the expected behavior or a bug?
Workaround
My current workaround is to move the tests into a separate module B as follows:
A/src/main/MyAnnotation.java
B/src/test/MyObject.java // relies on `MyAnnotation.java` to generate `MyObject_Impl.java`
B/src/test/Test.java (does something with `MyObject_Impl`)

Then have B's build.gradle contain:
testAnnotationProcessor project('A')

I believe this is because testAnnotationProcessor is empty. In 4.10, compileTestJava.options.annotationProcessorPath would then default to the testCompileClasspath, which would then include the annotation processor from the main source set. In 5.0 though, this fallback no longer happens, so the tests are compiled without any annotation processor.
You'd want to have testAnnotationProcessor extends runtimeClasspath or testImplementation; or something like that.

Thanks @tbroye! That seems to do the trick.
I'd still like someone from gradle to clarify whether this is intended tho.

Yes, this intended, Gradle ignores annotation processors on the compile classpath.


Add header and stub for User Messaging Platform SDK.

Description

Provide details of the change, and generalize the change in the PR title above.

Testing

Describe how you've tested these changes. Link any manually triggered Integration tests or CPP binary SDK Packaging Github Action workflows, if applicable.

Type of Change
Place an x the applicable box:

[ ] Bug fix. Add the issue # below if applicable.
[ ] New feature. A non-breaking change which adds functionality.
[ ] Other, such as a build process or documentation change.

Notes

Bug fixes and feature changes require an update to the Release Notes section of release_build_files/readme.md.
Read the contribution guidelines CONTRIBUTING.md.
Changes to the public API require an internal API review. If you'd like to help us make Firebase APIs better, please propose your change in a feature request so that we can discuss it together.

The public API looks good, I added several suggestions to make the method documentation closer to the phrasing of iOS and Android docs. (I will update the API proposal comments similarly shortly)

Thanks - I made all the changes you mentioned. PTAL.


Instantmotiontracking cannot use obj format files，How to make uuu file format ？

The .uuu format is a simple custom binary format designed to be trivially rendered using OpenGL, so sequenced .obj animations/files (or any other asset format) will need to be parsed into this format in order to be used by MediaPipe.  The LoadAnimation and LoadAnimationAndroid functions in gl_animation_overlay_calculator.cc can be used as reference for how the .uuu format is expected to be loaded.  In brief, the format expects an arbitrary number of animation frames, concatenated directly together, with each animation frame looking like:
HEADER
VERTICES
TEXTURE_COORDS
INDICES
The header consists of 3 int32 lengths, the sizes of the vertex data, the texcoord data, and the index data, respectively.  Let us call those N1, N2, and N3.  Then we expect N1 float32's for vertex information (x1,y1,z1,x2,y2,z2,etc.), followed by N2 float32's for texcoord information (u1,v1,u2,v2,u3,v3,etc.), followed by N3 shorts/int16's for triangle indices (a1,b1,c1,a2,b2,c2,etc.).  This is precisely the data that a glDrawElements call expects, when drawing triangles.

If I want to use my own obj file, just add the file header?

Unfortunately, no-- you'll need to use a script or program to parse the .obj file for the necessary information, and output it into the format above.

Therefore, we cannot modify the effect we want in this project. Can you open source the script?

@tyrmullen   followed by N3 shorts/int16's for triangle indices (a1,b1,c1,a2,b2,c2,etc.).
I found this line to say the uuu model got vertex indices.  I‘m not sure which I need to use.
I think maybe is the second one because the uuu format caculate normal information by code not load from uuu file.
Am I right?

@tyrmullen   followed by N3 shorts/int16's for triangle indices (a1,b1,c1,a2,b2,c2,etc.).
I found this line to say the uuu model got vertex indices.  I‘m not sure which I need to use.
I think maybe is the second one because the uuu format caculate normal information by code not load from uuu file.
Am I right?

With the next update of MediaPipe, we're hoping to release our internal obj --> uuu processing tools; so hopefully this will clarify the precise nature of our formats in full detail.

Hi @mmm2016, Could you please respond if you still looking resolution for the above query. Thanks!


chore: update docs

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.04%) to 80.698%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
4787

Relevant Lines:
5932

💛  - Coveralls


Add support Symfony 4

Symfony 4 released few weeks ago. Let's add support for it.

Added support of Symfony 4


Getting warning from ReactDOM when using React 18

Hello,
When upgrading to React 18 and using single-spa-react with the following configuration:
import React from 'react';
import ReactDOM from 'react-dom';
import singleSpaReact from 'single-spa-react';

import Root from './root.component';

const lifecycles = singleSpaReact({
  React,
  ReactDOM,
  rootComponent: Root,
  renderType: 'createRoot',
});

I get the following warning in the console: Warning: You are importing createRoot from "react-dom" which is not supported. You should instead import it from "react-dom/client".
However, the single-spa-react api is requiring ReactDOM to be passed. Would it be better to pass the createRoot function here or drop the requirement of passing ReactDOM when using React 18?

I unfortunately don't have time to today to setup an example project with React 18. If you can share one, that'd be a huge help. But looking at how we're using ReactDOM within single-spa-react, I think a possible workaround could be to import from react-dom/client, alias as ReactDOM, and pass that in.
import React from 'react';
- import ReactDOM from 'react-dom';
+ import * as ReactDOM from 'react-dom/client';
import singleSpaReact from 'single-spa-react';

import Root from './root.component';

const lifecycles = singleSpaReact({
  React,
  ReactDOM,
  rootComponent: Root,
  renderType: 'createRoot',
});

Or more verbosely but matches the new import syntax here.
import React from 'react';
- import ReactDOM from 'react-dom';
+ import * as ReactDOMClient from 'react-dom/client';
import singleSpaReact from 'single-spa-react';

import Root from './root.component';

const lifecycles = singleSpaReact({
  React,
-  ReactDOM,
+  ReactDOM: ReactDOMClient,
  rootComponent: Root,
  renderType: 'createRoot',
});

If that works, we can document or update the api change for React 18.

As the newest code shows, single-spa-react has already supported React with renderType set to createRoot, but it seems ReactDOM definition still remains old?

TS2740: Type 'typeof import("**/node_modules/@types/react-dom/client")' is missing the following properties from type 'typeof import("**/node_modules/@types/react-dom/index")': findDOMNode, unmountComponentAtNode, createPortal, flushSync, and 5 more.

I think I can run a demo to see if it this library is ready for React v18+?

@filoxo I test it with import ReactDOM from 'react-dom/client';, and it shows<EMAIL_ADDRESS>supports it in single-spa's rendering, but as I said above, maybe the type of ReactDOM should be changed to support React v18+, so that I don't need to add @ts-ignore comments:
singleSpaReact({
      React,
      // @ts-ignore
      ReactDOM,
      renderType: 'createRoot',
      ...
    }),

Oh right, TypeScript. Those can be updated here: https://github.com/single-spa/single-spa-react/blob/main/types/single-spa-react.d.ts#L9

Oh right, TypeScript. Those can be updated here: https://github.com/single-spa/single-spa-react/blob/main/types/single-spa-react.d.ts#L9

It seems type of ReactDOM may be ambiguous from two different source, maybe it can be passed from RootComponentProps and not infer from typeof? Or simply, I searched current single-spa-react, and it only uses createRoot in mount lifecycle, maybe additional union type can also work?
export function createRoot(container: Element | DocumentFragment, options?: RootOptions): Root;

ps: maybe some conditions should be updated again, since v18+ already is a stable version and functions like this https://github.com/single-spa/single-spa-react/blob/main/src/single-spa-react.js#L213 may not exist anymore?

Does anyone have an update on this? I'm still getting the error shown below—FYI I'm using layout engine. If I check the network tab, I can see the ReactVersion = 18. I've also listed my package.json and render code:

...
"react": "^18.0.0",
"react-dom": "^18.0.0",
...

import React from "react";
import * as ReactDOMClient from "react-dom/client";
import singleSpaReact from "single-spa-react";
import Root from "./root.component";

const lifecycles = singleSpaReact({
  React,
  ReactDOM: ReactDOMClient,
  rootComponent: Root,
  renderType: 'createRoot',
  errorBoundary(err, info, props) {
    // Customize the root error boundary for your microfrontend here.
    return null;
  },
});

export const { bootstrap, mount, unmount } = lifecycles;

Please ignore my message above, I was able to get it to work with the above code, my issue, I didn't override the importmap to point to localhost 🤦

@filoxo I do believe you initial comment fixes the warning. I'm wondering if the type definition file single-spa-react.d.ts should also be updated?
import * as React from "react";
- import * as ReactDOM from "react-dom";
+ import * as ReactDOM from "react-dom/client";
import { AppProps, CustomProps, LifeCycleFn } from "single-spa";

I can open a PR if that's right

I dove into it this weekend and found we hadn't known about hydrateRoot until the final React 18 release so I'll be looking to support that and and tests. That should really only affect users of TypeScript any anyone using hydrateRoot which is very likely no one at this time.
I think I also learned that this change could be farther reaching if simply swapping out that import like I had initially done. That would mean a breaking change for this helper library which can actually be avoided as, syntactically, all of this code should still be able to work for prior versions of React too. Hopefully that's helpful information.

@filoxo Thanks for looking into this further. I think this issue can be closed for now since everything is working. Let me know if there's anything to help out with with the updates

Thank you, much appreciated! And thanks also to everyone else for your input too!
I'd rather keep this ticket open and use it to communicate further updates. I've got a WIP branch I've yet to push up, as tests are what are taking a bit longer to figure out (and validate that both versions of React can be supported). However, I don't want to come across as "I'm the only one qualified to do this" so this is an open call if anyone would like to contribute/collaborate with me on this one. We can use this issue for that too.

So in bringing in single-spa-react v5.0.0 and using the alternative ReactDOM import you suggested works for me and removes the error, but after reading through this a bit, I'm still a little lost. Is this supposed to be the fix or is there further work to be done? I ask because, afaik, most React devs will reflexively reach for the former style of import.

I still haven't merged in the documentation changes because I'm at home recovering from covid, but could you review them and see if they answer your question? https://github.com/single-spa/single-spa.js.org/pull/549 that'd really help me validate.

I also failed to update this issue so<EMAIL_ADDRESS>release notes here.


[question] svg format

I love this tool! 🥇
Question: How did you export humaaans into SVG format?
I only see Invision and Sketch on the website :)

As Pablo says you can download the free version of invisionapp and export as eps for use in a vector editing program.


Overlay overlapps the popup menu created by PopupMenuButton

I have a bug with menu overlapping. If OverlayEntry's widget is active and I activate popup menu with PopupMenuButton it is draw under the overlay which is bug for me.
P.S. Unfortunately I will not be able to present the code sample, only this video.
https://user-images.githubusercontent.com/45417992/205594093-2929af05-6a67-43e5-ad35-991dcc78348f.mp4

Hello @bambinoua. Thank you for filing this issue. Can you please provide the following information?

A complete and minimal, reproducible example. Ideally, a single main.dart file that can be run directly. You do not need to provide any proprietary code. You can replicate the behavior in a new Flutter project and provide the contents of the main.dart file that way.
The output of flutter doctor -v
The output of flutter run -v

Please provide the information in the form of text. See how to make collapsible sections with Markdown here.

I could create a reproduceable code.

Minimal reproduceable code
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      title: 'Bug',
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key});

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  final link = LayerLink();

  OverlayEntry? entry;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('Bug'),
      ),
      body: Column(
        children: [
          Padding(
            padding: const EdgeInsets.all(8.0),
            child: Row(
              mainAxisAlignment: MainAxisAlignment.spaceBetween,
              children: [
                ElevatedButton(
                  onPressed: () {
                    if (entry == null) {
                      entry = OverlayEntry(
                        builder: (context) {
                          return CompositedTransformFollower(
                            link: link,
                            showWhenUnlinked: false,
                            child: const Material(
                              color: Colors.amber,
                              child: Center(
                                child: Text('Overlay'),
                              ),
                            ),
                          );
                        },
                      );
                      Overlay.of(context)!.insert(entry!);
                    } else {
                      entry!.remove();
                      entry = null;
                    }
                    setState(() {});
                  },
                  child: Center(
                      child: Text(
                          entry == null ? 'Insert Overlay' : 'Remove Overlay')),
                ),
                PopupMenuButton(
                  child: Container(
                    color: Colors.blue,
                    padding: const EdgeInsets.all(6),
                    child: const Text('Show Menu'),
                  ),
                  itemBuilder: (context) => List.generate(
                    5,
                    (index) => PopupMenuItem(
                      value: index,
                      child: Text('Item $index'),
                    ),
                  ),
                ),
              ],
            ),
          ),
          Expanded(
            child: CompositedTransformTarget(
              link: link,
              child: Container(
                alignment: Alignment.center,
                color: Colors.grey.shade300,
                child: const Text('Overlay Zone'),
              ),
            ),
          )
        ],
      ),
    );
  }
}

flutter -v doctor
[√] Flutter (Channel stable, 3.3.9, on Microsoft Windows [Version 10.0.19045.2251], locale ru-RU)
• Flutter version 3.3.9 on channel stable at D:\Programs\flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision b8f7f1f986 (2 weeks ago), 2022-11-23 06:43:51 +0900
• Engine revision 8f2221fbef
• Dart version 2.18.5
• DevTools version 2.15.0

Triage report
I can reproduce this issue on Master (3.7.0-3.0.pre.57).

Screenshot

Code Sample (Same as OP)
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      title: 'Bug',
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key});

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  final link = LayerLink();

  OverlayEntry? entry;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('Bug'),
      ),
      body: Column(
        children: [
          Padding(
            padding: const EdgeInsets.all(8.0),
            child: Row(
              mainAxisAlignment: MainAxisAlignment.spaceBetween,
              children: [
                ElevatedButton(
                  onPressed: () {
                    if (entry == null) {
                      entry = OverlayEntry(
                        builder: (context) {
                          return CompositedTransformFollower(
                            link: link,
                            showWhenUnlinked: false,
                            child: const Material(
                              color: Colors.amber,
                              child: Center(
                                child: Text('Overlay'),
                              ),
                            ),
                          );
                        },
                      );
                      Overlay.of(context)!.insert(entry!);
                    } else {
                      entry!.remove();
                      entry = null;
                    }
                    setState(() {});
                  },
                  child: Center(
                      child: Text(
                          entry == null ? 'Insert Overlay' : 'Remove Overlay')),
                ),
                PopupMenuButton(
                  child: Container(
                    color: Colors.blue,
                    padding: const EdgeInsets.all(6),
                    child: const Text('Show Menu'),
                  ),
                  itemBuilder: (context) => List.generate(
                    5,
                    (index) => PopupMenuItem(
                      value: index,
                      child: Text('Item $index'),
                    ),
                  ),
                ),
              ],
            ),
          ),
          Expanded(
            child: CompositedTransformTarget(
              link: link,
              child: Container(
                alignment: Alignment.center,
                color: Colors.grey.shade300,
                child: const Text('Overlay Zone'),
              ),
            ),
          )
        ],
      ),
    );
  }
}

CC @LongCatIsLooong

Any news on this issue?

@exaby73 @darrenaustin any updates on this issue which still is present?

Any updates? I don't know what to do

Just run into this issue as well.
There should be some way to control if the PopupMenuButton:s menu appears below or above existing OverlayEntry:s.
The current behavior of appearing below them is not very useful, feels more like a bug, but if there is a need to keep it, then there needs to be another flag that makes sure the PopupMenuButton appears above them, since the seRootNavigator: true does not help.

There is no obvious solution to fix this issue at the framework level.
PopupMenuButton shows the popup menu by adding a route to the closest navigator. The navigator manages the corresponding overlay entries.
This is fine until the app also inserts a custom OverlayEntry to the same Overlay used by the navigator.
When doing so, the custom OverlayEntry will always appear on top of overlay entries created by routes.
I tried adding a way to configure this behavior in https://github.com/flutter/flutter/pull/160336 but it breaks many customer tests.
When possible Overlay.wrap workaround, see https://github.com/flutter/flutter/issues/116504#issuecomment-2528109973, should help.
Migrating to MenuAnchor is probably the best solution, see https://api.flutter.dev/flutter/material/PopupMenuButton-class.html which points out that MenuAnchor is the preferred solution for Material 3.

I tried the Overlay.wrap workaround, but could not get it to work in our use case, maybe I was using it wrong somehow.
I was also planning on giving the MenuAnchor a try. Need to make it look the same as our customized PopupMenu, but it is quite flexible, so I should if I should get it to look the same or close enough to replace it. Also MenuAnchor hilghted item on top and bottom item does not overflow in the corners like PopupMenu does when using high border radius. So I can fix that as well in our use case if I replace it.
I was not sure though if it makes a difference with Z-order, but I will take your word for it if you say it works, and give it a shot 😃

@rydmike

I tried the Overlay.wrap workaround, but could not get it to work in our use case, maybe I was using it wrong somehow.

If your use case is on a public repo, I can have a look to see if the workaround can be applied.

This is in a private repo with +400kLoC Flutter/Dart code. I have been thinking of migrating our popup menus to MenuAnchor based ones anyway, this is a good reason to start doing it.

MenuAnchor works when you use consumeOutsideTap: true, but it is lacking alignment features making it basically not a viable solution.
Here is why, in situation where you e.g. a IconButton on end side of screen, with some padding so it does not touch the end, and want the menu to open under it, so that its end side, is aligned with the end side of the icon button, there is no way to this. You can move it around alignmentOffset which is in relation to its MenuStyle.alignment, but the ref point from the Menu is always from start top, so if you want to align end sides, which you want on a menu on the end side, you need to know the width of the opened menu, so you can compute the correct offset, and you won't know that until it is opened since its width depends on translated strings.
So then what can you do? Well you are forced to render a copy of it Offstage in a Stack, with the actual menu and measure its width in post frame callback that you can then use to compute the offset for the menu. All very bad ideas and super tedious.
The MenuAnchor needs features for what it should use alignment ref point so you can set end side of menu and opening widget to align as well. I could not find any easy way to this in current implementation.

Thanks for the feedback.
There are ongoing work on the MenuAnchor side (mainly related to DropdownMenu issues).
The most important effort is this refactoring that will help to address more complex issues.
In https://github.com/flutter/flutter/pull/154667, I added a way to position the MenuAnchor using a leader/follower which makes it possible to set the attachment point of the MenuAnchor. Maybe it is worth trying it?
FYI, the MenuAnchor.layerLink is still there but its DropdownMenu usage was reverted because it lacks intelligent positionning. See https://github.com/flutter/flutter/pull/158930 for the revert.
I have also some pending work to make the leader/follower positionning adaptive but it won't be merged soon, see https://github.com/flutter/flutter/pull/157921 (which is a draft to get feedback, it addresses only the vertical positionning but it might possible to extend it to horizontal positionning, nonetheless this is very experimental).
Back to your PopupMenu issues, if you can find a way to makes the custom OverlayEntry created by your app show up on a different Overlay than the one used by the Navigator it should help (if there are several Navigator, PopupMenu.useRootNavigator is probably needed).


Makefile cupy-cuda chainer

In the Makefile, I am not sure what line 36 is trying to do.
I get the following error:
miniconda3/envs/eend/bin/pip install cupy-cuda10==6.2.0 chainer==6.2.0 ERROR: Could not find a version that satisfies the requirement cupy-cuda10==6.2.0 ERROR: No matching distribution found for cupy-cuda10==6.2.0

I guess you have cuda 10.0 or 10.2. The line#36 should try to get cupy-cuda100 or cupy-cuda102.
https://github.com/hitachi-speech/EEND/blob/ddf4df4da55a493f550b6c45689b2d60a716c06d/tools/Makefile#L10
This line might have an issue. I found the awk part should be changed as follows:
CUDA_VERSION := $(shell $(CUDA_PATH)/bin/nvcc --version | tail -n1 | awk '{print substr($$5,1,length($$5)-1)}')

I changed line 10 as recommended. I also modified line 36 to:
miniconda3/envs/eend/bin/pip install cupy-cuda$(subst .,,$(CUDA_VERSION)) chainer==6.2.0


Light/Dark Mode issues on older versions of Safari 12,13, and 14

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[ ] I have tried the npm run clear or yarn clear command.
[ ] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[ ] I have tried creating a repro with https://new.docusaurus.io.
[ ] I have read the console error message carefully (if applicable).

Description
Howdy! We are using docusaurus for the https://excaliburjs.com documentation site. Some of our users have reported issues with code snippets showing both the light and dark mode versions at the same time in older versions of safari. The docusaurus project is in the /site directory in the example link.
See issue https://github.com/excaliburjs/Excalibur/issues/2981

With our investigation this seems to be because of the way that the :has pseudo class is used to show/hide the snippet. One possible fix is a simpler selector using backwards compatible css in docusaurus?

Reproducible demo
https://github.com/excaliburjs/Excalibur/tree/main/site
Steps to reproduce

Run the docusaurus site locally https://github.com/excaliburjs/Excalibur/tree/main/site with npm run start
On safari 12, 13, or 14 navigate to /docs/installation
Observe the duplicated snippets for both light and dark mode

Expected behavior
Either the light mode code snippet or the dark mode snippet should appear on older versions of safari (12, 13, 14)
Actual behavior
Both versions of the code snippet appear
Your environment

Public source code: https://github.com/excaliburjs/Excalibur/tree/main/site wit
Public site URL: https://excaliburjs.com
Docusaurus version used: 3.0.0
Environment name and version (e.g. Chrome 89, Node.js 16.4): Safari 12, 13, 14 & Node v20.12.1
Operating system and version (e.g. Ubuntu 20.04.2 LTS): Windows 11

Self-service

[ ] I'd be willing to fix this bug myself.

Probably fixed in v3.1.1+ already, with https://github.com/facebook/docusaurus/pull/9530
Please upgrade your site before reporting bugs, or provide a runnable repro using the latest version. Otherwise, it's hard to inspect and we might spend time uselessly trying to fix a problem that is already fixed.

I'll retest in that version to be sure


Custom connection.socketURL

Hi everyone please help me how to have a own local connection.socketURL? Please help me badly need it.

Check this out : https://github.com/muaz-khan/RTCMultiConnection/issues/629#issuecomment-421111152
Probably the one you might be looking for.
Cheers.


code for ensemble distillation

Nice work! I was wondering if you are planning to release code for ensemble distillation?

You can simply sum up all L_{crd}(T_i; S), with i indicating the i-th teacher.


btest: Add version 7.15.1

Add btest, the bandwidth test software for Windows developed by Mikrotik.

[x] I have read the Contributing Guide.

Closes #13452

/verify


Will the code be published?

Was just wondering if this code will be published? The project page linked here for the code, but the repository is empty.

Yes, the testing code is now released, and we will release our training code as well in the future. Thank you!

How great your work, is there any plan to still release the training code?

Sorry about the plan change, but we cancelled the plan for training code release due to licensing issues with the training dataset.


Make usePrometheus available from @graphql-mesh/plugin-prometheus

Description
usePrometheus is not available to import from @graphql-hive/gateway:
import {
  useJWT,
  useOpenTelemetry,
  // usePrometheus // Not available!
} from '@graphql-hive/gateway';
import usePrometheus from '@graphql-mesh/plugin-prometheus';

With this PR, it will be possible to:
import {
  useJWT,
  useOpenTelemetry,
  usePrometheus,
} from '@graphql-hive/gateway';

Type of change

[x] New feature (non-breaking change which adds functionality)

Covered here;
https://github.com/graphql-hive/gateway/pull/87


support for the Sass indented syntax

The SCSS syntax of Sass can be used quite easily, but not the indented syntax. In node-sass there is an option that can be set: indentedSyntax: true, but I couldn't figure out how to make that work with my roots project (might be a user error).

  There are workarounds to that issue, but it's not ideal. I have something mocked up here if someone wants to play with it. This was my first crack at a roots project, it includes some things like contentful integration and yaml front matter too. https://github.com/javaporter/roots

Yeah definitely, I'll look into enabling the sass syntax as well. This
particular compiler is unique in the fact that you use the same compiler
for two different languages with different extensions, so I'll have to
tinker around a bit to make sure it's smooth.
On Sat, Mar 7, 2015 at 9:46 PM Perry Kibler<EMAIL_ADDRESS>wrote:

Hmm, kind of works.  There are workarounds
to that issue, but it's not ideal. I have something mocked up here if
someone wants to play with it. This was my first crack at a roots project,
it includes some things like contentful integration and yaml front matter
too. https://github.com/javaporter/roots
—
Reply to this email directly or view it on GitHub
https://github.com/jenius/roots/issues/621#issuecomment-77729438.


🐛 FIX: INCORRECT NAVIGATIONAL LINK ON README [DOCS]

Description:

Before this PR the navigational link of public/Buttons is pointing towards https://github.com/Spyware007/Animating-Buttons/blob/main/src/components/Main/Main.jsx
Now in this PR i resolved the issue and modified the link to https://github.com/Spyware007/Animating-Buttons/tree/main/public/Buttons

This issue is assigned me under GSSoC'23, the Issue: #696
Reviewer:

@Spyware007

Hi, the link leads to your repository

Hello @PoorviShetty, Thanks for pointing that out!
Now i resolved that issue, can this PR can be merged now ?


Normalize markdown formatting across usage.md files

This preps the usage.md files for syntax highlighting in the CF docs by:

Removing indents from HTML that is intended to render
Correcting the places where four back ticks were used instead of 3
Fixing a few small markdown errors or weird copy and paste formatting (extra quotation marks, single quotes used instead of inline backticks)

Review
@Scotchester or @contolini

@cfpb/front-end-team-admin For folks working on components' usage.md files: this affects you.

Other than that one small note, :+1:

@Scotchester I just pushed that update to the CHANGELOG

Thanks!


editMessageReplyMarkup and editMessageText methods are not working

Hi,
I'm calling editMessageReplyMarkup method with these parameters:
var reply_markup = {"inline_keyboard":[[{"callback_data":"579a27fc082b43da63de74af,1","text":"1"},{"callback_data":"579a27fc082b43da63de74af,2","text":"2"}]]}
var message_id = 1789 //this is message_id property of a message which is sent by myBot with an inline reply markup

myBot.editMessageReplyMarkup(reply_markup, message_id)

And here is what i get as an error:
Unhandled rejection Error: 400 {"ok":false,"error_code":400,"description":"Bad Request: message identifier is not specified"}
    at /home/masoud/node.js/TCHBot/node_modules/node-telegram-bot-api/lib/telegram.js:186:17
    at tryCatcher (/home/masoud/node.js/TCHBot/node_modules/request-promise/node_modules/bluebird/js/main/util.js:26:23)
    at Promise._settlePromiseFromHandler (/home/masoud/node.js/TCHBot/node_modules/request-promise/node_modules/bluebird/js/main/promise.js:507:31)
    at Promise._settlePromiseAt (/home/masoud/node.js/TCHBot/node_modules/request-promise/node_modules/bluebird/js/main/promise.js:581:18)
    at Promise._settlePromises (/home/masoud/node.js/TCHBot/node_modules/request-promise/node_modules/bluebird/js/main/promise.js:697:14)
    at Async._drainQueue (/home/masoud/node.js/TCHBot/node_modules/request-promise/node_modules/bluebird/js/main/async.js:123:16)
    at Async._drainQueues (/home/masoud/node.js/TCHBot/node_modules/request-promise/node_modules/bluebird/js/main/async.js:133:10)
    at Async.drainQueues (/home/masoud/node.js/TCHBot/node_modules/request-promise/node_modules/bluebird/js/main/async.js:15:14)
    at process._tickCallback (node.js:419:13)

I have also tried replacing message_id with id property of the object that is presented by callback_query event and chat.id property of a message which is sent by myBot with an inline reply markup, and I got exactly the same error message.
I Also have the same problem with editMessageText method. Here is how I call it:
var message_id = 1789 //this is message_id property of a message which is sent by myBot with an inline reply markup
tchBot.editMessageText("edited! :)", message_id)

And here is what i get as an error:
Unhandled rejection Error: 400 {"ok":false,"error_code":400,"description":"Bad Request: Message text is empty"}
    at /home/masoud/node.js/TCHBot/node_modules/node-telegram-bot-api/lib/telegram.js:186:17
    at tryCatcher (/home/masoud/node.js/TCHBot/node_modules/request-promise/node_modules/bluebird/js/main/util.js:26:23)
    at Promise._settlePromiseFromHandler (/home/masoud/node.js/TCHBot/node_modules/request-promise/node_modules/bluebird/js/main/promise.js:507:31)
    at Promise._settlePromiseAt (/home/masoud/node.js/TCHBot/node_modules/request-promise/node_modules/bluebird/js/main/promise.js:581:18)
    at Promise._settlePromises (/home/masoud/node.js/TCHBot/node_modules/request-promise/node_modules/bluebird/js/main/promise.js:697:14)
    at Async._drainQueue (/home/masoud/node.js/TCHBot/node_modules/request-promise/node_modules/bluebird/js/main/async.js:123:16)
    at Async._drainQueues (/home/masoud/node.js/TCHBot/node_modules/request-promise/node_modules/bluebird/js/main/async.js:133:10)
    at Async.drainQueues (/home/masoud/node.js/TCHBot/node_modules/request-promise/node_modules/bluebird/js/main/async.js:15:14)
    at process._tickCallback (node.js:419:13)

In this case as well, I have tried to replace message_id with id property of the object that is presented by callback_query event and chat.id property of a message which is sent by myBot with an inline reply markup, and Again I got exactly the same error message.
I haven't tried editMessageCaption method yet.
So, can anyone tell me that is it my fault or something is wrong with the module?
Thank you very much for reading all this! :)
P.S: I have double checked all the parameters that i have passed to both methods.

Hey everyone,
Sorry for opening this issue! I was blind but now I see!
I have to pass message id as an object not as an integer or string!
Here is the true way to call editMessageTest for example:
var messahe_id = 1789
myBot.editMessageText("edited! :)", {message_id: messsge_id})


Problem with url in wordpress

Hi,
Im trying to use animsition in wordpress, but I think it not work when I give the full url..
What should I do?

Hi,
What you mean by not working with full URL ?
Have a look into this wordpress plugin - https://wordpress.org/support/plugin/page-transition to get idea if you want.
It is using same jquery plugin but not using latest version of this plugin yet. We will update later this month.
Thanks.

Hi,
I dont know if the full URL is the problem.. im getting "undefined" in the end of URL when I click in the link.. the page transition works ok..
What should I do?
Thanks

Im not using the plugin, couse I need the Overlay option

Would you please show an example somewhere if possible ?
Thanks.

I believe the problem has to do with pretty permalinks.
ex:  http://example.com/2012/post-name/
this will result in:
http://example.com/2012/post-name/undefined
Let me know if you find the solution to this

Hi there, possibly you using the li class for the linkElement rather than the a (easy mistake to make)!

That plugin is using the outdated 3.2.X version of animsition, I'd suggest or using cdnjs, adding to your theme or manually updating the plugin.
Let me poke @gauravpadia so he can know that the plugin is outdated. Also, it blocks using some new animations provided by the plugin, not to mention it just hides some features and options, like denying the use of an image and always writting a "Loading" on loading div.

Hi @Shirkit, due to some circumstances I was not able to update the plugin for last few months, but I am on to it and will update the wordpress plugin as soon as possible. Thanks. :)


FIX #6193 : bind enter and tab key on color picker input to close color board

Description

Binding Enter and Tab Key on color picker and close color popup.

Fixes #6193
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Tested locally.

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:red_circle: Total coverage has decreased
// Code coverage diff between base branch:release and head branch: FEATURE/6193-binding-key-events-color-picker

Status
File
% Stmts
% Branch
% Funcs
% Lines

:red_circle:
total
51.53 (-0.01)
33.55 (-0.01)
29.38 (0)
52.16 (-0.01)

:red_circle:
app/client/src/components/ads/ColorPickerComponent.tsx
32.56 (-5.28)
53.85 (-9.79)
0 (0)
35.9 (-4.1)

@ramsaptami can you test this deploy preview and give us a signoff

Tested and ok on deploy preview

/ok-to-test sha=bcd1188

/ok-to-test sha=bcd1188

@vibhandikyash please merge latest release here, Failing test fix is available here.
Also do request for a review as /ok-to-test is not fully functional right now

There is an issue in the DP

Focus on the colour picker field
type a colour ex: red
Press tab

It is observed that the colour picker of the previous input/field opens, and then if we press shift + tab another colour picker opens.
https://user-images.githubusercontent.com/11089579/128358463-5437baf8-4b0d-4363-b030-351363f4634e.mov

@SatishGandham As discussed yesterday on the call. Can you pl. update here?

@SatishGandham what is pending here? I have missed the trail

@vibhandikyash Tested the DP and working fine


Email notification changes

I have made the following changes:

Email notifications are sent to only creator and assigned person, not to everyone in the project.
Overdue email notification is sent to only the assigned persons.
A daily reminder email of tasks that are due this week.
Email template enhancements.

Would you like me to merge these changes to your codebase?

:+1:

You can always send a pull-request.

By the way, I'm currently refactoring the notification code to be more flexible...

The notifications are more flexible now in the master branch... Don't hesitate to send a pull-request for any possible improvements.

Hi, sorry if this is not the right place to ask this, but I wanted to know how to activate these email reminders in my kanboard install. I'm using 1.0.28
Thanks


We emit a signal so that it can be displayed as popup instead of a console.error

I was tempted to remove line #91 so that the metrics stays as last "polled" but I guess if it was done like that there was a reason?

Thanks @ROunofF!
I made some additions to your PR:

Upgrade angular-toastr to 1.6.0
Use a generic error message when the backend API returns an error
Set the toastr option preventOpenDuplicates to true
Only initialize the $rootScope.metrics object if undefined so it stays to the last polled value


HttpContext.SignInAsync sets null to Identity claims data

Describe the bug
In my project, I want to make use of the cookie based authentication without using the identity infrastructure. I have my own user,role and claim tables. So I was following this.
Our project is in a tfs repo. When any member in the team tries to run the app, we always get a exception caused from HttpContext.User.Identity is filled up with all null values. But when debug it or run it for several times, mostly values somehow filled up, or not. Details below;
To Reproduce
Steps to reproduce the behavior:

Using this version of ASP.NET Core 2.1
I have a middleware where I get a header value and and depend on that value get user data, then set these user data to Identity object.

UserSessionMiddleware
public async Task Invoke(HttpContext context, IUserService userService, ISessionHelper sessionHelper,
                                    ILogger<UserSessionMiddleware> logger)
{
    if (sessionHelper.IsAppSessionSet)
        await _next.Invoke(context);
    else
    {
        var employeeId = context.Request.Headers["headerVal"].ToString();
        UserEntity user = userService.GetUserData(employeeId);
        List<ClaimEntity> roleClaims = userService.GetUserRoleClaims(user.UserRoles.Select(ur => ur.RoleId).ToArray()).ToList();

        if (user == null)
        {
            await context.Response.WriteAsync($"User with Id: {employeeId} not found in database!");
            return;
        }
        #region Create user claims

        ClaimsIdentity claimsIdentity = new ClaimsIdentity(CreateUserClaims(user, roleClaims),
            CookieAuthenticationDefaults.AuthenticationScheme);

        ClaimsPrincipal claimsPrincipal = new ClaimsPrincipal(claimsIdentity);

        await context.SignInAsync(CookieAuthenticationDefaults.AuthenticationScheme, claimsPrincipal,
            new AuthenticationProperties { IsPersistent = true, ExpiresUtc = DateTime.UtcNow.AddDays(7) });

        #endregion
        sessionHelper.User = user;
        sessionHelper.IsAppSessionSet = true;

        var sessionId = context.Session.Id;
        logger.LogInformation($"SessionId: {sessionId}, {user.EmployeeId}/{user.FullName} has started session");
        await _next.Invoke(context);
    }
}

As the request reaches to the view component where to get user data and bind to the view elements, it throws null object exception. In fact when I inspect the context's User.Identity object after context.SignInAsync, I can see that a new object sits there however all values are null, as seen in the ss.

public class ProfileComponent : ViewComponent
{
    private ISessionHelper _sessionHelper;
    private IHttpContextAccessor _contextAccessor;

    public ProfileComponent(ISessionHelper sessionHelper, IHttpContextAccessor contextAccessor)
    {
        _sessionHelper = sessionHelper;
        _contextAccessor = contextAccessor;
    }
    public IViewComponentResult Invoke()
    {
        ClaimsIdentity identity = (ClaimsIdentity)_contextAccessor.HttpContext.User.Identity;
        dynamic user = new ExpandoObject();

        //this line throws exception
        user.EmployeeId = identity.FindFirst(ClaimTypes.NameIdentifier).Value;
        user.Name = identity.Name;
        user.Email = identity.FindFirst(ClaimTypes.Email).Value;
        user.Roles = identity.FindAll(ClaimTypes.Role).Select(i => i.Value).ToList();
        ViewBag.User = user;
        return View();
    }
}

Expected behavior
Expected behavior is to get claims data in view component properly and constantly. For now this code is working on my computer, and same not working on my firend's.

Is this question related to AspNetCore.Security?


Discussion: should download screen be mandatory, or the interview order block?

I think the concept of having an optional interview order block was to facilitate combining multiple interviews into one. But it doesn't get all of the way there, because you still need to remove the mandatory download screen.
I forgot that we had this slightly unusual structure. I think it could throw people off when they see other docassemble code.
The use of a named block is kind of handy to automate, because we show in the wizard-generated code how to do it. That could be a tricky concept otherwise.
An alternative approach would be to have two interview order blocks: one mandatory one that referenced the optional one. The mandatory block could be removed if the file is included in another file.
Even fancier: we could generate two YAML files: one with the mandatory block, and one that just has the includable stuff. But that feels over-optimized for a rare use case. Only a few files get included inside other files.

Currently wizard generates code like this:
id: interview order
code: |
  ...
  this_file_block = True
---
id: download
mandatory: True
need: this_file_block
question: |
  Your file is ready to download

Currently wizard generates code like this:
id: interview order
code: |
  ...
  this_file_block = True
---
id: download
mandatory: True
need: this_file_block
question: |
  Your file is ready to download

The last option was also the one I was on the fence about. It adds an extra bit of complexity. On the other hand, it seems an easier layer of complexity to overcome when compared with trying to combine interview packages that have mandatory code in them.
As I said, though, I don't have legal experience, so I'll defer to folks who do.
As far as order of code, I have found that model a bit confusing to think about, but if it's the only way to combine files then I dunno. How does it work when you want to exclude an attachment block, btw?

The last option was also the one I was on the fence about. It adds an extra bit of complexity. On the other hand, it seems an easier layer of complexity to overcome when compared with trying to combine interview packages that have mandatory code in them.
As I said, though, I don't have legal experience, so I'll defer to folks who do.
As far as order of code, I have found that model a bit confusing to think about, but if it's the only way to combine files then I dunno. How does it work when you want to exclude an attachment block, btw?

As far as order of code, I have found that model a bit confusing to think about

What does this mean?

As far as order of code, I have found that model a bit confusing to think about

What does this mean?

I'm leaning towards having two interview order blocks: one mandatory, and one named block, so someone can move just the mandatory block if they decide to make this a file that gets included in other interviews. Not too much overhead in the file, fits better docassemble style, and doesn't clutter the file with something people don't use very often while making that rare use case easier to handle.

I'm leaning towards having two interview order blocks: one mandatory, and one named block, so someone can move just the mandatory block if they decide to make this a file that gets included in other interviews. Not too much overhead in the file, fits better docassemble style, and doesn't clutter the file with something people don't use very often while making that rare use case easier to handle.

As far as order of code, I have found that model a bit confusing to think about

What does this mean?

This means that the document being required and the interview order code not being required is something I haven't found intuitive to think about. It's not a huge burden though, and might be covered by a bit of documentation.

Our new mandatory block does two things:

Trigger the variable order block
Trigger intro screen
Trigger the signature/preview blocks
Trigger the download

We need an ID/name for the new block.
Let's keep "interview order" block for the one, mandatory block. Let's rename the
internal / named block the "variable order" block.
Another idea: the umbrella order block for the mandatory one.
Standalone order block

store_variables_snapshot:

It does seem like it would belong in the interview order, but I'd think data would be gathered in an umbrella interview if that were happening.
Is there a piece of information that's more reliably required than address? It doesn't seem like it to me, but I know that the form I'm working wants to avoid collecting an address. Perhaps the referrer? This may be in its own issue eventually.

I think we need to discuss a bit more on an admin day.
For now, I agree it could go in the umbrella file block by default.


Fix for #2180

Fix for #2180

Tests should pass after this

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
0

Relevant Lines:
1367

💛  - Coveralls


Building and running examples (for Java) is not clear

I'm not a Java programmer, so I'm sure that's part of it, but I feel that the docs are lacking.  At least in regards to Java.
I'm trying to follow the docs here for building and running the examples, but after a few hours, I'm still not there.
I feel like a concrete example of how to use this would be helpful.  Here's what I've come up with so far:
Don't use the JDK that's already installed.  Instead:
wget --no-check-certificate --no-cookies - --header "Cookie: oraclelicense=accept-securebackup-cookie" "http://download.oracle.com/otn-pub/java/jdk/8u111-b14/jdk-8u111-linux-i586.tar.gz"
mkdir -p /usr/local
cd /usr/local
tar xzf ~/jdk-8u111-linux-i586.tar.gz
export JAVA_HOME=/usr/local/jdk1.8.0_111
mkdir -p /usr/local/bin
cd /usr/local/bin
ln -s $JAVA_HOME/bin/java

Then to get, say, the APA102 example to (almost?) run:
git clone repo https://github.com/intel-iot-devkit/upm.git
cd upm
mkdir build
cd build
cmake .. -DBUILDSWIGJAVA=ON -DBUILDEXAMPLES=ON
cd src/apa102
make
make install
cd ../../../examples/java
javac -cp /usr/local/lib/java/* APA102Sample.java

However, even at this point, it's not quite working:
root@edison:~/workspace/upm/examples/java# java APA102Sample
Exception in thread "main" java.lang.NoClassDefFoundError: upm_apa102/APA102
	at APA102Sample.main(APA102Sample.java:30)
Caused by: java.lang.ClassNotFoundException: upm_apa102.APA102
	at java.net.URLClassLoader.findClass(URLClassLoader.java:381)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
	at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:331)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
	... 1 more

Again, I'm sure if I were a Java programmer, I would (probably?) have not had a problem with this, but I feel like it wouldn't hurt to help with a concrete example about how to actually run a Java example from this repo.  I've spent hours and gone through countless tutorials/articles, but still haven't gotten it to work.  Even this seemingly informative article in the Java section says that it's not working.

Hi @solidcell, sorry to see you are having issues running the Java sensor examples, you're actually very close.
CLASSPATH is likely not set and hence why Java fails to load the library. You did figure out how to resolve this when compiling javac -cp /usr/local/lib/java/* APA102Sample.java, simply use the -cp /usr/local/lib/java/* option when running with java or set the environment variable beforehand. Keep in mind that the jars alone are not enough and LD_LIBRARY_PATH needs to be set to the location of libjavaupm_apa102.so (the one you compiled and installed).
Please note that Shawn's article is pretty old and our Java support has improved considerably since, especially when you use an IDE and a supported image on your device. For instance, with Intel System Studio you can create Java projects step by step, and we have several helpful samples in this repo. If you're using the IoT DevKit image consider updating the MRAA & UPM libraries with the built-in package manager, opkg, as described here.

Oh, when running you'll also want the current directory added to the path since it has your APA102Sample class, thus something like:
java -cp /usr/local/lib/java/upm_apa102.jar:. APA102Sample

Thanks @Propanu, that worked perfectly :)
I don't have anything at LD_LIBRARY_PATH, but it seems to work fine with just doing java -cp /usr/local/lib/java/upm_apa102.jar:. APA102Sample.
As for the IDE, I like not relying on a specific IDE or too much magic.


pip installation failed - Unable to find vcarsall.bat

I have tried to install pyproj with pip but keep getting errors when i try and install it. i have even downloaded the source code. I have checked and do have the necessary software to compile the module but it continues to fail.

@maplelm, you most likely need the Visual C++ Build Tools 2015 for Python 3.5 to get this to compile.  This used to be a lot harder to track down but Microsoft has put some effort into getting this visible for the Windows Python users in the world.
Blog Reference: Unable to find VCVARSALL.BAT

Refer to #216


Support nullary products

Needed this alongside https://github.com/haskell-beam/beam/pull/563

This probably needs to be reworked given upstream concerns

psql (9.6.15)
Type "help" for help.

postgres=# CREATE TABLE zero ();
CREATE TABLE
postgres=# \d zero
    Table "public.zero"
 Column | Type | Modifiers
--------+------+-----------

postgres=# insert into zero values ();
ERROR:  syntax error at or near ")"
LINE 1: insert into zero values ();
                                 ^
postgres=# insert into zero default values;
INSERT 0 1
postgres=# SELECT * FROM zero;
--
(1 row)

postgres=# insert into zero default values;
INSERT 0 1
postgres=# SELECT * FROM zero;
--
(2 rows)

I think this may require a backend-specific check to see if the resulting tables are sensible.


catch the exception when trying to getopts for a non-exist socket

Proposed Changes
Trying to fix the issue. It will catch the exception when trying to getopts for a non-exist/closed socket.
Types of Changes
What types of changes does your code introduce to this project?
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes issue #XXXX)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (correction or otherwise)
[ ] Cosmetics (whitespace, styles...)
[ ] DevOps (Build scripts, pipelines...)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating
the PR. If you're unsure about any of them, don't hesitate to ask on the
mailing list. We're here to help! This is simply a reminder of what we are
going to look for before merging your code.

[x] I have read the CODE_OF_CONDUCT.md document
[x] All tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if needed)
[x] Any dependent changes have been merged and published in related repositories
[ ] I have updated the changelog (At the bottom of the release version)
[x] I have squashed all my commits into one before merging

Further Comments
Based on our production experience, this exception will be thrown when there is an accept queue overflow.

@xiaolingtc Thanks so much! :)
can you please run rebar3 fmt and push that commit? (so that we pass the codestyle check in CI)


HBASE-23136 PartionedMobFileCompactor bulkloaded files shouldn't get …

…replicated (addressing buklload replication related issue raised in HBASE-22380)

Can we use a shorter commit message? :)

Should this new option be exposed on command-line?

Definitely! Will address that on a separate issue, since on this one I'm most concerned with avoiding getting compacted mob files replicated.

Can we use a shorter commit message? :)

We sure can, I will edit the message prior to merging it.


data object for combined metadata

Call it data/subjectdata.rda (save with high compression), also need man/subjectdata.Rd file. Needed for issues #10 and #30.
Make a note of this data.frame in the vignette.

Would this perhaps be a good place to use an internal sysdata.rda file (http://r-pkgs.had.co.nz/data.html#data-sysdata)?

close by 0a583f4

0a583f4 titled "changes to template.csv to make some variables numeric" seems to be unrelated to this issue?
Aha, I think it's c506f74. But two questions:

How does a user load this object? If it is exported and documented (and usually in data/, not R/), they can do data(combined_metadata)
Should there be a column saying which study the row is from?

Ah, I just saw that you had proposed this as an internal, hidden data file. What's the justification? I would think the participant metadata would be of potential use to the user, but you might be able to convince me otherwise with good reason...

Oops, it is c506f74 and there should be a row with the study name. I will be very difficult to find the data otherwise.
As for why we might prefer an internal data file, I see the curatedMetagenomicData function as the main interface for the package and not the data.frame itself. If we use the function to download a dataset we can then pass that on programmatically to something else. In essence, every user will want to download some data from the package and by using an internal file with curatedMetagenomicData function the process will be consistent. Also, the function should help us as we develop the Shiny app.
I can document and export the file if you think it is better that way but it just seems the cMD function and your CLI interface are a better way to interact with the data.

Users certainly should be required to interact with the metadata table, but here are a couple use cases for it I can think of:

user does an analysis spanning thousands of samples, but wants to make a descriptives table in a script without having to load all those datasets

user would like to count the number of certain types of samples available to see if analysis is worthwhile, before doing download

user would just like to open up the metadata in a spreadsheet or View() just to "see" what's  available. I've felt this way sometimes about TCGA metadata.

advanced user would like to do some sample selection methodology not supported by the curatedMetagenomicData() function.

And I don't see much downside, do you?

No downside to it. I will export and document the file and then it can be used both internally and externally.

The combined metadata is not exported yet but it is corrected to include the dataset_name. Will have it exported shortly but just wanted to mention that it's available now : )

After a bit of a struggle with automated documentation, this issue is closed by a664c26.


standard_init_linux.go:185: exec user process caused "no such file or directory" on WIndows 10

On Windows 10
docker version
Client:
 Version:      17.09.0-ce
 API version:  1.32
 Go version:   go1.8.3
 Git commit:   afdb6d4
 Built:        Tue Sep 26 22:40:09 2017
 OS/Arch:      windows/amd64

Server:
 Version:      17.09.0-ce
 API version:  1.32 (minimum version 1.12)
 Go version:   go1.8.3
 Git commit:   afdb6d4
 Built:        Tue Sep 26 22:45:38 2017
 OS/Arch:      linux/amd64
 Experimental: true

docker-compose version
docker-compose version 1.16.1, build 6d1ac219
docker-py version: 2.5.1
CPython version: 2.7.13
OpenSSL version: OpenSSL 1.0.2j  26 Sep 2016

After running the following:
step 1: git config --global core.autocrlf false
step 2: delete code and git clone again
I get an error running:
docker-compose -f odm-standalone.yml up

Starting odmondocker_odm-standalone_1 ...
Starting odmondocker_odm-standalone_1 ... done
Attaching to odmondocker_odm-standalone_1
odm-standalone_1  | standard_init_linux.go:185: exec user process caused "no such file or directory"
odmondocker_odm-standalone_1 exited with code 1

I have checked and rechecked that all the files now show with LF only and not CRLF as when I started. But I still get the issue.

verbose.txt
verbose-err.txt
The files attached provide the verbose output of the command (verbose.txt is the regular output, verbose-err.txt is the err output which seems to be used for the verbose logging).

Regards,
Eric (E.) Charpentier
P.Eng. MBA PMP
IBM ODM L2 Support Engineer
IBM Hybrid Cloud
E-mail<EMAIL_ADDRESS>                              8200 Warden Ave
Find me on:                                                    Markham, ON L6G 1C7
Canada
From:	Laurent GRATEAU<EMAIL_ADDRESS>To:	ODMDev/odm-ondocker<EMAIL_ADDRESS>Cc:	echarpibm<EMAIL_ADDRESS>Author
<EMAIL_ADDRESS>Date:	2017-10-19 14:58
Subject:	Re: [ODMDev/odm-ondocker] standard_init_linux.go:185: exec user
process caused "no such file or directory" on WIndows 10 (#78)
Hi,
Could you verify the layout ?
Should be something like that :
.dockerignore
artifacts.xml
buildcommand
configuration
doc
eclipse
epl-v10.html
executionserver
gettingstarted
images
jdk
license
notice.html
odm-ondocker
p2
plugins
profileTemplates
readme
shared
shortcuts
studio
teamserver
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Well after troubleshooting the issue for a while, here is what I have had to do to resolve.

In the docker application (in the system tray), enable containers (which were not enabled) (required a reboot)
In the .yml file I updated the version to 2.1 (version: '2.1')
There were other things that happened in between but essentially as far as I can tell, that was it.


Assigning input.value results in incorrect unmaskedvalue() result

Describe the bug
Assigning an unmasked string to the input.value results in an incorrect unmaskedvalue() result which contains the mask

Add a link to a codepen, jsfiddle or other example page which shows the problem
https://jsfiddle.net/djfb72q6/2/

As far as I could trace, generatedInput === undefined for mask characters which triggers the following condition
https://github.com/RobinHerbots/Inputmask/blob/69cf0a550cb3322f757e980b864a51172481995c/lib/maskScope.js#L1928

OS:
Windows 10

Browser
Chrome

Inputmask version
5.0.3

I have the same problem. Same as unmasked value and masked value in 5.0.3 version.
But there is no problem in 5.0.1 version.
<input type="text" id="kkno-mask" maxlength="19" autocomplete="new-password" data-parsley-no-focus data-parsley-group="kkdogrula" data-parsley-trigger-after-failure="null" data-parsley-kkartno required="" />
element.inputmask({
    "mask": ["4999 9999 9999 9999", "5999 9999 9999 9999", "6999 9999 9999 9999", "3999<PHONE_NUMBER>9"],
    "keepStatic": false,
    "greedy": false,
    "placeholder": "_",
    "clearIncomplete": false,
    "autoUnmask": true,
    onUnMask: function(maskedValue, unmaskedValue, opts){
        return maskedValue.substr(0, 1)+""+unmaskedValue;
    }
    .....other lines......
});

Version 5.0.3 onUnMask results:
// maskedValue: 4999 9999 9999 9999
// unmaskedValue: 4999 9999 9999 9999

Version 5.0.1 onUnMask results:
// maskedValue: 4999 9999 9999 9999
// unmaskedValue:<PHONE_NUMBER>999999

Looks like this has been fixed since 5.0.5

@MaximBalaganskiy i will test it. thank you.

Not reproducable in 5.0.5


Multiple Refactorings

This is a draft PR that contains multiple assorted tweaks and refactorings that I have implemented for a SEM.jl package over a last few weeks.
The goal of these changes was to

improve the performance and memory footprint for my use cases (few thousands of observed vars)
cleanup the public API (also to simplify building SEM models for my usescases)
cleanup the internal API to simplify maintenance

I realize that there's a lot of changes, so it is hard to review all that is going on in this PR, but I thought it still would be nice to submit it as a preview of all the potential changes that I have implemented.
In particular, these are the bigger pieces implemented:

introduce ParamArray type that incapsulates the logic of substituting parameter values into RAM matrices (what is currently done by A_inds, S_inds etc).
It simplifies the code, reduces code duplication and allows easy switching between dense/sparse matrices of substituted model parameters.
unify objective/gradient/hessian calculation of loss functions and implied objects into a single evaluate!() call (instead of upto 7 methods in the current version).
That greatly improves the maintenance and tweaking this code, and also maps better onto optimization packages (Optim.jl).
predict_latent_vars() for predicting factor scores per observation. At the moment regression and Bartlett methods are implemented.
tweaks to the external API: identifier() is renamed into param() for clarity, param_type into relation, consistently added support for vars, observed_vars, latent_vars methods to ParameterTable, RAMMatrices and other types
converting SEM.jl models into lavaan models; extracting parameters from lavaan models (helper function for the unit tests was moved into the main package code)
missing data handling: introduce SemObsMissingPattern object to simplify missing data-related code (including FIML)
multiple computation optimizations across the package (EM, ML, FIML, WLS, generic and symbolic implied objects): mostly to linear algebra, e.g. using 5-arg mul!() to avoid unnecessary allocations
some cleanups to the unit tests to improve reporting of the failed tests
use Julia 1.9 package extensions for supporting different optimizations backend (NLOpt.jl, ProximalOperators.jl, BlackBoxOptim.jl). This is both to reduce the number of the package dependencies and to streamline switching between the optimizers for the users.

Let me know what you think about this PR and how you would like these changes to be incorporated into the main tree (of course, if you are interested in that :) )
I have tried to make commits as granular as possible to allow selecting subset of commits around a particular feature into a smaller PRs.

I have benchmarked one of my models (~120 observed variables, generic imply, ML) 2 weeks ago.
With the main branch:
47.710711 seconds (1.55 M allocations: 35.937 GiB, 11.03% gc time)
47.635976 seconds (1.55 M allocations: 35.937 GiB, 10.97% gc time)

 * Candidate solution
    Final objective value:     2.002460e+02

 * Found with
    Algorithm:     L-BFGS

 * Convergence measures
    |x - x'|               = 8.45e-07 ≰ 0.0e+00
    |x - x'|/|x'|          = 6.51e-08 ≰ 0.0e+00
    |f(x) - f(x')|         = 0.00e+00 ≤ 0.0e+00
    |f(x) - f(x')|/|f(x')| = 0.00e+00 ≤ 0.0e+00
    |g(x)|                 = 2.88e-07 ≰ 1.0e-08

 * Work counters
    Seconds run:   48  (vs limit Inf)
    Iterations:    4279
    f(x) calls:    12529
    ∇f(x) calls:   12529

With this PR (earlier version):
20.348503 seconds (665.00 k allocations: 12.666 GiB, 9.55% gc time)
20.356397 seconds (665.00 k allocations: 12.666 GiB, 9.42% gc time)

Candidate solution
    Final objective value:     1.999197e+02

 * Found with
    Algorithm:     L-BFGS

 * Convergence measures
    |x - x'|               = 3.89e-07 ≰ 0.0e+00
    |x - x'|/|x'|          = 2.99e-08 ≰ 0.0e+00
    |f(x) - f(x')|         = 0.00e+00 ≤ 0.0e+00
    |f(x) - f(x')|/|f(x')| = 0.00e+00 ≤ 0.0e+00
    |g(x)|                 = 3.68e-08 ≰ 1.0e-08

 * Work counters
    Seconds run:   20  (vs limit Inf)
    Iterations:    2774
    f(x) calls:    8085
    ∇f(x) calls:   8085

The number of iterations is different, because the input data is slightly different due to imputation.
But for the main branch it's 89.7 iterations/sec, 8.39 Mb/iteration
For the PR: 136.3 iterations/sec, 4.54 Mb/iteration

These changes look amazing, thanks a lot for feeding them back. I will take a look at them and try to figure out what the smartest way of merging is.

I tried wrapping my head around how to format first and then merge, but everything I tried lead to quite large merge conflicts... So I guess it's easier to merge this first and format after...

I tried to wrap my head around how to format first and then merge, but everything I tried lead to quite large merge conflicts...

This would require a bit of scripting, but I think it's doable.
The idea is to create a parallel branch, which would recreate the history of this branch, but with the formatting applied.
Then at each iteration of the script, it would check out the specific commit from the unformatted branch (this branch), format the whole codebase and commit the resulting changes to the new branch.
I can do that, it would be a fun exercise :) We just have to agree on the formatting style, apply it to the master branch, and then I can reformat the commits in this PR.

Cool, if you would like to do that of course we can^^ I thought about using the default formatter style with some minor tweaks: https://github.com/StructuralEquationModels/StructuralEquationModels.jl/blob/feature/formatter/.JuliaFormatter.toml
Would you be happy with that?

Would you be happy with that?

I like always_for_in. :)
In principle, I would be happy with anything that you, as the author&maintainer, would decide, but if you can create a draft PR/branch that would show the reformatted code, I can give a specific feedback.

This branch is formatted accordingly: https://github.com/StructuralEquationModels/StructuralEquationModels.jl/tree/feature/formatter
Yes, always_for_in made me realize I used =, in and ∈, haha. The other options are just some minor changes that let you have multiple assignments / matrix definitions / ... aligned.
One minor thing is that formatter changes spaces between - and *, and sometimes in a weird way... But I guess that's okay.

This branch is formatted accordingly

Thank you! I like this style, even the condensing of formulas within the indices. :)
I personally also like to remove superfluous newlines, so that more code can fit on one screen (which remove_extra_newlines = true should implement), but I don't know if that would be too aggressive and remove any reasonable separation of the code blocks.

I added remove_extra_newlines = true, and also added whitespace_typedefs = true which changes for example struct SemEnsemble{N,T<:Tuple,V<:AbstractVector,D,I} <: AbstractSemCollection to struct SemEnsemble{N, T <: Tuple, V <: AbstractVector, D, I} <: AbstractSemCollection. If you want you can check again on the feature/formatter branch; and if you're okay with that I would go ahead and update the main branch accordingly.

I added remove_extra_newlines = true (I also like it better), and also added whitespace_typedefs = true.

Thank you, I like these updates. It's go to go from my side.
One thing I noticed -- the formatter adds explicit returns (which is a good thing), which exposes a few cases like return print(...), where returning the result of print was not intended.
This has to be replaced with
    print(...)
    return nothing

but it's not critical to do that immediately, it could be gradually fixed over time.

Okay, devel and main are both reformatted now, and since these are the only two branches the complete repo is reformatted.
(W.r.t. the return print: I think this was because I messed up; I played around with different styles, and the option always_use_return was applied altough it's not in the default style. I now re-applied the reformatting with the correct specifications in one go, so the return print thing is fixed.)

Okay, devel and main are both reformatted now, and since these are the only two branches the complete repo is reformatted.

Thanks! Later today I will try to implement per-commit formatting I suggested.
If that works, I'll probably force-push to this PR.


How to use terminal colors

Hi and thank you for providing this plugin!
I'm using chriskempson/base16-shell to set colors in the shell and then have other tools automatically pick them up.
Before switching to tree-sitter highlighting and this plugin, I was using chriskempson/base16-vim, which seems (or so I understand) to pick colors from the terminal using let s:cterm03 = "08"...
I tried setting such colors using require('base16-colorscheme').setup as documented but all colors turn gray. I tried setting
base03 = 08, base03 = '08'... but none work. RGB color codes such as base03='#a06949' work as intended.

Looks like we need to set ctermfg and ctermbg. In chriskempson/base16-vim, they are set with a number in [0, 21] unconditionally of the theme.
Is this an addition you'd be willing to add?

It sounds like there's some confusion here. Once you install this plugin, you need to set the specific theme you want to use: e.g., colorscheme base16-tomorrow-night. You shouldn't have to make any edits to the plugin itself.
In the plugin, the colors are set here: https://github.com/RRethy/nvim-base16/blob/f9b28409a6b6b6a4e6722fb3b7c866ae3e3b2866/lua/base16-colorscheme.lua#L58-L281. If you want to tweak any of these, you can override them in require('base16-colorscheme').setup.

Found out fnune/base16-vim fork of chriskempson/base16-vim is kept up to date with new neovim features (tree-sitter, LSP...) so that works for me.


[Hub Generated] Review request for Microsoft.DocumentDB to add version stable/2021-06-15

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
Contribution checklist:

[x] I commit to follow the Breaking Change Policy of “no breaking changes
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[x] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Please follow the link to find more details on PR review process.

Please fix LintDiff.

Loop in @msyyc @ArcturusZhang There is breaking changes report on python and cross-version.

@weidongxu-microsoft I approved the Go SDK breaking change. but however, the cross-version breaking changes need the breaking change review board to review and approve

"Cross-Version Breaking Changes" fail on stable to stable version upgrade.


booleanValid

part of the validity of the data in the JSON is checked in the constructor of the different GeometryObjects. The problem is however that the assertion error trigger in debugging/development and not in production. However, during the dev/debug they trigger and mess with the tests. On the other hand some of the tests on the polygons are unknown to me and I will need to wrap my head around them. The mixture does not help.
This issue is here for the conversation about the function.  @lukas-h, you may join the rescue effort any time. :-)

While the issue above is now solved (geojson test was checking all the test examples and would trigger error!) there is one single test that fails: './test/examples/booleans/valid/true/MultiPolygon/multipolygon-touch.geojson '
Since the function is supposed to "... check if the geometry is a valid according to the OGC Simple Feature Specification." according to L10, turfJS/b-valid and the standard states "2. The Boundaries of any 2 Polygons that are elements of a MultiPolygon may not ‘cross’ and may touch at only a finite number of points. (Note that crossing is prevented by assertion 1 above).
∀ M ∈ MultiPolygon, ∀ Pi, Pj ∈ M.Geometries(), ∀ ci ∈ Pi.Boundaries(), cj ∈ Pj.Boundaries()
ci ∩ cj = {p1, ….., pk | pi ∈ Point, 1 <= i <= k}"
and (multipolygon-touch.geojson)[https://github.com/Turfjs/turf/blob/master/packages/turf-boolean-valid/test/true/MultiPolygon/multipolygon-touch.geojson] touches apparently only in one point ('finite number of points), I conclude that we need to change the code to let this case pass.
The issue is I do not know what 'finite' should mean. Can we call one point finite? @lukas-h

@lukas-h - what do you think about this proposal: 42 seconds ago

booleanValid should not have any assertions, and should be solely based on boolean conditions, and the function returning false/true

The issue is I do not know what 'finite' should mean. Can we call one point finite? @lukas-h

Yes, let's do that.
And let's merge the feature now, with the warning, that is it not properly tested yet (early testing phase)


to_value should be bound on ToJson, not Serialize

All types that implement Serialize implement ToJson, but not all types that implement ToJson necessarily implement Serialize. So to_value() should be bound on ToJson, not Serialize, since that's all it really needs. That way I can implement ToJson on my custom types and then use them with the json!() macro.
For reference, I'm implementing ToJson on my type and not Serialize because my type can actually be serialized to both JSON and XML, but the serializations have very different representations, and I don't want to confuse anyone.

Or json! should use ToJson instead of to_value. Would that work for your use case?
I think there is value in consistency across various Serde crates and part of that is to_* functions that take Serialize as input.

I think that sounds reasonable. For my specific use case, having json! use ToJson instead of to_value() would certainly work.


Package request: mullvad-browser

Project description
The Mullvad Browser is a privacy-focused web browser developed in a collaboration between Mullvad VPN and the Tor Project. It’s designed to minimize tracking and fingerprinting. You could say it’s a Tor Browser to use without the Tor Network. Instead, you can use it with a trustworthy VPN. The idea is to provide one more alternative – beside the Tor Network – to browse the internet with more privacy. To get as many people as possible to fight the big data gathering of today. To free the internet from mass surveillance.
Metadata

homepage URL: https://mullvad.net/browser
source URL: https://github.com/mullvad/mullvad-browser
license: free
platforms: linux, darwin

that would be awesome! Lookin' forward to test it

Thank you very much, @felschr!


Build multiple QA pairs for long document

First of all thank you for a very useful package. My question is: if I have a long text document, is there any way bonito can create multiple QA pairs that cover the entire knowledge of the document? Looking at the examples, it looks like I need to break up the long document into smaller chunks manually by myself first, then feed each chunk to bonito to create a QA pair for that chunk.

Thank you for your interest in Bonito! You're right, though, unfortunately, Bonito can't generate multiple QA pairs from a long context input at the moment. Breaking the input into smaller chunks would be a nice solution. Thanks!


Fix string length

Fixed test.
Revert #27
Implement Ruby's String#length

I think that it is better to rename String::new() to String::new_usascii_unchecked().

I think that it is better to rename String::new() to String::new_usascii_unchecked().

Maybe so.  That would be a breaking change.  In https://github.com/d-unseductable/ruru/issues/67 it was discussed  to make new use the UTF-8 method internally.

I think renaming it to be explicit like you suggested is a good idea.  Any reason why you chose to have it as _unchecked?  Should the new_utf8 be changed to new_utf8_unchecked as well?

Rust's string is always UTF-8. Therefore, checking is unnecessary when converting Rust's string to Ruby's UTF-8 character string. However, when converting Rust's string to Ruby's US-ASCII, we need to check if it is correct. In that case, it will be the following API.
pub fn new_utf8(string: &str) -> Self {
    Self::from(string::new_utf8(string))
}

pub fn new_usascii_unchecked(string: &str) -> Self {
    Self::from(string::new(string))
}

pub fn new_usascii(string: &str) -> Self {
    if str.is_ascii() {
        new_usascii_unchecked(string)
    } else {
        panic!("This string contains multibyte.")
    }
}


Update README.md with LiquidityRemoved emit fixed and other small  ch…

Switched tokenAmount and ethWithdrawn in LiquidityRemoved emit to reflect the implemented LiquidityRemoved event.

	event LiquidityRemoved(
		address liquidityRemover,
		uint256 liquidityWithdrawn,
		uint256 tokensOutput,
		uint256 ethOutput
	);

Updated the Chackpoint 5: Liquidity by changing the statement that previously said k will be unchanged. As Checkpoint 3 says  "The k only changes as liquidity is added."
other small tiny grammar fixes

Thank you @GeemoCandama, lgtm!


Adding  Example

Pull Request Test Coverage Report for Build 35

3 of 3 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 92.126%

Totals

Change from base Build 31:
0.06%

Covered Lines:
117

Relevant Lines:
127

💛  - Coveralls


Rendering stops working if an XR session starts+stops without getting tracking

On both Android and iOS, rendering gets in a bad state if you start then stop XR without XR ever actually getting tracking. This can easily be reproed by just starting XR with the device laying flat on a desk. If you rotate the device 90 degrees (and the view/window is reset), then it starts rendering correctly again. This behavior was introduced with https://github.com/BabylonJS/BabylonNative/commit/23f066b567d5eb7054c984a0726e6bc59fe74d06.

When in the repro state, I can see that we are now drawing the equivalent of multiple frames in a single frame:

Compared with a working frame, we may be drawing hundreds of times:


hi Me  Again

so does anyone know how i can fix these errors?

not a bug with gorilla locomotion don't use this for that stuff


[Log View] Support secure VM logs

No extra work is needed for secure VM logs except https://github.com/bwsw/cloudstack-ui/pull/1494


Easily identify project by window skin/theming

I typically have > 6 separate instances of VS code running at one time.  Consequently, I waste a ton of time every day trying to figure out which project is which as I toggle back and forth through these different instances.
I need a way to quickly visually identify project metadata about each window at a glance.
I've been thinking about this for months - the easiest way would be if we could assign different colors to the outer window chrome and associate those with the project somehow.  A config file dedicated to this purpose local to the project would do the trick.
Here are some preliminary acceptance criteria for such a feature:

when I alt+tab I can clearly see the project/solution name in the window tile (currently this is almost always occluded bc of the way it's being rendered with other window meta)
when I alt+tab I can clearly discern the project/solution by the color of the window chrome (currently is all same color)
the above are also true for windows key+tab (desktops view)

this sounds a lot like the Peacock extension :)
https://www.peacockcode.dev/

@mroch great calll - big thanks.  Think that pretty much does it although didn't find a way yet to set peacock colors with a local config file (probably under my nose)


Fixed Link to Getting Started

The old link renders a 404 to https://jondot.github.io/formation/docs/docs/docs.html


Release Notes 2.1 iOS

https://docs.google.com/document/d/1QbRUTbCjdHpj-k4KGaAWwl76fg02qTtsShkvSl5S6oE/edit#heading=h.dy18gnowg4jx
coordinated with @gingervik, worth sharing with @azaidelson :)


[SMALLFIX] Minor optimization for Object UFS list performance

This PR saves a few RPCs to an object store for isDirectory and list calls.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/12613/
Test PASSed.

@calvinjia PTAL

LGTM

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/12631/
Test PASSed.

Merged build finished. Test PASSed.

LGTM, thanks for the improvement


Add formatting jobs to CI

As a public repository, Gematria gets unlimited free minutes for running jobs on Github CI. We've had good success using Github actions to ensure code is formatted properly/tests pass in MLGO, so I'm opening up this patch to do the same with Gematria so that it's less likely developers miss something.
Note that I'm using external dependencies here to make the setup easier, which may not be super desirable due to issues like maintenance concerns, but migrating off of them should be trivial. Something to note, however.

This is really cool, thanks!
Let's see how it works in practice, but unless the formatters change significantly, this should not generate a lot of maintenance work.


Enhance the Footer UI

Description
I noticed the footer is missing on your website. I would like to contribute by adding a professional footer to enhance the site's appearance and usability.
Screenshots

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a KWOC'24 contributor.

Code of Conduct

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

Issue 8 is already opened for the same, you can comment on that if you want to work @pragna7


Use  https://api.twitter.com/i/oauth2/authorize ... instead of https://twitter.com/i/oauth2/authorize? to disable deep links on mobile, which makes auth on mobile confusing for end users

Is your feature request related to a problem? Please describe.
The package is using https://twitter.com/i/oauth2/authorize? in generateOAuth2AuthLink at the moment. This url supports Twitter deep links. When the user tries to authenticate on a mobile device, deep links try to take the user to Twitter app. User needs to stay in the browser in order to complete authentication. If user goes to Twitter app, they cannot authenticate.
Please see this thread for more info: https://twittercommunity.com/t/disable-twitter-universal-deep-links/53580/3
Describe the solution you'd like
If we use api subdomain i.e. https://api.twitter.com/i/oauth2/authorize? ... instead of the current url in generateOAuth2AuthLink, it will not trigger deep links.
Describe alternatives you've considered
Additional context
generateAuthLink already uses api subdomain.

I am testing the fix on the repo I forked however api subdomain is not working for this url. Closing for now. If you know of another way to disable deep links for https://twitter.com/i/oauth2/authorize?... call we make from generateOAuth2AuthLink, plmk.


Option for skip gdal_fillnodata

Add an option to skip step gdal_fillnodata - to speed up processing for images that don't need this step.

workaround use -m 0 ?


CI: fix code coverage

Noticed only one of the two files was being processed. Tipped off by https://github.com/codecov/codecov-action/issues/325 that allowing automatic detection fixed their use and it appears to here as well?!

Hey, @jharley. I saw your comment over in https://github.com/codecov/codecov-action/issues/325 and was able to reproduce the issue.
Keep an eye on https://github.com/codecov/codecov-action/pull/791 since it'll fix the issue once it's merged.
In the meantime, here is a comparison of the behavior you were seeing running on a fork of this repository (diff):

https://github.com/mitchell-codecov/terraform-provider-honeycombio/runs/7513310451?check_suite_focus=true#step:10:30 (before)
https://github.com/mitchell-codecov/terraform-provider-honeycombio/runs/7514720191?check_suite_focus=true#step:8:24 (after)

Hey, @mitchell-codecov! Thanks for digging into it.. I missed the trailing whitespace but I'm sure I wouldn't have been the last either

Introduced by #195


[BUG] - No method find when migrating to Can Hub

🚨🚨 Before open the ticket please check if you can reproduce the bug in the sample code
No, I cannot reproduce the  bug in the sample code.

Lib Version [e.g. 1.1.0]
com.github.CanHub:Android-Image-Cropper:4.1.0

Describe the bug
A clear and concise description of what the bug is.
I am migrating from 'com.theartofdev.edmodo:android-image-cropper:2.8.+' to 'com.github.CanHub:Android-Image-Cropper:4.1.0'
Steps I followed:

added dependency.
added
import com.canhub.cropper.CropImage
import com.canhub.cropper.CropImageActivity

in my activity and the issue I'm facing is I cannot find --- getActivityResult(data) in CropImage class.  Also, getUri() not present in CropImage.ActivityResult. And cannot find activity in CropImage.
ScreenShots are attached for reference
Expected behavior
A clear and concise description of what you expected to happen.
Media

Smartphone (please complete the following information):
Running on Redmi Note: 7 Pro
Exact Error:
error: cannot find symbol
CropImage.ActivityResult result = CropImage.getActivityResult(data);
^
symbol:   method getActivityResult(Intent)
location: class CropImage

Next time, please add a title to your issue please =D
About your issue:

You need to use Activity Contracts, like we explain in the migration guide

=============== IMAGE ===============

=============== IMAGE ===============
How to solve it
Option 1
Use version 3.3.6 of the library, but this is not the most up to date version
Option 2
Use Activity contracts, like we have in the sample java
Would look something like this
private final ActivityResultLauncher<CropImageContractOptions> cropImage =
            registerForActivityResult(new CropImageContract(), presenter::onCropImageResult);

In this case you can create the method onCropImageResult on your presenter class or change presenter::onCropImageResult to a method in the same file, like egisterForActivityResult(new CropImageContract(), ::yourAnswerMethod);
That should have the signature:
public void yourAnswerMethod(@NonNull CropImageView.CropResult result) { ... }
And to call the crop, you will need:
CropImageContractOptions options = new CropImageContractOptions(null, new CropImageOptions())

cropImage.launch(options);

On the options you can use the settings you prefer, please check the sample code for it

@Canato I've changed my version to 3.3.6
My code is building fine and app is getting installed too, but my cropped image is not uploading.
result.getUriContent() is
content://com.google.android.apps.photos.contentprovider/0/1/content%3A%2F%2Fmedia%2Fexternal%2Fimages%2Fmedia%2F278973/ORIGINAL/NONE/image%2Fjpeg/39327663
Also,
result.getUriContent().getPath() is
/my_images/Pictures/cropped2207926145134794118.jpg
Do you think there is some issue with the path.

Getting this error in LogCat.
Unable to decode stream: java.io.FileNotFoundException: /my_images/Pictures/cropped981662640046948234.jpg: open failed: ENOENT (No such file or directory)

result.getUriContent().getPath(); should not be use, please use one of the methods we have in the README and on the sample codes, like:
result.uriContent for content://... schema or result.getUriFilePath(context, true) for file:// schema

Thanks a lot man! @Canato .  Using result.getUriFilePath(context, true) solved problem.

Happy that we got there \o/!!!