profile
viewpoint

agirton/gloom 17

A dark and gloomy pastel color syntax theme for Visual Studio Code

agirton/adventofcode 3

Trying out Rust by doing daily Advent of Code

agirton/Hook.js 1

Pull to refresh. For the web.

agirton/apex-ui 0

Apex monochrome Atom theme UI.

agirton/atom-autocomplete-modules 0

Autocomplete for require/import statements

agirton/atom-itunes 0

Displays the currently playing song in iTunes in Atom's status bar

agirton/babel-plugin-dynamic-import-webpack 0

Babel plugin to transpile import() to require.ensure, for Webpack

agirton/downshift 0

🏎 Primitives to build simple, flexible, WAI-ARIA compliant enhanced input React components

agirton/elm-architecture-tutorial 0

How to create modular Elm code that scales nicely with your app

issue commentNetflix/vmaf

Different VMAF score coming for different runs on same video

@slhck I would request to let the issue be there for few more days until we complete our investigation from our side. i will try to provide as much info as possible

tarun-discovery

comment created time in 25 minutes

issue commentNetflix/vmaf

Different VMAF score coming for different runs on same video

@li-zhi I suggest closing this issue, as it has nothing to do with VMAF itself, but ffmpeg and network transmission issues.

tarun-discovery

comment created time in 26 minutes

issue commentNetflix/vmaf

Different VMAF score coming for different runs on same video

This indicates a transmission issue. Without the requested additional information or a reproducible example, I am afraid no one else will be able to help you further though.

tarun-discovery

comment created time in 27 minutes

issue commentNetflix/conductor

Deploying Conductor with HA and DR in mind

@TwoUnderscorez thanks for sharing info regarding dynomite. But the activity in dynomite seems very low (just 3 PRs merged during 2020). And also, I believe netflix is no longer using dynomite as the backend for conductor ... at least not for the queues.

TwoUnderscorez

comment created time in 2 hours

issue commentJedWatson/react-select

The react-select works incorrectly when I remove the react-modal component from DOM

Clicking on the drop-down icon or navigating with the keyboard after closing the modal works fine.

Clicking the component as is (not the icon) does nothing. This is usually expected behavior if the <input> is already focused. The problem is that the onFocus handler on the <input> itself is not being triggered by react-modal returning the focus (i.e. inputElement.focus()).

I remember that this was a React issue, but I can't remember where I saw it mentioned.

fmva

comment created time in 2 hours

issue commentJedWatson/react-select

state value when no options are found

@cseas You could easily make a wrapper which would add this functionality.

Because the component adds a NoOptionsMessage component if there are no options found, we can create a custom component to set a state value if this component renders:

const NoOptionsMessage = ({ selectProps: { setNoOptions }, ...props }) => {
  React.useEffect(() => {
    setNoOptions(true);

    return () => setNoOptions(false);
  }, [setNoOptions]);

  return <components.NoOptionsMessage {...props} />;
}

const customControlStyle = (provided, state) => ({
  ...provided,
  color: state.selectProps.noOptions ? 'red' : null
});

const NoOptionsSelect = () => {
  const [noOptions, setNoOptions] = React.useState(false);

  return <Select
    setNoOptions={setNoOptions}
    noOptions={noOptions}
    components={{ NoOptionsMessage }}
    styles={{ control: customControlStyle }}
  />
}

Only caveat is that the state is being reset to false if the focus is lost / the menu is closed.

cseas

comment created time in 2 hours

push eventNetflix/sureal

Zhi Li

commit sha 50b785f0a881e643374f345a8a05d08d97d21f53

Fix __main__.

view details

push time in 4 hours

issue openedJedWatson/react-select

state value when no options are found

With isSearchable enabled, is there a state value available for when no options are found in the react-select search?

Use case: I'm trying to add some custom css based on whether options are found in the search or not. To be precise, I want to make some icons in a CustomControl component turn red when no options are found for the given search.

I can see from DefinitelyTyped that the only values available for state under control are isDisabled, isFocused, menuIsOpen.

Would it be possible to add another value called state.noOptions?

created time in 4 hours

pull request commentNetflix/conductor

