supportedViewports missing for APL in the skill.json validation

Description:

Steps to reproduce the issue:

Edit the skill.json
Add the interface

{
            "type": "ALEXA_PRESENTATION_APL",
            "supportedViewports": [
              {
                "mode": "HUB",
                "shape": "ROUND",
                "minWidth": 480,
                "maxWidth": 480,
                "minHeight": 480,
                "maxHeight": 480
              },
              {
                "mode": "HUB",
                "shape": "RECTANGLE",
                "minWidth": 1024,
                "maxWidth": 1024,
                "minHeight": 600,
                "maxHeight": 600
              },
              {
                "mode": "HUB",
                "shape": "RECTANGLE",
                "minWidth": 1280,
                "maxWidth": 1280,
                "minHeight": 800,
                "maxHeight": 800
              },
              {
                "mode": "TV",
                "shape": "RECTANGLE",
                "minWidth": 960,
                "maxWidth": 960,
                "minHeight": 540,
                "maxHeight": 540
              },
              {
                "mode": "HUB",
                "shape": "RECTANGLE",
                "minWidth": 960,
                "maxWidth": 960,
                "minHeight": 480,
                "maxHeight": 480
              }
            ]
          }

Observed result:
Warning: Property supportedViewports is not allowed.
Expected result:
No Warning because supportedViewports is required to add APL support to all Alexa devices since it is possible to add APL for single devices
Additional environment details (Ex: Windows, Mac, Amazon Linux etc)
Mac
VS Code version:
1.38.1

Hi @kgnuuk
Thanks for reporting this issue. Its a known json validation issue and We are currently working on it.
We inform you once we got it fixed.
Thanks,
Shen

Hi @kgnuuk
We just published v1.0.2 and this issue should be fixed after upgrading the extension. Will close it for now and please feel free to reopen if you meet any problems.
Thanks,
Shen


PostCSS support

PostCSS now is a big player at CSS processors market. More and more peoples start using it, and, I sure, many of them wants to use CSScomb with it. In many cases CSScomb in SCSS mode can process CSS with PostCSS plugins syntax, but sometimes it's fails and can't parse file.
PostCSS and CSScomb both going to their big releases. PostCSS 5.0 will support custom parsers. CSScomb 4.0, as I know, is heavy refactored. Maybe it's time to drop Gonzales PE and use PostCSS parsers for CSScomb?
PostCSS syntax can be very different because of used plugins and developers wants to have their source files organized.
This is example of PostCSS code, that current CSScomb (in Sublime Text) fails to process:
.button {
    height: 40px;
    border: solid transparent;
    border-width: 2px;
    border-radius: calc(@height / 2);
    line-height: calc(@height - @border-width * 2);
}

Fails because of @ in calc(). Used PostCSS plugins: postcss-property-lookup and postcss-calc.
More PostCSS-plugins syntaxes. Mixins are different from SCSS, for example, and there in no way to sort them now like we did it recently in this PR:
@define-mixin icon $name {
    padding-left: 16px;
    &::after {
        content: "";
        background-url: url(/icons/$(name).png);
    }
}

.search {
    @mixin icon search;
}

Following code taken from cssnext:
/* custom properties & var() */

:root {
    --fontSize: 1rem;
}
h1 {
    font-size: calc(var(--fontSize) * 2);
}

/* custom media queries */

@custom-media --small-viewport (max-width: 30em);

@media (--small-viewport) {
    /* styles for small viewport */
}

/* media queries ranges */

Allows to replace min-/max- with <= & >= (syntax easier to read)

@media (width >= 500px) and (width <= 1200px) {
    /* your styles */
}

/* or coupled with custom media queries */

@custom-media --only-medium-screen (width >= 500px) and (width <= 1200px);

@media (--only-medium-screen) {
    /* your styles */
}

What you think about it, @tonyganch and @ai?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/25675165-postcss-support?utm_campaign=plugin&utm_content=tracker%2F214563&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F214563&utm_medium=issues&utm_source=github).

We just dicussed it yesterday.
the current problem: PostCSS 5.0 has only CSS and SCSS parsers. In next month I have no plan to start Sass, Less and Stylus parsers.
ut I will write good docs about writing a parsers ;).

I’m interesting in CSScomb :heart: PostCSS. The file I have (I use postcss-nested):
.header
{
    color: #fff;
    background-color: $darkGray;

    .wrapper {
        display: flex;
        flex-direction: column;

        @media (--tablet)
        {
            flex-direction: row;
        }
    }
}

and the error I get at csscombing:
Please check the validity of the block starting from line #6

4 |     background-color: $darkGray;
6*|     .wrapper {
7 |         display: flex;
8 |         flex-direction: column;

@isqua if you use CSSComb in text editor, you can set syntax to SCSS and CSSComb will work with code you provide.

@hudochenkov, I use it from console, and I want to lint files too. I can rename all *.css files to *.scss, but I don’t think it’s a good idea. I’ve created an issue https://github.com/csscomb/core/issues/6 for defining syntax by extension in config.

To use CSSComb right now in Sublime Text we need to use scss scope in editor to make CSSComb work in limited mode (not all syntax changes CSSComb can parse). It's better than nothing :) Recently I've made PostCSS syntax highlighting for Sublime Text and forced to use source.scss in order to use CSSComb on files highlighted with this syntax. This also have downside as interference with SCSS syntax highlighting, because of using the same scope. While plans for PostCSS support in CSSComb is not known yet, can we add source.postcss scope as kinda alias for source.scss to CSSComb 3? It will provide same limited functionality as now, but it will add more independence to PostCSS syntax highlighting. What is the right way to do it?

@hudochenkov, I believe this is actually what @isqua has implemented: map one syntax to another.
Unfortunately, I'm away from my computer and will be back only in a week or so, I'll roll a release once I'm able to.

@tonyganch As I know @isqua added this for CSSComb 4. If you will be able to add this in current version of Sublime Text plugin it would be great :) Thank you for quick reply :)

Hi, do you know when will be released CSSComb 4?

@hudochenkov, I can't, I've created a total dependency mess there ¯\(ツ)/¯
I'm going to release<EMAIL_ADDRESS>with linters api and then add linters themselves in options as a major release. It does not make any sense to postpone features that are ready now any further.
// cc @senechaux

@hudochenkov, btw I'm looking for any help with this project or maybe even a new maintainer. If it sounds kinda interesting to you or you know a guy who knows a guy who is a good match, let me know :)

@tonyganch that's great! Can't wait :)
Sadly I doesn't know JS enough to support this huge project. We can ask in community accounts @jsunderhood and @cssunderhood at Twitter.

@hudochenkov, I'm trying to search among familiar faces first :)

@ai Doesn't postcss-sorting effectively replace CSScomb?

@danielbayley I think it could replace CSScomb. But CSScomb team don’t want to merge.

@danielbayley postcss-sorting effectively replace only CSSComb's sort-order option. And only for CSS, SCSS, Less and other syntaxes PostCSS can parse. It can't parse Sass (indent based syntax).

I have used postcss-sorting as visual studio code plugin with postcss-nested syntax successfuly

It can't parse Sass (indent based syntax)

It doesn't seem far off http://postcss.parts/?searchTerm=sass. Besides, SugarSS basically replaces that anyway.
I suppose the equivalent of CSScomb's functionality would be a series of PostCSS plugins, with sorting being just one…

@danielbayley yeap, code style features already can be implemented by stylelint and stylefmt.


Wadl Import Failed

Hi,
Importing the wadl from pentaho bi-platform doesen't work. I tried import from URL, File and Raw Text.
This is the link to the wadl.
http://javadoc.pentaho.com/bi-platform500/webservice500/application.wadl
Thnaks.

I think there may be some misunderstanding of what this project is.  This project is a plugin for ServiceStack to take APIs created in that framework (ServiceStack) and generate Postman collections.  This project doesn't import external service descriptions, but rather creates the JSON necessary for importing your ServiceStack service into Postman for testing the service.

My bad, in my search frenzy I didn't notice that I got to ServiceStack. I tought I was still on Postmans Issues page. Sorry for wasting your time. Please close this.

No worries!


Remove yarn install during build

Per the discussion in rails/cssbundling-rails#45, this parallels rails/cssbundling-rails#50 to remove yarn install from javascript:build. In its place, a FAQ is added for those that want to replicate prior behavior.

This is a breaking change, so I guess there should be a way to configure which behavior to use, but I would really like to see this merged. Alternatively, yarn install should have --frozen-lockfile added (which is also a breaking and less desirable change).

Same opposition as https://github.com/rails/cssbundling-rails/pull/50.


Fix JSONDecodeError import (solve #973)

Solve issue https://github.com/huggingface/huggingface_hub/issues/973.

Thank you for reporting the issue 😉

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Issues with the ping

The ping command sometimes doesn't return anything. Status channels display wrong status. (Reported by users)

The ping command has been fixed in be2acac748e2d05922944b4e88fd132c59a65d27. The ping library (minecraft-server-util) wasn't handling the request timing out. I wasn't able to reproduce the issue with the status channels locally.

fixed the auto pinger in commit 262d37c8e97a9882ecd5b25b4d8599083e520428 and the ping command will be fixed in a later update totally didnt forget to fix that


The up and down keys is not functioning in the profile page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to settings > preference > use the up and down keys on the keyboard and observe that they are functional
Repeat the above step for the "about" page in settings
Repeat the above step for the "profile" page in settings

Expected Result:
The up and down keys on the keyboard should be functional in the profile page
Actual Result:
The up and down keys on the keyboard are not functional in the profile page
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.59-2
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/c3b03307-1037-479e-8684-2f4e92e5c9c7
Expensify/Expensify Issue URL:
Issue reported by: @Yokabdk
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1692056740867719
View all open jobs on GitHub

Not sure I understand what the problem is or if this is needed.

@muttmore The arrow keys are not functional in the "profile" page which is inconsistency


Chore/allow protoc path from env var

Not sure if the included binary actually runs on OSX, but simple change to allow setting of path to the platform specific binary via environment variable.

Now we just wait for travis to see if it works! :)

🤔 It doesn't like the relative import reaching outside fo the tests package. I can just make it an absolute import since different configuration for test vs normal is prob not worth it. Uno momento
Travis is great btw

It is saying it is a top level package because you do have the python init.py

No __init__.py in the tests directory, which should make it not a package at all.
I haven't used pytest much so not sure how it interprets that. More familiar with nose and django test runner so not sure how no __init.py__ gets interpreted.
The absolute import is better anyway; easier to move them around. Passed this time though just need another review since i made a change.


Example route configuration and documentation

This is great, Plamen! Seems like a much more mature and sustainable way of organizing the application long-term. Thanks for getting this kicked off.


Closing Project dialog is shown for 30+seconds with Neon plugin enabled

Describe the bug
When switching projects or closing phpStorm the Closing Project dialog is shown for 30+ seconds and hangs the process

Environment (please complete the following information):

