Disable static result should be reworded to Enable static result

Describe the Bug with repro steps
Navigate to Testing tab of an action. Notice it says Disable static outputs with toggle off which gives a sense that it is enabled by default (not true). Lets follow v1 designer pattern and change the wording to Enable static outputs.
What type of Logic App Is this happening in?
Consumption (Portal)
Are you using new designer or old designer
New Designer
Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg
Yes
Workflow JSON
No response
Screenshots or Videos
No response
Browser
Chrome
Additional context
No response
AB#25408898

Will be fixed in our 10/27 release


deja-dup: 40.4 -> 40.6

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/deja-dup/versions.
meta.description for deja-dup is: '"A simple backup tool"'.
meta.homepage for deja-dup is: '"https://wiki.gnome.org/Apps/DejaDup"

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 40.6 with grep in /nix/store/si24h98c4i9x7snj1i5v7q5c05rbmz8l-deja-dup-40.6

directory tree listing: https://gist.github.com/978cc19edd2cce535d24b9006bf9ccdf

du listing: https://gist.github.com/ace769068d49d659cc0c1a933fbd67f9

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)
1 package rebuild(s)
1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
deja-dup

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/si24h98c4i9x7snj1i5v7q5c05rbmz8l-deja-dup-40.6 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A deja-dup https://github.com/r-ryantm/nixpkgs/archive/04ff52b772b6d873193be62a98ff34d250c2f3ec.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/si24h98c4i9x7snj1i5v7q5c05rbmz8l-deja-dup-40.6
ls -la /nix/store/si24h98c4i9x7snj1i5v7q5c05rbmz8l-deja-dup-40.6/bin

cc @jtojnar @joncojonathan for testing.

Changes:

https://gitlab.gnome.org/World/deja-dup/-/tags/40.5
https://gitlab.gnome.org/World/deja-dup/-/tags/40.6


Strange Behaviour for MoveMouseBy

Hello! I seem to be having some issues with the Mouse.MoveMouseBy method. The method seems to move the mouse cursor by about three times the values passed to the deltaX and deltaY parameters. Using a screen ruler and calling:
InputSimulator.Mouse.MoveMouseBy(1300, 0);

meaning 1300 pixels on the positive horizontal axis results in the mouse being moved all the way across the screen which is much more than 1300 pixels. This holds similarly true for the vertical axis:
InputSimulator.Mouse.MoveMouseBy(0, 300);

where the mouse ends up approximately somewhere beyond 900 pixels on the positive vertical axis.
For testing I have used a screen ruler that seems to be correct since I have made it extend all the way across my screen resolution on both horizontal and vertical axes and it fits perfectly, as well as the following simple code:
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
using WindowsInput;

namespace InputSimulatorPlusTest
{
    public partial class Form1 : Form
    {
        private readonly InputSimulator _inputSimulator;

        public Form1()
        {
            InitializeComponent();
            _inputSimulator = new InputSimulator();
        }

        private async void Form1_OnShown(object sender, EventArgs e)
        {
            await Task.Delay(10000);
            _inputSimulator.Mouse.MoveMouseBy(0, 300);
        }
    }
}

Is this a known issue, am I missing something and, if it is a bug, could it be fixed? Thanks!

For others passing by, the problem lies with the global mouse pointer acceleration. MoveMouseBy is applied before any mouse acceleration settings get applied such that the pointer will end up farther than what one would expect. You should look into setting a.) a default mouse acceleration and b.) disabling the enhanced pointer precision.


feat: add RockPi to config generator

Why

Support RockPi with custom config template that uses correct i2c bus i2c-7
How

add i2c-bus to config template
add rockpi env variable for generating different configs
add logic for populating i2c bus in template
update staging and production actions workflows to generate docker.config.rockpi and save to storage bucket
update test fixtures to add new i2c bus parameter
change test coverage check to 80%
update unit test to add i2c bus check
update README to add info about truncate and RockPi

References

Relates-to: NebraLtd/hm-miner#53
Relates-to: NebraLtd/hm-miner#31

Hmm seems I broke something @robputt 🤔


VirusTotal detects Trojan.PSW.Python.bp in ChronoPlotter.exe

Bringing this to your attention, in case you are not yet aware.
Reference: https://www.virustotal.com/gui/file/8bf43a3416b52ae0cc37e9032ccf4fd8598d3461656566d6675967d1c1a7ec22/detection

Thanks for the heads up. Yeah, anti-viruses haven't been playing too well with PyInstaller. I'm currently building the PyInstaller bootloader from source with each release (instead of using the prebuilt one) which has significantly helped the issue. Hitting zero false positives is on the list though, either through further code changes or maybe Windows code signing.

Cool. Perhaps this may help?
https://stackoverflow.com/questions/43777106/program-made-with-pyinstaller-now-seen-as-a-trojan-horse-by-avg/52054580#52054580

Thanks, yup I'm currently doing that actually https://github.com/mncoppola/ChronoPlotter/blob/main/azure-pipelines.yml#L41

Just an update, I'm mid-way through rewriting ChronoPlotter from the ground-up in C++. This will drop the PyInstaller dependency which will be a huge benefit (for a variety of reasons, including anti-virus false detections). It'll also make it easier to perform Windows code signing, which will additionally help with SmartScreen warnings on recent Windows.

I just released v2.0.0 which brings a full project rewrite in C++. This drops PyInstaller completely, which should solve this issue.


Add about menu

Similar to https://github.com/mueller-ma/PrepaidBalance/commit/9cc583cf375bd71fe904f941c0c8ace54ec6c96b

Hi, I've created a PR for this issue. #169


Add phenotyped file for testing downstream

This file works to test SpatialLDA , this should solve the issue #1130 @chiarasch

@kbestak  could you review?


Units of Measurement

First of all thanks for your awesome work.
I'd like to use 3DDFA in conjunctions with other models and I would need to convert the 3D models to a common system of reference. My problem is the following: I can't find any explanation of the units of measurement of the coordinates of the 3D models produced by 3DDFA. I've looked both in the papers and in the code, but the only seemingly related function I was able to find is similar_transform in utils.tddfa_utils.py, which I can guess makes a rescaling based on the bounding box size. I've tried turning it off, but the resulting scale of the model still doesn't match with meters, cm, ms, feet or inches.
The first paper mentions NCC, but they aren't definitely what's returned by TDDA.recon_vers(). There's also a scale factor f involved in the projections, but I can't figure out how to use it.
Could you help me clarify this issue?

3DDFA uses orthographic projection and the reconstruted mesh is aligned to pixel space.
You may use ICP or other methods to do the transformation.


Add 'annot_col' parameter to highest_expr_genes plotting function

The annot_col parameter allows to specify a column that will be shown on the y axis of the highest_expr_genes plot instead of var_names.
This is useful when working with ENSG identifiers as var_names and you have stored gene names in a separate column.

@falexwolf: Honestly, we should prepare and ship a scanpy 2.0 where all functions are converted to almost keyword-only. In this case there’s only one central important parameter apart from adata, so a * should be after n_top:
def highest_expr_genes(
    adata: AnnData,
    n_top: int = 30,
    *,
    show: Optional[bool] = None,
    ...
    **kwds,
)

Thanks for the PR.
One concern that I have is that similar solutions do not exists for other plotting functions. For coherence, ideally the annot_col argument should be available for other cases.
Thus, I think that a better and more generic approach would be to simply modify your genes names in the AnnData object and let all plotting functions use those names. For this, you simply do:
adata.var = adata.var.reset_index().set_index(annot_col)
# adata.var_names is automatically updated

Good point @fidelram. If you don’t want to modify your adata object permanently:
ad_plot = adata.copy()
ad_plot.var = adata.var.reset_index(drop=True).set_index(annot_col)
sc.pl.highest_expr_genes(ad_plot, ...)
del ad_plot

Is this accepable?