Added missing params to Go client Ack

Codecov Report

Merging #1942 into dev will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##                dev    #1942   +/-   ##
=========================================
  Coverage     65.57%   65.57%           
  Complexity     3928     3928           
=========================================
  Files           301      301           
  Lines         18527    18527           
  Branches       1685     1685           
=========================================
  Hits          12149    12149           
  Misses         5537     5537           
  Partials        841      841           

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 b51ce0a...2353871. Read the comment docs.

djcass44

comment created time in 4 hours

issue commentNetflix/conductor

unable to execute function that implements task using go client

Looks like I missed a bit when I last updated the client. @apanicker-nflx can you take a look at #1942

archthegit

comment created time in 4 hours

PR opened Netflix/conductor

Added missing params to Go client Ack

Go workers aren't able to correctly Ack that they've accepted a request due to missing parameters.

See #1923

+7 -3

0 comment

2 changed files

pr created time in 4 hours

pull request commentNetflix/lemur

Delegated CNAME DNS validation for ACME

Coverage Status

Coverage increased (+0.007%) to 59.383% when pulling af348b10128c1c1affa1c7a886ca7d659a8880e0 on cname_01 into ff837217206f4b9070bba97d3a8b77d1d8b37365 on master.

csine-nflx

comment created time in 4 hours

push eventNetflix/lemur

Emmanuel Garette

commit sha 9374adaa4632ed4e477f875cf036f8bb364d7707

do not create db_upgrade.log during migrations

view details

Hossein Shafagh

commit sha adf8f377188b00381642f13509aa8ffd901dd8cf

Merge branch 'master' into log_update

view details

Hossein Shafagh

commit sha 2b876f22a5b6908639b2aa9c4eab05588eb4534e

Merge branch 'master' into log_update

view details

Hossein Shafagh

commit sha 30915d30bec3a33c8eef6b5358ca47812b989ff2

Merge branch 'master' into log_update

view details

sayali

commit sha 2c22d42a57d7ac219fbacd6480d63cbf67af0b63

Modify description during reissue Include the certificate ID being reissued and mention that this is created by Lemur as part of reissue

view details

Hossein Shafagh

commit sha 2c1e7b19a225ad71e4a6802fd293469c5601d3d7

10x 10s delay might be too long for the load balancer request

view details

Hossein Shafagh

commit sha d233490c8aeb6996bb8fc936c9c86f3614fce142

simple retry

view details

Hossein Shafagh

commit sha 75bc3a5b20d6d0efcf0da236daf301fe29b12bdb

refactoring and adding retry

view details

Hossein Shafagh

commit sha 6891077501a9ca418b6747f0b13d57bc08a79759

readability

view details

Hossein Shafagh

commit sha 7e573d6d517d7f6189ff005ed072f321627b7b6f

fixing typo

view details

Hossein Shafagh

commit sha 9957120a7fd2c0befef39f166c6292f5b8f83e86

adding missing import

view details

Hossein Shafagh

commit sha 0e02abbb3791cbd844cd916812a99896c6823fce

Entrust just looks into CSR for RSA/EC key type

view details

Hossein Shafagh

commit sha f6554a9a1e79b9a3e688ba59438708b559efd835

typo, fixing abstract class complaints

view details

Emmanuel Garette

commit sha d7478a5c5cb00dc619e7f8bc7afc2780a1cb3e1a

use an alternative logger for the upgrade

view details

sayali

commit sha 392725ff309609c7695de5860a409a9f794415d0

Add description check in reissue unit test

view details

Hossein Shafagh

commit sha ab47db4cd4c1bc25ffc5cef0e3a0ed61b21a758d

Merge branch 'master' into log_update

view details

Hossein Shafagh

commit sha 4fffb8ba5b916d250e59f9226e327a9855e6fc88

Merge branch 'master' into log_update

view details

Hossein Shafagh

commit sha cb4f8144785390810cd927a4cebc1015e4cd9b32

Merge branch 'master' into issuer-retry

view details

Hossein Shafagh

commit sha 97f66276ec1789378f75a0e8d8ea9027f87d8674

Merge branch 'master' into log_update

view details

Hossein Shafagh

