Can't make Multi Row (& Nested) Layout functionnal

Hi !
I am working with Ionic and it's been a week that I found out Swiper and this library and it was a relief since they are exactly what I need ! But my problem is that I can't find out how to make things work :disappointed:
Swiper and ngx-swiper-wrapper are working well since I can see the rendering of some simple layouts, but when I want to make a multi-row layout, it just becomes crazy (my goal is to mix multi-row with nested layouts)
Right now I have put my code aside just to try with really simple code, here it is:
My .ts file :
import { Component } from '@angular/core';
import { SwiperConfigInterface } from 'ngx-swiper-wrapper';

@Component({
  selector: 'swiper',
  templateUrl: 'swiper.component.html'
})
export class SwiperPage {
  constructor() {  }

  public configTestColumn: SwiperConfigInterface = {
    slidesPerView: 3,
    slidesPerColumn: 2,
    spaceBetween: 30,

    pagination: '.swiper-pagination',
    paginationClickable: true,
    nextButton: '.swiper-button-next',
    prevButton: '.swiper-button-prev'
  };
}

My .html file :
<ion-header>
    <ion-navbar>
        <ion-title>
            Ionic Blank
        </ion-title>
    </ion-navbar>
</ion-header>
<ion-content padding>

Slide 1

Slide 2

Slide 3

Slide 4

Slide 5

Slide 6

Slide 7

Slide 8

Slide 9

Slide 10

</ion-content>

Result of this code :

I have also added those line in my index.html file because I saw on the Swiper website that it was required http://idangero.us/swiper/get-started/#.WeTbOXW0OV4 :
  <link href="../node_modules/swiper/dist/css/swiper.min.css" rel="stylesheet">

If anyone could help me I would be so grateful, thank you in advance. I would give you more details if needed !
Thanks,
Cyrille

I'll have a quick look later today or tomorrow that if there is some problem with the library styles etc.

Sorry that it took this long, but I quickly tried this setup and seems that you need to define the height for the swiper-slide via css. At least I did not see Swiper adjusting it automatically and its styles have default height as 100% hence the slide 1 is pushing the slide 2 out of the view.

Thank you for your answer !
I tried to explore the height of swiper-slide and swiper-container and I obtained those weird results about the position of the slides (I added a background color for clarity) :
Settings : 2 slides per view & 2 slides per column & direction horizontal
swiper-container : no height
swiper-slide : no height

swiper-container : height = auto
swiper-slide : no height

swiper-container : height = auto
swiper-slide : height = 0px
==> nothing displayed

swiper-container : height = auto
swiper-slide : height = 100px

swiper-container : height = auto
swiper-slide : height = auto
(this is the same rendering as container height auto & slide no height)

swiper-container : no height
swiper-slide : height = 0px

swiper-container : no height
swiper-slide : height = 10px

I'm okay with the height of the slides, but about the positionning I don't understand what is going on (why do we have this order, for instance slide 2 is not right after slide 1 horizontally neither vertically)

What version are you using of this library? Since that looks odd and it should not be this library which causes that. So I am guessing that it might be a bug in Swiper 3.x. The latest @next (5.0.0-5) release usese the new Swiper 4.x so it might be worth the shot.

I'm using version 4.6.6 of this library (the latest i got from npm).
But concerning Swiper I have version 3.4.2
I tried to install Swiper 4.x but the dependency is not what I expected :

Do you know when will this new release be available ? As you said, it might be worth the shot !

You can install the next release which uses Swiper 4.x by installing ngx-swiper-wrapper@next.

Hi again !
I've been working on something else these days so I just tried that new version this previous couple of days only. The result is that it still doesn't work unfortunately. But the good news is that I have been able to identify some issues about the behaviour of the slides, being clues about the part of the code that may be affected. My tests will complete a very long message though, since I have almost 20 screenshots :s
Here are my tests :

First of all, every bugs that I found involved slidesPerColumn option
1st bug : Last column mentioned by slidesPerView has a display issue

When: It appears when slidesPerView > 1 and slidesPerColumn > 1 and spaceBetween > 0
Why: I think this problem comes from spaceBetween option because it takes some width that is initially required for the widths of the slides, so the slide of the last column at the screen has no more space to display, thus it goes at the next line
Screenshots: Here are some screenshots to describe it better :

with spaceBetween = 0

with spaceBetween = 1

And it's the same result when I set slidesPerColumnFill to 'row' :

More: I don't know if this is done on purpose but the only way I see it usable is by mentioning an extra slide per view (let’s say you want slidesPerView = 3, you’ll then mention slidesPerView = 4), and using that extra width for spaceBetween and padding left and right option to adjust the display of your slides

2nd bug : slidesPerColumn "almost inactive when used with slidesPerColumnFill = 'row'

Why: My guess is that it has to deal with the code probably ?
Screenshots:

slidesPerColumnFill: 'row' & slidesPerColumn: 2
(In the following I increase the number of slides to see the difference)

slidesPerColumnFill: 'row' & slidesPerColumn: 20

slidesPerColumnFill: 'row' & slidesPerColumn: 56

slidesPerColumnFill: 'COLUMN' & slidesPerColumn: 56

slidesPerColumnFill: 'COLUMN' & slidesPerColumn: 20

slidesPerColumnFill: 'COLUMN' & slidesPerColumn: 19
(do not give attention to the specific position of each slide separately, only the global disposition is the matter here, I’ll come back later about the specific position of each slide)

slidesPerColumnFill: 'COLUMN' & slidesPerColumn: 2

slidesPerColumnFill: 'COLUMN' & slidesPerColumn: 1

More:
Clearly there is no difference to set the slidesPerColumn option to 2, 20, or 56 when the slidesPerColumnFill option is set to ‘row’, it’s like there are only 2 options possible :
==> first option triggered when slidesPerColumn equals 1 (or 0, both give the same result), resulting in a single row,
==> second option triggered when slidesPerColumn  > 1 (in the examples I gave: 2, 20, 56)
In the opposite, we can see the difference when the slidesPerColumnFill option is set to ‘column’, even though this case has also a problem :
==> If slidesPerColumn >= number of slides : then it behaves the way it has to do
==> If 2 <= slidesPerColumn < number of slides : then all slides are displayed into two columns, with a number of slides per column different from the parameter slidesPerColumn
==> If slidesPerColumn = 1 (or slidesPerColumn = 0) : then it behaves as it should, displaying slides on a single row (giving the possibility to actually slide through the slides)

3rd bug : slidesPerColumn working strangely when used with slidesPerColumnFill = ‘column’, slides that should be “displayed" outside the screen are actually being displayed on the screen, below their previous slides, and thus, not following the slidesPerColumn: number parameter

Why: My guess is that there may also be some mistakes in the code ?

Screenshots:

More: As you can notice, when a slide should be placed in a new column, it is actually appearing somewhere else, but that “somewhere else” isn’t random. If you pick Slide 5, it is supposed to be placed at the right of the Slide 3, but instead it is placed juste below Slide 1. But maybe this presents no contradiction in the code depending of the structure of the swiper-container, leading to this question : is it an one-dimensional array or a two-dimensional array ? If it is an one-dimensional array, then the position below Slide 1 corresponds to the next position of Slide 3.

I hope I have been clear on those points.
Actually my whole trouble comes from MultiRow Slider, with the parameter slidesPerColumn
Don't hesitate to tell me if I'm wrong anywhere !

So you are now using the version 5 of this library i.e. Swiper 4.x? It definitely looks like a bug and I am 99% sure that its not due to this wrapper so maybe you could make this report in Swiper issues. At least I don't have a clue that how Angular or this library could be effecting.

Thank you for your very fast reply !
Yep I'm using those version

