feat!: fetch base asset ID from chain

Closes #1948
Breaking Changes:

BaseAssetId is no longer exported by fuels. It must be fetched from the chain as it is configurable by the chain.

// Before
import { BaseAssetId } from `fuels`;

// After
const provider = await Provider.create(FUEL_NETWORK_URL);
const baseAssetId = provider.getBaseAssetId();

Instantiating a transaction request now requires passing the base asset ID. It no longer assumes a default value so should be fetched from the chain, then passed to the transaction request.

// Before
const scriptTxRequest = new ScriptTransactionRequest();
const createTxRequest = new CreateTransactionRequest();

// After
const scriptTxRequest = new ScriptTransactionRequest({ baseAssetId });
const createTxRequest = new CreateTransactionRequest({ baseAssetId });

CoinQuantityLike now requires an AssetId. Previously most of it's usages would default to the BaseAssetId, as this must now be fetched, then it must be passed to the type.

✨ A PR has been created under the rc-1976 tag on fuels-wallet repo.
https://github.com/FuelLabs/fuels-wallet/pull/1212

Sorry @danielbate I'm very late to the party here but is there a way for us to fetch the base asset ID just like the chainInfo and nodeInfo when we initialize the Provider, and then cache it? This way we can avoid the breaking change around having to fetch and pass it in via all the transactions. This breaking change really degrades the DX imo.

@Dhaiwat10 I guess that's the same thing I said here.

Not sure if I am missing something, but I believe we do not need the baseAssetId specified in the TransactionRequest

Agreed, but from a DX POV, we are now handing off the responsibility of ensuring the base asset is passed around correctly completely to the consumer. Previously we were using optional checks and default values for asset IDs that would then evaluate to the base asset.
I am actually more for this change, however when I initially went for making the property optional, and to hand off responsibility, it was deemed too risky.

If I understood correctly, this change is happening because we need the BaseAssetId when setting the assetId for a MessageInput. However, a MessageCoin already carries the assetId property and can be used when adding an OutputCoin or OutputChange to the Transaction

When we create the corresponding change output, we are creating it specifically for the base asset. Is there any risk in returning funds for the same asset as the message? This area is not very well tested.
But agreed, if we wanted to completely decouple that class from the concept of a base asset, addMessageInput right now seems to be the only area that makes a specific reference to it.

cc @Torres-ssf

When we create the corresponding change output, we are creating it specifically for the base asset. Is there any risk in returning funds for the same asset as the message? This area is not very well tested.

