Possibilty to add styles/class to root span was added

And now you can do like this:
<Flex column alignItems="stretch" css={ style.youtube }>
  <Youtube
    opts={ {height: "100%", width: "100%"} }
    rootStyle={ {height: "100%", width: "100%"} }
    videoId={ post.channel.unsubscribedTrailer }
  />
</Flex>

Hey @furagi
That element was inserted there to allow React to properly remove the player from the page, since the YouTube API replaces the div, causing React to be unable to find the original element.
Specifically a <span> was chosen as it shouldn't interfere with the presentation of the player on the page.
Because of this having a style property is potentially confusing as these styles are not necessarily also applied to the iframe contained in the span. However a root/wrapper-style property could be useful to certain people as long as it is made clear that this is not applying the style to the iframe.
If you could add some documentation about this property to README.md which clearly states that this is applying the style to the <span> and therefore can behave somewhat unexpectedly I don't see a reason why we shouldn't merge this.
You also added a rootClassName, i'd prefer it if we "forced" users to apply their classes to the iframe so unless there is a good reason to add it maybe you can remove it from this pull request.

@tjallingt right now I'm struggling because the inability of put a class or id in that span...  I guess i'll have to use a nth-child-* ... I think what @furagi did is seriously useful!

Hey @I-NOZex can you explain what you are trying to achieve by styling that span element?
By default in html span elements are displayed inline, meaning they shouldn't affect the styling of the elements inside or around them.
If you give me more information about your situation it could help make a better decision if we need to merge this pull request.


bugfix: mutable state should be reset if the operation is not success…

…fully

Coverage increased (+0.003%) to 64.862% when pulling a9a6e841e1127f65fcb24a8d91a33f33d78990d0 on bugfix-mutablestate-x into eed3c4d9bde3e45c2a587e739d0623320fe9c876 on master.


No-implicit-any checks do not work with destructuring assignment

function f([a], {b}) { // should be an error
}

Same problem here. Quite annoying because it's silently leaking "any"-s and developers don't notice it in order to manually add a type annotation.


Pipeline not triggerred/retagged for dockerhub build on quickstart:latest?!

What problem does your feature solve?
Pipeline not triggerred for dockerhub build on quickstart?!
https://hub.docker.com/r/stellar/quickstart/tags?page=&page_size=&ordering=&name=latest
What would you like to see?
latest should be rebuilt, with this commit https://github.com/stellar/quickstart/commit/6cc1a0b9af10f7bef8f849f0b459832fffa60bef ?!
or this tag is not used more?!
saw the pipeline triggered some build like v426-future-amd64 though
What alternatives are there?

It appears the most recent commit failed to build the arm64 image on the master branch, and so subsequently the latest tag wasn't pushed because it is a multiarch image and all archs need to be built to push it.

Rerunning the build to see if it was an intermittent issue:

https://github.com/stellar/quickstart/actions/runs/9181545406/job/25769027303

@jun0tpyrc The rerun succeeded and image is pushed. Please reopen if the issue continutes.


Cannot read properties of null (reading 'forEachFeatureAtCoordinate')

Describe the bug
forEachFeatureAtPixel errored in our application. Cannot read properties of null (reading 'forEachFeatureAtCoordinate')
I would expect (not 100% sure) that it was on a map where the target had just been removed.
https://github.com/openlayers/openlayers/blob/main/src/ol/PluggableMap.js#L635
To Reproduce
Steps to reproduce the behavior:
map.renderSync();
map.setTarget(undefined);
map.forEachFeatureAtPixel([1, 2], () => {});

Expected behavior
No error. Would it make sense to guard again a null renderer_ in PluggableMap or is it unsupported to call functions on the map when it has no target?

Guarding agains renderer_ being null does make sense, and adding |null too. This would mean that forEachFeatureAtPixel et al. would not return features when a map does not have a renderer (which happens when it does not have a target), which makes sense because without target, we don't have a way to know how a pixel translates to a coordinate.
Would you be able to create a pull request, @EvertEt?

Will do, 1 question:
What would be the preferred way to handle a nulll renderer_ in renderFrame_?
    this.renderer_.renderFrame(frameState);

renderFrame_ won't be called when there is no renderer. So a type cast should be fine.


[AutoPR applicationinsights/resource-manager] Add applicationinsights/resource-manager/readme.typescript.md

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4146

(message created by the CI based on PR content)


Story: Limit number of metrics/params/deps/outs displayed wrt experiments

From testing with a DVCGen repository of 1000 metrics+params combined:
https://user-images.githubusercontent.com/37993418/162337259-b5e6de58-af9a-45fa-8b3a-81b2f318b534.mov
Some ideas to improve performance:

Only display the first 100 metrics, first 100 params, and first 100 data fields in the table, leave the rest as unselected.
Introduce a config option to only display the first 100 metrics, first 100 params, and first 100 data fields across the entire extension. Users would have to select their desired combination on the first load of a workspace for each repository within that workspace.

The first step is to introduce a telemetry event to be able to quickly confirm the size of our user base's repositories. We need to capture the number of metrics/params/deps/outs that could possibly be displayed. This will then let us know how much effort we need to spend trying to fix this issue.
Related to https://github.com/iterative/studio/issues/3257

Closing placeholder.


"columnstore table does not allow a primary key constraint" is false?

This documentation states "By design, a columnstore table does not allow a primary key constraint."
This doesn't appear to be true. See this example:
CREATE TABLE t_account (
AccountKey int NOT NULL PRIMARY KEY NONCLUSTERED,
AccountDescription nvarchar (50),
AccountType nvarchar(50),
UnitSold int
);
CREATE CLUSTERED COLUMNSTORE INDEX t_account_cci ON t_account ;
This executed fine in SQL 2017. I think what the documentation intends to say is that the table cannot have a clustered primary key, since the table is a clustered columnstore  index.

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

ID: 3e021430-b8b5-1a96-1e71-c95d72580c0f
Version Independent ID: 9f8a8520-31ef-eb8f-3ff5-c92db7d69514
Content: Columnstore indexes - Data Warehouse - SQL Server
Content Source: docs/relational-databases/indexes/columnstore-indexes-data-warehouse.md
Product: sql
Technology: table-view-index
GitHub Login: @MikeRayMSFT
Microsoft Alias: mikeray

@samrueby -- Sam, thank you for your feedback and your researches.
@MikeRayMSFT -- please look into this issue 3938.


Preview Plan Removal

Description
Changes proposed in this pull request:

Preview plan removal.

Related issue(s)

/test pre-main-control-plane-reconciler-upgrade-kyma2-latest-to-main-gardener

/retest

/retest

/test pre-control-plane-components-kyma-environment-broker

/test pre-main-control-plane-gke-integration pre-main-control-plane-integration

/test pre-main-control-plane-protecode-guard

/test pre-main-control-plane-gke-integration pre-main-control-plane-integration

/test pre-control-plane-components-kyma-environment-broker pre-main-control-plane-gke-integration pre-main-control-plane-integration pre-main-control-plane-protecode-guard


suggestion: use Deno's in-built formatter

This means contributors don't have to download another tool they may not already have. You can migrate Prettier's settings to Deno's formatter settings. See https://docs.deno.com/runtime/manual/getting_started/configuration_file#fmt

Deno's fmt doesn't have as many features as I'd like, but it does seem to have a bit more than I remember so I'll look into it later.

3076754085f0b8d81734771a4b203dd5fab273f1


Remove legacy infinite loop blocker