commit sha 7a982a731a3aaad0cf8dc0a166ed411f61402a73

Merge branch 'master' into ecc_changes

view details

push time in 4 hours

PR opened Netflix/lemur

Delegated CNAME domain validation for ACME validation

The ACME protocol supports DNS delegation for validation records via CNAME, due to the nature of DNS,. Rather than creating TXT records in the target domain, CNAME records can be used to delegate them to to other domains. For example, lets say you want to request a certificate for foobar.com. Normally with DNS-01, a TXT record would need to be created at _acme-challenge.foobar.com. With CNAME delegation, you could instead create a CNAME at _acme-challenge.foobar.com that points to foobar.com.validationdomain.yourdomain.com, which is a CNAME containing the appropriate token. This capability is enabled by setting _ACME_ENABLE_DELEGATED_CNAME = True in your config file. When enabled, Lemur will follow CNAMEs and attempt to create TXT records in the target domains.

The advantages of a CNAME

+69 -23

0 comment

3 changed files

pr created time in 4 hours

push eventNetflix/lemur

csine-nflx

commit sha 33a006bbeba013e4e3f99c5f257458d9199c942a

fixing delete with optional validation

view details

push time in 5 hours

issue commentNetflix/conductor

Deploying Conductor with HA and DR in mind

@marosmars thanks for sharing! I'll definitely check out patroni and RDBMS persistence. By the way, according to #1763 dynomite is still being maintained.

TwoUnderscorez

comment created time in 5 hours

issue closedJedWatson/react-select

yarn info: The platform "linux" is incompatible with this module

npm doesn't work for trying to run the docs website locally as discussed in #4232 However, even when trying to run it with yarn, an info message appears indicating an incompatibility with linux.

I'm wondering whether this project is incompatible with linux or npm? Would npm have worked if I was on Windows? What's the "recommended" development environment?

Steps to reproduce

git clone https://github.com/JedWatson/react-select
cd react-select/docs
yarn

Screenshot

Selection_090

Operating system: Ubuntu 20.04.1 LTS

closed time in 6 hours

cseas

issue commentNetflix/vmaf

Different VMAF score coming for different runs on same video

@slhck finally able to replicate and noticed following error for scenarios where my vmaf error was different. Note: I was not able to reproduce it in standalone machine

\rframe=65451 fps= 33 q=-0.0 size=N/A time=00:43:38.04 bitrate=N/A speed=1.31x \rframe=65460 fps= 33 q=-0.0 size=N/A time=00:43:38.40 bitrate=N/A speed=1.31x \rframe=65469 fps= 33 q=-0.0 size=N/A time=00:43:38.76 bitrate=N/A speed=1.31x \rframe=65479 fps= 33 q=-0.0 size=N/A time=00:43:39.16 bitrate=N/A speed=1.31x \rframe=65487 fps= 33 q=-0.0 size=N/A time=00:43:39.48 bitrate=N/A speed=1.31x \rframe=65497 fps= 33 q=-0.0 size=N/A time=00:43:39.88 bitrate=N/A speed=1.31x \r[mxf @ 0x3e57f40] Packet corrupt (stream = 0, dts = 65502).\n[mpeg2video @ 0x3f73d40] ac-tex damaged at 35 24\n[mpeg2video @ 0x3f73d40] Warning MVs not available\n[mpeg2video @ 0x3f73d40] concealing 5280 DC, 5280 AC, 5280 MV errors in P frame\nframe=65503 fps= 33 q=-0.0 size=N/A time=00:43:40.12 bitrate=N/A speed=1.31x \rhttp://packaged-vod-assets.s3.amazonaws.com/b856b6ff-5f57-4f7a-b62b-2a851cb86eaf/MasterMezzanine.mxf?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=ASIAYOHIUN2NG6JFQARV%2F20201028%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=<#######<deletedt to safegaurd####>: corrupt decoded frame in stream 0\nframe=65549 fps= 33 q=-0.0 size=N/A time=00:43:41.96 bitrate=N/A speed=1.31x \rframe=65567 fps= 33 q=-0.0 size=N/A time=00:43:42.68 bitrate=N/A speed=1.31x \rframe=65575 fps= 33 q=-0.0 size=N/A time=00:43:43.00 bitrat

