Can straight.el do all the jobs of auto-package-update?

I've noticed that auto-package-update.el provides functionality for automatically updating the installed Emacs packages periodically. I use straight.el and enabled its use-package integration. As far as I am concerned, the following configuration is currently used for automatic package updates:
(use-package auto-package-update
  :config
  (setq auto-package-update-delete-old-versions t)
  (setq auto-package-update-hide-results t)
  (auto-package-update-maybe))

I want to know whether straight.el can do all the jobs of auto-package-update mentioned above?
Regards,
HY

There is no timer based update mechanism built into straight.el.
The timer-based approach would only work well in the case where no manual intervention is needed for an update.
From a quick glance it looks as if auto-package-update.el supports quelpa and package.el.
I'm sure it could be extended to work with straight, but that would require changes to its source.

Yes, and I'm not sure I would advise this kind of thing in general. straight.el is intended to make package configurations more reproducible, not less.


[Bug] Not translated uploder column in translatable model error on update

Bug report
What I did
Install Backpack 6, configure translatable models with translated columns but a not translated uploader column.
What I expected to happen
To be able to create and update translatable models with translated columns but a not translated uploader column.
What happened
On create it works as expected but on update I get a "Integrity constraint violation column cannot be null" on the uploader not translated column even when the validation and content are ok.
What I've already tried to fix it
I debugged the Backpack source code and found out that the problem is with the Backpack\CRUD\app\Library\Uploaders\Uploader::getOriginalValue method. It checks if the model is translatable and the translation is enabled but it doesn't check if the attribute is translatable. I will submit a PR with my fix.
Is it a bug in the latest version of Backpack?
After I run composer update backpack/crud the bug... is it still there? Yes
Backpack, Laravel, PHP, DB version
When I run php artisan backpack:version the output is:
PHP VERSION:
PHP 8.2.12 (cli) (built: Oct 26 2023 17:33:26) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.2.12, Copyright (c) Zend Technologies
with Zend OPcache v8.2.12, Copyright (c), by Zend Technologies
LARAVEL VERSION:
<IP_ADDRESS>
BACKPACK PACKAGE VERSIONS:
backpack/basset: 1.2.2
backpack/crud: 6.6.1
backpack/filemanager: 3.0.7
backpack/generators: v4.0.3
backpack/pro: 2.0.23
backpack/settings: 3.1.0
backpack/theme-tabler: 1.2.0

Hello @ablunier
Thanks for reporting this issue, I will check about it and let you know the results; in case you push a PR, I will try to check it too.
Thanks again.

I will close this issue due https://github.com/Laravel-Backpack/CRUD/pull/5467 is merged.
Thanks.


June'22 WP3.7 (UKD-UKD-S6) Inkind quarterly update/plan

08/JUL/22

this has been submitted, see 08/JUL/22 email from @sugatakaviraj for a copy

09/SEP/22

this has been marked as received in the IPC Contribution Tracker


Gridsearch params --start-latency and --emeg-t-start should be in seconds, not miliseconds

Trying to keep things in consistent units throughout where possible

@sharonho1215, this is in 'invokers/run_gridsearch.py' there is the code:
    parser.add_argument("--start-latency", type=float, default=-200,
                        help="Earliest latency to check in cross correlation")
    parser.add_argument("--emeg-t-start", type=float, default=-200,
                        help="Start of the emeg evoked files relative to the start of the transform")

it would be great if you could amend it to
   parser.add_argument("--start-latency", type=float, default=-0.2,
                       help="Earliest latency to check in cross correlation")
   parser.add_argument("--emeg-t-start", type=float, default=-0.2,
                       help="Start of the emeg evoked files relative to the start of the transform")

Note that you will likely have to change some other functions to also work in milliseconds as a result. But hopefully fairly straightforward - and it should be fairly easy to check, you can run a gridsearch to confirm that you get the same results as before.


Two-stage training evaluation

Hi, I have trained two models (stage 1 and stage 2) according to your instruction. However, when I run this command
CUDA_VISIBLE_DEVICES=0 python tools/test.py /home/guxunjia/Desktop/VAD/projects/configs/VAD/VAD_base_stage_2.py /home/guxunjia/Desktop/VAD/work_dirs/VAD_base_stage_2/epoch_12.pth --launcher none --eval bbox --tmpdir tmp
The following error occur:
projects.mmdet3d_plugin WARNING!!!!, Only can be used for obtain inference speed!!!! load checkpoint from local path: /home/guxunjia/Desktop/VAD/work_dirs/VAD_base_stage_2/epoch_12.pth [>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>] 81/81, 5.0 task/s, elapsed: 16s, ETA:     0sTraceback (most recent call last):   File "tools/test.py", line 294, in <module>     main()   File "tools/test.py", line 274, in main     print(dataset.evaluate(outputs['bbox_results'], **eval_kwargs))   File "/home/guxunjia/Desktop/VAD/projects/mmdet3d_plugin/datasets/nuscenes_vad_dataset.py", line 1786, in evaluate     result_dict['ADE_'+cls] = all_metric_dict['ADE_'+cls] / all_metric_dict['cnt_ade_'+cls] ZeroDivisionError: float division by zero
I am using the mini dataset for training and eval. I checked the 'all_metric_dict' and it shows the following:
(Pdb) all_metric_dict {'gt_car': 701.0, 'gt_pedestrian': 659.0, 'cnt_ade_car': 0.0, 'cnt_ade_pedestrian': 2.0, 'cnt_fde_car': 0.0, 'cnt_fde_pedestrian': 0.0, 'hit_car': 0.0, 'hit_pedestrian': 0.0, 'fp_car': 46.0, 'fp_pedestrian': 0.0, 'ADE_car': 0.0, 'ADE_pedestrian': tensor(2.8956), 'FDE_car': 0.0, 'FDE_pedestrian': 0.0, 'MR_car': 0.0, 'MR_pedestrian': 0.0}
I am wondering if this is normal since I am using the mini dataset for training (just trying out), and it will affect the performance of the model and leads to these zeros values. When I am using your checkpoint model, everything is fine. I am wondering what is the correct file/procedure for evaluating the two-stage model?
Thank you so much!

I also tried training end-to-end on the mini dataset, the problem still occurs. So I am guessing that the small dataset is not enough to train at least some minimal results for exploring?

Yes, the mini dataset of nuScenes is very small, and I think the original epoch configs are insufficient for convergence. If you want to try on the mini dataset, I think at least a much larger epoch is required.

Hello, when i run the test.py ,I found that you encountered such result_dict['ADE_'+cls] = all_metric_dict['ADE_'+cls] / all_metric_dict['cnt_ade_'+cls] ZeroDivisionError: float division by zero, so did I. Have you solved it?

@yuyuyuyuyuty I think the above explanation is the reason.

