White output

Hi guys,
As a default Mac console color settings user, I find very sad that elvis prints a lot of output in white :)
Is there a way to switch to default, or at least non-background color? I can submit a pull request if we settle on some setup. (Frankly speaking, I'd make as few colors as possible and just use proper formatting that is readable enough).
Thanks!

@doubleyou we would definitely appreciate a PR with some sort of output_format :: color | plain parameter added to elvis' configuration. You can discuss with @jfacorro where should that go exactly, most likely somewhere in the elvis.config file

@doubleyou I think it's a great idea to make the output a happier, colorful output :). Do you have an example on what parts of the output you would add colors to?

@jfacorro I'm pretty sure he wants to remove the colors, not add new ones.

@elbrujohalcon I see. The default background color for OS X terminal is white and elvis is printing most text in white.
@doubleyou All output is printed using the elvis_utils:print/2 function which in turn calls elvis_utils:parse_colors/1. In print one could check the configuration value output_format and if plain then remove all {{.*}} in the message instead of calling parse_colors and replacing them with the corresponding console color format.

@jfacorro We could also make a customizable color scheme I guess, something like
colors => {
    success => green_bold,
    failure => red_bold,
    file_path => terminal_default,
    ...
}

And maybe a global switch that defaults to
use_colors => true

@doubleyou that's a good idea, but I always recommend not using boolean switches, so... instead of use_colors :: boolean() I suggested output_format :: color | plain. It looks more declarative, IMHO.

Yeah, sounds good. Expect a PR from me soon then :)

@doubleyou any progress on this front? Should we still expect a PR from you?


Fixing wrong stubs on \MongoCollection

\MongoCollection::findOne() has a third parameter options (check docs).

May be you can add the contents of the "Currently supported options are:..." table? )

Any recommendations about the format ?
In the same file I found 3 different formats :

options in \MongoCollection::insert() : HTML format (dl, dd, dt)
options in \MongoCollection::remove() : HTML format (ul, li)
options in \MongoCollection::update() : plain text

Which one should I choose ?


[Task]: Integrations pod: Create pod history page

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks
As the team keeps growing and adding new members, while the older members may or may not remain with the team, it is important the team retains reasoning around any key decision that it made in the past e.g. why it chose to go with complexity based estimation as opposed to the time based estimation, why it chose to keep 2 week sprints instead of 1 week, why it chose to include items of 0.2 complexity etc.

https://www.notion.so/appsmith/Pod-Practices-1d9f33150dab4f72ab35788a355aaa56


#167510296 Fixes token error in authentication middleware

What does this PR do?
It resolves a token error in authentication middleware
Description of the task to be completed

Remove split function in the token verification method.

Relevant PT Stories:

Fix token error

How this can be tested

Clone the repository
Run npm install to install dependences
Run npm start and proceed to localhost://PORT/api/v1/houses and supply the token and required information

Screenshots
N/A

You can merge it!


[status_check] Unhandled exception when calling a custom check with syntax errors

**Output of the info page **
sudo service datadog-agent info
2017-08-09 15:31:26,268 | WARNING | dd.collector | utils.dockerutil(dockerutil.py:94) | Error while detecting orchestrator: Error while fetching server API version: ('Connection aborted.', error(2, 'No such file or directory'))
2017-08-09 15:31:26,304 | ERROR | dd.collector | collector(agent.py:610) | Uncaught error running the Agent
Traceback (most recent call last):
  File "/opt/datadog-agent/agent/agent.py", line 606, in <module>
    sys.exit(main())
  File "/opt/datadog-agent/agent/agent.py", line 532, in main
    return Agent.info(verbose=options.verbose)
  File "/opt/datadog-agent/agent/agent.py", line 219, in info
    return CollectorStatus.print_latest_status(verbose=verbose)
  File "/opt/datadog-agent/agent/checks/check_status.py", line 282, in print_latest_status
    message = module_status.render()
  File "/opt/datadog-agent/agent/checks/check_status.py", line 180, in render
    ] + ["", ""]
  File "/opt/datadog-agent/agent/checks/check_status.py", line 546, in body_lines
    '  ' + '-' * (len(cs.name) + 3 + len(cs.check_version))
TypeError: object of type 'NoneType' has no len()
Traceback (most recent call last):
  File "/opt/datadog-agent/agent/agent.py", line 606, in <module>
    sys.exit(main())
  File "/opt/datadog-agent/agent/agent.py", line 532, in main
    return Agent.info(verbose=options.verbose)
  File "/opt/datadog-agent/agent/agent.py", line 219, in info
    return CollectorStatus.print_latest_status(verbose=verbose)
  File "/opt/datadog-agent/agent/checks/check_status.py", line 282, in print_latest_status
    message = module_status.render()
  File "/opt/datadog-agent/agent/checks/check_status.py", line 180, in render
    ] + ["", ""]
  File "/opt/datadog-agent/agent/checks/check_status.py", line 546, in body_lines
    '  ' + '-' * (len(cs.name) + 3 + len(cs.check_version))
TypeError: object of type 'NoneType' has no len()
====================
Dogstatsd (v 5.16.0)
====================

  Status date: 2017-08-09 15:31:18 (9s ago)
  Pid: 14984
  Platform: Linux-4.4.0-81-generic-x86_64-with-Ubuntu-16.04-xenial
  Python Version: 2.7.13, 64bit
  Logs: <stderr>, /var/log/datadog/dogstatsd.log, syslog:/dev/log

  Flush count: 0
  Packet Count: 0
  Packets per second: 0
  Metric count: 0
  Event count: 0
  Service check count: 0

====================
Forwarder (v 5.16.0)
====================

  Status date: 2017-08-09 15:31:27 (0s ago)
  Pid: 14983
  Platform: Linux-4.4.0-81-generic-x86_64-with-Ubuntu-16.04-xenial
  Python Version: 2.7.13, 64bit
  Logs: <stderr>, /var/log/datadog/forwarder.log, syslog:/dev/log

  Queue Size: 606 bytes
  Queue Length: 1
  Flush Count: 3
  Transactions received: 2
  Transactions flushed: 1
  Transactions rejected: 0
  API Key Status: API Key is valid

======================
Trace Agent (v 5.16.0)
======================

  Pid: 14982
  Uptime: 13 seconds
  Mem alloc: 1028744 bytes

  Hostname: [redacted]
  Receiver: localhost:8126
  API Endpoint: https://trace.agent.datadoghq.com

  Bytes received (1 min): 0
  Traces received (1 min): 0
  Spans received (1 min): 0

  Bytes sent (1 min): 0
  Traces sent (1 min): 0
  Stats sent (1 min): 0

Additional environment details (Operating System, Cloud provider, etc):
4.4.0-81-generic #104-Ubuntu SMP Wed Jun 14 08:17:06 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
VERSION="16.04.3 LTS (Xenial Xerus)"
Steps to reproduce the issue:

Write a custom check such as /etc/dd-agent/checks.d/example.py with a syntax error:

from checks import AgentCheck

class ExampleCheck(AgentCheck):
    def check(self, instance):
    self.gauge('hello.world', 1)

Create the corresponding yaml file in /etc/dd-agent/conf.d/example.yaml:

init_config:

instances:
  [{}]

sudo /etc/init.d/datadog-agent restart
sudo /etc/init.d/datadog-agent info

Describe the results you received:
The above-described error output, ultimately leading only as far as error in line 546 of check_status.py:
TypeError: object of type 'NoneType' has no len()
Describe the results you expected:
A handled exception with meaningful output that stipulates there is a syntax error in a custom check.
Ideally it either raises the root-cause custom check exception that shows the responsible line of code, or provides information which custom check triggered the exception to allow separate debugging.
Additional information you deem important (e.g. issue happens only occasionally):
While this should be encountered with all types of syntax errors, in some specific cases where checking with Python for syntax errors on one host succeeds, but it fails on another, can result in confusion about what has caused the issue, leading to many hours of "black-box" debugging that can be avoided with an indication of the custom check having a syntax error when ran by the agent.

@insideClaw man, that was one awesome issue description. Thank you so much for the breakdown and investigation 🙇
All we have to do know is fix it. I'll make sure we do so  - should be a small quick fix - and ship in the next release.

Excellent, I'm glad you found it well put.
Also that I'm not misguided of it being an issue - the Datadog support person who I spoke to originally was convincing me that's normal python behaviour :)
Good luck with the fix and I'll be looking forward to your release.

@insideClaw a fix was merged, should be available in the upcoming release, due very soon. Thanks again! 👍

 [info] unknown ver if failed module load, custom if custom check.


Mock account provider quote error

{
  id: 'a066a4ab-cbd1-4a73-b911-fadd1087dcf2',
  paymentPointerId: '3807c74e-fcdf-4f6f-a548-c626d49b602b',
  receiver: 'https://peer-backend/accounts/pfry/incoming-payments/70f567d7-1f5e-4e30-9dd1-76903b547a5e',
  sendAmount: { value: '100', assetCode: 'USD', assetScale: 2 },
  receiveAmount: { value: '99', assetCode: 'USD', assetScale: 2 },
  createdAt: '2022-09-20T22:48:27.125Z',
  expiresAt: '1970-01-01T00:00:00.000Z',
  paymentType: 'FixedSend'
}

The mock account provider's /quotes endpoint gets the following error when processing the above quote
local-fynbos-1        | RangeError: The number 1.98 cannot be converted to a BigInt because it is not an integer
local-fynbos-1        |     at BigInt (<anonymous>)
local-fynbos-1        |     at action2 (/workspace/packages/mock-account-provider/build/index.js:514:16)
local-fynbos-1        |     at processTicksAndRejections (node:internal/process/task_queues:96:5)
local-fynbos-1        |     at Object.callRouteAction (/workspace/node_modules/.pnpm/@remix-run+server-runtime@1.6.8_biqbaboplfbrettd7655fr4n2y/node_modules/@remix-run/server-runtime/dist/data.js:40:14)
local-fynbos-1        |     at handleResourceRequest (/workspace/node_modules/.pnpm/@remix-run+server-runtime@1.6.8_biqbaboplfbrettd7655fr4n2y/node_modules/@remix-run/server-runtime/dist/server.js:442:14)
local-fynbos-1        |     at requestHandler (/workspace/node_modules/.pnpm/@remix-run+server-runtime@1.6.8_biqbaboplfbrettd7655fr4n2y/node_modules/@remix-run/server-runtime/dist/server.js:42:18)
local-fynbos-1        |     at /workspace/node_modules/.pnpm/@remix-run+express@1.6.8_nawawpklwqjvxe6axdd7matlce/node_modules/@remix-run/express/dist/server.js:39:22
local-fynbos-1        | POST /quotes 500 - - 7.610 ms

related to #571


autograph=False not respected when calling get_concrete_function()

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): reproduced on Ubuntu 18.04 and MacOS 10.14
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 1.14.0  v1.14.0-rc1-22-gaf24dc91b5
Python version: 2.7.16

Describe the current behavior
When passing in tf.function(..., autograph=False), console output indicates that autograph still attempts to autograph the function.
Describe the expected behavior
That the function is not going to be autographed.
Code to reproduce the issue
This encountered when passing in a function that autograph fails on. For example, a python function that is actually a wrapper around an R function.
This example is in R:

