Going to URL /Default/ gives "Undefined index: Default" PHP notice

I notice that if you navigate to /Default/ URL, you get PHP notice in Notice: Undefined index: Default in <path to vendor>/library/Zend/Controller/Dispatcher/Standard.php on line 397
I think issue is in Zend_Controller_Dispatcher_Standard::isValidModule() where there's two comparison, both are turned to lower cases first. But since default is not Default, its not working. When taking both strtolower() calls away, site return just 404 without stack trace.
I could provide PR, but I'm not 100% certain that that fix does not have any side effects, or are those strtolower() calls there for reason. Can someone confirm this is indeed a bug and I'm not doing something wrong?

From memory all ZF1 url have always been in lower casing and even camelCase controller/view classes are converted to lowercase dash separated.

Its more like problem for getting potential automated error notices from system, when bot (or human) goes to that URL. If someone would somehow have notices visible on production server, it would also expose used framework among installing path, but that's quite edge case and bad configuration. Unnecessary error logging is more of an issue.

I see your point now ^^
While I don't agree that notice should be shown under your production server, I agree that in such case it would have been wiser to return a 404.
I'm not sure stripping the lower case is the best fix here.

That's why I wanted to ask before making PR, since it looks like that lowercases are there for reason, and I'm not by any means expert with Zend logic. One option would be to check if array key exists before using it. That wouldn't break anything, but in most cases is quite unnecessary check. Best fix would be to somehow make Default to behave like any other word. But I have no idea how to achieve that, since I don't quite understand what exactly causes word Default to behave as it does.

Closing due to inactivity


Spans are not connected after upgrading spring-web-starter

Dear Community,
Different services spans can not be connected after upgrade below old version
opentracing-spring-jaeger-starter:0.1.4
opentracing-spring-web-starter:0.3.3

to new version
opentracing-spring-jaeger-starter:2.0.0
opentracing-spring-web-starter:2.1.0

Before the upgrading, everything is ok.
For example, Service A call B and C, we inject the tracing context to the HTTP header when calling Service B and C.  The sampler is ratelimiting 1. Span Calling Service B and Calling Service C are created in Service A and under the root span Service A.  Span Service B is recorded in service B. Span Service C is recorded in service C.
Before upgrading, we got below three level tracing: one root span, two level 2 spans and two level 3 spans,
Service A
  Calling Service B
    Service B
  Calling Service C
    Service C

Even we changed the sampler to any, above tracing is working as expected.
After upgrading, we can only got 1 root span and two level 2 spans which all belong to service A:
Service A
  Calling Service B
  Calling Service C

But when we tried to change the sampler to const 1 in Service A. we got three individual tracing:
1:
Service A
  Calling Service B
  Calling Service C

2:
GET
  Service B

3:
POST
  Service C

in tracing 2 and 3, there has two unknown span: GET and POST. When looked at the span detail. I found the field Component has the value java-spring-webclient which is also not seen in old version. I am guessing the Spring Webclient created a root span solely. I didn't look at the http packet to see what exactly the header are, but from the result, I think Service B and Service C treat the span as the root span. Is our injected span got overwritten? and also sometimes the results related to the sampler?
I am not sure this is the changes to new version or other changes resulted in the weird behavior. Thanks for your information and opinion.

hi @lubinson, what client/technology are you using to create request from service A to services B and C?
Ultimately would you be able to share a small reproducer?

@pavolloffay
Thanks for your reply. After a deep looking, I found the Spring Webflux Webclient is auto injected with a filter TracingExchangeFilterFunction. This actually starts a new Tracing (Tracing 2 and Tracing 3) to Service B or Service C, which the tracing starts with span  GET and POST. And I think this injection overwrote the tracing context we injected so the linked spans are broken.
After I removed the auto injected filter, everything is ok as before. I think this is a change of new version. Its better to make it configurable, especially for upgrading cases. And further, despite of my case,  I am not sure this injection created span will connect spans before and after?
The reproducer is a little bit hard, because it is integrated with Spring Boot and other code. But the basic case is All Services are spring boot applications instrumented with Jaeger, Client call Service A, Service A processes its business and then uses auto injected WebClient.builder() (this Webclient is the one mentioned above) to call Service B and Service C. Then you will see the happening. Pls be note that Service A processing has multiple threads switching, we use some context to take the root tracing context and inject it to the HTTP header then use Webclient to call Service B and C.
Thanks.

@lubinson Since you have already done an initial investigation and have an idea of how to fix the problem, care to create a PR for it?


Teamviewer detects "commercial use"; exits after 5 minutes

I get cut off after a few minutes and locked out because it says I'm using it commercially.
This doesn't happen when I connect to the same host with the Windows client via Virtualbox.
Host OS is Fedora 23.
Diagnostic info:
$ docker version
Client:
 Version:      1.11.0
 API version:  1.23
 Go version:   go1.5.4
 Git commit:   4dc5990
 Built:        Wed Apr 13 18:55:58 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.11.0
 API version:  1.23
 Go version:   go1.5.4
 Git commit:   4dc5990
 Built:        Wed Apr 13 18:55:58 2016
 OS/Arch:      linux/amd64

$ ./check-config.sh
warning: /proc/config.gz does not exist, searching other paths for kernel config ...
info: reading kernel config from /boot/config-4.3.5-300.fc23.x86_64 ...

Generally Necessary:
- cgroup hierarchy: properly mounted [/sys/fs/cgroup]
- CONFIG_NAMESPACES: enabled
- CONFIG_NET_NS: enabled
- CONFIG_PID_NS: enabled
- CONFIG_IPC_NS: enabled
- CONFIG_UTS_NS: enabled
- CONFIG_DEVPTS_MULTIPLE_INSTANCES: enabled
- CONFIG_CGROUPS: enabled
- CONFIG_CGROUP_CPUACCT: enabled
- CONFIG_CGROUP_DEVICE: enabled
- CONFIG_CGROUP_FREEZER: enabled
- CONFIG_CGROUP_SCHED: enabled
- CONFIG_CPUSETS: enabled
- CONFIG_MEMCG: enabled
- CONFIG_KEYS: enabled
- CONFIG_MACVLAN: enabled (as module)
- CONFIG_VETH: enabled (as module)
- CONFIG_BRIDGE: enabled (as module)
- CONFIG_BRIDGE_NETFILTER: enabled (as module)
- CONFIG_NF_NAT_IPV4: enabled (as module)
- CONFIG_IP_NF_FILTER: enabled
- CONFIG_IP_NF_TARGET_MASQUERADE: enabled (as module)
- CONFIG_NETFILTER_XT_MATCH_ADDRTYPE: enabled (as module)
- CONFIG_NETFILTER_XT_MATCH_CONNTRACK: enabled (as module)
- CONFIG_NF_NAT: enabled (as module)
- CONFIG_NF_NAT_NEEDED: enabled
- CONFIG_POSIX_MQUEUE: enabled

Optional Features:
- CONFIG_USER_NS: enabled
- CONFIG_SECCOMP: enabled
- CONFIG_CGROUP_PIDS: enabled
- CONFIG_MEMCG_KMEM: enabled
- CONFIG_MEMCG_SWAP: enabled
- CONFIG_MEMCG_SWAP_ENABLED: enabled
- CONFIG_BLK_CGROUP: enabled
- CONFIG_IOSCHED_CFQ: enabled
- CONFIG_BLK_DEV_THROTTLING: enabled
- CONFIG_CGROUP_PERF: enabled
- CONFIG_CGROUP_HUGETLB: enabled
- CONFIG_NET_CLS_CGROUP: enabled
- CONFIG_CGROUP_NET_PRIO: enabled
- CONFIG_CFS_BANDWIDTH: enabled
- CONFIG_FAIR_GROUP_SCHED: enabled
- CONFIG_RT_GROUP_SCHED: missing
- CONFIG_EXT3_FS: missing
- CONFIG_EXT3_FS_XATTR: missing
- CONFIG_EXT3_FS_POSIX_ACL: missing
- CONFIG_EXT3_FS_SECURITY: missing
    (enable these ext3 configs if you are using ext3 as backing filesystem)
- CONFIG_EXT4_FS: enabled
- CONFIG_EXT4_FS_POSIX_ACL: enabled
- CONFIG_EXT4_FS_SECURITY: enabled
- Network Drivers:
  - "overlay":
    - CONFIG_VXLAN: enabled (as module)
- Storage Drivers:
  - "aufs":
    - CONFIG_AUFS_FS: missing
  - "btrfs":
    - CONFIG_BTRFS_FS: enabled (as module)
  - "devicemapper":
    - CONFIG_BLK_DEV_DM: enabled
    - CONFIG_DM_THIN_PROVISIONING: enabled (as module)
  - "overlay":
    - CONFIG_OVERLAY_FS: enabled (as module)
  - "zfs":
    - /dev/zfs: missing
    - zfs command: missing
    - zpool command: missing

I'm not sure what to tell you, this is an issue with how teamviewer determines whether the application is being used commercially or not. My recommendation would be try and raise the concern with them. I could also try running the container with network host, although I'm not sure if that will have any baring on the issue. I'm not sure if they just use a mac address to determine if the machine is unique or not.


Extension not detecting all available kernels

Environment data

VS Code version: 1.81.1
Jupyter Extension version (available under the Extensions sidebar): v2023.7.1002162226
Python Extension version (available under the Extensions sidebar): v2023.14.0
OS (Linux Ubuntu) and version: 22.04
Python and/or Anaconda version: 3.11
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): XXX
Jupyter server running: Remote