@fidelram Actually, a similar solution does exist for other plotting functions. @falexwolf added the gene_symbols parameter to sc.pl.rank_genes_groups() and sc.pl.rank_genes_groups_violin() as I was having the same issue of using Ensembl gene IDs. I'm not sure if he regrets doing that now though or not... it's not consistently applied in either case.
In either case Ensembl gene IDs are more specific than HGNC/MGI gene symbols. For some users it may be important that this specificity is available (and you don't keep having to make copies of the data to plot). Also, if alternative splicing is investigated with single-cell data, then you cannot keep gene symbols as var_names throughout your dataset as a default. So this is a future-proof approach.

In this case it should probably be called gene_symbols as well, even though that’s not the best name for the parameter, just for consistency.

@flying-sheep, thanks for catching the argument positioning!
@fidelram, I definitely want to keep ENSG as main identifiers. As a related side note, why is actually everyone using gene symbols for single cell data? Though I'd rather have this supported by the plotting function, I could live with the workaround proposed by @flying-sheep.
@LuckyMD, thanks for pointing this out, I renamed the argument to gene_symbols for consistency.

@flying-sheep I completely agree that it may not be the best naming convention. That was what it was used for at the time though.
@grst The idea of using gene symbols as .var_names in scanpy was to make the software as user friendly as possible as far as I'm aware. Typically people care about the gene-level as the highest resolution. This is probably also due to the dominance of 3` enrichment protocols making it hard to detect signals at a higher resolution.

As a related side note, why is actually everyone using gene symbols for single cell data?

They are? They should really stop doing that, it’s a bad idea! I thought the field had moved past that long ago!
Looks good as it is!

I'm not sure if he regrets doing that now though or not... it's not consistently applied.

Yes, we should have some consistent way of adding this parameter. The best way would be an automatized approach that added the parameter to all plotting functions, maybe a decorator that effectively does this:
ad_plot = adata.copy()
ad_plot.var = adata.var.reset_index(drop=True).set_index(annot_col)
sc.pl.highest_expr_genes(ad_plot, ...)
del ad_plot

without creating a copy of the full data, though. A view would be much better. Some thoughts would be required...
I think the name gene_symbols for the parameter is OK.
I think that using gene symbols in the data analysis scripts for single-cell is also fine. Convenience is important. But for sure, we don't want to keep people who want this to be treated more rigorously from doing it.

I have an idea how to fix this without breaking everything: I think I can build a decorator that allows people to use parameters positionally, but sends a DeprecationWarning when they do, while the docs only report the new signature.

This sounds extremely sensible to me. I didn't actually want people to call things with positional arguments.

Let’s pay attention to not include any new function without * anymore, OK?

OK!

Oh it’s definitely OK! I meant that it needs a bit of imagination to put other things in there than actual gene symbols, but I feel like it’s faster to see!

Yes, but I hardly imagine that you need two layers of annotations for variables outside the specific case of ENSids and gene symbols. So it's good to be verbose about this specific case and people wont miss such functionality in other cases.


Add FOV circle

So that we can see preview what the aimbot looks like
https://i.imgur.com/YSSTsQZ.jpg

https://github.com/danielkrupinski/Osiris/issues/310
Here ya go.

+++1


navigator is undefined error

I get an error navigator is undefined in 5.2.0 and 5.1.x at this line
var isEdge = navigator.userAgent.indexOf('Edge') !== -1 && (!!navigator.msSaveBlob || !!navigator.msSaveOrOpenBlob);

In particular, 5.1.0 works.

Should be fixed now.


[UI > Settings] Fix You searching

also removed the youKey snakeCase mapping because it seems unnecessary (correct me if i'm wrong though)

I'm pretty sure they did it so that it was backwards compatible with panels UI, so please ensure panels UI pages still work

@acquitelol just checked, still works

@acquitelol just checked, still works

slayyy


Endianess issue?

Hi
when looking on your library I noticed that for CRC16 (ARC in your standards enumerator), the library returns {0x3D,0xBB} when inface it should be reversed {0xBB,0x3D}.
look at http://www.lammertbies.nl/comm/info/crc-calculation.html and other online calculators on the web. They all produce 0xBB3D and yours is the only one producing the wrong endianess

I'm afraid you are misunderstanding byte endianess and the way that hexadecimal values are expressed.
When we write the value 0xBB3D, we are expressing them in big-endian as that is how we naturally express numbers in everyday life -- 0xBB3D == 47,933 dec, just like how 0x003D == 61 dec.  However, most modern-day, laptop/desktop processors work in little-endian.  What this means is that when you read the value 0xBB3D as individual bytes that the computer stores, you'll read it as { 0x3D, 0xBB }.
To put it programmatically, run the following code:
...
var crc = new System.Data.HashFunction.CRCStandards.ARC();
var bytes = crc.ComputeHash(System.Text.Encoding.UTF8.GetBytes("123456789"));
var shortValue = BitConverter.ToUInt16(bytes);
// Should print BB3D
Console.WriteLine(shortValue.ToString("X"));
...

Hi Brandon,
I apologize for not explaining myself correctly (probably a moment of stupidity towards the weekend :-) ). What endianess does the library produce? is it always little? big? or does it vary based on platform? In order for me to compare the CRC the library produces to a CRC provided in a byte stream that has been coded using big endian I need to know the order to compare the bytes.

The CRC hash functions are somewhat unique because they are bit-wise hash functions instead of byte-wise hash functions.  The bit-endianness of the bytes will be whatever the endianness of the machine is, but that shouldn't matter because the only thing that should matter is the order of the bits.
Since they are bit-wise, the standards specify the endianness that input bytes should be processed and the endianness of the outputted bytes.  This is controlled by the reflectIn and reflectOut fields in https://github.com/brandondahler/Data.HashFunction/blob/master/CRC/Standards.json .  Since ARC specifies reflectIn and reflectOut as both true, it should process and produce a bitstream that is big-endian.
The good news for you is that this is already handled and you'll find that the bitstream produced by any other endianness machine will produce the same bitstream as you will receive if you do the following:
...
var crc = new System.Data.HashFunction.CRCStandards.ARC();
var bytes = crc.ComputeHash(System.Text.Encoding.UTF8.GetBytes("123456789"));
var bitArray = new BitArray(bytes);

// Should write<PHONE_NUMBER>111101
foreach (var bit in bitArray)
    Console.Write((bit ? 1 : 0));

Console.WriteLine();
...

The only thing you might need to be concerned with is the bit-endianness of the bytes when you transmit bytes between the systems -- either the sender needs to convert its bytes from its native endianness to the receivers endianness, or the reciever will need to convert the recieved bytes from the other's endianness its endianness.
Reguardless of the endianness of any of the machines, casting the bytes to a short on the producing machine and printing them out should produce the decimal value 47,933.


Cache filter function

This PR should be rejected since it contains breaking changes in colly.CacheDir. However, i just wanted to demonstrate how it works at the moment.

Useful functionality, thanks! Could you modify it to be an independent option from CacheDir? Then it wouldn't break the current API.

Something like colly.CacheDirWithFilter ?

Ok, i'm gonna need some help on dealing with my commits. My first commit on this PR kind of conflicts with latest one - i have tried squashing it, but i had to merge after that on my fork. How can i fix this in some nice way? I would go with fresh PR, but it seems like good learning moment ;-)

Why do you want to use reflection for testing? I think, a simple test case would be enough:

Setup a custom CacheFilter
Create two requests where one is allowed by the CacheFilter and the other one isn't.
Check if the cache file created for the cached request.

Another thought:
It would be good to support every type of requests, not only GET. We can achieve it if we define a default CacheFilter function for a collector instead of doing additional filtering in httpBackend.Cache.

@krzysztofantczak are you able to continue this PR?  Looks like good progress was made :+1:

@krzysztofantczak, this feature is really appreciable one, so +1 to @vosmith.

Closing due to inactivity.


PagedListView not working as expected inside CustomScrollView

Current behavior: when scrolling within the PagedListView the SliverAppBar will not collapse / expand. It only collapses / expands when scrolling within the SliverAppBar itself.
Expected behavior: the SliverAppBar expands / collapses correctly when scrolling anywhere on the screen.
If there is no TabBarView I could just add PagedSliverList, but that's not really an option here I think.
Scaffold(
        body: CustomScrollView(
      slivers: [
        SliverAppBar(),
        SliverFillRemaining(
          child: Column(
            children: [
              TabBar(
                controller: _controller,
                tabs: [Tab(text: "latest")],
              ),
              Expanded(
                child: TabBarView(
                  controller: _controller,
                  children: [PagedListView],
                ),
              )
            ],
          ),
        )
      ],
    ));

Current behavior:
https://user-images.githubusercontent.com/13279066/110826675-9bc44000-8295-11eb-8611-e5f4a5ad4668.mp4
Expected behavior:

Hi @Martibis .
Can you try replacing the PagedListView with a regular ListView with fake items?
That way we can know for sure if it's something with the package or not.
Thank you

Hi @Martibis .
Can you try replacing PagedListView with a regular ListView with fake items?
That way we can know for sure if it's something with the package or not.
Thank you

With a ListView I didn't have the issue, but I figured out why and it's not the package I think (just weird Flutter behavior). When I was using a PagedListView I was using a ScrollController in there so I would be able to scroll back to the top on a button press. Having a Scrollcontroller in there is what was causing this issue!

I want to implement similar behavior with PagedGridView.
Flutter automatically provides the behavior if you have SliverList or SliverGrid  inside CustomScrollView.
Anyone created this behavior?

im experiencing the same problem has anybody a solution

https://github.com/EdsonBueno/infinite_scroll_pagination/issues/47#issuecomment-775796276


Do not format the node definition JSON file after converging

It conflicts the format rule with other format tools, prettier, for example. The inconsistency is annoying.

it has to update the format, and it uses the format originally defined by the yajl-ruby project and used for the past 10 years in chef as being authoritative


Support for tables

Seems quill.js (our template editor) doesn't support html tables at all. We will not have the support for tables until Quill adds them unfortunately.
Related: https://github.com/quilljs/quill/issues/117

+1

+2

Since Quill is not moving here much let's try to find an alternative editor. @thomasgit123 will have a look at it.

Done in #265


Add PartialMerkleTree::calc_tree_height method

PR 3 in the merkle_tree::block series, builds on top #1388. This PR is the last 2 patches.

Patch one adds unit tests for the PartialMerkleTree::calc_tree_width method
Patch two adds a method calc_tree_height and reduces code duplication. No unit tests added because, now we are confident calc_tree_width is good calc_tree_height is trivial.

Rebased on top of #1388, no changes to the last 2 patches which make up this PR.

Seems no one is that keen on this one. Closing

Oh woops, its just that we have to wait for https://github.com/rust-bitcoin/rust-bitcoin/pull/1388


Move List test to use namespaced PHPUnit TestCase

This class throws an exception on PHPUnit 6 currently. All other test classes use the namespaced TestCase already.

Codecov Report

Merging #1324 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1324   +/-   ##
=======================================
  Coverage   74.84%   74.84%
=======================================
  Files          35       35
  Lines        4775     4775
=======================================
  Hits         3574     3574
  Misses       1201     1201

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 dadb88c...8fd0700. Read the comment docs.


Update LIS documentation

Fixed a wrong unit entry for the "forcing_variables.adoc" file
Added "Openwater" option for surface model types (may have been forgotten in the past)

Note: No issue was opened for this commit, since minor updates and easy to compare and merge.

Hi @karsenau
Are these errors also in the support/lisf-public-7.3 branch?  If so, I think this should go into that branch rather than master.

Hi @emkemp,
Yes, these omissions and errors are also in support/lisf-public-7.3 branch.
Should I check out a fresh support/lisf-public-7.3 branch and push the commits / PR there?
Thanks!

Hi @karsenau
Yes, please checkout support/lisf-public-7.3, change the appropriate files, and make a new PR.  That way only these specific changes will be marked by git for merging.
@jvgeiger will then migrate these back into master, covering both branches.

Hi @emkemp,
I have added and pushed the new commits via the support/LIS 7.3 branch (from a checkout from the support/LIS7.3 branch, with PR #881.
So if you want to go ahead and delete this pull request, or please guide in the best method to undo these commits and PR.
Thanks!


Discuss ways to avoid new Function in denodeify

I'd like to lockdown new Function and eval in favor of vm and other code execution mechanisms that can be checked via require or import.
This module seems to use them to create .length preserving wrapper functions that bridge callback code and Promises.
https://github.com/then/promise/blob/d980ed01b7a383bfec416c96095e2f40fd18ab34/src/node-extensions.js#L47
I think the semantics of denodeify could be implemented without runtime code generation using tricks like the below:
const { apply: rapply, get: rget } = Reflect;

const handlerForArgumentCount = [];

function denodeify(fn, argumentCount) {
  if (typeof fn !== 'function') {
    throw new TypeError(typeof fn);
  }
  if (argumentCount !== (argumentCount >>> 0)) {
    argumentCount = fn.length;
  }
  if (!handlerForArgumentCount[argumentCount]) {
    handlerForArgumentCount[argumentCount] = {
      get(target, prop, receiver) {
        if (prop === 'length') {
          return argumentCount;
        }
        return rget(target, prop, receiver);
      },
      apply(target, thisArgument, argumentsList) {
        return new Promise((rs, rj) => {
          const args = [];
          for (let i = 0; i < argumentCount; ++i) {
            args[i] = argumentsList[i];
          }
          args[argumentCount] = (err, res) => {
            if (err) {
              rj(err);
            } else {
              rs(res);
            }
          };
          const res = rapply(target, thisArgument, args);
          if (res && (typeof res === "object" || typeof res === "function")
              && typeof res.then === "function") {
            rs(res);
          }
        });
      },
    };
  }
  const handler = handlerForArgumentCount[argumentCount];
  return new Proxy(fn, handler);
}

Would you be interested in patches that avoid using new Function so can run on runtimes that turn that off?
I notice you check compatibility with
https://github.com/then/promise/blob/d980ed01b7a383bfec416c96095e2f40fd18ab34/.travis.yml#L4-L7
How tolerant is this project of feature detecting or loading different -impl.js files based on the availability of features like Proxy?
What would you need to see in terms of performance measurement before making a decision?

I'm not super keen to rely on Proxy, and from what I recall, the primary reason we use new Function is actually for performance.  In the case where we know the number of arguments up front, we can generate more optimised code that does not need to copy the arguments pseudo array.
One option might be to add a few prebuilt functions for 0, 1, 2, 3 and 4 arg functions.  We could then avoid the runtime code generation until faced with a function that takes a large number of arguments.
In terms of performance, I'd want to see us doing the same or better in all the bluebird benchmarks https://github.com/petkaantonov/bluebird/tree/master/benchmark
In terms of having multiple versions, we so have a version for "domains" support and a version that uses "setImmediate" directly instead of asap. These are generated using the build.js script. We could add another output for content-security-policy compliance, which could either use your Proxy approach, or just skip enforcing the length of returned functions.


Merge branch v3.0.0 into master

Reopen #131 (changes have already been reviewed in #122 and #124). Work is already done, but we decided to address #130 before.
Close #108
Close #116

LGTM


Huggingface API Compatibility Inquiry

I am curious to know if it is possible to utilize custom models or Huggingface model API endpoints through a custom API url similarly to how the Pydantic program handles Openrouter API calls as demonstrated in the "Models" section of the Pydantic documentation. Thank you for your time and consideration.

See #398, sounds like it should be possible soon once we work around a bug in hugging face's TGI project.


[v2] fix @font-face rule support

This PR :

add tests for the problem explained into #493
upgrade stylis to v1.1.3
do a minor fix on another test that was no longer valid after stylis upgrade

Maybe the test is superfluous because there is already one into stylis.
I just add one to validate that the problem is here before, and is gone after the upgrade...


uv pip freeze creates color in output

Issue
When tox calls the freeze routine, uv is placing color escape sequences in the output, resulting in the dependency list containing color escape sequences and breaking the constraints text file used in subsequent steps.
Environment

OS: macOS Sonoma 14.3.1
Terminal: kitty, tmux, zsh with True Color support

Minimal example
py38-mindeps: freeze> /Users/dporter/.pyenv/versions/3.11.8/bin/uv pip freeze
py38-mindeps: install_package_deps> /Users/dporter/.pyenv/versions/3.11.8/bin/uv pip install 'asyncssh<3,>=2.13.1' -c${project}/.tox/py38-mindeps/constraints.txt
error: Unexpected 'xpected '-c', '-e', '-r' or the start of a requirement at ${project}/.tox/py38-mindeps/constraints.txt:1:1

$ file .tox/py38-mindeps/constraints.txt
.tox/py38-mindeps/constraints.txt: ASCII text, with escape sequences

This doesn't make sense. This freeze is not used for install. Is only for user consumption.

This sounds like an uv bug similar to astral-sh/uv/issues/2017


Row order in observation matrices - ability to lock

This is tangentially related to the bug I posted a while ago (#1881), which has been closed.
Row order can be very important to the user, so please include an option to lock the user-defined row order from "edit observation matrices".
Or simply disable in observation_matrices/view, the ability to resort rows by clicking on OTUs.

@jlpereira I would suggest we keep the sort locked by default (like comprehensive form), and click to unlock.

@jlpereira in both cases, here in #1881. The observation_matrix_rows.position should be used as a primary sorting mechanism when the rows are displayed.

Already changed the order in edit/new matrix task, reopen if the problem persist


login security messed up boxes

for some reason this happend when I use the login security plugin when I remove it it gets fixed

I doubt this is related to LoginSecurity. the plugin does not change anything related to name tags


HAR Redactor

What feature or new tool do you think should be added to DevToys?
Analyze a HAR file for secrets and replace sensitive secrets with the word REDACTED.
Why do you think this is needed?
When working with various cloud companies, they frequently request HAR files to help debug the issue. Users unknowingly send in those files insecurely or they are stored in a requestor's unsecure system. This can lead to compromised credentials and leave a company open to security breaches. A best effort redacted HAR file reduces vulnerabilities but still allows a HAR file to be useful in assisting debugging complicated web troubleshooting.
Solution/Idea
Things that could be detected and redacted:

The signature of a JWT
Known/common query string parameters that contain a secret
Known variables used for Azure/AWS IAM based API access

Comments
No response

Closing as no upvote/traction on this issue. This tool be made as an extension using our SDK: https://devtoys.app/doc


Add drag & drop of links onto bar to save new links

Makes for a better user experience than manually editing in the settings

You already had this working, then I broke it and had to recreate it, but it's working again 👍

This is included in the current release (1.3.1) so I believe this issue can be closed.


V0.1.2

Codecov Report

Merging #11 (6337f11) into main (67e2b64) will increase coverage by 12.99%.
The diff coverage is 30.36%.

@@             Coverage Diff             @@
##             main      #11       +/-   ##
===========================================
+ Coverage   17.37%   30.36%   +12.99%
===========================================
  Files           4       11        +7
  Lines         541     1505      +964
===========================================
+ Hits           94      457      +363
- Misses        434     1017      +583
- Partials       13       31       +18

Impacted Files
Coverage Δ

cmd/commands/common.go
0.00% <0.00%> (ø)

cmd/commands/root.go
0.00% <0.00%> (ø)

cmd/commands/version.go
0.00% <0.00%> (ø)

pkg/git/provider.go
0.00% <0.00%> (ø)

cmd/commands/repo.go
2.39% <2.39%> (ø)

cmd/commands/app.go
20.27% <20.27%> (ø)

pkg/application/application.go
42.95% <42.95%> (ø)

pkg/git/provider_github.go
43.54% <43.54%> (ø)

cmd/commands/project.go
66.40% <66.40%> (ø)

pkg/git/repository.go
66.43% <66.43%> (ø)

... and 12 more

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 c2439f4...6337f11. Read the comment docs.


Project: separate out developmental precursor cells from stem cell heirarchy

There is a common pattern in CL in which adult cell types appear to arise only
from stem cells, often ones only found in adults. i.e. - developmental origins
are ignored.  The whole stem/precursor cell hierarchy needs to be reviewed with
this in mind.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 16 Jul 2014 at 10:04

WARNING: This issue has been automatically closed because it has not been updated in more than 3 years. Please re-open it if you still need this to be addressed – we are now getting some resources to deal with such issues.


Dev

Oops, was testing webhooks on my fork and accidentally opened a PR to your repo instead of mine. Please disregard :)


Docs: Update user quick-start readme and fix scripts

Motivation and Context

Why is this change required?  To make onboarding smoother.
What problem does it solve?  The quick-start local deployment steps were difficult to follow.
What scenario does it contribute to? Beginners trying out Chat Copilot locally.
If it fixes an open issue, please link to the issue here. Unknown.

Description
Main Chat Copilot Readme is updated to present a quick-start beginner use case.  This is the local deployment of Chat Copilot, not an Azure deployment.  It removes the redirect for the scripts, and formats instructions to improve readability.
Bash and PowerShell scripts are also fixed to use environment variable ASPNETCORE_ENVIRONMENT, which is necessary for appsettings.json override (by appsettings.Development.json) when running the backend for OpenAI.
Contribution Checklist

[ ] The code builds clean without any errors or warnings
[ ] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[ ] All unit tests pass, and I have added new tests where possible
[ ] I didn't break anyone :smile:

OpenAI and Azure OpenAI with Windows tested. Works.
Linux to be tested.

testing on linux and macos

closed in favor of PR #94


Update PropsEditor.js to use 'prop-types' package

First, great work on this package!
In my search to see which dependencies were still using React.PropTypes (and causing React 15.6 to yell at me), I found this little guy was behind the times. PropsEditor.js (and this package in general) isn't causing any warnings, but I figured it couldn't hurt to bring this last file into the modern React era 😄

[X] Import prop-types into file and replace React.PropTypes with PropTypes

Thanks for finding this and pushing a fix 👍
Cheers !


Fix mail commands

Do not merge yet.
Closes #70
Closes #97
Closes #105
Closes #126
Closes #127
Closes #129

Closed as this fix is not allowed for v1.4.


[Library Manager] please add TSC2007 sensor library to the Arduino library manager

Please add this repository:
https://github.com/adafruit/Adafruit_TSC2007
Thanks!

Hi @evaherrada. We have a new system for submitting libraries. The instructions are here:
https://github.com/arduino/library-registry#adding-a-library-to-library-manager
Please let me know if you have any questions.
Regards, Per


compiler: Added prefix and suffix support for Char

As discussed in #1734, here's the PR for the support of prefixed characters.
For now only u and a are supported, respectively to get the code point as an int and the ascii code as a byte.

factorize code
missing interpreter
use b for Byte prefix: b'z' is Byte, b"zz" will be Bytes

Fixed @privat's remarks

+1

Fixed syntax errors

+1


Audit package names and coherency parents

We should go through all of our Version.Details.xml entries and make sure we are using correct package names and coherency parents to prevent issues like https://github.com/dotnet/source-build/pull/1518.

[Triage]  This work was completed.


update to vs2017 hotness

@brockallen,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

thanks!


[FEATURE] Add forwards and backwards tabulation with Ctrl+Alt(Opt)+?S…

…hift

Can one of the admins verify this patch?


requirements command not finding utilities

When I run ./cli/build/install/ort/bin/ort requirements certain items are not found, but I am confident they are on the system. Can you help by describing how the algorithm for finding these tools works?
- ScanCode: Requires 'scancode' in no specific version. Tool not found.
Scancode is definitely installed and running.
- Pip: Requires 'pip' in no specific version. Tool not found.
- Pipenv: Requires 'pipenv' in version >=2018.10.9. Tool not found.

These work when invoked directly as pip or pipenv, but are not found by which.

but are not found by which.

That's the key point, ORT basically uses the same approach as which by simply looking at the PATH environment variable.
Things to check:

Maybe the PATH environment is different for the user running ORT than for the user you're running interactively. Maybe even a completely different shell is used.
Maybe your pip and pipenv are shell aliases, not executable files.

Feel free to reopen if you need further help, @lucasgonze.


Improve CI

Enable rust backtrace for better err debugging experience
Turn warnings into errors
Disable incremental build since CI always builds from scratch
Use sparse crates.io registries
Rm use of unmaintained actions-rs/toolchain
Speedup cargo-clippy with --no-deps
Use Swatinem/rust-cache@v2 for caching
Add workflow msrv.yml for checking msrv
Add workflow minimal.yml for checking with minimal version of deps
Fix shellcheck errors in run_tests.sh

Signed-off-by: Jiahao XU<EMAIL_ADDRESS>

I tried to re-enable miri, but it really takes too long.
We need to write tests specifically for running on miri, the existing code uses rayon and it is just too slow.
@silver-ymz Would you like to try adding this?

Ok. I'd like to try it.


docs: Fix a few typos

There are small typos in:

doc/internal/testing.rst
doc/programming_guide/time.rst

Fixes:

Should read imprecision rather than inprecision.
Should read decorated rather than docorated.

Semi-automated pull request generated by
https://github.com/timgates42/meticulous/blob/master/docs/NOTE.md

Thanks for that!


Added create a product

Added code and test cases to create a product for model and service.

Forgot to mention I also added SKU to the product table schema, since we had that in the original story requirements.


Fix vscode.commands.executeCommand() documentation

As noted in #80338, when executing a command that is contributed by an extension
that is not yet loaded results in serializing all arguments. Only the types
allowed to be passed to editor commands and JSON-serializable objects are
guaranteed to survive being passed to another extension.

Great! I like that solution much better.


sage: remove duplicated pkgconfig

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

This broke sage. The pkgconfig you removed lives at pkgs/development/python-modules/pkgconfig, it's not the alias to pkg-config.

By the way, the commit that added the "duplicated" pkg-config line explicitly says that sage breaks without it: https://github.com/NixOS/nixpkgs/commit/d2c4b5777ba22a51dcdc5f5a97e9d8e72e00d860


Include selection flags from dbt

We still need to
closes #86

[ ] https://github.com/fal-ai/fal/issues/86#issuecomment-1012398594
[ ] add information to readme
[x] tests
[ ] match flag format from dbt  (--select one two three vs --select one --select two --select three)

Done with tests @mkam039


ci: Add basic Pull Request checks.

This PR is intended to set up basic build / linting target for CI.
Some small naming changes may be required to get the build to work-- I'll update the PR as needed for that.

I'm going to add a basic smoke test check to this PR, so that we can make sure that our usage example more-or-less works.

@chendrix We finally have a basic Testcontainers-based smoke test for the API! :slightly_smiling_face:
It spins up an OPA container, loads in the Rego Playground RBAC policy + data, and then queries the OPA instance with an example request, using the Speakeasy SDK API, and asserts the results are correct.
It's ugly, but should serve as a good scaffold for later smoke tests. :sweat_smile:


cookies.remove  does not take effect

browser.cookies.getAll({
    url:"http://**/"
},function(){
     for (let cookie of cookies) {
        console.log(cookie.value);
    }

});
browser.cookies.remove({url:"http://**",name:"JSESSIONID"},function(results){
    console.log(results)
})

getAll can get the cookies data,but can not remove

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

ID: 06926489-8d16-3995-4965-810523179b4d
Version Independent ID: a6a96e02-8db8-98ab-0bd6-bb93a2b5b4ea
Content: Extensions - JSON manifest example - Microsoft Edge Development
Content Source: microsoft-edge/extensions/api-support/supported-manifest-keys/json-manifest-example.md
Product: microsoft-edge
GitHub Login: @abbycar
Microsoft Alias: abigailc

Investigating. Thanks @wzqiang1332

Considering its an old thread and we have rolled out a new Chromium-based Edge browser, I'll go ahead and close it. If you still are facing any issues, please feel free to reopen this issue or file a new one.


refactor: Only import 'react-native-maps' when not on web

This addresses an issue on expo Snack. Check this example snack.
It crashes on web preview mode with the following error:

I was able to pinpoint that this was caused whenever react-native-maps is imported on web. Actually importing a component was only happening from within the cluster.tsx, importing types is fine and does not cause any issues. I modified this to conditionally import from react-native-maps only when not on web, otherwise use the normal marker of this package.
This should no longer be needed after https://github.com/react-native-maps/react-native-maps/issues/4383 is fixed. But for now, this workaround will allow this package to be usable in snack.

@teovillanueva We need this to be able to use it with Draftbit since we rely on snack for a bunch of our stuff. Take a look when you can.

Hey, what do you think of renaming the current marker.tsx to marker.web.tsx and adding a marker.tsx that has export { Marker } from "react-native-maps" and then in cluster.tsx just importing from ../marker. Let me know if you can try that.

I think there is still no need to add the condition since webpack will just skip the marker.tsx file and include the marker.web.tsx making the export from react-native-maps dissapear from the web bundle.

You'd think so. But Snack seems to work differently than a traditional react native app and still throws the error. I'll test it out again, maybe that has changed since last I tested this, which was a while ago. I'll report back on what I am able to come up with in the end.

Sure! If that doesn't seem to work we'll go with the condition solution :) Im really excited to see Draftbit use the library

And we're excited to include it as well! Will be a great addition 😁

Turns out export { Marker } from 'react-native-maps'; is sufficient for the problem!


Parameter to MergeFrom() must be instance of same class issue with grpc versions 1.26.0

What version of gRPC and what language are you using?
Python 3.8.6, grpc 1.33.2, tried on any grpc version 1.26.0+; protobuf 3.13
What operating system (Linux, Windows,...) and version?
What runtime / compiler are you using (e.g. python version or version of gcc)
Python 3.8.6
What did you do?
I generated a pb file, compiled in python 3.8.6, called 'resource_pb2' that is used in a unit test. This pb file has a class called RecognitionResource that is called in the test.
What did you expect to see?
The test pass.
What did you see instead?
Starting gRPC bi-directional streaming session
Parameter to MergeFrom() must be instance of same class: expected RecognitionResource got RecognitionResource.
Anything else we should know about your project / environment?
I printed out all the Recognition resource class types that are called in that test, I do not see any difference in the classes or ids:
<class 'resource_pb2.RecognitionResource'><PHONE_NUMBER>760
<class 'resource_pb2.RecognitionResource'><PHONE_NUMBER>760
<class 'resource_pb2.RecognitionResource'><PHONE_NUMBER>760

This is unfortunately a ProtoBuf Python issue, you will have better luck at https://github.com/protocolbuffers/protobuf/issues.


Test config updates

When #172 is merged, it would be good to have a test for config updates.

Closing in favor of #42.


Modal with form inside a .row-fluid breaks input size

Hi! :)
If I define a  in a  container (I can't do without), the size of inputs in the form contained in the class="modal" are set in %, not in px, so the size is not what expected.
For a .span3, I get something like 23% width instead of 260px;
But since the modal's width is fixed (to 560px apparently), the inputs inside a modal should have a px defined width, not a percentage.
The problem seems to be located at less/mixins.less, around line 222.

Why is this a problem exactly? If you're using the fluid row, you'll get percentages. If you want a fixed width, use the default columns.

That's logical.
But a .modal have a fixed width, but inputs inside that .modal have a percentage width if the modal is defined inside a .row-fluid container. Resulting in smaller inputs than expected.

Here, take a look at this fiddle, it's exactly the problem:
http://jsfiddle.net/PmRfU/

I see two possible solutions:

Don't define the modal within a .row-fluid, move it somewhere else in your source
Change the .span3 within the modal to something more appropriate

I know I could've moved the modals some place else, but I feel this is more an issue than just a lazy move from me.
The reason is because since the modal have a fixed width, the inputs should also have a fixed width inside, not a percentage one.
I'm pretty sure that I won't be the only one to report this as an issue.
If you want, I can fork and pull a fix to this?

+1

Should be resolved in 2.0.1-wip—the fluid grid sizes are scoped to immediate descendants only.

Great! Thanks :)


