Adds a function for declaring a dataset that has access to Laravel

Howdy!
Occasionally, it can be extremely useful to access the Laravel application inside of a dataset. For example, you may want to return every route registered under a certain middleware to ensure that the middleware acts as expected on each route.
Or you may just want easy access to facades or helpers that go through the container, such as File, DB or Storage.
This PR adds a new function, laravelDataset. It accepts a name for the dataset and a Closure, which is passed the resolved Laravel application.
Example
laravelDataset('example', function ($app) {
    // Return and test against all routes defined in the application
    return Route::getRoutes()->getRoutes();
});

it('does something amazing', fn() => ...)->with('example');

This feature relies on there being a CreatesApplication trait, either in the application tests directory or in Orchestra\Testbench\Concerns. If those traits (and the subsequent createApplication method) don't exist, a BadMethodCallException will be thrown.
Kind Regards,
Luke

No plans for this.


Link preview image is missing

I went to post a pymc.io link in discourse and noticed that the preview discourse generated was missing at least one image (possibly two).  I know there are CSS things that you can add to get various social media sites to correctly preview links.  I suspect that this may be related to that (not exactly sure what discourse is trying to pull here):

PR merged and website rebuilt. Can you check it works now? This can't really be tested reliably on the previews

Hm, that doesn't seem to have changed anything.  Checking here, it looks like everything is detected ok and the only "complaint" is that that image aspect ratio is a bit wide.  But I'm not sure what's up with discourse.

I tried flushing and refreshing.  Let me try logging out and back in?

Nope :frowning_face:

That seems to be a problem with the favicon.  We're missing the main preview image.


[Widgets refactor] typing use enums

Nit: use enums when makes sense here

No benefit. I thought it would be stylistically consisted with https://github.com/huggingface/hub-docs/blob/5e3dcd184361d9552ea4c6f920fb49c23a3f690d/js/src/lib/interfaces/InferenceDisplayability.ts#L1-L14


add peer dependency to typescript

The rule .js files require both tslint and typescript.  However, tslint is a peerDependency while typescript is only a devDependency.  I added it as a peer dependency as well (similar to tslint).
My build system depends on the package.json specifying a dependency if it is used, so while for others this will usually not be a problem I kindly ask you to please merge it in.

Fair enough, haven't checked tslints package.json in a while.


Start implementation of prototype

After setting up the views (#9) you can start template by copying HTML from prototype and replacing dynamic content with variables or django forms.

done


Rc1-Rc2

#1227

@JunTaoLuo @DamianEdwards @Rick-Anderson Please review

See new DISQUS comments. Also, can you put the article title in the PR next time?

@danroth27 these changes looks accurate to me and we should update the document soon. If there are further changes we want to make (like re-ordering of sections), we should do that now or in a follow up PR.

@danroth27  need a tree rat

Starting at line 159:

The hosting configuration keys are now consistent with the command line, environment variables, and hosting.json values. The Microsoft.AspNet.Hosting.json configuration file was renamed to hosting.json.

hosting.json is no longer read by default, users need to explicitly add their own configuration sources and pass a configuration to WebHostBuilder. The default server URL and port are localhost:5000.

This seems to be going through changes that were made between RC1 and RC2. I think the first paragraph should be completely removed. The second paragraph should be updated to say:
To specify web host settings (like the server URL and port) through configuration you can create a new configuration instance and pass it to WebHostBuilder using the UseConfiguration method. The default server URL and port are localhost:5000.

Let's move the VS launch settings section to the very end.

:watch:

Here are the top level sections I think we should have:

Overview
Specify .NET Core SDK version
Namespace and package ID changes
Creating your web application host
Hosting configuration
Hosting service changes
Working with IIS
MVC
Configuration
Logging
Identity
Updating Launch Settings in Visual Studio

Other changes:

Rename "Global.json" to "Specify .NET Core SDK version" and move to the top just below the Overview.
Rename "IHostingEnvironment" section to "Hosting configuration"
Rename "Runtime Services Changes" to "Hosting services"
Changes in MVC -> MVC
Move the content of the Entity Framework section to the end of the Overview section and remove the Entity Framework section heading.
Identity API updates -> Identity
Remove the "WebHostBuilder API updates" section heading and merge it's contents with the "Create your web application host" section.

Rework the "Hosting configuration" section as follows:

Adding the following initial sentence: "You enable configuration for your web application host by calling the UseConfiguration method on WebHostBuilder and passing a configuration instance. The following configuration values can be specified:"
List the configuration keys: application, startupAssembly, detailedErrors, server, webroot, captureStartupErrors, server.urls, contentRoot.
Remove the sentence about env vars being prefixed by ASPNETCORE_ and the env var table.
Explain that to configure hosting using env vars you need to add env vars as a configuration source and optionally specify a prefix.
Specify that the new env var for specifying the environment is ASPNETCORE_ENVIRONMENT.

In the "Creating your web application host" section add the following sentence below the code snippet: "The application base path is now called the content root."

:watch:

cust writes
There is a change in how the routes are mapped to the application. Can I get some example of how to use new IRouteBuilder.MapRoute method.

:watch:

A couple more minor things, but once those are fixed :shipit:


Modify httpBatchStore so that writing values maintains some locality

NBS benefits from related chunks being near one another. Initially,
let's use write-order as a proxy for "related".
This patch contains a pretty heinous hack to allow sync to continue
putting chunks into httpBatchStore top-down without breaking
server-side validation. Work to fix this is tracked in #2982
This patch fixes #2968, at least for now

PTAL
Now that I've gone all the way through the patch, I'm not sure that having NomsBlockCache be distinct from NomsBlockStore makes a lot of sense. On one hand, it's a nice place to put the code that creates and deletes the temp directory that the cache data is in, and it means that ExtractChunks is not part of the NomsBlockStore public API. On the other hand...it's a pretty thin wrapper, and the guts of extractChunks really needs to be inside NomsBlockStore so it can get at nbs.tables and the mutex that protects it.
I'm trying a little bit of further cleanup, but I broke something while doing so. I'll continue trying to chase that down, but please go ahead and review

Diff Coverage
File Path

0%
new go/nbs/cache.go

0%
go/nbs/store.go

••••••••   80%
go/nbs/compacting_chunk_source.go

•••••••••  96%
go/datas/http_batch_store.go

•••••••••• 100%
go/nbs/table_set.go

•••••••••• 100%
go/datas/remote_database_client.go

•••••••••• 100%
go/nbs/table_reader.go

•••••••••• 100%
go/nbs/table.go

•••••••••• 100%
go/types/batch_store.go

•••••••••• 100%
go/nbs/mem_table.go

Powered by Codecov. Last update ed1e386...0b3e3a8

Actually, can you look again? I've squashed so that the second commit includes all the changes since yesterday, which includes getting rid of the old batch-put machinery, that flushInternal() hack, and some other things.

Still LG


Docs for flexibleSpace should specify that expandedHeight is also needed in SliverAppBar

flexibleSpace docs do not currently say that you also need to also specify the expandedHeight parameter to be able to work. (the expandedHeight parameter says that, but if you don't look there, you don't know)
Also possibly worth mentioning in the FlexibleSpaceBar class docs as well.

Although I've intended to fixed this as soon as there's a little time for nearly 9 months it's time to admit defeat. Should be an easy fix :-).

@efortuna I would like to work on this issue!

I was able to reproduce this in 1.21.0-1.0.pre.
As we can see on the current dev channel, the following property doc does not specify that the expandedHeight is also needed:
https://github.com/flutter/flutter/blob/f25bd9c55c48c139524139b477d04b13e9f36b2c/packages/flutter/lib/src/material/app_bar.dart#L1123-L1127

 flutter doctor -v
[✓] Flutter (Channel dev, 1.21.0-1.0.pre, on Mac OS X 10.15.5 19F101, locale en-AO)
    • Flutter version 1.21.0-1.0.pre at /Users/pedro/dev/SDKs/flutter_dev
    • Framework revision f25bd9c55c (5 days ago), 2020-07-14 20:26:01 -0400
    • Engine revision 99c2b3a245
    • Dart version 2.9.0 (build 2.9.0-21.0.dev 20bf2fcf56)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/pedro/Library/Android/sdk
    • Platform android-30, build-tools 30.0.0
    • Java binary at: /Users/pedro/Library/Application
      Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/193.6514223/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.5, Build version 11E608c
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = /Applications/Google Chrome.app/Contents/MacOS/google-chrome-unsafe

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] Android Studio (version 4.0)
    • Android Studio at /Users/pedro/Library/Application
      Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/193.6514223/Android Studio.app/Contents
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.47.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.12.1

[✓] Connected device (3 available)
    • macOS (desktop)  • macos      • darwin-x64     • Mac OS X 10.15.5 19F101
    • Web Server (web) • web-server • web-javascript • Flutter Tools
    • Chrome (web)     • chrome     • web-javascript • Google Chrome 83.0.4103.116

• No issues found!


Missing Region For adding existing AKS cluster

What kind of request is this (question/bug/enhancement/feature request):
Enhancement or Bug
Steps to reproduce (least amount of steps as possible):
Adding existing AKS cluster. Southeast Asia is not available for location selector
Result:
Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
Installation option (single install/HA):

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):
Machine type (cloud/VM/metal) and specifications (CPU/memory):
Kubernetes version (use kubectl version):

(paste the output here)

Docker version (use docker version):

(paste the output here)

This was added in https://github.com/rancher/rancher/issues/17054 and will be in the next release


Curl malformed url

Hello,
I'm trying to use php-vcr which will improve a lot the testability of our system, but I have a problem.
I get this error which I cannot understand:
Guzzle\Http\Exception\CurlException: [curl] 3: <url> malformed [url]
I'm testing an interaction with Amazon SQS in a Yii/PHPunit test suite.
I'm starting php-vcr in the bootstrap.php file like so:
\VCR\VCR::configure()->setCassettePath('fixtures');
\VCR\VCR::turnOn();

and then the setUpBeforeClass is already failing with the error I've said in the beginning.
public static function setUpBeforeClass() {
	\VCR\VCR::insertCassette('example');
	self::$helper = new AmazonSQSHelper( 'send-access-stk' );
	parent::setUpBeforeClass();
}

I tried to replace the Amazon call with this standard curl code:
$ch = curl_init("http://www.example.com/");
$fp = fopen("example_homepage.txt", "w");

