Unify Application and Package overrides - Allow Package overrides for more core features (same as application/)

I am building a multi-purpose theme package that will be a base (starting point) for all websites I build with concrete 5.7 for ease of development, deployment and maintenance.
Since concrete 5.7 is so flexible and highly customizable I am sure we could achieve this without having to rely/depend on overrides at the application level - that is, also allow overriding [almost] everything at the package level that can be overridden at the application level by following a priority based system: Core > Package > Application with the latter able to override previous controllers when they exist.
This will allow/enable just a single package installation to setup everything as required and then we can start customizing the site using the numerous available options: page design - theme customizer, design and custom template - custom block templates and pagetheme area/block classes.
The bare minimum support should include:-

overriding block controllers for core blocks (override only possible now at application/blocks/)
email template used by the form block (currently done here application/mail/block_form_submission.php).

A method of prioritizing or configuring which package overrides what when multiple packages try to do so should also be provided - maybe a config or dashboard setting to select a particular package (sort of master package) override (Not needed for application level overrides since they are overall) even if its just site specific (not for marketplace). Alternatively the active package theme could be given precedence when multiple packages provide an override that the theme provides.
I am not sure if overrodeCoreByPackage() would work here and whether updates to core blocks will still be available to the overrides in the package but I remember reading packages which override the core this way cannot be approved for marketplace.
Example code that should work but currently does not: override placed in \packages\fluid_theme\blocks\image_slider\controller.php
namespace Concrete\Package\FluidTheme\Block\ImageSlider;

use Concrete\Block\ImageSlider\Controller as ImageSliderBlockController; // coreBlockController to be extended/overriden

class Controller extends ImageSliderBlockController {

    public function registerViewAssets() {
      // bxslider asset group registered in package on_start
        $this->requireAsset('bxslider'); // View::getInstance()->requireAsset('xyz') in view.php not reliable with cache on
        //echo 'registerViewAssets() in package block override!'; // testing...
    }

}

References
http://documentation.concrete5.org/developers/working-with-blocks/working-with-existing-block-types/creating-additional-custom-view-templates/including-custom-templates-in-packages
http://documentation.concrete5.org/developers/working-with-blocks/working-with-existing-block-types/using-a-custom-controller-for-a-core-block-type

The above is based on the logic that the three main directories (concrete, packages, application) are each analogous to each other... a package is therefore a microcosm of an entire site, and since these directories repeat the general pattern with each containing /blocks, /css, /js, /elements, /themes, /views, /controllers etc. functionality should be provided/implemented in any of the following places with overrides at higher levels:-

core (basic functionality)
packages (generic/extended functionality)
application (site specific functionality)

You are right to point out that these things are difficult to override from a package, but I want to add that the way we override today skirts the line of an OCP violation. Instead of adding an "override" like structure, I'd like us to begin an effort to make these parts of our framework more extensible by design.
A lot of our blocks do a ton of fancy stuff in their controllers. Perhaps we can work to abstract the things that one would want to change with these classes and try to reduce their responsibilities as much as we can so that each aspect is easily swappable without having to extend the base class and override by implicit directory structure or arbitrary string bindings like Environment::overrideCoreByPackage('/some/thing', $package);
So here's what I'm thinking. For a start, lets make it so that there is an easy way to break into the block controller selection process and replace the original controller with code. Perhaps an event or some kind of manager or a command bus or something:
(This is kind of long winded, this is just to show how powerful this could be, we can make it simpler to do)
<?php

/** \Symfony\Component\EventDispatcher\EventDispatcher */
$events = $this->app->make('director');

// Create our custom factory class that will be in charge of converting core block controllers into our custom ones
$controller_factory = \My\Custom\Package\Block\Controller\Factory();

// Set up an event listener so that we can break into the resolver
$events->addListener('blockBeforeControllerResolve', function(\Concrete\Core\Block\Controller\ResolverEventInterface $event) use ($controller_factory) {
    $resolver_manager = $event->getResolverManager();

    // Now that we have the manager, lets add a new resolver to the pipeline
    $resolver_manager->addResolver('block_handle', function($block_type, callable $next) use ($controller_factory) {
        // We call $next here so that we get the result from the completed pipeline
        // Notice that we could swap out the $block_type (or whatever information gets passed), or just not call next at all.
        $block_controller = $next($block_type);

        // Now we convert the core block controller into our custom block controller and return that instead.
        return $controller_factory->createFromController($block_controller);
    });
});

Here's just a few ways I could imagine this being used:

Setting some or all blocks to cache always
Adding CSRF protection to block actions
Swapping out the block type completely
Adding error information to the controller's error array
Adding interesting filters to a specific block type

So setting up a resolver like this this would give us all the power we need for swapping out / overriding the block controllers without the need for implicit path matching or too many arbitrary strings that our IDE's don't understand.
As we move forward, we can also put in some effort to breaking down these controllers into smaller more specific classes so that we can do cool stuff like:
<?php

$manager->addResolver('content', function(ContentBlockType $content_block_type, callable $next) {
    // Swap out the editor handler with a new one for ckeditor instead of redactor
    $content_block_type->setEditorHandler(new CkeditorHandler());

    return $next($content_block_type);
});

without having to completely override the entire functionality of the content block or do refactor large swaths of the block controller.
Perhaps we can do something similar with the email templates, what do you guys think about all of this @joe-meyer @aembler @ChrisHougard @herent ?

I should also add that this is a similar idea to what I was trying to do with my block-factory branch on my personal repo, I didn't get too far with it.

Its great, its clever, its powerful, its sophisticated.
Its also very long winded and complex compared to just putting a file at a predefined location in /application or /packages/my_package/
So while I appreciate the ultimate flexibility of such a clever approach, it should not be at the expense of losing the simple approach.

@JohntheFish I am not suggesting we remove the overrides we have. In my examples, I would expect  $next($block_type) to be resolve a controller in the exact same way it does today to support backward compatibility.
That said, if someone broke the chain and returned a replacement controller it'd override your override.
Thanks!
Korvin

Good. Thanks for the reassurance.
A small request should this be implemented. Have a global flag in the dashboard that can be used to disable it globally, and hence all overrides (assuming overrides other than blocks will work in a similar way behind the scenes).
That way, whenever a support request comes in and the environment report shows many overrides, and the response is 'Do you still have the issue with overrides removed?', its just a flip of a dashboard setting to do so.

This is complete. In version 8 you can actually override and/or provide your own custom block templates in a package's theme's directory, not just in application.


Set the breakpoint where the parser switches from 2000 to 1900 manually

If I set the parser pattern "ddMMyy" and the formatting pattern "dd.MM.yyyy" to the enhanced-date-picker, the following input "101070" is parsed to "10.10.2070". I think it makes more sense to parse this input to "10.10.1970" in context of date of birth, for example.
Is there a way to manually set the breakpoint where the parser switches from 2000 to 1900?

This is already coming in Vaadin 23.3, https://github.com/vaadin/flow-components/pull/3914
Duplicate of #1107

PR is created for this one now:

https://github.com/vaadin/flow-components/pull/4354


uv cache clean doesn't work

Environment
uv installed via pipx.
$ uv --version
uv 0.1.5

$ uname -a
Darwin machine.local 23.3.0 Darwin Kernel Version 23.3.0: Wed Dec 20 21:33:31 PST 2023; root:xnu-10002.81.5~7/RELEASE_ARM64_T8112 arm64

Problem
According to the README, to clear the cache, I should be running uv cache clean.

To clear the global cache entirely, run uv cache clean.

But:
$ uv cache clean
error: unrecognized subcommand 'cache'

Usage: uv [OPTIONS] <COMMAND>

For more information, try '--help'.

Wait for uv 0.1.6 to be released, in the meantime you can run uv clean

Ah sorry, that's a disconnect between the README (docs) and the latest published version.


Add .pbt as an extension for Protocol Buffer Text

Description

Checklist:

[x] I am associating a language with a new file extension.

[x] The new extension is used in hundreds of repositories on GitHub.com

Search results for each extension:

https://github.com/search?utf8=✓&type=Code&ref=searchresults&q=extension%3Apbt+core+NOT+nothack

[x] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):

https://github.com/coreslinkous/TeaShop/blob/main/Data/ProjectSettings.pbt

Sample license(s): https://github.com/coreslinkous/TeaShop/blob/main/LICENSE

[ ] I have included a change to the heuristics to distinguish my language from others using the same extension.

[ ] I am adding a new language.

[ ] The extension of the new language is used in hundreds of repositories on GitHub.com.

Search results for each extension:

https://github.com/search?utf8=✓&type=Code&ref=searchresults&q=extension%3AFOOBAR+KEYWORDS+NOT+nothack

[ ] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):

[URL to each sample source, if applicable]

Sample license(s):

[ ] I have included a syntax highlighting grammar: https://github-lightshow.herokuapp.com/
[ ] I have updated the heuristics to distinguish my language from others using the same extension.

[ ] I am fixing a misclassified language

[ ] I have included a new sample for the misclassified language:

Sample source(s):

[URL to each sample source, if applicable]

Sample license(s):

[ ] I have included a change to the heuristics to distinguish my language from others using the same extension.

[ ] I am changing the source of a syntax highlighting grammar

Old: https://github-lightshow.herokuapp.com/
New: https://github-lightshow.herokuapp.com/

[ ] I am updating a grammar submodule

[ ] I am adding new or changing current functionality

[ ] I have added or updated the tests for the new or changed functionality.

[ ] I am changing the color associated with a language

[ ] I have obtained agreement from the wider language community on this color change.

[URL to public discussion]
[Optional: URL to official branding guidelines for the language]

I'm unsure about how to best deal with the fact that PowerBuilder also uses pbt

I'm unsure about how to best deal with the fact that PowerBuilder also uses pbt

We normally rely on heuristics if the structures of the languages are quite similar but have unique ways to distinguish them. In this case, I think they're sufficiently different that we can rely on the classifier. To increase the chances of the classifier getting it right, please add another Proto buffer .pbt sample. The "Classifier cross-validation" test should highlight if this is sufficient (it needs two samples per language).

@lildude Sorry for the bump in here but is there a way to know which version of linguist is currently deployed on GitHub? I just want to make sure there was no issue with my PR because I don't see any of the repos using the new attribution yet.

there a way to know which version of linguist is currently deployed on GitHub?

7.21.0... deployed this morning, however this PR shipped with 7.19.0 which was deployed waaaaay back in February.

I just want to make sure there was no issue with my PR because I don't see any of the repos using the new attribution yet.

