Attempt at implementing pathlib Path. Regression test failing because…

… of wrong path.

@SeBecker , could you please help me on this one. I tried to implement pathlib Path usage in soepy_config. However the regression test is failing because of a false pathname to the regression vault.
def test1():
    """This test runs a random selection of five regression tests from the our regression test
    battery.
    """

    fname = TEST_RESOURCES_DIR / "regression_vault.soepy.json"

  tests = json.load(open(fname))

E       FileNotFoundError: [Errno 2] No such file or directory: '/Users/boryanailieva/Projects/test/resources/regression_vault.soepy.json'
I do not understand where the problem is, because the TEST_RESOURCES_DIR path in soepy_config.py is correct. There must be some tiny detail that I am missing somewhere...

Also, do .pytest_cache files get ignored on your machine. They seem not to be ignored on mine.. How do we handle this? gitignore?

Please also add .idea/ for the local Pycharm configuration files to gitignore.

Thanks for the follow up. Sebastian and I will discuss the issue on
Wednesday in person and and then get back to you.
On Mon, Mar 4, 2019 at 11:47 AM Boryana<EMAIL_ADDRESS>wrote:

@boryana-ilieva commented on this pull request.
In soepy/python/soepy_config.py
https://github.com/OpenSourceEconomics/soepy/pull/15#discussion_r262007252
:

@@ -9,6 +10,6 @@
np.testing.assert_equal(sys.version_info[1] >= 5, True)

We rely on relative paths throughout the package.
-ROOT_DIR = os.path.dirname(os.path.realpath(file))
-PACKAGE_DIR = ROOT_DIR.replace("/python", "")
-TEST_RESOURCES_DIR = PACKAGE_DIR + "/test/resources"
+ROOT_DIR = Path.cwd()
@peisenha https://github.com/peisenha , I was aware of the link above,
but I am lost on how to adapt the very first line to my setup: "ROOT_DIR =
Path(respy.path[0])" to my setup. I am under the impression that with
Tobias' code it works after "import respy", but I cannot do "import soepy"..
@SeBecker https://github.com/SeBecker , thank you, at least I now know
what the problem is.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OpenSourceEconomics/soepy/pull/15#discussion_r262007252,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADcMJzaKk4FLZTB0om6ITDmepo5QEuvUks5vTPmxgaJpZM4bXPW_
.

--
Philipp Eisenhauer
Economist
Mail<EMAIL_ADDRESS>Web www.eisenhauer.io
Repository https://github.com/peisenha

@boryana-ilieva , I talked about the issue with @SeBecker and he will let you know how to proceed later today. If there are questions remaining afterwards, we can address them on our call tomorrow.

At this stage I implemented everything as advised by Sebastian but import soepy is still giving me a PackageNotFoundError. I am afraid we will have to have a look at this when in Bonn next week and possibly maybe ask Tobias.


lightline not working for me

I am having an issue getting lightline to work. I am using oh-my-zsh and this is my .vimrc
syntax on
set ruler
set number

set tabstop=4
set softtabstop=0
set shiftwidth=4
set expandtab

set mouse=a

if !has('gui_running')
    set t_Co=256
endif

set laststatus=2
set nocompatible              " be iMproved, required
filetype off                  " required

" set the runtime path to include Vundle and initialize
set rtp+=~/.vim/bundle/Vundle.vim
call vundle#begin()
" alternatively, pass a path where Vundle should install plugins
"call vundle#begin('~/some/path/here')

" let Vundle manage Vundle, required
Plugin 'VundleVim/Vundle.vim'

" The following are examples of different formats supported.
" Keep Plugin commands between vundle#begin/end.
" plugin on GitHub repo
Plugin 'tpope/vim-fugitive'
" plugin from http://vim-scripts.org/vim/scripts.html
" Plugin 'L9'
" Git plugin not hosted on GitHub
Plugin 'git://git.wincent.com/command-t.git'
" git repos on your local machine (i.e. when working on your own plugin)
" Plugin 'file:///home/gmarik/path/to/plugin'
" The sparkup vim script is in a subdirectory of this repo called vim.
" Pass the path to set the runtimepath properly.
Plugin 'rstacruz/sparkup', {'rtp': 'vim/'}
" Install L9 and avoid a Naming conflict if you've already installed a
" different version somewhere else.
" Plugin 'ascenator/L9', {'name': 'newL9'}
Plugin 'octol/vim-cpp-enhanced-highlight'
Plugin 'itchyny/lightlin.vim'
" All of your Plugins must be added before the following line
call vundle#end()            " required
filetype plugin indent on    " required
" To ignore plugin indent changes, instead use:
"filetype plugin on
"
" Brief help
" :PluginList       - lists configured plugins
" :PluginInstall    - installs plugins; append `!` to update or just :PluginUpdate
" :PluginSearch foo - searches for foo; append `!` to refresh local cache
" :PluginClean      - confirms removal of unused plugins; append `!` to auto-approve removal
"
" see :h vundle for more details or wiki for FAQ
" Put your non-Plugin stuff after this line```

The plugin name is lightline.vim, not lightlin.vim.


flutter upgrade (dart sdk download) can never complete because of incorrect curl command in update_dart_sdk.sh

I am in a country where accessing overseas network is very unstable (unreliable). My overseas connection is always interrupted/dropped/reset.
It becomes a serious issue when the download process can not be resumed.
I tried to execute flutter upgrade and it could never complete. When downloading dart sdk, it always met curl: (56) Unexpected EOF error message. I thought it should resume the download when I restart the upgrade process, but it didn't.
To further investigate the bug, I found out the curl command was called by the script flutter/bin/internal/update_dart_sdk.sh. And the curl command is incorrect in my linux OS. The curl command in the current update_dart_sdk.sh is:
curl -continue-at=- --location --output "$DART_SDK_ZIP" "$DART_SDK_URL" 2>&1

But it should be
curl --continue-at - --location --output "$DART_SDK_ZIP" "$DART_SDK_URL" 2>&1

Note: It is --continue-at - instead of -continue-at=-
After changing the incorrect curl command and re-run flutter upgrade several times, I can finally complete the dart sdk download process. Otherwise, if it always starts to download from scratch, the dart sdk download process can never complete.
Flutter version
[✓] Flutter (on Linux, locale en_US.UTF-8, channel master)
• Flutter at /home/song/opt/flutter
• Framework revision 5462ddb917 (4 hours ago), 2017-08-21 16:17:54 -0700
• Engine revision 600567ef15
• Tools Dart version 1.25.0-dev.11.0

Sounds like an easy win.  Thoughts @tvolkert @jason-simmons?

Although in my update_dart_sdk.sh
curl --continue-at - --location --output "$DART_SDK_ZIP" "$DART_SDK_URL" 2>&1
I am getting this error:

Downloading Dart SDK from Flutter engine ead227f118077d1f2b57842a32abaf105b573b8a...
/home/ngima/Android/flutter/bin/internal/update_dart_sdk.sh: line 54: curl: command not found

i have an issue on line 40 on the above file,
DART_SDK_BASE_URL="${FLUTTER_STORAGE_BASE_URL:-https://storage.googleapis.com}"
it says command not found if i run flutter doctor,and sdk starts downloading.


Ctrl+C'ing out of a Dart script breaks Windows console

In the following screenshot I executed the script test1.dart twice: The first time I let the script finish. The second time, I terminated the script in the middle of execution with Ctrl+C. When doing the latter, the Windows console breaks for subsequent commands that (try to) print to the console (see strange rectangular boxes in the screenshot).

I believe this is an undesired side-effect of the changes introduced to fix #28571.
/cc @zanderso

This is interesting: My default code page is 437. After Ctrl+C'ing out of a Dart script  to break my console, chcp claims that I am still on code page 437. However, explicitly running chcp 437 again seems to fix the console.

At least according to this:
https://sourceforge.net/projects/chcp-advanced/
chcp changes both input and output codepage, but only displays the input code page. So if the Dart VM was changing the output codepage and not touching the input codepage, then that could explain what we're seeing.

Yeah, looks like that's what going on:
Code pages before Ctrl+C'ing out of a Dart script:
C:\Users\goderbauer> [Win32.NativeMethods2]::GetConsoleCP()
437
C:\Users\goderbauer> [Win32.NativeMethods2]::GetConsoleOutputCP()
437
C:\Users\goderbauer> chcp
Active code page: 437

Code pages after Ctrl+C'ing out of a Dart script:
C:\src\flutter [windowsVersion]> [Win32.NativeMethods2]::GetConsoleCP()
437
C:\src\flutter [windowsVersion]> [Win32.NativeMethods2]::GetConsoleOutputCP()
65001
C:\src\flutter [windowsVersion]> chcp
Active code page: 437

As you suspected, the output code page does not get resetted. Fixing that will probably fix this output problem.

Registering a signal handler for Ctrl+C and calling exit from there seems to be a work around for this problem:
ProcessSignal.SIGINT.watch().listen((_) {
    exit(-1);
});

I think we should handle this in the VM. It's weird to make you register a handler that just exits. I'll take a look at this after landing my other change.

I can confirm that this is fixed now! Thanks! \o/


Added New Linking Objects Syntax to Example Apps

A very minor PR that brings the 'Backlinks' sample app in line with the new linking objects properties feature syntax.
/cc @bdash @jpsim

Thanks for the PR, @TimOliver! We'll need you to accept our Contributor License Agreement before we can merge your contributions.
Cheers,
The robots and humans at Realm

Thanks, Tim! :+1:


l4t_version_compatible() wrongly flags JetPack6 R36 images as compatible on JetPack5 R35 hosts

https://github.com/dusty-nv/jetson-containers/blob/93739890ec4f575c113e0c8cfad76a6e4e1a51d3/jetson_containers/l4t_version.py#L243
">=35" should be "==35" ??

