Failed assertion: line 93: '_pointers.containsKey(datum.pointer)' is not true

Steps to Reproduce
Running a flutter app under qemu. When I try and interact with the app with the mouse, I get this:
[00240.406] 03135.03217> [ERROR:../../lib/tonic/logging/dart_error.cc(16)] Unhandled exception:
[00240.406] 03135.03217> 'package:flutter/src/gestures/converter.dart': Failed assertion: line 93: '_pointers.containsKey(datum.pointer)' is not true.
[00240.406] 03135.03217> #0      _AssertionError._throwNew (dart:core-patch/errors_patch.dart:24)
[00240.407] 03135.03217> #1      _AssertionError._checkAssertion (dart:core-patch/errors_patch.dart:31)
[00240.407] 03135.03217> #2      PointerEventConverter.expand.<expand_sync_body> (package:flutter/src/gestures/converter.dart:93)
[00240.407] 03135.03217> #3      _SyncIterator.moveNext (dart:core-patch/core_patch.dart:66)
[00240.407] 03135.03217> #4      ListQueue.addAll (dart:collection/queue.dart:580)
[00240.407] 03135.03217> #5      BindingBase&SchedulerBinding&GestureBinding._handlePointerDataPacket (package:flutter/src/gestures/binding.dart:32)
[00240.407] 03135.03217> #6      _dispatchPointerDataPacket (file:///media/jimbe/mojossd/fuchsia/flutter/lib/ui/hooks.dart:73)

Flutter Doctor
[✓] Flutter (on Linux, channel master)
    • Flutter at /media/xyz/fuchsia/lib/flutter
    • Framework revision fd5a08941e (14 hours ago), 2016-11-14 16:54:24
    • Engine revision a3a13f4a94
    • Tools Dart version 1.21.0-dev.3.0

[✓] Android toolchain - develop for Android devices (Android SDK 23.0.2)
    • Android SDK at /usr/local/google/home/jimbe/Android/Sdk
    • Platform android-23, build-tools 23.0.2
    • OpenJDK Runtime Environment (build 1.8.0_102-google-v7-137518472-137512069)

[x] Flutter IDE Support (No supported IDEs installed)
    • IntelliJ - https://www.jetbrains.com/idea/

[✓] Connected devices
    • None

For more information about diagnosing and reporting Flutter bugs, please see https://flutter.io/bug-reports/.

Is this still an issue?

I assume this issue is stale. Please add a comment to have it reopened if you disagree.


Test bitcoind payment flow

This is somewhat inspired by #5, but simpler. Some code taken from JoinMarket (most of it written by me). Tests SatSale payment flow for bitcoind backend. Similar regtest tests can (and in future should) be added for CLN and LND too, but that's more complicated, as multiple Lightning nodes and channel between them needs to be created, there we likely should look at that lnbpb-testkit by @Kixunil. This also does not test frontend part as #5 does, just bitcoind class payment flow functions.
There is support to also test on different Bitcoin Core versions in CI, but that was too slow with my tests and often even didn't finish at all. Just using some recent version will be good enough for now IMO. We don't use many different and complicated RPCs, so probablity of them breaking between Core versions isn't so big.
Tests only basic flow, where user pays all requested amount in a single transactions. Tests for cases of underpay, overpay, payment in multiple transactions could be added later, in a follow-up PR.

Updated Bitcoin Core version from 24.0 to 24.0.1.


"load_state_filename" changed to "map_filename"

update documentation to reflect new arg name

I signed it!

@Chambana Thanks for the PR, but I think load_state_filename is the new argument name. See here.

You're correct.  I pulled master a few days ago and this must've been changed in the meantime with this merge:
https://github.com/googlecartographer/cartographer_ros/commit/b4e77051594a8c173e33cbb6bd389ac64201c8a2


Function inside then always gets called

When I change order of then and catch, function inside then always gets called.
for example, below code is totally fine
    return qwest.map(method, ROOT_URL + api, payload, options, null)
    .then(function(xhr, response)) {
        console.log(response);
    }
    .catch(function(e, xhr, response) {
        alert(response.message);
    });

but when I change order of  then and catch like below, then always gets called
    return qwest.map(method, ROOT_URL + api, payload, options, null)
    .catch(function(e, xhr, response) {
        alert(response.message);
    });
    .then(function(xhr, response)) {
        console.log(response);
    }

According to Promise specification this is right behavior.

Sorry, I wasn't aware of the behaviour. My mistake. I should've checked before create an issue.
for memo, the check code is below
// prints catch
new Promise(function(fulfill, reject) {reject()}).then(function() {console.log('then')}).catch(function() {console.log('catch')})

// prints catch, and then
new Promise(function(fulfill, reject) {reject()}).catch(function() {console.log('catch')}).then(function() {console.log('then')})


Remove -Swift import in DataModel.m

@rmutter Removed this line from the Jinja template that causes problems when you're running an Objective-C project with Objective-C generated signals code. They will have to add this line (along with adding a bridging header, etc.) if they decide to mix languages, but I feel like that isn't the path we should support by default.

@ZachGawlik makes sense to me and agree we no longer need it


I have a question about the paper.

Hi, Thanks for the interesting paper.
Problem: Row and Column
The size of the weight matrix W is n x m, and the per-column segmentation is performed as shown in the figure below, right?

The paper clearly states 'k is the number of selected columns.' Isn't m x k the correct notation?

Initially, I thought this was a typo. However, seeing 'rows' continually used instead of 'columns' in later mentions and in Algorithm 1 made me wonder if I had misunderstood something.

Thank you for your careful observation😄. $m \times k$ is the correct notation, cuz k is arranged vertically. We will update this typo in subsequent paper versions.

Thank you for your response.
So, should the 'rows' mentioned in the algorithm all be represented as 'columns'? I have spent a lot of time trying to understand this in some way 😭.

Sorry, I only noticed the second picture of your explanation, based on the equations mentioned in the paper: n represents rows and m denotes columns. So the right expression of this typo is $n \times k$.

Sorry, I only noticed the second picture of your explanation, based on the equations mentioned in the paper: n represents rows and m denotes columns. So the right expression of this typo is n×k.

The details of this selection part are explained in util/autoseatch.py file, the structural selection is based on the column-wise. Your initial insight is right,  and thanks again for your correction :-)


vim overwrites the symbolic link file of .viminfo

description
I made a symbolic link of .viminfo, it's opened and interpreted as normal. But once saving to this file, the symbolic link is replaced by a normal file with new contents. I'd see this behaviour as a bug.
reproduce snippet
cd ~
mv .viminfo /tmp/
ln -s /tmp/.viminfo
vim foo # make sure you type some commands which vim will save to .viminfo
ls -ali .viminfo # the symbolic file is changed to a normal file

closing then.

It wasn't clear to me what the resolution to this was, so I thought I'd follow up on this. In h:viminfo under Notes for Unix: it says:

The viminfo file cannot be a symbolic link.  This is to avoid security                                                                                issues.

So, a symlinked viminfo is simply not supported, by design. I found a couple references (including this issue and this old debian bug against vim 6) to suggestions that using backupcopy could work around this limitation, but I was not able to prevent vim from breaking the symlink using any permutation of backup/backupcopy/writebackup.

Yes, this is by design. I still have an old patch lying around that "removes" this feature, since this comes up once in a while: https://github.com/chrisbra/vim-mq-patches/blob/master/viminfo_resolve_symlinks
But don't think this is worth it.

I run NixOS with impermanence which involves symlinking files from my /home ramdisk to a permanent location elsewhere. Because of this Vim security feature I'm not able to persist my .viminfo this way. Are there details anywhere of the security issue that this is preventing? Does my system being impermanent negate it?
I'm trying to disable this feature so that Vim will respect the symlink, but the patch you posted didn't seem to work for me on version 9.0.1897.

yes, you can also use :set viminfofile nowadays


Update to 2.18.15 (LTR)

Update QGIS 2.x builds to latest release (2.18.x), which is also the new LTR version (updated from 2.14.x series).

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Level: Merge Conflicts

Closes #250
Level currently works but has only default user messages

Comment: Running git gud reset in the middle of a merge conflict completely breaks the Git Gud installation to the point where the only way to fix it is to delete the Git Gud installation directory. git gud init --force and git gud init --force --pretty-please do nothing.


#1892, for postgresql prepared statement

Fixes #1892.

Pull Request Test Coverage Report for Build 6785

2 of 57 (3.51%)  changed or added relevant lines in 13 files are covered.
5 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.07%) to 74.308%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-core/src/main/java/org/apache/shardingsphere/core/parsing/parser/sql/SQLParserFactory.java
1
2
50.0%

sharding-proxy/src/main/java/org/apache/shardingsphere/shardingproxy/transport/postgresql/codec/PostgreSQLPacketCodecEngine.java
0
1
0.0%

sharding-proxy/src/main/java/org/apache/shardingsphere/shardingproxy/transport/postgresql/packet/command/query/binary/ConnectionScopeBinaryStatementRegistry.java
0
1
0.0%

sharding-proxy/src/main/java/org/apache/shardingsphere/shardingproxy/transport/postgresql/packet/command/query/binary/bind/protocol/PostgreSQLStringBinaryProtocolValue.java
0
1
0.0%

sharding-proxy/src/main/java/org/apache/shardingsphere/shardingproxy/transport/postgresql/packet/command/query/binary/parse/PostgreSQLComParsePacket.java
0
1
0.0%

sharding-proxy/src/main/java/org/apache/shardingsphere/shardingproxy/transport/postgresql/packet/command/query/text/PostgreSQLDataRowPacket.java
0
1
0.0%

sharding-proxy/src/main/java/org/apache/shardingsphere/shardingproxy/frontend/postgresql/PostgreSQLFrontendEngine.java
0
4
0.0%

sharding-proxy/src/main/java/org/apache/shardingsphere/shardingproxy/backend/communication/jdbc/execute/callback/ProxyJDBCExecutePrepareCallback.java
0
5
0.0%

sharding-proxy/src/main/java/org/apache/shardingsphere/shardingproxy/transport/postgresql/packet/command/query/binary/bind/PostgreSQLComBindPacket.java
0
8
0.0%

sharding-proxy/src/main/java/org/apache/shardingsphere/shardingproxy/transport/postgresql/packet/handshake/PostgreSQLParameterStatusPacket.java
0
9
0.0%

Files with Coverage Reduction
New Missed Lines
%

sharding-proxy/src/main/java/org/apache/shardingsphere/shardingproxy/frontend/postgresql/PostgreSQLCommandExecutor.java
1
0.0%