> library(reticulate)
> library(tensorflow)
>
> Sys.setenv(AUTOGRAPH_VERBOSITY=10)
> reticulate::use_virtualenv("~/tf1")
>
> x <- tf$constant(1)
> add1 <- py_func(function() x + 1)
>
> fn <- tf$`function`(add1, autograph = FALSE)
>
> fn$get_concrete_function()
ERROR:tensorflow:Error converting <function initialize_variables at 0x134c82140>
Traceback (most recent call last):
  File "/Users/tomasz/tf1/lib/python2.7/site-packages/tensorflow/python/autograph/impl/api.py", line 524, in to_graph
    return conversion.convert(entity, program_ctx)
  File "/Users/tomasz/tf1/lib/python2.7/site-packages/tensorflow/python/autograph/impl/conversion.py", line 306, in convert
    entity, program_ctx, free_nonglobal_var_names)
  File "/Users/tomasz/tf1/lib/python2.7/site-packages/tensorflow/python/autograph/impl/conversion.py", line 229, in _convert_with_cache
    entity, program_ctx)
  File "/Users/tomasz/tf1/lib/python2.7/site-packages/tensorflow/python/autograph/impl/conversion.py", line 431, in convert_entity_to_ast
    nodes, name, entity_info = convert_func_to_ast(o, program_ctx)
  File "/Users/tomasz/tf1/lib/python2.7/site-packages/tensorflow/python/autograph/impl/conversion.py", line 624, in convert_func_to_ast
    node = node_to_graph(node, context)
  File "/Users/tomasz/tf1/lib/python2.7/site-packages/tensorflow/python/autograph/impl/conversion.py", line 657, in node_to_graph
    node = converter.standard_analysis(node, context, is_initial=True)
  File "/Users/tomasz/tf1/lib/python2.7/site-packages/tensorflow/python/autograph/core/converter.py", line 354, in standard_analysis
    node = qual_names.resolve(node)
  File "/Users/tomasz/tf1/lib/python2.7/site-packages/tensorflow/python/autograph/pyct/qual_names.py", line 254, in resolve
    return QnResolver().visit(node)
  File<EMAIL_ADDRESS>line 241, in visit
    return visitor(node)
  File<EMAIL_ADDRESS>line 297, in generic_visit
    value = self.visit(value)
  File<EMAIL_ADDRESS>line 241, in visit
    return visitor(node)
  File<EMAIL_ADDRESS>line 297, in generic_visit
    value = self.visit(value)
  File<EMAIL_ADDRESS>line 241, in visit
    return visitor(node)
  File<EMAIL_ADDRESS>line 306, in generic_visit
    new_node = self.visit(old_value)
  File<EMAIL_ADDRESS>line 241, in visit
    return visitor(node)
  File<EMAIL_ADDRESS>line 297, in generic_visit
    value = self.visit(value)
  File<EMAIL_ADDRESS>line 241, in visit
    return visitor(node)
  File "/Users/tomasz/tf1/lib/python2.7/site-packages/tensorflow/python/autograph/pyct/qual_names.py", line 236, in visit_Subscript
    if isinstance(s.value, gast.Num):
AttributeError: 'module' object has no attribute 'Num'
WARNING:tensorflow:Entity <function initialize_variables at 0x134c82140> could not be transformed and will be executed as-is. Please report this to the AutgoGraph team. When filing the bug, set the verbosity to 10 (on Linux, `export AUTOGRAPH_VERBOSITY=10`) and attach the full output. Cause: converting <function initialize_variables at 0x134c82140>: AttributeError: 'module' object has no attribute 'Num'
<tensorflow.python.eager.function.ConcreteFunction>

I just checked against TensorFlow version 1.15.0rc1 and this issue is fixed there.

Opening back up because this is not fixed in TensorFlow version 2.0.0rc1
> library(reticulate)
> library(tensorflow)
>
> Sys.setenv(AUTOGRAPH_VERBOSITY=10)
> reticulate::use_virtualenv("~/tf2")
>
> tf$version$VERSION
[1] "2.0.0-rc1"
> tf$version$GIT_VERSION
[1] "v2.0.0-rc0-101-gd2d2566eef"
>
> x <- tf$constant(1)
2019-09-20 00:03:35.132646: I tensorflow/core/platform/cpu_feature_guard.cc:142] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
2019-09-20 00:03:35.146449: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x7fc075f78db0 executing computations on platform Host. Devices:
2019-09-20 00:03:35.146465: I tensorflow/compiler/xla/service/service.cc:175]   StreamExecutor device (0): Host, Default Version
> add1 <- py_func(function() x + 1)
>
> fn <- tf$`function`(add1, autograph = FALSE)
>
> fn$get_concrete_function()
WARNING:tensorflow:Entity <function initialize_variables at 0x10adc18c0> could not be transformed and will be executed as-is. Please report this to the AutoGraph team. When filing the bug, set the verbosity to 10 (on Linux, `export AUTOGRAPH_VERBOSITY=10`) and attach the full output. Cause: 'module' object has no attribute 'Num'
<tensorflow.python.eager.function.ConcreteFunction>

@t-kalinowski,
I could reproduce the issue with Tf 2.0.0.rc1. Can you try with latest Tf 2.0 nightly version. Thanks!

Thanks,
This appears to be fixed in tf 2.0 nightly preview (tf version 2.0.0-dev20190920) and in tf version 1.15.0-rc1
However, this is currently broken in the tf v1 nightly (taken from pip install tf-nightly, tf version 1.15.0-dev20190821

TF 1.15 rc1 is the latest TF 1.X release so we can expect the fix to be included in final TF 1.15 version as well.
TF nightly version 1.15.0-dev20190821 was built on 08/21/2019 where as TF 1.15rc1 version was released on 09/16/2019.


[Coverity CID :203473]Null pointer dereferences in /subsys/usb/usb_descriptor.c

Static code scan issues seen in File: /subsys/usb/usb_descriptor.c
Category: Null pointer dereferences
Function: usb_fix_descriptor
Component: Other
CID: 203473
Please fix or provide comments to square it off in coverity in the link: https://scan9.coverity.com/reports.htm#v32951/p12996

Coverity is flagging that cfg_data is initialized to NULL and is passed to usb_validate_ep_cfg_data() in the USB_ENDPOINT_DESC case statement where a dereference occurs. Not knowing this code, I suspect that the while loop is processing the descriptor and as such it may assume that the bDescriptorType of USB_INTERFACE_DESC is invoked AND the code path routes through the code that initializes cfg_data.
Looks like a legit bug that should be fixed.


EmberLib :

Dear @KimonHoffmann , @mywave82 , @mkeuck
I am trying to mask some value from an enumeration and I am using '~' as recommended in the ember+ specification

My properties of the qualifiedParameter - enum
                    {
                        number: 4,
                        identifier: "ParameTypeEnumerationMasking",
                        description: "Enumeration parameter with masking some value",
                        value: 0,
                        access: "readWrite",
                        enumeration: "Line\n~Mic+48V\nMic",
                        isOnline: true,
                        default: 0,
                        type: "enum"
                    },

There is no effect in EmberPlusviewer and vsmStudio too.
enumeration: "Line\n~Mic+48V\nMic",

Maybe EmberPlusViewer and vsmstudio are not supporting the way of masking value.
I give an example, a codec manufacturer provides all the encoder codec available in the protocol but the customer buy only some of them, if I let the codec not licensed I will have an issue.
If I remove from the enum the encoder codec not licensed, If I need to add it in the future, I will have an issue with the mapping ID of the enum.
Would you like to verify if the way I added the '~' is correct ?
Best Regards,
Youssef

Dear @KimonHoffmann  , @mywave82
Just to confirm the ~is working on vsmstudio (missing refresh when I updated the emberTree) but EmberPlusViewer 2.4 doesn't work
Best Regards
Youssef


token validation process

@h1alexbel thank you for reporting this. I'll assign someone to take care of it soon.

@zoeself deregiste

@zoeself deregiste

@h1alexbel didn't understand that, sorry.

@zoeself deregister

@zoeself deregister

@h1alexbel ok, I've removed this task from scope. I'm not managing it anymore.

@l3r8yJ  take a look, please


Illegal Instruction(core dumped) on AWS

Hello people, I am trying to compile the dnn version of training car detection model on AWS and later I want to train my own model. But when I run the training binary I get Illegal instruction error (see image). Has anyone used dlib on AWS and experienced this error ? Either way, any ideas as to how to debug this ?
Thank you very much.

By the way, I ran the program with valgrind and found it is an out of memory error. Just pasting it here if anyone else has this issue.

ubuntu@ip-172-31-29-24:~/dlib/examples/build$ valgrind ./dnn_mmod_ex ../faces/
==13524== Memcheck, a memory error detector
==13524== Copyright (C) 2002-2015, and GNU GPL'd, by Julian Seward et al.
==13524== Using Valgrind-3.11.0 and LibVEX; rerun with -h for copyright info
==13524== Command: ./dnn_mmod_ex ../faces/
==13524==
num training images: 4
num testing images:  5
num detector windows: 1
detector window width by height: 40 x 40
overlap NMS IOU thresh:             0.170353
overlap NMS percent covered thresh: 0.33141
==13524== Warning: noted but unhandled ioctl 0x30000001 with no size/direction hints.
==13524==    This could cause spurious value errors to appear.
==13524==    See README_MISSING_SYSCALL_OR_IOCTL for guidance on writing a proper wrapper.
==13524== Warning: noted but unhandled ioctl 0x27 with no size/direction hints.
==13524==    This could cause spurious value errors to appear.
==13524==    See README_MISSING_SYSCALL_OR_IOCTL for guidance on writing a proper wrapper.
==13524== Warning: noted but unhandled ioctl 0x7ff with no size/direction hints.
==13524==    This could cause spurious value errors to appear.
==13524==    See README_MISSING_SYSCALL_OR_IOCTL for guidance on writing a proper wrapper.
==13524== Warning: noted but unhandled ioctl 0x25 with no size/direction hints.
==13524==    This could cause spurious value errors to appear.
==13524==    See README_MISSING_SYSCALL_OR_IOCTL for guidance on writing a proper wrapper.
==13524== Warning: noted but unhandled ioctl 0x17 with no size/direction hints.
==13524==    This could cause spurious value errors to appear.
==13524==    See README_MISSING_SYSCALL_OR_IOCTL for guidance on writing a proper wrapper.
==13524== Warning: set address range perms: large range [0x200000000, 0x700000000) (noaccess)
==13524== Warning: set address range perms: large range [0x900000000, 0xb00000000) (noaccess)
==13524== Thread 5:
==13524== Syscall param sched_setaffinity(mask) points to unaddressable byte(s)
==13524==    at 0x16D01499: syscall (syscall.S:38)
==13524==    by 0x16077028: __kmp_affinity_determine_capable (in /usr/local/lib/libiomp5.so)
==13524==    by 0x16057A24: __kmp_env_initialize(char const*) (in /usr/local/lib/libiomp5.so)
==13524==    by 0x16043E06: INTERNAL_23_______src_kmp_runtime_c_680c9f7a::kmp_do_serial_initialize() (in /usr/local/lib/libiomp5.so)
==13524==    by 0x16038F84: kmp_get_global_thread_id_reg (in /usr/local/lib/libiomp5.so)
==13524==    by 0x1602D875: omp_in_parallel@@VERSION (in /usr/local/lib/libiomp5.so)
==13524==    by 0x5EC8C29: dscal (in /usr/local/lib/libopenblas_haswellp-r0.3.0.dev.so)
==13524==    by 0x63FE733: dlarfg (in /usr/local/lib/libopenblas_haswellp-r0.3.0.dev.so)
==13524==    by 0x639D94D: dgebd2 (in /usr/local/lib/libopenblas_haswellp-r0.3.0.dev.so)
==13524==    by 0x639E0C2: dgebrd (in /usr/local/lib/libopenblas_haswellp-r0.3.0.dev.so)
==13524==    by 0x63B3BAF: dgesvd (in /usr/local/lib/libopenblas_haswellp-r0.3.0.dev.so)
==13524==    by 0x4E6000: int dlib::lapack::gesvd<double, 3l, 0l, 3l, 0l, 0l, 1l, 0l, 0l, dlib::memory_manager_stateless_kernel_1 >(char, char, dlib::matrix<double, 3l, 0l, dlib::memory_manager_stateless_kernel_1, dlib::row_major_layout>&, dlib::matrix<double, 0l, 1l, dlib::memory_manager_stateless_kernel_1, dlib::row_major_layout>&, dlib::matrix<double, 3l, 0l, dlib::memory_manager_stateless_kernel_1, dlib::row_major_layout>&, dlib::matrix<double, 0l, 0l, dlib::memory_manager_stateless_kernel_1, dlib::row_major_layout>&) (in /home/ubuntu/dlib/examples/build/dnn_mmod_ex)
==13524==  Address 0x0 is not stack'd, malloc'd or (recently) free'd
==13524==
step#: 0     learning rate: 0.1   average loss: 0            steps without apparent progress: 0
==13524== Warning: noted but unhandled ioctl 0x19 with no size/direction hints.
==13524==    This could cause spurious value errors to appear.
==13524==    See README_MISSING_SYSCALL_OR_IOCTL for guidance on writing a proper wrapper.
==13524== Warning: noted but unhandled ioctl 0x21 with no size/direction hints.
==13524==    This could cause spurious value errors to appear.
==13524==    See README_MISSING_SYSCALL_OR_IOCTL for guidance on writing a proper wrapper.
==13524== Warning: noted but unhandled ioctl 0x1b with no size/direction hints.
==13524==    This could cause spurious value errors to appear.
==13524==    See README_MISSING_SYSCALL_OR_IOCTL for guidance on writing a proper wrapper.
==13524== Warning: set address range perms: large range [0x201300000, 0x213aba000) (defined)
==13524== Warning: set address range perms: large range [0x213bc0000, 0x22637a000) (defined)
==13524== Warning: set address range perms: large range [0x226380000, 0x2565e6000) (defined)
==13524== Warning: set address range perms: large range [0x256700000, 0x286966000) (defined)
==13524== Warning: set address range perms: large range [0x2e0600000, 0x310866000) (defined)
==13524== Warning: noted but unhandled ioctl 0x42 with no size/direction hints.
==13524==    This could cause spurious value errors to appear.
==13524==    See README_MISSING_SYSCALL_OR_IOCTL for guidance on writing a proper wrapper.
==13524== Warning: noted but unhandled ioctl 0x22 with no size/direction hints.
==13524==    This could cause spurious value errors to appear.
==13524==    See README_MISSING_SYSCALL_OR_IOCTL for guidance on writing a proper wrapper.
==13524== Warning: set address range perms: large range [0x2e0600000, 0x310866000) (noaccess)
==13524== Warning: set address range perms: large range [0x201300000, 0x213aba000) (noaccess)
step#: 2     learning rate: 0.1   average loss: 0            steps without apparent progress: 0
==13524== Warning: set address range perms: large range [0x213bc0000, 0x22637a000) (noaccess)
==13524== Warning: set address range perms: large range [0x256700000, 0x286966000) (noaccess)
==13524== Warning: set address range perms: large range [0x226380000, 0x2565e6000) (noaccess)
Error while calling cudaMalloc(&data, new_size*sizeof(float)) in file /home/ubuntu/dlib/dlib/dnn/gpu_data.cpp:195. code: 2, reason: out of memory
==13524==
==13524== HEAP SUMMARY:
==13524==     in use at exit: 132,641,622 bytes in 99,175 blocks
==13524==   total heap usage: 211,084 allocs, 111,909 frees, 346,499,270 bytes allocated
==13524==
==13524== LEAK SUMMARY:
==13524==    definitely lost: 0 bytes in 0 blocks
==13524==    indirectly lost: 0 bytes in 0 blocks
==13524==      possibly lost: 578,392 bytes in 4,321 blocks
==13524==    still reachable: 132,063,230 bytes in 94,854 blocks
==13524==         suppressed: 0 bytes in 0 blocks
==13524== Rerun with --leak-check=full to see details of leaked memory
==13524==
==13524== For counts of detected and suppressed errors, rerun with: -v
==13524== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)