You need to push a change for your repo to be re-analysed to pick up any new languages etc.
It would also only appear as the language in a list of repos if this were the primary language of the repo.

Oh yes, I also forgot to say: the language is "data" so won't show in the language stats unless someone explicitly sets the linguist-detectable override. This applies to all "data" languages.
Syntax highlighting is however working and easy to check.

@lildude oh I see, thank you! I had assumed that it would be detectable by default, so I assume everyone who uses it will have to add the linguist-detectable attribute in their .gitattributes file still?

Yup, but only if they want it to show in their language stats.


Abstraction Layer for PostgreSQL

Issue Description
Implement an abstraction layer for PostgreSQL to improve database management and facilitate easier interactions with the database.
Expected Behavior
The application should seamlessly manage database operations through the abstraction layer, enhancing maintainability and flexibility.
Actual Behavior
Currently, the database interactions are tightly coupled with the implementation, making management more complex.

Hi, I understand that this is a comparatively complex issue, but i would like to go about it if it can be assigned. I tried searching for PostgreSQL in the codebase, but I found all the repositories in the /repo package extend the mongoRepository.
I wonder which file would be a good starting point to approach this issue. I would also highly appreciate if the issue description could be elaborated a bit.
Thanks!

@dhairya-quash Help @JishnuGoyal with understanding the issue in detail

@JishnuGoyal Kindly resume the conversation for any issue on our discord's open-source channel https://discord.gg/BuQPj8JY


what version should be used 0.3.0 or 0.2.6?

What is the suggested version for usage?

I would recommend to use 0.3.0.

Sorry I've just realized I forgot to push 0.3.0 to github after publishing to npm.


Plug-in not working with ios 4.0.0-dev

Been testing iOS 4.0 and noticed that the plugin is not compatible. Here are the compile errors:
nl.x-services.plugins.socialsharing/SocialSharing.m:33:24: No visible @interface for 'UIView' declares the selector 'stringByEvaluatingJavaScriptFromString:'
and
Plugins/nl.x-services.plugins.socialsharing/SocialSharing.m:606:24: Implicit declaration of function 'CDVNewBase64Decode' is invalid in C99
Any plans to make compatible with 4.0?

Thanks for letting me know, I'll do it shortly!

Sure. Here's the link to install ios 4.0 see https://shazronatadobe.wordpress.com

Fixed in 5.0.5.


Monster List is Difficult to Read

It's currently just displayed as a bunch of data in a series of UL.
I'd like it to either be a list of data in a more readable format, such as a table, or to list out the monster names and have the data displayed on a separate page.

Even if the data is displayed in a table, I want to include separate data pages with this issue's resolution.

At this point it's an entirely different issue, so see #12


Improve heal checker

Signed-off-by: denis-tingajkin<EMAIL_ADDRESS>
Description

Motivation and Context
How Has This Been Tested?

[ ] Covered by existing integration testing
[ ] Added integration testing to cover
[ ] Tested locally
[ ] Have not tested

Types of changes

[ ] 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)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

I prefer if that goes in after #1207.


SDL2_image: 2.6.3 -> 2.8.1

Changes:

https://github.com/libsdl-org/SDL_image/releases/tag/release-2.8.1
https://github.com/libsdl-org/SDL_image/releases/tag/release-2.8.0

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] 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

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 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

[ ] Fits CONTRIBUTING.md.

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

Let's move to draft until the failure is understood.

Check failures come from here: https://github.com/hedgewars/hw/blob/master/hedgewars/uTextures.pas#L235
Worth checking if SDL2_image changes stopped advertising alpha channel for some .png files.

The change that caused hedgewars regression is https://github.com/libsdl-org/SDL_image/commit/bf0821bcc71f16ba3c4282ec2e6aad29684faf57 . It exposes palette-based PNG files as palette-based surfaces (as opposed to directly encoded RGB-based surfaces). Seemingly working hack which is an equivalent of the revert:
--- a/src/IMG_stb.c
+++ b/src/IMG_stb.c
@@ -106,7 +106,7 @@ SDL_Surface *IMG_LoadSTB_RW(SDL_RWops *src)
             magic[14] == 'D' &&
             magic[15] == 'R' &&
             magic[25] == PNG_COLOR_INDEXED) {
-            use_palette = SDL_TRUE;
+            //use_palette = SDL_TRUE;
         }
     }
     SDL_RWseek(src, start, RW_SEEK_SET);

Other games like vcmi, fheroes2, chromium-bsu, wesnoth, blobwars work without the problems. Either because they don't peek at image internals or because they don't have such PNGs.

Added held back SDL2_image_2_6 and pinned hedgewars to it. Sent the email to hedgewars author (looks like their bugzilla is down for a while).
Ready for review.

Updated to 2.8.2 release


Update a LegalUser with no LegalRepresentativeAddress

For a legal user LegalRepresentativeAddress is optional, so you can create one without it. But when retrieving such an user, LegalRepresentativeAddress is not None, it got something like {"Region": null, "Country": null, "AddressLine2": null, "AddressLine1": null, "PostalCode": null, "City": null}. That's OK, but the thing is when you try to save your user (for example after you update some fields, it will send a request where LegalRepresentativeAddress is not null but either with actual data, just a dictionary with empty values so the API return an error.
How to reproduce the issue
from mangopay.resources import User, LegalUser

