Linking error: undefined reference to symbol 'pthread_create@@GLIBC_2.2.5'

[100%] Linking CXX executable VelodyneCapture
/usr/bin/ld: CMakeFiles/VelodyneCapture.dir/main.cpp.o: undefined reference to symbol 'pthread_create@@GLIBC_2.2.5'
//lib/x86_64-linux-gnu/libpthread.so.0: error adding symbols: DSO missing from command line

That's a problem caused by pthread library

$ readelf -s /lib/x86_64-linux-gnu/libpthread.so.0 | grep pthread_create
   206:<PHONE_NUMBER>007990  3720 FUNC    GLOBAL DEFAULT   14 pthread_create@@GLIBC_2.2.5
   119:<PHONE_NUMBER>007990  3720 FUNC    LOCAL  DEFAULT   14 __pthread_create_2_1
   495:<PHONE_NUMBER>007990  3720 FUNC    GLOBAL DEFAULT   14 pthread_create@@GLIBC_2.2

Maybe need to add a pthread finding L16-17 & L88

cmake_minimum_required( VERSION 3.5 )

add_compile_options(-std=c++11)
#set(CMAKE_CXX_STANDARD 11) # C++11...
#set(CMAKE_CXX_STANDARD_REQUIRED ON) #...is required...
#set(CMAKE_CXX_EXTENSIONS OFF) #...without compiler extensions like gnu++11

# Create Project
project( sample )
add_executable( VelodyneCapture VelodyneCapture.h main.cpp )

# Set StartUp Project
set_property( DIRECTORY PROPERTY VS_STARTUP_PROJECT "VelodyneCapture" )

# Find Packages
# Find Package pthread
find_package(Threads REQUIRED)

# Find Package Boost
set( BOOST_ROOT "C:/Program Files/boost" )
set( Boost_USE_STATIC_LIBS ON ) # Static Link Libraries ( libboost_* )
set( Boost_USE_MULTITHREADED ON ) # Multi Thread Libraries ( *-mt-* )
set( Boost_USE_STATIC_RUNTIME OFF ) # Static Runtime Libraries ( *-s* )
find_package( Boost COMPONENTS system date_time regex )

# If Direct Capture from Sensors, VelodyneCapture are required Boost.Asio.
# Please Define HAVE_BOOST in Preprocessor.
set( HAVE_BOOST )
if( Boost_FOUND )
  set( HAVE_BOOST "-DHAVE_BOOST" )
endif()

# Find Package PCAP
set( CMAKE_MODULE_PATH "${CMAKE_CURRENT_SOURCE_DIR}" ${CMAKE_MODULE_PATH} )
set( PCAP_DIR "C:/Program Files/WpdPack")
find_package( PCAP )

# If Capture from PCAP Files, VelodyneCapture are required PCAP.
# Please Define HAVE_PCAP in Preprocessor.
set( HAVE_PCAP )
if( PCAP_FOUND )
  set( HAVE_PCAP "-DHAVE_PCAP" )
endif()

# Find Package OpenCV
set( OpenCV_DIR "C:/Program Files/opencv/build" )
set( OpenCV_STATIC OFF )
find_package( OpenCV REQUIRED )

# Required Viz Module
if( OpenCV_FOUND )
  if( NOT "opencv_viz" IN_LIST OpenCV_LIBS )
    message( FATAL_ERROR "not found opencv_viz module." )
  endif()
endif()

if( OpenCV_STATIC AND MSVC )
  foreach(flag_var
        CMAKE_CXX_FLAGS CMAKE_CXX_FLAGS_DEBUG CMAKE_CXX_FLAGS_RELEASE
        CMAKE_CXX_FLAGS_MINSIZEREL CMAKE_CXX_FLAGS_RELWITHDEBINFO)
    if(${flag_var} MATCHES "/MD")
      string(REGEX REPLACE "/MD" "/MT" ${flag_var} "${${flag_var}}")
    endif(${flag_var} MATCHES "/MD")
  endforeach(flag_var)
endif()

# Set Properties
if( OpenCV_FOUND AND ( PCAP_FOUND OR Boost_FOUND ) )
  # Additional Include Directories
  include_directories( ${Boost_INCLUDE_DIRS} )
  include_directories( ${PCAP_INCLUDE_DIRS} )
  include_directories( ${OpenCV_INCLUDE_DIRS} )

  # Add Definitions
  add_definitions( ${Boost_LIB_DIAGNOSTIC_DEFINITIONS} )
  add_definitions( ${HAVE_BOOST} )
  add_definitions( ${HAVE_PCAP} )

  # Additional Library Directories
  link_directories( ${Boost_LIBRARY_DIRS} )
  link_directories( ${PCAP_LIBRARY_DIRS} )
  link_directories( ${OpenCV_LIB_DIR} )

  # Additional Dependencies
  target_link_libraries( VelodyneCapture ${Boost_LIBRARIES} )
  target_link_libraries( VelodyneCapture ${PCAP_LIBRARIES} )
  target_link_libraries( VelodyneCapture ${OpenCV_LIBS})
  target_link_libraries( VelodyneCapture ${CMAKE_THREAD_LIBS_INIT})
else()
  message( WARNING "VelodyneCapture need at least either Boost or PCAP." )
endif()

I was commit based on your report.
VelodyneCapture was able to support Linux.
Thank you for your reports.


Fixes a bug in cross domain handling.

Based on the documentation and the rest of this function, there is no options.uri.  I suspect this was missed as part of some previous refactor that moved the URI to its own variable?

Thanks, I don't really maintain the browser version, since it's such a bad idea to actually use it.


Only disable Network test on watchOS

Instead of wholesale disabling the test.
Related to: rdar://problem/40799651

@swift-ci test


Printing markdown cells in interactive window doesn't trigger scroll-to-bottom

Scrolling issue in the interactive window: doesn't stick to bottom when printing markdown cells.
Minimal example
# %%
print("Fill vertical space in interactive window")
for i in range(100):
    print(i)

# %%
print("trigger scroll to bottom")

# %% [markdown]
# # Test
# not visible unless you scroll down manually

Slightly related: #10249

Thanks. I've seen this too. I think it has to do with the markdown renderer showing its output async. When we check to see if we should scroll or not, the output isn't there yet.


Add new services for deploying tooling

We need roles for:

ethstats
tx-fuzz
landing_page
blockscout

in order to use in smaller testnets that dont have k8s setups

ethstats https://github.com/ethpandaops/ansible-collection-general/pull/61


Installation fails on Kubernetes v1.24

Description
Installation fails on Kubernetes v1.24
Go Version
go version go1.20.4 darwin/amd64
Current behavior
Installation following the readme fails on Kubernetes v1.24.
Errors when applying the kustomize build (both with latest Kustomize, or with v3.8.10) - it uses deprecated API versions for several resources.
Expected behavior
Installation succeeds on recent Kubernetes versions
Steps To Reproduce
Steps to reproduce the behavior:

Have a Kubernetes 1.24 cluster (probably affects several earlier versions since the API changes)
Install cert-manager and test it successfully according to cert-manager docs (the command in the readme failed, but the Helm chart from cert-manager installed fine)
Install newrelic-kubernetes-operator with kustomize
See error

Debug Output (if applicable)
~/kustomize build https://github.com/newrelic/newrelic-kubernetes-operator/config/default | kubectl apply -f -
+ kubectl apply -f -
namespace/newrelic-kubernetes-operator-system unchanged
role.rbac.authorization.k8s.io/newrelic-kubernetes-operator-leader-election-role unchanged
clusterrole.rbac.authorization.k8s.io/newrelic-kubernetes-operator-manager-role configured
clusterrole.rbac.authorization.k8s.io/newrelic-kubernetes-operator-proxy-role unchanged
clusterrole.rbac.authorization.k8s.io/newrelic-kubernetes-operator-secret-reader unchanged
rolebinding.rbac.authorization.k8s.io/newrelic-kubernetes-operator-leader-election-rolebinding unchanged
clusterrolebinding.rbac.authorization.k8s.io/newrelic-kubernetes-operator-manager-rolebinding unchanged
clusterrolebinding.rbac.authorization.k8s.io/newrelic-kubernetes-operator-proxy-rolebinding unchanged
clusterrolebinding.rbac.authorization.k8s.io/newrelic-kubernetes-operator-secrets-rolebinding unchanged
service/newrelic-kubernetes-operator-controller-manager-metrics-service unchanged
service/newrelic-kubernetes-operator-webhook-service unchanged
deployment.apps/newrelic-kubernetes-operator-controller-manager unchanged
resource mapping not found for name: "alertsapmconditions.nr.k8s.newrelic.com" namespace: "" from "STDIN": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1beta1"
ensure CRDs are installed first
resource mapping not found for name: "alertschannels.nr.k8s.newrelic.com" namespace: "" from "STDIN": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1beta1"
ensure CRDs are installed first
resource mapping not found for name: "alertsnrqlconditions.nr.k8s.newrelic.com" namespace: "" from "STDIN": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1beta1"
ensure CRDs are installed first
resource mapping not found for name: "alertspolicies.nr.k8s.newrelic.com" namespace: "" from "STDIN": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1beta1"
ensure CRDs are installed first
resource mapping not found for name: "apmalertconditions.nr.k8s.newrelic.com" namespace: "" from "STDIN": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1beta1"
ensure CRDs are installed first
resource mapping not found for name: "nrqlalertconditions.nr.k8s.newrelic.com" namespace: "" from "STDIN": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1beta1"
ensure CRDs are installed first
resource mapping not found for name: "policies.nr.k8s.newrelic.com" namespace: "" from "STDIN": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1beta1"
ensure CRDs are installed first
resource mapping not found for name: "newrelic-kubernetes-operator-serving-cert" namespace: "newrelic-kubernetes-operator-system" from "STDIN": no matches for kind "Certificate" in version "cert-manager.io/v1alpha2"
ensure CRDs are installed first
resource mapping not found for name: "newrelic-kubernetes-operator-selfsigned-issuer" namespace: "newrelic-kubernetes-operator-system" from "STDIN": no matches for kind "Issuer" in version "cert-manager.io/v1alpha2"
ensure CRDs are installed first
resource mapping not found for name: "newrelic-kubernetes-operator-mutating-webhook-configuration" namespace: "" from "STDIN": no matches for kind "MutatingWebhookConfiguration" in version "admissionregistration.k8s.io/v1beta1"
ensure CRDs are installed first
resource mapping not found for name: "newrelic-kubernetes-operator-validating-webhook-configuration" namespace: "" from "STDIN": no matches for kind "ValidatingWebhookConfiguration" in version "admissionregistration.k8s.io/v1beta1"
ensure CRDs are installed first