sharding-proxy/src/main/java/org/apache/shardingsphere/shardingproxy/transport/postgresql/packet/command/PostgreSQLCommandPacketType.java
2
0.0%

sharding-proxy/src/main/java/org/apache/shardingsphere/shardingproxy/transport/postgresql/packet/command/query/binary/bind/PostgreSQLComBindPacket.java
2
0.0%

Totals

Change from base Build 6783:
-0.07%

Covered Lines:
10632

Relevant Lines:
14308

💛  - Coveralls


[UI-NEXT] Fix wrong packaging locations

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Codecov Report

Merging #8851 (75704d3) into dev (7433dd2) will decrease coverage by 0.06%.
The diff coverage is n/a.

:exclamation: Current head 75704d3 differs from pull request most recent head ed65940. Consider uploading reports for the commit ed65940 to get more accurate results

@@             Coverage Diff              @@
##                dev    #8851      +/-   ##
============================================
- Coverage     45.36%   45.30%   -0.07%
+ Complexity     4084     4077       -7
============================================
  Files           702      702
  Lines         27113    27113
  Branches       2897     2897
============================================
- Hits          12301    12283      -18
- Misses        13657    13676      +19
+ Partials       1155     1154       -1

Impacted Files
Coverage Δ

...he/dolphinscheduler/common/enums/SqoopJobType.java
0.00% <0.00%> (-88.89%)
:arrow_down:

...olphinscheduler/rpc/remote/NettyServerHandler.java
69.04% <0.00%> (-11.91%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
50.70% <0.00%> (-2.82%)
:arrow_down:

...inscheduler/common/task/sqoop/SqoopParameters.java
74.00% <0.00%> (-2.00%)
:arrow_down:

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 7433dd2...ed65940. Read the comment docs.


Eliminate the use of dangerous fgets()

Reported by Cppcheck:

[src/common/sofa_config.c:972]: (error) Buffer is accessed out of bounds: osdisk_name

The corresponding code:
    if (fgets(osdisk_name, CFG_VALSET_STR_LEN, fp) != NULL) {
        if (osdisk_name[0] != 0) {
            _remove_char_str(osdisk_name, strlen(osdisk_name), '\n');
            set_config_item(STR_OSDISK_NAME, osdisk_name, NULL);
            strcpy(osdisk_ret, osdisk_name);
            syslog(LOG_INFO, "[SOFA] USERD INFO get os disk: %s\n",
                   osdisk_name);
            ret = 0;
        } else {

Quoted from Deprecated string input functions - fget:

The fgets ("file get string") function is similar to the gets function. This function is deprecated -- that means it is obsolete and it is strongly suggested you do not use it -- because it is dangerous. It is dangerous because if the input data contains a null character, you can't tell. Don't use fgets unless you know the data cannot contain a null. Don't use it to read files edited by the user because, if the user inserts a null character, you should either handle it properly or print a clear error message. Always use getline or getdelim instead of fgets if you can.

Thank you for your suggestions. Later we will review our codes and replace fgets() with getline() or getdelim() instead.


ci: add dependabot config

Dependabot will open MRs as soon as one of our pip dependencies is outdated.

Codecov Report

Merging #208 (7a3769d) into develop (29f6666) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #208   +/-   ##
========================================
  Coverage    96.65%   96.65%
========================================
  Files           22       22
  Lines         1077     1077
========================================
  Hits          1041     1041
  Misses          36       36

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 29f6666...7a3769d. Read the comment docs.

Tested it in my own repo - looks like that :)


Quadratic regularizer with nonzero baseline

Is your feature request related to a problem? Please describe.
Sometimes we might want to keep controls or trajectories close to some baseline. Currently, $||a||_R$ is allowed, but the new feature should allow for $||a - a_0||_R$.
Describe the solution you'd like
The QuadraticRegularizer objectives can be modified with an optional argument for the new feature. The objective function, gradient, and hessian can be modified to account for the shifted baseline.

This has been implemented on a feature branch, but needs to be tested and merged into main.
Some problem templates may want to allow access to this feature via a keyword argument.


Testing Account Controller is too difficult

I must be doing something wrong, because currently testing the default AccountController from the default asp.net authentication is very difficult.
Here is my test
        [Fact]
        public async Task Can_Add_A_User()
        {
            // Setup Identity with InMemoryStore
            var services = new ServiceCollection();

            services.AddEntityFramework()
                .AddInMemoryStore()
                .AddDbContext<InMemoryContext>();
            services.AddIdentity<ApplicationUser, IdentityRole>()
                .AddEntityFrameworkStores<InMemoryContext>();

            services.AddLogging();

            // Mock HttpContext
            var context = new Mock<HttpContext>();

            var authenticationManager = new Mock<AuthenticationManager>();
            context.Setup(c => c.Authentication).Returns(authenticationManager.Object);

            var response = new Mock<HttpResponse>();
            context.Setup(c => c.Response).Returns(response.Object);

            var contextAccessor = new Mock<IHttpContextAccessor>();
            contextAccessor.Setup(a => a.HttpContext).Returns(context.Object);
            services.AddInstance(contextAccessor.Object);

            var provider = services.BuildServiceProvider();

            var userManager = provider.GetRequiredService<UserManager<ApplicationUser>>();
            var signInManager = provider.GetRequiredService<SignInManager<ApplicationUser>>();

            var db = new InMemoryContext();

            using (var accounts = new AccountController(userManager, signInManager, db))
            {
                var registerViewModel = new RegisterViewModel
                {
                    ConfirmPassword = "Test-123",
                    Password = "Test-123",
                    Email =<EMAIL_ADDRESS>                };

                var result = await accounts.Register(registerViewModel);

                var actual = await userManager.FindByEmailAsync("test@test.com");

                Assert.NotNull(actual);
            }
        }

Can anyone give me a hint as to what I am doing wrong?

/cc @HaoK

/cc @divega ping.

This looks pretty similar to we do I think, there are a lot of services involved so I agree that mocking them all is a pain, but I'm not sure what alternative there is, see:
https://github.com/aspnet/MusicStore/blob/master/test/MusicStore.Test/ManageControllerTest.cs

Are you testing the application code in Register action or are you testing the Frameworks used in the Register action?

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


go fmt

I've run go fmt to standardise formatting (my editor runs go fmt on save, I think it's a good practice)

Duplicate of PR #173.


retry with disposing establish connection when stream deletion

This PR includes following things

add retry with disposing establish connection when stream deletion

https://github.com/grpc/grpc/blob/master/doc/connection-backoff.md


Bundle references into definitions

Let's say we have a schema that has external references to another schema's definitions:
foo.json
{
  "definitions": {
    "foo": {
      "properties": {
        "a": {
          "$ref": "./bar.json#/definitions/bar"
        },
        "b": {
          "$ref": "./bar.json#/definitions/bar"
        },
        "c": {
          "$ref": "./bar.json#/definitions/bar"
        }
      }
    }
  }
}

bar.json
{
  "definitions": {
    "bar": {
      "properties": {
        "a": {
          "type": "string"
        },
        "b": {
          "type": "number",
          "maximum": 20
        }
      }
    }
  }
}

When using parser.bundle('foo.json') to bundle the external references, this is the resulting schema:
{
  "definitions": {
    "foo": {
      "properties": {
        "a": {
          "properties": {
            "a": {
              "type": "string"
            },
            "b": {
              "type": "number",
              "maximum": 20
            }
          }
        },
        "b": {
          "$ref": "#/definitions/foo/properties/a"
        },
        "c": {
          "$ref": "#/definitions/foo/properties/a"
        }
      }
    }
  }
}

While this is a valid schema, it seems more straightforward to copy the definition for bar into definitions, especially if it is referenced in multiple places:
{
  "definitions": {
    "foo": {
      "properties": {
        "a": {
          "$ref": "#/definitions/bar"
        },
        "b": {
          "$ref": "#/definitions/bar"
        },
        "c": {
          "$ref": "#/definitions/bar"
        }
      }
    },
    "bar": {
      "properties": {
        "a": {
          "type": "string"
        },
        "b": {
          "type": "number",
          "maximum": 20
        }
      }
    }
  }
}

Can this be done somehow?

You can accomplish this by referencing bar in the definitions section, like this:
{
  "definitions": {
    "foo": {
      "properties": {
        "a": {
          "$ref": "./bar.json#/definitions/bar"
        },
        "b": {
          "$ref": "./bar.json#/definitions/bar"
        },
        "c": {
          "$ref": "./bar.json#/definitions/bar"
        }
      }
    }
  }
  "bar": {
    "$ref": "./bar.json#/definitions/bar"
  }
}

This ability is super helpful for creating files that are used by other class generator tools.
Are there any plans to have an option to create the shared definition entries automatically?
When maintaining multiple files, it's kind of hard to remember/enforce that the definition entry is created along with the other references. And if those references ever change you need to make sure to go update the definition entry as well.

@aaronschwartz - The behavior you're asking for doesn't sound (to me) like it adheres to the JSON Reference spec.  I also can't think of any straightforward way to implement it.
That said, if you can get it working, I'd be happy to review a PR.  As long as it doesn't break any existing functionality and is isolated to just the bundle() method, I think it'll be ok.

Adding my +1 here because the requested behavior is necessary for generating client libraries from a combined API definition. I also think it does adhere to the spec as far as I can tell, and results in a much cleaner result (in my opinion 🙂 ). I would love to see this as a configurable option in the bundle() method.
I will attempt to write up a PR in the next couple of weeks if I can find the time. Thanks for being open to it

This is a commonly requested feature but rewriting your JSON Schema is not what this tool is about. We've implemented some custom logic to take on this opinionated rewrite at stoplight.io and maybe we can release this as another OSS tool, but this is working as expected.

This is a commonly requested feature but rewriting your JSON Schema is not what this tool is about. We've implemented some custom logic to take on this opinionated rewrite at stoplight.io and maybe we can release this as another OSS tool, but this is working as expected.


PCUIC vs Ast.term

I just recently started looking into PCUIC and it looks really great. Would it be possible to replace the Template.Ast.term definition with this one? Are there features that exist in one but not in the other?

PCUIC is an alternative syntax to the TemplateCoq one, replacing one by the other would rather mean only keeping one I think.
If I'm correct, the point is that TemplateCoq follows rather closely the Coq implementation while PCUIC is supposed to be easier to reason on.
Ultimately it should have utilities to easily go from one to the other.

Yes. The original motivation was exactly that, I was thinking that you could play tricks with partially applied functions, but that breaks Coq's internal invariants.
It isn't important to do now, but personally I would be in favor of moving to a single, cleaner representation.

Hi guys,
I think, n-ary application makes a big difference in convenience for reasoning. I believe that the mkApps  smart constructor is used in various relations (and, IIRC, in substitution) to "normalize" various ways of nesting tApp with a list of arguments.
I am currently working with the WcbvEval.eval relation and run into the mkApps issue.
I was hoping that something like this should hold:
l <> [ ] ->
WcbvEval.eval Σ Γ (mkApps t l) t ->
WcbvEval.eval Σ Γ (tApp t l) t

and I can replace derivations of mkApps with just tApp, but apparently this is not true (at least without imposing restrictions on what is t)

You probably want to phrase a viewApp : term -> term * list term that would guarantee that the resulting term does not have a head tApp constructor.

Yes, good point. Actually, since I am translating to term from another type of expressions (with only binary applications), my translation guarantees that there are no empty or nested applications, I might be able to use that. Thanks!

I still struggle with my proof involving mkApps. I think, if I targeted PCUIC instead, my development would be much easier. So, the question is which Ast makes more sense to target in a long run? Also, if we keep both, would it be reasonable to translate PCUIC to TemplateCoq and prove correctness? For me this direction makes a lot of sense, since I am developing an embedding of functional languages in Coq, so making a certified chain of translations MyAst -> PCUIC -> TemplateCoq is probably better (or at least, simpler, because PCUIC looks better as a target). What do you think @gmalecha @TheoWinterhalter @mattam82 ?

It should be trivial to do the PCUIC -> Template translation, good to know we have a practical application for it too!

Personally, I think targeting PCUIC is the righy way to go, at least at the level that I understand things.

I think that is part of the goal of PCUIC, having something nicer to reason about.

I agree with Gregory and Théo.
Matthieu, would the translation require that stacked (binary) tApp in PCUIC get translated to one big Template Coq tApp (taking a term and a list)? That's not completely trivial anymore I think, but if you want going from TemplateCoq to PCUIC and back to be the identity on many terms it's necessary, no?

would the translation require that stacked (binary) tApp in PCUIC get translated to one big Template Coq tApp (taking a term and a list)?

This is exactly the part which I am struggling with :) At least in the proof involving WcbvEval.eval relation and evaluation in my language a "stacked" (or nested) application is evaluated in different "direction" than the application to a list of terms.