@yuyuyuyuyuty我认为上述解释就是原因。
I have run the original 60epoch set in the paper, do I still have to increase the epoch?


Upgrade hadoop-aws version

Important Read
Issue #308
What is the purpose of the pull request
Fix an access denied issue on copying a file in S3.
Brief change log
Bumped the hadoop-aws version from 3.2.2 to 3.3.1
Verify this pull request
This pull request is a library version change. Project builds correctly and migration from Delta to Iceberg works using S3.

Thanks @arifazmidd! @sagarlakshmipathy can you do a quick check that everything still works for you from this branch?

Can confirm the sync works when I built and ran with the new jar from this PR.
Spark configs:
pyspark --packages org.apache.iceberg:iceberg-spark-runtime-3.2_2.12:1.4.1,org.apache.hadoop:hadoop-aws:3.2.4 --conf "spark.sql.extensions=org.apache.iceberg.spark.extensions.IcebergSparkSessionExtensions" --conf "spark.sql.catalog.spark_catalog=org.apache.iceberg.spark.SparkSessionCatalog" --conf "spark.hadoop.fs.s3a.aws.credentials.provider=com.amazonaws.auth.profile.ProfileCredentialsProvider"

Spark code:
from pyspark.sql.types import *
# initialize the bucket
table_name = "people"
bucket_name = <bucket_name>
base_path = f"s3a://{bucket_name}/delta-dataset-new-aws-version"

records = [
(1, 'John', 25, 'NYC', '2023-09-28 00:00:00'),
(2, 'Emily', 30, 'SFO', '2023-09-28 00:00:00'),
(3, 'Michael', 35, 'ORD', '2023-09-28 00:00:00'),
(4, 'Andrew', 40, 'NYC', '2023-10-28 00:00:00'),
(5, 'Bob', 28, 'SEA', '2023-09-23 00:00:00'),
(6, 'Charlie', 31, 'DFW', '2023-08-29 00:00:00')
]
schema = StructType([
   StructField("id", IntegerType(), True),
   StructField("name", StringType(), True),
   StructField("age", IntegerType(), True),
   StructField("city", StringType(), True),
   StructField("create_ts", StringType(), True)
])

df = spark.createDataFrame(records, schema)

(
 df.write
 .format("delta")
 .partitionBy("city")
 .save(f"{base_path}/{table_name}")
)

config.yaml:
sourceFormat: DELTA
targetFormats:
  - HUDI
  - ICEBERG
datasets:
  -
    tableBasePath: s3://<bucket_name>/delta-dataset-new-aws-version/people/
    tableName: people

Logs:
2024-01-19 11:04:14 INFO  org.apache.iceberg.RemoveSnapshots:124 - Expiring snapshots older than: 2024-01-12T19:04:14.789+00:00<PHONE_NUMBER>789)
2024-01-19 11:04:14 INFO  org.apache.iceberg.RemoveSnapshots:309 - Committed snapshot changes
2024-01-19 11:04:14 INFO  org.apache.iceberg.RemoveSnapshots:328 - Cleaning up expired files (local, incremental)
2024-01-19 11:04:16 INFO  org.apache.iceberg.hadoop.HadoopTableOperations:164 - Committed a new metadata file s3://<bcuket_name_redacted>/delta-dataset-new-aws-version/people/metadata/v2.metadata.json
2024-01-19 11:04:17 INFO  io.onetable.client.OneTableClient:128 - Sync is successful for the following formats HUDI,ICEBERG


Mainnet syncing issues

Znode locks not present during sync
"Orphan" category for Znode reward transactions
Out-of-order processing of events
Slower than zcoind sync speed

Fixed in https://github.com/zcoinofficial/zcoin/commit/29e3c9344e994efd98b8a796f57a452147f700c9


Sessions no longer persisting

I noticed that after commit 09b09ebad480940f2b49f96bbfbea0647783025e (v5.1.39) my sessions were no longer persisting.
After investigation, I discovered that this is because a $minutes property (inactivity before session expiry) was added to the DatabaseSessionHandler class without a default value, which, when $this->destroy($sessionId); was added, led to automatic expiry of a session with no activity.
I am happy to fix this issue in a PR, but wondering what would be the preferred solution:
1. Only expire a session if $minutes is not null
2. Add a default $minutes value
Thoughts?

We intentionally expired all existing sessions.

Minutes must be non-null for security, going foward.

Add a default $minutes value

Sounds good to me. :+1:


Pugin does not resize in the dock window

Introduced in 0.8.0

The plugin does resize but is just sluggish to do so. The user must hold and drag only releasing the left click once the resize has occurred.


feat: Carousel component

Closes #
Checklist

[ ] I didn't over-scope my PR
[ ] My PR title matches the commit convention
[ ] I did not include breaking changes
[ ] I included unit tests that cover my changes
[ ] I added/updated the documentation about my changes
[ ] I made my own code-review before requesting one

Description of the changes

Technical highlight/advice

:tada: This PR is included in version 1.0.0-alpha.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-beta.113 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Smaller metadata - to master

Ingest-jobs on integration stack hit the 2K metadata limit on user-defined metadata.  The PR eliminates values of some metadata dictionary keys.  The keys are left as SPDB will not load correctly without them.

Deleting this PR and branch it has been overtaken by more recent changes to ingest-jobs.


Modify mod_static, support mime type.

Codecov Report

Merging #197 into develop will increase coverage by 0.02%.
The diff coverage is 74.46%.

@@             Coverage Diff             @@
##           develop     #197      +/-   ##
===========================================
+ Coverage     52.9%   52.93%   +0.02%
===========================================
  Files          217      218       +1
  Lines        21630    21671      +41
===========================================
+ Hits         11444    11471      +27
- Misses        8889     8897       +8
- Partials      1297     1303       +6

Impacted Files
Coverage Δ

bfe_modules/mod_static/conf_mod_static.go
61.11% <100%> (+4.86%)
:arrow_up:

bfe_modules/mod_static/mod_static.go
60.24% <72.41%> (+2.27%)
:arrow_up:

bfe_modules/mod_static/mime_type.go
75% <75%> (ø)

bfe_http2/server.go
71.34% <0%> (-0.33%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 29f6480...814f01d. Read the comment docs.


Is test_fits.py::test_out_filename meant to utilise tmp_path?

Describe the Bug
The test https://github.com/glass-dev/glass/blob/a101ab48d8e9fcc6be5dc3fdc491afe971beda4e/tests/test_fits.py#L93-L99 has tmp_path fixture but isn't using it. Is this intentional or an oversight?
To Reproduce
import glass
...

Expected Behaviour
No response
Actual Behaviour
No response
Version In Use
2024.2.dev38+gd9ced9d.d20240918
Additional Context
No response

Definitely an oversight, and probably how we ended up with *.Fits (with wrong case) in .gitignore. Can we fix the test, the filename, and .gitignore? :)

