Incorrect parsing of ksmg messages with continuation lines

currently expr matching needs to match until end of text:
https://github.com/kubernetes/node-problem-detector/blob/9fd58e318ff6c4e06b4a50094245ab22f4040c77/pkg/systemlogmonitor/log_buffer.go#L65-L67
ksmg log watcher removes ending newline, but doesn't remove newlines in between the message itself.
for example, here are messages I want to match:
I0505 00:40:26.905993       1 log_watcher_linux.go:108] got kernel message: {Priority:3 SequenceNumber:114864 Timestamp:2023-05-05 00:40:17.630052342 +0300 MSK m=+395.145047298 Message:mlx5_core 0000:31:00.0 ens2f0: Recovered 1 eqes on EQ 0x3b
 SUBSYSTEM=pci
 DEVICE=+pci:0000:31:00.0
}

judging from kernel docs:
https://www.kernel.org/doc/Documentation/ABI/testing/dev-kmsg

The human readable text string starts directly after the ';'
and is terminated by a '\n'. Untrusted values derived from
hardware or other facilities are printed, therefore
all non-printable characters and '' itself in the log message
are escaped by "\x00" C-style hex encoding.
A line starting with ' ', is a continuation line, adding
key/value pairs to the log message, which provide the machine
readable context of the message, for reliable processing in
userspace.
Example::
7,160,424069,-;pci_root PNP0A03:00: host bridge window [io  0x0000-0x0cf7] (ignored)
SUBSYSTEM=acpi
DEVICE=+acpi:PNP0A03:00
6,339,5140900,-;NET: Registered protocol family 10
30,340,5690716,-;udevd[80]: starting version 181

so it looks like those parts should be ignored by ksmg parser, but they're not
because of this you have to write kmsg matchers with [\S\s]* in the end instead of .* -- since regexp is not instantiated with flag s=true.
I'm not sure on what should the fix be, but at least:

kmsg-parser should be fixed to handle continuation line with kv pairs correctly https://github.com/euank/go-kmsg-parser
probably regex should be instantiated with s=true flag

this is still an issue afaik

this is still an issue

Hey @stek29 If no one is working on this then I can take on this but I need some more details and steps to reproduce issue and get started


WinUI 3 - The attachable property 'InputBindings' was not found in type 'Window'.

Hi, I am trying to figure out how to use this utility in a WinUI 3 application. I followed the quickstart guide and added this in my XAML view.

However, I am getting the error in the title. I found this thread (https://github.com/microsoft/WindowsAppSDK/discussions/1974) which contains dead links. I used the wayback machine to view those links and I'm still not sure how to get this working. Any assistance would be much appreciated.
Here are the two archived links in that thread for your reference:
https://web.archive.org/web/20211223095955/https://docs.microsoft.com/en-us/windows/winui/api/microsoft.ui.xaml.window?view=winui-3.0
https://web.archive.org/web/20211223084314/https://docs.microsoft.com/en-us/windows/winui/api/microsoft.ui.xaml.controls.page?view=winui-3.0

Hmm, I'm not all that familiar with WinUI 3 to be honest. I briefly looked into it when I was working on modernizing a WPF application but I don't remember too much about it.
Is WinUI 3 its own stand-alone framework or is it something that you integrated into an existing UI framework (e.g. WinForms, WPF, UWP, etc.)?
I think the issue is that, at the moment, HotkeyUtility is only intended to be used in a WPF application. If you look at the Window class in a WPF application, it contains a property called InputBindings. In your case, it looks like you might be using the Window class that belongs to the Microsoft.UI.Xaml namespace which doesn't contain an InputBindings property.

Thanks for your reply. WinUI 3 is indeed a standalone UI framework. It is XAML based, much like WPF, and intended to be the modern choice for windows development (instead of UWP, WPF). I was looking for a hotkey manager and yours looks like the most capable, which is also intended for WPF (the others were built around winforms and aren’t viable for winui 3). There is a lot of carryover between WPF and WinUI3, so I assumed a wpf hotkey manager would work with winui3.
You are correct about your assumption. Does your library require itself to partly be implemented in the view? If so, is there an alternative to using the InputBindings property? Perhaps this property has been updated into something else in winui3?

Does your library require itself to partly be implemented in the view?

Actually, you don't have to implement it in the view. You can also implement it in the code-behind/view model if you like:
using System;
using System.Windows.Input;
using HotkeyUtility;

protected override void OnInitialized(EventArgs e)
{
    Hotkey hotkey = new(Key.Space, ModifierKeys.Alt, HotkeyBinding_Pressed);
    HotkeyManager hotkeyManager = HotkeyManager.GetHotkeyManager();
    _ = hotkeyManager.TryAddHotkey(hotkey);
}

public void HotkeyBinding_Pressed(object sender, HotkeyEventArgs e)
{
    // Handle hotkey press here
}

Since you're using WinUI 3, I'm not sure if you'll have an OnInitialized method to override but I'm sure something similar exists.

Thank you for that. I have managed to get it working in my viewmodel. The singleton pattern really makes everything a lot easier. I was wondering if there was any way to record keypresses in a textbox to assign to a hotkey during runtime?

I was wondering if there was any way to record keypresses in a textbox to assign to a hotkey during runtime?

I actually implemented something exactly like that in my application I mentioned earlier.

The way you would handle it is by monitoring the PreviewKeyDown event on a TextBox control and using the keys the user types as the new hotkey.
Here's a slightly modified version of how I implemented it in my application:
// You're going to need to keep track of the current hotkey combination to replace it later.
private Hotkey MyHotkey { get; set; }

public void HotkeyTextBox_PreviewKeyDown(object sender, KeyEventArgs e)
{
    ModifierKeys modifiers = Keyboard.Modifiers;
    switch (modifiers)
    {
        // Selected when a modifier key was pressed (other than the Windows logo key).
        case not ModifierKeys.None when modifiers != ModifierKeys.Windows:
            Key key;
            switch (key = e.Key == Key.System ? e.SystemKey : e.Key)
            {
                // Selected when a key other than Shift, Control, or Alt is pressed.
                case < Key.LeftShift:
                case > Key.RightAlt:
                    // The F12 key is reserved for use by the debugger at all times, so it should not be registered as a hotkey. Even when you are not debugging an application, F12 is reserved in case a kernel-mode debugger or a just-in-time debugger is resident.
                    if (key == Key.F12)
                    {
                        break;
                    }

                    // Ensures that if the user tries registering a key binding that's
                    // clearly already registered, the request will simply be ignored.
                    if (key == MyHotkey.Key && modifiers == MyHotkey.Modifiers)
                    {
                        break;
                    }

                    HotkeyManager hotkeyManager = HotkeyManager.GetHotkeyManager();

                    // This checks to see if the hotkey binding you're replacing actually exists.
                    Hotkey hotkey = hotkeyManager.GetHotkeys().Find(MyHotkey.Key, MyHotkey.Modifiers);
                    if (hotkey is not null)
                    {
                        // If it does exist, we replace it here:
                        _ = TryReplaceHotkey(MyHotkey.Key, MyHotkey.Modifiers, key, modifiers);

                        // Keep track of the new hotkey binding
                        MyHotkey.Key = key;
                        MyHotkey.Modifiers = modifiers;
                    }

                    break;
            }

            break;
    }
}

That's it pretty much. You're probably going to have to manually update the text in the TextBox. You could probably have a property like:
public string HotkeyTextBoxInput { get; set; }

...and bind that to the Text property of the TextBox control. When you successfully replace the old hotkey, convert MyHotkey.Key and MyHotkey.Modifiers into a string representation and assign it to HotkeyTextBoxInput. This is a method I created to do that that you can use if you like:
private static string ConvertKeysToString(Key key, ModifierKeys modifiers)
{
    return modifiers.ToString().Replace(",", " +") + " + " + key.ToString();
}

Thank you for this. I ran into some trouble when using this for WinUI 3, but I have managed to resolve it after a lot of trial and error.
WinUI 3's PreviewKeyDown has an event of type KeyRoutedEventArgs rather than KeyEventArgs. Therefore, the type of e.Key is a Windows.System.VirtualKey enum rather than a System.Windows.Input.Key enum which is what your library requires. In order to fix this, the VirtualKey must be converted to a Key. This can be done by:
Key key = KeyInterop.KeyFromVirtualKey((int)e.Key);

Have you considered the case where the user wants to create a new hotkey during runtime? For example, if they want to create a new hotkey to do a math sum. I have tried to create a blank hotkey, but when I access MyHotkey.Key, it is null and I can not assign anything to it.
My thought process is that I can create a new blank hotkey in a click handler (like a button to add a new hotkey in a list) like this:
private void Button_Click(object sender, RoutedEventArgs e)
    {
        Hotkey hotkey = new(Key.None, ModifierKeys.None, ActivateHotkey(int.Parse(num1TextBox.Text), int.Parse(num2TextBox.Text)));
    }

    private void ActivateHotkey(int num1, int num2)
    {
        Debug.WriteLine(num1 + num2);
    }

However, the hotkey does not accept the ActivateHotkey function. Is another way of doing this to perhaps add num1 and num2 as a property to the Hotkey class, and store those during creation, so that they can be modified later too, and manipulated in the hotkeypressed function?


Minikube

:tada: This PR is included in version 0.5.1 :tada:
The release is available on:

PowerShell Gallery
GitHub release

Your semantic-release bot :package::rocket:


[SDC] Change the alignment to make boolean type options full screen width

Problem
Boolean questions are not full screen width
Solution
Change the alignment to make boolean type options full screen width
EmCare commit
https://github.com/google/android-fhir/commit/f1408650e8ff71ee52169c1825e58249cdfbe784
cc @santosh-pingle @kunjan8794 @khyativyasargus @jingtang10

@kunjan8794 @khyativyasargus
I see corresponding commit opened for this feature on your repository.
Are you going to create PR corresponding to main repository android fhir sdk?
Or you want me to look into it?
cc @jingtang10

@santosh-pingle Sure. We will create PR for this in main repository.

@khyativyasargus thanks!, is there any approximate date by which we can expect the pull request?

@santosh-pingle Yes, You can expect all PRs done by 24th April, Monday. Let me know if it's needed before that. Thanks!


Fix configuration loading

This updates the configuration loader to merge command-line options with a specified configuration task, taking in order of precedence:

Options specified on the command-line
Options specified in the configuration file
Built-in command-line option defaults

Also, exit with error when there is an error loading the configuration.
Fix issue #356

This looks great!


YAML-time!

Huge refactoring effort:

Add support for YAML files
Clean up frontmatter for the various markdown templates
Make it easier to re-use authors (teammates)
Delete tons of repeated code!

@protoEvangelion I need to show this off to somebody, but you're the only one who might care... basically figured out how the Gatsby team is doing it, and replicated for us! I'm really happy w/how much code we can reuse now!

@abelhancock and @victorvalle — check out the updates to the Events and Team md templates! Now it's a lot easier to add people, events, articles, etc to the site!

Authors.yaml
Offices.yaml

Are now single sources for data, these get mapped in the gatsby-config.js file using the id's (person name / office name) — and now we can query that instead of having a bunch of repeated fields in various markdown files!

Nice I like it! @plhnk sheck this out https://www.gatsbyjs.org/packages/gatsby-remark-reading-time/


Support for new root chain by letsencrypt

Update root chain with new ones specified in https://letsencrypt.org/certificates/

Dehydrated doesn't have hardcoded chains. It uses the chain as provided by the CA. You can even select alternate chains the CA is offering using the PREFERRED_CHAIN config value or the --preferred-chain CLI option.

While you're right, Lukas, this probably should be put on the top of readme (both github and else) since it does bite almost everyone providing external certs.
As a community service, instead of letting them find it in closed issues. :-)
Also, solution is spelled out in #842 .