I'm going to follow your advice and report this in Swiper issues, I'll let the "tag" here or the link to the new issue I will create (I don't know very well how this works, it's like the first time I'm posting an issue) and will close this issue then !

Yes please do link it here so in the case that Swiper developer can't find a bug or can give some insights what could be causing the problem. And you can leave this issue open until resolved in either Swiper's end or here.

Thank you very much for your help, let's do that this way then !

spaceBetween bug : https://github.com/nolimits4web/Swiper/issues/2330
slidesPerColumn + slidesPerColumnFill = 'row' : https://github.com/nolimits4web/Swiper/issues/2331
slidesPerColumn + slidesPerColumnFill = 'column' : https://github.com/nolimits4web/Swiper/issues/2332


create Dockerfile

Status
Ready/In Progress/In Hold (Reason for hold)
Related Issues
related: link to the issue
Description
A few sentences describing the overall goals of the pull request's commits.

Docker Image Ready - Dev
Docker automatic build at CircleCI has deployed your docker image: devdemisto/archer:<IP_ADDRESS>11
It is available now on docker hub at: https://hub.docker.com/r/devdemisto/archer/tags
Get started by pulling the image:
docker pull devdemisto/archer:<IP_ADDRESS>11

Docker Metadata

Image Size: 31.63 MB

Image ID: sha256:2a1b9f5c40f7067e8436aeeeb3d54f284493d6da18beca057de861c793553978

Created: 2020-08-10T09:41:54.3149786Z

Arch: linux/amd64

Command: ["python3"]

Environment:

PATH=/usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
LANG=C.UTF-8
GPG_KEY=E3FF2839C048B25C084DEBE9B26995E310250568
PYTHON_VERSION=3.8.3
PYTHON_PIP_VERSION=20.1.1
PYTHON_GET_PIP_URL=https://github.com/pypa/get-pip/raw/eff16c878c7fd6b688b9b4c4267695cf1a0bf01b/get-pip.py
PYTHON_GET_PIP_SHA256=b3153ec0cf7b7bbf9556932aa37e4981c35dc2a2c501d70d91d2795aa532be79
DOCKER_IMAGE=devdemisto/archer:<IP_ADDRESS>11

Labels:

org.opencontainers.image.authors:Demisto<EMAIL_ADDRESS>org.opencontainers.image.revision:62c846cb8e77d908da4444eeacd4dff5e5a03ec8
org.opencontainers.image.version:<IP_ADDRESS>11


Fix/auth: try all corresponding secrets until one works

closes #237

:tada: This PR is included in version 1.6.0-beta.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Source issue

Description
Fix parsing of source, by taking into account additional syntax.
Bug:
Creates an error in TS. Does not affect current highlighting, but if someone wants to use the source argument for highlighting it wouldn't be found when the error is triggered.
Details:
source
Allow quoted & interpolated strings as well as pipes in parenthesis as arguments to source. Also updates tests for source for the new grammar.

Creates an error in TS.

what does that mean?

Creates an error in TS.

what does that mean?

Creates an ERROR query in tree-sitter. This means there is undefined behaviour to the rules & TS just gaves up.
The problem essentially is that it doesn't parse the whole source something structure correctly. What happens is: it parses the source as a keyword & highlight it as such, it parses the something as whatever it is & highlight it as such. However it doesn't actually create the whole source structure, so if someone wants to create specific highlighting that is specific to the source file group they won't be able to.
This fixes this by adding a more syntax structures to the source rules.


Add note about libicu-dev to build instructions

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

Where's pull request? :P


Anoncreds - support for anoncreds and askar wallets concurrently

Support starting aca-py with both the credx and anoncreds libraries loaded, and support (in multitenancy) tenants with both askar and askar-anoncreds wallet types concurrently.  (This is necessary to allow tenants in a multi- environment to upgrade their sub-wallets from askar to askar-anoncreds at their convenience.  (Note that the upgrade function will be documented in a separate issue.)
When starting up aca-py, load both the credx and anoncreds libraries, and enable the API endpoints for both libraries (schema, credential definition and revocation).  Ensure that the endpoints are under different url's.  (Schema and cred def are currently under different url's - the "new" API's are under /anoncreds.  For revocation, the new endpoint will need to be either (a) moved to the /anoncreds url, with the schema and cred def, or (b) all new endpoints will need to be moved under some kind of "version" based url scheme.)  NOTE this needs to be confirmed.
In each API endpoint that is duplicated, add a check to each endpoint to make sure it is being called with the correct wallet type (for example for the new anoncreds-related API's, if the wallet type is askar then return an error code.)
When in multi-tenancy mode, create new tenant wallets using the overall aca-py's configured wallet type.  (For example if aca-py is started with --wallet-type askar then created tenant wallets with askar type, likewist with askar-anoncreds.)  NOTE this needs to be confirmed.
When running in multi-tenant mode, ensure that the tenant wallet's wallet type is in scope for all processing related to that tenant (i.e. Admin API calls and processing for received agent-to-agent messages).

My vote is for a versioned API approach.  IMO this will enable the API to remain consistent from a endpoint path perspective between versions where appropriate (such as with the schema, cred-def, and revocation endpoints) which will in-turn make migration from one API version to another easier for developers; consistent APIs with different parameters/functionality.
I think this will also enable:

Setting a default API, in the case an API version is not explicitly defined by the caller.  The default can be moved up as support for deprecated APIs is removed.
Making it easier to introduce new versions of existing APIs (aka introduce breaking changes), and deprecate unsupported API versions.
Easily eliminating deprecated API versions from subsequent releases.
The ability to provide deprecation warnings and migration recommendations to those calling deprecated APIs in a consistent manner.

When in multi-tenancy mode, create new tenant wallets using the overall aca-py's configured wallet type. (For example if aca-py is started with --wallet-type askar then created tenant wallets with askar type, likewist with askar-anoncreds.) NOTE this needs to be confirmed.

Agreed.  This is a logical default for creating tenant wallets.

@WadeBarnes what are the rules you are suggesting for the versions changing?

When any new endpoint is added, or when any endpoint changes, all update to v2?
Every endpoint is versioned separately?

When any new endpoint is added, or when any endpoint changes, all update to v2?
New APIs can be introduced into the current API version since they don't impact existing API endpoints.

Every endpoint is versioned separately?
They are versioned together as a set.

Details:

Only (breaking) changes to an existing endpoint trigger a new API version.  i.e. Changes to the APIs that impact existing implementations or existing functionality, or the removal of an endpoint.  Such as how the migration to askar-anoncreds impacts the Schema, Cred-Def, and Revocation functionality and endpoints.
New APIs (new functionality) can safely be added to an existing, the current, API version, since there are no changes to existing APIs, therefore existing implementations are not impacted.  So, there is no reason to complicate things by introducing a new API version.
When a new API version is introduced, it exposes the complete API.  Existing APIs that have not been changed, along with the ones that have.  This allows the previous API version to be marked as deprecated and eventually removed.  This also has a bit of a side affect for those using the default API version.  Provided the endpoints they are using are not impacted by breaking changes or removed things just continue to work.

Thanks, Wade.  That makes sense.
The downside of that approach is that when removing an API, everyone has to deploy a new controller, even if the endpoints they use have not changed. That will be worse when the motivation to bump the API version is required, but isolated to a small number of endpoints.  For example, in the current case, no verifiers are affected by the schema/creddef/revreg, but in order to get rid of the “old code” related to those endpoints, every controller (including verifiers) will have to update their code to change to the “/v1” endpoints.  Or we start being lienent on the rules (which is what I worry about) and we keep some “unversioned” endpoints (unchanged ones) and delete others.  That makes fuzzy the definition of what is the “vX API”.
Obviously there is no right/wrong answer. So I’m not hard fast against, but my experience points to the “rename endpoints” and avoiding the “API vX” definition as an easier approach in the long run.
A couple of other reasons that I think make the ACA-Py scenario a little different:

We only publish ACA-Py, and it is deployed by many others. We can’t (for example) monitor the use of specific endpoints as we could in centralized system to know when to, for example, remove old endpoints.
plugins that add Admin API endpoints add to the fun.  Should they be versioned as well?  Presumably, they can’t be via the same numbering scheme.

I don't think removing an API necessarily requires people to deploy a new controller.  That's where the default version comes in.  If the controller does not specify a version or the version the controller using does not exist any more, I see the call using the default API version.  Properly versioned APIs don't impose the use of the /v* in the path.  You can specify the API version in headers; OrgBook has this feature.

Ah…I was not aware of the API version in the header.  Could you point me to how that works?

https://github.com/bcgov/aries-vcr/blob/main/server/vcr-server/vcr_server/middleware/routing.py

The difference between orgbook api versioning and aca-py is that for orgbook the entire api is versioned, and for aca-py each protocol is versioned independently.  So there's no "overall api version" - if we use the version parameters in the header then it needs to be targeted at each protocol individually.  And since protocols can be loaded as plug-ins we have no overall control over what api's and versions that aca-py will be running.
The existing schema, cred def and revocation api's have no version info, so if we add the new version of the api's under a /v2 then we'll need to also move the existing api's to a /v1 url and build in handling for when a non-versioned api is called.  (As Wade mentions we can handle this the same was as for orgbook however it will need to be implemented for each protocol and not for aca-py overall.)

I've updated the main description per the comments, @swcurran @WadeBarnes @jamshale

For:
On aca-py startup, verify that the configured --wallet-type matches the actual database. If the wallet is askar and aca-py is started with --wallet-type askar-anoncreds, aca-py should shut down with an error message. Likewise if the wallet is askar-anoncreds and aca-py is started with --wallet-type askar.
I'm trying to determine a way to tell if the existing wallet/DB is askar or askar-anonreds, because they both use Askar storage. Anyone know of a record type I can look for? Or another approach?
I was looking at https://github.com/hyperledger/aries-cloudagent-python/issues/2384 but working with an encrypted DB, i'm not sure how I can use it. And that would only work for a DB/wallet that has a rev reg record.

I think we store a version number in the database, so we could store the storage type in the same way.  If there is no storage type, then assume it's "askar".  And then we could store a value for "anoncreds" (which we would store on creation of the wallet, or on upgrade from askar to anoncreds).

We do something similar in the upgrade.py that was updated here: https://github.com/hyperledger/aries-cloudagent-python/pull/2185/files
We are doing something similar, but instead of using the version number, we’re using the wallet type string.

And then we could store a value for "anoncreds" (which we would store on creation of the wallet, or on upgrade from askar to anoncreds).

Presumably we want to call it “askar-anoncreds” to match the —wallet-type option, right?

then assume it's "askar”.

Is there anything more we should do to prevent this from being run when the wallet-type is “indy”?  Or is that already considered?

This :point_up: makes sense. But, I am a bit unsure about what to do with and indy wallet and config. I was thinking if the record is None and the first config it gets is either indy or askar that it will use that as the storage type? It would only cause problems if the first config it receives it the wrong type. Say it's an indy wallet and first config is askar, or vice versa.
If it gets askar-anoncreds config and a non empty DB then it will assume the storage type is askar and fail. If the DB is empty then it can startup and set the storage type to askar-anoncreds.

But, I am a bit unsure about what to do with and indy wallet and config

I think if you try to open an Indy wallet using Askar it will throw an exception.
I suggest we just document that people will need to upgrade Indy to Askar (if they are still running an Indy wallet) and then we don't need to do anything else.


Refactor Webpack Config

I have modified your project to take advantage of Webpack, I hope it makes sense to you.
Keep in mind, Webpack if a Bundler. It loads CSS, Images, HTML, JS,
so I have added imports in the index.js file to import CSS and also images imports in the JS files.

Good to go!


scrollToBottom

Guys, I do the same thing in your library, but scrollBottom () does not work for me

can you help me?

Yes I am going to implement this


feat: add ethereum "query_block_transaction_count_by_number" RPC call

Implemented "query_block_transaction_count_by_number" RPC call.
Implemented:
-CLI Commands + tests
-Beerus Core implementation + tests
-RestAPI implementation + tests
Pull Request type

Please check the type of change your PR introduces:

[ ] Bugfix
[X] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Testing
[ ] Other (please describe):

What is the current behavior?

Issue Number: #75
What is the new behavior?

Does this introduce a breaking change?

[ ] Yes
[ ] No

Other information

Codecov Report
Base: 84.43% // Head: 84.31% // Decreases project coverage by -0.11% :warning:

Coverage data is based on head (e9ae266) compared to base (2cfd26d).
Patch coverage: 81.81% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #124      +/-   ##
==========================================
- Coverage   84.43%   84.31%   -0.12%
==========================================
  Files          21       21
  Lines         938      982      +44
==========================================
+ Hits          792      828      +36
- Misses        146      154       +8

Impacted Files
Coverage Δ

...ore/src/lightclient/ethereum/helios_lightclient.rs
28.57% <0.00%> (-3.87%)
:arrow_down:

beerus_core/src/lightclient/ethereum/mod.rs
100.00% <ø> (ø)

beerus_cli/src/model.rs
52.94% <50.00%> (-0.26%)
:arrow_down:

beerus_rest_api/src/api/ethereum/endpoints.rs
94.69% <95.00%> (+0.06%)
:arrow_up:

beerus_cli/src/ethereum/mod.rs
100.00% <100.00%> (ø)

beerus_cli/src/runner.rs
78.26% <100.00%> (+0.98%)
:arrow_up:

beerus_rest_api/src/api/ethereum/resp/mod.rs
100.00% <100.00%> (ø)

beerus_rest_api/src/lib.rs
89.28% <100.00%> (+0.39%)
:arrow_up:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


dockerBuildNative fails

Running gradlew dockerBuildNative on a project fails in step 10/11 with the following  error
Could not build image: COPY failed: stat /var/lib/docker/overlay2/c359be076403b377d0b41047e67284d7a39957e65117266c6a8c3e8572b3a38f/merged/home/app/application: no such file or directory

In step 7/11 it runs the following command
Step 7/11 : RUN native-image -H:Class=counter.Application -H:Name=application --no-fallback -cp /home/app/libs/*.jar:/home/app/resources:/home/app/application.jar

According to https://github.com/oracle/graal/issues/726, the -H:Name argument here is ignored ending up with a binary name based on the project instead.

Check whether you have an existing native-image.properties file that overrides the name

The existing native-image.properties in /src/main/resources/META_INF/native-image/counter/counter-application is being ignored. The content was already naming a different application name, but the DockerfileNative in /build/docker didn't reflect that.
Removing the existing native-image.properties gives me the same result in step 10

That is weird since creating a fresh app from https://micronaut.io/launch doesn't exhibit this behaviour

I can confirm that. I've used the CLI tool version 2.1.0
mn create-app counter --build=gradle --features=mongo-reactive,graalvm --java-version=11 --test=spock
I cannot find any distinct difference except the micronaut version.
Happy to close this ticket as my issue seems to be resolved with a new launch.


BasicTagTest.java:43-43: Write tests for other conditions.

The puzzle 36-ca71c469 in src/test/java/com/driver733/vkmusicuploader/wallpost/attachment/support/mp3filefromfile/basictag/BasicTagTest.java (lines 43-43) has to be resolved: "Write tests for other conditions."
The puzzle was created by Mikhail Yakushin on 22-Jun-17.
Estimate: 0 minutes, role: IMP.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 36-ca71c469 has disappeared from the source code, that's why I closed this issue.


Feature: Support for optional chaining

It would be beautiful if Helm could support optional chaining. Similar to JavaScript in latest standard ECMAScript 2020. See https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Optional_chaining
It would basically look like this:
spec:
  type: {{ .Values.service?.type | default "ClusterIP" }}

If service is not defined, it would short-circuit to undefined/false and thus default to "ClusterIP".
This could be used in other cases, such as:
spec:
  {{- if .Values.global?.internalIPFamily }}
  ipFamily: {{ .Values.global.internalIPFamily }}
  {{- end }}

It solves a few things.

default not working when parent values does not exist in a chain, giving errors, see https://github.com/helm/helm/issues/8026.
It would be able to get rid of the, in my opinion, not great best practice for nested or flat values, see https://helm.sh/docs/chart_best_practices/values/#flat-or-nested-values. Optional chaining would instead still make the code more readable (one-liner) not caring about value depth. It would also in my opinion make it easier to read YAML files with the nested structure, clearly separating levels, than to use camelCase.

@thernstig let me say, this is a great idea.
This part of the templates is something handled by text/template in the Go standard library. It isn't something Helm can add on top of that. The feature request would need to be filed over on the Go project.

@thernstig let me say, this is a great idea.
This part of the templates is something handled by text/template in the Go standard library. It isn't something Helm can add on top of that. The feature request would need to be filed over on the Go project.

Thanks @mattfarina, I submitted https://github.com/golang/go/issues/43608. Should we close this then, or keep it open for tracking purposes? If the feature is implemented, documentation for Helm should be updated.

Thanks @mattfarina, I submitted https://github.com/golang/go/issues/43608. Should we close this then, or keep it open for tracking purposes? If the feature is implemented, documentation for Helm should be updated.

The unstale bot should skip to automatically close threads that has X amounts of upvotes 😆

For whoever reads about this missing feature later:
{{- if .Values.global.cluster.vpcId }}

may give nil pointer evaluating interface {}.vpcId.
The solution is to use parentheses (explanation on StackOverflow):
{{- if (.Values.global.cluster).vpcId }}

You can use multiple parentheses, e.g. if global in the above example could also be nil.


Extend handling of range headers in the sample

When testing the R2 with this worker sample, I ran into some applications (e.g. VLC) failing to read the data and failing immediately, due to corrupted network stream errors.
This was because they perform range requests which have optionally the offset or end missing, which the original sample doesn't handle correctly. This adds proper handling for those headers, which fixes those applications not working with this sample.
It also adds the "accept-ranges" header. Some apps seem to be confused when it's missing, while some ignore it, but change their behavior (how they send the range headers) when it's present.
I'm not sure if there are any unexpected side effects to these changes, but from my testing so far, it seems to work pretty well and I figure it might help other be less confused when their apps fail to work with the sample.

The latest runtime release should remove the need for doing range header parsing yourself, you can simply pass the request headers into the onlyIf of R2GetOptions and the returned object has a range option that'll contain the actual returned range, in the event that your requested range couldn't be satisfied.
https://community.cloudflare.com/t/2022-8-12-workers-runtime-release-notes/410873?u=kiannh
The Demo Worker could probably be cleaned up to utilise those two features, since they've landed since that Worker was added to the documentation.

Yeah I agree with @KianNH. The worker should just use the available methods instead of doing parsing by hand.


Inconsistent install path for xtensor-blasConfig.cmake with respect to the xtensor family

Hi Guys,
As said in the title, xtensor-blasConfig.cmake is installed by default in share/cmake/xtensor-blas folder while for the rest of the family such files are installed in lib/cmake/family-name.
Of course, this not urgent :-) !
Cheers,
Thibaud.

Thanks @tkloczko for the report. Indeed, that was an oversight on our side. Just opened a fixup PR.


[TextField] Make fullWidth the default on form element?

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
Coming from https://github.com/marmelab/react-admin/issues/8796, it seems it's common for UI libraries to set fullWidth={true} as the default behavior (see benchmark). Maybe we should do the same for MUI's open-source projects.

https://stackblitz.com/edit/react-ts-fbuqu9?file=App.tsx
An alternative option would be to set a consistent min-width CSS property on all the inputs. However, it would be surprising to override, so unlikely to fly.
Examples 🌈

https://getbootstrap.com/docs/5.3/forms/form-control/
https://ant.design/components/input
https://chakra-ui.com/docs/components/input/usage
https://sancho-ui.com/components/form/#input
https://baseweb.design/components/input/
https://seek-oss.github.io/braid-design-system/components/TextField
https://balance.reckon.com/package/text-input
https://tailwindui.com/components/application-ui/forms/input-groups

Motivation 🔦
In most of the time, as a developer, you need to set a width to an input element. Sometimes, it's width: 100%, sometimes, it's not, but it's almost never the default value. So maybe we should make this breaking change.

The problem, as I described in https://github.com/marmelab/react-admin/issues/8796, is different: inputs have an inconsistent width by default, which makes the first version of a form always misaligned.
The solution you describe (a 100% width by default) would be a big breaking change. Currently, devs keep the default width when it's large enough to accommodate the content (e.g. for year, zip code, etc), and change it when the possible value is big (e.g. email). Your change would reverse this need for customization, and break all existing form layouts. Also, it would make forms unusable on large screens by default.
I'm +1 with a min-width equal to the current TextField width (computed by the browser using the default size of 20. It would only affect types of field Date and Select, which we do have to change width every time. I don't think a min width is hard to override.

I don't think a min width is hard to override.

The problem is that developers wouldn't understand why when they set thenwidth property, it's not applied correctly. They would need to know min-width takes precedence over width, they would need to know that the style is applied. IMHO, this is too much requirement. We have a prior art with the Tabs that has a min width and people who struggle to customize it.
I would personally struggle as I wouldn't be able to remember which element has a min width, and because there are others reason why the width might be wrong, I would need to dive into the dev tool => not a great experience.


Add support for 16 bit GrayScale TIFF

It would be nice if support to read and write 16 bit GrayScale TIFFs is added.

There is already read support for 16bit TIFFs but there is no support for writing any TIFF format.

nwin, I have been trying since last week, It cannot read a 16 bit GrayScale TIFF.

Ah right, I'm just guessing what you did. You need to read them using the TiffDecoder directly. If it still does not work, please file a bug report including the error message, minimal code sample, and if possible the file in question.

I'd like to propose a better API to handle this. For most uses you want one of two things:

An sRGB image that you will display on screen or send to another bit of code that only cares about image display
A linear buffer to be used for further processing in your own specialized code

So the best way to handle this would be to have two methods:

to_rgb() will always return an 8bit sRGB image that can just be displayed directly. If the source image is 16bit it gets gamma applied and 8bit converted. Image viewers will just use this and see all formats magically supported.
to_linear() will always return an f32 linear image with values between 0 and 1. 16bit images will just be converted to f32 in that range, 8bit images will do that but with gamma unapplied. Image processors will just use this and also have all formats supported without special casing each type of image.

Would that kind of API be accepted as a PR?

Sounds sensible except I would scale 16bit image to 1.0. Im currently working on a new image buffer library which provides some abstractions for exactly these uses cases (https://github.com/PistonDevelopers/image_buffer). I would prefer PR to it first instead of changing the API here.

When I say "16bit images will just be converted to f32 in that range" I mean turn 0..65535 into 0.0..1.0. Is that what you mean?


HPER_080: require RNG

Require hardware feature to create high quality random numbers. As per UEFI spec this capability should then be exposed by instance of EFI_RNG_PROTOCOL with at least one EFI RNG Algorithm supported.

Operating Systems use the EFI_RNG_PROTOCOL for entropy generation capabilities during early boot.

https://github.com/riscv-non-isa/riscv-server-platform/pull/20


SystemRegisterError not raised on caller's thread

If you try to bind to a hotkey that's in use by another program, system hotkey raises a RegisterException, but in a new thread, so it's not caught by any handlers you may have installed.
Sample, assumes you have another program bound to alt-shift-1:
from system_hotkey import SystemHotkey, SystemRegisterError

hk = SystemHotkey()

try: hk.register(('shift', 'alt', '1'), callback=lambda e: print('hi'))
except SystemRegisterError: print('Caught error')

input() # Hold program open until you press enter

Results:
$ python3 test.py
Unhandled exception in thread started by <bound method CallSerializer.call_functions of <system_hotkey.util.CallSerializer object at 0x7f46508a37f0>>
Traceback (most recent call last):
  File "/usr/local/lib/python3.5/dist-packages/system_hotkey/system_hotkey.py", line 790, in _xcb_the_grab
    xproto.GrabMode.Async, xproto.GrabMode.Async).check()
  File "/usr/local/lib/python3.5/dist-packages/xcffib/__init__.py", line 330, in check
    self.conn.request_check(self.sequence)
  File "/usr/local/lib/python3.5/dist-packages/xcffib/__init__.py", line 557, in wrapper
    return f(*args)
  File "/usr/local/lib/python3.5/dist-packages/xcffib/__init__.py", line 663, in request_check
    self._process_error(err)
  File "/usr/local/lib/python3.5/dist-packages/xcffib/__init__.py", line 634, in _process_error
    raise error(buf)
xcffib.xproto.AccessError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.5/dist-packages/system_hotkey/util.py", line 37, in call_functions
    func(*args, **kwargs)
  File "/usr/local/lib/python3.5/dist-packages/system_hotkey/system_hotkey.py", line 341, in register
    self._the_grab(keycode, masks)
  File "/usr/local/lib/python3.5/dist-packages/system_hotkey/system_hotkey.py", line 797, in _xcb_the_grab
    raise SystemRegisterError(msg)
system_hotkey.system_hotkey.SystemRegisterError: The bind could be in use elsewhere: 1

@Xcelled The last commit should have fixed the issue, i added some tests for it as well, are you able to test it on windows?

@timeyyy yes, at some point I can fire up my Windows machine. May not be for a day or two, though.


Not displayed after redirect from livewire component

Describe the bug
If a redirect is triggered from within a livewire component, the flasher output is never displayed as the request that would have triggered it originated within a livewire component, and the does not appear to be any mechanism available to delay the display until a future request in that case.
Whereas it is otherwise possible to define hops or delay stamps this does not appear to be possible from within a livewire component.
As a redirect causes a page reload, and as best I can tell presently display of flasher messages that originate within a livewire component occur as events rather than being added to the message bag in the session, the is presently no way to display a message after redirect.
To Reproduce
Steps to reproduce the behavior:

Add a flasher message within a livewire component, and then redirect to a different page, observe no message displayed
Add a flasher message within a livewire component and do NOT redirect, and observe message IS displayed

Expected behavior
I expect the notification to display after redirect, even if that requires me adding an extra flag when defining the message, so that for example when saving an entity and then redirecting to a list view - the save message is properly displayed.
Desktop (please complete the following information):

OS: macOS
Browser Firefox
Version 100

Hello @willvincent thanks for the details.
I just pushed the v1.0.4 which solve this redirection problem, can you check please ?
 php-flasher/flasher-laravel

you're welcome ^^


MANAGER GIT: Winter 18 Search Result Field Issue

Found by: Kevin Swiggum
Addressing CSG case: 17370350
Customer Issue Encountered:
When looking at the search results in a Winter 18 HEDA org you will see that for the column “Address” the word “USA” is partially cut off.
Technical Review of Issue:
Classifying this as a Winter 18 issue....and partially intended. Right now, if there are 3 lines in a search result field, the length of the 3rd line is cut off to match the max length of the first two lines. So the "USA" is getting cut off because the street and city/state/zip lines are really short.
They're doing this as a visual indicator to show that more data exists in a field than what they're seeing in the search results. The strange part is that the 3rd line cuts off based on the length of the top two lines rather than looking at the available width of the column.
Steps to Reproduce:

Log into a Winter 18 HEDA org
Create a Contact and include the following information for Address:
2a. Mailing Street = 1234
2b. Mailing City = test
2c. Mailing State/Province = test
2d. Mailing Zip/Postal Code = test
2e. Mailing Country = United States
Edit the Address Object Search Layout for "Search Results" and include the field "Mailing Address"
Copy and Paste the Address-ID into the Search bar and hit the Enter key
From the Search Results page click on the "Show More" link to expand additional Search options
Select Addresses
You will see where the issue is occurring in the Search Results view

Current Workaround:
None

**lurch: add

Tracking W-020923

Do we have any control over this behavior? I'm not sure its useful for us to log a bug here, when it should probably be sitting with core, no?

@kbromer - You are correct. We don't have control over this behavior since it seems to be a core issue. However, because this issue was reported via Case by a customer, I felt it necessary to create the Issue here so the HEDA community understands that this is not a HEDA issue but a Core issue. I've discussed this Issue with the HEDA PM and he will be escalating it to Core so they are aware of the impact. Once Core resolves the Issue we can close this Issue out along with the WI on our end.

@kbromer - Based on our discussion I've created a new label called: core bug.

Update!
The "fade" is an updated UI improvement in Winter '18. This is NOT a core bug but is working as designed.
Closing this WI now.


Put special patroni parameters to the bootstrap.

Some special patroni postgresql parameters, like max_connections,
should reside in the bootstrap.dcs.postgresql.parameters section
to come into effect.

Coverage decreased (-0.05%) to 11.29% when pulling f5550c337bf3064f074e9e1dcb53fd2eefc0e7e3 on fix_max_connections into e6d12b3480cd7a22bc9d48189b860eea8103fff0 on master.

Some special patroni postgresql parameters

What exactly makes these parameters special ?

:+1:

👍


Documentation - Crosslinking?

What's the best way to do cross-linking (e.g. between chapters)? Is there a way that plays nicely across platforms?  Or does cross-linking only work if I stick to Bookdown and don't try to port to Leanpub or Coursera?

By cross linking, do you mean providing a link to another chapter in a different chapter?

Yeah, linking from one chapter to another.  In the AnVIL books, there are a lot of places where it's nice to be able to say "for more information on X, see this chapter.

This does work across platforms now that we have iframes! But we should document this.

Done: https://github.com/jhudsl/OTTR_Template/wiki/Borrowing-chapters-between-courses


Query regarding the Bulk Transfer for verbs plugin

I have some doubts regarding the bulk transfer mechanism supported by Mercury:

In the overview section of documentation, it is mentioned that RPC layer is used for small messages. What is the maximum size that this transfer supports? Also, are the messages with small length sent with ibv_send_inline (not registering the memory region) method for verbs?
For bulk transfer, with verbs, is there a way to provide a previously registered MR region. In the examples, I see that the user is allocating the memory using malloc / calloc and sending the same buffer in the API. HG_Bulk_Create() registers the buffer as MR region. Can the same region be used again for data transfer?

Maximum size depends on the underlying transport, it can be queried using HG_Class_get_input_eager_size(). We do not call into verbs directly but rely on OFI to do that, which calls ibv_post_send(). You do not need to worry about memory registration at this level. When needed, registration is done internally.
You can call HG_Bulk_create() once and then do multiple transfers with it, that's right. It is actually recommended that you do it this way, outside of your transfer code path to avoid memory registration overheads.

Thanks for prompt response.

I ran HG_Class_get_input_eager_size() and it returned 4060 in my system, it means that HG_Forward can accept this much amount of user payload. But for verbs, the data is still going over RDMA. It means that the OFI will have to internally register the buffer being used in HG_Forward API?
Is this explanation correct? If yes, it would mean that every HG_Forward call will first register the input buffer, thus making the call slow and not desirable in fast path.

We have a pool of pre-registered buffers, when you send a message we use a buffer of that pool, which is why space is limited. There is no registration in that code path for small data.

We have a pool of pre-registered buffers, when you send a message we use a buffer of that pool, which is why space is limited. There is no registration in that code path for small data.

What is the size of the pre-registered pool and is it configurable?
Is there any queue mechanism within mercury in case of pool going out of space due to many outstanding requests or the request is failed back to the user?
If the request if failed, does it make more sense to initialise multiple mercury context with same IP but different port combination to have a larger pool?


MARC Conversion error message in Sinopia Production 3.4.1

Instance record title: Gedenkbuch für die Münchner Opfer der nationalsozialistischen "Euthanasie"-Morde
https://api.sinopia.io/resource/0dbc0974-67cd-488c-b03a-26691df19c44
The following MARC Conversion error message was display after clicking the MARC Conversion icon.

Please investigate it and let me know what caused this error.
Thanks.

@jermnelson This seems like a prod deploy issue.


fix: replaced defined() by function_exists()

Description
Fixes #1173
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

Please sign all your commits.
See:
https://github.com/codeigniter4/shield/pull/672#issuecomment-1468658726
More Info:
https://github.com/codeigniter4/CodeIgniter4/blob/develop/contributing/pull_request.md#signing

Please sign all your commits.
See: #672 (comment)
More Info: https://github.com/codeigniter4/CodeIgniter4/blob/develop/contributing/pull_request.md#signing

Sorry about that. It should be signed now

It should be signed now

No, see:
https://github.com/codeigniter4/shield/pull/672#issuecomment-1468722608

It should be signed now

No, see: #672 (comment)

Sorry, first time signing commits. Hopefully it is signed now

Sorry, first time signing commits. Hopefully it is signed now

No problem at all! Signing commits for the first time can be a bit tricky.  And thanks again for your contribution—it's much appreciated!
LGTM!

Awesome! Keep up the good work. Thank you

@FrancoisChaumont Please do not close PR. Wait for your code to be reviewed and merged by other team members.

@FrancoisChaumont Please do not close PR. Wait for your code to be reviewed and merged by other team members.

Apologies. Ok

@FrancoisChaumont Thank you!


Freeze Rows functionality

Hi there,
I noticed that there is the freezeColumns option. Is there an analogous functionality for rows, or a recommended way to achieve it?
Thanks

This would be a great feature if implemented.

Some thoughts for future jason. This is actually fairly easy to implement if you just render the freeze columns into the header canvas.

This is planned in https://github.com/glideapps/glide-data-grid/issues/823
Closing this issue to track there.


Recommended pattern for direct DOM manipulation in Blazor

Normally we just tell people not to do it because it's unsafe, but if your goal is to integrate with some 3rd-party JS library that produces DOM output, then we'd like to be able to describe what kind of thing is legit to do.
For example, we could elaborate on this comment and put it in docs.

It's very briefly mentioned in Components ...

OnAfterRenderAsync and OnAfterRender ... Use this stage to perform additional initialization steps using the rendered content, such as activating third-party JavaScript libraries that operate on the rendered DOM elements.

... and the "unsafe" advice that you mentioned under Capture references to elements in the JS interop topic ...

Do not use captured element references as a way of populating the DOM. Doing so may interfere with the declarative rendering model.

... but then (next sentence) it vaguely states ...

ElementReference ... the JavaScript-side code receives an HTMLElement instance, which it can use with normal DOM APIs.

I'll improve the content using your language/example.


Frontend exception when changing a user's full name after deactivating them

I saw this exception in production, and through looking at ReadlmAuditLog determined that the user was deactivated, and then their name was changed.  I guess I wasn't totally sure you could change names of deactivated users from our admin UI, but I guess this suggests that is possible.  Possible fixes include making sure we have deactivated user IDs in the realm present on the frontend so we can know to ignore data related to them.  I think we kinda need data on deactivated users to hydrate things like avatars anyway.
Got update_person event for unexpected user

Stacktrace:
Error
    at Object.a.error (/static/webpack-bundles/common-fb0216afcf56b783fef3.07be352d6e58.js:1:4890)

    at Object.update_person (/static/min/app.25c2e847f59b.js:1911:60)
       = static/js/user_events.js line 14 column 9

    at Object.dispatch_normal_event (/static/min/app.25c2e847f59b.js:1888:74)
       = static/js/server_events_dispatch.js line 158 column 13

    at /static/min/app.25c2e847f59b.js:1875:119
       = static/js/server_events.js line 86 column 20

    at Function.m.each.m.forEach (eval at e.exports (/static/webpack-bundles/common-fb0216afcf56b783fef3.07be352d6e58.js:1:577), <anonymous>:5:2350)

    at g (/static/min/app.25c2e847f59b.js:1874:734)
       = static/js/server_events.js line 90 column 5

    at success (/static/min/app.25c2e847f59b.js:1877:352)
       = static/js/server_events.js line 176 column 17

    at Object.a.success (/static/min/app.25c2e847f59b.js:1324:237)
       = static/js/channel.js line 64 column 16

    at Object.t [as success] (/static/webpack-bundles/common-fb0216afcf56b783fef3.07be352d6e58.js:1:3397)

    at i (eval at e.exports (/static/webpack-bundles/common-fb0216afcf56b783fef3.07be352d6e58.js:1:577), <anonymous>:2:27151)

    at Object.fireWith [as resolveWith] (eval at e.exports (/static/webpack-bundles/common-fb0216afcf56b783fef3.07be352d6e58.js:1:577), <anonymous>:2:27914)

    at z (eval at e.exports (/static/webpack-bundles/common-fb0216afcf56b783fef3.07be352d6e58.js:1:577), <anonymous>:4:12120)

    at XMLHttpRequest.eval (eval at e.exports (/static/webpack-bundles/common-fb0216afcf56b783fef3.07be352d6e58.js:1:577), <anonymous>:4:15680)

Currently, it is possible to change the name of a deactivated user due to a bug in progressive list updation that I had reported earlier as issue #5451. Fixing that issue should automatically fix this one.
Steps to reproduce this in dev environment:
1: Login as Iago.
2: Go to 'Users' in organization settings. Search for Zoe.
3: From the search results deactivate Zoe.
4: In another browser window login as Aaron.
5: Switch back to previous window.
6: Remove the search filter. Although Zoe's account has been deactivated, it will be shown as if it is active.
7: Now change the name of Zoe. You will get a traceback in Aaron's browser console.

Cool.  Probably worth fixing that UI code path as well as making it not thrown an exception if one does change such a name (There's a use case for that in editing how historical messages sent by someone are displayed)


Obsidian block reference doesn't work

Thanks for the great plugin, I'm just facing a little issue.
Unless I'm doing something wrong, the block reference within obsidian doesn't work, since the plugin prints the randomized block reference before the highlight in another paragraph.
I would think changing the 66th line in src/renderer/index.ts from
${HighlightIdBlockRefPrefix}${highlight.id}\n${trimmedHighlight};
to something like
${trimmedHighlight}${HighlightIdBlockRefPrefix}${highlight.id};
would do the trick.
edit: Upon doing some digging, I now see that these automated block references is a new addition purely for the plugin to keep the progress for re-syncing, and not for users to directly use in their writing process. Implementing the chance I suggested would then probably break the re-sync method as well. Still, it would have been nice if these block-references can be appended to the end of the quote, and modify the re-sync code appropriately, so that their purpose would be two-fold.

See:

This is definitely not expected behaviour. The block reference must be
before the highlight otherwise resync doesn't work.
Can you share more information about your setup. Your operating system and
if you are syncing from file or Amazon online.
I suspect the issue could be that I'm always using \n instead of \r\n
on Windows?
On Sat, 6 Nov 2021 at 1:14 AM, alicangok @.***> wrote:

Unless I'm doing something wrong, the block reference within obsidian
doesn't work, since the plugin prints the randomized block reference before
the highlight in another paragraph.
I would think changing the 66th line in src/renderer/index.ts from
${HighlightIdBlockRefPrefix}${highlight.id}\n${trimmedHighlight};
to something like
${trimmedHighlight}${HighlightIdBlockRefPrefix}${highlight.id};
would do the trick.
[image: image]
https://user-images.githubusercontent.com/26303095/140508623-8352aaab-eb49-4687-88df-15f09cfb5c51.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hadynz/obsidian-kindle-plugin/issues/93, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACNBQLACH65P75B2NMBZZDUKPDCDANCNFSM5HNWTIFQ
.

--
Sent from Gmail Mobile

This is not expected behaviour. The special ref- quotes are always inserted by the plugin before a highlight and is used as state information for the plugin to identify a given highlight and on order to do resync logic.
This was an opinionated design choice that had to be made go allow for portable resyncing to take place.
Can you give me more information on your setup so I can understand how you got this result. What OS are you using, what type of sync did you run and are you using a custom template?
You suggested moving the plugin's block reference to after the highlight. Curious to know how is that makes a difference in your workflow?

Thanks for your reply. Don't get me wrong, there is absolutely nothing wrong with the sync algorithm. In
Here's my workflow: Let's say I'm working on article.md, and I want to reference a quote from a book. Using your plugin, let's say I've extract the highlights to book_title.md, which contains the following block:
^ref-24706
Quote. -location:[123]()

Now, within article.md, when I want to reference this block, I can start typing " [[book_title^quote.. " , and obsidian will automatically assign a reference ID, such as 24c0a0, and your the original block becomes:
^ref-24706
Quote. -location:[123]()^24c0a0

So, this results in a rather redundant second reference for the same quote, which doesn't look elegant at all, as you can see in the following picture:

Instead, if the plugin could originally produce, and work with a template like this, it could be much more elegant.
Quote. -location:[123]()^ref-24706

Thanks

Thanks @alicangok for the detailed walkthrough of your idea.
I totally get it. I did not know that when you reference a block in another page, Obsidian will auto-magically generate and add a new block reference, or use an existing reference if one exists.
I agree... your suggestion of adding block references to the end of the annotation block is much better as it fits with the convention that Obsidian uses.
The challenge of implementing your suggestion, is that it is yet another breaking change in how the plugin generates annotations in files which sucks. I'll sleep over it, but I definitely take on your feedback and will implement it.

This has been now been implemented in 1.1.0. Please check it out and let me know if it resolves the issue you raised and works well in your workflow.
You might have to delete one of your books so the plugin can recreate it on new sync with block references printed end of line.

That was fast. Thank you, works perfect.


Vanishing datagrid vertical gridlines

Describe the bug
When a datagrid column is resized, and the user scrolls horizontally some of the vertical gridlines will vanish.
Returns to normal when the datagrid is in it's leftmost position.
To Reproduce

Turn on gridlines visibility
Turn on column resizing
Resize a column
Scroll the datagrid horizontally
See vertical gridlines vanish

Expected behavior
Grid lines should not vanish.
Screenshots

Normal

The whole vertical gridlines vanishes if it's close to the left screen

The header vertical gridlines vanishes the moment you resize

Desktop (please complete the following information):

OS: [e.g. Windows, Mac, Linux (State distribution)]
Version [e.g. 11.0.4]

Additional context

Sample:
DatagridLinesIssue.zip

👀 I can reproduce, thx.

👀 I can reproduce, thx.

This issue is also there in the default Fluent DataGrid style, but it only affects header cells (all cells below are fine):

saw it. I have no idea where it comes from atm. Propably some rounding issue during resize. If you have the time to dig into, you may try other rounding strategy in resize method (using samples/ControlCatalog).

Hello. Try setting the property UseLayoutRounding on False.
The problem is precisely rounding. Also, overriding the rounding method inside the table introduces errors. As a result, I cleared the overridden rounding method and use UseLauotRoinding=False

We might need to set UseLayoutRounding on the lines directly, if it helps.


fix: update CYA options and styles

Description
Changes:

Replace old links with the new ones as per task requirement.
Style updates

Notes:

Created LogoHeartConfigurable with configurable colour as a duplicate of LogoHeart as it is added to .gitignore.
Links were already shuffled by shuffle function in code so the order will differ from the one in Figma.

Test plan

Related issues

Fixes RET-1175

Backwards compatibility

Network scalability
If a new NetworkId and/or Network are added in the future, the changes in this PR will:

[ ] Continue to work without code changes, OR trigger a compilation error (guaranteeing we find it when a new network is added)

Congratulations with your first PR! 🎉

ah, i see that CYA is covered by e2e tests so those will need to be updated. you might be able to make the code changes and commit them here to run on the CI without testing locally (the changes might be obvious) but if not, lmk if i can help with getting the e2e tests running locally. the readme is pretty comprehensive, the only extra part is that you may need to change the device name in package.json (i don't have the iPhone SE (2nd generation) simulator, so i change the string to 3rd generation)

oh one more thing, maybe the PR title (which will be the commit message in the main branch) can be a bit more descriptive? e.g. update CYA options and styles? ✨

@kathaypacific @bakoushin Fixed all the comments.

@kathaypacific can I submit with "merge when ready" now?


launch.exe doesn't exist in VRChat versions <= 1078

The current fallback for launching the game and determining the location of the UserData folder relies on the existence of launch.exe, which in VRChat versions less than or equal to 1078 it does not exist. Instead, launch.exe is launch.bat.

Yea I had gotten ahead of myself completely forgotten about that, oh well release should be soon enough.


Custom renderers do not render output when loading an existing Notebook

VSCode Version:
OS Version:

Steps to Reproduce:
1.Register a custom renderer for text/html and custom pre-load script file.
2.Run a cell with text/html output will be rendered using custom renderer
3. Open a notebook with above cell information.
4. The preload script is not loaded, hence custom rendering fails.
5. Running the cell results in preload script being loaded

Does this issue occur when all extensions are disabled?: N/A
@rebornix /cc

Already fixed in master and will be shipped soon.

@DonJayamanne can you confirm this is working now?
Otherwise, @rebornix  how can I verify this?

I know how to verify this


OCPBUGS-18341: change required pod anti-affinity rule to preferred rule

What this PR does / why we need it:
Change the hypershift operator pod anti-affinity rule from required to preferred while keeping the default replica at 2 so that the hypershift operator can be installed on SNO and can be upgraded (rolling update) on 2-node OCP cluster.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
https://issues.redhat.com/browse/OCPBUGS-18341
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/jira refresh

/cherry-pick release-4.14

/lgtm

/cherry-pick release-4.14

/jira refresh

/jira refresh


feat(Rest): add patch method

Yay for methods :D

Rename commit
add tests
update docs

wanted to do that in the merge message
is planned
is planned
:smile:

That should be done before the merge message as it's still a record in versioning. Otherwise it looks pretty awesome

@RWOverdijk seems express doesn't like lowercase patch. i uppercased all default methods then

@doktordirk I use express and it doesn't care. Is this new in express 4 perhaps?

that with 4. but the problem is only with patch vs PATCH. the others work, just though changing the defaults to upper would still be BC. give 'patch' in our tests and see if it works, didn't respond for me

It's not BC if someone relies on it. I guess that's the thing with BC.. Either way I'm fine with using upper case.

👍

renamed first commit

Still :+1: haha


shift+enter

Is shift+enter supported? If not, is it possible to add it?
Expected result:
Hitting shift+enter inserts a BR in active P tag.
Actual result:
Hitting shift+enter does nothing.

you should use HardBreak extension


Godot 3.0 not recognizing old fonts

Arch Linux - Godot 3.0
Issue description:
Godot 3.0 does not recognize Godot 2.1.3 .fnt files as fonts. Godot 3.0 looks for a .font file, and if you try to make it load a .fnt file it spits out that it not a resource. Found while converting Godot 2.0 project to Godot 3.0.
FlappyGodot3.zip

yeah, unfortunately this was needed so we can add supprot for bmfont .fnt
files without godot trying to recognize them.
I suppose the exporter should rename those extensions..
On Tue, Jul 11, 2017 at 5:11 PM, BrainBlasted<EMAIL_ADDRESS>wrote:

Arch Linux - Godot 3.0
Issue description:
Godot 3.0 does not recognize Godot 2.1.3 .fnt files as fonts. Godot 3.0
looks for a .font file, and if you try to make it load a .fnt file it spits
out that it not a resource. Found while converting Godot 2.0 project to
Godot 3.0.
FlappyGodot3.zip
https://github.com/godotengine/godot/files/1140087/FlappyGodot3.zip
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/9610, or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z2xItIu5awrSkyT0EqggRq_36HY7fks5sM9cOgaJpZM4OUwKj
.

For me it does not work even after renaming. The editor doesn't recognize it as a resource.

Confirmed, I've implemented renaming the .fnt files to .font but they don't load. :/

Running into the same issue.  Can't find a way to get Godot 3.0 to recognize BMFont files.

has anybody found an answer to this problem?

@stubbsy345, should be fixed by #11869

@stubbsy345, should be fixed by #11869


Revising the content page

I'm not confident about this page, but it serves as a kind of NWTWWHB hub to show people that content is a focus. As we get the tutorial and CMS guides in, I think it will come into its own, and we can focus on linking out to examples.

Thanks, Dan! Lets get this into my branch!


Introducing Iterable, deprecating Value

Towards #2431
This is one of the bigger-ones. The type hierarchy evolves in a backward compatible way such that io.vavr.Iterable supersedes io.vavr.Value.

Codecov Report

Merging #2449 into master will increase coverage by <.01%.
The diff coverage is 60%.

@@             Coverage Diff              @@
##             master    #2449      +/-   ##
============================================
+ Coverage     95.02%   95.02%   +<.01%
- Complexity     5366     5367       +1
============================================
  Files            89       90       +1
  Lines         12744    12746       +2
  Branches       1591     1591
============================================
+ Hits          12110    12112       +2
  Misses          454      454
  Partials        180      180

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/io/vavr/collection/Seq.java
90% <ø> (ø)
40 <0> (ø)
:arrow_down:

src/main/java/io/vavr/concurrent/Future.java
97.65% <ø> (ø)
127 <0> (ø)
:arrow_down:

...c/main/java/io/vavr/collection/JavaConverters.java
91.12% <ø> (ø)
1 <0> (ø)
:arrow_down:

...in/java/io/vavr/collection/LinkedHashMultimap.java
96.61% <ø> (ø)
10 <0> (ø)
:arrow_down:

src/main/java/io/vavr/Lazy.java
100% <ø> (ø)
35 <0> (ø)
:arrow_down:

src/main/java/io/vavr/collection/TreeMultimap.java
91.3% <ø> (ø)
11 <0> (ø)
:arrow_down:

src/main/java/io/vavr/collection/BitSet.java
99.43% <ø> (ø)
82 <0> (ø)
:arrow_down:

src/main/java/io/vavr/collection/SortedMap.java
66.66% <ø> (ø)
2 <0> (ø)
:arrow_down:

src/main/java/io/vavr/control/Try.java
98.36% <ø> (ø)
103 <0> (ø)
:arrow_down:

...n/java/io/vavr/collection/HashArrayMappedTrie.java
97.02% <ø> (ø)
1 <0> (ø)
:arrow_down:

... and 41 more

Continue to review full report at Codecov.

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


SolidJS: migrate css prop solution to solid-styled

Migrate our approach for the css prop in the SolidJS generator from using https://github.com/solidjs/solid-styled-components to using https://github.com/lxsmnsyc/solid-styled (as recommended by some folks in the SolidJS Community)

Hey @samijaber, I would like to contribute to this.
I found this line , is responsible for importing the css wrapper from solid-styled-components, which needs to change to make the migration happen and we are injecting css into the component here using the style tag, does the css string also needs to be changed for it to work?
(Sorry for the newbie questions, just needed a heads up if this is the correct approach)

@akshatgarg12 I haven't had time to look: I am not familiar with the difference between both, so we'd need to read up the docs/API of both and see if there are differences between them, and go from there.
I actually also see that solid-styled requires Babel https://github.com/lxsmnsyc/solid-styled#babel, which would be an extra thing to setup for the SolidJS output to work properly. That does make me wonder if we should commit to it, although it sounds like this is the direction the SolidJS community is exploring...

Note to anyone who'll look into this. If possible, make the solution work on Solid components class="" rather than className="", because className quite likely could go away in future versions.

Please assign me, I will migrate css props to solid-styled. thanks

@Syed-Bilal-Haider-Engineer go for it!


Please Get a 'baseline.pth' model.

I will use 'vocal-remover' in my Capstone project.
but i dont find 'baseline.pth' model.
how can i get the model??

Oh..... Sorry... I find that~~~!!! Thank you~

You can also download new models here: https://github.com/Anjok07/ultimatevocalremovergui/releases/tag/v4.0.0

found all the assets here https://github.com/Anjok07/ultimatevocalremovergui/releases

found all the assets here https://github.com/Anjok07/ultimatevocalremovergui/releases


feat: convert maple shares to exit assets

https://app.shortcut.com/avantgardefinance/story/7733/mapleconvertsharestoexitassets?vc_group_by=day&ct_workflow=all&cf_workflow=500002430


Should not deselect disabled checked checkbox on click on unselected all

There is an issue we face where there are some rows which shows as checked and disabled , and there should not be any action expected on these rows , since we are not able to do any action on check of check box because those are disabled but when we click on unselected all it deselect the disabled selected rows but it should not do any action on disabled rows as per the expectation , not able to restrict this action by implementation of function call on "unselected all" click
please check below screen shots of current implementation , for more clarity
please suggest how we can implement this functionality , or its a bug , we are stuck here .
we are currently using version 4.x of react-bootstrap-table
At the initial point when TableSelect loads -

After click on "unselected all" checkbox click, disable checkbox also unchecked but it should not

@anujemc, what's the version you used?

I am currently using latest version of react-bootstrap-table , which is v2.5.2

ok, it's a bug, I'll fix it quickly.

Fixed on 2.5.3. Thanks

Thanks for your quick fix ..


Add bashate linting

This commit adds the linting-bashate.sh script which is used to do
syntax checks on our scripts. Some important pieces of code are
scripts and previously our linting only checked YAML and Python
files. Upstream Openstack-Ansible also already includes bashate.
The linting-bashate.sh script itself searches recursively through
our files and runs bashate on files that have a (ba)sh shebang at
the top of the file. The script ignores the same bashate errors as
upstream OSA. The tox.ini was modified in order to add the
bashate script to the list of tests run. The .travis.yml file was
modified in order to add the bashate tox enviroment.


Well done!

Well done!
Although this project may still be in its infancy, I hope you can use some build tools (such as rollup) to build npm packages.
I found that there are a lot of var in the code after pulling down and lost ts disclaimer, this is not very friendly for project-level usage.

I'm not quite sure what you're saying, sorry. There is already a npm package: https://www.npmjs.com/package/polywasm. And the presence of the var keyword is completely irrelevant to how you use this library. I'm closing this issue because it doesn't seem like anything needs to be done here.


Allow taking an snapshot of a local file

For an .html file stored locally I use python -m http.server 80 and then shot-scraper http://localhost/file but it would be handy to be able to just point shot-scraper to the file directly.

I agree, this would definitely be useful.

Deleting this check:
https://github.com/simonw/shot-scraper/blob/773377e4d0422b46ff302717f553ae5018b667f6/shot_scraper/cli.py#L352-L355
And then running the following worked:
shot-scraper file:/tmp/index.html

I had to provide both file:/ and a full path to the file though.
So the actual implementation of that should probably notice when the user provides a URL that doesn't start with http:// or https:// AND is a file that exists on disk and, if so, do the work of turning that into a file://full/path/to/filename.html.

Thanks for the quick response!
Shot-scraper has been a great help for transforming complex epub ebooks into easier to display series of images. Thanks a lot for this tool!

I'm intrigued! Hadn't considered there might be epub applications.

They use XHTML. Some are simple (say some images and texts) totally ok to reflow, some are complex. On these last group you have some comics that have images but also transparent text on top so it is readable for accessibility, some others have images without the text and text on top with complex transformations (tilting and such). There are rarely displayed properly in a simple epub reader. I'm still testing but with a little fine fiddling shot-scraper is giving me exactly what I needed (rendering one HTML page into an image).

Tested this out using the Firefox "save complete web page" option on https://daringfireball.net/ and then running:
shot-scraper /tmp/Daring\ Fireball.html --height 1200

Got this output (because I had run it a few times before so the filename had to change):
Screenshot of 'file:/tmp/Daring Fireball.html' written to 'tmp-DaringFireball-html.2.png'

Result:

This is great. Thanks a ton, Simon!
On Sun, Mar 13, 2022, 10:01 AM Simon Willison @.***>
wrote:

Closed #35 https://github.com/simonw/shot-scraper/issues/35 via df4498c
https://github.com/simonw/shot-scraper/commit/df4498c2ee29c17906ff1198bd16584bf60aca31
.
—
Reply to this email directly, view it on GitHub
https://github.com/simonw/shot-scraper/issues/35#event-6231368079, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AP46PCMDHKPKFBDDDZHTCZLU7YNPVANCNFSM5QS2HOYA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


[IBCDPE-516] Updates for v3 tag

Purpose: This pull request will create a new tagged version of this action (v3) with the following updates:

Updates the action-pipenv-aws-setup to use OIDC integration (see README updates for more details) which includes adding more inputs to specify, adding specific permissions and removes the method of using github secrets
Updates action-pipenv-aws-setup to v2
Updates checkout to v3
Updates setup-python to v4
Updates default python version to 3.10
Add more inputs to specify when integrating this action into your github actions workflow (i.e: aws_region)

Successful test run using the new action-pipenv-aws-setup for this branch here: successful upload-and-deploy run on recover's refactor_artifacts_gh_action branch


5.5 New Op Codes

Not sure if this is already being researched, but someone I know found the 5.5 opcodes to update the plugin, I do not know how useful they will be for this, but I will post them here anyway.
"ActorCast": "0x03c0",
"ActorControl": "0x03b5",
"ActorControlSelf": "0x03c0",
"AddStatusEffect": "0x01d7",
"AlwaysOnTop": false,
"GameServerIpRange": "<IP_ADDRESS>/24,<IP_ADDRESS>/24,<IP_ADDRESS>/24,<IP_ADDRESS>/8,<IP_ADDRESS>/8,<IP_ADDRESS>/12,<IP_ADDRESS>/16,",
"GameServerPortRange": "10000,10001-65535",
"LogAllIpcMessage": false,
"MedianRttCalculationCount": 1,
"RequestUseAction": "0x03b0",
"RequestUseAction2": "0x01a0",
"ShowControlWindow": false,
"ShowLoggingWindow": true,
"SkillResultResponse01": "0x0204",
"SkillResultResponse08": "0x01c7",
"SkillResultResponse16": "0x032a",
"SkillResultResponse24": "0x035c",
"SkillResultResponse32": "0x0d20",
"UseAllIpcMessageLogger": false,
"UseAutoAdjustingExtraDelay": false,
"UseEffectApplicationDelayLogger": false,
"UseHighLatencyMitigation": true,
"UseHighLatencyMitigationLogging": true,
"UseOpcodeFinder": true

I was running the following but I think at least one opcode is wrong, I could see it processing but GCD clipping was pretty bad, some seem correct tho.
self.SUBTYPE_RESPONSE_ACTOR_CAST = 0x00b6
self.SUBTYPE_RESPONSE_ACTOR_CONTROL = 0x03b5
self.SUBTYPE_RESPONSE_ACTOR_CONTROL_SELF = 0x03c0
self.SUBTYPE_RESPONSE_ACTION_RESULT = [0x0204, 0x01c7, 0x032a, 0x035c, 0x02c5]

self.SUBTYPE_REQUEST_ACTION = 0x0387

I've reflected the changes, but I only applied the new opcodes. Please let me know if something's not working right.

Looks like you've got duplicate opcodes for actor cast and actor control self. The commit has both at 0x3C0.

Yep, that was a copy & paste error. It should have been 0x00b6.

Sorry if this is a dumb question, but does the updated opcodes mean it should work for the 5.5 patch now?
Thanks for the good and timely work!

Yes, and if it doesn't work, try downloading the latest release again.

Everything looks good to me, thank you!


Update Nacos to 2.2.0 in Spring Cloud Alibaba 2.2.x branch

我们鼓励使用英文，如果不能直接使用，可以使用翻译软件，您仍旧可以保留中文原文。
We recommend using English. If you are non-native English speaker, you can use the translation software.
Which Component
eg. Nacos,
Is your feature request related to a problem? Please describe.
We are prepared to release a new version of Spring Cloud Alibaba in 2.2.x branch, we are plan to update some components.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Update Nacos to 2.2.0 in Spring Cloud Alibaba 2.2.x branch, and test the reliability.

I solved it.


Add flight return feature

Add a feature to check when return flights are available, this can be achieved by checking  the destinations list to only fetch possible routes.

Added in 1.2.0


ROX-26719: Refactor in preparation for Secured Cluster TLS Issuer

Description
Another refactor PR to prepare for Secured Cluster certificates refresh, following https://github.com/stackrox/stackrox/pull/12825 and https://github.com/stackrox/stackrox/pull/13023. Like these other 2 PRs, it does not change any behaviour.
Changes in this PR
Some files are moved out of the localscanner package into the more generic certrefresh package (so that they can be reused for secured cluster certificate refresh):

cert_refresher.go
cert_expiration.go

Because the CertificateRequester interface (used in cert_refresher.go)  was tightly coupled with a gRPC call type:
type CertificateRequester interface {
	Start()
	Stop()
	RequestCertificates(ctx context.Context) (*central.IssueLocalScannerCertsResponse, error)
}

I introduced an intermediate type called IssueCertsResponse, and moved it together with CertificateRequester into their own package (to prevent cyclic dependencies).
Finally, it also moves certrefresh/localscanner/tls_issuer.go to certrefresh/localscanner_tls_issuer.go. This might go against the logical structure of the directories, but it'll make it easier to share some common testing code with securedcluster_tls_issuer (coming in a future PR), and allows me to keep some internal types unexported. It also makes instantiating this component look a bit cleaner IMO: certrefresh.NewLocalScannerTLSIssuer vs localscanner.NewLocalScannerTLSIssuer previously.
Next steps
Following the merge of this PR, the actual implementation of certificate refresh for secured clusters will be next.

User-facing documentation

[x] CHANGELOG is updated OR update is not needed
[ ] documentation PR is created and is linked above OR is not needed

Testing and quality

[x] the change is production ready: the change is GA or otherwise the functionality is gated by a feature flag
[ ] CI results are inspected

Automated testing

[x] added unit tests
[ ] added e2e tests
[ ] added regression tests
[ ] added compatibility tests
[x] modified existing tests

How I validated my change

Did not add new functionality, existing tests should check for regressions.

/test ocp-4-17-nongroovy-e2e-tests

/test ocp-4-17-nongroovy-e2e-tests


Missing i2c slave functions.

Hello,
I believe that there is currently no helper function for proper i2c slave support.
To get a pico working as a i2c slave cache of 256 bytes, I had to do something like this:
#include <stdio.h>

#include "pico/stdlib.h"
#include "hardware/i2c.h"
#include "hardware/irq.h"

#define I2C0_SLAVE_ADDR 0x04

#define GPIO_SDA0 0
#define GPIO_SCK0 1

uint8_t ram_addr;
uint8_t ram[256];

void i2c0_irq_handler() {
    uint32_t status = i2c0_hw->intr_stat;
    uint32_t value;

    /* Check to see if we have received data from the I2C master. */
    if (status & I2C_IC_INTR_STAT_R_RX_FULL_BITS) {
        /* Read the data (this will clear the interrupt). */
        value = i2c0_hw->data_cmd;

        /* Check if this is the first byte we have received. */
        if (value & I2C_IC_DATA_CMD_FIRST_DATA_BYTE_BITS) {
            ram_addr = (uint8_t)(value & I2C_IC_DATA_CMD_BITS);
        }
        else {
            /* If it is not the first byte, store the data in the RAM. */
            ram[ram_addr++] = (uint8_t)(value & I2C_IC_DATA_CMD_BITS);
        }
    }

    /* Check if the I2C master is requesting data from us. */
    if (status & I2C_IC_INTR_STAT_R_RD_REQ_BITS) {
        /* Write the data from the current address in RAM. */
        i2c0_hw->data_cmd = (uint32_t)(ram[ram_addr++]);

        /* Clear the interrupt. */
        i2c0_hw->clr_rd_req;
    }
}

int main() {
    stdio_init_all();

    i2c_init(i2c0, 100e3);
    i2c_set_slave_mode(i2c0, true, I2C0_SLAVE_ADDR);

    gpio_set_function(GPIO_SDA0, GPIO_FUNC_I2C);
    gpio_set_function(GPIO_SCK0, GPIO_FUNC_I2C);
    gpio_pull_up(GPIO_SDA0);
    gpio_pull_up(GPIO_SCK0);

    ram_addr = 0;

    /* Select the interrupts we want. */
    i2c0_hw->intr_mask = I2C_IC_INTR_MASK_M_RD_REQ_BITS
        | I2C_IC_INTR_MASK_M_RX_FULL_BITS;

    /* Set up the interrupt handler. */
    irq_set_exclusive_handler(I2C0_IRQ, i2c0_irq_handler);
    /* Enable I2C interrupts. */
    irq_set_enabled(I2C0_IRQ, true);

    while (true) {
        tight_loop_contents();
    }

    return 0;
}

I feel like there should be a function that takes some of the work away, like selecting the interrupts and setting the irq_handler.

it is a reasonable request


Publish to Open VSX

Currently, it is publishing on Microsoft VS Code Marketplace only.
it would be nice to publish to Open VSX  registry open-vsx.org. it will allow usage of it in VS Codium and not only VS Code.

documentation: https://github.com/eclipse/openvsx/wiki/Publishing-Extensions

For security reasons, I think it is best to no tuse the GitHub actions pointed here https://github.com/eclipse/openvsx/wiki/Publishing-Extensions#github-action but to provide our own steps in our github action workflow. (I will provide this part one of these days)


Incorrect explanation of Unsafe usages

https://github.com/dalek-cryptography/subtle/blob/b4b070c3faf87cb8f324bd0ed0a5e5ec32d3a5b0/src/lib.rs#L223 remarks that u8 is neither Send nor Sync, however u8 has auto-trait implementations for both: https://doc.rust-lang.org/std/primitive.u8.html#impl-Sync

I think a good point is being raised here. I am not sure whether the documentation changed in the meantime, or whether I missed something last time. However, we could update the comment with new argumentation following the current requirements for volatile reads:

https://doc.rust-lang.org/core/ptr/fn.read_volatile.html#safety
https://doc.rust-lang.org/core/ptr/index.html#safety

IIUC it's bad to mix volatile and non-volatile reads.
The current implementation seems to be built on a non-volatile write (to create the u8) followed by a volatile read.
It seems like it would be better to use volatile_write to write the value to the Choice, and volatile_read to retrieve it, making Choice something of a VolatileCell type, although even that doesn't provide a proper abstraction when used with references, and without Pin it may change locations with moves.
Some other options include cmov-like predication instructions as well as #96


Expressions in ORDER BY

Description
I can use a "correlation name" in ORDER BY:
SELECT DATE_TRUNC('minute', arrow_cast("EventTime", 'Timestamp(Second, None)')) AS M, COUNT(*) AS "PageViews" FROM 'hits.parquet' WHERE "CounterID" = 62 AND "EventDate" >= '2013-07-14' AND "EventDate" <= '2013-07-15' AND "IsRefresh" = 0 AND "DontCountHits" = 0 GROUP BY DATE_TRUNC('minute', arrow_cast("EventTime", 'Timestamp(Second, None)')) ORDER BY M LIMIT 10 OFFSET 1000;
but not an expression:
SELECT DATE_TRUNC('minute', arrow_cast("EventTime", 'Timestamp(Second, None)')) AS M, COUNT(*) AS "PageViews" FROM 'hits.parquet' WHERE "CounterID" = 62 AND "EventDate" >= '2013-07-14' AND "EventDate" <= '2013-07-15' AND "IsRefresh" = 0 AND "DontCountHits" = 0 GROUP BY DATE_TRUNC('minute', arrow_cast("EventTime", 'Timestamp(Second, None)')) ORDER BY DATE_TRUNC('minute', arrow_cast("EventTime", 'Timestamp(Second, None)')) LIMIT 10 OFFSET 1000;
because it gives:
Error: Schema error: No field named "EventTime". Valid fields are m, "PageViews".
Note: this behavior might follow the SQL standard, but some engines,allow using expressions and aliases in every SQL clause.

For comparison, ClickHouse can process expressions and aliases successfully.


tsconfig skipLibCheck

I think the option for skipLibCheck should be set to true in tsconfig.json as when developing our libraries we probably don't care about issues with external libraries?

We can't assume that users of this library won't need to use external .d.ts files


runHTS.py does not work

When trying to run the example script I get the following error:
INFO:matplotlib.font_manager:font search path ['/opt/conda/lib/python3.6/site-packages/matplotlib/mpl-data/fonts/ttf', '/opt/conda/lib/python3.6/site-packages/matplotlib/mpl-data/fonts/afm', '/opt/conda/lib/python3.6/site-packages/matplotlib/mpl-data/fonts/pdfcorefonts']
INFO:matplotlib.font_manager:generated new fontManager
TypeError                                 Traceback (most recent call last)
 in
44 # NOTE: CVselect takes a while, so if you want results in minutes instead of half-hours pick a different method
45 ##
---> 46 myDict = hts(data2, 52, nodes, holidays = holidays, method = "FP", transform = "BoxCox")
47 ##
48 # This output is a dictionary of dataframes, so you can do any further analysis that you may want. It also allows you to plot the forecasts.
/opt/conda/lib/python3.6/site-packages/htsprophet/hts.py in hts(y, h, nodes, method, freq, transform, include_history, cap, capF, changepoints, n_changepoints, yearly_seasonality, weekly_seasonality, holidays, seasonality_prior_scale, holidays_prior_scale, changepoint_prior_scale, mcmc_samples, interval_width, uncertainty_samples, skipFitting, numThreads)
270         ynew = fitForecast(y, h, sumMat, nodes, method, freq, include_history, cap, capF, changepoints, n_changepoints,
271                            yearly_seasonality, weekly_seasonality, holidays, seasonality_prior_scale, holidays_prior_scale,
--> 272                            changepoint_prior_scale, mcmc_samples, interval_width, uncertainty_samples, boxcoxT, skipFitting)
273     ##
274     # Inverse boxcox the data
/opt/conda/lib/python3.6/site-packages/htsprophet/fitForecast.py in fitForecast(y, h, sumMat, nodes, method, freq, include_history, cap, capF, changepoints, n_changepoints, yearly_seasonality, weekly_seasonality, holidays, seasonality_prior_scale, holidays_prior_scale, changepoint_prior_scale, mcmc_samples, interval_width, uncertainty_samples, boxcoxT, skipFitting)
72                     growth = 'linear'
73                     m = Prophet(growth, changepoints1, n_changepoints1, yearly_seasonality, weekly_seasonality, holidays, seasonality_prior_scale,
---> 74                                 holidays_prior_scale, changepoint_prior_scale, mcmc_samples, interval_width, uncertainty_samples)
75                 else:
76                     growth = 'logistic'
/opt/conda/lib/python3.6/site-packages/fbprophet/forecaster.py in init(self, growth, changepoints, n_changepoints, changepoint_range, yearly_seasonality, weekly_seasonality, daily_seasonality, holidays, seasonality_mode, seasonality_prior_scale, holidays_prior_scale, changepoint_prior_scale, mcmc_samples, interval_width, uncertainty_samples)
140         self.component_modes = None
141         self.train_holiday_names = None
--> 142         self.validate_inputs()
143
144     def validate_inputs(self):
/opt/conda/lib/python3.6/site-packages/fbprophet/forecaster.py in validate_inputs(self)
147             raise ValueError(
148                 "Parameter 'growth' should be 'linear' or 'logistic'.")
--> 149         if ((self.changepoint_range < 0) or (self.changepoint_range > 1)):
150             raise ValueError("Parameter 'changepoint_range' must be in [0, 1]")
151         if self.holidays is not None:
TypeError: '<' not supported between instances of 'str' and 'int'

Try the solution in this link. It worked for me.
https://stackoverflow.com/questions/48450418/hts-prophet-holidays-issue/51400527#51400527

Open PR
#7

i am getting same error  while running runHTS.py. "TypeError: '<' not supported between instances of 'str' and 'int'"  even though i tried with keyword arguments
detailed error:

TypeError                                 Traceback (most recent call last)
 in
----> 1 myDict = hts(data2, 12, nodes,  method = "FP")
~\AppData\Local\Continuum\anaconda3\lib\site-packages\htsprophet\hts.py in hts(y, h, nodes, method, freq, transform, include_history, cap, capF, changepoints, n_changepoints, yearly_seasonality, weekly_seasonality, holidays, seasonality_prior_scale, holidays_prior_scale, changepoint_prior_scale, mcmc_samples, interval_width, uncertainty_samples, skipFitting, numThreads)
270         ynew = fitForecast(y, h, sumMat, nodes, method, freq, include_history, cap, capF, changepoints, n_changepoints,
271                            yearly_seasonality, weekly_seasonality, holidays, seasonality_prior_scale, holidays_prior_scale,
--> 272                            changepoint_prior_scale, mcmc_samples, interval_width, uncertainty_samples, boxcoxT, skipFitting)
273     ##
274     # Inverse boxcox the data
~\AppData\Local\Continuum\anaconda3\lib\site-packages\htsprophet\fitForecast.py in fitForecast(y, h, sumMat, nodes, method, freq, include_history, cap, capF, changepoints, n_changepoints, yearly_seasonality, weekly_seasonality, holidays, seasonality_prior_scale, holidays_prior_scale, changepoint_prior_scale, mcmc_samples, interval_width, uncertainty_samples, boxcoxT, skipFitting)
72                     growth = 'linear'
73                     m = Prophet(growth, changepoints1, n_changepoints1, yearly_seasonality, weekly_seasonality, holidays, seasonality_prior_scale,
---> 74                                 holidays_prior_scale, changepoint_prior_scale, mcmc_samples, interval_width, uncertainty_samples)
75                 else:
76                     growth = 'logistic'
~\AppData\Local\Continuum\anaconda3\lib\site-packages\fbprophet\forecaster.py in init(self, growth, changepoints, n_changepoints, changepoint_range, yearly_seasonality, weekly_seasonality, daily_seasonality, holidays, seasonality_mode, seasonality_prior_scale, holidays_prior_scale, changepoint_prior_scale, mcmc_samples, interval_width, uncertainty_samples)
140         self.component_modes = None
141         self.train_holiday_names = None
--> 142         self.validate_inputs()
143
144     def validate_inputs(self):
~\AppData\Local\Continuum\anaconda3\lib\site-packages\fbprophet\forecaster.py in validate_inputs(self)
147             raise ValueError(
148                 "Parameter 'growth' should be 'linear' or 'logistic'.")
--> 149         if ((self.changepoint_range < 0) or (self.changepoint_range > 1)):
150             raise ValueError("Parameter 'changepoint_range' must be in [0, 1]")
151         if self.holidays is not None:
TypeError: '<' not supported between instances of 'str' and 'int'

Same. Still not working for me. Already added keyword arguments as mentioned in #7 but still getting the same error.

Hi everyone, the solution I found that worked for me was to clone the repo and delete the backslash \ characters. Not sure what \ does in python


How can i add room key and user key for websocket connection?

Hi, I am trying to connect to an existing web server. I need to enter the user key and room key for websocket. How can I add?

Hi, Sorry for the late reply. I have a question. Can I ask about the existing web server in detail? Do you mean you want to add room key and user key for this server app?

@tkmn0 Hi!) Help please, I want to call to exact user, for this I have user id/key and room id/key. Now, where to enter them? In other words, how can I call to user B from users [A, B, C, D, E, ...]?


How to build a custom OTel receiver

I want to learn how to build a custom OpenTelemetry receiver.

Why

collect data from unsupported sources
learn how receivers work

Created a GitHub repo using the work at https://zmoog.dev/posts/how-to-build-a-custom-open-telemery-collector/

https://github.com/zmoog/custom-otel-collector

The OTel docs contains a detailed guide about how to build a (traces) receiver: https://opentelemetry.io/docs/collector/building/receiver/
Here is my journey.

Overview

Set up a Go module for the receiver
Design the config
Build the factory
Implement the receiver
Add the receiver to the collector
Running and debugging the receiver

I want to focus this note to building and running a working no-op receiver component, leaving all data model design to a dedicated note.

Set up a Go module for the receiver
In this note, I will scaffold a custom receiver for my solar panel inverter build by ZCS.
mkdir -p receiver/zcsreceiver
cd receiver/zcsreceiver

$ go mod init github.com/zmoog/custom-otel-collector/receiver/zcsreceiver
go: creating new go.mod: module github.com/zmoog/custom-otel-collector/receiver/zcsreceiver

cd ../..

go work use receiver/zcsreceiver

Design the config
Opening the collector's config.yaml file and start writing down the setting your ideal settings is a good starting point:
receivers:
  zcs: # this line represents the ID of your receiver

interval
Since the inverter data lives in the ZCS cloud, this receiver will periodically scrape the metrics from the ZCS cloud API.
At the moment we probably need at least an interval setting.
receivers:
  zcs: # this line represents the ID of your receiver
    interval: 1m

Authentication
The ZCS API requires authentication, so we probably need to also add dedicated options:
receivers:
  zcs: # this line represents the ID of your receiver
    interval: 1m
    auth:
      client_id: ${env:ZCS_CLIENT_ID}
      auth_key: ${env:ZCS_AUTH_KEY}

Filter
Finally, users may have one or more inverters, so we need to include an option to list the inverter serial numbers thay want to collect data from:
receivers:
  zcs: # this line represents the ID of your receiver
    interval: 1m
    auth:
      client_id: ${env:ZCS_CLIENT_ID}
      auth_key: ${env:ZCS_AUTH_KEY}
    thing_ids:
      - ${env:ZCS_THING_KEY}


Updates to correctly reference the default branch after renaming.

GitHub will automatically redirect links to the old branch name, however the URLs should eventually be updated once all of the repositories have been updated.

Signed-off-by: Wade Barnes<EMAIL_ADDRESS>

@swcurran, @andrewwhitehead, Please review, but do not merge.  I'll merge once the branch has been renamed.


Adding domain manually

Description
The submit button is unresponsive.

If I don't use the wild card, the toast for success appears, but then the modal doesn't close.

And when I go for adding the wild card, I cannot do it.

Note: currently https://demo.prem.ninja services are broken.

After restarting the server, I installed prem again and now it works on https://demo.prem.ninja on the first try, but domains are still not provisioned for services.

CC @sekulicd


Incorrect title is being submitted if it includes a semicolon

If you include a semicolon in the title, for example
Title; Title

It will only include the part before the first semicolon.

Interesting, are you allowed to do semicolons usually?

Semicolon works fine with the official Reddit app: https://www.reddit.com/r/test/comments/7zboc5/test_test_test/


Add support for throwable to Status

Mentioned in https://github.com/census-instrumentation/opencensus-java/pull/779#pullrequestreview-74281349.
Also, it would be nice to let the exporter send the stacktrace, too, at least in case of stackdriver trace (https://cloud.google.com/trace/docs/reference/v2/rpc/google.devtools.cloudtrace.v2#google.devtools.cloudtrace.v2.Span), not sure about other backends.

Definitely would be cool. We need to watch carefully where and which
parsing cost (of throwable) is added


country localized name is wong

country named east timor and the arabic name is not correct جمهورية الكونغو الديمقراطية ,the correct arabic name is تيمور الشرقية

please update this small issue

thanks

@nag4r Thank you for your contribution, Released in v2.0.27.


Bug fix/java pom unit test

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 6

Complexity increasing per file
==============================
- Ginger/GingerTest/POMs/JavaPOM.cs  5
- Ginger/GingerTest/POMs/POMsPOM.cs  2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 6

Complexity increasing per file
==============================
- Ginger/GingerTest/POMs/JavaPOM.cs  5
- Ginger/GingerTest/POMs/POMsPOM.cs  2

See the complete overview on Codacy