Expected behaviour
Only user based kernels are displayed, other jupyter kernels saved in different path e.g. /opt/kernels/kernel1 are not detected.
Actual behaviour
start a remote vs code session on  compute node, install both python and jupyter extensions, open a notebook select kernel ...
just an information all not detected kernels have a bash script as argv:
{
 "argv": [
     "start-kernel.sh",
     "{connection_file}"

and not directly
"argv": [
     "/home/user/kernels/new-kernel/bin/python",
     "-m",
     "ipykernel_launcher",
     "-f",
     "{connection_file}"

Please could you enable logging as follows:

Open settings & go into Jupyter->Logging
Change the value to verbose
Reload VS Code,
Attempt to repro the issue & then copy the output from the Jupyter output panel.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


chore(main): release pixels 0.22.0

:robot: I have created a release beep boop
0.22.0 (2024-08-30)
Features

pixels: add underline prop to Breadcrumb (16a6fa7)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

pixels-v0.22.0
:sunflower:


[helm]support config override in starrocksFESpec/starrocksBeSpec/starrocksCnSpec

Description
https://github.com/StarRocks/starrocks-kubernetes-operator/issues/494
Test
1) Test config spec setting
with values1.yaml, run
helm template --debug kube-starrocks --values kube-starrocks/values.yaml --values kube-starrocks/values-1.yaml

# spec for component be, provide storage and compute function.
starrocks:
  starrocksFESpec:
    configyaml:
      query_port: 9031
      foo: FEAdditionalValue
      additonal: values
  starrocksBeSpec:
    configyaml:
      be_port: 9061
      foo: BEAdditionalValue
  starrocksCluster:
    enabledCn: true
  starrocksCnSpec:
    configyaml:
      heartbeat_service_port: 9051
      foo: CNAdditionalValue

Got

2) Test config hash
with template below, a minor change in the configyaml values generate totally different hash value
fe-config-hash: "{{template "starrockscluster.fe.config.hash" . }}"
be-config-hash: "{{template "starrockscluster.be.config.hash" . }}"
cn-config-hash: "{{template "starrockscluster.cn.config.hash" . }}"

3) Test with quote in values
  starrocksFESpec:
    configyaml:
      LOG_DIR : ${STARROCKS_HOME}/log
      DATE : '"$(date +%Y%m%d-%H%M%S)"'
      JAVA_OPTS: '"-Dlog4j2.formatMsgNoLookups=true -Xmx8192m -XX:+UseG1GC -Xlog:gc*:${LOG_DIR}/fe.gc.log.$DATE:time"'
      mysql_service_nio_enabled: false

4) Test with configyaml not setting
Verified that it uses the string value in config field
5) Verified default value setting from configyaml

Related Issue(s)
N/A
Checklist
For operator, please complete the following checklist:

[ ] run make generate to generate the code.
[ ] run golangci-lint run to check the code style.
[ ] run make test to run UT.
[ ] run make manifests to update the yaml files of CRD.

For helm chart, please complete the following checklist:

[x] make sure you have updated the values.yaml
file of starrocks chart.
[x] In scripts directory, run bash create-parent-chart-values.sh to update the values.yaml file of the parent
chart( kube-starrocks chart).

I think there is a backward compatibility issue here. That is, if the user has set a custom config by config and then upgrades to use the latest helm chart, because the value of configyaml is always not empty, it will cause the user to use the default content of configyaml as the configuration for FE/BE/CN.

Looking at your code, you directly modified helm-charts/charts/kube-starrocks/values.yaml, should modify helm-charts/charts/kube-starrocks/charts/starrocks/values.yaml.
In scripts directory, run bash create-parent-chart-values.sh to update the values.yaml file of the parent
chart( kube-starrocks chart).

Can we merge the contents of config and configyaml to generate the final result? This way, there will only be one set of configurations in values.yaml. At the same time, allow users to add incremental configurations through configyaml.

I think there is a backward compatibility issue here. That is, if the user has set a custom config by config and then upgrades to use the latest helm chart, because the value of configyaml is always not empty, it will cause the user to use the default content of configyaml as the configuration for FE/BE/CN.

I can set configyaml to {} to address this issue. would this solution OK with you?

update

Can we merge the contents of config and configyaml to generate the final result? This way, there will only be one set of configurations in values.yaml. At the same time, allow users to add incremental configurations through configyaml.

I am afraid this hybrid mode will cause more confusion, and increase the chance of making mistakes

I think there is a backward compatibility issue here. That is, if the user has set a custom config by config and then upgrades to use the latest helm chart, because the value of configyaml is always not empty, it will cause the user to use the default content of configyaml as the configuration for FE/BE/CN.

I can set configyaml to {} to address this issue. would this solution OK with you?

This is LGTM. The method I mentioned seems quite difficult to implement, and splitting the configuration into two blocks doesn't seem like a good choice.

Can we merge the contents of config and configyaml to generate the final result? This way, there will only be one set of configurations in values.yaml. At the same time, allow users to add incremental configurations through configyaml.

if both config and configyaml has the same key, which value shall be overriden? I am afraid this hybrid mode will cause more confusion, and increase the chance of making mistakes

Agree with you


Auto completion inconsistency

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
Hey, over the past few years with Atom I've always discovered this bug but never eally had the motivation to do something about it. In the past few weeks I tried to fix it but havent really found a solution so I mad a gif and wanted to show you.
Basically im writing my code like usual, for example I write dis, then I press enter so Atom auto completes it to display: then I write for example fl for flex behind it.
But this does not always work. >>>

In the given example you can see that with justify-content it does not autocomplete the space-between, it just brings me to a new line.
Note that this is not an issue with justify-content itself, this also happens with color, display, or whatever.
If you write continously code it is really annoying.
Someone had similiar issues?
My mates who also use Atom at least have this issue as well. We usually always use enter to autocomplete the code, not tab.
We couldn't reproduce it in safe mode.
Reproduces how often: About 60% to 70% of the time.


daq.Gauge prop 'showCurrentValue' value is misaligned without 'label' property also

Running the "Current Value and Units" section example from the Dash DAQ Gauge chapter and omitting the label property results in a very misaligned current value:

With the label argument provided:

The current value is centered relative to the viewport instead of the component, I suspect because it's missing the flex rules in the direct child of the outermost .daq-gauge--light container (applied to a hashed styled-components class, but also using the daq-gauge--light__label class). This .daq-gauge--light__label, <div> container is missing its class attributes when there is no label prop provided to daq.Gauge (although the <div> element itself is there either way).
It makes sense to have use cases that display the currentValue property without the label property.
Minimal example:
import dash
import dash_daq as daq
import dash_html_components as html

app = dash.Dash(__name__)

app.layout = html.Div([
    daq.Gauge(
        showCurrentValue=True,
        # label="Default", # uncomment to have normal "currentValue" alignment
        value=6
    )
])

if __name__ == '__main__':
    app.run_server(debug=True)

Still broken as of 1.21.
Temporary workaround is to add a label string containing a single space in it (e.g. " ") to each gauge

Still broken as of 1.21.
Temporary workaround is to add a label string containing a single space in it (e.g. " ") to each gauge


Add languages to the extension description

On the extension description page https://chrome.google.com/webstore/detail/redmine-time-tracking/ldcanhhkffokndenejhafhlkapflgcjg, you need to add German and Russian languages.
https://yadi.sk/i/DmO2AsxQk3YMgA

Thanks for the hint.
I have adjusted the extension to list all languages. It will be displayed as soon as Google is done with the review.


Layout creation failing

I am following the tutorial on Customizing the layouts and components for your site
The layout creation is not working as expected.
After i run the command:
npm run create-layout -- --type <content type name>
The content types are fetched properly after providing credentials, but the layout creation fails.
Please find the logs below:
Pull started
Pull complete. Pulled 18 artifacts successfully. See wchtools-cli.log for more information on the operation.
Creating the layout, layout mapping, and the angular component.
D:\Watson_projects\WCH\wch-site-application\node_modules\ibm-wch-sdk-cli\node_modules\ts-simple-ast\dist\manipulation\code\getNewInsertCode.js:5
    const { structures, newCodes, parent, getSeparator, previousFormattingKind,
nextFormattingKind } = opts;
          ^

