"salmon" core-dumps

Version info

bcbio version (bcbio_nextgen.py --version): 1.2.2
OS name and version: Ubuntu 18.04.4 LTS

To Reproduce

Exact bcbio command you have used
/usr/local/bcbio/bin/salmon index -h
Your sample configuration file

more /usr/local/bcbio/config/install-params.yaml
aligners:
- bwa
- rtg
datatarget:
- variation
- rnaseq
- smallrna
genomes:
- hg38
isolate: false
tooldir: /usr/local/bcbio
toolplus: []

Observed behavior
Version Info:   This is the most recent version of salmon.
Illegal instruction (core dumped)

Expected behavior
Index
==========
Creates a salmon index.

Command Line Options:
  -v [ --version ]           print version string
  -h [ --help ]              produce help message
  -t [ --transcripts ] arg   Transcript fasta file.
  -k [ --kmerLen ] arg (=31) The size of k-mers that should be used for the
                             quasi index.
  -i [ --index ] arg         Salmon index.
  --gencode                  This flag will expect the input transcript fasta
                             to be in GENCODE format, and will split the
                             transcript name at the first '|' character.  These
                             reduced names will be used in the output and when
                             looking for these transcripts in a gene to
                             transcript GTF.
  -p [ --threads ] arg (=2)  Number of threads to use (only used for computing
                             bias features)
  --perfectHash              [quasi index only] Build the index using a perfect
                             hash rather than a dense hash.  This will require
                             less memory (especially during quantification),
                             but will take longer to construct
  --type arg (=quasi)        The type of index to build; options are "fmd" and
                             "quasi" "quasi" is recommended, and "fmd" may be
                             removed in the future
  -s [ --sasamp ] arg (=1)   The interval at which the suffix array should be
                             sampled. Smaller values are faster, but produce a
                             larger index. The default should be OK, unless
                             your transcriptome is huge. This value should be a
                             power of 2.

Log files
Please attach (10MB max): bcbio-nextgen.log, bcbio-nextgen-commands.log, and bcbio-nextgen-debug.log.
NA
Additional context
manager@brigante:~$ printenv PATH
/usr/local/bcbio/bin:/home/manager/bin:/home/manager/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin
manager@brigante:~$ which -a salmon
/usr/local/bcbio/bin/salmon
/usr/bin/salmon

manager@brigante:~$ /usr/local/bcbio/bin/salmon -v
salmon 1.1.0
manager@brigante:~$ /usr/bin/salmon -v
version : 0.7.2

Both salmon 1.1.0 and 1.00 from GitHub core-dump when attempting to build an index in our pipeline, but salmon 0.14.1 from GitHub works correctly and outputs a warning that a newer version of "salmon" is available. Unfortunately, I can't compile it from source under Ubuntu 18.04 because the version of "cmake" is too old:
root@firefly:/usr/local/src/salmon/salmon/build# cmake ..
CMake Error at CMakeLists.txt:1 (cmake_minimum_required):
  CMake 3.12 or higher is required.  You are running version 3.10.2

-- Configuring incomplete, errors occurred!

OK, I installed cmake 3.17 from KitWare: https://apt.kitware.com/
Attempting to compile salmon 1.1.0 from source...

Heya, I'd open up an issue on the Salmon repository, they'll be a lot more help getting Salmon built correctly for your setup.

OK, I'll do that - Thanks.
However, things appear to be working OK with "salmon" 0.14.1
What are the drawbacks of using the older version of "salmon"?

Hi @tony-travis,
The new version of Salmon has a supplementary alignment method that we're going to switch over to using by default, so it will be good to get it working. I see you are making progress with Rob over there, let us know how you fare. Thanks for following up with him, it is awesome getting this fixed.

Thanks Tony for getting that straightened out with Rob!


Adjust URL during inlining more selectively

 Issue by sigmundch
Originally opened as dart-lang/sdk#19770

The list indicates not just which attributes to filter, but on what elements:
https://code.google.com/p/dart/codesearch#dart/trunk/dart/pkg/polymer/lib/src/build/import_inliner.dart&l=406
The inliner works fine with normal DOM elements because those attributes are not used for other purposes, but if some custom element happens to use the same attribute names, we shouldn't try to fix it.
See discussion here:
https://groups.google.com/a/dartlang.org/d/msgid/web/6b798af4-d5d3-4825-8397-1fe8d6f786eb%40dartlang.org?utm_medium=email&utm_source=footer

 Comment by sigmundch

Removed Pkg-Polymer label.
Added Pkg-PolymerBuild label.

 Comment by sigmundch

Issue dart-lang/polymer-dart#167 has been merged into this issue.

 Comment by sigmundch

Set owner to @jakemac53.

 Comment by sigmundch

Removed Polymer-P-1 label.
Added Polymer-Milestone-Next label.

 Comment by sigmundch

Removed this from the 1.6 milestone.
Added Polymer-P-1 label.

 Comment by sigmundch

Added PolymerMilestone-Next label.

 Comment by sigmundch

Removed Polymer-Milestone-Next label.

 Comment by sigmundch

Issue dart-lang/polymer-dart#227 has been merged into this issue.

This comment was originally written by<EMAIL_ADDRESS>
I was experiencing the same exception when on polymer version >=0.11.0 <0.12.0. When using the version >=0.12.0 it works fine in Dartium, but throws a new exception in Chrome: "Uncaught TypeError: Cannot read property 'element' of undefined"
That is, after the fix, the code runs good in dartium but blows up in chrome after compiling the solution to java script.
By inspecting the element in chrome, I got:
'Resource interpreted as Script but transferred with MIME type application/dart'
With entry point (after a chain of anonymous functions):
  'isolate_helper.dart:97'
and the blow up at:
  'isolate_helper.dart:438'