Union "Types" don't seem to be generating

Working through a large query. It seems to choke when I need to hit a field that is a union type.  Relevant piece from schema
{
          "kind": "UNION",
          "name": "Asset",
          "description": null,
          "fields": null,
          "inputFields": null,
          "interfaces": null,
          "enumValues": null,
          "possibleTypes": [
            {
              "kind": "OBJECT",
              "name": "Article",
              "ofType": null
            },
            {
              "kind": "OBJECT",
              "name": "Image",
              "ofType": null
            },
            {
              "kind": "OBJECT",
              "name": "Video",
              "ofType": null
            },
            {
              "kind": "OBJECT",
              "name": "Slideshow",
              "ofType": null
            },
            {
              "kind": "OBJECT",
              "name": "InteractivePromo",
              "ofType": null
            },
            {
              "kind": "OBJECT",
              "name": "InteractiveEmbed",
              "ofType": null
            },
            {
              "kind": "OBJECT",
              "name": "Promo",
              "ofType": null
            },
            {
              "kind": "OBJECT",
              "name": "BlogPost",
              "ofType": null
            },
            {
              "kind": "OBJECT",
              "name": "ExternalUrl",
              "ofType": null
            },
            {
              "kind": "OBJECT",
              "name": "Recipe",
              "ofType": null
            },
            {
              "kind": "OBJECT",
              "name": "TimesTopic",
              "ofType": null
            }
          ]
        }

