Update TestController.psm1 SetGlobalVariables to add root user check

Description
Update TestController.psm1's function SetGlobalVariables to add root user check. If set as "root", exit the test framework directly. Not sure here is the best place to check vmUsername. XMLSecretFile also changes the vmUsername, it is called in PrepareTestEnviroment function of child class.
Issue #
If do not add root user check, it will get lots of error when running LisAv2 related with /home/xxx paths.
Although the main instruction guide demonstrates that do not use root user account. When new user uses LisAv2, they may ignore this requirement, then need long time to find the failure reason.
Thank you so much.
PR information

[ ] The title of the PR is clear and informative.
[ ] There are a small number of commits, each of which has an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[ ] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes in the description.
[ ] Verify PR checker results and fix any coding errors.
[ ] Included LISAv2 sample test results to demonstrate code functionality.

Quality of Code and Contribution Guidelines

[ ] I have read the contribution guidelines.

==========================================================
PS C:\Users\xuli\workspace\repo\LISAv2> .\Run-LisaV2.ps1 -TestPlatform HyperV -RGIdentifier 20190225 -OsVHD C:\RHEL-8.0.
0-20190213.0-x86_64-GEN1-A.vhdx -TestNames KVP-BASIC-CHECKS -VMGeneration 1
02/28/2019 09:51:59 : [INFO ] Autogenerated test ID: PY52
02/28/2019 09:52:00 : [INFO ] Found branch information and updated with  add_root_user_check
02/28/2019 09:52:00 : [INFO ] Path length is too long for current working directory 'C:\Users\xuli\workspace\repo\LISAv2
'
02/28/2019 09:52:00 : [INFO ] Copying current workspace to C:\LISAv2\PY52
02/28/2019 09:52:04 : [INFO ] Working directory has been changed to C:\LISAv2\PY52
02/28/2019 09:52:04 : [INFO ] Logging directory: C:\LISAv2\PY52\TestResults\2019-28-02-17-52-04-8558
02/28/2019 09:52:04 : [INFO ] Setting parameter: TestPlatform = HyperV
02/28/2019 09:52:04 : [INFO ] Setting parameter: RGIdentifier = 20190225
02/28/2019 09:52:04 : [INFO ] Setting parameter: OsVHD = C:\RHEL-A.vhdx
02/28/2019 09:52:04 : [INFO ] Setting parameter: TestNames = KVP-BASIC-CHECKS
02/28/2019 09:52:04 : [INFO ] Setting parameter: TestIterations = 1
02/28/2019 09:52:04 : [INFO ] Setting parameter: VMGeneration = 1
02/28/2019 09:52:04 : [INFO ] Test parameters for HyperV have been validated successfully. Continue running the test.
02/28/2019 09:52:04 : [ERROR] Failed to update configuration files. '-XMLSecretFile [FilePath]' is not provided.
02/28/2019 09:52:04 : [INFO ] Set Test Location to GlobalConfiguration.Global.HyperV.Hosts.ChildNodes[0].ServerName
02/28/2019 09:52:05 : [INFO ] Updated C:\LISAv2\PY52\XML\GlobalConfigurations.xml file.
02/28/2019 09:52:05 : [INFO ] ------------------------------------------------------------------
02/28/2019 09:52:05 : [INFO ] HyperV Host            : 2019-auto
02/28/2019 09:52:05 : [INFO ] Destination VHD Path   : VHDs_Destination_Path
02/28/2019 09:52:05 : [INFO ] ------------------------------------------------------------------
02/28/2019 09:52:05 : [INFO ] Setting global variables
02/28/2019 09:52:05 : [ERROR] EXCEPTION: Root user cannot be used in the test configuration
02/28/2019 09:52:05 : [ERROR] Source: Line 181 in script C:\Users\xuli\workspace\repo\LISAv2\TestControllers\TestControl
ler.psm1.
02/28/2019 09:52:05 : [INFO ] Copying all files back to original working directory: C:\Users\xuli\workspace\repo\LISAv2.
02/28/2019 09:52:17 : [INFO ] Setting workspace back to original location: C:\Users\xuli\workspace\repo\LISAv2
02/28/2019 09:52:17 : [INFO ] LISAv2 exit code: 1

This is test-cases related, though in some platforms/scenarios I'm thinking you can have them run directly with the root account.
So we should not limit this, at best maybe throw a warning but not fail.
Anyhow, this should be covered as part of the README, and not controlled from automation end.
@juhlee-microsoft

@chvalean Agreed. We have an upcoming requirement to use root user directly.
@juhlee-microsoft @chvalean We can talk about this offline.

@lixuemin2016 I am not sure if I understand your PR. Accessing root user priv can be done in Test Case script level, and I am not sure why you put this error handling in TestController. If you need root user priv in test run-time, please refer other script like using runAsSudo parameter. If you are ok with my feedback, please close the PR.

@chvalean @juhlee-microsoft  @iamshital
Thank you so much for all your review comments.
Let me explain more for this PR.
When tested on HyperV platform, I did not notice that "non-root" user should be set up, then configured LinxuUsername as "root" in the GlobalConfigurations.xml,  test case failed by lots of errors, since there was no /home/root folder.
[Error] Failed to execute: dmesg > /home/root/initialBootLogs.txt
[Error] Unable to identify /home/root/state.txt
[Error]  dmesg >/home/root/FinalBootLogs.txt,
[Error] /home/root/detectLinuxDistro.sh
[Error] Doing Diffie-Hellman group exchange xxxxxxx
[WARN] Download error, attempt, Retrying for download
In other words,  it cannot use "root" account in the GlobalConfigurations.xml. Here "root" does not mean root user priv in the test case. if set up "root" account, I think that how about exiting the framework directly, or giving warning to avoid lots of error message?
Now I agree to close this PR, looks that have an upcoming requirement to use root user directly.


Updating tedious in mssql docs sample

In order to get the current mssql-based SQL Server sample working on https://github.com/GoogleCloudPlatform/nodejs-docs-samples/tree/main/cloud-sql/sqlserver/mssql we'll need to make sure mssql is using a newer version of tedious, which supports the connector config property which in turn enable support to the Cloud SQL Node.js Connector.
Once mssql is updated upstream we should be able to update our SQL Server doc sample to start using the Node.js Connector package.
Related to: https://github.com/GoogleCloudPlatform/cloud-sql-nodejs-connector/issues/140