I also got this message:
'flushing 26 elements' at 'polymer.js:26' (I don't have a clue to what that means)
Some more info:
If I create my polymer element (that itself depends on core icons) in the web-root directory, then everything works fine and out of the box with polymer version >=0.11.0 <0.12.0. If I move my element to a directory in the web-root directory (i.e. components), then the element blows up (as described by this thread). Upgrading to the new polymer version >=0.12.0 fixes the problem in dartium, but now it blows up in chrome (when compiled to java script).

 Comment by jakemac53

dart-lang/sdk@4b85b6a43a154fa891fd213bef3c84aa1cd3fbb5 changes this so it will only affect attributes on standard html elements.

Added Fixed label.

 Comment by sigmundch

Hey Jan - we recently fixed the bug related to "Cannot read property 'element' of undefined". The issue was a bug in how the "dart:js" was being compiled by dart2js (issue dart-lang/sdk#20305).
The fix is already submitted and will be part of the next dev SDK release (1.6.0-dev.7.0), which I expect will be out today or tomorrow if all looks good. If you need to get past this issue in the meantime, there is a longer discussion and some workarounds described in issue dart-lang/polymer-dart#254.

 Comment by jakemac53

Thanks for the report Jan, I am pretty sure this is related to issue dart-lang/polymer-dart#257, and is not a direct consequence of the bug fix here. I will start looking into that shortly though , and follow up in that issue :).

This comment was originally written by<EMAIL_ADDRESS>
Further looking into the problem gives that no matter where I put my polymer element (web root or some folder under web root), it always blows up when running the code in chrome (that is, solution compiled to java script), polymer version >=0.12.0


Datasets conflicts with fsspec 2024.9

Describe the bug
Installing both in latest versions are not possible
pip install "datasets==3.0.1" "fsspec==2024.9.0"
But using older version of datasets is ok
pip install "datasets==1.24.4" "fsspec==2024.9.0"
Steps to reproduce the bug
pip install "datasets==3.0.1" "fsspec==2024.9.0"
Expected behavior
install both versions.
Environment info
debian 11.
python 3.10.15

Yes, I need to use the latest version of fsspec and datasets for my usecase.
https://github.com/fsspec/s3fs/pull/888#issuecomment-2404204606
https://github.com/apache/arrow/issues/34363#issuecomment-2403553473


Update app-service-move-limitations.md

Update the existing line 22 "You can only move Azure App Service certificate bound to a web without deleting the TLS bindings, as long as the certificate is moved with all other resources in the resource group".
Added this extra limitation at line 24 "You cannot move Free Managed certificates or third party certificates for across subscription migrations".

@abmoh-Microsoft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

The clarification about free managed certificates has already been added.
#please-close


Style/ArgumentsForwarding unsafe autocorrect example

Hi, we updated to rubocop 1.60.0 today and had several auto-correctable offenses for Style/ArgumentsForwarding. Running bundle exec rubocop --only Style/ArgumentsForwarding -A yielded updated code that does not do the same thing as it did before auto-correction.
The corrections that caused issues in our case were examples in which a block was forwarded to another method, along with additional arguments.
If this is already handled as a configuration option, I apologize for not searching hard enough.

Expected behavior
def use_time_zone(&)
  Time.use_zone(current_identity.time_zone, &)
end

This code should either require manual updating, or be corrected to not clobber the non-forwarded arguments (current_identity.time_zone, in this case)
Actual behavior
The above code is auto-corrected to:
def use_time_zone(...)
  Time.use_zone(...)
end

which does not perform the same.
Steps to reproduce the problem
Run bundle exec rubocop --only Style/ArgumentsForwarding -A on above code
RuboCop version
1.60.0 (using Parser <IP_ADDRESS>, rubocop-ast 1.30.0, running on ruby 3.2.2) [arm64-darwin21]
  - rubocop-performance 1.20.2
  - rubocop-rails 2.23.1
  - rubocop-rspec 2.26.1

The issue is caused by the same reason as #12618 and is planned to be resolved in #12624.


How to modify runtime_manager_dialog.py

I wonder how runtime_manager_dialog.py is created, is it generated by some tools? I just want to know if i want to modify the UI of runtime manager, should I directly modify the python code itself?  Thanks.

You have better modify the src/util/packages/runtime_manager/scripts/xxxx.yaml for UI change

UI was created using wx. However, just like @simon750324 mentioned, RTM code creates dynamically the UI reading the yaml files.


Your gem release includes previous release

Hey, I noticed, that in 0.0.5 there's a help_center-0.0.1.gem included in the release. Same with previous releases. This makes the gem size twice as big as it could be.
Cheers

@mensfeld  sorry about that, we will put more work into it later this week


#181 support tag for scalars

This PR adds a new support for scalar tags both in deserialization and serialization.
Along with the above change, the way of handling YAML version directive (such as %YAML 1.2) has also been changed without changing the public fkYAML APIs.
Note that tags for mappings and sequences are not yet supported, but they will soon be supported.

Pull Request Checklist
Read the CONTRIBUTING.md file for detailed information.

[x] Changes are described in the pull request or in a referenced issue.
[x] The test suite compiles and runs without any error.
[x] The code coverage on your branch is 100%.
[x] The documentation is updated if you added/changed a feature.

Please don't

The C++11 support varies between different compilers and versions. Please note the list of supported compilers. Some compilers like GCC 4.7 (and earlier), Clang 3.3 (and earlier), or Microsoft Visual Studio 13.0 and earlier are known not to work due to missing or incomplete C++11 support. Please refrain from proposing changes that work around these compiler's limitations with #ifdefs or other means.
Please refrain from proposing changes that would break YAML specifications. If you propose a conformant extension of YAML to be supported by the library, please motivate this extension.
Please do not open pull requests that address multiple issues.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

598 of 598 (100.0%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3449

Relevant Lines:
3449

💛  - Coveralls


Keyboard shortcuts widget in bottom-left of webapp UI is buggy

This widget:

Has a couple important bugs related to the tooltip hover behavior:

There's a weird flicker one can trigger by hovering over it in the right spot.
If you try to click on the keyboard icon to open "keyboard shortcuts" when hovered over roughly the top half of the icon, it doesn't work; I think the tooltip is eating the click.

I would like to work on this issue. If I am not wrong, this widget can be found in the templates/zerver/app/right_sidebar.html file? There is an  tag to show the keyboard icon there. However I am not being able to understand what exactly is causing the flickering effect you mentioned, or how I can resolve it. Any guidance from your side would be appreciated.

Great!  That sounds right to me.  I'm not sure what causes it either -- @vinitS101 might have some ideas.  But my guess is that we have logic to show a tooltip over it, and that if you find that logic (git grep for its selector will be helpful), you'll find the responsible code.

I am unable to reproduce the clicking issue, but I have experienced the flicker bug before. A solution I found for this was to replace it with proper js tooltips (like I've done in #14539 here).

Ok, so replacing the tooltip with a JS tooltip should fix the flickering then? If so I will try to implement this.

We've been actively replacing HTML titles with js tooltips so it's a change
we've wanted to do for a while now. It has the added advantage of fixing
this bug, primarily since you can have custom positioning of the tooltip to
avoid any clicking or flickering issues.
On Thu, Oct 1, 2020 at 2:33 PM Harsh Sharma<EMAIL_ADDRESS>wrote:

Ok, so replacing the tooltip with a JS tooltip should fix the flickering
then? If so I will try to implement this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip/issues/16438#issuecomment-701996924, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AGBH7O75HEBZ5STP3R4U6HLSIRAXHANCNFSM4R6ND6GQ
.

@vinitS101 In the PR you have written a function called create_generic_tooltips to create tooltips using jQuery .  However, when I tried to apply the code as written in the PR to the keyboard-icons  element, the tooltip is not displayed. I have tried changing the position of the tooltip as well, but to no avail. This code seems to be working for all the other tooltips though. I am completely clueless as to why it isn't working for the keyboard-icons  tooltip.
For reference, here is the code that was written in the PR:
$("#streams_filter_icon").on("mouseenter", (e) => {
        e.stopPropagation();

        const elem = $(e.currentTarget);
        const hotkey = '(q)';
        const title = i18n.t("Filter streams __hotkey__", {hotkey: hotkey});

        ui.create_generic_tooltip(elem, title, "left");
    });
$("#streams_filter_icon").on("mouseleave", (e) => {
        e.stopPropagation();
        $(e.currentTarget).tooltip('hide');
    });

Here is the ui.create_generic_tooltip function:
exports.create_generic_tooltip = function (elem, title, placement) {
    elem.tooltip({
        title: title,
        trigger: 'hover',
        placement: placement,
        animation: false,
    });
    elem.tooltip('show');
    $(".tooltip-arrow").remove();
};

This code works perfectly and displays a tooltip correctly without flickering problems. However, as I described above, when I apply this code for the keyboard-icons element, it doesn't show a tooltip. Can anyone please help me with this issue? Thanks for your help in advance 😄

@hs2361 That function is a part of the PR I shared, and it is yet to be merged. I refactored some of the existing code to create that re-usable fun 1 of 2 things to fix this:

Rebase on top of that commit from my PR and then proceed as you are right now.
or, don't use create_generic_tooltips and just use that code to get tooltips for your specific element (like we have been doing before this).

For the 2nd option, refer to the code that is replaced by this function in my original PR.

I did exactly what you mentioned in your second point, I took the code that you wrote for the create_generic_tooltips function and applied it to the keyboard-icons element, but the tooltip wasn't appearing. I tested the function on some of the other elements on the page and it seemed to be working there. I thought there was some kind of overlay on top of the tooltip due to which it wasn't seen, but even after changing the zIndex of the tooltip, it wasn't visible. Here is what I changed the element to:
<i class="fa fa-keyboard-o fa-2x" aria-label="{{ _('Keyboard shortcuts') }} (?)" id="keyboard-icon">
As you can see I followed what you've done in the PR. On the JS side, I wrote this:
$("#keyboard-icon").on("mouseenter", (e) => {
        e.stopPropagation();

        const elem = $(e.currentTarget);
        elem.innerText = "xyz"
        const hotkey = '(?)';
        const title = i18n.t("Keyboard Shortcuts __hotkey__", {hotkey: hotkey});

        elem.tooltip({
            title: title,
            trigger: 'hover',
            placement: 'top',
            animation: false,
        });

        elem.tooltip('show');
        $(".tooltip-arrow").remove();
});

$("#keyboard-icon").on("mouseleave", (e) => {
        e.stopPropagation();
        $(e.currentTarget).tooltip('hide');
});

Once again, this follows what you wrote in your PR, I just added in the code for the ui.create_generate_tooltip function. However, while this works for other elements on the page, the tooltip isn't shown on the element.
Do you know what I can do to resolve this?

It'd be much easier to review if you pushed a PR.
Did you remove the existing html tooltip from templates/zerver/app/right_sidebar.html?

It'd be much easier to review if you pushed a PR.
Did you remove the existing html tooltip from templates/zerver/app/right_sidebar.html?

Yes, I did remove the existing tooltip. I was hesitant to open a PR because I didn't want to push code that didn't work. Nevertheless, I will open one, and we could discuss it there.

@hs2361 @vinitS101 is this issue fixed? I would like to contribute.


Changes in preparation for 6.5.3 preview release

Codecov Report

Merging #710 into dev will decrease coverage by 0.08%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev     #710      +/-   ##
============================================
- Coverage     48.11%   48.03%   -0.09%
+ Complexity     2578     2576       -2
============================================
  Files           113      113
  Lines         26553    26553
  Branches       4457     4457
============================================
- Hits          12777    12754      -23
- Misses        11637    11664      +27
+ Partials       2139     2135       -4

Flag
Coverage Δ
Complexity Δ

#JDBC42
47.83% <ø> (-0.18%)
2562 <ø> (-9)

#JDBC43
47.99% <ø> (+0.09%)
2576 <ø> (+7)
:arrow_up:

Impacted Files
Coverage Δ
Complexity Δ

...a/com/microsoft/sqlserver/jdbc/SQLJdbcVersion.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...om/microsoft/sqlserver/jdbc/ReaderInputStream.java
42.69% <0%> (-4.5%)
14% <0%> (-2%)

...in/java/com/microsoft/sqlserver/jdbc/IOBuffer.java
54.78% <0%> (-0.67%)
0% <0%> (ø)

...m/microsoft/sqlserver/jdbc/SQLServerResultSet.java
33.04% <0%> (-0.08%)
244% <0%> (-1%)

...rc/main/java/com/microsoft/sqlserver/jdbc/dtv.java
63.3% <0%> (-0.06%)
0% <0%> (ø)

...ncurrentlinkedhashmap/ConcurrentLinkedHashMap.java
39% <0%> (+0.21%)
43% <0%> (ø)
:arrow_down:

...rc/main/java/com/microsoft/sqlserver/jdbc/DDC.java
46.08% <0%> (+0.67%)
108% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


Inline a once-used one-line function

Already done in OpenBSD.

@Pyrohh Thanks. Reasons not to manually replay diffs on Christmas morning.   :-)

I'm gonna rebase these into a single commit.

Merged/rebased together:
commit 5b191cffb2813639abfc489be402c0a7eced4fce
Author: Michael McConville<EMAIL_ADDRESS>Date:   Fri Dec 25 13:01:38 2015 -0500
Inline a once-used one-line function


chore: refactor pool_noodle loop to use cancellation token and config

Post the Veritech Jetstream changes, this refactors PoolNoodle to run over a single loop that works based on tasks in a queue. The veritech cancellation token is threaded through to we shutdown and clean up when veritech is shut down. We now only spin up as many tasks as there is max_concurrency to help manage back pressure (it's currently set to an arbitrarily high number).
This also does some quality of life changes, a config struct, better defaults, and more metrics to track things like the number of jobs waiting for instances when we're under enough pressure.
Here's an undersized pool fighting for its life.

/try

/try

/try

/try

/try

/try

/try

/try

/try

/try

/try

/try

/try

/try

/try

/try


Standardize code style using Black

This PR proposes to add Black into Koalas.

dev/linter-python detects if codes are formatted or not by Black
dev/reformat script that reformats the codes by using black.
It still keeps 100 line length style, and ignores E231 and E203 at pycodestyle. See also https://github.com/psf/black/issues/429 and https://github.com/psf/black/issues/1202. Black automatically formats and violates several rules.
Update contribution guide.

Codecov Report

Merging #1301 into master will decrease coverage by 0.02%.
The diff coverage is 93.94%.

@@            Coverage Diff             @@
##           master    #1301      +/-   ##
==========================================
- Coverage   95.06%   95.04%   -0.03%
==========================================
  Files          34       34
  Lines        7233     7238       +5
==========================================
+ Hits         6876     6879       +3
- Misses        357      359       +2

Impacted Files
Coverage Δ

databricks/koalas/missing/common.py
100% <ø> (ø)
:arrow_up:

databricks/koalas/frame.py
96.51% <ø> (+0.05%)
:arrow_up:

databricks/conftest.py
96.22% <100%> (+1.88%)
:arrow_up:

databricks/koalas/numpy_compat.py
90.9% <100%> (ø)
:arrow_up:

databricks/koalas/missing/frame.py
100% <100%> (ø)
:arrow_up:

databricks/koalas/usage_logging/usage_logger.py
100% <100%> (ø)
:arrow_up:

databricks/koalas/missing/__init__.py
100% <100%> (ø)
:arrow_up:

databricks/koalas/mlflow.py
94.87% <100%> (ø)
:arrow_up:

databricks/koalas/missing/groupby.py
100% <100%> (ø)
:arrow_up:

databricks/koalas/usage_logging/__init__.py
98.19% <100%> (ø)
:arrow_up:

... and 26 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 f8b60d2...38a701f. Read the comment docs.

cc @ueshin and @itholic

I manually pushed to keep the co-author - https://github.com/databricks/koalas/commit/30b33344cdf7c41e2a2061daed314cdd55ec3215
Let's see if it's really useful ...

nice!


Implement "loading" property in dummyVis

Make dummyVis have a loading boolean property. Clicking and dragging the X should set the loading property to true for one second. The purpose of this is to simulate asynchronous operations that may fetch data from a server.

Not sure this makes sense, may revisit later.


Cannot tab back into an open floating OverflowMenu

Environment

Operating system

Mac osX

Browser

Chrome v74.0

Automated testing tool and ruleset

Assistive technology used to verify

Keyboard navigation
Detailed description

What version of the Carbon Design System are you using?

10.1.0

What did you expect to happen?

Could navigate via keyboard back into an open floating OverflowMenu

What happened instead?

Tab focus never returns to the menu items

What WCAG 2.1 checkpoint does the issue violate?

Steps to reproduce the issue

Tab to menu trigger
Press space to open
Focus goes to primaryFocus item
Tab out of menu
Cannot tab back in

Please create a reduced test case in CodeSandbox
https://codesandbox.io/s/github/IBM/carbon-components-react/tree/master/examples/codesandbox

https://codesandbox.io/s/kon9z6o0rv

Hi 👋 thank you for reporting - Here's the way you won't be lost with navigation. First, you can add data-floating-menu-container attribute to the containing <div>. Then you can add an focusable element next so such <div>. Your App component will be like:
<>
  <div data-floating-menu-container>
    <OverflowMenu floatingMenu>
      <OverflowMenuItem primaryFocus>text1</OverflowMenuItem>
      <OverflowMenuItem>text2</OverflowMenuItem>
    </OverflowMenu>
  </div>
  <input type="text" class="bx--visually-hidden" />
</>

Hope this helps!

@asudoh - Hi there, I have updated the code sandbox supplied and the issue is still persisting.
Could you confirm whether the data-xyz attributes can be used with the React components?

Seems that your CodeBox hasn't been updated, the may explain why you are still seeing the problem. React JSX accepts data-* and aria-* with dash case. Once you replace the content with my markup I see the tab is not lost even if I tab away from the menu item.

@asudoh - the code box appears to be updated when I or a colleague opens the link. I can see in the Chrome debugging tool that the data-overflow-menu-container attribute is in the HTML but it is not working for me.
Is there a minimum version of carbon-components or carbon-components-react I need?

data-overflow-menu-container has been supported for long time. Let me clarify what I mentioned earlier a bit - When I put my markup into your CodeSandbox, the overflow menu started working in following way; Clicking on the overflow meun's trigger button puts the focus on the first item. Hitting tab then puts the focus back to the trigger button.

@asudoh - Ah, thank you - it seems the additional empty <> wrapping the data-overflow-menu-container <div> was the bit it needed to make it work. I'm assuming that adding data-overflow-menu-container to the root dom node was the problem?
Out of interest - I'm not seeing this in the react storybook. Is there a plan to update applicable storybook items to reference the need for the data-* attributes.

@nicthemighty Additional <input> is not necessary for most applications, given it's very rare that the overflow menu is the last element in the document. However, you still have a good point and will consider better documentation.


Convert dataset already on disk to webdataset

Hello,
I have a dataset that already is in ImageFolder format. Can I use this code to convert it to a webdataset?

No but there is an easy way to do this.
tar cf 0.tar your_folder/
You may create several such tars if you have many images
On Fri, Oct 14, 2022, 00:44 AlaaKhaddaj @.***> wrote:

Hello,
I have a dataset that already is in ImageFolder format. Can I use this
code to convert it to a webdataset?
—
Reply to this email directly, view it on GitHub
https://github.com/rom1504/img2dataset/issues/211, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAR437R63U76I6UVSHXKV4TWDCGETANCNFSM6AAAAAAREW7I64
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I see, thanks!
Also, if I have a custom dataset object with len and getitem, do you know how I can save it as a webdataset?

you can use tarwriter from webdataset


All sample files can use IPAM

Using IPAM reduces the complexity of required Ansible variables and minimises confusion.

/retest-required

recheck

/retest

/lgtm


crash when running two server 1 replicate and memory is full

I got my first crash, and I can reliably recreate this:

Start two disque-server and have them cluster meet
Connect to one of them (cluster meet...)
Add jobs like this: addjob q1 PAYLOAD replicate=1. I used 10KB payload.
Add jobs until memory is full, only one of the server is getting full
When the server that is full runs out of memory, it crashes

I first observed it with a 1GB maxmemory, then again with 100MB, so it's not the physical's server's memory limitation.
If I don't specify replicate, it works as expected, when the servers run out of memory, I get this:
NOREPL Timeout reached before replicating to the requested number of nodes
=== DISQUE BUG REPORT START: Cut & paste starting from here ===
28284:P 06 Jan 16:52:15.412 #     Disque 1.0-rc1 crashed by signal: 11
28284:P 06 Jan 16:52:15.412 #     Failed assertion: <no assertion failed> (<no file>:0)
28284:P 06 Jan 16:52:15.412 # --- STACK TRACE
./src/disque-server *:7770(logStackTrace+0x43)[0x42ab33]
./src/disque-server *:7770(dictAddRaw+0x14)[0x4135b4]
/lib64/libpthread.so.0[0x309d80f710]
./src/disque-server *:7770(dictAddRaw+0x14)[0x4135b4]
./src/disque-server *:7770(dictAdd+0x1e)[0x4137ae]
./src/disque-server *:7770(clusterProcessPacket+0x7ab)[0x42fbcb]
./src/disque-server *:7770(clusterReadHandler+0xdb)[0x43020b]
./src/disque-server *:7770(aeProcessEvents+0x13c)[0x4108fc]
./src/disque-server *:7770(aeMain+0x2b)[0x410bbb]
./src/disque-server *:7770(main+0x39d)[0x417d5d]
/lib64/libc.so.6(__libc_start_main+0xfd)[0x309d41ed5d]
./src/disque-server *:7770[0x40ff19]
28284:P 06 Jan 16:52:15.414 # --- INFO OUTPUT
28284:P 06 Jan 16:52:15.414 # # Server
disque_version:1.0-rc1
disque_git_sha1:7dad5666
disque_git_dirty:0
disque_build_id:df2612766b0503d8
os:Linux 2.6.32-504.12.2.el6.x86_64 x86_64
arch_bits:64
multiplexing_api:epoll
gcc_version:4.4.7
process_id:28284
run_id:6114fb4470d99c4aea5098221d34471dde35caf3
tcp_port:7770
uptime_in_seconds:79
uptime_in_days:0
hz:10
executable:/.local/work/mlongtin/github/disque/./src/disque-server
config_file:/.local/work/mlongtin/github/disque/d7770/disque.conf

# Clients
connected_clients:1
client_longest_output_list:0
client_biggest_input_buf:0
blocked_clients:0

# Memory
used_memory:75004472
used_memory_human:71.53M
used_memory_rss:83181568
used_memory_peak:75004472
used_memory_peak_human:71.53M
mem_fragmentation_ratio:1.11
mem_allocator:jemalloc-4.0.3

# Jobs
registered_jobs:6992

# Queues
registered_queues:1

# Persistence
loading:0
aof_enabled:0
aof_state:off
aof_rewrite_in_progress:0
aof_rewrite_scheduled:0
aof_last_rewrite_time_sec:-1
aof_current_rewrite_time_sec:-1
aof_last_bgrewrite_status:ok
aof_last_write_status:ok

# Stats
total_connections_received:1
total_commands_processed:6994
instantaneous_ops_per_sec:573
total_net_input_bytes:70374510
total_net_output_bytes:301098
instantaneous_input_kbps:5633.30
instantaneous_output_kbps:24.07
rejected_connections:0
latest_fork_usec:0

# CPU
used_cpu_sys:2.75
used_cpu_user:1.36
used_cpu_sys_children:0.00
used_cpu_user_children:0.00

# Commandstats
cmdstat_hello:calls=2,usec=29,usec_per_call=14.50
cmdstat_addjob:calls=6992,usec=270174,usec_per_call=38.64
hash_init_value:<PHONE_NUMBER>

28284:P 06 Jan 16:52:15.414 # --- CLIENT LIST OUTPUT
28284:P 06 Jan 16:52:15.414 # id=1 addr=<IP_ADDRESS>:48795 fd=11 name= age=68 idle=0 flags=N qbuf=0 qbuf-free=32768 obl=0 oll=0 omem=0 events=r cmd=addjob

28284:P 06 Jan 16:52:15.414 # --- REGISTERS
28284:P 06 Jan 16:52:15.414 #
RAX:00007f675c90d800 RBX:0000000000000000
RCX:0000000000000000 RDX:00007f6762463000
RDI:0000000000000000 RSI:00007f6762463000
RBP:0000000000000000 RSP:00007ffff70de2c0
R8 :00000000b32ca4f0 R9 :00007f675c7b30bd
R10:00000000ffffffdc R11:0000000000000009
R12:00007f6762463000 R13:00007f676241e3f0
R14:0000000000000000 R15:0000000000000005
RIP:00000000004135b4 EFL:0000000000010206
CSGSFS:0000000000000033
28284:P 06 Jan 16:52:15.414 # (00007ffff70de2cf) -><PHONE_NUMBER>000178
28284:P 06 Jan 16:52:15.414 # (00007ffff70de2ce) -> 00007f675c7b3005
28284:P 06 Jan 16:52:15.414 # (00007ffff70de2cd) -><PHONE_NUMBER>42fbcb
28284:P 06 Jan 16:52:15.414 # (00007ffff70de2cc) -> 00007f675c7b3005
28284:P 06 Jan 16:52:15.414 # (00007ffff70de2cb) -> 00007f675c90d800
28284:P 06 Jan 16:52:15.414 # (00007ffff70de2ca) -> 00007f6762463000
28284:P 06 Jan 16:52:15.414 # (00007ffff70de2c9) -><PHONE_NUMBER>4137ae
28284:P 06 Jan 16:52:15.414 # (00007ffff70de2c8) -><PHONE_NUMBER>000005
28284:P 06 Jan 16:52:15.414 # (00007ffff70de2c7) -><PHONE_NUMBER>000000
28284:P 06 Jan 16:52:15.414 # (00007ffff70de2c6) -> 00007f676241e3f0
28284:P 06 Jan 16:52:15.414 # (00007ffff70de2c5) -> 00007f6762463000
28284:P 06 Jan 16:52:15.414 # (00007ffff70de2c4) -> 00007f675c90d800
28284:P 06 Jan 16:52:15.414 # (00007ffff70de2c3) -><PHONE_NUMBER>000000
28284:P 06 Jan 16:52:15.414 # (00007ffff70de2c2) -><PHONE_NUMBER>000180
28284:P 06 Jan 16:52:15.414 # (00007ffff70de2c1) -> 00007f6762a00180
28284:P 06 Jan 16:52:15.414 # (00007ffff70de2c0) -><PHONE_NUMBER>00000f
28284:P 06 Jan 16:52:15.414 # --- FAST MEMORY TEST
28284:P 06 Jan 16:52:15.416 # Bio thread for job type #0 terminated
28284:P 06 Jan 16:52:15.416 # Bio thread for job type #1 terminated
Testing<PHONE_NUMBER>2
Testing 1e4a000 135168
Testing 309ce21000 4096
Testing 309d78f000 20480
Testing 309da19000 16384
Testing 7f675c600000 75497472
Testing 7f6760fff000 10485760
Testing 7f6761a00000 12582912
Testing 7f67626d0000 4096
Testing 7f6762a00000 2097152
Testing 7f6762c82000 16384
Testing 7f6762ca8000 8192
28284:P 06 Jan 16:52:16.845 # Fast memory test PASSED, however your memory can still be broken. Please run a memory test for several hours if possible.
28284:P 06 Jan 16:52:16.845 #
=== DISQUE BUG REPORT END. Make sure to include from START to END. ===

       Please report the crash by opening an issue on github:

           http://github.com/antirez/disque/issues

  Suspect RAM error? Use disque-server --test-memory to verify it.

Segmentation fault

Ok I can reproduce this very easily. Fixing.

p.s. Thanks for submitting!

Fixed, thank you.


Issues Related to the Discord Status Script

Describe your problem here.
Whenever I try to compile, I get these 2 errors
source/Discord.hx:63: characters 48-49 : Unexpected )
source/Discord.hx:63: characters 48-49 : Unexpected )
Can anyone help (or at least tell me the problem?)
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No

Either add or remove the ) at line 63

I tried that, but:
When I added an ), it gave me the same error
When I removed it, it gave me this error
source/Discord.hx:67: characters 2-7 : Unexpected trace
source/Discord.hx:4: characters 7-29 : Type not found : discord_rpc.DiscordRpc