SyntaxError: Unexpected token {
    at exports.runInThisContext (vm.js:53:16)
    at Module._compile (module.js:414:25)
    at Object.Module._extensions..js (module.js:442:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:313:12)
    at Module.require (module.js:366:17)
    at require (module.js:385:17)
    at Object.<anonymous> (D:\Watson_projects\WCH\wch-site-application\node_modules\ibm-wch-sdk-cli\node_modules\ts-simple-ast\dist\manipulation\code.js:6:10)
    at Module._compile (module.js:435:26)
    at Object.Module._extensions..js (module.js:442:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:313:12)
    at Module.require (module.js:366:17)
    at require (module.js:385:17)
    at Object.<anonymous> (D:\Watson_projects\WCH\wch-site-application\node_modules\ibm-wch-sdk-cli\node_modules\ts-simple-ast\dist\manipulation.js:7:10)
    at Module._compile (module.js:435:26)

npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "run" "create-layout" "--" "--type" "BookLaunch"
npm ERR! node v4.2.4
npm ERR! npm  v2.14.12
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>create-layout: `echo Pulling the content typ
es. Enter the user password for your WCH tenant: && wchtools pull -t -I --dir sr
c/wchLayouts && echo Creating the layout, layout mapping, and the angular compon
ent. && ibm-wch-sdk-cli create layout --data src/wchLayouts --src src/app --scss
 --flat "--type" "BookLaunch"`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>create-layout script 'echo Pul
ling the content types. Enter the user password for your WCH tenant: && wchtools
 pull -t -I --dir src/wchLayouts && echo Creating the layout, layout mapping, an
d the angular component. && ibm-wch-sdk-cli create layout --data src/wchLayouts
--src src/app --scss --flat "--type" "BookLaunch"'.
npm ERR! This is most likely a problem with the wch-site-application package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     echo Pulling the content types. Enter the user password for your WC
H tenant: && wchtools pull -t -I --dir src/wchLayouts && echo Creating the layou
t, layout mapping, and the angular component. && ibm-wch-sdk-cli create layout -
-data src/wchLayouts --src src/app --scss --flat "--type" "BookLaunch"
npm ERR! You can get their info via:
npm ERR!     npm owner ls wch-site-application
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     D:\Watson_projects\WCH\wch-site-application\npm-debug.log

It looks like the problem might be caused by using an old version of node. We require Node 6 or higher. Can you please retry with Node 6?

Also, I would recommend a recent version of Node6 (eg, 6.11.x) to pick up all security and functional fixes to node.  Early 6.x Node had some issues that caused problems for wchtools, but many are using wchtools on 6.10.x and 6.11.x.      I will ask the documentation team to fix that tutorial you refer to above that states Node4 or later.

@ThomasHurek @mburati The issue was resolved by upgrading Node to version 6 ( currently using 6.11.4).


ci: increase timeout for a few //test/common/stats:... tests.

Signed-off-by: Piotr Sikora<EMAIL_ADDRESS>

/retest

@PiotrSikora: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

test-tsan_envoy_release-1.5
25757b493afe73039161c50a7dbef1c83d22afba
link
/test test-tsan_envoy_release-1.5

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@PiotrSikora: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

test-tsan_envoy_release-1.5
25757b493afe73039161c50a7dbef1c83d22afba
link
/test test-tsan_envoy_release-1.5

test-release_envoy_release-1.5
25757b493afe73039161c50a7dbef1c83d22afba
link
/test test-release_envoy_release-1.5

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@PiotrSikora: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

test-tsan_envoy_release-1.5
25757b493afe73039161c50a7dbef1c83d22afba
link
/test test-tsan_envoy_release-1.5

test-release_envoy_release-1.5
25757b493afe73039161c50a7dbef1c83d22afba
link
/test test-release_envoy_release-1.5

test-asan_envoy_release-1.5
25757b493afe73039161c50a7dbef1c83d22afba
link
/test test-asan_envoy_release-1.5

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@PiotrSikora: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

test-release_envoy_release-1.5
25757b493afe73039161c50a7dbef1c83d22afba
link
/test test-release_envoy_release-1.5

test-asan_envoy_release-1.5
25757b493afe73039161c50a7dbef1c83d22afba
link
/test test-asan_envoy_release-1.5

test-tsan_envoy_release-1.5
25757b493afe73039161c50a7dbef1c83d22afba
link
/test test-tsan_envoy_release-1.5

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@PiotrSikora: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

test-asan_envoy_release-1.5
25757b493afe73039161c50a7dbef1c83d22afba
link
/test test-asan_envoy_release-1.5

test-tsan_envoy_release-1.5
25757b493afe73039161c50a7dbef1c83d22afba
link
/test test-tsan_envoy_release-1.5

test-release_envoy_release-1.5
25757b493afe73039161c50a7dbef1c83d22afba
link
/test test-release_envoy_release-1.5

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@PiotrSikora: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

test-tsan_envoy_release-1.5
25757b493afe73039161c50a7dbef1c83d22afba
link
/test test-tsan_envoy_release-1.5

test-release_envoy_release-1.5
25757b493afe73039161c50a7dbef1c83d22afba
link
/test test-release_envoy_release-1.5

test-asan_envoy_release-1.5
25757b493afe73039161c50a7dbef1c83d22afba
link
/test test-asan_envoy_release-1.5

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@PiotrSikora: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

test-tsan_envoy_release-1.5
25757b493afe73039161c50a7dbef1c83d22afba
link
/test test-tsan_envoy_release-1.5

test-release_envoy_release-1.5
25757b493afe73039161c50a7dbef1c83d22afba
link
/test test-release_envoy_release-1.5

test-asan_envoy_release-1.5
6fc3c2e70da9f2471770813ae097063256ecc0e8
link
/test test-asan_envoy_release-1.5

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@PiotrSikora: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

test-release_envoy_release-1.5
25757b493afe73039161c50a7dbef1c83d22afba
link
/test test-release_envoy_release-1.5

test-asan_envoy_release-1.5
6fc3c2e70da9f2471770813ae097063256ecc0e8
link
/test test-asan_envoy_release-1.5

test-tsan_envoy_release-1.5
6fc3c2e70da9f2471770813ae097063256ecc0e8
link
/test test-tsan_envoy_release-1.5

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@PiotrSikora: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

test-asan_envoy_release-1.5
6fc3c2e70da9f2471770813ae097063256ecc0e8
link
/test test-asan_envoy_release-1.5

test-tsan_envoy_release-1.5
6fc3c2e70da9f2471770813ae097063256ecc0e8
link
/test test-tsan_envoy_release-1.5

test-release_envoy_release-1.5
6fc3c2e70da9f2471770813ae097063256ecc0e8
link
/test test-release_envoy_release-1.5

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Add Rahul as a mentor for coala

CC @yash-nisar

I approve since @RJ722 is the current Google Code-in mentor and has been a GSoC student for coala.

Added in #127


⚠ ClamAV has degraded performance

In 2610a65, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 213 ms

Resolved: ClamAV performance has improved in a70836f after 17 minutes.


Should we deprecate $db->getEntityManager()?

Now that we have the ORM facade in the core, should we encourage its use?
I'm not sure if it was noticed but one of the comments in the pull request that implemented these (or the related issue) was that it would probably be wise to advise to use ORM::entityManager($context) instead. Or an alternative way of getting it such as $pkg->getEntityManager().
The reason for this is that one connection can have multiple entity managers attached to them but one entity manager always belongs to one connection. So the single entity manager instance attached to the default connecton might confuse things up.
I don't mean dropping the functions (straight away) but adding a @deprecated notice to the functions within the database connection class (Concrete\Core\Database\Connection).
I'm talking about these functions:
https://github.com/concrete5/concrete5-5.7.0/blob/2e26f4a21d33cc881248fa62cc54308d590a26fb/web/concrete/src/Database/Connection/Connection.php#L19-L25
https://github.com/concrete5/concrete5-5.7.0/blob/2e26f4a21d33cc881248fa62cc54308d590a26fb/web/concrete/src/Database/Connection/Connection.php#L27-L34

+1

What about just add a parameter to the existing getEntityManager($handle) and allow one to get a named instance of the entity manager they are looking for? If they don't specify, then we assume they want the default core or concrete5 entity manager. Just a thought.

@joe-meyer That's of course an option but I don't think the EntityManager particularly belongs to the connection class architecturally which is why the ORM facade was actually added in the first place:
https://github.com/concrete5/concrete5/pull/1700

And what that means is that the connection itself does not have to be aware of any entity managers attached to it.

Yeah, we should deprecate it.

OK, maybe we should keep this open then until this is resolved...?

Fix available in #3472

I added a big long interesting reply to the pull request. I'm interested in everyone's thoughts


Error: Couldn't find a result file

Getting this error when i try to use nexrender
Error: Couldn't find a result file     at ChildProcess.<anonymous><EMAIL_ADDRESS>The following is my json file:

{
"template": {
"src": "file:///E:/Kinetics-v01/kinetic-titles.aep",
"composition": "title01-textA"
},
"assets": [ {
"composition": "title01-textA",
"type": "data",
"layerName": "test",
"property": "Source Text",
"expression": "'Hello'"
}],
"actions":{
"postrender": [
{
"module": "@nexrender/action-encode",
"preset": "mp4",
"output": "encoded.mp4"
},
{
"module": "@nexrender/action-copy",
"input": "encoded.mp4",
"output": "E:/Kinetics-v01/myresult.mp4"
}
]
}
}

I looked the error up on render.js and it seems that the "outputFile" does not exist so it throws this error? but the output path I provided exists.

you can check the logs of the AE in the temp folder, and also check if the rendered file actually exists

you can check the logs of the AE in the temp folder, and also check if the rendered file actually exists

The log file is empty, it only has the aerender version. The rendered file doesn't exist.

something went wrong during rendering inside of after effects
the log should tell about it, at least it usually does

something went wrong during rendering inside of after effects
the log should tell about it, at least it usually does

Fixed by using AE CS6 instead of CC 2019.


GPU fix seems to have started failing on latest High Sierra Updates

Hi
Lately my MacBook Pro has started failing again with the GPU fix applied. The circumstances are a bit unclear to me, but I will try to help analyse as much as I can.  I know the Mid2010 version is not supported for higher versions than High Sierra and will be unnsupported from Agugust 2020, but I am hopeful I can continue using it until then or even later.
The GPU fix worked fine on High Sierra since I started using it in August 2018, but sometime during the spring 2019, after a an update from Apple I started experiencing reboots again (for me the problem mostly manifested itself with reboots, with occasional black screens only). The problem is now not as frequent as it was before the I started using the gpufix, but happens once in a while.
As the documentation states I apply the pathc after every mac os update. And sometimes i have to apply the gpufix a second time, if it crashes and reboots. So two applications of the patch for it to work more permanently.
It might of course not be a gpu problem this time, but seems to behave in the same manner as before I stared using the gpufix.
Is there a chance Apple changed things slightly, so that the gpufix needs an update to stay functional? Any help would be much appreciated.

Hey Timmy, from what I understand you over write the GPU fix overtime you install a software update. This means you have to reinstall the patch every time you do such an update.

On Oct 19, 2019, at 1:34 PM, Timmyyy<EMAIL_ADDRESS>wrote:
Hi
Lately my MacBook Pro has started failing again with the GPU fix applied. The circumstances are a bit unclear to me, but I will try to help analyse as much as I can. I know the Mid2010 version is not supported for higher versions than High Sierra and will be unnsupported from Agugust 2020, but I am hopeful I can continue using it until then or even later.
The GPU fix worked fine on High Sierra since I started using it in August 2018, but sometime during the spring 2019, after a an update from Apple I started experiencing reboots again (for me the problem mostly manifested itself with reboots, with occasional black screens only). The problem is now not as frequent as it was before the gpufix, but happens once in a while.
Sometimes it seems as if applying the fix twice(as in after the second crash and reboot), fixes the problem for a while. When a new mac update comes along another round of applying the fix twice again.
(I will mention I had to refresh the High Sierra installation, due to a bash scripting error on my part that started the classic "rm -rf" on a path that by mistake contained nothing but "/" :( my bad and extremely embarrassing for a professional programmer (my first of the type in 25 years, actually)
It might of course not be a gpu problem this time, but seems to behave in the same manner as before the gpufix was used.
Is there a chance Apple changed thing a little bit in macos, so that the gpu fix needs a small patch to stay functional? Any help would be much appreciated.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/julian-poidevin/MBPMid2010_GPUFix/issues/120?email_source=notifications&email_token=AMDF7PMHRM3IT52YYE7ODMTQPNVMLA5CNFSM4JCRLJD2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HS6OOMA, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMDF7PJ7KLCVO4MZGVSEMFTQPNVMLANCNFSM4JCRLJDQ.

@tfinneid : @jurgenkoppen is right, it's written on the app README.md (on the main page...) :

I am sorry for the misunderstanding but the problem is not solved. As I state in the description I apply the patch after every mac os update. The issue is now that even with the patch applied, it starts failing again. Which it did not do previously.
Please reopen the bug.

@tfinneid does the patcher says the patch succeeded ?

@julian-poidevin yes, the patcher do succeed. Is there any other log than the one in the main gpufix application window or is there some profiling or similar I could do when applying the patch the next times?

I am experiencing the same issue - but on Sierra. Installed the latest software update and reapplied the patch (successfully installed) and the reboots/black screens have been happening to me as well. I even tried to apply twice - going through the process all over again in terminal, etc...

update: installed it again and it seems to work. 3rd times a charm!

I dont understand, why is this bug report closed? As far as I can see there is no new version that fixes the problem, and what @21Slingshot is saying supports this bug report. That something has changed, presumably in later editions of MacOS, so that the patch must be applied several times ,after and OS update, before becoming effective. I am sure that is not what is desired. It should be enough to apply the patch once, after each OS update.

@julian-poidevin, please respond to my question. The bug is not properly addressed and I would like to understand why you think closing this bug report is solves the bug.

My MacBook has been upgraded to Sierra, not High Sierra.  I do sometimes see the patch "fail" and then I have to re-install it.  It may be that High Sierra has changed something to do with updates - meaning it "sees" the fact that you have changed a system file and hence does an unannounced update to correct that problem.  I suspect that happens every now and then on my Mac, because I have not been notified that a typical update needs to be installed.  Would it surprise you that Apple does something like that?
Note too that your laptop is ancient (as is mine).  The only long-term fix is to buy a current machine.  I am not happy about that either but I know that destiny awaits me.
If you have the skills and patience to determine why the patch is failing there are many Mac users who will cheer you on to discover what is happening and fix it.  In the meantime you might check to verify that "auto updates" are disabled.  And to determine if some kind of update is happening when you have not been notified.  While I cannot prove it I believe that happens now and then on my Mac.

@tfinneid Well, @Bl2c5FlA9 has said everything I guess. Thanks for answering @Bl2c5FlA9, I totally share your point.


Make Orchestrators more extensible and simplify the interface

Describe changes
This PR is our attempt at making ZenML orchestrators more easily extendable and abstracting away most of the internal tfx logic.
Orchestrators
The BaseOrchestrator interface now has a single abstract method called prepare_or_run_pipeline(...) which subclasses need to implement. In there they should either run the steps (e.g. the LocalOrchestrator) or write other code that runs the pipeline in a different environment (e.g. writing the kubeflow yaml and uploading it).
Actually running a step is now as simple as calling
self.run_step(step=zenml_step, run_name="some_string", pb2_pipeline=pb2_pipeline) and the BaseOrchestrator implementation takes care of creating the tfx launcher and actually executing it.
Airflow/Kubeflow: In addition to adapting this new BaseOrchestrator interface, we further simplified and cleaned the airflow and kubeflow orchestrators and got rid of the Airflow/KubeflowDagRunner and Airflow/KubeflowComponent.
Entrypoints
For more complex orchestrators we implemented a generic version of an entrypoint that executes a single step. This generic entrypoint (zenml.entrypoints.step_entrypoint.py) can be configured using a step entrypoint configuration (zenml.entrypoints.step_entrypoint_configuration.py).
Small changes

Disables logging when copying a profile. This was printing logs of stack/stack component registration everytime a docker image was built.

Note: As the focus of this PR was to make orchestrators more easily extensible, please make sure to also take a closer look at the comments/docstrings to check if we're missing any crucial information that users would need to implement a custom orchestrator.
Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[ ] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] If I have added an integration, I have updated the integrations table.
[ ] I have added tests to cover my changes.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Other (add details above)

LGTM!


Remove redundant contacts_params attribute from JaxSimModelData

The contacts_params attribute in the JaxSimModelData object is redundant, as the same data is accessible directly through JaxSimModel.contact_model.parameters. This duplication creates unnecessary complexity and increases maintenance overhead. To improve code simplicity and clarity, I'd propose to remove contacts_params from JaxSimModelData and utilizing model.contact_model.parameters directly within the contact models.
Additionally, this change will eliminate the need to initialize both the model and the data for each integration step when computing the contact forces. As a result, the JaxSimModelData instance will be lighter and more efficient, improving the overall performance of the simulation.
The main disadvantage of this change is that the contact parameters for the SoftContacts model can no longer be initialized automatically. However, the contact parameters attribute should be updated after the JaxSimModel object initialization as follows:
import jaxsim.api as js

model = js.model.JaxSimModel.build_from_model_description(model_description=...)

with model.editable(validate=False) as model:

    model.contact_model = jaxsim.rbda.contacts.SoftContacts.build(
        terrain=model.terrain,
        parameters=js.contact.estimate_good_contact_parameters(
            model=model,
            number_of_active_collidable_points_steady_state=4,
            static_friction_coefficient=1.0,
            damping_ratio=1.0,
            max_penetration=0.001,
        ),
    )

Nevertheless, a different way to automatically initialize the SoftContactsParams can be investigated.
An initial version of the proposal can be found at ami-iit/jaxsim@remove_data_contacts_params

In my opinion it would be better to keep the parameters in the JaxSimModelData object rather than the JaxSimModel object. Particularly for SoftContacts, including the terrain parameters in the domain randomization set can be extremely important for sim-to-real applications. If you move them to the model, a vectorized implementation cannot use a unique model operating on a batched data object having different contact parameters.
And this is also the reason why the static and non-static parameters have been split in #257.

In my opinion it would be better to keep the parameters in the JaxSimModelData object rather than the JaxSimModel object. Particularly for SoftContacts, including the terrain parameters in the domain randomization set can be extremely important for sim-to-real applications. If you move them to the model, a vectorized implementation cannot use a unique model operating on a batched data object having different contact parameters.
And this is also the reason why the static and non-static parameters have been split in #257.

Thanks for your suggestions Diego! I opened #299 accordingly


Don’t send WorkDoneProgressRequest to the client if it doesn’t support work done progress

The way this happens in VS Code is that we sometimes tried to send a WorkDoneProgressRequest before initialization was complete. At that stage VS Code isn’t ready to receive work done progress yet.

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test Windows

@swift-ci Please test Windows


Using 'properties' when you mean 'Named Values'

Git also has a feature called 'properties' that does a very different thing, and it was not immediately clear upon reading this that what you meant was API Management's 'Named Values' and not git properties.  The page here: https://docs.microsoft.com/en-us/azure/api-management/api-management-howto-properties similarly uses 'properties' and 'Named Values' interchangeably.  If you mean 'Named Values', say 'Named Values', and do it everywhere in the documentation for clarity and consistency.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 7d0120b7-a5fe-fead-e88b-cf1bab80ba73
Version Independent ID: 7371e474-727d-5bd1-600d-a502e9b17581
Content: Configure your API Management service using Git - Azure
Content Source: articles/api-management/api-management-configuration-repository-git.md
Service: api-management
GitHub Login: @vladvino
Microsoft Alias: apimpm

Hi @cpdohert Thank you for your feedback! We have assigned this issue to the author to review further and take the right course of action.

Good catch, thank you @cpdohert. Addressed in https://github.com/MicrosoftDocs/azure-docs-pr/pull/69232/commits/a4fd5359bbe4e9e5ec54116dcf073a7371698c2d. #please-close


PDF formular not displayed

Attach (recommended) or Link to PDF file here:
Unbenannt.1.pdf
Configuration:

Web browser and its version: Firefox Quantum 66.0.5 (64-Bit)
Operating system and its version: Win 10 Pro
PDF.js version: 1.10
Is a browser extension: Owncloud PDF Viewer

Steps to reproduce the problem:

Upload a pdf with Acrobat form inside
PDF.js displays a error message inside the pdf instead of the pdf.

What is the expected behavior? (add screenshot)
The PDF formular is displayed and can be filled out directly
What went wrong? (add screenshot)
Instead of the pdf the following text is formatted inside a pdf:

Please wait...
If this message is not eventually replaced by the proper contents of the document, your PDF
viewer may not be able to display this type of document.
You can upgrade to the latest version of Adobe Reader for Windows®, Mac, or Linux® by
visiting  http://www.adobe.com/go/reader_download.
For more assistance with Adobe Reader visit  http://www.adobe.com/go/acrreader.
Windows is either a registered trademark or a trademark of Microsoft Corporation in the United States and/or other countries. Mac is a trademark
of Apple Inc., registered in the United States and other countries. Linux is the registered trademark of Linus Torvalds in the U.S. and other
countries.

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension): https://github.com/owncloud/files_pdfviewer

WFM, using PDF.js version 2.2.177.

PDF.js version: 1.10

No such version number has ever existed, please find the latest releases at https://github.com/mozilla/pdf.js/releases

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension): https://github.com/owncloud/files_pdfviewer

