Icons are not aligned on MacOS.

The Setting Icon is not aligned with other icons in the right.
Idea Version: 2023.2.3
MacOS Version: MacOS Senoma
Other third-party themes has same issus. Not knowing wether this was a compatibility issue with lates idea version. Official Thems has no such problem.

Yes. This is a compatibility issue with new UI.
I have found the workaround and it will fix in the next release.
Please install EAP version here and check if this issue is solved:
https://plugins.jetbrains.com/plugin/18922-gerry-themes/versions/eap
https://plugins.jetbrains.com/plugin/19668-gerry-themes-pro/versions/eap

solved! 2023.1.72 EAP version works perfectly fine now.
Thank you.


Add Checks for Version 1.1.01850.1

Release Notes:

Changed the Azure endpoint to increase resiliency.
The NPS server must be able to communicate with the following URLs over port 443:
strongauthenticationservice.auth.microsoft.com (new)
adnotifications.windowsazure.com
Fixed an issue where event logs showed a username with incorrect encoding.
Radius values are now read as UTF-8 instead of attempting to detect encoding.
The registry entry USERNAME_ENCODING can be set to "Detect" to use the previous encoding detection behavior.
Improved cache management.
This reduces occurence of high CPU utilization in busy or long running systems.
Improved LDAP search performance for systems that have several forests specified in the registry entry LDAP_LOOKUP_FORESTS.
Support for Windows Server 2022.
Upgraded to Visual C++ redistributable for Visual Studio 2017.

Add Check for the following URL:

strongauthenticationservice.auth.microsoft.com (new)

I have been unable to download this latest version. The previous version is failing for NPS health check version status. I downloaded the latest version, but it still showing the previous version no. Please I need assiatnce.

@weyCC81 thank you! @Lawike2 please retry should be fixed now.


File System fixes for Windows and proper schema pickup for several passed class files

I tried to merge the 2 commits.

Hi @koh-osug,
I've done it for you.
Thanks a lot !

OK, great.


Multi-Root: Configure projects with Open Folder without a workspace (automatically)

Currently as I see it to use the Multi-Root feature. I have to use the option "Add Folder to Workspace" and have a workspace, which needs to be documented/explained.
I'd like to be just able to point to a root folder containing sub folders with independent CMakeList.txt projects and the extension to find them and configure them automatically without the hassle to create a workspace and add the folders one by one.
MyProjects/
     MyProject01/CMakeLists.txt
     MyProject02/CMakeLists.txt
     MyProject03/CMakeLists.txt

In that context "cmake.sourceDirectory" could point to an array of root CMakeLists.txt files

TLDR: +1 array of strings for sourceDirectory
This would also help my situation where I have sibling folders each with their own projects as in
Application/                 — Application project directory
  + Makefile / CMakeLists.txt    - Makefiles of the application project
  + build/
   - components/          — Components of the application project
     + testable/
  + main/                       — Main source files of the application project
  - test/                         — Test project directory
     + build/
     + Makefile / CMakeLists.txt    - Makefiles of the test project

Which is typical of how unit testing is done with espressif eps32
As a workaround I have "re-added" the test folder to create a workspace so that CMakeTools sees the separate $workspaceFolder's and allows me to build the two projects given their independent root CMakeLists.
While it's confusing to have the same folder represented twice (test folder as a root folder and test folder in the main application's root folder) -- it's worth it to get rid of the distracting red squiggles!


Update README to point to nio as an alternative library

Signed-off-by: Adam Beckmeyer<EMAIL_ADDRESS>

Looks good, wondering if we could make it even more prominent?

Good idea. I'll put it at the top of that section and use more forceful language. Something like "We strongly recommend using matrix-nio rather than this project".

Better?


Feature Alert: USB monitoring

Check whether it's possible in a sort-off real-time way.

Also for Overview

Probably better to include this in feature alert instead of a stand-alone function.


An internal error occurs STREAMA

Issue description
Hi,
I putted my local video files in the settings. When I want to add a movie from that file, I click on "Local file" but I can't select any files while there are .avi or .mp4
There is only one that I can choose (which is .mp4) but when I want to play it there is an error message which is :  "There seems to be a problem adding the video-file to the player. This is most likely due to a codec-problem. Try converting it to a compatible HTML5 codec, remove the currently attached file and re-add it. If the codecs are fine, check the error log of the server and the base URL in the settings."
So, I tried to just "Upload" them by dragging them in the dedicated menu but then a pop-up windows appears saying " an internal server occured" "File upload failed. Please close this pop-up and try again".
I am on Mac under Catalina 10.15.4
Thank you

Hi elaniee,
Can you please let me know if you are using the latest version of Streama v1.8.3 and some more information about your environment:
Streama Version:
Web browser:
And do you have multiple .mp4 files that you have tested this with?
Can you try to replicate this on another browser?
Also .avi videos will not work as they are not supported by the HTML5 codec you can convert them using the guide in the FAQ
Can you also test an upload via the via the pop out window rather than the drop window?
Please can you also submit a screenshot of any errors you encounter while troubleshooting this.
Many Thanks,

Also, please let us know if you're using any network shares or Virtualization/Containers.  I've had this issue as a result of these types of things!

same isue here. An internal error occurs . cant change logo  or ad movies
v1.9.1
firefox

Make sure that you have set up the upload directory correctly, this happened to me before.

I even can't upload movies neither upload logo as well, using Ubuntu 16 browser is firefox and in windows it's Chrome


chore: remove unload event from OTLPExporterBrowserBase

As discussed in https://github.com/open-telemetry/opentelemetry-js/pull/4326 we could remove any concept of a shutdown event and allow consuming code to handle that explicitly.
Which problem is this PR solving?
The 'unload' event is stopping any site using OTEL in JavaScript from taking advantage of back/forward cache.  And also Google is planning to deprecate unload.
Fixes # (issue)
Short description of the changes
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A

Checklist:

[ ] Followed the style guidelines of this project
[ ] Unit tests have been added
[ ] Documentation has been updated

I don't think we should just be removing the "unload" event. I understanding the ask but there is a bunch of nuances that are bing missed here.