If you want I can work on updating the docs (https://github.com/GoogleCloudPlatform/nodejs-docs-samples/tree/main/cloud-sql/sqlserver/mssql) to document usage with connector (in a similar manner that it is documented for tedious). Let me know.


registry_rebuild (and potentially) other projects missing some releases

Was running a routine composer update and noticed that registry_rebuild fails the dependency resolving process as Drupal packagist does not have any information about the latest recommended version of the project that was requested.
drupal.org says 2.2 is recommended while packagist only has up to 1.4.
This has happened before with another project and IIRC the releases parser was at fault at that time.

The dependency parsing on drupal packagist does not work in any case. Things are easier with a composer.json in the repo itself. I filled a issue in the registry_rebuild issue queue.
https://www.drupal.org/node/2622718#comment-10604008

:thumbsup:
Just curious: how does adding composer.json facilitate release info update?

We have this info-file parser library (https://github.com/drupal-composer/drupal-parse-composer/blob/master/src/Project.php#L50) and the parser stops if there is no valid info file. Drupal Packagist uses the composer.json in favour of the parser.

Package is up to date. https://packagist.drupal-composer.org/packages/drupal/registry_rebuild

Gotchya, and appreciate such a quick turnaround.

I just ran composer update with registry_rebuild pinned at 2.2 and it succeeded, however composer install then installs it under vendor/drupal/ instead of drush/. In my case this is configured via composer/installers install paths - vendor/drupal for type drupal-core, drush/{$name} for type drupal-drush:
"extra": {
  "installer-paths": {
    "vendor/drupal": [
      "type:drupal-core"
    ],

    // ...

    "drush/{$name}": [
      "type:drupal-drush"
    ],
  }
},

I checked your patch in the module's issue queue and project type seems to be correct there (drupal-drush), but any chance there's a discrepancy between that and the type value that the registry maintains?

Took me a while to understand the issue. The older releases of registry_rebuild are list on drupal packagist but as type library because these releases have no composer.json (https://packagist.drupal-composer.org/packages/drupal/registry_rebuild.json)
Please use the dev release (composer require drupal/registry_rebuild:7.2.x-dev) instead, this contains the committed composer.json from today.


Unable to run docker image

OS: Mac
Docker installed with some errors but completed with
Successfully built 7170abd7c78d
Successfully tagged manim:latest
gcc -pthread -fno-strict-aliasing -g -O2 -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -fPIC -DVERSION=1.3.5.dev0 -DHAVE_FREETYPE2 -Iagg2/include -Iagg2/font_freetype -I/usr/include -I/usr/include/freetype2 -I/usr/local/include/python2.7 -c aggdraw.cxx -o build/temp.linux-x86_64-2.7/aggdraw.o
cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for C/ObjC but not for C++
In file included from agg2/font_freetype/agg_font_freetype.h:28:0,
                 from aggdraw.cxx:84:
agg2/include/agg_scanline_storage_bin.h: In member function ‘unsigned int agg::scanline_storage_bin::byte_size() const’:
agg2/include/agg_scanline_storage_bin.h:247:38: warning: unused variable ‘sp’ [-Wunused-variable]
                     const span_data& sp = m_spans[span_idx++];
                                      ^
aggdraw.cxx: In function ‘PyObject* draw_new(PyObject*, PyObject*)’:
aggdraw.cxx:691:70: warning: deprecated conversion from string constant to ‘char*’ [-Wwrite-strings]
         PyObject* buffer = PyObject_CallMethod(image, "tobytes", NULL);
                                                                      ^
aggdraw.cxx: In function ‘agg::rgba8 getcolor(PyObject*, int)’:
aggdraw.cxx:924:72: warning: deprecated conversion from string constant to ‘char*’ [-Wwrite-strings]
         result = PyObject_CallFunction(aggdraw_getcolor_obj, "O", color);
                                                                        ^
aggdraw.cxx: In function ‘PyObject* draw_flush(DrawObject*, PyObject*)’:
aggdraw.cxx:1405:71: warning: deprecated conversion from string constant to ‘char*’ [-Wwrite-strings]
     result = PyObject_CallMethod(self->image, "frombytes", "N", buffer);
                                                                       ^
aggdraw.cxx:1405:71: warning: deprecated conversion from string constant to ‘char*’ [-Wwrite-strings]
In file included from agg2/font_freetype/agg_font_freetype.h:35:0,
                 from aggdraw.cxx:84:
agg2/include/agg_font_cache_manager.h: In instantiation of ‘void agg::font_cache_manager<FontEngine>::init_embedded_adaptors(const agg::glyph_cache*, double, double) [with FontEngine = agg::font_engine_freetype_int32]’:
aggdraw.cxx:490:60:   required from ‘void draw_adaptor<PixFmt>::drawtext(float*, PyObject*, FontObject*) [with PixFmt = agg::pixel_formats_rgba32<agg::order_rgba32>; PyObject = _object]’
aggdraw.cxx:2253:1:   required from here
agg2/include/agg_font_cache_manager.h:306:17: warning: enumeration value ‘glyph_data_invalid’ not handled in switch [-Wswitch]
                 switch(gl->data_type)
                 ^
In file included from aggdraw.cxx:86:0:
agg2/include/agg_path_storage.h: In function ‘PyObject* draw_ellipse(DrawObject*, PyObject*)’:
agg2/include/agg_path_storage.h:336:25: warning: ‘y’ may be used uninitialized in this function [-Wmaybe-uninitialized]
         *coord_ptr   = y;
                         ^
agg2/include/agg_path_storage.h:179:23: note: ‘y’ was declared here
             double x, y;
                       ^
agg2/include/agg_path_storage.h:335:25: warning: ‘x’ may be used uninitialized in this function [-Wmaybe-uninitialized]
         *coord_ptr++ = x;
                         ^
agg2/include/agg_path_storage.h:179:20: note: ‘x’ was declared here
             double x, y;
                    ^

Then trying to start the docker image
docker run --rm -v "$PWD/files":/app/files manim example_scenes.py WarpSquare

Traceback (most recent call last):
  File "extract_scene.py", line 14, in <module>
    from constants import *
  File "/app/constants.py", line 32, in <module>
    """)
Exception:
        Redefine MEDIA_DIR by changing the MEDIA_DIR
        environment constant or by changing
        media_dir.txt to point to a valid directory
        where movies and images will be written

I get this error even tho I have updated my MEDIA_DIR in constants.py
Any idea how to get this working in docker? (I also tried directly on mac with no luck)

Did you rebuild? Did the path exists?  You can try with /app, for example.

@damo1884 I had this issue too. The way I fixed it is by making a directory for the media_files and by passing in the correct environment variable that points at it.
I then couldn't access the output of the scene, so I had to share the media_files directory by way of a volume. Because I'm making frequent changes to the code, I've just decided to share my whole manim directory with the container. If you run the code below, it should do that for you.
mkdir media_files
docker run --rm --volume="$PWD:/app/" -e MEDIA_DIR=/app/media_files manim example_scenes.py WarpSquare

Solved, as @cnrmck gave the solution.


chore(ci): update dependencies for yocto build in ubuntu

Is this only a dependency update or are you tryiing to get the builds to succeed again?

I try to make it at least run. Not sure if I can get it to pass


[Compute] Add Compute Test for AutoPatches and Generating new SDK for compute-2020-06 version

This is merged to compute-2020-06 branch.
https://github.com/Azure/azure-sdk-for-net/commit/b927d329a3b95d34957d087ff3446011805c5d43


zephir built extension incorrectly on mac os 10.10

Hi,
I tried to install zephir on Mac OS 10.10.2 (Yosemite).
Everything is fine and I was able to build my dummy.so extension against PHP 5.4.35.
But when I add "extension=dummy.so" to php.ini and do:
php -c php.ini -m
I got this error message like this:
PHP Warning: PHP  Startup: Unable to load dynamic library '[path/to/dummy.so]' - dopen([path/to/dummy.so, 9): Symbol not found: _test_globals
Reference from: [path/to/dummy.so]
Expected in: flat namespace
In [path/to/dummy.so] in Unknown on line 0.
Any ideas on what happened? So far I tested the built process on fedora 21 with gcc and it works great, any ideas why on MacOS it is not working?
Thanks.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/9041150-zephir-built-extension-incorrectly-on-mac-os-10-10?utm_campaign=plugin&utm_content=tracker%2F280146&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F280146&utm_medium=issues&utm_source=github).

I guess I figured it out...
I was doing [path/to/zephir] install before to install zephir, when I am doing that, the built process is not working. But if I do: ./[path/to/zephir]/install-json and ./[path/to/zephir]/install, things start working...


Pass extra parameter to hooks in neutron-lbaas tempest jobs

This step is required for adding a functional job to neutron-lbaas.
Extra parameter would allow gate_hook and post_test_hook to
distinguish between tempest and functional jobs.

Can one of the admins verify this patch?

check

Many many thanks!


"Răzvani" is listed in the drop-down menu

Description:
"Răzvani" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Răzvani" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Răzvani, CĂLĂRAȘI (LEHLIU-GARĂ)".


feat(filter-sets): Saving filter sets in metadata

SUMMARY
This pr add feature to save filters sets in metadata and remove them from there. Also here fixed relevant bug that sometimes dashboard is updated when non-instant filter changed.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

https://user-images.githubusercontent.com/56388545/108347134-daac2c00-71e8-11eb-83f4-57544f6aefa5.mov
TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #13205 (29d0f24) into master (2ce7982) will increase coverage by 4.96%.
The diff coverage is 68.22%.

@@            Coverage Diff             @@
##           master   #13205      +/-   ##
==========================================
+ Coverage   53.06%   58.02%   +4.96%
==========================================
  Files         489      468      -21
  Lines       17314    15920    -1394
  Branches     4482     4088     -394
==========================================
+ Hits         9187     9237      +50
+ Misses       8127     6683    -1444

Flag
Coverage Δ

cypress
58.02% <68.22%> (+4.96%)
:arrow_up:

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

Impacted Files
Coverage Δ

...erset-frontend/src/SqlLab/components/ResultSet.tsx
4.16% <0.00%> (-0.09%)
:arrow_down:

...end/src/SqlLab/components/RunQueryActionButton.tsx
52.77% <ø> (ø)

superset-frontend/src/chart/ChartContainer.jsx
100.00% <ø> (ø)

superset-frontend/src/chart/ChartRenderer.jsx
75.67% <0.00%> (-1.04%)
:arrow_down:

...perset-frontend/src/common/components/Dropdown.tsx
50.00% <ø> (ø)

...t-frontend/src/common/components/Tooltip/index.tsx
100.00% <ø> (ø)

...perset-frontend/src/components/AlteredSliceTag.jsx
92.00% <ø> (ø)

superset-frontend/src/components/CachedLabel.jsx
42.10% <ø> (ø)

...ontend/src/components/CertifiedIconWithTooltip.tsx
0.00% <ø> (ø)

...perset-frontend/src/components/CopyToClipboard.jsx
57.14% <0.00%> (ø)

... and 148 more

Continue to review full report at Codecov.

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


feature/OP-17744_Resolved_Gate_issue

We prefer small, well tested pull requests.
Please refer to Contributing to Spinnaker.
When filling out a pull request, please consider the following:

Follow the commit message conventions found here.
Provide a descriptive summary for your changes.
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Add inline code comments to changes that might not be obvious.
Squash your commits as you keep adding changes.
Add a comment to @spinnaker/reviewers for review if your issue has been outstanding for more than 3 days.

Note that we are unlikely to accept pull requests that add features without prior discussion. The best way to propose a feature is to open an issue first and discuss your ideas there before implementing them.

This PR  triggered this build in this execution, please wait for it  and child pipelines to succeed before merging:  https://isd.prod.opsmx.net/deck/#/applications/gcp-gitprtrigger/executions/details/01GDWXCS2GT6T7RSHAZNQTBFKS

This PR  triggered this build in this execution, please merge to master only if this succeeds:  https://isd.prod.opsmx.net/deck/#/applications/gcp-gitprtrigger/executions/details/01GDWXDBGTEM112TBX123PFCJ5


Loading screen

Currently, if you are the first person in a given day to load the site, the site will fetch all the new data from campus dish. The user will be left with a blank screen for around 20/30 seconds. This is obviously not ideal behavior. The site should have a loading screen that is displayed after say 2 seconds stating that the content is loading, and to please wait or come back later.

Added loading screen in 62e58a409481f0752906da2bb4c502f6eec4c8a0. Also added the /testpages/loading page for testing the loading mechanism.


Create a Website for this Repo

Hi there.
Is there a website for this repo?
Because if you don't have, well, this repo can simply be turned into a website right away. Others will discover this project in that website.
Steps:

Go to Settings and look for GitHub Pages, scroll down. That's almost at the bottom.

You will see there: Branch:none, so you should change that to master because you have a README.md file in the master repo. This will be your page. Click Save first.

Then click Choose a theme, you select a predefined theme of your site.

Visit your site now! The URL will be https://kotcrab.github.io/vis-ui.

If you were amazed by that, simply read the documentation about GitHub Pages.

There is a website at https://github.com/kotcrab/vis-ui/blob/master/README.md :) Nothing else is really necessary

That's not a website. You simply look at the domain. It's different. Maybe you don't fully understand the difference between the two. Why do you think GitHub created GitHub Pages? It's different from simply having README.md in your GitHub Repo.

I'm not here to argue about the definition of a website. This project is not actively developed, it doesn't need anything else than this README page. Adding Github pages adds one more component to maintain.

I am just suggesting also, well maybe you can try GitHub Pages maybe sometime. It's not that hard to be maintained.


there is no option to set headers in session

AttributeError: 'builtins.Client' object has no attribute 'headers'

Fixed in v0.9.0


Accessing View Props From SetSelection

I see from #419 that TipTap now allows custom Nodes to override the default SetSelection method. From within that function, how do we access props such as view and node and other elements / attributes defined within get view()?
For example,
get view () {
   return {
       props: ["node", "view"],
       data: function () { return { text: "" }},
    }
}

setSelection(anchor, head) {
   this.node;
   this.view;
   this.text;
}

The scope of setSelection, as defined in the CreateDom method of the ComponentView, is set to this.vm, so to access node and view, use this.vm.node and this.vm.view.


Script for initial setup of new Vault cluster

We have added support for launching and upgrading Vault clusters running in an auto-scale group. When a new cluster is launched we need to initialize it and do initial setup so that it is usable with Pulumi for populating mounts and secret values. This can be managed via a script using the HVAC library.
The steps to execute are:

[ ] Initialize vault cluster with number of key shares and unseal threshold (preferably using GPG keys)
[ ] Set up audit device (file writing to stdout that will be captured by the SystemD journal)
[ ] Mount authentication backend(s) to be used by programmatic and human users. Initial setup should likely include username/password auth and create a user account for Pulumi
[ ] Revoke root token returned by initialization step

This snippet has some useful example code for the initialization step https://hvac.readthedocs.io/en/stable/usage/index.html#initialize-and-seal-unseal


Missing appropriate orientation and origin for prediction results

When running test.py the saved prediction results are missing appropriate orientation and origin since the output files are initiated with eye(4). This will cause issues when viewing the image data and the segmentation masks in most viewing software. Suggestion would be to remember the orientation and origin from the image volumes and to use this when initiating the output files, e.g. use img.get_affine() when reading the image data.

Thanks for your suggestion. I have fixed this issue by copying the orientation and origin information of the input image to the segmentation output.


No images with an error of 404

Hello All,
I am using Orthanc with OHIF v3. My search of a study always give me 404 error and does not display any images. I know the request goes to Orthanc successfully as I can confirm from the verbose. Any idea what could be the issue? I am using reverse-proxy to get to Orthanc server.
I am bit suspicious on the responseType as json. Is that okay?
browser error ...

request failed:
XMLHttpRequest { onreadystatechange: onreadystatechange(), readyState: 4, timeout: 0, withCredentials: false, upload: XMLHttpRequestUpload, responseURL: "https://dicomweb.mydomain.com/orthanc/dicom-web/studies?lim…tanceUID=1.2.392.200036.9<IP_ADDRESS>.12061.202302141090233", status: 404, statusText: "Not Found", responseType: "json", response: null }

Orthanc Web Config...

 "DicomWeb" : {
		"Enable" : true,
		"Root" : "/dicom-web/",
		"EnableWado" : true,
		"WadoRoot" : "/wado",
		"Host" : "",
                "PublicRoot" : "/orthanc/dicom-web/",
		"Ssl" : false,
                "StudiesMetadata" : "MainDicomTags",
                "SeriesMetadata" : "MainDicomTags"
    }

OHIF config.

  dataSources: [
    {
      friendlyName: 'Orthanc',
      namespace<EMAIL_ADDRESS>      sourceName: 'dicomweb',
      configuration: {
        name: 'orthanc',
	wadoUriRoot: 'https://dicomweb.mydomain.com/orthanc/wado',
	qidoRoot: 'https://dicomweb.mydomain.com/orthanc/dicom-web',
	wadoRoot: 'https://dicomweb.mydomain.com/orthanc/dicom-web',
        qidoSupportsIncludeField: false,
        supportsReject: false,
        imageRendering: 'wadors',
        thumbnailRendering: 'wadors',
        enableStudyLazyLoad: true,
        supportsFuzzyMatching: false,
        supportsWildcard: true,
        staticWado: true,
        singlepart: 'bulkdata,video,pdf',
      },
    },
  ],

Thank you so much!

wadoUriRoot: '/orthanc/wado',
qidoRoot: '/orthanc/dicom-web',
wadoRoot: '/orthanc/dicom-web',

Thank you Rana Asim Wajid for you kind support!
The changes you have recommended has made the viewer to work partially. The viewport loads the images and I could move between multiple series via keyboard shortcut, so this works. However, the thumbnail does not show/load any sequences. The 404 error still persist.
Any other suggestions?
Many thanks!

@richsmith123 What are left as bug here?


Cant get tls certificate??

failed to set up TLS certificates: www.egjesg3g3.online: obtaining certificate: [www.egjesg3g3.online] Obtain: [www.egjesg3g3.online] solving challenges: www.egjesg3g3.online: no solvers available for remaining challenges (configured=[tls-alpn-01 http-01] offered=[http-01 dns-01 tls-alpn-01] remaining=[dns-01]) (order=https://acme-v02.ap.i.letsncrypt.org/acme/order/18862228396/298159407636) (ca=https://acme-v02.ap.i.letsencrypt.org/directory)
Can someone help me try to fix this, I configured a ssl certificates on the sever my domain is pointed to, but it still comes up with this.

telegram itsdayztar


can use env SLIENT keep stdout clean , so can use with vim autoformat

Thanks for the PR! But have you tried using --quiet arg?
npx prettier --pn my-plugin --quiet --log-level silent -w <some-file>

If you use it together with builtin Prettier's --log-level silent, there is no output at all

vim autoformat use stdout not write file https://github.com/vim-autoformat/vim-autoformat

i see


TabMon service failed to start with multiple hosts

Hi there
I added two clusters as below in my TabMon.config file.

But after the above change, I am unable to start TabMon service. Excerpt of log file indicating the error is provided below.
2017-06-09 16:22:53,474 [8] INFO TabMon.TabMonAgent - TabMon stopped.
2017-06-09 16:22:53,474 [8] INFO Topshelf.Runtime.Windows.WindowsServiceHost - [Topshelf] Stopped
2017-06-09 16:22:56,521 [5] INFO TabMon.Config.TabMonConfigReader - Loading TabMon user configuration..
2017-06-09 16:22:56,537 [5] DEBUG TabMon.Config.TabMonConfigReader - Loading database configuration..
2017-06-09 16:22:56,552 [5] INFO TabMon.Config.TabMonConfigReader - Connecting to results database..
2017-06-09 16:22:56,693 [5] DEBUG TabMon.Helpers.HostnameHelper - Successfully resolved '<IP_ADDRESS>0' to 'SERVER1009'.
2017-06-09 16:22:56,693 [5] DEBUG TabMon.Helpers.HostnameHelper - Successfully resolved '<IP_ADDRESS>1' to 'SERVER991'.
2017-06-09 16:22:56,693 [5] INFO TabMon.Config.TabMonConfigReader - Successfully loaded TabMon config options! [Hosts='host server 1\SERVER1009,host server 2\SERVER991', Writer='Database Writer (Postgres Driver)', PollInterval='60', TableName='countersamples']
2017-06-09 16:22:56,693 [5] INFO TabMon.TabMonAgent - Initializing TabMon..
2017-06-09 16:22:56,693 [5] DEBUG TabMon.CounterConfig.CounterConfigLoader - Loading persistent performance counters from D:\Tableau\TabMon\Config\Counters.config..
2017-06-09 16:22:56,693 [5] DEBUG TabMon.CounterConfig.CounterConfigLoader - Loading performance counters from D:\Tableau\TabMon\Config\Counters.config..
2017-06-09 16:22:56,709 [5] DEBUG TabMon.CounterConfig.CounterConfigLoader - Successfully validated 'Config\Counters.config' against 'Resources\CountersConfig.xsd'.
2017-06-09 16:22:56,709 [5] DEBUG TabMon.CounterConfig.CounterConfigLoader - Loading Perfmon counters..
2017-06-09 16:22:57,037 [5] DEBUG TabMon.CounterConfig.CounterConfigLoader - Loaded 86 persistent Perfmon counters on SERVER1009.
2017-06-09 16:23:39,084 [5] ERROR TabMon.TabMonAgent - Failed to initialize counter sampler using counters from configuration file: The network path was not found.
Aborting..
2017-06-09 16:23:39,084 [5] FATAL Topshelf.Runtime.Windows.WindowsServiceHost - The service did not start successfully
Topshelf.TopshelfException: The service did not start successfully (returned false).
at Topshelf.Runtime.Windows.WindowsServiceHost.OnStart(String[] args)
2017-06-09 16:23:39,084 [1] ERROR Topshelf.HostFactory - The service terminated abnormally
System.NullReferenceException: Object reference not set to an instance of an object.
at TabMon.TabMonAgent.Dispose(Boolean disposing)
at TabMonService.TabMonServiceBootstrapper.Dispose(Boolean disposing)
at TabMonService.TabMonServiceBootstrapper.Dispose()
at Topshelf.Builders.ControlServiceBuilder1.ControlServiceHandle.Dispose()    at Topshelf.Runtime.Windows.WindowsServiceHost.Dispose(Boolean disposing)    at System.ComponentModel.Component.Dispose()    at System.ServiceProcess.ServiceBase.Run(ServiceBase[] services)    at System.ServiceProcess.ServiceBase.Run(ServiceBase service)    at Topshelf.Runtime.Windows.WindowsServiceHost.Run()    at Topshelf.HostFactory.Run(Action1 configureCallback)
Any help would be greatly appreciated.
Thanks

There are 2 things that I can think of that would cause this.

The remote registry service is not enabled on one of the hosts.
The two hosts in the clusters are not in the same subnet.

To verify these options please go on both hosts and check that the Remote Registry service is enabled and gather the IP addresses. Once done, please send the information.
Thanks,
Dan

Hi Dan
Unfortunately, due to company security policy I am unable to provide IP addresses but it's most likely the item #2.
I will try this on two servers that are in the same subnet.
Thanks
Sam

@SamanthaGun did you have any luck with servers on the same subnet?

Due to inactivity, I am going to close this issue out.

Apologies for the issue was caused due to item #1. This item can be closed now.

Sounds great. Thank you.


NativeLibraryConfiguration WithLogs

In version 0.11.2, WithLogs is missing
NativeLibraryConfiguration
.Instance
.WithCuda (false);
.WithLogs (true);
How should I print logs to determine if I am using a GPU

Please pass LogLevel.Debug to WithLogs.

Thank you, I have found a new way of writing
NativeLibraryConfig
   .Instance
   .WithCuda(false)
   .WithLogCallback((level, message) => {
       Console.WriteLine($"[llama {level}]: {message.TrimEnd('\n')}");
   });


treewide: https://cache.nixos.org/ -> https://cache.nixos.org

Motivation for this change
The latter seems to be marginally nicer.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @domenkozar

Yup.
On Tue, Nov 26, 2019 at 5:04 PM Yegor Timoshenko<EMAIL_ADDRESS>wrote:

@yegortimoshenko commented on this pull request.
In nixos/modules/services/misc/nix-daemon.nix
https://github.com/NixOS/nixpkgs/pull/74271#discussion_r350834397:

@@ -380,7 +380,7 @@ in
config = {

 nix.binaryCachePublicKeys = [ "cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=" ];

nix.binaryCaches = [ "https://cache.nixos.org/" ];

By relying, are you referring to
https://github.com/cachix/cachix/blob/275f732676c06aa23ae6c93049c5e54cb12ce30c/cachix/src/Cachix/Client/InstallationMode.hs#L158,
i.e. there will be two copies of the same binary cache listed in
/etc/nix/nix.conf?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/74271?email_source=notifications&email_token=AAA63AYITDVP52CUBWQES5TQVVCKHA5CNFSM4JRZYKR2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCNBBSSA#discussion_r350834397,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA63A27RJHMBF3XM6UO2ELQVVCKHANCNFSM4JRZYKRQ
.

Closing, see https://github.com/NixOS/nixpkgs/pull/74271#discussion_r350817015.

@domenkozar Thanks for the prompt feedback :)


chinese error msg dialogue box

Describe the issue
Since recently I get an error message when starting any VM in UTM, see screenshot.
I can't read chinese and I also can't copy and paste the error msg into google translate or so.
Configuration

UTM Version: 2.2.4
OS Version: monterey 12b9
Apple Silicon

IIRC I installed some earlier version from website / github .dmg download some time ago and later updated to a newer release using the same method.

Downloaded UTM 2.2.4 again (.dmg from homepage), copied it to applications, replacing the previous UTM (same version).
Now VMs start again. Strange.


No release for docs

When there is a push on the docs branch there is no need to release a new version of opentab.
Add this branch to the excluded branches in the release workflow.

The improvement made did not change the way the pipelines are triggered when docs is merged.


Issue 343

@homu  r+

:pushpin: Commit e81b733 has been approved by Vinatorul

:zap: Test exempted - status


Team 2 Review (MIS): Environment Variables lacking details

@samm82
Name: Team 2 Review
About: The Environment Variables sections in the MIS is not descriptive enough. They are lacking details such as type.
Summary
The environment variables section lacks descriptiveness. Environment Variables should have their type specified like any other variable. For example, in the TA follow-up, they mentioned that a Device Screen could have the type of a "2D Array of Pixels". Similarly, a device keyboard could be a 2D array of characters.

(example in figure taken from Text Communication Module)
Deadline

March 1, 2024

Estimated Work Hours
~3 hrs

Updated environment variables to be formatted the same as other variables

Updated in #310


create robot control react element to show the current state of the robot

sensor data, motor output etc..., position...

I implemented this already. Will add a PR today

#46 has an extremely simple version of this, only contains the motor power for now, but ill close this issue.


Coverage 3.14.0

Summary
Implements an annotation-based coverage approach that only uses local ids that are also in the annotation (and therefore can be seen in the cql coverage highlighting). This mitigates updates in version 3.14.0 of the translator which added many additional local ids that could otherwise impact coverage.
New behavior
Coverage calculations will be approximately the same as for translator 3.3.2. There may be some very small differences in in coverage percentages since this is a new coverage approach, but any coverage gaps should be identifiable for the end user.
Code changes

Small README updates
ClauseResultsHelpers.ts Update to add findAnnotationLocalIds function that identifies all "r"-keyed local ids within an annotation structure. Uses this to limit the final localIds to the ones that can be found in the annotation.
ClauseResultsHelpers.test.ts Updates to "Not Equivalent" and "Not Equal" and addition of a "Not Null" unit test to align with new localid approach
Adds 3.15.0 folder for new translations of existing easily translatable cql unit test fixtures (some existing cql fixtures do not have the right dependency library versions to be easily re-translated)

Testing guidance

Test with the 3.14.0 measures provided in https://github.com/projecttacoma/fqm-execution/issues/313 and confirm coverage percentages are restored. Maybe test with other measures with known past coverage issues.

I am not sure how much we care about this, but we should at the very least note it in the release notes. For CMS646, I noticed that the highlighting for v3.3.2 and v3.14.0 are slightly different. While the percentage is the same, looks like some aliases no longer get highlighted. Could be worth investigating if there is a fix and if not, at least noting it.
For example:

So it looks like you're seeing a change that was caused by the translator update and not by this PR as 3.3.2-translated measure results before and after this PR change have the same highlighting (alias highlighted), and 3.14.0-translated measure results before and after this PR change have the same highlighting (alias  un-highlighted).
Since this PR's task aims to resolve coverage and address the linked ticket, addressing highlighting may be out of scope (though the ticket does reference https://github.com/projecttacoma/fqm-execution/issues/285 as well, which is a highlighting issue).
I think we may also want to address highlighting separately because the fix is more likely to break backwards compatibility. Essentially, we have workarounds for highlighting aliases that use a plus 1 increment to identify the correct localid association (since aliases are identified within the annotation but not within the ELM). Initial testing shows that the relationships between the correct localids are at least sometimes a +2. Especially if we are aiming for backwards compatibility, this may take a new approach.


SkinComposer slows down the system on Mac OS

Hi,
When I launch the new version 35 of SkinComposer on my Mac OS Catalina 10.15.5, the whole system slows down. Mouse interaction are very slow...
However, CPU and memory do not seem to be at their limit.
Have you already noticed such slow-down on Mac OS ?
I don't if it may help: I am using Oracle JDK 14 :
java version "14.0.1" 2020-04-14
Java(TM) SE Runtime Environment (build 14.0.1+7)
Java HotSpot(TM) 64-Bit Server VM (build 14.0.1+7, mixed mode, sharing)

I haven't noticed anything, but that's because I'm used to my Mac being completely slow almost to the point of being unusable. It will continue to be my pain point for some time. JDK 14 is supposed to be fine. Can you try running it with the following paramaters?
java -jar -XstartOnFirstThread SkinComposer.jar

I included a hack to avoid having to use that, but I wonder if it's affecting performance. Specifying the flag should bypass the hack.

I have tested with -XstartOnFirstThread. But it does change the performances.
My Macbook is quite old (MacBook Pro Retina, 13 pouces, mi-2014 / 2,6 GHz Intel Core i5 double cœur).
But it still performs quite well with any other applications. For example, I can easily run Android Studio and emulate a device in the meantime.
What is strange is that both CPU and memory seem to be ok... I have never seen such a behavior before. I will continue to observe and analyse it. I let you in touch.
Thanks

I tried on Catalina 10.15.6 with OpenJDK 11, no problems. Machine is from 2018 though.

I can't reproduce this either. Sorry, I can't see anything I'm doing would cause this problem besides the general incompatibilities of trying to run Java/LWJGL3/libGDX on the notriously challenging Mac platform. I'm going to close this for now. Please do report if you have anything new to add. Thank you.

May be related to libgdx/libgdx#5893


Lords of the Fallen missing objects after Mesa + DXVK upgrade

Sorry to bother, but I can't understand if this is a Mesa problem or DXVK.
I'm using Manjaro Linux, and this game was running well with Mesa 18.3.5, LLVM 7.0.1, DXVK 1.0 and Wine-staging-nine 4.1.
I did a system upgrade that brought Mesa 19.0.1, LLVM 7.0.1, DXVK 1.0.1, but wine-staging-nine was not upgraded.
Since this upgrade I can't see object, enemies or buildings in game. The problem is visually exactly the same as described in #285 .
Software information
Lords Of The Fallen (GOG Version)
System information

GPU: AMD RX 580 4 GB
Driver: Mesa 19.0.1/LLVM 7.0.1
Wine version: wine-staging-nine 4.1
DXVK version: 1.0.1

Apitrace file(s)

Put a link here

Log files

d3d11.log:
LordsOfTheFallen_d3d11.log

dxgi.log:

This is a mesa regression and fixed in git master. AFAIK it only affects the 19.0.1 stable release, 19.0 should be fine.

Already fixed in mesa. There is no DXVK workaround, so you're going to have to compile for yourself for now or revert to Mesa 19.0.0.

Thank you all a lot, I'll notify my distro maintainers of the problem! ;)

Thanks for the info! Seems the same was affecting Esacpe from Tarkov windows reflection on Factory, right now I'm running with RADV_DEBUG=nohiz until mesa-git get updated in unofficial arch repo.


link back to Github from Storybook

hey @malcomio thanks for your PR. The base branch where we want to merge is release/1.0.0. I did try to change the base but unfortunately it has conflicts. Do you mind rebasing it, please?


What is the message format for a feature being removed?

Regarding https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#commit
I understand that the commit message footer must have the BREAKING CHANGE message. But what type ought to be used to indicate that a feature is being removed?, feat?

Generally, we don't remove features, unless

It is necessary to fix something else (e.g. a regression or a long-standing bug).
Type: fix, perf
It replaced/superceded by another feature.
Type: feat
It has been deprecated for a while and we decided to remove it.
Type: Not sure - we hardly ever do this

I'm going to close the issue, as it not really actionable.


Add a Gitter chat badge to Readme.md

felixge/node-form-data now has a Chat Room on Gitter
@alexindigo has just created a chat room. You can visit it here: https://gitter.im/felixge/node-form-data.
This pull-request adds this badge to your Readme.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Will be implemented manually. Thank you.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_a0a375fb-1718-437c-bd62-25f1f504c83e/pull/1 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target_a0a375fb-1718-437c-bd62-25f1f504c83e.git

Docs Build status updates of commit 5df49ee:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-RuningBuildFailed] Some unexpected errors happened when running build, please open a ticket in https://aka.ms/SiteHelp and include the error report for our team to troubleshoot

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


An unexpected error occurred when executing the action on the 'basic12sp4' machine. Please report this as a bug:

An error occurred while executing multiple actions in parallel.
Any errors that occurred are shown below.
An unexpected error occurred when executing the action on the
'basic12sp4' machine. Please report this as a bug:
undefined method `coerce' for an instance of String
/usr/share/vagrant/gems/gems/vagrant-libvirt-0.11.2/lib/vagrant-libvirt/util/byte_number.rb:35:in coerce' /usr/share/vagrant/gems/gems/vagrant-libvirt-0.11.2/lib/vagrant-libvirt/util/byte_number.rb:39:in <=>'
/usr/share/vagrant/gems/gems/vagrant-libvirt-0.11.2/lib/vagrant-libvirt/util/byte_number.rb:39:in <=>' /usr/share/vagrant/gems/gems/vagrant-libvirt-0.11.2/lib/vagrant-libvirt/action/handle_box_image.rb:95:in <'
/usr/share/vagrant/gems/gems/vagrant-libvirt-0.11.2/lib/vagrant-libvirt/action/handle_box_image.rb:95:in call' /usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/warden.rb:48:in call'
/usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/builtin/handle_box.rb:56:in call' /usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/warden.rb:48:in call'
/usr/share/vagrant/gems/gems/vagrant-libvirt-0.11.2/lib/vagrant-libvirt/action/handle_storage_pool.rb:63:in call' /usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/warden.rb:48:in call'
/usr/share/vagrant/gems/gems/vagrant-libvirt-0.11.2/lib/vagrant-libvirt/action/set_name_of_domain.rb:34:in call' /usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/warden.rb:48:in call'
/home/rmontes/.vagrant.d/gems/3.3.0/gems/vagrant-hostmanager-1.8.10/lib/vagrant-hostmanager/action/update_all.rb:24:in call' /usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/warden.rb:48:in call'
/usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/builtin/config_validate.rb:25:in call' /usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/warden.rb:48:in call'
/usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/builtin/provision.rb:80:in call' /usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/warden.rb:48:in call'
/usr/share/vagrant/gems/gems/vagrant-libvirt-0.11.2/lib/vagrant-libvirt/action/cleanup_on_failure.rb:21:in call' /usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/warden.rb:48:in call'
/usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/warden.rb:127:in block in finalize_action' /usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/warden.rb:48:in call'
/usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/builder.rb:180:in call' /usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/runner.rb:101:in block in run'
/usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/util/busy.rb:19:in busy' /usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/runner.rb:101:in run'
/usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/builtin/call.rb:53:in call' /usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/warden.rb:48:in call'
/usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/builtin/box_check_outdated.rb:31:in call' /usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/warden.rb:48:in call'
/usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/builtin/config_validate.rb:25:in call' /usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/warden.rb:48:in call'
/usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/builder.rb:180:in call' /usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/runner.rb:101:in block in run'
/usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/util/busy.rb:19:in busy' /usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/action/runner.rb:101:in run'
/usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/machine.rb:248:in action_raw' /usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/machine.rb:217:in block in action'
/usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/environment.rb:631:in lock' /usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/machine.rb:203:in call'
/usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/machine.rb:203:in action' /usr/share/vagrant/gems/gems/vagrant-2.3.7/lib/vagrant/batch_action.rb:86:in block (2 levels) in run'

I believe you've run into a known issue with the SLE 12 SP4 vagrant box image provided by SUSE. That image specifically had this issue and as far as I know it was never fixed. However, you can fix it manually by doing the following:

Determine the box version for SLE 12 SP4 being used on your system with this command: vagrant box list
Open this file: ~/.vagrant.d/boxes/SLES-12-SP4-Vagrant.x86_64/<version>/libvirt/metadata.json
Change from:

"virtual_size": "42"
to:
"virtual_size": 42
Then test to see if the issue has resolved.


Controls that rely on serializing data to XML cause Resx to XLIFF conversion to fail

There seems to be an issue present as of the Multilingual App Toolkit v4.2.1: when a TableLayoutPanel is in a Localizable form, the LayoutSettings property is stored in the Resource file with an XML header inside it:
<?xml version="1.0" encoding="utf-16"?>

This causes the MAT to fail on build, with the error:
Convert file [file].resx to XLIFF format failed. Message:Unsupported processing instruction '?xml' found during encoding. Please remove unsupported processing instructions from the xml.

The offending line(s) can be removed from the Resource file, but future builds/project loads will cause the line to be automatically reinserted and trigger the error again.
See also:

[Wayback Machine, old UserVoice community] fix convert problem '?xml' with embedded xml structure in resx file
[StackOverflow] Convert resx to XLIFF format failed
[Microsoft Q&A] TableLayoutPanel RESX property generation causes Multilingual App Toolkit to fail

Acknowledging bug

I've encountered another example with the DevExpress XtraReport. It stores its data source's XSD schema into a property called DataSourceSchema and the build fails with the same error.
What seems strange is that this property doesn't have a Localizable attribute. Shouldn't it be ignored? Nor does the report have Localizable set to true. The property looks like this when decompiled:
[Description("Gets or sets an XML/XSD file, containing a schema of a report’s data source.")]
[DXDisplayName(typeof(ResFinder), "DevExpress.XtraReports.UI.XtraReport.DataSourceSchema")]
[DefaultValue("")]
[SRCategory(ReportStringId.CatDesign)]
[Editor("DevExpress.Utils.UI.XmlSchemaEditor,DevExpress.Utils.v19.2.UI, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b88d1754d700e49a", typeof(UITypeEditor))]
[XtraSerializableProperty]
public string DataSourceSchema
{
	get { ... } set { ... }
}

In this case, it would resolve the problem if the property was completely ignored, either by detecting that the property isn't marked as localizable (which I guess wouldn't help the OP) or by having a custom setting where we could state which properties should be skipped, or even by ignoring properties that contain XML (since these break the build anyway).
Another thing I tried is to change the offending ?xml bit into ?xml in the RESX file, but I got the same error.
I was unable to find a reason for this behaviour, but I'm guessing it's because the Multilingual Editor doesn't support translating XML and we'd have to use the original designer inside the VS to localize that particular property. MAT would have to ignore the property when building the XLF file but also when putting its translated value from XLF back into RESX. Would this be doable? Just issue a warning like "properties containing XML are not supported, you have to find a different way to localize them." Maybe add an option in the settings to turn the warnings into errors.


Make window save its aspect ratio

Background image covers the whole widget even after resizing the window. So, this feature is not needed anymore.


Image save and upload

Allow users to select an image from the filesystem and have it work with slide background and the image element. Figure out image upload when saving presentation to Plotly server.

This should serialize the image into the JSON representation of the slides. Ideally we add compression during serialization.
Also includes image size cap.
Must handle JPG, PNG, GIF

@rgerstenberger Can this be closed? Image size cap is duplicated by https://github.com/FormidableLabs/spectacle-editor/issues/107.

Yes, can close.


Feature/hide users with 0 hours

@mauro-oto @etagwerker this PR fixes #30, check it out! Thanks

@schmierkov Other than that one comment it looks good!

@etagwerker thanks for the feedback, I always get confused which version we prefer. just updated the code, thanks! 👍

@schmierkov Great, thanks! 🍻

@schmierkov Could you solve #32 too? It'll be a very similar solution.

@etagwerker I didn't see that this exists for projects too! I will check asap, thanks for pointing it out!


fix(WebSocketSubject): add different input and output types

Description:
add different input and output types to WebSocketSubject

Related issue (if exists):
#5381

My guess is that this is a breaking change for somebody. While it wouldn't breaking explicitly typed usages of webSocket, I do think it may break implicitly typed versions of it. I don't have the time at the moment to figure out what those would be. It would be helpful if we had tslint tests around this before we merged this change. So we can assert that this change doesn't break any of those.

@benlesh I forgot to run api_guardian:update. The files are updated now.
I can create a separate pr for dtslint tests for webSocket, I do not see any for webSocket currently in the project (looked for them here https://github.com/ReactiveX/rxjs/tree/7.1.0/spec-dtslint/observables/dom).

Is the failing check a result of something I changed? Seems odd it says
npm ERR! Could not resolve dependency:
npm ERR! peer typescript@"^3" from<EMAIL_ADDRESS>

I rebased from master to hopefully fix the failing check due to npm package dependency.


Add link to designsystemet

Description
Added link and description for designsystemet.no in introduction in altinn storybook.
Related Issue(s)

Fixes: https://github.com/digdir/designsystem/issues/10

Verification

[ ] Your code builds clean without any errors or warnings
[ ] Manual testing done (required)
[ ] Relevant automated test added (if you find this hard, leave it and we'll help out)
[ ] All tests run green

Documentation

[ ] User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (4cac5ad) 95.51% compared to head (0cf33a3) 95.51%.
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #336   +/-   ##
=======================================
  Coverage   95.51%   95.51%
=======================================
  Files          29       29
  Lines         357      357
  Branches       95       95
=======================================
  Hits          341      341
  Misses         14       14
  Partials        2        2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


is it possible to show this themes in Rmarkdown html code chunk ?

Hi , It look cool. I want to share it in the Rmarkdown html code chunk with this themes .
Is it  possible ?

You could do that, but it would be a fair amount of work. The theme itself is css, and Rmarkdown supports css for code chunks. So you would take the theme file, and change the names of the different classes to match the code chunks, then create a new style sheet to load into the Rmarkdown (see here).
If you do get this working I'd love to see it!


How can I delete the users database to a reset/refreshed state?

How can I reset the database record if I want to pull new changes and restart from scratch? The UI always tells me that the user exists (but I could not remember my admin password). How can I delete the old entries from which file?
Thanks.

To reset the user database you can delete the /app/backend/data/ollama.db file of the container, wherever you have the data directory mounted on the host.

There are already users in this table. I guess this too needs documentation from the docs team out there. How do we change the password of a user? I don't see any password field in the table structure:
SELECT name FROM sqlite_master WHERE type='table';

chat
user
auth
modelfile
prompt
document
sqlite>

PRAGMA table_info(user);

0|id|VARCHAR(255)|1||0
1|name|VARCHAR(255)|1||0
2|email|VARCHAR(255)|1||0
3|role|VARCHAR(255)|1||0
4|profile_image_url|VARCHAR(255)|1||0
5|timestamp|DATE|1||0

My scenario is that on proxmox, I was migrating from one machine to another. I never put into account that /var/lib/docker/volumes/ollama-webui/_data/ollama.db is not part of the cluster or the virtual environment in pve, so that I only did rsync.
I believe we should have a migration docs because man, this repo is improving at lightning speed!
And it should include migrating the volume too.
Another question I have is the structure of the table. The user table doesn't seem to have a password. Accessing the sqlite db should also be documented. This not only makes the system much more user-friendly to devs, but also leave plenty of room for customization and improvement for wishlist and experimentation for a probable PR.

Perhaps a command that could be run on the container shell to reset the first admin account password as well? Deleting the whole database isn't exactly an ideal solution to a lost password if you've got other users, and chat histories you'd like to keep.

FYI, the db does not need any migration, and will continue to work after you pull the latest image.

If you'd like to start from scratch, you could always just delete the ollama.db file and you'll be able to make a new account. As for the password update, you can do so from the admin panel page but I presume you don't have access to it at the moment.


Fix RemoveRequestParameterGatewayFilterFactory to deal with query par…

The RemoveRequestParameterGatewayFilterFactory class builds up a new URI once it removes the request parameter outlined in the filter config, however currently expects the remaining
request parameters to all be encoded and the are not.
When the request query params are determined, they are decoded in AbstractServerHttpRequest and are not subsequently encoded again before
the filter builds up the URI.
Currently when the remaining request parameters have a character which requires encoding, you get an "Invalid character ' ' for QUERY_PARAM"
error.

Can you make this against the 2.2.x branch?

Can you make this against the 2.2.x branch?

done


Use ES module exports

Summary
This converts most of the commonjs usage into ESM exports.
I took care to not change to default export for entrypoints, meaning that those importing individual modules can still use require without issue.
Might consider adding no-commonjs lint rule (https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/no-commonjs.md), but there are still lots of places we have to use CJS?

Test plan
./jest

Codecov Report

Merging #4454 into master will increase coverage by 0.11%.
The diff coverage is 64.65%.

@@            Coverage Diff             @@
##           master    #4454      +/-   ##
==========================================
+ Coverage   56.74%   56.86%   +0.11%
==========================================
  Files         189      186       -3
  Lines        6485     6294     -191
  Branches        3        3
==========================================
- Hits         3680     3579     -101
+ Misses       2804     2714      -90
  Partials        1        1

Impacted Files
Coverage Δ

...kages/jest-circus/src/format_node_assert_errors.js
0% <ø> (ø)
:arrow_up:

...ackages/jest-haste-map/src/lib/extract_requires.js
100% <ø> (ø)
:arrow_up:

...ckages/jest-cli/src/reporters/coverage_reporter.js
54.45% <ø> (-0.45%)
:arrow_down:

packages/jest-util/src/null_console.js
0% <ø> (ø)
:arrow_up:

packages/jest-cli/src/pattern_prompt.js
100% <ø> (ø)
:arrow_up:

packages/jest-config/src/set_from_argv.js
95% <ø> (-0.24%)
:arrow_down:

packages/jest-cli/src/test_scheduler.js
16.94% <ø> (-0.7%)
:arrow_down:

packages/jest-runtime/src/should_instrument.js
58.82% <ø> (-4.34%)
:arrow_down:

packages/jest-cli/src/search_source.js
69.84% <ø> (-0.48%)
:arrow_down:

...es/jest-cli/src/lib/handle_deprecation_warnings.js
0% <ø> (ø)
:arrow_up:

... and 169 more

Continue to review full report at Codecov.

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

Woah, nice work @SimenB. Thank you for your continued efforts to modernize the codebase.


Added custom attribution of layers to viz.json

Fixes #5216.
@Kartones or @juanignaciosl, would any of you take a look at this please? It took me a while to write the test, but finally got some integration tests :weary:. Thanks for your help!

:+1:  although that test is only testing the Carto::Visualization model, not the old one...
Could you try building one forr https://github.com/CartoDB/cartodb/blob/master/spec/models/visualization/relator_spec.rb ? (even if with some heavy stubbing should be possible) I fear  old backend bugs

retest this please

@Kartones Please take a look at this commit that adds a very basic test for CartoDB::Visualisation::Relator#related_visualizations. I'm not very familiar with the underlying models so any feedback about how to improve that test will be welcome. Thanks!

:+1: is just to be sure it doesn't breaks until we're able to get rid of that backend


CSS

CSS
User Story
As a bootcamp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Study Guide
WHEN I view the study guide
THEN I see a dark blue header and footer, and four boxes with the shadow

Was able to successfully add, commit, push and merge mutliple requests without 100% using instructions.


Components not being registered anymore

I keep getting this error (and many others for all directives and components):
[Vue warn]: Unknown custom element:  - did you register the component correctly? For recursive components, make sure to provide the "name" option.
Although I am following the documentation on how to use collapse:
import VueCollapse from 'vue2-collapse';
Vue.use(VueCollapse);
Running Node 10.4.0 and NPM 6.1.0
Please help, I don't know what to do anymore. Thank you.

Does your application use babel?


testing more cases for Game init

Current coverage is 75.00% (diff: 100%)

Merging #46 into master will increase coverage by 7.04%

@@             master        #46   diff @@
==========================================
  Files             8          8
  Lines           284        284
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            193        213    +20
+ Misses           91         71    -20
  Partials          0          0

Powered by Codecov. Last update d78dc8c...ce0fae2


fix(docs): update command for destroying redis instance

from dokku postgres:destroy redis to dokku redis:destroy ledokku

Thanks @holamendi!


no associated IAM Roles

Hello,
I am trying to configure a coreos cluster, but get this error
module.vpc.aws_route_table_association.private.1: Creation complete
module.vpc.aws_route_table_association.private.0: Creation complete
module.vpc.aws_route_table_association.private.2: Creation complete
Error applying plan:

2 error(s) occurred:

* aws_instance.bastion: Error launching source instance: InvalidParameterValue: IAM Instance Profile "arn:aws:iam::378239092500:instance-profile/k8s-bastion-test" has no associated IAM Roles
	status code: 400, request id: 57dfa361-50e6-4ab6-a34c-5c0ff8067a53
* aws_instance.etcd.2: Error launching source instance: InvalidParameterValue: IAM Instance Profile "arn:aws:iam::378239092500:instance-profile/k8s-master-test" has no associated IAM Roles
	status code: 400, request id: e1b195ce-8d59-45e6-918e-388d521e4c36

Terraform does not automatically rollback in the face of errors.
Instead, your Terraform state file has been partially updated with
any resources that successfully completed. Please address the error
above and apply again to incrementally change your infrastructure.

any advice, much appreciated.

Just make all again, sometimes terraform fails to create the IAM Profile at the same time everything else is being created in parallel. That's somewhat of a known issue.


Add seperator lines between header, description and links in article view

It would be nice to have an option for showing lines, consisting of hyphens, which separate header lines, description and links in article view.

Hi! Thanks for the idea.
Why hyphens? It seems to me that they won't be too obvious right under the text. Wouldn't it be better to add completely empty line, to increase the spacing? I.e. instead of
Feed: Texts
Title: Lorem ipsum
Date: Tue, 22 Oct 2019 03:00:00 +0300
Link: https://example.com/lorem/ipsum.html

Consectetur dolores sint quia nihil velit in suscipit quo. Totam dolorum
possimus hic. Eveniet atque reiciendis dicta eius ut perferendis.

Repudiandae dolore sit unde nobis quas velit sunt cumque[1]. Voluptas facere
dolore natus consequatur doloribus sed eaque. Consequuntur eos animi atque.

Dolores consequatur quisquam asperiores animi consequuntur odit qui. Odio
reprehenderit officiis distinctio. Mollitia cumque in et. Vel deserunt quaerat
facere possimus laboriosam magnam. Cupiditate dolorum[2] et est.

Dolor qui qui sit enim[3] modi. Ut tempore enim ratione excepturi. Modi rem autem
voluptatem.

Et id voluptatem culpa minus et exercitationem.  Et
temporibus dignissimos est. Ut animi delectus modi quisquam dolore.

Links:
[1]: https://example.com/cumque/ (link)
[2]: https://example.com/dolorum/ (link)
[3]: https://example.com/enim/ (link)

Newsboat should display this:
Feed: Texts
Title: Lorem ipsum
Date: Tue, 22 Oct 2019 03:00:00 +0300
Link: https://example.com/lorem/ipsum.html

Consectetur dolores sint quia nihil velit in suscipit quo. Totam dolorum
possimus hic. Eveniet atque reiciendis dicta eius ut perferendis.

Repudiandae dolore sit unde nobis quas velit sunt cumque[1]. Voluptas facere
dolore natus consequatur doloribus sed eaque. Consequuntur eos animi atque.

Dolores consequatur quisquam asperiores animi consequuntur odit qui. Odio
reprehenderit officiis distinctio. Mollitia cumque in et. Vel deserunt quaerat
facere possimus laboriosam magnam. Cupiditate dolorum[2] et est.

Dolor qui qui sit enim[3] modi. Ut tempore enim ratione excepturi. Modi rem autem
voluptatem.

Et id voluptatem culpa minus et exercitationem.  Et
temporibus dignissimos est. Ut animi delectus modi quisquam dolore.

Links:
[1]: https://example.com/cumque/ (link)
[2]: https://example.com/dolorum/ (link)
[3]: https://example.com/enim/ (link)

Just one line. Subtle enough not to be intrusive (and not to annoy people that use smaller screens), yet noticeable enough to make a difference.
The only advantage of hyphens that I can think of is that they can be highlighted. But one could highlight the header just as easily.
Would that cover your use-case, @mglh?

Just one more empty line would make no (positive) impact to the readability, to my view. I still would prefer a separating line.
But you are right, adding more lines has to be considered in terms of wasting precious screen space -- "smaller screens" also affects me, since I use newsboat also on my cellphone, under termux.
To take that into account, what about making the char that is used choosable by the user? Including the possible use of newlines before and after the line, in all replacing the one empty line that delimits header and description by default. For example, a hypothetical option like this
delimiting-line "_\n" # underscore following newline

would lead to this:
Feed: Texts
Title: Lorem ipsum
Date: Tue, 22 Oct 2019 03:00:00 +0300
Link: https://example.com/lorem/ipsum.html
_________________________________________________________________________________

Consectetur dolores sint quia nihil velit in suscipit quo. Totam dolorum
possimus hic. Eveniet atque reiciendis dicta eius ut perferendis.

Repudiandae dolore sit unde nobis quas velit sunt cumque[1]. Voluptas facere
dolore natus consequatur doloribus sed eaque. Consequuntur eos animi atque.

Dolores consequatur quisquam asperiores animi consequuntur odit qui. Odio
reprehenderit officiis distinctio. Mollitia cumque in et. Vel deserunt quaerat
facere possimus laboriosam magnam. Cupiditate dolorum[2] et est.

Dolor qui qui sit enim[3] modi. Ut tempore enim ratione excepturi. Modi rem autem
voluptatem.

Et id voluptatem culpa minus et exercitationem.  Et
temporibus dignissimos est. Ut animi delectus modi quisquam dolore.
_________________________________________________________________________________

Links:
[1]: https://example.com/cumque/ (link)
[2]: https://example.com/dolorum/ (link)
[3]: https://example.com/enim/ (link)

I'd make the setting dumber: let the user write exactly what they want to see. We can mark it as non-wrappable in the pager, so if it's wider than the terminal, it'll still be a single line, just truncated. Your example will then look like this:
delimiting-line "_________________________________________________________________________________\n"

The default would be an empty string, of course, to keep current behaviour.
Would that be an acceptable solution for you, @mglh?

Hi, I would like to work on the enhancement, if no one else is.
As per my understanding of it, we need to add an optional and customizable separator at above 2 places. Should be non-wrappable so it does not overflow to next line.

Hi! Sorry, I'm busy with the ICFPC contest and don't have time to re-read this issue and figure out what should be done. It looks like we never got a word back from @mglh on this? I'll get back to this after Monday.

Upon re-reading, I don't feel like this feature is baked enough to be implemented yet. I'm not even sure it's needed; the current pager seems good enough, and if a user wants something else, they can override it in their config.
Can I help you pick some other issue to work on, @kartikeya100? What are you interested in? Implementing new features? Tracking down bugs? Refactoring, improving existing things? Figuring out designs for new features? We got a little bit of everything here, you can probably find something if you look through the list!

Honestly, I am picking those items that I guess are easy for me and does not require much assistance as this all is quite new to me and I don't want to be troublesome. But I am looking forward to fix some bugs, refactor code and then adding new features. I'll pick a new item now.


onSelected is not returning currentSelectingRows

selectedRows =  Set.from(stateManager!.currentSelectingRows.map((e) => e.sortIdx)).toList();
In version 4.0.0 this was working on the onSelected but in ver 5.0.4 it is returning an empty list.

Found issue in my code


More efficient audio extraction from Europa

Original Source: https://www.wikidata.org/wiki/Q1198588

For P T

Sorry that was ment to my own fork.


YouTube Video on Loop

Hey guys,
I'm fairly new to this, I have the player working as I need it to however I'd like to have it running on a loop. Is there an easy way to set this in the options?
This is what I have right now before the closing  tag
<script>plyr.setup({ 		autoplay: ["true"], 		hidecontrols: ["true"] 	}); 	</script>
This is a YOUTUBE video

You don't need to set those values as strings in an array. Also, hidecontrols is default to true
Try this
plyr.setup({
    autoplay: true,
    loop: true
});

Thanks Sam, but unfortunately that didn't get it to loop :-(

I'm also experiencing that the loop option doesn't work for a YouTube video. AFAIK the YouTube API doesn't provide a loop parameter, but you can achieve looping by making a playlist with only one video:
player.loadPlaylist(videoId);

The downside is that it reloads the video each time. Another possibility that I haven't explored, is to seek to position 0 (zero) and replay.
@SamPotts how is YouTube looping implemented on Plyr? Can you confirm it works?
(PS: this can be tested with this very short video: https://www.youtube.com/watch?v=tPEE9ZwTmy0)

I'm trying to workaround this by doing:
plyr.on('ended', () => plyr.restart());

The event handler is executed but the video doesn't restart. This is the debug log:
[Plyr] event: ended Object
[Plyr] event: timeupdate
[Plyr] event: seeking
[Plyr] Seeking to 0.0000 seconds
[Plyr] event: statechange

I'm also trying to restart the playback programmatically by calling plyr.restart() as well as plyr.rewind() && plyr.play() on the browser console but nothing seems to make it play again.
[Plyr] event: ended
[Plyr] event: statechange
$0.plyr.restart()
[Plyr] event: timeupdate
[Plyr] event: seeking
[Plyr] Seeking to 0.0000 seconds

(Notice that $0.plyr refers to a Plyr instance in my browser console.)

I found that in order for YouTube to replay, it needs a call to youtubePlayer.stopVideo() first. After that, calling plyr.play() works.

I've opened a PR that you can test which implements loop functionality on YouTube. Beware that since it was made against the develop branch, it uses the new loop config structure (config.loop.active).

I've updated my files with the develop branch and used your code, yet I'm seeing this:
(index):62 Uncaught TypeError: plyr.on is not a function

Develop is not production ready otherwise it'd be in the master branch.

@SamPotts I would suggest as an advice keeping the develop branch
production ready and all in-progress work to happen on a feature branch. In
that way, the latest features can be tested before they are released (as I
think @CudaCores intended).
Cheers!
El El mié, 10 de may. de 2017 a las 22:34, Sam Potts <
<EMAIL_ADDRESS>escribió:

Develop is not production ready otherwise it'd be in the master branch.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Selz/plyr/issues/560#issuecomment-300657867, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAdVgH06VCOqOEmOoI2TBXR2i9P14NI-ks5r4mW5gaJpZM4M-hjL
.

I use develop as a feature branch for developing. Once it's production ready then I'll create a staging or beta branch that would be production ready for testing. master is the production branch. It's a pretty standard setup. Yours may differ of course.

Can you let me know on here when and where the updated files are ready to be tested. Thanks!

Although the video looped, I didn't love the visual reset that took place in stopping the video completely. I achieved a more seamless looping effect seeking to 0 using YouTube seekTo method on ended.
plyr.on('ended', () => {
  plyr.getEmbed().seekTo(0);
  plyr.play();
});

There's still one or two frames of black, but this is pretty close to a seamless loop.

@gideonco I tried your code and it didn't do anything? No loop.
<script> 	plyr.setup({      autoplay: true,      loop: true }); 	plyr.on('ended', () => {     plyr.getEmbed().seekTo(0);     plyr.play(); }); 	</script>

Actually I'm seeing "plyr.on is not a function" should i be downloading a new version somewhere?

As I said ....

Develop is not production ready otherwise it'd be in the master branch.

I solved with this code:
var players = plyr.setup()[0];
players.on('ended', function(event) {
    console.log("fin");
    players.getEmbed().seekTo(0);
    players.play();
});

Another issue has been raised for this so I'll close this one and track it in the other one...
https://github.com/sampotts/plyr/issues/632


V0.15.0 update

v0.15.0 update with a couple of improvements to facilitate the transition

@epost do you still maintain this library?

Hey @sigma-andex! Yes, I'll have a closer look at this asap. It looks great, thanks! 🙏

@epost would you be interested in transferring  the maintainership of this repo?

Hi @sigma-andex, sorry about the long wait. I just had some time this afternoon to try out your PR, but I see that you've closed and deleted it. However, I'm still interested in merging it. Would you be interested in resubmitting?

Wonderful, thanks for your time and effort @sigma-andex! I'm running into some issues during building and test execution, so I'm looking into fixing those.


Reexport flow types from src

This way allows to avoid copying source files and just reuse them for flow types.
If you don't have enough time to maintain this project, I could take care at least about build step and make output smaller. This is important for me since this project is a dependency of react-beautiful-dnd
/cc @chenglou @nkbt

Ping guys. I want to make this project even better.

Yep, I certainly do. Life and things, you know ¯_(ツ)_/¯
Nik Butenko
<EMAIL_ADDRESS>http://github.com/nkbt

I want to make this project even better.

That's great!

Please don't say things like this.  Instead you could ping the maintainers again and offer to help maintain the project. If you still get no response then a fork would be appropriate (and perhaps a merge in the future).
I hope you all have a better day :)

Removing such a dependency is nice. Thanks for the PR @TrySound. I'll merge it. Sorry for the long wait

Yay. Cool. That worked. Sorry for my words.

All good =)


Migrate to the AWS-SDK v2

Hi @zwily This pull request updated the gem to use the AWS-SDK v2 gem and a second commit that is just style changes made by Rubocop. I can drop the second one if you would rather just the functional change.

P.S. This pull request removes the need of pull request #11

@tristanmorgan Sorry for the delay here. I'd love to merge this, but would prefer the rubocop commit be separate if you don't mind. A travis config and a rake task so the rubocopping is automatic would be nice too.

Hi @zwily I have dropped the Rubocop commit and rebased the branch.

Thanks! Seems to work for me.


[5.1] Router group helper

I've thought about this, but the name group is just too ambigious and common for a helper I think.

@taylorotwell So are get() and post()... :wink:

@lukasgeiter @taylorotwell Indeed

I don't think GET and POST are anywhere near as ambiguous as group.

If you really want this function, it's pretty easy to register it yourself in your app. :)

If you write it all in capitals or if it appears in a routing context. However the function name itself is just a very commonly used english verb (and noun as well in the case of post). I'm not for or against adding a group() helper, I just wanted to point out that in my opinion some helpers are already very ambiguous.


Remote modules fail to checkout git tag but succeed with commit hash

Description

An issue was recently uncovered where ITK fails to checkout updated remote modules when a named git tag is used. Checkout succeeds when the equivalent commit hash is provided instead.
https://github.com/InsightSoftwareConsortium/ITK/commit/065c5d853987d399887eee13b2dc92e56813088d attempted to use tags for ITKMinimalPathExtraction and ITKTubeTK modules, which resulted in CMake configuration failures: https://open.cdash.org/build/8323654
-- Performing Test CXX_HAS_DISABLE_OPTIMIZATION_FLAG - Failed
ATTENTION: You have enabled the use of FFTW. This library is distributed under a GPL license. By enabling this option, the ITK libraries binary that is built will be covered by a GPL license and so will any executable that is linked against these libraries.
-- Found Python3: /usr/bin/python3.8 (found suitable version "3.8.10", minimum required is "3.7") found components: Interpreter
-- AnalyzeObjectMap has been renamed to AnalyzeObjectLabelMap
fatal: Needed a single revision
CMake Error at CMake/ITKModuleRemote.cmake:81 (message):
  Failed to get the hash for tag
  '/home/kitware/Dashboards/Tests/ITK/Modules/Remote/MinimalPathExtraction'
Call Stack (most recent call first):
  CMake/ITKModuleRemote.cmake:137 (_git_update)
  CMake/ITKModuleRemote.cmake:236 (_fetch_with_git)
  Modules/Remote/MinimalPathExtraction.remote.cmake:45 (itk_fetch_module)
  Modules/Remote/CMakeLists.txt:10 (include)

-- Configuring incomplete, errors occurred!

Updating to use hashes in https://github.com/InsightSoftwareConsortium/ITK/commit/fd386e4c2911a18561bdce9151f8a719bfcc6883 succeeded.
Steps to Reproduce

Tag an ITK remote module in that module's git history, i.e. [ITKMinimalPathExtraction@v1.2.4](https://github.com/InsightSoftwareConsortium/ITKMinimalPathExtraction/releases/tag/v1.2.4)
Update the remote module's .remote.cmake file with the given tag string, such as at https://github.com/InsightSoftwareConsortium/ITK/blob/master/Modules/Remote/MinimalPathExtraction.remote.cmake#L50
Configure ITK:

> cd path/to/ITK-build
> cmake -DModule_MinimalPathExtraction:BOOL=ON path/to/ITK-src

Expected behavior

Configuration succeeds. The remote module repo is made available at ITK-src/Modules/Remote/ITKMinimalPathExtraction and is checked out at the given tag, i.e. v1.2.4.
Actual behavior

Module fails to checkout the given tag and CMake config fails. See description.
Reproducibility

100%
Versions

ITK v5.3.0
ITKMinimalPathExtraction v1.2.4
Environment

Ubuntu 20.04
CMake 3.22
Additional Information

It is possible that this issue is limited to existing ITK source trees where the remote module was already cloned in the past.

This may be related to differences in what recent versions of git fetch by default (I am not sure of the details offhand, but I have observed some).

Another data point, it seems that tag checkout failed in a clean build in an experimental HASI CI workflow run, so this may not be isolated to existing ITK source trees after all. https://github.com/KitwareMedical/HASI/actions/runs/3758800016/jobs/6387580347


Swiper .css function called with wrong parameters.

CSSStyleDeclaration object takes properties in the propertyName format not property-name
format... JQuery css function might user property-name format, but Dom7 does not seem to do so...
Line 548
if (arguments.length === 2 && typeof props === 'string') {
for (i = 0; i < this.length; i += 1) {
this$1[i].style[props] = value;
}
It sets the property directly in the CSSStyleDeclaration object on the swiper HTMLDomElement.
Line 1319 ... this is right
if (rtl) { slides.css({ marginLeft: '', marginTop: '' }); }
Line 1370
slide
.css(
("margin-" + (swiper.isHorizontal() ? 'top' : left')),
is wrong... and it should probably be
slide
      .css(
        ("margin" + (swiper.isHorizontal() ? 'Top' : 'Left')),

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: EXACT RELEASE VERSION OR COMMIT HASH, HERE.

Live Link or JSFiddle/Codepen or website with isssue: PREFERABLY (IF YOU WANT YOUR ISSUE TO BE RESOLVED ASAP).

Expected Behavior
EXPLAIN WHAT IS TO BE EXPECTED, HERE.
Actual Behavior
EXPLAIN WHAT IS ACTUALLY HAPPENING, HERE.
P.S. Remember, an issue is not the place to ask questions. You can use Stack Overflow
for that.
Before you open an issue, please check if a similar issue already exists or has been closed before.

Yeah, but it works correctly, open any browser and type in console:
document.body.style['margin-left'] = '1000px';

Sorry I do not open an issue without consideration, I could show u an
instance where it actually does not work....but it might be an edge case...
On Fri, Jun 14, 2019, 4:09 PM Vladimir Kharlampidi, <
<EMAIL_ADDRESS>wrote:

Yeah, but it works correctly, open any browser and type in console:
document.body.style['margin-left'] = '1000px';
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nolimits4web/swiper/issues/3099?email_source=notifications&email_token=AK6X3WDCLEGD3UPIJQJPIFLP2P3HPA5CNFSM4HX3FVOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXX22CI#issuecomment-502246665,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK6X3WEVE2VZTX7M6T2E4A3P2P3HPANCNFSM4HX3FVOA
.


iterator API for predictions

For data sets that are too huge to fit in memory or when they need to cascade with 3rd party decoders.

LGTM


Use "resourceid" value to generate anchor in the PDF

If a DITA topic uses the resourceid element:
http://docs.oasis-open.org/dita/v1.2/os/spec/langref/resourceid.html#resourceid
in its prolog:
  <prolog>
    <resourceid id="sqlid00375" appname="dbaccess"/>
  </prolog>

at least when creating PDF with Apache FOP we could use the fox:destination extension:
http://xmlgraphics.apache.org/fop/0.95/extensions.html
to add an anchor in the PDF. In this way, you could have some contextual help linking to various parts of a PDF document using these anchors.

There is a lot of theoretical discussion in this thread that I think got away from the original request -- automatically create anchors in PDF based on <resourceid> values.
There are couple of open design questions there if it is going to be implemented -- what to do with duplicates? All resource IDs or based off of some key (like appname="pdf")? Is there any validation?
Starting with 3.1 I've added the ability to easily insert custom anchors in a plugin, with #2984 and the customTopicAnchor mode that's called for every topic. I'm using that now in our customizations -- when the original topic ID is unique across topics in the map, I insert an anchor, and using that to link within a book.
Of course it's not automatic, but means the function requested here could now be done in a pretty straightforward plugin, with answers to the questions above handled per local rules.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 69 ms

Resolved: ClamAV performance has improved in 03092a0.


GH: Better error message when sender stream is not a stream URL

Improve the error message when the input stream cannot be parsed in the Sender component.
Currently, the last try/catch is dealing with it, which is unhelpful (plus, that one should only be used when something critical happens).
Originally discovered in our forum: https://speckle.community/t/grasshopper-send-and-receive-components-not-working/1663

Turns out message for missing or wrong stream was fine. This error was caused by a possible corrupted installation on the user's computer.
Check thread above for more info.
For reference, here's the stream error message:


arm: board: Add support for board Atmel SAM4S Xplained.

This commit provides support for Atmel SAM4S Xplained board
Pinmux driver is provided with initial support definitions.
External SRAM support is also provided.
Signed-off-by: fallrisk<EMAIL_ADDRESS>

Thanks for the update. This PR can also include all the files in arch/arm/soc/atmel_sam/sam4s folder which need to be changed. Only as a separate commit. I.e. you can submit this PR with two commits. Let me know if you have any problems.

Hey @mnkp, I have made the code in the soc directory into a new PR. @galak he said he preferred that I split the PR. I am going to post the SoC PR so you can have a look at it. When I get home I will make a branch to combine the PRs and test it again. I tested the code last night and it worked with the UART SAM. However, since I added CONFIG_SOC_ATMEL_SAM4S_EXT_MAINCK=y I need to test it again.

@mnkp, @galak I added an image for the SAM4S Xplained docs.

@dbkinder any update on doc review?