Running test suite under Windows fails because of line breaks

I just cloned markdown-it under Windows and executed the test suite. It failed because of "wrong" line breaks in all files. The option "autocrlf" in my Git config is set tot true. The error message that comes several hundred times is
Expected linebreaks to be 'LF' but found 'CRLF'  linebreak-style

I fixed this locally by removing following line from .eslintrc.yml:
linebreak-style:        2

My suggestions:
Remove the above mentioned line from .eslintrc.yml.
Or, if you want to enforce Unix line breaks on all development machines, add a .gitattributes file with e.g. following content:
* text eol=lf

Thanks for details.
There is editor config https://github.com/markdown-it/markdown-it/blob/master/.editorconfig, and it was enough for many years. I would not like to add extra configs, if possible.
May be you know many popular js projects, using .gitattribues? I'm open to new things, if those became mainstream.

I don't know, how mainstream .gitattributes is, but it is supported by Git for ages. The problem with .editorconfig and .eslintrc.yml is that Git does not care about them at the clone process, while the rules in .gitattributes are regarded while cloning.
I use .gitattributes mainly for files that are generated and/or manipulated by some tools, e.g. package.json.

Please, understand me right, here is stable process, working for many years. Personal opinion of single human is not enough to change it. To move forward, i need not feature description, but usage samples/stats in real projects.

Here is a list .gitattributes files after a quick search:

Angular
Atom
Bootstrap
Electron React Boilerplate
Electron
React
VS Code
Webpack

My personal project: MDView

Thank you for info. Looks reasonable.
Take a look, is it ok now?

Tests passed without need for any modification after a fresh clone. I checked some files manually and they have Unix line ending.
Thanks!


Fix some mistakes when using ignSignIn

Some old code doesn't respect the definition of RequireSignInView (the ignSignIn is forgotten).
After #24413, this PR will do more fixes, and rename the strange ignSignIn to optSignIn.
This PR is ready for review, I think we can postpone the "ignSignIn" renaming to another, to make this PR simple and clear.

This PR is ready for review