BM test run 3

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-benches
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2617959

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-benches
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2617958


Added the possibility to filter by GPU IDs

The standard nvidia-smi summary provides an option to display only some of the available GPUs. This feature is very useful in systems with many GPUs.
The standard nvidia-smi command is:
nvidia-smi -i <comma-serparated-gpu-ids>
For example, to display only GPU 1, 2 and 4:
nvidia-smi -i 1,2,4
I added the same exact -i argument, exploiting the nvidia-smi command directly.
Usage:
nvidia-htop.py -i <comma-separated-gpu-ids>
For example, to display only GPU 1, 2 and 4:
nvidia-htop.py -i 1,2,4
If a wrong ID list is passed, the command fails with the original nvidia-smi error message.
For example, asking for an non-existing GPU ID:
nvidia-htop.py -i 42 or nvidia-htop.py -i random,string
Return:
nvidia-smi exited with error code 6: No devices were found

And do you know since when the -i flag is supported by nvidia-smi? Has it been there forever?

My pleasure, I've been using the script for some time and figured I'd contribute with my little edit.
I added the usage string in the readme and in the Python script, thanks for pointing it out.
It's worth stressing out that this implementation sends the -i argument directly to the nvidia-smi call by subprocess.run. This means that the -i option would not work if this command is piped to nvidia-smi. It only works when nvidia-htop.py is called directly. Otherwise, this option is not considered.
We might look into filtering out the GPU IDs manually from the full nvidia-smi output, but that'd be kinda inefficient, I don't see much purpose to it if not for backward compatibility. Let me know what you think though.
I looked into testing the changes, but for the reason above I could not add new test cases, as they'd require nvidia-htop.py to be piped to a fake output txt.
I think I've used the native -i option on nvidia-smi for years, but I'm not sure when it was first introduced tbh.