This is (currently) specific to chromium browsers
Removing will cause any extension classes to have to either "listen" to the unload events ("unload", "beforeunload", "pagehide", "visibility") to ensure that any "batched" events are flushed.
This class is not listening to all possible "unload" events and the "unload" event is not always called either (we should at the very least have "pagehide" (for IOS) and "visibilitychange" (all other modern). While "unload" and "beforeunload" are required for "older" runtimes that don't support the other 2.

Rather than "remove", this "should" be configuration driven and we should include the others.
Happy to consider that the "default" is false (to not include by default), or to have code that "checks" if the runtime is Chromium based with version > the say 100 (or something -- not sure when it occurred).
And while it's being "deprecated" / "removed" the code should "handle" (identify) when the event was not able to be hooked (because the addEventListener returns false).
Simply, just Removing this should also be considered a breaking change because of the downstream functionality change.

I don't think we should just be removing the "unload" event. I understanding the ask but there is a bunch of nuances that are bing missed here.

I guess I'm confused, what was being proposed in your comment here? https://github.com/open-telemetry/opentelemetry-js/pull/4326#issuecomment-1864984629

I don't think we should just be removing the "unload" event. I understanding the ask but there is a bunch of nuances that are bing missed here.

I guess I'm confused, what was being proposed in your comment here? #4326 (comment)

This PR just deleting the code is not how this should be working. It should be behind a flag about whether to add / remove the unload events. And preferablly, that flag should be an array of the unload events (as listed above).

This PR just deleting the code is not how this should be working. It should be behind a flag about whether to add / remove the unload events. And preferablly, that flag should be an array of the unload events (as listed above).

Okay thanks for the further explanation, do you mind describing or pointing to an existing example of how flag is used in this repository?   Greping for flag I don't really see anything that is obvious to me how it would be used ... are you just talking about adding another property on experimental/packages/otlp-exporter-base/src/OTLPExporterBase.ts?

And while it's being "deprecated" / "removed" the code should "handle" (identify) when the event was not able to be hooked (because the addEventListener returns false).

I'm having a very difficult time trying to figure out what you're asking for ... can you tell me more about the addEventListener returning false?  In the MDN docs it says it just returns undefined and the lib dom types declaration says it's void return type:
declare function addEventListener(type: string, listener: EventListenerOrEventListenerObject, options?: boolean | AddEventListenerOptions): void;

I've updated the original PR to take an array as an optional argument -- otherwise it should perform just as it did before.

@eldavojohn we discussed this in the SIG today and concluded that we should be fine just removing the functionality, but we want to list it as a breaking change. The possible break is this:

A user extends the exporter and overrides the shutdown function, and does something which is usually called by the unload listener
We remove the unload event listener
That user's overridden shutdown function no longer gets called

Please let me know if you have questions. Can probably close #4326

Hey @dyladan thanks ... um, I keep reading "removing the functionality" and I'm just absolutely confused.  In the PR I closed, I removed the functionality.  That seemed to be very wrong.  So in this PR I allow the property to be set.  Which also seems to be wrong.  I understand the breaking change from "removing the functionality" but I am absolutely struggling with what that means if it doesn't mean deleting the event listener that is listening for the unload event.  I'm sorry becuase these seem like very atomic concepts and I'm unable to grasp them and I thank you for your time and patience!

I think you mixed up the PRs? This one doesn't allow any property to be set. This PR is the correct solution based on what we discussed in the meeting today.

I think you mixed up the PRs? This one doesn't allow any property to be set. This PR is the correct solution based on what we discussed in the meeting today.

Sorry I definitely did, I thought this PR was horrendously wrong as explained by @MSNev .  Please let me know if there's anything else I can do or any changes to this code.  Thanks again!

I think you mixed up the PRs? This one doesn't allow any property to be set. This PR is the correct solution based on what we discussed in the meeting today.

Sorry I definitely did, I thought this PR was horrendously wrong as explained by @MSNev . Please let me know if there's anything else I can do or any changes to this code. Thanks again!

We discussed this with @MSNev today and he is now convinced this is the correct approach


Errors while installing moveit2 from source

When I tried to build the master_compile branch from source using the README instructions, I am seeing a number of issues.

The distinction between the master branch and master_compile branch is unclear. Before announcing the progress, it is very important that users are able to reliably work of a master branch

running rosdep install -y --from-paths src --ignore-src --rosdistro ${ROS_DISTRO} to ensure I have the correct dependencies fails with:

ERROR: the following packages/stacks could not have their rosdep keys resolved
to system dependencies:
moveit_planners_ompl: Cannot locate rosdep definition for [ompl]

This causes the build to fail with

--- stderr: moveit_planners_ompl
CMake Warning at /home/mike/ws_acutronics/install/share/moveit_core/cmake/ament_cmake_export_libraries-extras.cmake:116 (message):
  Package 'moveit_core' exports library 'moveit_test_utils' which couldn't be
  found
Call Stack (most recent call first):
  /home/mike/ws_acutronics/install/share/moveit_core/cmake/moveit_coreConfig.cmake:38 (include)
  CMakeLists.txt:13 (find_package)

CMake Error at CMakeLists.txt:18 (find_package):
  By not providing "Findompl.cmake" in CMAKE_MODULE_PATH this project has
  asked CMake to find a package configuration file provided by "ompl", but
  CMake did not find one.

  Could not find a package configuration file provided by "ompl" with any of
  the following names:

    omplConfig.cmake
    ompl-config.cmake

  Add the installation prefix of "ompl" to CMAKE_PREFIX_PATH or set
  "ompl_DIR" to a directory containing one of the above files.  If "ompl"
  provides a separate development package or SDK, be sure it has been
  installed.

moveit_planners_ompl fails to build because it does not include moveit_msgs correctly

The distinction between the master branch and master_compile branch is unclear. Before announcing the progress, it is very important that users are able to reliably work of a master branch

+1, we'll update the README.md to clarify this point.

running rosdep install -y --from-paths src --ignore-src --rosdistro ${ROS_DISTRO} to ensure I have the correct dependencies fails with:

You're right this package is failing when you try to install using rosdep. You need to install libompl-dev using apt. We need to reflect this on the README.md as well.
We'll review the current status of the README.md to clarify all of this points. Thanks for your feedback!

Thanks for reporting @mlautman, @LanderU once fixed, can we refer in here to the commit that addresses the issues raised here please?
@mlautman, based on our experience, rosdep is discouraged at this point (@LanderU, correct me if I'm wrong). AFAIK, we've done a reasonable amount of work to try to fix deps that way but more effort is required. @mlautman feel free to jump in and help with rosdep if you consider this critical, I believe you guys have plenty of experience.

You're right this package is failing when you try to install using rosdep. You need to install libompl-dev using apt. We need to reflect this on the README.md as well.

based on our experience, rosdep is discouraged at this point

I don't understand this viewpoint. ROS2 uses rosdep extensively. Why wouldn't we use it here as well? Explicitly installing dependencies with apt is a short-term patch that doesn't actually achieve our goals for MoveIt 2. When we go to release a Debian package we will need to use rosdep.

AFAIK, we've done a reasonable amount of work to try to fix deps that way but more effort is required

What has been tried? In order to use rosdep with ompl, we need to add a Dashing track for ompl with bloom-release. This will add ompl to distribution.yaml which acts as a dictionary for rosdep.  I have ping'd Mark Moll to see if he can un-block us there.

I had this same issue. I've made some suggestions:
https://github.com/AcutronicRobotics/moveit2/pull/95

On second thought, if you are able to successfully build against libompl-dev we don't need to release ompl with bloom. I opened a PR to ros/distro that will enable us to install libompl-dev using the libompl-dev key in package.xml

@LanderU can we close this issue?


Need TelemetryConfiguration added to DI Services

Current guidance for utilizing custom telemetry w/ AppInsights is to get the TelemetryConfiguration object from the DI container and create a TelemetryClient instance from it. However, currently we aren't adding the config to the DI container.
I've worked around this locally by putting the following in Program.cs
public static void Main()
{
    var host = new HostBuilder()
        .ConfigureFunctionsWorkerDefaults()
        .ConfigureServices(b =>
        {
            var appInsightsInstrumentationKey = Environment.GetEnvironmentVariable(@"APPINSIGHTS_INSTRUMENTATIONKEY");
            if (!string.IsNullOrWhiteSpace(appInsightsInstrumentationKey))
            {
                b.AddSingleton(new TelemetryConfiguration(appInsightsInstrumentationKey));
            }
        })
        .Build();

    host.Run();
}

but we should probably bake this in so docs still flow

Related: #350 , #308

We used to use TelemetryClient injected automatically via DI in .NET 3.1 functions. Is there a particular reason that this doesn't propose adding that back to the DI container as well? Is it a fundamentally different thing? (We have just upgraded and are now creating it ourselves using the TelemtryConfiguration as above in each function but it's additional duplicated code per function)

We used to use TelemetryClient injected automatically via DI in .NET 3.1 functions. Is there a particular reason that this doesn't propose adding that back to the DI container as well? Is it a fundamentally different thing? (We have just upgraded and are now creating it ourselves using the TelemtryConfiguration as above in each function but it's additional duplicated code per function)

@stap123 in general for .net 3.1 you should be following this guidance. This issue is around that guidance not being possible in its current form due to the lack of TelemetryConfiguration in the DI container. You definitely shouldn't be re-creating TC every time a function launches.

Thank you for the clarification, we are doing that now anyway so all good 😃

Related to #619

We don't want the worker to be tied to any one logging solution so I don't think we'll ever do this directly in the host. Ideally, supporting something like AddApplicationInsightsTelemetryWorkerService(). This may "just work" well enough today -- we'll need to look through exactly what it does.
But we may have to introduce some similar API from a separate package that lets customers add this support to the worker (if the above API doesn't work as we'd like).

Totally fair as I know the ability to plumb in other logging/telemetry solutions has been an ask for a while.
That said, though, if this won't be supported we need to make sure our docs are steering people this way, so updates are needed there at a minimum.

This documentation for Isolated Process functions needs updating:
https://docs.microsoft.com/en-us/azure/azure-functions/functions-dotnet-dependency-injection#logging-services
"Don't register your own TelemetryConfiguration or TelemetryClient if you are using the built-in Application Insights functionality. If you need to configure your own TelemetryClient instance, create one via the injected TelemetryConfiguration as shown in Log custom telemetry in C# functions."

I'm going to close this out as we've got a solution for Application Insights in the worker now (currently in preview) that minimally wraps the ApplicationInisghtsWorkerServiceTelemetry. All configuration, etc, should be exactly the same as any other console application using this package.


Deleting account on the website

Under GDPR, I have exclusive rights to have my account removed from your grotesque platform.
Since you guys love keeping a dead website "alive" how about you provide a support method to accomplish this?
Failure to do this will assume liability under the General Data Protection Act.

Under GDPR, I have exclusive rights to have my account removed from your grotesque platform.
Since you guys love keeping a dead website "alive" how about you provide a support method to accomplish this?
Failure to do this will assume liability under the General Data Protection Act.

cc @ton31337

I thought the same.  Can be  very expensive, when someone report u. U Sell not existing  Subscription and noone can delete account

cc @fordnox

cc @fordnox


fix setup script

ADT extension has been published to the azure-iot public extension.
Follow these steps before you run the setup.ps1

az extension update --name azure-iot

Do you need to update the readme too then?


[EPIC] Change home page to show different spaces

Is your feature request related to a problem? Please describe.
The current navigation in the home page can lead to more clicks than what's necessary. This needs to be improved.
Describe the solution you'd like

[x] #866
[x] #867
[x] #868
[x] #869
[x] #870

Additional context
We could use https://getbootstrap.com/docs/4.0/components/card/ in a future task to make the home page look better. Thanks.

Completed. Closing. Thanks.


Let me configure whether to autocomplete single quotes in comments

VSCode Version: 1.2.1
OS Version: Windows 8.1

Steps to Reproduce:

Create a Python file (really, any language will do, I just happened to be using Python today). Add a commented-out line.
Type the word don't. Two apostrophes are inserted instead of one.

When I'm writing a comment, I'm more often using a single quote as an apostrophe in a word,
rather than writing a string or character literal. If I really want a pair of them in a comment, I'm OK typing two of them. I should be able to configure whether to auto-complete single quotes in comments.

Some other places where single quote doubling should be configurable, as the user's intent is probably to have just one single quote / apostrophe character:

Inside strings, where contractions like "don't" are far more likely than strings like "Unknown identifier '%s' on line %d". If I need two single quotes in a string, I'm quite happy to type them both.
Generic type parameters in F#, where Array<'T> is an array containing items of type 'T. The leading single quote means "This is a generic type parameter" in F#. So in F#, if I've typed an angle bracket, I would prefer for single quotes not to be doubled.