Of course!


???

open data Unit | unit
def Func => Unit -> Unit
def f {x : Unit} : Func => \y => x
def ff => f unit

In file test.aya:4:10 ->

  2 | def Func =>  Unit -> Unit
  3 | def f {x : Unit} : Func => \y => x
  4 | def ff => f unit
                ^----^

Error: Unable to apply the expression
         f unit
       because the type of what you applied is not a Pi type, but instead:
         Func
         (Normalized: Unit -> Unit)

???

In file test.aya:4:12 ->

  2 | def Func => Unit -> Unit
  3 | def f {x : Unit} : Func => \y => x
  4 | def ff => f unit
                  ^--^

Error: Unsolved meta x'
       in `f'
       in `f unit'

Fixed, oh yes


[top/dv] i2c_host_tx_rx_test

Integrate #14778 with i2c_host_tx_rx test.
Also make some minor clean-up to the i2c_agent.
#14778 will need to be merged before this one can go forward.
This PR is now WIP from the chip_if update.
It works, but there is some clean-up that has to be done inside chip_if

A number of comments for you :)
@matute can you have another look? There has been a lot of changes as I've finally got things working with the new chip_if.
I also removed a commit that has already been merged.

@jdonjdon @weicaiyang can you guys have another look as well?


Use apollo-cli to typegen client-side GraphQL qieries

They want to remove codegen completely (https://github.com/apollographql/apollo-tooling/issues/2614Z) in replacement of using http://graphql-code-generator.com. I have a working copy of this that I am pushing to PR using graphql-code-generator.


Question about functionnality

Hello!
i have tried jexcel for a couple hours and so far i love it
i'm doing a BOM (bill of material) web application for my company
working with it, i realise that i could not find some functionnality
could you tell me if i missed them?
-delete row
-prevent user from adding row and column
-responsive column width
i think those are the functionnality i need for the moment
can you help me?
thank you and have a nice day

also
is there a scroll functionnality?
like you limit with a max height and beyond that height the overflow stay hidden?
keeping the header frozen on top could be nice too

To prevent the adding row and column from the context menu, i have commented those line in jquery.jexcel.js
but i can still add a column when i TAB from the last column

Get the most recent updates, and add these properties in your configuration.
allowInsertRow:false,
allowInsertColumn:false,
To delete records right click on the record and then delete the full row.
About the overflow, it is in progress and should be released soon.
Responsive resizing won't be available soon, sorry ;(
Regards

thank you for your answer
i can live with the responsive for now
for the overflow, in the past i worked with DataTable @https://datatables.net/examples/index
i am "getting inspired" by how they do the scrollable part
i would recommand you to check it out (i am not developping that library)
for the moment i use jexcel to create 2 tables, one that i hide the tbody and the other i hide the thead
its as guetto as it can be XD but it work for now
loocking forward for that excellent plugin


feat: add weth subparser

This PR adds WETH parser to parse deposit() and withdraw(wad)
close #389

Thanks @Soptq this looks great! It looks like some tests are failing CI but otherwise this looks like exactly what we need
I will be mostly away from my computer for the next couple of days. I'll be back on Monday and will be to to take a closer look, test it locally and get it merged.

Thanks for adding test @Soptq.
It appears that the WETH withdrawal is being parsed by the Yearn parser, instead of the new one you created - we'll need to address that.

Fixed, it turns out to be a bug in the yearn parser, allowing the yearn parser to capture transactions even if they are not yearn-related.

@Soptq @0xApotheosis
I was just testing this PR locally and noticed some weth transactions unrelated to yearn are being processed by the yearn parser.
For example this TX:
https://etherscan.io/tx/0x29ed7ad51bbc568172d3d97bc79738a921e7dc669abbf15ab5198d97cc74614b
Gives:
{
  address: '0xE7F92E3d5FDe63C90A917e25854826873497ef3D',
  blockHash: '0x048e94784ee6376f08ce9e86e62ee304f3dfbb05b502f415ebc3ac090f939a1a',
  blockHeight: 14611926,
  blockTime:<PHONE_NUMBER>,
  caip2: 'eip155:1',
  chain: 'ethereum',
  confirmations: 42907,
  fee: { caip19: 'eip155:1/slip44:60', value: '1087028000000000' },
  status: 'confirmed',
  tradeDetails: undefined,
  transfers: [
    {
      caip19: 'eip155:1/slip44:60',
      from: '0xE7F92E3d5FDe63C90A917e25854826873497ef3D',
      to: '0xC02aaA39b223FE8D0A0e5C4F27eAD9083C756Cc2',
      type: 'send',
      value: '3264000000000000'
    }
  ],
  txid: '0x29ed7ad51bbc568172d3d97bc79738a921e7dc669abbf15ab5198d97cc74614b',
  data: { method: 'deposit', parser: 'yearn' }
}

Parser should be weth in this case because the tx has nothing to do with yearn and should be going through the weth parser.

https://etherscan.io/tx/0x29ed7ad51bbc568172d3d97bc79738a921e7dc669abbf15ab5198d97cc74614b

Hi @elmutt , I believe the transaction you tested is related to WETH. This transaction interacts with contract 0xc02aaa39b223fe8d0a0e5c4f27ead9083c756cc2, which is the contract address of WETH. In other words, this transaction is indeed calling the deposit function of the WETH contract instead of yearn.
I think yearn probably proxy some WETH deposit/withdraw related operations to the WETH contract.

Hi @elmutt , It seems I can't reproduce this problem. I added your provided transaction to the test:

https://github.com/shapeshift/unchained/blob/87c1461465c45481224024b7680a14ec588b59b9/node/packages/client/src/ethereum/parser/__tests__/mockData/wethDeposit2.ts#L1-L35
https://github.com/shapeshift/unchained/blob/87c1461465c45481224024b7680a14ec588b59b9/node/packages/client/src/ethereum/parser/__tests__/ethereum.test.ts#L1318-L1372
and the test succeeded without problems as CI suggested.

@0xApotheosis Could you be able to reproduce this problem?

@Soptq im looking into it a bit more now will let you know what I find

Hi @elmutt , It seems I can't reproduce this problem. I added your provided transaction to the test:

https://github.com/shapeshift/unchained/blob/87c1461465c45481224024b7680a14ec588b59b9/node/packages/client/src/ethereum/parser/__tests__/mockData/wethDeposit2.ts#L1-L35

https://github.com/shapeshift/unchained/blob/87c1461465c45481224024b7680a14ec588b59b9/node/packages/client/src/ethereum/parser/__tests__/ethereum.test.ts#L1318-L1372
and the test succeeded without problems as CI suggested.

@0xApotheosis Could you be able to reproduce this problem?

@Soptq I apologize, I was testing the wrong branch. On the correct branch the parser is correctly appearing as "weth"
I think I did find a small issue tho:

the "from" and "to" addresses on the weth transfers appear reversed. For example a weth "deposit" is creating new "weth" tokens for the user so the "to" address should be the users address and the "from" should be the contract address. (It looks reversed for withdraw as well, withdraw is a weth transfer from the user to the contract)

Otherwise this PR looks great and will get it merged & bounty paid after that last issue is fixed

@elmutt Done! Thanks for your suggestions ;)

