add missing bindings support for completable future via binding-conveyor-fn

when I added completable-future recently I forgot to add support for dynamic bindings via binding-conveyor-fn.
i've added binding-conveyor-fn support in this PR as well as added a test to verify it works.

Sorry, I'll reopen this; I really do mean to get to it, but I've moved houses, and I have other good excuses, and I haven't dealt with it. But I do intend to merge this. Sorry. Thanks for your patience.


fix(fsfreeze): improved error message when volume is not staged

Describe the issue
It its seen that if a node has supposedly recovered from an abrupt shutdown and the application is not running, in that case the the filesystem mayastor volume is published but it's not mounted on the node where the application is. Mayastor control-plane volume being published gives the csi-controller a wrong notion that the volume is under use i.e mounted thus it tries to issue fsfreeze to ensure fs consistency, but fsfreeze wouldn't succeed as it would search for the mount path to issue FIFREEZE ioctl. In such case the flow returns a very vague error of Terminated with error error=Cannot find volume: volume ID: e0316b8c-e4f4-454c-9803-c238dc68457c.
Changes
This PR enhances the error message stating what is the actual error i.e Failed to freeze volume as the filesystem volume is not mounted

bors merge


[ELE-129] Add to documentation a page with all configuration options on the dbt side

General Feedback, I’d like a definitive guide/page for configuration options and variables, along with their allowed values.
The CLI usage/commands page is helpful and straight forward. But configuration options on the dbt side, including variables and schema setup, are scattered about. For example in Slack there are references to setting the variables
disable_run_results ,disable_test_results but there is no reference to them in the Docs.
Slack: https://elementary-community.slack.com/archives/C02CTC89LAX/p1672946756861029
ELE-129

test comment

Hi! @Maayan-s! I’m one of the developers of Sweep(YC S23), a github app that solves issues by writing pull requests. We used Sweep https://github.com/sweepai/sweep to write this pull request https://github.com/elementary-data/elementary/pull/1027.


Acquire application token for Microsoft Graph API

I have an application that needs to be able to call Microsoft Graph API without a user.  It would use the ClientId and ClientSecret.  Can this be set up to work through IdentityServer4 using your Admin?

If you want to call Microsoft Graph API - you need retrieve an access token from their Identity Provider.
If you are looking for forwarding access token of external provider - take a look at these resources:
https://stackoverflow.com/a/45767061/943403
https://github.com/waqaskhan540/IdentityServerExternalAuth
http://docs.identityserver.io/en/latest/topics/extension_grants.html

Thank you.  I will have a look at that right away.

May I close it? Do you have any issues?

Feel free to reopen.


VM Application - DeployifNot exists

[Enter feedback here]
Documentation on how to create the policies for VM application scan and deploy if not exists into the Virtual Machines as remediation is necessary.

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

ID: 1990fefe-9aee-5205-04aa-02ad58016077
Version Independent ID: 9737fc6f-99a0-563a-b809-348326155834
Content: Create and deploy VM application packages (preview) - Azure Virtual Machines
Content Source: articles/virtual-machines/vm-applications-how-to.md
Service: virtual-machines
Sub-service: gallery
GitHub Login: @cynthn
Microsoft Alias: cynthn

@p9139010,
Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@p9139010,
Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@PriyankaRudramaina  Any update on this ? Is this documentation expected at any time soon?

@p9139010
This is under review by content author @cynthn- Can you please share your comments here?

@p9139010 p913 Thanks for the feedback! Can you give me a little bit more information about what you are looking for? Are you looking for instructions on how to apply a policy for VM apps to VMs? Or, are there details missing from the policy definition?
@ericd-mst-github FYI - maybe ask engineering to take a look at this too?

We are looking for documentation about how to install a package using VM application with the deployifnotexists Azure policy effect.
If we go to Microsoft documentation https://docs.microsoft.com/en-us/azure/virtual-machines/vm-applications it says :
VM applications can be used with the DeployIfNotExists policy.
However, when we look for documentation about how to use ARM template, or even how to use Azure policy to install VM application, we cannot find any documentation.

Hi all, I have checked with engineering and at this time we currently do not have additional documentation on this but will be looking into this.
I apologize for any delays and thank you for your patience!

Looks like @ericd-mst-github has this feedback and is working on this with engineering, so I'm going to close this for now. #please-close


Transaction reverted: function returned an unexpected amount of data from faked contract struct.

Describe the bug
When getting a faked struct from inside a real solidity contract this error is returned from faked struct.
Transaction reverted: function returned an unexpected amount of data
Reproduction steps
I wrote out the minimal test example. Let me know if you need more.
DebugMapping.sol
// SPDX-License-Identifier: MIT
pragma solidity ^0.8.0;

import "./IDebugMappingDeployed.sol";

contract DebugMapping {
    IDebugMappingDeployed private remoteDeployed;

    constructor(address _remoteDeployedAddress) {
        remoteDeployed = IDebugMappingDeployed(_remoteDeployedAddress);
    }

    function getStructFromDeployed(uint256 _item)
        public
        view
        returns (IDebugMappingDeployed.SomeStruct memory)
    {
        return (remoteDeployed.getStruct(_item));
    }
}

DebugMappingDeployed.sol
// SPDX-License-Identifier: MIT
pragma solidity ^0.8.0;

contract DebugMappingDeployed {
    struct SomeStruct {
        uint256 id;
        string name;
    }

    mapping(uint256 => SomeStruct) public getStruct;

    constructor() {
        getStruct[1] = SomeStruct({id: 3, name: "one"});
        getStruct[2] = SomeStruct({id: 2, name: "two"});
        getStruct[3] = SomeStruct({id: 1, name: "three"});
    }
}

IDebugMappingDeployed.sol
// SPDX-License-Identifier: MIT
pragma solidity ^0.8.0;

interface IDebugMappingDeployed {
    struct SomeStruct {
        uint256 id;
        string name;
    }

    function getStruct(uint256) external view returns (SomeStruct memory);
}

DebugMapping.ts (test)
import * as dotenv from "dotenv";

import { expect } from "chai";
import { ethers } from "hardhat";
import { FakeContract, smock } from "@defi-wonderland/smock";

import { Contract, ContractFactory } from "ethers";
import { SignerWithAddress } from "@nomiclabs/hardhat-ethers/signers";
import { DebugMappingDeployed } from "../typechain";

dotenv.config();

let contractFactory: ContractFactory;
let contract: Contract;
let debugMappingDeployed: FakeContract<DebugMappingDeployed>;
let owner: SignerWithAddress;

describe("DebugMapping", function () {
  before(async () => {
    [owner] = await ethers.getSigners();

    // Fake Contract
    debugMappingDeployed = await smock.fake<DebugMappingDeployed>(
      "DebugMappingDeployed"
    );

    // Set up this test contract
    contractFactory = await ethers.getContractFactory("DebugMapping", owner);
  });

  beforeEach(async () => {
    contract = await contractFactory.deploy(debugMappingDeployed.address);
    await contract.deployed();
  });

  it("Should return faked struct", async () => {
    debugMappingDeployed.getStruct.returns({ id: 1, name: "one" });

    const struct = await contract.getStructFromDeployed(1);
    console.log(struct);

    expect(await contract.getStructFromDeployed(1).id).to.equal(1);
  });
});

Expected behavior
The unit test "Should return faked struct" should successful execute await contract.getStructFromDeployed(1); and return {id: 1, name: "one" }.
System Specs:

OS: Windows 10
Package Version (or commit hash): 2.0.7

Additional context
When running with Mocha Test Explorer plugin for vscode.
Stack trace
Error: Transaction reverted: function returned an unexpected amount of data
    at DebugMapping.getStructFromDeployed (contracts/DebugMapping.sol:18)
    at HardhatNode._gatherTraces (node_modules\hardhat\src\internal\hardhat-network\provider\node.ts:1484:30)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
    at runNextTicks (internal/process/task_queues.js:62:3)
    at listOnTimeout (internal/timers.js:523:9)
    at processTimers (internal/timers.js:497:7)
    at async HardhatNode.runCall (node_modules\hardhat\src\internal\hardhat-network\provider\node.ts:503:20)
    at async EthModule._callAction (node_modules\hardhat\src\internal\hardhat-network\provider\modules\eth.ts:353:9)

When running with hardhat test
Stack trace
UnhandledPromiseRejectionWarning: Error: Failed to encode return value for getStruct
    at SafeProgrammableContract.encodeValue (C:\test\smart-contract\node_modules\@defi-wonderland\smock\src\logic\programmable-function-logic.ts:128:17)
    at SafeProgrammableContract.modifyAnswer (C:\test\smart-contract\node_modules\@defi-wonderland\smock\src\logic\programmable-function-logic.ts:97:52)
    at C:\test\smart-contract\node_modules\@defi-wonderland\smock\src\logic\programmable-function-logic.ts:42:18
    at Object.next (C:\test\smart-contract\node_modules\rxjs\src\internal\Subscriber.ts:194:14)
    at SafeSubscriber.Subscriber._next (C:\test\smart-contract\node_modules\rxjs\src\internal\Subscriber.ts:119:22)
    at SafeSubscriber.Subscriber.next (C:\test\smart-contract\node_modules\rxjs\src\internal\Subscriber.ts:75:12)
    at C:\test\smart-contract\node_modules\rxjs\src\internal\operators\withLatestFrom.ts:104:22
    at OperatorSubscriber._this._next (C:\test\smart-contract\node_modules\rxjs\src\internal\operators\OperatorSubscriber.ts:43:13)
    at OperatorSubscriber.Subscriber.next (C:\test\smart-contract\node_modules\rxjs\src\internal\Subscriber.ts:75:12)
    at C:\test\smart-contract\node_modules\rxjs\src\internal\operators\map.ts:56:20

I have discovered that in solidity 0.8.4 one can not return a struct from a mapping in the way I am attempting to do. The solution is to destructure the result when calling a public  mapping with struct.
Once again the bug is not with your library. 🤘
The solution when using the above test scenario.
IDebugMappingDeployed.sol
// SPDX-License-Identifier: MIT
pragma solidity ^0.8.0;

interface IDebugMappingDeployed {
    struct SomeStruct {
        uint256 id;
        string name;
    }

    function getStruct(uint256) external view returns (uint256 id, string memory name); // Destructured the return value
}

DebugMapping.sol
// SPDX-License-Identifier: MIT
pragma solidity ^0.8.0;

import "./IDebugMappingDeployed.sol";

contract DebugMapping {
    IDebugMappingDeployed private remoteDeployed;

    constructor(address _remoteDeployedAddress) {
        remoteDeployed = IDebugMappingDeployed(_remoteDeployedAddress);
    }

    function getStructFromDeployed(uint256 _item)
        public
        view
        returns (uint256 id, string memory name) // Returns the destructured return value
    {
        return (remoteDeployed.getStruct(_item));
    }
}


display a "Server buffering your request" when percentCompleted is 100

labeling this as a bug because hitting 100% completion when uploading thousands of files will cause the backend to hang for awhile (and the Loading.js page will remain displaying that same 100% completion message) to copy all the files to /datastore. this can be confusing to users


move alaska, hawaii, and puerto rico into frame with lower 48 states all in an albers

This function projects U.S. states and Puerto Rico into a frame that's easy to view for election maps, etc.
Sample usage:
update states_table
set the_geom = CDB_AlbersUSA(the_geom,state_identifier)

Produces a map like this:

I'm looking for feedback on:

Choosing appropriate projections
Placement and scale of the states
Whether more state identifiers should be listed for the second argument