curl_setopt($ch, CURLOPT_FILE, $fp);
curl_setopt($ch, CURLOPT_HEADER, 0);

curl_exec($ch);
curl_close($ch);
fclose($fp);

and it works fine. Do you have any suggestion on how to understand more about the problem?
Thanks a lot.

Since the latest master branch, php-vcr itself doesn't depend anymore on Guzzle. It might be, that this error is gone now. I know its been a while ;-)
If you find some time, please check if this issue can be closed.

Hi @adri thanks! I've tried again and now I've got this error:
VCR\VCRException: Invalid HTTP status '', expected format like: 'HTTP/1.1 200 OK'.
I bet is some weirdness in the Amazon library. I don't know if it's worth to keep the issue open :)
Sadly at the moment we cannot upgrade the library so I don't want to blame php-vcr for the problem ;)

@adri I managed to update the Amazon library and it looks like it's recording stuff!
The problem now is that it doesn't replay, maybe because the calls are time related? In all the requests there is a header like this: x-amz-date: 20150123T174129Z which obviously changes every time. That means I can't use php-vcr in this case?

Awesome! You can choose what you want the requests to match on, please see "request matching" in the docs. Does that help?

Ohhhh yes. Thanks a lot!


[Bug]: can’t remove triggered status

What happened?
Hi, when my Eufy Base gets triggered, I can change status and for a while it’s ok, but after some minutes it gets back to triggered. I have to reboot HomeBridge in order to fix the problem. Can I do some troubleshooting? Thanks!
Device Type
Station (Supported)
Plugin Version
beta (Best Effort)
HomeBridge Version
v1.4.x (Supported)
NodeJS Version
v16 (Supported)
Operating System
Raspbian 10+ (Supported)
Relevant log output
No response

Did you check this Common Issues pages ?

Yes, didn’t help :(

Able to reproduce please bear with me I have not spare time to works on

Stale and from the history of this issue I believe this is resolved. Please reopen if still relevant.


mapof   add tooManyOverflowBuckets  limits to grow table

Now mapof grows use factor. How about add constraint overflowBucket to limit too long bucket list.

Too long bucket chain is unlikely to happen in real-world applications. Do you have a non-artificial example when this happens for you?


fix Remove docker tag from non docker case test_dynamic_partition_mod_distribution_key

What problem does this PR solve?
Remove docker tag from non docker case test_dynamic_partition_mod_distribution_key
Release note
Remove docker tag from non docker case test_dynamic_partition_mod_distribution_key
Check List (For Author)

Test

[ ] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[x] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[x] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall


Implementation of Frame.Data() is not correct for all pixel formats

Thanks for the great work you have done with this library!
I tried to use the Frame.Data() method to copy the raw image data of a YUV420P stream and the program sometimes crashed with "unexpected fault address" (segmentation violation).
I tracked the problem down to the algorithm of the Frame.Data method:
func (f *Frame) Data() [NumDataPointers][]byte {
	b := [NumDataPointers][]byte{}
	for i := 0; i < int(NumDataPointers); i++ {
		b[i] = bytesFromC(func(size *C.int) *C.uint8_t {
			*size = f.c.linesize[i]
			if f.c.height > 0 {
				*size = *size * f.c.height
			} else if f.c.channels > 0 {
				*size = *size * f.c.channels
			}
			return f.c.data[i]
		})
	}
	return b
}

The YUV420P data consists of three planes. E.g. a 640x480 pixel frame, the planes have the following dimensions:
Y: 640x480
U: 320x240
V: 320x240
However, the implementation always multiplies the linesize of the plane with the image height which results in too much bytes being read from C. In the best case, the read image data contains garbage after the correct values, in the worst case, the program tries to read invalid memory addresses.
The AVPixFmtDescriptor struct describes the correct sizes for each plane. However the implementation is much more complicated...
So I would suggest to implement a simple function to receive the raw data of images as simple byte slice:
func (f *Frame) GetImageBuffer() ([]byte, error) {

	// retrieve the image buffer size
	bufferSize := C.av_image_get_buffer_size(C.enum_AVPixelFormat(f.c.format), f.c.width, f.c.height, 1)
	if bufferSize < 0 {
		return nil, newError(bufferSize)
	}

	// create a buffer and copy the raw image data into it
	buffer := make([]byte, int(bufferSize))
	n := C.av_image_copy_to_buffer((*C.uint8_t)(unsafe.Pointer(&buffer[0])), bufferSize, (**C.uint8_t)(unsafe.Pointer(&f.c.data)), (*C.int)(unsafe.Pointer(&f.c.linesize)), C.enum_AVPixelFormat(f.c.format), f.c.width, f.c.height, 1)
	if n < 0 {
		return nil, newError(n)
	}

	return buffer, nil
}

That method does not output multidimensional slices as Data() does and only works for images. I don't submit a pull request so far - maybe you want the Data() method to be fixed (which I don't have capacity for know). If you feel that my implementation of GetImageBuffer() is useful, I would kindly submit a PR.
Best regards!

Thanks for describing the problem so clearly 👍  To be honest, I've never trusted my implementation of Frame.Data(). However, I think this is best if it is fixed. If you don't feel like it, no problem. However you seem to have investigated it a good bit so maybe you could point me in the right direction : could you elaborate on The AVPixFmtDescriptor struct describes the correct sizes for each plane. However the implementation is much more complicated...? What would need to be done exactly?
As for your needs, I would be open to a PR that adds the 2 following functions:

Frame.ImageBufferSize() int
Frame.CopyImageToBuffer(size int) ([]byte, error)

Additionnaly, do you know a function that would copy a buffer to an image ? 🤔

I think the implementation of av_image_copy_to_buffer is a good reference to fix the Data() method:
It uses av_pix_fmt_desc_get (line 509) to get information about the pixel format, then determines the number of components and extracts the raw data without padding (starting at line 523).
If you mean a function to create an image.Image, I don't know a library which supports various formats. For my use case with YUV420P format I use the following algorithm:
buffer, _ := frame.GetImageBuffer()

w := frame.Width()
h := frame.Height()

img := &image.YCbCr{
	Y:              buffer[:w*h],
	Cb:             buffer[w*h : w*h+(w*h)/4],
	Cr:             buffer[w*h+(w*h)/4:],
	YStride:        frame.Linesize()[0],
	CStride:        frame.Linesize()[1],
	SubsampleRatio: image.YCbCrSubsampleRatio420,
	Rect:           image.Rect(0, 0, w, h),
}

However one would have to adapt it for all the formats one would like to support...

Thanks for taking the time to share that information 👍
Regarding your needs, I've thought about it a little bit more, and I think that would be a better idea to add only one function Frame.DataBytes() ([]byte, error) that would be simplified to something along the lines of:
if f.Height() > 0 && f.Width() > 0 {
  // TODO Get image buffer size
  // TODO Copy image to buffer
} else {
  return nil, errors.New("astiav: not implemented")
}

If you come up with a better solution than f.Height() > 0 && f.Width() > 0 to identify the frame as an image, I'm open to it. An alternative would be to provide the CodecContext or CodecParameter to the function to retrieve the MediaType but I'd rather keep the function signature simple.

Using dimensions to detect video frames seams OK to me. The documentation states that these fields are video only, so this should be fine and a simple solution.

FYI Frame.Data() now returns a *FrameData that allows to either get data as []byte or, if it's an image, get data as an image.Image.
I'll close this issue


Please, for god's sake, add an uninstall button

I've been trying all the ways to uninstall this and it won't work, just add an uninstall button, please.

There are several instruction sets on how to do so as well as a plugin that automatically does it. An uninstall button will not be added. It may be added to the installer but not to the UI inside discord.


chore: bump chess

made the uci::uciToMove more failsafe with very bad input to not have ub/crash.


MetricsContext does not return metric reference for Gauges or PerfCounters

The methods to create a gauge or performance counter metric don't return a reference to the metric that was just created, unlike all the other methods to create a metric.  Is this a bug, or is this intentional?
Also, it doesn't look like there's any way to dispose the performance counter instance that's created by the PerformanceCounterGauge, which doesn't implement IDisposable. Would it make sense to expose the perf counter reference as a property on the gauge? I could then dispose it manually when I'm done using it. This would additionally be useful to me because I want to access some of the performance counter's properties, specifically, its name, category, instance name, help text, and counter type.  Also, shouldn't PerformanceCounterGauge derive from GaugeImplementation?
Another unrelated thing I noticed, it seems like DerivedGauge is essentially identical to ScaledValueProvider with T : double. Is that the only difference, or am I misunderstanding something?
Thanks!

Closing this for now, feel free to re-open if more info is needed.


Preprocess markdown in metadata?

It'd be great if there were an option to preprocess markdown in metadata. At the moment it's a difficult thing to do with metalsmith.

Preprocessing markdown can easily be done with a separate plugin. Depending on what templating language you use, this can also be done with filters. For example for Nunjucks you can use something like this:
in the build file add marked
const marked = require('marked');

then build filter
const mdToHTML = (mdString) => {
  try {
    return marked.parse(mdString);
  } catch (e) {
    console.error('Error parsing markdown:', e);
    return mdString;
  }
}

then define engine options for the in-place and layouts plugins. these options are passed to the Nunjucks templating engine
const templateConfig = {
  engineOptions: {
    path: [`${workingDir}/layouts`],
    filters: {
      mdToHTML,
    },
  },
};

then apply the options to in-place
.use(inplace(templateConfig))

Now you can use this filter in the template like this
{{ info.prose | mdToHTML | safe }}


Install Lean4 on old Mac (OS version < 10.15)

Description
I've installed elan, and lean4-nightly already.
However, when I run the lean by typing  lean on terminal, then I see
following exceptions:
  Referenced from: <my personal folder path>/.elan/toolchains/leanprover--lean4---nightly/bin/../lib/lean/libleanshared.dylib (which was built for Mac OS X 10.15)
  Expected in: /usr/lib/libSystem.B.dylib

dyld: Symbol not found: ____chkstk_darwin
  Referenced from: <my personal folder path>/.elan/toolchains/leanprover--lean4---nightly/bin/../lib/lean/libleanshared.dylib (which was built for Mac OS X 10.15)
  Expected in: /usr/lib/libSystem.B.dylib

Abort trap: 6

