forge coverage shows 0.00% when using deployUUPSProxy

Hi,
I am using Upgrades.deployUUPSProxy to deploy an upgradeable contract. When running forge coverage it shows 0.00% for the upgradeable contract, even though I am certain to go through some of the branches:
| File                          | % Lines       | % Statements   | % Branches    | % Funcs       |
|-------------------------------|---------------|----------------|---------------|---------------|
| src/vaults/ERC20Vault.sol     | 0.00% (0/17)  | 0.00% (0/21)   | 100.00% (0/0) | 0.00% (0/14)  |

I tried disabling upgrade safety checks with the unsafeSkipAllChecks option but the output stays the same.
Here is what the deployment function looks like:
    function deployERC20Vault(
        address vaultAsset,
        string memory vaultName,
        string memory vaultSymbol,
        uint256 deployerKey,
        bool unsafeSkipAllChecks
    ) internal returns (address) {
        Options memory deployOptions;
        deployOptions.unsafeSkipAllChecks = unsafeSkipAllChecks;

        vm.startBroadcast(deployerKey);
        address proxy = Upgrades.deployUUPSProxy(
            "ERC20Vault.sol",
            abi.encodeCall(
                ERC20Vault.initialize,
                (
                    vm.addr(deployerKey),
                    IERC20(vaultAsset),
                    vaultName,
                    vaultSymbol
                )
            ),
            deployOptions
        );
        vm.stopBroadcast();

        return proxy;
    }

and a test where I call a function from the upgradeable contract through the proxy (vault is already cast as ERC20Vault beforehand):
    function testERC20VaultWithdrawAssets() public delegatorDeposited {
        // Act
        vm.prank(DELEGATOR);
        vault.withdraw(DEPOSIT_AMOUNT, DELEGATOR, DELEGATOR);

        // Assert
        assertEq(vault.totalAssets(), 0);
        assertEq(vaultAsset.balanceOf(DELEGATOR), STARTING_BALANCE);
        assertEq(vault.totalSupply(), 0);
    }

Here is the output I get when doing a more manual deployment with ERC1967Proxy:
| File                          | % Lines         | % Statements    | % Branches    | % Funcs        |
|-------------------------------|-----------------|-----------------|---------------|----------------|
| src/vaults/ERC20Vault.sol     | 76.47% (13/17)  | 71.43% (15/21)  | 100.00% (0/0) | 78.57% (11/14) |

Deployment function:
    function deployERC20Vault(
        address vaultAsset,
        string memory vaultName,
        string memory vaultSymbol,
        uint256 deployerKey,
        bool unsafeSkipAllChecks
    ) internal returns (address) {
        Options memory deployOptions;
        deployOptions.unsafeSkipAllChecks = unsafeSkipAllChecks;

        vm.startBroadcast(deployerKey);
        ERC20Vault vault = new ERC20Vault();
        ERC1967Proxy proxy = new ERC1967Proxy(
            address(vault),
            abi.encodeCall(
                ERC20Vault.initialize,
                (
                    vm.addr(deployerKey),
                    IERC20(vaultAsset),
                    vaultName,
                    vaultSymbol
                )
            )
        );
        vm.stopBroadcast();

        return address(proxy);
    }


[BUG] v7.3.1 script not working

Go Hass Agent Version
v7.3.1
Describe the bug
Maybe it's not a bug, i could miss something ?
My go-hass-agent starting well, no crash ,   but my script still not working, i have starting with --debug and i see :
RN Cannot run script error="could not unmarshal script output" script=/home/debian32/.config/com.github.joshuar.go-hass-agent/scripts/check_version_ha_agent_json.sh
the script have a good right, and the output is :
{   "schedule": "@every 1h",   "sensors": [     {       "sensor_name": "go hass agent version_installed",       "sensor_icon": "mdi:package",       "sensor_state": "7.3.1"     }   ] }
I would check the current go-hass-agent vesion installed  to automate  warning when new version.
and don't see any new sensor.
Thanks for this great tool ;)

Hey @Erreur32 that's super odd. I just tested on my machine by creating a script that simply echos that JSON output, and the agent accepts it and generates a sensor.
The "could not unmarshal script output" means the agent couldn't parse the output as either JSON, TOML or YAML (i.e., it doesn't recognise the output). Maybe the script is outputting some strange/unexpected whitespace?
You can use dasel to validate the JSON output of the script. Once you have dasel installed, a command like:
/home/debian32/.config/com.github.joshuar.go-hass-agent/scripts/check_version_ha_agent_json.sh > /tmp/script.output.json && dasel validate /tmp/script.output.json && rm /tmp/script.output.json

Should print out pass /tmp/script.output.json or a fail (some error)... if it failed to validate.
If it is possible, can you share the script and I can see if I can reproduce with the actual script?

Good point for validiting the json,  first time i need do that.
dasel validate .config/com.github.joshuar.go-hass-agent/version_installed.json
fail .config/com.github.joshuar.go-hass-agent/version_installed.json could not unmarshal data: could not decode object: invalid character '\x1b' in string literal
Error: 1 files failed validation

And apparently, the go-hass-agent version command  also returns this ANSI control sequence along with the actual version.
--> "v7.3.1[0m"
So I filtered it and now it seems to work :)
sed 's/\x1B[[0-9;]*[a-zA-Z]//g'
my working script (for example propose):
generate_json_output() {
    script_name="go hass agent version_installed"
    script_version=$(sudo -u debian32 /home/debian32/.local/bin/go-hass-agent version | grep -oP '(?<=go-hass-agent: ).*' | sed 's/\x1B\[[0-9;]*[a-zA-Z]//g')

    # Creation JSON file (creating a file , in case of)
    cat <<EOF > /home/debian32/.config/com.github.joshuar.go-hass-agent/version_installed.json
{
  "schedule": "@every 1h",
  "sensors": [
    {
      "sensor_name": "$script_name",
      "sensor_icon": "mdi:package",
      "sensor_state": "$script_version"
    }
  ]
}
EOF
}

generate_json_output

cat /home/debian32/.config/com.github.joshuar.go-hass-agent/version_installed.json

Question:  On launch, the script doesn't seem to update, so you have to wait until the next calendar to get an update, is that correct?

I'm glad it's now working! Yes, the output of the version command has some escape sequences to colour the output:

As an alternative to your script, I could add a sensor that shows the agent version that only updates whenever the agent is started?
Hmm, I think you can only remove a sensor if it is no longer provided by whatever was sending it. Otherwise you could just hide it. You can follow the instructions for customising entities:
Select the entity, either from the frontend or by selecting the info button next to the entity in the Developer Tools “States” tab.
Select the cog icon in the right corner of the entity’s dialog.
Toggle the "Enabled", "Visible" switches, or click "DELETE" depending on your preference.

As an alternative to your script, I could add a sensor that shows the agent version that only updates whenever the agent is started?

As there is no automatic update, this seems like a good idea to me.
i have made a card and script to check online version on Github also:


Change name of volume provisioned by openebs

This will change the name of the volume provisioned by OpenEBS.
Earlier, it used to create volume with name pvc-[unique-id]
which was quite difficult to recognize.Hence, changing it to
openebs-volume-[unique-id].
Closes : #356

Sample output after applying this change:
ubuntu@kubemaster-01:~/openebs/k8s$ kubectl get pods
NAME                                                              READY     STATUS    RESTARTS   AGE
jupyter-server-2764185079-zdqdx                                   1/1       Running   0          33m
maya-apiserver-1733699612-hcbgj                                   1/1       Running   0          35m
openebs-provisioner-1512667371-025df                              1/1       Running   0          35m
openebs-volume-d692ff6c-96db-11e7-ad39-021c6f7dbe9d-ctrl-3bh7tm   1/1       Running   0          33m
openebs-volume-d692ff6c-96db-11e7-ad39-021c6f7dbe9d-rep-246pvlc   1/1       Running   0          33m
openebs-volume-d692ff6c-96db-11e7-ad39-021c6f7dbe9d-rep-24kpzn5   1/1       Running   0          33m
ubuntu@kubemaster-01:~/openebs/k8s$ kubectl get pv
NAME                                                  CAPACITY   ACCESSMODES   RECLAIMPOLICY   STATUS    CLAIM                            STORAGECLASS      REASON    AGE
openebs-volume-d692ff6c-96db-11e7-ad39-021c6f7dbe9d   5G         RWO           Delete          Bound     default/jupyter-data-vol-claim   openebs-jupyter             33m
ubuntu@kubemaster-01:~/openebs/k8s$ kubectl get pvc
NAME                     STATUS    VOLUME                                                CAPACITY   ACCESSMODES   STORAGECLASS      AGE
jupyter-data-vol-claim   Bound     openebs-volume-d692ff6c-96db-11e7-ad39-021c6f7dbe9d   5G         RWO           openebs-jupyter   33m

At the moment we assume the PV has a deterministic name https://github.com/kubernetes-incubator/external-storage/blob/master/lib/controller/controller.go#L751 so that we don't create a volume if it was somehow already created. So this PR will change behavior of the provisioner somewhat, are you okay with that?
perhaps we can add an optional function to the Provisioner interface where you define getProvisionedVolumeNameForClaim yourself.

@satyamz  Ideally keeping pvc-{uniqueid} is good enough, do you have any specific requirement to be met by having a custom name ? One of the advantage is that, it gives an abstraction and at any point of user can define their PVC name as they want.

./assign @humblec
.

Hi, @humblec & @wongma7 ,  Our intention to change the name is simple, we want to recognize the volumes provisioned by OpenEBS provisioner. Just for usability purpose. If this not the good practice to follow and will break things up then I will not proceed with this.
@humblec :

One of the advantage is that, it gives an abstraction and at any point, user can define their PVC name as they want.

I agree with you on this. However, I think we can manage this behavior in the case with custom pvc name (if we follow the design suggested by @wongma7).
@wongma7 The method suggested by you sounds good, but I need to think more about the design. Currently, I am going through lib/controller.
I would love to have more pointers towards this apart from what you've already mentioned. I want to give it a try :slightly_smiling_face:

, Our intention to change the name is simple, we want to recognize the volumes provisioned by OpenEBS provisioner. Just for usability purpose
@satyamz There are other ways, have a annotation as in other provisioners.

@humblec I agree that annotations will definitely work in this case. As I was having look at kubernetes dashboard and I was thinking about how to let users know that what kind of volumes are provisioned as soon as users click on volumes.
I also liked the idea of having a generic method to the Provisioner interface which is better than only modifying single provisioner. If this is fine with you then shall I raise an issue and start working on it? and also let's close this PR and focus on the other.