The latter example is specific to the language: when writing F#, I'd like to be able to turn off single-quote-doubling inside angle brackets, but when writing other languages I'll want to keep single-quote-doubling inside angle brackets.
So I suggest making this behavior configurable by context: the configuration can list a series of contexts where the doubling should be turned off. Maybe the contexts can be names of syntax-highlighting regions, like "comment", or "fsharp generic-type-parameter". (The generic-type-parameter syntax-highlighting region doesn't exist right now in the F# syntax highlighter, but it probably will if this feature request gets implemented). Also, the latter suggestion is an "and", not an "or": it should be converted to a CSS selector like .fsharp.generic-type-parameter.

I can confirm that this behavior is still occurring in 1.4.0 on Windows 7 Enterprise SP1 x64. In my case it's noticeable within JavaScript comments

I also see this behaviour and agree that it's annoying.  I am using v1.4.0 with OSX 10.11.5.
It's particularly annoying inside comments.  If I type a comment such as
// this comment shouldn't add an extra apostrophe.
in a JS file, I get an extra single quote at the end of the line that appeared when I typed "shouldn't".
However, even outside of comments, I agree with rmunn's suggestion that the extra single quote should never be automatically added inside strings.

Same here for JavaScript. Doesn't make sense to have it automatically insert a second one.

Version 1.6.1 (primarily JS).  I'm finding all kinds of comments that end in a single quote

// TODO: this doesn't clear'

PLEASE FIX THIS, I have the same problem in python, and it'ts really annoying!!!!!
I'm using v1.7.1 on windows 10

For consistency, it would be nice to be able to configure this for (, {, [ too. I note it doesn't happen for <.

Currently—in Python—typing comments or inside a string doesn't cause a second ' to be inserted.

VSCode Version: Code 1.8.1 (ee428b0eead68bf0fb99ab5fdc4439be227b6281, 2016-12-19T14:42:49.304Z)
OS Version: Darwin x64 15.6.0
Extensions:

Extension
Author
Version

Bookmarks
alefragnani
0.10.2

django-html
bibhasdn
1.2.0

python
donjayamanne
0.5.5

gitlens
eamodio
1.4.1

tslint
eg2
0.8.1

cpptools
ms-vscode
0.9.3

preview-vscode
searKing
1.1.1

vscode-react-native
vsmobile
0.2.5

I'm using VS Code with latest updates as for today 31st May 2017. I'm using PHP, and whenever I type ', I get '' instead.

This is configurable by language extension. I don't see this issue in Python and JavaScript anymore and fixed it for PHP. Let me know if you still see any problem.


document a practical example

https://gist.github.com/mattvonrocketstein/580e04631af195edae0c9157f47758fe

Implemented an example app in b2f09ca95. Feel free to implement another example(s) under /examples folder and open a pull-request. I believe having them packaged as mix apps make 'em runnable from the box and easier to clone/reuse. Closing the issue for now.


Electron >=v4 "makeSingleInstance is not a function", replaced in Electron v4

https://github.com/wojtkowiak/meteor-desktop/blob/5aea4ccc6983c52e0196a01c38c9bc3a591e502d/skeleton/app.js#L131
If you have single instance enable the app.makeSingleInstance() function is used but it was replaced in Electron v4, see release notes https://www.electronjs.org/blog/electron-4-0. app.requestSingleInstanceLock() should be used instead

In the meantime anyone who stumbles on this can fix this yourself by just setting "singleInstance" to false in .desktop/settings.json, then adding this in your .desktop/desktop.js constructor() method:
    // ensure only a single instance of the app can be run
    const gotTheLock = app.requestSingleInstanceLock();
    if (!gotTheLock) {
      return app.quit();
    }


[Hub Generated] Review request for Microsoft.CognitiveServices to add version 2017-04-18

If you are a MSFT employee you can view your work branch via this link.
Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Can one of the admins verify this patch?

Automation for azure-sdk-for-python

Automation for azure-sdk-for-js

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-java

Automation for azure-sdk-for-node

Automation for azure-sdk-for-go

@anuchandy Please disregard the default failures here

Automation for azure-sdk-for-net
A PR has been created for you:
https://github.com/azure/azure-sdk-for-net/pull/5493
.NET SDK Commits:
https://github.com/adxsdknet/azure-sdk-for-net/pull/5493/commits/d0ff9dae223b90ba0cdaac44a5e0a67d26b96a28

@DavidLiCIG could you please update the PR as we discussed?

Closing due to inactivity. @DavidLiCIG please open a new PR when you're ready. thanks!


Clean up Dashboard > System and Settings > SEO & Statistics section

Fix #8562
URLs and Redirection

I reorganized a bit the order of the checkboxes, and simplified the handling of the pretty URLs in the controller.

concrete5 v8
concrete5 v9

Bulk SEO Updater

I split the search form and the result table in two tabs (so that when you are updating the SEO stuff in paginated results you don't have to scroll after the search form)
I greatly simplified the document structure (thanks to Vue)
I removed all those custom classes in flavor of the bootstrap default ones
The page ID was visible as a plain title attribute of the title and description input boxes (quite hidden and in the wrong place IMHO). I added it to the page summary (the gray box on the left)
It was not possible to filter pages that don't have a default description (CollectionVersions.cvDescription database field): I added it (it's the new No Default Description checckox); this, in addition to the existing No Meta Description checkbox, allows users to filter pages that don't have any description (neither the default one nor the meta one) - see #4278

concrete5 v8
concrete5 v9

Like before, it's usable in small devices

concrete5 v8
concrete5 v9

Tracking Codes
Nothing special changed here (except the usage of a monospace font, which makes the code more readable):

concrete5 v8
concrete5 v9

Excluded URL Word List

I fixed a bug when restoring the default word list (Urlify::$remove_list is an array of arrays, not a simple array)
I added a confirmation before restoring the default word list
I fixed the comparison of currently configured word list vs default word list (it was ugly and didn't work)

concrete5 v8
concrete5 v9

n/a

Search Index

I added a more good-looking confirmation dialog when users click the Clear Search Index button
I had to change some of the methods in the IndexedSearch class to static, since they are called statically (that is, without creating an instance of the class)

concrete5 v8
concrete5 v9

Statistics page
The Statistics page was removed 6 years ago in https://github.com/concrete5/concrete5/commit/3d9baa161c90c0a51ea4270626fdc40a4431a4b1 but we still had the controller and view files, so I removed them.

PS: this PR is not yet ready to be merged: the Bulk SEO Updater page needs Vue, but at the moment it's not available in dashboard pages.
At the moment there's this terrible hack in the concrete/single_pages/dashboard/system/seo/bulk.php file:
<script<EMAIL_ADDRESS>
I'm waiting for a solution from @KorvinSzanto.

PPS: I'm also waiting for #8645 for a proper way to associate the <label> elements to the related <input type="text">/<textarea> elements.

I rebased this pull request and added https://github.com/concrete5/concrete5/pull/8648/commits/44fab10d6d1ad1e94135a8e5ee4f3e09bab49e87?w=1: it's now ready for review/merge.

Really really nice work, as always.


Support Jammy

Support Jammy

Thanks for merging @robdimsdale ... I misclicked patch instead of minor so we'll have to fix that in post-production.


[ Feature Request ] Implement getters for Flow.Fog

Is your feature request related to a problem? Please describe.
Currently, you can only SET the fog colour and distance but not get it.
Describe the solution you'd like
In order to implement changing fog properties so that they are carried over when the game saves and loads: I need the Lua API to be extended to allow for this.
Describe alternatives you've considered
There are mentions of getting the fog properties in the code but its not finished as far as I can tell.
Additional context
n/a

you can get the values directly from the object.
Try this code:
local level = TEN.Flow.GetCurrentLevel()
print(tostring(level.fog.color))
print(tostring(level.fog.minDistance))
print(tostring(level.fog.maxDistance))

As david said, you can already do that with GetCurrentLevel(). For nodes, you can just have a variable with getting the level then implement a getter node which gets these values.


223 create animation component

Codecov Report

Merging #231 (dcb8d23) into master (619bd77) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #231   +/-   ##
=======================================
  Coverage   32.58%   32.58%
=======================================
  Files          98       98
  Lines        1903     1903
  Branches      935      935
=======================================
  Hits          620      620
  Misses        912      912
  Partials      371      371

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


File rename box was automated unfocus

VSCode Version: 1.52.1
OS Version: macOS 11.1

Steps to Reproduce:

Copy and then paste any file
Press enter to "rename" copied file
Wait for ~500ms you will see the rename box was automated unfocus

NOTE: This issue only occurs the first time the file was copied (within 500ms~1s). A workaround is waiting for 1s before start renaming the file.

Does this issue occur when all extensions are disabled?: Yes
Gif Image:

/duplicate and already fixed in Insiders, so the fix should arrive in Stable in a couple of weeks.

/duplicate and already fixed in Insiders, so the fix should arrive in Stable in a couple of weeks.


optimize service with listener port

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/test all

@wulianglongrd: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
1515f78e96eac50db20098f41e4fbb536c3f57e5
link
true
/test unit-tests-arm64

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

/test unit-tests-arm64

@wulianglongrd: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
1515f78e96eac50db20098f41e4fbb536c3f57e5
link
true
/test unit-tests-arm64

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


feat: increase code coverage

This PR increases the code coverage for the scripts, bringing it to 100%.

Summary by CodeRabbit

New Features

Added a function to ensure the existence of a directory before file operations.

Bug Fixes

Updated coverage reporting to exclude additional files from coverage metrics.

Tests

Introduced new test cases for enhanced error handling in file writing operations and directory creation.

/rtm


CSS minify breaks classes with exclamation mark or dollar sign

I've already seen in previous issues that the CSS optimizations are well-known for breaking CSS when special other special characters are being used.
But sadly, I noticed on my website (https://www.codebucket.de) that the classes which are responsible for column width are not correctly displaying the page content. As the class names contain dollar signs, I think that these are being ignored when doing the minify.
Here is an example of how the website should look like:

NGINX version: 1.12.1
ngx_pagespeed version: latest

I haven't seen $ being used in CSS identifiers.  Are you sure it's legal?  The (verbose) CSS3 description makes it look like it's a delimiter.
https://www.w3.org/TR/css-syntax-3/#consume-a-token0

I don't know, as I am not so much into CSS3, but apparently the theme author did use them 😕

Any updates on that?

Did you file a bug with the theme author about using legal CSS syntax?  Closing for now.


Add Network::to_core_arg() method

closes: #1207
Adds converting Network to its bitcoind equivalent.
The arguments for -chain can be found in the documentation and is one of the following:
main, test, signet, regtest

Thanks for your contribution!
error[E0277]: `[(&str, network::constants::Network); 4]` is not an iterator
   --> bitcoin/src/network/constants.rs:605:40
    |
605 |         for (expected_arg, network) in network_pairs {
    |                                        ^^^^^^^^^^^^^ borrow the array with `&` or call `.iter()` on it to iterate over it
    |
    = help: the trait `std::iter::Iterator` is not implemented for `[(&str, network::constants::Network); 4]`
    = note: arrays are not iterators, but slices like the following are: `&[1, 2, 3]`
    = note: required by `std::iter::IntoIterator::into_iter`

Oops, this won't compile with rustc 1.41, which is our MSRV. I think you just need to write network_pairs.iter() or something like that.

Thanks for the info. When I get back to my desk I'll fix this up and verify it works locally at MSRV

Thanks @connormullett, IMO we could just drop the unit test, the code is easy enough not to need it.

Thanks @connormullett, IMO we could just drop the unit test, the code is easy enough not to need it.

For sure, just trying to follow the contributing guidelines

For sure, just trying to follow the contributing guidelines

Thanks for trying man! One more request, can you squash this down into a single commit please.

Thanks for trying man! One more request, can you squash this down into a single commit please.

Sure

Might've messed something up trying to squash this. Never had to squash pre-merge. Normally I do it through github.

Looks like the squash worked, thanks! I am running my local tests and then will ack.

@connormullett could you rebase this on the current master? It looks like the recent merge of #1353 has caused a conflict.

Resolved conflicts and rebased

Remove newline from between tests


[build-script] Unify configuration params into one toolchain param

This shifts the paradigm of the build script from specifying every executable manually to building SwiftSyntax using an existing toolchain. This goes in sync with building SwiftSyntax using swift_build_support in the main Swift repository.

@swift-ci Please test and merge

Build failed
Swift Test OS X Platform
Git Sha - 94540ccacf6f89d737f729213006b6a38d59baa1

Let’s try again, now that the bot for SwiftSyntax PR testing has been updated.
@swift-ci Please test

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - 94540ccacf6f89d737f729213006b6a38d59baa1

https://github.com/apple/swift/pull/27934
@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - 94540ccacf6f89d737f729213006b6a38d59baa1

Re-triggering due to network outage.
https://github.com/apple/swift/pull/27934
@swift-ci Please test


Code for the "Writing an action server and client (C++)" tutorial does not compile

Hi!
The  "Writing an action server and client (C++)" tutorial gives a compile error with the constructor for the action server in the file action_tutorials_cpp/src/fibonacci_action_server.cpp. I got it to compile by using updated code from the following file demos/action_tutorials/action_tutorials_cpp/src/fibonacci_action_server.cpp instead of the embedded part which the tutorial suggest to copy into a local file. Specifically, the original part is this (line 21-32):
explicit FibonacciActionServer(const rclcpp::NodeOptions & options = rclcpp::NodeOptions())
  : Node("fibonacci_action_server", options)
  {
    using namespace std::placeholders;

    this->action_server_ = rclcpp_action::create_server<Fibonacci>(
      this,
      "fibonacci",
      std::bind(&FibonacciActionServer::handle_goal, this, _1, _2),
      std::bind(&FibonacciActionServer::handle_cancel, this, _1),
      std::bind(&FibonacciActionServer::handle_accepted, this, _1));
  }

And it should be updated into this (line 34-48):
explicit FibonacciActionServer(const rclcpp::NodeOptions & options = rclcpp::NodeOptions())
  : Node("fibonacci_action_server", options)
  {
    using namespace std::placeholders;

    this->action_server_ = rclcpp_action::create_server<Fibonacci>(
      this->get_node_base_interface(),
      this->get_node_clock_interface(),
      this->get_node_logging_interface(),
      this->get_node_waitables_interface(),
      "fibonacci",
      std::bind(&FibonacciActionServer::handle_goal, this, _1, _2),
      std::bind(&FibonacciActionServer::handle_cancel, this, _1),
      std::bind(&FibonacciActionServer::handle_accepted, this, _1));
  }

I would have made a fork with a pull-request, but I could not figure out how the code was embedded - Specifically, where the file, which the tutorial pulls from, is located.
Distro Dashing
OS Ubuntu 18.04
Note: I am using Ubuntu 18.04 and running Dashing, so maybe the code works on another distro - and in that case a switch-page (or what it is called in Github terms) should be made.
EDIT:
I was too quick in only reporting about the action server - The client got problems too... I found the file on the Github too demos/action_tutorials/action_tutorials_cpp/src/fibonacci_action_client.cpp - However, just copying the code over results in problems with "no match for 'operator'=" around line 71-73. So, instead I just replaced this part (line 21-31):
explicit FibonacciActionClient(const rclcpp::NodeOptions & options)
  : Node("fibonacci_action_client", options)
  {
    this->client_ptr_ = rclcpp_action::create_client<Fibonacci>(
      this,
      "fibonacci");

    this->timer_ = this->create_wall_timer(
      std::chrono::milliseconds(500),
      std::bind(&FibonacciActionClient::send_goal, this));
  }

With this: (line 36-39)
explicit FibonacciActionClient(const rclcpp::NodeOptions & node_options = rclcpp::NodeOptions())
  : Node("fibonacci_action_client", node_options)
  {
    this->client_ptr_ = rclcpp_action::create_client<Fibonacci>(
      this->get_node_base_interface(),
      this->get_node_graph_interface(),
      this->get_node_logging_interface(),
      this->get_node_waitables_interface(),
      "fibonacci");

    this->timer_ = this->create_wall_timer(
      std::chrono::milliseconds(500),
      std::bind(&FibonacciActionClient::send_goal, this));
  }

And then it compiles and the server/client talks with each other and the client gets the Fibonacci numbers as intended.
--
BR
Peter

See #1083, which should fix this for Dashing.


paper-icon-theme: 2017-11-20 -> 2018-06-24

Motivation for this change
Update version and build dependencies
Things done

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

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

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

cc @romildo
@GrahamcOfBorg  build paper-icon-theme

Failure on x86_64-darwin (full log)
Attempted: paper-icon-theme
Partial log (click to expand)

trying https://github.com/snwh/paper-icon-theme/archive/c7cd013fba06dd8fd5cdff9f885520e2923266b8.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   163    0   163    0     0    317      0 --:--:-- --:--:-- --:--:--   317
100 46.2M    0 46.2M    0     0  8182k      0 --:--:--  0:00:05 --:--:-- 10.5M
unpacking source archive /private/tmp/nix-build-source.drv-0/c7cd013fba06dd8fd5cdff9f885520e2923266b8.tar.gz
fixed-output derivation produced path '/nix/store/cz1kl9x1zad2fv7ickv4aksym5f4driz-source' with sha256 hash '01x6pds9gcy377p8gjlc5ps2bc4wygd74fnpnmz4h6wmwfr8g2yz' instead of the expected hash '0x45zkjnmbz904df63ph06npbm3phpgck4xwyymx8r8jgrfplk6v'
cannot build derivation '/nix/store/768pi5a812qxfw87i3sc2az9x65bv143-paper-icon-theme-2018-06-24.drv': 1 dependencies couldn't be built
error: build of '/nix/store/768pi5a812qxfw87i3sc2az9x65bv143-paper-icon-theme-2018-06-24.drv' failed

Success on x86_64-linux (full log)
Attempted: paper-icon-theme
Partial log (click to expand)
Installing subdir /build/source/Paper to /nix/store/d1q2wqxi8w0as0z90gx6i5js8ccb4j3h-paper-icon-theme-2018-06-24/share/icons/Paper
Installing subdir /build/source/Paper-Mono-Dark to /nix/store/d1q2wqxi8w0as0z90gx6i5js8ccb4j3h-paper-icon-theme-2018-06-24/share/icons/Paper-Mono-Dark
Running custom install script '/build/source/meson/post_install.py'
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/d1q2wqxi8w0as0z90gx6i5js8ccb4j3h-paper-icon-theme-2018-06-24
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/d1q2wqxi8w0as0z90gx6i5js8ccb4j3h-paper-icon-theme-2018-06-24
checking for references to /build in /nix/store/d1q2wqxi8w0as0z90gx6i5js8ccb4j3h-paper-icon-theme-2018-06-24...
/nix/store/d1q2wqxi8w0as0z90gx6i5js8ccb4j3h-paper-icon-theme-2018-06-24

Success on aarch64-linux (full log)
Attempted: paper-icon-theme
Partial log (click to expand)
Installing subdir /build/source/Paper to /nix/store/597mbgd096a7q0aps9rs7lrgn7ryvxcc-paper-icon-theme-2018-06-24/share/icons/Paper
Installing subdir /build/source/Paper-Mono-Dark to /nix/store/597mbgd096a7q0aps9rs7lrgn7ryvxcc-paper-icon-theme-2018-06-24/share/icons/Paper-Mono-Dark
Running custom install script '/build/source/meson/post_install.py'
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/597mbgd096a7q0aps9rs7lrgn7ryvxcc-paper-icon-theme-2018-06-24
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/597mbgd096a7q0aps9rs7lrgn7ryvxcc-paper-icon-theme-2018-06-24
checking for references to /build in /nix/store/597mbgd096a7q0aps9rs7lrgn7ryvxcc-paper-icon-theme-2018-06-24...
/nix/store/597mbgd096a7q0aps9rs7lrgn7ryvxcc-paper-icon-theme-2018-06-24

Looks good to me.

Thanks!


Server expects Sec-WebSocket-Accept

In the RFC the Sec-WebSocket-Accept

[...] is sent from the server to the client [...]

but in WebSocketConnection.cs the server expects it from the client. That seems to be the reason why it doesn't work with websocket-sharp.

Ohh I git confused with how this works, sorry.
The real problem was that WebSockets are not supported in Windows 7, this caused accept to be null.


Restrict users to find by one unique prefix and keyword

Previously:
find n/john n/sam -> Accepted
Now:
find n/john n/sam -> Rejected
This improves the clarity of the item the user wishes to find, by rejecting multiple keywords of the same prefix

The code will hinder the find with multiple tags and this should be fixed in a future PR.


Documentation is not being built for tags

🐛 Bug
The circleci build workflow for tags should be building and pushing a version of the documentation, but is not. Compare this nightly build which has build_docs and upload_docs steps, to this tag build which only has binary_win* builds.
I think the problem is in the regereate.py script where it only adds a master filter for windows, which triggers a tag filter as well:
https://github.com/pytorch/vision/blob/0013d9314cf1bd83eaf38c3ac6e0e9342fa99683/.circleci/regenerate.py#L40-L50
So the only builds that get the tag filter are the windows ones.
This is the same script for text and audio as well, so whatever fix is done here should be done for them as well.

Hi @mattip
Thanks for raising this. BTW, after the fix, do we still need to manually update the version selector?

do we still need to manually update the version selector?

Yes.

Docs are now being built. There are still some quirks: see #4754


upgrade code to use ocaml upgrade to 4.11.2

#8898 has all the needed changes, CI passes (with one expected exception).
Leaving it in draft status, until we have a plan to test and merge.


Uninstallation fails for dotnetcore and dotnetcore-runtime packages

When you perform try to perform a choco uninstall for the dotnetcore and dotnetcore-runtime packages they just fail.  This is a problem for those wanting to automate this via configuration management tools.

You're correct, there is no uninstall script. How's your auto-uninstall configured?

Hey @riezebosch ,
I'm using the stock options for the latest version of chocolatey.  No customizations performed.   Are you able to add an uninstall script for this?

Hey @riezebosch ,
Were u able to reproduce this?

I guess you can call this a repro.

I'm not sure if my problem was the same, but a choco upgrade wouldn't uninstall the old installation first. So when you tried to uninstall later on you'd get the warning stated above. I added a chocolateybeforemodify.ps where all current installations are uninstalled first.

Hey @riezebosch ,
I will give this a shot.  On a side note would it be possible to get up a seperate package that allows for side-by-side installs?


Basic support for WeMo Insight Switch

Tried to follow the design pattern, let me know if this is okay!

Hi there, thanks for your interest in this project and for the pull request as well. I don't have a Wemo Insight switch to test your changes. Can you please confirm that this works for you? Also please update the README to indicate the support for Wemo Insight switch. Just add it to the list of supported devices after #3 (Wemo Switch). Once you confirm that it works and update the README, I will merge your PR. Thanks!

Hey Arif, thanks for the great library! I've been struggling to find anything that works with the Insight. I've tested the following functions on my Insights successfully: getParams, On, Off, getBinaryState. I couldn't get GetPower, GetTodayKWH, or any of the other Insight-specific commands working, but those are all derivable from GetInsightParams.
I've updated the README.md and added an example.

Thanks for the changes and the example. As long as the methods in the DeviceInterface works for now it all good.


Request to add Citipointe Christian College

Request to add Citipointe Christian College. Website url is https://citipointe.qld.edu.au/

@sadaramk Pull request merged. Thank you!


tapToClose not working in andorid

cannot get andorid to work on tapToClose while ios works fine
<Drawer
              captureGestures={false}
              acceptPan={false}
              openDrawerThreshold={100}
              openDrawerOffset={0.2}
              tapToClose={true}
              acceptTap={true}
              ref="drawer"
              content={<ControlPanel closeDrawer={this.closeControlPanel}/>}
              closeDrawer={this.closeControlPanel}

              >

I just tested this and it is working for me. What version of android are you on? Just to be sure try v1.12.0

should be fixed in 1.12.0 but your testing and confirmation would be much appreciated

it works now, but partial of screen tap is captured by my another component in mainview. thanks!

@notlose not sure I follow. What partial of screen is being captured? Be aware that tapToClose applies specifically to the panCloseMask area, so you will want to make sure panCloseMask = openDrawerOffset


Relax upper bounds in cabal?

I'm successfully running dbus-introspect-hs with ghc 8 and template-haskell  2.11 - would you be OK with me sending a PR with updated bounds?

Hello
Please send a PR.

Now it builds with ghc8.


mimir: 2.8.0 -> 2.9.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for mimir is: Grafana Mimir provides horizontally scalable, highly available, multi-tenant, long-term storage for Prometheus.
meta.homepage for mimir is: https://github.com/grafana/mimir
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.9.0 with grep in /nix/store/brs63275ymplf4swbmg3q180iyi7abxi-mimir-2.9.0
found 2.9.0 in filename of file in /nix/store/brs63275ymplf4swbmg3q180iyi7abxi-mimir-2.9.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
mimir

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/brs63275ymplf4swbmg3q180iyi7abxi-mimir-2.9.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A mimir https://github.com/r-ryantm/nixpkgs/archive/08f2a5aed48cdf751f60ba29aad5202e0159c26e.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/brs63275ymplf4swbmg3q180iyi7abxi-mimir-2.9.0
ls -la /nix/store/brs63275ymplf4swbmg3q180iyi7abxi-mimir-2.9.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

mimir

Maintainer pings
cc @happysalada @bryanhonof for testing.

Result of nixpkgs-review pr 239448 run on x86_64-linux 1

1 package built:

mimir


chart freezed after serveral clicks of buttons in toolbox

Version
5.0.0-alpha.2
Steps to reproduce
in recent next version,  follow the sequences of clicks of buttons in toolbox freeze the chart

click stack view button
click line view button
click the stack(tiled now) button
now it is freezed

the magic type options are:
magicType: {
                    show: true,
                    type: ['line', 'bar', 'stack'],
                    title: {
                        line: 'line view',
                        bar: 'bar view',
                        stack: 'stack view',
                        tiled: 'tiled view',
                    },
                },

1

2

3

4

What is expected?
chart as normal
What is actually happening?
chart freezed!

Hi @pingf , seems I can't reproduce this issue in my example. Can you provide full option?

    <meta charset="utf-8">
    <title>Using ECharts</title>

hello world

and from the console, the error shows as above

and I've tested it again, the steps to reproduce this is simple(at least on my local machine)

click the line view button
click the stack view button
and now it freezes

Hi, I can't also reproduce it in this example https://gallery.echartsjs.com/editor.html?c=xb8NHHc3TB&v=1, is there anything missing?

well, it seems it only shows on my dev environment.
I use the latest code
echarts (next branch)
zrender (next branch)

@pingf I can reproduce it now:) Will fix it soon


Forcibly remove trailing "/" in homepage_url and repo_url

We want to make it easy to detect duplicate homepage_url and repo_url values.  Let's forcibly remove trailing "/" in those URLs (in both the current database, and attempts to edit in the future), so that duplicates are easier to detect.

Unfortunately we already have these situations, including in keys (where that can be a problem).  I think the first step is to quietly remove the trailing "/" when people create a badge entry, so it's less likely to happen by accident.  We can then slowly deal with editing & database adjustments.

All done!


fzf-tmux: Executes fzf in env-based shell

This allows executing fzf in a user-defined shell, rather than just using the bash. I noticed that both fzf and fzf-tmux has different behavior due to different shell environment (fzf runs in the commandline shell, but fzf-tmux always executes fzf in bash).
This behavior got prominent when using fzf.fish plugin that had a fish autoloaded function to preview file contents. In fzf it ran succesfully, but in fzf-tmux it threw an error due to missing function in bash.

What is the output of tmux show -g default-shell?


add final retouches on my page

Highlights

Checked linters✔
Documented work in a professional way✔
Styled my page ✔

Thanks youuuu!!!!!!!!!


Try to fix failing jenkins build

Codecov Report

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

@@            Coverage Diff            @@
##             master      #18   +/-   ##
=========================================
  Coverage          ?   55.77%
  Complexity        ?      406
=========================================
  Files             ?       59
  Lines             ?     2010
  Branches          ?      130
=========================================
  Hits              ?     1121
  Misses            ?      831
  Partials          ?       58

Continue to review full report at Codecov.

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

I have no clue why the Jenkins build fails. It works on my machine and on Travis. Something is wrong on Jenkins when it reads the keystore.jks file. The file seems to be corrupted ¯_(ツ)_/¯

jenkins retest this please


Community forum login

Logging in with a community forum account should be added as long as the user is in one of the devbuild accessing groups.

This is now also added.


=== problem

Hi,
I tried using WKZombie for this code
browser.open(url)

browser.get(by: .Id("Username"))
browser.setAttribute("value", value: username)
browser.get(by: .Id("Password"))
browser.setAttribute("value", value: password)
browser.get(by: .Id("site-login-form"))
browser.submit(then: .Wait(2.0))
browser.get(by: .Contains("href", "#studentmyday/schedule"))
browser.click(then: .Wait(2.0))
browser.getAll(by: .Id("accordionSchedules"))
=== handleResult

and it returns the error:
Binary operator '===' cannot be applied to operands of type 'Action<[HTMLElement]>' (aka 'Action<Array>') and '(Action<[HTMLTableRow]>) -> ()'
. All the methods are the same with the sample files and I am sure I have the update version. Any ideas?
Thanks!

Hi. I made some breaking changes to the === operator recently. Could you please try to change the signature of your handleResult function to this:
func handleResult(result: Result<[HTMLTableRow]>)
Hope that helps.

yea that works, thanks a lot!
I found another problem with my test app and would really need some help.. so it went through and i did a browser.dump(). However I found out that it just fetched an HTML source file where the truncated tags are all missing..
for example the phone it loads

    <input name="__RequestVerificationToken" type="hidden" value="aHkiUeYthjno_QJwArziB-TEZDs2jPtKqZ1XMeGWXlVevzO5ck2t8wwWifkxkfkJ3DqqaS8lYBkxPBrv-tdLJzl1D8_c58cs633wlTNFYT01">

but when I inspect the source code, there are html tags within  that I can't access through the WKzombie browser..
Any help would be appreciated!

Maybe you're trying to access the tags before the page loading (including JavaScript / Ajax) has completed? Did you try to increase the number of waiting seconds using the .Wait()-PostAction?

Ah thanks, it worked!


Add operation logs, change heatmap to horizontal style

Closing this PR temporarily to remove the experiment folder from history.


Add ecosystem field to ProjectSummaryResponse

In order to return a value for ecosystem from the projects list endpoint (as requested by https://github.com/phylum-dev/api/issues/292), we need to add that field here to the ProjectSummaryResponse struct. I also added a derivation for the sqlx::Type trait in this PR, which is required in order to return a value of this type in a RETURNING sql clause.
I will link the dependant API PR here once it is posted.

Taking a new approach in my API PR, and won't need the sqlx::Type derivation anymore. This PR is now just adding ecosystem to the ProjectSummaryResponse.


direct_association_only argument causes error

Both the role_users() function in the roleApi, and the user_roles() function in the userApi, offer a "direct_association_only" argument so that you can find only roles that have been assigned directly to a user (i.e., not indirectly via Group membership). However, setting this argument (to TRUE, True, true, 1, or FALSE) causes the following error:
Error in f(init, x[[i]]) : is.request(y) is not TRUE
So this call fails:

roleApi$role_users(config = sdk$oauthHeader,
role_id = "4",
direct_association_only = "TRUE")$content

But this succeeds:

roleApi$role_users(config = sdk$oauthHeader,
role_id = "4")$content

traceback() shows the following, which suggests the issue is maybe in httr and/or curl?

8: stop(simpleError(msg, call = if (p <- sys.parent(1L)) sys.call(p)))
7: stopifnot(is.request(x), is.request(y))
6: f(init, x[[i]])
5: Reduce(request_combine, extra, init = request())
4: request_build("GET", hu$url, as.request(config), ...)
3: httr::GET(url, queryParams, headers, ...)
2: self$apiClient$callApi(url = paste0(self$apiClient$basePath,
urlPath), method = "GET", queryParams = queryParams, headerParams = headerParams,
body = body, ...)
1: roleApi$role_users(config = sdk$oauthHeader, role_id = "4",
direct_association_only = "TRUE")

Also, just discovered - the "fields" argument has the same effect.


add watermark imagic to default tools

See if https://github.com/rasbt/watermark is useful for the template

add environment logging to schema (each log mark should contain information of the environment in which it was created in): https://github.com/City-of-Helsinki/ml_project_template/issues/32

watermark only supports python 3.7, so it will not be included in the template


add support for startproject command and alias for it

My friend tried to run "django startproject", I told him that the "django" command uses the "manage.py" not the "django-admin.py".
So I added support for this command, and the alias "sp" for it.
Since the "django-admin.py" and the "manage.py" have some of the same commands, I think is not necessary to add support for others.
Thanks.

Merged, thanks!


ocamlPackages.mlx: init at 0.9

https://github.com/ocaml-mlx/mlx
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[X] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[X] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Please split into two commits, one for adding you as maintainer and the other for the package.
I don't remember what the first commit should be named but the package commit should be ocamlPackages.mlx: init at 0.1.0

@ulrikstrid done

this version doesn't build on OCaml 5.3

@anmonteiro do you have a fix in the overlays or should we just set max version constraint?


"First time using FABRIC" appears again when I login from firefox.

This dialogue is the one that appears when a user signs in to FABRIC for the first time. It popped up to me when I tried working from firefox.

The portal/login has changed a lot since this version. I'm not sure this is repeatable anymore. Keep it in mind and look for it in the newer versions.

I tried Firefox one more time and it didn't happen :+1: I'll report if I ever catch it again.


JDK not picked up

Question

Context
Java version:
C:\Users\frmau>java -version
openjdk version "<IP_ADDRESS>" 2020-11-04
OpenJDK Runtime Environment AdoptOpenJDK (build <IP_ADDRESS>+1)
OpenJDK 64-Bit Server VM AdoptOpenJDK (build <IP_ADDRESS>+1, mixed mode)
Your operating system and platform:
Windows 10 64 bit

For some reason my question did not show up so I opened a new issue.


problems with the latest firmware of standard jk

I'm having problems with one of the latest standard versions of the jk bms, it doesn't detect several values, the cell delta is wrong, the SOC is not displayed and several other errors, on the contrary the voltage of my 16 cells is displayed correctly

I have configured this protocol

i have other 2 older jk-bms and all perfect, any solution?

Have you tried protocol_version: JK02_24S ?

Have you tried protocol_version: JK02_24S ?

i'll try thanks

Did you resolve the issue? The wrong protocol_version is the root cause for sure.


UI related gems are being required in all processes, consuming memory

Tranferred https://github.com/ManageIQ/cfme/issues/1270 from old repository.
This is old but most likely still relevant.
All process running through rails load these gems, so multiply these values by each worker/server process + any script/rails process we run.
7 MB for actionwebservice:
28907 root 20 0 323m 69m 6060 S 0.0 3.7 0:06.32 ruby
Add require => false to actionwebservice
+gem "actionwebservice", "=3.1.0", :require => false, :git => "git://github.com/ManageIQ/actionwebservice.git", :tag => "v3.1.0-1"
28915 root 20 0 314m 62m 6000 S 0.0 3.3 0:05.23 ruby
4 MB for haml-rails:
21079 root 20 0 324m 70m 6060 S 0.0 3.7 0:06.05 ruby
Add require => false to haml-rails
+gem 'haml-rails', "~> 0.4", :require => false
21068 root 20 0 320m 67m 6060 S 0.0 3.6 0:05.93 ruby
2 MB for prototype-rails:
29065 root 20 0 314m 62m 6000 S 0.0 3.3 0:05.45 ruby
Add require => false to prototype-rails
+gem "prototype-rails", "~>3.2.1", :require => false
29090 root 20 0 312m 61m 6000 S 77.2 3.3 0:05.24 ruby
Ruport, used by anything using MiqReport objects (UI, reporting, server process probably, etc.) and princely add less than 1 MB each by adding :require => false for them.

Most of this is solved by bundler groups on master


Listening to Location all time will cost battery?

Listening to Location all time will cost battery?

Please consult platform documentation or ask SO.


Fix #171: Do not include .babelrc in npm published package

This is the minimum required fix for #171. If you think it's a good idea, I can also add test/ to .npmignore.
If you don't care about including src/ in the published package, it's probably better to just whitelist dist in package.json : "files": ["dist"]

Good thinking. Yes, it’d be great to also add test/.  Thanks @thatmarvin

Np @krisk, done!

Can you publish this to npm please? 🤞🏼

@krish can you publish this update? I see an error every-time I re-install this package as a result of this.

Any update on this, please?


Contentful Integration + Tailwind upgrade

This is really Franzi's Ticket with the Contentful Integration. I've simply merged the Tailwind upgrade that I did in the other PR into the Contentful integration to get it up and running.
Access Tokens: Please remember when you checkout the branch to add the Contentful Access Token and Space ID which are located in Settings/API Keys on Contentful. If you need an invitation please ask Cedric.
@franziskapendzialek Please check that everything works as it did before from when you worked on it.

I've added some changes from another ticket to this branch, since future changes rely on the Tailwind upgrade to function correctly.
I've added this ticket to the PR: https://www.notion.so/hubblr/Change-Contact-Person-to-Simon-00c5a803702b4d3d9fca9f147e86cb32


The install script is interfering with other installs on macOS

I found that having cairo-lang installed with the project's install script was interfering with another install made by the commands that starknet_in_rust`s makefile run for compiling contracts and executing tests, making compilation fail and me unable to run tests.
Uninstalling cairo-lang with brew and removing the CARGO_MANIFEST_DIR var from my env solved the problem. I didn't investigate it yet but the problem may be in the env variable.

You had a problem with the installation of "cairo-lang" that was interfering with your "starknet_in_rust" project, causing errors during compilation and testing. The temporary solution was to uninstall "cairo-lang" and remove the "CARGO_MANIFEST_DIR" environment variable.
I'll write some possible issues that caused this problem:
Check if the "CARGO_MANIFEST_DIR" variable is set correctly and not causing conflicts.
Examine the dependencies and build tools of your "starknet_in_rust" project to ensure there are no conflicts with "cairo-lang."
Try to pinpoint the exact source of the conflict by uninstalling "cairo-lang" and checking if the problem persists.
If the issue continues, you might seek help from the online communities of the involved projects.
Check if both projects are up to date with the latest versions.
Verify system and dependency compatibility.
If "brew" is causing conflicts with "cairo-lang," consider using an alternative package manager or building "cairo-lang" from source.


How to execute from command line not pycharm

Hi @SBCV ,
I tried to execute python ssr/run_pipeline.py from SatelliteSurfaceReconstruction folder.
Also i tried python run_pipeline.py from ssr folder.
Got this module not found error in both case,
Traceback (most recent call last):   File "run_pipeline.py", line 3, in <module>     from ssr.utility.logging_extension import logger ModuleNotFoundError: No module named 'ssr'
In the Getting_started doc, I could not understand this ,
`
Add VisSatSatelliteStereo to the SatelliteSurfaceReconstruction project
For example in Pycharm
    Open SatelliteSurfaceReconstruction
    Open VisSatSatelliteStereo and attach it to SatelliteSurfaceReconstruction
    Go to File / Settings / Project: SatelliteSurfaceReconstruction / Project Dependencies
        Select SatelliteSurfaceReconstruction
        Make sure that VisSatSatelliteStereo is selected

`
Can you guide me about how to setup the whole folder structure for the full pipeline execution?
I'm attaching my folder structure below,

Hi, thank you for opening this issue - my explanation at this point should have been more precise.
As you can see in the source code, the library uses absolute and not relative imports (e.g. from ssr.utility.logging_extension import logger). Since we did not install the library,  your Python interpreter does not know where ssr is located.
One way to tell your python interpreter the location of the module, is to use the PYTHONPATH environment variable.
1.) Add ssr (in SatelliteSurfaceReconstruction) to the python interpreter
export PYTHONPATH="${PYTHONPATH}:/path/to/SatelliteSurfaceReconstruction"
2.) Add lib (in VisSatSatelliteStereo) to the python interpreter
export PYTHONPATH="${PYTHONPATH}:/path/to/VisSatSatelliteStereo"
If you use an IDE like pycharm, you can usually avoid messing around with the $PYTHONPATH environment variable, and define the dependencies within the IDE.
Hope this helps, I'll update the install instructions

Maybe to summarize:
export PYTHONPATH="${PYTHONPATH}:/path/to/SatelliteSurfaceReconstruction"
export PYTHONPATH="${PYTHONPATH}:/path/to/VisSatSatelliteStereo"
cd /path/to/SatelliteSurfaceReconstruction/ssr
python run_pipeline.py

I've updated the installation instructions - see this link

Thank you so much @SBCV for your quick response.
I'm just trying and let you know if I find any other issue.

Hi @SBCV ,
As you suggested, I tried that. When I run "python run_pipeline.py" first time, it's generating a new 'pipeline.cfg' and ending with some path error. So, I edited the pipeline.cfg as you have already mentioned here https://github.com/SBCV/SatelliteSurfaceReconstruction/blob/master/Getting_Started.md
But when i again i run it it's throwing the same error.
colmap_vissat_exe_fp: /path/to/ColmapForVisSat/build/src/exe/colmap Traceback (most recent call last):   File "run_pipeline.py", line 94, in <module>     vissat_pipeline = VisSatPipeline(pm)   File "/home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/ssr/sfm_mvs_rec/vissat_pipeline.py", line 19, in __init__     assert os.path.isfile(colmap_vissat_exe_fp) AssertionError
So,it's not getting the instances from the new pipeline.cfg as you can see the first line of my traceback where i have printed the path. So i hard coded the 'colmap_vissat_exe_fp' and the error went. But I would have to do it for all the instances that is edited in the pipeline.cfg.
Can you suggest where to pass the new config while running the 'run_pipeline.py'?
Also I need some clarification about the field to be edited in the pipeline.cfg.
As you suggested in the 'Getting_Started.md', we need to Install at least one of the following MVS & surface reconstruction libraries in which colmapvissat is already included. So i didn't installed others for surface reconstruction. I installed MVS_texturing as suggested by you. For meshlab, I just downloaded the latest zip file and extracted it.
Since I'm just using colmapvissat, so commented the 'mve_apps_dp',gdmr_bin_dp,openmvs_bin_dp.
And for meshlab, i just put the 'bin' folder of the extracted meshlab.tar downloaded from your provided link.
For meshing_backends, I have used "MeshingBackends.colmap_poisson".
But confused what to select for texturing_backends as i have not installed "openmvs" and "mve". I have just installed MVS_texturing for texture .
So finally my edited pipeline.cfg looks something like this,
`# This config file format follows and extends the syntax used by
https://docs.python.org/3/library/configparser.html
[DEFAULT]
=== USAGE ===
Everything that follows a "#" symbol is treated as a comment.
File and directory paths are denote with
"_fp" (file path) and
"_dp" # (directory path).
The paths MUST NOT contain any quotation marks.
Boolean variables (such as "reconstruct_sfm_mvs" or "extract_pan") can be
toggled with "1" and "0" or "True" and "False".
=== VisSat ===
colmap_vissat_exe_fp = '/home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/ColmapForVisSat/build/src/exe/colmap'
=== MVS, Mesh and Surface Reconstruction Libraries  ===
#mve_apps_dp = /path/to/mve/mve/apps
texrecon_apps_dp = '/home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/mvs-texturing/build/apps'
#gdmr_bin_dp = /path/to/pointfusion/pointfusion-0.2.0-Linux/bin
#openmvs_bin_dp = /path/to/openMVS/build/bin
colmap_exe_dp = '/home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/ColmapForVisSat/build/src/exe/colmap'
=== Meshlab ===
Make sure that Meshlab has access to the "meshlab_temp_dp" directory below
For example, the snap installation of Meshlab may have NO access to /tmp
meshlab_server_fp = '/home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/MeshLab2021.05-linux/usr/bin'
meshlab_temp_dp = '/home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/meshlab_temp'
=== Computation behavior ===
lazy = True
=== SfM / MVS Options ===
reconstruct_sfm_mvs = 1
=== Preparation Options ===
extract_pan = 1
extract_msi = 1
pan_sharpening = 1
depth_map_recovery = 1
skew_correction = 1
===
reconstruct_mesh = 1
texture_mesh = 1
meshing_backends = [
"MeshingBackends.colmap_poisson",
# "MeshingBackends.colmap_delaunay",
# "MeshingBackends.openmvs",
# "MeshingBackends.mve_fssr",
# "MeshingBackends.mve_gdmr"
]
Comment symbols can also be used within lists. For example to enable /
disable specific meshing backends, e.g.
meshing_backends = [
"MeshingBackends.colmap_poisson",
# "MeshingBackends.colmap_delaunay",
"MeshingBackends.openmvs",
# "MeshingBackends.mve_fssr",
"MeshingBackends.mve_gdmr"
]
texturing_backends = [
# "TexturingBackends.openmvs",    # Currently not supported
"TexturingBackends.mve",
#"TexturingBackends.mvs"
]
===== IARPA MVS3DM dataset =====
satellite_image_pan_dp ='/home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/satellite_imgs/WV3/PAN'
satellite_image_msi_dp ='/home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/satellite_imgs/WV3/MSI'
===== Site 1 (Used in the VisSat Evaluation) =====
workspace_vissat_dp ="/home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat"
workspace_ssr_dp ="/home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_ssr"
zone_number = 21
hemisphere = S
ul_easting = 354035.0
ul_northing = 6182717.0
width = 745.0
height = 682.0
alt_min = -20.0
alt_max = 100.0
===== Site 2 (Used in the VisSat Evaluation) =====
workspace_vissat_dp =
workspace_ssr_dp =
zone_number = 21
hemisphere = S
ul_easting = 354984.0
ul_northing = 6185506.0
width = 366.0
height = 350.0
alt_min = -20.0
alt_max = 80.0
===== Site 3 (Used in the VisSat Evaluation) =====
workspace_vissat_dp =
workspace_ssr_dp =
zone_number = 21
hemisphere = S
ul_easting = 355028.0
ul_northing = 6187389.0
width = 426.0
height = 397.0
alt_min = -20.0
alt_max = 90.0`
Thanks again.

Can you please add a code block around the config content in your previous message? Difficult to read it in the current state

Have you maybe accidentally changed the value in pipeline_template.cfg and not pipeline.cfg?
Can you go to line 62 in ssr/run_pipeline.py and add
print(ssr_config.config_fp). What path does it print?

As you said I checked it, i think it's getting the proper path for pipeline.cfg, but then why assertion error,
python run_pipeline.py  your print statement from run_pipeline.py : /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/ssr/configs/pipeline.cfg colmap_vissat_exe_fp from vissat_pipeline.py: '/home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/ColmapForVisSat/build/src/exe/colmap' Traceback (most recent call last):   File "run_pipeline.py", line 95, in <module>     vissat_pipeline = VisSatPipeline(pm)   File "/home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/ssr/sfm_mvs_rec/vissat_pipeline.py", line 19, in __init__     assert os.path.isfile(colmap_vissat_exe_fp) AssertionError
But I figured it out that all the paths should be without string quotes as in template pipeline,
so I changed all the paths from "/path/to/build/src/exe/colmap" to just  /path/to/build/src/exe/colmap
And the above Assertion Error went away.

So, I could start running the 'run_pipeline.py' somehow but I'm getting the below error,
`python run_pipeline.py
your print statement from run_pipeline.py : /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/ssr/configs/pipeline.cfg
colmap_vissat_exe_fp from vissat_pipeline.py: /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/ColmapForVisSat/build/src/exe/colmap
INFO:root:self.pm.vissat_config_fp: /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/VisSat.json
{'dataset_dir': '/home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/satellite_imgs/WV3/PAN', 'work_dir': '/home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat', 'bounding_box': {'zone_number': 21, 'hemisphere': 'S', 'ul_easting': 354035.0, 'ul_northing': 6182717.0, 'width': 745.0, 'height': 682.0}, 'steps_to_run': {'clean_data': True, 'crop_image': True, 'derive_approx': True, 'choose_subset': True, 'colmap_sfm_perspective': True, 'inspect_sfm_perspective': True, 'reparam_depth': True, 'colmap_mvs': True, 'aggregate_2p5d': True, 'aggregate_3d': True}, 'alt_min': -20.0, 'alt_max': 100.0}
INFO:root:dataset path: ['/home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/satellite_imgs/WV3/PAN']
INFO:root:will save files to folder: /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/cleaned_data
INFO:root:the standard format is: <7 char date><6 char time>-P1BS-<20 char product id>.NTF
INFO:root:cleaning 01SEP15WV031000015SEP01135603-P1BS-500497284040_01_P001_________AAE_0AAAAABPABP0.NTF
INFO:root:cleaning 02APR15WV031000015APR02134716-P1BS-500276959010_02_P001_________AAE_0AAAAABPABB0.NTF
INFO:root:cleaning 02APR15WV031000015APR02134718-P1BS-500497282050_01_P001_________AAE_0AAAAABPABJ0.NTF
INFO:root:cleaning 02APR15WV031000015APR02134802-P1BS-500276959010_02_P001_________AAE_0AAAAABPABC0.NTF
INFO:root:cleaning 02APR15WV031000015APR02134804-P1BS-500497284080_01_P001_________AAE_0AAAAABPABJ0.NTF
INFO:root:processed 5 items in total
INFO:root:Timer started at 2021-05-25 13:15:46, summarized at 2021-05-25 13:15:47, total elapsed 0.014337 minutes
description: Data cleaning Module
start: 2021-05-25 13:15:46
Data cleaning done: 2021-05-25 13:15:47, since_last: 0.014337 minutes, since_start: 0.014337 minutes
step clean_data:	finished in 0.014340516666666666 minutes
INFO:root:process 12958, cropping 1/5, ntf: /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/cleaned_data/15SEP01135603-P1BS-500497284040_01_P001.NTF
INFO:root:process 12957, cropping 0/5, ntf: /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/cleaned_data/15APR02134804-P1BS-500497284080_01_P001.NTF
INFO:root:process 12959, cropping 2/5, ntf: /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/cleaned_data/15APR02134718-P1BS-500497282050_01_P001.NTF
INFO:root:process 12960, cropping 3/5, ntf: /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/cleaned_data/15APR02134802-P1BS-500276959010_02_P001.NTF
INFO:root:process 12961, cropping 4/5, ntf: /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/cleaned_data/15APR02134716-P1BS-500276959010_02_P001.NTF
INFO:root:ntf image to cut: /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/cleaned_data/15SEP01135603-P1BS-500497284040_01_P001.NTF, width, height: 43008, 43008
INFO:root:cut image bounding box, ul_col, ul_row, width, height: 14690, 15355, 2057, 2091
INFO:root:png image to save: /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/tmp/12958:0001:15SEP01135603-P1BS-500497284040_01_P001.png
INFO:root:Running subprocess: gdal_translate -of png -ot UInt16 -srcwin 14690 15355 2057 2091 /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/cleaned_data/15SEP01135603-P1BS-500497284040_01_P001.NTF /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/tmp/12958:0001:15SEP01135603-P1BS-500497284040_01_P001.png
INFO:root:ntf image to cut: /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/cleaned_data/15APR02134802-P1BS-500276959010_02_P001.NTF, width, height: 43008, 29696
INFO:root:cut image bounding box, ul_col, ul_row, width, height: 24371, 3125, 1985, 2195
INFO:root:png image to save: /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/tmp/12960:0003:15APR02134802-P1BS-500276959010_02_P001.png
INFO:root:Running subprocess: gdal_translate -of png -ot UInt16 -srcwin 24371 3125 1985 2195 /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/cleaned_data/15APR02134802-P1BS-500276959010_02_P001.NTF /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/tmp/12960:0003:15APR02134802-P1BS-500276959010_02_P001.png
INFO:root:ntf image to cut: /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/cleaned_data/15APR02134716-P1BS-500276959010_02_P001.NTF, width, height: 43008, 28672
INFO:root:cut image bounding box, ul_col, ul_row, width, height: 24184, 2056, 1928, 2056
INFO:root:png image to save: /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/tmp/12961:0004:15APR02134716-P1BS-500276959010_02_P001.png
INFO:root:Running subprocess: gdal_translate -of png -ot UInt16 -srcwin 24184 2056 1928 2056 /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/cleaned_data/15APR02134716-P1BS-500276959010_02_P001.NTF /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/tmp/12961:0004:15APR02134716-P1BS-500276959010_02_P001.png
INFO:root:ntf image to cut: /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/cleaned_data/15APR02134804-P1BS-500497284080_01_P001.NTF, width, height: 43008, 36864
INFO:root:ntf image to cut: /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/cleaned_data/15APR02134718-P1BS-500497282050_01_P001.NTF, width, height: 43008, 36864
INFO:root:cut image bounding box, ul_col, ul_row, width, height: 24371, 8209, 1985, 2194
INFO:root:cut image bounding box, ul_col, ul_row, width, height: 24184, 9640, 1928, 2056
INFO:root:png image to save: /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/tmp/12957:0000:15APR02134804-P1BS-500497284080_01_P001.png
INFO:root:png image to save: /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/tmp/12959:0002:15APR02134718-P1BS-500497282050_01_P001.png
INFO:root:Running subprocess: gdal_translate -of png -ot UInt16 -srcwin 24371 8209 1985 2194 /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/cleaned_data/15APR02134804-P1BS-500497284080_01_P001.NTF /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/tmp/12957:0000:15APR02134804-P1BS-500497284080_01_P001.png
INFO:root:Running subprocess: gdal_translate -of png -ot UInt16 -srcwin 24184 9640 1928 2056 /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/cleaned_data/15APR02134718-P1BS-500497282050_01_P001.NTF /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/tmp/12959:0002:15APR02134718-P1BS-500497282050_01_P001.png
INFO:root:Input file size is 43008, 28672
INFO:root:Input file size is 43008, 29696
INFO:root:0...10...20...30...40...50...60...70...80...90...100 - done.
INFO:root:Subprocess finished
INFO:root:0...10...20...30...40...50...60...70...80...90...100 - done.
INFO:root:Subprocess finished
INFO:root:Input file size is 43008, 36864
INFO:root:0...10...20...30...40...50...60...70...80...90...100 - done.
INFO:root:Subprocess finished
INFO:root:Input file size is 43008, 36864
INFO:root:0...10...20...30...40...50...60...70...80...90...100 - done.
INFO:root:Subprocess finished
INFO:root:Input file size is 43008, 43008
INFO:root:0...10...20...30...40...50...60...70...80...90...100 - done.
INFO:root:Subprocess finished
INFO:root:Timer started at 2021-05-25 13:15:47, summarized at 2021-05-25 13:16:47, total elapsed 1.00136 minutes
description: Image cropping module
start: 2021-05-25 13:15:47
image cropping done: 2021-05-25 13:16:47, since_last: 1.00136 minutes, since_start: 1.00136 minutes
step crop_image:	finished in 1.0013645500000001 minutes
INFO:root:deriving an affine camera approximation...
INFO:root:scene coordinate frame is in lat, lon, alt
INFO:root:residual error (pixels): 0.0897340284458035
INFO:root:residual error (pixels): 0.08646687084550146
INFO:root:residual error (pixels): 0.077138199325807
INFO:root:residual error (pixels): 0.07556797422352711
INFO:root:residual error (pixels): 0.08467842272992213
INFO:root:deriving a perspective camera approximation...
INFO:root:scene coordinate frame is in ENU
INFO:root:singular values:  0.018201683090824313<PHONE_NUMBER>7251816 7356.49728738482 25689.95856535532 47122.19620156296 51920.66564592974 84131.86475960165 94850.92875029102<PHONE_NUMBER>218126 43629163.618809216 149413190.3496925 162789536.6106509
INFO:root:singular values:  0.018682439676534823<PHONE_NUMBER>62375 7357.848092753466 25690.614903828377 47134.17623404645 51929.71397724839 84126.80198616433 94851.04256576655<PHONE_NUMBER>105738 43617074.00565755 149381099.48553303 162756492.377518
INFO:root:singular values:  0.011817055831826207<PHONE_NUMBER>2006304 6861.006144527523 25432.101539450556 41565.65890723015 53014.38485674486 84735.28477259395 93748.79237338423<PHONE_NUMBER>262869 47479155.22177774 153972185.8213711 168356704.34668428
INFO:root:singular values:  0.011645058705925417<PHONE_NUMBER>3827414 6861.065503790191 25432.23589322226 41564.666785326895 53014.8152800425 84734.64207757187 93747.26441224132<PHONE_NUMBER>464184 47475804.616387114 153961409.18636802 168343462.12129146
INFO:root:singular values:  0.01922331325117253<PHONE_NUMBER>8630192 7360.334125429207 25702.49338093768 46263.672654665126 53643.58877406965 83800.69896814745 94477.66559785818<PHONE_NUMBER>186535 46160741.56484158 156300570.17355317 169943201.42871875
INFO:root:Timer started at 2021-05-25 13:16:47, summarized at 2021-05-25 13:16:49, total elapsed 0.0306161 minutes
description: Derive Approximation Module
start: 2021-05-25 13:16:47
Derive approximation done: 2021-05-25 13:16:49, since_last: 0.0306161 minutes, since_start: 0.0306161 minutes
step derive_approx:	finished in 0.0306676 minutes
step choose_subset:	finished in 9.9e-06 minutes
INFO:root:Running subprocess: colmap feature_extractor --database_path /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/colmap/sfm_perspective/database.db                                     --image_path /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/colmap/sfm_perspective/images                                     --ImageReader.camera_model PERSPECTIVE                                     --SiftExtraction.max_image_size 10000                                      --SiftExtraction.estimate_affine_shape 0                                     --SiftExtraction.domain_size_pooling 1                                     --SiftExtraction.max_num_features 25000                                     --SiftExtraction.num_threads 32                                     --SiftExtraction.use_gpu 1                                     --SiftExtraction.gpu_index -1
INFO:root:ERROR: flag 'logtostderr' was defined more than once (in files 'src/logging.cc' and '/home/ujjawal/intrinsic3d/glog/src/logging.cc').
INFO:root:Subprocess finished
INFO:root:Running subprocess: colmap exhaustive_matcher --database_path /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/colmap/sfm_perspective/database.db                                             --SiftMatching.guided_matching 1                                             --SiftMatching.num_threads 6                                             --SiftMatching.max_error 3                                             --SiftMatching.max_num_matches 30000                                             --SiftMatching.gpu_index -1
INFO:root:ERROR: flag 'logtostderr' was defined more than once (in files 'src/logging.cc' and '/home/ujjawal/intrinsic3d/glog/src/logging.cc').
INFO:root:Subprocess finished
Traceback (most recent call last):
File "run_pipeline.py", line 96, in
vissat_pipeline.run(reconstruct_sfm_mvs)
File "/home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/ssr/sfm_mvs_rec/vissat_pipeline.py", line 85, in run
pipeline.run()
File "/home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/VisSatSatelliteStereo/stereo_pipeline.py", line 115, in run
self.run_colmap_sfm_perspective()
File "/home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/VisSatSatelliteStereo/stereo_pipeline.py", line 346, in run_colmap_sfm_perspective
colmap_sfm_perspective.run_sfm(work_dir, sfm_dir, init_camera_file, weight)
File "/home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/VisSatSatelliteStereo/colmap_sfm_perspective.py", line 72, in run_sfm
reproj_err_threshold, reproj_err_threshold, reproj_err_threshold)
File "/home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/VisSatSatelliteStereo/colmap_sfm_commands.py", line 75, in run_point_triangulation
create_init_files(db_file, template_file, out_dir)
File "/home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/VisSatSatelliteStereo/colmap_sfm_utils.py", line 110, in create_init_files
table_images = db.execute("SELECT * FROM images")
sqlite3.OperationalError: no such table: images`
I have only downloaded some PAN and MSI images (not completely) as you can seen below,

And I'm using the same pipeline as mentioned above. (I just removed " " from the all the paths)

Please use code blocks to format your log properly

Sorry @SBCV ,
Comments edited.
Please have a look.

As you can see, the error appears in the VisSat library, so potentially you should open the issue there.
Looking at the log one sees that two error messages are thrown:
INFO:root:Running subprocess: colmap feature_extractor --database_path /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/colmap/sfm_perspective/database.db                                     --image_path /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/colmap/sfm_perspective/images                                     --ImageReader.camera_model PERSPECTIVE                                     --SiftExtraction.max_image_size 10000                                      --SiftExtraction.estimate_affine_shape 0                                     --SiftExtraction.domain_size_pooling 1                                     --SiftExtraction.max_num_features 25000                                     --SiftExtraction.num_threads 32                                     --SiftExtraction.use_gpu 1                                     --SiftExtraction.gpu_index -1
INFO:root:ERROR: flag 'logtostderr' was defined more than once (in files 'src/logging.cc' and '/home/ujjawal/intrinsic3d/glog/src/logging.cc').

INFO:root:Subprocess finished
INFO:root:Running subprocess: colmap exhaustive_matcher --database_path /home/ujjawal/my_work/object_recon/SatelliteSurfaceReconstruction/workspace_vissat/colmap/sfm_perspective/database.db                                             --SiftMatching.guided_matching 1                                             --SiftMatching.num_threads 6                                             --SiftMatching.max_error 3                                             --SiftMatching.max_num_matches 30000                                             --SiftMatching.gpu_index -1
INFO:root:ERROR: flag 'logtostderr' was defined more than once (in files 'src/logging.cc' and '/home/ujjawal/intrinsic3d/glog/src/logging.cc').

Maybe that could be an issue.
On the otherhand, during my experiments I observed that the --SiftExtraction.num_threads 32 caused problems on my system - see this issue.
Maybe it would be a good idea to directly execute the VisSat pipeline first (and then use SSR). This way it should be easier to detect the error, since VisSat allows you to run single reconstructions steps by modifying the corresponding config file
"clean_data": true,
"crop_image": true,
"derive_approx": true,
"choose_subset": true,
"colmap_sfm_perspective": true,
"inspect_sfm_perspective": false,
"reparam_depth": true,
"colmap_mvs": true,
"aggregate_2p5d": true,
aggregate_3d: false


TDR-3064 Remove Depreciated addFileMetaData Method

TDR-3064
Ticket is to remove a depreciated method in such a way as to not impact other services. This method seems to only be referenced internally, and tests pass locally.

[x] Method removed
[x] Graphql schema updated

Dependant repos merged, should be safe to merge changes


Find locations by bounding box

Given an Array of coordinates find the bounding box [nwLat, nwLng, seLat, seLng]:
{
  locations(boundingBox: [42.02238033615207,-76.4456118,38.88848331958911,-83.4768618]) {
    ...
  }
}

Fixed in 9870eab7057954e10e2b85136c39555b338926a3