MinGW-w64 build implemented

build llvm libs since the prebuilt llvm doesn't cointain them. I recommend to use ninja since if you encounter any errors in process you won't have to start the build from scratch. https://github.com/llvm-mirror/llvm/pull/64 also would be helpful for the first time.

check if llvm-config is built too. It must be built, but if it is not (I haven't managed to build it even though the it is present in build.ninja), add -l:LLVMBinaryFormat.dll -l:LLVMCore.dll -l:LLVMSupport.dll -l:LLVMIRReader.dll -l:LLVMTransformUtils.dll  -L . to CMAKE_*_LINKER_FLAGS and copy the mentioned libs to your build directory.

You can build coriander for Windows. Though I have not tested if it works.

you must pass --target=i686-pc-windows-gnu if you use MinGW on 32-bit windows and -I<paths to include dirs for MinGW>

Doesn't work
<censored1>\coriander\build>python ./bin\cocl.py --clang-home <censored2>\LLVM-7.0.0-win32 cuda_sample.cu
./bin\cocl.py --clang-home <censored2>\LLVM-7.0.0-win32 cuda_sample.cu
INCLUDES []
LIBS []
<censored2>\LLVM-7.0.0-win32\bin\clang++ -DUSE_CLEW -std=c++11 -x cuda -D__CORIANDERCC__ -D__CUDACC__ --cuda-gpu-arch=sm_30 -nocudalib -nocudainc --cuda-device-only -emit-llvm -O2 -S -Wno-gnu-anonymous-struct -Wno-nested-anon-types --target=i686-pc-windows-gnu -isysroot <censored2>/mingw32/lib/gcc/i686-w64-mingw32 -I<censored2>/LLVM-7.0.0-win32/lib/clang/7.0.0/include -I<censored2>/mingw32/opt/include -I<censored2>/mingw32/include -I<censored2>/mingw32/lib/gcc/include -I<censored2>/mingw32/lib/gcc/include/c++ -I<censored2>/mingw32/lib/gcc/include/c++/i686-w64-mingw32 -I<censored2>/mingw32/lib/gcc/i686-w64-mingw32/include -I<censored1>\coriander\build\include\cocl -include <censored1>\coriander\build\include\cocl\cocl.h -include <censored1>\coriander\build\include\cocl\fake_funcs.h -include <censored1>\coriander\build\include\cocl\cocl_deviceside.h -I<censored1>\coriander\build\include cuda_sample.cu -o cuda_sample-device-noopt.ll
In file included from <built-in>:1:
In file included from <censored1>\coriander\build\include\cocl\cocl.h:10:
In file included from <censored2>\mingw32\lib\gcc\i686-w64-mingw32\8.1.0\include\c++\stdexcept:38:
In file included from <censored2>\mingw32\lib\gcc\i686-w64-mingw32\8.1.0\include\c++\exception:143:
In file included from <censored2>\mingw32\lib\gcc\i686-w64-mingw32\8.1.0\include\c++\bits/exception_ptr.h:40:
<censored2>\LLVM-7.0.0-win32\lib\clang\7.0.0\include\cuda_wrappers\new:41:12: error: use of undeclared identifier 'malloc'
  return ::malloc(size);
         ~~^
<censored2>\LLVM-7.0.0-win32\lib\clang\7.0.0\include\cuda_wrappers\new:58:7: error: no type named 'free' in the global namespace
    ::free(ptr);
    ~~^
In file included from <built-in>:1:
In file included from <censored1>\coriander\build\include\cocl\cocl.h:14:
<censored1>\coriander\build\include\cocl/cocl_memory.h:3:10: fatal error: 'clew.h' file not found
#include <clew.h>
         ^~~~~~~~
3 errors generated when compiling for sm_30.
Traceback (most recent call last):
  File "./bin\cocl.py", line 347, in <module>
    '-o', '%s-device-noopt.ll' % OUTPUTBASEPATH
  File "./bin\cocl.py", line 282, in run
    print(check_output(cmdline_list))
  File "./bin\cocl.py", line 49, in check_output
    res = subprocess.check_output(cleaned_cmd_list)
  File "<censored3>\Anaconda3\lib\subprocess.py", line 336, in check_output
    **kwargs).stdout
  File "<censored3>\Anaconda3\lib\subprocess.py", line 418, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['<censored3>\\Portable_Apps\\LLVM-7.0.0-win32\\bin\\clang++', '-DUSE_CLEW', '-std=c++11', '-x', 'cuda', '-D__CORIANDERCC__', '-D__CUDACC__', '--cuda-gpu-arch=sm_30', '-nocudalib', '-nocudainc', '--cuda-device-only', '-emit-llvm', '-O2', '-S', '-Wno-gnu-anonymous-struct', '-Wno-nested-anon-types', '--target=i686-pc-windows-gnu', '-isysroot <censored2>/mingw32/lib/gcc/i686-w64-mingw32', '-I<censored2>/LLVM-7.0.0-win32/lib/clang/7.0.0/include', '-I<censored2>/mingw32/opt/include', '-I<censored2>/mingw32/include', '-I<censored2>/mingw32/lib/gcc/include', '-I<censored2>/mingw32/lib/gcc/include/c++', '-I<censored2>/mingw32/lib/gcc/include/c++/i686-w64-mingw32', '-I<censored2>/mingw32/lib/gcc/i686-w64-mingw32/include', '-I<censored1>\\coriander\\build\\include\\cocl', '-include', '<censored1>\\coriander\\build\\include\\cocl\\cocl.h', '-include', '<censored1>\\coriander\\build\\include\\cocl\\fake_funcs.h', '-include', '<censored1>\\coriander\\build\\include\\cocl\\cocl_deviceside.h', '-I<censored1>\\coriander\\build\\include', 'cuda_sample.cu', '-o', 'cuda_sample-device-noopt.ll']' returned non-zero exit status 1.

clew is in a git submodule. If you run something like git submodules update --init --recursive, does that change anything?

Yes, I have managed to fix that by uncommenting the line in py file. And some other problems are fixed with clang flags enabling msvc compatibility and some other. I guess the main problem for now is with cuda_wrappers. Anyway, this PR is about building, not functioning, so I guess it may be merged even if I cannot for now make clang compile even a simple app using coriander headers + MinGW.

Fair point. Thanks! :)


Segfault when visiting https://www.kriminalomsorgen.no/

My version of PhantomJS segfaults when visiting the HTTPS version of www.kriminalomsorgen.no.  The HTTP version do not segfault.  I suspect it is related to the fact that the site currently have an invalid SSL certificate.

Which version of PhantomJS are you using? Tip: run phantomjs --version.
1.9.0

What steps will reproduce the problem?

download https://github.com/ariya/phantomjs/raw/master/examples/netsniff.js and add a line "setTimeout(function() {phantom.exit();},10000);" at the start to ensure it do not hang forever.

run "phantomjs --ssl-protocol any netsniff.js https://www.kriminalomsorgen.no/"

observe segfault

Which operating system are you using?
Linux

Did you use binary PhantomJS or did you compile it from source?
Precompiled binaries.

Please provide any additional information below.

Running the process in valgrind show this problem near the end:
==11722== Invalid read of size 8
==11722==    at 0xED2986: ??? (in /usr/lib/phantomjs/phantomjs)
==11722==    by 0xED6E24: ??? (in /usr/lib/phantomjs/phantomjs)
==11722==    by 0xEFF2A6: ??? (in /usr/lib/phantomjs/phantomjs)
==11722==    by 0xF08B1E: ??? (in /usr/lib/phantomjs/phantomjs)
==11722==    by 0xEFF1D9: ??? (in /usr/lib/phantomjs/phantomjs)
==11722==    by 0xED2E4D: ??? (in /usr/lib/phantomjs/phantomjs)
==11722==    by 0x4EEC7B: ??? (in /usr/lib/phantomjs/phantomjs)
==11722==    by 0x5277D2: ??? (in /usr/lib/phantomjs/phantomjs)
==11722==    by 0x527AE2: ??? (in /usr/lib/phantomjs/phantomjs)
==11722==    by 0x5178B1: ??? (in /usr/lib/phantomjs/phantomjs)
==11722==    by 0x47929B: ??? (in /usr/lib/phantomjs/phantomjs)
==11722==    by 0x410E1C: ??? (in /usr/lib/phantomjs/phantomjs)
==11722==  Address 0x3a is not stack'd, malloc'd or (recently) free'd
Look like a null pointer to a struct is being accessed somewhere.

Please upgrade to PhantomJS 2. 1.9.x is no longer supported, and known to have many crasher bugs that are fixed in 2 version.


TypeError: indexes.forEach is not a function

First time I try to run "Binary Search", it gives this error:

I'm not sure but I was fixing something on Binary Search at that time.
Can you check if it happens again?

It seems fine now, closing then, thanks :)

You're welcome 👍


fix a tap gesture exception

Sample Code
import 'package:flutter/material.dart';

void main() {
  runApp(BugReport());
}

class BugReport extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Directionality(
      textDirection: TextDirection.ltr,
      child: GestureDetector(
        onTap: () {
          print('outer onTap.');
        },
        child: Column(
          children: [
            GestureDetector(
              onDoubleTap: () {
                print('inner onDoubleTap.');
              },
              child: SizedBox(
                width: 100,
                height: 100,
                child: Container(
                  color: const Color(0xFF00FF00),
                ),
              ),
            ),
            SizedBox(
              width: 100,
              height: 100,
              child: Container(
                color: const Color(0xFF0000FF),
              ),
            ),
          ],
        ),
      ),
    );
  }
}

Reproduce Step
Tap one time on the 1st box then tap the 2ond box as much as you can AND quickly.
Logs
======== Exception caught by gesture library =======================================================
The following _CastError was thrown while routing a pointer event:
Null check operator used on a null value

When the exception was thrown, this was the stack:
#0      BaseTapGestureRecognizer.handlePrimaryPointer (package:flutter/src/gestures/tap.dart:229:38)
#1      PrimaryPointerGestureRecognizer.handleEvent (package:flutter/src/gestures/recognizer.dart:475:9)
#2      PointerRouter._dispatch (package:flutter/src/gestures/pointer_router.dart:93:12)
#3      PointerRouter._dispatchEventToRoutes.<anonymous closure> (package:flutter/src/gestures/pointer_router.dart:138:9)
#4      _LinkedHashMapMixin.forEach (dart:collection-patch/compact_hash.dart:397:8)
...
====================================================================================================

Root Cause
The second tap down event will win the tap gesture before the first one because no one competes with it.
Then the recognizer will call acceptGesture -> _checkUp,
here _up is not null because the first point already up and follow into _reset and clear _down.
At last, the coming move event processing triggers the exception!
Fixes #73667