my_user = LegalUser(
    name='My Company',
<EMAIL_ADDRESS>    person_type='LEGAL',
    legal_person_type='BUSINESS',
    headquarters_address=Address(
        address_line_1='10 rue des champs',
        city='Nantes',
        postal_code='44000',
        country='FR',
    legal_representative_first_name='John',
    legal_representative_last_name='Doe',
    legal_representative_birthday=606441600,
    legal_representative_nationality='FR',
    legal_representative_country_of_residence='FR',
)
my_user.save()

pk = my_user.get_pk()

my_user2 = User.get(pk)
my_user2.save()     # <--- crash here, because API returns an error

The request sent by the SDK when doing the 2nd save()
HTTP Method: PUT
URL: https://api.sandbox.mangopay.com/v2.01/<ID_CLIENT>/users/legal/<ID_USER>
HEADERS:

Authorization: bearer
User-Agent: MangoPay V2 Python/None
Content-Type: application/json

DATA: {"LegalRepresentativeCountryOfResidence": "FR", "LegalPersonType": "BUSINESS", "LegalRepresentativeFirstName": "John", "LegalRepresentativeLastName": "Doe", "LegalRepresentativeBirthday": 606441600, "LegalRepresentativeAddress": {"Region": null, "Country": null, "AddressLine2": null, "AddressLine1": null, "PostalCode": null, "City": null}, "HeadquartersAddress": {"Region": null, "Country": "FR", "AddressLine2": null, "AddressLine1": "10 rue des champs", "PostalCode": "44000", "City": "Nantes"}, "Name": "My Company", "LegalRepresentativeNationality": "FR", "Email"<EMAIL_ADDRESS>"KYCLevel": "LIGHT", "PersonType": "LEGAL"}
API response
{"Message":"One or several required parameters are missing or incorrect. An incorrect resource ID also raises this kind of error.","Type":"param_error","Id":"2790e064-da7d-44e7-b71d-8af61d04bdbb#1504702107","Date":1504702108.0,"errors":{"LegalRepresentativeAddress.AddressLine1":"The AddressLine1 field is required.","LegalRepresentativeAddress.City":"The City field is required.","LegalRepresentativeAddress.Country":"The Country field is required."}}
Workaround
Set LegalRepresentativeAddress to None if its dictionary is actually empty values.
my_user2 = User.get(pk)
my_user2.LegalRepresentativeAddress = None
my_user2.save()

Hello @b0uh !
Thanks for your feedback.
I can easily understand it could be painful to process the way you have perfectly described above.
We could try to add an extra check for this kind of case, but we want to let our SDK the more simple as possible, and let every functional complexity to our API.
Because we don't provide any PATCH method (yet), it's difficult to easily handle this case, so your workaround seems to be the best solution today.


toc div fix

sometimes we want to use custom divs in left column, but current css settings dont allow that, becuase content will shift till the end of nearest content in right column. So if this line is only needed for one div, let it be specific by adding an id

Thanks! I think the div it's selecting though is the invisible div tocify creates above every header...they have different ids, I think?

I guess I'm just not sure that #toc is the right selector here, but agreed that a change should be made!

Well, maybe becuase I use kramdown I dont see it. I have only one of those divs in the top part of the page.

I think I'm going to close this since once we move away from tocify, we won't have dummy divs and won't need this any more, but thanks for the contribution!


Typo in wifi_enterprise README.md (IDFGH-8595)

Many thanks to the various authors for the wifi_enterprise example. It has been a big help to get me started with WPA-EAP using ESP32. There is however a typo in the command to generate client certificate in README.md in the section Steps to create enterprise openssl certs. The command in question is:
openssl ca -batch -keyfile ca.key -cert ca.pem -in client.csr -key (password) -out client.crt -extensions xpserver_ext -extfile xpextensions

The option -extensions should be xpclient_ext (it was correct prior to commit cd85c79d):
openssl ca -batch -keyfile ca.key -cert ca.pem -in client.csr -key (password) -out client.crt -extensions xpclient_ext -extfile xpextensions

In my tests where I used a Linux AP running hostapd version 2.9.0, having the incorrect X509 extension in the client cert will prevent hostapd from authenticating the client when using EAP-TLS.
Ref: https://github.com/espressif/esp-idf/blob/daf3da5e4bcd001fb339c2d5424077d819fc8051/examples/wifi/wifi_enterprise/README.md?plain=1#L74

Hi @smchan514 , Thank you for reporting the issue.


"Error starting gRPC call" when adding to basket

This might be a dumb question because I have not looked into gRPC very closely yet.
I'm getting this error when I click on "Add to basket" in the public facing app (https://localhost:44335/)
Grpc.Core.RpcException: 'Status(StatusCode="Unavailable", Detail="Error starting gRPC call. HttpRequestException: No connection could be made because the target machine actively refused it. (localhost:81) SocketException: No connection could be made because the target machine actively refused it.", DebugException="System.Net.Http.HttpRequestException: No connection could be made because the target machine actively refused it. (localhost:81)
Please give some guidance on how I can fix this. I just want to successfully run the app, then explore the code after that.
Thanks.

It seems like the catalog service is not up and running. Check the logs for the Catalog microservice please.

Hi @gterdem , thanks for responding.
This is the error I have in my log file:
2023-05-08 03:08:20.453 +08:00 [ERR] Failed executing DbCommand (2ms) [Parameters=[], CommandType='"Text"', CommandTimeout='30'] SELECT a."Id", a."DisplayName", a."ExtraProperties", a."Name" FROM "AbpPermissionGroups" AS a 2023-05-08 03:08:20.455 +08:00 [ERR] An exception occurred while iterating over the results of a query for context type 'EShopOnAbp.AdministrationService.EntityFrameworkCore.AdministrationServiceDbContext'. Npgsql.PostgresException (0x80004005): 42P01: relation "AbpPermissionGroups" does not exist

Closed by https://github.com/abpframework/eShopOnAbp/pull/170
Thank you!


[Question] is it possible for you to add dynamic shadows and will you make a discord server and what happened to you previous website?

for discord, it seems like I can't make it because I can't focus on taking care of it
for website, I'm redesigning but I'm too lazy to finish it

And what about dynamic shadows? Is it possible? It would make the shader perfect.


Groupby checkbox event is not getting captured

Working as expected in latest version.

Working as expected in latest version.


Does not seem to work with "ng serve"

Hi Arkon,
I am a bit new to this so maybe I am missing something and this is not a bug.  Maybe it needs to be integrated in a different way.
Anyway as other users may also be newbies I thought I would raise this ticket for it could alert you to the kind of problems your user base is having.
I used @angular/cli to create a new angular 2 project as follows:
 ng new ngsidebarexample
 cd ngsidebarexample
 npm install angularjs --save
 ng serve

Then I edited my app.component.ts as follows:
import { Component } from '@angular/core';

@Component({
  selector: 'app-root',
  templateUrl: './app.component.html',
  styleUrls: ['./app.component.css']
})
export class AppComponent {
  title = 'app works!';

  private _opened: boolean = false;

  private _toggleSidebar() {
    this._opened = !this._opened;
  }
}

I did not see any system js file so I thought I was not using it and so did not update it.  Your docs seem to indicate it is not mandatory according to your docs:
I updated my app.module.ts:
import { BrowserModule } from '@angular/platform-browser';
import { NgModule } from '@angular/core';
import { FormsModule } from '@angular/forms';
import { HttpModule } from '@angular/http';
import { SidebarModule } from 'ng-sidebar'

import { AppComponent } from './app.component';

@NgModule({
  declarations: [
    AppComponent
  ],
  imports: [
    BrowserModule,
    FormsModule,
    HttpModule,
    SidebarModule.forRoot()
  ],
  providers: [],
    bootstrap: [AppComponent]
 })
 export class AppModule { }

Then I opened browser at
 http://localhost:4200/

I saw the:
app works!
Toggle Sidebar button

Sidebar contents

If I click the sidebar button it disappeared but there was no sidebar
I think this example does not work with @anglular/cli project creation and ng serve
Not sure if this is a bug or I am completely ignorant about something?  Maybe this has to be integrated in a different way and does not work with "ng new" and "ng serve" ?
Sorry in advance if I am wasting time.
Thank you for your time

SystemJS is indeed an optional way of doing it. Since you're using the CLI tool, that uses Webpack internally instead of SystemJS. I haven't really looked into the changes with Angular 4 and AOT, but I suspect there's something wrong on my end for that, so I'll need to look into it.

Thanks for the info

Can you post the app.component.html template, for me it worked with ng-cli

@Nox-404 I'm assuming you're asking @javaspeak ?

@Nox-404 and @arkon
Thanks for your comments.
I ran
npm install --save ng-sidebar

again and this time it worked.
I may have previously somehow had an outdated previous version that did not work.
Thanks for your help.


[SPARK-29708][SQL] Correct aggregated values when grouping sets are duplicated

What changes were proposed in this pull request?

This pr intends to fix wrong aggregated values in GROUPING SETS when there are duplicated grouping sets in a query (e.g., GROUPING SETS ((k1),(k1))).
For example;
scala> spark.table("t").show()
+---+---+---+
| k1| k2|  v|
+---+---+---+
|  0|  0|  3|
+---+---+---+

scala> sql("""select grouping_id(), k1, k2, sum(v) from t group by grouping sets ((k1),(k1,k2),(k2,k1),(k1,k2))""").show()
+-------------+---+----+------+
|grouping_id()| k1|  k2|sum(v)|
+-------------+---+----+------+
|            0|  0|   0|     9| <---- wrong aggregate value and the correct answer is `3`
|            1|  0|null|     3|
+-------------+---+----+------+

// PostgreSQL case
postgres=#  select k1, k2, sum(v) from t group by grouping sets ((k1),(k1,k2),(k2,k1),(k1,k2));
 k1 |  k2  | sum
----+------+-----
  0 |    0 |   3
  0 |    0 |   3
  0 |    0 |   3
  0 | NULL |   3
(4 rows)

// Hive case
hive> select GROUPING__ID, k1, k2, sum(v) from t group by k1, k2 grouping sets ((k1),(k1,k2),(k2,k1),(k1,k2));
1	0	NULL	3
0	0	0	3

This pr adds code to dedupe grouping sets in the Analyzer.
Why are the changes needed?

To fix bugs.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

The existing tests.

Currently, I followed the hive result because of the simpler fix. But, we might need to check results in the other DBMS-like systems, too (Oracle and DB2?). I just want to know the Oracle answer, but I cannot because I don't have a Oracle instance. Anyone can help me and check it?

Test build #115607 has started for PR 26961 at commit 16d99c7.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/20406/
Test PASSed.

Test build #115607 has finished for PR 26961 at commit 16d99c7.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/115607/
Test FAILed.

retest this please

Test build #115613 has started for PR 26961 at commit 16d99c7.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/20412/
Test PASSed.

Test build #115613 has finished for PR 26961 at commit 16d99c7.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/115613/
Test FAILed.

Test build #115625 has started for PR 26961 at commit 19a36b3.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/20425/
Test PASSed.

Test build #115625 has finished for PR 26961 at commit 19a36b3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/115625/
Test PASSed.

Retest this please.

cc @cloud-fan

Test build #116625 has started for PR 26961 at commit 19a36b3.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/21406/
Test PASSed.

The result of SQL server:

I think pgsql is corrected.

Test build #116625 has finished for PR 26961 at commit 19a36b3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/116625/
Test PASSed.

ok, I'll follow the pgSQL behaviour.

Test build #116686 has started for PR 26961 at commit cdcc4d0.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/21465/
Test PASSed.

Test build #116686 has finished for PR 26961 at commit cdcc4d0.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/116686/
Test FAILed.

retest this please

Test build #116692 has started for PR 26961 at commit cdcc4d0.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/21473/
Test PASSed.


Mock limit and patch in the AnsibleTowerClient::JobTemplate

@bdunne, @bzwei Please review.

LGTM :+1:

:+1:  LGTM


[S3C-3567] Create Account Request/Response Marshaller in AWS SDK sugg…

…ested format

Hello saitharunthodupunuru,
My role is to assist you with the merge of this
pull request. Please type @bert-e help to get information
on this process, or consult the user documentation.
Status report is not available.

Waiting for approval
The following approvals are needed before I can proceed with the merge:

the author

2 peers


build issues?

Hi,
Thanks for your work on this. I look forward to see the final product. I have the following issue on Ubuntu. When trying to do the ./build.sh I get the following errors:
osboxes@osboxes:~/iptvx$ ./build.sh
src/config.c:32:23: fatal error: libconfig.h: No such file or directory
#include <libconfig.h>
^
compilation terminated.
src/epg.c:23:23: fatal error: libconfig.h: No such file or directory
#include <libconfig.h>
^
compilation terminated.
src/main.c:24:23: fatal error: libconfig.h: No such file or directory
#include <libconfig.h>
^
compilation terminated.
src/util.c:32:23: fatal error: curl/curl.h: No such file or directory
#include <curl/curl.h>
^
compilation terminated.
src/video.c:19:21: fatal error: vlc/vlc.h: No such file or directory
#include <vlc/vlc.h>
^
compilation terminated.
src/window.c:25:27: fatal error: SDL/SDL_image.h: No such file or directory
#include <SDL/SDL_image.h>
^
compilation terminated.
Can you please help?
Many thanks Dorin

Hi dorinilut,
Current status of the code
First of all just a quick info: The current status is "alpha 0.2" and any useful status of the software will not exist until "beta 0.5" which I think I'll finish around March 2017. Anything before "beta 0.5" should be considered "a bunch of code that may compile on your system".
What can be used
If you're still keen to see the current WIP outcome, you can try to compile it, but you need to check the compiling script (build.sh) to include the correct libraries on your system. I build it on OpenSUSE 42.1 and if you're using a DEB-system like Debian or Ubuntu your development libraries might have different names. You might even be required to change them in the code before compiling.
Your concrete issue
The issue you are having is that libvlc, SDLimage, libcurl and libconfig development versions are not present in your system or the compiler cannot find it in your include path. On Debian/Ubuntu the packages end with "-devel" and on SuSE etc. they end with "-dev". So check if you have them.
Just a quick warning: the current status contains config files from development which may cause the program to segfault under some circumstances. It still is a development version and proper functionality cannot be guaranteed at the moment.
Thank you a lot for your interest in my project! Once the beta is ready for release, there will also be a website under iptvx.org and binary packages (RPM and DEB) are also planned.
I am closing this ticket as the missing development libraries are your build issue here.
Cheers, Jan

Thanks for your quick reply. . I will follow your recommendations. if you
want a stable source for your IPTV testing I have a tvheadend server where
I can provide access. My final intention is to use your software on a
raspberry as a simple TV interface.
I am really looking forward to see the final build and provide testing
using multiple systems.
Dorin
Dorin Ilut
Mobile1: +31647071114
Mobile2: +40727777895
Skype: dorin.ilut.2
On Mon, Jan 30, 2017 at 11:57 AM, Jan Kammerath<EMAIL_ADDRESS>wrote:

Hi dorinilut,
Current status of the code
First of all just a quick info: The current status is "alpha 0.2" and any
useful status of the software will not exist until "beta 0.5" which I think
I'll finish around March 2017. Anything before "beta 0.5" should be
considered "a bunch of code that may compile on your system".
What can be used
If you're still keen to see the current WIP outcome, you can try to
compile it, but you need to check the compiling script (build.sh) to
include the correct libraries on your system. I build it on OpenSUSE 42.1
and if you're using a DEB-system like Debian or Ubuntu your development
libraries might have different names. You might even be required to change
them in the code before compiling.
Your concrete issue
The issue you are having is that libvlc, SDLimage, libcurl and libconfig
development versions are not present in your system or the compiler cannot
find it in your include path. On Debian/Ubuntu the packages end with
"-devel" and on SuSE etc. they end with "-dev". So check if you have them.
Just a quick warning: the current status contains config files from
development which may cause the program to segfault under some
circumstances. It still is a development version and proper functionality
cannot be guaranteed at the moment.
Thank you a lot for your interest in my project! Once the beta is ready
for release, there will also be a website under iptvx.org and binary
packages (RPM and DEB) are also planned.
I am closing this ticket as the missing development libraries are your
build issue here.
Cheers, Jan
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jankammerath/iptvx/issues/18#issuecomment-276033476,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYQDq71yq9A1K8v7CbyCRUnsAs6SDJQlks5rXcIYgaJpZM4LxLtE
.


Revamp keptn logging

Use fluent-bit for logging. Configuration in:stdout, out:http. Target is the DBAccessService (find a better name while implementing this), the is a wrapper service to a NoSQL database, e.g. MongoDB. The same service is used by the bridge to retrieves data from the database.
Default the DBAccessService is shipped with MongoDB. It should be possible going forward to use for example DynamoDB for that, by providing a different plugin to the DBAccessService.
Implement a distributor that subscribes to all topics and stores all NATS events in the database.
While doing that, standardize logging in all components and rework the logging instructions for custom services.

The setup of the new architecture is already completed in https://github.com/keptn/installer/tree/feature/507/keptn-logging . We do have to wait for an adapted version of the bridge that retrieves the log messages from the new datastore. As soon as this is ready, the feature branch in the installer repo is ready to be released.

Some comments on logging
#1: In a simple new artifacdt run I have 74 log messages from pitometer saying "Evaluation is passed" -> please reduce to 1
#2: Pitometer Result: I want to have a single log entry that tells me whether Pitomter has failed or passed INCLUDING the actual score, e.g: Pitometer EValuation PASS - total Score 75
#3: I also want to see the Pitometer Result + Score in the Dynatrace Event


oss-directory: Automatic factory validation

Describe the feature you'd like to request
When a factory is added we should validate that it's a factory.
Describe the solution you'd like
A factory is a smart contract that deploys other contracts. All contract addresses should be reviewed to determine if they are a factory or not. If they are a factory, then the contracts deployed by them should be counted as part of their downstream impact (but should not be separate artifacts).
Describe alternatives you've considered
N/A

Decomposition of #1034


Documentation page consuming too many cpu cycles

When I browse the documentation website (only when it is an active tab) with Firefox it consumes too many cpu cycles for no reasons.
Prior Issues
This closed issue seems to be related.
What is the current behavior?
Energy impact is medium to high for no reason in the task manager of Firefox 78.3.0esr. Fan gets noisy.
Steps to Reproduce
Just browse the documentation pages using Firefox. Apart from 78.3.0esr I haven't tried other versions. The issue doesn't appear using Chrome 86.
What is the expected behavior?
It's a static website with no fancy stuff, it shouldn't consume that many cpu cycles.
Environment Details
I'm using Debian 10 and Firefox 78.3.0esr.

Does this maybe have something to do with the CodeSandboxes being loaded? Does this happen on all pages, or just specific pages?

How can I check if CodeSandboxes are loaded ?
It doesn't seem to happen on all pages. For example it happens on https://redux.js.org/tutorials/essentials/part-1-overview-concepts but not on https://redux.js.org/recipes/configuring-your-store.
After disabling my cache and refreshing, consumption got down to 7, which is better but still high. Also, consumption seems to increase whenever I move my mouse on the page, even if it's just over some normal text. Could over-consumption be due to some kind of mouse tracking script ?

Hmm. The tutorial page doesn't have any CodeSandboxes, but it does have a large animated gif.

Indeed it's quite large, but opening the GIF in a standalone tab doesn't make my cpu sweat. Thank you for your help though

Yeah.  If you or anyone else can actually pinpoint a pattern in which pages are burning more CPU, and ideally why it's happening, let us know, but I don't think there's anything specific we can do atm.


Automate Releases

Please check if the PR fulfills these requirements

[x] Followed the Contributing guidelines.
[x] Tests for the changes have been added (for bug fixes/features) with 100% code coverage.
[x] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
This automates the versioning fully and takes care of packaging dependencies correctly

Hello
It seems more elaborate than my PR
However maybe you can divide in multiple PR

one for editing markdown file
one for adding rimraf
one for updating/adding github action workflows

It would be more easier to review and address potential issues or suggestions for improvement
Thanks ❤


ISC License

Hello can you add ISC License
https://opensource.org/licenses/ISC

Hi, I can see ISC license here: public static int ISC = 5;
But not here: Version 1.0.4.

I know. That is because I have not made the new release yet.

Added in Version 1.0.5.


Add log target to send profiling info and counters to statsd

https://github.com/etsy/statsd/blob/master/docs/metric_types.md

Looks like I've started to implement this feature + RUM as a separate extension.

this usecase is a nice to have as an extension


Implicit operator from Result to TValue

Would you consider an implicit operator from Result<TValue> to TValue which would return a succeeded result?

Thanks for giving feedback. On this topic I have a strong opinion, because I have a Domain Driven Design background and made a lot of experiences about Value Object conversions via implicit/explicit operators.
Implicit operators should succeed successfully in each case. For example int to decimal is safe, no information loss - implicit operator exists. Decimal to int is not safe because there is an information loss - no implicit operator, only an explicit operator exists.
A result object can be

in success state => Result.Ok(4) to TValue is safe.
in failed state => Result.Fail("Error") to TValue is not safe because a failed result has no value. If you try to access the Value of a failed state then an exception occur. Only via ValueOrNull property you can access the Value without an exception.

With this in mind a conversion from a result object to TValue is not safe in each case. Therefore I would not add an implicit operator from Result to TValue.
But maybe I don't see the whole picture. Please show me your scenario.

Thanks for giving feedback. On this topic I have a strong opinion, because I have a Domain Driven Design background and made a lot of experiences about Value Object conversions via implicit/explicit operators.
Implicit operators should succeed successfully in each case. For example int to decimal is safe, no information loss - implicit operator exists. Decimal to int is not safe because there is an information loss - no implicit operator, only an explicit operator exists.
A result object can be

in success state => Result.Ok(4) to TValue is safe.
in failed state => Result.Fail("Error") to TValue is not safe because a failed result has no value. If you try to access the Value of a failed state then an exception occur. Only via ValueOrNull property you can access the Value without an exception.

With this in mind a conversion from a result object to TValue is not safe in each case. Therefore I would not add an implicit operator from Result to TValue.
But maybe I don't see the whole picture. Please show me your scenario.

Hello!
Sorry I dont quite understand, which of these is ok?
It sounds like when you said:

in failed state => Result.Fail("Error") to TValue is not safe because a failed result has no value. If you try to access the Value of a failed state then an exception occur. Only via ValueOrNull property you can access the Value without an exception.

That  return Result.Fail("Item does not exist") is dangerous and will have errors?

Also, the two methods marked in green underline:

Result.Ok(itemDto)
itemDto.ToResult()

Are marked as 'Obselete', and the warning says to perform a .ToResult() instead, which is what I did in the second example and I still get the warning.
I don't understand this:

How do I return a successful Result with an object value?
How do I return a failed Result such as  return Result.Fail("Item does not exist")

Can you please update your examples?
Thanks for the great library! 🚀

Hello, I'am on vacation so my answer is short. To return an success result with a value you have to use Result as Return type of the method. In your case T is the type of itemDto. Then Result.Ok(..) is right and you have to change the fail method to Result.Fail(...) You find that all in the readme.


clean install, constant errors in the console, but app works

After a clean download of the repo, npm install and npm start... it "works" correctly but there are continual errors in the console.
GET http://localhost/info?t=1449285338378 net::ERR_CONNECTION_REFUSED
[WDS] Disconnected!

The URL being used in the failed GET is "localhost" but missing the port. And nothing is listening for that request. I can't figure out where the issue was coming from.

Thanks for reporting, did came across it.
will be fixed on next milestone.
For the meanwhile you can fix it on your own project, check out the comments on #4

Duplicate of #4


Die prototype

[x] Neutral dice
[x] Multiple dices
[x] Combo recognition and clarification

Tested and only implemented one combo (Three of a kind)


Add documentation page

The documentation page is missing, please complete the documentation.

Hi @opozo we updated the template and added a link to the documentation page in the latest commit.


Omit ...{:type} annotations from search index

Fixes #2913 by omitting the {:type} annotations from the search index when they appear in inline code.

@dimaMachina Are there any other changes needed on this PR?

I am currently busy with other tasks, no worries I will include this change in next release when I will back to clean up Nextra for contributions


[UI] Chatroom

Chat
Chat is a component used to display chat rooms based on pushstate
https://github.com/Semantic-Org/Semantic-UI/blob/ui-chatroom/src/definitions/modules/chatroom.js
Required for 1.x

Create documentation
Create examples
Major recode
Figure out pusher connection or switch to another provider

Required for 2.0
TBD

you can consider using pubnub

what is happening with this?

@jlukic there is some progress in this module? this would be a good one, even if would be only the html and css and some JS to change the person or options like the comments

+1

+1

Is there any updates about this? 👍

no


Error while pulling image: Get https://index.docker.io/v1/repositories/oracle/jdk/images: read tcp -> read: connection reset by peer

Description
Error while pulling image: Get https://index.docker.io/v1/repositories/oracle/jdk/images: read tcp  read: connection reset by peer
when trying to run docker
Checking if required packages are present and valid...
fmw_<IP_ADDRESS>.0_wls_quick_Disk1_1of1.zip: OK
Building image 'oracle/weblogic:12.2.1-developer' ...
Proxy  '' ...
Sending build context to Docker daemon 2.637 GB
Step 1 : FROM oracle/jdk:8
Pulling repository docker.io/oracle/jdk
Error while pulling image: Get https://index.docker.io/v1/repositories/oracle/jdk/images: read tcp  read: connection reset by peer
There was an error building the image.
its alwys giving error, this even   not working for hello world
Steps to reproduce the issue:
1.Download weblogic image dependencies
2.execute sh buildDockerImage.sh -v 12.2.1 -d
3.
Describe the results you received:
Checking if required packages are present and valid...
fmw_<IP_ADDRESS>.0_wls_quick_Disk1_1of1.zip: OK
Building image 'oracle/weblogic:12.2.1-developer' ...
Proxy  '' ...
Sending build context to Docker daemon 2.637 GB
Step 1 : FROM oracle/jdk:8
Pulling repository docker.io/oracle/jdk
Error while pulling image: Get https://index.docker.io/v1/repositories/oracle/jdk/images: read tcp : read: connection reset by peer
There was an error building the image
Describe the results you expected:
it should download the image
Additional information you deem important (e.g. issue happens only occasionally):
Output of docker version:

docker version
Client:
Version:      1.10.3
API version:  1.22
Go version:   go1.5.3
Git commit:   57bf6fd
Built:        Fri May 13 01:38:36 2016
OS/Arch:      linux/amd64

Server:
Version:      1.10.3
API version:  1.22
Go version:   go1.5.3
Git commit:   57bf6fd
Built:        Fri May 13 01:38:36 2016
OS/Arch:      linux/amd64

Containers: 0
 Running: 0
 Paused: 0
 Stopped: 0
Images: 0
Server Version: 1.10.3
Storage Driver: devicemapper
 Pool Name: docker-253:0-67923214-pool
 Pool Blocksize: 65.54 kB
 Base Device Size: 10.74 GB
 Backing Filesystem: xfs
 Data file: /dev/loop0
 Metadata file: /dev/loop1
 Data Space Used: 11.8 MB
 Data Space Total: 107.4 GB
 Data Space Available: 31.7 GB
 Metadata Space Used: 581.6 kB
 Metadata Space Total: 2.147 GB
 Metadata Space Available: 2.147 GB
 Udev Sync Supported: true
 Deferred Removal Enabled: false
 Deferred Deletion Enabled: false
 Deferred Deleted Device Count: 0
 Data loop file: /var/lib/docker/devicemapper/devicemapper/data
 WARNING: Usage of loopback devices is strongly discouraged for production use. Either use `--storage-opt dm.thinpooldev` or use `--storage-opt dm.no_warn_on_loop_devices=true` to suppress this warning.
 Metadata loop file: /var/lib/docker/devicemapper/devicemapper/metadata
 Library Version: 1.02.107-RHEL7 (2016-06-09)
Execution Driver: native-0.2
Logging Driver: json-file
Plugins:
 Volume: local
 Network: bridge null host
Kernel Version: 3.10.0-327.36.3.el7.x86_64
Operating System: Oracle Linux Server 7.2
OSType: linux
Architecture: x86_64
CPUs: 6
Total Memory: 19.45 GiB
Name: hostname
Http Proxy: http://hostname:2343
WARNING: bridge-nf-call-iptables is disabled
WARNING: bridge-nf-call-ip6tables is disabled

Additional environment details (AWS, VirtualBox, physical, etc.):

It looks like the oracle/jdk repository no longer exists on Docker Hub, IIRC it may have been deprecated by them (https://hub.docker.com/r/oracle/jdk is a 404);
docker pull oracle/jdk:8
Pulling repository docker.io/oracle/jdk
Error: image oracle/jdk:8 not found

Probably needs to be updated to https://hub.docker.com/r/oracle/openjdk/
I'll close this issue, because this is not a bug in docker, but due to changes made by Oracle. Feel free to comment after I closed


[ntf-core] Fix builiding on x86_64 Linux

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[x] The "supports" clause reflects platforms that may be fixed by this new version
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Attempting to run vcpkg x-add-version --all caused the following:
error: can't load port microsoft-signalr
error: while loading /Users/tfoxhall/repos/vcpkg/ports/microsoft-signalr/vcpkg.json:
$.default-features[0]: mismatched type: expected an identifierinternal error: /tmp/vcpkg-20230411-5267-16p751l/vcpkg-tool-2023-04-07/src/vcpkg/commands.add-version.cpp(412): vcpkg has crashed; no additional details are available.

I can open this as a separate issue if this is a real problem.

error: can't load port microsoft-signalr
error: while loading /Users/tfoxhall/repos/vcpkg/ports/microsoft-signalr/vcpkg.json:
$.default-features[0]: mismatched type: expected an identifierinternal error: /tmp/vcpkg-20230411-5267-16p751l/vcpkg-tool-2023-04-07/src/vcpkg/commands.add-version.cpp(412): vcpkg has crashed; no additional details are available.

I can open this as a separate issue if this is a real problem.

I can't reproduce this error locally.

Thanks for the feedback, I've applied the suggestions to fix the usage text.
Perhaps unrelated to the current thread, is the standard of practice for ports providing multiple targets to export them under a namespace? Would it be best practice to have the port export something like NtfCore::ntc and NtfCore::nts? Or is this mostly up to the discretion of the port? Is there a way to check for collisions in target name exports?

For CMake config, best practice is to use namespaces. A main advantage is that CMake can understand at configuration time that NtfCore::ntc must be an target. If it doesn't exist, it will not generate the build system but ask if you forgot some find_package. In contrast, ntc might als be a library name spec  if no such target exists. Then the build must proceed until the link step to let the linker resolve the name spec. And even if the linker can resolve it, it might not be what you want (debug vs. release, shared vs. static, port vs. system, ...)

Are we good to go here?

@hallfox Thanks for the fix!
@dg0yt Thanks for the review!
@LilyWangLL Thanks for the testing!


Add files via upload

Uploaded the klayout file (EBeam_shahazhameed.gds)

Hi,
I just realized that my cell name is not as per the instructions but my file is already uploaded to the submission folder in openEBL-2024-02. Could you please let me know how to delete this file and upload the latest one? Or can I upload the new file with the same file name (EBeam_shahazhameed.gds) but with the modified cell name? Please let me know.
These are the details of my file for your reference:
Currently uploaded file name: EBeam_shahazhameed.gds
Currently uploaded layout cell name: EBeam_shahazhameed_new_v1 (something like that , not sure because I modified the name now and cannot see the old one)
New file to be uploaded: EBeam_shahazhameed.gds Cell name: EBeam_shahazhameed_v1
I am attaching to this email the latest file that I intend to upload.
Thanks and regards,
Shahaz
De: "Hang Bobby Zou" @.>
À: "SiEPIC/openEBL-2024-02" @.>
Cc: "shahazhameed" @.>, "Author" @.>
Envoyé: Lundi 29 Janvier 2024 20:25:35
Objet: Re: [SiEPIC/openEBL-2024-02] Add files via upload (PR #46)
Merged [ https://github.com/SiEPIC/openEBL-2024-02/pull/46 | #46 ] into main.
—
Reply to this email directly, [ https://github.com/SiEPIC/openEBL-2024-02/pull/46#event-11632292200 | view it on GitHub ] , or [ https://github.com/notifications/unsubscribe-auth/BFWBUZVLELKWZMQZTMI44ITYQ7ZS7AVCNFSM6AAAAABCP5WINCVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJRGYZTEMRZGIZDAMA | unsubscribe ] .
You are receiving this because you authored the thread. Message ID: @.***>


Support cron processing in Ska3

Do we want to migrate the fid data itself into sqlite, hdf5, or other?


Make allowed_classes configurable when unserializing formState

With a new configuration option it is possible to add class names to allowed_classes  for unserialize() formState within FormRuntime::initializeFormStateFromRequest
closes #143

I'm just not sure about the target branch. IIRC features can only go into a 5.3 release!?

I can change the target branch to master, that is fine by me. @bwaidelich  Should I?

I can change the target branch to master

yes please. Thanks

@ger4003 Just a little update: I'm currently reviewing this and the change works great, but we are currently discussing whether the allowed_classes option could be skipped altogether. I will keep you updated

#close in favor of #145


Curve geometry types

Is your feature request related to a problem? Please describe.
We've began developing a webapplication in which we're looking to not only visualize but ultimately also interactively edit our stored geometries. We're not trying to reinvent the wheel and it seemed that OpenLayers was a good fit for this purpose. It did turnout though that there's no support for geometries that contain or solely consist of arcs.
Describe the solution you'd like
A solution that is often brought up for this problem is to stroke any arc before adding it to any OpenLayers map. However we believe editing the arc interactively would than become troublesome. Also, we would ultimately need to convert this stroked arc back to an actual arc. In short, this seems like a far from ideal solution to us.
Therefore we've began digging and attempting to extend the list of geometry types in OpenLayers. In a first attempt we've focused on adding a CircularString geometry type (as in WKT/WKB) and being able to visualize it. So far it seems that we're doing alright. Other more complex types like a CurvePolygon are on our list.
Anyway, we're willing to build support for these geometry types ourselves. What we like to know is whether or not you would be interested in this functionality as well? If not, we might need to change our strategy a bit. If yes, we would like to contribute. In return we'd probably need a bit of guidance along the way every now and then as we're no OpenLayers or Javascript experts.
Perphaps (or hopefully in fact) you've become interested in what we're attempting to do. What we've done so far you can find here.

Hi @sweco-nlahoo. Adding WKT CIRCULARSTRING to OpenLayers would be a good addition.
Could you open a PR with the work you made on this subject?

@nboisteault we were actually just talking about opening a PR one of these days. My colleague has been busy these days fine-tuning our work, adding missing pieces, etc. We'll definitely come back to you on this in the upcoming weeks.


feat: add ecr image finding table

This table is used to get findings on a specific tag, in a specific repository allowing joins with others data sources
Integration test logs

Logs
Add passing integration test logs here

Example ecr image tag findings
select
    repository_name
    ,image_tag
    ,image_scan_completed_at
    ,vulnerability_source_updated_at
    ,finding
    ,partition
    ,repository_name
    ,account_id
from services.aws_ecr_image_findings where repository_name = 'my-ecr-repository-name' and image_tag = 'v1.0.0'

Get findings on images tags pushed in the last 24 hours
select
    finding.*
from (
    select
        repository_name
        ,jsonb_array_elements_text(image_tags) as image_tag
    from services.aws_ecr_image i
    where
        i.image_pushed_at > now() - interval '24' hour
) images
left outer join services.aws_ecr_image_findings finding
on images.repository_name = finding.repository_name and images.image_tag = finding.image_tag

Hey @aminvielledebatAtBedrock , thanks for working on and opening this PR!
Are you ready for us to review the PR yet, or are you still planning on pushing in some changes (since the PR is in draft mode)?

Nope @cbruno10. It needs unit tests and documentation.
I opened it in draft to have your point of view about this new table.
It will return N rows for N findings for a specific image tag, and allow us to join rows with images from our kubernetes clusters without aws rate limiting (whereas table_aws_ecr_repository returns all findings for all images tags).
If you're interested about it, i can add documentation and tests quickly.

@aminvielledebatAtBedrock I think this new table is a great idea, I actually didn't even realize we return all image scan findings in the image_scanning_findings column in the aws_ecr_repository table. We actually made a similar change recently in that same table, where we had a column image_details, but we decided to create the aws_ecr_image table and plan on deprecating the image_details column actually.
One design aspect we're considering for this table is should it require a user to specify the image ID/tag and the repository (like they are in this PR), or should we leverage the ParentHydrate function, which allows us to re-use functions to list parent resources, eliminating the need for required key columns.
For instance, in the aws_ecr_image table, we use a ParentHydrate to list all repositories first, and then list images in each repository. If a user passes in a specific repository name though, it'll only get images in that repository.
In this case though, it'd potentially be 3 levels of nesting though, e.g., Repository -> Image -> Scan Findings, which is a lot, and like you mentioned, may lead to throttling.
If you're able to share, approximately how many repositories, images, and findings do you have in your environments where you'd be using this table? And do most of your accounts/environments have a similar number, or are there varied amounts?

No problem !
We have :
~250 images deployed in production (even more in other environments / tenants...). These production images are stored in only one service account.
~200 repositories for production in this account (~700 with non production)
~ an average of about 80 findings pointed by ecr scan (when you take account low and informational findings)
~ an ecr lifecycle policy wich delete oldest images when you have more than 300 image tags in your repository
So we have 200 x 300 x 80 = 480 0000  findings record only for production :disappointed:
ParentHydrate doesn't work in that case :'(

Thanks @aminvielledebatAtBedrock for the info!
Agree that trying to list that many image findings isn't really scalable, and even if we did get through all of the throttling exceptions, it'd take quite a bit of time to get all of the results.
With that in mind, I think requiring the repository and image digest or tag makes sense. I have a few other suggestions/questions for the table as well:

Can you please update the table name and filename to be aws_ecr_image_scan_finding? Usually our table names are singular, and I think adding the word scan makes sense, as the API + types include that word as well.
Since the API also allows a user to pass in the image digest, can you please add a new column and optional qual image_digest? In the list call, if neither an image digest or tag was passed, the function should return an error.
Looking at the response syntax for the API, there's the imageScanStatus field, was there a reason this was excluded as a column (image_scan_status)?
Instead of having a single column finding, I think it'd be better if each field from the ImageScanFinding type was its own column, e.g., attributes, description, name, etc.
The other columns, like image_scan_completed_at and vulnerability_source_updated_at would still be columns as well.
Also from the response syntax, I can see there are two sets of findings, enhanceFindings and findings. Both type of finding seems like there's a sufficient number of columns to be their own tables, e.g., aws_ecr_image_scan_finding, aws_ecr_image_enhanced_scan_finding. Does this also seem reasonable to you, based on your knowledge of enhanced findings vs. findings?

Hi @cbruno10 !
I did the changes on the aws_ecr_image_scan_finding table , excepting one : allowing usage of image_digest as a new optional qual.
I'm not able to use it when i join results from another table... I spend a lot of time without finding any answer to my issue.
Maybe you've got a good idea...
The plugin.HydrateData parameter is always empty (as expected because we don't have a parentHydrate), plugin.QueryData.KeyColumnQuals is always empty too on Optional fields image_digest and image_tag with the following queries :
select
  images.repository_name,
  images.image_tag,
  findings.*
from
  (
    select
      repository_name,
      jsonb_array_elements_text(image_tags) as image_tag
    from
      services.aws_ecr_image i
    where
      i.image_pushed_at > now() - interval '24' hour
  )
  images
  inner join
    services.aws_ecr_image_scan_finding findings
    on images.repository_name = findings.repository_name
    and images.image_tag = findings.image_tag

Returns Error: Ecr findings called without repositoryName or imageTag or imageDigest (SQLSTATE HV000)
select
  *
from
  services.aws_ecr_image_scan_finding f
where
  (
    f.repository_name,
    f.image_tag
  )
  in
  (
    select
      repository_name,
      image_tag
    from
      services.aws_ecr_image i
    where
      i.image_pushed_at > now() - interval '24' hour
    group by
      1,
      2
  )

returns Error: rpc error: code = Internal desc = 'List' call for table 'aws_ecr_image_scan_finding' is missing 1 required qual: column:'repository_name' operator: =  (SQLSTATE HV000)
If you have any suggestions please advise me :pray:

Thanks @aminvielledebatAtBedrock for making the changes so quickly and adding this table!


Moving to bootstrap 4 and cleaning up UI things.

This PR attempts to move our documentation site to bootstrap 4.3.1 as well as updating our webpack script.
This is my first step to start implementing the new UI for the docs site.

@stanislas-m i just fixed the video not showing, please let me know if there is something else before merging this one.


CONTRIBUTING.md Fix syntax for atom docs link

Missing single character parenthesis ( to fix syntax for atom docs link

Why:
Closes https://github.com/github/docs/issues/10274

What's being changed:

Check off the following:

[ ] I have reviewed my changes in staging (look for the latest deployment event in your pull request's timeline, then click View deployment).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

Closing as duplicate of another open PR https://github.com/github/docs/pull/10256

@achekerylla I am opening this PR as the other one is now closed 🐬 Thanks so much for opening a PR! I'll get this triaged for review :zap:


load all images from a post with multiple photos

Hi,
Some posts may have one images like this one.
https://www.instagram.com/p/BUwtRaTg30c/
(it contains 9 images in one post). Is there method I can use to grap all the images in this post? Thanks for your help first!

@mingtam The Instagram API was updated in April 2017 with carousel  that returns url for all photos that are part of the initial link. Check the response code (specifically the carousel_media portion):
https://www.instagram.com/developer/endpoints/media/#get_media
I haven't integrated this functionality yet, but I believe that it only works if you have public_content permission, thus you can't integrate the carousel if you're testing in the sandbox.

By any chances this has been integrated yet? I am looking to get carousel posts off instagram but the plugin returns only the first image of the carousel. Is there any way I can have access to the rest of the images to use in template?

@pfvqueiros Not yet! I've set up an issue to track the progress of it here: #650
But we'll probably address #646 and #649 before we tackle carousel images.


Unable to start Spegel on Talos 1.8 and ContainerdV2

Spegel version
v0.0.27
Kubernetes distribution
Talos
Kubernetes version
v1.30.0
CNI
Cilium
Describe the bug
I saw that ContainerdV2 is not yet fully supported, but Spegel is failing to start with a different error than the one already mentioned:
failed to generate spec: failed to mkdir "/etc/containerd/certs.d": mkdir /etc/containerd/certs.d: read-only file system

Not sure if this is due to Talos1.8 or containerdV2. Spegel works fine on Talos1.7.x which uses containerdV1.
Would appreciate any hints as to what I should look into as a workaround.

I thought I had set that correctly but I now see that I have a typo.. Thank you!
For reference: https://github.com/broersma-forslund/homelab/blob/31e7c4dbb96c1bf2f09390c9977428650ced1461/apps/cluster/spegel/values.yaml#L15
That's missing a letter 🫣


multiplication distributive law

Is there any possible to get "multiplication distributive law" express?
for example,
input : (A+B)(C+D)
output: AC+AD+BC+B*D
I knew matheclipse ,but it run too slowly

This library is only concerned with parsing and evaluating mathematical expressions, not modifying/simplifying/transforming them. So, I'd say no.


(BUG) The time on the counter is displayed more than 24 hours after clicking on the claim button

Description:  The time on the counter is displayed more than 24 hours after clicking on the claim button
Steps to reproduce:

Open the https://goodqa.netlify.app/
Click on the claim button on the Home screen
Click on the Claim button
Close the popup
Click on the Claim button
Pay attention to the time  on the counter

Expected result:
The time on the counter is displayed 24 hours after clicking on the claim button
Actual result:
The time on the counter is displayed more than 24 hours after clicking on the claim button
Used_ID: "0xC2e0fCf5D1bF0b684F3445CC34fA4548C80A45e8"
Environment:  Prod 1.11.7
Devices list: Desktop- Windows 10 // Chrome 87.0.4280.88
Additional info:
email<EMAIL_ADDRESS>pass: Tester_88888
Video: https://www.dropbox.com/s/njsjtn8rhqkgmf1/Time after claime on Prod.mp4?dl=0

this is fixed in the new claim screen you ahve test on QA


Unused folder and file autodelete not working

I've also asked this as a StackOverflow question here: https://stackoverflow.com/questions/45040572/why-is-django-photologue-not-automatically-deleting-images-from-the-project-dire
According to https://github.com/jdriscoll/django-photologue/wiki, Photologue cleans up after itself. Unused files and folders are deleted automatically.
However, when I add a picture in the admin interface and then delete it, the image files are not deleted from the associated folder. I am working on a development server using python manage.py runserver, and I have tried deleting the image, exiting the server, running python manage.py migrate as well as python manage.py makemigrations (because why not?) and the image files are still there in the /media/photologue/photos/ folder as if nothing happened.
I have set up my MEDIA and STATIC settings according to the recommendations of https://timmyomahony.com/blog/static-vs-media-and-root-vs-path-in-django/ and set up Photologue according to their Read-the-Docs page. Before discovering this issue I fixed a problem with broken thumbnails by implementing the recommendations here: https://stackoverflow.com/questions/40575506/django-photologue-thumbnails-not-loading.
Am I misunderstanding the meaning of deleted automatically or is there something I can do to fix this behavior? To be clear, I want the delete command in the admin interface to also remove the physical files in the associated directory.

Just realized this is a Django question, not a Photologue question. See https://stackoverflow.com/questions/16041232/django-delete-filefield. Closed.


create a custom layout

Hello, is there a way to create our own layouts to use with the app? I loved the app and I would like to create a custom layout for myself

You can create a custom layout yourself, this is exactly what my fork does.


Demos - Grid Toolbar Not Rendering

Short description
The demo pages for Grid with toolbars are not working. There is no error in the console.
What is current behavior
Toolbars are not rendering.
What is desired behavior
Toolbars render.
Link to jsfidle/codepan with sample code
...
Steps to reproduce or sample
...

Its fixed now.

Fixed


Can't commit due to eslint bug

Reproduce:

do yarn bootstrap-clean
do git commit -a -m "update"

pauliusguzas@Pauliuss-MacBook-Pro graybox % git commit -a -m "update"
husky > pre-commit (node v16.14.2)
$ yarn compile && yarn lint && yarn test
$ npx lerna run compile
lerna notice cli v6.6.2

> @flexn/graybox:compile

$ tsc -b tsconfig.json

 ———————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————

 >  Lerna (powered by Nx)   Successfully ran target compile for project @flexn/graybox (1s)

$ npx eslint ./packages

Oops! Something went wrong! :(

ESLint: 8.40.0

ESLint couldn't find the plugin "@typescript-eslint/eslint-plugin".

(The package "@typescript-eslint/eslint-plugin" was not found when loaded as a Node module from the directory "/Users/pauliusguzas/Desktop/graybox".)

It's likely that the plugin isn't installed correctly. Try reinstalling by running the following:

    npm install @typescript-eslint/eslint-plugin@latest --save-dev

The plugin "@typescript-eslint/eslint-plugin" was referenced from the config file in ".eslintrc.js » @flexn/eslint-config".

If you still can't figure out the problem, please stop by https://eslint.org/chat/help to chat with the team.

error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
husky > pre-commit hook failed (add --no-verify to bypass)
pauliusguzas@Pauliuss-MacBook-Pro graybox %

pauliusguzas@Pauliuss-MacBook-Pro graybox % git commit -a -m "update"
husky > pre-commit (node v16.14.2)
$ yarn compile && yarn lint && yarn test
$ npx lerna run compile
lerna notice cli v6.6.2

> @flexn/graybox:compile

$ tsc -b tsconfig.json

 ——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————

 >  Lerna (powered by Nx)   Successfully ran target compile for project @flexn/graybox (1s)

$ npx eslint ./packages
$ npx jest --coverage
● Validation Error:

  Preset react-native not found.

  Configuration Documentation:
  https://jestjs.io/docs/configuration

error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
husky > pre-commit hook failed (add --no-verify to bypass)
pauliusguzas@Pauliuss-MacBook-Pro graybox %

@pavjacko still getting error, though different this time

fixed

released in 0.21.0


Use GitHub Actions

@phryneas

thanks!

Whoops, this failed. I don't know why or if it's related to GitHub Actions though.


WebP support?

Hello,
Is that possible to add WebP support to your great tool?

While technically the library in use (libvips) can handle WebP and a variety of other formats, the project is pretty much hard-coded to only work with jpeg.
I would need to take a deeper look but files like photo_filter.rb, rename_photos.rb etc would need to implement code depending on the image format by the file's extension.

Good news, months ago I already played around with replacing Exifr with Exiftools which supports more than jpeg and tiff files. I dropped a WebP file in the dev tree and it worked right away. :) Going to prepare a merge after some more testing.

@waschinski website building seems to be longer than before. But it works se well 🎉

@waschinski website building seems to be longer than before. But it works se well 🎉

Thanks for the feedback, great to hear it works well so far. :)
I have seen some rendering issues on the demo page but nothing I can reproduce consistently on that remote side nor locally. If you happen to come across something strange please let me know.

That's a pretty huge increase. My guess is that exifr was a lot faster than exiftool but I would need to get some confirmation on that.

The exiftool website is mentioning performance and ways to improve it. I will try looking into options provided by the [exiftool.rb](https://github.com/exiftool-rb/exiftool.rb.


Die Hard was released in 1988

In MoviesProvider.cs in the Samples, it says 1998.  (I didn't want to sully your great work by submitting this as a bug. :) ).
Excellent work though, thanks.

Hi @SteveSmith16384 I fixed it. :D
Thanks for the appreciation!


Q: Best practice for managing links (Boost development use case)

Your Windows build number: Windows 10 Version 1709 (Build 16299) with latest Ubuntu installed from the Microsoft Store.

TL;TR: Shall I mklink in Windows or ln -s in WSL to get links right for development of a (C/C++) project?
Goal
I aim to set up for Boost C++ libraries development workflow with support
for building inside both environments, Windows and WSL. Specifically, to target

Windows building from VS2017 (MSVC) and CLion (MinGW)
WSL building from VS2017 (GCC) and CLion (GCC)

I will maintain separate parallel clonse of Boost git repositoru:

d:\boost.win - for the Windows builds
d:\boost.wsl - for the WSL builds

Although, single source tree might work well, who knows, I feel like it is cleaner and safer way to go.
Preparation
The Boost getting started shows the following
steps to clone and prepare the Boost source code tree:
git clone --recursive https://github.com/boostorg/boost.git
cd boost
git checkout develop # or whatever branch you want to use
bootstrap            # ./bootstrap.sh or .\bootstrap.bat, builds b2 program
b2 headers           # fills virtual /boost directory with links to libraries/headers

The interesting part is the last step: b2 headers.
What it does is also explained in the getting started:

/boost sub-directory is now a "virtual" directory, containing links to the headers.
The command b2 headers creates or recreates the contents of this /boost directory.

So, b2 header creates links using mklink or ln -s, depending on OS:
mklink-or-dir boost\numeric\ublas
symbolic link created for boost\numeric\ublas <<===>> ..\..\libs\numeric\ublas\include\boost\numeric\ublas

Questions
Given that b2 headers manages the links:

Shall I prepare both of the source trees in Windows?

d:
cd \

git clone --recursive https://github.com/boostorg/boost.git boost.win
cd boost.win & b2 headers

git clone --recursive https://github.com/boostorg/boost.git boost.wsl
cd boost.wsl & b2 headers

Or, Shall I prepare the boost.wsl tree in WSL?

c:\> ubuntu.exe
cd /mnt/d
git clone --recursive https://github.com/boostorg/boost.git boost.wsl
cd boost.wsl & b2 headers

Which option is best for things like using IDE in Windows to cross-platform build, browse code, etc. in WSL?

If I go for the option 1. all the links are correctly visible and resolved for both environments:

Windows - all good

D:\boost.wsl> dir boost
 Volume in drive D is Data
 Volume Serial Number is 1CFA-E5AD

 Directory of D:\boost.wsl\boost

2018-04-08  12:00    <DIR>          .
2018-04-08  12:00    <DIR>          ..
2018-04-08  11:59    <SYMLINKD>     accumulators [..\libs\accumulators\include\boost\accumulators]
2018-04-08  11:59    <SYMLINKD>     algorithm [..\libs\algorithm\include\boost\algorithm]
...

D:\boost.wsl> dir boost\core\
 Volume in drive D is Data
 Volume Serial Number is 1CFA-E5AD

 Directory of d:\boost.wsl\boost\core

2018-04-09  15:25    <DIR>          .
2018-04-09  15:25    <DIR>          ..
2018-04-09  15:25             6,149 addressof.hpp
2018-04-09  15:25             1,588 checked_delete.hpp
...

WSL - all good too

$ ls -l /mnt/d/boost.wsl/boost
total 0
lrwxrwxrwx 0 root root   0 Apr 8 11:59 accumulators -> ../libs/accumulators/include/boost/accumulators
lrwxrwxrwx 0 root root   0 Apr 8 11:59 algorithm -> ../libs/algorithm/include/boost/algorithm
...

$ ls -1 /mnt/d/boost.wsl/boost/core
addressof.hpp
checked_delete.hpp
...

If I go for the option 2. then the links created in WSL are JUNCTIONS-s visible in the Explorer as flat files (same as in https://github.com/Microsoft/WSL/issues/2898#issuecomment-362511510) and there seem to be problem with accessing them from Windows:

Windows

D:\boost.wsl> dir boost
 Volume in drive D is Data
 Volume Serial Number is 1CFA-E5AD

 Directory of D:\boost.wsl\boost

2018-04-08  12:13    <DIR>          .
2018-04-08  12:13    <DIR>          ..
2018-04-08  12:13    <JUNCTION>     accumulators [...]
2018-04-08  12:13    <JUNCTION>     algorithm [...]

D:\boost.wsl> dir boost\core\
 Volume in drive D is Data
 Volume Serial Number is 1CFA-E5AD

 Directory of d:\boost.wsl\boost\core

File Not Found

D:\boost.wsl> cd boost\core\
The directory name is invalid

WSL - of course, all good as expected

Possibly, those questions are also relevant for use case of git-managed symlinks: git clone -c core.symlinks=true --recursive ....

With the upcoming release of SCU, you will be able to launch VS Code from WSL if you want to go down that path noting GUI is still not officially supported. Since MS has done some work for Windows VS Code interop with WSL, I think this will be your best bet. You can even bring up the VS Code CLI for whichever WSL distribution you are working in and works better than the default console at this point in time. Also, should you ever need to reinstall a distro to resolve an issue, having the project stored on Windows side will prevent it from being removed when you uninstall and save you time and effort of backing up your data before hand if it's even possible for you to do so. Also, interop has greatly improved with SCU so I recommend upgrading once it's released. There may still be issues but at that point, an strace would determine where the issue lies and is most likely already known if it's similar to git-managed symlinks. I would check out the developer blogs to see what's new with SCU.

@DarthSpock

With the upcoming release of SCU, you will be able to launch VS Code from WSL if you want to go down that path noting GUI is still not officially supported

I do not require running any editor from inside WSL.
I clearly stated I'm aiming at the VS2017, that is the Targeting the Windows Subsystem for Linux from Visual Studio, ie. the "Linux development with C++" feature. Although the CLion use case I mentioned is exactly the same, let's forget about CLion to keep the issue foused.
Please, let's forget the VS Code.

VS2017 is available for both and I was addressing your number 3 for IDE. Sorry for providing a tidbit that may or may not be useful to you. There are many things in the blogs that could prove useful to your case. For example, with symlinks, there is much improved support such as / vs \ not being a factor. Another is Linux permissions being preserved on Windows from WSL. Case sensitivity on a per directory level is also available.
So shorter answer, which I probably should of done in first place:

Shall I prepare both of the source trees in Windows?

Yes
Or, Shall I prepare the boost.wsl tree in WSL?
No, and I was trying explain benefits of 1 vice 2 in less-technical terms. I won't make that mistake again.

If I go for the option 2. then the links created in WSL are JUNCTIONS-s visible in the Explorer as flat files

Per the referenced issue:

It seems to work if you have a recent Insiders version and developer mode activated.

SCU contains all fixes and features from Insiders obviously so therefore as stated above, upgrade and enable developer mode.

It appears no one has a particular opinion on how you should approach your scenario.


pool should use sync.Pool

the use of channels to control the pool is fine, but IMHO it's much better to use the sync.Pool from the stdlib

further more, a pointer to a channel is not usable using the receive/send operator <-


DatabaseMetaData.getDatabaseProductVersion should return the DB edition too.

Is your feature request related to a problem? If so, please give a short summary of the problem and how the feature would resolve it
We have some code that does different stuff depending on the DB being enterprise or standard edition. For oracle we can use getDatabaseProductVersion which returns the edition too, it would be good if sqlserver would to the same.
Describe the preferred solution
Have "Standard/Enterprise Edition" within the version string.

That sounds like a breach of contract https://docs.oracle.com/javase/8/docs/api/java/sql/DatabaseMetaData.html#getDatabaseProductVersion--
You would need to use https://docs.oracle.com/javase/8/docs/api/java/sql/DatabaseMetaData.html#getDatabaseProductName--  for getting  the product name

I would also rather expect the information about the edition to be returned by getDatabaseProductName, I have mentioned the other method only because in case of oracle it is returned by that method.
According to my test none of the methods returns the database edition. Do you get the edition as part of getDatabaseProductName?

but you've asked for a modification of getDatabaseProductVersion and not getDatabaseProductName
as you can see in the code getDatabaseProductName returns a hardcoded stringinstead of return what select @@version produces.
see also https://sqlhints.com/tag/sql-version-to-product-name-mapping/

I've asked for the modification of getDatabaseProductVersion because oracle is putting the edition information in there already, but I do agree that it should have been getDatabaseProductName all along.
And yes, I already looked up getDatabaseProductName and noticed that it returns a hard coded string.
In any case I would like to have the information about the DB edition, regardless of the method that returns it.

Is the suggestion to return @@VERSION from getDatabaseProductName()?

@ulvii Yes.

Thanks @JSamir , we will look into this.


Use "closed" instead of "disconnected"

Addresses https://github.com/crossbeam-rs/crossbeam/issues/256#issuecomment-444606458

My main worry about this change is that it would break too much existing code for very little benefit. To see if that is true, I cloned the Servo repository and grepped it for 'disconnected'. I only found one instance of breakage:
https://github.com/servo/servo/blob/a5a21a59addae0df6d9e050f17d44399db04fec3/components/script/dom/window.rs#L1735
This makes me feel better considering that Servo is a big codebase. Still... I'm a bit worried that other projects might be matching on error types more often. Crossbeam might have reached the point where it's so widespread that stability is preferable over minor API improvements.
What do you think?

Hmm, given that the current term is not wrong, the cost of changing this will probably be greater than the benefit. (I found nearly 50 "Try(Send|Recv)Error::Disconnected" in tikv.)
What I was thinking was mainly about the impact on methods that may be added in the future, such as close and is_closed. (As time goes by, it becomes more difficult to standardize terms within the ecosystem. But in reality, it may already be impossible...)

Yeah I think that ship has sailed :) We'll probably never agree on a 'standard' channel interface, but that's okay. I don't think it's a super important problem worth solving. We're fine with a bunch similar but slightly different interfaces...

Ok! Closing in favor of the above opinion that the benefits of doing this are not enough.


Cannot find Mapbox API at https://a.tiles.mapbox.com

When I am trying to launch the Mapbox Studio, in the landing page it says Connect to Mapbox. I click on Connect option and the above error message comes. Any suggestion to resolve it?

@Ranajay Are you behind a proxy?

Hi there, got the same problem. I test your suggestions on post 860 https://github.com/mapbox/mapbox-studio-classic/issues/860
i got no proxy server (but it's from my workstation, maybe others restrictions)
No automatic configuration script
Can't ping a.tiles.mapbox.com, requests send but none received
I can  access https://a.tiles.mapbox.com through a web browser, from chrome it says {"api":"mapbox"} and from IE it tries to dl a .json file.
Thanks in advance for the help :)

@Faethor thanks for the detailed information.
Since you cannot ping from the command line, I think there is something blocking your requests to https://a.tiles.mapbox.com
1.)
Can you look at those sites and see, if they think you are behind a proxy:
http://amibehindaproxy.com/
http://www.lagado.com/proxy-test
http://whatismyipaddress.com/proxy-check
2.)
Try to uncheck Automatically detect settings.
Are you still able to access the internet and https://a.tiles.mapbox.com/ in particular?
If you can't, then there is some automatic network configuration going on.

@BergWerkGIS
none of these site detect a proxy server
After unchecking the Automatially detect settings, no access to  https://a.tiles.mapbox.com/ or other internet website

Thanks, now we know for sure, there is something blocking your traffic.
In case there is an IT department where you are, best guess would be to ask them.
If this is not the case, you might have an additional firewall (Norton, McAfee, ...), that you have to configure manually.
Or switch off temporarily to see, if it works then.

Deactivate my Windows firewall from Avira Antivrius, no change. no ping return from a.tiles.mapbox.com
I'll try to reach our IT guy.
Thanks a lot

I am experiencing the same problem: Cannot find Mapbox API at https://a.tiles.mapbox.com I am behind a corporate firewall but not behind a proxy plus I can access the website via browser which then shows {"api":"mapbox"}.
Any suggestions?

@Steeefan that proves, that something is blocking Mapbox Studio Classic's internet access.
If you are sure, you don't have a proxy your corporate firewall might be configured to only let pass traffic that has proper user-agents set for each request.
We rely on several 3rd party modules, that we have no control of, and not all of those set the a proper user-agent.
If this is the case, your best bet is to work with your IT department to not block Mapbox Studio Classic's requests.
Before you do that, try these:

open a console and enter ping a.tiles.mapbox.com. Do you get something like below, or an error message?

Reply from <IP_ADDRESS>: bytes=32 time=47ms TTL=52
Reply from <IP_ADDRESS>: bytes=32 time=47ms TTL=52
Reply from <IP_ADDRESS>: bytes=32 time=48ms TTL=52
Reply from <IP_ADDRESS>: bytes=32 time=48ms TTL=52

try the services linked above, if they think you are behind a proxy. If they are positive you need to find out your proxy settings.

I already tried the services. They tell me No proxyserver detected., This request appears NOT to have come via a proxy. and Proxy server not detected.
a.tiles.mapbox.comis available via browser to me but indeed not via ping:
C:\>ping a.tiles.mapbox.com
Ping wird ausgeführt für d3eju24r2ptc5d.cloudfront.net [<IP_ADDRESS>] mit 32 Bytes Daten:
Zeitüberschreitung der Anforderung. //4x, timeout
I also tried pinging the IP address <IP_ADDRESS> you were referred to while pinging: no luck.
Thanks for pointing out the obvious -.- After being able to reach a.tiles.mapbox.com in a browser I didn't feel the need to ping.

Hi BergWerkGIS, i tried everything from the above, I'm behind no proxy (determined by all of these sites) and i am able to ping a.tiles.mapbox.com got 7-14ms ping on that.
Do you have any clue what else this could be?

@Chricky do you have a firewall. Maybe that is blocking Mapbox Studio Classic traffic?

I'm using Windows Firewall. But if it was blocking the connection the ping shouldn't work too?

@Chricky open a console and try this:
cd "C:\Program Files\mapbox-studio\resources\app\vendor\"
node -e "var h=require('https');h.get('https://a.tiles.mapbox.com',function(r){console.log('ok:',r.statusCode);}).on('error',function(e){console.log(e.message);});"

Is the output
ok: 200

or something else?
If something else, paste the whole message here.

The output is:
connect ETIMEDOUT

@Chricky That shows that something is blocking Mapbox Studio Classic's internet access.
If you are absolutely sure that there is nothing else between Studio and the internet, that might intercept connections, the only thing I can think of, is that the installer was not able to modify the firewall rules.
Goto Control Panel -> Windows Firewall with Advanced Security -> Inbound Rules.
Check if there is an entry for Evented I/O for V8 Javascript.
If there is, make sure it points to C:\Program Files\mapbox-studio\resources\app\vendor\node.exe.
If there is not, create a new inbound Rule:

Program
This program path -> C:\Program Files\mapbox-studio\resources\app\vendor\node.exe
Allow the connection
Just Private, uncheck Domain and Public
Name -> Evented I/O for V8 Javascript

@BergWerkGIS There were 2 Rules, both set to domain. I unchecked Domain there and Checked Home/Work and i still get the same error :(

@Chricky out of ideas at the moment. We just know, something is blocking traffic.
Maybe try to switch off Windows Firewall completely (just for testing, not recommending this for everyday use) and try again, if node can connect with above procedure.


rake Thor::Base error after adding guard-rspec to Rails 4 mountable engine gemspec

I am using a --mountable Rails engine and rspec. If I add this to my gemspec and run bundle s.add_development_dependency 'guard-rspec' then the rake command (with or without bundle exec) does this:
~/projects/project_name(branch:master*) » rake
rake aborted!
NameError: uninitialized constant Thor::Base
/Users/brian_b/projects/project_name/Rakefile:22:in `<top (required)>'
(See full trace by running task with --trace)

If I instead specify it with  s.add_dependency 'guard-rspec' then it will work fine. This is obviously not optimal, as clients shouldn't have to install guard-rspec to run the engine.
REPRODUCTION: This is with a brand new fresh project with no modifications, after following this guide. http://viget.com/extend/rails-engine-testing-with-rspec-capybara-and-factorygirl The instructions can be followed here for an exact replication of the issue, just add guard-rspec as a development_dependency.

Seems fixed (closed: https://github.com/bundler/bundler/issues/3205) - please reopen/comment if not. Thanks!

@bbugh  your trick (moving Bundler::GemHelper.install_tasks to the top) did the trick for me, with regards to a Rails gem, using minitest


When brush is >1 tile, only 1 tile is shown when hovering over the map

This is not very intuitive: when I first encountered this behaviour, I thought I can’t use brushes >1 tile in R48.

I'm actually pretty sure this is fixed in master by now, but do check first.

Just tested this, it works like a charm now, thanks!

Close as it apparently works.


eventFlowConfig.json not auto created in VS2017

@Lybecker : Thanks for your contribution to the Azure documentation! The author, @dkkapur, has been notified to review your proposed change.

@Lybecker thanks!
#sign-off


Cannot observe properties of nested elements in Safari 8.0 (iPad, iOS 8.1.3) in Polymer.dart

 Issue by austinov
Originally opened as dart-lang/sdk#22275

Attachment:
issue.tar.gz (1.68 KB)

 Comment by austinov

This effect is due presence of affirmative or dismissive attributes on paper-button in paper-action-dialog.
Without them the parent-element observes change of the property in the child-element.
I replaced the paper-action-dialog to paper-dialog and it works on Safari.