query (it seems to not recognize that "Article" is an "Asset", Asset Type never gets generated

query SimpleQuery($id:String!)
 {
     program(id:$id) {
       firstLoad {
         ...onBlock
       }
     }
   }

   fragment onBlock on Block {
     title
     showTitle
     showMore
     showSection
     firstLoad {
       ... onAssetsConnection
     }
     moreAssets {
       ... onAssetsConnection
     }
   }

   fragment onAssetsConnection on AssetsConnection {
     edges {
       node {

         ... on Article {
           promo_summary
         }
       }
     }
   }

Logs:
/Users/Nakhimovich/apollo-android-copy/apollo-sample/build/generated/source/apollo/fragment/OnAssetsConnection.java:14: error: package type does not exist
import type.Asset;
           ^
/Users/Nakhimovich/apollo-android-copy/apollo-sample/build/generated/source/apollo/fragment/OnAssetsConnection.java:78: error: cannot find symbol
    private @Nullable Asset node;
                      ^
  symbol:   class Asset
  location: class Edge
/Users/Nakhimovich/apollo-android-copy/apollo-sample/build/generated/source/apollo/fragment/OnAssetsConnection.java:80: error: cannot find symbol
    public Edge(@Nullable Asset node) {
                          ^
  symbol:   class Asset
  location: class Edge
/Users/Nakhimovich/apollo-android-copy/apollo-sample/build/generated/source/apollo/fragment/OnAssetsConnection.java:84: error: cannot find symbol
    public @Nullable Asset node() {
                     ^
  symbol:   class Asset
  location: class Edge
/Users/Nakhimovich/apollo-android-copy/apollo-sample/build/generated/source/apollo/fragment/OnAssetsConnection.java:130: error: cannot find symbol
      @Nonnull Edge create(@Nullable Asset node);
                                     ^
  symbol:   class Asset
  location: interface Creator
/Users/Nakhimovich/apollo-android-copy/apollo-sample/build/generated/source/apollo/fragment/OnAssetsConnection.java:121: error: cannot find symbol
        Asset node;
        ^
  symbol:   class Asset
  location: class __ContentValues
/Users/Nakhimovich/apollo-android-copy/apollo-sample/build/generated/source/apollo/fragment/OnAssetsConnection.java:66: error: cannot find symbol
      public @Nonnull Edge create(@Nullable Asset node) {
                                            ^
  symbol: class Asset
/Users/Nakhimovich/apollo-android-copy/apollo-sample/build/generated/source/apollo/fragment/OnAssetsConnection.java:92: error: cannot find symbol
        Field.forObject("node", "node", null, true, new Field.ObjectReader<Asset>() {
                                                                           ^
  symbol:   class Asset
  location: class Mapper
/Users/Nakhimovich/apollo-android-copy/apollo-sample/build/generated/source/apollo/fragment/OnAssetsConnection.java:93: error: cannot find symbol
          @Override public Asset read(final ResponseReader reader) throws IOException {
                           ^
  symbol: class Asset
/Users/Nakhimovich/apollo-android-copy/apollo-sample/build/generated/source/apollo/fragment/OnAssetsConnection.java:94: error: package Asset does not exist
            return new Asset.Mapper(factory.assetFactory()).map(reader);
                            ^
/Users/Nakhimovich/apollo-android-copy/apollo-sample/build/generated/source/apollo/fragment/OnAssetsConnection.java:94: error: cannot find symbol
            return new Asset.Mapper(factory.assetFactory()).map(reader);
                                           ^
  symbol:   method assetFactory()
  location: variable factory of type Factory
/Users/Nakhimovich/apollo-android-copy/apollo-sample/build/generated/source/apollo/fragment/OnAssetsConnection.java:111: error: cannot find symbol
                contentValues.node = (Asset) value;
                                      ^
  symbol: class Asset

@digitalbuddha could you pls provide the generated IR.json file as well

{
	"operations": [
		{
			"filePath": "/Users/Nakhimovich/apollo-android-copy/apollo-sample/src/main/graphql/com/example/foo.graphql",
			"operationName": "SimpleQuery",
			"operationType": "query",
			"variables": [
				{
					"name": "id",
					"type": "String!"
				}
			],
			"source": "query SimpleQuery($id: String!) {\n  program(id: $id) {\n    __typename\n    firstLoad {\n      __typename\n      ...onBlock\n    }\n  }\n}",
			"fields": [
				{
					"responseName": "program",
					"fieldName": "program",
					"type": "Program",
					"args": [
						{
							"name": "id",
							"value": {
								"kind": "Variable",
								"variableName": "id"
							}
						}
					],
					"fields": [
						{
							"responseName": "firstLoad",
							"fieldName": "firstLoad",
							"type": "[Programmable]",
							"fields": [],
							"fragmentSpreads": [
								"onBlock"
							],
							"inlineFragments": []
						}
					],
					"fragmentSpreads": [],
					"inlineFragments": []
				}
			],
			"fragmentsReferenced": [
				"onBlock",
				"onAssetsConnection"
			]
		}
	],
	"fragments": [
		{
			"filePath": "/Users/Nakhimovich/apollo-android-copy/apollo-sample/src/main/graphql/com/example/foo.graphql",
			"fragmentName": "onBlock",
			"source": "fragment onBlock on Block {\n  __typename\n  title\n  showTitle\n  showMore\n  showSection\n  firstLoad {\n    __typename\n    ...onAssetsConnection\n  }\n  moreAssets {\n    __typename\n    ...onAssetsConnection\n  }\n}",
			"typeCondition": "Block",
			"fields": [
				{
					"responseName": "title",
					"fieldName": "title",
					"type": "String"
				},
				{
					"responseName": "showTitle",
					"fieldName": "showTitle",
					"type": "Boolean"
				},
				{
					"responseName": "showMore",
					"fieldName": "showMore",
					"type": "Boolean!"
				},
				{
					"responseName": "showSection",
					"fieldName": "showSection",
					"type": "Boolean"
				},
				{
					"responseName": "firstLoad",
					"fieldName": "firstLoad",
					"type": "AssetsConnection",
					"fields": [],
					"fragmentSpreads": [
						"onAssetsConnection"
					],
					"inlineFragments": []
				},
				{
					"responseName": "moreAssets",
					"fieldName": "moreAssets",
					"type": "AssetsConnection",
					"fields": [],
					"fragmentSpreads": [
						"onAssetsConnection"
					],
					"inlineFragments": []
				}
			],
			"fragmentSpreads": [],
			"inlineFragments": [],
			"fragmentsReferenced": [
				"onAssetsConnection"
			]
		},
		{
			"filePath": "/Users/Nakhimovich/apollo-android-copy/apollo-sample/src/main/graphql/com/example/foo.graphql",
			"fragmentName": "onAssetsConnection",
			"source": "fragment onAssetsConnection on AssetsConnection {\n  __typename\n  edges {\n    __typename\n    node {\n      __typename\n      ... on Article {\n        __typename\n        promo_summary\n      }\n    }\n  }\n}",
			"typeCondition": "AssetsConnection",
			"fields": [
				{
					"responseName": "edges",
					"fieldName": "edges",
					"type": "[AssetsEdge]",
					"fields": [
						{
							"responseName": "node",
							"fieldName": "node",
							"type": "Asset",
							"fields": [],
							"fragmentSpreads": [],
							"inlineFragments": [
								{
									"typeCondition": "Article",
									"fields": [
										{
											"responseName": "promo_summary",
											"fieldName": "promo_summary",
											"type": "String"
										}
									],
									"fragmentSpreads": []
								}
							]
						}
					],
					"fragmentSpreads": [],
					"inlineFragments": []
				}
			],
			"fragmentSpreads": [],
			"inlineFragments": [],
			"fragmentsReferenced": []
		}
	],
	"typesUsed": []
}

As I understood Asset should be interface. So we need generate it as interface, plus Article should implement Asset

Yup that seems reasonable

Interestingly enough if I move the Article out to its own fragment (instead of inline), everything compiles correctly.

query SimpleQuery($id:String!)
 {
     program(id:$id) {
       firstLoad {
         ...onBlock
       }
     }
   }

   fragment onBlock on Block {
     title
     showTitle
     showMore
     showSection
     firstLoad {
       ... onAssetsConnection
     }
     moreAssets {
       ... onAssetsConnection
     }
   }

   fragment onAssetsConnection on AssetsConnection {
     edges {
        node {
         ... assetFragment
       }
     }
   }

   fragment assetFragment on Article {
                promo_summary

   }

Even though it compiles I never get anything marshalled into assetFragment Node

I'm not sure what is going on with the generated Java code, but I don't think we need special handling for union types. They are similar to GraphQL interfaces, except that they don't define any fields of themselves.
Right now on iOS, we don't generate types for them, just as we don't generate types for interfaces. (There is no type generated for the Character interface from the Star Wars schema either for example.) We only generate types for inline fragments defined on concrete object types, and make these accessible through as<Type> methods.
So given the following query:
query Search {
  search(text: "an") {
    ... on Human {
      name
      height
    }
    ... on Starship {
      name
      length
    }
  }
}

The types generated are:
public struct Data: GraphQLMappable {
    public let search: [Search?]?

    public init(reader: GraphQLResultReader) throws {
      search = try reader.optionalList(for: Field(responseName: "search", arguments: ["text": "an"]))
    }

    public struct Search: GraphQLMappable {
      public let __typename: String

      public let asHuman: AsHuman?
      public let asStarship: AsStarship?

      public init(reader: GraphQLResultReader) throws {
        __typename = try reader.value(for: Field(responseName: "__typename"))

        asHuman = try AsHuman(reader: reader, ifTypeMatches: __typename)
        asStarship = try AsStarship(reader: reader, ifTypeMatches: __typename)
      }

      public struct AsHuman: GraphQLConditionalFragment {
        public static let possibleTypes = ["Human"]

        public let __typename: String
        public let name: String
        public let height: Double?

        public init(reader: GraphQLResultReader) throws {
          __typename = try reader.value(for: Field(responseName: "__typename"))
          name = try reader.value(for: Field(responseName: "name"))
          height = try reader.optionalValue(for: Field(responseName: "height"))
        }
      }

      public struct AsStarship: GraphQLConditionalFragment {
        public static let possibleTypes = ["Starship"]

        public let __typename: String
        public let name: String
        public let length: Double?

        public init(reader: GraphQLResultReader) throws {
          __typename = try reader.value(for: Field(responseName: "__typename"))
          name = try reader.value(for: Field(responseName: "name"))
          length = try reader.optionalValue(for: Field(responseName: "length"))
        }
      }
    }
  }
}

Not sure at difference on android is. I am getting import statements for a class that is not getting generated.
import type.Asset;
Similar it is trying to access that class within mappers as seen in my traces above.  Is there something else that would help you debug @martijnwalraven

I think this is something that will have to fixed on the Java code generation side. Maybe @sav007 knows what is going on?

Happy to add union types to typesUsed, but I don't think the code generator needs to do anything with it, unless the Android code works differently from the iOS code. Let's hear what @sav007 or @brianPlummer has to say first.

@digitalbuddha if it's only an import statement but the class isn't referenced anywhere in the generated code class, then it might have to do with the way we're adding import for types and fragment packages.
If the import is there and the class is referenced in the generated code class, it's probably the way apollo-compiler generates the classes.

I think its a few other places too. Check out the stacktrace above. I think the mappers and object readers are using it as a generic

@martijnwalraven it was an issue on our end, so will fix that


Request to add "Doctruyenonline" - vietnamese

Source name
Doctruyenonline
Source link
https://doctruyenonline.vn/truyen-tranh
Source language
Vietnamese - Tiếng Việt
Other details
Manhua - Manhwa - Manga
Acknowledgements

[X] I have checked that the extension does not already exist by searching the extension listing and verified it does not appear there.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a meaningful title with the source name.
[X] I will fill out all of the requested information in this form.

New URL: https://bingtruyenviet.com/


Moving default subnet for docker network to overlays subnets

Creating a docker network with subnets like 172.17-31.x.x/16 and 192.168.x.x/20 could create conflicts with local networks like discussed in https://github.com/moby/moby/issues/21776#issuecomment-206133109.
Using <IP_ADDRESS>/8 derivated subnets will not expose to this conflicts, already other projects used this network range to overlay networks (like as istance https://docs.mesosphere.com/1.10/overview/design/overlay/).
Signed-off-by: Cristiano Casella<EMAIL_ADDRESS>

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "master" git@github.com:cristianocasella/libnetwork.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@c592ee4). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #2072   +/-   ##
=========================================
  Coverage          ?   40.51%
=========================================
  Files             ?      138
  Lines             ?    22172
  Branches          ?        0
=========================================
  Hits              ?     8983
  Misses            ?    11872
  Partials          ?     1317

Impacted Files
Coverage Δ

ipamutils/utils.go
100% <100%> (ø)

Continue to review full report at Codecov.

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

Sorry if I'm being a newb here but isn't <IP_ADDRESS>/8 assigned to IBM? Have they publicly released this space for this (or any kind) of use? An obvious existing public use of this is the quad9 public DNS resolver system (https://www.quad9.net/) at <IP_ADDRESS> and <IP_ADDRESS>.

Does https://github.com/docker/libnetwork/pull/2058 address this issue?  If so, should we close this?

Yes. Our defaultAddressPool feature will take care of this issue.

But this is for overlay networks, not local networks, correct?

Yes. right now we are supporting only local scope bbut global scope which takes care of overlay networks is in our pipeline.
Sorry for the confusion.

Sorry @thaJeztah, @selansen, but I can't tell if you are in agreement or disagreement.  Is it correct that the https://github.com/docker/libnetwork/pull/2058 only changes default overlay network addresses or does it only manage node local addresses?  To me, it looks like the latter, but I stand to be corrected.

Thanks for your support. With #36396 this can be closed.


critical bug: edit service, and then save, selector miss, 100% reproduce!

Describe your issue here
critical bug: edit service, and then save, selector miss, 100% reproduce!

Useful
Info

Versions
Rancher v2.2.3-v2.2.5 UI: v2.2.41

Route
containers.index

Note: Repro in older v2.2.5, create service in UI. Edit as YAML and change selector. Save. Edit again - check selector.

@loganhz I'm having a hard time reproducing. I've reviewed your PR and see your steps mentioned but am not seeing the issue in v2.2.5. Any idea what I'm missing?

Tried to create nginx workload, it creates service for me automatically, attempt to View/Edit YAML and change selector and it works, remove selector it works.
Try to delete the nginx service and import YAML and paste in with the selector and it works
Try to create nginx service with import YAML and don't include the selector first time (create) then edit YAML and add it in and it works.

I wasn't successful in finding a situation where the selector did not work when edit as YAML or import YAML, etc.

@davidnuzik

Create a workload
Go to the Edit page for the auto created svc in Service Discovery page by clicking Edit instead of Edit as YAML.
Just click Save without changing anything
Click View YAML to check if the selector field is missing in the yaml.

Thanks Logan.
Reproduction:
v2.2.5

Create an nginx workoad with port mapping.
Rancher automatically creates service. View/Edit in YAML this service, notice our selector is correct currently:

Cancel out of the view/edit YAML. Edit the service. Save (make no changes).
View/Edit in YAML again. The selector is missing now!

Validation:
master-head (v2.3) (8/12/19)

Perform the steps per the above reproduction section again.
Upon the last step we can see the selector is still there now. (The issue is resolved).


APPT-XXXX: Loading spinners

Creates Loading Spinner components according to the nhsuk-frontend backlog and uses them to replace submit buttons during form submissions.

 May not be needed.

It's a good question, certainly doable.
I think component tests would be fairly pointless as there's no behaviour to test, and testing for css is icky.
We could test that the spinners are used by the form components by mocking out the useForm<> object and setting isSubmitting and isSubmitSuccessful to true.
There might be a smart way to handle the async nature of all this. Naturally we don't know how long the submission is going to take, and our tests await each assertion to accommodate this delay. So writing "First you see the see then you see the next page" might be flaky. I'll have a quick play with mocking the useForm return and see how it goes :)


Epic: seamless tenant migration between pageservers

Motivation
This follows on from https://github.com/neondatabase/neon/issues/5050.  Whereas #5050 makes it safe to attach a tenant to a new pageserver without detaching the old one, this epic will make a tenant migration seamless from postgres's point of view.
See RFC: https://github.com/neondatabase/neon/pull/5029
DoD
For failover (i.e. unplanned migration), only most recent LSNs become unavailable for reads.  They become available again in a time bounded by the time taken for the new pageserver to download the writes since the remote_consistent_lsn on the old pageserver.
For planned migrations, postgres experiences no loss of read availability whatsoever.
Tasks

[ ] Compute: https://github.com/neondatabase/neon/issues/4578
[ ] Pageserver: implement warm secondary mode
[ ] Continuously inject migrations on staging.
[ ] Control plane: update database schema to handle multiple locations per Project, and assign secondary locations
[ ] Pageserver: roll out more instances to accomodate secondary locations
[ ] Pageserver: implement flush & AttachedStale notification API, AttachedMulti mode
[ ] Pageserver: create automated tests that demonstrate the migration process
[ ] Control plane: implement seamless migration procedure

Before we can enable generation usage and these, we will need to have a way to restrict consumption_metrics.rs from a pageserver with previous generation number.

This is on pause to avoid merging more code until we can deploy existing changes, which will require https://github.com/neondatabase/cloud/issues/6600

@jcsp do you have an issue somewhere on re-attach and maybe validate modifications? As far as I understood from the RFC, we should return mode in /re-attach response, but generation number there should also become optional for the secondaries then.

I just responded to the similar ping on https://github.com/neondatabase/cloud/issues/9614#issuecomment-1911064489 . There's no change to validate.
Including attachment states and/or secondary tenants in /re-attach responses is not essential for correctness here -- that's more of an optimization to enable tenants omitted in /re-attach to smoothly transition to secondary rather than detaching.


Links inside gridstack item not registering "focus" event on left click

I stumbled upon a bug while trying to place a Bootstrap popover inside a gridstack item. It seems that left click "focus" events aren't being registered but right clicking raises it normally even though there's role="button".
A minimal test case is the following:

         Dismissible popover

$('[data-toggle="popover"]').on('focus', function(e) {         console.log('focused');       });
When you left click nothing is logged to the console. Right click and all is good. Removing the element from gridstack html markup fixes the problem.

Could you throw this into a jsfiddle? gridstack should bubble up click events. I've got all kinds of crazy stuff inside of a gridstack item and haven't had problems. If you could put a running example together, I'd be happy to help.

https://jsfiddle.net/ctto3m1n/6/
This is pretty much the setup I'm using now, only that I attach the popover to the body.
Another interesting thing I saw while putting this example together is that if I shrink the result pane width until the panels occupy 12 columns, the popovers work. If the result pane width is such that the panels are side by side, the focus event isn't triggered.

I'm getting an error in the console with this fiddle. Looks like you may be missing lodash, perhaps? Could you fix it up? I have some idea of what may be causing an issue, but if we have a small play area (jsfiddle!) where we can make sure we're seeing the same thing, we'll know it to be working.

Fixed, sorry I neglected to update.

It's still giving me that error in the console when I go to the link you had posted:
https://www.dropbox.com/s/rchg61m2sjvn75u/Screenshot 2016-02-24 09.24.58.png?dl=0

https://jsfiddle.net/jjim50/h18c2fae/5/
This should work, sorry again.

It appears that jquery-ui is swallowing up the focus. I'm sure this is related to resize and/or drag. However, you may get around this easily. Change data-trigger="focus" to data-trigger="focus click". Let me know if that helps.

Changed it, it works on click now but clicking outside the popup doesn't dismiss it. This is a problem because I attach the popup to the body. The popup has a a button inside that triggers a modal. Modal displays but the popup z-index positions it above the modal.
So I'll have to go bug hunting in jquery-ui most likely?

I think that might be the case, unfortunately. One option you could try is to set the grid to not be resizable or draggable when you click on the popup. Then, when the popup is exited, you could again allow it. The easiest way to do this is probably to use the setStatic(true) method, then setStatic(false) when you're done. I haven't tried this out, but I'd imagine it would work.
Just a quick note - this method will only work in v0.2.5-dev. It's a feature/bug fix that was added recently.

For reference, another workaround would be to use a <button> tag instead of .


Light pop-in issues

Description

i dont really know how to explain this beyond visuals. i might be insane or very dumb
Screenshots
harsh light: (near arrivals, or in bridge is easy to test)

non-harsh light: (near tool storage is where i tested)

Additional context

Is this related to #2379?

fixed with paulvs


doesn't have to be vege

I know it was supposed to be vegetarian, but it needs some ground beef in my tacos

:-) Let us test it out.


docker swarm: dns resolution fails

From @mcastelino on April 27, 2017 17:37
When running Clear Container based containers in a docker swarm, DNS resolution does not work both for internal and external DNS when the DNS resolution is performed from within the Clear Container.
This is due to the way the DNS resolution is implemented within docker swarm.
DNS Resolution in Swarm
All docker swarm containers have the DNS resolver set to <IP_ADDRESS>:53
Docker swarm has an internal DNS based load balancer that RRs the DNS requests to spread load.
That runs on the localhost on the host bound to a host port specific to the container.
https://github.com/docker/libnetwork/blob/5ac04367ae7b0b12c33bed5f5b395bd4c104fff9/sandbox.go#L815
There is a iptables rule injected into the container namespace which is used to implement the docker DNS load balancer/resolver. That way <IP_ADDRESS>:53 maps to a specific port on which the corresponding resolver is running.
	-A DOCKER_OUTPUT -d <IP_ADDRESS>/32 -p tcp -m tcp --dport 53 -j DNAT --to-destination <IP_ADDRESS>:41343
	-A DOCKER_OUTPUT -d <IP_ADDRESS>/32 -p udp -m udp --dport 53 -j DNAT --to-destination <IP_ADDRESS>:43411
	-A DOCKER_POSTROUTING -s <IP_ADDRESS>/32 -p tcp -m tcp --sport 41343 -j SNAT --to-source :53
	-A DOCKER_POSTROUTING -s <IP_ADDRESS>/32 -p udp -m udp --sport 43411 -j SNAT --to-source :53

Here the DNS request is NATed to a container specific TCP and UDP port.
The resolver in this case is dockerd
 netstat -plunt
Active Internet connections (only servers)
Proto Recv-Q Send-Q Local Address           Foreign Address         State       PID/Program name
tcp        0      0 <IP_ADDRESS>:41343        <IP_ADDRESS>:*               LISTEN      14447/dockerd
udp        0      0 <IP_ADDRESS>:43411        <IP_ADDRESS>:*                           14447/dockerd

In the case of clear containers there is currently no way for the DNS request from within the VM to talk to the dockerd running on the host side. The only host connectivity that the VM has is via the docker_gwbridge. However the DNS resolver running within the network namespace is not reachable via the VM.
Network setup with Clear Containers

+---------------------------------+                +--------------------------------+
|   ingress sbox                  |                |                                |
|                    +            |                |                       +        |
|                    +-----------------------------------------------------+        |
|            I IP    |            |                |                       +--------------+
|                    +----+       |                |           +-----------+        |
|                    +    |       |                |       over|ay box     |        |
|                         |       |                |           |                    |
+---------------------------------+                +--------------------------------+
                          |                                    |
                          |                                    |
                          |                                    |            host container ns
                          |                   +--------------------------------------------+
                          |                   |        +-+     |                           |
                          |                   |        | +-----+     +-----------------+   |
                          |                   |        | |           |    IP           |   |
                          |                   |        | +--------------+ VIP          |   |
                          |         Resolver-----+     +-+           |                 |   |
     docker_gw_bridge     |       <IP_ADDRESS>  |                      |                 |   |
               +          |                   |       +-+R IP        |                 |   |
               +----------+                   |       | +---------------+ HIP          |   |
       H GW IP +--------------------------------------+ |            |                 |   |
               |                              |       +-+            +-----------------+   |
               +       default gw             |                    /etc/resolv.conf (127..)|
                                              +--------------------------------------------+

Internal DNS Resolution
Internal DNS resolution is handled completely by dockerd. So dockerd directly responds to the DNS request from the container process for any cluster local resource.
External DNS Resolution
External DNS resolution is not handled by dockerd. When dockerd is unable to resolve the name to a cluster local resource it will then perform a DNS resolution using the host's resolv.conf.
Hence the DNS resolution process for external name is
process -> dockerd:43411  -> host (via the namespace) -> external DNS -> host -> dockerd -> container.

Here you will notice,  dockerd sends packets out from within the namespace to the host via the interface bound to the docker_gwbridge.
In the case of clear containers as there is network connectivity  between the container network namespace and the host, this request can never be fulfilled.
Work around for External DNS
For external DNS resolution, the resolv.conf can be updated to point to a external DNS resolver. This will ensure that the external DNS resolution works
Fixing this issue in Clear Containers
The long term plan is to proxy the internal DNS requests from within the VM to dockerd. On failure of the DNS resolution the resolution has to be performed from within the VM to the host resolver. However assuming that the host resolver is the right resolver to use in the case of dockerd resolution failure may not be a correct assumption. Also this results in longer resolution times as dockerd takes a significant amount of time to fail the external DNS request.
Copied from original issue: 01org/cc-oci-runtime#854

From @mcastelino on April 27, 2017 17:56
@devimc We need to add this to our release notes.

From @devimc on April 27, 2017 18:49
great! thanks @mcastelino and nice description

Initial proposal for support for DNS resolution for clear containers with docker swarm.
The internal DNS resolution can be supported by running a DNS proxy agent within each Clear Container VM, which then forwards the request to the host side network namespace to the docker DNS resolver. This can be done in one of two ways

Assign a unique IP address to the integration bridge (which is not assigned an IP today), and use that to send the DNS requests to the host side namespace (this has been been verified to work using socat)

Note: Rancher VM also does something similar

Send the packets using a custom protocol over a virtual serial port and then run an agent in the host namespace to further forward the request to the docker DNS resolver.

However neither of these methods will solve the resolution of external DNS. Today the docker DNS resolver running in the host namespace does another round of DNS resolution using the host DNS configuration for external DNS, but it does the same through the namespace. In the case of clear containers there is no network connectivity between the host and the host side container namespace.
To support this there are two options

Add the host side DNS resolution entries to the container resolv.conf. Doing so results in the container trying the query via the host through the VM (this has been verified to work).
- However this means we are now modifying the resolv.conf which was created by docker. And we need to do this only in the case of docker swarm.
- Also it takes a long time to resolve, as the docker DNS resolution has to fail, which takes time
The dns proxy agent we run inside the VM does this resolution through the host.

The challenge is that DNS works fine when not running in a swarm (and it works when we run CC in kubernetes). So the agent will need to detect this and only activate itself when running in a swarm which will end up being a very specific hack!
If there are better ideas I would welcome them before I jump into this implementation
@sameo @dlespiau @rbradford


Replace Core Plot range plot with internal implementation

When many events are recorded, drawing performance is really bad. I believe an internal implementation can be achieved relatively easily, with much improved performance and focus.
The following are required:

[x] Research macOS drawing techniques and performance characteristics
[x] Implement a POC which is able to take a large existing dataset and draw it correctly, at significantly better performance
[x] Add plot range capabilities, similar to Core Plot, including manipulation (zoom/magnification, scroll)
[x] Implement selection of samples
[ ] Support sample & range highlighting
[x] Support Touch Bar.

Setting self.layer.drawsAsynchronously = YES; and drawing all the lines is very fast. Distincting by line color is much faster than naively setting the color before each line draw.

Improvement is ~0.002s to draw 19000+ lines vs 1.5s+.


Fix wrong pandas imports & fix setup.py typo

This fixes the following error:
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/pyfinance/utils.py", line 78, in <module>
    from pandas.tseries.frequencies import FreqGroup, get_freq_code
ImportError: cannot import name 'FreqGroup' from 'pandas.tseries.frequencies' (/usr/local/lib/python3.8/site-packages/pandas/tseries/frequencies.py)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/usr/local/lib/python3.8/site-packages/pyfinance/__init__.py", line 32, in <module>
    from .returns import TFrame, TSeries  # noqa
  File "/usr/local/lib/python3.8/site-packages/pyfinance/returns.py", line 42, in <module>
    from pyfinance import ols, utils
  File "/usr/local/lib/python3.8/site-packages/pyfinance/ols.py", line 15, in <module>
    from pyfinance import utils
  File "/usr/local/lib/python3.8/site-packages/pyfinance/utils.py", line 80, in <module>
    from pandas._libs.tslibs.frequencies import FreqGroup, get_freq_code
ModuleNotFoundError: No module named 'pandas._libs.tslibs.frequencies'

Traceback (most recent call last):
File "C:/quantzhang_2018-12-13/nuts/BOND/macro Quantamental/bond/III bond beta CFETS.py", line 893, in
from pyfinance import ols
File "C:\Program Files\Python35\lib\site-packages\pyfinance_init_.py", line 32, in
from .returns import TFrame, TSeries  # noqa
File "C:\Program Files\Python35\lib\site-packages\pyfinance\returns.py", line 40, in
from pyfinance import ols, utils
File "C:\Program Files\Python35\lib\site-packages\pyfinance\ols.py", line 15, in
from pyfinance import utils
File "C:\Program Files\Python35\lib\site-packages\pyfinance\utils.py", line 64, in
from pandas.tseries.frequencies import FreqGroup, get_freq_code
ImportError: cannot import name 'FreqGroup'

@quantzhang
Python 3.8.6 (default, Oct 27 2020, 08:57:44)
[Clang 12.0.0 (clang-1<IP_ADDRESS>)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> from pandas._libs.tslibs.dtypes import FreqGroup
>>> from pandas.tests.tslibs.test_period_asfreq import get_freq_code
>>> import pandas as pd
>>> pd.__version__
'1.1.3'
>>>

Thanks for finding this. I've addressed it in version 1.3.0, which is now available on PyPI. Apologies it was not done sooner.


vdn-har-convertor-jmeter-tool v3.2

vdn-har-convertor-jmeter-tool v3.1, use library har-to-jmeter-convertor-2.2 that correct bugs.

Remove the header 'Content-length' because the length is computed by JMeter when the request is created.
POST or PUT could have query string and body with content so add query string to the path.
Set Content Encoding to UFT-8 for POST or PUT method and request Content-Type : application/json.
Add body data content in Record.xml for PUT and PATCH methods.

Sorry i find a bug for POST with Query String and body JSON.
Don't push this version

OK vdn-har-convertor-jmeter-tool version 3.2

Thanks Andrey


Make bootstrap title configurable and sticky

Useful for playlists for instance:

bootstrap modal title shows playlist title
player shows video title

Or with single clips one can put the title just in the modal and avoid
hiding fp-title via CSS.

@nnarhinen - can be closed if you go for https://github.com/flowplayer/flowplayer-overlay/pull/3 where it is included or if you have objections.


feat: add channel parameter for puppeteer.launch

Add 'channel' parameter for Puppeteer.launch
Resolves #7340
Google Chrome is often used for using puppeteer as a browser automation tool,
We have to specify executablePath in such case. However we can't share the code among Windows/macOS users, because executablePath is different.
microsoft/playwright already introduced the channel parameter for solving this problem.
Let's provide the similar functionality for Puppeteer users.
const puppeteer = require('puppeteer');

puppeteer.launch({ channel: 'chrome-canary', headless: false }).then(async (browser) => {
    const page = await browser.newPage();
    await page.goto('https://github.com/puppeteer');
    await browser.close();
});

@googlebot I signed it!

Thank you for your king review. I considered again about the priority of channel and executablePath.

[1] executablePath  should be ignored when channel is specified
[2] executablePath should be respected even when channel is specified

[1] is actually adopted in Playwright. This is really simple. I think [1] is better.
The use-case of [2] is like below:
When some users changed the install path of Chrome,  channel: chrome fails to detect Chrome. Without modifying the script source code, users can set the install path with PUPPETEER_EXECUTABLE_PATH=/path/to/chrome
However in [2], users can also specify MS Edge as executablePath even when channel: chrome.
If the script developer intends to the script is assumed to work with Chrome, PUPPETEER_EXECUTABLE_PATH=/path/to/msedge  is just a source of worry.
This is just my observation but I didn't see the people who change the Chrome install path, and I thought [2] is useful for only a few people. IMO, simplicity is better than considering the edge case.
So I suggedt the spec [1] executablePath  should be ignored when channel is specified

Regarding the priority: Let's make them mutually exclusive.
If one of them is specified, the other one can not be set - that way, people don't run into unexpected behaviour.
Can you add an assert for that?

Regarding the priority: Let's make them mutually exclusive.
If one of them is specified, the other one can not be set - that way, people don't run into unexpected behaviour.

That's a good idea. I will add assertion for it.

Could you re-run GitHub actions? The failed test is caused by flaky testcase.

Sure :)

Rebased.

Your change landed 🎉
Thanks for your contribution @YusukeIwaki!

Any particular reason this feature only checks Program Files and not also Program Files (x86) on Windows? My Chrome installation is located in the latter. According to this it was the default location until 2020. For existing installations, it seems like the only (non-hacky) way to move it is to uninstall and reinstall.
Playwright seems to look in both (as well as in AppData/Local, which I seem to remember was indeed where the executable lived at one point), if that's any motivation: https://github.com/microsoft/playwright/blob/b6eb917c29b743e8d63004ae7a24974404461198/packages/playwright-core/src/utils/registry.ts#L461


Delete always available, even when you cannot delete

Describe the bug, optionally with screenshots
You cannot delete a message in a DM you did not start, but the option is available.
To Reproduce
Steps to reproduce the behavior:

Send message in a DM started by someone else
Click on delete button for DM
Nothing happens, delete remains available

Expected behavior
We should not be given the option to do something we cannot do. Can we actually delete a message from a DM not owned by us? If not, do not give users the option.

There isn't any "ownership" of DMs, it's just a log on your ship, like SMS.
Are you clicking delete on your own DM or someone else's?

Manifests as #2375


remake: Tensorlfow cpp warn log supression for pytest

remake of #8702 / seems there was some problem with my poetry version that messed up dependencies
Proposed changes:

fixes #7762
added pytest-env dependency
added TF_CPP_MIN_LOG_LEVEL=2 as default to pytest.ini to suppress tensorflow C++ code warnings, info, and debug by default

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

To see functionality compare log of this branchs ubuntu run to main branch ubuntu run. On main branch, you'll see tf logs of the type
2021-05-18 13:15:45.501852: W tensorflow/stream_executor/platform/default/dso_loader.cc:59] Could not load dynamic library 'libcudart.so.10.1'; dlerror: libcudart.so.10.1: cannot open shared object file: No such file or directory
2021-05-18 13:15:45.501895: I tensorflow/stream_executor/cuda/cudart_stub.cc:29] Ignore above cudart dlerror if you do not have a GPU set up on your machine.

but not on this one

Just to be clear: This warnings currently don't show in the CLI - only during testing?

Yes. In the CLI the logs are successfully surpressed as you had shown me. They do show up however when using the --debug flag which I think is fine.

@wochinge alrighty, just did the make file change


Pre-commit hook fails on windows when file only contains suppressed lines

On windows, after 1 line comment suppression in java file, try to commit.
Pre-commit hook fails.
There must be at least one range -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal com.cosium.code:maven-git-code-format:1.39:on-pre-commit (default-cli) on project core-additional-health-insurance: There must be at least one range
..
Caused by: java.lang.IllegalArgumentException: There must be at least one range
    at com.cosium.code.format.formatter.LineRanges.<init> (LineRanges.java:19)
    at com.cosium.code.format.formatter.LineRanges.of (LineRanges.java:30)
    at com.cosium.code.format.git.GitIndexEntry$EntryFormatter.computeLineRanges (GitIndexEntry.java:191)

This seems to be related to two factors :

on windows file is considered as partially staged even if no unstaged change exists
partially staged file format fails if no new line exists because it detects no line range to format

I guess this can also be reproduced on unix with this process :

suppress comment line on file and stage it
modify the file again but don't stage to produce unstaged change
try to commit staged modification

Platform :

plugin version 2.1
git version 2.25.0.windows.1
Apache Maven 3.6.2 (40f52333136460af0dc0d7232c0dc0bcf0d9e117; 2019-08-27T17:06:16+02:00)
Java version: 11.0.2, vendor: Oracle Corporation
OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows"

Are you using version 2.1? If not try it please.

Test was done on 2.1 :
 There must be at least one range -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal com.cosium.code:git-code-format-maven-plugin:2.1:on-pre-commit (default-cli) on project


Failed to watch /v1, Kind=Secret: failed to list /v1, Kind=Secret

Hello,
I deployed ESO on a Openshift cluster version 4.12, using OLM (Operator Lifecycle Manager).
On lab, no problem.
But on production, the "controller-manager" pod, deployed with OLM, that deploy the ESO helm chart based on the operatorconfig CR, consume huge memory (over 2 GB) and CPU.
In the logs, I can see these errors:
E1005 14:57:27.947872 1 reflector.go:138] pkg/mod/k8s.io/client-go@v0.22.2/tools/cache/reflector.go:167: Failed to watch /v1, Kind=Secret: failed to list /v1, Kind=Secret: stream error when reading response body, may be caused by closed connection. Please retry. Original error: stream error: stream ID 2949; INTERNAL_ERROR
{"level":"info","ts":1696517848.088788,"logger":"helm.controller","msg":"Reconciled release","namespace":"mpms-external-secrets-operator","name":"mpms","apiVersion":"operator.external-secrets.io/v1alpha1","kind":"OperatorConfig","release":"mpms"}
I1005 14:58:31.238828 1 request.go:665] Waited for 1.097264099s due to client-side throttling, not priority and fairness, request: GET:https://<IP_ADDRESS>:443/apis/build.openshift.io/v1?timeout=32s
{"level":"info","ts":1696517918.8514946,"logger":"helm.controller","msg":"Reconciled release","namespace":"mpms-external-secrets-operator","name":"mpms","apiVersion":"operator.external-secrets.io/v1alpha1","kind":"OperatorConfig","release":"mpms"}
I1005 14:59:09.665397 1 trace.go:205] Trace[503815121]: "Reflector ListAndWatch" name:pkg/mod/k8s.io/client-go@v0.22.2/tools/cache/reflector.go:167 (05-Oct-2023 14:58:05.331) (total time: 64334ms):
Trace[503815121]: [1m4.334004469s] [1m4.334004469s] END
E1005 14:59:09.665426 1 reflector.go:138] pkg/mod/k8s.io/client-go@v0.22.2/tools/cache/reflector.go:167: Failed to watch /v1, Kind=Secret: failed to list /v1, Kind=Secret: stream error when reading response body, may be caused by closed connection. Please retry. Original error: stream error: stream ID 3459; INTERNAL_ERROR
I1005 14:59:40.118865 1 request.go:665] Waited for 1.045434445s due to client-side throttling, not priority and fairness, request: GET:https://<IP_ADDRESS>:443/apis/flows.knative.dev/v1?timeout=32s
{"level":"info","ts":1696517985.8971891,"logger":"helm.controller","msg":"Reconciled release","namespace":"mpms-external-secrets-operator","name":"mpms","apiVersion":"operator.external-secrets.io/v1alpha1","kind":"OperatorConfig","release":"mpms"}
I1005 15:00:47.889320 1 request.go:665] Waited for 1.041447353s due to client-side throttling, not priority and fairness, request: GET:https://<IP_ADDRESS>:443/apis/kafka.strimzi.io/v1beta1?timeout=32s
I1005 15:00:55.460965 1 trace.go:205] Trace[1134291501]: "Reflector ListAndWatch" name:pkg/mod/k8s.io/client-go@v0.22.2/tools/cache/reflector.go:167 (05-Oct-2023 14:59:54.692) (total time: 60768ms):
Trace[1134291501]: [1m0.768885778s] [1m0.768885778s] END

