[FLINK-8516] Allow for custom hash function for shard to subtask mapping in Kinesis consumer

What is the purpose of the change
Allow the user to customize Kinesis shard to subtask assignment in the Kinesis consumer.
Brief change log
Added pluggable shard assigner.
Verifying this change
Added unit test.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation
Javadoc

@tzulitai the PR is ready for review now

@tzulitai What is the proposed migration test going to assert? The assigner does not influence how state is saved and restored. Even when the assigner returns invalid index, the modulus will ensure that the shard gets assigned.

@tzulitai please see changes and couple questions.

@tweise regarding what the proposed migration test is going to assert:

The assigner does not influence how state is saved and restored. Even when the assigner returns invalid index, the modulus will ensure that the shard gets assigned.

This is exactly what I think the test is verifying, without any knowledge of what internally is happening. Using a different custom assigner across restores should not result in any state lose / change.

@tzulitai Assignment and restore are orthogonal; this PR doesn't change restore. I therefore don't see the need to add another migration test for it. There is also no change to how the index is computed by default, it is just that part of what happened earlier in isThisSubtaskShouldSubscribeTo is now in the assigner. Should I add a (trivial) unit test that asserts that isThisSubtaskShouldSubscribeTo applies modulus to assigner returned value that falls outside the subtask index range?

@tweise ok, I think I can agree on your last comment.

Should I add a (trivial) unit test that asserts that isThisSubtaskShouldSubscribeTo applies modulus to assigner returned value that falls outside the subtask index range?

That sounds good! Once we add that, I think we're good to go.

@tzulitai done adding test
(CI failure BackPressureStatsTrackerITCase.testBackPressuredProducer unrelated to this PR)

@tzulitai any update?

@tweise changes look good, will merge this to master.
Thanks a lot for the work!

@tzulitai thanks for the review!


Hasher is not invariant on small writes with .write vs .write_u8

Consider the following program:
use ahash; // 0.7.6
use std::hash::Hasher;

fn main() {
    let mut hash1 = ahash::AHasher::default();
    let mut hash2 = ahash::AHasher::default();

    let input = b"12345".as_ref();
    hash1.write(input);
    for k in input.iter() {
        hash2.write_u8(*k);
    }
    assert_eq!(hash1.finish(), hash2.finish());
}

We would assume hash1 and hash2 produce the same value, having the same sequence of bytes written, however the assertion fails.
https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=8d4837e9666e0c75293a387435885200
The assertion passes using the std:: SipHasher
https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=7d89099aa02615fe5ce5e8065767fcf9

Small sized .write() is also not consistent
https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=1b56400d019ae6c46bc756e328ad6159

As in aside, wyhash, fxhash are also not immune to this symptom of stretching single or small writes to much larger hash states than intended. The stdlib hasher is the only one so far that gets this correct so far.
A naive buffering-before-flushing hasher would require fn finish(&mut self) which isn't directly possible.

This is not a guaranteed property of the Hasher trait. So it is not accurate to refer to this as being "correct". It is simply an additional property which sipHash happens to offer that ahash, wyhash, and fxhash do not. As there are MAJOR performance gains from not supporting this feature, and because it is not required by the trait or by any  hashmap implementation that I am aware of, supporting it is not a priority.

Curious question: Why should the two be the same?  I'd actually assume them to be different.
One is a slice of five bytes.
One is five consecutive single bytes.
Even the types are different.  Why should they hash to the same thing?
Case in point:  consider:
struct MyStruct(i32, i32);

let x = MyStruct(1,2);
let y: i64 = 0x0000000100000002;

x.hash() == y.hash() ?

The fact that two types have the same byte representation does not mean that they should hash to the same thing...

This is in fact called out explicetly in the final paragraph of the top section here: https://doc.rust-lang.org/std/hash/trait.Hasher.html


java.lang.RuntimeException: Unable to pre-dex

Application working fine when I am build and run but I am facing below error when generate release APK using ionic cordova build --release android command
Execution failed for task ':transformClassesWithPreDexForRelease'.
> com.android.build.api.transform.TransformException: java.lang.RuntimeException
: java.lang.RuntimeException: Unable to pre-dex 'C:\Users\Dhaval\.gradle\caches\
modules-2\files-2.1\xerces\xmlParserAPIs\2.6.2\65acede1e5305bd2b92213d7b5761328c
6f4fd9\xmlParserAPIs-2.6.2.jar' to 'E:\Relax.java\application\SanManDriverIonicO
ldDesign\platforms\android\build\intermediates\transforms\preDex\release\32.jar'

Full stack trace :
trouble processing "javax/xml/namespace/QName.class":
Ill-advised or mistaken usage of a core class (java.* or javax.*)
when not building a core library.
This is often due to inadvertently including a core library file
in your application's project, when using an IDE (such as
Eclipse). If you are sure you're not intentionally defining a
core class, then this is the most likely explanation of what's
going on.
However, you might actually be trying to define a class in a core
namespace, the source of which you may have taken, for example,
from a non-Android virtual machine project. This will most
assuredly not work. At a minimum, it jeopardizes the
compatibility of your app with future versions of the platform.
It is also often of questionable legality.
If you really intend to build a core library -- which is only
appropriate as part of creating a full virtual m

Hi @dhavaljivani! Are you still having this problem? Can you run the ionic info command from your project folder and paste the output here so we can have more information about your environment? Thanks.

Im having the same issue. cordova build works fine, but cordova build --release does not.
Here is my ionic info
`cli packages: (/home/tunte/WebProjects/Spot/dogwalkmobileapp/node_modules)
@ionic/cli-utils  : 1.19.2
ionic (Ionic CLI) : 3.20.0

global packages:
cordova (Cordova CLI) : 8.0.0

local packages:
Cordova Platforms : android 6.4.0
Ionic Framework   : ionic1 1.2.4

System:
Node : v8.10.0
npm  : 5.6.0
OS   : Linux 4.13

Environment Variables:
ANDROID_HOME : not set

Misc:
backend : pro

`

@UISTRepo Can you paste your error output? Thanks.

Its identical as dhavaljivani reported it above, only the paths are different because i use Linux.
I solved it by downgrading cordova android platform from 6.4.0 to 6.3.0.
This was caused by mauron85 background geolocation plugin.
Everything was fine when you build it for debug, but when you build it for release it throws that error.

@UISTRepo Then the problem seems to be related to cordova-plugin-mauron85-background-geolocation and not Ionic.
@dhavaljivani Are you also using this plugin? If not, what are the plugins you're using? You can paste the output of cordova plugin ls.

@FdezRomero Same Issue here
> cordova plugin ls call-number 0.0.2 "Cordova Call Number Plugin" com.synconset.imagepicker 2.1.8 "ImagePicker" cordova-plugin-appavailability 0.4.2 "AppAvailability" cordova-plugin-compat 1.2.0 "Compat" cordova-plugin-console 1.1.0 "Console" cordova-plugin-device 1.1.7 "Device" cordova-plugin-facebook4 1.9.1 "Facebook Connect" cordova-plugin-file 4.3.3 "File" cordova-plugin-geolocation 4.0.1 "Geolocation" cordova-plugin-googlemaps 1.4.5 "cordova-googlemaps-plugin" cordova-plugin-googleplus 5.2.1 "Google SignIn" cordova-plugin-inappbrowser 1.7.2 "InAppBrowser" cordova-plugin-splashscreen 4.1.0 "Splashscreen" cordova-plugin-statusbar 2.4.1 "StatusBar" cordova-plugin-whitelist 1.3.3 "Whitelist" cordova-plugin-x-toast 2.6.0 "Toast" cordova.plugins.diagnostic 3.9.1 "Diagnostic" ionic-plugin-keyboard 2.2.1 "Keyboard" phonegap-plugin-push 1.10.6 "PushPlugin" twitter-connect-plugin 0.6.0 "Twitter Connect"

cordova plugin ls
call-number 0.0.2 "Cordova Call Number Plugin"
com.synconset.imagepicker 2.1.8 "ImagePicker"
cordova-plugin-appavailability 0.4.2 "AppAvailability"
cordova-plugin-compat 1.2.0 "Compat"
cordova-plugin-console 1.1.0 "Console"
cordova-plugin-device 1.1.7 "Device"
cordova-plugin-facebook4 1.9.1 "Facebook Connect"
cordova-plugin-file 4.3.3 "File"
cordova-plugin-geolocation 4.0.1 "Geolocation"
cordova-plugin-googlemaps 1.4.5 "cordova-googlemaps-plugin"
cordova-plugin-googleplus 5.2.1 "Google SignIn"
cordova-plugin-inappbrowser 1.7.2 "InAppBrowser"
cordova-plugin-splashscreen 4.1.0 "Splashscreen"
cordova-plugin-statusbar 2.4.1 "StatusBar"
cordova-plugin-whitelist 1.3.3 "Whitelist"
cordova-plugin-x-toast 2.6.0 "Toast"
cordova.plugins.diagnostic 3.9.1 "Diagnostic"
ionic-plugin-keyboard 2.2.1 "Keyboard"
phonegap-plugin-push 1.10.6 "PushPlugin"
twitter-connect-plugin 0.6.0 "Twitter Connect"

@aelshemy16 You have some plugins that have been deprecated and are no longer necessary, like cordova-plugin-compat and cordova-plugin-console: https://cordova.apache.org/news/2017/09/22/plugins-release.html
Can you post of the output of ionic info?
Thanks


Refactor Package.swift to support sub modules

Introduce sub modules in Package.swift, similar to how pod subspecs are organized

Being tested in the temp repo

is this still active ? would love to see spm support sub modules.

@mikechout this is something that I'd like to do but just haven't had the time. You're more than welcome to create a PR.

Cool, I'll try to have a look and see if i can PR

@mikechout have a look at https://github.com/SwifterSwift/SwifterSwift/pull/1187 and tell me what you think.

Cool, it looks good, thanks for the quick implement. Thanks~~


Invisible menu & nav buttons

[x ] I am on the latest Hyper.app version
[x ] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name:
Windows 8.1 Enterprise
Hyper.app version:
2.0.0
Link of a Gist with the contents of your .hyper.js:
Note - Issue was present with the default config
https://gist.github.com/mkempster22/b40f9e3f9a5b285cd3f7d8bf96dadf00
Relevant information from devtools (CMD+ALT+I on macOS, CTRL+SHIFT+I elsewhere):
N/A
The issue is reproducible in vanilla Hyper.app:
No plugins etc, changed shell to bash. issue was present before this

Issue

I've installed hyper a couple of times but each time all of the top nav buttons (file, minimise, close etc.) are invisible, I am still able to click them by guessing where they are and if I change the config to left the also move to the correct place but they still don't show.
I installed this on a windows 10 computer at home and they were showing fine.

This is probably an electron issue and will most likely not get fixed for windows 8. Upgrade your OS :)


H264 Loop back fails

I tweaked the Huebot sample code to receive H264 frames instead of the NV12 frames from the video socket and I tried to send back the frame over the video socket. However, I haven't been able to successfully get the media to loopback to the teams meeting.
I changed the following in the Huebot

