Ignore auto sized axes to avoid circular dependency

This is a retry of #1862. See also that PR.
Accessing parent size while computing RequireParentSizeToFit is a circular dependency. isComputingChildrenSizeDependencies was placed to fix at least stack overflow but there are observable issues for some edge cases (#1803 AutoSizeWithRotation / AutoSizeWithShear cases).
FillFlowContainer and SpriteText (new implementation) adds edges to the dependency graph and also makes a cycle.
In this PR, define: an axis is auto-sized (directly or indirectly) if and only

AutoSizeAxes is set for the axis or,
RelativeSizeAxes is set for the axis and parent is auto-sized for the axis.

Then, regard auto-sized axes sizes as zero.
I have to duplicate the whole chain of dependencies (LayoutRectangleBeforeParentAutoSize etc.) to actually avoid the dependency cycle. This is not good for eyes but I couldn't come up with any other ways.
Current implementation determines whether an axis was auto-sized by comparing to zero but this is not technically correct although I couldn't find issues in the real world. Another possible implementation is use of Infinity or NaN but I'm not sure about this is better or not.

Fix #1803's AutoSizeWithRotation2-3.
AutoSizeWithRotation1 and AutoSizeWithShear are not passing due to another invalidation issue.

RelativeSizeAxes is set for the axis and parent is auto-sized for the axis.
Why the new case of allowing autosize and relative size simultaneously?

Current implementation determines whether an axis was auto-sized by comparing to zero but this is not technically correct although I couldn't find issues in the real world.
Since the framework is meant to be usable by anyone and to always perform correctly, we need a better argument/validation than "couldn't find issues". We should be covering all use-cases, i.e. make the framework perform as expected across all valid API parameters.

@Tom94

Why the new case of allowing autosize and relative size simultaneously?

I don't. Maybe an example is needed. Consider this case where B is a child of A.
B = { RelativeSizeAxes = Axes.X };
A = { AutoSizeAxes = Axes.X, Children = new [] { B, new Box { ... } } };

For A, X axis is auto-sized directly. For B, X axis is auto-sized indirectly.

Naming is pretty arbitrary and I don't think name represents the function (able to understand what is going on without deeply looking into the code) even with comments.
This suggests either my approach is off or the underlying problem is inherently complex. I'm still not able to come up with simpler solution to it and if anyone have idea then I would appreciate.

Closing to investigate better solution.


Harvest Festival 0.5.25 mod translation into Brazilian Portuguese

Below is the translation of the mod Harvest Festival 0.5.25 (Brazilian Portuguese)
pt_BR.lang.txt

It should be sent as a pull request not like this.

Sorry.


Fix linting issues

Part of #42

[x] Add cargo fmt and cargo clippy to CI
[x] Fix fmt issues
[ ] Fix clippy issues
[x] Remove #![allow(dead_code)] and #![allow(unused_imports)] from lib.rs

@DaviRain-Su can you please review this.
While trying to address some of the clippy warnings (and running 1.42 locally to get the same as the CI), I realized the ics23.rs file was manually editted and doesn't match what comes from cargo protoc, which will cause issues in the future keeping them in sync. I tried regenerating it with the script and got all kinds of errors, even in std mode.
Otherwise the rest was just making clippy happy basically


Wrong taxon column in robinson_and_holton_2012 taxa.csv

taxon in taxa.csv doesn't match taxlables in summary.trees.

Ahh, I guess I went off the IDs in the lexibank dataset... fixed in https://github.com/D-PLACE/dplace-data/pull/270


Add DbtDocsGenerateOperator and extend DbtTestOperator

Extend DbtTestOperator to allow usage of --data (data test) and --schema (schema test)
Added new DbtDocsGenerateOperator

Thanks @ayobamshy!

Hi I know it's been a while, but just curious if it's possible to have a new version release with DbtDocsGenerateOperator included?


Load turnover maps for the spatially explicit mode

Implements turnover maps for the spatially explicit scenario, from which a per-location turnover rate is loaded.
To support this extra parameter, the Classical algorithm and non-skipping Gillespie algorithm have been merged (as both were doing almost the same already) such that the Gillespie algorithm is used in general, but when turnover is uniform the more optimised Classical algorithm is used instead.

@jrosindell @fernandeslucasd I am now ready to merge this branch into main - are there any more features you require beforehand (specific to turnover maps)?


Selected date is not selected again after coming back in the month

When a date is selected, one subsequently swipes two or more months in the future or in in the past, and then comes back to the original month, the originally selected date isn't selected anymore.

Fixed in 1.0.5


Reading single epochs from a fif file

I noticed that reading single epochs is significantly slower when the raw data is saved as a fif file vs. an edf file. Also, the reading time from fif files linearly increases with how far in the recording the epochs are, which is not the case with edf files.
I'm interested in getting this reading time as low as possible since I have to repeatedly load single epochs. I was wondering whether this difference between fif and edf is expected (maybe part of it is due to the lower precision of edf files).
Steps to reproduce
I created a gist that creates .fif, .edf and .hdf5 files out of the same fake raw data, and then times the loading of each individual epoch: https://gist.github.com/hubertjb/1b434688dba296d25eba8fab2866a50f
I included direct reading from an hdf5 file as a baseline.
Expected results
I was expecting fif and edf to yield similar results. Specifically, I was expecting the loading time to be more or less constant across the recording for fif.
Actual results
As shown below, reading from fif is slower than from edf, and reading times increase as the epochs get further from the beginning of the recording.

Mean loading times are
{'edf': 0.0028056514029409373,
'fif': 0.008060473086787204,
'hdf5': 0.0009250746053807876}
Additional information
Platform:      Linux-4.15.0-70-generic-x86_64-with-debian-buster-sid
Python:        3.7.6 (default, Jan  8 2020, 19:59:22)  [GCC 7.3.0]
Executable:    /storage/store/work/hjacobba/miniconda3/envs/braindecode/bin/python
CPU:           x86_64: 72 cores
Memory:        376.5 GB
mne:           0.20.dev0
numpy:         1.18.1 {blas=openblas, lapack=openblas}
scipy:         1.4.1
matplotlib:    3.1.3 {backend=module://ipykernel.pylab.backend_inline}
sklearn:       0.22.1
numba:         0.48.0
nibabel:       Not found
cupy:          Not found
pandas:        1.0.1
dipy:          Not found
mayavi:        Not found
pyvista:       Not found
vtk:           Not found
@agramfort @larsoner

I was expecting fif and edf to yield similar results. Specifically, I was expecting the loading time to be more or less constant across the recording for fif.

In FIF the raw data are stored in "tags" of (usually) 1 sec duration so in principle I don't see why it should be slower. I agree it's worth looking into. @hubertjb do you want to look into it? If not I can give it a shot

... actually from looking at the FIF code I already suspect a culprit. I'll open a WIP PR.


cannot get wordplate/extended-acf to render Gutenberg blocks

ACF Pro is very proud of the fact that you can jump right in and create Gutenberg blocks using field groups. I set it up in a wordplate project, using extended-acf, and following the ACF Pro blocks guide. I've double- and triple- checked it, but I can't get the blocks to render.
Here's what I have:
theme/functions.php
...
// Action Bar
function init_block_action_bar () {
  if (function_exists('acf_register_block_type')) {
    acf_register_block_type(array(
      'name' => 'action-bar',
      'title' => __('Action Bar'),
      'description' => __('Full width bar layout with title and CTA'),
      'render_template' => 'partials/blocks/action-bar/action-bar.php',
      'category' => 'layout',
      'icon' => 'align-wide',
      'mode' => 'edit',
      'keywords' => array('action bar', 'cta'),
      'post_types' => array('post', 'page'),
    ));
  }
}
add_action('acf/init', 'init_block_action_bar');
...

theme/partials/blocks/action-bar/action-bar.php
<?php
/**
 * Action Bar Block Template
 * @param    array $block The block settings and attributes
 * @param    string $content The block inner HTML (empty)
 * @param    bool $is_preview True druing AJAX preview
 * @param    (int|string) $post_id The post ID this block is saved to
 *
 * @package  newco
 * @since    newco 0.5
*/

$id = 'action-bar-' . $block['id'];
if (!empty($block['anchor'])) {
  $id = $block['anchor'];
}
$className = 'action-bar';
if (!empty($block['className'])) {
  $className .= ' ' . $block['className'];
}
if (!empty($block['align'])) {
  $className .= ' align' . $block['align'];
}

$bg_color = get_field('bg_color');
$bg_color_class = ' bgcolor--' . $bg_color;

$padding_classes = '';
if (get_field('add_padding')) {
  $padding = get_field('padding');
  $top_padding_class = $padding['top_padding'] == 0
    ? ''
    : ' spacer__layout--pt-' . $padding['top_padding'];
  $padding_classes .= $top_padding_class;
  $bottom_padding_class = $padding['bottom_padding'] == 0
    ? ''
    : ' spacer__layout--pb-' . $padding['bottom_padding'];
  $padding_classes .= $bottom_padding_class;
}

$title = get_field('title');
$cta = get_field('cta');
$target = $cta['target'] == '_blank'
  ? ' target="' . esc_attr($cta['target']) . '" rel="noreferrer"'
  : '';
?>

<section id="<?=$id;?>">
  <div class="container-fluid<?=$bg_color_class;?><?=$padding_classes;?>">

        <div class="<?=$className;?>">

            <?=$title;?>
            <a class="button button--secondary spacer__element--ml-48" href="<?=esc_url($cta['url']);?>"<?=$target;?>><?=$cta['title'];?>

In the CMS,  I added a Block: Action Bar field group with the corresponding inputs:

So far so good. Now if I edit a Page, and add a block, my Action Bar block appears as an option. Beautiful, right? Now I select Action Bar, the ACF fields appear and I can fill them out. So cool! Then I update the edit page and reload the rendered page and...sad trombone...no Gutenberg block. However, in the browser inspector, where the block should appear, the data from the block shows up as commented JSON, like so:

This is where I get stuck. I can't figure out how to get this data to render as the template file. I did a test in a vanilla WordPress ACF Pro instance and the blocks render as expected, easy-peasy. Is there something in wordplate/extended-acf that is preventing this from working? Is it a scoping issue? Not sure where to go from here.

It doesn't look like you've used wordplate/extended-acf. If you're using this library field group shouldn't be visible in WordPress dashboard.

Is there something in wordplate/extended-acf that is preventing this from working?

All I can say is that we've tested this library with Gutenberg blocks and we know that it works. I can't help you debug your application.

It doesn't look like you've used wordplate/extended-acf. If you're using this library field group shouldn't be visible in WordPress dashboard.

Is there something in wordplate/extended-acf that is preventing this from working?

All I can say is that we've tested this library with Gutenberg blocks and we know that it works. I can't help you debug your application.

@albeeboy here is an minimal example for creating a text block with extended-acf and ACF ...
<?php

declare(strict_types=1);

use WordPlate\Acf\Fields\Wysiwyg;
use WordPlate\Acf\Location;

acf_register_block_type([
    'name' => 'Text',
    'title' => __('Text'),
    'description' => __('A text block.'),
    'render_template' => get_theme_file_path('includes/acf/blocks/text/template.php'),
    'enqueue_style' => get_asset_file_uri('assets/blocks/text.css'),
    'post_types' => ['page'],
    'category' => 'common',
    'icon' => 'editor-textcolor',
    'align' => 'wide',
]);

register_extended_field_group([
    'title' => 'Text Block',
    'fields' => [
        Wysiwyg::make('Text')
            ->instructions('Add the text block content.')
            ->mediaUpload(false)
            ->toolbar('text')
            ->required(),
    ],
    'location' => [
        Location::if('block', 'acf/text')
    ],
]);

@albeeboy here is an minimal example for creating a text block with extended-acf and ACF ...
<?php

declare(strict_types=1);

use WordPlate\Acf\Fields\Wysiwyg;
use WordPlate\Acf\Location;

acf_register_block_type([
    'name' => 'Text',
    'title' => __('Text'),
    'description' => __('A text block.'),
    'render_template' => get_theme_file_path('includes/acf/blocks/text/template.php'),
    'enqueue_style' => get_asset_file_uri('assets/blocks/text.css'),
    'post_types' => ['page'],
    'category' => 'common',
    'icon' => 'editor-textcolor',
    'align' => 'wide',
]);

register_extended_field_group([
    'title' => 'Text Block',
    'fields' => [
        Wysiwyg::make('Text')
            ->instructions('Add the text block content.')
            ->mediaUpload(false)
            ->toolbar('text')
            ->required(),
    ],
    'location' => [
        Location::if('block', 'acf/text')
    ],
]);


added C++ support

Added C++ snippets from 30 Seconds of C++

I'll need to add an option to support c++ first. Then we can merge this, however the folder for c++ can't be named "c++" it should be "cpp". Also if there is a repo where you have all these snippets could you link to it here so others can help with migrating the snippets.

Ok I have changed the directory name to cpp
I will be add C++ snippets to this directory.
Also ping me when you will be adding  support for C++ , I will provide you with the logo :smiley:

@Bhupesh-V Hey man, I've started merging all PRs and I'm just about to merge C++ snippets, I don't however want to release them before we have at least 20-30 snippets. So the question is, do you have more snippets or is this all? 😄

@Bhupesh-V Hey man, I've started merging all PRs and I'm just about to merge C++ snippets, I don't however want to release them before we have at least 20-30 snippets. So the question is, do you have more snippets or is this all?

Hey :smile: sure , I have added more snippets(26 in total) also do I have to push a template for C++ Snippets in the templates/ directory ?
I will also ping you once the Logo is ready

@Bhupesh-V Hey, it would be great if you could also push the template, as for the logo there is no need for it. If you are taking the snippets from a separate repo do send me a link so I can backlink to the original repo in readme and on other places :D

@Bhupesh-V Also I see you are using ** notation to mark sections in the template, you should check other templates, you'll se they use #### for section names and ### for snippet title. We should keep these the same across the templates.


Does sol2 support assigning lua functions to function pointers?

Hi
I would like to assign lua functions to c++ function pointers.
vec3 (*script)(float, float, float, vec3) lua   = &lua_["Idle"]["animate"];

As I understand, the api currrently returns a decayed value from lua. I was wondering if the api is able to convert a sol function object to a c++ function and then assigned to a function pointer.
Thanks
Rebecca.

It's impossible, because raw pointer to function cannot have state other than its address. Use std::function instead

How do you convert a sol::function object to a std::function object then?

How do you convert a sol::function object to a std::function object then?

The following example is from https://sol2.readthedocs.io/en/latest/tutorial/all-the-things.html
	// get a function
	sol::function a_function = lua["a_function"];
	int value_is_100 = a_function();
	// convertible to std::function
	std::function<int()> a_std_function = a_function;
	int value_is_still_100 = a_std_function();
	c_assert(value_is_100 == 100);
	c_assert(value_is_still_100 == 100);


Update all dependencies to latest, especially ic-stable-structures and Boa

[ ] Use crazy script to update all candid files in Azle

This is currently blocked by the Finalize CDK Inteface and Finalize Syntax epics.

No longer blocked for me, just need to implement Vec

Actively working on this now


Attributes for components

Going forward, it would be useful to expose some mechanism of adding attributes to components (and other things like instances) to control code generation. For example, we can put an attribute to force generation of an FSM with a particular delay:
#[slow_fsm=5]
comp main<G:10>(...) {...}

The attribute can then be used by the lowering pass to generate the FSM of the specified delay (and circumvent the need for #273 for some time).

A quick question on attributes before we implement them should be about scoping of different attributes. For example, say I have a two-event component as follows:
#[slow_fsm=5]
comp main<G: 10, L: 9>(...) -> (...) {...}

which is currently not super realistic to create currently but nonetheless exists, and could become important in dynamic filament. A component level attribute like slow_fsm would not be able to distinguish between the two events and which one to apply the slow FSM to.
Even further we may have attributes we want at the port level, such as information about how bundles could be lowered at the top level. The simplest example of this is with the spiral DFTs, where the verilog emitted can have say, 4 input ports for a 16 point FFT where the first input represents the 0th, 4th, 8th, 12th input in each cycle. We have discussed before possibly having some form of lowering where a bundle like
A[4]: for<i> ['G+i*4, 'G+i*4+1] 32

would be lowered into a single port that is reused at different times for different behaviors. This behavior could be decided via attribute, where #[collapse_bundle] or something similar could decide whether a bundle is collapsed into one port or split into multiple.

Of course component level attributes could come first but I do think this is an idea that might be worth exploring in the future.

I think the approach you've taken---to implement the simplest thing first (component-level attributes)---is probably the right one. As we add more features to the language (real modules, real scoping rules), we can investigate more fine-grained attributes.


Must send back sentinel value when updating a SearchIndexerDataSourceConnection

Through testing, I found that we cannot send back a null or empty SearchIndexerDataSourceConnection.ConnectingString ("credentials"). Instead, you have to use an undocument sentinel value of "" (include brackets). I opened Azure/azure-rest-api-specs#9877 to make sure this gets documented, but it something we'll need to do for GA to support CreateOrUpdateDataSourceConnection properly.

The issue has been fixed. Closed the issue


Request a tag

Hi,
I hope you can tag the releases as a good habit can keep your project in good status.
I'm going to package it into Fedora & EPEL.
Thanks.

@cicku . The project has now some new board members (me included)  and we will try to take care of these stuff in the sfuture.
Did you manage to pack it into Fedora and EPel??

Why not 😃

why not what??

A new release tag has been created on github containing the release binaries
https://github.com/SoftEtherVPN/SoftEtherVPN/releases/tag/v4.24-9651-beta
This will hopefully be automated in the future


BLD: bump blaze

Bump blaze to a bug fix that fixes comparison of timestamps. In this bug, the server serialized timestamps incorrectly -- it appended tz to tz naive timestamps, which resulted in matching timestamps being missed.

Coverage remained the same at 87.464% when pulling 8faab75459c3c23f7c74250dc187143098882956 on bump-blaze into a006b4bbab1f4d92724bfa268b92f0a4f1868e54 on master.


Would like to know which version added the data index to data value of Boxplot tooltip

What problem does this feature solve?
Sorry to bother, I would like to ask for the change in the new version of echarts(don't know which version). I improved the echart version from v3.4 to v4.4 and I found a change in the new version but not found in the echart official website.
In the v4.4 the data of Boxplot tooltip would add the index as default.
Just like the data below:
data:[5,0,200,400,600,800,400]
But in v3.4 it is shown below:
data:[0,200,400,600,800,400]
5 is the array index of  the tooltip data
I want to know in whick version do you make the change and why to do that.Thanks a lot.
What does the proposed API look like?
I've tried on the offical website, it wiil also show the array index.

I came across the same issue. Echarts version 4.2.1
It always put dataIndex to the beginning of the data array.
The demo is here: https://jsfiddle.net/dmLc587n/1/


chore: bdd-nh-1585578474 to 0.0.1

chore: Promote bdd-nh-1585578474 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Writing empty .LAZ makes python crash

import pylas
las = pylas.create()
las.write('empty.las') # Ok
las.write('empty.laz') # crash

This was a bug in laz-perf that was fixed.


Strapi

Directly trying to finally get the E2E test to run inside the CI 😅

For these bits:
     config.entry.main = [workspaceRoot + `/${root}/.strapi/client/app.js`]
     config.output.path = workspaceRoot + `/${outputPath}/build`

you might be better off trying to utilise the existing paths and see if you can mutate them into what you want, but i think that would be a nice to have :)

@joshuaellis good one, thanks!


feat(htlc-coordinator): new htlc coordinator

Resolve #953
Depends on #582
Depends on #580
Depends on #978

Fixed most of the issues, just the openapi specs and types express remaining.

Codecov Report

Merging #976 (69c9b47) into main (e2cd07a) will decrease coverage by 0.00%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main     #976      +/-   ##
==========================================
- Coverage   71.76%   71.75%   -0.01%
==========================================
  Files         270      270
  Lines        9420     9422       +2
  Branches     1113     1114       +1
==========================================
+ Hits         6760     6761       +1
- Misses       2087     2088       +1
  Partials      573      573

Impacted Files
Coverage Δ

.../src/main/typescript/plugin-htlc-eth-besu-erc20.ts
92.40% <50.00%> (-1.11%)
: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 e2cd07a...69c9b47. Read the comment docs.

@jagpreetsinghsasan Which test file is the end to end one that does the entire tx?

extensions/cactus-plugin-htlc-coordinator/src/test/typescript/integration/plugin-htlc-coordinator/withdraw-counterparty-endpoint.test.ts

@jagpreetsinghsasan Got it, thank you! Are there assertions for both parties balances at the end of the test? I see this one at the end but not for the other party (maybe it's there, I'm not sure)
t.equal(responseFinalBalance.callOutput, "10", "balance of account is 10 OK");

@jagpreetsinghsasan Got it, thank you! Are there assertions for both parties balances at the end of the test? I see this one at the end but not for the other party (maybe it's there, I'm not sure)
t.equal(responseFinalBalance.callOutput, "10", "balance of account is 10 OK");

I forgot to add the other party assertion as well. I will add it and re-request for review then.

@petermetz
Fixed the DCO checks, the review comments.
Hope this time everything's right 😄


Command plugins

Hi,
The issue I've been facing is that of aggregating coverage data at a package level. There are several PRs for this, but in fact, existing APIs make it easy enough to do as a user -- almost.
I'd like to have the option to define my own subcommands -- in particular, here, "coverage packages" or something of the sort.
I have, in fact, written this tool, by copying the cmdline and summary modules from within coverage and modifying them. But it feels awkward.
This may require some serious refactoring of the cmdline code.

Can you say more about what your subcommand does? I'm happy to support tool building, though I'm not sure adding subcommands is necessary: why not your own command that does what you need?  Are there extra APIs you would need?

It's basically #356 applied to the report subcommand. So, it shares a lot of the configuration and command-line options with the existing report. I'll check again if I could have done it more elegantly than copying-and-modifying; it seemed the fastest way to do it, but it feels quite awkward.

Hello :) Sorry for reviving an old topic, but I searched and this is exactly what I'm looking for. Also sorry for the long post.
A bit of context:

I'm quite familiar with using CoveragePy but mostly from its wrapper pytest-cov. This is relevant for I had no sense of the architecture of coverage, I initially thought the report format was just an option (and linked sub-options) and not the whole command (see --cov-report doc)
I'm trying to build something that generates a markdown coverage report in a way that's suitable for posting as a GitHub PR. I'd guess that suggesting a Markdown command could make sense, but I was planning to embed the GitHub API part in it too, so as to simplify my usecase

So I found that CoveragePy supported Plugins, yay, but there was no "Reporter" plugin type. I thought maybe I could contribute a new type of plugin! But then looking a bit deeper, I tried to think about the API, compared to what we have now: Imagine we add a Markdown plugin that would simply do the same as JSON but in markdown format, and have it declared in .coveragerc, alongside with its specific options.

In terms of CLI, what would be the expect call for computing coverage and generating a markdown report. coverage markdown ? But that would be a whole new subcommand, which means plugins could add arbitrary subcommands (what @shaib suggested in this ticket). coverage custom markdown or something ? I'm thinking maybe coverage {xml|json|annotate|report} should be a shorthand for coverage result {xml|json|annotate|report} and coverage result would also accept any type of result that would be statically defined by plugins ? (I said result because reportwas already taken, but that might not be the best name).
In terms of Python API, we have Coverage().html_report(), Coverage().json_report(), Coverage().annotate(), Coverage().report() and I believe we could have Coverage().results(type, **kwargs) which would then be compatible with arbitrary type, be them official ones or plugin ones.

Two places in the documentation that tell me the idea that a generic results command would make sense:

In https://coverage.readthedocs.io/en/coverage-5.5/cmd.html#command-line-usage, the resultcommands stand out by all being phrased with "... with coverage results."
In https://coverage.readthedocs.io/en/coverage-5.5/api_coverage.html, all corresponding methods have "See report() for other arguments."

I see that plugins can already have arbitrary options through config file, so this shall give plenty of space for configuring a Reporter plugin. We could imagine that reporter plugins would be able to implement a method that returns an argparse object, though.
So what do you think. I'm ready to try and draft a PR if you're interested!

I want to make it possible for you to build the tools you are thinking about.  Reporting tools should be able to use the existing coverage API in a separate tool, not a plugin.  What would be bad about writing markdown as a standalone tool?

This would mean e.g. I wouldn't be able to use it from pytest-cov (in one go, as it's done today: pytest cov does both the coverage computation and the report generation) without pytest-cov being specifically aware about my tool, but that can be circumvented by having pytest-cov just generate the coverage info and then using my tool directly instead of pytest-cov. I guess it's possible too.

To be perfectly honest, I've never understood why pytest-cov needs to handle the reporting phase as well.  It seems to me to be jamming too much stuff into one tool.  The complex command-line syntax for the reporting options is a clue that it's in the wrong place.  People should use tools like make, tox, or even shell scripts if they want to do a number of things with one command.

There's a certain serenity in being able to use pytest directly in projects without having to use tools over it, both for the sake of having a standardized experience across projects, and being able to easily add pytest CLI args. Makefile is not great with adding arbitrary arguments, a shell script can be better but is not always built this way. There's always the PYTEST_ADDOPTS envvar. I do understand your doubts, but IMHO python misses the one and only great entrypoint tool that satisfies everyone.
(just to be clear, I'm perfectly ok with your answer)


All GBs are bonus barrels

Recently came to the conclusion that this isn't something that we'd want. Not worth the time investment for a niche setting


Pivot table generation fails when no Columns and multiple Data are provided.

PR Details
Hi,
I've been using excelize and found pivot table generation does not work when multiple Data are provided and no Columns are provided. Please see the detail below.

Description
Here is a pivot table that I've been trying to generate with excelize.

And the base data for pivot table

Month
Year
Type
Sales
Sales1
Sales2
Sales3
Sales4
Sales5
Sales6
Region

Jun
2017
Produce
4059
2081
1318
4425
2540
456
3300
North

Aug
2017
Dairy
4728
3274
1211
1445
3237
4106
495
North

Sep
2018
Produce
4947
3287
2888
2790
3015
541
408
South

Dec
2017
Meat
1737
631
1485
26
1413
3090
194
South

Feb
2017
Beverages
4324
1159
1353
1957
3721
2189
2199
East

Feb
2019
Beverages
4703
4355
2451
3510
2605
156
3266
East

Jun
2019
Produce
746
1563
4376
4002
4718
447
94
West

Aug
2018
Meat
3623
953
1137
3133
4241
59
3033
South

Dec
2018
Beverages
4336
2546
4107
2940
1503
552
4843
West

Mar
2017
Produce
1515
4757
3687
3010
3410
285
3590
East

Jul
2018
Produce
3582
384
1297
4267
1137
4271
894
North

Nov
2017
Produce
2066
1270
493
3086
4819
3981
1052
South

Feb
2019
Produce
3749
1528
2818
4384
2903
1224
4547
East

May
2019
Produce
540
786
2051
3076
3640
2351
3844
East

Jun
2018
Dairy
90
1602
2258
3767
3231
2578
3154
North

Apr
2017
Meat
2743
1968
1166
3710
4535
440
4904
North

Feb
2017
Produce
3039
3430
4513
4700
1359
1720
783
North

Jan
2018
Beverages
565
4162
1829
2920
2048
1756
695
North

Sep
2019
Dairy
1092
3831
2577
2886
320
399
1162
South

Jan
2018
Produce
4103
4888
3318
3756
2019
2807
1157
East

Apr
2018
Dairy
3795
417
1393
3470
3996
4386
2632
East

Sep
2018
Dairy
1661
60
2420
1079
2954
1464
2060
South

Oct
2019
Meat
600
2039
4637
1533
561
2804
1685
West

Aug
2019
Beverages
1040
3662
740
1000
4284
3173
235
South

Jun
2017
Beverages
2869
3070
1336
3338
3472
1544
2395
North

Oct
2017
Dairy
1606
4648
3352
2420
3622
371
1117
South

Jun
2018
Produce
1592
129
4231
3721
855
4511
343
West

Jul
2017
Produce
3408
2008
528
3653
1774
3284
2375
North

Nov
2018
Beverages
4467
1114
4658
259
3546
4723
1724
South

Feb
2018
Produce
1531
3146
4983
928
516
4926
2477
East

In case you need to sum of Sales, Sales1, Sales2... by "Type"(rows), The code would be like below,
if err := f.AddPivotTable(&excelize.PivotTableOption {
                DataRange:       "Example!$A$1:$K$31",
                PivotTableRange: "Example!$A$32:$K$65",
		Rows: []excelize.PivotTableField{
			{Data: "Type"}},
		Filter:  []excelize.PivotTableField{},
		Columns: []excelize.PivotTableField{},
		Data: []excelize.PivotTableField{
			{Data: "Sales", Name: "Sum of Sales", Subtotal: "Sum"},
			{Data: "Sales1", Name: "Sum of Sales1", Subtotal: "Sum"},
			{Data: "Sales2", Name: "Sum of Sales2", Subtotal: "Sum"},
			{Data: "Sales3", Name: "Sum of Sales3", Subtotal: "Sum"},
			{Data: "Sales4", Name: "Sum of Sales4", Subtotal: "Sum"},
			{Data: "Sales5", Name: "Sum of Sales5", Subtotal: "Sum"},
			{Data: "Sales6", Name: "Sum of Sales6", Subtotal: "Sum"},
		},
		RowGrandTotals: true,
		ColGrandTotals: true,
		ShowDrill:      true,
		ShowRowHeaders: true,
		ShowColHeaders: true,
		ShowLastColumn: true,
	}); err != nil {
		fmt.Println(err)
	}

but this won't generate proper pivot table and end up with pivot table removed in the generated excel.
After doing comparison of "\xl\pivotTables\pivotTable1.xml",
the one that is working(generated manually) got colFields generated like,
<colFields count="1"> 	<field x="-2"/> </colFields>
but the one by excelize doesn't. (only colItems were generated)

I am not sure If I am following the standard, but I couldn't find what's "<field x="-2">" the value of x means.
here is the addition in order to generate colFields when there is no Columns input
func (f *File) addPivotColFields(pt *xlsxPivotTableDefinition, opt *PivotTableOption) error {
	pt.ColFields = &xlsxColFields{}
	if len(opt.Columns) == 0 {
		//in order to create pivot table in case there is no input from Columns
		pt.ColFields.Count = 1
		pt.ColFields.Field = append(pt.ColFields.Field, &xlsxField{
			X: -2,
		})
		return nil
	}

Related Issue

Motivation and Context

How Has This Been Tested
go test TestAddPivotTable

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Codecov Report

Merging #708 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #708   +/-   ##
=======================================
  Coverage   95.52%   95.52%
=======================================
  Files          31       31
  Lines        8379     8383    +4
=======================================
+ Hits         8004     8008    +4
  Misses        226      226
  Partials      149      149

Impacted Files
Coverage Δ

pivotTable.go
98.75% <100.00%> (+0.01%)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks for reviewing, I have removed files generated by go test.

This commit, break the fact to have multiple columns and multi data.
Now we can have :  no columns and many data, but we can't have both of them (many columns and many data).
it still ok for one column, one data.

This commit, break the fact to have multiple columns and multi data.
Now we can have : no columns and many data, but we can't have both of them (many columns and many data).
it still ok for one column, one data.

Hi, May I ask a sample of pivot table that you are trying to generate by excel not with this package? with many columns and many data.
The reason that I am asking this is because I couldn't find a way to generate a pivot table with many columns and many data.
here is a pivot table that I tried with sample data in excelize doc.

it's seem to be the same kind of problem,
with this code
if err := ef.AddPivotTable(&excelize.PivotTableOption{
		DataRange:       "Sheet1!$A$1:$T$" + strconv.Itoa(i),
		PivotTableRange: "TCD!$G$2:$M$34",
		Rows: []excelize.PivotTableField{{Data: "Designation", DefaultSubtotal: false},
			{Data: "Sous_reference", DefaultSubtotal: false},
			{Data: "Type_prev", DefaultSubtotal: false},
			{Data: "Commentaire", DefaultSubtotal: false}},
		Filter:         []excelize.PivotTableField{{Data: "Code_pays"}},
		Columns:        []excelize.PivotTableField{{Data: "Annee", DefaultSubtotal: false}, {Data: "Trimestre", DefaultSubtotal: false}, {Data: "Mois", DefaultSubtotal: false}},
		Data:           []excelize.PivotTableField{{Data: "Qte_real", Name: "QTE REALISE", Subtotal: "Sum"}, {Data: "Mnt_real", Name: "MNT REALISE", Subtotal: "Sum"}},
		RowGrandTotals: true,
		//ColGrandTotals: true,
		ShowDrill:      true,
		ShowRowHeaders: true,
		ShowColHeaders: true,
		ShowLastColumn: true,
	}); err != nil {
		fmt.Println(err)
	}

After doing comparison of "\xl\pivotTables\pivotTable1.xml",
the one that is working(generated manually) got colFields generated like,
<colFields count="4">
		<field x="5"/>
		<field x="6"/>
		<field x="7"/>
		<field x="-2"/>
	</colFields>

The one by exelize is :
<colFields count="3">
		<field x="5"/>
		<field x="6"/>
		<field x="7"/>
	</colFields>

I looks like the issue you are having is related to the following lines.
https://github.com/360EntSecGroup-Skylar/excelize/blob/c49222023756d6740877f286cb7d3e7a5a0950eb/pivotTable.go#L478
The commit for this PR was before the line pivotTable.go#L478, when only Columns length is zero.

Hi, May I ask a sample of pivot table that you are trying to generate by excel not with this package? with many columns and many data.
The reason that I am asking this is because I couldn't find a way to generate a pivot table with many columns and many data.
here is a pivot table that I tried with sample data in excelize doc.

exemple.xlsx


Why not to start with the PiMuxicBox Image?

Wouldn't it be easier to start with a http://www.pimusicbox.com/ Image instead of a plain debian?

ok - makes sense - but the image boots much faster...
I started again with a mini installation and there we go!
Boot time is now much lower...

ok - makes sense - but the image boots much faster...
I started again with a mini installation and there we go!
Boot time is now much lower...

How much?
I get 12 sec with the recommended installation until the boot up sound is ... but i would be interested tio reduce the time even more (Raspi 3 a+, MFRC522, i2s Dac).
Does phoniebox install (via one click install) without problems?

OK - my boottimes are much slower.
I have a rpi2
It takes 12 seconds until the rfid beeps and another 30 seconds until boot up sound.
With gui it was twice the time...
I have a rpi3b+ lying around - can i boot the same image or do i have tonsetbit up individually?
I could test boot time there.
Install scripts ran through without problems i think.


Update to enumflags2 0.7

Maintenance.

Nice!
I don't know if it's a conscious choice, but the discriminants can be populated automatically by enumflags2 now, so they're optional, e.g. MyVariant, instead of `MyVariant = 1 << 2,".

Yeah, but we are very careful that we use the correct integer in the variants. They are from the TDS manual.


VS 2022 Local Debug Durable Functions Issues "Error in LeaseManagerStarter task. Exception: Microsoft.WindowsAzure.Storage.StorageException: The response ended prematurely, with at least 128 additional bytes expected."

Which service(blob, file, queue, table) does this issue concern?
N/A
Which version of the Azurite was used?
Latest Visual Studio 2022 Enterprise
Where do you get Azurite? (npm, DockerHub, NuGet, Visual Studio Code Extension)
Visual Studio Code Extension
What's the Node.js version?
N/A
What problem was encountered?
I have an Azure Functions v3 application that I have been developing and running for several years. In VS 2019 I could debug locally without issue. When running the application in VS 2022 and debugging I ran into several issues.
There was one issue related to AzureWebJobsSecretStorageType that I solved by adding "AzureWebJobsSecretStorageType": "files" to my local.settings.json based on this post.
https://stackoverflow.com/questions/67728499/azure-functions-there-was-an-error-performing-a-read-operation-on-the-blob-stor
But after that I am still seeing errors that seem related to Durable Functions and I am not finding any answers online so far.
One issue is a timer function listener that triggers a Durable Functions clean up is unable to start due to Azure.Storage.Blobs: Service request failed. But then right after it seems to be able to start just fine. Why would it not start and then start?
Second is an "Error in LeaseManagerStarter task. Exception: Microsoft.WindowsAzure.Storage.StorageException: The response ended prematurely, with at least 128 additional bytes expected." which then leads to a "Error in LeaseManagerStarter task. Exception: System.InvalidOperationException: AppLeaseManager has already started"
I had previously posted about this in the Azure Functions Durable Function discussion which has more details:
https://github.com/Azure/azure-functions-durable-extension/discussions/2025
@davidmrdavid suggested that I bring the discussion here.
The problem started when I moved the solution to VS 2022. The project works fine in VS 2019. I understand that is because VS 2019 uses the Azure Storage Emulator and VS 2022 uses Azurite. So, either VS 2022 should just work or there should be some migration steps documented someplace on how to convert a project from Azure Storage Emulator to Azurite, or there is something else going wrong.
Steps to reproduce the issue?
Debug my project in VS 2022.
Have you found a mitigation/solution?
It was suggested that if I run Azure Storage Emulator before launching VS 2022 that I can debug the project without issue but that it will be using Azure Storage Emulator rather than Azurite. Since Azure Storage Emulator is deprecated I would like to get things working with Azurite.
Thanks

Hi Andrew,
Thanks for raising this.
Would you be able to start Azurite manually and share a debug log?
You can find the details on how to activate it here:
https://github.com/Azure/Azurite#debug-log-configuration
That will show us which requests are failing and why.
Alternatively, if you can share a sample which reproduces the issue, I can run it locally and debug the issue.
Many thanks!

Hi Edwin. I can even seem to run Azurite from the command line. I get this error:
C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\IDE\Extensions\Microsoft\Azure Storage Emulator>azurite -d Azurite_ACv3.log
Azurite Blob service is starting at http://<IP_ADDRESS>:10000
Exit due to unhandled error: Error: EPERM: operation not permitted, open 'C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\IDE\Extensions\Microsoft\Azure Storage Emulator_azurite_db_blob_.json~'
From: Edwin Huber @.>
Sent: Wednesday, April 27, 2022 1:24 AM
To: Azure/Azurite @.>
Cc: Andrew Hawes @.>; Author @.>
Subject: Re: [Azure/Azurite] VS 2022 Local Debug Durable Functions Issues "Error in LeaseManagerStarter task. Exception: Microsoft.WindowsAzure.Storage.StorageException: The response ended prematurely, with at least 128 additional bytes expected." (Issue #1447)
Hi Andrew,
Thanks for raising this.
Would you be able to start Azurite manually and share a debug log?
You can find the details on how to activate it here:
https://github.com/Azure/Azurite#debug-log-configurationhttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2FAzurite%23debug-log-configuration&data=05|01|Andrew.Hawes%40ClarityMIS.com|32da44063917419f5e1408da28168df0|1f87f22d99f94bc9b392bb0e2bb46d4c|0|0|637866374588723468|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=qUVsEujigXRYrkvSY0m1Fsm4naRfZhKVXU5iMMMvrTI%3D&reserved=0
That will show us which requests are failing and why.
Alternatively, if you can share a sample which reproduces the issue, I can run it locally and debug the issue.
Many thanks!

Reply to this email directly, view it on GitHubhttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2FAzurite%2Fissues%2F1447%23issuecomment-1110591435&data=05|01|Andrew.Hawes%40ClarityMIS.com|32da44063917419f5e1408da28168df0|1f87f22d99f94bc9b392bb0e2bb46d4c|0|0|637866374588723468|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=MIDoi42%2FVz7EBzEYQVaedfBBohy9kXgKFqkwNA0GDQM%3D&reserved=0, or unsubscribehttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABN4FEGXHZ3XT3W7ALDTZMLVHDMQ7ANCNFSM5UMBBMPQ&data=05|01|Andrew.Hawes%40ClarityMIS.com|32da44063917419f5e1408da28168df0|1f87f22d99f94bc9b392bb0e2bb46d4c|0|0|637866374588723468|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=Yxt%2FIk1HhrO3sos9%2FAhUADxVX32jThSYvVlIehbZ2Wc%3D&reserved=0.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

If visual studio and another instance of Azurite are running, then it can't get a second handle to open the files used to persist the data.
Please first reboot, then start Azurite from Command Line, then start your VS Code / Visual Studio project.

I rebooted and started up Teams, Outlook, and Chrome and the opened a new Command Prompt and changed to the "C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\IDE\Extensions\Microsoft\Azure Storage Emulator" folder and then ran "azurite -d C:\Users\Andrew.Hawes\DownloadsAzurite_ACv3.log" and got the same error.
Azurite Blob service is starting at http://<IP_ADDRESS>:10000
Exit due to unhandled error: Error: EPERM: operation not permitted, open 'C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\IDE\Extensions\Microsoft\Azure Storage Emulator_azurite_db_blob_.json~'
From: Edwin Huber @.>
Sent: Wednesday, April 27, 2022 2:05 PM
To: Azure/Azurite @.>
Cc: Andrew Hawes @.>; Author @.>
Subject: Re: [Azure/Azurite] VS 2022 Local Debug Durable Functions Issues "Error in LeaseManagerStarter task. Exception: Microsoft.WindowsAzure.Storage.StorageException: The response ended prematurely, with at least 128 additional bytes expected." (Issue #1447)
If visual studio and another instance of Azurite are running, then it can't get a second handle to open the files used to persist the data.
Please first reboot, then start Azurite from Command Line, then start your VS Code / Visual Studio project.

Reply to this email directly, view it on GitHubhttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2FAzurite%2Fissues%2F1447%23issuecomment-1111375106&data=05|01|Andrew.Hawes%40ClarityMIS.com|48239debc40f4129048408da2880cb22|1f87f22d99f94bc9b392bb0e2bb46d4c|0|0|637866830874140464|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=topokxasJEab2Wpf7cUuTXhOvteKshlnuARNU9CVUPA%3D&reserved=0, or unsubscribehttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABN4FEFG5M35OEOVLB5Q4B3VHGFU3ANCNFSM5UMBBMPQ&data=05|01|Andrew.Hawes%40ClarityMIS.com|48239debc40f4129048408da2880cb22|1f87f22d99f94bc9b392bb0e2bb46d4c|0|0|637866830874140464|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=6NZQyUz1RSgVe%2BAw1U8gl06OugeyhAhB%2FzBCLw1kLuQ%3D&reserved=0.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

@edwin-huber, any feedback on my comments.

@ahawes-clarity this looks like a permissions issue, the database engine cannot open the DB location.
You could try running in an administrative window, but this is an unusual problem.
We are looking at long term options for this type of issue.
@Sti2nd Regarding the functions project, there is a release pending to resolve this, you need to upgrade Visual Studio to upgrade Azurite.

Thanks @edwin-huber that seemed to work. Running Azurite in a administrative command window before launching VS removed the errors I was seeing. I have a log file I can send you but am concerned about posting it here. How can I get it to you if you feel it might be helpful?
Does this give you more insight into what the problem is and a possible resolution. I really don't want to have to follow these steps to get things working each time.

@edwin-huber I am on Visual Studio 2022 version 17.1.6. For some reason my VS can't check for updates right now.
@ahawes-clarity Since running another version of Azurite outside of Visual Studio fixes the problem for me, did you try running Azurite without administrative privileges before opening Visual Studio?

@Sti2nd , yes I did try running Azurite before VS but it only worked when running Azurite as an administrator.

@ahawes-clarity I don't think a debug log will show us anything we don't already know.
The symptoms point to a permission issue on the file system.
The error states that it is unable to open the file at the location 'C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\IDE\Extensions\Microsoft\Azure Storage Emulator\
You could check with windows explorer what permissions are set there.
I ran the following PowerShell commands in Windows Terminal / PowerShell:
cd C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\IDE\Extensions\Microsoft
get-childitem . | where-object {($_.name -eq "Azure Storage Emulator" -and $_.PSIsContainer)} | get-ACL | fl

My permissions are:
Path   : Microsoft.PowerShell.Core\FileSystem::C:\Program Files\Microsoft Visual
         Studio\2022\Enterprise\Common7\IDE\Extensions\Microsoft\Azure Storage Emulator
Owner  : BUILTIN\Administrators
Group  : DESKTOP\<my local username>
Access : NT SERVICE\TrustedInstaller Allow  FullControl
         NT SERVICE\TrustedInstaller Allow  268435456
         NT AUTHORITY\SYSTEM Allow  FullControl
         NT AUTHORITY\SYSTEM Allow  268435456
         BUILTIN\Administrators Allow  FullControl
         BUILTIN\Administrators Allow  268435456
         BUILTIN\Users Allow  ReadAndExecute, Synchronize
         BUILTIN\Users Allow  -1610612736
         CREATOR OWNER Allow  268435456
         APPLICATION PACKAGE AUTHORITY\ALL APPLICATION PACKAGES Allow  ReadAndExecute, Synchronize
         APPLICATION PACKAGE AUTHORITY\ALL APPLICATION PACKAGES Allow  -1610612736
         APPLICATION PACKAGE AUTHORITY\ALL RESTRICTED APPLICATION PACKAGES Allow  ReadAndExecute, Synchronize
         APPLICATION PACKAGE AUTHORITY\ALL RESTRICTED APPLICATION PACKAGES Allow  -1610612736

@edwin-huber mine look the same.
Path   : Microsoft.PowerShell.Core\FileSystem::C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\IDE\Extensions\Microsoft\Azure Storage Emulator Owner  : BUILTIN\Administrators Group  : Access : NT SERVICE\TrustedInstaller Allow  FullControl          NT SERVICE\TrustedInstaller Allow  268435456          NT AUTHORITY\SYSTEM Allow  FullControl          NT AUTHORITY\SYSTEM Allow  268435456          BUILTIN\Administrators Allow  FullControl          BUILTIN\Administrators Allow  268435456          BUILTIN\Users Allow  ReadAndExecute, Synchronize          BUILTIN\Users Allow  -1610612736          CREATOR OWNER Allow  268435456          APPLICATION PACKAGE AUTHORITY\ALL APPLICATION PACKAGES Allow  ReadAndExecute, Synchronize          APPLICATION PACKAGE AUTHORITY\ALL APPLICATION PACKAGES Allow  -1610612736          APPLICATION PACKAGE AUTHORITY\ALL RESTRICTED APPLICATION PACKAGES Allow  ReadAndExecute, Synchronize          APPLICATION PACKAGE AUTHORITY\ALL RESTRICTED APPLICATION PACKAGES Allow  -1610612736

Hi @ahawes-clarity did you remove / redact the "Group" entry, or is it blank on export?

@edwin-huber it was blank on export

In that case, please use the explorer to do the following:

Browser to this folder: C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\IDE\Extensions\Microsoft
right click on the folder Azure Storage Emulator and choose "properties"
choose the "Security" tab
and click the "advanced" button
at the top, where it says owner, click the "Change" button
add yourself back as "owner" of the folder by searching for your local user name, or domain user name depending on how you log into Windows
click "check names"
click "OK"
Click "Apply "

I can do that but first please tell me why I would need to do that. Currently the folder is owned by the local computer's Administrators Group.

The idea is that it will allow access for the process without needing to elevate the permissions to administrator.

I set the Owner of the folder to the Users Group

I then opened up VS 2022 and ran my Azure Functions project.
I received the same errors as shown in the initial post of this thread.
I closed down VS 2022.
I opened a command window in normal mode (not administrator) and tried to start azurite and received the following error:
Azurite Blob service is starting at http://<IP_ADDRESS>:10000
Exit due to unhandled error: Error: EPERM: operation not permitted, open 'C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\IDE\Extensions\Microsoft\Azure Storage Emulator\__azurite_db_blob__.json~'

I ran the same powershell command as before to verify the permissions.
Path   : Microsoft.PowerShell.Core\FileSystem::C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\IDE\Extensions\Microsoft\Azure Storage Emulator
Owner  : BUILTIN\Users
Group  :
Access : NT SERVICE\TrustedInstaller Allow  FullControl
         NT SERVICE\TrustedInstaller Allow  268435456
         NT AUTHORITY\SYSTEM Allow  FullControl
         NT AUTHORITY\SYSTEM Allow  268435456
         BUILTIN\Administrators Allow  FullControl
         BUILTIN\Administrators Allow  268435456
         BUILTIN\Users Allow  ReadAndExecute, Synchronize
         BUILTIN\Users Allow  -1610612736
         CREATOR OWNER Allow  268435456
         APPLICATION PACKAGE AUTHORITY\ALL APPLICATION PACKAGES Allow  ReadAndExecute, Synchronize
         APPLICATION PACKAGE AUTHORITY\ALL APPLICATION PACKAGES Allow  -1610612736
         APPLICATION PACKAGE AUTHORITY\ALL RESTRICTED APPLICATION PACKAGES Allow  ReadAndExecute, Synchronize
         APPLICATION PACKAGE AUTHORITY\ALL RESTRICTED APPLICATION PACKAGES Allow  -1610612736
Audit  :
Sddl   : O:BUG:DUD:AI(A;ID;FA;;;S-1-5-80-956008885-3418522649-1831038044-1853292631-2271478464)(A;CIIOID;GA;;;S-1-5-80-956008885-3418522649-1831038044-1853292631-2271478464)(A;ID;FA;;;SY)(A;OICIIOID;GA;;;SY)(A;I
         D;FA;;;BA)(A;OICIIOID;GA;;;BA)(A;ID;0x1200a9;;;BU)(A;OICIIOID;GXGR;;;BU)(A;OICIIOID;GA;;;CO)(A;ID;0x1200a9;;;AC)(A;OICIIOID;GXGR;;;AC)(A;ID;0x1200a9;;;S-1-15-2-2)(A;OICIIOID;GXGR;;;S-1-15-2-2)

I then opened a new command window as administrator and started azurite successfully.
So, it seems that setting the folder ownership to the Users group did not solve the problem.

OK, many thanks for taking the time to try it.
I have just reproduced this on my own system.
I am always working from the main branch, and so don't start it from the visual studio download location.
Looking at the folder, it is marked "read-only", which would explain why this only works when starting as administrator.
However, I am unable to change this restriction.
I need to research and follow up with my colleagues who own the VS distribution mechanism, to see what the expected behavior and folders in this path should look like.
Please bear with us.

@ahawes-clarity
"C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\Extensions\Microsoft\Azure Storage Emulator" seems to be read only be design. I have same read only settings cross my different Visual Studio 2022 installations.
If you want to start 'azurite.exe', please make a directory like C:\Azurite to store data for Azurite. Refer to https://github.com/microsoftdocs/azure-docs/issues/83901
cd C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\Extensions\Microsoft\Azure Storage Emulator
azurite --silent --location c:\azurite --debug c:\azurite\debug.log

@ahawes-clarity Can you share the steps to reproduce the issue in Visual Studio 2022 (instead of starting azurite from the extension path)? I'll forward the steps to Visual Studio team for trouble shoot.

@XiaoningLiu the steps are quite simple.

Run my Azure Functions v3 project locally in debug mode from VS 2022.

I am not sure if there is something about my project that may be a factor. It is a project that has evolved over the years. Works fine in VS 2019 and Azure Storage Emulator. It does use Durable Functions.
    <TargetFramework>netcoreapp3.1</TargetFramework>
    <AzureFunctionsVersion>v3</AzureFunctionsVersion>
    <PackageReference Include="Microsoft.Azure.Functions.Extensions" Version="1.1.0" />
    <PackageReference Include="Microsoft.Azure.WebJobs.Extensions.Storage" Version="4.0.3" />
    <PackageReference Include="Microsoft.EntityFrameworkCore.SqlServer" Version="3.1.9" />
    <PackageReference Include="Microsoft.NET.Sdk.Functions" Version="3.0.13" />

@edwin-huber says he was able to reproduce the issue so maybe he has more info.

Hi @ahawes-clarity,
thanks, yes, Xiaoning and I have spoken.
I was able to reproduce the error in my environment, by trying to start the executable azurite.exe from the command line.
@XiaoningLiu the steps to reproduce the error are here:
https://docs.microsoft.com/en-us/azure/storage/common/storage-use-azurite?tabs=visual-studio
The VS documentation specifically states to start from the command line, which does not work without the --location parameter.
When I start one of my function apps, the location parameter is automatically included, which stores the data files in a different location, this would then also solve the issue for people starting the emulator from command line:
azurite.exe --location "C:\Users\mylocaluser\AppData\Local\Temp\Azurite" --debug "C:\Users\mylocaluser\AppData\Local\Temp\Azurite\debug.log"
We need to check why the location parameter is not being used in the migrated / upgraded function apps...

Many thanks for the tip, we shall look into it!
@XiaoningLiu FYI!

I randomly reproduced this issue again by clearing out my NuGet cache. After getting everything rebuilt I was getting errors and had to delete those temp files again. Not sure it's relevant but figured I'd mention it just in case.

But it quickly stopped working for me 😠 I guess I could delete the folder again, but getting seriously tired of the hassle with Azure Functions now

OK, so I get the same issue and it seems that it's due to another Azurite process running on the same port but it seems isolated from any builds or something. Still, running the below steps seem to work:

Close Visual Studio

Try the following PS Code (sorry for the crappy code!):
cd "C:\Program Files\Microsoft Visual Studio\2022\Professional\Common7\IDE\Extensions\Microsoft\Azure Storage Emulator"
$azStop = .\azurite.exe stop
## Find processes running on port 10000
$portUse = Get-Process -Id (Get-NetTCPConnection -LocalPort 10000).OwningProcess;
$portUse
## Kill processes' on this port
foreach($port in $portUse) {
    if($portUse.Id -ne 0) {
        $x = taskkill /F /PID $portUse.Id
        $x
   }
}

run .\azurite.exe start then reopen VS. It should now build & serve your app with no errors.

I have encountered the same error today, and after trying the suggestions in this thread it still didn't work.
I then tried updating to the latest version of Visual Studio (Microsoft Visual Studio Community 2022 (64-bit) - Preview, Version 17.6.0 Preview 6.0), and then reinstalling the Azure development feature from the Visual Studio installer.
Everything seems to be working fine now.

All these issues can be fixed by following these steps ( Running in admin mode is not an option)

Clearing the NPM cache %APPDATA%\npm-cache, delete everything in this folder
npm install azure
Then start azurite it should work

For more details refer to his article
https://learn.microsoft.com/en-us/archive/blogs/matt-harrington/how-to-fix-node-js-npm-permission-problems


enable savepoint with tidb 6.2.0

Signed-off-by: Weizhen Wang<EMAIL_ADDRESS>

Could we also turn support_transaction on? I remember the reason to disable it is because not support savepoint(see the link in code). And various tests could also work.


Virtual Parts and Cut List Items Cause Exceptions

Parts and assemblies with virtual parts and/or cut list items cause exceptions to be thrown by the AutoGetReferencedAddin::getVersionConflicts method.

That solved the Cut List and Sheet objects.
Can we add the GetLocalVersionNo2() call into the try block as well to cover the Virtual Components?
int localVersion = retrievedFile.GetLocalVersionNo2(selection.mbsPath, out bool obsolete); Is throwing a 'The file could not be found.' exception on virtual components. the selection.mbsPath is showing the file in the vault folder of the parent assembly. I'm guessing the file is not found because it's not actually there? If I understand correctly the virtual component file is "embedded" in the assembly file so it's really not any concern to us. It would be nice to also filter these out of the selection set as well. I don't see any way to set up the SelectionList to ignore object types or how to test each one. In the database Documents table the virtual components have ObjectTypeID = 2.

That solved the Cut List and Sheet objects.
Can we add the GetLocalVersionNo2() call into the try block as well to cover the Virtual Components?

Done.


mouse pressed is mouse released and vice versa

In GlobalMouseHook.java "mousePressed" is called after the mouse has gone up and "mouseReleased" is called after the mouse has gone down...

What a bug. Fixed with 8d1ab72708f237692867c2fa1d52a841e5c8a74d and version 2.3. Thank you!


Added guide for GitLab CI

This guide came out of a discussion in https://github.com/devcontainers/ci/issues/205. While GitHub has a devcontainer action, nothing like this exists for GitLab CI. This caused a bit of a gap in documentation on how to get advanced devcontainer functionality into GitLab CI. This PR is designed to close that gap.

Tagging @bamurtaugh for review

Thanks so much for opening this @raginjason, this looks like a great guide! And thanks @samruddhikhandale for the quick review ✨ - I agree with the suggestions she mentions.


License

Hey, what is the license of the project?

Thank you, the project follows the MIT license


Monorepo and Splitting Out To Different Packages

Goals

We've wanted to move to a monorepo for quite some time. This is so we can move the docs app out of the project for core components, and possible move things into our monorepo to ensure compatability — for example chrome tools or the eslint plugin.
We've discussed many times splitting Observable off to be a standalone package.
We may want to consider splitting rxjs/testing, rxjs/ajax, etc, off into their own packages as well.
We want to do this in the slowest, most easy to migrate to way possible.

Monorepo and multiple packages

I think we should try to button up new feature work and the like for version 7 (which we are very close to doing, if we're not done already) and move to work on version 8 in earnest.
In version 8, we'll keep the same exports (with regards to rxjs, rxjs/operators, rxjs/ajax, et al) but split all of those off into their own packages under @rxjs/observable, @rxjs/operators, @rxjs/ajax, etc. We'll want to bikeshed ideas like @rxjs/subjects or @rxjs/multicasting etc. There's a lot of ground to cover there, and I don't want to get bogged down on that in this discussion.
In version 8 we're going to reexport from the new packages. For example, rxjs will export { Observable } from '@rxjs/observable'; etc.

Standalone Observable

Moving away from lift: If we want a standalone Observable, we should move away from lift, because it requires too much knowledge of the inner working of RxJS in order to create custom operators that leverage it properly.
We can start supporting Observable that doesn't have lift as a non-breaking change anytime. This is completely contained within our operate utility function now. We may even start supporting this in v7, as it should be completely innocuous. It's just a different call pattern. I actually have some work done around this, and everything looks solid.
I think we're going to need to leave pipe on our standalone observable package, as it's central to our particular design for how to use RxJS. Even though pipe might not be immediately useful to users of Observable that don't care about operators. (libraries that export Observable, etc).

Long Term:

We wouldn't want to force people to move over to @rxjs/observable et al until version 9.
Operator creation needs to be consumerized. We have our out internal way of creating operators, and we need to come up with a way of making that consumable and understandable to the masses. It has some advantages, like properly unsubscribing from firehose inner observables. (Currently it's impossible to do this properly with plan observer implementations) (A token/signal approach would help this)
Considerations around using abort signals, etc.

Core Team:

Monorepo - 👍
Creating "mini-packages" like @rxjs/observable, et al - 👍
a. Level of granularity? What do we do with things like mergeInternals? What about operators derived from other operators (mergeMap and concatMap for example)? ❓

I've posted this on #6367, but I feel it should also be seen here, sorry for duplicate:
Wouldn't publishing multiple packages for what is essentially the same code actually increase risk of non tree shakable use-case and thus increase bundle size ?
As seen with lodash, if one of my dependency decides to depends on the future @rxjs/observable, but my app depends on the current rxjs, then all the code from @rxjs/observable will essentially be bundled twice, won't it ?
And of course it would be much worse if we actually publish a package for each operator. Then a single operator could be bundled three times if my deps happened to depend on rxjs, @rxjs/operators and say @rxjs/operators-map.
I could also imagine compatibility issues if one of my dependency depends on an old version of  @rxjs/observable and my app pass one of those object to a new, incompatible version of an operator coming from the latest version of rxjs. This kind of issue already happened with the graphql package where they had to add manual check in their code to throw error if it happens.
That seems a lot of tricky tradeoffs to attempt to solve the issue of bundle size that is already properly solved by tree shaking.
Is that really worth it ? :thinking:

@benlesh would you please share the rxjs team opinions about the risks mentioned in my previous post?
How could we avoid fragmenting the ecosystem and actually increase the bundle size if things are available from multiple packages at once ?
How can we solve incompatibilities across future version of rxjs, that are transient dependencies, and that will still have to collaborate with each other?
Also what do you think about the fact that lodash seems to be going into the opposite direction, single ESM tree-shakable package, https://github.com/lodash/lodash/issues/5107#issuecomment-1106683823 ?

(Sorry originally posted this in the roadmap issue, before realising this issue is probably more appropriate)
I just wanted to contribute to the discussion of packaging ... I've been successfully using rxjs 7.5.x in Deno via esm.sh, but the way things are imported isn't ideal, ie. everything is import { ... } from 'rxjs' (where rxjs is actually mapped to an esm.sh url in an import map) ... treeshaking isn't an option in this situation, so the whole of rxjs ends up being loaded whether used or not.
I'm not bothered really whether ops are individually packaged or not, but would prefer a situation where they are at least exposed publicly as individual modules, so I can do something like: import { concatMap } from 'rxjs/op/concatMap'; and in turn that imports only on what it needs (not the whole of rxjs) ... I know rxjs modules already exist like this, but they are all under an internal module structure.


Add widthUnits & sizeUnits to core layers

Change List

Add widthUnits & sizeUnits to LineLayer, ArcLayer, PathLayer, PointCloudLayer, ScatterplotLayer, PolygonLayer and GeoJsonLayer
Revert recent changes made to the ContourLayer API, restore support for per-threshold stroke widths
Docs

Coverage increased (+0.04%) to 59.0% when pulling da869c8aa2a18041e261e69f88ae27965705794c on api-updates into a219ac2bed39c50084f2be606b4d937a855a65ce on master.


Support full-precision embeddings in half precision inference mode

Cast embeddings down to the correct precision

Interesting. Does it still work with the same memory footprint as the half precision implementation?

Interesting. Does it still work with the same memory footprint as the half precision implementation?

Yes, I use this to run inference with custom embeddings on an 8GB GPU, because it is not enough VRAM for full-precision.

lg2m. thanks for the fix!

Thanks @sajattack for this PR. Now embeddings.pt finally works on my 8GB VRAM GPU!


Corrected incorrect offset value in alias format documention

CI tests seem to be failing due to unrelated reason
also Reserved (set to zero) in one of the alias data samples I was looking at this value contained:
0x00000000  ff fe 00 00 00 00 00 00  00 00 ff ff ff ff        ..............

@freakboy3742 I would recommend to create a GitHub PR template so it is clear to me what information you would like to have with the PR. Given the change was small and straightforward I assumed there was sufficient context.
Also see: https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/creating-a-pull-request-template-for-your-repository


Update data-classification-activity-explorer-available-events.md

@chrfox
We have now released PowerBI data on Activity Explorer and now label activities from PowerBI show on the Activity Explorer

@chrfox Please merge this PR. Thanks


Deploy timeout

'sous deploy' times out 30 minutes after successfully writing to the GDM (by way of nested 'sous plumbing status' call).

Ideally the context should be passed through to each http request through the sub pollers etc, however that would be significant extra work so I'm not including it in this PR. The core sous.HTTPClient interface methods should probably be changed to accept a context, and at that point we can pass the context down.

Codecov Report

Merging #340 into master will decrease coverage by -0.01%.
The diff coverage is 68%.

@@            Coverage Diff             @@
##           master     #340      +/-   ##
==========================================
- Coverage   61.64%   61.64%   -0.01%
==========================================
  Files         161      161
  Lines        8706     8726      +20
==========================================
+ Hits         5367     5379      +12
- Misses       2898     2905       +7
- Partials      441      442       +1

Impacted Files
Coverage Δ

cli/sous_plumbing_status.go
5.55% <0%> (-0.7%)
:x:

lib/status_poller.go
82.77% <77.27%> (-0.48%)
:x:

lib/auto_resolver.go
88.05% <0%> (-1.5%)
:x:

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 70f0db3...103cf2f. Read the comment docs.


[Office add-ins march 2022] Remove consuming graph services demo

Category

[x] Content fix
[ ] New article

Related issues
None
Contents of the Pull Request
Update blog post since demo was removed.

@LuiseFreese do you know how to fix the Markdown Lint network errors in this PR? It looks like some links to valid URLs fail for some reason in the link check.

Hey there @davidchesnut - Don't worry about the Linkchecker. As we use a static site generator (Hugo), the entire site gets built every time we push/merge. That also means that all links of all articles get a check.
Sometimes, perfectly fine working links are flagged as network error, sometimes, we see that an SSL certificate has expired. I will take care of that :-)


File version for WPFLocalizationExtension and XAMLMarkupExtensions is <IP_ADDRESS>

File version for WPFLocalizationExtension and XAMLMarkupExtensions is <IP_ADDRESS>. When checking the details of WPFLocalizationExtension.dll and XAMLMarkupExtensions.dll the file version seems to be not set. This breaks the installer for our software, which checks the file version or product version. Previous version had the file and product version set to the nuget package version.

@alanwol  I will check this asap

@alanwol thanks, please test the new nuget WPFLocalizeExtension 3.3.1
If you have any issues, just reopen the ticket

I've checked the new packages and they have the file version and product version set now. I hope this solves the issue with the installer, in any case I consider this issue resolved.


Remove jcenter

Hey there!
Please remove jcenter() from your buildscript repositories, as it is deprecated and regularly causes problems.
Other RN libraries also disable the buildscript block when the project is not the root project (when it's included as a library): https://github.com/react-native-netinfo/react-native-netinfo/blob/master/android/build.gradle#L5

https://github.com/PostHog/posthog-react-native/pull/60

Closing this with the release of v2.0.0 as we now have a completely refactored "pure" Javascript implementation which avoids native crossover issues like these. Closing


feat(website-backend): replace acf pro feature by using custom plugin

Closes #266

@manuelportela I've updated everything on stage (cms and website).


Getting or setting Position does nothing on UWP

On UWP, it is possible to get or set the position property but the property is unrelated to the state of the video.
Setting the position does not cause the video position to change, and the property does not change as the video is playing.
I have not tested on other platforms.

Same on Windows 8.1

Fixed for UWP 1.0.2017.419. Keeping open for WinRT/wp81

Renderers are now shared, fixed across Windows

On WinRT/wp81.
Setting position seems to work, but it doesn't like to be set to the same position twice. Nothing happens in this case.
A workaround is to add a small random number when setting.
Can give minimal example if needed.


Added some null-checks and updated $position for ids

Null-Check for !position and $leaderboard

Udpated $position for myself when not on leaderboard

Prevented crash on !new championship when no one is in the leaderboard

Working position for myself when i have something caught

Working position for someone else

I hope i haven't forgotten some proofs or tests

Just noticed console.log(`User ID ${id} found in guild.`); is dead code.

Fixes #9


Fixed #25687 -- Documented how to add database function support to third-party backends.

Updated from https://github.com/django/django/pull/5574.

I'd rather replace microsoft by sqlserver in the function names and variables.

This is copied from django-mssql but I'd be okay with it, although in my mind "Microsoft" is the vendor, not "SQL Server." Of course this could be problematic as Microsoft might have several database offerings.

Yes, MS Access comes to mind. We have the precedent of Oracle, which is both a company name and a product. As for me, the database product name makes more sense for the vendor property. But I wouldn't fight for it.


pipenv install does not detect dependencies in setup.cfg

pipenv, version 11.0.7
I have the following project:

https://github.com/HearthSim/twitch-hdt-ebs/tree/7cb8315eb2740e520ac74b0dc93ab252a8199f69

I ran pipenv install at the root.
Now with most projects, pipenv detects requirements.txt. But before pipenv, I was using setup.cfg to define dependencies as it's less icky than requirements.txt and I can do pip install ..
Expected result
pipenv should have detected the dependencies in setup.cfg install_requires
Actual result
An empty venv and pipfile were created.

Pipenv reads dependencies from Pipfile, which does not take setup.py (and by extension setup.cfg) into account, because they serve different purposes. Read the docs.

@uranusjr but pipenv does read dependencies from requirements.txt on first install.

requirements.txt and setup.py serve different purposes. Pipfile replaces common usages of requirements.txt, but not setup.py. Further reading.

$ pipenv install -e .

@kennethreitz Thanks, that makes sense.

Not sure if this is cause to reopen the ticket, but @kennethreitz 's answer of pipenv install -e . only works for projects with a setup.py file.
I'd love to see support for setup.cfg declared dependencies in the same way.

It appears that setup.cfg is now handled by pipenv install -e, but it does not appear to understand the options.extras_require setuptools option.
The required incantation appears to be
pipenv install -e '.[test,debug]'

for example, assuming extras_require contains test and debug groups. This will result in a Pipfile entry like
[packages]
mylocalpkg = {editable = true, extras = ["debug", "test"], path = "."}


New version: Starpine.Screenbox version 0.14.3

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.9 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

This package is confirmed to be working but needs manual validation as mentioned in https://github.com/microsoft/winget-pkgs/pull/188154#issuecomment-2455980992


Replace google-api-python-client by protobuf

Let me prefix that one by saying that I haven't been testing it extensively, it just seems to be working as expected.
This PR would fix https://github.com/valkey-io/valkey-glide/issues/2298. Can you just trigger the test suite and see what happens? They seem to be passing fine there: https://github.com/Rafiot/valkey-glide/actions/runs/10905533315

@Rafiot could you please push your branch to this repo? We need to confirm that module test CI also passes. It runs only on a branches on this repository (not on forks).
Please also add changelog entry.

@Yury-Fridlyand I'm not sure to understand what you mean by pushing my branch to this repo. Isn't it what I did with the PR? I don't have write access so I cannot push a branch directly.
I'll update the changelog.

I run CI for you, all pass: https://github.com/valkey-io/valkey-glide/actions/runs/10909859375
I had to add extra fix for CI from #2308

LGTM
Thanks a lot @Rafiot! I will merge it since i guess you don't have the permissions.
And thanks @Yury-Fridlyand for the extra mile.

\o/ Thank you!


[Security] Workflow main.yml is using vulnerable action actions/checkout

The workflow main.yml is referencing action actions/checkout using references v1. However this reference is missing the commit a6747255bd19d7a757dbdda8c654a9f84db19839 which may contain fix to the some vulnerability.
The vulnerability fix that is missing by actions version could be related to:
(1) CVE fix
(2) upgrade of vulnerable dependency
(3) fix to secret leak and others.
Please consider to update the reference to the action.

Thank you for good issue :) I will fix it


Please disable demo for theme toha

HI,
Please disable demo for theme toha. The exampleSite contents is not sufficient to show the features of the theme. Furthermore, the Homepage url already point to a demo site hosted in Github Pages.  It's becoming increasingly difficult to keep the exampleSite and the homepage site in sync.
Thanks

Hello Emruz,
as requested, your theme's demo has been disabled in https://github.com/gohugoio/hugoThemes/commit/58b6a026579a3b1b696e39220e7e1c604e25f6fc.

Furthermore, the Homepage url already point to a demo site hosted in Github Pages.

That's good for users to find your self-hosted demo 👍


Editor misbehavior with option_d.py

When loading option_d.py (as in the README), the colormap is mostly out of gamut (is that the right term?).

When using "Load in viewer":
Traceback (most recent call last):
  File "/home/robatt/Projects/_external/viscm/viscm/gui.py", line 1319, in loadviewer
    v = viscm(cm, name=self.viscm_editor.name, figure=newfig)
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/robatt/Projects/_external/viscm/viscm/gui.py", line 265, in __init__
    Jpapbp = self._sRGB1_to_uniform(RGB)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/robatt/.local/share/miniconda3/envs/viscm/lib/python3.11/site-packages/colorspacious/transform_graph.py", line 581, in __call__
    x = transform(x, **kwargs)
        ^^^^^^^^^^^^^^^^^^^^^^
  File "/home/robatt/.local/share/miniconda3/envs/viscm/lib/python3.11/site-packages/colorspacious/conversion.py", line 86, in _XYZ100_to_CIECAM02
    return ciecam02_space.XYZ100_to_CIECAM02(XYZ100)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/robatt/.local/share/miniconda3/envs/viscm/lib/python3.11/site-packages/colorspacious/ciecam02.py", line 215, in XYZ100_to_CIECAM02
    / (tmp + (h_i[i + 1] - hprime) / e_i[i + 1]))
              ~~~^^^^^^^
IndexError: index 5 is out of bounds for axis 0 with size 5

Just got done testing at 65b657f (before the pile of recent changes) and both symptoms reproduce there as well. I'll add this to the list of pre-release tasks, just going to focus on Qt6 for now.

This was me being dumb. I forgot that this colormap uses Bezier splines. I didn't actually do what it says in the README.


Implement cross-referencing

Currently, every Talon object which is analysed by the Talon domain is given a unique name, but I've not had the opportunity to implement the ability to resolve cross-references yet.
This is especially important since it would enable us, for example, to populate the documentation for a tag with the contexts and Talon files that implement it (see #11).
For an example, see add_target_and_index and resolve_xref in Developing a “recipe” extension.

On the example is stores all of the indexes, directives, and the domain all in the same file. The current setup of the repo has the directives in /src/talondoc/sphinx/directives and the domain in the parent folder. Would it be better to add a second child folder to the sphinx directory for the indexes or should they be collocated with their respective directives?

I'm happy for you to put them under directives.

Two quick questions:

Where in the domain is the objects data stored?

In the example the function self.get_objects() that is used in the resolve_xref() returns the data stored in the self.data, and that data is populated in an add_recipe function. I don't see an equivalent set of functions defined in the domain class but i did see it has a registry variable that looks like the anylisis folder build the data from. Should I pull the objects from the regisry?

How do I compute the unique name for the objects in the directive's add_target_and_index function?

In the handle_signiture function for hte objectdescriptions it uses the self.talon.registry.lookup_default function  to return an object called default and then runs the function desc_qualname and passes in its name attribute and a signode.

Where in the domain is the objects data stored?
- In the example the function `self.get_objects()` that is used in the `resolve_xref()` returns the data stored in the self.data, and that data is populated in an add_recipe function. I don't see an equivalent set of functions defined in the domain class but i did see it has a registry variable that looks like the anylisis folder build the data from. Should I pull the objects from the regisry?

You should use the typed API to grab objects from the registry:
https://github.com/wenkokke/talondoc/blob/71d88e22b83713ef417e03417085c1c29a32a180/src/talondoc/analysis/registry/__init__.py#L290-L315
This resolves a type and a name to an object or object group. You have to pass the type of the object as the first argument and the name as the second.

How do I compute the unique name for the objects in the directive's add_target_and_index function?
In the handle_signiture function for hte objectdescriptions it uses the self.talon.registry.lookup_default function  to return an object called default and then runs the function desc_qualname and passes in its name attribute and a signode.

I'd suggest using a combination of the name and the object's class name, e.g., action:user.auto_insert. That's enough to reconstruct the class and name, which lets you use the lookup function.


Removing unused thirdparty dependency - query-selector.

@tractorcow  - part of the Open Sorcerers internal backlog. This is already included as part of the thirdparty/jquery-entwine dist files.

Restarting tests.


DeprecationWarning: The binary mode of fromstring is deprecated, as it behaves surprisingly on unicode inputs. Use frombuffer instead 1.6rc0

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
Ubuntu 16.04
Tensorflow installed from source, latest HEAD from master (CPU)
1.6rc0
Python 3.5.2
Bazel 0.10.1
gcc-5

Describe the problem
When using tensorflow, I got hundred of DeprecationWarnings: This message exactly

/home/jerome/.local/lib/python3.5/site-packages/tensorflow/python/framework/tensor_util.py:560: DeprecationWarning: The binary mode of fromstring is deprecated, as it behaves surprisingly on unicode inputs. Use frombuffer instead
return np.fromstring(tensor.tensor_content, dtype=dtype).reshape(shape)

EDIT : fixed replacing line 560 in tensor_util.py by :
return np.frombuffer(tensor.tensor_content, dtype=dtype).reshape(shape)
Thanks for any kind of help :)

import warnings
warnings.simplefilter("ignore", DeprecationWarning)

I used this at the start of my code which suppresses the warning.

I assume this is a duplicate issue of #17020, which was fixed already.

Since the issue has been fixed in https://github.com/tensorflow/tensorflow/commit/f5c581f0f4649898ed00650fe98c7ef344e0f240. Let me close this one. Feel free to reopen if the issue persists. Thanks for the contribution!

Which version of tensorflow should I to fix this?

Follow the link by yongtang. There is says it is fixed since version v1.7.0-rc0.

import warnings
warnings.simplefilter("ignore", DeprecationWarning)

I used this at the start of my code which suppresses the warning.

where to use this particularly

import warnings
warnings.simplefilter("ignore", DeprecationWarning)

I used this at the start of my code which suppresses the warning.

where to use this particularly

before all the imports in your script file.


feat: add nix flake

Adding a Nix flake to the repo -- for now, it'll read the requirements.txt file and create an environment accordingly. Future versions could, for example, include openJDK, for when Java support is developed, and so on.

@Stamp9 Could you confirm whether every single dependency mentioned in requirements.txt is used? I'm asking because, for the flake to (currently) work, I commented out 4 packages which aren't in nixPkgs, ran the example script in the README (for MetaMask) and it worked. As such, I think we should investigate whether every single dependency here is needed, and remove the ones which aren't
BTW, I'll add back soon the assertion of python package versions in the flake, after we know every python dependency which is effectively needed (and their versions)

@Stamp9 Could you confirm whether every single dependency mentioned in requirements.txt is used? I'm asking because, for the flake to (currently) work, I commented out 4 packages which aren't in nixPkgs, ran the example script in the README (for MetaMask) and it worked. As such, I think we should investigate whether every single dependency here is needed, and remove the ones which aren't
BTW, I'll add back soon the assertion of python package versions in the flake, after we know every python dependency which is effectively needed (and their versions)

Hi, @randomicecube, thanks for checking! I re-generate the requirements.txt by using pipreqs, feel free to check.

@Stamp9 should be fine now, thanks!

👍

Hi, @randomicecube, I re-generate the requirements file in virtual environment by using pip freeze to fix issue https://github.com/chains-project/dirty-waters/issues/23#issuecomment-2433220443, please let me know if there's any issues, thanks!


Fix Catalog being empty after subsequent opens

Signed-off-by: Sebastian Malton<EMAIL_ADDRESS>

@Nokel81 could you describe the problem a bit and how this fixes it?

Yes, do the following on master:

Run make dev
View the catalogue
See kube sync items
Close window
Reopen window from tray
View the catalogue
See empty list

Yes, do the following on master:

Run make dev
View the catalogue
See kube sync items
Close window
Reopen window from tray
View the catalogue
See empty list

Haha, this is fun, you noticed this in my mobx6-migration branch, but it was broken in master :D
Btw, fix was just to add this:
  ipcMain.on(IpcRendererNavigationEvents.LOADED, () => {
    KubeconfigSyncManager.createInstance().startSync();

@Nokel81 do we really need all these changes for a fix? And why don't we target master branch for a fix?

@jakolehm No we don't really need all of these. I probably should have made two PRs.
As to why not targeting master? I am targeting future master as I would like get all these PRs in.

Just the bug fix is now in #2815 I'll keep this as a cleanup PR.

Closing this, will make a new PR with just the cleanups.


Compiling error

I am attempting to build this dependency for mod_sts and am receiving a compile error:
src/sts.c: In function ‘oauth2_sts_cfg_free’:
src/sts.c:240:3: error: implicit declaration of function ‘oauth2_cache_free’; did you mean ‘oauth2_cache_set’? [-Werror=implicit-function-declaration]
oauth2_cache_free(NULL, cfg->cache);
^~~~~~~~~~~~~~~~~
oauth2_cache_set
src/sts.c: In function ‘sts_cfg_set_cache’:
src/sts.c:410:49: error: passing argument 3 of ‘oauth2_cache_init’ from incompatible pointer type [-Werror=incompatible-pointer-types]
cfg->cache = oauth2_cache_init(cfg->log, type, options);
^~~~~~~
In file included from src/sts.c:27:0:
/usr/local/include/oauth2/cache.h:54:17: note: expected ‘const oauth2_nv_list_t * {aka const struct oauth2_nv_list_t *}’ but argument is of type ‘const char *’
oauth2_cache_t *oauth2_cache_init(oauth2_log_t *log, const char *type,
^~~~~~~~~~~~~~~~~
I'm following the build instructions from: https://gist.github.com/zamd/24e4dff10ac234162a43908334b2422b
On Ubuntu

I'm in the middle of updating master; you should use https://github.com/zmartzone/liboauth2/commit/8d829cc058ab40594e2f63e785bc92182b0e763d for now

@zandbelt, for mod_sts?

it should be working now with the master branches of mod_sts, liboauth2-sts and liboauth2

Thanks! I have it all compiled. About to start testing/using it.


Open up port 8080, for light hotspots

Untested! Just getting started

Think this is good to go. Port 8080 is being set and automatically set to the GRPC_PORT per #validator-ops channel. https://github.com/helium/miner/blob/master/config/val.config.src#L40
However when i try to telnet to that port i get no response.. maybe it's not activated yet?


Fix index page alignment

After the changes to grids, the index page is misaligned. This commit
fixes the alignment by removing padding that's unnecessary.
Before (desktop):

After (desktop):

Before (mobile):

After (mobile):

/cc @edds @fofr

LGTM :+1:


botId required for authorization, but users.info not immediately consistent after oauth.access returns

Description
Describe your issue here.
What type of issue is this? (place an x in one of the [ ])

[x] bug
[ ] enhancement (feature request)
[ ] question
[ ] documentation related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report
Filling out the following details about bugs will help us solve your issue sooner.
Reproducible in:
package version: 1.4.1
node version: 13.1.0
OS version(s): 10.15.1
Steps to reproduce:

implement a custom oauth handler
on oauth success, immediately try to fetch the users.info using the newly acquired bot.bot_access_token and bot.bot_user_id.

Expected result:
I expect to get back the bot user's info to get the bot_id from user.profile.bot_id.
Actual result:
I get a user_not_found response. Trying again a second later, it works.
Not knowing how it works under the hood, my suspicion is that this is an issue of eventual consistency. I may be hitting one server that is doing reads on an out of date database after the oauth server did the write.
My workaround is to lazily fetch the bot_id in the authorize function if it's missing, and then update the database. This could lead to database contention and is non-ideal.
Other thoughts:
I would prefer to get the bot_id in the oauth success handler alongside bot_user_id, and not need to do another call.
I would also like it made clear that botId doesn't need to be returned from authorize if you pass ignoreSelf: false in initialization (I think this is true but haven't tested it). I don't need ignoreSelf for my use case, so I could have saved myself a lot of trouble if I could have skipped this debugging session.
#196 is related to this, but they didn't mention the problem of not being able to immediately access this info.

I get a user_not_found response. Trying again a second later, it works. Not knowing how it works under the hood, my suspicion is that this is an issue of eventual consistency.

I've never experienced this but, as you guessed, it looks like a consistency issue on the server-side. I hate to say this, but Slack apps have to have a workaround like the one you did to deal with the problem.

I would prefer to get the bot_id in the oauth success handler alongside bot_user_id, and not need to do another call.

Indeed, it's better to have bot_id in the response. I don't have anything to share about this at the moment but will share this feedback internally.

I would also like it made clear that botId doesn't need to be returned from authorize if you pass ignoreSelf: false in initialization (I think this is true but haven't tested it). I don't need ignoreSelf for my use case, so I could have saved myself a lot of trouble if I could have skipped this debugging session.

I believe ignoreSelf: false should work as you expect.

@seratch thanks for the update -
so to clarify does replacing users.info call with auth.test solve the consistency issue?

@pertschuk I believe so. As I haven't reproduced your situation yet, I'm not 100% sure though. auth.test should behave more firmly.

@seratch so I had this issue some days back and the comment here helped me, auth.test would actually send the bot_id as part of the object returned, is it possible to update the documentation to reflect this, I have seen more than 4 issues around authorization, this , this , this and some others,  I feel the details on here is not clear enough, after struggling for days with help from the issues above, I would like to make this contribution if I can be pointed to the direct direction, I can open a pull request to make the authorization flow clearer.
Also, this is my first time here so if it is inappropriate to point this out here, I can open a separate issue for it.

Bolt v2.1 came out with its built-in OAuth support almost three weeks ago.

https://github.com/slackapi/bolt-js/releases/tag/%40slack%2Fbolt%402.1.0
https://slack.dev/bolt-js/concepts#authenticating-oauth

The module provides a much smoother way to implement OAuth flows so that developers no longer deal with the details mentioned in this issue.
If you have feedback on the OAuth module, let us know in a new GitHub issue! Allow me to close this issue now.

@seratch Can you please expose Installation interface which is required by fetchInstallation ? There are a lot of interfaces that some methods require which are not exposed by the bolt sdk :/ This forces typescript users to cast to any. The View interface is a good example of this.

@realappie that is a good suggestion. Can you create a new issue for exporting Installation? For now, you could import the Installation interface from the @slack/oauth package directly.

@realappie I have a PR open where I am exporting Installation in bolt-js. https://github.com/slackapi/bolt-js/pull/586


Requirement for the P matrix

Hi,
On the website , I notice that it is required for the P matrix to be upper triangular when we use the "osqp_update_P" function. I have two questions:

Does the osqp solver only work for upper triangular P matrix?
What should I update my P matrix in the OSQPWorkspace if it is not upper triangular?

Thanks,
Hanxiang

If I use "DenseToCSCMatrix" to convert P from Eigen::MatrixXd to c_float*, is P already satisfied the input requirement?

Please use our forum  for questions on how to use OSQP.


AlienFx Crashes to Desktop

AlienFx Crashes to Desktop when on COLORS page, Add ZONE, SELECT FROM LIST. After selecting keys and then hit APPLY, crash occurs.

Confirm. Looks really strange, WHERE it really happened. Will fix ASAP.

Fixed - re-download last release.

Works Great, sir. Thank you.


sypder support bookmarks?

From<EMAIL_ADDRESS>on 2014-03-20T01:32:29Z
I use pyscripter before, it can shift + ctrl + num set bookmarks and ctrl + num go to bookmark, very convenient, can sypyder also add such features?
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1759

Do we want to implement this at some point @ccordoba12?

It can really useful! Just leave this open in case someone else wants to implement it.


Legion unit composition

Move on from just building all the Legion units to actually building them in a smart way

Naval platoons will need to be completely overhauled as they make assumptions about roles that Legion doesn't adhere to.

Naval platoons overhauled. More work will be required, and vanilla could probably use a polish but more test data will be needed for how to respond to Legion

Some further tweaks have also been made to the land comps.

I think there's too much diversity in the groups. Needs to be pickier about what to use and have some real focus.

Experimentation reveals that Legion does better when excluding Lancers from land attacks and possibly keeping the Peacekeepers excluded from land attacks too.

Tightening up the T2 choices to differentiate between specialised and frontline. Minimum requirements now in place.

Updated vanilla naval to account for Legion

Happy enough with unit comp now