tarun-discovery

comment created time in 6 hours

issue commentNetflix/vmaf

building static library - docs outdated

Is there a static only build please?

tessus

comment created time in 6 hours

create barnchNetflix/lemur

branch : cname_01

created branch time in 6 hours

issue closedNetflix/eureka

High Severity Vulnerabilities due to use of upgrade jettison-1.3.7.jar

The jar jettison-1.3.7.jar is used by eureka-client. It's reported to have Vulnerabilities: CVE-2020-10663 and CVE-2020-7712.

The proposed solution is to upgrade jettison to version 1.4.x.

closed time in 8 hours

aviramle

issue commentNetflix/eureka

High Severity Vulnerabilities due to use of upgrade jettison-1.3.7.jar

Thanks, this has been bumped.

aviramle

comment created time in 8 hours

issue closedNetflix/eureka

There is a vulnerability in xerces 2.4.0 ,upgrade recommended

https://github.com/Netflix/eureka/blob/821beb4b2e0204544c70017e1e883fcf4aa02c9e/eureka-server/build.gradle#L11

CVE-2012-0881 CVE-2013-4002 CVE-2009-2625

Recommended upgrade version:2.12.0.SP02

closed time in 8 hours

QiAnXinCodeSafe

issue commentNetflix/eureka

There is a vulnerability in xerces 2.4.0 ,upgrade recommended

Thanks, this has been bumped.

QiAnXinCodeSafe

comment created time in 8 hours

delete branch Netflix/eureka

delete branch : troshko/deps

delete time in 8 hours

push eventNetflix/eureka

troshko111

commit sha f92ee228f9a7b2e1a76f8d75de0a4708353a79f8

Update vulnerable deps #1357 #1351 (#1358)

view details

push time in 8 hours

PR merged Netflix/eureka

Reviewers
Update vulnerable deps #1357 #1351
+3 -3

0 comment

2 changed files

troshko111

pr closed time in 8 hours

pull request commentNetflix/conductor

Fix WorkflowRepairService to evaluate the possible task states for Si…

Codecov Report

Merging #1941 into dev will decrease coverage by 0.07%. The diff coverage is 85.71%.

Impacted file tree graph

@@             Coverage Diff              @@
##                dev    #1941      +/-   ##
============================================
- Coverage     65.57%   65.49%   -0.08%     
- Complexity     3928     3934       +6     
============================================
  Files           301      301              
  Lines         18527    18529       +2     
  Branches       1685     1686       +1     
============================================
- Hits          12149    12136      -13     
- Misses         5537     5553      +16     
+ Partials        841      840       -1     
Impacted Files Coverage Δ Complexity Δ
...onductor/core/execution/WorkflowRepairService.java 70.27% <85.71%> (+1.69%) 13.00 <9.00> (+5.00)
...ductor/dao/cassandra/CassandraEventHandlerDAO.java 65.78% <0.00%> (-10.53%) 15.00% <0.00%> (ø%)
.../conductor/dao/cassandra/CassandraMetadataDAO.java 53.88% <0.00%> (-4.15%) 24.00% <0.00%> (ø%)
...e/execution/tasks/SystemTaskWorkerCoordinator.java 83.33% <0.00%> (+2.08%) 18.00% <0.00%> (+1.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 b51ce0a...4fa0b49. Read the comment docs.

kishorebanala

comment created time in 8 hours

push eventNetflix/atlas

brharrington

commit sha a8bd7c21d1dfcc9fe227a75f337803620c17b1db

ensure image writer is disposed (#1216) This should cleanup imageio* files in the temp directory for image caching. Still need to investigate why they are being created in the first place since we disable the on disk caching (`ImageIO.setUseCache(false)`).

view details

push time in 9 hours

PR merged Netflix/atlas

ensure image writer is disposed

This should cleanup imageio* files in the temp directory for image caching. Still need to investigate why they are being created in the first place since we disable the on disk caching (ImageIO.setUseCache(false)).

+29 -21

0 comment

1 changed file

brharrington

pr closed time in 9 hours

more