PhpStorm 2023.1
0.5.3 updated today

To Reproduce
Steps to reproduce the behavior (or attach video):

Open a project
Switch to another project using cmd+o and select project - or just quit phpStorm
See that the closing dialog shows for a very very long time

Additional context
Disabling the Neon plugin fixes the problem

Same problem here

@mesour Any update?

I found the problem. I will make new version until the end of the week at most.

Please fix it 😇

@badpenguin yes. Sorry, I didn't have time for that. I keep it in my mind.

Fixed in 0.5.4 it will be available in 2 business days

0.5.4 Works for me, thanks


Tested and working idp's

So the documentation states:

Passport-SAML has been tested to work with Onelogin, Okta, Shibboleth, SimpleSAMLphp based Identity Providers, and with Active Directory Federation Services.

But i wanted to start a tread where people could give feedback if they found it working with these and other not noted services
Remember to check if its been stated already, then if you have it working give it a 👍 and if you have it not working please use the 👎  instead, that way we keep the issue short and tidy

Onelogin is working 👍

I've tested it with Onelogin, Okta, ADFS and many Shibboleth servers.
This module is generally spec-compliant, so it should work with spec-compliant servers. However, it has some missing features. The issue I've run into is not so much with different providers, it's when the idP wants to use some feature that's missing or not well supported here.  So far I've got about a dozen or so integrations working with it. The one missing feature I needed was better support for multiple SAML providers configured at once, and I patched the module to improve that support.
The existing issues posted in this issue tracker provide a good sense of the places where people have run into rough edges.

I'm using this branch to dynamically handle configs:
https://github.com/bergie/passport-saml/pull/276

I'm closing this. Just because it works for one person with their IdP in one configuration isn't a guarantee it will work for you with your configuration. A more useful reference could be a list of SAML spec features that we support, or a list of SAML features that are known to be missing.
Also, we have a wiki for collaborative documentation.


The _Init() method doesn't work in C#

Godot version: 3.1.1 and 30e16fff57cfdeb79f8e9ee6a7516e7d7cd6d376
OS/device including version: Ubuntu 18.04 64-bit
Issue description: It seems that _Init doesn't work at all in C#. @neikeq
Steps to reproduce: Open and run the reproduction project, which contains the following:
public override void _Init()
{
    GD.Print("This text is never printed.");
}

In 3.1.1: The code in _Init() isn't executed at all.
In master: 'TestNode._Init()': no suitable method found to override (CS0115)
Minimal reproduction project: OverrideVoidInit.zip

What would _init be for in C# since it has constructors?

Duplicate of #22633
_Init never did anything in C#, and as I recently commented:

I'm removing _Init for now in #29140, but I plan to add a working implementation in the future ...

For API parity, I think _Init() should exist for C#. Or documentation should explain how to replicate the functionality with a C# constructor.


Fix missing .ConfigureAwait(false)

Ensure we always call .ConfigureAwait(false) and enforce from now on.
All code changes were purely mechanicall, made by running the code fix over the project in VS.
Very important and timely change. Can you tell its connect/perspective season 😬

For example, if code is in VS and has used JTF.SwitchToMainThreadAsync()

To be honest, I was pleasantly surprised that there wasn't any violations of this in our CPS code that I had to review, though that also has me slightly worries that we're doing the wrong thing there :)

Updated rzls, and did another thorough check of the changes in the Workspaces layer, to ensure they're not introducing anything odd.


types(defineSlots): Support for generic keyof slots

related https://github.com/vuejs/language-tools/issues/3141
See comment: https://github.com/vuejs/language-tools/issues/3141#issuecomment-1546549681
Currently defineSlots with keyof generic has the error of // Type '{}' has no call signatures.ts(2349),  this PR fixes that error.
The issue is caused by Readonly, it seems it changes the type for calling the function. I think the type readonly safeness on typescript is less important than having generics working properly.

<template>
  // error here
  <slot :name="`label:${tab.code!}`" :tab="tab" :label="tab.label">
    <slot name="label" :tab="tab" :label="tab.label">
      {{ tab.label }}
    </slot>
  </slot>
</template>

@pikax Hi, I tried it and it seems not fix https://github.com/vuejs/language-tools/issues/3141 now. Is there anything I miss?

In that demo. I update vue to 3.3.4 and Change StrictUnwrapSlotsType's Readonly<T> to T. And I found

<template>

            <slot :name="`header-${col.name}`" :col="col"></slot>

</template>

@pikax I want to point out that we also have additional slot that is concated using & like in your PR's test but that isn't present in your self contained example.
Additionally, I want to point out that user might separate component's metas like props, slots, emits in another ts file and import it in SFC like I do in my UI lib anu. Thing to consider it when we import object from another file we need to use as const sometimes to make it work as expected. e.g. vue-component-meta's docs: https://github.com/vuejs/language-tools/tree/master/packages/vue-component-meta#required (notice as const)

Any news on merging this?
I really miss this feature =)


Support for file/directory exclusion

Is there support for exclusion criteria; such as for files or directories?

I should add that the files I want to exclude are JSON streams such as ndjson, jsonl, or concatenated JSON. These will cause a failure with this Action, as they match the pattern
{
    // valid JSON object 1
}
{
    // valid JSON object 2
}

Currently there is no mechanism to specify exclusion rule, but it should be possible to exclude targets with the proper writing of regexp.

Thanks for the reply. I will look into the regex pattern to exclude files.
Just as a fyi, Ruby's Yajl, seems to be a drop in replacement for JSON that also handles JSON streams if you ever want to extend the JSON functionality.
https://github.com/brianmario/yajl-ruby


Update README with new names

JSONValue -> JSON
JSONString -> String
Also reword a section or two for clarity
I don't think I'm going to merge this till we start pushing 1.0
releases, since that will lead to confusion with people using Argo now.
Maybe we use this branch to collect all README updates?

Besides me contemplating the true meaning of words , this is :+1:

This update is going to be way more substantial with all of the other changes coming down the pike. I'm going to close this in favor of a larger overhaul later on.


Upgrade faker to version 1.6.0

More info: https://libraries.io/rubygems/faker/1.6.0

Prefer https://github.com/24pullrequests/24pullrequests/issues/970


attacker.target work only if the NPC is casting spells

attacker.target should return the best target that the NPC is fighting with. But right now, if the NPC isnt casting magic, the target will always be -1. It used to work on older X versions.

oh for NPC not for player

oh for NPC not for player

Yup, NPCs

fix sent

Good, thanks!


Added windows support

Require this PR https://github.com/ros2/rcutils/pull/473 in rcutils
The code just compile, I didn't try the code yet.

Good and bad news,
On Linux this changes are not generating any new failures, but on windows I can see the following:

test_services__rmw_zenoh_cpp
test_subcription__rmw_zenoh_cpp
test_service_event_publisher__rmw_zenoh_cpp
test_gt_type_description_service__rmw_zenoh_cpp
test_services__rmw_zenoh_cpp (Timeout)

Just in case, change base to rolling-0.11

Closing this one, as we merged support for Windows into rolling in #312.


The target for json output is hardcoded in doc-sync even if it is overriden by .cargo/config

This is the tool I was looking for, thanks for sharing!
I have one issue through. When the target directory (for executing out-of-source builds) is specified in .cargo/config, this app is unable to find JSON output anymore (as it is automatically redirected by rustc outside of ./target dir which is hardcoded here in CLI).
I believe the best way would be to add another verb to CLI toolset, like json-to-markdown which would take already built JSON files (by given directory name) and convert them to markdown (by given output directory). This way it can be integrated with build scripts.

This falls under the "doesn't work with workspaces" category. Feel free to make a PR, but I don't think I'll be working on this.
Also in the README, a library is planned in the future for use in build.rs.


Add small guide for using whitelists + blacklists on a server

With #3332 merged #1608 is complete and servers can now configure whitelists & blacklists. But we don't have a small section for that yet in https://github.com/MovingBlocks/Terasology/blob/develop/docs/Playing.md - goal of this easy issue is to test out the new functionality and add the instructions :-)
Additionally Playing.md's multiplayer section has only grown over time to where it takes some scrolling to get to the actual "How to play" part - I'd like to see the entire multiplayer section moved to the bottom so the normal playing instructions come first.

Another tiny related doc item we might as well do as part of this: add the new toggleSunHalting command from #3312 to the sample list of commands since it is pretty useful to know about.

Hi!
I've been looking into this issue and the whitelist / blacklist functionality, and I may have found a small bug.
When running Terasology on the latest build, the whitelist and blacklist JSON files do not generate. What I eventually found was that only headless servers generate them, and normal clients will not. Once they're generated everything works fine, but I wasn't sure if this was intentional. If it is, I'll definitely include it in the changes to Playing.md. If not, it's a simple one line change to fix that I could include in my PR.
Thanks!

Hey @Inei1,
Thanks for the explanation, that makes a lot more sense!
I'll take a look at creating the lists when hosting a game from a non headless server, and check out if ServerConnectListManager needs to be changed at all.

As an aside, should the code changes be part of a different issue/PR? This issue is about the changes to Playing.md (which are pretty much done).
I'm new to GitHub and open source in general, so I'm not exactly sure how the process goes, and would definitely appreciate any advice.

Hey @AdamAboudi welcome to open source :-)
In this specific case it doesn't really matter. Technically yes you can submit one doc-only PR and later on a separate code PR that doesn't address a specific issue unlike the doc tweak. But if you imagine both can be ready in short order they can be handled in a single PR since they're small and related :-)
We do generally suggest multiple PRs for bigger independent issues. That's in case one gets stuck in code review with changes needed while another is perfectly fine and ready to merge.

Hey @Cervator, and thanks for the advice!
I thought the changes were small enough to go together, but just wanted to make sure since one was a doc issue and the other a coding issue.
Thanks again!


Definition of Abstraction and Encapsulation are incorrect.