Additional Context
Updating these references to the current v1 APIs does not solve this, as there are other errors reported by kustomize with recent kustomize versions. Running kustomize edit fix does not automatically fix those errors.
I tested with a version of kustomize, 3.8.10, from 2021. This resolved the kustomize errors seen with the new version but showed a different error:
Error: no matches for IdId admissionregistration.k8s.io_v1_MutatingWebhookConfiguration|~X|mutating-webhook-configuration; failed to find unique target for patch admissionregistration.k8s.io_v1_MutatingWebhookConfiguration|mutating-webhook-configuration
error: no objects passed to apply

So there seems to be something else needed.

Same here. If this is an abandoned project, it would be worth noting it in the README.

I just invested some time to fix the apply via kustomize: https://github.com/woehrl01/newrelic-kubernetes-operator/pull/1/files
The problem is now that all the go modules are outdated so that the webhook validation fails with:
Error from server (InternalError): error when creating "./newrelic-op.yaml": Internal error occurred: failed calling webhook "malertspolicy.kb.io": received invalid webhook response: expected response.uid="06efc0e3-2c46-404b-94bc-0adfc97d117e", got ""

I am uncertain if it's worth the trouble to update all those dependencies, too.
One question for the maintainers of this library, will a PR be reviewed and accepted? @thande

@thande I migrated the CRDs to work with current v1 api extensions.
https://github.com/newrelic/newrelic-kubernetes-operator/pull/152

@woehrl01 Checkout my PR, I fixed the admission hooks. Tested, it works ;)


Pysat removing literals during cardinality encoding

bv = boolvar(shape=3)
Model(sum(bv) < 10).solve('pysat')
print(bv.value())
When any solution is valid pysat returns None, instead of a random solution

This has to do with Pysat's built-in cardinality encodings (changed title accordingly).
Like many issues we have when optimizing some expressions, the variables are never seen by the solver as the cardinality encoding will simply return an empty list (equivalent to True)
We can either special case this in the Pysat solver interface, or (probably less efficient) call self.solver_vars(self.user_vars.keys()) before the solve-call to make sure all user variables are known to the underlying solver.

This issue is solved by #463, closing.


Documented the case of a scoped package

Description
Thanks to @phndiaye's pull request, I realized that the codemod has two bugs:

The workspaces field in the root package.json should list the package locations, not the package names. A bug would have surfaced when (1) the package is scoped or (2) an end-developer passes the argument --addon-location, --test-app-location, or --test-app-name.
The regular expression for computing the relative path is incorrect when the package is scoped.

References

https://docs.npmjs.com/cli/v7/using-npm/workspaces#defining-workspaces

Workspaces are usually defined via the workspaces property of the package.json file, e.g:
{
  "name": "my-workspaces-powered-project",
  "workspaces": [
    "workspace-a"
  ]
}

Given the above package.json example living at a current working directory . that contains a folder named workspace-a that itself contains a package.json inside it, defining a Node.js package, e.g:
.
+-- package.json
`-- workspace-a
   `-- package.json

https://classic.yarnpkg.com/lang/en/docs/workspaces/#toc-tips-tricks

