Command palette should return the "File: Compare New Untitled Text Files" command when I search for "diff" (fix #208897)

@bpasero is this for April or March milestone ?

@deepak1556 apri


Fix to RichTextEffect visibility - Label Now Accounts for Skipped Characters.

A picture is easier to describe this issue than words. Picture and original issue can be found here: #35729
Basically, rich text effects allowed for character visibility changes. While doing so would work properly, the rich text label would render the next word in an offset accounting for the hidden characters (leaving a huge space after hidden characters.)
This patch fixes this issue by keeping track of the amount of backtrack necessary per line.
Example of it working:

As an aside, is there a real reason that we're using #defines besides carefully crafted static inline functions? I understand the use case of macros over inline functions (no need for typechecks or long argument lists) but it also adds a lot of room for error when working on the rather large _process_line method.
Bugsquad edit: Fixes #35729.

Did some playing around and this doesn't seem to be fully fixed, though I'm not entirely clear on the problem this time. If the gif below there seem to be extra spaces appearing after the text processes.
This is a test [corrupt to=*rich*text]**********[/corrupt]effect.

extends RichTextEffect
class_name CorruptTextEffect

export var wait_time : = 0.0
export var transform_time : = 1.0
export var index_time_offset : = 0.6

var bbcode : = "corrupt"

func _process_custom_fx(char_fx : CharFXTransform):
	# Spaces don't work in rich text arguments, so use * as a space
	var to_text : String = char_fx.env.get("to", "").replace("*", " ")
	var hide : int = ord(char_fx.env.get("hide", "*"))

	var delay : = wait_time + (index_time_offset * char_fx.relative_index)

	if char_fx.elapsed_time >= delay:
		if to_text:
			var percent : = (char_fx.elapsed_time - delay) / transform_time
			if percent < 1.0:
				char_fx.character = randi() % 100
				# Ensure we don't randomize to the hidden character
				if char_fx.character == hide:
					char_fx.character += 1
			else:
				# If the target string is shorter than the source, hide trailing characters
				if to_text.length() >= char_fx.relative_index + 1:
					char_fx.character = ord(to_text[char_fx.relative_index])
				else:
					char_fx.character = hide
		else:
			char_fx.character = randi() % 100
			# Ensure we don't randomize to the hidden character
			if char_fx.character == hide:
				char_fx.character += 1

	if char_fx.character == hide:
		char_fx.visible = false
	return true

@BanchouBoo The problem with the above example is when you have a long set of visible strings (**********) it calculates the line length preemptively before running text effects. In other words, it will break to a new line preemptively (at least in my example project scenario.) I'm not sure in that particular circumstance there's a way to gracefully double back a whole line (though I could be mistaken, I'll take a look at it more later.)
If you could refactor the above script to account for breaks in the target corruption (for example, **** ****) it would probably have the desired behavior and would work as expected. Simply changing your example to: This is a test [corrupt to=rich*text]**** ****[/corrupt] effect. works on my end.

# Spaces don't work in rich text arguments, so use * as a space

This is probably worth its own bug report, albeit I'm not sure on the most elegant way to implement that into my rich text argument parser. Perhaps a syntax like to=string("rich text") would be acceptable?

Thanks!

Cherry-picked for 3.2.1.


Sort by what is seen on the page

If innerHTML is used, as is now, eg. table-rows with a link in the sorting column's table-data are sorted by the target href or whatever attribute is there in the source instead of the link text

Hmm, this is bad, it removes all the links!

Well, turns out the following: Do not make the table-data the element to sort, but the link inside. Source sample: Link Text	 I had the class declared on the td, which will produce strange results.

As an aside; the behaviour can be used to sort according to invisible criteria: just wrap the textContent in a span with attribute e.g. title that will override what is seen on the page.

At last, this /feature/ makes it possible, to sort on what is NOT visible ;) Wrap the sorting field in a  and it sorts on the attribute!


email adress not valid

If you want to write an email to request a new supported R package in Power BI, there is always an email coming back that it is not allowed to send emails to<EMAIL_ADDRESS>my email I sent out gone to<EMAIL_ADDRESS>but there seems to be a forwarding to the<EMAIL_ADDRESS>Soo: Where shall I write to to get my request placed?

@m123j123 Can you go to https://community.powerbi.com/ to provide your request for product feedback?


Client API for getting all active .watch() queries

It would be nice for clients to be able to cancel watch queries all at once, for example when navigating away from a page in an SPA, you may want to clean up resources from the previous page

I think you probably want a mechanism to get all active queries. Given that .fetch() queries are also built on top of streams, cleaning those up makes sense too.

I am looking for a way to unsubscribe and close active cursors when using sendRequest, im able to unsubscribe but it doesnt close the cursors until i reload the page, but maybe that is another thing than this?

So i guess close it even though done isnt send from the request handler

@stellanhaglund is this in the context of #515 ?

Yeah i didnt see that.
Looks like its what im talking about but doesnt look like it solves my
problem all the way yet
torsdag 26 maj 2016 skrev Josh Kuhn<EMAIL_ADDRESS>
@stellanhaglund https://github.com/stellanhaglund is this in the
context of #515 https://github.com/rethinkdb/horizon/pull/515 ?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/rethinkdb/horizon/issues/441#issuecomment-221897721

We are now tracking this information in the Socket, and will likely have everything we need for this with #9


Does Flutter support push notifications?

Can I use a flutter to realize android's push notification?

Yes, Flutter has a plugin for push notifications.
To receive push messages on both Android & iOS, you can use the Firebase Messaging package.

@roughike Thank you!

Firebase is not really a solution. It's blocked in many countries.

This should get more visibility

Is there support for local notification?

@yuehernkang perhaps https://pub.dartlang.org/packages/scheduled_notifications ?

@zoechi That only supports Android.

@yuehernkang If you just need basic local notifications (custom text and actions), you can try this plugin: https://github.com/mitchhymel/local_notifications . Should work on both Android and ios.

Thanks to everyone who offered suggestions. I don't think there's any action required here.

So it's clear, firebase is google's long term plan for push notifiactions, they've entirely depricated GCM. "We will turn off most GCM services in April 2019"
https://developers.google.com/cloud-messaging/faq

@collinjackson Do you have plans to add support for push notification without Firebase? Would be good to see a solution for IOS without this dependency.

Imho it is a terrible decision by the devs to limit push notifications to firebase. Notifications are something my app should be able to send to the user without some third party dependency and the need for an internet connection.

See my comment @pixelsnake, it's not a third party dependency, at least for Apple, it's Google owned and the official way to do them, they are deprecating gcm (google cloud messaging).  At that point, iOS is easier to setup using firebase since you're already using them for good, so I can see why they did it this way.  If you really want to use Apple's push notifications directly then you can do so by coding that part in iOS and use platform channels if needed

I help work on OneSignal and we created a push SDK for Flutter https://github.com/OneSignal/OneSignal-Flutter-SDK

@lifenautjoe I have the same issue for iOS, any news?

I'm not sure why I'm getting thumbs downed.  I am stating what I'm hearing from google.  Let me know if you think I'm wrong or why you have a problem with something.  But to maybe clarify one thing further
As linked above: https://developers.google.com/cloud-messaging/faq GCM is depricated so there won't be flutter support for this.
This references issue "push notifications" which seems to be what people are discussing - pushing notifications from a server to the phone.  If you're interested in local or scheduled notifications that's a different matter.  I find those to be usefull less often than ones that originate from a server, but there are cases for them and I haven't looked into what is required to implement those in a flutter application

how about to overwrite the some functions in AppDelegate file, then when didRegisterForRemoteNotificationsWithDeviceToken finish with token, we could use channel to tell dart that we have got the token, then in dart, we could get token and submit it to the server.
i don't know if this way could be work. so does any one resolved  this problem??

The devs aren't willfully neglecting this feature - #13328 Is where "generic push notifications" is being tracked

I'm not sure why I'm getting thumbs downed. I am stating what I'm hearing from google. Let me know if you think I'm wrong or why you have a problem with something. But to maybe clarify one thing further
As linked above: https://developers.google.com/cloud-messaging/faq GCM is depricated so there won't be flutter support for this.
This references issue "push notifications" which seems to be what people are discussing - pushing notifications from a server to the phone. If you're interested in local or scheduled notifications that's a different matter. I find those to be usefull less often than ones that originate from a server, but there are cases for them and I haven't looked into what is required to implement those in a flutter application

What about silent push notifications? I was under the impression Firebase didn't support them. I'm also wicked unhappy to have to support multiple push engines on my servers. (sigh) :(

I have shared my knowledge on integrating a firebase push notification with flutter
Link: Flutter push notification

It still does not support Rich Notifications (Notifications with Image) on iOS, Anybody has workaround for this???

@ghostrick
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/962. Any further collaboration will be done there.

Is it possible to make push notifications with Amazon SNS?

Is it possible to make push notifications with Amazon SNS?

FCM is the only way to send a push notification to an Android phone and APNS is the only way to send a push notification to iOS.


Add int8 and int16 support for FILL operator

This PR adds int8 and int16 support for the FILL operator to support quantization.

I have updated the PR to resolve merge conflicts caused by 49752e0f82cd. Feel free to comment @thaink as I have changed additional files to integrate the INT8 support.

I have updated the PR to resolve merge conflicts caused by 49752e0f82cd. Feel free to comment @thaink as I have changed additional files to integrate the INT8 support.

@teijeong Does fill op need quantization support?

@teijeong Does fill op need quantization support?

@thaink The broader quantization support is better since that makes us to run inferences with avoiding the quantization/dequantization pairs. Please review this and they have models that have Fill ops.

@thaink The broader quantization support is better since that makes us to run inferences with avoiding the quantization/dequantization pairs. Please review this and they have models that have Fill ops.


Xiaomi device tracker

Description: The change is intended to add support for Xiaomi Router as device tracker
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#1751
Example entry for configuration.yaml (if applicable):
device_tracker:
  - platform: xiaomi
    host: <IP_ADDRESS>
    username: admin
    password: 1234

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

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

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

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[NA] New dependencies have been added to the REQUIREMENTS variable ([example][ex-requir]).
[NA] New dependencies are only imported inside functions that use them ([example][ex-import]).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.
[x] Tests have been added to verify that the new code works.

Looks good! :dolphin:


feat(labs-react): export Options interface

I have the following code to avoid repeating passing React on every createComponent call.
import * as React from 'react';
import { createComponent as createReactComponent } from '@lit-labs/react';
import type { EventName } from '@lit-labs/react';

type EventNames = Record<string, EventName | string>;

type Constructor<T> = new () => T;

export const createComponent = <
	I extends HTMLElement,
	E extends EventNames = {},
>(options: {
	tagName: string;
	elementClass: Constructor<I>;
	events?: E;
	displayName?: string;
}) => {
	return createReactComponent<I, E>({ react: React, ...options });
};

I want to be able to
import * as React from 'react';
import { createComponent as createReactComponent } from '@lit-labs/react';
import type { EventName, Options } from '@lit-labs/react';

type EventNames = Record<string, EventName | string>;

export const createComponent = <
	I extends HTMLElement,
	E extends EventNames = {},
>(options: Omit<Options<I, E>, 'react'> ) => {
	return createReactComponent<I, E>({ react: React, ...options });
};

This PR fixes createCompoennt typo and exports the Options interface.

Thank you for the PR! I think exporting option type is fine but unsure if we want to accept it right now as we're in the middle of moving the source code over as part of graduation. @justinfagnani @rictic WDYT? Should we get this in before graduation?
Another workaround is to use Parameters like
type Options = Parameters<typeof createComponent>[0];

There's no issues with merging changes in now, I'm being careful in the graduate changes so that changes will naturally migrate over to their new locations when we merge into main

Thank you, @augustjk, for your suggestion. However, I encountered a TypeScript error when attempting to use the Parameters helper for the first time.

Exported variable 'createComponent' has or is using name 'Options' from external module ".../node_modules/@lit-labs/react/create-component" but cannot be named. ts(4023)

The following is the code I used
import * as React from 'react';
import type { EventName } from '@lit-labs/react';

import { createComponent as createReactComponent } from '@lit-labs/react';

type EventNames = Record<string, EventName | string>;

export const createComponent = <
	I extends HTMLElement,
	// eslint-disable-next-line @typescript-eslint/ban-types
	E extends EventNames = {},
>({
	tagName,
	elementClass,
	events,
	displayName,
}: Omit<Parameters<typeof createReactComponent<I, E>>[0], 'react'>) => {
	return createReactComponent<I, E>({
		react: React,
		tagName,
		elementClass,
		events,
		displayName,
	});
};

PS: I've omitted some other unrelated changes

Thanks for the update @rivajunior . We'll get this released soon.

Released in<EMAIL_ADDRESS>


error when listing aux plugins

./owtf.py -l net
              __       ___
             /\ \__  /'___\

___   __  __  \ \ ,/\ _/
/ `/\ /\ /\ \ \ /\ \ ,\
/\ _\ \ \ _/ _/ \ \ _\ \ _/
\ _/\ _x/'\ _\ _\
//  ////   /_/ /_/
Offensive (Web, etc) Testing Framework: An OWASP+PTES-focused try to unite great tools and make pen testing more efficient @owtfp http://owtf.org
Author: Abraham Aranguren<EMAIL_ADDRESS>- http://7-a.org - Twitter: @7a_
[] Loading framework please wait..
[] Loading Config from: /root/Desktop/webs/owtf/framework/config/framework_config.cfg ..
Traceback (most recent call last):
File "./owtf.py", line 210, in
ProcessOptions(sys.argv[1:], Core)
File "./owtf.py", line 160, in ProcessOptions
PluginTypesForGroup = Core.Config.Plugin.GetTypesForGroup(PluginGroup)
File "/root/Desktop/webs/owtf/framework/config/plugin.py", line 43, in GetTypesForGroup
for PluginType, Plugins in self.AllPlugins[PluginGroup].items():
AttributeError: 'list' object has no attribute 'items'

Thank you for the bug report am06!
net plugins are not implemented yet but I agree that this should be handled more gracefully, will try to fix it this week.

sure, no problem.
I didn't know what's exactly the problem or how it should work, because i couldn't find much details to read.....


[Improvment]: Notifications & Resend

What happened?
Hi ! I'm opening this issue because I would want to improve this package by adding support for Notifications in addition to Mails, but I want to be sure that it is in line with what you envisage for the future. I'd also like to add support for Resend.
Is this something you've thought about? Or are you used to using Mailables for emails and Notifications for everything else, without mixing the two?
How to reproduce the bug
PS : Discussions are note enabled on this repository, then Ask a question and Request a feature buttons are not working.

Package Version

PHP Version
8.3
Laravel Version
11.25
Which operating systems does with happen with?
No response
Notes
No response

Hi @benjaminhaeberli,
Thanks for your message! I'm not sure if I understand your question completely.
Do you ask if we want to add features to log (specifically) notifications and resend them if needed?
If so, that would be out of scope, as this is focussed on e-mail only.
But.. if your sending e-mail notifications, it will already automatically logged by this package already and you can resend them if you want, because it is essentially an email.

Hi @markvaneijk ! My questions weren't clear sorry, I had two things in mind :

Add support Mail Notifications
Only the the Mail Facade and  forMailable are supported right ?

Add support for Resend (the email service provider)
Only Postmark is supported right ?

Sorry for the confusion !

Every mail sent using Laravel is supported, it listens to an event that Laravel dispatches when sending an email. Using all framework related methods this will be fine, also for Mail Notifications 👍🏼
A driver for Resend is in the works.

Damn, perfect ! I close this issue then, thanks for your answer :)


RemoveIncludeParentheses not working

I have excluded following setting but still when I format the code it removes the parentheses from include and include_once statements.
"exclude": ["RemoveIncludeParentheses"]

Hi Team,Any updates on this issue?


ffmpeg does not work/load

Hi, trying to run my script that uses pyglet to play a video, and am getting errors about the avutil library.
Am on M1 Mac,
Python 3.11.6
pyglet==2.0.14
ffmpeg==1.4
ffmpeg-python==0.2.0
python-ffmpeg==2.0.10
pyglet.media.have_ffmpeg() says False although latest ffmpeg installed and in path
And when trying to do this import, it gives the error below.
Thanks for any help in advance!
TP
from pyglet.media.codecs.ffmpeg import FFmpegDecoder
ImportError                               Traceback (most recent call last)
Cell In[3], line 1
----> 1 from pyglet.media.codecs.ffmpeg import FFmpegDecoder
File /Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/pyglet/media/codecs/ffmpeg.py:15
13 from .base import AudioData, SourceInfo, StaticSource
14 from .base import StreamingSource, VideoFormat, AudioFormat
---> 15 from .ffmpeg_lib import *
16 from ..exceptions import MediaFormatException
19 class FileInfo:
File /Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/pyglet/media/codecs/ffmpeg_lib/init.py:2
1 """FFmpeg wrapping"""
----> 2 from .libavcodec import *
3 from .libavutil import *
4 from .libavformat import *
File /Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/pyglet/media/codecs/ffmpeg_lib/libavcodec.py:11
9 from pyglet.util import debug_print
10 from . import compat
---> 11 from . import libavutil
13 _debug = debug_print('debug_media')
15 avcodec = pyglet.lib.load_library(
16     'avcodec',
17     win32=('avcodec-60', 'avcodec-59', 'avcodec-58'),
18     darwin=('avcodec.60', 'avcodec.59', 'avcodec.58')
19 )
File /Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/pyglet/media/codecs/ffmpeg_lib/libavutil.py:13
9 from . import compat
11 _debug = debug_print('debug_media')
---> 13 avutil = pyglet.lib.load_library(
14     'avutil',
15     win32=('avutil-58', 'avutil-57', 'avutil-56'),
16     darwin=('avutil.58', 'avutil.57', 'avutil.56')
17 )
19 avutil.avutil_version.restype = c_int
20 compat.set_version('avutil', avutil.avutil_version() >> 16)
File /Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/pyglet/lib.py:135, in LibraryLoader.load_library(self, *names, **kwargs)
132             if _debug_lib:
133                 print(f"Unexpected error loading library {name}: {str(o)}")
--> 135 raise ImportError(f'Library "{names[0]}" not found.')
ImportError: Library "avutil" not found.

You might have followed https://ffmpeg.org//download.html to the unofficial mac builds, it's possible you installed a prerelease version (pseudo-7.x.x) whose set of libraries are too recent.
Try installing ffmpeg 6.1.1, which should be the latest available ffmpeg release right now. That one includes avutil/avcodec etc. at versions supported by pyglet.
And fwiw the python packages are all bindings that do not contain the ffmpeg libraries, so you may want to uninstall those again.

Hi, thanks. No, I actually have ffmpeg 6.1.1. installed. I tried to install pyglet-1.5.28, there the problem with avutil does not occur. But when I reinstall the current version, either with pip3 or if I compile it, there is still the same issue.
when I want to do this:
from pyglet.media.codecs.ffmpeg import FFmpegDecoder
I get this:
File ~/Library/Python/3.11/lib/python/site-packages/pyglet-2.0.14-py3.11.egg/pyglet/media/codecs/ffmpeg.py:15
     13 from .base import AudioData, SourceInfo, StaticSource
     14 from .base import StreamingSource, VideoFormat, AudioFormat
---> 15 from .ffmpeg_lib import *
     16 from ..exceptions import MediaFormatException
     19 class FileInfo:

File ~/Library/Python/3.11/lib/python/site-packages/pyglet-2.0.14-py3.11.egg/pyglet/media/codecs/ffmpeg_lib/__init__.py:2
      1 """FFmpeg wrapping"""
----> 2 from .libavcodec import *
      3 from .libavutil import *
      4 from .libavformat import *

File ~/Library/Python/3.11/lib/python/site-packages/pyglet-2.0.14-py3.11.egg/pyglet/media/codecs/ffmpeg_lib/libavcodec.py:11
      9 from pyglet.util import debug_print
     10 from . import compat
---> 11 from . import libavutil
     13 _debug = debug_print('debug_media')
     15 avcodec = pyglet.lib.load_library(
     16     'avcodec',
     17     win32=('avcodec-60', 'avcodec-59', 'avcodec-58'),
     18     darwin=('avcodec.60', 'avcodec.59', 'avcodec.58')
     19 )

File ~/Library/Python/3.11/lib/python/site-packages/pyglet-2.0.14-py3.11.egg/pyglet/media/codecs/ffmpeg_lib/libavutil.py:13
      9 from . import compat
     11 _debug = debug_print('debug_media')
---> 13 avutil = pyglet.lib.load_library(
     14     'avutil',
     15     win32=('avutil-58', 'avutil-57', 'avutil-56'),
     16     darwin=('avutil.58', 'avutil.57', 'avutil.56')
     17 )
     19 avutil.avutil_version.restype = c_int
     20 compat.set_version('avutil', avutil.avutil_version() >> 16)

File ~/Library/Python/3.11/lib/python/site-packages/pyglet-2.0.14-py3.11.egg/pyglet/lib.py:135, in LibraryLoader.load_library(self, *names, **kwargs)
    132             if _debug_lib:
    133                 print(f"Unexpected error loading library {name}: {str(o)}")
--> 135 raise ImportError(f'Library "{names[0]}" not found.')

ImportError: Library "avutil" not found.

ImportError is just an error saying it can't find your ffmpeg files. You should only be trying to import the decoder manually if pyglet.media.have_ffmpeg() is True. You need to ensure your ffmpegs dylib files are in the correct path and being found. If you still can't get it found by your system, you may have to move it locally, refer to the documentation here: https://pyglet.readthedocs.io/en/latest/programming_guide/media.html#ffmpeg-installation

Closing as user installation error.


Setup for project

In this pull request, I made the basic linters and others setup for the project.

Thanks @AhmedNazirMusah


Embedded language parser with context-dependant "boundaries"

I hope the issue title isn't too confusing. This is closely related to #803.
I'm trying to build a parser for a language like this:
FUNCTION x {
    SELECT * FROM t;
}

NAMESPACE y {
    FUNCTION z {
        SELECT * FROM t WHERE k = $1;
    }
}

In short: the "global" context is like a namespace, namespaces can contain other namespaces and functions and functions contain raw SQL code that's passed onto an external parser.
With the help of #803 I found out about multi-mode parsers, but I can't figure out how to apply them here. The { character can open both function and namespace 'blocks' so I can't differentiate there. I also don't see an efficient way to change mode on the FUNCTION or NAMESPACE keywords because there's an identifier token in the middle (from the lexer's point of view SQL could also be a bunch of identifiers).
I've tried more things than I can remember, but the only solution that I haven't explicitly seen failing and that I can make sense of in my head goes like this:

FUNCTION pushes the "function declaration mode"
In this mode only identifier and "function block open/close" tokens are used
This "function block opener" token pushes the "SQL mode"
In "SQL mode" there's a single custom pattern matcher that matches everything up to an (unescaped) } and pops the "SQL mode" again
The lexer would now be in "function declaration mode" again, matching the "function block close" token (popping the "function declaration mode")

Although this system makes sense in my head, I feel like it's not actually going to work. It seems unnecessarily convoluted to me, especially the fact that I'd need 2 separate "block opener" tokens that have the same pattern (but different push-/pop-modes).
Am I just overthinking it and is this actually a valid solution? Now that I've written it out like this it doesn't look so bad anymore, but I feel like I'm abusing the multi-mode functionality here. Is there a better solution that I should be aware of?

Why are multiple lexer modes needed for this language?
Are you trying to avoid Token sets conflicts?

If you do need a multi-mode lexer here you could use custom token patterns that include additional logic to define "complex" boundaries.
e.g: A function boundary is a "{" followed by WS and then "FUNCTION" keyword.

Why are multiple lexer modes needed for this language?
Are you trying to avoid Token sets conflicts?

I want to grab all SQL code as a single token (so I can easily process it externally), but it contains valid identifiers (including NAMESPACE and FUNCTION keywords), so I need to make chevrotain aware that it should not try to interpret anything between FUNCTION *identifier* { and an unescaped }.

e.g: A function boundary is a "{" followed by WS and then "FUNCTION" keyword.

I'm not sure what you mean here. Shouldn't the order be reversed?

This may not even need a custom pattern, RegExp positive lookahead should be able to support this:

Although lookahead is not what I'm looking for, lookbehind could possibly be it. I'll see if I can make it work.

O.k. I think I am beginning to understand.
I still think that if you want to tokenize the whole SQL statement as a single token  you will need lookahead.
Basically a custom token pattern that given an opening curly parenthesis will "scan" ahead
until the matching closing curly parenthesis and tokenize that entire block as a single token.

{
SELECT * FROM t WHERE k = $1;
}

The problem is that I can't differentiate on "a curly brace followed by SELECT" because any valid SQL statement should be allowed, that's why I think lookbehind would make more sense. With this new knowledge on lookahead/behind regexes I agree that multiple modes are unnecessary.

The problem is that I can't differentiate on "a curly brace followed by SELECT" because any valid SQL statement should be allowed

o.k. If you can build the logic to differentiate the scenarios using lookbehind its good.
Do you control the language definition? Can to specify some form of unique separators for
embedded SQL that would be simpler to identify?

I tried using lookbehinds and ended up with the following pattern:
/(?<=FUNCTION(\s+)([a-zA-Z]\w*)(\s*)){(\\}|[^}])*}/

It looks like it works 😄 (apart from not being able to use "first char" optimization, see the regexp-to-ast issue I made for that).
I wonder if it'd be possible to do this more efficiently using custom pattern matcher functions though. Would it be possible to make the matcher function "look behind itself" and check what kind of tokens precede it?
This avoids the monstrosity of a regex above and especially the repeating parts of it (the part for FUNCTION and identifier), which leads to better maintainability.
I think I tried something like that before, but I ran into strange issues with it telling me that "blockOpen" could never be matched because the custom pattern was above it or something. I'm currently on a different machine so I can't try this myself right now.

Do you control the language definition? Can to specify some form of unique separators for
embedded SQL that would be simpler to identify?

I do and I could, but I want to keep things as simple as possible (as long as it's possible parsing-wise of course).

I wonder if it'd be possible to do this more efficiently using custom pattern matcher functions though. Would it be possible to make the matcher function "look behind itself" and check what kind of tokens precede it?

You get the previously matched tokens and the whole text so why not?

https://sap.github.io/chevrotain/documentation/6_1_0/globals.html#custompatternmatcherfunc

I do and I could, but I want to keep things as simple as possible (as long as it's possible parsing-wise of course).

Almost everything is possible with enough effort but simplicity also has its perks.

https://www.python.org/dev/peps/pep-3099/

The parser won't be more complex than LL(1).
Simple is better than complex. This idea extends to the parser. Restricting Python's grammar to an LL(1) parser is a blessing, not a curse. It puts us in handcuffs that prevent us from going overboard and ending up with funky grammar rules like some other dynamic languages that will go unnamed, such as Perl.

Almost everything is possible with enough effort but simplicity also has its perks.

That's always something to consider of course, but I'm willing to sacrifice a little extra parsing-complexity here for additional end-user-simplicity/consistency.

I ended up implementing this using a custom pattern matcher function like this:
export const sqlBlock = createToken({
	name: 'SqlBlock',
	line_breaks: false,
	start_chars_hint: ['{'],

	pattern: (text: string, startOffset: number, tokens: IToken[]) => {
		let allowed = false; //determines if a sqlBlock is allowed here, false by default for the 'global context'
		for (let i = tokens.length-1; i > 0; i--) {
			if (tokenMatcher(tokens[i], FUNCTION)) {
				allowed = true;
				break; //we're dealing with a function, sqlBlock is allowed
			} else if (tokenMatcher(tokens[i], NAMESPACE)) {
				allowed = false;
				break; //we know we're dealing with a namespace, so a sqlBlock can't be allowed
			}
		}

		return allowed ? /{(\\}|[^}])*}/.exec(text.substring(startOffset)) : null; //this pattern matches anything between and including '{' and an unescaped '}'
	},
});

It's been working great so far without a regex that makes me want to puke 😄

This might be a little out-of-place, but I really want to thank you for your amazing help and work on this library. You created something truly amazing (and blazingly fast) for the community, and so far using it has been the best experience with open source software I've had in a very long time (maybe even ever). It makes me want to use chevrotain more and more, but its purpose is too specific for that unfortunately...
The documentation is very good and you provide excellent (and amazingly quick) help for dumb people like me. You really know your stuff and you're just, well, friendly.
Unfortunately this is a rare sight in the community these days (at least in my experience), so I just wanted to take the time to thank you for your amazing work.
I'd love to contribute to this project in the future, but I'm afraid I'm just not skilled enough to be of any value other than fixing typing mistakes 😅

Does not  /{(\\}|[^}])*}/ regExp assume that the closing curly brace inside the block is always escaped?
Is this always true? could for example a curly brace appear unescaped inside a string literal ?

his might be a little out-of-place, but I really want to thank you for your amazing help and work on this library. You created something truly amazing (and blazingly fast) for the community

You are welcome 😄

I'd love to contribute to this project in the future, but I'm afraid I'm just not skilled enough to be of any value other than fixing typing mistakes 😅

I doubt it is a skill issue but rather simply unfamiliarity with a complex library.
However it is possible to contribute more substantial features even without deep understanding of the library as long as those features flows have clear entry/exit points and some separation from the inner complexity.
Some possible examples for such features

Ability to export grammar as EBNF text.
feature request: ability to save Regex capture groups for tokens

This is creating a wrapper around customTokens that exposed capturing groups using the payload property.

Generation of d.ts signatures for CST Parsers

More complex but still does not go too deep into the "guts" of Chevrotain.

Does not /{(\}|[^}])*}/ regExp assume that the closing curly brace inside the block is always escaped?
Is this always true? could for example a curly brace appear unescaped inside a string literal ?

Yup, a '}' inside a string literal would have to be escaped as well. This isn't the "highest level of convenience" but it makes parsing (or lexing) a lot easier. Since curly braces are very rare in actual SQL code I'm willing to sacrifice that small convenience for a much simpler parsing approach.

However it is possible to contribute more substantial features even without deep understanding of the library as long as those features flows have clear entry/exit points and some separation from the inner complexity.

I'll take a look at some of the issues you linked when I have some more time on my hands, it certainly looks very interesting!

Yup, a '}' inside a string literal would have to be escaped as well. This isn't the "highest level of convenience" but it makes parsing (or lexing) a lot easier. Since curly braces are very rare in actual SQL code I'm willing to sacrifice that small convenience for a much simpler parsing approach.

If that ever becomes a problem you could replace the regExp with your own custom logic that is capable of ignoring the contents of string literals inside the SQL block.


Fill width text field with leading icon overwrites icon

From https://samprof.github.io/MatBlazor/TextField - the text overwrites the icon:

I guess the text should be right of the icon, the same way it is with a normal width text field:

This css is a temporary fix, I will submit a PR for the permanant fix
div.mdc-text-field--fullwidth-with-leading-icon:not(.mdc-text-field--textarea) > input.mdc-text-field__input {
    padding: 20px 16px 6px 48px !important;
}

It looks like the like SSB:  https://samprof.github.io/MatBlazor/ is still on Version 2.3.0 which has the issue.
CSB demo version 2.5.0-develop-004 the padding is fixed but the label is now missing:
https://www.matblazor.com/TextField
https://www.matblazor.com/textfield


[CON-231] Autocomplete cannot read file on Windows SSH to Linux

Before submitting your bug report

[X] I believe this is a bug. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that reports the same bug
[X] I've seen the troubleshooting guide on the Continue Docs

Relevant environment info
- OS:Windows, SSH to Linux
- Continue:*
- IDE:VS Code

Description
Happens after opening parenthesis is typed, caused by attempting to use LSP to find the definition of the file
To reproduce
No response
Log output
Error generating autocompletion:  Error: Unable to read file '\path\to\file' (Error: Unable to resolve nonexistent file '\path\to\file')
    at n.F (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:657:19968)
    at n.D (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:657:19763)
    at async l.M (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2370:42596)
    at async l.readStream (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2370:42353)
    at async L.Cb (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2359:13615)
    at async L.resolve (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2359:12062)
    at async s.W (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2370:38746)
    at async u.r (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2359:25720)
    at async k.acquire (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:37:3320)
    at async d.createModelReference (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2359:27297)

From SyncLinear.com | CON-231

Same issue "Error loading submenu items from file: EntryNotFound (FileSystemError): Unable to resolve nonexistent file 'vscode-remote://wsl/home/gianluca/projects/srm/workspace/directus/api/extensions/hooks/init-config-data/data/1#srm_types'"


Update Set-CsTeamsMeetingPolicy.md

Correct default for webinars

Docs Build status updates of commit 3d539a4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Set-CsTeamsMeetingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@yogkumgit fyi I have merged this.


Chore: Separate "Cloud" and "GlareDB" collections

Description

To differentiate between docs that correspond to GlareDB the database, and the Cloud service.
What this enables

Impacts

While working on this I opened #87 which I think would be really helpful at the "top" level of the docs. We could have a more extensive/alternate version on the landing website for "pricing".
We could also use this in the repo README (both at https://github.com/GlareDB/glaredb.github.io and https://github.com/GlareDB/glaredb.
LMK what you think @scsmithr .
Could either use an illustration chart (so long as appending to it is easy) or just MD tables

Agree on the comparison chart. Would be very helpful.


Fix database error that occurs during wallet deletion.

Issue Number
#1242
Overview
This PR adjusts the implementation of API.Server.deleteWallet so that Cardano.DB.destroyDBLayer is only called once during its execution.
This can be verified by inspecting the logs before and after applying this PR.
Before:
[cardano-wallet.api:Info:255] [2020-01-07 05:02:04.59 UTC] [RequestId 1] [DELETE] /v2/wallets/67f6430300c403a0a0f2920c2a7c4cd565118a9d
[cardano-wallet.database:Debug:255] [2020-01-07 05:02:04.59 UTC] Closing database (/home/jsk/.local/share/cardano-wallet/jormungandr/testnet/wallets/seq.67f6430300c403a0a0f2920c2a7c4cd565118a9d.sqlite)
[cardano-wallet.database:Debug:185] [2020-01-07 05:02:04.63 UTC] Closing database (/home/jsk/.local/share/cardano-wallet/jormungandr/testnet/wallets/seq.67f6430300c403a0a0f2920c2a7c4cd565118a9d.sqlite)
[cardano-wallet.worker:Error:185] [2020-01-07 05:02:04.63 UTC] 67f64303: Worker has exited unexpectedly: SQLite3 returned ErrorMisuse while attempting to perform close: bad parameter or other API misuse
[cardano-wallet.api:Info:255] [2020-01-07 05:02:04.63 UTC] [RequestId 1] 204 No Content in 0.043179776s
[cardano-wallet.api:Debug:255] [2020-01-07 05:02:04.63 UTC] [RequestId 1] Completed response to API request

After:
[cardano-wallet.api:Info:212] [2020-01-07 05:15:22.31 UTC] [RequestId 1] [DELETE] /v2/wallets/4152d133208bb9f2f3ae8ef76a53086c59709c2b
[cardano-wallet.database:Debug:127] [2020-01-07 05:15:22.31 UTC] Closing database (/home/jsk/.local/share/cardano-wallet/jormungandr/testnet/wallets/seq.4152d133208bb9f2f3ae8ef76a53086c59709c2b.sqlite)
[cardano-wallet.api:Info:212] [2020-01-07 05:15:22.31 UTC] [RequestId 1] 204 No Content in 0.001427646s
[cardano-wallet.api:Debug:212] [2020-01-07 05:15:22.31 UTC] [RequestId 1] Completed response to API request

Comments
The second call to Cardano.DB.destroyDBLayer happens implicitly, as a result of calling liftIO $ Registry.remove re wid.

@jonathanknowles there's an inherent race-condition here unfortunately. While one thread is deleting the database file from disk, another thread might be trying to read or write to this very database. This is why removeDatabase made sure of destroying the database layer first, preventing any other thread from reading / writing to it.
See my comment in #1242

@jonathanknowles there's an inherent race-condition here unfortunately. While one thread is deleting the database file from disk, another thread might be trying to read or write to this very database. This is why removeDatabase made sure of destroying the database layer first, preventing any other thread from reading / writing to it.
See my comment in #1242

I did actually test for this condition (on Linux), by actively deleting the DB files and then continuing to issue operations that affect the DB. These operations continue to succeed without any errors, presumably because the underlying DB creates RW file descriptors for the files at the point when the connection is initiated, only releasing those descriptors once the "close" command has been issued.
So even though we issue a "delete" operation, the delete simply unlinks the files from the .local/share/cardano-wallet/jormungandr/testnet/wallets/ directory, but the files still remain accessible to the DB while it holds on to its file descriptors.
However, for Windows, I suspect this behaviour is completely different: from memory, I recall that Windows will prevent a process from deleting a file if that file is open (for reading or writing) by another process.
Therefore, I agree with your assessment that we need to do something else: I've updated this PR to make the destroyDBLayer idempotent.

bors r+


no date validation in rules for client-side

public function rules() {
return [
[[['description'], 'string','max' => 80],
['birthday'], 'date', 'format' =>'Y/m/d']];
}
In the example above, you see that every types have their own client-side validation, but there is no such thing for 'date' data-type.

Try:public function rules() {
return [
[['description'], 'string','max' => 80],
[['birthday'], 'date', 'format' =>'Y/m/d'],
];
}

ok but not client-side validation

You could extend the DateValidator to add your own client side validation.
It's probably too awkward to add as different date pickers work in different ways.

date validation on client side is quite complex, as it would need to re implement icu format parser in JS. we currently do not have plans to do this.

What about implement it using Moment.js?

That's OK but is out of framework scope.

Why? For example we have integration with Bootstrap for a Form Builder. Why we can't have integration with Moment?

We probably can. Will re-consider for 2.1.1.


Improve actor getRollOptions performance

This PR aims to improve the performance of the actors getRollOptions function.
Performance is quite difficult to measure in the best of times, but from my profiling, this change alone means the function only takes about 60% the time to execute.
Absolute terms are even more complicated as they depend on even more variables, but in my test world this shaves off about 17ms when changing turns in the encounter tracker. This function is also a hot path when adding effects or manipulating actors in other ways.
previous function:

with Object.keys

The reason Object.keys is faster in this case is due to the fact that no additional arrays have to be created for the [key, value] pair.

I'm kinda annoyed that they don't optimize out the array destructuring.

Huh, interestingly, both Firefox and Safari optimize this much better than Chrome.
On my system (MacBook, M1 family), chrome is about twice as fast when using Object.keys instead of Object.entries, with 3k total runs in the attached jsperf benchmark. Safari and Firefox both have negligable performance differences between the two (about 4-5%). Both also run much faster overall, with 6k iterations for safari and 9k in firefox.
That means in this micro benchmark, Firefox for Object.entries is more than 6 times faster than chrome.
I still think that it makes sense to consider this change in this place, as even in Firefox Object.keys is at least as fast as Object.entries and I don't think that the code quality suffers too much from this one change. Maybe some day V8 will optimize this better overall.
JSPerf for reference: https://jsperf.app/yabije

The interesting bit is that its not even the destructuring, its Object.entries() itself. We'll have to see where we can begin incorporating maps, but I can imagine that the initialization of a map is where most of its expense is.


What the difference between class average IoU (mcIoU) and instance average IoU (mIoU) for ShapeNetPart dataset?

Hi,thanks for your great work, can you tell me what the difference between class average IoU (mcIoU) and instance average IoU (mIoU) for ShapeNetPart dataset?How to distinguish them in the codes?

First we compute IoU for all instance object. The instance mIoU is the average off all instance IoU. To get the class mcIoU, we first get the the IoU for each class (see Table 5 in paper supplementatry). THe mcIoU is the average of all the class IoU.
You can look a the function compare_convergences_multisegment in plot_convergence.py where both metrics are computed:
https://github.com/HuguesTHOMAS/KPConv/blob/07861d8e40244c20ede5a362052e01f61dcc012a/plot_convergence.py#L352

First we compute IoU for all instance object. The instance mIoU is the average off all instance IoU. To get the class mcIoU, we first get the the IoU for each class (see Table 5 in paper supplementatry). THe mcIoU is the average of all the class IoU.
You can look a the function compare_convergences_multisegment in plot_convergence.py where both metrics are computed:
https://github.com/HuguesTHOMAS/KPConv/blob/07861d8e40244c20ede5a362052e01f61dcc012a/plot_convergence.py#L352

Thanks!


8.x logging.yaml rolling enabled options dont work as expected

From the docs its expected that you should be able to use the previous numeric roll options, 0/1/2/etc, or be able to use the old lua version of log.roll.none, log.roll.both, etc. In testing it seems like none of those work and the only way I could get the roll option to work was to specify like 'both', 'size'

Looks like this was resolved? Please remember to add e.g "fixes #5080" to the PR commit message that fixes an issue.


Exception when importing expired credentials

Since #48, importing expired credentials fails since UNKNOWN_IDENTITY.name throws an exception.

Reinstitute the expired creds check in import_credentials
maybe Make UNKNOWN_IDENTITY its own class that implements the name property in a useful way

Fixed by #67


6.0 Feedback: Lights turn off during deck cal

Overview
I initiated a deck cal on v6.0.0-beta.0. When I got to the initial pick up tip step, the lights turned off. The lights should be on for the entire calibration sequence.
Current Behavior
No response
Expected Behavior
No response
Steps To Reproduce
No response
Operating system
No response
Robot setup or anything else?
No response

Sorry, I turned off the light when @emilywools tried to do deck cal.
I'm 100 sure that the light always turns on when an OT-2 does deck cal, pipette offset cal, and tip length since I've done them so many times since last month and that hasn't happened even once.


Build time issue/bug?

Hi guys,
first - thanks for Drone, it's really awesome.
My issue - I'm running Docker-in-Docker on Kubernetes using Helm chart & latest version (0.8.4), and I'm running into two time-related issues:

each time I trigger a build, timer starts at 00:50 (50 seconds) and goes down to 00:00 and then continues onwards as normal
each time I trigger a build, some steps (not only clone) would start at 59:10 and continue onwards until reaching 00:00 and continue as normal (the task completion time appears to be correct).

Screenshot:

I checked time difference between my local machine and nodes and it appears to be around ~50 seconds, so that's that.
However, is it an expected behaviour? Thought I'd check!

@NaurisSadovskis hey there, we track issues with the user interface in github.com/drone/drone-ui. Since this repository does not have any of the UI rendering code, it wouldn't make sense to track such questions here.

Thanks, I'll open an issue there, thank you!


IE Constant Refreshing

In IE11 (or any other version of IE), my initial frame loads in properly, though sometimes subsequent page loads cause the frame to constantly reset, grabbing a different height value every time.
Link:
Settings:
enablePublicMethods     : false,
				heightCalculationMethod	: 'max',
				interval				: -33,
				autoResize				: true,
				checkOrigin				: false,

To replicate: In IE, initial page loads in fine, on left hand side, click 'Search', next page loads in, and infinitely refreshes.
I tried changing the interval to 0, -X, and X and it didn't seem to fix the problem.
Any ideas?
EDIT: Also, as a side/unrelated question, how would I go about positioning the parent frame to the top of the page after each resize? (sometimes on a long page, if you click a link, you stay at the bottom of the page on the parent frame, leaving some people lost!

Dealing with the same issue here. IE9+ working fine with autoResize: true. But in IE8 iframe height is constantly increasing. autoResize: false solves this, but what if we actually need autoResize: true? As far as I understand when we set autoResize: true iframe is resizing automatically when we make changes to the iframe DOM. This seems like very common need, so I am wondering if this can be solved without needing to set autoResize: false.
I also found that this CSS inside iframe can cause this issue:
html,
body {
    height: 100%;
    }

But I think this is valid CSS and in the perfect world it should not cause issues like this...


Change webp images on home page to something Safari on Catalina can render properly

https://web.dev/serve-images-webp/
We can continue to serve webp to browsers that support it via:
<picture>
  <source type="image/webp" srcset="flower.webp">
  <source type="image/jpeg" srcset="flower.jpg">
  <img src="flower.jpg" alt="">
</picture>


[HUDI-840]Clean blank file created by HoodieLogFormatWriter

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
When roll over is true,  HoodieLogFormatWriter will create next version log file. But it always left a blank file when close.
Brief change log
(for example:)

Clean blank file created by HoodieLogFormatWriter

Verify this pull request
This pull request is a trivial rework / code cleanup without any test coverage.
Committer checklist

[x] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@yanghua please have a review when free.

@n3nash could you also review this since it touches the common log format writer as well

@hddong This PR has conflicts files.

@yanghua had rebase this.

@vinothchandar :
https://github.com/apache/hudi/blob/0cb24e4a2defd8e639437b6cd145a26f038ef1af/hudi-common/src/main/java/org/apache/hudi/common/table/TableSchemaResolver.java#L382
readSchemaFromLogFile may get the blank file and the blank file had been read before in HoodieLogFileCommand(modified to avoid reading the blank files)

@hddong : I went ahead and  redid this change in the interest of time :)
Instead of deleting on close, I have made changes to lazily create the log file when appending next block. This should avoid creating empty files.
(cc @vinothchandar )

LGTM


decode async problem

trying to return
decodeAsync(response.body) and i get
core.js:7187 ERROR Error: Uncaught (in promise): TypeError: e.getReader is not a function
TypeError: e.getReader is not a function

import { decodeAsync } from "@msgpack/msgpack";

const MSGPACK_TYPE = "application/x-msgpack";

const response = await fetch(url);
const contentType = response.headers.get("Content-Type");
if (contentType && contentType.startsWith(MSGPACK_TYPE) && response.body != null) {
  const object = await decodeAsync(response.body);
  // do something with object
} else { /* handle errors */ }

Above example not work, the same error :)

What's your environment and the version of @msgpack/msgpack?

Close it because of no responses. Will re-open it if necessary.

@gfx I’ve also run into this issue, I’m on my phone right now and I’ll update my specs later but as far as I know I’m developing in TS on react native and as of writing this I believe it’s the latest version of messagepack

@Genhain Is your first error is "TypeError: e.getReader is not a function"? Otherwise, you should file a new issue instead of replying a closed issue.
I also want more details:

msgpack-javascript version
platform / environment  on which  your program runs
minimum code to reproduce this issue (if possible)


1.0.2 version. Correct version of peer @mate-academy/monaco-editor

Task linked: CU-861mhjb0r Update monaco-editor to the newest version


TypeError: operand 'prototype' property is not an object

Got that error when launching the plugin compiled on my own

Can confirm, I've reverted the commit that caused this.
@SorsOps let's look into this on Monday and re-implement #1688

I'll close this issue as it's been fixed.


[Migrated] [Feature] Use lambda-warmer-py decorator to keep more than one lambda instance warm.

Originally from: https://github.com/Miserlou/Zappa/issues/1790 by M0dM
Hi,
Context
I am using python3.6 and I want 100 lambda to be warm at all time.
Expected Behavior
Keep 100 lambda warm to serve 100 requests concurrently without cold-start.
Actual Behavior
Current keep warm implementation  is only keeping warm a single lambda instance using lambdawrapper could make this a lot more customizable.
Possible Fix
Wrap lambda function using this project ?
https://github.com/robhowley/lambda-warmer-py

This issue seems duplicated with #343. Can we close one to keep the discussion in one place only ?
I wanted to know if someone found a solution to this because I'm really interested about this feature.
AWS Provisionned Concurrency seems to be THE solution but it's not really cheap...


A few errors when trying to start chisel locally

Hello
I have tried cloning the repository then doing the following:
I used yarn to install dependencies first but I get the following error when doing yarn start_dev:
ERROR in ./src/components/elements/MarkdownEditor/MarkdownEditor.js
Module not found: Error: Can't resolve 'react-mde' in '/path/to/folder/src/components/elements/MarkdownEditor'
 @ ./src/components/elements/MarkdownEditor/MarkdownEditor.js 18:0-48 73:6-15 80:33-41
 @ ./src/components/mainArea/content/ContentEdit/elements/ContentString.js
 @ ./src/components/mainArea/content/ContentEdit/ContentEdit.js
 @ ./src/containers/MainArea/Content/ContentEdit/ContentEditContainer.js
 @ ./src/index.js
 @ multi index webpack-hot-middleware/client?reload=true

ERROR in ./src/components/elements/MarkdownEditor/MarkdownEditor.js
Module not found: Error: Can't resolve 'react-mde/lib/styles/css/react-mde-all.css' in '/path/to/folder/src/components/elements/MarkdownEditor'
 @ ./src/components/elements/MarkdownEditor/MarkdownEditor.js 22:0-52
 @ ./src/components/mainArea/content/ContentEdit/elements/ContentString.js
 @ ./src/components/mainArea/content/ContentEdit/ContentEdit.js
 @ ./src/containers/MainArea/Content/ContentEdit/ContentEditContainer.js
 @ ./src/index.js
 @ multi index webpack-hot-middleware/client?reload=true

After getting this, I removed the node_modules folder and tried installing dependencies with npm i getting the following when doing the command:
✗ npm i
npm ERR! code 128
npm ERR! Command failed: /usr/local/bin/git checkout 84fdb197373ecfcb3114876f9da5def8f14d6941
npm ERR! fatal: reference is not a tree: 84fdb197373ecfcb3114876f9da5def8f14d6941
npm ERR!

npm ERR!      /Users/johhansantana/.npm/_logs/2018-07-23T14_37_47_518Z-debug.log

nevermind, did yarn local_run and it worked. Sorry for the noise.


Fix issue that block production doesn't work if block time config exc…

Fix EDGE-409
Description
This PR fixes the issue that the round change always happens if block time configuration exceeds 10 seconds because minimum IBFT timeout is 10 seconds.
Adds new CLI parameters ibft-base-timeout and fix timeout calculation
Changes include

[x] Bugfix (non-breaking change that solves an issue)
[ ] Hotfix (change that solves an urgent issue, and requires immediate attention)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)

Breaking changes
Please complete this section if any breaking changes have been made, otherwise delete it
Checklist

[x] I have assigned this PR to myself
[x] I have added at least 1 reviewer
[x] I have added the relevant labels
[ ] I have updated the official documentation
[ ] I have added sufficient documentation in code

Testing

[x] I have tested this code with the official test suite
[ ] I have tested this code manually

Manual tests
Please complete this section if you ran manual tests for this functionality, otherwise delete it
Documentation update
Please link the documentation update PR in this section if it's present, otherwise delete it
Additional comments
Please post additional comments in this section if you have them, otherwise delete it

I will create a PR on doc repo after a few members approve


Refactor Data Loader to prevent middleman from crashing due to invalid…

… data file.
Use case:
I use middleman to generate some error pages for our proxies. Normally I use middleman server to have a "live" preview while I change the data file with the contents for all error messages. While I work on the error messages it can happend that they contain invalid YAML. Unfortunately that makes the preview server crash. I refactored the data loader a bit.
I use the same pattern here like I did in some components of the preview server. Would you mind to merge it to v4 yourself?

:+1:

This just got refactored on master by @Arcovion, so I don't think it will apply cleanly. I do like this approach, though.

Thanks anyway. :-)


(FACT-1224) Add beaker node defs for Win10(x86,x64)

[skip-ci]

This will be merged down from stable when https://github.com/puppetlabs/facter/pull/1186 is merged.


Use selected theme to set default highlight color setting value

References and relevant issues
Follow up of #6689
Description
Get a default value for the highlight color setting from the current selected theme. The setting is set to the theme corresponding value only if the user has not changed the setting before.
Color exploration
Theme current color

light theme color: rgb(253, 240, 148)
dark theme color: rgb(0, 122, 204)

A preview:

New border_highlight color from the themes

light theme color: rgb(206, 196, 121)
dark theme color: rgb(135, 206, 235)

A preview;

Notes:
Couple of things I'm not completely sure about so I left them as TODOs for the moment and so any feedback related to them is appreciated!:

[x] ~The themes have a highlight value but is a kind of dark color (dark theme: rgb(106, 115, 128), light theme: rgb(163, 158, 156)). Checking, seems like a more suitable color from the current values available in the themes is the current color (dark theme: rgb(0, 122, 204), light theme: rgb(253, 240, 148)). Should the current color from the themes be used or should a new value be added something like data_highlight? If a new value is added to the themes what value it should have for dark and light themes?~
* Go with a new border_highlight color value to set more suitable colors (light theme current color doesn't have much contrast/looks quite invisible)
[x]  ~This PR as it is only covers using the theme to get a color, should also a value from the theme reflect the default highlight thickness value?~
* Use the theme just to get a color, get more feedback regarding the theme usage
[ ] To enable the highlight widgets to get the theme color when the theme changes I had to add some events connections in the AppearanceSettings (adding a _highlight_changed method connected to the highlight.events). Not totally sure if there is a better approach for that or maybe is not necessay and I'm missing something besides the changes done in the AppearanceSettings.update and AppearanceSettings.__setattr__ methods to handle the highlight.highlight_color value when the theme being selected changes 🤔

The themes have a highlight value but is a kind of dark color (dark theme: rgb(106, 115, 128), light theme: rgb(163, 158, 156)). Checking, seems like a more suitable color from the current values available in the themes is the current color (dark theme: rgb(0, 122, 204), light theme: rgb(253, 240, 148)). Should the current color from the themes be used or should a new value be added something like data_highlight? If a new value is added to the themes what value it should have for dark and light themes?

Pretty sure that highlight is for buttons and icons, etc. So IMO we should leave it and make a border_highlight?

I kinda like using the "current" color, it would match the meaning in the GUI sense. On the other hand, that yellow looks invisible, so I'm not sure where actually that's getting used in the gui...
Ultimately, not strong feelings for me here :shrug:

This PR as it is only covers using the theme to get a color, should also a value from the theme reflect the default highlight thickness value?

This seems wrong to me, it feels "outside" the scope of the theme. Buyt maybe it's a hint that the whole thing shouldn't be part of the theme? ^^' sorry, conflicting thoughts :P Let's see what others have to say.

Thanks for the feedback! So I would say:

Pretty sure that highlight is for buttons and icons, etc. So IMO we should leave it and make a border_highlight for the new functionality?

That makes sense :+1:

I kinda like using the "current" color, it would match the meaning in the GUI sense. On the other hand, that yellow looks invisible, so I'm not sure where actually that's getting used in the gui...
Ultimately, not strong feelings for me here 🤷

I think the style files use the theme current color but with some processing done via the darken and lighten functions (which most probably makes things look better 🤔)

This seems wrong to me, it feels "outside" the scope of the theme. Buyt maybe it's a hint that the whole thing shouldn't be part of the theme? ^^' sorry, conflicting thoughts :P Let's see what others have to say.

Yep, having the thickness seems pretty strange to me too 😅 I guess the color value at least is something more related with what the theme provides (colors that the interface uses) so kind of makes sense 🤔 but yep happy to hear what others think too :+1:
In the main time, for the light theme I think the color used for things with darken(current) is something like rgb(206, 196, 121) so I will use that in a new border_highlight color. For the dark theme, I was thinking in using the previous default highlight color, so I will set the skyblue color (rgb(135, 206, 235)) as border_highlight there.
Let me know if you have any further feedback and thanks again for giving this a check!

rgb(206, 196, 121) so I will use that in a new border_highlight color.

Bleh! I kinda hate the way that looks. I'd prefer a slightly darker skyblue (same for the Dark theme, the skyblue is too faint IMO).
(If I had my druthers, in light theme the highlight in the layer list and buttons would also be blue, rather than the yellow, but that's a different issue...)
As I said in the issue, the highlight is "close" to the data being visualized and versus which it needs to provide contrast, which is why I think it should have some sane defaults but be user-editable. It's less important it coordinate with the rest of the viewer window -- as opposed to things like widgets and buttons. So in the theme or not, for me isn't a big deal.
I think if color is in the theme, then so should the thickness, it's just more intuitive I think given the widget presentation.
(I personally think 1 as default is too thin because it's 1 screen pixel so it's super faint on my displays and flat out disappears when zoomed out a bit -- maybe this could consider the DPI scaling? Different PR I guess--I'll keep resetting it to 2 or 3.)
I will say I think the behavior with switching themes is strange:
When I open the preferences in fresh install (default Dark) I have sky-blue in the widget and it highlights as that.
If I switch to Light it shows sky blue in the widget -- not updated? But the actual highlights will be the new sandy color. So it's a bit unclear what's going on.

I rather dislike the way that looks on shapes with the default grey shape edge. It looks better with other shape colors... I'd prefer blue, but this sandy color is consistent with the theme....
(If I had my druthers, in light theme the highlight in the layer list and buttons would also be blue, rather than the yellow, but that's a different issue...)

Maybe an issue regarding a revision for the theme colors could be worthy (I think the yellow and white color combination is quite low contrast so indeed it could be improved towards a better color combination)

As I said in the issue, the highlight is "close" to the data being visualized and versus which it needs to provide contrast, which is why I think it should have some sane defaults but be user-editable. It's much less important it coordinate with the rest of the viewer window (aka the theme) -- as opposed to things like widgets and buttons. So in the theme or not, for me isn't a big deal.

Maybe for the moment is better to just choose a good default color for the highlight that works with the light and dark theme instead of trying to connect it directly with the theme?

I think if color is in the theme, then so should the thickness, it's just more intuitive I think given the widget presentation.
(I personally think 1 as default is too thin because it's 1 screen pixel so it's super faint on my displays and flat out disappears when zoomed out a bit -- maybe this could consider the DPI scaling? Different PR I guess--I'll keep resetting it to 2 or 3.)

Seems to me that connecting the highlight to the theme is not the best then. Maybe actually was is needed instead is to define better default values for the highlight setting (not just color but also thickness)?

I will say I think the behavior with switching themes is strange:
When I open the preferences in fresh install (default Dark) I have sky-blue in the widget and it highlights as that.
If I switch to Light it shows sky blue in the widget -- not updated? But the actual highlights will be the new sandy color. So it's a bit unclear what's going on.

I was able to reproduce this and indeed that's not the intended behavior with the changes here. The idea would be for the widget previewing the highlight color to show the theme defined color (dark theme -> skyblue; light theme -> sandy color ) if the user has not manually changed the selected color and the current selected theme is changed. I accepted the suggestion to change array_equal for allclose and that worked for one run but then I got again the widget not updating when I checked what would happen with clean settings (running napari --reset).  Seems like something else is needed to trigger a widget update for the highlight color after the theme setting changes 🤔

Thanks for exploring this @dalthviz! But yup, after seeing the above conversion, color me convinced :wink:

Created #6781 so closing this. Thanks for the time checking this and all the feedback @psobolewskiPhD @brisvag !


[Bug]  JSON upload not working

Is there an existing issue for this?

[X] Yes, I have searched the existing issues and none of them match my problem.

Product Variant
Cloud (https://rxresu.me)
Current Behavior
When I upload a JSON file it validates it but it doesn't upload any data. I've tried the reactive resume v1 and v2 json file types and still get nothing.

Expected Behavior
I am not sure, i have not used this before
Steps To Reproduce
No response
What browsers are you seeing the problem on?
Chrome
What template are you using?
None
Anything else?
No response

Im also getting this problem

with latest version (main branch on git) the problem doesn't exists. With latest docker release the issue is also present for me.
@AmruthPillai maybe publishing 4.2.0?


Virus Total Report for URL and EXE Check - Please Resolve

Dear Sir,
I have watached the video on Chirs Titus youtube channel and installed it on the system. the browser feels nice but since I want to open some banking website so I thought to check once for virus on virus total webite and below are the results from the virus total webite.
Please remove the bugs & correct the EXE so it will not show the warning on the visus total website.
Result for URL Check on the Wesite provided

Result When I installled it on system and checked its EXE from virus total.

Mark as duplicate


NTP.getTime not public

Good morning, using your Lib as replacement for NTPClientLib.
Everything is running fine till now.
Using NTP.getTime() results in
C:\Users\raffi\Documents\Arduino\libraries\ESPNtpClient-main\src/ESPNtpClient.h:153:10: error: 'void NTPClient::getTime()' is protected
 void getTime ();

Looking in ESPNtpLib.h you commented:
/**
* Starts a NTP time request to server. Returns a time in UNIX time format. Normally only called from library.
* Kept in public section to allow direct NTP request.
*/
void getTime ();
Is there another function to manually fire a request or what i am doing wrong?
Regards,
Ralf

Just fixed in last release. Why do you need that? I'd like to know the use case.

Hi, thanks for your fast reply.
The picture shows a part of a web-interface.
If the user clicks the Button, time sync should happen manually.
Von: Germán Martín<EMAIL_ADDRESS>Gesendet: Dienstag, 1. Dezember 2020 22:26
An: gmag11/ESPNtpClient
Cc: RaffiCologne; Author
Betreff: Re: [gmag11/ESPNtpClient] NTP.getTime not public (#4)
Just fixed in last release. Why do you need that? I'd like to know the use case.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/gmag11/ESPNtpClient/issues/4#issuecomment-736830411 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AR5PUBLXMSFCJFC3EPSSZK3SSVNOPANCNFSM4UIXF3PA .Das Bild wurde vom Absender entfernt.

Thanks, then I close this issue


Update SketchAggregator.java for removing duplicated parentheses

Fixes nothing.
Description
It seems that here comes a couple of deplicated parentheses.
Is it better to remove it for a better view?

Key changed/added classes in this PR

no class is modified.

This PR has:

[x] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

LGTM failed caused by read time out. Should i push a commit to trigger the CI to try again?
Log from LGTM analysis
[2021-04-08 15:46:43] [build-stdout] [2021-04-08 15:46:43] [autobuild] [ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:3.0.0:check (validate) on project druid: Execution validate of goal org.apache.maven.plugins:maven-checkstyle-plugin:3.0.0:check failed: Plugin org.apache.maven.plugins:maven-checkstyle-plugin:3.0.0 or one of its dependencies could not be resolved: Failed to collect dependencies at org.apache.maven.plugins:maven-checkstyle-plugin:jar:3.0.0 -> org.apache.maven:maven-core:jar:3.0 -> org.apache.maven:maven-settings-builder:jar:3.0 -> org.sonatype.plexus:plexus-sec-dispatcher:jar:1.3: Failed to read artifact descriptor for org.sonatype.plexus:plexus-sec-dispatcher:jar:1.3: Could not transfer artifact org.sonatype.plexus:plexus-sec-dispatcher:pom:1.3 from/to central (https://repo.maven.apache.org/maven2): Read timed out -> [Help 1]

@suneet-s
Hello, all checks have passed. Could you review again?

Thanks @MakDon


fixed tooltip

Is there a way to fixed tooltip?
I need this to indicate a update in on page

What do you mean by "fixed tooltip"?

Bootstrap has a similar behavior but use javascript

This can be done, but would require a few changes. I may take a look at it this week.

Commit 58d2cae added this option. Now you can use the attribute data-balloon-visible to force tooltips visible even when not hovering the element.


Feature Request: fading on color change in Static mode

First of all, many many thanks Tobias for this, i searched a lot for a valid code and i've been using yours for a year by now. So thanks again!
I was wondering if it's possibile to edit/add a fading between colors in static mode, because i doubt a little it could be hardware related. I'm no programmer but feel free to point me where to edit the code, I'll try my best.
I use mqtt to control the board, via openHab. (pin 12, 300 led ws2811)
Many thanks in advice!

You can use "rainbow" effect and slow the speed to your liking? Or are you referring to fading when you select a new color? All animations are dependent on WS2812FX, and I would ask you to refer to https://github.com/kitesurfer1404/WS2812FX/issues/124 to get some idea if latter is your intention.

You can use "rainbow" effect and slow the speed to your liking? Or are you referring to fading when you select a new color? All animations are dependent on WS2812FX, and I would ask you to refer to kitesurfer1404/WS2812FX#124 to get some idea if latter is your intention.

Thanks for the reply!
Yes i was referring to the static mode, so it would be great to see a fading between the old color and the new selected one ( i mean, like commercial products).
Many thanks for the link, i'll check it!
To be honest, i thought at this feature because the fastest way i found to control the nodemcu+mclighting with Google Assistant was by "cloning" (in openHab) the commands received by a Yeelight lamp, and in testing phase i spotted the difference (the yeelight fades, mclighting not).
I'll post my result, if positive!

+1 for this feature, and in addition, support for the "transition" feature from Home Assistant, which lets you set how many seconds the transition should take. Currently, if you define a transition time from HA, the lights don't react at all.


Getting an error while building and pushing container images to registry

Hello.
I am following this article https://learn.microsoft.com/en-us/azure/aks/tutorial-kubernetes-prepare-acr?tabs=azure-powershell
When I try to build and push a container image using this command:
az acr build --registry $ACRNAME --image aks-store-demo/product-service:latest ./src/product-service/
I am getting the following error message:
"'./src/product-service/' doesn't exist."
Am I doing something wrong?  Can you help me with that?
Thanks.

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

ID: 6afe8c14-efa4-31a6-9fd9-e68f2d878226
Version Independent ID: c67857a5-ee1b-3ed2-f281-dd62c4c4c6f3
Content: Kubernetes on Azure tutorial - Create an Azure Container Registry and build images - Azure Kubernetes Service
Content Source: articles/aks/tutorial-kubernetes-prepare-acr.md
Service: azure-kubernetes-service
GitHub Login: @schaffererin
Microsoft Alias: schaffererin

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

#reassign:schaffererin

Hi, @DenizFeital. Thank you for your feedback.
I just ran through the testing of this tutorial (and the one previous) and didn't run into any issues pulling the image in question. Did you follow all steps in https://learn.microsoft.com/en-us/azure/aks/tutorial-kubernetes-prepare-app?tabs=azure-cli? Did you change into the "aks-store-demo" directory?
As there are no issues with the published steps, I'm going to close out this issue. Please feel free to open a new issue if you run into any future problems with the AKS content.
Thanks again! #please-close


Laravel 7 with ionic 5 have problem about Cors Policy

When i try to post or get something on the API it return always  XMLHttpRequest at **************' from origin '******' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.
This is my middleware CORS :
    {
        return $next($request)
            ->header('Access-Control-Allow-Origin', "*")
            ->header('Access-Control-Allow-Methods', "GET, POST, PUT, PATCH, DELETE, OPTIONS")
            ->header('Access-Control-Allow-Headers', "Origin, X-Requested-With, Accept, Content-Type, Authorization");
    }```

And this is my Kernel :

protected $middleware = [
        // \App\Http\Middleware\TrustHosts::class,
        \App\Http\Middleware\TrustProxies::class,
        // \Fruitcake\Cors\HandleCors::class,
        \App\Http\Middleware\CheckForMaintenanceMode::class,
        \Illuminate\Foundation\Http\Middleware\ValidatePostSize::class,
        \App\Http\Middleware\TrimStrings::class,
        \Illuminate\Foundation\Http\Middleware\ConvertEmptyStringsToNull::class,
        \App\Http\Middleware\Cors::class,
    ];

Can someone help me ?

So you created your own middleware?

i already use Fruitcake cors
But the same issus so i create my own middleware

@barryvdh  can you help me please ?

I'm having a similar issue using this config/cors.php hitting an endpoint like https://site.com/api/endpoint.  Seems like an issue with POST using a header with a Bearer token and post body.  We're also using Laravel 7 with Ionic 5.
<?php

return [

    /*
    |--------------------------------------------------------------------------
    | Cross-Origin Resource Sharing (CORS) Configuration
    |--------------------------------------------------------------------------
    |
    | Here you may configure your settings for cross-origin resource sharing
    | or "CORS". This determines what cross-origin operations may execute
    | in web browsers. You are free to adjust these settings as needed.
    |
    | To learn more: https://developer.mozilla.org/en-US/docs/Web/HTTP/CORS
    |
    */

    'paths' => ['api/*'],

    'allowed_methods' => ['*'],

    'allowed_origins' => ['*'],

    'allowed_origins_patterns' => [],

    'allowed_headers' => ['*'],

    'exposed_headers' => [],

    'max_age' => 0,

    'supports_credentials' => false,

];

I was able to fix this with Ionic doing this.
https://forum.ionicframework.com/t/ionic-4-laravel-cors/165763/6
        const headers = new HttpHeaders({
          'Authorization': "Bearer " + token,
          'Content-Type': 'application/json',
          'Access-Control-Allow-Origin': '*',
          'Access-Control-Allow-Headers': '*',
          'Accept': 'application/json, text/plain'
        });

@davidquon I'm not sure how that is relevant, because this is a PHP package, not nodejs.

@barryvdh This issue specified using Ionic 5 (client) to talk with a Laravel 7 API (server) which is using fruitcake/laravel-cors which is what is having issues when sending an authorization header like below which I've done with prior versions of Laravel when using spatie/laravel-cors.
      const headers = new HttpHeaders({
        'Authorization': "Bearer " + token,
        'Content-Type': 'application/json'
      });

If I send a POST with a header like https://github.com/fruitcake/laravel-cors/issues/477#issuecomment-664709886 then fruitcake/laravel-cors works as expected otherwise I receive a CORS error as specified in this issue.  I thought this information might be helpful in determining the problem while also providing a workaround for others as well.  While the JS code may not be correct for every language as a work around sending that type of header will hopefully help fix others having this CORS issue as well.

Just to be a bit clearer https://github.com/fruitcake/laravel-cors/issues/477#issuecomment-664709886 is what I did using Ionic 5 (client-side) to make the Laravel 7 API (server-side) return the correct data and work around the CORS issue specified.  I did not create my own middleware though for CORS and configured fruitcake/laravel-cors like this https://github.com/fruitcake/laravel-cors/issues/477#issuecomment-662146566 which should in theory accept all incoming requests unless I'm configuring something incorrectly.  Thanks for the help @barryvdh.

Setting supports_credentials to true helped solve it as well for me per this comment.
https://github.com/fruitcake/laravel-cors/issues/478#issuecomment-664827785
Then I'm able to set the header to this in Ionic on the client-side:
      const headers = new HttpHeaders({
        'Authorization': "Bearer " + token,
        'Content-Type': 'application/json'
      });

and have the Laravel API respond correctly.
I saw supports_credentials but reading the comment above it thought it was for something else.  😥  Thanks for the help @barryvdh as I was hoping it was something like that.  👍

Hello i use ionic 4  for the front and laravel 7 for the backend  and i have this error  when i login  with the front : Access to XMLHttpRequest at 'http://localhost:8000/api/connexion' from origin 'http://localhost:8100' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource.       but i dont understand before the app run correctely when i use composer update for another  fonctionnality of my app mobile, the cors is wrong

+1

Is there any news on this? I have been using the older Laravel package for years without issues without issue, but for whatever reason I cannot get this package to work. The specific message I am getting is  Access-Control-Allow-Origin cannot contain more than one origin., which seems weird. I really cannot tell if this issue is being generated from a bad response, or if there was a recent security change on mobile devices that is now causing this.

That most likely is because you are adding headers anywhere else (nginx/apache, php `header())

As a note... It actually turned out to be caused by a temp block of code that was added to the project 5 years ago, which I did not find until today! @barryvdh thanks for the suggestion, and I appreciate the time you put into these projects.

I'm having a similar issue using this config/cors.php hitting an endpoint like https://site.com/api/endpoint. Seems like an issue with POST using a header with a Bearer token and post body. We're also using Laravel 7 with Ionic 5.
<?php

return [

    /*
    |--------------------------------------------------------------------------
    | Cross-Origin Resource Sharing (CORS) Configuration
    |--------------------------------------------------------------------------
    |
    | Here you may configure your settings for cross-origin resource sharing
    | or "CORS". This determines what cross-origin operations may execute
    | in web browsers. You are free to adjust these settings as needed.
    |
    | To learn more: https://developer.mozilla.org/en-US/docs/Web/HTTP/CORS
    |
    */

    'paths' => ['api/*'],

    'allowed_methods' => ['*'],

    'allowed_origins' => ['*'],

    'allowed_origins_patterns' => [],

    'allowed_headers' => ['*'],

    'exposed_headers' => [],

    'max_age' => 0,

    'supports_credentials' => false,

];

Hi guys! In my case the problem is paths. I use Passport so my route to authorize is /oauth/token. In paths i have pattern only for api/. So i add all path * instead of api/. Everything works!


Update links to mainnet

Make sure that for the mainnet launch, all links that point to testnet or stagenet are updated to point to mainnet.

Will include in launch smoke test.


Changes Made for New DB Actions - WMS 875

WMS ID - 875
DB Action Changes for the New UI

Please review and approve the changes for this live lab. DB Action changes requested by the DB Team. Deadline May 29

Please check now..


IPR now with some common info

This works now. Beautifyid the layout with tabs and soft line breaks., clearing some PEP8 glitches while digging around.

Ok.  Sorry it took so long for me to get to this. This looks good except for one thing:  You're basically repeating the same code twice.  Maybe you could make a MakeDspyParams(...) method which returns the string with the dspyparams and call it from both places?

That make sense. I'll do it.


Update

many chore

请规范pr信息及commit规范，详细参考contributor.md

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Webview Can support cookie?

In some issues here, i see vscode developer commented that webview couldn't support cookie.
But in some websites as below, i can see cookie set successed like this.
But again my http website can't work.
Why???

See #120869

See #120869

0.0  I got you. But the websites used internally by the company are all HTTP, not HTTPS.
Does it mean the websites should change to https and set cookie 'Samesite=None; Security'

It's worth trying.

We have a great extension developer community over on GitHub discussions and Slack where extension authors help each other. This is a great place for you to ask questions and find support.
Happy Coding!


Review all Icons to Ensure Consistency in Format and Design

We should review all the icons in the current package and clean house - anything that needs improving/replacing, and we might want to think about separating the 3rd party logos into their own directory.

Related PR https://github.com/rancher/icons/pull/45


Why is the Google calendar url being modified to /full instead of /basic? -> 403 forbidden

Dear ui calendar team,
I'm trying to get the events from my (or any) google calendar feed but with no success.
I keep getting 403 forbidden errors in the inspector because the url as given in the config (which ends in /basic) is never called, the url is being rewritten to /full which always returns 403 forbidden errors.
Even the online demo has this problem.. I've tried searching where this replacement is being done but without success.
Can anyone elaborate on how this is possible?
Kind regards,
Erik aka 4levels

Ok, I just found where this replacement is being done, in the included gcal.js on line 49
It seems like the gcal.js is way outdated (version 1.5.3 while fullcalendar is being set to 2.1.1)
Seems like this issue is thus related to the issue #230..


Use new AuthenticationStrategyInterface

Depends on https://github.com/cyberark/conjur-api-python/pull/22
Desired Outcome
Use new AuthenticationStrategy interface created in https://github.com/cyberark/conjur-api-python/pull/20. This lays the groundwork for adding LDAP authentication support.
Implemented Changes

Pass instance of AuthenticationStrategyInterface to Client constructor
Add optional authn_type field to .conjurrc (defaults to authn)

Connected Issue/Story
CyberArk internal issue link: ONYX-20429
Changelog

[ ] The CHANGELOG has been updated, or
[x] This PR does not include user-facing changes and doesn't require a
CHANGELOG update

Test coverage

[x] This PR includes new unit and integration tests to go with the code
changes, or
[ ] The changes in this PR do not require tests

Documentation

[ ] Docs (e.g. READMEs) were updated in this PR
[ ] A follow-up issue to update official docs has been filed here: insert issue ID
[x] This PR does not require updating any documentation

Behavior

[ ] This PR changes product behavior and has been reviewed by a PO, or
[ ] These changes are part of a larger initiative that will be reviewed later, or
[x] No behavior was changed with this PR

Security

[ ] Security architect has reviewed the changes in this PR,
[ ] These changes are part of a larger initiative with a separate security review, or
[x] There are no security aspects to these changes

@cyberark/community-and-integrations-team Can we please get a review on this so we can move forward on #411?


How to evaluate my own trained model?

Hi,
I've trained a MultiNet2 (segmentation and detection) model. I was wondering how I can evaluate it on the validation set?
There is no evaluate.py in the root folder. I tried to run the evaluate.py in the submodule folder but didn't succeed. Perhaps I missed something. I'd appreciate it if you could give me some instructions. Thanks.

Actually, I got evaluation results in the log file when training the model. But there are "raw" and "smooth" results. Which one shall I use? Thanks.


Upgrade walls when walls are maxed

When the walls are maxed out you still have a button that says "Upgrade 11".

Let me know if the project kicks off again. :smile:


Delete onboarding message

closes #120
Summary:

Edit mode  and exit edit mode features are added.
In the edit mode, a trash icon is shown in marker tooltip.
On clicking, it deletes the onboarding message
When all the onboarding messages for a onboarding stages is deleted. The onboarding stage is also removed.

Screenshots:

merged with #127


Templing with classes does not fail if expected

Given a similar template as you have in the documentation "How to write a template".
title: String
interactive: Boolean
seats: Int
occupancy: Float
duration: Duration
`abstract`: String
class Event {
-  name: String
+  name: String?
-  year: Int
+  year: Int?
}
event: Event
instructors: Listing<String>
class Session {
  time: Duration
  date: String
}
sessions: Listing<Session>
assistants: Mapping<String, String>

The implementation leaves out the event property:
title = "Pkl: Configure your Systems in New Ways"
interactive = true
seats = 100
occupancy = 0.85
duration = 1.5.h
`abstract` = """
  With more systems to configure, the software industry is drowning in repetitive and brittle configuration files.
  YAML and other configuration formats have been turned into programming languages against their will.
  Unsurprisingly, they don’t live up to the task.
  Pkl puts you back in control.
  """
-event {
-  name = "Migrating Birds between hemispheres"
-  year = 2024
-}
instructors {
  "Kate Sparrow"
  "Jerome Owl"
}
sessions {
  new {
    date = "2/1/2024"
    time = 30.min
  }
  new {
    date = "2/1/2024"
    time = 30.min
  }
}
assistants {
  ["kevin"] = "Kevin Parrot"
  ["betty"] = "Betty Harrier"
}

Expected:
Error with message

Tried to read property event but its value is undefined.

But I get:
Not error, outpout contains
event {
  name = null
  year = null
}

For me looks wrong.
event: Event from the template is not overriden so it should fail.
Fun fact:
If I leave a property within Event non-nullable, it will fail with the correct error message:

Tried to read property name but its value is undefined.
...
The above error occurred when rendering path event.name of module "..."

Another (simpler) example:
Given the following template:
class On {
  first: Int?
  second: Int?
  third: Int?
}

on: On

My template expect to override On.
However, non of the properties of On are required to set.
Implementation:

☝ No joke, just the amend statement of course 😁
What will be rendered?
on {
  first = null
  second = null
  third = null
}

Technically this behaviour seems to be correct.
It might be the case that pkl will "simply" add the "null class" for you to the implementation.
Because not defining on at all or using on {} will lead to the same output.
The latter (on {}) is perfectly fine and shouldn't print an error,
Nevertheless, I think we should fail here because it behaves differently to other errors that says "value is undefined".

Most non-primitive types in Pkl have a default value. The default value of nullable types is null.
If you want to force people to give a value, you have two options:

Make the property non-null. This indicates that it has to be provided and can't be left empty.
Make the default an error: first: Int? = throw("error: no value provided for property 'first'")

Is this invalid output?
on: {}

If this is valid, then there is nothing to do here.
If it this invalid, then you will want to think about what the validation rules are, and define them in your template.
For example, maybe you want to say that at least one sub-property is defined. If so, then you can do:
local const hasAtLeastOneProperty = (it: On) -> it.first != null || it.second != null || it.third != null

on: On(hasAtLeastOneProperty)

Then, Pkl will throw if some property is not defined.
Alternatively, if on: {} is valid, but you would rather omit this from your output altogether if it's empty. In that case, you can just make the property nullable, and have your renderer omit null properties.
For the YAML use-case, it already does omit null properties so nothing else needs to be set.
on: On?

output {
  renderer = new YamlRenderer {}
}

@stackoverflow I was not aware of the concept of default properties 🤯 thanks for sharing! Know it make sense!
@bioball thanks for the explanation!
Indeed, on {}without a single property set is invalid. Seems I wanna introduce a logic you provided.
Thanks for your support here guys.
Much appreciated.
Going to close it as it everything works as expected.


feat(website-frontend): add new homepage

#385
Hacktoberfest results:
Big thanks to @xandervedder @jensk-dev @JairJosafath @Aidan-Marcus
This is the first stage of the complete website rework using lit-element and the new stryker component library.

The repository page is temporary but

A few things that are missing from this PR that should be added:

consistent prefixing of components, using the sme prefix.
a testing suite that's either testing component logic using a simulated dom like vitest with happydom, or an in-browser test runner like karma or @web/test-runner
generated source maps for .js files. Can be generated by setting build.sourcemap to true in vite.config.ts
generated declaration maps for .ts files. Can be generated by setting the declaration and declarationMap properties to true in tsconfig.json

magic css values should really use the closest representation of the value in tailwind. E.g. width: 13px should be converted to @apply w-3 which resolves as width: 12px

Oh and also run the linter 🙂.

To make it a bit more easier for our selves, I will transfer all the issues from the fork over to the real repository. Let me know if you still want to contribute to this, I will add you as a contributor 🙂.

Yes, I'd like to contribute! Please add me to the list
Regards ,
Jair
On Fri, Oct 27, 2023, 10:04 AM Xander Vedder @.***>
wrote:

To make it a bit more easier for our selves, I will transfer all the
issues from the fork over to the real repository. Let me know if you still
want to contribute to this, I will add you as a contributor 🙂.
—
Reply to this email directly, view it on GitHub
https://github.com/stryker-mutator/stryker-dashboard/pull/392#issuecomment-1782478575,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN3KIYLPXVOUTRC4N3OJLLLYBNTHVAVCNFSM6AAAAAA6APS7ZWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBSGQ3TQNJXGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Superseded by #401 401


feather is breaking the build

https://github.com/cosmology-tech/cosmos-kit/actions/runs/4826644011/jobs/8598666858

PR is up, waiting for TFL ..


Persist app settings and support deployment from directory

https://user-images.githubusercontent.com/17034772/221951809-e10be35c-db03-4c62-9c22-8077a6433fb5.mov

THIS IS AMAZING!


feat(morgan): add ECS tracing fields if APM is in use

Refs: #35

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Build Cause: Pull request #40 opened

Start Time: 2021-01-20T19:54:53.979+0000

Duration: 8 min 5 sec

Commit: 4dfeede9ac1c0df56ed4bd4eeb22be8fc83a6aca

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Build Cause: Pull request #40 opened

Start Time: 2021-01-20T19:54:53.979+0000

Duration: 8 min 5 sec

Commit: 4dfeede9ac1c0df56ed4bd4eeb22be8fc83a6aca