@Soptq Thanks for the the great work! Will get bounty paid out asap.


Write tests for all representations and frameworks

Loading a dataset into the framework's data loader and running a dummy for loop should be enough

closed with 6a473fdad7f71fa2f9b39e7e0110afd1d9ae3518


File upload not working with unicode characters

Calling slack.files.upload('File 1.txt') will work, but slack.files.upload('File ∑.txt') gives the following error:

slacker.Error: no_file_data

I'm using Python 3

Just tried with cURL and it worked. This could be an issue with requests https://github.com/kennethreitz/requests/issues/2411

Thanks, it definitely looks like that's the case (though more accurately, Slack isn't properly handling an RFC standard that Requests implements). I'm taking it up with Slack.


style: 恢复指标列头对齐方式和数据单元格保持一致

👀 PR includes
🎨 Enhance

[ ] Code style optimization
[ ] Refactoring
[x] Change the UI
[ ] Improve the performance
[ ] Type optimization

🖼 Screenshot

Before
After

❌
✅

:tada: This PR is included in version @antv/s2-v1.12.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update to sistr_cmd v1.0.2

Add test with more results output (-MM)

This works great to me, thanks @peterk87


Cursor does not stay in position after exiting insert mode

Observed behavior: When I add text in insert mode (i.e. hit "i", type some text) and then hit Escape to get out of insert mode, the cursor goes to the top of the class (when editing a class in Typescript and the previous cursor position is inside the class definition) or the top of the document. When editing a JSON, Markdown, or plain text file, the cursor returns to the top of the document.
Expected behavior: The cursor stays at its current position after hitting "Escape".
Note that to reproduce this behavior, you have to insert some text. Just hitting "i" and then "Escape" doesn't do it.

That's not correct behavior. Try with empty init.vim

That fixes it, which suggests that it's something in my init.vim, copied below:
set runtimepath^=~/.vim runtimepath+=~/.vim/after
let &packpath = &runtimepath
source ~/.vim/vimrc

vimrc attached.
vimrc.txt

i'm not sure, but probably this block is culprit:
if has("autocmd")
	augroup MyResetCursor
	autocmd BufReadPost *
				\ if line("'\"") > 0 && line("'\"") <= line("$") |
				\   exe "normal! g`\"" |
				\ endif
	augroup END
endif

Yes, that seems to have been the culprit. I've had that in my .vimrc for a
long time before this started happening, though - what changed?
On Mon, Aug 24, 2020 at 12:55 PM Alexey Svetliakov<EMAIL_ADDRESS>wrote:

i'm not sure, but probably this block is culprit:
if has("autocmd")
augroup MyResetCursor
autocmd BufReadPost *
\ if line("'"") > 0 && line("'"") <= line("$") |
\   exe "normal! g`"" |
\ endif
augroup END
endif
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_asvetliakov_vscode-2Dneovim_issues_321-23issuecomment-2D679275982&d=DwMCaQ&c=jGUuvAdBXp_VqQ6t0yah2g&r=TTXL98RIPQNjLdDSgeT8-yL45AVQu-9EyYWqGxjCNjqPZing27cmWboo1qWURfVf&m=SexPTTLdiUIz8ANQP0VWRfVLI1qCmgFrsjxlZl4hcso&s=qV4Pan5SwDSTZvmJY_O_aVwKBTz23JDPObGyRWBqArc&e=,
or unsubscribe
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AMOAKPQTZLQB7EDPF75CWNDSCKSPHANCNFSM4QJXUDWQ&d=DwMCaQ&c=jGUuvAdBXp_VqQ6t0yah2g&r=TTXL98RIPQNjLdDSgeT8-yL45AVQu-9EyYWqGxjCNjqPZing27cmWboo1qWURfVf&m=SexPTTLdiUIz8ANQP0VWRfVLI1qCmgFrsjxlZl4hcso&s=LOCcNd1psEfjwdzBjVV2wYmflN2mT2pUZha20bgevEw&e=
.

Are you sure 100%? Because that seems neovim bug then - BufReadPost is triggered after calling nvim_buf_set_lines (which happens after you exit insert mode) which is wrong. You should report this to neovim repo, in meantime you can disable this (and few more things) for the extension with exists('g:vscode')

Yes, I'm 100% sure. It was fine as late as this past Friday.
On Mon, Aug 24, 2020 at 1:07 PM Alexey Svetliakov<EMAIL_ADDRESS>wrote:

Are you sure 100%? Because that seems neovim bug then - BufReadPost is
triggered after calling nvim_buf_set_lines (which happens after you exit
insert mode) which is wrong. You should report this to neovim repo, in
meantime you can disable this (and few more things) for the extension with
exists('g:vscode')
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_asvetliakov_vscode-2Dneovim_issues_321-23issuecomment-2D679282183&d=DwMCaQ&c=jGUuvAdBXp_VqQ6t0yah2g&r=TTXL98RIPQNjLdDSgeT8-yL45AVQu-9EyYWqGxjCNjqPZing27cmWboo1qWURfVf&m=gAYsOIuXb7PHcY_UyKvtEz8WBkyh-feywYS33xDPXNM&s=o2k4hOcdEWO3Avaw-bJ1nfEkNQclst_guJc68gMThMQ&e=,
or unsubscribe
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AMOAKPRYNR7SY6CALC6PYA3SCKT67ANCNFSM4QJXUDWQ&d=DwMCaQ&c=jGUuvAdBXp_VqQ6t0yah2g&r=TTXL98RIPQNjLdDSgeT8-yL45AVQu-9EyYWqGxjCNjqPZing27cmWboo1qWURfVf&m=gAYsOIuXb7PHcY_UyKvtEz8WBkyh-feywYS33xDPXNM&s=2rwPEdRw7OrRleEBcrDMrJIug2iIltCZflRNsyYfkRo&e=
.

You mean you have been using this extension and it didn't happen previously? Ah, then it's related to . repeat operator work released these weekends - after exiting the insert mode, the extension creates empty buffer with window, replay changes here and closes it. That's why BufReadPost is being fired and your cursor is messed.
So that's not neovim problem then. Anyway, this block doesn't make sense with vscode, so feel free to wrap it with check above.

Having the same issue from around the same time.

@ashchan Investigate your init.vim for any cursor setting/resetting commands. Nothing i can do here and they don't make sense anyway - neovim sees any opened file as new scratch file. Any command restoring state won't work and harm.
Closing the issue since it's the configuration issue

@asvetliakov Thanks. Just checked my vimrc and found the same configuration.


support for cancel feature for clearing text box ???

Are you reporting a bug?
.It is not a bug its a suggestion.

Please create a Codepen that demonstrates your issue. You can start from the Basic example.

Provide the steps to reproduce the issue, e.g.:

Focus on the input field
Type c, and wait for suggestions to appear
Press Escape

Observed behaviour: Suggestions stay open
Expected behaviour: Suggestions should be closed

Are you making a feature request?

Please describe your use case from user journey point of view, e.g.:

In my application, when user highlights suggestions (using the mouse or keyboard), I'd like to display additional information about the highlighted suggestion alongside the Autosuggest.

If you have ideas how to extend the Autosuggest API to support your new feature, please share!

If you know any examples online that already implement such functionality, please share a link.

Is there a feature to clear the text with a cross button inside the text box ??

please update ??

Clear button is not part of the Autosuggest.
Here is one example for implementation.

This below example isn't complete as I've removed most of the Autosuggest functions, but I got this functionality by displaying a clear button when this.state.value is >= 1, then setting value to empty string in state when the clear button is pressed. Fairly simple to implement.
class Search extends Component {
  constructor(props) {
    super(props);

    this.handleClearClick = this.handleClearClick.bind(this);

    this.state = {
      value: '',
      suggestions: [],
    };
  }

  handleClearClick() {
    this.setState({
      value: '',
    });
  }

  render() {
    const { value, suggestions } = this.state;

    const inputProps = {
      placeholder: 'search',
      value,
      onChange: this.onChange,
    };

    let clearButton;
    if (value.length >= 1) {
      clearButton = (
        <button
          className="close-icon"
          onClick={this.handleClearClick}
        />
      );
    }

  return (

        <Autosuggest
          id="search-box"
          suggestions={suggestions}
          onSuggestionsFetchRequested={this.onSuggestionsFetchRequested}
          onSuggestionsClearRequested={this.onSuggestionsClearRequested}
          getSuggestionValue={getSuggestionValue}
          renderSuggestion={renderSuggestion}
          inputProps={inputProps}
        />

          {clearButton}

    );
  }
}


Special Attributes on Fenced Code Blocks

Parsedown Extra supports fenced code blocks and special attributes on titles, but sadly not the combination of both (which is helpful for code highlighters, for example).
I'm not sure if you're planning to support the whole Markdown Extra spec, but are you still considering to support a wider range of attributes?

Ok – I guess I found something about this in an extension – working this out.

Ok – I didn't find any docs on this, but writing
~~~~~~~~~ html
Link Text
~~~~~~~~~

outputs

        <a href="#anchor">Link Text</a>

that is, prepending the string with language-and adding it as a class.
That's not Markdown Extra spec but a safe implementation of the technique – closing thread.

Just to clarify, this behaviour is from Parsedown itself adhering to the way GitHub implements GFM.

Ah right! I thought something like that while writing the issue comment. I guess that's simply how these edge cases evolve.


Auto-detect and configure minimum SSL version available.

Running some test with a SSL3 server, I found that even by defining the MBEDTLS_SSL_PROTO_SSL3 macro, the library doesn't detect automatically this version as minimum available. I suspect that  ssl_tls.c lines 7286 and 7287, that is
conf->min_major_ver = MBEDTLS_SSL_MAJOR_VERSION_3;
conf->min_minor_ver = MBEDTLS_SSL_MINOR_VERSION_1; /* TLS 1.0 */

should be changed to
conf->min_major_ver = MBEDTLS_SSL_MIN_MAJOR_VERSION;
conf->min_minor_ver = MBEDTLS_SSL_MIN_MINOR_VERSION;

Am I correct? Or should one explicitly call mbedtls_ssl_conf_min_version()?

ARM Internal Ref: IOTSSL-1072


deny list based filtering

Text to be checked/linted is added to a lintreq. To determine whether text should be linted I check if it corresponds to an allowed highlight group using tree-sitter queries, a allow list based system:
For any tree-sitter subtree (parser:for_each_tree)
     iter over highlights (hl_query:iter_captures)
         add to lintreq if allowed hl_group

Some languages, such as latex, need text linted that is not highlighted. For this we need a deny list style operation.
change
Configuration must be language dependent
Instead of always running in allow list mode the event handler on_event should change to deny list mode depending on config
language dependent config
options:

switch based on parser lang
switch based on buffer lang (an autocommand?)
switch based on file extension (see 25ec193a40ff0a0729aadf6a7c9491f5be3c8e5c)

deny list style on_event
split lintreq builder into allow builder and deny builder. Both still build into a lintreq

replaced with treesitter query based linting doing away with hl groups entirely


Background images do not work at all.

I cannot see any background images in the style manager and if I set one it does not show up in the canvas.

Background images are not specific to this plugin, maybe you should check GrapeJS docs for this. Or is there something I am missing? Could you please elaborate on what you are trying to accomplish with Bootstrap 4 and GrapesJS with background images?

I'm closing this issue, if you think that the background error if caused by this plugin feel free to reopen it again.


Is clangd supposed to work on pure C projects?

This is not clear to me based on the website and pages for IDE plugins. The reason I am asking is that I cannot get it to work with several C projects I have tried. If it should work I will open a new issue with the problem.

Yes it works fine with C projects.

Thank you, making new issue then.

Actually, this could be made more obvious in the documentation. If a maintainer thinks this is not needed then feel free to close again.


Implement a splash screen

The application should display a splash screen like Visual Studio does on Windows 10.

https://imgur.com/a/4cyRKpX

No need for a splash screen when it loads fast.

I'm not sure a splash screen is needed but I think the about window should get some love and maybe this could be used for that?

No need for a splash screen when it loads fast.

On my end it takes around 6 seconds to load everything and I have a decent computer and SSD.
It's just a suggestion but I'd rather look at a pretty splash screen than have the VS Code window load up and have to wait for the code view, extensions and other things to fully load up before I can even start using the program.

/feature-request


test(cdc): Add defaultvalue integration test

What problem does this PR solve?
Issue Number: close #4223
What is changed and how it works?
Check List
Tests

Integration test

Related changes

Need to cherry-pick to the release branch

Release note
`None`.

/run-integration-test
/run-kafka-integration-test

/run-integration-test
/run-kafka-integration-test
/run-dm-compatibility-test
/run-dm-integration-test

Codecov Report

Merging #4224 (04c9445) into master (08da001) will decrease coverage by 0.0019%.
The diff coverage is 51.4480%.

Flag
Coverage Δ

cdc
58.6709% <56.8994%> (+0.0261%)
:arrow_up:

dm
52.1704% <40.5451%> (-0.0841%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #4224        +/-   ##
================================================
- Coverage   55.1722%   55.1703%   -0.0020%
================================================
  Files           485        491         +6
  Lines         59829      60915      +1086
================================================
+ Hits          33009      33607       +598
- Misses        23484      23891       +407
- Partials       3336       3417        +81

/run-integration-test
/run-kafka-integration-test
/run-dm-compatibility-test
/run-dm-integration-test

/run-kafka-integration-test

/run-integration-test
/run-kafka-integration-test
/run-dm-compatibility-test
/run-dm-integration-test

/run-integration-test
/run-kafka-integration-test
/run-dm-compatibility-test
/run-dm-integration-test

/merge

/run-integration-test
/run-kafka-integration-test
/run-dm-compatibility-test
/run-dm-integration-test


Update the 3.0.0 changelog

We want to further clarify the event tracking changes in the 3.0 release. I'm updating the master branch but I won't try to update the 3.0.0 tag.
Preview link

Coverage remained the same at 99.949% when pulling 9afb0b36a1d7221cd8cb05d847abd54b91d1f69a on nikhil/update-3.0.0-changelog into cdc25993f2b77e02171d7638cca84a2b691298f8 on master.


Readded Stinson filter

This looks great :)

For some reason, this look a bit different on my system than on yours

I retested it and notice that. I do apologize for not noticing it sooner. I have added another commit, and ran the test. I have changed the settings and this one is a lot better.

I made similar comments on #251 -- there are a few PRs on this filter right now, just waiting on the first one to adjust their filter

Added the requested changes :)


