odo push fails for s2i components

/kind bug

What versions of software are you using?

Running odo 2.0.0 from OpenShift Connector 0.2.0
Actual behavior

Create a new NodeJS component - s2i image
Push the component to OpenShift cluster 4.5.9
Cluster is running via CodeReady Containers

Expected behavior
The s2i component should be pushed to the cluster.
Any logs, error output, etc?
It stucks in Waiting for scheduling. As the logs shows it is due to insufficient memory.
bash-3.2$ "/Users/mohitsuman/.vscode/extensions/redhat.vscode-openshift-connector-0.2.0/out/tools/darwin/odo" push -v 4
I1002 15:41:23.802327    7480 util.go:730] HTTPGetRequest: https://raw.githubusercontent.com/openshift/odo/master/build/VERSION
I1002 15:41:23.802842    7480 util.go:425] path devfile.yaml doesn't exist, skipping it
I1002 15:41:23.803858    7480 util.go:751] Response will be cached in /var/folders/tl/c76ptxnn43bcmbjp25w4kkjr0000gn/T/odohttpcache for 1h0m0s
I1002 15:41:23.804252    7480 util.go:764] Cached response used.
I1002 15:41:23.858217    7480 common_push.go:165] SourceLocation: ./
I1002 15:41:23.858282    7480 common_push.go:173] Source Path: /Users/mohitsuman/demo-0.2.0-s2i
I1002 15:41:23.891126    7480 util.go:425] path devfile.yaml doesn't exist, skipping it
Validation
 •  Checking component  ...
I1002 15:41:23.893258    7480 occlient.go:2781] Getting DeploymentConfig: comp-new-demo-app
I1002 15:41:23.967121    7480 component.go:532] Checking source location: ./
I1002 15:41:23.967200    7480 component.go:561] Validating configured memory values
I1002 15:41:23.967211    7480 component.go:561] Validating configured cpu values
 ✓  Checking component [76ms]
I1002 15:41:23.967692    7480 util.go:425] path devfile.yaml doesn't exist, skipping it

Configuration changes
 •  Retrieving component data  ...
I1002 15:41:24.049131    7480 occlient.go:2781] Getting DeploymentConfig: comp-new-demo-app
I1002 15:41:24.066872    7480 pushed_component.go:198] Source for component comp-new is  (local)
I1002 15:41:24.066907    7480 occlient.go:2781] Getting DeploymentConfig: comp-new-demo-app
I1002 15:41:24.086972    7480 component.go:1226] Updating component comp-new, from local to ./ (local).
 ✓  Retrieving component data [119ms]
 •  Applying configuration  ...
I1002 15:41:24.115841    7480 occlient.go:764] Found exact image tag match for nodejs:latest
 ✓  Applying configuration [106ms]

Applying URL changes
I1002 15:41:24.205398    7480 url.go:323] Listing routes with label selector: app.kubernetes.io/part-of=demo-app,app.kubernetes.io/instance=comp-new
I1002 15:41:24.205472    7480 occlient.go:2640] Listing routes with label selector: app.kubernetes.io/part-of=demo-app,app.kubernetes.io/instance=comp-new
 ✓  URLs are synced with the cluster, no changes are required.

Pushing to component comp-new of type local
I1002 15:41:24.226361    7480 file_indexer.go:214] file added: /Users/mohitsuman/demo-0.2.0-s2i/.gitignore
I1002 15:41:24.228477    7480 file_indexer.go:202] .odo or .git directory detected, skipping it
 ✓  Checking file changes for pushing [4ms]
I1002 15:41:24.228695    7480 common_push.go:245] List of files to be deleted: +[]
I1002 15:41:24.229155    7480 common_push.go:268] Copying directory /Users/mohitsuman/demo-0.2.0-s2i to pod
I1002 15:41:24.229717    7480 component.go:650] PushLocal: componentName: comp-new, applicationName: demo-app, path: /Users/mohitsuman/demo-0.2.0-s2i, files: [/Users/mohitsuman/demo-0.2.0-s2i/.gitignore], delFiles: [], isForcePush: false
I1002 15:41:24.249506    7480 preference.go:209] The path for preference file is /Users/mohitsuman/.odo/preference.yaml
I1002 15:41:24.250104    7480 occlient.go:1871] Waiting for deploymentconfig=comp-new-demo-app pod
 •  Waiting for component to start  ...

I1002 15:41:24.303550    7480 occlient.go:1908] Pod comp-new-demo-app-1-8hzct is Running
 ✓  Waiting for component to start [54ms]
I1002 15:41:24.304846    7480 occlient.go:1826] Quitting collect events
 •  Syncing files to the component  ...
I1002 15:41:24.305056    7480 component.go:716] Copying files /Users/mohitsuman/demo-0.2.0-s2i/.gitignore to pod
I1002 15:41:24.306362    7480 sync.go:34] CopyFile arguments: localPath /Users/mohitsuman/demo-0.2.0-s2i, dest /tmp/src/demo-0.2.0-s2i, targetPath /tmp/src, copyFiles [/Users/mohitsuman/demo-0.2.0-s2i/.gitignore], globalExps [/Users/mohitsuman/demo-0.2.0-s2i/.git /Users/mohitsuman/demo-0.2.0-s2i/.odo/odo-file-index.json]
I1002 15:41:24.306699    7480 sync.go:75] makeTar arguments: srcPath: /Users/mohitsuman/demo-0.2.0-s2i, destPath: /tmp/src/demo-0.2.0-s2i, files: [/Users/mohitsuman/demo-0.2.0-s2i/.gitignore]
I1002 15:41:24.306810    7480 sync.go:89] Got abs path: /Users/mohitsuman/demo-0.2.0-s2i/.gitignore
I1002 15:41:24.306826    7480 sync.go:90] Making /Users/mohitsuman/demo-0.2.0-s2i relative to /Users/mohitsuman/demo-0.2.0-s2i/.gitignore
I1002 15:41:24.306845    7480 sync.go:102] makeTar srcFile: demo-0.2.0-s2i/.gitignore
I1002 15:41:24.306857    7480 sync.go:103] makeTar destFile: .gitignore
I1002 15:41:24.306889    7480 occlient.go:3011] Executing command tar xf - -C /tmp/src
I1002 15:41:24.307395    7480 sync.go:121] recursiveTar arguments: srcBase: /Users/mohitsuman, srcFile: demo-0.2.0-s2i/.gitignore, destBase: /tmp/src, destFile: .gitignore
I1002 15:41:24.363292    7480 sync.go:127] Corrected destinations: base: /tmp/src file: .gitignore
 ✓  Syncing files to the component [504ms]
 •  Building component  ...
I1002 15:41:24.811111    7480 exec.go:27] Executing command [/opt/odo/bin/assemble-and-restart] for pod: comp-new-demo-app-1-8hzct in container:
+ set -eo pipefail
+ export ODO_UTILS_DIR=/opt/odo
+ ODO_UTILS_DIR=/opt/odo
+ export ODO_IMAGE_MAPPINGS_FILE=/opt/odo/language-scripts/image-mappings.json
+ ODO_IMAGE_MAPPINGS_FILE=/opt/odo/language-scripts/image-mappings.json
++ /opt/odo/bin/getlanguage
+ IMAGE_LANG=nodejs
+ '[' '!' -z /opt/app-root/src ']'
+ '[' /tmp '!=' /opt/app-root/src ']'
+ '[' -n /opt/app-root/src-backup ']'
+ '[' '!' -d /opt/app-root/src-backup ']'
+ mkdir -p /opt/app-root/src-backup/src
+ rsync -rlO /tmp/src/. /opt/app-root/src-backup/src/
+ b_IFS='
'
+ b_OFS=
+ OIFS='
'
+ IFS='
'
++ ls -A /tmp/src/
+ for file in '`ls -A ${ODO_S2I_SRC_BIN_PATH}/src/`'
+ rm -fr /opt/app-root/src/.dummy
+ for file in '`ls -A ${ODO_S2I_SRC_BIN_PATH}/src/`'
+ rm -fr /opt/app-root/src/.gitignore
+ OIFS=
+ IFS='
'
+ '[' -n rhscl/nodejs-12-rhel7 ']'
+ '[' rhscl/nodejs-12-rhel7 == redhat-openjdk-18/openjdk18-openshift ']'
+ '[' -f /tmp/src/.s2i/bin/assemble ']'
+ '[' -n nodejs ']'
+ /opt/odo/language-scripts/nodejs/dev-assemble
+ DEV_MODE=true
+ REFRESH_WORKING_DIR=/tmp/refresh/
+ PACKAGE_SHA_FILE=/tmp/refresh//package.json.sha1
+ mkdir -p /tmp/refresh/
+ '[' -z '' ']'
+ '[' true == true ']'
+ export NODE_ENV=development
+ NODE_ENV=development
+ '[' development == production ']'
+ export DEV_MODE=true
+ DEV_MODE=true
+ '[' true == true ']'
+ set -x
+ echo '---> Installing application source'
+ cp -Rf /tmp/src/. ./
---> Installing application source
+ '[' true == true ']'
+ '[' -f /tmp/refresh//package.json.sha1 ']'
+ echo '---> package.json modified'
+ sha1sum package.json
---> package.json modified
sha1sum: package.json: No such file or directory
I1002 15:41:25.923727    7480 exec.go:43] ExecuteCommand returned an an err: error while streaming command: command terminated with exit code 1. for command '[/opt/odo/bin/assemble-and-restart]'. output: + set -eo pipefail
+ export ODO_UTILS_DIR=/opt/odo
+ ODO_UTILS_DIR=/opt/odo
+ export ODO_IMAGE_MAPPINGS_FILE=/opt/odo/language-scripts/image-mappings.json
+ ODO_IMAGE_MAPPINGS_FILE=/opt/odo/language-scripts/image-mappings.json
++ /opt/odo/bin/getlanguage
+ IMAGE_LANG=nodejs
+ '[' '!' -z /opt/app-root/src ']'
+ '[' /tmp '!=' /opt/app-root/src ']'
+ '[' -n /opt/app-root/src-backup ']'
+ '[' '!' -d /opt/app-root/src-backup ']'
+ mkdir -p /opt/app-root/src-backup/src
+ rsync -rlO /tmp/src/. /opt/app-root/src-backup/src/
+ b_IFS='
'
+ b_OFS=
+ OIFS='
'
+ IFS='
'
++ ls -A /tmp/src/
+ for file in '`ls -A ${ODO_S2I_SRC_BIN_PATH}/src/`'
+ rm -fr /opt/app-root/src/.dummy
+ for file in '`ls -A ${ODO_S2I_SRC_BIN_PATH}/src/`'
+ rm -fr /opt/app-root/src/.gitignore
+ OIFS=
+ IFS='
'
+ '[' -n rhscl/nodejs-12-rhel7 ']'
+ '[' rhscl/nodejs-12-rhel7 == redhat-openjdk-18/openjdk18-openshift ']'
+ '[' -f /tmp/src/.s2i/bin/assemble ']'
+ '[' -n nodejs ']'
+ /opt/odo/language-scripts/nodejs/dev-assemble
+ DEV_MODE=true
+ REFRESH_WORKING_DIR=/tmp/refresh/
+ PACKAGE_SHA_FILE=/tmp/refresh//package.json.sha1
+ mkdir -p /tmp/refresh/
+ '[' -z '' ']'
+ '[' true == true ']'
+ export NODE_ENV=development
+ NODE_ENV=development
+ '[' development == production ']'
+ export DEV_MODE=true
+ DEV_MODE=true
+ '[' true == true ']'
+ set -x
+ echo '---> Installing application source'
+ cp -Rf /tmp/src/. ./
---> Installing application source
+ '[' true == true ']'
+ '[' -f /tmp/refresh//package.json.sha1 ']'
+ echo '---> package.json modified'
+ sha1sum package.json
---> package.json modified
sha1sum: package.json: No such file or directory
 ✗  Building component [1s]
 ✗  command terminated with exit code 1