cc @makella @andrewxhill

Does it work line your example, overwriting thegeom, it does it only work
if you alias the result as thegeomweb?
On Sep 29, 2015 12:58 AM, "Andy Eschbacher"<EMAIL_ADDRESS>wrote:

This function projects U.S. states and Puerto Rico into a frame that's
easy to view for election maps, etc.
Sample usage:
update states_tableset the_geom = CDB_AlbersUSA(the_geom,state_identifier)
Produces a map like this:
[image: screen shot 2015-09-28 at 21 56 55]
https://cloud.githubusercontent.com/assets/1041056/10155575/e142a8ce-662b-11e5-88d4-e98792563abe.png
I'm looking for feedback on:

Choosing appropriate projections
Placement and scale of the states
Whether more state identifiers should be listed for the second
argument

cc @makella https://github.com/makella @andrewxhill
https://github.com/andrewxhill
You can view, comment on, or merge this pull request online at:
https://github.com/CartoDB/cartodb-postgresql/pull/167
Commit Summary

first
refactored
project to wgs84 instead
fixed output errors

File Changes

A scripts-available/CDB_Albers.sql
https://github.com/CartoDB/cartodb-postgresql/pull/167/files#diff-0
(87)

Patch Links:

https://github.com/CartoDB/cartodb-postgresql/pull/167.patch
https://github.com/CartoDB/cartodb-postgresql/pull/167.diff

—
Reply to this email directly or view it on GitHub
https://github.com/CartoDB/cartodb-postgresql/pull/167.

I can easily change it to be agnostic to the input geometry, which might be more natural: the srid you put in is the srid you get out.
updating the_geom_webmercator works if it's transformed:
update states_table
set the_geom_webmercator = ST_Transform(CDB_AlbersUSA(the_geom,state_identifier),3857)

But that doesn't trigger the_geom column to be recalculated.

Yep, got it got, I missed the srid parameter. Looks great!
On Tue, Sep 29, 2015 at 12:12 PM, Andy Eschbacher<EMAIL_ADDRESS>wrote:

I can easily change it to be agnostic to the input geometry, which might
be more natural: the srid you put in is the srid you get out.
updating the_geom_webmercator works if it's transformed:
update states_tableset the_geom_webmercator = ST_Transform(CDB_AlbersUSA(the_geom,state_identifier),3857)
But that doesn't trigger the_geom column to be recalculated.
—
Reply to this email directly or view it on GitHub
https://github.com/CartoDB/cartodb-postgresql/pull/167#issuecomment-144107376
.

--
andrewxhill http://twitter.com/andrewxhill
http://cartodb.com

@rochoa @javisantana, what do you think of this? Also, how should I test it?

Why this needs to be a function in the extension and not a table in common data?

Yeah. Where should I save this function for later reference, then?

Because any user that wants to transform their own data to match will need
to perform the same transform exactly. (Imagine any point dataset)
On Oct 2, 2015 12:04 PM, "javi santana"<EMAIL_ADDRESS>wrote:

Why this needs to be a function in the extension and not a table in common
data?
—
Reply to this email directly or view it on GitHub
https://github.com/CartoDB/cartodb-postgresql/pull/167#issuecomment-145072219
.

I think we should do something more generic here, we know a lot of countries have this problem, so why now having a table with the codes <-> transformations (and projection), like
CDB_Country('spain', the_geom, admin_identifier)
CDB_Country('usa', the_geom, admin_identifier)

and the a table like
create table cdb_countries (
  name text,
  region text,
  offset,
  projection_srid
)

maybe sounds overkill but adding so concrete functionality to a core extension seems weird (not against it tho)

Ok, I'll:

make a list of candidate countries (or continents) that could use this type of transformation
look into the 'standard' way to alter the projection
fit it into the format proposed above

@ohasselblad there was a comment from Cindy B about the Census maps that she was a part of making... apparently AK (the state itself) prefers to have AK above the contiguous US. I know you and I had discussed the idea of having a couple of different layouts for templates including what USGS and Census do.
On a design note: she also said that I should make sure to shadow in Canada, Mexico in the background.
Example of one her maps:

i see this working when you have control of the non-map layout too, but for
more generic purposes i feel like we still need to move alaska below. space
packing + familiarity
On Mon, Oct 19, 2015 at 8:39 AM, Mamata Akella<EMAIL_ADDRESS>wrote:

@ohasselblad https://github.com/ohasselblad there was a comment from
Cindy B about the Census maps that she was a part of making... apparently
AK (the state itself) prefers to have AK above the contiguous US. I know
you and I had discussed the idea of having a couple of different layouts
for templates including what USGS and Census do.
On a design note: she also said that I should make sure to shadow in
Canada, Mexico in the background.
Example of one her maps:
[image: screen shot 2015-10-19 at 8 34 34 am]
https://cloud.githubusercontent.com/assets/1566273/10577844/41e17a0a-763c-11e5-90da-fbc0766331af.png
—
Reply to this email directly or view it on GitHub
https://github.com/CartoDB/cartodb-postgresql/pull/167#issuecomment-149202101
.

--
andrewxhill http://twitter.com/andrewxhill
http://cartodb.com

And I did make Alaska larger than it is in most other projections like this!
On Mon, Oct 19, 2015 at 6:26 AM Andrew W. Hill<EMAIL_ADDRESS>wrote:

i see this working when you have control of the non-map layout too, but for
more generic purposes i feel like we still need to move alaska below. space
packing + familiarity
On Mon, Oct 19, 2015 at 8:39 AM, Mamata Akella<EMAIL_ADDRESS>wrote:

@ohasselblad https://github.com/ohasselblad there was a comment from
Cindy B about the Census maps that she was a part of making... apparently
AK (the state itself) prefers to have AK above the contiguous US. I know
you and I had discussed the idea of having a couple of different layouts
for templates including what USGS and Census do.
On a design note: she also said that I should make sure to shadow in
Canada, Mexico in the background.
Example of one her maps:
[image: screen shot 2015-10-19 at 8 34 34 am]
<
https://cloud.githubusercontent.com/assets/1566273/10577844/41e17a0a-763c-11e5-90da-fbc0766331af.png
—
Reply to this email directly or view it on GitHub
<
https://github.com/CartoDB/cartodb-postgresql/pull/167#issuecomment-149202101
.

--
andrewxhill http://twitter.com/andrewxhill
http://cartodb.com
—
Reply to this email directly or view it on GitHub
https://github.com/CartoDB/cartodb-postgresql/pull/167#issuecomment-149212521
.

Hey there, I'm happy to unearth this :)
I've been using this, very useful and thank you
https://team.cartodb.com/u/nerikcarto/viz/36e410e6-0574-11e6-9b33-0e31c9be1b51/embed_map
One thing that is noticed when mapping the Zika outbreak: american territories are missing (American Samoas and US Virgin Islands). They are important, no?
This would be awesome if merged - even with only the USA, but a with generic name.

A quick way to add separator lines :
    -- puerto rico
    SELECT ST_Transform(ST_GeomFromText('LINESTRING(13 1.5,16 3)', 4236), 3857) as the_geom_webmercator
    union all
    -- hawaii
    SELECT ST_Transform(ST_GeomFromText('LINESTRING(-12 7.8,-5 1.6)', 4236), 3857) as the_geom_webmercator
    union all
    -- alaska
    SELECT ST_Transform(ST_GeomFromText('LINESTRING(-21 10,-9 -1)', 4236), 3857) as the_geom_webmercator

Stale


bias not found in checkpoint

model: models/pwcnet-sm-6-2-multisteps-chairsthingsmix/pwcnet.ckpt-592000
gpu_devices = []
controller = '/device:CPU:0'
windows 8.1
python 3.6
tensorflow 1.13
running: pwcnet_predict_from_img_pairs.py
full error output:
tensorflow.python.framework.errors_impl.NotFoundError: Restoring from checkpoint failed. This is most likely due to a Variable name or other graph key that is missing from the checkpoint. Please ensure that you have not altered the graph expected based on the checkpoint. Original error:
Key pwcnet/ctxt/dc_conv31/bias not found in checkpoint
[[node save/RestoreV2 (defined at C:\PROJECTS\SASMOB - hídas projekt\optical_flow\tfoptflow-master\tfoptflow\model_base.py:119) ]]
Caused by op 'save/RestoreV2', defined at:
File "C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\Common7\IDE\Extensions\Microsoft\Python\Core\ptvsd_launcher.py", line 89, in
vspd.debug(filename, port_num, debug_id, debug_options, run_as)
File "C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\Common7\IDE\Extensions\Microsoft\Python\Core\ptvsd\debugger.py", line 2631, in debug
exec_file(file, globals_obj)
File "C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\Common7\IDE\Extensions\Microsoft\Python\Core\ptvsd\util.py", line 119, in exec_file
exec_code(code, file, global_variables)
File "C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\Common7\IDE\Extensions\Microsoft\Python\Core\ptvsd\util.py", line 95, in exec_code
exec(code_obj, global_variables)
File "C:\PROJECTS\SASMOB - hídas projekt\optical_flow\tfoptflow-master\tfoptflow\pwcnet_predict_from_img_pairs.py", line 58, in
nn = ModelPWCNet(mode='test', options=nn_opts)
File "C:\PROJECTS\SASMOB - hídas projekt\optical_flow\tfoptflow-master\tfoptflow\model_pwcnet.py", line 231, in init
super().init(name, mode, session, options)
File "C:\PROJECTS\SASMOB - hídas projekt\optical_flow\tfoptflow-master\tfoptflow\model_base.py", line 66, in init
self.build_graph()
File "C:\PROJECTS\SASMOB - hídas projekt\optical_flow\tfoptflow-master\tfoptflow\model_base.py", line 266, in build_graph
self.init_saver()
File "C:\PROJECTS\SASMOB - hídas projekt\optical_flow\tfoptflow-master\tfoptflow\model_base.py", line 119, in init_saver
self.saver = tf.train.Saver()
File "C:\Users\BAndras\Anaconda3\lib\site-packages\tensorflow\python\training\saver.py", line 832, in init
self.build()
File "C:\Users\BAndras\Anaconda3\lib\site-packages\tensorflow\python\training\saver.py", line 844, in build
self._build(self._filename, build_save=True, build_restore=True)
File "C:\Users\BAndras\Anaconda3\lib\site-packages\tensorflow\python\training\saver.py", line 881, in _build
build_save=build_save, build_restore=build_restore)
File "C:\Users\BAndras\Anaconda3\lib\site-packages\tensorflow\python\training\saver.py", line 513, in _build_internal
restore_sequentially, reshape)
File "C:\Users\BAndras\Anaconda3\lib\site-packages\tensorflow\python\training\saver.py", line 332, in _AddRestoreOps
restore_sequentially)
File "C:\Users\BAndras\Anaconda3\lib\site-packages\tensorflow\python\training\saver.py", line 580, in bulk_restore
return io_ops.restore_v2(filename_tensor, names, slices, dtypes)
File "C:\Users\BAndras\Anaconda3\lib\site-packages\tensorflow\python\ops\gen_io_ops.py", line 1655, in restore_v2
name=name)
File "C:\Users\BAndras\Anaconda3\lib\site-packages\tensorflow\python\framework\op_def_library.py", line 788, in _apply_op_helper
op_def=op_def)
File "C:\Users\BAndras\Anaconda3\lib\site-packages\tensorflow\python\util\deprecation.py", line 507, in new_func
return func(*kwargs)
File "C:\Users\BAndras\Anaconda3\lib\site-packages\tensorflow\python\framework\ops.py", line 3300, in create_op
op_def=op_def)
File "C:\Users\BAndras\Anaconda3\lib\site-packages\tensorflow\python\framework\ops.py", line 1801, in init
self._traceback = tf_stack.extract_stack()
NotFoundError (see above for traceback): Restoring from checkpoint failed. This is most likely due to a Variable name or other graph key that is missing from the checkpoint. Please ensure that you have not altered the graph expected based on the checkpoint. Original error:
Key pwcnet/ctxt/dc_conv31/bias not found in checkpoint
[[node save/RestoreV2 (defined at C:\PROJECTS\SASMOB - hídas projekt\optical_flow\tfoptflow-master\tfoptflow\model_base.py:119) ]]
output until error:
C:\Users\BAndras\Anaconda3\lib\site-packages\h5py_init_.py:34: FutureWarning:
Conversion of the second argument of issubdtype from float to np.floating i
s deprecated. In future, it will be treated as np.float64 == np.dtype(float).ty pe.
from ._conv import register_converters as _register_converters
Building model...
WARNING:tensorflow:From C:\PROJECTS\SASMOB - hídas projekt\optical_flow\tfoptflo
w-master\tfoptflow\model_pwcnet.py:1094: conv2d (from tensorflow.python.layers.c
onvolutional) is deprecated and will be removed in a future version.
Instructions for updating:
Use keras.layers.conv2d instead.
WARNING:tensorflow:From C:\Users\BAndras\Anaconda3\lib\site-packages\tensorflow
python\framework\op_def_library.py:263: colocate_with (from tensorflow.python.fr
amework.ops) is deprecated and will be removed in a future version.
Instructions for updating:
Colocations handled automatically by placer.
WARNING:tensorflow:From C:\PROJECTS\SASMOB - hídas projekt\optical_flow\tfoptflo
w-master\tfoptflow\model_pwcnet.py:1221: conv2d_transpose (from tensorflow.pytho
n.layers.convolutional) is deprecated and will be removed in a future version.
Instructions for updating:
Use keras.layers.conv2d_transpose instead.
... model built.
Loading model checkpoint c:/PROJECTS/SASMOB - hídas projekt/optical_flow/tfoptfl
ow-master/tfoptflow/models/pwcnet-sm-6-2-multisteps-chairsthingsmix/pwcnet.ckpt-
592000 for eval or testing...
WARNING:tensorflow:From C:\Users\BAndras\Anaconda3\lib\site-packages\tensorflow
python\training\saver.py:1266: checkpoint_exists (from tensorflow.python.trainin
g.checkpoint_management) is deprecated and will be removed in a future version.
Instructions for updating:
Use standard file APIs to check for files with this prefix.
INFO:tensorflow:Restoring parameters from c:/PROJECTS/SASMOB - hídas projekt/opt
ical_flow/tfoptflow-master/tfoptflow/models/pwcnet-sm-6-2-multisteps-chairsthing
smix/pwcnet.ckpt-592000
2019-03-28 12:13:28.455200: W tensorflow/core/framework/op_kernel.cc:1401] OP_RE
QUIRES failed at save_restore_v2_ops.cc:184 : Not found: Key pwcnet/ctxt/dc_conv
31/bias not found in checkpoint