@dkwingsmt Hi
In fact, the correct behavior of this scene needs to be confirmed, whether the onTap callback should be triggered twice or once (ignore the first tap, although it also won the gesture competition).
My solution is to ignore the first tap gesture.
If the right behavior is that each tap will trigger the callback we should let the tap gesture recognizer cash all of the pointers.

@dkwingsmt Hi
In fact, the correct behavior of this scene needs to be confirmed, whether the onTap callback should be triggered twice or once (ignore the first tap, although it also won the gesture competition).
My solution is to ignore the first tap gesture.
If the right behavior is that each tap will trigger the callback we should let the tap gesture recognizer cash all of the pointers.

I agree with the direction of the behavior, mostly because I think the double tap should be treated as a single tap followed by another single tap, therefore the first single tap should override the standalone tap gesture.
Though, I wonder if there's a better solution to the fix. The current fix looks like you're "patching" an incorrect state; it'll be better if the incorrectness can be detected and found in the first place (i.e. if _down is different from _up then reset _up and do necessary cleanups before accepting _down.)
Also, it seems to me that the test can be done on the recognizer level (i.e. using testGesture and put into double_tap_test.dart). Can you give it a try?
If not, this PR is looking good anyway. Let me know if you to merge as it is.

I agree with the direction of the behavior, mostly because I think the double tap should be treated as a single tap followed by another single tap, therefore the first single tap should override the standalone tap gesture.
Though, I wonder if there's a better solution to the fix. The current fix looks like you're "patching" an incorrect state; it'll be better if the incorrectness can be detected and found in the first place (i.e. if _down is different from _up then reset _up and do necessary cleanups before accepting _down.)
Also, it seems to me that the test can be done on the recognizer level (i.e. using testGesture and put into double_tap_test.dart). Can you give it a try?
If not, this PR is looking good anyway. Let me know if you to merge as it is.

But I still have a little doubt. If there is no second tap down (or after a period of time), then the first tap will win the competition and trigger the ontap callback, right? Based on this, why does a quick second tap affect the results of the first arena?

But I still have a little doubt. If there is no second tap down (or after a period of time), then the first tap will win the competition and trigger the ontap callback, right? Based on this, why does a quick second tap affect the results of the first arena?

@xu-baolin I understand your concern. Although, it's more like

The DoubleTap grabbed the ownership for the first tap, and Tap no longer has access to it.
The DoubleTap then ditched it because it didn't receive the 2nd tap. It can not transfer the ownership to Tap by this time either.

If DoubleTap has a "onFirstTap" callback (and it should), this scenario will trigger 2 taps callbacks.

@xu-baolin I understand your concern. Although, it's more like

The DoubleTap grabbed the ownership for the first tap, and Tap no longer has access to it.
The DoubleTap then ditched it because it didn't receive the 2nd tap. It can not transfer the ownership to Tap by this time either.

If DoubleTap has a "onFirstTap" callback (and it should), this scenario will trigger 2 taps callbacks.


change S3 bucket from old dvc-share to new dvc-public/{prefix}

We decided to migrate from an old S3 bucket dvc-share to use certain prefixes (folders) in a new dvc-public bucket for use in documentation. The prefixes identified now are:

code: For example code archives
data: For sample datasets
 remote: For using as an S3 remote (read-only in examples)

The main use for the bucket is to wget (download) files or use as DVC remote (read-only) in the examples throughout command references and Get Started section (as well as in https://dvc.org/doc/commands-reference/get/url) but there are other uses.

[ ] The engine has some hardcoded HTTP 301 redirects pointing to the old bucket. We need to change those.
[ ] All the examples that use either redirected dvc.org URLS or the old bucket directly need to be updated and tested.

Spanned from #487

Also found that

[ ] http://man.dvc.org/get-url gets redirected to https://dvc.org/doc/commands-reference/get/url (404 Not Found)

@jorgeorpinel let's create PRs for simple things like get-url right away ... this way we can move faster and granular

Yes I'm on that process now. Creating git branches etc (:

Yeah I shouldn't have created this issue. Sorry, my mind slipped! I guess I wanted to leave a record of the actions we're taking thus the full description above ^.
Set to be closed automatically with the new PR now.


Introduce Megalinter

What is the context of this PR?
Install and fix issues for megalinter
How to review
documentation https://confluence.ons.gov.uk/display/NWP/Megalinter?flashId=428895879
Follow-up Actions

LGTM - just tested it out! https://confluence.ons.gov.uk/display/NWP/Megalinter?flashId=428895879 will need to be updated as make docker-build and make docker-start is no longer used.


Escape event boundary + focus return to activeElement

It's been tricky trying to work around the event order of returning focus on esc from outside of details-dialog code. So I changed the code here to return focus to activeElement (in most case it'd be summary) instead of querying directly for summary.
I'd have added tests for this but looks activeElement is always body unless focus is called directly on elements. 🤔

(ﾉಥ益ಥ）ﾉ https://bugs.webkit.org/show_bug.cgi?id=112968


[bct] Investigate streamlining check reporting

Can we streamline the way we're reporting checks to not have exceptions like we do in the case of reporting module checks?

One solution is to remove module info from reported checks, however this is useful data for sorting changes based on module and helps track whether checks are in the aio or sync modules.

the CodeReporter class should simply return the list of breaking changes and folks can process the list and report the changes however they want. By default the tool can just do a simple formatting call on the checker message? Or should we have a message api in each checker


cleanup NotFound string errors

#1443  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Feature idea: Modify VSCode environment to match conanbuild.sh/conanrun.sh

Hi afri-bit!
First of all: I like your extension! The quality looks really good. 👍
In the company I'm working for, we're also using Conan (2) to manage C and C++ dependencies (most of the times using CMake).
We want to provide our developers a very smooth development experience. More precisely we want to have all extensions working within an Conan environment.
For e.g. VSCode CMake extension this works out of the box by the presets generated by Conan itself. But if other extensions (or even the Terminal) rely on some PATHs entries (or other environment variables) we don't have a sound solution at the moment.
Our workaround at the moment is: I've written a small generator that creates a .env-File and use direnv to modify the VSCode environment.
Are you interested to add such a feature to your extension? For sure, I would implement and test the feature (And discuss the approach with you upfront). 🤓
What do you think?

Hi @hackenbergstefan
thanks for using the extension, really glad you like it :)
I am open for any kind of features, that can help the developers generally. Being honest with you, this extension also doesn't contain so many workflows to work with conan properly, until now it would be just a helper extension for the developer. So if this extension can even help you and other developers to work even smoother with conan and also eventually cmake, I am really in!
Maybe to begin with it, could you provide me a small detailed explanation and/or playground repo to start experimenting and see the pain point.
PS: A pull request is also welcome 😺

Hi!
Sorry that I didn't respond for such a long time. Too much work... 🙈
One of our main use cases for Conan is developing and packaging libraries for baremetal embedded devices. We're using Conan to install the entire toolchain and other tools like CMake. A prototypical Conanfile could look like this:
class FooLibRecipe(ConanFile):
    ...
    def requirements(self):
        self.requires(
            "cmake/3.27.7",
            run=True,
            build=True,
            visible=False,
        )
        self.requires(
            "my_baremetal_toolchain/0.27.0",  # includes arm-non-eabi-gcc
            run=True,
            build=True,
            visible=False,
        )

Let's assume we're working on a vanilla system where even CMake is not installed but we want to use all cool VSCode Extensions like CMake-Extension. As VSCode is not aware of the proper PATH ingredients from Conan-Cache CMake is not available by default. (Even things like debugging the baremetal code would not work, if arm-none-eabi-gdb is installed by Conan.)
One solution is to start VSCode after setting the proper environment variables:
$ conan install . -pr:h baremetal -g VirtualBuildEnv
$ . build/baremetal/generators/conanbuild.sh
$ code .

But that's not sexy 😁
This is where VSConan comes into play! :-) VSConan could provide two commands VSConan: Activate BuildEnv and VSConan: Activate RunEnv to modify VSCode's environment properly.
I've already played around yesterday and it's not too difficult. Modification of process.env (for subprocesses) and context.environmentVariableCollection works quite well.
Maybe I'm able to provide a draft implementation within the next days...

Hi @hackenbergstefan,
too focused on your pull request, I almost forgot this issue 😄
Have you managed to provide a simple use case project, where I can play around a bit with the conan project and the environment?
Are you talking about conan2 or conan1?

Hey!
I've recorded a simple gif to demonstrate the use case. The recording uses the proof-of-concept implementation on my fork (https://github.com/hackenbergstefan/vsconan/tree/feature/environment)

In text: We want to "install" certain tools, e.g. CMake, compiler toolchains, ..., via Conan. conan install generates  conanrun.sh and conanbuild.sh which adjusts the environment variables, e.g. PATH, properly. But the changes in the environment are not reflected in VSCode.
In the recording I demonstrate how VSConan: Activate BuildEnv adjusts VSCode's terminal environment which makes CMake usable in the terminal.
Regarding the Conan compatibility: For me Conan 2 is sufficient ;-) But the use case is version independent and can be imho implemented Conan version agnostic.

Hey @hackenbergstefan,
looks really nice, great work! Hopefully my code is clear enough for you, so you didn't have difficulties to implement such a big feature :) Another pull request candidate 😄
It looks promising, i like the idea!
We need to take care of the UX also in case of the change between conan version / profile.
Have you tried to have multiple conan projects in one VSCode workspace? This is one of many points of consideration :)
I will take a look and the code, meanwhile would you like to make a pull request for this - I won't leave your name as behind as contributor for this matter :)  I would suggest a PR to develop branch, if you don't mind.

Sorry for the delay...
I've filed a PR against develop. I've left out the buttons in the statusbar for the first PR. Unfortunately it's quite big 🙈

Thanks for the PR 😃 I will take a look at your awesome work :)
PS: Meanwhile, maybe you could support me and my extension by providing feedback in the microsoft marketplace https://marketplace.visualstudio.com/items?itemName=afri-bit.vsconan, that would be really helpful to improve the existence of this extension. Thanks in advance :)

@hackenbergstefan The PR has been merged - https://github.com/afri-bit/vsconan/pull/44
Many thanks for your contribution and cooperation :)
I will close the issue for now.


Media Library - Keyboard Navigation

Feature Description
As an author I want to be able to navigate the media library via keyboard

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implement keyboard navigation.
Keyboard should trigger infinite scroll loading for next page.

QA Instructions

By keyboard navigation, do you mean being able to use the Tab key to cycle between focused elements?
If not, can you elaborate on how the navigation would work?

Tab key wouldn't work. With the infinite scrolling in the media library, you would never be able to get out of it. So should be arrow keys. See discussion at #248 about this.

@pbakaus Can you take the logic piece here since you're doing a holistic pass with kb navigation. I'll work to supply the focus state with Victor. Will update once we have the designs locked this Friday.

