provide/replace/conflict support

I'm trying to integrate this awesome plugin into my CMS.
It comes with some components already bundled (jQuery, Polymer, UIkit, etc.), but since they are not installed via this plugin I need to prevent duplicated installation to avoid conflicts.
In my case composer.json is generated on-fly, I've tried two forms, in both cases jQuery was installed:(
First:
{
    "repositories": [
        {
            "type": "package",
            "package": {
                "name": "modules/WebSockets",
                "version": "0.31.0+build-49",
                "require": {
                    "bower-asset/jquery": "^2.1",
                    "bower-asset/bootstrap": "^3"
                },
                "dist": {
                    "url": "/web/cscms.org/www/components/modules/Composer/empty.zip",
                    "type": "zip"
                }
            }
        },
        {
            "type": "package",
            "package": {
                "name": "plugins/Composer_assets",
                "version": "0.1.0+build-1",
                "require": {
                    "fxp/composer-asset-plugin": "^1.0.2",
                    "oyejorge/less.php": "~1.7",
                    "leafo/scssphp": "^0.1.7"
                },
                "dist": {
                    "url": "/web/cscms.org/www/components/modules/Composer/empty.zip",
                    "type": "zip"
                },
                "provide": {
                    "bower-asset/jquery": "3.0.0-alpha",
                    "npm-asset/jquery": "3.0.0-alpha",
                    "bower-asset/polymer": "0.5.5",
                    "npm-asset/polymer": "0.5.5",
                    "bower-asset/uikit": "2.21",
                    "npm-asset/uikit": "2.21",
                    "bower-asset/webcomponentsjs": "0.7.3",
                    "npm-asset/webcomponentsjs": "0.7.3",
                    "bower-asset/webcomponents.js": "0.7.3",
                    "npm-asset/webcomponents.js": "0.7.3",
                    "bower-asset/jsSHA": "1.6.1",
                    "npm-asset/jsSHA": "1.6.1",
                    "bower-asset/jssha": "1.6.1",
                    "npm-asset/jssha": "1.6.1"
                }
            }
        }
    ],
    "require": {
        "modules/WebSockets": "0.31.0+build-49",
        "plugins/Composer_assets": "0.1.0+build-1"
    }
}

Second:
{
    "repositories": [
        {
            "type": "package",
            "package": {
                "name": "modules/WebSockets",
                "version": "0.31.0+build-49",
                "require": {
                    "bower-asset/jquery": "^2.1",
                    "bower-asset/bootstrap": "^3"
                },
                "dist": {
                    "url": "/web/cscms.org/www/components/modules/Composer/empty.zip",
                    "type": "zip"
                }
            }
        },
        {
            "type": "package",
            "package": {
                "name": "plugins/Composer_assets",
                "version": "0.1.0+build-1",
                "require": {
                    "fxp/composer-asset-plugin": "^1.0.2",
                    "oyejorge/less.php": "~1.7",
                    "leafo/scssphp": "^0.1.7"
                },
                "dist": {
                    "url": "/web/cscms.org/www/components/modules/Composer/empty.zip",
                    "type": "zip"
                }
            }
        }
    ],
    "require": {
        "modules/WebSockets": "0.31.0+build-49",
        "plugins/Composer_assets": "0.1.0+build-1"
    },
    "provide": {
        "bower-asset/jquery": "3.0.0-alpha",
        "npm-asset/jquery": "3.0.0-alpha",
        "bower-asset/polymer": "0.5.5",
        "npm-asset/polymer": "0.5.5",
        "bower-asset/uikit": "2.21",
        "npm-asset/uikit": "2.21",
        "bower-asset/webcomponentsjs": "0.7.3",
        "npm-asset/webcomponentsjs": "0.7.3",
        "bower-asset/webcomponents.js": "0.7.3",
        "npm-asset/webcomponents.js": "0.7.3",
        "bower-asset/jsSHA": "1.6.1",
        "npm-asset/jsSHA": "1.6.1",
        "bower-asset/jssha": "1.6.1",
        "npm-asset/jssha": "1.6.1"
    }
}

Is there any possibility to explain that jQuery is already installed or provided by some other package, and do not even download it, just ignore from the beginning?
I'm using version 1.0.2.

Bower and NPM are two separate systems, so it is normal that you have jquery installed 2 times, once for NPM, and another time for Bower.