Dehydrated doesn't have hardcoded chains. It uses the chain as provided by the CA. You can even select alternate chains the CA is offering using the PREFERRED_CHAIN config value or the --preferred-chain CLI option.

Thanks , readme is not very clear with this


testing new coding methods

Hi - is there a standard metric that's been established to evaluate the performance of the various categorical encoding methods? In my own work I use a generalization of 1-hot encoding I call k-hot encoding, where k denotes the number of activate bits. I'd be keen to evaluate it on a standard metric if one exists to see if it's any good for more general tasks.

Hey,
K-hot sounds somewhat, but not quite, similar to the BaseN encoding we have here. I'd be interested to see how they compare in practice, K-hot may be worth adding into the library to see.
As for evaluation, there are some heuristics, like total dimensionality (so compression), and distortion (stdev of l2 distance between random points once encoded, should be very low). There is also the option of comparing eventual model performance on a mix of datasets (shown in the examples directory). Formally, I'm not sure of a much better way, but I'd be very interested to find one.

Moved this into a more specific #46, if you'd like to implement your k-hot algorithm, please open a separate issue for that.


about loss surface random direction

https://github.com/g-benton/hessian-eff-dim/blob/temp/hess/loss_surfaces/loss_surfaces.py
line57 should be ' dir2 = dir2 - dir1.mul(vu).div(uu)' ?

Good catch - fixed.


Added Azure.RSV.Immutable

PR Summary
Fixes #2386
Added Azure.RSV.Immutable rule.
PR Checklist

[x] PR has a meaningful title
[x] Summarized changes
[x] Change is not breaking
[x] This PR is ready to merge and is not Work in Progress
Rule changes

[x] Unit tests created/ updated
[x] Rule documentation created/ updated
[x] Link to a filed issue
[x] Change log has been updated with change under unreleased section

Other code changes

[ ] Unit tests created/ updated
[ ] Link to a filed issue
[ ] Change log has been updated with change under unreleased section

@BernieWhite Let me know.

@BernieWhite Thanks for the great feedback, I have adjusted accordingly now.


[CI] fix ocaml setup

hopefully this will fix issues we see in CI

this greatly increases the time CI takes, and it seems like we don't have the ocaml error anymore


feat: return signatures when signing message if they are present in the response

Part of https://github.com/safe-global/web-core/issues/1886
Overview
In order to enable a synchronous EIP-1271 signing flow, we have to return the signatures which are returned from the signing request.
The plan is to wait in the Safe{Wallet} web interface until the message is fully signed and then return the preparedSignature from our tx service.
This will make a lot of Dapps work with Safe off-chain signatures.

Could you please add a changeset and remove the version change? The release process is documented here:
https://github.com/safe-global/safe-apps-sdk/blob/main/docs/releases.md
You can skip the pre part unless you want to release a beta version first.

LGTM but there's something wrong with the tests. Also don't forget to push the dist folder so the package can be used directly from github 👍

We updated to node 18 which supports window.crypto. The polyfill in the jest setup broke the test.
I removed it and now they run.

Hey @schmanu i guess you removed the getRandomValues because it is now implemented in jsdom right. In any case, please remove it as well here then


Chat Support

Issue
Has anyone had success in implementing the (somewhat) new chat functionality?  I see and have been able to use the following endpoints to send a chat request and view the last message sent in the chat, but I can't find any endpoints to actually send chat messages.

PUT /v3/chat/accept
PUT /v3/chat/{id}/block
DELETE /v3/chat/{id}
PUT /v3/chat/reject
PUT /v3/chat/settings
GET /v3/chat/notifications/new
GET /v3/chat/notifications
GET /v3/chat/{id}/details
PUT /v3/chat/request
PUT /v3/chat/notifications/{notificationId}/read
PUT /v3/posts/{postId}/toggleChatRequests

I thought that /v3/chat/{id}/details might provide a transcript of the chat, but it only shows the last message sent.  Anybody have any suggestions on the endpoint used to send chat messages?

Sending and receiving messages is done via web sockets.


Certain "In" animations dont fade in when used as the initial effect.

Certain "In" animations seem to not "fade in" correctly when used as the first effect:
flash,bounce,shake,tada,swing,wobble, pulse are some examples.
The effect still works, but only after the full text kindof "slams" in all at once.   If however I use an effect like "fadeInBigLeft" first, all subsequent effects work properly.  The easiest way I've found to show this behavior is to edit the example playground (index.html) and change the line:
$form.find('[data-key="effect"][data-type="in"]').val('fadeInLeftBig');

to
$form.find('[data-key="effect"][data-type="in"]').val('flash');
I think this only changes the initial "In" Effect.
Basically I'm seeing the same issue in a HTML i'm doing where i just want to use the "flash" effect on some text but want it to fade in nicely from left to right
If i'm doing something wrong please let me know.  Otherwise I'd even settle for a workaround.
Thanks.

All this lib does is use animations defined by animate.css: https://daneden.github.io/animate.css/.
The flash,bounce,shake,tada,swing,wobble, pulse are not really designed for fading in and out.