The reason is we have about 20,000 secrets in the Kubernetes cluster and the controller-manager tries to get them.
This is the default behavior when controller-manager are developed using the operator SDK.
Cert-Manager has the same issue: https://github.com/cert-manager/cert-manager/issues/3229
Some info:
$ oc version
Client Version: 4.12.18
Kustomize Version: v4.5.7
Kubernetes Version: v1.25.12+26bab08

apiVersion: operators.coreos.com/v1alpha1
kind: Subscription
metadata:
  name: external-secrets-operator
spec:
  channel: stable
  installPlanApproval: Manual
  name: external-secrets-operator
  source: community-operators
  sourceNamespace: openshift-marketplace
  startingCSV: external-secrets-operator.v0.9.5
  config:
    resources:
      requests:
        cpu: 100m
        memory: 1Gi
      limits:
        cpu: 300m
        memory: 3Gi

Could you investigate on that issue ?
BR.

Apparently, you use an old version of the operator SDK helm image.
https://github.com/external-secrets/external-secrets-helm-operator/blob/main/Dockerfile#L2
You should upgrade it ?

Hi,
I did the following test : I created a new helm controller image by replacing the From docker image to version 1.31.0
# Build the manager binary
FROM quay.io/operator-framework/helm-operator:v1.31.0