Incorrect reference?  content definition reference is not in any base file:   api.signuporsigninsam

Technical profile is missing:

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

ID: a49122f8-0ba8-08f1-c8cf-52634dfd0d0e
Version Independent ID: ea718f8a-e004-47b5-2feb-1eade3835301
Content: Conditional Access technical profiles in custom policies - Azure AD B2C
Content Source: articles/active-directory-b2c/conditional-access-technical-profile.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

@RideMo Thank you for pointing this out . We will investigate and update the thread.

@RideMo  We are working on it in separate issue #62857 , please subscribe to this issue.
Closing it as duplicate.


Create wiki page about database model

https://github.com/ita-social-projects/WhatBackend/wiki/Database-ER-Diagram


trailing quote formatted differently on multiple Format calls

EXPECTED:
var x = foo() '

ACTUAL:
var x = foo()'

This will alternate between these two formats as you ctrl+k+d
Test Name: TestInvalidTrailingQuote
Test FullName: AnalysisTests.FormatterTests.TestInvalidTrailingQuote
Test Source: e:\DevDiv\Offcycle\xTVS\Open_Source\Nodejs_Main\Nodejs\Tests\AnalysisTests\FormatterTests.cs : line 324
Test Outcome: Failed
Test Duration: 0:00:00.0109026

Result Message: Assert.AreEqual failed. Expected:<var x = foo() '>. Actual:<var x = foo()'>. First and Second call to format had different results...
Result StackTrace:
at AnalysisTests.FormatterTests.TestCode(String code, String expected, FormattingOptions options) in e:\DevDiv\Offcycle\xTVS\Open_Source\Nodejs_Main\Nodejs\Tests\AnalysisTests\FormatterTests.cs:line 2043
at AnalysisTests.FormatterTests.TestInvalidTrailingQuote() in e:\DevDiv\Offcycle\xTVS\Open_Source\Nodejs_Main\Nodejs\Tests\AnalysisTests\FormatterTests.cs:line 325