Changed the CreateLocalMediaSession (in Bot.cs) to set the ReceiveColorFormat to VideoColorFormat.H264 instead of VideoColorFormat.NV12. And also changed the SupportedSendVideoFormats to include the supported H264 formats instead of the VN12 formats.
Change the GetSendVideoFormat (in Utilities.cs) to return the appropriate H264 format for a given width.
Changed OnVideoMediaReceived to copy the incoming H264 buffer ti another buffer and send it to the remote. The Bot does not change the hue or take snapshots.
Following is the changed code I made to create a copy of the H264 buffer
// Use the real length of the data (Media may send us a larger buffer)
VideoFormat sendVideoFormat = e.Buffer.VideoFormat.GetSendVideoFormat();
int size = (int)e.Buffer.Length;
IntPtr ptr = Marshal.AllocHGlobal(size);
CopyMemory(ptr, e.Buffer.Data, size);
var videoSendBuffer = new VideoSendBuffer(ptr, size, sendVideoFormat);
this.Call.GetLocalMediaSession().VideoSocket.Send(videoSendBuffer);

The looped back video is not seen in the teams meeting.
The send on the first few frames does not throw any exception however, subsequent ones continuously throws the following exception
{
"Timestamp": "2018-11-08T10:01:09.4860889-06:00",
"ProviderName": "Sample.HueBot",
"Id": 1,
"Message": "Sample.HueBot(CallHandler.cs:317 OnVideoMediaReceived) 2018-11-08T16:01:13.6035038Z: [301a0400-326b-4d67-bc79-3aa784ce0f21] Exception in VideoMediaReceived
exception: Exception of type 'Microsoft.Skype.Bots.Media.SkypeMediaException' was thrown.
Deliver video source media buffer
StackTrace:    at Microsoft.Rtc.Internal.Media.MediaApi.VideoSourceConsumer.DeliverBuffer(MediaBuffer A_0)
at Microsoft.Skype.Bots.Media.SourceConsumerWrapper.DeliverBuffer(MediaBuffer buffer, BufferDisposeDelegate bufferDisposeDelegate)
, Properties: null",
"ProcessId": 8776,
"Level": "Informational",
"Keywords": "0x0000F00000000000",
"EventName": "Message",
"ActivityID": "0000518f-0001-0000-4822-0000ffdcd7b5",
"RelatedActivityID": null,
"Payload": {
"message": "Sample.HueBot(CallHandler.cs:317 OnVideoMediaReceived) 2018-11-08T16:01:13.6035038Z: [301a0400-326b-4d67-bc79-3aa784ce0f21] Exception in VideoMediaReceived
exception: Exception of type 'Microsoft.Skype.Bots.Media.SkypeMediaException' was thrown.
Deliver video source media buffer
StackTrace:    at Microsoft.Rtc.Internal.Media.MediaApi.VideoSourceConsumer.DeliverBuffer(MediaBuffer A_0)
at Microsoft.Skype.Bots.Media.SourceConsumerWrapper.DeliverBuffer(MediaBuffer buffer, BufferDisposeDelegate bufferDisposeDelegate)
, Properties: null"
}
}
If someone could throw light on what could be going wrong here would be great.

Could you start looking through our other sample that demonstrates H.264 streaming?  It is not an echo bot scenario like what you are attempting (great idea btw) but plays back pre-recorded streams.
https://github.com/microsoftgraph/microsoft-graph-comms-samples/blob/master/Samples/LocalMediaSamples/AudioVideoPlaybackBot/FrontEnd/Bot/BotMediaStream.cs

@vganapathy-lifesize
Hello,
It is not appropriate to loop back video using the H.264 encoded video format, especially if the bot is joined to a meeting or group call. Use of the H.264 format is to support advanced scenarios where the  bot is receiving its video source from its own H. 264 video encoder, and can respond promptly to incoming VideoKeyFrameNeeded events (to supply an IDR frame) and changes in the PreferredEncodedVideoSourceFormats property (given by the VideoSendStatusChanged event) to adapt to requested resolution/framerate/bitrate. The bot must ensure that it does not send video at a bitrate exceeding the requested VideoFormat.BitRate value.
If joined to a meeting, the bot should also be able to supply H.264 encoded video in multiple formats concurrently (per its VideoSocketSettings.MaxConcurrentSendStreams setting); for example, the bot may be requested to supply its video source at both 720p/30fps and 240p/15fps simultaneously.
With loopback, the bot does not have sufficient control over what video resolution size, framerate, and bitrate it is receiving to ensure it can satisfy what formats it is required to send.
Thanks,
Stephen

@ssulzer Thanks for your clarification. Fair enough, I can perfectly understand what you are saying. That's eventually what we are trying to get to. Right now I am at a very initial stage trying to see how I can get a H264 frame out of the Bot and  also to send a H264 frame to it. Loop back as you say may not be a good idea, as it cannot service the key frame, resolution or bitrate requests originating from the far end. I need to generate a video stream of my own to deal with all the dependencies you have mentioned above.
I am trying to understand right now is how to get a frame out and how to feed it a frame. I have basic question on receiving a frame which you can help me answer. The VideoMediaReceivedEventArgs object that I receive in the VideoMediaReceived callback has  Buffer member which internally has a Data (IntPtr). Is that an unencrypted and depacketized H264 Frame? Or is it a reference to an instance of some other class like H264Frame?

@vganapathy-lifesize
The frame data is an unencrypted, depacketized H.264 elementary stream. VideoMediaBuffer.Data points to the start code of the first NALU of the frame and the whole buffer contains all the NALUs of the frame in sequence separated by their start codes.

@ssulzer is there a github repo I can transfer this issue to?

@ksikorsk
No. This issue can be closed: we advise not to do video loop back using the encoded H.264 video format. Loop back will work reliably only when using the "raw" NV12 video format. Thanks.

I am trying to get one .mp4 file from byte array  with .h264 color format but I am not able to do that. i am using this code to write the video file:  Stream t = new FileStream("C:\ganesha.MP4", FileMode.Create);
BinaryWriter b = new BinaryWriter(t);
b.Write(arrbyte);
t.Close();
        After i see your post where it is mentioned that we should use Nv12 in place of h264 so i used that but i am getting error:    (Array dimensions exceeded supported range). How we can fix this issue. We are stuck here.


New lint: avoid AsyncValue.when/co inside providers

Don't:
@riverpod
Future<int> dep(DepRef ref) async {...}

@riverpod
int? fn(FnRef ref) {
  return ref.watch(depProvider).when(
    data: (value) => value * 2,
    loading: () => null,
    error: (err, stack) => null,
  );
}

Do:
@riverpod
Future<int> dep(DepRef ref) async {...}

@riverpod
Future<int> fn(FnRef ref) async {
  final value = await ref.watch(depProvider.future);
  return value * 2;
}

This may have some false positives. But users could disable the lint if they wish to.

when/map are straight up discouraged now that we have pattern-matching. So no need for this


ExtractAPMEntityDashboardScreenshots failed with System.ComponentModel.Win32Exception