All the more reason to do this once and for all then. :)

Another thing that looks a bit strange to me: mkApps is also used in substitution. Because of this I didn't have much of a choice when I defined the translation from my language to TemplateCoq Ast. If I didn't collapse stacked applications, my substitution would not be compatible with the TemplateCoq substitution.

Yep, that's the root of the issue, you need to preserve the mkApps invariant everywhere and this gets pretty messy (and ultimately, boring). Let's give it a try. @yforster I think the translation will be structurally recursive, if defined as follows:
Fix trans (t : PCUIC.term) {struct t} : Ast.term :=
  match t' with
  | ... => trans u'
  | tApp f u => mkApps (trans f) [trans u]

It shouldn't be too hard to prove [pcuic_to_coq (coq_to_pcuic t) ~= t]  (where ~= is equality plus [(c : t) ~= (fun x : t => x) c] for the cast translation). Or do you see a problem with this?

I think there's a problem, yes.
 And I think in some cases, this is an issue. For instance the tCase rule in Ast.red1 expects a tConstruct b (mkApps ...), but that's not going to be true anymore.
So I guess you need something like
Fix trans (t : PCUIC.term) {struct t} : Ast.term :=
  match t' with
  | ... => trans u'
  | tApp f u => transApps f [trans u]
... end
with
transApp (t : PCUIC.term) (L : list Ast.term) : Ast.term :=
  match t with
  | tApp l r => transApp l (trans r :: L)
  | t => mkApps (trans t) L

I.e. you have to make sure that when converting stacked PCUIC.tApp you put all of the right hand sides in the list, instead of producing stacked Ast.tApp with singleton lists.
Otherwise reducible terms might not be reducible anymore after the roundtrip, and I would guess there are problems with typing as well.

Hmm, but my mkApps (trans f) [trans u] will "unstack" the applications in f if there are any. You're right that we should be careful in the ML to ensure that we don't build ill-formed terms made of Ast.tApp though.

Oops, you're right. I had a wrong definition of mkApps in mind, this is indeed doing the unstacking properly, so everything is fine on this front.

Out of curiosity, what kind of language are you translating @aa755 ? Clearly mkApps is giving trouble to everyone :) I believe we should however keep both representations for the time being. It seems I need a change of interpretation of fixpoints blocks also to ease reasoning in PCUIC, and it’ll be nice to mechanically check I’m proving things about an equivalent theory.