[copied from codeplex https://nodejstools.codeplex.com/workitem/1877]

This is invalid code anyways, so I'm not super inclined to prioritize it. We should investigate if we find it's affecting a more mainline scenario.

This test (TestInvalidTrailingQuote) seems to be passing and I can't repo this behavior on NTVS Alpha 1.2, with both Full Intellisense and ES6 preview.
Please reopen if I interpreted this bug incorrectly.


is the Google services are Missing?  Iam getting this error while trying to login with my account.

Did you follow the instructions in step 4 - " Set up the Google Photos Library API"?
You need to register an application in the Firebase console, download your own google services configuration and copy it into the project.


3486 Kusto Server Icon in Recent Connections

Added logic to render server icon when provider is Kusto in asyncServerTreeRenderer > set

This PR fixes #

Coverage decreased (-0.005%) to 40.407% when pulling 3f90d501e7e62d60e46139f72180101299c8d124 on 3486-Kusto_Recent_Connections_Icon into 8b52e7200c4a69bab7d5ad7f7c35185477b2b35c on main.

Closing PR for now since priorities have shifted.


programs.fish: use buildPackages for building completions/translations

Motivation for this change
A nixos system couldn't be cross compiled with programs.fish enabled, because it was trying to use the target system's python/babelfish to build the completions/translations.
This patch makes it use the host system's python & babelfish.
Tested on x86_64-linux, targeting armv7l-linux, and also tested for x86_64-linux for x86_64-linux to make sure it didn't break anything. Although I don't normally use babelfish so I'm not 100% sure it's working as usual, but the shell seems to be initialized correctly and /etc/fish/setEnvironment.fish is populated normally.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Hello, both issues have been fixed. Thanks for the PR


fix: make modifier keys work

Resolves issue https://github.com/HakCat-Tech/RubberNugget/issues/35
Keys CTRL, ALT, SHIFT are modifier keys, and need to be passed to the keyboard.sendPress function

I looked into this issue, and realized older versions of the EspTinyUSB library have an issue where key modifiers aren't appended to the keycode array in hidkeyboard.cpp.
I forgot I circumvented this issue by modifying the library locally, but I've published the updated file here.  Could you take a look at this and tell me if it's a viable fix?

If so, we can just point the Docker image to our forked version of the EspTinyUSB library for now, and make a PR to the official repo after optimizing it :P

Whoops, notifications were going to wrong email address. Should be fixed now
A few questions for you:

Can you tell me a little more about the issue you're talking about?
Why do modifiers need to be added to the keycode array? Does this have something to do with the maximum number of keys that can be pressed simultaneously?

Hey Brandon! I got busy chasing down bugs in a different Nugget project.
A few months ago I ran into the same issue with modifiers not working so I looked into the source code of the EspTinyUSB library, and traced down the issue to the hidkeyboard.cpp file.  The standard NKRO seems to be 6 keys from most sources I could gather, and the keycode array defined here has 6 elements allocated to hold the keystrokes and modifiers.
Problem is tho, I don't think there was a variable keeping track of the index of modifiers or multiple keypresses and only the first index was being addressed. However, shift still worked since technically the shift modifier w/ a character counts as a single char?  I forget but can look into this more.  Let me know if that makes sense :P

Also I completely forgot abt this issue until I went to merge your PR and dug around in my local codebase for the EspTinyUSB library.  So I might be wrong abt some of the details

Hey! Sounds like we might be talking about a few different issues here. There's the six-max keypress thing, and then there's this issue I've seen where mod+key doesn't work, e.g. ctrl-s

Just tried it out...the Nugget doesn't seem to have any problems pressing six keys at a time. Maybe we should merge this because less broken > more broken


Set The Date manually after object is created

I have a form where I use the plugin for several fields. If I change the textbox value using javascript the plugin shows using "today" as current value and Im not able to find the way to work around this.
Is there any way I can some function so it triggers the "selectedDate" event with a custom value?

Hi bathan,
I've updated the js file to facilitate what you need. After you update the textbox value just call $('#myinput').dcalendarpicker('setDate');, where myinput is the id of your textbox.

Hey! Thanks a lot for this change! It does work but now I have a different problem. Im using latin american date format {format: 'dd/mm/yyyy'} . When I create the dcalendarpicker object I specify that format.
So the text input has a value of 03/08/2016 for today , and when I run your new function its changed to 8/3/2016
Apparently the problem is in value = new Date(that.calendar.prev().val()); inside setDate function . I believe maybe you should check date format before creating the Date object?
Sorry for the annoyance and thanks a lot again for taking time to look at this.

Hi, actually I've tried the format you're using and it's running fine. Can you post the snippet of your code where you set the date and call the function I made?

HI!
The code is built using a blade template in laravel, that is why its mixed with php code.
$('#{{ "st_".strtolower($stage_name)."_starts_at" }}').focus(function(e) {
                    if($(this).val() == '' ) {
                        var previous_stage = $(this).data('previous-stage-name');
                        if (previous_stage) {
                            var prev_finish_date = $('#st_' + previous_stage + '_ends_at').val();
                            if (prev_finish_date) {

                                $(this).val(prev_finish_date);
                                $(this).dcalendarpicker('setDate');
                            }
                        }
                    }
                });```

The fields are date ranges for a process different stages. Each stage has a start and end date, and the idea here is to "suggest" a start date that will be the value of the previos stage end date. So when the user focuses on the field, it checks if it has a `data-previous-step-name` and gets the value from that input box.

Thanks!

Hi,
There shouldn't be any problem with the format as I always call the format function.
If you initially defined the format of your date input like: $('#{{ "st_".strtolower($stage_name)."_starts_at" }}').dcalendarpicker({format: 'dd/mm/yyyy'}), there shouldn't be any problem with the formatting.
And I assume your date input is static and not dynamically added.

Im initializing the plugin like this :
$('#{{ "st_".strtolower($stage_name)."starts_at"
}}').dcalendarpicker({format: 'dd/mm/yyyy'});
$('#{{ "st".strtolower($stage_name)."_ends_at"
}}').dcalendarpicker({format: 'dd/mm/yyyy'});
Ill look into it tomorrow. Thanks!!
On Fri, Aug 5, 2016 at 12:02 AM, Dionlee Uy<EMAIL_ADDRESS>wrote:

Hi,
There shouldn't be any problem with the format as I always call the format
function.
If you initially defined the format of your date input like: $('#{{
"st_".strtolower($stage_name)."_starts_at" }}').dcalendarpicker({format:
'dd/mm/yyyy'}), there shouldn't be any problem with the formatting.
And I assume your date input is static and not dynamically added.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dmuy/DCalendar/issues/1#issuecomment-237743827, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAvPW8aiiq5EpN77f11cRnsBbg58Mww5ks5qcqe-gaJpZM4Ja7rz
.

What I was able to debug is that in the setDate function :
setDate : function() {
var that = this,
value = new Date(that.calendar.prev().val());
if the date is for example today : 05/08/2016 , witch according to the
format is August 5 2016, it gets converted to May 8 2016. Attaching some
screenshots of the Chrome Developer Tools
Value of the text input : 05/08/2016
[image: Inline image 1]
Value of the value variable witch was converted to May 08 2016 instead.
[image: Inline image 2]
Hope this helps!
On Fri, Aug 5, 2016 at 12:10 AM, Matias Banzas<EMAIL_ADDRESS>wrote:

Im initializing the plugin like this :
$('#{{ "st_".strtolower($stage_name)."starts_at" }}').dcalendarpicker({format: 'dd/mm/yyyy'});
$('#{{ "st".strtolower($stage_name)."_ends_at" }}').dcalendarpicker({format: 'dd/mm/yyyy'});
Ill look into it tomorrow. Thanks!!
On Fri, Aug 5, 2016 at 12:02 AM, Dionlee Uy<EMAIL_ADDRESS>wrote:

Hi,
There shouldn't be any problem with the format as I always call the
format function.
If you initially defined the format of your date input like: $('#{{
"st_".strtolower($stage_name)."_starts_at" }}').dcalendarpicker({format:
'dd/mm/yyyy'}), there shouldn't be any problem with the formatting.
And I assume your date input is static and not dynamically added.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dmuy/DCalendar/issues/1#issuecomment-237743827, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAvPW8aiiq5EpN77f11cRnsBbg58Mww5ks5qcqe-gaJpZM4Ja7rz
.

Oh, right. That's the problem, initializing new Date object using different format. I'll look into this to find a solution. Thanks :)

Hi @bathan, I've updated the js file. I added a handler for the parsing of date value. Hope this helps. :)

Yay! It works like a charm!! Thanks a lot!!!
On Fri, Aug 5, 2016 at 9:12 AM, Dionlee Uy<EMAIL_ADDRESS>wrote:

Hi @bathan https://github.com/bathan, I've updated the js file. I added
a handler for the parsing of date value. Hope this helps. :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dmuy/DCalendar/issues/1#issuecomment-237834976, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAvPW1Hn6Kg9ZnHI6OccfGmtm-KqqONHks5qcyi4gaJpZM4Ja7rz
.


can't find margin field and borderRadius field.

I want to remove the margins that are around the bottom nav bar.

Hello,
We have added this feature. You can check in our latest version.


shadow

Hello, I have noted that the shadow is more hight to border and when I align for example in a Row a Button, I have to work for align.

Hi I have verificated Dopdownserch it have shodow inside on the edge,

I'm went to see your xample code and your example app also has the same error:

Hi I tryed but I dont found solution, can You post an simple code for Example?

Sorry my error I have put inside In to SizedBox.


Reduce multiplicities in .NET scans

ODC groups libraries by file content (actually by hashes). This works well in Java world.
This is unfortunately not the case in the .NET world. When I resolve libraries using Nuget and perform a scan on it, I usually get many files for one library:

The nupkg file.
DLLs for multiple .NET versions in the nuget file
The same DLLs extracted (Easiest to get rid of, and ODC handles them well, because they are duplicates.)

My goal is to get rid of multiplied items in the reports. I can perform some preprocessing or modify the ODC code. What is the best approach for that? My preferred idea how to implement it:
Don't treat nupkg as a standard ZIP. Files in nupkg archive would be analyzed, but the evidence would be added to the nupkg bundle. This does not skip the extracted DLLs (the #3), but they might be addressed in multiple ways, e.g. by blacklisting hashes of files (maybe just DLLs) found in a nupkg archive or by removing the lib directory before running ODC.
I was also thinking about some special mode that treats a directory as a single item (with some Merkle-tree-based hashing), which should achieve similar results as grouping by nupkg, but this approach would be more complex and more general. While the increased complexity is rather obvious (not only for the Merkle-tree hashing, but mainly for deciding which directories to group), I am not sure if it has any real benefits.
Should I start implementing the nupkg grouping?
Original thread: https://groups.google.com/forum/#!topic/dependency-check/bOsmmCmKfnM

Sorry for the delayed response on this. Absolutely the dependencies should be combined. This is similar to an issue we have with the ruby analyzers - see #477. I think a similiar approach could work for both (see comments in the other issue on how we might combine these). However, if after looking at the code you think there is a better place to combine the analyzers let me know. If you have time to work on a PR I would truly appreciate it.
Best Regards,
--Jeremy

Hi Jeremy,
the #477 approach looks to be essentially the same to the mine, just with mode detailed.
I'd like to work on it, but I won't probably do it this week.
Regards,
Vít Šesták 'v6ak'

I guess Dependency objects don't have a concept of a parent dependency? Trying to figure out the most elegant way to handle this, and I'm not sure the interior analyzers that "matter" (AssemblyAnalyzer, for example) have a way of "knowing" that this is actually within an extracted nupkg to add the evidence to that.

Other analyzers set the packagePath and then the dependency bundling
analyzer (?) Combines them.  Not in front of code right now or i could
point to the exact lines.  Problem is the assembly analyzer likely cant set
the package path.
Jeremy
On Nov 12, 2016 7:24 PM, "colezlaw"<EMAIL_ADDRESS>wrote:

I guess Dependency objects don't have a concept of a parent dependency?
Trying to figure out the most elegant way to handle this, and I'm not sure
the interior analyzers that "matter" (AssemblyAnalyzer, for example) have a
way of "knowing" that this is actually within an extracted nupkg to add the
evidence to that.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jeremylong/DependencyCheck/issues/453#issuecomment-260157865,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0qwqFdmUVrLW27bPGz2SbfdbD-7-eAks5q9liygaJpZM4Hiooo
.

Ah - it probably could, since it just sees a DLL - I just didn't know about
setPackagePath(dependency). I'm trying to decipher what the
DependencyBundlingAnalyzer does - maybe that will sort everything out for
me.
On Sat, Nov 12, 2016 at 8:57 PM Jeremy Long<EMAIL_ADDRESS>wrote:

Other analyzers set the packagePath and then the dependency bundling
analyzer (?) Combines them. Not in front of code right now or i could
point to the exact lines. Problem is the assembly analyzer likely cant set
the package path.
Jeremy
On Nov 12, 2016 7:24 PM, "colezlaw"<EMAIL_ADDRESS>wrote:

I guess Dependency objects don't have a concept of a parent dependency?
Trying to figure out the most elegant way to handle this, and I'm not
sure
the interior analyzers that "matter" (AssemblyAnalyzer, for example)
have a
way of "knowing" that this is actually within an extracted nupkg to add
the
evidence to that.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<
https://github.com/jeremylong/DependencyCheck/issues/453#issuecomment-260157865
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AA0qwqFdmUVrLW27bPGz2SbfdbD-7-eAks5q9liygaJpZM4Hiooo
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jeremylong/DependencyCheck/issues/453#issuecomment-260161383,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdyESAA0nvc_SKOgpmnIvusMOQdB5itks5q9m6cgaJpZM4Hiooo
.

I think the package path may be named incorrectly too. That came in as part
of a PR to solve this issue for Ruby and it was later extended for another
language (python?). Don't have the code in front of me again - I'll take a
look in the morning.
--Jeremy
On Mon, Nov 14, 2016 at 6:38 PM, colezlaw<EMAIL_ADDRESS>wrote:

Ah - it probably could, since it just sees a DLL - I just didn't know about
setPackagePath(dependency). I'm trying to decipher what the
DependencyBundlingAnalyzer does - maybe that will sort everything out for
me.
On Sat, Nov 12, 2016 at 8:57 PM Jeremy Long<EMAIL_ADDRESS>wrote:

Other analyzers set the packagePath and then the dependency bundling
analyzer (?) Combines them. Not in front of code right now or i could
point to the exact lines. Problem is the assembly analyzer likely cant
set
the package path.
Jeremy
On Nov 12, 2016 7:24 PM, "colezlaw"<EMAIL_ADDRESS>wrote:

I guess Dependency objects don't have a concept of a parent dependency?
Trying to figure out the most elegant way to handle this, and I'm not
sure
the interior analyzers that "matter" (AssemblyAnalyzer, for example)
have a
way of "knowing" that this is actually within an extracted nupkg to add
the
evidence to that.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<
https://github.com/jeremylong/DependencyCheck/issues/453#
issuecomment-260157865
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/
AA0qwqFdmUVrLW27bPGz2SbfdbD-7-eAks5q9liygaJpZM4Hiooo
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/jeremylong/DependencyCheck/issues/453#issuecomment-
260161383>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAdyESAA0nvc_
SKOgpmnIvusMOQdB5itks5q9m6cgaJpZM4Hiooo>

.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jeremylong/DependencyCheck/issues/453#issuecomment-260499856,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0qwlmalaeT_PMqFjhOByvF4UhXsBMiks5q-PETgaJpZM4Hiooo
.


Simple framework for gas benchmarking

Gas usage benchmarking is a recurring topic, it would be nice to have simple utility for writing such benchmarks. It should be very simple to use, and with very much single responsibility - checking gas cost of performed calls. Under the hood real vm shall be used, with some help of its test utils, but benches on they own should not perform any verification (the behavior of contract is to be tested in UTs, and MTs, possibly some integration tests).
As a bonus it would be nice if the framework would report results in some common format, like json - it would make it probably much easier to integrate it with tools like grafana in future.

Benchmarking is in general a specific / punctual task. I mean, you usually want to measure the performance of some specific, isolated functionality.
For measuring the overall cost of execute, instantiate, etc. as we want to do in #505, perhaps a good option is to do it in a conventional way. That is, a CPU benchmark, in the context of a unit test (or multi-test). Instead of a gas usage benchmark, in the context of an integration test.

@hashedone Is this really an ng contracts/Sylvia issue? This doesn't sound right

@de-husk implemented a framework, have a look: https://github.com/de-husk/cosm-orc