Expected behavior:
What I expected  is, of course, lean4 is working well.
Actual behavior:
But as what I've written above, it doesn't work on mac OS version < 10.15
Reproduces how often: [What percentage of the time does it reproduce?]
100 %
Versions
lean itself doesn't work, but I know that its nightly version of lean4.
Additional Information
There is nothing more.
My vs code Lean4 extension doesn't work, too, but i guess that above issue causes such problem.

We do not support end-of-life OS versions.


Neato component updates break the botvac connected compatibility

Home Assistant release with the issue:
0.77.3
Last working Home Assistant release (if known):
~0.64
Operating environment (Hass.io/Docker/Windows/etc.):
virtual_env / raspbian
Component/platform:
neato component with botvac connected series
Description of problem:
The new neato component update breaks the compatibility to the botvac connected series

the new return_to_base service doesn't work. The workaround is that the botvac has to be paused first in order to call the return_to_base service.
if the botvac has been paused, upon resuming with the "start" service, the status changes falsely to "Return to start" (should be Turbo House cleaning or Eco House cleaning). This status will show for a long time, even the botvac has been docked (until the component resets?!). The action in the UI for the vacuum switch is also wrong - "Return to Dock".
if you return it after pausing it, sometimes the status "eco docking" / "turbo docking" or even 'Please empty dust bin' appear instead of "Returning to start". The question is, what should we expect in the state attribute, the alerts or the actions?
In the cases above the wrong states will stay for some time (even other actions are undertaken). In all other cases the states change swift and correspondingly upon the action.

Hey thanks for reporting these issues:

the new return_to_base service doesn't work. The workaround is that the botvac has to be paused first in order to call the return_to_base service.

Ah yes the old service call would first pause then return to base, as we have new services I had split them up accordingly.  I will put in a fix to first pause the botvac if its not paused already so this can work like it did in the past.

if the botvac has been paused, upon resuming with the "start" service, the status changes falsely to "Return to start" (should be Turbo House cleaning or Eco House cleaning). This status will show for a long time, even the botvac has been docked (until the component resets?!). The action in the UI for the vacuum switch is also wrong - "Return to Dock".

So the botvac actually checks in every 5 minutes now to the cloud as we needed to reduce the cloud calls.  From the version that you upgraded from we were hammering the cloud and we needed to cut down on the amount.
https://github.com/home-assistant/home-assistant/commit/6cabbd2592df6df4b5197c19173b07491e9f76bb#diff-72b7776753ac10efe7ae9f30338dd8f7R125
The UI issue would need to be filed over in the polymer repo but it is also more generic to all vacuums.
https://github.com/home-assistant/home-assistant-polymer

if you return it after pausing it, sometimes the status "eco docking" / "turbo docking" or even 'Please empty dust bin' appear instead of "Returning to start". The question is, what should we expect in the state attribute, the alerts or the actions?

The status reflected for the botvac changes according to whatever Neato is reporting.  If you get an alert to empty the dustbin that will override the docking status.
https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/neato.py#L46

In the cases above the wrong states will stay for some time (even other actions are undertaken). In all other cases the states change swift and correspondingly upon the action.

This is because we needed to cut down on the cloud calls as botvacs were getting disconnected due to the amount of calls.

@dshokouhi - Thanks for your response & time


Cannot download pre-trained models

I can't download any pre-trained model file from BaiDuNetdisk. Can you please make those available to download in Google Drive, just like results folder?

Yes, I also tried to download it. Writes that there is no permission to download.

Please download the pre-trained model from: https://drive.google.com/drive/folders/1OkdakKKIMRGnKH8mxuFi_qI9sa903CD2?usp=share_link.


Rules for Text Resizing

In order to keep design from breaking and maintain hierarchy the following rules should be applied:

H2: max-size = 22px (up from 18px)
H3: max-size = 20px (up from 16px)
H4: max-size = 18px (up from 16px)
Para: stays at 16px (no resizing)
Small text: max-size = 16px (up from 14px)

Line-heights should stay proportional. Is this currently measured in ems?

@daralthus ^ for your reference

We can do something like this in ./components/Text.js to reverse the effect of accessibilty font scaling after a certain ratio
h2: {
    fontFamily: "Poppins-SemiBold",
    fontSize:
      PixelRatio.getFontScale() < 1.222 ? 18 : 22 / PixelRatio.getFontScale(),
    lineHeight:
      PixelRatio.getFontScale() < 1.222 ? 24 : 26 / PixelRatio.getFontScale(),
    includeFontPadding: false
  },


Infra: docker-compose repository

Summary
Hotfix for local docker-compose setup.
Details and comments
Gateway and repository has the same application name and same table programs. Therefore migrations in both applications were creating the same table in the same database. Migration passed as it was thinking that those table existed already.
This PR is a hotfix to make docker-compose with  full, default and jupyter profile work.
Real fix will be implemented within https://github.com/Qiskit-Extensions/quantum-serverless/issues/555
Closes #552

I'm still getting the error

@psschwei with profile full? or without profile?

full

ok, looks like I may have been caching things... deleting and recreating got it working

Tomorrow we will create proper fix :)


Accept kwargs in buy and sell

Previously buy and sell accepted a single dict of arbitrary parameters.
That worked, but it wasn't very pythonic.
This commit changes them both to accept arbitrary kwargs. The
functionality is the same, but now it looks like python.
I have also taken the opportunity to format the lines I altered in
accordance with PEP-8.
Cf. #8

Great start! As you mentioned, two things: 1) Create a place_order method (public?) to be inherited by buy/sell. 2) Require product_id in all related methods.

Why should they go in the same PR? This PR is just about kwargs.

Lastly, please make these changes in the master branch as the development branch is very stale. I know this isn't good practice, but we can clean it up if multiple people from the community come together on this.

No. I'm not going to fix your entire development branch for you. I'm grateful you wrote this project, and thought I'd contribute a few small improvements. I don't see what the problem is about maintaining your development branch, or why you'd keep it around and talk about it in your documentation if you don't want people to use it.

Closing this PR in favor of #42.

Hi Bryan I understand yourconfusion. I wasn't asking you to cleanse it up, but rather catching you up to speed on these branches. I do not expect you or anyone else to help update the branches. Thanks for your help and explanation on your changes. They are invaluable additions to this project.


Update domains.json

Hi! What is your PR about?

[ ] Add or edit a skin
[x] Add a custom domain for your space
[ ] Add an alias to migrate your space

Added code is between domains. It should work.


Wrong Chartboost banner orientation with AIR SDK <IP_ADDRESS>3

Hi,
In my application I use landscape orientation on iOS.
When I build application with AIR SDK <IP_ADDRESS>3 chartboost banner pops-up in portrait mode while it must be in landscape.
When I build application with AIR SDK <IP_ADDRESS> everything works as expected. But AIR SDK <IP_ADDRESS> is an old build, but AIR SDK <IP_ADDRESS>3 must be supported by ANE.

We have the same problem. It seems this happens only on devices running iOS 8.

@TurboMask there is workaround: Compile your project with AIR SDK <IP_ADDRESS>.
Anyways it must be fixed to work properly with newer AIR SDK.

@minimedj Is your application in the Chartboost dashboard configured for landscape only?


Shared caching for identical function types in relationship checking

Improve caching of relation checks for simple function types.

@typescript-bot perf test this

No positive effects on performance from this experiment. Closing the PR.


bug: Navigation to tabbed view with state.cache = false breaks history

Type: bug
Platform: all
The same view appears twice in history and breaks back button behavior on android when:
Go to a view with tabs whose parent state has "cache: false" and "abstract: true";
Go back;
Go to the same view with tabs again;
Try to go back.
You can reproduce here:
http://codepen.io/anon/pen/JdVKJZ?editors=101
The problems seems to be the detection of 'dupNav' now that the view.historyId is new each time the view is rendered.

Greetings @israelrios!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

Since the view is abstract, you're not really navigating to that state, but it's child, so this a gray area. You really should't need to use cache: false here.

Without the cache = false I had problems with stale content in the tabs.

Then you would want to move that to the nested states.

There are other problems with tabs, even when cache=false. Check:

Do you have an example of that?

http://codepen.io/anon/pen/pJBdMW?editors=101

I'm not sure what I should be seeing here?
It looks to be working fine.

The second time we go to the "About" tab the title should be "About 2" but it is "About 1".

Hi @israelrios 😄 ,
Please use the life cycle events to access state parameter data when using cached views.  Controllers are only created once.
$scope.$on("$ionicView.enter", function(event, data){
   $scope.id = data.stateParams.id;
});

I don't think there is a legit issue here.  Everything seems to be working fine for me.  Please let me know if I am mistaken.
Thanks,
Dan


How can I delete an accidentally added team?

I put in the the whole link for my team instead of just the idea and now I have two teams that look exactly the same - but one wokrs nad hte other doesn't.
Maybe the extension could check the validity of the input first?

@ogierm I had to update the code manually in my vscode node_modules to allow me to do this as I made this mistake too.
I added the following to the extensions files Mine where found in
HomeDirectory.vscode\extensions\idered.figma-0.1.1\
package.json
{
	"command": "figma.removeTeams",
	"title": "Remove All Teams",
	"category": "Figma"
}

extension.js
    // FIGMA: Remove all Teams
    context.subscriptions.push(vscode.commands.registerCommand("figma.removeTeams", () => __awaiter(this, void 0, void 0, function* () {
        console.log(store.teams);
            store.removeTeams();
            tree.provider.refresh();
            console.log(store.teams);
        })));

and finally
store.js
    removeTeams() {
        self.teams = [];
    },

After you have updated the files you just need to disable and re-enable the extension and you should have a new command that your can run which will remove all the teams.
Cmd-Shift-P / Ctrl-Shift-P  Figma: Remove all Teams

Hey, sorry for late response. I don't have any spare time lately to continue work on this. Repo is open for PR's


[MsTeams]Input.Number is not allowing more than 10 numbers

Copied bug from: https://github.com/microsoft/AdaptiveCards/issues/3957
Details from original post:
Platform

.NetCore
BotFramework V4

Host

MS Teams

Hi All,
I am using Input.Number property to collect the phone number from the user. If the user enters the wrong number then I am pre-populating the same Adaptive card filling the number entered by the user.
If the value of number is more than 10 digits, input.number property is not allowing that number.
Please have a look at the Json:-
 {"type":"AdaptiveCard","version":"1.0","body":[{"type":"TextBlock","text":"My mobile number"},{"type":"Input.Number","placeholder":"","value":"**962213213213213**","id":"asdas"}],"$schema":"http://adaptivecards.io/schemas/adaptive-card.json"}