Should be fixed??


Samples get saved on sample location

 Issue by dakshika
Monday Jun 12, 2017 at 05:45 GMT
Originally opened as https://github.com/ballerina-lang/composer/issues/1915

When you save a sample it will save under sample location and override the file.
ballerina-tools-0.88/samples/echoService/echoService.bal
This will kill the sample behavior and if some user change accidentally and they can't revert back to original due to this bug
Tested on ballerina-tools-0.88

 Comment by dakshika
Tuesday Jun 27, 2017 at 08:50 GMT

this has to fix below issue as well.

unsaved sample name on breadcrumb should be "sampleName" or "Untitled"

 Comment by dakshika
Thursday Jul 06, 2017 at 10:34 GMT

similar issue https://github.com/ballerinalang/composer/issues/1832

Composer is discontinued hence closing the issue.


Exporting pck file from command line with --quit or --disable-render-loop or from a server build hangs on thumbnail generation

Godot version:
Latest Master
OS/device including version:
Problem observed both on Archlinux and Debian, but is more a design issue/limitation, so will appear everywhere.
Issue description:
When running the command:
godot  --path path/to/game --export "pck" test.pck
Godot will freeze - but only if you've never opened the project before. If you have opened the project in the Godot editor, then it exports successfully.  If you have opened it in the editor, you can force it to happen by deleting the thumbnails from .cache/godot.
This problem is because the server build does not draw the display, and the thumbnailer waits for a draw to happen. As a result, it gets stuck in this loop (which is present in a couple of preview generators in editor_preview_plugins.cpp):
        preview_done = false;
	VS::get_singleton()->request_frame_drawn_callback(const_cast<EditorFontPreviewPlugin *>(this), "_preview_done", Variant());

	while (!preview_done) {
		OS::get_singleton()->delay_usec(10);
	}