That's a different project, and while it may be using the PDF.js library it's not something that we can provide support for; please contact that project directly for support instead.

I suspect that you uploaded the wrong file. The "please wait" text isn't in the PDF at all. I know it from XFA PDFs.

Closing since this is a duplicate of #1773.

I suspect that you uploaded the wrong file. The "please wait" text isn't in the PDF at all. I know it from XFA PDFs.

Yes, the attached PDF is an Example. Instead of the inside I see the text I discribed inside the pdf. I didn't attach the "wrong" pdf. And yes it is with XFA PDFs.

Closing since this is a duplicate of #1773.

sorry, I didn't find this duplicate as I just checked the summary descriptions and #1773 is open since 2012.
Is there any plan to get pdf forms supported (after 7 years as open issue)?

AcroForms support is only experimental and not enabled yet (tracked in #7613).


Upgrade to cats-effect, fs2 3.0.0-M3 builds

This adds some minor api cleanup as well. It shouldn't affect anyone downstream too badly, I think

I think at this point the main unresolved question is whether CE3 will recommend using Dispatcher as an implicit. Currently it's not really set up for that, but it makes this code clearer to do so. But if the library instructs people to pass it explicitly, I'll want to change the signatures or do the api change like I was thinking earlier

recommendation from CE team is to create locally, it's cheap
https://gitter.im/typelevel/cats-effect-dev?at=5fbbc54afe857d71dc8dad83

WDYT about wrapping all ops in a trait and creating a dispatcher as part of that? Then all the methods inside could hopefully return raw streams/sinks (with no constraints, which is a good thing anyway)?

I think it's reasonable, but I wasn't sure. Some thoughts in #84

Anything that is blocking this currently? I have been using @Daenyth code is isolation. Seems to do solve my issue since I am using CE3.

I haven't had time to work on this lately, and I'm no longer using it at work.
I'd say this is probably "good enough" to push just to have something out, but I don't have bandwidth to roll it out


[iOS] MDCCard's ripple effect requires test on iOS

Material Components Card has ripple when Interactable = true. Further test is required when having subviews with background. Subviews should process touch to the parent in order to correctly show the ripple.

Fixed in 386543e920d12831653a9cbd87e35901e762c43f


Let CMake detect toolkits/compilers by default

Addresses issue #59 and fixes cross-platform (i.e. Windows) builds without changing environment variables.
Still missing FPGA part @definelicht.

Nice job Tal, should've done this a long time ago. I'll do the FPGA part.


How can I get frames from Local Camera Track  [Help Wanted]

Description
** Not an issue, a question
I want to get the frames from the local video preview view so I can apply face detection using either Apple Vision or OpenCV. I understand for Apple Vision, they receive a CVSample Buffer that contains a single CVImageBuffer. I was trying to create a custom renderer but don't know what to pass for the VideoFrame. Does the localVideoTrack contain the most recent rendered frame? How do we get the VideoFrame from the localVideoTrack or previewView so I can pass it to an iOS Vision framework during a video chat? When I try to pass self.previewView.frame to the renderFrame function I get the error "Cannot convert value of type 'CGRRect' to expected argument type 'VideoFrame''. I've noticed some other comments about this but it wasn't very clear to me or the suggestions led to some deprecated code. Any steps or advice would be greatly appreciated!
let imageBuff = renderFrame(self.previewView.frame)
func renderFrame(_ frame: VideoFrame) {
    let imageBuffer = frame.imageBuffer
    return
}

Thank you.
Xcode
[11.4.1]
iOS Version
[13.4.1]
iOS Device
[iPhone X]

I personally haven't been able to get this info from the Twilio SDK directly - instead if you create your own "VideoSource" you can own the AVCaptureSession yourself. Then just assign your own class to its sample buffer and forward along the buffer to both consumers - Twilio & your own Vision / OpenCV code.
The twilio capturer is super straightforward to set up:
final class CustomFrameCapturer: NSObject, VideoSource {

    public var isScreencast: Bool
    public weak var sink: VideoSink?

    init(isScreencast: Bool) {
        self.isScreencast = isScreencast
        super.init()
    }

    func requestOutputFormat(_ outputFormat: VideoFormat) {
        sink?.onVideoFormatRequest(outputFormat)
    }

    func deliverCapturedBuffer(
        buffer: CVPixelBuffer,
        orientation: VideoOrientation,
        timestamp: Date = Date()
    ) {
        guard
            let frame = VideoFrame(
                timeInterval: timestamp.timeIntervalSinceReferenceDate,
                buffer: buffer,
                orientation: orientation
            )
        else { return }

        // The consumer retains the CVPixelBuffer and will own it as the buffer flows through the video pipeline.
        sink?.onVideoFrame(frame)
    }
}

when you're creating the real track:

    private lazy var cameraVideoSource = CustomFrameCapturer(isScreencast: false)
    private lazy var cameraTrack = LocalVideoTrack(
        source: cameraVideoSource,
        enabled: true,
        name: "Camera"
    )

then just call:
cameraVideoSource.deliverCapturedBuffer(buffer: buffer, orientation: .up, timestamp: exposureTime)

Most of the times I've found its easier to do any rotations before sending the buffer into the twilio video track.

@ksuhr1 Writing custom VideoSource is one way, however easier ways is to implement a VideoRenderer on a track which doesn't require you to write a custom VideoSource.
We have a work-in-progress PR https://github.com/twilio/video-quickstart-ios/pull/286 which demonstrates custom VideoRenderer implementation. Here is the renderFrame implementation example from the PR.
Let me know if you have any questions.

@AdiAyyakad  Thank you so much for your input! I will try this out

@piyushtank Is there anyway I can checkout this VideoRenderer code and see how all the files work together when running on an iphone? I am trying to figure out how to incorporate the the renderFrame function with the original code of this video-quickstart code. Thank you for your response!

@ksuhr1 did you end up finding a good solution to do this? Currently opting to go for @AdiAyyakad's suggestion.


Udacity bootstrap aws fixes

-F ssh.config needs set in the ProxyCommand too, otherwise we can't
use the nat as an ssh proxy, 10.* addresses get added to known_hosts on
each run, poluting that file for subsequent runs, and the ProxyCommand
attempts to negotiate host key checking.

Environment:
Ubuntu 15.05
OpenSSH_6.7p1 Ubuntu-5ubuntu1, OpenSSL 1.0.1f 6 Jan 2014

make sed command portable

@pmbauer thanks! This might also help you https://github.com/Capgemini/Apollo/commit/eeeb6089f644a4b72265fe838a8f6a30d0ab8640 (in latest master)

Yep, thanks. Using eeeb608


feat(authentication): add LDAP authentication support

This PR introduces a new Authentication section in the Settings of Portainer. It will allow you to choose between Internal (managed by Portainer) and LDAP authentication.
Close #677

You can test this feature using the image portainer/portainer:pr1093 (Linux amd64 build).

Will test that this week

I started my preexisting portainer container with the image portainer/portainer:pr1093.
Within the user management, the password fields were missing and I could not see any options for ldap.

Go to settings, and you will see the new section for LDAP
Sent from my iPad
On 9/08/2017, at 11:47 AM, DimeOne <notifications<EMAIL_ADDRESS>wrote:
I started my preexisting portainer container with the image portainer/portainer:pr1093.
Within the user management, the password fields were missing and I could not see any options for ldap.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_portainer_portainer_pull_1093-23issuecomment-2D321112529&d=DQMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=0nFvQgf1d29kt20P90oNp0mUXWRpskl8btdLmEWDwAA&s=3xZfRP9tlvksnuFSxnGsCmNlPprhh5k5ReFgUzjaXl0&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AWGrldffgtXchwsojWtkFFwRcfSmuFO4ks5sWPOCgaJpZM4Oufnc&d=DQMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=0nFvQgf1d29kt20P90oNp0mUXWRpskl8btdLmEWDwAA&s=g98XCUd_lqHf6AIasbEL6XGbwncNfBA6ZsTYSM4KhFM&e=.

Within the user management, the password fields were missing and I could not see any options for ldap.

When LDAP is enabled, Portainer will automatically hide password management. So that would mean that LDAP is already enabled on your instance.

It seems I must have been blind (or too tired). I was looking at Settings but must have overlooked the Authentication Menuitem.
I was not able to get it working like I did with other applications. This is my home lab which is using a Microsoft Active Directory running on a Windows Server 2012R2. I have been able to configure the connection using other tools like Jenkins, All Certificates are issued by my own pki and are working fine. LDAP has been tested on 389+3268(plain+starttls), 636+3269(SSL) using LDAP Admin.

Not able to login
Even though the connection seems to work while testing it, I was not able to login to portainer using ldap. (Neither with plain nor TLS)
If I create the user as administrator before logging in, I receive:
"http error: User not found or too many entries returned (code=500)"
Otherwise it says:
"http error: Invalid credentials (code=400)"

This may be due to the missing User search configurations.

incomplete TLS Settings
I was trying to configure a tls connection on port 389 which uses starttls. I uploaded only the certificate chain and saved the settings, without being able to test the connection. ( Due to missing tls cert and key ?! ) Upon reloading the settings, all tls items are checked and the test connection button is active. Testing the connections returns an error, that the certificate cannot be found.

I assume that this tls is only used for client authentication?

SSL Connection
After failing to establish a starttls connection, I tested an SSL connection on port 636, which failed too.
Error on failure was "Connection reset by peer"

Setting the protocol explicitly to ldaps:// within the LDAP URL is not supported either.

@DimeOne thanks for the feedback !

Not able to login
Even though the connection seems to work while testing it, I was not able to login to portainer using ldap. (Neither with plain nor TLS)
If I create the user as administrator before logging in, I receive:
"http error: User not found or too many entries returned (code=500)"
Otherwise it says:
"http error: Invalid credentials (code=400)"

There was an issue that was fixed this morning where the User search configurations would not appear when using previous Portainer data (you can pull again portainer/portainer:pr1093 to retrieve the fix).
There User not found or too many entries returned (code=500) error is raised when the LDAP search query returns either no or too many entries. Tho without search settings, it's highly probable that this would be raised.

incomplete TLS Settings
I was trying to configure a tls connection on port 389 which uses starttls. I uploaded only the certificate chain and saved the settings, without being able to test the connection. ( Due to missing tls cert and key ?! ) Upon reloading the settings, all tls items are checked and the test connection button is active. Testing the connections returns an error, that the certificate cannot be found.

SSL Connection
After failing to establish a starttls connection, I tested an SSL connection on port 636, which failed too.
Error on failure was "Connection reset by peer"

I was not able to test the TLS connection, indeed all 3 TLS fields are mandatory so I'm pretty sure it won't work without the TLS trio.

Setting the protocol explicitly to ldaps:// within the LDAP URL is not supported either.

You don't need to specify the protocol in the URL field.
You can ping me on Slack for a more reactive support / communication.

After pulling the latest pr1093 I was able to get LDAP login to work (without tls/ssl).
For AD I used following user search configurations:
BaseDN: CN=Users,DC=example,DC=org
Username attribute: sAMAccountName
Filter: (objectClass=user)
I had to add the users manually, but then I was able to login with said users.
If you could give me a reference to the documentation of the library you used for ldap, I may take a look at encryption.

@DimeOne nice to hear that !
I used the following library for the implementation: https://github.com/go-ldap/ldap (godoc available at https://godoc.org/gopkg.in/ldap.v2)
I've specifically used https://godoc.org/gopkg.in/ldap.v2#DialTLS to establish a TLS connection.
See the createConnection method here: https://github.com/portainer/portainer/blob/feat677-ldap-support/api/ldap/ldap.go#L51
Globally, all the ldap implementation is available in https://github.com/portainer/portainer/blob/feat677-ldap-support/api/ldap/ldap.go

Within the dokumentation is a StartTLS with a basic example that skips certificate verification, more secure options would include a certificate chain, servername within the config: https://godoc.org/crypto/tls#Config
func (l *Conn) StartTLS(config *tls.Config) error
err = l.StartTLS(&tls.Config{InsecureSkipVerify: true})
if err != nil {
log.Fatal(err)
}
The changes I would make:

Change "TLS" switch to "Use Encryption"
Add new Switch for "TLS vs StartTLS"
Add new Switch for "Skip Verification"
require Certificate Chain when no Skip Verification is specified and use in tls.Config.RootCAs
Set servername in tls.Config.Servername
Make Cert and ClientCert optional for Client Authentication - ( bind_dn and password would then be optional )

That should allow using ldap with encryption and validation with TLS and StartTLS. I'm not familiar with go and could provide a patch for that, but I'd be happy to give feedback and test it at home and work :)