@samitron7 actually passing this over to Fontaine makes me most sense here - I don't think I know her Github handle, but I'll get an email thread going. My A11Y doc focuses on general guidelines. The feature specific work still needs to happen in the feature specs.

I think infinite scrolling and a11y are very much at odds, and from my research on the subject it's extremely hard if not impossible to build and infinite scrolling feed that eg: screen reader users can find usable. There's a few problems:

Users can't use tab to move around because as Pascal mentioned they would never be able to get out of the list (to navigate to other parts of the editor).
AFAIK you can't also suddenly ask them to use the Up/down/left/right keys when they happen to land in the list because they have no way of knowing that their method of input is supposed to change. So tabs should work.

I think that a good way around this is instead of automatically loading the next page, we should have a "Load more images" button at the bottom, which when clicked loads the next page (similar to what Google Search does). Then you keep on tabbing until you reach the end of the second page, click that button again, etc. WDYT @samitron7 ?

AFAIK you can't also suddenly ask them to use the Up/down/left/right keys when they happen to land in the list because they have no way of knowing that their method of input is supposed to change. So tabs should work.

I disagree. You can inform users that they can interact with a list using arrow keys, and you should do that. In fact, it's pretty common and encouraged. See "Managing Focus" section of the ARIA spec:

When the container or its active descendant has focus, the user may navigate through the container by pressing additional keys, such as the arrow keys, to change the currently active descendant. Any additional press of the main navigation key (generally the TAB key) will move out of the container to the next widget.

You cannot expect user to tab through 30 items before they can get to the next important area on the screen. The grid widget might be a suitable way for us to announce that navigation style, but there are others too. The examples list "a set of search results" as a possible use case.
You are right about the infinite scrolling part. It's difficult to get this right in an accessible way. WordPress core is even considering removing infinite scrolling for its media library. Some of the issues mentioned there:

it's impossible or very hard for keyboard users to reach content placed after an infinite scrolling region
no audible feedback or instructions about how infinite scrolling works, the current and total number of items, or when new items get loaded

So my tl;dr is:

Arrow keys for navigation
Don't mess with tabs
"load more" button instead of infinite scrolling

Note: there's the feed ARIA role for infinite scrolling, but it's not really supported by assistive technologies today

Arrow keys for navigation
This sounds good to me

Don't mess with tabs
Also sounds good mainly because we should try to reserve tab from tabbing to next section and not within items of a section

load more" button instead of infinite scrolling
//This is the only one I would like to find a better solution to but I say for now we can do the load more button. I've been tinkering with the idea of opening up a accessibility helper menu/mode where we can expose shortcut keys to help users move between large section without hitting the tab keys. Ie you enter menu and from menu you hit X key to go to Design panel or media panel inside of tabbing X times to nav between sections/actions. But I don't have time to explore this area now. @pbakaus Any thoughts on this?

re: load more: I'm not sure this is the right compromise here.. but what we could try is that we don't trigger infinite scroll when the user is using a keyboard (instead, we only trigger infinite scroll if we detect that a) the trigger element at the bottom is visible and b) there were scroll events in a short time period before on the list...or something).
This is a little tricky because I'm not sure we can reliably distinguish scroll from keyboard and scroll from pan/mouse, but we could try.

AFAIK you can't also suddenly ask them to use the Up/down/left/right keys when they happen to land in the list because they have no way of knowing that their method of input is supposed to change. So tabs should work.

I disagree. You can inform users that they can interact with a list using arrow keys, and you should do that. In fact, it's pretty common and encouraged. See "Managing Focus" section of the ARIA spec:

When the container or its active descendant has focus, the user may navigate through the container by pressing additional keys, such as the arrow keys, to change the currently active descendant. Any additional press of the main navigation key (generally the TAB key) will move out of the container to the next widget.

You cannot expect user to tab through 30 items before they can get to the next important area on the screen. The grid widget might be a suitable way for us to announce that navigation style, but there are others too. The examples list "a set of search results" as a possible use case.
You are right about the infinite scrolling part. It's difficult to get this right in an accessible way. WordPress core is even considering removing infinite scrolling for its media library. Some of the issues mentioned there:

it's impossible or very hard for keyboard users to reach content placed after an infinite scrolling region
no audible feedback or instructions about how infinite scrolling works, the current and total number of items, or when new items get loaded

So my tl;dr is:

Arrow keys for navigation
Don't mess with tabs
"load more" button instead of infinite scrolling

Note: there's the feed ARIA role for infinite scrolling, but it's not really supported by assistive technologies today

Thanks for those links and examples, I wasn't aware of the "composite" widgets in the Aria spec, looks like the right approach!


Issues on command regarding to folder path

There are issues when I am trying to use following command:
❯ pl open the folder in intellij idea
💡 Command:
  I'm sorry, but as an AI language model, I don't have access to your local files. However, the command to open a folder in IntelliJ IDEA would be: `idea /path/to/folder` (replace `/path/to/folder` with the actual path to the folder).

If I change the wording to current folder, the result is like:
❯ pl open current folder in intellij idea
💡 Command:
  I do not know.

Expected result:
idea .

Context:
alias pl="please -l"