Looks like you didn't understand what I mean.
I've specified that jQuery is already installed:
    "provide": {
        "bower-asset/jquery": "3.0.0-alpha",
        "npm-asset/jquery": "3.0.0-alpha",

But anyway it was installed additionally via Bower by dependencies, I want to account at least this simple case when situation is quite obvious.
It seems to be a bug in dependencies resolution process.

I probably have not understood your problem ... and I still don't understand it.
The plugin solves nothing, It's the Solver SAT of Composer that makes the analysis. The plugin simply gets the URL via the NPM and Bower registries, and it creates the package for each repository. Basically, it simulates the presence of a file package.json, and creates a VCS Repository automatically. That is what do the plugin.
You talk about Bower, So if I understand correctly, you install Jquery with Bower, but jquery is also installed by the Composer plugin. Is that right? In this case, it's normal.

Not exactly.
I know, that in environment where everything is installed jQuery is already present (manually, without package managers, it is just present).
That is why I want to avoid duplicated installation.
For this reason I've added lines:
    "provide": {
        "bower-asset/jquery": "3.0.0-alpha",
        "npm-asset/jquery": "3.0.0-alpha",

with the hope that solver will see in case when some package depends on bower-asset/jquery (just jquery in Bower terms), jquery will be skipped from installation as already present.
This works fine for Composer packages, but conflicts have no effect on Bower and/or NPM decisions.

If you have a technical solution to achieve what you want, I am open to any proposal.

I'm not sure how exactly this plugin integrates with Composer currently.
Who manages dependencies resolution, do you feed everything to Composer or do that for Bower and NPM separately inside plugin alone?
If Composer's dependency resolution system would have all information, it would work already. If plugin does this by itself - than it should read provide/replace/conflict fields in addition to require, convert them into Bower/NPM format and use them in resolution process.

I've specified that jQuery is already installed:

shouldn't that provide definition go into the root of your composer.json instead of the repository part?

Why? It might be part of any package. Also I've tried to put it into root of composer.json (look at second part of first message), it didn't work either.

"bower-asset/jquery": "3.0.0-alpha", "npm-asset/jquery": "3.0.0-alpha",

The problem here seems to be that you provide jquery of version 3.0.0-alpha, which does not match the requirement of the first package so another one gets installed.
I played around a bit and found that only replace works and only if you put it in the root:
As far as I see this is solely related to composer and not the asset plugin.
{
    "repositories": [
        {
            "type": "package",
            "package": {
                "name": "plugins/Composer_assets",
                "version": "0.1.0+build-1",
                "require": {
                    "fxp/composer-asset-plugin": "^1.0.2",
                    "oyejorge/less.php": "~1.7",
                    "leafo/scssphp": "^0.1.7"
                },
                "dist": {
                    "url": "/tmp/composer/empty.zip",
                    "type": "zip"
                }
            }
        },
        {
            "type": "package",
            "package": {
                "name": "modules/WebSockets",
                "version": "0.31.0+build-49",
                "require": {
                    "bower-asset/jquery": "^2.1",
                    "bower-asset/bootstrap": "^3"
                },
                "dist": {
                    "url": "/tmp/composer/empty.zip",
                    "type": "zip"
                }
            }
        }
    ],
    "require": {
        "modules/WebSockets": "0.31.0+build-49",
        "plugins/Composer_assets": "0.1.0+build-1"
    },
	"replace": {
		"bower-asset/jquery": "*",
		"npm-asset/jquery": "*",
		"bower-asset/polymer": "0.5.5",
		"npm-asset/polymer": "0.5.5",
		"bower-asset/uikit": "2.21",
		"npm-asset/uikit": "2.21",
		"bower-asset/webcomponentsjs": "0.7.3",
		"npm-asset/webcomponentsjs": "0.7.3",
		"bower-asset/webcomponents.js": "0.7.3",
		"npm-asset/webcomponents.js": "0.7.3",
		"bower-asset/jsSHA": "1.6.1",
		"npm-asset/jsSHA": "1.6.1",
		"bower-asset/jssha": "1.6.1",
		"npm-asset/jssha": "1.6.1"
	}
}

The problem here seems to be that you provide jquery of version 3.0.0-alpha, which does not match the requirement of the first package so another one gets installed.

Normally it should cause dependencies resolution problem and result in error since composer doesn't allow two packages with the same name to be installed, isn't it?
Also look at https://github.com/francoispluchino/composer-asset-plugin/blob/master/Resources/doc/index.md#usage-with-multiple-versions-of-the-same-dependency
But whose bug is that?

But whose bug is that?

if you can reproduce it without having the asset plugin installed it is a composer issue, otherwise we need to look where the issue is when the plugin is installed.

@nazar-pc Can this question be closed?

@francoispluchino, I didn't check what @cebe recommended yet, let me try that first.

Seems to be Composer issue, reported as https://github.com/composer/composer/issues/4334


Add fuzzy search examples

Based on Lunch Break video
https://deploy-preview-717--zealous-morse-14392b.netlify.app/docs/3.7/arangosearch-fuzzy-search.html

@Simran-B LGTM, but would be nice to mention that similarity also affects overall score BM25/TFIDF


fix: Just to force a new release for angular components

This is being done mainly to force NPM to update to a new angular components release so that we can get updated documentation on NPM.
This has a side effect that I don't believe the -f is actually necessary

:tada: This PR is included in version 2.0.0-alpha.12 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.0.0-alpha.114 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-alpha.154 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-alpha.21 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Bi-directional protocol and serverbound API

Right now the network implementation and API are entirely client-sided. In order to re-implement server-side components (#26) and implement unit testing (#2), the network stack should support crafting packets for both directions.
The public API needs to be reworked a bit to support this.

Bi-directional parsing is now implemented under the rups::proto module (#29)


Fix/sdl 0189 changes according review

Changes for 3 comments:
softbuttons
according to suggestion from Colin for resetTimeoutValue as a single state
unused variable

@LiliiaShlikhtLuxoft resetTimeoutValue still used here

@LiliiaShlikhtLuxoft What is the sense to nesting a resetPeriod into resetTimeout ?

@LiliiaShlikhtLuxoft Please resolve conflicts


Trouble with package virtualbox

Hello.
I have installed я virtualbox4dsm v.6.0.8-130520 package at my device synology DS412+(cedarview),  OS: DSM 6.2.2-24922 Update 2.
But this package doesn't start. I run it and see load indication for short time but it doesn't start then.
What could be a problem and how can i fix it?
Thank you.

In your virtualbox shared folder you should have scripts.log file. It should give you an idea of what is going on. If you can't figure it out add it to the issue here so I can take a look.

Thank you for your answer! Unfortunately I can't figured out my problem. If you'll help me with it, I'll be grateful! You can find a logfile in attached.
scripts.log

Can you post content of /etc/vbox/vbox.cfg and output of following commands
 lsmod | grep vbox and  ps aux | grep VirtualBox

/etc/vbox/vbox.cfg
No such directory in my device(
lsmod attached
lsmod_log.txt

Other files I can't attached because I can't find such directory

You should run
lsmod | grep vbox
and then
ps aux | grep VirtualBox
but I guess problem is missing /etc/vbox/vbox.cfg which is required.
I suggest you first uninstall virtualbox package and then install again.

lsmod | grep vbox

ps aux | grep VirtualBox

Should I really REinstall virtualbox?

Not reinstall,  you must first uninstall then install.  Config file /etc/vbox/vbox.cfg is absolutely necessary for virtualbox.  Alternatively you can create that file by hand, but uninstall and then install will confirm that there is no problem with package. If it is not working after un/install process then for some reason package is not installing properly on your nas. Here is sample

I deleted virtualbox4dsm and reInstall it. After installation, I launched it. Log of this operation you can find in attached( scripts.log)
**** Start at Mon Aug 19 11:27:20 MSK 2019 **** Starting VirtualBox kernel modules Loading driver from: /var/packages/virtualbox4dsm/target/drivers/cedarview/3.10.105 vboxdrv loaded successfully vboxnetflt loaded successfully vboxnetadp loaded successfully vboxpci loaded successfully Starting VirtualBox kernel modules. Done! Starting VirtualBox web service stop vm's is not enabled. add  VBOXWEB_USER=vbox in /etc/vbox/vbox.cfg to enable vbox wol starting Traceback (most recent call last):   File "/var/packages/virtualbox4dsm/target/wol/vboxwolservice.py", line 229, in <module>     svc.start(iniFile)   File "/var/packages/virtualbox4dsm/target/wol/vboxwolservice.py", line 78, in start     self.run()   File "/var/packages/virtualbox4dsm/target/wol/vboxwolservice.py", line 94, in run     url = self.ConfigSectionMap('Credentials')['url']   File "/var/packages/virtualbox4dsm/target/wol/vboxwolservice.py", line 62, in ConfigSectionMap     options = self.Config.options(section)   File "/usr/lib/python2.7/ConfigParser.py", line 279, in options     raise NoSectionError(section) ConfigParser.NoSectionError: No section: 'Credentials' vbox wol done **** Start finished at Mon Aug 19 11:27:24 MSK 2019 ****
Thus the application doesn't start, moreover /etc/vbox/vbox.cfg doesn't create with installation

That is weird. Well to fix this you can manually create vbox.cfg. Unfortunately creating package for arch that you cant test is not easy, a lot if stuff can go wrong.
Reason why installation is broken might be found in install log file, try to execute
cat /var/log/synopkg.log | grep virtualbox4dsm
as root.

I created /etc/vbox/vbox.cfg, then queried cat /var/log/synopkg.log | grep virtualbox4dsm , log in attached
log.txt
I started virtualbox4dsm and it doesn't work. Log in attached.
scripts (2).log
What do you think should I do next?

Well this is strange, it looks like install finished successful as far as I see from log file. I forgot to say to create /etc/vbox/vboxwolservice.ini, that will remove this traceback error. but what is problematic is this

su: user root
does not exist
what do you get when you execute?
/opt/VirtualBox/VBoxManage list vms

Thank you for your help!
I've created  /etc/vbox/vboxwolservice.ini. I already have root rights.
When I executed this
su - root -s /bin/sh -c '/opt/VirtualBox/VBoxManage list vms'
got this output
root@xxxxx:~# su - root -s /bin/sh -c '/opt/VirtualBox/VBoxManage list vms'
/opt/VirtualBox/VBoxManage: /lib/libstdc++.so.6: no version information availabl                            e (required by /opt/VirtualBox/VBoxManage)
/opt/VirtualBox/VBoxManage: /lib/libstdc++.so.6: no version information availabl                            e (required by /opt/VirtualBox/VBoxManage)
/opt/VirtualBox/VBoxManage: /lib/libstdc++.so.6: no version information availabl                            e (required by /opt/VirtualBox/VBoxDDU.so)
/opt/VirtualBox/VBoxManage: /lib/libstdc++.so.6: no version information availabl                            e (required by /opt/VirtualBox/VBoxRT.so)
/opt/VirtualBox/VBoxManage: /lib/libstdc++.so.6: no version information availabl                            e (required by /opt/VirtualBox/VBoxRT.so)
/opt/VirtualBox/VBoxManage: /lib/libstdc++.so.6: no version information availabl                            e (required by /opt/VirtualBox/VBoxXPCOM.so)
/opt/VirtualBox/VBoxManage: /lib/libstdc++.so.6: no version information availabl                            e (required by /opt/VirtualBox/VBoxXPCOM.so)

And after executed this query ps aux | grep VirtualBox
Got this
/etc/vbox$ ps aux | grep VirtualBox
root      7708  0.0  0.0   9008   660 ?        S    10:47   0:00 grep VirtualBox

What do you get for output when you execute?
sudo /var/packages/virtualbox4dsm/target/scripts/vboxweb-service.sh start

root@xxxxx:~# sudo /var/packages/virtualbox4dsm/target/scripts/vboxweb-service.sh start
Starting VirtualBox web service
 does not exist
Error 0 setting webservice authentication library to null
Starting FAIL!
Starting VirtualBox web service. Done!

For some reason web service can't start what do you get after running
sudo /opt/VirtualBox/vboxwebsrv

I got it
root@xxxxx:~# sudo /opt/VirtualBox/vboxwebsrv
/opt/VirtualBox/vboxwebsrv: /lib/libstdc++.so.6: no version information available (required by /opt/VirtualBox/vboxwebsrv)
/opt/VirtualBox/vboxwebsrv: /lib/libstdc++.so.6: no version information available (required by /opt/VirtualBox/vboxwebsrv)
/opt/VirtualBox/vboxwebsrv: /lib/libstdc++.so.6: no version information available (required by /opt/VirtualBox/VBoxRT.so)
/opt/VirtualBox/vboxwebsrv: /lib/libstdc++.so.6: no version information available (required by /opt/VirtualBox/VBoxRT.so)
/opt/VirtualBox/vboxwebsrv: /lib/libstdc++.so.6: no version information available (required by /opt/VirtualBox/VBoxXPCOM.so)
/opt/VirtualBox/vboxwebsrv: /lib/libstdc++.so.6: no version information available (required by /opt/VirtualBox/VBoxXPCOM.so)
Oracle VM VirtualBox web service Version 6.0.8
(C) 2007-2019 Oracle Corporation
All rights reserved.
00:00:00.017232 main     VirtualBox web service 6.0.8 r130520 linux.amd64 (May 13 2019 16:01:09) release log
00:00:00.017258 main     Log opened 2019-08-22T09:50:55.240462000Z
00:00:00.017263 main     Build Type: release
00:00:00.017282 main     OS Product: Linux
00:00:00.017286 main     OS Release: 3.10.105
00:00:00.017289 main     OS Version: #24922 SMP Wed Jul 3 16:35:48 CST 2019
00:00:00.017396 main     DMI Product Name: GRANITE WELL
00:00:00.017421 main     DMI Product Version: 1.0
00:00:00.017723 main     Host RAM: 3947MB (3.8GB) total, 2226MB (2.1GB) available
00:00:00.017775 main     Executable: /opt/VirtualBox/vboxwebsrv
00:00:00.017778 main     Process ID: 30188
00:00:00.017780 main     Package type: LINUX_64BITS_GENERIC
00:00:00.090025 main     IPC socket path: /tmp/.vbox-root-ipc/ipcd
00:00:00.681957 SQPmp    Socket connection successful: host = default (localhost), port = 18083, master socket = 8
00:00:05.612784 Watchdog Statistics: 0 websessions, 0 references

Unfortunately I'm out of ideas. Judging on your output web service is working, there is something that is problematic with your setup which prevents starting web service. I don't know if that is specific to your setup or general for cedarview. Without actual box it is impossible for me to debug.
Btw. after running sudo /opt/VirtualBox/vboxwebsrv you should be able to open phpvirtualbox if configuration file for it is correct. What you could do, if phpvbox is working is to modify vboxweb-service.sh where you would replace content of start method like this
`start() {
if [ ! -f ${PIDFILE} ]; then
echo "Starting VirtualBox web service";
[ -z "$VBOXWEB_USER" ] && exit 0
    check_single_user $VBOXWEB_USER
    vboxdrvrunning || {
        fail_msg "VirtualBox kernel module not loaded!"
        exit 0
    }

    /opt/VirtualBox/vboxwebsrv --background > /dev/null 2>&1

    # ugly: wait until the final process has forked
    sleep 1
    PID=`pidof vboxwebsrv 2>/dev/null`
    if [ -n "$PID" ]; then
        echo "$PID" > $PIDFILE
        RETVAL=0
    echo "Starting SUCCESS!"
    else
        RETVAL=1
        echo "Starting FAIL!"
    fi
else
	echo "It looks like VirtualBox is running, delete ${PIDFILE} if it's not.";
fi

echo "Starting VirtualBox web service. Done!";
return $RETVAL

}`

Due to inactivity I'm closing this issue.


Fix: tiny visual glitch in poke with connector errorType

Description
Tiny fix

Our SOC-2 Type-II status requires us to review all code. Please assign a reviewer with PMRR so that someone can review post-merge :pray:

LGTM

Our SOC-2 Type-II status requires us to review all code. Please assign a reviewer with PMRR so that someone can review post-merge 🙏

Oups sorry about that. Will do next time.


dysk: 2.6.1 -> 2.7.1

Diff: https://github.com/Canop/dysk/compare/v2.6.1...v2.7.1
Changelog: https://github.com/Canop/dysk/blob/v2.7.1/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Are the manpage and the shell completions automatically copied to the right place?

They weren't, but now they are


fix(nix): use builtins.path instead of builtins.filterSource

Jenkins CI uses '@' in its workspace directory name. Nix seems to not like that. builtins.path { name = ... } lets you choose the directory name when copying the source files to the nix store directory.

bors try

bors cancel

bors try

bors cancel

bors merge

bors merge


feat: export fetchMock

Export RequestMockFactory so that user can mock the request as same as we do within the repo

Thanks for the suggestion @inca , I've updated as you suggested 👍


JS Integration tests

JS Integration tests would be a great way to both have examples and do full tests from top to bottom. Cross extension to RPC calls to testing the api-augmentation.
Examples

https://github.com/AcalaNetwork/Acala/tree/master/ts-tests

Notes

@demisx to help with the CI portion
mocha is suggested as the most common for web3 uses
May need to setup a manual sealing version

This may be useful:  https://joelhooks.com/jest-and-github-actions

What folder/file structure would work best?  Some tests will be straightforward but others may require more than one script file and data files.
src/tests
          ├── test1
          │       ├── func.js
          │       ├── func.test.js
          │       └── .....
          ├── test2
          │       ├── func.js
          │       ├── func.test.js
          │       ├── data.json
          │       └── .....
          ├── ....

do we have an idea about what workflows we should be testing?

do we have an idea about what workflows we should be testing?

Now that we are more stable we could have a deployment integration test. I think that is the only integration test we would want.
Wouldn't be too hard to setup with the docker container running as a service in GitHub Actions.

do we have an idea about what workflows we should be testing?

I think most of them will be basic happy path. Create a schema, create a message, create an msa.
Perhaps also some replay attack e2e tests

Closed with #637


How to turn off validation in Swagger UI?

{"schemaValidationMessages":[{"level":"error","message":"Can't read from file http://<private host name>:9000/docs/swagger.json"}]}

Is your swagger spec showing up? This error message swagger UI gave isn't very helpful 😞

The validator badge in Swagger UI uses the publicly hosted version of the swagger validator. If you want to update it to either turn off the badge or point to a locally hosted validator which is able to reach your endpoint, you can set the validatorUrl property in the Swagger UI index.html page. See the Swagger UI github page for more info.
If you are using docker to host your swagger UI endpoint, see this docker hub image. I recently made a contribution that allows you to override the validator within the image.

Hi, My case is little different , I am using Swagger UI Webjars-3.14.2 to generate Swagger UI JS files in my project structure dynamically, as a result if i introduce index.html file with validateURL:null attribute, the original from web jars retains back every time upon mvn compile. Could you put me in right direction.


Dev page has out of date info about docs.

The development page currently references the old and new doc pages, but the old is deprecated or retired?
Currently the docs folder serves two sites: the new site that is under construction and the current site that is deployed to readthedocs.io. The new site is currently not available for preview just yet. For the current site, documentation is generated automatically by readthedocs when pushing to master, but you can also generate a local copy by running:

dev_tools/docs/build-rtd-docs.sh
The HTML output will go into the dev_tools/rtd_docs/sphinx/_build directory.

Also is it possible to generate docs for the new site in any form locally?
P2 - we should do it in the next couple of quarters

The development page currently references the old and new doc pages, but the old is deprecated or retired?

I think we thought about retiring in the past, but there were some packages depending on us that needed our RTD (or something like that) does anyone know if this is still the case ? @vtomole ?

Also is it possible to generate docs for the new site in any form locally?

Unfortunately no, you can run notebooks locally, but there is no way to stage them for viewing as they would appear on the site without being a Googler.

Is this fixed?

If the merged removal is sufficient, I think so!

If the merged removal is sufficient, I think so!


Integration with tryshowtime

As we have artists and we would like to help highlight them, their artwork seamlessly, we should have integration with them
As this should be an effort against RMRK "protocol", we can be helpful here once we will have our own API.
ref https://twitter.com/plha_se_hlasi/status/1404789137694203911

They don't exist anymore


DOC: HuggingFaceEmbeddings support for HF_HOME environment variable

URL
https://api.python.langchain.com/en/latest/embeddings/langchain_huggingface.embeddings.huggingface.HuggingFaceEmbeddings.html#langchain_huggingface.embeddings.huggingface.HuggingFaceEmbeddings
Checklist

[X] I added a very descriptive title to this issue.
[X] I included a link to the documentation page I am referring to (if applicable).

Issue with current documentation:
Currently, the documentation of the cache_folder keyword only mentions the SENTENCE_TRANSFORMERS_HOME environment variable. It appears that the HF_HOME variable is also considered and takes precedence over SENTENCE_TRANSFORMERS_HOME if no cache_folder keyword is provided. I tested this on Linux with current versions of all involved modules.
Idea or request for content:
The documentation should be amended to include handling of HF_HOME.

@Jofthomas is this coming from within huggingface sdk?


Ability to easily add a FieldAlias for the current property mapping

I would love to figure out how to easily create a FieldAlias for the current property that I'm mapping. Something like this:
map.Properties(p => p
  .Keyword(f => f.Name(u => u.SomeProperty).Alias("some")));

instead of this:
map.Properties(p => p
  .Keyword(f => f.Name(u => u.SomeProperty))
  .FieldAlias(a => a.Name("some").Path(f => f.SomeProperty));

I was trying to dig into the property descriptors to see if there was some way I could reach outside of the current mapping to add another mapping, but that doesn't appear to be possible. Thoughts?

Closing due to age/inactivity. Please let us know if this issue is still a relevant priority for your usage?


Clean up temporary testing reliance on @tnozicka resources

Noted in the codebase in TODOs.
OpenShiftServiceTestBase.createProjectAndApplyTemplate

CatapultResourceIT

CatapultIT

@tnozicka In fixing this issue to set the template file to be https://raw.githubusercontent.com/redhat-kontinuity/jboss-eap-quickstarts/kontinu8/helloworld/.openshift-ci_cd/pipeline-template.yaml, getting:
Caused by: java.lang.RuntimeException: Failed to create BuildConfig from openshift-project-template.json. io.fabric8.kubernetes.client.KubernetesClientException: Failure executing: POST at: https://localhost:8443/oapi/v1/namespaces/test-project-1470864109348/buildconfigs. Message: Jenkins pipeline template openshift/jenkins not found. Received status: Status(apiVersion=v1, code=500, details=null, kind=Status, message=Jenkins pipeline template openshift/jenkins not found, metadata=ListMeta(resourceVersion=null, selfLink=null, additionalProperties={}), reason=null, status=Failure, additionalProperties={}).

This running against Origin:master built locally.  Ideas?

@tnozicka Changes above which generate the error

Ignore @tnozicka : I need to configure locally to add the template:
https://gist.github.com/ALRubinger/c93a84aab735aaf2eeb78f0e27f20e0a
...but at the moment can't get either "admin" or "system" users to do it.  Working on it.

6f7ce66


A page for the framework would be nice to have

I just stumbled upon this framework and I see it as a really good click it would be nice to have a webpage something similar to echo https://echo.labstack.com/ like getting started i think they have the best documentation among any framework I have seen.
Also, I know that fasthttprouter still lacks the support for http2 is http2 supported in this even though like it's built on top of it because lots of people including me will be hesitant to adopt it for a full-blown production use

Hi @gopherine,
Sorry for delay.
I know that a website is a good idea, but now i don't have too much time to do it now.
But I try to it do as soon as possible, even a basic website.
About http2 is under development in fasthttp, and I don't know when it will be ready. Sorry!


Azure doesn't show tables in a server

Issue Type: Feature Request
On my left hand menu, it only shows the server IP Address. When I click to the drop down arrow, it didn't show any thing.
Azure Data Studio version: azuredatastudio 1.24.0 (eb3c0f5d9df86b9490606aab7fffaa253b0aca4d, 2020-11-10T23:35:27.682Z)
OS version: Darwin x64 20.1.0

Duplicate of https://github.com/microsoft/azuredatastudio/issues/12992


Problems after running

Error occurred after the operation, did not show the running route, and should be 1 green and 4 Blue points, only 4 blue point now, and now run after landing the flash back phenomenon, the authors hope to see this message.

It doesn't matter at all! Any how, there is no such a checker to check whether the route is valid.
It only matters whether the total distance of your account increase or not.

Obsessive compulsive disorder, did not show the map is not good.


True secure with HTTPS

HTTPS is currently unsecure. It is necessary to add certificate validation in future versions. Read more: https://maakbaas.com/esp8266-iot-framework/logs/https-requests

Added in v1.0.0


chore(playwright): add playwright

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Code with `` and equation $$ can't work

It seems description delete them automatically, how should I set the option to turn it off?

I have used remark and rehype to do it by myself, but it's worth some options in my view as description may be use rendered in abstract in some article list card.

In fact, I find a easier way to get description for me. We can get full fresh markdown string from post.body. So if you are migrated from hexo or hugo, just get substring before<!-- more -->.

Great catch @LeenHawk ! This is definitely something I want to add support for, currently I use  mdast-to-string to get the text of a node which sanitizes all HTML/markdown from the string.
What should the API for this could look like? Currently I am thinking of :

Exposing the mdast node to the transform function

export default defineConfig({
  markdown: {
    remarkPlugins: [
      [remarkDescription, {
        transform: (text, { node }) => toHtml(toHast(node))
      }]
    ]
  }
});

This option gives you complete freedom over how the node is transform into text, but it is a but more advanced to setup and you have to parse the mdast to html yourself

Add a html option to toggle between text and HTML

export default defineConfig({
  markdown: {
    remarkPlugins: [
      [remarkDescription, {
        html: true
      }]
    ]
  }
});

This option is a lot simpler, but it is less flexible than the first and would require more dependencies by default

I think first, for someone migrated from hexo or hugo like me, we have made <!-- more --> in all post, so we just get the substring before this label and rendered it. You should focus those need to more options.
It's my code for this:
import { type CollectionEntry } from "astro:content";
import {unified} from 'unified'
import remarkParse from 'remark-parse'
import remarkMath from 'remark-math'
import remarkRehype from 'remark-rehype'
import rehypeMathjax from 'rehype-mathjax'
import rehypeStringify from 'rehype-stringify'
function outputBeforeMore(str) {
  const index = str.search(/<!--\s*more\s*-->/i);
  if (index !== -1) {
    return str.substring(0, index);
  }
  return str;
}
async function getdescription(post: CollectionEntry<"posts">) {
  const descriptionraw:string = outputBeforeMore(post.body);
  const description=await unified()
  .use(remarkParse)
  .use(remarkMath)
  .use(remarkRehype)
  .use(rehypeMathjax)
  .use(rehypeStringify)
  .process(descriptionraw)
  return String(description);
}

The only inconvenient thing is I don't find astro render string API, the remote markdown seems to be dealt with importing all package in frontmatter by myself.

In fact, a more tricky but feasible way used in hugo is you only parse automatically when user don't add <!--more--> label, and it works well. You don't need much work for this choice and don't need to add options for it like second option.

Ahh, interesting! You use the node option with remarkDirective to emulate this use-case, for example:
import { defineConfig } from 'astro/config';
import remarkDescription from 'astro-remark-description'
import remarkDirective from 'remark-directive'

export default defineConfig({
  markdown: {
    remarkPlugins: [
      remarkDirective,
      [remarkDescription, {
        node: node => node.type === 'containerDirective' && node.name === 'description'
      }]
    ]
  }
});

# Heading

Some text

:::description
Some more text
:::

I should add an example of this use case in the README
For the option to render the node to HTML I think I am going to add both options I mentioned, that way you can easily parse it to HTML or do something custom using the transform function

I played around with this a little more and I was able to get the to work with the <!-- more --> comment using a configuration like this:
import { defineConfig } from 'astro/config';
import remarkDescription from 'astro-remark-description'

// https://astro.build/config
export default defineConfig({
  markdown: {
    remarkPlugins: [
      [remarkDescription, {
        node: (node, i, parent) => {
          const sibling = parent?.children[i + 1]
          return sibling?.type === 'html' && sibling?.value === '<!-- more -->'
        }
      }]
    ]
  }
});

I played around with this a little more and I was able to get the to work with the <!-- more --> comment using a configuration like this:
import { defineConfig } from 'astro/config';
import remarkDescription from 'astro-remark-description'

// https://astro.build/config
export default defineConfig({
  markdown: {
    remarkPlugins: [
      [remarkDescription, {
        node: (node, i, parent) => {
          const sibling = parent?.children[i + 1]
          return sibling?.type === 'html' && sibling?.value === '<!-- more -->'
        }
      }]
    ]
  }
});

Fantastic work, thank you.

I am going to keep this issue open until I create a PR to add HTML rendering as a feature


fix: Correct version pin to support 1.4+ and protect from upcoming provider v5 changes

What this PR does / why we need it:

Correct version pin to support 1.4+ and protect from upcoming provider v5 changes

The main implementation version requirements have been updated to constrain the AWS provider to stay within v4 in order to avoid issues with the upcoming v5 changes. The Terraform version requirements have been updated to support 1.4+ but restrict from a 2.0 release (if that were to happen - no plans at this time)

Which issue(s) this PR fixes:
Fixes #394
Quality checks

[ ] My content adheres to the style guidelines
[ ] I ran make test or make e2e-test and it was successful

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Closing as coming rearchitecture work will address this or make it no longer applicable


Rename App

Rename App
Update script delivery
Update privacy policy
Update CGU

Hot fix on 0.11.0 version

@B3nz01d I think can be closed now ?


Datatables stateSaving "conflicts"

If stateSave is enabled when page is refreshed and different data is shown in table, it doesn't unhide previously hidden columns. E.g.
There are columns A, B and C. C column is empty.
New query is performed page is refreshed and column C now longer empty, however it remains hidden.

So you're saying its the same table, but completely different data? Is the data different in just that one page you're on, or the entire table?

It's same table, but data in there is different. Plugin indeed hides only empty columns, but it doesn't unhide columns which were hidden previously if page gets refreshed. I don't know what would be nice solution for this as someone might had hidden some columns "manually", otherwise I would add:
                if (emptyCount === data.toArray().length) {
                    api.column(this.index()).visible(false);
                } else {
                    api.column(this.index()).visible(true);
                }

Sorry, but im still confused. You're using the stateSave feature of DT, and hide a column (which will tell DT to hide it on reload), but you want HEC plugin to unhide that column?

Yes I am using stateSave. If HEC hides column it never gets unhidden, so dependant on the data at some point someone can get table where all data is hidden: e.g.:
Table has column A, B and C:
Query1:
Column C is empty and gets hidden. State is saved.
Query2:
Column B is now empty, C is not empty. B gets hidden, C remains hidden because of "saveState".
Query3:
Column A is now empty, B and C are not empty. However none of columns are shown at this stage.
state() contains structure where its defined which columns are hidden.
As I have mentioned some user might have set column C for example hidden "on purpose".

Ok, this one will take a little more work than the other request you made, but ill definitely look into it

@Lapkritinis, im working on this now, trying to think of the best way to approach it. Do you think columns that were hidden via stateSave should be overridden on the initial page load, and set as visible? How about the opposite?
I think maybe HideEmptyColumns should override the column visibility in the StateSave all together.

Ok, this was fixed on the last update, heres the relevant code


[DOC] Orphaned doc links in OpenSearch Dashboards

What do you want to do?

[ ] Request a change to existing documentation
[ ] Add new documentation
[ ] Report a technical problem with the documentation
[x] Other

Tell us about your request. Provide a summary of the request and all versions that are affected.
In OpenSearch Dashboards, we have a centralized location where we define all documentation links. Within that file, there's an entire section that lists links in the application that don't actually lead anywhere specific (they all point to the documentation site root):
https://github.com/opensearch-project/OpenSearch-Dashboards/blob/4fd67de27eb1d942450ee8272d2de2bb8ef2f96c/src/core/public/doc_links/doc_links_service.ts#L396-L428
          logstash: `${OPENSEARCH_WEBSITE_DOCS}`,
          winlogbeat: `${OPENSEARCH_WEBSITE_DOCS}`,
          siem: `${OPENSEARCH_WEBSITE_DOCS}`,
          indexPatterns: {
            loadingData: `${OPENSEARCH_WEBSITE_DOCS}`,
            introduction: `${OPENSEARCH_WEBSITE_DOCS}`,
          },
          management: {
            opensearchDashboardsGeneralSettings: `${OPENSEARCH_WEBSITE_DOCS}`,
            opensearchDashboardsSearchSettings: `${OPENSEARCH_WEBSITE_DOCS}`,
            dashboardSettings: `${OPENSEARCH_WEBSITE_DOCS}`,
          },
          scriptedFields: {
            scriptFields: `${OPENSEARCH_WEBSITE_DOCS}`,
            scriptAggs: `${OPENSEARCH_WEBSITE_DOCS}`,
            painless: `${OPENSEARCH_WEBSITE_DOCS}`,
            painlessApi: `${OPENSEARCH_WEBSITE_DOCS}`,
            painlessSyntax: `${OPENSEARCH_WEBSITE_DOCS}`,
            luceneExpressions: `${OPENSEARCH_WEBSITE_DOCS}`,
          },
          visualize: {
            guide: `${OPENSEARCH_WEBSITE_DOCS}`,
            timelineDeprecation: `${OPENSEARCH_WEBSITE_DOCS}`,
          },
          addData: `${OPENSEARCH_WEBSITE_DOCS}`,
          vega: `${OPENSEARCH_DASHBOARDS_VERSIONED_DOCS}`,
          dateMath: `${OPENSEARCH_WEBSITE_DOCS}`,

Some of these need to be created (such as vega, which is tracked in https://github.com/opensearch-project/documentation-website/issues/1178). Others may already exist or could be linked to existing OpenSource docs. For example, a user suggested that dateMath could likely be linked to https://opensearch.org/docs/latest/opensearch/supported-field-types/date/#full-date-formats. We may also decide to remove some of these links altogether if they are no longer relevant.
What other resources are available? Provide links to related issues, POCs, steps for testing, etc.
Related to https://github.com/opensearch-project/documentation-website/issues/751

@joshuarrrr I'm working on this issue

@joshuarrrr I've updated links related to this issue. Below are the updates. Will you review these to make sure the links are to the appropriate topic? For example, painless. Also, I don't have permission to edit file in the Dashboards repo. How do I update the file? Note that some of these items need documentation. For instance, Dashboards management and timeline deprecation.
logstash: https://opensearch.org/docs/latest/tools/logstash/index/,
winlogbeat: https://opensearch.org/docs/latest/tools/index/,
siem: https://opensearch.org/docs/latest/security-analytics/index/,
indexPatterns: {
loadingData: https://opensearch.org/docs/latest/im-plugin/index/,
introduction: https://opensearch.org/docs/latest/im-plugin/index/)
management: {
opensearchDashboardsGeneralSettings: ${OPENSEARCH_WEBSITE_DOCS},
opensearchDashboardsSearchSettings: ${OPENSEARCH_WEBSITE_DOCS},
dashboardSettings: ${OPENSEARCH_WEBSITE_DOCS},
},
scriptedFields: {
scriptFields: https://opensearch.org/docs/latest/api-reference/script-apis/index/,
scriptAggs: https://opensearch.org/docs/latest/aggregations/pipeline-agg/,
painless: https://opensearch.org/docs/latest/api-reference/script-apis/exec-stored-script/,
painlessApi: https://opensearch.org/docs/latest/api-reference/script-apis/exec-script/,
painlessSyntax: https://opensearch.org/docs/latest/data-prepper/pipelines/expression-syntax/,
luceneExpressions: https://opensearch.org/docs/latest/search-plugins/knn/filter-search-knn/,
},
visualize: {
guide: https://opensearch.org/docs/latest/dashboards/visualize/viz-index/,
timelineDeprecation: ${OPENSEARCH_WEBSITE_DOCS},
},
addData: https://opensearch.org/docs/latest/dashboards/discover/index-discover/,
vega: https://opensearch.org/docs/latest/dashboards/visualize/vega/,
dateMath: https://opensearch.org/docs/latest/field-types/date/,


Training with Sparsity

This process can be fine-tuned with a "Normal Trained"  model?
I prune InceptionV3 by fine-tuning with a pre-trained ImageNet model, and find it slow to be sparse(base_lr=0.01, 30epoch). I wonder whether if it is possible to use fine-tuning.
And how do you think InceptionV3 which has many branchs? I find it's a trouble to implement InceptionV3prune.py.

It is better to train from start with sparsity. We haven't tried fine-tuning with sparsity on a pretrained model before.
To implement pruning, you just need to collect all the scaling factors in bn layers. Use model.modules() to go through all the modules.

@Eric-mingjie , always thanks for quick answer.
I have implemented InceptionV3prune.py, but found accuracy drops a lot after prune.
The trouble of InceptionV3prune.py is that it has many branchs. So it means the cfg_mask can't be read one by one in turn. My main modification is to solve this problem.

The significant drop in accuracy could be due to that the model is not sparse. You can analyze the magnitude of scaling factors in the models.

@Eric-mingjie ,yes. The model is not sparse enough.
One more question,  druning the sparsity training, the channel_selection is necessary?

The use of channel_selection depends on the architecture you use and the way you want to do pruning. For VGG, no channel selection is needed. For ResNet and DenseNet, channel selection is normally needed.
However, if you use mask implementation, channel selection is not needed for any model.


Add possibility to use Role Import as the value for role attribute

https://github.com/FidelLimited/serverless-plugin-warmup/blob/d4b2bd91c766b2f51b880e545d990dee768e6150/src/index.js#L159

Hi @azhurtbayev-klikatech ,
Can you elaborate a bit on what exactly are you asking for?
I don't really understand.

Hello @juanjoDiaz ,
At the moment, the only way to specify a role for the Warmup function is to specify it as string value.
I use serverless additional stacks plugin to define all project roles within a separate stack.
Now, I try to define a role for warmup function like this:
custom:
  warmup:
    name: ${{self:custom.appName}}-warmup
    role: {"Fn::ImportValue": "${{self:custom.appName}}-warmup-role-arn"}

In this case, I cannot apply this role to the function, because it is not of a string type.

Thanks for such a quick fix!
You're just like Flash ;)


fix(ui): Convert Tailwind CSS config to ESM

Convert the Tailwind CSS config file from CJS to ESM syntax to fix the build with Node 22.12.0 1.

@lamppu @Etsija @mmurto According to the linked ticket another option would have been to change the file extension from .js to .cjs. If you would have preferred that for some reason, feel free to make a follow-up PR.


Update python version for databuilder

Summary of Changes
Update the readme
Tests
What tests did you add or modify and why? If no tests were added or modified, explain why. Remove this line
Documentation
What documentation did you add or modify and why? Add any relevant links then remove this line
CheckList
Make sure you have checked all steps below to ensure a timely review.

[ ] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[ ] PR includes a summary of changes.
[ ] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

[ ] PR passes make test

Codecov Report

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

@@            Coverage Diff             @@
##           master     #229      +/-   ##
==========================================
- Coverage   75.94%   75.70%   -0.24%
==========================================
  Files          90       90
  Lines        3791     3791
  Branches      347      329      -18
==========================================
- Hits         2879     2870       -9
- Misses        817      821       +4
- Partials       95      100       +5

Impacted Files
Coverage Δ

databuilder/loader/file_system_neo4j_csv_loader.py
84.33% <0.00%> (-4.82%)
:arrow_down:

...ilder/transformer/regex_str_replace_transformer.py
91.66% <0.00%> (-4.17%)
:arrow_down:

...abuilder/extractor/snowflake_metadata_extractor.py
93.44% <0.00%> (-1.64%)
:arrow_down:

...tabuilder/extractor/postgres_metadata_extractor.py
93.54% <0.00%> (-1.62%)
:arrow_down:

databuilder/publisher/neo4j_csv_publisher.py
84.15% <0.00%> (-1.00%)
:arrow_down:

databuilder/task/neo4j_staleness_removal_task.py
56.98% <0.00%> (ø)

Continue to review full report at Codecov.

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


Add deserialize_default_from_empty_object.

In addition to deserialize_default_from_null, this also treats empty objects ({}) as null.
Also, I fixed the deserialize_default_from_null. It would actually treat an Err as null because I accidentally simplified the code too much and moved the .unwrap_or from an Option to a Result.

You mean undefined as in absent? I think you still have to add #[serde(default)] to handle absent fields.

You mean undefined as in absent? I think you still have to add #[serde(default)] to handle absent fields.

No no, I meant this:
{ "value": undefined }

Is this valid json? If it is, should we also treat undefined in the way we do null and {}?

Is this valid json?

No, JSON only supports null.

@reitermarkus Thank you!


add a trigger for updating a specific column

Is it possible to add to the library support for triggers that are triggered only when certain columns in the table are updated?
expected sql of such a trigger:
CREATE TRIGGER trg_update_trigger
AFTER UPDATE OF column1 ON your_table
FOR EACH ROW
EXECUTE FUNCTION your_update_function();

syntax how it would look like:
modelBuilder.Entity<YourEntity>()
            .AfterUpdateOf(x => x.column1
                .Action(x => x.ExecuteRawSql("some sql")));

You can try something like
modelBuilder.Entity<YourEntity>()
            .AfterUpdate(trigger => trigger
                .Condition(refs => refs.Old.Column1 != refs.New.Column1)
                .ExecuteRawSql("some sql"));

that's what I did, but this code generates the following sql
CREATE FUNCTION myFunction() RETURNS trigger as $LC_TRIGGER_AFTER_UPDATE_MYFUNCTION$
BEGIN
    IF NEW."Column1" <> OLD."Column1" OR NEW."Column2" <> OLD."Column2" THEN
    ....
    END IF;
RETURN NEW;
END;
$LC_TRIGGER_AFTER_UPDATE_MYFUNCTION$ LANGUAGE plpgsql;

CREATE TRIGGER LC_TRIGGER_AFTER_UPDATE_MYFUNCTION AFTER UPDATE
ON your_table
FOR EACH ROW EXECUTE PROCEDURE myFunction();

and I need the trigger condition to be triggered not in the trigger body, as it is here.
CREATE TRIGGER trg_update_trigger
AFTER UPDATE OF column1 ON your_table
FOR EACH ROW
EXECUTE FUNCTION your_update_function();

is there any possibility with the next update to add support for triggers on certain table columns?


Add compact refract as a format

Closing since compact refract is no longer specified in Refract specification. Can re-open if/once compact refract is revived.


Register without bit-fields

How to define control register without bit-fields? All examples I have encountered contain bit-fields within the register. However, sometimes (most often?) there is only single bit-field with the same width as the register width.
The following works
  register {
    name 'reg0'
    bit_field { name 'foo'; bit_assignment width: 32; type :rw  }
  }

but is the following one also supposed to work?
  register {
    name 'foo'
    width 32
    type rw
  }

I also would like to omit the width if the field data occupies whole register.

RgGen does not support registers without bit fields.
Therefore, you need to define a bit field even if a register has single bit field only.
Implementation of RgGen supposes that a register must have bit fields so a lot of changes is needed to support such feature. So I don't intend to implement such feature.

@m-kru ,
Can you close this issue if you have no further topics?


improve latex and fix --- separator issue

This is really two PRs..and I am lazy to split them.

improve latex detection, allow, for example, $\text{$x$}$

fix the separator problem mentioned in https://github.com/jonschlinkert/sublime-markdown-extended/commit/e864385465ea0a2c921f2fd04dacc78850bf1fd. If the first row doesn't contain ---, it will never include the front matter.

@randy3k I added you as a collaborator, feel free to merge! thanks again!

@thanks, the latex fix may need more work, there are many special cases to take care of.
Will merge it once it is done.

Thanks @randy3k :cat2:

agreed! thanks @randy3k!


fixed an issue in TestWALRestoreCorrupted

Fixed a small issue with test case "TestWALRestoreCorrupted" as the test routine and cut method may update the same segment file concurrently and resulting in error sometime.

👍


Missing title tag on the page

Make sure that all pages have proper title
Fix these:

Same for meta description


HARP-11894 Fixing partially missing tiles

The problem was that the perspective division looses information about
whether the tile was behind the camera or behind the far plane, which
means we can't sort the tiles correctly. Tiles behind the near plane
are therefore given a distance of -1 and those past the far plane
are given a distance of +1. Note, we know that the tiles are visible
because they intersect the frustum.
Signed-off-by: Jonathan Stichbury<EMAIL_ADDRESS>

retest this please

retest this please

retest this please

retest this please


[OPPRO-347] Enable caching for Velox data connector

What changes were proposed in this pull request?
This patch enables the caching feature for Velox. Cache file(s) will be generated on Spark context initialization, based on the configurations, velox will asynchronously caching the contents when reading from hdfs/s3.  The feature should be helpful if reading from slow/remote storage.
Below configurations are added:
    spark.gluten.sql.columnar.backend.velox.cacheEnabled // enable or disable velox cache, default off
    spark.gluten.sql.columnar.backend.velox.cachePath  // the folder to store the cache files, default to /tmp
    spark.gluten.sql.columnar.backend.velox.cacheSize  // the total size of the cache, default to 1GB
    spark.gluten.sql.columnar.backend.velox.cacheShards // the shards of the cache, default to 1
    spark.gluten.sql.columnar.backend.velox.cacheIOThreads // the threads for cache promoting, default to 1

Note the current patch does not reuse the cache from last run - a new cache file will be generated for each new spark-context.
Signed-off-by: Yuan Zhou<EMAIL_ADDRESS>How was this patch tested?
manually test

Hi @akashsha1 could you please help to review this?

Hi @akashsha1 could you please help to review this?

sure. @paul-amonson please review as well.

I don't see any code issues but I do have a question:  The velox configuration variables, specifically "spark.gluten.sql.columnar.backend.velox.cacheEnabled", are not found in the facebook velox source tree or the special gluten velox source tree. How is configuration passed to and consumed by velox?

I don't see any code issues but I do have a question: The velox configuration variables, specifically "spark.gluten.sql.columnar.backend.velox.cacheEnabled", are not found in the facebook velox source tree or the special gluten velox source tree. How is configuration passed to and consumed by velox?

This was answered out of band. Thanks.

Can we merge this PR?

should we set spark.gluten.sql.columnar.backend.velox.cacheIOThreads  to 5 as Orri suggested? @akashsha1 what's the test result?

should we set spark.gluten.sql.columnar.backend.velox.cacheIOThreads to 5 as Orri suggested? @akashsha1 what's the test result?

Don't have the result yet. Let's merge the PR first, and we can iterate based on finding to update the default value.
@zhouyuan - can you provide me a gluten jar to validate the efficiency of this change?


feat(zig): snippets

Adds Simple Zig Snippets

Looking for recommendations to add more snippets

Zig users, let me know if you need more snippets.

ops there the title is back

Well since no one wanted added snippets, I am going to look for used things instead.

I have fixed the link I will do a git rebase when I have a chance

I am currently out of town so I am not able to rebase the pr.

I have rebased it and fixed the link.

@OkelleyDevelopment do you want me to squash my commits or it's fine?

i'm encountered error when using this :(

 i've encountered error when using this :(

Can you provide the line number, so I can make a pr to fix this snippet.

 i've encountered error when using this :(

Found the cause issue, fixed in #531. Sorry for the mess of prs, I to rename a branch and etc.


Video Conference does not work on Huawei p10 (Android 7)

Hi Muaz,
I just install android version of video conference from Google Play on huawei p10 (Android 7.0)
but it does not work anymore. only i see leave button and video does not play.
please note that i check app permissions and all thing is OK.
I just install on some samsung devices and all of them work fine.
Also, I build android version using crosswalk web view, but does not work.
have any idea?

In case Anyone wondering about this issue :
Starting from Android 6+, you need to request permissions on runtime , it not sufficient to just declare them in the manifest .
For that, I use Cordova permission plugin,  Found Here.
To avoid problems , I ask for runtime permissions list on DeviceReady this way:
`var permissions = cordova.plugins.permissions;
  var list = [
    permissions.MODIFY_AUDIO_SETTINGS,
    permissions.RECORD_AUDIO,
    permissions.CAMERA,
    permissions.WRITE_EXTERNAL_STORAGE,
    permissions.ACCESS_NETWORK_STATE,
    permissions.INTERNET
  ];
  list.forEach(function (perm) {
    permissions.checkPermission(perm, function( status ){
      debug("status of cheking==="+deser(status));
        if ( status.hasPermission ) {
            debug("Yes :D -> "+perm);
        }
        else {
            debug("No :( -> " +perm);
            permissions.requestPermission(perm, function ( status ) {
              debug("status of requesting==="+deser(status));
            }, function (err) {
              debug('Error request=='+ deser(err));
            });
        }
    },function (err) {
      debug('Error check=='+ deser(err));
    });
  });

function debug(src){
console.log(src);
}
// to read JSON objects
function deser(src){
return JSON.stringify(src);
}`

I'm using same code in the video-conferencing and scalable-broadcast apps.

https://webrtcweb.com/cordova-apps/video-conferencing/www/js/index.js


Automatically generate feature gates for all peripherals

This PR adds cfg(feature = "...") gates for all the peripherals that are generated. It means that if people only want to use certain peripherals, that they don't need to compile all of them which can take a very long time.
This PR is not yet finished (I want to see CI though :wink: ) – ideally there should also be a Cargo.toml skeleton that's generated or even just a .txt file which includes all available feature flags for users to expose in the Cargo.toml.

Thanks for the PR, @spacekookie.

This PR is not yet finished (I want to see CI though wink )

We have disabled running CI on PRs in this repo because the test suite takes very long to run and CI resources are shared with other repos under the rust-embedded org. However, I have you granted you permissions to test this manually by including bors try in your comment. You can use that to manually test this PR.

This PR adds cfg(feature = "...") gates for all the peripherals that are generated.

In that case you should also update the test script to check the generated crates with --all-features to continue checking that code generation for peripherals works as intended.
https://github.com/rust-embedded/svd2rust/blob/77ce4d59fcd49eceedc9b9b26d8c99c59cdd04d3/ci/script.sh#L21

bors try

bors try

:see_no_evil:
bors try
please

cc/ @jamesmunns @japaric I renamed the file it generates again because it seems to be causing issues with CI.
I could always just call it a txt file that people copy stuff out of. However, I'm not 100% why Cargo.toml even collides with anything.
Maybe you have some better insights on how it works and/or how to proceed from here on out.

bors try

bors try

bors try

bors try

bors try

bors try

bors try
Apologies to anyone with email notifications.

Notes to self:

Clean up the "cat Cargo.toml" line
Talk to @therealprof about pinning dependencies
Refactor the Cargo.toml.bkp dance?

bors try

bors try

bors try

bors try

@japaric it works for Cortex m, but causes problems for Msp430. Right now it shouldn't be a breaking change, since I've added a CLI flag to enable the behavior, so by default current behavior should be the same.
I can take some time tomorrow to try and close this out, or at least make it specifically only work for Cortex m for now.

@jamesmunns Thanks for the update / clarification. My preference would be to not add features that affect codegen. The rationale is that most people use pre-generated crates (so they don't have to compile svd2rust when compiling a device crate) and there would be no way for them to pick between "all peripherals" and "peripherals behind Cargo features". This could to alternate device crates popping up on crates.io (e.g. nrf51 and nrf51-peripherals).

Okay, I really think I got it this time,
bors try

bors try

bors try

bors try p=10

bors retry

bors r-

bors r+

bors try

bors try
(please?)

I think its because it worked one time? I can close this PR, squash my changes, and open a new one anyway, if thats easier.

Closing PR because of how gross this has gotten.


Bot also replies in dm

Describe the bug
the bot will reply on the command even if its dm
To Reproduce
Dm the bot cm!help
It will reply
Expected Behavior
No response
Screenshot/ Video
https://tanshadow.is-inside.me/JNGa5Jqn.png
Desktop (please complete the following information)
No response
Additional context
No response

I guess commands work in DM :thinking:

my second issue has the solution for this one

Commands should not work in dms imo

Commands should not work in dms imo

Nope, that's okay. It becomes handy sometimes

Commands should not work in dms imo

Nope, that's okay. It becomes handy sometimes

What would you do after discord sets it to be as-if no messages can be read... They have that on their roadmap... Also replying in dms requires special setup after v13 this means they want us to avoid the use of dms in bots

What would you do after discord sets it to be as-if no messages can be read... They have that on their roadmap... Also replying in dms requires special setup after v13 this means they want us to avoid the use of dms in bots

Oh, I was not aware of this. :thinking: Then this makes sense. Please make a relative or for fixing this


Task: Style Table View

Describe the task
Style the table view per the mockups

The property list table view doesn't seem like a high priority right now considering all the other things right now. I wrote a ticket to hide it for now. https://github.com/CodeForPhilly/vacant-lots-proj/issues/249 So let's hold all work on this.

@brandonfcohen1 I moved this back to the Backlog.

@thansidwell  @nlebovits  Not clear on what the desired style was - doesn't look like there is an update to the table view of the properties (that's what I'm assuming you're referring to.) Should we close this?

@Moylena This doesn't apply anymore. I'll just archive this.


Allow pre existing config ref in NFDeployment

Tests done

Test with pre-existing config ref. Done

/approve

/lgtm
/approve


_pickle.UnpicklingError: pickle data was truncated

I am trying to get RL4LMs to work, and to achieve this, I've made the docker image using the instructions in the README file. After building the container, I tried running the following command in the container(under Quick start):
python scripts/training/train_text_generation.py --config_path scripts/training/task_configs/summarization/t5_ppo.yml
(docker run -it rl4lms /bin/sh
followed by
python scripts/training/train_text_generation.py --config_path scripts/training/task_configs/summarization/t5_ppo.yml)
However, I'm getting an UnpicklingError. The complete debugging info right before the error is as follows:
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-03-02 17:33:13.379149: W tensorflow/compiler/xla/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer.so.7'; dlerror: libnvinfer.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /usr/local/nvidia/lib:/usr/local/nvidia/lib64
2023-03-02 17:33:13.379385: W tensorflow/compiler/xla/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer_plugin.so.7'; dlerror: libnvinfer_plugin.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /usr/local/nvidia/lib:/usr/local/nvidia/lib64
2023-03-02 17:33:13.379401: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Cannot dlopen some TensorRT libraries. If you would like to use Nvidia GPU with TensorRT, please make sure the missing libraries mentioned above are installed properly.
2023-03-02 17:33:45.988942: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-03-02 17:33:48.806522: W tensorflow/compiler/xla/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer.so.7'; dlerror: libnvinfer.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /usr/local/nvidia/lib:/usr/local/nvidia/lib64
2023-03-02 17:33:48.806802: W tensorflow/compiler/xla/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer_plugin.so.7'; dlerror: libnvinfer_plugin.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /usr/local/nvidia/lib:/usr/local/nvidia/lib64
2023-03-02 17:33:48.806862: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Cannot dlopen some TensorRT libraries. If you would like to use Nvidia GPU with TensorRT, please make sure the missing libraries mentioned above are installed properly.
Killed
# Traceback (most recent call last):
  File "/opt/conda/lib/python3.8/multiprocessing/forkserver.py", line 280, in main
    code = _serve_one(child_r, fds,
  File "/opt/conda/lib/python3.8/multiprocessing/forkserver.py", line 319, in _serve_one
    code = spawn._main(child_r, parent_sentinel)
  File "/opt/conda/lib/python3.8/multiprocessing/spawn.py", line 126, in _main
    self = reduction.pickle.load(from_parent)
_pickle.UnpicklingError: pickle data was truncated

I am using a macbook pro 2019 with intel chip 2.4 GHz Quad-Core Intel Core i5. Should I make any changes to the Dockerfile to be able to run this or is there any other script I can try?
note: I've also tried installing this on my machine (and a different machine with M1 chip) but have run into issues with incompatibility of specific package versions with python 3.10 and 3.11 I have on these two machines.

Hi, I met the same problem. Have you remember how to solve it?


How to disable this addon for a single story?

For other stories, to disable a plugin (in the example, named centered), I do the following:
export const Simple = () => {
  return
Hello World
;
};

Simple.story = {
  parameters: {
    centered: {
      disable: true,
    },
  },
};

To my understanding, I am using component story format (CSF): https://storybook.js.org/docs/formats/component-story-format/
I have the JSX addon enabled globally, for all stories. Is it possible to disable the addon for a single story in CSF mode?
Context: I am rendering a table component, and the story is serving as a performance test with 1 million table rows. Creating the JSX for this story is expensive (which is totally reasonable) and the browser slows down. Since the story is only for performance, I don't need the JSX addon tab.

I def feel like you should be able to. But If it doesn't I'll happily merge a PR!


Allow organization url to be passed.

Overview/Summary
Feature: Ability to pass organization  url as organization parameter. This would allow the ability to pass pipeline variable System.CollectionUri.
This PR fixes/adds/changes/removes

Organization on connect will strip URL to get the Organization Name

As part of this Pull Request I have

[X] Checked for duplicate Pull Requests
[ ] Associated it with relevant issues, for tracking and closure.
[X] Ensured my code/branch is up-to-date with the latest changes in the main branch
[X] Performed testing and provided evidence.
[ ] Verified build scripts work.
[X] Updated relevant and associated documentation.

@bjompen I've made these changes. Feel free to make any edits.

Added another mock, but this is awesome! Thank you so much for the contribution!
Merged.


Is it possible to get segmentation pixel masks?

Hi, Thanks for the tutorial. Is it possible to get actual pixels which are detected as hands instead of the bounding box as of your codes?

Need to train mask rcnn model to do that


[Bug] LunarNvim integration issues

Hi, first of all, congratulations on this great plugin.
I'm encountering some difficulties integrating it in the LunarVim config package.
lua\plugins.lua:
return require("packer").startup(function(use)
    ....
    use {'ray-x/navigator.lua', requires = {'ray-x/guihua.lua', run = 'cd lua/fzy && make'}, opt = true}
    ....
    require_plugin("navigator.lua")
    ....
end)

lua\init.lua:
....
require('my-navigator-lua')
....

lua\my-navigator-lua\init.lua:
require'navigator'.setup()

The fact is that it seems to install correctly (no errors reported), but it reports an error when starting Neovim:
Errore/i eseguendo C:\Work\MEGA\Portable\Neovim\config\init.vim:
riga  511:
E5108: Error executing lua ...er\opt\navigator.lua\lua\navigator\lspclient\clients.lua:49: table index is nil

Help with running it in LunarVim would be greatly appreciated!

Also referenced here:
https://github.com/ChristianChiarulli/LunarVim/issues/482

It seems navigator failed to handle pathname in windows. (I am not using windows ATM)
Could you help me check the following output  in your neovim cmd
: lua print(vim.fn.expand("$HOME"))
My output is /User/rayx
And this:
: lua print(vim.fn.expand("$HOME") .. "/.config/nvim")
My output is /User/rayx/.config/nvim
And this:
lua print(vim.fn.isdirectory(vim.fn.expand("$HOME") .. "/.config/nvim"))
My output is 1
Also this
: lua print(vim.fn.stdpath('config'))
I doubt that windows may output sth total different

Thanks for the prompt reply:
lua print(vim.fn.expand("$HOME")):
C:\Users\Myusername

My output is /User/rayx

lua print(vim.fn.expand("$HOME") .. "/.config/nvim"):
C:\Users\Myusername/.config/nvim

My output is /User/rayx/.config/nvim

lua print(vim.fn.isdirectory(vim.fn.expand("$HOME") .. "/.config/nvim"))
0

My output is 1

lua print(vim.fn.stdpath('config'))
C:\Users\Myusername\AppData\Local\nvim

Also add some code may help prevent error throw. Please update the plugin as well.

I didn't understand what actions I should take for doing as you suggested.

run:
: PackerUpdate
That will allow you update all plugins

Now it seems to have properly loaded. Great work.
I'll test it right now.

Hi,  @pidgeon777
Can you confirm the issue been solved in the latest version?

Thank you @ray-x , this issue has been solved, I had a chance to do more testing and I found some other issues which I just reported.


Games without content getting ratings.

I don't know if it's worry about the deadline, but it seems dubious behavior to submit games without content.
Perhaps the rules should say something about it.
There's more than one case in LD53 with this, but at least first one responded to comment:

And the first example above, the authors have made entries to LD before so it's hopefully not related to the spam situation.

Most games seems to have gotten content now but this one still seems to be a placeholder:
https://ldjam.com/events/ludum-dare/53/iter-vitae

I think it may help if I don't allow publishing games without something in the links section, or an upload.

I think it would be great to block publishing without any links (or upload) and saving after published without links (or upload).
A somewhat related phenomenon that I saw reports of games where the links return 404. Unfortunately I wasn't online when it was discussed and haven't gotten the links to the examples. Will post it here if I do get them.
Anyways, it would be nice if the server when saving a game could do a get_headers($url, 1) for all supplied links and ensure that there are no 4xx or 5xx responses before accepting the save.

Fair enough, and you're right, checking just to warn would still be useful.


Add unit test case

Signed-off-by: ZouYu<EMAIL_ADDRESS>
What this PR does / why we need it:
Add unit test case for TestEqual() in internal/urlutil/urlutil_test.go
Special notes for your reviewer:
If applicable:

[ ] this PR contains documentation
[x] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

Hi there. What’s the reason for these unit tests? Is there a case where this behaviour is not covered by other tests, or some code path this test case covers?

@bacongobbler , this test cases cover these code path.

How is that not tested by the test case on line 58?

I think so.
Without `line 59-60`
Running tool: /usr/local/go/bin/go test -timeout 30s -coverprofile=/tmp/vscode-gonkzK6N/go-code-cover helm.sh/helm/v3/internal/urlutil

ok  	helm.sh/helm/v3/internal/urlutil	0.002s	coverage: 75.0% of statements

Running tool: /usr/local/go/bin/go test -timeout 30s -coverprofile=/tmp/vscode-gonkzK6N/go-code-cover helm.sh/helm/v3/internal/urlutil

ok  	helm.sh/helm/v3/internal/urlutil	0.004s	coverage: 87.5% of statements


right off canvas + visibilty class bug

If you add a visibility class to the section tag of a right off canvas menu button the button bugs
Go to the "Tab Bar Section" -> "Right Section" example and add a visibilty class like "visible-for-small-up" to the right section to see it:
http://foundation.zurb.com/docs/components/offcanvas.html

Hmm visible-for-small-up seems to be screwing that up, but also doesn;t exist as a class. Thats just showing it for every screen size.
Using another seems to work.


Encrypt and decrypt an object

when I encrypt an object instead of only a string as in the example , I get no error but when tried to decrypt it , I only get the output as [object object], I can't get the real data , am I doing something wrong or hybrid-crypto-js can only encrypt and decrypt simple strings

If not, just stringify the object before encrypting then parse it back again after decrypting?

Hybrid-crypto-js only supports basic strings. Object could be converted into string using JSON.stringify() before encrypring and parsed after decryption using JSON.parse().


Make player health 20.0 by default

Objective
When testing making a minigame example, I noticed the player always spawned with 6 health, which is the default for living entities in Valence.
It would also cause a desync between the client and server, because the client implies that it has 20.0 health upon spawning.

The CI failures are unrelated.

In that case, let's close this and open an issue for it instead.


Nested slots

I haven't found a solution for nesting slots with synonyms
Like
@[paymentPeriod:hour]
	hour
	h

@[paymentPeriod:day]
	day
	d

@[paymentPeriod]
	@[paymentPeriod:hour]
        @[paymentPeriod:day]

I want to use nested perymentPeriod slot like this
No,[ per|per?] @[paymentPeriod]

But right now I get
- No, [[](paymentPeriod)d](paymentPeriod:day)
- No, [[da](paymentPeriod)y](paymentPeriod:day)
- No, [[](paymentPeriod)h](paymentPeriod:hour)
- No, [[hou](paymentPeriod)r](paymentPeriod:hour)
...

Hi,
Nesting slots is not a supported feature, because it is not clear what the generated sentence should look like. To me, using the templates you showed, the slot @[paymentPeriod] should generate something like this: [[hour](paymentPeriod:hour)](paymentPeriod). As you may know, this is not supported by Rasa.
To avoid having this kind of problems, I will add an error if Chatette detects nested slots in your templates.
Now, seeing your template, I think I understand what you want to be generated: you would like a slot called paymentPeriod that could contain either hour, h, day or d, is that right?
If so, the following template will achieve this:
~[paymentPeriod:hour]
	hour
	h
~[paymentPeriod:day]
	day
	d

@[paymentPeriod]
        ~[paymentPeriod:hour]
        ~[paymentPeriod:day]

On the other hand, if you want a slot called paymentPeriod:hour that could have the values hour or h and another one called paymentPeriod:day that could have the values day or d, then I'm afraid you will have to make 2 different slots and use them wherever you want, as in the following template:
@[paymentPeriod:hour]
	hour
	h
@[paymentPeriod:day]
	day
	d

~[some alias]
        ... [@[paymentPeriod:hour]|@[paymentPeriod:day]] ...

If what you're looking for is some kind of "categories of slots" (e.g. hour and day being 2 slots that are part of a paymentPeriod category), I am afraid Rasa doesn't support this kind of things. Therefore, Chatette doesn't support it neither. That being said, I might add something like this in the far future, but I'll need to find a solution to "translate" them to something Rasa would understand.
Hope this helps!

Right now I use the approach
No, @[salary_sum][ per|per?] [@[paymentPeriod:hour]|@[paymentPeriod:day]|@[paymentPeriod:week]|@[paymentPeriod:month]|@[paymentPeriod:year]]

to generate
- No, [4500.00](salary_sum) per [annum](paymentPeriod:year)
- No, [4500.00](salary_sum) per [d](paymentPeriod:day)
- No, [4500.00](salary_sum) per [day](paymentPeriod:day)
- No, [4500.00](salary_sum) per [h](paymentPeriod:hour)
- No, [4500.00](salary_sum) per [hour](paymentPeriod:hour)

but I want to make the template shorter
like
No, @[salary_sum][ per|per?] @[paymentPeriod]

I  see. Then, I suppose your solution is a good one. To make it easier to reuse your slots, I would suggest using a template such as this:
... your slot definitions ...
~[paymentPeriod]
   @[paymentPeriod:day]
   @[paymentPeriod:hour]
   ...

You can then use this alias as if it was a slot, rather than needing to write the same list of slots over and over again ;)
For the record, it is also sometimes useful to use variations of slots (but I’m guessing this will not be a solution in your case), like this:
@[paymentPeriod#hour]
   hour
   h
@[paymentPeriod#day]
   day
   d

You can then use the slot @[paymentPeriod] as if it was defined, and the generator will choose one of the variations at random. However, in that case, the name of the generated entity will be paymentPeriod and not the name of the chosen variation.
I understand that in your case, none of the solutions I suggested is totally satisfying. I’ll see if I can make some changes that would help in your situation, because I guess such cases must come up often.
Thanks for your questions :)

Thank you


如何把文字放到圆圈内，或者在圆圈内放背景图！

[ ] I am using English in this issue. 在这个 Issue 中我使用了英文（强烈建议）。

General Questions

PLEASE MAKE SURE OF ALL THE FOLLOWING OPTIONS IN REQUIRED FIELDS ARE TICKED (with x)!
Otherwise, the issue will not be answered.
And think before you tick. :)
请确保以下每项都打上勾了！并且确保都做了这些事哦~ 这将大量简化我们的工作流程，使你的问题更快速得到解答。

[ ] Required: I have read the document and examples and tried to solve it by myself. （必填）我读过了文档和教程，并且曾试图自己解决问题。
[ ] Required: I have searched for similar issues and that didn't help. （必填）我搜索过 issue 但是没有帮助。
[ ] Required: I have tried with the latest version of ECharts and still have this problem. （必填）我试过最新版本的 ECharts，还是存在这个问题。

In this issue, I have provided information with: 在这个 issue 中我提供了以下信息：

[x] Required: issue type；（必填）issue 类型 ✅
[x] Required: one sentence description in issue details；（必填）一句话的问题描述
[x] Required: demo；（必填）能反映问题的例子（如果你想提问或报 bug）
[x] Required: ECharts version；ECharts 版本

Issue Type

[ ] I have a question to ask about how to use ECharts to ...；我想提问如何使用 ECharts 实现某功能
[ ] I have a bug to report；我想要报 bug
[ ] I have a feature to request, e.g.: I'd like a new feature that ...；我需要一个新功能
[ ] I have a feature to enhance, e.g.: The current feature should be improved in the way that ...；我希望改进某个功能
[ ] There's something wrong with the documents；文档有些问题
[ ] Others, or I'm not sure which issue types to choose here；其他，或我不知道应该选什么类型

Issue Details

{DESCRIBE_HERE}
Expected Behavior

{DESCRIBE_HERE}
Current Behavior

{DESCRIBE_HERE}
Online Example

THIS IS REQUIRED FOR ALL BUG REPORTS AND QUESTIONS!!! 如果是提问或报 bug 一定要提供这一项！
{ADD_LINK_OF_DEMO_HERE}

{PASTE_RELATED_CODE_HERE}
Topics

[ ] Legend
[ ] Tooltip
[ ] Event
[ ] Performance
[ ] SVG
[ ] Map
[ ] ECharts GL
[ ] Third-party libraries, e.g.: Vue.js, React

Anything Else We Need to Know

{DESCRIBE_HERE}
Environment

ECharts version；ECharts 版本: {DESCRIBE_HERE}

[ ] It happens only on certain browsers or operating systems. 对于特定浏览器或操作系统才会出现的问题，请提供相应环境信息：{BROWSER_VERSION_OR_OS_INFORMATION_HERE}

如何把文字放到圆圈内，或者在圆圈内放背景图！

@wwwsoftwares You can refer to this example.
http://gallery.echartsjs.com/editor.html?c=xrJXD6fWhl

@wwwsoftwares
Sorry, border-radius is supported in image.

怎么加这个样式


add: ckpool package

The test currently fails with:
  shrinking RPATHs of ELF executables and libraries in /nix/store/5ji6fxrysnw974kvfx4gz6r9asizw049-ckpool-unknown
  shrinking /nix/store/5ji6fxrysnw974kvfx4gz6r9asizw049-ckpool-unknown/lib/libjansson.so.4.14.0
  shrinking /nix/store/5ji6fxrysnw974kvfx4gz6r9asizw049-ckpool-unknown/bin/notifier
  shrinking /nix/store/5ji6fxrysnw974kvfx4gz6r9asizw049-ckpool-unknown/bin/ckpmsg
  shrinking /nix/store/5ji6fxrysnw974kvfx4gz6r9asizw049-ckpool-unknown/bin/ckpool
  checking for references to /build/ in /nix/store/5ji6fxrysnw974kvfx4gz6r9asizw049-ckpool-unknown...
  RPATH of binary /nix/store/5ji6fxrysnw974kvfx4gz6r9asizw049-ckpool-unknown/bin/notifier contains a forbidden reference to /build/
  /nix/store/nkskw6n4swnlxjrwv47mpb4a4h8r85wg-stdenv-linux/setup: line 74: pop_var_context: head of shell_variables not a function context
  /nix/store/nkskw6n4swnlxjrwv47mpb4a4h8r85wg-stdenv-linux/setup: line 1422: pop_var_context: head of shell_variables not a function context
  /nix/store/nkskw6n4swnlxjrwv47mpb4a4h8r85wg-stdenv-linux/setup: line 1561: pop_var_context: head of shell_variables not a function context

The test currently fails with:
  shrinking RPATHs of ELF executables and libraries in /nix/store/5ji6fxrysnw974kvfx4gz6r9asizw049-ckpool-unknown
  shrinking /nix/store/5ji6fxrysnw974kvfx4gz6r9asizw049-ckpool-unknown/lib/libjansson.so.4.14.0
  shrinking /nix/store/5ji6fxrysnw974kvfx4gz6r9asizw049-ckpool-unknown/bin/notifier
  shrinking /nix/store/5ji6fxrysnw974kvfx4gz6r9asizw049-ckpool-unknown/bin/ckpmsg
  shrinking /nix/store/5ji6fxrysnw974kvfx4gz6r9asizw049-ckpool-unknown/bin/ckpool
  checking for references to /build/ in /nix/store/5ji6fxrysnw974kvfx4gz6r9asizw049-ckpool-unknown...
  RPATH of binary /nix/store/5ji6fxrysnw974kvfx4gz6r9asizw049-ckpool-unknown/bin/notifier contains a forbidden reference to /build/
  /nix/store/nkskw6n4swnlxjrwv47mpb4a4h8r85wg-stdenv-linux/setup: line 74: pop_var_context: head of shell_variables not a function context
  /nix/store/nkskw6n4swnlxjrwv47mpb4a4h8r85wg-stdenv-linux/setup: line 1422: pop_var_context: head of shell_variables not a function context
  /nix/store/nkskw6n4swnlxjrwv47mpb4a4h8r85wg-stdenv-linux/setup: line 1561: pop_var_context: head of shell_variables not a function context

solved this by configuring with --enable-static and using the generated libjansson.a


Added running tests in Docker image

run for Python 3.6 Linux build only

Note that I have skipped the one failing real FS test if inside a Docker container, though it also might be related to the Ubuntu version.
I'm also not too happy to run this together with the Python 3.6 build - I would prefer a separate build, but couldn't get it to mark alongside the matrix configuration in Travis.CI.
Maybe I will add a new matrix entry instead...

Ok, I moved the Docker build into an extra matrix entry, this is better now. I will merge this now.


Only build snippets on latest TFM

Also fixes #33975 by reformatting snippets in sdk/core.

API change check
API changes are not detected in this pull request.


Hover class occasionally doesn't get removed

This is clear when hiding the buttons at all times, except on hover/active. Just randomly moving the mouse over the table will show/hide most buttons, but occasionally some will get stuck. By inspecting the DOM we can see that the hover class doesn't get removed.
Here's a demo:
http://jsfiddle.net/nfvs/1mcfx0jg/3/

My initial guess is that this has the same root cause as https://github.com/zeroclipboard/zeroclipboard/issues/463

I thought that as well at first, but I thought I read in that bug that it was only about Chrome on OS X, and this is reproducible on Chrome, Firefox and IE on Windows. Re-reading it again, it might indeed be related.

+1


comparing tool gives wrong result for large files

VSCode Version: 1.28.1
OS Version: Windows 10

Steps to Reproduce:

Prepare one text file, say 14741KB large. Name it as "log1.txt". Each line of log1.txt may be like this:

	i=1, 0.000000*1.000000+0.000000=0.000000

Copy log1.txt to log2.txt, and change some lines of log2.txt to be diffirent than log1.txt's;

Putting log1.txt and log2.txt under same directory, say "log_cmp";

Using VSCode open "log_cmp" folder, and compare these two files

Does this issue occur when all extensions are disabled?: Didn't try that
Screenshot:

Note
If I decrease the compared files to only contain 5000 lines, the comparing tool works correctly.

The diffing algorithm is implemented to be time sensitive. If computing the diff takes longer than 5 seconds, then the returned result is a valid diff, but is not very precise.


Y24-166: Add new endpoint on V2 API for creating and querying Asset Audits

Closes https://github.com/sanger/asset_audits/issues/621
Changes proposed in this pull request

Create API V2 Asset Audit controller and resource
Update routing for the new controller

Instructions for Reviewers
[All PRs] - Confirm PR template filled
[Feature Branches] - Review code

Another thing I thought of, by the way.  The originating issue for this work says that the endpoints for Sequencescape V2 API should be documented.  I don't know if we have existing documents for the API.  If so, we need to add this new one to the documentation.  If we don't have documentation, we probably need to look at how to auto-generate them.


Pull latest code from "CU-86bxy3dkd - Develop Frontend UI for Project Details Configuration"

Task linked: CU-86bxy3dkd Develop Frontend UI for Project Details Configuration


AWS: store Terraform state remotely

[x] Enable S3 bucket versioning
[ ] Figure out S3 key path

Had to make some decisions around hardcoding variables or making the init/build process more involved for the end user. Opted to hardcode some values, documented everything in code and in the README, and hopefully life will be easier for the end user.


avoid deprecated new Buffer() ctor

@dbaeumer new Buffer() is deprecated and triggers tons of deprecation warnings in our Electron 3.0.x builds.
Refs: https://nodesource.com/blog/understanding-the-buffer-deprecation-in-node-js-10/

Thanks.


Font sizes in the header for longer service/org names look too small

Bug Report
What is the issue?
Font sizes in the header for longer service/org names look too small. The current example in the FEL  very small font when the browser width is "small mobile" (320px width).

Similarly, services such as https://www.digitalregulations.innovation.nhs.uk/ have long names which need additional testing with the new style of header.

AI and Digital Regulations Service header on my phone

@anandamaryon1 , for info - in case it's related to the work you're doing on headings.


Composing sagas that use selectors

We're trying to build a bunch of Saga-using components which can be embedded at various different levels of a store hierarchy. For example:

An ItemEditor component with it own React view, Redux reducers, and sagas that only deal with the editing of a single item
A MetaItemEditor component which embeds an ItemEditors, and also has some additional reducers and sagas to edit some metadata around that item

The sagas in ItemEditor don't know or care that they're inside of MetaItemEditor. In fact, there are other parts of the codebase where we just insert a single ItemEditor all by itself, so this works. But ItemEditor's sagas sometimes need to query the state of its store.
In Saga 0.9.x, we were able to just use getState and override it. We created a helper called combineSagas:
// Combine a collection of sagas into one saga, pass them a selector function
// to pull a relevant piece of state, and then start them all
export const combineSagas = (sagas, selector) => function*(getState) {
    const sagaArray = Array.isArray(sagas) ? sagas : Object.values(sagas);
    const boundSelector = () => selector(getState());
    for (let s of sagaArray){
        yield fork (s, boundSelector);
    }
};

and used it like this:
const watchItemEditor = combineSagas(
    itemEditorSagas, state => state.itemEditorState
);

But 0.10.x has made it more difficult. Now that we have to use select effects, the only way to override the getState function is now doing it directly inside runSaga. So we've ended up having to write our own custom middleware instead of using the built in sagaMiddleware:
// Combine a collection of sagas into one saga, pass them a selector function
// to pull a relevant piece of state, and then start them all
class CombinedSagas {
    constructor(sagas, selector) {
        const sagaArray = Array.isArray(sagas) ? sagas : Object.values(sagas);
        this.generator = function*() {
            for (let s of sagaArray){
                yield fork(s);
            }
        };
        this.selector = getter => () => selector(getter());
    }
}
export const combineSagas = (sagas, selector) =>
    new CombinedSagas(sagas, selector);

// Saga middleware that will correctly process CombinedSagas
export function buildCombinableSagaMiddleware(sagas){
    const sagaArray = Array.isArray(sagas) ? sagas : Object.values(sagas);
    let listeners = new Set();
    // Create middleware that will pull a Redux store
    return ({dispatch, getState}) => {
        // For each saga
        for (let saga of sagaArray) {
            // Check if this is a CombinedSagas
            const isCombined = saga instanceof CombinedSagas;
            // If it is, pull the sagsa from the generator prop
            const generator = isCombined ? saga.generator() : saga();
            const subscribe = (l) => {
                listeners.add(l);
                return () => listeners.delete(l);
            };
            runSaga(generator, {
                subscribe, dispatch,
                // If this is a CombinedSagas, set its getState to the
                // passed in selector; otherwise just give it the entire state
                getState: isCombined ? saga.selector(getState) : getState
            });
        }
        return next => action => {
            const result = next(action);
            listeners.forEach(l => l(action));
            return result;
        };
    };
}

This works, but it's pretty unwieldy. I'm wondering if there's a better way to do this that can still take advantage of the built-in sagaMiddleware.

You can pass getState explicitly as argument in sagaMiddleware.run
function* rootSaga(getState) {
  ...
}
const store = createStore(...)
sagaMiddleware.run(rootSaga, store.getState)

Aha, I see. So that would replicate the behavior of getState in 0.9.x.
I guess that works, but I'd prefer to use the new pattern of select effects. The problem, as I see it, is that it's difficult to override where select gets its state from at any level below the root saga. I don't know whether that's a problem with Redux-Saga, or if I've just been following an anti-pattern this whole time.

Sorry for the late replay.
As you said, overriding select effect is not supported. But I think passing a selector to the child saga would do the same thing
function* megaItemEditorSaga() {
  yield fork(childSaga, itemSelector)
}

function* itemEditorSaga(itemSelector) {
  yield select(itemSelector)
}


Sphinx docs

Summary

This PR addresses Jira ticket CISLUNAR-328
This PR addresses some issues that came up after deploying the documentation website onto our main branch. After much debugging, I realized that the .readthedocs.yaml file had to be moved up a directory for it to be recognized, which then allowed for the correct python version to be recognized for package installations.
Apologies for the large diff... it seems for some reason that some source files have a redundant diff (ie no changes are made but a diff still shows up)
Testing

Tested via deployment to the docs website
Notes

I also added saved a v0.1 version of the docs website by using git tags... pretty interesting stuff.
Comments

[ ] Add an x between the brackets if you commented your code well!
Not applicable, except for docstrings

I noticed "main module" is blank. Is that by design? https://cislunarsim.readthedocs.io/en/latest/main.html

I noticed "main module" is blank. Is that expected behavior? https://cislunarsim.readthedocs.io/en/latest/main.html

Ah, good observation. There were some other things I also saw that weren't being rendered. I think something happened after I merged main into this branch. I'll look into it

I noticed "main module" is blank. Is that expected behavior? https://cislunarsim.readthedocs.io/en/latest/main.html

Ah, good observation. There were some other things I also saw that weren't being rendered. I think something happened after I merged main into this branch. I'll look into it

@andrew032011 I fixed the issue. It was caused by a missing import. Everything should look good now!


Weather info

This is a great whatchface. It would be perfect showing wheater and activity  info. Thanks for your work

@trandafile
Thanks a lot. In the first place I wanna keep the face compact. Let me first fix some bugs, make a release with the weather info instead the activity info and than I will think about your proposal (actually I already have a concept in mind :))

@trandafile
I'm releasing version 1.7 where the weather info moved to the top right corner (you can choose between battery and weather). I hope it works for you :)


bootstrap-decorator not included when using browserify

Hi,
I've created a new project and installed angular-schema-forms via npm.
Included it
require('angular-schema-form');

When running browserify app.js it is missing bootstrap-decorator.js code? Not sure if this is meant to be included?

This has been recently reported #435. Pull requests welcome.


Add authorization header

I'm testing a local SPARQL endpoint that requires an Authorization header. Is it possible to add the header at lodlive.profile.js? If so, can you provide any examples please?

No, it is not possible.
LodLive uses JSONP call to handle Cross origin requests limitations... you cannot add any Authorization header to jsonp calls, what about a proxy?
you can use apache to convert a request parameter (that can be added in lodlive's configuration) into an Header
https://dino.ciuffetti.info/2015/11/http-request-header-query-string-parameter-apache-reverse-proxy/

OK, I will try the proxy approach, thanks.
The link you provided is not working though


Transparent mode failure: Runtime Error (could not resolve original destination)

Problem Description
as title description, i want to use Custom Routing of Transparent Proxying, but can't regular work

Steps to reproduce the behavior:
the NAT Gateway and MitmProxy are 2 devices，NAT Gateway is a desk route(IP is <IP_ADDRESS>)，MitmProxy is a pc（macbook pro,IP is <IP_ADDRESS>,Default Gateway is <IP_ADDRESS>），the test phone is android OS(IP is <IP_ADDRESS>,Default Gateway is <IP_ADDRESS>), all devices are working normally.

as pc, run mitmproxy as command :
mitmweb --mode transparent --showhost -p 9999
as route, run iptables rules
iptables -t nat -A PREROUTING  -s <IP_ADDRESS> -p tcp --dport 80 -j DNAT --to-destination <IP_ADDRESS>:9999;
iptables -t nat -A PREROUTING  -s <IP_ADDRESS> -p tcp --dport 443 -j DNAT --to-destination <IP_ADDRESS>:9999;
as phone, open browser address as http://www.ip138.com:8080/search.asp?action=mobile

System Information
Mitmproxy: 5.0.1
Python:    3.8.1
OpenSSL:   OpenSSL 1.1.1d  10 Sep 2019
Platform:  macOS-10.13.4-x86_64-i386-64bit

and i see the pf.py, looks like code line 24 doesn't work

have the same problem

Please take a look at https://docs.mitmproxy.org/stable/howto-transparent/. This seems to be a routing question, not a mitmproxy bug.


operator project-quay (3.8.8)

Add 3.8.8 to community operators

/merge possible

/merge possible