@DimeOne
Sweet, let me make some changes and I'll ping you to give it a try :-)

@DimeOne would you be able to join me on Slack as I've got multiple questions ? https://portainer.io/slack/

Never used slack before, I'll figure it out in a few minutes

Thanks everyone for your feedback on this !
@DimeOne Now that it's merged into develop, do you mind testing again the TLS settings ? (I did some cleanup and want to be sure I did not broke anything). You should be able to use portainer/portainer:develop.

Does this work now with Windows containers?

@artisticcheese our instable build is not available as a Windows container yet (only linux amd64 build). Tho it will be released this week !

I get the following after setup of ldap. Is there a limit of the # of users portainer can handle on it's side? I'll dig in a bit more when I have time.
{"err":"User not found or too many entries returned"}

Everything seems to work if I restrict the results to just a single user via the filter option in the "User search configurations".
(uid=%s) <--- doesn't work
(uid=beenanner) <--- does work

@beenanner if you want to use (uid=%s) you can just set the value uid in the username attribute field. This is the attribute on which the username will be matched.

Nice @deviantony, I tried your suggestion and just removed the filter and everything is working! Awesome job and welcomed feature!


Tokenizer

Warning
The tokenizer class you load from this checkpoint is not the same type as the class this function is called from. It may result in unexpected tokenization.
The tokenizer class you load from this checkpoint is 'LLaMATokenizer'.
The class this function is called from is 'LlamaTokenizer'.