Yes, @satyamz I think a new method would be nice. IIRC you are not the first to want to name the pvs something more useful. However, we should make the method optional by creating a new interface that extends the Provisioner one. I'm not sure what it would be called. https://stackoverflow.com/questions/44577539/go-optional-methods-in-an-interface/44577611#44577611 Let's discuss further if need be in the issue

@wongma7  That's great! I am going to close this PR. I will create the new issue for the addition of optional method. Thanks, @wongma7  & @humblec


Maintenance: Support for Node.js 20

Summary
Node.js 20 official docker image is available in preview: https://github.com/aws/aws-lambda-base-images/issues/91#issuecomment-1737694321. We can start preparing to support the new runtime in Powertools for AWS Lambda (TypeScript).
Why is this needed?
Because we aim at supporting the new runtime the same day it's launched.
Which area does this relate to?
No response
Solution
No response
Acknowledgment

[X] This request meets Powertools for AWS Lambda (TypeScript) Tenets
[ ] Should this be considered in other Powertools for AWS Lambda languages? i.e. Python, Java, and .NET

Future readers
Please react with 👍 and your use case to help us understand customer demand.

This is now released under v1.15.0 version!


report: Final Thoughts (part of Conclusion)

Completed as of rev 35cec5953ea33a7a14380f6c99c11ca260258c71. Actual time spent: 2h.


Is it possible to have integration tests and app run on 2 different repositories

Hello,
I have 2 repositories, one with all my app code written and another one with my end to end integration tests to automate my mobile application. I want to maintain 2 repositories because i plan to outsource my testing to another company in the future.
Is this something can be achieved in flutter using integration tests library.
PS: I'm using integration tests library to automate my mobile application.

Hi, @akramcshive
This issue seems like a discussion for either StackOverflow or one of the other flutter communities in https://flutter.dev/community#community-grid.
If you have tried something already and are running into issues, kindly update the issue to reflect the issues you are experiencing.
Closing from here as this is neither an issue with flutter nor a feature request.
If you disagree, kindly leave a comment below and I will reopen.
Thank you


fix(app): fix infinitely hanging return to dashboard

Possibly closes RABR-672

Overview
We gate the "return to dashboard" by a success close run response, but we probably should do this just onSettled.

Test Plan and Hands on Testing

Ran protocols on the ODD and clicked return to dashboard.

Review requests

does this seem reasonable?

Risk assessment
low

Yeah it makes sense to me. I'd be a bit worried that we'd orphan runs though, do we know why the close failed?

No, I don't. I'll discuss with @SyntaxColoring and dig around a bit myself before merging!

Hm, so you're suggesting that the close request raced against something else that was also closing the run, and lost, and that's what made the spinner spin infinitely?

Or the server should return a 2xx for that PATCH instead of a 409. Doing current: false on a run that's already current: false is just as much a successful no-op as it is a conflict.


#9 fixed: public url used for cloning

avoid access rights check for the git clone command by using the public url

Good catch, @lifecoderua! Merging


Fixed impossible array return type hints

A quick pass on the impossible array returns, as per #132
Keeps happy:

PHPStan, as configured by the project
PhpStorm, at current stable (2022.3.1)
Psalm, at dev-master, specifically about this issue (I had trouble with stable phar crashing against the project, not sure why, Psalm phar is generally touchy for me)

Psalm has more things to complain about, but a lot of them are in tests and would probably need to be added to CI for a serious pass. Not sure if there is interest, with PHPStan in use?
Fixes #132

Indeed I will not start using psalm has PHPStan check is already enough IMHO. Thanks for the PR


Fix reply filter for open_revs

Current filter removing all not_found's from the final reply in presence of any {ok, Doc} in it, which is not always the correct behaviour, since we can have missing doc for one of the passed revisions. In general, in absence of latest attribute, we are expecting one reply per revision which could be either #doc or not found.
This fix's making sure that we are removing not_found's only for the revisions that also have {ok, Doc} reply.
COUCHDB-3097

+1 after adding a comment or tweaking that bit of logic to be a bit less subtle.

@davisp I did both, does it look better now?

100% better. I kept staring at the {not_found, missing} tuple in both clauses of the fold and my brain just couldn't process what was going on for awhile. This is much better.
+1 and good find on that.


Build workflow trying to push container image on pull_request triggers

The container build workflow tries to push the image when it's only run on a PR even though we are excluding it in the job step logic with an if statement.

I might have been using the wrong syntax. I have {{ }} around the expression currently, but in this example they don't wrap it in curly braces.

I removed the curly braces around the if expression in c89eda2. I'll open a test PR to see if this issue is resolved.

The curly braces didn't make a difference, but I was able to resolve this by using
if: ${{ github.event_name == 'push' || github.event_name == 'workflow_dispatch' }}
instead of
if: ${{ github.event_name != 'pull_request' }}
No idea why the latter doesn't work considering it evaluates correctly in the docker push step. Oh well, it's working now.


feat: add encodeable utilities for chart

🏆 Enhancements
Define encoding config for a visualization