I'm sure that "Encapsulation means that a group of related properties, methods, and other members are treated as a single unit or object." But It's confusing to define that "Encapsulation means hiding the unnecessary details from type consumers." Because "hiding the unnecessary details from type consumers" is right for abstraction also. The difference is Encapsulation hides details from accessing, Abstraction hides details for unnecessaries.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ece1648c-e8a7-153d-a22b-350a619aa26d
Version Independent ID: a0c9221a-cc55-4c31-4f54-0e5fbdcbce25
Content: Object-Oriented Programming (C#)
Content Source: docs/csharp/tutorials/intro-to-csharp/object-oriented-programming.md
Product: dotnet-csharp
Technology: csharp-get-started
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Related to https://github.com/dotnet/docs/pull/22888 and https://github.com/dotnet/docs/pull/23315.

I'll add this to our backlog.
We've had several issues written for these definitions. It's clear they can be expressed better.

OK, we need to put this to rest quickly because anyone who is looking to learn C# for a job is going to fail their first interview question and then hate the language and then it will die and we will all be left with PHP or Ruby.   The actual literal definition of the word Encapsulation is 'the action of enclosing something in or as if in a capsule'.   That is about the same as making a single unit from other points of data.
Abstraction or Abstract is from a Latin word meaning "pulled away, detached,".  Think of a transmission on a car, you can see the shifter but you are not seeing the gears move in an automatic car. Think of this as an abstraction.
Final point, if you do some Googling all the blogs and forums that discuss this, follow this documentation. The world sees  Encapsulation as things placed into a single unit of data and Abstraction as hiding things or only showing what we want to show.
At this point changing these terms would be like telling the world the sky is no longer blue, it's green.

@davidbcoulson I agree that we need to address this quickly. (It was on my list for this week).
My concern with simply switching the definitions back again is that the verb "hide" applies to both encapsulation and abstraction. In your example above, you can think of an automatic transmission as encapsulating the gears and the mechanisms needed to change gear ratios. I like your statement here (grammar edits mine):

Encapsulation as is things placed into a single unit of data and Abstraction as is hiding things or only showing what we want to show.

For our purposes here, I would focus on the following key points for this terms:

Abstraction focuses on identifying the important components of a system. You'll use abstractions when you create classes to implement important behavior for your program.
Encapsulation focuses on hiding the internal complexities of a system. You'll use encapsulation when you determine what the public interface for a class should contain.

Let me know if you want to update your PR along those lines. (If you do, note that the definitions are restated at the end of the article as well as the introduction.)

I would like to address the first sentence and the list. These were always referred to as "pillars" not "techniques" for me and that seems to be the theme else were. The happy medium, I would suggest calling them "principles".  Please let me know your thoughts.
C# is an object-oriented programming language. The four basic principles of object-oriented programming are:

Encapsulation - Ability to organize internal complexities of a system into single objects.
Abstraction - Ability to access or restrict access to properties, methods, or members of something encapsulated.
Inheritance - Ability to create new encapsulations based on existing encapsulations.
Polymorphism - Ability to implement inherited properties or methods in different ways across multiple encapsulations.

@davidbcoulson I like it.  I'll make one edit:

Encapsulation - Ability to organize internal complexities of a system into single objects.
Abstraction - Ability to access or restrict access to properties, methods, or members of something encapsulated.
Inheritance - Ability to create new abstractions based on existing abstractions.
Polymorphism - Ability to implement inherited properties or methods in different ways across multiple abstractions.

My reason for replacing "encapsulation" with "abstraction" in the last two is this: A sub-class refines the abstraction of a base class. (A car is a more specific type of vehicle).
If you make those updates to your PR, I'd appreciate it. It will be a major improvement. Thanks for the thoughtful discussion.

@BillWagner Thanks for the discussion. I like this definition.

Encapsulation - Ability to organize internal complexities of a system into single objects.
Abstraction - Ability to access or restrict access to properties, methods, or members of something encapsulated.
Inheritance - Ability to create new abstractions based on existing abstractions.
Polymorphism - Ability to implement inherited properties or methods in different ways across multiple abstractions.

Please note that people are confusing about does OOP has abstraction.
Abstraction a.k.a Data abstraction is a step in system design. And Encapsulation, Inheritance, Polymorphism are three key techniques that OOP proposes to achieve Data abstraction. So, Abstraction is the bridge linking the business logic with software architectures.
Ref: Abstraction is not a principle of Object-Oriented Programming

@BillWagner I have updated the PR. Thanks for working with me on this.


[minesweeper] use event iterator; add arrow keys to minesweeper

Replaces the single byte reads with the termion event iterator, and allows arrow keys to be used for moving.

Thanks!


Why is it that the Euclidean distance function in ClusterAidedComputing calculates the L1 distance?

I made a mistake. I should use the European distance here. It's written as the Manhattan distance here

OK.Thank you very much for your reply


fix(backend): update dependencies

@stebenz @eliobischof i hope it's okay to also update the versions of the operator

superseeded by https://github.com/caos/zitadel/pull/2136


polyup adds 'fit' css rule instead of using --layout-fit

I'm not sure why polyup makes style rules for the layout attributes at all (I'd love to know)[0], but it does this for most of the layouts, but not for 'fit', even though the rule is identical :
https://github.com/PolymerElements/iron-flex-layout/blob/master/iron-flex-layout.html#L271
    --layout-fit: {
      position: absolute;
      top: 0;
      right: 0;
      bottom: 0;
      left: 0;
    };

This is what Polyup adds :
      [fit] {
        position: absolute;
        top: 0;
        right: 0;
        bottom: 0;
        left: 0;
      }

The other layout attributes are replaced like this:
      [layout][horizontal] {
        @apply(--layout-horizontal);
      }

so, I would have expected :
      [fit] {
        @apply(--layout-fit);
      }

So :

is there a reason for not using classes as described in the migration guide, and,
is there a reason for not using --layout-fit?

Thanks.

Huh, I thought that fit didn't exist in iron-flex-layout. Looks like I just missed it!


Encoding

Hi, thank you for this great Module!
In German we have "Umlaute" like äöü and so on. I did'nt found any way to handle this with your Module.
Can you give me a hint or fix somethin to do this?
Thank you very much.

Yes,
Make sure to use UTF-8 with BOM as enoding of your .ps1 file.

I don't know how to use your Screenshot.
Within the Script i tried with this Lines, but no Success:
[System.Text.Encoding]::UTF8
$OutputEncoding = [System.Text.Encoding]::UTF8
$PSDefaultParameterValues = @{ '*:Encoding' = 'utf8' }
How can i set the File or into the Script to UTF-8 with BOM?
Thanks for your Help.

No, You have to use editor such as VSCode and set encoding of a file that way. Script has nothing to do with it.
Download VSCode, download PowerShell Extension (you will thank me for that) and then simply change encoding of a file.

Ok, thank you very much. I already work with vscode for a longer time, and it was a fight to change the encoding. But with "save file with encoding" it worked well. Nice.


24 hour date format does not work

Problem
The date pipe does not respect the 24 hour format (it always shows 12 hours format)
Context
Verified on iOS emulator / 9.1

tns-core-modules: 2.3.0
tns-ios: 2.3.0

How to reproduce it

Create a sample application (tns create Sample --ng) and for example introduce the same:
app.component.ts:

import {Component} from "@angular/core";

@Component({
    selector: "my-app",
    templateUrl: "app.component.html",
})
export class AppComponent {
    public myDate = new Date("Tue Nov 15 2016 21:02:49 GMT-0300 (UYT)");
    ...
}

app.component.html
<StackLayout>
    ...

</StackLayout>

You should see H:m uses 12 hours format instead of 24, check out the screenshot:

Verified on plain angular just in case and it does work:

Please note also the 12 hours format is incorrect, even for 'jm' it appears "09 PM02" instead of "9:02 PM", i.e. PM wrongly placed and 09 instead of just 9.

Hey @ignaciolarranaga
In Angular2, there are some predefined DatePipes which you can use.
For example:
<StackLayout>

</StackLayout>

In your case you can create the Date Pipe like this

Hi @NickIliev, I might not explain it clearly, if you place this:

It doesn't show a 24 hour format, but instead the hour in 12 hour format but without the AM/PM
I also include in the same example the other format:

to show that the variable actually has the right value (21:02)

@ignaciolarranaga Yes it took me a while to get into the problem but now it is clear - the 24format is not working.Furthermore, I've notice the following at my side (reproducible in both iOS and Android)
if myDate is created with new Date() the result will be the exact hour but always in 12-hour format.
However, if myDate is created with new Date("Tue Nov 15 2016 21:02:49 GMT-0300 (UYT)"); it will be parsed as totally wrong date (at my side the result is 16.Nov.2016 02.0249 AM

Oh, got it. I didn't notice that, probably with the epoch time would be better to reproduce it (but quite less readable).
I didn't put myDate = new Date() because it will depend on the time you executed :)

I've just made a fix inside nativescript-intl plugin. Version 0.0.5 should be OK.
@ignaciolarranaga Could you please try it on your side. Please note that nativescript-angular has dependency to this plugin so you have to install 0.0.5 version before installing nativescript-angular (at least with node 6+ it works).

thanks @NickIliev it works great

I had the same issue and the below hack got it right
<Label *ngIf="student.created"
    <FormattedString>

    </FormattedString>

I face the same issue. Any fix other than Date Pipes?
I want to display my date as dd/mm  and not dd/mm/yyy, so DatePipe may not work in my case.

Solution is really simple, just use 'HH' for 24hour format and 'hh' for 12.
{{"field | date: 'HH.mm'" }}
@shaikhspear16
For custom formats just use multiple interpolations like...
{{myDate | date: 'dd'}}/{{myDate | date: 'MM'}}

Solution is really simple, just use 'HH' for 24hour format and 'hh' for 12.
{{"field | date: 'HH.mm'" }}
@shaikhspear16
For custom formats just use multiple interpolations like...
{{myDate | date: 'dd'}}/{{myDate | date: 'MM'}}

this solved my issue, thanks. it should have been in the official docs


Hosted service

I spotted barcoders on reddit and thought it looked solid, so I quickly hacked together a hosted version of it and hosted it on our startup's platform here:

If you're interested in owning/maintaining it on our platform, I'm happy to help make that happen. Otherwise, I just thought I'd share another way in which your work is being highlighted, and I'll keep an eye out for updates that we can take advantage of. Feel free to close this issue. :-)

Thanks. Your platform seems pretty cool. I'll add a link to it on my README. What is involved in me taking over maintenance of it on your platform? I was planning to make some performance updates to the SVG output soon that will impact your hosted version.

Thanks. Glad you think the platform seems cool. :blush:
We still lack a feature to directly transfer ownership, so in this case, since the wrapper I wrote was pretty minimal, the easiest approach is to just create a new "algorithm" on the platform, add these 2 dependencies:
error-chain = "0.9.0"
barcoders = {version = "0.7.0", features = ["svg"]}