I see that, but on the (unedited) demo page.  (https://jschr.github.io/textillate/), it seems like an additional effect is occurring with the effects I mentioned.
I've even noticed that they seem to fade in differently based on the effect type.  For example:
sequence - the letters fade in from left to right
reverse - from right to left
sync:  - all together
shuffle - randomly
Actually it's more like a reveal than a fade in.
If that is not the desired behavior and simply a side effect or bug, i wish i could reproduce it.  It provides an extremely nice effect.

@quizjax If I understand correctly, what you're seeing is expected. Sequence will wait for the previous letter to finish animating before animating the next, reverse is the opposite, sync will animate all letters at the same time and shuffle is like sequence but picks a random letter after the previous letter is done animating.
Hope that makes sense.

Also, for any effect designated as an in animation, the initial state of the letter will be set to visibility: hidden. Maybe thats what you're referring too?

Thanks @jschr ,
Yes I think that's what i'm referring to.  It seems like on the (unedited) playground page, the initial state is "visibility:hidden", so those letters get revealed as they are also doing their core IN animation (like a bounce, or flash).  All looks great.
However, (to demonstrate the issue i'm experiencing) if you edit the source on that page and change the default starting "in" effect to "bounce" like this:
$form.find('[data-key="effect"][data-type="in"]').val('bounce')
It seems like the initial state of all IN animations, is no longer "visibility:hidden".
I feel like something is not getting initialized properly.
Sorry if i'm not explaining this well.  Just started using this library today and it's really good.   I spent hours trying to figure out why my "bounce" looked different than the "bounce" on the test page.
Thanks for your help.

Here's a fiddle with initial value set to bounce: http://jsfiddle.net/jschr/81xb9nyy/
What I'm seeing there is expected. I should amend what I set in the previous post, in sequence mode each character doesn't wait for the previous to finish animating - it uses the in.delay config option. So what you see in the fiddle is each character's visibility set to hidden initially, then it's set to visible after 50ms * letterIndex and animated.
If you're seeing different results in your case, I'd be happy to investigate if you can provide a jsfiddle showing the problem.

I'm not seeing the initial value set to bounce.   Seeing:

var quotes = ["this is quote one", "quote two"];
$('.tlt')
.html(
'' +
quotes.map(function (quote) {
return '' + quote + '';
}).join('') +
''
)
.textillate();
Is this right?  I'd like to see exactly what you're seeing.Thanks.

Accidentally pasted the wrong link initially: http://jsfiddle.net/jschr/rkm47qvr/

Yes that looks great.  I will use that method.
Just for my own knowledge however, I modified that fiddle as follows:
http://jsfiddle.net/quizjax/6dsro0o1/1/
I put another animation below yours with the method i was trying to use.
Can you tell me why my method doesn't reveal from the left like yours does?  Is there a simple fix or does it just not work that way.
Again thanks for your help.

Each textillate instance needs its own container: http://jsfiddle.net/jschr/68185m5p/

Was that link an update?  Didn't see any changes.

hm.. you should be seeing "testing 123" bounce in after an initial delay of 1000ms, are you not?

Anyways, your html needs to be and it should work:

Some Title

Another Title

testing 123

I actually see the code change - but the result is the same on the right.   Tried Chrome, IE, Firefox.
The "testing 123" is always on the screen, and just bounces at 1 sec - but your text both reveals and bounces from the left.
Are you seeing it differently?  Maybe i got an old version of a library.

  I'll just use your method above.
Thanks for your patience

I get what you're trying to show me now and I think there actually is an issue. I definitely would expect the effects to be equivalent: http://jsfiddle.net/jschr/s7Lysmnw/
Seems there's a difference between the setting options with a JS config and HTML attributes that I wouldn't expect. I'm not sure which actually makes most sense by default but the HTML attribute way is definitely smoother.
I've opened up a new issue for it: https://github.com/jschr/textillate/issues/90
Thanks

I think I get what quizjax was trying to do.
The text needs to be revealed one by one during the data in effect.
It does not display like in the playground for some effects such as pulse
It happens naturally though when using the html attribute (data-in-effect:'pulse') just not with a JS config in { effect: 'pulse' }
I got around it by utilizing the two methods, Like so
http://jsfiddle.net/tvsrezzm/

@yesse33 your comment helped me!
I think the whole point is inconsistent behavior between js config effect: in and DOM attribute data-in-effect
I ended up doing
$('.tlt li').each(function(i, el){
	$(el)
		.attr('data-in-effect', 'swing')
		.attr('data-out-effect', 'hinge');
});

$('.tlt').textillate({
  "loop": true,
  "in": {
    "shuffle": false,
    "reverse": false,
    "sync": false
  },
  "out": {
    "shuffle": true,
    "reverse": false,
    "sync": false
  }
});


occupations vs affiliations update

In listperson (in data directory, not the one in archive), relevant affiliation elements have been changed to occupation @type="sk". They contain a @key containing an organisation key.
For these, create:

F51_Pursuit
URI as usual ([person-URI]/occupation/[n])
rdfs:label: [Person Name] + occupation/text()[1], e. g. "Friedell, Egon arbeitet für Nachtlicht"

Also create

E4_Period
URI derived from other individual-URIs, with addition: https://sk.acdh.oeaw.ac.at/ + [occupation/@key] + /period
rdfs:label: "Existence of [occupation/orgName/text()]"

Modeling:
E21_Person p14i_performed F51_Pursuit
[...]
.
F51_Pursuit p10_falls_within E4_Period
[...]
.

Also:
All affiliations are to be ignored.
Also, all temporal attributes in occupation @type="sk" (@when-iso, @notBefore-iso, @notAfter-iso) are to be ignored.
If this poses a problem for the script, these elements / attributes can be deleted.

code wise fixed with https://github.com/semantic-kraus/lk-data/commit/a4cb0ab660036e7e0e270f152a284e62c346e954
ToDo: dates ranges are serialised in a stupid way out of pmb


Resolve the doc differences for the configuration options

When we moved to auto-generated docs, it looks like some small bits of documentation may have been lost.
Compare https://github.com/istio-ecosystem/authservice/blob/master/docs/README.md with the table from https://github.com/istio-ecosystem/authservice/tree/6dfd1ebd5a61034238fe73d791cb7cbf5262d733/bookinfo-example and resolve the diferences by updating the proto files (the source of the auto-generated docs).

This was fixed by https://github.com/istio-ecosystem/authservice/commit/70bda2877448a32f0c069fab0bbb92cde46739ea#diff-e70100059cc3a7376740bbe9751a082e


Potential Memory Leak?

Hi, I am trying to figure out some memory leak problems in my server application, and I notice (from heapdump) that deferredConnection in lib/feedback.js is retaining quite some memory by holding references to lots of closure object. I go and read the source code and find the following snippet that may be causing the problem, I wonder if it is really the cause or is it just me using the library wrongly.
In lib/feedback.js
/**
 * Connect to the feedback service, also initialise the timer if an interval is specified.
 */
Feedback.prototype.start = function () {
	debug("Starting feedback service");
	this.cancel();
	if (this.options.interval > 0) {
		debug("Feedback service interval set at: %d", this.options.interval);
		this.interval = setInterval(this.request.bind(this), this.options.interval * 1000);
	}
	this.request();
};

/**
 * @private
 */
Feedback.prototype.request = function () {
	debug("Performing feedback request");
	this.createSocket().fail(function (error) {
		this.emit("feedbackError", error);
	}.bind(this));
};

It seems that each time request is called (in interval), a new fail handler is attached to the promise returned by this.createSocket() (if it is still the same promise) and it is not freed.
I am still using an older version(1.5.2) of node-apn, but that part of code is about the same. Not sure if that matters.

This is a fair observation, however it is incorrect for request to be called more than once on a single connection. When a request is performed it will connect to APNS and close the connection when all data is received. The close event handler disposes of the createSocket promise ensuring there is no memory leak. There should be little need to set the interval to less than 1 hour and for most applications once per day would probably suffice, thereby avoiding any memory leaks in this case.
Thanks for getting in touch though!


How to resume trainning?

Hi, it's really awesome work. So I'd like to train on my own dataset, but I accidently interrupt the trainning process. I have read the source code about resume checkpoint part, it is difficult for me to load checkpoint because I found the model seem like didn't save the right format (it only saved weight not optimizers and schedulers).
Can you give the method to resume trainning?

thank you for your attention, you can set the checkpoint path in here to resume training, the load function is in here


Remove the invocation to no longer existing python unit tests.

This is a quick fix to get the docker file build again.

This is entirely superseded by #9; my inclination is to not bother with it. @tbetbetbe do you agree?


Highstock component example not working

Version
nuxt-highcharts: 3.0.2
nuxt3: 3.0.0-27468803.23e7afb
highcharts: 10.0.0
Nuxt configuration
mode:

[x] universal
[ ] spa

Nuxt configuration for nuxt-highcharts

export default defineNuxtConfig({
  modules: ["nuxt-highcharts"]
});

Reproduction Link

https://codesandbox.io/s/priceless-sanne-3ccmhg?file=/nuxt.config.ts
Steps to reproduce
Minimal setup with Nuxt3 and the provided Highstock example.
What is Expected?
Example should work.
What is actually happening?

The stockChart does not ever render. The following error is thrown (can be seen in the dev tools console):
Uncaught (in promise) TypeError: SeriesRegistry is undefined
    NuxtJS 5
        <anonymous>
        _registerModule
        <anonymous>
        <anonymous>
        <anonymous>
[indicators-all.src.js:39:22](https://3ccmhg.sse.codesandbox.io/_nuxt/node_modules/nuxt-highcharts/node_modules/highcharts/indicators/indicators-all.src.js?v=ed8cceaf)

It appears the line of code that is causing the issue is here: https://github.com/richardeschloss/nuxt-highcharts/blob/feat/v3/lib/components.js#L34
However I'm not sure if this is a nuxt-highcharts issue or if its a Highcharts issue. I did also try with Highcharts version 9.3.0 (instead of 10.0.0) but that did not fix the problem.
Performance analysis?

Hi, https://github.com/richardeschloss/nuxt-highcharts/tree/feat/v3 contains the working stockChart example (and just re-tested). It's currently tested against Highcharts 9.x. There are plans to make it compatible with Highcharts 10.x (released 18 days ago).
nuxt-highcharts lists "highcharts 9.x" as a dependency, so installing just "nuxt-highcharts@next" should install the compatible version of highcharts. You may have to remove package-lock.json and node_modules before installing to get the dependencies fixed.

https://github.com/richardeschloss/nuxt-highcharts/tree/feat/v3-hc10 pushed and tested against highcharts v10. Highstock works.

I'm not sure what I'm doing wrong, I updated my codesandbox so it only installs nuxt-highcharts@next and doesn't have Highcharts as a dependency, but it's still throwing the same error. Is it a Nuxt3 issue?

According to the console output in that particular codesandbox, there is an unhandled error exception occurring in the pages/index.vue mounted hook. It appears as though, while in the codesandbox environment, there is an error fetching the "ohlcv.json" data resulting in empty series data (and no chart).
From the console output:
"[Vue warn]: Unhandled error during execution of mounted hook
at <Highstock options=
{chart: Object, rangeSelector: Object, title: Object, series: Array(0)}
"
I'm able to run your exact code locally and get it to work. I'm guessing to get it to work right in codesandbox, you may want to try fetching the full url https://3ccmhg.sse.codesandbox.io/ohlcv.json

Hi, im facing the same issue, and im unable to fix the error even using the full url, I can see in the Vue developer extension that the data is correctliny being fetched, but im not sure why its not displaying

@g-tejas can you please share your most recent code sandbox?

I have copy pasted the code from the sandbox provided in the earlier comments, into my local project. Its not working on my local project, with/without the full URL. I think the only different would be the dependencies and how they were installed. May i confirm how i can go about installing nuxt-highcharts properly for the latest 3.0.3 version? I deleted the lockfiles, node_modules, and reinstalled (without explicity stating main highcharts) since you said that highcharts is a dependency of nuxt-highcharts, and it will auto install. I am using nuxt 3.0.0 rc3.

Hi, nuxt 3 rc introduced (breaking changes)[https://github.com/nuxt/framework/discussions/2883]) that this module has not yet accounted for. The main change breaking the code is the naming convention change for the runtime config.  It's on my roadmap to support that version as soon as I can but will be delayed by other financial / work obligations I have. However, I'm not sure how much I should do just yet because the nuxt3 project is iterating much faster (and it's possible that whatever changes I make to support the rc version today will work one day but not another).
For the time being though, nuxt3 at 3.0.0-27365391.7479eac works with this library:

npm i<EMAIL_ADDRESS>
Cloning this git repo and switching the known working branch feat/v3 can also be helpful.

Thank you for your help! I'll try your solution of changing the nuxt version to the one mentioned. Also, i see from the nuxt official roadmap that, stable version of nuxt3 will be out in june, which is not too far away. So i guess that would be the endpoint? Either ways, i really appreciate you doing this, and making this amazing plugin, thanks alot.

My goal is to get the support before they roll out the stable version in June.

Released support for nuxt3rc version:

npm i nuxt-socket-io@nuxt3rc

Tested against highcharts 10.0.0.

Thank you!!  I have a small problem, whenever I install<EMAIL_ADDRESS>its installing highcharts 10.1.0 instead of 10.0.0. Should i just manually specify it in package.json and yarn install it? Also whats the difference. between rc3 and the one shown below (i cloned your branch)

I wasnt able to do it on code sandbox because it kept using nuxt 2 but, i have uploaded it on github,
https://github.com/g-tejas/highcharts-test

 works perfectly, but highstock doesn't. im not sure what im doing wrong, thanks for helping out

Thank you!! I have a small problem, whenever I install<EMAIL_ADDRESS>its installing highcharts 10.1.0 instead of 10.0.0. Should i just manually specify it in package.json and yarn install it? Also whats the difference. between rc3 and the one shown below (i cloned your branch)

Hi, I just pushed 3.0.5. which should clamp the highcharts version to 10.0.0 (there appears to be some bug in the future version with setTicks which causes issues). You can also manually install that version with "npm i<EMAIL_ADDRESS>.
The version of Nuxt being tested against is:
https://github.com/richardeschloss/nuxt-highcharts/blob/c23660525c7a8fb796d6618c012055bc57b0f802/package.json#L41
Which will resolve to whatever the latest "release candidate for Nuxt3" is. So that specific version can change many times a day (and it's possible that those changes can break existing code).

I wasnt able to do it on code sandbox because it kept using nuxt 2 but, i have uploaded it on github, https://github.com/g-tejas/highcharts-test  works perfectly, but highstock doesn't. im not sure what im doing wrong, thanks for helping out

I'm glad to see progress being made! I believe highstock isn't working because that chart expects data to be formatted a certain way (in "ohlc" format):
  data: [
    [ epochTime, open, high, low, close ],
    [ epochTime, open, high, low, close ]
  ]

Having [1,2,3] would not work because it's invalid stock chart data.
https://github.com/richardeschloss/nuxt-highcharts/blob/d17c2cf6a1453a15a46a0f392ab29ab35a021bd9/components/charts/StockChart.vue#L39

Still have no luck sir, I copied the exact code over, but i still get the same cannot read properties of undefined error. Furthermore, this codesandbox was able to plot stock chart with just 1,2, 3 . To replicate this issue, I ran "npx nuxi init test" to create a freshinstall of nuxt project, installed the latest nuxt-highcharts version as u stated. Added the module into the nuxt config. Copied the ohlcv json + Stock chart vue file into my app.vue. Could this be a typescript issue?

Locally, clone this repo, switch to the working branch (feat/v3-rc) and run "npm run dev" and select the stock chart from the app. It will work..
My guess is you may need to uninstall highcharts and re-install the version 10.0.0.

Yes, ive checked the highcharts version, its 10.0.0. Also i can clone and it works, but I still want to add this plugin to my existing project, which i already have the code for, so i would need to do it via the yarn add method instead of just cloning.
Thanks for the help anyway

In another local project, it should still work with npm install and yarn add. I can test on my end too

Unfortunately i wasnt able to, (from a fresh new nuxt project).

Ok, I can repro this on my end, I re-opened the issue and will investigate.

Ok, the feature in #43 seems to responsible for partially breaking things. Importing indicators seems to depend on existing global variables to be defined, so I'll figure out the better place to put that code. In the meantime, a non-ideal workaround is to comment out the indicators and it will work.
Comment out:
https://github.com/richardeschloss/nuxt-highcharts/blob/14dba971c48730f8947bc987edce64e77f2f291f/lib/components.js#L34
and
https://github.com/richardeschloss/nuxt-highcharts/blob/14dba971c48730f8947bc987edce64e77f2f291f/lib/components.js#L36
A better solution should be coming eventually

Ok, I was able to fix it on my end. I release 3.0.6 with what seems to be working code now.  Tested against 10.0.0 . (it's on the roadmap to support highcharts lts)

Thank you, it works now! Really appreciate all the help and quick responses!


ARROW-17518: [CI][Doc][Python] Update glob to detect arrow development version from git

Reproduced locally:
$ git describe --dirty --tags --long --match "apache-arrow-[0-9].*"
apache-arrow-9.0.0.dev-641-g0d5bb92-dirty
$ git describe --dirty --tags --long --match "apache-arrow-[0-9]*.*"
apache-arrow-10.0.0.dev-114-g0d5bb92-dirty

Also, can you please rebase to get a passing AppVeyor build?

As it can be seen on the builds: https://github.com/apache/arrow/runs/8010907934?check_suite_focus=true#step:6:1322
the new version is selected correctly:
ARROW_GIT_DESCRIPTION=apache-arrow-10.0.0.dev-120-g4e1922569
or
pyarrow==10.0.0.dev120+g4e1922569
which was not correct previously.


[1.19.2-Forge] Raw Material and Storage Block tags

Added

Raw Material and Storage Block Tags

Fixed END_ZINC_ORE missing from NEEDS_IRON_TOOL and MINABLE_WITH_PICKAXE tags.
Bumped version with 2 instead of 1.

Updated in current version.  Thanks for highlighting it.


Update data-factory-build-your-first-pipeline-using-editor.md

Should be inputdata instead of scripts as I found when doing this tutorial

Hi @rishigoutam, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@spelluru
fyi

This does not sound right. The HQL file should be in the script folder.

From the Tutorial Overview page: The HQL file should be in the script folder, not the input folder. Please revert the changes if you can. Thanks.
e.Repeat the previous two steps to upload the partitionweblogs.hql file to the script folder of the adfgetstarted container. Here is the command:
Copy
AzCopy /Source:. /Dest:https://.blob.core.windows.net/adfgetstarted/script /DestKey:  /Pattern:partitionweblogs

I reverted with PR https://github.com/Azure/azure-content/pull/5421
It should be in time to take effect. Thanks.


Adjust cbor_types() for unknowable conditions

e..g extern types, custom serialization, etc
not done right now as it impacts both #228 and [TODO: insert WIP @custom_serialize PR when it's pushed]

Possibly break @custom_serialize into two categories: one which respects the CBOR type, and one that is 100% arbitrary.
It would be nice to be able to specify this for __CDDL_CODEGEN_EXTERN_TYPE__ too. Right now we have to provide dummy definitions in a separate dependency folder then delete them after instead of using __CDDL_CODEGEN_EXTERN_TYPE__ when the type is either a basic group OR it is a primitive. We still use it for fully-contained types (i.e. ones that contain their encoding information and are just regular structs).


SubscriptionTerm is missing termUnit property

Describe the bug
Missing property used by the marketplace APIs.
To Reproduce

SubscriptionTerm is missing termUnit of type TermUnitEnum (which exists in models).

Expected behavior

SubscriptionTerm contains termUnit property of type TermUnitEnum.
Getter and setter methods exist for SubscriptionTerm.termUnit property.

Thank you @Tomanow
Fixed pas part of #36


Sometimes duplicate keys appear in useList after database insertion

I've noticed this issue intermittently:
When a new record is added to the realtime database, it shows up multiple times in the list when using useList. When I refresh the page, it will go back to only being shown once. This has happened for multiple different views of useList (a data table, a custom component, etc), and I'm confident it's caused by a single key appearing multiple times in the data snapshot.
This has happened in two different apps of mine, a normal react project and a react native project.
I haven't been able to consistently reproduce this, but it's happened frequently enough that I bet it happens to other users. Has anyone noticed this happening to them?
(This is a very useful library by the way, thank you for creating it!)

I see the same, and also with useListVals.

I have the same problem, do you have any news?

Hi - from my experience so far, it only seems to happen in development environments, not in production.
I haven't had time to look at this repo's source code yet, but as the bug appeared after upgrading to React 18 (I think), my guess it that it has something to do with React running effects twice in development mode now, and that interfering with how RFH works.
It hasn't been a breaking issue for us as, as mentioned, it only appears in dev for me (as far as I can see, at least), but still, it's very annoying.

Seeing it myself with NextJS app. Kind of concerning, makes me think the hooks don't clean themselves properly when unmounting.

I'm seeing this with useListVals in a multiplayer game. When another client adds a player object to a list, the first client sees 5 copies of that player object. Debugging useList, these instances are kept in internal state. This is in developer mode.

+1

+1

even worse, I get triggered twice for some of the keys, first with the old v.val() and then with the old value. So I have to manually ignore the second trigger...

I got the same problem with useListVal in dev mode (firebase emulator).
It happens after inserting entries into, or removing entries from the db.
Programmatically as well as in the emulator UI view.

+1 when using useListVals

Also happens here when using useListVals

I fixed this problem by replacing useEffect cleanup function in useList.ts file to:
return () => {       off(ref)     };

We're also seeing duplicate keys with useList. It's annoying having to check for and remove duplicates. Any ETA on this?

+1

Hello,
after spending hours to understand why it's duplicating, i finally found the reason:
As explained in this thread, https://github.com/facebook/react/issues/16295,
and from documentation here: https://legacy.reactjs.org/docs/strict-mode.html#detecting-unexpected-side-effects, react trigger reducer function twice when using React strict mode (that is the case in development).
As functions addChild, removeChild and changeChild are not pure, there is an unexpected side effect when reducer is called twice.

#298
I added a PR to manage the issue


Generate a local site to host all reference docs

By Cli vela reference to generate reference docs for all workload
types and traits, and host them in a local website.
To fix #880

@wonderflow I've made a quick e2e solution PoC to feature #880. Please take a look at it. I will store capability reference to ConfigMap later. Thanks.
$ vela reference

Open a browser to visit the reference docs site which is hosted locally.

@wonderflow I've made a quick e2e solution PoC to feature #880. Please take a look at it. I will store capability reference to ConfigMap later. Thanks.
$ vela reference

Open a browser to visit the reference docs site which is hosted locally.

My 2c is to show the doc of independent workload type or trait instead of full list.
$ vela show webservice

Then it just pop up this page: https://kubevela.io/#/en/developers/references/workload-types/webservice

My 2c is to show the doc of independent workload type or trait instead of full list.
$ vela show webservice

Then it just pop up this page: https://kubevela.io/#/en/developers/references/workload-types/webservice

My 2c is to show the doc of independent workload type or trait instead of full list.

Sure, I can show the exact doc for a workload type or trait.

$ vela show webservice

Then it just pop up this page: https://kubevela.io/#/en/developers/references/workload-types/webservice

Am I understanding correctly?

for built-in workload type or trait, we should redirect to kubevela.io.
for extended workload type or trait, we can load the responding doc in our local hosted doc site <IP_ADDRESS>:18080.

My 2c is to show the doc of independent workload type or trait instead of full list.

Sure, I can show the exact doc for a workload type or trait.

$ vela show webservice

Then it just pop up this page: https://kubevela.io/#/en/developers/references/workload-types/webservice

Am I understanding correctly?

for built-in workload type or trait, we should redirect to kubevela.io.
for extended workload type or trait, we can load the responding doc in our local hosted doc site <IP_ADDRESS>:18080.

@zzxwill

for built-in workload type or trait, we should redirect to kubevela.io

AFAICT, it is good enough to show the reference docs as the website does, not about showing a link or redirecting via browser.
Currently, the reference docs in the website is auto generated and put into the website files. We can do the same for the CLI/UI as well by saving those data on server side. For the long term, we should let the /definitions API endpoint return the documentation too. For now, the CLI doesn't talk to the API Server yet, so we could probably save it as a ConfigMap on k8s.

@zzxwill

for built-in workload type or trait, we should redirect to kubevela.io

AFAICT, it is good enough to show the reference docs as the website does, not about showing a link or redirecting via browser.
Currently, the reference docs in the website is auto generated and put into the website files. We can do the same for the CLI/UI as well by saving those data on server side. For the long term, we should let the /definitions API endpoint return the documentation too. For now, the CLI doesn't talk to the API Server yet, so we could probably save it as a ConfigMap on k8s.

The general idea to show docs in a generated website is good. But most cli user maybe don't have a GUI browser, what if the user only have a terminal console and hope to see the docs?

The general idea to show docs in a generated website is good. But most cli user maybe don't have a GUI browser, what if the user only have a terminal console and hope to see the docs?

By the way, we're improving the code quality for the whole project, we need unit-tests before merge.

By the way, we're improving the code quality for the whole project, we need unit-tests before merge.

The general idea to show docs in a generated website is good. But most cli user maybe don't have a GUI browser, what if the user only have a terminal console and hope to see the docs?
So I think the cli still need to show a simple workload/trait reference in terminal level.

Let me implement this feature and try to make it as simple and clear as the website does, but it will take more time for the table-style formatting.

The general idea to show docs in a generated website is good. But most cli user maybe don't have a GUI browser, what if the user only have a terminal console and hope to see the docs?
So I think the cli still need to show a simple workload/trait reference in terminal level.

Let me implement this feature and try to make it as simple and clear as the website does, but it will take more time for the table-style formatting.

By the way, we're improving the code quality for the whole project, we need unit-tests before merge.

Of course, working on it. Thanks for the kind reminder.

By the way, we're improving the code quality for the whole project, we need unit-tests before merge.

Of course, working on it. Thanks for the kind reminder.

Seems CodeCov is broken as the data  of the latest commit is not refreshed.

And the detailed coverage statistics could not be visited.

Seems CodeCov is broken as the data  of the latest commit is not refreshed.

And the detailed coverage statistics could not be visited.

The general idea to show docs in a generated website is good. But most cli user maybe don't have a GUI browser, what if the user only have a terminal console and hope to see the docs?
So I think the cli still need to show a simple workload/trait reference in terminal level.

Here is a snapshot to display webservice on console.

The general idea to show docs in a generated website is good. But most cli user maybe don't have a GUI browser, what if the user only have a terminal console and hope to see the docs?
So I think the cli still need to show a simple workload/trait reference in terminal level.

Here is a snapshot to display webservice on console.

Seems CodeCov is broken as the data of the latest commit is not refreshed.

If the unit-test broken, the coverage data doesn't make sense so it won't refresh.

Seems CodeCov is broken as the data of the latest commit is not refreshed.

If the unit-test broken, the coverage data doesn't make sense so it won't refresh.

The general idea to show docs in a generated website is good. But most cli user maybe don't have a GUI browser, what if the user only have a terminal console and hope to see the docs?
So I think the cli still need to show a simple workload/trait reference in terminal level.

Here is a snapshot to display webservice on console.

It will also close #896

The general idea to show docs in a generated website is good. But most cli user maybe don't have a GUI browser, what if the user only have a terminal console and hope to see the docs?
So I think the cli still need to show a simple workload/trait reference in terminal level.

Here is a snapshot to display webservice on console.

It will also close #896


Cleanup internally shared functions and rearrange tests folder

This is just to make things a teeny bit tidier (primarily for myself).

:tada: This PR is included in version 0.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Small Page Size PDF Not Scaling to Fit Screen

Thank you for creating this wonderful package.
I am using PdfDocumentViewBuilder and PdfPageView, but I noticed that a certain PDF file does not scale to fit the screen.
Below is a minimal example.
Is this the intended behavior?

Image

Expected Result
Actual Result

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

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

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

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
        useMaterial3: true,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});

  final String title;

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

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        backgroundColor: Theme.of(context).colorScheme.inversePrimary,
        title: Text(widget.title),
      ),
      body: PdfDocumentViewBuilder.asset(
        'assets/sample2.pdf',
        builder: (context, document) {
          return PdfPageView(
            document: document,
            pageNumber: 1,
          );
        },
      ),
    );
  }
}

flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.22.2, on macOS 14.5 23F79
    darwin-arm64, locale ja-JP)
[✓] Android toolchain - develop for Android devices
    (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2023.3)
[✓] VS Code (version 1.90.0)
[✓] Connected device (5 available)
[✓] Network resources

• No issues found!

Additional Information

The Expected Result uses maximumDpi: 450, in the PdfPageView.
This PDF file was downloaded from here.
This issue has been observed on both Android and iOS.
pdfrx version is 1.0.68

I could not download the PDF anyway...

Ah, I found if the screen size is larger than 300 dpi, the page image may be rendered smaller than the page placeholder :(

1.0.69 contains the fix.

Thank you for your response.
After updating to version 1.0.69, the PDF is now properly scaled to fit the screen.
Thank you for your great work!


add tmio/tmx links

It would be nice to have links to the trackmania.io and trackmania.exchange (if uploaded) pages in the embedded message. Probably just have settings for them

thank you for your suggestion.
done 👍


S2U-34 Improvements to self, peer and grouped rubrics

Thanks for all the good fixed Bernie, let me know when do you want to merge this.

The build is broken for a different reason, I've re-try it.


Epic: Allow field values to depend on multiple other field values

Epic
Want to be able to do things like:
Number 1 > Number 2 and Number 1 < Number 3.
{
  "field": "Open",
  "is": "lessThan",
  "otherField": "High"
}
{
  "field": "Open",
  "is": "greaterThan",
  "otherField": "Close"
}

Not currently supported:
Number 1 = Number 2 - Number 3
{
  "field": "bar",
  "is": "equalTo",
  "otherField": "foo",
  "offsetField": "foobar"
}

Likewise for dates and any other fields that can do single dependencies for.
There is already a story for this at #1240

Closing as a duplicate as now tracking in a miletsone (milestone 6)


JIT error in debug mode

The JIT is throwing an assertion failure in debug mode, and also throwing what seems to be an unrelated error in the Release Static build, even though it works in Release build.

Fixed in #70


Menu Show / Hide Change in Behavior in 0.14.4 vs 0.14.3

Current behavior
I have a top navigation menu system that is all Ariakit.
This works perfectly in 0.14.3, but in .4, I see the menus overlap awkwardly and show multiple at a time. I see that showOnHover is set to true, but they aren't coming down properly in 0.14.4.
Expected behavior
Like it did in 0.14.3
Possible solutions
I think this has something to do with the transition animation, which I set on my mui components to "unset" previously to stop this in earlier versions.

Thanks @diegohaz - Was some new [data-leave] styling / attribute adjusted that might be the clue? I have it nailed dead to rights on the version change from .3 to .4.

Hi @httpete! I reviewed this and other related issues you've had before, and it seems they keep recurring. If you provide a reproduction, we can create an internal example of your use case with tests to ensure this won't break again in the future—or if it does, we'll know exactly why.
Here's an example where we test very specific WordPress use cases in Ariakit to ensure we won't break them: https://github.com/ariakit/ariakit/blob/d955bf36167ff0d9175dd69caac39ae30c9962d3/examples/menu-wordpress-modal/menu.tsx

It's been very solid for some time. It is a recursive N level deep Menu system, it was done before the MenuProvider was in Ariakit, so I will try to refactor to that using the Submenu example as a guide. I am doing what that suggests, iterating over the tree and selectively rendering with the render prop:
    <AriaMenuItem
      render={
        itemHasSubmenuItems ? (
          <Menu menuItem={menuItem} />
        ) : (
          <MenuItem onClick={handleOnClick} />
        )
      }
      ```

I tried raking off all of the render props (which were using Mui ListItem and Mui/Buttons) and it still fails. There is just something going on with the way they come down in .4.  I have a simple case where I loop and call menu.hide() on them and they just come down slowly and overlap.

I produced this codesandbox, with example code right from yours, and it overlaps exactly the same way.
Hover quickly over "Find" And "Speech" at the link below.
@benrodrs
https://codesandbox.io/p/sandbox/happy-meninsky-5vvt8v

@benrodrs  - Sorry to nudge but this is really going to break a lot of people and I want to raise it to your attention. My proof is solid, it is introduced in this release.

@diegohaz Is this the same issue? Downgrading to 0.14.3 fixes it.
If not, please LMK and I'll open a separate issue to track this bug.

@bengry It's the same issue. I'm working on a fix.

If you want to upgrade and can't wait for the next release, you can work around this issue by applying this onMouseLeave prop to the *Anchor or MenuItem component that triggers the hovercard or submenu.
  // TODO: Remove when https://github.com/ariakit/ariakit/issues/4247 is fixed
  onMouseLeave={(event) => {
    const view = event.currentTarget.ownerDocument.defaultView;
    const nextEvent = new MouseEvent("mouseleave", {...event, view });
    event.target.dispatchEvent(nextEvent);
  }}

I have to re-open this bug. It is still there in 0.4.15.

@benrodrs It is still there. I had the old version cached.


Unit test for Issue 70

Adding unit test to check that issue https://github.com/klarna/HiveRunner/issues/70 doesn't happen anymore.

Fixes #70


Load more than 25 documents

How do you get it to load more than 25 documents in a batch? It seems to process that and then shuts down.  Any way to tell it to process x number?
Still getting errors paring json response from LLM
Jun 11 18:46:06.877 ERRO Error parsing llm response json expected , or } at line 6 column 18
Jun 11 18:46:48.638 ERRO Error parsing llm response json key must be a string at line 4 column 1
Jun 11 18:48:41.691 ERRO Error parsing llm response json key must be a string at line 7 column 32
bjoern5urban/doclytics:v1.2.0-rc.1

Hi @madman2012, sorry for the late response. Can you elaborate on the problem a bit ? Does the whole application run never process more than 25 documents ?

Hi thanks for coming back.  It’s it only requests and processes 25 documents then shuts down.

From: b-urb @.>
Sent: Tuesday, August 13, 2024 2:37:12 PM
To: B-urb/doclytics @.>
Cc: madman2012 @.>; Mention @.>
Subject: Re: [B-urb/doclytics] Load more than 25 documents (Issue #55)
Hi @madman2012https://github.com/madman2012, sorry for the late response. Can you elaborate on the problem a bit ? Does the whole application run never process more than 25 documents ?
—
Reply to this email directly, view it on GitHubhttps://github.com/B-urb/doclytics/issues/55#issuecomment-2286881693, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABMS5U6TZNCKTCINOIEBXU3ZRJG5RAVCNFSM6AAAAABJE5DCJKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBWHA4DCNRZGM.
You are receiving this because you were mentioned.Message ID: @.***>

Thanks for pointing this out, indeed paperless itself seems to have a pagination with a limit of 25. This stems from the underlying django api framework which is why(unfortunately) it is not documented withing paperless itself. I created a fix and will try to merge it asap.


Commissioner/Joiner Router port usage is incorrect in Joiner Authentication

According to the the 1.1 spec, during the Joiner Authentication process, the Joiner Router should send to the Border Agent from an ephemeral port to port MM (61631), and should receive replies from the Border Agent on this ephemeral port.
joiner_router.cpp is not listening on this ephemeral port after sending the RLY_RX. Further, the Commissioner is then processing the DTLS-ClientHello and sending the RLY_TX to a hard-coded port (MM, 61631), where the Joiner Router is listening (which keeps things working in OpenThread, but is incompatible with the spec).
So there are two parts to this issue:

The joiner router should be listening/expecting the RLY_TX reply on whatever port it sends from rather than only on 61631. My (and I assume other 3rd party) off mesh border agent/commissioner which don't send to port 61631 end up going through the generic HandleUdpReceive path which breaks the joiner authentication.
The commissioner/border agent (these two roles seem to be treated the same in the codebase) should not be sending to a hard coded port 61631. A joiner router that is waiting for the RLY_TX on the port it sent from would not expect this port to be used for the reply, so OpenThread would be incompatible with third party Joiner Routers.

Thread 1.1 Section <IP_ADDRESS> indicates that the RLY_TX.ntf message from Border Agent to Joiner Router is sent to the MM port.  Based on Thread 1.1 Section 10.3.1, I believe the intent is for all CoAP Request messages to have a well-known destination port.  Can you indicate where the spec indicates that RLY_TX.ntf messages are sent to the ephemeral port?  Maybe you found an inconsistency in the spec.
Also, OpenThread does not currently support the Thread Border Agent (we are currently working on this feature).  The existing Commissioner role is the "On-Mesh Commissioner" role.

According to Figure 8-5 (page 223 in Thread 1.1) the RLY_TX should from the MM port, but be sent to UDP destination port: Eph(3). This seems to indicate the Spec has contradicting information.
The reason Jorge encountered this is because he is adding the Windows Border Agent code.

I hadn't looked at <IP_ADDRESS> that you pointed out. Another figure that I hadn't examined too closely was <IP_ADDRESS>.1, which agrees with the current implementation. So we have 2 sources that agree with the current implementation and that disagrees.

Yes, I agree that Chapter 8 is inconsistent regarding this.  Please file a SPEC JIRA so that the Thread Group's Errata Committee can work to resolve this.

@jwhui I'm not familiar with that process, where would I file this?

The Thread Group JIRA portal is at https://threadgroup.atlassian.net/
Let me know if you'd like me to file the JIRA on your behalf.

@jwhui I don't have access to that group, could you please file it? Thanks.

I have created SPEC-701.  Thanks for finding this spec issue!


Clock: Store time and timezone in AsyncLocal<> instead of ThreadStatic.

Make Clock also work with async/await.
Solves #303

@Sjaaky What happens now if multiple tests run in parallel with different using blocks?


Example code function "logMessageText" warnings

Hi team,
I'm moving this from here: https://github.com/MicrosoftDocs/bot-docs.de-DE/issues/3
@chrismade commented 2 days ago
Can someone pls have a look on the example code function "logMessageText" on page:
https://docs.microsoft.com/en-us/azure/bot-service/bot-builder-howto-v4-storage?tabs=javascript&view=azure-bot-service-4.0
when following the instructions:
build an echo bot using botbuilder:
yo botbuilder -N "botlog02" -D "test options to log conversations" -T echo
test --> everything is as expected when running locally on bot emulator 4.4.0
next copy and paste "bot.js" from its first appearance at the very top of the page and overwrite the existing "bot.js" - then testing again, you will see this warning (which says it will become an breaking error in the future so I guess this should be fixed):
(node:14516) UnhandledPromiseRejectionWarning: TypeError: Cannot perform 'set' on a proxy that has been revoked
    at adapter.sendActivities.then (D:\daten\bots\botlog02\botlog-02\node_modules\botbuilder-core\lib\turnContext.js:176:36)
    at process._tickCallback (internal/process/next_tick.js:68:7)
(node:14516) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a
catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:14516) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

When you comment out the call of function "logMessageText" in "this.onMessage" then no warning appears -
This issue appears even before you proceed and start working with CosmosDB or blob storage - and it will not go away with any of these enhancements.
Usually I don't care much on warnings in example code as it is usually a trade off to make code lean to see what matters - and to catch all imaginable scenarios which obviously make the code more complex.
What worries me even more is that I cannot fix it myself - I can comment out all "turnContext.sendActivity" inside the function which makes the warning(s) go away - but I also would like to understand why the warning comes up at all - Good news: The "turnContext.sendActivity" inside the function are not essential - it is rather for a better understanding to info the user what is going on - so if there is not fix for this issue at least make a box into the doc page that users should ignore the warning and remove all "turnContext.sendActivity" before they move towards production
my node version is:
node --version
v10.14.0

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

ID: 7538ed7a-5bb8-abc0-23e7-5b118e2fd47f
Version Independent ID: f6181862-6527-6698-afcd-4f6d240123da
Content: Write directly to storage - Bot Service
Content Source: articles/v4sdk/bot-builder-howto-v4-storage.md
Service: bot-service
Sub-service: sdk
GitHub Login: @DeniseMak
Microsoft Alias: v-demak

I've labeled this as a v4 enhancement for our dev team to consider. Thanks!

Hi Cecilia,
Thanks for your feedback. I am currently looking into this issue and will get back to you soon.
Best,
Yan

Hi @chrismade
Thanks for your feedback after I checked the old issue you opened here.
I'd like to double check my understanding of the issue you reported here before we could proceed for a solution:
You followed this article

You built an echo bot using botbuilder:
yo botbuilder -N "botlog02" -D "test options to log conversations" -T echo
You tested the echo bot and things worked.

Then you copy and paste the "bot.js" file to some other location and you overwrote the existing "bot.js" file (not the one you copied and pasted).  Things didn't work.

Please let me know if I have understood this part of your question correctly.
Best,
Yan

Hi @chrismade FYI
I also explored a bit on this article and tested the bot. I did not come to the point of using Cosmos DB.
This is what I got so far:

I checked out a JavaScript echo bot from the samples repo.

I overwrote my bot.js file following instructions here. I double checked the code before I tested the bot.

I tested the bot in Emulator and got this:

@zxyanliu thx for picking this up - I confirm your summary above is correct #1 you create a simple bot using "yo" and then #2 overwrite bot.js with code from the Microsoft example - you can still compile the code and get something useful / working - however, in console window (where you start 'node index.js' ) you will see warnings - which are not easy to fix - example code people use to learn from should not have warnings

Hi -
Thanks for your reply. May I ask which Microsoft example did you follow when overwrite your bot.js file?
Thanks.
Yan

Hi @chrismade
I also have some confusion here:
What did you mean when you said "get something useful / working", could you please specify?
Did you mean you got your bot running and got some warnings in the console as well?
Thanks.
Yan

Hi @zxyanliu - it was the first or second example code on that page - if I remember correctly it started with using memorystorage and then explained how to replace it then with cosmosDB and blobstorage - with much more complicated code - it was definitely one of the memorystorage examples - and yes, the code ran and showed the expected behaviour but also showed a warning in the console - and especially this one is my concern:
[DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
because it is indicating that the code will no longer run in future - and I'm still unsure where the unhandled promise rejection is - pls also have a look where the issue started originally on the german page where I filed the issue originally already in english language - before it was closed there and copied into the english section.

Hi @chrismade
Thanks for letting me know the example code on the article page.
I have also read the issue you filed on the German page and I totally understand the concern you have when you see such an error.
I will do my best to run through the process again tomorrow to see if I will see similar warnings and get back to you with any updates.
Best,
Yan

Hi @chrismade
I followed what you did:

Create a JS bot using Yeoman
The echo bot runs and tested with success.
I overwrote the bot.js file followed this tutorial
Yes, I got the same type of warning.

We need further support to see if this warning will have any effect that @chrismade have mentioned.
Thanks.

Simply editing lines 41, 43, 47, 56, 58, 63 to be await turnContext.sendActivity(...) makes the warnings go away :)

Happy Console

Thanks! @Zerryth
I tried and it worked! No more warnings!
@chrismade Please feel free to try what @Zerryth has recommended. This is what I got:

Tested in Emulator.

thanks, so simple  - then please fix the examples in the docu - then close this case as solved - thank you so much for your support

Hi -
@chrismade I have created PR 1164 to fix the bugs. You will be able to see the updates in a day or two in the article and then I will close the issue.
Thanks again for your feedback to help us improve our docs. We really appreciate it.
Vielen Dank!
Yan

Hi - @chrismade
The issue was resolved and content has been updated and live now Write directly to storage.
Therefore closing this issue.
Thanks for your feedback, we really appreciate it.
Yan


Dyld Symbol not found

Receiving the error below after a successful install. Not sure if it's down to my system config but thought I would share with you to see if you had any ideas. Not sure what other information you might need!

macOS 10.10
mono  v <IP_ADDRESS>

~|⇒  brew cask reinstall expresso
Warning: Calling brew cask reinstall is deprecated! Use brew reinstall instead.
==> Downloading https://github.com/sttz/expresso/releases/download/1.0.2/expresso-1.0.2.zip
Already downloaded: /Users/MMS/Library/Caches/Homebrew/downloads/ec49d826036c4efe760072f3fa0f9a563f24fa0fd51cf1ba4785e7096e681618--expresso-1.0.2.zip
==> Verifying SHA-256 checksum for Cask 'expresso'.
==> Uninstalling Cask expresso
==> Unlinking Binary '/usr/local/bin/expresso'.
==> Purging files for version 1.0.2 of Cask expresso
==> Installing Cask expresso
==> Linking Binary 'expresso' to '/usr/local/bin/expresso'.
🍺  expresso was successfully installed!

~|⇒  expresso
dyld: Symbol not found: _kSecKeyAlgorithmRSAEncryptionRaw
  Referenced from: /usr/local/bin/expresso
  Expected in: /System/Library/Frameworks/Security.framework/Versions/A/Security
 in /usr/local/bin/expresso
[1]    6306 trace trap  expresso

Hey, you're trying to use the pre-built binary that is built using CoreRT. CoreRT is experimental and they don't publish any official system requirements but the missing symbol was only added in macOS 10.12. So I suspect your version of macOS 10.10 is not supported by CoreRT.
I suggest you try to install from source using the formula instead. It uses mono, which should support macOS 10.9+ but I am not set up to test on macOS 10.10 to actually verify.
brew cask install sttz/tap/expresso

Great thank you - everything working well now!


Auto-indentation on ENTER only works in { braced } blocks

Environment data
VS Code version: 1.79.2
C# Extension version: 1.25.7
⚠ Actually in this extension which is a fork of the official extension; but presumably it happens here too (please confirm).
OmniSharp using mono: 6.12.0
Dotnet Information
.NET SDK (reflecting any global.json):
 Version:   6.0.119
 Commit:    044cde2ce0

Runtime Environment:
 OS Name:     ubuntu
 OS Version:  23.04
 OS Platform: Linux
 RID:         ubuntu.23.04-x64
 Base Path:   /usr/lib/dotnet/sdk/6.0.119/

global.json file:
  Not found

Host:
  Version:      6.0.19
  Architecture: x64
  Commit:       e37fab9fc9

.NET SDKs installed:
  6.0.119 [/usr/lib/dotnet/sdk]

.NET runtimes installed:
  Microsoft.AspNetCore.App 6.0.19 [/usr/lib/dotnet/shared/Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 6.0.19 [/usr/lib/dotnet/shared/Microsoft.NETCore.App]

Settings

"C# formatter" setting was disabled, but enabling it (and restarting the IDE) didn't make any difference. It seems the formatter works only when I explicitly call it, not when typing ENTER.
"Format on type" setting is enabled, but "format on save" is not.

Steps to reproduce

Make sure "editor.autoIndent": "full" is set for C#
Optionally create .editorconfig with:

root = true
[*.cs]
indent_style = space
indent_size = 2
csharp_indent_case_contents = true
csharp_indent_switch_labels = false

Create A.cs file on VSCode
Paste the code below and press ENTER after each code line to check how indentation is bad when you don't use braces. It's going to where I put the comments.

class A {
  // CORRECT: Auto-indented here
  internal void f() =>
  // BAD: Should probably increase indentation here
    f(false);
    // WRONG: Did not decrease indentation
  void f(bool b) {
    // CORRECT: Auto-indented here
    if (b) {
      // CORRECT: Auto-indented here
    }
    if (b)
    // WRONG: Did not increase indentation
      f(!b);
      // WRONG: Did not decrease indentation
    else
    // WRONG: Did not increase indentation
      f(b);
      // WRONG: Did not decrease indentation
    for (; b; b = !b)
    // WRONG: Did not increase indentation
      f(b);
      // WRONG: Did not decrease indentation
    while (b)
    // WRONG: Did not increase indentation
      f(b = !b);
      // WRONG: Did not decrease indentation
    switch (b.GetHashCode()) {
      // ACCEPTABLE: But I have `csharp_indent_switch_labels = false` in .editorconfig
      case 0:
      // ACCEPTABLE: But I have `csharp_indent_case_contents = true` in .editorconfig
      break;
      // CORRECT: Kept indentation here
      case 1:
        break;
        // BAD: Ideally, smart analysis could revert to label indentation here
    case 2:
    // WRONG: Did not increase indentation
    return;
    // CORRECT: Kept indentation here
    default:
      return;
      // BAD: Ideally, smart analysis could revert to label indentation here
    }
  }
}

Expected behavior
It should auto-add indentation properly when you press ENTER, as described in the code above.
Actual behavior
Indentation is as described above, which gives you a really bad coding experience (specially if you indent with spaces).
Additional context
Note that when you hover the folding regions, only the braces generate folding. There is no folding for non-braced statements like if, for, etc. This could be related to why the editor is not indenting correctly. But it's also happening on the function with expression body despite it having folding, so maybe not.
I don't know if this should be implemented through language defined brackets, onEnterRules or indentationRules. In any case, if you want to do a "smart" analysis, you could probably improve indentation even inside a switch block, so after a "top-level" break, continue, goto or return statement, it would revert to label indentation.

@arkalyanms The VS Code team is not interested in discussing this matter with us.

Maybe this could be brought up at the LSP meeting since we were previously requesting a new LSP API. This has come up a few times before and it seemed to be contingent on https://github.com/microsoft/vscode/issues/13960 being implemented.

Maybe this could be brought up at the LSP meeting since we were previously requesting a new LSP API. This has come up a few times before and it seemed to be contingent on microsoft/vscode#13960 being implemented.

Yes thats my plan too.

it seemed to be contingent on virtual space being implemented.

Sorry if I misunderstood, but I believe this has no dependency on virtual space.
Other languages like JS/TS currently already do this just fine in VSCode - the indentation goes where it should go regardless of braces - though it's not perfect (e.g., it doesn't unindent after the unbraced statement line, even with ;).

@geekley To be fair to the C# extension, your scenarios fail in ts/js with FormatOnType disabled. Which makes sense because indentation is currently determined based on open brackets and they have a similar set of brackets.

JS/TS example
class A {
  // CORRECT: Auto-indented here
  l = () =>
    // BAD: Should probably increase indentation here
    f(false);
  // WRONG: Did not decrease indentation
  f(arg) {
    // CORRECT: Auto-indented here
    if (b) {
      // CORRECT: Auto-indented here
    }
    if (b)
      // WRONG: Did not increase indentation
      this.f(!b);
    // WRONG: Did not decrease indentation
    else
      // WRONG: Did not increase indentation
      this.f(b);
    // WRONG: Did not decrease indentation
    for (; b; b = !b)
      // WRONG: Did not increase indentation
      this.f(b);
    // WRONG: Did not decrease indentation
    while (b)
      // WRONG: Did not increase indentation
      this.f(b = !b);
    // WRONG: Did not decrease indentation
  }
}

Ah, actually I see that @geekly had FormatOnType enabled. JS/TS does much better in that scenario. O# doesn't perform any formatting on Enter. Which brings me to this old PR which is where I remembered the discussion around virtual space.

Yes, I just tested here and you're right. Interesting. So that behavior is provided by a formatter? JS/TS completely ignores editor.autoIndent setting, it seems that it does indentation "right" even if you set it to keep or none.
But I believe VSCode extensions can still provide this independently of formatOnType, right? Otherwise there would be no point to the autoIndent setting. In any case, my main point is that this feature is orthogonal to virtual space availability.


Not a responsive designed site?

Is it just me, or is this not a responsive designed site?   When I view it in Chrome in Responsive mode it is not able to display the pizza choices properly and the top nav does not fit.

Does it even matter? This is an example app . No one is going to test how it looks on various devices or if pizza can have nails as toppings , thats not the point.

Yes it matters.  I have watched the presentations and heard numerous times in the presentations "Modern web applications for modern browsers".   Guess what, a modern web app is responsive.   And guess what, I checked.   So I would classify this as not a modern web app for modern browsers and question whether blazor can do it or not.   This is an example app of what can be done with Blazor is it not?   If it's not an example of what can be done with Blazor than what is it?   Isn't Blazor supposed to be the next Client Side Framework for the future?     This is one of the first things folks looking into Blazor are going to experience and they and I are having an experience that doesn't support a responsive design.  Just little old me thinks this is very short sighted to put out a demo for modern web browsers and modern client side development, using the latest and greatest framework that is not responsive.

Cool. Fix css , do pull request.  It's open source after all.

I won't consider this an "Issue" but a "nice to have"

Idk but maybe you are mixing some different concepts here. Blazor still are a framework for "Modern web applications for modern browsers" this was not to do with css and responsiveness.  As @RafaDom pointed this is not an issue

CSS changes required for responsiveness have absolutely nothing to do with blazor.

I get all that.
But think of the C-levels that one might be presenting the latest sample framework project to, and they ask, is it responsive, because all our sites are today.
And I have to say, “this sample is not, but trust me, it’s just some CSS”….
They won’t get it.
What I don’t get, is why anyone or anyone group, would make the decision to put a sample together, and not take the effort to make it responsive….I can’t connect the dots on that…
Anyway…moving on….
From: Menasheh Peromsik<EMAIL_ADDRESS>Sent: Thursday, July 25, 2019 10:28 AM
To: dotnet-presentations/blazor-workshop<EMAIL_ADDRESS>Cc: Bernie Quick<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [dotnet-presentations/blazor-workshop] Not a responsive designed site? (#98)
CSS changes required for responsiveness have absolutely nothing to do with blazor.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/dotnet-presentations/blazor-workshop/issues/98?email_source=notifications&email_token=AD6YAJLVVXTSYLAXBNHVP2LQBHA7BA5CNFSM4HYPCAS2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2Z2RZQ#issuecomment-515090662, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AD6YAJOCNMPJCV57AHUHKULQBHA7BANCNFSM4HYPCASQ.
CONFIDENTIAL: The information contained in this email communication is confidential information intended only for the use of the addressee. Unauthorized use, disclosure or copying of this communication is strictly prohibited and may be unlawful. If you have received this communication in error, please notify us immediately by return email and destroy all copies of this communication, including all attachments.

@stevesandersonms Thoughts on this?

It wasn't a design goal, but if someone wants to contribute an improvement, and they can do so in a sufficiently clear, simple, and high-quality way, that would be great.


s3 Scaleway backup

How to setup s3 Scaleway backup? Is it possible?
I try
backupURL: https://<<my bucket name>>.s3.fr-par.scw.cloud
  backupCredentials:
    AWS_ACCESS_KEY_ID: SCW*******
    AWS_SECRET_ACCESS_KEY: ************
    AWS_REGION: fr-par
    AWS_ACL: private
    AWS_STORAGE_CLASS: standard
    S3_ENDPOINT: s3.fr-par.scw.cloud
    S3_PROVIDER: Scaleway

This did not work
Create Google Drive service-account.json file.
Create rclone.conf file.


AttributeError: 'numpy.bool_' object has no attribute 'variables'

I am new to cvxpy and modelling the optimization problem. I followed the basic example "control" and developed my coding example but there's an error. Please give me answer to figure out this problem..
Traceback (most recent call last):
File "C:\Users\soyeon\Desktop\2018 Fall\test1006.py", line 111, in
prob = cvx.Problem(cvx.Minimize(objective), constrlist)
File "C:\Users\soyeon\Miniconda3\lib\site-packages\cvxpy\problems\problem.py", line 78, in init
self.size_metrics = SizeMetrics(self)
File "C:\Users\soyeon\Miniconda3\lib\site-packages\cvxpy\problems\problem.py", line 672, in init
for var in problem.variables():
File "C:\Users\soyeon\Miniconda3\lib\site-packages\cvxpy\problems\problem.py", line 145, in variables
vars += constr.variables()
AttributeError: 'numpy.bool_' object has no attribute 'variables'

The stack trace makes it seem like the issue is with your code, not CVXPY. In particular, it indicates that one of your constraints in constrlist is a NumPy object. As such, I’m closing this issue.
Feel free to post this question (along with your source code) on StackOverflow. If you tage the question with “cvxpy”, you should receive a response.


URL in description and README broken

The URL in the repository description and README are pointing to gh-dev-bchealth's copy of the game instead of yours.
Please fix both, so they point to your copy of the game at https://gh-dev-bchealth.github.io/github-games-Jacob-warnes.

Updated urls


Fix research overview while research in progress

This fixes the bug i mentioned in PR #11

I just noticed there are some changes missing on my fork, i merged them in my branch.


consider queuing earliest available set in event?

to pick up sets in earlier rounds that were delayed for various reasons


Rename directorClass to directorType

Spec: https://material-motion.gitbooks.io/material-motion-starmap/content/specifications/interactions/transitions/platform/ios/TransitionController.html

🎊 My diff is ready for review at http://codereview.cc/D1786

No longer planning to do this.


promql Go import problem: use of vendored package not allowed

What did you do?
I've tried to import the github.com/prometheus/common/model package so I can construct a promql.AlertStmt from scratch. Here is the demo code that I'm using:
package main

import (
	"github.com/prometheus/common/model"
	"github.com/prometheus/prometheus/promql"
)

func main() {
	_ = &promql.AlertStmt{
		Annotations: model.LabelSet{},
	}
}

Building this package is not possible because promql uses a vendored package and thus I've got this error:
./demo.go:10: cannot use "github.com/prometheus/common/model".LabelSet literal (type "github.com/prometheus/common/model".LabelSet) as type "github.com/prometheus/prometheus/vendor/github.com/prometheus/common/model".LabelSet in field value

This is ok, because they really are two different packages (one is vendored, and the other one is pulled by me and put into GOPATH), but if you go and use the vendored import path:
package main

import (
	"github.com/prometheus/prometheus/promql"
	"github.com/prometheus/prometheus/vendor/github.com/prometheus/common/model"
)

func main() {
	_ = &promql.AlertStmt{
		Annotations: model.LabelSet{},
	}
}

Now you get the error:
package demo
	imports github.com/prometheus/prometheus/vendor/github.com/prometheus/common/model: use of vendored package not allowed

What did you expect to see?
I want to be able to create a custom promql.AlertStmt with the Annotations field
What did you see instead? Under which circumstances?
As described above, it's not possible to create such a value
Environment
go version go1.6 darwin/amd64

System information:

Darwin Fatihs-MacBook-Pro.local 15.5.0 x86_64

Prometheus version:

I'm using latest master with Git SHA a08943e6d347119bfb1b50e738fca577e5733290

A local solution for me would be moving the vendor folder content to GOPATH and remove the vendor  folder.
Also here is a conversation I had with dave cheney https://twitter.com/fatih/status/740883554264096772

The problem here is that you are using the promql package as a library, which it was not intended to so far. Or in other words: We vendor in the prometheus/prometheus repo because it is not considered to contain libraries (in contrast to prometheus/common).
More and more people are using parts of prometheus/prometheus as libraries, though.
We have to look into how to make that easier.

Yes it is intended to be used as a library. I fought hard to design it as one and it is in wide use by several companies.
It is arguably a design flaw in the vendoring mechanism and as far as I know there are vendoring tools handling thise case by stripping vendoring from packages you vendor yourself.

Wouldn't promql then be a candidate to move into it's own repo? Or into prometheus/common? It's not easy for externals to spot which part of an application repo are meant as libraries and which not.
WRT to nested vendoring: Sometimes, nested vendoring is desired (for any identifiers that do not show up in the API of the library and are therefore completely contained). It would be hard for a vendoring tool to distinguish between the two cases.

PromQL is pretty tightly bound to Prometheus, and we're not willing to offer any guarantees around API stability, so I think where it is is fine.

I think the better solution would be move out promql into a separate repo and then vendor it. That way everyone can make use of it. I'm not sure though what the better solution would be, as @brian-brazil mentioned just looking at the code today, they indeed a very tightly bound.
But just a side note, the way promql was laid out, with it's documentation and co, It was intended from the beginning as a library as @fabxc mentioned. It's a joy to use it and so far I'm really happy with it. We use it here at DigitalOcean for our internal systems and was quite happy to find that we could use the parser and manipulate the AST.
That's why I think the stability should be guaranteed

That's why I think the stability should be guaranteed

That's a completely separate discussion.

@brian-brazil well you said it's not guaranteed and I said it should. Not sure why you wrote this comment. Happy to discuss it in another issue.

So yes, nothing is wrong with the vendoring in prometheus/prometheus. We could close this issue and open a new one titled "Move promql into a different repository?", or we could simply rename this issue and have the discussion here (as it has already happened anyway).

There's no such thing as "meant as library" if we design each package in a manner that it exposes a well-designed API for the behavior it aims to implement.
Locality of packages is merely to simplify making modifications as we need them ourselves (i.e. no constant revendoring) and to indicate to potential users the API instability that comes with it.
I'm not comfortable moving PromQL out at this point. It has an API that makes it useful for external use cases but is not there yet for a stable package.
I think we have to count on vendor tooling here to work around the general issue.

OK, vendoring declared as "still unsolved", and this issue declared closed. :)

Thanks for the clarification. I've created a custom script to extract the promql package into a new import path which can be used by any third party package for a given prometheus version. Until this is solved by the team itself I'm going to use it.

@beorn7
In order to host both a reproducible cmd build and have packages that can be imported elsewhere, I see two options in general for packages and repos to take in Go:

Move the vendor folder in the cmd/ folder. Then after updating a package, either periodically, or in a pre-commit hook or similar, run cd cmd && govendor update github.com/prometheus/prometheus/... which will copy over the package files into the vendor folder makeing a clean cmd build.
Have packages that import the prometheus use a tool like govendor that will also flatten the vendor imports out.
(Not recommended) You could also add a line vendor/*/ in the .gitignore file (and not check-in the dependencies) and add a post update hook in git to run govendor sync to copy the dependencies in to the vendor folder.

I'm working on having other options for the future, but that's about where we are right now. One of the "benefits" of option (1) is you could decouple your cmd release from your importable packages schedule.

Move the vendor folder in the cmd/ folder. Then after updating a package, either periodically, or in a pre-commit hook or similar, run cd cmd && govendor update github.com/prometheus/prometheus/... which will copy over the package files into the vendor folder makeing a clean cmd build.

This is essentially "self-vendoring" and would require the constant re-vendoring @fabxc mentioned before. It's equivalent to moving the cmd directory into its own repo, where then everything, including prometheus/prometheus, is vendored. It has the advantage of not requiring one repo per package, but it has a great potential of confusion.

Have packages that import the prometheus use a tool like govendor that will also flatten the vendor imports out.

That's what @fabxc meant by "I think we have to count on vendor tooling here to work around the general issue."


type int problematic for size, start and end indicators

Large strings produce weird results when the range of int is exceeded
due to undefined behaviour regarding integer overflow.
Could you either change the type to unsigned int or make
the type configurable (a la
#!c
#ifndef INDEX_TYPE
#define INDEX_TYPE int
#endif

typedef struct {
	jsmntype_t type;
	INDEX_TYPE start;
	INDEX_TYPE end;
	INDEX_TYPE size;
#ifdef JSMN_PARENT_LINKS
	INDEX_TYPE parent;
#endif
} jsmntok_t;

Bitbucket: https://bitbucket.org/zserge/jsmn/issue/44
Originally reported by: Sebastian Freundt
Originally created at: 2015-03-13T15:39:19.116

I don't think unsigned int is an option, because we need a special value for "undefined", which is now -1.
I wonder if there is any practical platform where this problem matters?
E.g. on most 32-bit systems int is 2^32, which should work fine for JSONs less than 2GB.
On smaller systems like AVRs int is 16 bit addressing up to 32KB strings, which is still a lot for a tiny 8-bit CPU with a few kilobytes of RAM.

You could make it an ssize_t. It's signed, so -1 still works, but it has a much larger range on many systems.

First of all, this is an awesome little library, thanks Zserge.
I also don't care much for non ascii strings, and it is obvious that this library wouldn't help with that, so that's really not a problem.
But having a signed overflow problem is does not seem like an obvious thing while looking at the documentation, or the code. I would even say it is misleading. If the code can ever fail, that is a bug, period. According to Murphy's law of software development, if it can ever fail, it will fail. If it is ever going to be a security problem, and people see it was known and ignored - those people are going to be angry. Please, at least state it in documentation.
By the way, I don't see a problem with using size_t instead of int. The special value can be SIZE_MAX instead of -1  -- not that SIZE_MAX definitely can not be a valid index in any array. The largest theoretical valid index is SIZE_MAX-1 . The current state of things is pretty ugly: size_t len argument used in json_parse, unsigned int pos used for indexing, signed int used for returning results. This is very incosistent, ugly. Please consider it, it would not take away any of the simplicity.
Also, joeljk13: ssize_t is a type in POSIX C, while size_t and SIZE_MAX are available in freestanding ISO C as well -- thus pretty much everywhere.

I just tried what happens, if json_pares is given a too-large string, an output from a test program:
token[0] start: 0 end: -2147482627 size: 2
token[1] start: 2 end: 6 size: 1
token[2] start: 9 end: -2147482646 size: 0
token[3] start: -2147482642 end: -2147482637 size: 1
token[4] start: -2147482634 end: -2147482629 size: 0
Now, as a simple hypothetical example, image a software, that just reads data from a network socket, tries to json_parse that data using jsmn. An attacker somehow manages to feed over 2 gigabytes of data, so the attacker can creates a token that starts at index -2 billion. While trying to access this token, the said software segfaults. I know, this is not something that is going to happen a lot, it's just the first thing I can come up with in 10 minutes.

Simple solution: Don't allow ridiculously large input?
If you are reading data from a socket and the client is giving you 2+Gb of data... that connection needs to close();
The maximum size of any input is limited by the maximum index we can hold in token.start/end
+1 for adding this to the documentation.
+1 for using unsigned and say (unsigned)-1 for undefined (solves the problem of negative index)
Having the size configurable is a plus, but even using size_t does not solve the problem; theoretically even size_t will eventually wrap.
On the other hand, since the input buffer is held in memory in it's entirety the system will run out of memory first.


Adds a PIP exceeded alarm and wires it with basic GUI visuals

This is more progress towards #366 .
It is wired with the color of the pressure graph and background of the PIP measurement.
To play with this, launch the GUI and set the commanded PIP to a few units less than the measured one, they'll highlight in red.
Next step (tomorrow) will be to wire with @paulovap 's alarm banner PR.

Thanks Justin. @paulovap - I'm gonna merge this for now, but please take a look; if you have any comments, I'm happy to address them tomorrow.


Add draft Smart Answer with all question types

Trello card
This is not intended to ever be live, it was specifically done to make it easier to work on all the front-end elements. Therefore no tests are necessary.
I would also like to add simple error validation to all elements. That might be added in a separate PR.
I added four different date questions, because it is very likely we will have more than one type of date question in the future.
Expected changes
None on production. On other environments a new Smart Answer gets added under /all-smart-answer-questions.

A quick thought: it might be worth adding a link to each question page from the recently added index page which is only available in development.

I'm not sure if the index page would be a good place, it'd be muddling the purposes a bit. I wonder if it makes sense to link all the questions on the landing page for this flow?
Then again, all the pages are already linked from the results page via the "Previous answers" section. I could add a link to that on the landing page? E.g. /all-smart-answer-questions/y/mithrandir,olorin,tharkun/mordor/2016-07-27/1970-01-01/2020-02-29/0000-12-25/9999999.0/yes/WC2B%206NH/500000.0-month/world/0/0.5

FYI, I just made a change and force-pushed. The change is adding a second multiple choice question (which_boolean_choice) as a Yes/No question will look different to others, and changing the previous which_choice question to not be a Yes/No/Maybe question.

This looks good to me. 👍

Thanks, I just rebased with origin/master and also added a link to the results on the landing page to make it easy to navigate to all the different questions individually.