The workspaces field is an array containing the paths to each workspace. Since it might be tedious to keep track of each of them, this field also accepts glob patterns! For example, Babel reference all of their packages through a single packages/* directive.

https://pnpm.io/pnpm-workspace_yaml

pnpm-workspace.yaml defines the root of the workspace and enables you to include / exclude directories from the workspace. By default, all packages of all subdirectories are included.
For example:
packages:
  # all packages in direct subdirs of packages/
  - 'packages/*'
  # all packages in subdirs of components/
  - 'components/**'
  # exclude packages that are inside test directories
  - '!**/test/**'

Thanks for this, @ijlee2 ! It will help a lot for the next addons I'm going to convert :)


Release/0.0.1

merge

Nice work!
I suppose you open the PR by mistake, but if you are interested, we are open to accept new contributors.
Regards,


Lottie autoPlay jest Invariant Violation

Description
If i add the autoPlay prop to a LottieView, jest returns this:
Invariant Violation: Unable to find node on an unmounted component.
Steps to Reproduce
Lottie:
<LottieView
    style={[
        styles.image,
    ]}
    source={require('src/assets/lotties/AllowAlerts.json')}
    autoPlay
    loop
/>

Jest:
import React from 'react';
import AllowNotificationsModal from '@allow-notifications-modal';
import renderer from 'react-test-renderer';

describe('Allow Notifications Modal', () => {
    it('should renders correctly', () => {
        const allowNotificationsModal = renderer.create(
            <AllowNotificationsModal />,
        ).toJSON();
        expect(allowNotificationsModal).toMatchSnapshot();
    });
});

**Expected behavior:
The test should pass
**Actual behavior:
Jest returns Invariant Violation: Unable to find node on an unmounted component.

Is there a proper solution to this?
I have very similar code to the aforementioned issue but my console is showing the error:
`    PrettyFormatPluginError: Invalid string lengthRangeError: Invalid string length
  at printObjectProperties (node_modules/jest-snapshot/node_modules/pretty-format/build/collections.js:172:47)`

which is coming from my Jest snapshot comparison which includes a Lottie

This issue is still around. I would recommend to reopen it.


update bindings.js for supporting Solus

Fixes #856

Coverage remained the same at 99.319% when pulling 9ae8918727472c40338f458b49cc6b745e2b2f04 on ekremkaraca:patch-1 into ec617f24896ba977174bb5658aeb8d07b97cd2f8 on lovell:master.

Thank you!


docs: docs structure improvement

Signed-off-by: Daniel Hu<EMAIL_ADDRESS>Pre-Checklist
Note: please complete ALL items in the following checklist.

[x] I have read through the CONTRIBUTING.md documentation.
[x] My code has the necessary comments and documentation (if needed).
[x] I have added relevant tests

Description

move /tutorials/best-proctices to /best-proctices;
move /tutorials/autocomplete.md to /commands/autocomplete.md;
all links updated accordingly.

TODO
multi-version won't works well. I have made some attempts, including the mike way and the json way. I also tried to test with the fork version after retyping the tag, considering that the old version did not have these configurations, but it did not solve the problem perfectly.
some ref:

https://squidfunk.github.io/mkdocs-material/setup/setting-up-versioning/
https://github.com/squidfunk/mkdocs-material/discussions/3566

Let's try multi-version support later together. Merging now.


Introduce TimeOut waiter

Let me know if any change should be done to be merged

changes done

Thank you @tyx!


Installing Elixir Error - Uncaught error in rebar_core

kiex install 1.4.0                                                                     2.3.1
Downloading elixir version 1.4.0
Installing elixir version 1.4.0
Already on 'master'
Your branch is up-to-date with 'origin/master'.
Switched to a new branch 'v1.4.0'
From https://github.com/elixir-lang/elixir

tag               v1.4.0     -> FETCH_HEAD
Already up-to-date.
cd lib/elixir && "/home/milk/.kiex/builds/elixir-git/rebar" clean
==> elixir (clean)
rm -rf ebin
rm -rf lib//ebin
rm -rf lib/elixir/test/ebin
rm -rf lib//tmp
rm -rf lib/mix/test/fixtures/git_repo
rm -rf lib/mix/test/fixtures/deps_on_git_repo
rm -rf lib/mix/test/fixtures/git_rebar
rm -rf lib/elixir/src/elixir.app.src
make clean_man
make[1]: Entering directory '/home/milk/.kiex/builds/elixir-git'
rm -f man/elixir.1
rm -f man/iex.1
make[1]: Leaving directory '/home/milk/.kiex/builds/elixir-git'
Uncaught error in rebar_core: {error,
{erl_interface,include,
"code:lib_dir(erl_interface)is unable to find the erl_interface library."}}
Makefile:69: recipe for target 'erlang' failed
make: *** [erlang] Error 1
Uncaught error in rebar_core: {error,
{erl_interface,include,
"code:lib_dir(erl_interface)is unable to find the erl_interface library."}}
Makefile:69: recipe for target 'erlang' failed
make: *** [erlang] Error 1
Problem with Elixir build.  Aborting!

Solve it! Removed the Elixir OS installation, reinstall just Erlang and try install Elixir with Kiex. Done! It's works. :)

What OS and version?  What version of Elixir was installed with the OS?

@taylor Fedora Core 25 and Elixir 1.3.1 release 1.fc25


fix incorrect labels for watching resources.

Please provide a description for what this PR is for.
resolve: https://github.com/istio/istio/issues/23603
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ X ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

In response to a cherrypick label: new pull request created: #24481


FR Open file passed via command line argument

It would be great if icy draw could open a file it was passed on the command line, like every other program ever.

implemented.


Ldamata/fix least request lb not fair

Commit Message: Fix Least requests LB when doing a random pick so it removes already chosen hosts from the random function to remove the chance of selecting the same host again when dealing with a small amount of hosts. Also, when the number of choices is smaller or equal the number of hosts, use full scan of least used instead.
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
This Release changes the default behaviour of Least Request Load Balancer doing a full scan when the number of choices is more than equal the size of hosts and also adds a new option on the envoy::extensions::load_balancing_policies::least_request::v3::LeastRequest configuration to always do a full scan. Allowing a full scan instead of random making choices reduces the chance of selecting a host that doesn't have least requests when the number of hosts is smaller.
Platform Specific Features:
[Optional Runtime guard:]
Fixes #11004
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/assign

/wait

/wait

Please merge main and add a release note. Then we can land this. Thanks.
/wait

Not sure If I need someone else to approve since "envoyproxy/api-shepherds must approve for any API change" is pending.

perhaps @lizan needs to take a look again?

We are reverting this change since we noticed that there is a significant impact on hosts with a small number of hosts caused by the deterministic loop which makes every client pick the same hosts. I think we need a better understanding of the impact with a load simulation for this change to be re-applied. Please consider adding some independent sampling to the algorithm and we can do a more thorough review again.

I understand that changing the behaviour would have impacted people, but It is not clear to me that adding an option to enable full scan would have a bad impact since it is explicitly selecting the host with least requests. Wondering if we can put add it back?

The API will be kept and new implementation is still welcome. And @tonya11en is working a simulated system to ensure we can get a more reasonable implementation in the future.
Thanks for your contribution and so sorry for that we need to revert it. We know it taken you lots of time. :(
Hope we can bring it back soon.

If the patch is limited to only the “full scan” flag, it should do what you want without affecting the existing selection behavior.
On Fri, Nov 10, 2023, at 6:17 AM, code wrote:

The API will be kept and new implementation is still welcome. And @tonya11en https://github.com/tonya11en is working a simulated system to ensure we can get a more reasonable implementation in the future.
Thanks for your contribution and so sorry for that we need to revert it. We know it taken you lots of time. :(
Hope we can bring it back soon.
—
Reply to this email directly, view it on GitHub https://github.com/envoyproxy/envoy/pull/29873#issuecomment-1805822182, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAIOZ7QPECF7Z7BNKBKEMJTYDYZPRAVCNFSM6AAAAAA5MROLXKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBVHAZDEMJYGI.
You are receiving this because you were mentioned.Message ID: @.***>

Hello, I have this PR open that changes the behaviour for a random index: https://github.com/envoyproxy/envoy/pull/31146


Add CD bit to flags.

Hi, I'm writing a utility that needs to capture and decode DNS packets and am using this library. I'm finding lots of packets don't decode properly and am fixing things one at a time.
This adds support for parsing packets with the CD bit set. See this[0] for details. It's similar to the previous AD patch.
[0] - https://tools.ietf.org/html/rfc6895

Looks good. Thanks!


Please help me with this basic part I don't understand

I tried to make a wand
For example
/wand create
/wand name
/wand add (spel)
But when I click right or left, it doesn't open the magic store and when I use magic, what should I do next?

Try /wand configure inherit base_wand
I'm not entirely sure that'll work, but basically if you're trying to make something that works like the survival wand you will need to configure a lot more stuff. Key bindings, enchantment table behavior, mana functionality, etc- all of these things are something you have to set up.
If you're just trying to get and use a wand, just do /mgive wand and use the built-in one.
If you want to make your own custom wand, it's much easier to use the editor and start with one of the premade templates. Try /meditor wand mywand for that.


Project Ready for Marking

WORK DONE:

Inquirer package prompts employee questions
Employee class and Manager, Engineer, and Intern subclasses all passing tests via jest
Questions looping properly and validated
styled HTML file generated

adjusted email validation regex
updated readme


Triggering error manually works, but the class is not changed

validettaObj.window.open.call(validettaObj, inputElement, errorMessage); this triggers error manually & it works very nice.
But the parent element still uses validetta-valid instead of validetta-error. Not big issue, but i think would be nice if it can be fixed from the library instead of patching it manually.

Hi @bersujamu
I know about that. It will be fixed in the next version.
Thank you

This can be followed #44.


Introduce settings panel and enabling/disabling functionality

Meant to implement https://github.com/hypothesis/wp-hypothesis/issues/3. I've been doing wordpress stuff recently and realized this could be completed fairly quickly... so I just did it.

Adds Hypothesis to the wordpress settings menu.
Allows for admin users to configure which pages, posts, or categories the embed code is loaded on.

Looks like this:

Not being familiar with WordPress plugins it's hard to review, but it looks okay. Please clean up the whitespace, though. It seems to be a mix of tabs and spaces right now.

White space cleaned up!

Woo! Super cool :-)
On Mon, Jun 22, 2015, 18:33 Jake Hartnell<EMAIL_ADDRESS>wrote:

White space cleaned up!
—
Reply to this email directly or view it on GitHub
https://github.com/hypothesis/wp-hypothesis/pull/5#issuecomment-114319852
.

LGTM.

@JakeHartnell you're amazing!!! :smile:
Thanks for tackling this! It should give us a nice foundation to build up from.
Much thanks!
:gift:


Content-Length is missing from response header.

Content-Length is missing from response header.
Response from FastEndpoint:
HTTP/1.1 200 OK
Content-Type: application/json
Date: Tue, 12 Apr 2022 12:56:56 GMT
Server: Microsoft-IIS/10.0
Content-Encoding: gzip
Transfer-Encoding: chunked
Vary: Accept-Encoding
X-Powered-By: ASP.NET

Response from Asp.Net MVC
HTTP/1.1 200 OK
Content-Length: 672
Content-Type: application/json; charset=utf-8
Date: Tue, 12 Apr 2022 12:59:12 GMT
Server: Microsoft-IIS/10.0
Content-Encoding: gzip
Vary: Accept-Encoding
X-Powered-By: ASP.NET

minimal api doesn't usually send conten-length header afaik for ex:
app.MapGet("test-min", () => new { test = "yes" });

most likely because there's a performance hit incurred if you have to send the content-lenght.
if it's essential you can incurr that cost and configure fastenpoints to send the conten-lenght with a custom response serializer func like the following:
app.UseFastEndpoints(c =>
{
    c.ResponseSerializer = async (rsp, dto, contentType, jCtx, cancellation) =>
    {
        using var source = new MemoryStream();
        await JsonSerializer.SerializeAsync(source, dto);
        rsp.ContentType = contentType;
        rsp.ContentLength = source.Length;
        source.Position = 0;
        await source.CopyToAsync(rsp.Body);
    };
});


Fit into windows system setting

Description of the new feature / enhancement
Fit into windows system setting, directly running as a system fucntion.
Scenario when this would be used?
Easy to make changes;
Running as a service or something, not annoyed with a window based setting window.
Supporting information
No response

Can you explain more about the scenario?  Trying to be sure i'm understanding the full ask / user scenario versus guessing.
/needinfo

Sorry for late reply; Just think no need to start the program as a independent one, just make the configuration as part of system configuration item. Thank you

do yo

sorry, trying to be sure i fully understand you ask.  /needinfo
Is it you want PowerToys Run to directly allow for system settings to be adjusted?
if so:
Settings: I don't think we'd do this as the options / experiences change from Win10 and 11.  It is more effective to transition to there.
Services, if you have the Services plugin running for PowerToys Run, you can already adjust those directly.


About BIAN Customer Offer Service Domain

Hello, Sir,
I'm BaoYu Wang,  Thanks a lot for your wonderful Coreless Banking Demo,  Two questions need your advice help.

"Customer Offer" BIAN Service Domain is mentioned in Readme, but In model directory, the code file name is "CustomerOfferEvent.java" , just wonder they mean the same thing？
Which BIAN back-end Service is simulated in Camel to responde to request from "PartyRoutingProfileTest.java", How that Java code work as API provider?

Hi @wangbyk,
The implementation of the Coreless Demo is outdated. You have to look at the https://github.com/open-accelerators/mercury repository.
There, the model is generated and packaged in the customer-offer-common module from the OpenAPI schema, currently v9 and published as a GitHub package here
Currently we have implemented 3 examples consuming the mercury-framework. They're also available in the examples folder

Hi @ruromero,
OK, got it,  thanks a lot for your kindly update, We'll study it later.


/bin/bash: node: command not found

my yml

On my macOS /bin/bash: node: command not found

Getting the same. Looks like node.js is not supported by default

Hi Folks, sorry for the delay.
If you don't have Node.js on the Agent, the easiest thing to do is to use the Docker Node.js example.


How to add whitelist patter for purgecss?

How do I add purgecss options to this plugin ? example for vue transition, I would like to do some whitelist on purgecss:
      whitelistPatterns: [         /-enter-active$/,         /-leave-active$/,         /-enter$/,         /-leave-to$/       ],

Pass it in to options.purgeConfig.


azblob: STG 90 Preview

Preview release for azblob up to STG 90

[x] The purpose of this PR is explained in this or a referenced issue.
[x] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[x] Tests are included and/or updated for code changes.
[x] Updates to module CHANGELOG.md are included.
[x] MIT license headers are included in each file.

/azp run go - azblob


Prerender static index.html

Is it possible to use prerender on static .html files? eveything i have seen so far seems to trigger it from an ASP.net view.
We currently serve up several angular apps that are managed via bower. So they will be mapping asp.net mvc routes to specific index.html files.
i was thinking that there should be a way to do this through asp.net middle ware?

Is it possible to use prerender on static .html files? eveything i have seen so far seems to trigger it from an ASP.net view.

Currently all the templates and samples do work by injecting prerendered content into a Razor view's output. I'm guessing you want one of these two alternatives:

At build time, emitting static .html files to disk, containing the rendered SPA's HTML output. When loaded into visitors' browsers, it will be entirely static, non-interactive content.
Or, your goal is to use prerendering at runtime to produce entire HTML documents, not involving Razor at all

If your requirement is (2), then that's something that others have requested too (see #182). We may implement something like that soon, or perhaps someone in the community will do so.
If your requirement is (1), then that's pretty unusual and hasn't been requested by anyone else. It would certainly be possible to do it by writing some custom code to invoke the prerendering APIs on your dev machines (e.g., from a command-line tool) and writing the output to disk. We don't have any plans to bake such a feature into the templates because it would be a very unusual use case and nobody else has asked for it. But I'm sure you could figure out a way if it's important to your project.

BTW if I'm misinterpreting your requirement, please re-open and clarify what you're looking for that might be different from the two options I've mentioned above :)

@SteveSandersonMS its more along the lines of number 2. We have static html files for our angular apps that we serve up with asp.net but not via razor. we would use ASP.net more for a backend web api and server site routing.
so i would imagine you would have some sort of middle ware for the routes that would use prerender on the way out.


jdk8 hotspot zLinux & Windows JDI test failures: UnsupportedOperationException: target does not support popping frames

Since 17th March these have started failing:
https://trss.adoptopenjdk.net/deepHistory?testId=605bd5605730424dbcb6e571
eg.https://ci.adoptopenjdk.net/job/Test_openjdk8_hs_sanity.openjdk_s390x_linux/464/testReport/junit/com_sun_jdi_PopSynchronousTest/java/PopSynchronousTest/
run args: [PopSynchronousTarg]
TEST pop from 'a' to 'a'
java.lang.UnsupportedOperationException: target does not support popping frames
	at com.sun.tools.jdi.ThreadReferenceImpl.popFrames(ThreadReferenceImpl.java:539)
	at PopSynchronousTest.pop(PopSynchronousTest.java:216)
	at PopSynchronousTest.reenter(PopSynchronousTest.java:223)
	at PopSynchronousTest.runTests(PopSynchronousTest.java:252)
	at TestScaffold.startTests(TestScaffold.java:429)
	at PopSynchronousTest.main(PopSynchronousTest.java:113)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at com.sun.javatest.regtest.agent.MainActionHelper$AgentVMRunnable.run(MainActionHelper.java:312)
	at java.lang.Thread.run(Thread.java:748)

JavaTest Message: Test threw exception: java.lang.UnsupportedOperationException

These failing tests don't look as though they were run prior to 17th Mar: https://ci.adoptopenjdk.net/job/Test_openjdk8_hs_sanity.openjdk_s390x_linux/461/testReport/

https://github.com/AdoptOpenJDK/openjdk-tests/commit/af6c98f26a0bea29e610519420b297a3e545a21a#diff-9e2b6489dae12580d3f0543dea5e3457a060f3327902b5b31edf187b133a1110
@M-Davies I will re-exclude for Windows and s390xLinux

jdk8 zLinux is a "zero" (no-jit) impl which does not support pop-frames see:
https://github.com/AdoptOpenJDK/openjdk-jdk8u/blob/a1d853a5d797f38f7be8c260f4a83c1b3fe48adc/hotspot/src/share/vm/prims/jvmtiManageCapabilities.cpp#L121

Windows failure is a different reason, not popFrame:
19:20:38  STDERR:
19:20:38  ShellScaffold.sh: Version
19:20:38  E:/workspace/Test_openjdk8_hs_sanity.openjdk_x86-64_windows/openjdk-tests/openjdk/openjdk-jdk/jdk/test/com/sun/jdi/ShellScaffold.sh: line 277: C:\Windows/tmp.6622: Permission denied
19:20:38  cat: 'C:\Windows/tmp.6622': No such file or directory
19:20:38  mkdir: cannot create directory '/aa6622': Permission denied
19:20:38  touch: cannot touch '/aa6622/jxdbOutput.txt': No such file or directory
19:20:38  E:/workspace/Test_openjdk8_hs_sanity.openjdk_x86-64_windows/openjdk-tests/openjdk/openjdk-jdk/jdk/test/com/sun/jdi/ShellScaffold.sh: line 380: cd: /aa6622: No such file or directory
19:20:38  tail: cannot open '/aa6622/jxdbOutput.txt' for reading: No such file or directory
19:20:38  tee: /aa6622/jxdbOutput.txt: No such file or directory
19:20:38  tail: cannot open '/aa6622/jxdbOutput.txt' for reading: No such file or directory
19:20:38  --Top process (6622) is dead.  We better die too
19:20:38  --Top process (6622) is dead.  We better die too

I've created a separate issue for Windows which is a different problem: https://github.com/AdoptOpenJDK/openjdk-tests/issues/2415

Permanent exclude due to jdk8 hs zero vm limitation: https://github.com/AdoptOpenJDK/openjdk-tests/pull/2416


[Bug]  tooltip repeat rendering

Version
5.2.2
Link to Minimal Reproduction
No response
Steps to Reproduce
1.triggerOn=mousemove
2.formatter: function(params) {
return ;
}
3.Viewing Network Requests
Current Behavior
Render should not be repeated on the same data
Expected Behavior
Render should not be repeated on the same data
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
No response

formatter: function(params) {
return <img src='xxxxx'>;
}

The tooltip will empty the previous content on each render. In this case, you should create DOM outside the formatter callback and return it. BTW, the "repeated rendering" generally occurs only when the cache is disabled, for example, you opened the devtools and enabled Disable cache or the image URL or its query parameter is changed.


net: unable to bind the specified network card to establish a TCP connection

Failure to bind to the specified network card will occur on some machines. I wanted to connect from <IP_ADDRESS>, but actually went to the <IP_ADDRESS> network card, but the source address of the packet was <IP_ADDRESS>. Very strange.
It's been on two devices, but the other devices are fine. The systems of the two devices in question are: win7 ultimate and win10 enterprise. Other devices have no problems with such systems.
Is it unclear whether golang supports bind? Or is there some other software that affects bind?
golang version : go1.14
netAddr := &net.UDPAddr{}
netAddr.(*net.UDPAddr).IP = net.ParseIP("<IP_ADDRESS>")

dialer := net.Dialer{LocalAddr: netAddr, Timeout: 5 * time.Second}
dialer.Dial(network, address)

Thank you for raising this issue. Can you please provide a runnable code sample that someone who does not have access to your environment can use to verify the issue you are seeing.
Thank you

Setting the LocalAddr field will cause the net package to call bind.  I'm not sure this is a Go problem.

You don't bind a socket to a network card, you bind it to a local IP address (and port). The routing system (using the routing table) determines where the traffic goes. So, it is not strange that the source address for outgoing packets is the address you bound the socket to. That is always the case. Now, Windows by default uses the strong host model, which means that outgoing packets must have a source address that is assigned to the outgoing network interface. This rule is also used to make the routing decision, so if you force the local IP address by binding a socket to an address, then for an outgoing packet a route will be selected that satisfies the requirement. So if the routing table contains several possible routes to a destination, for example Wi-Fi and Ethernet both with a default route, then if you bind a socket to the Wi-Fi address, then packets will be routed using the default route on the Wi-Fi interface even if the Ethernet interface has a default route with a lower metric. This is because the strong host model does not allow packets going out on the Ethernet interface with the source address assigned to the Wi-Fi interface. If I interpret @caleb-letsgo 's issue correctly, this behavior was what was expected. Now for the Windows devices where it doesn't work this way, these devices probably are configured to use the weak send model instead. Note that most other systems such as Linux do not support the strong host model at all, so they always work as your two problematic Windows systems.

Thank you very much。After the test certification, what you said is right, awesome!


rust-analyzer not running or installed on remote machine yet no diagnostics

I have a new Pocketbeagle that I tried bringing up Rust on.  The Rust stuff is fine.
Then I tried to install the remote stuff for Visual Studio Code.  It claims to have installed fine.
However, I get no "Rust..." entries in the command palette, nor do I get any error.  It simply says installed even though there is no rust-analyzer binary on the remote computer anywhere.
Last time, I at least got a notification saying that my platform had no binaries and that I had to use cargo xtask, et al. to compile it up.
Thanks.

Closed in favor of #5091.


Fix dependencies

PR's key points
…
How to review this PR?
…
Related Issues
…

          1 Error

:no_entry_sign:

fastlane-plugin-mm_toolkit.gemspec
32
Dependencies should be sorted in an alphabetical order within their section of the gemspec. Dependency danger-rubocop should appear before pry.

  Generated by :no_entry_sign: Danger


Querying deprecated fields on introspection

Altair querying deprecated fields (onOperation, onFragment, onField) on introspect schema
{
	"errors": [
		{
			"message": "Cannot query field \"onOperation\" on type \"__Directive\".",
			"extensions": {
				"category": "graphql"
			},
			"locations": [
				{
					"line": 17,
					"column": 9
				}
			]
		},
		{
			"message": "Cannot query field \"onFragment\" on type \"__Directive\".",
			"extensions": {
				"category": "graphql"
			},
			"locations": [
				{
					"line": 18,
					"column": 9
				}
			]
		},
		{
			"message": "Cannot query field \"onField\" on type \"__Directive\".",
			"extensions": {
				"category": "graphql"
			},
			"locations": [
				{
					"line": 19,
					"column": 9
				}
			]
		}
	]
}

Desktop:

OS: Windows 10
Browser Chrome
Platform: Chrome extension and desktop appp
Version: 4.2 on Chrome extension and 4.3.1 on Desktop App

Could you provide some more context? I'm not sure I understand the issue here.

Going to close this issue as stale. Feel free to comment here if that is not the case.


Input cleared on validation on a mobile device

Versions:

VueJs: 2.5.16
Vee-Validate: 2.0.8

Description:
When on a mobile device, when an input fails the validation (like email), the input field is cleared of it's value.  Unless I'm doing something wrong.
Steps To Reproduce:
Create an input with v-validate="'required|email'" and try to type into the input on a mobile device (Android in my case).  Once the validation has fired, the error message is generated and the input value is cleared.  So basically you can't type anything, unless you add a delay using data-vv-delay="1000", which will clear it after 1 second.
Any ideas?

Could you create an example on codesandbox, and check if its similar to #1251

Is there anything in your module that clears the input for any reason? just trying to get a clue as to what is going on here as it works perfectly on any desktop browser.

I've investigated this problem and found that on a mobile device, Vue does not update the data as you type when using 2 way binding (v-model), it updates much less frequently.  So what happens is the update back to the model hasn't happened yet and the validation fires.  When the control is updated to show the error, the empty data from the data model is written back into the input, clearing it.
Maybe a way to fix it would be to make sure all the v-model fields are updated back to the data at the start of the validation, is there a way to do that?

Currently, vee-validate uses Vue.$watch API if the model is present, and HTML5 listeners if it's not. If Vue's listener handler occurs after the validation the input will seem to be locked since it triggers DOM updates due to the errors object adding/removing messages.
The idea is to detect when v-model triggeres the value commitment and validate afterward, for example, .lazy is handled to validate on change instead of input for mobile devices I have not yet found which event is being used on the v-model. I've seen composing events being used but I have to read up on them more.

Is there something we can do to prevent this until a fix has been implemented ? I am currently facing this exact problem.

On the actual input in the Vue template adding data-vv-validate-on="blur" also solves the problem, although obviously validation doesn't occur until focus is lost, which was fine for my implementation.

@logaretm When will this issue be fixed? It's really annoying for our users.

When you make a hefty donation or provide a PR.

Haven't found the time to implement a proper fix, you could always validate
on change event and it should work.
On Wed, 20 Jun 2018, 11:15 Arend de Boer<EMAIL_ADDRESS>wrote:

When you make a hefty donation or provide a PR.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/baianat/vee-validate/issues/1281#issuecomment-398681511,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-KSmsLvnDjlXfF68Qoy1QI1dupVEeKks5t-hKZgaJpZM4TlkrS
.

@logaretm Will it break anything or change the behaviour in any direction if we use the change event?

The change event occurs when the value of the input is committed, typically
with the blur event, so the validation will be less aggressive, validating
only when the user leaves the input. Changing this as the default behaviour
may produce unintended results for the current users. So you should make
the decision of when to va
On Thu, 21 Jun 2018, 11:49 pSchaub<EMAIL_ADDRESS>wrote:

@logaretm https://github.com/logaretm Will it break anything or change
the behaviour in any direction if we use the change event?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/baianat/vee-validate/issues/1281#issuecomment-399043979,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-KSpWcV9wyfhyoO52tP7GD7DpfrcONks5t-2wsgaJpZM4TlkrS
.

I revisited this issue with 2.1.0-beta.2 on Moto-Z Andriod 8.0, it seems to happen when using v-model on a non-existing model at the component ctor options, which won't be fixed.
Please, whoever has this issue create an example so I can confirm it, or maybe I missed a test case.
I will close this for now since it occurs due to improper model binding and already has a workaround by using the change event.

@logaretm Can you explain what "on a non-existing model at the component ctor options" means?

I mean its not initialized in the data:
<template>
    <form @submit.prevent="validateForm">
    <input type="text" v-model="name" v-valiname="'required'" name="name">
    <pre>
      {{ errors.first('name') }}
    </pre>
    </form>
</template>

<script>
export default {
  data: () => ({
    val: ''
  })
}
</script>

The name model does not exist and will trigger this behavior if it is not the case for you please produce a reliable demonstration so I can better understand the problem.

I still have the same problem with Motorola and Huawey :( some solution apart from .lazy , thanks thanks

I found that it has no problem on Firefox, but it is not stable in chrome. When it is an object's property, if the property is null, it may not be able to input. If you give the property a default value, there will be no problem.

@kent666a thx. I added empty '' string instead null and now it works :) Thx for save my day.
Btw -> Validation Provider + Vee validate 2.1.6 + Vue 2.5.22

I will close this for now since it occurs due to improper model binding and already has a workaround by using the change event.

Changing event will not help us , as its your library, in all cases it must work on all platforms,because I loved vee-validate just because of keyup validation and my issue is, input gets cleared if validation is DETECTED only incognito android chrome,and works like charm on all devices, changing to event catches validation on focus out and submit,i need a fix for my issue without changing behaviour


split metric in pre- and post- sync/reduce computation steps

🚀 Feature
split metric in pre- and post- sync/reduce computation steps
decide what to reduce how (usually, this resolves in gathering results of the pre-sync outputs and aggregating them)
Motivation

Pitch

Alternatives

Additional context

I guess this can be close with the new interface in place


Getting translation issues in the sort options on the Category carousel edit page.

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
master
Issue Description
Getting translation issues in the sort options on the Category carousel edit page.
Preconditions

Admin should be logged in.
Admin should have created a category carousel.

Steps to reproduce

Go to 'Admin panel'
Click on 'Edit Category Carousel' inside the Settings/Theme
See the error

Expected Result
It should be shown the proper translation.
Actual Result
getting translation issues. Here
Screenshots
No response
Additional context
No response

@Amitk-webkul Assign this issue to me.


[SPARK-17654] [SQL] Propagate bucketing information for Hive tables to planner

What changes were proposed in this pull request?
This PR depends on https://github.com/apache/spark/pull/15300 and includes following changes to have better planning for Hive bucketed tables:

HiveTableScanExec now exposes outputPartitioning and outputOrdering as per bucketing spec.
InsertIntoHiveTable now exposes requiredChildDistribution and requiredChildOrdering based on the target table's bucketing spec.

Despite of this PR, Spark still won't produce bucketed data as per Hive's bucketing guarantees, but will allow writes IFF user wishes to do so without caring about bucketing guarantees. I am incrementally working on closing the gaps to have complete Hive bucketing support in Spark but those will be separate PRs (eg. PR to add Hive's hashing function https://github.com/apache/spark/pull/15047)
How was this patch tested?
This PR depends on https://github.com/apache/spark/pull/15300 to let Spark create hive bucketed tables. Once that gets in, I can write unit tests for this PR.

Test build #65862 has started for PR 15229 at commit 8726cc6.

Test build #65862 has finished for PR 15229 at commit 8726cc6.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65862/
Test FAILed.

Merged build finished. Test FAILed.

Test build #66114 has started for PR 15229 at commit db82040.

Test build #66114 has finished for PR 15229 at commit db82040.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66114/
Test FAILed.

Merged build finished. Test FAILed.

Test build #66125 has started for PR 15229 at commit 9b61e39.

Test build #66128 has started for PR 15229 at commit 23986a8.

Test build #66128 has finished for PR 15229 at commit 23986a8.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66128/
Test FAILed.

Test build #66125 has finished for PR 15229 at commit 9b61e39.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66125/
Test PASSed.

@tejasapatil how does HIve store partitioning files?

Hi @rxin,
In Hive you have two levels, the partition and the buckets.
The partitons are translated to folders on HDFS, for example:
/apps/hive/warehouse/model_table/date=6

Where model_table is the name of the table and date is the partition.

If you create a table like this on Hive:
create table events (
  timestamp: long,
  userId: String,
  event: String
)
partitioned by (event_date int)
clustered by (userId) sorted by (userId, timestamp) into 10 buckets;

Then when it will be only 10 files per partition and all the events for one user will be only on one partition and sorted by time.
If you insert data on this table using the next query on Hive you will see that the clustering policy is respected:
set hive.enforce.bucketing = true;  -- (Note: Not needed in Hive 2.x onward)
from  event_feed_source e
insert overwrite table events
partition (event_date = 20170307)
select e.*, 20170307
where event_day = 20170307;

However... if you do the next insert with Spark:
sqlContext.sql("insert overwrite table events partition (event_date = 20170307) select e.*,1 from event_feed_source e")

You will see that the data is stored with the same partitioning as it is on the source dataset.
What is the benefit of respecting the Hive clustering policy?
The main benefit is to avoid shuffle and have a control on the number or partitions.
To give an example we have a pipeline that reads thousands of events per user and save them into another table (model), so it means the events table is going to have x times more data than the model table (imagine a factor of 10x).
First point is, if the source data are clustered properly we can read all the events per user without shuffle (I mean to do something like events.groupBy(user).mapValues(_.sortBy(timestamp) will be done without shuffle).
Second point is when we generate the model RDD/Dataser from the event RDD/Dataset. Spark respects the source partitioning (unless you indicate otherwise) which means... is going to save into Hive 10 times the number of files for the model as needed (not respecting the clustering policy on Hive).
This implies that we have 10x more partitions than needed and also that the queries over the model table are not "clustered"... which means full scan every time we need to do a query (a full scan over 10 times the optimal number of partitions).
I hope I clarify the point on Hive clusters ;)

@carlos-verdes : Thanks for the information. This is moved under an umbrella jira (SPARK-19256) which has a proposal : https://docs.google.com/document/d/1a8IDh23RAkrkg9YYAeO51F4aGO8-xAlupKwdshve2fc/edit
I believe all your requirements are captured in the proposal. If not, let me know. Meanwhile, I will close this PR and re-open when the right pieces are together.


[Site Request] olympics.com

Checklist

[x] I'm reporting a new site support request
[x] I've verified that I'm running yt-dlp version 2021.09.02
[x] I've checked that all provided URLs are alive and playable in a browser
[ ] I've checked that none of provided URLs violate any copyrights
[ ] The provided URLs do not contain any DRM to the best of my knowledge
[x] I've searched the bugtracker for similar site support requests including closed ones

Example URLs

Single video: https://olympics.com/tokyo-2020/en/replay/164b045b-431e-478c-8c1c-0629fe05b823/jumping-team-final
Single video: https://olympics.com/tokyo-2020/en/replay/300622eb-abc0-43ea-b03b-c5f2d429ec7b/jumping-team-qualifier

The above video URLs are not supported by yt-dlp. None of the video from olympics.com is supported.

Please consider adding this website.
Thanks.

I can get all the data but formats
first req to get token https://appovptok.ovpobs.tv/api/identity/app/token?api_key=OTk5NDcxOjpvY3N3LWFwaXVzZXI%3D&api_secret=ODY4ODM2MjE3ODMwYmVjNTAxMWZlMDJiMTYxZmY0MjFiMjMwMjllMjJmNDA1YWRiYzA5ODcxYTZjZTljZDkxOTo6NTM2NWIzNjRlMTM1ZmI2YWNjNmYzMGMzOGM3NzZhZTY%3D
then to this https://appocswtok.ovpobs.tv/api/schedule-sessions/164b045b-431e-478c-8c1c-0629fe05b823?fields%5BCommentary%5D=title%2Cname%2CsortOrder&fields%5BScheduleSession%5D=title%2Ccommentaries%2Cstate%2CsuperSession%2Cstart%2Cend%2CcompetitionDate%2CbroadcastStart%2CbroadcastEnd%2CbroadcastPublished%2CbroadcastUnpublished%2CtimelineMarkers%2CferVod%2CdisciplineCode%2Corganisations%2Cstream&fields%5BSpriteSheet%5D=columns%2Crows%2Csource%2Ctiles&fields%5BStream%5D=streamType%2CmanifestUrl%2Ctt%2CpromotionStatus%2Calias%2Cpath%2Cstream%2Cstart%2Cend&fields%5BVod%5D=title%2CvideoType%2CspriteSheet%2Cthumbnail&include=commentaries%2CferVod.spriteSheet%2Cstream with header x-obs-app-token and value as result of previous req
then parse attributes and make req like  https://meteringtok.ovpobs.tv/api/playback-sessions?tt=nqjP3Sn93Dmzs4R1ehZfBrLEsJm2At8guMCzCV%2FbS0w%3D&start=1628330134&end=1628339825&alias=tok1000804&stream=ch31&type=vod with same header as above
This will give u the manifest url
I'm an stuck here, i can't get to connect to it even with same headers as webrequest...even tried cookies.

Are you trying to get the manifest manually? If so, the issue could be that the link is expiring faster than you can test it. See https://www.reddit.com/r/youtubedl/comments/p1e8eo/olympic_videos/h8ecq7c

The site has new/videos and sports playlists aswell but i've only added replay extractor.


Reset doesn't work after first submission using new method

Hey,
Really enjoying using conform. Just trying out the new reset method in v0.7.0 and can't get it to work, after the second submission it won't reset.
Here is a minimal example: https://codesandbox.io/p/sandbox/jovial-field-h3qckx?embed=1&file=%2Fapp%2Froot.tsx%3A43%2C18
If you submit a value in the text box it will be cleared the first time, afterwards it won't get cleared if you try another input.
Pretty sure it's something to do with passing in the last submission, and probably a useEffect or something under the hood not being updated with new deps.

Hi @Mattinton, I am glad you like it :)
Thanks for the report! I overlooked the situation after the first reset happen. If the submission has been successful consecutively, the internal useEffect call will not be executed as it is still receiving the same null value and thus no reset happens.
Here is a PR (#194) with an alternative solution.

Hey, thanks for coming back to me so fast. Looked over the PR, that'll do the trick!
Looking forward to it getting merged 😄

Fixed on v0.7.2.


Cycle detection in linked list

Implementation of hare and tortoise algorithm to detect cycle in linked list.

I would like to implement it as part of GSSOC 2021 for C++

Please assign this problem to me I would like to implement it under GSSOC 2021 using java

I want to work on this issue. I can code it in Python with correct comments and code flow Please assign me this project in python

Hello, I am from GSSoC 2021, I would like to contribute to this issue. Could you assign me this? I will implement it in C++

Hello, I am from GSSoC 2021, I would like work on  this issue. Could you assign me this? I will implement it in python

I am a GSSOC'2021 participant. Please assign me this work.I am good at C++,C,Java and python.Also good at algorithm and data structures.I will write programmes with detailed explanation and proper commenting.

I am a GSSOC'S participant and would like to work on the issue .
I have a good understanding about C and C++ language , thus want to implement hare and tortoise algo in C++ language.
The code will be properly indented with proper comments and description.

I am a GSSoC'21 participant.
I assure you I can handle this issue as I am extremely good at DSA, problem-solving, and C++.
It would be really great if you assign this issue to me.
Hoping for a good outcome from your side.

I'm a GSSoC'21 participant and I would like to work on this issue in Java language. Please assign it to me.

Hey, I'm a GSSOC'21 participant. I would like to work on this issue in C++, please assign it to me.

Hey, I'm a GSSOC'21 participant. I would like to work on this issue in C++, please assign it to me.

I am a GSSOC'2021 participant. Please assign me this work.I am good at C++,C,Java and python.Also good at algorithm and data structures.I will write programmes with detailed explanation and proper commenting.

do it in python


feat(config): add Namron 4512757

Added config file for Namron zwave thermostat 4512757

@kjellml80 did you intend to close this PR?

Doing Format Document in VSCode (we recommend this for a reason) should fix one of those.
The other check fails because the resulting config file isn't valid. You're defining tons of partial parameters for parameter number 3 - I don't think this is intended.
For example these should just be "#": "4" and "#": "8", according to the manual:

The other numbers have to be converted from hexadecimal (0x...) to normal decimal numbers though. We only use hexadecimal for the partial parameter bitmasks.

Besides that, it doesn't look too bad. I can give you a better review when you're done with that.

Thanks for the pointers. That made me understand the paramInformation part.

The file is semantically correct, so now we can look at consistency with other files. I'll highlight some things in a review.


Openstack Logs in Grafana

This PR contains the dashboard that when combined with the right KA patches, should give you logs in the openstack dashboard

WIP

Is it WIP?

WIP

Is it WIP?

@markgoddard Updated

This PR contains the dashboard that when combined with the right KA patches, should give you logs in the openstack dashboard

Do we have these patches in our antelope fork? What would happen in the dashboard if grafana had been deployed without them?

@technowhizz Is this ready to go in yet? If not, could you link the changes that still need to merge?

I tried testing this, but I get an empty panel:

It could be a config error with my env, but I can view logs via the the OpenSearch plugin for Grafana.
This is on master / latest Grafana OSS release. That could also be related.

This has been fixed now @dougszumski

This is still blocked on the Kolla upstream commit - we need to be better at chasing these on a regular basis

The tweak looks good - works for me now. Thanks

I think this can merge now

@dougszumski @technowhizz can we still merge this?


Disable noobaa management on client for dev preview

Disable env var default return value to false in order to not force noobaa management on the client while feature is in dev preview

/lgtm

/lgtm

/cherry-pick release-4.17

/cherry-pick release-4.17


HDDS-2593 - DatanodeAdminMonitor should track under replicated containers and complete the admin workflow accordingly

What changes were proposed in this pull request?
This PR builds on HDDS-2459 and chances the DatanodeAdminMonitor to use the newly provided ContainerReplicaCount object to track the under replicated containers on a node and move the node to the next stages in the decommission / maintenance workflow when they are all sufficiently replicated.
Once the replication is completed, the node will be moved to either complete (ending the decommission process) or go into a pending state waiting on maintenance to complete.
This PR is a significant refactor of the DatanodeAdminMonitor, but previously it was incomplete so the bulk of the code is new.
There are also a few minor refactors to other classes:

Removed the isHealthy() method from ReplicationManager and put it into ContainerReplicaCounts.
Pulled SimpleMockNodeManager into its own class (previously an inner class in TestReplicationManager) and enhanced it somewhat. HDDS-2673 is raised to consider merging it with MockNodeManager
ContainerReplicaCount now also takes a container object inorder to implement the isHealthy method.
TestDatanodeAdminMonitor now mocks all dependencies to allow for simpler and faster tests, avoiding MiniCluster.

Note that these changes do not consider the new commands which must be sent to the Datanodes to change their admin state as part of HDDS-2592. Those enhancement will be made as part of that Jira.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-2593
How was this patch tested?
Various new unit tests have been added.

Single failing unit test is:
[ERROR] testMultipleBlockAllocationWithClosedContainer(org.apache.hadoop.hdds.scm.block.TestBlockManager)  Time elapsed: 2.7 s  <<< ERROR!
org.apache.hadoop.hdds.scm.pipeline.InsufficientDatanodesException: Cannot create pipeline of factor 3 using 2 nodes.
	at org.apache.hadoop.hdds.scm.pipeline.RatisPipelineProvider.create(RatisPipelineProvider.java:153)
	at org.apache.hadoop.hdds.scm.pipeline.PipelineFactory.create(PipelineFactory.java:58)
	at org.apache.hadoop.hdds.scm.pipeline.SCMPipelineManager.createPipeline(SCMPipelineManager.java:155)
	at org.apache.hadoop.hdds.scm.block.TestBlockManager.testMultipleBlockAllocationWithClosedContainer(TestBlockManager.java:306)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)

Which is fixed by HDDS-2646 and 6c1a9ff05 (Addendum HDDS-2646) on master.

The unit test failure is still this one, unrelated to the changes I believe:
[ERROR] testMultipleBlockAllocationWithClosedContainer(org.apache.hadoop.hdds.scm.block.TestBlockManager)  Time elapsed: 2.248 s  <<< ERROR!
org.apache.hadoop.hdds.scm.pipeline.InsufficientDatanodesException: Cannot create pipeline of factor 3 using 2 nodes.
	at org.apache.hadoop.hdds.scm.pipeline.RatisPipelineProvider.create(RatisPipelineProvider.java:153)
	at org.apache.hadoop.hdds.scm.pipeline.PipelineFactory.create(PipelineFactory.java:58)
	at org.apache.hadoop.hdds.scm.pipeline.SCMPipelineManager.createPipeline(SCMPipelineManager.java:155)
	at org.apache.hadoop.hdds.scm.block.TestBlockManager.testMultipleBlockAllocationWithClosedContainer(TestBlockManager.java:306)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:47)
	at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
	at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:44)```

It should be fixed when we next merge master into this branch, as I believe that test has been fixed on master now.

I agree there is a chance we could move the calculation into replication manager and track the metrics of under-replicated blocks in the NodeManager, but for now I think we should move forward and commit this and once we have a working decommission solution we can look to optimised and refactor it further.

If you are happy can you please go ahead and commit this change?

That test is still failing event after merging master to the branch. I though these commits would have fixed it, but I was confusing them with something else:
Addendum HDDS-2646. Start acceptance tests only if at least one THREE pipeline is available (#282)
HDDS-2646. Start acceptance tests only if at least one THREE pipeline is available (#282)

I think this is a problem with the MockNodeManager handing the new operational states on this branch. I will look at it further.

This is committed in af2efa54c2fa3c65050320afd7f9108f08a53ffb to the HDDS-1880-Decom feature branch, but PR would be closed automatically (due to Closes #... commit message) by GitHub only when the commit reaches master.  Can we close it manually?

This is committed in af2efa5 to the HDDS-1880-Decom feature branch, but PR would be closed automatically (due to Closes #... commit message) by GitHub only when the commit reaches master. Can we close it manually?

It is fine with me to close this one. I think @elek merges the PRs manually from the CLI rather than using the button here in github, which is possibly why this PR did not get closed automatically.


Switch from travis to github workflows

This PR replaces travis with a github workflow
It includes the same features compared to travis except testing the GDAL master branch (I did not figure out how to mark a job with allow_failure). But as the GDAL master job is typically not cached and thus the most time/resource consuming, it would probably be removed from travis anyway.
The action is designed to emulate a travis environment. Thus, no changes to the proj/gdal install script were necessary. Also, this would allow to easily switch back to travis.
The github workflows only work once this PR is merged.
The workflow in action can be seen here: https://github.com/rbuffat/Fiona/actions/runs/374781042

I did not figure out how to mark a job with allow_failure

See eg here how we did this for pygeos: https://github.com/pygeos/pygeos/blob/62ddf67cf623c6f66da3cc9b3f941763b810a664/.github/workflows/test-linux.yml#L90

@jorisvandenbossche Thanks for the template, this helped a lot!

@sgillies The github action seem to still work. One of the limitations of github actions is that the cache is only available for one week:

A repository can have up to 5GB of caches. Once the 5GB limit is reached, older caches will be evicted based on when the cache was last accessed. Caches that are not accessed within the last week will also be evicted.

https://github.com/actions/cache
An alternative to caching gdal would be to use docker container. Gdal seems to provide a range of different docker images already: https://github.com/OSGeo/gdal/tree/master/gdal/docker It should be possible to create tailored containers for our CI based on these containers. But for now, it is probably better to have a working CI as soon as possible.

@sgillies The github action seem to still work. One of the limitations of github actions is that the cache is only available for one week:

A repository can have up to 5GB of caches. Once the 5GB limit is reached, older caches will be evicted based on when the cache was last accessed. Caches that are not accessed within the last week will also be evicted.

https://github.com/actions/cache
An alternative to caching gdal would be to use docker container. Gdal seems to provide a range of different docker images already: https://github.com/OSGeo/gdal/tree/master/gdal/docker It should be possible to create tailored containers for our CI based on these containers. But for now, it is probably better to have a working CI as soon as possible.

@rbuffat thank you!

@rbuffat thank you!


Added String Category and Single Number for Array

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ x] New algorithm
[ ] Optimization in previous algorithms
[ ] Code style update
[ ] Refactor
[ ] Documentation
[ ] Other, please describe:

Briefly describe the changes in this PR
Added String Category and Single Number for Array

Hi @canhuynh1998,
Does this have an issue opened?

Hi @ming-tsai, sorry for the confusion. I think this is the #420


Allow inherit_from to accept a glob

This PR allows inherit_from to accept a glob.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Benchmarking
I ran benchmarks on rails to determine if this change causes a performance regression.
10x bundle exec rubocop --show-cop=Style/EmptyMethod
This reflects the basic startup time of Rubocop to process configurations and resolve inheritance.

I'm wondering if in some cases the order in which the extra files get required won't be an issue. Any thoughts on this?

@alexevanczuk The CI's failing because a few lines are now too long.

@alexevanczuk The CI's failing because a few lines are now too long.

@bbatsov Thanks! Fixed. I'm having trouble seeing the results from CI. When I click the ❌ next to CircleCI Pipeline I just see:
Could not find a usable config.yml, you may have revoked the CircleCI OAuth app.
Please sign out of CircleCI and log back in with your VCS before triggering a new pipeline.

Let me know if there's something I should be doing differently!

Here's what we got from the CircleCI support team.

Contributors who get this error need to re-authenticate with CircleCI because their GitHub OAuth token has expired.
https://support.circleci.com/hc/en-us/articles/360051228052-How-to-Perform-a-Re-Authentication

Here's what we got from the CircleCI support team.

Contributors who get this error need to re-authenticate with CircleCI because their GitHub OAuth token has expired.
https://support.circleci.com/hc/en-us/articles/360051228052-How-to-Perform-a-Re-Authentication

Thanks @bbatsov . I reauthorized on circle CI and made some good progress. Now I'm just seeing...

A failure for cc-upload-coverage:

Error: response from https://api.codeclimate.com/v1/test_reports.
HTTP 401: You are not authorized for this action
Usage:
  cc-test-reporter upload-coverage [flags]

Some workflows that need to be approved it look like

Seems we're in business now. Thanks!


composer ? and what is your suggestion for the database ?

I'm a new student in the web developer field and I need more help. Thank you very much
I have an error message for php artisan... should I install composer? Another question is how (yarn) can install or work.

Hi @aschhkan
Did you read this section in the readme before? https://github.com/guillaumebriday/laravel-blog#installation
I think you can get everything you need to install the project on your own machine 👍


Couldn't right click on empty space - Bug

Hey, @duke7553 I couldn't right click on the empty spaces. I can right click on the files but not on the empty spaces.

Fixed


Check for location existence

Fixes https://github.com/HubSpot/PlanOut.js/issues/8
Actual change: https://github.com/HubSpot/PlanOut.js/pull/9/files#diff-b07eff5971fb3611f37107522ba01edaR12

This latest version should now be on npm, so you should be able to use PlanOut.js in node now. Thanks again for this.


Enable orchestration message override

In previous versions of MIQ, it was possible to override the Service Template Provision Task failure message for orchestration tasks from within Automate.
The normalized_status function in app/models/orchestration_stack/status.rb overrides any failure message set in Automate, with 'Stack creation failed'.
This PR allows for any message other that 'Service_Template_Provisioning failed' to be set.

/cc @bzwei

@supernoodz app/models/orchestration_stack/status.rb is backend code, it should not handle messages from automate.
Can you explain with more details concerning what have been changed when you said "In previous version of MIQ, it was possible to override ... within Automate"? What is the difference before and after?
If you want to override the message in automate you can do in update_serviceprovision_status.rb automate script. Pending #12342 enhances the showing of error message. Does this satisfy your need? If not, this is the place where you can have your enhancement.
Finally, "Service_Template_Provisioning failed" should be an indication that the error comes from automate, while other error messages come from the provider.

@bzwei I have found for Cloud Orchestration service provisioning tasks that I can no longer set the message in update_serviceprovision_status.rb, as any message value is being overridden by the normalized_status function. Previously, this was possible.

@supernoodz can you post the code how you set the message in update_serviceprovision_status.rb? I don't know any changes made in this area recently. @tinaafitz @billfitzgerald0120 any idea?
Is there any reason you want to change Service_Template_Provisioning failed to Stack creation failed?

@bzwei  This is the original scenario [1]. I'm checking for Azure stack errors in check_provisioned, using set_state_var to store the error and then setting prov.message in update_serviceprovision_status.
Service_Template_Provisioning failed seems to be the standard failure message, so in this case use the default normalized_status function behaviour. Otherwise, use the custom stack error.
[1] http://www.tigeriq.co/2016/02/17/detecting-azure-orchestration-errors-with-cloudforms/

@supernoodz We recently had an enhancement to show properly the error message from Azure provisioning. Hopefully your posted extra work is no longer needed. Please give it a try #12064

@bzwei Tested #12064 and error message looks good, thanks. Closing PR.


lvm tests - do not merge

Signed-off-by: Thomas Parrott<EMAIL_ADDRESS>

Testsuite passed


Inline Comments

GFM Allows inline HTML-Style comments, but these are not honored by Hyperbook.
Oh, and Blockquotes aren't rendered properly: https://hyperbook.openpatch.org/markdown-reference#blockquotes

I  think GFM does not have anything specific for comments, but GitHib for example allows to use raw HTML code in markdown. Currently in hyperbook this is not allowed. There are three options in my opionon:

allow HTML code
create a new directive for comments

I am not keen on allowing HTML code since I will not have control over the style and can not ensure custom HTML in the future, because I do not know what is used out there.
Directives for inline and block comments might therefore be better. They also have another benefit, I can just ignore them when the hyperbook is build. Thus comments live only in the raw markdown files, but not in the HTML files.

I will take a look at blockqoutes.

I thought about it a bit more and search how markdown comments can be handled. I think the best "native" way is described here: https://www.jamestharpe.com/markdown-comments/ and looks like this:
[]: # (This is a comment)
[]: # "And this is a comment"
[]: # 'Also this is a comment'
[//]: # (Yet another comment)
[comment]: # (Still another comment)

For the moment, I would suggest using this, since it is compatible with all markdown flavors and does not require a new syntax element. I will add a paragraph about comments to the documentation.

interesting, thank you :)


elm-format autosave didn't work for me

caveat: I can live without it. just reporting this with the assumption that you'd want to know.
I got this error whenever I attempt to save a file:
Error detected while processing function elm#util#EchoStored:
line    2:
E488: Trailing characters

my guess is that the problem may come from the fact that I didn't install elm with -g. I'm just very skittish about installing anything globally. but, if that doesn't explain it, I'm on vim 7.3, and the relevant section of my .vimrc looked like this:
" elm-format setup; from https://github.com/avh4/elm-format
call plug#begin('~/.vim/plugged')
Plug 'elmcast/elm-vim'
call plug#end()
let g:elm_format_autosave = 1

This looks like a bug in something I merged today. Thanks for the report. What operating system are you on?

god, this is so embarassing. OS X 10.8.5, vim --version gives me: VIM - Vi IMproved 7.3 (2010 Aug 15, compiled Oct 23 2012 13:50:52). I hate upgrading and I've got some graphics stuff on this machine I really like.

Seeing the same issue.
OS X 10.11.6
VIM 7.3
No issue running :ElmFormat directly. Error appears when writing and formatting automatically on save via let g:elm_format_autosave = 1.
Disabling format on write via let g:elm_format_autosave = 0 stops the error. Commenting out the following line in autoload/elm/util.vim also stops the error:
call function('elm#util#' . s:echo_func_name)(s:echo_title, s:echo_msg)

I got this today "error why processing Elm#format".
Removed elm-vim and :PlugInstall.

Again the same problem. Goting with
let g:elm_format_autosave = 0
autocmd BufWritePost *.elm silent! !elm-format --yes %:p


Splitview divider is white?

There is a white vertical divider between master and detail views right now, any idea what caused this? Looks weird.

Bug Report Dump (Auto-generated)

Version 1.19.0<PHONE_NUMBER>)
Device: iPad7,3 (iOS 11.3)
TestFlight: false

Only when collapsed is safer in that case I suppose.
Sent with GitHawk

@BasThomas think this is worth resubmitting 1.19 for?
Sent with GitHawk

We can do a followup 1.19.1 maybe?
Sent with GitHawk

Fixed in https://github.com/GitHawkApp/GitHawk/pull/1989


fix #215

I fixed the network error issue. If a user choose wrong network, it automatically ask user to change network.

I am not sure about why asking user to switch a network is sort of an issue. In general, it is needed to let users know which network our platform runs on. Another reason what I am thinking is that we just deployed smart contracts on ethereum network. so we don't want users to interact with other networks except mainnet. Would you @21CenturyTarzan give us more context/reason about your pull request like how does the error will generate user compliants?
@pavlovcik how do you think of this pr?

Hi, I removed the hardcoded part by using .env file.

@21CenturyTarzan We will continue PR review next week.

looks good to me @21CenturyTarzan . by the way, would you create a pr to development branch, not the main branch?

looks good to me @21CenturyTarzan . by the way, would you create a pull request for the development branch, not the main branch?

fyi you should be able to do this

@21CenturyTarzan why did you make the mainnet chain id a variable? It should never change from "1"

Ok. I will try to create pull request for the development branch.

@21CenturyTarzan why did you make the mainnet chain id a variable? It should never change from 1

The dapp should be able to run on any networks except mainnet. so hardcoded values need to be removed.

@21CenturyTarzan why did you make the mainnet chain id a variable? It should never change from 1

The dapp should be able to run on any networks except mainnet. so hardcoded values need to be removed.

The only plans for official deployments currently are Mainnet (1) and Optimism (10)

the wrong network button hasn't been added yet, currently it just results in a black screen

we didn't get any response from @21CenturyTarzan. but we can make the simple bounty for this @pavlovcik


⚠ Bofrost.lu has degraded performance

In d902ea2, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1138 ms

Resolved: Bofrost.lu performance has improved in 1fc7f5a after 13 minutes.


Provide simplified approach for different Azure clouds

Provide simplified approach for different Azure clouds similar to .NET SDK where you can define the type of cloud and it will return the respective endpoints for authentication, etc.
We want to support storage for all clouds and are now using endpoint suffixes but ideally we just define the type of cloud and it resolves the endpoint suffix for us.

Relates to https://github.com/kedacore/keda/pull/1863

Found it in Azure/go-autorest: https://github.com/Azure/go-autorest/blob/master/autorest/azure/environments.go

That'd exactly what I meant, thanks!

@tomkerkhove thanks for reaching out!
As far as I know, all the Storage SDKs are consistent in the way that it accepts an endpoint explicitly.
Is the package posted by @amirschw sufficient for your use case? If yes please close this issue, thanks!

We went that route, thanks.


License?

What's the license on this? Please add a license.

Added LICENSE file. This is licensed under the MIT License. Thanks for the reminder!
--Matt


Ideation

Here I will brainstorm and plan my ideas for my RPG project.

Okay, first of all, let's think of what I want to include in my RPG.
-Character for the user
-main plot/quest

Better IDEA YAYYYYY ^^

First you start off by doing a wholesome innocent task. Such as collecting flowers in a a field, to give to your mom as a birthday gift. However, when you collect all flowers you get jumpscared by a smiling dark flower that makes a sudden loud noise.

This is what I want the game to be like:

It starts off with an intro such as:
What a beautiful day it is today ^^ But oh no!  It's your sisters birthday and you completely forgot to get get her a present! Hurry up and collect the flowers for your sister's

okay, MEIN SCHEIß teaMmItGLIeD hat kB darauf (fettes L aber na ja), deswegen überlege ich mir jz was anderes (hab auch schon eine Idee.

I was planning on programming a game such as Tomb of the Mask.

While I would really like to a program a game like this, there are still some changes I would make, including:
-Changing the navigation to wasd, as I really like using those keys, when playing games
-Set the view of the maze to a certain point so that it doesn't move when playing the game
-change the characters (aka the yellow thingy you play as and the monsters)
Of course, over time I might find more things I'd like to change.


LookupError when calculating gradient of gradient with RNN on GPU

System information

Have I written custom code (as opposed to using a stock
example script provided in TensorFlow): yes
OS Platform and Distribution (e.g.,
Linux Ubuntu 16.04): Ubuntu 18.04.3 LTS
TensorFlow installed from (source or
binary): binary (Anaconda/PIP)
TensorFlow version (use command below):
GIT_VERSION v1.12.1-25080-gca585e7
VERSION  2.2.0-dev20200218
Python version: 3.6.10
CUDA/cuDNN version: CUDA 10.2, cuDNN 7.6.2
GPU model and memory: GeForce RTX 2080 Ti, 12GB VRAM

Describe the current behavior
When trying to implement a gradient penalty for a WGAN-GP, which requires to calculate a gradient of a tensor, which itself depends on a gradient, the program terminates with a LookupError and tells me

LookupError: gradient registry has no entry for: CudnnRNNBackprop

This occurs only on the GPU version of Tensorflow and only if a recurrent layer is used (GRU or LSTM, doesn't matter). On the CPU, the provided minimum working example runs as expected. The error does not occur when training a "normal" model with a recurrent layer on the GPU (no gradient penalty).
I tried with tensorflow-gpu 2.0 (installed via conda), tensorflow-gpu 2.1 (installed via pip) and tf-nightly-gpu 2.2 (more specific: 2.2.0-dev20200218) (installed via pip). The error is the same for all GPU versions.
The operation works with tensorflow-gpu, if the parameter unroll of the GRU-Layer is set to True. Since this disables the usage of the cuDNN implementation of the GRU-Layer (deviation from default parameters), this does not solve the actual problem, but might indicate that there is just a small bug in the interface to cuDNN.
Standalone code to reproduce the issue
# this code does not make much sense, but is shorter than providing a full optimization loop for a WGAN_GP and produces the same error
import tensorflow as tf
import tensorflow.keras as k
import tensorflow.keras.layers as kl
import numpy as np

physical_devices = tf.config.experimental.list_physical_devices('GPU')
tf.config.experimental.set_memory_growth(physical_devices[0], True)

def gradient_penalty(model, input_data):
    # get gradient
    input_data = tf.convert_to_tensor(input_data)
    with tf.GradientTape() as t:
        t.watch(input_data)
        pred = model(input_data)
    grad = t.gradient(pred, [input_data])[0]
    # define gradient penalty
    slopes = tf.sqrt(tf.reduce_sum(tf.square(grad), axis=[1, 2]))
    gp = tf.reduce_mean((slopes - 1.) ** 2)
    return gp

if __name__ == "__main__":
    # model with recurrent layer
    model = k.Sequential([kl.InputLayer(input_shape=(50, 20)), kl.GRU(100), kl.Dense(1)])
    # Optimizer
    opt = tf.optimizers.Adam()
    # Dummy data
    data = np.random.normal(0, 1, (8, 50, 20)).astype(np.float32)
    # Optimize
    with tf.GradientTape() as tape:
        gp = gradient_penalty(model=model, input_data=data)
    grad = tape.gradient(gp, model.trainable_variables)
    opt.apply_gradients(zip(grad, model.trainable_variables))

Other info / logs
Full log:
/home/hendrik/anaconda3/envs/MLS22/bin/python /home/hendrik/PycharmProjects/GAN/MinimumMinimumWorkingExample.py
2020-02-26 13:30:08.149540: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcuda.so.1
2020-02-26 13:30:08.172329: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-02-26 13:30:08.172601: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1558] Found device 0 with properties:
pciBusID: 0000:01:00.0 name: GeForce RTX 2080 Ti computeCapability: 7.5
coreClock: 1.545GHz coreCount: 68 deviceMemorySize: 10.76GiB deviceMemoryBandwidth: 573.69GiB/s
2020-02-26 13:30:08.172716: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudart.so.10.1
2020-02-26 13:30:08.173554: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcublas.so.10
2020-02-26 13:30:08.174413: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcufft.so.10
2020-02-26 13:30:08.174551: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcurand.so.10
2020-02-26 13:30:08.175425: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcusolver.so.10
2020-02-26 13:30:08.175952: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcusparse.so.10
2020-02-26 13:30:08.177916: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudnn.so.7
2020-02-26 13:30:08.177978: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-02-26 13:30:08.178270: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-02-26 13:30:08.178512: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1700] Adding visible gpu devices: 0
2020-02-26 13:30:08.187368: I tensorflow/core/platform/cpu_feature_guard.cc:143] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
2020-02-26 13:30:08.208283: I tensorflow/core/platform/profile_utils/cpu_utils.cc:102] CPU Frequency:<PHONE_NUMBER> Hz
2020-02-26 13:30:08.208513: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x55cac97d38c0 initialized for platform Host (this does not guarantee that XLA will be used). Devices:
2020-02-26 13:30:08.208523: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Host, Default Version
2020-02-26 13:30:08.267483: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-02-26 13:30:08.267804: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x55cac97f6830 initialized for platform CUDA (this does not guarantee that XLA will be used). Devices:
2020-02-26 13:30:08.267814: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): GeForce RTX 2080 Ti, Compute Capability 7.5
2020-02-26 13:30:08.267915: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-02-26 13:30:08.268226: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1558] Found device 0 with properties:
pciBusID: 0000:01:00.0 name: GeForce RTX 2080 Ti computeCapability: 7.5
coreClock: 1.545GHz coreCount: 68 deviceMemorySize: 10.76GiB deviceMemoryBandwidth: 573.69GiB/s
2020-02-26 13:30:08.268246: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudart.so.10.1
2020-02-26 13:30:08.268253: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcublas.so.10
2020-02-26 13:30:08.268259: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcufft.so.10
2020-02-26 13:30:08.268265: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcurand.so.10
2020-02-26 13:30:08.268271: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcusolver.so.10
2020-02-26 13:30:08.268276: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcusparse.so.10
2020-02-26 13:30:08.268282: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudnn.so.7
2020-02-26 13:30:08.268308: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-02-26 13:30:08.268554: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-02-26 13:30:08.268778: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1700] Adding visible gpu devices: 0
2020-02-26 13:30:08.268795: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudart.so.10.1
2020-02-26 13:30:08.269304: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1099] Device interconnect StreamExecutor with strength 1 edge matrix:
2020-02-26 13:30:08.269310: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1105]      0
2020-02-26 13:30:08.269314: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1118] 0:   N
2020-02-26 13:30:08.269362: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-02-26 13:30:08.269613: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-02-26 13:30:08.269856: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1244] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 9813 MB memory) -> physical GPU (device: 0, name: GeForce RTX 2080 Ti, pci bus id: 0000:01:00.0, compute capability: 7.5)
2020-02-26 13:30:08.667776: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudnn.so.7
2020-02-26 13:30:09.284739: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcublas.so.10
Traceback (most recent call last):
  File "/home/hendrik/PycharmProjects/GAN/MinimumMinimumWorkingExample.py", line 31, in <module>
    grad = tape.gradient(gp, model.trainable_variables)
  File "/home/hendrik/anaconda3/envs/MLS22/lib/python3.6/site-packages/tensorflow/python/eager/backprop.py", line 1048, in gradient
    unconnected_gradients=unconnected_gradients)
  File "/home/hendrik/anaconda3/envs/MLS22/lib/python3.6/site-packages/tensorflow/python/eager/imperative_grad.py", line 77, in imperative_grad
    compat.as_str(unconnected_gradients.value))
  File "/home/hendrik/anaconda3/envs/MLS22/lib/python3.6/site-packages/tensorflow/python/eager/backprop.py", line 145, in _gradient_function
    grad_fn = ops._gradient_registry.lookup(op_name)  # pylint: disable=protected-access
  File "/home/hendrik/anaconda3/envs/MLS22/lib/python3.6/site-packages/tensorflow/python/framework/registry.py", line 97, in lookup
    "%s registry has no entry for: %s" % (self._name, name))
LookupError: gradient registry has no entry for: CudnnRNNBackprop

Process finished with exit code 1

Thanks in advance for your time!

I have tried on colab with TF version 2.2.0-dev20200226 and was able to reproduce the issue.Please, find the gist here. Thanks!

Hi @HendrikLaux, Thanks for reporting the issue.
The root cause is that cudnnRNN kernel doesn't have higher order gradient support, which is the issue you are hitting. I am not sure if Nvidia is going to add high order gradient support the cudnn kernel, for now we can only work around the issue (eg use unroll=True).

Note that unroll requite the input to have a static time sequence. If you want to build your model with dynamic time sequence length, you can disable the cudnn backend by:
gru_layer = tf.keras.layers.GRU(10)
gru_layer._could_use_gpu_kernel = False.

Hey qlzh727,
thank you for the clarification of this issue! Do you think the issue is worth passing to Nvidia, or is the problem too minor and the use cases too limited for this?

@houtoms from Nvidia for this.

@houtoms @qlzh727
Any updates on this issue? :slightly_smiling_face:

Thanks for the update. Let me sync with our team and put some update here.

@HendrikLaux We don't have it on our plan in the near term to support higher order gradient RNN. But I can help issue a RFE if you think this is needed because, for example, the performance of it is the bottleneck.

This support would be very nice to have.


Manual approval required for workflow run<PHONE_NUMBER>: Aprove of release on Prod environment

Please approve or deny the deployment of the relaese version to the Prod environment.
Workflow is pending manual review.
URL: https://github.com/Water-XP-Scrumban-Fall/BigProject/actions/runs/6264654985
Required approvers: [DrShadowPLay]
Respond "approved", "approve", "lgtm", "yes" to continue workflow or "denied", "deny", "no" to cancel.

yes

All approvers have approved, continuing workflow and closing this issue.


Adding some more utility python script

Adding some home grown python scripts will continue to add more scripts

@geekcomputers  can you please review my pull request and if possible please merge it


Can't get firefox version >47 to work in travis

So frist off, it works great locally, no issues. But... I've had some trouble getting firefox version >47 to load the profile at all in travis. I have tried saucelabs, and the same occurred. Any help would greatly be appreciated.
 Thanks!

Due to https://wiki.mozilla.org/Add-ons/Extension_Signing#Timeline
Unfortunately latest dev build and nightly is running into: https://github.com/mozilla/geckodriver/issues/647 for me
So I went with an unbranded build, which has the xpinstall.signatures.required setting available.
And of course neither the travis firefox addon nor saucelabs supports these unbranded builds...
Hacked around it with the following (link cribbed from here):
before_install:
  - "wget -O /tmp/firefox-unbranded.tar.bz2 http://archive.mozilla.org/pub/firefox/tinderbox-builds/mozilla-release-linux64-add-on-devel/1488485271/firefox-52.0.en-US.linux-x86_64-add-on-devel.tar.bz2"
  - "tar xf /tmp/firefox-unbranded.tar.bz2 -C /tmp"
- "export PATH=/tmp/firefox:$PATH"

Not an issue with this project afterall. And it is quite useful. Thanks!


MiPay does not return proper transaction data

The raw response string contains no values for all parameters except the transaction ref which is copied back. An empty transaction ID is hence obtained by the library. This may be a blocker for client applications towards functionality.

@drenther I feel this issue can be closed. To the best I understand the intent and the UPI specs, this behaviour of the MiPay app is not due to any parameters sent in the intent call. It's not a forensic idea though; but just based on the behaviour of several apps covered in my testing.