Caused by tokenizer = LlamaTokenizer.from_pretrained("decapoda-research/llama-7b-hf", cache_dir="./cache/")
Protential solution is modify tokenizer = LLaMATokenizer.from_pretrained("decapoda-research/llama-7b-hf", cache_dir="./cache/")
or modifypath_to_cache_hfmodel/models--decapoda-research--llama-7b-hf/snapshots/5f98eefcc80e437ef68d457ad7bf167c2c6a1348/tokenizer_config.json
change
{"bos_token": "", "eos_token": "", "model_max_length":<PHONE_NUMBER>000000019884624838656, "tokenizer_class": "LlamaTokenizer", "unk_token": ""} to {"bos_token": "", "eos_token": "", "model_max_length":<PHONE_NUMBER>000000019884624838656, "tokenizer_class": "LLaMATokenizer", "unk_token": ""}

Cc @zoidbb

The cleanest solution would be to convert the original LLaMA weights to HF ones with the script included in the main Transformers branch, but it requires a fair bit of ram.

Yep, this leads to some tokenization artifacts and has started producing garbage outputs.


Wordpress App CrashLoopBackOff

This issue is a:

[ ] Bug Report

If the issue concerns an existing Marketplace application, please tag the maintainer of the application (found in manifest.yaml) below:
Bug Reports
If the issue is a Bug Report, please detail the following:

Expected behaviour
Pod to run as expected
Encountered behaviour
CrashLoopBackOff
Error in log:
[mpm_prefork:notice] [pid 1] AH00170: caught SIGWINCH, shutting down gracefully
The steps to recreate the behaviour that you are reporting
Deploy the app from the marketplace

@saiyam1814 can you reach out to the maintainers please.

@javicv would be great if you can have a look at this issue !

Sorry @saiyam1814  I can't, please contact @DoNnMyTh instead

Will look at it on weekend


Add Amazon Bedrock Anthropic Claude v1 and v2 assistants

Additional assistant if you want to use Claudev1 or v2 hosted via Amazon Bedrock.
You need to set AWS environment variables for your AWS credentials (i.e export AWS_ACCESS_KEY_ID, AWS_SECRET_ACCESS_KEY)
Code is currently region specific to where Amazon Bedrock is hosted.

This PR is for https://github.com/Quansight/ragna/issues/171

Hi there. I have no strong opinions where this goes, and only wanted to contribute if you found it useful. I guess I was confused by the term 'additional infrastructure' given that Bedrock acts kind of like Anthropic, OpenAI, so apologies for the misunderstanding. I put this together for folks who might want to try this out but be using Claude via Bedrock rather than directly via Anthropic. Hope this makes sense.

I'm not sure what a local extension means but if you mean a local fork then if that is the recommendation of the project then I have no issues with that.

apologies for the misunderstanding.

No worries here.

I put this together for folks who might want to try this out but be using Claude via Bedrock rather than directly via Anthropic. Hope this makes sense.

Yeah, and that is fine. Like I stated above, maybe we can have a ragna-aws package in the future that contains just this functionality. However, since we won't accept the PR as is in the near future, I'm going to close this. If our stance on this changes, we can always re-open.

I'm not sure what a local extension means but if you mean a local fork then if that is the recommendation of the project then I have no issues with that.

What I mean is that you can use the implementation you have with Ragna without it being part of the core package. Just throw your implementation in a ragna_bedrock.py module (make sure that is on the PYTHONPATH; you can test with python -c import ragna_bedrock.py) and add this to your configuration file
[core]
# ...
assistants = [
    # ...
    "ragna_bedrock.AmazonBedRockClaude",
    "ragna_bedrock.AmazonBedRockClaudev1",
]

If you start the UI now, you should be able to select and use them. If not, please open an issue.


Refactor template pod annotations

What this PR does / why we need it:
This PR attempts to increase the readability of the pod annotation generation.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/cc @EdDev @phoracek

/cc @EdDev @phoracek

Addressed @EdDev 's comments in this forced push.
Rebased latest master this forced push.

/retest

/cc @enp0s3 could you take a look at this PR ?

@maiqueb Sorry for the late response, will review it

/hold

@maiqueb Would you prefer to add unit tests for the multus annotation in this PR or in a separate PR?

@maiqueb Would you prefer to add unit tests for the multus annotation in this PR or in a separate PR?

I can do it now. I'll remove the hold; please lgtm if you're on board with them @enp0s3 .
Thanks for the review.
/hold cancel

@EdDev is there an action point from me here ? Would you rather have me remove these unit tests ? I think they add some value.
@enp0s3 are you OK with the proposed unit tests ?

@EdDev is there an action point from me here ? Would you rather have me remove these unit tests ? I think they add some value.

I consider testing private functions a bad practice which costs more pain than gain.
It makes sense to accept this when no other choices exists, however, in this case you are saying that the same code flow is already examined by other tests (which I hope exercise the public/exported functions).
IMO you need to reason for make the test code so coupled with the implementation and that is the text that is missing from my part.
I have not voted here as I do not want to block this change, it is much better than before. I have even lgtm it when these tests were not included.

@EdDev is there an action point from me here ? Would you rather have me remove these unit tests ? I think they add some value.

I consider testing private functions a bad practice which costs more pain than gain.
It makes sense to accept this when no other choices exists, however, in this case you are saying that the same code flow is already examined by other tests (which I hope exercise the public/exported functions).
IMO you need to reason for make the test code so coupled with the implementation and that is the text that is missing from my part.

Is it enough If I export the methods on the pool thing ?
What reasoning are you after ? This multus annotation pool is, literally a one trick pony. It only needs to generate a string. And I don't imagine it being called from anywhere else.

I have not voted here as I do not want to block this change, it is much better than before. I have even lgtm it when these tests were not included.


chore: bdd-nh-1586218869 to 0.0.1

chore: Promote bdd-nh-1586218869 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Readme changelog link broken

The changelog in the README doc links to https://docs.nativebase.io/changelog, which gives a 404 error.

@jacraven thanks for reporting, docs are up now. Check it out


Remove wrong path from README

I tested locally and it seems the metrics endpoint isn't under /auth/ so fixing docs to reflect that.

@uberspot what Keycloak version are you testing with? I believe the auth prefix was present when I tested this.

The auth prefix gets removed by default in the Quarkus distribution of Keycloak, it's still present in the legacy wildfly distribution. Version 19 is still distributed in both Quarkus and Wildfly, starting with version 20 there will only be the Quarkus distribution.

Thansk @uberspot , fixed in #160


The bad pixel mask algorithm is flagging all the reference pixels as dead

I ran the jwst_reffiles.bad_pixel_mask code on a set of NIRISS internal flat ramp slope images, and when I looked at the results I found that all the reference pixels are flagged with flag value 1024 and flag value 1 (i.e. DO_NOT_USE) in the output bad pixel mask file.  Whilst it is true that the reference pixels are dead, this should not be flagged in the bad pixel file.

Addressed in #59. Refpix are now stripped off of the mean slope image before sending it to the dead, low QE, and open pixel searches.


Inputting an invalid commit ID builds a passing HTML report with no commit information.

This could be misleading, a warning should be displayed and the HTML report should not be generated if the git id is invalid.

Output error in invalid commit ID and invalid git repository. @neelimas

Output error in invalid commit ID and invalid git repository. @neelimas


Initialization scripts in docker-entrypoint-initdb.d/ not working for Windows containers

In the documentation for the container image in section "Initializing a fresh instance" it is described how to initialize the image with custom JavaScript files. Those can be located in a docker-entrypoint-initdb.d directory.
However, this feature is not yet supported nor working on windows.
Instead just a todo-comment was added to the corresponding Dockerfile.
If the feature is not added yet, I think, it would be good to have a note mentioning this in the documentation.

So far as I know, the environment variables "MONGO_INITDB_ROOT_USERNAME" and "MONGO_INITDB_ROOT_PASSWORD" also don't show any effect in the Windows image.

https://github.com/docker-library/mongo/issues/291#issuecomment-410813034

Thanks for your reply.

If the feature is not added yet, I think, it would be good to have a note mentioning this in the documentation.

I created an issue for changing the docs: https://github.com/docker-library/docs/issues/2271
Issue closed.


RTL in html

Add direction attribute with value rtl in case of arabic language to html tag, so that <html lang="ar"> will become <html dir="rtl" lang="ar"> instead.

If you didn't notice, the class HTMLNode has a method to set the writing direction which is ::setWritingDir()
So, it is possible to set writing direction by using something like that:
$doc = new HTMLDoc();
$doc->getDocumentRoot()->setWritingDir('ltr')

Do you think that I should add a default value for the attribute or keep it empty?

You can default it to ltr unless specified.


Linking structures to their AST nodes to have access to their positions