This is the error message from Teams App studio:-

Please help.
Thanks.

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

ID: 3fb4394d-25b1-8752-3996-fe3a2934732d
Version Independent ID: 6d391381-89d0-117c-27cc-00043bd26a36
Content: Cards reference - Teams
Content Source: msteams-platform/task-modules-and-cards/cards/cards-reference.md
Product: msteams
GitHub Login: @surbhigupta12
Microsoft Alias: surbhigupta12

@Chekrii - We are looking into this I will get back to you soon.

We are closing this issue for now, Please feel free to reopen it, if required.


Execution of export-config fails with error code 10000

Describe the bug
Execution of export-config fails with error code 10000
To Reproduce
Steps to reproduce the behavior:

Run the command:
cf-mgmt export-config --config-dir=./config/myenv --system-domain=my-env.x.x.com --user-id=cf-mgmt-user --client-secret=xxxxx

Expected behavior
... Collects all the data but ends with
2020/02/06 16:50:00 E0206 16:50:00.559481 452 export_config.go:49] Export failed with error:  {
"description": "Unknown request",
"error_code": "CF-NotFound",
"code": 10000
}
error: {
"description": "Unknown request",
"error_code": "CF-NotFound",
"code": 10000
}
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: Linux
Browser: NA
Version
cf-mgmt version
Version: [1.0.41], Commit: [b06f99729dd28b0da3b4d82a9482799d37c437ef]
cf-mgmt-config version
Version: [1.0.41], Commit: [b06f99729dd28b0da3b4d82a9482799d37c437ef]

Additional context
This completes without any issues in PCF environments but seems to never complete successfully in OSCF environments. It seems to be something at the very end that fails since all the Quotas are properly collected before it fails.

Looking through the code it appears the failures are probably on "DefaultProtectedOrgs"
Not sure what or how this is determined. Looked through the v2 and v3 apis, I cannot find anything different.

@findneville
Can you turn on debug and provide logs...
LOG_LEVEL=debug cf-mgmt export-config --config-dir=./config/myenv --system-domain=my-env.x.x.com --user-id=cf-mgmt-user --client-secret=xxxxx

@calebwashburn
The output I have pasted and screenshot are with the LOG_LEVEL set. The below is the output and there is nothing else. Logs before it fails are below. Let me know if we need more info
2020/02/07 09:16:23 I0207 09:16:23.163968 16665 yaml_config.go:433] Writing out orgQuota 1000G
2020/02/07 09:16:23 E0207 09:16:23.188502 16665 export_config.go:49] Export failed with error:  {
"description": "Unknown request",
"error_code": "CF-NotFound",
"code": 10000
}
error: {
"description": "Unknown request",
"error_code": "CF-NotFound",
"code": 10000
}

The final 2 steps in the export are to get shared domains and routing groups.  No logging exists to indicate which one we are getting so will add that shortly.  But these are making calls to the following....
cf curl /v2/shared_domains

cf curl /routing/v1/router_groups

@findneville  Added more logging to help debug this - https://github.com/pivotalservices/cf-mgmt/releases/tag/v1.0.42

Thx @calebwashburn .. Trying now

So you are right. Its the router-groups where things are failing. We do not have TCP routing enabled.
2020/02/07 13:16:43 I0207 13:16:43.30692 20225 exportconfig.go:295] Listing Router Groups
2020/02/07 13:16:43 E0207 13:16:43.320324 20225 export_config.go:49] Export failed with error:  Getting routing groups: {
"description": "Unknown request",
"error_code": "CF-NotFound",
"code": 10000
}
error: Getting routing groups: {
"description": "Unknown request",
"error_code": "CF-NotFound",
"code": 10000
}
==
Running cf router-groups gives me errors too
cf router-groups
Getting router groups as xxx ...
FAILED
Failed fetching router groups.
Error performing request: Get /v1/router_groups: unsupported protocol scheme ""

Added new flag to allow skipping router groups in https://github.com/pivotalservices/cf-mgmt/releases/tag/v1.0.43

Works.. Thx


1.7.x players kicked upon sleeping in a bed

When a player who is on a 1.7 protocol client tries to sleep in a bed, they are kicked instantly with this message in their client http://prntscr.com/mmp2xr
There is no proper error in console when the player is kicked it only displays a disconnect message.

You need to provide information about your setup so that we can actually attempt to replicate this and look into it.


Download assets skipped

please fix my download assets are being skipped while downloading

I'm experiencing the same issue with all txt assets.
Course: https://www.udemy.com/course/di-ioc-dotnet/
Logs: https://haste.laiteux.dev/zkkspfpr

so how can we get help from udemy or software ? any fixes ?
On Thu, Jul 16, 2020 at 5:18 AM Matt<EMAIL_ADDRESS>wrote:

I'm experiencing the same issue with all txt assets.
Course: https://www.udemy.com/course/di-ioc-dotnet/
Logs: https://haste.laiteux.dev/zkkspfpr
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/r0oth3x49/udemy-dl/issues/526#issuecomment-659072714,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANPE5HRPDOMETC7AVLRQ3ZDR3Y55XANCNFSM4O2MUHHA
.

duplicate of #524


Add missing input bindings to DX11.

LGTM, but I want to run some tests and try to understand a few things before merging it.
It's not clear to me why the imgui-rs Key enum doesn't have all the alphanumeric characters and that's part of the reason I have never implemented this in the first place, going for an external input control system in my implementors (i.e. via GetAsyncKeyState in ER and DS3 tools).
Furthermore, we're still on imgui-rs 0.8.0 by choice as the newer versions have broken docs and also, iirc, have some breaking changes despite only being patch version bumps.
I'd like to make sure that that's heading in the right direction, or alternatively sidestep that and rely on imgui-sys directly for this specific part.