ENV HOME=/opt/helm
COPY watches.yaml ${HOME}/watches.yaml
COPY helm-charts  ${HOME}/helm-charts
WORKDIR ${HOME}

Then I directly updated the CSV (ClusterServiceVersion) on the cluster, and replaced the default image by my image.
Here are the memory consumption of the helm controller:
Before: default v0.9.5

After: v0.9.5 with helm controller 1.31.0

It's better but there is still a lot of memory consumption for a basic helm controller.
One reason that should explain better performance in operator-sdk 1.31.0 here: https://github.com/operator-framework/operator-sdk/pull/6354
Any advice ?

I just tested the last operator-sdk/helm-controller version 1.32.0 available since this night and it fixes the issue:

The 1.32.0 contains this PR: https://github.com/operator-framework/operator-sdk/pull/6377

This issue has been fixed by the ESO version 0.9.6 which uses operator-sdk-helm controller version 1.32.0.
Thanks.


Translate Interpolated Strings to Concatenation as a Macro

Summary
The core language doesn't want to have to deal with variable length structures, but the case of an interpolated string is one of these. Instead, the macro system should be used to
Value
By doing this, we remove a point of significant complexity in the core language.
Specification

[ ] Create a macro to translate an interpolated string to a series of string concatenations.
[ ] It must only output raw strings (potentially including escapes), concatenation operators (+) and expressions. The 'interpolated string' entity should no longer exist.