Thanks for pointing out it doesn't work in pipe mode. I've added a warning. I also added the test, but it only tests that the option is accepted, nothing more.

Released in version 1.1.0. Thanks for your contribution!

Great! Thank you!


Create FUNDING.yml

That FUNDING.yml file could be setup org-wide via the special .github repository 🤔

Thanks! 🙏 @cyyynthia


Upgrade flutter map dependency

Hey @Wackymax
Could you update the PR to directly update flutter_map to ^7.0.2 and latlong2 to version 0.9.1. Both are the newest versions.

I didn't want to do that because it would force everyone using this lib to update to that version. The ^ will automatically let flutter pub use the latest version if it can.On 29 Jul 2024, at 10:55, Koboo @.***> wrote:
Hey @Wackymax
Could you update the PR to directly update flutter_map to ^7.0.2 and latlong2 to version 0.9.1. Both are the newest versions.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

I've tried this PR with the example and the map comes up blank. It's working fine with the main branch. Anyone had success with the example app?

merged #206, thanks for the contribution


Feature: Rename everything from Post to Repo

Type of feature
🧑💻 Refactor
Current behavior
This project started as post submissions, but in reality they are all Repos!
Suggested solution
Please rename PostWrap to RepoWrap, and etc. Keep in mind that this is a breaking change that should point to beta.