@mohitsuman the logs that you provided doesn't mention Waiting for scheduling are you sure that you pasted the correct log output?

I1002 15:41:24.278575    7480 occlient.go:1842] Warning Event: Count: 11, Reason: FailedScheduling, Message: 0/1 nodes are available: 1 Insufficient memory.
This is an issue with clutter. There is not enough memory. Nothing that we can do from odo side. You will have to increases crc memory.
Feel free  to reopen if increasing memory doesn't help
/close


9398.2 Update master_integration_fields.sql to add migration script for adding masterIntegrationFields for joyentTritonKey integration

https://github.com/Shippable/pm/issues/9398

rc verified


Implement GZIP/batches upload of data

This is already done in the Go library, and it's more of an optimization.

Done as part of 0deebffe75ba366b5033e6beeac249b4377428be. Closing.


Name of the Repo

ROS will generate warnings during the compilation if the package name start with a capital letter. Maybe consider change the name of the repo to qualisys instead.
For feature/catkin branch, I have already change the name of the package to qualisys in files like CMakeList.txt and package.xml. Simply change the name of the package once it is cloned, otherwise it won't compile.

I'll rename the repository to qualisys, make sure you change the remote repository url in your git config to point to the new one.


Be more aggressive about cleaning up scroll cursors.

Codecov Report

Merging #19 into master will increase coverage by 0.05%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##           master      #19      +/-   ##
==========================================
+ Coverage   60.70%   60.76%   +0.05%
==========================================
  Files         115      115
  Lines        9824     9821       -3
==========================================
+ Hits         5964     5968       +4
+ Misses       3860     3853       -7

Impacted Files
Coverage Δ

assemblyline/filestore/transport/azure.py
61.76% <75.00%> (-0.24%)
:arrow_down:

assemblyline/datastore/stores/es_store.py
71.00% <100.00%> (+0.08%)
:arrow_up:

assemblyline/filestore/__init__.py
63.00% <100.00%> (ø)

assemblyline/remote/datatypes/__init__.py
69.49% <0.00%> (-0.51%)
:arrow_down:

assemblyline/remote/datatypes/queues/comms.py
77.77% <0.00%> (-0.49%)
:arrow_down:

assemblyline/run/pubsub_reader.py
0.00% <0.00%> (ø)

assemblyline/datastore/helper.py
26.11% <0.00%> (+0.04%)
:arrow_up:

assemblyline/run/cli.py
24.54% <0.00%> (+0.04%)
:arrow_up:

assemblyline/common/backupmanager.py
60.41% <0.00%> (+0.25%)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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


chore: Correct list of provisioned resources formatting

What does this PR do?

Fix list of provisioned resources formatting

More

[ ] Yes, I have tested the PR using my local account setup  (Provide any test evidence report under Additional Notes)
[ ] Yes, I have added a new example under examples to support my PR
[ ] Yes, I have created another PR for add-ons under add-ons repo (if applicable)
[ ] Yes, I have updated the docs for this feature
[ ] Yes, I ran pre-commit run -a with this PR

Note: Not all the PRs required examples and docs except a new pattern or add-on added.
For Moderators

[ ] E2E Test successfully complete before merge?

Additional Notes


Crash when click on any image

Hello,
Every time, I do a research and when I click on any image, the software crash.

You can see the logs here.
And I'm really sorry for my shortly message but I'm very bad in english, I'm french.
Thank for advance

im having the same problem with Grabber version 7.11.0


Jinan/message editing

What

Adjust Css for editing box
Add handlers for edit message
Update message style to flex, make it easier to caculate parent width
Separate code of chatMessage
Reuse logic of input box
Adjust rtl use case

Why

How Tested

Process & policy checklist

[ ] I have updated the project documentation to reflect my changes if necessary.
[ ] I have read the CONTRIBUTING documentation.

Is this a breaking change?

[ ] This change causes current functionality to break.

Tried to run the storybook linked in comments.
Got this error.
FYI @mgamis-msft

Tried to run the storybook linked in comments.
Got this error.
FYI @mgamis-msft

It got fixed by new PR, thanks @mgamis-msft


flash should be disabled by default and the flash button should show the correct state on launch

This issue has been now resolved. (Reference: 282317822de707fa9e258e4b17bbe265727d88f2 and 85299c7612bd4cc5298acc4589e36c5e9577c874)


Person Verifier Showcase

Scope:

Verifier only (no credential issuance)
Accept Person Credential for DEV/TEST/PROD

Deployed at https://bc-wallet-demo-dev.apps.silver.devops.gov.bc.ca/digital-trust/showcase/person
assigning to Alex and Kim for UX work

@wadeking98, while we are waiting for UX work, please remove "picture" and "Expiry Date Dateint" attributes form the proof request

Are we still using this now that we have the lawyer2 demo?

@cvarjao Is this Showcase different from lawyer 2?

@hannahgov, yes. This is a verifier only. The lawyer 2 also issues demo credentials. But, this is till an idea phase, the new lawyer 2 may satisfy


Implement new AppendMissingKey method on error struct

A new append method  added to the ProviderError struct to better handle for adding "missing keys" to provider errors.
Closes #56

Coverage decreased (-1.3%) to 98.738% when pulling 5cdf7161e730f793f82de4e69487381ae7330b6c on #56-new-error-append-keys-method into 7275941f3c39fe1b85399cc8730042b5479654c5 on v2.0.

*please ignore till i fix coverage.


Powertoys Run UX improvements

Along with reducing the height and font size can we have list items to be shown categorically? as of now all result are shown in plain list one after another.
Proposed :

Reduce font and UI size overall (Event the dropdown box!)
Show items in categories.

Eg.
A search "Code" Shows result as below.

It will be more Intuitive if It were shown in categories as :

Apps
Files & Folders
Others

and may be each section can have its own configuration of limit of files being shown?

So basically, just like the standard Windows search. Brings us back to the question: where's the difference between them? One issue has this as a title (don't know the number though). PT Run is turning in to a copy of Search.

So basically, just like the standard Windows search. Brings us back to the question: where's the difference between them? One issue has this as a title (don't know the number though). PT Run is turning in to a copy of Search.

The main reason I went looking for tools like Listary was that Search just doesn't work.
I cannot reliably use it to find what I want to find.
As a user, if Run turns into a copy of Search that works then that's still a win.

PT Run is much more powerful if you take it's extensibility into account. It can resolve math expressions offline, for example, use prefixes for special queries or commands, and more. If you take a look at all the improvements issues on this repo, you should have an overview of how it's shaping up to be much more powerful than Windows Search.

@Jay-o-Way I am not sure how you are comparing the Powertoys Run tool with windows search? I mean just having options to show items categorically it becomes bing search? As said by @wazybr It's more powerful utility to run/launch command locally and   what I recommended is just a UI tweak to what all we currently have and I feel nothing wrong into that.

Also another improvement can be done like dividing result into two panes one text and other possibly a description (Preview/Summary pane) one provided by Cerebro App and below list of quick options like one provided by windows search menu context screen pane.

below list of quick options like one provided by windows search menu context screen pane

This seems like my suggestion in #4657. I think the Listary way of using the context menu entries is better, it will show more actions and would seamlessly include customization to the context menu entries.

To be very frank I Googled to see how does it works and it's nice and quick. 👍 But Here, I though of having a preview pane with possible options listed directly below preview. You can directly navigate to result item with down arrow and then to options with right arrow. Also, we can have them quick links as 'f' for file in explorer which can be again customized 😅

@shirkerohit
we don't track multiple suggestions in one issue.
Closing this in favor of https://github.com/microsoft/PowerToys/issues/4794 for "Reduce font and UI size overall (Event the dropdown box!)" and in favor of https://github.com/microsoft/PowerToys/issues/8913 for "Show items in categories."
Thanks.

@shirkerohit
we don't track multiple suggestions in one issue.
Closing this in favor of https://github.com/microsoft/PowerToys/issues/4794 for "Reduce font and UI size overall (Event the dropdown box!)" and in favor of https://github.com/microsoft/PowerToys/issues/8913 for "Show items in categories."
Thanks.


Make the text visible in the text field on dark mode

I tried to type "sample text" in the text field while on dark mode but the text seems too difficult to see, I think you should change the text color only in the text fields when on dark mode. Besides, it seems there's no issue on light mode. Great work po kuys!. sana gumaling din ako sa ganyang level.

Thanks kuys. Will do. gagaling ka po lods. let curiousity do the work.


Implement BitbucketServerProvider.get_line_link

User description
Implements missing BitbucketServerProvider.get_line_link so links in PR review comments generate proper URLs.
Works with #1059
#1058

PR Type
enhancement

Description

Implemented the get_line_link method in BitbucketServerProvider to generate proper URLs for line links in PR review comments.
The method constructs URLs based on the file and line number, with special handling for cases where the line number is -1.

Changes walkthrough 📝
Relevant filesEnhancement

bitbucket_server_provider.pyImplement `get_line_link` method for BitbucketServerProvider

pr_agent/git_providers/bitbucket_server_provider.py
Added get_line_link method to generate proper URLs for line links.  Constructed URLs based on file and line number.  Included error handling for cases where line number is -1.

+7/-0

💡 PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

cc
@taisyalobanov


Remove vars/main.yml and move vars to defaults/main.yml

This would be the proper way to do it in an Ansible role, otherwise you have to change vars/main.yml to change the version. If you add it to the vars in your playbook it doesn't override it. Without these changes you can't specify the vars in your playbook and have the version change.

@lunarthegrey the changes caused Travis CI to fail. This might be related to the postgres_exporter release bump in 9bd9d34? Could you try either reverting the the version increment or fixing the tests ?

@sbesson, strange as I deployed this in production with the changes on Debian 10. I will make a commit reverting the version change and see what happens. I couldn't get the exporter to work with the previous version at all, had some obscure error that I didn't save.

@lunarthegrey looking at the upstream library, I suspect the test failures were due to https://github.com/wrouesnel/postgres_exporter/pull/215 which breaks the exporter metrics format in order to support multi-server by introducing the server label.
I see reverting to the current version sufficed to fix the tests. Thoughts about getting this in as a patch release allowing consumers like you to override the version of the library via playbook vars  for now? /cc @joshmoore @manics