Context: https://github.com/istio/istio/issues/14242
Original PR: https://github.com/istio/istio/pull/15833/files
Back then, we had an issue where the traffic for inbound could match the
outbound listener, which would lead to it being redirected through
passthrough cluster back to itself, indefinitely. This was in the 1.1.x
times. Now, we have a split listener, so this issue should never occur,
as we have another fix for this case
(https://github.com/istio/istio/pull/15906).
This PR cleans up the code/config so we are no longer sending an
additional filter chain on every listener.
I have tested this using the original regression test
(https://github.com/istio/tools/tree/master/perf/stability/looper) as
well as deploying this in the same config that I spotted the original
issue in, and have detected no issues with this disabled. However, I am
not 100% sure, so I will place a hold on this until I get multiple
approvals, since this is a pretty high impact change if I am wrong and
this is not safe to remove.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
3afe56219e9e1a84d3d03e1ebce2b12516d4210f
link
/test unit-tests_istio

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

memory usage over 12hr with this set:

trying to figure out now if its caused by this or something else

I am fairly sure the leak is a telemetry v2 issue and not this PR, will investigate more


Directive resolver input arguments type ignores enumPrefix = false

Describe the bug
The generated directive arguments ignore enumPrefix = false. This results in the wrong type name when typesPrefix is set.
To Reproduce
https://codesandbox.io/s/pedantic-lalande-q0g1qr?file=/types.ts
Set enumPrefix = false and typesPrefix = I.
Define a directive with an input argument that is an enum and generate the resolvers.

My GraphQL schema:

enum TestEnum {
    ONE
    TWO
}

directive @debug(in: TestEnum) on FIELD_DEFINITION | OBJECT

My GraphQL operations:
None

My codegen.yml config file:

schema: schema.graphql
generates:
  types.ts:
    plugins:
      - typescript
      - typescript-resolvers
    config:
      typesPrefix: I
      enumPrefix: false
      defaultMapper: unknown

Setting defaultMapper works around another name collision with the default IResolversTypes and IResolversParentTypes. Normally I'd map the model types.
Expected behavior
Does not add the types prefix to the enum type in the input args type for the directive resolver.
Environment:
https://codesandbox.io/s/pedantic-lalande-q0g1qr?file=/types.ts
Additional context
I don't actually need the directives so a skip directives option would work as well.
I may be able to get a hook to rename it as a workaround for now.

Specifically generating this code:
export type IDebugDirectiveArgs = {
  in?: Maybe<ITestEnum>;
};

Instead of this code:
export type IDebugDirectiveArgs = {
  in?: Maybe<TestEnum>;
};


Upgrade gradle to 4.7

Upgrade gradle and the lombok plugin (because the old plugin used a deprecated gradle api)

Hm, I'm not sure, if my change broke that test.

@guenhter Travis seems to be a bit flaky, I don't think it was caused by your change


file-roller: 3.40.0 -> 3.41.90

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Change log: https://gitlab.gnome.org/GNOME/file-roller/-/compare/3.40.0...3.41.90
This breaks pantheon.file-roller and it should be okay to drop the package. @ofborg build pantheon.file-roller
NEWS says this is a beta release, are there anything that motivates you to bump this package now?

NEWS says this is a beta release, are there anything that motivates you to bump this package now? (Otherwise this should already be part of https://github.com/NixOS/nixpkgs/pull/160343)

Oh, I'm not aware of that. Thanks for pointing it! I'll close this one.

This breaks pantheon.file-roller and it should be okay to just drop all elementary specific stuff and alias pantheon.file-roller to the vanilla one.

Actually I almost forget to do that, I opened #163828.


Provide screen reader support for the filters

Screen reader experience when setting and changing filters is poor.
System: Windows 10 build 1803, with NVDA and Narrator.
To replicate:
Run an Ace report
Move to the Violations tab and select it
Move further down to Filter by button which should be collapsed. Expand it (with space) and with a screen reader you hear "Expanded".
Tab into the filter boxes and you will hear "Edit".
There are four filter boxes across the screen, these are impact, rule set, rule and location. This text is shown on the screen in each edit box. As you move between the boxes it is not obvious with a screen reader which one you are in. With NVDA it says edit, Narrator is silent.
Alt + down opens a list. With NVDA the list immediately disappears if you try to move down it. With Narrator it behaves better, but the lists are completely silent.
If you manage to add a filter it is added to the edit box, but this is not announced by the screen reader. It is not possible to determine which filters are set.

Development note: the current filters are created with react-select (https://react-select.com/home)

There is a chance the component we use will get updated with better a11y support, but it's hard to tell from the issues what the timeline is:
https://github.com/JedWatson/react-select/issues/2456

Seems edit boxes are not labelled by their corresponding headings. Also, the tooltips read, for example "press arrows to open a menu" (to select an option like moderate and so on, are not localized (always read in English).
While this is nt fixed, baybe useful some kind of documentation explaining that screen readers may prefer to use browser mode (in NVDA), virtual cursor in JAWS, etc.


scx_git: 641d6357bb604008bdac5190b854e68dc7667d88 -> b4db5ee9f6611c2f…

Finally, nixpkgs PR coming soon once I figure out the patches to cherry-pick.

LGTM!


Ability to delete messages

There's been an issue on twister-html that belongs in core: https://github.com/miguelfreitas/twister-html/issues/114 I wrote a bit of a response detailing a way to do this.

Would it be possible to put in code into the client to honour remote deletion flags?
I know it could be removed and recompiled, but having it as an action for official binaries would help new users adjust better.

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512
I personally am against giving a false impression that something has
been deleted. I would put more effort into educating the up-front
about these differences and why they exist.
On 02/03/15 05:00, rodneyrod wrote:

Would it be possible to put in code into the client to honour
remote deletion flags?
I know it could be removed and recompiled, but having it as an
action for official binaries would help new users adjust better.
--- Reply to this email directly or view it on GitHub:
https://github.com/miguelfreitas/twister-core/issues/253#issuecomment-76655698

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1
iQIcBAEBCgAGBQJU9FepAAoJEBN/k2mDKuUl0DwQALp0sCeNUdx6g1EJfR3h3Co8
sZ5uLJmAwVlvOqHsBQOz7RmjRcVhTit7MzqxSJv5JO950k5ZSgvAOeVua7M5D/uF
fbnfMXocOALJ4+d4S9JBnk6c8F83YhLW1lSihZLSWz7JL4zC6G5yrRuy5i9lM5yS
yIaoZ1hBRmn/KiiNL4lq5+yt87lSt0DMBc67kElEjv4ZYW9u0p29hLdX3z3x6Abd
tRHKyo2Y3xiirN+bgkTtNN384tdXP4bMo580clAQfV1wlAP+ff6SdUpeUI4ZZWzv
vDVRCnFuOnra1+JiXCsTMkb+R/hr0c403X+OyfFD1fc7v3mVB4o/mkonJDSBVQ76
Yj0K5Cvvr0xoIjCIONEESp/BVYFJfkQ8uYerduNuGpHaX4NHpsJt7kYSy/YyK7Uq
qvUMF+zi99Cu619iZkgD+tiNehK1eQXqMTvQ5wgAVTE0JT0mtB0jPq+YIzoMdSDq
nn1zkCrOCi7a9/+Y9ty3iwNNiVGhRkLLbiQ2PqryCDbxqqMQBrIVT/c5QyDjbcFH
OfM0uzSnb4G2DbyPjnoTF4Lv0XoXvQ6Ez1Lfzu202+2T6Rgf9IOiefq1yDJP2Eub
V0SmOwYKoJmNlXnzVZ2z5FX0EV/DrAEV3B99Fj3/jcArDlftJi0CIkrYgvHm0ouh
zJQBgq3t4aAo7/OeyK69
=VMjG
-----END PGP SIGNATURE-----


V2.x Echo here/leaving not triggering

Hello, I just started using Laravel-Websockets and decided to check out v2 as I'll be needing redis and multi-server support.
I noticed that Laravel Echo .here() and .leaving() are not being triggered. It was working in version 1 when I first tested, updated to v2 and both functions have stopped working. I have backed down to v1 again and both functions work again.
I realise this is still in beta, just wanted to report or see if it was a known issue.

An update on my issue.
I updated back to v2 and changed the LARAVEL_WEBSOCKETS_REPLICATION_DRIVER back to local and everything is working as expected.
When switching back to Redis, I can see the key/value pairs in my Redis DB that relate to the Laravel-Websocket connections so up to this point it is working. What I can't get to work is broadcasting these pairs that will no doubt trigger the here/leaving functions.
Any pointers on what we should be doing within Laravel to keep an eye on Redis? Usually I have Horizon running but so far this hasn't solved anything.

We are aware that this is caused by the fact that the Redis replication doesn't work with the presence channels and we're working on a fix, as v2 is needed to reach stability.
Horizon is used for queues, and it has nothing to do with the Redis on the Pub/Sub side.

.leaving() do get triggered, but .here() is not. I have to investigate this.

.leaving() do get triggered, but .here() is not. I have to investigate this.

I can confirm, .leaving() does work, .here() does not. Hopefully a fix can be found soon.
Keep up the great work!

2.x should fix the .here() method :D
Read here the changelog: https://github.com/beyondcode/laravel-websockets/pull/447
Make sure to get a new copy of the config fule.

Nice, the .here() method is working.
I have an issue with broadcasts duplicate but will create another thread if I can't figure out what's going on.

Also just ran into this issue.. .here() works but .leaving() does not and as a result .here() shows a ton of users that are no longer active
Why is this closed @rennokki ?

@vesper8 I will look into it tomorrow. I have reopened this for now as the problems were signaled for .leaving().
If you can, try to use the redis mode and see if the bug goes away. If it goes, then the local channel manager should be further debugged.

Hey @rennokki,
I can confirm that using the redis mode makes leaving() work correctly so that's a yay
However I noticed that prior to switching to redis, I had already "joined" with a user who's browser I have closed long ago, but this user remains "online" now
I don't know how to flush this "online" cache ? I don't want to flush all of redis.. and besides this user must be stuck in the "local" mode somehow
It would be useful if there was a command for flushing this data.. you could pass the app id or app name to the command and it would flush everything related to that app ?
On a side note.. I tried running websockets:flush, not realizing that this is only related to statistics, and I received this error:
Flushing the collected WebSocket Statistics...

   Illuminate\Contracts\Container\BindingResolutionException

  Target [BeyondCode\LaravelWebSockets\Contracts\ChannelManager] is not instantiable.

  at vendor/laravel/framework/src/Illuminate/Container/Container.php:1017
    1013▕         } else {
    1014▕             $message = "Target [$concrete] is not instantiable.";
    1015▕         }
    1016▕
  ➜ 1017▕         throw new BindingResolutionException($message);
    1018▕     }
    1019▕
    1020▕     /**
    1021▕      * Throw an exception for an unresolvable primitive.

      +32 vendor frames
  33  artisan:37
      Illuminate\Foundation\Console\Kernel::handle(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))

It would also be useful if the dashboard could confirm which "mode" is being used
Of course it would be super awesome if the dashboard would report more statistics.. such as displaying all channels currently subscribed too.. including public, private and presence channels, and displaying how many connections are included in each

According to the Pusher protocol, there is a maximum amount of 120 seconds since a connection should not pong in order to be automatically be closed: https://pusher.com/docs/channels/library_auth_reference/pusher-websockets-protocol#recommendations-for-client-libraries
Did the connection last more than 120s after you realized the user won't go away from the prsence channel?

Yes.. it's been 13 hours and .here() is still showing users from 13+ hours ago

any update on this @rennokki ? It's been 2 weeks and those users are are still stuck inside the "online" presence channel even though I've rebooted multiple times since then.
At least can you give me an idea of what cache I need to flush to get rid of them?

Figured out how to delete that key. Also made this helper command to flush out all websocket-related keys from the redis cache
<?php

namespace App\Console\Commands;

use Illuminate\Console\Command;
use Illuminate\Support\Facades\Redis;

class FlushWebsockets extends Command
{
    protected $signature = 'websockets:flush';

    protected $description = 'Flush Websockets';

    public function handle()
    {
        $apps = config('websockets.apps');

        foreach ($apps as $app) {
            $redisKeys = collect(Redis::keys(sprintf('*%s*', $app['id'])))->each(function ($fullKey) use ($app) {
                $key = str_replace(config('database.redis.options.prefix'), '', $fullKey);
                $this->info(sprintf('[%s] Deleting redis key <%s>', $app['name'], $key));
                Redis::del($key);
            });
        }
    }
}

Maybe it would be worth including this command in this package?

I will need to find a way to integrate this command for the Redis part only, or at least write an endpoint in the server that can be HTTP called and flush them using some kind of authorization.
Enhancement: Regarding the issues with the flushing, I assume your cluster doesn't allow you to call SMEMBERS, SREM, ZREM, or HDEL since these are the commands used to remove users, either from the presence or keeping tracking of the socket IDs internally in the Redis instance.
There is a Redis-related bugfix too prior to -beta.30. Not sure if this really helps.

@vesper8 So the issue was actually because of a hard stop of the process. I have implemented a while back a way to listen to PCNTL signals and make sure to close any new connections to that specific instance and mark all connections as closed to the channel manager.
However, I plan on fixing the stale data regarding the stored presence channel users by searching for users that have sockets opened, older than 2 minutes (as described in Pusher Protocol Reference), so this way we can clean up all the stale members.

excellent catch @rennokki : ) so happy this project is evolving the way it is and largely thanks to your efforts! As always I look forward to your future releases. Keep up the fine work!

@vesper8 So the issue was actually because of a hard stop of the process. I have implemented a while back a way to listen to PCNTL signals and make sure to close any new connections to that specific instance and mark all connections as closed to the channel manager.
However, I plan on fixing the stale data regarding the stored presence channel users by searching for users that have sockets opened, older than 2 minutes (as described in Pusher Protocol Reference), so this way we can clean up all the stale members.

This is still happening to me all the time.. I have users still in the presence channel that don't even exist anymore in the users table. So I'm guessing you haven't had the chance to implement the above fix yet. I hope it's on the roadmap to be added soon.
In the meantime is there a workaround short of manually flushing Redis every so often?


bosh deployment error on vsphere

Can anyone please help me on this?
Thank You

We got it. cluster is to be in "[]"
eg: [CF-Cluster]


[Improve][Transform] Ignore error with jsonpath transform

Purpose of this pull request
[Improve][Transform] Ignore error with jsonpath transform
Does this PR introduce any user-facing change?
Add new config item
How was this patch tested?
Added
Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist
Add ci label in label-scope-conf
Add e2e testcase in seatunnel-e2e
Update connector plugin_config

[ ] Update the release-note.

good idea


Create encoding-specific conformance classes

This would follow the approach used in OGC API - Features, Tiles, Maps...
Proposed conformance classes:

CIS JSON (+RDF to include JSON-LD @context ?)

netCDF

GeoTIFF

CoverageJSON

LAS/LAZ?

Zarr?

JPEG2000 with GML/JP2?

JPEG XL?? (eventually: how to add GML/JP2 or JSON in JPEG XL?)

PNG? (to populate GeoPackage Gridded Coverage Extension) -- with factor and offset query parameters

Cloud native conformance class indicating a cloud optimized profile and support for HTTP Range (+ presence of overviews?) #93

@jerstlouis The right hierarchy would be

CIS

JSON
RDF
netCDF
GeoTIFF
etc.

CoverageJSON

Shouldn't this be extensible? If you prescribe allowed options, what it someone defines COG 2.0?

@m-mohr Yes, it is always fully extensible.
As with OGC API - Features, Tiles, EDR and others, even though some conformance classes are defined in the standard itself to facilitate declaring support for common formats, the standard is clear that an HTTP Accept header can always be used to negotiate any other format (and if necessary, a future extension/part to the standard could define a conformance class for those).

Great to hear. If I can request all file formats through media types (as long as there is one), why do they need conformance classes?

@m-mohr For the following reasons:

Following in the footsteps of Features which had GeoJSON, GML
To provide a list of formats from which several implementations will at least support one of
Potentially as a way to procure an implementation that supports at least one or more of those common formats
As an easy way for clients to know beforehand which formats are available (at least for those common formats) -- related issue: https://github.com/opengeospatial/ogcapi-common/issues/160

An initial draft of the encoding-specific requirements classes have been merged with #172.

So how do I detect a unique list all supported file formats? I'm implementing a client and it must let the user choose the file format he wants to load.
As far as I understand:

Get list of conformance classes and filter the file format specific ones out. Q: How do I know which ones are file formats if there can always be non-OGC conformance classes, too?
Get all links with a specific rel type and list the media types. Q1: Is there a way to map conformance classes to media types? Q2: What happens if I want to expose STAC JSON and Records JSON, which both currently just use the application/json media type? Q3: What is meant to happen if I find a link without a type property?

@m-mohr Tough questions ;) But they apply equally to all OGC API standards, so something to also discuss in Common and/or with the Cross-SWG issue discussions. But trying to answer each point here:

Q: How do I know which ones are file formats if there can always be non-OGC conformance classes, too?

Not possible for those non-OGC conformance classes in a generic way. Clients that support specific formats can easily identify conformance classes that they recognize whether part of the standard or coming some somewhere else. The reason to include these conformance classes in the first place is to at least have a standard way to identify common formats expected to be supported by several implementations, so at least all clients can recognize these. Generically recognizing conformance classes outside of the standard is not possible.

Q1: What happens if I want to expose STAC JSON and Records JSON, which both currently just use the application/json media type?

On one hand this could be the general problem that application/json is really not an encoding implying a particular logical model, like XML, it's a generic transport for arbitrary logical models, and negotiating a particular profile really needs more information, either using a different media type or with the negotiation by profile ( https://github.com/opengeospatial/ogcapi-common/issues/8 ). In this specific case of STAC vs. Records though it may be related to the question of whether STAC is a profile of records ( has that been resolved in https://github.com/opengeospatial/ogcapi-records/issues/178 ?). Is this related to https://github.com/opengeospatial/ogcapi-records/issues/276 ? I am not clear how any of this relates to Coverages though ? Is this about /items?

Q2: What is meant to happen if I find a link without a type property?

If a link does not include the type property, then the resource likely supports multiple (possibly several) representations, and the client should use an Accept: header with those representations that it supports. If this is /coverage and the implementation lists those encoding requirements classes in /conformance, then the implementation supports all of these. The client should assume that the server does not support those encodings if the requirements classes defined in the standard are not listed.

I guess the relation types in links take preference over the conformance classes in such a case? Or should they be merged?

Do you mean the type and not the rel here? The type specifies the type for a particular link (e.g., /coverage?f=coveragejson), but there should either be no type, or a link of each type listed in the conformance classes. The two should be consistent.

If yes, is there a way to map conformance classes to media types?

There is a table in the latest draft that does exactly this:
https://github.com/opengeospatial/ogcapi-coverages/blob/master/standard/clause_14_encodings.adoc#media-types

Not possible for those non-OGC conformance classes in a generic way.

Okay, so this is not a reliable way to generate a list of supported file formats. Let's look at the other ones then (i.e. the links).

On one hand this could be the general problem that application/json is really not an encoding implying a particular logical model, like XML [...]  I am not clear how any of this relates to Coverages though ? Is this about /items?

It was just meant as examples to visualize the issue better. But in principle this can occur for all media types. If I want to support two different flavors of ZARR or netCDF, the same applies basically. I just think that content negotiation by media types (IANA is in priciple relatively strict with "additions" to media types) might get problematic/ambiguous.

In this specific case of STAC vs. Records though it may be related to the question of whether STAC is a profile of records ( has that been resolved in opengeospatial/ogcapi-records#178 ?).

STAC will not be a profile of Records in the first iteration, but might be in the future. It is closely aligned but not fully.

Is this related to opengeospatial/ogcapi-records#276 ?

Yes.

If a link does not include the type property, then the resource likely supports multiple (possibly several) representations, and the client should use an Accept: header with those representations that it supports. If this is /coverage and the implementation lists those encoding requirements classes in /conformance, then the implementation supports all of these. The client should assume that the server does not support those encodings if the requirements classes defined in the standard are not listed.

So If I write a client such as the GDC Web Editor which let's the choice to the user and in principle supports all media types, what shall I do? I want the user to choose from all supported file formats.
Should I send a HEAD .../coverage request and send header Accept: */*, but what do I get back then? The Content Type is usually just a single value, not a list, right?

Do you mean the type and not the rel here? The type specifies the type for a particular link (e.g., /coverage?f=coveragejson), but there should either be no type, or a link of each type listed in the conformance classes. The two should be consistent.

I meant if I have a list of supported file formats from the conformance classes and one from the type in the coverage links, which list should I show to users in the GDC Web Editor? I assumed the links have preference, although I can't show a list if there's no type set (see above).

https://github.com/opengeospatial/ogcapi-coverages/blob/master/standard/clause_14_encodings.adoc#media-types

Good to know, thanks.

@jerstlouis Copied from another Gitlab issue to have everything in one place:

Regarding the media type when no type is included, it should be considered multiple, with all of the listed coverage encodings supported as listed in /conformance.

Above, you've said above I can't reliably read that from the conformance classes. So it looks like there is no definitive way to get a list of supported file formats right now, right? This should be fixed, I think.

Let's look at the other ones then (i.e. the links).

That's not a reliable way either because several implementations may opt to have a single /coverage link without a type.

So If I write a client such as the GDC Web Editor which let's the choice to the user and in principle supports all media types, what shall I do? I want the user to choose from all supported file formats.

Should I send a HEAD .../coverage request and send header Accept: /, but what do I get back then? The Content Type is usually just a single value, not a list, right?

I think this is related to our earlier discussion about at which point you care about which media type to use.
In the end, whether in a particular hop of a workflow, or in the end-user client application, something will need to visualize or analyze that coverage data, and this will support a specific list of formats.
This is where the Accept: ... list should ultimately come from, and would never be */*.
In the case of the GDC editor, is there a need to ask the user to pick one?
It may be the list of the formats that the browser supports, or a predetermined list of formats that a particular GDC profile requires to be implemented.

Otherwise, you/I can't write a user-friendly client.

Just as you said earlier, possibly in a slightly different context:

the user although they probably don't care which file format is used internally to retrieve the data. They just care what comes out at the end... :-)

I don't agree with your point that the user needs to specify the file format for the coverage request.

Ideally, I think a user-friendly client should not require the user to deal with formats :)

I meant if I have a list of supported file formats from the conformance classes and one from the type in the coverage links, which list should I show to users in the GDC Web Editor? I assumed the links have preference, although I can't show a list if there's no type set (see above).

I think the list of supported formats to show would be all of those from the conformance classes, plus any additional conformance classes not in the standard that the client is aware of, plus any type ones that are not already mapped to any of these.

So it looks like there is no definitive way to get a list of supported file formats right now, right? This should be fixed, I think.

Nothing better than the above, but this applies exactly the same to all OGC API data access standards, including Features.

Ideally, I think a user-friendly client should not require the user to deal with formats :)

No! As a user I want to choose whether my end result is a COG or a netCDF.
In-between the steps/Internally it doesn't care, but the end result I want to choose. And to make an informed decision I need to know upfront which file formats are available. There's no reliable way for this in OGC APIs it seems, so it should be added. Where should such a feature be requested? Here? In Commons?

As a user I want to choose whether my end result is a COG or a netCDF.

In a connected environment the end-user should not need to care and just see results or get an answer to their question, but this is getting philosophical :) Of course the need for intermediate exports of data still persists, so this is a use case to addres.

And to make an informed decision I need to know upfront which file formats are available

As stated above the client can get this right now:

all of those from the conformance classes,
plus any additional conformance classes not in the standard that the client is aware of,
plus any type ones that are not already mapped to any of these.

This may not seem ideal, but it's not that bad in practice.
If OGC API - Common - Part 1 is supported, and the client supports dynamically parsing the particular API definition language used to describe the API (e.g., OpenAPI) for the deployed implementation, then of course the client can also look at the paths responses media types.

Where should such a feature be requested? Here? In Commons?

This issue https://github.com/opengeospatial/ogcapi-common/issues/160 that I linked to on GitLab is the central place for this discussion I believe. Perhaps optional requirement classes to define requirements for HEAD and/or OPTIONS and/or something similar to the list of media types available in openEO could be a solution? The list of media type would need to be resource type-specific.

This may not seem ideal

It is not ideal, it is not reliable and I can't build a client on top of it.

something similar to the list of media types available in openEO could be a solution? The list of media type would need to be resource type-specific.

openEO defines the file formats globally for input and output respectively. We don't really have different resource types (except for a broad distinction between raster, vector, table and "other",

https://github.com/opengeospatial/ogcapi-common/issues/160

Thanks, added a short comment there, linking back to this discussion.

@m-mohr

it is not reliable and I can't build a client on top of it.

The above strategy (standard/client-recognized conformance classes + type) is fully reliable (with the only exception of media types not part of the standard conformance classes and of which the client is not aware), and I think a client can detect / offer that choice, though it might be a bit of work, so it is not clear what you mean by "I can't build a client on top of it".
Are you referring specifically to that exception, do you mean it is significantly more work than it should be, or is there really a technical limitation that I am missing?

We don't really have different resource types (except for a broad distinction between raster, vector, table and "other",

For OGC APIs, strictly for "data"  we at least havecoverages, maps, feature collections, individual features, vector tiles, coverage tiles, map tiles.
Then there are all the metadata resources like collection lists / collection descriptions, process lists / process descriptions...

@m-mohr I concur with your statement that the user should be able to 1) get a list of available formats and 2) choose the one they want. Scanning conformance classes to get this information seems overly complex :-)

Feb 24 code sprint: do we need a separate conformance class with requirements for GML/JP2 ?

Here are some more that eventually may need conformance classes: https://gdal.org/en/latest/drivers/raster/index.html
And that's why this doesn't scale well and there needs to be more than just conformance classes for file format discovery.


Failure output for dart fix --compare-to-golden is difficult to parse

When testing dart fixes with

dart fix --compare-to-golden

a failure outputs the entire test file's expected output and the entirety of the actual output.
We have many many fixes and tests in the framework, and when this fails it is very difficult to debug hundreds of lines of before/after code.
I always just copy paste it into a diff-er to find out what did not turn out as expected.
Can dart fix truncate this failure output so it is easier to debug where a transformation is not working?

We should certainly be able to remove common lines from the beginning and end. We might be able to do even better than that.


PR: Don't run continue when starting debug if there is a breakpoint in the first line with code.

Fixes #4681
I implement another approach that could be less error prone than manually checking for the first line of code.
I take advantage that when ipdb stops, the editor goto the line being debugged.
This could be improved when addressing #528 and adding a variable (or method) that will return the current line being debugged.

@rlaverde, unfortunately this doesn't fix #4681 in all situations. I'll open a new PR with a new proposal.


Mac OS X installation issue

Hi ...I am quite new to all this ...and while installing the steps given in the installation,  I am facing following error. I have tried to install using all the three methods given in https://github.com/VisionLabs/torch-opencv/wiki/Installation but getting same error. I have also installed opencv3 using brew.
CMake Error at CMakeLists.txt:22 (FIND_PACKAGE):
Could not find a configuration file for package "OpenCV" that is compatible
with requested version "3.1".
The following configuration files were considered but not accepted:
/usr/local/share/OpenCV/OpenCVConfig.cmake, version: 2.4.13

-- Configuring incomplete, errors occurred!
See also "/Users/viral/torch-opencv/build/CMakeFiles/CMakeOutput.log".
make: *** No targets specified and no makefile found.  Stop.
Error: Build error: Failed building.

have you tried
brew instal opencv3 --with-contrib
OpenCV_DIR=/usr/local/Cellar/opencv3/3.1.0/share/OpenCV luarocks install cv

@szagoruyko yes I have already tried that solution. It is giving the same error.

Last login: Sun Jul 17 18:05:02 on ttys000
VIRALs-MacBook-Pro:~ viral$ brew instal opencv3 --with-contrib
Warning: homebrew/science/opencv3-3.1.0_3 already installed
VIRALs-MacBook-Pro:~ viral$ OpenCV_DIR=/usr/local/Cellar/opencv3/3.1.0/share/OpenCV luarocks install cv
Installing https://raw.githubusercontent.com/torch/rocks/master/cv-scm-1.rockspec...
Using https://raw.githubusercontent.com/torch/rocks/master/cv-scm-1.rockspec... switching to 'build' mode
Cloning into 'torch-opencv'...
remote: Counting objects: 202, done.
remote: Compressing objects: 100% (180/180), done.
remote: Total 202 (delta 40), reused 76 (delta 19), pack-reused 0
Receiving objects: 100% (202/202), 2.03 MiB | 2.02 MiB/s, done.
Resolving deltas: 100% (40/40), done.
Checking connectivity... done.
cmake -E make_directory build;
cd build;
cmake .. -DCMAKE_BUILD_TYPE=Release -DCMAKE_PREFIX_PATH="/Users/viral/torch/install/bin/.." -DCMAKE_INSTALL_PREFIX="/Users/viral/torch/install/lib/luarocks/rocks/cv/scm-1";
make
-- The C compiler identification is AppleClang <IP_ADDRESS>30031
-- The CXX compiler identification is AppleClang <IP_ADDRESS>30031
-- Check for working C compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc
-- Check for working C compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++
-- Check for working CXX compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
CMake Error at CMakeLists.txt:22 (FIND_PACKAGE):
Could not find a configuration file for package "OpenCV" that is compatible
with requested version "3.1".
The following configuration files were considered but not accepted:
/usr/local/share/OpenCV/OpenCVConfig.cmake, version: 2.4.13

-- Configuring incomplete, errors occurred!
See also "/tmp/luarocks_cv-scm-1-6393/torch-opencv/build/CMakeFiles/CMakeOutput.log".
make: *** No targets specified and no makefile found.  Stop.
Error: Build error: Failed building.
VIRALs-MacBook-Pro:~ viral$

I need to uninstall previous opencv3 using brew uninstall opencv3
then followed the steps again and I guess it is working now... thank you @szagoruyko


Meltano Invoke/Select does not invalidate catalog cache after reinstalling a tap

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2627
Originally created by @macoop2363 on 2021-03-04 21:55:13

What is the current bug behavior?
Taps are using cached properties for meltano invoke/select even if the tap's properties have changed after meltano install.
What is the expected correct behavior?
When running meltano install, a tap's properties cache should be invalidated so that meltano invoke/select should effectively "rediscover" the tap's properties.
Steps to reproduce
# install all taps
1. meltano install
2. meltano invoke tap-custom-tap
# update tap-custom-tap repo with some new schema properties
3. meltano install tap-custom-tap
4. meltano invoke tap-custom-tap

Relevant logs and/or screenshots
After running meltano install extractor tap-s3-toast
> meltano --log-level=debug invoke tap-s3-toast
[2021-03-04 21:28:53,618] [1|MainThread|root] [DEBUG] Creating engine <meltano.core.project.Project object at 0x7f510102d160>@sqlite:////projects/.meltano/meltano.db
[2021-03-04 21:28:54,642] [1|MainThread|root] [DEBUG] Created configuration at /projects/.meltano/run/tap-s3-toast/tap.config.json
[2021-03-04 21:28:54,643] [1|MainThread|root] [DEBUG] Could not find tap.properties.json in /projects/.meltano/extractors/tap-s3-toast/tap.properties.json, skipping.
[2021-03-04 21:28:54,643] [1|MainThread|root] [DEBUG] Could not find tap.properties.cache_key in /projects/.meltano/extractors/tap-s3-toast/tap.properties.cache_key, skipping.
[2021-03-04 21:28:54,643] [1|MainThread|root] [DEBUG] Could not find state.json in /projects/.meltano/extractors/tap-s3-toast/state.json, skipping.
[2021-03-04 21:28:54,643] [1|MainThread|meltano.core.plugin.singer.tap] [DEBUG] Using cached catalog file

Possible fixes
According to @DouweM and my Slack conversation, meltano invoke/select should have some logic to invalidate caches. Other options are removing the cache when running meltano install either by default or with an optional cli flag.
Further regression test
Ensure we automatically catch similar issues in the future

[ ] Write additional adequate test cases and submit test results
[ ] Test results should be reviewed by a person from the team

View 12 previous comments from the original issue on GitLab


d3.layout  is undefined

i tried to use d3.layout.tree() function, it works in the jsfiddle but not in my local pc, what should I do? i have linked d3.min.js too

Please read the CHANGES document in the d3 repo. The layouts were renamed in d3 4.0.


Project Status

Hello,
I'm currently starting out on integrating with HubSpot and decided to use this project, but straight away have found the nuget doesn't contain implementation for OAuth2. (Last updated Aug 2018). I can see the README shows an example using a OAuth connection. So its unclear if this project is still active. If it is, is it recommended to clone the repo directly and is it in a good state to use in a production environment?
Thanks

Hi @chrismoutray,
You are correct, the NuGet package currently does not support OAuth2. We have not released the OAuth features as of yet. Yes, if you need updated features, the recommended approach is to build from source and integrate it into your solution.
Although there are some users using this repo in a production environment, it's really up to you to decide that through testing of the features that you need.
We are currently in a preview of the full 1.0, so we cannot guarantee full compatibility with what you are trying to accomplish. If you find some specific issues, please open a new issue and we will try to tackle that there.
We are working with what we have to try and release a 1.0 package soon.

Thank you for reaching out @chrismoutray!

@Psypher9 Thank you for the quick response and really appreciate the effort put into this project.


[icons-test] load dynamic of icons

Compare to how MUI handled it? https://mui.com/material-ui/icons/


Zod enums create optional schema values

Description
Using nativeEnum or enum in 2.0 with strict = true produces a schema with optional values.
Version
2.0.0
Reproducible
tsconfig.json
{
  "compilerOptions": {
    "strict": true
  },
  "extends": ["./.svelte-kit/tsconfig.json"]
}

Produces optional value (value?: Type | undefined)
import { defaults } from 'sveltekit-superforms';
import { zod } from 'sveltekit-superforms/adapters';
import { z } from 'zod';

enum MyEnum {
  ValueA = 'ValueA',
  ValueB = 'ValueB',
}

const mySchema = z.object({
  value: z.nativeEnum(MyEnum).default(MyEnum.ValueA),
});
type MyType = z.infer<typeof mySchema>;
// type MyType = {
//   value: MyEnum;
// }

const myFunction = (_type: MyType) => {
  // no-op
};

myFunction(mySchema.parse({}));

const values = defaults(zod(mySchema));
// values.data: {
//   value?: MyEnum | undefined;
// }
myFunction(values.data);
//         ^ Argument of type '{ value?: MyEnum | undefined; }' is not assignable to parameter of type '{ value: MyEnum; }'.
//           Types of property 'value' are incompatible.
//           Type 'MyEnum | undefined' is not assignable to type 'MyEnum'.
//           Type 'undefined' is not assignable to type 'MyEnum'.

Previously working code (repro)
import { superValidateSync } from 'sveltekit-superforms/client';
import { z } from 'zod';

enum MyEnum {
  ValueA = 'ValueA',
  ValueB = 'ValueB',
}

const mySchema = z.object({
  value: z.nativeEnum(MyEnum).default(MyEnum.ValueA),
});
type MyType = z.infer<typeof mySchema>;
// type MyType = {
//   value: MyEnum;
// }

const myFunction = (_type: MyType) => {
  // no-op
};

myFunction(mySchema.parse({}));

const values = superValidateSync(mySchema);
// values.data: {
//   value: MyEnum;
// }
myFunction(values.data);

Note: Wasn't able to create a reproducible in stackblitz due to no parsing of .svelte files being configured

Maybe same as #335

Yes, this is a problem, but hard to solve. For the type to be correct, if an enum field is not optional or required, it must have a default value in the schema. But an existing default value creates an optional type, as that is the standard is most "toJSONSchema" libraries.
Previously, I was considering using the first enum value as the default, but that may not always be the intent. But I also understand that enums must be able to be required somehow. It's not a breaking change, so it can be a solution.

It's not related to Superforms or Zod, but a type problem that doesn't map to form input. The inferred schema type is { gender: Gender }, not Gender | undefined, so it must always have one of the gender values.
And then in the form, with bind:value, it will use the default value. Defining a Default won't work either if you want to require only the Male or Female values.
For the next release, the first enum value will be used (as in v1), so enums can be required again. And this may vary between libraries (their JSON Schema generators), but for Zod, if the default value doesn't match one of the enums, it will set the field to required, so it will work in this case!
const schema = z.object({
  letter: z.enum(['a', 'b', 'c']).default('' as any)
});

It's not related to Superforms or Zod, but a type problem that doesn't map to form input. The inferred schema type is { gender: Gender }, not Gender | undefined, so it must always have one of the gender values.
And then in the form, with bind:value, it will use the default value. Defining a Default won't work either if you want to require only the Male or Female values.
For the next release, the first enum value will be used (as in v1), so enums can be required again. And this may vary between libraries (their JSON Schema generators), but for Zod, if the default value doesn't match one of the enums, it will set the field to required, so it will work in this case!
const schema = z.object({
  letter: z.enum(['a', 'b', 'c']).default('' as 'a')
});

Is it a bug that this doesn't work. I just tested it in 2.3.0 and I get the same issue using zod.
const schema = z.object({
  letter: z.enum(['a', 'b', 'c']).default('' as 'a'),
});
const adapter = zod(schema);
const parsed: SuperValidated<Infer<typeof schema>> = defaults(adapter);
//    ^ Type 'SuperValidated<{ letter?: "a" | "b" | "c" | undefined; }, any>' is not assignable to type 'SuperValidated<{ letter: "a" | "b" | "c"; }>'.

From my understanding it worked in v1.0, but this doesn't work in v2.0.
The other way works by modifying the adapter, but it's pretty annoying as then the validation isn't entirely in the schema.

Use the other method instead: https://github.com/ciscoheat/sveltekit-superforms/issues/333#issuecomment-1938881450

Until there's a way to modify the inferred type with an option, it's how it must be. There's no way to use default without making the field optional. That's how most validation libraries work


Add receipts and fix BroadcastTx tests

With a bit of related refactoring.
Closes #109

FAIL	github.com/eris-ltd/eris-db/txs [build failed]

Oops, yep fixed

LGTM


testing

Are you experiencing an issue with...
shields.io
🐞 Description
I just want to trigger running npm run badge from GH actions :D
🔗 Link to the badge
https://img.shields.io/liberapay/gives/Changaco
💡 Possible Solution
No response

wicked cool


Timeline.setItems causes _onUpdate which checks and renders items one by one

Callstack:
Timeline.setItems
ItemSet.setItems
ItemSet._onUpdate
forEach
...
Group.add
Group._checkIfVisible
RangeItem.show
...

I think it makes sense to add a flag to prevent one-by-one updates.
Using timeline.setItems(null); timeline.setItems(null); does not help

Itemset.setItems removes all items and adds again :(
https://github.com/yotamberk/timeline-plus/blob/master/lib/timeline/component/ItemSet.js#L862-L864

TL;DR:
DataView + DataSet produces somewhat usable result (10K items in the dataset, and artificial cap of 1K displayed items in timeline), however timeline still spends noticeable amount of time in _updateItemsInRange/_traceVisible.
It looks like the proper approach would be to implement tiling. That is use multiple pre-computed datasets, so the timeline does not have to process more than 100 (or whatever) elements.

Closing the issue for now as proper resolution to Timeline.setItems is to avoid use of the method.
timeline should probably produce a warning when Timeline.setItems is called multiple times with lots of items.


Look to run local tests under travis

looks like we ought to be able to run local tests under travis based on GoogleCloudPlatform/openjdk-runtime#35

Fixed by #113


How to install jpype on pypy3?

I got the following error when I try to install JPype1 on pypy3. Does anybody know how to fix the problem? Thanks.
$ pypy3 -mpip install -U JPype1
Collecting JPype1
  Using cached JPype1-0.7.5.tar.gz (645 kB)
Installing collected packages: JPype1
    Running setup.py install for JPype1 ... error
    ERROR: Command errored out with exit status 1:
     command: /usr/local/bin/pypy3 -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/private/tmp/mktemp/pip-install-6pnf_iup/JPype1/setup.py'"'"'; __file__='"'"'/private/tmp/mktemp/pip-install-6pnf_iup/JPype1/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' install --record /private/tmp/mktemp/pip-record-594dwb7h/install-record.txt --single-version-externally-managed --compile --install-headers /usr/local/Cellar/pypy3/7.3.1_1/libexec/include/JPype1
         cwd: /private/tmp/mktemp/pip-install-6pnf_iup/JPype1/
    Complete output (61 lines):
    running install
    running build
    running build_py
    creating build
    creating build/lib.macosx-10.7-x86_64-3.6
    creating build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/_jcollection.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/_classpath.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/_jio.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/_pykeywords.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/_jproxy.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/_gui.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/_jmethod.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/nio.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/_jstring.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/__init__.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/types.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/beans.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/_jvmfinder.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/imports.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/_jcustomizer.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/_core.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/_jinit.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/_jarray.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/_jobject.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/pickle.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/_jclass.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/_jexception.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/reflect.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    copying jpype/_jpackage.py -> build/lib.macosx-10.7-x86_64-3.6/jpype
    running build_ext
    running build_java
    Using Jar cache
    creating build/lib
    creating build/lib/org
    creating build/lib/org/jpype
    creating build/lib/org/jpype/classloader
    copying native/jars/org/jpype/classloader/JPypeClassLoader.class -> build/lib/org/jpype/classloader
    copying native/jars/org.jpype.jar -> build/lib
    running build_thunk
    Building thunks
      including thunk build/lib/org/jpype/classloader/JPypeClassLoader.class
      including thunk build/lib/org.jpype.jar
    Call build extensions
    Call build ext
    building '_jpype' extension
    creating build/temp.macosx-10.7-x86_64-3.6
    creating build/temp.macosx-10.7-x86_64-3.6/build
    creating build/temp.macosx-10.7-x86_64-3.6/build/src
    creating build/temp.macosx-10.7-x86_64-3.6/native
    creating build/temp.macosx-10.7-x86_64-3.6/native/python
    creating build/temp.macosx-10.7-x86_64-3.6/native/common
    gcc -pthread -arch x86_64 -DNDEBUG -O2 -fPIC -DMACOSX=1 -Inative/common/include -Inative/python/include -Ibuild/src -Inative/jni_include -I/usr/local/Cellar/pypy3/7.3.1_1/libexec/include -c build/src/jp_thunk.cpp -o build/temp.macosx-10.7-x86_64-3.6/build/src/jp_thunk.o -g0 -std=c++11 -ggdb
    warning: include path for stdlibc++ headers not found; pass '-stdlib=libc++' on the command line to use the libc++ standard library instead [-Wstdlibcxx-not-found]
    In file included from build/src/jp_thunk.cpp:1:
    In file included from build/src/jp_thunk.h:3:
    native/common/include/jpype.h:64:10: fatal error: 'map' file not found
    #include <map>
             ^~~~~
    1 warning and 1 error generated.
    error: command 'gcc' failed with exit status 1
    ----------------------------------------
ERROR: Command errored out with exit status 1: /usr/local/bin/pypy3 -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/private/tmp/mktemp/pip-install-6pnf_iup/JPype1/setup.py'"'"'; __file__='"'"'/private/tmp/mktemp/pip-install-6pnf_iup/JPype1/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' install --record /private/tmp/mktemp/pip-record-594dwb7h/install-record.txt --single-version-externally-managed --compile --install-headers /usr/local/Cellar/pypy3/7.3.1_1/libexec/include/JPype1 Check the logs for full command output.

As far as I am aware JPype does not currently work with PyPy.  The problem that you are reporting is that it is compiling with gcc rather the g++.  Likely this has something to do with how PyPy was build.   I have successful built it by adding a few minor modifications to the platform.py file to force it to build with the correct options.  Unfortunately after that suffers a segmentation fault during the tests due to small differences between the PyPy and the CPython object model implementation.  They don't seem insurmountable, but I don't know enough about the PyPy object model resolve the differences myself.   The code for PyPy is in python and uses the JIT compiler approach to make C code.  This obfuscates source of the errors enough that I haven't been about to puzzle out where it went wrong as the logs reported just don't make much sense to me.
Assuming someone with skills is writing PyPy modules could weigh in on how to debug issues with the JIT compiler it should be possible to run.   I made a lot of modifications to correct earlier problem with the Python interaction layer that was leading to PyPy fails and it compiles just fine (with the proper flags).  Further we now have a way to link the GC so that JPype and PyPy can work together in terms of the memory space.   But we are still missing some pieces of the object model to pull it off.


Call hierarchy lacks detail

re https://github.com/microsoft/vscode/issues/89386

use ts3.8-beta
show callers of mergeSort
the callers are shown without any detail making it hard to understand/process the information

@mjbvz This is typescript that needs to provide this information

@jrieken just to clarify, you mean showing the file path for all entries?

I would expect to see the "container", e.g the class that contains a function, or the module, or the file path

Thanks for the clarification. I've opened https://github.com/microsoft/TypeScript/issues/37061 to track this request upstream


Missing loading state when fetching a story with useStoryblok hook

When fetching a story with the useStoryblok hook in React, I'm not able to tell if the request is still in progress or if it failed, because in both cases an empty object is returned.

Expected Behavior
Return an isLoading state when fetching a story with the useStoryblok hook. This would make it easier to render a loading screen, when isLoading is true and a 404 page when the request failed.
Current Behavior
At the moment the hook returns an empty object initially and when the request has failed.
Steps to Reproduce

Create a new react app and initialize with Storyblok
Add the following code to App.tsx:

function App() {
  const slug = pathname === '/' ? 'home' : pathname(replace('/', '');

  const story = useStoryblok(slug, { version: "draft" });

  if (!story?.content) return
Loading ...

  return (
    <StoryblokComponent blok={story.content} />
  )

}

And that is what I would want to achiev:
function App() {
  const slug = pathname === '/' ? 'home' : pathname(replace('/', '');

  const { story, isLoading } = useStoryblok(slug, { version: "draft" });

  if (!story?.content && isLoading) return
Loading ...

  if (!story?.content) return
404 not found

  return (
    <StoryblokComponent blok={story.content} />
  )

}

If this is also achievable with the current implementation of the hook, please enlighten me!

Hey @oliverstreissi thanks for sharing the suggestion. We will discuss the idea with the team and will come back to you

Tagging @alexjoverm for visibility

@oliverstreissi
This custom behavior is achievable with something like this:
import React, { useState, useEffect } from "react"
import { storyblokInit, apiPlugin, StoryblokComponent } from "@storyblok/react";

const { storyblokApi } = storyblokInit({
  accessToken: "YOUR_ACCESS_TOKEN",
  use: [apiPlugin],
});

function MyComponent({slug}){
    const [loading, setLoading] = useState(false);
    const [data, setData] = useState(null);
    useEffect(()=>{
        setLoading(true);
        storyblokApi.get(slug, { version: "draft" }).then((response)=>{
            setLoading(false);
            setData(response.data)
        }).catch({
            setLoading(false);
            setData(null)
        });
    }, [slug])

    if(loading){
        return
Loading

    }
    if(!!!data){
        return
404

    }
    // Depends on data returned
    return <StoryblokComponent blok={data} />

}

@fgiuliani yes with the update it's working, thanks!


Migrations: Opt out from IS NOT NULL in index creation

When creating an index in migrations it would be nice if it was possible to specify on a per column basis whether the column should be generated as NOT NULL. Right now all columns are NOT NULL if the index is not clustered.
More generally, it would be nice if provider specific details were implemented in seperate overrideable methods, so that one could implement this with a custom migrations SQL generator. For example:
protected override void Generate(CreateIndexOperation operation, IModel model, MigrationCommandListBuilder builder)
        {
            Check.NotNull(operation, nameof(operation));
            Check.NotNull(builder, nameof(builder));

            base.Generate(operation, model, builder, terminate: false);
            AppendIndexWhere(operation, model, builder);

            EndStatement(builder);
        }

protected virtual void AppendIndexWhere(CreateIndexOperation operation, IModel model, MigrationCommandListBuilder builder)
{
            var clustered = operation[SqlServerFullAnnotationNames.Instance.Clustered] as bool?;
            if (operation.IsUnique
                && (clustered != true))
            {
                builder.Append(" WHERE ");
                for (var i = 0; i < operation.Columns.Length; i++)
                {
                    if (i != 0)
                    {
                        builder.Append(" AND ");
                    }

                    builder
                        .Append(SqlGenerationHelper.DelimitIdentifier(operation.Columns[i]))
                        .Append(" IS NOT NULL");
                }
            }
}

In this way I can call base.Generate() from a derived custom SQL generator and override the provider behaviour where needed by overriding each step. This is because I cannot call base.base.Generate() from my custom SQL generator, so it is currently an all or nothing situation.

You can override AppendIndexWhere in derived class and opt out of NOT NULL condition. Is it not working for you?

@smitpatel  Exactly, the problem is that AppendIndexWhere() does not exist, but is my proposal, where provider specific stuff is separeted out from the Generate() body, so that one can opt out :-)

Oh sorry. I thought those methods were in current codebase.
The reason provider specific details are not in separate method because it is provider specific SqlGenerator class. Providers derive from base SqlGenerator and modify the SqlGenerator in the way required for the specific provider.
I am not sure if you should be deriving from SqlServerMigrationsSqlGenerator class and not the base one.

I am deriving from SqlServerMigrationsSqlGenerator, but I don't want the Generate() from that, but the one from its base MigrationsSqlGenerator. If only the Generate() functions in MigrationsSqlGenerator were public instead of protected, then I could instantiate my own MigrationsSqlGenerator and call the Generate()'s directly.

Related to https://github.com/aspnet/EntityFramework/issues/5532

Note for triage, we thought this was a dupe but we had closed the other report of this as "won't fix". However, we've seen several folks not like the redundant IS NOT NULL specification now, so we should reconsider.

We'll avoid creating the redundant checks, tracking this with https://github.com/aspnet/EntityFramework/issues/5532.


Is there an alternate for setting video orientation of the output file?

This is not an issue but a query.
public void setOrientationHint (int degrees) is the method used by the MediaRecorder API to set the orientation hint for output video playback and this should be called before preparing MediaRecorder object. I want to know if I can set Orientation of the video other than using MediaRecorder.setOrientationHint.
Thanks
Ravi

This sample has been migrated to a new location (check README for more information).
As recommended by GitHub, we are closing all issues and pull requests now that this older repo will be archived.
If you still see this issue in the updated repo, please reopen the issue/PR there. Thank you!


reporting bug issues

Looking forward for reporting a security issue:
Please report security issues by sending an email to<EMAIL_ADDRESS>instead of raising a GitHub issue.
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

You have not reported an actual issue.  I am closing this for now.  If you have any further information to provide, you can request it to be reopened.


Add missing command line flags to README

Several command line flags are not described in the README. Command line flags may overwrite environment variables, this change clarifies which flags overwrite environment variables.

I added the flags based on this code


azpainter: new package at 2.1.6

Description of package
Full color painting software for illustration drawing.

[x] I agree to maintain this package.
[x] I agree to notify @dylanaraps if I can no longer maintain this package.

Nice!


Add Random Forest to learning.ipynb

@norvig and @MrDupin, Random Forest Algorithm is not added in learning.ipynb. Can I add that in the python notebook?

Sure, that would be a great addition! Make sure to add the section close to the Decision Tree One.

@norvig, Can you please review my pr

@MrDupin please review my pr

@norvig and @MrDupin, I also find that the functions which are written for testing learners (eg : err_ratio, train_test_split, grade_learner and cross_validation) are no where explained in the python notebook. Are those needed to be added?

I will review your PR sometime tomorrow, I am a bit busy at the moment.
As for the evaluating functions, I don't think they need to be explained in the notebooks, their docstrings will do just fine.

@CharuChhimpa cross_validation isn't usable in its current state, nor are the functions that derive from it  (learning_curve for example). It won't be useful adding notebook sections for the functions that don't work. See #735 , #730 , #566 and this issue for details. train_test_split just splits a dataset into the training and testing set. I can't imagine what a notebook section for this function might look like. You can go ahead if you have some ideas, but hold back on cross_validation for now.


"... Cannot find or open the PDB file." ... Did I break something in my app?

While running locally with IIS Express, an issue has come up (possibly due to something I fouled up), where I'm seeing many, many output lines like this:

'iisexpress.exe' (CoreCLR: dnx.coreclr.managed): Loaded 'C:\Users\abcde.dnx\runtimes\dnx-coreclr-win-x<IP_ADDRESS>-beta6\bin\System.Security.Cryptography.Hashing.Algorithms.dll'. Cannot find or open the PDB file.

The app is unresponsive as the runtime or IIS Express hits msdl.microsoft.com making requests for PDB files but reaches 404's, and yields ...
The program '[14616] iisexpress.exe' has exited with code -1 (0xffffffff).

... when I stop the debugger. Any idea what I might have broken? ... or is this an issue?
MVC.Core Web API app (dnxcore50 only) on dnx-coreclr-win-x<IP_ADDRESS>-beta6 in VS 2015 RTM
  "dependencies": {
    "Microsoft.AspNet.Mvc.Core": "6.0.0-beta6",
    "Microsoft.AspNet.Server.IIS": "1.0.0-beta6",
    "System.Runtime.Serialization.Primitives": "4.0.10-beta-*",
    "Microsoft.AspNet.Mvc.Formatters.Json": "6.0.0-beta6",
    "WindowsAzure.Storage": "5.0.1-preview",
    "Microsoft.Framework.Configuration": "1.0.0-beta6",
    "xunit.runner.aspnet": "2.0.0-aspnet-beta6-*"
  },

Occurs in multiple applications. Paused the debugger and received this ...


Add command to open NuVoc in web browser

Thanks for contributing this plugin!
I use the default F1 binding in JQT to open NuVoc quite frequently. This provides the command, but I didn't add a default key binding.
I also chose the wording "Open NuVoc Help in Browser" to ensure that typing j and help would match this entry for users looking for J help via the VS Code command palette.

@semperos Sounds good.
Will take a look and hopefully get it included this weekend.

I've implemented your suggestion in Typescript (canonical development).
Closing in favour of #14


Better file matching?

I would expect any one of the globs /home/dev/devenv/dappsys/src/sol/token/old_asset/*, /home/dev/devenv/dappsys/src/sol/token/old_asset/*.*, and /home/dev/devenv/dappsys/src/sol/token/old_asset/*.sol to match the path /home/dev/devenv/dappsys/src/sol/token/old_asset/asset0.sol, but it appears that none of them presently do.
Edit: I'm assuming this is a bug, not a feature. Please correct me if I'm wrong here.

What is your base directory?

Ah, okay. That appears to have been the piece I was missing. I'd been operating under the impression it matched files based on file.path. Subtracting file.base from my globs fixed it.


What is the equivalent flags for Google Authenticator yes to all?

Hi @ThomasHabets ; I want to create a large number of VPN accounts so I wrote a small tool to automate this.
 This restricts you to one login about every 30s, but it increases
your chances to notice or even prevent man-in-the-middle attacks (y/n) y
By default, tokens are good for 30 seconds and in order to compensate for
possible time-skew between the client and the server, we allow an extra
token before and after the current time. If you experience problems with poor
time synchronization, you can increase the window from its default
size of 1:30min to about 4min. Do you want to do so (y/n) y
If the computer that you are logging into isn't hardened against brute-force
login attempts, you can enable rate-limiting for the authentication module.
By default, this limits attackers to no more than 3 login attempts every 30s.
Do you want to enable rate-limiting (y/n) y

I want to use the flags to redirect the output to a file; but I can't find a right combination.
Mine is like
google-authenticator -t -d -f -r 3 -R 30 -W > ~/test.txt
and not work :(
Please help me.

Oh, you need the QR code? Yeah that's only rendered if stdout is a TTY.
Try this:
script -c "google-authenticator -s blah.secret -t -d -f -r 3 -R 30 -W" test.txt

Hi; I will try tomorrow and feedback later(I dont have remote access). Thanks for the quick reply. BTW, can you explain a bit about your command? I see the flags is quite the same so I dont want you misunderstand my question?

script is a tool that creates a fake tty that saves a copy of everything sent to that terminal to a file.

That's a new thing for me but I think you misunderstand my question.
Here's how I create QR code for my company:

google-authenticator
Question 1: y
...
Question n: y (yes to all)

I want to have an one line command that have the same meaning with the above process.

I'm not following. You have a command that doesn't ask you any follow-up questions.
If you just want to simulate answering "y", then how about:  yes y | google-authenticator ?

Look very promising :+1:
I will feedback tomorrow. Thanks for the kind support.


Suggestion: Add .gitattributes ignore rules to getting started content

I've seen several folks recently mention/complain that the generated code makes code review more difficult.
I'd suggest we include a suggestion to exclude the generated parts of the ent subdirectory as described in https://docs.github.com/en/github/administering-a-repository/managing-repository-settings/customizing-how-changed-files-appear-on-github

Actually, as the ent-generated files use the typical generated header this probably isn't actually accomplishing anything. Perhaps we should close.

Hi, how do we exclude (auto collapse) generated files by using .gitattributes?
I don't know how to exclude all folders except ent/schema :smile:

Can we add a custom postfix/prefix for generated files? So that the config for ignoring will be more simpler? Moreover, it will help us omit generated files when searching text inside the project.


feat: translate legal text

Can be merged if you add at least a single sentence in the description of the Pr to explain why this change is necessary.


Foundation 6.4.1: Changing $grid-column-count value in _settings.scss doesn't work?

Hello Zurb-Team,
we're using Foundation 6.4.1 with the new xy-grid. We need a 16 columns grid and wanted to change the value of the variable $grid-column-count in the file settings.scss - like it worked like a charm in the past. When we built our markup and we wanted e.g. to create a 2-col-layout for medium up, we still have to use "medium-6" for both cells - even if the value of $grid-column-count has already been changed to 16. Is this a known issue (I couldn't find anything in all Github Issues) or are we doing something wrong?
Thank you very much and kind regards

@ginocremer For XY grid you need to change $grid-columns found here.
$grid-column-count is for the old 2 grids. Hope that helps!

You're right, thank you very much for your help and your fast answer! Perhaps it could be helpful for others to write a short comment behind like "// Legacy - Use $grid-columns for XY instead" - I just read "3. The Grid" and for me the grid is...the grid :D

You might be referring to a settings file that you had previously - carried over form 6.3.1. This can be bettered handles through migration notes since a news settings file wouldn't have the old settings.
We'll be adding this specificity for the next major release.


Fix typo

Thanks for @polka/compression, looks great!
Quick PR to fix the typo in the type

@lukeed Thanks for the review and merge!

Thanks!


Add kubelet log config DaemonSet to tools

@SergeyKanzhelev Can you help review? Thanks!
Verified the PR manually:

Apply node selector label to worker nodes
SSH on to the worker node, check the kubelet config does not have log config
Apply the daemonset
SSH on to the worker node, check the kubelet config has the log config, and check the kubelet has just restarted from kubelet log

Accidentally squashed the changes I had. So now there is no diff. Hope it is not making it too hard to review...


Can't find kubernetes plugin after startup the docker

After startup the docker, can not find kubernetes plugin in Manage Plugin page & search/filter.

sorry, it's already included.


Some fixes for bugs I encountered when trying to use this plugin.

See commit for details.
If you don't agree with the default stemmer option I added I can make a new PR without it.

Coverage decreased (-0.1%) to 75.783% when pulling 363e6ff5f8c052b14fd6d68276a1abe4186abbbc on Cyriuz:master into 31e25f4354792ed4cad5617024260fe9d87be4c0 on easyops-cn:master.

Hey @Cyriuz Thanks for your work, 👍
Could you follow the Conventional Commits, so we can let our release bot to generate change log automatically, like: feat: add an option to ... fix: .... Thanks.

And next time, we would better to make different PRs for non-relevant features/fixes. Thus we can launch them independently. Like here we seem to get stuck with one fix, while other fixes seem to be good, but they can't be merged.

Yeah sure, I can fixup the commit messages, thanks for the link! And yeah you're right, I was a bit lazy with this one, I'll split them in the future :)

Thanks, released in v0.24.0. By the way I just manually created a commit to mark these release notes.

There is a quick fix for indexing content between h1 and h2, and been released in v0.24.0.


can't run on mac os

Tisp:Error: Electron failed to install correctly, please delete node_modules/electron and try installing again.
I haved deleted and installed again and again,but did't work.
how bad it is!

Thanks for reaching out!
Because we treat our issues list as the team's backlog, we close issues that are questions since they don't represent a task needing to be completed. This forum is for raising bugs or requesting new features, and for most questions about Electron there are a lot of more appropriate community options. There are helpful people in this Discord or the subreddit that should be willing to point you in the right direction.


FenceValueCPUVirtualAddress(PagingQueue)/HwQueueProgressFenceCPUVirtualAddress/FenceValueGPUVirtualAddress(D3DKMT_OPENSYNCOBJECTFROMNTHANDLE2) should be mapped read-only

Windows Version
Microsoft Windows [Version 10.0.19044.3570]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu 22.04
Other Software
No response
Repro Steps
#include <wsl/winadapter.h>
#include <d3dkmthk.h>

#include <unistd.h>
#include <linux/unistd.h>
#include <sys/mman.h>
#include <sys/eventfd.h>

#include <cstdio>

D3DKMT_HANDLE                   gAdapter;
D3DKMT_HANDLE                   gDevice;
D3DKMT_HANDLE                   gPagingQueue;
D3DKMT_HANDLE                   gPagingFence;
uint64_t                                    gPagingFenceAddr;

int main (int argc, char *argv[]) {
        D3DKMT_ADAPTERINFO adapter = { };

	D3DKMT_ENUMADAPTERS2 enum_adapter = {
		.NumAdapters = 1,
		.pAdapters = &adapter,
	};

	NTSTATUS ret = D3DKMTEnumAdapters2(&enum_adapter);
	if (ret != STATUS_SUCCESS) {
		printf("D3DKMTEnumAdapters2 failed, ret = %08x\n", ret);
		return -1;
	}

	D3DKMT_OPENADAPTERFROMLUID open_adapter = {
		.AdapterLuid = adapter.AdapterLuid,
	};

	ret = D3DKMTOpenAdapterFromLuid(&open_adapter);
	if (ret != STATUS_SUCCESS) {
		printf("D3DKMTOpenAdapterFromLuid failed, ret = %08x\n", ret);
		return -1;
	}
	gAdapter = open_adapter.hAdapter;

	D3DKMT_CREATEDEVICE create_device = {
		.hAdapter = gAdapter,
	};

	ret = D3DKMTCreateDevice(&create_device);
	if (ret != STATUS_SUCCESS) {
		printf("D3DKMTCreateDevice failed, ret = %08x\n", ret);
		return -1;
	}
	gDevice = create_device.hDevice;

        D3DKMT_CREATEPAGINGQUEUE create_page_queue = { };
	create_page_queue.hDevice = gDevice;
	create_page_queue.Priority = D3DDDI_PAGINGQUEUE_PRIORITY_NORMAL;

	ret = D3DKMTCreatePagingQueue(&create_page_queue);
	if (ret != STATUS_SUCCESS) {
		printf("D3DKMTCreatePagingQueue failed, ret = %08x\n", ret);
		return -1;
	}

	gPagingQueue = create_page_queue.hPagingQueue;
	gPagingFence = create_page_queue.hSyncObject;
	gPagingFenceAddr = (uint64_t)create_page_queue.FenceValueCPUVirtualAddress;

        *(uint64_t *)gPagingFenceAddr  = 0;

        return 0;
}

Expected Behavior
*(uint64_t *)gPagingFenceAddr  = 0; should trigger Segmentation fault (core dumped).
Per d3dkmthk.h.
https://github.com/microsoft/libdxg/blob/main/include/d3dkmthk.h#L4788
D3DKMT_PTR(VOID*,           FenceValueCPUVirtualAddress);   // out: Read-only mapping of the fence value for the CPU
https://github.com/microsoft/libdxg/blob/main/include/d3dkmthk.h#L5138
D3DKMT_PTR(VOID*,           HwQueueProgressFenceCPUVirtualAddress); // out: Read-only mapping of the queue progress fence value for the CPU
https://github.com/microsoft/libdxg/blob/main/include/d3dkmthk.h#L1528
D3DKMT_PTR(VOID*,       FenceValueCPUVirtualAddress);           // out: Read-only mapping of the fence value for the CPU
Actual Behavior
*(uint64_t *)gPagingFenceAddr  = 0; succeed.
Diagnostic Logs
No response

All of them are mapped read/write in dxgkrnl linux driver,
https://github.com/microsoft/WSL2-Linux-Kernel/blob/linux-msft-wsl-5.15.y/drivers/hv/dxgkrnl/dxgvmbus.c#L1319
https://github.com/microsoft/WSL2-Linux-Kernel/blob/linux-msft-wsl-5.15.y/drivers/hv/dxgkrnl/dxgvmbus.c#L3716
https://github.com/microsoft/WSL2-Linux-Kernel/blob/linux-msft-wsl-5.15.y/drivers/hv/dxgkrnl/dxgvmbus.c#L793


Would it be a good idea to bring zlib-ng to the attention of the Linux Kernel folks?

A recent GCC bug lead to an effort to modernize the zlib library used in the Linux Kernel (see: http://lkml.iu.edu/hypermail/linux/kernel/2105.0/01493.html). I am not subscribed to the list and my name doesn't carry any weight anyway, so maybe one of the interested parties here (Google? ARM? IBM?) could pitch the idea of using zlib-ng as a drop-in replacement and enjoy some of the benefits which come with it?
Of course only if it would make sense.

Thanks for letting us know, I will try and reach out.

Linus responded regarding the zlib changes and there are some specific modifications made to their zlib implementation that are necessary for the kernel, namely no dynamic allocations using malloc.

The problem with the kernel side of zlib is that we can't just act as if it's just another regular library: the kernel tends to have some restrictions on what we can do anyway, but when it comes to zlib (and some other code), those restrictions are even worse than usual because it's used fairly early during boot for uncompressing early data.

Anybody can view their zlib implementation that is in the kernel and it might benefit us to take a look at their changes:
https://github.com/torvalds/linux/tree/master/lib/zlib_deflate
https://github.com/torvalds/linux/tree/master/lib/zlib_inflate


Error in compiling: process didn't exit successfully: rustc -vV (signal: 7, SIGBUS: access to undefined memory)

Previously I have compiled and deployed my code. Now I am having this issue after formatting my computer from Window 11 Pro to Window 11 Home.
When I run anchor test or anchor build, it shows the error below:
error: process didn't exit successfully: rustc -vV (signal: 7, SIGBUS: access to undefined memory)
Anyone face this issue and able to give a method to fix it? I am using Window 11 Pro and WSL2 to compile my code.

I fix this by reinstalling WSL2 as well as my Ubuntu, then all packages required.
Found (https://github.com/rust-lang/cargo/issues/10359). It happened because during release of solana-bpf I closed it without having a proper installation.
Suggest using rm -rf ~/.cache/solana/v1.21/ works but I can't find the file version of solana cache in Ubuntu. I am not expertise in Ubuntu command so I reinstall all my WSL2 and setup again my WSL2.
Fix.


[Bug] A way for custom enemies to be stomped by boxes but not players

As of the moment "killedbyboxes": true, "stompable": false just means any boxes dropped on the enemy will bounce up and down, making the stomp noise but not actually killing the enemy.
I'd like for a way to configure a CE such that when a player steps on it, the player just uses the enemy as ground with the player nor the enemy being harmed in the process, but when a cube steps on it, the enemy is stomped and killed. I don't believe that's currently possible.

I'm treating this as a bug because it doesn't make sense that one property depends on the other. They should have been independent all along.

Fixed in #63


Update intro.md

The types section of the intro had "bit rotted".

I'm not sure it is worth having this as "Most importantly"?

LGTM


Proposed: Extension classes with Broadcast Channels

Stacked on #35
This PR takes some opinionated steps forward:

it defines a few of the peer message types required in the system (still remaining things like cosign helpers and maybe others) (#7)
it suggests a split of the base class into separate APIs for wallet owners and wallet intermediaries - the pay method of the OwnerClient is implemented (#6)
it sets up two BroadcastChannels for each channel - one for communication between the channel participants, and one for the receipt of messages from outside of the channel. (#18)

Follow-on work to get to an end-to-end payment running is

to make IntermediaryClient which can receive, inspect, and decide whether to honor a ForwardPayment message (should reject if the sender does not actually have funds to cover the added HTLC)
to make the OwnerClient responsive to RequestInvoice messages

PR is lacking in test coverage - hence draft.

@NiloCK I'm on board with this plan.


Does not work on Vivaldi

Vivaldi is a Chrome browser. It installs and appears to work. I granted the permission but, it doesn't actually enable it.

Disregard


CUDNN Dilation

What is the motivations for having a custom dilation function rather than using pytorchs inbuilt dilation?

I am wondering if you have any data/intuition on whether using Pytorchs inbuilt dilated convs would speed up training time a lot? I am noticing that the network is immensely slow to train even with just 1-2M parameters...


Script for dataset compilation

Thanks for sharing your training code. Could you kindly share the script for compiling the dataset. I want to reproduce your work and want to ensure everything is the same.

Hi, there is an error in the model, as pointed out in this reply. Nevertheless, the data compile code is uploaded.


bind not finding zones.override and does not start

With the latest update of bind, it does not start anymore.
ab77/bind:latest           "/bin/sh -c ./run.sh"    6 hours ago         Restarting (1) 20 seconds ago
The output of
sudo docker logs -f bind
`drwxr-sr-x   2 root bind 4096 Dec  5 16:18 .
drwxr-xr-x 121 root root 4096 Dec  5 16:20 ..
-rw-r--r--   1 root root 3954 Nov  8 13:00 bind.keys
-rw-r--r--   1 root root  237 Nov  8 13:00 db.0
-rw-r--r--   1 root root  271 Nov  8 13:00 db.127
-rw-r--r--   1 root root  237 Nov  8 13:00 db.255
-rw-r--r--   1 root root  353 Nov  8 13:00 db.empty
-rw-r--r--   1 root root  270 Nov  8 13:00 db.local
-rw-r--r--   1 root root 3171 Nov  8 13:00 db.root
-rw-r--r--   1 root bind  463 Nov  8 13:00 named.conf
-rw-r--r--   1 root bind  490 Nov  8 13:00 named.conf.default-zones
-rw-r--r--   1 root root  268 Dec  5 16:15 named.conf.local
-rw-r--r--   1 root root 2431 Dec  5 16:15 named.conf.logging
-rw-r--r--   1 root root 1214 Dec  5 16:15 named.conf.options
-rw-r--r--   1 root root  130 Dec  5 16:15 named.recursion.conf
-rw-r-----   1 bind bind   77 Dec  5 16:17 rndc.key
-rw-r--r--   1 root root 1317 Nov  8 13:00 zones.rfc1918
/etc/bind/named.conf.local:14: open: /etc/bind/zones.override: file not found`
It used to be working before and zones.override is still in
/opt/netflix-proxy/data/conf
How can I configure bind to use the zones.override file again?
Thanks!

I agree with you
Something is not right with this update, I did a new install and edited bind files but it does not work

The file(s) are now located in ~/netflix-proxy/docker-bind/.

/root/netflix-proxy
root@ubuntu-512mb-nyc3-01:~/netflix-proxy# head docker-bind/zones.override
zone "netflix.com." {
    type master;
    file "/etc/bind/db.override";
};

zone "netflix.net." {
    type master;
    file "/etc/bind/db.override";
};

root@ubuntu-512mb-nyc3-01:~/netflix-proxy# docker ps -a
CONTAINER ID        IMAGE               COMMAND                  CREATED              STATUS              PORTS               NAMES
e114d19e6134        andyshinn/dnsmasq   "dnsmasq -k --log-fa…"   About a minute ago   Up 54 seconds                           caching-resolver
ddf84762b3a6        abiosoft/caddy      "/usr/bin/caddy --co…"   About a minute ago   Up 54 seconds                           caddy
25c0a889b6c9        ab77/bind           "/bin/sh -c ./run.sh"    About a minute ago   Up 54 seconds                           bind
0c79112a486a        ab77/sniproxy       "/bin/sh -c ./run.sh"    About a minute ago   Up 54 seconds                           sniproxy
b81695d2d69c        andyshinn/dnsmasq   "dnsmasq -k --port=5…"   About a minute ago   Up 54 seconds                           dnsmasq
root@ubuntu-512mb-nyc3-01:~/netflix-proxy#```

In addition, bind is being replaced with dnsmasq. So all the domain config will shortly be defined in dnsmasq.conf.

I sure know that it is now ~/netflix-proxy/docker-bind/ and I pointed it at my other SNIProxies (for Netflix and BBC) and it wasn't working

OK, I had to remove docker and do a complete reinstall. This solved it.
However, now it seems that you just moved to a new setup as you mentioned a couple of hours ago (dnsmasq beeing now the only docker containers). How do I configure zones.override now?
Thanks!

The project has now moved to dnsmasq to provide DNS services. BIND was a complete overkill and highly inflexible for this specific purpose.
You need to edit dnsmasq.conf followed by docker restart dnsmasq.

Thank you! However, I cannot seem to find a dnsmasq.conf in the netflix-proxy directory. Where could I find it?

Am 06.12.2017 um 14:30 schrieb Anton Belodedenko<EMAIL_ADDRESS>The project has now moved to dnsmasq to provide DNS services. BIND was a complete overkill and highly inflexible for this specific purpose.
You need to edit dnsmasq.conf followed by docker restart dnsmasq.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/ab77/netflix-proxy/issues/281#issuecomment-349640113, or mute the thread https://github.com/notifications/unsubscribe-auth/AFeqSMV_ADd_jJ-ivJokKBngrrzqgtRbks5s9pb5gaJpZM4Q3Cgv.

Try re-building on a clean image and check ~/netflix-proxy

Thanks! It’s working fine again!
Just one more question: I submitted a number of DynDNS names and the proxy resolved their IP-Adresses. However, when I tried to connect from one of these IP-Adresses, I got the admin screen.
Is there a problem with DynDNS?
Thanks,
Roland

Am 06.12.2017 um 14:40 schrieb Anton Belodedenko<EMAIL_ADDRESS>Try re-building on a clean image and check ~/netflix-proxy
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/ab77/netflix-proxy/issues/281#issuecomment-349642472, or mute the thread https://github.com/notifications/unsubscribe-auth/AFeqSH0Mq1nCpjEYLA9MNACsgdcXkDmtks5s9plTgaJpZM4Q3Cgv.

Not sure, check iptables, etc.

Please create a new issue descibing the problem in detail @drallgood so it can be tracked separately.
It is possible something in the new build affected DDNS, but I've not tested it.


new branch

Hi,
can a maintainer please create a new branch, ubuntu20.04, so I can create a PR against this branch ?
Or shall I create a PR against master ?
Thanks
Colin

Creating the PR against master branch would be great. Thanks.


Wrong type for CL_DEVICE_GLOBAL_MEM_CACHELINE_SIZE

device::get_info<CL_DEVICE_GLOBAL_MEM_CACHELINE_SIZE> is specialized for cl_ulong instead of for cl_uint

Fixed in #826


🪲[Techdebt] polish layout fees

What?
There was a minor bug where the ops address would show up  instead of the sent address. Also the fees were off by a factor

Cool! Thank you! What is ops address? What is the difference between ops address and send address?

The ops address is the address users are sending the service fee.  It is the way we are funding the MWEB, redesign and staff grants

@josikie  TYVM