This far there were only 3 types of structures linked to their AST nodes. Now adding some more expressions and literals to their AST node to have access to their token's position in the query.
Why this can be useful? If you're using JSqlParser to eventually present a query that was modified or maybe show different parts of it in different colors. In that case, you'll have to know the exact position of each of the tokens.
Regarding memory consumption - I have a test set of ~2000 real world queries that I'm testing my changes with. I'm running 100 iterations of that set to make sure nothing changes over time and relatively high volume.
Running the tests on a machine with low resources (1 CPU & 1GB RAM), to make sure there is no excessive use of memory - if something works on that machine, it provides a good indication it will work on strong production environment.

Coverage remained the same at 87.041% when pulling 1212f9dfd0de8d493350d171b111a6f3643138a3 on sh-tomer:link_structures_to_ast_nodes into b1abc6ff39e9c2a5499083b40db59fb4b2af7ff7 on JSQLParser:master.

Agreed and merging right now ...


utils.slugger chokes on unicode

Apparently, having a unicode encoded file works, but if it contains unicode caracters it doesn't.
stjeans1 ~ $ smart_dispatch.py -x -f file.txt -q qwork@mp2 -t 1:10:00:00 launch
Traceback (most recent call last):
File "/home/stjeans1/.local/bin/smart_dispatch.py", line 180, in
main()
File "/home/stjeans1/.local/bin/smart_dispatch.py", line 74, in main
log_filename = os.path.join(path_job_logs, smartdispatch.generate_name_from_command(command, max_length_arg=30))
File "/home/stjeans1/.local/lib/python2.7/site-packages/smartdispatch/smartdispatch.py", line 41, in generate_name_from_command
name = '_'.join([utils.slugify(argvalue)[max_length_arg:] for argvalue in command.split()])
File "/home/stjeans1/.local/lib/python2.7/site-packages/smartdispatch/utils.py", line 33, in slugify
value = unicodedata.normalize('NFKD', unicode(value)).encode('ascii', 'ignore').decode('ascii')
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 1: ordinal not in range(128)
where file .txt contains
echo "éé"

Also fails directly from the command line.
germainm@mp2-ip08 ~$ smart_dispatch.py -q qtest@mp2 launch echo "éé"
Traceback (most recent call last):
  File "/home/larochel_group/.local/bin/smart_dispatch.py", line 199, in <module>
    main()
  File "/home/larochel_group/.local/bin/smart_dispatch.py", line 43, in main
    jobname = smartdispatch.generate_name_from_command(command, max_length=235)
  File "/home/larochel_group/.local/lib/python2.7/site-packages/smartdispatch/smartdispatch.py", line 39, in generate_name_from_command
    name += '_'.join([utils.slugify(argvalue)[max_length_arg:] for argvalue in command.split()])
  File "/home/larochel_group/.local/lib/python2.7/site-packages/smartdispatch/utils.py", line 60, in slugify
    value = unicodedata.normalize('NFKD', unicode(value)).encode('ascii', 'ignore').decode('ascii')
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 0: ordinal not in range(128)


[SPARK-25797][SQL][DOCS] Add migration doc for solving issues caused by view canonicalization approach change

What changes were proposed in this pull request?
Since Spark 2.2, view definitions are stored in a different way from prior versions. This may cause Spark unable to read views created by prior versions. See SPARK-25797 for more details.
Basically, we have 2 options.

Make Spark 2.2+ able to get older view definitions back. Since the expanded text is buggy and unusable, we have to use original text (this is possible with SPARK-25459). However, because older Spark versions don't save the context for the database, we cannot always get correct view definitions without view default database.
Recreate the views by ALTER VIEW AS or CREATE OR REPLACE VIEW AS.

This PR aims to add migration doc to help users troubleshoot this issue by above option 2.
How was this patch tested?
N/A.
Docs are generated and checked locally
cd docs
SKIP_API=1 jekyll serve --watch

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Test build #98077 has started for PR 22846 at commit a6f4d54.

Test build #98077 has finished for PR 22846 at commit a6f4d54.

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

Merged build finished. Test PASSed.

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

@jiangxb1987 @cloud-fan @gatorsmile Could you please kindly review this when you have time?

LGTM, merging to master/2.4!
it conflicts with 2.3, can you send a new PR? thanks!

@cloud-fan Sure, I will send a new PR for 2.3. Thanks you for review this.

@cloud-fan PR for 2.3 is submitted. Please see #22851.


[Event Request] Codeunit 7009 CopyFromToPriceListLine - OnBeforeInsertPriceListLine

Describe the request
Dear ALAppExtensions team,
On behalf of 4PS I would like to request integration event 'OnBeforeInsertPriceListLine' to be added to procedure InsertPriceListLine in codeunit 7009 CopyFromToPriceListLine.
    local procedure InsertPriceListLine(var PriceListLine: Record "Price List Line")
    begin
        InitLineNo(PriceListLine);
        OnBeforeInsertPriceListLine(PriceListLine); //new
        PriceListLine.Insert(true);
    end;

    [IntegrationEvent(false, false)]
    local procedure OnBeforeInsertPriceListLine(var PriceListLine: Record "Price List Line")
    begin
        //new
    end;

Additional context
We have functionality to make it possible to have the same Price List Lines in several companies. Therefore we have added some code to the OnInsert of Price List Line that takes care of inserting the Price List Line in other companies.
But when the procedure InsertPriceListLine of codeunit 7009 CopyFromToPriceListLine is called, inserting the Price List Line should not be done in other companies. Therefore we need an event on this place, to make sure that inserting in other companies is skipped. Unfortunately we cannot use the event OnAfterInitLineNo, because this event is also called in situations where it is necessary to insert the Price List Line in other companies.
When inserting the Price List Header, we have the same issue, but there is already an event OnBeforeInsertHeader.
Internal work item: AB#524157

Work completed on the 11-04-2024.


Unable to update full list property when using component.

Hello, I dynamically fetch the data from my API on vue's ready function. Before that my list contains only one null element. I also use a component to show the data.
On ready I get from my API 10 elements and I paginate the results to show only 4 of them. This works anytime, however if I try to use next, prev or change function it will show the null value from the beggining.
I tried to instantiate Vue with 3 elements on the list ["test1", "test2", "test3"] and after changing the page it will show only those 3 elements not paginated data from API.
Also if you use full[listName] in the component's v-for it will not show the data fetched later via API but the one you instantiated with vue.
I tried to use
this.fullOffers = response.data;
self.$set("fullOffers", response.data);

on the parent element with no luck. I think that the component's full data is not being updated.

Here is the event list.

As you can see the fullOffers prop is being updated only at the beggining and it does not change after the data being fetched via API call. I am unable to update it from the parent's scope.