Hi Shumin ... unfortunately not all commands will work. Please be aware that please does not know your folder structure and that we cannot change the way GPT4 is working internally here. Although the command seems logical to us, it might not be for the language model. You might tweak the prompts a bit, but every extra token will cost extra money for EVERY command. :-(

I tried to tweak the prompt a bit so it can refer current directory as . or $PWD but did not succeed. Maybe it just does not have enough data for intellij idea or GPT3.5 is not smart enough :joy:

Have you tried with the proxy and GPT4?

Seems to work with GPT-4.
`please open IntelliJ in the local folder
💡 Command:
intellij-idea-community .
❗ What should I do? [use arrow keys to navigate]
[I] Invoke   [C] Copy to clipboard   [A] Abort`

Thanks for experimenting. In some outputs it can also output something like intellij-idea . in GPT3, although the correct command is idea .
I also tried the prompt like open the folder in vscode and it worked. So probably it is because the lack of data.


cmd/link: test failure in TestLldbPython from commit  594eae5 onward

What version of Go are you using (go version)?
go version devel +b459e00454 Sat Jun 16 13:35:45 2018 +0000 linux/amd64
This is the current tip as I file this bug report, but this issue happens from 594eae5 onward; that commit is for issue #11799.
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOOS="linux"

Specifically I'm using Fedora 27, which may matter for this particular failure.
What did you do?
During an all.bash build, I get a test failure reporting unexpected lldb output.
ok      regexp/syntax   0.260s
--- FAIL: TestLldbPython (4.51s)
    runtime-lldb_test.go:178: Unexpected lldb output:
        Created target
        Created breakpoint
        Process launched
        Process state 10
FAIL
FAIL    runtime 23.172s

The Fedora 27 version of lldb appears to be 5.0.2. Based on poking around the Python lldb module, process state 10 appears to be eStateExited on my machine.
I'd be happy to add additional debugging information to the Python that runtime-lldb_test.go uses, or generally test changes.

Thank you @siebenmann for the report and for testing out the latest!
/cc @heschik @ianlancetaylor @aclements

LLDB only added support for zdebug sections last month: https://reviews.llvm.org/rL332162

We basically have three options: roll back compression, tell everyone to update lldb, or skip the test. I don't want to roll back compression. Updating lldb is going to be a hassle -- I'm not even sure if that change has hit a release yet. So I'm inclined to skip the test and we can re-enable it once lldb has released the change @aarzilli linked to.

I agree. Can we detect when we should skip the test so it's not just permanently skipped?

Okay, I skipped the test if the breakpoint doesn't match any locations. Not perfect but the best I could come up with.
@siebenmann, please try the above CL? I think it'll fix it but the test times out on my machine.

@heschik, your CL fixes the issue on my Fedora machine; all tests now pass and Go tip (+ the change) builds without problems.


Carousel: sliding wrong direction

components: Carousel
reactstrap version #9.0.1
react version #17.0.2
bootstrap version #5.1.3

What is happening?
Carousel slide in wrong direction ie. only from left to right.
What should be happening?
Slides should be sliding the same direction as indicators or control ie. moving from right to left and left to right accordingly.
Steps to reproduce issue
I created a carousel basing of Reactstrap v.8 code which works fine in  v.8.
However the difference could be observed also in examples:
https://reactstrap.github.io/?path=/docs/components-carousel--carousel - wrong
https://deploy-preview-2356--reactstrap.netlify.app/components/carousel/ - OK
Thanks for your help in advance.

We are also experiencing this

Hi,
I have investigated in order to find where the problem comes from.
The sliding direction is defined in the transformation properties in 2 differents CSS classes. One for the left to right moving and the other one for the right to left moving. Affectation of the right CSS classes is implemented in the lines 69 - 71 in the CarouselItem.js file:
          const { direction } = this.context;
          const isActive = (status === TransitionStatuses.ENTERED) || (status === TransitionStatuses.EXITING);
          const directionClassName = (status === TransitionStatuses.ENTERING || status === TransitionStatuses.EXITING) &&
            this.state.startAnimation &&
            (direction === 'end' ? 'carousel-item-start' : 'carousel-item-end');

There is a condition on the direction variable. The issue is that's the direction value in the this.context is always undefined (this.context contains { direction: undefined }). So the condition always returns the same class. And so this slides always in the same direction :(
The context value comes from the Carousel parent component where the CarouselContext.Provider is used. For me, the direction state is always set and defined by the static method getDerivedStateFromProps. This is this state.direction that is put in the context. Normally the value should be accessed in the children components, but not...
I don't understand why the direction (ie. this.context.direction in the CarouselItem.js) has an undefined value.
Does anyone have a clue?

Fixed: https://deploy-preview-2461--reactstrap.netlify.app/?path=/docs/components-carousel--carousel

hi @illiteratewriter , our first fork/PR was already behind by 40 commits due to age and had a lot of conflicts, so we've decided to re-sync and re-integrate our changes on top of 9.1.3.
here is our new PR: https://github.com/reactstrap/reactstrap/pull/2606


Ratnakeraditya04 patch 1

Please fill the details before contributing.

⚠Important!!

[x] Done reading CONTRIBUTING.md?

Adding Code PRs?🔥
⚠Do not add multiple files for the same solution!!!

[ ] Added problem statement(Readme.md) with solution?

[ ] Contributing new file?
[ ] Appending your code in already existing file?

From Platform:

[ ] Leetcode
[ ] GFG
[ ] Interviewbit
[ ] Others(only acceptable after DISCUSSIONS)

Migration PRs?🤝

[ ] Done deleteing file from home directory?
[ ] Added file with correct name and correct directory?

I errored in sending the Pull Request.
Sincere Apologies from my side.


steps for how to use weave-wiper script in kubernetes cluster.

Hi Team,
As I have been experiencing IP leakage issue in k8s 1.12.2 & 1.13.2 clusters very frequently.  I found someone created a script called wiper.bash to release unused IP's in k8s.  I did not find steps how to use it in order to release unused IPs.
I cloned the git clone https://github.com/ocadotechnology/weave-wiper.git and executed "sh wiper.bash".  However, I ended up with below error message.
[root@ip-172-31-0-133 weave-wiper]# sh wiper.bash
wiper.bash: line 97: `T_ids_wrong_files_validation_containers-json': not a valid identifier
[root@ip-172-31-0-133 weave-wiper]#
Could you help me how to use it in kubernetes clsuter.

@kasim457 Closing the bug as per slack discussion to open a issue in weave-wiper repo.


Decrease memory usage

The stress memory example increase from 1.68 GB to 2.42 GB in memory usage, likely after supervisors were implemented.

#318 decreased the memory usage drastically.


[CI][C++] Google protobuf fails to compile on source verification jobs

Describe the bug, including details regarding any error messages, version, and platform.
There are several failures on the source verification jobs. Some of the jobs were failing on the past but some of them have started to fail during the last 3 days:
verify-rc-source-cpp-linux-almalinux-8-amd64
verify-rc-source-cpp-linux-ubuntu-20.04-amd64
verify-rc-source-cpp-linux-ubuntu-22.04-amd64
verify-rc-source-integration-linux-almalinux-8-amd64
verify-rc-source-integration-linux-conda-latest-amd64
verify-rc-source-integration-linux-ubuntu-20.04-amd64
verify-rc-source-integration-linux-ubuntu-22.04-amd64
verify-rc-source-integration-macos-amd64
verify-rc-source-integration-macos-conda-amd64
verify-rc-source-python-linux-almalinux-8-amd64
verify-rc-source-python-linux-ubuntu-20.04-amd64
verify-rc-source-python-linux-ubuntu-22.04-amd64
verify-rc-source-ruby-linux-almalinux-8-amd64
verify-rc-source-ruby-linux-ubuntu-20.04-amd64
verify-rc-source-ruby-linux-ubuntu-22.04-amd64
The changes introduced when the new jobs started failing are the following: https://github.com/apache/arrow/compare/51817917e1436c8799ed382d160798060cd76652...fa36cde1105c8c82ef6407cb991b83d1ca67258a
The current stack trace:
 [190/205] Building CXX object CMakeFiles/libprotobuf-lite.dir/tmp/arrow-HEAD.K9cQf/cpp-build/protobuf_ep-prefix/src/protobuf_ep/src/google/protobuf/message_lite.cc.o
In file included from /usr/include/string.h:495,
                 from /tmp/arrow-HEAD.K9cQf/cpp-build/protobuf_ep-prefix/src/protobuf_ep/src/google/protobuf/stubs/port.h:39,
                 from /tmp/arrow-HEAD.K9cQf/cpp-build/protobuf_ep-prefix/src/protobuf_ep/src/google/protobuf/stubs/common.h:48,
                 from /tmp/arrow-HEAD.K9cQf/cpp-build/protobuf_ep-prefix/src/protobuf_ep/src/google/protobuf/message_lite.h:46,
                 from /tmp/arrow-HEAD.K9cQf/cpp-build/protobuf_ep-prefix/src/protobuf_ep/src/google/protobuf/message_lite.cc:36:
In function 'void* memcpy(void*, const void*, size_t)',
    inlined from 'uint8_t* google::protobuf::io::EpsCopyOutputStream::WriteRaw(const void*, int, uint8_t*)' at /tmp/arrow-HEAD.K9cQf/cpp-build/protobuf_ep-prefix/src/protobuf_ep/src/google/protobuf/io/coded_stream.h:684:16,
    inlined from 'virtual uint8_t* google::protobuf::internal::ImplicitWeakMessage::_InternalSerialize(uint8_t*, google::protobuf::io::EpsCopyOutputStream*) const' at /tmp/arrow-HEAD.K9cQf/cpp-build/protobuf_ep-prefix/src/protobuf_ep/src/google/protobuf/implicit_weak_message.h:103:28,
    inlined from 'bool google::protobuf::MessageLite::SerializePartialToZeroCopyStream(google::protobuf::io::ZeroCopyOutputStream*) const' at /tmp/arrow-HEAD.K9cQf/cpp-build/protobuf_ep-prefix/src/protobuf_ep/src/google/protobuf/message_lite.cc:411:30:
/usr/include/x86_64-linux-gnu/bits/string_fortified.h:34:33: warning: 'void* __builtin___memcpy_chk(void*, const void*, long unsigned int, long unsigned int)' specified size between<PHONE_NUMBER>1562067968 and<PHONE_NUMBER>3709551615 exceeds maximum object size<PHONE_NUMBER>854775807 [-Wstringop-overflow=]
   34 |   return __builtin___memcpy_chk (__dest, __src, __len, __bos0 (__dest));
      |          ~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Component(s)
C++, Continuous Integration

Hmm. This may be related to the recent https://apache.jfrog.io/ problem.
https://github.com/ursacomputing/crossbow/actions/runs/8409407364/job/23026685294#step:6:4807
-- verifying file...
       file='/tmp/arrow-HEAD.aCHiU/cpp-build/boost_ep-prefix/src/boost_1_81_0.tar.gz'
-- SHA256 hash of
    /tmp/arrow-HEAD.aCHiU/cpp-build/boost_ep-prefix/src/boost_1_81_0.tar.gz
  does not match expected value
    expected: '9e0ffae35528c35f90468997bc8d99500bf179cbae355415a89a600c38e13574'
      actual: '205666dea9f6a7cfed87c7a6dfbeb52a2c1b9de55712c9c1a87735d7181452b6'
-- Hash mismatch, removing...
CMake Error at boost_ep-stamp/download-boost_ep.cmake:170 (message):
  Each download failed!

Yes, it seems to be the case as I retried some jobs that are successful now. I am closing it.


Issue with downloading pics from twitter link

The bot is having trouble downloading multiple pics from twitter links, just downloading the first pic.
Example tweet: https://twitter.com/channel_1231/status/991726700676333568?s=21

fixes in https://github.com/Seklfreak/discord-image-downloader-go/releases/tag/v1.28


LiDAR demo bags

Thank you for your excellent work. However, I didn't see the LiDAR demo bags in the outdoor folder of the raw sensor data. Have they not been uploaded yet?

Thanks for catching this! I did not realize that the upload was interrupted. I just re-uploaded it. You can find it here: https://drive.google.com/drive/folders/1O9MnN7HuVifYdxr01L_CAuOfSKppmTtK


Crate naming and folder structure, continued

@brenzi asked me to give some input on the changes we've made in #348 . Since those changes were merged (PR #378 ) and the issue closed, I decided to open another issue, to discuss the potential next iteration of re-naming.
primitives

We have a primitives folder (with itp- prefix) but also a sidechain-primitives crate that is named its-primitives

First option would be to have it moved to primitives and name it itp-sidechain ?
Otherwise we need a naming to distinguish the top-level primitives from the ones nested in other crates (something like core-primitives was already discussed?)
In light of refactoring the e-call and o-call crates (#316 #288), I'd suggest adding an enclave-tunnel sub-folder to primitives and place all the o-call and e-call crates in there

core

We have a core/ folder with itc prefix, but also a primitives/core crate. Maybe re-name itp-core to itp-types, as suggested by @clangenb in #348

untrusted worker / service

I'm not entirely sold on the service name for the untrusted worker. It should be more clear that this is the main entry point of the entire worker, and I don't feel like service does it justice.
Did we have a good name for this component in the past, or was it just called the 'untrusted worker'? I believe our goal will be to make this component as lean as possible, so it should end up being a simple worker- 'runner', 'wrapper', 'vessel' or 'hull'?

client / cli

The client crate was re-named to cli, but not the folder that contains it. We'd probably want to rename the folder to cli as well?

folder name should be cli too, yes
service can be worker or validateer, that's why I want to drop worker as a toplevel name
I agree with you suggestions re 'primitives'
@clangenb ?

Primitives

I don't really have an opinion here. I think I slightly prefer to keep it in sidechain and name it its-sidechain-primitives. It carries redundant information, but at least you don't mix it up.

core

Yes, core should be merged into types. This was already agreed on, but somehow forgotten in the process. Good catch!

worker

I agree with service being suboptimal, also because substrate has a service, which is not the main binary. So, we use the term differently.

I'd prefer not to have it name: `vessel' or 'hull' because  these are seldom used terms.

Wrapper does not imply a binary for me, so I am against that too.

runner would somehow work out.

validateer. I thought this was not an option because we also want to run it without sidechain features? However, it does still perform light validation of the parentchain regardless, so it's maybe not bad after all. What I'd like about validateer, is that it uniquely stands for our project! So I vote for this one.

cli

yes, this should be renamed.

Overall, good input @mullefel!

service is suboptimal, yes. But not unacceptable IMO. validateer is simply not what a worker is so I object. runner doesn't strike me as appropriate neither. Is main a nogo? Or server?
@mullefel please summarize the actions here so we can assign @echevrier to finish this today

Worker is also used as repository name and in the document. We should also consider how to use it there

primitives

I can definitely see why it makes sense to keep the sidechain primitives in the sidechain sub-folder and then be called its-primitives.
Since we might have more components in the future that have their own primitives, it might make more sense to re-name our primitives folder to something like core-primitives, base-primitives or common-primitives ? What do you think?

core

Looking that the primitives/core crate now, are the remaining types all related to RPC? If so, we could re-name it to itp-rpc? Or would you rather still merge it into primitives/types @clangenb ?

untrusted worker / service

I think I would still prefer service over main. server is probably on the same level as service for me. I'd say we keep service for now and see if we can think of something better in the future?

@brenzi: I suggest I'll close this issue once the discussion is over, and open a new one for @echevrier with all the tasks that resulted from the discussion - if that is okay?

yes: its-primitvesin the sidechain folder
yes: rename primitives folder to core-primitives
yes: keep service

@mullefel please summarize with everything above and assign @echevrier . no new issue needed IMO. just a clear list of what should be done.

Summary and Tasks
for @echevrier
primitives

[ ]  move content of primitives/core crate to primitives/types, and then remove primitives/core
[ ]  re-name primitives folder to core-primitives

client / cli

[ ] re-name client folder to cli

What about the crate name of the primitives if we rename the primitives folder to core-primitives ? It remains itp-*?

@mullefel please check the remaining worker.
There are still some worker that have not been renamed, those I'm not sure if they have to become service

not all occurrences of "worker" must be replaced

the crates in core-primitives will keep the itp-* prefix
I think we keep the current state of service and worker as is


Require of different mod with Lua snippet (Lua API global Variable Viewer (gvv))

Hi there,
First of all, huge thank you for all the work you have been doing for typed-factorio!
I'm currently developing a mod which I would like to debug by using: https://mods.factorio.com/mod/gvv
The mod requires the use of the following lua snippet:
    if script.active_mods["gvv"] then
        require("__gvv__.gvv")()
    end

I have been struggeling but so far I have managed to get it working by doing the following:
//debug.lua
local debug = {}

function debug:setup_gvv()
    if script.active_mods["gvv"] then
        require("__gvv__.gvv")()
    end
end

return debug

// control.ts
setup_gvv();

This works and I am able to use GVV succesfully. However I keep receiving the following error in the console of my IDE
error TSTL: Could not resolve lua source files for require path '__gvv__.gvv' in file debug.lua.
Am I importing it wrong or is there a different way to translate the Lua snippet to Typescript?
Very much thank you in advance!

This is an issue with TypescriptToLua, you can find more details here (give it an upvote): https://github.com/TypeScriptToLua/TypeScriptToLua/issues/1152
As a workaround, either extract "__gvv__.gvv" to a local variable before passing it to require, or change it to "@NoResolution:__gvv__.gvv"

It worked, thank you so much!
Upvoted the thread as well 👍


BigInt error in production build, works fine in dev mode

Describe the bug
Adding the following line causes an error in a site built with react-scripts build, but works fine with react-scripts start

Did you try recovering your dependencies?

yes
Which terms did you search for in User Guide?

(Write your answer here if relevant.)
Environment

Environment Info:
current version of create-react-app: 4.0.3
running from C:\Users\lain\AppData\Local\npm-cache_npx\c67e74de0542c87c\node_modules\create-react-app
System:
OS: Windows 10 10.0.19042
CPU: (32) x64 AMD Ryzen 9 5950X 16-Core Processor
Binaries:
Node: 16.13.0 - C:\Program Files\nodejs\node.EXE
Yarn: 1.22.17 - ~\AppData\Roaming\npm\yarn.CMD
npm: 8.1.0 - C:\Program Files\nodejs\npm.CMD
Browsers:
Chrome: Not Found
Edge: Spartan (44.19041.1266.0), Chromium (96.0.1054.41)
Internet Explorer: 11.0.19041.1202
npmPackages:
react: ^17.0.2 => 17.0.2
react-dom: ^17.0.2 => 17.0.2
react-scripts: 4.0.3 => 4.0.3
npmGlobalPackages:
create-react-app: Not Found
Steps to reproduce

run npx create-react-app .
add const TEST = 2n ** 3n; to the top of index.js
run yarn build and serve build to see the error
run yarn start to see it work correctly

Expected behavior

The page loads without error.
Actual behavior

The page does not load, there is an error in the console.
Reproducible demo

https://github.com/0xlain/cra-bug

I was encountering this issue and I was able to overcome it by installing the @babel/plugin-syntax-bigint package. So I ran yarn add @babel/plugin-syntax-bigint, then in my babel.config.json file I added it to the plugins array like so:
  "plugins": ["@babel/plugin-syntax-bigint"]
}```

Also, wherever you are deploying it to don't forget to clear your cache/delete node_modules

https://babeljs.io/docs/en/babel-plugin-syntax-bigint

That fix didn't work for me unfortunately. @0xlain did you ever figure out a solution to this issue?

That fix didn't work for me unfortunately. @0xlain did you ever figure out a solution to this issue?

Nope, sorry.

I found a fix in the end, instead of 2n ** 3n I did Math.pow(2n, 3n).


[MRESOLVER-327] Allow use of system properties to configure HttpClient

But do not encourage users about this, quite the opposite, drive them toward documentation how to configure transports, for example this page: https://maven.apache.org/guides/mini/guide-proxies.html

https://issues.apache.org/jira/browse/MRESOLVER-327

Am merging this as doco was updated, and is also safe as all this is disabled by default. The only reason for this change is to make possible exact use case like the reported in https://issues.apache.org/jira/browse/MNG-7721 to work with transport-http, nothing more and nothing less.


Use serial console instead of OCR

VirtualBox can expose a serial port, and *BSD guests can use it at bootup. That should be used instead of screen scraping text.

In the FreeBSD image,
cat <<EOF > /boot/loader.conf
boot_multicons=YES
boot_serial=YES
comconsole_speed=9600
EOF

serial.log
Then with VirtualBox
vboxmanage modifyvm freebsd --uart1 0x3F8 4 --uartmode1 file serial.log

then I get (attached)

For compatibility, this could probably just check for a hardcoded serial log name and fallback to current method if it's missing


Removed code that ignored function arguments when comparing the exp...

...ected and actual args in 'toBeCalledWith'
Also see: https://github.com/facebook/jest/issues/211

This stuff has changed quite a lot so I don't think this pull request is appropriate any longer. Feel free to re-open if you think this still makes sense and please rebase on master!


Jquery 3.x issues

It seems that visibly is not compatible with JQuery 3.x
I get it only working with version 2.2.4 but when using JQuery 3.x the related select box can't be change anymore

I'm also trying to get this little great little plugin working with jQuery 3.x
Any suggestions?

Hey guys,
I have not tested this at all with jQuery 3.x, I will see if I can find cause.  If you have any jsfiddles that demonstrate your issues to assist that will surely help.


mercurial: 3.7.1 -> 3.7.3 for multiple CVEs

Please backport to 16.03
Things done:

[x] Tested using sandboxing (nix-build --option build-use-chroot true or nix.useChroot on NixOS)
Built on platform(s)

[x] NixOS
[x] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

CVE-2016-3068
Blake Burkhart discovered that Mercurial allows URLs for Git
subrepositories that could result in arbitrary code execution on
clone.

CVE-2016-3069
Blake Burkhart discovered that Mercurial allows arbitrary code
execution when converting Git repositories with specially
crafted names.

CVE-2016-3630
It was discovered that Mercurial does not properly perform bounds-
checking in its binary delta decoder, which may be exploitable for
remote code execution via clone, push or pull.

@edolstra can you cherry-pick to the release?

Picked to release at 3c534c83cedc4cb448c0688f5016f4f4a27c3fd9

Thank you!


feat: documenting library compatiblity & BeforeExecuting Feature

@bzbetty this is still marked as draft. Is there a dot point list of what you'd like to see added/completed to finish this off?

Was waiting on info about hooking into pre execution, but it sounds like you didn't have much luck there?

so having a quick look at the preexecute hook,
would it not go in
ExecutableGraphQLStatement.ExecuteExpressionAsync()
between
var lambdaExpression = Expression.Lambda(expression, parameters.ToArray());
and
Compile().DynamicInvoke
Essentially a new property on ExecutionOptions ?
ExecutionOptions.OnBeforeExecute(LambdaExpression);
Thoughts?

That is what I first though and quickly added it. Issue is that not all expressions have a IEnumerable<>/IQueryable<> result. Hence you can not cal any of the extension methods designed for that.
e.g. GraphQL
query Movie($id: ID!) {
   movie(id: $id) { name }
}

this creates a similar expression to this
ctx.Movies.Where(m => m.Id == args.id).FirstOrDefault()

You can not just wrap that in a method call expression that takes a IEnumerable<>/IQueryable<> as the argument. Depending on the schema configuration that expression might actually be more like
something == null ? null : new {
   field = something.Field
}

I haven't spent much more time thinking about it yet, but I think the few places we know we've finished a IEnumerable<>/IQueryable<> expression and are about to wrap in a null check or similar we can do the call back then.

#253

tested out ExpressionOptimizer a bit, doesn't seem that useful in my case, although might help someone else I guess.

Related to this but more to method fields as query field using [GraphQLField] I wanted to get some thoughts...
Consider added these to the Movie type in the Demo project
        [GraphQLField]
        public uint DirectorAgeAtRelease => (uint)((Released - Director.Dob).Days / 365);

        [GraphQLField]
        public uint[] AgesOfActorsAtRelease()
        {
            var ages = new List<uint>();
            foreach (var actor in Actors)
            {
                ages.Add((uint)((Released - actor.Person.Dob).Days / 365));
            }
            return ages.ToArray();
        }

By default EF doesn't handle this. The expression built by EntityGraphQL calls the method. EF at least knows this and tries to execute it after it has fetched what it does understand.
Issue is that in DirectorAgeAtRelease the Director property will be null as EF doesn't know to fetch it. In AgesOfActorsAtRelease the Actors property will be null for the same reason.
One solution is Microsoft.EntityFrameworkCore.Proxies to lazy load the properties, which comes with its own tradeoffs.
Another one is EntityFrameworkCore.Projectables which will work for DirectorAgeAtRelease by not for AgesOfActorsAtRelease. At least it will give a compiler error if you try to add [Projectable] to DirectorAgeAtRelease. But if you leave it off, and you use EF it will fail.
I feel like including EntityFrameworkCore.Projectables in the library for this would be useful, but then this library probably shouldn't be pushing the user down a single path. I.e. if they want to use lazy loading.
So that leaves me with somehow making it really clear in docs etc the limitation of marking a method with `[GraphQLField] when using EF and the options around that.
I think the last point is the right direction, but just wanted to write out my thought and get some other thoughts on it, if you have any @bzbetty?