[ ] Rename PostGrid to RepoGrid
[ ] Rename PostWrap RepoWrap
[ ] Rename PostList RepoList

Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[X] I agree to follow this project's Contribution Docs

Hey can you assign me this issue?

Hey can you assign me this issue?

You assign yourself actually. Read here  on how to do it :)

As @ddsuhaimi mentioned, Please review the CONTIBUTING.md for info on how to assign yourself to the issue.

.take

I completed this on my branch #336. Hoping someone can complete this on beta before that PR lands.


[WFRP4e System] Login bugs

Hi.
My player can't get past the login screen.
I have made a Port Forward, and checked that the port is open.
I have tried myself on my Ipad
When he picks his assigned player from the login screen, and types the password, it doens't react.
He is login in from a new chrome browser.
What to do?From<EMAIL_ADDRESS>3.4.1wfrp4e-core: 1.3.1

Not a WFRP issue.
Please use the Foundry's troubleshooting channels on discord


bring manifests up to date with deis/charts

in particular, the replication controller manifest needs some work
https://github.com/deis/charts/blob/master/deis/manifests/deis-builder-rc.yaml

Would be nice if we also had the update-manifests task from the other makefiles


Add a progress bar for file pinning

@flyingzumwalt brought up that it would be extremely useful to have a way to track the progress of a pin call. We can simply reuse the same model we have to track progress for ipfs add.
Another thing that can be extremely useful to this is to add a way to pause and resume a pin call.

Looks like it already been fixed in https://github.com/ipfs/go-ipfs/issues/736
ipfs pin add --progress /ipfs/QmWY4K...ZXKT


FeedAHelp: Buy Domain on a yearly bases

Buy domain www.feedahelp.com | www.feedahelpless.com | www.feedahelp.dk
Whatever suits the best
I think DK is easier, hope you get lucky with .com

We have bought the domain: www.feedahelp.com

FeedaHelp Vercel account has been created

We can close it it works, let see hwo it goes


Make action bar styleable

hahah i know right
On Fri, Nov 10, 2017 at 5:03 PM, skeie<EMAIL_ADDRESS>wrote:

@skeie approved this pull request.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/taxfix/native-navigation/pull/26#pullrequestreview-75801979,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAqlYyV9cJZRXSt9mWqwKe4mA1C4w0cjks5s1HPSgaJpZM4QZvYU
.


Maven - UpdateMavenModel incompatible with CI-Friendly Versions

Consider running OpenRewrite on a multi-module project, using the ${revision} pattern described here: https://maven.apache.org/maven-ci-friendly.html
If using recipes which trigger UpdateMavenModel, then you'll receive log output like this:
[INFO] --- rewrite-maven-plugin:4.27.0:dryRun (default-cli) @ sample ---
[INFO] Using active recipe(s) [my.maven.modifying.recipe]
[INFO] Using active styles(s) []
[INFO] Validating active recipes...
[INFO] Project [sample-parent] Resolving Poms...
[INFO] Project [sample-parent] Parsing Source Files
[INFO] Project [sample-rest] Resolving Poms...
[INFO] Project [sample-rest] Parsing Source Files
[INFO] Project [sample-web] Resolving Poms...
[INFO] Project [sample-web] Parsing Source Files
[INFO] Project [sample-app] Resolving Poms...
[INFO] Project [sample-app] Parsing Source Files
[INFO] Project [sample] Resolving Poms...
[INFO] Project [sample] Parsing Source Files
[INFO] Running recipe(s)...
[WARNING] Illegal character in path at index 73: file:/C:/Users/me/.m2/repository/net/company/sample/sample-parent/${revision}/sample-parent-${revision}.pom
[DEBUG] java.lang.IllegalArgumentException: Illegal character in path at index 73: file:/C:/Users/me/.m2/repository/net/company/sample/sample-parent/${revision}/sample-parent-${revision}.pom
    at java.net.URI.create (URI.java:883)
    at org.openrewrite.maven.internal.MavenPomDownloader.download (MavenPomDownloader.java:239)
    at org.openrewrite.maven.tree.ResolvedPom$Resolver.resolveParentPropertiesAndRepositoriesRecursively (ResolvedPom.java:351)
    at org.openrewrite.maven.tree.ResolvedPom$Resolver.resolveParentsRecursively (ResolvedPom.java:308)
    at org.openrewrite.maven.tree.ResolvedPom$Resolver.resolve (ResolvedPom.java:297)
    at org.openrewrite.maven.tree.ResolvedPom.resolve (ResolvedPom.java:142)
    at org.openrewrite.maven.UpdateMavenModel.updateResult (UpdateMavenModel.java:133)
    at org.openrewrite.maven.UpdateMavenModel.lambda$visitDocument$3 (UpdateMavenModel.java:107)
    at org.openrewrite.marker.Markers.lambda$computeByType$0 (Markers.java:96)
    at org.openrewrite.internal.ListUtils.lambda$map$0 (ListUtils.java:138)
    at org.openrewrite.internal.ListUtils.map (ListUtils.java:120)
    at org.openrewrite.internal.ListUtils.map (ListUtils.java:138)
    at org.openrewrite.marker.Markers.computeByType (Markers.java:91)
    at org.openrewrite.maven.UpdateMavenModel.visitDocument (UpdateMavenModel.java:40)
[WARNING] These recipes would make changes to sample-parent\pom.xml:
[...]

Overall recipe execution continues, but the maven model updates are cut off by the exception, impacting recipe correctness.

Debugging: I believe the issue is in the condition at ~MavenParser:128, causing the MavenResolutionResult.parent field to not be set for projects using these special placeholders for their version. This prevents the MavenPomDownloader later created during recipe execution to not be aware of other projectPoms.
I'm looking into a fix; need to decide how robust we need to be with validating the version and relativePath tags for this case.

@nmck257 I have some time to dig into this. I suspect we just missing a getValue() and therefore there is no property place holder replacements.

@tkvangorder cool -- yeah, if there's a more root-cause fix than my PR (#2050), then by all means.

Should be closed with https://github.com/openrewrite/rewrite/commit/86f9f63bde54f9ed87185f0eed121ef249a80aa9

I'm seeing issues with this again, albeit different exceptions. My previous PRs seem to have not 100% covered it.
@Test
fun `can connect project poms when using CI-friendly version properties`() = rewriteRun(
    pomXml("""
        <?xml version="1.0" encoding="UTF-8"?>
        <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
                 xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd">
          <modelVersion>4.0.0</modelVersion>
          <groupId>net.sample</groupId>
          <artifactId>sample</artifactId>
          <version>${'$'}{revision}</version>
          <packaging>pom</packaging>

          <modules>
            <module>sample-parent</module>
            <module>sample-app</module>
            <module>sample-rest</module>
            <module>sample-web</module>
          </modules>

          <properties>
            <revision>0.0.0-SNAPSHOT</revision>
          </properties>
        </project>
    """.trimIndent()) { spec -> spec.path("pom.xml") },
    pomXml("""
        <?xml version="1.0" encoding="UTF-8"?>
        <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
                 xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd">
          <modelVersion>4.0.0</modelVersion>
          <artifactId>sample-parent</artifactId>
          <groupId>net.sample</groupId>
          <version>${'$'}{revision}</version>
          <packaging>pom</packaging>

          <properties>
            <revision>0.0.0-SNAPSHOT</revision>
          </properties>

          <dependencyManagement>
            <dependencies>
              <dependency>
                <groupId>net.sample</groupId>
                <artifactId>sample-web</artifactId>
                <version>${'$'}{project.version}</version>
              </dependency>
              <dependency>
                <groupId>net.sample</groupId>
                <artifactId>sample-rest</artifactId>
                <version>${'$'}{project.version}</version>
              </dependency>
            </dependencies>
          </dependencyManagement>
        </project>
    """.trimIndent()) { spec -> spec.path("parent/pom.xml") },
    pomXml("""
        <?xml version="1.0" encoding="UTF-8"?>
        <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
                 xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd">

          <modelVersion>4.0.0</modelVersion>
          <artifactId>sample-app</artifactId>
          <packaging>jar</packaging>

          <parent>
            <groupId>net.sample</groupId>
            <artifactId>sample-parent</artifactId>
            <version>${'$'}{revision}</version>
            <relativePath>../parent/pom.xml</relativePath>
          </parent>

          <dependencies>
            <dependency>
              <groupId>net.sample</groupId>
              <artifactId>sample-rest</artifactId>
            </dependency>

            <dependency>
              <groupId>net.sample</groupId>
              <artifactId>sample-web</artifactId>
            </dependency>
          </dependencies>
        </project>
    """.trimIndent()) { spec -> spec.path("app/pom.xml") },
    pomXml("""
        <?xml version="1.0" encoding="UTF-8"?>
        <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
                 xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd">

          <modelVersion>4.0.0</modelVersion>
          <artifactId>sample-rest</artifactId>
          <packaging>jar</packaging>

          <parent>
            <groupId>net.sample</groupId>
            <artifactId>sample-parent</artifactId>
            <version>${'$'}{revision}</version>
            <relativePath>../parent/pom.xml</relativePath>
          </parent>
        </project>
    """.trimIndent()) { spec -> spec.path("rest/pom.xml") },
    pomXml("""
        <?xml version="1.0" encoding="UTF-8"?>
        <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
                 xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd">

          <modelVersion>4.0.0</modelVersion>
          <artifactId>sample-web</artifactId>
          <packaging>jar</packaging>

          <parent>
            <groupId>net.sample</groupId>
            <artifactId>sample-parent</artifactId>
            <version>${'$'}{revision}</version>
            <relativePath>../parent/pom.xml</relativePath>
          </parent>
        </project>
    """.trimIndent()) { spec -> spec.path("web/pom.xml") },
)

...yields:
java.lang.AssertionError: Failed to parse sources or run recipe
[...]
Caused by: org.openrewrite.maven.internal.MavenDownloadingException: Unable to download dependency [net.sample:sample-rest:0.0.0-SNAPSHOT] from the following repositories :
[...] // my repositories