Downloaded <IP_ADDRESS> for osx-core to run on macOS 10.15.1 (Catalina)
Ran xattr -d com.apple.quarantine AppDynamics.DEXTER.<IP_ADDRESS>-osx-core.zip
Copied DefaultJob.json and used all of these values for the job for one (large) app
Invoked ./AppDynamics.Dexter -j ~/Google\ Drive/Tools/DEXTER/prod-1205-0700.json  -o ~/Documents/Tools/DEXTEROut/
The job ended with this output. I'll drop the dashboard part of the report for now, not sure what perms needed to invoke the chromedriver; if I chmod it 755 would that suffice?
logs.zip
ExtractAPMEntityDashboardScreenshots(32): Starting
Input.EntityDashboards=True
ExtractAPMEntityDashboardScreenshots(32): [1/1], https://siteone.saas.appdynamics.com Unity Enterprise(3105) [APM]
Trying to open Chrome Web Driver from /Users/dskeen/Google Drive/Tools/DEXTER/AppDynamics.DEXTER.<IP_ADDRESS>-osx-core/core-osx/ChromeDriver/78
System.ComponentModel.Win32Exception (13): Permission denied
at System.Diagnostics.Process.ForkAndExecProcess(String filename, String[] argv, String[] envp, String cwd, Boolean redirectStdin, Boolean redirectStdout, Boolean redirectStderr, Boolean setCredentials, UInt32 userId, UInt32 groupId, UInt32[] groups, Int32& stdinFd, Int32& stdoutFd, Int32& stderrFd, Boolean usesTerminal, Boolean throwOnNoExec)
at System.Diagnostics.Process.StartCore(ProcessStartInfo startInfo)
at System.Diagnostics.Process.Start()
at OpenQA.Selenium.DriverService.Start()
at OpenQA.Selenium.Remote.DriverServiceCommandExecutor.Execute(Command commandToExecute)
at OpenQA.Selenium.Remote.RemoteWebDriver.Execute(String driverCommandToExecute, Dictionary`2 parameters)
at OpenQA.Selenium.Remote.RemoteWebDriver.StartSession(ICapabilities desiredCapabilities)
at OpenQA.Selenium.Remote.RemoteWebDriver..ctor(ICommandExecutor commandExecutor, ICapabilities desiredCapabilities)
at OpenQA.Selenium.Chrome.ChromeDriver..ctor(ChromeDriverService service, ChromeOptions options, TimeSpan commandTimeout)
at OpenQA.Selenium.Chrome.ChromeDriver..ctor(String chromeDriverDirectory, ChromeOptions options, TimeSpan commandTimeout)
at OpenQA.Selenium.Chrome.ChromeDriver..ctor(String chromeDriverDirectory, ChromeOptions options)
at AppDynamics.Dexter.ProcessingSteps.ExtractAPMEntityDashboardScreenshots.Execute(ProgramOptions programOptions, JobConfiguration jobConfiguration) in C:\appdynamics\AppDynamics.DEXTER\src\ProcessingSteps\Extract\ExtractAPMEntityDashboardScreenshots.cs:line 179
ExtractAPMEntityDashboardScreenshots(32): [1/1], https://siteone.saas.appdynamics.com Unity Enterprise duration 00:00:00.5873023 (587 ms)
ExtractAPMEntityDashboardScreenshots(32): total duration 00:00:00.5948291 (594 ms)
If you need support, please review https://github.com/Appdynamics/AppDynamics.DEXTER/wiki#getting-support and send the logs
Application execution took 00:48:47.5013999 (2927501 ms)
DSKEEN-M-841M:Google Drive/Tools/DEXTER/AppDynamics.DEXTER.<IP_ADDRESS>-osx-core/core-osx>

logs.zip

You already ran the xattr command, that should have made the thing unstarted.
Can you run the executable from /Users/dskeen/Google Drive/Tools/DEXTER/AppDynamics.DEXTER.<IP_ADDRESS>-osx-core/core-osx/ChromeDriver/78 yourself and see what happens?

To run it from the command line I have to chmod the file. Should re-run the dashboard report with these file perms?
DSKEEN-M-841M:Google Drive/Tools/DEXTER/AppDynamics.DEXTER.<IP_ADDRESS>-osx-core/core-osx/ChromeDriver/78/mac64> ./chromedriver
-bash: ./chromedriver: Permission denied
DSKEEN-M-841M:Google Drive/Tools/DEXTER/AppDynamics.DEXTER.<IP_ADDRESS>-osx-core/core-osx/ChromeDriver/78/mac64> chmod +x chromedriver
DSKEEN-M-841M:Google Drive/Tools/DEXTER/AppDynamics.DEXTER.<IP_ADDRESS>-osx-core/core-osx/ChromeDriver/78/mac64> ./chromedriver
Starting ChromeDriver 78.0.3904.11 (eaaae9de6b8999773fa33f92ce1e1bbe294437cf-refs/branch-heads/3904@{#86}) on port 9515
Only local connections are allowed.
Please protect ports used by ChromeDriver and related test frameworks to prevent access by malicious code.
[1575571661.641][WARNING]: FromSockAddr failed on netmask
^C
DSKEEN-M-841M:Google Drive/Tools/DEXTER/AppDynamics.DEXTER.<IP_ADDRESS>-osx-core/core-osx/ChromeDriver/78/mac64>

Yes, please, adjust the Status parameter https://github.com/Appdynamics/AppDynamics.DEXTER/wiki/Restart-Job-From-Specific-Step with https://github.com/Appdynamics/AppDynamics.DEXTER/wiki/Phases-and-Job-Steps#extractapmentitydashboardscreenshots-32 and restart

I hit the same problem with appdynamics.dexter.<IP_ADDRESS>-osx-core/core-osx/ChromeDriver/80/mac64/chromedriver. I set perms to 755 and restarted the same job. This time I hit a new problem with chromedriver:
ExtractAPMEntityDashboardScreenshots(32): Starting
Input.EntityDashboards=True
ExtractAPMEntityDashboardScreenshots(32): [1/1], https://siteone.saas.appdynamics.com Unity Enterprise(3105) [APM]
Trying to open Chrome Web Driver from /Users/dskeen/Documents/Tools/DEXTER/appdynamics.dexter.<IP_ADDRESS>-osx-core/core-osx/ChromeDriver/80
Starting ChromeDriver 80.0.3987.16 (320f6526c1632ad4f205ebce69b99a062ed78647-refs/branch-heads/3987@{#185}) on port 60587
Only local connections are allowed.
Please protect ports used by ChromeDriver and related test frameworks to prevent access by malicious code.
[1578081284.960][WARNING]: FromSockAddr failed on netmask
Unable to create Web Driver from /Users/dskeen/Documents/Tools/DEXTER/appdynamics.dexter.<IP_ADDRESS>-osx-core/core-osx/ChromeDriver/80/mac64
Trying to open Chrome Web Driver from /Users/dskeen/Documents/Tools/DEXTER/appdynamics.dexter.<IP_ADDRESS>-osx-core/core-osx/ChromeDriver/79
System.ComponentModel.Win32Exception (13): Permission denied
at System.Diagnostics.Process.ForkAndExecProcess(String filename, String[] argv, String[] envp, String cwd, Boolean redirectStdin, Boolean redirectStdout, Boolean redirectStderr, Boolean setCredentials, UInt32 userId, UInt32 groupId, UInt32[] groups, Int32& stdinFd, Int32& stdoutFd, Int32& stderrFd, Boolean usesTerminal, Boolean throwOnNoExec)
at System.Diagnostics.Process.StartCore(ProcessStartInfo startInfo)
at System.Diagnostics.Process.Start()
at OpenQA.Selenium.DriverService.Start()
at OpenQA.Selenium.Remote.DriverServiceCommandExecutor.Execute(Command commandToExecute)
at OpenQA.Selenium.Remote.RemoteWebDriver.Execute(String driverCommandToExecute, Dictionary`2 parameters)
at OpenQA.Selenium.Remote.RemoteWebDriver.StartSession(ICapabilities desiredCapabilities)
at OpenQA.Selenium.Remote.RemoteWebDriver..ctor(ICommandExecutor commandExecutor, ICapabilities desiredCapabilities)
at OpenQA.Selenium.Chrome.ChromeDriver..ctor(ChromeDriverService service, ChromeOptions options, TimeSpan commandTimeout)
at OpenQA.Selenium.Chrome.ChromeDriver..ctor(String chromeDriverDirectory, ChromeOptions options, TimeSpan commandTimeout)
at OpenQA.Selenium.Chrome.ChromeDriver..ctor(String chromeDriverDirectory, ChromeOptions options)
at AppDynamics.Dexter.ProcessingSteps.ExtractAPMEntityDashboardScreenshots.Execute(ProgramOptions programOptions, JobConfiguration jobConfiguration) in C:\appdynamics\AppDynamics.DEXTER\src\ProcessingSteps\Extract\ExtractAPMEntityDashboardScreenshots.cs:line 179
ExtractAPMEntityDashboardScreenshots(32): [1/1], https://siteone.saas.appdynamics.com Unity Enterprise duration 00:00:00.8221364 (822 ms)
ExtractAPMEntityDashboardScreenshots(32): total duration 00:00:00.8297320 (829 ms)
If you need support, please review https://github.com/Appdynamics/AppDynamics.DEXTER/wiki#getting-support and send the logs
Application execution took 00:03:53.4257994 (233425 ms)
When I invoke chromedriver by itself:
DSKEEN-M-841M:Documents/Tools/DEXTER/appdynamics.dexter.<IP_ADDRESS>-osx-core/core-osx/ChromeDriver/80/mac64> ./chromedriver
Starting ChromeDriver 80.0.3987.16 (320f6526c1632ad4f205ebce69b99a062ed78647-refs/branch-heads/3987@{#185}) on port 9515
Only local connections are allowed.
Please protect ports used by ChromeDriver and related test frameworks to prevent access by malicious code.
[1578081394.327][WARNING]: FromSockAddr failed on netmask
^C
I will leave this job where it is if you can figure out a workaround.

Fixed in https://github.com/Appdynamics/AppDynamics.DEXTER/releases/tag/<IP_ADDRESS>


Included TimeZone in AnniversaryDate Instructions

Based on the test suite, the AnniversaryDate function needs to return the current year's anniversary date in UTC timezone. However, the current instructions have no mention of any such requirement. Updating the instruction to fix this.

Thanks for the fix!


Sync master and develop after 2.0 release

Hi @s-martin
what would be the best way to go about the syncing of master and develop. Are they synced now? My feeling is that the best way would be ... extreme ...?

delete master
rename develop into master
create new develop from master

But I am not sure if that is necessary at the moment. With this merge you synced master issues back into develop?

master is in sync with develop, i.e. develop contains all changes from develop and master.
We can continue working on develop and when the next release is ready we can merge again develop to master. As long as we don't apply any hotfixes to master we don't need to merge master to develop.
I don't think we need to delete and rename the branches.


old video linked in the getting started game

https://workshops.hackclub.com/sprig_first_game/ links the old video which shows the old editor
this is the correct video : https://www.youtube.com/watch?v=ZOPYB6dw4Os

hmmm imo the correct video doesn't cover everything the old video covers

Yeah there needs to be an updated video for this if we are going to change the link.
https://www.youtube.com/watch?v=1UTLS4aO9bQ

@LucasHT22 can you please look at this and close this if it has already been fixed?

@grymmy We still don't have a video that covers what the old one covers, so it's not finished

@karamassie do we expect this video to be created?

@LucasHT22  what content is missing in the new video, pls?

this is actually fixed!


Question on opa test options

Is it possible to pass options to opa test?
https://www.openpolicyagent.org/docs/latest/cli/#options-12
Specifically we're trying to get additional info from test failures, so maybe --verbose and --explain would be useful.

Is how you've defined --strict as an arg in opa_check be the pattern to follow for adding --verbose and --explain to opa_test

Hello. Good question. I see 3 1/2 possibilities

We can add dedicated attributes just as you suggested, similar to the --strict flag.
We can add a more general opts attribute for arbitrary options.
We can add flags for the command line, something like --@rules_opa//:test_verbose and --@rules_opa//:test_explain so that I can be set dynamically
3.5. Reuse some exiting flag provided by bazel

What do you think would work best for you?

I think I'd be familiar with how to apply 1.
opa_test(
    name = "opa_lib_test",
    size = "small",
    srcs = glob(["opa/**/*_test.rego"]),
    deps = [":opa_lib"],
    strict = True,
)

I'm not familiar with how the others would be accomplished, but otherwise don't have a strong preference other than a desire to get increased visibility of failing tests.  As it currently is we don't get any details beyond that the test FAILED.  And are looking for typical information such as the values that were compared being output.


Introducing script to generate bug stat visualizations

Uses github API to query bug information
For now it Popen's curl, would be better
to use the requests module.
Even when authentication seemed to be happening
with requests, I was still being severly rate
limited.
The command line curl binary does not have this problem.
It may have to do with proxy support, not sure.

plots look great!
Might help add some artificial stalls to limit the query rate on the github API - should help limit the number of empty strings returned by the github servers.

Cleaned up the colors and added a 100 ms stall between queries.


[FLINK-19801] Adding virtual channels for rescaling unaligned checkpoints.

What is the purpose of the change
This PR concludes rescaling for unaligned checkpoints.
Brief change log

Uses the InflightDataRescalingDescriptor introduced by FLINK-19533.
Adds VirtualChannelSelector event and adds demultiplexing on input side.
Emits VirtualChannelSelector on recovery on input and output side and replicates data according to channel mapping in rescaling descriptor.
Adds check for custom partitioner and allow UC rescaling for supported cases.

Verifying this change
Adds rescaling tests in UnalignedCheckpointITCase. Enables unaligned checkpoint in all e2e tests.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented) (separate PR)

@flinkbot run azure


fix(z-order): import cdk overlay.scss so we can update the z-order

z-order of the cdk was 1000 which was conflicting with several of our cmponents now it's 1060
fixing for angularjs for now .. will need to fix anguarl

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots:
Before (If applicable):

After:

Types of changes

[x ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation update
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

Checklist:

[x ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x ] All new and existing tests passed.

@pauljeter  .. done!


Normalize/ css reset?

On firefox I noticed spurious margins on block elements like body, headings, figure and perhaps many others. In the code I don't see any css reset or normalization. Is this by design or an omission?

I'll normalize some margins on some elements in v1.0.0 👍


How to use whatsapp quick reply button

can you please help me to use quick reply buttons for whatsapp custom connector . I can't create an interactive template like whatsapp quick reply buttons. how to send a template data action file to customer connector or What should I do for use whatsapp templates.

I'm Fixed this issue. Thank you!


Slider not movable after reaching to maximum or minimum

After the slider reaches to the minimum value or maximum value, it is not movable anymore. It just freezes to that particular point. Is there any way to solve this?

It's a zIndex problem :P, the second circle its alway on foreground.

So there is no way to solve this?


long word problem

the screen automatically extends when a long word comes to accommodate background styles, but the original text comes on next line.
i think we should put the styling also in textarea

I think it is because size of color ing block do not concide with textarea

regexr.com and regexpal.com both are using codemirror, I think we have to do same,
tried both editable div and textarea none of them working properly.

solution found from codemirror and present inside this pen
https://codepen.io/ashishpatel0720/pen/QmrzMg


feat(go-namespaces): Split generated Go files into multiple packages

This commit overhauls the WorldGenerator file structure in the tinygo bindgen. Key changes include:

Splitting generated Go files into different packages for each interface, imported functions, and exported functions.
Simplifying function and symbol names by removing namespaces.
Adding a package_name option to Go bindgen for prefixed package names. Defaults to main if not specified.

Internally, the C files are placed at a special Go package named "c_files_", and a dummy Go file is generated to expose the C APIs. Each interface generates a Go package that imports this special "c_files_" package to get the C apis. It still has to import the C header file which is also placed at "c_files_" by setting CFLAGS to locate where that is for that Go package.
This restructuring enhances usability by making symbol and function names more readable and less verbose, significantly improving the user experience.
TODO

[ ] Generate a dummy main file for each codegen test, and that main file needs to import all the generated Go packages.
[ ] Enable the rest of the runtime tests
[ ] Figure out how to import Go packages to implement type aliasing
[ ] More testing with wasi:http

This work has been paused for a few months since there is a new pure-Go implementation of the wit-bindgen-go undergoing.


Merge main to vnext

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: cchen-vertica:white_check_mark: roypaulin:x: Matt SpilchenMatt Spilchen seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

You should resolve the history. Only the last 2 commits are new.


create a contact page

pl assign me this issue

@ShravaniAK we already have a contact page .... if you can just add the navbar to it (present in the index.html) that would be a great help

hii @RIKI-05  I want to add navbar on the contact page . pls assign it to me

Hi please assign me this issue, I would like to  work on this.


Set-DbaDatabaseOwner additional checks

Set-DbaDatabaseOwner needs additional checks for Offline and ReadOnly status, currently throws error and stops cmdlet if the database is in those states. Also check for new owner already being mapped to the db, this also throws an error and stops cmdlet.

I have a couple thoughts on resolving this:

Throw a warning and continue if a mapped login is detected (like the other checks)
Add a -Force switch. If -Force is declared, it will drop the user from the database and then set the owner. Otherwise it will throw a warning.

Thoughts?

Agree with the first one.
For the second one, why drop the user and not only set him as dbowner ?
Imagine someone set 'user1' (that have datareader permissions) as dbowner
but he was wrong and have to set 'user2' as dbowner, now, the user 'user1'
do not have the permissions (has lost datareader) he had before.
This makes sense to you?
No dia domingo, 21 de agosto de 2016, MikeFal<EMAIL_ADDRESS>escreveu:

I have a couple thoughts on resolving this:

Throw a warning and continue if a mapped login is detected (like the
other checks)
Add a -Force switch. If -Force is declared, it will drop the user
from the database and then set the owner. Otherwise it will throw a warning.

Thoughts?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ctrlbold/dbatools/issues/156#issuecomment-241266283,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASnfI98O4NMTLmddc-9uYZClY5uuac8Tks5qiHjHgaJpZM4JnazA
.

--
Cumprimentos,
Cláudio Silva

e-mail<EMAIL_ADDRESS>linkedin: https://pt.linkedin.com/in/claudioessilva
twitter: http://www.twitter.com/claudioessilva

You can't set a login as database owner if it exists as a user in the database. Adding it to the db_owner role is not the same as setting the database owner to a login. That's why we'd have to drop the user. I agree with your concerns that if the user gets dropped, you lose any permissions the user had. This is why they would need to use -Force to perform the user drop.
Powershell is a sharp knife. Be careful you don't cut yourself with it. ;)
Of course, I'm open to not doing this and simply throwing a warning if the user is in the database.

That's true @MikeFal! I had forgotten this one!
That said I agree with the second one too :smile:

Additionally, it's not possible for AD groups to be database owners is it? I think we came across this in the DbaJobOwner cmdlets as well but it was fixed there, might need a check for that too.

@MikeFal would you be happy for me to attempt to fix the issue I had with AD groups as database owners in Set-DbaDatabaseOwner? I think it was fixed in the "JobOwner" cmdlets you wrote, but not here, so I can probably just fix it with code copied or inspired from what you've written elsewhere.


Build error with CMake and MSYS2 mingw-w64

If attempt to use cmake to build CppUTest on msys2 I get following compiler errors:
../src/Platforms/Gcc/UtestPlatform.cpp: In function 'int IsNanImplementation(double)':
../src/Platforms/Gcc/UtestPlatform.cpp:237:12: error: conversion to 'float' from 'int' may alter its value [-Werror=conversion]
     return isnan((float)d);
            ^
../src/Platforms/Gcc/UtestPlatform.cpp:237:12: error: conversion to 'float' from 'int' may alter its value [-Werror=conversion]
../src/Platforms/Gcc/UtestPlatform.cpp:237:12: error: conversion to 'float' from 'int' may alter its value [-Werror=conversion]
../src/Platforms/Gcc/UtestPlatform.cpp:237:12: error: conversion to 'int' from 'float' may alter its value [-Werror=float-conversion]

Note that while cmake fails the autotools build works fine on msys2.
Steps to reproduce:

Get MSYS2 and follow their install intructions
Install mingw-w64, cmake and ninja with pacman -S mingw-w64-x86_64-gcc mingw-w64-x86_64-cmake mingw-w64-x86_64-ninja
Open MinGW-w64 Win64 Shell and execute cmake -G Ninja . && ninja in the CppUTest folder

For a conversion to / from float to int to occur at all, you must be attracting a very weird implementation of isnan(). Personally, I doubt that is anything to do with CppUTest.
For example, the "normal" declaration would be something like:
#define isnan(x) (sizeof (x) == sizeof (float) ? __isnanf (x)	\
		  : sizeof (x) == sizeof (double) ? __isnan (x)	\
		  : __isnanl (x))

according to this page  (so, strictly speaking, the cast to double is unnecessary), which resolves to something like:
__CRT_INLINE int __cdecl __isnan (double _x)

which is a lean match to the call within CppUTest. Could you please dig up your math.h and check the definition of isnan()?

P.S. Unfortunately, I can't personally take the time to set up an entire build environment to reproduce your problem.

The definition for isnan() for mingw-w64:
#define isnan(x) \
__mingw_choose_expr (                                         \
  __mingw_types_compatible_p (__typeof__ (x), double),            \
    __isnan(x),                                                 \
    __mingw_choose_expr (                                     \
      __mingw_types_compatible_p (__typeof__ (x), float),         \
        __isnanf(x),                                            \
    __mingw_choose_expr (                                     \
      __mingw_types_compatible_p (__typeof__ (x), long double),   \
        __isnanl(x),                                            \
    __dfp_expansion(__isnan,(__builtin_trap(),x),x))))

The according function is resovled with C++ templates defined in line 75.
Like you said it should resolve to extern int __cdecl __isnanf (float) because the parameter for isnan() is casted to float in UtestPlatform.cpp like return isnan((float)d);
I don’t really know why the compiler throws those conversion errors when calling isnan() like that. A simliar error occurs if I change the line to return isnan(d);:
../src/Platforms/Gcc/UtestPlatform.cpp: In function 'int IsNanImplementation(double)':
../src/Platforms/Gcc/UtestPlatform.cpp:237:12: error: conversion to 'float' from 'double' may alter its value [-Werror=float-conversion]
     return isnan(d);
            ^
../src/Platforms/Gcc/UtestPlatform.cpp:237:12: error: conversion to 'int' from 'double' may alter its value [-Werror=float-conversion]

I wonder why the autotools build works flawlessly. It should be using the exact same header file.

Very strange indeed. The first error in particular, since the macro would imply that it chooses the correct version if its three internal isnan definitions. As for the 'double' to 'int' conversion, I have no idea. It is almost as if it it is digging up some strange definition elsewhere. But where, and why?

It might be interesting to compile with -E instead of -c to see what is the actual preprocessor output....

What happens if you `#include  and call std::isnan() ?

diff --git a/src/Platforms/Gcc/UtestPlatform.cpp b/src/Platforms/Gcc/UtestPlatform.cpp
index 447b801..809fdad 100644
--- a/src/Platforms/Gcc/UtestPlatform.cpp
+++ b/src/Platforms/Gcc/UtestPlatform.cpp
@@ -38,7 +38,7 @@
 #include <stdarg.h>
 #include <setjmp.h>
 #include <string.h>
-#include <math.h>
+#include <cmath>
 #include <ctype.h>
 #include <unistd.h>
 #include <signal.h>
@@ -232,13 +232,8 @@ void (*PlatformSpecificFree)(void* memory) = free;
 void* (*PlatformSpecificMemCpy)(void*, const void*, size_t) = memcpy;
 void* (*PlatformSpecificMemset)(void*, int, size_t) = memset;

-static int IsNanImplementation(double d)
-{
-    return isnan((float)d);
-}
-
 double (*PlatformSpecificFabs)(double) = fabs;
-int (*PlatformSpecificIsNan)(double) = IsNanImplementation;
+int (*PlatformSpecificIsNan)(double) = std::isnan;
 int (*PlatformSpecificAtExit)(void(*func)(void)) = atexit;  /// this was undefined before

 static PlatformSpecificMutex PThreadMutexCreate(void)

This change fixes the cmake build and at the same time breaks the autotools based build:
../src/Platforms/Gcc/UtestPlatform.cpp:41:17: fatal error: cmath: No such file or directory
 #include <cmath>
                 ^

Is UtestPlatform.cpp compiled as C code here?

You'd have to keep the IsNanImplementation(double d) wrapper, because using the function pointer breaks template-based Clang (sigh).
Nice that it fixes cmake :-)
Now that's one weird error again. I don't know what to say... It is compiled as C++.

I get the same error (cmath not found) with autotools and clang(++).

Looks like -nostdinc++ compiler flag in CPPUTEST_CXXFLAGS causes this behavoir. When I remove it everything builds fine.

This one happens when the -nostdinc++ option is present on the compiler command line. This is strange, since it is not the default. Running ../configure without arguments should remove it

Too bad though, because CppUTest needs to be able to build with the  -nostdinc++ flag also -- so we can't use this solution, unfortunately.

The only fix I could come up with would be to use ugly #ifdef code, which I don’t think is the right way.
As this is clearly a mingw-w64 bug I’ll close this issue and file a bug report at their issue tracker.
Thank you very much for your help! :-)

Hello,
I have the same problem.
@somemetricprefix, did you opened a bug on the MinGW side? I'm using 4.9.2-posix-sjlj.
Can you give more explanation about your "ugly #ifdef code" solution? thanks!

@arcadien
The MinGW bug is here.
Unfurtunalety don’t have the code for a workaround anymore. All I remember is that I got it compiling for MinGW but breaking other platforms at the same time.


Region options on the main page doesn't filter properly

STEPS TO REPRODUCE:

head to https://detroit-public-dev.netlify.app/
on that page there are the region options, if you click on one you should navigate to a filter version of the listing grid. Filtered by the region selected
this filtering doesn't occur

EXPECTED RESULTS:
Clicking on those region options should take you to a correctly filtered set
QA NOTES
Testing new coming soon filter (https://detroit-public-dev.netlify.app/listings)
Head to public site, go to view all listings and opt to add a filter
there should be a new coming soon under availability that is functional
Fix for region options (https://detroit-public-dev.netlify.app/)
Head to the root page for the public site
notice at the bottom there are the regions laid out in a carousel
Click on one of the region options
the filter should function properly and should play nicely with the new filter form

LGTM


FAILURE: macos/grpc_basictests_c_cpp alts_concurrent_connectivity_test

prod:grpc/core/master/macos/grpc_basictests_c_cpp has been failing since Nov 12, 2020 because of alts_concurrent_connectivity_test consistently. log
Since this test had been disabled between Oct 26 and Nov 12, this could be caused by in-between changes. Or this could be caused by new mojave mac machine with xcode 11.

 appears to show that two first test runs were done successfully.

This test passed locally after increasing the limit of open files. To be sure, following is the result of ulimit on both high-sierra and mojave mac kokoro machines. Both have the same limits.
macos_external-prod-dynamic (mojave)
++ launchctl limit maxfiles
	maxfiles    500000         500000
++ ulimit -a
core file size          (blocks, -c) 0
data seg size           (kbytes, -d) unlimited
file size               (blocks, -f) unlimited
max locked memory       (kbytes, -l) unlimited
max memory size         (kbytes, -m) unlimited
open files                      (-n) 500000
pipe size            (512 bytes, -p) 1
stack size              (kbytes, -s) 8192
cpu time               (seconds, -t) unlimited
max user processes              (-u) 2500
virtual memory          (kbytes, -v) unlimited

macos_external-prod-high-sierra
++ launchctl limit maxfiles
	maxfiles    500000         500000
++ ulimit -a
core file size          (blocks, -c) 0
data seg size           (kbytes, -d) unlimited
file size               (blocks, -f) unlimited
max locked memory       (kbytes, -l) unlimited
max memory size         (kbytes, -m) unlimited
open files                      (-n) 500000
pipe size            (512 bytes, -p) 1
stack size              (kbytes, -s) 8192
cpu time               (seconds, -t) unlimited
max user processes              (-u) 2500
virtual memory          (kbytes, -v) unlimited

Test result isn't clear on why this test failed. It doesn't have any obvious failing message and the final test end code was called.
I1113 11:05:36.035056000<PHONE_NUMBER>50560 subchannel.cc:1040]           Failed to connect to channel, retrying
E1113 11:05:36.041442000<PHONE_NUMBER> test_config.cc:194]                Timeout in waiting for gRPC shutdown
Stack trace:
    @        0x10d53e105  (unknown)  grpc_core::GetCurrentStackTrace()
    @        0x10d539526  (unknown)  gpr_default_log()
    @        0x10d538d79  (unknown)  gpr_log_message
    @        0x10d539366  (unknown)  gpr_log
    @        0x10d0c731a  (unknown)  grpc::testing::TestEnvironment::~TestEnvironment()
    @        0x10d0c7435  (unknown)  grpc::testing::TestEnvironment::~TestEnvironment()
    @        0x10ce332a9  (unknown)  main
    @     0x7fff6374b3d5  (unknown)  start

I1113 11:05:36.051401000<PHONE_NUMBER> test_config.cc:205]                TestEnvironment ends
real        21.39
user        11.13
sys          1.11

Closing (obsolete)


Partially valid JSON issue

In our client/server WS communication we use partially valid JSON (e.g. 42["action",["event1","event2"]]) I'd like to have ability to save such messages and preview pop-up with highlighted JSON.
I disabled options that validate JSON in messages (send/save)

but I'm not able to save it because it fails when I press "Save" button.

Also, I'd like to have "preview" pop-up with JSON highlighted

If I push a new branch that addresses this issue, could you clone, build, and try it out in Chrome? The instructions to do that are in the README. Since this isn’t a feature I’d use, I’d like to get your feedback before I publish it.

Sure, thank you for the awesome plugin :-)
By the way, about PING/PONG - we could add section under options/preferences saying "enable PING/PONG messages", then in options/messages (or new dynamic section options/heartbeat) have text fields where PING/PONG message could be specified, as well as pingInterval etc.

Please clone branch 1.3.0, build it using npm run build:chrome, install it locally, and let me know if it meets your needs. I still have to fix failing tests and add new ones, so don't worry about those yet. I just want to make sure this is what you're after before going any further. It looks pretty simple, but I'm not sure how to improve it.
I'm still thinking about the ping/pong issue, but I think your idea above is the only way to do it.
Thanks for your help!

Thanks for the update. Save messages works fine now. :+1:
Pop-up works differently than I expected. Might be misunderstanding. This is how it works now

The behavior I'd expect

In other words, I'd like to "try" the most outer valid JSON in response. 42["valid","json"]43 should be displayed and highlighted in pop-up as ["valid","json"]. Every JSON starts with [ or { and finishes with ] or } so maybe recursively we try to find valid JSON starting from the most outer possible combination. Sound like a crazy requirement but I'd be happy if we just could skip some invalid part in the beginning. Hope examples help you to better understand my expectation.
Thank you in advance :-)

Thanks for the feedback, especially the images. I understand now what you’re looking for, but I want to spend a day or two thinking about it. Maybe there are other use cases I haven’t considered. I’ll probably ask you to take another look before I publish a new version, so stay tuned.

Please retry the latest 1.3.0 commit and let me know what you think. Most of the changes are here: https://github.com/kensiprell/browser-websocket-client/blob/9638f0b1f7988d864be1a7e68604f68bb3ad4ffa/src/main/index.js#L1044-L1080.
Here are some preview images:

I just pushed a new commit for 1.3.0. It now looks like: https://github.com/kensiprell/browser-websocket-client/blob/09d4135d03569bc7c94c48d24f98723030cd1c0c/src/main/index.js#L1043-L1075

I've just checked. It works great I love it, thank you very much :+1:

Fixed in version 1.3.0.


[Proposal] Allow adapter specific data to be stored inside Tesla.Env

Example:
#little change to adapter
defmodule Tesla.Adapter.Hackney do
    def call(env, opts) do
      with  {:ok, status, headers, body, location} <- request(env, opts || []) do
        %{env | status:   status,
                headers:  headers,
                body:     body,
                private: %{final_location: location}}
      end
    end

    defp handle({:ok, status, headers, ref}) do
      with  location <> "" <- :hackney.location(ref),
               {:ok, body} <- :hackney.body(ref) do
        {:ok, status, headers, body, location}
      end
    end
  end

#usage
iex(1)> Tesla.get("https://github.com/teamon/tesla", opts: [follow_redirect: true])
%Tesla.Env{
  url: "https://github.com/teamon/tesla",
  private: %{final_location:  "https://github.com/teamon/tesla"}
}

iex(2)> Tesla.get("https://goo.gl/8hfJ7F", opts: [follow_redirect: true])
%Tesla.Env{
  url: "https://goo.gl/8hfJ7F",
  private: %{final_location:  "https://github.com/teamon/tesla"}
}

While I do get the idea you could use Tesla.put_opt(:location, ...) and
then access the value with env.opts.location.
Is there a reason why opts might not be a good solution?
On Wed, 25 Jan 2017 at 01:38, Tobiasz Małecki<EMAIL_ADDRESS>wrote:

Example:
#little change to adapter
defmodule Tesla.Adapter.Hackney do
def call(env, opts) do
with  {:ok, status, headers, body, location} <- request(env, opts || []) do
%{env | status:   status,
headers:  headers,
body:     body,
private: %{final_location: location}}
end
end
defp handle({:ok, status, headers, ref}) do
  with  location <> "" <- :hackney.location(ref),
           {:ok, body} <- :hackney.body(ref) do
    {:ok, status, headers, body, location}
  end
end

end
#usage
iex(1)> Tesla.get("https://github.com/teamon/tesla", opts: [follow_redirect: true])
%Tesla.Env{
url: "https://github.com/teamon/tesla",
private: %{final_location:  "https://github.com/teamon/tesla"}
}
iex(2)> Tesla.get("https://goo.gl/8hfJ7F", opts: [follow_redirect: true])
%Tesla.Env{
url: "https://goo.gl/8hfJ7F",
private: %{final_location:  "https://github.com/teamon/tesla"}
}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/teamon/tesla/issues/48, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAfk8tV-nq-xqoqvo6hb-R7Vmlb66e3ks5rVplqgaJpZM4Ls93b
.

Sure, opts could be used, but I think it's would be cleaner to not mix different entities under same key.
For me :opts is write-only for user and read-only for Tesla when :private will be opposite.
What do I mean by this?
When you do Tesla.get(url, opts: [something: true]) you probably don't care about what's saved under :opts key in Tesla.Env response, because you know what you have passed in function call. On the other hand Tesla shouldn't use data from :private when making request, because this field would allow to store inconsistent data(specific to one adapter/middleware), which from end user perspective may be useful.
#some pseudocode to make this 100% clear

important_in_request = [follow_redirect: true]
ignored_by_tesla = [random_key: :value]

%Tesla.Env{
   opts: _user_probably_dont_care,
   private: additional_data_that_user_may_be_interested_in
} = Tesla.get(url, opts: important_in_request, private: ignored_by_tesla)

I agree mixing opts with private might not be a good idea.
One question remains - what should be the desired adapter's behaviour and rules for using private properties? (Assuming client code does not modify adapters)
In case of "follow redirects" I think this should be a tesla middleware like this

My first idea for private properties can be described as "allow to store anything useful, don't merge to master anything that tries to read from it". I'll think about it more later, but unfortunately I'm little busy at the moment.
about FollowRedirects middleware
I like this, it will allow me to remove custom implementation from my library
one note: httpc default for autoredirect is true, you should probably disable it inside this middleware
btw I made middleware for data compression
https://github.com/amatalai/link_preview/blob/1.0.0/lib/compression_middleware.ex
It's work in progress now, but I think that I'll have time to finish it in next few days (I'll make separate issue tomorrow to discuss details)

@amatalai sooooo, what are we doing with this? ;)

I wish I could create PR with some proposal to push work on this forward, but lately I don't have time to do that.

I need to revisit this idea. With ability to add middleware before adapter it should be easy to save raw headers etc with simple middleware and without any breaking changes.

As mentioned in #104 there was always a possibility to add something right around adapter :)

Please see possible solution in #37

Yeah, this wouldn't be useful at the moment


userguide: PV docs clarify Azure storage account restriction

mirrors: https://github.com/kubernetes/kubernetes/pull/40945
Just clarifies a usage constraint regarding AzureDisk and dynamic provisioning.
cc: @rootfs

This change is

/lgtm


Phrase suggestion not working on DART elastic_client ?

Hi,
I am trying to get phrases suggestions on my Flutter app using client.search() from https://pub.dev/packages/elastic_client
Works fine on REST API :
`{
"suggest":{
"text": "Britih",
"content": {
"phrase": {
"field": "content",
"highlight": {
"pre_tag": "",
"post_tag": ""
}
}
}
}
}`
However, when I replicate this request on dart/Flutter, I get the following error:
[ERROR:flutter/lib/ui/ui_dart_state.cc(209)] Unhandled Exception: type 'Null' is not a subtype of type 'int' in type cast #0      Client.search.<anonymous closure>.<anonymous closure>.<anonymous closure> (package:elastic_client/src/_client.dart:331:29) #1      MappedListIterable.elementAt (dart:_internal/iterable.dart:413:31) #2      ListIterator.moveNext (dart:_internal/iterable.dart:342:26)
It seems to me that in _client.dart, frequency is expected but elastic return none.
return SuggestHitOption(
            m['text'] as String,
            m['score'] as double,
            freq: m['freq'] as int, <== ligne 331
            highlighted: m['highlighted'] as String,
          );

Thanks for reporting this. It looks like an easy fix to make it optional... would you be interested in a PR?

(Or rather always-returning 0 or -1 as a frequency when it is not present)

Yes :) thanks
Le lun. 18 avr. 2022, 14:46, István Soós @.***> a
écrit :

Thanks for reporting this. It looks like an easy fix to make it
optional... would you be interested in a PR?
—
Reply to this email directly, view it on GitHub
https://github.com/isoos/elastic_client/issues/60#issuecomment-1101381048,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOCBXORX2ELIEMHMDKRB3TDVFVKQZANCNFSM5TVXKQ2Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks Istvan


ui: fix prime ad storage duration

Fix the comma prime signup ad in the UI: the prime cloud storage duration was bumped to a year quite a long time ago.

Nice catch, though rlogs/full res stuff is only kept for two weeks still. I'll merge this to match the website, and we can figure out how to better communicate the different policies for rlogs and qlogs.


Change application fn mode()

Change application mode() to mode(&mut self) -> Option for window mode controll more easily

Closing as we discussed in a Zulip thread.


DockerComposeContainer fails due to updated docker-compose naming scheme

As of the release of docker-compose 1.23.0 the naming scheme of container changed. See this issue for more information: https://github.com/docker/compose/issues/6316
This seems to brake stuff in testcontainers DockerComposeContainer/SocatContainer implementation.
When trying to launch a docker-compose script we end up getting an exception like this: 'org.testcontainers.containers.ContainerLaunchException: Aborting attempt to link to container se2uw5jwht66_smtp-server_1 as it is not runnining'
This is due to the fact that the container name isn't really 'se2uw5jwht66_smtp-server_1' but rather 'se2uw5jwht66_smtp-server_1_e71d646caccf' following the new naming scheme in docker-compose.

Thanks @m-thread, yes, we've become aware of this unfortunate problem 😭. For now we'd recommend people hold back on updating local installations of docker-compose, and stay with 1.22.x.
We'll need to implement a solution to this, as it looks like the new behaviour is here to stay.

New release of docker-compose today:
https://github.com/docker/compose/releases/tag/1.23.2
Seems like they backed off from the breaking change for now. At least partially.

Yes you are right, I will close this for now in the light of docker-compose 1.23.2 release.
All Testcontainers user are advised to update their docker-compose version accordingly.
Thanks to everyone contributing in this issue by providing details.


refactor(banner): sync storybook and Volto component

@eea-jenkins build storybook

@eea-jenkins build storybook

@eea-jenkins build storybook

@eea-jenkins build docusaurus

@eea-jenkins build storybook

@eea-jenkins build storybook

@eea-jenkins build all

@eea-jenkins build storybook


Controller for jumpTo page

Hello, thanks for this great plugin,
However, I was wondering why there is no controller for jump to the desired page.
I go through the code of the plugin, but I don't see any controller. As a reader, I thought, it should have a controller to track the read page.

Similarly, I would love to be able to set the page. I was wondering if we have access to the scrollview height, or the ability to control the scrollview?
My pages are uniform so if I knew that I had 10 pages (which I would know), and I knew how tall the scrollview is... I could approximate what page the user is on, and then I could jump to the page I want them to be on, if I could set or move the scroll view.
Is this possible?
Thanks
jjb


Officially support minio as a provider

In this PR:

Support MINIO as storage provider. Can just use s3 and s3ForcePathStyle: true and it'll work

fixes #149 in official manner (solutions already provided in comments on that issue)

Issues reference:

https://github.com/ducktors/turborepo-remote-cache/issues/149

Checklist:

x ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

[x] Have you lint your code with pnpm lint locally prior to submission?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your core changes, as applicable?

Any ideas on writing a good test? The s3 tests do s3ForcePathStyle for the s3 tests, and that's all Minio support requires in addition to s3. I could just copy the s3 tests and change the provider but it's gonna have the same output for that

[x] Have you successfully ran build with pnpm build of your changes locally?
[x] Have you successfully ran tests with pnpm test of your changes locally?
[x] Have you commit using Conventional Commits?

@all-contributors add @EWhite613 on code

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[coq] simple-io: 1.0.0 -> 1.2.0; QuickChick: init at 1.1.0 for Coq 8.9

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build coqPackages.QuickChick coqPackages_8_9.QuickChick


DOB will save and change state even if incorrect format/age

From user:

This may be random incident, but I tried to complete surveys and, as one example I was missing one complete answer for About Me b/c my DOB was incorrect.  It has only two numbers for year.  I could not change it so had to put prefer not to answer to complete the survey.

@pmanko All set on the front-end. If the locked DOB question for that user is not just a spot issue, will we need to run another data migration just for that question?

@krand7 The fixes seem to work. When I looked into this problem, the DOB question was not the one missing in that survey. The question shows up locked as it should (since it has been previously answered). However, there were two things that stood out that you might want to take a look at:

[ ] The styling for prefer not to answer on locked does not indicate that the box is locked
[ ] Client-side validation of the DOB field still occurs even though the text box is locked. We should not have client-side validation of locked answers.


m4b-tool: init at 0.4.2

Motivation for this change
Adds m4b-tool - cli for audiobook manipulation. Requires fdkaac added in #87952.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Should be rebased on master, and you should be able to drop the commit for the lib and for addition of the maintainer entry :)

All done :)

I just came to think... Is there a point of building three different packages of this package? One for each version of PHP.
I mean, since it's just a tool it should work fine to just use a single version. Some other tools it actually makes a difference for (such as composer, phpcs, phpcbf, phpstan etc). But this doesn't interact with actual PHP code.
So I'm thinking that it would fit better in like pkgs/tools/audio/m4b-tool and be registred in all-packages.nix as a normal package. And then just depend on the default php attribute. That way we only get one variant of the package.

It's a fair point. I couldn't see many php packages using that appeoach so that's why I went with php-packages.nix. Looking at python packages, however, it does seem that your approach is taken in many cases - httpie, certbot, streamlink for example (but there are some exceptions, see youtube-dl).
Making this change would also allow for ~fdkaac~ fdk-aac-encoder and fdk-enabled ffmpeg to be optional, but default, dependencies. Most of the functionality of m4b-tool seems to require them but is is possible to perform some operations without. Saves users who know what they want a heavy footprint and possibly long build for ffmpeg-full. I'll start looking into it.

Ok I have spun m4b-tool out as its own package and made fdk-aac-encoder an optional default dependency. I have also added a comment explaining how to build with ffmpeg with full FDK aac support, but it builds against ffmpeg by default.

Result of nixpkgs-review pr 87957 run on x86_64-linux 1

1 package built:

m4b-tool

Result of nixpkgs-review pr 87957 run on x86_64-linux 1

1 package built:

m4b-tool

Formatted with nixpkgs-fmt this time!

I think this might be best revisited when there is a new release, master is quite behind now.


Static widgets

Depends on #13
Originally planned for 3.0, static widgets allow widgets to be exported using nbconvert.  The mechanism of state storage has yet to be decided.

+1. If I'm interpreting correctly, it's a widget that you can embed on any webpage/get raw HTML. I would love to see this and am currently using the unsupported ipywidgets.

+1. Using Nbconvert or Nbviewer with widgets would be neat feature for teaching people without access to ipython/jupyter.

I'm confused, is it correct that currently nothing like this on Jake's blog is possible with the official ipywidgets? https://jakevdp.github.io/blog/2013/12/05/static-interactive-widgets/

This looks great. Looking forward!

This looks great guys btw. Looking forward!

+1

+1

https://github.com/jakevdp/ipywidgets-static
The problem is solved here, though native widgets look much batter. Is anyone working on this right now?

@georgebv You can tell from his own example that it no longer works. Would absolutely love to see this feature though!

+1

How is the feature described in this ticket related/different from the feature to have interactive widgets embedded into static web pages illustrated here?

The point is not having widgets in the static web page. It is about making them interact with the output (by storing all the possible output combinations in the html file).

I know @SylvainCorlay is going to write something (post) about this soon, see this comment: https://github.com/ipython/ipywidgets/issues/758#issuecomment-246200132

Will be watching this; I have a reveal.js presentation .html file and would like an interactive slider/plot combination to be usable on my static site as opposed to only while using RISE.

@SylvainCorlay, can you confirm whether there will soon be static export of ipywidgets?  We're at this very moment trying to find a different widget library so that we can switch to it for that feature alone, but if we should be staying with ipywidgets and just need to hold on for a few days longer, please let us know!

Yes! There is already. For example on http://jupyter.org/widgets, widgets are live.
With 6.0 in a few days, widgets will also be embeddable in sphinx documentation.
Nbconvert will follow.

Great, thanks!  The live widgets at that link are great, but none of those are for the standard sliders, selection boxes, text entry boxes, etc. that we are using ipywidgets for. Presumably those are also now supported?  Nbconvert is indeed my main use case, but we can probably work with Sphinx if that's the only currently supported way.

All of these are supported. The widgets in that page are built upon ipywidgets. I will add a tab with some simple dropdowns and buttons.
We have had static widgets since 5.0. The nbconvert part is the only reason why this issue is not closed yet.

Sounds great.  Getting nbconvert support is crucial for people even to know that static support is available.  Presumably documentation is also needed before closing the issue?

Yes. We are currently formalizing the spec for the format of the serialized widgets. cf #901

With the goal of serialization being to generate and insert that HTML snippet directly, without user cutting and pasting?  That does sound crucial for this to be usable in practice, thanks.

I don't understand your question, but the html snippet feature will remain for those who want to embed in webpages, blogs.
The nbconvert question is more about the choice of how the widget state is stored in the notebook format.

I just mean that nbconvert converts .ipynb to .html, and presumably when one does so one won't need to paste in the HTML snippets that create the widgets, by hand?  Basically I just want the widgets to be preserved in the exported file, with the values they had at that time as their starting point, whether or not they are hooked up to anything that actually does something in the exported HTML.  E.g. here's an example of a bunch of ipywidgets generated with static PNGs using nbconvert: https://anaconda.org/jbednar/paramnb/notebook
But that only works because of the embedded PNGs, whereas embedding the actual widgets would be so much cleaner.  Basically I just want to be able to save things as .html and have it all work, insofar as it is possible. Perhaps @philippjfr and @jlstevens could explain better.

👍 got that. Stay tuned. :)

Fabulous!  I'll hand you a drink as you sprint by, if I see you!

How is the sprint coming?  You guys must be winded by now if you've been running flat out for this long! :-)

@jbednar

we have working static widgets in sphinx docs

which is only waiting for the PR to get merged in another repo.

All the pieces are completed for nbviewer except an outstanding PR in nbconvert.

That sounds great. Can't wait!

Very nice. Just a quickie: i assume the readthedocs version is not yet working because of some final tweaks or am I maybe missing a plugin (using Safari on 10.11.6)?

Indeed, stay tuned for the rtd rendering.

Widgets are rendering in RTD http://ipywidgets.readthedocs.io/en/latest/examples/Widget List.html

Wow, nice! Out of curiousity, what is the chosen mechanism for state storage?

@akhmerov check out the section of the documentation on embedding that got improved at the same time.
http://ipywidgets.readthedocs.io/en/latest/embedding.html

Looks great!
But: The linked widgets are not linked in RTD:
http://ipywidgets.readthedocs.io/en/latest/examples/Widget Basics.html#Linking-two-similar-widgets

@michaelaye anything that requires a roundtrip to the backend will not work.
However, jslink and jsdlink do work on RTD.

@michaelaye for things that require a kernel, we are thinking of using mybinder as a kernel provider (without the notebook) for embedded widgets. I got some example working but the documentation would probably not be the main purpose.

We now support widgets rendering on nbviewer
Example here: http://nbviewer.jupyter.org/github/ipython/ipywidgets/blob/master/docs/source/examples/Widget Events.ipynb
This requires the widgets to be generated from the ipywidgets 6.0 (beta) or the current state of master and will be included in the next release.

Really really glad to solve this issue. Support of custom widgets and some css glitches will be worked on before the 6.0.
Thank you to everyone involved. @minrk @michaelpacer @jasongrout @takluyver @jdfreder.

@SylvainCorlay Great stuff! Thanks so much for your work on this!
One question: in the nbviewer link at the top if I click on the button or hit enter in the submit field nothing happens. Is that expected? Thanks!

Yes. In these static embeddings, there is no python backend. Only the pure js things will work.
We are also working on a kernel-backed thing for full fledges apps.

Gotcha! One more question: Is it possible to change plots using a radio button or a slider? Similar to http://nbviewer.jupyter.org/github/jakevdp/ipywidgets/blob/master/example.ipynb#Example-4:-Matplotlib-Figures

You can check www.github.com/Bloomberg/bqplot .. its built for doing things like this
----- Original Message -----
From: Ino de Bruijn<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>At: 21-Dec-2016 14:31:52
Gotcha! One more question: Is it possible to change plots using a radio button or a slider? Similar to http://nbviewer.jupyter.org/github/jakevdp/ipywidgets/blob/master/example.ipynb#Example-4:-Matplotlib-Figures
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


git ignores portainer container proxy settings when pulling App Templates

Bug description
When I atempt to install an Application Template such as "Iron Functions" in Portainer which depends on the https://github.com/portainer/templates the deployment eventually fails as git times out attempting to access the resource.
Expected behavior
git should use the configured proxy to successfully complete the deployment.
Steps to reproduce the issue:

Be behind an omnipresent corporate firewall that will - by default - block server access to github.com
Launch Portainer on docker hosted on such a blocked server, using either ~/.docker/config.json, --env passed to docker-compose, or an environment section in docker compose to set HTTPS_PROXY to the correct environment variable.
With Portainer pointing at some docker swarm or instance, attempt to install Iron Functions or some other built in App Template that is hosted in the portainer/templates repository

Technical details:

Portainer version: 1.22.1
Docker version (managed by Portainer): 19.03.3
Platform (windows/linux): Redhat
Command used to start Portainer: docker-compose up
Browser: Safari

Additional context
One way to set the proxies for launched containers is to ensure that ~/.docker/config.json contains the proxy settings:
{
	"proxies: {
		"httpProxy": "http://<IP_ADDRESS>:8080",
		"httpsProxy": "http://<IP_ADDRESS>:8080",
		"noProxy": "localhost,<IP_ADDRESS>,<IP_ADDRESS>/12,.example.com"
	}
}

Proxy settings are also configured directly on the Portainer service - to little apparent avail:
version: "3.7"

services:
  portainer:
    image: portainer/portainer
    ports:
      - "9100:9000"
      - "8000:8000"
    volumes:
      - data:/data
      - /var/run/docker.sock:/var/run/docker.sock
      - /etc/ssl/certs/ca-bundle.crt:/certs/ca.pem
    environment:
      GIT_SSL_NO_VERIFY: 1
      HTTPS_PROXY: http://<IP_ADDRESS>:8080
      HTTP_PROXY: http://<IP_ADDRESS>:8080
      NO_PROXY: localhost,<IP_ADDRESS>,<IP_ADDRESS>/12,.example.com
      https_proxy: http://<IP_ADDRESS>:8080
      http_proxy:  http://<IP_ADDRESS>:8080
      np_proxy: localhost,<IP_ADDRESS>,<IP_ADDRESS>/12,.example.com
      GIT_CURL_VERBOSE: 1

volumes:
  data:

docker logs portainer
2019/10/21 06:12:31 http error: Unable to clone git repository (err=Get https://github.com/portainer/templates/info/refs?service=git-upload-pack: dial tcp <IP_ADDRESS>:443: connect: connection timed out) (code=500)

which is completely consistent with what happens if I ssh to the server and try to wget https://github.com/portainer/templates/info/refs?service=git-upload-pack from the console without setting the proxies first.

Hi there,
We haven't tested deploying app templates when portainer is behind a corporate firewall.
Are you sure that configuring the docker proxy will allow requests to github? As far as I can find from some quick googling, setting the proxy like you have is used by docker to pull images from dockerhub and other registries

Setting these environment variables is the general way to set the proxy used by most tools; git, curl, wget and so on. For various reasons both cases need to be set as libcurl and tooling that relies on it prefers the lowercase variant.
So, for example, to test this, I could ssh into a container and execute the following:
 $ wget https://github.com/portainer/templates/...
 timeout connecting to....
 $ export https_proxy=http://<IP_ADDRESS>:8080
 $ wget https://github.com/portainer/templates/...
 index.html saved.

The same applies to git. It is, of course, preferable to set git config variables via git config --global http.proxy ... but the portainer container is so weird i've been unable to map that config file in anywhere where it gets used.

Thanks for your feedback on this, we will need to test it.

@deviantony I think I have confirmed this without needing to set up an HTTP proxy.

Deploy the stack above
Console into the portainer container using CNTR and execute wget https://github.com/portainer/templates/info/refs?service=git-upload-pack
See Connecting to <IP_ADDRESS>:8080... failed: Connection timed out.
Deploy the ironfunctions stack with /data/compose empty (implying a git clone has to occur)
See that it deploys correctly (I.E. git is ignoring http_proxy settings)

Further evidence that Portainer is respecting the env vars, I see [WARN] [exec,extensions] [message: unable to retrieve extensions manifest via Internet. Extensions will be retrieved from local cache and might not be up to date] [err: Get https://portainer-io-assets.sfo2.digitaloceanspaces.com/extensions-1.23.2.json: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)] meaning portainer is trying to retrieve extensions using this proxy that I don't have configured

@itsconquest this issue seems that it's about git not respecting proxy settings, your test cases just prove that any external request is failing.
Not sure if it's related.

Any news about this bug?
I'm facing the same issue trying to deploy a docker-compose from git.

I have the same bug when I try to deploy a stack from git and the docker host is placed behind a proxy. This bug occurs on Linux and Windows platforms when environment variables for the proxy are passed to the portainer container. Is it in the process of being resolved or is there a way around it ?

I would have thought that setting the environment variable should have worked, since we're using go-git: https://github.com/src-d/go-git/issues/774
But I haven't tried it myself.

I've tried it as well, and the environment variable indeed is ignored.
docker run --name portainer -d -p 8000:8000 -p 9000:9000 --env HTTP_PROXY=http://<IP_ADDRESS>:3128 --env HTTPS_PROXY=http://<IP_ADDRESS>:3128 -v /var/run/docker.sock:/var/run/docker.sock -v portainer_data:/data portainerci/portainer:2.5

Results in:

One thing to try might be to use our alpine image instead of the default Portainer image, e.g. portainer/portainer-ce:2.5.0-alpine (be careful with the known issues on that release, we're working on 2.5.1 right now and it should be available today).
 It might be a problem with our scratch based image only but I'm not 100% sure.

Hello,
I tried it :
docker run -d -p 8000:8000 -p 8083:9000 -e HTTP_PROXY=myproxy:8080 -e HTTPS_PROXY=myproxy:8080 --name=portainer --restart=always -v /var/run/docker.sock:/var/run/docker.sock -v qva:/data portainer/portainer-ce:2.5.0-alpine
Still have the timeout
2021/05/26 22:17:35 http error: Unable to create custom template (err=Get https://qvarquet:***@mygitlab.com/repo//info/refs?service=git-upload-pack: dial tcp: lookup http://mygitlab.com on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:53669-><IP_ADDRESS>:53: i/o timeout) (code=500)

@Quentinvarquet thanks for the confirmation, I wasn't sure if that would solve the issue.

Any news about this issue ?

@Quentinvarquet we are considering addressing this as part of our CI/CD efforts, but we don't have any timeline for it at the moment.

We have a preview version available. Please give it a try by using the image portainerci/portainer:pr5281
And let us know how it's working.
Note that this is a development build and should not be used in a production environment.
Example of how to specify the proxy details:
docker run --name portainer -d -p 8000:8000 -p 9000:9000 --env HTTP_PROXY=http://<IP_ADDRESS>:3128 --env HTTPS_PROXY=http://<IP_ADDRESS>:3128 -v /var/run/docker.sock:/var/run/docker.sock -v portainer_data:/data portainerci/portainer:pr5281

Hello, it's working


Merge stage to main

Test URLs:

Before: https://main--express--adobecom.hlx.page/express/
After: https://stage--express--adobecom.hlx.page/express/

Codecov Report

Merging #420 (a4499cf) into main (09cadc2) will decrease coverage by 0.02%.
Report is 8 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #420      +/-   ##
==========================================
- Coverage   31.28%   31.27%   -0.02%
==========================================
  Files         124      124
  Lines       25126    25126
==========================================
- Hits         7860     7857       -3
- Misses      17266    17269       +3

see 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


chore(*) fix lints

Fixed linting across all files.

@tharun208 thanks for this one! Can we enable this lint as part of the CI so it does not allow merging non-compliant changes?

yes, @nickolaev thinking the same. I will work on this 👍

@tharun208 please rebase to check it against CI now!

@nickolaev, I rebased and it is not running the kuma-commit workflow

Maybe it will work on new PRs, can you please close this one and create it once again?


Fixes cos url

Could you tell me more about this change? Is there documentation on the Google Cloud site with a list of these? Sorry, it's been a while since I made this, and I'm pretty sure I was just referencing an upstream list.

Current images: https://cloud.google.com/compute/docs/images
Rename announce: https://groups.google.com/forum/#!topic/gce-announce/v9JSnre92I8
Release history: https://cloud.google.com/container-optimized-os/docs/release-notes (you can see the rename gci to cos)

You can also try running your test :) they will fail ;)

@stephenplusplus poke :)

Sorry! Thank you! Merged!

Published under 0.3.4.


Ability to expose simulated kubestate-metrics endpoint for added church and cardinality

Many Victoriametrics users are likely monitoring k8s resources and kubestate-metrics is often notorious for high cardinality and churn.  In some cases, users may have specific use cases where they need to map additional k8s labels and/or annotations as metric labels which can lead to substantial memory consumption.  It would be great to additionally to have the ability to expose an endpoint similar to kubestate-metrics which simulates a configurable level of churn for resources of type deployment, job, and statefulset.  The first two are most relevant as environments where a large amount of jobs are scheduled throughout the day (e.g.: Prefect 2), this can lead to a massive amounts of churn in a 24h period.
The ability to combine this along with the current simulated node_exporter metrics would give an even better idea as to what to expect from VM with regards to performance and resource usage.

Thanks for the feature request! So far this benchmark doesn't have endpoint for adjusting churn rate by resource type.
But it has settings to configure the churn rate of the scraped targets - see https://github.com/VictoriaMetrics/prometheus-benchmark/blob/f6a69052413618c607758d5469e43e508792aff7/chart/values.yaml#L23-L38


Change calendar locale

Hello,
This is an awesome library!! I was wondering if you are able to allow the GLCalendarView to set a custom locale for its NSCalendar variable (which is readonly)?
Currently, it is using the currentLocale from the device. Which is not possible to override.
Thanks!

You can access the global calendar by [GLDateUtils calendar]


When I turn hotspot on nothing works

I turn hotspot on and all it says is send this to victim: and there is nothing after that

bro please leave a screenshot of the error

bro please leave a screenshot of the error

I can’t get it to work

I can’t get it to work

Bro please install curl before running this tool..
cmd
pkg install curl ; apt-get install curl


added couchbase probe for couchnode package

#494

Codecov Report

Merging #498 into master will decrease coverage by 0.35%.
The diff coverage is 29.72%.

@@            Coverage Diff             @@
##           master     #498      +/-   ##
==========================================
- Coverage    60.8%   60.45%   -0.36%
==========================================
  Files          49       50       +1
  Lines        3215     3252      +37
==========================================
+ Hits         1955     1966      +11
- Misses       1260     1286      +26

Impacted Files
Coverage Δ

probes/couchbase-probe.js
29.72% <29.72%> (ø)

Continue to review full report at Codecov.

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

I'm not sure what is making codecov fail.

@astub before we can accept your change request, please can you read and accept
https://github.com/RuntimeTools/appmetrics/blob/master/CONTRIBUTING.md

@astub if your test can be run without needing a database to be set up (looks like a mock one?) then you can add it to the list of tests we run in package.json  https://github.com/RuntimeTools/appmetrics/blob/master/package.json#L32.  That would also fix the codecov decrease.

@sjanuary Hi. It uses couchbase.Mock so it does not need a couchbase server. I'm unsure what else i'm missing for this PR.

@astub - you need to rebase as master has changed since this PR and there are conflicts. Also if you rename your test file to end in "tests.js" then it will be run automatically - see https://github.com/RuntimeTools/appmetrics/blob/master/package.json#L32

@sjanuary Thanks for your feedback. Anything else I need to improve/fix?

@astub please could you also add a description the couchbase event to the Readme?  It needs to go in the table at the start and also here.

closing this as its a copy of #506


Feature Request: Be able to show the individual rolls

Showing the total works great, it'd be nice to also be able to show the individual results, or a populated expression: 4d10dl2 + 5 would give you [1*,7,4*,5] + 5.
Maybe a renderedExpression property on Result?
I tried looking at this, but typescript/javascript are not my forte, and I got lost in the AST/Expressions.
Really cool project, even cooler when I saw the 5th ed e-tools you've put together!

Thanks for the feedback! That makes a lot of sense. There's currently an empty printer.ts file that I was going to use for rendering an expression! I'll see when I can get some time to look into it.

This is now in 1.4.0 (52d958f81c69963c9a623c0fb3c27245663b6df5), which is available in NPM. Hopefully it works as you expect! It will certainly work for simple expressions, but lengthier dice rolls will probably get complex rather quickly.

Awesome! Works great!


Test collection hanging on openSUSE Tumbleweed x86_64

My builds halt at
[   30s] + /usr/bin/python3 setup.py test
[   31s] running pytest
[   31s] running egg_info
[   31s] writing grpcio_opentracing.egg-info/PKG-INFO
[   31s] writing dependency_links to grpcio_opentracing.egg-info/dependency_links.txt
[   31s] writing requirements to grpcio_opentracing.egg-info/requires.txt
[   31s] writing top-level names to grpcio_opentracing.egg-info/top_level.txt
[   31s] reading manifest file 'grpcio_opentracing.egg-info/SOURCES.txt'
[   31s] writing manifest file 'grpcio_opentracing.egg-info/SOURCES.txt'
[   31s] running build_ext
[   31s] ============================= test session starts ==============================
[   31s] platform linux -- Python 3.8.6, pytest-6.1.2, py-1.9.0, pluggy-0.13.1
[   31s] rootdir: /home/abuild/rpmbuild/BUILD/python-grpc-1.1.4
[   31s] collected 43 items

Build project at https://build.opensuse.org/package/show/home:jayvdb:opentracing/python-grpcio-opentracing , where full logs can be obtained for the builds.
The only obvious difference is Python 3.8 instead of the 3.6 used locally here in CI.

OK, this gets wierder.
https://build.opensuse.org/package/show/home:jayvdb:py-submit/python-grpcio-opentracing is a clone of the above build project, with more target architectures and versions of openSUSE, and the i586 build work, and the Python 3.6 builds work.


pull request for e2e test at 12/02/2016 21:45:04

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


npm install for client shows vulnerabilities fixing them breaks client

Fresh clone of fabcar-blockchain-sample.
I run npm install:
jjsretina:client jjs$ npm install

><EMAIL_ADDRESS>install /Users/jjs/Documents/Villanova/ECE8498Spring2019/Week13/fabcar-blockchain-sample/web-app/client/node_modules/fsevents
> node install

node-pre-gyp WARN Using request for node-pre-gyp https download
[fsevents] Success: "/Users/jjs/Documents/Villanova/ECE8498Spring2019/Week13/fabcar-blockchain-sample/web-app/client/node_modules/fsevents/lib/binding/Release/node-v64-darwin-x64/fse.node" is installed via remote

><EMAIL_ADDRESS>install /Users/jjs/Documents/Villanova/ECE8498Spring2019/Week13/fabcar-blockchain-sample/web-app/client/node_modules/node-sass
> node scripts/install.js

Cached binary found at /Users/jjs/.npm/node-sass/4.11.0/darwin-x64-64_binding.node

><EMAIL_ADDRESS>postinstall /Users/jjs/Documents/Villanova/ECE8498Spring2019/Week13/fabcar-blockchain-sample/web-app/client/node_modules/node-sass
> node scripts/build.js

Binary found at /Users/jjs/Documents/Villanova/ECE8498Spring2019/Week13/fabcar-blockchain-sample/web-app/client/node_modules/node-sass/vendor/darwin-x64-64/binding.node
Testing binary
Binary is fine
added 1198 packages from 1330 contributors and audited 40441 packages in 12.027s
found 9 vulnerabilities (2 low, 7 high)
  run `npm audit fix` to fix them, or `npm audit` for details

If I do npm audit fix, the app will not open:
jjsretina:client jjs$ npm start

><EMAIL_ADDRESS>start /Users/jjs/Documents/Villanova/ECE8498Spring2019/Week13/fabcar-blockchain-sample/web-app/client
> ng serve

** Angular Live Development Server is listening on localhost:4200, open your browser on http://localhost:4200/ **

Date: 2019-04-29T01:44:30.551Z
Hash: 84be1010ddee4fbede4c
Time: 5392ms
chunk {main} main.js, main.js.map (main) 140 kB [initial] [rendered]
chunk {polyfills} polyfills.js, polyfills.js.map (polyfills) 237 kB [initial] [rendered]
chunk {runtime} runtime.js, runtime.js.map (runtime) 6.08 kB [entry] [rendered]
chunk {styles} styles.js, styles.js.map (styles) 16.4 kB [initial] [rendered]
chunk {vendor} vendor.js, vendor.js.map (vendor) 3.46 MB [initial] [rendered]

ERROR in ./src/styles.css (./node_modules/@angular-devkit/build-angular/src/angular-cli-files/plugins/raw-css-loader.js!./node_modules/postcss-loader/src??embedded!./src/styles.css)
Module build failed (from ./node_modules/postcss-loader/src/index.js):
SyntaxError

(1:4) Unknown word

> 1 | // SCSS Color Palette Variables
    |    ^
  2 | $ibm-color__black-100: #000000;
  3 | $ibm-color__blue-10: #EDF4FF;

ℹ ｢wdm｣: Failed to compile.

Any ideas?

@jsolderitsch - we will have to ignore the vulnerabilities at the moment - because IBP requires npm 5.5.1 and the audit fix requires npm 6 - so it is not compatible.

@jsolderitsch - so I have updated the version of the libraries that were causing the vulnerabilities and the app is still working for me - please clone or download the latest version and give it another try. tx


Add documentation for some Iterator methods

I think in general it would be good to highlight that they return new iterators, in the examples too (which are just not right in many cases, sorry :/)

@jhass okay, it's no problem :)
Bun could you show not right examples?

@davydovanton are you looking to pick this up again? Otherwise I might take it and fix it :)

@PragTob yep, sorry, I have some problems with other project (deadline :sweat:). I'll fix this tomorrow

@davydovanton no worries, just checking in :)

Ping, could free any time by now? :)

@jhass I'm so sorry. I completed my GSoC project. :disappointed: I'll update this in the coming days :sweat:

@davydovanton if you don't make it in the coming days, any objections to me picking up this PR, rebasing it and adding other documentation for new methods? :) Want to get those docs in!

@davydovanton There's really no pressure in you doing this. If other things have more priority you should definitely do those! There are plenty of other things where you can later contribute.
@PragTob You can go ahead and submit a separate pull request for the iterator docs :-)

closed by #1352


delete "test scope"

为了支撑本地调试，还需要将mysql的依赖scope注释掉test

Notice:  the english and other version docs need to modify together

No follow-up feedback A new PR has been merged #https://github.com/apache/incubator-linkis-website/pull/609


Use util functions to handle function name case issues

Thank you

@sarsamurmu By the way, this PR also fixes the case problem of jsx related function names in the is util. Could you please release a new version again?

Ok, I have published a new version. I hope it helps with your project 👍


Allow first param of createComponent(WithProxy) to be optional css rule object

Type: Feature

Packages
Version

fela
6.x

Description
I often have static css rules. In these cases, it would be nice to pass in an object to createComponent and createComponentWithProxy instead of being required to wrap it in a function.

Proposal
I would like to see createComponent and createComponentWithProxy changed to accept either an object or function as their first parameter. The other params would not change.
Code Example:
Current function-only param
createComponent(() => ({
  some: 'rule',
  another: 'rule'
}));

Proposed optional object param
createComponent({ some: 'rule', another: 'rule' });

You're using version 6 right? The newly released Fela 10+ already supports plain object usage :P
I'll add a test later to show that. For now I'll close this issue.

Well..exellenct! ;)