Minimal reproduction project:
Any project will exhibit this, so long as you remove the thumbnail cache first.
Currently this is preventing automated-export of projects from a headless environment.

Here is an example project.
texture-server-export-fail.zip
Do not open with the godot editor, because then it will work. If you do, you should delete ~/.cache/godot to re-experience the bug.
Instead, inside there are four shell scripts:
godot --path $(dirname "$0") --export "pck" test1.pck --disable-render-loop
godot --path $(dirname "$0") --quit --export "pck" test2.pck
godot --path $(dirname "$0") --export "pck" test3.pck
godot-server --path $(dirname "$0") --export "pck" test4.pck -q

All of these leave godot frozen.


⚠ Roblox Site has degraded performance

In d296dcb, Roblox Site (https://www.roblox.com) experienced degraded performance:

HTTP code: 503
Response time: 14793 ms

Resolved: Roblox Site performance has improved in a341cce after 28 minutes.


[stable/openvpn] Updated README Client Cert install instructions

Added bash shell md for logging the openvpn pod
Modified kubectl get svc instructions
Replaced the jsonpath to search for hostname instead of ip
Replaced namespace with default, due parse error thrown in kubectl v1.5.2

@VDuda are you able to continue with this?

Closing as this has been stale for more than a week.
https://github.com/kubernetes/charts#stale-pull-requests


Use mapserver in signer

Use an internal implementation of the trillian map server.
In the keyserver:

Use GetLeaves to retrieve leaf values and inclusion proofs.
Use GetSignedMapRoot to retrieve the current epoch.  (Replaces sths object)

In the signer:

Collect all mutations that have occurred since the last epoch.
GetLeaves for the respective indexes
Verify and Apply the mutations.
Call SetLeaves to create a new epoch with the new values.

TODO:

Pass an expected epoch field to make sure that the read/write cycle to the mapserver is atomic. google/trillian#559
Use mastership election to make sure that only one signer is active at any point. #565

Contributes to #486

Codecov Report

Merging #564 into master will increase coverage by 0.35%.
The diff coverage is 58.03%.

@@            Coverage Diff             @@
##           master     #564      +/-   ##
==========================================
+ Coverage   59.36%   59.71%   +0.35%
==========================================
  Files          29       30       +1
  Lines        2124     2140      +16
==========================================
+ Hits         1261     1278      +17
  Misses        593      593
+ Partials      270      269       -1

Impacted Files
Coverage Δ

core/client/kt/verify.go
27.9% <0%> (ø)
:arrow_up:

core/mapserver/mapserver.go
50.6% <100%> (+0.6%)
:arrow_up:

core/mapserver/readonly.go
41.66% <41.66%> (ø)

core/keyserver/keyserver.go
42.85% <61.9%> (+6.16%)
:arrow_up:

integration/testutil.go
70.64% <69.23%> (+0.94%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #564 into master will increase coverage by 0.48%.
The diff coverage is 57.25%.

@@            Coverage Diff             @@
##           master     #564      +/-   ##
==========================================
+ Coverage   59.36%   59.85%   +0.48%
==========================================
  Files          29       30       +1
  Lines        2124     2142      +18
==========================================
+ Hits         1261     1282      +21
+ Misses        593      591       -2
+ Partials      270      269       -1

Impacted Files
Coverage Δ

core/client/kt/verify.go
27.9% <0%> (ø)
:arrow_up:

core/mapserver/mapserver.go
55.22% <100%> (+5.22%)
:arrow_up:

core/mapserver/readonly.go
42.18% <42.18%> (ø)

core/keyserver/keyserver.go
43.75% <65.21%> (+7.05%)
:arrow_up:

integration/testutil.go
70.64% <69.23%> (+0.94%)
:arrow_up:

Continue to review full report at Codecov.

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

Addressed comments. PTAL

Codecov Report

Merging #564 into master will increase coverage by 0.57%.
The diff coverage is 60.3%.

@@            Coverage Diff             @@
##           master     #564      +/-   ##
==========================================
+ Coverage   59.36%   59.94%   +0.57%
==========================================
  Files          29       30       +1
  Lines        2124     2142      +18
==========================================
+ Hits         1261     1284      +23
+ Misses        593      589       -4
+ Partials      270      269       -1

Impacted Files
Coverage Δ

core/client/kt/verify.go
27.9% <0%> (ø)
:arrow_up:

core/mapserver/readonly.go
51.56% <51.56%> (ø)

core/keyserver/keyserver.go
43.75% <65.21%> (+7.05%)
:arrow_up:

integration/testutil.go
70.64% <69.23%> (+0.94%)
:arrow_up:

core/mapserver/mapserver.go
49.25% <86.66%> (-0.75%)
:arrow_down:

Continue to review full report at Codecov.

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


Scrollbar jumps back and forth

The scrollbar often jumps back and forth before starting the animation.
This is best explained in a short video: react-scrollchor.mp4

@xehpuk sorry! I don't get it

There you go, with 5 fps:

As you can see, the scrollbar jumps to the top of the page for one frame.

In the example "Home" Scrollchor has to set to "",   that's why, in this case, scroll jump to the top of the page. Just like will do <a href="" ...

It jumps to the top before scrolling from bottom to top.

@xehpuk I still don't see the behaviour that you describe. Maybe is relate to your browser or default animation function, easeOutQuad

Finally I see it!
I will investigate it.  Any help on is welcome.

The example in localhost works well
Maybe is relate to github gh-pages hosting. I had to tweak the example a little for deploy on gh-pages
Can you test it in your own server?

I will close this for now.
Please, reopen it if needed.

@sohbr  in what browser do you tested?

I also used different easing functions and it produced the same effects.

Maybe is related to this: https://github.com/bySabi/react-scrollchor/issues/30 , do you set some offset value on animate prop??


Add an option to ignore blueprints on the quick trash all

Could you add an option for the quick trash all to ignore blueprints?  It's nice to use the shortcut but not when it trashes all my blueprints into my network.

Heh, apparently I removed the Quick-trash-all excludes option and never re-added it. Oops.


Applying rune to a weapon while dual wielding causes incorrect damage rolls

Specifically, I applied the shocking rune to a dueling pistol while also wielding the erraticannon. Now when rolling damage for one of the weapons when clicking the button in the chat it rolls the damage for the other weapon.

This also occurs when testing with a second dueling pistol.
Damage is rolled correctly when not dual wielding.
Damage is also rolled correctly when duel wielding two guns that are not the weapon with the rune.

A bit more testing: damage is rolled correctly when done through the character sheet

Unable to reproduce this issue. Try with no modules enabled.


Allow syntactic abstraction of refinements via "Interpreted" Functions dfn

I've keenly felt the absence of this feature for awhile -- it lets you lift common patterns into flux-level functions which are really macros that get inlined into the relevant places in the specifications. See

flux-tests/tests/pos/surface/ealias00.rs
flux-tests/tests/pos/surface/ealias01.rs
flux-tests/tests/pos/surface/ealias02.rs

(and neg versions)
Mostly self contained in the expand.rs so we can easily revert later.
Note: The one thing I cannot figure out is the bizarre DefinitionCycle error - which is thrown if you have cyclic / mutually recursive dfn . Currently, I get some strange cannot find fluent bundle run-time error, so just using the panic! till this gets sorted out later.

btw, this PR was somewhat motivated by wanting to do this example
https://github.com/liquid-rust/flux/pull/238/commits/43bfa891dccf727df6358ef21d7141ffd69aa143#diff-b41377276614e16abfd86f1948720ceef91e72bae75241b28898b0b49cad0e9a
it would be nice I think at some future point to not have to "duplicate" the definitions of is_leap_year and such... maybe with some macro tricks?

@nilehmann - I shifted all the stuff to rty level - very civilized & pleasant to use the TypeFolder API!

Still have to keep the Span to fail for cyclic defns - in theory could check that in FHir but that seems like gross duplication…

I missed the bit about re keeping spans in rty -- by "this is cleaner" do you mean (a) "keeping the Span in rtyor (b) using theMap-- I totally forgot about theMapthing, so yes we can definitely use that, but didn't follow which one you thought is cleaner -- (I assume you meant using theMapto avoid duplicating theSpan`?)
Btw, the real puzzler for me is the "missing fluent bundle" thing, I could never figure that out so have left the cycle error as a panic for now :-( ...

I think (b) is cleaner, but if it makes the code much more complicated having the Span in rty is not that big of a deal I guess.
Were you emitting the error with FluxSession::emit or with tcx.sess().emit()? this is one reason I can think of you could be getting a missing fluent bundle. You have to use the the emit in FluxSession


Fix: edit spelling of "Philadelphia" where needed (Dashboard show?)

Edit spelling of "Philadelphia" where needed (Dashboard show page / result cards?)
Edit landing page text:  "smoke detectors" --> "code compliant"


Sendgrid: email setup and daily email

Propose
We have a SendGrid account but need to set it up. Use cases will grow here, but to start we want to be able to send the following automated messages:

Daily email when there was new data in the last 24 hours
Weekly digest that calls out key points ... ex: data that hasn't been downloaded over the last 7 days

Requirements

Get SendGrid license
Create accounts for team [Matt, James, Drew, ... ]
Create<EMAIL_ADDRESS>and get access to inbox
Backend data connection
Email content + logic
HTML templates
Partners should be able to opt out of some emails

Matt to switch things over for tomorrow 8/12, to go live on Mon 8/15.


No. of times a sorted array is rotated using Binary search

issue : #169
This program counts the number of times a sorted array is rotated using Binary search.
This is done using Java.

@tarun26091999 the first one is from another branch. In this PR there is only one issue

@tarun26091999 I have sent 2 PRs so it is like that

What are there 2 files? Shouldn’t it be only 1?
Also, you need to update respective readme too

@Amit366 You have to resolve the conflicts otherwise we can't merge your PR

@Rohan27s I am not getting the option for resolving

@Rohan27s atleast now merge it . Its been open since when.


Check Availability Fails

The check availability button has an error and is not running. I have checked the App monitor in PowerApps and get a Bad Gateway error and this in the app.

@AlanMRichards the most common reason for this error is that the data connections and variables inside the 'CheckTeamAvailability' flow have not been updated.
Did you update these as per the documentation?
Thanks

Hi! I had the same issue. The problem was I didn't modify the List Name under Check Team Availability Flow > Check site exists response > Check for existing requests matching URL. The Deployment Guide say:

But it doesn't say anything about modifying the "List Name" field. This field must be replaced by "Teams Requests", that is the List Name.
I hope it helps! Thanks!

There is only the
Team request Approval flow but not an Check Team Availability flow after importing the zip file?

What did i do wrong?
Thanks

There is only the
Team request Approval flow but not an Check Team Availability flow after importing the zip file?

What did i do wrong?
Thanks

The "Check Team Availability" flow is imported when you import the Power App package as it is linked to the Power Apps (Step 3: Set up Power Apps). When you import this package, you should see the following:

I also see the badgateway error message. I have tried multiple browsers and signed-in as different Flow owners. I have deleted and recreated my SharePoint Flow connection. I have recreated the steps in the Flow where it references SharePoint, "Check if site exists" and "check for existing requests matching url". I confirmed they are pointing to the correct SPO site and in the 2nd step, the correct list on that site. I gave the user account site Owner and Site Collection Admin to the Request Teams Site
I'm still getting the same error.

I am struggling with this same thing. I have checked and rechecked my flow and cannot see what I am missing. My flow completes successfully but my app logic fails on "bad gateway". Anyone have some possible ideas on this?

@alexc-MSFT; I've taken a look at this and the variable for "TeamsRequestSiteURL" is being used as the input for the "Send an HTTP request to SharePoint" step. This is incorrect.

Instead; the full tenant name should be used "https://m365x765471.sharepoint.com" with no site defined. Then you get the desired "true" or "false" result and not a 404.
Hope this is useful.
Cheers.

b/lists/ge

When I use the URL as you directed I am getting the following error:
{
"status": 401,
"message": "{"error_description":"Exception of type 'Microsoft.IdentityModel.Tokens.AudienceUriValidationFailedException' was thrown."}\r\nclientRequestId: b1e2fc0c-c7cd-4e44-a6b6-5ff06d1ddfa7\r\nserviceRequestId: 3f409fa0-e0ab-6000-4db6-fced5cc9ae54",
"source": "https://m365x765471.sharepoint.com/_api/SP.Site.Exists",
"errors": []
}

I still get the bad gateway error here, any ideas?


Env -> Envs in example json

Love the tool, but this tripped me up for quite a while. It looks like it's actually parsing "envs" and not "env" for environment variables in the deploy.  Let me know if you disagree or if I'm misunderstanding something!

Hi @BrianNewsom , you are right, it's a better name. But I guess we need to add some backward compatibility warnings so people are not surprised. Gonna add it when have time. Thanks for the suggestion and glad this tool really helps you.

Thanks for your response!
Just to clarify - when I used the Env keyword, my environment variables were not passed at all to the server.  I was trying to make this update to the readme assuming that "Envs" is the expected keyword, as that is what I have now and the parameters are correctly passed.
I didn't mean to propose a change, but rather simply update the documentation. But maybe I am confused as to what is really happening in the app.

Oops, sorry, I misunderstood it. Merged. Thanks again. :+1: :D


Separate Firebird v3 from earlier Firebird versions

Description
Firebird 3 has some differences with respect to the created SQL statements. Currently there is no way to distinguish versions prio v3 from v3 and onwards
Expected behavior
The Firebird 3 behaviour should get explictily available by using its own short name "firebird3" and by creating the necessary test cases.
Additional context
The Firebird3Database class should be extended and all test cases should be implemented for it as well.

Added Risk Medium Tag due the volume of code changes but Firebird v3 changes look pretty well isolated.  Nice use of integration tests!

Closing this as this is not work we plan to implement. For anyone wanting to use Firebird with Liquibase, please read our documentation.


[BonusApp] Fedimint

As from talks with the development team at BTCPrague of https://fedimint.org the fedrated eCash mint is now in a stable state to be deployed to node distribution like RaspiBlitz. As a bonus App caretaker we got recommended https://github.com/Kodylow @Kodylow
The user docs can be found here: https://fedibtc.github.io/fedi-docs/ and there is a good starting tutorial how to setup a federation with setup-leader & setup-followers: https://fedibtc.github.io/fedi-docs/docs/launch-fedimint
Any detailed questions can be asked on the Fedi-Developer-Discord:
https://discord.gg/qwGFpU4Thq

So it makes sense to offer two options side-by-side:

Fedi-Guardian (create/join)
Fedi-Lightning Gateway

In talks with the Fedi team there seems two special problems to solve that RaspiBlitz could take care of in the process of setup:
The guardians (or a lightning gateway) needs to be addressable by fixed domain or IP. Tor can be an option for clients addressing a guardian - but especially for the connection between the guardians a more performant connection is needed. So RaspiBlitz could offer an integrated solution like tailscale (or simlar) to build/join a VPN network between guardians.
Also an alternative to Tor for public client connection with domain/IP:port would be best and is a general topic that makes sense to improve options.

PR to add tailscale to RaspiBlitz: #4712

^ once that lands I can work on the fedimint integrations, thanks for the tag!

Hey @Kodylow - we are doing on the 10th of October a RaspiBlitz Hackday in Berlin (one day before the BTC++ eCash event). It would be great if you or somebody of your team can join and exchange/hack with us on adding Fedimint to the RaspiBlitz: https://www.meetup.com/bitcoin-lab-berlin/events/303230291


grammar and such

just some items i found in readme
good looking project, though :confetti_ball:

Thank you!


Error detection, error first convention?

I would expect error handling to be supported at the module level, otherwise I end up having to wrap the module myself just to add a simple if (data.error) to each callback.
// query a resource/subresource
yf.{resource}.{subresource} (
  {possible argument(s)},
  function cb(err, data) {
    // handle an error
    if (err) {
      console.error(err);
    }
    // do your thing
  }
);

It's been a while, but I've finally started on this! Hopefully will have some updates up soon, been tough since the baby came :D


Close box shows when input is hidden

Describe the bug
When a class is set on the input and that class hides the input, the X close button still shows. I've been unsuccessful at targeting this specifically with CSS to hide it.
To Reproduce

Set input-class-name to a class that is "display: none"

Expected behavior
If the input is hidden, the X button should not show.
Demonstration
https://stackblitz.com/edit/vuepic-vue-datepicker-f3qkwq?file=src/components/Playground.vue

Thank you! :clearable="false" did the trick. Didn't catch that in the documentation, but it's there.


Variant pathogenicity statements should have a classification

Variant pathogenicity is reported using classification terms like "Pathogenic", "Benign", etc. There used to be a property for this, but I can no longer find it.

Do you mean as an enum? I see the classification field here, for example. Though this has been turned into a Coding and some examples where such a code is provided would be helpful. And if a coding doesn't exist, it might be worth recreating the Mappable Entity class that allowed for non-coded entities so they may be representable as strings.

Nevermind, I see it now in core-im-source.yaml. Although I am curious why its on all statements, rather than a special type of ClassificationStatement like before

I think that @mbrush and @larrybabb flattened a lot of the VA Spec abstract classes in their rework leading up to Connect.
I agree that the "Statement" class doesn't seem like a great fit for the classification attribute, and this is a decision I would like some additional clarity / refinement on.


Ignoring changes when calling changeState!

We have a "command" client custom element that takes a couple of different  commands from the server and acts on the client.
One of the command is morph_url that tells the client to call puppet.network.changeState to a provided url.
However, the custom element also resets the command parameter to '' in order to be able to receive the exactly same command again. But that change is never sent to server causing the servers and clients data model to differ!
Pseudo code in custom element, command is bounded to the"Model.Command$
commandChanged: function () {
   parsecommand();
   this.set('command', '');
   window.tu.puppetDirtyCheck(); (Utilityfunction to force the puppet  check that otherwise only reacts on click)
   ....
  ....
   puppet.network.changeState(parameterfromcommand);

}

Do you have notify: true in the command property declaration?

Yes! As I said, the client model is updated. Its just not sent to server.

Am I right, that the change is not being sent to the server even with the next patches from the client side?

Yepp, thats right. The client and server model differs even after a couple of patches.

That's definitely an error then.
@malx122 if you could create a small example which demonstration of the issue, it would be very useful.
@warpech take a look please.

I think the first post describing the problem good enough. If you have trouble to repeat it locally, call me and I will show it.

Does it work when you click on the page background or trigger window mouseup event as described here? https://github.com/PuppetJs/PuppetJs#generating-patches-based-on-local-changes

Aha, I guess this is what window.tu.puppetDirtyCheck(); does.

Can you reproduce it in a small app, like https://github.com/warpech/SimplePuppet or https://github.com/StarcounterSamples/KitchenSink?

Not sure, Ivar found out that it works if he wraps the set command in an async:
(which of course should not be needed)
this.async( function() {
     this.set('command', '');
     window.tu.puppetDirtyCheck();
 }).bind(this);

The whole problem is that you "unobserve" the model while applying data patches from server. That is not ok since applying data patches may trigger javascript methods that makes other model changes themself.
This problem has grown alot since this issue was created. We now have many sections of our code dependent on this working. And the workaround no longer works (not sure it ever worked for more than some lucky timing moments).

@tomalec Question to you as the OT master... Do you think it is possible to apply remote patches without calling unobserve ... observe?

But we do not want those patches to trigger sending them back to server, do we?
So in some meaning we want unobserving it while applying.
Regarding

The whole problem is that you "unobserve" the model while applying data patches from server. That is not ok since applying data patches may trigger javascript methods that makes other model changes themself.

OT could handle changes applied in-between applications of consecutive patches.
So that's more a problem of JSON observer than JSON Patch OT.
As after application of incoming patch (possible synchronous changes), once we unobserve, we would need to compare whether something new (and different from incoming patch) was introduced.

Discussed with @tomalec: Solution seems to be to reintroduce  object a mirror object in puppet-polymer-client that is not known to PuppetJs.

Today I got a new idea :bulb: that might work with https://github.com/PuppetJs/puppet-polymer-client/issues/9
puppet-polymer-client does not have a mirror of puppet.obj. Instead, it is enough if it stops using this.set to apply the patches. Instead, it should:

use regular jsonpatch.apply to apply the patches. This happens between unobserve and observe)
use Polymer's notifyPath to announce the patches to Polymer. This happens in PuppetJs onRemoteChange. I think that this callback (and onLocalChange) should be moved after observe() to avoid side effects like described in the OT

Polymer's notifyPath is already used by us to notify template is="dom-bind"s in nested partials, so this should work OK.

I'd like to remid you that we did so in previous version of puppet-polymer-client ;) and we decided to switch not to fight with Polymer, but use it's public and more stable API.

Not exactly the same. puppet-polymer-client was using its own jsonpatch.apply based on Polymer's set since the second commit in this repo
So right now we are still fighting with Polymer in every template is="dom-bind" other than the root one. With the proposed change, we will use the same principle for the root template that is used for every other partial template.

Sorry, my bad it was dom-bind-notifier

So right now we are still fighting with Polymer in every template is="dom-bind" other than the root one. With the proposed change, we will use the same principle for the root template that is used for every other partial template.

Fair point

@tomalec @warpech Our current approach is to:

modify json-patch to return removed elements on application
and puppet to propagate this info in callbacks
let puppet use json-patch to apply the patches
notify polymer afterwards
polymer requires us to provide it with removed original elements when notifying about changes in arrays. This is reason for p1
should any client update the model in callback, those changes can be safely fed back to puppet

But it turns out that between puppet and json-patch sits a queue which can defer patch application - it's not guaranteed that patches are applied when they are sent to this queue. This makes p. 1a more complicated

I don't think queue will complicate a thing. You will simply trigger the
event once it's eventually applied.
3 cze 2016 21:09 "Marcin Goliński"<EMAIL_ADDRESS>napisał(a):

@tomalec https://github.com/tomalec @warpech
https://github.com/warpech Our current approach is to:

modify json-patch to return removed elements on application

and puppet to propagate this info in callbacks

let puppet use json-patch to apply the patches
notify polymer afterwards

polymer requires us to provide it with removed original elements
when notifying about changes in arrays. This is reason for p1

should any client update the model in callback, those changes can
be safely fed back to puppet

But it turns out that between puppet and json-patch sits a queue which can
defer patch application - it's not guaranteed that patches are applied when
they are sent to this queue. This makes p. 1a more complicated
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PuppetJs/PuppetJs/issues/85#issuecomment-223667166,
or mute the thread
https://github.com/notifications/unsubscribe/AABEGxtdEfR08BJ4jzbs_Uw-BqKXYWx0ks5qIHwCgaJpZM4IPqQh
.

This should be fixed now in new release


Add Research Labs

Considering the recent surge in NLP research, I propose the listing of some of the top labs that are carrying cutting-edge research in NLP (no endorsements, here) like Stanford NLP group, University of Edinburgh NLP group etc.
@NirantK how does this sound?

Sounds good. Raise a pull request? Mention their recent work/ticket to fame
as well?
On Sat 4 May, 2019, 12:50 Dhruv Apte<EMAIL_ADDRESS>wrote:

Considering the recent surge in NLP research, I propose the listing of
some of the top labs that are carrying cutting-edge research in NLP (no
endorsements, here) like Stanford NLP group, University of Edinburgh NLP
group etc.
@NirantK https://github.com/NirantK how does this sound?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/keon/awesome-nlp/issues/174, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAYZUPP6QW4O5QGG77PSSZTPTU2LZANCNFSM4HKYJQMA
.

Merged #176


Add Sanitiser::Strategy::SanitisingError to the list of excluded errors

⚠ Make sure you release a new version of this gem after merging your changes. ⚠
Refer to the existing docs if you are making changes to the supported Ruby versions.
What
Do not send Sanitiser::Strategy::SanitisingError to Sentry. This error should still be raised in all environments.
This error is raised in some apps when an incorrect (wrong UTF-8 encoding) character is encountered in a query string or in a cookie.
Why
Trello card

I'm closing this PR as an alternative approach will be used.


Flutter 1.12.13+hotfix.8 : Problem with R8 shrinker and plugin flutter_inappwebview

Environment
**Flutter version: Flutter 1.12.13+hotfix.8
**Plugin version: Master branch
Description
Expected behavior:
initialOptions should be set when running in release mode
Current behavior:
options.crossplatform.useShouldOverrideUrlLoading = null even if set to true in the initialOptions.
Steps to reproduce

set minifyEnabled true in android/app/build.gradle:

buildTypes {
  release {
    minifyEnabled true
  }
}

in the inappwebview widget add in initialOptions:
useShouldOverrideUrlLoading: true,

and  add :
shouldOverrideUrlLoading: (InAppWebViewController controller,
             ShouldOverrideUrlLoadingRequest request) async {
           print("Override available: ${request.url}");
          return ShouldOverrideUrlLoadingAction.ALLOW;
},

Then run flutter run --release
shouldOverrideUrlLoading will never been called because useShouldOverrideUrlLoading is null

If minifyEnabled is set to false, it works as expected
Same issue when using :

flutter build appbundle
Using:
flutter build appbundle --no-shrink solves the problem, seems like R8 is messing with something.
doc: flutter.dev

[✓] Flutter (Channel stable, v1.12.13+hotfix.9, on Mac OS X 10.14.6 18G3020, locale en-ES)
    • Flutter version 1.12.13+hotfix.9 at /Users/arnaud/dev/flutter
    • Framework revision f139b11009 (4 days ago), 2020-03-30 13:57:30 -0700
    • Engine revision af51afceb8
    • Dart version 2.7.2

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.0)
    • Android SDK at /Users/arnaud/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.0
    • ANDROID_HOME = /Users/arnaud/Library/Android/sdk
    • Java binary at: /Users/arnaud/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/191.6010548/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.3.1, Build version 11C504
    • CocoaPods version 1.8.4

[✓] Android Studio (version 3.5)
    • Android Studio at /Users/arnaud/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/191.6010548/Android Studio.app/Contents
    • Flutter plugin version 41.1.2
    • Dart plugin version 191.8423
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] VS Code (version 1.43.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.8.1

[✓] Connected device (2 available)
    • Redmi Note 8 Pro          • jjkrdyir8l4s8ls4 • android-arm64 • Android 9 (API 28)
    • Android SDK built for x86 • emulator-5554    • android-x86   • Android 8.1.0 (API 27) (emulator)

• No issues found!

I've open an issue also on the github project of the plugin, but i'm not sure if the error is from R8 or the plugin himself: plugin  issue#315

Hi @arnaudelub
This is 3rd party plugin and  you've already raised the issue on theirgithub
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you

well, as the issue occurs only when minification is used, yes, i disagree, the third party plugin is working fine.
There are the same probabilities that the issue is coming from flutter as from the plugin in my opinion.
Thank you for your help

Hi @arnaudelub
R8 is enabled by default and  you don't need to add
minifyEnabled true
if you're experiencing this issue without third party plugin and without addingminifyEnabled true, please comment below and i will reopen it
Thank you

i know it's enabled by default, that's why i said in the issue that flutter build appbundle --no-shrink   can be used as workaround


Proposal: Arch dependent Docker file support for arch specific gccgo builds

Gccgo build for non x86 arch may require changes to Dockerfile that makes docker build difficult from upstream for all archs.
To enable the build, one idea is to have

Default directory for Dockerfile for the archs that do not need any changes (hack/arch/default)
Whichever archs need separate Dockerfile can have hack/arch//Dockerfile
Corresponding changes to Makefile to copy the Dockerfile from appropriate place.

Note: [ my early effort was to not move the current Dockerfile and give
docker build -t "$(DOCKER_IMAGE)"
but above idea failed since built image will fail to have hack/dind wrapper

With the new -f option on "docker build" it might make a solution a bit easier

Thanks Doug, looking into that option

Thanks again. -f option made everything very easy now.
I have the changes now in https://github.com/ktraghavendra/docker/commit/e8107ac885e5fc3be9ba77be004786ee4b7b7ebd .

Does your PR reference https://github.com/ktraghavendra/docker/commit/e8107ac885e5fc3be9ba77be004786ee4b7b7ebd ?  I think it still points to the old commit 7ed30bfadb322be6c0e777db3463e1213e1a1494

Hi Doug, Sorry for confusion, but 7ed30bf is the correct commit with updated changes of -f option
(
https://github.com/ktraghavendra/docker/commit/e8107ac885e5fc3be9ba77be004786ee4b7b7ebd did not have issue number in the log message and it is updated now in https://github.com/ktraghavendra/docker/commit/7ed30bfadb322be6c0e777db3463e1213e1a1494, otherwise patch is same in both the commits )

Some of us were chatting about this the other day and one of the issues with having separate Dockerfiles for each arch is that its going to be a real pain to keep the common stuff in-sync between all of them.
Several options were discussed, including:

some kind of INCLUDE command for Dockerfiles. While this might be a useful feature in general, for this situation it might not be that useful because of the number of arch-specific steps that are intertwined in the Dockerfile would mean a ton of INCLUDE statements all over the place - which felt awkward.
define some sort of conditional logic (like an IF block) so that we can continue to use a single Dockerfile but have arch-specific sections clearly delineated.
@brahmaroutu is working on a proposal and should have something soon to share.

Thanks for the update, I think that makes sense.
Thinking loud further,
where should we have the cgo, gccgo builds ? (i.e., should we have gccgo built conditionally for non -golang archs) in Dockerfile or better to have prebuilt image available with gccgo.

Any update on this?
@duglin If we use some sort of conditional logic, what should it be like for the arch specified base image? Should it be like:
if $ARCH = "ppc64" then
FROM ppc64/ubuntu:14.04
else if ...

Or should we use the same
FROM ubuntu:14.04
and let docker hub detect which arch the request comes from and return the specified image?

@brahmaroutu Is there another proposal I can follow with?

@brahmaroutu can give you more information but I believe the current plans are to have unique Dockerfiles per arch.

@hqhq Please see my latest PR #17459 we are using separate Dockerfile for each ARCH.

Do we need to keep this issue open? I think general consensus at this moment is to use separate Dockerfiles for different architectures

@thaJeztah Agree that we can close this.

Thanks @ktraghavendra !


Multi Channel CC V1 devices are interviewed with a newer Multi Channel CC

which causes them to never respond

It was a parsing issue because of invalid responses.
duplicate of #938


More featureful data type checking

.isReadingFor(0) is a convenient method for checking that a response is for a particular tag.
What if I want to run the same method for multiple tags?
I propose that .isReadingFor take n arguments that allow for as many tags as the developer wants.
Also, if the same code is run no matter the tag .isReadingFor should accept zero arguments allowing for the type to be checked without consideration for the tag.

So you're proposing something like:
def isReadingFor(self, *tags):
    return self.tag in tags

def isStatusFor(self, *tags):
    return self.tag in tags

(and eventually making it snake-case 😄 )

Your example will work for allowing something like .isReadingFor(2,3,4). 🎉

If we want to accept .isReadingFor([1,2,3]) then we need to flatten the input. I believe that could be done with
import itertools
def isReadingFor(self, *tags):
    return self.tag in itertools.chain(tags)

I don't want to pass an array. I like the earlier example. 👍🏻


the first webcat national user

leave this issue open for the internet historical societies and for visibility
23:20 $ webcat maxogden
YO REQSHARK!


cite error

cannot find "model" in "ac" library, and cannot find related Q&A in google or baidu

ohhhh, ac is your pkg hhh

Here ~


More endpoints and stability changes.

A previous pull request also implements the dungeon and raid encounters, either my implementation has to be changed or we can disregard that pull request and simply merge this (the implementation is slightly different):
https://github.com/wingyu/rbattlenet/pull/16

Thanks for the PR! I really appreciate it


Access denied exception with RazorEngine 3.5

Hi,
We're using RazorEngine to generate emails in an ASP.Net Web Application. Since updating our code to version 3.5, the template compiling fails with the following stack trace:

Access is denied. (Exception from HRESULT: 0x80070005 (E_ACCESSDENIED))

The easiest way to test this is simply to create a blank website in Visual Studio, add the RazorEngine package, and plug the basic "Hello World" sample template. With 3.4.x, the code will run just fine. I've tracked the error to the CompileType method, when the process tries to load the generated assembly in the Windows Temp folder.

C:\Windows\Temp\RazorEngine_grceyklj.1f1\CompiledRazorTemplates.Dynamic.RazorEngine_6aa243ea4a104b1f8d7036145a4c87a0.dll
Event Class:	File System
Result:	ACCESS DENIED
Impersonating:	DMZ\iusr_web

Note that our websites are using ASP.NET Impersonation, and the impersonated user is "iuser_web" ,from our DMZ domain. Giving this user read access to the C:\Windows\Temp\ folder on the local machines fix the problem, and the templates are generated correctly. However, I'm not sure if this is the right course of action, since this used to work perfectly in 3.4.x.
Is this a new requirement for 3.5 ?
Thanks,
Shaun

First of all thanks for  this detailed error report!
Yes, there were changes in that part of RazorEngine:
Instead of relying on the build in temporary compilation we now specify the path manually.

Helps with debugging templates (and to provide nice stack traces).
The default loading mechanism of CodeDom (Load(byte[])) defeats the sandbox.
Allows custom caching provider to pick up the assembly and cache it in the filesystem.
Unfortunately it seems to fail in your scenario, but I'm not sure why that is the case. Especially I cannot see why you can write in that directory but not read?

The first things that come into my mind:

Can you try GACing the RazorEngine assembly? (If you do not to run templates with full trust use the isolation API)
Would it help you if there was an API to specify the path manually?

The reason specifying the path manually is problematic: If you do not use the Isolation API you cannot clean up that path! There is no way to unload the Assembly and therefore to release the file lock.
I also decided agains a single "RazorEngine" directory in Temp as that would potentially lead to permission problems when used in different user accounts. That's why RazorEngine creates the folders starting with "RazorEngine_". It would be enough to allow RazorEngine to read those. In fact it should be enough if RazorEngine could read the folders it created itself... Any idea why that is not the case?

Thanks for the reply.

Unfortunately it seems to fail in your scenario, but I'm not sure why that is the case. Especially I cannot see why you can write in that directory but not read?

I dug a little deeper and found something interesting : the call to CompileAssemblyFromSource doesn't carry forward the impersonated user, CSC.exe is thus executed under the AppPool identity. The DLL is written using that identity, but the impersonated user doesn't have read access to the resulting DLL.
I have tried running the website with Full trust, and also tried adding RazorEngine to the GAC. The results are the same, the impersonated user gets ignored when CompileAssemblyFromSource is called. I'm not sure about the implications of using the Isolation API, do you think this could possibly fix my problem?
I understand the design decision behind these changes, and in the meantime I'm running tests with read rights explicitly given to the Temp folder. Let me know how I can be of further help.

Can you try with the https://github.com/Antaris/RazorEngine/tree/feature/workaround_impersonation branch?
git clone -b feature/workaround_impersonation git@github.com:Antaris/RazorEngine.git
cd RazorEngine
build.cmd

Should build you the assembly in the build\net45 folder.

The branch fixed the issue!
The impersonated user is no longer being used, and the templates are compiled correctly.
Thanks a lot.

Good, thanks for testing this. I will make a 3.5.1 release when I have fixed the mono build.
Until then, if you want to use the official version you can work around the bug yourself:
(new PermissionSet(PermissionState.Unrestricted)).Assert();

WindowsImpersonationContext wic = WindowsIdentity.Impersonate(IntPtr.Zero);
try
{
    // Failing razorengine code.
}
finally
{
    wic.Undo();
}

Sadly I found no way to unit test this to prevent future regressions :(. Do you have an idea?

For my use case, we'd need to properly mock IIS behavior when using ASP.Net Impersonation. Unit testing and mocking HttpContext over Impersonation can be a total PITA. All my suggestions would amount to integrated testing, sorry.

I think its "ok" without unit test as there is a workaround, so even if we introduce this again in the future: one can use the code from above to work around this.


MyWorkDrive Gallery App Configuration shows wrong sample reply URL

[Enter feedback here]
Hello, please note in the gallery app for MyWorkDrive, on the page where customer would enter their reply URL it shows in invalid pattern: it shows:
Patterns: https://..com/account/login.saml underneath Reply URL in the sample app.
This pattern belongs underneath in the sign-on URL. The correct pattern is /SAML/AssertionConsumerService.aspx for reply URL.

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

ID: 5becc0e1-f452-92ab-c607-5e0af7f1848b
Version Independent ID: 4e4dcf30-76a4-7c8f-d98d-1e974dbacce8
Content: Tutorial: Azure Active Directory integration with MyWorkDrive
Content Source: articles/active-directory/saas-apps/myworkdrive-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @jeevansd
Microsoft Alias: jeedes

@dgordoncal
Thanks for your feedback! We will investigate and update as appropriate.

@dgordoncal
Thanks for your feedback! We will investigate and update as appropriate.

Great Thanks!  Let me know if you have any questions.  Your article is fine, just the Gallery App got updated somehow..

Great Thanks!  Let me know if you have any questions.  Your article is fine, just the Gallery App got updated somehow..

@v-hagamp and @chetansriv Are we fixing this as part of the bigger effort?

@v-hagamp and @chetansriv Are we fixing this as part of the bigger effort?

@dgordoncal
As you said, there is nothing to change in the article and we updated the Reply URL pattern in the Gallery, it may reflects soon.
so, am closing this.
#please-close

@dgordoncal
As you said, there is nothing to change in the article and we updated the Reply URL pattern in the Gallery, it may reflects soon.
so, am closing this.
#please-close


Update direct dependencies, most important metal-stack/v which fixes …

…a urgent --version bug, and google api dependencies, still ldflags are not set as required.
next step must be to fix ldflags

Thanks for that.


No proper pagination support with query builder

I'm currently migrating from TypeORM and I've found that MikroORM lacks pagination support for QueryBuilder. Well, it works, until I need to add any leftJoinAndSelect().
There are a few issues actually (when we add leftJoinAndSelect()):

getCount() returns wrong count
limit() also limits joined relations
orderBy() causes SQL error (when used with or without groupBy())

Using findAndCount() or QueryFlag.PAGINATE is not an option for me.
Let's imagine such a simple example:
@Entity()
export class User {
  @PrimaryKey()
  id: number;

  @OneToMany(() => Comment, (comment) => comment.user)
  comments = new Collection<Comment>(this);
}

And imagine that we have 2 users, each of them has 10 comments.
Query 1:
const query = getEntityManager()
  .getRepository(User)
  .qb('user')
  .select('*')
  .leftJoinAndSelect('user.comments', 'comments')
  .where({ id: 1 }); // let's select just one user here

const count = await query.getCount(); // returns 10, should be 1

Query 2:
const query = getEntityManager()
  .getRepository(User)
  .qb('user')
  .select('*')
  .leftJoinAndSelect('user.comments', 'comments')
  .limit(2);

const data = await query.getResultList();
// 1) data contains only 1 user (should be all 2)
// 2) data[0].comments contains only 2 comments

Query 3. Let's try adding orderBy():
const query = getEntityManager()
  .getRepository(User)
  .qb('user')
  .select('*')
  .leftJoinAndSelect('user.comments', 'comments')
  .limit(2)
  .orderBy({ 'user.id': 'DESC' });

// SQL Error:
// select count("user"."id") as "count" from "users" as "user" left join "comments" as "comments" on "user"."id" = "comments"."user_id" where "user"."id" = 1 order by "user"."id" desc -
// column "user.id" must appear in the GROUP BY clause or be used in an aggregate function

Query 4. Let's add groupBy():
const query = getEntityManager()
  .getRepository(User)
  .qb('user')
  .select('*')
  .leftJoinAndSelect('user.comments', 'comments')
  .limit(2)
  .groupBy('user.id')
  .orderBy({ 'user.id': 'DESC' });

// SQL Error:
// select "user".*, "comments"."id" as "comments__id", "comments"."created_at" as "comments__created_at", "comments"."updated_at" as "comments__updated_at", "comments"."user_id" as "comments__user_id", "comments"."text" as "comments__text", "comments"."nft_id" as "comments__nft_id", "comments"."parent_id" as "comments__parent_id" from "users" as "user" left join "comments" as "comments" on "user"."id" = "comments"."user_id" where "user"."id" = 1 group by "user"."id" order by "user"."id" desc limit 2 -
//  column "comments.id" must appear in the GROUP BY clause or be used in an aggregate function

That's it. TypeORM handles pagination nicely with their skip() and take() that we should use instead of limit() and offset() if we have at least one leftJoinAndSelect() on the query builder. They wrap the count query into SELECT COUNT(DISTINCT("user"."id")) as "cnt" FROM (...main query here...) and also do an additional query without limits to avoid limiting joined relations (though I don't know all the details).
Is there some way around these issues right now?
Versions

Dependency
Version

node
16.13.2

typescript
4.5.5

mikro-orm
5.0.4

your-driver
PostgreSQL

They wrap the count query into SELECT COUNT(DISTINCT("user"."id")) as "cnt" FROM (...main query here...) and also do an additional query without limits to avoid limiting joined relations (though I don't know all the details).

Sounds exactly like what QueryFlag.PAGINATE does, why do you say its not an option for you?

QueryFlag.PAGINATE is not an option because it's usable only inside find query.  And query builder gives more control over the select query. Sometimes I need to add subselects for "virtual" columns and sometimes I have some complex filters with subqueries. Also, there is just one paragraph about pagination in the docs right now so I might not know something.

QueryFlag.PAGINATE is not an option because it's usable only inside find query.

You can use it with QB too via qb.setFlag(). The em.find() method basically just calls QB, most of the features are implemented in the driver/QB (e.g. autojoining).

Seems like adding setFlag(QueryFlag.PAGINATE) prevents limiting joined relations in getResultList() but getCount() still counts everything like before. And it still not possible to do orderBy().

Then please provide details for that. I am not going to add skip/take, I don't see a reason for different API doing the same thing, instead I do want to enable this paginate flag based on what is joined.
Looking at qb.getCount(), that indeed resets limit/offset (and it is not connected to the paginate flag anyhow). Why would you want to use count query together with pagination? You can just call qb.execute() if you don't like that. The reason for this is that you can clone a QB (that already has a limit/offset) and call getCount() on it, which should ignore the limit/offset.

Minimal pagination interface looks like this:
{
  limit: number,
  offset: number,
  total: number,
  data: any[],
}

We need total here to be able to count the total pages available based on limit.
The problem with getCount() is that it counts joined relations when it should not.
I just discovered that if I call it like this:
query.getCount('DISTINCT("user"."id")')

then it works the way I need.
I suppose it could do this distinct wrapping automatically.

We need total here to be able to count the total pages available based on limit.

You will need a separate call to get the total number of results, I hope we don't need to argue about that. I know there are some driver specific ways to do it in a single query (in mysql at least), tried it some time ago and it is much worse performance wise than a separate query.

I just discovered that if I call it like this:

There is a second boolean argument to enable disctinct query, in both qb.count() and qb.getCount(). Enabling it by default is technically a breaking change, so no plans to do it any time soon.
I am still waiting for details regarding what is not working with paginate + order by.

Right, I didn't notice that second arg at first. So query.getCount('user.id', true) is much better.
About orderBy(). The query:
const query = getEntityManager()
  .getRepository(User)
  .qb('user')
  .select('*')
  .leftJoinAndSelect('user.comments', 'comments')
  .orderBy({ 'user.id': 'DESC' })
  .setFlag(QueryFlag.PAGINATE);

const count = await query.getCount('user.id', true);

This query fails if I call getCount() on it:
select count(distinct("user"."id")) as "count" from "users" as "user" left join "comments" as "comments" on "user"."id" = "comments"."user_id" order by "user"."id" desc -
column "user.id" must appear in the GROUP BY clause or be used in an aggregate function

It's not critical because I can move orderBy() into the .orderBy(...).getResultList() call.
Though getCount() could also strip "ORDER BY" as well as it does it with limit/offset, I suppose.

Right. Thanks for your help.


Import BEL named complexes into BE namespace and relations

Here is a list of BEL named complexes and their members:
complex_table.txt
based on
http://resource.belframework.org/belframework/1.0/resource/named-complexes.xbel

Added in f8d49d6e89d.

Whoops--thought this was about families, not complexes. Deleting previous commit ref.

Done in #12


App-rdm: use invenio-pages service [REQUIRES: inveniosoftware/invenio-pages#72]

closes: #2023
requires: https://github.com/inveniosoftware/invenio-pages/pull/72

Changes from last review: change system_identity for superuser_access.

Reverted to using system_identity, as using superuser-access required to use a problematic hack.


Bandaged BD stopped loading

I started up discord today, and BD was not integrated/injected (or whatever you want to call it.) I tried a reload and still didn't inject. i checked and all of the files are still there. Is it abandoned?

Using the repair tool included with the installer fixed it for me.

Using the repair tool included with the installer fixed it for me.

fixed for me too, thanks!

Yep just need to reinject. Not sure how you would surmise that this project has been abandoned...


REACT18 TS - add support for children props

REACT18 TS - Children are no longer  implicit property of an Component.
Children has to now be specify explicitly on the props declaration.

@ederelk Thanks for making the PR. Hope they merge soon!

Unfortunately, it seems that I have lost the ability to publish this package somehow. You might want to consider either using patch-package (https://www.npmjs.com/package/patch-package) or switching to a different thing, like IntersectionObserver (maybe via react-intersection-observer https://www.npmjs.com/package/react-intersection-observer).

I managed to push v10.3.0 🎉


Add light dropdown-menu for Navigation

I could not find a way to add a light dropdown in the navigation like this:
http://esri.github.io/calcite-web/documentation/components/#dropdowns
or like in AGOL:
https://zrhdata.mapsqa.arcgis.com/home/webmap/viewer.html?useExisting=1

Is there a way to do it with this version?

Hey @SaschaBrunnerCH, currently dropdowns are "roll your own". We've talked about making an uncontrolled dropdown component with a simplified implementation, but right now the closest thing would be the Popover. The targetEl property can be modified to display a button similar to what your screenshot shows.
Here's an example of what that might look like:
targetEl={
  <Button
    transparent
    icon={<ChevronDownIcon />}
    onClick={this.togglePopover}
  />
}

Let me know if this doesn't meet your requirements or you need more details.

@codylawson Thanks for pointing me to the right direction. This works for me:
https://gist.github.com/SaschaBrunnerCH/b25185b6a53224e6b584a1ddbe36a148


865 update logs and metrics

This PR modifies the koaJsonRpc in order to log errors with response codes other than 200s.

Codecov Report
Base: 74.34% // Head: 74.15% // Decreases project coverage by -0.20% :warning:

Coverage data is based on head (3ec88f5) compared to base (f946030).
Patch coverage: 65.21% of modified lines in pull request are covered.

:exclamation: Current head 3ec88f5 differs from pull request most recent head 7cc27e6. Consider uploading reports for the commit 7cc27e6 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #938      +/-   ##
==========================================
- Coverage   74.34%   74.15%   -0.20%
==========================================
  Files          28       28
  Lines        1844     1861      +17
  Branches      349      349
==========================================
+ Hits         1371     1380       +9
- Misses        372      380       +8
  Partials      101      101

Impacted Files
Coverage Δ

packages/server/src/koaJsonRpc/index.ts
61.62% <60.00%> (-1.61%)
:arrow_down:

packages/server/src/server.ts
74.17% <100.00%> (-0.13%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Question: Why is mutation document needed when you have all the details in schema.json?

Question in the title :)
Why is mutation document needed when you have all the details in schema.json?

Hey @Richard87,
of course, react-apollo-form could generate the query for you, it's a good point!
I keep this ticket as a "feature request" and will implement it ASAP.
Thanks!


[TAB REQUEST] Microsoft Defender Update Check

Describe the solution you'd like
A toolset that integrates with Microsoft Defender in two key ways

Check for Defender Definition Update availability or compliance
Run Windows Defender definition updates (using Microsoft.Update.Session APIs)

Describe the command flow intended

Under functions, user chooses “Defender Definition Update”
System runs a push for Defender Definition updates using Windows Update service.

Additional context
https://learn.microsoft.com/en-us/windows/win32/wua_sdk/searching--downloading--and-installing-updates

This might actually be quick

This is good to go


slskd: 0.18.2 -> 0.19.5

Description of changes
Major: https://github.com/slskd/slskd/releases/tag/0.19.0
Patch: https://github.com/slskd/slskd/releases/tag/0.19.5
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Hi @melvyn2, Do you want to add yourself as a maintainer too?

Thank you for the invitation! I'll add myself in the refactor PR instead to keep this one focused.


🐛🐛  Bug Report: AssertionError  assert response is not None

Describe your problem

群发消息错误

Reproduce your problem
k = 0
for room in rooms:
   if room is not None:
     k = k + 1
     if k >= 7:
        await asyncio.sleep(random.randint(60, 120))
        k = 0
      await asyncio.sleep(random.randint(5, 20))
      await room.say(msg)

Error info
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/apscheduler/executors/base_py3.py", line 30, in run_coroutine_job
    retval = await job.func(*job.args, **job.kwargs)
  File "wechaty_client_pad.py", line 305, in execute
    await room.say(msg)
  File "/usr/local/lib/python3.7/site-packages/wechaty/user/room.py", line 289, in say
    mention_ids=mention_ids
  File "/usr/local/lib/python3.7/site-packages/wechaty_puppet_service/puppet.py", line 337, in message_send_text
    text=message, mentonal_ids=mention_ids)
  File "/usr/local/lib/python3.7/site-packages/wechaty_grpc/wechaty/__init__.py", line 438, in message_send_text
    "/wechaty.Puppet/MessageSendText", request, puppet.MessageSendTextResponse
  File "/usr/local/lib/python3.7/site-packages/betterproto/__init__.py", line 1125, in _unary_unary
    assert response is not None
AssertionError

msg是文本么？ say只支持文本和filebox。。。

from the above error log info, we can't find out the key error to help you fix this issue. Please post more detail info:

the version of wechaty related packages, you can use command: pip list | grep wechaty
the type of msg which can be: str, UrlLink, FileBox, MiniProgram

version of wechaty：
wechaty                0.8.25
wechaty-grpc           0.20.19
wechaty-plugin-contrib  0.0.12
wechaty-puppet         0.3.dev10
wechaty-puppet-service  0.8.5
the type of msg is str

str


Hacktoberfest: Swipe: A Peer to Peer Chat Application

A P2P Chat application in Node JS using Budo/Watchify module to bundle for browser. Tested on Mozilla Firefox 53.x Can be extended to Live streaming over network as well as quick file sharing.
Ref: #1
Signed-off-by: Progyan Bhattacharya<EMAIL_ADDRESS>

Awesome 👍 💯