Hey @Lucassith! I'm not sure I fully understand the context of your problem, but I guess here's the solution.
So, let's take the example of langs list. Let's say that I want to fetch the list from an API, and I want to pass the fetched data to a component which will paginate them and display them.
This means first I need to define that component (I'll call it test-component):
Vue.component('testComponent', {
  template: '#langs-template',

  props: ['fullLangs'],

  data () {
    return {
      langs: []
    }
  }
})

Note how I accept the fullLangs as a prop instead of langs (that's the trick to make it work).
And here's the template for the component:
<template id="langs-template">
  <!-- data -->
  <ul v-paginate:3="langs">
    <li v-for="lang in langs">
      {{ lang }}
    </li>
  </ul>

  <!-- links -->
  <a @click="prevLangsPage()" href="#">
    Prev
  </a>
  |
  <a @click="nextLangsPage()" href="#">
    Next
  </a>
</template>

Now in your parent component you'll need to use that component like this:
<test-component :full-langs="fetchedLangs"></test-component>

Notice how I passed an array called fetchedLangs, which we haven't created yet. Let's do this now in the parent component:
new Vue({
  el: '#app',
  data: {
    fetchedLangs: []
  },

  ready () {
    this.$http.get('/langs')
      .then(function (response) {
        this.fetchedLangs = response.data
      })
  }
});

With that, everything should work as expected, because when the fetchedLangs are filled with the data from the API it'll be automatically assigned to fullLangs in the child component which will change the full version of the list, so it gets paginated properly by the plugin.
Hope this solves your problem.
Thanks for using the plugin :).

Yeah you're right, now it works, thank you! :+1:
Also please take a look on the first issue "inside a component using props". I was following your comment there.
Great plugin, Thank you!


ISSUE-851: Add missing string UDFs in streamline

Added Substring, Trim, Ltrim, Rtrim, Position, Overlay.
Made some backend changes to support overloaded UDFs

With the proposed changes there can be multiple entries (one per overload) in the UDF table (and the UDF api) for a single function. Some changes are required in the UI side to handle this and let the user select the function with the right number of arguments. cc @shahsank3t
The function names of the streamline provided udfs are appendend with _FN to be consistent and avoid conflicts with the calcite defaults. That way when we expose the SQL editor, user can choose to use the streamline functions or the SQL standard ones. E.g. SELECT SUBSTRING(stringField FROM 10) FROM inputStream vs SELECT SUBSTRING_FN(stringField, 10) FROM inputStream. We need to provide some migration sql for the users who would have created rules with the older function names.
This includes changes from https://github.com/hortonworks/streamline/pull/850
Fixes #851

+1


Blake2b: Use OSSL_DIGEST_PARAM_SIZE as settable instead of XOFLEN

BLAKE2 is not really an extensible output function unlike SHAKE as the digest size must be set during the context initialization. Thus it makes no sense to use OSSL_DIGEST_PARAM_XOFLEN.
It is still necessary to use the EVP_DigestFinalXOF() function to get the output for non-default sizes as EVP_DigestFinal() cannot handle non-default sizes.
Fixes #22488
Checklist

[x] documentation is added or updated
[x] tests are added or updated

Having to use EVP_DigestFinalXOF goes against using XOFLEN being "wrong" in the sense that we have an XOF named API that has to used. I know it isn't XOF as discussed - but we should have a non-XOF named API for the final that works like this then.

Having to use EVP_DigestFinalXOF goes against using XOFLEN being "wrong" in the sense that we have an XOF named API that has to used. I know it isn't XOF as discussed - but we should have a non-XOF named API for the final that works like this then.

Do we want such last minute API addition? I do not like this.

IMO we could just rename/alias the EVP_DigestFinalXOF as the name is misleading anyway.

The OTC discussion was about this not being an XOF as the reason to change the parameter name - but if we do that and then the user still has to call a function with XOF in its name that seems inconsistent to me. Renaming the function as that part of it isn't XOF would make sense - but you will need other views expressed on this. This itself is a last minute change and I don't think it does far enough to achieve the objective (of not packaging something up that isn't XOF in an XOF facade)

I still believe this is better than what we currently have in the code. We can add the alias to EVP_DigestSignXOF later.
Alternatively we live with what we have as is.

I've added the alias as that is relatively trivial.

Now we can start the bikeshedding about the alias name.

I don't understand why EVP_DigestFinal_ex() can't be modified to allow output lengths that vary from the default. Sure, there's the EVP_MD_get_size() call, but it can be modified to retrieve the actual length, and retain the EVP_MD_get_size() as a fallback, can it not?

This would be rather non-trivial change - it would require us to make the OSSL_DIGEST_PARAM_SIZE as a ctx gettable and adding a helper function to get it. I can try to do it as an alternative.

I don't like any of the options facing us - 1) do nothing, 2) accept this PR, 3) change to not using the final-xof
3 seems like the right longer term solution but is too late for 3.2 beta and hence 3.2 itself and 2 is reasonable and 1 wasn't the OTC meeting view.
I lean towards this PR as is.

The alternative in #22491 is preferable to this PR IMO.

The alternative in #22491 is preferable to this PR IMO.

Agreed

I am closing this in favor of https://github.com/openssl/openssl/pull/22491


Padding for text label of extralines

Hello and thank u for great chart library!
It would be nice to have padding around text, because for now for make text look nicer we have to use spaces on both sides of text.
Example (with padding):

Example (how it works now):

THank you for your help!

You can use tooltipPadding property in the LineTouchTooltipData.

@imaNNeo thank u for your response.
Sorry if my question wasn't clear from the beginning.
Purple line on screenshots above is HorizontalLine. And HorizontalLine hasn't paddings.

So you want to render a text with some padding around it?

yes, i want to render a text with some padding around it

There is a padding property in the VerticalLineLabel. Does it help?

@imaNNeo thank u for your suggest solution.
VerticalLineLabel.padding add padding to whole label.
Example:
HorizontalLineLabel.padding = EdgeInsets.zero

HorizontalLineLabel.padding = EdgeInsets.symmetric(horizontal: 5)

I am looking for padding for text inside label. So it would be nice looking color frame around text.
Example:
text with padding

text without padding

@imaNNeo VerticalLineLabel -> labelResolver, return only String. Why not a Widget? For example, I would like to put an icon.

@imaNNeo I could take a look at this. We could add margin/padding for all of the chart tooltips as well?

take a look at this. We could add margin/padding for all of the cha

Hi @k0psutin, Yes exactly. We can have padding/margin for everything


2 fixes for Watson Assistant:

use first DNS name as commonName if commonName is empty
parse out IP addresses from dnsNames if they exist and move into
ipAddresses field

sample snippet to quickly test dns sanitization: https://play.golang.org/p/iQtVM8osprZ

/cherry-pick  release-efix


[Feature][Dashboard][OpenAPI] auto-render frontend form for WorkloadDefinition/TraitDefinition

Requirement
In order to making workloads/traits to render their parameters form for application developers without writing specified frontend code for each of them, we expect a mechanism of automatically rendering workload/trait form.
For example, we'd like to automatically render parameters for workload webservice as below.

Possible solutions
https://github.com/oam-dev/kubevela/issues/577
Input
WorkloadDefinition/TraitDefinition, ie, their CUE parameter.
parameter: {
	// +usage=Which image would you like to use for your service
	// +short=i
	image: string

	cmd?: [...string]

	// +usage=Which port do you want customer traffic sent to
	// +short=p
	port: *80 | int

	env?: [...{
		name:   string
		value?: string
		valueFrom?: {
			secretKeyRef: {
				name: string
				key:  string
			}
		}
	}]
	// +usage=Number of CPU units for the service, like `0.5` (0.5 CPU core), `1` (1 CPU core)
	cpu?: string
}

Output
Front-end parameters forms like it shown as below.

@majian159 and @zzxwill will work on it.


console.log console.info show as errors in chromium

For some reason, chromium seems to think console.log() and console.info() are errors and shows them after you reload the extension as errors.
They are currently just there for diagnosis purposes.

hmm, I have a hard time finding out why console.log and console.info will show as errors in chrome://extensions


Support option with dot

Support use a custom message type of one option.
In old version, there are something wrong when parse file like this:
extend google.protobuf.MethodOptions {
  optional MyMessage my_method_option = 50006;
}

message MyMessage {
  optional int32 foo = 1;
  optional string bar = 2;
}

message RequestType {}
message ResponseType {}

service MyService {
  rpc MyMethod(RequestType) returns(ResponseType) {
    option (my_method_option).foo = 567;
    option (my_method_option).bar = "Some string";
  }
}

code from https://developers.google.com/protocol-buffers/docs/proto#options

My thought is that the parser should fail if a non-existent option is referenced. How difficult would this be?

@cpdeethree i thought options were up to the user to define? is there a whitelist somewhere?

@cpdeethree feel free to open another PR for that feature btw if possible. merging this now to fix the bug :)

@wanglam awesome stuff (with tests even!). released in 3.3.1


Add Logging and FunSuite test-utilities

Logging allows logging to be enabled for a test.
FunSuite is a base test class that pulls in all of our test utilties, intended
to replace use of scalatest's FunSuite.  This can be done incrementally/later.

merged https://github.com/BuoyantIO/linkerd/commit/0ff4fc2e5fbeaf1bd27b5aa241ad6c1966a1a0ad


Source problem

Hi!
I've tried to use this ext, but no matter what kind of source i gave to it, it always said, there is nothing to watch.
It seems in Inotify.php construct function it tries to check the inner "source" var. If it's not null ( what will never happen at start ), it will try to add that source:
==================================
public function __construct(string|null $source = null, Closure|null $event = null)
{
  //if (! is_null($this->source)) { //original, this will always be empty at the start
  if (! is_null($source)) {            // it started to work after i changed it to this
    $this->source(new InotifySource($source));
  }
  ...
}

===================================
Am I doing something wrong or is it a bug?
My code:
$inotify = new Inotify( "/tmp/test_dir/", function (Event $event) {
   $event->on(IN_CREATE, function (Watcher $watcher) {
      Log::info("It was creation");
    });
});
$inotify->watch();

Can I get the filename somehow out of the $watcher object?

@Wall0fDeath You're right, I forgot the property. I'll fix it.


Add function for deep merge?

I did this a few days ago in a fork:
https://github.com/crudh/seamless-immutable/commit/8552c98fd177cdf48d560758caa1ab8bee7df92f
mergeDeep is like the normal merge function but it merges all children objects too. Right now there is no code reuse from the normal merge (except for 2 lines it is a copy and paste of merge) and the tests are simple copies from the normal merge.
Before cleaning it up for my fork I just wanted to know if there is any interest to pull it back into master? If so, are there any preferences for naming, code reuse and so on?

I dig it! Test duplication is no problem, but from an API perspective I'd prefer if this were an optional boolean passed to merge to enable deep merging.
Would love to see a Pull Request for this. :+1:

Ok! Instead of a boolean, what to you think about a mergeWith function or similar that takes a config object first (which the normal merge can use under the covers) so more config can be added later if needed?
obj1.mergeWith({deep: true}, obj2);
I have a use case where I would like to supply a custom merger sometimes, that could fit into this strategy.
But if you don't like it I will do a PR with the boolean.

IMO it's usually nicer to have configs or optional flags as the last argument in a function and not the first.

@RangerMauve I agree in general, but if the With in the function name is for merging with the config I think it is natural if the mandatory config is first. Especially if it is a merge with multiple objects like:
obj1.mergeWith({deep: true}, obj2, obj3, obj4, obj5);
But it doesn't matter much either way. Any thoughts @rtfeldman?

@crudh That's a good point, actually. And it'd work well once rest paramters become mainstream.

Good points all around. My preference is to change the existing merge as follows:

It now always takes exactly one or exactly two arguments.
The first argument works the same as always: you can pass it one object to merge, or an array of them.
The second argument is optional, and is a configuration object that currently supports only {deep: true}

Sound good?

Sounds good to me!


VM loses time when host sleeps

Issue Type

Bug Report / Support Request

Your Environment
Vagrant 1.8.7
VirtualBox 5.1.10r112026
ansible <IP_ADDRESS>
  config file = /Users/wt/.ansible.cfg
  configured module search path = Default w/o overrides

Your OS

macOS 10.11.6

Full console output
n/a
Summary
What's your technique for keeping the time of the Vagrant box in sync with the host?
Mine steadily loses time every time the host Mac sleeps.
There is a StackOverflow question about this, but the top-voted answer doesn't work for me with DrupalVM for some reason ("Could not find a registered machine named" either the id or the name).

Does it work if you create a Vagrantfile.local with:
config.vm.provider :virtualbox do |v|
  v.customize ['guestproperty', 'set', :id, '/VirtualBox/GuestAdd/VBoxService/--timesync-threshold', 10000]
end

@oxyc it doesn't seem to make any difference - I've added that file, taken the VM up and down, also run vagrant provision on it, and if I sleep the host Mac for a few minutes then wake it again, the clock is still where it was when it slept, even waiting 10 seconds+ for it to reset.

@wturrell it might.  There's also vagrant global-status --prune that maybe can help.

Worth noting, VBoxManage has a different vm name than vagran's machine name. Run VBoxManage list vms to see the names.
I tested adding a Vagrantfile.local with:
config.vm.provider :virtualbox do |v|
  v.customize ['guestproperty', 'set', :id, '/VirtualBox/GuestAdd/VBoxService/--timesync-threshold', 10000]
end

If I put my computer to sleep the VM time is off until I run vagrant reload, at that point it syncs.

My example had a "typo", it should be --timesync-set-threshold, not --timesync-threshold:
config.vm.provider :virtualbox do |v|
  v.customize ['guestproperty', 'set', :id, '/VirtualBox/GuestAdd/VBoxService/--timesync-set-threshold', 10000]
end

I can confirm this fixed the sync for me. As soon as I wake my host, the VM's clock is correct.

I am trying this. Looks good. I need one week to see how it goes.

@oxyc vagrant global-status --prune has cleaned up the duplicate entry, thanks.  Meanwhile the time started syncing (after a while, for no obvious reason) with your first snippet of code...

@wturrell - Thanks for the update! And @oxyc thanks for the info! I'll close this issue out as it seems there is a resolution.

I believe following https://github.com/geerlingguy/drupal-vm/issues/1097#issuecomment-271213478 has solved the issue for me.