I was wondering that issue myself too. I saw this issue https://github.com/imgui-rs/imgui-rs/issues/455 which it seems to explain that those specified keys are for imgui-rs to handle directly.
For what its worth it, I checked some imgui backends (like the SDL and the WINIT and they only init those same keys in the same way. Without this patch, input fields won't capture those special keys (for example backspace, arrow keys, etc). The other inputs work fine.
It sounds more like a limitation if imgui-rs itself rather than imgui

Oh, yeah, indeed it is probably something specific to imgui-rs. That's why I suggested we find out a way to initialize all the keys in the appropriate way by peeling off a layer and using imgui-sys directly -- even simply checking out how the underlying index values are laid out, building our own #[repr(usize)] enum and using that for the indices.
I feel like I don't know that part of imgui well enough yet, but I don't think it'll be super hard to figure out.

Turns out I'd already done the exact same thing for DX12 here 😅 I need to do some refactoring.

Yeah, I noticed that as well but forgot to point it out haha. I can see a bit of code duplication that maybe could be reduced but I'm not sure if it would be prettier.

I think the style of the two hooks could be at least made more similar, but there are some very fundamental differences that completely prevent factoring common code together in a meaningful way.


Admob with real bannerId

Can't able to see real ads But test is working.
<AdMobBanner
adSize="smartBannerPortrait"
adUnitID={bannerid}
onAdFailedToLoad={error =>console.log(error)}
/>

Got same issue here, even example from master has error

same issue. Get error says no ad to show. Test ad Unit ID works, so I wonder if they don't allow real ads for dev?

In Admob doc it says

Before loading ads, apps should initialize the Mobile Ads SDK by calling the configureWithApplicationID: class method in GADMobileAds, and passing it their AdMob App ID. This only needs to be done once, ideally at app launch. You can find the App ID for your app in the AdMob UI.

There is no option to set this in the Component. In native, to set configureWithApplicationID
import GoogleMobileAds
…

@UIApplicationMain
class AppDelegate: UIResponder, UIApplicationDelegate {

  var window: UIWindow?

  func application(_ application: UIApplication,
      didFinishLaunchingWithOptions launchOptions: [UIApplicationLaunchOptionsKey: Any]?) -> Bool {
    // Initialize the Google Mobile Ads SDK.
    // Sample AdMob app ID: ca-app-pub-3940256099942544~1458002511
    GADMobileAds.configure(withApplicationID: "YOUR_ADMOB_APP_ID")

    return true
  }

}

This is different from  adUnitID which we initialize in the App.jsLike this
 <AdMobBanner
          adSize="fullBanner"
          adUnitID="ca-app-pub-3940256099942544/6300978111"
          testDevices={[AdMobBanner.simulatorId]}
 />

Did anyone found any workaround for this?

There is no workaround Needed.  It appears setting the application id is optional.
In dev if you want to see "real" ads then do not set device test Id anywhere and use your real ad unit Ids.  The ads will be pulled from the real admob network with a "Test Ad" label shown at the top center.
In prod, the "Test ad" label will be removed automatically.


altmetrics badge

Hi there,
I'm trying to modify the theme code to display the Altmetrics badge on each paper to get something like this. So far I was able to do it (sort of) in the publication pages by modifying the academic-admin code so it adds the html code at the end of each index.md if there is a DOI available.
I'm pretty sure it's not the best way so I'm wondering which files dictate the way the citations are styled in the Publication page and which one sets the outline for each single publication page (where the abstract appears).
Cheers,
Xabi

Please use the forums or chat for questions/troubleshooting. Also refer to the Academic and Hugo Documentation.


Nemesyst Version 2 Code Rework

This will add:

[x] use of new mongo MongoDB handler library
[x] be python 2-3 compatible
[ ] use password hashing
[x] read the docs integration

This will change:

[x] lines of code to be fewer
[x] code to be cleaner and pep8 compliant
[x] number of args will be fewer and more consistent
[ ] creates more dependencies for cryptography

Reference for password cryptography: https://passlib.readthedocs.io/en/stable/index.html

Looks good. There is still functionality to carry over to version 2 and we still need templates and examples for various neural networks but its all there, all the core functionality for version 2 has been implemented, in its raw form.


Reliance AGM stream videos not working, maybe HLS at fault ??

Please follow the guide below

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

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

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

Before submitting an issue make sure you have:

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

What is the purpose of your issue?

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

I first tried to see what options are available to me.  I remember HLS having issues earlier. Maybe something similar here at work :(
It basically throttles the download to less than 20 kbps, have no idea why :(

This was a live stream at the time of writing.


Prototypical diagnostic facility

Submisison Checklist

[x] Run tests: ./runCmdStanTests.py src/test
[x] Declare copyright holder and open-source license: see below

Summary:
Introduces a prototypical diagnostic facility.  The exact text and thresholds can be modified per user feedback.  Addresses #540.
Intended Effect:
Provides a compliment to bin/stansummary that analyzes the output instead of simply displaying it.
How to Verify:
./runCmdStanTests.py src/test/interface/diagnose_test.cpp
Side Effects:
None.
Documentation:
None added -- we should iterate a few times before exposing this to users via the manual.
Reviewer Suggestions:
@syclik
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): Columbia University
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

Changes pushed.  Right now the only thing that could be pushed to Stan is the E-BFMI calculation, but even that would be awkward given that the chains object doesn't discriminate between its variables.  At the moment I think it's best to keep it in the interfaces.
max_limit should be the max_treedepth argument for the input chains, but that's not saved in the stan_csv_reader so it cannot be accessed programmatically.  I went with 10 as it's the CmdStan default.

Cool!  Just waiting on @syclik's approval for the changes.


New Feature :User Profile Page

I can utilize the getUserDetails function to write a DB query which will fetch user details based on username as it is unique. As per schema defined in the project.
Result of the query will be the document , I will parse necessary details and return the results from controller
Child of #17
@YashKumarVerma

All functions from model would return only JSON data that controllers would process. Please ensure that.

All functions from model would return only JSON data that controllers would process. Please ensure that.

I make sure to keep that in mind.

as it seems "create user" functionality is not present and the , and the schema is not importing to local (no db named user). What is expected ? Because my task is dependent on this.

as it seems "create user" functionality is not present and the , and the schema is not importing to local (no db named user). What is expected ? Because my task is dependent on this.

There is a schema file in the users module (https://github.com/LockedUp-Coders/nostalgify/blob/master/src/modules/user/schema.ts). All these fields need to be shown in the profile page.

updates ?

Have you finished it @ausome197  ??

@YashKumarVerma please reasign this

@YashKumarVerma please reasign this
Thanks for your concern.
I'am still working on this issue , I'am facing problems in this task which are taking longer than usual to resolve , since its my first time working on backend.


unrecognized options or missing parameter tls-crypt {2.3.4}

Hello
I am having problem connecting to pivpn on my raspberry pi After run sudo pivpn -d, I got this:
=============================================
::::		Self check		 ::::
:: [OK] IP forwarding is enabled
:: [OK] Iptables MASQUERADE rule set
:: [OK] OpenVPN is running
:: [OK] OpenVPN is enabled (it will automatically start on reboot)
:: [ERR] OpenVPN is not listening, try to restart now? [Y/n] y
Done
[INFO] Run pivpn -d again to see if we detect issues
=============================================
::::      Snippet of the server log      ::::
Jun  7 14:20:31 raspberrypi ovpn-server[2072]: Options error: Unrecognized option or missing parameter(s) in /etc/openvpn/server.conf:23: tls-crypt (2.3.4)
Jun  7 14:20:31 raspberrypi ovpn-server[2072]: Use --help for more information.
Jun  7 14:21:32 raspberrypi ovpn-server[338]: Options error: Unrecognized option or missing parameter(s) in /etc/openvpn/server.conf:23: tls-crypt (2.3.4)
Jun  7 14:21:32 raspberrypi ovpn-server[338]: Use --help for more information.
Jun  7 14:36:25 raspberrypi ovpn-server[1164]: Options error: Unrecognized option or missing parameter(s) in /etc/openvpn/server.conf:23: tls-crypt (2.3.4)
Jun  7 14:36:25 raspberrypi ovpn-server[1164]: Use --help for more information.
=============================================
::::		Debug complete		 ::::

I tried to search the internet but no luck.
Did somebody know how to solve this ?

as per https://github.com/pivpn/pivpn/issues/759
Closing thread.
please fill the issue template properly and open a new issue or reopen this issue if needed.


authentication support

to secure the server tunnel we should be able to set credentials
wstunnel --server --credentials mycredentials.htpasswd  wss://<IP_ADDRESS>:443
then the client wstunnel wss://myuser:mypass@server:443

Hello,
Supporting all the possible hash for htpasswd will be a lot of work and possibly contains a lot of issues.

Hello,
Sorry for the late response, I haven't seen your issue in my mailbox.
Regarding your feature request, I will not implement it as I will require a lot of work in order to support all the possible hash function forhtpasswd, but also the feature you request is feasible if you setup a reverse-proxy (i.e: nginx) in front of wstunnel in order to do that.
Ok this require a bit more of work/setup than just spawning wstunnel, but the project aims to stay as simple as possible while remaining useful to bypass firewalls.


Add sortShapeProp option to sort-prop-types rule

This allows the rule to enforce the sorting rules in a shape prop object.
I think it makes more sense to happen by default, but I expect it would be better to wait until the next major version for that. This PR should allow the rule to satisfy the request in #1476.

Please rebase this one more time; tests should pass now :-)


Update jdk jigsaw version to 111 and fix some examples

Thanks a lot for this repository, it helped me get started with jigsaw really quickly. :)
I've encountered some issues related to older version of jdk jigsaw (b80). I've changed it to newest version (111) and fixed some examples. I've commented my changes in the commits.
I have tested changes on mac osx (el capitan) and on ubuntu docker image.
One example that still does not work is 08_XModules_Overries:
src/java.base/java/util/concurrent/ConcurrentHashMap.java:3: error: ConcurrentHashMap is not abstract and does not override abstract method entrySet() in Map
public class ConcurrentHashMap implements java.util.Map {

I can fix it later in another pull request and maybe add some more examples.
Thanks :)

Thanks for all the fixes @kamkor, this repo is many months old and with the moving changes in Jigsaw we are expecting parts of the instructions to change or not work. Thanks for your contributions.


PauseChildThreads() sometimes hangs

I think this is related to #53 (cc/ @jamespic) . I am consistently seeing Pyflame hang when profiling threads on my Chromebook. This is running Debian Jessie in a chroot, and the host kernel is 3.14.0. IIRC I did test this branch on a regular Jessie host, so I think it might be related to the ChromeOS kernel? I'll have to check.
What I am seeing is that it goes to pause both threads in a Python process running test_threaded_sleeper.py, and gets stuck on the second thread. By "stuck" I mean the call to wait() that is made after the PTRACE_ATTACH never completes. The full stack is like:
(gdb) bt
#0  0x00007f0358495ab2 in __libc_wait (stat_loc=0x0) at ../sysdeps/unix/sysv/linux/wait.c:30
#1  0x00000000004065eb in pyflame::PtraceAttach (pid=pid@entry=26554) at ptrace.cc:42
#2  0x00000000004079a1 in PauseChildThreads (pid=26553) at ptrace.cc:209
#3  pyflame::PtraceCallFunction (pid=26553, addr=4794543) at ptrace.cc:222
#4  0x000000000040a5f1 in pyflame::PyFrob::DetectPython (this=this@entry=0x7fffd588a130) at pyfrob.cc:135
#5  0x0000000000403845 in main (argc=<optimized out>, argv=<optimized out>) at pyflame.cc:268

This is a blocker to tag a new release.

Hmm. I don't have a Chromebook, so it'll be difficult to reproduce. I think the key question is what state the second thread is in. Is it already stopped, or otherwise not allowed to run (in which case the fix is to not bother attaching), or is it being stopped by the attach call, but for some reason pyflame isn't notified (a race maybe? maybe there's some way we can check if it's already stopped before we wait), or is it just falling to attach?

Seems to work on regular Jessie (with kernel 3.16.0-4-amd64). I'll poke around on my Chromebook some more and see if I can figure out what's going on.

I created a branch called "wait" that prints some debugging info.
On my chromebook exhibiting the bug I see: https://gist.github.com/eklitzke/c7bdc74b162b5a9eb3c3ff70a141d84a
On a regular jessie (without the bug) I see: https://gist.github.com/eklitzke/e84dc1993fc0a81bb8311268357d335f
On regular jessie what happens is each thread/process starts out in the sleeping state (S), then moves to the running state (R) after the PTRACE_ATTACH, and then moves to the traced state (t) after the call to wait().
On my chromebook what happens is the main process goes S -> t -> t, and then the next thread goes S -> t -> wait fails here.
I'm guessing this is some kind of kernel change, either between 3.14 and 3.16, or perhaps some patches Google applies on the Chromebook kernels. If I can't reproduce with a stock 3.14 kernel then we can ignore this I guess.

It would certainly make sense for this to be ChromeOS specific. I know they're big on sandboxing, so they enable a lot of the kernel's security controls, as well as adding a few of their own. I'll see if I can get a Chromium OS VM running, and try to get to the bottom of this.

I spent some time building kernels today, and it looks like something changed between kernel 3.15.10 and kernel 3.16.0. Here's how to reproduce:

Install Debian Jessie in a VM
Download https://www.kernel.org/pub/linux/kernel/v3.0/linux-3.15.10.tar.xz
Build the kernel using these instructions: https://www.debian.org/releases/jessie/i386/ch08s06.html.en (see below for details)
Reboot the VM, click the "Advanced options" choice in the first GRUB menu, and then select your 3.15.10
Check your kernel version with uname -a
In one session in the VM run python tests/threaded_sleeper.py and in the other session run sudo pyflame <PIDOFSLEEPER>

Here's specifically how I built a kernel, after you've installed the build dependencies in the previous link:
tar -xvJf linux-3.15.10.tar.xz
cd linux-3.15.0
cp /boot/config-3.16.0-4-amd64 .config  # or whatever config you have in /boot
make oldconfig   # hit ENTER for all questions, there should only be a few of them
make-kpkg clean
fakeroot make-kpkg --initrd --revision=1.0.custom kernel_image
sudo dpkg -i ../*.deb

I don't think this is high priority because there aren't any widely used distros with kernels this old (maybe RHEL/Centos?). But there's probably a simple fix here related to the semantics of ptrace/wait changing. I looked a bit at the kernel changelogs and saw a number of ptrace changes, but none that looked relevant.

Interestingly the bug happens with a stock 3.16.0 kernel. The kernel that comes with Jessie (which does not have this bug) purports to be 3.16.0-4, which means it's 3.16.0 + a bunch of patches that Debian has likely backported.
Here's a stock kernel.org 3.16.0 kernel that I built for amd64 if you want to try to repro the issue: http://eklitzke.org/linux-image-3.16.0_1.0.custom_amd64.deb

If it affects 3.16.0, but not 3.16.0-4, that should help us track it down. Bisecting those Debian fixes should pin it down to a small number of changes. I'll try and find some time take a look later today.

The Debian changelogs show that it includes https://patchwork.kernel.org/patch/8593281/, which seems like the culprit. It's a workaround for lazy developers (in the original bug the developers of init, or in this case me) failing to use the __WALL flag when waiting on child threads. It should just be a question of adding the __WALL flag to the waitpid call options. I'll try it out when I get home.

I'm not so sure, because if that was the bug I'd expect the waitid branch (which I just pushed) to not have this bug. The relevant code there:
static void do_wait(pid_t pid) {
  std::ostringstream ss;
  siginfo_t infop;
  if (waitid(P_ALL, pid, &infop, __WALL) == -1) {
    ss << "Failed to waitid() on pid " << pid << ": " << strerror(errno);
    throw PtraceException(ss.str());
  }
  if (infop.si_status != CLD_STOPPED) {
    ss << "Expected waitid() for pid " << pid << " to yield CLD_STOPPED";
    throw PtraceException(ss.str());
  }
  if (infop.si_pid != pid) {
    ss << "Expected waitid() to return pid " << pid << ", instead got " << infop.si_pid;
    throw PtraceException(ss.str());
  }
}

I'm trying to bisect the kernel, and it's something relatively recent (e.g. I saw a 4.5 kernel with the issue). It could also be that I misunderstand the waitid bug.

So, it is definitely the patch you linked to earlier. I built the kernel at that git revision, and the one before, and once that commit is in the kernel the problem no longer surfaces.
What I don't understand is that according to the bug report, the code should work on old kernels when using waitid() with  __WALL. So I'm not sure what error I made in my waittid branch.

Some interesting commentary on this change here: https://lwn.net/Articles/688624/

The article mentions a subtlety of __WALL that I missed reading the docs: it can't be used with waitid (the docs are less clear what it can be used for, although waitpid seems the most likely guess).
So it seems like the docs are suggesting that a similar call with waitpid may work on the older kernels, although I'm becoming increasingly confused by the semantics of these calls. I fear I may have to actually look at the implementation in the kernel source.

The relevant code in the kernel is in the patch you already linked to. From what I can see, the old code does check for __WALL (see the red lines from the diff). However, the LWN post says the opposite:

and, besides, the waitid() system call, unlike wait4(), does not even accept the __WALL flag

GDB has been able to debug multi-threaded programs forever, so there must be a way to do this by doing whatever GDB has done historically. I'll experiment with using wait4() instead of waitpid() or waitid().

It looks like what people have done in the past (before the ptrace patch was merged) was to literally send SIGSTOP to the main process, which stops all threads. Then you can go in and PTRACE_SEIZE the already-stopped threads without waiting for them. This is really annoying, especially given that the much saner code you already wrote works.
I already added a --no-threads option in another recent commit. I'm a little bit worried about enabling a feature that will break kernels that are a year old, since a lot of people run older kernels in production, and don't necessarily apply security updates regularly. I think maybe the right thing to do here is to invert the option (so that by default multiple threads aren't traced), and then add an option --threads or the like for people who want multi-threaded profiling, with a note that it may not work on older kernels.
What do you think?

Yeah, that sounds reasonable. Multithreaded Python code is not all that common, and unpatched systems are terrifyingly common. Plus, inverting the options means that the new version doesn't break anyone's expectations (there'll be someone relying on (idle) who'll raise a bug if it's not the default).
And I agree that explicitly supporting unpatched kernels is not worthwhile. The old approach is messy and racy, and it seems like most distros are backporting security fixes like this one (although it's interesting that Chrome OS doesn't, given their usually laser focus on security - maybe their init is already immune to the issue this fixes).

Oh, and one other thing I gleaned from the docs: if you specify P_ALL, the process id is ignored. Just mentioning since I spotted that we're using an explicit process id.


bump nbsphinx & fix labels

Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

What does this PR do?
resolving for https://github.com/Lightning-AI/lightning/pull/17924
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Could be an alternative? https://myst-nb.readthedocs.io/en/latest/


USB Endpoint sturct 'opaque' variable very obscure

in other_usb_controls loops, the variable 'opaque' is set as such
e->opaque = 1;
this causes a compiler warning as it is setting a pointer to literal 1.
new method without warning, and comment what the opaque variable does
note: cnlohr mentioned "Honestly it should be a #define".

Turns out that was a bug, I got my USB stacks mixed up.  It was only in cdc_exp.c
I fixed it.


FR: Logs component defaults to all containers

Describe the problem/challenge you have
Using the Logs component (via NewLogs) seems to require me to enumerate all of a pod's containers.
Describe the solution you'd like
I'd like to be able to leave that up to Octant, and just request logs for all containers.
Extra-ideally, those logs could be combined/collated into one stream, as if I'd called kubectl logs <pod> --all-containers, but just being able to have all containers listed in the dropdown by default would be fine.
NewLogs could take container names as an option, or could take a component.AllContainers sentinel value, or just always show all logs -- I'm not sure if there's a use case for a Logs component that only shows allows logs from some containers.
Environment:

Octant version (use octant version): 02/05/2020 nightly
Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.4", GitCommit:"67d2fcf276fcd9cf743ad4be9a9ef5828adc082f", GitTreeState:"clean", BuildDate:"2019-09-18T14:51:13Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"15+", GitVersion:"v1.15.4-gke.22", GitCommit:"a6ba43f5a24ac29e631bb627c9b2a719c4e93638", GitTreeState:"clean", BuildDate:"2019-11-26T00:40:25Z", GoVersion:"go1.12.11b4", Compiler:"gc", Platform:"linux/amd64"}

OS (macOS 10.15, Windows 10, Ubuntu 19.10 etc): macOS 10.14.6 (18G2022)

Yeah, that seems reasonable. Thanks @ImJasonH

Changed the NewLogs signature to accept only a single argument: pod for which logs are collected.  That way caller will get all logs ant they can later be filtered through dropdown in the UI or additional metadata that will be available after #668 is completed.

After the initial investigation and the work that Milan did, I think this feature is large enough that it deserves some thought. I will write a proper proposal for this work today and get it up as a PR so folks can have some time to review it and discuss.

@GuessWhoSamFoo pointed out https://github.com/wercker/stern which is a tool that does work around joining multiple log streams together.


Insert list of Item at once

How to insert list of items at once

i have large amount of data if you don't mind may  i get any example of it ?
like i have  9000 list of item and want to store at one go

The above example should be sufficient to outline how you would do it. Just create all the instances that you want to add as unmanaged objects in a list and add them in one transaction with
realm.write {
    list.map { copyToRealm(it) }
}

Closing this due to inactivity. If more help is need please add more comments or open a new issue.


Move files containing postal codes, municipalities and word list to separate folder

They are located in root atm.

On it

#167


Fix vulnerability and package relationship for RHEL

To reduce disk utilization, we separated vulnerabilities from the related packages, and we related them via the vulnerability's name (ex: CVE-2022-1650). However, when a vulnerability (for example CVE-2022-1650) has a different severity for different packages/environments, this becomes a problem, as multiple entries from the vuln table will be returned. We just simply took the first entry, which means for something like dotnet, the vulnerability entry we returned could have been attributed with severity Important instead of the correct severity, Moderate.
This PR fixes the relationship between vuln and vuln_package by using the vulnerability's hash as the "foreign key" instead of just the name. That way, we attribute the package to the correct vulnerability entry, which will give us the correct severity.
Creating a migration for this would have been a headache, as we need to correctly align the entries in vuln_package to vuln, as they are not correct now. This does not seem very easy to do with pure SQL. However, ScannerDB is currently not persisted, so we can get away with not performing a DB migration.
These changes will not affect live Scanners, as those Scanners come with their own already-made genesis dump and any diff we perform comparing the genesis dumps with the most up-to-date data do not involve the Postgres tables, but rather the Go struct RHELv2Vulnerability, which it not affected.

/retest

Need to check and test if instead of updating existing entries we just make new entries in the tables. It is preferred to update existing entries, so not continuously increase the DB size upon update

Going to close this to rethink if there is another/better way to do this

Replaced by https://github.com/stackrox/scanner/pull/895


PLAT-619: fix TestVirtual_Constructor_BadClassRef

- What I did
test relied  on order of messages. Now it checks just that all required messaged has come
- How I did it
- How to verify it
- Description for the changelog

bors r+

bors merge


Do you need crave token to build asop rom

HI I wanted to ask if you needed crave token to build ASOP rom ?
is it normal to wait 6 hours without getting a build done?
if I sign up to github enterprise does the runner excepend longer?
many thanks

HI I wanted to ask if you needed crave token to build ASOP rom ?

No, The team clarified that this was a bug in the official chat

is it normal to wait 6 hours without getting a build done?

What do you mean by this?

if I sign up to github enterprise does the runner excepend longer?

I don't think so, considering the docs don't mention anything like that.


remove age 1 column in age 2+ survey age proportion table

The age 1 column should be removed to not introduce confusion in Table 11 as this is an age-2+ survey.

They're all zeros though (and were last year). Maybe we can just tweak the caption to say age-2+.

Though I see that some refs to Table 11 should be Table 12. Will fix.


Wrong URL for package

Hello,
Our package Braket.jl moved parent organizations from awslabs to amazon-braket and it looks like the entry at juliapackages.com didn't update. Any way to get this fixed?

Updated now: https://juliapackages.com/p/Braket


head function

no more rightToMaybe, listToMaybe, maybeToMaybe etc....

:shaved_ice:

(just added a minor cosmetic correction https://github.com/ambiata/p/commit/c2569ccabe8cbe26b17a39cdd6579c59be9eac57)

great addition :rocket:

Great. @domdere would it be useful to implement last as well?

@etorreborre not sure tbh, dont know how common that use case is. but i think @markhibberd  would argue that if last was being used frequently enough to warrant its existence, then its a smell that we arent using the right data structure (for e.g we should be using a snoc list maybe)
but i think we can get rid of leftToMaybe and its ilk if bifunctors is added to p and we compose head with [Clown] (https://hackage.haskell.org/package/bifunctors-<IP_ADDRESS>/docs/Data-Bifunctor-Clown.html). Again don't want to bother with it unless there are more use cases for it other than Either

@domdere We added a dependency on either last week, so p already depends on bifunctors, just need to re-export things if you want.


Which graphql client fits better with Juniper?

I would like to know which graphql client fits better with Juniper, Apollo GraphQL client  OR Relay Modern?

It should work fine with either. Note that Juniper doesn't directly support Relay in the library, you either need to encode the Relay semantics yourself or use a helper library like https://github.com/bbqsrc/juniper-relay. We also have a task to make integrate better with graphql-client: https://github.com/graphql-rust/graphql-client/issues/20


Is it possible to retrieve a list of indexed documents (paths and/or title/metadata), and the vector store itself, from a GPTVectorStoreIndex?

Hi: I mistakenly asked https://github.com/jerryjliu/llama_index/issues/3249 but have now realised the correct question is: is there a way to query my GPTVectorStoreIndex in order to retrieve the list of documents that were originally indexed?  And also their text snippets?  And also to fetch the vectors their snippets mapped to?

I see that the (0.6.0) index persisted on disk contains: docstore.json, index_store.json and vector_store.json, but they don't seem to contain file paths or title metadata from the original documents, so maybe that's not captured and stored?

I was able to get the vectors from my index like this:
        vector_store = index.storage_context.vector_store
        vector_store_dict = vector_store.to_dict()
        embedding_dict = vector_store_dict['embedding_dict']
        vectors = numpy.array(list(embedding_dict.values()))

but it seems a bit non-portable: is there a more direct method?

The closest I could get to a list of the indexed documents is:
        index_store = self.index.storage_context.index_store
        index_store_dict = index_store.to_dict()
        doc_ids = list(index_store_dict['index_store/data'].values())[0]['__data__']['doc_id_dict']

which is a list of ids only.  I don't know if I can use those ids to look up paths or document metadata, but that would be great!  (Path at least.). Any ideas?

The vectors themselves are not exactly easily exposed. As you saw there, you can get the doc_ids. Then using those ids, I think you can fetch the nodes from the docstore
Can I ask why you need this information though? Just for debugging?
Another neat feature is you can do something like this, which will only retrieve the nodes that would have been sent to the LLM (but doesn't actually call the LLM). The source nodes will also include the similarity score as computed against your query string.
query_engine = index.as_query_engine(response_mode='no_text')
response = query_engine.query("my query")
print(response.source_nodes)

Thanks!  I'm trying to cluster (all) the vectors, then generate a description (label) for each cluster by sending (just) the vectors in each cluster to GPT to summarize, then associate the vectors with the original documents and classify each document by applying a sort of weighted sum of its cluster-labeled snippets.  Not sure how useful that will be, but I want to try!  I've got the vectors now (although I'm bit worried that the nested structure I'm getting them from might change without warning in the future!), and I'm able to cluster them, but I don't know how to associate the vectors (via their nodes) back to the original documents yet...

Should be able to associate them using the doc ids 🤔
vector_store._data.text_id_to_doc_id -> this will help map the doc ids to text ids
vector_store._data.embedding_dict  -> this is accessed using text ids

Thx!  I didn't see any document titles or file names/paths in the embedding_dict though...

Ah yea, then there's that issue.
Tbh for this task, you might be better off generating the embeddings yourself outside of llama index 😅

OK, thanks: it's not a must-have at this point.

Related to this, is it possible to get a list of nodes from an existing index? I am trying to load saved vector index state into another vector index.


[STORM-3667] make exceptions more understandable to submitters when topology submission fails due to incorrect worker max heap size config

What is the purpose of the change
https://issues.apache.org/jira/browse/STORM-3667
When submitting a topology failed because of incorrect max worker heap size config, the exception returned to submitter is not easy to understand: "Internal error processing submitTopology"
How was the change tested
With
bin/storm jar /home/y/lib64/jars/storm-starter.jar  org.apache.storm.starter.WordCountTopology -c topology.worker.max.heap.size.mb=100  -c topology.component.resources.onheap.memory.mb=120 wc

the exception message is much more clear to submitters now:
22:34:43.683 [main] ERROR o.a.s.StormSubmitter - Topology submission exception: Topology will not be able to be successfully scheduled: Config TOPOLOGY_WORKER_MAX_HEAP_SIZE_MB=100.0 < 120.0 (Largest memory requirement of a component in the topology). Perhaps set TOPOLOGY_WORKER_MAX_HEAP_SIZE_MB to a larger amount
java.lang.RuntimeException: InvalidTopologyException(msg:Topology will not be able to be successfully scheduled: Config TOPOLOGY_WORKER_MAX_HEAP_SIZE_MB=100.0 < 120.0 (Largest memory requirement of a component in the topology). Perhaps set TOPOLOGY_WORKER_MAX_HEAP_SIZE_MB to a larger amount)
	at org.apache.storm.StormSubmitter.submitTopologyAs(StormSubmitter.java:291)
	at org.apache.storm.StormSubmitter.submitTopology(StormSubmitter.java:210)
	at org.apache.storm.StormSubmitter.submitTopology(StormSubmitter.java:173)
	at org.apache.storm.topology.ConfigurableTopology.submit(ConfigurableTopology.java:119)
	at org.apache.storm.starter.WordCountTopology.run(WordCountTopology.java:58)
	at org.apache.storm.topology.ConfigurableTopology.start(ConfigurableTopology.java:68)
	at org.apache.storm.starter.WordCountTopology.main(WordCountTopology.java:36)
Caused by: InvalidTopologyException(msg:Topology will not be able to be successfully scheduled: Config TOPOLOGY_WORKER_MAX_HEAP_SIZE_MB=100.0 < 120.0 (Largest memory requirement of a component in the topology). Perhaps set TOPOLOGY_WORKER_MAX_HEAP_SIZE_MB to a larger amount)
	at org.apache.storm.generated.Nimbus$submitTopology_result$submitTopology_resultStandardScheme.read(Nimbus.java:9427)
	at org.apache.storm.generated.Nimbus$submitTopology_result$submitTopology_resultStandardScheme.read(Nimbus.java:9404)
	at org.apache.storm.generated.Nimbus$submitTopology_result.read(Nimbus.java:9338)
	at org.apache.storm.thrift.TServiceClient.receiveBase(TServiceClient.java:88)
	at org.apache.storm.generated.Nimbus$Client.recv_submitTopology(Nimbus.java:319)
	at org.apache.storm.generated.Nimbus$Client.submitTopology(Nimbus.java:303)
	at org.apache.storm.StormSubmitter.submitTopologyInDistributeMode(StormSubmitter.java:344)
	at org.apache.storm.StormSubmitter.submitTopologyAs(StormSubmitter.java:279)
	... 6 more

The change looks okay except, the validation should be performed only if Nimbus is using ResourceAwareSchduler
https://github.com/apache/storm/blob/master/storm-server/src/main/java/org/apache/storm/daemon/nimbus/Nimbus.java#L3150
Some condition like should help avoid that if (underlyingScheduler instanceof ResourceAwareScheduler)

The change looks okay except, the validation should be performed only if Nimbus is using ResourceAwareSchduler
https://github.com/apache/storm/blob/master/storm-server/src/main/java/org/apache/storm/daemon/nimbus/Nimbus.java#L3150
Some condition like should help avoid that if (underlyingScheduler instanceof ResourceAwareScheduler)

I will file a separate Jira.


Http operation impl

Http operation DSL based on @kitlangton's zio-api. Notable changes to zio-api:

removed the parts related to http request handling
merged the Input and Params type in API. This way it has an Input and Output type that correlates directly with the types on zio-flow's Operation.

The idea is that query parameters, paths segments, headers and the request body will all be part of the Activity's input, so it can be used like this:
  val setUserData = Activity[(Int, UserData), Unit](
    "setUserData",
    "set user data",
    Operation.Http(
      "https://example.com",
      API.post("users" / int).input[UserData]
    ),
    check = ???,
    compensate = ZFlow.succeed(())
  )

  val flow = for {
    id   <- ZFlow.succeed(1)
    data <- ZFlow.succeed(UserData("Albert"))
    _    <- setUserData(id, data)
  } yield ()

Two issues I am still working on:

Defining the schema on the Http operation's Input type. I don't know if this schema should be built from the ground-up and defined on every RequestInput in path.scala, or we should just require an implicit Schema[Input] when creating the Http operation.
The Http operation, with API itself  has to be serializable. One tricky thing with implementing that is that in Path.scala, the description of request inputs are having unzipping functions as fields, and I don't know yet how this description should be serialised.

I pushed an implementation of schema instance for the Http operation DSL that compiles, though tests are still failing on that. I think I can make it work with a few changes.

The Http operation, with API itself  has to be serializable. One tricky thing with implementing that is that in Path.scala, the description of request inputs are having unzipping functions as fields, and I don't know yet how this description should be serialised.

You can turn the Zippable trait into a sealed trait.

Thanks @kitlangton. I did exactly that in the meantime. :) Now the HTTP operations are serialisable, there is still some failing test which I am working on.

I think the general direction of this PR is good, but I'd appreciate some feedback about the details. Also asked @kitlangton wether we should update zio-api directly and use it as a dependency here.


[doc] Adjust PFM url in tutorial

Codecov Report

Merging #2361 (28d0d5f) into master (1407ae7) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2361   +/-   ##
=======================================
  Coverage   86.14%   86.14%
=======================================
  Files          56       56
  Lines       10225    10225
=======================================
  Hits         8808     8808
  Misses       1417     1417

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1407ae7...28d0d5f. Read the comment docs.


Allow concourse to push release commits

/kind bug
What this PR does / why we need it:
Allow concourse to push release commits
Which issue(s) this PR fixes:
Release jobs in concourse are failing because it is unable to push release commits:
2022-03-30 13:58:29,565 [WARNING] step.release: An error occured while applying step_name='Create Release Commit' e=GitCommandError(['git', 'push', '--porcelain', 'tbdjkrreveld', 'f82dced03a904a398cc5184e32729211f499621e:release-v1.41'], 1, b"error: failed to push some refs to 'ssh://github.com/gardener/gardener'")

ref https://concourse.ci.gardener.cloud/teams/gardener/pipelines/gardener-release-v1.41/jobs/release-v1.41-release-job/builds/9#L624ed8af:13
Special notes for your reviewer:
I'm not sure, if the team is correctly specified. But let's try :)

/lgtm


Optimize output of debug instrumentation flush

The Issue
#5023
How This PR Solves The Issue
Manual Testing Instructions
Automated Testing Overview

Related Issue Link(s)
Release/Deployment Notes

Instead of the EOF warning a propper message is now printed if the cache is empty


Request: Provide XLS BOM?

Hi there! This project looks great! My only request is to upload an XLS BOM so that people who don't want to source the individual surface mount components can just have the fab assemble that part.
Thanks for your hard work!

Will work on it. The BOM that Kicad generates is not that useful. I will create something manually. Not too many parts to list.

Added BOM.  https://github.com/zapta/simple_stepper_motor_analyzer/tree/master/kicad

Thanks!

Crap, I forgot, they also need a CPL (pick and place) file. If that's not easy for you to produce, don't worry about it.

Ben, is this is what you are looking for?  https://support.jlcpcb.com/article/84-how-to-generate-the-bom-and-centroid-file-from-kicad
I can try going over the instructions and export.  BTW, I looked at SMD service at JLCPCB but am not sure they have the exact Alegro current sensors.

Yeah, that's it! Hmmm you checked on LCSC.com? That's unfortunate. I wonder if I'd have better luck through PCBway?

@bepstein111 Did you have chance to generate that CPL file?
I found Alegro current sensor at https://www.aliexpress.com/item/4000178092630.html

Be very careful of aliexpress vendors.  There is a reasonable chance you will get a fake chip. Personally, I would not trust them - way too many stories of remarked chips that aren't even close to what they claim to be.

Here is another one. This is witha  a different package that most makers
would consider easier to solder (will require a minor PCB change).
https://www.aliexpress.com/item/4000178147058.html
On Sat, Apr 24, 2021 at 11:48 PM phil-barrett @.***>
wrote:

Be very careful of aliexpress vendors. There is a reasonable chance you
will get a fake chip. Personally, I would not trust them - way too many
stories of remarked chips that aren't even close to what they claim to be.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/zapta/simple_stepper_motor_analyzer/issues/2#issuecomment-826268865,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQVMQL76VEHAOUHKYE6P6DTKO3KXANCNFSM43DINNXA
.

FYI, I just released an upgraded design which is based on the Raspberry Pi Pico. It also includes gerbers, bom and pos files for JLCPCB SMD service, for the parts they do carry.
https://github.com/zapta/simple_stepper_motor_analyzer


Cannot use

I noticed a file script-hoisting.test.ts in the test case. I try to use the same syntax in example, but I failed.
I tried a lot of ways. It seems that this part is all broken.
↓↓↓↓↓↓↓↓↓↓↓You can see examples here↓↓↓↓↓↓↓↓↓↓↓
demo
↑↑↑↑↑↑↑↑↑↑↑You can see examples here↑↑↑↑↑↑↑↑↑↑↑

@FuBaooo I am working a new PR and I have switched in your test file as the repo's primary test mechanism as it supports your view that this is not working as it should.

note that you can get everything to work if you're secondary script blocks are just <script> blocks and not <script setup> blocks, right?

I haven't tried <script> yet, but I think <script setup> should work normally since it is in the test case.
I tried to copy the usage method in the test example, but failed.I think since it is in the test case, it should work

note that you can get everything to work if you're secondary script blocks are just <script> blocks and not <script setup> blocks, right? This is the difference which you introduced in repeating the <script setup> block multiple times which I'm not sure would ever have worked. Does it say it should work in the docs?

The test case I had before worked fine but your test case added multiple <script setup> blocks which did not work. I think in general it would be a better idea for people to add <script> blocks in their code simply because the cardinality of <script> is 1:M but that said I have updated the test to the more aggressive test you introduced and I have all tests passing there too now with a little refactoring. This code isn't released yet but will be soon.

this is fixed in v0.16.0


Stop pipeline when ExampleValidator finds anomalies

In cases where any anomalies that are found by ExampleValidator should stop the pipeline execution, the standard components don't have a way of stopping it.  That's because none of the standard components depend on the results from ExampleValidator.
We can write a custom component to take the ExampleAnomalies artifact from ExampleValidator, along with the Examples artifact from ExampleGen, and parse the ExampleAnomalies artifact.

If no anomalies are found, the component passes through the Examples artifact from ExampleGen
If any anomalies are found, the component will fail

Downstream components which depend on the Examples artifact from ExampleGen can instead depend on the Examples artifact from this new custom component.  When it fails, the pipeline will stop.
An enhancement of this would be to either include a user code module to check the type of anomaly and decide whether to fail or not, or some sort of configuration parameter.

this sounds very reasonable. what is the current status of this idea?

and is there a way to terminate pipeline from a component? maybe just raise an error?

sounds good and sounds easy to implement I guess. What is your plan on this?

I don't have a plan to work on this, it's a project idea for someone to pick up.  I agree, it sounds fairly simple, except for the part about user code to check the type of anomaly and decide whether to fail or not, or some sort of configuration parameter.  Even that shouldn't be too hard.


onResize and onResizeCapture are required for virtually all components

Those are all the dependencies of my Sanity Studio project:
{
  "dependencies": {
    "@sanity/color-input": "^3.0.1",
    "@sanity/icons": "^2.1.0",
    "@sanity/table": "^1.0.1",
    "@sanity/ui": "^1.0.1",
    "@sanity/util": "^3.0.6",
    "@sanity/uuid": "^3.0.1",
    "@sanity/vision": "^3.0.6",
    "csv-reader": "^1.0.10",
    "lodash.get": "^4.4.2",
    "lodash.merge": "^4.6.2",
    "match-sorter": "^6.3.1",
    "mocha": "^10.2.0",
    "react": "^18.2.0",
    "react-dom": "^18.2.0",
    "react-icons": "^4.7.1",
    "react-is": "^18.2.0",
    "react-query": "^3.39.2",
    "react-virtuoso": "^3.1.5",
    "sanity": "^3.0.6",
    "sanity-plugin-datatable": "^1.1.0",
    "styled-components": "^5.3.6"
  },
  "devDependencies": {
<EMAIL_ADDRESS>"^4.4.7",
<EMAIL_ADDRESS>"^4.6.7",
    "@types/mocha": "^10.0.1",
    "@typescript-eslint/eslint-plugin": "^5.46.1",
    "@typescript-eslint/parser": "^5.46.1",
    "eslint": "^8.29.0",
    "eslint-config-prettier": "^8.5.0",
    "eslint-config-sanity": "^6.0.0",
    "eslint-plugin-react": "^7.31.11",
    "mocha-junit-reporter": "^2.2.0",
    "prettier": "^2.8.1",
    "ts-node": "^10.9.1",
    "typescript": "^4.9.4"
  }
}

Almost all components have TypeScript errors (in this screenshots, all but <Stack/>):

The error is:
Type '{ children: string; }' is missing the following properties from type 'Pick<TextProps & Omit<HTMLProps<HTMLDivElement>, "size" | "as">, "className" | "color" | "height" | "id" | "lang" | "max" | ... 353 more ... | keyof TextProps>': onResize, onResizeCapture ts(2739)

Adding onResize={undefined} onResizeCapture={undefined} on the components removes the error but it's not optimal.
Am I the only one having this issue? :)

Hello @thobas-dnvgl – this is caused by an unfortunate mismatch between older and newer versions of @types/react caused by this PR: https://github.com/DefinitelyTyped/DefinitelyTyped/pull/63076
The solution is to update your @types/react dependency to the latest version.


Should IDs be globally unique?

An example is a persist annotation + an exec unit having the same ID

We'll go with globally unique IDs going forward


Feature request: analogue stick text input

I use my DS4 as a remote control for my pc sometimes but whenever i need to type something (which happens a lot on a computer designed for keyboard and mouse input) i need to go over to the keyboard to get anywhere. I have seen solutions for this on console games that give you the ability to write efficiently by moving the two analogue sticks in different ways, aka Twostick, Quikwrite and by other weird names that most people haven't heard of. I think the ability to pull up a keyboard interface of sorts by pushing a button of choice would be a gamechanger and huge differentiator for the software that incorporates it, and I know that i would stop swapping back and forth if you implemented something like this.

Controller Companion works well with DS4Windows and has a built in keyboard interface that works well. I've been using it for a while, definitley worth the £1.99. I originally used it with an xbox 360 controller but it works just as well with the DS4.


Cask request: (Fix) Charles

Cask details
The cask in question in Charles to install Charles Proxy.
The Cask is currently broken because Charles Proxy requires a different type of URL. The new URLs require making a post to a page and parsing the data. This seems a bit brittle so in lieu of making a PR, I wanted to ask what to do first.

What's wrong with the current URL?
Yes, the download page serves up https://charles-52f.kxcdn.com/release/4.0/charles-proxy-4.0.dmg, but the URL in the Cask still seems to be valid

You are correct @adidalal. It appears that older versions are not working. The latest 3x and 4x are both OK but 3.11.2 returns a 404. I believe there's nothing to do unless we want to put a warning in about updating if a 404 is detected.


consider modularizing APIs

Should we have

one plugin covering the entirety of the monarch API/KG

generic endpoint for multiple entity types
multiple endpoints

separate plugins for different types, e.g. disease, gene

My reading of the OAI docs suggest that 2 is best. This might be a bit more complicated for deployment, juggling multiple fastapi endpoints...
but perhaps the thing to do is focus on a disease endpoints; however, this may limit certain options such as chatgpt reasoning about how to chain multiple endpoints together (e.g. find all pairs of interacting genes in the same disease...)

I haven't tested it but I suspect the chaining could happen if both plugins are enabled. But that would mean users would have to know about the different plugins and actively choose to enable them though, so option 1 seems better to me. Which part of the docs are mentioning it?
I'm not sure about 1a vs 1b though - whether the model is better able to handle a few flexible endpoints, or more special purpose endpoints, I dunno.

Closing as we are live as of #45


Codebase modified to work with Redmine 2.0.x.

It works with Redmine 2.0.x, but not 1.4.x...

could you modify to work with redmine 3.x

I also subscribe to request to make the plugin compatible with Redmine 3.x