No objections to getting the move from main to defaults in and tagged as 0.2.2.

@sbesson, nice find. Seems good to me! Then people can override it in their playbooks anyways like I do.

Many thanks for your contribution @lunarthegrey. I pushed two additional commits to document the new variables in the readme and update the top-level license.
The new version of the role has been tagged and is now available from Galaxy - https://galaxy.ansible.com/ome/prometheus_postgres.

Of course @sbesson, thanks for maintaining this role. It seems like the only maintained Ansible role for postgres_exporter currently. Also the best one, the YAML is solid and methods aren't hacky.


Error while running a file: 'runfile' is not defined

From<EMAIL_ADDRESS>on 2013-07-24T08:32:47Z
Spyder Version:  2.2.0
Python Version:  2.7.5
Qt Version:      4.8.4, PyQt4 (API v2) 4.9.6 on Windows
What steps will reproduce the problem?

Running any script when using the setting "Execute in current Python or IPython interpreter" What is the expected output? What do you see instead? Expected output is for the script to run and either perform correctly or return some sort of error.  Instead, I get the following:

Imported NumPy 1.7.1, SciPy 0.12.0, Matplotlib 1.2.1 + guidata 1.6.1, guiqwt 2.3.1
Type "scientific" for more details.
>>> runfile(r'C:\Python27\Lookup.py', wdir=r'C:\Python27')
Traceback (most recent call last):
File "", line 1, in <module>
NameError: name 'runfile' is not defined
>>>
Please provide any additional information below
. Changing the run setting to "Execute in a new dedicated Python interpreter" fixes the error and runs the script but I need to run in the script under the first setting ("Execute in current Python or IPython interpreter")
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1494

From ccordoba12 on 2013-08-09T12:50:11Z
This is very very strange but I don't know how to tackle it (because it's working fine for me).
Are you seeing an error printed in the first line of the console that reads something like:
"import sitecustomize fails"
Summary: Error while running a file: 'runfile' is not defined  <span class="oldvalue"> (was: 'runFile' is not defined) </span>
Labels: Cat-Console MS-v2.3

From<EMAIL_ADDRESS>on 2013-08-09T12:53:42Z
No errors at all. This is exactly what I see when I start up Spyder:
Python 2.7.5 (default, May 15 2013, 22:43:36) [MSC v.1500 32 bit (Intel)] on win32
Type "help", "copyright", "credits" or "license" for more information.
Imported NumPy 1.7.1, SciPy 0.12.0, Matplotlib 1.2.1 + guidata 1.6.1, guiqwt 2.3.1
Type "scientific" for more details.
>>>

From ccordoba12 on 2014-05-04T15:10:08Z
This was an error in WinPython. Here it's explained why https://groups.google.com/d/msg/spyderlib/myEBR_DWnFw/aUde6yOH-ZYJ
Status: WontFix
Labels: -MS-v2.3

From<EMAIL_ADDRESS>on 2014-01-13T04:32:56Z
well.. try re-installing spyder. make sure to delete its temporary data after uninstalling the current version.

From<EMAIL_ADDRESS>on 2014-02-15T01:21:56Z
You can restart the consol of the interpreter. And after it your scripts will be start normaly. In my situation it helps:) However, I don't now main reason.

From ccordoba12 on 2014-05-05T07:29:34Z
issue #1419 has been merged into this issue.


FALCON-790 - Falcon UI to enable entity/process/feed edits and management

@kenneththo  I believe this is already committed, can you please close this pull request?


Webapp revamp - Landing page

Changes

Apply new design to the landing page for mobile and desktop

Context

Relates to #710
Tests

I've checked my work by:

[ ] Adding new tests or adjusting existing tests
[X] Testing manually

@HonkingGoose feel free to approve the snapshots in Happo if you are happy with them!


Ignore non camel case property names

While using camel case is common practice in JavaScript, some libraries or data may not follow this rule. Easing the camelcase ESLint rule removes errors associated with providing non camel case properties. For example, changing the camelcase setting will allow the following argument structure provided to a function:
{
  primary_variant: {...}
}

By changing the rule in .eslintrc we would avoid adding /* eslint-disable camelcase */ in modules.

@tniezg good point, I believe at the same time we should remove places where we // eslint-disable this rule

👍  I'll remove eslint-disable in this PR.

👍 I'll remove eslint-disable in this PR.

Thank you!


az configure: input validation for defaults

Is your feature request related to a problem? Please describe.
Yes, a feature request for adding input validation. As pointed out in https://github.com/Azure/azure-docs-cli-python/issues/1061,  currently the input values are not validated. For example, you can execute az configure --defaults location=blah without any immediate issues, but that doesn't make sense and will potentially cause troubles, down the line.
Describe the solution you'd like
Check input against known, valid values and report invalid values back to user.
Describe alternatives you've considered
The current status is an, albeit non-desirable, alternative. A low-hanging fruit would be to just warn the user that the input values are not validated.

@jiasli  add to S162.

location varies depending on Resource Providers. Some services may not be available in all locations.
Moreover, if CLI limits the location in az configure, once a new data center becomes available or a service becomes available in some data centers, users can't use them immediately and need to wait for CLI to be updated accordingly.
That's why we decided not to implement this check but leave the check to individual commands instead. Thanks for understanding.


Missing sapflow in inventory

C9
~- C11~
~- F13~

Needs to be IDed in the field and updated in inventory.csv

C9 doesn't have a tag and is technically out of plot but i suspect its 1451

@bpbond can you please check for Tag 1451 in the Control plot and see if that is C9? We are missing C9 in the inventory sheet and I suspect it is 1451, but not sure

Also - I found 1829, which had tree tag secured by flagging, and re-mounted the tag. Can you add this in inventory file notes please? Thanks.

Ok C9 now has tag 1456. It is out of plot, ACRU, 24.0 cm dbh.


Power toys

ℹ Computer information

PowerToys version:
PowerToy Utility:
Running PowerToys as Admin:
Windows build number: [run "winver"]

📝 Provide detailed reproduction steps (if any)

…
…
…

✔ Expected result
What is the expected result of the above steps?
❌ Actual result
What is the actual result of the above steps?
📷 Screenshots
Are there any useful screenshots? WinKey+Shift+S and then just paste them directly into the form

Closing since it doesn't contain any information.


Customization of {{noteCreated}}

when I set the Path template to : Resources/YouTube/{{channelName}}/{{noteCreated}} {{title}}.md
the Filename looks like this: "2024-05-18, 1121 Video Title XYZ" is there a way to get rid of the (strangely formatted) time?

changed it as a hotfix for me in the youtube.ts like this: noteCreated: moment().format('YYYY-MM-DD'),

Yes, it's strange that I've added "HH:mm" to format, I'll fix this

Fixed in latest version


Configure docker to use the json log format

Fixes #72
Store docker logs in JSON format, with storage size quota

/lgtm


#2938 Show task details in sequence details

Show sequence tasks filtered by stage and show progress details of each task in tiles.

Codecov Report

Merging #3030 (42be6c1) into master (76645f8) will increase coverage by 5.41%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3030      +/-   ##
==========================================
+ Coverage   36.96%   42.37%   +5.41%
==========================================
  Files         113       10     -103
  Lines        6969     1076    -5893
  Branches      225        0     -225
==========================================
- Hits         2576      456    -2120
+ Misses       4059      564    -3495
+ Partials      334       56     -278

Flag
Coverage Δ

moduleA
?

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

Impacted Files
Coverage Δ

helm-service/controller/action_handler.go

...notification-bar/ktb-notification-bar.component.ts

statistics-service/api/statistics.go

lighthouse-service/event_handler/common.go

statistics-service/api/event.go

shipyard-controller/common/pagination.go

bridge/client/app/_models/notification.ts

helm-service/pkg/mesh/istio.go

helm-service/controller/deployment_handler.go

bridge/client/app/_services/mock-data.service.ts

... and 81 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #3030 (42be6c1) into master (76645f8) will increase coverage by 5.41%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3030      +/-   ##
==========================================
+ Coverage   36.96%   42.37%   +5.41%
==========================================
  Files         113       10     -103
  Lines        6969     1076    -5893
  Branches      225        0     -225
==========================================
- Hits         2576      456    -2120
+ Misses       4059      564    -3495
+ Partials      334       56     -278

Flag
Coverage Δ

moduleA
?

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

Impacted Files
Coverage Δ

helm-service/controller/action_handler.go

...notification-bar/ktb-notification-bar.component.ts

statistics-service/api/statistics.go

lighthouse-service/event_handler/common.go

statistics-service/api/event.go

shipyard-controller/common/pagination.go

bridge/client/app/_models/notification.ts

helm-service/pkg/mesh/istio.go

helm-service/controller/deployment_handler.go

bridge/client/app/_services/mock-data.service.ts

... and 81 more

Continue to review full report at Codecov.

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

Show sequence tasks filtered by stage and show progress details of each task in tiles.

When expanding a task, you see the details (started, finished, message) and the payload of each step.

The result of sequence tasks should be taken from .finished instead of .triggered traces


Add package.json test script

