Not validating MeterSigGrp with @copyof

When using @copyof on <MeterSigGrp> I get a validation error as <MeterSigGrp> should have a @func and a <MeterSig> which would obviously break the idea of having @copyof.
Or am I missing something here?
<?xml version="1.0" encoding="UTF-8"?>
<?xml-model href="https://raw.githubusercontent.com/music-encoding/music-encoding/master/schemata/mei-all.rng" type="application/xml" schematypens="http://relaxng.org/ns/structure/1.0"?>
<?xml-model href="https://raw.githubusercontent.com/music-encoding/music-encoding/master/schemata/mei-all.rng" type="application/xml" schematypens="http://purl.oclc.org/dsdl/schematron"?>
<mei xmlns="http://www.music-encoding.org/ns/mei" meiversion="3.0.0">
    <meiHead>
        <fileDesc>
            <titleStmt>
                <title>MeterSigGrpTest</title>
            </titleStmt>
            <pubStmt/>
            <sourceDesc>
                <source>
                </source>
            </sourceDesc>
        </fileDesc>
    </meiHead>
    <music>
        <body>
            <mdiv>
                <score>
                    <scoreDef key.sig="3f">
                        <staffGrp>
                            <staffDef n="1" xml:id="P1" label="Voice" lines="5" clef.line="2" clef.shape="G">
                                <meterSigGrp xml:id="meterSigGrp1" func="mixed">
                                    <meterSig count="2+3" unit="8"/>
                                    <meterSig count="1" unit="8"/>
                                </meterSigGrp>
                            </staffDef>
                        </staffGrp>
                    </scoreDef>
                    <section>
                        <measure n="1">
                            <staff n="1">
                                <layer n="1">
                                </layer>
                            </staff>
                        </measure>
                        <staffDef n="1"><meterSig count="2" unit="2"/></staffDef>
                        <measure n="2">
                            <staff n="1">
                                <layer n="1">
                                </layer>
                            </staff>
                        </measure>
                        <staffDef n="1"><meterSigGrp copyof="#meterSigGrp1"/></staffDef>
                        <measure n="3">
                            <staff n="1">
                                <layer n="1">
                                </layer>
                            </staff>
                        </measure>
                    </section>
                </score>
            </mdiv>
        </body>
    </music>
</mei>

@rettinghaus: @copy can't be used on any element that requires certain attributes or content.  The requirements on <meterSigGrp> could be relaxed in order to allow the use of @copy.  But, the trade off is that the function and content of <meterSigGrp> could no longer be validated via simple parsing.  That is, the presence of @func and <meterSig> children would have to be enforced via Schematron.  Despite our frequent use of Schematron, I'm reluctant to start using it for "primary" parsing; that is, those aspects than encoders/developers should expect to be present in the basic schema without invoking an "outside" process.  I'm well aware that there's a conflict here, but for the work-around for any case where @copy can't be used can be handled by simply repeating the entity pointed to by @copy.  This isn't a "cop out", it's just making the encoding explicit.

@pe-ro: So if @copyof can't be used here the Guidelines IMHO should clearly state this.
(Why has <meterSigGrp> to be a member of att.common.anl in the first place?)

It doesn't have to be a member of att.common.anl.  Dropping its membership is the easiest solution.


net/smtp: How to include friendly-from, friendly-to addresses?

Hi, I am trying to include "friendly" email addresses like Bob Dole<EMAIL_ADDRESS>for example.
https://wordtothewise.com/2014/12/friendly-email-addresses/
However, net/smtp (or possibly Lotus Notes or Gmail) silently strips out the friendly portion. The actual, raw email in my inbox is addressed simply<EMAIL_ADDRESS>where I want Bob Dole<EMAIL_ADDRESS>I wonder if there a specific syntax required for getting friendly addresses through the net/smtp Mail(), Rcpt() methods. Digging into the implementation code, however, it looks like the original authors may not have accounted for friendly addresses.
https://golang.org/src/net/smtp/smtp.go#L246
Looks like net/smtp's validation here is rather limited. No validation is performed for the addresses, neither the formal address nor any friendly portion. No RFC5322 regex is applied by net/smtp. Rather, only CRLF checks are applied.
https://www.ietf.org/rfc/rfc5322.txt
In any case, the email address that the Go user passes to Mail() or Rcpt() is automatically wrapped in angle brackets, mangling any friendly portion. At which point, Lotus Notes or Gmail seems to simply sanitize the address, dropping the friendly part entirely.
Third party libraries like https://github.com/emersion/go-smtp have copied and pasted code from net/smtp, and so have similar runtime behavior, unfortunately.
Can we please update the Mail(), Rcpt() code to account for optional friendly from and to addresses?
I realize that at this point in time, by having always wrapped the addresses in angle brackets, Go users will expect that to continue at least until Go v2 releases. So any patch to these methods can't simply pass the address through as written.
An updated implementation accounting for friendly addresses, would involve first splitting the argument into friendly and formal addresses, check for the existence of a friendly part, and only then insert angle brackets around the formal address, when not already present. And again, RFC5322 validation should probably be performed around each formal email address, in order to catch mistakes earlier.
Anybody know a third party library that does this better, instead of hardcoding angle brackets around these addresses? The current winner on Stack Overflow bypasses these methods and just writes the entire email as a byte array :P
https://stackoverflow.com/a/46369988/10251641

As far as I know the SMTP level just uses e-mail addresses without comments.  When you talk about "friendly" e-mail addresses I think you are talking about the contents of the message, which is not the same as SMTP.  SMTP is the transport layer that carries the message.  Don't confuse the SMTP MAIL FROM and RCPT TO commands with what the user will actually see.
I'm going to close this issue because there is nothing to do.  If you want to discuss this further, I recommend that you use a forum; see https://golang.org/wiki/Questions.

Thank you Ian.
Ah, I think I understand. I will try the From: ... syntax in the message body rather than the top level headers. Would be nice for the net/smtp library to provide for this out of the box. Anyway, I will give this manual way a try.


Prepare for Renovate with reusable workflows

Q
A

QA
yes

Description
Adding Renovate to Laminas, Mezzio repositories. https://gist.github.com/weierophinney/49ca8ae2b916d898dd3d4162b26ff41b

Configure minimum supported PHP version
Update composer dependencies
Update to reusable Continuous Integration workflow
Update to reusable Automatic Releases workflow

I'd say its fine with 2.0.x
It does support PHP 8.1 already and is used in recent versions (as 2.0.x-dev) to provide PHP 8.1 support.
It's time to create a 2.0 release here.
This is only meant to be a dev-dependency for storage adapters and thus it should be easy to update.

Thanks, @ghostwriter


⚠ Bofrost.at has degraded performance

In e492355, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1217 ms

Resolved: Bofrost.at performance has improved in 6b5fef6 after 6 minutes.


Dynamic Configuration

What is the feature request? What problem does it solve?
Currently, administrators of VDK Control Service configure data jobs using either:

a configuration type of plugin of VDK SDK
Update Control Service configuration with the helm chart vdkOptions value then upgrade the helm release and then re-deploy all jobs that need the new configuration.
Configuration can be set per data job, for all data jobs.
It's a requirement that job X sees only configuration for job X (or not for Y)

It look like this:
We store the configuration in k8s secrets resource and mount it in the Control Service  on deploy of job set them as environment variables:

Issues with those approaches are:

The first approach does not allow for configuration-sensitive configuration (passwords)
The second approach is very long and error-prone. It may take hours to fully re-deploy 1000 jobs for example
Local and "cloud" (deployed) job configuration may differ

The main issue we want to fix with this change is the time to configure .
Suggested solution
While the plans are for https://github.com/vmware/versatile-data-kit/issues/832 to solve this fully.
Having a full-blown System Config API is still a lot of work.
The proposal here is to do much smaller change that would enable administrators to configure jobs much quicker (without needing to redeploy each job):
The operator would still edit the helm chart values.yaml, with a slightly different format.
The key is the job name or special keys like "all" which donates all jobs
And the value is the same format as config.ini of a data job
vdkOptions:
    job-1: |
       [vdk]
       VDK_CONFIG = one
    job-2: |
       [vdk]
      VDK_CONFIG = two
   all: |
       [vdk]
       VDK_CONFIG = three