This code was added because all MessageCoin has as the assetId the Base asset.
MessageCoins are bridged ETH, brought to Fuel from the main layer.
For instance, take a look at the messages that we define at chainConfig:
    "messages": [
      {
        "sender": "0xc43454aa38dd91f88109a4b7aef5efb96ce34e3f24992fe0f81d233ca686f80f",
        "recipient": "0x69a2b736b60159b43bb8a4f98c0589f6da5fa3a3d101e8e269c499eb942753ba",
        "nonce": "0101010101010101010101010101010101010101010101010101010101010101",
        "amount": "0x000000000000FFFF",
        "data": "",
        "da_height": "0x00"
      },
      ...

As you can see there is no assetId. If you start the node using this chain config and try the following query on the playground:
query {
  coinsToSpend(owner: "0x69a2b736b60159b43bb8a4f98c0589f6da5fa3a3d101e8e269c499eb942753ba", queryPerAsset: [{ assetId: "0x0000000000000000000000000000000000000000000000000000000000000000", amount: "1"}]) {
  	... on MessageCoin {
      amount
      assetId
      nonce
    }
  }
}

The query will return the specific message, as a MessageCoin, carrying the assetId property.

Thanks @Torres-ssf, that makes sense.
If we're happy then, we'll hand off ownership of the base asset ID completely to the consumer. I'll make the TransactionRequest have no knowledge of the base asset, and instead it just passes asset IDs.


Remove xerrors dependency and use stdlib.

I apologize, but we can't accept this change. See https://github.com/google/go-cmp/pull/225#issuecomment-650431592

I understand that it needs to support Go 1.8. As go mod has version control, could we raise the version number to drop the dependency? If users want to use it under Go 1.14, they could pin the version 0.5.x.

See https://github.com/google/go-cmp/pull/225#issuecomment-650434186

Thanks for clarifying.


auto-formatting seems to be broken with svg style attribute

VSCode Version:1.8.1
OS Version: OSX Sierra 10.12.2

Steps to Reproduce:
Create a plain html file with this contents, I've stripped this down from a full HTML page to this

This HTML seems to produce incorrect indentation on the <g> tag, caused by the style attribute

<svg>
    <g>
        <rect style="width:200px;"></rect>
</g>
</svg>

this HTML seems to indent fine (I've removed the style on the rect tag)

<svg>
    <g>
        <rect></rect>
    </g>
</svg>

duplicate of #17849


Hiero: Effect Categories - OP-7397

Changelog Description
This PR introduces Effect Categories for the Hiero settings. This allows studios to split effect stacks into meaningful subsets.
Testing notes:

Setup Effect Categories in Hiero addon settings (see documentation).
Setup a clip in Hiero with effects corresponding to the settings.
Publish clip and effects.
Load into Nuke and verify the effects are grouped correctly.

Docs https://github.com/ynput/ayon-documentation/pull/123

Can we also add default values as you were showing in documentation into the variable DEFAULT_PUBLISH_PLUGIN_SETTINGS?

I would maybe argue that we should not have any defaults to maintain backwards compatibility?

It is not collecting soft effect nodes and then those are not serialized into json files.

@jakubjezek001 I'm not entirely sure what you meant here, but I've fixed the case where effects are not categorized, in which case we just dump the effects into the "main" effects.


Stripping whitespaces in launch file name

Basic Info

Info
Please fill out this column

Issue(s) this addresses
#151

ROS2 version tested on
Humble

Aerial platform tested on
-

Description of contribution in a few bullet points

Launch file name contained white-spaces so it wasn't being installed correctly

Codecov Report

Merging #152 (15456a0) into main (43a3f8d) will decrease coverage by 0.09%.
The diff coverage is n/a.

:exclamation: Current head 15456a0 differs from pull request most recent head b23c143. Consider uploading reports for the commit b23c143 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff            @@
##            main    #152      +/-   ##
========================================
- Coverage   4.62%   4.54%   -0.09%
========================================
  Files         98      95       -3
  Lines       4995    4977      -18
  Branches     513     528      +15
========================================
- Hits         231     226       -5
+ Misses      4564    4535      -29
- Partials     200     216      +16

Flag
Coverage Δ

unittests
4.54% <ø> (-0.09%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

as2_core/src/platform_state_machine.cpp
42.42% <0.00%> (-13.14%)
:arrow_down:

as2_controller/tests/find_mode_match_test.cpp
45.65% <0.00%> (-6.53%)
:arrow_down:

...state_estimator/plugins/mocap/tests/mocap_test.cpp
42.22% <0.00%> (-1.26%)
:arrow_down:

as2_core/src/utils/tf_utils.cpp
19.38% <0.00%> (ø)

as2_core/src/aerial_platform.cpp
0.00% <0.00%> (ø)

as2_core/src/utils/yaml_utils.cpp
0.00% <0.00%> (ø)

as2_controller/src/controller_manager.cpp
0.00% <0.00%> (ø)

as2_core/include/as2_core/aerial_platform.hpp
0.00% <0.00%> (ø)

as2_core/include/as2_core/utils/yaml_utils.hpp
0.00% <0.00%> (ø)

as2_motion_reference_handlers/src/speed_motion.cpp
0.00% <0.00%> (ø)

... and 40 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


User configuration file

Is your feature request related to a problem? Please describe.
For some features (#21, #8 or #22) it will be good to have a user-space configuration file.
Describe the solution you'd like

JSON (?) based configuration file for user-based settings, as

#21 Support CSP connections
#8 hosting environments (may be different / relatively static)
#22 Proxy support
Azure certificate / ServicePrincipals
...

Additional context

Where to store user based if module is system-wide?
Removing when uninstalling module?
Init if not existing by template?

If adding a dependency is an option then I believe PSFrameWork has a good configuration system or one of the other config projects out there

Plan

Format: PSD1 as it is more readable as JSON / XML and natively supported (YAML isn't)
https://medium.com/@ssg/powershell-accidentally-created-a-nice-configuration-format-3efde5448090
http://ramblingcookiemonster.github.io/PowerShell-Configuration-Data/
Scope: supporting a user scope at $Env:LocalAppData
using a fixed module for config handling,

PoshCode looks promising
PSFramework seems to be huge, but many functions (https://psframework.org/documentation/documents/psframework/configuration.html)
ScriptConfig as an alternative, but then JSON, not PSD1

realized

$env:LOCALPPDATA\Connect-MS365\Connect-MS365.Config.psd1
initialized if not available
switch to reinitializeconfig
read config

PSD1 file over XML / json as commentable and better readable
not using any framework as too many drawbacks


UNIQUE constraint failed: storage_pools.name

Distributor ID:	Ubuntu
Description:	Ubuntu 16.04.2 LTS
Release:	16.04
Codename:	xenial

lxd 2.9.1-0ubuntu3~ubuntu16.04.1~ppa1

After upgrade to LXD 2.9, lxd deamon has error on start:
% sudo /usr/bin/lxd --group lxd --debug
INFO[02-20|09:39:49] LXD 2.9.1 is starting in normal mode     path=/var/lib/lxd
WARN[02-20|09:39:49] CGroup memory swap accounting is disabled, swap limits will be ignored.
INFO[02-20|09:39:49] Default uid/gid map:
INFO[02-20|09:39:49]  - u 0<PHONE_NUMBER>6
INFO[02-20|09:39:49]  - g 0<PHONE_NUMBER>6
DBUG[02-20|09:39:49] Initializing and checking storage pool "vg0".
DBUG[02-20|09:39:49] StoragePoolInit                          driver=storage/lvm
DBUG[02-20|09:39:49] StoragePoolCheck                         driver=storage/lvm
DBUG[02-20|09:39:49] Applying patch: storage_api
error: UNIQUE constraint failed: storage_pools.name

https://github.com/lxc/lxd/issues/2906

That's tracked as #2907.


Examples: exchange entire RTCSessionDescription

Resolves #39
TODO:

[ ] Test gstreamer-receive
[ ] Test SFU

Can someone test these last two examples? I'm having trouble running them on Windows.

Pull Request Test Coverage Report for Build 1228

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 31.062%

Totals

Change from base Build 1227:
0.0%

Covered Lines:
2179

Relevant Lines:
7015

💛  - Coveralls

@backkem can you also update the jsfiddle links?
Or maybe just wait until https://github.com/jsfiddle/jsfiddle-issues/issues/1397 is fixed, then we can use the embed feature again!

@backkem can you also update the jsfiddle links?

Yea, I'll update them over the weekend.


Import-DbaCsv - Add support for Azure SQL Database

Removed the code that called 'use ' since Azure SQL doesn't allow that to be called.
Updated to use Connect-DbaInstance with the -Database param to set the db connection context. If the database does not exist an error will be returned "cannot open database".

Type of Change

[x] Bug fix (non-breaking change, fixes #6855 )
[ ] New feature (non-breaking change, adds functionality, fixes # )
[ ] Breaking change (effects multiple commands or functionality, fixes # )
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[x] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose

Minor fix to support Azure SQL.
Approach

Commands to test

$serverAzureTest = Connect-DbaInstance -ConnectionString "Data Source=TCP:yourserver.database.windows.net,1433;MultipleActiveResultSets=False;Connect Timeout=30;Encrypt=True;TrustServerCertificate=False;User Id=youruserid;Password=strongpassword;Database=TestDB"
$result = Import-DbaCsv -Path "your sample tab delimited file" -SqlInstance $serverAzureTest -Database TestDB -Delimiter `t -Table TestTable -Schema Test -Truncate -AutoCreateTable -Verbose

The internal function is used in all of our commands. Making a change to that will have to be in a PR by itself so we can verify what breaks and what does not.

Ok, that makes sense. The change I made to Connect-SqlInstance seems to have affected other commands based on the appveyor results, so I've reverted the change.
I switched the Import-DbaCsv back to using Connect-DbaInstance since it handles the boundary case I ran into. The boundary case is:

Connect to a server with one database specified (See the commands under 'Commands to test' above)
Re-use that same server object and call Import-DbaCsv with an invalid db name.

When this scenario occurs Import-DbaCsv doesn't catch the invalid db name since Connect-SqlInstance just returns the existing server object.
If there is a strong preference to not use Connect-DbaInstance I can add code to check if the db exists.

That edge case is irrelevant and will break on many other commands.

Connect-SqlInstance just returns the existing server object.

They is why the import function code was manipulating the connection.
I can fully review and test this later, I'm just looking at this from my phone right now so can't get the full picture.

Just adding some changes based on the discussion and checking the appveyor results. No hurry on the review since we've got the holiday weekend.

Just adding some changes based on the discussion and checking the appveyor results. No hurry on the review since we've got the holiday weekend.

@wsmelton, let me know if any additional changes are needed.

@wsmelton, let me know if any additional changes are needed.

Test results between dev branch and PR:
[PS 5.1] [16:28:44] [  9.6 s ][ git:development ] C:\git\dbatools
 [12] > Import-DbaCsv -FullPath C:\temp\testexport.csv -SqlInstance sql -SqlCredential $containerCred -Database tempdb -Table dbatoolslci_testImportdev -Schema dbo -Truncate -AutoCreateTable -NoProgress

ComputerName  : SQL
InstanceName  : MSSQLSERVER
SqlInstance   : SQL
Database      : tempdb
Table         : dbatoolslci_testImportdev
Schema        : dbo
RowsCopied    : 6347
Elapsed       : 1.98 s
RowsPerSecond : 3211
Path          : C:\temp\testexport.csv

[PS 5.1] [16:29:14] [  2.0 s ][ git:development ] C:\git\dbatools
 [13] > remove-module dbatools
[PS 5.1] [16:29:22] [ 45.0ms ][ git:development ] C:\git\dbatools
 [14] > git branch
  development
  ga-repo-folder-restructure
  master
  pr/lancasteradam/7054
* pr/lancasteradam/7064
[PS 5.1] [16:29:45] [842.0ms ][ git:pr/lancasteradam/7064 ] C:\git\dbatools
 [15] > ipmo .\dbatools.psd1 -force
[PS 5.1] [16:30:03] [ 10.4 s ][ git:pr/lancasteradam/7064 ] C:\git\dbatools
 [16] > Import-DbaCsv -FullPath C:\temp\testexport.csv -SqlInstance sql -SqlCredential $containerCred -Database tempdb -Table dbatoolslci_testImportpr -Schema dbo -Truncate -AutoCreateTable -NoProgress

ComputerName  : SQL
InstanceName  : MSSQLSERVER
SqlInstance   : SQL
Database      : tempdb
Table         : dbatoolslci_testImportpr
Schema        : dbo
RowsCopied    : 6347
Elapsed       : 937.56 ms
RowsPerSecond : 6774
Path          : C:\temp\testexport.csv

CSV file utilized:
testexport.txt

Test results between dev branch and PR:
[PS 5.1] [16:28:44] [  9.6 s ][ git:development ] C:\git\dbatools
 [12] > Import-DbaCsv -FullPath C:\temp\testexport.csv -SqlInstance sql -SqlCredential $containerCred -Database tempdb -Table dbatoolslci_testImportdev -Schema dbo -Truncate -AutoCreateTable -NoProgress

ComputerName  : SQL
InstanceName  : MSSQLSERVER
SqlInstance   : SQL
Database      : tempdb
Table         : dbatoolslci_testImportdev
Schema        : dbo
RowsCopied    : 6347
Elapsed       : 1.98 s
RowsPerSecond : 3211
Path          : C:\temp\testexport.csv

[PS 5.1] [16:29:14] [  2.0 s ][ git:development ] C:\git\dbatools
 [13] > remove-module dbatools
[PS 5.1] [16:29:22] [ 45.0ms ][ git:development ] C:\git\dbatools
 [14] > git branch
  development
  ga-repo-folder-restructure
  master
  pr/lancasteradam/7054
* pr/lancasteradam/7064
[PS 5.1] [16:29:45] [842.0ms ][ git:pr/lancasteradam/7064 ] C:\git\dbatools
 [15] > ipmo .\dbatools.psd1 -force
[PS 5.1] [16:30:03] [ 10.4 s ][ git:pr/lancasteradam/7064 ] C:\git\dbatools
 [16] > Import-DbaCsv -FullPath C:\temp\testexport.csv -SqlInstance sql -SqlCredential $containerCred -Database tempdb -Table dbatoolslci_testImportpr -Schema dbo -Truncate -AutoCreateTable -NoProgress

ComputerName  : SQL
InstanceName  : MSSQLSERVER
SqlInstance   : SQL
Database      : tempdb
Table         : dbatoolslci_testImportpr
Schema        : dbo
RowsCopied    : 6347
Elapsed       : 937.56 ms
RowsPerSecond : 6774
Path          : C:\temp\testexport.csv

CSV file utilized:
testexport.txt

Test results on the file against Azure SQL:
[PS 5.1] [16:18:16] [ 18.1 m ][ git:pr/lancasteradam/7064 ] C:\git\dbatools
 [7] > Import-DbaCsv -FullPath C:\temp\testexport.csv -SqlInstance thy.database.windows.net -SqlCredential (Get-Credential) -Database ss -Table dbatoolslci_testImportpr -Schema dbo -Truncate -AutoCreateTable -NoProgress

cmdlet Get-Credential at command pipeline position 1
Supply values for the following parameters:
Credential

ComputerName  : thy.database.windows.net
SqlInstance   : thy
Database      : ss
Table         : dbatoolslci_testImportpr
Schema        : dbo
RowsCopied    : 6347
Elapsed       : 7.92 s
Path          : C:\temp\testexport.csv

Test results on the file against Azure SQL:
[PS 5.1] [16:18:16] [ 18.1 m ][ git:pr/lancasteradam/7064 ] C:\git\dbatools
 [7] > Import-DbaCsv -FullPath C:\temp\testexport.csv -SqlInstance thy.database.windows.net -SqlCredential (Get-Credential) -Database ss -Table dbatoolslci_testImportpr -Schema dbo -Truncate -AutoCreateTable -NoProgress

cmdlet Get-Credential at command pipeline position 1
Supply values for the following parameters:
Credential

ComputerName  : thy.database.windows.net
SqlInstance   : thy
Database      : ss
Table         : dbatoolslci_testImportpr
Schema        : dbo
RowsCopied    : 6347
Elapsed       : 7.92 s
Path          : C:\temp\testexport.csv


Cannot work with compressed pcd

Hi there,
Thank you for the great work! I met the error below, do you have any idea of this error? Thank you!
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/bo/miniconda3/envs/pytorch/lib/python3.8/site-packages/pypcd4/pypcd4.py", line 261, in from_path
    return PointCloud.from_fileobj(fp)
  File "/home/bo/miniconda3/envs/pytorch/lib/python3.8/site-packages/pypcd4/pypcd4.py", line 233, in from_fileobj
    pc_data = _parse_pc_data(fp, metadata)
  File "/home/bo/miniconda3/envs/pytorch/lib/python3.8/site-packages/pypcd4/pypcd4.py", line 187, in _parse_pc_data
    buffer = lzf.decompress(fp.read(compressed_size), uncompressed_size)
AttributeError: module 'lzf' has no attribute 'decompress'

Seems like a python version issue, not sure. I used python 3.8 in conda

I think I can confirm that it is a python version issue. I created one clean python 3.8 env and one clean python 3.11 in conda.
Python 3.8
(tmp2) ✔ ~/projects/LocNDF [main|…3]
18:45 $ python
Python 3.8.19 (default, Mar 20 2024, 19:58:24)
[GCC 11.2.0] :: Anaconda, Inc. on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from pypcd4 import PointCloud
>>> pcd = PointCloud.from_path("ColumbiaPark/2018-10-11/pcds/1000.pcd")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/bo/miniconda3/envs/tmp2/lib/python3.8/site-packages/pypcd4/pypcd4.py", line 261, in from_path
    return PointCloud.from_fileobj(fp)
  File "/home/bo/miniconda3/envs/tmp2/lib/python3.8/site-packages/pypcd4/pypcd4.py", line 233, in from_fileobj
    pc_data = _parse_pc_data(fp, metadata)
  File "/home/bo/miniconda3/envs/tmp2/lib/python3.8/site-packages/pypcd4/pypcd4.py", line 187, in _parse_pc_data
    buffer = lzf.decompress(fp.read(compressed_size), uncompressed_size)
AttributeError: module 'lzf' has no attribute 'decompress'
>>>

Python 3.11
18:42 $ python
Python 3.11.9 (main, Apr 19 2024, 16:48:06) [GCC 11.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from pypcd4 import PointCloud
>>> pcd = PointCloud.from_path("ColumbiaPark/2018-10-11/pcds/1000.pcd")
>>> pcd.numpy()
array([[-1.99902172e+01, -3.02818966e+00, -2.31144094e+00,
         1.43000000e+02,  1.22324355e+09],
       [-2.14173756e+01, -3.97245312e+00, -2.38133454e+00,
         1.51000000e+02,  1.22324355e+09],
       [-4.40083427e+01, -1.29852591e+01,  2.70792931e-01,
         2.16000000e+02,  1.22324355e+09],
       ...,
       [-1.48014488e+01, -4.26814842e+00, -2.21298957e+00,
         9.00000000e+01,  1.22324355e+09],
       [-1.00869751e+01, -2.12456894e+00, -2.10196733e+00,
         8.10000000e+01,  1.22324355e+09],
       [-1.05974884e+01, -2.81252408e+00, -2.12533903e+00,
         9.70000000e+01,  1.22324355e+09]])
>>>

Maybe the root issue is from python-lzf?

It is more of an issue of conda. python-lzf does not work well for conda python 3.8. However, it works well for native Ubuntu python 3.8.

An update: I finally realized that lzf is installed instead of python-lzf.


[client] Client cannot be closed when reconnecting bugfix

To reproduce the bug:

start client with

not existing server address
exponential back off strategy (infinite or just long enough)

stop client
client state is ClientConnecting, and client still tries to reconnect even though it was closed

@mmatczuk Thanks!


Feature Request: DT plug-ins and extensions

It will be really useful if the next versions include the filtering and file export plugins and extensions:
http://www.datatables.net/extensions/tabletools/
http://www.datatables.net/plug-ins/filtering/

Most plugins should work without changes needed in this package. Filtering won't because this package uses serverSide option and the filtering plugin docs state that the plugin has no effect when doing server side processing.
If there are certain filtering features you want, you could suggest those specifically and we may or may not be able to add support for it.

what would be the proper folder to place the extensions ? sorry for the rookie question but I am still somewhat lost with this

Probably /client/lib

it uses some sort of flash magic for the the csv/excel export stuff I cant get to work I think I'll just give up. Thanks bunch for trying to help anyway.


Update tar.lua

fix a bug occured when unrar some .tar files using this module

What was the bug? I look at the code and I can't see how this change changes the behaviour (although I might still be asleep).

It does change behavior (the original code breaks the for loop, the new one doesn't).
@usstwxy, do you have an example tar that reproduces the bug? if it's not too big we might include it in the test cases.

Here is a .tar file which could produce the bug in my machine.
The reason is that sometimes the last character of octal in function
octal_to_number is not a number.
    octal        #octal

001751 7
001751 7
00001156234 12
12561100475 12
015063 6
000000 7
000000 7

2015-09-22 0:27 GMT+08:00 Hisham Muhammad<EMAIL_ADDRESS>
It does change behavior (the original code breaks the for loop, the new
one doesn't).
@usstwxy https://github.com/usstwxy, do you have an example tar that
reproduces the bug? if it's not too big we might include it in the test
cases.
—
Reply to this email directly or view it on GitHub
https://github.com/keplerproject/luarocks/pull/426#issuecomment-142034823
.

the attachment didn't come through.

It seems that github doesn't allow tar attachment. You can get it by this link https://drive.google.com/file/d/0B0jziWAfhoBOSDYyV08yMGlQRFNBTTdYajBTand2ZDlHYTdJ/view?usp=sharing

off topic, but while reviewing this I noticed that a header reading failure is reported (util.printerr(err)) but not returned (it just ends up throwing on the next line)

For anyone watching at home, this is the current/old behaviour:
$ lua -e 'print(require "luarocks.tools.tar".untar("k.tar", "."))'
chmod: invalid mode: ‘000664 ’
Try 'chmod --help' for more information.
Invalid header magic ±ÁRÑð
lua: /usr/share/lua/5.3/luarocks/tools/tar.lua:105: attempt to index a boolean value (local 'header')
stack traceback:
	/usr/share/lua/5.3/luarocks/tools/tar.lua:105: in function 'luarocks.tools.tar.untar'
	(command line):1: in main chunk
	[C]: in ?

While this is the new behaviour (with this PR):
$ lua -e 'print(require "tar".untar("k.tar", "."))'
chmod: invalid mode: ‘000664 ’
Try 'chmod --help' for more information.
chmod: invalid mode: ‘000664 ’
Try 'chmod --help' for more information.
chmod: invalid mode: ‘000664 ’
Try 'chmod --help' for more information.
true

You can get it by this link https://drive.google.com/file/d/0B0jziWAfhoBOSDYyV08yMGlQRFNBTTdYajBTand2ZDlHYTdJ/view?usp=sharing

This fails to download with Firefox (it opens, but the download button in that page redirects to a page that fails)...

Try this link: https://www.dropbox.com/s/o5mocki9urw7bm1/k.tar?dl=0

@ignacio danke schön! :D

Is this module (luarocks.tools.tar) actually used anywhere in LuaRocks?

@mpeterv: not really. It was part of a grand plan that I had back in pre-1.0 days, of replacing all UnxUtils tools in the Windows port with Lua modules (with as little dependencies on binary modules as possible — I was trying to go with "Kepler modules" alone, remember that LuaRocks was born as part of Kepler). UnxUtils was meant to be a stop-gap solution, but it "just worked" and here it is after all these years.


autocomplete method when a value is selected

Hi, is there a method that identifies the value that is selected? For example if I write "goo" and the dropdown shows me Google and I click it, is there a way to get that clicked value ?
$('input.autocomplete').autocomplete({
    data: {
      "Apple": null,
      "Microsoft": null,
      "Google": 'http://placehold.it/250x250'
    }
  });

Nevermind I check the materialize.js and found the autocomplete plugin. I use this to get what I mentioned before
 $(".autocomplete-content").on('click', 'li', function () {
    var selectedval = $(this).text();
    alert("Value "+selectedval );
  });


[Feature] Automatic partition support async mv

What type of PR is this：

[ ] BugFix
[x] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

run starrocks_fe_unittest

run starrocks_fe_unittest

run starrocks_fe_unittest

[FE PR Coverage Check]
:heart_eyes: pass : 5 / 6 (83.33%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/OlapTable.java
5
6
83.33%
[863]

run starrocks_admit_test

@Mergifyio backport branch-3.0


Altintity operator schema auto-creation not working when new replica(clickhouse pod) added on k8s cluster

Hi,
It has been mentioned on "https://github.com/Altinity/clickhouse-operator/blob/master/docs/schema_migration.md"
that when cluster is scaled up and new ClickHouse instances are ready, clickhouse-operator automatically creates schema.
If replica is added:

Other replicas at the same shard are analyzed for replicated tables
Databases for replicated tables are created
Replicated tables are created
Then the same logic as to adding shard applies

I am using latest version of altinity operator(altinity/clickhouse-operator:0.24.2), clickhouse server(clickhouse/clickhouse-server:23.7) and zookeeper ( docker.io/zookeeper:3.8.4).
I deployed clickhouse on k8s cluster with 1 shard and 1 replica. Then, I created the replicated merge tree table and inserted few rows:
CREATE TABLE default.events_local ON CLUSTER `{cluster}`
(
    `event_date` Date,
    `event_type` Int32,
    `article_id` Int32,
    `title` String
)
ENGINE = ReplicatedMergeTree('/clickhouse/chi-1/chi-1/tables/{shard}/default/events_local', '{replica}')
PARTITION BY toYYYYMM(event_date)
ORDER BY (event_type, article_id)

Post that, I scaled the setup to have 1 more replica. I expected that new replica will have the schema auto-created but I do not see schema automatically created on newly created replica pod.
pod/chi-chi-1-chi-1-0-0-0                                      1/1     Running   0          42m
pod/chi-chi-1-chi-1-0-1-0                                      1/1     Running   0          24m

Is there any specific configuration needed for automatic schema creation on new replica? Please help as I am stuck on this and tried many things mentioned on docs like schemaPolicy, internalReplication etc.
Here is my ClickHouseInstallation yaml file content:
apiVersion: "clickhouse.altinity.com/v1"
kind: ClickHouseInstallation
metadata:
  name: chi-1
  namespace: clickhouse
spec:
  configuration:
    zookeeper:
      nodes:
        - host: "zookeeper-0.zookeepers.clickhouse.svc.cluster.local"
          port: 2181
    clusters:
      - name: chi-1
        schemaPolicy:
          replica: All
        layout:
          shardsCount: 1
          replicasCount: 2
        templates:
          podTemplate: clickhouse-stable
          dataVolumeClaimTemplate: clickhouse-data-volume
    users:
      default/password: password
      default/profile: default
      default/quota: default
      default/networks/ip:
        - <IP_ADDRESS>/0
        - ::/0
  templates:
    volumeClaimTemplates:
      - name: clickhouse-data-volume
        spec:
          storageClassName: vmsp-default
          accessModes:
            - ReadWriteOnce
          resources:
            requests:
              storage: 500Mi
    podTemplates:
      - name: clickhouse-stable
        spec:
          containers:
            - name: clickhouse
              image: clickhouse/clickhouse-server:23.7
              resources:
                requests:
                  memory: "500Mi"
                  cpu: "500m"
                limits:
                  memory: "1024Mi"
                  cpu: "1000m"

zookeeper.yaml content:
# Setup Service to provide access to Zookeeper for clients
apiVersion: v1
kind: Service
metadata:
  namespace: clickhouse
  # DNS would be like zookeeper.zoons
  name: zookeeper
  labels:
    app: zookeeper
spec:
  ports:
    - port: 2181
      name: client
    - port: 7000
      name: prometheus
  selector:
    app: zookeeper
    what: node
---
# Setup Headless Service for StatefulSet
apiVersion: v1
kind: Service
metadata:
  namespace: clickhouse
  # DNS would be like zookeeper-0.zookeepers.etc
  name: zookeepers
  labels:
    app: zookeeper
spec:
  ports:
    - port: 2888
      name: server
    - port: 3888
      name: leader-election
  clusterIP: None
  selector:
    app: zookeeper
    what: node
---
# Setup max number of unavailable pods in StatefulSet
apiVersion: policy/v1
kind: PodDisruptionBudget
metadata:
  namespace: clickhouse
  name: zookeeper-pod-disruption-budget
spec:
  selector:
    matchLabels:
      app: zookeeper
  maxUnavailable: 1
---
# Setup Zookeeper StatefulSet
# Possible params:
# 1. replicas
# 2. memory
# 3. cpu
# 4. storage
# 5. storageClassName
# 6. user to run app
apiVersion: apps/v1
kind: StatefulSet
metadata:
  namespace: clickhouse
  # nodes would be named as zookeeper-0, zookeeper-1, zookeeper-2
  name: zookeeper
  labels:
    app: zookeeper
spec:
  selector:
    matchLabels:
      app: zookeeper
  serviceName: zookeepers
  replicas: 1
  updateStrategy:
    type: RollingUpdate
  podManagementPolicy: OrderedReady
  template:
    metadata:
      namespace: clickhouse
      labels:
        app: zookeeper
        what: node
      annotations:
        prometheus.io/port: '7000'
        prometheus.io/scrape: 'true'
    spec:
      containers:
        - name: kubernetes-zookeeper
          imagePullPolicy: IfNotPresent
          image: "localhost:5000/images/zookeeper:3.8.4"
          ports:
            - containerPort: 2181
              name: client
            - containerPort: 2888
              name: server
            - containerPort: 3888
              name: leader-election
            - containerPort: 7000
              name: prometheus
          env:
            - name: SERVERS
              value: "1"

          # See those links for proper startup settings:
          # https://github.com/kow3ns/kubernetes-zookeeper/blob/master/docker/scripts/start-zookeeper
          # https://clickhouse.yandex/docs/en/operations/tips/#zookeeper
          # https://github.com/ClickHouse/ClickHouse/issues/11781
          command:
            - bash
            - -x
            - -c
            - |
              HOST=`hostname -s` &&
              DOMAIN=`hostname -d` &&
              CLIENT_PORT=2181 &&
              SERVER_PORT=2888 &&
              ELECTION_PORT=3888 &&
              PROMETHEUS_PORT=7000 &&
              ZOO_DATA_DIR=/var/lib/zookeeper/data &&
              ZOO_DATA_LOG_DIR=/var/lib/zookeeper/datalog &&
              {
                echo "clientPort=${CLIENT_PORT}"
                echo 'tickTime=2000'
                echo 'initLimit=300'
                echo 'syncLimit=10'
                echo 'maxClientCnxns=2000'
                echo 'maxTimeToWaitForEpoch=2000'
                echo 'maxSessionTimeout=60000000'
                echo "dataDir=${ZOO_DATA_DIR}"
                echo "dataLogDir=${ZOO_DATA_LOG_DIR}"
                echo 'autopurge.snapRetainCount=10'
                echo 'autopurge.purgeInterval=1'
                echo 'preAllocSize=131072'
                echo 'snapCount=3000000'
                echo 'leaderServes=yes'
                echo 'standaloneEnabled=false'
                echo '4lw.commands.whitelist=*'
                echo 'metricsProvider.className=org.apache.zookeeper.metrics.prometheus.PrometheusMetricsProvider'
                echo "metricsProvider.httpPort=${PROMETHEUS_PORT}"
                echo "skipACL=true"
                echo "fastleader.maxNotificationInterval=10000"
              } > /conf/zoo.cfg &&
              {
                echo "zookeeper.root.logger=CONSOLE"
                echo "zookeeper.console.threshold=INFO"
                echo "log4j.rootLogger=\${zookeeper.root.logger}"
                echo "log4j.appender.CONSOLE=org.apache.log4j.ConsoleAppender"
                echo "log4j.appender.CONSOLE.Threshold=\${zookeeper.console.threshold}"
                echo "log4j.appender.CONSOLE.layout=org.apache.log4j.PatternLayout"
                echo "log4j.appender.CONSOLE.layout.ConversionPattern=%d{ISO8601} - %-5p [%t:%C{1}@%L] - %m%n"
              } > /conf/log4j.properties &&
              echo 'JVMFLAGS="-Xms128M -Xmx1G -XX:ActiveProcessorCount=8 -XX:+AlwaysPreTouch -Djute.maxbuffer=8388608 -XX:MaxGCPauseMillis=50"' > /conf/java.env &&
              if [[ $HOST =~ (.*)-([0-9]+)$ ]]; then
                  NAME=${BASH_REMATCH[1]} &&
                  ORD=${BASH_REMATCH[2]};
              else
                  echo "Failed to parse name and ordinal of Pod" &&
                  exit 1;
              fi &&
              mkdir -pv ${ZOO_DATA_DIR} &&
              mkdir -pv ${ZOO_DATA_LOG_DIR} &&
              whoami &&
              chown -Rv zookeeper "$ZOO_DATA_DIR" "$ZOO_DATA_LOG_DIR" &&
              export MY_ID=$((ORD+1)) &&
              echo $MY_ID > $ZOO_DATA_DIR/myid &&
              for (( i=1; i<=$SERVERS; i++ )); do
                  echo "server.$i=$NAME-$((i-1)).$DOMAIN:$SERVER_PORT:$ELECTION_PORT" >> /conf/zoo.cfg;
              done &&
              if [[ $SERVERS -eq 1 ]]; then
                  echo "group.1=1" >> /conf/zoo.cfg;
              else
                  echo "group.1=1:2:3" >> /conf/zoo.cfg;
              fi &&
              for (( i=1; i<=$SERVERS; i++ )); do
                  WEIGHT=1
                  if [[ $i == 1 ]]; then
                    WEIGHT=10
                  fi
                  echo "weight.$i=$WEIGHT" >> /conf/zoo.cfg;
              done &&
              zkServer.sh start-foreground
          readinessProbe:
            exec:
              command:
                - bash
                - -c
                - '
                  IFS=;
                  MNTR=$(exec 3<>/dev/tcp/<IP_ADDRESS>/2181 ; printf "mntr" >&3 ; tee <&3; exec 3<&- ;);
                  while [[ "$MNTR" == "This ZooKeeper instance is not currently serving requests" ]];
                  do
                    echo "wait mntr works";
                    sleep 1;
                    MNTR=$(exec 3<>/dev/tcp/<IP_ADDRESS>/2181 ; printf "mntr" >&3 ; tee <&3; exec 3<&- ;);
                  done;
                  STATE=$(echo -e $MNTR | grep zk_server_state | cut -d " " -f 2);
                  if [[ "$STATE" =~ "leader" ]]; then
                    echo "check leader state";
                    SYNCED_FOLLOWERS=$(echo -e $MNTR | grep zk_synced_followers | awk -F"[[:space:]]+" "{print \$2}" | cut -d "." -f 1);
                    if [[ "$SYNCED_FOLLOWERS" != "0" ]]; then
                      ./bin/zkCli.sh ls /;
                      exit $?;
                    else
                      exit 0;
                    fi;
                  elif [[ "$STATE" =~ "follower" ]]; then
                    echo "check follower state";
                    PEER_STATE=$(echo -e $MNTR | grep zk_peer_state);
                    if [[ "$PEER_STATE" =~ "following - broadcast" ]]; then
                      ./bin/zkCli.sh ls /;
                      exit $?;
                    else
                      exit 1;
                    fi;
                  else
                    exit 1;
                  fi
                   '
            initialDelaySeconds: 10
            periodSeconds: 60
            timeoutSeconds: 60
          livenessProbe:
            exec:
              command:
                - bash
                - -xc
                - 'date && OK=$(exec 3<>/dev/tcp/<IP_ADDRESS>/2181 ; printf "ruok" >&3 ; IFS=; tee <&3; exec 3<&- ;); if [[ "$OK" == "imok" ]]; then exit 0; else exit 1; fi'
            initialDelaySeconds: 10
            periodSeconds: 30
            timeoutSeconds: 5
          volumeMounts:
            - name: datadir-vol
              mountPath: /var/lib/zookeeper
      # Run as a non-privileged user
      securityContext:
        runAsUser: 1000
        fsGroup: 1000
  volumeClaimTemplates:
    - metadata:
        namespace: clickhouse
        name: datadir-vol
      spec:
        storageClassName: standard
        accessModes:
          - ReadWriteOnce
        resources:
          requests:
            storage: 500Mi

Here is the content of zookeeper path which did not change after adding 1 more replica( I expect it to contain the entry of chi-chi-1-chi-1-0-1 as well :
[zk: localhost:2181(CONNECTED) 32] ls /clickhouse/chi-1/chi-1/tables/0/default/events_local/replicas
[chi-chi-1-chi-1-0-0]

Please suggest what configuration I am missing. I am looking for auto-creation of schema and data replication on newly created clcikhouse pod.

@ggangwar , that should work by default. Schema policy is not needed in CHI. Do you have clickhouse-operator log?
Side note -- clickhouse/clickhouse-server:23.7 is not recommended. Use LTS versions or Altinity Stable builds.

Hi @alex-zaitsev I removed scehamPolicy setting and using LTS of clickhouse-server but still the same behaviour. The newly created replica is not having the table automatically created.
Following are the steps in my setup:
1. Deployed 1-node zookeeper
 $ k get po | grep zookeeper
zookeeper-0                                                1/1     Running   0          71m

2. Deployed the clickhouse using altinity operator having 1 shard and 2 replica
 $ k get po  | grep clickhouse
ch-altinity-clickhouse-operator-7cd579b65f-qgtwv   1/1     Running   0          72m

 $ k get po | grep chi
chi-chi-1-chi-1-0-0-0                                      1/1     Running   0          72m
chi-chi-1-chi-1-0-1-0                                      1/1     Running   0          71m

3. created the ReplicatedMergeTree table on one of the replica(chi-chi-1-chi-1-0-0-0) and inserted few entries:
CREATE TABLE events_local ON CLUSTER `{cluster}`
(
    `event_date` Date,
    `event_type` Int32,
    `article_id` Int32,
    `title` String
)
ENGINE = ReplicatedMergeTree('/clickhouse/default/tables/{shard}/events_local', '{replica}')
PARTITION BY toYYYYMM(event_date)
ORDER BY (event_type, article_id)

INSERT INTO events_local (event_date, event_type, article_id, title)
VALUES ('2024-12-01', 1, 101, 'First Event'),
       ('2024-12-02', 2, 102, 'Second Event');

4. The table automatically created and data persisted in other replica(chi-chi-1-chi-1-0-1-0) as well:
SELECT *
FROM events_local

Query id: 3686134e-3290-4414-8cd7-ccd078743753

   ┌─event_date─┬─event_type─┬─article_id─┬─title────────┐
1. │ 2024-12-01 │          1 │        101 │ First Event  │
2. │ 2024-12-02 │          2 │        102 │ Second Event │
   └────────────┴────────────┴────────────┴──────────────┘

5. Edited CHI and increased replicated count to 3. The new replica created:
 $ k get po | grep chi
chi-chi-1-chi-1-0-0-0                                      1/1     Running   0          72m
chi-chi-1-chi-1-0-1-0                                      1/1     Running   0          71m
chi-chi-1-chi-1-0-2-0                                      1/1     Running   0          34m

6. But, new replica does not have the events_local table in it:
chi-chi-1-chi-1-0-2-0.chi-chi-1-chi-1-0-2.default.svc.cluster.local :) \d

SHOW TABLES

Query id: cabddc7e-27ac-4eb9-8443-0bfcfe3586b0

Ok.

0 rows in set. Elapsed: 0.003 sec.

7. Also, it does not have entry in zookeeper path:
zk: localhost:2181(CONNECTED) 37] ls /clickhouse/default/tables/0/events_local/replicas
[chi-chi-1-chi-1-0-0, chi-chi-1-chi-1-0-1]

I waited for 10 mins to see if table automatically created or not. But, But, events_local did not create automatically and also zookeeper path entry did not change to contain new replica

Please help me with the issue.

This is quite urgent, can anyone please help on this issue?

Hi, I have figured out the issue. The problem was the missing configuration for clickhouse_operator user in CHI yaml file.
The auto-replication worked post updating users section in CHI yaml file like this:
    users:
      default:
        password: "password"
        profile: "default"
        quota: "default"
        networksIP:
          - <IP_ADDRESS>/0
          - ::/0
      clickhouse_operator:
        password: "password"
        profile: "default"
        quota: "default"
        grants:
          - CREATE
          - ALTER
          - DROP
          - INSERT
          - SELECT
          - SHOW
          - SYSTEM
          - RELOAD
          - REPLICATE
        networksIP:
          - <IP_ADDRESS>/0
          - ::/0


Trying to get ifkey in "on_ppp_changed" through NETIF_PPP_STATUS event handler causes crash (IDFGH-4147)

Environment

Development Kit:      [none]
Kit version (for WroverKit/PicoKit/DevKitC): [v1|v2|v3|v4]
Module or chip used:  [ESP32]
IDF version (run git describe --tags to find it):
v4.3-dev-472-gcf056a7d0
Build System:         [idf.py]
Compiler version (run xtensa-esp32-elf-gcc --version to find it):
xtensa-esp32-elf-gcc (crosstool-NG esp-2020r2) 8.2.0
Operating System:     [Linux]
(Windows only) environment type: [MSYS2 mingw32|ESP Command Prompt|Plain Command Prompt|PowerShell].
Using an IDE?: [No]
Power Supply:         [Battery]

Problem Description
I am trying to get the ifkey in the on_ppp_changed which seems to receive it from the even as eventdata.
I have multiple PPP links and on_ppp_changed gets called if any one of these change. I would like to distinguish which PPP either using the ifkey or if someone can suggest another method. Currenltly it seems like the event_data is the neftif object but I cannot seem to get the ifkey from it.
Expected Behavior
I expect it to get the netif ifkey
Actual Behavior
crashes with guru med
Steps to reproduce

step1
Build pppos app with bg96 or sim800 modem

change on_ppp_changed to
static void on_ppp_changed(void *arg, esp_event_base_t event_base,
                           int32_t event_id, void *event_data)
{
    esp_netif_t *netif = event_data;

    const char *ifkey = esp_netif_get_ifkey(netif);
    if(strcmp(ifkey, "PPP_DEF") == 0)
     {
          printf ("%s",ifkey)
     }
}

Debug Logs
Guru Meditation Error: Core  0 panic'ed (LoadStoreError). Exception was unhandled.

Core  0 register dump:
PC      : 0x40001277  PS      : 0x00060130  A0      : 0x800db6c5  A1      : 0x3ffc1000
A2      : 0x4008603c  A3      : 0x3f4051e4  A4      : 0x00060120  A5      : 0x3ffc1070
0x4008603c: prvGetCurMaxSizeByteBuf at /opt/esp/idf/components/esp_ringbuf/ringbuf.c:716

A6      : 0x00000103  A7      : 0x3ffbb434  A8      : 0x80086bd4  A9      : 0x3ffc0fd0
A10     : 0x00000003  A11     : 0x00060123  A12     : 0x00060120  A13     : 0x00060123
A14     : 0x00000001  A15     : 0x00060023  SAR     : 0x00000004  EXCCAUSE: 0x00000003
EXCVADDR: 0x4008603c  LBEG    : 0x4000c2e0  LEND    : 0x4000c2f6  LCOUNT  : 0x00000000
0x4008603c: prvGetCurMaxSizeByteBuf at /opt/esp/idf/components/esp_ringbuf/ringbuf.c:716

Backtrace:0x40001274:0x3ffc1000 0x400db6c2:0x3ffc1010 0x40166d7d:0x3ffc1030 0x40166822:0x3ffc1060 0x401668fd:0x3ffc10a0 0x400868c1:0x3ffc10c0
0x400db6c2: on_ppp_changed at /project/build/../main/app_pppos.c:145

0x40166d7d: handler_execute at /opt/esp/idf/components/esp_event/esp_event.c:145

0x40166822: esp_event_loop_run at /opt/esp/idf/components/esp_event/esp_event.c:582 (discriminator 3)

0x401668fd: esp_event_loop_run_task at /opt/esp/idf/components/esp_event/esp_event.c:115

0x400868c1: vPortTaskWrapper at /opt/esp/idf/components/freertos/xtensa/port.c:169

Hi @teekaytk
Thank you for reporting this issue. There's a bug in the PPP netif event reporting (the event data contains the actual netif object content instead of it's pointer). Please see the below patch fixing this issue:
0001-esp-netif-Fix-PPP-netif-event-posting-to-include-esp.patch.txt

Thanks @david-cermak
This worked!

Reopen until the fix is available on GitHub, when the issue will be automatically updated.


uniform TEXTURE_PIXEL_SIZE bugged

Godot version:
3.2.2 stable
3.2.3 stable
3.2.4 beta6 (everywhere act same)
OS/device including version:
confirmed in AMD and Nvidia
WIndows and Linux
bug works with Angle in Web
Issue description:
when Viewport size changed by "some" event like a fullscreen event or double click on window header
TEXTURE_PIXEL_SIZE uniform has wrong value
Screenshots:

second image from Chromium browser
Values on image:
two text lines on top - is viewports size from gdscript
White number - VIEWPORT_SIZE uniform in 3d scene
Yellow number - iResolution value, is vec2 iResolution=1./TEXTURE_PIXEL_SIZE;
Aqua number - textureSize(TEXTURE,0) value
Expected:
all numbers should be the same
Real result - vec2 iResolution=1./TEXTURE_PIXEL_SIZE; not correct
Steps to reproduce:
Launch my web build in your webbrowser link to web build and double click on browser header, or F11 to fullscreen page. Or open source project and launch in Godot, and double click on launched window header.
Minimal reproduction project:
texture_size_bug.zip

Update - just have tested bug in AMD, bug works there same like on Nvidia, so it Godot bug.

Update - just have tested bug in AMD, bug works there same like on Nvidia, so it Godot bug.

I tested the link to web build and the result is always correct(all numbers same in any resolution, or fullscreen).
I use Nvidia, Google Chrome

I tested the link to web build and the result is always correct(all numbers same in any resolution, or fullscreen).
I use Nvidia, Google Chrome

I tested the link to web build and the result is always correct(all numbers same in any resolution, or fullscreen).
I use Nvidia, Google Chrome

you sure?
can you drag by mouse "tab" of opened my web-example (to make it new window) and then double click on window header, are numbers still same?
I tested it on 3 PC with diferent hardware and OS, in Nviida and AMD

update: I find in some situations(the Chrome window is very small), it will be not correct.

yes but it also same when you going full screen (F11) or just double click on header

I tested the link to web build and the result is always correct(all numbers same in any resolution, or fullscreen).
I use Nvidia, Google Chrome

you sure?
can you drag by mouse "tab" of opened my web-example (to make it new window) and then double click on window header, are numbers still same?
I tested it on 3 PC with diferent hardware and OS, in Nviida and AMD

update: I find in some situations(the Chrome window is very small), it will be not correct.

yes but it also same when you going full screen (F11) or just double click on header

you sure?
yes, i am sure. I tested as you described and the results are the same always

you sure?
yes, i am sure. I tested as you described and the results are the same always

okey thank for tesing

okey thank for tesing

**this is not just-html5 bug @Chaosus **
this bug happens in exe or native Godot version
WebGL build is only "fastest way to test it"

**this is not just-html5 bug @Chaosus **
this bug happens in exe or native Godot version
WebGL build is only "fastest way to test it"


Last gift getting cut in list of gifts

https://user-images.githubusercontent.com/68264080/142812191-51891f64-3d2e-4528-82b9-3779b048de1a.mp4

Still having the same issue on live app v2.0.5(381)

Verified this issue on staging app v2.0.5(382)


[docs] How to get the screen size

How to get the screen size

In javascript use this https://tauri.app/v1/api/js/modules/window#currentmonitor to get the monitor and monitor.size to get its size.
In Rust: basically the same starting with https://docs.rs/tauri/latest/tauri/window/struct.Window.html#method.current_monitor


Laravel validation - show the error messages

I'm starting in the VueJS world and I really loved this package!
I would like to use this package with Laravel and Quasar (Vue, vuex, axios and vuelidate).
I'm making APIs available for query, inclusion, editing, and deletion of records but I want to perform server-side checks (Laravel validations).
I'd like some help on how to show error messages from Laravel's validations.
Can anybody help me ?

This validation don't support object message as validation of jQuery
But it is support customize show/hide error message by a component ( in their document http://prntscr.com/kowun3 ). i think it's more flexible for vue. Because vue component is not a pure html, so generating automatic "" message error like jQuery will occur some issues don't expect


Add continuation tokens

Resolves #132.

Closing since .token was added to 2.4 in #140.  Thanks for the great work!


fix(bufferline): add missing highlight groups for tab separator

This commit enables customization of the tab_separator and tab_separator_selected options for Bufferline. These allow for styling the tab entries that appear on the end of the bufferline.
fixes #581

It looks like there were more groups missing, I've gone ahead and added all of them here 👍

Could you please rebase your branch from master and then squash your changes in a single commit please ?

@GaetanLepage can do! I have a few more changes coming for other modules as I work on porting things. I'll revisit the PRs and try to break things apart.

I have quite a few more changes and I'm going to break them apart and draft new PRs. Sorry for the churn!


[MINOR] Propagate failed cleaner status

Change Logs
PR Cloned from: https://github.com/apache/hudi/pull/5965
This PR changes the HoodieCleaner utility so that errors will propagate to spark + YARN (or whatever RM you are using).
Currently, if the cleaner fails for any reason the application status (in YARN) is not set to FAILED. When running the cleaner outside of the write process (such as DeltaStreamer) this result in a job that cannot be tracked by the coordinating system. We are using Apache Airflow to run the cleaner in parallel on our tables. If the cleaner fails, we need the application status to be set to FAILED so that the airflow sensor for the job can alert us.
There was an old issue and PR created to catch all cleaner errors and log them. According to the ticket, the change was done because the cleaner process was hanging during certain scenarios. I have not been able to reproduce this behavior. There is no information about the environment on the ticket (possibly only occurred under spark 2.x?)
Impact
Minor impact. This would only negatively impact users who currently rely on failures in the async cleaner to return SUCCESS in resource managers like YARN.
Risk level (write none, low medium or high below)
If medium or high, explain what verification was done to mitigate the risks.
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@vinothchandar FYI I moved the PR over to this new PR

Any update on the status of this review?

@vinothchandar sorry for the ping but you were familiar with the previous PR. Would you mind taking a look and merging if all looks OK?


S5 (FMEA Table) Components column does not mention components in S3(System Boundaries and Components)

The components mentioned in Section 3:

Do not align with the components referenced in the FMEA table (i.e. General, Risk Prediction) making it difficult to trace hazards from the table back to the aforementioned components. It would be nice to see the components also included in the table, and if there are more then please provide a short description of the component in Section 3 to provide visibility into the root of the hazards.

Risks in our system can travel through and cover multiple of these aspects so as a result it would not make sense to sub categories them by these 4 labels, instead we thought general made more sense.


Fix code style issue #25 and #17

Closing to lack of activity, feel free to pick it up. Although, I'm not sure we wanna support wns in the future.


TOC entries only generated when online

Steps to reproduce:

ensure machine is connected to the Internet.

find or write a Markdown document that should generate a good TOC, e.g.
this gist.

use gh-md-toc to generate a TOC for it, to stdout.

$ ~/bin/gh-md-toc ~/Desktop/burnout.md

Table of Contents
=================

* ["The Modern Day Sisyphus: \#libtech Burnout and You": Becky Yoose](#the-modern-day-sisyphus-libtech-burnout-and-you-becky-yoose)
  * [Abstract & slides](#abstract--slides)
  * [twelve phases of burnout](#twelve-phases-of-burnout)
  * [what's special about \#libtech burnout](#whats-special-about-libtech-burnout)
  * [how to prevent/recover](#how-to-preventrecover)
  * [\.\.\.but this isn't about what the victim should do](#but-this-isnt-about-what-the-victim-should-do)
    * [co\-workers](#co-workers)
    * [managers](#managers)
    * [community](#community)
  * [community privilege](#community-privilege)

Created by [gh-md-toc](https://github.com/ekalinin/github-markdown-toc.go)
$

turn off networking

attempt to generate the TOC again.

Expected:

same TOC is generated as in step 3.

Actual:

only header is generated:

$ ~/bin/gh-md-toc ~/Desktop/burnout.md
Table of Contents
Created by gh-md-toc
$

## Notes

- MacOS X 10.10.5
- go version go1.6 darwin/amd64
- `gh-md-toc` 5.0 built from commit 0523167dffc9cedcbc55996e3717716760bb4454

Hey @dmolesUC3
Yeah, it can work only when the Internet is on. As I didn't want to write a parser MD → HTML.
For converting markdown into plain HTML, I use GitHub's API. That's why this tool always needs an internet connection.

OK. It might be nice to get an error message and document that in the README.


GitHub, Outlook and WhatsApp desktop notifications

[ ] Github (don't know if this can be implemented notifier-for-github)
[ ] Outlook doesn't show desktop notification
[ ] WhatsApp doesn't show profile photo on desktop notification like Facebook Messenger

A toggle for desktop notifications appears in the quick settings panel in Firefox and Chromium, but not in Ferdi. It also doesn't matter which outlook service is used. (O365 and non) I don't know if that's helpful for anyone who wants to look into this.

A toggle for desktop notifications appears in the quick settings panel in Firefox and Chromium, but not in Ferdi. It also doesn't matter which outlook service is used. (O365 and non) I don't know if that's helpful for anyone who wants to look into this.

I've also narrowed it down to webpack://owa/./packages/common/controls/packages/controls/owa-webpush-notifications/lib/isWebPushNotificationSupported.ts:
import { isFeatureEnabled } from 'owa-feature-flags';
import {
    isBrowserChrome,
    isBrowserFirefox,
    isBrowserEDGECHROMIUM,
    isBrowserSafari,
} from 'owa-user-agent';
import getUserConfiguration from 'owa-session-store/lib/actions/getUserConfiguration';
import { getOwaWorkload, OwaWorkload } from 'owa-workloads';

export const isWebPushNotificationSupported = (windowObj: Window): boolean => {
    if (
        !isFeatureEnabled('fwk-webPushNotification') &&
        !isFeatureEnabled('fwk-webPushNotificationV2')
    ) {
        return false;
    }
    if (
        !windowObj.navigator ||
        windowObj.navigator.serviceWorker == null ||
        windowObj.PushManager == null
    ) {
        return false;
    }
    return (
        (isBrowserChrome() ||
            isBrowserFirefox() ||
            (isFeatureEnabled('fwk-webPushNotificationExtended') &&
                (isBrowserEDGECHROMIUM() || isBrowserSafari()))) &&
        !getUserConfiguration().SessionSettings.IsUserCultureRightToLeft &&
        (getOwaWorkload() == OwaWorkload.Mail || getOwaWorkload() == OwaWorkload.Calendar)
    );
};

Luckily someone forgot to remove some of the source maps.

Here is their detection algorithm (changed to be executable from the console)
(function () {
  function getUserAgentString() {
    return typeof navigator != "undefined" && navigator.userAgent
      ? navigator.userAgent
      : "";
  }

  // Algorithm copied from Aria
  const Browser = {
    MSIE: "MSIE",
    CHROME: "Chrome",
    FIREFOX: "Firefox",
    SAFARI: "Safari",
    EDGE: "Edge",
    EDGE_CHROMIUM: "Edge Anaheim",
    ELECTRON: "Electron",
    SKYPE_SHELL: "SkypeShell",
    PHANTOMJS: "PhantomJS",
    OPERA: "Opera",
    UNKNOWN: "Unknown",
  };

  function getBrowserInfo() {
    let browser = Browser.UNKNOWN;
    let browserVersion;

    const userAgentString = getUserAgentString();
    if (/OPR\//.test(userAgentString)) {
      browser = Browser.OPERA;
      browserVersion = getNonIEBrowserVersion("OPR");
    } else if (/PhantomJS/.test(userAgentString)) {
      browser = Browser.PHANTOMJS;
    } else if (/Edge/.test(userAgentString)) {
      browser = Browser.EDGE;
    } else if (/Edg/.test(userAgentString)) {
      browser = Browser.EDGE_CHROMIUM;
      browserVersion = getNonIEBrowserVersion("Edg");
    } else if (/Electron/.test(userAgentString)) {
      browser = Browser.ELECTRON;
    } else if (/Chrome/.test(userAgentString)) {
      browser = Browser.CHROME;
    } else if (/Trident/.test(userAgentString)) {
      browser = Browser.MSIE;
      browserVersion = getIEBrowserVersion();
    } else if (/Firefox/.test(userAgentString)) {
      browser = Browser.FIREFOX;
    } else if (/Safari/.test(userAgentString)) {
      browser = Browser.SAFARI;
      browserVersion = getNonIEBrowserVersion("Version");
    } else if (/SkypeShell/.test(userAgentString)) {
      browser = Browser.SKYPE_SHELL;
    }

    return {
      browser,
      browserVersion: browserVersion || getNonIEBrowserVersion(browser),
    };
  }

  function getNonIEBrowserVersion(prefix) {
    let match = getUserAgentString().match(
      new RegExp(prefix + "/([\\d,\\.]+)")
    );

    return !match ? [] : parseVersionString(match[1]);
  }

  function getIEBrowserVersion() {
    let match = getUserAgentString().match(/MSIE ([\d,\.]+)/);
    if (match) {
      return parseVersionString(match[1]);
    }
    match = getUserAgentString().match(/rv:([\d,\.]+)/);
    return !match ? [] : parseVersionString(match[1]);
  }

  function parseVersionString(version) {
    return version.split(/[\.,]/).map((num) => parseInt(num));
  }
  console.info(getBrowserInfo());
})();

And here is our problem: {browser: "Electron", browserVersion: Array(0)}
So we "just" need to change the user agent string.

This seems to be already done in the most recent versions of the services, but turning it on results in the following: Uncaught (in promise) DOMException: Registration failed - push service not available

It might be possible to just simply use https://github.com/MatthieuLemoine/electron-push-receiver but I might need support from the ferdi team on where to implement that

It might be possible to just simply use https://github.com/MatthieuLemoine/electron-push-receiver but I might need support from the ferdi team on where to implement that

Not stale


Ensure view compilation inherits the right settings from the project's compilation settings

From @cesarbs in https://github.com/aspnet/Razor/issues/598:
If you strong name a web app, Razor fails to compile pages:
An error occurred during the compilation of a resource required to process this request. Please review the following specific error details and modify your source code appropriately.

/Views/Home/Index.cshtml

Error signing output with public key from file 'C:\Users\cesars\Documents\Visual Studio 2015\Projects\RC1StrongNameValidation\src\RC1StrongNameValidation\Key.snk' -- Assembly signing not supported.

Apparently this happens because no strong name provider is being set for Roslyn (see https://github.com/dotnet/roslyn/issues/4953 and https://github.com/aspnet/dnx/blob/1bb5a1d24fcd7d21d2278528c243e2b9f181c43d/src/Microsoft.Dnx.Compilation.CSharp/RoslynCompiler.cs#L205).

@pranavkm parking this on your plate, though I'm not sure how actionable this issue is right now.
@davidfowl can this be done already for CLI?

This is likely to be covered by https://github.com/aspnet/Mvc/issues/3633

Ah, ok, let's hold onto this bug anyway to ensure the scenario is covered after the full switch.

We switched over to the CLI in https://github.com/aspnet/Mvc/commit/80b6996701ca240b8b4b1ba8f56c7ecae3e69b60 and haven't seen issues with this feature.


fix: Listing price: I set the price 1 DOT it listed item for 100 DOTs

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Bugfix

Needs QA check

@kodadot/qa-guild please review

Context

[x] Closes #7343
[ ] Requires deployment <snek/rubick/worker>

Did your issue had any of the "$" label on it?

[x] My DOT address: Payout

Screenshot 📸

[ ] My fix has changed UI

Copilot Summary

🤖 Generated by Copilot at dad5d9a
Fixed list price display for different chains by using decimals from useChain in ListingCartModal.vue.

🤖 Generated by Copilot at dad5d9a

useChain has decimals
Balance shows list price right
Autumn of chains

@Jarsen136 I'm not sure if the problem comes from this PR, but transaction fails when i try to mint it, even if i have the proper balance on Asset Hub
https://assethub-polkadot.subscan.io/tx/0x95edc539021ecc7dedcb2f8384166bf8970ada33fe5ed9e89269e149d3ce03f9
I was able to mint properly on canary although i could not reproduce Luuu issue

@Jarsen136 I'm not sure if the problem comes from this PR, but transaction fails when i try to mint it, even if i have the proper balance on Asset Hub
https://assethub-polkadot.subscan.io/tx/0x95edc539021ecc7dedcb2f8384166bf8970ada33fe5ed9e89269e149d3ce03f9

It's not related to this PR. I have not changed anything about minting on this PR. Would you mind trying again?

I was able to mint properly on canary although i could not reproduce Luuu issue

Listing any nft on ahp network could reproduce this bug.


Use exec_module instead of load_module

DeprecationWarning: the load_module() method is deprecated and slated for removal in Python 3.12; use exec_module() instead

https://docs.python.org/3/library/importlib.html#importlib.machinery.SourcelessFileLoader.load_module
https://docs.python.org/3/library/importlib.html#importlib.abc.Loader.load_module

A bit of googling for python importlib the load_module() method is deprecated and slated for removal in Python 3.12; use exec_module() instead gave me these pointeres:
https://bugs.python.org/issue43540
https://github.com/sktime/skbase/pull/190

I would like to contribute to this

I encountered an issue when running pytest on the repository. Specifically, the test suite is failing with the following error: [
repr/tests/run_test.py::test_get_module - crepr.crepr.CreprError: Error: file 'tests/classes/kw_only_test.py' not found.
failed crepr/tests/run_test.py::test_get_init_args - crepr. crepr.CreprError: Error: file 'tests/classes/kw_only_test.py' not found.
]
I reviewed the get_module function in crepr.py to ensure it is correctly located and loaded
Can you please help me deal with this pytest error

merge develop into your fork, should be fixed now


Active status & maintainers ?

Hi, great library. Thank you for the hard work.
Its seems there is not much activity in the last year. It would be helpful I think to know:

Is this still in active development
Is there a road map to achieve if so
Do you need maintainers
If you do, what is the process of being a maintainer, and do you have time to support maintainers on board

Maybe it's time for us to migrate to hadolint, looks like Super-Linter has also done a corresponding migration.


[Question]: Can you generate an accurate solidity contract from bytecode if you already have the ABI?

Version
0.4.5
Please check the following

[X] This is not a duplicate question
[x] I have checked the wiki beforehand

Operating System
Linux
Ask your question here
Can you generate a more accurate solidity contract if you already have the ABI (and the bytecode)? My problem is I want to decompile a few contracts, I already have the ABI (+bytecode) for them, but don't have access to the contracts. Maybe this could be a potential new feature if not implemented, perhaps by adding an --abi flag with an abi input to the heimdall compile command.
Thank you for your time and for the awesome tool!

Hey! Thanks for opening this!
Would heimdall only use the provided abi to resolve signatures (i.e. skipping lookups)?
I suppose it would be more accurate this way and wouldn't have to inherit argument types from bytecode, which would improve overall decompilation.
Very interesting! I'll take a crack at it this weekend.

Just to follow up here, will likely include in 0.5.0 or 0.5.1, (next 1 or 2 builds)


#UI-1 update(ui-alert): ui-alert in react

@rebeccaOberto don't forget to export the component from src/index.ts also


Add version: WinDirStat.WinDirStat.Beta version <IP_ADDRESS>4-10-11

Pull request has been created with WinGet Releaser :rocket:
Microsoft Reviewers: Open in CodeFlow

Sorry for the noise, this was still a test. Thus closing. Next one should be good to go.


Migrate falco Slack to CNCF Slack

The Slack instance should be migrated to the CNCF Slack to keep it vendor neutral.
We created a #falco channel on slack.cncf.io

Closing in favor of #983 as we are discussing this today and coming up with a plan to migrate.


KAFKA-10185: Restoration info logging

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Unrelated failure:
org.apache.kafka.connect.mirror.MirrorConnectorsIntegrationTest.testReplication


No module named _pywrap_tensorflow_internal

Initially I had tensorflow 2.3 but due to
module 'tensorflow' has no attribute 'GPUOptions'
changed to tensorflow 1.7
and now iam getting the following error and I am unable to proceed with my proj.Though they have mentioned the link I am kind off lost.If anyone has any suggestions regarding the issue it would be much appreciated.Thank You in advance
ImportError: No module named '_pywrap_tensorflow_internal'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\pnc\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "C:\Users\pnc\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 28, in
_pywrap_tensorflow_internal = swig_import_helper()
File "C:\Users\pnc\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 20, in swig_import_helper
import _pywrap_tensorflow_internal
ModuleNotFoundError: No module named '_pywrap_tensorflow_internal'
Failed to load the native TensorFlow runtime.
See https://www.tensorflow.org/install/install_sources#common_installation_problems
for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.

remove tensorflow completely,
pip uninstall tensorflow' pip uninstall tensorflow-gpu'
then install tensorflow, it should work fine.

It worked for me after doing this command :
pip install tensorflow-gpu == 1.15.5

I was previously with tensorflow-gpu 1.15.0 when it was showing the error


dashboard/config: refactor linux config system

Currently we have configs for upstream kernel, some stable releases, Android, ChromeOS and some other kernels; multiplied by KASAN/KMSAN/KCSAN/KMEMLEAK, plus separate config for USB tree. Some of these are produced and updated manually, some -- with the help of assorted scripts. Some of them are not updated consistently, or at all (e.g. nobody cares about stable configs enough to update them). We also need way more configs: e.g. baseline configs for config bisection, configs for other architectures, configs for other sanitizers as they get ported to other kernels. It would be good to switch some configs to SLUB and keep some to use SLAB, this would require 2 upstream configs, which is too hard to maintain now.
Ultimately we may have several hundreds of configs. Managing them manually is not feasible: too resource intensive, tedious and error prone.
Design proposal for kernel configuration system:
https://docs.google.com/document/d/134XWCo9yDmxIJB_YMAMV1al7diYYDVFZsHoTKULO_aM/edit#
Additionally we have several uses for a kconfig package, which would be able to parse kernel kconfig files and do some manipulations with config files (most importantly, understanding dependencies between config parameters).
Namely, USB config scripts use some unversioned third-party library to work with dependencies:
https://github.com/google/syzkaller/blob/cf0c8d051223a94af93bcf76b6c227de2a1d6f35/dashboard/config/generate-config-usb.sh
https://github.com/google/syzkaller/blob/cf0c8d051223a94af93bcf76b6c227de2a1d6f35/dashboard/config/kconfiglib-merge-usb-configs.py
Script for generation of baseline configs uses some sed's and grep's to understand kconfig dependencies:
https://github.com/google/syzkaller/blob/cf0c8d051223a94af93bcf76b6c227de2a1d6f35/dashboard/config/generate-baseline-configs.sh
The config-bisect.pl script does a very trivial thing, mishandles dependencies and costs us too much to integrate as compared to what it actually does:
https://github.com/google/syzkaller/blob/cf0c8d051223a94af93bcf76b6c227de2a1d6f35/pkg/vcs/linux.go#L302
In the long run we need a package that understands kconfig/dependencies. It would allow us to implement all of this in a better and maintainable way.

The config-bisect.pl script does a very trivial thing, mishandles dependencies and costs us too much to integrate as compared to what it actually does:
https://github.com/google/syzkaller/blob/cf0c8d051223a94af93bcf76b6c227de2a1d6f35/pkg/vcs/linux.go#L302
In the long run we need a package that understands kconfig/dependencies. It would allow us to implement all of this in a better and maintainable way.

There are these patches from Dan Carpenter and Jukka Kaartinen:
https://www.spinics.net/lists/linux-kbuild/msg25447.html
https://www.spinics.net/lists/linux-kbuild/msg25446.html
What do you think about using these together with config_bisect.pl? I've been struggling how to integrate them with Syzkaller bisection. Even thought we would get them integrated in mainline we still don't have them in older kernels.

Fixed

Fixed


Add HackerOne provisioning tutorial page

This adds a document explaining how to setup SCIM on HackerOne.
See https://docs.hackerone.com/en/articles/9010541-scim-provisioning-for-hackerone for more information.

@microsoft-github-policy-service agree company="HackerOne"

@derrudi77 - Thanks for your pull request. A member of the content team will review it.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@DeRRudi77 SCIM/Provisioning is not enabled in the Entra ID gallery app for HackerOne. So, we can't add SCIM tutorial for it.
#please-close

@v-missam how can I enable it?

@DeRRudi77 you need to reach out to the Hacker One team, and they need to get in touch with Microsoft Provisioning team to get provisioning service on that app.


Cancelling Llama CPP Generation Using Agent's astream_events

          > @TheJerryChang will it also stop the llm`s completion process? I am using langchain llama cpp with conversationalretrievalchain

Did you solve this? It seems I'm facing the same issue.

@TheJerryChang will it also stop the llm`s completion process? I am using langchain llama cpp with conversationalretrievalchain

For my use case, Im using Chat model and did not try with completion process. How do you do the streaming? Did you use chain.stream()? I personally think as long as it supports calling chain.stream() to proceed the streaming, it then could be interrupted by raising an exception during the iteration

I'm using the .astream_events method of AgentExecutor. I'm trying to figure out how to stream an agent response but also be able to cancel it before it finishes generating.
Originally posted by @Spider-netizen in https://github.com/langchain-ai/langchain/issues/11959#issuecomment-1975388015
Hi,
Can this be achieved?
Thanks.

Hi langchain team, any update on this?


[DNM] Test gen kobj fast

pyelftools can't be parallelized so we can't process multiple CUs at once. This is the main bottleneck whenever we need to iterate through the whole ELF file.


google.drive.files.update() fail with Google Apps Script Bound to Google Sheets

I have a Scripts Bound to Google Sheets. when I edit it, the url is like: https://script.google.com/a/mydomain.com/macros/d/Md-MytdOSdIJ9XDzO7J5kDJpRT9d30hqB/edit .
I use node-google-apps-script( https://github.com/danthareja/node-google-apps-script ) to export files from it. it could be exported by id Md-MytdOSdIJ9XDzO7J5kDJpRT9d30hqB successful.
But when I try to import back, I got a error:
Pushing back up to Google Drive...
An error occured while running upload command: null
Upload failed.

it seems like that google.drive.files.update()  does not support bounded scripts.

another issue here: https://github.com/danthareja/node-google-apps-script/issues/24

For now at least, closing per @cjs-cynic-net's comment as it appears the deficiency is in the v2 Google Drive API, not google-api-nodejs-client. Perhaps the v3 API works better? gapps would need to try it.


getting error when pressing download button (sample app code)

getting the following error when pressing download button (step 2 of the quickstart - sample app code): given data uri is not formatted correctly with data uri syntax

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

ID: e20117b0-215d-d6aa-584e-89d3b6141980
Version Independent ID: 427b8f4b-a590-be9b-e3a7-ed37add89295
Content: Azure Cosmos DB: Build a todo app with Xamarin
Content Source: articles/cosmos-db/create-sql-api-xamarin-dotnet.md
Service: cosmos-db
Sub-service: cosmosdb-sql
GitHub Login: @anfeldma-ms
Microsoft Alias: anfeldma

@arizzi12 Our Engineering team is aware of this issue. The fix has been pushed. Please check in 24 hours and if the issue still persists please get back to us and we will check with the team.

@arizzi12 Our Engineering team is aware of this issue. The fix has been pushed. Please check in 24 hours and if the issue still persists please get back to us and we will check with the team.

@arizzi12 Please convey if you are still facing the issue.

@arizzi12 Please convey if you are still facing the issue.

@arizzi12 We are going ahead and closing this issue. Please get back to us if you have any further issues.

@arizzi12 We are going ahead and closing this issue. Please get back to us if you have any further issues.

@arizzi12 The issue is resolved now and code samples are available for download via the portal.
Thank you again for taking the time to report this issue.

@arizzi12 The issue is resolved now and code samples are available for download via the portal.
Thank you again for taking the time to report this issue.


Complete copy edit pass

@rotycenh

OPS Build status updates of commit d5ac217:
:clock10: Preparing: average preparing time is 45 sec(s)

OPS Build status updates of commit d5ac217:
:clock10: Incremental building: average incremental building time is 22 sec(s)

OPS Build status updates of commit d5ac217:
:warning: Validation status: warnings

File
Status
Preview URL
Details

bread/toc.json
:warning:Warning

Details

toc.json
:warning:Warning

Details

building-blocks/extending-templates/toc.json
:warning:Warning

Details

docs/cloud-adoption/infrastructure/virtual-machines/basic-workload.md
:white_check_mark:Succeeded
View

bread/toc.json

[Warning] Error happen when converting bread/toc.json to Pdf. Details: Could not find a part of the path 'T:\oujm\bread\toc.json'.

toc.json

[Warning] Error happen when converting toc.json to Pdf. Details: Could not find file 'T:\oujm\toc.json'.

building-blocks/extending-templates/toc.json

[Warning] Error happen when converting building-blocks/extending-templates/toc.json to Pdf. Details: Could not find a part of the path 'T:\oujm\building-blocks\extending-templates\toc.json'.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Proposal: Add sensor device classes

Description:
We should add device classes to sensors, just like we have for binary sensors.
Previously, we would try to distinguish / group sensors by unit of measurement in, for example, the history panel in the frontend. However, this can result in a graph that contains both battery levels and humidity (both %).
As an example, I've updated 3 sensors.
We will update the frontend to recognize device classes for sensors and automatically assign icons.
I think that we should be a little conservative in adding device classes. We should only add them for the ones that are common.

We could also use this to group data for data collectors like influxdb and datadog. Limitation is that the device class isn't available per state attribute.

Just checking: this will impact history panel only for now ?

It will impact nothing yet because history panel has not been updated yet.


puppet node graph error

There were no resources created in the node graph. Is there an include statement missing?

Hi @jrbast , could you fill out the information in the new issue request? It will help us answer your question. Otherwise, all I have is the error message above.

How do I do that.. never done this before??
Thank you,
Jeff
Jeff Bast
IT Server Analyst Lead
TECH - Hybrid Cloud Platform Services
<EMAIL_ADDRESS>3101 S. Packerland Drive, Green Bay, WI 54306
•<PHONE_NUMBER>  •<PHONE_NUMBER> ext.5928368  • bastj<EMAIL_ADDRESS>Working hours: Monday thru Friday 7:00am - 4:00pm
Mail-stop: US.GRB.01.03.11
The information contained in this e-mail is privileged and confidential, and for use only of the addressees of same and/or those who might have been specifically authorized to read it.  If you have received this e-mail by error, please destroy it and notify the sender, to his/her e-mail address.  Any non-authorized dissemination, distribution or reproduction of this message is strictly prohibited, and subject to the sanctions established in the applicable laws.
From: James Pogran<EMAIL_ADDRESS>Sent: Thursday, November 07, 2019 9:36 AM
To: lingua-pupuli/puppet-vscode<EMAIL_ADDRESS>Cc: Bast, Jeff<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [lingua-pupuli/puppet-vscode] puppet node graph error (#577)
Hi @jrbasthttps://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjrbast&data=02|01|bastj%40schneider.com|f9fa49ed0cbc4e527dd808d763982364|e439976157cf420a98c2ca0f44ce8267|0|1|637087377385938275&sdata=DxHoY6MhwJ4u4MbU6Xz24BBgRgOd0WnXpToLdxAL4wA%3D&reserved=0 , could you fill out the information in the new issue request? It will help us answer your question. Otherwise, all I have is the error message above.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Flingua-pupuli%2Fpuppet-vscode%2Fissues%2F577%3Femail_source%3Dnotifications%26email_token%3DAGPVMFODMJFG2KWIIN7QUY3QSQYURA5CNFSM4JJ5PF22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDMZX4Y%23issuecomment-551132147&data=02|01|bastj%40schneider.com|f9fa49ed0cbc4e527dd808d763982364|e439976157cf420a98c2ca0f44ce8267|0|1|637087377385948274&sdata=963zxGBsFtIsaZ1DBklDoBXqrjYKlXji2ZPjH%2FjOF4s%3D&reserved=0, or unsubscribehttps://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAGPVMFLYPHCPE4CE33RI5STQSQYURANCNFSM4JJ5PF2Q&data=02|01|bastj%40schneider.com|f9fa49ed0cbc4e527dd808d763982364|e439976157cf420a98c2ca0f44ce8267|0|1|637087377385958274&sdata=bHQNXTn7GF3YEBhQA3U53rhewzAd3LUgeLojGDqp9yo%3D&reserved=0.

No worries! When you create a new issue with https://github.com/lingua-pupuli/puppet-vscode/issues/new/choose, it presents several choices. Choosing bug report lists this template, which you can then answer inline:

What Versions are you running?

What is Expected?
How Did You Get This To Happen? (Steps to Reproduce)

Output Log

Duplicate of #578


Wip first draft of a prune command.

What do you think of it now.
TODO:

tests

Well, it seems too much logic in Command to me.
You should think to a Command like if it were a Controller: so it should be as thin as possible, while all logic should be pushed in Model, if possible.

I didn't moved it there yet because the distinction between the responsibilities of a package, a repository and the bowerphp file.
Not sure where to put what.

If you are in doubt, put it in Bowerphp class.
Also, try to avoid private methods in Command, if possible.
Also, direct access to bower.json file from a Command is a big big bad smell.

Hi @mikeSimonson, any news about this PR?

Hi @garak
It will be done by next week.

Sorry Mike, this PR is now too old. Feel free to open a new one.
Thank you.


[CLOUDSTACK-10020] Changes to make marvin work with projects and VPCs

Currently there are a lot of problems when running Marvin with projects and VPCs enabled. Most of them are related to the fact that some functions don't take the projectid or the vpcid parameter. This PR is an attempt to fix those problems and have better testability of setups with VPC and projects.

ACS CI BVT Run
Sumarry:
Build Number 1031
Hypervisor xenserver
NetworkType Advanced
Passed=95
Failed=17
Skipped=12
Link to logs Folder (search by build_no): https://www.dropbox.com/sh/r2si930m8xxzavs/AAAzNrnoF1fC3auFrvsKo_8-a?dl=0
Failed tests:

test_routers.py

test_03_restart_network_cleanup Failing since 10 runs

test_04_restart_network_wo_cleanup Failing since 10 runs

test_05_router_basic Failing since 10 runs

test_06_router_advanced Failing since 10 runs

test_08_start_router Failing since 10 runs

test_09_reboot_router Failing since 10 runs

test_network.py

test_reboot_router Failing since 10 runs

test_router_dns.py

test_router_dns_guestipquery Failed

test_deploy_vm_iso.py

test_deploy_vm_from_iso Failing since 85 runs

test_list_ids_parameter.py

ContextSuite context=TestListIdsParams>:setup Failing since 48 runs

test_volumes.py

test_06_download_detached_volume Failed

test_nic.py

test_01_nic Failing since 10 runs

test_vm_life_cycle.py

test_10_attachAndDetach_iso Failing since 85 runs

test_routers_network_ops.py

test_01_isolate_network_FW_PF_default_routes_egress_true Failing since 118 runs

test_02_isolate_network_FW_PF_default_routes_egress_false Failing since 118 runs

test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Failing since 115 runs

test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failing since 115 runs

Skipped tests:
test_vm_nic_adapter_vmxnet3
test_01_verify_libvirt
test_02_verify_libvirt_after_restart
test_03_verify_libvirt_attach_disk
test_04_verify_guest_lspci
test_05_change_vm_ostype_restart
test_06_verify_guest_lspci_again
test_static_role_account_acls
test_11_ss_nfs_version_on_ssvm
test_nested_virtualization_vmware
test_3d_gpu_support
test_deploy_vgpu_enabled_vm
Passed test suits:
test_deploy_vm_with_userdata.py
test_affinity_groups_projects.py
test_portable_publicip.py
test_vm_snapshots.py
test_over_provisioning.py
test_global_settings.py
test_router_dnsservice.py
test_scale_vm.py
test_service_offerings.py
test_routers_iptables_default_policy.py
test_loadbalance.py
test_reset_vm_on_reboot.py
test_deploy_vms_with_varied_deploymentplanners.py
test_non_contigiousvlan.py
test_login.py
test_public_ip_range.py
test_multipleips_per_nic.py
test_metrics_api.py
test_regions.py
test_affinity_groups.py
test_network_acl.py
test_pvlan.py
test_deploy_vm_root_resize.py
test_resource_detail.py
test_secondary_storage.py
test_disk_offerings.py

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1014

@blueorangutan test

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-1415)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 44675 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2206-t1415-kvm-centos7.zip
Intermitten failure detected: /marvin/tests/smoke/test_deploy_virtio_scsi_vm.py
Intermitten failure detected: /marvin/tests/smoke/test_iso.py
Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermitten failure detected: /marvin/tests/smoke/test_router_dhcphosts.py
Intermitten failure detected: /marvin/tests/smoke/test_routers_network_ops.py
Intermitten failure detected: /marvin/tests/smoke/test_snapshots.py
Intermitten failure detected: /marvin/tests/smoke/test_vpc_vpn.py
Test completed. 51 look OK, 6 have error(s)

Test
Result
Time (s)
Test File

test_01_vpc_remote_access_vpn
Failure
81.08
test_vpc_vpn.py

test_04_rvpc_privategw_static_routes
Failure
416.55
test_privategw_acl.py

test_05_iso_permissions
Failure
0.05
test_iso.py

test_02_edit_iso
Failure
0.05
test_iso.py

ContextSuite context=TestSnapshotRootDisk>:setup
Error
0.00
test_snapshots.py

ContextSuite context=TestRedundantIsolateNetworks>:setup
Error
1817.24
test_routers_network_ops.py

ContextSuite context=TestDeployVirtioSCSIVM>:teardown
Error
78.79
test_deploy_virtio_scsi_vm.py

test_change_service_offering_for_vm_with_snapshots
Skipped
0.00
test_vm_snapshots.py

test_09_copy_delete_template
Skipped
0.01
test_templates.py

test_06_copy_template
Skipped
0.00
test_templates.py

test_static_role_account_acls
Skipped
0.02
test_staticroles.py

test_11_ss_nfs_version_on_ssvm
Skipped
0.02
test_ssvm.py

test_01_scale_vm
Skipped
0.00
test_scale_vm.py

test_01_primary_storage_iscsi
Skipped
0.03
test_primary_storage.py

test_vm_nic_adapter_vmxnet3
Skipped
0.00
test_nic_adapter_type.py

test_nested_virtualization_vmware
Skipped
0.00
test_nested_virtualization.py

test_06_copy_iso
Skipped
0.00
test_iso.py

test_deploy_vgpu_enabled_vm
Skipped
0.03
test_deploy_vgpu_enabled_vm.py

test_3d_gpu_support
Skipped
0.03
test_deploy_vgpu_enabled_vm.py

Additional review requested -- @nvazquez @DaanHoogland @borisstoyanov

@borisstoyanov We have tests that are internal where we use Projects and VPCs. I'll post those tests to the community as they get more mature

@syed is this still actual? are you still working on it?

@blueorangutan package

@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✖centos6 ✖centos7 ✔debian. JID-1029

Packaging result: ✖centos6 ✔centos7 ✔debian. JID-1031

@blueorangutan test

@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-1217)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 48506 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2206-t1217-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Smoke tests completed. 83 look OK, 0 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

let's see if this still works
@blueorangutan package

@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2260

@blueorangutan test

@DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-3046)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 43048 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2206-t3046-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_accounts.py
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Smoke tests completed. 83 look OK, 2 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_03_deploy_and_upgrade_kubernetes_cluster
Failure
254.49
test_kubernetes_clusters.py

test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL
Failure
311.11
test_vpc_redundant.py


Fix handling of staticfile settings if cloud provider and whitenoise are selected

The code prevents automatically assuming collectfast strategy and respective staticfile location when whitenoise is selected.
It implements conditional handling for these case:

Cloud provider ONLY
Whitenoise ONLY
Cloud provider AND Whitenoise

Fixes #4925

Thanks for the fix! Finally found some time to review and give it a spin.


[Feature Request] Some Features

An INI config file within the application that allows modification of only the current folder, not all subfolders. Users can toggle Enable/Disable it through context menu.
Add a separate entry to the menu for setting default drop effect.

desktop.ini
[.ShellClassInfo]
DefaultDropEffect = N
 1 - Copy
 2 - Move
 4 - Create a shortcut

3 Add another entry to menu for setting a icon (that's already specified in INI config file) as folder icon.
4. Include an option during the installation that makes it only display with Shift+Right-Click.
Concept:

Config.ini
# set predefined icon
Icon=Path\to\icon.ico,0

# Do not change this line manually, set it through context menu instead.
IncludeSub=1

Great suggestions. You're basically proposing a whole desktop.ini settings manager. Intriguing.
I'm currently working on some WinSetView enhancements, so it may be some time before I circle back to this one.
Although this could be more fun to work on. I'll see what I can do.
Thanks!


HTML Report additions

Addresses issue #2853 . Git hash and title now added to HTML report.
Addresses issue #2844. I replicated the the statistics that get displayed in the console in the HTML report. I originally intended the info to be at the top of the page but that needed a lot more work than putting it at the bottom, so I went with that instead.

Thanks for your feedback, @oktomus. I've attempted to address most of it in my latest commit.

Main changes are:

Results Class

Contains methods for setting and accessing data needed for results statistics information

git_information method

Refactored out from __write_header

Various name and style changes suggested by @oktomus

Thanks for the changes @russkev :)

Thanks for the kind words @oktomus I'm happy to help. Also thank you for your detailed reviews. Having read Clean Code, I very much appreciate the importance of good naming so I'm thankful to have guidance in that area.
I just made a few tweaks. Mainly changing a few names and adding some comments.
Also, what do you think about also implementing a __report_success method in ReportWriter so that the report lists all tests that were run?

Also, what do you think about also implementing a __report_success method in ReportWriter so that the report lists all tests that were run?

You mean including successful tests in the html report ? Right now, showing only the failed tests is very useful because you can easily see if something is wrong or not. For a complete report, there is always the txt report : http://testsuite.appleseedhq.net:8080/current-job/sandbox/tests/test scenes/testsuite_script_log.txt. But maybe you have a design in mind that allow to get all the tests, while still being able to quickly see the failures ? Although I'm not sure how seeing all tests could be useful

I have to admit, I haven't spent much time playing with the tool in practice so I'm not familiar with how it's used. I thought that it would be nice to have all the information from the tests in one nicely presented html file instead of split up over multiple documents.
The way I was thinking of doing it was to have the successful tests be just a single line and the failures would have the comparison images, maybe some red text, so that it would be easy to differentiate.

Okay, I've made those changes and merged the latest master. Let me know if there's anything else I can do.

Thanks a lot @russkev !
@dictoon I think we can merge it now. What do you say ?

I think we can merge it now. What do you say ?

Yes!
Thanks @russkev for your work, and @oktomus for the reviews.


Preserve data in json file

Preserved data for Book related class

Thank you for your kind words! We're truly committed to achieving our capstone project's big goal, and it's fantastic to hear that our efforts and implementations are making a positive impact🚀🙌