There is currently no test script in the package.json.  The only thing holding back is that this project does not use jasmine so one cannot easily use jasmine test/* as the test script but running node as the command, there is no way to have node run each script of a directory recursively unless one creates a batch script to automate it.
The plan is to create a batch script to run node on test/* and one that is done we can then go further and beyond as to activating it on travis-ci.

TBD


MICNO-274: Changed logging from ERROR to WARNING for 404 not found exceptions

Good point, @DanielElisenberg! 👍🏻
I think I would go for option 1.
As you say, they are all "expected", and is cluttering what is "actual" potential errors with the application, if I am getting this right.


Add Explicit/Offline marking mechanism for Upgrade

resolve #2201
Pending extensive testing with docker-test scripts

In the config, explicit_upgrade is the flag or key and it's value can be critical [meaning error and stop] or warning [log and proceed]. With warning, if there are versions which are not marked as explicit and are in scope for applying the upgrade then upgrade for these versions will proceed [after that it will continue the ACA-Py start up process]. With warning, if there are version which are not marked as explicit and are in scope for applying the upgrade then no upgrade will be performed and version in storage not updated.
"v0.7.2": {
    ...,
    "explicit_upgrade": "warning",
},
"v0.7.3": {
    ...,
    "explicit_upgrade": "critical",
},
"v0.7.1": {
    ...,
},

@shaangill025 — what is left for this one to leave “Draft” state?  Thanks

As I think about it — probably best to leave this to post-0.8.1 as it is not needed.

Tested the two new configuration values and got the expected results (Error and stop when "critical", Logged warning when "warning").
Approved.


6rd support upgrade original patch to freebsd 11

Goodmorning,
I still would like to have proper 6rd support but lack the skills to do this myself.
https://forum.opnsense.org/index.php?topic=4566.msg17514#msg17514
And the original patch:
https://github.com/pfsense/FreeBSD-src/commit/62498dd06a33a82a579d1cc113c2fa04d995ac91
https://redmine.pfsense.org/issues/7272
I can give access to my setup if needed.
My provider is telfort (KPN) in the Netherlands which has support for 6rd.
Another option could be to support it through a gif interface like this post:
https://blog.feld.me/posts/2015/02/ipv6-via-6rd-on-freebsd/
The only problem then would be not to have access to hosts on the same 6rd implementation.
With best regards,
William van de Velde

Hello opnsense team.
It looks like the issue has been resolved within pfsense.
Could there be an effort to get this code back into opnsense?
Best regards,
William

Hi @cobradevil,
pfSense did not release a single patch for their upcoming 2.4 version since July 24 despite their claims that they are open source and actively working on it:
https://github.com/pfsense/FreeBSD-src/tree/RELENG_2_4
Until this changes or hits FreeBSD, there is nothing we will do, because the patch they claim they have is not available publicly.
pfSense did not respond to my inquiries to why they do not open source their changes anymore.
Thus, I am closing this until further notice.
Cheers,
Franco

Hi @fichtner
Ok did not notice their commit change policy .. weird company.
Thanks for the info.
Best regards,
William

If something changes, we are glad to revive this. But I have no willingness to go through the drama that happened in 2014 one more time in a slightly different wrapping. If they want to mess with their community and ultimately the FreeBSD community at large they are certainly free to try. Last time it created OPNsense, something they did not expect. ;)

Any news on this? Has pfSense finally released their 6rd patches?

@paride no, see https://forum.pfsense.org/index.php?topic=138822.0
contributors raising the issue are expelled from the community rather than the simple solution of bringing the source code back or saying that it's not open source anymore. ;)

If you have any news on this like testing the functionality then give me a message.
Best regards,
William

I'm also available to test stuff, just ping me. Given the number of providers using 6rd, I think it's very important for OPNsense to support it.

I see this has been removed from the 18.7 milestone. Is there any news on when this will be implemented? I am looking to migrate from pfsense to opnsense but this feature would be needed to get ipv6 connectivity.

Sorry, GitHub added „projects“ but the global projects are not visible to end users. We initially switched from milestone to project „18.7“, but didn’t know it looked like there was no designation at all now.

Ah, no worries. Glad it didn't fall off the radar for the project as a whole.

If you need any help testing, this is a highly desired feature for me. I'm in the states with CenturyLink Gigabit Fiber service, a dedicated /29 block of IPv4 addresses from them (currently 1 unused that I can assign to testing OPNsense+6RD out for a while), and they use 6RD for their IPv6 implementation. I have dedicated lab hardware I can shove on this connection and open up SSH or other access to any devs that need to poke around (since I know not everyone has access to a live 6RD network)

Alright, this is the patch not yet added to the 18.7 release candidate  https://github.com/opnsense/src/commit/2bab086d4
You can test this update on an all up-to-date 18.1.x like so:
opnsense-update -bkr 18.7-stftest -n "snapshots/dummy"
/usr/local/etc/rc.reboot
I would expect there is more to do in the GUI but if this doesn't crash we have a way forward. Please note that I have no viable test setup whatsoever so we need your help in bringing it back.
Cheers,
Franco

Hello Franco,
I can confirm that the wan_stf interface is back but it does not work yet.
Not sure what is wrong but this is the first attempt so I wanted to let you know it seems to work.
Best regards,
William

Hi William,
Splendid! Can you poke the system log for ifconfig errors?
And maybe an ifconfig output for wan_stf from a working system and our system to compare.
Best case this is only a minor hiccup in the interface management code.
Thanks,
Franco

Ok, after looking at the working setup from 16.7 I saw my error in the 6rd prefix.
I have now a working setup with the patch you gave me without adjustments.
Anyone else too confirm this working without interfering the rest of the system?
Best regards,
William

This sounds shiny. 1-2 more confirms would be awesome indeed. :)

Sounds like I need to migrate my firewall when I get home :)

I can provide test images if you need them.... easy to check with live mode boot without clobbering your install

Not a bad idea, if thats something easy to roll together and I can just boot it off a usb stick I will give it a go.

yeah, of course... amd64 in serial or vga ?

amd64 vga

Thanks @fichtner! If I update to 18.7-stftest, will it seamlessly update to 18.7 once it's out? I'll be away from home for about 10 days, but then I'll be glad to give it a try.

It will, but make sure to lock the kernel and base under system: settings: firmware: packages. It will unlock for the major upgrade and I'll make sure to land this in the final images (maybe not RC so careful).

amd64/vga for 18.1.10 with stf changes: https://pkg.opnsense.org/FreeBSD:11:amd64/snapshots/OPNsense-18.1.10-stftest-OpenSSL-vga-amd64.img.bz2

Thank you, I will give it a try tonight when I get home.

So, I had all intentions to getting this tested tonight, however windows is causing me issues getting the image onto my usb drive. It may not be until this weekend before I can report back.

Not working here, no wan_stf interface showing up. Found this in the log:
opnsense: /interfaces.php: The interface IPv4 '<IP_ADDRESS>' address on interface 'em1_vlan12' is not public, not configuring 6RD tunnel
I think that public IPv4 network addresses should not be a prerequisite, as providers who are using DS Lite are typically using private networks.

@toyowheelin take your time 👍
@Uica I agree. Parts of the code already do this so I've committed c3a4e45a86 which will be in 18.1.11.

That's the first time I have ever seen anybody using that address space. I suspect that opnsense uses a bogon filter on the public interface like pfsense. If that's true I suspect turning off the bogon filter on the wan interface will allow stf to work.

is_private_ip() checks for <IP_ADDRESS>/10 so that happens. But there was one spot in 6RD code that already had it commented out so.... :)

I can't get it to work with swisscom 6rd. I don't know if it is because swisscom changed something since last time I made it work or if it is a problem with opnsense...
I get the wan_stf interface, everything seems configured correctly but I when I run packet capture on internet facing interface no packets goes to the 6rd border gateway and I get "Network is Down" errors. Nothing wrong in dmesg.
# uname -a
FreeBSD XXXXXXXXX 11.1-RELEASE-p11 FreeBSD 11.1-RELEASE-p11  2bab086d417(stf)  amd64

wan_stf: flags=4041<UP,RUNNING,LINK2> metric 0 mtu 1280
	inet6 2a02:120X:XXXX:XXXX:: prefixlen 60
	nd6 options=1<PERFORMNUD>
	v4net <IP_ADDRESS>/32 -> tv4br <IP_ADDRESS>
	groups: stf

## The prefix is 2a02::1200/28 and border gateway is 6rd.swisscom.com the complete ipv4 is embedded into the ipv6 address

PING6(56=40+8+8 bytes) 2a02:120X:XXXX:XXXX:: --> 2a00:1450:400a:801::200e
ping6: sendmsg: Network is down
ping6: wrote google.com 16 chars, ret=-1

I just don't know which diagnostics to run more than that.

Forgot the gateway info:
# netstat -r -n
Routing tables

Internet:
Destination        Gateway            Flags     Netif Expire
default            Y.Y.Y.1         UGS         re0
X.X.X.0/23      link#1             U           re0
X.X.X.X       link#1             UHS         lo0
[...]
<IP_ADDRESS>         Y.Y.Y.1         UGHS        re0

Internet6:
Destination                       Gateway                       Flags     Netif Expire
default                           2a02:120X:XXXX:XXXX::c105:1d01 UGS    wan_stf
::1                               link#5                        UH          lo0
2a02:120X:XXXX:XXXX::             link#11                       UHS         lo0
2a02:120X:XXXX:XXXX::/60          link#11                       U       wan_stf
[...]

stf patch was merged into 18.7-RC1, thanks for the feedback so far!
@RyuunoAelia not sure about swisscom yet, it would be best to have a larger discussion here as soon as RC1 is out in a week hopefully https://forum.opnsense.org/index.php?board=30.0

I tested again with 18.1.11, and can confirm that it works now also with private networks. This is great news - I can finally get rid of pfSense! (Well, I can get rid of that once the Denverton drivers are backported, different topic, but that seems to be under way).
Thanks!

@Uica thanks for confirming! Denverton ? For network? The stf test kernel also has the network backports from 11.2... can you check? If not we still need to do something, I'm not entirely sure.

@fichtner: Yes, for network. Good to know that the backports are supposed to be there! I will test this, but it'll take some time, since I only have that Denverton hardware in my production system, which is currently running pfSense. I'll need to plan the migration carefully, so I can stay on that system if it works and wouldn't have to reinstall / restore the old system.

well we do have a live system usb install... if you need one based on 18.1.11 let me know.

@Uica amd64 ... vga or serial?

amd64 vga

@Uica there you go https://pkg.opnsense.org/FreeBSD:11:amd64/snapshots/OPNsense-18.1.11-stf-OpenSSL-vga-amd64.img.bz2

Thanks! I will test that tonight.

@fichtner: I have tested the live system. The good news is that the backported Denverton network driver worked; 6rd with those interfaces looked good.
The bad news is that the hard disk was not recognized; several error messages about AHCI timeouts. The Denverton AHCI driver should probably be backported, too. Want me to open a new issue about that?

Is pfSense already on 11.2 ? I don't feel so great about backporting AHCI. Is this not solvable with BIOS tweaking the controller ?

pfSense releases are not yet on 11.2, development snapshots for 2.4.4 are.
I’ve tried both, the development snapshot and the latest release with the network kernel module (and only that one) loaded manually. Both worked and recognized the Denverton AHCI driver.
I would prefer not to tweak the BIOS settings, as it works well with pfSense, but if that’s the only way to migrate anytime soon, I’m willing to try.

I'm just curious at this time to hear how much it differs from FreeBSD, because 11.2 is barely out. Let me check the sources again...

okay, so this would be our backport, but not yet tested https://github.com/opnsense/src/commit/4d136a0d91
can we move this discussion to https://github.com/opnsense/core/issues/2473 ?

@fichtner  I quintuple-checked my configuration with as many sources as I could but still didn't find any problem with it. Is it possible to get an stf module with debugging enabled (since this is a compile-time option)? I tested in my installed system by enabling your update source so I can do it again no problem.

btw I made some comments on the patch for things I find weird I'll let you have a look if these are accurate.

@RyuunoAelia thanks for the source code review! I'll comment when I have more time because I would assume that it works in general but we can always go back and improve it on our own. I don't have a setup to test so we have to be careful about this though.
For now, would you mind opening a ticket for your specific case and insert all info about your connection / ISP again? And does this in fact work in pfSense for you (which version) or defunct there as well? I don't want to assume it, but there is the possibility that this code is not the code that is used to build the binary versions giving earlier trouble with the visibility of the code and their owner's attitude towards open source.
I'll look into the debug thing, but it will take till Monday to give you such a kernel.

For the "ticket" could you be more specific where I should open it?
As for switching to pfsense I would rather let that as a last resort since swisscom is really a pain with custom routers since their infrastructure is made to control the routers with TR-069 and gives the router a token that needs to be passed in Option dhcp-class-identifier and pfsense did not support the format of that option correctly (full of commas everywhere). This is why I switched to opnsense in the first place since your dhcp configuration generation code was much smarter than the pfsense one.
If I do not set this DHCP option correctly, I will have a few hours downtime on my internet connection (EACH time I change routers) resolved by a phone call to the technical support of swisscom getting to the support level 2/3 and having them whitelist my custom router...
No problem waiting a bit for the debug. Last time I had 6rd working for swisscom was during their test phase some years ago.

Ticket here in core is fine and I understand your motivation. We can make this work eventually. Worst case there are Swisscom contacts that I can ask if they know something. 😊

For whatever reason, I couldn't get this to work on 18.1.11 with the patch, but upgrading to 18.7-RC1 today, everything is working beautifully.
ISP: CenturyLink Gigabit FttH (United States)
As a note for others trying: make sure your IPv6 prefix is correct. I was trying to configure this thing based on some notes in some online forums, and they had the ISP's prefix wrong.
A note for the UI. In one of my tests, I put in the IPv6 prefix without specifying the bit mask length (because they're the same field, and I forgot about that). Maybe make this into two separate fields with a drop-down for the IPv6 bitmask, just like the IPv4 bitmask?
Also, another interesting issue. Setting up 6RD on the WAN interface creates the 6RD virtual interface. In the UI, you can "assign" and then configure the interface from there. This causes all sorts of issues, such as the IPv6 address being lost. Perhaps it would be best to now allow manual assignment of the virtual interface in the UI?

@darkain yay, thanks for the update! We're collecting all the bits and pieces and hopefully land all of them in said beautifully working 18.7 release. FWIW, everything we discussed up until now will be in 18.7-RC2 for further testing. :)

A note for the UI. In one of my tests, I put in the IPv6 prefix without specifying the bit mask length (because they're the same field, and I forgot about that). Maybe make this into two separate fields with a drop-down for the IPv6 bitmask, just like the IPv4 bitmask?

I'm adding proper validation while not changing the field's input. It's too late to split the prefix from the input, but fixes your issue nevertheless.

Also, another interesting issue. Setting up 6RD on the WAN interface creates the 6RD virtual interface. In the UI, you can "assign" and then configure the interface from there. This causes all sorts of issues, such as the IPv6 address being lost. Perhaps it would be best to now allow manual assignment of the virtual interface in the UI?

Yes, it makes total sense to avoid selecting them as they already belong to an interface.
Both fixed in cc2902e4d -- you can try via:
opnsense-patch cc2902e4d


CosmosClientBuilder with TokenCredential throws ArgumentNullException

System.ArgumentNullException when building with CosmosClientBuilder
After creating a CosmosClientBuilder with TokenCredential a System.ArgumentNullException "Value cannot be null. (Parameter 'uriString')" occurs when Build is called.
var builder = new CosmosClientBuilder(
                accountEndpoint: databaseHost,
                tokenCredential: new DefaultAzureCredential());

builder.Build();

Environment summary
Occurs in version 3.22.1
Remarks
Comparing the master with the source of 3.22.1 revealed that the issue has already been fixed in the master branch.
The ctor does not set the field accountEndpoint in the source of 3.22.1, this has been solved in the master. Releasing a new version of the package will fix this issue.

This was already fixed in https://github.com/Azure/azure-cosmos-dotnet-v3/pull/2844. We are starting to work on the next release so a 3.23.0 will have the fix and should be done in the next week or two.

Fixed in https://github.com/Azure/azure-cosmos-dotnet-v3/releases/tag/3.23.0


SaturnMail Label Editor

From<EMAIL_ADDRESS>on November 06, 2013 10:47:53
Add support edit creating/editing user labels.
Create improved label selection View.
Original issue: http://code.google.com/p/foam-framework/issues/detail?id=55

Obsolete.


Collect code coverage in CI

Closes #292

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@d1a775b). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #294   +/-   ##
=========================================
  Coverage          ?   71.89%
=========================================
  Files             ?       33
  Lines             ?     4658
  Branches          ?        0
=========================================
  Hits              ?     3349
  Misses            ?     1309
  Partials          ?        0

Continue to review full report at Codecov.

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


Add envFrom and remove lookup

Add envFrom to allow providing environment variables from Secrets, and remove usage of the Helm lookup function for merging Secrets into the configuration.
This removes the custom checksum calculation as well. The checksum had included the optional configuration secrets, causing pod restarts whether configuration was updated through the chart or through a secret. Since this required retrieving Secrets from the target cluster through the Helm lookup function, it caused the chart to be incompatible with release pipelines in which a manifest is generated in an environment that does not have access to the target cluster.

@willmostly please rebase

Just a note - this is a breaking change, which reminded me that we should also clarify somewhere in the docs that the chart's version number is not following semantic versioning.

We can just do that in the readme for the chart .. or in the page I will work on for the Trino Gateway docs


X_ZSH_HIGHLIGHT_DIRS_BLACKLIST only works with a single path

If I add multiple directories to the blacklist array, the blacklist stops working. For example:
X_ZSH_HIGHLIGHT_DIRS_BLACKLIST+=("$HOME/mnt")
X_ZSH_HIGHLIGHT_DIRS_BLACKLIST+=("$HOME/Network")

It works fine without the second line there.

Fixed by 1eedb23. Thanks for the report!


change pod event log level to debug

The Pod event log is at Info causing lot of these log lines in medium sized clusters. This PR changes it to debug

@howardjohn does this make sense or you wanted it to be at Info for any specific reason?

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

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
67ab66bee34dcaad4a02e057af0f90c525c72a57
link
/test integ-security-k8s-tests_istio

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.

/test integ-security-k8s-tests_istio

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

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
67ab66bee34dcaad4a02e057af0f90c525c72a57
link
/test integ-security-k8s-tests_istio

integ-local-tests_istio
67ab66bee34dcaad4a02e057af0f90c525c72a57
link
/test integ-local-tests_istio

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.

/test integ-local-tests_istio

/cherry-pick release-1.6

@hzxuzhonghu: once the present PR merges, I will cherry-pick it on top of release-1.6 in a new PR and assign it to you.

In response to this:

/cherry-pick release-1.6

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.

@hzxuzhonghu: new pull request created: #23597

In response to this:

/cherry-pick release-1.6

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.


Clarify interactions between output.libraryTarget and output.library

@TheLarkInn as promised, I expanded on the entire output.libraryTarget section and checked a few things that kind of related to this.  However, I really don't like how when I had sectioned output.libraryTarget behaviors (i.e. the variable, object assignment or the formalized module definition systems) in a sub heading still ended up being on the sidebar which probably is a bad thing in terms of consistency. Though this is the first pass that attempted to correct the various major misgivings I had about the entire thing about that.  Please review carefully because I am by no means experienced at dealing with all the various intricacies of the various JavaScript module systems.

@metatoaster Cool! From a docs side this looks good to me :)

What about the new sections defined under the output.libraryTarget showing up on the same level as the navigation tree when rendered?  I am a bit worried that doesn't look too good, and I am not sure how to go about it while keeping these headings to group them by these categories to help with comprehension.  If this is fine then I supposed no harm done.


[dart] Target of URI doesn't exist: 'package:connectivity/connectivity.dart'. [uri_does_not_exist]

in main.dart
import 'package:connectivity/connectivity.dart';
in pubs.yaml
connectivity: ^0.3.2
editior vs code
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel beta, v0.7.3, on Microsoft Windows [Version 10.0.17134.407], locale en-US)
[√] Android toolchain - develop for Android devices (Android SDK 28.0.3)
[√] Android Studio (version 3.2)
[√] Connected devices (1 available)
• No issues found!

just close and reopen the folder to solve.............................

Thanks. Works, but don't know why or how.


Update

Please update the plugin

Please make a video on how to update your plugin. So that we don't disturb you again

Please update the plugin

I suggest you to implement it yourself - it should be easier now with new version.
https://docs.godotengine.org/en/stable/tutorials/platform/android/android_plugin.html?highlight=plugin
At the moment I don't have time for this.

Are you on discord or Twitter. I want to connect with you

Did you ever make it work?
Is there any update?


User with null email cannot be sorted (happens for anonymous users)

The fix is to instead sort by ID or anonymous ID, whichever is non-null.

Fixed in e4962854ed00a8450c6c01925dec5bf0eedb65e4


Crystal 0.16.0 Could not compile the Library.

Using the latest Crystal 0.16.0 on a test project, I faced this issue:
Error in ./libs/mysql/mysql/types.cr:65: can't extend non-abstract struct MySQL::Types::Value
struct Blob < Value
       ^

Hi @GuneshRaj
Thanks for reporting!
This is fixed in the latest release 0.4.3.


Revert "chore: Remove security services initialization for mongodb"

Reverts edgexfoundry/edgex-go#2567

This was just a test GitHub Revert... Actual work already completed


There is a vulnerability in  Nimbus-JOSE-JWT 5.14,upgrade recommended

https://github.com/google/data-transfer-project/blob/ef53a7e92ef77a3efe0bdc378262215d0fe7498d/extensions/security/portability-security-jwe/build.gradle#L23
CVE-2019-17195 CVE-2022-21449 CVE-2022-21434
Recommended upgrade version：9.21.1

Resolved by #1237


SSL error after AWS Aurora Update

After upgrading our AWS Aurora servers today (us-east-1 region), we cannot connect anymore using SSL: "Amazon RDS".
In my opinion the RDS SSL certificate in https://github.com/mysqljs/mysql/blob/96fdd0566b654436624e2375c7b6604b1f50f825/lib/protocol/constants/ssl_profiles.js has to be updated.

That's interesting. I can certainly update them (or you can even make a pull request to update). Can you link me to the public certificate that I need to add/update?

We will try to download the new RDS certificates and patch our local mySQL NPM to see if it works.
I will comment the outcome here and we'll either prepare a PR or supply the certificates.

Ok. I got answer from AWS. It is not a problem with the certificates - so node mysql is ok!
In the case that anyone else runs into the problem:
The AWS Aurora patch is broken, but there is a temporary workaround
"The temporary workaround is to disable Diffie-Hellman ciphers suites in the impacted MySQL client.
mysql -hyourhost -uyouruser -p{} --ssl-ca=rds-combined-ca-bundle.pem --ssl-verify-server-cert --ssl-cipher='DEFAULT:!EDH:!DHE'"

We were also having ssl handshake errors when connecting to an Amazon RDS Aurora backend using the "Amazon RDS" ssl config in this module. The errors we were getting were:
{
      "code": "HANDSHAKE_SSL_ERROR",
      "message": "140283231541056:error:14082174:SSL routines:ssl3_check_cert_and_algorithm:dh key too small:../deps/openssl/openssl/ssl/s3_clnt.c:3617:"
}

This seems to coincide with an update to the Aurora engine on 10-26:
The fix was relatively simple (using @HyperBrain's response from AWS), but less than obvious, especially if you're less adept at SSL in general. We downloaded the AWS combined certificate into our repository and update our ssl config as:
const fs = require('fs');
const amazonCA = fs.readFileSync('./rds-combined-ca-bundle.pem');
const sslConfig = {
    ca: amazonCA,
    ciphers: 'DEFAULT:!EDH:!DHE', // from @HyperBrain's AWS response
};

var connection = mysql.createConnection({
    host: '...',
    ssl : sslConfig,
});

This workaround is fine for now and we'll work with AWS to come up with a more permanent solution and update the issue / file a PR if it's something that can be addressed directly in this module.

and update the issue / file a PR if it's something that can be addressed directly in this module

Please do!

Since the workaround was working, we didn't focus too much on the issue until today and I had it slightly wrong. The 10-26 update actually fixed the issue. It's only a problem on the 1.8 (fixed in 1.8.1, broken in 1.8) version of the aurora engine, so the easy fix is to just update your aurora instances. If that's easier said than done, the workaround above is sufficient, but I don't want to change the module and add complexity that's only needed for one patch revision of the aurora engine for RDS.


Add runtime flag for specifying trust stores to install to

I wish there was a flag to specify which trust stores to install to, for example only to Firefox's. The current default is to install in system (requires sudo), Chrome (which I don't use), and Firefox trust stores.

Works great, thanks!
$ TRUST_STORES=nss $(go env GOPATH)/bin/mkcert -install
Using the local CA at "/home/aorth/.local/share/mkcert" ✨
The local CA is now installed in the Firefox and/or Chrome/Chromium trust store (requires browser restart)! 🦊

$ TRUST_STORES=nss $(go env GOPATH)/bin/mkcert -uninstall
Using the local CA at "/home/aorth/.local/share/mkcert" ✨
The local CA is now uninstalled from the Firefox and/or Chrome/Chromium trust store(s)! 👋


Consider replacing flatlist with Shopify/flash-list ?

I use production app. When I load other works (about 4 pages), it will be very lag, and the app will be crashed after that.

hey @ducdh-dev thanks for calling out, we are aware and working on this issue, on Android it is especially bad.

Linking here the first part of a series of PRs that will swap out the old implementation with the MasonryFlashList


Revert "I9 new radiation engine daysim by paul"

Reverts architecture-building-systems/CEAforArcGIS#500

to add #500 back, just create a new PR (but first update from master and fix all the errors...)


docs(@angular/cli): fix universal example webpack config

If you run that webpack config currently it'll throw an exception

@CaerusKaru fixed :D

LGTM 👍
cc @hansl @MarkPieszak
(PS, you can add Fixes #8183 to your PR description to close that issue when this gets merged)

Nice catch wow ! 👨💻👍


Resolved invalid syntax in resizeimage.py

There were unnecessary brackets in some function that were throwing invalid syntax error.
Line numbers: 118, 142, 165

thanks @amitrao11 for this path, and very sorry for this silly error.


Windows 32/64 Github Action Builds

Fixes #142
This PR includes minor changes to the Bazel build process and new GitHub Actions/Workflows to facilitate building the Windows 32/64 release zip files using the Visual Studio toolset ensuring it's compatibility on all Windows systems (not just in minGW).  The Bazel changes should be compatible with existing internal build processes and should not affect the non-Windows releases controlled by Google's internal build system. They are only so we can pass the "--cpu" argument in Github Actions and properly name the output release file.
See the release created by the tag submitted with this PR at:
https://github.com/yinzara/protobuf-javascript/releases/v3.21.2
I was actually successful in building the Mac OS x86/ARM and Linux x86_64 releases as well.
https://github.com/yinzara/protobuf-javascript/releases/tag/v3.21.2-yinzara-2
Unfortunately building the other architectures would either require a new toolchain or potentially using https://github.com/uraimo/run-on-arch-action. However, either was beyond the scope I was tackling and both wouldn't be able to produce the x86_32 architecture.
I've disabled the Mac OS and Linux builds in the github action workflow file for now (with comment explaining why).
If a release was already created via an internal tool before the tag is pushed to Github, this should properly add to the existing release (instead of creating a new one) because of the svenstaro/upload-release-action@v2 behavior.
I tested this by rerunning the release step of the build and it only updated the executables without creating a new release.
Initial release:
https://github.com/yinzara/protobuf-javascript/actions/runs/4379577450/attempts/1
Rebuild:
https://github.com/yinzara/protobuf-javascript/actions/runs/4379577450
I suggest after this PR is merged that someone take the auto build win32/win64.zip files and update the existing 3.21.2 release so that people downloading the new files will have the fixed version that works in Powershell and Windows apps.

Overall this looks fantastic. Thank you for putting it together!

Any chance we could get the existing win32 and win64 zip files in the 3.21.2 release updated with the versions built by the main action build?

Yes, planning to, just ran out of time the other day.  I'll ping the bug thread when it's done.


runner: Implement validator support

This PR is a PoC for #13 to get a quick preview of my idea.
Welcome any comments.

ping @skyzh to take a look.

Please let me know if we need an RFC for this feature.

By the way, can we go a step further? e.g., add a Result associated type for DB, so that we can pass any type apart from String as query result. (Not needed in this PR)

Oh, so nice. Let's do it.

By the way, can we go a step further? e.g., add a Result associated type for DB, so that we can pass any type apart from String as query result. (Not needed in this PR)

Created a new issue #16 to track it.

Also, I would recommend adding an example for validator in examples folder.

Added. Maybe we should move some of them into tests instead.


Staff email notifications on intervention added

Not part of current project plan but something to look at in the future.

Moved to https://github.com/studentinsights/studentinsights/issues/15.


Make Stanford tagger heed class path

NLTK was issuing a command looking a lot like this:
/usr/bin/java -mx1000m -cp "stanford-ner-2015-12-09/stanford-ner.jar:stanford-ner-2015-12-09/bin/*" edu.stanford.nlp.ie.crf.CRFClassifier  -loadClassifier stanford-ner-2015-12-09/classifiers/english.muc.7class.distsim.crf.ser.gz -textFile sample.txt -outputFormat slashTags -tokenizerFactory edu.stanford.nlp.process.WhitespaceTokenizer -tokenizerOptions "tokenizeNLs=false"
CRFClassifier invoked on Wed Jun 08 17:03:10 EDT 2016 with arguments:
   -loadClassifier stanford-ner-2015-12-09/classifiers/english.muc.7class.distsim.crf.ser.gz -textFile sample.txt -outputFormat slashTags -tokenizerFactory edu.stanford.nlp.process.WhitespaceTokenizer -tokenizerOptions tokenizeNLs=false
tokenizerFactory=edu.stanford.nlp.process.WhitespaceTokenizer
tokenizerOptions=tokenizeNLs=false
loadClassifier=stanford-ner-2015-12-09/classifiers/english.muc.7class.distsim.crf.ser.gz
textFile=sample.txt
outputFormat=slashTags
Exception in thread "main" java.lang.NoClassDefFoundError: org/slf4j/LoggerFactory
	at edu.stanford.nlp.io.IOUtils.<clinit>(IOUtils.java:42)
	at edu.stanford.nlp.ie.AbstractSequenceClassifier.loadClassifier(AbstractSequenceClassifier.java:1484)
	at edu.stanford.nlp.ie.AbstractSequenceClassifier.loadClassifierNoExceptions(AbstractSequenceClassifier.java:1497)
	at edu.stanford.nlp.ie.crf.CRFClassifier.main(CRFClassifier.java:3015)
Caused by: java.lang.ClassNotFoundException: org.slf4j.LoggerFactory
	at java.net.URLClassLoader$1.run(URLClassLoader.java:372)
	at java.net.URLClassLoader$1.run(URLClassLoader.java:361)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.net.URLClassLoader.findClass(URLClassLoader.java:360)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
	at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:308)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
	... 4 more

However, if I correct what is passed to -cp to the following -cp "stanford-ner-2015-12-09/stanford-ner.jar:stanford-ner-2015-12-09/bin/*", it works:
/usr/bin/java -mx1000m -cp "stanford-ner-2015-12-09/stanford-ner.jar:stanford-ner-2015-12-09/lib/*" edu.stanford.nlp.ie.crf.CRFClassifier  -loadClassifier stanford-ner-2015-12-09/classifiers/english.muc.7class.distsim.crf.ser.gz -textFile sample.txt -outputFormat slashTags -tokenizerFactory edu.stanford.nlp.process.WhitespaceTokenizer -tokenizerOptions "tokenizeNLs=false"
CRFClassifier invoked on Wed Jun 08 17:12:34 EDT 2016 with arguments:
   -loadClassifier stanford-ner-2015-12-09/classifiers/english.muc.7class.distsim.crf.ser.gz -textFile sample.txt -outputFormat slashTags -tokenizerFactory edu.stanford.nlp.process.WhitespaceTokenizer -tokenizerOptions tokenizeNLs=false
tokenizerFactory=edu.stanford.nlp.process.WhitespaceTokenizer
tokenizerOptions=tokenizeNLs=false
loadClassifier=stanford-ner-2015-12-09/classifiers/english.muc.7class.distsim.crf.ser.gz
textFile=sample.txt
outputFormat=slashTags
Loading classifier from stanford-ner-2015-12-09/classifiers/english.muc.7class.distsim.crf.ser.gz ... done [0.8 sec].
Hello./O Bugs/O Bunny/O is/O a/O wascally/O wabbit./O Elmer/PERSON Fudd/PERSON works/O for/O the/O FBI./O
CRFClassifier tagged 13 words in 1 documents at 481.48 words per second.

where sample.txt is created thus echo "Hello. Bugs Bunny is a wascally wabbit. Elmer Fudd works for the FBI." >> sample.txt.
But I could not work out how to pass information to the Java command, as it seems that no environment variables are referenced in the appropriate place. I made an edit in one place and all works with appropriate setting of CLASSPATH.

This seems related to this.

I added a third commit to make code robust to unspecified CLASSPATH and to use information on Stanford JAR even if not specified on CLASSPATH.

Note that the develop branch does not work with my test code. But if I apply my commits to 3.2.1, it works. I have a branch (https://github.com/iangow/nltk/tree/iangow) that is 3.2.1 plus the commits on heed-classpath-stanford branch.

@iangow Thank you for looking into the Stanford NER Tagger. The Stanford wrapper should move towards using the Stanford CoreNLP API, please take a look at #1249
Also, please note that the if-else condition that you've added would be appending/changing how self._stanford_jar is instantiated. The better solution would be to improve the  NLTK internals code works for nltk.internals.find_jar.

Thanks. All works now. Honestly, I thought I was having the problem with 3.2.1 (git checkout 3.2.1 followed by sudo python setup.py install), but now I cannot replicate it. So problem solved.


Fix a subtle bug in DBSCAN noise marking

Situation: a core point is connected to only boundary points. However,
those boundary are connected to other core points. So it may happen that
the boundary points are assigned to other clusters, resulting in a
cluster with a single point (core).
The bug was that those single core clusters were marked as noise.
The fix is ugly as heck, but I don't know how to do better.

CI failure is not a failure, but a warning from a container build.


Make Error "undeclared identifier 'MAP_POPULATE'"

Can you help me how to fix this erro

Are you running this on Linux? I have not tested this on any other platform. It appears you don't have some of the required headers that define MAP_POPULATE.

Thank you for the prompt reply.
I had a compilation error because I was running in MacOS.
I tried in Linux and it runs perfectly. Thank you.


added quick-start.sh & extra.sh, fixed issue with aarch64_CM-agent.sh…

… & removed image:mongo-infra:0.1 from docker-compose.yml

fixes #42


Perform and check methods should be better suggested.

perform and check methods are not in bold in suggestions. Fix that.

not applicable anymore


Fixes #505 : fix the scrollbar in library

Related: https://github.com/readium/readium-desktop/pull/498

Initial problem: flickering!
https://github.com/readium/readium-desktop/issues/294

Original attempt to fix the scrollbar: https://github.com/readium/readium-desktop/pull/362

This seems to work fine, let's merge. If the flickering problem reappears, we will try another CSS technique.


client: render posts server side

Fixes #22
There was some code that I removed that checked whether the post was ok or not but didn't actually align with the structure of the post object itself.
I've replaced that logic with some simpler logic to check if the post is good or not.
While testing, I found a bug that allows you to view private posts as if they are public/unlisted. I'll make an issue about that shortly.

LGTM 🎉


Drop vector column from ResultSet when not selected

See for some context https://github.com/stargate/stargate/pull/2760. Essentially, we want to filter out vector columns when they are not selected. This logic will be reverted once we get a fix in stargate.
Note: it was not easy to write a test. I verified the vector tests pass as expected.

I essentially copied the logic from https://github.com/datastax/cassandra/pull/728, and made it vector specific to limit the blast radius.

This did not work for prepared statements. https://github.com/stargate/stargate/pull/2760 did work for them, so we are just pushing through the correct solution without merging this.


Logout route added, Env added to Secret string

This is a new pull after cloning develop branch, removing conflicts, adding log out route and creating env for super secret string.

Is this root necessary? The token can be simply removed o front-end IMO. Is this PR is a result of some issue? Please enlighten me @sounak07 @skyshader


Refunds on Adyen_Payment 2.3.0 are factor of 10 too high

And thus results in an exception. Creditmemo is created though, but with value of 10 times too high.
Notification from order with total amount of € 1, it creates a creditmemo with a value of € 100:
2015-07-03T10:50:54+00:00 DEBUG (7): Array
(
    [processResponse begin] => Begin to process Notification
    [response] => Array
        (
            [amount] => Array
                (
                    [currency] => EUR
                    [value] => 100
                )

            [eventCode] => REFUND
            [eventDate] => 2015-07-03T12:49:54+02:00
            [merchantAccountCode] => AceTateNL
            [merchantReference] => 100011429
            [originalReference] =><PHONE_NUMBER>654745
            [paymentMethod] => mc
            [pspReference] =><PHONE_NUMBER>948389
            [success] => true
        )

    [_updateOrder] => Updating the order
    [_addStatusHistoryComment amount] => amount notification:100 amount order:100
    [_addStatusHistoryComment] => Created comment history for this notification
    [_updateAdyenAttributes] => Updating the Adyen attributes of the order
    [_processNotification] => Processing the notification
    [_refundOrder] => Refunding the order
    [_refundOrder error] => Error creating credit memo error message is: Maximale bedrag voor terugbetaling is € 1,00
    [_setRefundAuthorized] => Status update to default status or refund_authorized status if this is set
    [_storeNotification] => Notification is saved in adyen_event_data table
    [__pid] => 24121
)

We're on Magento <IP_ADDRESS>.

Hi,
The value send to adyen is the amount of the credit memo. Is the credit memo amount as well 10 times higher? If this is the case something else in your magento is going wrong.
Regards,
Rik
Adyen

This refund was sent from Adyen control panel if I'm not mistaken.

Hi Leeuwd,
Oke thanks for your respond. It is always better to do the refund from inside Magento. I will look into this.
Regards,
Rik
Adyen

Hi Leeuwd,
This is fixed. See change below.
Regards,
Rik
Adyen

Great


Character counter font doesn't reset

What is the issue

Exceeding 500 characters changes the character counter's font. Going back under 500 doesn't reset it. Changing zoom levels does.
How to reproduce

Set your zoom to 0.9x
Open two splits (for comparison)
Type 501 characters in one
And one in the other one
Replace the 501 characters with one
Observe that the font looks slightly different

Screenshots

It's easier to see with more characters.

OS and Chatterino Version

Chatterino + Dankerino patches 826c56fe2cec1b1b2a6647ad6a19d3d3b76641c9 (based on upstream 74960bf4)

I've seen this issue couple times myself, but couldn't reproduce it properly. Thanks for providing good repro steps :+1:

Actually, unsetting the custom font size itself is the cause of this behavior.


Request: Potions as Bonus Action

Is it possible to get a Rule Change Option to make Potion a Bonus Action? Like Critical Role does it in their campaigns?

Added a new Cunning Feat that might cover this scenario.


Static analysis?

As far as I know, there are no hardcore static analyzers for Rust yet (like Coverity or PRQA for C++), due to the language and its ecosystem being so brand-new and young.
However, there are some decent-looking linters out there, like https://github.com/rust-lang-nursery/rust-clippy.
@kjetilkjeka do you think it would make sense to run Clippy against the library sources in the CI environment? Do you know other (better?) static analysis solutions for Rust?
Since we're aiming to make a highly robust implementation, having static analysis set up early is very important for the project.

Clippy seem great, I think we definitely should run it in CI. I will configure travis one of the next days. I'm not well versed in the static analysis options in Rust, but it seems like it's more common to use clippy than not.
As I'm sure you know, the Rust compiler also gives statically proven memory guarantees (which is amazing).
We should probably ask ourselves. With the lints Clippy provide and the guarantees (+lints) rustc provides, what more do we want from static analysis? After answering that we're probably in a good shape for looking into what kind of tools that currently exists and will exist in close future.

Well, what we need from the static analysis is quite simple: we want it to point out possible mistakes and well-known issues in the code. People like Coverity, MISRA, or PRQA gather their knowledge of that from the empirical data accumulated over the years of application of C++ in various industries. That data is probably not going to be available for Rust in the years to come, so I would expect less from its static analysis tools.

Travis is now running a build with clippy, the results looks like this: https://travis-ci.org/UAVCAN/uavcan.rs/jobs/265996551
It won't fail the build but this is possible to achieve by for instance failing on warnings when running clippy. For this to be practical, I will have to go through the current warnings and fix them or disable lints where. The practical approach is probably to manually disallow some specified lints instead. I guess the approach would then be to first remove warnings and then disallow failures for lints seperately (one at a time). What do you think about this @pavel-kirienko?
It also looks like it's possible to get a code coverage report for rust projects with codecov. I will look into this. But if you (or anyone else?) have experiences with this tool I would love to hear them.

The proposed approach with Clippy looks good to me. Eventually, the failures do need to be disallowed (as in they need to break the build), otherwise people won't be paying attention to the output of the static analysis, I think.

It's also possible to run rustfmt in CI to fail builds if the code is not formatted correctly. I'm going to test/look into this to find out if it's useful.

#27 makes CI break for even more lints. There is still some more I wish to add, but I'm currently doing it incrementally to make sure it doesn't interfere with productivity.

here is the threaded SimpleNodeexample. The interface name of vcan0 is currently hardcoded. This, amongst a lot of things, will be improved before publishing.


Indeterminate progress indicators should not be active when collapsed

Template10 / Templates (Project) / Minimal / Views / Shell.xaml
Things like ProgressRing and indeterminate progress bars should not be IsActive="True" unless they are visible.  Unless they are specifically collapsed, then the animations actually are still running, causing potential perf issues.  Collapsing a parent container (e.g., grid, panel, etc.) isn't enough as that doesn't trigger turning activity off for them.

Daniel warned me of this, too. We'll fix it asap.

The code-behind implemented this - alas we forgot to update the XAML. Fixed.

Now that's asap!


Dialog buttons hidden behind keyboard

On some phones, the EditText dialog has buttons hidden behind keyboard.
But normal dialogs in other apps moves away whe keyboard appears. Why not here?

Could you show a screenshot? Can't reproduce it on my devices. However, I don't do anything with the dialog handling, so it's up to the base library, whatever happens with the displayed dialog.

On Nexus 5X (Android 7.1), it actualy shrinks the part of dialog with EditText:

But on one fo my customer's phone with Android 4 or 5, the keyboard is above the dialog and the suggestions bar hides the dialog's buttons. Sorry, I can't get the screenshot at the moment.
Maybe this should be another issue your library fixes.
I found this on Stack Overflow.
Maybe just add this to onCreate() of your dialog implementation:
getDialog().getWindow().setSoftInputMode(WindowManager.LayoutParams.SOFT_INPUT_ADJUST_RESIZE);

Still cannot reproduce. I tried it on 7.0 and 5.0.1 but the dialog works good: if the content is too high, it moves up til possible, the bottom part is not touching the keyboard area, and the content is scrolling. Otherwise the dialog displays normally with all its content and buttons visible. Do you have this problem with the sample app too? I think you set the windowSoftInputMode attribute (or something similar) on your Activity and it changes the behavior of the resizing methods.

@Pitel Do you have this problem with the sample app too?

I have the very same issue.

Same issue, the problem is the same that report @little-fish

@ornolisvt Is this present in v1.1.0?

@ornolisvt Is this present in v1.1.0?

I´m using Xamarin Android with Support libraries, not AndroidX libraries


Document update needed:-   '--sdk-auth' has been deprecated and will be removed in a future release.

for the given article

While attempting to create a service principal using the az ad sp create-for-rbac command in the Azure CLI, the provided command, along with correct credentials, executes with a warning and an error:

'--sdk-auth' has been deprecated and will be removed in a future release.
aslo an  Error occurred in request., InvalidSchema:

The document needs an upgrade with valid format if there is any replacement for --sdk-auth has been made.
[Enter feedback here]

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

ID: 7aef74eb-a8c5-0251-82f6-0cba6f58d712
Version Independent ID: fa7b29e1-f341-e7ac-f4a8-d04602e2cd0f
Content: Use GitHub Actions to deploy a static site to Azure Storage
Content Source: articles/storage/blobs/storage-blobs-static-site-github-actions.md
Service: azure-storage
GitHub Login: @juliakm
Microsoft Alias: jukullam

@surajyadav1108
As discussion with author and product team advised us to keep this command in. If any changes required, it will be addressed in future.
if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

Okay, got it. Everything else worked fine for me based on the documentation. Thanks for letting me know.


Update sbt-scalajs, scalajs-compiler to 1.0.0

Updates

org.scala-js:sbt-scalajs
org.scala-js:scalajs-compiler

from 0.6.32 to 1.0.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-js" } ]

labels: library-update, semver-major

Codecov Report

Merging #368 into master will decrease coverage by 0.37%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #368      +/-   ##
==========================================
- Coverage   95.92%   95.54%   -0.38%
==========================================
  Files          15       15
  Lines         761      764       +3
  Branches       45       45
==========================================
  Hits          730      730
- Misses         31       34       +3

Impacted Files
Coverage Δ

core/src/main/scala/io/iteratee/Enumerator.scala
94.96% <0%> (-2.1%)
:arrow_down:

Continue to review full report at Codecov.

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


Fix tests and scripts

Fix tests & scripts parsing
Add appveyor script
Add missing pypi mocks
Update Lockfile
Switch to pathlib2 as fallback

This contains #1909 right? I’ll rework the script-parsing part mentioned about a little.

@uranusjr I'm not sure it contained all of #1909 so you may want to verify that
I just reorganized and squashed everything

FYI I will be merging this when it finishes unless there is an objection. I am open to suggestions about dealing with the extremely large artifacts we are now mocking.

I cherry-picked changes not included previous. Fingers crossed. (If this does not work I’ll revert and work on them later.)

I'd be worried that keeping the matrix in appveyor.yml up to date (and in sync what's specified in .travis.yml) is going to prove tricky, leading to inevitable gaps in CI coverage.
Is there any way to reduce the copy-pasta in the config?  (I'm unfamiliar with Appveyor, but will take a look at their docs.)

The situation is indeed bad, but unfortunately I don’t know of any bullet-proof solutions :(

I had a quick look at the docs, and it seems that one may be able to leverage multiple matrix dimensions with environment variables.  That approach would at least reduce a big chunk of copy-paste.
I'm not at a computer this afternoon, but I may try experimenting with this later.

Please don't change the appveyor config, it was very difficult to get it working.

Sure.  But you understand the concern, I guess?  There's a ton of copy-paste in there, which is definitely going to break silently at some point in future.  IIRC there's already been a recent incident of "tests are silently failing on CI", so the paranoia (as someone who would like to be able to rely on Pipenv in a user context) isn't completely unwarranted.
I realise I'm moaning a lot, so as a minor constructive suggestion - could the TEST_SUITEs be simplified?  For example, could they be named e.g. a, b, c, d in the CI configs, and the mappings specified elsewhere (e.g. in run_tests.sh)?  This would also allow for a programmatic / more robust way to deal with the not case.

@tsiq-oliverc I’m all for it if there is a better solution. I couldn’t find one.

In this PR I want tests passing consistently so that we can merge other PRs
Making the appveyor config more agreeable to you personally is not high on my priority list, that is not a personal qualm with you but a literal allocation of resources issue. Could it be configured better? Maybe. I tried many different arrangements and talked to several people about it, and I also consulted the appveyor configs of about 50 large projects and I couldn't find any reference materials that had a better organization system
My ideal case is kind of fine the way things are, which is that we catch everything in the last integration test anyway, so I'm just struggling to see the problem i guess.

@techalchemy - For sure, I concur that getting the tests back to green is a priority.  I'd be proposing to submit a PR once this has merged (in any case, a PR against a PR from a GitHub fork is too complex for me to handle!)
I suppose what I'm saying is that as complexity / entropy / duplication grows, the probability of divergence / bugs approaches 1.  That's just the reality I'm familiar with.  And CI is the last line of defence, so there's nothing further that could catch it in this case.
But given there's no direct way in Appveyor to clean this up, my proposed PR would be something like factoring out the duplicated lists of test markers (in both Travis and Appveyor config) to a single source of truth, and then also eliminating the duplication due to the not list.

 But yeah I was thinking of making a single list for each of the 4 other runs and then combining them in a separate place for the "not" use case
I'm gonna just merge this one for now sowe can start merging other things.  There's still a test in here that we have skipped, but it needs to be fixed.


No runtime available error

I'm constructing tokio runtime myself to make certain async methods synchronous in my code. I'm hitting this error when I switch from local to remote.
https://github.com/tursodatabase/libsql/blob/508274d9a68851fa468dfb5f555f7aecb815563a/libsql/src/hrana/hyper.rs#L91
Any option to fix this without using a global runtime?

Closing in favour of #806


timekeep: fix header include

it is required for memset during karin build process
Signed-off-by: Humberto Borba<EMAIL_ADDRESS>

Mandatory for Marshmallow build


HSEARCH-4300 Fix build timeout after 1 hour when testing against DB2

https://hibernate.atlassian.net/browse/HSEARCH-4300
Also includes a few follow-up fixes for HSEARCH-4303, since we can now actually test on Oracle.
Some tests are down from 4+ minutes to ~10 seconds on DB2, because we do much fewer DDL operations. Hopefully that will be enough to bring test execution on DB2/Oracle back under 1 hour.

Test execution on all databases: https://ci.hibernate.org/blue/organizations/jenkins/hibernate-search/detail/PR-2649/2/

Looks like it was a success; when compared to the last build without timeout (which included fewer tests, that's why db2 still ran in under 1 hour):

postgresql down from 8min30 to 5min
mariadb down from 9min to 6min
mysql down from 13min to 6min30
mssql down from 10min to 6min
oracle down from 16min to 8min
db2 down from 36min to 13min

Merged!


Enable configuring java GC and heap dump

For debugging crashes when ODL runs out of ram or gets a sigkill we need should generate heap dumps.  Config needed to startup file:
-XX:+HeapDumpOnOutOfMemoryError  -XX:HeapDumpPath=

@trozet @dfarrell07 we already have -XX:+HeapDumpOnOutOfMemoryError on DEFAULT_JAVA_OPTS in bin/karaf (at least on an ODL dist built locally today from an upstream current master = future Nitrogen release, based on Karaf 4 - but AFAIK I've added this quite a while ago already).
Dunno what HeapDumpPath is for - are you sure we need that? Probably to set the dir for the *.hprof ... without that it probably goes into the "working directory" of the Karaf JVM proc - not good enough? Not writeable, not working? (see http://docs.oracle.com/javase/8/docs/technotes/tools/unix/java.html and https://docs.oracle.com/javase/8/docs/technotes/guides/troubleshoot/clopts001.html)
Unless proven otherwise, I'm not sure this is really required.


Query failed on loaded collection

Milvus version: dev version.
Cli: v0.2.0

connect milvus
query on already loaded collection
throw Error: <BaseException: (code=5, message=collection has already been loaded, and the number of replicas 1 is not same as the request's 0. Should release first then reload with the new number of replicas)>

query on unloaded collection, every working fine.

fixed in #12


:bomb: Bug: command dtm show config -p harbor print the wrong config sample

What Happened?

./dtm show config -p harbor

---
# core config
varFile: "" # If not empty, use the specified external variables config file
toolFile: "" # If not empty, use the specified external tools config file
state: # state config, backend can be local or s3
  backend: local
  options:
    stateFile: devstream.state

---
# plugins config
tools:
  - name: github-repo-scaffolding-golang
    instanceID: default
    options:
      owner: YOUR_GITHUB_USERNAME_CASE_SENSITIVE
      repo: go-webapp-devstream-demo
      branch: main
      image_repo: YOUR_DOCKER_USERNAME/go-webapp-devstream-demo
  - name: githubactions-golang
    instanceID: default
    dependsOn: ["github-repo-scaffolding-golang.default"]
    options:
      owner: YOUR_GITHUB_USERNAME_CASE_SENSITIVE
      repo: go-webapp-devstream-demo
      language:
        name: go
        version: "1.18"
      branch: main
      build:
        enable: True
      test:
        enable: True
        coverage:
          enable: True
      docker:
        enable: False

How to Reproduce?
./dtm show config -p harbor
Anything else
No response
DevStream Version
latest

The reason occurs in the logic of #794, the template param has the default value quickstart , and the template is judged first in the Show function, so that the logic will always output the configuration of quickstart.
https://github.com/devstream-io/devstream/blob/7d186cca87464a3d807ec3f363d80edbaa1acbcb/cmd/devstream/show.go#L64
https://github.com/devstream-io/devstream/blob/68054bedd91467175ab80a8ea152fb1fbe848c8f/internal/pkg/show/config/config.go#L11-L14
The quickest solution is to remove the default value of the template parameter.
Or we can discuss how to implement this more elegantly 👏


Small documentation updates

Looking at the DEV_README.md, it looks like the instructions there assume that the dll has been copied to our "source" directory first. However, this should not be necessary. If, in those instructions, you replace the two instances of "source/libnrniv.dll" with the location of the libnrniv.dll within the NEURON installation folder, I expect the instructions to work just fine.

When trying to run build_interface on windows, with mingw selected as both c and c++ compiler, I still got an error:
An error occurred while initializing child process:  While attempting to execute "REDACTED\matlabneuroninterface\2a03-961f-b749-17a2.exe":
CreateProcessW: Access is denied
So, we should add in the documentation, where mentioning (how to) run build_interface, that you need Administrator rights, and / or must have started Matlab as an adminstrator user, in order to be able to build the interface. Since at least clibgen.generateLibraryDefinition() needs that.

Fixed in edovanveen-phase2a_docupdates (https://github.com/edovanveen/matlabneuroninterface/tree/phase2a_docupdates).

Closed by pull request #42


Do not send splitTestRunVersion if there is no SplitTestRunName

Expected behavior
If there is no A/B test implicated in the pipeline, do not send data related to a splitTest.
Actual behavior
Usage Analytics will refuse the event if there is a SplitTestRunVersion but no SplitTestRunName.
Right now on every event, the name of the pipeline is sent as the SplitTestRunVersion.
Steps to reproduce the behavior
Run any scenario, it should refuse the usage analytics event.

This is a bug that was introduced with https://github.com/coveo/uabot/pull/73
Sending the pipeline all the time with each event as splitTestRunVersion makes the UA service refuse the event if there is no AB test setup on that pipeline.
Fix: Only send splitTestRunVersion if there is a splitTestRunName also included.


Start work for dispatch for caveats by adding MembershipSet

The MembershipSet helper type provides helper methods for easily combining the results of downstream dispatch checks to determine the membership of resources for which determined results (MEMBERSHIP) or caveated results (CAVEAT_MEMBERSHIP) apply, along with the caveat expressions and their context(s)
First part of #846

Updated


SPARQL Endpoint

An icon for a SPARQL endpoint would be cool aswell

I added

https://github.com/k00ni/semicon/blob/master/icons/svg/sparql_endpoint.svg


How do I update the map with new Markers?

Hello!
I have on my app an input bar that user can type the address, and then it should show a new pin for that address. The only issue is that it is only showing the Marker if I drag the map around or zoom in/out on it. I am pretty sure the issue is on GoogleMaps.ready stuff I am using to show this new marker. It is like this:
GoogleMaps.ready('map', function(map) {
     var marker = new google.maps.Marker({
          position: new google.maps.LatLng(newPositionLat,newPositionLng),
          title:"Hello World4",
          map: map.instance
     });
});

This code works, it is showing the Marker, not automatically, but only when I zoom in/out and the map need to "re-render". Is there some kind of GoogleMaps.update or something that automatically adds this new marker as soon I create it?

Hey @educkf take a look at this example to see how's it's done

@educkf Did that solve the problem?