Out of these using helm templating language (https://helm.sh/docs/chart_template_guide/control_structures/#looping-with-the-range-action) the following would be created
apiVersion: v1
kind: Secret
metadata:
  name: vdk-options-secrets
type: Opaque
data:
  job-1: "configuration ini file of job 2 in base64"
  job-2: "configuration init file of job 2 in base64"
  all: "configuration ini file for all jobs in base64"

Then this would be mounted on the CronJob (hence Job/Pod ) of the Data Job
spec:
  template:
    spec:
      containers:
      - name: job-1
        image: job-image
        volumeMounts:
            - name: secrets
              mountPath: "/etc/secrets"
              readOnly: true
      volumes:
        - name: secrets
          secret:
            secretName: vdk-options-secrets
            items:
              - key: job-1
                path: vdk-options.ini
              - key: all
                path: vdk-options-all.ini
            optional: true

Thos would mount the vdk options.ini if it exists or it's mapped as empty file if not (thanks to optional=True)
And finally, we extend vdk run to be able to support multiple configuration files
export VDK_CONFIG_FILES="/etc/secrets/vdk-options.ini;/etc/secrets/vdk-options-all.ini"

VDK_CONFIG_FILES would have higher priority that user provided job-specific config file.
Additional context
Add any other context or screenshots about the feature request here.
I think this could server as incremental step toward https://github.com/vmware/versatile-data-kit/issues/832 as then we can create a API logic that would set those through an API instead of through helm.

Thanks for the proposal I like it. The actual configuration files will probably be stored in the git repository through the gitcrypt or something similar and then passed to the helm chart through our CI/CD. Is that correct?

Thanks for the proposal I like it. The actual configuration files will probably be stored in the git repository through the gitcrypt or something similar and then passed to the helm chart through our CI/CD. Is that correct?

That's not technically part of the proposal since it depends on the deployment. But yes, keeping the configuration in gitcrypt is a good option. Possibility is to use Hashicorp Vault as well .

Triaged. We will keep it, as being an important feature that we would like to address.


new: update plugins

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If contributing rules or changes to rules, please make sure to also uncomment one of the following line:

/kind rule-update

/kind rule-create

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area engine

/area rules

/area tests

/area proposals

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

update: updated plugins to latest stable released version

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: FedeDP
To complete the pull request process, please assign jasondellaluce after the PR has been reviewed.
You can assign the PR to them by writing /assign @jasondellaluce in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.32.0

/milestone 0.32.0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: afd84a5e451a6aca0d7976fbedf8f4d3a2cfe2a9

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, jasondellaluce, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Data integrity crash

ReActiveAndroid Version: 1.4.3
Bug or Feature Request: Got a data integrity crash after updating data model and database version
Description:
java.lang.IllegalStateException: ReActiveAndroid cannot verify the data integrity. Looks like you've changed schema but forgot to update the version number. You can simply fix this by increasing the version number.
        at com.reactiveandroid.internal.database.ReActiveOpenHelper.checkIdentity(ReActiveOpenHelper.java:71)
        at com.reactiveandroid.internal.database.ReActiveOpenHelper.onOpen(ReActiveOpenHelper.java:51)
        at android.database.sqlite.SQLiteOpenHelper.getDatabaseLocked(SQLiteOpenHelper.java:411)
        at android.database.sqlite.SQLiteOpenHelper.getWritableDatabase(SQLiteOpenHelper.java:298)
        at com.reactiveandroid.internal.database.DatabaseInfo.getWritableDatabase(DatabaseInfo.java:65)
        at com.reactiveandroid.ReActiveAndroid.getWritableDatabaseForTable(ReActiveAndroid.java:131)
        at com.reactiveandroid.internal.utils.QueryUtils.execSQL(QueryUtils.java:27)
        at com.reactiveandroid.query.ExecutableQueryBase.execute(ExecutableQueryBase.java:21)

I deleted a field from my model, and incremented the database version:
@Database(name = "propertyforce", version = 19)
public class AppDatabase {

It was 18 before.  Here's how I'm initializing:
DatabaseConfig appDatabase = new DatabaseConfig.Builder(AppDatabase.class)
                        .addModelClasses(LogEntry.class, FarmReport.class, FarmReportProperty.class)
                        .disableMigrationsChecking() // Potentially destructive, so replace with migrations if adding any tables that need to persist
                        .addTypeSerializers(JsonSerializer.class)
                        .build();
                ReActiveAndroid.init(new ReActiveConfig.Builder(context)
                        .addDatabaseConfigs(appDatabase)
                        .build());

So I shouldn't need any migrations.  Why is this happening?

Same page here, increasing the version number is not working at all. Thanks for helping.


Port the functions humanize_bytes

We want to port the function humanize_bytes from the mack project.
Here is a link to the function: https://github.com/MrPowers/mack/blob/cf250e29fa366cd7a0d31a0bc55edb1c74ebbd75/mack/init.py#L521
The purpose of this helper function is to make large numbers of bytes on delta tables human readable.

Commenting for reference of this other issue.


Add parent_controller configuration option

This uses https://github.com/thoughtbot/clearance/pull/784 as a starting point and then also:

Does a rebase against changes in master since then;
Makes small changes to avoid some deprecation warnings that would not be generated at the time the original PR was opened, around class names vs strings of class names

this is awesome!


Criterion <IP_ADDRESS> not producing HTML reports

With criterion <IP_ADDRESS>, with the benchmark run as (in a cabal sandbox) I get:
> dist/build/bench/bench --no-gc -o bench.html
benchmarking Write to ByteString/Binary
...
...
bench: TemplateNotFound "default"

There also seems to be no entry in the changelog.md file for the <IP_ADDRESS> release.

I haven't been able to reproduce the particular error you're seeing (TemplateNotFound "default") with or without a sandbox, but this is with my own code. Do you mind sharing the code that you're trying to run?
Separately, we should also consolidate all the changelog entries after <IP_ADDRESS> into <IP_ADDRESS>, since there weren't any Hackage releases between those version numbers.

git clone https://github.com/erikd/fastpack
cd fastpack
git co 52bbe547aeafb4b41 -b test
cabal sandbox init
cabal install --dependencies-only --enable-benchmarks
cabal configure  --enable-benchmarks
cabal build
dist/build/bench/bench -o bench.html

With that procedure I get TemplateNotFound every time.

Hrm, I've tried that on two Ubuntu 14.04 machines (one in a Docker image) with GHC 8.0.1 / cabal-install-<IP_ADDRESS>, but couldn't reproduce it.
It's a bit baffling that this happens, since templates/default.tpl should always just be loaded, since it's listed as a data-file in criterion.cabal.
For clarification, when you say you get this error occasionally, does the error happen when you run dist/build/bench/bench -o bench.html successively after installing it once? Or does it require reinstalling the sandbox to trigger the non-determinacy?

This seems to be related with whether --no-gc is added to the command line when I run the benchmark. Adding is seems to make bad things happen (silently).

Bah, figured it out. I sometimes use a custom template and the terms in the default templates that are replaced had changed.
     <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
     <title>criterion report</title>
     <script language="javascript" type="text/javascript">
-      {{#include}}js/jquery-2.1.1.min.js{{/include}}
+      {{js-jquery}}
     </script>
     <script language="javascript" type="text/javascript">
-      {{#include}}js/jquery.flot-0.8.3.min.js{{/include}}
+      {{js-flot}}
     </script>
     <script language="javascript" type="text/javascript">
       {{#include}}js/jquery.criterion.js{{/include}}

Would be nice if there was an actual error or warning printed when the template replacement failed.

You mentioned that --no-gc was an important part of the equation in https://github.com/bos/criterion/issues/126#issuecomment-260777656, but in https://github.com/bos/criterion/issues/127#issuecomment-260884000 you seemed to be doubting that claim.
Also, even after trying out --template (as you recommended in https://github.com/bos/criterion/issues/127#issuecomment-261043624), I still am unable to reproduce this issue. Is this still affecting you? If so, can you give me some guidance on how one might trigger it?

I can't right now. I'll try some more after I work.

I've managed to re-produce this problem. Might as well close it.


Make daily pull request per 6 hours

The deployment in getrector.org is already per-6 hours: 00:05, 06:05, 12:05, 18:05.
This PR update the daily Update Rector Dependencies per 00:00, 06:00, 12:00, 18:00.
The 5 minutes different is to ensure PR merged first.

All checks have passed 🎉 @TomasVotruba it is ready for review.


Fix clean commands not work.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks! I forgot about this functionality.


Wireframes for Editing Color Correction/Mosaic

The current editing interface is going to move to a more unified browse UI based on the projects workflow that is half-implemented.
Some basic design notes/wireframes are needed so that we can implement it.

This should include a workflow for creating, editing, viewing user defined color modes (and by extension, selecting bands for greyscale tiles)

This was done I believe


Not Triggering Firefox ExternalInterface Callback

I'm not sure of the best way to get you a minimal example to demo this, but embedding a Flash movie in Firefox doesn't seem to trigger ExternalInterface callbacks, nor can I call exposed methods from the Flash movie.
The SWF I'm using is from here: https://github.com/mangui/flashls/blob/dev/bin/debug/flashlsChromeless.swf
I have a component with the following relevant methods:
componentWillMount() {
    window.VideoPlayer = window.VideoPlayer || {};
    window.VideoPlayer[this.props.video.id] = {
        callback: _.bind(this.flashCallback, this)
    };
}

componentDidMount() {
    this.$plugin = this.refs.flash.getFPDOMNode();
}

render() {
    let vars = {
        callback: `window.VideoPlayer[${this.props.video.id}].callback`
    };

    return (
        <ReactSWF
            allowScriptAccess="always"
            flashVars={ vars }
            id={ `videoPlayer_${this.props.video.id}` }
            key={ `videoPlayer_${this.props.video.id}` }
            ref="flash"
            src="/assets/flash/VideoPlayer.swf"
            wmode="transparent"
        />
    );
}

flashCallback(event, args) {
    this.log.debug({ args }, `Flash Player :: ${event}`);

    if (_.isFunction(this[`on${eventName}`])) {
        this[`on${eventName}`].apply(this, args);
    }
}

This all works correctly in Chrome at the very least. Firefox doesn't seem to ever call the callback, nor can I call this.$plugin.methodOnFlashMovie() like I would expect.

I have a project myself in which I rely heavily on ExternalInterface and it works across all browsers, including FireFox, and there is no special code in ReactSWF for ExternalInterface other than getFPDOMNode() for returning a reference to the actual node.
So all I can really say at this point is that it's probably something on your end... perhaps you could check that ExternalInterface.available really is true? Could it be that you're adding the callbacks before Event.ADDED_TO_STAGE has occured?

Alright, I'll see if I can pull the SWF source and try debugging it. I know that the SWF works in Firefox from here: http://www.flashls.org/latest/examples/chromeless/ but it seems to be using the old <object><embed /></object> style of adding flash embeds.

:rage: so it seems that Firefox doesn't like loading flash files from localhost. I deployed to our development server and it loads just fine.
Sorry for the false alarm

No problem :)
You may be able to work around that in the SWF by allowing script requests from all hosts (*) via some system function somewhere. But I'm not entirely sure.


chore: remove minimal and workflow images

Linked PR

https://github.com/Substra/substra-tools/pull/86 (main)
https://github.com/Substra/substra-tests/pull/284
https://github.com/owkin/substra-ci/pull/129
https://github.com/Substra/substra-backend/pull/736
https://github.com/Substra/substra/pull/389

CI ✅
Doc + SDK -> https://github.com/owkin/substra-ci/actions/runs/6250601543
MNIST -> https://github.com/owkin/substra-ci/actions/runs/6250262317
SubstraFL + Camelyon -> https://github.com/owkin/substra-ci/actions/runs/6249659677
closes FL-1214

The plan is to remove those images, then to add a new one based on python-slim and rename the current one "cuda" or someting?

The first plan was to simplify the current docker images. Then the management of the python based one can be done here or directly on substrafl. This is to be defined.


Haystack uses obsolete methods for working around old bugs.

Haystack uses some strange methods to avoid old bugs in Haystack. It has been through several revisions and this code is not relevant. The old methods are affecting SOLR updates in asynchronous mode.

@hyi you had mentioned HayStack upgrades possibly fixing other problems you can link your issues here or comment

@cuahsimarko It turns out we are using the latest HayStack version 2.8.1 that works with our current outdated django version 1.11. The next HayStack version only works with Django 2.2 or above, so upgrading HayStack is not an option for us. But it appears discovery actually works in my local development environment now although I cannot explain what I did to make it work.

@hyi Given that HayStack cannot be upgraded and discovery is working, should this issue be closed?

@Castronova I'll defer this to @alvacouch to see if this issue can be closed or not.


[Alignment]

How can we reproduce this bug?
Write out the HTML (or Inky code) that causes the issue.
What did you expect to happen?
What happened instead?
What email clients does this happen in?

@DanielRuf would it be possible to get this issue closed? I think it is a duplicate of #924


Storage client change to support multipart in existing version

Changed the method in storage client file for multipart feature to  worked with existing 2.3.0 storage client version


17979 - Countries of citizenship control

*Issue:*https://github.com/bcgov/entity/issues/
Description of changes:

adding countries of citizenships control
basic component tests

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namex license (Apache 2.0).

will add more cypress tests in the main app today

/gcbrun

Temporary Url for review: https://bcregistry-btr-dev--pr-47-2s8uilgf.web.app

Looks good to me, just needs the lint fix

/gcbrun

Temporary Url for review: https://bcregistry-btr-dev--pr-47-2s8uilgf.web.app


fix(errors): flatten sdk problem chains to reduce hash complexity

Previously, any SDKProblem instance could be used as the "caused by" for another SDKProblem instance and its ID would become a part of the hash. This meant that problems created between service SDKs and the Go SDK Core would have a lot of complexity - the error scenarios tracked would be unnecessarily granular and numerous.
This change updates the system to maintain one problem scenario per context (where a context is either HTTP, SDK, or Terraform) rather than per every component within a given context.
It also provides for keeping knowledge of problem scenarios originating in the core for easier debugging and easier attaching of HTTP errors to service SDK problems.

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


Redux Store Constructs for Every Test

Our React-Redux App uses create-react-app. We’re trying to back-apply Jest/Enzyme/redux-mock-store to a large code base. Problem is that for every test, Jest loads src/store.js where the redux store is created to its initial state. I’d like to suppress all but the first iteration of that construction. Or, supply the mock instead of letting  src/store.js use the reducers.
This is killing me. What in Jest, create-react-app, or ??? initiates this and how to override? Any suggestions?

Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions. You'll find more on Jest docs help page. Thank you :)


Widget type UI display with customization option

Suggested by Giuseppe Pappalardo.
The major refactoring needed is to transition all the way to RiverPod and restructure so each tile would be a stream filtered by RiverPod. RiverPod would be also needed to tackle #333 and #127. Probably I'll fix #333 first.
We have several other planned UI changes which could be somewhat conflicting. We need to decide if we should aim right away for a full widget type UI (such as GymTrakr or Wahoo):

#228
#242
#278
#279

The first two tickets would be covered if the user doesn't configure the related tiles to show. The average ticket is covered by allowing average tiles for each non cumulative measurements. We'd also need to think how the graphs would show. Would the tiles be per row, or rather rectangular ones which are half row width only. This tile system would also solve landscape layouts which is right now pretty rigid: two columns of fixed list measurements.
If the tiles are row width then graphs can function as they are right now: by being an accordion. If not then maybe we'll return to the ancient way and there would be only one dedicated graph spot? Or maybe we'd allow graph widgets which span two spaces and take up a row?

This would go against a fixed line-by-line type setting which has the value and the average on the same row #278


Allow disabling of CORS

It is desirable to disable CORS for development purposes in order to
validate same-origin policy and web security concepts. Additionally,
json-server could be used to show proxy concepts.
This change adds a new CLI option --no-cors which disables CORS by not
adding the CORS middleware. The change is backwards compatible since the
default behavior, i.e. adding CORS headers, is retained.

Thank you :)


⚠ RbxVeri has degraded performance

In d3c78fc, RbxVeri (https://verify.triza.dev) experienced degraded performance:

HTTP code: 200
Response time: 2830 ms

Resolved: RbxVeri performance has improved in c16542d.


C# 11 Raw string literals not work in Blazor @code functions

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
It show me syntax error.

Expected Behavior
No syntax error.
Steps To Reproduce
No response
Exceptions (if any)
No response
.NET Version
.NET 7
Anything else?
No response

This is likely a duplicate of https://github.com/dotnet/razor/issues/7084.


Closing a websocket connection cleanly

It is currently impossible to close the websocket connection normally. It should be done according to this spec, which is the version that browsers with current support implement:
To close the connection cleanly, a frame consisting of just a 0xFF
byte followed by a 0x00 byte is sent from one peer to ask that the
other peer close the connection.

For now, the js client will force connections to terminate.

I think @0x009922 originally introduced this issue so we might ask him if more details are needed.

The issue seems to be that on our server side in handle_subscription we don't handle Close message.


Normalization is dropping third 0 digit

Important fact: Phony only works with international phone numbers. It needs the country code to be able to identify the number's country and rules.
What I did
Phony.normalize('(330) 458-1111')
Phony.normalize('(700) 458-1111')
What happened
=> '334581111'
=> '7004581111'
What I expected to happen and why
=> '3304581111'
=> '7004581111'

Did you see the important fact?
On Friday, 5 May 2017, dbeqiraj<EMAIL_ADDRESS>wrote:

Important fact: Phony only works with international phone numbers. It
needs the country code to be able to identify the number's country and
rules.
# What I did
Phony.normalize('(330) 458-1111')
Phony.normalize('(700) 458-1111')

What happened
=> '334581111'
=> '7004581111'
*
# What I expected to happen and why
=> '3304581111'
=> '7004581111'
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/floere/phony/issues/373, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAALzvty-5sqjhElA-cqEVY20mtcrITTks5r2z3HgaJpZM4NSDmF
.

No :P
So... nothing to do, right?

Probably not 😊 Unless there's something else?
On Friday, 5 May 2017, dbeqiraj<EMAIL_ADDRESS>wrote:

No :P
So... nothing to do, right?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/floere/phony/issues/373#issuecomment-299499027, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAALzhgcy4yq-v5YNIVmTGG-ZEefzKuTks5r20KcgaJpZM4NSDmF
.


Pager overlapping items if SurfaceView is involved

Describe the bug
After migrating to the new, LazyList based pager (0.19), all of our SurfaceView content is broken. As you scroll, items are often drawn one on top of the other. Note that

it was working before LazyList migration
we're not using contentPadding / itemSpacing, not a duplicate of #793
only happens with SurfaceView. I have rewritten our rendering logic to optionally use TextureView and in that case everything works fine.

To Reproduce
Checkout this branch: https://github.com/natario1/Playground/tree/compose-pager-sv
It uses GLSurfaceView and a random GLES cube renderer that I found online for simplicity, but the issue exists with simple SurfaceView as well, in video playback for example.
I don't have enough knowledge to understand whether this is a pager issue or a LazyList issue. I have included both in the sample app so that you can play with it.
Expected behavior
Surfaces are drawn at the correct position.
Screenshots?
See how the previous item (blue background) is partially drawn on top of the current item (red background).

Environment:

Android OS version: API 30, but likely all of them
Device: Pixel 4, but likely all of them
Accompanist version: 0.19

Before this issue is automatically closed, I just want to add that I am also affected by this bug. I use Exoplayer in a HorizontalPager.

FWIW I spent many hours on this and I think it might be caused by this https://issuetracker.google.com/issues/204328111 - also reported by me, also took days to spot and diagnose, also no answer 😄 in the end I just rewrote pager from scratch to match our use case and it was way simpler than debugging this stuff.

Yes, it is the same issue as https://issuetracker.google.com/issues/204328111. I will close this one, please follow the main bug


Refactor handle process

Refactor handle process and update collection handles

closing ticket, decision to change /collection/ to /object/


Refactor Captcha widget and write tests.

Q
A

Is bugfix?
no

New feature?
no

Breaks BC?
yes/no

Tests pass?
yes

Fixed issues

Suggest all the logic associated with the generation of images render in drivers

Generally the changes are good from my point of view.

Test for performents
Code test.
require '/www/yii2/framework/Yii.php';

use yii\captcha\Captcha;

$times  = [];
for ($i = 0; $i < 100000; $i++) {
    $startTime = microtime(true);
    Captcha::checkRequirements();
    $endTime = microtime(true);
    $times[] = $endTime - $startTime;
}

echo 'Time   AVG:' . array_sum($times) / count($times);

old code
Time   AVG: 0.00036538505792618
new code (used model)
Time   AVG: 0.00047585074424744

@vladis84 seems it's not finished. Right? According to your diagram there should be an ImageRendererInterface such as:
interface ImageRenderererInterface
{
    public function renderImage($code); // proabably need more options such as width and height in the signature
}

And places such as https://github.com/vladis84/yii2/blob/40cf0dd7da4e23582a9a844af8a30bc43d09f8d7/framework/captcha/CaptchaAction.php#L254 should be moved into GdRenderer and ImagickRenderer.

I suggest two version
1 make a module from captcha
2 fix the code using the factory and composition (write diagram)
I consider the first correct

What's on the diagram is correct. We don't really need Yii's module for that if you've meant it and not module as a set of related classes...

Okay. I implement the code from the diagram

Pull request close?

Hello.
Where can be used yii\captcha\Captcha::checkRequirements()) except yii\captcha\CaptchaAction?

@samdark @SilverFire
Refactor Captcha widget is ready.

Not pass test for php7.1.
Error: Class 'ImagickPixel' not found

@samdark
Part of the remarks fixed.

PHP Fatal error:  Call to undefined function yii\captcha\drivers\imagettfbbox() in /home/travis/build/yiisoft/yii2/framework/captcha/drivers/GdDriver.php on line 41
http://stackoverflow.com/questions/7290958/php-fatal-error-call-to-undefined-function-imagettftext

@samdark
Part of the remarks fixed.

I think the design overall is now good. Need additional reviews. After these are done I can work on polishing naming and docs.

I consider the refactoring of such scale can not be accepted at 2.0.x - it should be moved to 2.1 milestone.

Since this has moved to 2.1 and already contains some more complexity and multiple components, captcha would be a candidate to move out of the core into an extension. Opinions?

https://github.com/yiisoft/yii2/pull/13839#discussion_r107698500

@cebe yes, that's the plan.

Migrated to #14632


Downloads

Use this issue to discuss Downloads in the ONS Design System

An improvement for this component would be to:
Add tabindex="-1" for the image thumbnail link.
We have an aria-hidden="true" attribute, which is great because we don’t want screen readers reading out the link twice.
However a keyboard user would still have to tab twice for the thumbnail link and the heading link.
They do the same thing. Adding tabindex="-1" will resolve this.

An improvement for this component would be to:
Add tabindex="-1" for the image thumbnail link.
We have an aria-hidden="true" attribute, which is great because we don’t want screen readers reading out the link twice.
However a keyboard user would still have to tab twice for the thumbnail link and the heading link.
They do the same thing. Adding tabindex="-1" will resolve this.

Sounds good Ive raised a PR to fix this

We need to update the thumbnail path to allow for a smallPath and a largePath.
Currently the macro expects a small and large folder (hardcoded into macro). However, with the way Craft does image modifications it names the folders with an underscore.
Allowing for a smallPath and a largePath would sort this issue out and also make the component more flexible.

We need the ability to have a fallback image if no image specified as part of the component.


(Bug) Step 1: selected strategy isn't kept after refreshing or moving back/forward

Closes #1170

Black Duck Security Report
Merging #1173 into integration/#1082-new-design will not change security risk.
Click here to see full report

Pull Request Test Coverage Report for Build 3254

8 of 20 (40.0%)  changed or added relevant lines in 6 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 22.725%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/components/Common/CrowdsalesList.js
0
1
0.0%

src/components/Crowdsales/index.js
0
1
0.0%

src/components/StepOne/index.js
2
12
16.67%

Files with Coverage Reduction
New Missed Lines
%

src/components/StepOne/index.js
1
50.0%

Totals

Change from base Build 3205:
-0.02%

Covered Lines:
1042

Relevant Lines:
4014

💛  - Coveralls

@mariano-aguero it seems, a deploy to Netlify failed ☝

@vbaranov

it seems, a deploy to Netlify failed

The netlify API failed, now is functional
Regards


Update the import paths for Caddy

This pull request fixes #18 by making caddy-net compatible with Caddy v1.0.1+ (see epicagency/caddy-expires#6 for reference) by updating the import paths for Caddy.

Thanks @whalehub


Free PLP token early when processing bulk load requests

Description
ParameterToken is allocated to read incoming PLP  data.  While token itself is freed promptly, the PlpData records inside it are not freed. Also the token itself can be leaked when the record is null.
Backend memory usage importing 2 million records (half of them are nulls, see details in linked issue), without the patch:

With patch applied:

Issues Resolved
#2462
Test Scenarios Covered
There are no functional changes, so no new tests.
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.
Signed-off-by: Alex Kasko<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 11 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 72.531%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

contrib/babelfishpg_tds/src/backend/tds/tdsbulkload.c
0
11
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
41114

Relevant Lines:
56685

💛  - Coveralls

We should merge this fix on BABEL_2_X_DEV and BABEL_3_X_DEV


currentResX/Y seems incorrect

My goal is to differentiate between native resolution of a screen and it's scaled current resolution:
$ node
Welcome to Node.js v12.10.0.
Type ".help" for more information.
> const si = require('systeminformation');
undefined
> si.graphics().then(data => console.log(data))
Promise { <pending> }
> {
  controllers: [
    {
      vendor: 'Intel',
      model: 'Intel Iris Plus Graphics 650',
      bus: 'Built-In',
      vram: 1536,
      vramDynamic: true
    },
    { vendor: '', model: '', bus: '', vram: -1, vramDynamic: false }
  ],
  displays: [
    {
      vendor: '',
      model: 'Color LCD',
      main: false,
      builtin: false,
      connection: '',
      sizex: -1,
      sizey: -1,
      pixeldepth: 30,
      resolutionx: 2560,
      resolutiony: 1600,
      currentResX: 2560,
      currentResY: 1600,
      positionX: 0,
      positionY: 0,
      currentRefreshRate: -1
    },
    {
      vendor: '',
      model: 'LG QHD',
      main: true,
      builtin: false,
      connection: 'DisplayPort',
      sizex: -1,
      sizey: -1,
      pixeldepth: 30,
      resolutionx: 2560,
      resolutiony: 1440,
      currentResX: 2560,
      currentResY: 1440,
      positionX: 0,
      positionY: 0,
      currentRefreshRate: -1
    }
  ]
}

The resolution on my macbook's retina screen is 1680x1050 -- I don't think it's possible to get 2560x1600 native resolution on the 13.3" screen.
If we're just using system_profiler SPDisplaysDataType here, it won't provide the scaled resolution we're looking for.

Fortunately (as you have a Retina display) your MacBook Pro really has a native screen resolution of 2560 x 1600. See also https://www.apple.com/macbook-pro-13/specs/ ... under "Display" in the first text line you should see "... 2560‑by‑1600 native resolution ...". Also on your Mac, in the apple menu - About my Mac - in the display tab you should also see this. So I also tested it here on my machine and it also shows the correct resolution. Closing it here.

@sebhildebrandt fwiw, a reliable way to get scaled resolution is this: https://wiki.libsdl.org/SDL_GetCurrentDisplayMode
If you don't want to use SDL, I am sure you can look at the source of how they accomplish it.


Accessing Map values crashes React app, even after .toJS()

Packages: Redux (3.0.0), React (15.2.0), React-Redux (4.0.0), and Immutable (3.8.1)
What happened
When trying to access any methods on an Immutable.Map that's been passed down via Props from my Redux store, my app crashes.  I'm using hot-reloader, and as soon as it re-builds, my console spits out 2 warnings and an error:
warning.js:44 Warning: There is an internal error in the React performance measurement code.  It should be a string (for DOM elements) or a ReactClass (for composite components).warning @ warning.js:44createElement @ ReactElementValidator.js:166renderError @ main.js:66render @ main.js:72(anonymous function) @ main.js:75hotApply @ bootstrap ad5d90a…:518cb @ process-update.js:52hotUpdateDownloaded @ bootstrap ad5d90a…:334hotAddUpdateChunk @ bootstrap ad5d90a…:306webpackHotUpdateCallback @ bootstrap ad5d90a…:27(anonymous function) @ 0.2552c86….hot-update.js:1
process-update.js:115 [HMR] Cannot check for update (Full reload needed)
process-update.js:116 [HMR] Invariant Violation: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: object.
    at invariant (http://<IP_ADDRESS>:3000/vendor.ad5d90acbf535202d2ce.js:1393:16)
    at ReactCompositeComponentWrapper.instantiateReactComponent [as _instantiateReactComponent] (http://<IP_ADDRESS>:3000/app.ad5d90acbf535202d2ce.js:19109:103)
    at ReactCompositeComponentWrapper.performInitialMount (http://<IP_ADDRESS>:3000/app.ad5d90acbf535202d2ce.js:19562:23)
    at ReactCompositeComponentWrapper.mountComponent (http://<IP_ADDRESS>:3000/app.ad5d90acbf535202d2ce.js:19436:22)
    at Object.mountComponent (http://<IP_ADDRESS>:3000/app.ad5d90acbf535202d2ce.js:11586:36)
    at mountComponentIntoNode (http://<IP_ADDRESS>:3000/app.ad5d90acbf535202d2ce.js:24688:33)
    at ReactReconcileTransaction.perform (http://<IP_ADDRESS>:3000/app.ad5d90acbf535202d2ce.js:12607:21)
    at batchedMountComponentIntoNode (http://<IP_ADDRESS>:3000/app.ad5d90acbf535202d2ce.js:24709:16)
    at ReactDefaultBatchingStrategyTransaction.perform (http://<IP_ADDRESS>:3000/app.ad5d90acbf535202d2ce.js:12607:21)
    at Object.batchedUpdates (http://<IP_ADDRESS>:3000/app.ad5d90acbf535202d2ce.js:21773:20)

Uncaught TypeError: Cannot read property '__reactInternalInstance$4heywo6qdln2sth55h7inyu8fr' of nullgetClosestInstanceFromNode @ ReactDOMComponentTree.js:107findParent @ ReactEventListener.js:39handleTopLevelImpl @ ReactEventListener.js:68perform @ Transaction.js:138batchedUpdates @ ReactDefaultBatchingStrategy.js:63batchedUpdates @ ReactUpdates.js:98dispatchEvent @ ReactEventListener.js:150
After that, reloading yields the following warnings/errors:
Warning: There is an internal error in the React performance measurement code.
However, console.log(Map) works fine, and the console indicates that it's a regular Immutable.js map.
What's even more bizarre, I think, is that the error happens even if I try the following:

invoke .toJS() and try to access the object properties via normal JS. Interestingly, no matter where in the chain from reducer to console.log() I do this, it still has the same result.
Assign each property to a new JS object. eg,

let tempObj = {
propA: Map.get('propA'),
propB: Map.get('propB'),
}

On doing this, console.log(tempObj.propA) crashes in the same fashion.
How to reproduce
I wish I could be of more help here, but I'm absolutely stumped at why this is happening or how to reproduce it in isolation.
I cloned my repo and added some detailed notes to the readme, and I'm happy to discuss w/ anyone that's interested.
The issue happened in a totally different part of the project the other day. I was able to code around it, but this time I figured it wasn't just a fluke.

I've done some more tinkering/weeping w/ this. I was able to fix it for my purposes.
I had the component structured like this (pseudocode):
this.state={modalBody: null, isModalOpen: false}

// sets content to the state, which is passed via props to Modal
handleClick(modalText)={this.setState({isModalOpen: true, modalBody: modalText})

// List render method
Immutable.List.map((item) => {
  <CustomListItem item={item} modalOpener={this.handleClick}  />
}

<Modal content={this.state.modalBody} open={this.state.isModalOpen} />

// <ListIitem>
_clickHandle() {
  this.props.modalOpener(this.props.item)
}

render() {
<ListItem
  onTouchTap={this.clickHandle}
  primaryText={this.props.item.get('text')}
/>
}

^^(which is a pattern i got from an eslint plugin doc. Note that to avoid creating an instance of the click handler and dialog for each item generated by .map(), I've created an intermediate component.
I think the problem lies in the fact that I was passing the item generated by map() outside of the map() anonymous function, but I've been unable to confirm this theory.
However, when I changed it to something like this:
this.state={modalBody: null, isModalOpen: false}

// sets content to the state, which is passed via props to Modal
handleClick(modalText)={this.setState({isModalOpen: true, modalBody: modalText})

// List render method
Immutable.List.map((item) => {
  <CustomListItem
    item={item}
    modalOpener={() => {this.handleClick(item.get('text')}}
  />

<Modal content={this.state.modalBody} open={this.state.isModalOpen} />
}

...things work ostensibly fine. This is obviously not optimal as it's generating a <Modal> for every list  item (vice creating one just for the clicked list item), but it's working for now.
The other things I tried:

forEach() instead of map(). I thought maybe it had something to do w/ side effects, but no dice.
connect()-ing the <Modal> component itself to the Redux store, then just passing an ID string from the .map() function. Unfortunately this also crashed (just as before, I even tried creating a brand new string object to sever the tie to the original data structure, but nothin'-doin'.

Closing this aging issue - sorry I hope you were able to figure out what was going wrong with your app. If you found something related to Immutable.js in there, please open another specific issue


Option to double (triple, ...) click same button to iterate over character

Hi,
I would prefer to have an option to enabling iterating over the character instead of entering it twice. I know that this of course breaks the idea of word predicition, however another perspective is to "free it" from the saved word (so if you expected that the word is not within the dictionary you can directly start to enter it without predection.
In order to enter a letter twice a small delay or something like this could be watched by the application to proceed to next letter.
Background: I already tried your app successfully* on an Android TV and want to use the standard remote which has a numpad to avoid iterating. Because here entering only keywords is enough most, less effort of predicition is needed but avoiding multiple clicks.
*some more optimization should be done for easyness Android TV and I am happy to support programmind and testing (hope you are interested in this too?). But first step by step and I think this feature, which is even helpful for others too, might be quite easy to add?
Generally: Thanks for your app! I couldn't find such an app in the play store although I am wondering that none other TV user is interested / missing this but enters letters with a soft keyboard and the d-pad of the remote...

Have you tried using ABC mode, instead of Predictive? In ABC mode, pressing a number key indeed cycles the possible letters and there is no prediction. I guess, this is what you are looking for... ?
Alternatively, TT9 provides more sophisticated ways of typing words that are not in the dictionary or words that are similar to existing dictionary words. For example, you could type "Anakin", by starting of with "Ana", then using the right arrow to keep adding letters to the word stem. See the advanced typing rules in the manual.
As for your other question: sorry, no, I do not plan to support TV sets. These devices are of little interest to me. I don't even have one. However, any pull requests are welcome. I didn't quite get what optimizations you have in mind, but as long as they don't change the experience on a phone, they should be OK.

Oh, thank you for pointing me to this option! This was exactly what I am looking for. I thought "input mode" means "input method" - so to change back to e.g. the native soft keyboard. And because I thought I found all options in the settings, I didn't looked for a manual... Sorry for bothering you.
Just as an idea, not sure whether it is useful in praxis: Did you thought about a "mixed mode"? So ABC mode for input, but offering prediction, too, in order to shorten longer input.
Regarding TV: No worries. Then I will play around with a fork and try not to change the behaviour of phone. Some thoughts I have are to use the bigger screen to show more options (or even the 3x4 main input buttons for learning which letter is on which button) and especially to offer more/other buttons for the hotkeys (e.g. I don't have a "#" or "*" or any "F" buttons on the remote)

Oh, thank you for pointing me to this option! This was exactly what I am looking for. I thought "input mode" means "input method" - so to change back to e.g. the native soft keyboard. And because I thought I found all options in the settings, I didn't looked for a manual... Sorry for bothering you.

I understand. As a programmer, I never read the manual myself.

So ABC mode for input, but offering prediction, too, in order to shorten longer input.

This is not possible, unless I completely change the way the application works. By design, keyboards have a single suggestion list. I am using it either for displaying the words in Predictive mode or displaying the letters in ABC mode. In order to display both letters and words, there would have to be a secondary list and each has to rewired depending on the item source.
Anyway, I find it difficult to appreciate the idea. I mean, T9 is meant to reduce the key presses to 1 per letter. However, typing in that "mixed" mode you suggest, one would still need to press a single key multiple times to select the desired letters, which kind of defeats the purpose of T9 and suggestions. Having 17 years of experience with barphones, I definitely feel how input is going to take longer, not shorter. Perhaps, the experience with a remote control is different... I don't know... it makes me scratch my head... :thinking:

Some thoughts I have are to use the bigger screen to show more options (or even the 3x4 main input buttons for learning which letter is on which button)

I do want to create a full touchscreen version of TT9 and make it usable on a smartphone, because this is the typing method for thumbs, while QWERTY is for 10 fingers. Unfortunately, with my skills, it would take at least a month and I don't have that much free time now.

... offer more/other buttons for the hotkeys (e.g. I don't have a "#" or "*" or any "F" buttons on the remote)

Extra key support is not a bad idea. I just don't have the device to verify them.
In order to add new ones, you need to add them to the beginning SectionKeymap.java. It's the file that takes care of the entire key settings section. Do any validation in the same file as needed. Also, if you need to override the defaults, you can do it in: SettingsStore.setDefaultKeys().
However, you may want for PR #141, because it fixes some bugs with key handling and also introduces some changes in that particular file. You'd probably want to avoid code conflicts...

Having 17 years of experience with barphones, I definitely feel how input is going to take longer, not shorter. Perhaps, the experience with a remote control is different

I am absolutley convinced that an experienced user is way faster with true T9 than a "mixed" mode and I don't think that a remote will change this dramatically. It's just the different usecase and that saving new custom words is always more complicate, so for a TV I think it's not worth (a user will most times search only once for a movie title and on the other hand in movie titles will often be unknown words). Actually my wording for the "mixed" mode was not perfect and maybe misleading: I didn't mean suggestion like in T9, but additional auto-completion hints to shorten up long words (like in the advanced filtering for predicition mode), which can be chosen with right arrow. But well, if it's complicate to implement: As I wrote, just an idea to have in mind as another option :)

I do want to create a full touchscreen version of TT9 and make it usable on a smartphone, because this is the typing method for thumbs, while QWERTY is for 10 fingers. Unfortunately, with my skills, it would take at least a month and I don't have that much free time now.

Ok, then I will try my luck, and hope that an IME is not completley different than other android apps regarding GUI. I will let you know when I get it working. But may take some time, too...

In order to add new ones, you need to add them to the beginning SectionKeymap.java...

Thank you for your hints!

I didn't mean suggestion like in T9, but additional auto-completion hints to shorten up long words (like in the advanced filtering for predicition mode), which can be chosen with right arrow. But well, if it's complicate to implement: As I wrote, just an idea to have in mind as another option :)

This is currently not possible. ABC mode does not use the database, so it cannot provide any other suggestions but the letters. In order to achieve what you are describing, ModeABC has to be refactored to query the database like ModePredictive. Or even better, use it as a base to create a new "hybrid" or "mixed" mode.
Still, I can't really wrap my head around this intermediate scenario. You can either type letter by letter or type entire words. I hope you can complete what you have in mind. I would be curious to see it.


Revert "Build: Update build.sh"

This reverts commit f1fd4d9da8d1a6353d208ed6ed5f44649ff5a8b1.
Fixes: #1792

lgtm

LGTM. It seems that we still need to push to ${registry}/${name}

I think it might be part of the tag_string (with the -t  removed) rather than ${registry}/${name} as that wasn't created by the buildx step before


Spanner ADO.NET Ability to set properties in any order for SpannerParameter

Currently you need to specify the SpannerDbType before setting Value.
Per .NET guidelines, you should be able to set properties in any order.

Fixed in #1135.


changed function to intergrate map and made traffic function param th…

…e general city name

MORE EDITS AND CLOSURE PLOTS COMING


Create cobalt.rs

Adds the Cobalt static site generator. It's written in Rust. It has a decent following.

Sorry this PR got so stale. We had some plan updates for the page around that time. I went ahead create a new PR to rebase the latest master: https://github.com/netlify/staticgen/pull/241
Cobalt is now live on the site
http://www.staticgen.com/cobalt


Problems disappear temporarily when file is closed and reportingScope is set to workspace

I think this might be controlled by VS Code but filing it here just in case.
When  "mypy-type-checker.reportingScope" is set to "workspace" and you have multiple files with problems reported by mypy, closing the files will make them disappear from the problems window temporarily. Once you open another file, it shows back up.
https://github.com/microsoft/vscode-mypy/assets/45497113/a3323846-d698-4895-b057-469a48bb2d01

Can you set this "mypy-type-checker.trace.server": "verbose"? and collect the logs?

Actually, I think I know what is going on. I will address this.

Actually, I think I know what is going on. I will address this.

Am still seeing this today (along with other general problems with the reported problems not showing up in the Problems tab), so is safe to say this wasn't ever addressed?
FWIW, seems to only be a problem when using workspace scope and daemon. Switching the mypy-type-checker.preferDaemon to false and everything seems to work as expected.

The issues with Daemon are with mypy itself. The daemon is known to have cashing bugs, hopefully future version of mypy address it.


Don't use silverlight Mono.Cecil config in Release

Release|Any CPU is using the winphone_Release config of Mono.Cecil.  This caused me some strangeness with missing metadata due to #if SILVERLIGHT.

Looks good, thanks! Sorry for taking so long to look at it.


Fix ABI regression in task groups due to Sendable requirements.

Top-level functions still encode inferred requirements in the mangled name.
In this case, Sendable has no ABI impact beyond the effect on the mangled
name, so fix the mangled names to maintain ABI.
Fixes rdar://83617067.

@swift-ci please smoke test


fetch-canonical_data_syncer: Download v0.17.0, not latest

Pinging @ErikSchierboom - could you please check the changes to the Powershell script? In particular, I'm led to believe that it already follows redirects (we needed to add --location in the bash script), but I haven't tested it.

This commit alters the fetch-canonical_data_syncer scripts so that
they will stay working after we make changes to the
canonical-data-syncer repo.
The plan for those changes is:

Rename the existing canonical-data-syncer repo to configlet-v3.
Use that repo to develop configlet for Exercism v3. The previous
functionality will be adapted into a configlet-style CLI with a
sync subcommand.
When we are ready, rename the existing configlet repo to
configlet-v2, and the configlet-v3 repo to configlet.

After step 1, GitHub will redirect canonical-data-syncer traffic to
the renamed repo. However, each fetch script was implemented to download
the latest release, and in future the latest release will not be
backwards-compatible with canonical-data-syncer: it will instead have
a configlet-style CLI with a sync subcommand.
Therefore, this commit alters each fetch script so that it downloads the
final release of canonical-data-syncer. We hard-code the download by
id, rather than by tag, giving us the option to start the
new configlet versioning at v0.1.0. We must also add the
--location flag to the first curl command so that it follows
redirects. The request in the .ps1 script already follows redirects.
In the future, track repos can remove each fetch-canonical_data_syncer
script by either:

adding the upcoming new version of the  fetch-configlet script
or simply waiting until the sync functionality is available in the
repo located at exercism/configlet.

The PowerShell script does indeed handle redirects out of the box. I've just verified this.

Great. Thanks.

The PowerShell script does indeed handle redirects out of the box. I've just verified this.

Great. Thanks.

I've merged this PR as this will allow us to move forward with the development of configlet v3. If there are any questions about this change, feel free to let me know.

I've merged this PR as this will allow us to move forward with the development of configlet v3. If there are any questions about this change, feel free to let me know.


Fix incorrect available bytes when change space in StorageDir

What changes are proposed in this pull request?
Fix incorrect available bytes when change space in StorageDir.
Why are the changes needed?
Otherwise the value of available bytes will greater than real available bytes And also make it thread-safe.
Does this PR introduce any user facing changes?
No user facing changes.

Codecov Report

Merging #14382 (eb372ac) into master (f24cc71) will decrease coverage by 18.53%.
The diff coverage is 63.80%.

@@              Coverage Diff              @@
##             master   #14382       +/-   ##
=============================================
- Coverage     35.49%   16.96%   -18.54%
+ Complexity     7672     2706     -4966
=============================================
  Files          1493     1495        +2
  Lines         87361    87479      +118
  Branches      10420    10425        +5
=============================================
- Hits          31012    14841    -16171
- Misses        53747    71399    +17652
+ Partials       2602     1239     -1363

Impacted Files
Coverage Δ

.../java/alluxio/master/block/DefaultBlockMaster.java
41.10% <0.00%> (+0.40%)
:arrow_up:

...a/alluxio/worker/block/meta/DefaultStorageDir.java
21.25% <0.00%> (-67.93%)
:arrow_down:

...main/java/alluxio/master/job/plan/PlanTracker.java
0.00% <0.00%> (-66.28%)
:arrow_down:

...in/java/alluxio/master/AlluxioExecutorService.java
33.33% <33.33%> (ø)

...in/java/alluxio/master/ExecutorServiceBuilder.java
55.40% <55.40%> (ø)

...common/src/main/java/alluxio/conf/PropertyKey.java
98.05% <100.00%> (-1.39%)
:arrow_down:

...n/src/main/java/alluxio/metrics/MetricsSystem.java
51.50% <100.00%> (-9.94%)
:arrow_down:

...main/java/alluxio/master/AlluxioMasterProcess.java
67.64% <100.00%> (-2.15%)
:arrow_down:

...mon/src/main/java/alluxio/shell/CommandReturn.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...mon/src/main/java/alluxio/util/ExceptionUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 571 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 86a7738...eb372ac. Read the comment docs.

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.7 please

alluxio-bot, cherry-pick this to branch-2.7 please


Microsoft Defender says: Trojan:Win32/Wacatac.B!ml

Hi,
I don't know if it is a temporary problem ... but at the moment the Win11 Microsoft defender says the current klipper_estimator.exe (v 3.7.0) has a Trojan:Win32/Wacatac.B!ml and quarantains the file

With best regards,
freakyDude

Hi,
I guess this is a false positive from Windows Defender. I really have no way of fixing that. You can try compiling the tool yourself following the README file.
Best regards,
Lasse

The windows defender stops with 3.7.1 to detect the "virus"


Adding calculator for implementing section formula

I would like to add a calculator for implementing the section formula in the coordinate geometry section i.e. finding the coordinate point which divides the joining of two points (x1,y1) and (x2,y2) in the ratio m:n.

Go ahead
Deadline : 23 April 2021


Improving the account recovery claims modifications

Is your suggestion related to an experience? Please describe.
Once a user sets the recovery email and configures 2 security questions, the same user can update the sensitive claims without verifying it’s the user who is initiating this change request. This applies for the email recovery field as well. In the below GIF it shows how an authenticated user can modify previously configured account’s recovery options easily without verifying.

Describe the improvement
Set a precondition to reauthenticate when adding/modifying security questions or when changing the recovery email.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Default n_neighbors parameter does not match documentation in README

The documentation in README.md says the default number of neighbors scales up with the number of samples comprising the dataset.

n_neighbors: the number of neighbors considered in the k-Nearest Neighbor graph. Default to 10 for dataset whose sample size is smaller than 10000. For large dataset whose sample size (n) is larger than 10000, the default value is: 10 + 15 * (log10(n) - 4).

But this documentation is incorrect! While this behaviour can be enabled by supplying n_neighbors=None to PaCMAP, that is not the default value of the parameter and hence not the default behaviour. Instead, the default number of neighbors is always n_neighbors=10, irrespective of the size of the dataset.
If the automatic scaling behaviour should actually be the default behaviour, the default value for n_neighbors in the PaCMAP class should be changed to n_neighbors=None. On the otherhand, if the current behaviour (n_neighbors=10 for all dataset sizes) is desired, the README should be corrected to reflect the actual default behaviour, whilst still indicating that automatic scaling can optionally be enabled by explicitly supplying n_neighbors=None.
If the default parameter is changed to n_neighbors=None, I recommend this change is done with great care as it would constitute a breaking change to the API (all code which does not specify n_neighbors explicitly would change its behaviour with the new release). i.e. IMHO Such a change would need to be made in a new major release of pacmap, and be well-documented.
Additionally, I think it is confusing for new users that the example code-block in the README uses the n_neighbors=None behaviour, when this is not the default behaviour.
embedding = pacmap.PaCMAP(n_components=2, n_neighbors=None, MN_ratio=0.5, FP_ratio=2.0)

Hi there, thank you for your interest in PaCMAP, and my sincerest apologies to any inconvenience you've experienced. The autoselection of n_neighbors feature was brought into PaCMAP in v0.4.0, and at that time we also set it as the default selection. This change was retracted only a few days later in v0.4.1, as we received complaints on the breaking change. Previously, the release notes was displayed at the very beginning of the README file, so the users will be notified of the change in the default value, but it was refactored into a standalone file in an update about one month ago, which leads to the situation. I'm so sorry for any problems you've experienced and I'm working on a hotfix to solve this discrepancy. The default value for n_neighbors will still be set to 10, and we will fix the documentation to ensure this is displayed properly.


when using mouse wheel, the scroll.direction was bogus

by temporarily setting _lastEvent to draggable, the direction is correctly
calculated when there is no velicity

This behavior is still a bit odd, I think:
scroll = new ScrollComponent
	width: Screen.width
	height: 100
	mouseWheelEnabled: true
	scrollHorizontal: false

layerA = new Layer
	image: Utils.randomImage()
	parent: scroll.content

scroll.onScroll ->
	print "scroll", @.direction
	print "draggable", @.draggable.direction

Using a trackpad, if you scroll by moving two fingers up and down (only works when mouseWheelEnabled, the scroll direction will be fine, but the draggable direction is null. When you scroll the content by dragging, the scroll direction is undefined, but the draggable direction is correct.
I think the null value is actually correct, because you’re not dragging the layer, but scrolling it. I would expect the scroll direction to always have a value though.

not that before, these were simply always returning "down" ...
when doing this, it is slightly better
scroll.onMove ->
	print "scroll", @.direction
	print "draggable", @.draggable.direction

but I will see about fixing this

Yes, sorry for highjacking the pull request :) I think this works well and should be merged and the point I brought up is a ‘new’ issue.


Update(fix) wallet-server.md

Updated HW requirements, I don't think it was possible to get hub-server running at all on the old values. New values used contain some amount of guessing.
The current steps left room for a lot of guessing of what exactly to do.(Like when to exactly start lbrycrdd, or the info that it doesn't matter when different things are started/stopped) This wasn't completely obvious for me. I added bit more details to the guide, which hopefully makes it easier to get hub running for people.
As version 17.4.6 requires much less RAM, I added it to the guide. I don't know if there are other things than RAM, for people to care, when they choose the version to use. Probably could use some piece of text for how to choose?
Updated word "wallet server" to "hub server" when referring to whole thing.
Experience with old guide:
It took me ~8 days total to get hub-server running, mainly because lbrycrd crashing for lack of RAM(had 16GB) and there was no info about blockchain snapshot on this guide(I synced from start and did --reindex after crashes instead, this took most of the time).
After needing to ask help and trying with 17.4.6, it took me ~2 days to get everything running. And that was when letting lbrycrd fully sync(from snapshot) before starting docker stuff( I didn't want more unexpected crashes that make lbrycrd want --reindex). In the end, on 17.4.6 it was very straight-forward to get things going.  (Luckily I had 315GB of storage on server, instead of 200GB as in guide) Currently it's using 8.7 GB of RAM, with this ES_JAVA_OPTS=-Xms2g -Xmx2g. Forgot it on lower value from earlier tries if that matters
EDIT:
Yesterday .lbrycrd took 145GB (value mentioned in guide for what to expect the lbrycrd needing)
Now it takes 139GB
There has been reboot between those du -sh .lbrycrd commands. Don't think I have touched anything in .lbrycrd manually.

thanks @keikari!


Testing

FIXME: description.
Fixes #42.

Regarding Testing different targets, I would say we do this on a per-backend basis so we would have tests for cmake, npm, pnpm and lfc

I would add the other backends tests in a seperate pr

The CPP Tests are failing becuase the wrong path is in
LF_SRC_PKG_PATH:UNINITIALIZED=/home/revol-xut/workspace/lf/lingua-franca/test
It should always point to the src dir

@lhstrh Last Thing I did this weekend was looking into why the typescript CI is still red, and I think its the dockerized tests.

Okay the next problem with the typescript target is that the federated code gen is dropped under fed-gen and not src-gen

Okay, the next problem with the typescript target is that the federated code gen is dropped under fed-gen and not src-gen. Is the fed-gen folder so structurally different from the src-gen folder that its not feasible to remove the fed-gen folder ? @lhstrh

Yes, it is. The fed-gen/src folder actually has generated LF code in it, which is then compiled down to target code in a second step.

@lhstrh Would it be possible to rename fed-gen to src-gen


Refactor the Chalice back-end into smaller Python modules

api/msam/app.py is big and needs decomposition into modules.

Refactored app.py into 10 source files (including app.py)


Working horizontal and vertical regridder.

This is a working horizontal and vertical regridder. I need to push some tests, but that'll come later.
Its worth banking this change for all to see.
Note that, we need python-stratify to be pushed to the conda-forge or scitools channels and added to the ESMValTool environment.
I've added the UnstructuredNearest horizontal regridding scheme and that means we require to depend on iris>=1.12
We may want to separate the vertical interpolation from the horizontal regridding. But that will come out of future discussions regarding the structure and design of the backend preprocessing - which at the moment is in a state of flux.

Ping @axel-lauer and @mattiarighi


neopt should be using CMake instead of make

Recommend targeting version 3.10 as a minimum requirement. Ubuntu 18.04 has version 3.10.2. VS2019 has a custom build of v3.14. I forget what VS2017 has, but I'm pretty sure it's later than 3.10

Seems about right for OSX too. I don't recall installing it but I got 3.10.2 on my system

@rodoufu do you think this is easily done?

The first thing I've suggested is the first issue =)
@igormcoelho ,  haven't seen anything too complicated in the makefiles, so I don't think it will be that difficult.
The first time I saw the project I was going to send a PR for that, but you said you were changing a lot of stuff so I decided to wait for a little.


Fix musescore URL

Fixes: #1573

Add "extract_dir": "MuseScore $majorVersion" into autoupdate property.

Also fix checkver to match releases/MuseScore-([\\d\\.])/ from https://musescore.org/en/download/musescore.msi to get correct version without manually adding 0.

Hi ruifigueira
(apologies for the duplicate post, still finding my feet)
Is there a reason why the Version variable was not brought in in line with the URL? e.g. I believe
"version": "3.0"
should be
"version": "3.0.0"
The next version is expected to be 3.0.1

Thanks for updating the script. I just wanted to point out a few consequences of these changes:

Renaming MuseScore3.exe to MuseScore.exe means that it is not possible to run two versions at the same time (see this and this). This is why MuseScore.exe is versioned, but if versioning interferes with scoop's functionality then it is of course up to you guys to decide which functionality you wish to prioritise.

Checking the version from https://musescore.org/en/download/musescore.msi rather than from https://musescore.org/en/download will detect a new version before the MuseScore developers are ready to publish it. Usually they wait a few days after uploading the new version to allow people to test it before publishing it in the download page, and if problems are found they will upload a new version to the same URL. I have contacted the MuseScore team to request that they provide a stable API to the current version that exactly matches the version in the URL.

Hm, for Firefox this is true, but other programs like 7-Zip and MuseScore don't care about this. :smile:

if problems are found they will upload a new version to the same URL

If they keep the version (3.0.0) and just update the, then this will break the manifest anyway.
Musescore itself uses this appcast.xml to check for a new version including a build number. Using the full version.build would be the best way.
Version 2 used https://update.musescore.org/update_win_stable.xml, so I guess the URL will change for version 4 anyway. 😁

https://github.com/lukesampson/scoop-extras/commit/a1f1d02141cf350a8dee071449e43b3b9e408caa

Thanks for fixing! Good idea to use the sparkle appcast!

If they keep the version (3.0.0) and just update the file, then this will break the manifest anyway.

Indeed, but they wouldn't do this after changing the "current stable version is" text on the download page, which is what my regex was originally checking. However, the appcast is a better solution because there is a small possibility that they do a Windows-only release, e.g. to get the existing program working on a new "version" of Windows (Windows 10 is supposed to be the last version of Windows but you know what I mean - after a big update).

Hm, for Firefox this is true, but other programs like 7-Zip and MuseScore don't care about this.

Interesting. This is probably because MuseScore 3 uses a different version of Qt to MuseScore 2. Probably whoever changed the executable name was going from prior knowledge that names sometimes cause conflicts and didn't test it in this specific case, or tested it but decided not to take the risk.

So I spoke to the MuseScore developers again and they gave me some more reasons for versioning the MuseScore3.exe executable. These include:

Conflicts in the Windows Registry
Easy distinguishability in the Open With list
To allow MuseScore 2 to remain available in the Windows Store

None of these reasons are relevant to Scoop.


New release

Hi, would it be possible to push up a new release to crates.io?
There are a few features that have been added in the last few weeks that I would like to use in my project, but it seems the last release was a month ago.

Right, I actually just bumped the version to 0.3.1 so as to be able to make a release, should land tomorrow hopefully.

Thank you!

0.3.1 should be available on crates.io now.


current quota exceeded

I have a paid subscription ($20 per month), created a api key via https://platform.openai.com/account/api-keys, created a OPENAI_API_KEY environment variable in Windows 11 with the api key value.
I am getting the following error when running :ChatGPT:
You exceeded your current quota, please check your plan and billing details.

@azinsharaf Try creating a new session , that solved the issue for me.

@miguelmoraperea  creating a new session deletes the error message but it happens again after submitting a new query.

@azinsharaf In that case, I think you might need to set up a billing account with ChatGPT API. "ChatGPT and ChatGPT Plus subscriptions are billed separately. The API has its own pricing, which can be found at https://openai.com/pricing. The ChatGPT Plus subscription covers usage on chat.openai.com only..." according to their website.

i re-entered my credit card info and it started to work.


Can't view the bookinfo services in Jaeger UI after the latest Istio deployed

After I followed the link and installed the latest Istio into our Kubernetes platform, I checked the Istio pod status as the following:
root@m-1:~/istio-1.1.0-rc.4/samples/bookinfo# kubectl get po -n istio-system
NAME                                      READY     STATUS      RESTARTS   AGE
istio-citadel-5bf5488468-vn5w5            1/1       Running     0          12m
istio-galley-96d888b56-sxbcc              1/1       Running     0          12m
istio-ingressgateway-665ffb67b8-bz5r9     1/1       Running     0          12m
istio-ingressgateway-665ffb67b8-d8xk9     1/1       Running     0          9m
istio-init-crd-10-hgkl7                   0/1       Completed   0          13m
istio-init-crd-11-lbk6z                   0/1       Completed   0          13m
istio-init-crd-certmanager-10-9gksp       0/1       Completed   0          13m
istio-init-crd-certmanager-11-fd9w8       0/1       Completed   0          13m
istio-pilot-5f4fd55b86-4dk4m              2/2       Running     0          12m
istio-policy-5674979c46-brnsr             2/2       Running     2          12m
istio-sidecar-injector-744f68bf5f-r4c6l   1/1       Running     0          12m
istio-telemetry-577955c48d-l8j9r          2/2       Running     2          12m
istio-tracing-75dd89b8b4-kqdvq            1/1       Running     0          12m
kiali-5d68f4c676-57mz5                    1/1       Running     0          12m
prometheus-89bc5668c-q4jsm                1/1       Running     0          12m
Then I deployed the sample: bookinfo into this env.
Your release is named istio.
To get started running application with Istio, execute the following steps:

Label namespace that application object will be deployed to by the following command (take default namespace as an example)

$ kubectl label namespace default istio-injection=enabled
$ kubectl get namespace -L istio-injection

Deploy your applications

$ kubectl apply -f .yaml
For more information on running Istio, visit:
https://istio.io/

Have you tried increasing the trace sampling rate? https://preliminary.istio.io/docs/tasks/telemetry/distributed-tracing/overview/#trace-sampling

as you said, once I increase the sampling rate to 100, it is working fine. thank you so much!


github actions events-that-trigger-workflows

Description
when push to origin/master, always run ci. change to triggered actions.
Related links

https://help.github.com/ja/actions/reference/events-that-trigger-workflows

fix.


Update landscape.yml

removed redundant category
Signed off: Rob Armstrong<EMAIL_ADDRESS>

removed redundant SubCategory text.
Preview looks good.  LGTM
Signed off: Rob Armstrong<EMAIL_ADDRESS>


feat: ability to set wallpaper texpath in config

Describe your PR, what does it fix/add?
This pr adds the ability to change the wallpaper in the configuration file.
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
No
Is it ready for merging, or does it need work?
This should be ready

no? Use hyprpaper


amplify env checkout  should update aws-exports.js file

Describe the bug
When you do amplify env checkout <env> the aws-exports.js file does not update, it will still contain settings of the previous environment.
To work around this, you can do amplify init and choose an existing environment and choose the  you checkout to.
To Reproduce

Check aws-exports.js
Do amplify env checkout <env>
Check aws-exports.js
Realize that it hasn't changed to reflect the environment settings you are in.

Expected behavior
amplify env checkout <env> should update aws-exports.js file to reflect the settings of the current environment you checkout to.
Additional context
To work around this, you can do amplify init and choose an existing environment and choose the <env> you checkout to.

@aprilmintacpineda it seems to be a valid request, I marked it as a bug, thanks for reporting it.

@aprilmintacpineda This should be fixed in the latest version of the CLI. Please report back if you're still seeing this issue.

I am currently on CLI version 4.11.0 and I am still having this issue. Creating a new env through amplify add env did not update aws-exports.js file. Running amplify init or amplify env checkout <environmentName> does not update the aws-exports.js either.

Still not solved amplify -v 4.18.1
Disappointing.

still not solved in
amplify -v
4.40.0

still not solved in
amplify -v
4.40.0

Hmmm, maybe it's a recurring bug? @attilah @kaustavghosh06?

Hmmm, maybe it's a recurring bug? @attilah @kaustavghosh06?

If you run checkout with restore it will update (v4.44.0)
amplify env checkout dev --restore

Still not solved as of today... Amplify 4.46.1. Pulled a fresh copy of my repo on a new computer, ran amplify init, choose the proper environment and whatnot, finally ran amplify pull and the aws-export.js file is not there.


npm package does not match github repo

Prerequisites

[X] I have written a descriptive issue title
[X] I have searched existing issues to ensure the issue has not already been raised

Issue
Package installed via npm does not include the "without" function. It seems like that npm package does not match the github repo.
const S = require('fluent-json-schema')

const wholeSchema = S.object()
  .prop('name', S.string())
  .prop('age', S.number())

const schemaWithoutAge = wholeSchema.without(["age"]) // wholeSchema.without is not a function

True, I will deploy the latest later this evening
On Wed, 2 Mar 2022 at 15:45, Utku K @.***> wrote:

Prerequisites

I have written a descriptive issue title
I have searched existing issues to ensure the issue has not already
been raised

Issue
Package installed via npm does not include the "without" function. It
seems like that npm package does not match the github repo.
const S = require('fluent-json-schema')
const wholeSchema = S.object()
.prop('name', S.string())
.prop('age', S.number())
const schemaWithoutAge = wholeSchema.without(["age"]) // testSchema.without is not a function
—
Reply to this email directly, view it on GitHub
https://github.com/fastify/fluent-json-schema/issues/167, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJN2LEGNWPM34S3YTYYEMDU56EKHANCNFSM5PX2XTDQ
.
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 are subscribed to this thread.Message
ID: @.***>

--
Sent from mobile phone

Thank you very much 🙏

@ukocaman done :)


How to use in Unity Project and keep the project (not git) in sync with this git?

Sorry am quite new to Hololens.
There is this git which has HoloToolKit and there is HoloToolKit from https://github.com/Microsoft/MixedRealityToolkit-Unity
Which should I be using.
Also for my Unity project (not git). How to ensure I can get the latest updates from this git?
Thank you.

Hi markleeyw,
You're right that HoloToolkit appears in these two different locations. The MRTK (Mixed Reality Tool Kit) is still very much in development. It is currently a branch of Holotoolkit github. In the meantime, I recommend downloading the master branch of either MRDesignLabs_Unity (this page), or the main branch of HoloToolkit. If you're new to HoloLens, I recommend getting familiar with HoloToolKit first, and then moving into MRDesignLabs. MRDesignLabs is a more advanced and robust tool, while HoloToolkit is a good confidence builder to get you familiar with many core concepts.
When you do get into MRDesignLabs_Unity (this page), you can keep the tool up to date by updating the submodule. There are instructions for initializing and updating the submodule on the ReadMe/Front page of MRDesignLabs_Unity (https://github.com/Microsoft/MRDesignLabs_Unity).
Hope that helps.