Can we refactor to readable requireSignIn and ignoreSignIn (assuming it stands for that?

Can we refactor to readable requireSignIn and ignoreSignIn (assuming it stands for that?

ignore is still not that right. My idea is:

This PR is ready for review, I think we can postpone the "ignSignIn" renaming to another, to make this PR simple and clear.

req vs opt seems clear enough, but let's do it in next PR, to keep this PR simple

Can we refactor to readable requireSignIn and ignoreSignIn (assuming it stands for that?

ignore is still not that right. My idea is:

rename them to "optSignIn" ...
I think we can postpone the "ignSignIn" renaming to another, to make this PR simple and clear.

req vs opt seems clear enough, but let's do it in next PR, to keep this PR simple

require and optional please. I hate pointless abbreviations in code.

require and optional please. I hate pointless abbreviations in code.

Really? You wrote df dib .....
But next PR please.

To get an agreement quickly, I started a vote in maintainer channel.

"reqSIgnIn" vs "optSignIn" (optional)
2 or use full name: "requireSignIn" vs "optionalSignIn"

require and optional please. I hate pointless abbreviations in code.

Really? You wrote df dib ..... (so I assumed that req and opt could be fine)
But next PR please (for the renaming)

Well in the CSS case it had a performance benefit, in go code, naming of functions does not affect performance 😆


MISSING_SEMI with pageencoding

I'm getting  the message [cflint] MISSING_SEMI: End of statement(;) expected after pageencoding
component {
    pageencoding "UTF-8";
    //...
}

Is this Lucee only, or ACF as well?

It's ACF.
https://helpx.adobe.com/coldfusion/developing-applications/the-cfml-programming-language/extending-coldfusion-pages-with-cfml-scripting/defining-components-and-functions-in-cfscript.html

fixed in cfparser, version added to dev branch, test added too


RVC-UI working with M1 MacOS, ffmpeg issue

So I got everything setup with the RVC-UI on my Mac and it launches successfully
but when I try to use a .pth file and convert audio, I get an error with the ffmpeg.
Any assistance??

I got the same error while working with RVC-WebUI.
Upon investigation,I found that there are hidden files in the audio(training dataset) directory.I named it as 'work'
In my case,there were ".DS_Store" files.
The ".DS_Store" file is a system file created by macOS Finder to store custom attributes of a folder. It's not an audio file, so attempting to load it as audio is causing the error.
to delete the files(macos)
find . -name ".DS_Store" -print -delete

The error is resolved.Hope it helps you.


Add conda recipe and supporting fixes

This change adds a conda recipe that (1) fetches dependencies, (2) builds a binary deepdive package, and (3) configures a conda environment with deepdive's required environment variables.
Benefits of conda packaging include:
A. Installation of deepdive and dependencies without sudo or root privileges,
B. Fast (binary) installs,
C. Can declare deepdive as a dependency in other conda projects,
D. Sandboxed deepdive instances inside conda environments,
E. Eventual removal of custom bundled-dependency code, and
F. Eventual removal of brittle, manually-managed PATH, PYTHONPATH, [...] environment variables
Supporting fixes on this branch include:
a. Making bundled dependencies optional,
b. Python 3 compatibility fixes, and
c. Fixes to the spouse example
Note that not all dependency packages are available on public repositories, and we can't turn off bundled dependencies or turn on automatic conda packaging for deepdive until they are.

LGTM!

@shahin Thanks for the fixes and conda packaging.  In addition to my inline suggestions, could you add a simple note on maintaining the conda package in doc/developer.md?
Most benefits of conda you listed sounded like tautology to me, i.e., "conda packaging is good for using conda," not solving any new problem, but the first reason is good enough for a complete switch: "A. Installation of dependencies without sudo."
Also, please keep in mind that DD has to allow two different PYTHONPATHs (or environments in general): user/app's and deepdive's own.  So far, the distinction has not been clear throughout DD's code, often causing strange UDF issues or DD crashes.  Naively forcing all DD users to share a single PYTHONPATH and PATH with conda will make certain UDFs impossible to use with DD.  Greenplum is a good example of such nightmare as users are misguided to switch to their internal python for accessing the database.

Most benefits of conda you listed sounded like tautology to me, i.e., "conda packaging is good for using conda,"

Ha ha, I did use the word "conda" there many times. It probably reads tautologically because the word is semantically ambiguous. "conda" refers to any subset of:

A package format,
A dependency system,
An environment manager.

On ubuntu you have deb/apt/nothing (or chroot?), in Python you have setup.py/pip/virtualenv, in Rust you have crate/cargo/multirust. The package format in this PR buys us a ticket to the dependency system (in C) and the environment manager (in D) the same way that Python packaging gets you pip and virtualenv. The conda/conda/conda naming situation we have here is pretty unfortunate.

Also, please keep in mind that DD has to allow two different PYTHONPATHs (or environments in general): user/app's and deepdive's own [...] Naively forcing all DD users to share a single PYTHONPATH and PATH with conda will make certain UDFs impossible to use with DD. Greenplum is a good example of such nightmare as users are misguided to switch to their internal python for accessing the database.

This isn't quite clear to me yet.
Greenplum is a process that runs as a service. So on the one hand it's entitled to its own runtime environment, but on the other it's a huge pain to write and debug Greenplum UDFs because they depend on this special runtime environment that interacts with user environments in brittle ways. I think this is what your last sentence is referring to.
Ideally, Greenplum would be installed in its own conda environment that can be activated and deactivated anywhere by anyone (I have no intention of doing this atm). UDFs would be written, tested, and deployed inside this conda environment.
It sounds like you're saying the Deepdive situation is similar to Greenplum's. Is that right? I'd be grateful for more details or a pointer to a good example UDF.

@shahin could you merge or rebase master?


Public methods should be test methods

Category: Design
Severity: Info
Enabled by default: false
Use cases: a public method is not maked with [TestMethod] or a derived attribute.
AB#2078797

@Evangelink should it work only if that public method is inside a TestClass?


Cleaning up the clang formatting workflows.

@Jake-Carter
What do you think about removing clang-format-run.yml? This workflow is intended to be a manual dispatch on a branch other than main. We don't use it very often and it doesn't seem necessary, since the formatting checks only run when you create a PR or push to main, and I don't think we want to run this on the main branch directly.
clang-format-run-pr.yml is intended to be run withing a PR. We use this much more frequently.

I'm having troubling creating something that will allow us to share the REGEX with the linter workflow.

/clang-format-run

/clang-format-run

@Jake-Carter
What do you think about removing clang-format-run.yml? This workflow is intended to be a manual dispatch on a branch other than main. We don't use it very often and it doesn't seem necessary, since the formatting checks only run when you create a PR or push to main, and I don't think we want to run this on the main branch directly.
clang-format-run-pr.yml is intended to be run withing a PR. We use this much more frequently.

I think we can remove it.  If I need to manually format my branch I will run the formatter locally.

/clang-format-run


Apparently Google doesn't like https everywhere afterall.

Fixes #46 #49 #48

mmm, @ThePixelDeveloper now you reverted "everything"?
Note my comment in https://github.com/ThePixelDeveloper/Sitemap/issues/43#issue-241813996

The namespace http://www.sitemaps.org/schemas/sitemap/0.9 should not change, but the XML Schema is currently loaded from http://www.sitemaps.org/schemas/sitemap/0.9/siteindex.xsd

It is perfectly possible to use:

namespace: http://www.sitemaps.org/schemas/sitemap/0.9
location of the XML Schema: https://www.sitemaps.org/schemas/sitemap/0.9/siteindex.xsd

xsi:schemaLocation="http://www.sitemaps.org/schemas/sitemap/0.9 https://www.sitemaps.org/schemas/sitemap/0.9/siteindex.xsd" xmlns="http://www.sitemaps.org/schemas/sitemap/0.9"

And indeed, this namespace should be http-only: http://www.w3.org/1999/xhtml

I'll take another look over lunch.

great, only change required:

http://www.sitemaps.org/schemas/sitemap/0.9/siteindex.xsd

should become

https://www.sitemaps.org/schemas/sitemap/0.9/siteindex.xsd

appears to be 3 changes in 3 files


[Candidate 3v3 Tournament C ZH v1] too many craters

Observed behavior
The map has too many craters / bullet holes. This distracts and does not look good.
Expected behavior
The map has just a few craters / bullet holes.
Screenshots

fixed


Is 1080p fullscreen possible

Can you write a one-sentence description of your request?
1920x1080 full screen. Is it possible?
What is the current behavior?
Fullscreen defaults to 1280x720.
What is the expected behavior?
1080p fullscreen
What is the motivation / use case for changing the behavior?
Native resolution on more modern displays.

For context, the app is natively 720p in essence because HD-era Taiko no Tatsujin arcade cabinets run 720p and in turn because all TJAPlayer3 skins came to exist in 720p and no other resolution.
I actually have long-term plans to shift the app towards running at 4K internally since that would be even more future-proof and scales down to 1080p and 720p nicely (/2 and /3 respectively) but there is a pile of work to do first.
Along the way, I intend to start decoupling things like the image buffer dimensions from the coordinate system, to decouple animation from the integer coordinate system, etc. so that the app can go through a phase where the coordinate system and assets are all still 720p but things display and animate nicely on higher-resolution displays.
I currently don't have timelines for any of the above, but it is a part of the (as-yet unpublished) roadmap. I'll keep your issue open here as a reminder.


initial webpack-dev-server setup

Steps needed to run:
npm run build
npm run devserver
As you can see this will point to the public folder but not sure how we should set it up so it can point to server/index.js

This is awesome @joshhornby! I think I have an idea of how to get this all working. Will test it out in a couple of hours or tomorrow morning!

@joshhornby Thanks again for the PR. I've used this as a reference and created another branch. I'll close this in favour of https://github.com/choonkending/react-webpack-node/issues/37.

No worries @choonkending. Will be pulling this in tomorrow! Great work!


[OT-2.0] Can't Use Extension Point dita.xsl.html.cover

Trying to add overrides to this extension point fails because the extension point is specified after templates in map2html-coverImpl_template.xsl.
This code:
<dita:extension id="dita.xsl.html.cover" behavior="org.dita.dost.platform.ImportXSLAction" xmlns:dita="http://dita-ot.sourceforge.net"/>
is at the end of map2html-coverImpl_template.xsl, after templates, so on build you get an error that all import statements must occur at the start.
To fix:

Move the templates from map2html-coverImpl_template.xsl to another XSL file.
Specify an xsl:import to the new XSL file before the extension point.

I'll fork & submit a fix later. Trying to get other work done, too. :-)

Fixed in pull request #1896


[mattermost] Add 9.10

See https://github.com/mattermost/mattermost/releases/tag/v9.10.0.

Still a prerelease, see https://github.com/mattermost/mattermost/releases/tag/v9.10.0.
Maybe we should migrate auto configuration to github releases ?


The "Scale Cluster" button is Enable during the process of deleting the cluster

The "Scale Cluster" button is Enable during the process of deleting the cluster. You can use this button, but cluster anyway will be deleted.
Preconditions:
At least one cluster should be created.
Steps to reproduce:

Open the Oshinko Spark Cluster Management home page.
Click on the "Actions" button.
Click on the "Delete Cluster" button, and click on the "Delete" button in the opened window.
Click again on the "Actions" button and click on the "Scale Cluster" button.
Input the Number of workers value equals 2 and click on the "Scale" button.

Actual result: The workers count were not scaled and cluster was deleted.
Expected result: The "Scale Cluster" button should be disabled after starting the process of deleting the cluster.

I agree here, we should disable the "scale" option unless the cluster is in the "running" state.


Move command definitions from root to cmd/

The commands were located in the project root. Community "best practice" suggests placing them under cmd/.
There's still a "main.go" for convenience to run go run . when developing.
Also includes some refactoring to how the commands and their options are instantiated.
This makes the "go install" hint in README.md more regular looking.

Looks like I have managed to break the history links, moved files appear as delete + new here.

And now it's completely broken


Component - Web3 Gallery

[ ] Web3Gallery
[ ] NFT Collection Item
[ ] NFT Item

Design review: 3.2-Lists
Please check and update:

[ ] Add the "Loading" control
[ ] Add the cursor
[ ] UI bug on Display Grid component when search:


{AKS} Fix azdev style issues for internal files

This checklist is used to make sure that common guidelines for a pull request are followed.

Fix azdev style issues for internal files (may refer to PR #7112).

Related command

General Guidelines

[ ] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[ ] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

AKS

please resolve code conflicts

It is better to rerun some related tests to ensure this change would not affect the original logic.

Queued full live test to validate the change.


Card effect

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

103 of 103 (100.0%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 99.406%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
669

Relevant Lines:
673

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

103 of 103 (100.0%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 99.406%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
669

Relevant Lines:
673

💛  - Coveralls


Scorex version

I found that Scorex doesn't have own version.
ScorexApiRoute located in Lagonaki and returns scorex version from Constants defined in Lagonaki project.
What do think about to have two verions in future - for Scorex itself and Lagonaki app ?

Scorex have it's own version, you may found in https://github.com/ScorexProject/Scorex/releases or in maven (http://mvnrepository.com/artifact/org.consensusresearch/scorex-basics_2.11).

Yes, sure. But http api returns it from lagonaki.conf. Theres is no http api in Scorex

You may use a lot of different libraries, the should not provide http method, that return their version. Moreover, scorex is modular framework, you may use scorex-basics-1.2.4, scorex-transaction-1.1.0 and scorex-consensus-1.2.3  in your project, there is no single Scorex version.
If you want to have API method providing version of scorex (and other libraries) - you may write your own.


Leaderboard page not showing results for unverified users?

Turns out it doesn't matter whether they're unverified or verified. It doesn't show results either way.

It turns out the opinions were just not approved, so they weren't showing up in the leaderboard page. Not a bug!


(packageName) choco install wamp-server failed

Chocolatey Version
2.2.2
Chocolatey License
None
Package Version
Not read.
Current Behaviour
In the context of https://github.com/Benjamin-Loison/YouTube-operational-API/issues/267#issuecomment-2052245776

After clicking OK get:
choco install wamp-server

Chocolatey v2.2.2
3 validations performed. 2 success(es), 1 warning(s), and 0 error(s).

Validation Warnings:
 - System Cache directory is not locked down to administrators.
   Remove the directory 'C:\ProgramData\ChocolateyHttpCache' to have
   Chocolatey CLI create it with the proper permissions.

Installing the following packages:
wamp-server
By installing, you accept licenses for the packages.
Progress: Downloading autohotkey.portable 2.0.11... 100%

autohotkey.portable v2.0.11 [Approved]
autohotkey.portable package files install completed. Performing other installation steps.
The package autohotkey.portable wants to run 'chocolateyInstall.ps1'.
Note: If you don't run this script, the installation will fail.
Note: To confirm automatically next time, use '-y' or consider:
choco feature enable -n allowGlobalConfirmation
Do you want to run the script?([Y]es/[A]ll - yes to all/[N]o/[P]rint): A

Extracting C:\ProgramData\chocolatey\lib\autohotkey.portable\tools\AutoHotkey_2.0.11.zip to C:\ProgramData\chocolatey\lib\autohotkey.portable\tools...
C:\ProgramData\chocolatey\lib\autohotkey.portable\tools
Removing ANSI-32 version
 ShimGen has successfully created a shim for AutoHotkey.exe
 The install of autohotkey.portable was successful.
  Software installed to 'C:\ProgramData\chocolatey\lib\autohotkey.portable\tools'
Progress: Downloading vcredist2013 12.0.40660.20180427... 100%

vcredist2013 v12.0.40660.20180427 [Approved]
vcredist2013 package files install completed. Performing other installation steps.
Downloading vcredist2013 64 bit
  from 'https://download.microsoft.com/download/0/5/6/056dcda9-d667-4e27-8001-8a0c6971d6b1/vcredist_x64.exe'
Progress: 100% - Completed download of C:\Users\Benjamin Loison\AppData\Local\Temp\chocolatey\vcredist2013\12.0.40660.20180427\vcredist_x64.exe (6.87 MB).
Download of vcredist_x64.exe (6.87 MB) completed.
Hashes match.
Installing vcredist2013...
vcredist2013 has been installed.
Downloading vcredist2013
  from 'https://download.microsoft.com/download/0/5/6/056dcda9-d667-4e27-8001-8a0c6971d6b1/vcredist_x86.exe'
Progress: 100% - Completed download of C:\Users\Benjamin Loison\AppData\Local\Temp\chocolatey\vcredist2013\12.0.40660.20180427\vcredist_x86.exe (6.21 MB).
Download of vcredist_x86.exe (6.21 MB) completed.
Hashes match.
Installing vcredist2013...
vcredist2013 has been installed.
  vcredist2013 may be able to be automatically uninstalled.
 The install of vcredist2013 was successful.
  Software installed as 'exe', install location is likely default.
Progress: Downloading wamp-server 3.3.2... 100%

wamp-server v3.3.2 [Approved]
wamp-server package files install completed. Performing other installation steps.
WARNING: Url has SSL/TLS available, switching to HTTPS for download
Downloading wamp-server 64 bit
  from 'https://wampserver.aviatechno.net/files/install/wampserver3.3.2_x64.exe'
Progress: 100% - Completed download of C:\Users\Benjamin Loison\AppData\Local\Temp\chocolatey\wamp-server\3.3.2\wampserver3.3.2_x64.exe (320.09 MB).
Download of wampserver3.3.2_x64.exe (320.09 MB) completed.
Hashes match.
Installing wamp-server...

ERROR: Running ["C:\Users\Benjamin Loison\AppData\Local\Temp\chocolatey\wamp-server\3.3.2\wampserver3.3.2_x64.exe" /DIR="C:\wamp" /VERYSILENT /SUPPRESSMSGBOXES ] was not successful. Exit code was '1'. See log for possible error messages.
The install of wamp-server was NOT successful.
Error while running 'C:\ProgramData\chocolatey\lib\wamp-server\tools\chocolateyInstall.ps1'.
 See log for details.

Chocolatey installed 2/3 packages. 1 packages failed.
 See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).

Failures
 - wamp-server (exited 1) - Error while running 'C:\ProgramData\chocolatey\lib\wamp-server\tools\chocolateyInstall.ps1'.
 See log for details.

Expected Behaviour
No response
Steps To Reproduce
Not read.
Environment
- Operating System:
- PowerShell Version:
- Shell:

Chocolatey Log
Not read.

Anything else?
No response

Same issue on my Pegasus Ubuntu Windows (trust) VirtualBox virtual machine which was not having any WampServer installed.

Same issue when considering https://sourceforge.net/projects/wampserver/files/latest/download.

Best way to install Visual C++ Redistributable Packages on https://wampserver.aviatechno.net/ solves this issue.


Splash screen loading, then wallet closes

as title states.. the splash screen of Pulsar wallet shows.. with "loading wallet...." at the bottom.. and then disappears and closes.
it was working and operating perfectly for several days.. and i accidently closed the wallet.. (which shouldn't be a problem to begin with) and went to open it back up.. and this issue is what I am met with..
I've tried everything I know to do.. from uninstalling and reinstalling.. deleting temp folders... make sure of firewall exceptions.. everything..
this is the only thing it will do now... just the splash screen and then closes

i fixed the issue.. i deleted everything i could find in the windows registry, (_run regedit - \HKEY_LOCAL_MACHINE\SOFTWARE\Classes_) and seems to work fine now, my coins and everything safe and sound

I fixed the issue.. I deleted everything i could find in the windows registry that was pulsar related, (run regedit - and deleted pulsar directory in HKEY_LOCAL_MACHINE\SOFTWARE\Classes\pulsar) and seems to work fine now, my coins and everything safe and sound

Hello, sorry for I didnt see your issue at time. Glad it is fixed now, and ty for explaining how you<ve fixed it


Bug report: Cannot create a new SharePoint list using example command with schemaXml option

Description
Attempts to create a new list on an existing SharePoint online team site fails with 'Error: Invalid list template.'
Unfortunately there is no information provided in the error response from SharePoint to help determine the issue with the schemaXml.
The same issue occurs if using the schemaXml example from the documentation page, https://pnp.github.io/cli-microsoft365/cmd/spo/list/list-add/
Steps to reproduce
At an ubuntu terminal, running under wsl:

Install using 'npm install -g @pnp/cli-microsoft365'
Login to SP site using 'm365 login'
Run the following command, copied from the example given in documentation page https://pnp.github.io/cli-microsoft365/cmd/spo/list/list-add/, changing the --webUrl argument to refer to an existing site:

m365 spo list add --title Announcements --baseTemplate Announcements --webUrl https://watconsdev.sharepoint.com/sites/bfwvendors --schemaXml '<List xmlns:ows="Microsoft SharePoint" Title="List1" FolderCreation="FALSE" Direction="$Resources:Direction;" Url="Lists/List1" BaseType="0" xmlns="http://schemas.microsoft.com/sharepoint/"><MetaData><ContentTypes><ContentTypeRef ID="0x01"><Folder TargetName="Item" /></ContentTypeRef><ContentTypeRef ID="0x0120" /></ContentTypes><Fields><Field ID="{fa564e0f-0c70-4ab9-b863-0177e6ddd247}" Type="Text" Name="Title" DisplayName="$Resources:core,Title;" Required="TRUE" SourceID="http://schemas.microsoft.com/sharepoint/v3" StaticName="Title" MaxLength="255" /></Fields><Views><View BaseViewID="0" Type="HTML" MobileView="TRUE" TabularView="FALSE"><Toolbar Type="Standard" /><XslLink Default="TRUE">main.xsl</XslLink><RowLimit Paged="TRUE">30</RowLimit><ViewFields><FieldRef Name="LinkTitleNoMenu"></FieldRef></ViewFields><Query><OrderBy><FieldRef Name="Modified" Ascending="FALSE"></FieldRef></OrderBy></Query><ParameterBindings><ParameterBinding Name="AddNewAnnouncement" Location="Resource(wss,addnewitem)" /><ParameterBinding Name="NoAnnouncements" Location="Resource(wss,noXinviewofY_LIST)" /><ParameterBinding Name="NoAnnouncementsHowTo" Location="Resource(wss,noXinviewofY_ONET_HOME)" /></ParameterBindings></View><View BaseViewID="1" Type="HTML" WebPartZoneID="Main" DisplayName="$Resources:core,objectiv_schema_mwsidcamlidC24;" DefaultView="TRUE" MobileView="TRUE" MobileDefaultView="TRUE" SetupPath="pages\viewpage.aspx" ImageUrl="/_layouts/15/images/generic.png?rev=23" Url="AllItems.aspx"><Toolbar Type="Standard" /><XslLink Default="TRUE">main.xsl</XslLink><JSLink>clienttemplates.js</JSLink><RowLimit Paged="TRUE">30</RowLimit><ViewFields><FieldRef Name="LinkTitle"></FieldRef></ViewFields><Query><OrderBy><FieldRef Name="ID"></FieldRef></OrderBy></Query><ParameterBindings><ParameterBinding Name="NoAnnouncements" Location="Resource(wss,noXinviewofY_LIST)" /><ParameterBinding Name="NoAnnouncementsHowTo" Location="Resource(wss,noXinviewofY_DEFAULT)" /></ParameterBindings></View></Views><Forms><Form Type="DisplayForm" Url="DispForm.aspx" SetupPath="pages\form.aspx" WebPartZoneID="Main" /><Form Type="EditForm" Url="EditForm.aspx" SetupPath="pages\form.aspx" WebPartZoneID="Main" /><Form Type="NewForm" Url="NewForm.aspx" SetupPath="pages\form.aspx" WebPartZoneID="Main" /></Forms></MetaData></List>'
Expected results
A new SharePoint list called Announcements is created.
Actual results
An error result is given by m365:
Error: Invalid list template.
Diagnostics
Executing command spo list add with options {"options":{"debug":true,"title":"Announcements","baseTemplate":"Announcements","webUrl":"https://watconsdev.sharepoint.com/sites/bfwvendors","schemaXml":"<List xmlns:ows=\"Microsoft SharePoint\" Title=\"List1\" FolderCreation=\"FALSE\" Direction=\"$Resources:Direction;\" Url=\"Lists/List1\" BaseType=\"0\" xmlns=\"http://schemas.microsoft.com/sharepoint/\"><MetaData><ContentTypes><ContentTypeRef ID=\"0x01\"><Folder TargetName=\"Item\" /></ContentTypeRef><ContentTypeRef ID=\"0x0120\" /></ContentTypes><Fields><Field ID=\"{fa564e0f-0c70-4ab9-b863-0177e6ddd247}\" Type=\"Text\" Name=\"Title\" DisplayName=\"$Resources:core,Title;\" Required=\"TRUE\" SourceID=\"http://schemas.microsoft.com/sharepoint/v3\" StaticName=\"Title\" MaxLength=\"255\" /></Fields><Views><View BaseViewID=\"0\" Type=\"HTML\" MobileView=\"TRUE\" TabularView=\"FALSE\"><Toolbar Type=\"Standard\" /><XslLink Default=\"TRUE\">main.xsl</XslLink><RowLimit Paged=\"TRUE\">30</RowLimit><ViewFields><FieldRef Name=\"LinkTitleNoMenu\"></FieldRef></ViewFields><Query><OrderBy><FieldRef Name=\"Modified\" Ascending=\"FALSE\"></FieldRef></OrderBy></Query><ParameterBindings><ParameterBinding Name=\"AddNewAnnouncement\" Location=\"Resource(wss,addnewitem)\" /><ParameterBinding Name=\"NoAnnouncements\" Location=\"Resource(wss,noXinviewofY_LIST)\" /><ParameterBinding Name=\"NoAnnouncementsHowTo\" Location=\"Resource(wss,noXinviewofY_ONET_HOME)\" /></ParameterBindings></View><View BaseViewID=\"1\" Type=\"HTML\" WebPartZoneID=\"Main\" DisplayName=\"$Resources:core,objectiv_schema_mwsidcamlidC24;\" DefaultView=\"TRUE\" MobileView=\"TRUE\" MobileDefaultView=\"TRUE\" SetupPath=\"pages\\viewpage.aspx\" ImageUrl=\"/_layouts/15/images/generic.png?rev=23\" Url=\"AllItems.aspx\"><Toolbar Type=\"Standard\" /><XslLink Default=\"TRUE\">main.xsl</XslLink><JSLink>clienttemplates.js</JSLink><RowLimit Paged=\"TRUE\">30</RowLimit><ViewFields><FieldRef Name=\"LinkTitle\"></FieldRef></ViewFields><Query><OrderBy><FieldRef Name=\"ID\"></FieldRef></OrderBy></Query><ParameterBindings><ParameterBinding Name=\"NoAnnouncements\" Location=\"Resource(wss,noXinviewofY_LIST)\" /><ParameterBinding Name=\"NoAnnouncementsHowTo\" Location=\"Resource(wss,noXinviewofY_DEFAULT)\" /></ParameterBindings></View></Views><Forms><Form Type=\"DisplayForm\" Url=\"DispForm.aspx\" SetupPath=\"pages\\form.aspx\" WebPartZoneID=\"Main\" /><Form Type=\"EditForm\" Url=\"EditForm.aspx\" SetupPath=\"pages\\form.aspx\" WebPartZoneID=\"Main\" /><Form Type=\"NewForm\" Url=\"NewForm.aspx\" SetupPath=\"pages\\form.aspx\" WebPartZoneID=\"Main\" /></Forms></MetaData></List>"}}
Creating list in site at https://watconsdev.sharepoint.com/sites/bfwvendors...
Existing access token eyXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX still valid. Returning...

      }
    }
  }
}
Error: Invalid list template.

CLI for Microsoft 365 version
v4.2.0
nodejs version
v14.17.0
Operating system (environment)
Windows
Shell
bash
cli doctor
{                                                                                "os": {                                                                                "platform": "linux",                                                                                "version": "#1 SMP Tue Jun 23 12:58:10 UTC 2020",                                                                                "release": "4.19.128-microsoft-standard"                                                                                },                                                                                "cliVersion": "4.2.0",                                                                                "nodeVersion": "v14.17.0",                                                                                "cliAadAppId": "31359c7f-bd7e-475c-86db-fdb8c937548e",                                                                                "cliAadAppTenant": "common",                                                                                "authMode": "DeviceCode",                                                                                "cliEnvironment": "",                                                                                "roles": [],                                                                                "scopes": [                                                                                "AllSites.FullControl",                                                                                "AppCatalog.ReadWrite.All",                                                                                "ChannelMember.ReadWrite.All",                                                                                "ChannelMessage.Send",                                                                                "ChannelSettings.ReadWrite.All",                                                                                "Directory.AccessAsUser.All",                                                                                "Directory.ReadWrite.All",                                                                                "Group.ReadWrite.All",                                                                                "IdentityProvider.ReadWrite.All",                                                                                "Mail.ReadWrite",                                                                                "Mail.Send",                                                                                "Reports.Read.All",                                                                                "Tasks.ReadWrite",                                                                                "Team.Create",                                                                                "TeamMember.ReadWrite.All",                                                                                "TeamsApp.ReadWrite.All",                                                                                "TeamsAppInstallation.ReadWriteForUser",                                                                                "TeamSettings.ReadWrite.All",                                                                                "TeamsTab.ReadWrite.All",                                                                                "TermStore.ReadWrite.All",                                                                                "User.Invite.All",                                                                                "User.ReadWrite.All",                                                                                "profile",                                                                                "openid",                                                                                "email",                                                                                "AllSites.FullControl",                                                                                "AppCatalog.ReadWrite.All",                                                                                "ChannelMember.ReadWrite.All",                                                                                "ChannelMessage.Send",                                                                                "ChannelSettings.ReadWrite.All",                                                                                "Directory.AccessAsUser.All",                                                                                "Directory.ReadWrite.All",                                                                                "Group.ReadWrite.All",                                                                                "IdentityProvider.ReadWrite.All",                                                                                "Mail.ReadWrite",                                                                                "Mail.Send",                                                                                "Reports.Read.All",                                                                                "Tasks.ReadWrite",                                                                                "Team.Create",                                                                                "TeamMember.ReadWrite.All",                                                                                "TeamsApp.ReadWrite.All",                                                                                "TeamsAppInstallation.ReadWriteForUser",                                                                                "TeamSettings.ReadWrite.All",                                                                                "TeamsTab.ReadWrite.All",                                                                                "TermStore.ReadWrite.All",                                                                                "User.Invite.All",                                                                                "User.ReadWrite.All"                                                                                ]                                                                                }
Additional Info
No response

Hey @danwatford, sorry for the trouble and thank you for taking the time to submit the issue. Let us have a look at it and get back to you as soon as we know more.

I tried using the same schemaXml to create a SP list with pnpjs (code below).
A similar error suggesting there was an error with the list template was given: Invalid list template.
I expect the problem is really down to the content of the schema, but I haven't been successful in constructing any list schemas of my own so was hoping the example would have been a good starting point.
import * as fs from "fs";
import * as path from "path";
import "dotenv/config";
import { SPFetchClient } from "@pnp/nodejs-commonjs";
import { sp } from "@pnp/sp-commonjs";
import "@pnp/odata-commonjs/debug";
import "@pnp/sp-commonjs/webs";
import "@pnp/sp-commonjs/lists";
import validateEnv from "./utils/validateEnv";

validateEnv();

sp.setup({
  sp: {
    fetchClientFactory: () => {
      const fc = new SPFetchClient(
        process.env["SITE_URL"],
        process.env["CLIENT_ID"],
        process.env["CLIENT_SECRET"]
      );

      return {
        fetch: (url, options) => {
          console.log("FETCHING", url, JSON.stringify(options));
          console.log("BODY", options.body);
          return fc.fetch(url, options);
        },
      };
    },
  },
});

async function createList() {
  const schemaBuffer = fs.readFileSync(
    path.join(__dirname, "..", "ApplicationsListSchema.xml")
  );

  try {
    const addListResult = await sp.web.lists.add(
      "DLW Test 1",
      "First attempt at adding list",
      100,
      false,
      {
        SchemaXml: schemaBuffer.toString("utf-8"),
      }
    );

    console.log(addListResult);
  } catch (err) {
    console.error(JSON.stringify(err));
  }
}

createList();

One area to look at might be how double-quotes in the schema xml are being escaped for inclusion in the JSON body of the request sent to spo.
From the output of --debug, notice that only the first double-quote in xmlns:ows="Microsoft SharePoint" is escaped.
"data": {
"Title": "Announcements",
"BaseTemplate": 104,
"SchemaXml": "<List xmlns:ows=\"Microsoft SharePoint" Title="List1" FolderCreation="FALSE" Direction="$Resources:Direction;" Url="Lists/List1" BaseType="0"

Actually, ignore my previous comment, some backslashes were getting dropped in rendering my first post to this issue. Now fixed.
Instead it looks like the first double-quote in the schema xml is getting double escaped (note the 3 backslashes after xmlns:ows):
"SchemaXml": "<List xmlns:ows=\\\"Microsoft SharePoint\" Title=\"List1\"

@danwatford thank you for the additional information and also comparing the functionality with PnPjs. If you have a List Schema that works with the RAW rest API or any other method (PnPjs, PnP PowerShell, CSOM) then it would be a great starting point for us to check if everything's working as intended.

FWIW, I can reproduce the issue you mentioned using the sample from our docs. I also tried to use a different list schema and it's failing with the same error as well. If we can reproduce the issue with schemas that we're pretty sure used to work in the past, then it could indicate that something's changed in the SharePoint API that we're using.

@danwatford thank you for the additional information and also comparing the functionality with PnPjs. If you have a List Schema that works with the RAW rest API or any other method (PnPjs, PnP PowerShell, CSOM) then it would be a great starting point for us to check if everything's working as intended.

Hi @waldekmastykarz , unfortunately I haven't been able to get any schemaXml content working for list creation. I haven't got as far as using the raw rest-api yet.

FWIW, I can reproduce the issue you mentioned using the sample from our docs. I also tried to use a different list schema and it's failing with the same error as well. If we can reproduce the issue with schemas that we're pretty sure used to work in the past, then it could indicate that something's changed in the SharePoint API that we're using.

@waldekmastykarz , that's good news.
I did see something in how the schemaXml command line was being processed before making REST request to SPO:
    if (options.schemaXml) {
      requestBody.SchemaXml = options.schemaXml.replace('\\', '\\\\').replace('"', '\\"');
    }

In the calls to the replace methods, only the first occurrence of the pattern will be replaced, hence the differences in escaping of the double quotes in my earlier comments.
To replace all occurrences we need to use regular expressions:
replace(/\\/g, '\\\\').replace(/"/g, '\\"')
But I'm not sure what these replacements are trying to achieve and couldn't find any information on how the SPO API expects SchemaXml to be encoded.

Thank you for the extra information @danwatford. It could be indeed the case that our replacement logic is faulty. If I recall correctly, this used to work as intended and I even used it in the past myself, but it could be that something has changed. Let's try to find a working list schema first, so that we have a reliable sample to test the API and then the CLI with.

Will have a look upcoming days; this might be a bigger problem with our current implementation. I had similar issues when creating new list columns from XML, updating an existing view, and sending adaptive cards.
It might be related to Windows vs Linux as the commands for creating columns seems to be working on Unix. But haven't investigated to much so will spend some time checking a few more commands that work with complex XML before diving into a solution.

Will have a look upcoming days; this might be a bigger problem with our current implementation.

If this is the issue, then it's coming from the underlying shell that requires a specific sequence for escaping quotes and it's nothing that we could do about. One workaround for such cases is to store the payload in a file and pass the argument from a file using @. That way you don't need to worry about escaping quotes.

So for the list view and list columns all work properly, turns out I just made a typo in escaping a column name resulting in an error.   Same for adaptive cards. they are just really complex to get the escaped json correct. Looks like it still goes wrong on schemaXml though.

Looks like it still goes wrong on schemaXml though.

Even when you pass contents from a file?

Hey @appieschot, are you still looking into this?

Resetting as available for picking due to lack of response

I should be able to have a look at this

Strong work @MathijsVerbeeck!

After doing some research, I've noticed that this is a common issue. I've also had a look at different PnP repositories, and none of these support creating a list by using the SchemaXML option.
I have found one example of a user which will 'parse' the XML from the SchemaXML and will set all the possible list options upon creating. The example can be found below: https://reshmeeauckloo.wordpress.com/2016/10/22/create-lists-libraries-from-schema-as-xml-using-csom/
Also when testing this using CSOM in a simple console application the same error is thrown, even while using the most simple SchemaXML that you can imagine. I have read examples of users for whom this works, but that was on very old environments (SP2010/SP2013), and while using SSOM code. Can't really imagine that it has every worked unfortunately.
Any other thoughts?

@jansenbe, @patrick-rodgers do you happen to know if it still should be possible to create a SharePoint List from SchemaXml? It seems that all our attempts and samples that we can find are failing. I wonder if anything has changed or if this method shouldn't have been used in the first place. We'd appreciate your help.

I am not aware that it doesn't work but haven't tried it in many years.
Looking at the source the Create REST method still accepts a custom schemaXML and doesn't appear to have changed. Have you tried with a super basic schemaXML?

Thanks for your reply, @patrick-rodgers. I think @MathijsVerbeeck tried a basic schema XML. @MathijsVerbeeck, would you mind sharing what you tried here?

I did indeed @waldekmastykarz. I've tried using the following Schema:
<List Title='CustomList' BaseType='0'></List>
Which is the most basic I suppose. I also tried adding the property Type to specify the list template, but that didn't help either.

@waldekmastykarz Any idea on how we continue with this issue? Should we stop supporting the option --schemaXml?

If we can't get it to work for any case, let's update the docs and mark the option as deprecated, and to be removed in the next major version of the CLI.


Request: Disable user-interaction in filtereditor

(Original issue 19 created by None on 2011-07-05T08:06:20+00:00)
What steps will reproduce the problem?
1. Add a predefined filter
2. Call setEnabled(false)
3. The user can still delete the filter and choose the nofilter option.

What is the expected output? What do you see instead?
I would like to provide a predefined filter and totally disable user-interaction.

What version of the product are you using? On what operating system?
4.1.4

Please provide any additional information below.
Allow the user to see what filter is in effect, but don't allow changing it in any
way.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 2011-07-05 08:06:20

(Original issue 19 last updated on 2015-04-26T00:23:15.020079+00:00)
(Issue automaticaly closed due to status in Bitbucket: resolved)

(Original comment by coderazzi on 2011-07-29T11:52:35+00:00)
(No text was entered with this change)

Original issue reported on code.google.com by coderazzi on 2011-07-29 11:52:35

(Original comment by coderazzi on 2011-11-10T16:18:43+00:00)
Implementation is ready for release (4.2.0).
New API methods on IFilterEditor interface:
    /**
     * Enables or disables the user's interaction; if disabled, the control
     * is disabled but the associated filter remains in place.
     */
    void setUserInteractionEnabled(boolean enable);

    /** Returns the user interaction mode. */
    boolean isUserInteractionEnabled();

The look and feel of an editor with user interaction disabled is not distinguishable
from an editor that is disabled, as I do not foresee the case where some editors are
disabled, and others have the content fixed (and if it would be possible, still would
not like to expand further the possible colors combinations)

Original issue reported on code.google.com by coderazzi on 2011-11-10 16:18:43

(Original comment by coderazzi on 2011-11-10T16:19:14+00:00)
(No text was entered with this change)

Original issue reported on code.google.com by coderazzi on 2011-11-10 16:19:14

(Original comment by coderazzi on 2011-11-14T23:13:24+00:00)
(No text was entered with this change)

Original issue reported on code.google.com by coderazzi on 2011-11-14 23:13:24

(Original comment by None on 2011-11-15T11:15:43+00:00)
Perfect. Works just as expected. Thanks

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 2011-11-15 11:15:43

(Original comment by coderazzi on 2015-04-26T00:23:15.028087+00:00)
Already resolved (previously, before migrating to BitBucket)


Disable writing to config file

Describe the solution you'd like
Add an option to create and manipulate a configuration file in memory without writing it to a file.

Will be implemented on ConfigHandler v3


Fixed compile error when ros-indigo-opencv3 is installed

I was unable to compile the marti_common when ros-indigo-opencv3 was installed. This patch fixed the issue.

Good catch.  I've noticed that swri_image_util, swri_opencv_util, and swri_transform_util also all depend on OpenCV 2 but do not specify the version number; could you add those to your pull request?

I updated the CMakeFiles.txt. It would be great if you could merge it into your repository.

LGTM. @pjreed, should we add a Travis test for this?

Yeah, probably.  The easiest way to test it would probably be to just manually install the ros-indigo-opencv3 packge before compling; it's very obvious when it fails.


grammar fix in arbitrary.smap documentation

merged as part of https://github.com/jsverify/jsverify/pull/277

thanks!


[Settings] Migrate Shortcut Guide Settings Tests

Summary of the Pull Request

Migrated shortcut guide settings test to a .Net Standard project.
Target PR: #5754

PR Checklist

[ ] Applies to #xxx
[x] CLA signed. If not, go over here and sign the CLA
[x] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Validation Steps Performed

Run tests and verify output on the settings UI.

 Im not sure if thats something I can help with, @crutkas FYI.

@tkoup did you subscribe to the PR?


Surprising 'path' handling in $VIMRUNTIME/ftplugin/ruby.vim

Steps to reproduce
Default behaviour (no filetype detection)
The default value of 'path' includes . and ,,, which allow us to find files in the directory of the current file and the working directory. This makes it possible to use gf and other commands that depend on 'path' for project-wide navigation out of the box.

Create a test directory: $ mkdir testrubypath.

Change to that directory: $ cd $_.

Create two Ruby files: $ touch a.rb b.rb.

Open a.rb in a barebones Vim: $ vim -u NONE a.rb.

Check the global value of 'path' with :set path?. The output should be .,/usr/include,,.

Add the line below to the buffer and write it:
require 'b.rb'

Press gf on the filename. You should jump to b.rb.

Current behaviour (uses $VIMRUNTIME/ftplugin/ruby.vim)
The Ruby ftplugin overrides the default 'path' value in a way that makes it impossible to use gf and other commands that depend on 'path' for project-wide navigation.

In the same directory as above, start a clean Vim: $ vim --clean.

Check the global value of 'path' with :set path?. The output should be .,/usr/include,,.

Open a.rb: :edit a.rb.

Check the local value of 'path' with :setlocal path?. The output should be a comma-separated list of Ruby-specific directories. Something like this (YMMV):
path=/Library/Ruby/Site/2.3.0,/Library/Ruby/Site/2.3.0/x86_64-darwin18,/Library/Ruby/Site/2.3.0/universal-darwin18,/Library/Ruby/Site,/System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib/ruby/vendor_ruby/2.3.0,[...]

Note that the default value as disappeared, as well as the possibility to find files in the directory of the current file and the working directory.

Press gf on the filename. You should get an E345 error.

Expected behaviour

In the same directory as above, start a clean Vim: $ vim --clean.

Check the global value of 'path' with :set path?. The output should be .,/usr/include,,.

Open a.rb: :edit a.rb.

Check the local value of 'path' with :setlocal path?. The output should have a comma-separated list of Ruby-specific directories appended to the default value. Something like this (YMMV):
path=.,/usr/include,,/Library/Ruby/Site/2.3.0,/Library/Ruby/Site/2.3.0/x86_64-darwin18,/Library/Ruby/Site/2.3.0/universal-darwin18,/Library/Ruby/Site,/System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib/ruby/vendor_ruby/2.3.0,[...]

Press gf on the filename. You should jump to b.rb.

Operating system
Unix-like system
Version of Vim
8.2.3455
Logs and stack traces
No response

In $VIMRUNTIME/ftplugin/ruby.vim, there is the following function:
function! s:build_path(path) abort
  let path = join(map(copy(a:path), 'v:val ==# "." ? "" : v:val'), ',')
  if &g:path !~# '\v^%(\.,)=%(/%(usr|emx)/include,)=,$'
    let path = substitute(&g:path,',,$',',','') . ',' . path
  endif
  return path
endfunction

The path argument is a list of Ruby-related directories obtained via introspection in another function.
The conditional checks the global value of 'path'.
If it is different from the default value, the function returns it, slightly modified, with the list of Ruby-related paths is appended to it.
If it is the default value, the function ignores it and returns the list of Ruby-related paths.

That logic is quite surprising. I would expect the ftplugin to always append its list to the global 'path', no matter what it is set to.

have you asked @tpope as the maintainer about it?

The output should have a comma-separated list of Ruby-specific directories appended to the default value.

I would expect the ftplugin to always append its list to the global 'path', no matter what it is set to.

The ruby filetype plugin started setting the 'path' option 16 years ago.  At that time already, it ignored the default value of 'path':
if !exists("s:rubypath")
  if executable("ruby")
    if &shellxquote == "'"
      let s:rubypath = system('ruby -e "print $:.join(%q{,})"' )
    else
      let s:rubypath = system("ruby -e 'print $:.join(%q{,})'" )
    endif
    let s:rubypath = substitute(s:rubypath,',.$',',,','')
  else
    " If we can't call ruby to get its path, just default to using the
    " current directory and the directory of the current file.
    let s:rubypath = ".,,"
  endif
endif

let &l:path = s:rubypath

It did set 'path' with .,, (which would be enough for your example to work as expected), but only if Vim could not find an executable named ruby.
The change which breaks your example was committed 10 years ago.  Before then, when there was a ruby executable, the path included the directory of the current file unconditionally, as well as the current working directory on the condition that the output of $ ruby -e "print $:.join(%q{,})" contained a dot (e.g. foo,.,bar):
let s:ruby_path = '.,' . substitute(s:ruby_path, '\%(^\|,\)\.\%(,\|$\)', ',,', '')
                  ^--^                                                    ^^

After that, it no longer included neither of them:
let s:ruby_path = substitute(&g:path,',,$',',','') . ',' . s:ruby_path

As for the reason why, you would have to read this thread, in particular this comment:

Regarding clobbering the global path, it occurs to me there's a very obvious compromise I had not considered. What if we kill the global path if it's the default (.,/usr/include,,), but append to it otherwise? Seems win-win to me.
But while we're slaughtering sacred cows, there's another aspect of the path I want to touch. By default, it includes both the current working directory and the current file's containing directory. Now when I'm in ~/src/myapp and I'm editing lib/foo/bar.rb, I want neither ~/src/myapp nor ~/src/myapp/lib/foo added to my path.

While I was reading the plugin, I found another possible issue.
Minimal example:
vim9script
["require 'b.rb'"]->writefile('/tmp/a.rb')
['you are in b.rb']->writefile('/tmp/b.rb')
filetype plugin on
set path=.,,xxx
cnoreabbrev find invalid
edit /tmp/a.rb

Expected result: Vim loads /tmp/b.rb.
Actual result: E492: Not an editor command: invalid b.rb is given.
The issue is caused by:

the gf mapping which is recursive, and presses some keys – including a space – while in command-line mode (where there can be abbreviations)
the command-line mode abbreviation which – in practice – could be installed on the user side

The plugin could be made more reliable by prefixing the spaces with <C-V> to suppress any possible expansion of abbreviations.  As a suggestion, here is a patch:
diff --git a/runtime/ftplugin/ruby.vim b/runtime/ftplugin/ruby.vim
index 4a476fd8c..bb3e76d09 100644
--- a/runtime/ftplugin/ruby.vim
+++ b/runtime/ftplugin/ruby.vim
@@ -220,10 +220,10 @@ if !exists("g:no_plugin_maps") && !exists("g:no_ruby_maps")
   call s:map('n', '<silent>', '<C-W>}      <SID>:exe v:count1."ptag <Plug><ctag>"<CR>')
   call s:map('n', '<silent>', '<C-W>g}     <SID>:exe        "ptjump <Plug><ctag>"<CR>')

-  call s:map('n', '<silent>', 'gf           <SID>c:find <Plug><cfile><CR>')
-  call s:map('n', '<silent>', '<C-W>f      <SID>c:sfind <Plug><cfile><CR>')
-  call s:map('n', '<silent>', '<C-W><C-F>  <SID>c:sfind <Plug><cfile><CR>')
-  call s:map('n', '<silent>', '<C-W>gf   <SID>c:tabfind <Plug><cfile><CR>')
+  call s:map('n', '<silent>', 'gf           <SID>c:find<C-V> <Plug><cfile><CR>')
+  call s:map('n', '<silent>', '<C-W>f      <SID>c:sfind<C-V> <Plug><cfile><CR>')
+  call s:map('n', '<silent>', '<C-W><C-F>  <SID>c:sfind<C-V> <Plug><cfile><CR>')
+  call s:map('n', '<silent>', '<C-W>gf   <SID>c:tabfind<C-V> <Plug><cfile><CR>')
 endif

 let &cpo = s:cpo_save

Entering the command-line with <Cmd> instead of a colon would be even better; it would fix the issue and let us drop many <silent>/<C-U>, but then we would lose <C-R>=, which the plugin relies on.
cc @tpope

Well, I don't do Ruby so I never faced that problem. I opened this issue after trying to answer this SO question and what I found seemed insane to me: a built-in ftplugin purposefully breaking a built-in feature. How nice.
Anyway, I don't have skin in the game so whatever the pope says…

and what I found seemed insane to me: a built-in ftplugin purposefully breaking a built-in feature. How nice.

Why so emotional (again)?

A bit of context: The initial 2005 implementation (which pre-dates my involvement) is not without precedent: it's more a or less a direct copy of ftplugin/perl.vim.  One notable difference is that Ruby at the time, unlike Perl, included the current working directory in the load path.  That changed in Ruby 1.9.2, released in 2010, and my subsequent 2011 change took that into account.  That is to say, the 2011 change was not a change in philosophy on the inclusion of the empty entry ,,.
The Stack Overflow post in question is a special case where the fix works more or less by coincidence.  If you are editing a file named lib/a.rb, and you press gf on require 'a/b, then yes, having . in 'path' will enable jumping to lib/a/b.rb.  But if you're editing lib/a/b.rb, then that same change won't enable you to jump from require 'a/b/c to lib/a/b/c.rb.  Instead it would enable you to jump to lib/a/a/b/c.rb, which is not how require works. The correct fix is to add /path/to/lib to 'path', but that's a hard problem, better suited to a plugin than a runtime file.
To quickly touch on a few other points:

If the ftplugin were to "always append its list to the global 'path'", then the overwhelming majority of users would end up with /usr/include in there taking precedence over everything else.  I consider this to be self-evidently absurd.
The find abbreviation bug is valid, but throwing <C-V> in there would replace it with a <C-V> map bug.  I'm sure we can find another solution.  I would encourage opening an issue at https://github.com/vim-ruby/vim-ruby.
Other feedback is best submitted to https://github.com/vim-ruby/vim-ruby as well.

If the ftplugin were to "always append its list to the global 'path'", then the overwhelming majority of users would end up with /usr/include in there taking precedence over everything else. I consider this to be self-evidently absurd.

Was setlocal path-=/usr/include, considered at some point?

I don't know the details of Ruby, but assuming that "gf" is only used in a "require" statement is wrong.  The file name could be in a comment or in a string, and the user can expect "gf" to try find that file.  Therefore adding the current directory ",," and especially the directory of the file "." is useful.  Putting these at the end will avoid that the current behavior for using "gf" on a require of a library file changes.  Dropping /usr/include makes sense.  Thus the logic would be to include ",," if 'path' already has ",," and include "." if 'path' already had ".".

This is actually accounted for somewhat in the gf map that is also provided. Filenames preceded by require are special cased. There's other special cases for things like require_relative which use a directory of the file behavior identical to ".".
This approach obviously not and can never be perfect. I'll ensure your proposed change makes it into the next upstream sync.

So let's close this hear, assuming it will be handled at https://github.com/vim-ruby/vim-ruby
Note to interested readers: If there are other issues, please directly use the upstream repository to discuss.
Thanks!


Add timja to winstone

Description
Adding myself to winstone a core team maintained component, need to sort out a core team here :p

Submitter checklist for adding or changing permissions

Always

[ ] Add link to plugin/component Git repository in description above

For a newly hosted plugin only

[ ] Add link to resolved HOSTING issue in description above

For a new permissions file only

[ ] Make sure the file is created in permissions/ directory
[ ] artifactId (pom.xml) is used for name (permissions YAML file).
[ ] groupId / artifactId (pom.xml) are correctly represented in path (permissions YAML file)
[ ] Check that the file is named plugin-${artifactId}.yml for plugins

When adding new uploaders (this includes newly created permissions files)

[ ] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[ ] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[ ] Make sure to @mention the users being added so their GitHub account names are known if they require GitHub merge access (see below).
[ ] All newly added users have logged in to Artifactory at least once

Reviewer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
[ ] If security contacts are changed (this includes add/remove), ping the security officer (currently @Wadeck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

@olamy @oleg-nenashev if you get a chance can you retroactively approve this?

@olamy @oleg-nenashev if you get a chance can you retroactively approve this?

@halkeye it's a core component, I already had 'admin' access as part of the core team.
Also see https://github.com/jenkins-infra/repository-permissions-updater/pull/2369 where it was made explicit


Set Query Param results in a 404 Page Not Found.

I am using the query param to try and filter out some results and this module works when querying the params but not when trying to update them
const [categoryParam, setCategoryParam] = useQueryParam(
    'category',
    StringParam
  )

<button
                type="button"
                onClick={() => {
                  setCategoryParam(category)
                }}
>My Button</button>

When I try this, and click the button, I get immediately redirected to the 404 page saying the page doesn't exist. What's stranger is if I type something into the gatsby develop's 404 Page search box, as soon as I type a letter it loads the page correctly.
Why is this module causing a 404 when setting a query param?

Having the same issue; also reported in https://github.com/alexluong/gatsby-packages/issues/41
Theres a solution that involves wrapping the page element with <QueryParamProvider> manually, and it fixes the 404 error but introduces a page flicker effect on transition instead, which isn't optimal...
Any chance for a fix on the Gatsby plugin soon?

The first workaround described in https://github.com/alexluong/gatsby-packages/issues/41#issuecomment-961913780 also fixes the 404 issue without the flicker effect, but feels like a bit of a hack..


Tickets/sitcom 1120 Simple Conversion of Jupyternotebook.

I directly converted the Jupyter Notebook to the technote for this version as there would be a chance that we need this notebook very soon.
https://sitcomtn-143.lsst.io/v/SITCOM-1120/index.html
Now I am thinking that I will use tags (what you showed me before) so
v0.1   - simply converted one from the original notebook  (this version)
For v.0.2 - apply TN-102 structure with a little more description.

move some functions to the separated Python code (and call the functions)
add an abstract, some background, and more descriptions
add just a few more descriptions and apply some structure on tn-102

Hi Yijung ! Thanks for this conversion.


std_msgs: 0.5.10-0 in 'indigo/distribution.yaml' [bloom]

Increasing version of package(s) in repository std_msgs to 0.5.10-0:

upstream repository: git@github.com:ros/std_msgs.git
release repository: https://github.com/ros-gbp/std_msgs-release.git
distro file: indigo/distribution.yaml
bloom version: 0.5.21
previous version for package: 0.5.9-1

Waiting for coordinated release.


Fix Spacebar double pausing on Safari

Related:
Fixes https://github.com/vidstack/player/issues/788
Description:
Fix spacebar pausing and unpausing on Safari and only does it once instead.
Ready?
No.
Anything Else?
Find fix on Chrome browsers and see if there are any problem with Firefox.
Review Process:
Test on Safari, Chrome, and Firefox, as well as any browsers with planned support.

Actually https://github.com/vidstack/player/commit/14ae6a38a0630bdf4b4309201561110add639173 only fixed chromium and not safari so the issue still persists.


.log.resources

Hi there,
so when I enter 'log.resources' as extension, it will not work.
It will only work with 'resources' however, that would also include files like 'LogConfigTemplate.xml.resources' which you would not want to delete.
So any ideas about that?

Perhaps it is an idea to also allow subfolder selection, so you can create multiple jobs, and skip some subfolders.
I would like to empty the _default/Logs folder of .log.resources files
But not the _default/Logs/LogConfig folder for example.

And perhaps I would also like to empty App_Data/ExtensionPackages


Insert at specific position

The Table positions are documented to only be set when they are parsed from a document. Is it possible in insert a table at a specific place of a document?
E.g. if I want to add a [profile.custom] before or after [profile.release] in https://github.com/CosmWasm/cosmwasm/blob/v1.0.0-beta4/contracts/hackatom/Cargo.toml#L13-L22, is that possible?
Right now I do
                let mut profile = toml_edit::Table::new();
                profile.set_implicit(true);
                profile.insert("custom", toml_edit::Item::Table(toml_edit::Table::new()));
                doc.insert("profile", toml_edit::Item::Table(profile));

which appends [profile.custom] at the end of the document.

The Table positions are documented to only be set when they are parsed from a document. Is it possible in insert a table at a specific place of a document?

Which piece of documentation are you referring to?  set_position exists to be able to control the ordering of all tables within a document.

Ah, cool. I guess it was a combination of
    // used for putting tables back in their original order when serialising.
    // Will be None when the Table wasn't parsed from a file.
    pub(crate) position: Option<usize>,

    /// Sets the position of the `Table` within the `Document`.
    ///
    /// Setting the position of a table will only affect output when
    /// `Document::to_string_in_original_order` is used.
    pub fn set_position(&mut self, position: usize) {

and
    /// The position of the `Table` within the `Document`.
    ///
    /// Returns `None` if the `Table` was created manually (i.e. not via parsing)
    /// in which case its position is set automatically.
    pub fn position(&self) -> Option<usize> {
        self.position
    }

that made me think the position is exclusive to parsed elements.
But it seems to work with something like this 🚀
                let mut profile = toml_edit::Table::new();
                profile.set_implicit(true);
                profile.insert("release", toml_edit::Item::Table(toml_edit::Table::new()));
                profile.set_position(3);
                doc.insert("profile", toml_edit::Item::Table(profile));

If I insert a new Table into a document, do I have to manually adjust the positions of all elements later on?

If I insert a new Table into a document, do I have to manually adjust the positions of all elements later on?

Yes.  Something we could do to improve this is to sort by (pos, path) and not just pos.  This would allow people to reuse an existing tables position and get more predictable results.

I guess what confused me most is that the position information is stored in the child instead of the parent. I would have expected that in the insert method.

I guess what confused me most is that the position information is stored in the child instead of the parent. I would have expected that in the insert method.

Hopefully the explanation that the position is global helps clarify why its on the table and not on the parent.
It also can't be on insert because position handling of key-value pairs is different (local to the table its visually under, in the ideal case but its not fully implemented for dotted keys)

As for the original request of this issue, because of the difference in how table vs key-value pair positioning works (not just how implemented), I'm inclined to close this out.

Thanks a lot for the help. Much appreciated.


Model forecasting score function

Input: array of problems, array of observables, time array, dataset array (same as datafit, [S => data for S, I => data for I])
Output: Array L2 norms of forecasts against the data for each model
model_forecast_score

Need a model forecast visualization as well


Added Virtualenv checks to superpack_install script. Allows use in a Virtualenv, without breaking non-virtualenv use.

Virtualenv comes with setuptools, so ez_setup is not needed. In a Virtualenv, the Python path is changed (and set in the environment by the activate script). Virtualenv installation does not require (and should not use) sudo.

Thanks Barry,
I have since replaced this script with one that installs packages from git, rather than from my Dropbox using curl. However, I will integrate your suggestion. I was thinking about using hash or command to check for the existence of required tools before trying to install them.

Sure, no need to take the patch as is. Using hash or command makes sense to me. If you'd like me to update the patch, I'm happy to, or you can just take what you want.

Have a look at the changes in 42e53ffeb0419332c6d08a974a6c55512e00c615
Let me know if there are issues with Virtualenv.

Chris, Sorry for the delayed reply. Looks good. I'll submit a new pull request if we have any additional changes (none at the moment). Thanks!


cv::Exception when trying to calibrate camera

I installed the whole package and checked if both libfreenect2 and iai_kinect2 works. They work both with opengl and opencl. My problem is when I try to calibrate the camera I get the following error:
Start settings:
       Mode: calibrate
     Source: color
      Board: chess
 Dimensions: 5 x 7
 Field size: 0.03
Dist. model: 5 coefficients
Topic color: /kinect2/hd/image_mono
   Topic ir: /kinect2/sd/image_ir
Topic depth: /kinect2/sd/image_depth
       Path: ./

restoring files...
starting calibration...
calibrating intrinsics...
OpenCV Error: Assertion failed (nimages > 0) in calibrateCamera, file /usr/share/OpenCV/opencv-2.4.11/modules/calib3d/src/calibration.cpp, line 3415
terminate called after throwing an instance of 'cv::Exception'
  what():  /usr/share/OpenCV/opencv-2.4.11/modules/calib3d/src/calibration.cpp:3415: error: (-215) nimages > 0 in function calibrateCamera

The order of the commands is exactly as mentioned in Kinect2 Calibration:
rosrun kinect2_bridge kinect2_bridge _fps_limit:=2
rosrun kinect2_calibration kinect2_calibration chess5x7x0.03 record color
rosrun kinect2_calibration kinect2_calibration chess5x7x0.03 calibrate color

Also, sometimes I get the error:
GL_MAX_RECTANGLE_TEXTURE_SIZE is too small: 0

but, if I run it again it works normally

No images are restored and a calibration with no input data will fail. You need to run the record color first and record data. There should be a couple of files created in the current directory. Then execute calibrate color in the same directory.

Thanks for the quick reply; I tried to run it but, it doesn't save any data in the folder.
Also, I noticed that when it prompts me to input a option it seems that it doesn't accept the input.
I run:
rosrun kinect2_bridge kinect2_bridge _fps_limit:=2
rosrun kinect2_calibration kinect2_calibration chess5x7x0.03 record color
rosrun kinect2_calibration kinect2_calibration chess5x7x0.03 record data

From record color I get the below message:
Start settings:
       Mode: record
     Source: color
      Board: chess
 Dimensions: 5 x 7
 Field size: 0.03
Dist. model: 5 coefficients
Topic color: /kinect2/hd/image_mono
   Topic ir: /kinect2/sd/image_ir
Topic depth: /kinect2/sd/image_depth
       Path: ./

starting recorder...
Controls:
   [ESC, q] - Exit
 [SPACE, s] - Save current frame
        [l] - decreas min and max value for IR value rage
        [h] - increas min and max value for IR value rage
        [1] - decreas min value for IR value rage
        [2] - increas min value for IR value rage
        [3] - decreas max value for IR value rage
        [4] - increas max value for IR value rage
init done
opengl support available

and from record data the below message:
Start settings:
       Mode: record
     Source: sync
      Board: chess
 Dimensions: 5 x 7
 Field size: 0.03
Dist. model: 5 coefficients
Topic color: /data/hd/image_mono
   Topic ir: /data/sd/image_ir
Topic depth: /data/sd/image_depth
       Path: ./

starting recorder...
Controls:
   [ESC, q] - Exit
 [SPACE, s] - Save current frame
        [l] - decreas min and max value for IR value rage
        [h] - increas min and max value for IR value rage
        [1] - decreas min value for IR value rage
        [2] - increas min value for IR value rage
        [3] - decreas max value for IR value rage
        [4] - increas max value for IR value rage

I think I found the problem... The data that I should get from the calibration are: one .png image and xxxx_color_points.yaml?
if yes, then I was inserting the argument for saving the frame from the terminal rather from the window... I did not know that you used waitKey() to wait an input.
I am sorry for the inconvenience and thanks for the help .


First run of dsc_script will fail if powershell 4.0 is not installed

@jdmundrawala, a quick fix might be to move this detection to execution phase rather than compile:
http://www.getchef.com/blog/2014/09/08/chef-11-16-gets-into-powershell-dsc/#comment-1582426718.
See this comment:
Regarding dsc_script and the Powershell 4 requirement, it seems that on a vanilla 2008 machine (no PS4) this presents a problem using this in the very first run; even if the very first recipe in the run list is the installation of PS4, if any other recipe contains the usage of dsc_script, the compile phase will break with the message 'Powershell 4.0 or higher
was not detected on your system and is required to use the dsc_script resource' thus preventing the installation of PS4, thus preventing from ever finishing the run.

looks fixed.


Pattern Matcher Tests

This is a reopen of #4744 because I'm confident that the closing comment was for another pr as this pr only dealt with testing the existing pattern matcher functionality. There was no line getbacks or functionality changes. Once this pr gets merged to master, I can be able to prove that #4670 introduces no functional changes.

i don't like this, it's pretty ugly and i will be sad to maintain this. like what is test_max_214 and what's this formatting?


Sprockets::FileNotFound in Spree::Admin::GeneralSettings#edit

Hi, @manuca! Thanks for the module :smiley_cat:
I'm new to Spree (and Rails) so this is likely my fault, but here it goes.
This is what my Gemfile looks like.
gem 'spree_mercado_pago', github: 'manuca/spree_mercado_pago'#, branch: '2-4-stable'

And I'm getting the following error when trying to configure the module.

Ideas?
Thanks!

@skqr thanks. Did you do?
bundle exec rails g spree_mercado_pago:install
I'm currently taking some days out for holidays but if this doesn't solve it promise I'll get back to this as soon as I'm back to work.

Hi, @manuca. Thanks for the swift reply!
Here's my whole command log, it includes the VM boot and the Rails server start.
➜  spree_vagrant git:(master) ✗ vagrant up
Bringing machine 'default' up with 'virtualbox' provider...
==> default: Checking if box 'chef/ubuntu-14.04' is up to date...
==> default: Clearing any previously set forwarded ports...
==> default: Clearing any previously set network interfaces...
==> default: Preparing network interfaces based on configuration...
    default: Adapter 1: nat
==> default: Forwarding ports...
    default: 3000 => 3001 (adapter 1)
    default: 22 => 2222 (adapter 1)
==> default: Booting VM...
==> default: Waiting for machine to boot. This may take a few minutes...
    default: SSH address: <IP_ADDRESS>:2222
    default: SSH username: vagrant
    default: SSH auth method: private key
    default: Warning: Connection timeout. Retrying...
    default: Warning: Connection timeout. Retrying...
    default: Warning: Remote connection disconnect. Retrying...
==> default: Machine booted and ready!
==> default: Checking for guest additions in VM...
==> default: Mounting shared folders...
    default: /vagrant => /home/skqr/Projects/SKQR/spree_vagrant
==> default: Machine already provisioned. Run `vagrant provision` or use the `--provision`
==> default: to force provisioning. Provisioners marked to run always will still run.

➜  spree_vagrant git:(master) ✗ vagrant ssh
Welcome to Ubuntu 14.04.1 LTS (GNU/Linux 3.13.0-24-generic x86_64)

 * Documentation:  https://help.ubuntu.com/
Last login: Tue Jan  6 21:49:02 2015 from <IP_ADDRESS>

Here's the command running.
vagrant@vagrant:~$ cd /vagrant/spree/
vagrant@vagrant:/vagrant/spree$ bundle exec rails g spree_mercado_pago:install
[WARNING] You are not setting Devise.secret_key within your application!
You must set this in config/initializers/devise.rb. Here's an example:

Devise.secret_key = "8d9b1f9a891b4e3eaae86456d21ed79e135040ecf150eaa93954b3a016eebe7073e8a6bc8574f4649f07fd298f4831ca4964"
      append  vendor/assets/javascripts/spree/frontend/all.js
      append  vendor/assets/javascripts/spree/backend/all.js
      insert  vendor/assets/stylesheets/spree/frontend/all.css
      insert  vendor/assets/stylesheets/spree/backend/all.css
         run  bundle exec rake railties:install:migrations FROM=spree_mercado_pago from "."
Would you like to run the migrations now? [Y/n] y
         run  bundle exec rake db:migrate from "."
[WARNING] You are not setting Devise.secret_key within your application!
You must set this in config/initializers/devise.rb. Here's an example:

Devise.secret_key = "ccb763bcb637864c11bf29161d5ebeaf3f9e00e129a597482ff16dc240d1839edb67f8e50a8df410442e5a2db4fb828160de"

vagrant@vagrant:/vagrant/spree$ rails server
=> Booting WEBrick
=> Rails 4.1.8 application starting in development on http://<IP_ADDRESS>:3000
=> Run `rails server -h` for more startup options
=> Notice: server is listening on all interfaces (<IP_ADDRESS>). Consider using <IP_ADDRESS> (--binding option)
=> Ctrl-C to shutdown server
[WARNING] You are not setting Devise.secret_key within your application!
You must set this in config/initializers/devise.rb. Here's an example:

Devise.secret_key = "32eb6959e933ac3480b9ba9b4d912c8519e6d8c663199eb63a35271a8d3421a437973365cad77d6bf67223259a3cb6fc6559"
[2015-01-10 20:49:25] INFO  WEBrick 1.3.1
[2015-01-10 20:49:25] INFO  ruby 2.1.5 (2014-11-13) [x86_64-linux]
[2015-01-10 20:49:25] INFO  WEBrick::HTTPServer#start: pid=1596 port=3000

The server appears to start properly, but I still get the same error. Am I missing something that might generate the missing asset file?
Thanks again!

I get the same error...
I already run:
bundle exec rails g spree_mercado_pago:install

Hi Guys, Im getting the same error.
Problem: this line
 *= require spree/frontend/spree_mercado_pago

From frontend/backend all.css and all.js
Also I detected after bundle exec rails g spree_mercado_pago:install and db:migrate there is no migration to run.
I checked at psql, table mercado_pago_notifications do not exist. When I tried to test IPN, POST request to /mercado_pago/ipn said that the table is missing.
I think there´s some problem with the installation.
Im using spree 2-3-stable.
Thanks!

Guys sorry for the delay, just took a general look at this issue. It seems the paths generated on installation for the assets inside the gem are messed up. It would be great if you could try a branch I opened to fix this issue.
https://github.com/manuca/spree_mercado_pago/tree/fix-issue-4
Could you all try this and tell me if it solves the issue?
Thanks

https://github.com/manuca/spree_mercado_pago/tree/fix-issue-4 fixed the issues with css/js files! Thanks a lot.


Implementation in ES6

Hi,
I'm trying to use your library in a ES6 project. But I can't seem to get it to work. I'm not sure how and where dc-addons is able to modify the dc object before I try to use it. Can you give me a pointer on how to import the library correctly and then use it, in an ES6 class?

@akuji1993 I use this library in a jspm ES6 project successfully. I had to configure the libraries to load using the global format - https://github.com/jspm/registry/wiki/Configuring-Packages-for-jspm#package-module-format.
Are you using a bundler like jspm or webpack?

Thanks, I'll check out your link and will tell you if it worked.
I'm using webpack for this project.

Update on the webpack situation. I can't get it to work in any way. Do you have anything you can point me towards @Tim-Intellipharm ? Otherwise I pretty much have to force it to work by forking and customizing this.

In case somebody finds this and wants at least SOME pointers. I tried to do the following:

Copy the entire library into my project, surrounding the functions in dc-addon.js with

export const extendDc = (dc) => {...}

Since that is not sufficient (some variables are missing), I'm using (https://www.npmjs.com/package/extend) extend to add them, before defining extendDc(), like this:
 import L from 'leaflet'; import extend from 'extend'; import LMarkerCluster from 'leaflet.markercluster';

If you do those two steps, the library SEEMS to work, but throws the above error with lat of null.

If any changes are needed to dc.js to better support dc-addons, please let me know.
Our UMD support is in this file: https://github.com/dc-js/dc.js/blob/develop/src/footer.js
One of the (many) weird things about JS modules is that the webpack module loader seems to go through the AMD codepath by default (probably the global format is a workaround for this). (See dc-js/dc.js#1214 & dc-js/dc.js#1261)
D3 itself has had problems being a module yet letting others add to its namespace; this went back and forth in v3 and I think in v4 Mike just settled on Rollup and decided users of other module loaders were on their own. When I finally upgrade to d3v4 I intend to follow the same pattern.

@akuji1993 Would it be possible for you to share some of your code or create a plunkr replicating the issue? Or even the stack trace from the error? This problem may even be coming from the way you are using the dc-addons library.

Just wanted to let you know, I haven't forgotten about this, just chasing some deadlines right now. I'll get around to this and share some hopefully useful code example with you guys.

Does anyone have working code for getting dc-addons working in environments like webpack?

Hey,
I have a solution that is super easy.  It might not be as "clean" as you want but it works.
ES6 DC.js leaflet  ,
ES6 DC.js
I hope these example help you.
Or if anyone has any way to make them better please let me know.
Cheers!


Nest Settings tab under main MediaConch tab

@verwinv could you provide more information here. In what location of the UI are you referencing? What is the reason for the proposal?

Hi,
I suppose it is the one in the navigation bar.
This settings will be merged in the graphical part (see #263 for more information), one day.


Entity/Outcome should be selected after creation

If an entity/outcome is already selected,
and you create a new entity/outcome
the new entity/outcome is created
BUT the previously selected entity/outcome remains selected
The selection should be changed to the newly created object.

Implemented d2e791a0ee7b59e014d4fafc456a2866de039f10


:seedling: add stack info to osv-scanner error

What kind of change does this PR introduce?
feature

[x] PR title follows the guidelines defined in our pull request documentation

What is the current behavior?
only print a error string for osv-scanner panic error
What is the new behavior (if this is a feature change)?**
also print the stacktrace

[ ] Tests for the changes have been added (for bug fixes/features)

Which issue(s) this PR fixes
Updates #4171
Special notes for your reviewer
Does this PR introduce a user-facing change?
NONE

One e2e test failed, reaching Github speed limit, not related with this patch.

Thank you!

I'm wondering when will we got a new release.

I'm wondering when will we got a new release.

For our CLI and action, we are targeting end of the month for the full, non-release candidate v5. This code is already deployed in our infrastructure though.


What lock files we should keep?

Hello everyone,
Some repos currently have yarn.lock, others have package-lock.json. Should we keep both of them in order to support npm and Yarn at the same time?
Thanks

@racse1 no, we should stick to one or the other.

I noticed this as well. I think it'd be simpler to stick with package-lock.json—there's no reference in the docs of using Yarn to install.

@frekyll I agree. Even though Yarn is pretty popular, it's better to stick to the official package manager in my opinion.

Hi @simonsmith @giuseppeg, I'd like to hear from you guys what you think, so we're completely on the same page 😄
Again, I'd personally go with npm (it's not that slow nowadays), but nevertheless I'm looking forward to hear form you and then I'll need to prepare a few pull requests in order to close this one.

Seeing as we use npm in all of the scripts we should probably settle on package-lock.json

is it problematic to have both?

is it problematic to have both?

I would say so. Unnecessary and confusing to the user. I agree we should stick with package-lock and consider yarn if do a big migration.

@racse1 Is it causing any issues at the moment?

@simonsmith I don't think so 😄 Just a bit inconvenient, I guess.
I thought it'd be nice to finally close this one either by switching to package-lock.json everywhere or by just closing the issue as it'll be fixed with monorepo approach anyway...

Closing this one as it seems to be fixed now 👍


Fix isDrive

I was looking at "TODO" comments that could be eliminated and noticed the
one preceding the sole use of isDrive(). Which caused me to notice its
implementation is broken. The s[1] < 'z' shnould be s[0] < 'z'
Rather than make the mechanical fix simplify the implementation by using
unicode.IsLetter. That will probably be marginally slower but this
function should never appear in a hot execution path.

FYI: I considered adding a dependency on https://github.com/scott-ainsworth/go-ascii/ to minimize the run-time overhead. I decided that, for Elvish, the cost of that dependency was greater than the slower builtin unicode package.


Fix python version requirement

Fixes version requirement as != 3 will never get called unless python 2 or 4 etc is used.

Took me so long lmao


tag update

I'm submitting a ...
[ x] bug report
[ ] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Summary

Other information (e.g. detailed explanation, stack traces, related issues, suggestions how to fix, links for us to have context, eg. StackOverflow, personal fork, etc.)

Could you please merge the tag update request ?
https://github.com/solana-labs/token-list/pull/7209
update tag #7209

merged


Implement Total size feature

Total size for directories in long format is implemented with this PR for unix. For non-unix environments, there is no exact equivalent for what GNU displays so I've left it unimplemented at the moment instead of printing some other value.
Partially fixes #1872

Fails on Windows

failures:

---- test_ls::test_ls_long_total_size stdout ----
current_directory_resolved:
touch: C:\Users\RUNNER~1\AppData\Local\Temp\.tmpRgTsbE\test-long
open(append): C:\Users\RUNNER~1\AppData\Local\Temp\.tmpRgTsbE\test-long
touch: C:\Users\RUNNER~1\AppData\Local\Temp\.tmpRgTsbE\test-long2
open(append): C:\Users\RUNNER~1\AppData\Local\Temp\.tmpRgTsbE\test-long2
run: D:\a\coreutils\coreutils\target\x86_64-pc-windows-msvc\debug\coreutils.exe ls -l
thread 'test_ls::test_ls_long_total_size' panicked at 'assertion failed: self.stdout_str().contains(cmp.as_ref())', tests\common\util.rs:298:9

@sylvestre Thanks - I'd missed to disable the test for windows.
Current PR only implements this for unix - I don't see a straight-forward way to have this on windows (we could print file size directly but I'd like to hear some opinions on whether we want to go that way)
I also see MSRV failures - I'll do another update to fix those

I've updated to use md.len for windows and added corresponding test.
I've also renamed some functions to more appropriate names.


Playwright support?

Is your feature request related to a problem? Please describe.
Just wondering if Playwright is more stable than selenium? https://github.com/microsoft/playwright
Describe the solution you'd like
Maybe be able to switch from Selenium to Playwright with a config setting.
Describe alternatives you've considered
N/A
Additional context
N/A

https://twitter.com/lefthandedgoat/status/1458281358208475137

ping

Hi, my mind keeps wandering around the idea.
It looks loke introduction to F# DSL
https://www.codeproject.com/Articles/39031/Making-DSLs-in-F
for the newbies like me.
Canopy is way too good to start another scraping project with the plain Playwright. I just can't force myself to load these Node s.t into VScode and start from scratch.


Added : Pattern 37 JAVA

Java pattern 37
Closes: #60

@LAKSHIBRO closing thir pr as you have made 2 prs for the same file


"Ambiguous use of 'createPairs'"

In Requests.swift, I am getting this error: "Ambiguous use of 'createPairs'"
It seems to be confused about the public functions createPairs(_ key: String) -> Array<HTTPPair> inside the dictionary and array extensions.
Any ideas what could fix this? Thanks!

Check your Array

Which array? This error occurs inside the requests file that I pulled from github. I didn't do anything with it yet

We need some more information here. Maybe the code that you are having a problem with. Are you implementing the HTTPParameterProtocol to doing custom?


HTML in data-description

Is there any way to use HTML tags in the data description?
I thought about an additional option like description-html -> true / false

Any updates on this?


SFM to ray

Using a single kernel takes ages to compile and I'm not sure there is much performance gain, as well the sharding machinery is getting stuck I think. Using >500GB for sharded predict over T/C despite using scan_dims={'T', 'C'}. I think this can be simplified and the expense of breaking up the computation into streaming actor layers, rather than a single JAX kernel.
The transition is relatively simple. For each SFM core step we refactor the get_state to provide the initial state, then for each layer we spin up N[layer] idenical actors which feed off a single queue, which uses object refs to pass data. The __reduce__ function is necessary for pytrees to pass efficiently.
Thus as we execute the DAG each step has a supervisor actor that is responsible for dishing out work. Serial steps, e.g. simulating dish, can have single worker actors responsible for the work. Thus, the size of each layer is controllable. Only horizontal layers will take advantage of scale.

Currently, performance for a single solint (6s, 40chan) without bright source prediction is :
Total run time: 1203.54s
Initialisation time: 610.28s
Compilation time: 59.41s
Run time: 530.97s

roughly 100x slower than real time. I think we can and must do better. We require real-time. Some part of this must be related to non-optimal JAX compilation. Combine with isoloated performance tests to see what we should be getting.


A11y for List component

The List component should

use semantically correct elements
support keyboard navigation
accessible by AT

Resolved by using React Aria GridList: a3fa465ff12b0cd8fc505a0d040229a66344a632


Replace condp with case on .ordinal

Should be a bit faster (seems like 3%).

I wonder why *warn-on-reflection* is set to false in project.clj as I wondered if this introduced any reflections (which aren't good for performance and also break GraalVM native-image compilation).

I had set *warn-on-reflection* to true and this does not introduce any reflections (not even at macroexpansion time, which might not even matter).

@nilern Thanks for checking :)
Have you considered just writing a case + (.getName (class ...)) and dispatching on strings? Imo that would make the code more understandable/hacky with perhaps the same per boost.

Have you considered just writing a case + (.getName (class ...)) and dispatching on strings? Imo that would make the code more understandable/less hacky with perhaps the same perf boost.

That sounds more hacky to me and surely it is fastest to dispatch on ints.

@nilern Fair point. I had trouble parsing what was going on in your macro, but I expanded it using some test examples locally. Clever.

I wish we could just switch on enums directly, like Java. But I was determined to find a way.

Interestingly, this is slightly slower for benchmarks for me with the merging of #174 (and merging master in to use the latest Jackson dependencies). Is it slower for you with these changes as well?

Sounds good, I'll close this for now, we can always re-open in the future if needed!


horizon_token not saved to localstorage

Server version: 2.0.0
Client version: 2.0.0
const horizon = Horizon({authType: 'token'});
    if (!horizon.hasAuthToken()) {
      horizon.authEndpoint('google').subscribe((endpoint) => {
        window.location.replace(endpoint);
      });
    } else {
      console.log(1);
    }

Browser redirect with "horizon_token" in query param and a new user inserted to users table, However, token is not save to localStorage.
Where did I do wrong?

just figured it out
In the redirection page, I have to instantiate a horizon client to fire the following, which is in the horizon client
const tokenStorage = new TokenStorage({ authType, path })
tokenStorage.setAuthFromQueryParams()


url_for appends ? if called with empty query argument

Expected Behavior
If url_for is called with a query argument that is an empty list (and no other query arguments), the URL should have no query part at all, including no trailing question mark. (When called with a nonempty list, it adds several query arguments with the same name, which is what I expect; when called with None, it omits the question mark already.)
@app.route('/')
def index():
    return flask.url_for('index', extra_param=[])

Full example: https://github.com/lucaswerkmeister/repro-flask-url_for-empty
Actual Behavior
The URL has a question mark at the end (in the above example: /?).
Environment

Python version: 3.9.1
Flask version: 1.1.2
Werkzeug version: 1.0.1

Workaround
Replace the empty list with None by appending or None to the argument. (This also works in Jinja2 templates.)

Went back to 0.14 and this didn't happen there. git bisect points to #1281, which introduced the new code generator for URL building.

Went back to 0.14 and this didn't happen there. git bisect points to #1281, which introduced the new code generator for URL building.