@mattam82 were you asking me about the language I am translating? (I mean, Abhishek didn't comment on this issue). I am working of embedding of mini-ml style (plus inductives and pattern-matching) language in Coq. The idea is to use it as a source for lifting functional programs to Coq in a controlled way (as opposed to hs-to-coq source-to-source approach). I work with Bas Spitters on verifying smart contract languages and many of this languages have functional "core".
I agree, keeping both makes sense, since TemplateCoq syntax should follow the OCaml implementation closely. So having translation PCUIC <-> TemplateCoq would be nice to have (along with proofs of correctness).

I've ported parts of my development to target PCUIC, but it turns out I just pushed the issue with mkApps to another corner. Now inverting anything involving PCUICWcbvEval.eval relation is painful because of the cases with mkApps in conclusion (e.g. cases like tConstruct). Specifically, when proving a case for application, I invert the hypothesis with eval, and one of the cases I have the following in my context: mkApps f agrs = tApp (T⟦ e1⟧) (T⟦ e2⟧), where T⟦ - ⟧ is a translation function. I want to learn something about f from that, but don't see any way of doing this.
I am thinking about defining a relation almost as PCUICWcbvEval.eval, but with values in a separate syntactic category. These values will include vConstr i c args, where args is a list of values. This is how I did it in the semantics to my language. But if I do this, I'll have to show that my relation is also a sub-relation of a small step relation of PCUIC (up to converting values to terms).

I had similar issues, so I added a function to count applications in order to deduce injectivity properties on mkApps, see https://github.com/TheoWinterhalter/template-coq/blob/pcuic-conversion/pcuic/theories/PCUICSafeLemmata.v#L685 and the few lemmata that follow.
I don't know if it can help you.

Thanks @TheoWinterhalter, I'll look if I can use these.

I decided to go back to my translation directly to TemplateCoq AST, since inversion of PCUICWcbvEval seems too problematic. Also, I think, way of evaluating applied constructors does not match with my evaluation of constructors.
With TemplateCoq WcbvEval I see two main issues:

eval_beta evaluates nary application from left to right. This doesn't match the way I do it in my interpreter. Since I have binary application, I do recursion "on the left" traversing nested applications. I fixed this by  "emulating" the nary application evaluation in the interpreter.

usage of mkApps in eval_beta (and, I beleive, in eval_fix). This stuff I don't know how to fix really. I see that mkApps is used in subst as well, so, it looks like if I use the following function instead of mkApps:

Definition mkApps1 (t : term) (ts : list term):=
  match ts with
  | [] => t
  | _ => tApp t ts
  end.

If any nested application should be flattened by subst. Also, terms produces by my translation are guaranteed to have all the application flattened and have no empty applications.
I played a bit with the WcbvEval relation with mkApps replaced with mkApps1; it seems to work for some cases of applications. And with this modified relation I can prove the lambda application case.
How wrong this modification can be, what do you think? Maybe it is possible to prove that this modified relation and the original one give the same results on terms with "well-formed" application.

Ok, I think I understand that I still get nested tApps even though the substitution flattens nested applications. But why it's bad to have nested tApps? I understand that different ways of nesting will correspond to different derivations of WcbvEval.eval relation. But why this is important?

I've modified the WcbvEval.eval relation to make the evaluation of applications more explicit. I've added a mutually defined eval_apps relation that takes care of evaluation of nary applications. Here is a gist link:
https://gist.github.com/annenkov/b89a075598ea84e0f3d3560004a12819
This relation follows the same way as the typing relation with typing_spine.
However, I am not sure how to evaluate fixpoints. I would appreciate some feedback on this.
@gmalecha @TheoWinterhalter @mattam82

An alternative (an maybe a better one) would be to use the following cbv evaluation relation for PCUIC:
https://gist.github.com/annenkov/5b2af547fb7ee018dc18161068071dc7
In this values are a separate type, distinct from term. The conversion function is used to convert values back to terms for substitution. I think, more controlled use of mkApps (only in the conversion function) makes this relation more well-behaved, and so, potentially easier to use. However, I the fixpoint case is indeed would be nice to simplify as @mattam82  mentioned. I my version, I assume that fixpoints are defined by recursion on the first arg.

I'm not sure we can escape mkApps, even if the split of values and terms is somewhat nice in your latest WcbvEval. Can you show where the original problem is when you need to invert on PCUICWcbvEval, e.g. which lemma are you proving?

Hi @mattam82 , sorry for the delay with my reply. The repo with my development is not (yet) public, but I gathered all relevant pieces here: https://gist.github.com/annenkov/10dedceffec2d02da9c094dd0dd62e2e.
Basically, I am proving that interpreter for our language gives the same result as PCUICWcbvEval.eval on the translated terms (with the translation given by expr_to_pcuic) for terminating programs. It seems that the case for the application of lambda works fine, but for the constructor application I have to invert hypothesis like mkApps f agrs = tApp (T⟦ e1⟧) (T⟦ e2⟧) hoping to say something about f.
Basically all the rules with the conclusion of the form Σ;;; Γ |- mkApps f a ⇓mkApps something a' (⇓ is a notation for WcbvEval) are problematic. It seems, these rules "emulate" n-ary application by requiring a term being evaluated to be an iterated application and then evaluating all the terms in a list a in "one go". This is different from what's happening in my interpreter: there, the nested application would be evaluated recursively, essentially in the same way as I did in the alternative version of WcbvEval with separate type for values.


Remove need to click in to initiate

I tried the below and it turns the screen black. I've made it work so that it stays permanent after first click. How can i make it permanent from the beginning?
this line, move it to the bottom of a function and change it to pointer.moved = Math.abs(pointer.deltaX) > 0 || Math.abs(pointer.deltaY) > 0;
You will need to change this function:
function updatePointerMoveData (pointer, posX, posY) {
pointer.moved = pointer.down;
pointer.prevTexcoordX = pointer.texcoordX;
pointer.prevTexcoordY = pointer.texcoordY;
pointer.texcoordX = posX / canvas.width;
pointer.texcoordY = 1.0 - posY / canvas.height;
pointer.deltaX = correctDeltaX(pointer.texcoordX - pointer.prevTexcoordX);
pointer.deltaY = correctDeltaY(pointer.texcoordY - pointer.prevTexcoordY);
}
to this:
function updatePointerMoveData (pointer, posX, posY) {
pointer.prevTexcoordX = pointer.texcoordX;
pointer.prevTexcoordY = pointer.texcoordY;
pointer.texcoordX = posX / canvas.width;
pointer.texcoordY = 1.0 - posY / canvas.height;
pointer.deltaX = correctDeltaX(pointer.texcoordX - pointer.prevTexcoordX);
pointer.deltaY = correctDeltaY(pointer.texcoordY - pointer.prevTexcoordY);
pointer.moved = Math.abs(pointer.deltaX) > 0 || Math.abs(pointer.deltaY) > 0;
}
Here's my code:
script.txt

I added ES module for this amazing project, it's Hover to activate by default, can also be Click:
import WebglFluid from 'webgl-fluid'

WebglFluid(document.querySelector('canvas'), {
  TRIGGER: 'hover', // Can be change to 'click'
})

I added a delay for listener so the black screen didn't occur for now.

@cloydlau Hi, I am also having the issue where I want this to work right at the beginning rather than clicking one time first. Can you let me know how your ES module works, and how I can change it for the JS?

@rckychen Hello, here's the example if you mean using it directly in the browser:
<!DOCTYPE html>
<html lang="en">
<head>
  <style>
  canvas {
      width: 100vw;
      height: 100vh;
  }
  </style>
</head>
<body>
<canvas/>
<script<EMAIL_ADDRESS><script>
window['webgl-fluid'].default(document.querySelector('canvas'))
</script>
</body>
</html>

Hi sorry, I'm new to web dev, so am a little confused. To allow the effect to start immediately, do I need to change that script or something else? @cloydlau

@rckychen It does immediately start by default, you can check the document here.

@rckychen It does immediately start by default, you can check the document here.

No it does not, this TRIGGER line is not in the script.js file by default:  "TRIGGER: 'hover', // Can be change to 'click'" and when I add it nothing happens.

@rckychen It does immediately start by default, you can check the document here.

No it does not, this TRIGGER line is not in the script.js file by default: "TRIGGER: 'hover', // Can be change to 'click'" and when I add it nothing happens.

see https://github.com/cloydlau/webgl-fluid/issues/2

@rckychen It does immediately start by default, you can check the document here.

No it does not, this TRIGGER line is not in the script.js file by default: "TRIGGER: 'hover', // Can be change to 'click'" and when I add it nothing happens.

see cloydlau/webgl-fluid#2

Thanks, I posted before I realized this was a different file.   My bad.


We need more docs!

The documentation hasn't been updated for a few months, and does not reflect the state of the project. I'm removing the docs/ folder from the master repo for now, and pushing it into the branch 'bcyphers/docs' (#80) to avoid confusing newcomers to the project. The folder will be added back to master once it is closer to being ready.

A few docs have been added with #80, but there's a lot more to do to get full documentation coverage.
A few TODO's:

A page which explains how to configure a datarun from top to bottom, including explanations of all the fields in each one of the config files and how the config files interact with command line args
An explanation page for each major component of ATM, and a page that ties them all together. We kinda have one for the Modelhub (database.py), but need them for Worker and Model at least.
Automatic documentation of all the public-facing API functions. That probably starts with model.py, enter_data.py, and worker.py.

The Readme is now completely out of date. I'm on a Mac, so I don't know if the configuration is tuned to this Os.
After I copied the config from the git clone to the egg in the virtual env, and the data from OpenML, I got this, which looks like a mismatch between the run configuration and the pandas expectation.
File "pandas/_libs/hashtable_class_helper.pxi", line 1273, in pandas._libs.hashtable.PyObjectHashTable.get_item
KeyError: u'class'

I don't mind poor documentation, but at least having one known good path to get going would be great.
Is this easier to run in Linux? Or is there another preferred way?

Did you download data from OpenML or use the one that is available here in the repo? While we point to the source data from OpenML, OpenML data does not name the target variable as 'class'.

Hi @gducharme , I think this could be filed as a separate issue, thanks!

Thanks, using the data from the project did the trick. I'm still not clear why I'm having to put in the data/templates and methods from the cloned repo into the venv egg.

I'm still not clear why I'm having to put in the data/templates and methods from the cloned repo into the venv egg.

@gducharme That's a consequence of a known issue: https://github.com/HDI-Project/ATM/issues/91
There is already a PR pending to be merged that fixes it.

We've improved the documentation since this issue


Automatic versioning handling through SwiftPM plugin

Hi,
it's always felt a bit weird to me to have to manually hardcode and maintain the version number as part of my main command, e.g.
@main
struct Command: ParsableCommand {
    static let configuration = CommandConfiguration(
        // ...
        version: "1.0.0",
        // ...
    )
}

I'd typically have my version stored in git tags and the string here and it's a (admittedly) minor pain if I forget to update the version in code and only update the tag.
Swift Package Manager plugins now offer the ability of generating code at compile-time and thus also generate this version information based on the available git repo metadata. There are some packages already that do just that (see for example here, here, and here). It would be great however to have such a plugin built directly into and distributed as part of swift-argument-parser so that I could optionally automatically reference a version string pulled from my git repo there.
Thanks for considering!

Hi @kiliankoe, thanks for the suggestion! It seems like there are so many different ways of managing version numbers that it would be hard to provide all that flexibility in this particular package. From what I can see, all three of those plugins that you referenced would integrate well with ArgumentParser's CommandConfiguration.version setting, but each one uses a different way of generating and surfacing the package/project metadata! I think having other package supply that information is probably the right situation right now.
If you have a proposed design or more information about a standard way of providing versioning information, I'd be happy to discuss further!


Support Firefox on Android

What will it take?  I've got a Galaxy Tab S4 that needs some love.

I'll investigate when I'm done with the bugfixes for 1.54, and multi-item support for Imgur/Instagram (about half done right now, I think).
I've got an Nvidia Shield tablet on Nougat I could use for testing, and I've been looking for an excuse to fix my Android development environment.

Was I not supposed to release 1.54 already?  haha.  Thom was getting pushy.

I'm digging into this now.  It seems there's something Firefox/Android doesn't like about having subfolders inside the extension folder.  It acts like ext/, scripts/, styles/ aren't there and it won't load anything from them, so I can get the unstyled options HTML to show but that's about it.  If it move all the files into the root, it works.  I double checked that I'm including the folders in my zip file.  Any ideas?

Could be something weird going on with manifest.json on Android. Maybe there's format changes for the Android version of Firefox?

Yeah I dunno.  I wrote a PowerShell script to flatten the file tree and fix the references inside the files.  That works for now.
It seems that they want me to use browser.storage.sync instead of localStorage for settings, so I'm going to just insert special code paths for Android to do that without changing what we're doing on desktop.

Looks like it works pretty well all things considered. Another win for WebExtensions I guess.
I'm pretty curious how the performance is on mobile devices given how we're reprocessing rendered DOM nodes.

Seems pretty good to me.  I don't have any performance complaints on my Tab S4.  Once I got the settings fixed, everything seems to basically work fine.


Update wiki for docker on windows

Short Description
To run the bot + map with Docker on Windows, there are a few steps which are not described.
The correct command to run the bot container AND the map container, without Docker's compose are the following:
Bot container (change to mount the web directory):
 docker run --name=bot1-pokego --rm -it -v $(pwd)/configs/config.json:/usr/src/app/configs/config.json -v $(pwd)/web/:/usr/src/app/web/ pokemongo-bot
Map container (no change):
docker run --name=bot1-pokegoweb --rm -it --volumes-from bot1-pokego -p 8000:8000 -v $(pwd)/configs/userdata.js:/usr/src/app/web/userdata.js -w /usr/src/app/web python:2.7 python -m SimpleHTTPServer
Then to access the host of the VM:
docker-machine ip default
Why not just use Docker compose ?
The rendering in the Docker console is not colored.
How it would help others
I spent a few hours yesterday to understand why the map wasn't working.  It was my first time using Docker, and I didn't have enough understanding of it to spot my mistake. A lot of others users may be in the same case.
Info
This proposition follow this issue: #https://github.com/PokemonGoF/PokemonGo-Bot/issues/4083

@supercourgette Can you make a pull request for this?

Done with the best of my aptitude (since I'm not an expert) here : #4136


Need a URL input type and associated validator

import repo requires a valid URL input and fqdn_or_ip does not work here, text is too wide open and could allow errors on the back end

implemented in cnc commit: https://github.com/PaloAltoNetworks/pan-cnc/commit/9878fa0150f64ef06d8ab5ba01de254e00d991ec


sloth-app: init at 3.2

Description of changes

Mac app that shows all open files, directories, sockets, pipes and devices https://sveinbjorn.org/sloth
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 299107 run on aarch64-darwin 1

1 package built:

sloth-app


Address faceCMC

Fix #142
Uses the SF built in CMC to determine the mana cost of the cards (so it will be as expected).
Add faceCMC which will be based on the face of the card.

Far // Away will report 2 and 3 for face.
Delver will report 1 and 0 for the faces.
Budoka Gardener and Dokai will be 2 and 0 for the faces.

DGM.json.txt
V17.json.txt
C18.json.txt

This needs documentation.
CC: @ancestral


⚡ Standardization of rep

Hi @danielbayley, we need to adjust a few things first to move forward, ok? ✨
First, we need to update the screenshot!
Can you take a single capture at the closest possible resolution to 1728x1117?
We also need a separate .md with installation instructions to show the theme on the site.
I ask for these changes to standardize with the other repositories, see an example; this is also important because it helps other potential users who would like to contribute too! 😁
Let me know if something isn't possible or if you have any questions! 🙂

@luxonauta I’m guessing this issue was auto-generated? I don’t think these changes apply to this particular repo, since it’s only for the Homebrew installers…

I believe this is the case! I'm closing the Issue.


ln/path: use Draw2D instead of Cairo for a pure-Go build

This partially addresses fogleman/ln#3, with the benefit of not
requiring OpenGL.  I'm sure the performance is suboptimal, but this
was so simple I couldn't resist.

cool!  let me know if you have concerns, or if there is anything in particular I could do to test this better.

Just trying to figure out why draw2d doesn't respect the join & cap style.

Looks like draw2d is just using this under the covers:
https://github.com/golang/freetype
I might see about using that directly instead.
Thanks for the contribution! cairo can be a pain to install for some people.

Thank you! Super cool library you have here


Support multi-wiki configurations

Currently the script only supports one wiki with one user at a time. This should be changed to allow the script to create backups for multiple wikis at once, and should allow for different user accounts to be used on different wikis. I'm thinking of some sort of config like:
CONFIG = {
  "global": {
    "user": "Username",
    "OAUTH2_TOKEN": "TOKEN HERE",
  },
  "somewiki": {
    "url": "https://somewiki.miraheze.org"
  },
  "someotherwiki": {
    "user": "Other user",
    "password": "PASSWORD",
    "url": "https://someotherwiki.miraheze.org"
  }
}

Done in 9af034a


Adding functions to compute the visual width of east asian characters

Implement function to compute the East Asian Width Property of codepoints as decribed in issue 6.
I am new not only to roc but to functional programming in general, so I expect this code to be unidiomatic. I am open to criticism and to fix any issues.
The results of the function have been tested against other open source libraries.

Thank you for your review!
I added a commit in which I incorporated the functions for getting the EAW property and the width of code points into the CodePoint module, but I'm not sure if that is the way to do it.
I have also added some tests to both the generated module and it's generator.


PLT-7979 Provide support for sending ledger events over a TChan

This is used by a PR in marconi

I'd like to refactor runNode so that the socket-based event streaming code is outside it and is provided via a handler in the list of handlers. However, I'll do that in a follow-up PR.

I've fixed the linting errors (all of which were worth fixing) but we can't do much about the remaining errors, especially the ones that are unrelated to our changes.


fix(sort): reverse directions and better animation

Fixes #6198

I haven't been able to reproduce it with a simple test case (and can't upload my regular site), but with this patch in place (beta 11), I'm seeing the sort header "stem" and indicator appearing horizontally next to one another rather than vertically.
I was able to fix this by (with the debugger) adding a "flex-direction: column" style to the "mat-sort-header-arrow" class.
Would it be possible to do so in the official release?

@mcalmus Please open an issue for bug reports.


⚠ ClamAV has degraded performance

In ca51dd2, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 88 ms

Resolved: ClamAV performance has improved in 193f57a.


Set up issue tracker templates

Flutter is using the new GitHub issue templates: https://github.com/flutter/flutter/issues/new/choose
It would be good to set up something similar for the SDK repo.

Looks like we have a template set up now.


KAS-ECC Component mode and some info for non-Component mode

Plus fixed a bug in TOTP password generation when there was a NULL in the string which caused the key length to be wrong.

TOTP fix looks good 👍


Giving References

A big part of #929
Frontend checklist

[x] Formatted my code with yarn format && yarn lint --fix
[x] There are no warnings from yarn lint
[x] There are no console warnings when running the app
[x] Added any new components to storybook
[x] Added tests where relevant
Has some initial tests, but needs a followup to add more
[x] All tests pass
[x] Clicked around my changes running locally and it works
see below!
[x] Checked Desktop, Mobile and Tablet screen sizes

There is a problem with the host request references, which I think might come from the backend but otherwise I need another set of eyes to look at this. After a confirmed host request has occurred, the reference is available for both parties. After the first person submits a reference, the other person involved with that host request no longer has a reference available. I've tried myself and can't seem to fix it. Depending on what comes up during the review process we might have to raise another Issue to deal with the backend bug (if that's where the problem lies).
Otherwise, clicking around locally works. Here is leaving a friend reference on Desktop:
https://user-images.githubusercontent.com/35485667/118649311-a6dea380-b7e3-11eb-876b-fc7f2bafbbbf.mov
And with a mobile screen size:
https://user-images.githubusercontent.com/35485667/118649826-3be19c80-b7e4-11eb-9d5a-4a27132ea784.mov

Could you rebase this onto develop or merge develop here to get it up to date?

Should be good to review now

Also I just noticed - why are these files under communities? It's probably more appropriate to put these under profile/view as this is not a community feature.

That does make more sense, honestly I think since the issue was listed under the Communities sprint my brain just sort of ran with that and didn't question it 😅 I've moved everything over to profile view and will continue with the rest of you guys' comments with the files in their right place.

sorry for the large volume of comments!

No, thank you for your time!

Regarding the rating, if anyone has suggestions for improving this text please feel free to chime in! It's pretty important for our rating system and I'm not a native speaker 🙂

That bug was now fixed in #1306, let me know if you find any more!

I made the slider a controlled component but still ended up having to use useEffect so that the color is correctly set when someone goes back. Let me know if this is still hard to follow or not up to convention!

it doesn't do the red-to-green change on every value update

I pulled this down and commented out the handleSliderChange line, and it all seems fine to me?

I see what happened now – I use a dark reader because of an eye condition and it's with the dark reader that the color change doesn't work anymore.
With handleSliderChange:

Without handleSliderChange (plus dark reader):

But indeed without handleSliderChange and with normal usage:

Since this is probably then an issue on the dark reader's front should we just ignore it? I am not very sure of the accessibility implications, if it is important/impactful enough to hold on to the imperative code.
Sorry for the confusion, I usually do remember to turn off the dark reader before manually testing the UI but I must have just forgotten when I made that change.

Okay. Let me know.


Inline sorting of traversal result in SELECT projection

OrientDB Version: 3.1.8 (docker)
I don't think this exists yet, so this is probably a feature request.
It would be great to have a way to sort the partial result set returned by a traverse function (e.g. out()). This would make it a lot easier to control the order of nested traversal projections.
Example
SELECT
  /* as method, string argument -> single sort */
  out("hasContainer").sort("id:asc"):{
    id,
    size,
    /* as method, array argument -> multiple sort */
    out("hasContent").sort(["promisedDeliveryDate:desc", "id:asc"]):{
      id,
      dispatchDate,
      promisedDeliveryDate,
      /* Maybe as a function? I think it is a less readable syntax. */
      sort(out("hasLineItems"), "lineId:asc"):{
        lineId,
        name,
        quantity
      } AS lineItems
    } AS contents
  } AS containers
FROM Manifest;

Duplicates #6715.


Support firmware downloading (reading fw from device)

Is your feature request related to a problem? Please describe.
I have an idea that's mostly a curiosity right now: Can we build a firmware database, with easily-usable binaries for different device types? (We probably can't... licensing and whatnot).
Part of this idea involves gathering firmwares that can be tricky to get. Downloading a firmware from a device, would help a lot. Devices with newer firmwares could help efforts to flash older devices, even when the original manufacturer refuses or can't provide new firmware images.
Describe the solution you'd like
In the docs for the z-wave COMMAND_CLASS_FIRMWARE_UPDATE_MD CC, it says:

The Firmware Update Meta Data Command Class may be used to transfer a firmware image to or from a Z-Wave node.

This is for version 4+, and most of the devices I've seen are version 3 - but here's to hoping.
If it was possible, an API to download a firmware image from the device would be beneficial. The downloaded images would be as raw as they could be, without any additional packing or whatnot from manufacturers.. :)
Describe alternatives you've considered
N/A
Additional context
Entirely theoretical, it should be possible to flash one device via. another.. Hmm..
I originally wrote about this on ozw.

For reference:

Seems I missed that entirely 😬

For reference:

Seems I missed that entirely 😬

I'm envisioning the pirate database of firmwares now. Keyed by manufacterer/product ids, and firmware versions.. :D

I'm envisioning the pirate database of firmwares now. Keyed by manufacterer/product ids, and firmware versions.. :D

Yeah, I don't think many devices will offer their firmwares for download if they aren't already available freely.

Yeah, I don't think many devices will offer their firmwares for download if they aren't already available freely.

Spoilsport. :P

From: AlCalzone<EMAIL_ADDRESS>Sent: Tuesday, January 12, 2021 12:32:44 AM
To: zwave-js/node-zwave-js<EMAIL_ADDRESS>Cc: Michael Bisbjerg<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [zwave-js/node-zwave-js] Support firmware downloading (reading fw from device) (#1354)
Yeah, I don't think many devices will offer their firmwares for download if they aren't already available freely.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/zwave-js/node-zwave-js/issues/1354#issuecomment-758290856, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAH2YJ4HA4CF2X7EJRWYSULSZODBZANCNFSM4V6JMJ3Q.

Spoilsport. :P

From: AlCalzone<EMAIL_ADDRESS>Sent: Tuesday, January 12, 2021 12:32:44 AM
To: zwave-js/node-zwave-js<EMAIL_ADDRESS>Cc: Michael Bisbjerg<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [zwave-js/node-zwave-js] Support firmware downloading (reading fw from device) (#1354)
Yeah, I don't think many devices will offer their firmwares for download if they aren't already available freely.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/zwave-js/node-zwave-js/issues/1354#issuecomment-758290856, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAH2YJ4HA4CF2X7EJRWYSULSZODBZANCNFSM4V6JMJ3Q.

I think this is a bad idea as it exposes the project to potential copyright violations for little benefit. It’s one thing to let the user retrieve the firmware and another to distribute that binary without permission.

I think this is a bad idea as it exposes the project to potential copyright violations for little benefit. It’s one thing to let the user retrieve the firmware and another to distribute that binary without permission.

While making it possible doesnt mean that this will be a problem before anyone makes FW public. I know Fibaro doesnt make FW available, forcing people to buy their hub to get updates.

While making it possible doesnt mean that this will be a problem before anyone makes FW public. I know Fibaro doesnt make FW available, forcing people to buy their hub to get updates.

I think this is a bad idea as it exposes the project to potential legal wrangling for little benefit. It’s one thing to let the user retrieve the firmware and another to distribute that binary without permission.

I wouldn't say zwavejs should make the store.. Merely implement the API to fetch the fw.. :)

I think this is a bad idea as it exposes the project to potential legal wrangling for little benefit. It’s one thing to let the user retrieve the firmware and another to distribute that binary without permission.

I wouldn't say zwavejs should make the store.. Merely implement the API to fetch the fw.. :)

@LordMike I agreed, even more so if this is apart of the (official?) Z-Wave specification. Haveing feature parity would be great

@LordMike I agreed, even more so if this is apart of the (official?) Z-Wave specification. Haveing feature parity would be great

I think this is a bad idea as it exposes the project to potential legal wrangling for little benefit. It’s one thing to let the user retrieve the firmware and another to distribute that binary without permission.

I wouldn't say zwavejs should make the store.. Merely implement the API to fetch the fw.. :)

Ok, to be clear I don’t think the function is an issue, but there reason for doing so is to implement the full specification.

I think this is a bad idea as it exposes the project to potential legal wrangling for little benefit. It’s one thing to let the user retrieve the firmware and another to distribute that binary without permission.

I wouldn't say zwavejs should make the store.. Merely implement the API to fetch the fw.. :)

Ok, to be clear I don’t think the function is an issue, but there reason for doing so is to implement the full specification.

Can we build a firmware database, with easily-usable binaries for different device types? (We probably can't... licensing and whatnot).

You would probably be OK legally if you provide firmware images as a public service as it should be allowed according to "fair use" doctrine as long as you only provide the binary files unmodified "as is" so that they can really only be flashed on the same product.
FYI, the Zigbee2MQTT community does almost exactly this for Zigbee devices OTA firmware update images that are not publicly available to be downloaded directly via an URL from the manufacturers, so that can perform OTA upgrades via zigbee-herdsman:
https://github.com/Koenkk/zigbee-OTA
Zigbee2MQTT, however, does not have a feature to download an FW image from a device, (that part is done manually today).
Repo has a JSON file with an index that gets updated each time a new OTA firmware update image is added to that GitHub repo:
https://github.com/Koenkk/zigbee-OTA/blob/master/index.json
Their collection is limited because they have decided to not collect/provide OTA firmware update images for Zigbee devices where the manufacturer already publicly provides OTA firmware via an URL.
https://github.com/Koenkk/zigbee2mqtt/issues/1696
https://github.com/Koenkk/zigbee-herdsman/issues/60
There is also a similar discussion about creating an OTA community repository for zigpy where they talk about PROS and CONS here:
https://github.com/zigpy/zigpy/issues/154

Can we build a firmware database, with easily-usable binaries for different device types? (We probably can't... licensing and whatnot).

Part of this idea involves gathering firmwares that can be tricky to get.

I'm envisioning the pirate database of firmwares now. Keyed by manufacterer/product ids, and firmware versions.. :D

FYI, the Zigbee2MQTT community does almost exactly this for Zigbee devices OTA firmware update images that are not publicly available to be downloaded directly via an URL from the manufacturers, so that can perform OTA upgrades via zigbee-herdsman:
https://github.com/Koenkk/zigbee-OTA
Zigbee2MQTT, however, does not have a feature to download an FW image from a device, (that part is done manually today).
Repo has a JSON file with an index that gets updated each time a new OTA firmware update image is added to that GitHub repo:
https://github.com/Koenkk/zigbee-OTA/blob/master/index.json
Their collection is limited because they have decided to not collect/provide OTA firmware update images for Zigbee devices where the manufacturer already publicly provides OTA firmware via an URL.
https://github.com/Koenkk/zigbee2mqtt/discussions/2921
https://github.com/Koenkk/zigbee2mqtt/issues/1696
https://github.com/Koenkk/zigbee-herdsman/issues/60
There is also a similar discussion about creating an OTA community repository for zigpy where they talk about PROS and CONS here:
https://github.com/zigpy/zigpy/issues/154

I think this is a bad idea as it exposes the project to potential legal wrangling for little benefit. It’s one thing to let the user retrieve the firmware and another to distribute that binary without permission.

While making it possible doesnt mean that this will be a problem before anyone makes FW public. I know Fibaro doesnt make FW available, forcing people to buy their hub to get updates.

It is probably OK legally if you provide firmware images in a repo as a public service as it should be allowed according to "fair use" doctrine as long as you only provide the binary files unmodified "as is" so that they can really only be flashed on the same product.
https://en.wikipedia.org/wiki/Fair_use

This is just wrong. And I AM a lawyer.

It is probably OK legally if you provide firmware images in a repo as a public service as it should be allowed according to "fair use" doctrine as long as you only provide the binary files unmodified "as is" so that they can really only be flashed on the same product.
https://en.wikipedia.org/wiki/Fair_use

This is just wrong. And I AM a lawyer. Fair use doesn't cover this.

Fair use might be the wrong term, I am not a lawyer, but do you really think that some the Zigbee2MQTT project is running a huge risk of being sued for providing Zigbee device OTA images which are not publicly available via URL from the original manufacturer?
Again, it is not like the Zigbee2MQTT project is hacking the firmware, they are providing those OTA firmware image files unmodified which means that "as is" they are only useful to users that own exact same type and model of product from that manufacturer.
If it then more likely that any manufacturer who would not be happy with their firmware images being provided in a GitHub repository by a third-party like this would only submit a cease and desist letter?
In that case that cease and desist letter would probably only come from one of those manufacturers, and if so it would be easy to comply by simply removing the image files from that manufacturer.
PS: Please do not blow this all out of propositions as I was just stating it as a fact that Zigbee2MQTT has such OTA collection which from that software can only be used to update the firmware of exact same products which the manufacturer intended it for.

GitHub isn't the proper forum for a legal debate. I think a copyright letter would go to GitHub and GitHub can and does remove or limit accounts for hosting copyrighted material. You're also confusing the likelihood of enforcement with legality. Many manufacturers do in fact go after websites in such situations.

Side note: The entire issue depends on the devices actually responding to the download commands.

I think a copyright letter would go to GitHub and GitHub can and does remove or limit accounts for hosting copyrighted material. You're also confusing the likelihood of enforcement with legality. Many manufacturers do in fact go after websites in such situations.
Regardless, I would hope and assume that it is a non-issue in practice for this use-case and thus a moot point here. That is, the fear of potential legal consequences should hopefully not stop anyone here from considering the idea of implementing the same type of concept of hosting a repository with a collection of OTA image for Z-Wave devices that Zigbee2MQTT community has here today:
https://github.com/Koenkk/zigbee-OTA

GitHub isn't the proper forum for a legal debate.

I agree.

This forum isn't meant for this type of debate. We're going to stop spamming everyone now.

Now the the timeout is over and thread cleaned up, I'm unlocking this again. In speaking with @AlCalzone we have no interest in hosting firmware images. Let's please limit discussion to the requested function to download firmware from the device, if doing so is even possible.

I landed on this issue because I was hoping this functionality existed. I have several otherwise identical devices with different firmware versions that were obtained at different times. The oldest firmware has an acknowledged bug but the company has not released an upgrade while newer devices come with a patched version. All I want to be able to do is copy the firmware from one device to another, which it seems I should have a right to do as an owner of the device. This seems like a pretty clear cut legitimate use case.

And by the way, thanks to the developers of zwave-js for the great work. I'm in the process of moving from SmartThings to Home Assistant and so far, the process has been very smooth. Working with the combination of zwave-js and Home Assistant has been a vastly better experience so far than working with SmartThings.

I landed on this issue because I was hoping this functionality existed. I have several otherwise identical devices with different firmware versions that were obtained at different times. The oldest firmware has an acknowledged bug but the company has not released an upgrade while newer devices come with a patched version. All I want to be able to do is copy the firmware from one device to another, which it seems I should have a right to do as an owner of the device. This seems like a pretty clear cut legitimate use case.

Iirc, the ability to download a firmware was removed in the FIRMWARE_MD v3 or something like that. At least, that's what I remember from the docs, that older versions had this idea that you could do stuff with the firmware, while later versions went away from it. I can only suspect that the original intent was like yours, while the later intent was influenced by "proprietary" software - but I'm guessing.. :)

Oh Wow. I must have completely misunderstood the docs for that command. I was certain they removed it :O
My bad.
Mike.

From: kpine @.>
Sent: Tuesday, January 24, 2023 3:56:46 PM
To: zwave-js/node-zwave-js @.>
Cc: Michael Bisbjerg @.>; Mention @.>
Subject: Re: [zwave-js/node-zwave-js] Support firmware downloading (reading fw from device) (#1354)
According to the spec, your device must support Firmware Update Meta Data Command Class v5 or later for it to be possible to download its firmware (the functionality hasn't been removed in later versions). If you use Home Assistant, you can download the device diagnostics file and it will tell you which version of the CC is supported, I could not find this information in ZUI. You can also search the Z-Wave Products DBhttps://products.z-wavealliance.org/regions and sometimes it tells the version.

v1-4: The Firmware Update Meta Data Command Class may be used to transfer a firmware image to a ZWave device.
v5-8: The Firmware Update Meta Data Command Class may be used to transfer a firmware image to or from a Z-Wave node.

The Firmware Update Meta Data Command Class, version 5 also introduces the support firmware download from the Z-Wave node to the controller.
The following commands are introduced:

Firmware Update Meta Data Prepare Get Command
Firmware Update Meta Data Prepare Report Command

—
Reply to this email directly, view it on GitHubhttps://github.com/zwave-js/node-zwave-js/issues/1354#issuecomment-1402081989, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAH2YJ4EE2OL7XA6X7X2H73WT7US5ANCNFSM4V6JMJ3Q.
You are receiving this because you were mentioned.Message ID: @.***>

If I'm looking at the right thing in the diagnostics file, the Firmware Update Meta Data version for the device whose firmware I wanted to download is v4. So yeah, guess it's not going to happen. The version for the devices with the older firmware is v2! Thanks for your help, I really appreciate your work on this.


Add mocks of optional libraries for autodoc

This should fix the last of the ReadTheDocs autodoc build errors.

Pull Request Test Coverage Report for Build 65

0 of 0   changed or added relevant lines in 0 files are covered.
75 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.4%) to 88.104%

Files with Coverage Reduction
New Missed Lines
%

tortoise/backends/base/executor.py
1
95.53%

tortoise/backends/asyncpg/schema_generator.py
3
100.0%

tortoise/backends/asyncpg/executor.py
9
100.0%

tortoise/backends/asyncpg/client.py
62
68.83%

Totals

Change from base Build 63:
-0.4%

Covered Lines:
1599

Relevant Lines:
1765

💛  - Coveralls

@Zeliboba5 Please review/merge.
I have no idea why the coverage changed, as this didn't actually touch any code.


CD example source code "AI Techniques for Game Programming"

Hello :)
Hey Crazy Skady can you share the original CD source code of "AI Techniques for Game Programming by Matt". I bought used and don't have the CD. In the author site don't have any link to download the source code example. T_T
Can you help me :) Thank you!
regards
J

Hi, John
Absolutely, give me an email, I will email it to you. :)
BR/Wei

Hi!
just a casualness but Mat answered my email today :D he send me a link.
Thank you crazyskady.

by the way I forked you repo just for study :) thanks

:) You are always welcome.