The version check (L389) continues to be thorny... in this case, in the relevant iteration, the left side is ${revision} and the right side is 0.0.0-SNAPSHOT, as the (non-resolved) pom object doesn't have all its inherited properties available for resolution.
https://github.com/openrewrite/rewrite/blob/175a35850b98d4b73dd7c763ad83a0d5ad94042a/rewrite-maven/src/main/java/org/openrewrite/maven/internal/MavenPomDownloader.java#L384-L392
A naive solution could be removing this version check.
A more intricate solution could be refactoring this method (and MavenParser::parseInputs) such that we use module interdependencies to inform project pom resolution order and make the resolved-thus-far project poms available to the MavenPomDownloader::download method, thus allowing better property resolution for the version check. Currently, project pom resolution order is arbitrary, and module analysis happens after resolution.

cc @jkschneider
I've drafted changes to resolve the project poms in interdependency-order.
And, I've started changing the MavenPomDownloader to use ResolvedPoms (whichever ones are available) for projectPoms instead of raw Poms. But, things start falling down there -- using ResolvedPoms works well for this property resolution use case, but the recursive pom downloads which happen during resolution (which just needs raw poms) will break if we don't provide the full list of project poms.
Relevant lines which misbehave if the projectPoms collection is incomplete:
https://github.com/openrewrite/rewrite/blob/175a35850b98d4b73dd7c763ad83a0d5ad94042a/rewrite-maven/src/main/java/org/openrewrite/maven/internal/MavenPomDownloader.java#L394-L410
This issue pushes us towards providing the MavenPomDownloader both the full list of raw project poms, and also the partial list of resolved-thus-far poms.
Or -- noting that Maven does not allow arbitrary properties in the version field, only the CI-friendly version properties revision, sha1, or changelist (ref) -- we could continue passing MavenPomDownloader only the raw project poms, and add a little logic to the section mentioned in my previous comment to do a "partial, property-only resolution". Basically, identify where a project pom's parent is also a project pom, and use that to get the set of transitive properties, ignoring any transitive properties which may come from non-project parents.
I'm leaning towards that last approach. And, with that in place, I don't actually see a clear benefit to resolving the project poms in interdependency-order, but if you're interested in that code, I could open a separate PR for further evaluation.


added All Cities endpoint implementation

develop
GitHub Board
Issue link
Create new endpoint for obtains list of cities #574
Story link
Page overview #544
Code reviewers

[ ] @teaFunny
[ ] @V-Kaidash

Summary of issue

[x] Implement api endpoint contorller for getting all cities.
[x] Implement api endpoint contorller for getting cities by region.
[x] Implement api endpoint contorller for getting region by city.
[x] Implement service for all above
[x] Implement repository for all above
[x] Add tests for all above.

Summary of change

[x] Implemented api endpoint contorller for getting all cities.
[x] Implemented api endpoint contorller for getting cities by region.
[x] Implemented api endpoint contorller for getting region by city.
[x] Implemented service for all above
[x] Implemented repository for all above
[x] Added tests for all above.

Please add some demo on this feature. Thanks

https://drive.google.com/file/d/1HTS62XcuwPeRTC4_V1aZgEY3pOxvldno/view?usp=sharing


Use common logging framework

Delivered by #3.
The SDK now provides the logger implementation for the Simple Data Pipe. To use the logger common modules (that are not processing pipe runs) and that are not included in the SDK should do the following:
var sdpLog = require('simple-data-pipe-sdk').logging.getLogger('sdp_common');
sdpLog.info('...');

Modules included in the SDK should
var sdpLog = require('../logging/sdpLogger').getLogger('sdp_common');
sdpLog.info('...');

The rotating log file(s) are located in /home/vcap/app/logs/ and named sdp_common.<date>-<HHMM>.log


Fixed math issues

Mistakes were made :facepunch:

det er elif, ikke else if :) @hptk

@gigglearrows Thanks, I write too much C...

no


Running tests as part of the build

I think the GitHub actions do not run the tests, but so it is difficult to catch issues introduced by PRs

implemented in #74


Sorting on data of Cell Template

I have an Id field in my ui grid. The data for that comes from a custom template that prints number 1,2,3 in order. But due to some reasons I am not able to perform search or sorting on that field. Is that normal behavior?

You'd be better to insert the id field into the data than to use a custom template.  ui-grid searches and sorts on the underlying field, it doesn't know that you've changed the template to render a value in there that wasn't in the data.

Id can be random nos. They dont follow a particular order. So directly showing id will not be useful in some cases. From a user perspective I was thinking to have a sequential nos in first column independent of id and have a sorting on that.

So add an extra field to the data called seq, define it as a column { name: 'seq', displayName: 'ID' }, and then they can sort/filter on it.  There is no need to include the real id in your columns at all.

For example: http://plnkr.co/edit/Qdn5oYmfPbxB1hCVLcJe?p=preview

Thanks Paul for suggestion. It works as expected. Closing issue.


Message-generating tools only work if last statement in codeblock

It tried to do something like:
screenshot()
view_image()

But it never took the screenshot, because the generator wasn't run, and had nowhere to emit messages.
See also https://github.com/ErikBjare/gptme/issues/186

Should probably just get rid of the Message-generating tools altogether, and make a special solution for the view_image() tool (or just rely on the assumption that it will almost always be the last statement).
Started doing the former in https://github.com/ErikBjare/gptme/commit/65bdb8ad3291cd4a4be691255d3533e6413bc676


VSCode plugin

It lacks a VSCode plugin to help intellisense

@HerryYT
You can look into this project, but it's under development https://github.com/vectier/tslombok


Configuration docs

This section could become its own page eventually, as we add things we want to configure.

Merging this so I can refer to these docs in the auth plugin docs.


Update documentation to match minikube's current behaviour

Updating docs as per minikube's #631

This change is

Signed CLA.

LGTM

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Create a Shopcart DB

As a developer
I need a database to store shopcart data
So that we can persist shopcart data between customer sessions
Details and Assumptions

A model for the shopcart exists
A unique id is generated for each shopcart
Only 1 shopcart can exist per customer

Acceptance Criteria
Given a customer ID
When I perform a query to the Shopcart DB using the customer ID
Then I receive the shopcart associated with the customer

This story is not required


Create Microsoft

What is the purpose of this PR?

Thanks for contributing, but I'm not quite sure what you're trying to achieve here. Closing the PR.


fix dependency issue that causes github security warning

@dmilet the twilio library 2.x is now deprecated and an associated library hoek causes a github security warning (which you probably received, too).
This PR fixes upgrades the twilio library, but is still a WIP, as I need to confirm if any changes are needed as here:
https://www.twilio.com/docs/libraries/node/migration-guide

updated code to use promised-based v3 twilio api


Markdown tables are won't be rendered

Summary
Markdown tables are won't be rendered.
Environment Information

Device Name: Nexus 5
OS Version: Android 7.1.2
Mattermost App Version: 1.0
Mattermost Server Version: 3.10.0

Steps to reproduce
Write this:
| A | B |
|---|---|
| a | b |

Expected behavior
This should be rendered (like in the Mattermost web app):

Observed behavior
This will be rendered:

Possible fixes
I think marked can render tables. Maybe you have to enable table support: https://github.com/chjj/marked#tables

Thanks @sedrubal!
We still need to add support for tables - we have a ticket here to track the issue: https://mattermost.atlassian.net/browse/RN-175

also is not working in ios
we rely on that feature a lot, and forced to use "classic" version of the app, where it works

it also appears that markdown images are not supported as well.

Markdown tables and images are scheduled to be included with release 1.3. Im closing this issue but feel free to re-open if needed


maint: updating ansys-sphinx-theme version

The search engine is not running correctly on the stable branch. This should fix the issue.
Pinging @Revathyvenugopal162 for visibility.

This will increase the search result speed.


Failed to create consumer while using proxy and Prefix Subscription Authentication Mode

Describe the bug
I am using pulsar cluster on k8s, deployed by pulsar-helm-chart, having proxy component and using JWT for authentication.
If one namespace change the subscription authentication mode from None to Prefix, the consume can not subscribe the topic even having the valid role prefix.
To Reproduce
Steps to reproduce the behavior:

Using pulsar cluster on k8s, having proxy component and using JWT for authentication
Set the subscription authentication mode of one namespace from None to Prefix, e.g. namespace test/auth
Add a subscription roleA-test of topic persistent://test/auth/topic
Use role roleA to consume the topic persistent://test/auth/topic

Expected behavior
The consumer should subscribe and consume the topic normally.
Screenshots
The consumer get the error org.apache.pulsar.client.api.PulsarClientException$AuthorizationException: Failed to subscribe ...

Desktop (please complete the following information):

OS: Linux

Additional context
The auth part of helm chart values:
auth:
  authentication:
    enabled: true
    provider: "jwt"
    jwt:
      # Enable JWT authentication
      # If the token is generated by a secret key, set the usingSecretKey as true.
      # If the token is generated by a private key, set the usingSecretKey as false.
      usingSecretKey: true
  authorization:
    enabled: true
  superUsers:
    # broker to broker communication
    broker: "broker-admin"
    # proxy to broker communication
    proxy: "proxy-admin"
    # pulsar-admin client to broker/proxy communication
    client: "admin"

The broker log:
20:31:48.269 [pulsar-io-25-4] INFO  org.apache.pulsar.broker.service.ServerCnx - New connection from /<IP_ADDRESS>:52986
20:31:48.281 [pulsar-io-25-4] INFO  org.apache.pulsar.broker.service.ServerCnx - [/<IP_ADDRESS>:52986] org.apache.pulsar.broker.PulsarServerException: Failed to create consumer - The subscription name needs to be prefixed by the authentication role, like proxy-admin-xxxx for topic: persistent://test/auth/p1-partition-0 with role roleA