I wouldn't include either by default in the library, but I think calling them out in documentation and/examples makes a lot of sense.


Create/update logic and homeId blockers

Exploring the Atlas API to create/update an entity, it seems that a self-defined GUID cannot be used (putting a GUID into the request body forces Atlas to attempt an update, which then immediately fails as the GUID provided does not yet exist).
Furthermore, even when leaving the GUID off but including a homeId, the homeId is not stored and entities are still fully mutable (ie. via the UI).
These seem to be fundamental blockers to achieving anything other than a purely read-only connector for Apache Atlas...

Actually, this was specifically for a glossary and a category...  Maybe the Glossary API must explicitly be used for these particular entities?

... or not -- doesn't seem that homeId even exists on the Glossary API objects / payloads.

Clarification: above points mean that Atlas cannot currently handle reference copies; however, it would likely still be capable of handling master (Atlas-homed) metadata -- with the slight caveat of #3 meaning that fully master metadata support could not (yet) extend to glossary-related entities.
Open question is whether there would be demand for homing metadata in Atlas via the Egeria APIs rather than via Atlas UIs / APIs directly -- otherwise there may be little benefit to extending to cover only master metadata (and still lacking reference copies).

@cmgrote This is addressed in https://issues.apache.org/jira/browse/ATLAS-3755 with a patch available. Would be great if you could test it. Note that it is now a polict item whether someone can create/update system attributes like homeId.
Glossary/Category API has not been adjusted yet


"Connection Error: Unable to find Agda Language Server" Error downloading language server?