For example, the following string:
x = 'The result is: `fn bar baz`, as you can see.'

Should be translated to:
x = "The result is: " + fn bar baz + ", as you can see."

Acceptance Criteria & Test Cases

This macro has been created and works as expected.

This will be implemented using the new macro system.


Javascript capstone - Week#5 - Pokemon-world-app

Greetings, friend reviewer!
You may notice that we've removed the cards from our project since we are only 2 partners working together.

Marco Parra

Julius

This is the link to our GitHub project.
Best regards!

Hi team,
Good job so far!
There are some issues that you still need to work on to go to the next project but you are almost there!
Required Changes ♻

[ ] Cards should be assigned to team members. There should see a circle with the user's avatar on each card. See the example below:

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Approved project! ✅ 🥳 🎈 🎉🎉🎉🎉🎉🎉🎉🎉
Hi there,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Typo causes warnings for native SVG feDistantLight tag

Vue version
3.2.5
Link to minimal reproduction
https://sfc.vuejs.org/#eNptUk1PwzAM/StROAASa7fxcZgKEgKJCzeuvYTObTPSJEqcDZj233GyrvtgvdR+z3l+Trzmz9ZmywB8xguEziqB8FRqxgq/bNhKzrF9LPl0PC45a0E2Le7SVFViUUuF4JicE+GD7on4FTW8yroOHt7jQakbJjVVfZjgKnhzwrayIl0HPqioq2IZAelP5aPKKOOIWLUS4UC51/YoNCZtJn5lF5LXh2gVFCwFShPb3Ucg37vK/9ki7sDzi+ms8bHh0I2dNX7MT/cDHBLGghOYphBOYtsBnp79mhA5OcGiHBk/wm7PYHdbbBiQxksP0qc9WElXKWDRRbqO6nuIfnYRnVMUO5hvExKhNDh1dXlBD3t5PXQpcloOiop8WBl+wyVdm8NRJ2y28EbTRq3TivSEL/mMJSRitHIxL3mLaP0sz31dxT1c+My4Jqcoc0Gj7CAD340+nVl5cCS8HX9T6g3f/AH0F9Uk
Steps to reproduce
See the log output and note the [Vue warn]: Failed to resolve component: feDistantLight message.
What is expected?
The feDistantLight should be recognized as a native SVG tag.
What is actually happening?
The feDistantLight seems to be mistaken as a component.
System Info
System:
    OS: Linux 6.1 Arch Linux
    CPU: (12) x64 AMD Ryzen 5 PRO 4650U with Radeon Graphics
    Memory: 10.55 GB / 30.58 GB
    Container: Yes
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.13.0 - /usr/bin/node
    Yarn: 1.22.19 - /usr/bin/yarn
    npm: 8.19.2 - /usr/bin/npm
  Browsers:
    Chromium: 109.0.5414.74
    Firefox: 108.0.2
  npmPackages:
    vue: ^3.2.45 => 3.2.45

Any additional comments?
I believe the problem originates in the domTagConfig.ts which contains an SVG tag called feDistanceLight instead of feDistantLight (distance vs distant). I’m not aware of a feDistanceLight SVG tag and believe this is a simple typo.

Maybe it's a problem with a misspelled word
wdyt? @LinusBorg

Yep, sure looks like it. Just send a PR :)


Bug: correct eslint tooling

Describe the bug
Eslint fixing rules works very well after #269 however there are some issues that were hanging for a while:

vite eslint cache issue
IDE auto-config for vsc, gitpod, etc
ignore .eslintrc.cjs for other editors

Steps to reproduce

Clone the project
Run npm ci && npm start in a terminal window/vsc/intellij/etc
Introduce a linting error and check the terminal window
Correct code linting error and check the terminal, the error should still be there

Affected services
hot.opensauced.pizza
Platforms
No response
Browsers
No response
Environment
No response
Additional context
auto-run the formatter on commit (could be opened as a separate issue if it proves difficult)
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[X] I agree to follow this project's Contribution Docs

I'll figure this out over the weekend, want to use gitpod and more vsc for tutorial purposes 🍕


embed New Computer Modern into binary

This is the font which most closely resembles the default LaTeX font and thus will be the go-to for many users switching to typst. It should be embedded and supported as first-class.
Especially considering that the font is supported by default on the Web app, it is a strange experience to compile a document locally only to find that the font is different without warning...

the default LaTeX font

The default for both Lualatex and Xelatex is Latin Modern.

@drupol The additional Italic and BoldItalic variants were downloaded from: https://ctan.org/pkg/newcomputermodern?lang=en

Ok thanks.
Preparing a PR to include that font in Nix in the meantime, I'll update this comment when it's done.

Hmm, the binary is already half fonts, half code in size. Maybe we can find another way to distribute often-used fonts, because otherwise it will just keep growing as people request other fonts.

Is there a good reason to include more than one sane default set of fonts in the binary?

Hmm, the binary is already half fonts @laurmaedje

This is a good point. From some quick testing, zstd was able to compress the font files down by about 50%. Then we could embed the compressed files and decompress at runtime.

otherwise it will just keep growing as people request other fonts

The same compelling interesting that exists for this font, as the singular default LaTeX font, will not exist for future requests. I ran into this issue this while migrating my first document to Typst from a very simple LaTeX file. I think it makes sense to optimize for these kinds of migrations if Typst is to successfully gain adoption by converting LaTeX users.

Is there a good reason to include more than one sane default set of fonts in the binary? @Enivex

Yes. If Typst intends to convert LaTeX users, it has a strong interest in offering comparable capabilities out of the box. This includes the one default LaTeX text/math font.
As a separate note: I was surprised to find no compiler warning or error when the target font was missing.

Maybe we can find another way to distribute often-used fonts

I think the platform specific package managers make the most sense here– although it's worth noting that I was unable to find New Computer Modern on homebrew for macOS. The unfortunate side effect of course, is that system fonts become an implicit dependency of a project. Maybe it would make sense to have a system package à la brew install typst-fonts that just redistributes a large list of popular fonts, similar to mactex?

Alright.


Get custom domain name

Custom domain name purchased and assigned to GitHub Pages repo under https://github.com/OpenDataScotland/jkan


Small changes to use the role from CentOS

Description of the Change
In order to override package names, I moved the package definition array out of vars/main.yml and into defaults/main.yml. This will make it possible for me to override whats defined in the array. I need to override it when using the role in CentOS since package "ruby-dev" is named "ruby-devel".
Benefits
Better support for other OS:es than Ubuntu
Possible Drawbacks
None i know about
Applicable Issues
Use the role in CentOS

Could this be merged?

Could this be merged ?


"Create file" and "Create folder" buttons in file pane should not create items, only queue them to be made if name form is submitted