Ahh thanks @frostydonut , you are correct!  Just fixed this in https://github.com/dusty-nv/jetson-containers/commit/2b178c8ce897556035ecdbf49bb91c1716716ac1
Note that if it pulled any r36.2.0 images to your system, you should remove them with docker rmi, because autotag will defer to your local images first (and for those it doesn't do version checks).  Once you have r36.2.0 removed, it will check dockerhub again for r35 images if you pull the latest jetson-containers that includes the patch above.

Thanks, Dusty! Awesome resource btw. This got me dusting-off an old Xavier
NX and playing with some cool new stuff again. Thanks for bestowing this on
the world! Cheers!
On Thu, Dec 14, 2023 at 7:23 AM Dustin Franklin @.***>
wrote:

Ahh thanks @frostydonut https://github.com/frostydonut , you are
correct! Just fixed this in 2b178c8
https://github.com/dusty-nv/jetson-containers/commit/2b178c8ce897556035ecdbf49bb91c1716716ac1
Note that if it pulled any r36.2.0 images to your system, you should
remove them with docker rmi, because autotag will defer to your local
images first (and for those it doesn't do version checks). Once you have
r36.2.0 removed, it will check dockerhub again for r35 images if you pull
the latest jetson-containers that includes the patch above.
—
Reply to this email directly, view it on GitHub
https://github.com/dusty-nv/jetson-containers/issues/353#issuecomment-1856049109,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BEFLIKPAKVWC7CZLZOBVYOLYJMKVJAVCNFSM6AAAAABAUFZO2KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJWGA2DSMJQHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @dusty-nv,  what are the ramifications of running r36.x images on an r35.x host? I've just sat down to try and use the humble-ros-base-l4t-r36.2.0 on my Xavier NX. My thought (hope) was that I could run the majority of my ROS nodes in that newer humble container, and then put the few which use GPU in to a separate roxy/r35.x container

@frostydonut no problem, happy to be of service!
@adamwhats the r36 images aren't compatible with r35, not for GPU at least.  Otherwise, yea you can run whatever as long as it was built for aarch64.  And if you aren't using GPU for some nodes, then you can just run the upstream OSRF ROS containers for those:   https://github.com/osrf/docker_images   my guess is you would prefer one with a 22.04 base since that has tier-1 support for Humble and their debian packages.


Content designer recruitment

Recruit content designer for service manual team

We have 5 CVs. Georgina and Amy to review.

@amyj2110 @georginaplatt, Susie Wood (content designer) has offered to join an interview panel, when we get to that stage.


Gl error fix

Fixed video error check
Found and fixed openGL error

SonarQube analysis reported 9 issues

 9 minor

Watch the comments in this conversation to review them.
8 extra issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 AncientFungus:LGEPR: Cppcheck cannot find all the include files (use --check-config for details)
 RenderManager.cpp#L495: The function 'GetAnimations' is never used.
 RenderManager.cpp#L513: The function 'GetDirectionalLights' is never used.
 RenderManager.cpp#L532: The function 'GetLenses' is never used.
 RenderManager.cpp#L552: The function 'GetMaterials' is never used.
 RenderManager.cpp#L570: The function 'GetMeshes' is never used.
 RenderManager.cpp#L589: The function 'GetPointLights' is never used.
 RenderManager.cpp#L611: The function 'GetSpotLights' is never used.

SonarQube analysis reported 9 issues

 9 minor

Watch the comments in this conversation to review them.
8 extra issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 AncientFungus:LGEPR: Cppcheck cannot find all the include files (use --check-config for details)
 RenderManager.cpp#L495: The function 'GetAnimations' is never used.
 RenderManager.cpp#L513: The function 'GetDirectionalLights' is never used.
 RenderManager.cpp#L532: The function 'GetLenses' is never used.
 RenderManager.cpp#L552: The function 'GetMaterials' is never used.
 RenderManager.cpp#L570: The function 'GetMeshes' is never used.
 RenderManager.cpp#L589: The function 'GetPointLights' is never used.
 RenderManager.cpp#L611: The function 'GetSpotLights' is never used.

SonarQube analysis reported 9 issues

 9 minor

Watch the comments in this conversation to review them.
8 extra issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 AncientFungus:LGEPR: Cppcheck cannot find all the include files (use --check-config for details)
 RenderManager.cpp#L495: The function 'GetAnimations' is never used.
 RenderManager.cpp#L513: The function 'GetDirectionalLights' is never used.
 RenderManager.cpp#L532: The function 'GetLenses' is never used.
 RenderManager.cpp#L552: The function 'GetMaterials' is never used.
 RenderManager.cpp#L570: The function 'GetMeshes' is never used.
 RenderManager.cpp#L589: The function 'GetPointLights' is never used.
 RenderManager.cpp#L611: The function 'GetSpotLights' is never used.


textTrackStyle for iOS/Android

Hi there,
First of all: great lib :)
We're currently evaluating if THEOplayer is the right way to go for us.
One potential blocker is that textTrackStyle is not available for iOS/Android according to the docs:
/**
 * The text track style API.
 *
 * @remarks
 * Only available for Web.
 */
readonly textTrackStyle: TextTrackStyle;

Is there some other way to achieve this? If not, are there any plans to support this? If so, is there a timeline for it?
Thanks!

Hi @kamilafsar , thanks for reaching out!
Indeed, the texttrack styling API of react-native-theoplayer is currently only available for Web, but the native iOS/Android SDKs already include this feature, so the remaining work is bridging it towards react-native.
We do not have this work on our roadmap currently, but we frequently adjust the planning based on customer needs. I'd suggest you contact us offline to see if we can work something out. I can bring you in contact with our sales team.


enh: run slack webhooks on all our slack_configurations' data sources

This PR starts a workflow for each SlackConfigurations' connectorIds when we receive a Slack webhook.
Otherwise if we have multiple manaaged data sources from multiple workspace pointing to the same slack team we would not update all of them (just one).
Note: we do receive only one webhook even if we connected from multiple data sources, it's because it's one webhook per Slack app per slack team.
r? @lasryaric

@lasryaric I refactored the code a bit trying to make it more readable and standardizing on the use of apiError. So please review carefully :pray:

Invariants I tried to preserve:

all errors through apiError
all 200 are preceded by a logger.info

If a workflow start fails for one we return an error which will retry the webhook but this is generally unlikely to fail.

@lasryaric what would be the tests you suggest?
Connect in dev with ngrok test that we get the updates, leaving a channel / joning a channel trigger the workflow ?

@lasryaric updated r? PTAL

@lasryaric tested in dev with Dust staging app. Confirmed channel_join and message work as expected
channel_left or channel_deleted I could not test because I don't believe we've registered for these events in Dust staging and checked that they are not registered for our production Dust app either.
Is that expected?


rdf4j-rio-rdfjson incompatible with Jackson 2.7+

Currently the RDF4J code is using a method in rdf4j-rio-rdfjson (Lf2SpacesIndenter) that doesn't exist in Jackson 2.7+ as Jackson have a policy where they remove public APIs across minor versions. It is simple to fix without changing the version of Jackson that legal has approved for distribution, but will change the range of Jackson versions that we are compatible with. They had deprecated the API before introducing a new named public API that does exactly the same thing in a newer version and the replacement exists in 2.6 that we have legal approval for.

Tracking issue for asking jackson maintainers what their public API versioning policy is:
https://github.com/FasterXML/jackson-core/issues/308
The commit that broke compatibility is:
https://github.com/FasterXML/jackson-core/commit/f8d8856524fbaf9b72245ee84fd20c4afa996620

I will work on a pull request for this.


Scala 2.12.2 support added

I've had to rewrite a deprecated scala.concurrent.util.Unsafe class with Scala and put it into your project. By the way, It will not work on Java 9.
Also I've removed a deprecated -optimise flag from compiler parameters for Scala 2.12 because it would lead to compilation errors.
Besides I've updated versions of scalatest and scalacheck, because previous versions have no builds for Scala 2.12.

Great! Thank you!


[Build] Automate Atmosphere Deploy

See https://github.com/percolatestudio/atmosphere/issues/395#issuecomment-120751201.
I thought everything was automated...

This needs to be added to admin scripts used to register component repos, as part of deploy process.
https://github.com/Semantic-Org/Semantic-UI/blob/master/tasks/admin/register.js
I wish they just auto-updated with tagged git releases...

Oh wait. I forgot we're using autopublish.

Will have to see how this goes..

2.0.4 updated fine
https://atmospherejs.com/semantic/ui-css

I manually pubilshed the css release. Autopublish is not working for the rest. I contacted the author he's on vacation. Fix will come soon

Could you push 2.0.7?

Published, auto-publish is still failing..

Jack, any progress on this?
https://atmospherejs.com/semantic/ui-css is two releases behind.

Any plans on updating the atmosphere package to the latest version soon? A workaround if not would be helpful.


lxc config string in config file is limited in 16 bytes

lxc config string should be less than 16 bytes.
For example, if lxc.network.veth.pair is 'leebaokleebaok-0', container will start failed.
Now, we use UserName-ID-Offset as lxc.network.veth.pair and this may exceed the limited length.

that means if we can not ask lxc change its 16 bytes setting, and probably we can not, we have to use a inner id, instead of he username, i.e., userid-id-offset. the userid could be 6 bytes long numerical, which means a max of 999,999 users, enough for an organisation? or 6 bytes long num+alphabet, a max of 36**6=2,176,782,336.
the inner id could be the key from the usertable.

or the ID in UserName-ID-Offset need not be a global id, but a local id, starting from 0 for each user.

Now, just use ClusterID-ContainerID as lxc.network.veth.pair name


If I want to implement SiganalR on Android, how can I achieve this?

Hello
In the use of.Net Core, I found that  SignalR doesn't seem to provide support for the Android version.
If I want to implement SiganalR on Android, how can I achieve this?

@BrennanConroy or @anurse, Do you have any guidance to provide?

The SignalR client fully supports Xamarin. Are you seeing errors using it?

@anurse  I tried to request chat hub, but the resulting network request was fed back to http405 error.

I'm going to need a lot more detail as to how your server is configured and how your client is written. I suggest you take a look at our Xamarin client sample: https://github.com/aspnet/SignalR-samples/tree/master/Xamarin
If you're still stuck, please collect server and client traces as suggested in the Diagnostics Guide and provide an example of your client and server (Startup, as well as the Hub) code.
We've got examples of SignalR working fine in Xamarin scenarios so we need to try and track down what might be different about your scenario :).

@anurse do we have a public sample we can point to?

Yep, the one I mentioned in my comment: https://github.com/aspnet/SignalR-samples/tree/master/Xamarin

Closing this, assuming the sample provided in preceding comment helped to resolve the issue.  Feel free to reopen if that is not the case.


Tag modal component

Extract tag modal as a separate component.

@demamoh Have you had a chance to look at this PR?  It would be nice to get it merged before the meeting on Thursday.

@tmciver  Will be reviewing now. Should wrap up in a bit.


Question: how to handle camelCase for entity name in variable name

var fooSaaSBar = ... or var fooSaasBar?
And while we're at it, how do we handle something like company names in full and acronym form? Specifically something like:
fooIBMBar vs fooIbmBar
fooGitHubBar vs fooGithubBar

In camel case style, the capital letters indicate the start of a new word. I would personally do fooSaasBar, fooIbmBar, and fooGithubBar since you want to treat "SAAS", "IBM", and "GitHub" as a single word. For the same reason, I also do peerId instead of peerID, for example.


Added workflows to publish snapshots and releases to Maven Central.

Signed-off-by: Daniel (dB.) Doubrovkine<EMAIL_ADDRESS>Highly inspired from https://github.com/aws/random-cut-forest-by-aws/pull/334 and https://docs.github.com/en/actions/publishing-packages/publishing-java-packages-with-maven.
Will need MAVEN_GPG_PRIVATE_KEY, MAVEN_GPG_PASSPHRASE, OSSRH_USERNAME and OSSRH_TOKEN in repo secrets. There's a pretty good explanation in https://dzone.com/articles/how-to-publish-artifacts-to-maven-central of how to get all those things if you need help.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@dblock could you have a look now? I've updated the release process so if you have all the needed tools installed in your machine by running make release 2 pull request will be raised one for the release and one for the next working version as draft. Maintainer have to merge them in order, everything else should be automatic. The idea being if this works well an automation can be added to avoid the PRs and do the same directly on master.
If it looks good to you I'll write the RELEASING.md, I'll just hold it in case some changes are needed. The commands were mostly written by heart or tested in Linux, if you are using a different platform they might not work exactly the same.

Btw, if you want to push some limits and really make something spectacular, you could implement all of the stuff in Makefile in GitHub actions. For example, when a release happens (a tag is created) a workflow would make a PR to increment for the next development iteration. Here's an example: https://github.com/opensearch-project/OpenSearch/blob/main/.github/workflows/version.yml.

One more thing. Any reason not to create a SNAPSHOT with every build automatically?

@dblock on your comments for the PR:

One more thing. Any reason not to create a SNAPSHOT build with every build automatically? Do we need a Makefile target?

This is the case at the moment. On push to master a there is a deploy to Maven, I modified the workflow so it will happen regardless of it being SNAPSHOT or not. If it's not SNAPTHOT then a tag is generated, but it will deploy to Maven every push to master.

Btw, if you want to push some limits and really make something spectacular, you could implement all of the stuff in Makefile in GitHub actions. The release would be a 1-click workflow executed on demand that opens a PR to make a release (won't need tools like gh, makefiles, etc.). And when a release happened (a tag was created) a workflow would make a PR to increment for the next development iteration. I don't know of a maven repo that has all these, but here's an example of something triggered on tags that opens PRs: https://github.com/opensearch-project/OpenSearch/blob/main/.github/workflows/version.yml.

That's the idea, but I thought I'd do it in 2 steps. First make it work manually to enable releasing and create a new ticket to automate it.  That will trigger the workflow that already exists and it's fully automated. But that will take me a bit longer as I haven't played much with GitHub actions and I didn't want to keep holding the release on it being perfect, but it will def be soon. Thank you for the example, it's helpful. WDYT?

@dblock on your comments for the PR:

One more thing. Any reason not to create a SNAPSHOT build with every build automatically? Do we need a Makefile target?

This is the case at the moment. On push to master a there is a deploy to Maven, I modified the workflow so it will happen regardless of it being SNAPSHOT or not. If it's not SNAPTHOT then a tag is generated, but it will deploy to Maven every push to master.

Btw, if you want to push some limits and really make something spectacular, you could implement all of the stuff in Makefile in GitHub actions. The release would be a 1-click workflow executed on demand that opens a PR to make a release (won't need tools like gh, makefiles, etc.). And when a release happened (a tag was created) a workflow would make a PR to increment for the next development iteration. I don't know of a maven repo that has all these, but here's an example of something triggered on tags that opens PRs: https://github.com/opensearch-project/OpenSearch/blob/main/.github/workflows/version.yml.

That's the idea, but I thought I'd do it in 2 steps. First make it work manually to enable releasing and create a new ticket to automate it.  That will trigger the workflow that already exists and it's fully automated. But that will take me a bit longer as I haven't played much with GitHub actions and I didn't want to keep holding the release on it being perfect, but it will def be soon. Thank you for the example, it's helpful. WDYT?

I agree that we don't need PRs where we can just commit/release, and this is simpler!
If you want to merge this change as is and make a release, that's more than OK with me. Up to you.

Good, I'll tackle it separately then: https://github.com/acm19/aws-request-signing-apache-interceptor/issues/53. I want to try the release and make sure that works. Also there are some outstanding problems we haven't solved yet.


use of unresolved identifier 'GenericResponseSerializer'

Why am I getting an error? I just upgraded to swift 2 and I am referencing the pod below to get the latest.
pod 'Alamofire', '~> 3.0.0-beta.1'
public protocol ResponseObjectSerializable {
    init?(response: NSHTTPURLResponse, representation: AnyObject)
}

extension Request {
    public func responseObject<T: ResponseObjectSerializable>(completionHandler: (NSURLRequest, NSHTTPURLResponse?, T?, NSError?) -> Void) -> Self {
        let responseSerializer = GenericResponseSerializer<T> { request, response, data in
            let JSONResponseSerializer = Request.JSONResponseSerializer(options: .AllowFragments)
            let (JSON: AnyObject, serializationError) = JSONResponseSerializer.serializeResponse(request, response, data)

            if let response = response, JSON: AnyObject = JSON {
                return (T(response: response, representation: JSON), nil)
            } else {
                return (nil, serializationError)
            }
        }

        return response(responseSerializer: responseSerializer, completionHandler: completionHandler)
    }

}

GenericResponseSerializer has been renamed ResponseSerializer. For cases like this, make sure you check the documentation for update, or, if it hasn't been updated, check the source for the various provided serializers, as they have to use the same API as our custom ones.

@jshier Thanks for the response. I have been spending the last 40 minutes to solve this issue. Yes I no longer get the original error but now my code is not working. I tried look at the source but It was not helping me as mush as i thought it would. Can you show me a working example please?

Thanks for helping out here @jshier. 👍🏼
@ballerabdude I would suggest you drop down to the Alamofire 2.0.2 branch where we have all the sample code updated in the README and a migration guide put together. The Alamofire 3.0.0-beta.1 is a pre-release. I don't anticipate we'll cut the official 3.0 release here for at least another week, maybe two.


Android: No longer try to free the native activity when toggling the virtual keyboard

This fixes the issue described here.
In short, the line is an error, since we don't copy the clazz member/object, so it shouldn't be freed.

This PR has been added to my merge list, meaning it will be merged soon, unless someone raises any concerns.


DRILL-6806: Moving code for a HashAgg partition into separate class.

Goal
The goal of this change is to move code for managing a partition in HashAgg into a seperate class, similar to the HashPartition class in HashJoinBatch. This has several advantages:

Ease of testing: We can test basic partition logic like spilling independently of the rest of the HashAgg operator.
Reducing generated code: Currently HashAggTemplate contains the bulk of the code for HashAgg and it is a generated class. The larger the HashAggTemplate class is, the more space that the generated code for HashAgg consumes. Additionally generated code is more difficult to debug during testing, so moving code out of HashAggTemplate into a non-generated class simplifies debugging.
Memory Calculations: Moving the code for a partition into a separate class enables us to simplify and unit test the memory calculations in HashAgg.

The original design doc describing the motivation behind the change and how it relates to memory calculations is here.
Implementation HashAggPartition
In order to achieve this I introduced some interfaces:

HashAggPartition: This is the interface for a partition. It is useful to provide an interface for the partition so that we can test other pieces of operator logic with a mock partition.
HashAggMemoryCalculator: This currently encapsulates the logic for reserving and releasing space space in HashAgg's allocator at various points in the operator's execution. This interface was created so that the HashAggPartition could rely on a clear calculator interface that could be swapped out with different implementations for testing.
HashAggBatchAllocator: This interface includes the method that is used to allocate space for an outgoing batch. This interface is implemented by HashAggTemplate and is required by the HashAggPartition class. The interface is necessary because HashAggTemplate is generated at runtime, while the HashAggPartition class is not generated. So HashAggPartition cannot use methods implemented by HashAggTemplate unless they are included in an interface.
HashAggBatchHolder: The BatchHolder class is an inner class of HashAggTemplate and is generated. Since the code for HashAggPartition is not generated it can only interact with a BatchHolder through an interface. So the HashAggBatchHolder interface was added.

Then the relevant code for HashAggPartition, and HashAggMemoryCalculator calculator were moved out of HashAggTemplate and into implementation classes HashAggPartitionImpl and HashAggMemoryCalculatorImpl.
The rest of the code changes focus on replacing the usages of the following arrays with a single HashAggPartition array in HashAggTemplate:

hashTables
batchHolders
outBatchIndex
writers
spilledBatchesCount
spillFiles

Next Steps
This change is a first step to moving the code for a partition into a separate HashAggPartition class. I did not move all of it in this change in order to make review easier and to minimize the risk for bugs. A follow up change will complete moving all the code relevant to a partition into HashAggPartition. The main body of code that remains to be moved is most of the HashAggTemplate.checkGroupAndAggrValues function.
Additionally some debugging statements which printed out various partition stats were removed in this change. After the code for HashAggTemplate.checkGroupAndAggrValues is moved into HashAggPartition, I will make the stats available to print for debugging on the HashAggPartition class itself.

@Ben-Zvi Please review. This also fixes the code generation errors that I introduced in a previous commit.

@Ben-Zvi Added description of the changes

@ilooner Could we proceed with this PR?

@vdiravka still need to address @Ben-Zvi 's comments. I will try to get it done early next week.

@vdiravka Got overloaded with some deadlines for another project. I won't be able to finish this for this release.


NotImplementedError: Wrong number or type of arguments for overloaded function 'new_GaussianKernel'.

I have two set of images each having 50 images with shape (299 , 299, 3). So basically two vectors with (50, 299 , 299, 3).
Since I am interested of running a two sample MMD test for them, I reshape them to two vectors of (50, 268203) and run the following:

p_val, stat, samps, bandwidth, threshold, binary_test_result = rbf_mmd_test(knn_neighbours,
                                                                            recovered_space)

In which the rbf_mmd_test is the following:
def rbf_mmd_test(X, Y, bandwidth='median', null_samples=1000,
                 median_samples=1000, cache_size=32):
    if bandwidth == 'median':
        from sklearn.metrics.pairwise import euclidean_distances
        sub = lambda feats, n: feats[np.random.choice(
            feats.shape[0], min(feats.shape[0], n), replace=False)]
        Z = np.r_[sub(X, median_samples // 2), sub(Y, median_samples // 2)]
        D2 = euclidean_distances(Z, squared=True)
        upper = D2[np.triu_indices_from(D2, k=1)]
        kernel_width = np.median(upper, overwrite_input=True)
        bandwidth = np.sqrt(kernel_width / 2)
        # sigma = median / sqrt(2); works better, sometimes at least
        del Z, D2, upper
    else:
        kernel_width = 2 * bandwidth**2

    mmd = sg.QuadraticTimeMMD()
    mmd.set_p(sg.RealFeatures(X.T.astype(np.float64)))
    mmd.set_q(sg.RealFeatures(Y.T.astype(np.float64)))
    mmd.set_kernel(sg.GaussianKernel(cache_size, kernel_width))

    mmd.set_num_null_samples(null_samples)
    samps = mmd.sample_null()
    stat = mmd.compute_statistic()

    alpha=0.05
    threshold=mmd.compute_threshold(alpha)

    binary_test_result=mmd.perform_test(alpha)

    p_val = np.mean(stat <= samps)

    return p_val, stat, samps, bandwidth, threshold, binary_test_result

I am receiving the following error:
NotImplementedError                       Traceback (most recent call last)
<ipython-input-320-e9d27a80e98d> in <module>
      2
      3
----> 4 p_val, stat, samps, bandwidth, threshold, binary_test_result = rbf_mmd_test(a , b)

~/code/lime_analysis/utils.py in rbf_mmd_test(X, Y, bandwidth, null_samples, median_samples, cache_size)
     34     mmd.set_p(sg.RealFeatures(X.T.astype(np.float64)))
     35     mmd.set_q(sg.RealFeatures(Y.T.astype(np.float64)))
---> 36     mmd.set_kernel(sg.GaussianKernel(cache_size, kernel_width))
     37
     38     mmd.set_num_null_samples(null_samples)

NotImplementedError: Wrong number or type of arguments for overloaded function 'new_GaussianKernel'.
  Possible C/C++ prototypes are:
    shogun::CGaussianKernel::CGaussianKernel()
    shogun::CGaussianKernel::CGaussianKernel(float64_t)
    shogun::CGaussianKernel::CGaussianKernel(int32_t,float64_t)
    shogun::CGaussianKernel::CGaussianKernel(shogun::CDotFeatures *,shogun::CDotFeatures *,float64_t,int32_t)
    shogun::CGaussianKernel::CGaussianKernel(shogun::CDotFeatures *,shogun::CDotFeatures *,float64_t)

What can cause this? I am using the same rbf_mmd_test for other type of vectors and it works just fine.

@ambodi could you check that cache_size and kernel_width are of type int and float respectively?


Add second YT query for info about each video

Add a second query to the Youtube API to pull more detailed info about each video retrieved by the first query

Test


update README.md

This gets a little closer. It obviously has to update after we do a release.

Nice, thank you!


Update readme - incorrect path

$ wget https://github.com/bwssytems/ha-bridge/releases/download/{VERSION}/ha-bridge-{VERSION}.jar
should read
$ wget https://github.com/bwssytems/ha-bridge/releases/download/v{VERSION}/ha-bridge-{VERSION}.jar

I can add this to my README fix pull request.

@Lorccan - i looked through the README.MD file and only found 2 lines (74 and 80) with wget commands. Here they are - and they both already have the "v" before the version.  Can you pull the latest code and check again? Please update this issue with your test results.
I did notice the instructions were there twice so I'm going to submit a patch to remove the duplicate lines.
Create the directory and make sure that ha-bridge-4.5.0.jar is in your /home/pi/habridge directory.
pi@raspberrypi:~ $ mkdir habridge
pi@raspberrypi:~ $ cd habridge

pi@raspberrypi:~/habridge $ wget https://github.com/bwssytems/ha-bridge/releases/download/v4.5.0/ha-bridge-4.5.0.jar

Create the directory and make sure that ha-bridge-4.5.0.jar is in your /home/pi/habridge directory.
pi@raspberrypi:~ $ mkdir habridge
pi@raspberrypi:~ $ cd habridge
pi@raspberrypi:~/habridge $ wget https://github.com/bwssytems/ha-bridge/releases/download/v4.5.0/ha-bridge-4.5.0.jar

Sorry, I am pointing you in the wrong place.  It's here: https://github.com/bwssytems/ha-bridge/wiki/HA-Bridge-FAQs under the heading 'How do I update the ha-bridge jar file?'

That's an easy one - it's just a wiki page so anyone with a github account can edit it.  This is resolved.  Thanks for the suggestion and feedback.

DIdn't realise I could edit.  I'll do so in future.  Thanks.


(WIP) Integration tests using Ginkgo

@mrunalp @runcom Opened this so you can see my progress as we go along. I've managed to hack together enough to start an embedded ocid daemon (inside the test process), which listens on a socket.
Next step is to copy some code from the pod start command and make sure we can start pods based on a given test config, starting with run ocic pod run --config "$TESTDATA"/sandbox_config.json
It's really sketchy at this point but nothing seems impossible at this point. What would you think about using Go global variables for test data? (i.e. embedded in the code rather than deserialized from JSON files)

Why use global variables instead of json? You want to motivate that?
PR looks good!

On Jan 16, 2017, at 6:13 PM, Jonathan Yu<EMAIL_ADDRESS>wrote:
@mrunalp @runcom Opened this so you can see my progress as we go along. I've managed to hack together enough to start an embedded ocid daemon (inside the test process), which listens on a socket.
Next step is to copy some code from the pod start command and make sure we can start pods based on a given test config, starting with run ocic pod run --config "$TESTDATA"/sandbox_config.json
It's really sketchy at this point but nothing seems impossible at this point. What would you think about using Go global variables for test data? (i.e. embedded in the code rather than deserialized from JSON files)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@mrunalp Mostly because it'd allow us to avoid dealing with deserialization code or finding the test files. It'd eliminate a lot of boilerplate for opening files, decoding JSON, etc. Another nice benefit is that it gives strong type checking (and completion, if you use an IDE) -- a typo in the field name would result in a compile error, not a test failure later. I'm assuming that our deserialization codepaths are already well-tested by unit tests (if not, we could add those)
On the other hand, the test files might be useful as examples, though I could argue that test data and examples are distinct: while all examples can be used as tests, the converse isn't true (examples are supposed to be exemplary, and test code often isn't)
It was just kind of a side thought. I guess we should leave that to explore in another PR instead.

Sounds good😀

On Jan 16, 2017, at 8:57 PM, Jonathan Yu<EMAIL_ADDRESS>wrote:
@mrunalp Mostly because it'd allow us to avoid dealing with deserialization code or finding the test files. It'd eliminate a lot of boilerplate for opening files, decoding JSON, etc. Another nice benefit is that it gives strong type checking (and completion, if you use an IDE) -- a typo in the field name would result in a compile error, not a test failure later. I'm assuming that our deserialization codepaths are already well-tested by unit tests (if not, we could add those)
On the other hand, the test files might be useful as examples, though I could argue that test data and examples are distinct: while all examples can be used as tests, the converse isn't true (examples are supposed to be exemplary, and test code often isn't)
It was just kind of a side thought. I guess we should leave that to explore in another PR instead.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

shall we just close this in favor of https://github.com/kubernetes-incubator/cri-tools and contribute there (our tests also)?

Yeah, I think we can close this one.

Makes sense to have the tests in a common place.


convolution VS correlation convolution

I think the

tensorflow.nn.conv2d

use the correlation convolution, but i need convolution in my layer. So do you have any advice?

@lannisite110 Please provide details about what platform you are using (operating system, architecture). Also include your TensorFlow version. Also, did you compile from source or install a binary?
Make sure you also include the exact command if possible to produce the output included in your test case. If you are unclear what to include see the issue template displayed in the Github new issue template.
We ask for this in the issue submission template, because it is really difficult to help without that information. Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


CVE-2007-4559 Patch

Patching CVE-2007-4559
Hi, we are security researchers from the Advanced Research Center at Trellix. We have began a campaign to patch a widespread bug named CVE-2007-4559. CVE-2007-4559 is a 15 year old bug in the Python tarfile package. By using extract() or extractall() on a tarfile object without sanitizing input, a maliciously crafted .tar file could perform a directory path traversal attack. We found at least one unsantized extractall() in your codebase and are providing a patch for you via pull request. The patch essentially checks to see if all tarfile members will be extracted safely and throws an exception otherwise. We encourage you to use this patch or your own solution to secure against CVE-2007-4559. Further technical information about the vulnerability can be found in this blog.
If you have further questions you may contact us through this projects lead researcher Kasimir Schulz.

Many thanks for your effort to check this repository!
However, in this particular case, this is not an issue. The extractall() calls are in the test suite. They are applied to trusted tar files with well known content: the tar files are created by the test suite itself. All members in these tar files are known to have sane paths.


Doc: Add Clearcoat, Transmission, recent unlit changes.

Updates for Blender 2.91

Committed in https://developer.blender.org/rBM7266


Can anyone reproduce the mae as same as the papper

I have try it in wiki and imdb, the result is far from expection. Can anyone reproduce it?

Do you able to get age, more likely, as a combination of two values from y?

conv5->flatten(no pooling)->feats->...
get better results

The result in morph is still far from the paper,and can anyone reproduce it ?what is the mae usually?

The result in morph is still far from the paper,and can anyone reproduce it ?what is the mae usually?

I got 3.15(Internal Evaluation).

I got 3.15(Internal Evaluation).


Make task templates configurable

I started implementing a command to run tests, but the noticed there's a command in the command palette called "Run Test Task". It seems like it'd be a good idea to reuse this, since otherwise my "Run Dart tests" command appears even when there's not a Dart project open.
However, I'm struggling to find how I can actually fire this up. If I execute that command it says No task runner configured and when I click the option to configure it gives me a list of existing task runners. I figure I need to add a new item to this list that the uesr can select, which would then pre-fill their tasks.json with the correct things for executing tests.
However, I can't figure out how to contribute something to that list?

Seems these might be hard-coded?
https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/parts/tasks/common/taskTemplates.ts
Are there plans to extend this? If not, what's the best way to handle this so that the end user doesn't need to configure things manually to run tests?

Having ac contribution mechanism to the tasks.json templates is a fair request.

Is there a good way to handle this in the meantime? I'm trying to avoid adding more commands while we can't restrict them to when our extension is activating, I'm worried about polluting the command list with Dart-specific things :-/

No actually not.

I haven't tried it yet, but does #8937 help with this now? Seems the settings API may allow us to write to tasks.json?

@DanTup yes the settings API allows you to write the tasks.json. But asking extensions to do this might not be a good idea in general since it would require that all extensions are fair in terms of manipulating the tasks.json file..

@dbaeumer I thought that was the reason for these changes? I'm currently using this for launch.json changes (see #15062 for an example); is it something that should generally be avoided?

@DanTup I think it depends on. For the test running case I would say yes. What should extension do if two extensions want to execute something when the user requests running tests. We have on the next iteration plan an item to think about how to continue with the task framework and one idea is that extensions can participate in Build, Run Test, Clean, Rebuild All, without contributing a task.

@dbaeumer

We have on the next iteration plan an item to think about how to continue with the task framework and one idea is that extensions can participate in Build, Run Test, Clean, Rebuild All, without contributing a task.

This sounds great! Is there an Issue about this I can subscribe to? Are you considering better support than just "run a process and show the output" for tests (such as a structured pane that allows click to jump to tests, etc.)? I know you don't want to turn into a fat IDE, but I think there's a lot that could be done to aid/encourage testing =)

@DanTup: https://github.com/Microsoft/vscode/issues/15179. But it is work in progress right now

@dbaeumer Thanks, I'll add some comments about things I'd like to see (though I suspect they're very similar to TypeScript).

See also #9505

🙁 In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


[Interpreter] Fix: constant assignmen

Fixes #12015
Original code by @asterite in #12015 (comment).

A spec would be nice.

FYI a nice way to convey authorship is adding this line to the bottom of the commit description:
Co-authored-by: Ary <...>
-this is what GitHub itself adds, there's no other special co-authorship feature that you wouldn't be able to reproduce.
Certainly looks better than appending "by @asterite"
I am excluding the email part from the above example (don't know what it helps to protect from spam, mostly superstition at this point...), so please go to Crystal repo and do git log to search and find out the full line.

I think eventually we could change the behavior of assigning a constant in the REPL (not in interpreted mode) to evaluate the constant's value right away and return the value, but for now this change is fine :-)


Fix execution order of settings populateDefaults

refs https://github.com/TryGhost/Team/issues/221
Context
populateDefaults call in Ghost's pre-boot phase causes settings table population before migrations have a chance to set correct values first (longer explanation available here). The method is called in pre-boot only to allow "frontend" to initialize in a minimal way to serve styled 503 page.
Solution
The problem of the "frontend" boot before the "server" has a chance to initialize will be solved by flipping the dependence of the "server" on the "frontend". "server" will serve it's own 503 page until the boot is complete ("db.ready" event?) and then will notify the "frontend" to start it's initialization.
Todos

[x] introduce independent 503 page served directly from Ghost server
[x] initialize "frontend" after migrations and populateDefaults have finished execution
[ ] change boot sequence to call populateDefaults after migrations (https://github.com/TryGhost/Ghost/pull/12587)
[x] design review of maintenance page
[ ] ~allow to add site icon to maintenance page (reference: https://github.com/TryGhost/Core/commit/fb0d0f09ba58703ee54683140acd07acea470316)~ the plan has been scrapped because of too much effort needed to reach the effect

[ ] clean up the maintenance page

Related to the issue is Ghost's boot sequence rewrite - https://github.com/TryGhost/Ghost/compare/main...ErisDS:boot-spike.

An updated boot sequence has landed in main and this initializes the settings defaults after the migrations are done.


Set title / double action bug / pullRequest tests / lock issue

Fixes #21
Fixes #24
Fixes #15
Fixes #28
Summary of changes:

Added go test to Dockerfile.
Added a check that the issue_comment webhook action isn't deleted before taking action, as this was causing Derek to re-trigger when comments originally aimed at Derek deleted.
Added a check that the pull_request webhook action isn't closed before taking action, as this was causing Derek to perform the dco_check unnecessarily.
Added tests to cover some of the pullRequestHander methods.
Implemented Derek set title:, Derek lock & Derek unlock commands
Updated README.md to cover new features, marking the lock & title features as complete and adding two additional features.
Extracted a method in the open/close switch clause to assess requested state against current state and return whether the requested transition is valid.  Added a test for this.
Added checking to setTitle, close, reopen, lock & unlock so that API calls are only made when necessary hence minimizing the effect on API rate limits.
Broadened the scope of the Issue type to enable capturing details required for evaluating whether requests to the API are necessary.

Currently deployed at rgee0/Test for evaluation.
n.b. The github comparison views  aren't displaying the changes at all well.  It doesn't seem to be lining the line numbers up correctly.

@alexellis I've changed validAction to be validTransition.
I've also repointed the rgee0 [bot] on rgee0/Test at an 'always on' endpoint for testing

Thanks 👍

Derek add label: enhancement

Derek add label: enhancement

@alexellis assessState is now checkTransition.  also rebased to account for recent changes.


Factor our PASV component

The logic for the PASV listener/server adds too much complexity to the FtpConnection component and should be abstracted into it's own library. Ideally FtpConnection doesn't need to care how the data connection is established (active or passive) it just needs to be able to read/write, open/close and subscribe to events.
I'm in the process of doing this refactor however I'm wondering about a few things related to passive connections and the FTP spec.
Assumptions I'm making (are these safe to assume?):

There will be no more than one data connection per control connection.
The data connection will originate from the same host (IP address) as the control connection.
After server sends 227 Entered Passive Mode with an address/port, I can expect a connection from the expected host (within a certain time) and once that connection is received (or the timeout happens) then I can free the listener again (either close it or put it back into a pool of available listeners).
If I get an unexpected connection on a listener I can reject it.
I might have more clients than available listening ports. I can use the same listener for multiple clients as long as those clients have different host addresses because I will know which incoming connection belongs to which control connection based on IP.
If client(s) on the same host (or behind the same NAT device) open concurrent control connections, several of which send PASV command at or around the same time, then I will need to reply with different ports. If there are no more available listener ports, I assume I can leave them waiting until one becomes available. Or Is there a busy/retry response I can send?

Is there anything in the spec I'm mis-understanding or overlooking? Is this how other FTP servers are implemented?
Any thoughts @asylumfunk, @oleksiyk, @mk-pmb, @joshbuddy, @addrummond, @pye2k, @phillipgreenii, @coderbuzz?

Caveat, I don't know what your design is looking like, so I'm just purely commenting on your assumptions as you've written them. Forgive me if I'm misinterpreting your words.
RE: assuming connections will originate from certain IPs (points 2-4)

I don't think you can assume this. It's probably not very common today, but you could conceivably be a client, and instruct two remote FTP servers to connect to each other, then move bytes directly between the two servers without the client ever touching it.
Example (this is going to be hard...)

client (C) connects to server 1 (S1); and in another process connects to server 2 (S2)
C instructs S1 to open a PASV connection; S1 replies with a listening port (S1:P)
C instructs S2 to connect to S1:P via the PORT command
(at this point, S2 is connected to S1)
C instructs S1 to wait for a file via STOR
C instructs S2 to send a file via RETR (which ultimately goes over the connection established to S1:P

It's been awhile since I've done this, but I swear that there had been programs back in the day that did this.
Anyway, point being, connections don't necessarily come from the same IP as the control connection (though, I'd reckon that they almost always do)

RE: multiplexing connections (point 5)

Sounds doable, but also sounds messy and complex. Maybe you've come up with an elegant way of tracking this?

RE: busy response (point 6)

I think the standard code for server being busy is 425 Can't open data connection

instruct two remote FTP servers to connect to each other

Sounds like FXP, and that was my first thought when I read the assumption.
Beware of the FTP bounce, which Wikipedia says is why

Nearly all modern FTP server programs are configured by default to refuse PORT commands that would connect to any host but the originating host

_

There will be no more than one data connection per control connection.

I'm not sure about this, but I think it can be assumed, since afaik the responses for transfer success or failure aren't designed to indicate which of potential multiple transfers is meant. Seems like the designers considered it obvious, which would make sense if there's only one data channel ever.
This assumes that ftpd will create a fresh new control connection object on REIN. If instead the old one is kept and reset, "LOGOUT (QUIT)" in section 4.1.1 sounds like you'd indeed have multiple data connections.

If I get an unexpected connection on a listener I can reject it.

Yes. Optionally send ads. ;-)

I might have more clients than available listening ports. […] will know which incoming connection belongs to which control connection based on IP.

Yes, except for FXP. Wikipedia states that FXP is a subset of RFC 959 FTP, so it'd be nice to support it, but as said above, configs usually deny it.

If client(s) on the same host (or behind the same NAT device) open concurrent control connections, several of which send PASV command at or around the same time, then I will need to reply with different ports.

Yes, that or admit failure. 425 looks appropriate, best with a reason.

Thanks for the in-depth feedback, it's much appreciated. @pye2k: that's a very good explanation of how the FTP protocol can be used to transfer between two remote hosts directly. Now that I read your example, I've heard of this before. But as you mentioned "It's probably not very common today" and since, according to @mk-pmb's reference, "Nearly all modern FTP server[s] ... refuse PORT commands that would connect to any host but the originating host" I think I'm happy to leave FXP unsupported. Especially since there are well-documented vulnerabilities like FTP Bounce.
It never sat well with me that a third party can so easily guess a data port (especially since ports are selected sequentially) and just connect to it. I know FTP is utterly insecure anyway, but checking that the right host is connecting before sending/receiving data seems like the least we could do.
@mk-pmb: I need to look into REIN but I don't think that will be an issue, because what I meant about "no more than one data connection per control connection" is really "no more than one concurrent data connection per control connection". So I'm OK if control connections are reused, as long as data connections are closed first.

As I understand the LOGOUT part of the RFC, the main benefit of REIN is that the operator doesn't have to wait for transfer complete, but can initiate the next login and transfer while the first one is still in progress. To me that REIN design looks like it's trying to help me upload a file to a lot of servers when I only have a very limited shell with no terminal multiplexer, minimal job control, some unfamiliar shell, somehow managed to install netcat and I'm in a hurry because I don't trust my shell access to survive another 10 minutes. Or if that shell happens to be PHP racing against its per-request time limit.

I tried to make a cool demo of this scenario but my favourite FTP didn't like it:

502 REIN not implemented.

That sure is one way to avoid the simultaneous connections.

Interesting. So the use-cases seem pretty slim, but that's an interesting feature. Shows how many edge-case features there are in such an old protocol!

I've started this work on branch refactor which started with pulling components into their own modules and then went down the path of ES6 and babel and now I'm working on the PASV refactor on that branch. Any feedback as we go will be appreciated!

I've committed my initial implementation of this PASV component to the refactor branch. See this commit for the code (although keep in mind I will possibly rewrite commits on that branch).


Vors/common

** don't merge **
I'm going to merge it with --ff

This change is

Merged in master with --ff


lidar->camera conversion and 3d corners

What changes are proposed in this pull request?
(Please fill in changes proposed in this fix)
How is this patch tested? If it is not, please explain why.
(Details)
Release Notes
Is this a user-facing change that should be mentioned in the release notes?

[ ] No. You can skip the rest of this section.
[ ] Yes. Give a description of this change to be included in the release
notes for FiftyOne users.

(Details in 1-2 sentences. You can just refer to another PR with a description
if this PR is part of a larger change.)
What areas of FiftyOne does this PR affect?

[ ] App: FiftyOne application changes
[ ] Build: Build and test infrastructure changes
[ ] Core: Core fiftyone Python library changes
[ ] Documentation: FiftyOne documentation changes
[ ] Other

Hi @brimoor , these methods will be used in the 3d computer vision tutorial. When I finish the complete draft of the tutorial, we can revisit and see if it just makes sense for me to define in the context of the tutorial and not add to the code base.

I don't actually think this is necessary for my tutorial. Will just write the needed code in the jupyter notebook


Calling _portal.CreateMarketplace fails with error unless Region is specified

Using latest 'develop' and standard seed template. Calling /seed endpoint fails when creating a new Marketplace:
"Data": "Call failed with status code 400 (Bad Request): PUT https://portal.ordercloud.io/api/v1/organizations/200422f7-8adf-4fdc-bccc-08d3cfffd158"
However, if I add the code below to the "new Marketplace()" initializer:
Region = new Region()
                    {
                        AzureRegion = "westus",
                        Name = "Azure Us-West",
                        Id = "usw"
                    }

All works as expected. Is Region a required property now?

Yes, I believe it is

@crhistianramirez, it appears the Region has been added to the Marketplace class and I believe we can now close this issue.


GitHub Profile Review

GitHub Profile Link
https://GitHub.com/Fatao

I hope to learn alot from this community


Update to teku develop branch to test mainnet merge config

https://github.com/ConsenSys/teku/commit/eedecbfb66a397fd3dabe2b9375d9f62990af951

All checks passed, so should be fine for the next release. Will open a new PR when the actual release comes out though.


operator for handling queues and topics

Disclaimer: this is the wrong repo for the ticket, but the k8s operator isn't public, so nowhere to register it.
It would be great if the operator: https://www.ibm.com/support/knowledgecenter/SSFKSJ_9.2.0/com.ibm.mq.ctr.doc/ctr_using_operator.htm could handle creation/parameters of queues and topics, and not only queue-managers.
This would make MQ truly user-friendly, as end-users could be self-serviced with queues via the k8s api.

Hi @davidkarlsen .  I think you're right, that would be a great feature.  The best way to help get this prioritized is to raise an official Request for Enhancement (RFE).  The process is outlined here.  This makes the feature request "official", and means it gets properly prioritized with other work.
As you say, this repo isn't the right place, as it's for the MQ sample image, so I propose you raise the RFE and that we close this issue.

that site doesn't respond, and issues reported there are usually just ignored.

It eventually responded and here is the RFE: https://www.ibm.com/developerworks/rfe/execute?use_case=viewRfe&CR_ID=149603


Routing: Milestones and Features

Summary
The Data Hub - Routing project will provide the infrastructure to send data generated from the Data Input App. To achieve this, we will need to build the ability to send the data and to reformat the data to adhere to unique needs of each PHD.
Milestones to achieve an MVP

Fake data
Test data + Auth
MVP: Real data + Auth
Post MVP milestones
Monitoring
Easier configurability

Sending and transforming
"Once data is collected by the Data Input app, the Routing app will send it to the correct PHD in the correct format."

App can figure out where to send the data the proper jurisdiction
App can transform the data to send the fields required by the target PHD, in the required format

Authentication and security
"The app securely connects with each PHD. Along the way, data is protected."

App is able to save authentication information for each individual PHD
App is able to authenticate with different types of configurations
App has proper restrictions on what data is sent/accessible

Configuration: Pilot customers
"For our first pilot customers, we know the requirements to connect with each jurisdiction"

What are the data requirements for each PHD
What are the logistics/timeline for connecting with each of them

--
Monitoring
"As a human interacting with the app, I can get helpful updates about the status of data I expect to be sent"

Receipts back to sender
...

Configuration: scalability
"As USDS/CDC works with more PHDs, we can easily add more jurisdictions to the Routing app"

App can save settings for each unique jurisdiction
...

So we don't lose track - comment from Rick on previous issue: https://github.com/CDCgov/prime-central/issues/65#issuecomment-713009264


fix github output types

Codecov Report

Merging #63 (ef8d793) into master (1909725) will increase coverage by 7.09%.
The diff coverage is 16.66%.

@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
+ Coverage   53.49%   60.59%   +7.09%
==========================================
  Files          46       29      -17
  Lines        1172      637     -535
  Branches      243      136     -107
==========================================
- Hits          627      386     -241
+ Misses        545      251     -294

Impacted Files
Coverage Δ

...kages/bundlemon/src/main/outputs/outputs/github.ts
17.64% <0.00%> (ø)

packages/bundlemon/src/main/utils/configUtils.ts
30.30% <0.00%> (ø)

...ckages/bundlemon/src/main/utils/validationUtils.ts
27.27% <50.00%> (-2.73%)
:arrow_down:

service/src/utils/hashUtils.ts

service/src/utils/promiseUtils.ts

service/src/controllers/utils/markdownUtils.ts

service/src/routes/api/v1.ts

service/src/framework/mongo.ts

service/src/consts/commitRecords.ts

service/src/app.ts

... and 10 more

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 1909725...ef8d793. Read the comment docs.


Driver cannot be attached to shipment.

-Create new shipment.
-Select driver in the last field. (Even though it's hiding behind keyboard, it's super hard to select, I'll raise separate issue for this)
-save the object.
-go back and check the object again.
Observed: driver not attached to the object.

Yes, seems to be happening on ios

Yes, seems to be happening on ios


fix: font-family Hex issue

Preserve quotes in font values to prevent Hex conversion by clean-css.
Close #6306

Types of changes

[X ] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[ ] I have added tests to cover my changes (if not applicable, please state why)
[ ] All new and existing tests are passing.

Codecov Report

Merging #7763 into dev will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##              dev    #7763   +/-   ##
=======================================
  Coverage   68.83%   68.83%
=======================================
  Files          90       90
  Lines        3821     3821
  Branches     1034     1034
=======================================
  Hits         2630     2630
  Misses        968      968
  Partials      223      223

Flag
Coverage Δ

#unittests
68.83% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

packages/webpack/src/utils/postcss.js
84.52% <100.00%> (ø)

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 45f204c...9c2006f. Read the comment docs.

Oh, very nice! 👍
Maybe it is worth adding a small test for it? 🤔

@farnabaz Would be nice if we can also have a test for it indeed. Merged to include in 2.14 (tomorrow)


Made dialog messages fullscreen

@ellisonbg
Should be at least part of a fix to #317
something something pix something something didn't happen

Okeedoke it seems I have broken something, I'll look into it

Thanks!


nginx module doesn't log to journal

Issue description
It seems that if you use the default config with services.nginx.enable = true, it creates a log folder that doesn't get picked up by journald. Is that intentional? It seems unintuitive when everything else goes to the journal by default.

I guess this can be merged into https://github.com/NixOS/nixpkgs/issues/34378.


go2rtc live_provider missing

I have just upgraded to Frigate 0.12 - installed as HA addon.
Also upgraded the Frigate integration to 4.0.0 and this hass card to 5.2.0.
I'm trying to modify my dashboard to use go2rtc instead of jsmpeg for the Frigate cameras.
Previously (and still) working:
- type: custom:frigate-card
  cameras:
    - camera_entity: camera.front
      live_provider: frigate-jsmpeg

Not working:
- type: custom:frigate-card
  cameras:
    - camera_entity: camera.front
      live_provider: go2rtc

Error that shows is: card custom:frigate-card Error: Invalid configuration: [ "cameras[0] -> live_provider"]
Additionally, if I try to configure the card via the gui, I cannot even select "go2rtc" from the dropdown list of live_providers.
I'm sure I'm missing something obvious but can't figure it out.
Thanks in advance

I've resolved this myself.
It was down to caching in Chrome.  After multiple failed attempts to clear site data, cookies, force refresh, etc, finally fixed by following https://github.com/dermotduffy/frigate-hass-card#chrome-does-not-update-card-version-after-upgrade to Empty Cache and Hard Reload.


ngx-device-detector issue

Checklist

[x] I tested all changes and all related features with different users (student, tutor, instructor, admin)

Motivation and Context
The new update of the ngx-device-detector causes an error for the guided tour since the isDesktop call is not working (on Mac devices), see https://github.com/KoderLabs/ngx-device-detector/issues/115
I will write a test the covers this issue in a follow-up PR.
Steps for Testing

Log in to Artemis
Start Guided Tutorial is displayed in the account menu

Thanks for the fix.


Reimplement share modal

We could change the icons of the unavailable options or something, right? Also the line-height of the text looks weird

Hey, I suggest this changes making the unavailable options more evident. What do you think @saleiva ?

Too much text IMO. Also, there is missing the highlight for sharing internally within the org since it's the only action that can be done

Hey, I upload here the WIP, I'll see some things tomorrow with @noguerol (specially the get links section). Also about the header: not final, we need to ensure that this new "layout" works on all cases.

@josecruz feels weird that the most common actions are the same size than the more expert ones. It might be worth to change how they visually look like. If not, I'd go with the same organization that we had before (4 in a row, horizontally distributed).

Also, it's very weird to have the buttons so far from the other actionable items.
I'd take the search within the list so we can relate to the results. It make no sense to have the searchbox at that level since you can only search users.
We need to prepare the case that the user is not in an organization account.
We need the case where the map is private and the options are disabled.

Also, it's very weird to have the buttons so far from the other actionable items.

I made a more vertical-ish version of the get links cards giving it more whitespace (better to read) and making it close to the publish buttons:

I'd take the search within the list so we can relate to the results. It make no sense to have the searchbox at that level since you can only search users.

Yep agree, I was just testing different positions for the search. Also, regarding to the invite users list, I removed the buttons on that part to make it "auto-save" (frontend ppl gonna hate me)

We need to prepare the case that the user is not in an organization account.

We just remove one tab:

We need the case where the map is private and the options are disabled.

I think "Privacy" should be the first tab, as is the starting point of the workflow. This doesn't mean is the the default one. It will be nice to add some logic to open "share" by dafault once privacy has been setted.
Under privacy passwd needs explanation
Under privavy ill rather order the boxes from "private" to "public", as private is the default state
Ill prefer "Publish" instead of "get links"
Ill prefer "OK" instead of "Publish" for the main cta and thus no "done" button
I prefer shorter texts under "Get links" cards
Can't we have "edit" as a general option for the whole organization?

1. Org. Users

As you can see we separated this into 2 tabs for org. users.

"Publish" tab will have the same elements for links that we have now but with some UI Changes:

Also we made some changes for privacy (dropdown still WIP) but the main idea is to make actionable that tag to show a dropdown:

2. Regular users

The difference is that in this version we don't use tabs (but same header behaviour):

Ping @nobuti due he's working on related stuff on the #8915

Hey @saleiva and @nobuti I uploaded a link to invision with more cases: https://invis.io/5G4TKZHEJ#/174674656_05_Publish_-_00-00

Does the Share button remain in the UI?

@nobuti Yep

Hey @josecruz some pending stuff

We are lacking the design of the process when selecting a password protected privacy type.
We should decide what happens with the datasets so we unify. I propose we just open the privacy dropdown when clicking on the privacy tag.
We should decide what happens when clicking on the privacy tag on the dashboard for maps and  datasets. For maps at least we should open the shame share modal (without the UPDATE option) but what to do with the datasets?

Hey @saleiva:

We are lacking the design of the process when selecting a password protected privacy type.

I think this is already designed: https://invis.io/64800HMTN#/174678414_05_Publish_-00-04-_Password

We should decide what happens with the datasets so we unify. I propose we just open the privacy dropdown when clicking on the privacy tag.

We should decide what happens when clicking on the privacy tag on the dashboard for maps and datasets. For maps at least we should open the shame share modal (without the UPDATE option) but what to do with the datasets?

I would just use the dropdown solution on this case and keep the modal on the rest of the UI until we have time to replace it with the dropdown everywhere (so we can test if the dropdown solution really works)

Sounds good as an intermediate solution, but I'd try to implement the drop down there from the beginning if it's not too hard

@saleiva @josecruz I've found some issues integrating the current behaviour, mostly related with errors:

When updating, how any possible error is shown?
Right now when sharing inside the organisation, all changes are sent when the user press "Save settings", because if the user press "Escape" all the changes are gone, like the rest of modals. We can make it the same way using the "Done" button, but again, errors could happen. Not sure if we should change the label at least. For me, "Done" has an immediate result, closer to "Close". What worries me if that the "Done" button has not the same behaviour in both tabs. When publishing, it just closes the modal. When sharing, it has to wait to save the data, and, if nothing happens, then close the modal. Not sure if I'm too fussy here.

When updating, how any possible error is shown?

Working on it

Right now when sharing inside the organisation, all changes are sent when the user press "Save settings", because if the user press "Escape" all the changes are gone, like the rest of modals. We can make it the same way using the "Done" button, but again, errors could happen. Not sure if we should change the label at least. For me, "Done" has an immediate result, closer to "Close". What worries me if that the "Done" button has not the same behaviour in both tabs. When publishing, it just closes the modal. When sharing, it has to wait to save the data, and, if nothing happens, then close the modal. Not sure if I'm too fussy here.

So, the idea here is to auto-save the changes so you don't need a button for saving changes when sharing.
Anyway can we try it on staging so we can test it? TY!

Hey @josecruz, @saleiva, we have two pending things about design, could you take care of them as soon as possible pretty please? Those are:

Groups: AFAIK, we haven't taken into account this stuff. If a group has been changed its permission, how should appear in the info close to map/dataset privacy?. Also, group icon in the shared list should appear different than the organization one, right?
Organization: similar to the previous one, but with the whole organization.

Take into account at that point we don't have the info about how many users there are in the organization or within a group.
Take into

Hey @xavijam!
For the info close to map/dataset privacy I would just update the number with the total shared people (included groups and so on).
Also for groups, I would just maintain the current design that we have but changing the copy of the number of members for this (just as we have it on the user group settings):


Run apiview checks in pr to detect public api changes

You can try these changes at https://cadlplayground.z22.web.core.windows.net/prs/3283/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/prs/3283/

API change check
APIView has identified API level changes in this PR and created following API reviews.
Microsoft.Generator.CSharp


fix(package): update kronos-service to version 3.2.11

https://greenkeeper.io/

[Current coverage][cc-pull] is 76.64%

Merging [#175][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #175   diff @@
==========================================
  Files             1          1
  Lines           137        137
  Methods          24         24
  Messages          0          0
  Branches         18         18
==========================================
  Hits            105        105
  Misses           32         32
  Partials          0          0

Powered by Codecov. Last updated by [136ef86...441a344][cc-compare]
[cc-base-branch]: https://codecov.io/gh/Kronos-Integration/kronos-service-consul/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/Kronos-Integration/kronos-service-consul/compare/136ef86154ea6574f814bcf650802869f775f26a...441a344044d92971e421b3691b0eb2a645bdfb68
[cc-pull]: https://codecov.io/gh/Kronos-Integration/kronos-service-consul/pull/175?src=pr


socket-step now uses the ip-parameter + store step relies on an active location

LGTM (Looks good to me)!


Installation

Is it a valid and builded plugin? I couldn't install it in IDEA 2017

Someone else has put it in the plugin repository: https://github.com/changdaye

After adding this plugin, I get this error messqge:


Add suport for N dimensions.

It's okay and powerfull to have 3 dimensions but what if we can have N dimensions?
https://en.wikipedia.org/wiki/Hypercube

duplidacted #4


Contrast ratios should account for alpha transparency

Requested on social media:

https://c.im/@cwilcox808/110833633913519185
https://bne.social/@james/110839978330469012

Look at contrast-ratio, originally made by Lea Verou. It handles foreground and background colors with alpha. When the background color has alpha, the contrast value has "error bars;" I think what it does is calculate what the resulting background color would be if there was black behind it, what it would be with white behind it, then calculate the contrast of those vs. the foreground color. The +- number  represents those black/white extremes and the main number is the middle point between them.

Example: https://www.siegemedia.com/contrast-ratio#white-on-hsl(0deg 0%25 0%25 %2F 0.1)
Source: https://github.com/siege-media/contrast-ratio

Similarly, CCAe does alpha (but only for the foreground, since if you start doing it for background as well, you have to always make an assumption (generally, white) for what's behind that particular colour https://github.com/ThePacielloGroup/CCAe

Because #106 will handle background colors with alpha, this will just be limited to addressing alpha in the foreground color.
In essence, contrast is a measure of the difference between two displayed colors. Foreground colors with alpha are not displayed as transparent, but rather as a third color mixed with the background color. Ideally, we would want to get the contrast between the background and the mixed color, which requires us to compute a mixed color.
In practice, that mixed color is computed at display time, based on the display's settings. If we calculate the mixed color in the space that the other colors are defined in, we're not doing a similar calculation. This can lead to a range of results, even between monitors on the same computer.
Because color mixing for monitors is done in an RGB space, we'll do all the mixing in srgb. This matches the current behavior of both CCAe and contrast-ratio.
Despite the fuzziness that comes into play when alpha is involved, it is useful for designers to see an approximation of what the mixed color will be and what its contrast is, so that they can see approximately what level of transparency still passes.

We may just want to mark this as approximate in some way?

@mirisuzanne @stacyk
Here are the warning-related Figma file links.
Note: I don't know why Figma is suddenly displaying green lines around various bits of the mockups, and I haven't found the way to turn that view off yet.
Out of Gamut + Format Groups
https://www.figma.com/file/JtQTGIpwoaOcqcWfsoeOk1/Browser-bugs-%26-other-warnings?type=design&node-id=83%3A11&mode=design&t=4CwzTDrU2fakSiQ4-1
Incorrect implementation + Expanded Color Issue Info
https://www.figma.com/file/JtQTGIpwoaOcqcWfsoeOk1/Browser-bugs-%26-other-warnings?type=design&node-id=12%3A138&mode=design&t=4CwzTDrU2fakSiQ4-1
Background Alpha Alert + Collapsed Color Issues
https://www.figma.com/file/JtQTGIpwoaOcqcWfsoeOk1/Browser-bugs-%26-other-warnings?type=design&node-id=12%3A243&mode=design&t=4CwzTDrU2fakSiQ4-1
Foreground Alpha Alert
https://www.figma.com/file/JtQTGIpwoaOcqcWfsoeOk1/Browser-bugs-%26-other-warnings?type=design&node-id=12%3A348&mode=design&t=4CwzTDrU2fakSiQ4-1
Mobile Alerts + Color Issue Info
https://www.figma.com/file/JtQTGIpwoaOcqcWfsoeOk1/Browser-bugs-%26-other-warnings?type=design&node-id=83%3A10&mode=design&t=4CwzTDrU2fakSiQ4-1
Mobile Alerts + Color Issue Info 2
https://www.figma.com/file/JtQTGIpwoaOcqcWfsoeOk1/Browser-bugs-%26-other-warnings?type=design&node-id=83%3A11&mode=design&t=4CwzTDrU2fakSiQ4-1

@SondraE  I think the first and last link are the same, no? Is that the intention?

https://www.figma.com/file/JtQTGIpwoaOcqcWfsoeOk1/Browser-bugs-%26-other-warnings?type=design&node-id=5-146&mode=design&t=fz0z6iKhkUnjtRs7-4 this might be the link for the first one

This is the same set of issues and links provided in the other story, right? I guess we don't need to break out individual stories - but it might be helpful to clarify which designs are related to which issue :)


Missing endpoints in localtargets.* A records

Steps to reproduce

Deploy two cross-communicating ohmyglb setup locally
$ make deploy-full-local-setup
Check generated localtargets.* dnsendpoint conf

$ kubectl -n test-gslb get dnsendpoints test-gslb -o yaml
...
spec:
  endpoints:
  - dnsName: localtargets.app3.cloud.example.com
    recordTTL: 30
    recordType: A
    targets:
    - <IP_ADDRESS>
    - <IP_ADDRESS>
    - <IP_ADDRESS>
...

Check if coredns returns matching A records

dig +short @localhost localtargets.app3.cloud.example.com
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>

This is expected result. After some time localtargets.* can 'lose' one of the records in the following way:

localtargets.* dnsendpoint conf is always consistent

$ kubectl -n test-gslb get dnsendpoints test-gslb -o yaml
...
spec:
  endpoints:
  - dnsName: localtargets.app3.cloud.example.com
    recordTTL: 30
    recordType: A
    targets:
    - <IP_ADDRESS>
    - <IP_ADDRESS>
    - <IP_ADDRESS>
...

Meanwhile actual DNS response might lose one of the A records

dig +short @localhost localtargets.app3.cloud.example.com
<IP_ADDRESS>
<IP_ADDRESS>

Issue is not really deterministic in its behaviour .  Meanwhile we faced it several times over multiple deployments
In case of 2 cluster setup only single cluster is affected effectively making exposed through coredns only 5 out of 6 k8s worker.
DNSEndpoint CR generation looks always correct so the problem is somewhere in etcd coredns backend area.
make debug-test-etcd can help in debugging this issue runtime.

Example of degraded localtargets from within etcd:
/ # etcdctl get '' --from-key
/skydns/com/example/cloud/failover/16fccd7b
{"host":"<IP_ADDRESS>","ttl":30,"targetstrip":1}
/skydns/com/example/cloud/failover/24f6061f
{"host":"<IP_ADDRESS>","ttl":30,"targetstrip":1}
/skydns/com/example/cloud/failover/6c871674
{"host":"<IP_ADDRESS>","ttl":30,"targetstrip":1}
/skydns/com/example/cloud/failover/localtargets/13ee1210
{"host":"<IP_ADDRESS>","ttl":30,"targetstrip":1}
/skydns/com/example/cloud/failover/localtargets/2485c7ba
{"host":"<IP_ADDRESS>","ttl":30,"targetstrip":1}
/skydns/com/example/cloud/failover/localtargets/315f5958
{"host":"<IP_ADDRESS>","ttl":30,"targetstrip":1}

Notice the unexpected duplication of <IP_ADDRESS>
At the same time related DNSEndpoint is fully ok:
k -n test-gslb get dnsendpoints -o yaml
....
    - dnsName: failover.cloud.example.com
      recordTTL: 30
      recordType: A
      targets:
      - <IP_ADDRESS>
      - <IP_ADDRESS>
      - <IP_ADDRESS>

So the problem seems to be fully isolated  to upstream components of external-dns+etcd ... coredns reads whatever it has in etcd backend so it is behaving correctly.
Digging into external-dns -> etcd propagation code ...

Made $ etcdctl del '' --from-key effectively wiping out local etcd database
Then it was gslb reconcile
External-dns logs during the even :
time="2020-03-19T21:32:52Z" level=info msg="Add/set key /skydns/com/example/cloud/failover/4088e0a8 to Host=<IP_ADDRESS>, Text=, TTL=30"
time="2020-03-19T21:32:52Z" level=info msg="Add/set key /skydns/com/example/cloud/failover/461535c9 to Host=<IP_ADDRESS>, Text=, TTL=30"
time="2020-03-19T21:32:52Z" level=info msg="Add/set key /skydns/com/example/cloud/failover/68ec81be to Host=<IP_ADDRESS>, Text=, TTL=30"
time="2020-03-19T21:32:52Z" level=info msg="Add/set key /skydns/com/example/cloud/failover/localtargets/57c491cc to Host=<IP_ADDRESS>, Text=, TTL=30"
time="2020-03-19T21:32:52Z" level=info msg="Add/set key /skydns/com/example/cloud/failover/localtargets/5afed140 to Host=<IP_ADDRESS>, Text=, TTL=30"
time="2020-03-19T21:32:52Z" level=info msg="Add/set key /skydns/com/example/cloud/failover/localtargets/6a0a107c to Host=<IP_ADDRESS>, Text=, TTL=30"
time="2020-03-19T21:33:52Z" level=info msg="Add/set key /skydns/com/example/cloud/failover/68ec81be to Host=<IP_ADDRESS>, Text=, TTL=30"
time="2020-03-19T21:33:52Z" level=info msg="Add/set key /skydns/com/example/cloud/failover/68ec81be to Host=<IP_ADDRESS>, Text=, TTL=30"
time="2020-03-19T21:33:52Z" level=info msg="Add/set key /skydns/com/example/cloud/failover/68ec81be to Host=<IP_ADDRESS>, Text=, TTL=30"
time="2020-03-19T21:33:52Z" level=info msg="Add/set key /skydns/com/example/cloud/failover/localtargets/6a0a107c to Host=<IP_ADDRESS>, Text=, TTL=30"
time="2020-03-19T21:33:52Z" level=info msg="Add/set key /skydns/com/example/cloud/failover/localtargets/6a0a107c to Host=<IP_ADDRESS>, Text=, TTL=30"
time="2020-03-19T21:33:52Z" level=info msg="Add/set key /skydns/com/example/cloud/failover/localtargets/6a0a107c to Host=<IP_ADDRESS>, Text=, TTL=30"

So the first chunk of the log looks totally correct - each entry at different path and different hash at the end. Consequently we observe weird output of different values ending up at the same hash.
Surprisingly after the wipe out and reconcile it did fix itself:
# etcdctl get '' --from-key
/skydns/com/example/cloud/failover/4088e0a8
{"host":"<IP_ADDRESS>","ttl":30,"targetstrip":1}
/skydns/com/example/cloud/failover/461535c9
{"host":"<IP_ADDRESS>","ttl":30,"targetstrip":1}
/skydns/com/example/cloud/failover/68ec81be
{"host":"<IP_ADDRESS>","ttl":30,"targetstrip":1}
/skydns/com/example/cloud/failover/localtargets/57c491cc
{"host":"<IP_ADDRESS>","ttl":30,"targetstrip":1}
/skydns/com/example/cloud/failover/localtargets/5afed140
{"host":"<IP_ADDRESS>","ttl":30,"targetstrip":1}
/skydns/com/example/cloud/failover/localtargets/6a0a107c
{"host":"<IP_ADDRESS>","ttl":30,"targetstrip":1}

No dups.
Something really weird happens here.


now dev with inspect faild

I want to debug the function locally by vscode.
but with the NODE_OPTIONS=--inspect env seems run faild

NODE_OPTIONS=--inspect npx now dev
Debugger listening on ws://<IP_ADDRESS>:9229/ca9bd141-e033-4cca-9926-930fb07b80e0
For help, see: https://nodejs.org/en/docs/inspector
> Now CLI 16.7.3 dev (beta) — https://zeit.co/feedback
> Creating initial build
Starting inspector on <IP_ADDRESS>:9229 failed: address already in use
> Error! Failed to update builders: Command failed with exit code 12: /Users/fox/.nvm/versions/node/v12.13.0/bin/node /Users/fox/Library/Caches/co.zeit.now/dev/builders/yarn add --exact --no-lockfile --non-interactive @now/next@latest @now/build-utils@latest

lsof -i:9229

is empty

success without inspect env
npx now dev
> Now CLI 16.7.3 dev (beta) — https://zeit.co/feedback
> Creating initial build
> Using external babel configuration
> Location: "/Volumes/data/workspace/github/foxmn/web-tools/packages/show-user-agent/.babelrc"
[ info ]  bundled successfully, waiting for typecheck results...
> Success! Build completed
> Ready! Available at http://localhost:3000
[ ready ] compiled successfully
[ event ] build page: /next/dist/pages/_error
[ wait ]  compiling ...
[ info ]  bundled successfully, waiting for typecheck results...
[ ready ] compiled successfully
[ event ] build page: /
[ wait ]  compiling ...
[ info ]  bundled successfully, waiting for typecheck results...
[ ready ] compiled successfully

This is not supported.
See related issues #2864 and #2077.


Modal not wide enough, and has too much padding

Description of the bug
The modal component is not wide enough to fit the content we would like to have in it. It also has too much padding. See this phtoto for reference:

Also, found this in the code, and it seems like the max-width and padding issue is a result of this:

Steps To Reproduce
See photos above 😄
Additional Information
No response

Thanks for bringing this to our attention!
If you are using entirely custom content like this we advice just not use Modal.Content and just use it directly inside Modal

We'll move the top/bottom paddings away from Modal to a children selector instead.
We had some discussion on how to handle width as we want the modal fill up the full screen under a certain width, but also have this customisable. We'll have a look at how we want to best solve this as we would like to solve this using CSS, but haven't come up with a good way yet 😅

Thanks for bringing this to our attention!
If you are using entirely custom content like this we advice just not use Modal.Content and just use it directly inside Modal

We'll move the top/bottom paddings away from Modal to a children selector instead.
We had some discussion on how to handle width as we want the modal fill up the full screen under a certain width, but also have this customisable. We'll have a look at how we want to best solve this as we would like to solve this using CSS, but haven't come up with a good way yet 😅

First point also relates to a bug, defined in #1116


Add BUILD_PRESET alias as NITRO_PRESET

IMO it makes sense to have it as an alias of NITRO_PRESET in term of DX.
Also noted that Analog.js also have this alias.

Might be worth moving it to nuxt/cli repo

https://github.com/unjs/nitro/issues/1860


autoscaler should use the service network to communcate with guest kube-apiserver

related to #83 the autoscaler kubeconfig should use the service network to talk to the guest kube-apiserver, not the external API endpoint
@elmiko

closed by https://github.com/openshift/hypershift/pull/246


feat(imports) Support imported function

See README.md or examples to see the API (come with this PR).
The API is based on cgo.

bors r+

bors r+


TypeError: Cannot read property 'fn' of undefined react

I am using the jquery like this
import OwlCarousel from 'react-owl-carousel';
import $ from 'jquery';

window.jQuery = $;
window.$ = $;
global.jQuery = $;

but I am still facing this issue
TypeError: Cannot read property 'fn' of undefined react
kindly please help me on this, please...?
jquery version is 3.4.0

i get this issue too, only solution that worked for me was to inject query in your 'index.html' like so
<script
  src="https://code.jquery.com/jquery-3.4.1.min.js" </script>

I inject jquery in index.html file but still have same issue.

I inject jquery in index.html file but still have same issue.

hi, did you resolved this issues.

I inject jquery in index.html file but still have same issue.

hi, did you resolved this issues.

I have the same error even after injecting inside. Does anybody know how to overcome this problem?

do, npm i jquery, worked for me

i get this issue too, only solution that worked for me was to inject query in your 'index.html' like so
<script
  src="https://code.jquery.com/jquery-3.4.1.min.js" </script

i get this issue too, only solution that worked for me was to inject query in your 'index.html' like so
<script
  src="https://code.jquery.com/jquery-3.4.1.min.js" </script>

same solution worked for me

Make sure to inject the script jquery tag in the head portion.

still not resolve...

include this is plugin array path : Project\node_modules\react-scripts\config\webpack.config.js
new webpack.ProvidePlugin({         $: 'jquery',         jQuery: 'jquery',         'window.jQuery': 'jquery',       }),

inject this query into your "index.html"

i get this issue too, only solution that worked for me was to inject query in your 'index.html' like so
<script
  src="https://code.jquery.com/jquery-3.4.1.min.js" </script>

same solution worked for me

thanks work for me

this is the correct script


Swipe to reveal cells

Hey,
Am I correct in assuming that swipe to delete won't work with this anymore since the tableView and the cells are rotated?
Thanks

Thank you for your reporting!
I've fixed at #5
But you have to pay attention swipe direction is left to right.


No Force sensor dashboard

Hi, I have just downloaded this package to use it for FT45_M4 iiT sensor, but I noticed differences in FT_CAN_messages.slx and your readme file.
In the .slx file:

the general management window doesn't have "set can datarate" tab.
FT management window and send FT Msg doesn't exist!

Hi @haniehes,
Thanks for raising this issue! I removed the dashboard elements you mentioned because of internal usage purposes, but you rightly point out that it does not match the readme, so I'll add all that stuff back.
While I do that, you can roll back to a previous version of the model by checking out the commit https://github.com/icub-tech-iit/ft-sensors-simulink/commit/11fb0835bb68722719b334ba9e1abafe53dbd756 with:
git checkout 11fb083

That version should match the readme.

Hi @haniehes , I created a PR to roll back the changes https://github.com/icub-tech-iit/ft-sensors-simulink/pull/9


api: add list command

Follows https://github.com/iterative/dvc.org/pull/967
Relates:

Pull#2346
Issue#2509

❗ Please read the guidelines in the Contributing to the Documentation list if you make any substantial changes to the documentation or JS engine.
🐛 Please make sure to mention Fix #issue (if applicable) in the description of the PR. This causes GitHub to close it automatically when the PR is merged.
Please chose to allow us to edit your branch when creating the PR.
Thank you for the contribution - we'll try to review it as soon as possible. 🙏

[ ] We should also update other refs to mention/link to this one. import and get are the obvious ones, any others? 🤔

I rewrote the fist half @shcheklein. Please check when you can and I'll address your feedback at that time along with a review of the examples. Hoping we also get answers to the few questions I added above by then. Thanks!

p.s. Thanks again for the initial version @JIoJIaJIu! Keep in mind that even if I rewrite significant parts of it, it would be much harder without having your initial draft/submission. 👍

@jorgeorpinel @JIoJIaJIu looks great!
@jorgeorpinel should we also update the data registry to include the list?

takes too long ... merging this, please make other fixes as a regular update


Extra quotes get added when trying to add a closing quote in json file

In a json file

add double quotes, closing quotes get added
delete the closing quotes
add the closing quote yourself

Observe that another double quotes get added.
Repros without any extensions as well

Same problem as #26637