Sorry, this is for an other github project.


Document incompatibilities between bst releases

See original issue on GitLab
In GitLab by [Gitlab user @jjardon] on Feb 5, 2018, 13:44
bst 1.0 -> 1.2:

Config file default to not push to cache servers, you have tu specify "push: true" (See #224)
cache-key has been changed (See #326)
changes in the default value of build-root (See #414)
ostree-based cache server is gone

bst 1.2 -> 1.4:

TBD

In GitLab by [Gitlab user @jjardon] on Mar 27, 2018, 16:58
changed the description

In GitLab by [Gitlab user @toscalix] on May 30, 2018, 17:42
marked this issue as related to #326

In GitLab by [Gitlab user @jjardon] on May 31, 2018, 10:50
changed the description

In GitLab by [Gitlab user @toscalix] on May 31, 2018, 10:55
marked this issue as related to #414

In GitLab by [Gitlab user @toscalix] on May 31, 2018, 10:57
This documentation has an impact on Baserock. Check #414

In GitLab by [Gitlab user @jjardon] on Jul 17, 2018, 15:03
changed the description

In GitLab by [Gitlab user @toscalix] on Aug 15, 2018, 15:30
marked this issue as related to #224

In GitLab by [Gitlab user @jjardon] on Sep 20, 2018, 11:57
changed title from Document incompatibilities between bst{--1.0 and current master branch-} to Document incompatibilities between bst{+ releases+}

In GitLab by [Gitlab user @jjardon] on Sep 20, 2018, 11:57
changed the description

In GitLab by [Gitlab user @jjardon] on Sep 20, 2018, 11:57
changed the description

In GitLab by [Gitlab user @toscalix] on Sep 24, 2018, 09:56
assigned to [Gitlab user @jjardon]

In GitLab by [Gitlab user @jjardon] on Sep 24, 2018, 14:38
[Gitlab user @toscalix] what location will be a good place to put this?

In GitLab by [Gitlab user @jjardon] on Sep 24, 2018, 14:38
assigned to [Gitlab user @toscalix]

In GitLab by [Gitlab user @jjardon] on Sep 24, 2018, 14:38
unassigned [Gitlab user @jjardon]

In GitLab by [Gitlab user @toscalix] on Sep 24, 2018, 16:06
In my opinion, this should go to Known Issues web page, in a v1.0 section, since we have them identified, and we know workarounds, right [Gitlab user @jjardon] ?

In GitLab by [Gitlab user @jjardon] on Sep 24, 2018, 16:17
[Gitlab user @toscalix] I think it should go more in a special "migration" guide or something. People that wants to upgrade to a new version not necessarily will go to "known bugs",
I think it should go at https://buildstream.build/feature.html, so all the info about what you have to consider about using 1.2 is in a single place

In GitLab by [Gitlab user @jjardon] on Sep 24, 2018, 16:17
assigned to [Gitlab user @jjardon] and unassigned [Gitlab user @toscalix]

In GitLab by [Gitlab user @toscalix] on Sep 24, 2018, 16:25
Sadly the installation guide does not have an upgrade section, which is where it should belong.
I would prefer to add them to the known issues and make reference to them in the feature page through a link.

In GitLab by [Gitlab user @jjardon] on Sep 24, 2018, 16:30
mentioned in merge request website!92

In GitLab by [Gitlab user @jjardon] on Sep 24, 2018, 16:31
assigned to [Gitlab user @toscalix] and unassigned [Gitlab user @jjardon]

In GitLab by [Gitlab user @jjardon] on Sep 24, 2018, 16:31
Done: https://gitlab.com/BuildStream/website/merge_requests/92, [Gitlab user @toscalix] please review

In GitLab by [Gitlab user @LaurenceUrhegyi] on Sep 24, 2018, 17:50
Thanks for kicking it off, [Gitlab user @jjardon] !
I have corrected some typos only, would be great to also have [Gitlab user @juergbi] [Gitlab user @tlater] [Gitlab user @tristanvb] also take a look at the MR.

In GitLab by [Gitlab user @toscalix] on Sep 25, 2018, 08:43
closed via commit website[Gitlab user @0920a449aed01c06aefe4abbb34ca856cd3232ea]

In GitLab by [Gitlab user @toscalix] on Sep 25, 2018, 15:55
mentioned in commit website[Gitlab user @0920a449aed01c06aefe4abbb34ca856cd3232ea]

In GitLab by [Gitlab user @toscalix] on Nov 1, 2018, 15:56
marked this issue as related to #743


Add "cancel" event

Hi,
the "cancel" event should be very helpful, mainly for revert changes after changes introduced by "move" event.

I think cancel event should be added. Currently is it not possible to use hide alongside with change event.
Here's the example:
$('#colorpicker"]').spectrum({
  color: '#f4ece2',
  preferredFormat: 'hex',
  showInput: true,
  move: function(color) {
    $('body').css('backgroundColor', color.toHexString());
  },
  hide: function(color) {
    $('body').removeAttr('style');
  },
  change: function(color) {
    $('body').css('backgroundColor', color.toHexString());
  }
});

The expected behavior is:

Moving the cursor will change the background color,
Clicking "choose" button, the selected color will be applied to body (doesn't work),
Clicking "cancel" (or outside colorpicker)  will discard the changes.

Maybe you are right.

any update on this?

Hi @cekerholic, here's how I handle a similar situation :
$('#colorpicker').spectrum({
  show: function() {
    $(this).data('changed', false);
  },
  change: function(color) {
    $(this).data('changed', true);
  },
  hide: function(color) {
    if($(this).data('changed')) {
        // changed
    } else {
        // cancelled
    }
  }
});

This works because change event is fired before the hide event.

Thanks @ctekin, works like a charm :+1:

Using hide is a bad idea because it triggers both when a new color was chosen and when you cancel (incl. clicking outside the picker). There really needs to be a cancel (or revert) event that triggers when clicking Cancel or clicking outside the picker.
My problem right now is that I need to know whether the user canceled or not, in order to know if I should update another input field - and I can't do that without hacking it.

+1 for cancel event. you can simulate it with detecting a change between show and hide, but it's possible that they have opened and clicked "choose" without making any changes, my code will incorrectly detect this as a "cancel"

Hi folks, i need help , my scenario is bit different to this topic , i am using spectrum on a div and how can i detect if user clicks on select or cancel button? suppose a person selects the color and click on select color will be applied but next time when he again opens the picker and dont want to change the color now here i need help how can i get the selected color if he clicks on "select", and i have multiple spectrum objects on same page for different purpose. thanks.

@ctekin, thank you. I believe it would make sense to add a cancel event though (was looking for it in the doc, didn't found it, and had to search through the issues, almost posted a new one, until I found this "hack" of thought of sort).

Yeah - I know it's very old topic, but for any poor souls ending their search for close event here, I solved it this way:
$('#colorpicker').spectrum({
  hide: function(color) {
    if (document.activeElement.classList.contains("sp-cancel")) {
        // cancel was clicked
    }
  }
});


[VKVertScrollBox] Scroll position is off when using Android 12

As per title. The controls are slightly below where they should be, at least for me

Any update on this issue, please?
I've also encountered this on a Samsung Galaxy Z Fold3 5G running Android 12.
However, weirdly enough... positioning is correct on the large screen (i.e. when the device is open) but not correct when using the smaller front screen.
Let me know if you'd like to see some screen shots.
Thanks in advance.

Thanks for the reminder - I may get to this in the next couple of days

That would be much appreciated.

An update: I started looking at this a day or so ago - it seems on later versions of Android the reported size of the virtual keyboard is not accurate, or some other factor is causing the problem. Still looking into it..

Please see: https://github.com/DelphiWorlds/Playground/tree/main/Demos/VKObserver


Actions an hour

I'm always curious about how much money an hour I make as I level up when skilling so I'd like for actions an hour to be listed in the same tab as xp/hr so instead of having to divide xp/hr by xp per action it would be nice to see how much actions I do in an hour.

There is already actions/hr when you hover over the progress bar


Create sample NBS order ORM & ADT messages, based on any existing HL7 standards

Accidentally created -- closing


Other locale support

Hi,
I didn't see any support to other languages / locales.
Example: longBusinessSummary in other language different from English.
Is there any plans?

Do LLMs solve this?


[docs] Update the Code of Conduct version

The project is currently using the Code of Conduct adapted from Contributor Covenant, version 1.0.0 which is outdated. The latest one is v2.1.
Please let me know whether you want to update to the latest version as I might be able to add a PR for that.

That would be fabulous, looking forward to the PR


Mathjax3 does not support the syntax for '//' line breaks

Issue Summary
The MathJAX3 upgrade does not support the syntax for '//' line breaks, so we have to use things like $\displaylines{x = A + B \ y = b + c}$ to wrap lines, which is not very nice to us. Is there any way to configure MathJAX3 to support '//'?
Technical details:

MathJax Version: 3.2
Client OS: (e.g., Mac OS X 10.8.4)
Browser: (e.g., Chrome 29.0.1547.57)

I am using the following MathJax configuration:
MathJax = {
    tex: {
      inlineMath: [['$', '$']],   // inlineMath: [['$', '$'], ['\\(', '\\)']], // 行内公式选择符
      displayMath: [['$\begin', '\end$'], ['\\[\begin', '\end\\]']],
      processEscapes: true,
      processEnvironments: false,
      packages: ['base', 'ams', 'noerrors', 'noundefined', 'cancel', 'color', 'bbox', 'mhchem']
    },
    options: {
      skipHtmlTags: ['script', 'noscript', 'style', 'textarea', 'pre', 'code', 'a'],
      ignoreHtmlClass: 'comment-content'
    },
    loader: {
      load: ['[tex]/ams', '[tex]/noerrors', '[tex]/noundefined', '[tex]/cancel', '[tex]/color', '[tex]/bbox', '[tex]/mhchem']
    }
};

and loading MathJax via
<script<EMAIL_ADDRESS>
Supporting information:
Is there any syntax other than '//' that is not supported after upgrading to MathJAX3?  Or can you give me a copy of the latex syntax table supported by MathJax? Thank you!

This is a duplicate of #2312.
We have just released v4.0.0-alpha.1, which includes the line-breaking implementation.  You may wish to give that a try.
Also, I believe you mean \\ rather than //, right?
There is a list of supported LaTeX commands in the MathJax documentation.


I don't understand 1

I don't understand anything about devops.
I don't know how to use docker.

issue resolved


chore(main): release 0.3.3

:robot: I have created a release beep boop
0.3.3 (2023-11-09)
Bug Fixes

neo4j docker Java must be pinned to jdk 17 (25c0f32)
rm datasets file input, and fix version output since it changed in v15 (d2c4d90)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/socialgene/sgnf/releases/tag/v0.3.3 :sunflower:


Migrate to ChannelBuffers.push

This is a reland of https://github.com/flutter/flutter/pull/76288 which was reverted in https://github.com/flutter/flutter/pull/81226 due to google3 issues. Those should be solved by cl/370540746.
Also includes code to remove a legacy TODO whose referenced issue (#51885) is now closed.
See #77454, #79061, and #80003 for earlier discussions. This was last reverted in https://github.com/flutter/flutter/pull/81316.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

cc @goderbauer who reviewed the previous PRs that this lands; this does not have any code changes from the last two PRs that landed as far as I am aware.

RSLGTM (reland)


Update delay

Pull Request instructions:

Please follow the update protocol
Answer the questions below in detail. Your responses will be emailed to experimenters.
If the experimenters must do anything new, provide detailed step by step instructions on the wiki
Use markdown syntax in order for your comments to be rendered reliably in the email: "1." instead of "1)", use four spaces for indents.
If you use the keyword "skip email" in the title, it will skip the email updates
Merges from "develop" into "production_testing" should use the keyword "production merge" in the title for reliable indexing of updates
Merges from "production_testing" into "main" should use the keyword "update main"

Describe changes:
The previous task logic was that if a lick was detected, the delay would be restarted.
The new logic is the ITI and delay will restart if a lick is detected during the delay period. The duration of the new ITI and delay is the same as in the current trial.
Old:

New:

What issues or discussions does this update address?
The animal shows very high early licking rate. The new delay logic (similar to what Sue used before. The difference is that in Sue's case the new ITI and delay were redrew from exponential distribution) may help reduce the early licking rate.
Related to discussion
https://github.com/AllenNeuralDynamics/aind-behavior-blog/issues/309
https://github.com/AllenNeuralDynamics/aind-behavior-blog/discussions/323
Describe the expected change in behavior from the perspective of the experimenter

The task structure is changed. Licking during the delay will restart the entire ITI and delay.
The format of B_DelayStartTime and B_DelayStartTimeHarp is changed from a matrix to a list, and  each trial may have multiple values due to repeated delays.

Describe the outcome of testing this update on a rig in 447

[x] Tested on my own computer
[ ] Tested in 447

@ZhixiaoSu @hanhou I am updating the delay period. The proposed logic is when we detect early licks during the delay period, the entire ITI and delay will be restarted with the same parameters of the current trial. Here, I don't plan to redraw the exponential distribution of the ITI and delay. Otherwise, it will make the code more complex. Do you have any concerns?

@ZhixiaoSu @alexpiet This is ready for review. Hi Sue, can we find a time to test it together?

Yes, let's test it together! I think it may make a big change in how some animals do the task. When testing the lick sensors, I was touching the lick tubes constantly to get more 'licks' and I feel every time I stop for around 1s, I hear a go cue. This is probably because of the 1s fixed punishment. If a mouse also feels this way, it can explain why they have more iti lickings: they thought go cue comes when they keep licking and then stop.

"When testing the lick sensors, I was touching the lick tubes constantly to get more 'licks' and I feel every time I stop for around 1s, I hear a go cue. "

You mean it's like a hallucination?

Yes, let's test it together! I think it may make a big change in how some animals do the task.

Yes, I think it's a big change.

The task structure is changed. Licking during the delay will restart the entire ITI and delay.

Why structure it this way? Why not simply extend the ITI after each lick?

The format of B_DelayStartTime and B_DelayStartTimeHarp is changed from a matrix to a list, and each trial may have multiple values due to repeated delays. TransferToNWB.py needs to be updated to make it compatible with this change.

I dont understand the need to change the structure. Can you explain? Either way, you should update TransferToNWB.py before merging.

The task structure is changed. Licking during the delay will restart the entire ITI and delay.

Why structure it this way? Why not simply extend the ITI after each lick?

The format of B_DelayStartTime and B_DelayStartTimeHarp is changed from a matrix to a list, and each trial may have multiple values due to repeated delays. TransferToNWB.py needs to be updated to make it compatible with this change.

I dont understand the need to change the structure. Can you explain? Either way, you should update TransferToNWB.py before merging.

Mice showed very high early licking rate (~50%), which is detrimental to experiments (ephys, ophys and optogenetics ...). Previously, we usually use small delay duration or fixed delay duration, and we think this is one of the reasons of high early licking rate. The punishment was minimal for small delay duration and some mice showed regular licking pattern (for example 2Hz) for fixed delay duration.
Sue restarted the entire trial when detecting early licking. And mice trained with this task structure also showed low early licking rate (caveat here is there are other factor may also contribute to early licking, for example, sound decibel). This PR replicates Sue's task structure.

The new delay structure has two advantages compared to the current one:

Time out punishment is longer, so the mouse got more punishment when it licks before go cue comes;
Time out length is unpredictable, so the mouse won't have the 'hallicination' that go cue comes in ~1s when it stops licking after continuous licking.

The task structure is changed. Licking during the delay will restart the entire ITI and delay.

Why structure it this way? Why not simply extend the ITI after each lick?

Because this requirement can be too strict for the mouse. It can be hard for them to hold the licks right after harvesting reward.

The new delay structure has two advantages compared to the current one:

Time out punishment is longer, so the mouse got more punishment when it licks before go cue comes;
Time out length is unpredictable, so the mouse won't have the 'hallicination' that go cue comes in ~1s when it stops licking after continuous licking.

The task structure is changed. Licking during the delay will restart the entire ITI and delay.

Why structure it this way? Why not simply extend the ITI after each lick?

Because this requirement can be too strict for the mouse. It can be hard for them to hold the licks right after harvesting reward.

Can you draw a diagram of how timing changes?
I'm ok with whatever the distribution of timing you want, I'm mostly concerned about how you implemented it and breaking downstream code (like TransferToNWB)

The new delay structure has two advantages compared to the current one:

Time out punishment is longer, so the mouse got more punishment when it licks before go cue comes;
Time out length is unpredictable, so the mouse won't have the 'hallicination' that go cue comes in ~1s when it stops licking after continuous licking.

The task structure is changed. Licking during the delay will restart the entire ITI and delay.

Why structure it this way? Why not simply extend the ITI after each lick?

Because this requirement can be too strict for the mouse. It can be hard for them to hold the licks right after harvesting reward.

Can you draw a diagram of how timing changes?
Borrowing Han's slide:

The shaded area is time out punishment for early licks. In current version. it is a fixed value, 1s (another nolick DelayPeriod). We want to change it to exponential distribution + DelayPerod, so that the punishment is longer and timing of go cue is less predictable.

I'm ok with whatever the distribution of timing you want, I'm mostly concerned about how you implemented it and breaking downstream code (like TransferToNWB)

I agree. And I think that depends on how trials are defined in this task.

I also have some questions about the implementation

Is there a simpler way to implement the change without modifying the bonsai workflow too much? For example, can you add a general "early_lick_punishment" timeout node with a flexible length? This node is triggered by an early lick and always goes back to the start of the delay period. By doing this, you don't need to combine the "ITI" and "delay" into a new "ITIandDelay" node. There are at least two advantages:

after all, the early lick punishment should not be equated with restarting an ITI, given that we are not technically initiating a new trial and thus it doesn't make sense to create a new inter-trial-interval,
by not locking the early lick punishment to any existing trial structure like ITI or delay, we have the full control of the early lick punishment length on the trial-by-trial basis. For example, we can predefined its length in bonsai and pass it via the OSC message at the start of each trial like you do with other parameters. Then we still have the flexibility to set it to be either a new delay or a new ITI or any other values. (btw, did we generate exponential distribution for ITI and delay inside bonsai or in the GUI?)

If what I describe above is not feasible, is there any room for improvement in your current implementation?

I'm not a bonsai expert, so can you explain the operators in the yellow circles? What is GoCue2, DelayStart2, Pass2? What are Licks and Licks2? The left and right lick spouts? Why do we have an additional purple Pass for the upper branch of Licks?

A general comment: it would be great to always ask our bonsai expert @bruno-f-cruz to review out bonsai workflow.

after all, the early lick punishment should not be equated with restarting an ITI, given that we are not technically initiating a new trial and thus it doesn't make sense to create a new inter-trial-interval,

You mean just restart the delay, not the ITI+delay?

by not locking the early lick punishment to any existing trial structure like ITI or delay, we have the full control of the early lick punishment length on the trial-by-trial basis. For example, we can predefined its length in the GUI and pass it via the OSC message at the start of each trial like you do with other parameters. Then we still have the flexibility to set it to be either a new delay or a new ITI or any other values. (btw, did we generate exponential distribution for ITI and delay inside bonsai or in the GUI?)

This is related to my initial question "I am updating the delay period. The proposed logic is when we detect early licks during the delay period, the entire ITI and delay will be restarted with the same parameters of the current trial. Here, I don't plan to redraw the exponential distribution of the ITI and delay. Otherwise, it will make the code more complex. Do you have any concerns?"
If we redrew the exponential distribution for ITI and delay for early licking, it would make the code much more complex. Right now, we actually have few changes.

yellow circles? What is GoCue2, DelayStart2, Pass2? What are Licks and Licks2?

The Pass node (yellow) is controlling if Licks can go through it to restart the ITI+Delay. The Pass node is controlled by the GoCue, DelayStart and purple Pass nodes. The original GoCue is a behavior subject, which will be triggered every time the ITIAndDelay (SelectMany) node is triggered, and this is not desired. So, I transferred the original GoCue, DelayStart and purple Pass nodes to publish subject to get the GoCue2, DelayStart2 and Pass2. GoCue2 will close the yellow Pass node, DelayStart2 will open the yellow Pass node and the Pass2 will close the Pass node.
I'm happy to go through with you more details.
That would be great if @bruno-f-cruz could help review the workflow.

This change of trial structure is significant and we should be very cautious. I suggest we go through these steps:

test the task logic with hand thoroughly
test with at least two mice and for at least one week of training, preferably one naive and one well-trained. This means only two days on production_test as usual may not be enough and we may need to make a special case for testing this change.
As Alex said, before even merging to develop, please also make corresponding changes in TransferToNWB. After some test data are collected at step 2, test NWB conversion and make sure all downstream pipeline still works, ie, the session should show up in the app. If not, I'll have to make further modifications to the pipeline.
Importantly, we should also make sure the modified TransferToNWB and downstream pipeline are still fully compatible with previous data format.

Sue and I will test it in the behavior rig.
Let's see how we implement this. I think Sue already have a lot of sessions trained with this task structure.
3 and 4. I've updated the GUI code so that we don't need to update the TransferNWB.py. I realize it's best to leave the TransferNWB.py file intact if possible.

soon

The current licking pattern is terrible. We cannot make a meaningful interpretation from this.

soon

The current licking pattern is awful. We cannot make a meaningful interpretation from this.

I mean for later experiments (optogenetics, ephys ...)

@XX-Yin @ZhixiaoSu This PR was a big update that @hanhou and I had both expressed some concerns about. In the future, before merging a PR, please get explicit approval from anyone who expresses concerns.

one is only merging to development, but not to production (main) yet.

FYI, It was pushed to develop last week, so its part of this week's testing (today), and will be merged into main tomorrow unless we raise an issue.
I understand the urgency of fixing the mouse motivation. I think @hanhou and I mostly had concerns about how the data was being organized internally. Han was out last week, and I was deferring to his expertise. So we had a communication breakdown.
The current situation is fine, I just wanted to raise the issue for future updates.

@alexpiet I don't change the NWB file, so it should maintain the downstream analysis. I agree we need more agreements to merge cases like this in the future. Let's talk about it more tomorrow.


Why digest fields are displayed as none Type?

$ docker images --digests=true
REPOSITORY                         TAG                 DIGEST              IMAGE ID            CREATED             VIRTUAL SIZE
docker.io/mysql                    5.6.26              <none>              a128139aadf2        2 weeks ago         283.8 MB
docker.io/redis                    latest              <none>              0ff407d5a7d9        3 weeks ago         109.5 MB
docker.io/registry                 2.0.1               <none>              f9684be4155a        4 weeks ago         548.6 MB
docker.io/busybox                  latest              <none>              8c2e06607696        3 months ago        2.43 MB
gcr.io/google_containers/pause     0.8.0               <none>              6c4579af347b        12 months ago       239.8 kB

and where will use digest?

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

please try docker images --no-trunc | head

Yeah, I don't understand this either.
ping @stevvooe

@aaae The digest provides a content-address for docker images. The missing values are an unfortunate side-effect of how we store and manage image digests. Currently, these are stored as tags and only under certain conditions. Images that don't have a digest (which can be all of them) will have the digest displayed as <none>.
As we better integrate content-addressability into the daemon itself, these values will become more stable and be available for all images.

Should we keep this issue open? The original question has been answered

@thaJeztah We should fix this bug.
@aaronlehmann @tonistiigi Will we get this fixed in the future, such that a digest is available for every image?

In the near future, the image ID will become a content address. In this sense, a digest will be available for every image.
However, the current digest field is used to identify manifest digests for images that have been pulled by digest rather than by tag. This concept will still exist when images are content-addressable. So this particular digest field will still only apply to some images.

However, the current digest field is used to identify manifest digests for images that have been pulled by digest rather than by tag. This concept will still exist when images are content-addressable. So this particular digest field will still only apply to some images.

This was more of a side-effect of the implementation rather than an intended feature. If we can remove this behavior, and have a proper digest for everything, it would be a lot better.

It seems this issue still exist.  When will we would fix this ?
root@dccee5a197a7:/go/src/github.com/docker/docker# docker images
REPOSITORY          TAG                 IMAGE ID            CREATED             SIZE
ouruser/sinatra     devel               ffccd5f4334b        6 minutes ago       452.4 MB
ouruser/sinatra     v2                  ffccd5f4334b        6 minutes ago       452.4 MB
ubuntu              latest              1c9b046c2850        10 days ago         187.9 MB
centos              latest              61b442687d68        3 weeks ago         196.6 MB
busybox             latest              d9551b4026f0        5 weeks ago         1.113 MB
nate/dockviz        latest              3668e74867b8        9 weeks ago         5.947 MB
training/sinatra    latest              49d952a36c58        19 months ago       447 MB
root@dccee5a197a7:/go/src/github.com/docker/docker# docker version
Client:
 Version:      1.10.0-dev
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   67929bd
 Built:        Fri Jan 15 03:09:06 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.10.0-dev
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   67929bd
 Built:        Fri Jan 15 03:09:06 2016
 OS/Arch:      linux/amd64
root@dccee5a197a7:/go/src/github.com/docker/docker# docker images --digests
REPOSITORY          TAG                 DIGEST              IMAGE ID            CREATED             SIZE
ouruser/sinatra     devel               <none>              ffccd5f4334b        About an hour ago   452.4 MB
ouruser/sinatra     v2                  <none>              ffccd5f4334b        About an hour ago   452.4 MB
ubuntu              latest              <none>              1c9b046c2850        10 days ago         187.9 MB
centos              latest              <none>              61b442687d68        3 weeks ago         196.6 MB
busybox             latest              <none>              d9551b4026f0        5 weeks ago         1.113 MB
nate/dockviz        latest              <none>              3668e74867b8        9 weeks ago         5.947 MB
training/sinatra    latest              <none>              49d952a36c58        19 months ago       447 MB
root@dccee5a197a7:/go/src/github.com/docker/docker# 447 MB

@aaronlehmann Shouldn't these digests exist in the latest version?

Perhaps they should be combined, i.e. currently an image has either a tag or a digest in the list, in that case the head could show TAG / DIGEST

@thaJeztah They are in no way equivalent and we should not be reinforcing that idea.

I can confirm this still is an issue with Docker 1.10.1. It prevents any reasonable form of automation with Docker Content Trust, as the only time the digest is available is in the output from docker push, e.g.
# docker push redacted.example.com/namespace/image:tag
[The push refers to a repository [redacted.example.com/namespace/image]
b40428476b7a: Layer already exists
6057236047d9: Layer already exists
dbcbac958d69: Layer already exists
ba60a0e9f154: Layer already exists
4c7a05b9f3c4: Layer already exists
5f70bf18a086: Layer already exists
4915f908919a: Layer already exists
af6d4541205c: Layer already exists
b90cf634e321: Layer already exists
b7fb86c085cb: Layer already exists
e7788833b83a: Layer already exists
27b7abd92fd9: Layer already exists
5c5b99a3d7e9: Layer already exists
ed18fe2d435e: Layer already exists
628a915caf0d: Layer already exists
tag: digest: sha256:293bdd0ada30f83d00179cb72123b9377ddf566d7b14fa8f4704184f3ecb90a7 size: 21037
Signing and pushing trust metadata
Enter passphrase for repository key with ID d19614a (redacted.example.com/namespace/image): ](url)

Neither docker images --digests nor docker inspect redacted.example.com/namespace/image:tag will reveal the digest.

Agreed that this is still a bug:

The digests should always be available.
The digests available should represent the identifier available in a remote registry.

Should we close #20931  @thaJeztah

Also, would expect that after pulling an image by tag and digest that they would show as one image that has a tag and a digest, but that is not the case.
$ docker images --digests
REPOSITORY                     TAG                 DIGEST                                                                    IMAGE ID            CREATED             SIZE
debian                         jessie              <none>                                                                    f50f9524513f        5 days ago          125.1 MB
debian                         <none>              sha256:a9c958be96d7d40df920e7041608f2f017af81800ca5ad23e327bc402626b58e   f50f9524513f        5 days ago          125.1 MB

@HuKeping yes, let me close that one

As the present code, in a DCT pull, daemon will first pull the image by digests and then tag it with a tag, latest for example. So there are two images after a DCT pull and one with  in tag field and the other with  in digest field.
How about take the digest info with it when doing the tag action and delete the digest one when finish the tag action?

Any milestone for this?

I believe this may be preventing uploading tags on existing images without checking all the layers?

right now,  this is still a bug

Ping @dmcgowan will this be resolved by https://github.com/docker/docker/pull/23996, or are more changes needed?

#23996 does address this by ensuring that anytime a manifest is created or pulled the digest is stored. This will reduce the occurrence of "" in the digest field, but they will still occur before an image manifest is created or seen. The UX is also greatly improved in regards to images which do have a digest, no longer will the output show tag and digest as 2 different lines, but combined together when referring to the same image id.
As for more changes needed, I think we will have to wait and see. Ideally we always have a digest but that is not possible without having the distribution artifact hashes, which are not currently calculated on commit.

Run a local test, #23996 seems works for me, thanks!

The digest field will no longer be <none> after push and pull operations. This field can now be used after push to get the manifest digest which was pushed to the registry. Closing this issue as resolved.

@dmcgowan, I'm still seeing this after pushing.
$ docker push example.com/foo/bar:master
The push refers to a repository [example.com/foo/bar]
9ce98538809e: Layer already exists
91a8f3990fa4: Layer already exists
837094365fb4: Layer already exists
9708155555f8: Layer already exists
b91611bea4a8: Layer already exists
148b5a1af9ed: Layer already exists
011b303988d2: Layer already exists
master: digest: sha256:164b5fc83e34c2c1d59a430c7b64ecd010902f13c62ba9dcc50efb771c4060b2 size: 12564

$ docker images --digests example.com/foo/bar:master
REPOSITORY                 TAG           DIGEST              IMAGE ID            CREATED             SIZE
example.com/foo/bar   master              <none>              d594640ae3af        About an hour ago   63.02 MB

$ docker images --no-trunc example.com/foo/bar:master
REPOSITORY                         TAG                 IMAGE ID                                                                  CREATED             SIZE
example.com/foo/bar   master              sha256:d594640ae3af27c8552d1274ce766107c3bfbefe9585d42f24f2ca7d2476a6f9   About an hour ago   63.02 MB

$ docker version
Client:
 Version:      1.12.2-rc1
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   45bed2c
 Built:        Tue Sep 27 23:38:15 2016
 OS/Arch:      darwin/amd64
 Experimental: true

Server:
 Version:      1.12.3
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   6b644ec
 Built:        Wed Oct 26 22:01:48 2016
 OS/Arch:      linux/amd64

And it looks like I'm getting two different hashes, sha256:d594640ae3af27c8552d1274ce766107c3bfbefe9585d42f24f2ca7d2476a6f9 and sha256:164b5fc83e34c2c1d59a430c7b64ecd010902f13c62ba9dcc50efb771c4060b2, seems odd?

It seems that the 'image id' shown by docker images --no-trunc is not a digest (confirmed in https://github.com/docker/distribution/issues/1662).
$ docker images --no-trunc example.com/foo/bar:master
REPOSITORY                         TAG                 IMAGE ID                                                                  CREATED             SIZE
example.com/foo/bar   master              sha256:d594640ae3af27c8552d1274ce766107c3bfbefe9585d42f24f2ca7d2476a6f9   About an hour ago   63.02 MB

$ docker run --rm example.com/foo/bar:master@sha256:d594640ae3af27c8552d1274ce766107c3bfbefe9585d42f24f2ca7d2476a6f9
Unable to find image 'example.com/foo/bar@sha256:d594640ae3af27c8552d1274ce766107c3bfbefe9585d42f24f2ca7d2476a6f9' locally
docker: Error response from daemon: manifest unknown: manifest unknown.
See 'docker run --help'.

$ docker push example.com/foo/bar:master
The push refers to a repository [example.com/foo/bar]
9ce98538809e: Layer already exists
91a8f3990fa4: Layer already exists
837094365fb4: Layer already exists
9708155555f8: Layer already exists
b91611bea4a8: Layer already exists
148b5a1af9ed: Layer already exists
011b303988d2: Layer already exists
master: digest: sha256:164b5fc83e34c2c1d59a430c7b64ecd010902f13c62ba9dcc50efb771c4060b2 size: 12564

$ docker run --rm example.com/foo/bar:master@sha256:164b5fc83e34c2c1d59a430c7b64ecd010902f13c62ba9dcc50efb771c4060b2
^C⏎

(the second docker run ... worked)
So is there a way to get the real digest for an image that has been pushed already (apart from docker push ...)?
WTF: just ran a couple more commands
$ docker images --digests quay.io/cerberos/hubnet-frontend:master
REPOSITORY                         TAG                 DIGEST              IMAGE ID            CREATED             SIZE
quay.io/cerberos/hubnet-frontend   master              <none>              d594640ae3af        2 hours ago         63.02 MB

This is the command I pasted ^^^, no digest, buuuuuut
$ docker images --digests quay.io/cerberos/hubnet-frontend
REPOSITORY                         TAG                 DIGEST                                                                    IMAGE ID            CREATED             SIZE
quay.io/cerberos/hubnet-frontend   master              sha256:164b5fc83e34c2c1d59a430c7b64ecd010902f13c62ba9dcc50efb771c4060b2   d594640ae3af        2 hours ago         63.02 MB
quay.io/cerberos/hubnet-frontend   <none>              sha256:6fd65f7141dc61adeb9f262784b1aca9848866d2d4b68c3a10ecf0aa2d1edef6   1c270a40466d        3 days ago          63.02 MB
quay.io/cerberos/hubnet-frontend   <none>              sha256:0b9e94db5aaedb6f6407778d803e99b0e7dd2e710842611873411717dd51e7b8   c42d40e9c60b        3 days ago          63.02 MB
quay.io/cerberos/hubnet-frontend   <none>              sha256:a5aee68356dba08f0576396d8cbd23cf48630391e00d3197c2b237cbedc8a7a2   998b383b2bcb        2 weeks ago         63.02 MB
quay.io/cerberos/hubnet-frontend   <none>              sha256:076ff9b7dc7cadf2a74d7bf5b7226c0b69ef18e9d29834c2c2e0f3efca019aba   a6df32f2ad19        2 weeks ago         62.08 MB
quay.io/cerberos/hubnet-frontend   <none>              sha256:a8a4411a3f6f89c9215a24adfa5ae3c62d04052c53039fd594002a78118ecf1d   7bc456afeef5        2 weeks ago         62.08 MB
quay.io/cerberos/hubnet-frontend   <none>              sha256:eb448ad5b5d1a6b8d425e90c37056c82c90809ffbf3e6486f601351893fe2785   973e019e1f45        4 weeks ago         429.8 MB

I'm going to create a separate issue for digest not shown when image tag specified.

@j0hnsmith thanks for creating separate issue. Excluding the digest when listing a specific tag vs including when no tag is given is not intended behavior.

https://github.com/docker/docker/issues/29901


Where is the link of the app?

Is this working where is the link?

The usage description of the app is here. Unfortunately, it is still currently work-in-progress.


The latest DockerDesktopEdge <IP_ADDRESS> does not show up

Brief description of your issue
A winget search docker still shows a previous version of Docker Desktop Edge, although a PR to update the package version is already merged.
Steps to reproduce
$ winget search docker

Expected behavior
$ winget search docker
Name                Id                       Version       Matched
--------------------------------------------------------------------------
Docker Desktop      Docker.DockerDesktop     <IP_ADDRESS>.45519 Moniker: docker
Docker Desktop Edge Docker.DockerDesktopEdge <IP_ADDRESS>.46329

Actual behavior
$ winget search docker
Name                Id                       Version       Matched
--------------------------------------------------------------------------
Docker Desktop      Docker.DockerDesktop     <IP_ADDRESS>.45519 Moniker: docker
Docker Desktop Edge Docker.DockerDesktopEdge <IP_ADDRESS>.46268

It still shows DockerDesktopEdge <IP_ADDRESS> instead of <IP_ADDRESS>  🤔
The PR for <IP_ADDRESS> 11 days ago: https://github.com/microsoft/winget-pkgs/pull/2121
The PR for <IP_ADDRESS> 5 days ago: https://github.com/microsoft/winget-pkgs/pull/2173
Environment
$ winget --info
Windows Package Manager v0.1.41331 Vorschau
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19041.329
Package: Microsoft.DesktopAppInstaller v1.0.41331.0

Links:
  Privacy Statement: https://aka.ms/winget-privacy
  License agreement: https://aka.ms/winget-license
  3rd Party Notices: https://aka.ms/winget-3rdPartyNotice
  Homepage:          https://aka.ms/winget

No merges after 1th July is showing up.

This and #2274 seem like duplicates, and the root question is: How often can we expect the winget 'source' to get updated with merged-in packages once a user does  winget source update? Understandably this product has not launched 1.0 yet, so

What can we expect in terms of time-to-show-up from now until 1.0 launches?
Is there a planned target time-to-show-up for 1.0?

Thanks!

From my own PR experience you don’t have to do winget source update (or it does it in the background very often).
New merges should show up after a few hours max.
Something is clearly broken because its been working like it should until 1th july.


blocked services list slow to load

Description
Loading the list of blocked services can sometimes take 6-7 seconds, is there a way to troubleshoot or improve this? I remember early versions of this app where all screens loaded very fast. There is no slowdown when i go to the :8080 admin interface of adguard in my browser (on the same wifi network). Happy to provide more (debug) info if needed.
And, many thanks you for this great app !

try this
https://cdn.discordapp.com/attachments/1277836331501883449/1280291965879193731/fix.zip?ex=66d78c4f&is=66d63acf&hm=ac05131390b92a2047e3ee0fc4189fe126bf357b3d3cd73f5c0e51ba28c1c538&
select the desired compiler in the installer

I haven't seen that issue on my device/adguard instance

Description
Loading the list of blocked services can sometimes take 6-7 seconds, is there a way to troubleshoot or improve this? I remember early versions of this app where all screens loaded very fast. There is no slowdown when i go to the :8080 admin interface of adguard in my browser (on the same wifi network). Happy to provide more (debug) info if needed.
And, many thanks you for this great app !

I have also been using it for a long time, but I haven't encountered such an issue yet. It is most likely a problem caused by your host.

Good to know. It's strange that the desktop browser loads instantly almost but the app does not. I'm running adguard on an edgerouter-X.

As @mikropsoft said, I think it's a problem with your instance. No one else has reported something similar and I haven't seen such behavior on testing.

I think so yes, but the fact that i can load the adguard website in desktop firefox instantly has to mean something.


Updating Sphinx and other doc dep versions

On OSX, I tried a fresh install and came across the following error when attempting to build the docs via make html:

Updating the documentation libraries fixes the issue.

I was able to reproduce this bug in a macOS environment, but not inside the vagrant-wagtail-develop with the same libraries versions. :/
Extended Traceback:
# Sphinx version: 1.5.2
# Python version: 3.5.1 (CPython)
# Docutils version: 0.13.1 release
# Jinja2 version: 2.9.5
# Last messages:

# Loaded extensions:
Traceback (most recent call last):
  File "/Users/chimeno/Documents/experiments/testing/lib/python3.5/site-packages/sphinx/cmdline.py", line 295, in main
    opts.warningiserror, opts.tags, opts.verbosity, opts.jobs)
  File "/Users/chimeno/Documents/experiments/testing/lib/python3.5/site-packages/sphinx/application.py", line 199, in __init__
    self.setup_extension(extension)
  File "/Users/chimeno/Documents/experiments/testing/lib/python3.5/site-packages/sphinx/application.py", line 512, in setup_extension
    mod = __import__(extension, None, None, ['setup'])
  File "/Users/chimeno/Documents/experiments/testing/lib/python3.5/site-packages/sphinxcontrib/spelling/__init__.py", line 3, in <module>
    from .builder import SpellingBuilder
  File "/Users/chimeno/Documents/experiments/testing/lib/python3.5/site-packages/sphinxcontrib/spelling/builder.py", line 16, in <module>
    from enchant.tokenize import EmailFilter, WikiWordFilter
  File "/Users/chimeno/Documents/experiments/testing/lib/python3.5/site-packages/enchant/__init__.py", line 92, in <module>
    from enchant import _enchant as _e
  File "/Users/chimeno/Documents/experiments/testing/lib/python3.5/site-packages/enchant/_enchant.py", line 121, in <module>
    prefix_dir.contents = c_char_p(e_dir)
TypeError: bytes or integer address expected instead of str instance

After upgrading pyenchant to 1.6.8 the issue was resolved as mentioned above.

Looks like our Drone setup doesn't like installing pyenchant 1.6.8 - possibly because it isn't available as a .whl package and the .tar.gz has extra installation requirements. Please could someone figure out the extra incantations needed to get it to work, or see if 1.6.7 (which does have a .whl) is sufficient to avoid the error?

As far as I know, the 1.6.7 doesn't have a .whl for macOS. And the are only .whl for macOS and Win in 1.6.8.
I guess the best option could be to install the c  enchant library in the Drone setup, but I'm not acquainted with how to do that.

Doh, I'm overlooking the obvious here... Drone doesn't do anything with docs, so there's actually no reason to run pip install -e .[testing,docs] at all. Will remove the docs part and see if that fixes it...


The path "root.parameters.dimensions.Order Date.includeTime" cannot contain an empty value, but got null.

https://connection.keboola.com/admin/projects/5980/jobs/528926736?q=
Zrejme to vzniklo po nacteni LDM

duplicate of https://github.com/keboola/gooddata-writer-v3/issues/54


Sprint 57 ns 4227 create proxy box

Please ignore. I did not mean to open it against your fork.
However, feel free to take a look at the work we've added!


[KeyVault] - Consider providing a type for ISO8601 durations

Key Rotation at the service level has and will continue to use to ISO8601 durations. Currently we use strings to represent these; however, this can be a source of confusion and errors as developers must author them by hand.
To alleviate such issues, consider (along with your language architects) whether it makes sense to provide a structure that can convert to and from ISO8601 durations.
Initial thoughts on the API (in TS):
class Duration {
  static parse(input: string): Duration
  toISOString(): string

  years?: number
  months?: number
  weeks?: number
  hours?: number
  minutes?: number
  seconds?: number
}

Duration.parse("P3Y6M4DT12H30M5S").toISOString() => "P3Y6M4DT12H30M5S"

But of course this will change per language, and some languages might not want this in which case feel free to close the issue.

After discussion, we'll not be doing this for .NET in time for the next release but will consider it in the future. For now, we'll keep the property type a string.

There's been enough push back on this I'm closing. We can always revisit if we get enough customer demand.


Serve swagger from root

Currently, the root url is a 404. Instead, serve the swagger page.

Coverage decreased (-0.09%) to 73.905% when pulling a9e8fb556062c7604d90062c5032a5eb2f82b234 on gr/swagger-root into b1b43155cccb9def36657897719c442adddb03c5 on develop.


1.5 Backport MTL-2348 .editorconfig and assign-ncn-images.sh

Backports the extra parts from #4647 to 1.5.

/backport release/1.6


Add display-metrics-changed handler to place window on screen

Add display-metrics-changed handler to place window on screen when metrics change while the application is running.
Before submitting, please confirm you've

[x] read and understood our Contributing Guidelines
[x] completed Mattermost Contributor Agreement
[x] executed npm run lint:js for proper code formatting

Please provide the following information:
Summary
I added a listener for display-metrics-changed to handle changes while the app is running.
However the event doe snot fire on my OS so I'm not able to test this (+ I can't reproduce the bug on Linux so there is actually no bug to fix in my case).
However this might fix #471 if it works correctly...
Test Cases
See #471, place window on a dedicated monitor, remove that monitor. Hope the event fires and the window is moved to an existing screen.
Additional Notes
Not really tested, event does not fire on my OS. I hope it does on Window...

@jnugh Big thanks for this PR!
@yuya-oc can you help review/test this PR?

Sure. But it's hard today, so I will test tomorrow if possible.

Yeah for sure, not urgent 👍

I tested the artifact of https://circleci.com/gh/mattermost/desktop/940#artifacts . It looks the problem does not exist even when the event is not fired.
Possibly this might be needed to be solved by upstream Electron in future.

I have tested both of v3.6.0 and the artifact on Windows 10 by following the walk-through. It seems the problem happens when the window is minimized.
In that case, the problem was not fixed by this PR (in other words, this PR's approach might not be appropriate). Possibly a minimized window might have a special characteristic.

Thanks @yuya-oc, great work investigating this!
@jnugh would you have any thoughts on the most recent findings?

Running the check on window restore might b an option I guess...
Jason Blais<EMAIL_ADDRESS>schrieb am Mi., 19. Apr. 2017, 02:21:

Thanks @yuya-oc https://github.com/yuya-oc, great work investigating
this!
@jnugh https://github.com/jnugh would you have any thoughts on the most
recent findings?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mattermost/desktop/pull/509#issuecomment-295023548,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIv7ZA4C_NIITorIi4BVIVFCQllK0xSks5rxVOigaJpZM4M3_0l
.

Updated the commit. Added a restore handler (and use this globally across all browser windows).

Thanks Jonas! @yuya-oc can you help take a look?

As stated initially - this has not been verified to work correctly as I have no possibility to test on windows at the moment.
My test was to place the window off-screen (by closing the app and changing the restore coordinates) and check if it will be restored on-screen, which it did in my case. However that worked with prior versions of this PR as well and didn't fix the issue on windows.
So if this attempt does not work, someone else might be able to debug this further, by actually using windows for that ;)

Thanks @jnugh!
Here is the result.

Move the window to secondary monitor.
Minimize the window.
Start Remote Desktop session.
Restore the window.
The window should move to (0, 0), but it's restored at other coordinates.
Minimize and restore the window again. It's located at (0 ,0).

In my logging, the position was valid in getValidWindowPosition on display-metrics-changed and restore of step 3 and 4. But the position becomes invalid (too large x for me) on step 6. To solve this, I needed to remove display-metrics-changed. So only restore event was sufficient for me.
Does using only restore event make sense also on Linux?

I guess display-metrics-changed is called before the new screen size is actually calculated. A timeout might work also but when waiting for restore is enough I guess we should go with it.
So the Desktop I use does not even place windows off screen when removing a display but only when the window is reopened after removing the display it has been on previously.
So yeah as long as it works on windows this should be okay. As the position is still checked on window create.
I will update the commit.

Looks good for me on both of Windows and Mac. On Mac, originally the problem doesn't reproduce and I could not find any regression.

@jasonblais Is there anything to test in your side?

I don't have multiple monitors to try..but the work here looks good!
Thanks @jnugh and @yuya-oc! 👍

Thanks all!


Plugin Config Window not showing

Hi there,
My plugin config window isn't showing when i type in /eglo
Thank you!

I ve the same error, do u have endwalker? i haven't it and i think thats the error maybe, a friend that has endwalker works perfectly , i hope they gave us a answer soon!
Thank u too :D

Please test the new release to see if it is fixed

Same problem here. I'm testing the trial and I can install the plugin, but the config menu doesn't open either way.

Same problem here. I'm testing the trial and I can install the plugin, but the config menu doesn't open either way. Others plugins open with no problem.
Edit: Closing the FPS counter solved my problem.

wait, what? i'll look into this.

I solved this bug with the new version released, but i needed reboot the game and could see the configuration window

I solved this bug with the new version released, but i needed reboot the game and could see the configuration window

good to know. i still need to fix another issue i found in that window but it shouldn't stop ppl from using the plugin. In case of any error, please delete the Echoglossian.json file inside the %appdata%\XIVLauncher\pluginConfigs folder and restart the game

I already deleted json file (and everything too xD) but I cant open the config window to select language. :/

Fixed in the latest release. If you still get the error, reload the game after installing the plugin


fix(build): disable tsetse rule ban-expect-truthy-promise

Description:
Fixes Bazel build of rxjs with latest rules_typescript
Related issue (if exists):

Would you elaborate bit more details around changes?


Fix FTDI array-ouf-of-bounds crash

Crash happens when bad data (odd length) was received from the USB device. See commit messages for details.

Added another commit which fixes a data corruption bug with FT4232H chips. See https://github.com/XCSoar/XCSoar/issues/1031 for details


Kiali is not displaying trafic on the graph

Hi everyone,
First, thank you for creating Rancher, it's a pleasure to work on the product :)
My first bug report here/
THe Kiali Graph is not displaying trafic, the pod logs show that a TS label is missing.
Hope you can fix it !
Thanks.
What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):
Install Rancher 2.4.5
Install Istio 1.5.8 with Istio ingress Gateway
Deploy a pod, with a Service, DestinationRUle and Istio Gateway
Access the deployed app
Result:
Open Kiali, trafic is not displayed on graph
Other details that may be helpful:
In the Kiali pod logs :
Skipping {destination_service="workload1-service.fmr-istio.svc.cluster.local", destination_service_name="workload1-service", destination_service_namespace="fmr-istio", destination_workload="workload1", destination_workload_namespace="fmr-istio", request_protocol="http", response_code="200", response_flags="-", source_workload="istio-ingressgateway", source_workload_namespace="istio-system"}, missing expected TS labels
Same bug found here : https://github.com/istio/istio/issues/24924
Some one say that going with 1.18 of Kiali fix the bug, but apparently, 1.18 is not meant to be with Istio 1.5
https://github.com/kiali/kiali/issues/1007
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): 2.4.5
Installation option (single install/HA): Single

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Infrastructure Provider Azure
Machine type (cloud/VM/metal) and specifications (CPU/memory): Cloud (3x 8vCPU, 32GO RAM)
Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.0", GitCommit:"9e991415386e4cf155a24b1da15becaa390438d8", GitTreeState:"clean", BuildDate:"2020-03-25T14:58:59Z", GoVersion:"go1.13.8", Compiler:"gc", Platform:"windows/amd64"}

Server Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.6", GitCommit:"dff82dc0de47299ab66c83c626e08b245ab19037", GitTreeState:"clean", BuildDate:"2020-07-15T16:51:04Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):

19.3.12

Yes this was mentioned in the release notes. We can keep this issue open and use to track the upstream istio issue.

rancher/rancher:v2.4.6-rc4
Istio 1.5.9

Deploy a cluster
Enable Istio 1.5.9 in a cluster (or upgrade from a previous version)
Deploy some services and access
Traffic shows correctly in the Kiali graph


BUG: run_algorithm with no data source should default

to 'quantopian-quandl' bundle
For https://github.com/quantopian/zipline/issues/1374
I considered adding a unit test.  Because of its dependence on the global load of a bundle, it would depend on ingesting the real quantopian-quandl bundle first, or we'd need to register a replacement bundle globally for the span of the test.  So I haven't included a unit test here.

Coverage decreased (-0.006%) to 86.652% when pulling 91d7ded23e03006e7f9028649ad1f19fd51a406a on run_algo-defaults into df07f67614e2bc54c5474749ea5974e1c03e95b9 on master.

lgtm, do you think there is an easy test for this?

Not easy, at least. 😄

Coverage remained the same at 86.659% when pulling 3bdba2ec50a3d4e1f7df0f62b504a4924a31c255 on run_algo-defaults into df07f67614e2bc54c5474749ea5974e1c03e95b9 on master.


Render the Material widget with the physical model layer

@abarth

Feedback addressed - PTAL

Does this turn on the fancy new shadows? (i.e. Will this cause pixel-rendering changes for our internal clients who are tracking image diffs?  FYI @tvolkert)

Yes, this changes how shadows are rendered

This caused a number of major performance regressions.


Issue with load

Dear Team,
Server:
OS: ubuntu 18.04
Hosting Provider: AWS
CPU: 16vCPU
Server CPU Load during the session: 30 percent
Server and APP are running in different hosts
client:
Google Chrome Version 90 and above
Network  Latency : less than 50ms
CPU usage : Not more than 70 percent
we have conducted a VC Session with more than 200 participants with LASTN=5 and Mobile LASTN=1 once login to the session i dont hear anyone at all and all the options seems behaving weird and there are disconnections too.
Unmute and other options wont work at all clicking one option opens something else   also below are the continual error running logs on the server.
almost all people joined the meeting with Audio and video mute
All participants having the same issue and finally switched over to zoom meeting
Jun 12 09:19:39 ip-10-0-1-142 server.js[439697]: 2021-06-12T09:19:39.774Z edumeet-server:ERROR:Room "request" failed [error:"Error: Peer not yet joined at Room._handleSocketRequest (/var/lib/jenkins/workspace/edumeet/lib/Room.js:1269:12) at Socket. (/var/lib/jenkins/workspace/edumeet/lib/Room.js:809:9) at Socket.emit (events.js:315:20) at Socket.EventEmitter.emit (domain.js:467:12) at /var/lib/jenkins/workspace/edumeet/node_modules/socket.io/lib/socket.js:528:12 at processTicksAndRejections (internal/process/task_queues.js:75:11)"]
Jun 12 09:19:40 ip-10-0-1-142 server.js[439697]: 2021-06-12T09:19:40.262Z edumeet-server:ERROR:Room "request" failed [error:"Error: Peer not yet joined at Room._handleSocketRequest (/var/lib/jenkins/workspace/edumeet/lib/Room.js:1250:12) at Socket. (/var/lib/jenkins/workspace/edumeet/lib/Room.js:809:9) at Socket.emit (events.js:315:20) at Socket.EventEmitter.emit (domain.js:467:12) at /var/lib/jenkins/workspace/edumeet/node_modules/socket.io/lib/socket.js:528:12 at processTicksAndRejections (internal/process/task_queues.js:75:11)"]
Jun 12 09:19:40 ip-10-0-1-142 server.js[439697]: 2021-06-12T09:19:40.263Z edumeet-server:ERROR:Room "request" failed [error:"Error: Peer not yet joined at Room._handleSocketRequest (/var/lib/jenkins/workspace/edumeet/lib/Room.js:1269:12) at Socket. (/var/lib/jenkins/workspace/edumeet/lib/Room.js:809:9) at Socket.emit (events.js:315:20) at Socket.EventEmitter.emit (domain.js:467:12) at /var/lib/jenkins/workspace/edumeet/node_modules/socket.io/lib/socket.js:528:12 at processTicksAndRejections (internal/process/task_queues.js:75:11)"]
Jun 12 09:19:40 ip-10-0-1-142 server.js[439697]: 2021-06-12T09:19:40.449Z edumeet-server:ERROR:Room "request" failed [error:"NotFoundInMediasoupError: consumer with id "3248da8c-f671-4e41-bfa0-4db56a9d7737" not found at Room._handleSocketRequest (/var/lib/jenkins/workspace/edumeet/lib/Room.js:1275:12) at Socket. (/var/lib/jenkins/workspace/edumeet/lib/Room.js:809:9) at Socket.emit (events.js:315:20) at Socket.EventEmitter.emit (domain.js:467:12) at /var/lib/jenkins/workspace/edumeet/node_modules/socket.io/lib/socket.js:528:12 at processTicksAndRejections (internal/process/task_queues.js:75:11)"]
Jun 12 09:19:41 ip-10-0-1-142 server.js[439697]: 2021-06-12T09:19:41.306Z edumeet-server:ERROR:Room "request" failed [error:"NotFoundInMediasoupError: consumer with id "11be91aa-feba-4b0e-93c1-2726eb5cc864" not found at Room._handleSocketRequest (/var/lib/jenkins/workspace/edumeet/lib/Room.js:1275:12) at Socket. (/var/lib/jenkins/workspace/edumeet/lib/Room.js:809:9) at Socket.emit (events.js:315:20) at Socket.EventEmitter.emit (domain.js:467:12) at /var/lib/jenkins/workspace/edumeet/node_modules/socket.io/lib/socket.js:528:12 at processTicksAndRejections (internal/process/task_queues.js:75:11)"]
Jun 12 09:19:41 ip-10-0-1-142 server.js[439697]: 2021-06-12T09:19:41.824Z edumeet-server:ERROR:Room "request" failed [error:"NotFoundInMediasoupError: consumer with id "8c48bc6a-6464-4e9d-a73c-1829804ff0ba" not found at Room._handleSocketRequest (/var/lib/jenkins/workspace/edumeet/lib/Room.js:1275:12) at Socket. (/var/lib/jenkins/workspace/edumeet/lib/Room.js:809:9) at Socket.emit (events.js:315:20) at Socket.EventEmitter.emit (domain.js:467:12) at /var/lib/jenkins/workspace/edumeet/node_modules/socket.io/lib/socket.js:528:12 at processTicksAndRejections (internal/process/task_queues.js:75:11)"]

++ #862

we have again conducted meeting again with more than 200 users login from mobiles, desktops and laptops all are real participants like last time we faced same issue like before, in UI there is a flow of notifications of user join and sometimes error on server request
Server:
OS: ubuntu 18.04
Hosting Provider: AWS
CPU: 16vCPU
Ram Usage:  3GB out of 32GB
Server CPU Load during the session: 30 percent
Server and APP are running in different hosts
client:
Google Chrome Version 90 and above
Network Latency : less than 50ms
CPU usage : Not more than 70 percent
LAST N=5 and Mobile LASTN=1
below is the same log found
Jun 23 06:59:25 ip-10-0-1-142 server.js[601]: 2021-06-23T06:59:25.741Z edumeet-server:ERROR:Room "request" failed [error:"NotFoundInMediasoupError: consumer with id "b0f627af-47fe-4d8b-bb18-61cfdf310c57" not found at Room._handleSocketRequest (/var/lib/jenkins/workspace/edumeet/server/lib/Room.js:1275:12) at Socket. (/var/lib/jenkins/workspace/edumeet/lib/Room.js:809:9) at Socket.emit (events.js:315:20) at Socket.EventEmitter.emit (domain.js:467:12) at /var/lib/jenkins/workspace/edumeet/node_modules/socket.io/lib/socket.js:528:12 at processTicksAndRejections (internal/process/task_queues.js:75:11)"]
Jun 23 06:59:25 ip-10-0-1-142 server.js[601]: 2021-06-23T06:59:25.796Z edumeet-server:ERROR:Room "request" failed [error:"NotFoundInMediasoupError: consumer with id "215172ab-0721-44cc-93da-9166ee99d503" not found at Room._handleSocketRequest (/var/lib/jenkins/workspace/edumeet/server/lib/Room.js:1275:12) at Socket. (/var/lib/jenkins/workspace/edumeet/lib/Room.js:809:9) at Socket.emit (events.js:315:20) at Socket.EventEmitter.emit (domain.js:467:12) at /var/lib/jenkins/workspace/edumeet/node_modules/socket.io/lib/socket.js:528:12 at processTicksAndRejections (internal/process/task_queues.js:75:11)"]
Jun 23 06:59:25 ip-10-0-1-142 server.js[601]: 2021-06-23T06:59:25.898Z edumeet-server:ERROR:Room "request" failed [error:"NotFoundInMediasoupError: consumer with id "e59d2fb4-cfe1-4961-a430-4275a1e1576d" not found at Room._handleSocketRequest (/var/lib/jenkins/workspace/edumeet/server/lib/Room.js:1275:12) at Socket. (/var/lib/jenkins/workspace/edumeet/lib/Room.js:809:9) at Socket.emit (events.js:315:20) at Socket.EventEmitter.emit (domain.js:467:12) at /var/lib/jenkins/workspace/edumeet/node_modules/socket.io/lib/socket.js:528:12 at processTicksAndRejections (internal/process/task_queues.js:75:11)"]
Jun 23 06:59:25 ip-10-0-1-142 server.js[601]: 2021-06-23T06:59:25.905Z edumeet-server:ERROR:Room "request" failed [error:"NotFoundInMediasoupError: consumer with id "f7779349-9529-4c1a-8541-91fc3282e81e" not found at Room._handleSocketRequest (/var/lib/jenkins/workspace/edumeet/server/lib/Room.js:1275:12) at Socket. (/var/lib/jenkins/workspace/edumeet/lib/Room.js:809:9) at Socket.emit (events.js:315:20) at Socket.EventEmitter.emit (domain.js:467:12) at /var/lib/jenkins/workspace/edumeet/node_modules/socket.io/lib/socket.js:528:12 at processTicksAndRejections (internal/process/task_queues.js:75:11)"]
Jun 23 06:59:25 ip-10-0-1-142 server.js[601]: 2021-06-23T06:59:25.927Z edumeet-server:ERROR:Room "request" failed [error:"NotFoundInMediasoupError: consumer with id "5d8fd94d-06b8-48c1-9b57-9a21e9a58904" not found at Room._handleSocketRequest (/var/lib/jenkins/workspace/edumeet/server/lib/Room.js:1275:12) at Socket. (/var/lib/jenkins/workspace/edumeet/lib/Room.js:809:9) at Socket.emit (events.js:315:20) at Socket.EventEmitter.emit (domain.js:467:12) at /var/lib/jenkins/workspace/edumeet/node_modules/socket.io/lib/socket.js:528:12 at processTicksAndRejections (internal/process/task_queues.js:75:11)"]
Jun 23 06:59:25 ip-10-0-1-142 server.js[601]: 2021-06-23T06:59:25.973Z edumeet-server:ERROR:Room "request" failed [error:"NotFoundInMediasoupError: consumer with id "14f99a0b-8c66-4e7e-8360-3d27662b0425" not found at Room._handleSocketRequest (/var/lib/jenkins/workspace/edumeet/server/lib/Room.js:1275:12) at Socket. (/var/lib/jenkins/workspace/edumeet/lib/Room.js:809:9) at Socket.emit (events.js:315:20) at Socket.EventEmitter.emit (domain.js:467:12) at /var/lib/jenkins/workspace/edumeet/node_modules/socket.io/lib/socket.js:528:12 at processTicksAndRejections (internal/process/task_queue
Again it is shifted to Google meet
Thanks
Alex

Anyone is able to do load test with more than 100 participants with LASTN 5, i see there are drops in connections and after disconnection and connect back it takes so much time to get audio from the participants and also enabling audio mute takes so much time to respond.
Thanks


Calling Request from a state machine WhenEnter handler fails to compile on 8.0

Is this a bug report?
Yes
Can you also reproduce the problem with the latest version?
Yes, on 8.0.1.
Environment

Dotnet version: 6.0
Operating system: Windows
IDE (Rider, VS Code, Visual Studio): Visual Studio

Steps to Reproduce
The following code compiles under 7.x but fails to compile under 8.0.1:
WhenEnter(StartingExecution, x => x
    .Request(Execute, context => context.Init<ExecuteRule>(new { RuleId = context.Saga.CorrelationId }))
    .TransitionTo(Executing)
);

Expected Behavior
Code compiles.
Actual Behavior
error CS0029: Cannot implicitly convert type 'System.Threading.Tasks.Task<MassTransit.SendTuple<SagaWhenEnterDispatchIssue.ExecuteRule>>' to 'System.Uri'
error CS1662: Cannot convert lambda expression to intended delegate type because some of the return types in the block are not implicitly convertible to the delegate return type

Reproducible Demo
https://github.com/danports/SagaWhenEnterDispatchIssue

Easy enough fix, just missed a few extension methods.

Awesome, thanks!


Show a popup next to a button when it is clicked

Implements show a popup next to a button when it is clicked.

Looks good but I think you want to remove debugger

Yes :). It's removed


[Feature Request] Ability to run tests across multiple regions

Is your feature request related to a problem? Please describe.
I am testing web application firewall behaviour and would like to be able to send load test traffic from multiple regions/geos.
Describe the solution you'd like
The ability to schedule a test to run across multiple regions, each region with its own compute, but with the same test plan.
Describe alternatives you've considered
I could automate this using a GitHub action, but it would be tricky to coordinate. I could use az CLI to do this, if az CLI supported tests. See also #122 .
AB#1686946

@DanielLarsenNZ Thank you for the feedback. This feature to generate load from multiple regions (geographically distributed load) is on our roadmap.
Meanwhile, you can try running simultaneous tests from multiple regions. Az CLI support is on our roadmap. However, you can try automation using APIs and SDKs
API: https://learn.microsoft.com/en-us/rest/api/loadtesting/
JavaScript: https://learn.microsoft.com/en-us/javascript/api/overview/azure/load-testing?view=azure-node-preview
Dotnet: https://learn.microsoft.com/en-us/dotnet/api/overview/azure/loadtestservice?view=azure-dotnet-preview
Python: https://learn.microsoft.com/en-us/python/api/overview/azure/load-testing?view=azure-python-preview
Java: https://learn.microsoft.com/en-us/java/api/overview/azure/developer-loadtesting-readme?view=azure-java-preview

Thanks so much!


Translate from English

I translate change language from English but when I click another link it will change to English back and can not change language anymore except back to home page.

Hi, please provide the steps to reproduce.

In the home page, I change from English to other language but when I click on a category the language will change to English again.

It sounds like the old storefront from legacy-views? We don't have this functionality in Saleor Storefront yet.