I've gotten this issue on two different machines now, both running Linux Mint, on the primary one I tried the issue appeared when I pointed agda-mode to both agda 2.6.3 and agda 2.6.4.
Here are the error messages from all the attempts:
Trying to connect to localhost:4096 : Error: connect ECONNREFUSED <IP_ADDRESS>:4096
Trying to find the command "als": Cannot find the executalbe on PATH
Trying to download prebuilt from GitHub: Cannot get release info from GitHub:
Cannot parse downloaded file as JSON:

Request forbidden by administrative rules. Please make sure your request has a User-Agent header (https://docs.github.com/en/rest/overview/resources-in-the-rest-api#user-agent-required). Check https://developer.github.com for other possible causes.

I've tried manually downloading the ubuntu release and making it available in my PATH, but when I do that I get errors as described in THIS issue for the agda language server. Maybe I was supposed to put the release somewhere else but I wasn't sure how it's supposed to be installed if installed manually.
edit: if relevant, this is my first time using emacs and I'm relatively new to all things cabal

https://github.com/agda/agda-language-server/issues/15 is the duplicate of https://github.com/banacorn/agda-mode-vscode/issues/122. I also stumbled across this one and waiting for a solution :).

Hey guys, sorry for the inconvenience!
I've just attended that last Agda Implementors' Meeting.
We're working on this now!

Fixed in v0.4.7

It works, thank you!


SharedComponentPropertiesBindingsValuesProvider should support LazyBindings

Required Information

AEM Version, including Service Packs, Cumulative Fix Packs, etc: AEM 6.5, AEM CS
ACS AEM Commons Version: all
Reproducible on Latest? yes

Expected Behavior
The SharedComponentPropertiesBindingsValuesProvider always resolves all values before they are added to the bindings; this happens irrespective if these values are required or not.
When using Lazybindings (https://sling.apache.org/apidocs/sling12/org/apache/sling/api/scripting/LazyBindings.html) this could be avoided, and values are only resolved when they are needed.
Actual Behavior
the values are always computed, leading into performance degradation.

@joerghoh TL;DR: To take advantage of LazyBindings, we have to implement the Supplier type it introduces. Unless we use reflection to access the LazyBindings type, this will require an upgrade of our org.apache.sling.api dependency version to at least 2.22.0, which will require dropping support for AEM versions prior to 6.5 SP7.
org.apache.sling.api.scripting.LazyBindings was introduced in org.apache.sling.api:2.22.0. Between this and its previous release artifact, the org.apache.sling.api.scripting package exported version was bumped from 2.3.2 to 2.5.0, which will introduce a breaking change to the Import-Package range for deployment to previous AEM releases (the current version of org.apache.sling.api we depend on is 2.16.4, which exports the .scripting package at version 2.3.1) . The latest version of this package in the apache/sling-org-apache-sling-api master branch is still 2.5.0.
In this project, the org.apache.sling.api dependency version is transitively managed by the io.wcm.maven:aem-dependencies pom import. In both our master branch and the acs-aem-commons-6.0.0 branch, that pom version is currently set to <IP_ADDRESS>4. This pom import dependency will need to be upgraded to at least <IP_ADDRESS>0 to bring along the necessary org.apache.sling.api version without adding an explicit dependency to the acs-aem-commons-bundle. The latest incremental version of the 6.5.7 (AEM 6.5 SP7) pom is <IP_ADDRESS>2, while the latest AEM SP pom is <IP_ADDRESS>0. The same org.apache.sling.api:2.22.0 dependency version is defined for the entire range between these versions, so while I will likely include an upgrade to <IP_ADDRESS>2 in my PR, it may spark a discussion about which AEM 6.5 SP level the project should support when the decision is made to drop support for AEM 6.4.


Update README.md Tishanth

Comments better show the creators thoughts while creating some code while also making stuff easier for readers.


Reference another local library in a project.json file

Maybe not an issue, but I have been trying to figure out if I can do this.
I have an asp.net app and in it's project.json file I want to reference a package that I am also developing locally (in another path) as I have not published it to nuget yet, no want to while developing it.
Anyway to do this? Or any ideas?
Thanks

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


question

hello, I have installed all, but any mistakes are still returned, what do I need to do?
D:\Users\anaconda\python.exe E:/知识/GS-Meta-init/run.py
Traceback (most recent call last):
File "E:\知识\GS-Meta-init\run.py", line 65, in
main()
File "E:\知识\GS-Meta-init\run.py", line 55, in main
torch.cuda.set_device(int(args.gpu))
File "D:\Users\anaconda\lib\site-packages\torch\cuda_init_.py", line 404, in set_device
torch._C._cuda_setDevice(device)
AttributeError: module 'torch._C' has no attribute '_cuda_setDevice'

Hi, this may help you. https://github.com/pytorch/pytorch/issues/33655
If your server does not support GPU, please set the gpu parameter in https://github.com/HICAI-ZJU/GS-Meta/blob/main/args_parser.py#L13 to -1 to use CPU.


Maximum file size 8 Gigabyte (8000 MB)

So 8GB or unlimited?
https://skysend.hns.siasky.net/#b-ZAD8OE8NwtYKY6_G7xBeSnaE_WZkSyektnj_bNOBQ3JiaQ+9U2kuWYzcSAy4WODhCos7z7d+U3UNObRyP85bcike3ixu5IDIHjYMj/CUKYUzLmW

No limit is enforced, but problems may occur with larger files (especially in the browser)

No limit is enforced, but problems may occur with larger files (especially in the browser)

Understood, thank you.

Understood, thank you.


Inconsistent Spot Dimensions

Hi,
I am using the same dataset and your BLEEP code to reproduce your results. However, after training, I found that the ground truth spot expression has a shape of (4992, 3559) whereas the predicted spot expression has a shape of (4992, 256) as in training phase, we trained both image encoder and spot encoder to the dimension of 256. I am assuming that in your BLEEP inference.ipynb and BLEEP main.py, "data/filtered_expression_matrices/3/harmony_matrix.npy" is the same expression matrix that we have used for the BLEEP training part. In the end, I am getting different embedding dimensions. Please share your data (only harmony_matrix) so that I can see where I am wrong or guide me to reproduce the correct results.[
Here is my COLAB file for your reference.
https://colab.research.google.com/drive/1hp-1CWZbDCgXMf-LXwbcH8HhZQHJHtH-?usp=sharing](url)

Hi, the data associated with the experiments were in review but are now public, so I have released a version of it on github along with the preprocessing script and other relevant outputs. Please let me know if you run into any additional issues.
The ground truth spot expression is a spot x gene matrix, where as the embedding from the image and spot encoder are 256d representations which are used for neighbor matching.
Also, it's recommended to filter the raw expression matrices you are currently using with the tissue_pos_list.csv (the column after barcodes) for detected spots prior to training the model. Please let me know if there are any other questions!

Hi, I took a brief look at the code. The problem likely occurred at the hvg filtering step. Since your preprocessing script modified the order and total number of gene features of the input matrices, the resulting hvg_matrix.npy features no longer line up across the 4 slices. I would recommend following the preprocessing script I've uploaded.
Regarding overfitting, the best models tend to occur between epoch 20-80 from my experience. Let me know if you are able to reproduce.

Hi, thanks for your reply. However, the outputs shown in the code file (I've attached above in the comment) are produced by your uploaded data, i.e. harmony_matrices and tissue_position_list directly into the CLIPDataset() for training and testing.  So, the data produced above in the form of pre-processing is not being used.

Hi,
I did another iteration. Now seems like the model is not overfitting and converged. However, the mean co-relation is very low (0.0002856....), even lower than the overfitted model (0.015). You can see in the code that I am using your uploaded data. Also, it is the 4th time, I ran the same model on the same data and got these converged results. It seems like there is an issue with setting up the seed, or the model is not generalizing well. As per your advice, I set the epochs to 100.
See the code in the link:
https://livemanchesterac-my.sharepoint.com/:u:/g/personal/muhammadahtazaz_ahsan_postgrad_manchester_ac_uk/EVWGmr1Otj5EtWCflCtT_SwBHnVRHKvkSsfHGkeuJkR_1A?e=PpaCTX
What do you say?

My apologies, I didn't realize in your notebook the data was later switched to the ones uploaded. Looking at the results of the first notebook more closely, you did successfully reproduce the results to a large extent. Your best model was around epoch 30 and cell wise correlation looks good. The inference script outputs the mean correlation across all genes but that's not the reported value in the manuscript. The low correlation mean across all genes is expected due to the ill-posed nature of the problem. We only expect models to predict the expression patterns of a small handful of genes faithfully.
So instead, during evaluation, we focused on reporting and comparing the performances on marker genes, hegs and hvgs due to their practical relevance. I pushed a modified version of the inference code to calculate these. Here are the results from a model I previously trained using the same github code. They are not so far off from the values reported in table 1 in the manuscript, and likely from that of your trained model. As for the remaining difference in results, it may be due to the environment or the way distributed training is performed on the cluster. Of note, training is done using 4 gpus each with 128 batch size without concatenating the logits together at each step.

Hi, I got the following results. Please have a look and suggest what to do. I have updated the inference code that you provided yesterday. Only for marker genes, my model is giving correlation value of 0.170 which is not too far from 0.25.

I am using only one GPU (A100) machine to train the model. Should I change the batch size to 128 and reduce the number of epochs, say 35? Also, Why the model is behaving randomly? For multiple iterations on the same data and giving different results.

Hi, the results you got from the 4th run is strange. The model likely collapsed to a trivial solution. I have never experienced this. It may be due to the lack of distributed training where loss is computed from a effective batch size of 128 and then reduced across 4 instances before backpropagation. This likely offers more stability during training and better performance than one instance of batch size 512 due to the smoothed loss being dependent on the embeddings.

Hi, thanks for timely reply. I got the following results. Please have a look and suggest what to do. I have updated the inference code that you provided yesterday. Only for marker genes, my model is giving correlation value of 0.170 which is not too far from 0.25.

I am using only one GPU (A100) machine to train the model. Should I change the batch size to 128 and reduce the number of epochs, say 35? Also, **Why the model is behaving randomly? For multiple iterations on the same data and giving different results.**

Hi, thanks for the reply again. What do you say about these results? Also, why mean co-relation is greater than maximum correlation?
Also, I performed another experiment, in which I took 199 genes (common and highly variables across all cells) and its results are below.  Again, the mean corelation is higher than the maximum. Can you please explain?

Hi, the mean is computed across cell-wise correlations, whereas the max correlation is calculated from correlation gene-wise. Regarding your results above, I suspect it's also partly due to the difference in distributed training. During my experiments I didn't concatenate the embeddings produced from each individual batch of 128 prior to calculating loss, but rather only reduce the loss itself. Hence, it is functionally different from one batch of 512. This may lead to better results compared to your reproduction.


[Task]: Hypothesis for Start with data Excel/CSV File upload

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks
Complete the hypothesis doc with all the different proposed solutions.
Ref- https://www.notion.so/appsmith/By-allowing-users-to-upload-data-from-Excel-CSV-file-we-ll-increase-the-of-users-who-get-activate-6afc83b06d3a4137b29690ccc23d61e7

Have added the solution designs in the notion doc after discussion with @sumitsum


Require Bitwasp "^1.0"

Require Bitwasp "^1.0" for old version causing conflict on composer

Looks fine, tests still work.  thanks for the PR!


Assert.Equal() Failure
Expected: 1
Actual:   0
   at Microsoft.AspNetCore.Hosting.WebHostTests.WebHostShutsDownWhenTokenTriggers()

http://aspnetci/viewLog.html?buildId=208061&buildTypeId=XPlat_Windows_Win8_Universe&tab=buildLog

cc @Tratcher

Duplicate https://github.com/aspnet/Hosting/issues/1024


Candidate was blocked due to SAFETY at the time when the BLOCK_NONE setting was set

Description of the bug:
[GoogleGenerativeAI Error]: Candidate was blocked due to SAFETY
result.response.candidates[0].safetyRatings has the following value
 [
  { category: 'HARM_CATEGORY_SEXUALLY_EXPLICIT', probability: 'LOW' },
  { category: 'HARM_CATEGORY_HATE_SPEECH', probability: 'NEGLIGIBLE' },
  { category: 'HARM_CATEGORY_HARASSMENT', probability: 'HIGH' },
  {
    category: 'HARM_CATEGORY_DANGEROUS_CONTENT',
    probability: 'NEGLIGIBLE'
  }
]

My model.safetySettings
[
  { category: 'HARM_CATEGORY_HATE_SPEECH', threshold: 'BLOCK_NONE' },
  {
    category: 'HARM_CATEGORY_SEXUALLY_EXPLICIT',
    threshold: 'BLOCK_NONE'
  },
  { category: 'HARM_CATEGORY_HARASSMENT', threshold: 'BLOCK_NONE' },
  {
    category: 'HARM_CATEGORY_DANGEROUS_CONTENT',
    threshold: 'BLOCK_NONE'
  }
  ]

Actual vs expected behavior:
Content should not be blocked.
Any other information you'd like to share?
No response

Hello, @kpripper ! Well...even if you set it to 'BLOCK_NONE', you are still subject to the rules of the Google pretrained model, to which you don't have access. This model, which we don't have access to, governs the blocked categories in the pretrained model, following Google's rules and norms. I believe you can find these guidelines in the section where you look for your security settings and when you agree to use the API. I hope this helps! Regards! (:

@kpripper Could you share more information please? What was your input + API call?

Chamidu@Chamidu-PC:~/Documents/UI 2 WEB$ node app.js
Server is running on http://localhost:3000
Error processing image: GoogleGenerativeAIResponseError: [GoogleGenerativeAI Error]: Candidate was blocked due to RECITATION
at response.text (/home/Chamidu/node_modules/@google/generative-ai/dist/index.js:256:23)
at /home/Chamidu/Documents/UI 2 WEB/app.js:75:27
at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
response: {
candidates: [ [Object] ],
promptFeedback: { safetyRatings: [Array] },
text: [Function (anonymous)]
}
}
### anyone Can help fix that???

I have similar issue..
GoogleGenerativeAIResponseError: [GoogleGenerativeAI Error]: Candidate was blocked due to RECITATION

I am sending the prompt: "who are you"

my prompt is just what is 1 + 1, got the same error

I got the same error with Gemini 1.5 Pro V1beta chat api.
GoogleGenerativeAIResponseError: [GoogleGenerativeAI Error]: Text not available. Response was blocked due to OTHER
my safety settings are const safetySettings = [   {      category: HarmCategory.HARM_CATEGORY_HARASSMENT, threshold: HarmBlockThreshold.BLOCK_NONE,   },     {     category: HarmCategory.HARM_CATEGORY_HATE_SPEECH, threshold: HarmBlockThreshold.BLOCK_NONE,   },     {     category: HarmCategory.HARM_CATEGORY_SEXUALLY_EXPLICIT, threshold: HarmBlockThreshold.BLOCK_NONE,   },     {     category: HarmCategory.HARM_CATEGORY_DANGEROUS_CONTENT, threshold: HarmBlockThreshold.BLOCK_NONE,   }, ];
my prompt is (in Turkish):
translate following to English: Opere meme Ca tanısıyla izlenen hastada sağ meme mastektomizedir. Mastektomi lojunda belirgin kitle lezyonu saptanmadı.
I'm getting same error both in Google Aı Studio, and in my nodejs program using api

i got the same error

My language is portuguese, i send a prompt to gemini that says: "Gostaria de um resumo sobre a série pousando no amor", returns this error,
Error: [GoogleGenerativeAI Error]: Candidate was blocked due to SAFETY at Object.response.functionCalls. But, if i send a prompt message saying: "Gostaria de um resumo sobre a série Crashing landing on you", returns the resume successfully, i dont understand...

It would be good to know why this occurs. The same prompt works sometimes and sometimes not..

Turns out you can circumvent this by giving better prompts. I had the issue for several days without any solution and bypassed it by giving a more detailed and 'less harmful' prompt but with the same content. The issue just vanished. :D

I was considering gemini but this is making it almost unusable. Please do not block overly aggressively. The most simple prompts are blocked while it works for every other provider.


make selection of auto attenuation control enable or disable of attenuation option

This refers to the new mission editor and may not be feasible or necessary.

was able to accomplish this. However, when a new mission is loaded (opened) the columns revert to the default state for that column.  Need to add a loop that checks each individual cell for each scan and changes the state.


App will crash when exit

Cause we destroy m_pMiniThingCore firstly, and some data structure at the same time;
when we destroy m_pMiniThingQtWorkThread after that, we access those data structure again, this lead nullptr;
Refine at e418dfaea043c8aa50.

Refine at https://github.com/AlanoSong/MiniThing/commit/e418dfaea043c8aa5057e0872d66946ab241fab5.


Import alias doesn't work with React app

Expected Behavior
TLM options --includePaths and --aliases should worked correctly when TLM is used in a typescript React app.
Current Behavior
If you clone this repo and run the "basic example" command, then it works fine. But if you create a React typescript app and run the same command (after copying the example folder of course), then you get the following error:
Cannot find module 'alias' from 'C:\Projekty\test\react-app-tlm\examples\basic\feature-b' (examples/basic/feature-b/style.less[1:0])
'variables' wasn't found. Tried - examples\basic\feature-a\variables.less,npm://variables,npm://variables.less,variables.less (examples/basic/feature-a/style.less[1:0])

Possible Solution
Maybe this has nothing to do with React and it only works when you clone this repo (and it runs in "developer mode"), and doesn't work when you install it with yarn or npm.
Steps to Reproduce (for bugs)

Create a new typescript React app with yarn create react-app my-app --template typescript
Install LESS and TLM with yarn add -D less typed-less-modules
Copy the "examples" folder from this repo to the new project
Run the example command: yarn tlm "examples/basic/**/*.less" --includePaths examples/basic/core --aliases.~alias examples/basic/core/variables

Context
I'm trying to generate typed style annotations for a typescript React app.
Your Environment
Host OS: Windows 10
IDE Used: VS Code
Commands ran from: Windows Power Shell

Version used: Repo cloned from current master branch, module installed<EMAIL_ADDRESS>Operating System and versions: Windows 10
Link to your project: (no link, follow the 4 steps to reproduce to get the same repo as I have)

Close due to repo archived, see details in https://github.com/qiniu/typed-less-modules/issues/30#issuecomment-2298113479


Error in training with ddp backend and checkpoint_callback

🐛 Bug

I am getting the following error when I am using checkpoint_callback or early_stop_callback  in ddp mode with multi-GPUs. However, If I remove checkpoint_callback and early_stop_callback, then my code executes without any error.
Code sample
Here is a sample google colab notebook (Please note that this colab error is not exactly the same actual error): Colab
Traceback (most recent call last):
  File "../model.py", line 489, in <module>
    main(args=args)
  File "../model.py", line 357, in main
    trainer.fit(model)
  File "/home/machine/.local/lib/python3.7/site-packages/pytorch_lightning/trainer/trainer.py", line 738, in fit
    mp.spawn(self.ddp_train, nprocs=self.num_processes, args=(model,))
  File "/home/machine/.local/lib/python3.7/site-packages/torch/multiprocessing/spawn.py", line 200, in spawn
    return start_processes(fn, args, nprocs, join, daemon, start_method='spawn')
  File "/home/machine/.local/lib/python3.7/site-packages/torch/multiprocessing/spawn.py", line 149, in start_processes
    process.start()
  File "/usr/lib/python3.7/multiprocessing/process.py", line 112, in start
    self._popen = self._Popen(self)
  File "/usr/lib/python3.7/multiprocessing/context.py", line 284, in _Popen
    return Popen(process_obj)
  File "/usr/lib/python3.7/multiprocessing/popen_spawn_posix.py", line 32, in __init__
    super().__init__(process_obj)
  File "/usr/lib/python3.7/multiprocessing/popen_fork.py", line 20, in __init__
    self._launch(process_obj)
  File "/usr/lib/python3.7/multiprocessing/popen_spawn_posix.py", line 47, in _launch
    reduction.dump(process_obj, fp)
  File "/usr/lib/python3.7/multiprocessing/reduction.py", line 60, in dump
    ForkingPickler(file, protocol).dump(obj)
_pickle.PicklingError: Can't pickle <class 'torch._C._VariableFunctions'>: it's not the same object as torch._C._VariableFunctions

Environment

PyTorch Version : 1.5
PyTorch Lightning Version : 0.7.3rc3
OS (e.g., Linux): Ubuntu 18.04
How you installed PyTorch (conda, pip, source): pip
Python version: 3.7.5
CUDA/cuDNN version: 10.2
GPU models and configuration: RTX 6000

I have just update from

PyTorch 1.4 & Pytorch Lightning 0.7.3

where "ddp" & ModelCheckpoint worked just fine to

PyTorch 1.5 & Pytorch Lightning 0.7.4

where I hit the same error.

I have updated to PyTorch 1.5 & Pytorch Lightning 0.7.5 and "ddp" in combination with ModelCheckpoint seems to work. Possibly a duplicate of #1628.

resolved in 0.7.5

Thanks @williamFalcon for resolving the issue. But, still I am getting the same error in 0.7.5, you can this colab notebook to reproduce. Please let me know, whether I am missing anything. Thanks.

@mmiakashs mind open a new issue?


Set max line length back to "79" to adhere to the PEP 8 code styling

Reasoning and conclusion can be found here:
https://github.com/thenewboston-developers/thenewboston-python-client/pull/40#discussion_r525297531

This should be done after #48 is merged

I'll do this one

@velibor7  Someone (@sdhiaeddine) is already working on this. Maybe you can give him assistance if he requeres it. The PR is open for a couple of days.
PR: https://github.com/thenewboston-developers/thenewboston-python-client/pull/43

Ohh I was in a rush to get issues done, I'm here if you need help.
Also if you are you using visual studio code add vertical line helper:
"editor.rulers": [79],
"workbench.colorCustomizations": {
"editorRuler.foreground": "#ff5555d8"
},

Hey @velibor7 , yes please. I still have problem fixing some "E999 SyntaxError: invalid syntax"
Please check the commit dfe0f05 #43

Do you want to do a call on discord or skype so that we don't clutter this issue?

@sdhiaeddine Can you send me a message on slack so that we don't clutter this issue? I tried, but I can't find you there, just search me by my name :)

514983fc77ebf39bedb82de179d2312497757e1a0e4276835844479dbfa80e5f


Win10 BSOD when connecting to mouse on Win10 host running https://github.com/dorssel/usbipd-win

I am trying a mix of your https://github.com/vadimgrn/usbip-win2/releases/download/v.<IP_ADDRESS>/usbip-win2-<IP_ADDRESS>-Debug.exe usbip client talking to https://github.com/dorssel/usbipd-win's usbipd host.
On the host I bind the mouse and start the daemon:
C:\Program Files\usbipd-win>usbipd list
Connected:
BUSID  VID:PID    DEVICE                                                        STATE
8-2    1b1c:1b8b  USB Input Device                                              Not shared

C:\Program Files\usbipd-win>usbipd bind -b 8-2

C:\Program Files\usbipd-win>usbipd server
....

On the client I list and attach:
C:\Program Files\usbip-win2>usbip.exe list -r <IP_ADDRESS>
Exportable USB devices
======================
 - <IP_ADDRESS>
        8-2: Corsair : unknown product (1b1c:1b8b)
           : USB\VID_1B1C&PID_1B8B\1602D03FAF7A04C55F40C0F0F5001C06
           : (Defined at Interface level) (00/00/00)
           :  0 - Human Interface Device/Boot Interface Subclass/Mouse (03/01/02)
           :  1 - (Defined at Interface level) (00/00/00)

C:\Program Files\usbip-win2>usbip.exe attach -r <IP_ADDRESS> -b 8-2

The connection is established, but soon after the client PC BSODs in usbip2_vhci.sys
Interestingly, after reproing this a few times, my usbipd.exe binary is now a 0 byte file!
I checked Windows Defender and nothing has been reported as being quarantined.
I then tried to uninstall and reinstall usbipd-win2 and now it bluescreens during every uninstall attempt. :/
FWIW, The client PC that  is BSODing also has cezanne's usbip client and host configured, so perhaps it is some conflict between the two apps?

I followed @vadimgrn 's instructions at https://github.com/vadimgrn/usbip-win2/issues/11#issuecomment-1181589639 and here is the log.
BSOD_usbip2_vhci.txt
Hope it is good enough.

Duh, I now also see the comments in the readme.md (RTFM) about how to collect logs.
Let me know if the above logs are OK; if not I will get a better one.

You should make a log as described in README.md

Fixed, works. It could be considered as the issue of usbipd because genuine Linux server behaves differently. The workaround has applied.
usbip-win2-<IP_ADDRESS>.zip

Run as Admininstrator these commands to remove the driver manually
pnputil /remove-device /deviceid USBIP\root /subtree
FOR /F %P IN ('findstr /m "CatalogFile=usbip2_vhci.cat" C:\Windows\INF\oem*.inf') DO pnputil.exe /delete-driver %~nxP /uninstall

After that remove folder "C:\Program Files\usbip-win2".

Thanks @vadimgrn Looking in to this today....

@vadimgrn Confirmed works! Thanks!

Fixed


switcher: correct sufficient stack space check

Having chopped the stack to its post callee-save-register spill address, csp might have fewer than STACK_ENTRY_RESERVED_SPACE bytes of length. Subtraction by that value would then make the count of available bytes, held in t2, negative.  The test, unsigned tp >= t2, thus risks misinterpreting a negative t2.
On the other side of the comparison, we have, in tp, the number of required bytes between 0 and 0x7F8 (the result of an unsigned byte load left shifted by 3).  Given this extremely limited range, we can move the subtraction of STACK_ENTRY_RESERVED_SPACE from t2 to being an addition on tp without risk of carrying into the sign bit.

Given that you've remembered to consider overflow is it worth putting the reasoning why it is safe in a comment?

Now with 100% more commentary in-line, not just in commit metadata.


logo carousel links difficult to use

overview
@tusharshahrs called out during ux office hours that he was having difficulty following through links in the homepage logo carousel, several of which link to case studies and youtube content. it seems like while the carousel is auto scrolling that clicking behavior is not respected.
screen recording
takes three clicks to follow the link through

Perhaps when hovered, the scrolling could stop to make this easier?


feat: merge from dev to main branch

Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

hot fix on main branch to solve critical issues.


Add bootstrap service, and stop allowing unauthenticated access when auth is enabled and no admin user exists

Background
Currently a new influxd with auth-enabled set to true has the surprising behavior of allowing unauthenticated access, presumably in order to allow the first user who finds the new server to be able to bootstrap it.
Unauthenticated installers combined with modern techniques such as use of Let's Encrypt can have surprising side-effects.
See for example this DefCon presentation last year as it relates to dealing with unauthenticated WordPress installations, which follow a similar anti-pattern.
Relevant certificate search for registered certs with a name beginning with influx.*: https://crt.sh/?q=influx.%25
There are likely a lot of publicly exposed servers out there - defaults are important (separate to this PR, it would be good to consider enabling auth by default).
Proposed change
This PR does two things:

Fixes the TODO to stop allowing unauthenticated access when AdminUserExists() returns false.

Provides [bootstrap] stanza to the TOML configuration that allows an operator to specify a username / password for an admin user, along with a list of seed databases to create:

[bootstrap]
  admin_username = "admin"
  admin_password = "s3cr4t"
  seed_databases = ["foo", "bar", "baz"]

No default values are set, so not specifying these will not create a bootstrap user. These must be explicitly set to apply.
Note also that being able to specify bootstrap data via configuration is a pattern that works well with many orchestration layers such as Bosh which we intend to use for integration with Prometheus in our CloudFoundry monitoring stack.
When this PR lands, we'll update our fork of the influxdb-boshrelease to add.
Thank you for your consideration.

@corylanou as your GitHub handle was on the TODO that this addresses.

Gentle ping?

This looks like a useful and very sane pattern to adopt.
@aeijdenberg we'll need you to sign our CLA before we can review or accept the PR.
I think we can target this for 1.6, if okay with @timhallinflux and @rbetts.

Thanks @mark-rushakoff - CLA is now signed.

Hello, can you split the bootstrap portions from the security portions? I'm interested in merging the "Stop allowing unauthenticated access when auth is enabled and no admin user exists" portion, but less interested in merging the bootstrap portion. The kind of bootstrapping that I see in the code seems like it is something that is better served by puppet/chef/ansible than the database itself and I would like to separate them so this can get into 1.6.
Although I do think it would be interesting to allow configuring the schema in the configuration file. I would like to separate it though so the security fix (which I would count as a kind of bug) is separated from the bootstrap (which would act more as a feature).

@jsternberg , yes it should be easy enough to split it out, and I've started to so - however I'm running into problems with the tests as a result. The bootstrap functionality also made it easier.
In any case, I've added what I've had time to do today to a new PR (https://github.com/influxdata/influxdb/pull/9797), and I've renamed this one slightly to disambiguate.

Thanks for splitting it out, but on review, it looks like there's still some discussion about what the final goal of this is and how we can get there without breaking everyone's current bootstrap process for authentication.
It seems to me there are 3 options.

Allow a user to only run CREATE USER when auth is enabled and there is not admin users. The CREATE USER command needs to create an admin user which can then be used for subsequent commands. Add an optional bootstrap section so that this initial command can require authentication if the user supplies it. This is the version that I would not consider a breaking change, even though it will technically break some workflows. The workflows it breaks are completely invalid and I think they're worthy of invalidating.
The above, but require the bootstrap section or generate unique credentials on startup and output them to the log. Jenkins will generate unique credentials for its bootstrap user and output them to the log so there's precedent for this. If the bootstrap section is filled out, we would not generate credentials. This is a breaking change.
Force the user to set bootstrap credentials. This is a breaking change.

The first one is insecure by default and doesn't really solve the issue, but two and three would break a lot of authentication bootstrap methods at the moment and I don't think we can merge that kind of thing this close to when we release 1.6.
CC @timhallinflux


Document "tee" command in perserving CLI output

logpipe will swallow all logs. We should document the use of tee to place logs into a file before logpipe gets it.
Something like
my-program |& tee logs.txt | logpipe

Added a whole shell redirection page.


Not able to install

Version: 1.0.0 (beta5)
Environment: React
Type: Bug

Description
Please try to add as much detail as possible.
Not able to install. See Bug report.
I am on Mac OSX - El Capitan.
React - 0.15.0
Meteor - 1.3

Actual Behavior
npm ERR! Darwin 15.3.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install" "react-look" "--save"
npm ERR! node v4.0.0
npm ERR! npm  v2.14.2
npm ERR! code EPEERINVALID

npm ERR! peerinvalid The package<EMAIL_ADDRESS>does not satisfy its siblings' peerDependencies requirements!
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>|| 15.x.x
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>

Proposal
Any information on how this might be achieved.
Code Example:
// your proposal here

Oh well, seems like those errors are just peer dependency errors right? There should be no problem with this, just ignore it and start coding I guess. npm's peer dependency logic is a bit strange.
If there still is a problem which let's you not use Look, feel free to reopen this issue :)

Thanks for the quick response. Yes, it is strange that it is throwing this error especially as your package doesn't depend explicitly on the earlier version. Will go ahead and code.


Website: turn into proper several page websites

Multisite

turn one-pages into several page-website

Course content

specify the course content based on the lecturers

Venue and costs

add information about travel, venue, and costs

Registration

add information about the registration requirements and  how to apply

@FGaiser I am not sure if I understand what you mean by "expected working hours"? Like an hour donation? I think we can do up to 6 * 45 min blocks per day, which is roughly 22 hours of working.


Choices type of command arg?

Is there a way to force the user to provide an option based on an enumerated list?
Such as,
find-pet --type[=cat|dog]

@jessevdk I would really love this. I feel that this is the only thing missing in this package

I would really like to avoid adding more features at this point. That said, this is something that I like. Note though that you can of course already enforce this by yourself without support in the library, by making the option a callback function and doing the verification in there.

by making the option a callback function and doing the verification in there.

Yes, but I am more hoping for an overall UI which means including showing the choices in the help message which is automatically generated.