stdenvAdapters.useLibsFrom: optionally specify coreutils

Description of changes

Allows specification of coreutils when using stdenvAdapters.useLibsFrom. Additionally, switches to an attribute set to make the purpose and order of the arguments more apparent.
When cross-compiling, wrapCCWith by default uses coreutils from pkgs, not buildPackages, causing cross-compilation builds under wrapped environments to fail to run because common utilities like date and time are built for hostPlatform, not buildPlatform.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

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

[!
Template log archival command:
PR=281371; \
RUN_NUMBER=13; \
SYSTEM="aarch64-linux"; \
CUDA_CAPABILITIES="7_5"; \
tar -cvf "$SYSTEM-cap-$CUDA_CAPABILITIES-pr-$PR-$RUN_NUMBER.tar.zst" \
  -I "zstd -T0 --ultra -22" \
  -C "$HOME/.cache/nixpkgs-review/pr-$PR-$RUN_NUMBER" \
  .

Logs are made available at https://drive.google.com/drive/folders/1GgABhwa2ooKeZXMqf5He6PcyClNrE6cQ?usp=share_link

✅ aarch64-darwin
Result of nixpkgs-review pr 281371 --extra-nixpkgs-config '{     allowUnfree = true;     allowBroken = false;     cudaSupport = false;     cudaCapabilities = [ ];   }' run on aarch64-darwin 1
✅ x86_64-darwin
Result of nixpkgs-review pr 281371 --extra-nixpkgs-config '{     allowUnfree = true;     allowBroken = false;     cudaSupport = false;     cudaCapabilities = [ ];   }' run on x86_64-darwin 1
✅ aarch64-linux
✅ CUDA (Jetson)
Result of nixpkgs-review pr 281371 --extra-nixpkgs-config '{     allowUnfree = true;     allowBroken = false;     cudaSupport = true;     cudaCapabilities = [ "7.5" ];   }' run on aarch64-linux 1
✅ CUDA (Non-Jetson)
Result of nixpkgs-review pr 281371 --extra-nixpkgs-config '{     allowUnfree = true;     allowBroken = false;     cudaSupport = true;     cudaCapabilities = [ "7.2" ];   }' run on aarch64-linux 1
✅ Non-CUDA
Result of nixpkgs-review pr 281371 --extra-nixpkgs-config '{     allowUnfree = true;     allowBroken = false;     cudaSupport = false;     cudaCapabilities = [ ];   }' run on aarch64-linux 1
✅ x86_64-linux
✅ CUDA (Non-Jetson)
Result of nixpkgs-review pr 281371 --extra-nixpkgs-config '{     allowUnfree = true;     allowBroken = false;     cudaSupport = true;     cudaCapabilities = [ "7.2" ];   }' run on x86_64-linux 1
✅ Non-CUDA
Result of nixpkgs-review pr 281371 --extra-nixpkgs-config '{     allowUnfree = true;     allowBroken = false;     cudaSupport = false;     cudaCapabilities = [ ];   }' run on x86_64-linux 1

Interesting! I now wonder what would buildPackages.wrapCCWith do in the useLibsFrom definition

Updated, rebased, and renamed the PR.
@SomeoneSerge can you give it another round of review? I minimized the changes so all this PR does is pass coreutils from targetStdenv.

@SomeoneSerge I added your comment fix and included a reference to #283517. Can you take another look? Pending your approval and CI passing, I'd like to merge this.


Updated Simulator to send DeviceId in body of event message.

Also added Hive query that shows count of events by device.
Connected to #26.

Hi @francischeung, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (fcheung). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;


增加缓冲区大小，处理换行符

Checklist

[ ] npm test passes and diff coverage over 90%
[ ] tests and/or benchmarks are included
[ ] documentation is changed or added
[ ] commit message follows commit guidelines

Affected core subsystem(s)

Description of change

Codecov Report

Merging #2 into master will increase coverage by 0.81%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master       #2      +/-   ##
==========================================
+ Coverage   81.53%   82.35%   +0.81%
==========================================
  Files           4        4
  Lines          65       68       +3
==========================================
+ Hits           53       56       +3
  Misses         12       12

Impacted Files
Coverage Δ

app/extend/utils/sse_utils.js
94.28% <100%> (+0.53%)
:arrow_up:

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 908b2c3...6c4d5dc. Read the comment docs.


Cannot get a list of VIT running PIDs with vit --list-pids

For some reason, I cannot use the refresh all instances documented here: https://github.com/vit-project/vit/blob/2.x/CUSTOMIZE.md#auto-refreshing-vits-interface
Checklist:
Ubuntu 21.10
I followed the instructions and added:
pid_dir =/tmp/vit_pids
In my INI config.
vit-external-refresh.sh is executable and in my $PATH
on-exit-refresh-vit.sh is present in taskwarrior hooks folder
Versions used:
vit --version = 2.2.0
task --version =2.6.2
When I start vit --list-pids instead of the running PIDs it just starts a new instance of VIT (even if an instance is already running). The folder /tmp/vit_pids is never created and even after manually creating it, it still remains empty.
Am I missing anything here?

What you're reporting is not making a whole lot of sense.
The parser code is a pretty simple decision tree, it's not clear to me how executing vit with the --list-pids arg would not run the correct branch of code: https://github.com/vit-project/vit/blob/2.x/vit/option_parser.py#L45-L53
You can add this in the parse_options() method right after the options are parsed:
    import debug
    debug.console(options)

You can see in my install it correctly sets the list_pids option, which takes it down the branch to grab and output the pids:
$ vit
Namespace(list_actions=False, list_pids=False)
$ vit --list-pids
Namespace(list_actions=False, list_pids=True)
1485335
1344548

Closing due to inactivity


Upgrade to electron 0.30.7

Upgrades Electron to prevent a file handle leak on Windows.
Refs https://github.com/atom/electron/issues/2871
Closes https://github.com/atom/atom/issues/8834
Closes https://github.com/atom/atom/issues/8882

/cc @nathansobo

/cc @zcbenz thanks for the quick release of this fix :+1:

Thanks @kevinsawicki for diagnosing this and @zcbenz for addressing it promptly.

Yeah, I'm so glad you guys tracked this down. Awesome :sparkles:.


Fix restart bug after theme or language change

do not restart app if user entered language menu and selected same language
after theme or language was changed: an unexpected restart of app happens when we completely exit settings: add a dialog warning before the restart
after language or theme change: immediately restart, important for language

Previously:

when we enter language, even if we do not change it, app will restart
restart was really unexpected (no warning) and only happening after exit settings to main app. For language, it is better to restart immediately to avoid layouts not properly applied on all activities and context. Less code maintenance later. Apply language without restart is possible, but code maintenance to ensure it is not worth it. users usually set the language once and restart is ok.

Thank you for your changes.
Merge your changes. However, the results of changes made in ActivitySetting will still be reflected in ActivitySetting.
We believe it's important for users to make decisions based on the results of the changes they've made.
Best regards.

thank you, works perfectly


daos: copy hdf5 containers to and from POSIX filesystems

If daos-api=hdf5, then h5repack is invoked. This is for copying to/from DAOS HDF5 containers. If the daos-api flag is not set, then the default is to move hdf5 files to DAOS POSIX containers. Also, h5repack is serial, so if more than one rank is specified when an HDF5 container is the source or destination the program will exit with an error explaining only one process is supported currently.
Signed-off-by: Danielle Sikich<EMAIL_ADDRESS>

This PR should be merged after #457

Leaving a note here that the functionality in this PR is only supported if commit dc6306dbed9698d9f6e218c5b3ec15c7f9d9044c from the HDF5 DAOS VOL is present. Also, this PR needs commit 506a8bda1ce81dd039e10e809f716c7f2d544a9b for DAOS.

This looks good.  I made a couple comments, but nothing to hold up the merge.  Thanks.

This looks good. I made a couple comments, but nothing to hold up the merge. Thanks.

@adammoody thanks!


Convert autorest gulp files to invoke/tox

Current autorest is designed around gulp as task invoker:
https://github.com/Azure/autorest.python/tree/master/.gulp
Since autorest for Python is now in Python, gulp is not the preferred choice. Let's move to something more Python friendly.

Task was done part of https://github.com/Azure/autorest.python/issues/114 by @iscai-msft


Feature/ cdssm

add cdssm preprocessor for #328

Codecov Report

Merging #396 into 2.0 will not change coverage.
The diff coverage is 100%.

@@         Coverage Diff         @@
##            2.0   #396   +/-   ##
===================================
  Coverage   100%   100%
===================================
  Files        35     36    +1
  Lines      1038   1109   +71
===================================
+ Hits       1038   1109   +71

Impacted Files
Coverage Δ

matchzoo/preprocessor/__init__.py
100% <100%> (ø)
:arrow_up:

matchzoo/preprocessor/cdssm_preprocessor.py
100% <100%> (ø)

matchzoo/preprocessor/process_units.py
100% <100%> (ø)
:arrow_up:

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 99dc94a...13ffcff. Read the comment docs.

unfixed text length (means window number) data can not feed in cdssm model


1.20.1 Quilt crashes

So i try to use the mod with the quilt loader but it just doesn't even load, it just straight up breaks the game before it starts, is this iusse due to a dependency i need or is there something else going on.
For the record it doesn't show any error code, just the minecraft launcher with the following text.

The crash log will be in your .minecraft folder. I can't help without one.

I'm unsure of if they are having the same issue as me, but this issue is the closest I could find and very recent.
I believe this is just an issue with the Quilt Kotlin Libraries having a lower version of FLK than this mod requires.
The latest release of QKL labeled as compatible with 1.20.1 includes version 1.10.10 of FLK, whereas particle core and fzzy config require 1.10.18.
This can be fixed by just installing version 4.0.0 or higher of QKL, as despite it being labeled as only compatible with 1.20.4 it will still work on 1.20.1
Crash log with ver. 2.1.3 of QKL: https://mclo.gs/BwYAWHw
Related issue in QKL repo that made me aware of the mc version weirdness: https://github.com/QuiltMC/quilt-kotlin-libraries/issues/93
Related issue asking for QKL to include several compatible versions on modrinth: https://github.com/QuiltMC/quilt-kotlin-libraries/issues/95


Phantom crash

There's a peculiar phantom crash happening in react_on_rails tutorial app test suite
Crash occurs at spec/features/shared/contexts.rb:51
When test execution briefly stopped with byebug before line:51 - test passes
Strace (got full 30k+ lines log too)
What seems to be telling:
open("/home/lapsa/.rvm/gems/ruby-2.3.1/gems/websocket-driver-0.6.4/lib/websocket/driver/hybi.rb", O_RDONLY|O_NONBLOCK|O_CLOEXEC) = 16
(..)
read(16, "module WebSocket\n  class Driver\n"..., 8192) = 8192
read(16, "ror, \"Unrecognized frame opcode:"..., 8192) = 3890

Found on stackoverflow something seemingly related about frame opcodes and web sockets
My uneducated guess is that either:

something (what?) is using outdated WebSocket protocol
or there is some kind of race condition problem in Capybara::Poltergeist::WebSocketServer

Any ideas what's going on?

The most likely cause of this is PhantomJS which we have no control over.  What version are you using?  Could you try running it with the PhantomJS 2.5 beta - https://groups.google.com/forum/#!topic/phantomjs/AefOuwkgBh0 -, and master branch of Poltergeist?

Used PhantomJS 2.1.1
Having some troubles testing beta version at the moment.
Unsure how to get that up and running on Arch Linux.

Any update on this? Were you able to try PhantomJS 2.5 ?

Sorry... Real life happened and completely forgot about this.
Will give it another shot this weekend.


Cut / Copy is not working

I started a new instance. Then ran 'docker swarm init'. The command worked but I could not highlight the text then right-click to copy. Nor could I use ^c.
Is there another keyboard combo I should be using?

Ctrl+shift+c/v
sent from mobile
On Dec 2, 2016 12:42 AM, "David Medinets"<EMAIL_ADDRESS>wrote:

I started a new instance. Then ran 'docker swarm init'. The command worked
but I could not highlight the text then right-click to copy. Nor could I
use ^c.
Is there another keyboard combo I should be using?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/franela/play-with-docker/issues/65, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ABgV2pK4dE6LhdkgKqb5DhaxCIlg2CpAks5rD5OigaJpZM4LCKzE
.

@medined agreed, it's a real issue on Firefox using Windows, because I'm able to copy using right click, but I'm not able to paste after this... but at least, I can copy/paste using right click on Chrome using Windows
@marcosnils this trick doesn't work on Windows (either using Chrome or Firefox)

@Oliboy50 haven't tested on windows. I'll give it a try

@marcosnils after trying harder, it seems that CTRL+SHIFT+V work on Chrome, but CTRL+SHIFT+C opens the Chrome console panel instead of copy stuff

@Oliboy50 @akalipetis I believe this could be configured in xterm.js but we haven't looked in detail. Woudl you like to investigate and send a PR maybe?

Fixed.
Added new shortcut to copy with ctrl + alt + v. Couldn't change default pasting due tu browser permissions.
Also, just found that ctrl + insert (copy) and shitft + insert(paste) should also work.

I tried what you suggested, but none of this worked in Firefox on Windows :/

@Oliboy50 ctrl + shift +v should work for pasting as well as shift + insert. Can you please verify?

@marcosnils

ctrl + shift + v is the Adblock Plus default shortcut for opening the filtered elements list of the current page... so it does something else for me
shift + insert does not work... However, it prints ^[[2~ when I use it twice

ctrl + shift + v is the Adblock Plus default shortcut for opening the filtered elements list of the current page... so it does something else for me

I don't see that there's much we can do if a non-default browser plugin is interfering with xterm.js shortcuts =(. Maybe you can change Adblock plus default shortcut?

@marcosnils I deactivated the Adblock Plus shortcut, but ctrl + shift + v does not work, it does nothing at all :(

seems like there is an issue opened on xterm.js side https://github.com/sourcelair/xterm.js/issues/418

@Oliboy50 interesting. I'll check it out and come back to you here.

@Oliboy50 just downloaded latest firefox 51.0.1 in Windows 10 and shift + insert works perfectly. ctrl + shift + v doesn't work though.

There is a paste button in the Firefox all-purpose dropdown menu, just above the "zoom-in +", which works with xterm.js, when using Firefox-on-windows.

Ctrl + Insert to copy
Shift + Insert to paste

:thumbsup:

ctrl+shift+c ....this is command open inspector of the page
using Ubuntu+Firefox
is there another command or solution for that?

@alaagalal yes. Use ctrl + insert to copy and shift + insert to paste.

Is it possible to just use Ctrl+C and Ctrl+V for copying and pasting?

Nope because the browser overrides it. You need to use ctrl + insert and shift + insert

Nope because the browser overrides it. You need to use ctrl + insert and shift + insert

It's quite strange. ctrl+insert doesn't work for copying and shift+insert works for pasting. What can I check for getting more data? Thanks

Ctrl + Insert to copy
Shift + Insert to paste

this worked perfectly fine for me

Hi,
I am facing the same problem on Win10. I tried all mentioned tricks in the browsers Chrome, Firefox and Edge.
The editor (e.g. Nano, Vim) inside a PWD master is blocking as far as the Shift+Insert or context menu function for paste is used.
Furthermore, I tried this also on another laptop (Win 7) with the same negative result.
I tried some other solutions like, using the PWD Editor, but there I can just drag the file, which I would like to upload and see it's content. Further functions like import, save, etc. is not available.
Lastly I tried to use Putty / WinScp to upload the config file (docker-compose.yaml) but this attempt, fails during establishing the ssh connection, with the error " server send pubic-key".
Some other ideas, to upload or paste file contents to PWD?
Thanks and BR,
Peter

I tried some other solutions like, using the PWD Editor, but there I can just drag the file, which I would like to upload and see it's content. Further functions like import, save, etc. is not available.

You can just drag & drop a file into the terminal and it'll get uploaded to the current working directory you're standing in.

I am facing the same problem on Win10. I tried all mentioned tricks in the browsers Chrome, Firefox and Edge.

Strange.. I've tested window setups in the past and never seen issues with copy / paste. I don't have a windows machine, but once I get one I'll try to repro

Thanks a lot. Drag & Drop into the terminal window did the job. Great.

to copy (CTRL+INSERT)
to paste (SHIFT+INSERT)

in 2021 following 2 works well
Ctrl + Insert to copy
Shift + Insert to paste

in 2021 following 2 works well
Ctrl + Insert to copy
Shift + Insert to paste

thanks marcos

The following worked for me using Microsoft Edge browser CTRL + FN + E to copy CTRL + SHIFT + v to paste

Thanks This works

The following worked for me using Microsoft Edge browser CTRL + FN + E to copy CTRL + SHIFT + v to paste

Thank you! This works.

on windows with MS-Edge browser following cmd will work
COPY with ctrl+Insert+ c Paste with ctrl+SHift+v

I have a hp elitebook 840 g8 and it's keyboard does not have the INS key, what else should I do to perform the copy command?

to copy (CTRL+INSERT) to paste (SHIFT+INSERT)

Not working in chrome.
Only ctrl+shift+v working for paste but unable to find copy shortcut.

to copy (CTRL+INSERT) to paste (SHIFT+INSERT)

Working in Chrome on Windows 11👍

@marcosnils after trying harder, it seems that CTRL+SHIFT+V work on Chrome, but CTRL+SHIFT+C opens the Chrome console panel instead of copy stuff

Thanksssss

I am on window 10, browser chrome
labs.play-with-docker.com provides linux
the commands worked for me are -
for copy       ct+fn+insert
for paste      ct+shift+v

Ctrl+shift+c/v sent from mobile
…

If you are using a browser like Opera, this shortcut is a bit of a torture :) Because they are shortcut for developer tools.
I suppose that ctrl & shift + insert is useful in all browser.

Ctrl+shift+c/v sent from mobile
…

@marcosnils If you are using a browser like Opera, this shortcut is like a bit torture :) Because they are shortcut for developer tools.
I suppose that ctrl & shift + insert is useful in all browser.

Ctrl + Insert to copy
Shift + Insert to paste

this worked perfectly fine for me

Thanks. this is working windows 10 in crome

Microsft Edge
CTRL + FN + E to copy (couldn't get to work)
CTRL + SHIFT + v to paste (worked)
Chrome (Both commands worked)
Ctrl + Insert to copy
Shift + Insert to paste

to copy (CTRL+INSERT) to paste (SHIFT+INSERT)
simple, great and useful. may thanks

I am on window 10, browser chrome labs.play-with-docker.com provides linux the commands worked for me are - for copy ct+fn+insert for paste ct+shift+v

yes its working for me on windows machine

Windows Operating System | Chrome Browser

COPY = CONTROL + INSERT
PASTE = SHIFT + INSERT


Fix start point and end point of removing sriov_ bridge stanza.

This fixes start point and end point of removing sriov_ bridge stanza.
deploy script removed sriov_ bridge stanza overly.
On OpenNebula 4.12.1, oned creates bridge stanza like:
(snip)
			<mac address='02:00:c0:a8:01:96'/>
                        <model type='virtio'/>
                </interface>
                <interface type='bridge'>
                        <source bridge='sriov_pt'/>
                        <mac address='02:00:0a:01:80:01'/>
                        <model type='virtio'/>
                </interface>
                <graphics type='vnc' listen='<IP_ADDRESS>' port='6001'/>
        </devices>
(snip)

line=$[$line - 2] points at previous </interface> line,
and count=$[$count + 3] points at next <graphics type='vnc'  line.

Added a fix to dynamically find the boundaries of the stanza to be removed.


Can you upload

No description provided.

Can you upload what?

No description provided.

Can you upload the "train_vls.py" file, thanks a lot.

Thank you for the concern of our work.
I have some deadlines recently. I will start to organize the code when I am available. Notice that in the last commits, we extended our method to support image editing using StyleGAN and a pretrained language model CLIP, which frees us from training another text encoder.


Add link to github repo on website

I'd like to close this issue. I would like to know what page or on which section should the github repo link be added?

@mannyanebi You can follow the Jekyll Layout and Themes
One option is update the "footer" and the "config.yml" with the GitHub and Discord links

Oh nice. I'd check out then. Thanks

I definitely think the footer is a good option.

@mannyanebi can pull the PR #63 and send the change to the PR
Just need uncomment the GitHub url for social links in the [config.yml](https://github.com/oleksis/blackpythondevs.com/blob/6f31e545c945e25e82b27e2670f3e5c1a373a843/_config.yml#L41-L57]

Thanks @oleksis. This helped me alot. I'm taking a crash course on jekyll to learn and understand developing with jekyll.

More Discussion about the website stacks


automating shell commands and ec encode/decode

My goal is to be able to periodically check ec volumes and decode the ones that have had a certain threshold of files deleted in order to compact them and then re-encode when they fill back up again.
It looks like volume file deletion data is available within some of the shell commands, but I'm not sure how to automate any of those since they are in an interactive shell. Is there a way to run them from an external script or can what I want to do be done in the http api? Or am I thinking about this the wrong way?

Does this work for you?
echo "lock; ec.decode .... ; unlock" | weed shell

yeah that works, I can totally work with that, thank you!

I'm actually wondering now, is it even possible to get the amount of deleted files/bytes for an ec volume without converting back to a normal volume first? seems like the http api only gives info about normal volumes