Each visualization has one or more channels that can be encoded, such as x, y, color, stroke, fill. These channels configurations should be consistent/sharable across all visualizations. For example, x and y usually have scales and axes. Scales have domain, range. Axes have label, format. Color can support categorical, sequential or fixed color. All channels can support fixed value, field name in the data, or sometimes aggregates (haven't implemented the aggregate yet).
Each encoding channel config is based on vega-lite encoding grammar. (See line chart example, vega-lite encoding documentation)
Only adopt a small subset of vega-lite encoding grammar for now as we don’t have enough time (and may not intend) to provide 100% feature of vega-lite.

Implement TypeScript types for the encoding config.

First attempt was using all types from vega-lite package (XXXFieldDef) directly but ended up getting too overwhelmed.
Now adopting the primitive types from vega-lite then write higher-level types that has same name but only with the subset of the fields, so we can keep that of what we support and slowly adopt more in the future instead of taking it all now which our encoder does not know how to handle and also make it harder to implement the encoder. (preview)

Write an encoder that parses the encoding config mention above and become a helper during visualization rendering.

There is a base Encoder class and each visualization such as line chart will implement its own Encoder that extends from the base class. See: base Encoder class, Line Chart's Encoder

Long shot

Once all these encoding configs code are a bit more stable, it could be extracted to @superset-ui/encodeable.

Addressed most of the comments.


Added setters for callback and gateway.

Reformatted

Thank you @lpasek. New version 1.3 was just released based on your contribution.


Java Driver: added documentation about ArangoCursor.close

Reverted changes to 3.4, 3.5, 3.6, 3.8. These versions will later be symlinked by #487


feat(search) Show a no results placeholder when no results are available

Adds placeholder text ("No Results Found") when no search results are available.

Note: this result does not have a hover state as there are no actions that can be taken with it
Before:

After:

This is awesome, thank you!


Extraneous temporary and copy in implicit byref parameter

Using the tuple form of a call that has a byref parameter, if the surrounding code is sufficiently complex, the compiler emits an extraneous temporary and copy.
open System
open System.Collections.Generic

[<EntryPoint>]
let main argv =
    let dict = Dictionary<int, DateTime>()
    let result, elem = dict.TryGetValue(0)
    let b =
        if result then
            if elem.Day > 0 then
                Console.WriteLine ""
                true
            else false
        else false
    0

Expected behavior
Only one local is generated and no extra copy happens, i.e. generated code is equivalent to:
    let mutable V_1 = DateTime()
    let result = dict.TryGetValue(0, &V_1)
    let b =
        if result then
            if V_1.Day > 0 then ...

Actual behavior
Looking at the IL, this generates 2 DateTime locals:
       [1] valuetype [mscorlib]System.DateTime V_1,
       [2] valuetype [mscorlib]System.DateTime V_2,

V_1 is passed into the call, then V_1 is copied into V_2, then V_2 is used. In effect this is equivalent to the following code:
    let mutable V_1 = DateTime()
    let result = dict.TryGetValue(0, &V_1)
    let V_2 = V_1
    let b =
        if result then
            if V_2.Day > 0 then ...

I tried simplifying this example as much as possible, but at this point removing anything results in good generated code, without the copy, so it seems to depend a lot on the surrounding logic.
This makes a big difference in code that heavily relies on passing large structs by reference, e.g. System.Numerics.Vectors.Matrix4x4 etc; it wastes stack space and time copying. The JIT doesn't optimize this away.
Known workarounds
Don't use the tupled form, use a mutable local and pass it explicitly.
Related information
Visual Studio 2017 15.2 (26430.6).

Intellisense even don't show, that you can do this though it does not complain if you use such syntax.

@asik If you get a chance to try out https://github.com/Microsoft/visualfsharp/pull/5110 on some performance benchmarks for System.Numerics.Vectors.Matrix4x4 etc. that would be great

Closing this, as #5110 is merged.


verilog preproc: Assert `ifdef_already_satisfied' failed in frontends/verilog/preproc.cc:758.

Run the Verilog frontend on the following:
`ifndef a
`ifdef b
`endif
`else
`endif

I'm pretty convinced the easiest way to get this right is to just implement a stack with the three possible states. It makes things a lot easier to reason about. This is what I did for the sv2v preprocessor.

Eh, it's not that hard to fix with the current model, I'll have a patch in a few minutes.
The stack would help with better error reporting though (like checking for double else)


Expected non-null name in imported array child

Bug description
The ConstVector cannot be converted to the ArrowSchema. This results in an exception being thrown, specifically Expected non-null name in imported array child. The exception occurs because the name of the value schema is not set here. Even when setting the name to 'values', the following exception still persists: Operator::getOutput failed for [operator: TableWrite, plan node ID: 2]: NotImplemented: Unhandled type for Arrow to Parquet schema conversion: run_end_encoded<run_ends: int32, values: int64>.
  ArrowSchema arrowSchema;
   const bool isScalar = (type->size() == 0);
   const bool constantSize = 100;

   // If scalar, create the constant vector directly; if complex type, create a
   // complex vector first, then wrap it in a dictionary.
   auto constantVector = isScalar
       ? BaseVector::createConstant(
             type, variant(type->kind()), constantSize, pool_.get())
       : BaseVector::wrapInConstant(
             constantSize,
             3, // index to use for the constant
             BaseVector::create(type, 100, pool_.get()));

   velox::exportToArrow(constantVector, arrowSchema);
   arrow::ImportSchema(&arrowSchema);

System information
Velox System Info v0.0.2
Commit: 98d6c053c065cabe7e2071e040a9680efb4255f6
CMake Version: 3.16.3
System: Linux-5.4.0-156-generic
Arch: x86_64
C++ Compiler: /usr/bin/c++
C++ Compiler Version: 9.4.0
C Compiler: /usr/bin/cc
C Compiler Version: 9.4.0
CMake Prefix Path: /usr/local;/usr;/;/usr;/usr/local;/usr/X11R6;/usr/pkg;/opt
Relevant logs
No response

@mbasmanova @majetideepak Can you help to look this issue? Thanks.

CC: @Yuhta @pedroerp @kgpai

@mbasmanova @majetideepak @Yuhta @pedroerp @kgpai Do you have any input? Thanks.

@JkSelf, @czentgr opened an issue here https://github.com/facebookincubator/velox/issues/7457.
The solution is to flatten the dictionary and constant vectors. PR https://github.com/facebookincubator/velox/pull/7025

@majetideepak I guess there is also a point to be made that if we don't flat constants while writing to Parquet, we could be smarter and leverage the constant encoding while writing - say we simply write it as a RLE to the Parquet file.

@pedroerp You are right. Now that we moved the Parquet writer code to Velox, we can implement writing Velox constant vector as Parquet RLE
https://parquet.apache.org/docs/file-format/data-pages/encodings/#a-namerlearun-length-encoding--bit-packing-hybrid-rle--3

The Parquet RLE piece is currently not implemented in Arrow as well. https://github.com/facebookincubator/velox/issues/7189#issuecomment-1800915223

#7802 fixes the Arrow conversion issue. Cc: @JkSelf

@pedroerp Thanks for your fix. With https://github.com/facebookincubator/velox/pull/7802 ,  Expected non-null name in imported array child exception no longer exists. However, the following exception occurs:
Operator::getOutput failed for [operator: TableWrite, plan node ID: 2]: NotImplemented: Unhandled type for Arrow to Parquet schema conversion: run_end_encoded<run_ends: int32, values: int64>.

@pedroerp Got it. We currently support the DataWriting operator in Gluten. However, we are awaiting resolution on this issue for static partition write. Are there any plans for upstream support of writing Constant/REE in the Parquet writer in the near future? Thanks.

@majetideepak and @czentgr are the ones working on Parquet reader/writer lately. Do you guys have any plans to look at this issue?

@majetideepak Thanks for your reply. https://github.com/facebookincubator/velox/pull/7025 can address my issues. Does converting the vector to a flattened form affect the performance or not?

Does converting the vector to a flattened form affect the performance or not?

Unfortunately, the performance will degrade. We will prioritize implementing RLE Parquet.

@majetideepak Got it. Thanks for your reply.


attribute names of foreign key condition must have the same name

The following commands in tuts give the wrong error:
authors := relation{tuple{name "agentM"}}
kernel_req := relation{tuple{author "moin", no 0}}
foreign key author_exits kernel_req{author} in authors{name}
ERR: AttributeNamesMismatchError (fromList ["author","name"])

I would expect that the condition is not accepted because of the missing author, not because of the wrong attribute names.
A test with the the same attribute names in both relations gives the correct error.
I use the latest version in the branch support_ghc_8.2

@testexplode brought up this issue on our IRC channel and I asked him to open a ticket. After I mistakenly assumed that this is bug, I have an explanation as to why this behavior is expected.
In the case of a single attribute foreign key constraint, the two attributes must obviously (unambiguously) be the attributes in question to match so the names of the attributes are largely irrelevant. However, if the constraint refers to two-or-more attributes in both sides of the constraint, then, because attributes in a relation do not a specific ordering, then matching attributes for a potential union is ambiguous. For example, for foreign key constraint "x{a,b} in y{c,d}" it is not clear whether "a" maps to "c" or "d". Furthermore, the types of a,b,c, an d could be identical, so guessing based on the types would fail as well.
In this case, it makes sense that the relational algebra would require that the two sides of the constraint be unionable, that is, of the same type/attributes which includes the names of the attributes.
Still, this issue caught me and a user off-guard, so I will improve the documentation and the returned error message.

I cherry-picked the better error message patch to the "support_GHC_8.2" branch.


zulip: init at 4.0.0

Motivation for this change
Add the Zulip chat client to nixpkgs.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
New package.

thanks!


Configure interfaces in zones

This allows you to attach interfaces zo zones

I fixed the whitespace, but please hold the merge - I will verify if other cases also actually work as I expect...

Now it behaves correctly with arrays of interfaces - if you are satisfied with the indenting, the code is ready to merge

Thanks @jovandeginste  - will test it too and hopefully merge shortly, thanks for the contribution


Update symptoms questionnaire as per definition based extraction.

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #625
Description
Clear and concise code change description.
Alternative(s) considered
no
Type
Choose one: (Feature)
Screenshots (if applicable)
Checklist

[x] I have read and acknowledged the Code of conduct
[x] I have read How to Contribute
[x] I have read the Developer's guide
[x] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA
[x] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project
[x] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally
[x] I have built and run the reference app(s) to verify my change fixes the issue and/or does not break the reference app(s)

Codecov Report

Merging #688 (966b6d4) into master (7d9c57f) will decrease coverage by 0.24%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #688      +/-   ##
============================================
- Coverage     54.15%   53.91%   -0.25%
+ Complexity      178      175       -3
============================================
  Files            44       44
  Lines           818      818
  Branches        117      117
============================================
- Hits            443      441       -2
  Misses          308      308
- Partials         67       69       +2

Impacted Files
Coverage Δ

...va/com/google/android/fhir/db/impl/DatabaseImpl.kt
35.55% <0.00%> (-2.23%)
:arrow_down:

.../google/android/fhir/db/impl/dao/LocalChangeDao.kt
66.66% <0.00%> (-2.09%)
: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 7d9c57f...966b6d4. Read the comment docs.


chore: Change version to v1.5.2

Pull Request
Description
Release v1.5.2 of the LPVS project.
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Code cleanup/refactoring
[X] Documentation update
[ ] This change requires a documentation update
[ ] CI system update
[ ] Test Coverage update

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 92.57%. Comparing base (b017074) to head (eb25982).

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #554   +/-   ##
=========================================
  Coverage     92.57%   92.57%
  Complexity      529      529
=========================================
  Files            48       48
  Lines          1737     1737
  Branches        208      208
=========================================
  Hits           1608     1608
  Misses           64       64
  Partials         65       65

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add issue template for EF.Docs

cc @rowanmiller

May be worth mentioning to file issues here for documentation request?

Ping. Worth adding this issue template or no?

:shipit:


Simple latex

Thanks for the great library, is there a way to use unicode_to_latex to get a simple character form, for example, x instead of \ensuremath{\mathit{x}}?

Sure, you can define your own UnicodeToLatexConversionRule (see https://pylatexenc.readthedocs.io/en/latest/latexencode/) with either a RULE_DICT rule type and a manual mapping for a subset of the characters you're interested in (e.g. the mathematical letters), or a RULE_REGEX or RULE_CALLABLE with some logic to replace a set of input characters.

@phfaist I think a simple option to disable the addition of only these commands would be awesome. (E.g. because the MediaWiki Math extension doesn't recognize \ensuremath{}. )

do you mean in the command line tool or in the API?

@phfaist In the API.


Issue #1060, #1205 and #1233: removes local variables if no declaration was found and adds out-of-scope validator

This PR is part of this OpenRoberta/blockly#68 PR and fixes following issues:

removing the declaration block of a local variable did not remove the "assingStmt" and "var" blocks when they are not in scope
out-of-scope variables did not stop the lab from executing and generating false code.

This is now fixed by:

removing "assignStmt" and "var" blocks as soon as their declaration block is removed.
preventing the execution and generation when variables are detected which are not in scope by implementing an out-of-scope variable validator

already merged


Release 1.7.0

As title, all pending works are done already, time to release latest version !

@EragonJ
Great work!
I've researched a bit and I can confirm that Kaku can stream YouTube video at maximum resolution of 720p and 60 FPS video isn't supported. Is there anything you can do about this?

@reddit-is-fun Kaku is mainly based on https://github.com/rg3/youtube-dl and I use the default best option (Because we support many different platforms and they may all have different formats, so I leave the decision to the library itself)
Maybe you can go through more details about that lib and see what we can do here !
:+1:

The audio volume isn't saved (YouTube), I hope you can fix that at some point. The volume always resets to 50% after closing the program.

@reddit-is-fun hmmm ~~ ok i'll try to make it in this milestone.

@neighborhood999 do you have Windows machine ? Right now there is problem when using Kaku on Windows but I don't have machine to test and see what's going on xDD

Yep, I have windows machine, I will try later XD.

@reddit-is-fun I decided to postpone that to next milestone because we need to focus on fixing the critical issue that Kaku doesn't work on Windows. This version has been delayed a lot and I hope this can be fixed soon. Sorry about that :) I'll make a new issue for it.

@neighborhood999 I can successfully run built Kaku on Windows 10 (friend's machine), how about yours there ??

@EragonJ still failed. I already use node 5.1.1 and electron-prebuilt 0.36.0.
My windows machine is 8.1 version, so what's problem in here?

Noticed that there is one critical bug happened in youtube-dl which I need to solve first. Just a note here to let you all know why 1.7.0 is not released yet.

Finally.


Go through all remaining TODOs in the code

As I am adding tests I am also adding TODOs here and there for later refactoring work that may or may not happen. At some point we should go through these TODOs and prioritise.

With #325 and #327, there are roughly two types of TODOs left:

TODOs for new/better tests. Always relevant, but not problematic.
Recent TODOs, about changes to features that we're already planning to add.

As such, I don't think we need this issue any more. Most things tracked here have already been tackled.


Ohai is slow when Chef-Client runs it under a service

When running Chef-Client as a service on Windows, Ohai is very slow. Specifically, the perl and C plugins take quite a while. This behavior only repros running as a service (System Local account)
[2015-04-20T15:03:13-07:00] DEBUG: No hints present for azure.
[2015-04-20T15:03:13-07:00] INFO: #<Ohai::NamedPlugin::Azure:0x3468828> took 0.0
[2015-04-20T15:03:18-07:00] INFO: #<Ohai::NamedPlugin::C:0x3468618> took 4.828096
[2015-04-20T15:03:18-07:00] INFO: #<Ohai::NamedPlugin::Elixir:0x3472698> took 0.032143
[2015-04-20T15:03:18-07:00] INFO: #<Ohai::NamedPlugin::Erlang:0x3472500> took 0.031568
[2015-04-20T15:03:18-07:00] INFO: #<Ohai::NamedPlugin::Go:0x3471b70> took 0.045661
[2015-04-20T15:03:18-07:00] INFO: #<Ohai::NamedPlugin::Groovy:0x3471930> took 0.032917
[2015-04-20T15:03:18-07:00] INFO: #<Ohai::NamedPlugin::Java:0x3470838> took 0.029592
[2015-04-20T15:03:18-07:00] INFO: #<Ohai::NamedPlugin::Languages:0x347b9f8> took 0.0
[2015-04-20T15:03:18-07:00] INFO: #<Ohai::NamedPlugin::Lua:0x347b1e8> took 0.032176
[2015-04-20T15:03:18-07:00] INFO: #<Ohai::NamedPlugin::Mono:0x347b050> took 0.03032
[2015-04-20T15:03:18-07:00] INFO: #<Ohai::NamedPlugin::Nodejs:0x347aa50> took 0.031291
[2015-04-20T15:03:28-07:00] INFO: #<Ohai::NamedPlugin::Perl:0x3479e50> took 9.890613
[2015-04-20T15:03:28-07:00] INFO: #<Ohai::NamedPlugin::PHP:0x3479b50> took 0.046857
[2015-04-20T15:03:29-07:00] INFO: #<Ohai::NamedPlugin::Powershell:0x34795c8> took 0.999975
[2015-04-20T15:03:29-07:00] INFO: #<Ohai::NamedPlugin::Python:0x3478ea8> took 0.032473
[2015-04-20T15:03:29-07:00] INFO: #<Ohai::NamedPlugin::Ruby:0x3478788> took 0.312138
[2015-04-20T15:03:29-07:00] INFO: #<Ohai::NamedPlugin::Rust:0x3478560> took 0.03041
[2015-04-20T15:03:29-07:00] INFO: #<Ohai::NamedPlugin::Chef:0x34682b8> took 0.0
[2015-04-20T15:03:29-07:00] INFO: #<Ohai::NamedPlugin::Ohai:0x347a840> took 0.0
[2015-04-20T15:03:29-07:00] DEBUG: has_ec2_mac? == false

This may affect other scenarios as well, not just running in a service context.

Hasn't been replicated, and my guess is this is general ruby / appbundling / windows perf issues that have been somewhat addressed in Chef-15 and we could do more work around in general.


IMG2IMG picture quality (compression,artifacts)

Describe the bug
I've been doing a lot of img2img and i've been noticing that often the quality of the resulting image contains a lot of what seems JPG compression, compared to how pristine the input image was. Ofcourse only noticeable when using masking, as img2img on the complete image would result in something completely new and not comparable.
To Reproduce
Steps to reproduce the behavior:
use an original image and a masked image, create an img2img and compare the quality of the non-masked area.
Expected behavior
Less compression and artifacts of the non masked area of the image

That's an interesting observation. Are you using the output of img2img as the input to another round repeatedly? I suppose that what could be happening is that the image is being compressed and uncompressed multiple times. PNG actually does use lossy compression, so this is a real concern.
If you think this is what we might be seeing, then we can change the compression level when we write out the generated PNG files to something less aggressive. I'm hoping to get back to working on inpainting tomorrow (Sunday) and I can tweak compression then. Just tell me how to best trigger the problem.

i'm actually not sure what i'm seeing, testing it with a really close up face now, it's like it slightly does change the entire photo and not just the part that was masked. If I overlay them in photoshop , the hairs have slightly shifted , the lip creases, the teeth. and I only masked her earring and replaced it. So it's either compression or SD is doing its best to recreate the original but only gets it 99%

and here look with face that isn't as close up , only masked the balloon and changed it , the girls eyes get deformed

here's a girl not in close up , masked the balloon and switched it out. notice the resulting image where the face of the girl gets deformed anyway.

For the first closeup one, it looks to me like a lack of interpolation on the original image. See the eyelashes made of hard pixels, and the edges of the teeth. The other differences are only noise.
The second one is affected clearly outside the region.

SD downscales all its input images to 64x64 (whether they're pure noise, or a weighted combination of an image + noise), does its magic, and then upscales back up to 512x512 at the other end (somehow). It seems likely the way that the masking works is that it's preventing changes to the 64x64 version, but it's still passing through SD's downscale and upscale process. If so it could in theory be changed to blend back in the original 512x512 version in the unmasked regions, with an opacity gradient around the edge of the mask.

Do you see the non-masked regions changing when both the init image and the mask (if you are using one) are exactly 512x512 (or whatever your specified destination dimensions are)? Otherwise there's going to be some resizing and resampling going on, and information will be lost. In addition, it doesn't help that we are using the image library's default PNG compression level of 9. This causes artifacts that become increasingly evident as you pass the image through many img2img cycles.
In any case, inpainting is a bit of a black box to me, and I'm hoping to work more on it with guidance from AUTOMATIC111's work. He seems to have solved most of the issues. Anybody reading this who wants to help would be most welcome to!

here's a test with 512x512 , original , masked and generated
labrador is original,
border collie the generated one
as you can see, the woman got destroyed with pixels

SD struggles to encode and decode small faces unfortunately, using the internal description of weights that it understands an image by before rebuilding it at the other.
It's something I've been discovering while doing textual inversion - all my embeddings are (correctly) fitting to messed up faces since that's what the training data turns into when it goes into SD's model (I'm trying to counteract it by writing custom training prompts for those images where they are described as blurry/messed up/corrupted faces so that the embedding is trained to produce those faces with that description, and hopefully can produce more normal faces when that description isn't used in actual prompts).

I know it's been a while, but this feature request finally came to the top of my queue. I have just posted PR #1127 which adds a --png_compression option that you can use to reduce the compression level or turn it off entirely. This should fix the JPG-like compression artifacts.

It sounds like this might not be entirely from writing pngs with compression (unless Pillow does that in-memory). I'm experiencing this on the node branch as well (where everything is kept in-memory). The quality loss from img2img with a mask is pretty substantial.

Turns out that this was a problem in the Autoencoder. If you get and load the VAE file vae-ft-mse-840000-ema-pruned.ckpt, it goes a long way towards improving the img2img quality.


Fix npc spawning at avatar position while the app is loading

Hide npc rendering till physics enabled

Works Fine.

I think we want to make the whole npc app, including the character controller, to be in the correct place, not just make it invisible while loading.
I think this is more about setting the position/matrixWorld early enough directly from the initialized app.

Fixed npc position by update avatar instead of hiding on loading time.


[T4A3][F09-B2]Bernard Tan Ke Xuan

Ready for review

Some comments added. Please close the PR after reading comments.


Update cats-effect, cats-effect-laws, ... to 3.3.5

Updates

org.typelevel:cats-effect
org.typelevel:cats-effect-laws
org.typelevel:cats-effect-testkit

from 3.3.1 to 3.3.5.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel" } ]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #135.


az account get-access-token failing for resource 'https://database.windows.net/'

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az account get-access-token
Errors:
Get Token request returned http error: 400 and server response: {"error":"invalid_resource","error_description":"AADSTS500011: The resource principal named 'https://database.windows.net/' was not found in the tenant named ***. This can happen if the application has not been installed by the administrator of the tenant or consented to by any user in the tenant. You might have sent your authentication request to the wrong tenant.\r\nTrace ID: *****\r\nCorrelation ID: *****\r\nTimestamp: 2020-05-06 13:30:38Z","error_codes":[500011],"timestamp":"2020-05-06 13:30:38Z","trace_id":"*******","correlation_id":"*******","error_uri":"https://login.microsoftonline.com/error?code=500011"}
Traceback (most recent call last):
pip-install-7cl6xofh\knack\knack\cli.py, ln 215, in invoke
...

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az account get-access-token --subscription {} --resource 'https://database.windows.net/'

Expected Behavior
An access token is returned I can use for Azure sql authentication
Environment Summary
Windows-10-10.0.18362-SP0
Python 3.6.6
Installer: MSI

azure-cli 2.5.1

Extensions:
azure-devops 0.16.0

Additional Context

add to S170

To shed a little light, the following will get me a token:
az account get-access-token --subscription {} --resource https://database.windows.net
But this won't:
az account get-access-token --subscription {} --resource 'https://database.windows.net/'

I am not able to reproduce it anymore:
> az account get-access-token --resource https://database.windows.net
{
  "accessToken": "...",
  "expiresOn": "2020-07-15 12:16:16.784885",
  "subscription": "0b1f6471-1bf0-4dda-aec3-cb9272f09590",
  "tenant": "54826b22-38d6-4fb2-bad9-b7b93a3e9c5a",
  "tokenType": "Bearer"
}

> az account get-access-token --resource https://database.windows.net/
{
  "accessToken": "...",
  "expiresOn": "2020-07-15 12:16:23.451931",
  "subscription": "0b1f6471-1bf0-4dda-aec3-cb9272f09590",
  "tenant": "54826b22-38d6-4fb2-bad9-b7b93a3e9c5a",
  "tokenType": "Bearer"
}

For SQL service, now both resource IDs are supported:

https://database.windows.net
https://database.windows.net/

If I remember correctly, there is a known issue that some resource providers require the trailing slash / while some don't:
https://github.com/Azure/azure-cli/blob/81ff7bfad67383a9d8aba2e2e0bd5a7457b0f8b9/src/azure-cli-core/azure/cli/core/cloud.py#L243-L259
+ @azureSQLGitHub to help confirm.


hyperium says failed to patch optifine at the end of the installation what can I do? help me please I am new

when i try to install hyperium it tells me what i put in the title i am using mac help me please

optifine???

this is not the hyperium game, this is hyperion ambilight...

thanks @tpmodding


ADO 2178475

Docs Build status updates of commit 3e41343:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-gp/installation/installation-checklist.md
:bulb:Suggestion
View
Details

dynamics-gp/upgrade/company-data-conversion.md
:bulb:Suggestion
View
Details

dynamics-gp/upgrade/system-requirements.md
:bulb:Suggestion
View
Details

dynamics-gp/index.md
:white_check_mark:Succeeded
View

dynamics-gp/installation/microsoft-sql-server-configuration.md
:white_check_mark:Succeeded
View

dynamics-gp/resources.md
:white_check_mark:Succeeded
View

dynamics-gp/upgrade/introduction.md
:white_check_mark:Succeeded
View

dynamics-gp/upgrade/system-preparation.md
:white_check_mark:Succeeded
View

dynamics-gp/upgrade/upgrade-checklist.md
:white_check_mark:Succeeded
View

dynamics-gp/web-components/introduction.md
:white_check_mark:Succeeded
View

dynamics-gp/installation/installation-checklist.md

Line 74, Column 7: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: /sql/?view=sql-server-2017. Otherwise, remove the view parameter.

dynamics-gp/upgrade/company-data-conversion.md

Line 245, Column 1: [Suggestion-column-header-missing] Column headers are required for tables. Add appropriate header text. NOTE: This Suggestion will become a Warning on 1/29/21.

dynamics-gp/upgrade/system-requirements.md

Line 2, Column 8: [Suggestion-duplicate-titles] Title 'System requirements - Dynamics GP' is duplicated in: 'upgrade/system-requirements.md(2,8), web-components/prerequisite-software.md(2,8)'. Titles must be unique within a docset.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Cherry-pick #59 into release/2.0 branch

Signed-off-by: Niladri Halder<EMAIL_ADDRESS>This PR cherry-picks #59 into release/2.0 branch.

bors try

bors try

@niladrih the upgrade operator is not for release/2.0? If not then we don't need this cherry pick?

Tested and verified that OnDelete updateStrategy make manual-delete available on the same helm upgrade as the one in which it is introduced, i.e. changing the updateStrategy to OnDelete on 2.1.0 helm chart will not delete the io-engine pods when upgrading from 2.0.0 to 2.1.0, there is no need to change the upgrade strategy on the previous release (as I had assumed). Closing this PR.


[Improvement] Submit the module to packagist.org

To improve the accessibility of your module you can submit it at packagist.org. In this way the user does not have to add a repository node to his composer.json.

Hi @lamasfoker,
I added core, contact-sync and order-sync modules to packagist.org
I also updated the README of the modules adding a preferential route for installation via composer with repman.io


bugfix(#2462)

Description
fixes #2462
Contribution Checklist:

[ ] The pull request only addresses one issue or adds one feature.
[ ] The pull request does not introduce any breaking changes
[ ] I have added screenshots or gifs to help explain the change if applicable.
[ ] I have read the contribution guidelines.
[ ] Create an issue and link to the pull request.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.
Publishing to New Package Managers
Please see here for more information.

Merged.
Thanks for taking care of this @sanjai0py !


Linux AOT via flutter build bundle

Building an application on a Linux machine using flutter build bundle is problematic.

The debug build will register and run the Linux Dart code.  Stuart Morgan has stated this is definitely a bug.  I'm building the bundle on Linux why would it not include Linux platform specific code?
Using the built bundle to create an AOT will drop the Linux Dart code, and the Linux specific Dart code is not executed.

When running flutter build bundle on a Linux machine (platform type is literally Linux), why would it not include the Linux specific Dart code?
Given flutter build linux is currently not capable of cross compiling, one cannot run flutter build linux and use the libapp.so (it's host only).
This is impacting developers targeting AGL Flutter.  We (AGL/Toyota) want support for Linux specific Dart when using flutter build bundle for both Debug images and AOT images.
We want flutter build bundle and AOT generation to build/run the Linux specific Dart code regardless of the native CMake stuff.
@Hixie @timsneath @cmc5788

The debug build will register and run the Linux Dart code.  Stuart Morgan has stated this is definitely a bug.  I'm building the bundle on Linux why would it not include Linux platform specific code?

I have answered this at length in both https://github.com/flutter/flutter/issues/103660 and our discussion in Discord. Capturing the core bit from the latter:

The thing that is not running is not "the Linux Dart code", it is the 1P-Linux-embedding-defined plugin registration function for the class that is declared in the 1P Linux embedding metadata for that plugin (in its pubspec).

Continuing to conflate "Linux-specific Dart" and "Flutter-1P-Linux-embedding-specific Dart" isn't constructive.

We (AGL/Toyota) want support for Linux specific Dart

That support already exists; you can write Linux-specific Dart using Platform.isLinux checks.
If what you want is the ability to auto-generate Dart plugin registration code for your custom embedding, as I've said before we're happy to review designs for such a system, as part of the custom embedding hook support in the tool.
What you are specifically requesting here though, which is for your embedding to get Flutter-1P-Linux-embedding-specific Dart, is a wontfix.

@jwinarske Thanks for filing the bug. For the record, @stuartmorgan is the TL for this area of the code so he's the one who should make the final call on this.
It sounds like there's an ambiguity in your request. We support several Linux-based platforms, for example Android and GTK. Plugins for different platforms are quite different (for example, plugins on Android are typically in Java, whereas on GTK they are in C!).
The core problem, I think, is that we don't yet have extensible logic in the tool so that you can easily add new platforms. We should definitely work on that. https://github.com/flutter/flutter/issues/25377 is where we're tracking that issue. You can discuss this in #hackers-tool on our Discord. As a heavy user of a custom embedding, you may be interested in getting involved in this work; we haven't really started on it in earnest yet but it is an area we are very interested in.

@Hixie Thanks for the constructive response, I'll look into the referenced issue and Discord channel.

Just a heads up.  This was a tooling issue.  Resolved in my AOT generation sequence:
https://github.com/meta-flutter/meta-flutter/blob/kirkstone/conf/include/flutter-app.inc#L228-L326
What I did uncover in my investigation to resolving this was that some apps worked fine with gen_snapshot obfuscation disabled, some did not.  The Flutter Package examples worked, while Gallery and Wonders did not.  Enabling obfuscation enabled Gallery and Wonders to work as expected.  I raised this on the hacker-tools discord channel.


Error 410: Gone (VRT.BE)

Hello, can you help to download that ? :)
With the subtitles it would be perfect.
Thank you in advance.

C:\Users\hehe\Downloads>youtube-dl https://www.vrt.be/vrtnu/a-z/belpop/10/belpop-s10a1
[VrtNU] belpop-s10a1: Downloading webpage
[VrtNU] belpop-s10a1: Downloading JSON metadata
[Canvas] pbs-pub-85aaef91-3bd4-496e-8be3-706e26f698f3$vid-4af6ec24-8876-4409-b003-57fd23adff63: Downloading asset JSON
WARNING: Unable to download asset JSON: HTTP Error 410: Gone
[Canvas] pbs-pub-85aaef91-3bd4-496e-8be3-706e26f698f3$vid-4af6ec24-8876-4409-b003-57fd23adff63: Downloading token
[Canvas] pbs-pub-85aaef91-3bd4-496e-8be3-706e26f698f3$vid-4af6ec24-8876-4409-b003-57fd23adff63: Downloading video JSON
ERROR: An extractor error has occurred. (caused by KeyError('title',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Same problem here

Also not working here,
youtube-dl is up-to-date (2020.11.12)

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://www.vrt.be/vrtnu/a-z/corona-talks/1/corona-talks-s1a1/', '--username', 'PRIVATE', '--password', 'PRIVATE', '--verbose']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2020.11.12
[debug] Python version 3.4.4 (CPython) - Windows-7-6.1.7600-SP0
[debug] exe versions: ffmpeg git-2020-05-10-fc99a24
[debug] Proxy map: {}
[VrtNU] Logging in
[VrtNU] Requesting a token
[VrtNU] corona-talks-s1a1: Downloading webpage
[VrtNU] corona-talks-s1a1: Downloading JSON metadata
ERROR: Unable to download JSON metadata: HTTP Error 410: Gone (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpwrmvn5tt\build\youtube_dl\extractor\common.py", line 632, in _request_webpage
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpwrmvn5tt\build\youtube_dl\YoutubeDL.py", line 2238, in urlopen
  File "C:\Python\Python34\lib\urllib\request.py", line 470, in open
  File "C:\Python\Python34\lib\urllib\request.py", line 580, in http_response
  File "C:\Python\Python34\lib\urllib\request.py", line 508, in error
  File "C:\Python\Python34\lib\urllib\request.py", line 442, in _call_chain
  File "C:\Python\Python34\lib\urllib\request.py", line 588, in http_error_default

Still not working with latest version. Is there an alternative to download the clips provided from vrtnu.be ?

There's a thing you can do to download clips from vrtnu.be, you open the inspector and look for a file ending in .mpd which will come when the video is playing, so you can play the video for a few seconds and download the clip with that mpd file

Hello everyone,
Actually I found just a few days ago a website which does the job.
It downloads in SD but anyways, I did not find anything else. I can give you later the URL as I am at work right now.
If you ever find a solution to grab the subtitles I'll always be interested.
See you later!

Hello everyone,
Actually I found just a few days ago a website which does the job.
It downloads in SD but anyways, I did not find anything else. I can give you later the URL as I am at work right now.
If you ever find a solution to grab the subtitles I'll always be interested.
See you later!

Happy to hear there are other solutions. Can you provide the link to the website which does the job?
About the subtitles: aren't these hard coded in the video on vrtnu? I've downloaded with youtubedl in the past with vrtnu.be and the subs are always hard coded in the movie. (if applicable)

Just updated to new version
youtube-dl is up-to-date (2020.11.17)
Error stays
[debug] System config: [] [debug] User config: [] [debug] Custom config: [] [debug] Command-line args: ['https://www.vrt.be/vrtnu/a-z/corona-talks/1/corona-talks-s1a1/', '--username', 'PRIVATE', '--password', 'PRIVATE', '--verbose'] [debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252 [debug] youtube-dl version 2020.11.17 [debug] Python version 3.4.4 (CPython) - Windows-7-6.1.7600-SP0 [debug] exe versions: ffmpeg git-2020-05-10-fc99a24 [debug] Proxy map: {} [VrtNU] Logging in [VrtNU] Requesting a token [VrtNU] corona-talks-s1a1: Downloading webpage [VrtNU] corona-talks-s1a1: Downloading JSON metadata ERROR: Unable to download JSON metadata: HTTP Error 410: Gone (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpa0zbt1ts\build\youtube_dl\extractor\common.py", line 632, in _request_webpage   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpa0zbt1ts\build\youtube_dl\YoutubeDL.py", line 2238, in urlopen   File "C:\Python\Python34\lib\urllib\request.py", line 470, in open   File "C:\Python\Python34\lib\urllib\request.py", line 580, in http_response   File "C:\Python\Python34\lib\urllib\request.py", line 508, in error   File "C:\Python\Python34\lib\urllib\request.py", line 442, in _call_chain   File "C:\Python\Python34\lib\urllib\request.py", line 588, in http_error_default

The merge/commit has failed. Cam you retry?

Same error here as well, youtube-dl is up-to-date (2020.11.19). Any idea how to fix it?

Problem still exists in version (2020.11.19)
ERROR: Unable to download JSON metadata: HTTP Error 410: Gone (caused by HTTPError()); please report this issue on https://yt-dl.org/bug ...

Well, should I retry or it won't work ?

The fix provided in #27053 seems to work for me, so I guess the fix is either compiling your own version with those additions or waiting for it to be merged

Still not working with latest version. Is there an alternative to download the clips provided from vrtnu.be ?

downloadgemist.nl

Hello, updated to 2020.11.24, not working.
What is the normal delay for a merge to be incoroporated in the official build?

Updated to version 29.11, unfortunately not working.

#26957 hasn't been merged yet, so it's logical that it's not working

What is the normal delay for a merge to be incoroporated in the official build?

It will probably take some time. Clone the repo, enter the directory, and run something like (assuming your user/passwd is in .netrc):
python -m youtube_dl -n https://www.vrt.be/vrtnu/a-z/terzake.relevant/

+1, broken for me as well.

2020.12.31 still not working

With version 2021.01.03 still not working, I guess the fix hasn't merged yet?
C:\YoutubeDL> .\youtube-dl.exe --update
youtube-dl is up-to-date (2021.01.03)
C:\YoutubeDL> .\youtube-dl.exe -F https://www.vrt.be/vrtnu/a-z/spookuil/2018/spookuil/ --verbose
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-F', 'https://www.vrt.be/vrtnu/a-z/spookuil/2018/spookuil/', '--verbose']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2021.01.03
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.19041
[debug] exe versions: ffmpeg git-2020-03-24-e5d25d1
[debug] Proxy map: {}
[VrtNU] spookuil: Downloading webpage
[VrtNU] spookuil: Downloading JSON metadata
ERROR: Unable to download JSON metadata: HTTP Error 410: Gone (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp7n_m4vnl\build\youtube_dl\extractor\common.py", line 632, in _request_webpage
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp7n_m4vnl\build\youtube_dl\YoutubeDL.py", line 2248, in urlopen
  File "C:\Python\Python34\lib\urllib\request.py", line 470, in open
  File "C:\Python\Python34\lib\urllib\request.py", line 580, in http_response
  File "C:\Python\Python34\lib\urllib\request.py", line 508, in error
  File "C:\Python\Python34\lib\urllib\request.py", line 442, in _call_chain
  File "C:\Python\Python34\lib\urllib\request.py", line 588, in http_error_default

With version 2021.01.03 still not working, I guess the fix hasn't merged yet?
C:\YoutubeDL> .\youtube-dl.exe --update
youtube-dl is up-to-date (2021.01.03)
C:\YoutubeDL> .\youtube-dl.exe -F https://www.vrt.be/vrtnu/a-z/spookuil/2018/spookuil/ --verbose
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-F', 'https://www.vrt.be/vrtnu/a-z/spookuil/2018/spookuil/', '--verbose']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2021.01.03
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.19041
[debug] exe versions: ffmpeg git-2020-03-24-e5d25d1
[debug] Proxy map: {}
[VrtNU] spookuil: Downloading webpage
[VrtNU] spookuil: Downloading JSON metadata
ERROR: Unable to download JSON metadata: HTTP Error 410: Gone (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp7n_m4vnl\build\youtube_dl\extractor\common.py", line 632, in _request_webpage
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp7n_m4vnl\build\youtube_dl\YoutubeDL.py", line 2248, in urlopen
  File "C:\Python\Python34\lib\urllib\request.py", line 470, in open
  File "C:\Python\Python34\lib\urllib\request.py", line 580, in http_response
  File "C:\Python\Python34\lib\urllib\request.py", line 508, in error
  File "C:\Python\Python34\lib\urllib\request.py", line 442, in _call_chain
  File "C:\Python\Python34\lib\urllib\request.py", line 588, in http_error_default

the fix will be included in the next release.

the fix will be included in the next release.


GKE credentials are not shown

Hi, we are using GKE plugin version 0.6 and credentials plugin 2.1.18
User tries to create a new job and adds the Deploy in GKE build step. He creates new credential of type "Google service account from private key". After creating the credential the user is unable to see it in the selection box. He does not even see the none option. However, admin users (meaning user with full permissions) can see the credential in the box. If the user is granted with admin permissions then he can also see the credential.

What permissions does your Google service account currently have?

Hi ,
includedPermissions:

container.apiServices.get
container.apiServices.list
container.clusters.get
container.clusters.getCredentials
container.clusters.list
resourcemanager.projects.get

Is this really related to service user acocunt ? This seems like Jenkins is somehow unable to show defined credentials in Service Account Credentials select box.

Hi, thanks for the answer. When I mentioned permissions I mean Jenkins permissions. I don't think Google service account permissions have something to do. These are the permissions the user has in the Jenkins folder (last row)

https://github.com/jenkinsci/google-kubernetes-engine-plugin/blob/bd42d72d6063c6464dec9c84ea25669d57dc44ab/src/main/java/com/google/jenkins/plugins/k8sengine/KubernetesEngineBuilder.java#L371
As you can see here the permission we currently check for is Item.Configure, which I think is carried over from something similar in the GCE plugin, and can be changed to something more permissive like Credentials-View. Also looks like we don't provide a none option in the dropdown in that case either, so thanks for pointing that out.

Thanks for finding the issue. Are you planning then to change it?
El mié., 29 may. 2019 18:10, Stephen Shank<EMAIL_ADDRESS>escribió:

https://github.com/jenkinsci/google-kubernetes-engine-plugin/blob/bd42d72d6063c6464dec9c84ea25669d57dc44ab/src/main/java/com/google/jenkins/plugins/k8sengine/KubernetesEngineBuilder.java#L371
As you can see here the permission we currently check for is
Item.Configure, which I think is carried over from something similar in the
GCE plugin, and can be changed to something more permissive like
Credentials-View. Also looks like we don't provide a none option in the
dropdown in that case either, so thanks for pointing that out.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jenkinsci/google-kubernetes-engine-plugin/issues/79?email_source=notifications&email_token=AGABEMONPJMRWUP3A3R55KLPX2THHA5CNFSM4HP4VM2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWP2ZZA#issuecomment-497003748,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGABEMPXCTHLBY4PYARH4DTPX2THHANCNFSM4HP4VM2A
.

I'll try to get a change out today.

Thanks
El mié., 29 may. 2019 18:48, Rachel Yen<EMAIL_ADDRESS>escribió:

I'll try to get a change out today.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jenkinsci/google-kubernetes-engine-plugin/issues/79?email_source=notifications&email_token=AGABEMMAVKVFMZDHIHVLN2TPX2XUTA5CNFSM4HP4VM2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWP6KFI#issuecomment-497018133,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGABEMMW4GW4LYEK745T7MTPX2XUTANCNFSM4HP4VM2A
.

Hi, by when can we expect a solution for this problem?

Hello, kindly please any update on this issue?
FYI @MartinHajducik
I just checked, and there is still not plugin update available,
thanks for answer

Hi,
Do we have any update here ?

I will cut a release of the plugin today, after which it will arrive in the update center.

@GolanTrevize10 @pavenova @MartinHajducik  Release 0.6.1 has been submitted with the fix, look for it in the update center soon.

Are we doing something wrong ? We are now using  release 0.6.1 . When we add credentials inside of "Deploy to Kubernetes Engine build step" , it is added in project credentials store, but cannot be listed .

Only credentails under located under global store are visible in select menu

Hi, @MartinHajducik, the use of the global domain is intended at the moment. See IAM Credentials for the instructions we provide on creating credentials.
Can you explain a bit more about how you added the credentials to the project credentials store? I am able to add global credentials from that build step, but have not configured other domains.

Hi he means the folder credentials store
On Fri, 7 Jun 2019, 20:44 Stephen Shank<EMAIL_ADDRESS>wrote:

Hi, @MartinHajducik https://github.com/MartinHajducik, the use of the
global domain is intended at the moment. See IAM Credentials
https://github.com/jenkinsci/google-kubernetes-engine-plugin/blob/develop/docs/Home.md#iam-credentials
for the instructions we provide on creating credentials.
Can you explain a bit more about how you added the credentials to the
project credentials store? I am able to add global credentials from that
build step, but have not configured other domains.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jenkinsci/google-kubernetes-engine-plugin/issues/79?email_source=notifications&email_token=AGABEMNOLVFNFN5YVDTPN6TPZKUCJA5CNFSM4HP4VM2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXGVRBQ#issuecomment-499996806,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGABEMJYWJIQ4IASNEZBUITPZKUCJANCNFSM4HP4VM2A
.

Hi @MartinHajducik I wanted to follow up on this to make sure you're able to get things working with the latest release (0.6.1). Please let me know. Thanks.


How to run it on Linux?

Hello,
I'm here because I wanted to run Edgeware++ on Linux, I got exited, but I couldn't run it because it doesn't recognize the file existing or directory to the file. Is there anyway to fix this and get to gooning?
From,
Seiryū

Never Mind, had to download most of the requirements manually. But "vlc fails to load" keeps popping up on Terminal, and when I tried installing "pygobject", it doesn't work.

The VLC warning should not be a problem unless you want to use the VLC mode to play videos, in which case you will have to install VLC separately first.

when I tried installing "pygobject", it doesn't work.

What do you mean by this? Do you get any warnings or errors?

Hello, I decided to switch to Arch Linux, and I'm having trouble to get it to work. I'm wondering if anyone in the community who knows Arch Linux stuff that are able to help me?

i know arch linux

i use arch linux everyday

Closing this for now, but if you still have problems, feel free to open another issue.


[Feat]Implement Chatbot Feature

Description
Description: Implement a chatbot feature that assists users by generating relevant titles and descriptions for their content. This feature will improve user experience by helping users create optimized and engaging text based on their input.
Proposed Solution:
User Prompting: The chatbot should ask users key questions about their content, purpose, and audience.
Natural Language Processing (NLP): Use NLP to process responses and generate suitable titles and descriptions.
Customizable Output: Allow users to tweak the generated output to meet their specific needs.
Benefits:
Provides users with quick, optimized title and description suggestions.
Improves engagement by generating descriptive, engaging, and SEO-friendly text.
Screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

Hey @Ankul8471, ChatBOT is a very good feature, but I don’t think it is needed on our website right now. Once the website is fully functional and in working condition, I will let you know. Currently, we are working on the payment gateway, so I will inform you if there is a need in the future. Please raise any other issue, which I am happy to assign to you, but make sure the issue you raise is valid and really needs to be addressed on our website; otherwise, invalid issues will be closed. Thank you!

Ok I understand it but you assign me then it will help me a lot leaderboard and gssoc and also assigned some in future then why not now

@Ankul8471, don't worry. We retrieved your issues from the closed section and will add you there if a chatbot is needed.
Ankul, tell me one thing. In a previous issue, I reopened the issue for multilingual language support and assigned it to you, asking you to star our repository, which you acknowledged. However, I still noticed that you haven't starred it. Is this the right way to contribute to open source?

I also assume you haven't read our contributing guidelines. Please don't do that.

Ok I will do this


Request to get a User access token

Hi,
How can I request Request to get a User access token to set it for eBay.catalog.createChangeRequest.
I have tried like this but not working
eBay.application.getOAuthToken({
        grant_type: 'authorization_code',
        scope: 'https://api.ebay.com/oauth/api_scope'
    }).then(token => {

});

What exactly error are you facing?

@ineffablep
Documentation is available here.
https://developer.ebay.com/api-docs/static/oauth-tokens.html
Duplicate of #10

@ineffablep
#14
Support has been extended in the same package. You can find the example here :
https://github.com/bhushankumarl/eBay-node-client/tree/master/examples/javaScript/user


Windows 10: Electron and Atom cannot properly be pinned to task bar

run latest Windows 10
pin either electron or atom to the taskbar
click it to open the app

=> you end up having 2 icons
=> somehow the app when started does not get associated to the pinned entry in the taskbar
=> it does not reproduce on Windows 8.x
I wonder if your change in https://github.com/atom/electron/commit/fb6c80d12ecbb99a481e1018a0d1d56c3fc4dce3 could have an impact here. Imho it is used to find out if an application belongs to the same process group or not: https://msdn.microsoft.com/en-us/library/windows/desktop/dd378422(v=vs.85).aspx

So, this change shouldn't have any impact unless you're explicitly calling the method - the idea was that unless you called SetAppUserModelId, nothing would change. However, we've been noticing issues with this too, and one thing that I was wondering is that, do AppUserModelIDs get inherited? (i.e. if I set this in browser process, will renderer have the same ID?)

I see, so we are not calling this method and I am not sure about Atom/Electron calling it. Thats an interesting theory about the child process getting another entry in the task bar. However when i was right clicking on this second task entry to check which command was running, it was actually the same as the pinned one.

@bpasero Electron will end up calling it, but it has done so for at least a year or so. Another tricky bit is, that the Squirrel shortcut we generate has its own AppID on Windows, in order to try to fix pinning. The reason I added that PR is so that we could make Electron's AppID follow along with the Squirrel one.

Basically Pinning is a nightmarish feature that will continue to troll us forever

Calling SetAppUserModelId in renderer process might be able to fix this.

@zcbenz I'd really love to see this fixed, as it's driving me absolutely crazy.

Calling SetAppUserModelId in renderer process might be able to fix this

I was thinking about it too, basically we have to "fake" AppUserModel inheritance if it doesn't happen already.

I am experiencing this with both Atom and an Electron app. Pretty unacceptable for a production app, so hopefully this can be figured out and fixed pretty quickly and it's not a limitation with Windows 10 itself.

+1, I just upgraded from Windows 8.1 to 10 and this has been happening.  Hope you guys figure out what's causing this - it's not that bothersome but the two icons look kinda funny sometimes.

It seems there's a fix here. Can that be incorporated into Atom's next release?

@bnb Already done :)

@paulcbetts Im getting the same bug and its nice to know this'll be fixed soon.

Does this still occur for Electron >= v0.30.2?

@zcbenz yes, I just tried to pin the bare Electron 0.31.0 on Windows 10 and it reproduces.

Btw I also tried with app.setAppUserModelId("Electron") and it does not seem to make a difference.

@bpasero Do you have a shortcut and does that shortcut have an app user model ID already?

@paulcbetts is there a way to find out what app user model ID windows is using when you pin an app from a shortcut? In my comment today I was just using the electron.exe as it comes from the GitHub releases page without shortcut and I just pinned it to the taskbar. However, in another scenario I was testing with VSCode where we indeed have a proper shortcut generated. I was hoping that the task bar pinning would not produce 2 entries if I programmatically set the ID to be the same as whatever the shortcut is using. But I could not find an ID to make it work....

If you look at the command line parameters on a renderer process via Process Explorer you can see what Electron is using. Since you use Squirrel to install, you need to set it to com.squirrel.PACKAGE_ID.YOUR_EXE_WITHOUT_DOT_EXE, so for us it's com.squirrel.slack.Slack

@paulcbetts indeed it works once I set it to "com.squirrel.code.Code". Any reason you guys don't close this bug and fix it on the Atom side by using the setAppUserModelId() call?
Out of curiosity, where is the com.squirrel name defined in Squirrel for the User App Id?

@bpasero in Atom, this is already fixed but hasn't gone out yet since it relies on new Electron. Here's the PR that adds the user model ID stuff - https://github.com/Squirrel/Squirrel.Windows/pull/247

@paulcbetts got it. I noticed one downside to calling setAppUserModelId(): My user tasks are no longer showing up when I right click on the task bar icon. I wonder if a pinned application needs to register its tasks differently, e.g. via the shortcut, instead of programmatically from within the application?

Could you possibly have user tasks that were created before you made the setAppUserModelId change?

@paulcbetts I tried both variants: first setting the user task and then the app id, and vice versa. In both scenarios, my task does not show up anymore. Once I dont set an app id, the task shows up.

Looks like with the latest release of Atom 1.0.12 it can no longer be pinned to the desktop. It was pinned fine prior on two machines I use that I use. I know this issue is still open but is anyone else seeing this behavior?

"Pinned to the Desktop"? What does this mean?

Taskbar, LOL!

This bug in Atom was an Atom bug, from the Electron side everything works correctly, reopen if someone finds that this isn't the case

I didn't reopen or open this issue but was just using it as a sounding board since it was still open. Yeah pinning was fixed in Atom either in 1.0.13 or .14 at least it seems to be working now.

@paulcbetts Could you have a look at https://github.com/maxogden/electron-packager/issues/166 and determine if the 3rd bullet warrants reopening this case?
Dunno about the second bullet, if that is electron or electron-packager. The first bullet for sure is only the electron-packager
Basically I see duplicate icons on Electron and Windows exe's created with electron-packager with the latest version of electron (0.34.1 by now)

@p-bakker Are you using Squirrel as your installer on Windows?

I'm using electron-packager which, to my understanding, uses Squirrel

Right..... I took the com.squirrel.myCompany.mycompany stuff from some comment on a case...
I see now that squirrel is only used when using the auto-updater functionality, which I'm not right now (want to, but for now I'd be happy to just get things working correctly without auto-updater functionality)
But, as I mentioned, I see the same duplicate icon thing when just running my app in electron (commandline 'electron .')

@p-bakker Yep - because you set your app model ID to be something other than what the shortcut sets, you're seeing this bug.

I removed the app.setAppUserModelId(...); call: makes no difference
I've now also tested it just running electron, with the default app (which says: Welcome to Electron To run your app with Electron, execute the following command under your Console (or Terminal): ....) and I get the same double icon issue there

No idea if this is the case here, but it seems there's also a bug in Windows 10 that causes duplicate icons for pinned apps in the TaskBar. This issue will apparently be fixed in the next update to Windows 10: https://blogs.windows.com/windowsexperience/2015/10/12/announcing-windows-10-insider-preview-build-10565/

k, went through the exercise of setting up a VM with the Windows 10 Insider Preview build 10565, downloaded electron-v0.34.1-win32-x64.zip, extracted it and ran Electron, then pinned it: no duplicate icon!
To make sure I did the same procedure on my own machine running the currently released version of Windows 10: there I get duplicate icons.
So, pretty save to say this duplicate icon business is a Windows 10 bug and should be solved in the next Windows 10 update, which is expected to be released to the general public somewhere next month (nov '15).
So no need to reopen this case

Updated Windows 10 yesterday: the duplicate icon issue is gone now, so it was indeed a Windows 10 issue


Add ! prefix to "set" topics

Attributes topics have the "$" prefix.
I would like us to prefix the command topics, e.g. "set", with "!" or another appropriate character.
The reason is to make it easier to distinguish the calls going to the device from the controller.
Distinguish between state updates and commands.
(Command topics will not have the Retain bit set.)
Examples:

Setting light intensity from the controller to 50%

Controller publish
CTR: homie/686f6d6965/light/intensity/!set "50"
Device receives, and applies the change, then sends a property update
DEV: homie/686f6d6965/light/intensity "50"

Update device publishing interval to every 2.0 seconds

Controller publish
CTR: homie/686f6d6965/$interval/!set "2.0"
Device receives, and applies the change, then sends a attribute update
DEV: homie/686f6d6965/$interval "2.0"

Reboot device (implementation specific)

Controller publish
CTR: homie/686f6d6965/$implementation/!reboot ""
Device reboots

Hello @oeysteinhansen,
MQTT topic names normally consist of words/letters and numbers. It is highly unusual to use special characters like a "!". The dollar sign has a special meaning, see: http://www.hivemq.com/blog/mqtt-essentials-part-5-mqtt-topics-best-practices
The Homie convention already relies on the idea of speaking keywords. I feel the word "set" or "reboot" already imply a command.
Secondly the Homie convention defines how devices can communicate with each other. A certain knowledge is needed to be able to interact using the convention. This includes the topics structure and commands. The fact that the topic "set" is to be used by the counterpart for command messages is already known at this point.
With this scenario in mind, adding an additional character doesn't make a difference.
So in conclusion, I don't think that an unusual character added to command topic names will improve the Homie convention.
May I ask where your idea originates from? Maybe a related aspect of the Homie convention is misleading?

I had several thoughts in mind when suggesting the prefix. I did not want to jam everything into this issue, but I can describe some of  them her.
I can see that the mechanism for detecting nodes and properties are by a separate attributes $nodes and $properites. I would introduce that all topic not starting with a special character ($ or ! ) is the ID of the node / property. Eliminating the need for the $nodes and $properties attributes.
I would introduce that a node / property does not directly have a value, instead all values are represented as attributes.
Example:
homie/super-car/engine/$name → "Car engine"
homie/super-car/engine/$type → "V8"
homie/super-car/engine/speed/$value → "3.7"
homie/super-car/engine/direction/$value → "0.15"
homie/super-car/engine/temperature/$value → "21.5"
Future expansions:
Nodes and Properties would be equal. Both having describing attributes.
Nodes could have sub-nodes, allowing adding gateways, or domains.
Array of Nodes can be represented as sub-nodes instead of the _INDEX postfix of the NodeID.
Example:
homie/super-car/lights/$name → "Lights"
homie/super-car/lights/$type → "array"
homie/super-car/lights/0/$name → "Back lights"
homie/super-car/lights/0/intensity/$name → "Intensity"
homie/super-car/lights/0/intensity/$value → "0"
homie/super-car/lights/1/$name → "Front lights"
homie/super-car/lights/1/intensity/$name → "Intensity"
homie/super-car/lights/1/intensity/$value → "100"

@oeysteinhansen I understand the concern, but unfortunately we have to freeze the evolution at some point. The 2.1.0 is not released yet, and if we never stop changing things, it will never be. So let's put that aside for now. 😉

That is why an active maintainer group is important. I did criticize the idea and think it is important to get a second opinion (yes, I can be wrong). With @marvinroger 's and your response this is in fact an issue we can reject.
@oeysteinhansen thanks for your input!

@oeysteinhansen: Thank you for your input :)
I hope you don't feel offended. I'm worried about the maintainership of this project and picked your issue as reference for one of many stale issues. You can follow and comment the discussion in the open pull request :)


fft2 ignores overwrite_x=True

The overwrite_x keyword actually has no effect if one uses the convenience function for the 2D fft.
It seems the normal fft (1D) is not affected, I did not test this for fftn.
Also, the docs could be irritating because the keyword is called 'overwrite_x' and in the description is written "contents of x can be destroyed".
Reproducing code example:
import numpy as np
from scipy.fftpack import fft2

x = np.ones((4, 6), dtype=np.complex128)
print("state before inplace fft2:\n", x)

print("FFT:\n", fft2(x))

fft2(x, overwrite_x=True)
print("state after inplace fft2:\n", x)

Output
state before inplace fft2:
 [[ 1.+0.j  1.+0.j  1.+0.j  1.+0.j  1.+0.j  1.+0.j]
 [ 1.+0.j  1.+0.j  1.+0.j  1.+0.j  1.+0.j  1.+0.j]
 [ 1.+0.j  1.+0.j  1.+0.j  1.+0.j  1.+0.j  1.+0.j]
 [ 1.+0.j  1.+0.j  1.+0.j  1.+0.j  1.+0.j  1.+0.j]]
FFT:
 [[ 24.+0.j   0.+0.j   0.+0.j   0.+0.j   0.+0.j   0.+0.j]
 [  0.+0.j   0.+0.j   0.+0.j   0.+0.j   0.+0.j   0.+0.j]
 [  0.+0.j   0.+0.j   0.+0.j   0.+0.j   0.+0.j   0.+0.j]
 [  0.+0.j   0.+0.j   0.+0.j   0.+0.j   0.+0.j   0.+0.j]]
state after inplace fft2:
 [[ 1.+0.j  1.+0.j  1.+0.j  1.+0.j  1.+0.j  1.+0.j]
 [ 1.+0.j  1.+0.j  1.+0.j  1.+0.j  1.+0.j  1.+0.j]
 [ 1.+0.j  1.+0.j  1.+0.j  1.+0.j  1.+0.j  1.+0.j]
 [ 1.+0.j  1.+0.j  1.+0.j  1.+0.j  1.+0.j  1.+0.j]]

Scipy/Numpy/Python version information:
python  3.6.4
numpy  1.13.3
scipy  1.0.0

The overwrite_* parameters in all routines mean that the output may be
rewritten, in case it is useful for performance, but it is not a
guarantee that this occurs.

i want to take up this issue can some one please guide me how to start and i have experience in python & numpy.

does this mean fft2(x, overwrite_x=True) should overwrite the x with result of fft2(x).

Yes, but unfortunately this is by design. If there is a up/down casting required then the intermediate arrays are used and those are the ones that get overwritten. Hence it won't be possible to fix this unless the whole fft library is rewritten.

Ah I see, it is still kind of strange as the behavior is not really intuitive (an argument that may or may not cause your input to be overwritten?), numpy won't give you such an argument:
https://docs.scipy.org/doc/numpy-1.14.0/reference/generated/numpy.fft.fft2.html#numpy.fft.fft2
The solution could maybe to just drop the parameter until a more flexible fft library is available?

I'm not sure if all cases are supported via NumPy too but that's because NumPy uses the C bindings of fftpack, that is fftpack_lite , similar to LAPACK handling. SciPy uses directly the Fortran source. We have recently added a note at http://scipy.github.io/devdocs/linalg.lapack.html#module-scipy.linalg.lapack and same applies here. It's a datatype dependent overwrite via f2py and not related to fft library itself.

But shouldn't than the input be overwritten if the input array fulfills the requirements?
I thought if I make the array F continuous it should than be overwritten:
In [36]: x = np.ones((4, 6), order='F', dtype=np.complex128)

In [37]: fft2(x, overwrite_x=True)
Out[37]:
array([[24.+0.j,  0.+0.j,  0.+0.j,  0.+0.j,  0.+0.j,  0.+0.j],
       [ 0.+0.j,  0.+0.j,  0.+0.j,  0.+0.j,  0.+0.j,  0.+0.j],
       [ 0.+0.j,  0.+0.j,  0.+0.j,  0.+0.j,  0.+0.j,  0.+0.j],
       [ 0.+0.j,  0.+0.j,  0.+0.j,  0.+0.j,  0.+0.j,  0.+0.j]])

In [38]: x
Out[38]:
array([[1.+0.j, 1.+0.j, 1.+0.j, 1.+0.j, 1.+0.j, 1.+0.j],
       [1.+0.j, 1.+0.j, 1.+0.j, 1.+0.j, 1.+0.j, 1.+0.j],
       [1.+0.j, 1.+0.j, 1.+0.j, 1.+0.j, 1.+0.j, 1.+0.j],
       [1.+0.j, 1.+0.j, 1.+0.j, 1.+0.j, 1.+0.j, 1.+0.j]])

Whether the array gets overwritten or not with overwrite_xxx=True is
implementation detail in most of the functions. In particular, the f2py
guarantees don't apply here as you find out.

Clearly, these flags would best not have been added in the first place
as the performance advantage is not that clear.
However, they're been in the API close to 20 years, and removing them
is again a question of breaking old code vs. cosmetic reasons.

This is of course true, maybe an explicit hint in the documentation would than be in order to not use/avoid this flags?
I teach sometimes classes in python and when I show them a page of the scipy docs and have to immediately tell them "but well don't use the overwrite keywords or at least test them beforehand" makes the strong advertisement I'm doing beforehand for numpy+scipy somewhat awkward.

The description of these parameters in the documentation can of course
be made more clear on that they exist only for performance reasons, and
will not necessarily result to overwriting.

will the following changes be made to documentation ? if so i would like to take up the issue.


No locker when creating Azure Composer

Question
Every composer in composer.go uses a memory locker, but not Azure. Was that forgotten, or is this on purpose?

Thank you for reporting!


Social Feeds for autoshare in social networks

I've tried to implement the solution from Eduardo Boucas (repo) to automatically share content from Jekyll to social platforms when a new post is posted, but I think I've failed more of less.
I can make the feed to take the content in IFTTT, but I can make the IFTTT to be able to post an image at the same time. What is more, I would like that the tool posts:

Tittle of the post.
URL of the post
Tags of the post as a twitter hashtags (without blank spaces)
Feature image in the tweet, so it's more appealing.

I've read about how ITFFF take care of the image to post it here, but I don't really know if it's working at all.

I've implemented it's own feed for twitter.
here:

8d6c86811a0d94032875fd730c9bacf05be41bde
42755bfc9f499f2d35c1950378bddb82d23339db
af28a74f69713f994a7967066b821f7c1fdaf405
3f5969c6f7934cb1363159629eee9d9e56e46e10
d473133b184941b0dbb213286f2f3b50de1ef7c3
07e696bf87e9cd66ef5adb73ac0f5ad4a5af21df


Update SeamOS routes

Would be really nice to automate this ;)

yea or we could skip the javascript module entirely and put the route definitions in seamos for now

e.g. allow an override


Docs don't load in latest Firefox (85)

Since updating to Firefox 85 I can't view Rust docs. The package page loads, but none of the content gets fleshed out. Also notable and probably related, my long standing preference for the dark theme isn't working either. The light theme loads by default and switching it to dark only works for one page, then disappears.
I'm guessing some new cookie or fingerprinting protection is blocking whatever the site uses for storing that preference and presumably also whatever mechanism dynamically loads actually content.

After several days frustration having to switch to Chromium every time I landed on a docs.rs link, I just tried to screen shot this problem so you could see what was and wasn't loading. And for some reason half way through picking a sample project it magically started working.
My best guess is that a third party extension was responsible for the blockage and it updated in the background at just that moment, but at this point I can't prove or demonstrate anything. I'll re-open in I can pass on anything reproducible.

Followup: uBlock Origin did indeed update in the last half hour (now 1.33.2, released today). This must have been some incompatibility between the previous release of uBlock Origin and the latest Firefox.


Problem with scaffolding

When I try scaffold, I get error:
There was an error running the selected code generator: 'There was an error creating a DbContext, the file C:\Users\Asus\source\repos\ContosoUniversity\Data\SchoolContext.cs  already exists'

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

ID: d0026f69-7e5d-0d3e-5586-16ecc0656ff0
Version Independent ID: 3cb264da-2ff1-1256-572f-aa4ebca6e255
Content: Razor Pages with EF Core in ASP.NET Core - Read Related Data - 6 of 8
Content Source: aspnetcore/data/ef-rp/read-related-data.md
Product: aspnet-core
Technology: aspnetcore-data
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

That's a great error message to help you figure out what you did wrong.

When I try scaffold, I get error:
There was an error running the selected code generator: 'There was an error creating a DbContext, the file C:\Users\Asus\source\repos\ContosoUniversity\Data\SchoolContext.cs already exists'
Document Details
Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d0026f69-7e5d-0d3e-5586-16ecc0656ff0
Version Independent ID: 3cb264da-2ff1-1256-572f-aa4ebca6e255
Content: Razor Pages with EF Core in ASP.NET Core - Read Related Data - 6 of 8
Content Source: aspnetcore/data/ef-rp/read-related-data.md
Product: aspnet-core
Technology: aspnetcore-data
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

had the same problem.
just replace this
--dataContext AppMvcBasica.Models.ApplicationDbContext
to this
--dataContext ApplicationDbContext
example:
dotnet aspnet-codegenerator controller --force -async -name FornecedoresController -m Fornecedor --dataContext ApplicationDbContext --relativeFolderPath Controllers --useDefaultLayout --referenceScriptLibraries


The test suite is stuck

The test suite is stuck

As shown above, the execution steps have been stuck there, but my backend has not received the request
I have achieved the report state, but still cannot pass the test. reportstate:https://homegraph.googleapis.com/v1/devices:reportStateAndNotificatio

Did you hear the test suite run verbally?

i don't understand

When running the test suit, the backend did not receive the request

Test suite I run on a web page

Have you fulfilled the prerequisites?

Closing as obsolete.

Closing as obsolete.


Various UI and configuration improvements

Apart from the podspec this looks O.K. Without this addition I would merge right away. Let me explain: If I would add the Podspec I would also need to maintain the pod within Cocoapods (e.g. push updates to the trunk...). That's not something I want to do now or in the future as (from my opinion) SPM is the way to go.

I get it. I'm currently working on converting all of my libraries to SPM for the same reason.