then copy the lib.rs source, hit compile, then publish, and edit the description to your liking. Then let me know, and I'll effectively hide mine and redirect it to yours. It's worth noting, the edit-compile experience for rust in the web editor isn't great (long compiles) so I'd definitely recommend cloning (https://git.algorithmia.com/git/USERNAME/ALGONAME.git) for any local development.

Ok, thanks. I'll do this soon.


Helm: headless-service might require publishNotReadyAddresses: true

Hello,
as far as I understand, the nodes on kubernetes uses the DNS names from headless-service for communication. The DNS for this service is only available if the node is healty. It might be that DNS is required for communication before the node gets healthy (so that other nodes can sync with it). If so, the headless-service should have "publishNotReadyAddresses: true" set.
Kind regards,
Michael.

@micw thanks... is this basically the same issue as here https://github.com/vernemq/docker-vernemq/issues/261?
Do you have a PR?

@ioolkos I can do one (it's just a one-liner) but I'm not 100% sure if that is one of the issues causing kubernetes vmq-clusters to fail (I'm still on https://github.com/vernemq/vernemq/issues/1698).
Without the publishNotReadyAddresses: true option, the behaviour is the following:

Cluster has 3 nodes (node1, node2, node3)
each node knows the other nodes by DNS name
if node 1 goes down, the DNS name "node1" is removed
If node comes up again, the DNS name "node1" is not available until node1 gets "OK" state (which is checked via it's health endpoint)
node1 is still able to connect the other 2 nodes via DNS but the other nodes can only answer to it's IP address

So if that option is required or not hevily depends on the internals of cluster building until the "OK" state. If you have deep insight, you might be able to answer this - I do not ;-)
Kind regards,
Michael.

@ioolkos This isn't the same issue as #261


Lazy loading loads all components in first load

Version
3.10.0
Reproduction link
http://asd.com
Environment info
Windows

Steps to reproduce
See below
What is expected?
Lazy loading should only be load when spesific route visited.
What is actually happening?
Lazy loading is loading all routes.

chainWebpack: config => {
        config.plugins.delete('prefetch');
}

const LoginView = import( /* webpackChunkName: "LoginView" */ '@/views/auth/LoginView.vue');
const RegisterView = import( /* webpackChunkName: "RegisterView" */ '@/views/auth/RegisterView.vue');
const ForgotPasswordView = import( /* webpackChunkName: "ForgotPasswordView" */ '@/views/auth/ForgotPasswordView.vue');
const ResetPasswordView = import( /* webpackChunkName: "ResetPasswordView" */ '@/views/auth/ResetPasswordView.vue');
const HistoryView = import( /* webpackChunkName: "HistoryView" */ '@/views/history/HistoryView.vue');
const DepositView = import( /* webpackChunkName: "DepositView" */ '@/views/deposit/DepositView.vue');
const MoneyGramView = import( /* webpackChunkName: "MoneyGramView" */ '@/views/deposit/MoneyGramView.vue');

{
            name: 'login',
            path: '/login',
            component: () => LoginView
        },
        {
            name: 'register',
            path: '/register',
            component: () => RegisterView
        },

This is not how lazy loading works. out have to wrap the import() in a function but You call it immediately.


Language Server - makeChild

Adds a way for the language server to supply not only the content of the current script, but also all the script that will be imported by it

Request for help noted,


Fix import test

What does this PR do?
Import tests are failing on main, seems to be from an external library, best I could find different/slow is the transformers import.
We'll see what this shows however.
I've also tweaked the threshold to give us a few more datapoints in the tree.
Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.
@SunMarc

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


[FZ Editor] Custom layout creator, no way to delete zone with keyboard

From https://github.com/microsoft/PowerToys/issues/5776

In “Custom layout creator”:
After adding a zone, no way to keyboard navigate to new zone’s UI (ie, no way to delete it with keyboard only)

https://github.com/microsoft/PowerToys/issues/7105 tracks the same problem for grid-based layouts.

Related: #7105

Resolved with the release of v0.41.


will this work when the webapp is in the background on mobile ? (question)

Seeing the service worker isn't actively pulling from a backend, and is implemented for the sole purpose of allowing local notifications on mobile Chrome, I don't believe it will. However, I just had someone ask me a similar question, so you might find the answer useful. Attached is a screenshot of that convo.


Can't install on GX <IP_ADDRESS>

I recently tried to get the Branch https://github.com/mollie/gambio/tree/3.5-4.0 aka. release v1.0.1 working on Gambio GX <IP_ADDRESS>. The branch states to be compatible with GX 3.5 - 4.0 though Gambio <IP_ADDRESS> wouldn't recognize the new module. All required caches were emptied after uploading the files to the webserver. Is the version confirmed to be working with GX <IP_ADDRESS>? Thanks in advance.

Hi @me-marcel,
Thanks for reaching us.
Our team has tested the plugin on different GX3 versions including 3.10.x.x.
Did you upload the content of folder gambio-1.0.x to the root of Gambio shop? Only the content of folder gambio-1.0.x inside the downloaded zip should be uploaded to the root (please check the image below).

Thanks in advance.
Best regards.

Hi @logeecom,
of course I followed the instructions. Since this is the exact process described within the README.md of the plugin I followed it step by step and cleared all caches after uploading the files shown in the screenshot above. Still the plugin doesn't show up as module, legacy module or payment method within GX v<IP_ADDRESS>.
Kind regards

Hi @me-marcel,
Thanks for the detailed explanation.
Could you please create a support ticket on Mollie Support?
Please keep in mind that we will need admin and FTP access to check why Mollie doesn't show up as a module.
Thank you so much for your attention and participation.
Best regards.

I surely can 👍🏻 Thanks for taking time to deal with this issue.


Template for modulo operations

One of the most common codes that I've looked up over time is a mod template that I wrote many years ago for a private repository. It is useful enough to be used around once every second competition.
I cleaned up my old code, and added some extra features to it. I think this could be a super useful template to have on PyRival!
Note that I have yet to add tests to it, I will do that tomorrow.
@cheran-senthil What do you think about this template? I think it is both very simple and super useful to have.

As a small note on the fast mod mul. Locally on my windows computer using PyPy the code runs around a factor of 10 faster with modmul compared to doing a * b % MOD.

With the new version of PyPy on Windows, what are your thoughts on this moving forward?

With the new version of PyPy on Windows, what are your thoughts on this moving forward?

Once CF updates (assuming it ever updates) then we can remove all the fast modmul stuff from the template. But I still think this template has its uses even after that.


Linking to the article

Linking to the article

It was done by adding the URL to this repo into the paper.


Poxy runs into "File name too long" OSError

Environment
version and/or commit hash:
Poxy via pip install on Ubuntu 22.04:
$ poxy --version
0.12.3

Describe the bug
I just tried to get a basic config going to generate docs from:
https://github.com/tim-janik/anklang/tree/trunk/ase
But that soon runs into (warnings omitted):
$ poxy
Poxy v0.12.3
Reading /opt/src/anklang/poxy.toml
Generating XML files with Doxygen 1.9.1
Post-processing XML files

*************

Traceback (most recent call last):
  File "/usr/local/lib/python3.10/dist-packages/poxy/main.py", line 339, in main
    run(
  File "/usr/local/lib/python3.10/dist-packages/poxy/run.py", line 1621, in run
    postprocess_xml(context)
  File "/usr/local/lib/python3.10/dist-packages/poxy/run.py", line 726, in postprocess_xml
    if f.exists():
  File "/usr/lib/python3.10/pathlib.py", line 1290, in exists
    self.stat()
  File "/usr/lib/python3.10/pathlib.py", line 1097, in stat
    return self._accessor.stat(self, follow_symlinks=follow_symlinks)
OSError: [Errno 36] File name too long: '/tmp/poxy/opt_src_anklang/xml/namespace_ase_1_1_writ_converter_3_01_t_00_01_r_e_q_u_i_r_e_sv_3_01_9std_1_1is__base__of_3_01_serializable_00_01_t_01_4_1_1value_01_6_6_9_jsonipc_1_1_derives_vector_3_01_t_01_4_1_1value_01_6_6_9_jsonipc_1_1_derives_shared_ptr_3_01_t_01_4_1_1value_01_6_6std_1_1is__class_3_01_t_01_4.xml'

*************

You appear to have triggered an internal bug!
Please re-run poxy with --bug-report and file an issue at github.com/marzer/poxy/issues
Many thanks!

*************

Additional information
The poxy.toml file is basically copyied from the toml example.
--bug-report

[x] I have attached the zip file generated when using the --bug-report option

poxy_bug_report.zip

Oh, wow. That's an xml file generated by doxygen, not by poxy specifically. I'm actually not sure how to avoid that, short of simplifying/refactoring the source C++ so the doxygen symbol path is shorter. Can you show me some of the C++ code that is generating this? Specifically it looks like you're using a macro REQUIRES to do some SFINAE; any snippet with that would be helpful.

(cc @mosra - get a load of this behemoth filename Doxygen generated 😅)

Ok so I did some digging into your serialize.hh and found the culprit:
https://github.com/tim-janik/anklang/blob/3e64716fc0bc6dbb444f8bef12983b05f31af4d4/ase/serialize.hh#L454-L459
Since these are template specializations there's no way to make their Doxygen symbols shorter without also losing information (as the specialization's definition is it's identity) - you can't do any clever preprocessor tricks to simplify it without also losing unique specializations.
That being said, since practically all template specializations are used simply as a form of static polymorphism and don't actually change interfaces, you can typically avoid emitting documentation for them entirely. My suggestion here would be to tell doxygen to ignore them entirely using a @cond/@endcond block, e.g.:

/// @cond

template<typename T>
struct WritConverter<T, /* specialization 1 */> { /* ... */};

template<typename T>
struct WritConverter<T, /* specialization 2 */> { /* ... */};

/// @endcond

Oh, wow, what an enormous file name! That's an xml file generated by doxygen, not by poxy specifically, so I'm actually not sure how to avoid that, short of simplifying/refactoring the source C++ to make the doxygen symbol path is shorter.
Can you show me some of the C++ code that is generating this? Specifically it looks like you're using a macro REQUIRES to do some SFINAE; any snippet with that would be helpful.

It is the REQUIRESv<> macro, e.g. here: https://github.com/tim-janik/anklang/blob/trunk/ase/serialize.hh#L387
I can hack around that atm with:
sed 's/REQUIRESv<[^>]*>/REQUIRES<true>/' -i ase/serialize.hh

Which actually allows me to generate documentation for the ase/ dir with poxy.
PS: I just fail to get any file listed in the /files.html index, even adding @file doesn't help. Short of extract_all=true, but that includes anon namespaces even though poxy sets EXTRACT_ANON_NSPACES   = NO. Do you need a separate bug report for that or am I missing some config option?

Ok so I did some digging into your serialize.hh and found the culprit:
https://github.com/tim-janik/anklang/blob/3e64716fc0bc6dbb444f8bef12983b05f31af4d4/ase/serialize.hh#L454-L459
Since these are template specializations there's no way to make their Doxygen symbols shorter without also losing information (as the specialization's definition is it's identity) - you can't do any clever preprocessor tricks to simplify it without also losing unique specializations.

Yeah, is there a way to turn that into a Doxygen bug instead?
E.g. is poxy using a config that triggers Doxygen into genrating too long file names?

It is the REQUIRESv<> macro

Looks like we replied at much the same time. See my suggestion above.

PS: I just fail to get any file listed in the /files.html index, even adding https://github.com/file?type=source doesn't help.

That's an m.css feature, see here. In short, entities need to have some documentation (a brief, detail, a remark, function args, something) in order to appear in the generated HTML. In the case of files, merely using @file is insufficient, since that just tells doxygen "the following commands relate to this file", and isn't itself a piece of documentation - you'd need a @brief or similar.

E.g. is poxy using a config that triggers Doxygen into genrating too long file names?

Nothing specific - I'm not overriding any options that would impact filename length. My guess is that python's filename handling in Pathlib is stricter on file name length than it needs to be, hence why Doxygen is OK but poxy falls down here. That's a guess, though. I haven't encountered this before. It's also reasonable to suggest that it is a bug in Doxygen for them to be generating filenames that long instead of using a hash or something because holy crap, but that doesn't help here 😅
I can do some long symbol renaming in the XML preprocess step but it's a nontrivial fix so it will have to wait until this weekend at the earliest. Is the @cond/@endcond workaround a reasonable one in the short-term?

OH! I've just realized that doxygen has the SHORT_NAMES option, which might actually already be a nice built-in fix for this. I'll do some testing with it at some point over the next few days.

PS: I just fail to get any file listed in the /files.html index, even adding https://github.com/file?type=source doesn't help.

That's an m.css feature, see here. In short, entities need to have some documentation (a brief, detail, a remark, function args, something) in order to appear in the generated HTML. In the case of files, merely using @file is insufficient, since that just tells doxygen "the following commands relate to this file", and isn't itself a piece of documentation - you'd need a @brief or similar, e.g.:
/// @file
/// @brief This is where the serializer lives.

Thanks for that. But even pasting your snippet literally into my header file still yields an empty files.html index. Can you please point me to any working example of poxy generating a non-empty files.html ?

I can do some long symbol renaming in the XML preprocess step but it's a nontrivial fix so it will have to wait until this weekend at the earliest. Is the @cond/@endcond workaround a reasonable one in the short-term?

Thanks, there is really no need to rush anything for me, I'm just evaluating poxy atm, not using it in production or so ;-)
Also, it is really just one template case, the one in: https://github.com/tim-janik/anklang/blob/trunk/ase/serialize.hh#L455
I'll probably just comment it out for Doxygen based generations, as you said that won't affect the documentation much.

Can you please point me to any working example of poxy generating a non-empty files.html ?

Here you go:

HTML: https://marzer.github.io/muu/files.html
one of the @files: https://github.com/marzer/muu/blob/master/include/muu/accumulator.h

Depending on how your project is structured you may also need a documentation file for the containing directory for the child files to appear, e.g. https://github.com/marzer/muu/blob/master/include/muu/folder.dox

@wroyca

I'm unaware of doxygen internals specifically, but surely we can intercept in the middle?

Yup, it's possible. The 'XML v2' stuff I was working on did this to stabilise some IDs and it worked OK. The main issue here would be python falling down with long filenames; since Doxygen uses the IDs as filenames, I'd need to be able to interact with the filesystem to rename them, and it appears that even forming a path to do that will cause the OSError. There's probably a workaround (e.g. invoking the shell), but it looks like the SHORT_NAMES might already have me covered 😄

FYI, for excessively long names there's another option, CASE_SENSE_NAMES, which is NO on OSes with case-insensitive default filesystems (Windows, macOS) and causes the crazy long underscored names to appear. Unless your codebase contains classes, namespaces or files that differ only by case (which is rather unlikely I think), it should be safe to set to YES. Compared to SHORT_NAMES you'll get the URLs still somewhat readable with that option.

sed 's/REQUIRESv<[^>]*>/REQUIRES<true>/'

What I often do for such constructs is listing them in the PREDEFINED option and defining them to empty Ah, alright, it's not really a macro due to the <>. Then this is a useless suggestion.

I can do some long symbol renaming in the XML preprocess step but it's a nontrivial fix

I want to look into that for m.css itself eventually, because I suspect it'll need a lot more than just XML preprocess. And I could possibly implement INLINE_SIMPLE_STRUCTS and other useful output reorganization features alongside that.


porting changes back to main

Codecov Report

Merging #10979 (56d1a78) into main (84625f0) will increase coverage by 0%.
The diff coverage is n/a.

@@          Coverage Diff           @@
##            main   #10979   +/-   ##
======================================
  Coverage     63%      63%
======================================
  Files        482      482
  Lines      33851    33851
  Branches    5518     5518
======================================
+ Hits       21463    21472    +9
+ Misses     10344    10333   -11
- Partials    2044     2046    +2

Impacted Files
Coverage Δ

src/notebooks/outputs/tracebackFormatter.ts
93% <0%> (-7%)
:arrow_down:

src/kernels/jupyter/baseKernelConnectionWrapper.ts
35% <0%> (-3%)
:arrow_down:

src/standalone/api/kernelApi.ts
72% <0%> (-2%)
:arrow_down:

src/kernels/common/baseJupyterSession.ts
80% <0%> (-1%)
:arrow_down:

src/notebooks/debugger/kernelDebugAdapterBase.ts
77% <0%> (+1%)
:arrow_up:

...tive-window/debugger/jupyter/kernelDebugAdapter.ts
60% <0%> (+1%)
:arrow_up:

.../standalone/survey/dataScienceSurveyBanner.node.ts
69% <0%> (+3%)
:arrow_up:

src/notebooks/debugger/debuggingManagerBase.ts
73% <0%> (+4%)
:arrow_up:

...kernels/jupyter/preferredRemoteKernelIdProvider.ts
97% <0%> (+4%)
:arrow_up:

src/notebooks/debugger/debugger.ts
77% <0%> (+11%)
:arrow_up:


Actions.refresh is not working on intital Scene

Version
Tell us which versions you are using:
"react": "16.8.3",
"react-native": "0.59.3",
"react-native-gesture-handler": "github:kmagiera/react-native-gesture-handler#ff74e0edadd62518b45d85f251f527307457ab9e",
"react-native-router-flux": "4.1.0-beta.2",
Expected behaviour
If I click on to "change title" it does not refresh or change the title , however if I move and then click on it , it works .
app.js
import React, {Component} from 'react';
import {Platform, StyleSheet, Text, View, TouchableOpacity} from 'react-native';
import { Scene, Router, Actions, Stack } from 'react-native-router-flux';
import { StackViewStyleInterpolator } from 'react-navigation-stack';
import Page from './Page'
const transitionConfig = () => ({
  screenInterpolator: StackViewStyleInterpolator.forFadeFromBottomAndroid,
});
type Props = {};
const stateHandler = (prevState, newState, action) => {
  console.log('onStateChange: ACTION:', action);
};

export default class App extends Component<Props> {
  render() {
    return (
      <View style={{ flex: 1 }}>
      <Router key="router" onStateChange={stateHandler} sceneStyle={styles.scene}>
      <Stack key="root" titleStyle={{ alignSelf: 'center' }}>
          <Scene key="page" component={Page} title="Page" />
          <Scene key="page2" component={Page} title="Page2" />
        </Stack>
      </Router>
      </View>
    );
  }
}

const styles = StyleSheet.create({
  container: {
    flex: 1,
    backgroundColor: 'transparent',
    justifyContent: 'center',
    alignItems: 'center',
  },
  scene: {
    backgroundColor: '#F5FCFF',
    shadowOpacity: 1,
    shadowRadius: 3,
  },
  tabBarStyle: {
    backgroundColor: '#eee',
  },
  tabBarSelectedItemStyle: {
    backgroundColor: '#ddd',
  },
});

page.js
import React, {Component} from 'react';
import {Platform, StyleSheet, Text, View, TouchableOpacity} from 'react-native';
import { Actions } from 'react-native-router-flux'
export default class App extends Component<Props> {
  render() {
    return (
      <View style={styles.container}>
        <TouchableOpacity onPress={()=> Actions.refresh({ title: 'Cool!', key: Math.random() }) }>
        <Text style={styles.welcome}>Change Title</Text>
        </TouchableOpacity>

        <TouchableOpacity onPress={()=> Actions.page2() }>
        <Text style={styles.welcome}>Move ></Text>
        </TouchableOpacity>

      </View>
    );
  }
}
const styles = StyleSheet.create({
  container: {
    flex: 1,
    justifyContent: 'center',
    alignItems: 'center',
    backgroundColor: '#F5FCFF',
  },
  welcome: {
    fontSize: 20,
    textAlign: 'center',
    margin: 10,
  },
  instructions: {
    textAlign: 'center',
    color: '#333333',
    marginBottom: 5,
  },
});

Could you try latest beta?

On 6 Apr 2019, at 10:08, Andrew<EMAIL_ADDRESS>wrote:
Version
Tell us which versions you are using:
"react": "16.8.3",
"react-native": "0.59.3",
"react-native-gesture-handler": "github:kmagiera/react-native-gesture-handler#ff74e0edadd62518b45d85f251f527307457ab9e",
"react-native-router-flux": "4.1.0-beta.2",
Expected behaviour
If I click on to "change title" it does not refresh or change the title , however if I move and then click on it , it works .
app.js
import React, {Component} from 'react';
import {Platform, StyleSheet, Text, View, TouchableOpacity} from 'react-native';
import { Scene, Router, Actions, Stack } from 'react-native-router-flux';
import { StackViewStyleInterpolator } from 'react-navigation-stack';
import Page from './Page'
const transitionConfig = () => ({
screenInterpolator: StackViewStyleInterpolator.forFadeFromBottomAndroid,
});
type Props = {};
const stateHandler = (prevState, newState, action) => {
console.log('onStateChange: ACTION:', action);
};
export default class App extends Component {
render() {
return (
<View style={{ flex: 1 }}>

<Stack key="root" titleStyle={{ alignSelf: 'center' }}>

);
}
}
const styles = StyleSheet.create({
container: {
flex: 1,
backgroundColor: 'transparent',
justifyContent: 'center',
alignItems: 'center',
},
scene: {
backgroundColor: '#F5FCFF',
shadowOpacity: 1,
shadowRadius: 3,
},
tabBarStyle: {
backgroundColor: '#eee',
},
tabBarSelectedItemStyle: {
backgroundColor: '#ddd',
},
});
page.js
import React, {Component} from 'react';
import {Platform, StyleSheet, Text, View, TouchableOpacity} from 'react-native';
import { Actions } from 'react-native-router-flux'
export default class App extends Component {
render() {
return (

<TouchableOpacity onPress={()=> Actions.refresh({ title: 'Cool!', key: Math.random() }) }>
Change Title

    <TouchableOpacity onPress={()=> Actions.page2() }>
    <Text style={styles.welcome}>Move ></Text>
    </TouchableOpacity>

  </View>
);

}
}
const styles = StyleSheet.create({
container: {
flex: 1,
justifyContent: 'center',
alignItems: 'center',
backgroundColor: '#F5FCFF',
},
welcome: {
fontSize: 20,
textAlign: 'center',
margin: 10,
},
instructions: {
textAlign: 'center',
color: '#333333',
marginBottom: 5,
},
});
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/aksonov/react-native-router-flux/issues/3461, or mute the thread https://github.com/notifications/unsubscribe-auth/ABQpcatykxWHVk_Dtdo7s49Qz3i-LmkHks5veFXugaJpZM4cgFCu.

isn't it latest beta ? I could not find anywhere in release about latest beta version.

Try beta.5

On 6 Apr 2019, at 11:47, Andrew<EMAIL_ADDRESS>wrote:
isn't it latest beta ? I could not find anywhere in release about latest beta version.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

did not work . even I tried to use replace, it reflects title however, it didn't reload the stuff .

I have this EXACT same problem as well, I opened an issue for this too: https://github.com/aksonov/react-native-router-flux/issues/3478
@aksonov

Same problem here, what do we do @aksonov ?
This have worked in another version im sure.

Hey @Friis1978, I suggested 2 possible solutions for this issue on my other thread: https://github.com/aksonov/react-native-router-flux/issues/3478#issuecomment-488011253. It's the best we can do until we hear back from the library author about the issue. Perhaps Actions.refresh is being deprecated and my solutions are meant to be the correct way to go.

@Twinbird24 ok, i saw your solutions, but it didnt make sence to me how to use it with the title, where and how can I set the title ?

You can set the title in any method you want inside your scene component, in the example I am setting it in the onEnter method, although I have tested if it will work in other methods, like an on press callback method, for example (it should be easy to try though).
The other example, I am setting the prop directly from the Scene component (which is probably located in your root index file or something). For the title (I have not tested this, so you'll have to try it yourself) you can do something like this:
title={() => {
       if (Actions.refs.mySceneKey) {
         Actions.refs.mySceneKey.myComponentMethodThatReturnsAString();
       }
     }}

However, in this last example, I'm not sure if you'll be able to re-trigger the method and change the title again once the component is already mounted.

Hi @Twinbird24
Thanks for your suggestions, I found another method, using a custom component to set the title based on a object in the redux reducer. Something like this:
<Scene                         key="guardpage"                         component={GuardList}                         renderTitle={() => (                             <View style={styles.headerLogoView}>                                 <Logo title="guardspagetitle" />                             </View>                         )}                     />
In this 'Logo' component I used 'react-native-i18n' to translate the title into the phones language. Like this:
`
if (this.props.txtValues) {
I18n.translations = this.props.txtValues;
locale = I18n.locale;
this.props.translateIntro(locale, subscriptionFeaturesTranslation, subscriptionTermsTranslation, termsTranslation, slidesTranslation);
}
const locale = I18n.locale;
const txt = this.props.title;
const values = this.props.txtValues[locale];
const title = values[txt];
`
And then I used this title in the render().
This is also a way to set the title based on the language.
Thank you.

I found something that works. I have a method that is called after a short timeout from componentDidMount. That method calls
Actions.home();
("home" is the key of my initial route) and then
Actions.refresh({title: "whatever"});
These appear to do nothing, but then the next time I call the same methods in the same order, but with a different title, the title is actually replaced.
Hope this helps, lemme know if it isn't clear


Config Attribute: Always recompile build script

When applied on script build script assembly should always be recompiled.

             [AlwaysRecompileScript]          public class BuildScript : DefaultBuildScript          {          }


Can't compile v2.1.13

Hello,
I just downloaded tar.gz source code, tried to compile it on Ubuntu 16.04 but failed. AMD drivers are installed as other miners are working well with all my cards.
I have the following error :
tiix@geonosis ~/cate/MVis-tokenminer-1.1.13/build % make
[  0%] Built target BuildInfo.h
[ 39%] Built target devcore
[ 49%] Built target ethash
[ 50%] Building CXX object libethash-cl/CMakeFiles/ethash-cl.dir/ethash_cl_miner.cpp.o
In file included from /home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../libethcore/Miner.h:36:0,
                 from /home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../libethcore/Ethash.h:31,
                 from /home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../libethcore/EthashAux.h:28,
                 from /home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../libethcore/EthashGPUMiner.h:28,
                 from /home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/ethash_cl_miner.h:20,
                 from /home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/ethash_cl_miner.cpp:40:
/home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../ethminer/ADLUtils.h:65:72: error: ‘ADL_CONTEXT_HANDLE’ has not been declared
  typedef int(*ADL2_MAIN_CONTROL_CREATE)(ADL_MAIN_MALLOC_CALLBACK, int, ADL_CONTEXT_HANDLE*);
                                                                        ^
/home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../ethminer/ADLUtils.h:66:51: error: ‘ADL_CONTEXT_HANDLE’ has not been declared
  typedef int(*ADL2_ADAPTER_NUMBEROFADAPTERS_GET) (ADL_CONTEXT_HANDLE, int*);
                                                   ^
/home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../ethminer/ADLUtils.h:67:46: error: ‘ADL_CONTEXT_HANDLE’ has not been declared
  typedef int(*ADL2_ADAPTER_ADAPTERINFO_GET) (ADL_CONTEXT_HANDLE, LPAdapterInfo, int);
                                              ^
/home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../ethminer/ADLUtils.h:68:41: error: ‘ADL_CONTEXT_HANDLE’ has not been declared
  typedef int(*ADL2_ADAPTER_ACTIVE_GET) (ADL_CONTEXT_HANDLE, int, int*);
                                         ^
/home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../ethminer/ADLUtils.h:69:37: error: ‘ADL_CONTEXT_HANDLE’ has not been declared
  typedef int(*ADL2_OVERDRIVE_CAPS) (ADL_CONTEXT_HANDLE, int iAdapterIndex, int *iSupported, int *iEnabled, int *iVersion);
                                     ^
/home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../ethminer/ADLUtils.h:70:55: error: ‘ADL_CONTEXT_HANDLE’ has not been declared
  typedef int(*ADL2_OVERDRIVE6_THERMALCONTROLLER_CAPS)(ADL_CONTEXT_HANDLE, int iAdapterIndex, ADLOD6ThermalControllerCaps *lpThermalControllerCaps);
                                                       ^
/home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../ethminer/ADLUtils.h:70:94: error: ‘ADLOD6ThermalControllerCaps’ has not been declared
  typedef int(*ADL2_OVERDRIVE6_THERMALCONTROLLER_CAPS)(ADL_CONTEXT_HANDLE, int iAdapterIndex, ADLOD6ThermalControllerCaps *lpThermalControllerCaps);
                                                                                ^
In file included from /home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../libethcore/Miner.h:36:0,
                 from /home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../libethcore/Ethash.h:31,
                 from /home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../libethcore/EthashAux.h:28,
                 from /home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../libethcore/EthashGPUMiner.h:28,
                 from /home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/ethash_cl_miner.h:20,
                 from /home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/ethash_cl_miner.cpp:40:
/home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../ethminer/ADLUtils.h:71:48: error: ‘ADL_CONTEXT_HANDLE’ has not been declared
  typedef int(*ADL2_OVERDRIVE6_TEMPERATURE_GET)(ADL_CONTEXT_HANDLE, int iAdapterIndex, int *lpTemperature);
                                                ^
/home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../ethminer/ADLUtils.h:72:45: error: ‘ADL_CONTEXT_HANDLE’ has not been declared
  typedef int(*ADL2_OVERDRIVE6_FANSPEED_GET)(ADL_CONTEXT_HANDLE, int iAdapterIndex, ADLOD6FanSpeedInfo *lpFanSpeedInfo);
                                             ^
/home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../ethminer/ADLUtils.h:72:84: error: ‘ADLOD6FanSpeedInfo’ has not been declared
  typedef int(*ADL2_OVERDRIVE6_FANSPEED_GET)(ADL_CONTEXT_HANDLE, int iAdapterIndex, ADLOD6FanSpeedInfo *lpFanSpeedInfo);
                                                                                ^
/home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../ethminer/ADLUtils.h:73:49: error: ‘ADL_CONTEXT_HANDLE’ has not been declared
  typedef int(*ADL2_OVERDRIVEN_TEMPERATURE_GET) (ADL_CONTEXT_HANDLE, int, int, int*);
                                                 ^
/home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../ethminer/ADLUtils.h:74:48: error: ‘ADL_CONTEXT_HANDLE’ has not been declared
  typedef int(*ADL2_OVERDRIVEN_FANCONTROL_GET) (ADL_CONTEXT_HANDLE, int, ADLODNFanControl*);
                                                ^
/home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../ethminer/ADLUtils.h:74:73: error: ‘ADLODNFanControl’ has not been declared
  typedef int(*ADL2_OVERDRIVEN_FANCONTROL_GET) (ADL_CONTEXT_HANDLE, int, ADLODNFanControl*);
                                                                         ^
/home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../ethminer/ADLUtils.h:75:48: error: ‘ADL_CONTEXT_HANDLE’ has not been declared
  typedef int(*ADL2_OVERDRIVEN_FANCONTROL_SET) (ADL_CONTEXT_HANDLE, int, ADLODNFanControl*);
                                                ^
/home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../ethminer/ADLUtils.h:75:73: error: ‘ADLODNFanControl’ has not been declared
  typedef int(*ADL2_OVERDRIVEN_FANCONTROL_SET) (ADL_CONTEXT_HANDLE, int, ADLODNFanControl*);
                                                                         ^
/home/tiix/cate/MVis-tokenminer-1.1.13/libethash-cl/../ethminer/ADLUtils.h:98:2: error: ‘ADL_CONTEXT_HANDLE’ does not name a type
  ADL_CONTEXT_HANDLE context = NULL;
  ^
libethash-cl/CMakeFiles/ethash-cl.dir/build.make:66: recipe for target 'libethash-cl/CMakeFiles/ethash-cl.dir/ethash_cl_miner.cpp.o' failed
make[2]: *** [libethash-cl/CMakeFiles/ethash-cl.dir/ethash_cl_miner.cpp.o] Error 1
CMakeFiles/Makefile2:265: recipe for target 'libethash-cl/CMakeFiles/ethash-cl.dir/all' failed
make[1]: *** [libethash-cl/CMakeFiles/ethash-cl.dir/all] Error 2
Makefile:127: recipe for target 'all' failed
make: *** [all] Error 2

Did someone encountered the same issue ?
Thank you !

Did you follow the instructions in the readme, specifically these ones?
- download the AMD ADL SDK from the AMD website, and extract it to a temporary folder
- copy all 3 .h files from the <adl_package>/include/ folder to <mvis_folder>/extdep/include/amd_adl/
  (create subfolders as necessary)

Hmm.  AMD website seems kind of messed up right now.  Can't find any of the SDK pages.
Here's the files I have on my system:
amd_adl_includes.zip

Okay so I had an bad version.
It works well with your libraries. Thank you very much !


Fix bug and performance of getrf_tntpiv

This fixes an accuracy bug when origin=d and target=d.  Additionally, I broke up some of the tasks into multiple pieces to expose more parallelism.
Maybe this will address #70?
On Frontier, with 4 nodes, CPU-only MPI, and n=250 000, this gives a 28% speedup.  (I didn't test GPU-aware MPI because it was giving me issues.  I think due to the usual multi-threading bug in Cray MPICH.)

I discovered that using rocm/5.3.0 instead of rocm/5.1.0 fixes this issue.  But, this is still beneficial for the performance boost.

Can you please re-run CI on AMD?

What happened to the CI?
"Workflow runs completed with no jobs"

I was able to get the CI to run.  The one error is the usual issue with QR/LQ.


Changes

So, lets start this off!
Need everyone's input!!
here are the current main issues that this PR is going to solve.
#80 #62 #89 #24
Okay this is going to be a mega RP...
So its this PR is just to get the discussion around this started..
So what does this fix?
All the issues above :)
What am I proposing to change and why?
GET
So first off, we are going to change the way the whole "find" system work's, for the better I think...
So we are going to change the way we handle simple "get" queries the reason why is that the current way is just an awful (for a lack of a better word) way of doing it...
here is some of reasoning behind it.
So to do a simple "model.find( primaryKey )" it has to do multiple "get" requests to get it...
Why not just do "waterline:collectionname:primarykey:value" query? who knows... but what I'm proposing is that we move it to a simple find method...
CREATE
Now, the next step is going to be changing the "set/create" method, so this method is way to complicated for what it actually does, and it confuses new comers to the project, so cleaning that function up and removing some of the complexity and move them to lower methods or just an ultimate clean up.
We also need to give the users the ability to set the prefix, so instead of defaults to "waterline" you can actually set it, obviously it will continue to default to "waterline" but it would be a nice change for people that want to use it...
Uniqueness
So at the current state, we manually check if the the value is unique when we do most crud functions, which is fine, but there is a bug around the .save method... so we need to make a better or adapt the current methods...
Now lets try to keep redis consistency...
So we are going to change .find with .get and .find will just map to .get this will keep with redis methodology and the same for create vs .set
The last and final thing we need TTL,
So if we add a method called setTTL or something, we can start getting TTL support ( YAY! )
I'm sure I'm missing other thing's but would love everyone's options on this!
I'm just going to keep it here... so please no one merge this one just yet... ill continue to add to it!

I try to focus on atomistic changes to sails-redis instead of overhauling the whole process.
I propose to adress TTL issues first without changing the rest of the behaviour, to keep PR and review easier.
userland expire
Using the following custom callback afterCreate(), I can set an expire on my keys :
afterCreate: function (createdEntry, cb) {
    var self = this;
    self.native(function (err, connection) {
      console.log('waterline:'+ self.tableName + ':' + self.primaryKey + ':' + createdEntry[self.primaryKey]);
      if (!err)
        connection.expire('waterline:'+ self.tableName + ':' + self.primaryKey + ':' + createdEntry[self.primaryKey], 10);
    });
    cb();
  }

Question :
Is there a better way to access 'waterline:'+ self.tableName + ':' + self.primaryKey + ':' + createdEntry[self.primaryKey]
custom primaryKey
If I use a model with :
module.exports = {
  autoPK: false,
  attributes: {
    email: {
      type: 'email',
      required: true,
      primaryKey: true
    }
}

I can set expire with the previous snippet, but the primary key is also stored in the indexed set :
index.js line 482
addToSet: ['saveKeyData', function(next, results) {

      var index = self.schema.indexKey(name, primary);

      // Grab the key to use for this record
      var noSqlKey = self.schema.recordKey(name, primary, data[primary]);

      self.connection.sadd(index, noSqlKey, function(err) {
        if(err) return next(err);
        next();
      });

    }]

Now once the key expires, I can't create a new one with the same primaryKey value, since there is still an entry in self.schema.indexKey(name, primary)
So, I thought of a very simple fix :
Here, can we set an expire on the index set values too ? That way, if the unique index disappear at the same time as the key I want to expire(), the unique throw would disappear.
Then we could easily do a method setTTL() that would expire the 'main' key and the 'unique index' key in one method.
I notice the addToSet async uses redis.sadd()
I will try today to acess the individual values in the set. I don't know yet if we can expire data inside of a set.
If we can't, we might use another storage pattern that allows it.

Ok so it seems we can't set expire on items in a set :
https://quickleft.com/blog/how-to-create-and-expire-list-items-in-redis/
So maybe we might just use a prefix index namespace to store, request and expire unique keys

index.js
multi.sismember(idx.keyName, data[idx.name]);

sismember checks if an item is part of a set.

@particlebanana
What to you think of keeping the unique checks and storage conceptually, but to store relevant data outside of sets so that we can set expire() on the unique index individual keys as part of a setTTL method ?

With a little more detail in the proposition :
in index.js line 755
Database.prototype._uniqueConstraint = function _uniqueConstraint(collectionName, data, cb) {
 // ...
  indices.forEach(function(idx) {
    if(!hasOwnProperty(data, idx.name)) return;
    // --- multi.sismember(idx.keyName, data[idx.name]);
    //+++
    multi.get([idx.keyName, data[idx.name]].join(':'));
  });
// ...
};

@Ryanc1256 sorry it took so long to reply! I think these changes are much needed and for the better. Make it a breaking change - maybe bump it to 1.0.0.
I want to make sure you have whatever you need to make sails-redis work for you and everyone else who is using it. Going to shoot you an email in a min with some ideas.
@Kallikrein I defer to @Ryanc1256 on structural changes.

Okay, Sorry it's been a while... I'm starting to work on it now...
So i've already tweaked it to
#84 so this PR will resolve that...

@Ryanc1256 What is missing to get this pull request accepted?
And, one more question, do you believe your master branch is safe to run in production?
Thank you.

@ianko Hey,
im waiting for them to pull in a pull request on another module (https://github.com/balderdashy/waterline-criteria/pull/28), so i have to wait until that one gets pulled in...
And I would say it's 'safe' as its just at the moment its just has a way faster find method.... but then again i haven't tested everything, so i would also say that it isn't at the same time.


Gradient Enhanced Neural Networks (GENN)

Added GENN to the SMT project. Feedback welcome.

:x: Build SMT 0.2.462 failed (commit https://github.com/SMTorg/SMT/commit/61797cbb41 by @)

:x: Build SMT 0.2.465 failed (commit https://github.com/SMTorg/SMT/commit/d178955750 by @)

:x: Build SMT 0.2.466 failed (commit https://github.com/SMTorg/SMT/commit/5652e1eee5 by @)


Improvements to Neural Style Transfer

Incorporates the improvements suggested in https://github.com/fchollet/keras/pull/6872
Specifically -

Uses VGG 19 rather than VGG16.
Uses block5_conv2 (conv5_2) layer for better results, in contrast to conv4_2 as suggested by the original paper.
Image initialization is now done using the content image itself, rather than noise.

Please provide an example image (or two) generated with the default settings.

Will do. Since it takes quite some time to generate images, will 20 iterations of the algorithm be sufficient?

Here is the output after 20 iterations, on the style image given below :
Style - "Patterned Leaf"

The following images were with default settings, run for 20 iterations each.
Old model (VGG16, noise initialization, conv4_2) -

New model (VGG19, content image initialization, conv5_2) -

The following images are from my repository (using Network.py, which is a replica of the Keras example with variables exposed), with default settings (VGG 19, content initialization, conv5_2, Total Variation Loss weight of 8.5e-5, some have color preservation post processing (which does not affect style transfer)).
Each example has the result in the center, the content in the upper right corner and the style in the bottom right corner. Open and zoom in for finer details which are obtained using conv5_2 and VGG 19, and no noise in the images due to content initialization.
Please disregard the text, it was for a demonstration -

LGTM, thanks


Design the CTA Guideline page

As an IBM.com Library adopter

I need a guideline to follow and effectively create the CTA for my day-to-day work

so that I can not only do my day to day work without confusion but also support a better user experience on IBM.com.

Tasks

[ ] Design the page with finalized content
[ ] Craft the Dos and Don'ts examples.

Acceptance criteria

[ ] Design is ready and reviewed with a developer

Task update:
The initial design is in work progress and needs content edits. Jim is working on the edit via issue #353

Placement section content updated.

@larahanlon2 @wonilsuhibm Can you take a look and let me know what do you think? Thank you.

@JennySanchez The design looks good to me, but I have a few concerns regarding the content. Where are we with the content?

@wonilsuhibm The content in the design above is the latest. The detail of Jim's work can be viewed via #328.

Note: We have additional edits on the content - task issue #356 created for the work.


Flyway 10 missing Automatic-Module-Name in MANIFEST.MF

Which version and edition of Flyway are you using?
Community 10.0.0
If this is not the latest version, can you reproduce the issue with the latest one as well? (Many bugs are fixed in newer releases and upgrading will often resolve the issue)
This is the latest version.
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Command-line, Java API, Maven plugin
Which database are you using? (Type & version)
MySQL Community 8.2.0
Which operating system are you using?
Linux
What did you do? (Please include the content causing the issue, any relevant configuration settings, the SQL statement(s) that failed (if any), and the command you ran)
Updated the flyway dependency from 9.22.3 to 10.0.0. There were no changes.
What did you expect to see?
Flyway application compiles successfully, runs migrations successfully.
What did you see instead?
On compilation, get the following error:
error: module not found: org.flywaydb.core

I noticed that you switched from using the Apache Maven Bundle Plugin 5.1.8 to the Maven Jar Plugin 3.2.0 for this release, it appears you haven't migrated everything necessary to use the Maven Jar Plugin correctly: flyway-core-10.0.0.jar is missing the Automatic-Module-Name: org.flywaydb.core entry in MANIFEST.MF. It is required for every JAR on the module path that has no module descriptor.
This causes code that previously compiled against flyway-core-9.22.3.jar (which did have the Automatic-Module-Name) to fail with the above message.

Hi @kebeda ,  Automatic-Module-Name is optional, and it's removed from all Flyway plugins.

@JasonLuo-Redgate How is it optional? Without it, there is no way to access the JAR from the module path! This doesn't fit the definition of optional as I understand it.
Also, I'm not talking about plugins, I'm talking about flyway-core.
Practically speaking, how do I get code that doesn't live on the class path to reference the Flyway API without this?

Hi @kebeda , this change is aimed to fix another issue here #3342.

@JasonLuo-Redgate Removing the Automatic-Module-Name from org.flywaydb:flyway-core:10.0.0 does not fix #3342, but it does hide the fact that you didn't address it at the expense of breaking all Java Migrations written in Java 9+! The proper fix to #3342 would have been to simply add an  Automatic-Module-Name of  flyway-sqlserver to org.flywaydb:flyway-sqlserver:10.0.0 and a value of flyway-mysqlserver to org.flywaydb:flyway-mysql:10.0.0.
The current configuration is unworkable, and must be addressed if you ever want to support a version of Java higher than 9.

Hi @kebeda , based on my understanding, if Automatic-Module-Name is not presented the automatic module name is inferred from the file name.

@kebeda I am currently investigating a resolution to this matter. This change was made with best intentions to try and progress the long standing issue of https://github.com/flyway/flyway/issues/3196 (and because adding Automatic-Module-Name did not have the full positive effect that we had originally intended). As part of that previous issue, user https://github.com/cowwoc has provided a PR that looks promising (but obviously needs to be updated for V10).
I am hoping having the module descriptors will remove the need for the Automatic-Module-Name, and if not, I will seriously look at reverting this change.
I do apologise for this not making it into the release notes and I do understand your frustrations. I will update this post as soon as we have something worthwhile to share.

@madninjaskillz If you are introducing module-info.java's to all your published artifacts, and they match the name you gave for the module historically in the Automatic-Module-Name manifest entry, there should be no problems, besides the issues mentioned above, which by necessity will need to break. The goal should be to break as little code as possible, while maintaining the promises made in prior releases.
I'm concerned because I see no hint of this in the code base that is checked in, so I'm confused...are we still talking JPMS modules, or something else?


Is there any way to enter data for the 'statistics_short_term` table?

Checklist

[X] I have filled out the template to the best of my ability.
[X] This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
[X] This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.
I would like to import short term statistics (i.e., once every 5 minutes) as well as the hourly statistics that your integration currently covers.
Describe the solution you'd like
Similar code with ability to upload a file with short term statistics data to be entered into the statistics_short_term table.
(Ideally, also a REST API so json encoded short term statistics could be added via an API rather than uploading a file)
Describe alternatives you've considered
Manually adding stuff to the SQLite database or purging the recorder.
Additional context

I am not aware of an HA-interface I could use for this. I would need a method similar to async_import_statistics / async_add_external_statistics which also works for short term statistics. If you find that out, I could give it a try.

No feedback, close.


Publishing

22/03/2024 submission deadline; after that date, publish the completed project.

Published on 21/03/2024.


add an aligned TSM model

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.HydrogenSulfate seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


REUSE_DATASET_IF_EXISTS don't work

Describe the bug
I use the following code  to download natural_question dataset. Even though I have completely download it, the next time I run this code, the new download procedure will start and cover the original /data/lxy/NQ
config=datasets.DownloadConfig(resume_download=True,max_retries=100,cache_dir=r'/data/lxy/NQ',download_desc='NQ')
data=datasets.load_dataset('natural_questions',cache_dir=r'/data/lxy/NQ',download_config=config,download_mode=DownloadMode.REUSE_DATASET_IF_EXISTS)

Since I don't have apache_beam installed, it throw a exception. After I pip install apache_beam ,the download restart..

Steps to reproduce the bug
run this two line code
config=datasets.DownloadConfig(resume_download=True,max_retries=100,cache_dir=r'/data/lxy/NQ',download_desc='NQ')
data=datasets.load_dataset('natural_questions',cache_dir=r'/data/lxy/NQ',download_config=config,download_mode=DownloadMode.REUSE_DATASET_IF_EXISTS)
Expected behavior
Download behavior can be correctly follow DownloadMode
Environment info

datasets version: 2.14.4
Platform: Linux-3.10.0-1160.88.1.el7.x86_64-x86_64-with-glibc2.17
Python version: 3.9.17
Huggingface_hub version: 0.16.4
PyArrow version: 11.0.0
Pandas version: 2.0.3

Hi! Unfortunately, the current behavior is to delete the downloaded data when this error happens. So, I've opened a PR that removes the problematic import to avoid losing data due to apache_beam not being installed (we host the preprocessed version of natual_questions on the HF GCS, so requiring apache_beam in that case doesn't make sense)

Thanks for your reply. I met another question that I set export HF_DATASETS_CACHE=/data/lxy/.cache , but each time I run load_datasets, the datasets module still looking for NQ in the wrong default cache dir '/home/lxy/.cache' 。How to avoid this incorrect behavior. I am sure HF_DATASETS_CACHE was set correctly since I use echo & to check it.

You need to set this variable before the datasets import. Then, you can use import datasets; datasets.config.HF_DATASETS_CACHE to verify the cache location.


dea_ng varz exposed customer data vulnerable to eavesdropping (plain http protected by basic auth)

Most Cf components varz when sanitized to only include non sensitive stats data. Cf
https://www.pivotaltracker.com/story/show/65300526
https://www.pivotaltracker.com/story/show/65042430
However the dea varz still include some sensitive details about containers such as:

app name
app routes (uris)
security groups (egress_network_rules)
container memory stats

Currently varz are always exposed in plain http, exposing the customer data to eavesdropping risks if intruders are able to sniff traffic to DEA
https://github.com/cloudfoundry/vcap-common/blob/4ae2f57d5a79f80b4d245b87e048f8a37335dab6/lib/vcap/component.rb#L131-L145
The varz endpoint provides useful diagnostic data used by admin-ui. This would be fine if it could support an option to be started on https, and potentially support mutual ssl auth in its yml config (i.e. expect clients to offer SSL cert to connect to).
In terms of implementing secret rotation, it seems that the basic auth (username and password) are automatically generated as hinted into
https://groups.google.com/a/cloudfoundry.org/forum/#!searchin/vcap-dev/varz/vcap-dev/rycUnc0TS38/FCmgtebOD08J however I could not yet verify this into the code:

Specs don't describe the password being setteable
https://bosh.io/jobs/dea_next?source=github.com/cloudfoundry/cf-release&version=224
the yml template does not include status hash
https://github.com/cloudfoundry/cf-release/blob/master/jobs/dea_next/templates/dea.yml.erb
However dea config confirms status options are optional
https://github.com/cloudfoundry/dea_ng/blob/d710ce11a539293e5cd78231f5722514288530e8/lib/dea/config.rb#L80-L84

So restarting dea should automatically rotate varz basic auth, so no issues there.

It's worth noting that Collector and /varz are being deprecated: https://github.com/cloudfoundry/collector

/cc @fraenkel @sykesm

thanks @Amit-PivotalLabs for the follow up. Refined a bit the issue desc to remind varz deprecation plans. What about conditionally enabling varz to be turned off or start on https during this deprecation phase?

Hey @gberche-orange
I know you mentioned the /varz data being useful, but are you saying you'd be okay with a solution that allows just disabling populating this data in /varz altogether?  Given that collector is being deprecated, and indeed the DEAs will eventually give way to Diego, I'm hoping to find the simplest thing that will meet your needs.

Thanks Amit. For my particular case, the DEA replacement by Diego are indeed satisfactory (production workloads with security requirements will come after our diego transition).
Sorry for the noise on this issue. Feel free to close it if there is no other people impacted.

@gberche-orange great, glad that works for you. Will close out.


UserProfile component triggers hard navs on Nextjs (and others)

Preliminary Checks

[X] I have reviewed the documentation: https://clerk.com/docs
[X] I have searched for existing issues: https://github.com/clerk/javascript/issues
[X] I have not already reached out to Clerk support via email or Discord (if you have, no need to open an issue here)
[X] This issue is not a question, general help request, or anything other than a bug report directly related to Clerk. Please ask questions in our Discord community: https://clerk.com/discord.

Reproduction

Commit here https://github.com/arlyon/litehouse/commit/30921dd9b0a290b75a84cefbdd038a5f6a48f29c
and you can validate the behaviour here https://litehouse.arlyon.dev/profile
source here: https://github.com/arlyon/litehouse/blob/main/site/app/(auth)/profile/[[...profile]]/page.tsx

Publishable key
pk_test_cHJvLW9yeXgtMjcuY2xlcmsuYWNjb3VudHMuZGV2JA
Description
It would be nice if the router was able to specialize for frameworks and default to a client side nav in supported cases rather than a hard serverside one. This seems to already be the case for navigation between sign in and sign up.
Steps to reproduce:

Visit a self hosted profile
Click to a different tab
Observe hard navigation

Expected behavior:
I am using nextjs, so it would be nice if this leveraged the framework to make the navigation seamless.
Actual behavior:
A hard nav.
Environment
N/A

Are you using the latest version of the nextjs SDK at the moment? We think we fixed this in the latest version, just want to confirm.

5.1.5 still displays the behaviour for me

Commit here https://github.com/arlyon/litehouse/commit/30921dd9b0a290b75a84cefbdd038a5f6a48f29c
and you can validate the behaviour here https://litehouse.arlyon.dev/profile
source here: https://github.com/arlyon/litehouse/blob/main/site/app/(auth)/profile/[[...profile]]/page.tsx

Nothing here seems out of place. Project doesn't need any particular setup for reproing. Required env vars are as follows
NEXT_PUBLIC_CLERK_PUBLISHABLE_KEY=<redacted>
CLERK_SECRET_KEY=<redacted>
NEXT_PUBLIC_CLERK_SIGN_IN_URL="/sign-in"
NEXT_PUBLIC_CLERK_SIGN_UP_URL="/sign-up"

Hello 👋
In an effort to keep our GitHub issues clean and focused, we close any issues that are awaiting a reproduction after 8 days on inactivity, and it has been 7 days. This issue will be closed tomorrow unless a reproduction is provided. If it takes longer than this to get a reproduction, that's ok, just drop a comment and we will remove the Stale label.
How to create a minimal reproduction
Thanks for being a part of the Clerk community! 🙏

After 8 days without a reproduction being supplied, we are closing this issue. Keep in mind, I'm just a robot, so if I've closed this issue in error, please reply here and my human colleagues will reopen it. Likewise if a reproduction is prepared after it has been closed.

Repro has been given, see the profile.tsx page above.

This isn't a minimal reproduction unfortunately, it's a link to part of an entire built-out app. Our instructions for creating a minimal reproduction linked above should outline how to make this happen!

@jescalan please reopen https://github.com/arlyon/clerk-hard-nav

set env vars from a clerk app
visit main page, sign in
visit /profile, switch between profile submenus
observe hard nav

Same applies to orgs fwiw

Hey @arlyon - I just pulled down your reproduction and tested it, and there were no hard navigations performed when switching between profile tabs. Opening the developer console while the navigation is being performed shows dom elements being patched directly, and the network tab shows that there is no hard navigation running.
Some follow up questions:

I used the keys for a test clerk app I have set up. Did you have a different experience when using the reproduction? If so, the only difference would be the keys, so it may have to do with some configuration in the app you were using to test, though I can't really imagine what it would be.
Do we have different definitions of "hard nav" perhaps? I was seeing the app re-render, which sometimes causes a brief flicker on the screen as the dom is updated. Or are you interpreting the change in url as a hard navigation? If so, next.js normally will update the url directly even for a client-side navigation using the history api - seeing a change in the URL no longer means that a full navigation was performed after the introduction of this browser API.

Screenshot for reference

Hello 👋
We currently close issues after 40 days of inactivity. It's been 30 days since the last update here. If we missed this issue, please reply here. Otherwise, we'll close this issue in 10 days.
As a friendly reminder: The best way to see an issue fixed is to open a pull request. If you're not sure how to do that, please check out our contributing guide.
Thanks for being a part of the Clerk community! 🙏

Hello again 👋
After 40 days of no activity, we'll close this issue. Keep in mind, I'm just a robot, so if I've closed this issue in error, please reply here and my human colleagues will reopen it.
As a friendly reminder: The best way to see an issue fixed is to open a pull request. If you're not sure how to do that, please check out our contributing guide.
Thanks for being a part of the Clerk community! 🙏