Dig into the code, the broker will check the operation in org.apache.pulsar.broker.service.ServerCnx#isTopicOperationAllowed
In this case:
service.isAuthorizationEnabled() is true, originalPrincipal is roleA, authRole is proxy-admin.
isProxyAuthorizedFuture will be true, isAuthorizedFuture will throw the PulsarServerException(actually throws in org.apache.pulsar.broker.authorization.PulsarAuthorizationProvider#canConsumeAsync, passing proxy-admin as role will throw the exception), which results the failure of consumer.
One simple solution is that, permissionFuture.complete(false) instead of permissionFuture.completeExceptionally(ex), not throwing the exception in org.apache.pulsar.broker.authorization.PulsarAuthorizationProvider#canConsumeAsync


Reply for button on message

Hi, can I start a dialogue by replaying the message with the button that I started.
Example:

I have a handler that starts a dialog with the show_mode method=ShowMode.SEND
I click on it. A dialog is launched that in reply contains the message on which the button was clicked.

hello, i have same question

Same question.

@Tishka17 This is a case when pressing a button on a message outside the stack causes the keyboard on it to be deleted. We need your help in correcting the logic.
Example start:
await DialogManager.start(CreationTask.name, mode=StartMode.RESET_STACK, show_mode=ShowMode.SEND)
Logs:
2024-05-27 11:42:55,309 - aiogram_dialog.dialog - DEBUG - Dialog start: <State 'CreateTask:name'> (Dialog 'CreateTask')
2024-05-27 11:42:55,311 - aiogram_dialog.dialog - DEBUG - Dialog render (Dialog 'CreateTask')
2024-05-27 11:42:55,311 - aiogram_dialog.window - DEBUG - Show window: <Window(<State 'CreateTask:name'>)>
2024-05-27 11:42:55,313 - aiogram_dialog.manager.message_manager - DEBUG - Send new message, because: mode=ShowMode.SEND, has old_message=True
2024-05-27 11:42:55,313 - aiogram_dialog.manager.message_manager - DEBUG - remove_inline_kbd in ...
2024-05-27 11:42:55,416 - aiogram_dialog.manager.message_manager - DEBUG - send_text to

This was an intended behavior to handle outdated keyboards, but I guess in your case it's not always expected.
There is no direct way to disable deleting the keyboard, but you cat try starting dialog using BgManager: dialog_manager.bg().start(...)


Vague Prework task 2. Ensure your laptop up to the task of coding

This task contains vague requirements with no instructions and should be more specific.
What is the minimum hard drive space and memory?
Should we define memory?
How do they eliminate viruses and can we recommend any programs?
References https://github.com/codefellows/code-101-prework/issues/75

From CF site:
Material Requirements
You are required to supply your own laptop that meets the below requirements. Chromebooks are not suitable for this course, since you can’t directly control installed software. Most students use Macs. You are responsible for the upkeep and maintenance of your computer throughout the course. The course pre-work will have specific instructions for configuring your computer to be ready for the first day of class.
64-bit Operating System, like macOS or Windows 10.
Fully up to date, with the latest version of the operating system and all security updates installed.
At least 50GB of free space on the hard drive.
At least 8GB of RAM. 16GB RAM strongly preferred.
Free of viruses, and in general working order.

https://www.codefellows.org/courses/code-201/foundations-of-software-development/#material-requirements
THAT IS THE MAGIC SAUCE

lgtm


example of using skipif as a parametrize mark

This PR is just for you to see an idea of what was I explaining you. Of course this is a dirty way of doing it.

Actually you should be able to write something like this:
from mne.utils import requires_mayavi, requires_vispy

skipif_mayavi_is_missing = requries_mayavi(lambda *args: None)
skipif_vispy_is_missing = requries_vispy(lambda *args: None)

@pytest.mark.parametrize('backend', [
    pytest.param('mayavi', marks=skipif_mayavi_is_missing),
    pytest.param('vispy', marks=skipif_vispy_is_missing),
    pytest.param('foo', marks=pytest.mark.xfail(raises=ValueError)),
])
def test_backend_enviroment_setup(backend, backend_mocker, monkeypatch):
    # bla bla

I think it's minimal and elegant, I will try it.
do you think it will as flexible as the pytest.mark.skipif alternative?

@requres_mayavi returns pytest.mark.skipif. That's why I name it skipif_bla

what I'm actually doing is undoing the decorator.


resurrection of the Mempool interface refactoring

This is a work in progress to resurrect the work of this PR https://github.com/bitcoin/bitcoin/pull/11775

After working on this I understand that we should rewrite how the Signal is implemented. Currently, it is coupled with the CVerificationInterface, but now that we have the second interface, we should move this into another
design.
I was thinking of using a struct CSignal and add and remove a signal, with the current interface but with the only problem is that we should feel the signal as needed.
I do not know if exist a better design for this, so I will leave cook a little bit


removed Target Release Notes

As the feature appear as launched in the roadmap, and since Standard release customer already saw that, removing those line is probably required
https://www.microsoft.com/it-it/microsoft-365/roadmap?rtc=1&filters=&searchterms=QR

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks

@kwekuako Please merge this PR. Thanks


Heading title

The titles of the tables "Users", "Groups", "Roles" (and other..!) should start with a capital letter.
(Same problem in French & english!)

Thx


WIP: Add websock command to query device for triggers

Description:
Add websock command to query device for triggers.
Triggers can be provided by the integration that provided the device or the entity integrations that the device has entities with.
As an example of the latter, add triggers to light.
TODO: Add an example of the former.
This PR is intended as initial step towards home-assistant/architecture#178
Example:
# Ask for triggers for device:
{"id": 1, "type": "automation/device_automation/list_triggers", "device_id": "46d461813d264be185bad875ea907166"}
# Two triggers returned:
{"id": 1, "type": "result", "success": true, "result": {"triggers": [{"platform": "device", "domain": "light", "type": "turn_on", "entity_id": "light.brown_light"}, {"platform": "device", "domain": "light", "type": "turn_off", "entity_id": "light.brown_light"}]}}

The returned trigger can be used in an automation:
trigger:
 - platform: 'device'
   domain: 'light'
   type: turn_off
   entity_id: light.disco_room
action:
   service: homeassistant.turn_on
   entity_id: light.sonoff

Checklist:

[x] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works. - TODO

I think that overall this looks great.
For translations, would we do something like device_automation.<domain>.<type> ?

@balloob thanks for reviewing! All fixed now + tests added.


API draft

This is the initial draft for our proposed new API. A brief overview of the existing API is also provided below for reference. The API should serve two uses cases:
High-level
This use case is the simplest one where the API consumer wants to embed a PDF reader in their app, much like Scribd's embedded reader. The reader UI can be minimal: no page scrolling, toolbar, etc - simply next/previous page buttons and page # selector.
Basically the consumer gives a div and a document src, and pdf.js embeds the aforementioned reader in that div, perhaps with optional parameters to enable/disable features (text layer, search, annotations):
PDFJS.embed(div, 'my-super-doc.pdf', {
  selection: true,
  search: false,
  annotations: true,
  passwordHandler: function() { /* ... */ return password; }
});

The .embed() interface does all the job of downloading the file and inserting all features into the div.
Low-level
This is the API for consumers who need more detailed control. It can mirror the current API:
Current API
This is a summary of calls currently made by the web/ application to PDFJS objects. They are summarized here because it's not clear from core.js which methods are meant to be called externally and which are for internal use only.
PDFDoc()

var pdf = new PDFJS.PDFDoc(data)
pdf.numPages
pdf.fingerprint
pdf.catalog.destinations
pdf.catalog.documentOutline

Page()

var page = pdf.getPage(num)
page.width
page.height
page.stats
page.ref
page.view (from here on the calls are actually content.* in viewer.js)
page.getAnnotations()
page.rotatePoint(x, y, reverse)
page.startRendering(ctx, callback, textlayer)

We probably need to pass a callback for passwords from users to handle encrypted PDFs:
PDFJS.embed(div, 'my-super-doc.pdf', {
  selection: true,
  search: false,
  annotations: true,
  passwordHandler: function() { /* ... */ return password; }
});

To enable users to build "addons" like #1119 #1148 which need to extract information at a low-level, perhaps we should add:
page.getIRcode()

This would return the internal intermediate representation of the PDF page, enabling users to easily extract text, annotations, etc, from a JavaScript object.

Only wanted to throw in some thoughts, without solutions so far, but to keep them considered during API design phase:

On low-memory devices, I'd like to control the amount of stored parsed pages in a PDFDoc, to parse/purge on demand
Should to API plan ahead for handling progressive/partial loading?
Event emitter to hook into internal flows, being able to cancel/mutate event data (onparse, onbeforeparse, onbeforerender, onrender, etc). This could also lead into a …
Plugin architecture (text rendering, clickable links, copy images, etc) for core and viewer (just an idea :))

I think a pdf.js API is a great idea, and extracting various PDF objects is likely to come up again and again.  I think the proposed API of a list of IR codes is likely to be a much better match for extracting vector graphics and images than for extracting text, however.
Highlight/underline annotation extraction in particular seems to require rendering the PDF (though I'm open to suggestions otherwise).  Highlighted/underlined areas are given simply as a series of bounding boxes; it's hard to know what falls inside those boxes without rendering the entire page.
Extracting all the text of the pdf (like the pdftotext program does) probably requires rendering, too, since many "words" are rendered as a sequence of individual characters with complicated inter-character spacing.  Since this rendering code is already in pdf.js, I thought it made the most sense to augment rendering a bit.
I understand that this adds some complexity (and slowdown) for a use-case that isn't directly related to PDF viewing.  To deal with the slowdown, I'd be happy to factor the annotation extraction stuff into a separate function called from showText() and showSpacedText(), and have this turned off by default.  I'm also happy to maintain my fork outside the pdf.js project.

While we're on the subject of alternate use cases for pdf.js, another side project I'd like to start soon is to modify pdf.js to update the Author/Title/Keywords metadata inside a PDF so that the research papers in my library have proper PDF metadata, which makes desktop search much more effective.  Of course, writing out new PDF files is even less related to PDF viewing ;-).

This sounds like a great idea. A couple of comments.
For the high level API, how does the consumer change pages? Are there buttons in the viewer embedded in the div? I think it would be nice if the consumer could programmatically change pages. One example was if there was a synchronized sound track and pdf, the controlling page could automatically change the pdf page whenever a certain point in the sound track is hit.
As for the lower level API, two calls that would be of interest to me would be to get all the text and all the images in the page. As @devietti pointed out, getting the text would require at least some partial rendering of the page, but this seems like it could be done on demand. Getting the images could be done by drawing the images on canvas objects, and then returning the array of canvases. Also, Im not sure how useful the IR stream by itself it, but it seems easy enough to implement.

I'm very interesting in this.  And I'd rather not run or include a large set of .js files.  Is there any way to build pdf.js for third parties?  And what particular directory should we be running/building/including from for third party apps?

I'm very interesting in this. And I'd rather not run or include a large set of .js files. Is there any way to build pdf.js for third parties? And what particular directory should we be running/building/including from for third party apps?


Fix an issue related to "-" chars in title

When attempting to export my notes I received the following error.
./export-notes.scpt:8297:8314: execution error: Can’t get text 1 thru -2 of "-". (-1728)
I added a try catch so that if this error occurs it falls back to removing all "-" characters.

Hi @bebrws - Thanks for this.
Can you raise an issue and explain the problem, and include information about the filename text before it was changed by the script, so I can try to replicate? I'd like to identify the root cause before accepting the fix - thanks.


File Explorer > Send To > Add folder to PATH

📝 Provide a description of the new feature
The new-ish "Environment Variables" PATH editor is really nice, much better than the old days of finding your way in the semicolon delimited textbox.
I'm setting up a new computer and installing many command line utilities that don't update the PATH during the install, and often need to add a folder to the PATH and thought it would be a nice addition to the PowerToys to have a File Explorer > Right click on a folder(or folders?) > Send To > Add to PATH.
Maybe have one entry for System variables "Add to System PATH" and for user variables "Add to {username} PATH"