Currently, clicking these buttons in the file pane UI immediately creates a new file or folder with the name "Untitled" and an index suffix, then sets that new item to be in "renaming mode" so you can immediately set its name.

Instead, we should follow the example of VS Code. Clicking those buttons opens a UI form to set its name, and if you submit this form the file or folder is then created. If you cancel with Esc or by blurring focus on the input no file system access occurs.
This should include an Electron E2E test or Vitest component test of the behavior.

Duplicate of #3248


First init freeze; Blackbox when animated (Android)

Hi there,
I've added react-native-maps inside to a one tab managed by react-native-scrollable-tab-view component. With that component you can navigate between tabs like you do in Youtube or Periscope apps.
But when I navigate from one tab to another one which contains map component, it freezes for a moment (can be seen on the video below). Then, if I tap and scroll between the tabs (the both tabs are visible), the black container behind the map is visible (this container isn't mine, it belongs to the map).
Here is the link to the video where both of issues can be clearly seen.
Is there any way to solve these issues or any work around?
Thank you for your afford.

The freeze comes from the fact that the View was not created before you scrolled. This is surely because your tab manager creates it only when it needs to be rendered.
For the blackbox, avoid putting a MapView in a swipable container for the moment.

@VinceBT Thank you for your reply.
The freeze I kinda solved with setting property loadingEnabled=true for <MapView /> component.
As for blackbox, I can't avoid putting MapView in a swipable container because I want to get a Periscope similar behavior for the map. It would be cool to swipe horizontal tabs with one contained a map. Can you point where can be a problem with blackbox? Is it because of native map can't re-render fast enough when moving? Or is it because of JS<->Native bridge or React Native is slow? Is there any tricks or hacks that would help here?
And I have one more question.
Now the all area of the MapView is draggable. Is it possible to make it all draggable but except 10-15 pixels on the border to make it possible to kinda bubble up user events? Like we can drag over the map but swipe it's tab on the borders -> http://i.imgur.com/R59QJCh.png

I think the problem comes from the bridge between Native and JS.
I know that when you put a View on top of the MapView it lets touch events pass if it has no backgroundColor or if there is nothing to be rendered inside.
Try adding after your MapView :
<View style={{ position: 'absolute', bottom: 0, left: 0, right: 0, height: 100, backgroundColor: 'transparent' }} />

The backgroundColor: 'transparent' is a quick trick to render an invisible View that can receive touch events.

I've tried this hack but it didn't help... ViewPagerAndroid  is used as a container for tabs and it consumes a bunch of Views but doesn't react to nested components like a View on top of MapView

Even if your page view is designed like this ? :
<View style={{ flex: 1, position: 'relative' }}>
  <MapView
    style={{ flex: 1 }}
    ...otherProps />
  <View style={{
    position: 'absolute',
    bottom: 0, left: 0, right: 0,
    height: 100, backgroundColor: 'transparent' }} />
</View>

If I make a page like this, the drag events from visible are propagated to underlying view only it it's not overlapped by MapView. I mean, if there's a free space between MapView and underlying View, it is draggable via transparent top View. But drag events made over the MapView neither makes page is being swiped nor changes the map's position

On latest (0.19.0) does not happen


How to generate a general affinity map (or .dlg file)?

Hi,
I am working with MolDrug (which is a drug-oriented optimizer on the chemical space). During the optimization steps I am generation new molecules that I do not know what type of atoms they will have.
One of the project is related with a zinc metalloprotein. I was following the Docking with zinc metalloproteins tutorial. However the step 3. Generating affinity maps will be too time consuming. I am wondering if there is a way to generate only one time the grid maps for all the possible ligand_types of atoms that accept AutoDock and use this grid for all the generated molecules.
Of course, if the new genereated molecule does not have any valid AutoDock atom type we could just not perform the docking.
A lot of thanks in advance.

Hi, @diogomart
Thanks for your quick replay. I have a final question. In the script, if I got it right, the definition of the atom types for the receptor is:
supported_atypes = ['HD', 'C', 'A', 'N', 'NA', 'OA', 'F', 'P', 'SA', 'S', Cl', 'Br', 'I', 'Mg', 'Ca', 'Mn', 'Fe', 'Zn', 'H', 'OC']
And for the ligand is the line inside of the string definition of the gpf:
ligand_types HD C A N NA OA F P SA S Cl Br I H
I guess that it is not expected that the ligand has metal ions but what about OC atom type. What is that atom.?

The OC is an experimental type, the preparation tools described in the docs do not use it. Autogrid will complain about OC atoms in the receptor.

I guess that it is not expected that the ligand has metal ions.

That's correct, this script does not expect metals in the ligand.

Thanks again! I will close here


Nested Navigation isn't working correctly in Nav 2.0, when it comes to more than 3 nested Pages

If I want to navigate to 3 nested pages it works well

/A/B/C (like from A-> B, B->C ) while using  Get.rootDelegate.toNamed method
/A/B/C/D (like from A-> B, B->C works till here, from C to D it does not work ), lets say I have only static files without parameters

**Reproduction code
GetPage(
          preventDuplicates: true,
          name: "/home",
          page: () => HomeView(),
          bindings: [
            HomeBinding(),
          ],
          title: "Home",
          children: [
            GetPage(
                middlewares: [
                  EnsureAuthMiddleware(),
                ],
                title: "Dashboard",
                name: "/dashboard",
                page: () => DashboardView(),
                binding: DashboardBinding(),
                children: [
                   GetPage(
                    title: "page C",
                    name: "/pageC",
                    middlewares: [
                      EnsureAuthMiddleware(),
                    ],
                    page: () => PageC(),
                    binding: PageCBinding(),
                    children: [
                      GetPage(
                        title: "PageD",
                        name: "/pageD",
                        page: () => PageD(),
                        binding: PageDBinding(),
                      ),
                    ],
                  ),
                ]),

To Reproduce
Steps to reproduce the behavior:
// until pageC If I write below code it navigates
           InkWell(
                onTap: () {
                  Get.rootDelegate.toNamed('/home/dashboard/pageC');
                }

// If I am in PageC then  call the below code
           Get.rootDelegate.toNamed('/home/dashboard/pageC/pageD'); // this code seems to be not working..

Flutter Version:
2.8.1
Getx Version:
^4.6.1
Describe on which device you found the bug:
ex: Simulators iOS 15.2... Android api 30
I am not sure whether I am using proper navigator 2.0 methods correctly.

Yes, I am also facing the same issue.

@ahmednfwela Is it a default behavior or... , could you please help me out with the above ?

I will have to see a complete reproduction code to help

@ahmednfwela
example_nav2_nested.zip
please find the attachment. example file uses getx version 4.6.1. Please look for goto PageC button inside the dashboard screen.
And
I have tried to clone the latest Getx Link repo,  nav2 example it has some issues while pressing bottom bars. it does not respond.

it's because of an issue that has been brought up multiple times, you forgot to specify anchorRoute

@jonataslaw your implementation is causing problems for long routes, please revert it to my implementation here https://github.com/jonataslaw/getx/pull/1652/commits/360c76c12da951d75073b0afec16fcc611664f64

@ahmednfwela  thank you so much for the above will give a try..  please check the example_nav2 project. it does not work correctly.


Python print statement output is shown in logs after the complete execution of script

Expected Behavior
The print statement output should be shown Immediately in the logs when the print line is executed
Actual Behavior
The print statement output is shown in the logs after the execution of complete script
Steps to Reproduce the Problem

Create a simple task which consists of python script

apiVersion: tekton.dev/v1alpha1
kind: Task
metadata:
  name: check-python-script
spec:
  steps:
    - name: python-example
      image: python  # contains python
      script: |
        #!/usr/bin/env python3
        import time
        for i in range(100):
          time.sleep(1)
          print('Sleeping for 1 second')

Start the task using command tkn task start check-python-script
check the logs of the task using the command tkn taskrun logs -f

Additional Info
Tested similar scenario with shell script using the echo command, the echo output were shown immediately in the logs

I'm reasonably sure this is a python problem rather than a Tekton problem.  Try updating the print statement to the following:
print('Sleeping for 1 second', flush=True)

For me this makes the print statements appear every second.  I'm going to close this issue but please feel free to reopen if you think there's something Tekton can do more here.


BatchNorm does not correctly compute mean/variance across workers when they have different batch sizes

System information
Not relevant
Problem you have encountered:
The code at
https://github.com/google/flax/blob/9fe386508e35c31ca3e1f151bf1a4a87d71486ae/flax/linen/normalization.py#L82-L93
shows that the cross-device mean is computed by the "mean of per-device mean". This is correct only if each device has the same number of elements to normalize over. If they have different batch sizes (or different spatial dimensions in case of images), the result mean & variance is no longer the same as normalizing all samples together.
What you expected to happen:
Although this feels mathematically wrong, in my experience in detection models, this behavior is OK  as long as we compute correct gradients. I also mentioned this in Appendix A.6 of https://arxiv.org/pdf/2105.07576.pdf.
So to me it may be fine to keep things as is, as long as this is documented clearly. But fixing the behavior is probably better.

Hey @ppwwyyxx

If they have different batch sizes (or different spatial dimensions in case of images), the result mean & variance is no longer the same as normalizing all samples together.

To my knowledge there is no way of getting different array shapes per device in jax given the SPMD design. Is there a realistic scenario you have in mind?

@cgarciae I'm new to jax so I might be wrong, but to my knowledge, when using pmap in a multi-host / multi-process set up, each host independently processes its own data and can have data with different shapes, e.g.:
      x = jnp.ones((2, 4))

      def func(x):
          y = x[:jax.process_index()+1]
          print("Y:", y.shape, y)
          sum = jax.lax.psum(y.sum(), axis_name="i")
          return sum

      print(jax.pmap(func, axis_name="i")(x))

when using pmap in a multi-host / multi-process environment, each host independently processes its own data and can have data with different shapes

Each host compiles the program independently indeed. But correct behavior depends on each host compiling an identical program (same ops and shapes). It might sometimes work if you violate this assumption but it is by not garantueed.

Fair enough, I'll close this then!


Docs/managers

Codecov Report

Merging #15 (30a9568) into master (3b409fa) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #15   +/-   ##
=======================================
  Coverage   94.04%   94.04%
=======================================
  Files          12       12
  Lines         856      857    +1
  Branches       54       54
=======================================
+ Hits          805      806    +1
  Misses         42       42
  Partials        9        9

Impacted Files
Coverage Δ

fastapi_mongodb/managers.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


[entt] Update to 3.11.0

Describe the pull request

What does your PR fix?

Updates EnTT to 3.11.0

Which triplets are supported/not supported? Have you updated the CI baseline?

No change to triplet support, tested on x64-windows

Does your PR follow the maintainer guide?

Yes

@Loghorn, when I tested the feature experimental , I got this error, Is this intentional?

@JonLiu1993 Yes: the experimental feature is intended to get the latest experimental branch from the EnTT repository, so it doesn't specify a commit or tag, but just the branch name; in this case vcpkg requires that you add the --head parameter to the command line: vcpkg install entt[experimental] --head

Feature experimental is tested successfully in the following triplet:

x86-windows
x64-windows
x64-windows-static

Thank you!