If you'd like to see this feature implemented, add a 👍 reaction to this post.

Thanks for the suggestion. Duplicate of #2189.


query the example of 'complex-resource'

I have tried to add some mock data for the complex-resource example, which has its structure as,
var order = {
  custom: {
    id: String,
    name: String
  },
  orderItems: [{
    quantity: Number,
    product: {
      id: String,
      name: String,
      price: Number
    }
  }]
};

The question is, how can I get the result that the orderItems have their products with the prices higher than 10.00, for example?
I have tried the queries like, without luck.
1. GET orders?$filter=price gt 10
2. GET orders?$filter=product/price gt 10
3. GET orders?$filter=orderItems/product/price gt 10

the results are always empty, was this a bug or something I was missing, any ideas?
{"value":[]}
The mock data:
{ "_id" : "fb107243-5f4d-4d47-8b35-245fa85127cd", "orderItems" : [ { "quantity" : 100, "_id" : ObjectId("57e303cdd70bda6419c9922d"), "product" : { "id" : "item123", "name" : "ITEMA", "price" : 10 } } ], "custom" : { "id" : "abc123", "name" : "ABC" } }
{ "_id" : "1c1ceeea-8c75-443e-8706-fa696ec5acf6", "orderItems" : [ { "quantity" : 200, "_id" : ObjectId("57e30426d70bda6419c9922e"), "product" : { "id" : "item234", "name" : "ITEMB", "price" : 15 } } ], "custom" : { "id" : "bcd123", "name" : "BCD" } }
{ "_id" : "a8dd0f23-46d9-4f6e-bffa-9709e7c23b87", "orderItems" : [ { "quantity" : 300, "_id" : ObjectId("57e30454d70bda6419c9922f"), "product" : { "id" : "item345", "name" : "ITEMC", "price" : 20 } } ], "custom" : { "id" : "efg123", "name" : "EFG" } }
{ "_id" : "316efd3f-0b30-4ad1-ac22-2640a613142e", "orderItems" : [ { "quantity" : 400, "_id" : ObjectId("57e30474d70bda6419c99230"), "product" : { "id" : "item456", "name" : "ITEMD", "price" : 30 } } ], "custom" : { "id" : "hij123", "name" : "HIJ" } }

Sorry, it's not support this feature, it can only support a simple model query. I will try to add it on this weekend.

try
GET orders?$filter=orderItems.product.price gt 10

Hi @TossShinHwa, many thanks for your quick reply!
It seems working well before I add following new orderItems to the collection,
{"orderItems":[{"quantity":1000,"product":{"id":"item777","name":"ITEM7","price":10}},{"quantity":2000,"product":{"id":"item888","name":"ITEM8","price":15}},{"quantity":3000,"product":{"id":"item999","name":"ITEM9","price":20}}],"custom":{"id":"xyz123","name":"XZY"}}
In this case, if I query,
GET orders?$filter=orderItems.product.price lt 15
The response would always include the entire array, if there is at least one price in the array matches the query. I wonder if I have to use $expand or any/all functions to further filter (refine) the result in this case?
The response of above query (price less than 15):
{"value":[{"orderItems":[{"quantity":100,"product":{"id":"item123","name":"ITEMA","price":10},"id":"57e303cdd70bda6419c9922d"}],"custom":{"id":"abc123","name":"ABC"},"id":"fb107243-5f4d-4d47-8b35-245fa85127cd"},**{"orderItems":[{"quantity":1000,"product":{"id":"item777","name":"ITEM7","price":10}},{"quantity":2000,"product":{"id":"item888","name":"ITEM8","price":15}},{"quantity":3000,"product":{"id":"item999","name":"ITEM9","price":20}}]**,"custom":{"id":"xyz123","name":"XZY"},"id":"57e941c0a8dbeb4c8c8f317c"}]}
The response of query (price greater than 15):
{"value":[{"orderItems":[{"quantity":300,"product":{"id":"item345","name":"ITEMC","price":20},"id":"57e30454d70bda6419c9922f"}],"custom":{"id":"efg123","name":"EFG"},"id":"a8dd0f23-46d9-4f6e-bffa-9709e7c23b87"},{"orderItems":[{"quantity":400,"product":{"id":"item456","name":"ITEMD","price":30},"id":"57e30474d70bda6419c99230"}],"custom":{"id":"hij123","name":"HIJ"},"id":"316efd3f-0b30-4ad1-ac22-2640a613142e"},**{"orderItems":[{"quantity":1000,"product":{"id":"item777","name":"ITEM7","price":10}},{"quantity":2000,"product":{"id":"item888","name":"ITEM8","price":15}},{"quantity":3000,"product":{"id":"item999","name":"ITEM9","price":20}}]**,"custom":{"id":"xyz123","name":"XZY"},"id":"57e941c0a8dbeb4c8c8f317c"}]}

Unfortunately, it's not support $expand / all / any to query entities. But there has a way to make it work even it ugly.
You can add a ACTION for this resource, it's use mongoDB API to filter data. example:
    server.resource('order', { product: [{ price: Number }] })
    .action('/all-item-greater', (req, res, next) => {
      const { price } = req.query;
      const $elemMatch = { price: { $gt: price } };
      server.resources.order.find()
      .select({ product: { $elemMatch } })
      .exec((err, data) => res.jsonp(data));
    });

Full version.

Thanks for the reply! but I met a small problem with const { price },
  const { price }  = req.query;
        ^

SyntaxError: Unexpected token {
    at exports.runInThisContext (vm.js:53:16)
    at Module._compile (module.js:373:25)
    at Object.Module._extensions..js (module.js:416:10)
    at Module.load (module.js:343:32)
    at Function.Module._load (module.js:300:12)
    at Function.Module.runMain (module.js:441:10)
    at startup (node.js:139:18)
    at node.js:974:3

ES6 syntax:
const { price }  = req.query;

Same as ES5 syntax:
var price = req.query.price;

Hi ! The ES5 syntax seems working, whereas the query cannot get a result in my test..
GET orders(316efd3f-0b30-4ad1-ac22-2640a613142e)/all-item-greater?price=10
The response,
Cannot GET /orders(316efd3f-0b30-4ad1-ac22-2640a613142e)/all-item-greater?price=10
The data, I assume ACTION should also work in the case of non-multiple orderItems.
GET orders(316efd3f-0b30-4ad1-ac22-2640a613142e)/

{"orderItems":[{"quantity":400,"_id":"57e30474d70bda6419c99230","product":{"id":"item456","name":"ITEMD","price":30}}],"custom":{"id":"hij123","name":"HIJ"},"id":"316efd3f-0b30-4ad1-ac22-2640a613142e"}

Strange, it should be working. plz check your code is same as the example.


View to delete or edit a referral

@pprolance, maybe want for single page app upgrade.

@vincentdavis
as you added "High Priority" on this task, can I start working on this or I already have to wait for single page app upgrade?

@pprolancer Start working on this please


BUILD_SHARED_LIBS should trigger dynamic build for OpenSSL in Linux

Added possibility to build OpenSSL dynamic in linux using the BUILD_SHARED_LIBS=ON argument.

Thanks, please update PACKAGE_INTERNAL_DEPS_ID. See https://docs.hunter.sh/en/latest/reference/user-modules/hunter_download.html for details.

https://github.com/ruslo/hunter/pull/703


Integrator class uses method not defined in "xml-apis-1.4.01.jar"

It uses this method "XMLInputFactory.newFactory()" which is defined in the JRE 1.8 default classes but is not defined in the similar implementation from "xml-apis-1.4.01.jar". Publishing probably works because the Java default classes are usually stronger than the extra JAR libraries from the classpath.
But when compiling the "dos.jar" if for example the compilation classpath favors "xml-apis-1.4.01.jar" instead of the runtime libraries, you get a compilation error.
I suggest replacing "newFactory()"=>"newInstance()" which does the same thing.

Fixed by #3273


[ Feat ] Add transform option

This PR is related to #6 .

Adds transform option.
Updates playground/fileselector.php and Docs/fileselector.md

Sorry I didn't merge this earlier - I totally forgot about it. This is great, thx again for helping out by contributing! I really appreciate it!

Sorry I didn't merge this earlier - I totally forgot about it. This is great, thx again for helping out by contributing! I really appreciate it!

@ProjektGopher  No problem.  I thought you were on summer vacation😎.
Glad to be of help.


1004 fix style issues up031 ruff lint

First onto dependency-updates , maybe needs to be rebased later to dev-2.2.0.
Fix for discussion referred in #1004

Description

Related issue: #1004
Motivation and Context

How has this been tested?

Screenshots (if appropriate)
Types of Changes

[ ] 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)
[x] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Checklist

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

Why are all the commits from dependency-updates in this PR?

thanks, something went wrong there. I need to fix this branch

re-did the commits on fresh dependency-updates branch.
Now waiting for tests to pass..

this one can be merged right? @jochenklar


[now dev] Allow custom 404 pages via routes

This matches the behavior in production, which allows a dest to be
provided when defining a status: 404 in the routes configuration.
Related to #2638.

This one doesn't really touch routes, but rather the handling of the result of the routing logic.

Codecov Report

:exclamation: No coverage uploaded for pull request base (canary@eeed5a8). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             canary   #2689   +/-   ##
========================================
  Coverage          ?   8.24%
========================================
  Files             ?     260
  Lines             ?   10514
  Branches          ?    1339
========================================
  Hits              ?     867
  Misses            ?    9589
  Partials          ?      58

Impacted Files
Coverage Δ

src/util/dev/server.ts
12.42% <ø> (ø)

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 eeed5a8...612e199. Read the comment docs.


Write jsonrpc and cli tests

As a developer, I want to know that every jsonrpc and cli command works so that I don't break the daemon's UI by accident.
As a user, I want the jsonrpc and cli documentation to be correct so that I know how to interact with the daemon.

I'm gonna leave this out of the milestone-ing. Lemme know when it's ready for us to look at.

Sure. I've been quite busy recently, but I'm free now so I'll probably get it done by the end of this week.


Switch to GHA and psalm

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Fixed issues
-

Need readme badges corrected. See how it's done in template.

Need readme badges corrected. See how it's done in template.

Ah badges are forgotten...


If disabling an alert, it stays on

I disabled this alert 10 seconds earlier and got a successfully stored toast message. But the alert stays red. As a user, I would expect it to return to the non-triggered state right away.

It worked on the detailed sensor view but when I returned to the dashboard, the sensor was still showing red font (alert on). A page refresh fixed it.